Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev environmental exposure #1455

Open
wants to merge 103 commits into
base: main
Choose a base branch
from
Open

Dev environmental exposure #1455

wants to merge 103 commits into from

Conversation

jc2341
Copy link
Contributor

@jc2341 jc2341 commented Jan 24, 2025

No description provided.

jc2341 added 30 commits December 8, 2024 22:14
…-feeature endpoint and trajectory endpoint, in case data stored in multiple stacks
@jc2341 jc2341 force-pushed the dev-Environmental-Exposure branch from ceec71c to 55b2c82 Compare January 29, 2025 16:08
jc2341 added 16 commits January 31, 2025 12:02
… used in exposure_utils.py to avoid hardcode of query command
…anage functions used in exposure calculation
…l, utilities function and registerted exposure blueprint
@jc2341 jc2341 requested a review from sm453 February 4, 2025 12:01
@sm453
Copy link
Member

sm453 commented Feb 6, 2025

Please also comment on the apparent redundancy of the files resources/ontop.obda and resources/obda/features.obda. They seem identical, or have I missed something? Are they both needed?

@jc2341
Copy link
Contributor Author

jc2341 commented Feb 6, 2025

Please also comment on the apparent redundancy of the files resources/ontop.obda and resources/obda/features.obda. They seem identical, or have I missed something? Are they both needed?

Only resources/ontop.obda is necessary as it is requested in the Dockerfile. Redundant mappings and query templates have been removed. Please check commit [729e4ec] for details if needed

@jc2341 jc2341 requested a review from sm453 February 7, 2025 00:03
Copy link
Member

@sm453 sm453 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The docker image does not build. It throws the following error:

6.394 INFO: pip is looking at multiple versions of geopandas to determine which version is compatible with other requirements. This could take a while.
6.396 ERROR: Cannot install Fenland-Trajectory-Agent and fenland-trajectory-agent==0.1.1 because these package versions have conflicting dependencies.
6.397
6.397 The conflict is caused by:
6.397     fenland-trajectory-agent 0.1.1 depends on numpy==1.21.6
6.397     geopandas 1.0.1 depends on numpy>=1.22
6.397
6.397 To fix this you could try to:
6.397 1. loosen the range of package versions you've specified
6.397 2. remove package versions to allow pip to attempt to solve the dependency conflict
6.397
6.398 ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

Please resolve this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants