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

Integration of ncWMS2 correction on native grids #499

Open
PBrockmann opened this issue May 6, 2024 · 5 comments
Open

Integration of ncWMS2 correction on native grids #499

PBrockmann opened this issue May 6, 2024 · 5 comments

Comments

@PBrockmann
Copy link

It would be very useful to get the correction proposed from
Reading-eScience-Centre/ncwms#99
integrated into tds.

@tdrwenski
Copy link
Contributor

I see mentioned here that the new edal-java snapshot fixed your issue. So just to be sure I understood correctly (since that issue you linked to had a long discussion), are you asking that we update our version of edal-java to incorporate those recent changes?

@PBrockmann
Copy link
Author

PBrockmann commented May 7, 2024

Yes, it would be very nice from you.
Perhaps after having waited for the correction of an existing bug that I have detailled in the ncwms issue (there is a gap at wrapping line).

@tdrwenski
Copy link
Contributor

tdrwenski commented May 7, 2024

Okay no problem, just ping us again when the fix is there and we can ensure it gets into the edal-java version used in the TDS.

@tdrwenski
Copy link
Contributor

any update on this issue?

@PBrockmann
Copy link
Author

Nope... Too bad.

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

No branches or pull requests

2 participants