Add ability to select path segments using Python `re` regexprs by AlainLich · Pull Request #146 · dpath-maintainers/dpath-python

AlainLich · 2021-06-01T15:18:26Z

Hi,
In case there is an interest, this allows to use python re regular expressions, like in following examples:

import dpath
dpath.options.DPATH_ACCEPT_RE_REGEXP = True
js = loadJson()
selPath = 'Config/{(Env|Cmd)}'
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = [ re.compile('(Config|Graph)') , re.compile('(Env|Cmd|Data)') ]
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = '{(Config|Graph)}/{(Env|Cmd|Data)}'
x = dpath.util.search(js.lod, selPath)
print(x)

I found this useful navigating JSON.

Remarks:

. This runs under Python3 as shown on https://github.com/AlainLich/dpath-python/actions/runs/896380770 ;
. the trickwith dpath.options.DPATH_ACCEPT_RE_REGEXP, means the option has to be enabled make this compatible
with earlier versions and will not break if users have path segments in curly brackets.

WARNING: this will not work under Python2.7, since it requires Python3 features like 'f' strings; however I might
do a compatible version if there is interest.

Changes (most in random tests):

quite straightforward changes in dpath/segments.py
added DPATH_ACCEPT_RE_REGEXP in options as explained above
in dpath/util.pyre.compile extended regexprs once when they are found
added `test/test_path_ext.py ̀ to test this extension
made all tests using hypothesis run under unittest, in particular this permits to select and reproduce a single test function
debugged a rare situation in the random test generation, reproduced in file issues/err_walk.py where a path segment
gets generated which is interpreted by the bash globbing function resulting in a test failure (e.g. ̀seg[0]) which is globbed into something accepting seg0` but not its literal version with the brackets

…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Still needed: test and documentation of added functionality

Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

…ing to diagnostics Cleaned up formatting

…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

…H_ACCEPT_RE_REGEXP does not break things

…t discovery in nose2)

…if Flake still complains

moomoohk · 2022-12-04T13:40:55Z

@AlainLich Hey, are you still interested in working on this PR?

AlainLich · 2022-12-04T17:37:56Z

@moomoohk Hello, I have not touched this for a year and a half, could be interested to work on this PR, likely at a slow pace, since this would facilitate my own use (loading via pip ).

However, only on Python3... not interested in Python2 compatibility and such.

moomoohk · 2022-12-04T17:59:08Z

That's great to hear :)
Slow pace is alright by me, as well as Python 3+. On that note, I've decided to drop Python 2 support entirely a while ago!
Minimum supported version is 3.7 as of now.
I've refactored the codebase quite a bit in the last year and a half. Mainly modernizing, upgrading to Python 3 and blowing some dust off...
If you have any questions feel free to reach out here or on Gitter.

AlainLich · 2022-12-04T19:55:20Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

moomoohk · 2022-12-04T19:57:16Z

Yes indeed. Please start by merging master into your branch and take it from there.

AlainLich · 2022-12-13T11:18:12Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

AlainLich · 2022-12-13T11:20:19Z

Hi, I have entered a new PR #177 with revised code integrated in the current commit for branch master.

This PR should be closed.

AlainLich · 2021-06-01T15:18:26Z

Hi,
In case there is an interest, this allows to use python re regular expressions, like in following examples:

import dpath
dpath.options.DPATH_ACCEPT_RE_REGEXP = True
js = loadJson()
selPath = 'Config/{(Env|Cmd)}'
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = [ re.compile('(Config|Graph)') , re.compile('(Env|Cmd|Data)') ]
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = '{(Config|Graph)}/{(Env|Cmd|Data)}'
x = dpath.util.search(js.lod, selPath)
print(x)

I found this useful navigating JSON.

Remarks:

. This runs under Python3 as shown on https://github.com/AlainLich/dpath-python/actions/runs/896380770 ;
. the trickwith dpath.options.DPATH_ACCEPT_RE_REGEXP, means the option has to be enabled make this compatible
with earlier versions and will not break if users have path segments in curly brackets.

WARNING: this will not work under Python2.7, since it requires Python3 features like 'f' strings; however I might
do a compatible version if there is interest.

Changes (most in random tests):

quite straightforward changes in dpath/segments.py
added DPATH_ACCEPT_RE_REGEXP in options as explained above
in dpath/util.pyre.compile extended regexprs once when they are found
added `test/test_path_ext.py ̀ to test this extension
made all tests using hypothesis run under unittest, in particular this permits to select and reproduce a single test function
debugged a rare situation in the random test generation, reproduced in file issues/err_walk.py where a path segment
gets generated which is interpreted by the bash globbing function resulting in a test failure (e.g. ̀seg[0]) which is globbed into something accepting seg0` but not its literal version with the brackets

…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Still needed: test and documentation of added functionality

Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

…ing to diagnostics Cleaned up formatting

…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

…H_ACCEPT_RE_REGEXP does not break things

…t discovery in nose2)

…if Flake still complains

moomoohk · 2022-12-04T13:40:55Z

@AlainLich Hey, are you still interested in working on this PR?

AlainLich · 2022-12-04T17:37:56Z

@moomoohk Hello, I have not touched this for a year and a half, could be interested to work on this PR, likely at a slow pace, since this would facilitate my own use (loading via pip ).

However, only on Python3... not interested in Python2 compatibility and such.

moomoohk · 2022-12-04T17:59:08Z

That's great to hear :)
Slow pace is alright by me, as well as Python 3+. On that note, I've decided to drop Python 2 support entirely a while ago!
Minimum supported version is 3.7 as of now.
I've refactored the codebase quite a bit in the last year and a half. Mainly modernizing, upgrading to Python 3 and blowing some dust off...
If you have any questions feel free to reach out here or on Gitter.

AlainLich · 2022-12-04T19:55:20Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

moomoohk · 2022-12-04T19:57:16Z

Yes indeed. Please start by merging master into your branch and take it from there.

AlainLich · 2022-12-13T11:18:12Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

AlainLich · 2022-12-13T11:20:19Z

Hi, I have entered a new PR #177 with revised code integrated in the current commit for branch master.

This PR should be closed.

AlainLich · 2021-06-01T15:18:26Z

Hi,
In case there is an interest, this allows to use python re regular expressions, like in following examples:

import dpath
dpath.options.DPATH_ACCEPT_RE_REGEXP = True
js = loadJson()
selPath = 'Config/{(Env|Cmd)}'
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = [ re.compile('(Config|Graph)') , re.compile('(Env|Cmd|Data)') ]
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = '{(Config|Graph)}/{(Env|Cmd|Data)}'
x = dpath.util.search(js.lod, selPath)
print(x)

I found this useful navigating JSON.

Remarks:

. This runs under Python3 as shown on https://github.com/AlainLich/dpath-python/actions/runs/896380770 ;
. the trickwith dpath.options.DPATH_ACCEPT_RE_REGEXP, means the option has to be enabled make this compatible
with earlier versions and will not break if users have path segments in curly brackets.

WARNING: this will not work under Python2.7, since it requires Python3 features like 'f' strings; however I might
do a compatible version if there is interest.

Changes (most in random tests):

quite straightforward changes in dpath/segments.py
added DPATH_ACCEPT_RE_REGEXP in options as explained above
in dpath/util.pyre.compile extended regexprs once when they are found
added `test/test_path_ext.py ̀ to test this extension
made all tests using hypothesis run under unittest, in particular this permits to select and reproduce a single test function
debugged a rare situation in the random test generation, reproduced in file issues/err_walk.py where a path segment
gets generated which is interpreted by the bash globbing function resulting in a test failure (e.g. ̀seg[0]) which is globbed into something accepting seg0` but not its literal version with the brackets

…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Still needed: test and documentation of added functionality

Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

…ing to diagnostics Cleaned up formatting

…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

…H_ACCEPT_RE_REGEXP does not break things

…t discovery in nose2)

…if Flake still complains

moomoohk · 2022-12-04T13:40:55Z

@AlainLich Hey, are you still interested in working on this PR?

AlainLich · 2022-12-04T17:37:56Z

@moomoohk Hello, I have not touched this for a year and a half, could be interested to work on this PR, likely at a slow pace, since this would facilitate my own use (loading via pip ).

However, only on Python3... not interested in Python2 compatibility and such.

moomoohk · 2022-12-04T17:59:08Z

That's great to hear :)
Slow pace is alright by me, as well as Python 3+. On that note, I've decided to drop Python 2 support entirely a while ago!
Minimum supported version is 3.7 as of now.
I've refactored the codebase quite a bit in the last year and a half. Mainly modernizing, upgrading to Python 3 and blowing some dust off...
If you have any questions feel free to reach out here or on Gitter.

AlainLich · 2022-12-04T19:55:20Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

moomoohk · 2022-12-04T19:57:16Z

Yes indeed. Please start by merging master into your branch and take it from there.

AlainLich · 2022-12-13T11:18:12Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

AlainLich · 2022-12-13T11:20:19Z

Hi, I have entered a new PR #177 with revised code integrated in the current commit for branch master.

This PR should be closed.

AlainLich · 2021-06-01T15:18:26Z

Hi,
In case there is an interest, this allows to use python re regular expressions, like in following examples:

