Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Odd output in Shiny App #35

Closed
aguynamedryan opened this issue Oct 13, 2021 · 3 comments · Fixed by #37
Closed

Odd output in Shiny App #35

aguynamedryan opened this issue Oct 13, 2021 · 3 comments · Fixed by #37

Comments

@aguynamedryan
Copy link

I love this regex tester! Thanks for making and sharing it.

When I was working on a file-extension stripper, I saw some odd output. Check it out here: https://spannbaueradam.shinyapps.io/r_regex_tester/_w_f725e36c/?pattern=%5C%5C.(txt%7Cgz)&test_str=some/dumb/path.txt.gz%0Asome/dumb/path2%0Asome/dumb/path.001.txt.gz

Thanks again!

@AdamSpannbauer
Copy link
Owner

AdamSpannbauer commented Oct 13, 2021

For copy paste when I get in a place to test:

matching_pattern = '\\.(txt|gz)'
test_str = 'some/dumb/path.txt.gz\nsome/dumb/path2\nsome/dumb/path.001.txt.gz'


UPDATE:

A more minimal example to reproduce what seems to be the same buggy behavior:

matching_pattern = 'a(b|c)'
test_str = 'abacab'

@AdamSpannbauer
Copy link
Owner

AdamSpannbauer commented Oct 13, 2021

Oh and thank you for the kind words! Looking into this and I'll either make progress in this branch or I'll report back here.


EDIT: I also want to be upfront that I personally only have motivation on this project for general upkeep. If this turns out to be something sour in the foundation then it might just stay as a bug.... 🤷‍♂️

@AdamSpannbauer
Copy link
Owner

@aguynamedryan I think that your example should be functioning now in the live app.

thanks for reporting and let me know if you run into any further issues!

your test link:
https://spannbaueradam.shinyapps.io/r_regex_tester/_w_f725e36c/?pattern=%5C%5C.(txt%7Cgz)&test_str=some/dumb/path.txt.gz%0Asome/dumb/path2%0Asome/dumb/path.001.txt.gz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants