Community

    • Login
    • Search
    • Recent
    • Tags
    • Popular
    • Users
    • Groups
    • Search

    Regex] Rounding numbers python script does not run properly

    Help wanted · · · – – – · · ·
    4
    28
    1254
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • unfassbarstephan
      unfassbarstephan last edited by

      Hi,

      unfortunately i can’t comment on the thread https://notepad-plus-plus.org/community/topic/15100/regex-rounding-numbers , so i’m asking my question here.

      First of all I want to thank @PeterJones for the nice Python script for rounding numbers in Notepad++.

      However, the script does not run correctly for unknown reasons. The following series of numbers (coordinates):

      5494577.00001803319901228 5650290.99997399840503931, 5494577.00001803319901228 5650288.99997399933636189, 5494575.00001802947372198 5650288.99997399933636189, 5494575.00001802947372198 5650286.99997399840503931, 5494573.00001803319901228 5650286.99997399840503931

      is converted to:

      5494577.0000 5650290.99997399840503931, 5494577.0000 5650288.99997399933636189, 5494575.0000 5650288.99997399933636189, 5494575.0000 5650286.99997399840503931, 5494573.0000 5650286.99997399840503931

      Unfortunately I could not fix the error in the script. What is exactly the problem?

      Thank you in advance!

      Sincerly,

      Stephan

      Ekopalypse 1 Reply Last reply Reply Quote 0
      • Ekopalypse
        Ekopalypse @unfassbarstephan last edited by

        @unfassbarstephan

        the issue might be the recalculation of the next start point.
        May I propose a different approach?

        editor.rereplace("-*\d+\.\d{4,}", lambda m: round(float(m.group(0)) , 3))
        

        Just this line, nothing else.
        rereplace takes care that the next start position
        is calculated according to the replaced text.

        Alan Kilborn 1 Reply Last reply Reply Quote 3
        • Alan Kilborn
          Alan Kilborn @Ekopalypse last edited by Alan Kilborn

          @Ekopalypse

          When I try the one-liner on the user’s data, I get this as a result (I don’t think it is the intent?):

          5494577.0 5650291.0, 5494577.0 5650289.0, 5494575.0 5650289.0, 5494575.0 5650287.0, 5494573.0 5650287.0

          But maybe it is…it would have been nice if the OP had said exactly what they were trying to achieve!

          Ekopalypse 1 Reply Last reply Reply Quote 2
          • Ekopalypse
            Ekopalypse @Alan Kilborn last edited by Ekopalypse

            @Alan-Kilborn

            if it is really needed to show 3 digit zeros then a slight modification will do it. :-)

            editor1.rereplace("-*\d+\.\d{4,}", lambda m: '{:0.3f}'.format(round(float(m.group(0)), 3)))
            
            Alan Kilborn 1 Reply Last reply Reply Quote 2
            • Alan Kilborn
              Alan Kilborn @Ekopalypse last edited by Alan Kilborn

              @Ekopalypse

              Ah. The .0 or .000 seems like extra noise to me, but maybe it is required for OP’s need (often wish people would explain their need better).

              But, very nice one-liner Eko! It’s a clip-and-saver.

              A switch from editor to editor1 ? Methinks you might be too focused on which view is active. I pretty much ignore editor1 and editor2 in my programming unless I really need them. :)

              Ekopalypse 1 Reply Last reply Reply Quote 2
              • Ekopalypse
                Ekopalypse @Alan Kilborn last edited by Ekopalypse

                @Alan-Kilborn

                :-D I constantly keep forgetting this, yes, it should be editor, thx for the head up.

                editor.rereplace("-*\d+\.\d{4,}", lambda m: '{:0.3f}'.format(round(float(m.group(0)), 3)))
                

                Yes, to me it looks awkward too but as you said, it might be in the interested of the OP.

                1 Reply Last reply Reply Quote 2
                • Alan Kilborn
                  Alan Kilborn last edited by

                  Just for fun (I say that a lot), here’s a version that does a prompted replace with preview (wouldn’t it be great if N++ itself offered a preview to its [regex] replacement?). Strange but I’ve never coded this kind of thing before for N++; one more thing to stash in the bag-of-tricks…

                  def main():
                  
                      while True:
                          match_list = []
                          editor.research("-*\d+\.\d{4,}", lambda m: match_list.append(m.span(0)), 0, editor.getCurrentPos(), editor.getTextLength(), 1)
                          if len(match_list) == 0: return
                          editor.setSelection(match_list[0][1], match_list[0][0]); editor.scrollCaret()
                          repl_text = '{:0.3f}'.format(round(float(editor.getSelText()), 3))
                          result = notepad.messageBox('Replace selected text with\r\n\r\n{}\r\n\r\n???'.format(repl_text), '', MESSAGEBOXFLAGS.YESNOCANCEL)
                          if result == MESSAGEBOXFLAGS.RESULTCANCEL: return
                          if result == MESSAGEBOXFLAGS.RESULTYES: editor.replaceSel(repl_text)
                  
                  main()
                  

                  Imgur

                  1 Reply Last reply Reply Quote 3
                  • Alan Kilborn
                    Alan Kilborn last edited by

                    …still having fun with this…so much can be done to make interactive replace much better than what N++ can do; sample:

                    Imgur

                    1 Reply Last reply Reply Quote 3
                    • unfassbarstephan
                      unfassbarstephan last edited by

                      Wow, thank you dir your help! I will try this approach.

                      1 Reply Last reply Reply Quote 0
                      • unfassbarstephan
                        unfassbarstephan last edited by

                        @Alan-Kilborn said:

                        …still having fun with this…so much can be done to make interactive replace much better than what N++ can do; sample:

                        The one-line-solution works like a charm.

                        Would you also publish your code for the interactive replace? That would be great.

                        Short question: What does “???” mean?

                        Ekopalypse Alan Kilborn 2 Replies Last reply Reply Quote 0
                        • Ekopalypse
                          Ekopalypse @unfassbarstephan last edited by

                          @unfassbarstephan

                          Short question: What does “???” mean?

                          This is Alan’s wasteful use of the question mark,
                          for which it is also used here, to ask a question. :-D

                          1 Reply Last reply Reply Quote 3
                          • Alan Kilborn
                            Alan Kilborn @unfassbarstephan last edited by

                            @unfassbarstephan said:

                            Would you also publish your code for the interactive replace?

                            I did…for your specific case. I’m still working on a generic version; may never finish it… :(

                            @Ekopalypse said:

                            Alan’s wasteful use of the question mark

                            Wasteful? It doesn’t seem right without it; I did consider it this way as well, maybe better?:

                            Replace selected text with? :

                            1 Reply Last reply Reply Quote 2
                            • PeterJones
                              PeterJones last edited by

                              @Alan-Kilborn said:

                              Replace selected text with? :

                              Maybe Would you like to replace the selected text with the following text?

                              Alan Kilborn 1 Reply Last reply Reply Quote 1
                              • Alan Kilborn
                                Alan Kilborn @PeterJones last edited by

                                @PeterJones

                                Well, Pythonscript sizes the dialog box according to contents, so I was trying to keep it as narrow and short as possible to avoid covering too much of the Notepad++ main window. But…I’ve already added more info to make it taller, so why not wider as well? :)

                                1 Reply Last reply Reply Quote 1
                                • Alan Kilborn
                                  Alan Kilborn last edited by

                                  Would you also publish your code for the interactive replace?

                                  I’m still working on a generic version; may never finish it

                                  Follow-up:

                                  There may not be a reasonable way to script this (the “generic version”); here’s why: There doesn’t seem to be a way to do a “boost-ish” replacement option on a Python string. It’s only possible to do it on in-place text.

                                  What I want to do in the code is pull a copy of the matched text into a Python string and then do a Notepad++ -like replacement operation on that. Sure I can do a Python “re” replacement on it, but that isn’t always going to be equivalent.

                                  Digging deeper, I see there is an open issue on this kind of thing in the Pythonscript code pages, here. But it hasn’t been acted on so it isn’t a reality.

                                  If anyone has thoughts on this, I’m all ears. @Ekopalypse ?

                                  Ekopalypse 1 Reply Last reply Reply Quote 0
                                  • Alan Kilborn
                                    Alan Kilborn last edited by

                                    Ugh. Clearly I meant this Pythonscript issue: https://github.com/bruderstein/PythonScript/issues/101 in the previous post, but it is too late for an edit. :(

                                    1 Reply Last reply Reply Quote 1
                                    • Ekopalypse
                                      Ekopalypse @Alan Kilborn last edited by

                                      @Alan-Kilborn

                                      one idea might be to search for matches and replace only those which are
                                      in the same range as the selected text. Something like

                                      if start_pos_selected text <= match_start and end_pos_selected_text >= match_end:
                                          replaceSelText
                                      

                                      Or did I misunderstood your goal? :-(

                                      Alan Kilborn 1 Reply Last reply Reply Quote 0
                                      • Alan Kilborn
                                        Alan Kilborn @Ekopalypse last edited by

                                        @Ekopalypse said:

                                        did I misunderstood your goal?

                                        Maybe that part. :)

                                        To be able to offer a preview of the yet-to-be-done replacement, I need to know what that replacement will be. I can’t do that without actually doing the replacement. Catch-22. I could do it and then undo it, I suppose, but that gets messy. Mostly it should be OK, but sometimes with regexes Boost can differ from Python’s re.

                                        Ekopalypse 1 Reply Last reply Reply Quote 1
                                        • Ekopalypse
                                          Ekopalypse @Alan Kilborn last edited by

                                          @Alan-Kilborn

                                          what about using a hidden scintilla instance?

                                          Alan Kilborn 2 Replies Last reply Reply Quote 0
                                          • Alan Kilborn
                                            Alan Kilborn @Ekopalypse last edited by

                                            @Ekopalypse said:

                                            what about using a hidden scintilla instance?

                                            That also sounds “messy” (or “involved”) but if you have some snippet to show me it is not so bad…?

                                            Ekopalypse 2 Replies Last reply Reply Quote 0
                                            • First post
                                              Last post
                                            Copyright © 2014 NodeBB Forums | Contributors