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

Supports covar when some are empty #148

Merged
merged 2 commits into from
Jul 12, 2020
Merged

Conversation

jlaura
Copy link
Collaborator

@jlaura jlaura commented Jul 9, 2020

No description provided.

@jessemapel
Copy link
Contributor

@jlaura You should add a test for this case.

@coveralls
Copy link

coveralls commented Jul 9, 2020

Coverage Status

Coverage increased (+0.2%) to 50.623% when pulling a4fe307 on jlaura:master into 0127ed8 on USGS-Astrogeology:master.

@jlaura
Copy link
Collaborator Author

jlaura commented Jul 12, 2020

@jessemapel Basic test added that covers when None is passed and when a 2x3 ndarray is passed. We don't do any sort of logic checking for shape/values so this covers the cases we need to test I believe.

@jessemapel jessemapel merged commit ec59585 into DOI-USGS:master Jul 12, 2020
Kelvinrr pushed a commit to Kelvinrr/plio that referenced this pull request May 26, 2022
* Supports covar when some are empty

* Adds tests for aprioriCovar
ladoramkershner pushed a commit to ladoramkershner/plio that referenced this pull request Jun 7, 2022
* Supports covar when some are empty

* Adds tests for aprioriCovar
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

Successfully merging this pull request may close these issues.

None yet

4 participants