read executed line to get indentation level and expand code #10

mbroedl · 2018-07-26T12:06:57Z

Should solve #3 in line with what @nikitakit mentioned in nteract/hydrogen#862.
It reads the first line of the selection and appends the indentation to the code passed.
This removes both limitations mentioned in the README.

Further solves nteract/hydrogen#1341 and nteract/hydrogen#1344 I believe.
However, I couldn't find a way to move the tick bubble to the correct line (except for adding a watch and moving it by cheating).
It expands the code line by line if certain (configurable) keywords are found.
Maybe you have an idea about the tickbox?

kylebarron · 2018-07-26T18:01:47Z

This looks quite good. Especially the if, elif, else support.

I'll use this for a bit and let you know if I find any bugs.

kylebarron · 2018-07-26T19:11:52Z

Here's a bug. Put your cursor anywhere on the line x = 2 and run Hydrogen: Run And Move Down/Shift+Enter. It works with Hydrogen:Run though. I'm not sure what the difference is.

x = 2
if x == 1:
    print('a')
else:
    print('b')
# File "<ipython-input-17-c80c1a5b0b20>", line 2
#     print('a')
#     ^
# IndentationError: unexpected indent

Also, I'm not sure if it's possible, but it would be great for Hydrogen: Run And Move Down to move the cursor to the end of the if/else statements.

Before:

After:

mbroedl · 2018-07-26T22:12:36Z

@kylebarron thanks for testing!!
I have to admit I never used the run and move down before.
run executes but stays where you had your cursor before, whereas the former moves the cursor immediately to the end of the called code.

So I checked, and it seems like the cursor is moved before hydrogen-python is called which messes up the knowledge of the plugin where it is.
That's why the run and move down messes up: it executes x=2 then thinks it is in the if line already (bc the cursor is moved down, and it checks for the line number) and adds the print statement afterwards. — I could instead match the executed code in the editor, but certainly that would be a bit more hacky. :/

It would be nice if there was a way to intercept the events fired to hydrogen, so as to know what key combination/command was used... I shall have a look at this. Which action (run / run + move down) was called is crucial to knowing when to move the cursor further. But it shouldn't be too difficult.

mbroedl · 2018-07-27T00:38:12Z

@kylebarron The run and move down behaviour should be fixed now. Unless you have a selection range and want to run and move down, in which case the script understands it as run and move down without selection as the selection is not preserved.

So the displayed take of your example results in 'b' and moves down to the print(x) line. To avoid that would mean to build a complete wrapper around hydrogen... and admittedly, the cases where you would want to execute the if part of a loop and not the else part but then have your cursor at the else part to execute it immediately afterwards might be limited. At least that's what I hope, and thus that this limitation is okay...

I haven't trialled the other run-options yet.

kylebarron · 2018-07-27T16:14:39Z

Am I missing something or is this a regression? It no longer runs the entire if/else clause: I'm using the base run here.

mbroedl · 2018-07-27T16:32:15Z

In my atom it works. Are you sure its the right package loaded?
e.g. are you in dev mode in case you installed it to dev mode?

kylebarron · 2018-07-27T16:35:12Z

Oh sorry, yesterday I disabled then uninstalled the package. Today when installing it again it stayed disabled. So I just had to enable it in the settings again. I'll let you know if I find any other bugs

kylebarron · 2018-07-27T17:25:14Z

Another error:

x = 2
if x == 2:
    print('a')
    print('b')
else:
    print('c')

If you put your cursor anywhere on the if x == 2 line, and hit Run and Move Down, I get:

Uncaught TypeError: Cannot read property 'replace' of undefined

At /home/kyle/.atom/packages/hydrogen-python/lib/main.js:110

TypeError: Cannot read property 'replace' of undefined
    at PythonKernelMod.execute (/packages/hydrogen-python/lib/main.js:110:58)
    at MiddlewareAdapter.execute (/packages/Hydrogen/lib/kernel.js:149:24)
    at Kernel.execute (/packages/Hydrogen/lib/kernel.js:287:33)
    at Object._createResultBubble (/packages/Hydrogen/lib/main.js:391:12)
    at Object.createResultBubble (/packages/Hydrogen/lib/main.js:351:12)
    at Object.run (/packages/Hydrogen/lib/main.js:447:12)
    at HTMLElement.hydrogenRunAndMoveDown (/packages/Hydrogen/lib/main.js:108:50)
    at CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app/src/command-registry.js:384:49)
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/usr/share/atom/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:621:22)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/usr/share/atom/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:412:28)
    at WindowEventHandler.handleDocumentKeyEvent (/usr/share/atom/resources/app/src/window-event-handler.js:110:40)

kylebarron · 2018-07-27T17:30:55Z

Another separate error. If there's any indented whitespace on a line inside an if/else block, it fails with a Python IndentationError.

if x == 2:
    print('a')
    print('b')
   
# 4 spaces ^
#
# File "<ipython-input-17-42c9438ba923>", line 2
#     print('a')
#         ^
# IndentationError: expected an indented block

kylebarron · 2018-07-27T17:31:35Z

Also fails when the if block is already indented:

class test(object):
    def __init__(self, arg):

        if x == 2:
            print('a')
                
        else:
            print('c')
        
        # Error when line after print('a') has 4, 8, or 12 spaces

kylebarron · 2018-07-27T17:34:11Z

It also doesn't work generally if there's any space between the if and else clauses.

x = 2
if x != 2:
    print('a')

else:
    print('c')

With Run on the if line, I just get the check from Python saying None. With Run and Move down on the if line, I get IndentationError, even when the line between if and else is not indented at all.

kylebarron · 2018-07-27T17:35:20Z

I'm worried that any solution will be hacky and will have myriad edge cases that don't work.

nikitakit · 2018-07-29T01:56:13Z

@mbroedl Thanks for taking a look at the indentation level issues!

