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

clear command does not work #72

Closed
avylove opened this issue Jun 28, 2017 · 21 comments
Closed

clear command does not work #72

avylove opened this issue Jun 28, 2017 · 21 comments
Assignees
Milestone

Comments

@avylove
Copy link

avylove commented Jun 28, 2017

The clear command does not seem to function. I'm not sure if this is a xterm.js bug or a spyder-terminal bug.

What steps will reproduce the problem?

  1. Launch terminal

What is the expected output? What do you see instead?

Expected:

[avylove@localhost Project]$      

Actual:

Loading...                              
[avylove@localhost Project]$ cd /home/avram/Desktop/Audrey      
[avylove@localhost Project]$ clear
[avylove@localhost Project]$

Please provide any additional information below
Neither clear nor reset commands seem to do anything. I tried changing the clearTerm() function from to user term.clear(), which removed the clear command from the terminal, but still didn't clear the terminal.

Versions and main components

  • Terminal Version: Included
  • Spyder Version: 0.1.2
  • Python Version: 3.5.3
  • Operating system: Fedora 25
@ccordoba12 ccordoba12 added this to the v0.2 milestone Jun 28, 2017
@ccordoba12
Copy link
Member

@avylove, thanks for opening the issue.

@andfoy
Copy link
Member

andfoy commented Jul 4, 2017

@avylove Does PR #78 fixes your problem?

@andfoy andfoy added the os:linux label Jul 5, 2017
@andfoy andfoy self-assigned this Jul 5, 2017
@avylove
Copy link
Author

avylove commented Jul 5, 2017

It doesn't seem to. Both this version and the original one work fine when launched outside of Spyder, but don't work both during initialization and if I open additional terminals once Spyder is up.

@andfoy
Copy link
Member

andfoy commented Jul 5, 2017

Does the console output keeps the same?

@avylove
Copy link
Author

avylove commented Jul 5, 2017

Yes, same output.

@ccordoba12
Copy link
Member

ccordoba12 commented Jul 5, 2017 via email

@avylove
Copy link
Author

avylove commented Jul 5, 2017

It seems like something about the Spyder integration interferes with the clear command. I'm not sure it's a timing issue because the clear command does not clear the screen even after I can interact with the console, yet it works fine when the server is run independently and the console is accessed through Chrome.

@andfoy
Copy link
Member

andfoy commented Jul 5, 2017

Does this happens only with the first console, or does it also happens with new terminal tabs?

@avylove
Copy link
Author

avylove commented Jul 5, 2017

New terminal tabs as well

@ccordoba12 ccordoba12 modified the milestones: v0.2.1, v0.2 Jul 6, 2017
@andfoy andfoy modified the milestones: v0.2, v0.2.1 Jul 17, 2017
@andfoy
Copy link
Member

andfoy commented Jul 17, 2017

@avylove Does PR #93 fixes your problem?

@avylove
Copy link
Author

avylove commented Jul 17, 2017

Well I have mixed news. Turns out the issue must have been in Spyder because I don't have the issue with 0.1.2 when I bootstrap Spyder 3.x. Wish I had thought to do that before. I found this out because I installed the spyder-terminal master branch and didn't have the spyder-terminal plugin available in Spyder, so I thought maybe there was a dependency on the latest. Well it didn't work there either. Not sure what the issue is. How do I see some output?

@andfoy
Copy link
Member

andfoy commented Jul 17, 2017

Have you tried to open an internal console and see if there is some output there?

@avylove
Copy link
Author

avylove commented Jul 17, 2017

First thing I tried. Nothing there.

@andfoy
Copy link
Member

andfoy commented Jul 17, 2017

How did you installed the plugin?

@andfoy
Copy link
Member

andfoy commented Jul 17, 2017

Is the plugin visible? Sometimes the plugin might be not be shown because its visibility is disabled

@avylove
Copy link
Author

avylove commented Jul 17, 2017

It wasn't visible or on the panes menu. I just realized I was using your fork instead of the master branch. Trying with the master branch now and having issues with bower not being in the path. Installing with pip. Added --no-deps because it was failing with the Spyder version. Spyder isn't installed, just launching with bootstap script.

pip3 install --user --no-deps git+https://github.com/spyder-ide/spyder-terminal.git

Is there a better way to go about this?

@andfoy
Copy link
Member

andfoy commented Jul 17, 2017

Well, to install bower, you just need to execute:

npm install -g bower

@andfoy
Copy link
Member

andfoy commented Jul 17, 2017

I would prefer to emit the following commands:

# From spyder master branch
python setup.py install

# From spyder-terminal master branch
pip install -U .

@avylove
Copy link
Author

avylove commented Jul 17, 2017

Well I don't install things globally that aren't in rpms, but I did run:

npm install bower

Then I added ~/node_modules/.bin to my PATH. It installed and everything looks fine with spyder-terminal@master and spyder@3.x.

@andfoy
Copy link
Member

andfoy commented Jul 17, 2017

@avylove Great to hear that is working!

@andfoy
Copy link
Member

andfoy commented Jul 17, 2017

Release v0.2 of spyder-terminal will come on the following days

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

3 participants