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

2023 and before 1 changes #16

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

Bonnarel
Copy link
Collaborator

Add transform parameters, pixel ranges, Dataset format changes, product type reduction

Bonnarel and others added 4 commits February 24, 2021 17:50
RElated to issue ivoa-std#4 this Commit introduces a couple of subsections for new rebinning / reprojecting parameters in SODA
Copy link
Collaborator

@pdowler pdowler left a comment

Choose a reason for hiding this comment

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

Some organisational suggestions:

  • change DOCTYPE back to WD in the Makefile
  • re-order the filtering param sections so that MOC comes after POLYGON and PIXELS comes at the end
  • isn't DPTYPE a kind of transformation (in the general sense)? Move it to the "Transform" section?

I will comment on individual items separately.

adding sub-sampling by picking up "1 upon n" pixels in the area of interest
moving DPTYPE subsection to Transform and reorder the filtering section according to Pat's comments
@Bonnarel
Copy link
Collaborator Author

Some organisational suggestions:

* change DOCTYPE back to WD in the Makefile

* re-order the filtering param sections so that MOC comes after POLYGON and PIXELS comes at the end

* isn't DPTYPE a kind of transformation (in the general sense)? Move it to the "Transform" section?

I will comment on individual items separately.

Changes made

Working draft 1.1 first version
clarifying behavior for DPTYPE parameter
- PIXELS and parameter multiplicity
- RESPONSEFORMAT = application/hips specificities
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