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

Make Adaguc call async. #345

Merged
merged 10 commits into from
Mar 7, 2024
Merged

Make Adaguc call async. #345

merged 10 commits into from
Mar 7, 2024

Conversation

lukas-phaf
Copy link
Collaborator

Also fix the postgres volume location in the docker compose file.

Also fix the postgres volume location in the docker compose file.
@maartenplieger maartenplieger marked this pull request as ready for review March 7, 2024 09:45
mgrunbauer
mgrunbauer previously approved these changes Mar 7, 2024
python/lib/adaguc/CGIRunner.py Outdated Show resolved Hide resolved
@@ -5,11 +5,12 @@ ADAGUC_PORT=443
ADAGUC_DATA_DIR=${HOME}/adaguc-docker/adaguc-data
ADAGUC_AUTOWMS_DIR=${HOME}/adaguc-docker/adaguc-autowms
ADAGUC_DATASET_DIR=${HOME}/adaguc-docker/adaguc-datasets
ADAGUC_NUMPARALLELPROCESSES=8
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please set this to 4

@@ -259,8 +259,8 @@ async def get_collection_position(
}


@cached(cache=TTLCache(maxsize=1024, ttl=60))
def get_collectioninfo_for_id(
# @cached(cache=TTLCache(maxsize=1024, ttl=60))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove this comment?

return collection


@cached(cache=TTLCache(maxsize=1024, ttl=60))
def get_capabilities(collname):
# @cached(cache=TTLCache(maxsize=1024, ttl=60))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove comments?

@@ -171,12 +171,12 @@ def get_dimensions(layer, skip_dims=None):
return dims


@cached(cache=cache)
def get_parameters(collname):
# @cached(cache=cache)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove comments?

@mgrunbauer mgrunbauer dismissed their stale review March 7, 2024 11:07

Infinite loop does not work properly, and EDR caching is much slower.

Copy link
Collaborator

@mgrunbauer mgrunbauer left a comment

Choose a reason for hiding this comment

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

🎉

@mgrunbauer mgrunbauer merged commit 017d70c into master Mar 7, 2024
2 checks passed
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.

3 participants