In addition to some of the edge cases that @kylebarron pointed out, I also want to mention the "run cell and move down" command (there's also plain "run cell" variant). Do you think you can get your approach to work with the full range of hydrogen commands? If you're having trouble figuring out which command triggered the code running, maybe something in the command-history package or other atom packages for tracking command execution can be helpful.

Even if the changes here only work for a particular workflow (e.g. one that avoids using certain commands), it may still be worth shipping them but they'd need to be hidden behind a configuration flag. I want to avoid users encountering these issues and then reporting them to the main hydrogen bug tracker. I think this has happened a few times already with bugs in code I wrote.

Alternatively, might you be interested in modifying hydrogen a bit to get a cleaner solution? It should be possible to have the "code manager" file in hydrogen expose some options/callbacks to the plugin API, so that the plugin can operate on full information about which code is being run instead of having to piece things together after-the-fact.

mbroedl · 2018-07-29T21:11:08Z

@kylebarron Thanks a lot for the elaborate bug reports! I shall have a look at them soon. Apart from the first one, all are indeed a bit more hacky. — You're totally right about the edge cases... but then anything that works better than now is an advancement.

@nikitakit Is there any documentation what the run-cell commands should do? I could not quite figure that out yet? — I totally understand your concerns about the config and bug trackers. Couldn't agree more.
I believe indeed that the best solution would be to allow a plugin to intercept the code manager, simply because this way the response-bubble would also move, which would be one of the big downsides of any external solution. I could imagine that a simple check here:
https://github.com/nteract/hydrogen/blob/master/lib/code-manager.js#L313
Something like

if (plugins[grammar].findCodeBlock) {
    return plugins[grammar].findCodeBlock(editor, row);
}

But I'm probably overlooking a lot of complexity here?
The other option would be to have the plugin pass on some kind of regex or function to identify when to continue code expansion to the bottom, and regexes how to identify cells? Not sure if that would be a use case for other languages?
Further, I would assume it wouldn't do any harm to remove an equal amount of indentation from every line in hydrogen directly?
— Too many questions, haha.

kylebarron · 2018-07-29T22:19:05Z

It would be a huge step forward to fix these issues that Hydrogen has with Python. However with Hydrogen proper, I always know that there's a way for me to run a specific section of code, and when unexpected bugs occur with hydrogen-python, it's easiest to just turn the extension off. With if/else blocks within a function, that might mean I have to select the entire block, fully dedent it, and then run it. But it'll always work with Hydrogen proper.

There's an open issue about removing an equal amount of indentation from all lines here: nteract/hydrogen#862. Now that I read the thread again, I think a solution along the lines of nteract/hydrogen#862 (comment) would be ideal and such a PR could be accepted. I may try to delve into that and see how hard it would be.

kylebarron · 2018-07-30T02:59:13Z

@mbroedl You may want to look at my PR for Hydrogen related to common leading whitespace: nteract/hydrogen#1363

I spent hours on a relatively in-depth solution. Then I discovered that removing 7 characters would fix it.

mbroedl · 2018-07-30T08:16:16Z

@kylebarron thanks for taking the initiative. — As said the only problem is now: where to intercept with a possible API, as hydrogen-python can not anymore guess the indentation level. Also, one would want to strip the whitespace again after the expanded code is selected.
I'm not quite familiar about the order of calling though.

kylebarron · 2018-07-30T13:15:50Z

Sorry if I confused you... I meant to say that instead of the in-depth solution of trying to remove common whitespace, if I just take off the .trim() and don't remove any common whitespace, then everything works fine. The PR is not the branch in my second link above.

The reason why the current Hydrogen fails is because it removes the leading whitespace from the beginning of the selection but nowhere else.

kylebarron · 2018-07-31T13:37:28Z

My idea for expanding a selection based on if/elif/else is:

Get location of cursor. Note what buffer column it's in.
Find next buffer row where there is text before or on the same column as above.
If the text is in a prior column, stop.
If the text is in the same column, check if the next four characters are elif or else, in which case keep searching, recursively in the case of elif.

This is my idea for what might support indented if/else clauses as well.

I might first try to implement this in a my init script, having it create the full selection that I would then send to Hydrogen.

mbroedl · 2018-07-31T14:24:02Z

@kylebarron
Thanks for sharing your thoughts.
Are you thinking of creating a dynamic cursor selection before sending off the command to hydrogen?
Make sure to ignore lines with comments or white space even when they have less indentation, etc. This chunk of the PR has something similar (slightly modified):

      let nextRowText = e.lineTextForBufferRow(lastRow + 1);
      const expandCode = [' '].concat(atom.config.get('hydrogen-python.expandCode')).join('|');
      while (nextRowText.match(new RegExp(`^(${indent}(${expandCode})|\s*#|\s$)`))) {
          code += `\n${nextRowText}`;
          lastRow += 1;
          nextRowText = e.lineTextForBufferRow(lastRow + 1);
      }

With the default for the setting being: ["else", "elif", "\\}", "\\]", "\\)"]

kylebarron · 2018-07-31T15:01:17Z

I was just describing creating a cursor selection before sending off to Hydrogen because I'm not yet familiar with the Hydrogen extension APIs.

That chunk does look similar to what I had in mind.

Are there changes you want to make to this PR given the changed functionality in Hydrogen?

mbroedl · 2018-07-31T15:52:07Z

The change actually caused some trouble because the right trim was gone, and it took me a while to realise.
The first error you mentioned above might have been related to the code including the last line of the buffer? This is now solved, as well as all the other cases you mentioned.

I believe it works well now!

I included a default=false setting to enable code expansion. So make sure you tick that box before testing.

Imo, the next step would be now to fiddle into the hydrogen API that the plugin is asked what code execute before it is executed.

kylebarron · 2018-07-31T16:16:26Z

Yes, there was some discussion whether to keep the right trim or not, but the merge removed the right trim as well.

So the plugin isn't currently able to change the code before executing? At what point does it get control from Hydrogen?

mbroedl · 2018-07-31T17:00:32Z

Well, it changes the code before executing, but it doesn't seem to be able to tell hydrogen that the code has changed. Does that make sense?

Just tracing the plugin thing through the hydrogen files:
The plugin seems to intercept as middleware here:
https://github.com/nteract/hydrogen/blob/master/lib/plugin-api/hydrogen-kernel.js#L56
These are the interface options for middleware:
https://github.com/nteract/hydrogen/blob/master/lib/plugin-api/hydrogen-types.js#L29
And here is the kernel (including calls to middleware) implemented:
https://github.com/nteract/hydrogen/blob/ac02fa91c81b850642db948ed5787af2771dc4b3/lib/kernel.js

It seems as if the middleware intercepts right before sending the code to the kernel.
Thus hydrogen keeps its internal idea of where to place the bubble.
Ideally for our purpose, one could return the code and the range of it so that hydrogen knows where to place the bubble. But I couldn't find out how the kernel and the output (or line, even) are linked.

mbroedl · 2018-08-01T21:05:27Z

Thanks, @kylebarron . I could reproduce the error, put in a check for the last line when filtering for regex + breakpoints, and now the error disappeared.
Let me know if there is anything more!

nikitakit · 2018-08-03T06:01:00Z

Thanks for keeping up with this, and I'm happy that the trim changes have been merged into hydrogen core!

I'll take a closer look at the code and try running it this weekend. I appreciate that you put this feature behind a default-false config, which should make it possible to ship this out faster and see what people think.

kylebarron · 2018-08-03T16:27:59Z

@mbroedl

I hit another "Uncaught TypeError: Cannot destructure property lineText of 'undefined' or 'null'.". This time it occurs specifically when running code that has been folded. So it works when my cursor is on the class line with the code

class test(object):
    def __init__(self, arg):
        super(test, self).__init__()
        self.arg = arg

    def helloworld():
        x = 2
        if x == 2:
            print('hello world')
        else:
            print('not 2')

    def test2():
        try:
            pass
        except Exception as e:
            raise


if __name__ == '__main__':
    main()

But it fails when the def lines are folded and my cursor is on the class line.

class test(object):
    def __init__(self, arg):
    # folded

    def helloworld():
    # folded

    def test2():
    # folded

if __name__ == '__main__':
    main()

I'm guessing that this has something to deal with using tokensForScreenRow. And then screen vs buffer coordinates might be off? It doesn't appear that there's a tokensForBufferRow function.

mbroedl · 2018-08-03T17:04:45Z

Thanks for the detailed report @kylebarron .
No tokens for screen row, but this worked e.tokenizedBuffer.tokenizedLines[rownum].tokens, although then they are in the point syntax (foo.breakpoint) instead of in the css syntax (syntax--foo syntax--breakpoint).

I also added except and finally to the defaults. Looking at a list of keywords I think these four (else, elif, except, finally) should be all for which python continues on the same indentation level. Or did I miss anything?

kylebarron · 2018-08-03T17:08:01Z

I think that's all. The regex extensibility is great. I added plt\. so that I could run

plt.plot()
plt.xlabel()
plt.ylabel()
plt.show()

without having to select the lines.

mbroedl · 2018-08-03T17:13:33Z

Oh, that's sweet! I hope it won't ever surprise you!
It might be a nice example in the README file?

kylebarron · 2018-08-03T17:46:49Z

Yeah it could be a nice user-defined example

nikitakit

I just tested this out (with the master version of hydrogen) and everything is working as expected so far!

The next step before merging would be to clean up the code and make sure the README/config docs are up-to-date. I have some non-exhaustive suggestions in my review.

At some point we'll also need to remove "Feature: Strip common leading whitespace" from the README, since the hydrogen patch is mostly taking care of that. Would you like to take care of that in this PR?

nikitakit · 2018-08-07T04:14:18Z

README.md

-
-(Pull requests are welcome to fix the first issue; the second probably can't be fixed without changing the Hydrogen package)
+There is also an experimental option to expand the code passed on by hydrogen to include else/elif or closing brackets, which until now is only possible when selecting the code.
+However, the tick mark can not be moved to the right location as of yet.


Can you add some examples to the readme, and document the relevant config options needed to enable this feature?

nikitakit · 2018-08-07T04:14:52Z

lib/main.js

-import React from "react";
-import ReactDOM from "react-dom";
-import v4 from "uuid/v4";
+import React from 'react';


What is changing all the quotes here? Is it an automated tool you're running, or did you actually replace all of these manually?

That is the eslint airbnb base spec that you defined in the package.json. That spec enforces single quotes, and my atom therefore auto-corrected it while saving (I always tried to not commit these parts, but I seem to have missed that once).

nikitakit · 2018-08-07T04:18:45Z

package.json

+  },
+  "configSchema": {
+    "expandCode": {
+      "title": "[experimental] expand code",


This needs a clearer name and description. Right now the name seems a bit cryptic and doesn't directly relate to how people would want to use it.

Possible names I'm thinking of include "Extend code to execute", though that still isn't quite right. Any thoughts?

Also, the title should be capitalized in the same way as other settings in Atom.

nikitakit · 2018-08-07T04:19:38Z

package.json

+      "title": "Code to expand",
+      "type": "array",
+      "default": ["else", "elif", "except", "finally", "\\}", "\\]", "\\)"],
+      "description": "If the above is enabled, this list will be passed on to a regex, so experts may define their own tokens. In the default setting, else, elif, as well as all closing braces are expanded on."


I'd say "you" instead of "experts"

nikitakit · 2018-08-07T04:45:23Z

lib/main.js

+    const selection = e.getSelectedBufferRange();
+    const isSelection = !selection.start.isEqual(selection.end);
+    if (atom.config.get('hydrogen-python.expandCode') && !isSelection) {
+    // trim right to avoid empty lines at the end of the selection


This is a very long block of code that only executes if the particular setting is enabled. I recommend moving it into its own function.

I wanted to do that, but I couldn't figure out what to do with the next variable. As in: would I make it part of the export and use next as first argument, or would I rather create a separate function? Please let me know what you'd prefer.

My original thought was to have a function that accepts the code (as a string) and returns a modified string.

nikitakit · 2018-08-07T04:46:45Z

lib/main.js

+      code = code.trimRight();
+
+      // read editor settings
+      const isRunWithoutMove = code.includes(e.lineTextForBufferRow(selection.start.row).trim()) || e.lineTextForBufferRow(selection.start.row).match(/^\s+#/);


e.lineTextForBufferRow(selection.start.row) is used twice here, and once more in the code below. Can you make it a variable?

nikitakit · 2018-08-07T04:47:43Z

lib/main.js

+    // trim right to avoid empty lines at the end of the selection
+      code = code.trimRight();
+
+      // read editor settings


Comment about reading settings doesn't seem to apply to the code below

Nevermind. Though if you read the config setting to a variable instead of inlining it, the code might be more readable.

nikitakit · 2018-08-07T04:48:57Z

lib/main.js

+      const expandCode = [' '].concat(atom.config.get('hydrogen-python.expandCodeList')).join('|');
+      const re = new RegExp(`^(${indent}(${expandCode})|\s*#|\s*$)`);
+      const isBreakpoint = rownum =>
+        e.tokenizedBuffer.tokenizedLines[rownum].tokens.reduce((l, t) =>


Are there better variable names than l,t,ll,tt? (For that matter, e is probably not the best choice for naming the editor, either)

nikitakit · 2018-08-07T05:05:30Z

One more note (doesn't have to be in this PR): if this feature were extended to handle regexes before the currently-selected code, as opposed to just after, it would also be able to properly attach decorators to functions. See nteract/hydrogen#77 for details.

mbroedl · 2018-08-08T12:06:26Z

@nikitakit Thanks for the elaborate review. Will work on them next week!

kylebarron · 2018-08-08T19:57:27Z

Here's another bug:

kylebarron · 2018-08-09T22:37:32Z

@mbroedl This might be the same bug. If I have my cursor at for and run Run and Move Down, it tries to add all the text in the selection apparently

        for (Token, in_line), log_line in zip(syn_chunks, log):
            # Split log_line on \r\n.
            if str(Token) != 'Token.MatchingBracket.Other':
                # Since I'm sending one line at a time, and since it's not a
                # block, the first line should equal the text sent
                # The assert is just a sanity check for now.
                log_l = log_line.split('\r\n')
                assert log_l[0] == in_line
                log_all.extend(log_l[1:])
                log_all.append('')

        log
        syn_chunks


    def export_graph(self):
        """
        NOTE: unclear whether I actually need to save all the graphs individually, or if I can just overwrite one. Since I'm just writing them to load them into Python, and the next graph shouldn't start writing until I'm done reading the previous one into Python, I can probably just overwrite the same file.
        """
        # Export graph to file
        cmd = 'qui graph export `"{0}/graph{1}.{2}"\' , as({2}) replace'.format(
            self.cache_dir, self.graph_counter, self.graph_format)
        self.do(cmd)

kylebarron · 2018-08-09T22:39:39Z

I forgot to mention... It happens specifically when I have exactly 2 empty lines between syn_chunks and def export_graph. When I have 1 or 3 empty lines between them it works fine.

The console thinks it only expanded by one line:

mbroedl · 2018-08-19T13:36:09Z

@nikitakit I've worked through your comments and now it should be adjusted accordingly.

@kylebarron Thanks for the detailed bug report. After fixing the first one, I could not reproduce the second one,so I assume it's been fixed now as well!

mbroedl · 2018-08-19T14:19:37Z

@nikitakit The latest commit also includes decorators in the code expansion.

kylebarron · 2018-08-19T15:02:11Z

package.json

+      "description": "If the above setting is enabled, this list will be passed to a regex. Any of these items in the list need to occur on the same intentation level as the first line. You may define your own custom elements to modify the code to your preferred behaviour. In the default setting, else, elif, except, finally, as well as all closing braces are expanded on."
+  },
+  "prependCodeList": {
+    "title": "Code to Expand Before the Execution",


Selection makes more sense to me than Execution here

I don't think "Selection" is quite right. If there is code selected, the expansion features won't necessarily activate.

nikitakit · 2018-08-24T04:50:15Z

Just had a chance to test the feature out out today. Everything appears to work as advertised, and the decorator handling is a nice and useful addition!

I'm going to merge this now and put out a new release soon. Unfortunately I didn't get the chance to test this super-thoroughly because I don't have any active projects where I'm using hydrogen every day. I hope that you'd be willing to take a look if any github issues are filed after this feature is out!

I understand that there are some limitations because hydrogen doesn't provide the right API for this sort of feature. For example, having a selection and calling run-and-move-down will result in the selection going away. My view on this is there are diminishing returns to building increasingly complex workarounds, and any further effort might be better spent on improving hydrogen core to fix the core problem.

The two directions I'd consider exploring are:

A plugin API that explicitly deals with modifying execution regions. That way hydrogen can directly provide which region in the buffer it intends to execute and why (run at cursor vs. run selection vs run cell), and a plugin can modify the start and endpoints of the executed region
Using the new tree-sitter parsers. Currently hydrogen determines which code to run using Atom's code-folding APIs, which are based purely on indentation. The new tree-sitter parsers provide a syntax-tree view of the code, and there might be a language-agnostic manner to pull out code regions to execute in hydrogen itself.

mbroedl · 2018-08-25T08:56:37Z

Thanks for merging @nikitakit .
Yeah, happy to keep a look at issues, in case there's some coming.

For example, having a selection and calling run-and-move-down will result in the selection going away.

I believe this is the default behaviour of hydrogen, rather than hydrogen-python.

I agree that hydrogen should be the target for major future improvements!
Tree sitter looks pretty nice and solid for this purpose, and it should cover all the originally intended use cases, but not so the nice side effects such as extending on e.g. plt.. Otherwise, the plugin API would always work as well, but would probably be a lot more effort to maintain for many languages.

read executed line to get indentation level and expand code

d482adb

mbroedl added 2 commits July 27, 2018 02:27

fix run and move down behaviour

d619c6b

continue expanding after comment

1475c72

fix end-of-file run-and-move-down behaviour

1507b97

mbroedl added 2 commits July 31, 2018 17:24

fixed bugs outlined in PR + adjusted to new hydrogen master

99a225d

add default=false setting

9cd2464

mbroedl added 2 commits August 3, 2018 18:57

fix breakpoint detection in folded lines

db79d16

add except/finally to defaults

9f463b2

simplify breakpoint search

fb86044

nikitakit requested changes Aug 7, 2018

View reviewed changes

mbroedl added 2 commits August 19, 2018 15:33

reworked to fix bugs + comments

bb39e52

remove debug messages

c489a05

extend code to top (i.e. decorators)

fdc1049

mbroedl mentioned this pull request Aug 19, 2018

Python: Running function definitions with decorators ignores the decorator nteract/hydrogen#77

Closed

kylebarron reviewed Aug 19, 2018

View reviewed changes

nikitakit approved these changes Aug 24, 2018

View reviewed changes

nikitakit merged commit c152803 into nikitakit:master Aug 24, 2018

This was referenced Sep 7, 2018

Else block always runs kylebarron/stata_kernel#150

Open

Add foldEndPattern to settings atom/language-python#274

Merged

Modify foldEndPattern in Atom settings MagicStack/MagicPython#161

Merged

read executed line to get indentation level and expand code #10

read executed line to get indentation level and expand code #10

Conversation

mbroedl commented Jul 26, 2018

kylebarron commented Jul 26, 2018

kylebarron commented Jul 26, 2018

mbroedl commented Jul 26, 2018

mbroedl commented Jul 27, 2018 • edited Loading

kylebarron commented Jul 27, 2018

mbroedl commented Jul 27, 2018

kylebarron commented Jul 27, 2018 • edited Loading

kylebarron commented Jul 27, 2018 • edited Loading

kylebarron commented Jul 27, 2018 • edited Loading

kylebarron commented Jul 27, 2018

kylebarron commented Jul 27, 2018

kylebarron commented Jul 27, 2018

nikitakit commented Jul 29, 2018

mbroedl commented Jul 29, 2018

kylebarron commented Jul 29, 2018

kylebarron commented Jul 30, 2018 • edited Loading

mbroedl commented Jul 30, 2018

kylebarron commented Jul 30, 2018

kylebarron commented Jul 31, 2018 • edited Loading

mbroedl commented Jul 31, 2018

kylebarron commented Jul 31, 2018

mbroedl commented Jul 31, 2018 • edited Loading

kylebarron commented Jul 31, 2018

mbroedl commented Jul 31, 2018

mbroedl commented Aug 1, 2018

nikitakit commented Aug 3, 2018

kylebarron commented Aug 3, 2018 • edited Loading

mbroedl commented Aug 3, 2018

kylebarron commented Aug 3, 2018

mbroedl commented Aug 3, 2018

kylebarron commented Aug 3, 2018

nikitakit left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

nikitakit commented Aug 7, 2018

mbroedl commented Aug 8, 2018 • edited Loading

kylebarron commented Aug 8, 2018

kylebarron commented Aug 9, 2018

kylebarron commented Aug 9, 2018

mbroedl commented Aug 19, 2018

mbroedl commented Aug 19, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

nikitakit commented Aug 24, 2018

mbroedl commented Aug 25, 2018 • edited Loading

mbroedl commented Jul 27, 2018 •

edited

Loading

kylebarron commented Jul 27, 2018 •

edited

Loading

kylebarron commented Jul 27, 2018 •

edited

Loading

kylebarron commented Jul 27, 2018 •

edited

Loading

kylebarron commented Jul 30, 2018 •

edited

Loading

kylebarron commented Jul 31, 2018 •

edited

Loading

mbroedl commented Jul 31, 2018 •

edited

Loading

kylebarron commented Aug 3, 2018 •

edited

Loading

mbroedl commented Aug 8, 2018 •

edited

Loading

mbroedl commented Aug 25, 2018 •

edited

Loading