Skip to content

Simplify and improve handling of parameters - #1051

Closed
nden wants to merge 6 commits into
astropy:masterfrom
nden:master
Closed

nden wants to merge 6 commits into
astropy:masterfrom
nden:master

Conversation

@nden

@nden nden commented May 4, 2013

Copy link
Copy Markdown
Contributor
  • Allow a parameter of type array
  • Allow models to have parameters of mixed type, e.g. a number and an array
  • Add some validation for multiple parameter sets
  • Allow Gaussian models to have multiple parameter sets

nden added 4 commits May 3, 2013 20:20
- Allow a parameter of type array
- Allow models to have parameters of mixed type, e.g. a number and an array
- Add some validation for multiple parameter sets
- Allow Gaussian models to have multiple parameter sets
@nden

nden commented May 7, 2013

Copy link
Copy Markdown
Contributor Author

Hmm, I added the last two commit by mistake here and I made a separate PR on a branch.
Should I just merge this one and close the other PR?
These all fix minor internal problems.
@eteq In cases like this (no API changes, still just fixing bugs, no new features) should I wait for someone else to look at it before I merge and for how long?

@nden

nden commented May 9, 2013

Copy link
Copy Markdown
Contributor Author

Closing this one, deprecated by #1075

@nden nden closed this May 9, 2013
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.

1 participant