import dpath
dpath.options.DPATH_ACCEPT_RE_REGEXP = True
js = loadJson()
selPath = 'Config/{(Env|Cmd)}'
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = [ re.compile('(Config|Graph)') , re.compile('(Env|Cmd|Data)') ]
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = '{(Config|Graph)}/{(Env|Cmd|Data)}'
x = dpath.util.search(js.lod, selPath)
print(x)

I found this useful navigating JSON.

Remarks:

. This runs under Python3 as shown on https://github.com/AlainLich/dpath-python/actions/runs/896380770 ;
. the trickwith dpath.options.DPATH_ACCEPT_RE_REGEXP, means the option has to be enabled make this compatible
with earlier versions and will not break if users have path segments in curly brackets.

WARNING: this will not work under Python2.7, since it requires Python3 features like 'f' strings; however I might
do a compatible version if there is interest.

Changes (most in random tests):

quite straightforward changes in dpath/segments.py
added DPATH_ACCEPT_RE_REGEXP in options as explained above
in dpath/util.pyre.compile extended regexprs once when they are found
added `test/test_path_ext.py ̀ to test this extension
made all tests using hypothesis run under unittest, in particular this permits to select and reproduce a single test function
debugged a rare situation in the random test generation, reproduced in file issues/err_walk.py where a path segment
gets generated which is interpreted by the bash globbing function resulting in a test failure (e.g. ̀seg[0]) which is globbed into something accepting seg0` but not its literal version with the brackets

…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Still needed: test and documentation of added functionality

Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

…ing to diagnostics Cleaned up formatting

…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

…H_ACCEPT_RE_REGEXP does not break things

…t discovery in nose2)

…if Flake still complains

moomoohk · 2022-12-04T13:40:55Z

@AlainLich Hey, are you still interested in working on this PR?

AlainLich · 2022-12-04T17:37:56Z

@moomoohk Hello, I have not touched this for a year and a half, could be interested to work on this PR, likely at a slow pace, since this would facilitate my own use (loading via pip ).

However, only on Python3... not interested in Python2 compatibility and such.

moomoohk · 2022-12-04T17:59:08Z

That's great to hear :)
Slow pace is alright by me, as well as Python 3+. On that note, I've decided to drop Python 2 support entirely a while ago!
Minimum supported version is 3.7 as of now.
I've refactored the codebase quite a bit in the last year and a half. Mainly modernizing, upgrading to Python 3 and blowing some dust off...
If you have any questions feel free to reach out here or on Gitter.

AlainLich · 2022-12-04T19:55:20Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

moomoohk · 2022-12-04T19:57:16Z

Yes indeed. Please start by merging master into your branch and take it from there.

AlainLich · 2022-12-13T11:18:12Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

AlainLich · 2022-12-13T11:20:19Z

Hi, I have entered a new PR #177 with revised code integrated in the current commit for branch master.

This PR should be closed.

AlainLich · 2021-06-01T15:18:26Z

Hi,
In case there is an interest, this allows to use python re regular expressions, like in following examples:

import dpath
dpath.options.DPATH_ACCEPT_RE_REGEXP = True
js = loadJson()
selPath = 'Config/{(Env|Cmd)}'
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = [ re.compile('(Config|Graph)') , re.compile('(Env|Cmd|Data)') ]
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = '{(Config|Graph)}/{(Env|Cmd|Data)}'
x = dpath.util.search(js.lod, selPath)
print(x)

I found this useful navigating JSON.

Remarks:

. This runs under Python3 as shown on https://github.com/AlainLich/dpath-python/actions/runs/896380770 ;
. the trickwith dpath.options.DPATH_ACCEPT_RE_REGEXP, means the option has to be enabled make this compatible
with earlier versions and will not break if users have path segments in curly brackets.

WARNING: this will not work under Python2.7, since it requires Python3 features like 'f' strings; however I might
do a compatible version if there is interest.

Changes (most in random tests):

quite straightforward changes in dpath/segments.py
added DPATH_ACCEPT_RE_REGEXP in options as explained above
in dpath/util.pyre.compile extended regexprs once when they are found
added `test/test_path_ext.py ̀ to test this extension
made all tests using hypothesis run under unittest, in particular this permits to select and reproduce a single test function
debugged a rare situation in the random test generation, reproduced in file issues/err_walk.py where a path segment
gets generated which is interpreted by the bash globbing function resulting in a test failure (e.g. ̀seg[0]) which is globbed into something accepting seg0` but not its literal version with the brackets

…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Still needed: test and documentation of added functionality

Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

…ing to diagnostics Cleaned up formatting

…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

…H_ACCEPT_RE_REGEXP does not break things

…t discovery in nose2)

…if Flake still complains

moomoohk · 2022-12-04T13:40:55Z

@AlainLich Hey, are you still interested in working on this PR?

AlainLich · 2022-12-04T17:37:56Z

@moomoohk Hello, I have not touched this for a year and a half, could be interested to work on this PR, likely at a slow pace, since this would facilitate my own use (loading via pip ).

However, only on Python3... not interested in Python2 compatibility and such.

moomoohk · 2022-12-04T17:59:08Z

That's great to hear :)
Slow pace is alright by me, as well as Python 3+. On that note, I've decided to drop Python 2 support entirely a while ago!
Minimum supported version is 3.7 as of now.
I've refactored the codebase quite a bit in the last year and a half. Mainly modernizing, upgrading to Python 3 and blowing some dust off...
If you have any questions feel free to reach out here or on Gitter.

AlainLich · 2022-12-04T19:55:20Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

moomoohk · 2022-12-04T19:57:16Z

Yes indeed. Please start by merging master into your branch and take it from there.

AlainLich · 2022-12-13T11:18:12Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

AlainLich · 2022-12-13T11:20:19Z

Hi, I have entered a new PR #177 with revised code integrated in the current commit for branch master.

This PR should be closed.

AlainLich · 2021-06-01T15:18:26Z

Hi,
In case there is an interest, this allows to use python re regular expressions, like in following examples:

import dpath
dpath.options.DPATH_ACCEPT_RE_REGEXP = True
js = loadJson()
selPath = 'Config/{(Env|Cmd)}'
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = [ re.compile('(Config|Graph)') , re.compile('(Env|Cmd|Data)') ]
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = '{(Config|Graph)}/{(Env|Cmd|Data)}'
x = dpath.util.search(js.lod, selPath)
print(x)

I found this useful navigating JSON.

Remarks:

. This runs under Python3 as shown on https://github.com/AlainLich/dpath-python/actions/runs/896380770 ;
. the trickwith dpath.options.DPATH_ACCEPT_RE_REGEXP, means the option has to be enabled make this compatible
with earlier versions and will not break if users have path segments in curly brackets.

WARNING: this will not work under Python2.7, since it requires Python3 features like 'f' strings; however I might
do a compatible version if there is interest.

Changes (most in random tests):

quite straightforward changes in dpath/segments.py
added DPATH_ACCEPT_RE_REGEXP in options as explained above
in dpath/util.pyre.compile extended regexprs once when they are found
added `test/test_path_ext.py ̀ to test this extension
made all tests using hypothesis run under unittest, in particular this permits to select and reproduce a single test function
debugged a rare situation in the random test generation, reproduced in file issues/err_walk.py where a path segment
gets generated which is interpreted by the bash globbing function resulting in a test failure (e.g. ̀seg[0]) which is globbed into something accepting seg0` but not its literal version with the brackets

…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Still needed: test and documentation of added functionality

Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

…ing to diagnostics Cleaned up formatting

…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

…H_ACCEPT_RE_REGEXP does not break things

…t discovery in nose2)

…if Flake still complains

moomoohk · 2022-12-04T13:40:55Z

@AlainLich Hey, are you still interested in working on this PR?

AlainLich · 2022-12-04T17:37:56Z

@moomoohk Hello, I have not touched this for a year and a half, could be interested to work on this PR, likely at a slow pace, since this would facilitate my own use (loading via pip ).

However, only on Python3... not interested in Python2 compatibility and such.

moomoohk · 2022-12-04T17:59:08Z

That's great to hear :)
Slow pace is alright by me, as well as Python 3+. On that note, I've decided to drop Python 2 support entirely a while ago!
Minimum supported version is 3.7 as of now.
I've refactored the codebase quite a bit in the last year and a half. Mainly modernizing, upgrading to Python 3 and blowing some dust off...
If you have any questions feel free to reach out here or on Gitter.

AlainLich · 2022-12-04T19:55:20Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

moomoohk · 2022-12-04T19:57:16Z

Yes indeed. Please start by merging master into your branch and take it from there.

AlainLich · 2022-12-13T11:18:12Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

AlainLich · 2022-12-13T11:20:19Z

Hi, I have entered a new PR #177 with revised code integrated in the current commit for branch master.

This PR should be closed.

AlainLich · 2021-06-01T15:18:26Z

Hi,
In case there is an interest, this allows to use python re regular expressions, like in following examples:

import dpath
dpath.options.DPATH_ACCEPT_RE_REGEXP = True
js = loadJson()
selPath = 'Config/{(Env|Cmd)}'
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = [ re.compile('(Config|Graph)') , re.compile('(Env|Cmd|Data)') ]
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = '{(Config|Graph)}/{(Env|Cmd|Data)}'
x = dpath.util.search(js.lod, selPath)
print(x)

I found this useful navigating JSON.

Remarks:

. This runs under Python3 as shown on https://github.com/AlainLich/dpath-python/actions/runs/896380770 ;
. the trickwith dpath.options.DPATH_ACCEPT_RE_REGEXP, means the option has to be enabled make this compatible
with earlier versions and will not break if users have path segments in curly brackets.

WARNING: this will not work under Python2.7, since it requires Python3 features like 'f' strings; however I might
do a compatible version if there is interest.

Changes (most in random tests):

quite straightforward changes in dpath/segments.py
added DPATH_ACCEPT_RE_REGEXP in options as explained above
in dpath/util.pyre.compile extended regexprs once when they are found
added `test/test_path_ext.py ̀ to test this extension
made all tests using hypothesis run under unittest, in particular this permits to select and reproduce a single test function
debugged a rare situation in the random test generation, reproduced in file issues/err_walk.py where a path segment
gets generated which is interpreted by the bash globbing function resulting in a test failure (e.g. ̀seg[0]) which is globbed into something accepting seg0` but not its literal version with the brackets

…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Still needed: test and documentation of added functionality

Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

…ing to diagnostics Cleaned up formatting

…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

…H_ACCEPT_RE_REGEXP does not break things

…t discovery in nose2)

…if Flake still complains

moomoohk · 2022-12-04T13:40:55Z

@AlainLich Hey, are you still interested in working on this PR?

AlainLich · 2022-12-04T17:37:56Z

@moomoohk Hello, I have not touched this for a year and a half, could be interested to work on this PR, likely at a slow pace, since this would facilitate my own use (loading via pip ).

However, only on Python3... not interested in Python2 compatibility and such.

moomoohk · 2022-12-04T17:59:08Z

That's great to hear :)
Slow pace is alright by me, as well as Python 3+. On that note, I've decided to drop Python 2 support entirely a while ago!
Minimum supported version is 3.7 as of now.
I've refactored the codebase quite a bit in the last year and a half. Mainly modernizing, upgrading to Python 3 and blowing some dust off...
If you have any questions feel free to reach out here or on Gitter.

AlainLich · 2022-12-04T19:55:20Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

moomoohk · 2022-12-04T19:57:16Z

Yes indeed. Please start by merging master into your branch and take it from there.

AlainLich · 2022-12-13T11:18:12Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

AlainLich · 2022-12-13T11:20:19Z

Hi, I have entered a new PR #177 with revised code integrated in the current commit for branch master.

This PR should be closed.

AlainLich · 2021-06-01T15:18:26Z

Hi,
In case there is an interest, this allows to use python re regular expressions, like in following examples:

import dpath
dpath.options.DPATH_ACCEPT_RE_REGEXP = True
js = loadJson()
selPath = 'Config/{(Env|Cmd)}'
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = [ re.compile('(Config|Graph)') , re.compile('(Env|Cmd|Data)') ]
x = dpath.util.search(js.lod, selPath)
print(x)
selPath = '{(Config|Graph)}/{(Env|Cmd|Data)}'
x = dpath.util.search(js.lod, selPath)
print(x)

I found this useful navigating JSON.

Remarks:

. This runs under Python3 as shown on https://github.com/AlainLich/dpath-python/actions/runs/896380770 ;
. the trickwith dpath.options.DPATH_ACCEPT_RE_REGEXP, means the option has to be enabled make this compatible
with earlier versions and will not break if users have path segments in curly brackets.

WARNING: this will not work under Python2.7, since it requires Python3 features like 'f' strings; however I might
do a compatible version if there is interest.

Changes (most in random tests):

quite straightforward changes in dpath/segments.py
added DPATH_ACCEPT_RE_REGEXP in options as explained above
in dpath/util.pyre.compile extended regexprs once when they are found
added `test/test_path_ext.py ̀ to test this extension
made all tests using hypothesis run under unittest, in particular this permits to select and reproduce a single test function
debugged a rare situation in the random test generation, reproduced in file issues/err_walk.py where a path segment
gets generated which is interpreted by the bash globbing function resulting in a test failure (e.g. ̀seg[0]) which is globbed into something accepting seg0` but not its literal version with the brackets

…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Still needed: test and documentation of added functionality

Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

…ing to diagnostics Cleaned up formatting

…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

…H_ACCEPT_RE_REGEXP does not break things

…t discovery in nose2)

…if Flake still complains

moomoohk · 2022-12-04T13:40:55Z

@AlainLich Hey, are you still interested in working on this PR?

AlainLich · 2022-12-04T17:37:56Z

@moomoohk Hello, I have not touched this for a year and a half, could be interested to work on this PR, likely at a slow pace, since this would facilitate my own use (loading via pip ).

However, only on Python3... not interested in Python2 compatibility and such.

moomoohk · 2022-12-04T17:59:08Z

That's great to hear :)
Slow pace is alright by me, as well as Python 3+. On that note, I've decided to drop Python 2 support entirely a while ago!
Minimum supported version is 3.7 as of now.
I've refactored the codebase quite a bit in the last year and a half. Mainly modernizing, upgrading to Python 3 and blowing some dust off...
If you have any questions feel free to reach out here or on Gitter.

AlainLich · 2022-12-04T19:55:20Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

moomoohk · 2022-12-04T19:57:16Z

Yes indeed. Please start by merging master into your branch and take it from there.

AlainLich · 2022-12-13T11:18:12Z

How do you suggest to proceed. I suppose the simpler is for me to pull the latest from branch masteror any other branch you find more relevant and look at the changes. Then I will try to assess what needs to be changed to integrate the PR 'as is'.

Please advise.

AlainLich · 2022-12-13T11:20:19Z

Hi, I have entered a new PR #177 with revised code integrated in the current commit for branch master.

This PR should be closed.

AlainLich added 14 commits April 20, 2021 17:04

Added ability to use regexp ('re') to specify path segments, tweaked …
df5ac0a
…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Added support for Github's Actions (in .github/workflows)
f99f882
Still needed: test and documentation of added functionality

Added test for re regexp feature, documentation.
5f13e7d
Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

Resolved a Python2.7 issue in dpath/segment.py; added some tests lead…
7d1a011
…ing to diagnostics Cleaned up formatting

Correct YML error
856afe4

Corrected YAML, now relying on yamllint
801d5c7

Well yaml is tricky
b3ebbbf

Trying to better control test duration
0d26277

Compatibility with requirement to enable regexp feature, test improve…
b0a7c5a
…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

Chased rare error in random test generated tests tests/test_segment.p…
2677126
…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

Extended testing (random test generation) to check that enabling DPAT…
1539f4c
…H_ACCEPT_RE_REGEXP does not break things

Added Tox tests, (not successful attempt to improve/dispense with tes…
c68fbdd
…t discovery in nose2)

Trial to sort out loading pypy in workflow action
380e6c3

Run Flake8; make several improvements, make workflow successful even …
c29643e
…if Flake still complains

moomoohk assigned AlainLich Dec 5, 2022

AlainLich mentioned this pull request Dec 13, 2022
Add ability to select path segments using Python re regexprs; follow on to PR#146 #177
Closed

moomoohk closed this Dec 13, 2022

AlainLich added 14 commits April 20, 2021 17:04

Added ability to use regexp ('re') to specify path segments, tweaked …
df5ac0a
…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Added support for Github's Actions (in .github/workflows)
f99f882
Still needed: test and documentation of added functionality

Added test for re regexp feature, documentation.
5f13e7d
Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

Resolved a Python2.7 issue in dpath/segment.py; added some tests lead…
7d1a011
…ing to diagnostics Cleaned up formatting

Correct YML error
856afe4

Corrected YAML, now relying on yamllint
801d5c7

Well yaml is tricky
b3ebbbf

Trying to better control test duration
0d26277

Compatibility with requirement to enable regexp feature, test improve…
b0a7c5a
…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

Chased rare error in random test generated tests tests/test_segment.p…
2677126
…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

Extended testing (random test generation) to check that enabling DPAT…
1539f4c
…H_ACCEPT_RE_REGEXP does not break things

Added Tox tests, (not successful attempt to improve/dispense with tes…
c68fbdd
…t discovery in nose2)

Trial to sort out loading pypy in workflow action
380e6c3

Run Flake8; make several improvements, make workflow successful even …
c29643e
…if Flake still complains

moomoohk assigned AlainLich Dec 5, 2022

AlainLich mentioned this pull request Dec 13, 2022
Add ability to select path segments using Python re regexprs; follow on to PR#146 #177
Closed

moomoohk closed this Dec 13, 2022

AlainLich added 14 commits April 20, 2021 17:04

Added ability to use regexp ('re') to specify path segments, tweaked …
df5ac0a
…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Added support for Github's Actions (in .github/workflows)
f99f882
Still needed: test and documentation of added functionality

Added test for re regexp feature, documentation.
5f13e7d
Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

Resolved a Python2.7 issue in dpath/segment.py; added some tests lead…
7d1a011
…ing to diagnostics Cleaned up formatting

Correct YML error
856afe4

Corrected YAML, now relying on yamllint
801d5c7

Well yaml is tricky
b3ebbbf

Trying to better control test duration
0d26277

Compatibility with requirement to enable regexp feature, test improve…
b0a7c5a
…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

Chased rare error in random test generated tests tests/test_segment.p…
2677126
…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

Extended testing (random test generation) to check that enabling DPAT…
1539f4c
…H_ACCEPT_RE_REGEXP does not break things

Added Tox tests, (not successful attempt to improve/dispense with tes…
c68fbdd
…t discovery in nose2)

Trial to sort out loading pypy in workflow action
380e6c3

