-
Notifications
You must be signed in to change notification settings - Fork 84
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
0.19.0rc #1077
Comments
When should the I will not have time to work on this before next week. So feel free to make an early RC without it. |
@pc494 yea I've been making a new class rather than adding to the exisisting classes mostly because the orientation mapping results are slightly different. I've now added in the |
So I think that we are quickly approaching the point where we should try to make a
Other Misc. Stuff that I might try to work on (To be updated later):
|
All good with me. At this point I have to add text explanations, but I will be away from tomorrow including June 4th.. Should this notebook replace any existing demo notebook? As I understand it, the segmentation code used for MgO nanocrystal segmentation will be removed in future versions? If so, would it be a good idea to replace 06 with the Ag example and also remove the machine learning in 01 to make the demos more clearly structured? @grigoryn94 might also be interested in this post. |
My plan was to create a "ML" folder to help organize the growing number of tutorials. The MgO segmentation will stay at least in some capcity, but I think that we should think about how to make it more general :). There are lots of new tools with sklearn, skimage etc that are continually evoloving and I think the idea is that it is fairly difficult to keep up with external tools adding new features to pyxem to keep up. That's kind of my goal with things like https://pyxem.readthedocs.io/en/stable/reference/generated/pyxem.signals.DiffractionVectors.cluster.html#pyxem.signals.DiffractionVectors.cluster which uses any of the methods avilable in https://scikit-learn.org/stable/modules/clustering.html or with the
Thanks for pinging @ grigoryn94 as well :) |
Is there anyone who can help with the review process a little bit over the next couple of days? It would be nice to merge some of the outstanding PR's and make a rc in the next couple of days so we can make a full release 0.19.0 release before the weekend. Right now we have:
@pc494 @ericpre @hakonanes I realize you all are quite busy at the moment but I wouldn't say no if any of you have any time to review or offer some suggestions. I think I owe all of you beer/coffee or both next time I see you :) |
I have time on Wednesday and the coming weekend. 🍺 sounds good! |
Is your feature request related to a problem? Please describe.
It might be nice to make a RC for 0.19.0 with the orientation mapping that way we can test and make sure that things are good/working the way that we want them to be.
Describe the solution you'd like
to_crystal_map
functionThe text was updated successfully, but these errors were encountered: