Add SetPrintLevel to support changing print level of libbpf messages

Closes #6
Depend on https://github.com/aquasecurity/libbpfgo/pull/270

Ready for review.
A demo for this function: https://github.com/mozillazg/libbpfgo/pull/6
$ sudo ./main-static
libbpf: loading main.bpf.o
libbpf: elf: section(3) license, size 13, link 0, flags 3, type=1
libbpf: license of main.bpf.o is Dual BSD/GPL
libbpf: elf: section(4) .maps, size 32, link 0, flags 3, type=1
libbpf: elf: section(9) .BTF, size 470, link 0, flags 0, type=1
libbpf: elf: section(13) .symtab, size 168, link 1, flags 0, type=2
libbpf: looking for externs among 7 symbols...
libbpf: collected 0 externs total
libbpf: map 'numbers': at sec_idx 4, offset 0.
libbpf: map 'numbers': found type = 1.
libbpf: map 'numbers': found key [8], sz = 4.
libbpf: map 'numbers': found value [8], sz = 4.
libbpf: map 'numbers': found max_entries = 5.
libbpf: map 'numbers': created successfully, fd=8

@mozillazg Thanks for this PR and sorry for the delay in getting to it.
This seems awesome, but as we've considered to publicise an interface to callbacks, perhaps this has become old for analysis. Would you mind of reviewing https://github.com/aquasecurity/libbpfgo/pull/284. Then, if you agree, I would close this.

Then, if you agree, I would close this.

Or, if you do prefer, you can tackle SetPrintLevel on top of #284 or wait it get into and rebase this.


LDAP connection

Hello,
is it possible to connect the local osf to an local ldap server, so the user can log in with the same user data there using for all the other network applications.
Best
Marius

@sloria?

We use Apereo CAS 4.1 for all authentication to the OSF. One would need to modify the cas-overlay's (https://github.com/CenterForOpenScience/cas-overlay) OSF Authentication module to interface with LDAP rather than the MongoDB backend. Further documentation on CAS and LDAP integration can be found here:  https://apereo.github.io/cas/4.1.x/index.html


Update to Bevy 0.10

(when it comes out)

In progress

Done! https://github.com/Andrewp2/bevy_camera_shake/commit/f3f085d80660a0a36b467b942fc0de675f2c77f8

Any chance of updating this for 0.11?

@musjj Updated for 0.11

Thank you!


Update documentation and tutorial to liqoctl

Description
This PR updates the documentation, in particular w.r.t. the addition of liqoctl as preferred way to install and configure liqo.
How Has This Been Tested?
Not applicable.

/doc-build

/doc-build

/doc-build

/doc-build

/doc-build

/test

/merge


log related regression

In Zipkin 2.15, when there were http timeouts, we didn't flood the console with warning messages. We incremented metrics instead. Now, for every request attempt that times out, we get a warn line. We don't get log lines for any other type of failure unless you set ES_HTTP_LOGGING.
I'd like someone who uses elasticsearch in production to comment if this is ok.
cc @huydx @jcarres-mdsol @jcchavezs @jonathan-lo (I think you use ES in production)
cc @anuraaga @trustin (about the error handling)
Example warn logs
zipkin                      | 2019-08-17 12:16:38.308  WARN 1 --- [orker-epoll-2-4] c.l.a.c.HttpResponseDecoder              : Unexpected exception:
zipkin                      |
zipkin                      | com.linecorp.armeria.client.ResponseTimeoutException: null
zipkin                      |
zipkin                      | 2019-08-17 12:16:40.251  WARN 1 --- [orker-epoll-2-6] c.l.a.c.HttpResponseDecoder              : Unexpected exception:
zipkin                      |
zipkin                      | com.linecorp.armeria.client.ResponseTimeoutException: null
zipkin                      |
zipkin                      | 2019-08-17 12:16:41.656  WARN 1 --- [orker-epoll-2-7] c.l.a.c.HttpResponseDecoder              : Unexpected exception:
zipkin                      |
zipkin                      | com.linecorp.armeria.client.ResponseTimeoutException: null
zipkin                      |
zipkin                      | 2019-08-17 12:16:42.561  WARN 1 --- [orker-epoll-2-8] c.l.a.c.HttpResponseDecoder              : Unexpected exception:
zipkin                      |
zipkin                      | com.linecorp.armeria.client.ResponseTimeoutException: null
zipkin                      |
zipkin                      | 2019-08-17 12:16:45.190  WARN 1 --- [orker-epoll-2-1] c.l.a.c.HttpResponseDecoder              : Unexpected exception:
zipkin                      |
zipkin                      | com.linecorp.armeria.client.ResponseTimeoutException: null
zipkin                      |
zipkin                      | 2019-08-17 12:16:48.888  WARN 1 --- [orker-epoll-2-3] c.l.a.c.HttpResponseDecoder              : Unexpected exception:
zipkin                      |
zipkin                      | com.linecorp.armeria.client.ResponseTimeoutException: null
zipkin                      |
zipkin                      | 2019-08-17 12:16:48.892  WARN 1 --- [orker-epoll-2-2] c.l.a.c.HttpResponseDecoder              : Unexpected exception:
zipkin                      |
zipkin                      | com.linecorp.armeria.client.ResponseTimeoutException: null
zipkin                      |

FWIW the way the old okhttp logger worked was that in the logger it would show if there was a timeout.
ex. like this:
zipkin                      | 2019-08-17 12:36:26.032  INFO 1 --- [search:9200/...] z.e.ElasticsearchStorage                 : --> POST http://elasticsearch:9200/_bulk http/1.1 (-1-byte body)
zipkin                      | 2019-08-17 12:36:27.622  INFO 1 --- [search:9200/...] z.e.ElasticsearchStorage                 : <-- HTTP FAILED: java.net.SocketTimeoutException: timeout

what was nice about this is that you have context. also, you can use the same logging category for any kind of http failure, regardless of if that was status or timeout.

Now, what happens is we have (with the same config) a summary message plus also the WARN. The thing to pay attention to when deciding if this is ok or not is that this is request-scoped and WARN level will always be in console unless we turn it off. In an overload situation you can be filling up logs. however, if they aren't stored anywhere, like shipped to elasticsearch, getting a WARN line every few milliseconds may not be a problem.
Here's the current log with BASIC on.
zipkin                      | 2019-08-17 12:39:20.000  INFO 1 --- [orker-epoll-2-4] c.l.a.c.l.LoggingClient                  : Request: {startTime=2019-08-17T12:39:19.994Z(1566045559994257), length=4970KiB(5089733B), duration=6198?s(6198300ns), scheme=none+h1c, headers=[]}
zipkin                      | 2019-08-17 12:39:20.117  WARN 1 --- [orker-epoll-2-4] c.l.a.c.l.LoggingClient                  : Response: {startTime=2019-08-17T12:39:20.117Z(1566045560117742), length=0B, duration=0ns, cause=com.linecorp.armeria.client.ResponseTimeoutException, headers=[]}
zipkin                      |
zipkin                      | com.linecorp.armeria.client.ResponseTimeoutException: null

You'll notice that while the info has enough data to describe what the timeout is about the WARN one does not. it just says a timeout occurred. We can guess that it was ES falling over though.

Filed https://github.com/line/armeria/issues/2000 since I think it's just a bug

we can also consider putting tracing alway-on, but in no-op mode so at least we get trace/span IDs in logs. Not sure it will cover this exception, but it might help validate if timeouts are working properly, or firing too soon, or something else. right now, obviously we cannot correlate

Also filed https://github.com/line/armeria/issues/2001 I think it's better if armeria generated a request ID for logs to not have to tweak trace sampling just to associate logs.


ID third_party_kickout:
User feedback: language
(third_party_kickout)

Question ID
third_party_kickout

Details
Whenever, possible, we want to use bold sparingly, just on a few words rather than a whole sentence. We also want to be consistent in how we refer to buttons. I'll add an image of how references to the "undo" button should look.

Variable being sought
third_party_kickout

Package version
0.0.6

Filename
docassemble.MLHMotionRegardingChangeOfDomicile:data/questions/motion_regarding_change_of_domicile.yml


ENH: ndimage: log(n) implementation for 1D rank filter

Reference issue
Closes gh-20026
What does this implement/fix?
reducing the 1D rank filter run time from complexity from n to log n.
this is used in multiple functions:
https://docs.scipy.org/doc/scipy/reference/generated/scipy.signal.medfilt.html
https://docs.scipy.org/doc/scipy/reference/generated/scipy.ndimage.median_filter.html
https://docs.scipy.org/doc/scipy/reference/generated/scipy.ndimage.rank_filter.html
https://docs.scipy.org/doc/scipy-1.12.0/reference/generated/scipy.ndimage.percentile_filter.html

I see many failed tests due to issues that seem to be not related to the current pull request. Is there anything I can/should do about it?

If test failures are really unrelated, you can simply ignore them. However, I had a quick look and all the ones I saw do look related (e.g. TestNdimageInterpolation)

@rgommers any Idea why do you think it is related? It is in the same submodule but I did not interfere with this function TestNdimageInterpolation

I will correct it and resubmit.

Thanks. Just to make sure: you can push new commits to the same branch that you opened this PR from, and they will show up here. No need to open a second PR.

