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

is_valid attribute is not needed anymore #308

Open
capooti opened this issue Jun 30, 2017 · 2 comments
Open

is_valid attribute is not needed anymore #308

capooti opened this issue Jun 30, 2017 · 2 comments
Assignees
Labels
Milestone

Comments

@capooti
Copy link
Contributor

capooti commented Jun 30, 2017

We can get rid of this attribute.

We can now associate an issue to any Layer/Service which is problematic. We should have a script which create issues for service/layers which are now marked as not valid.

When harvesting the service/layer, if there are conditions that previously marked the service/layer as invalid, they should now associate an issue to the same service/layer.

@capooti capooti added this to the 0.4 milestone Jun 30, 2017
@capooti
Copy link
Contributor Author

capooti commented Jul 3, 2017

The script which will create issues should become a celery task

@capooti
Copy link
Contributor Author

capooti commented Jul 10, 2017

@thaorell can you work on this as well?

Basically, the update_validity methods for Service and Layer in aggregator/models.py are obsolete, as now we use the Issue model to set a Service or Layer which is problematic.

The update_validity code should be now be part of a new Celery task, let's name it set_invalid_layers, which, running on a periodic basis, will set an issue for layers when:

  • layer is not in a supported SRS
  • layer is part of WMTS
  • layer is part of a service which contains 'noaa' in url
  • layer falls within an extent which is close to 0, 0

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

No branches or pull requests

3 participants