Quest 2 VR app malfunctions when I press the oculus button to activate the system menu

I have duct-taped together a Rust application that uses OpenXR and glutin to draw a rainbow triangle floating in VR space.
When I press the Oculus button to summon the system menu the system menu appears, but the rest of the environment turns black (I guess my rendering code is no longer being called?) I see this in the logs:
2023-06-21 17:40:37.000 8139-10064/rust.glutin_openxr1 I/VrApi: FPS=72/72,Prd=22ms,Tear=0,Early=0,Stale=0,Stale2/5/10/max=0/0/0/0,VSnc=0,Lat=-1,Fov=0D,CPU4/GPU=2/2,1382/490MHz,OC=FF,TA=0/0/0,SP=N/N/N,Mem=1353MHz,Free=2711MB,PLS=0,Temp=32.0C/0.0C,TW=1.16ms,App=0.52ms,GD=0.13ms,CPU&GPU=1.27ms,LCnt=2(DR72,LM2),GPU%=0.16,CPU%=0.24(W0.27),DSF=1.00,CFL=18.87/23.72,LD=0,SF=1.00
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr: thread '<unnamed>' panicked at 'internal error: entered unreachable code: AInputQueue_preDispatchEvent returned non-boolean -1', /home/thoth/.cargo/registry/src/github.com-1ecc6299db9ec823/ndk-0.7.0/src/input_queue.rs:82:18
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr: stack backtrace:
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    0:       0x789fb42258 - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    1:       0x789fb5cb44 - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    2:       0x789fb3f230 - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    3:       0x789fb4105c - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    4:       0x789fb43500 - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    5:       0x789fb4312c - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    6:       0x789fb43bbc - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    7:       0x789fb4396c - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    8:       0x789fb42728 - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:    9:       0x789fb436b8 - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   10:       0x789fa2849c - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   11:       0x789fb28d68 - <unknown>
2023-06-21 17:40:37.875 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   12:       0x789fa42938 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   13:       0x789fa44ab0 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   14:       0x789fa3a5e0 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   15:       0x789fa3a268 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   16:       0x789fa41c30 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   17:       0x789fa449a8 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   18:       0x789fa3a018 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   19:       0x789fa3c094 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   20:       0x789fa5adec - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   21:       0x789fa37fc8 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   22:       0x789fb12c5c - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   23:       0x789fb1db74 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   24:       0x789fb1b3f0 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   25:       0x789fb1e2d0 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   26:       0x789fb1f944 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   27:       0x789fb1ff20 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   28:       0x789fb1f890 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   29:       0x789fb1bebc - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   30:       0x789fb1af34 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   31:       0x789fb16198 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   32:       0x789fb4565c - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   33:       0x79bb73edf0 - <unknown>
2023-06-21 17:40:37.876 8139-8165/rust.glutin_openxr1 I/RustStdoutStderr:   34:       0x79bb6eb030 - <unknown>
2023-06-21 17:40:37.877 8139-8166/rust.glutin_openxr1 I/OpenXR: ------------ xrDestroySession [start] -----------
2023-06-21 17:40:37.877 8139-8166/rust.glutin_openxr1 W/Telemetry: [OpenXR] xrDestroySession: Session is in running state.;
2023-06-21 17:40:37.877 8139-8166/rust.glutin_openxr1 I/XrRuntimeUtils: XrRuntimeUtils::EndSession
2023-06-21 17:40:37.878 8139-8166/rust.glutin_openxr1 I/TREX: UnregisterInterface: locking mapMutex_
2023-06-21 17:40:37.878 8139-8166/rust.glutin_openxr1 I/TREX: UnregisterInterface: locking mapMutex_ complete
2023-06-21 17:40:37.878 8139-8166/rust.glutin_openxr1 I/TREX: UnregisterInterface: locking mapMutex_
2023-06-21 17:40:37.878 8139-8166/rust.glutin_openxr1 I/TREX: UnregisterInterface: locking mapMutex_ complete
2023-06-21 17:40:37.878 8139-8166/rust.glutin_openxr1 I/TREX: UnregisterInterface: locki

@mutantbob sorry for the inconvenience. If you had searched the repo for this we've already uncovered that the Quest 2 specifically modified Android's reference implementation: https://github.com/rust-mobile/ndk/issues/322
And we fixed that in https://github.com/rust-mobile/ndk/pull/325, which will appear in the next breaking release in the coming month or 2.

Is it possible for me to try the patched version in my project?  I added
[patch.crates-io]
ndk = {path="../../../vendor/ndk/ndk"}
ndk-sys = {path="../../../vendor/ndk/ndk-sys"}

to my Cargo.toml, but got     = note: perhaps two different versions of cratendk_sys are being used? in a compile error, so evidently that wasn't completely effective.  cargo tree shows me that the android-activity crate is using ndk-sys v0.4.1+23.1.7779620 while the master branch in this repo is 0.4.0+25.0.8775105 which seems odd (master having a lower version number than a released crate).

shows me that the android-activity crate is using ndk-sys v0.4.1+23.1.7779620 while the master branch in this repo is 0.4.0+25.0.8775105 which seems odd (master having a lower version number than a released crate).

Yes, because of a horrible mess with this crate getting republished on the same version by out CI, once with and once without version metadata.
I've ran into the patch situation myself as well and have now pushed<PHONE_NUMBER>41819fc2e96c65f94f4433212398f6 to combat this: this resets master to the latest ndk-sys version published, and it doesn't matter that the NDK version metadata is temporarily going down as it was mismatching the source anyway.
Other refs:
https://github.com/rust-mobile/ndk/issues/369#issuecomment-1324763607
https://github.com/rust-mobile/ndk/issues/368
https://github.com/rust-mobile/ndk/pull/370
https://github.com/rust-lang/cargo/issues/11412

I dusted off the old project to see if I could make use of the fixed ndk crate using a [patch.crates-io].
I was not successful.
If i check out the ndk-0.7.0 tag it does not include the fix so I get the error:
2023-10-08 16:17:30.805 16473-16500/rust.vr_gorgon E/RustPanic: internal error: entered unreachable code: AInputQueue_preDispatchEvent returned non-boolean -1

If I check out 6bc7fce867881004d11d1ce4eaf1895b6b318a2b (the last tag before the ndk crate version got incremented to 0.8)
   Compiling android-activity v0.4.3
error[E0412]: cannot find type `size_t` in crate `ndk_sys`
     --> /home/thoth/.cargo/registry/src/index.crates.io-6f17d22bba15001f/android-activity-0.4.3/src/native_activity/glue.rs:683:28
      |
683   |     out_len: *mut ndk_sys::size_t,
      |                            ^^^^^^
      |
     ::: /home/thoth/vendor/ndk/ndk-sys/src/ffi_aarch64.rs:14170:1
      |
14170 | pub type sig_t = __sighandler_t;
      | -------------- similarly named type alias `sig_t` defined here
      |
help: a type alias with a similar name exists
      |
683   |     out_len: *mut ndk_sys::sig_t,
      |                            ~~~~~
help: consider importing this type alias
      |
5     + use libc::size_t;
      |
help: if you import `size_t`, refer to it directly
      |
683   -     out_len: *mut ndk_sys::size_t,
683   +     out_len: *mut size_t,
      |

error[E0412]: cannot find type `size_t` in crate `ndk_sys`
     --> /home/thoth/.cargo/registry/src/index.crates.io-6f17d22bba15001f/android-activity-0.4.3/src/native_activity/glue.rs:691:40
      |
691   |             *out_len = len as ndk_sys::size_t;
      |                                        ^^^^^^
      |
     ::: /home/thoth/vendor/ndk/ndk-sys/src/ffi_aarch64.rs:14170:1
      |
14170 | pub type sig_t = __sighandler_t;
      | -------------- similarly named type alias `sig_t` defined here
      |
help: a type alias with a similar name exists
      |
691   |             *out_len = len as ndk_sys::sig_t;
      |                                        ~~~~~
help: consider importing this type alias
      |
5     + use libc::size_t;
      |
help: if you import `size_t`, refer to it directly
      |
691   -             *out_len = len as ndk_sys::size_t;
691   +             *out_len = len as size_t;
      |

I get the same errors if I use the fd3dee9 .
The compile errors are in the android-activity 0.4.3 crate, so it might be challenging to untangle the dependencies and find a working combination.

@mutantbob you might be better off switching to the beta releases on crates.io for both the ndk and android-activity, where the latter also uses a compatible ndk release instead of patching into an incompatible version.


When launching extension tests, if another Code instance has a folder open used for tests, it silently does not open

This one is a bit of a weird edge case. I was working on my tests and noticed that all of a sudden they weren't working right, and Code wasn't opening the folder it was told.
My launch config looks like this:
{
	"name": "Launch Tests (General)",
	"type": "extensionHost",
	"request": "launch",
	"runtimeExecutable": "${execPath}",
	"args": [
		"${workspaceRoot}/test/test_projects/hello_world",
		"--extensionDevelopmentPath=${workspaceRoot}",
		"--extensionTestsPath=${workspaceRoot}/out/test/general"
	],
	"env": {
		"DART_CODE_DISABLE_ANALYTICS": "true"
	},
	"stopOnEntry": false,
	"sourceMaps": true,
	"outFiles": [
		"${workspaceRoot}/out/test/**/*.js"
	],
	"preLaunchTask": "npm: watch"
},

The problem turned out to be that I'd opened "${workspaceRoot}/test/test_projects/hello_world" in another Code window so I could modify the test project while working on the tests. With this open, when I run the tests, it spawns a Code window but that window silently does not open anything.
It would be nice if this could be handled in a better way than a silent failure, or if I could pass a flag in my launch.json to force Code to open a folder, even if it's already open in another window somewhere (after all, the folder is normally open anyway, it's just a sub-folder of my main Code window rather than having its own instance).

No plan to fix this, also might be fixed in latest


Support device names with underscores (such as 'Default_Monitor') in JSONHelpers.isValidDeviceId

