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

Remove EA_MIRROR #77

Merged
merged 1 commit into from
Nov 28, 2023
Merged

Remove EA_MIRROR #77

merged 1 commit into from
Nov 28, 2023

Conversation

hamogu
Copy link
Member

@hamogu hamogu commented Nov 28, 2023

For a long time, the docs have said:

This simple model is of historical interest only and is no longer accurate or supported. This option should not be selected.

This PR removes the EA mirror model.
It reduces maintenance (though small, it's not zero) and I find that any option that exists will be used by a user at some point and then they get inaccurate results. It's safer to remove an option that expressively should not be used any longer.

closes #75

For a long time, the docs have said:
> This simple model is of historical interest only and is no longer accurate or supported. This option should not be selected.

This PR removes the EA mirror model.
It reduces maintenance (though small, it's not zero) and I find that any option that exists will be used by a user at some point and then they get inaccurate results. It's safer to remove an option that expressively should not be used any longer.

closes Chandra-MARX#75
@hamogu hamogu merged commit 4b7e931 into Chandra-MARX:main Nov 28, 2023
1 check passed
@hamogu hamogu deleted the remove_eamirror branch November 28, 2023 15:20
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.

Remove eamirror model
1 participant