-
Notifications
You must be signed in to change notification settings - Fork 52
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
orix 0.6.0 #145
Comments
This sounds good! I'm ready to review any coming PR. Regarding the projections module, I plan for us to put spherical, Gnomonic, and the modified Lambert projections from kikuchipy into that module. The setup is based on MTEX' spherical projection, with a projection and possibly an inverse projection. It takes vector3d or numpy arrays of arbitrary shapes with the last axis of length 3 (x, y, z). I hope we can adopt this setup (or perhaps something even better) for the projections module in orix. And reserve all plotting stuff for the |
I can add a For kikuchipy, I find it simpler to write documentation and test new functionality when having some example datasets available in our This would then be importable like (perhaps with a shorter name) >>> from orix import data as orix_data
>>> xmap = orix_data.small_super_duplex_stainless_steel()
>>> xmap
Phase Orientations Name Space group Point group Proper point group Color
1 5657 (48.4%) austenite/austenite None 432 432 tab:blue
2 6043 (51.6%) ferrite/ferrite None 432 432 tab:orange
Properties: iq, dp
Scan unit: um |
I think #167 should be added as well, I will have a look at this in the next couple of days. |
Ah cool, I think mentally I had marked them as "safe to do as patch" but it would be great to have them for |
Agree, they are safe to do as patches. We could add them to 0.6.1, will see how easy they are to fix. The issues you mentioned are more important! |
Also, @hakonanes I think we should put |
Yeah, definitely, I will have a look around the internet and see if there is a simple solution. |
@pc494, I can set up this release if you don't have time. |
Update on this: the work in #193 is proving a bit more substantial than previously thought. Given that #175 isn't really fixed until that's merged I'm holding off releasing for now. I'm optimistic about the chance I can finish it in the next couple of days. But if not our release may be delayed. Anyone with strong feelings about timings, now would be a good time to shout. |
The work you're doing in #193 is important, so I'm totally OK with a delay. |
When #193 goes in I'm going to tidy up and get a release out. |
Sounds good! |
Released! |
UPDATED: New list, with a planned Mid-May release:
0.6.0
Pushed to 0.7.0
Previous Text in this Issue
After conversations this week, a plan for orix 0.6.0 has solidified.
We're expecting solutions and implementation for:
I think the main discussion here is about the final bullet point. Correct me if I am mistaken, but I think @din14970 was expecting to put a streographic projection (in a projections module?) and a (rough) IPZ plotter somewhere. Meanwhile
@hakonanes was planning RGB colouring (and a general framework for such colouring schemes).
and then this means that:
@din14970 can work on stereographic projects without any new code going in. However we probably want to drop the IPZ plotting into the framework @hakonanes is planning. Am I missing something here?
The text was updated successfully, but these errors were encountered: