added helpText for minHeight field of image slider elements

1. Why is this change necessary?
You can set the minHeight of the image slider elements but if you don't add the unit, the minHeight will not work. The helpText gives an example of a valid input. If a user makes an input with spaces it also will not work. You can prevent that by trimming the spaces out of the input.
2. What does this change do, exactly?
Adding a helpText to the image slider and the gallery slider cms-element for the minHeight input and trimming spaces of  the input.
3. Describe each step to reproduce the issue or behaviour.
Create a new Layout in the shopping experience drag and drop the cms image slider or the gallery slider and set a min height of 200 and don't add a unit. The minHeight will not work because of the missing unit. Repeat this and now make an input with "200 px" this will also not work because of the blank space.
4. Please link to the relevant issues (if any).
See this issue
5. Checklist

[ ] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-23929
Please use this issue to track the state of your pull request.

Thanks again for your contribution, your PR is merged now! 🎉 💙


enhance comments, javadoc and logging message for ExecutionChannelHandler

What is the purpose of the change
a follow up for pull request https://github.com/apache/incubator-dubbo/pull/1449
Brief changelog
enhance comments, javadoc and logging message for ExecutionChannelHandler
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[x] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #1722 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #1722      +/-   ##
============================================
- Coverage     35.55%   35.52%   -0.03%
+ Complexity     3785     3783       -2
============================================
  Files           628      628
  Lines         30980    30980
  Branches       5447     5447
============================================
- Hits          11015    11006       -9
- Misses        18071    18079       +8
- Partials       1894     1895       +1

Impacted Files
Coverage Δ
Complexity Δ

.../dispatcher/execution/ExecutionChannelHandler.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ubbo/rpc/protocol/dubbo/ChannelWrappedInvoker.java
41.66% <0%> (-8.34%)
3% <0%> (ø)

...om/alibaba/dubbo/rpc/filter/ActiveLimitFilter.java
83.33% <0%> (-5.56%)
6% <0%> (-1%)

.../java/com/alibaba/dubbo/config/ProtocolConfig.java
34.02% <0%> (-3.48%)
39% <0%> (-1%)

.../java/com/alibaba/dubbo/config/AbstractConfig.java
37.67% <0%> (-0.71%)
60% <0%> (ø)

...java/com/alibaba/dubbo/config/ReferenceConfig.java
47.97% <0%> (+0.73%)
36% <0%> (ø)
:arrow_down:

.../com/alibaba/dubbo/monitor/dubbo/DubboMonitor.java
76.63% <0%> (+1.86%)
7% <0%> (ø)
:arrow_down:

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 080e8ef...a62d327. Read the comment docs.


Installation of Notebook dependencies failed with error:

Issue Type: Bug
Reinstalling notebook dependencies give the following error:
Error: EPRTM: operation not permitted, unlink '****\vcruntime140.dll'
Azure Data Studio version: azuredatastudio 1.3.6-insider (efd809971ff50befee6535315f19c495af46c71a, 2018-12-04T00:09:27.621Z)
OS version: Windows_NT x64 10.0.17763

Same as issue #3120


Update conf.py

Set linenos style to "table" so that when you select code to copy/paste, it doesn't include the line numbers.

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Separates the code from the line numbers in our readthedocs documentation.
Why should this Pull Request be merged?
We've been doing usability studies re: the python API and every user has ended up here: https://niflexlogger-automation.readthedocs.io/en/latest/getting_started.html#examples
When they try to copy/paste the code, it includes line numbers. When they go to delete them, the indentation gets messed up, and python cares a lot about indendtation.
What testing has been done?
Ran tox -e docs on my machine and it looks good now.
Before:

After:

Apparently this is a relatively recent regression (reported June 19th): https://github.com/readthedocs/sphinx_rtd_theme/issues/1168
And there has been a fix submitted to the sphinx-rtd-theme repo, but there's not a "stable" release with the fix in it yet. 1.0.0rc1 exists and seems to work, though: https://pypi.org/project/sphinx-rtd-theme/#history
Also, it seems this html_codeblock_linenos_style flag has been deprecated as of "Sphinx 4.0".
Bumping the docs/requirements.txt of sphinx-rtd-theme to require version ">=1.0.0rc1" also seems to fix this issue.
I was worried this would require a script somewhere to explicitly call "pip install --pre" to access prerelease versions, but I tested and it seems to just work. So that feels like the right way to fix this. Any objections?


Milestone 2 feedback

The midpoint idea is really good. When deciding on a midpoint, have the algorithm lean toward places with attractions such as downtowns; so that you don't meet in the middle of nowhere.
We also like the skating idea, especially for finding the best route for safe skating.
-Team 13

Reviewed last 2 storyboards (7 & 8)
I liked the midpoint idea - I think I could see myself using that feature, not necessarily the app. As for the storyboard for hotspots, the issue that arises is security. How would I be validating that a person I meet is verified? Another issue that comes to mind is validity. Maybe having a system of upvoting can help ensure that one person's hotspot is truly the locale's real hot spot. Looks good!

Review #5
This is a pretty cool application but it seems like you need a good user base location all the restrooms
Review #6
This is a really neat map idea so no students will get lost however schools should have these already if their students actually tried and looked it up.

Team 12
Storyboard 3 Feedback:
I like the idea of having routes for biking/skating, since it allows a safer and more optimal navigation for people who ride these instead of walking or driving.  I am unsure if these paths are generated using Google maps, or user generated.  Also, I like the idea of notifying users of terrain based items, such as hills; maybe users can create their own "points of significance" so other users can look at it.
Storyboard 4 Feedback:
I like the idea of having a calculator purely for gas, and it would be very useful for people who travel a lot by car or for people traveling for Spring Break.  I think the app should take local gas prices and choose the most optimal price along their path.  Users should also be able to leave comments on gas stations so other users can see if it is it worth it to go to a certain gas location or wait for another one.

Team #12
Storyboard 1:
I like the idea that students can easily find parking spot by using the app. I think it will be cool if people can lock the parking spot for 1 minute so that two users won't aim at one same parking spot.
Storyboard 2:
I really like this idea since I'm an international student and sometimes it is difficult to figure out the right time abroad. But this seems can be done with a simple clock..

Team #11 (Kool Kats) - Trung:
Storyboard 1:

When a person leaves a parking spot, they will very likely forget to report on the app that they left. Thus, the app needs to somehow detect this automatically or it would contain incorrect information.
In some parking structure with no internet, how can you do this?
Storyboard 2:
This could be done like built-in clock app on your phone. It does not have to relate to maps.
Storyboard 3:
Good idea overall. However, I would suggest a feature that lets user report routes that they usually skate.
Storyboard 4:
I used this app before.

Team #11
Storyboard 1:
The storyboard is very easy to understand. However, the storyboard could include a bit more to give an understanding of the potential user's environment and background. Otherwise, the idea seems good.
Storyboard 2:
The idea could be a bit more complex. We feel like a problem such as figuring out the time zone could already be easily solved with the given clock app/function on our phones. Storyboard was again concise and easy to understand.
Storyboard 3:
Storyboard is a bit hard to read. Adding borders of some sort may make it more readable. This idea is very interesting. We are wondering, however, whether the app chooses a route for us or recomends several possible routes.
Storyboard 4:
Same as storyboard 3, it would be great if it was sectioned off to make it more readable. The idea is pretty good, however, one thing to consider would be to add another feature to make it multi-faceted. This is with the consideration that MapQuest has something similar where the user can input the MPG of their car to figure out gas price.

Team 14 - C5 - JSOT
Augmented Reality Information - Allen
Good concept, but unclear about how you're getting data (real time, trend based, crowd sourced, etc.)
TimeZone Calculator - Crystal Jiao
It's not any different from a clock app. Don't really see map integration. Other user would have to keep location services on or keep updating where they're at.
Biking/Skate Route - Daniel Sung
We're not sure how to feel about this.
Like Google Maps for skateboards. Would also need to be heavily user generated/crowd sourced because road conditions change frequently.
Gas Cost Calculator - Daniel Sung
Good concept, but there's a lot of variables to take into account. Gas prices between cities/states, traffic, specific models of cars, and others.

Team JSOT (C5) Feedback:
Storyboard 5:
This addresses a pretty real problem--I'm sure everyone has had trouble finding bathrooms before. I think there could be more functionality (e.g. rating bathrooms for cleanliness, facilities, etc.) to make the app more fleshed out. I think the storyboard clearly identifies a problem and shows how your app would solve the problem.
Storyboard 6:
This definitely identifies a problem, but it doesn't seem like a very persistent one. Once someone learns where their classes are and the fastest route between them, there would be no need to use the app again. Furthermore, if you wanted to provide the truly fastest route between classrooms, the app would have to constantly keep track of foot traffic. I also feel that Google Maps already solves the issue of finding the shortest path, so you would have to differentiate this app somehow.
Storyboard 7:
I like the idea of finding popular hotspots. Since Google can already provide tourists with popular tourist locations, I think that you could make this app unique by using it to find "underground" or lesser-known hotspots. It could be a "best-of" location finder that is based on crowdsourced data.
Storyboard 8:
This storyboard identifies a problem for people who live far away from each other, but it does not really apply to friends who live close by. I think you could make this app more applicable to frequent meetups by having it suggest different meeting locations every time that are still within equal distance of all attendees.


[New feature] llama.cpp support

We are currently running out of bandwidth. Contributors to help integrate Medusa into llama.cpp would be greatly appreciated :)

What type of skills would be needed? Any specific grunt work in mind?

It looks like a lot of the groundwork is being laid out here:
https://github.com/ggerganov/llama.cpp/pull/3228

It looks like a lot of the groundwork is being laid out here with the parallel decoding implementation: ggerganov/llama.cpp#3228

Yeah, that's also what I thought. The tree attention implementation in llama.cpp should be very helpful for integrating Medusa. And I guess the main challenge is the familiarity with their codebase.

We are almost done in supporting medusa in llama.cpp. now working on attention mask part and few other things.

I would like to help with this. Is there any place where I can contact the group behind this and ask questions?

I would like to help with finalizing the support for this. Is there any place where I can contact the group behind this project and ask questions?

Hi @kalomaze , that would be great, just sent a friend request on Discord :)

I would like to help with finalizing the support for this. Is there any place where I can contact the group behind this project and ask questions?

Hi @kalomaze , that would be great, just sent a friend request on Discord :) (I assumed your user name is kalomaze right?)

Sorry, what was your tag? I'm coming back to this now, I figured GitHub would have sent a notification.


Test Types: Return support for disabling test types via the active flag

Return the ability to remove a given test type from being used, but without deleting the test type. This change has the following impacts for test types with a False active status:

Import forms will not list inactive test types
Creating/editing a test will not allow for an inactive test type to be selected

[sc-6869]

I'm not 100% sure, this will work in multi-Django deployment (k8s, number of pods >1). signal will probably trigger reload of PARSERS only in the container in which Test_type has been disabled.

That is a good point. Maybe it really would be better to totally move away from the "in memory" list of parsers. I'll see what I can do


[SciPy]: Test compiled packages at CI

We need to make a release covering PR: https://github.com/lfortran/lfortran/pull/2741
cc @certik @Shaikh-Ubaid

Sure, I just made a new release: https://github.com/lfortran/lfortran/releases/tag/v0.29.0

I think this has to be now updated in conda and then tested here in this respository.


fix: tabs indicator

Jira: LF-7780
Indicator should now be visible

Now the shadow shouldn't be cut off - note, this seemed to be a bit of clash between the boxShadow and absolute positioning

Looks good!


Circle Annotation without Image captions

With the 'extended' annotation style the caption shows up but this is not the case with the 'circle' style annotation.

Alt text is also missing in this scenario.

Fixed in v0.36.0


chore: update to Expo SDK 37.0.0

Since the release of  latest Expo SDK, this PR update the Expo SDK to 37.x.x and the related dependencies.

I have upgraded the RN version to 37.0.1 as per your instructions @byCedric. Thank you so much for your time and reviewing these PRs. :)

Anytime Aman ❤ Thanks for making and maintaining this!


[Blog] write a blog about the launch of using "latest" in distribution urls

Describe the blog post your would like to write
We support using "latest" keyword in the distribution url.
https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/1.3.0/latest/linux/arm64/builds/opensearch/manifest.yml
As a contrast, previously we have to provide a Jenkins build number (e.g 1210)
https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/1.3.0/1210/linux/arm64/builds/opensearch/manifest.yml
See this https://github.com/opensearch-project/opensearch-build/issues/1492 for more details
What is the title of the blog post?
"latest" keyword is now supported in distribution urls
Who are the authors?
Who will be listed as the authors of this blog post
https://github.com/tianleh
What is the proposed posting date?
3/18/2022

Hey @tianleh - I like it.
I think you'll need to setup the reason why this is important in the blog post. Average user of OpenSearch is probably not super familiar with distribution URLs.
Let me know how I can support!

@tianleh Any word on this post?

@tianleh Any word on this post?

@stockholmux Thanks for following up on this. Still trying to figure out the process.


Big error after calibration

Required Info

Camera Model
D435i

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10/MacOS

Platform
PC

SDK Version
{ legacy / 2.50.0}

Language
{python}

RS was calibrated with on Chip and dynamic tool (with RGB). I used my phone Redmi Note 8T.
Result in chip was 0.00 which is the best result one can obtain thanks to pattern provided here (https://dev.intelrealsense.com/docs/self-calibration-for-depth-cameras). During the dynamic part, I didn't change distance between pattern and camera plane.
Also, phone was kept straight and whole scene was covered (15 pictures were spread).
I tried it multiple times and each of the results is horrible. Below I put comparison of d435 and d435i (1000MM).
.
As far as I know, RMS should be close to 9 MM (according to Intel).

Hi @Obrepal   Could you confirm a couple of points about the target image that you were using, please?

If you were using a target on an Android phone with the Dynamic Calibration tool, did you use the Android target app on the Google Play Store at the link below that the user guide for the tool recommends?

https://play.google.com/store/apps/details?id=com.intel.realsenseviewer17613&gl=GB

The On-Chip calibration tool is used to improve depth image quality.  If you are aiming to improve depth accuracy then the Tare calibration tool should be used instead.  This has a different printable target image to the On-Chip calibration tool that can be downloaded at the link below in two different paper sizes.

https://dev.intelrealsense.com/docs/self-calibration-for-depth-cameras#ground-truth-targets-and-printing--mounting-instructions

You could also try using a tool built in to the RealSense Viewer to reset the camera to its factory-new default calibration to see whether this provides the desired image quality / accuracy.  Instructions for doing so can be found at https://github.com/IntelRealSense/librealsense/issues/10182#issuecomment-1019854487

Yes, I downloaded app via google store (Dynamic Target tool version 1.0)
Ok, should I print it or it can be displayed on monitor?
Also how accurate it has to be to improve performance (I do not have laser just tape measure)?

The calibration bar image provided by the Android app should be displayed on the screen of the Android device and not printed.  The camera should be pointed at the image on the Android screen.
Aside from using the Dynamic Calibration tool in a location with good lighting, there is not much else that can be done to improve its performance further.
The Tare calibration tool offers more flexibility, as it has a Get button to custom-adjust the depth calibration for the distance of the camera from the observed surface before clicking the 'Calibrate' button to begin calibration.

Yeah, in Dynamic I used screen of phone.
Ok, I will try it and let you know.
Thanks :)
śr., 15 cze 2022, 13:11 użytkownik MartyG-RealSense <
@.***> napisał:

The calibration bar image provided by the Android app should be displayed
on the screen of the Android device and not printed. The camera should be
pointed at the image on the Android screen.
Aside from using the Dynamic Calibration tool in a location with good
lighting, there is not much else that can be done to improve its
performance further.
The Tare calibration tool offers more flexibility, as it has a Get
button to custom-adjust the depth calibration for the distance of the
camera from the observed surface before clicking the 'Calibrate' button to
begin calibration.
[image: image]
https://user-images.githubusercontent.com/41145062/173812872-2144da61-de9c-4e8b-9aea-dfa8eb9cdd6c.png
—
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/10594#issuecomment-1156337427,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALRKFXBUZ2K3QSEPKFGLI7TVPG27PANCNFSM5YYQSCQA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @Obrepal  Do you require further assistance with this case, please?  Thanks!

Hi @MartyG-RealSense I think it was improved results are not as good as expected but it will do it.

Comparing the estimated RMSE curve for D435 on Intel's official graph from their camera tuning guide to your own graph, they seem to be similar at the 4000 mm distance.  Intel's graph is clearly marked as 60 mm RMSE at 4000, whilst your own curve looks to be around 60 at that distance if a ruler is put over the chart at the halfway poit between 0 and 100.

I am using 640x480 insteas of 848x480 thus the difference
wt., 21 cze 2022, 14:17 użytkownik MartyG-RealSense <
@.***> napisał:

Comparing the estimated RMSE curve for D435 on Intel's official graph from
their camera tuning guide to your own graph, they seem to be similar at the
4000 mm distance. Intel's graph is clearly marked as 60 mm RMSE at 4000,
whilst your own curve looks to be around 60 at that distance if a ruler is
put over the chart at the halfway poit between 0 and 100.
[image: image]
https://github.com/assets/41145062/a4c94546-b925-414f-908c-38728c90112c
—
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/10594#issuecomment-1161670677,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALRKFXDYAHW7J47VDF7BH73VQGXFHANCNFSM5YYQSCQA
.
You are receiving this because you were mentioned.Message ID:
@.***>

In general, as resolution is increased the depth accuracy increases, and accuracy decreases as resolution is reduced.  The optimal depth accuracy resolution for the D435i camera model is 848x480.  I do not believe that there will be a huge reduction in depth accuracy at 640x480 though.
RMS error may reduce if the camera's IR emitter and its dot pattern projection are disabled.  If the scene that the camera is observing is well lit then the camera can use the ambient light in the scene to analyze surfaces for depth detail instead of analyzing the dot pattern.

Hi @Obrepal  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


first public catalog example

First pass at public ZTF source catalog ingestor.

@dmitryduev Can I bump this?


Plugin is not working with IE9

When I try to run the example page on IE9, it crashes with an error:

Thanks!!


Overview component

Added FontAwesome using libman
Added some color variables an retrofitted them to BudyStories
Updated Overview component to look like the website

@all-contributors please add @smabuk for code


⚠ ClamAV has degraded performance

In 04eb034, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 73 ms

Resolved: ClamAV performance has improved in ded9ef5.


Fix tick positioning in checkboxes

What is the context of this PR?
Tick positioning is slightly off on when checkboxes are ticked. This will reposition the tick.
How to review
Compare checked checkbox on this branch with master

 Convert radio and checkbox input sizes from rem to px to avoid decimal sizes


ExePackage elements need DetectCondition attribute

In the example in the Samples wiki, the ExePackage element does not have a DetectCondition attribute. You need a DetectCondition for UninstallCommand to be called. Honestly it is a long standing mistake in the WiX Toolset that the DetectCondition was not required that is planned to be fixed in WiX Toolset v4.
Would be good to add the DetectCondition attribute to the example.

Will do. With previous releases there was no cheap way to detect extensions' presence but the new query API could make that possible now for v3s (doesn't have to load the entire Extension Manager API). Is that something BA functions could enable?

Actually, that still wouldn't work for previous extension versions or on previous releases. If the DetectCondition were always false, it would at least attempt to install the extension each time which is almost a no-op (incurs detection time across all installed versions) to VSIXInstaller of the extension is installed.

This came up today in a discussion. Basically, there's no conceivable way to author a DetectCondition that works for multiple instances. This is why in the WIP I recommend generating install commands at runtime, which currently to my knowledge is not supported by Burn nor am I advocating that it should be.
However, to support uninstall, would it be possible to rely on a BA function? It seems, however, only a single BA function is supported and not something that can be provided by an extension.

I have been trying to incorporate the bootstrapper into a WiX bundle for our internal extensions. It has been rather troublesome so far. We've had issues with the installer failing if we're just doing something like:
<ExePackage DisplayName="VS Ext 1" Id="Vsix1" SourceFile="VSIXBootstrapper.exe" InstallCommand="/q ext1.vsix" Permanent="yes" DetectCondition="WixBundleInstalled">
	<PayloadGroupRef Id="Vsix1" />
	<ExitCode Behavior="success" Value="0" />
	<ExitCode Behavior="success" Value="2" />
	<ExitCode Behavior="success" Value="2003" />
</ExePackage>

I don't exactly know why, but it sometimes this fails if the extension is already installed, so I proceeded to add this:
<ExePackage DisplayName="Remove VS Ext 1" Id="RemoveVsix1" SourceFile="VSIXBootstrapper.exe" InstallCommand="/q /u:MY-GUID" UninstallCommand="/q /u:MY-GUID" DetectCondition="WixBundleInstalled">
	<PayloadGroupRef Id="VSIXBootstrapper" />
	<ExitCode Behavior="success" Value="0" />
	<ExitCode Behavior="success" Value="2" />
	<ExitCode Behavior="success" Value="1002" />
</ExePackage>

That has worked, except that an upgrade now uninstalls the extension, installs the extension, and then runs the uninstaller of the previous version, which uninstalls the extension again.
So I copied the entire standard bootstrapper code and modified it to set a variable if the installer is called as part of an upgrade. I'm using that variable in the detect condition which now looks like this: WixBundleInstalled AND NOT IsRunAsPartOfUpgrade.
Everything seemed to work now, but upgrading the installation takes ages thanks to all the stuff the VSIX installer is doing during every action.
Got another bug report today. The uninstaller failed:
[5318:9FA4][2022-10-04T16:36:01]i301: Applying execute package: RemoveVsix1, action: Uninstall, path: C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\VSIXBootstrapper.exe, arguments: '"C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\VSIXBootstrapper.exe" /q /u:RemoveVsix1-GUID'
[5318:9FA4][2022-10-04T16:36:42]i319: Applied execute package: RemoveVsix1, result: 0x0, restart: None
[5318:9FA4][2022-10-04T16:36:42]i301: Applying execute package: RemoveVsix2, action: Uninstall, path: C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\VSIXBootstrapper.exe, arguments: '"C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\VSIXBootstrapper.exe" /q /u:RemoveVsix2-GUID'
[5318:9FA4][2022-10-04T16:37:25]e000: Error 0xe0434352: Process returned error: 0xe0434352
[5318:9FA4][2022-10-04T16:37:25]e000: Error 0xe0434352: Failed to configure per-user EXE package.
[5318:9FA4][2022-10-04T16:37:25]i319: Applied execute package: RemoveVsix2, result: 0xe0434352, restart: None
[5318:9FA4][2022-10-04T16:37:25]e000: Error 0xe0434352: Failed to execute EXE package.
[5318:9FA4][2022-10-04T16:37:25]i351: Removing cached package: RemoveVsix2, from path: C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\
[5318:9FA4][2022-10-04T16:37:25]i301: Applying rollback package: RemoveVsix1, action: Install, path: C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\VSIXBootstrapper.exe, arguments: '"C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\VSIXBootstrapper.exe" /q /u:RemoveVsix1-GUID'
[5318:9FA4][2022-10-04T16:37:25]e000: Error 0x80070003: Failed to CreateProcess on path: C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\VSIXBootstrapper.exe
[5318:9FA4][2022-10-04T16:37:25]e000: Error 0x80070003: Failed to configure per-user EXE package.
[5318:9FA4][2022-10-04T16:37:25]i319: Applied rollback package: RemoveVsix1, result: 0x80070003, restart: None
[5318:9FA4][2022-10-04T16:37:25]i351: Removing cached package: RemoveVsix1, from path: C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\
[5318:9FA4][2022-10-04T16:37:25]w353: Unable to remove cached package: RemoveVsix1, from path: C:\Users\user\AppData\Local\Package Cache\PACKAGE-CACHE-GUID\, reason: 0x80070003. Continuing...
[5318:9FA4][2022-10-04T16:37:25]i372: Session end, registration key: SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\{BUNDLE-GUID}, resume: ARP, restart: None, disable resume: No
[5318:9FA4][2022-10-04T16:37:25]i371: Updating session, registration key: SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\{BUNDLE-GUID}, resume: ARP, restart initiated: No, disable resume: No
[5318:9FA4][2022-10-04T16:37:25]i399: Apply complete, result: 0xe0434352, restart: None, ba requested restart:  No

Some kind of .NET exception happened in the VSIX Installer, then the cache directory... got cleared? No real idea what's happening there, but we can't uninstall.
Actual guidance on how to use the VSIX Bootstrapper in a WiX bundle would be nice because that wiki page is just wrong and leaves everyone trying to implement such an installer to a lot of guesswork. I'm almost tempted to just write my own VSIX bootstrapper that does all this stuff internally because trying to wrap WiX around this utility is painful to say the least.

The VSIXBootstrapper is just a thin wrapper around VSIXInstaller. If VSIXInstaller failed, as you noted above, please file a bug report at https://developercommunity.visualstudio.com.

I'm aware that I should file a bug report there for that specific issue, but thanks for mentioning it. I'll try to gather more data about that issue and might post the bug report then.

How to write the DetectCondition expression? Colud you provide the example? I'm confused with this attribute.


Create new Empty bot - .NET SDK installation is required which has already been installed

Describe the bug

Version

Browser

[ ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[ ] Windows 10
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

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

Expected behavior

Screenshots

Additional context

Hi @osedakov-amc, can you provide your Composer version? Also, if you are on an x64 machine, can you try installing the x64 version of the .NET Core SDK here?

@osedakov-amc Can you also share the output of the following commands:

dotnet --info
dotnet nuget list source

@lauren-mills @dmvtech Yes, sure. After uninstalling old SDK versions this issue becomes to an error. It appears every time when I try to create a new empty bot

HI @osedakov-amc If you run dotnet build from the bot's project directory, does it throw a more useful error?

Closing due to lack of response. If you are still experiencing the issue and need assistance, please comment or reopen.


Implement registration/authorization

[ ] watch video https://www.youtube.com/watch?v=FSUa8Vd-td0
[ ] implement
[ ] say frontenders what to do

Moving to sprint 2

Now it is implemented simplest authorization without security


Make plugin version optional in version catalogs

In some cases, one can apply a plugin in plugin { ... } block without specifying a version.
However, at the moment it's not possible to leave version unspecified in a version catalog and then apply via alias. Under the hood alias calls version unconditionally and then fails on null or empty value.
Use case for not specifying plugin versions in version catalog:

a plugin may be applied from an included build (incl. buildSrc)
plugin version may be pinned via settings, a settings plugin or an init script via pluginManagement
a plugin may be discovered in build script classpath
other situations and mechanisms unknown to me

Even if version is not specified, version catalog can be used as a bill of materials or a reference list. Besides, generated DSL accessors work as a typo prevention mechanism.

Context

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[x] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

Thank you for your contribution!
Please add tests to cover your changes.

@bot-gradle test this

Sorry but I'm afraid you're not allowed to do this.

Hello, @ov7a!
Sorry for triggering the workflows. It takes hours for me to run tests locally whereas your CI completes within minutes due to build caching.
I've taken another look at the offending code fragment, and I've realised there is undesirable code duplication: the interface has default method implementation which is okay for me, and two identical overrides that fail me and should be removed IMO.
For the time being, please review the PR as as.
Providing tests is still a work in progress on me, but frankly, it's not trivial given the current coverage.

@unshare See CONTRIBUTING.md: you can benefit from the remote cache too. Also, you can select tests specific to the area of your changes, no need to re-run all tests on each iteration.
Your changes are only about removing some lines. Is it expected?

@ov7a Thanks for the pointer to the build cache. Too late for this engagement: I've generated my own local one. But thanks.
With regard to the delete-only nature of this PR: yes, this is my intention. The overriden implementation don't handle missing version whereas the default implementation in the interface does it correctly, IMO.
See: https://github.com/gradle/gradle/blob/30fa5bccd047c28bf63c318915bc575c7e7e32b7/subprojects/core-api/src/main/java/org/gradle/plugin/use/PluginDependenciesSpec.java#L139

Okay, I asked just to clarify this and be 100% sure.
While I understand where are you coming from with that change, and it seems to be fine, please make sure that all new scenarios are also covered. See integTests.

Thank you for your contribution!
This PR has a valid DCO. The relevant team for this area will confirm the design of the implementation choices.

Sorry, this isn't something we're currently working on and won't have time to take a look at it for awhile.

Guys, really merge this PR. Stop adding new features. Please complete already existing ones.

Forgive me, I am on my phone, but:
plugins {
id(libs.plugins.abc.get().pluginId)
}
Works fine on Gradle latest.
Please take a look at the patch itself and the conversation above.

Everyone, I'm sorry for the delay implementing tests. It's not a top priority for me.
Gradle have requested the test cases for ALL use cases. And the target milestone isn't in the immediate future.
 I.e. they deny having bugs to the utmost and hesitate to accept outside contributions fearing regression.
Most of us witness such a behavior on our day jobs. We should just keep calm and carry on the constructive dialogue.

@unshare I shared a workaround in case people are hitting this issue and waiting on your PR. Good luck getting it merged.

Maybe focus your wrath on Gradle instead of other users who are browsing in your PR and trying to be helpful.
I don't think they are going to merge it.

@sgammon No it does NOT work. I have this in a precompiled script plugin plugins { //    id "org.hidetake.ssh"     id(libs.plugins.gradle.ssh.plugin.get().pluginId) } and even though I'm using a catalog your workaround doesn't work with groovy ```
FAILURE: Build failed with an exception.

Where:
Script '/home/mavek/Documenti/workspace/java/rebirth-gradle-deploy-plugin/src/main/groovy/it.rebirthproject.rebirth-ssh-deploy.gradle' line: 3

What went wrong:
Could not compile script '/home/mavek/Documenti/workspace/java/rebirth-gradle-deploy-plugin/src/main/groovy/it.rebirthproject.rebirth-ssh-deploy.gradle'.

startup failed:
script '/home/mavek/Documenti/workspace/java/rebirth-gradle-deploy-plugin/src/main/groovy/it.rebirthproject.rebirth-ssh-deploy.gradle': 3: argument list must be exactly 1 literal String or String with property replacement

For more information on the plugins {} block, please refer to https://docs.gradle.org/8.6/userguide/plugins.html#sec:plugins_block in the Gradle documentation.
@ line 3, column 5.
id(libs.plugins.gradle.ssh.plugin.get().pluginId)
^
1 error

@mavek87 Where in particular is it failing to find a symbol? I can't tell from your comment; if you fix the code blocks I may be able to help.

@sgammon now the code in the comment should be fixed

@mavek87 I can't quite tell what's going on, but here are the steps I would try:
1) If you are in a buildSrc project
You will need to make your catalog available to your precompiled plugins. You can do this with:
dependencies {
  implementation(files(libs.javaClass.superclass.protectionDomain.codeSource.location))
}

This assumes your version catalog symbol is libs. If it's, for example, other, you would do files(other.javaClass....

This fix is from the Gradle team; you can read more here
2) If you are in a precompiled Kotlin plugin, you may need to create the accessor
You can try this with:
import org.gradle.accessors.dm.LibrariesForLibs
// ...
val libs = the<LibrariesForLibs>()

// now you can access `libs.x.y.z` etc

3) If the error is in your IDE, you may need to suppress DSL scope violations
You can do this with:
@file:Suppress("DSL_SCOPE_VIOLATION")

These code samples are in Kotlin, I'm sorry, I'm not very good at Groovy. But they should work roughly the same way. It seems you are building a Gradle plug-in, right? If you can access libs, you should be able to do this trick because
libs.plugins.gradle.ssh.plugin.get().pluginId

Should be returning a string. Here's what I get for a similar plugin in my IDE:

For the following mapping in the catalog:
[versions]
kotlin-sdk = "1.9.22"
# ...

[plugins]
kotlin-multiplatform = { id = "org.jetbrains.kotlin.multiplatform", version.ref = "kotlin-sdk" }
# ...

I am running Gradle 8.6 and 8.7-RC2 in this case.

Oh, one other thing. If you have catalog structures that are nested, you may need to use a provider. For example
[versions]
# ...

[plugins]
example-plugin = { ... }
example-plugin-nested = { ... }

To access example-plugin's plugin ID with libs, sometimes I need to do:
libs.plugins.example.plugin.asProvider().get().pluginId

@sgammon No it does NOT work. I have this in a precompiled script plugin ```plugins {
//    id "org.hidetake.ssh"
id(libs.plugins.gradle.ssh.plugin.get().pluginId)

}``` and even though I'm using a catalog your workaround doesn't work with groovy

FAILURE: Build failed with an exception.

* Where:

Script '~/Documenti/workspace/java/rebirth-gradle-deploy-plugin/src/main/groovy/it.rebirthproject.rebirth-ssh-deploy.gradle' line: 3

* What went wrong:

Could not compile script '~/Documenti/workspace/java/rebirth-gradle-deploy-plugin/src/main/groovy/it.rebirthproject.rebirth-ssh-deploy.gradle'.

> startup failed:

  script '~/Documenti/workspace/java/rebirth-gradle-deploy-plugin/src/main/groovy/it.rebirthproject.rebirth-ssh-deploy.gradle': 3: argument list must be exactly 1 literal String or String with property replacement

  For more information on the plugins {} block, please refer to https://docs.gradle.org/8.6/userguide/plugins.html#sec:plugins_block in the Gradle documentation.

   @ line 3, column 5.

         id(libs.plugins.gradle.ssh.plugin.get().pluginId)

         ^

  1 error

Your problem is totally unrelated to the PR and the workaround provided, which indeed, works.
Look for version catalogs in included builds and how the plugin block works.

@JavierSegoviaCordoba The workaround is related because it lets you select an existing plugin ID or version, as a string, without repeating yourself. You can pull it from the catalog, even in an included build. It obviously depends where the consuming build is on disk, because you may need to add the catalog to settings.gradle[.kts], etc.
The workaround works for me. I fear Gradle will not merge this because it means specifying no plugin version in the catalog, which may be re-used by other projects. In the interest of actually unblocking developers, I'm just trying to help, but thank you for the pointers.

@JavierSegoviaCordoba The workaround is related because it lets you select an existing plugin ID or version, as a string, without repeating yourself. You can pull it from the catalog, even in an included build. It obviously depends where the consuming build is on disk, because you may need to add the catalog to settings.gradle[.kts], etc.
The workaround works for me. I fear Gradle will not merge this because it means specifying no plugin version in the catalog, which may be re-used by other projects. In the interest of actually unblocking developers, I'm just trying to help, but thank you for the pointers.

It is totally unrelated.
The plugins block is extracted, and analyzed separately, so it is impossible to use a catalog inside the plugins block in a precompiled script plugin in an included build or in the sources of a regular project.
It will not work, neither id or alias, with or without this PR being merged.

The plugins block is extracted, and analyzed separately, so it is impossible to use a catalog inside the plugins block in a precompiled script plugin in an included build or in the sources of a regular project.

It is not without this workaround from the Gradle team. I am accessing my catalogs fine in my precompiled scripts.

@JavierSegoviaCordoba Technically you can circumvent use of the plugins { } block in any capacity, at all, by simply providing a plugin implementation of some kind on the classpath. Then such plugins can be invoked with id(...) or even apply(plugin = ...), as you probably know. My workaround is related because it allows re-use of the ID from an existing catalog. This can be accomplished in different ways from all the places you named above, just not gracefully.
I don't actually know why you are attacking my workaround. At the worst case it won't work, sorry. At the best case someone can find some use for it. Gradle can be very confusing even for experts; there is no sense in us fighting about an issue that we are not empowered to fix.

Again, it will not work. Read all posts in the issue you shared, the author of the hack-workaround explicitly says it does not work inside plugins block.
And apply or any variant like pluginManager does not work in the same way plugins block works, they are not extracted ahead in order to generate any accessor like plugins block does. If you uses the plugins block in a precompiled script plugin and you apply the kotlin plugin, you will see all the accessors for the Kotlin extension, kotlin, the compile tasks, tasks.compileKotlin and so on. That only happens due how plugins block works.
So this PR will not fix that issue, but it is not even aiming to fix that issue as it is unrelated. And the hack-workaround is limited and will not be able to work inside plugins until Gradle fixes the other issue. Not sure if it will be even possible to fix the part of the plugins block due a chicken and egg situation.

Again, it will not work.

To do what, fix the PR? I never claimed it would. People arrive here searching Google for different errors. I did, right?

the author of the hack-workaround explicitly says it does not work inside plugins block.

Thanks, I addressed that

And apply or any variant like pluginManager does not work in the same way plugins block works, they are not extracted ahead in order to generate any accessor like plugins block does

Yes, it sounds like we will have to live without accessors until Gradle merges this and perhaps some other fixes. Accessors are cosmetic

you will see all the accessors for the Kotlin extension

the<KotlinProjectExtension>().apply {
  // ...
}

... the compile tasks, tasks.compileKotlin...

tasks.named("compileKotlin", KotlinCompile::class).configure {
  // ...
}

...not even aiming to fix that issue as it is unrelated...

From OP (bolding mine):
In some cases, one can apply a plugin in plugin { ... } block without specifying a version.
However, at the moment it's not possible to leave version unspecified in a version catalog and then apply via alias
The workaround I have used for this case in the past is the one I posted. Relevance is subjective, anyway. Do you work for Gradle? Maybe JetBrains? If not, why are you here? To tell me my workaround, offered in good faith, was bad? Okay.

Not sure if it will be even possible to fix the part of the plugins block due a chicken and egg situation.

I still don't see an issue. A plugin is an implementation on the classpath, addressable by a plugin ID, which resides at a known spot on disk. You then apply a plugin via various means, the plugins { ... } block is just one of these.

... PD: I am not attacking your workaround, indeed I am defending it...

Also you:

Your problem is totally unrelated to the PR
It is totally unrelated

... my reply was to the person who said your workaround...

You might want to tag who you are talking to because my comment is immediately above yours.

@sgammon This is my project. Gradle init and choose always groovy. No buildSrc, nothing. Just one included project (named "plugin")
this is my settings gradle in the root
pluginManagement {
    repositories {
        gradlePluginPortal()
    }
}

dependencyResolutionManagement {
    versionCatalogs {
        libs {
            plugin('gradle-ssh-plugin', 'org.hidetake.ssh').version("2.11.2")
        }
    }
}

rootProject.name = 'example-gradle'
include('plugin')

In the build.gradle of the "plugin" subproject i have just
plugins {
       id 'groovy-gradle-plugin'
}

And in the precompiled script I have just this
plugins {
//    id "org.hidetake.ssh"
    id(libs.plugins.gradle.ssh.plugin.get().pluginId)
}

If I use the string it works. If I try with the "workaround" I receive the same error I posted you before.

@mavek87 @JavierSegoviaCordoba is right that your issue may be different; if you post it in a new issue (I'm sorry to ask, but it will be cleaner and friendlier to ppl here), I'm happy to give a shot helping out. I'll review the code you've posted here as well.

@sgammon I didn't tag you, I quoted the other person message who said the workaround you shared does not work, and I said it works. For some reason, you replied me.
The rest of tags is to explain why the workaround will work in any situation except the plugins block in precompiled plugins and why it is unrelated to this PR.
I still do not understand why being able to apply a plugin without a version is related to a hack-workaround to use catalogs in any place except plugins block in precompiled script plugins (with or without version, none works), but for some reason, you pursue that goal.

@JavierSegoviaCordoba When you are replying to an email from GitHub, it will enclose all sorts of context; so I have no way of knowing if you are meaning to quote him or not.
Yes, I see now that I was confused. I appreciate that you are not attacking the workaround I posted.
I encouraged this Gradle user to file another bug so we can discuss it there without polluting this discussion.
Ultimately I just want everyone's Gradle builds to work smoothly. Sorry for misidentifying your intentions and tone. In the future, it would help someone in my position to clearly tag who you are talking to.

I still do not understand why being able to apply a plugin without a version is related to a hack-workaround to use catalogs in any place except plugins block

I do not presume to know every reason why a developer wants to do something. If they want a catalog somewhere, there are usually ways to do it. Catalogs are useful and help with less repetition.
I don't see any problem and I don't think your understanding is needed.

@sgammon that clarifies the misunderstood. To be honest I haven't replied via email in GitHub before so I didn't have any clue about that :)

@JavierSegoviaCordoba No problem, sorry as well. Neither of us work for Gradle or own this PR, though, so we should probably continue discussing elsewhere if needed. Cheers.

