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

Request documentation on Pitch Bin default values (Ps) #47

Open
Gregg140 opened this issue Dec 5, 2017 · 2 comments
Open

Request documentation on Pitch Bin default values (Ps) #47

Gregg140 opened this issue Dec 5, 2017 · 2 comments

Comments

@Gregg140
Copy link

Gregg140 commented Dec 5, 2017

I have a general question regarding adding pitch bins in a .json file:

This is a fragment of a json file I'm working with. I want to ad a 120 degree pitch bin - which I've done in P_pitches:

...................
"init_kwargs": {
"P_pitches": [
45,
60,
90,
105,
120, <- This pitch bin added
130,
150,
180
],
"Ps": [
0.58,
0.5,
0.41,
0.7,
<- Need to add value here
1.0,
0.9,
0.79
], .................

But I also need to add a value in Ps. Best I can get from the documentation is that these are 'default values'. They seem to be less than one. Can you describe how they are used and how one would arrive at a default value when adding another pitch bin?

Thanks

@taldcroft
Copy link
Member

@Gregg140 - these are the values that are used to create a XijaModel component object, e.g. when instantiating a SolarHeat class object. They typically then get updated later, hence the designation as default parameters.

Here is the relevant code that should make it more clear:

def _set_from_model_spec(self, model_spec):

@taldcroft
Copy link
Member

A pull request to update the documentation would be most welcome! I probably can't make this a priority any time soon, so if you or ACIS does not plan on this then go ahead close this issue.

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

2 participants