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

mnt: changed all origo references to origin #365

Merged
merged 1 commit into from
Aug 18, 2021
Merged

mnt: changed all origo references to origin #365

merged 1 commit into from
Aug 18, 2021

Conversation

zerothi
Copy link
Owner

@zerothi zerothi commented Aug 12, 2021

The usage of origo is a Danish word. So all is now transferred
to origin. Some of the routines still enable origo, but are
marked as deprecated.

This will probably break some codes.

Updated change-log

@pfebrer please have a look at this as it touches your code... Any comments are welcome!

@zerothi
Copy link
Owner Author

zerothi commented Aug 12, 2021

@tfrederiksen @sofiasanz you may have interest in this since Hubbard uses origo in some places.
As far as I recall it was @tfrederiksen who suggested me to change the name. I think this should fix all issues in the naming, whilst still retaining a few routines for getting/setting via the old origo name.

@pfebrer
Copy link
Contributor

pfebrer commented Aug 13, 2021

Seems fine to me!

@pfebrer
Copy link
Contributor

pfebrer commented Aug 17, 2021

Actually, could you wait for #312 to be merged or remove the changes in the visualization files (I would make the changes in my branch then). It would be a nightmare to rebase my branch after these changes 😅

@zerothi
Copy link
Owner Author

zerothi commented Aug 17, 2021

Yeah, I'll wait for #312.

@sofiasanz @tfrederiksen I'll merge this once #312 is in. Since you haven't complained (:smile:) I guess it is ok?

The usage of origo is a Danish word. So all is now transferred
to origin. Some of the routines still enable origo, but are
marked as deprecated.

This will probably break some codes.

Updated change-log
@zerothi zerothi merged commit c941b4e into master Aug 18, 2021
@zerothi zerothi deleted the origo-origin branch August 18, 2021 06:51
@tfrederiksen
Copy link
Contributor

Looks good to me. Sorry for the late response.

@zerothi
Copy link
Owner Author

zerothi commented Aug 18, 2021

No worries! :) Thanks thomas!

pfebrer pushed a commit to pfebrer/sisl that referenced this pull request Sep 8, 2021
The usage of origo is a Danish word. So all is now transferred
to origin. Some of the routines still enable origo, but are
marked as deprecated.

This will probably break some codes.

Updated change-log
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.

SuperCell.origo has no unique function
3 participants