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.

Sign up for GitHub

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

Jump to bottom

Cos angle of incident is missing in radiation analysis #8

Open
mostaphaRoudsari opened this issue Nov 11, 2013 · 2 comments
Open

Cos angle of incident is missing in radiation analysis #8

mostaphaRoudsari opened this issue Nov 11, 2013 · 2 comments
Assignees
@mostaphaRoudsari
Labels
bug

Comments

@mostaphaRoudsari
Copy link
Contributor

No description provided.

@theo-armour
Copy link
Member

Mostapha

Over the next couple of days I plan to build an app that uses AJAX to read the data from New_York_Central_Prk_Obs_Belv_NY_USA_SKY.json as an external file - as well as accepting data from the address input text field. This should make for a faster load of the app and make editing the app easier.

So then if you make edits to generateTestPoints() - or some such funtion - it should be easy to do a merge the two efforts.

Does that sound OK?

Theo

@mostaphaRoudsari
Copy link
Contributor Author

Hi @theo-armour. That sound perfect to me. Right now I have the sky copied into the runRadAnalysis which is not the best way to do it. Let me know when you have an example and I'll update the analysis component.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mostaphaRoudsari mostaphaRoudsari

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@theo-armour @mostaphaRoudsari