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

added drop down using AccordionWidget #336

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

mvnnn
Copy link

@mvnnn mvnnn commented Mar 11, 2016

issue #205

  • There are no merge conflicts.
  • If there is a related issue, a reference to that issue is in the
    commit message.
  • Unit tests have been added for the new feature.
  • The PR passes tests both locally (run nosetests) and on Travis CI.
  • All public methods and classes have docstrings. (We use the numpydoc
    format
    .)
  • An explanation has been added to the online documentation. (docs
    directory)
  • The code follows PEP8 guidelines. (use a linter, e.g.
    pylint, to check your code)
  • The new feature is documented in the Release
    Notes
    .
  • The code is backwards compatible. (All public methods/classes must
    follow deprecation cycles.)
  • All reviewer comments have been addressed.

itemList = []
page1 = []
for item in self._initial_conditions_text_widgets:
itemList.append(widgets.Text(description=item.name))
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to add value here. i try to find value using dir(symbol) but i didn't find this.

widgets.Text(description=item.name, value = ***)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@moorepants how to get symbol value . you have any idea ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The initial condition symbol and value are stored in self._system.initial_conditions.

@oliverlee
Copy link
Contributor

Do you have an example that illustrates this change?

@mvnnn
Copy link
Author

mvnnn commented Mar 11, 2016

example --->examples/chaos_pendulum/chaos_pendulum.py

@oliverlee
Copy link
Contributor

It looks like you implement an accordion view but only use a single page which doesn't help with the original issue. Also please reference the issue number in this PR.

screen shot 2016-03-11 at 16 31 13

If you're going to go with accordion view, I think it would look better to always use it and just add pages as necessary.

@mvnnn
Copy link
Author

mvnnn commented Mar 12, 2016

@oliverlee in this case scroller is best option. i will try to add scroller using iframe.

@moorepants
Copy link
Member

The above looks pretty good to me. You just need to make a box for the initial conditions too.

@mvnnn
Copy link
Author

mvnnn commented Mar 19, 2016

screenshot from 2016-03-19 21 48 01
@moorepants that's looks good.

@moorepants
Copy link
Member

I think an accordion for each of the initial conditions and the constants is preferable. Some systems have long lists of states too.

@mvnnn
Copy link
Author

mvnnn commented Mar 25, 2016

@moorepants How to get Constants value ?

@moorepants
Copy link
Member

I don't understand what you are asking.

_itemList = []
_initial_item = []
for item in self._initial_conditions_text_widgets:
_itemList.append(widgets.Text(description=item.name))
Copy link
Author

@mvnnn mvnnn Apr 17, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for delay, We need to add value property here but i don't know how to get value of item.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants