bad escape \s in Drain.py #84

moesjo · 2022-08-12T09:02:46Z

When running the demo script for drain to following error occures:
re.error: bad escape \s at position 2
Changing line 339 in Drain.py from
template_regex = re.sub(r'\\ +', r'\s+', template_regex)
to
template_regex = re.sub(r'\\ +', r'\\s+', template_regex)
fixed the problem for me.

The text was updated successfully, but these errors were encountered:

moesjo · 2022-08-16T06:36:10Z

Already included in several pull requests.

shoaib-intro · 2022-09-07T14:07:15Z

works with drain same issue with spell any idea?

moesjo · 2022-09-07T14:16:44Z

I think for spell it's pretty much the same, there should be a line similar to the one shown above and when you fill in the \ it should work.
Maybe you should also consider using one of the forks of this project, there are some which already include the fixes for these issues (see pull-requests).

shoaib-intro · 2022-09-07T17:13:44Z

I think for spell it's pretty much the same, there should be a line similar to the one shown above and when you fill in the \ it should work.
Maybe you should also consider using one of the forks of this project, there are some which already include the fixes for these issues (see pull-requests).

Thanks for reply but I even tried to change drain parameter code with spell does not works....even tried to change concerning line as well....spell keeps working for days but does not bring any output

moesjo · 2022-09-08T05:10:40Z

So i looked it up and from the syntax point of view you only have to change line 312 in Spell.py from

    splitter = re.sub(' +', '\s+', splitters[k])

to

    splitter = re.sub(' +', '\\\s+', splitters[k])

shoaib-intro · 2022-09-08T06:27:52Z

So i looked it up and from the syntax point of view you only have to change line 312 in Spell.py from
    splitter = re.sub(' +', '\s+', splitters[k])
to
    splitter = re.sub(' +', '\\\s+', splitters[k])

I have already tried this one, this works but only parameters extraction does not work however it works some other part of dataset but on some dataset having same size does not extract parameters.

moesjo · 2022-09-08T06:30:21Z

Okay, unfortunately I can't help you with this problem but i hope you'll find a fix for that!

* Fix PRs: #86, #85, #83, #80, #78, #65, #57, #53, #52, #51, #49 * Fix issues: #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82

* Fix PRs: fix #86, fix #85, #83, #80, #78, #65, #57, #53, #52, #51, #49 * Fix issues: #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82

* Fix PRs: merge #86, merge #85, merge #83, merge #80, merge #78, merge #65, merge #57, merge #53, merge #52, merge #51, merge #49 * Fix issues: #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82

* Close PRs: close #86, #85, #83, #80, #78, #65, #57, #53, #52, #51, #49 * Fix issues: fix #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82

* Close PRs: close #86, #85, #83, #80, #78, #65, #57, #53, #52, #51, #49 * Fix issues: fix #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82 * Release pypi package wheel

* Close PRs: close #86, #85, #83, #80, #78, #65, #57, #53, #52, #51, #49 * Fix issues: fix #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82 * Release package wheel to pypi

moesjo closed this as completed Aug 16, 2022

xpai pushed a commit that referenced this issue Sep 5, 2023

Update logparser v1.0.0 to support Python 3 compatibility

a22c324

* Fix PRs: #86, #85, #83, #80, #78, #65, #57, #53, #52, #51, #49 * Fix issues: #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82

xpai pushed a commit that referenced this issue Sep 5, 2023

Update logparser v1.0.0 to support Python 3 compatibility

641b5d3

* Fix PRs: #86, #85, #83, #80, #78, #65, #57, #53, #52, #51, #49 * Fix issues: #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82

xpai pushed a commit that referenced this issue Sep 5, 2023

Update logparser v1.0.0 to support Python 3 compatibility

be4fa3e

* Fix PRs: fix #86, fix #85, #83, #80, #78, #65, #57, #53, #52, #51, #49 * Fix issues: #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82

xpai pushed a commit that referenced this issue Sep 5, 2023

Update logparser v1.0.0 to support Python 3 compatibility

3bfb46f

* Close PRs: close #86, #85, #83, #80, #78, #65, #57, #53, #52, #51, #49 * Fix issues: fix #98, #79, #72, #84, #75, #71, #69, #56, #50, #45, #16, #82

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

bad escape \s in Drain.py #84

bad escape \s in Drain.py #84

moesjo commented Aug 12, 2022

moesjo commented Aug 16, 2022

shoaib-intro commented Sep 7, 2022

moesjo commented Sep 7, 2022

shoaib-intro commented Sep 7, 2022

moesjo commented Sep 8, 2022

shoaib-intro commented Sep 8, 2022

moesjo commented Sep 8, 2022

bad escape \s in Drain.py #84

bad escape \s in Drain.py #84

Comments

moesjo commented Aug 12, 2022

moesjo commented Aug 16, 2022

shoaib-intro commented Sep 7, 2022

moesjo commented Sep 7, 2022

shoaib-intro commented Sep 7, 2022

moesjo commented Sep 8, 2022

shoaib-intro commented Sep 8, 2022

moesjo commented Sep 8, 2022