-
-
Notifications
You must be signed in to change notification settings - Fork 41
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
enum missing in javax.measure.MetricPrefix #255
Comments
This was discussed and deciced upon in #250, @desruisseaux also said in that ticket, it should not be a big issue. Seshat does not yet implement it, but I trust @desruisseaux will find a way to work with the new API, please clearly say, if that would suddenly be a problem. I trust there can either be patches to Snapshot 31 or a Snapshot 32, changing the API dependency temporarily to 2.1.3 (as long as you don't already embrace the 4 new enums) or fixing whatever reference (I assume it's a resource bundle like Indriya's messages.properties you might have. We don't forsee a 3.x API anytime soon (unless maybe the JDK some day offered a true value type or another breaking change in its type system) so 2.2 as opposed to 2.1.x seemed a viable step for that, after all there are also 4 new prefixes that did not exixt before. In theory a backport (2.1.4) of the API with the 4 new prefixes added, but the old GeoTools is also no stranger to its own breaking changes, see GeoTools 30.x in upgrade. |
Thanks everyone for commenting so fast. For my purposes, I am OK, since I copied the sources and just added an enum. It is puzzling how GeoTools (snapshot 31) was even able to compile. I'm not sure what the correct action should be, but I'm leaning towards GeoTools making an adjustment. |
In version of unit-api.jar 2.0, the sources have an enum for DEKA. In the latest source it was removed. I'm not sure why as the related static method is retained. I needed to add it back in order to get the latest version of GeoTools (Snapshot 31) to run (compiles fine). I'm not sure where the issue actually is, but I thought I would start here as DEKA appears half implemented.
The text was updated successfully, but these errors were encountered: