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

better docstring #370

Merged
merged 1 commit into from
Feb 20, 2025
Merged

Conversation

Peter230655
Copy link
Contributor

@Peter230655 Peter230655 commented Feb 20, 2025

I tried to make the docstrings confirm to format given in #348

NB:
It was a bit difficult here to get a solution, see the Note in the docstrings.
Therefore I did not try

fname = f'.....'
if os.path.exists(fname):
solution = np.loadtxt(fname)

else:
inital_guess = .....

@moorepants moorepants merged commit c7e46d2 into csu-hmc:master Feb 20, 2025
22 checks passed
@Peter230655 Peter230655 deleted the particle-in-tube-1 branch February 20, 2025 19:55
@moorepants
Copy link
Member

The CI builds failed on master with "The job was not started because recent account payments have failed or your spending limit needs to be increased. Please check the 'Billing & plans' section in your settings."

I was not aware there were any limits for open source projects. That is news to me.

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.

2 participants