Opened 7 years ago

Closed 7 years ago

#17203 closed defect (fixed)

Make sage -notebook=ipython land by default in pwd

Reported by: slabbe Owned by:
Priority: major Milestone: sage-6.4
Component: notebook Keywords:
Cc: Merged in:
Authors: Sébastien Labbé Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: e8f739c (Commits, GitHub, GitLab) Commit: e8f739c38be79155ffa6a6ca8e73a8854c3af6f9
Dependencies: Stopgaps:

Status badges

Description (last modified by slabbe)

Since #16996 the default directory of sage -notebook=ipython is DOT_SAGE/notebooks_ipython/ following the old notebook. In the old Sage notebook, each worksheet was a directory and even each single cell was stored in its own directory. Therefore, there was no notion of "present working directory" at all. With the ipython notebook, each worksheet lives in one present working directory chosen by the user. That is one of the main point of the ipython notebook philosophy and we should not change that in Sage. See the post of Fernando Perez about difference between the ipython notebook and the Sage notebook:

"This is a key difference of our approach and the Sage notebook, so it' worth clarifying what I mean, the key point being the execution model and its relation to the filesystem. The Sage notebook took the route of using the filesystem for notebook operations, so you can't meaningfully use 'ls' in it or move around the filesystem yourself with 'cd', because Sage will always execute your code in hidden directories with each cell actually being a separate subdirectory. This is a perfectly valid approach and has a number of very good consequences for the Sage notebook, but it is also very different from the IPython model where we always keep the user very close to the filesystem and OS. For us, it's really important that you can access local scripts, use %run, see arbitrary files conveniently, etc., as these are routine needs in data analysis and numerical simulation." (Fernando Perez, The IPython notebook: a historical retrospective, January 2012)

Let's change the default directory so that it is equal to the present working directory (pwd) as it is the nice way of using ipython.

See the related discussion and vote on sage-devel.

Change History (6)

comment:1 Changed 7 years ago by slabbe

  • Branch set to u/slabbe/17203
  • Commit set to 9ee8d469119195ec6841cfffb8190cd6cc95c374
  • Status changed from new to needs_review

New commits:

06d90fdMerge branch 'u/vbraun/sage_ipynb' of into ipynb
de657f4Merge branch 'u/vbraun/sage_ipynb' of into ipynb
776620dMerge branch 'u/vbraun/sage_ipynb' of into ipynb
ff07faeMerge branch 'u/vbraun/sage_ipynb' of into ipynb
1910b64Merge branch 'u/vbraun/sage_ipynb' of into ipynb
9ee8d46Trac #17203: make pwd the default directory for the ipython notebook
Version 0, edited 7 years ago by slabbe (next)

comment:2 Changed 7 years ago by slabbe

My branch based on top of #16996 have conflicts. I will compile 6.4.beta6 during the night and fix the conflict tomorrow.

comment:3 Changed 7 years ago by git

  • Commit changed from 9ee8d469119195ec6841cfffb8190cd6cc95c374 to e8f739c38be79155ffa6a6ca8e73a8854c3af6f9

Branch pushed to git repo; I updated commit sha1. New commits:

e8f739cTrac #17203: make pwd the default directory for the ipython notebook

comment:4 Changed 7 years ago by slabbe

  • Description modified (diff)

comment:5 Changed 7 years ago by vbraun

  • Authors set to Sébastien Labbé
  • Reviewers set to Volker Braun
  • Status changed from needs_review to positive_review

comment:6 Changed 7 years ago by vbraun

  • Branch changed from u/slabbe/17203 to e8f739c38be79155ffa6a6ca8e73a8854c3af6f9
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.