Cr flux 4comp fix

[athakuri22]

adding 4 component fluxes for GST and GSF.

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (38261e3) 100.00% compared to head (0b15640) 99.86%.
Report is 7 commits behind head on main.

❗ Current head 0b15640 differs from pull request most recent head 069df80. Consider uploading reports for the commit 069df80 to get more accurate results

Files	Patch %	Lines
src/simweights/_fluxes.py	87.50%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##              main      #28      +/-   ##
===========================================
- Coverage   100.00%   99.86%   -0.14%     
===========================================
  Files           12       12              
  Lines          739      747       +8     
===========================================
+ Hits           739      746       +7     
- Misses           0        1       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kjmeagher]

@ athakuri22, Thanks for submitting a PR, the code you submitted looks good. But there a few things you need to do before I can merge this PR. Please read the CONTRIBUTING guide. Specifically make sure all of the pre-commits pass as well as adding data to flux_values.json for the two flux models you added and push the changes to this branch.

[kjmeagher]

To get rid of the pylint error you can add #pylint: disable=invalid-name to the line where the class is defined. We already have fluxes with _IT suffixes designated icetop and GlobalFitGSTIT would be worse. You also need to add the new fluxes to__init__.py so people can import them and add them to test_fluxes.py so they are tested

[athakuri22]

#pylint: disable=invalid-name worked to get rid of pylint error.
I also added the new fluxes to __init__.py
Thank you.

[kjmeagher]

Thanks for the update. You still need to add tests for GlobalFitGST_IT and GlobalSplineFit_IT to tests/test_fluxes.py so the data you added to flux_values.json actually gets used

[kjmeagher]

also you can ignore that failure from (3.11, macos-12). that looks like nonsense from pytables

[kjmeagher]

it looks like you put the wrong values in flux_values.json for GlobalSplineFit_IT

[athakuri22]

Yes, you are right. GlobalSplineFit_IT on flux_values.json was wrong.
I have corrected that now. Thank you.

[kjmeagher]

closes #24