Summary of the Pull Request
Fixes #1821

References
I think this could be related to #1807

PR Checklist

[x] Applies to #1821
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #1821

Detailed Description of the Pull Request / Additional comments
Fixes the JSONHelper::isValidDeviceId by splitting the first section on the '#' instead of by '_'. This allows to use underscores in monitor device names.

Validation Steps Performed

Perhaps you should add a test like this:
TEST_METHOD (DeviceIdWithoutHashInName)
{
    const auto deviceId = L"LOCALDISPLAY_5120_1440_{00000000-0000-0000-0000-000000000000}";
    Assert::IsTrue(isValidDeviceId(deviceId));
}

@ivan100sic
sure.

Nice work! Works as expected!

@GeertvanHorrik if you can add the suggested test it would be great, but we can also add it after the PR is merged. We would like to merge it today.

Will do it now.

Done, ready for another review.


WebClient.Execute Timeout

Hi Tim
Just wondering if you could help me... I have a request time out on one server running Excel 2016 16.0.42
"ERROR - WebClient.Execute: -2147210493 (11011 / 80042b03), An error occurred during execute
-2147012866 (80072efe): The connection with the server was terminated abnormally"
The exact same request on another server running Excel Office 365 16.0.12 works perfectly fine and I have tested the same request with Postman and the data is returned on both servers as expected. Both running v4.1.5.
Do you have any ideas? Appreciate your help.
Jason

Turns out it was a 64bit compatibility issue.
I have to change
Set web_Http = CreateObject("WinHttp.WinHttpRequest.5.1")
to
#If VBA7 Then    Set web_Http = CreateObject("MSXML2.ServerXMLHTTP60") #Else     Set web_Http = CreateObject("WinHttp.WinHttpRequest.5.1") #End If
with a few other changes for setting options...

I ran into this same issue. @jasongray what were the other few changes that you had to make for setting options?

I ran into this same issue. @jasongray what were the other few changes that you had to make for setting options?

WebClient - PrepareHttpRequest function:
#If Win64 Then         Set web_Http = CreateObject("MSXML2.serverXMLHTTP.5.0")     #Else         Set web_Http = CreateObject("WinHttp.WinHttpRequest.5.1")     #End If
then i changed the option calls under the Me.Insecure if statement.
#If Win64 Then         web_Http.setOption 2, 13056         #Else         web_Http.Option........ #End
serverXMLHTTP doesnt allow for redirects so there is no point having it...
#If Win32 Then     web_Http.Option(web_WinHttpRequestOption.web_WinHttpRequestOption_EnableRedirects) = Me.FollowRedirects     #End If
this was a band aid solution for a client running 64bit office version. It may not work for all cases..?

Hi @jasongray
After implementing your mentioned solution I still can't get it to work.
Can you pls post the complete modified code of WebClient - PrepareHttpRequest function?
Maybe I missed something.

I ran into this same issue. @jasongray what were the other few changes that you had to make for setting options?

WebClient - PrepareHttpRequest function:
#If Win64 Then Set web_Http = CreateObject("MSXML2.serverXMLHTTP.5.0") #Else Set web_Http = CreateObject("WinHttp.WinHttpRequest.5.1") #End If
then i changed the option calls under the Me.Insecure if statement.
#If Win64 Then web_Http.setOption 2, 13056 #Else web_Http.Option........ #End
serverXMLHTTP doesnt allow for redirects so there is no point having it...
#If Win32 Then web_Http.Option(web_WinHttpRequestOption.web_WinHttpRequestOption_EnableRedirects) = Me.FollowRedirects #End If
this was a band aid solution for a client running 64bit office version. It may not work for all cases..?


Datafusion upstream merge

Merge upstream/main into the datafusion-sql-planner branch after the recent release.

@ayushdg Does the last commit address what you were talking about?

@ayushdg Does the last commit address what you were talking about?

It does but I now realize it's still missing some of the old logic that was updated for timestampdiff, eg places like this.
To ensure that some of these changes don't slip through, I'm actually considering just including all the changes made to #495 and then removing bit that are not needed in a followup. Apologies if that increases the amount work needed on this pr.


Split cryptnono docs into a how-to and a topic

This PR splits up the cryptnono docs into a how-to and a topic from a single how-to. I am recommending this approach because

It can be overwhelming to have this much information on a single page - smaller chunks are generally more digestible
There is information here (such as future work, R2R considerations) that are important to have documented, but are superfluous to the steps an engineer should take to enable this feature. Hence the recommendation to move them into a topic section that is linked to from the how-to.

📚 Documentation preview 📚: https://2i2c-pilot-hubs--31.org.readthedocs.build/en/31/

Moved into https://github.com/2i2c-org/infrastructure/pull/3681


⚠ Minecraft has degraded performance

In 6183412, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1148 ms

Resolved: Minecraft performance has improved in b9214f9.


[BUGFIX release]: ancestorHostAddons return value

Without this, you cannot build engines with who's names are built-in array methods such as "filter".
The return type of the function is an Object not an Array

Ping @rwjblue @trentmwillis


Rendering to light DOM is not works properly. When parent custom-element is rerendering, child custom-element content is lost.

Example: https://www.webpackbin.com/bins/-KoOzIcTUxPE7oDkLAWq
I described this problem in https://github.com/developit/preact/issues/654
Please vote for updates in preact :) and apply this changes to skatejs

The problem do not appear if I switch the render to https://github.com/snabbdom/snabbdom + https://github.com/Swizz/snabbdom-pragma. So I wait for #1217 :)

Interesting! There's a branch with some of the work already done. It's at the point where we've prepped it to move. @chrisdarroch knows the details but if you see something you might be able to help on, happy to discuss. I opened #1218 to discuss etc there.

Hey @majo44, we just merged moving out the preact renderer. Can you see if there's anything else we need to do here? I can't tell if there is or not. Thanks!

This appears to be fixed when using Skate 5. Here's the updated example, @majo44  https://www.webpackbin.com/bins/-KuYJlTdy2SWIN_FVu7J

@chrisdarroch
Thank you.
But unfortunately  as I see this example not working properly, the attribute change is not triggering the second render. I will look at this a bit later.

hmm, I'll have to leave that part with you and @treshugart. Given that the child is rendering static text and isn't invoking the h() function, I'm not sure whether I'd expect the child to re-render after changing the attribute on the parent. I'm pretty new to react/preact, so I'm missing a load of context to understand what should be expected behaviour here.

Updated bin: https://www.webpackbin.com/bins/-KuYl9Fid-xH_0M81njR.
Ok, so this really isn't an issue with Skate. Sure, the content is lost but this is due to Preact's diffing algorithm. Why not just use Shadow DOM and polyfill it?
Alternatively, you could use something like this: https://gist.github.com/treshugart/e19c2be255448cd31d261f2e8db2127c, but YMMV with it. I'm ambivalent about making it "a thing", as in giving it a repo because of how close we really are to having shadow DOM everywhere.
Here's an updated bin with the pseudo shadow DOM base class: https://www.webpackbin.com/bins/-KuYx4lYlP6D_5JWqbGJ. No CSS encapsulation is happening there, but it is possible. The key with that implementation is that you must implement childNodes.

I'm going to close this now since this is really a renderer issue, not Skate, but I'm more than happy to discuss this further here, or separately.


Project Saadhna Readme Update

Hi Romin,
I have updated the README.md in the root folder of Project Saadhna, if you want any other changes, please let me know.
Thank you,
Bhaarat Krishnan

Thank you

No mention Romin
On Wed, 24 Jan, 2024, 7:31 am Romin Irani, @.***> wrote:

Thank you
—
Reply to this email directly, view it on GitHub
https://github.com/rominirani/googlecloud-indiacommunity-contributions/pull/55#issuecomment-1907225359,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVGCWZOCHKBV3NWLOVWHVRLYQBTQNAVCNFSM6AAAAABCHMTNWKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBXGIZDKMZVHE
.
You are receiving this because you authored the thread.Message ID:
<rominirani/googlecloud-indiacommunity-contributions/pull/55/c1907225359@
github.com>


support building swift/obj-c targets on macos

Right now the swift/obj-c targets cannot build on their own, relying on being wrapped in a ios_test or similar in order to
build with the right libraries. This makes some few surgical changes in order to let the targets build on their own. This opens up for running swift_binaries etc. that consume EM, with a few of the iOS specific bits disabled.
Risk Level: Low, macos only
Testing: Verified that we can build swift_binaries that target
        "@envoy_mobile//library/swift:ios_lib",
        "@envoy_mobile//library/objective-c:envoy_engine_objc_lib",

on macos
Docs Changes: n/a
Release Notes: n/a

This broke reachability on iOS too. Fix here is to enable it for both iOS & macOS: https://github.com/envoyproxy/envoy-mobile/pull/2114


Getting Report Parameters from Report Server

Hey there,
First off, thank you for the great work you have done.
Is there a way to call the function: ReportViewer.ServerReport.GetParameters();
I would like to be able to build the parameter list and hiding the default list.
Thank you!

Hi,
No unfortunately you cannot do that out of the box. The HTML helper renders an iframe to an ASP.NET page hosting Report Viewer control. You can get the source code and add your logic to https://github.com/ilich/MvcReportViewer/blob/master/MvcReportViewer/MvcReportViewer.aspx.cs file


Unexpected keyword argument 'line_terminator' in .to_csv()

From https://github.com/cemanetwork/appmar/issues/7, issue opened by @murataksel:
There is a call to to_csv in the method extract_series. The call uses the keyword argument line_terminator='\n'
In Pandas 1.5, line_terminator was changed to lineterminator. A prompt solution is to downgrade Pandas to <1.5 (e.g. 1.4.4). For forward compatibility, the method call in APPMAR could be changed to lineterminator.
TODO:

Change line_terminator to lineterminator in appmar2.py
Create a new release (minor update)
Create a package and upload it to PyPI (for easy pip install)

