-
Couldn't load subscription status.
- Fork 52
Proposed changes to ng_to_nasa #148
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
Conversation
|
This pull request introduces 1 alert when merging d8b680a into 38ff274 - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging 571d9c6 into 38ff274 - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging 10f9308 into 38ff274 - view on LGTM.com new alerts:
|
@teubert I am good with getting rid of atleast_1d. It was something that Cason put in because of mismatched datatypes that he ran into when we were working on MCMC but it didn't affect the vectorization. |
A few proposed changes: