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

Issue with cloud detector pickle file #9

Closed
jgomezdans opened this issue Apr 9, 2021 · 3 comments
Closed

Issue with cloud detector pickle file #9

jgomezdans opened this issue Apr 9, 2021 · 3 comments

Comments

@jgomezdans
Copy link
Contributor

I have sklearn 0.24.1 installed, but it appears that the cloud detector pickle file is incompatible with this:

ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-1-3016c41f79c7> in <module>
----> 1 import SIAC

~/SIAC/SIAC/__init__.py in <module>
----> 1 from .SIAC_S2 import SIAC_S2
      2 from .SIAC_L8 import SIAC_L8
      3 from .the_aerosol import solve_aerosol
      4 from .the_correction import atmospheric_correction
      5 from .atmo_solver import solving_atmo_paras

~/SIAC/SIAC/SIAC_S2.py in <module>
     11 from datetime import datetime
     12 from SIAC.create_logger import create_logger
---> 13 from SIAC.s2_preprocessing import s2_pre_processing
     14 from SIAC.the_aerosol import solve_aerosol
     15 from SIAC.the_correction import atmospheric_correction

~/SIAC/SIAC/s2_preprocessing.py in <module>
     30 file_path = os.path.dirname(os.path.realpath(__file__))
     31 gc.disable()
---> 32 cl = pkl.load(open(file_path + '/data/sen2cloud_detector.pkl', 'rb'))
     33 gc.enable()
     34 cl.n_jobs = multiprocessing.cpu_count()

ModuleNotFoundError: No module named 'sklearn.preprocessing.label'

This might just need "re-pickling" the cloud detector again.

@jgomezdans
Copy link
Contributor Author

I solved this by installing an older version of scikit-learn (0.23.1), but worth noting.

@MarcYin
Copy link
Owner

MarcYin commented Apr 10, 2021

Thanks for reporting this! There is a new version of CNN cloud and shadow mask trained by myself, which will be used in next version of the code and I think this issue should be resolved after I commit it...

MarcYin added a commit that referenced this issue Jun 8, 2021
MarcYin added a commit that referenced this issue Jun 8, 2021
scikit-learn version control related to issue #9
@MarcYin MarcYin mentioned this issue May 16, 2022
@MarcYin
Copy link
Owner

MarcYin commented Jul 8, 2022

This has been fixed.

@MarcYin MarcYin closed this as completed Jul 8, 2022
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

2 participants