Geographical sun addon #49359

Closed
opened 2016-09-15 02:12:48 +02:00 by stephen leger · 8 comments
Member

Blender Version
Broken: all

Short description of error
According https://de.wikipedia.org/wiki/Sonnenstand
the coords must be Ostern Positive
"Für einen Ort auf der geografischen Länge lambda (nach Osten positiv gezählt) ist der Stundenwinkel des Frühlingspunkts"
Osten in deutsch means East and not West !!! so : eg Zürich must be on positive side and Mexico on negative side.

But they are all West positive so they are badly broken and result using presets are all wrongs !

Result of algo dosent match sample on reference page. Reverted longitude sign does match.

How to fix
invert sign of all presets longitudes so they are East positives.

**Blender Version** Broken: all **Short description of error** According https://de.wikipedia.org/wiki/Sonnenstand the coords must be Ostern Positive "Für einen Ort auf der geografischen Länge lambda (nach Osten positiv gezählt) ist der Stundenwinkel des Frühlingspunkts" Osten in deutsch means East and not West !!! so : eg Zürich must be on positive side and Mexico on negative side. But they are all West positive so they are badly broken and result using presets are all wrongs ! Result of algo dosent match sample on reference page. Reverted longitude sign does match. **How to fix** invert sign of all presets longitudes so they are East positives.
Author
Member

Changed status to: 'Open'

Changed status to: 'Open'
Author
Member

Added subscriber: @StephenLeger

Added subscriber: @StephenLeger
Member

Added subscriber: @BrendonMurphy

Added subscriber: @BrendonMurphy
Member

If you would like to make & show changes to this addon, please post a diff here

If you would like to make & show changes to this addon, please post a diff here
Member

Changed status from 'Open' to: 'Archived'

Changed status from 'Open' to: 'Archived'
Brendon Murphy self-assigned this 2016-09-15 08:48:54 +02:00
Member

Changed status from 'Archived' to: 'Open'

Changed status from 'Archived' to: 'Open'
Member

Changed status from 'Open' to: 'Archived'

Changed status from 'Open' to: 'Archived'
Member

looking at the code in the script, this https://de.wikipedia.org/wiki/Sonnenstand seems to be already used?
archiving for now, apology's for confusion as there's also a Sun Position addon that I loosely maintain.
certainly these type scripts are worth getting right in the 2.8 series.
Thanks for report.

looking at the code in the script, this https://de.wikipedia.org/wiki/Sonnenstand seems to be already used? archiving for now, apology's for confusion as there's also a Sun Position addon that I loosely maintain. certainly these type scripts are worth getting right in the 2.8 series. Thanks for report.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#49359
No description provided.