I found that .to_csv uses os.linesep by default as line terminator character, so no need to specify a character. Using the default makes the code more robust since os.linesep is \n for linux and \r\n for Windows.
TODO:

Create a new release (minor update)
Create a package and upload it to PyPI (for easy pip install/update)

Did both of them in appmar2.py for windows OS.

Change the line_terminator to lineterminator in appmar2.py
Change \n to \r\n
Now it works.

Murat,
Glad to know that it worked. In the next few days I will create the minor release and update the PyPI package. Thank you very much for the feedback. Please, let me know if I can close this issue or if there is something more to check.
If you have any other questions, feel free to open a new issue; or if it is something beyond APPMAR (e.g. research/academia), you can even send me an email for us to share some ideas: diego.casas [at] ufpr.br
I will be happy to help
Diego

Diego Hi,
Sure. You can close it I guess. Still downloading the whole data for process. I will keep you update while using your tool.
Best.
Murat.


CONFIGURATIONS: Add support for different language models

OpenAI supports multiple LLM's for the same API.
We probably want to add an enum or something to the ApiConfigurations class that allows the consumer to pick a model to use.
I would do what the GPT site does itself and "default" to the current best choice that's not a beta so that this value is not required but having the choice would open up more options for consumers.
I've noticed this is in the request model, but my thinking is that this should be part of the client config so that a full conversation is given to a single LLM for conversational consistency.

https://platform.openai.com/docs/api-reference/models/list

I think this was covered pretty well over recent live stream videos.
I'm gonna close this ticket unless @hassanhabib wishes to keep it open for some reason.


Add about us page

Inform me if anything else needs to be added

Was it assigned to me? I can't find any information regarding that. By the way, I know the issue number I commented on. Can you please tell What should I do now?

if you do any changes or any thing then show me output
as screenshot and connect with us in discord

Can I please get the discord link?

I will share that.

Hello! I haven't received it yet. Have you shared it anywhere?

Just wait

https://discord.gg/8zGnu4YE


Special characters not supported

Tried using it in my french project and characters like é,è,à,ï and other accents aren't supported. I wonder if there's a quick fix.
Thanks

See the source code of custom filter demo → https://taufik-nurrohman.js.org/tag-picker/tag-picker.f.html
Make your own filter this way:
picker.f = function(text) {
    return text.toLowerCase().replace(/* your regex pattern to excludes */, '-');
};

Thanks a lot ! Works well !


add jwt plugin

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1541 (ee7e9e4) into master (1ba85d6) will decrease coverage by 0.74%.
The diff coverage is 44.68%.

@@             Coverage Diff              @@
##             master    #1541      +/-   ##
============================================
- Coverage     65.67%   64.93%   -0.75%
+ Complexity     2272     2269       -3
============================================
  Files           447      451       +4
  Lines          9385     9431      +46
  Branches        938      943       +5
============================================
- Hits           6164     6124      -40
- Misses         2749     2836      +87
+ Partials        472      471       -1

Impacted Files
Coverage Δ

...che/shenyu/plugin/jwt/lambda/ThrowingFunction.java
0.00% <0.00%> (ø)

...n/java/org/apache/shenyu/plugin/jwt/JwtPlugin.java
32.35% <32.35%> (ø)

...ava/org/apache/shenyu/common/enums/PluginEnum.java
100.00% <100.00%> (ø)

...shenyu/plugin/jwt/handle/JwtPluginDataHandler.java
100.00% <100.00%> (ø)

...oot/starter/plugin/jwt/JwtPluginConfiguration.java
100.00% <100.00%> (ø)

...enyu/client/tars/TarsServiceBeanPostProcessor.java
32.00% <0.00%> (-58.67%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
66.66% <0.00%> (-27.78%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
52.63% <0.00%> (-11.58%)
:arrow_down:

...in/alibaba/dubbo/cache/ApplicationConfigCache.java
75.86% <0.00%> (-4.60%)
:arrow_down:

...he/shenyu/plugin/grpc/client/ShenyuGrpcClient.java
9.09% <0.00%> (-4.55%)
:arrow_down:

... and 6 more

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 1ba85d6...ee7e9e4. Read the comment docs.


[DreamBooth] Optional cell not optional

It says to skip the cell if one doesn't want to upload class images, but skipping causes the "Start DreamBooth" cell to throw errors.
Running the cell and then canceling the upload seems to work fine, though.

fixed, update the colab and let me know if it's fine

Seems to work now, thank you!

Great !


A few changes to histogram functionality

@mronian, I was looking at the histogram functionality and noticed a few changes that seemed to make things simpler and faster. It's probably better if you review each commit separately; the first one just moves some code around, which makes it hard to see what I actually changed.
This is WIP because right now this won't pass the tests in #512 due to some problem in ColorTypes that I don't have time to track down right now, but if I comment out one line in that package then a suitable version of your tests do pass. So no one should merge this until I have time (not today) to fix the other problem. (Fixed in https://github.com/JuliaGraphics/ColorTypes.jl/pull/41, though we still need to tag a new version before tests will pass.)

Coverage increased (+0.5%) to 75.457% when pulling 59ecb29770ef6c391a561211a9cc8dbf7539b961 on teh/hist_cleanup into 132999a56e604e8ac3011658b31fbe5d5b447a6d on master.

If you could merge this, I will change the tests in #512 to test the new functions.

Coverage increased (+0.4%) to 75.356% when pulling dfce601b69f1be4f1bb053e25e97ca407c4e4e04 on teh/hist_cleanup into 132999a56e604e8ac3011658b31fbe5d5b447a6d on master.

Test failures are independent of this PR (in julia itself or in other packages).

Just referencing this issue here JuliaGraphics/ColorVectorSpace.jl#44 so that we fix this once the Gray{Float64} is handled.


allow .slack.com domain

Towards: https://github.com/giantswarm/giantswarm/issues/30612
On installations behind a proxy, the Slack API (at https:/slack.com/api) is currently not reachable. This is required in order for Alertmanager to be able to post alerts on Slack using the new https://slack.com/api/chat.postMessage API
This PR add slack.com in the list of allowed domains and also keeps compatibility with hooks.slack.com currently used by most installations (this can later be removed when all Alertmanager are using the new Slack API).

@giantswarm/team-phoenix is this enough to allow .slack.com on all installations or does this change needs to happen somewhere else for our customer installations ?


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3118

Questions ?
Please refer to the Backport tool documentation


Added Workflow Settings  - Fix for Issue #43

Hi @NikCharlebois, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

I'll pump this one through a quick test this week, I also want to review the tests as well to see if we should update the scenarios in the test methods for the web app object to account for these new properties as well. If you could have a quick look and see if you can add to that first as well that would be great.

Thanks Brian. Are you talking about the xSharePoint.xSPWebApplication.Tests.ps1 script or the actual Test-TargetResource function of the Resource?

Sorry should have specified - was talking about the tests.ps1 file. I've been meaning to review this resource to look at what happens with optional parameters where they aren't specified in the configuration (for example, I manually turn on one of the workflow options, but don't specify a setting for it in the DSC - does the DSC default to false as the default for a bool value, and then turn it off for me or does it ignore it and leave the current setting). This is a wider issue across some of the other resources we have but I wanted to throw it out there to discuss seeing as we are adding more of these types of values so we can get it sorted before we get it out of hand.

I've got us covered for this :). I will be sending another pull request� later today to add the throttling params to web application, and I've added code to handle the case where optional params are not passed to $PSBoud
Sent from my BlackBerry 10 smartphone on the Rogers network.
From: Brian Farnhill
Sent: Wednesday, August 5, 2015 5:53 AM
To: PowerShell/xSharePoint
Reply To: PowerShell/xSharePoint
Cc: Charlebois-Laprade, Nikolas
Subject: Re: [xSharePoint] Added Workflow Settings - Fix for Issue #43 (#63)
Sorry should have specified - was talking about the tests.ps1 file. I've been meaning to review this resource to look at what happens with optional parameters where they aren't specified in the configuration (for example, I manually turn on one of the workflow options, but don't specify a setting for it in the DSC - does the DSC default to false as the default for a bool value, and then turn it off for me or does it ignore it and leave the current setting). This is a wider issue across some of the other resources we have but I wanted to throw it out there to discuss seeing as we are adding more of these types of values so we can get it sorted before we get it out of hand.
�
Reply to this email directly or view it on GitHubhttps://github.com/PowerShell/xSharePoint/pull/63#issuecomment-127937438.


Face detection using EXIF

This is a table of all camera models that need to be checked.
The info here is the make of the camera and some exif data which is normally associated with the make.

Sony - FacesDetected, Face1Position
Fujifilm - FacesDetected, FacePositions, Face1Name, Face1Category, Face1Birthday
Nikon - FacesDetected, FaceDetectFrameSize, Face1Position,
Panasonic - FacesDetected, Face1Position, RecognizedFace1Name, RecognizedFace1Age
Pentax - FacesDetected, Face1Position, FacePosition
Sanyo - FacesDetected, FacePosition
Casio - FacesDetected, FaceDetectFrameSize, Face1Position
Ricoh - FacesDetected, FaceDetectFrameSize, Face1Position
Canon
Old versions -  FacesDetected, FaceDetectFrameSize, FaceWidth, Face1Position
New versions - ValidAFPoints, AFImageWidth, AFImageHeight, AFAreaXPositions, AFAreaYPositions, PrimaryAFPoint
Olympus - FacesDetected, FaceDetectArea, MaxFaces, FaceDetectFrameSize, FaceDetectFrameCrop

Note that the way the position is written in FacePosition is different for various makes and models.
The perl code given on exiftool forum can be found here: http://u88.n24.queensu.ca/pub/facetest.pl

What about asking the users on:
https://en.wikipedia.org/wiki/Wikipedia:Requested_pictures
to check their cameras and provide examples?

https://commons.wikimedia.org/wiki/Commons:Picture_requests

After a fair about of digging around, I found the following images on commons where I could get exif data:

Sony
https://commons.wikimedia.org/wiki/File:2015_NAMM_Show_-_DSC00396_(16344792356).jpg
Fujifilm
https://commons.wikimedia.org/wiki/File:Gabriele_Paolini_Roma_20-9-2015_2.jpg
Nikon
https://commons.wikimedia.org/wiki/File:Alexander_Fomichev_2011-10-27.JPG
Panasonic - FacesDetected, Face1Position, RecognizedFace1Name, RecognizedFace1Age
https://commons.wikimedia.org/wiki/File:Adriana_Cerretelli.JPG (seems corrupt ? Faces Detected is 0, but Face1Position is not 0)
https://commons.wikimedia.org/wiki/File:Anra_Kennedy_at_the_editathon_at_the_Royal_Armouries_2016-03-24_01.jpg (FacesDetected is set to 0 ...? But Face1Position is there)
https://commons.wikimedia.org/wiki/File:9th_Coventry_Wikimeet_07.jpg (FacesDetected is set to 0 ...? But Face1Position is there and seems credible !)
Pentax
https://commons.wikimedia.org/wiki/File:Quentin_Bryce_and_Neil_Willmett.JPG
TODO Sanyo - FacesDetected, FacePosition
TODO Casio - FacesDetected, FaceDetectFrameSize, Face1Position
TODO Ricoh - FacesDetected, FaceDetectFrameSize, Face1Position
Canon
TODO Old versions -  FacesDetected, FaceDetectFrameSize, FaceWidth, Face1Position
New versions
https://commons.wikimedia.org/wiki/File:Annagrah-2_041.JPG
https://commons.wikimedia.org/wiki/File:Bangla_Wikipedia_10_year_Founding_Anniversary_Conference_2015_(130).JPG (EOS model, EOS models have different position encoding)
Olympus - FacesDetected, FaceDetectArea, MaxFaces
https://commons.wikimedia.org/wiki/File:Myriam_Chalek.jpg


ReferenceError: sid is not defined

When I try to compare state to other state using the TypeScript abstraction, finish: { whenAny: s['count'].gt(s['limit']) } I get the following stacktrace.
Stack Trace:
ReferenceError: sid is not defined
    at Object.that.define (/Users/enriched/git/scratch/node_modules/durable/libjs/durable.js:142:21)
    at Object.that.define (/Users/enriched/git/scratch/node_modules/durable/libjs/durable.js:150:45)
    at Object.that.define (/Users/enriched/git/scratch/node_modules/durable/libjs/durable.js:390:41)
    at Object.that.define (/Users/enriched/git/scratch/node_modules/durable/libjs/durable.js:810:30)
    at Object.that.define (/Users/enriched/git/scratch/node_modules/durable/libjs/durable.js:853:52)
    at Object.that.define (/Users/enriched/git/scratch/node_modules/durable/libjs/durable.js:906:64)
    at module.exports.createHost (/Users/enriched/git/scratch/node_modules/durable/libjs/durable.js:931:62)
    at Object.module.exports.runAll (/Users/enriched/git/scratch/node_modules/durable/libjs/durable.js:951:25)
    at Object.<anonymous> (/Users/enriched/git/scratch/built/index.js:57:11)
    at Module._compile (module.js:425:26)

When I change to finish: { whenAny: s['count'].gt(10) } it works as expected.

Hi Rich, I'm sorry for taking so long on my reply. Can you please try writing the expression above as follows?
finish: { whenAny: s['count'].gt(c.s['limit']) }
Please note the c.s. In your case the code should be throwing a meaningful exception. I will fix that.

Hi Jesus, thanks for the reply and sorry for the false alarm.  Using c.s appears to work, although the c.s generates an error in Typescript because 's' doesn't exist on the Model type.  I can do (c as any).s['limit'] for now and should be able to move forward 👍


Merge Checkout with Cart

I want to merge this change because it closes #2004. Checkout class is removed and all data needed for that process are now saved in database within Cart model. Also, the business logic standing behind handling shipping and billing addresses in checkout is simplified.
TODO's

[ ] Simplify and refactor existing tests
[ ] Test new functionalities

Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[ ] JavaScript code quality checks pass: eslint.

This looks good! One thing that I'm not convinced of is the file structure, particularly the nested /cart/checkout/ directory. I think "checkout" is a more general notion and "cart" is only a part of the whole flow, so maybe this app should be called "checkout" instead?

I still don't get one part of the Checkout code: why are cart session expiring after 10 years?
https://github.com/mirumee/saleor/blob/e2fb44dfcb466413bfc0c402d8b7e36f2cfcbca9/saleor/cart/utils.py#L20-L23
In France, we have a 13 months limit for cookies, so I always have to lower this value in production.

10 years sounds... interesting. I'd say a month is more than enough for a cart. I'd also like to know why we can't store it in session instead of a separate cookie.


[New game]: 2048 game

🎮 Game Request
Game logic and basic description
2048 is a popular single-player sliding tile puzzle game where the objective is to combine numbered tiles on a grid to create a tile with the number 2048.
The goal is to create a tile with the number 2048. However,  the players can continue to play to achieve higher scores by creating larger tiles.
Point down the features
Game points
How to Play??
->The game is played on a 4x4 grid.
->Tiles: The grid starts with two tiles, each displaying either a "2" or a "4". New tiles appear with each move, either "2" or "4".
->Moves: Players can slide tiles left, right, up, or down using the arrow keys.
->Combining Tiles: When two tiles with the same number collide while moving, they merge into a single tile with a value equal to the sum of the two original tiles (e.g., two "2" tiles merge to form a "4" tile).
->Game Over The game ends when there are no empty spaces on the grid and no possible moves left to combine tiles
->The game ends when the grid is full and no valid moves are left (i.e., no adjacent tiles can be merged, and no empty spots are available).
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @nikki-05 , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#4564

Hey @nikki-05 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @nikki-05 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @nikki-05!
We are already having a similar game request in #4603 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.


Changed GitLab adapter to retrieve only the accessible projects

The reasoning behind this change is that making the adapter to retrieve
"all" projects would require the API user to have administration access
level, while getting just the accessible project would enable the users
of Packages to create an user with a more controlled set of permissions
(ie: "Reporter" permission on a group or individual repositories).

Of course, Master access would be needed to enable or disable webhooks. The administrator could add them manually when enabling the packages, though.

+1

Thanks!


[Bug] I can't play in remote play, black screen or error

Checklist

[X] I will only use English in my report.
[X] I have used the search function for open and closed issues to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

Questions

[ ] xCloud officially supports my country/region.
[ ] The bug doesn't happen when I disable Better xCloud script.
[ ] The bug didn't happen in previous Better xCloud version (name which one).

Device
Laptop
Device
DESKTOP-2HRVHVQ
Operating System
Windows 10
Android app/Browser Version
Opera GX Version LVL 5 (core: 109.0.5097.142)
Better xCloud Version
5.4.2
Game list
Remote Play
Reproduction Steps

I open the page 2. I hit the remote play icon 3. I hit connect 4. Error

Screenshot/video

Dup #229


User EOF symbol

Allow user to put a token that is interpreted as the end of a file where-ever they chose.
This can be as simple as an open /* comments out the rest of the file. This wouldn't speed up the scanner but the scanner isn't the slow bit at the moment.

This already works with /*


Fill in method profile

I decided to fill a lot more content than I originally planned. The only profile that is lacking most of its implementation is the method profile. I should fill in useful methods for this before the release.

Should be fairly straightforward. It's just implementing the remaining abstractions for the method profile in the same style as the other profile classes. It already has some of the content.


Create Terraform Plans For Generating Test Infrastructure in GCP

Similar to https://github.com/magneticstain/tf-ip2cr

Work for compute engine vms is complete

GCLB support added to tf: https://github.com/magneticstain/tf-ip2cr-gcp/commit/e0f9b666dd706e762403cb8b9f956da6768b34a6

CloudSQL generation added: https://github.com/magneticstain/tf-ip2cr-gcp/commit/5939f74411d3cd20a9075feae5a5f7467c42a249


Follow day night theme in editor

I don't have time to prepare this for merge
Just posting it here for the community and closing
I don't have time for cosmetics only usability
This does what I need

It would be better if we could set a separate color schemes for dark and light themes in settings


capacity.total_report sometimes don't return all information

Installation information

jumpscale version: development
jumpscale_lib : commit e6f7728a1b8a6639f785f9d1d76cd36afdf15f31
operating system: Ubuntu 18

Detailed description
using node zero-os with ip '<IP_ADDRESS>'
on ipython
from jumpscale import j
node = j.clients.zos.get('test', data={'host': '<IP_ADDRESS>'})
node.capacity.total_report()

Actual Result

I hadn't all info for the same node except once.

first time: no info about processor and motherboard.
second time: no info about processor.
third time: no info about processor.
forth time: no info about processor and memory.
fifth time: no info about processor and motherboard.
sixth time: had all info.
seventh time: no info about processor and memory.

ok that is indeed maybe a bug, but the actuall information we case about is the resouce units from the report. Can you check that these are always the same and correct ?

memory size for example:

8192 MB is the correct one
null
No Module Installed
don't have the same info

memory size for example:

8192 MB is the correct one
null
No Module Installed
don't have the same info

??
That's not what I was talking about on the total_repo object, you have some properties: HRU,CRU,MRU,SRU this is what is used to register the capacity

@zaibon

When I get 0 CRU's

When I get 1 CRU

When I get all (8) CRU's


support create TCP connection with timeout

Hi stjepang
Please review it. thank you.

Oh, maybe I can use futures::select macro.

Thanks for the PR and sorry for late reply! I don't think timeouts are necessasry for establishing TCP connections because async Rust already provides tools for creating timeouts on arbitrary futures.
And there is now an example on how to use timeouts in the crate root :) https://docs.rs/async-io/0.1.11/async_io/#examples

Oh, that's great. thank you for your hard work.


Disabled State

Hello,
In Photon, buttons have an active class available to them. It would be valuable if a disabled class was also available. An example of a disabled button can be seen in the Finder in OS X.

Open the Finder.
Click "Documents"
In the toolbar, you'll probably notice that the "Edit Tags" and "Share the selected items" buttons are disabled until you select one or more files.

FWIW, an attribute selector would fit here as well/better.

@sbruchmann Agreed!

Thanks for the PR @iamdade

Fixed in 78236888ba306ee81c10b8d873e2d8ab816adce9


Add option for LTO

Sometimes it is interesting to see and discuss the generated assembly for code with LTO turned on. For example, this code:
extern crate itertools;
use itertools::Itertools;

pub fn foo(x: u32, y: u32) -> bool {
    vec![x, y].into_iter().dedup().count() == 1
}

gets compiled down to just x == y when compiled with --release and lto = true, which is pretty wild. But there's no way to share or discuss that easily online, since this site doesn't allow for LTO and godbolt's site doesn't allow for external crate imports. So, it would be nice if this site allowed the users to turn on LTO.
If the top bar is getting too crowded, maybe it makes sense to have a drop-down box where one can specify additional, less-used compiler options? I'm not sure how best to go about this from a UI standpoint but it would be nice to have.

/cc @chandlore and #272


Update XRI package

upgrade XR Interaction toolkit to 2.2.0
check in the interaction list and move to new XRInteractionToolkit folder (this will prevent asking about upgrading upon opening project for the first time and creating a new folder)
Check in all samples: previously just uploaded a subset of the samples... if I upload them all you have access to more stuff to use such as the Device Simulator.

Thanks @joseph-kaile, this is great! Very glad to remove the message about upgrading.
The only thing I'm not sure about are the additional samples - let's talk in standup about whether or not they're something we want to include in the project. An argument could be made either way - providing the user with more out-of-the-box functionality for the scenes they'll build off the project, vs only including the minimum required for our sample project.


Have add-secrets / rotate-secrets visibly tick through each item

Right now, longer sets of credentials appear to be stalling for long periods of time when, in fact, they're generating several 4096-bit RSA keys (which can take awhile). Listing down each item as its being performed (in a style similar to check-secrets) would help the user see that progress is being made.

This will be delivered in v2.7.0

This was delivered in v2.7.x


No easy way to import main function

Hi,
I found no straight forward way to use the main function of the script (convert kml to geojson) inside a python program, as an imported module. The workaround I use is this (based on ways to test program on click)
import kml2geojson.kml2geojson
from click import CliRunner

runner = CliRunner()
runner.invoke(kml2geojson.kml2geojson.main, ["file.kml", "./"])

Trying to use kml2geojson.main(kml_path="file.kml", output_dir="./") directly result in error  in click module.
Traceback (most recent call last):
  File "./script.py", line 41, in <module>
    kml2geojson.kml2geojson.main(kml_path="file.kml", output_dir="./")
  File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 695, in main
    with self.make_context(prog_name, args, **extra) as ctx:
  File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 618, in make_context
    ctx = Context(self, info_name=info_name, parent=parent, **extra)
TypeError: __init__() got an unexpected keyword argument 'output_dir'

It would be great if the main function could be used directly on a python script, not just as CLI program.
If you agree, I could separate the main() function with the CLI parser and send PR request. I think it will make the main() possible to be called in other module. 😄

Hi adhikasp, i just noticed your issue.
Sorry for my delayed response.
Yes, good idea to separate that functionality.
I was a Click newbie when i wrote kml2geojson.
Now that i have a little more experience, i think that a clean way to do the separation would be create another module called cli.py in which a copy of the main function would reside.
I took that approach when i wrote the Wavetrace package.
I'll work on the separation tomorrow.

Done. Give version 4.0.0 a whirl.


INITIAL_CALLS_STREAM_BATCH_SIZE

Description

Fixes WB-NNNNN
Fixes #NNNN

What does the PR do? Include a concise description of the PR contents.
Testing
How was this PR tested?

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3169  👈 (View in Graphite)
#3168
#3167
master

This stack of pull requests is managed by Graphite. Learn more about stacking.


Error: Payment id is invalid

The payment works using the Mollie payment method. But when the user is getting redirected after paying, they see a fullscreen error:
An unexpected error occurred:
Error executing API call (request): The payment id is invalid.

The user is redirected to the url, see below (3 is the latest order). I also see that error when visiting that page. When entering "4 (none existing order)" I get redirected to empty shopping cart.
domein.ext/index.php/checkout/ordercompletion/3
The order is valid because I see a succesfull order in Mollie and in the Community Store. The user also gets an e-mail that the order was made. But they see an error after payment.
Any Idea what is going wrong?

I think this one is best raised on the specific issues list for the Mollie payment method:
https://github.com/concrete5-community-store/community_store_mollie/issues
There's been quite a few updates to this payment method just recently, and a release put out, are you using latest release here?   https://github.com/concrete5-community-store/community_store_mollie/releases
I don't know a huge amount about this payment method - @Jozzeh may be able shed some light on this error.

I cleared the cache and looked in their Mollie account for problems. At their websiteprofile they put in their domein.ext instead of shop.domein.ext. I made a test order and did not received any errors. I guess that was the problem.
Thanks you for the information.


Tech task – Remove duplicate ID's in tool tips

See screengrab of accessibility doc below.

Rolled up into #215


Missing path in Getting Started documentation

It appears that with the release of Carla 0.9.3 in the Getting Started documentation the following needs to be added to PYTHONPATH:
${CARLA_ROOT}/PythonAPI/agents/navigation
Also, it would be good to explicitly add to the instructions the step to pip install networkx .

I think this should go to CARLA directly, as networkx is a requirement for the CARLA agents. I add @nsubiron in the loop for this.

Updated the documentation. Should be correct now.


dbTable

dbTable should be in the code as per https://github.com/Azure/azure-sqldb-spark

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

ID: 3a43d656-dbdf-ae04-f88d-488aa1dc78c5
Version Independent ID: 155c9eb2-92df-1619-4284-17c46a419f1f
Content: Spark Connector with Azure SQL Database and SQL Server - Azure SQL Database
Content Source: articles/sql-database/sql-database-spark-connector.md
Service: sql-database
Sub-service: development
GitHub Login: @denzilribeiro
Microsoft Alias: denzilr

@marktab
Thanks for your feedback! We will investigate and update as appropriate.

@marktab Thank you for the feedback.
This is being assigned to the content author to evaluate and update the documentation as appropriate.

FYI -- the specific area is under AAD, the call is expecting a table to be named
https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/sql-database/sql-database-spark-connector.md#connect-spark-to-azure-sql-database-using-aad-authentication


Preloaded Time and date IOS styling Issue

https://jsfiddle.net/tj2Lo4z7/
Your Environment

flatpickr version used: current
Browser name and version: Safari for IOS
OS and version: IOS 16.2

I have noticed an issue on my website with preloaded time and time dates where they are getting shaddow class that is cause them to be styled differently. Please see the image attached (from flatpickr.org/examples).

Steps to Recreate:

on Safari for IOS head to https://flatpickr.js.org/examples/
Scroll down to preloaded time and dates
observe the issue

Seeing the same thing.


Take into account element's "maxlength" attr for the JS value setter

Otherwise additional issues could appear in case if there isn't server-side value length validation.

@shimkiv Thanks, will merge it!
Two questions:

Why "Math.abs" is needed?  I guess "maxlength" cannot be negative.
Why this should be optional? I suggest to enable this behaviour by default.

I also suggest to add another feature to "setValue": if element is readonly or disable, setValue() should do nothing, or even throw an exception.

@asolntsev thank you !

Why "Math.abs" is needed? I guess "maxlength" cannot be negative.

just in case :)

Why this should be optional? I suggest to enable this behaviour by default.

to maintain backward compatibility (for those who are already use JS set val feature)


what does --gc do in tests?

The tests run as tap --gc test/js/*-test.js. What does the --gc flag do?

You mean --expose_gc?

Nope I mean --gc :) (Check package.json)

--gc is a shorthand parameter for --expose-gc in the tap module so that the gc() function is present in tests.

Thanks for quick reply guys! Closing.


Add documentation when modules cache is corrupted

Is your feature request related to a problem? Please describe.
Sometimes nf-core lint crash due to the /.config/nfcore/nf-core/modules/ is corrupted
Describe the solution you'd like
Will need documentation to identify the problem and add the command to be executed to correct the problem.
Describe alternatives you've considered
I could be also be done automatically when tools crash

See https://nfcore.slack.com/archives/CE6SDBX2A/p1679053254975209
for more infos

Solve with #1693


Occasional endless loop in unit test

An excerpt from TestParquetMetadataConverter.randomTestFilterMetaData:

    for (int j = 0; j < 100; j++) {
      // ...
      int splitSize = random.nextInt(10000);
      // ...
      verifyAllFilters(melead tadata(rgs), splitSize);
      // ...
    }

An excerpt from verifyAllFilters(FileMetaData md, long splitWidth):

    for (long start = 0; start < fileSize(md); start += splitWidth) {

When random.nextInt(10000); returns 0, this will result in an infinite loop.
Reporter: Zoltan Ivanfi / @zivanfi
Assignee: Zoltan Ivanfi / @zivanfi
Related issues:

Release Parquet Java 1.10 (blocks)

Note: This issue was originally created as PARQUET-1208. Please see the migration documentation for further details.

Zoltan Ivanfi / @zivanfi:
https://github.com/apache/parquet-mr/pull/455


[MB-11580] Add helper function to get history log display name

Jira ticket for this change
Summary
From this worksheet we need to map the operationId or eventName to a human readable name in the history log.
Design:

Zeplin
Refer to the data content miro board to see what needs to be shown

Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps

Access the
Login as a

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

Screenshots

hey @jacquelineIO is this ready for review now?

Fails

:no_entry_sign:

New files have been created under one of the legacy directories
(src/shared or src/scenes). Please relocate them according to the file structure described here.
View the frontend file org ADR for more information

Messages

:book:
:link: MB-11580

  Generated by :no_entry_sign: dangerJS against 4b8dbef2d083e2b2af698cc4a06744d594a6639f


this._formData.on is not a function

Any clue what is going on? Stack trace looks like it gets thrown at .field?
  handleImageUpload(file) {
        let upload = request.post(CLOUDINARY_UPLOAD_URL)
                            .field('upload_preset', CLOUDINARY_UPLOAD_PRESET)
                            .field('file', file);

        upload.end((err, response) => {
          if (err) {
            console.error(err);
          }
          console.log(response);

          if (response.body.secure_url !== '') {
            this.setState({
              uploadedFileCloudinaryUrl: response.body.secure_url
            });
          }
        });
      }

This could only happen here, in Node-JS-only bit of code:
    this._formData = new FormData();
    this._formData.on('error', function(err) {

which would happen if FormData was incorrectly imported. Are you perhaps using Webpack or something similar that could accidentally create a mix of Node-only and browser-only modules?

We now have a getter that creates this object automatically. Hopefully it'll work.

@pornel
Having this exact same issue on both 3.5.3-beta.1 and 3.5.2
I'm on the meteor framework so using Meteor's webpack build compiler. Do you have any suggestions on tweaking an image upload request like in acomito's example for when this._formData is undefined?
Thanks for your help.

Webpack has a target setting, which has to be "browser" or "node" accordingly to the environment the code is going to run in.


CI/CD - accumulo

Package and deliver to the customer...
Dev Server...
Accumulo
!story G<IP_ADDRESS>
!component CI/CDA

Likely not going to make it into this project.


SCCP-361-Base-Sepolia

Hi @noisekit hoping to send 361 on both mainnet and testnet on both arbitrum and base, on Friday after pyth confirms price readiness

Test Issue is unrelated and being fixed in separate pr


Update csv for resource exclusion

What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
ArgoCD prunes TaskRun and PipelineRun objects if pipeline is manged by it. To avoid the pruning, an exclusion rule needs to be added explicitly. An example below. The exclusion should happen by default.
resourceExclusions: |
    - apiGroups:
      - tekton.dev
      clusters:
      - '*'
      kinds:
      - TaskRun
      - PipelineRun

Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
https://issues.redhat.com/browse/GITOPS-1876
Fixes #?
GITOPS-1876
Test acceptance criteria:

[ ] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:
Create the catalog source image and test by creating a new instance from the console.
NOTE : This will only work for Argo instance created from console not CLI

/retest

/test v4.8-e2e

The resourcesExclusions will only be set when the CR is created from the Operator console and doesn't have any effect on CRs created declaratively. Are we okay with restricting it to the console?

Thanks @saumeya , This PR will be automatically merged once the tests get passed

The resourcesExclusions will only be set when the CR is created from the Operator console and doesn't have any effect on CRs created declaratively. Are we okay with restricting it to the console?

Yes, I confirmed with @wtam2018 about the exception

/test v4.7-e2e


error while obfuscating luau

for some scripts that i obfuscate i get this strange and cryptic error
i need this fixed because im trying bypass roblox's moderation

this is a lua syntax error.
check your script to obfuscate is ok.

Luau is not supported.

i managed to fix it in the end, thanks


Update versioning-policy.md to add the type addition condition

Following this internal discussion: https://meilisearch.slack.com/archives/CD7Q2UKGB/p1670853892183669
@brunoocasali and @irevoire I would like to have your review on this 😇

For your information @meilisearch/engine-team


recommend plotting of data?

Great resource, Huang! Im looking forward to also using your latest ANCOM-BC. How do you recommend plotting the abundance data? This question is a follow-up to your suggestion above from May 29th...
"I agree with samd1993, you can visualize the ANCOM results by the relative abundance plot (actually this is what we have been doing in our group when using ANCOM), but keep in mind that ANCOM is designed for testing absolute abundances, not relative abundances (values between 0 and 1), you need to be cautious in interpreting results using relative abundances."
Do you recommend plotting transformed absolute count data to take into account library sizes? So, like EdgeR TMM transformation or using the processed feature table form ANCOM-II. Or do you just take the raw data and transform it for plotting relative abundance?
Originally posted by @KMKemp in https://github.com/FrederickHuangLin/ANCOM/issues/11#issuecomment-740679276

Hi @KMKemp,
Thanks for your interest in ANCOM and ANCOMBC! Per your question, here are my suggestions:

Visualize the normalized abundance data. As you mentioned, you can first normalize the observed abundance data by some normalization factors (e.g. TMM), and then plot the normalized abundance data. However, ANCOM does not have the normalization step, so there are no normalization factors generated by ANCOM. If you are using ANCOM-BC, you can follow section 3.47 of the vignette here to obtain normalization factors (sampling fractions) and normalized abundance data.

Visualize the relative abundance data, which is also commonly done in microbiome research. For example, the "microbiome" R package has a few functions for visualizing relative abundances.

Visualize the centered log-ratio (CLR) instead of relative abundances using volcano plots as shown in the tutorial, you can also find an eaxmple here in QIIME2.

Best,
Huang

Thank you for the clarity and helpful suggestions! I'm closing this out!


fix: Move all componentWillUnMount functionality to respective events

Closes #1515
Move all functionality from componentWillUnMount lifecycle
@acao can you review this PR

awesome work @harshithpabbati ! |
we just need to make sure to debounce the more high-throughput events to avoid performance/render-blocking issues, as invoking lots of storage.set() events can sometimes lead to issues like that
as for the delay, 500-1000ms should be fine. these values only need to be persisted in case the user navigates away or refreshes. this will essentially delay the write until the user has finished, or if there's lots of dragging or variables editing going on, it will only write to ls every 500ms-1000ms, rather than writing to localstorage 60 times a second or more

@acao should I debounce the whole function ?

@harshithpabbati no, only the storage write events

if you debounce the setState() events, we get weird issues with resizing panes as you can see

@harshithpabbati no, only the storage write events

ok, I will update that

if you debounce the setState() events, we get weird issues with resizing panes as you can see

oh, yeah true 👍

@harshithpabbati why did you close this PR?

@harshithpabbati why did you close this PR?

That's because we found a better solution than this.

@harshithpabbati i'm not sure that's the case, we just found a better way to reproduce the error

Oh is it ? Then cool!

@harshithpabbati i'm not sure that's the case, we just found a better way to reproduce the error. is there a conversation that i did't see where you and Mateo decided his solution was better?

No, we didn't have any discussion

@harshithpabbati yes, as i mentioned in discord, we will wait until both solutions are ready, and make sure we can reproduce the error with both and see if one doesn't solve it better than the other. both proposals are welcome.

Okay, so there is an error when I use denounce
If I make it like this debounce(500, this._storage.set('editorFlex', JSON.stringify(this.state.editorFlex)));
then this error is showing up
Argument of type '{ isQuotaError: boolean; error: any; }' is not assignable to parameter of type '(...args: any[]) => any'.
Type '{ isQuotaError: boolean; error: any; }' provides no match for the signature '(...args: any[]): any'.ts(2345)
Should I make a function for storage set and then pass the variables ?
@acao can you help me with this?

Done updated @acao :)

looking good @harshithpabbati !
as a rule of thumb, any event that has its events triggered many many times a second, such as resize or keyboard stroke, should debounce all the localStorage.setItem() calls (via storage.set())

looking good @harshithpabbati !
as a rule of thumb, any event that has its events triggered many many times a second, such as resize or keyboard stroke, should debounce all the localStorage.setItem() calls (via storage.set())

Sure, I will update it soon

@acao is this PR helpful ?

@harshithpabbati it might still be?

lol oops! tried merging via UI, fixing it now. gonna try and see if this resolves our user's issue


Visual block selection doesn't begin at start of line

I get a problem similar to https://github.com/lukas-reineke/indent-blankline.nvim/issues/41 but it seems to be specific to certain color schemes. I can't see anything in working color schemes that are making it work so I was hoping you could provide insight on if this is a bug (in blankline or the color scheme) or if color schemes need to implement something special to get this working right and what that special something is so I can customize it.
f.ex. the following vimrc doesn't work in neovim 0.5.1 and kitty 0.23.1 (displays behavior similar to the above bug where visual block selections don't show where the indent is located):
call plug#begin()
Plug 'lukas-reineke/indent-blankline.nvim'
Plug 'overcache/NeoSolarized'
call plug#end()
set termguicolors
syntax enable
set background=dark
colorscheme NeoSolarized

NeoSolarized sets the Visual highlight group to gui=reverse
Virtualtext does not support that yet, I think.
You can change it to normal colors.
For example :highlight Visual guibg=#839496 gui=None guifg=#002b36

Thanks, that works!
I couldn't find anything that looked like a relevant bug in neovim; do we need to file one?

I was wrong, virtual text does support this.
But the plugin turns mixing of gui attributes off on purpose. This is so the indent guides don't inherent things like underline or italic.
You could overwrite the highlight groups to remove this, but it will then not look good with the above cases.
highlight IndentBlanklineChar gui=None
highlight IndentBlanklineSpaceChar gui=None
highlight IndentBlanklineSpaceCharBlankline gui=None
highlight IndentBlanklineContextChar gui=None

Thanks for the workarounds; I appreciate it. If there's not a way for you to only allow mixing certain attributes commonly used for theming, does it make sense to request this from neovim?

does it make sense to request this from neovim?

Doesn't hurt to ask.
Probably the best way to support this would be to add negative gui options. Like noitalic
So a highlight group can set this, all gui options get combined, but noitalic overwrites if the underlying highlight group had italic


Find a simpler way to send QUBO to server

https://github.com/SupertechLabs/applications-superstaq/blob/374db4248901c6ebef3773a388d00bf833bb869b/applications_superstaq/qubo.py#L19

This is the simplest way to send QUBOs to the server. The keys of the QUBOs are tuples which are not valid JSON  keys hence why convert_qubo_to_model converts them to strings before sending them to the server. Serilaizing using pickle or the other ways @richrines1 has suggested are not simple. They are more likely to break this functionality in the future.

@vtomole what's wrong with list(qubo.items())?

what's wrong with list(qubo.items())?

Maybe.
@dhirpalshah Please try this out.


Error: run as uid (20000) is not supported on Windows

I am trying to run otel-collector-agent to collect logs for windows in our eks cluster.
I have logsEngine set to "otel"
The pod is failing to come up with this error:
Error: run as uid (20000) is not supported on Windows
Am I missing something?  Any input is appreciated.

Make sure to set isWindows=true in values.yaml
https://github.com/signalfx/splunk-otel-collector-chart/blob/b2dfe48d82b760369c4d3a3f98acb413473cbff4/helm-charts/splunk-otel-collector/values.yaml#L810

@jvoravong That is set to true I'm afraid
isWindows: true

Ah, looks like i have to use use runAsUserName in windows to get passed that error,


[Bug] All three dashboards have no data on the logs and connection tabs when accessing through nginx reverse proxy.

Verify Steps

[X] Tracker 我已经在 Issue Tracker 中找过我要提出的问题
[X] Branch 我知道 OpenClash 的 Dev 分支切换开关位于插件设置-版本更新中，或者我会手动下载并安装 Dev 分支的 OpenClash
[X] Latest 我已经使用最新 Dev 版本测试过，问题依旧存在
[X] Relevant 我知道 OpenClash 与 内核(Core)、控制面板(Dashboard)、在线订阅转换(Subconverter)等项目之间无直接关系，仅相互调用
[X] Definite 这确实是 OpenClash 出现的问题
[X] Contributors 我有能力协助 OpenClash 开发并解决此问题
[ ] Meaningless 我提交的是无意义的催促更新或修复请求

OpenClash Version
v0.46.050-beta
Bug on Environment
Immortalwrt
OpenWrt Version
ImmortalWrt 23.05.1 r27304
Bug on Platform
Linux-amd64(x86-64)
Describe the Bug
All three dashboards have no data on the logs and connection tabs when accessing through nginx reverse proxy.
I have configured the Dashboard Control and all the other tabs of all three dashboards work just fine, only the log and connection tabs have no data.
But accessing through ip address has no issues.
Is there any header parameter I need to set for my nginx reverse proxy?
My nginx conf:
  set $forward_scheme http;
  set $server         "openclash.ip";
  set $port           9090;

  listen 80;
listen [::]:80;

listen 443 ssl http2;
listen [::]:443 ssl http2;

  server_name clash.dashboard;
...

To Reproduce
Access openclash dashboard through nginx reverse proxy.
OpenClash Log
No relevant logs

OpenClash Config
No response
Expected Behavior
Have data on connections and logs when accessing all dashboard through nginx reverse proxy.
Additional Context
No response

https://github.com/MetaCubeX/mihomo/blob/9de9f1ef518206e43e9f8300cfc724d8e128d049/docs/config.yaml#L58

https://github.com/MetaCubeX/mihomo/blob/9de9f1ef518206e43e9f8300cfc724d8e128d049/docs/config.yaml#L58

I did that. But the connection tab is still not working. I got Browser can’t establish a connection to the server at wss://my.domain.tld/connections?token=mytoken. in the browser console log. Do I have to add something to the nginx conf for upgrading HTTP to WebSocket?

https://github.com/vernesong/OpenClash/pull/2386#issuecomment-1491389058


pass more flags to run_cloud.sh, for easier testing

Found this very handy when testing out https://github.com/uProxy/uproxy/pull/2868 and the latest release.
Rather than cloning the repo and switching to the development branch, etc., it let me run:
curl https://raw.githubusercontent.com/uProxy/uproxy/trevj-curl-installer-additions/install-cloud.sh | sh -s -- -u -s uproxy/sshd-alpine -z uproxy/firefox-stable -b trevj-alpine-ssh

I think this will be useful in future, too.

This change is

👍 Excellent, yes this will be helpful!


Support Server Sent Events (SSE) ?

Hi there,
I seems that there is no native way to do Server-Sent Events in Elm. I was wondering if, adding this feature, is in the roadmap ?
I have found an implementation here : https://github.com/gpremer/elm-sse-ports

I'd also love to have this feature built in.

That would be a great subscription to have. Wouldn't be hard to use ports for with EventSource but would be a cool addition to this package definitely.

I’m happy using ports for this.


[Bundle] Parallel bundle perf improvements

Description
Improving the performance of parallel bundles given most recent performance tests.
From what it was found, changes made are:

Replacing call to Task.WaitAll to Task.WhenAll.
Removing the explicit GC call as it was causing performance issues, and it's no more required.

Related issues
Addresses [issue #].
Testing
Describe how this change was tested.
FHIR Team Checklist

[x] CI is green before merge
Review squash-merge requirements

Semver Change (docs)
Patch|Skip|Feature|Breaking (reason)

                // The following Task.WaitAll should wait for all requests to finish.

Please replace by WhenAll

Refers to: src/Microsoft.Health.Fhir.Shared.Api/Features/Resources/Bundle/BundleHandlerParallelOperations.cs:143 in fdcf5bf. [](commit_id = fdcf5bf0480e9a662f0a491b1116bcae770a2f2d, deletion_comment = False)


When using the vscode twinny plugin with remote-ssh for remote development, code suggestions are not working.

Describe the bug
When using the vscode twinny plugin with remote-ssh for remote development, code suggestions are not working. However, code suggestions work fine when working locally. chat
To Reproduce
Steps to reproduce the behavior:
Open Visual Studio Code.
Install the Twinny plugin.
Connect to a remote server using the remote-ssh extension.
Open a file for editing on the remote server.
Start typing and try to get code suggestions.
Notice that code suggestions are not appearing.
Expected behavior
Code suggestions should appear while typing, regardless of whether working locally or remotely.
Desktop
OS: [e.g. Windows 11]
Vscode:1.88.0
twinny: v3.11.3
Additional context
Apart from the issue with code suggestions in the Twinny plugin and remote-ssh, all other functionalities in Visual Studio Code seem to be working fine. I can still chat and interact with the editor without any problems. The issue seems to be isolated to the code suggestion feature when working with remote development.

Looks like the plugin is attempting to contact a server that's local to your editor for completion.
You can:

run ollama locally
setup port forwarding (so your remote ollama is accessible locally)
point twinny to your remote ollama (possibly insecure)

Hey, thanks for reporting.
Please help me understand correctly. The chat works but not the auto-complete?  Or nothing at all?

Stale

I'm also running into this issue.
This is the specific error I'm seeing:
  ERR cannot open vscode-remote://ssh-remote%2Bdev-vm/embeded.embeddedhtml. Detail: Unable to read file 'vscode-remote://ssh-remote+dev-vm/embeded.embeddedhtml' (Error: Unable to resolve nonexistent file 'vscode-remote://ssh-remote+dev-vm/embeded.embeddedhtml'): CodeExpectedError: cannot open vscode-remote://ssh-remote%2Bdev-vm/embeded.embeddedhtml. Detail: Unable to read file 'vscode-remote://ssh-remote+dev-vm/embeded.embeddedhtml' (Error: Unable to resolve nonexistent file 'vscode-remote://ssh-remote+dev-vm/embeded.embeddedhtml')
    at n.$tryOpenDocument (vscode-file://vscode-app/usr/share/code/resources/app/out/vs/workbench/workbench.desktop.main.js:2195:62578)

Which I can't make heads or tales of. My "workaround" is to use Llama Coder for completions and Twinny for chat.

can confirm if a provide is using remote ssh i get
  ERR cannot open file:///embeded.embeddedhtml. Detail: Unable to read file '/embeded.embeddedhtml' (Error: Unable to resolve nonexistent file '/embeded.embeddedhtml'): CodeExpectedError: cannot open file:///embeded.embeddedhtml. Detail: Unable to read file '/embeded.embeddedhtml' (Error: Unable to resolve nonexistent file '/embeded.embeddedhtml')
    at OFe.$tryOpenDocument (vscode-file://vscode-app/Applications/Visual%20Studio%20Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:615:8481)


Show different color on html tag name and tag attributes

Now in html file, the colors of tag name and tag attributes are the same, I suggest to make them different so that the code looks more beautiful.

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


RHBPMS-4220: Added RoleRegistry and registered principals that come from Keycloak remote logins as roles.

Fixed issue that causing the security principals not being added as roles in the current user instance, so remote login (eg: keycloak) were failing due to authorizations failed, as users had no roles. In order to do this I've backported from master branch the RoleRegistry necessary stuff.
See https://issues.jboss.org/browse/RHBPMS-4220
Please @dgutierr, as commented with you this fix, can you please verify? Thanks!

jenkins rebuild this ( got crazy with some signature check failed on an artifact... :)

Hi @Roger,
The implementation looks great!
Aside from that, I think tests should be added for the JAASAuthenticacion and GroupAdapterAuthorizationSource classes in order to guarantee that the retrieval of groups & roles has proper coverage.

Hi @dgutierr , absolutely agree :) Added new commit with the tests for it! Added new commit with test as well for https://github.com/uberfire/uberfire-extensions/pull/301. Thanks!

@romartin I think the build is failing because in 0.9.x the new java 7+ nio api is not available, as it is on master. That explains why you get these errors in the log:
08:53:47 [ERROR] /home/jenkins/workspace/uberfire-pullrequests-0.9.x-and-older/uberfire-backend/uberfire-backend-server/src/main/java/org/uberfire/backend/server/security/RoleLoader.java:76: Undefined reference: java.io.File java.nio.file.Path.toFile()
08:53:47 [ERROR] /home/jenkins/workspace/uberfire-pullrequests-0.9.x-and-older/uberfire-backend/uberfire-backend-server/src/main/java/org/uberfire/backend/server/security/WebAppSettings.java:32: Undefined reference: java.nio.file.FileSystem java.nio.file.FileSystems.getDefault()
08:53:47 [ERROR] /home/jenkins/workspace/uberfire-pullrequests-0.9.x-and-older/uberfire-backend/uberfire-backend-server/src/main/java/org/uberfire/backend/server/security/WebAppSettings.java:32: Undefined reference: String java.nio.file.FileSystem.getSeparator()
08:53:48 [ERROR] /home/jenkins/workspace/uberfire-pullrequests-0.9.x-and-older/uberfire-backend/uberfire-backend-server/src/main/java/org/uberfire/backend/server/security/WebAppSettings.java:89: Undefined reference: java.nio.file.Path java.nio.file.Paths.get(String, String[])
08:53:48 [INFO]
The solution is to use the legacy UF NIO api backport.

@dgutierr, @romartin you should be able to reproduce the same fail when using the full profile (even when running with JDK 8). So something like mvn clean install -Dfull (which will add the animal-sniffer execution into the build)

ahh ok thanks for the update @psiroky !! I didnt realized about this, trying to fix now. Thanks!

Hi @dgutierr , @psiroky - thanks for the updates!
Fixed the build issue and added a  commit again. Checked other dependant PRs as well after this fix. Hope jenkins will not complain now :)

yeahh passed :)

+1

This appears to have broken tests in kie-wb-common and drools-wb:-
https://kie-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/drools-wb-pullrequests-6.x/101/testReport/junit/org.drools.workbench.jcr2vfsmigration/MortgagesRepoImportTest/testImportMortgagesRepo/
https://kie-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/kie-wb-common-pullrequests-6.x/165/testReport/org.kie.workbench.common.screens.explorer.backend.server/ExplorerServiceImplTest/explorerServiceShouldBeADependentBean/

@manstis @romartin @dgutierr I will revert both commits involved.

All three commits that is.


Fix/miduweb 849

@MrBuggy The WithFacet.js has been extensively refactored. I will take care of it and implement it. Please don't solve a merge conflict! Just leave it as it is. Thanks. cc/ @Weedshaker @Nilsilino


show collection didn't work for my on 3.4.0

Show collection didn't work on 3.4.0 Docker Image.
i can see the information throw db.collectionname.find() command, but when i execute the command show collections it doesn't show anything.

Do you have any data in your instance?  In MongoDB, collections and databases only exist when they have data inside them:
$ docker pull mongo:3.4
3.4: Pulling from library/mongo

Digest: sha256:d929ffc5d0871712198be82c475397e5e9c71f3d2a9fa4b304f623ad09ad204b
Status: Image is up to date for mongo:3.4
$ docker run --name some-mongo -d mongo:3.4
bd6b629894607e76bdf0f06890966e07c64ebef42dd2a931a170e572bae2b64b
$ docker run -it --link some-mongo:mongo --rm mongo:3.4 mongo mongo/test
MongoDB shell version v3.4.0
connecting to: mongodb://mongo/test
MongoDB server version: 3.4.0
Welcome to the MongoDB shell.
For interactive help, type "help".
For more comprehensive documentation, see
	http://docs.mongodb.org/
Questions? Try the support group
	http://groups.google.com/group/mongodb-user
Server has startup warnings:
2016-12-10T18:20:14.026+0000 I STORAGE  [initandlisten]
2016-12-10T18:20:14.374+0000 I CONTROL  [initandlisten]
2016-12-10T18:20:14.374+0000 I CONTROL  [initandlisten] ** WARNING: Access control is not enabled for the database.
2016-12-10T18:20:14.375+0000 I CONTROL  [initandlisten] **          Read and write access to data and configuration is unrestricted.
2016-12-10T18:20:14.375+0000 I CONTROL  [initandlisten]
> db.testcol.save({"a":"b"});
WriteResult({ "nInserted" : 1 })
> show collections
testcol


Electron.NET doesn't work with dotnet 8.0 on Mac M1 ?

Version:
(everything newest)
NuGet[ ElectronNET.API ] 23.6.1
NuGet[ ElectronNET.CLI] 23.6.1
node.js v20.11.0
Dotnet 8.0.101
Mac OS: 14.2.1 (23C71)  - M1 processor.

Target:
Mac OS, but later will try for Windows too.

I had perfectly working Blazor app contained in electron, but now I can't figure out how to run it - after updating to dotnet 8.0. I have made a new projects from scratch and have exactly the same behaviour:
After running 'electronize start' - the app is somehow starting as its icon is showing in the dock, but there is now Electron window visible - whatever I do or click. Last line at console after run:
...
ElectronHostHook handling started...
Invoke electron - in dir: /Users/pedropuchalski/Downloads/ElectroBlazorApp/obj/Host/node_modules/.bin
./electron "../../main.js"
2024-02-09 15:01:39.300 Electron[8712:1561502] WARNING: Secure coding is not enabled for restorable state! Enable secure coding by implementing NSApplicationDelegate.applicationSupportsSecureRestorableState: and returning YES.
Electron Socket IO Port: 8000
Electron Socket started on port 8000 at ::1
ASP.NET Core Port: 8001
So no errors... anything. Example code I use: https://github.com/pedropuchalski/ElectroBlazorApp
fulll StartingLog.txt

I had the same issue and it was due to the app not detecting arm architecture. Try running electronize start /target custom "osx-arm64;mac" (assuming you have the arm SDK installed). You can see my issue in #820.

Same issue on macOS with M1 Pro, and the command given by sajmonr doesn't work.


Fix: No notification on web if window is not active

Details

Desktop shows the notification but web does not when we focus on the tab then open another app to blur the window. That is caused by different implementations of Visibility.isVisible() function across platforms. This PR fixes the issue.
Fixed Issues

$ https://github.com/Expensify/App/issues/17242
PROPOSAL: https://github.com/Expensify/App/issues/17242#issuecomment-1507879211
Tests

Login with account A on Web
Go to any chat with account B
Open any other app so that the browser window gets defocused
Login with account B on another device
Send message from that device to account A
Verify that the account A's device receives new message notification

[x] Verify that no errors appear in the JS console

Offline tests

Same above
QA Steps

Login with account A on Web
Go to any chat with account B
Open any other app so that the browser window gets defocused
Login with account B on another device
Send message from that device to account A
Verify that the account A's device receives new message notification

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Screencast from 17-04-2023 15:59:02.webm

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

@luacmartins @Santhosh-Sellavel One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

Note: This only happens on Web so I only included Web screenshot. Please let me know if there were ways to enable notification on mWeb. What do you think if I didn't included other platforms' screenshots?

@Santhosh-Sellavel all yours!

@Santhosh-Sellavel Just a friendly reminder in case you missed this.

@Santhosh-Sellavel Just a friendly reminder in case you missed this.

@tienifr
Sorry we discussed this internally and forgot to leave a comment here, everything perfectly works fine for me on the staging itself and I'm not sure what we are trying to do here.
cc: @luacmartins

@luacmartins I still can produce it, can you describe your context in detail?
Screencast from 19-04-2023 15:33:18.webm

@Santhosh-Sellavel I didn't get notified when the window is unfocused/not active. Reproduced on Staging.
Screencast from 19-04-2023 15:37:53.webm

Alright, I got to reproduce the issue now, but the issue is not the same as it was described slightly differently.
We receive notifications even when the window is not active. We don't show when the same chat is active for which the notifications are received receiving the notification is

@tienifr Doesn't it affect mWeb also?

We receive notifications even when the window is not active. We don't show when the same chat is active

Absolutely! We don't show when that chat is opened and the window is active, to be more concise.

Supposedly you can enable notifications on Chrome mWeb, but I couldn't get it to work 😕

@luacmartins Thanks! I already tried but it yielded no result. In Chrome mobile (Android & iOS), we can't manually add allowed sites for notifications. Instead, the site must ask for permission first. While it's different on Computer where we can manually add or remove any site.
Reference: https://support.google.com/chrome/answer/3220216?hl=en&co=GENIE.Platform%3DAndroid&oco=1

Alright if the notification is itself not working on mWeb it's a different problem, can you create a new tracking issue for that @luacmartins? And let's move forward here

Are we asking for permission to show notifications on mWeb?

Pretty sure mWeb is completely unsupported for now

@Julesssss is mWeb notification covered anywhere in the notifications project? do we already have an issue for it?

@cead22 no, it's not something we care about

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/luacmartins in version: 1.3.2-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

FYI: this PR introduced this regression.

@cristipaval I'm confused how this cause a regression

@tienifr @Santhosh-Sellavel @luacmartins  I am reverting this PR since automatic authentication with the magic link is a very important feature.

@cristipaval I'm confused about how this causes a regression, can you explain please with a bit more context thanks!

@Santhosh-Sellavel Please see follow here this link to the Slack conversation with more context.


Support for AnchorSheetBehavior

I am planning to integrate a custom bottom sheet that supports an anchor point, after playing with the default BottomSheetBehavior, i realized that is quite limited, and doesn't support an anchor point, this is a crucial feature since i was design this library, i still didn't achieve a full google maps experience, an anchor point would allow us to expand full screen and a middle anchor.
So, here is some thoughts about it, if there's anyone interested in discuss it.

Uses the custom implementation from miguelhincapie/CustomBottomSheetBehavior
No breaking changes on the existing BottomSheetBehavior.
Adds AnchorSheetBehavior, BackdropBottomSheet and AppbarLayout components.
AnchorSheetBehavior has STATE_ANCHOR_POINT state and a anchorPoint prop to define a fixed anchor position.
AppBarLayout component that handles a scroll behavior, and allow us to connect a ToolbarAndroid from react-native to sets the title when the anchor sheet is expanded.
BackdropBottomSheet renders a parallax View behind the anchor sheet.
FloatingActionButton supports a custom behavior that hides when its near to the toolbar.

I have been working on a concept on anchorSheet branch, and i am very excited with it, there's still some stuff to be done, i'm planning to release it soon, also looking for feedback from anyone who are interested.

Ok, that's is how it looks like.

beta published 🎉


Mobile: Requirements

JIRA Link: https://patternfly.atlassian.net/browse/RHUAP-265

Closing - mobile will be after first release.