@JavierSegoviaCordoba the reason is because I dont want to type and duplicate something I also have in my catalog yet ("org.hidetake.ssh")

@mavek87 you could name it something like "Issues with Version Catalog accessors for precompiled plugin" and tag me.

@JavierSegoviaCordoba the reason is because I dont want to type and duplicate something I also have in my catalog yet ("org.hidetake.ssh")

The only way is not using plugins block. I am not a groovy user (my knowledge about it is almost zero), maybe using apply or applying the plugins with pluginManager does not force you to access the extensions as you would do in Kotlin with the or configure. If not, you will need to use those methods as you would be losing the accessors in the precompiled plugin.

@JavierSegoviaCordoba Groovy has dynamic property resolution features that Kotlin doesn't have, so it is entirely possible that the accessors do work in Groovy. Groovy can, in some cases, also reach across private boundaries. Spock does this for testing. So I would not actually bet on that (this is coming from another heavy Kotlin user, I don't know for sure).

@sgammon I opened a new issue: https://github.com/gradle/gradle/issues/28371
Unfortunately I'm the opposite, I really dont understand nothing about Kotlin. Trying for days to look in the documentation for this and other problems I'm facing but it's really though


Case-insensitively duplicate branch names in Git repo

The AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet repository contains some branches whose names are the same except for upper vs. lower case. This makes Git unable to distinguish those branches from each other when I clone the repository to a case-insensitive file system such as on Windows.
c56835c28710898d814ebdc2163b2d4988523a0d        refs/heads/geok/updateversion
c08ae2c2e72363da7ec848db4142e96aa05fb023        refs/heads/geok/updateVersion
e9ab9737e275bf6e84ce76f307b4c7aac95fdc21        refs/heads/xiao/RemoveIncorrectLogMessage
6a783a1d0386cbe992496831709774e70d599690        refs/heads/Xiao/RemoveIncorrectLogMessage

In total, there are five branches whose names start with xiao/, and two branches whose names start with Xiao/. It would be best to choose either xiao or Xiao and then use that consistently.
Now when I run git remote update --prune, I get these spurious changes every time:
$ git remote update --prune
Fetching origin
From https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet
 - [deleted]           (none)     -> origin/Xiao/EnableDeterministicBuildsforCIBuild
 - [deleted]           (none)     -> origin/Xiao/IntegrateServerNonceValidation
 - [deleted]           (none)     -> origin/Xiao/deterministic
 - [deleted]           (none)     -> origin/Xiao/fixsecuritybugs
 + e9ab9737...6a783a1d Xiao/RemoveIncorrectLogMessage -> origin/Xiao/RemoveIncorrectLogMessage  (forced update)
 * [new branch]        geok/updateversion   -> origin/geok/updateversion
 * [new branch]        xiao/EnableDeterministicBuildsforCIBuild -> origin/xiao/EnableDeterministicBuildsforCIBuild
 * [new branch]        xiao/IntegrateServerNonceValidation -> origin/xiao/IntegrateServerNonceValidation
 * [new branch]        xiao/RemoveIncorrectLogMessage -> origin/xiao/RemoveIncorrectLogMessage
 * [new branch]        xiao/deterministic   -> origin/xiao/deterministic
 * [new branch]        xiao/fixsecuritybugs -> origin/xiao/fixsecuritybugs

Using git version 2.33.0.windows.2.

@KalleOlaviNiemitalo thanks for noticing this and pointing it out.
I will inform the team.


Fix quattro-tl group name and add OTG owners

Fix team name for featureprofiles-quattro-tl
Add new owner/group which is configured to be autoassigned for reviews in /otg-tests/ folders.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 45.603%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
586

Relevant Lines:
1285

💛  - Coveralls


Add codecov

This works.  Not quite sure if I need to do something more to get codecov reports to link here.  I'll merge and we can keep playing with it.


KBFS (K:) is unavailable [Windows 10]

Keybase GUI Version: 1.0.40-20180205105656+493a528
I'm having trouble viewing the KBFS in Windows Explorer on Windows 10.  I have tried reinstalling the KBFS as a troubleshooting step, but this has not solved the issue.  I can still send messages fine.
my log id: 891a68ddaa5027ea5d10951c

thanks @s3cpat.  cc @zander

cc @zanderz

Possible that it is a similar issue to #10431 , now that I'm browsing through other issues

I've changed my drive letter to X: and restarted keybase, this seems to let me access the KBFS.

i've got the same issue for quite some time now. It sometimes works, but mostly doesn't.
How did you manage to change the drive letter?
I found no setting for this in the keybase client, nor in the context menu for the drive.
And it's not shown in the disk management control panel, where I'd do it for physical drives.

i've "resolved" the issue for me by disabling the autostart.
This is pure conjecture, but I'm guessing that some other process tries to access it while the startup isn't completed yet, which probably deadlocks the program somehow.
So probably try disabling virus scanners or backup programs for that location if somebody want to  "solve" the problem on their end. It was most likely duplicati in my case.

@wohlben sorry for the delay, changed the drive letter with:
keybase kbfsmount set x

in cmd

@s3cpat This just caused my K: to change to a 'Local Disk' (still unavailable) and did not mount anything to X:

for me the combination of

updating Dokan (to 1.3.1)

disableing autostart

renaming the drive:
  keybase kbfsmount set X:

did it. Hope this is gonna last :)


Databricks sql analytics broken with pyodbc

in tag 1.0.0 databricks with pyodbc worked very well
But after this code its stopped working
https://github.com/apache/superset/blob/423ff50768cd1d70fc63a29ac3312916d4aaf65e/superset/db_engine_specs/databricks.py#L21
Solution add support for pyodbc as well not only hive as SQL analytics relies on that

@saurabnigam feel free to open a PR for the regression, we will review your code.

Closing as stale, but happy to re-open or review a  PR if there's a need.


Initial i18n support

Add support for I18n translations through the translate and t methods on views that include Phlex::I18n.

Closing this in favour of #251.


SCM commit button: Codicons should not be localized

Testing #153366
I noticed this:
https://github.com/microsoft/vscode/blob/ade80ca128741e2aa97e836a1ff0ad7cf8d67fa7/extensions/git/src/actionButton.ts#L80-L100
The proper way to localize here would be:
title = '$(arrow-up) ' + localize('scm button commit and push title', "Commit & Push");

Otherwise localization extensions could change the icon or mess up the syntax.

I'd also recommend pulling the icons out of the localize strings all together in other spots like:
https://github.com/microsoft/vscode/blob/ade80ca128741e2aa97e836a1ff0ad7cf8d67fa7/extensions/git/src/actionButton.ts#L144
https://github.com/microsoft/vscode/blob/ade80ca128741e2aa97e836a1ff0ad7cf8d67fa7/extensions/git/src/actionButton.ts#L177
They should be within the localize string only if you want the position of the icon localized.

Closing this issue as this work is being tracked by a debt item - https://github.com/microsoft/vscode/issues/153743


Allow returning the final prompts that are sent to LLMs

Problem
Prompts are formatted and merged (e.g. adding ---context---\n) before being sent to LLMs. The final prompts sent to LLMs can vary depending on the type of chains used and can be quite different from the original inputs to the chains. However, the user cannot see such differences directly, making debugging more difficult.
Changes

Allow returning the final prompts as part of LLMResult.
Final prompts here are collected before being sent to specific API vendors (e.g. OpenAI). Therefore the addition to LLMResult not ChatResult.
Final prompts are returned with the _call() method of LLMChain in /chains/llm.py.

Limitations

This is not yet implemented on all usage of LLMResult (i.e. only implemented in /chat_models not /llms)
Due to the use of predict() instead of _call() in most of the chains, final prompts will not be returned there.
Async versions are not implemented
If you like the idea, I can fix the above limitations

Who can review?
Tag maintainers/contributors who might be interested:
@hwchase17
@agola11

Can I get some feedback so that I can decide whether to keep working on this feature or not? Thank you so much!
@hwchase17
@agola11

I believe this is implemented pretty well by callbacks, or even easier in LangSmith!
Closing because the PR wouldn't line up with the current directory structure of the library (would need to be in /libs/langchain/langchain instead of /langchain). Feel free to reopen against the current head if it's still relevant!


Fix newick output

Summary by Sourcery
Fix the Newick output formatting by removing quotes from the string representation of the tree dictionary. Enhance the 'ctree' function to handle cases where the tree is not generated by printing an error message and exiting. Update the test for 'ctree' to include the '-hp' flag in the arguments.
Bug Fixes:

Fix the Newick tree output by ensuring the string representation of the tree dictionary is correctly formatted without quotes.

Enhancements:

Add a check in the 'ctree' function to print an error message and exit if the tree is not generated.

Tests:

Update the 'test_ctree' function to include the '-hp' flag in the arguments for testing.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

3 of 5 (60.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 91.816%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/diverse_seq/cli.py
1
3
33.33%

Files with Coverage Reduction
New Missed Lines
%

src/diverse_seq/distance.py
1
91.27%

src/diverse_seq/cli.py
1
85.03%

Totals

Change from base Build<PHONE_NUMBER>3:
-0.1%

Covered Lines:
1178

Relevant Lines:
1283

💛  - Coveralls


meghana-pr.md

hi gssoc team,
i had made 2 pr's below, there is no response for the pr i made, kindly check and assign the points
level 3
https://github.com/shivamyadavrgipt/Social_media_management/pull/230#issuecomment-2466554596
level 2
https://github.com/BamaCharanChhandogi/GitFinder/pull/112#issuecomment-2466544164
@sanjay-kv @MastanSayyad kindly look into it

@aslams2020 the admin assigned me level 3 while assigning the issue and u gave level 2 while merging for the pr

Added level3 for it


mk_function incompatibility with OrdinaryDiffEq

Thanks for this great package!
I am trying to build differential equations at run-time and then solve them with OrdinaryDiffEq. Based on the documentation the following should work.
using OrdinaryDiffEq
import OrdinaryDiffEq: solve
using GeneralizedGenerated

# define the RHS the old fashioned way
f1(du, u, p, t) = begin
    du[1] = p[1]*1/u[1]
end
u0 = [1.0]
p = [1.0]
t = (0.0,1.0)
du = similar(u0)
f1(du, u0, p, 0)
@show du
prob = ODEProblem(f1, u0, t, p)
solve(prob, Tsit5())
println("The old fashioned way works")

# use an anonymous function and eval it.
fex = :((du, u, p, t) -> begin
        du[1] = p[1]*1/u[1]
        end)

f2 = eval(fex)
prob = ODEProblem(f2, u0, t, p)
soln = solve(prob, Tsit5())
println("Works fine with eval")

#use gg mk_function
f3   = mk_function(fex)

f3(du, u0, p, 0)
@show du
println("We can call f3")
prob = ODEProblem(f3, u0, t, p)
# this throws a an Error
soln = solve(prob, Tsit5())

The top of the stack trace is:
ERROR: LoadError: Input arguments too few.
Stacktrace:
 [1] error(::String) at ./error.jl:33
 [2] _ass_positional_args!(::Array{Expr,1}, ::DataStructures.Cons{GeneralizedGenerated.Argument}, ::Int64, ::Symbol) at /Users/jfairbanks6/.julia/packages/GeneralizedGenerated/x3uMp/src/runtime_funcs.jl:57
 [3] #s15#27(::Any, ::Any, ::Any, ::Any) at /Users/jfairbanks6/.julia/packages/GeneralizedGenerated/x3uMp/src/runtime_funcs.jl:72

Is this user error or an incompatibility with OrdinaryDiffEq?
@chrisrackauckas have you used GG with ODEs?
Relevant Versions:

Julia 1.0.3
6b9d7cbe] GeneralizedGenerated v0.1.3
1dea7af3] OrdinaryDiffEq v5.17.2

