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

After upgrade/uavo import, before reset: UI panes consider themselves unsaved #1871

Open
mlyle opened this issue Jul 1, 2017 · 9 comments
Open

Comments

@mlyle
Copy link
Member

mlyle commented Jul 1, 2017

You're supposed to reboot anyways-- but if you don't, you get wonky results.

@mlyle mlyle added this to the Wired milestone Jul 1, 2017
@tracernz
Copy link
Member

tracernz commented Jul 2, 2017

The wizard should reboot at the end IMO, that would also result in automatically bringing up the setup wizard if you don't have actuator settings (although the upgrader still doesn't handle no settings properly either so that won't be perfect).

@mlyle
Copy link
Member Author

mlyle commented Jul 2, 2017

Yes, but caution needs to be used here-- I think that reboot could race with commits if done naively currently

@tracernz
Copy link
Member

tracernz commented Jul 2, 2017

Yeah, would be safe with a 5 second timer or something?

@tracernz
Copy link
Member

tracernz commented Jan 1, 2018

I prepared a fix for this (tracernz@df0f763)... but I can't actually repro this with an import or an upgrade on current next (be3307d). Can you please try next, and if you can still repro, try that branch?

@tracernz tracernz modified the milestones: Wired, Post-Wired Jan 31, 2018
@mlyle
Copy link
Member Author

mlyle commented Mar 17, 2018

I have a recent repro clicking through input stuff ---- will try to repro and check out your fix.

@mlyle
Copy link
Member Author

mlyle commented Mar 17, 2018

Actually it's output stuff.

@mlyle
Copy link
Member Author

mlyle commented Mar 17, 2018

I can still repro with the fix. Procedure: import a UAV file that changes actuatorsettings from defaults->nondefaults or vice versa. (Importing the same one doesn't trigger it!) Then click to output pane, then somewhere else.

@mlyle
Copy link
Member Author

mlyle commented Apr 19, 2018

So, to note: the output pane is one of the few that doesn't auto-build relations.

@mlyle
Copy link
Member Author

mlyle commented May 9, 2018

Another one: stabilization acts "unsaved" on connection of pipxtreme. My guess, the backing objects are missing and it calculates new values and seems unsaved.

@mlyle mlyle modified the milestones: Inconceivable, Ludicrous May 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants