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

Release ARTS3 #56

Open
erikssonpatrick opened this issue Sep 6, 2019 · 3 comments
Open

Release ARTS3 #56

erikssonpatrick opened this issue Sep 6, 2019 · 3 comments
Assignees
Milestone

Comments

@erikssonpatrick
Copy link
Contributor

No description provided.

@erikssonpatrick
Copy link
Contributor Author

erikssonpatrick commented Oct 25, 2019

Things to fix and consider for ARTS-3 (for now draft for meeting later today):

Mandatory:

Would be very nice:

@riclarsson
Copy link
Contributor

riclarsson commented Oct 28, 2019

Patrick,

Please can you clarify Remove all obsolete code and features? What defines this? Because, to me, it seems we should first of all remove all control files that are not run or included by the test-setup. We cannot guarantee that they are working anyways. Then we would have to remove all methods that are not used in any of the remaining control files? They could be working and they could be used by others so we need an opinion on each of them. An opinion here means that someone with the opinion creates a feature test-file to allow the method to remain in ARTS. But what is next? Do we remove all inner functions of ARTS that are not used? Or do we even rearrange so that the lot of the helper-functions that don't really need a header because they are only called once has their header removed?

About Complete switch to new "radiative transfer system" #41 , I sent in some code a while back that mostly solves this. There is some problems but the difference between new and old method seems smaller than between either and the perturbed case. So there is probably something fundamental wrong in both.

@erikssonpatrick
Copy link
Contributor Author

All: Of course, you can not edit my post. So let's go for this. Add your points in a comment and I update my initial comment. And when you think your stuff is correctly reflected in the list, remove your comment. All to not get a very long thread here.

Richard: Added ARTSCAT-6. Yes, #41 is basically closed. I just wait for a last test by Simon before closing that issue.
When it comes to the control files we provide, I did not add much. The main reason is that I don't think we manage to make a proper revision of those files before March. But I agree that cfiles not used should be removed, or be included in some test. Added this as mandatory. I don't think we shall make Python mandatory, but that can be discussed. And started a new category.
I mainly meant that if there are internal functions that are not used, they shall be removed. There are likely still also WSMs that are replaced by newer ones, and they shall be removed.
I can not see that we can define a test for exactly every option in ARTS ...

@olemke olemke pinned this issue Sep 4, 2020
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

6 participants