Hi, James. Thanks for your kind words and the feedback. I'll debug this to detect the problems and make a patch.

ModelingToolkit.jl is using GG and it seems to work there, so 🤷♂

A function produced by GG has only one method, whose signature is similar to this
f(args…; kwargs…) = ...

If the following works
f3   = mk_function(fex)

f3(du, u0, p, 0)
@show du
println("We can call f3")
prob = ODEProblem(t -> f3(t), u0, t, p)
soln = solve(prob, Tsit5())

, I'd think this issue is caused by ODEProblembut not the GG's function representation.

what about ODEProblem{true}(...)?

what about ODEProblem{true}(...)?

I'm away from my computer now. Will try this later.

This works
f4 = (du, u, p, t) -> f3(du,  u, p, t)
prob = ODEProblem(f4, u0, t, p)
soln = solve(prob, Tsit5())

totally solves my problem. I made a PR to provide an example for new users! Thanks so much for this julia internals witchcraft.


chore:updates the readme with the latest kpt commands

Updates the README.md:

the kpt link to point to kpt.dev
kpt fn run replaced with kpt fn eval

Removes the policies\constraints folder as running the bundlers fails since the folder is already present.

@morgante - please review


TBChi: GlowFork.TaskQueue

Implement an Initialization Task Queue. LoadingState will initialize a TaskQueue with configurable number of runners.
Implement an IUpdateStatus (???) task wrapper that pushes an update to the assigned delegate.

Complete


Implement Rewrite for [ast::Attribute]

r? @nrc @marcusklaas

A test would be nice, maybe one where the inner comment is rewritten?

Updated with a test. There isn't any problem to rewrite comments.
Thanks for the review, I cherry-picked this from an obsolete branch, and my review wasn't careful enough...
Is everything clear ?

Brilliant!
r+


Enable Elasticsearch bboss plugin to fix bug #6048

Enable Elasticsearch bboss plugin to fix bug #6048

Codecov Report

Merging #6077 into master will increase coverage by 0.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6077      +/-   ##
==========================================
+ Coverage   40.76%   41.33%   +0.56%
==========================================
  Files        3020     3036      +16
  Lines       90572    91802    +1230
  Branches    11819    12007     +188
==========================================
+ Hits        36926    37942    +1016
- Misses      50484    50647     +163
- Partials     3162     3213      +51

Impacted Files
Coverage Δ

...t/collector/receiver/grpc/service/StatService.java
17.39% <0%> (-3.54%)
:arrow_down:

...t/collector/receiver/grpc/service/SpanService.java
17.14% <0%> (-3.32%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
82.92% <0%> (-2.44%)
:arrow_down:

...nt/collector/config/StatReceiverConfiguration.java
73.45% <0%> (-1.71%)
:arrow_down:

...pinpoint/profiler/monitor/DeadlockMonitorTask.java
18.6% <0%> (-1.17%)
:arrow_down:

...vercorp/pinpoint/common/util/DefaultSqlParser.java
85.92% <0%> (ø)

...orp/pinpoint/loader/trace/TraceMetadataLoader.java
81.06% <0%> (ø)

...int/loader/trace/AnnotationKeyMatcherRegistry.java
62.5% <0%> (ø)

...avercorp/pinpoint/common/util/DefaultTimeSlot.java
100% <0%> (ø)

...rp/pinpoint/common/util/DefaultApiDescription.java
91.17% <0%> (ø)

... and 27 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 af464b1...bcc3b3e. Read the comment docs.


Using a src/ dir for the next codebase doesn't seem supported

What version of this package are you using?
v2.0.4
What operating system, Node.js, and npm version?
macOS Ventura 13.2, node 18.x, yarn 3.5.0
What happened?
Using the new next app directory nested under a src/ directory displays the following error:
Unhandled Runtime Error
Error: Cannot read properties of undefined (reading '_context')

Moving the directory to the root of the project works as expected.
Seems related to how next-translate-plugin is looking for the source code.
What did you expect to happen?
No error
Are you willing to submit a pull request to fix this bug?
Yes if pointed to the right direction 🤷

@synoptase can you try next-translate-plugin 2.0.5-canary.1 prerelease to verify that is working fine now?

@aralroca : works like a charm :) Thank you!
I'm guessing that the switch from /app to /pages will be progressive thus, cool  libraries like yours need to go along that switch.


XIONE-1313 Add DeviceIdentification implementation for Realtek

Reason for change:

Add DeviceIdentification implementation for Realtek SOC
Test Procedure:
Add DeviceIdentification in PACKAGECONFIG in rdkservices_git.bbappend for Realtek SOC layer
and build RDKServices, DeviceIdentification plug-in should be build and works.
Risks: Low

Change-Id: I588a563d77a78a94ed6395d4dc2082cc7e454e23
Signed-off-by: Mark Yang<EMAIL_ADDRESS>

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


Disk Cache size and memory cache

Hello, it will be very useful if there is an option to control the size of the cache and an option to disable the disk cache and enable memory ram cache for more speed because some HDDs are worn and old....
Thank you so much :)

If you want to avoid disk writes entirely you should be using incognito mode. There are still a significant number of writes because of factors such as browsing history that need to be persisted onto disk.

@qua3k Thank you for your advise :)


Import fonts locally

The Regional Court of Munich I has determined the unlawfulness of the remote integration of Google Fonts. As a result, all fonts must be integrated locally in all web services used in Germany.
Goal: => Local integration of all fonts

Fixed in v1.0.1


add option to set enforce_https

Please merge ?

Here we go! v1.4.0 has been just released. :)

Thank you for being so responsive !!
On Wed, Dec 23, 2020 at 1:34 AM Anton Babenko<EMAIL_ADDRESS>wrote:

Here we go! v1.4.0
https://github.com/terraform-community-modules/tf_aws_elasticsearch/releases/v1.4.0
has been just released. :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/terraform-community-modules/tf_aws_elasticsearch/pull/47#issuecomment-750036247,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGWZJFDDN55UPFDCP6Y7Z3SWGZ7BANCNFSM4OU2Y2ZQ
.


.NET Core task prints ##[warning]Could not set the code paging of due to following error: {} on container

Current state
When using the DotNetCoreCLI@2 task on a container (here: mcr.microsoft.com/dotnet/sdk:6.0) it succeeds, but prints the following warning:

##[warning]Could not set the code paging of due to following error: {} on container

Screenshot:

Desired state
Should not print a warning (= same behavior as without using a container to run the build)
Environment

This is on the hosted service
vmImage: windows-latest
container: mcr.microsoft.com/dotnet/sdk:6.0

Anyone else seeing this?

I also experience this


docs(release): add e2e tests check to release docs

Should we expose the url?

Should we expose the url?

I have removed the URL.


Angular example with form tags keypress error

-I have provided a code example.
Using the angular example from the documentation I changed the
form.component template from

to
<form class="myCF" #myCF></form>
- If I encountered an issue I have provided: a description of the actual and expected behavior
The form actually works as normal, however, when I have lastpass plugin installed using chrome I receive the following error on each enter keypress on each form input:
onloadwff.js:71 Uncaught TypeError: Cannot read property 'type' of undefined     at setFieldValue (onloadwff.js:71)     at HTMLFormElement.formKeydownListener (onloadwff.js:71)
This error is documented on another plugin
-what version of Conversational Forms I am using
1.0.1
-what browser and version I am using
Chrome Version 78.0.3904.108 (Official Build) (64-bit)
-the output of the browser console
onloadwff.js:71 Uncaught TypeError: Cannot read property 'type' of undefined     at setFieldValue (onloadwff.js:71)     at HTMLFormElement.formKeydownListener (onloadwff.js:71)

@keyboardbreaker this bug comes from your lastpass extension


Fix UID and GID when providing IDs instead of names

Hi there!
I stumbled upon this problem wanting to set my god task as a user given it's uid instead of it's name.
There might be a smarter way to do this, but this PR handles the case.

Hi @eric, I've rebased this PR. Any chance to get this into master soon?

Thanks for the review @eric! I've updated the PR.

Looks good!


Add workbench.action.quickOpenNavigateNextInWindowPicker?

It would be nice if Quick Switch Window quick pick would allow repeating the keybinding to move the selection through the quick pick, so it would be the Window analogue to ctrl+tab for Editors.

Already possible today:
{
	"key": "ctrl+w",
	"command": "workbench.action.quickSwitchWindow"
},
{
	"key": "ctrl+w",
	"command": "workbench.action.quickOpenNavigateNext",
	"when": "inQuickOpen"
},
{
	"key": "ctrl+shift+w",
	"command": "workbench.action.quickOpenNavigatePrevious",
	"when": "inQuickOpen"
}


Loading extension remove redundant code regarding legacy Detours

Nothing is added to this list:
https://github.com/CitiesSkylinesMods/TMPE/blob/06eda8d0f006ae7ab603bd0f878af9521fe4b8e1/TLM/TLM/LoadingExtension.cs#L279
The list is used in multiple places in two files. All this code can be deleted.
My guess is that this is legacy code that has stuck along. We use Harmony now.
PS: We are not planning to use this list in future are we?

related #730


Test with action_mean in cleanrl

I want to use action_mean (instead of random sampling) to test the already trained neural network model in CleanRL. The results are very poor, but using the original random sampling method for testing yields very good results. I would like to ask why this might be happening?
The testing code is modified as shown in the uploaded image, using action_mean for testing.

Hello @LjXiii, I am not sure why the specific RL framework is not yielding desired results. I had tried the same with Sample-Factory and was getting expected results.  Do other frameworks also cause a similar behavior?

Closing this issue.
@LjXiii  Please let me know if this is still not resolved and I will reopen it.


Possible problem of declaring some elements

hi, I would like to make a small project using your code, only problem is that when I try to build it I encounter several errors related to the initialization of some varibles.
I report the output of PlatformIO below:
Verbose mode can be enabled via -v, --verbose option
CONFIGURATION: https://docs.platformio.org/page/boards/espressif32/esp32dev.html
PLATFORM: Espressif 32 (3.5.0) > Espressif ESP32 Dev Module
HARDWARE: ESP32 240MHz, 320KB RAM, 4MB Flash
DEBUG: Current (esp-prog) External (esp-prog, iot-bus-jtag, jlink, minimodule, olimex-arm-usb-ocd, olimex-arm-usb-ocd-h, olimex-arm-usb-tiny-h, olimex-jtag-tiny, tumpa)
PACKAGES:

framework-arduinoespressif32 @ 3.10006.210326 (1.0.6)
tool-esptoolpy @ 1.30100.210531 (3.1.0)
toolchain-xtensa32 @ 2.50200.97 (5.2.0)
LDF: Library Dependency Finder -> https://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 36 compatible libraries
Scanning dependencies...
Dependency Graph
|-- YACL @ 1.0.3
|-- Crypto @ 0.4.0
|-- micro-ecc @ 1.0.0
|-- Adafruit BusIO @ 1.14.5
|-- Adafruit SSD1306 @ 2.5.9
|-- Adafruit MPR121 @ 1.1.3
|-- SparkFun ATECCX08a Arduino Library @ 1.3.1+sha.71c6c3c
|-- ESP32 BLE Arduino @ 1.0.1
|-- Adafruit GFX Library @ 1.11.9
|-- Wire @ 1.0.1
Building in release mode
Compiling .pio\build\esp32dev\src\ble\device.cpp.o
Compiling .pio\build\esp32dev\src\ctap\clientpin.cpp.o
Compiling .pio\build\esp32dev\src\ctap\common.cpp.o
Compiling .pio\build\esp32dev\src\ctap\ctap.cpp.o
Compiling .pio\build\esp32dev\src\ctap\getassertion.cpp.o
Compiling .pio\build\esp32dev\src\ctap\getinfo.cpp.o
Compiling .pio\build\esp32dev\src\ctap\makecredential.cpp.o
Compiling .pio\build\esp32dev\src\ctap\reset.cpp.o
Compiling .pio\build\esp32dev\src\fido2\authenticator\authenticator.cpp.o
Compiling .pio\build\esp32dev\src\fido2\authenticator\clientpin.cpp.o
Compiling .pio\build\esp32dev\src\fido2\authenticator\common.cpp.o
Compiling .pio\build\esp32dev\src\fido2\authenticator\getassertion.cpp.o
In file included from src\ctap\common.cpp:5:0:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
In file included from src\ctap\ctap.cpp:5:0:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
In file included from src\ctap\clientpin.cpp:8:0:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
In file included from src\ctap\reset.cpp:5:0:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
src\ble\device.cpp: In function 'void BLE::init()':
src\ble\device.cpp:13:26: error: 'DEVICE_NAME' was not declared in this scope
#define GATT_DEVICE_NAME DEVICE_NAME
^
src\ble\device.cpp:26:25: note: in expansion of macro 'GATT_DEVICE_NAME'
BLEDevice::init(GATT_DEVICE_NAME);
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
In file included from src\ctap\getassertion.cpp:7:0:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
In file included from include/fido2/authenticator/authenticator.h:7:0,
from src\fido2\authenticator\authenticator.cpp:5:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
In file included from src\ctap\getinfo.cpp:7:0:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
In file included from include/fido2/authenticator/authenticator.h:7:0,
from src\fido2\authenticator\clientpin.cpp:3:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
In file included from include/fido2/authenticator/authenticator.h:7:0,
from src\ctap\makecredential.cpp:7:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
In file included from include/fido2/authenticator/authenticator.h:7:0,
from src\fido2\authenticator\getassertion.cpp:3:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
src\ctap\getassertion.cpp: In function 'FIDO2::CTAP::Status FIDO2::CTAP::Request::parseAllowList(CBOR&, FIDO2::CTAP::Request::GetAssertion*)':
src\ctap\getassertion.cpp:52:81: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
if(!cborId.is_bytestring() || cborId.get_bytestring_len() > CREDENTIAL_ID_LENGTH)
^
src\ctap\getassertion.cpp:57:60: error: request for member 'value' in 'cd.std::unique_ptr<_Tp, _Dp>::operator-><FIDO2::CTAP::PublicKeyCredentialDescriptor, std::default_deleteFIDO2::CTAP::PublicKeyCredentialDescriptor >()->FIDO2::CTAP::PublicKeyCredentialDescriptor::credentialId', which is of non-class type 'int'
cborId.get_bytestring(cd->credentialId.value);
^
In file included from include/fido2/authenticator/authenticator.h:7:0,
from src\fido2\authenticator\common.cpp:3:
include/fido2/ctap/ctap.h:128:34: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
uint8_t credentialId[CREDENTIAL_ID_LENGTH];
^
include/fido2/ctap/ctap.h:159:25: error: 'CREDENTIAL_ID_LENGTH' was not declared in this scope
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
include/fido2/ctap/ctap.h:159:45: error: template argument 1 is invalid
FixedBuffer<CREDENTIAL_ID_LENGTH> credentialId;
^
*** [.pio\build\esp32dev\src\ble\device.cpp.o] Error 1
*** [.pio\build\esp32dev\src\ctap\common.cpp.o] Error 1
*** [.pio\build\esp32dev\src\ctap\ctap.cpp.o] Error 1
src\ctap\makecredential.cpp: In function 'FIDO2::CTAP::Status FIDO2::CTAP::Request::parseExcludeList(CBOR&, FIDO2::CTAP::Request::MakeCredential*)':
src\ctap\makecredential.cpp:113:70: error: request for member 'maxLength' in 'credDescriptor.FIDO2::CTAP::PublicKeyCredentialDescriptor::credentialId', which is of non-class type 'int'
if (length > credDescriptor.credentialId.maxLength)
^
src\ctap\makecredential.cpp:117:57: error: request for member 'alloc' in 'credDescriptor.FIDO2::CTAP::PublicKeyCredentialDescriptor::credentialId', which is of non-class type 'int'
credDescriptor.credentialId.alloc(length);
^
src\ctap\makecredential.cpp:118:78: error: request for member 'value' in 'credDescriptor.FIDO2::CTAP::PublicKeyCredentialDescriptor::credentialId', which is of non-class type 'int'
value.get_bytestring(credDescriptor.credentialId.value);
^
src\ctap\makecredential.cpp:130:82: error: request for member 'length' in 'credDescriptor.FIDO2::CTAP::PublicKeyCredentialDescriptor::credentialId', which is of non-class type 'int'
if (credDescriptor.type == "" || credDescriptor.credentialId.length == 0)
^
*** [.pio\build\esp32dev\src\ctap\reset.cpp.o] Error 1
*** [.pio\build\esp32dev\src\fido2\authenticator\authenticator.cpp.o] Error 1
*** [.pio\build\esp32dev\src\fido2\authenticator\getassertion.cpp.o] Error 1
*** [.pio\build\esp32dev\src\ctap\getassertion.cpp.o] Error 1
*** [.pio\build\esp32dev\src\ctap\clientpin.cpp.o] Error 1
*** [.pio\build\esp32dev\src\fido2\authenticator\common.cpp.o] Error 1
*** [.pio\build\esp32dev\src\ctap\getinfo.cpp.o] Error 1
*** [.pio\build\esp32dev\src\fido2\authenticator\clientpin.cpp.o] Error 1
*** [.pio\build\esp32dev\src\ctap\makecredential.cpp.o] Error 1`

As you can see the main problem is that it can't find the statement of "CREDENTIAL_ID_LENGTH".
I followed the instructions given in the README.md file and on github to buid the code.
I cloned the code via git.
I hope you guys can help me.

ok, found the problem.
config.h is saved as "config.h.example" and this prevents the content from being imported.
if u rename the file all start working fine!

ok, found the problem.
config.h is saved as "config.h.example" and this prevents the content from being imported.
if u rename the file all start working fine!


update Protocol Buffers v3.1.0 -> v3.2.0

Can you modify the travis script too? So we know it doesn't fail if we update to 3.2.0.
https://github.com/AeonLucid/POGOProtos/blob/master/.travis.yml

ok AeonLucid  ready

Thanks 👍


Please add the racket topic

Hi,
Please add the racket topic - I think others will be interested and this will help them find it at
https://github.com/topics/racket
https://help.github.com/en/articles/classifying-your-repository-with-topics
Kind regards,
Stephen

Thanks for the suggestion. Done.


Clean Gameview: Add option to enable blur

Clean Gameview
Adds an option to enable blur on the game bar.

Looks cool! Thanks a bunch


make db-prepare-artix7 fails on Ubuntu 22.04

`(base) james@james-System-Product-Name:~/Desktop/Casino/prjxray$ make db-prepare-artix7
Preparing artix7 files
Traceback (most recent call last):
File "/home/james/Desktop/Casino/prjxray/utils/create_environment.py", line 14, in
from prjxray.util import get_part_information, get_part_resources, get_fabric_for_part
ModuleNotFoundError: No module named 'prjxray'
Traceback (most recent call last):
File "./utils/update_parts.py", line 12, in
import yaml
ModuleNotFoundError: No module named 'yaml'
make: *** [Makefile:190: db-prepare-artix7] Error 1
`

`(base) james@james-System-Product-Name:~/Desktop/Casino/prjxray$ make db-prepare-artix7
Preparing artix7 files
Traceback (most recent call last):
File "/home/james/Desktop/Casino/prjxray/utils/create_environment.py", line 14, in
from prjxray.util import get_part_information, get_part_resources, get_fabric_for_part
ModuleNotFoundError: No module named 'prjxray'
Traceback (most recent call last):
File "./utils/update_parts.py", line 12, in
import yaml
ModuleNotFoundError: No module named 'yaml'
make: *** [Makefile:190: db-prepare-artix7] Error 1
`

Also getting this error

Preparing artix7 files
============================
Traceback (most recent call last):
  File "/home/james/Desktop/Casino/prjxray/utils/create_environment.py", line 14, in <module>
    from prjxray.util import get_part_information, get_part_resources, get_fabric_for_part
ModuleNotFoundError: No module named 'prjxray'
Traceback (most recent call last):
  File "./utils/update_parts.py", line 15, in <module>
    from prjxray import util
ModuleNotFoundError: No module named 'prjxray'
make: *** [Makefile:190: db-prepare-artix7] Error 1

This was a Python dependency problem specific to my machine


Redesign constants.py and more Configuration-Options - Ideas

Currently the file constants.py sets some values manually, loads the config/config.json (hardcoded) and sets global constants in python.
This should be redesigned, so that even more can be configured manually, defaulting to values which are not set.
The file config.json can be then expanded (if needed) accordingly.
In Addition to this: It should be more configurable using additional command line arguments as --configFold= and similar. (this will be added!)
I would be happy to hear more suggestions and Ideas!

The constants.py has been changed. Now multiple instances can be run on the same Computer (which is much easier for debugging). To use another Config-Folder, use --conf PATH. There are also more Options available. Check them out with --help.
Feel free to propose other Configuration-Options here!


insured -> token precision changes

1 - modify conversion direction such that XCTD_TO_USDC_RATE (e.g. 0.14 so that for each USDC we get 7 XCTD)
2 - use arbitrary precision to support any decimal difference between tokens

#91


:warning:  Remove IPAM deployment logic

When making IPAm into a provider for CAPI, I needed to change the namespace where IPAM is living and how IPAM is deployed. This PR is made to match those changes in IPAM. There are also changes to metal-dev-env. These are the related PRs:

IPAM: https://github.com/metal3-io/ip-address-manager/pull/692
metal3-de-env: https://github.com/metal3-io/metal3-dev-env/pull/1459

All three PRs' changes need to be tested together. Check test in the metal-dev-env PR.

What this PR does / why we need it: To be compatible with IPAM changes

Should this be marked a breaking change as it is switching namespaces and hence upgrades are affected?

/test metal3-ubuntu-e2e-integration-test-main

/hold
This needs to go in to 1.10 cycle, so holding until 1.9 is cut.


Update docs

This is heavily inspired by https://github.com/arturo-lang/arturo?tab=readme-ov-file#the-language. I also added the Discord link to the installation docs in case further assistance is needed.

Closes #111


[djl-ui] Address security warning and upgrade to djl 0.6.0

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

shouldn't we also update the existing package-lock.json file for websocket-extensions ?

shouldn't we also update the existing package-lock.json file for websocket-extensions ?

package-lock.json is generated file, we don't need checkin this file.


OWIN classes missing

It would be useful to add documentation for OWIN related classes like IAppBuilder, StartupOptions etc.

@milivojm Where's the doc?

@milivojm Where's the doc?

Issue is about missing documentation. I guess that means there aren't any.

(@millivojm I didn't quite understand this issue, because I'm unfamiliar with OWIN.)  Is https://docs.microsoft.com/en-us/aspnet/core/fundamentals/owin?view=aspnetcore-6.0 what you're searching for?

IAppBuilder for example.

@milivojm Please submit feedback by clicking on the button at the doc's bottom.


Update collectfast to 2.0.0

This PR updates Collectfast from 1.3.2 to 2.0.0.

Running the bot with an API key allows it to query pyup.io's API for changelogs and insecure packages. This is highly recommended for production use. Learn More

@browniebroke 2.0.0 unfortunately had a pretty bad bug that is now fixed in 2.0.1. I recommend skipping 2.0.0.

Thanks for the heads up!


[BugFix] fix element_memory_usage of ArrayColumn

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #17035
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[x] 2.3
[x] 2.2

run starrocks_admit_test

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.3

@Mergifyio backport branch-2.2

run starrocks_admit_test


Database corruption error

Describe the bug
After initial sync, node kept running until CL client (Lighthouse) started giving me this message every block
un 19 10:19:19.394 WARN Failed to sync chain built on invalid parent, advice: check execution node for corruption then restart it and Lighthouse, parent_root: 0xdfad422d2e6a858bd55b45d586d913da8b6d0545bfeed7c1c8c914a066e1d6b3
At this point the node was not running stages, only printing peer updates.
I restarted it with Lighthouse and here is the error I got
2 023-06-19T10:25:50.344592Z ERROR execute{stage=Bodies}: sync::pipeline: Stage encountered a fatal error: An internal database error occurred: Database write error code: -30796. stage=Bodies
2023-06-19T10:25:50.368062Z  INFO reth::cli: Wrote network peers to file peers_file="/root/.local/share/reth/mainnet/known-peers.json"
Error: A stage encountered an irrecoverable error.

Caused by:
   0: An internal database error occurred: Database write error code: -30796
   1: Database write error code: -30796

Location:
    bin/reth/src/node/mod.rs:392:9

Steps to reproduce
idk
Node logs
No response
Platform(s)
Linux (x86)
What version/commit are you on?
a8be68a82c20ecf6b50ea66d8a706a3493c9bce0
Code of Conduct

[X] I agree to follow the Code of Conduct

Possibly related to https://github.com/paradigmxyz/reth/issues/3087


Import of Books with subtitles ":" break markdown.

It looks like the output from books which have a colon in the title break markdown when viewed in Obsidian. Fixed manually by changing the colon to "SPACE DASH SPACE" " - " so that they show up correctly.
Its and easy fix from our end but having to do that after every export is teadious.
I found this while checking the spell lists. The spells below are teh first ones that i came across with the issue and the text below them is how I fixed them.
Acid Stream:
"Unearthed Arcana:"
"Unearthed Arcana - "
Blade Of Disaster:
"Icewind Dale:"
"Icewind Dale - "
Blade Of Disaster: Doesnt import Classes)
borrowed-knowledge
"Strixhaven:"
"Strixhaven - "
Gust:
"Eberron: Rising"
"Eberron - Rising"
Linked Glyphs
"Adventures in the Forgotten Realms:"
"Adventures in the Forgotten Realms - "

I will double check how I am importing books, and how slugifying works. Colons should not break the universe.
Thank you for the report, will have a look.

Ok. I found one thing that explains the missing classes (though I don't understand how your fix fixed it... )
"break markdown"
What exactly is broken? Are there any errors or warnings in the output?

If there's a colon in the meta data in obsidian it breaks the front
matter.l displaying correctly...
On Sat, 15 Oct 2022, 01:44 Erin Schnabel, @.***> wrote:

Ok. I found one thing that explains the missing classes (though I don't
understand how your fix fixed it... )
"break markdown"
What exactly is broken? Are there any errors or warnings in the output?
—
Reply to this email directly, view it on GitHub
https://github.com/ebullient/json5e-convert-cli/issues/17#issuecomment-1279615293,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABL7VCCMXI2L2WNRR7W7WADWDH46LANCNFSM6AAAAAARD66JHM
.
You are receiving this because you authored the thread.Message ID:
@.***>

ok. Most things in the front matter should already be quoted and/or slugified.. but this is more specific. Thanks

Do you have a custom template, by chance? I can not reproduce this. I don't have any colons in the metadata anywhere.

Hiya, Yes custom template do for example below my custom Spell template
(also have custom monster template as well)
...
spell_name: Alter Self
spell_level: 2nd-level
spell_school: Transmutation
spell_ritual:
spell_cast_time: 1 action
spell_range: Self
spell_components: V, S
spell_duration: Concentration, up to 1 hour
spell_classes: Artificer, Druid (Moon), Sorcerer, Wizard
spell_source: Player's Handbook p. 211
![[Transmutation.png|60]]
=this.spell_name
=this.spell_level, =this.spell_school =this.spell_ritual

Casting time: =this.spell_cast_time
Range: =this.spell_range
Components: =this.spell_components
Duration: =this.spell_duration

So my data is pulled through and pushed to Front Matter - that way you can
also use the Data View plugin to process the info as required.
And then in my actual template below that I reference the data with
`=this.spell_source" for instance. but if a source has a colon in teh title
(like Frostmaiden) then this functionality breaks.
Does that explain it a bit clearer?
C
On Sat, 15 Oct 2022 at 20:00, Erin Schnabel @.***>
wrote:

Do you have a custom template, by chance? I can not reproduce this. I
don't have any colons in the metadata anywhere.
—
Reply to this email directly, view it on GitHub
https://github.com/ebullient/json5e-convert-cli/issues/17#issuecomment-1279809744,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABL7VCGZGRX24A5VGOMPPSLWDL5M5ANCNFSM6AAAAAARD66JHM
.
You are receiving this because you authored the thread.Message ID:
@.***>

AH! This makes more sense! Can you surround the to-be-replaced value with quotes in the YAML metadata?
spell_name: "Alter Self"

Quotes are not always required, but in the face of potential messy characters, they can solve the problem.
Additionally: would you prefer sources as a list you can concatenate yourself, rather than as one pre-made string?

Perfect! Added quotes aroud the data in teh template and now works perfectly, thank you!
spell_source: "{resource.source}"


Opened by accident, please ignore

@vaxerski Would it be possible to delete this?
Sorry to clutter up the issue tracker

github doesn't allow us to do that. It's fine, leave it as is.


KYLIN-4656 Guava classpath conflict caused by kylin-jdbc 3.1.0 jar

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[x] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Pull Request Test Coverage Report for Build 6171

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 28.05%

Totals

Change from base Build 6168:
0.003%

Covered Lines:
26245

Relevant Lines:
93566

💛  - Coveralls

LGTM