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

Magstatics state fn update #27

Merged
merged 3 commits into from
May 25, 2023
Merged

Conversation

berianjames
Copy link
Contributor

linted the file πŸ” @spollok sorry it has to happen at some point πŸ˜΅β€πŸ’«

Corrects the state function data load in run_simluation to match that in iterate_magnetization. I believe this works even though the loaded files have different data shapes (I have run this locally, but good also to confirm.)

Copy link
Contributor

@spollok spollok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fully agree that we should start with consistent linting πŸ‘
I will also hop on the Ruff linter then 🧹
And the idea with new Tile proxy class is super nice! Makes accessing the individual tile much smoother.

@berianjames berianjames merged commit 5282e9d into master May 25, 2023
@berianjames berianjames deleted the magstatics-state-fn-update branch May 25, 2023 11:49
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

Successfully merging this pull request may close these issues.

2 participants