-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add lenunit
option to GeoIO.load
#100
Comments
Hi @kramsretlow, it is good to see you. We should probably add a Would you like to try adding the option yourself? You can forward it to any affected format, and we can review it. |
Hi @juliohm , thanks for remembering me 😄 I will try to make some time in the next while to work on it, though I can't promise it'll get done. So far I was still unable to find the point in the source where the meters unit magically joined my mesh coordinates--will need to check it more deeply. I'm really enjoying using Meshes.jl for some computation job I'm doing on 3D parts. If one comes to Meshes.jl from a direction not related to geographical data, it feels slightly odd to need to go to GeoIO.jl (which is clearly geostats related) to load a mesh. Not a problem, but just a comment from user's perspective. In any case, thanks for your work on these packages! |
The units are added in the Point(1mm, 2mm)
You can add a unit to a pre-existing tuple with: u = mm
Point(x * u, y * u)
Glad you are enjoying it :)
GeoIO.jl is not tied to GeoStats.jl (the package), but is certainly related to GeoStats.jl (the framework). It doesn't depend on the "stats" part of "geostats". Think of it as a simple tool to load geospatial data as geotables, which can be used for any purpose beyond geospatial data science and geostatistical modeling. |
lenunit
option to GeoIO.load
@kramsretlow the option should be available now. Please update the environment. I will leave the issue open because there some other formats also support the option, but we didn't have time to implement yet:
|
/bounty $20 |
💎 $20 bounty • JuliaEarthSteps to solve:
Thank you for contributing to JuliaEarth/GeoIO.jl! Add a bounty • Share on socials
|
/attempt #100
|
@mobley-trent it is partially fixed. The option is not available in the formats listed above in a check list. |
I see that |
The option is passed to the inner functions, but it is being currently ignored. See the csvread for example: Line 5 in 8201229
The option is not used inside the function. |
Fixed for all backends. |
I have an .obj file that was exported from a CAD program. The CAD software was using mm as the unit. If I import it as a
SimpleMesh
usingGeoIO.load(filename).geometry
, it comes out with units of meters. Is it (or could it be) possible to specify the units in the call toload
? And/or, is there an easy way to change the units of a mesh after creation? Right now I am using the inelegant way of loading the mesh, then creating a new mesh with afix_part_units
function defined like this:I've tried searching the docs and source, but I can't find the functionalities I'm after. Sorry if I've missed something.
I'm not familiar with all of the file formats, but I guess this point could apply to some other "unitless" formats as well.
The text was updated successfully, but these errors were encountered: