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

This script is failing in 2.7 but it could work if you fix the print in line 275 #3

Open
mjtueros opened this issue Jun 5, 2019 · 5 comments

Comments

@mjtueros
Copy link

mjtueros commented Jun 5, 2019

and then you would be so famous!

@mjtueros
Copy link
Author

mjtueros commented Jun 5, 2019

but please be sure not to use only 3 decimals for the time, as this might be insuficient in some cases
better go to 5 o 6.

@niess
Copy link
Member

niess commented Jun 5, 2019

I guess that this is the faulty line?
Makes me think: It would be good to have a common logger as well. This would also allow to handle such compatibility problems.

@niess
Copy link
Member

niess commented Jun 5, 2019

It looks like astropy does provide one. But I don't get how you actually log something? To me it just says how to configure the logger.

@azilles
Copy link
Contributor

azilles commented Jun 13, 2019

The astropy logger seems to be easy to use. I will see that I implement this after starting with the new framework.

@niess
Copy link
Member

niess commented Jun 13, 2019

@azilles, about the new framework, I have merged the libs, store and tools packages in a core grand package. That was too much maintenance having specific packages for all of those. Was spending more time managing the packages than actually writing code :P Was planning to integrate radio_simus as well once everything will be stabilised. If that OK with you?

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

3 participants