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
New layout #64
New layout #64
Conversation
Merge performed on MacBook Pro. PyBERT does not run, do to missing Simbeor-SDK env. var.
On my windows machine right now and when I click on Results -> Outputs, it causes PyBERT to crash with a LinePlot error:
Besides that, I really like the tab window in a tab window view! Makes the tabs much less clustered. Might even make sense to have "Config", "Tx", Rx", "Channel", and etc... I know Tx/Rx impedance is used to solve the channel impulse but now we have Tx controls on both the config and channel tabs. I could even seen the main tab having a visual of the full channel (Tx+Channel + Rx) that kinda highlights the current config. across the other tabs. Excited to see what |
Hi David,
Thanks for the feedback!
I’ve replied below, to preserve context.
Thanks,
-db
On Oct 26, 2019, at 5:51 PM, David Patterson ***@***.***> wrote:
On my windows machine right now: The AMI checkboxes for both TX and RX don't seem to work.
They only become enabled after you’ve loaded a DLL and AMI file; did you do that?
Clicking on Results -> Outputs causes PyBERT to crash with a LinePlot error:
chaco\lineplot.py:255: UserWarning: Chaco.LinePlot: len(value) 958 - len(index) 400 = 558
% (len(value), len(index), size_diff))
I see this often, but it doesn’t cause PyBERT to crash.
Is it actually causing PyBERT to crash, in your case?
Besides that, I really like the tab window in a tab window view! Makes the tabs much less clustered.
Thanks! :)
Might even make sense to have "Config", "Tx", Rx", "Channel", and etc... I know Tx/Rx impedance is used to solve the channel impulse but now we have Tx controls on both the config and channel tabs.
Haha! Great minds think alike. ;-)
Yeah, I definitely want to consolidate the Tx/Rx split.
Maybe we could knock around a few ideas sometime soon?
I could even seen the main tab having a visual of the full channel (Tx+Channel + Rx) that kinda highlights the current config. across the other tabs.
Oh, I REALLY like that idea.
Excited to see what draw_channel will do for the channel view. Is that part of your simbeor work or do you plan to incorporate that for the current channel view?
Yes, this new release is sort of an “un-Simbeor’d” version of the simbeor branch.
I wasn’t careful to separate the work I’m doing with Yuriy and crew from the new layout changes.
And, since we’re not quite ready to release the fully Simbeor-integrated version, I had to do some surgery. :(
… —
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub <#64?email_source=notifications&email_token=AAH7XGX5NDZF2H5DC2ZYQPTQQTQ2NA5CNFSM4JFG76YKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECKT7BQ#issuecomment-546652038>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAH7XGSJB5AL6Y4EV6MOLALQQTQ2NANCNFSM4JFG76YA>.
|
David, I merged, even though our conversation hasn't been brought to closure, because I want to get started on the Tx/Rx split. And I want to be using the new layout, to do that work. If we find something that is really broken, we'll address it on a separate maintenance branch; okay? Thanks, |
Yeah, I forgot about that with the AMI models. My develop branch has changed that some. So disregard.
It did crash in this case but it could be a different exception that is being masked. Doesn't crash on my main machine. (osx)
Sure, I'll reply to your email and setup a time. |
Hi David,
I'm creating this pull request, before merging, to give you a chance to review my changes.
Note that (I think) there're no technical changes here; only aesthetic ones.
I've given the GUI a complete make-over, taking advantage of the availability of "sub-tabbed" frames, in the Enthought Traits/UI framework.
(I've been waiting for this capability for a while now.)
Cheers,
-db