Run Flake8; make several improvements, make workflow successful even …
c29643e
…if Flake still complains

moomoohk assigned AlainLich Dec 5, 2022

AlainLich mentioned this pull request Dec 13, 2022
Add ability to select path segments using Python re regexprs; follow on to PR#146 #177
Closed

moomoohk closed this Dec 13, 2022

AlainLich added 14 commits April 20, 2021 17:04

Added ability to use regexp ('re') to specify path segments, tweaked …
df5ac0a
…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Added support for Github's Actions (in .github/workflows)
f99f882
Still needed: test and documentation of added functionality

Added test for re regexp feature, documentation.
5f13e7d
Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

Resolved a Python2.7 issue in dpath/segment.py; added some tests lead…
7d1a011
…ing to diagnostics Cleaned up formatting

Correct YML error
856afe4

Corrected YAML, now relying on yamllint
801d5c7

Well yaml is tricky
b3ebbbf

Trying to better control test duration
0d26277

Compatibility with requirement to enable regexp feature, test improve…
b0a7c5a
…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

Chased rare error in random test generated tests tests/test_segment.p…
2677126
…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

Extended testing (random test generation) to check that enabling DPAT…
1539f4c
…H_ACCEPT_RE_REGEXP does not break things

Added Tox tests, (not successful attempt to improve/dispense with tes…
c68fbdd
…t discovery in nose2)

Trial to sort out loading pypy in workflow action
380e6c3

Run Flake8; make several improvements, make workflow successful even …
c29643e
…if Flake still complains

moomoohk assigned AlainLich Dec 5, 2022

AlainLich mentioned this pull request Dec 13, 2022
Add ability to select path segments using Python re regexprs; follow on to PR#146 #177
Closed

moomoohk closed this Dec 13, 2022

AlainLich added 14 commits April 20, 2021 17:04

Added ability to use regexp ('re') to specify path segments, tweaked …
df5ac0a
…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Added support for Github's Actions (in .github/workflows)
f99f882
Still needed: test and documentation of added functionality

Added test for re regexp feature, documentation.
5f13e7d
Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

Resolved a Python2.7 issue in dpath/segment.py; added some tests lead…
7d1a011
…ing to diagnostics Cleaned up formatting

Correct YML error
856afe4

Corrected YAML, now relying on yamllint
801d5c7

Well yaml is tricky
b3ebbbf

Trying to better control test duration
0d26277

Compatibility with requirement to enable regexp feature, test improve…
b0a7c5a
…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

Chased rare error in random test generated tests tests/test_segment.p…
2677126
…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

Extended testing (random test generation) to check that enabling DPAT…
1539f4c
…H_ACCEPT_RE_REGEXP does not break things

Added Tox tests, (not successful attempt to improve/dispense with tes…
c68fbdd
…t discovery in nose2)

Trial to sort out loading pypy in workflow action
380e6c3

Run Flake8; make several improvements, make workflow successful even …
c29643e
…if Flake still complains

moomoohk assigned AlainLich Dec 5, 2022

AlainLich mentioned this pull request Dec 13, 2022
Add ability to select path segments using Python re regexprs; follow on to PR#146 #177
Closed

moomoohk closed this Dec 13, 2022

AlainLich added 14 commits April 20, 2021 17:04

Added ability to use regexp ('re') to specify path segments, tweaked …
df5ac0a
…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Added support for Github's Actions (in .github/workflows)
f99f882
Still needed: test and documentation of added functionality

Added test for re regexp feature, documentation.
5f13e7d
Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

Resolved a Python2.7 issue in dpath/segment.py; added some tests lead…
7d1a011
…ing to diagnostics Cleaned up formatting

Correct YML error
856afe4

Corrected YAML, now relying on yamllint
801d5c7

Well yaml is tricky
b3ebbbf

Trying to better control test duration
0d26277

Compatibility with requirement to enable regexp feature, test improve…
b0a7c5a
…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

Chased rare error in random test generated tests tests/test_segment.p…
2677126
…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

Extended testing (random test generation) to check that enabling DPAT…
1539f4c
…H_ACCEPT_RE_REGEXP does not break things

Added Tox tests, (not successful attempt to improve/dispense with tes…
c68fbdd
…t discovery in nose2)

Trial to sort out loading pypy in workflow action
380e6c3

Run Flake8; make several improvements, make workflow successful even …
c29643e
…if Flake still complains

moomoohk assigned AlainLich Dec 5, 2022

AlainLich mentioned this pull request Dec 13, 2022
Add ability to select path segments using Python re regexprs; follow on to PR#146 #177
Closed

moomoohk closed this Dec 13, 2022

AlainLich added 14 commits April 20, 2021 17:04

