Trust dialog prompt appears even when disabled in settings

Type: Bug

Prerequisites:
Have the setting "Security -> Workspace -> Trust: Enabled" active, and the setting "Security -> Workspace -> Trust: Startup Prompt" set to "never".

Steps:

Open Visual Studio and wait a moment.

Expected result:
No trust dialog should appear.

Actual result:
The trust dialog appears.

Basically I can't make the trust dialog not appear every time I open VS Code or when I open a workspace. This has started happening when updating to version 1.71.0, it didn't happen with the previous version (I can't remember which one, but it shouldn't have been a very old one). However, this has happened at least once before, after some update (I can't remember which); I only was able to fix it by reinstalling VS Code, no update of the settings managed to do the trick. I didn't change any setting (or basically anything) before the issue started happening again.

VS Code version: Code 1.71.0 (784b0177c56c607789f9638da7b6bf3230d47a8c, 2022-09-01T07:36:10.600Z)
OS version: Windows_NT x64 10.0.19044
Modes: Restricted
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-10900KF CPU @ 3.70GHz (20 x 3696)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.86GB (10.70GB free)

Process Argv
--crash-reporter-id 37fe75ef-cce2-4d86-8391-c6c5553b1e3f

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

hexeditor
ms-
1.9.8

I tried to reinstall vscode, but I still see the trust pop-up window.

@guation, this workspace trust prompt is coming from the tasks service. As the dialog states, listing and running tasks can result in workspace code being executed so the task service asks for workspace trust. Assuming that your trust the workspace files, if you click "Trust Workspace & Continue" you should not see this dialog again.
Adding @meganrogge to see if there is a way to reduce the number of prompts.

@lszomoru So the "Trust: Startup Prompt" setting actually has no effect on the prompt triggered by tasks, and there's no equivalent setting to disable it (or at least to disable listing tasks)?

I believe this is already fixed, can you see if you can reproduce in the Insiders build?
/label info-needed

I believe this is already fixed, can you see if you can reproduce in the Insiders build?

Yes, I can reproduce it in the insiders build, this version:

Version: 1.72.0-insider (user setup)
Commit: 2d27f8db6a19f93529787731b0efa8f001478f6d

/duplicate #156183

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


test failure

https://drone-publish.rancher.io/rancher/rancher/122/1/2
=================================== FAILURES =================================== | 478s
-- | --
8862 | ____________________ test_pod_security_policy_template_del _____________________ | 478s
8863 | [gw2] linux -- Python 3.7.1 /go/src/github.com/rancher/rancher/tests/.tox/py37/bin/python3.7 | 478s
8864 |   | 478s
8865 | admin_mc = <core.conftest.ManagementContext object at 0x7fe2978bb978> | 478s
8866 | admin_pc = <core.conftest.ProjectContext object at 0x7fe29405c390> | 478s
8867 | remove_resource = <function remove_resource.<locals>._cleanup at 0x7fe2942c1620> | 478s
8868 |   | 478s
8869 | def test_pod_security_policy_template_del(admin_mc, admin_pc, remove_resource): | 478s
8870 | """ Test for pod security policy template binding correctly | 478s
8871 | ref https://github.com/rancher/rancher/issues/15728 | 478s
8872 | ref https://localhost:8443/v3/podsecuritypolicytemplates | 478s
8873 | """ | 478s
8874 | api_client = admin_mc.client | 478s
8875 | # these create a mock pspts... not valid for real psp's | 478s
8876 |   | 478s
8877 | pspt_proj = create_pspt(api_client) | 478s
8878 | # add a finalizer to delete the pspt | 478s
8879 | remove_resource(pspt_proj) | 478s
8880 |   | 478s
8881 | # creates a project and handles cleanup | 478s
8882 | proj = admin_pc.project | 478s
8883 | # this will retry 3 times if there is an ApiError | 478s
8884 | api_client.action(obj=proj, action_name="setpodsecuritypolicytemplate", | 478s
8885 | > podSecurityPolicyTemplateId=pspt_proj.id) | 478s
8886 |   | 478s
8887 | test_pod_security_policies.py:108: | 478s
8888 | _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ | 478s
8889 | ../.tox/py37/lib/python3.7/site-packages/rancher.py:471: in action | 478s
8890 | return self._post_and_retry(url, *args, **kw) | 478s
8891 | ../.tox/py37/lib/python3.7/site-packages/rancher.py:430: in _post_and_retry | 478s
8892 | raise e | 478s
8893 | ../.tox/py37/lib/python3.7/site-packages/rancher.py:423: in _post_and_retry | 478s
8894 | return self._post(url, data=self._to_dict(*args, **kw)) | 478s
8895 | ../.tox/py37/lib/python3.7/site-packages/rancher.py:68: in wrapped | 478s
8896 | return fn(*args, **kw) | 478s
8897 | ../.tox/py37/lib/python3.7/site-packages/rancher.py:319: in _post | 478s
8898 | self._error(r.text) | 478s
8899 | _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ | 478s
8900 |   | 478s
8901 | self = <rancher.Client object at 0x7fe297920198> | 478s
8902 | text = '{"baseType":"error","code":"ServerError","message":"error updating PSPT ID: Operation cannot be fulfilled on projects...bject has been modified; please apply your changes to the latest version and try again","status":500,"type":"error"}\n' | 478s
8903 |   | 478s
8904 | def _error(self, text): | 478s
8905 | > raise ApiError(self._unmarshall(text)) | 478s
8906 | E rancher.ApiError: (ApiError(...), 'ServerError : error updating PSPT ID: Operation cannot be fulfilled on projects.management.cattle.io "p-l8wjx": the object has been modified; please apply your changes to the latest version and try again\n\t{\'baseType\': \'error\', \'code\': \'ServerError\', \'message\': \'error updating PSPT ID: Operation cannot be fulfilled on projects.management.cattle.io "p-l8wjx": the object has been modified; please apply your changes to the latest version and try again\', \'status\': 500, \'type\': \'error\'}') | 478s
8907 |   | 478s
8908 | ../.tox/py37/lib/python3.7/site-packages/rancher.py:299: ApiError

How to replicate via stress test:
add dummy parametrize flag
@pytest.mark.parametrize("count",range(1,500))
def test_pod_security_policy_template_del(admin_mc, admin_pc, remove_resource,count):

run pytest -n 10 -s -vv -k test_pod_security_policy_template_del
Observe test failures after some time.

nothing to test, closing


activity.isDestroyed() is not available in API 16

Describe the bug
activity.isDestroyed() is not available in API 16. This change was introduced in a recent PR. Attached patch fixes it.
react-native-fast-image+7.0.2.patch.txt
To Reproduce
Steps to reproduce the behavior if possible, or a link to a reproduction repo:

Build app
Run on API 16
App crashes

Expected behavior
App should not crash
Screenshots
If applicable, add screenshots to help explain your problem.
Dependency versions

React Native version: 0.59.10
React version: 16.8.3
React Native Fast Image version: 7.0.2

Note:  I had to use 7.0.2 because I'm using jetifier for another library.

#552 appears same as the patch here

#552 appears to be the same as the patch in this issue


Does CORS-Anywhere work with URLs that are "protected" by web access control products like Oracle OAM, CA Sitemender, etc.?

Hi,
We have a number of situations where our users use (XHR/Fetch) clients to access resources (URLs) that are on different domains, and where those resources are "protected" by something like a "web agent" (e.g., Oracle OAM webgate, CA Siteminder webagent, etc.).  These web agents typically use redirects to cause the incoming browser request to produce a request to a different URL, which then communicates with the web access control product's server, so something like, in the case of these XHR clients:
XHR client (in browser) ==> Request to protected URL (in a different domain than the server that served the client code)
Access Product Web agent ==> Sends 302/redirect to client to a different Access product endpoint
XHR client follows the redirect (this request would have "Origin: null" due to the redirect)
Access product server consumes the request, "authenticates" the user, and sends 302/redirect to client, together with some Set-Cookie
XHR client ==> Request to protected URL but with Access product cookies
The above flow is somewhat high-level, but would a CORS-Anywhere server work with this scenario?
Thanks,
Jim

Hi,
I just found this on the help on the demo page:
Redirects are automatically followed. For debugging purposes, each followed redirect results
in the addition of a X-CORS-Redirect-n header, where n starts at 1. These headers are not
accessible by the XMLHttpRequest API.
After 5 redirects, redirects are not followed any more. The redirect response is sent back
to the browser, which can choose to follow the redirect (handled automatically by the browser).

But the README.md on the github project page says
This package does not put any restrictions on the http methods or headers, except for cookies. Requesting user credentials is disallowed. The app can be configured to require a header for proxying a request, for example to avoid a direct visit from the browser.
The protocols for the web access control products also rely on sending cookies and also query parameters during the authentication process, so do you think the out-of-box CORS-Anywhere would work?
Also I wanted to test, using your demo, but when entering the URL to the demo I am getting this:
GET http://test.whatever.com:7777/target/index.html
404 Not Found

Not found because of proxy error: Error: getaddrinfo ENOTFOUND test.whatever.com

Is that because, to use the demo, that your demo needs to be able to resolve the hostname in the URL that we enter?
Thanks,
Jim

The protocols for the web access control products also rely on sending cookies and also query parameters during the authentication process, so do you think the out-of-box CORS-Anywhere would work?

No. It is not secure to enable cookies when the proxy is used to access multiple websites.

EDIT: I should mention that the "test.whatever.com" hostname is a hostname that is in the c:\windows\system32\drivers\etc\hosts file of the Windows workstation that I am running the browser from. I was hoping that the hostname in the URL that I entered into the demo page would get resolved by that hosts file, but it sounds like the hostname actually has to be resolvable by (maybe) your demo server itself? Is that the case?

A third-party server cannot look in your local hosts file. That would be quite a security issue on your end.
CORS Anywhere is a public proxy that can only access publicly accessible resources.
If you host CORS Anywhere within your intranet, then your instance would also be able to access those resources. But be very careful with access control: any website on a client in your network can then read any public (as in available without further authentication) resource within the network.

Also, can an IP address be used in the URL that is entered into the demo page? Or, must it be a FQDN?

An IP address or host name is valid. The list of valid TLDs is stored in https://github.com/Rob--W/cors-anywhere/blob/master/lib/regexp-top-level-domain.js

Hi,i
I have started testing now with a test scenario, where my Javascript/XHR app is using the CORS Anywhere double URL to access a resource/URL that is hosted in a different domain and the resource is protected by an OAM webgate.  I wasn't sure if I should put this post in this issue, or in the other "closed" issue, but decided it might fit better here?
This test is using CORS Anywhere that is deployed on one of my test servers.
Before I started testing with the protected resource, I have an almost identical "unprotected" test setup where the Javascript/XHR (in xhrtest/xhr-fakewava.html) is accessing a resource that is NOT protected, and when I test with this "unprotected" setup, the test works, i.e., the Javascript/XHR is able to retrieve the resource, using URL:
http://192.168.xxx.yy:8080/http://fakewava.whatever.com:7777/wavatarget/index.html
So then I made a new target resource, "wavatarget-charlieeastweb05/index.html" that is hosted on a machine that has an OAM webgate.
I use an almost identifical HTML page with the Javascript/XHR, "xhrtest/xhr-fakewava-protectedpage.html".  The only difference is the double-URL is different:
http://<IP_ADDRESS>:8080/http://charlieeastweb05......com:7777/wavatarget-charlieeastweb05/index.html
However, when I use the page with the XHR pointing to the protected resource, I get a 404 error, and in the browser web developer=>network=>Response, it has the following message:
Not found because of proxy error: Error: self signed certificate in certificate chain
I was searching the Issues and found issue 123, that mentions the same error, from that thread, it looks like that problem was fixed awhile ago?
As I mentioned above, with a WAM like OAM, when a resource is protected, and a request is made for the resource, OAM will cause a 302/redirect, and in fact, in the Apache access_log, the last request I see shows a 302 response and the Location is set to one of the OAM endpoints:
"+++LOCATION+++++ https://charlieeastweb04....com:14430/oam/server/.... +++++++++++++"
I am guessing that when I do this test (XHR accessing protected resource), the browser is being re-directed to that OAM URL and then the error that is being shown in the browser web developer=>network=>Response occurs (the "self signed certificate in certificate chain"), but I not sure why that would happen, because when I point the same browser directory to the protected resource URL, I get a cert popup and after selecting a certificate, I can access the page.
When that error occurs, can you tell me which component is getting the error?  Is it the CORS Anywhere itself?  Also which certificate chain is that error referring to?
Jim

Hi Rob,
I think I almost have CORS Anywhere working with a test OAM scenario, but:
I currently am still having to do the "export NODE_TLS_REJECT_UNAUTHORIZED='0'" to avoid the "self-signed certificate in chain" problem.  Is it possible to tweak the server.js or the CORS Anywhere code to import one of our CA certs so that I don't have to do that export?
I am not 100% sure yet, but for my test with the protected resource, it is getting through the most of the flow, but I am still getting an "ENOENT"/404 error at the end.  That error SEEMS to be saying that there is a problem with the hostname, but I stood up a new DNS server for this testing.
OAM tends to return a 404 error when authentication fails, so I don't know for sure if the 404 error is because of an authentication error, or if there is because of something else like the name resolution.
Another possibility is that the problem may be that cookies that are normally created as part of the OAM authentication (and which are used for authorization) are gone.  Actually at the end, the browser doesn't seem to have any cookies at all.  The lack of those cookies could also be causing the 404 error response.
Is there any way that I can modify the server.js (or maybe something else), to NOT drop the cookies?
Thanks,
Jim

FYI, after re-examining some pcap files that I captured earlier, I am seeing "hints" that the redirects are actually occurring.  I don't see (yet) the actual redirected requests themselves, but I am seeing the "X-CORS-Redirect-1" etc. response headers in one of the responses and also the "X-final-url" header.
I am guessing that the reason that I don't see the actual requests corresponding to those URLs is that I haven't configured Wireshark to decrypt the SSL yet, which I am attempting to do now.
I gather that the "x-final-url" means that is the final redirect in the chain of redirects?  If so, the URL in that "x-final-url" header should not be the last URL in the chain of redirects (there should be more non-SSL redirects after the 2 SSL redirects that I see now).
Jim

Hi Rob,
FYI, I wanted to update my situation:

I had come to the conclusion that the reason that I haven't been able to see all of the requests/responses in Wireshark was that our dev environment is on AWS and promiscuous monitoring doesn't work on AWS.
So, I am now setting up a new environment on VirtualBox.  I am almost done with that and I will try to recreate the problem and hopefully be able to actually see all the requests and responses, and I will post back here with more info.

Thanks,
Jim

If I removed the:
   'cookie',
    'cookie2',

Would that allow the cookies to not be dropped?

The cookie would not be dropped, but cookies are still stripped in the library. This is hard-coded at https://github.com/Rob--W/cors-anywhere/blob/70aaa22b3f9ad30c8566024bf25484fd1ed9bda9/lib/cors-anywhere.js#L213-L215
It is not secure at all to remove this, because it can result in leakage of credentials between proxied websites. I strongly advice against it, as I mentioned at https://github.com/Rob--W/cors-anywhere/pull/154#issuecomment-468649353

Hi Rob,
Here's an update.  I determined that the reason I wasn't able to see most of the request/response pairs before was because our dev environment is on AWS, and promiscuous monitoring doesn't work on AWS, so I have now put together a test environment that is running under VirtualBox.
I think I now have a scenario that is almost close to the scenario that we were having earlier, and I have been able to capture packet captures.
Even to get to this point, I had to add some Redirect in some (2 I think)  in my Apache, because requests were coming in with "Origin" request headers, but the responses did not have the CORs  response headers.
It also looks like there are two places where there are requests with "Origin" headers with values, where the response is a 401,  but the response doesn't have the CORS response headers.  I think that because those 2 requests have responses, but don't have the CORS response headers, that is causing the the authentication to fail (this scenario is using BASIC authentication).
Would it be all right to send you the PCAP file?
Thanks,
Jim

Hi,
I was able to find a different (what Oracle calls) "authentication scheme", which doesn't need redirects, so I changed the protection on the target URL in OAM to use that authentication scheme.  This authentication scheme is using HTTP BASIC authentication (where you get a popup window to enter username and password).
When I tested going directly (using a browser) to that protected resource, sure enough there are no redirects.  I get the BASIC popup, enter my username and password, and then the browser receives the protected page.
So I changed my test so that my Javascript/XHR does a GET on that protected URL with the CORS Anywhere URL (http://xxx:8080/) pre-pended to the protected URL.
However when I test that, I don't get the Basic popup.
Looking at the wireshark capture,  I see the 401 response that has the "www-authenticate: Basic realm=xxxx" response header, which is supposed to be what causes the browser to present the popup window, so I've been looking at the 401 response when using the javascript/xhr and CORS Anywhere vs. going directly to the protected URL using a browser.
Right now, the only thing that I see is:
1) 401 Response when request to protected resource is using Javascript/XHR:

Frame 355: 934 bytes on wire (7472 bits), 934 bytes captured (7472 bits) on interface \Device\NPF_{A65DD5E0-F324-4BF0-8115-255A8EC064BD}, id 0
Ethernet II, Src: PcsCompu_4e:e2:d7 (08:00:27:4e:e2:d7), Dst: PcsCompu_4d:6c:d9 (08:00:27:4d:6c:d9)
Internet Protocol Version 4, Src: <IP_ADDRESS>, Dst: <IP_ADDRESS>
Transmission Control Protocol, Src Port: 7777, Dst Port: 56016, Seq: 1, Ack: 413, Len: 868
Hypertext Transfer Protocol
    HTTP/1.1 401 Unauthorized\r\n
    Date: Sun, 03 Oct 2021 13:04:01 GMT\r\n
    Server: Apache/2.4.29 (Unix) OpenSSL/1.0.2k-fips\r\n
    Access-Control-Allow-Origin: http://centos-apache1.whatever.com:7777\r\n
    Access-Control-Allow-Credentials: true\r\n
    Access-Control-Allow-Methods: GET, POST, OPTIONS\r\n
    Access-Control-Allow-Headers: Origin, Content-Type, Accept\r\n

    WWW-Authenticate: Basic realm="ATNSCHEME-BasicSessionlessScheme"\r\n

    Content-Length: 381\r\n

    Connection: close\r\n

    Content-Type: text/html; charset=iso-8859-1\r\n
    \r\n
    [HTTP response 1/1]
    [Time since request: 10.115751000 seconds]
    [Request in frame: 188]
    [Request URI: http://centos-apache3.whatever.com:7777/oamprotectedtarget/index.html]
    File Data: 381 bytes
Line-based text data: text/html (12 lines)

2) 401 Response when request to protected resource is directly from browser to protected resource:

Frame 261: 889 bytes on wire (7112 bits), 889 bytes captured (7112 bits) on interface \Device\NPF_{A65DD5E0-F324-4BF0-8115-255A8EC064BD}, id 0
Ethernet II, Src: PcsCompu_4e:e2:d7 (08:00:27:4e:e2:d7), Dst: PcsCompu_a8:ad:d1 (08:00:27:a8:ad:d1)
Internet Protocol Version 4, Src: <IP_ADDRESS>, Dst: <IP_ADDRESS>
Transmission Control Protocol, Src Port: 7777, Dst Port: 55025, Seq: 1, Ack: 480, Len: 835
Hypertext Transfer Protocol
    HTTP/1.1 401 Unauthorized\r\n
    Date: Sun, 03 Oct 2021 13:06:29 GMT\r\n
    Server: Apache/2.4.29 (Unix) OpenSSL/1.0.2k-fips\r\n
    Access-Control-Allow-Credentials: true\r\n
    Access-Control-Allow-Methods: GET, POST, OPTIONS\r\n
    Access-Control-Allow-Headers: Origin, Content-Type, Accept\r\n

    WWW-Authenticate: Basic realm="ATNSCHEME-BasicSessionlessScheme"\r\n

    Content-Length: 381\r\n

    Keep-Alive: timeout=5, max=100\r\n
    Connection: Keep-Alive\r\n

    Content-Type: text/html; charset=iso-8859-1\r\n
    \r\n
    [HTTP response 1/1]
    [Time since request: 10.104103000 seconds]
    [Request in frame: 111]
    [Request URI: http://centos-apache3.whatever.com:7777/oamprotectedtarget/index.html]
    File Data: 381 bytes
Line-based text data: text/html (12 lines)

In the above, for the case where the request is from Javascript+XHR going through CORS Anywhere, to the protected resource, the 401 response has:
Connection: close
but when using a browser to go to the protected resource, the 401 response has:
    Keep-Alive: timeout=5, max=100\r\n
    Connection: Keep-Alive\r\n

I've been trying to configure the Apache that is hosting the protected URL (an Apache server).   I can get the Apache to inject the "Keep-Alive: timeout=5, max=100" response header using the Apache "Header" directive, but it seems like there is no way to replace the "Connection: close" with "Connection: Keep-Alive" (I can ADD to the Connection header, but I cannot remove the "close").
The reason that I am posting this is that I cannot determine for sure where the "Connection" response header is coming from.
I don't think it is from the Apache that is hosting the target page, because that doesn't change between the 2 different cases.
So I am wondering if it is possible that that "Connection: close" response header is being set in the response by CORS Anywhere?
Thanks,
Jim

Hi Rob,
I just found this thread in SO:
https://stackoverflow.com/questions/18499465/cors-and-http-basic-auth
and specifically the response from "Brock Allen" on Aug 29, 2013:
"If you're requesting credentials then the server must respond with the specific origin in the Access-Control-Allow-Origin response header (and thus can't use the wildcard *). Of course it would then also need to respond with Access-Control-Allow-Credentials response header too."
And then I checked the 401 response that is going back to the browser in my Wireshark captures, and that 401 response does have:
access-control-allow-origin: *
So perhaps that (because of the *) may be preventing the browser from popping up the login window?
I am not 100% sure where that response header is coming from, but I'm guessing that it may be from CORS Anywhere?
If so, could CORS Anywhere be able to send back a header that doesn't have "*", but rather the value from the original "Origin" request header?
FYI, here is the request captured:
Frame 165: 511 bytes on wire (4088 bits), 511 bytes captured (4088 bits) on interface \Device\NPF_{A65DD5E0-F324-4BF0-8115-255A8EC064BD}, id 0
Ethernet II, Src: PcsCompu_a8:ad:d1 (08:00:27:a8:ad:d1), Dst: PcsCompu_4d:6c:d9 (08:00:27:4d:6c:d9)
Internet Protocol Version 4, Src: <IP_ADDRESS>, Dst: <IP_ADDRESS>
Transmission Control Protocol, Src Port: 52934, Dst Port: 8080, Seq: 1, Ack: 1, Len: 457
Hypertext Transfer Protocol
    GET /http://centos-apache3.whatever.com:7777/oamprotectedtarget/index.html HTTP/1.1\r\n
    Host: centos-apache1.whatever.com:8080\r\n
    Connection: keep-alive\r\n
    User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/94.0.4606.61 Safari/537.36\r\n
    Accept: */*\r\n
    Origin: http://centos-apache1.whatever.com:7777\r\n
    Referer: http://centos-apache1.whatever.com:7777/\r\n
    Accept-Encoding: gzip, deflate\r\n
    Accept-Language: en-US,en;q=0.9\r\n
    \r\n
    [Full request URI: http://centos-apache1.whatever.com:8080/http://centos-apache3.whatever.com:7777/oamprotectedtarget/index.html]
    [HTTP request 1/1]
    [Response in frame: 181]

and here's the 401 response (to the BROWSER):
Frame 181: 1322 bytes on wire (10576 bits), 1322 bytes captured (10576 bits) on interface \Device\NPF_{A65DD5E0-F324-4BF0-8115-255A8EC064BD}, id 0
Ethernet II, Src: PcsCompu_4d:6c:d9 (08:00:27:4d:6c:d9), Dst: PcsCompu_a8:ad:d1 (08:00:27:a8:ad:d1)
Internet Protocol Version 4, Src: <IP_ADDRESS>, Dst: <IP_ADDRESS>
Transmission Control Protocol, Src Port: 8080, Dst Port: 52934, Seq: 1, Ack: 458, Len: 1268
Hypertext Transfer Protocol
    HTTP/1.1 401 Unauthorized\r\n
    x-request-url: http://centos-apache3.whatever.com:7777/oamprotectedtarget/index.html\r\n
    date: Mon, 04 Oct 2021 13:16:56 GMT\r\n
    server: Apache/2.4.29 (Unix) OpenSSL/1.0.2k-fips\r\n
    access-control-allow-origin: *\r\n
    access-control-allow-credentials: true\r\n
    access-control-allow-methods: GET, POST, OPTIONS\r\n
    access-control-allow-headers: Origin, Content-Type, Accept\r\n
    keep-alive: timeout=7, max=100\r\n
    www-authenticate: Basic realm="ATNSCHEME-BasicSessionless"\r\n
    content-length: 381\r\n
    connection: close\r\n
    content-type: text/html; charset=iso-8859-1\r\n
    x-final-url: http://centos-apache3.whatever.com:7777/oamprotectedtarget/index.html\r\n
     [truncated]access-control-expose-headers: date,server,access-control-allow-origin,access-control-allow-credentials,access-control-allow-methods,access-control-allow-headers,keep-alive,www-authenticate,content-length,connection,content-ty
    \r\n
    [HTTP response 1/1]
    [Time since request: 0.008985000 seconds]
    [Request in frame: 165]
    [Request URI: http://centos-apache1.whatever.com:8080/http://centos-apache3.whatever.com:7777/oamprotectedtarget/index.html]
    File Data: 381 bytes
Line-based text data: text/html (12 lines)

So if that access-control-allow-origin header is from CORS Anywhere, could somehow CORS Anywhere be able to send back:
access-control-allow-origin: http://centos-apache1.whatever.com:7777\r\n
instead of:
access-control-allow-origin: http://centos-apache1.whatever.com:7777\r\n
??
Thanks,
Jim


EvaluatorException for sql query

Version: 1.5.0
I get an EvaluatorException when running an SQL query from the console;
eg; SELECT _id FROM User;
The cause of error seems to be

"missing ; before statement"

but I have a ; in the query

Uncaught EvaluatorException
Object.shadow$klass: ClassObject.shadow$monitor: -1961575788RhinoException.columnNumber: 19RhinoException.interpreterLineData: nullRhinoException.interpreterStackInfo: nullRhinoException.lineNumber: 1RhinoException.lineSource: "SELECT _id FROM User;"RhinoException.sourceName: "chrome"Throwable.cause: EvaluatorExceptionThrowable.detailMessage: "missing ; before statement"Throwable.stackState: arrayThrowable.stackTrace: arrayThrowable.suppressedExceptions: List

I'm facing the same problem.
Have included semicolons before and after my statement but nothing changed.

Can you try a simple javascript expression 2 + 3 and see if it evaluates?
Also try disabling stetho-js-rhino.

2 + 3 does evaluate.

Also seeing this issue, didn't before.

Also seeing this.

The console doesn't support arbitrary SQL statements, but it does support JavaScript statements via stetho-js-rhino.  To use SQL statements, go to the Resources tab and select WebSQL.

I think this is not working in version 1.5.0 and it throws "missing ; before statement". I just added
    compile 'com.facebook.stetho:stetho:1.5.0'
    compile 'com.facebook.stetho:stetho-js-rhino:1.5.0'

and
       Stetho.initializeWithDefaults(this);
in my Application class.
Ran the app -> chrome://inspect -> Clicked on inspect app -> Resources tab -> "select * from $table"
_____/\\\\\\\\\\\_______________________________________________/\\\_______________________
 ___/\\\/////////\\\____________________________________________\/\\\_______________________
  __\//\\\______\///______/\\\_________________________/\\\______\/\\\_______________________
   ___\////\\\__________/\\\\\\\\\\\_____/\\\\\\\\___/\\\\\\\\\\\_\/\\\_____________/\\\\\____
    ______\////\\\______\////\\\////____/\\\/////\\\_\////\\\////__\/\\\\\\\\\\____/\\\///\\\__
     _________\////\\\______\/\\\_______/\\\\\\\\\\\_____\/\\\______\/\\\/////\\\__/\\\__\//\\\_
      __/\\\______\//\\\_____\/\\\_/\\__\//\\///////______\/\\\_/\\__\/\\\___\/\\\_\//\\\__/\\\__
       _\///\\\\\\\\\\\/______\//\\\\\____\//\\\\\\\\\\____\//\\\\\___\/\\\___\/\\\__\///\\\\\/___
        ___\///////////_________\/////______\//////////______\/////____\///____\///_____\/////_____
         Welcome to Stetho
          Attached to last.seen.hider.for.whatsapp

select * from Chats;
Uncaught EvaluatorExceptionObject.shadow$_klass_: ClassObject.shadow$_monitor_: -1943706498RhinoException.columnNumber: 20RhinoException.interpreterLineData: nullRhinoException.interpreterStackInfo: nullRhinoException.lineNumber: 1RhinoException.lineSource: "select * from Chats;"RhinoException.sourceName: "chrome"Throwable.backtrace: arrayThrowable.cause: EvaluatorExceptionThrowable.detailMessage: "missing ; before statement"Throwable.stackTrace: arrayThrowable.suppressedExceptions: List

Ok, If you were stuck just like me. You might be entering the query in the javascript console. The queries work only the box below the Resources Tab and not the Console . Everyone does silly mistakes ;)

@VihaanVerma89 I've been follow your instruction, but nothing displayed on Resource Tab

I'm having this problem too.

I need a picture of what Vihaan described.
I'm not having any luck executing sqlite commands... picture of what I'm seeing:
https://drive.google.com/open?id=161OjhQTUhwL2_CwTUALKb7Pekd5Nha7s

I watched the video before and was exactly what I'm doing but it doesn't work.
When I hit enter, the "query editor" just break the line and not "evaluate expression".
I tried enter, shift + enter and ctrl + enter.  =/

You have to click on the db and use the console at the beginning.


【代码迁移】任务27-2：Analysis of Black Hole Solutions in Parabolic Class Usin…

运行过程：

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


New Extension: PlayCover Community Edition

Description
This extension allows you to use PlayCover via RayCast

Sideloading
Show and open apps installed with PlayCover
Launch PlayCover

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @JoseMoreville 👋
Could you try to make some new screenshots from your MacBooks screen? Looks like you used an external monitor to make them 😊

Hi @JoseMoreville 👋
Could you try to make some new screenshots from your MacBooks screen? Looks like you used an external monitor to make them 😊

Hi @pernielsentikaer  , i'm on it

I can help with the screenshots @JoseMoreville 😄

One thing I noticed is the Sideload with PlayCover where you have to drag a file in, the first time you do that without the Keep window always visible during development turned on, then Raycast closes and unloads the extension.

We're working on a file/folder picker, we might need to wait for that before this would work.

One of the commands throws an error If you don't have PlayCover installed, whereas the other one handles this. Can you look into handling that too here?

I'm looking forward to testing this extension again 🔥

Thanks @pernielsentikaer I'll be working tomorrow according to the feedback provided and tag you here as soon as it is done. Thanks again for the screenshot help, I appreciate it.


 We should show a notification when that happens.

This is handled now.


[MLS Spec change] Remove explicit version pin on HPKE (#512)

Link to the exact changes
https://github.com/mlswg/mls-protocol/pull/512
Description of the changes

HPKE is in the RFC Editor queue, so there shouldn't be any further changes.
Fixes #500

editorial (and HPKE is an RFC by now)


Impl From traits for conversion between Array and std::vec::Vec (in favor of Array::to_vec)

Motivation
In the js-sys crate, I noticed the Array type implemented a to_vec method, and wanted to propose implementing the From trait in favor of this method to work nicer with Rust's type system.
I also found conversion between the JS Array type and Rust's Vec<...> may be common enough to warrant these trait implementations for easier conversion.
Proposed Solution
To put it simply, I would add the following impl's:
impl From<Vec<JsValue>> for Array {
    #[inline]
    fn from(value: Vec<JsValue>) -> Self {
        value.into_iter().collect()
    }
}

impl From<Array> for Vec<JsValue> {
    #[inline]
    fn from(value: Array) -> Self {
        value.into_iter().collect()
    }
}

A downside to this is that it would "hide" the performance impact if a user was doing the conversion implicitly (i.e. passing it to a function that takes an Into<Array> or Into<Vec<JsValue>>), since this conversion would have to cross the JS/Rust boundary for each access. This would have to be well documented in these implementations.
Alternatives
I attempted to do this for the borrowed versions of each, but I ran into some conflicts with trait implementations.
For example, when trying to impl<A: AsRef<[JsValue]>> From<A> for Array, the compiler warns about possible upstream future trait implementations. So I found specifying the owned values to be the most feasible. I'm very open to hearing what approaches others might have!

I don't believe we should do this as it's not really supported, as it has to go through the FFI boundary for each element. This hides the performance impact, as you already pointed out.
Thank you for the suggestion though!


Fixing the documentation

Hi,
This PR fixes the documentation

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   70.77%   70.77%
=======================================
  Files           7        7
  Lines         154      154
  Branches       23       23
=======================================
  Hits          109      109
  Misses         39       39
  Partials        6        6

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4dfbc2c...6c1de6b. Read the comment docs.


List of PHP Podcasts

Let's make a page listing all the good PHP podcasts.
@peterkokot have shared a list of podcast with me already on fb group. the list can be found below if you have any suggestions please feel free to comment here or create a PR.
PHP ones:

https://www.phproundtable.com/
https://www.phppodcasts.com/
https://phptownhall.com/

There are also frameworks specific ones:

http://www.soundofsymfony.com/
http://www.laravelpodcast.com/

:+1: Sounds good to me. It will be added here:
https://github.com/wwphp-fb/php-resources

PHP Podcasts chapter added via eec51bfb8ab7ecf2e726fb923fcf4d29e37ce7ab
In case there is some missing, let me know.

looks nice. I'll ask people around to see if I can find more good podcasts :smiley:


support multi-cpi configuration

some quick hack was done on: https://github.com/cloudfoundry-incubator/bosh-google-cpi-release/tree/wip-cpi-config-support
this allows to use cpi-config (one use case is to have multiple projects be managed by the same director).

It would be useful if the same director can manage multiple GCP projects.

@cppforlife would like to get this from wip to merged. Would you mind opening a PR for this branch so we can review there?
One other question: is there a practical maximum for the number of CPIs a director will support? 10s? 100s?

I'd like to second what @alwaysastudent said. Being able to deploy into multiple GCP Projects in conjunction with isolation segments would help us quite a bit in figuring out who generates what costs etc.


fix(tee): correct previous fix for race condition in batch locking

What ❔
Commit a7dc0ed5007f6b2f789f4c61cb3d137843151860 (PR #3342) was supposed to fix a race condition in batch locking by introducing SQL row-locking, but it didn't work as expected.

Now we are switching back to coarser-grained table-level locking as originally suggested by Harald. The original fix was hard to test unless deployed to stage due to the undeterministic nature of the problem, so we needed to merge it to the main branch to properly test it.
Why ❔
To fix the bug that only activates after running zksync-tee-prover on multiple instances.
Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[x] Code has been formatted via zkstack dev fmt and zkstack dev lint.

@haraldh, let's get this merged. ✅


provide make install option (at least for GNU/Linux)

Summary:
I'm trying to create an AUR PKGBUILD file to install cmdstan from my package manager and to allow other Arch Linux users to stay up to date. I've got the git cloning/building parts configured, but I don't understand the directory structure of the project. It would be great if the makefile provided a simple make install command that puts the resources in the correct locations.
Reproducible Steps:
Here's my work in progress PKGBUILD file:
If you save this on an arch distro, you can makepkg -si while in the directory. This will download/build/compress the package but I'm not sure where to put the subcomponents of the file.
# Maintainer: Ilja Kocken <iljakocken <at> gmail <dot> com>

pkgname=cmdstan-git
_pkgname=cmdstan
pkgver=v2.25.0.r126.g07ceb18
pkgrel=1
pkgdesc='The command line interface to Stan'
arch=('x86_64')
license=('BSD-3')
depends=()
makedepends=('git')
url="https://mc-stan.org/cmdstanr/articles/cmdstanr.html"
# https://github.com/stan-dev/cmdstan/releases/download/v2.25.0/cmdstan-2.25.0.tar.gz
source=("${pkgname%-*}::git+https://github.com/stan-dev/cmdstan.git")
sha256sums=('SKIP')
provides=('cmdstan')
conflicts=('cmdstan')

pkgver() {
    cd "$_pkgname"
    (
        set -o pipefail
        git describe --long 2> /dev/null | sed 's/\([^-]*-g\)/r\1/;s/-/./g' ||
            printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)"
    )
}

prepare() {
  cd "${_pkgname}"
  # this gets all the sub-directories
  make stan-update
}

build() {
  cd "${_pkgname}"
  # the -j4 option can be set in /etc/makepkg.conf
  make build
}

package() {
  # unfortunately there is no simple make install for cmdstan
  install -dm0755 "${_pkgdir}/usr/lib/stan"
  install -dm0755 "${_pkgdir}/usr/src/stan"

  cp -a --no-preserve=ownership "${_pkgname}/lib" "${pkgdir}/usr/lib/stan"
  cp -a --no-preserve=ownership "${_pkgname}/src" "${pkgdir}/usr/src/stan"
  cp -a --no-preserve=ownership "${_pkgname}/bin" "${pkgdir}/usr/bin/"

  cd "${_pkgdir}
  install -Dm644 LICENSE "${pkgdir}/usr/share/licenses/${pkgname}/LICENSE"
  install -Dm644 README.md "${pkgdir}/usr/share/doc/${pkgname}/README.md"
}

Current Output:
This installs only the lib, src, and bin directories, but I'm not sure if they can link to each other correctly and whether missing the root directory of the project misses essential components.
Expected Output:
I was hoping to put a very simple
cd "${_pkgname}"
make install

in the package function.
Additional Information:
Here's some more info on making AUR packages for Arch: https://wiki.archlinux.org/index.php/Arch_package_guidelines
Current Version:
v2.25.0

cmdstan has not been designed to be installed into different locations. I don't disagree with your ambitions and intentions here, but things are just not setup this way. I doubt that it would be simple to do. So maybe just stay with install things into /opt/cmdstan for now (and foreseeable future).

cmdstan has not been designed to be installed into different locations. I don't disagree with your ambitions and intentions here, but things are just not setup this way. I doubt that it would be simple to do. So maybe just stay with install things into /opt/cmdstan for now (and foreseeable future).

Thanks for the info. Does the cmdstan directory have to be editable? Because if so, then it's not desirable to install it in /opt because then you'd need sudo access whenever you're running anything.

Thanks for the info. Does the cmdstan directory have to be editable? Because if so, then it's not desirable to install it in /opt because then you'd need sudo access whenever you're running anything.

No, it does not need to be editable once the compiler flags are set and you have build at least one model (because of the pre-compiled headers).

No, it does not need to be editable once the compiler flags are set and you have build at least one model (because of the pre-compiled headers).

OK thanks! That means that this pkgbuild works correctly now :)
I guess I'll close the issue since it appears that it's not on the roadmap to re-structure the package to work with regular system directories.
# Maintainer: Ilja Kocken <iljakocken <at> gmail <dot> com>

pkgname=cmdstan-git
_pkgname=cmdstan
pkgver=v2.25.0.r126.g07ceb18
pkgrel=1
pkgdesc='The command line interface to Stan'
arch=('x86_64')
license=('BSD-3')
depends=()
makedepends=('git')
url="https://mc-stan.org/cmdstanr"
source=("${pkgname%-*}::git+https://github.com/stan-dev/cmdstan.git")
sha256sums=('SKIP')
provides=('cmdstan')
conflicts=('cmdstan')

pkgver() {
    cd "$_pkgname"
    (
        set -o pipefail
        git describe --long 2> /dev/null | sed 's/\([^-]*-g\)/r\1/;s/-/./g' ||
            printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)"
    )
}

prepare() {
  cd "${_pkgname}"
  make stan-update
}

build() {
  cd "${_pkgname}"
  make build
  make examples/bernoulli/bernoulli
}

package() {
  install -dm0755 "${pkgdir}/opt"
  cp -a --no-preserve=ownership "${_pkgname}" "${pkgdir}/opt"

  cd "${_pkgname}"
  install -Dm644 LICENSE "${pkgdir}/usr/share/licenses/${_pkgname}/LICENSE"
  install -Dm644 README.md "${pkgdir}/usr/share/doc/${_pkgname}/README.md"
}

OK thanks! That means that this pkgbuild works correctly now :)
I guess I'll close the issue since it appears that it's not on the roadmap to re-structure the package to work with regular system directories.
# Maintainer: Ilja Kocken <iljakocken <at> gmail <dot> com>

pkgname=cmdstan-git
_pkgname=cmdstan
pkgver=v2.25.0.r126.g07ceb18
pkgrel=1
pkgdesc='The command line interface to Stan'
arch=('x86_64')
license=('BSD-3')
depends=()
makedepends=('git')
url="https://mc-stan.org/cmdstanr"
source=("${pkgname%-*}::git+https://github.com/stan-dev/cmdstan.git")
sha256sums=('SKIP')
provides=('cmdstan')
conflicts=('cmdstan')

pkgver() {
    cd "$_pkgname"
    (
        set -o pipefail
        git describe --long 2> /dev/null | sed 's/\([^-]*-g\)/r\1/;s/-/./g' ||
            printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)"
    )
}

prepare() {
  cd "${_pkgname}"
  make stan-update
}

build() {
  cd "${_pkgname}"
  make build
  make examples/bernoulli/bernoulli
}

package() {
  install -dm0755 "${pkgdir}/opt"
  cp -a --no-preserve=ownership "${_pkgname}" "${pkgdir}/opt"

  cd "${_pkgname}"
  install -Dm644 LICENSE "${pkgdir}/usr/share/licenses/${_pkgname}/LICENSE"
  install -Dm644 README.md "${pkgdir}/usr/share/doc/${_pkgname}/README.md"
}

Hmm it looks like perhaps it does need the permissions to be set to the user. This is the output of running a simple model:
This is cmdstanr version <IP_ADDRESS>00
- Online documentation and vignettes at mc-stan.org/cmdstanr
- CmdStan path set to: /opt/cmdstan
- Use set_cmdstan_path() to change the path
Compiling Stan program...
<built-in>: fatal error: opening dependency file stan/src/stan/model/model_header_threads.d: Permission denied
compilation terminated.
Assembler messages:
Fatal error: can't create src/cmdstan/main_threads.o: Permission denied
make: *** [make/program:14: src/cmdstan/main_threads.o] Error 1
Error: An error occured during compilation! See the message above for more information.

Hmm it looks like perhaps it does need the permissions to be set to the user. This is the output of running a simple model:
This is cmdstanr version <IP_ADDRESS>00
- Online documentation and vignettes at mc-stan.org/cmdstanr
- CmdStan path set to: /opt/cmdstan
- Use set_cmdstan_path() to change the path
Compiling Stan program...
<built-in>: fatal error: opening dependency file stan/src/stan/model/model_header_threads.d: Permission denied
compilation terminated.
Assembler messages:
Fatal error: can't create src/cmdstan/main_threads.o: Permission denied
make: *** [make/program:14: src/cmdstan/main_threads.o] Error 1
Error: An error occured during compilation! See the message above for more information.

Like I say... you need to install things and then compile a model, so
as root do:
make build
make example/bernoulli/bernoulli
then you should be able to compile as a user other models

Like I say... you need to install things and then compile a model, so
as root do:
make build
make example/bernoulli/bernoulli
then you should be able to compile as a user other models

I had those instructions included in the PKGBUILD file, so root already did that during the build. But while running another model it was now complaining that it didn't have permission.

I had those instructions included in the PKGBUILD file, so root already did that during the build. But while running another model it was now complaining that it didn't have permission.

The first time you use threads it will bulid main_threads.o

The first time you use threads it will bulid main_threads.o

for the CmdStanPy and CmdStanR packages, the default location for the CmdStan install is in the user's home directory, in hidden directory .cmdstan and/or .cmdstanr (although there's an issue on CmdStanR to change to name .cmdstan), precisely because users typically don't have admin perms on their machine.

for the CmdStanPy and CmdStanR packages, the default location for the CmdStan install is in the user's home directory, in hidden directory .cmdstan and/or .cmdstanr (although there's an issue on CmdStanR to change to name .cmdstan), precisely because users typically don't have admin perms on their machine.

Ahh ok. Yeah in my model I did set it to run with multiple chains/cores.
Ok looks like I'm going to give up on this effort and in stead rely on the cmdstanr::install_cmdstanr() function.
Thanks everyone for chiming in.

Ahh ok. Yeah in my model I did set it to run with multiple chains/cores.
Ok looks like I'm going to give up on this effort and in stead rely on the cmdstanr::install_cmdstanr() function.
Thanks everyone for chiming in.


mesageEdit: Show topic input even in topic narrow.

Sometimes message ref is undefined, so it is not focused. Thus
use editMessage object to check for topic input visibility.

Why do we want to show the topic input though?
PS. The complexity of the logic might need unit-tests at this point.

Why do we want to show the topic input though?

Then there is no way to edit/change message topic.
Many users are facing issue of accidentally sending message to wrong topic on mobile. So this will be helpful.

PS. The complexity of the logic might need unit-tests at this point.

Extracted logic and added tests.

OK, cool. That makes sense.
We can definitely improve the whole approach to editing a message from a coding perspective, it is messy, but that is another PR for another time. We should do it as a separate route so going back works naturally.

Thanks @jainkuniya !
Let's keep the getCanSelectTopic logic in the same file/module -- it only makes sense to read it in the context of ComposeBox, so the extra indirection of going to another file (starting from either one) to read what's happening just gets in the way. A separate function is good because it enables tests, though.
As this Boolean logic gets more complex, another useful technique that will help keep it readable is to use if and early return to break it up -- this is another thing we gain from splitting into a function. So something like
if (editMessage) {
  return isStreamOrTopicNarrow(narrow);
}
if (!isStreamNarrow(narrow)) {
  return false;
}
return isMessageFocused || isTopicFocused;

Merged -- thanks!
For clarity, I separated out the commit that does the refactor from the one that actually makes the behavior change. Take a look.


Chattier configuration errors

First off thanks for the library guys, appreciate it.
Second, it seems like there could be some more useful error messages and/or defaults around configuration. For example:
# mm-config.json

{
  "host": "localhost",
  "port": "27017",
  "db": "test"
}

# 1-foogration.js

module.exports.id = "FOOGRATION";

module.exports.up = function (done) {
  // use this.db for MongoDB communication, and this.log() for logging
  done();
};

module.exports.down = function (done) {
  // use this.db for MongoDB communication, and this.log() for logging
  done();
};

Now this doesn't actually change anything in the DB but when it's run it simply hangs, a connection is made to mongo but nothing is printed to the log.
mm migrate # and... nothing

As you may have noticed my config was missing the collection key, and adding it does solve the issue and result in a successful run but perhaps in these negative cases the library could communicate to the user, "hey you're missing some required config"?
Happy to add this functionality myself if requested, opening this issue in the meantime just to let you guys know.
Thanks again!

Hey, thanks for reporting!
It's been a while since I had a chance to work with this library, unfortunately :
You're completely right, there should be a default or an error thrown. A PR is welcome.

@emirotin may i ask what you use for migrations nowadays?

I don't use MongoDB these days at all.

I've returned to this module and made the following changes

the collection param now defaults to _migrations which should be a safe default IMO
other required config options are validated

This will be part of the 0.8.0 release to be published tonight.


Snapcast: v0.20.0 → v0.23.0, including various fixes to nixos/snapserver

Motivation for this change

New release.
The current module has out-of-date option examples and is missing stream types.

Things done
No significant change to package closure since there is no new dependency.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@fpletz gentile ping :-)

Result of nixpkgs-review pr 98962 run on x86_64-linux 1

1 package blacklisted:

tests.nixos-functions.nixos-test

1 package built:

snapcast

Hi @fpletz. Any chance you could give this PR a review please? It's currently preventing the use of new snapcast sources such as "tcp".

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/461

If absolutely nobody cares to object…

This pull request looks very good to me. Thank you for your work, @Zopieux! I'm sorry that merging took so long. Generally I am very happy with the fast turnaround time we have around here, but obviously there are big exceptions and I feel your frustration.

Thanks for the swift resolution in the end!

I'm sorry that merging took so long.

No worries, open-source work is best-effort.


[BUG] Results on Chatbot Preview Page are different from dashboard chat window and API response.

Describe the bug
Hello, I am a user who is using the Flowise service well. I would like to inquire about the chat message window through "share chatbot". When I ask a question in the chat window of the dashboard configured with a chatflow in Flowise, the answer is generated well based on the upserted resources. However, on the preview page generated through "share chatbot", the answers are not accurate or different from those in the dashboard chat window.
I wonder if there is a difference in caching per session or in the way API calls are made.
I always support the Flowise project and team. Thank you.
To Reproduce
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
Flow
test Chatflow.json
Setup

Installation: Docker
Flowise Version: 1.5.0
OS: Linux
Browser: Chrome

Additional context

Are you able to provide some screenshot examples of the difference you're experiencing?

Here you are, Thanks.

What if you make the same call via the api using postman?

there is no difference between the external and internal chat logic. I'd recommend using analytics to see if the step by steps are the same:


Harmony: ActionWrapper

PR includes

[x] Feature

Related issues
Resolve #998
Related issues:

https://github.com/taskany-inc/issues/issues/2416

QA Instructions, Screenshots, Recordings
https://github.com/taskany-inc/bricks/assets/95316053/802aee74-3f77-4169-ad21-d57e98ba54e9

/canary

/canary

/canary

/canary

/canary

/canary


Exam: Students Can File Complaints for Text Exercises Before the Review Period

Describe the bug
In exams, it seems that students can file complaints for text exercises as soon as the results are published, even if the review period has not begun. For programming exercises, it seems to work as expected.
This is not a major problem, it is just strange that it works differently than expected.
To Reproduce

Conduct an exam, grade it.
Set the result publication date, and set the review period to only start later
Once the results are published, from student view you can file complaints for text exercises (even though the review period has not started)

Expected behavior
Students can only file complaints for text exercises once the review period started.
Screenshots
No response
What browsers are you seeing the problem on?
Chrome
Additional context
To see the problem in action, one (w/ admin access to the TUM instance) can currently observe it on the FPV exam (course No. 182) before the start of the review period today at 1:30pm.
Relevant log output
No response

This affects Text- / Modelling- and File-Upload Exercises.
The complaint component is wrongly shown on the results - subpage of the exercise. Here, the complaint can be filed at any time because the complaint-component doesn't receive the exam information and thus cannot determine the student review period. The corresponding text-/modelling-/file-upload component also has no exam information.
Thus, the complaint component is displayed, as for normal course exercises, after the assessment due date, which is independent of the exam review period. That's also the reason the wrong hint with the limited number of complaints is shown to the student.
With the PR #4988 the complaint component is hidden for exam exercises and students can use the indented way to complain about an exam exercise - via the exam summary page and within the student review window.


MP borate export across plasma membrane

Borate is required in small quantities but is toxic to microbes in larger quantities. Therefore have borate PM exporter
GO:0140159    borate export across plasma membrane -> add detoxification

@tberardini will this work for plants where the transport of boron on an organismal level is critical? Is the export always detoxification or is it sometimes exported from say root cells into the plant vascular system for use elsewhere?
We need to be very careful about creating instance relationships in the ontology.

Ah plants. Good point!

Actually, I meant to specify borate, but I think the same argument applies, does it not?

@tberardini if you still agree, can you close this ticket?

I think this one nails it shut.
PMID:18603465
Plus it mentions my favorite plant structure, the Casparian Strip.


Allow show for RefernceField link

Resolves #247

I'd prefer a linkToRecord prop, with a default value of true.
Also, please update the documentation for ReferenceField
Lastly, new features PR should be done against next, not master.

Hi, we are using a ReferenceArrayField and apparently the linkType={false} isn't operational on this one. Is this intended ?


RTOS: Cortex A - stack larger than 64k

The version for Cortex-M was fixed via #826. It should be also applied to Cortex A version.
Snippet from P_TCB:
  /* Hardware dependant part: specific for Cortex processor                  */
  U8     stack_frame;             /* Stack frame: 0x1 Basic/Extended, 0x2 FP stacked/not stacked */
  U8     reserved;
  U16    priv_stack;              /* Private stack size, 0= system assigned  */
  U32    tsk_stack;               /* Current task Stack pointer (R13)        */
  U32    *stack;                  /* Pointer to Task Stack memory block      */

  /* Task entry point used for uVision debugger                              */
  FUNCP  ptask;                   /* Task entry address                      */
} *P_TCB;

priv_stack is there U16.

Fixed via #826

@0xc0170 I thought you opened this bug to track the propagating of change 826 to the Cortex-A branch of the RTX code?

Hello,
Thanks for your comment.
We have a plan to modify this problem in Cortex-A.
We will update RTOS to support NEON.
We will modify both issues in the same pull request.

Hello,
We updated RTOS to support NEON.
(https://github.com/mbedmicro/mbed/pull/1326)
We have modified this issues in the update.

@TomoYamanaka Means this should be fixed on master  ,and this can be closed?

Hello,
Yes, This thread can be closed since the fix of this isuue was reflected on master.
Please close this thread.

OK, I checked, should be fixed. Thanks


C++11 port

I ported the code to Standard C++11.
It use no platform specific features such as compiler intrinsics, Mac OS X specific high resolution timers.
I used C++11's atomic access library for replacement of compiler intrinsics.
It shouldn't be optimized away and in my environment, It's as strong as original.
I also added a simple Makefile.

Close it for better alternative PR.


Fixed the A-Z bookmarks

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Welshmilla sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for fixing this! That was a lot of work - we appreciate it!


Update travis config to test on node 6

This should get our node environments to match between internal tests and Travis runs

@facebook-github-bot shipit-ninja


Upgrade aiohttp to 2.1.0

Codecov Report

Merging #12 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   95.37%   95.37%
=======================================
  Files           5        5
  Lines         303      303
=======================================
  Hits          289      289
  Misses         14       14

Impacted Files
Coverage Δ

aiosparql/client.py
95.4% <100%> (ø)
:arrow_up:

aiosparql/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa8c02f...0758841. Read the comment docs.


Numbers past 53 bits break toCurrency and toNumber

See here:

Upon further investigation, this is actually a problem with javascript's toFixed() function used by i18n-js here:
https://github.com/fnando/i18n-js/blob/6431b926cf4c05a8946116968d3d40823dd4ac99/app/assets/javascripts/i18n.js#L48
And the reason toFixed() breaks on numbers like this is because JS only supports 53 bit integers which<PHONE_NUMBER>00020000 is larger than. See more info on this here: http://2ality.com/2012/07/large-integers.html.
Ideally we would want to find a patch for this that doesn't break rounding for imprecise floating points (which appears to be the reason toFixed() was added to this line here: https://github.com/fnando/i18n-js/blob/6431b926cf4c05a8946116968d3d40823dd4ac99/app/assets/javascripts/i18n.js#L48 ).
Not as ideally we could add a disclaimer for this known issue instead or in the meantime. I'm happy to help in any way I can on this!

Do you any known patch that solves this issue?
I have never encountered any large enough number yet

Added to known issues until there is a permanent fix available
https://github.com/fnando/i18n-js/commit/608d5526c6a0512cfe7d8685466d7717b768bafa

I wonder how the "warning" be sent
Since this library might be used in browser / NodeJS process / even mobile app (screenshot in #513)


MD014 does not allow command output to start with empty line

If the command output starts with an empty line, MD014 throws an error since change https://github.com/DavidAnson/markdownlint/commit/c4f77e021dc93626bc8ad465d2b70cb2a448af12.
Some command outputs just start with an empty line, so I think this should be allowed, for example:
$ dig github.com

; <<>> DiG 9 <<>> github.com
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 166
;; flags: qr rd ra; QUERY: 1, ANSWER: 1, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 65394
;; QUESTION SECTION:
;github.com.			IN	A

;; ANSWER SECTION:
github.com.		13	IN	A	<IP_ADDRESS>

;; Query time: 14 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Tue Dec 24 11:37:17 CET 2019
;; MSG SIZE  rcvd: 55

I'm not sure if this deserves its own issue, but there is also the problem of commands that do not have output, like cd. If cd is in the middle of a list of commands, the $ seems appropriate:
$ mkdir test
mkdir: created directory 'test'
$ cd test
$ ls test

When some commands do have output and others do not, the $ still indicates which lines are new commands are which lines are output. In the linked guide, the second "good" example shows a line without output: https://cirosantilli.com/markdown-style-guide#dollar-signs-in-shell-code

@davidystephenson While the intent of the style guide is clear, I don't see a good way for a simple rule likeMD014 to differentiate the cd scenario from a violation.￼ I don't think I've heard much about this special case in the past, so it may be rare in practice and something that needs to be excluded by the inline comment syntax.￼

To me, based on what is shown in the second example, the intent of the style guide is "if the block uses $ to mark commands, at least 1 command must show output without $." I happen to write a lot of command blocks in my work that require specifying the exact command order and output of basic terminal patterns. Often a few command do not have output, so this scenario arises often for me. I won't create another issue if these comments already express the point for future reference.

"if the block uses $ to mark commands, at least 1 command must show output without $."

That’s an interesting way to think about this, thank you! I will keep that in mind when I go to fix this and see if it helps to do what you want without violating the general guidance.


Measure-FleetCoreWorkload

Hi,
When i started this command, i have this error
VERBOSE: Operation '' complete.
WARNING: .: 2021/09/29-08:45:13.844 Node NCY-HYV1: No disks found to be used for cache
WARNING: .: 2021/09/29-08:45:13.844 Node NCY-HYV2: No disks found to be used for cache
2021-09-29T08:45:14: READ CACHE WARMUP: capacity devices are not caching reads, no warmup to perform
2021-09-29T08:45:19: Starting run for ComputeTemplate=A1v2 DataDiskBytes=153545080832 FleetVMPercent=100 MemoryStartupBytes=2147483648 PowerScheme=HighPerformance ProcessorCount=1 VMAlignmentPct=100 Workload=General4KWriteRatio0
2021-09-29T08:45:20: START Go Epoch: 1
2021-09-29T08:45:20: CLEAR pause at Go
2021-09-29T08:45:23: SLEEP TO RUN CHECK (17.44 seconds)
2021-09-29T08:45:40: RUN CHECK Go Epoch: 1
2021-09-29T08:45:40: ERROR: done-vm-base-NCY-HYV1-004 is already done
2021-09-29T08:45:40: ERROR: done-vm-base-NCY-HYV1-005 is already done
2021-09-29T08:45:40: ERROR: done-vm-base-NCY-HYV1-011 is already done
2021-09-29T08:45:40: RUN CHECK 1 : 3/32 done (0.02s, total 0.02s)
Transcript stopped, output file is C:\ClusterStorage\collect\result\coreworkload.log
Unexpected early completion of load, please check profile and virtual machines for errors
At C:\Program Files\WindowsPowerShell\Modules\vmfleet\<IP_ADDRESS>\VMFleet.psm1:4961 char:17

...             throw "Unexpected early completion of load, please check  ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (Unexpected earl...ines for errors:String) [], RuntimeException
FullyQualifiedErrorId : Unexpected early completion of load, please check profile and virtual machines for error
s

Acknowledging.
When this happens, can you please connect to one of those VMs (virtmgmt.msc / vmconnect) and show me what's being reported on the powershell controller's log output?
In the near/medium term I am looking to provide a way to extract the controller logs directly to make this diagnosis easier.

I encounter the same issue and the console show "STOPPING (reason: new run file)".

If you can confirm this is occurring with Windows Server 2019 ("RS5"), this is fixed in 2.0.2 - RS5 is now supported and confirmed to work.


Crash in Android

In some cases I get ORN exception in GridViewAdapter in ItemCount property. The solution is to make sure you check Items is not null so I suggest you to change following code from:
   public override int ItemCount
    {
        get
        {
            return _flattenedItems != null ?  _flattenedItems.Count() : Items.Cast<object>().Count();
        }
    }

To
public override int ItemCount
    {
        get
        {
            if (_flattenedItems != null)
                return _flattenedItems.Count();
            else if (Items != null)
                return Items.Cast<object>().Count();
            else
                return 0;
        }
    }

Note above ItemCount property located in GridViewAdapter class

#32


[Bug]: Traefik does not use custom SSL certificates

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Custom certificates that have been created through a certification authority other than LetsEncrypt seemingly cannot be used on any domain. After adding a custom ssl certificate in the SSL Certificate section of the settings menu traefik is still using the default self signed certificate for the coolify front-end as well as any of my applications. It ignores the custom certificate even though the certificate was created for the specified domain in the settings menu. Traefik then tries to auto generate a new certificate through LetsEncrypt which fails immediately because our institution's CAA record prevents LetsEncrypt from generating one. In the final setup I would like to use a wildcard certificate (not created through LetsEncrypt) to serve all of our applications over HTTPS.
Traefik Error when restarting

Steps To Reproduce

After logging into the Coolify front-end go to Settings
In the Coolify Settings section set the URL (FQDN) to your DNS
Navigate to SSL Certificates
Add a custom SSL certificate created by a certification authority other than LetsEncrypt

Expected behaviour:
-> Traefik should now use the supplied SSL certificate for the specified URL (FQDN). This is not the case.
Restarting Traefik and Coolify after uploading the custom certificate does not fix the issue.
Version
3.11.4

@andrasbacsai is there any news on how far down the docket fixing this issue is, or if this is at all something you are planning to address? As of now this sadly prevents us from using coofily in production.

Same thing here. This completely blocks usage of Coolify for me.

Is there any documentation on this process? I would also prefer to add custom generated certs using a local certificate authority

Is there any documentation on this process? I would also prefer to add custom generated certs using a local certificate authority

No, and it doesn't work currently, anyway.

It looks like the certs are correctly uploaded and saved into
/coolify-traefik-letsencrypt/_data/custom.
Maybe as a quick fix, is there a way to tweak the Treafik config somewhere?

@syrenius I also searched around coolify's docker container before I opened this issue and found the same thing. There might be a way to configure traeffik from inside the container but I didn't want to spend too much time on a hacky solution like this as I don't know traeffik that well. Yes, it might be possible but it will most definitely not be a viable solution in the long run as this should in my opinion work "out of the box" as it's a foundational feature for a platform like coolify. Unfortunately, the maintainer @andrasbacsai has been entirely unresponsive to any of my posts here or on the Discord channel (even as to just let users know if this feature is at all planned), so that I have given up on using this project. If you find a way to make it work it would be nice if you could post your solution here though.

I'm closing this issue, because tracking features will be moved to GitHub discussions. Github issues will be only for bugs.
Version 4 is in full focus. v3 related issues will be closed, if it is not a critical bug.
Thanks for your understanding.

But isn't this critical? It literally made it completely impossible for me to use Coolify in any way and seems like core functionality.
Will this be fixed in v4?


random error

I get these error sometimes while trying to use phantoms rasterize.js to output html page to pdf, meanwhile it still outputs the pdf correctly.   I am using phantoms 1.9.6
2014-02-28 13:52:00.643 phantomjs[13070:d07] *** WARNING: Method userSpaceScaleFactor in class NSView is deprecated on 10.7 and later. It should not be used in new applications. Use convertRectToBacking: instead.
Feb 28 13:52:01  phantomjs[13070] : CGContextSetShouldAntialias: invalid context 0x0. This is a serious error. This application, or a library it uses, is using an invalid context  and is thereby contributing to an overall degradation of system stability and reliability. This notice is a courtesy: please fix this problem. It will become a fatal error in an upcoming update.
Feb 28 13:52:01  phantomjs[13070] : CGContextSetShouldSmoothFonts: invalid context 0x0. This is a serious error. This application, or a library it uses, is using an invalid context  and is thereby contributing to an overall degradation of system stability and reliability. This notice is a courtesy: please fix this problem. It will become a fatal error in an upcoming update.
Feb 28 13:52:01 phantomjs[13070] : CGContextSetFillColorSpace: invalid context 0x0. This is a serious error. This application, or a library it uses, is using an invalid context  and is thereby contributing to an overall degradation of system stability and reliability. This notice is a courtesy: please fix this problem. It will become a fatal error in an upcoming update.
Feb 28 13:52:01 phantomjs[13070] : CGContextSetFillColor: invalid context 0x0. This is a serious error. This application, or a library it uses, is using an invalid context  and is thereby contributing to an overall degradation of system stability and reliability. This notice is a courtesy: please fix this problem. It will become a fatal error in an upcoming update.

I'm getting this on Yosemite using v2.0.0 of phantomjs
Attempting to use Wraith visual diff capture


Plush Sash of Guzbah incorrectly shows two available sockets

Describe the bug
The item Plush Sash of Guzbah has one available gem socket, but is incorrectly displayed with two.
Expected behavior
Only one yellow socket should be available.
Sim Links and Screenshots

Desktop (please complete the following information):

OS: Windows
Browser: Firefox
Version 106.0.1 (64-Bit)

It's a belt buckle, all the belts assume you get one


Comparison with hierarchical clustering

Hi Xiaohang,
I am curious about how you apply hierarchical clustering on such a large dataset since the best complexity is only O(n^2). Do you use any technique to accelerate hierarchical clustering?
By the way I'm also interested in how CDP performs against hierarchical clustering based on a single face embedding, I'd appreciate it if you can offer some comparison.
Thanks a lot!

We use faiss https://github.com/facebookresearch/faiss to search k-NN. It decreases the complexity to NxK. Hierarchical merging is only performed twice, and the metric is the distance of the centers of two clusters. According to our observation,  further merging brings minor improvements.

With single model, the mediator is not applicable, then CDP devolves into simply thresholding the k-NN graph and dynamically searching connected components. It much weaker than the full setting of CDP. Comparison can be found in Table 2 in our paper http://openaccess.thecvf.com/content_ECCV_2018/papers/Xiaohang_Zhan_Consensus-Driven_Propagation_in_ECCV_2018_paper.pdf. Clustering achieves recall/prec/fscore 0.558/0.95/0.703, and CDP with committee number = 0 yields 0.68/0.829/0.747. CDP is slightly better than clustering.

Thank you for your time!
So it is not a standard Agglomerative Clustering.  Putting aside the complexity, I believe a standard Agglomerative Clustering will achieve a satisfactory performance.

You are right.


typo:fix tables to table

only one table, line 43 tables should be table

/ok-to-test

/cc @errordeveloper Can you see why this chart is failing? See the tests for detailed logs.

/retest

/lgtm


fix(no-void): enable allowAsStatement

Fix #370
What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[x] Bug fix
[ ] New feature
[ ] Other, please explain:
What changes did you make? (Give an overview)
Enable allowAsASStatement in no-void
Which issue (if any) does this pull request address?
Fixes #370
Is there anything you'd like reviewers to focus on?
Run yarn lint in
https://codesandbox.io/s/quirky-hopper-sxtqw
to see the fixed behaviors.

It seems the CI error is due to https://github.blog/2021-02-02-npm-7-is-now-generally-available/#peer-dependencies and commitlint-config-awesome has unmatch peer dependency of @commitlint/cli. I can do either:

add --legacy-peer-deps to the install script
keep using npm 6
downgrade @commitlint/cli to ^8.0.0
wait for rebasing on upgraded master branch

The CI problem should be handled independently.


feat(frontend): support grouping sets

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Related Issue https://github.com/risingwavelabs/risingwave/issues/9284
Currently, the group by clause can only support one and only one grouping sets expression.
The optimizer use GroupingSetsToExpandRule to convert an Agg with grouping sets into an Agg without grouping sets.
The schema of an Agg with grouping sets is the union of all grouping columns pulsing the agg calls.

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR contains user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

Can you add Sqlsmith generation support?

Can you add Sqlsmith generation support?

Sure, I will reach out to @kwannoel for more information about sqlsmith tests later.


更新

Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

请先拉取最新的dev分支，完成 merge 后再 force push
教程：https://blog.csdn.net/zhongzunfa/article/details/80344585

重新整理一下再提交吧。先关了。

重新整理一下再提交吧。先关了。


Fix for flaky TestGameServerAllocationMetaDataPatch

Cache eventual consistency in the controller means that this test can be flaky. Implemented a polling request as allocation will work eventually.
Closes #725

Gentle bump - on a mission to de-flake the CI system, and this keeps rearing it's head.


Add missing reference table link to Getting Started

https://styled-system.com/getting-started#other-props

Awesome, thanks!


Updating _noise2a and _noise4a to output multidimensional arrays

Updating the _noise2a and _noise4a (noticed another pull request for _noise3a so have omitted in this) functions to output a multidimensional array. Given the inputs are numpy arrays of indices/coordinates then it will iterate over them, constructing a multidimensional array of the generated noise, like in the example code supplied in the repo. Have also added some docstrings and typehints in the functions.

Hi thank you for this.
Would you mind remove the docstring from the _noiseXarray funcs? They're not ment to be used directly so it's just confusing letting them have docstrings.

Hi no worries, just to confirm, do you mean to remove the docstring from the functions in the class (noiseXarray) or the private ones in the file that the class functions point to (_noiseXa)?

Whoops misstyped. Only the private ones (_noiseXa).
--
Alex
On Sun, 30 Jan 2022, at 14:09, Daniel Shrimpton wrote:

Hi no worries, just to confirm, do you mean to remove the docstring
from the functions in the class (noiseXarray) or the private ones in
the file that the class functions point to (_noiseXa)?
—
Reply to this email directly, view it on GitHub
https://github.com/lmas/opensimplex/pull/25#issuecomment-1025140293,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBDCXNDSWBFBLXEQ5HXBM3UYUZ7LANCNFSM5NDAB5OQ.
You are receiving this because you commented.Message ID:
@.***>

I have removed the docstrings from the private functions :)

Nice thanks man.
Holding off pulling for now, need to wait for a reply on the other PR.
--
Alex
On Sun, 30 Jan 2022, at 14:14, Daniel Shrimpton wrote:

I have removed the docstrings from the private functions :)
—
Reply to this email directly, view it on GitHub
https://github.com/lmas/opensimplex/pull/25#issuecomment-1025141124,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBDCXME54YBFGBGHFHP5QTUYU2TXANCNFSM5NDAB5OQ.
You are receiving this because you commented.Message ID:
@.***>

Sure thing, no worries at all


[Unclear] Custom Omnipay gateway

Hey guys,
First of all awesome package! Thanks a lot! I'm sorry to ask this here but i can't figuur out if it's possible to add Omnipay gateways directly into the gateway array of laravel-shop like so:
'gateways' => [
    'paypal'            =>  Amsgames\LaravelShopGatewayPaypal\GatewayPayPal::class,
    'paypalExpress'     =>  Amsgames\LaravelShopGatewayPaypal\GatewayPayPalExpress::class,
    'someOmnipayGateway' => Vendor\Package\SomeOmnipayPackage::class,
],

Your documentation only states: Install Omnipay Gateway to enable other payment services but i can't find any thing in the source code that would handle a custom omnipay gateway.
Am i correct in assuming that a gateway class is needed that extents Amsgames\LaravelShop\Core\PaymentGateway and defines the required functions using the custom gateway omnipay package?
Thanks,
Raoul

I'm sorry! I realized that the 'Omnipay Gateway' link is not a link to thephpleague/omnipay but a package explaining exactly what i needed to know!


Fix Issue #245

@exabrial Hi could you please add a bit more context to the commit message.
@clebertsuconic I will leave this one with you since you have been involved in #245.

The commit message should be something like:
ARTEMIS-### Any textual explaining what's being done...
We don't simply rely on the Links from JIRA for instance...  the jira might be replaced some day.. and when someone is looking from git logs.. it gets impracticable to look for changes and looking for regressions if we only used jira links.
If you can't do it I will treat this as a patch and rebase during merge... but you would probably lose ownership of your commit on the logs.

@exabrial Hi, any chance you could update the commit message and we'll get this merged.
Thanks

I'm not completely sure it's correct actually... a better way would be go
make the connection class an interface, rather than a final class.
Also at Devoxx UK in two weeks I'm leading a workshop with the goal is to
get this working 100%. If any of you will be available, or present
physically, that'd be awesome!
On May 19, 2016 5:30 AM, "Martyn Taylor"<EMAIL_ADDRESS>wrote:

@exabrial https://github.com/exabrial Hi, any chance you could update
the commit message and we'll get this merged.
Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/apache/activemq-artemis/pull/496#issuecomment-220273790

@exabrial I'd be happy to help.  I can talk to you on IRC #activemq or on the dev list.

@exabrial I am going to close this PR based on our discussion.  We can reopen later if need be.

@exabrial Actually doesn't look like I have permissions to close this PR.  Could you please close it.
Thanks


chore(RHTAPWATCH-766): add spacerequest cleaner app

Add app to contain a cronjob that will periodically run process for cleaning spacerequests

/retest-required

/retest-required

/retest-required

/approve
/lgtm


reset after power on

Hi!
I have one hardware that is working, and I build the Dallas example to know how this new lib work.
But, on the load static OneWireNg *ow = new OneWireNg_CurrentPlatform(OW_PIN, false); I get reset.. on the ESP8266. Cause 1: normal boot after power-on
load 0x4010f000, len 1384, room 16
tail 8
chksum 0x2d
csum 0x2d
vb3acf27f
~ld
But, if I change it to true:
static OneWireNg *ow = new OneWireNg_ArduinoESP8266(OW_PIN, true);
The issue of the reset don`t return. I Have pull UP 3K3.
This library is very robust.. I loved it! Congradulations!
Best Regards

@adbensi Thanks for the feedback. You probably found a severe bug. Will be fixed soon.

Hi!
If I can do something, please tell me.. I made another sample with no block
loop and detect sensor ID too, If you like to extend samples code
Thanks a lot!!
Em qui, 20 de fev de 2020 18:13, Piotr Stolarz<EMAIL_ADDRESS>escreveu:

Reopened #2 https://github.com/pstolarz/OneWireNg/issues/2.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pstolarz/OneWireNg/issues/2?email_source=notifications&email_token=ACCTHGFZAZWBC2YCESZR243RD3W73A5CNFSM4KXXR6GKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWZESTPY#event-3058248127,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCTHGHZVPII4ACLYFHFINTRD3W73ANCNFSM4KXXR6GA
.

@adbensi @MHz000 Could you check my last commit on master with fix for the ESP8266 crash. I'd be grateful for test with and without internal pull-up resistor configured - last argument of the OneWireNg_CurrentPlatform() constructor set to true / false. Let me know about results...

Bingo!
Chapeau, chapeau!
After changing to
static OneWireNg *ow = new OneWireNg_CurrentPlatform(OW_PIN, PWR_CTRL_PIN,
true);
#else
static OneWireNg *ow = new OneWireNg_CurrentPlatform(OW_PIN, true);
no crash any more (while set to false system crashed)
arduino serial monitor reads:
28:49:49:CC:3:0:0:28 -> DS18B20
Scratchpad: 73:1:4B:46:7F:FF:D:10:41
Temp: 23.187 C
Congratulations!
What next? Checking the DS2438? May I ask you for an strip down example to
run the check I'll be glad to support developent of your lib. Stripped down
version means: no parsite power, no address checking, just one device on
the wire. Keep it simple as it can be makes error checking easy.
Jochen
Am Fr., 21. Feb. 2020 um 00:39 Uhr schrieb Piotr Stolarz <
<EMAIL_ADDRESS>
@adbensi https://github.com/adbensi @MHz000 https://github.com/MHz000
Could you check my last commit on master with fix for the ESP8266 crash.
I'd be grateful for test with and without internal pull-up resistor
configured - last argument of the OneWireNg_CurrentPlatform() constructor
set to true / false. Let me know about results...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pstolarz/OneWireNg/issues/2?email_source=notifications&email_token=AB42SGSEUJKIY6EQN2WQS5TRD4ICZA5CNFSM4KXXR6GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMQ3SKQ#issuecomment-589412650,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB42SGXP5E2FTHOKSZT3NIDRD4ICZANCNFSM4KXXR6GA
.

Thanks @MHz000 but it looks like you're still working on 0.4.0 version. Could you confirm that on the latest master commit both variants work fine?

downloaded from github, include .zip lib from downloads, compiling the
example
=> Error compiling for board Generic ESP8266 Module.
:-)
my fault??
Am Fr., 21. Feb. 2020 um 16:02 Uhr schrieb Piotr Stolarz <
<EMAIL_ADDRESS>
Thanks @MHz000 https://github.com/MHz000 but it looks like you're still
working on 0.4.0 version. Could you confirm that on the latest master
commit both variants work fine?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pstolarz/OneWireNg/issues/2?email_source=notifications&email_token=AB42SGS6AXIPCLLHTKS6CFTRD7UI7A5CNFSM4KXXR6GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMS7OTI#issuecomment-589690701,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB42SGTPUMDAP3OAA4EFLZ3RD7UI7ANCNFSM4KXXR6GA
.

@adbensi @MHz000 Could you check my last commit on master with fix for the ESP8266 crash.

@pstolarz No more reset in the false condition :)
But I no longer have the hardware to run the long test.
I will assemble a new unit exclusively for this purpose.
Some sensors, independent of the library used, work more properly when the power supply is interrupted before starting the reading continuously. I am considering this in my layout .. the point is that I have not found another way to make a sensor work if it doesn’t start on the first attemp. Then this harware fix solve theses issues.
May be bad sensor?
I have a lot of sensors that don´t work .. without stamp brand/model, did You hear about this?
I'll be glad to support developent of your lib too.
I put one file to test it..
https://github.com/adbensi/OneWireNg/pull/1/files#diff-acd88268549e9aa747404ae7c530abcd
Best Regards!

Some sensors, independent of the library used, work more properly when the power supply is interrupted before starting the reading continuously. I am considering this in my layout .. the point is that I have not found another way to make a sensor work if it doesn’t start on the first attemp.

Could you check my updated Dallas_Temperature example (last master commit) if it fixes the issue?

Yes, I will do it, I want to understand why too.
I,ll be back :D
Thanks a lot!
Em sex., 21 de fev. de 2020 às 16:43, Piotr Stolarz <
<EMAIL_ADDRESS>escreveu:

Some sensors, independent of the library used, work more properly when the
power supply is interrupted before starting the reading continuously. I am
considering this in my layout .. the point is that I have not found another
way to make a sensor work if it doesn’t start on the first attemp.
Could you check my updated Dallas_Temperature example (last master commit)
if it fixes the issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pstolarz/OneWireNg/issues/2?email_source=notifications&email_token=ACCTHGEKXUOVH2PIHNKPES3REAVGPA5CNFSM4KXXR6GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMT3SNI#issuecomment-589805877,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCTHGD7D2Y4YN5PU557ZF3REAVGPANCNFSM4KXXR6GA
.

I'm closing this issue, since the problem it concerns looks to be fixed. I created a new version of the library 0.4.1 with the bug fixed. Thanks guys for the report and tests.
Piotrek