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

0.19.0rc #1077

Open
1 of 3 tasks
CSSFrancis opened this issue May 13, 2024 · 8 comments
Open
1 of 3 tasks

0.19.0rc #1077

CSSFrancis opened this issue May 13, 2024 · 8 comments

Comments

@CSSFrancis
Copy link
Member

CSSFrancis commented May 13, 2024

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

@magnunor
Copy link
Collaborator

When should the DPCSignal changes be included? 0.19.0? Or later?

I will not have time to work on this before next week. So feel free to make an early RC without it.

@pc494
Copy link
Member

pc494 commented May 25, 2024

  • Make to_crystal_map function

Did you know about?
image

@CSSFrancis
Copy link
Member Author

Did you know about?

@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 to_crystal_map method.

@CSSFrancis CSSFrancis pinned this issue May 28, 2024
@CSSFrancis
Copy link
Member Author

CSSFrancis commented May 29, 2024

So I think that we are quickly approaching the point where we should try to make a .rc for 0.19.0. My plan is to:

Other Misc. Stuff that I might try to work on (To be updated later):

@tinabe
Copy link
Contributor

tinabe commented May 29, 2024

  * [ ]  Update ML Notebook @tinabe It might be good to add your notebook for ML to the Pyxem demos!

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.

@CSSFrancis
Copy link
Member Author

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?

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 filter method. Hopefully, then we can focus more on adding new tutorials and using pyxem in a more modular way rather in a strictly linear workflow like fashion.

@grigoryn94 might also be interested in this post.

Thanks for pinging @ grigoryn94 as well :)

@CSSFrancis
Copy link
Member Author

CSSFrancis commented Jun 3, 2024

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 :)

@hakonanes
Copy link
Member

I have time on Wednesday and the coming weekend. 🍺 sounds good!

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

5 participants