I had a quick look at the code, and the heavy use of templating and fused types jumped out to me. It results in a too-large size of the new extension module, it's immediately almost the largest one in the ndimage module:
% ls -lh build/scipy/ndimage/*.so | awk '{print $5, $9}'
51K build/scipy/ndimage/_ctest.cpython-311-darwin.so
105K build/scipy/ndimage/_cytest.cpython-311-darwin.so
161K build/scipy/ndimage/_nd_image.cpython-311-darwin.so
372K build/scipy/ndimage/_ni_label.cpython-311-darwin.so
345K build/scipy/ndimage/_rank_filter_1d.cpython-311-darwin.so

Could this be limited to a subset of types? What does _nd_image.rank_filter support, and why would it be necessary to extend that already-supported set?

Started working on https://ideone.com/8VVEa, I optimized by [...]

The license of this code looks a little problematic. It has a broken link to something that (from the mit-license in the link adddress) indicates the author licensed it as MIT, but that is a bit inconclusive. Is there something more definite for this code? If not, it'd be good to contact the original author to try to clarify that this is really MIT-licensed code.

The performance improvement looks promising! If the scaling changed from N to log(N), the usual way to demonstrate that is to plot runtime as a function of kernel size (or input array size). E.g., like the graphs in https://github.com/rgommers/explore-array-computing/blob/master/explore_xnd.ipynb). I think the vertical axis of your plot is more or less like that, but a bit hard to read. Is it log(N) in kernel size, and is there any change for array input size or filter order?

E.g., like the graphs in

The vertical axis is the kernel size but the improvement for the median filter is better than for large/small ranks. I wanted to demonstrate both of the aspects. In general, I understand that you are going to perform independent testing so this was for initial illustration only. Thanks for the comment anyway. I will consider it in the future.

I will correct it and resubmit.

Thanks. Just to make sure: you can push new commits to the same branch that you opened this PR from, and they will show up here. No need to open a second PR.

I am sorry, it was 3am and my brain was totally crashed :)

I had a quick look at the code, and the heavy use of templating and fused types jumped out to me. It results in a too-large size of the new extension module, it's immediately almost the largest one in the ndimage module:
% ls -lh build/scipy/ndimage/*.so | awk '{print $5, $9}'
51K build/scipy/ndimage/_ctest.cpython-311-darwin.so
105K build/scipy/ndimage/_cytest.cpython-311-darwin.so
161K build/scipy/ndimage/_nd_image.cpython-311-darwin.so
372K build/scipy/ndimage/_ni_label.cpython-311-darwin.so
345K build/scipy/ndimage/_rank_filter_1d.cpython-311-darwin.so

Could this be limited to a subset of types? What does _nd_image.rank_filter support, and why would it be necessary to extend that already-supported set?

Started working on https://ideone.com/8VVEa, I optimized by [...]

The license of this code looks a little problematic. It has a broken link to something that (from the mit-license in the link adddress) indicates the author licensed it as MIT, but that is a bit inconclusive. Is there something more definite for this code? If not, it'd be good to contact the original author to try to clarify that this is really MIT-licensed code.

I have tried to set only three types float32, float64 and int64. For the rest of the fixed point types I allocate memory and cast to int64 (maybe view would be faster?). Anyway, I do not see large variation in the size. Are you sure it is related to the Cython's type fashion?

I had a quick look at the code, and the heavy use of templating and fused types jumped out to me. It results in a too-large size of the new extension module, it's immediately almost the largest one in the ndimage module:
% ls -lh build/scipy/ndimage/*.so | awk '{print $5, $9}'
51K build/scipy/ndimage/_ctest.cpython-311-darwin.so
105K build/scipy/ndimage/_cytest.cpython-311-darwin.so
161K build/scipy/ndimage/_nd_image.cpython-311-darwin.so
372K build/scipy/ndimage/_ni_label.cpython-311-darwin.so
345K build/scipy/ndimage/_rank_filter_1d.cpython-311-darwin.so

Could this be limited to a subset of types? What does _nd_image.rank_filter support, and why would it be necessary to extend that already-supported set?

Started working on https://ideone.com/8VVEa, I optimized by [...]

The license of this code looks a little problematic. It has a broken link to something that (from the mit-license in the link adddress) indicates the author licensed it as MIT, but that is a bit inconclusive. Is there something more definite for this code? If not, it'd be good to contact the original author to try to clarify that this is really MIT-licensed code.

Concerning the license, the code contained in the referred link is also contained in Stackoverlow. Does it solve the problem?
https://stackoverflow.com/a/5971248/8443371
https://stackoverflow.com/a/5970314/8443371

Concerning the license, the code contained in the referred link is also contained in Stackoverlow. Does it solve the problem?

Unfortunately not. From http://scipy.github.io/devdocs/dev/core-dev/index.html#licensing: "For instance, code published on StackOverflow is covered by a CC-BY-SA license, which is not compatible due to the share-alike clause. These contributions cannot be accepted for inclusion in SciPy unless the original code author is willing to (re)license his/her code under the modified BSD (or compatible) license"

t the original aut

I think that I found the original and it seems ok. Please approve @rgommers

I think that I found the original and it seems ok. Please approve @rgommers

Nice digging. Agreed, that looks like the original and states clearly enough that it is MIT-licensed. So all good from that perspective.

hose are basic questions.

They aren't. Related things are under rather active discussion (see https://github.com/scipy/scipy/issues/20334#issuecomment-2027980164 for a sample), and the jury is still out on what's the recommended way to wrap C/C++ kernels going forward. Up until very recently the standing recommendation was cython, just like you did. Maybe you are the perfect person to move this forward?
So I think you've at least two options, and none of them pretty :-).

people from the C++ land like pybind11. I've no experience with it, personally, and it would be interesting to see if it produces smaller binaries. It may or may not, we'll need to see and compare.
the (majority of the) rest of ndimage constructs the extensions manually, https://github.com/scipy/scipy/blob/main/scipy/ndimage/src/nd_image.c

Currently, I reduced the usage of the fused types to the main types only (int64/float32/double), the rest of the types are casted to those types (leading to some instability in performance but keeping solid improvement over the current implementation).

This is the thing we do pretty much everywhere, and is pretty standard and necessary. Cython adds overhead indeed, but not templating over a large set of types in C++ is .
I tested again, on Linux x86-64 this time:
$ # on Linux x86-64
$ python dev.py build -C-Dbuildtype=release
$ ls -lh build/scipy/ndimage/*.so | awk '{print $5, $9}'17K build/scipy/ndimage/_ctest.cpython-311-x86_64-linux-gnu.so
84K build/scipy/ndimage/_cytest.cpython-311-x86_64-linux-gnu.so
135K build/scipy/ndimage/_nd_image.cpython-311-x86_64-linux-gnu.so
394K build/scipy/ndimage/_ni_label.cpython-311-x86_64-linux-gnu.so
263K build/scipy/ndimage/_rank_filter_1d.cpython-311-x86_64-linux-gnu.so

Turning off release mode (our default python dev.py build):
1,9M build/scipy/ndimage/_ni_label.cpython-311-x86_64-linux-gnu.so
1,2M build/scipy/ndimage/_rank_filter_1d.cpython-311-x86_64-linux-gnu.so

263 kb is still surprisingly large after removal of most of the fused types usage. I'm actually a little puzzled by what is happening there. I did a quick test doubling the number of types under ctypedef fused numeric_t:, and it only increases the size by 31 kb (~12%). Reducing it to only double reduced the size to 241 kb, so only a ~10% reduction.
Changing def statements to cdef already helps more, it's down to 209 kb for only cdef rank_filter_1d_cpp_api and down to 161 kb for cdef rank_filter_1d. So it's probably useful to do all the input validation and selection of the correct type to call in a pure Python file, and only use 1-D memoryview in the Cython code rather than using numpy functions through the Python API. There really isn't much of a speed gain of doing that kind of thing in Cython.
Making rank_filter_1d_cpp_api work on a memoryview also avoids having it return a Python object (out_arr), and it can then be annotated noexcept nogil. cython -a may help show where Python is used in the .pyx file (see docs).
pybind11 or using the C API directly could be a nice comparison, but it looks to me like optimizing the Cython code first should be the next step. The C++ code here is clean/short enough that this should be doable in Cython I'd hope.

Hi,
I have implemented the linkage via the Numpy-C API. From my tests, there is no other way providing the same speed and file size:

I have tested all the directions suggested to Cython and it is a dead-end with respect to file size.
An experienced user of pybind11 indicated it creates smaller files but it is slower.

A direct (GCC with flags) compilation produces smaller .so than the compilation via meason.build. I assume it is related to some interference by Cython (I have seen that Cython appears in .so file name therefore I assume we are using it as the compilation engine).
Currently, the file size for the main three datatypes (float32, double, int64) is 23K for a compilation flag -O1 and 27K for the package default compilation. If I am using a direct compilation there is no file size variation. I assume that the addition in the file size is another Cython artifact - as far as I see, no boost in performance is seen.
I prefer to use the default package compilation, avoiding any surprises or additional maintenance in the future.

No, just depends on the optimization level etc. Release builds are -O3, dev.py builds are -O2 -g.

for manual compilation, I did not see a file size change...

@rgommers I have made all the required modifications, hopefully in the way you expected me to.

I am not sure I am following the errors. Seems like the tests are passing in my local machine (Mac/intel i7). Also seems that they pass on the mainstream environments. Three failures are:

two on Windows
Another failure is not clear to me. I see that the test failed but I can not find the error message...

How to debug Windows failure if I work on Mac?
Any clue about the third failure?

Given that the CI failures are seen on Windows and 32-bit linux, and that the failures are the same for all three confugurations, and that they are observed for numpy 1.23 and 1.26 (so a numpy version issue is unlikely), one common feature of the three configs are default integers being 32-bit. So I'd suspect there is an assumption that int is int64 somewhere.
================================== FAILURES ===================================
_______________________ TestNdimageFilters.test_rank02 ________________________
[gw1] win32 -- Python 3.10.11 C:\hostedtoolcache\windows\Python\3.10.11\x64\python.exe
scipy\ndimage\tests\test_filters.py:1354: in test_rank02
    output = ndimage.rank_filter(array, 1, size=[3])
        array      = array([1, 2, 3, 4, 5])
        self       = <scipy.ndimage.tests.test_filters.TestNdimageFilters object at 0x00000238CFEFC490>
scipy\ndimage\_filters.py:1561: in rank_filter
    return _rank_filter(input, rank, size, footprint, output, mode, cval,
        axes       = None
        cval       = 0.0
        footprint  = None
        input      = array([1, 2, 3, 4, 5])
        mode       = 'reflect'
        origin     = 0
        output     = None
        rank       = 1
        size       = [3]
scipy\ndimage\_filters.py:1512: in _rank_filter
    np.copyto(output, x_out, casting='unsafe')
        axes       = (0,)
        cval       = 0
        filter_size = 3
        footprint  = array([ True,  True,  True])
        fshape     = [3]
        input      = array([1, 2, 3, 4, 5])
        lenf       = 3
        mode       = 2
        num_axes   = 1
        operation  = 'rank'
        origin     = 0
        origins    = [0]
        output     = array([0, 0, 0, 0, 0])
        rank       = 1
        size       = [3]
        sizes      = [3]
        temp_needed = False
        x          = array([1, 2, 3, 4, 5], dtype=int64)
        x_out      = array([[[[[                   1,                    2,
                              3,                    4,
        ...965652251621,
<PHONE_NUMBER>526144586,  4593[923](https://github.com/scipy/scipy/actions/runs/9229378152/job/25395889773?pr=20543#step:7:924)821497038668,
<PHONE_NUMBER>625236646]]]]], dtype=int64)
<__array_function__ internals>:180: in copyto
    ???
E   ValueError: could not broadcast input array from shape (2,3,4,5) into shape (5,)
        args       = (array([0, 0, 0, 0, 0]), array([[[[[                   1,                    2,
                              3,      ...65652251621,
            45[926](https://github.com/scipy/scipy/actions/runs/9229378152/job/25395889773?pr=20543#step:7:927)05167526144586,<PHONE_NUMBER>497038668,
<PHONE_NUMBER>625236646]]]]], dtype=int64))
        dispatcher = <function copyto at 0x00000238C01EF2E0>
        implementation = <built-in function copyto>
        kwargs     = {'casting': 'unsafe'}
        public_api = <function copyto at 0x00000238C01EF490>
        relevant_args = (array([0, 0, 0, 0, 0]), array([[[[[                   1,                    2,
                              3,      ...51621,
<PHONE_NUMBER>526144586,<PHONE_NUMBER>497038668,
            4589[937](https://github.com/scipy/scipy/actions/runs/9229378152/job/25395889773?pr=20543#step:7:938)883625236646]]]]], dtype=int64), None)
_______________________ TestNdimageFilters.test_rank04 ________________________
[gw1] win32 -- Python 3.10.11 C:\hostedtoolcache\windows\Python\3.10.11\x64\python.exe
scipy\ndimage\tests\test_filters.py:1371: in test_rank04
    output = ndimage.rank_filter(array, 1, size=3)
        array      = array([3, 2, 5, 1, 4])
        expected   = [3, 3, 2, 4, 4]
        self       = <scipy.ndimage.tests.test_filters.TestNdimageFilters object at 0x00000238CFEFC7F0>
scipy\ndimage\_filters.py:1561: in rank_filter
    return _rank_filter(input, rank, size, footprint, output, mode, cval,
        axes       = None
        cval       = 0.0
        footprint  = None
        input      = array([3, 2, 5, 1, 4])
        mode       = 'reflect'
        origin     = 0
        output     = None
        rank       = 1
        size       = 3
scipy\ndimage\_filters.py:1512: in _rank_filter
    np.copyto(output, x_out, casting='unsafe')
        axes       = (0,)
        cval       = 0
        filter_size = 3
        footprint  = array([ True,  True,  True])
        fshape     = [3]
        input      = array([3, 2, 5, 1, 4])
        lenf       = 3
        mode       = 2
        num_axes   = 1
        operation  = 'rank'
        origin     = 0
        origins    = [0]
        output     = array([0, 0, 0, 0, 0])
        rank       = 1
        size       = 3
        sizes      = [3]
        temp_needed = False
        x          = array([3, 2, 5, 1, 4], dtype=int64)
        x_out      = array([[[[[                   3,                    3,
                              2,                    4]],

     ... [[<PHONE_NUMBER>216342016,                    0,
<PHONE_NUMBER>216342016,                    0]]]]], dtype=int64)
<__array_function__ internals>:180: in copyto
    ???
E   ValueError: could not broadcast input array from shape (3,2,5,1,4) into shape (5,)
        args       = (array([0, 0, 0, 0, 0]), array([[[[[                   3,                    3,
                              2,      ...[[<PHONE_NUMBER>216342016,                    0,
<PHONE_NUMBER>216342016,                    0]]]]], dtype=int64))
        dispatcher = <function copyto at 0x00000238C01EF2E0>
        implementation = <built-in function copyto>
        kwargs     = {'casting': 'unsafe'}
        public_api = <function copyto at 0x00000238C01EF490>
        relevant_args = (array([0, 0, 0, 0, 0]), array([[[[[                   3,                    3,
                              2,      ...6322717216342016,                    0,
<PHONE_NUMBER>216342016,                    0]]]]], dtype=int64), None)
_______________________ TestNdimageFilters.test_rank05 ________________________
[gw1] win32 -- Python 3.10.11 C:\hostedtoolcache\windows\Python\3.10.11\x64\python.exe
scipy\ndimage\tests\test_filters.py:1381: in test_rank05
    output = ndimage.rank_filter(array, -2, size=3)
        array      = array([3, 2, 5, 1, 4])
        expected   = [3, 3, 2, 4, 4]
        self       = <scipy.ndimage.tests.test_filters.TestNdimageFilters object at 0x00000238CFEFC9A0>
scipy\ndimage\_filters.py:1561: in rank_filter
    return _rank_filter(input, rank, size, footprint, output, mode, cval,
        axes       = None
        cval       = 0.0
        footprint  = None
        input      = array([3, 2, 5, 1, 4])
        mode       = 'reflect'
        origin     = 0
        output     = None
        rank       = -2
        size       = 3
scipy\ndimage\_filters.py:1512: in _rank_filter
    np.copyto(output, x_out, casting='unsafe')
        axes       = (0,)
        cval       = 0
        filter_size = 3
        footprint  = array([ True,  True,  True])
        fshape     = [3]
        input      = array([3, 2, 5, 1, 4])
        lenf       = 3
        mode       = 2
        num_axes   = 1
        operation  = 'rank'
        origin     = 0
        origins    = [0]
        output     = array([0, 0, 0, 0, 0])
        rank       = 1
        size       = 3
        sizes      = [3]
        temp_needed = False
        x          = array([3, 2, 5, 1, 4], dtype=int64)
        x_out      = array([[[[[                   3,                    3,
                              2,                    4]],

     ... [[                   0,                    0,
                              0,                    0]]]]], dtype=int64)
<__array_function__ internals>:180: in copyto
    ???
E   ValueError: could not broadcast input array from shape (3,2,5,1,4) into shape (5,)
        args       = (array([0, 0, 0, 0, 0]), array([[[[[                   3,                    3,
                              2,      ...[[                   0,                    0,
                              0,                    0]]]]], dtype=int64))
        dispatcher = <function copyto at 0x00000238C01EF2E0>
        implementation = <built-in function copyto>
        kwargs     = {'casting': 'unsafe'}
        public_api = <function copyto at 0x00000238C01EF490>
        relevant_args = (array([0, 0, 0, 0, 0]), array([[[[[                   3,                    3,
                              2,      ...               0,                    0,
                              0,                    0]]]]], dtype=int64), None)

Given that the CI failures are seen on Windows and 32-bit linux, and that the failures are the same for all three confugurations, and that they are observed for numpy 1.23 and 1.26 (so a numpy version issue is unlikely), one common feature of the three configs are default integers being 32-bit. So I'd suspect there is an assumption that int is int64 somewhere.

Agree that the problem is related to this. If I look at the test_rank02 test case, for example, it starts with
array = np.array([1, 2, 3, 4, 5])

which I think will result in an array of int32 on 32-bit platforms and int64 on 64-bit platforms. It seems only the latter is working as expected.
I can get the same failure on a 64-bit system if I change the first line of the test_rank02 test case to explicitly use int32 data
    def test_rank02(self):
        array = np.array([1, 2, 3, 4, 5], dtype=np.int32)
        output = ndimage.rank_filter(array, 1, size=[3])
        assert_array_almost_equal(array, output)
        output = ndimage.percentile_filter(array, 50, size=3)
        assert_array_almost_equal(array, output)
        output = ndimage.median_filter(array, (3,))
        assert_array_almost_equal(array, output)

will comment inline with the fix

seems like the failure has nothing to do with the modified code, right?
AILED scipy\optimize\tests\test__differential_evolution.py::TestDifferentialEvolutionSolver::test_strategy_fn

seems like the failure has nothing to do with the modified code, right?
FAILED scipy\optimize\tests\test__differential_evolution.py::TestDifferentialEvolutionSolver::test_strategy_fn

Exactly. This is unrelated and seen elsewhere and is likely to disappear on a rebase on main.

seems like the failure has nothing to do with the modified code, right?
FAILED scipy\optimize\tests\test__differential_evolution.py::TestDifferentialEvolutionSolver::test_strategy_fn

Exactly. This is unrelated and seen elsewhere and is likely to disappear on a rebase on main. Is safe to ignore here anyhow.

should I rebase or wait for other comments? I understand we like to avoid redundant tests to save sources but I am not sure...

Just ignore it.

different not related errors:

FAILED scipy\optimize\tests\test__differential_evolution.py::TestDifferentialEvolutionSolver::test_strategy_fn
FAILED scipy/integrate/tests/test_quadpack.py::TestQuad::test_complex - AssertionError: assert 1 == 3
waiting for your inputs

Just confirming that the DE test failure is unrelated and will be fixed by gh-20814.
I'll look out for that quadpack failure in other PRs though. That looks new to me.

It passed all tests.
What is the next step?

This is looking pretty good now, should be close to ready to merge. @grlee77 thanks for having a look - I just wanted to check, with the test you suggested having been added, did you have any other concerns or are you happy for this PR to move ahead?

@rgommers: it looks good to me now, approved

@rgommers what is next after the approval by @grlee77 ?

Does this also close #4878?

@dschmitz89 it does not close it as it is an implementation for 1d only. Anyway, the current implementation is lean in memory usage, for Vector length N it allocates two additional N length vectors of integers (for indexing).

This nearly slipped under the radar after Greg's approval, https://github.com/scipy/scipy/pull/20543#issuecomment-2162970175.
I kicked the CI to double-check it's still green, and added to the 1.15 milestone so that it does not slip again.
@ggkogan sorry for the radio silence for this long! It's generally OK to ping a PR when we stop being responsive like this. Typically an approval by a domain expert means somebody pushes the merge button, but in this case an unfortunate series of events happened, which I'd summarize as "life interfered" and leave it at that.

This nearly slipped under the radar after Greg's approval, #20543 (comment).
I kicked the CI to double-check it's still green, and added to the 1.15 milestone so that it does not slip again.
@ggkogan sorry for the radio silence for this long! It's generally OK to ping a PR when we stop being responsive like this. Typically an approval by a domain expert means somebody pushes the merge button, but in this case an unfortunate series of events happened, which I'd summarize as "life interfered" and leave it at that.

Oh, I itrepetaitied as no interest in the outcomes and stopped the development of the Hampel filter.
Will try to recover it now..

@ev-br, there is a new test named Linux PyTorch/JAX/xp-strict CPU' that fails in multiple places with ValueError: WRITEBACKIFCOPY base is read-only`. I am not sure how to handle it. I assume that I should either modify my code(I do not know how) or modify the tests. Would you please point me to similar cases so I can learn from examples?

Hm, so the error are
FAILED scipy/ndimage/tests/test_filters.py::TestNdimageFilters::test_rank02[jax.numpy] - ValueError: WRITEBACKIFCOPY base is read-only
FAILED scipy/ndimage/tests/test_filters.py::TestNdimageFilters::test_rank04[jax.numpy] - ValueError: WRITEBACKIFCOPY base is read-only
FAILED scipy/ndimage/tests/test_filters.py::TestNdimageFilters::test_rank05[jax.numpy] - ValueError: WRITEBACKIFCOPY base is read-only
FAILED scipy/ndimage/tests/test_filters.py::test_valid_origins[jax.numpy] - ValueError: WRITEBACKIFCOPY base is read-only

I cannot reproduce them locally with python 3.11, jax 0.4.25 and numpy 2.1.0, so have to guess. Can you repro them on your system? The relevant incantation is something like $ python dev.py test -s ndimage -b jax.numpy, and you'll need jax CPU installed.
All failures are with jax.numpy inputs, which are immutable, and something is trying to mutate them. Given that it does not happen on main, I've got to ask if your c++ code is modifying input or footprint? If it does, this is something to fix in your code.
Otherwise what likely happens is this: jax.numpy array inputs are being converted to numpy via np.asarray at the top of rank_filter. It's possible that some inputs are not converted and your c++ code receives a jax array. Then it's a bug in rank_filter and the fix is to add a relevant np.asarray before calling  _rank_filter_1d.rank_filter.
If still not, and all jax arrays are converted and it just so happens that your code really needs to modify an argument in-place, then the only fix is to skip failing tests via @skip_skip_backends("jax.numpy", reason="mention the reason"). There are several instances of this pattern in test_filters.py.

It looks like _rank_filter_1d.rank_filter assumes that x is mutable

@lucascolley , I am not sure if you see any modifications that I perform on the input. I have tested with a long random input array and found no changes on values. Maybe you assume some kind of definition such as const keyword?
Anyway, for testing of the problem, I have implemented copy on the input as default. It seems like a rough solution which is suitable for testing rather than finding where I actually modify the value. In case it passes, I would still appreciate if you point me to a precise location as I do not have the access to a system fitting the failed test.

@ev-br I did not modify test_rank09 which actually tests other functions such as rolling min. IMO, we should open an issue on it but it is out of scope of the current PR

xp_assert_equal(actual, desired) checks that actual and desired have the same namespace, so it'll error out if actual is a torch tensor and expected is a numpy array.
@ev-br I did not see this comment and now I have got the errors. My question is, are we expected to be capable providing any other array type except numpy array? Should we provide it only in case it was given to us in the input?

@ev-br , we have passed all the new tests...
What is the next step?

All relevant CI is finally green. The main algorithmic part was approved by Greg, so let's merge this then. Thank you @ggkogan and keep them coming! Thanks to all reviewers, too.


ASU-1559 | Update docker entrypoint to compile messages, fix Hitas payment template

Codecov Report

Merging #274 (69ee725) into develop (7695f9d) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #274   +/-   ##
========================================
  Coverage    93.29%   93.29%
========================================
  Files          118      118
  Lines         4236     4238    +2
  Branches       377      378    +1
========================================
+ Hits          3952     3954    +2
  Misses         205      205
  Partials        79       79

Impacted Files
Coverage Δ

application_form/pdf/hitas.py
96.77% <ø> (-0.06%)
:arrow_down:

invoicing/api/serializers.py
93.80% <0.00%> (+0.22%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Some error occurred while compiling sqlite3mc. c using Bcc32c

Some error occurred while compiling sqlite3mc. c using Bcc32c:
[bcc32c Error] sqlite3mc. c (273457):'_builtin_ia32aeskigenassist128 'needs target feature aes
[bcc32c Error] sqlite3mc. c (273457): always_inline function 'mm_setr_epi32' requires target feature 'mmx', but would be inline into function 'aesGenKeyEncryptInternal' that is compiled without support for 'mmx'

Some error occurred while compiling sqlite3mc. c using Bcc32c: [bcc32c Error] sqlite3mc. c (273457):'_builtin_ia32aeskigenassist128 'needs target feature aes [bcc32c Error] sqlite3mc. c (273457): always_inline function 'mm_setr_epi32' requires target feature 'mmx', but would be inline into function 'aesGenKeyEncryptInternal' that is compiled without support for 'mmx'

The compile time errors have to do with the compilation of the AES hardware support code. The error messages tell you exactly what is missing:

target feature aes
target feature mmx

That is, you will have to specify additional compiler options to enable those target features. For the gcc compiler the options are -maes -msse4.2, but I don't know what the corresponding options for bcc32c would be.
Alternatively, you can avoid the compile errors by specifying the preprocessor symbol SQLITE3MC_OMIT_AES_HARDWARE_SUPPORT. However, this will then disable native AES hardware support.

I couldn't find the AES and MMX options for bcc32c, so I used SQLITE3MC-OMIT-AES-HARDWARESUPPORT to compile it, but there was some error when linking:
[ilink32 Error] Error: Unresolved external '_rand_s' referenced from D:\PROJECTS\SQLITE3MC\WIN32\RELEASE\SQLITE3MC.OBJ
[ilink32 Error] Error: Unresolved external '_ASCON_P6ROUNDS' referenced from D:\PROJECTS\SQLITE3MC\WIN32\RELEASE\SQLITE3MC.OBJ
[ilink32 Error] Error: Unresolved external '_ascon_absorb' referenced from D:\PROJECTS\SQLITE3MC\WIN32\RELEASE\SQLITE3MC.OBJ
[ilink32 Error] Error: Unresolved external '_ASCON_P12ROUNDS' referenced from D:\PROJECTS\SQLITE3MC\WIN32\RELEASE\SQLITE3MC.OBJ

In the meantime I installed the C++Builder community edition for testing. However, I've never seen such a crippled C++ development environment before. Depending on the selected build (debug vs release, Win32 vs Win64) I got various different error messages.

I couldn't find the AES and MMX options for bcc32c, so I used SQLITE3MC-OMIT-AES-HARDWARESUPPORT to compile it,

I haven't found corresponding options either, but selecting the clang compiler instead of the classic compiler allowed me to compile the AES hardware support code without issues.

but there was some error when linking: [ilink32 Error] Error: Unresolved external '_rand_s' referenced from D:\PROJECTS\SQLITE3MC\WIN32\RELEASE\SQLITE3MC.OBJ

Well, C++Builder obviously doesn't implement the rand_s function. To overcome this obstacle you have to define the preprocessor symbol SQLITE3MC_USE_RAND_S=0.

[ilink32 Error] Error: Unresolved external '_ASCON_P6ROUNDS' referenced from D:\PROJECTS\SQLITE3MC\WIN32\RELEASE\SQLITE3MC.OBJ [ilink32 Error] Error: Unresolved external '_ascon_absorb' referenced from D:\PROJECTS\SQLITE3MC\WIN32\RELEASE\SQLITE3MC.OBJ [ilink32 Error] Error: Unresolved external '_ASCON_P12ROUNDS' referenced from D:\PROJECTS\SQLITE3MC\WIN32\RELEASE\SQLITE3MC.OBJ

For whatever reason the C++Builder compiler refuses to compile the code marked as inline. As mentioned above I used the compiler in clang mode, but the compiler does not define the preprocessor symbol __CLANG__. If I add the preprocessor symbol __CLANG__ manually, the code compiles without errors.
At first I had tried to build the Win32 variant, but the linker complained about an unresolved reference. I found out that C++Builder provides it only for Win64 (no idea why not for Win32). So I switched to Win64.
Nevertheless the linker now complained about not being able to load the system link library advapi32.lib. The reason is that C++Builder does not provide it for all build configurations, and - what's worse - does not name it consistently: in some build configurations it is named advapi32.lib, but in some advapi32.a.
After these "experiences" I come to the conclusion that I will not add official support for C++Builder - it is simply too cumbersome to overcome the deficiencies of that compiler.
You may be able to get it to work somehow with the configuration you need. However, I would recommend to use a different compiler (like Visual C++ or MinGW-W64).

Add the following three lines at the beginning of the sqlite3mc. c file, and compile them successfully.

#Define _ CLANG__
#Define SQLITE3MC-USE-RAND_S 0
#Define SQLITE3MC-OMIT-AES-HARDWARESUPPORT

Thank you for your patient guidance and for creating the great project SQLite3MultipleCiphers.

Add the following three lines at the beginning of the sqlite3mc. c file, and compile them successfully.
#define __CLANG__
#define SQLITE3MC_USE_RAND_S 0
#define SQLITE3MC_OMIT_AES_HARDWARE_SUPPORT

Good to hear that defining those preprocessor symbols allowed you to successfully compile with the bcc32c compiler.
Specifying the symbol SQLITE3MC_OMIT_AES_HARDWARE_SUPPORT should not be necessary - but it may depend on the actual version of bcc32c.
Please understand that I will not add these defines to the source code - at least not unconditionally.

Thank you for your patient guidance and for creating the great project SQLite3MultipleCiphers.

You are welcome. I'll close the issue now as resolved.


RequestError: Error with undefined response object

I'm not sure if this is an issue or not, but I'm getting "RequestError: Error" without any other identifying information in the error object:
{
  name: 'RequestError',
  message: 'Error',
  cause: Error,
  error: Error,
  options:
   { uri: <someURI>,
     gzip: true,
     proxy: <someProxyURI>
     rejectUnauthorized: false,
     transform: [Function: transform],
     callback: [Function: RP$callback],
     simple: true,
     resolveWithFullResponse: false,
     transform2xxOnly: false },
  response: undefined
}

I've never seen this before, nor do I have any idea what the issue is based on this response. Am I missing something?
Using 4.2.4 on Node 10.16.0
EDIT: It's also worth noting that my transform function isn't even being called on this error, but I believe the docs suggest that it should be.

Figured out what was causing the problem, but not why: Setting gzip true.
Removing that fixed the issue, which is odd because the I had to add that in a previous version of node to get the parsing to work.
Still uncertain if this is a bug, but leaving open in case you want more details to look into it.


Can't stop Edit Sessions

Testing #161860
I've gotten into a state where my Source Control view is showing two repositories even though I've just opened code-insiders in one.

Trying Turn Off Edit Sessions doesn't seem to help.
Even odder, there are no stored Edit Sessions

Signing out from GitHub doesn't seem to help either

I found the "Close repository" command which removed the vscode-docs repo from the Source Control view.
I can I'm surprised it was added when I opened the Edit Sessions view

Did you happen to use the Continue On > Clone Locally flow at some point? Opening the Edit Sessions view should not have anything to do with this.

Hmm, the second repo came back when I reloaded the window.
I see "Resuming edit session" on the Status bar

Nope, only playing with Edit sessions.

"Resuming edit session" will pretty much appear all the time when you first open a window unless you explicitly disable it, but it doesn't do anything with checking out git repositories. Is there anything in the Git or Log (Edit Sessions) channel, and do you recall what you've tried with Edit Sessions so far?

I was working with Edit Sessions in the vscode-docs repo and then I switched to working in vscode-website. The Edit Sessions view correctly (I believe) showed sessions from both repositories.
Not sure why the vscode-docs repo was added to the Source Control view

OK, might have been a red herring.
The vscode-website repo clones the vscode-docs repo if you build the website locally. The vscode-docs repo shown in the Source Control view is for that sub-repo.
What is odd is that Stable doesn't show the sub-repo and I don't think I've noticed that before with the 1.72 Insiders but something might have changed recently to include sub-repos in the Source Control view?

git.autoRepositoryDetection controls the behavior. If I set it to false, VS Code doesn't pick up the sub-repo.
I wonder if an issue was fixed recently with repo detection

Closing this as not related to Edit Sessions. Seems to be a change in sub-repo detection. I'll follow up with Lad

Yeah this landed via https://github.com/microsoft/vscode/pull/159291


Deferrals not working on some users. Skipped Straight to restart

It is working great for the most part! But I had a user come to me with an issue where they did not get a deferral option. It gave them the "you have deferred the maximum number of 5 times. A required software updated will automatically restart this computer in about 5 minutes." Which unfortunately came at an inopportune time. I have included the log with the notable events starting at 14:57 :mysuper.log
and the deferment plist:

JamfProID
$JSSID
MenuDefer
300,1800,3600,7200
FocusCount
5
HardCount
5
FocusDays
2
DisplayRedraw
10800
DisplayIcon
/private/var/tmp/icon_for_superman/icon.icns
IconSizeIbm
128
IconSizeJamf
128
ForceRestart

You shouldn't be running super v2 on that system... please see the latest release notes... https://github.com/Macjutsu/super/releases


Help Wanted: Omnibus docs are mis-leading, claiming for Debian but it only support Ubuntu

I tried to test omnibus per the directives at the web site and it gives this error
ERROR: Unsupported ubuntu release: "bullseye"

I think there should be clear notification it only supports Ubuntu, not Debian.

@nab-77 for visibility

@tolgaulas thanks for the feedback. Indeed this could be clearer.

@tolgaulas - Are you open to creating a docs PR with this correction and clarification? We'd welcome your contribution!

Hi @cwarnermm, can I work on this issue?

Thank you, @aniketh-varma! We welcome your contribution to this ticket! I've assigned it to you.

@cwarnermm As there's no progress on this issue for the past 2 weeks, I would like to pick this up. Thanks.

@aniketh-varma - Are you actively working on this issue? If you're not, another contributor has expressed interest in picking it up. Thanks.

Can you please assign this issue to me @cwarnermm ? I am looking forward to contributing to this issue.


please support arco design

please support arco design
https://arco.design/vue/docs/start

Hi,
Thank you for the suggestion, I'm always open to support new UI frameworks.
Are you interested in implementing it yourself? I could write a short guide on how to add a framework to vue-js-cron.

@abichinger yes, i like do it

Here is the guide: https://github.com/abichinger/vue-js-cron/blob/main/CONTRIBUTING.md#add-a-new-ui-framework
Please let me know if anything is unclear.


The region used in the settings.yaml is no't used

Even if you change the region configured on your setting.yaml, it won't use it.
It will use you region on your ~/.aws/config instead.

this has been fixed in e79e5fc8b09427e100fa9d6d42d5be0fe2ba31bf

Fixed in e79e5fc8b09427e100fa9d6d42d5be0fe2ba31bf


If only a single input files exists skip mosaicing.

This will probably be an issue based on resolution changes during reproject. Closing as not going to be worked.


Reason for 981172 (PCRE limits exceeded) ?

Greetings,
We are experiencing 981172 for simple requests like the below however we are unable to replicate when handcrafting the request using the cookies reported my modsecurity. A simple request to the resource below for example does not cause modsecurity to complain either.

[15/Mar/2016:19:55:16 --0400] Vuig5KwQAikAABojW@IAAADM <IP_ADDRESS> 63429 <IP_ADDRESS> 443
--7b334e38-B--
GET /resources/images/svg/loader.svg HTTP/1.1
Host: bh.sample.com
Connection: keep-alive
Accept: image/webp,image/,/*;q=0.8
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36
Referer: https://bh.sample.com/resources/css/app.css?version=1.2124.0
Accept-Encoding: gzip, deflate, sdch
Accept-Language: en-US,en;q=0.8
Cookie: JSESSIONID=BA7126354A1B1E378E295DA5DBD0C6A3.sample-app3; connect.sid=s%3A8Z-CWL1eO0k1H7ZJtaE-4lqP-1_OdNSy.vgumV%2BsDeAhBrvy%2FUAFVp8K4PJR6%2FPX0zHdogQI%2Bip8; _ga=GA1.2.2048708085.1413477326; _gat=1
--7b334e38-F--
HTTP/1.1 200 OK
Last-Modified: Tue, 15 Mar 2016 17:22:09 GMT
Accept-Ranges: bytes
Content-Length: 1673
Cache-Control: max-age=2592000, no-store
Expires: Thu, 14 Apr 2016 23:55:16 GMT
X-Frame-Options: sameorigin
X-XSS-Protection: 1; mode=block
X-Content-Type-Options: nosniff
X-WebKit-CSP: default-src 'self'
X-Permitted-Cross-Domain-Policies: master-only
Content-Security-Policy: default-src 'self' 'unsafe-inline'; script-src 'self' https://www.google-analytics.com; img-src 'self' https://www.google-analytics.com
Strict-Transport-Security: max-age=31536000; includeSubDomains
Keep-Alive: timeout=15, max=98
Connection: Keep-Alive
Content-Type: image/svg+xml
--7b334e38-E--
--7b334e38-H--
Message: Rule 7f883dc48ae8 [id "981172"][file "/etc/modsecurity/modsecurity_crs_41_sql_injection_attacks.conf"][line "157"] - Execution error - PCRE limits exceeded (-8): (null).
Stopwatch:<PHONE_NUMBER>267757 8497 (- - -)
Stopwatch2:<PHONE_NUMBER>267757 8497; combined=7524, p1=308, p2=7205, p3=2, p4=0, p5=8, sr=85, sw=1, l=0, gc=0
Response-Body-Transformed: Dechunked
Producer: ModSecurity for Apache/2.7.7 (http://www.modsecurity.org/); OWASP_CRS/2.2.8.
Server: Apache
Engine-Mode: "DETECTION_ONLY"
--7b334e38-Z--

For now it would look like we will have to exclude the rule:
SecRule REQUEST_URI "@contains /" "phase:1,t:none,pass,id:'17',nolog,ctl:ruleRemoveById=981172"
Any help would be really appreciated. Best regards,

Nestor

In general this has to do with the steps required to process the regex rules. It is possible to increase these such that these will not occur. The problem is that this is a balance between causing a DOS condition where extremely complicated input (to parse) can be provided by an attacker versus how deep you want your regex to process. In general tuning SecPcreMatchLimitRecursion and SecPcreMatchLimit will take care of the issue :) Please see the reference monitor for more details on this: https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual#secpcrematchlimit

Thanks for the feedback. The question is why this request will not generate any alert when manually crafted? It will appear like the alert does not happen 100% of the time as even though I see the alerts for some requests I am unable to further replicate them.

I am unsure of why you can't reproduce these requests - this rule seems to deal with REQUEST_COOKIES values and REQUEST_COOKIES_NAMEs


fix #91: change the duration of a snapshot proposal to three days

fixes #91

fixes #91

Looks good 👍🏼
Can you change the base branch to dev instead of main?

@Sarahtonein done!

Thanks mate :)


Transaction signing/change verification handling when PSBT does not contain wallet fingerprint

https://github.com/SeedSigner/seedsigner/blob/6dbdcdaa14996b1644fd20668bb7efbb0ad7208d/src/seedsigner/views/psbt_views.py#L319-L321
Context from @kdmukai "It's a combination of: a wallet that doesn't know the master fingerprint and the not-ideal way we bail on trying to verify change when no fingerprint is provided. Looks like we were smart enough to add in the sanity check to catch this, but couldn't figure out the best way to handle it if it ever happened. Now that I understand the broader context better (this BlueWallet scenario), I think we can make improvements."
User reported issue: I have a mixing(whirlpool) hot wallet on Sparrow. I have a watch only wallet on blue wallet. I put the seed words into seed signer (Version 0.60) for Blue wallet. Native segwit was used to set up the BlueWallet wallet, originally created on a coldcard. I got a receive address for the mixing wallet and I put it in blue wallet. It gave me a psbt qr code to scan.
I scanned it with my seed signer (blue wallet seed loaded on it). The transaction info came up. The correct amount and receive address were listed.
When I got to the end to sign it goes to this screen.

@kdmukai (Keith) asks "does BlueWallet know the seed's master fingerprint? I'll bet that field is blank in BlueWallet AND there's change coming back in the tx." user screenshot confirms at least the missing fingerprint

Temporary workaround described by Keith: "I thought there was a way to add it, but maybe you just have to re-import.
You can use SeedSigner to export the xpub again and import it as a separate wallet (I think BlueWallet will treat it as a separate wallet). But this time BlueWallet will know the fingerprint. If all looks good (same balance as expected, etc), you could then remove the original wallet.
I would guess that the seed in the SeedSigner has the question mark icon when you select which seed to try to sign the PSBT with. Same problem: can't know for sure if that's the right seed if the psbt doesn't supply the fingerprint." (<--later confirmed by user)

This is likely the same issues I have. Slightly different scenario.
I self selected a 12 word seed phrase, I created a watch only wallet entirely in bluewallet. Created a psbt transaction in bluewallet. Entered my seed into seedsigner and scanned bluewallet to sign. Received the same error message.


Should export ES5-compatible for npm, otherwise babel transform is required externally

The ES2015-rewritten 6.0.0 has issues when using via webpack. Usually, babel transform is only applied to the app code, and the node_modules directory is excluded entirely. This does not work with 6.0.0 compared to ES5-compatible 5.2.0 -- webpack bundles the ES2015 code of the node_modules/qs right into the output, which then breaks the uglify step.
Please provide ES5-compatible build without browserify and point package.json's main to it.
This can be done in the prepublish step. Look how it's implemented.
Related: #128, #129, #108.

Just use the previous version for now. They are the same.

@hueniverse yes, I do this for now.

i ran into this issue while updating my dependencies and saw the latest was 6.0.0 ... unfortunately this caused me to see the error SyntaxError: let is a reserved identifier and it was due to the issue @sompylasar describes. this should probably not be published in npm right now.

I suggest you learn how semver works and don't upgrade to a new major version without understanding what broke.

@hueniverse i am aware of how semver works, thank you. thought you might want to know that there's more than one person that would like to use qs latest with other their other ES5 transpiled libraries.

You can also add the path to qs in your array of included files for the babel transformer. Then you could exclude all of the other node_modules, but include qs.

@arb

You can also add the path to qs in your array of included files for the babel transformer

how would one do that actually? If node_modules is excluded using exclude parameter - you cannot include anything from it.

When you are setting up the loader, if you are excluding "node_modules", there is also an include option where you can pass an array of values. Add the path to QS into that array. (This is just speculating as I haven't done this, but I think that should work.)
module: {
  loaders: [
    {
      test: /\.js?$/,
      exclude: /node_modules/,
      include: [
        'path/to/your/app',
        'node_modules/qs'
      ]
      loader: 'babel',
      query: {
        presets: ['es2015']
      }
    }
  ]
}

Something along those lines. You'll have to play around with the include paths but that is the general gist of it.

There are certainly workarounds but this does not actually resolve the
issue.

There is no way to resolve this because there is no issue. This module is ES6 now. If you need ES5, either use an older version or configure your transpiler to deal with it for you.

https://github.com/nodejs/help/issues/8

http://jonnyreeves.co.uk/2015/babel-npm/

I'm aware of the many methods available for maintaining an ES5 version of this module. I have no interest in doing so. If you really want an ES5 version, build and publish one. I'm happy to link to it from here.

I see. Good luck then.

@arb that was my exact point: if a path is in exclude - there is no way to make it loaded. That's what exclude is for - to exclude.

Come on, it's a regex, It's not that hard to exclude /node_modules\/(?!qs)/...
I stand with @nlf, it's your problem if you want to take it into an ES5-only environment, this module explicitly says it uses node 4+ and YOU have ways to deal with it.

I'd like to state that in some cases "configure your transpiler to deal with it for you" is actually a ridiculous suggestion.
I'm using browserify, but:

qs hasn't declared something like "browserify": {"transform": [["babelify", {"presets": ["es2015"]}]]} so browserify doesn't work out of the box and I can't add this to the qs package myself.
"linking" to a 3rd party published browser build won't change anything since browserify wouldn't pick up on it, it's not actually part of the module.
The "other" option of configuring babelify as a global transform, doesn't actually work. Or rather, whether it works or not depends entirely on every other random module you depend on.

When you ignore files, that doesn't actually tell the transform to go and ignore entire modules. It's a babel level option and subject to other babel level options defined elsewhere. So babel still goes through the tree and parses other options like those in .babelrc files belonging to every single module you depend on.
This sort of works... up until you hit one of the modules that is transpiled during prepublish using babel@5, and then this happens Unhandled rejection ReferenceError: [BABEL] /.../node_modules/react-leaflet/lib/index.js: Unknown option: /.../node_modules/react-leaflet/.babelrc.optional while parsing file: /.../node_modules/react-leaflet/lib/index.js

It seems to me that as this is an official hapi module there should be an official hapi policy in place for module dependencies.
My vote is make it work for the widest possible audience. At least until ES5 is way down the long tail.

we have a discussion along those lines going here

Had the same problem & spent more time researching it than it took to fix it with webpack:
module: {
  loaders: [
        {
          exclude: /node_modules/,
          loader: "babel-loader",
          test: /\.js$/,
        },
        {
          include: /node_modules\/qs/),
          loader: "babel-loader",
          test: /\.js$/,
        },
    ],
  }
}

Happy copy-pasting!

come on, it's a regex, It's not that hard to exclude /node_modules/(?!qs)/...
I stand with @nlf, it's your problem if you want to take it into an ES5-only environment, this module explicitly says it uses node 4+ and YOU have ways to deal with it.

Well, this module is meant to work in browsers, so forcing every dependent package or app to make a special case for this one untranspiled package is spreading the burden and confusion.

I think you could also replace include: /node_modules\/qs/) with include: require.resolve('qs') as well which would be a little clearer what you're trying to do.


Get rid of dropping bypassed connections during soft-reconfig

In fact, this is reverting the https://github.com/AdguardTeam/CoreLibs/issues/1003. We should do it in order to don't irritating users in particular when filters and/or userscripts are updated

Guys, please note that this must be a hotfix to v1.4. We shouldn't roll out v1.5 yet.

core/pull-requests/1475
no need to test


Add app authy.com

Added Authy app.
The form is here.
They ask you to check if you've disabled 2FA on all accounts you've added to the app.
Then, they ask you to enter your phone number and type a code from SMS.
After that, they'll show you a list of sites and again ask you to check if you've disabled 2FA.
Finally, they'll send you an e-mail with links to continue or cancel the process. Cancellation link will be active for the next 30 days.

Thanks for another entry!


[docs] Prevent layout shift when rendering ads

[x] I have followed (at least) the PR section of the contributing guide.

Fixes https://github.com/mui-org/material-ui/issues/17887

I will follow up to solve an edge case, we don't display the ad on all the pages, for instance, it's excluded from the installation page.

I see, I guess the style rules should rather go in here? Anyway, feel free to work on this, I won't have time to work on this again until tonight.

What do you think of this solution?

Ok for me

@oliviertassinari 👍 Ok, that works well, I've turned my adblocker off.

I have created a reminder in 2 months, I'm curious to see how the click/display and user/display metric evolves.

I've turned my adblocker off.

This means an extra ~$0.03 for us at the end of the months. Thanks 😁.


Error deploying the website (GitHub action)

Acknowledge the following

[x] I carefully read and followed the Getting Started guide.
[x] I read through FAQ and searched through the past issues, none of which addressed my issue.
[x] The issue I am raising is a potential bug in al-folio and not just a usage question.  [For usage questions, please use gitter chat instead of raising an issue.]

Describe the bug
I am getting following error in the GitHub action:
Please commit those files or stash them before proceeding.
29212
Aborting.
29213
Error: Process completed with exit code 1.

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

System (please complete the following information):

OS: [e.g. iOS]
Browser (and its version) [e.g. chrome, safari]
Jekyll version [e.g. 3.8.7]
Ruby version [e.g. 2.6.5]

Additional context
Add any other context about the problem here.

@nadgowdas, I'm not able to reproduce the issue. Could you confirm that you followed the installation instructions?

@alshedivat i tried debugging it and realized it was because github has started naming default branch as main instead of master. I changed bin/deploy and I was able to host my pages.
I really like your theme, thanks for putting it up.


TN-351 formatting for message.py, next_step.py, organization.py, performer.py

Hello @John0Randolph! Thanks for submitting the PR.

In the file portal/models/organization.py, following are the PEP8 issues :

Line 288:26: E127 continuation line over-indented for visual indent


Add auto reconnect section to SignalR JS client docs

Addresses #11093 for the SignalR TS/JS client.

⬆📆

Additional feedback addressed.

I'm just going to assume this is OK. Ping me if you need me to make follow up changes in another PR.


Support for batch on fargate?

This was recently announced at re:Invent and I've been trying to get it to work without much luck. Are there plans to support this or is Fargate not in the cards because of the lack of support for gpu etc?

@bcgalvin Great timing! I am working on a PR for that. Unfortunately, you won't be able to specify resources at the grain that batch + ecs allows but it will allow for quicker job start-up times.

This is huge when it comes to the quicker job start up time


Create some tests

[ ] grab some sample CSV files to test against
[ ] import CSV to the database

So what are the things that fail the most on the parser?
Realistically...not much. It's really resilient.
Maybe a better purpose for tests is to make it easier to swap out components.

Use PHPUnit.


Getting data from old commits

I have installed the latest version 0.6.0, but it doesn't correspond to the latest commit. That's why I am getting old data. Kindly update version on npm to fix this.
Got the following log when I tried yarn intall iso-country-currency@fcad24696ef16eeff67f1be3993e9f8850c4fdde
Couldn't find any versions for "iso-country-currency" that matches "fcad24696ef16eeff67f1be3993e9f8850c4fdde"

Hi @usmanghani-em,
I've updated the version, please let me know if the issue is fixed

@AlexDid fixed, thank you very much 👍🏻


create_database(ctx, database_name; source=source_database_name) fails with a 404, despite the source DB existing

The below script will replicate the issue:
using RAI
using UUIDs

ctx = Context(load_config())
source_db_name = "source-db-$(uuid1())"
clone_db_name = "clone-db-$(uuid1())"
engine_name = "engine-$(uuid1())"

current_db_names = [d.name for d in list_databases(ctx)]
@assert !in(source_db_name, current_db_names)
@assert !in(clone_db_name, current_db_names)

function blocking_create_database(ctx, database_name, engine_name; source=nothing)
    is_term_state(state) = state == "CREATED" || occursin("FAILED", state)
    rsp = create_database(ctx, database_name, engine_name; overwrite=true, source=source)
    # Only continue when the database is finished creating.
    while !is_term_state(get(rsp, "state", ""))
        sleep(3)
        rsp = get_database(ctx, database_name)
    end
    @info "Created DB $database_name"
end

function blocking_create_database(ctx, database_name; source=nothing)
    is_term_state(state) = state == "CREATED" || occursin("FAILED", state)
    rsp = create_database(ctx, database_name; source=source)
    # Only continue when the database is finished creating.
    while !is_term_state(get(rsp, "state", ""))
        sleep(3)
        rsp = get_database(ctx, database_name)
    end
    @info "Created DB $database_name"
end

function blocking_create_engine(ctx, engine_name)
    is_term_state(state) = state == "PROVISIONED" || occursin("FAILED", state)
    rsp = create_engine(ctx, engine_name)
    while !is_term_state(get(rsp, "state", ""))
        sleep(5)
        rsp = get_engine(ctx, engine_name)
    end
    @info "Created engine $engine_name"
end

blocking_create_engine(ctx, engine_name)

# These two calls work, but raise a deprecation warning to stop using an engine parameter
blocking_create_database(ctx, source_db_name, engine_name)
blocking_create_database(ctx, clone_db_name, engine_name; source=source_db_name)

current_db_names = [d.name for d in list_databases(ctx)]
@assert in(source_db_name, current_db_names)
@assert in(clone_db_name, current_db_names)

delete_database(ctx, source_db_name)
delete_database(ctx, clone_db_name)

current_db_names = [d.name for d in list_databases(ctx)]
@assert !in(source_db_name, current_db_names)
@assert !in(clone_db_name, current_db_names)

# The first call works, but the second fails
blocking_create_database(ctx, source_db_name)
try
    blocking_create_database(ctx, clone_db_name; source=source_db_name)
catch e
    @error "I fail with a 404 error" exception = e
end

# But the source_db_name DB does exist!
current_db_names = [d.name for d in list_databases(ctx)]
@assert in(source_db_name, current_db_names)
@assert !in(clone_db_name, current_db_names)

Aha, good find!
It turns out that this is actually not an issue with the SDK, it's due to a current limitation in the RAI system itself: https://github.com/RelationalAI/relationalai-infra/issues/2542. Due to this limitation, you currently cannot clone from a DB until you have "touched" it, by issuing at least one query against it.
We need to very clearly document this limitation in the SDK docstring for create_database(). I will co-opt this issue to discuss that instead.

Note: This issue has been migrated to https://relationalai.atlassian.net/browse/RAI-6568.
This link is only accessible to employees of RelationalAI.


Lint/MixedRegexpCaptureTypes fails for multiline regex with comments

Expected behavior
I would expect Rubocop to not flag multi-line regular expressions with comments as invalid.
Actual behavior
End up with the following error:
Lint/MixedRegexpCaptureTypes: Do not mix named captures and numbered captures in a Regexp literal.

Steps to reproduce the problem
You can recreate the issue as follows:
# frozen_string_literal: true

SUCCESS = /
  (?<key>\A.+)
/x.freeze

FAILURE = /
  (?<key>\A.+)  # Key (anchored to start of line).
/x.freeze

If you save the above to example.rb and run rubocop example.rb, you'll get the following error:
Inspecting 1 file
W

Offenses:

example.rb:3:19: W: Lint/MixedRegexpCaptureTypes: Do not mix named captures and numbered captures in a Regexp literal.
PATTERN_FAILURE = / ...
                  ^

1 file inspected, 1 offense detected, 1 offense auto-correctable

RuboCop version
0.92.0 (using Parser <IP_ADDRESS>, rubocop-ast 0.7.1, running on ruby 2.7.2 x86_64-darwin19)

I think this was fixed by #8803 already!

:bow: @dvandersluis Perfect. I was searching for open/closed issues related to Lint/MixedRegexpCaptureType but didn't see that at all. I look forward to the next release.


Out of memory - 1060 6GB while Sampling

RuntimeError: CUDA error: out of memory
I am running sampling for birds model (python main.py --cfg cfg/eval_bird.yml --gpu 0)

try to kill other processes running on same GPU, else you have to increase RAM.

Change the batch size defined in cfg/eval_bird.yml.
Updated parameters :
TRAIN:
    FLAG: True
    BATCH_SIZE: 16

Change the batch size defined in cfg/eval_bird.yml.
Updated parameters :
TRAIN:
    FLAG: True
    BATCH_SIZE: 16

should I set flag true? Does it means I need to retrain the nerwork? but I just need to running a sample

Change the batch size defined in cfg/eval_bird.yml.
Updated parameters :
TRAIN:
    FLAG: True
    BATCH_SIZE: 16

should I set flag true? Does it means I need to retrain the nerwork? but I just need to running a sample

Should I set it as true? Same question

RuntimeError: CUDA error: out of memory
I am running sampling for birds model (python main.py --cfg cfg/eval_bird.yml --gpu 0)

Did you solve it?


saved dart code automatically reset

Issue Type: Bug
If saved a code and worked fine then opened next day , it will not working.
VS Code version: Code 1.69.2 (Universal) (3b889b090b5ad5793f524b5d1d39fda662b96a2a, 2022-07-18T16:12:57.074Z)
OS version: Darwin arm64 21.5.0
Restricted Mode: No
Extensions (8)

Extension
Author (truncated)
Version

flutter-snippets
ale
3.0.0

dart-code
Dar
3.44.0

flutter
Dar
3.44.0

bloc
Fel
6.6.1

pubspec-assist
jer
2.3.2

vscode-lottie
lot
1.0.5

hive-object-converter
mrg
1.0.1

material-icon-theme
PKi
4.19.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411cf:30525689
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vscscmwlcmt:30465135
cppdebug:30492333
pylanb8912:30529769
vsclangdc:30486549
c4g48928:30535728

/needsMoreInfo
Please provide exact steps to reproduce the issue.

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

@gjsjohnmurray Please checkout this video ,no error warnings for without library import and writing new code and running old cached code.
https://user-images.githubusercontent.com/63861163/181695067-06f2ca87-68ef-445a-9f4a-eeb820f764cf.mp4

I think your repro depends on having certain extensions installed. If so, please report the issue to the author of the relevant extension.
If you have a repro that demonstrates the issue even when all user-installed extensions are disabled, please post it here as step by step instructions.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines.
Happy Coding!

@gjsjohnmurray it's issue with GitHub connection , no extension problem, GitHub is using without extensions , right ?  issue reproduce when connect GitHub

@akshaynjarangal it's not clear from the gif what the problem is, but regardless, this seems related to the dart extension you have installed. I'm not sure where GitHub comes into this as you suggest.


ImportError: cannot import name 'AbstractSummaryAction'

Hello, I received an approval to use the gated preview of Azure's Abstractive Summarization and I'm following the sample code from here - https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/textanalytics/azure-ai-textanalytics/samples/sample_abstract_summary.py
However I get an import error:
from azure.ai.textanalytics import (
ImportError: cannot import name 'AbstractSummaryAction'
versions used:
azure-ai-textanalytics 5.2.0b4
azure-common           1.1.28
azure-core             1.24.2
I also tried the --pre version and it still doesn't import 'AbstractSummaryAction'

pip install azure-ai-textanalytics==5.3.0b1

This doesn't work, here is the error:
Could not find a version that satisfies the requirement azure-ai-textanalytics==5.3.0b1 (from versions: 1.0.0b1, 1.0.0b2, 1.0.0b3, 1.0.0b4, 1.0.0b5, 1.0.0b6, 1.0.0, 5.0.0, 5.1.0b1, 5.1.0b2, 5.1.0b3, 5.1.0b4, 5.1.0b5, 5.1.0b6, 5.1.0b7, 5.1.0, 5.2.0b1, 5.2.0b2, 5.2.0b3, 5.2.0b4)
No matching distribution found for azure-ai-textanalytics==5.3.0b1
Please note I'm using a python virtual environment.

@csaras84 which version of Python are you using? azure-ai-textanalytics==5.3.0b1 requires Python 3.7 or greater.

@csaras84 glad to hear this was resolved. I'm going to close out this issue since I see you opened a new issue regarding the difference in results. I will respond on that issue https://github.com/Azure/azure-sdk-for-python/issues/27683


Split components into separate repos

First, thanks so much for all of this -- these components are super valuable.
I'm sure there are very good reasons to have this all in one repo, and I know splitting it up would be pretty annoying to do. But NPM maintainers have steadfastly refused to support monorepo layouts, making it difficult to work with/on the individual components and include them in other projects

Won't fix, if anyone wants a master build, can clone the repo + build. I place the burden on developers that need that in those case since it would be unmaintainable for me to have 30 different repos

Fair enough, thanks for the reply/explanation


fix the integration testing framework for devs not using english-based systems.

Context and purpose of the change
This pull request fixes the testing framework for devs not using english-based systems.
An integration test failed when the framework is used with the local numerical formatting variable not set to en_US.
This PR adds and set explicitly the LC_NUMERIC variable to overcome this issue.
Brief Changelog
Set LC_NUMERIC=en_US.UTF-8 in dockernet/config.sh so this is used by integration_tests.bats.
Author's Checklist
I have...

[x ] Run and PASSED locally all GAIA integration tests
[ ] If the change is contentful, I either:

[ ] Added a new unit test OR
[ ] Added test cases to existing unit tests

[ ] OR this change is a trivial rework / code cleanup without any test coverage

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] manually tested (if applicable)
[ ] confirmed the author wrote unit tests for new logic
[ ] reviewed documentation exists and is accurate

Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[ ] Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md?
[ ] This pull request updates existing proto field values (and require a backend and frontend migration)?
[ ] Does this pull request change existing proto field names (and require a frontend migration)?
How is the feature or change documented?

[ ] not applicable
[ ] jira ticket XXX
[ ] specification (x/<module>/spec/)
[ ] README.md
[ ] not documented

The failing test is  [INTEGRATION-BASIC-GAIA] rewards are being reinvested, exchange rate updating .
The error is:
 (in test file dockernet/tests/integration_tests.bats, line 323)
redemption_rate_increased=$(( $(FLOOR $(DECMUL $redemption_rate $MULT)) > $(FLOOR $(DECMUL 1.00000000000000000 $MULT))))' failed
printf: 1011943.883549492794000000: invalid number
printf: 1000000.00000000000000000: invalid number

Test fails because of the separator (English uses dot as separator, Italian and others use comma as separator). Forcing English as local setting for the test configuration solves this issue.


Hprotoc build should be fixed

I added hprotoc-building to travis and fixed it, so should be good now

No, it's not:
[ 1 of 13] Compiling Text.ProtocolBuffers.ProtoCompile.Identifiers ( Text/ProtocolBuffers/ProtoCompile/Identifiers.hs, dist/build/Text/ProtocolBuffers/ProtoCompile/Idntifiers.o )

Text/ProtocolBuffers/ProtoCompile/Identifiers.hs:90:24:
    Ambiguous occurrence ‘uncons’
    It could refer to either ‘Text.ProtocolBuffers.ProtoCompile.Identifiers.uncons’,
                             defined at Text/ProtocolBuffers/ProtoCompile/Identifiers.hs:102:3
                          or ‘Data.List.uncons’,
                             imported from ‘Data.List’ at Text/ProtocolBuffers/ProtoCompile/Identifiers.hs:23:1-16
                             (and originally defined in ‘GHC.List’)
cabal: Error: some packages failed to install:
hprotoc-2.1.2 failed during the building phase. The exception was:
ExitFailure 1

I'm going to have to delay a week or so before trying this again, constant build failures are really causing trouble.

@snoyberg sorry, I'm only gaining experience with all this right now. Forgot to do a hackage release after I fixed it in master, will do now.

@snoyberg released 2.1.3 which has this fix inside


Add check that hot-switching from default to alternate Source is "safe"

When ProjectIdentifier.Source is empty for a given project, it's possible for a later dependency on that same project to come along and specify a ProjectIdentifier.Source, and the solver will transparently allow the switch to the alternate source location.
This is useful because it allows (one) project to specify an alternate source location - primarily, a fork - and have other dependers on that project just go along for the ride. Not having this feature would make specifying a fork of a shared dep nearly impossible; all the dependers on it would need to explicitly specify the fork. However, right now, we don't really do any checks to ensure that this hot-switching of source locations is actually safe - we just make a simple check, and off we go.
TBH, I'm not even sure what "safe" would even mean here, as the express purpose of having a fork is to use a modified version of the code, which necessarily means relying on a different version. We need better, clearer definitions.

This issue was moved to golang/dep#428


Extensible::constructExtensions() is easy to omit and silently fails

We should change this to either:

lazy-call on ->extend() or ->__call()
Trigger an exception if either of the above is invoked without ->constructExtensions() being invoked.

The same issue exists, albeit to a lesser degree, with CustomMethods::defineMethods() method.

This method is now redundant as extensions are auto-constructed.


Elide cmdline

this should be merged after #1666 - the last commit is the only new change.
also requires https://github.com/rancher/os-kernel/pull/48
this is for #1537 - @janeczku

@SvenDowideit need to obtain customer feedback

@SvenDowideit need clarification. Given a cmdline:
kernel ${base-url}/vmlinuz rancher.state.dev=LABEL=RANCHER_STATE rancher.state.autoformat=[/dev/sda] -- rancher.cloud_init.datasources=[url:http://example.com/cloud-config]

@janeczku It will need to be stored in the ros config - would they prefer all in one single string, or as an array? (i don't think we can do it as a tree)

@SvenDowideit

ok - as a proposal - what if we added the elided cmdline parts as a string to rancher.environment.CMDLINE_EXTRA - that way it can be added to services

Does that work with ad-hoc (cloud-config) services? If yes then this would be definitely helpful for them.

excellent (yes, I think the environment key works for both in os-config and service.yml based services - the only pain would be using system-docker run

next up, a test and doc example of how to get the EXTRA_CMDLINE into a service using the environment

ok, I'm going to merge this for 0.9.0-rc2 so we can try it out.

@SvenDowideit I don't think rancher.environment k/v are passed to the console service environment currently. Do you think it makes sense though?

IDK - is there a reason why we'd want the non-root user in the console to access this information by default?

@SvenDowideit yeah better to leave the current behavior not setting it in the container env.


Thank you!

Thanks for the integration. Works well for my heat pump "WPE-I 08 HK230 Premium". :-)

Mann thanks for the kind worden.


Create VisualizedPath add-on for Godot 4.0

Create a add-on/plugin for Godot4 that can easily create a visualized path, for game applications, such as showing intended direction or actions in turn based games.  The path must support texturing to allow material shaders that can communicate direction as well as decoration of beginning and ending of the path.
The path must be able to have an "up" side and be able to bend, or turn between points to maintain that "up" at each point.

Initial creation of the asset has been completed.

Accepted by the Asset Lib moderators, and verified that downloads and examples work when pulled from the Asset Library


Add release notes for Chocolatey Extension 4.2.0

Description Of Changes
Adds release notes for Chocolatey Licensed Extension 4.2.0.
Motivation and Context
There is a new release of Chocolatey Licensed Extension that needs to. be documented.
Testing

[x] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[ ] Minor documentation fix (typos etc.).
[x] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[ ] New documentation page added.

Change Checklist

[ ] Requires a change to menu structure (top or left hand side)/
[ ] Menu structure has been updated

Related Issue
n/a

This is currently referencing an xref added in #480. Once that's merged I'll rebase this for review.

@corbob I have made a new commit into this PR branch, as well as a few suggestions. Let me know if you have any questions.


I don't know exactly how to use this tool

Pls can someone tell me how to fix this problem
and please modify the readme file for better understanding for newbies like me.

pip install python-slugify in console.
Follow instructions to get cookie, add it to example.py.
For more convenient way than editing example.py every time you can comment line 10 (add # at the start), and add this line at the end:
dl.download_course_by_class_id((input("Course id: "))
ID is a 9 digit number at the end of URL of course.

yes it works! How can I download subtitles also.

Execute pip install python-slugify in console.
Follow instructions to get cookie, add it to example.py.
For more convenient way than editing example.py every time, you can comment line 10 (add # at the start) and add this at the end:
while True:
    dl.download_course_by_class_id((input("Course id: "))

ID is a 9 digit number at the end of URL of course. You can paste it to the console with right mouse click.

yeah . Thanks. it works! Can you pls tell me how to download subtitles also?

Execute pip install python-slugify in console.
Follow instructions to get cookie, add it to example.py.
For more convenient way than editing example.py every time, you can comment line 10 (add # at the start) and add this at the end:
while True:
    dl.download_course_by_class_id((input("Course id: "))

ID is a 9 digit number at the end of URL of course. You can paste it to the console with right mouse click.

I'm getting this error while using while loop command:

Guys please help me. I am just trying to download my class. I dont know anything about coding. I spent an hour to find out I need python. Please help and tell me how exactly I need to the thing to download my steps.


DOC: outline cibuildwheel on ci

Addresses one of the points in https://github.com/scipy/scipy/pull/16842#issuecomment-1224058895 by describing when the wheels GH Action runs.

Thanks @andyfaff. The CI error was caused by the branch being too far behind main - I've pushed a rebase.


Second pull request before jumping to use templates and Plotly

It has a more stylized look in the plot visualization part of the app. I added 9 additional plots and removed 3 unnecessary ones. Url navigation is also fully implemented with a very basic but functioning error handling.

Closing, this has been superseded by #887.


Grid class system is not working for Nebular Theme v4.0.0

Issue type
I'm submitting a ...  (check one with "x")

[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Issue description
Current behavior:
I have placed the .row and .show-grid class on two cards in my content layout. However, the
Expected behavior:
I expect for the two cards to be laid side-by-side on screens size tablet and larger. And only stack when the screen goes smaller (<768px).
Steps to reproduce:
You can see the problem in the StackBlitz sample shown below. The BaristaMachineComponent template has a container with the Grid classes but nothing is happening.
Related code:
https://stackblitz.com/github/guillermoarellano/ngx-baristamatic-demo/tree/324a61dab67ea04a5a2d48b9be3a4a2faf33fc69?file=src%2Fapp%2Fbarista-machine%2Fbarista-machine.component.html

    <nb-card [size]="'medium'">
      <nb-card-header>Drinks Menu</nb-card-header>
      <nb-card-body>

          <button nbButton
                  (click)="handleMakeDrink(drink.name)"
                  [disabled]="!drink.inStock">{{drink.name}}, {{drink.cost}}</button>

      </nb-card-body>
    </nb-card>

    <nb-card [size]="'medium'">
      <nb-card-header>Inventory</nb-card-header>
      <nb-list>
        <nb-list-item *ngFor="let item of inventory">
          {{item.ingredient}} - {{item.quantity}}
        </nb-list-item>
      </nb-list>
    </nb-card>
    <button nbButton status="warning" (click)="handleRestockInventory()">Restock inventory</button>

Other information:
npm, node, OS, Browser

Angular, Nebular
    "@angular/animations": "~8.0.0",
    "@angular/cdk": "^8.0.0",
    "@angular/common": "~8.0.0",
    "@angular/compiler": "~8.0.0",
    "@angular/core": "~8.0.0",
    "@angular/forms": "~8.0.0",
    "@angular/platform-browser": "~8.0.0",
    "@angular/platform-browser-dynamic": "~8.0.0",
    "@angular/router": "~8.0.0",
    "@nebular/eva-icons": "4.0.0",
    "@nebular/theme": "^4.0.0",
    "eva-icons": "^1.1.1",

can you update angular8??

Hi Yuki,
I have the latest version of Angular 8 already. Can you see my StackBlitz code base? Have you seen this issue before? I have never had problems when using the Bootstrap CSS library and it's Grid classes. Wondering if I configured the app correctly when I installed it. I used the Angular CLI ng add command install Nebular theme.
Guillermo

Thanks for your relpy,Because I try to use branch (feat updatet-to 8) The style is very strange，
I want to update old project, not create new project ,

add<EMAIL_ADDRESS>

git clone  ngx-admin
then
1: add<EMAIL_ADDRESS>2:add @nebular/theme
3: others  ,so ??

add<EMAIL_ADDRESS>add @nebular/theme
add<EMAIL_ADDRESS>if using auth and security, you can add @nebular/security and @nebular/auth.
Ensure that you using Node 12 or latest.

Resolved:
The solution was to import both @nebular/bootstrap and regular bootstrap into the project. And then modify the src/styles.scss pull in the Bootstrap mixins and grids.

npm install bootstrap @nebular/bootstrap
Modify styles.scss

See this file -- https://stackblitz.com/github/guillermoarellano/ngx-baristamatic-demo/tree/fix-bootstrap-grid?file=src%2Fstyles.scss

Thanks to @LeonardHorng for the tip. 👍

Anybody know how to submit updates/enhancements to the documentation?

<EMAIL_ADDRESS>is deprecated, should we import regular boostrap 4 files?

Cant use the grid too.. what i need to do?

Cant use the grid too.. what i need to do?

@dcruz1990
I just imported regular bootstrap 4 css, and it can live among the @nebular project wthout problems. so you can be able to use grid system


Set required proto fields for ruletypes

Summary
Ref #4952
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[x] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 55.225% (-0.006%) from 55.231%
when pulling b884a186f9dda8b0c163153bdcb51a945890f4c4 on eleftherias:openapi-ruletypes
into 1eb3600ccf32b9e57b3b85ac927fd392bb9a546d on mindersec:main.


Fix license badge link in README

Fixes the license badge link in the README.

thanks for the typo fix 👌🏾


Temporary claim value cannot be set as the sub claim in the ID token, however, it is set it in the self-contained access token.

Describe the issue:
Using adaptive authentication, temporary claim values can be set for a claim as shown below(useful when userstore is readonly),
user.claims['http://wso2.org/claims/im'] = "test123";

This temporary claim value cannot be set as the sub claim in the ID token, however, it is being set in the self-contained access token.
How to reproduce:

Get a WUM updated IS-5.10.0 pack, Setup an OIDC service provider with the token issuer JWT.
In the requested claims add the http://wso2.org/claims/im as a requested claim.
As the Subject Claim URI add http://wso2.org/claims/im.
Add the following adaptive script.

 var onLoginRequest = function(context) {
     executeStep(1, {
         onSuccess: function (context) {
                       var user = context.currentKnownSubject;
                       user.claims['http://wso2.org/claims/im'] = "test123";
                       Log.info("claim"+ user.claims['http://wso2.org/claims/im']);
         }
     });
 };

Initiate a request to get an access token and id token(grant types tried implicit, code). Check the difference in the sub value in the tokens.

Even though a temp value for the subject claim is set, the value for the claim is being retrieved from the user store. In code[1], we try to get it from a cache, however, there is seems to be no subject stored in the cache.
Expected behavior:
The temporary value which is set for the claim should be set as the sub in the id token, like in the self-contained access token to be consistent.
Environment information :

Product Version: [IS 5.10.0]

[1]https://github.com/wso2-extensions/identity-inbound-auth-oauth/blob/master/components/org.wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/openidconnect/DefaultIDTokenBuilder.java#L326

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Text copy paste issue

When you copy some text and if in the there is '<' this bracket then the following word or line is not copied.

#include < iostream >
This line should be pasted but instead of this it only shows
#include
I have to write  separately or any thing which is followed by '<' .
In fact in this issue report page its not showing the word or line followed by '<', that is why I have to write < iostream > this with a space in between.
I m using this on chrome on my laptop.

use safari or the darkweb especially sector 4 in the coding 9th projection

The words are actually limited and it depends on the font you're using. Some fonts support < and some don't and if it doesn't support that letter then the fallback is rendered.
If you want to use that bracket, unfortunately you will have to find a font that supports it.


Added styling to default template

I've just noticed that the GDPR link was removed from the current page. Would it be possible to add it back in? Thank you.


Arm64 architecture missing

Getting the same error(s). Is there a fix for this:

Hello,
Did you use git-lfs as described in documentation here: https://github.com/blinkinput/blinkinput-ios/blob/master/README.md#integration-without-cocoapods
Regards

I downloaded the zip file.

You're best bet is to install it manually. Download the framework, add it to your project, add the necessary libraries into your project file like so:

Already done:


Date modified: Handle empty slot

Freshdesk ticket
Issue
gcds-date-modifed renders when no date is available
Recommendation

Display console message
Suppress render

re: freshdesk ticker 18992
(following so I can advise the client when complete :D)

Needs review but ready to be done

Still to be released: https://github.com/cds-snc/gcds-components/pull/668


Testing

[x] Add Test runner
[ ] Site tests
[ ] Browse tests
[x] Login tests
[ ] Profile tests
[ ] Base tests
[ ] Datastream tests
[ ] Home
[ ] GoogleMap

Bug Fixes & Improvements

[x] Pinia store not being removed on logout
[x] Table pagination
[x] Token refresh not working since Maurier's May19th commits
[x] Make user organization changes reactively
[x] GoogleMap component shouldn't reload on each filter keypress
[x] GoogleMap doesn't show on Single Site page after user updates their site
[x] GoogleMap default Africa site still shows on load
[x] If Browse page filter returns no results, display message and show no results
[x] Browse clear button should work on every press, not like a toggle switch
[x] Manage metadata page needs to await for API data
[x] On account creation, user doesn't get logged in automatically
[x] SingleSite page requires datastream ownership, whcih shouldn't be the case
[x] When creating a new processing level, the select dropdown shows the id
[ ] Unit dropdown loads too slow
[x] List sorting
[x] User shouldn't have edit access to ownerless datastream metadata
[ ] Make datastream name & description concatenation of information
[ ] Anonymous user is always directed to the home page because of the interceptor. There needs to be allowance on pages that allow guests

The list rendering performance issue seems to be with Vuetify itself - https://github.com/vuetifyjs/vuetify/issues/16318
I'll wait until the next version of Vuetify to see if that fixes it


Great Guide.

Love the guide! I'm building a Collaborative project, and this was a great stepping stone. I think it would be good to have a small section that steps through how to implement this onto your own model/project as well.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4a79cabb-52f4-1cd5-d473-174391c73de3
Version Independent ID: fa30affe-a552-a265-6626-855b2326bf02
Content: Holograms 240 - Mixed Reality
Content Source: mixed-reality-docs/holograms-240.md
Service: windows
GitHub Login: @keveleigh
Microsoft Alias: kurtie

@keveleigh @davidkline-ms do you know if there's anything we could do to update the course with tips on how you can sample the source code to apply it to your own project? Or does MRTK have snippets you can use outside of the full app codebase?

I would also be interested in something like this. I'm trying to tweak parts of the tutorial to fit my project's needs and it's turning out to be a lot more difficult than I had anticipated.

+1 :D

Closing this one as HoloToolkit is deprecated, but if anyone has follow-ups please post and we'll reopen. Thanks!