search

desihub / timedomain

Time-domain code.
MIT License
3 stars 14 forks source link

cnn_classify using old obsdate/tilenumber keywords #15

Open AlexGKim opened 3 years ago

AlexGKim commented 3 years ago

@apalmese , is this line correct? The length of a string is the number of characters?

https://github.com/desihub/timedomain/blob/master/cronjobs/cnn_classify_data.py#L86

Have you tested the case where args.obsdates_tilenumbers is None?

AlexGKim commented 3 years ago

I think this may cause a problem here as well.

https://github.com/desihub/timedomain/blob/master/cronjobs/cnn_classify_data.py#L222