Rename Param and Obs - Githubissues

jobrachem commented 10 months ago

Closes #53

Notes:

Getting the documentation right was a little tricky, because Sphinx does not work well with two objects of the same name (except for case differences) in the same module.
To deal with this, I moved Param and Obs to the liesel.model.legacy module.
The import paths for users are not affected. I made sure that Param and Obs are imported to liesel.model.__init__.py, so they can still be used as lsl.Param and lsl.Obs
The two functions are marked as deprecated, issue FutureWarnings and are marked to be removed in v0.4.0, see #129

jobrachem commented 10 months ago

Hi @wiep, would you do the review on this one?

jobrachem commented 10 months ago

Thanks! I think that's not necessary.

I just added a changelog entry. If you approve, feel free to merge and delete the branch.

wiep commented 10 months ago

great. Just a note: We need to publish a new version when we update the manuscript on arxiv so that the manuscript and software are in sync.

liesel-devs / liesel