Add pyforest to first CODE cell and not markdown or other about pyforest HOT 6 CLOSED

Never mind everyone, I added the feature myself with PR #46.

from pyforest.

Seconding this! I'm sure many of use have the first cell in markdown as the title block. Wondering what type of fix would be needed? Not familiar with JS, but I'm assuming this chunk in src/pyforest/_importable.py is where it needs to go:

display(
        Javascript(
            """
        if (window._pyforest_update_imports_cell) {{ window._pyforest_update_imports_cell({!r}); }}
    """.format(
                "\n".join(statements)
            )
        )
    )

from pyforest.

Happy to hear that you also thought about this :)

I am afraid, that the fix needs to be in JS and not in Python.
Most likely somewhere around here in src/pyforest/static/utils.js

	function setup_lab(notebookTracker) {
		window._pyforest_update_imports_cell = function (imports_string) {
			var cell = notebookTracker.currentWidget.model.cells.get(0).value;
			cell.text = get_new_cell_content(imports_string, cell.text);
		};
	}

	function setup_notebook(Jupyter) {
		window._pyforest_update_imports_cell = function (imports_string) {
			var cell_doc = Jupyter.notebook.get_cell(0).code_mirror.getDoc();
			cell_doc.setValue(get_new_cell_content(imports_string, cell_doc.getValue()));
		};
	}

Because this is the position where pyforest selects the cell to be updated. Currently, it is just the first cell.

from pyforest.

Mmmm. Adding this and using the return value should do it. https://gist.github.com/tnwei/6005afe55cb8898ba87daa36fce23392

from pyforest.

Submitted a pull request, feedback appreciated!

from pyforest.

Just for the protocol: the last state of your PR #29 was that you need to find a way to make your PR work on JupyterLab. Any progress here? Can someone else help?

from pyforest.