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 steps to start the gui in linux #248

Closed
wants to merge 2 commits into from

Conversation

gautam1168
Copy link
Contributor

These steps work when starting the gui from linux using anaconda setup.

These steps work when starting the gui from linux using anaconda setup. 
I have not yet checked this on windows (which I don't think has environment variables or
with the virtualenv setup).
@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling cd1e858 on gautam1168:patch-1 into b298780 on tardis-sn:master.

@wkerzendorf
Copy link
Member

have you made sure that these documents build with sphinx?

I have made some changes to ensure that the docs build with sphinx.
@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling e2ec886 on gautam1168:patch-1 into b298780 on tardis-sn:master.

@gautam1168
Copy link
Contributor Author

I have now tested that the changes I made build with sphinx. For doing this
test
I cloned the repo at github and added the changes to running.rst in docs
folder.
Then I ran make html, and checked that I am getting the right output by
opening
the running.html that was generated. Is that what you wanted?

Also there are a few packages which need to be installed to run the gui
that don't
get installed when the conda create is passed the file

https://raw.githubusercontent.com/tardis-sn/tardis/master/conda-requirements

These are

ipython 3.0.0
pyside 1.2.1
shiboken 1.2.1

Should I include instructions to install these in the running.rst, or
would these be added to the conda-requirements file?

On Sat, Mar 7, 2015 at 5:29 AM, landscape-bot notifications@github.com
wrote:

[image: Code Health] https://landscape.io/diff/106963
Code quality remained the same when pulling e2ec886
gautam1168@e2ec886
on gautam1168:patch-1
into b298780
b298780
on tardis-sn:master
.


Reply to this email directly or view it on GitHub
#248 (comment).

@wkerzendorf
Copy link
Member

just add instructions to running rst

@gautam1168
Copy link
Contributor Author

I have now created another pull request which in addition to an updated running.rst has an updated gui.py, which can work with both PySide and PyQt4. So I am closing this request. I will redit the running.rst on the other pull request to include instructions on installation of the packages.

@gautam1168 gautam1168 closed this Mar 7, 2015
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.

None yet

3 participants