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

Sometimes the AP/PE point doesn't have ap/pe written on it. #2156

Closed
ghost opened this issue May 11, 2019 · 3 comments · Fixed by #2162
Closed

Sometimes the AP/PE point doesn't have ap/pe written on it. #2156

ghost opened this issue May 11, 2019 · 3 comments · Fixed by #2162
Labels
Milestone

Comments

@ghost
Copy link

ghost commented May 11, 2019

https://i.imgur.com/16j8Rhw.png
Yeah, again, a small visual glitch I didn't notice in previous versions. Am I getting a little annoying by keep reporting minor issues?

@eggrobin
Copy link
Member

eggrobin commented May 11, 2019

I have encountered this kind of issue before (while developing Fano), I thought I had managed to get rid of all of its occurences; apparently I missed some.
Note that any changes to the behaviour of the Ap/Pe/AN/DN nodes almost certainly date back to Fano, when we fixed some bugs with them.

Can you provide reproduction steps? “sometimes” is not very specific, and unless I can deterministically reproduce the issue, it is going to be hard to figure out what is going on.

Sorry, something went wrong.

@eggrobin
Copy link
Member

I appear to have reproduced this by changing between reference frames a lot; I may have an idea as to what is going on, I will add some traces and investigate.

Sorry, something went wrong.

@eggrobin eggrobin added the bug label May 11, 2019
@pleroy
Copy link
Member

pleroy commented May 11, 2019

By the way, please keep reporting these issues: they are very useful to us and we appreciate that you take the time to make Principia better.

Sorry, something went wrong.

@pleroy pleroy added this to the Fatou milestone May 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants