Rename `tf_peak_model_s1_cnn` to `tf_model_s1_cnn`

dachengx commented 1 year ago

What does the code in this PR do / what does it improve?

This name is following https://github.com/XENONnT/straxen/blob/b3e776f632f224808d823fe924dbde05f1fcdf28/straxen/plugins/peaks/peak_positions_cnn.py#L16-L26

We do not need to update the plugin version because it is lineage depends on plugin configs.

Can you briefly describe how it works?

Can you give a minimal working example (or illustrate with a figure)?

Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

coveralls commented 1 year ago

coverage: 93.534% (-0.01%) from 93.545% when pulling 6078a8bdb8c4365a97ecea3e3e9a93db68daef9d on tf_model_s1_cnn_name into b3e776f632f224808d823fe924dbde05f1fcdf28 on master.

dachengx commented 1 year ago

I will merge it. This PR WILL change the lineage. But fortunately only of peak_s1_positions_cnn.

XENONnT / straxen