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

orix 0.6.0 #145

Closed
7 of 9 tasks
pc494 opened this issue Dec 8, 2020 · 15 comments
Closed
7 of 9 tasks

orix 0.6.0 #145

pc494 opened this issue Dec 8, 2020 · 15 comments

Comments

@pc494
Copy link
Member

pc494 commented Dec 8, 2020

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?

@hakonanes
Copy link
Member

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 plot module, since the projections won't only be used for plotting!

@hakonanes
Copy link
Member

I can add a orix.data module with a few small example datasets, if it's desirable and if you can give me some suggestions?

For kikuchipy, I find it simpler to write documentation and test new functionality when having some example datasets available in our data module.

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

@pc494
Copy link
Member Author

pc494 commented Apr 21, 2021

I think the current consensus is to leave the orientation colouring to a 0.7.0 release & instead push on and try and release a 0.6.0 as we now have a good batch of features that have not yet shipped. I think the main things that are left are #139 and #140 (again) plus the newly noticed #175.

@hakonanes
Copy link
Member

I think #167 should be added as well, I will have a look at this in the next couple of days.

@hakonanes
Copy link
Member

I'm also nearly done with fixing #159 and #160.

@pc494
Copy link
Member Author

pc494 commented Apr 21, 2021

Ah cool, I think mentally I had marked them as "safe to do as patch" but it would be great to have them for 0.6.0. Shall we target mid-May for a release?

@hakonanes
Copy link
Member

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!

@pc494
Copy link
Member Author

pc494 commented Apr 29, 2021

Also, @hakonanes I think we should put .ipynb back in the .gitignore, if one changes the docs it's not that hard to find the -f command and it clogs up the commit list as it is because the unique IDs sometimes get changed.

@hakonanes
Copy link
Member

Yeah, definitely, I will have a look around the internet and see if there is a simple solution.

@hakonanes
Copy link
Member

@pc494, I can set up this release if you don't have time.

@pc494
Copy link
Member Author

pc494 commented May 16, 2021

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.

@hakonanes
Copy link
Member

The work you're doing in #193 is important, so I'm totally OK with a delay.

@pc494
Copy link
Member Author

pc494 commented May 21, 2021

When #193 goes in I'm going to tidy up and get a release out.

@hakonanes
Copy link
Member

Sounds good!

@hakonanes
Copy link
Member

Released!

@hakonanes hakonanes unpinned this issue May 26, 2021
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