Added ability to use regexp ('re') to specify path segments, tweaked …
df5ac0a
…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Added support for Github's Actions (in .github/workflows)
f99f882
Still needed: test and documentation of added functionality

Added test for re regexp feature, documentation.
5f13e7d
Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

Resolved a Python2.7 issue in dpath/segment.py; added some tests lead…
7d1a011
…ing to diagnostics Cleaned up formatting

Correct YML error
856afe4

Corrected YAML, now relying on yamllint
801d5c7

Well yaml is tricky
b3ebbbf

Trying to better control test duration
0d26277

Compatibility with requirement to enable regexp feature, test improve…
b0a7c5a
…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

Chased rare error in random test generated tests tests/test_segment.p…
2677126
…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

Extended testing (random test generation) to check that enabling DPAT…
1539f4c
…H_ACCEPT_RE_REGEXP does not break things

Added Tox tests, (not successful attempt to improve/dispense with tes…
c68fbdd
…t discovery in nose2)

Trial to sort out loading pypy in workflow action
380e6c3

Run Flake8; make several improvements, make workflow successful even …
c29643e
…if Flake still complains

moomoohk assigned AlainLich Dec 5, 2022

AlainLich mentioned this pull request Dec 13, 2022
Add ability to select path segments using Python re regexprs; follow on to PR#146 #177
Closed

moomoohk closed this Dec 13, 2022

AlainLich added 14 commits April 20, 2021 17:04

Added ability to use regexp ('re') to specify path segments, tweaked …
df5ac0a
…tests in conformance with options.ALLOW_EMPTY_STRING_KEYS Still required to add tests for regext path segments. Ran tests with Python-3 and Python-2.7 (nose)

Added support for Github's Actions (in .github/workflows)
f99f882
Still needed: test and documentation of added functionality

Added test for re regexp feature, documentation.
5f13e7d
Random test has shown 1 issue not related to new feature, non repeatable suspect binary string with single null byte;documented in issues/nonrepeatableErr.txt.

Resolved a Python2.7 issue in dpath/segment.py; added some tests lead…
7d1a011
…ing to diagnostics Cleaned up formatting

Correct YML error
856afe4

Corrected YAML, now relying on yamllint
801d5c7

Well yaml is tricky
b3ebbbf

Trying to better control test duration
0d26277

Compatibility with requirement to enable regexp feature, test improve…
b0a7c5a
…ments Uncontrolled test complexity apparently caused by translation of "***.."" into costly regexp in test framework; now simplified.

Chased rare error in random test generated tests tests/test_segment.p…
2677126
…y, test generator coming up with keys interpreted literally when walking tree, but interpreted as a bash glob when matching. See in file issues/err_walk.py. (e.g '[0]' is also a glob meaning '0' ...)

Extended testing (random test generation) to check that enabling DPAT…
1539f4c
…H_ACCEPT_RE_REGEXP does not break things

Added Tox tests, (not successful attempt to improve/dispense with tes…
c68fbdd
…t discovery in nose2)

Trial to sort out loading pypy in workflow action
380e6c3

Run Flake8; make several improvements, make workflow successful even …
c29643e
…if Flake still complains

moomoohk assigned AlainLich Dec 5, 2022

AlainLich mentioned this pull request Dec 13, 2022
Add ability to select path segments using Python re regexprs; follow on to PR#146 #177
Closed

moomoohk closed this Dec 13, 2022

Uh oh!

Conversation

AlainLich commented Jun 1, 2021

Uh oh!

moomoohk commented Dec 4, 2022

Uh oh!

AlainLich commented Dec 4, 2022

Uh oh!

moomoohk commented Dec 4, 2022

Uh oh!

AlainLich commented Dec 4, 2022

Uh oh!

moomoohk commented Dec 4, 2022

Uh oh!

AlainLich commented Dec 13, 2022

Uh oh!

AlainLich commented Dec 13, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

AlainLich commented Jun 1, 2021

Uh oh!

moomoohk commented Dec 4, 2022

Uh oh!

AlainLich commented Dec 4, 2022

Uh oh!

moomoohk commented Dec 4, 2022

Uh oh!

AlainLich commented Dec 4, 2022

Uh oh!

moomoohk commented Dec 4, 2022

Uh oh!

AlainLich commented Dec 13, 2022

Uh oh!

AlainLich commented Dec 13, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

AlainLich commented Jun 1, 2021

Uh oh!

moomoohk commented Dec 4, 2022

Uh oh!

AlainLich commented Dec 4, 2022

Uh oh!

moomoohk commented Dec 4, 2022

Uh oh!

AlainLich commented Dec 4, 2022

Uh oh!

moomoohk commented Dec 4, 2022

Uh oh!

AlainLich commented Dec 13, 2022

Uh oh!

AlainLich commented Dec 13, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation