reconciler/managed: avoid temporary data loss to managed on annotation update

Description of your changes
When storing critical annotations, the retry loop always dropped changes to the managed resource object, i.e.
part of the reconcile loop's effects on managed were just lost, always. This PR changes this to try one update
of the resource without reset (aka client get call), and only after that fall back into best-effort code path. In effect, this
might save a number of (expensive) reconciles.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Do you think it's worth "E2E" testing this? It would have to be manual at the moment, unfortunately. (i.e. Build a provider using the updated reconciler.)

Do you think it's worth "E2E" testing this?

That would be a tricky e2e. Certainly doable, but on the borderline of a test belonging into e2e.

Actually, we should only retry with a Get on conflict. Let me change that before merging. On any other error we can just keep the old object and retry.

Have extended the unit tests.


version jetson 4.4 v3.7.5  cannot save area map

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, feature request, or anything other than a bug report directly related to this project.

Description
When moving to version 3.7.5. and traying to save map using zed.disable_positional_tracking(self.outMapFile) map is not saved.
Steps to Reproduce

run positional tracking
tracking_params = sl.PositionalTrackingParameters()

tracking_params.area_file_path = inMap
    status = zed.enable_positional_tracking(tracking_params)

No file saved
...
Expected Result
saved map file
Actual Result
Nothing happens.
in version 3.7.4. a map is saved
ZED Camera model
ZED2i
Environment
Jetson Linux jetpack 4.4
ZED SDK 3.7.5 for jetpack 4.4

Anything else?
No response

Hi @shlomi-amitai
The code that you posted is correct to open an existing area map file.
To save the area map you can call the saveAreaMap function or call the disablePositionalTracking function

I'm sorry, pasted the wrong code,
that' what I'm doing
zed.disable_positional_tracking(self.outMapFile)
and yet , it is not saved.
When switching to 3.7.4, it works (but then other issues come up...)

We are going to test this odd behavior.
Can you try to use the saveAreaMap function to test if it works?

tried this as well, still not dumping the file.

Faced the same issue


Rice document update crossreferences

To include bibliographic localities, and to use container references (sections for table/note/example references).

Document in readme that container labels will be included in cross references to formulas and notes consistently; they are not in the Rice document.

I have had to do the extensions requested in https://github.com/riboseinc/isodoc-models/issues/23; will undo if that proposal is rejected.


Inactive IAM Collaborator Deletion Required - Username: jasper.jackson

The Collaborator-Inactivity-Monitoring workflow has detected that the username jasper.jackson is inactive for more than 180 days.
Consult this documentation which describes the process for deleting the collaborator user.

Removed via PR https://github.com/ministryofjustice/modernisation-platform/pull/6936


chore(package): update dev dependencies

Coverage remained the same at 97.872% when pulling caab81ffd30a1df843a143af827f7aaedf94d3a2 on dev-deps into b98db407121be4bcbbc0eb7594ce349394c5095b on master.


type Annotation Action (Github Action).

This PR closes the following issue:
closes #21

@Karim-53 Please review my changes...


[azeventgrid] Trim the Error struct to just be the Code and a .Error() string func

The latest Event Grid drop includes a change which exports an Error in FailedLockToken.
Our policy on this is to trim these error types to only the fields we know are strongly guaranteed to have programmatic data (ie, Code), leaving the other fields as unexported, but "stringized" in Error() string.

Did this back in February. :)


Support for v2 symbols

Addresses #44.
This is still WIP and I will come back to this as soon as I have the time.

This is now ready to get merged.


Implement generating alternatives using metaheuristics

Pull request details
Describe the changes made in this pull request
Implements finding alternative solutions using metaheuristics. Creates an alternative generating problem from a JuMP model that can be solved by an algorithm implemented in Metaheuristics.jl.
List of related issues or pull requests
Closes #2
Collaboration confirmation
As a contributor I confirm

[x] The documentation is up to date with the changes introduced in this Pull Request (or NA)
[ ] Tests are passing
[x] Lint is passing

@abelsiqueira as you can see this pull request fails on Julia 1.6. The reason for this is that the Metaheuristics package I use requires a minimum Julia version of 1.7. Using an earlier version of Metaheuristics.jl gives errors in the current code. What do you think is the best way to resolve this? Should we add a compatibility requirement in the project.toml that julia should be at least 1.7? Then we should also update the continuous integration.

Hi @marnoldus, there are two possibilities:

What you described, change 1.6 to 1.7 in all places it appears.
Use Package extensions to only load the relevant metaheurisitics functions if the Metaheuristics package is loaded (which should be impossible in 1.6 because it's broken) and/or check the version number and act accordingly at package load time.

Solution 1 is simpler and straightforward. I prefer it.
The drawback is that the other packages are requiring 1.6, so there is some incompatibility - expectations are broken.
Solution 2 involves too many moving parts and is hard to maintain. It requires manual intervention when the LTS version of Julia changes, and it will create the false expectation that the package works normally on 1.6, which is not the case.
For me it's a clear choice to drop 1.6, but I'm tagging @clizbe for approval.

Solution 1 sounds better. It just means the rest of Tulipa requires Julia 1.6 or later, while MGA requires Julia 1.7 or later, right?

Yes

Thanks for all the work! I added some comments while reading the code for the first time. Most of them are about clarification from an outsider point of view, Although I can understand most after some googling/testing, but it would be nice if there can be more comments so I don't have to jump between things 😄 .
Not sure if you're still around to follow this up @marnoldus but I tag you both so you are both informed @greg-neustroev.
And there is an issue #22 with my comments for the existing code.

Great! Thanks for this PR.


[rom_e2e] Fix jtag test timing out due to no being able to halt

Please see https://github.com/lowRISC/opentitan/issues/17377

Maybe this PR should also update the test's DV twin: chip_sw_rom_e2e_jtag_debug_vseq.

Closing this PR as the workaround is no longer necessary as the RTL will be fixed as discussed in the issue #17377


Update screenshot

Update screenshots to use iPhone 6 simulator

Is it just the single screenshot in the readme? I didn't see any others in the project, but just wanted to make sure it was the one

Yes, just the single screenshot.
Yes, you can make these other changes too.


Port status should be 'closed'

A port scan reports closed ports as 'close', rather than 'closed'

bump version 1.0.1 for that
thank you for feedback.
cheers


Show workspace owner in workspace list

@kylecarbs are you still working on this or can I take it on?


Analytics kartik

Checking in my changes on to the main ga-analytics branch.

Please make sure to keep up with the develop branch and merge changes in there into this branch to make sure you're working with the latest code

Please make sure to keep up with the develop branch and merge changes in there into this branch to make sure you're working with the latest code


[Draft] Next credential collision with AD credentials

User Story
Log in with local creds that match AD creds and you get extra powers!

T-shirt size: S(mall), M(edium), L(arge)

User Story
Currently with the ability to log in with either NEXT creds or corporate AD creds, it seems that I can create a NExt acct that matches my AD acct and get all the privileges tied to my AD creds.  While removing the UI switch to allow me to pick NEXT/AD will hide this issue, we really should be more robust about rejecting this on login.

Description
Possible escalation of privilege hack here...

Acceptance Criteria/Tests
May be tricky to demo this one after we disable radio button on login page -- need to show that a next acct with a different PW than my AD acct doesn't get my AD level of access.

appears to be duplicated in #1331 now...  let's close this one since that one is in current sprint.


create spdxid enum

Overview
This PR creates an SPDX id enum using enumeratum and replaces the SPDX license checker library we were using. This was necessary because the list of valid SPDX identifiers changes over time, and drift from the list in our library, which hasn't been updated in five years, meant that we were missing some ids, which is important when we reject all values that we don't know how to parse.
Checklist

[x] New tests have been added or existing tests have been modified
[x] Changelog updated

Testing

confirm the instructions at the top of SpdxId make sense

Closes #164

Nice update here -- I didn't really think about the list of valid SPDX ids changing much... I think your change is a better solution than refinement in this case for the reasons you mentioned.


ERROR: Couldn't connect to Docker daemon at http+docker://localhost - is it running?  If it's at a non-standard location, specify the URL with the DOCKER_HOST environment variable.

** PLEASE ONLY USE THIS ISSUE TRACKER TO SUBMIT ISSUES WITH THE EXAMPLE VOTING APP **

If you have a bug working with Docker itself, not related to these labs, please file the bug on the Docker repo *
If you would like general support figuring out how to do something with Docker, please use the Docker Slack channel. If you're not on that channel, sign up for the Docker Community and you'll get an invite. *
Or go to the Docker Forums *

Please provide the following information so we can assess the issue you're having
Description
run docker-compose up,then error shows:
ERROR: Couldn't connect to Docker daemon at http+docker://localhost - is it running?

If it's at a non-standard location, specify the URL with the DOCKER_HOST environment variable.

Steps to reproduce the issue, if relevant:
1.
2.
3.

We're sorry for the late response. Linux images have been refreshed, cleaned up, and are now multi-platform. If you're still having issues with the latest images, feel free to reopen this ticket.


Can you add coordinate change?

This is a really handful extension in VSCode, thanks!
However, is it possible for you to add coordinate system change as a feature?
Some pcd files use a different XYZ system and with current viewer we can only rotate in a fixed XYZ coordinate and XY grid.

Thanks for opening this issue!
I plan to add that feature, but it may take some time to implement.

I add rotation and reverse features. The features are as follows.

pcviewer.reverseCoordinate: This feature flips objects(point cloud and axis helper) by changing values to true. This array order is [x, y. z].
pcviewer.rotateCoordinate: This feature rotates objects by modifying rotation angles(degrees). This array order is [x, y, z].

original data

I writed "pcviewer.rotateCoordinate": [-90,0,0] into setting.json.


Boost 1.58.0 soon with updated and defined support for ARM/iOS (32+64 bit)

As defined in the BOOST WIP spec.
http://www.boost.org/users/history/version_1_58_0.html

Updated Libraries
Context:
support for ARM/iOS (32+64 bit)
class execution_context (C++11)

This is great news and should fix a lot of the smaller errors and compiler issues we have found.

ofxiOSBoost updated for 1.58.0


fix: minor grammar issues

Fix minor grammar issues on file packages/docs/api/interfaces/pinia_testing.TestingOptions.md

Of course! but, which branch/repo is the source code? 😅

same branch: v2, packages/pinia/src

Thanks for your guidance 😄


Bug: engine with custom payload header

Addressing #304

Thanks for the fix!

@PatGod we need engine test coverage for this, we can address this later.

In reply to: 904856504


[Main page] In the first application load page does not load completely

Environment: macOS Big Sur 11.5.1; Chrome version 99.0.4844.51.
Reproducible: always.
Build found: gcp-0.0.94
Preconditions
Pozashkillya page is closed
Steps to reproduce

Open the application
Refresh the page

Actual result
Page loaded completely only after refresh.
https://user-images.githubusercontent.com/97882565/179743316-1f0666a3-8288-43cc-a2ec-821f5e9de9e4.mov
Expected result
Page should load completely always.
Labels to be added
"Bug", Priority ("pri: high"), Severity ("severity: Major"), Type ("UI, "Functional")

fixed

This bug is not reproduced.


Revert "Build with amd64 compatible image"

Reverts base-org/web#925

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


collection .kind, .apiVersion inconsistent between return forms

Splitting general .get_plural interface problems from #307.  #307 will be only about wrong get_endpoints.kind == "Endpoint" and get_security_context_constraints.kind == "SecurityContextConstraint".

Getting a single entity get_foo gives us kind, apiVersion on the resource itself, in all result formats. :+1:
Listing multiple get_foos is inconsistent.

as: :parsed, as: :parsed_symbolized modes give what k8s gives: kind == "FooList" and apiVersion set on the top level, no kind nor apiVersion inside individual resources in items array.
as: :ros default mode historically tries to help by setting kind == "Foo" (without "List"!).  However no apiVersion at all :-1:!  And again no kind nor apiVersion inside individual resources.

I think kubeclient dropping "List" on top level is now harmful, given the inconsistency between modes, but I'm not sure it's worth breaking compatibility.
I think k8s API behavior of setting kind "FooList" on array but omitting kind "Foo" on individual items is unfriendly, it's just side effect of Go's static typing.
Proposal
I want get_foo to have same shape as get_foos[3].  I want individual items from get_foos to be later directly usable in create/update (as well as exporting to yaml).

[ ] In all modes except :raw, get_foos shall copy apiVersion into each item inside the array.
[ ] In all modes except :raw, get_foos shall set kind in each item inside the array to the individual kind "Foo".
The top-level behavior will remain.

Are there downsides?
What if someone wants to export get_foos(as: :parsed) as a whole that can be fed to kubectl, the extrenaous kind and apiVersion will cause problems?
Turns out kubectl get foos -o yaml behaves differently from k8s API: it sets a generic kind: List at top level instead of kind: FooList and adds kind and apiVerison inside each element.  And kubectl create accepts such list without complaints!  And with kind: FooList at top level, it still accepts those.  (In fact kubectl 1.10 accepts any other list kind at top level!  And oc 3.9 even a non-list kind: Pod at top level. That's probably 1.10 having more validation and not oc/kubectl difference.)
So seems the proposal is in line with the friendlier behavior kubectl already adds on top of the API.

ref equivalent python lib issue https://github.com/kubernetes-client/python/issues/745.

[ ] the list kind is not guaranteed to be singular kind + "List" :frowning_face:
relation between the two not available in discovery info we now get, in principle can be deduced from openapi.  not worth it :-1:
but when we make request we already know version & kind, so can just use that!?

[ ] what about v1beta1 -> v1 and similar normalizations  Does API always returns same version that we requested?

These points are all minor, and imperfect fix would still be help 99% cases.


Any chance to display an image?

Like popup quick view window for an image in memory, rather than saving it to disk before programmatically open it via "Preview" app. Is that possible now? Or would you consider implementing this feature?

You can do this by using StringIO for the file contents. I do this to generate QR Codes that pop-up in preview. I've made a PR to add this functionality #89

I took a look at your PR, and it seems to be previewing an image on disk. I was initially thinking about directly previewing an image in memory, like by transforming an BytesIO content into an NSImage instance or something.

Oh, I see. If you submit a Pull Request with the changes I'd be happy to review it.

Actually that's the problem: I cannot show the image.
First I tried your code in #89, and got this error:
2018-12-01 11:43:28.040 Python[42369:18111772] Traceback (most recent call last):
  File "/Users/joker/Works/rumps/rumps/rumps.py", line 1035, in callback_
    return _call_as_function_or_method(callback, self)
  File "/Users/joker/Works/rumps/rumps/rumps.py", line 384, in _call_as_function_or_method
    return method(event)
  File "simple.py", line 20, in sayhi
    rumps.show_image('/Users/joker/Downloads/256.png')
  File "/Users/joker/Works/rumps/rumps/rumps.py", line 212, in show_image
    return _nsimage_window_from_file(image)
  File "/Users/joker/Works/rumps/rumps/rumps.py", line 277, in _nsimage_window_from_file
    window = NSImageView.alloc().init(image)
TypeError: Need 0 arguments, got 1

I worked around by using this instead of NSImageView.alloc().init(image):
window = NSImageView.alloc().init()
window.setImage_(image)

This time I got no error, but the image did not show up either.
Now that I know how to converting raw bytes into an NSImage instance, it would be nice if I could be able to test it.
data = NSData.dataWithBytes_length_(raw_bytes, len(raw_bytes))
image = NSImage.alloc().initWithData_(data)


Can not find function resize2D_as

https://github.com/TiagoCortinhal/SalsaNext/blob/f67cc27ece3b37b8b8e3f305d813e5a0faeef4d1/train/tasks/semantic/modules/adf.py#L379-L380

Hey @EthanZhangYi thanks for pointing it out.
I added the missing function.


Lambda

I'm trying to use a lambda and am having trouble compiling it, is there any way to use lambda'so yet?
I want to convert https://github.com/carsonwright/spark-test to mirah

Is mirah active?

Yes. It is active. We did a release in November, and have been working on
nailing down things for another release since then.
Unfortunately, static importing single methods is unsupported right now. I
was able to get a hello world working by using your pom and doing the
following.
I created a
$ cat src/hello.mirah`

import static spark.Spark.*

get("/hello") do |req, res|
  "Hello World"
end

Then I created a file containing the dependency classpath, and ran mirah
with that classpath.
$ mvn -Dmdep.outputFile=myClasspath dependency:build-classpath

$ mirah -cp `cat myClasspath` src/hello.mirah
[Thread-4] INFO spark.webserver.SparkServer - == Spark has ignited ...
[Thread-4] INFO spark.webserver.SparkServer - >> Listening on <IP_ADDRESS>:4567
[Thread-4] INFO org.eclipse.jetty.server.Server - jetty-9.0.2.v20130417
[Thread-4] INFO org.eclipse.jetty.server.ServerConnector - Started
ServerConnector@6eb459e6{HTTP/1.1}{<IP_ADDRESS>:4567}

There's a patch for the maven plugin to bump the mirahc version to one that
supports java 8. Until that's merged / published, you can't use maven to
build and run directly. If you want to try the patched maven plugin, you
could try installing https://github.com/mirah/maven-mirah-plugin/pull/8
On Wed, Mar 18, 2015 at 7:45 AM, Carson Wright<EMAIL_ADDRESS>wrote:

Is mirah active?
—
Reply to this email directly or view it on GitHub
https://github.com/mirah/mirah/issues/287#issuecomment-82978062.

--
-Nick Howard
http://blog.baroquebobcat.com/

Thank you very much, this project looks amazing, I'm really excited about watching what ya'll are doing.


Rewrite bootimage to make bootloader a "normal" dependency

This changes the way the bootloader dependency is specified in a breaking way. The new version is already released as 0.5.0-beta. Please report any problems you find and let me know what you think!
Migration guide
The plan is to link this guide from the error messages.
How it worked in version 0.4
In 0.4, the bootloader dependency was defined in the following way:
# in your Cargo.toml

[package.metadata.bootimage.bootloader]
name = "bootloader"                 # The bootloader crate name
precompiled = false                 # Whether the bootloader crate is precompiled
version = ""                        # The bootloader version that should be used
git = ""                            # Use the bootloader from this git repository
branch = ""                         # The git branch to use (defaults to master)
path = ""                           # Use the bootloader from this local path
target = "x86_64-bootloader.json"   # Target triple for compiling the bootloader

When the package.metadata.bootimage.bootloader table was missing completely, it defaulted to:
[package.metadata.bootimage.bootloader]
name = "bootloader_precompiled"
precompiled = true

The New Way in 0.5
Instead of specifying the bootloader dependency in a package.metadata section and letting bootimage interpret it, we just specify the bootloader as a normal cargo dependency:
# in your Cargo.toml

[dependencies]
bootloader = "0.2.0-alpha"

The bootloader configuration table has shrunk to:
[package.metadata.bootimage.bootloader]
name = "bootloader"                 # The crate name of the bootloader dependency
target = "x86_64-bootloader.json"   # Target triple for compiling the bootloader

TODO

[x] Adjust the help and error messages
[x] Add support for precompiled bootloaders (e.g. via a cargo feature on the bootloader)

Alternatively: Make the bootloader build with LLD (maybe sometime in the future)

[x] Add a lib.rs to the bootloader, that re-exports the BootInformation type of the os_bootinfo crate (done in https://github.com/rust-osdev/bootloader/pull/20)

This way we solve the version mismatch problem (that the kernel uses a different bootinfo version than the bootloader).

To test the new bootimage in combination with the new bootloader:
> cargo install bootimage --version "^0.5.0-beta" --force

In your Cargo.toml add either:
[dependencies]
bootloader = "0.2.0-beta"    # only works on linux

or the precompiled variant:
[dependencies]
bootloader_precompiled = "0.2.0-beta"

Replace your _start function with the following:
#[macro_use]
extern crate bootloader;

#[cfg(not(test))]
entry_point!(kernel_main);

fn kernel_main(boot_info: &'static bootloader::bootinfo::BootInfo) -> ! {…}

Try bootimage run, bootimage test, cargo test, etc. and report your findings.

Published as version 0.5

@phil-opp
bootimage run, bootimage test, cargo test.
Everything is OK. But I don't know what has been changed.
[dependencies]
volatile = "0.2"
spin = "0.4"
uart_16550 = "0.1.0"
x86_64 = "0.2"
bootloader = "0.2" # only works on linux
bootloader_precompiled = "0.2"

@yjhmelody The problem is that the bootloader does not link with the LLD linker, because it doesn't support some relocation that is generated for the 16 bit code. So we're currently use the linux system linker for linking, which means that it does not work on Windows and macOS.
This is the reason that we have the bootloader_precompiled. Like the name says, it's already compiled so it should work on all OSes. When you're using this version, you don't need the bootloader dependency. So try to remove it:
[dependencies]
volatile = "0.2"
spin = "0.4"
uart_16550 = "0.1.0"
x86_64 = "0.2"
bootloader_precompiled = "0.2"


Fixes apply debug console font size setting to input field

This PR fixes #90714

@jeanp413 great PR as usual, thank you very much for creating this.
I have left a comment in the code changes. Also I tried this out and seems to work quite well, noticed one issue though: for larger font sizes the input chevron is not focused. We probably hard code the size somewhere, and the font change breaks this. Can you maybe look into improving the centering of the chevron on the left?

@jeanp413 your comment makes sense. Once we resolve the centering of the chevron (if possible) I would be open to merge in this PR.

Pushed some changes to center de input chevron, let me know if the css is ok

@jeanp413 Yeah, css looks great. Let's merge this.
Thanks a lot 🍻 ☀


chore(release): 1.148.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 853fc48f986390adcc5694d47bc51cb9f693b5ed
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 853fc48f986390adcc5694d47bc51cb9f693b5ed
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: fbffeb94b0cb003c1a1787a3f65d2dda61261bfe
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: fbffeb94b0cb003c1a1787a3f65d2dda61261bfe
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[Bug]: Git app, commit number shows up as 1, while the modal shows 3 commits

Is there an existing issue for this?

[X] I have searched the existing issues

Description
On a branch, I make application forkable [Share and embed], I change Theme, I create a new page and add a few widgets. and then refresh the page. Now I see that the count on the plus sign shows up as 1, but on invoking the modal, the number of changes show up as 3.
Steps To Reproduce
https://github.com/appsmithorg/appsmith/assets/101863839/9d783dcf-9f77-4737-aa25-578faa72302c
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Release

Application url
Shared with @nayan-rafiq  and @brayn003


ROCM

Hi! Is there a way to install sglang in rocm devices?

Hi @BasDiaz thank you for your interest. Currently, only NVIDIA GPU sm80+ is supported.

Hi @BasDiaz thank you for your interest. Currently, only NVIDIA GPU sm80+ is supported.

Hi, Does AMD GPU support now? ,I think there are quite a few ROCM-based questions in the issue


Stop supporting debug.logToDebugConsole command

Currently we support a 'debug.logToDebugConsole' command.
https://github.com/Microsoft/vscode/blob/a38db8e0a5ca9b651fd0484faefb17d706400cac/src/vs/workbench/parts/debug/electron-browser/debugCommands.ts#L25
Instead we should introduce API and deprecate this command.

Removing this only makes sense in november. First in october we have to create a solution for this using the debug api.

I will do this on October 23 ;-)

I can also look into this, just wrote down a note for myself :)

I've created an API request for this.

Since the API will be proposed in November, we cannot delete this in November.

@weinand should I create and assign this to the December milestone?

Let's wait until we know more about Xmas...

It seems to only support string message, but I need to display an object to be able to navigate properties as we do when using "Debugger evaluation" in vscode.
I didn't find a way to do this from an extension. Is it possible ?

@Alphapage That is only possible from a debug adapter. As an example, you can checkout node debug 2 and find when he sends the variableReference in the OutputEvent if that is what you are looking for.

@Alphapage expandable objects need an active debug session behind them (so that the UI can request more data on expand). When outputting an object from an extension, then there is not necessarily a debug session active.
So constraining the API to string is intentional.

As you told, I think I was going to the wrong direction: I just need to add an UI panel to the extension to be able to navigate the object.
Thank you very much.


FATAL: Error, call to 'patchelf' failed (PySide6 + QML)

Nuitka version, full Python version, flavor, OS, etc. as output by this
command (it does more than you think, and we are adding more all the time):

<IP_ADDRESS>
Commercial: None
Python: 3.10.1 (main, Jan  4 2022, 05:20:14) [GCC 9.3.0]
Flavor: Unknown
Executable: /home/am/prm/python/nuitka-issue/.venv/bin/python
OS: Linux
Arch: x86_64
Distribution: Ubuntu 20.04.3

How did you install Nuitka and Python

pip install nuitka

The specific PyPI names and versions

Nuitka==<IP_ADDRESS>
PySide6==6.2.3
shiboken6==6.2.3

This is the basic hello world of QML from two files main.py and main.qml.
Build command fails with a fatal error when calling patchelf

# main.py
from PySide6.QtGui import QGuiApplication
from PySide6.QtQml import QQmlApplicationEngine

if __name__ == '__main__':
    app = QGuiApplication()
    engine = QQmlApplicationEngine()
    engine.load('main.qml')
    app.exec()

// main.qml
import QtQuick
import QtQuick.Window

Window {
    visible: true

    Rectangle {
        width: 200
        height: 200

        Text {
            text: "Hello World"
        }
    }
}

Build command and output
 python -m nuitka --standalone --plugin-enable=pyside6 --include-qt-plugins=qml main.py

(.venv) am@lp:nuitka-issue$ python -m nuitka --standalone --plugin-enable=pyside6 --include-qt-plugins=qml main.py
Nuitka-Options:INFO: Used command line options: --standalone --plugin-enable=pyside6 --include-qt-plugins=qml main.py
Nuitka-Options:INFO: Detected static libpython to exist, consider '--static-libpython=yes' for better performance, but errors may happen.
Nuitka:INFO: Starting Python compilation with Nuitka '<IP_ADDRESS>' on Python '3.10' commercial None.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'PySide6.QtQml' added 'shiboken6,shiboken6.Shiboken,os,enum,PySide6.QtCore,PySide6.QtNetwork'.
Nuitka-Plugins:INFO: pyside6: Injecting post-module load code for module 'PySide6.QtCore':
Nuitka-Plugins:INFO: pyside6:     Setting Qt library path to distribution folder. We need to avoid loading target
Nuitka-Plugins:INFO: pyside6:     system Qt plugins, which may be from another Qt version.
Nuitka-Plugins:INFO: pyside6: Implicit dependencies of module 'PySide6.QtQml' added 'PySide6.QtCore,PySide6.QtNetwork,PySide6.QtQuick'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'PySide6.QtQuick' added 'shiboken6,shiboken6.Shiboken,enum,PySide6.QtCore,PySide6.QtGui,PySide6.QtOpenGL,PySide6.QtQml'.
Nuitka-Plugins:INFO: pyside6: Implicit dependencies of module 'PySide6.QtQuick' added 'PySide6.QtCore,PySide6.QtNetwork,PySide6.QtGui,PySide6.QtQml'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'PySide6.QtOpenGL' added 'shiboken6,shiboken6.Shiboken,enum,PySide6.QtCore,PySide6.QtGui'.
Nuitka-Plugins:INFO: pyside6: Implicit dependencies of module 'PySide6.QtOpenGL' added 'PySide6.QtCore,PySide6.QtGui,PySide6.QtWidgets'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'PySide6.QtWidgets' added 'shiboken6,shiboken6.Shiboken,enum,PySide6.QtCore,PySide6.QtGui'.
Nuitka-Plugins:INFO: pyside6: Implicit dependencies of module 'PySide6.QtWidgets' added 'PySide6.QtGui'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'PySide6.QtNetwork' added 'shiboken6,shiboken6.Shiboken,enum,PySide6.QtCore'.
Nuitka-Plugins:INFO: pyside6: Implicit dependencies of module 'PySide6.QtNetwork' added 'PySide6.QtCore'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'PySide6.QtCore' added 'shiboken6,shiboken6.Shiboken,os,enum'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'PySide6.QtGui' added 'shiboken6,shiboken6.Shiboken,os,enum,PySide6.QtCore'.
Nuitka-Plugins:INFO: pyside6: Implicit dependencies of module 'PySide6.QtGui' added 'PySide6.QtCore'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'socket' added '_socket'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'curses' added '_curses'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'ctypes' added '_ctypes'.
Nuitka:INFO: Completed Python level compilation and optimization.
Nuitka:INFO: Generating source code for C backend compiler.
Nuitka:INFO: Running data composer tool for optimal constant value handling.
Nuitka:INFO: Running C compilation via Scons.
Nuitka-Scons:INFO: Backend C compiler: gcc (gcc).
Nuitka-Scons:INFO: Slow C compilation detected, used 60s so far, this might indicate scalability problems.
Nuitka-Scons:INFO: Running gcc -o /home/am/prm/python/nuitka-issue/main.dist/main -z noexecstack -export-dynamic -s -Wl,-R,'$ORIGIN' -Wl,--disable-new-dtags<EMAIL_ADDRESS>-L/opt/Python-3.10.1/lib -ldl -lm -lpython3.10 -ldl -lpthread -lutil -lrt -lm took 77.05 seconds
Nuitka-Scons:WARNING: You are not using ccache.
Nuitka-Plugins:INFO: pyside6: Including Qt plugins 'qml' below 'PySide6/qt-plugins'.
FATAL: Error, call to 'patchelf' failed: ['/usr/bin/patchelf', '--set-rpath', '$ORIGIN/../../../../..', 'main.dist/PySide6/qml/Qt/labs/animation/labsanimationplugin.debug'] -> b"patchelf: patchelf.cc:1023: void setSubstr(std::string&, unsigned int, const string&): Assertion `pos + t.size() <= s.size()' failed.".

You might want to research what is wrong with patchelf from Ubuntu here. I wonder if it's a length issue, or if it's something to do with that .debug file not being real.

That said, you don't want to build standalone on the newest system on Linux, but the oldest you want to support. The resulting binary is going to run on 20.04 or higher, maybe not what you want.

Probably related to https://github.com/NixOS/patchelf/issues/207.
I updated the patchelf version (0.10 -> 0.14.3) and the error is gone. Thanks!

Yup, I would be closing this as invalid, because it's not a Nuitka bug really. But there are two things:
a) Nuitka needs to make you aware that standalone on Linux built on the system you build on, will have limits.
b) We can definitely just give both chrpath and patchelf a go. Nuitka has had chrpath usage before, and
could easily just restore that and try both, one working will be good enough.

I am choosing to give up on the chrpath fallback after reading its man page again. It's just not going to work as reliable as I want, while having test coverage is made harder.

Having this issue on
1.2.6
Commercial: None
Python: 3.10.6 (main, Nov 14 2022, 16:10:14) [GCC 11.3.0]
Flavor: Debian Python
Executable: /home/nir/.cache/pypoetry/virtualenvs/tzv5hob-ui-5N8t_BVY-py3.10/bin/python
OS: Linux
Arch: x86_64
Distribution: Ubuntu (based on Debian) 22.04.1

deps:
attrs              22.1.0 Classes Without Boilerplate
packaging          22.0   Core utilities for Python packages
pyside6            6.4.1  Python bindings for the Qt cross-platform application and UI framework
pyside6-addons     6.4.1  Python bindings for the Qt cross-platform application and UI framework (Addons)
pyside6-essentials 6.4.1  Python bindings for the Qt cross-platform application and UI framework (Essentials)
qtpy               2.3.0  Provides an abstraction layer on top of the various Qt bindings (PyQt5/6 and PySide2/6).
shiboken6          6.4.1  Python/C++ bindings helper module

@nrbnlulu posting below closed issues is a sure-fire way to get ignored, please open a new issue


DS-4271: Replaced brackets with double quotes in SolrServiceImpl.

DSpace 6 versions of the fix provided in:
https://jira.duraspace.org/browse/DS-4271
Fixes #7611
Quite a small fix, but it has been open for a little while now, so took the liberty to create a quick PR
(More information is present in the JIRA ticket itself)

Agree this is a good fix for a problem I didn't even realize we had. Tested with a contains filter with spaces on a subject term in our staging repository.
Before, 293 results, of which many are not what the user would expect:

After, 162 results, of which all are exactly matching the search term:

Looks good to me for DSpace 6.4. Merge!

I haven't tested this but it's got +3 @kshepherd

@J4bbi I think this is a good one for 6.4. It's at +3 for some months now.

Ported to main for DSpace 7.x here: https://github.com/DSpace/DSpace/pull/8284


Commit body in COMMIT_OVERRIDE

The readme shows this, which works as expected:
BEGIN_COMMIT_OVERRIDE
feat: add ability to override merged commit message
fix: another message
chore: a third message
END_COMMIT_OVERRIDE

I would think the library would also accept a description of the change, but it's not taking. I've tried with various new line placements to no avail.
BEGIN_COMMIT_OVERRIDE
feat: add ability to override merged commit message
this is a description of the feat
chore: a third message
this is a description of the chore
END_COMMIT_OVERRIDE

Is it currently not possible to override and/or set the description in a comment override?
Thanks for the help!

Is the extended message meant to show up somewhere in the release notes / changelog? I used your example above (for testing) in the body of https://github.com/catppuccin/github-file-explorer-icons/pull/118, and then re-ran my Release Please workflow (which successfully completed and logs showed no issues). However, the release PR doesn't contain anything about my "Some extra note." extended description


Main content refactor (figures)

This doesnt run for me, if you do a docker-compose up in the client it complains that:
./src/components/atoms/descriptors/descriptors.module.scss:16:1
Syntax error: Selector "img" is not pure (pure selectors must contain at least one local class or id)

  14 | }
  15 |
> 16 | img {
     |     ^
  17 |   max-height: 100%;
  18 |   max-width: 100%;

This passed all linting and it should accept this

Sorry Dave, I'm still getting the issue @will-byrne mentioned

I'll take it out but not liking that we are being told we cannot have this as it is more of an opinion.
There will be other selectors that will need to be put in after Thursday so will need to allow them in the stylesheets

Same deal with bare figure

For reference, this isn't linting failures. The style can't be built for production - it's a hard failure. And it's a failure because the selector is part of a component, but isn't scoped to the component. Like a selector lacking BEM-style selector in a sass file alongside a component, just enforced by the build-stage because it has no name to attach it to.


Always --pull when building Dockerfile

Because the oct image is tagged as latest, we should always --pull our base images:
https://docs.docker.com/engine/reference/commandline/pull/

I wonder if the trade-off we are making here is worth it. While this solution makes the image building process more "robust" is introducing a big inefficiency due to the fact that there are three FROM instructions in the Dockerfile, meaning that those three images will have to be pulled every time the image is rebuilt. Then, I also see that the --no-cache option is being used, making the process even slower.
The OCT image does not change much, so before introducing this change I'd reconsider if it's really necessary.

All CI tests function without the use of a cache on a fresh operating system installation. I wouldn't be worried about performance on a local machine.

I first noticed this on my QE runner machines.  We were using an oct image that was 6 weeks old (roughly the same time I started up the qe runners for the first time).

All CI tests function without the use of a cache on a fresh operating system installation. I wouldn't be worried about performance on a local machine.

Yes, the CI workflow starts from scratch, so it will take the same amount of time because it has to pull the images anyway. I was thinking about a local development cycle where you modify one line and have to pull 3 images and go through all the stages in the Dockerfile in order to compile the code. And then you want to do it again...
But it's ok, one can always call docker build directly and leave those Makefile tasks for the CI.


Option to suppress notification

There should be an option to disable the "Give a Start on GitHub" notification each time one starts VS Code 🙂

Grüße aus Ravensburg 🥳

The notification is only appearing once every 50 startups. It is not intended to get that popup every time. This should also be the case for you. I can't explain why this is coming up every time on your setup.
Anyways I disabled the notification completely with the recent update after you clicked the "Give a Start" button once.
Liebe Grüße zurrück aus Konstanz 😊


max int64 and uint64 handling

Problem statement
Max int64 and uint64 seems to be wrong even if correct in the json schema definition
go test -race ./...
# github.com/opencontainers/image-spec/specsgo
specsgo/int64_validator.go:17: constant -9223372036854776000 overflows int64
specsgo/int64_validator.go:21: constant<PHONE_NUMBER>854776000 overflows int64
specsgo/uint64_validator.go:21: constant<PHONE_NUMBER>3709552000 overflows int64
?       github.com/opencontainers/image-spec/cmd/oci-image-tool [no test files]
?       github.com/opencontainers/image-spec/image  [no test files]
ok      github.com/opencontainers/image-spec/schema 1.139s
Makefile:91: recipe for target 'test' failed
make: *** [test] Error 2

The schema def is:
"int64": {
 20       "type": "integer",
 21       "minimum": -9223372036854775808,
 22       "maximum":<PHONE_NUMBER>854775807
 23     },

and
 39     "uint64": {
 40       "type": "integer",
 41       "minimum": 0,
 42       "maximum":<PHONE_NUMBER>3709551615
 43     },

Swagger specification
Steps to reproduce

this is a tricky one because json only does floats and the spec structs are modeled as float64 which makes it to into a lossy scientific notation. This should use a json.Number construct but that is going to break other things.
https://github.com/go-openapi/spec/blob/master/schema.go#L586-L588
great catch though

What should we do with this one @casualjim ?

not a priority to fix to me. people can use format: uint64 where we're not bound by json's limitations on size

Since swagger validate inspects these values, I think an error or warning should be provided at the spec valdiation stage. @todo: need testing to figure out if it is not already the case.


Deployment template not working with bing-search failure

Describe the bug
I tried two ways to deploy the app but got similar errors.

Manual deployment: I got a warning Warning BCP081: Resource type "Microsoft.Bing/accounts@2020-06-10" does not have types available.. The script run a long time then failed.
Using the deployment template: One deployment task failed: function-copichat-7tz57cfyqmyry-websearcher-plugin/default.

I suspect it's related to the bing-search component. Not sure if it's the same problem as this issue: #491
Thanks.
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/microsoft/chat-copilot/blob/main/scripts/deploy/README.md
Click on Deploy to Azure
Fill the fields and deploy
See error

Expected behavior
The deployment should complete without errors.
Screenshots
If applicable, add screenshots to help explain your problem.

Platform

OS: [e.g. Windows, Mac]
IDE: [e.g. Visual Studio, VS Code]
Language: [e.g. C#, Python]
Source: [e.g. NuGet package version 0.1.0, pip package version 0.1.0, main branch of repository]

Additional context
Add any other context about the problem here.

Same issue as I'm experiencing. "Warning BCP081: Resource type "Microsoft.Bing/accounts@2020-06-10" does not have types available." is also what I get

@TaoChenOSU Thanks. The version is Bicep CLI version 0.22.6 (d62b94db31)

I guess Bicep doesn't support Bing search? I saw a warning as follows:

Adding to this:
Bicep CLI version 0.21.1  > upgraded to > Bicep CLI v0.22.6 Did not result in a different outcome
Warning BCP081: Resource type "Microsoft.Bing/accounts@2020-06-10" does not have types available.
{"status":"Failed","error":{"code":"DeploymentFailed","target":"/subscriptions/[REDACTED]/resourceGroups/rg-[REDACTED]/providers/Microsoft.Resources/deployments/[REDACTED]","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":[{"code":"BadRequest","target":"/subscriptions/[REDACTED]/resourceGroups/rg-[REDACTED]/providers/Microsoft.Resources/deployments/[REDACTED]","message":"{\r\n  \"Code\": \"BadRequest\",\r\n  \"Message\": \"Encountered an error (ServiceUnavailable) from host runtime.\",\r\n  \"Target\": null,\r\n  \"Details\": [\r\n    {\r\n      \"Message\": \"Encountered an error (ServiceUnavailable) from host runtime.\"\r\n    },\r\n    {\r\n      \"Code\": \"BadRequest\"\r\n    },\r\n    {\r\n      \"ErrorEntity\": {\r\n        \"Code\": \"BadRequest\",\r\n        \"Message\": \"Encountered an error (ServiceUnavailable) from host runtime.\"\r\n      }\r\n    }\r\n  ],\r\n  \"Innererror\": null\r\n}"}]}}

This is likely due to a race condition in getting the function host key @glahaye

Or a race condition caused by a clash of App Insights resources and config of the Bing Search Plugin Host.
I have a potential fix if my theory is correct.  Testing it right now.

The Web Searcher plugin has become optional (default to NOT deploy) when you deploy using the deploy-azure scripts. PR: https://github.com/microsoft/chat-copilot/pull/517.
Closing as the PR will address this issue. Feel free to reopen if you have further questions.

Thanks @TaoChenOSU . But I got another error:

{"status":"Failed","error":{"code":"DeploymentFailed","target":"/subscriptions/76760d52-2d57-4b56-9ba6-f0f5044f60a6/resourceGroups/rg-chat-copilot/providers/Microsoft.Resources/deployments/chat-copilot","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":[{"code":"ResourceDeploymentFailure","target":"/subscriptions/76760d52-2d57-4b56-9ba6-f0f5044f60a6/resourceGroups/rg-chat-copilot/providers/Microsoft.Web/sites/app-copichat-7tz57cfyqmyry-memorypipeline/extensions/MSDeploy","message":"The resource write operation failed to complete successfully, because it reached terminal provisioning state 'failed'.","details":[{"code":"Failed","message":"Deployment was interrupted and the process running it was terminated unexpectedly, if an ARM template is used, the likely cause is a race condition. To solve the race condition, make all resources that could potentially restart the site (app settings or site config changes for example) have a dependency on MSDeploy resource. For more info please read the information here: http://go.microsoft.com/fwlink/?LinkId=808141"}]}]}}

Not sure if I need to create a new issue? Thanks.

The failed one is app-copichat-7tz57cfyqmyry-memorypipeline/MSDeploy.

ERROR TYPE
Deployment was interrupted and the process running it was terminated unexpectedly, if an ARM template is used, the likely cause is a race condition. To solve the race condition, make all resources that could potentially restart the site (app settings or site config changes for example) have a dependency on MSDeploy resource. For more info please read the information here: http://go.microsoft.com/fwlink/?LinkId=808141 (Code: Failed)

@yanxiaodi We are aware of this issue and hope to have a fix today.
In the meantime, you can work around the problem by simply deploying again (without deleting the resources of the failed deployment).


[Bug Report] Argmax invalid result with padding

Describe the bug
A clear and concise description of what the bug is.
When there is padding, the result of argmax is incorrect.
To Reproduce
Steps to reproduce the behavior:

Copy below code

# SPDX-FileCopyrightText: © 2023 Tenstorrent Inc.

# SPDX-License-Identifier: Apache-2.0

import torch
import pytest
import tt_lib
from loguru import logger
from tests.tt_eager.python_api_testing.sweep_tests import comparison_funcs

@pytest.mark.parametrize(
    "input_shapes",
    (
        (torch.Size([1, 1, 1, 10])),
    ),
)
@pytest.mark.parametrize(
    "dim",
    (3,),
)
@pytest.mark.parametrize("all", (True,))
class TestArgmax:
    def test_argmax(self, input_shapes, dim, all, device):
        torch.manual_seed(10)

        input_data = torch.tensor([5,3,2,1,10,9,6,7,4,8]).reshape(input_shapes).bfloat16()
        print("input_data", input_data)
        input_tensor = (
            tt_lib.tensor.Tensor(input_data, tt_lib.tensor.DataType.BFLOAT16).pad_to_tile(100).to(tt_lib.tensor.Layout.TILE).to(device)
        )
        tt_output_tensor_on_device = tt_lib.tensor.argmax(input_tensor, dim=dim, all=all)

        tt_out_tensor = tt_output_tensor_on_device.cpu().to(tt_lib.tensor.Layout.ROW_MAJOR).to_torch()
        # tt_out_tensor = tt_output_tensor_on_device.cpu().to(tt_lib.tensor.Layout.ROW_MAJOR)
        if all:
            golden_tensor = torch.argmax(input_data)
            tt_out_tensor = tt_out_tensor[0, 0, 0, 0]
        else:
            golden_tensor = torch.argmax(input_data, dim=dim)
            if dim == 1 or dim == -3 or dim == 0 or dim == -4:
                tt_out_tensor = tt_out_tensor[0]
            else:
                if input_shapes[1] != 1 or input_shapes[0] != 1:
                    tt_out_tensor = tt_out_tensor[0]
                else:
                    tt_out_tensor = tt_out_tensor[0, 0, 0]

        pt_out_tensor = golden_tensor
        comp_pass, comp_out = comparison_funcs.comp_pcc(pt_out_tensor, tt_out_tensor, pcc=0.99)
        comp_all, _ = comparison_funcs.comp_allclose(pt_out_tensor, tt_out_tensor, atol=4, rtol=1e-1)
        logger.info(comp_pass)
        logger.info(comp_all)
        logger.info(comp_out)
        status = comp_pass | comp_all

        print("pt_out_tensor", pt_out_tensor)
        print("tt_out_tensor", tt_out_tensor)
        assert status

Check result

input_data tensor([[[[ 5.,  3.,  2.,  1., 10.,  9.,  6.,  7.,  4.,  8.]]]], dtype=torch.bfloat16)
pt_out_tensor tensor(4)
tt_out_tensor tensor(10., dtype=torch.bfloat16)

Expected behavior
A clear and concise description of what you expected to happen.
It seems that when the value of padding is greater than the value of the tensor, argmax returns the index of the padding value.
Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following environment information:

OS: [e.g. Ubuntu 20.04]
Version of software (eg. commit a059360c5f356621d37963009ccd5a8089a71432)
Arch: wormhole_b0

Additional context
Add any other context about the problem here.

Does this one belong to you @umadevimcw , let me know?

@jliangTT Yes will check it out

@hschoi4448 The problem is with the above logic you have used. Since argmax is to find the maximum index, you should not pad the input with 100. Also, you are sending the Padded input to TT and the unpadded input to Pytorch which is not an apple to apple comparison.
In the above code either use -1.0 or -inf as pad value. If you change that the test is passing
input_tensor = (
            tt_lib.tensor.Tensor(input_data, tt_lib.tensor.DataType.BFLOAT16).pad_to_tile(-1.0).to(tt_lib.tensor.Layout.TILE).to(device)
        )

`---- ttSiliconDevice::init_hugepage: bind_area_to_memory_nodeset() failed (physical_device_id: 0 ch: 0). Hugepage allocation is not on NumaNode matching TT Device. Side-Effect is decreased Device->Host perf (Issue #893).
                  Metal | INFO     | Initializing device 0. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 0 is:   1202 MHz
input_data tensor([[[[ 5.,  3.,  2.,  1., 10.,  9.,  6.,  7.,  4.,  8.]]]],
       dtype=torch.bfloat16)
2024-04-29 07:20:21.422 | INFO     | tests.tt_eager.python_api_testing.sweep_tests.pytests.tt_dnn.test_argmax:test_argmax:52 - True
2024-04-29 07:20:21.422 | INFO     | tests.tt_eager.python_api_testing.sweep_tests.pytests.tt_dnn.test_argmax:test_argmax:53 - True
2024-04-29 07:20:21.422 | INFO     | tests.tt_eager.python_api_testing.sweep_tests.pytests.tt_dnn.test_argmax:test_argmax:54 - Max ATOL Delta: 0.0, Max RTOL Delta: 0.0, PCC: 1.0
pt_out_tensor tensor(4)
tt_out_tensor tensor(4., dtype=torch.bfloat16)
PASSED                  Metal | INFO     | Closing device 0
                  Metal | INFO     | Disabling and clearing program cache on device 0

================================================================= PASSES =================================================================
======================================================== short test summary info =========================================================
PASSED tests/tt_eager/python_api_testing/sweep_tests/pytests/tt_dnn/test_argmax.py::TestArgmax::test_argmax[True-3-input_shapes0]
=========================================================== 1 passed in 8.13s ============================================================
                 Device | INFO     | Closing user mode device drivers`

@umadevimcw
the request for argmax is to be able to work with padded tensors and to ignore the padding, that is argmax only on the unpadded regionof the tensor, ie the orignal shape of the tensor should be used to determine which values of the padded tensor are considered in argmax.
This can be done by argmax internally masking the padded region with -inf, within the CB. This will allow it to work with any padding, even if it's 100 or garbage.

@davorchap May I know which file I have to take a look in GroupNorm?

@davorchap By default we have -inf as padding for the reduce max op. On top of it do you want to mask the padded value and work? Since the user will pad the files, do you have any specific method to differentiate between padded and unpadded shape while processing the Tensor?

@davorchap @hschoi4448 Do you want us to test only for the [1,1,1,10] this kind of tensors? Is it always N, C, H values will always be 1 ?

@davorchap @hschoi4448 Do you want us to test only for the [1,1,1,10] this kind of tensors? Is it always N, C, H values will always be 1 ?

as far as I know, in MNIST, the argmax input tensor size is (BatchSize, 10). Since we cannot accept a 2D tensor as input, I am planning to use a tensor of size (1, 1, BatchSize, 10).

@hschoi4448 #8202 This PR holds the masking of user pad value implementation. I have added test files also.
when the shapes are 1,1,10,10 -> after padding 1,1,32,32, the index return by the implementation will account the padded shape. In such cases, we must send a padded input to the torch.
Please test and let me know, so that I can merge it to main branch : umadevimcw/argmax_with_padding

Recently, during testing, I discovered an issue where the output dtype of argmax is not an integer type. When inspecting torch.argmax, the output dtype is LongTensor, but as TT currently lacks a long type, it seems like it should return a signed integer type for now.
Would you prefer to address this issue within this current one, or would it be better for me to create a separate issue for this?"
@davorchap @umadevimcw @razorback3

@tt-aho https://github.com/tenstorrent/tt-metal/issues/7571#issuecomment-2114068845 any comments on this?
@hschoi4448 Do you want the argmax output in Integer format?

@hschoi4448 Do you want the argmax output in Integer format?

Yes, as following the torch spec 😄

@hschoi4448 Can you retest this PR #8202 (it doesn't include the int format output)?

@hschoi4448 Can you retest this PR #8202 (it doesn't include the int format output)?

I have run the tests on the WH equipment and confirmed that they passed. 👍

@hschoi4448 I have merged the PR #8202 hence closing this issue. Kindly create an separate issue for the int format support


subscriptions_enabled=true is necessary to enable the websockets func…

…tionality in hedera-json-rpc-relay.  This should always be true when deploying the websockets
Description:
This PR modifies the SUBSCRIPTIONS_ENABLED key/value in the websockets configmap.   This value needs to be true to enable the websocket functionality.

Codecov Report
Patch and project coverage have no change.

Comparison is base (570562c) 78.09% compared to head (5ab8443) 78.09%.

:exclamation: Current head 5ab8443 differs from pull request most recent head cbb5e1f. Consider uploading reports for the commit cbb5e1f to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1536   +/-   ##
=======================================
  Coverage   78.09%   78.09%
=======================================
  Files          32       32
  Lines        2584     2584
  Branches      537      537
=======================================
  Hits         2018     2018
  Misses        401      401
  Partials      165      165

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add bigger designs to DRC tests

Add RISCV from SpyDrNet to test cases for PyTest.

Before we do this, we need to do a SpyDrNet release that includes the larger netlists.


Chore: Test Coverage Enhancements

Changes proposed in this pull request:

Added support of default trait for each term
~100% Test coverage

Codecov Report

Merging #10 (f256a0c) into main (4fb6cb7) will increase coverage by 1.82%.
The diff coverage is 0.00%.

:exclamation: Current head f256a0c differs from pull request most recent head 07da3b8. Consider uploading reports for the commit 07da3b8 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   23.60%   25.43%   +1.82%
==========================================
  Files          10        9       -1
  Lines         593      570      -23
==========================================
+ Hits          140      145       +5
+ Misses        453      425      -28

Impacted Files
Coverage Δ

src/ast/cdl/module_term.rs
50.00% <0.00%> (-16.67%)
:arrow_down:

src/ast/cdl/returns_term.rs
100.00% <ø> (ø)

src/ast/cdl/service_term.rs
55.55% <0.00%> (-1.59%)
:arrow_down:

src/ast/mod.rs
18.50% <0.00%> (+0.03%)
:arrow_up:

src/visitor/visitor.rs
60.00% <0.00%> (+20.00%)
:arrow_up:

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


Add more tests

Pull Request Test Coverage Report for Build 34

2 of 8 (25.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 33.929%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

test/icon_test.rb
1
2
50.0%

test/ops_tray_test.rb
1
6
16.67%

Totals

Change from base Build 30:
-0.07%

Covered Lines:
19

Relevant Lines:
56

💛  - Coveralls


feat: add phalcon 5.0.4 release

update phalcon with release 5.0.4

:tada: This PR is included in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add support for ellipsoid tileset loader

Supports https://github.com/CesiumGS/cesium-native/pull/908 by providing a new "From WGS84 Ellipsoid" option in the Cesium3DTileset's "Tileset Source" dropdown.

Thank you @jqntn!


How to use Redis.OM.AspNetCore

I want to use the IRedisConnectionProvider through injection but I do not see how I can use the Redis.OM.AspNetCore namespace and AddRedis. The only public package I can install is Redis.OM. How can I solve this?

You can just add IRedisConnectionProvider as a singleton service.
builder.Services.AddSingleton<IRedisConnectionProvider>(new RedisConnectionProvider("redis://localhost:6379"));

Alright, so the IServiceCollection extension is not intended to be used publicly?

It's not in the library. So no.

Alright, thx for the info!


PF2 Toolbox 1.7.9 - the Settings modal dialog doesn't open

Tested with no mods (other than libwrapper just in case).

Hey @apoapostolov - can you unistall and reinstall Toolbox to verify this for me. I am unable to reproduce this with version 1.7.9 locally.

Yes, did so. Same behavior. Tested with Culprit and no mods.

Do you see any errors in the console?

d {description: undefined, fileName: undefined, lineNumber: undefined, endLineNumber: undefined, message: "An error occurred while rendering SettingsApp 148:…s/settings-app/tabs/About.html could not be found", …}description: undefinedendLineNumber: undefinedfileName: undefinedlineNumber: undefinedmessage: "An error occurred while rendering SettingsApp 148: The partial modules/pf2e-lootgen/templates/settings-app/tabs/About.html could not be found"name: "Error"number: undefinedstack: "Error: An error occurred while rendering SettingsApp 148: The partial modules/pf2e-lootgen/templates/settings-app/tabs/About.html could not be found↵    at Object.h (http://localhost:30000/scripts/handlebars.min.js:27:18644)↵    at Object.c [as invokePartial] (http://localhost:30000/scripts/handlebars.min.js:27:15036)↵    at Object.eval [as main] (eval at createFunctionContext (http://localhost:30000/scripts/handlebars.min.js:29:1808), :11:28)↵    at c (http://localhost:30000/scripts/handlebars.min.js:27:15429)↵    at d (http://localhost:30000/scripts/handlebars.min.js:27:15744)↵    at e (http://localhost:30000/scripts/handlebars.min.js:28:22748)↵    at renderTemplate (http://localhost:30000/scripts/foundry.js:3627:10)↵    at async SettingsApp._renderInner (http://localhost:30000/scripts/foundry.js:2213:16)↵    at async SettingsApp._renderInner (http://localhost:30000/scripts/foundry.js:2740:18)↵    at async SettingsApp._render (http://localhost:30000/scripts/foundry.js:2048:19)"proto: Error
at Object. (http://localhost:30000/scripts/handlebars.min.js:27:5241)
at b (http://localhost:30000/scripts/handlebars.min.js:27:330)
at Object. (http://localhost:30000/scripts/handlebars.min.js:27:2069)
at b (http://localhost:30000/scripts/handlebars.min.js:27:330)
at Object. (http://localhost:30000/scripts/handlebars.min.js:27:1435)
at b (http://localhost:30000/scripts/handlebars.min.js:27:330)
at Object. (http://localhost:30000/scripts/handlebars.min.js:27:792)
at b (http://localhost:30000/scripts/handlebars.min.js:27:330)
at http://localhost:30000/scripts/handlebars.min.js:27:417
at http://localhost:30000/scripts/handlebars.min.js:27:422
(anonymous) @ foundry.js:2006
Promise.catch (async)
render @ foundry.js:2004
onClickSubmenuWrapper @ colorSetting.js:20
🎁SettingsConfig.prototype._onClickSubmenu#0 @ libWrapper-wrapper.js:160
dispatch @ jquery.min.js:2
v.handle @ jquery.min.js:2

@apoapostolov Ok this should be fixed in 1.7.11

Thanks!


Broken documentation links

Vue devtools version
6.6.4
Link to minimal reproduction
N/A
Steps to reproduce & screenshots
See files in the repository
What is expected?

Correct links:
devtools-v6 docs: https://devtools-v6.vuejs.org/
devtools-next docs: https://devtools.vuejs.org/

devtools-v6 directory name instead of devtools.

What is actually happening?

Links in this repository navigate to devtools-next documentation page instead of devtools-v6. These websites have different /docs structure, resulting in 404 errors.

Wrong default directory name in packages\docs\src\guide\contributing.md

System Info
N/A

Any additional comments?
No response

Resolved by https://github.com/vuejs/devtools-v6/pull/2190


fix(deps): update terraform terraform-ibm-modules/cos/ibm to v8.11.5

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cos/ibm (source)
module
patch
8.11.4 -> 8.11.5

Release Notes

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v8.11.5
Compare Source
Bug Fixes

deps: update terraform-module (#712) (b0933f2)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 1.4.14 :tada:
The release is available on:

GitHub release
v1.4.14

Your semantic-release bot :package::rocket:


Roblox Links Faster (Question)

Are there ways to optimize the connection with roblox links?
I mean, there are some games where the use of private servers is very favored, and the use of mobile gives an advantage in this,
You can take the discord link and enter directly in the game, unlike the PC that has to open the browser page, afterwards load the command to open launcher and then the launcher opens
Is there a possibility to do this in roblox PC? or just optimize the current PC process?

Are you talking about deeplinks? If by "discord link" you mean the launch links that Bloxstrap uses for Discord rich presence, then yeah, that's what those are.

I don't know if it's the same thing, you know when they send you a link to the privates servers? in the case of roblox and discord, it directly opens the browser, and then the launcher, which takes a lot of time, in fact, I don't even know if it is possible to improve this since it depends on the link ,correct?

What I want to talk about is, Is there a way I can click on a roblox private server link and have it bypass the website and go straight to the launcher? if so, is there a way to make it work with any link? (or at least the reliable ones)

Oh, you're talking about optimizing private server links. So, turns out that Roblox deeplinks have a parameter for private server access codes, which you can use to join a private server.
roblox://experiences/start?placeId=PLACE_ID_HERE&accessCode=ACCESS_CODE_HERE
You can find the access code of a private server using this API:
https://games.roblox.com/v1/games/ACCESS_CODE_HERE/private-servers
That should allow you to make deeplinks that can be used to join private servers without having to go through the Roblox website first.

With your experience, would it be possible to do this automatically? as a Launcher Feature, (I don't think so, because it depends on the link)
maybe a browser extension that automatically does this? or an extension of discord?
Because although there is a way to do this, it's impractical when trying to quickly join someone else's server.

Not as a launcher feature, but yeah, a browser extension or something like that would be possible. Though, I thought the idea is that you shouldn't have to figure out what the deeplink is when joining, you should kind of just already know what it is?

I'm very layman in relation to this type of subject, at the time that I played "crafwars" I was very irritated with the fact of never being able to enter a server fast enough, then I discovered the mobile technique, in which you leave roblox opens in the background and enters a private discord link, and not only does it not open the browser (because it is mobile) but it also does not open the launcher because it is already open, so it goes straight to loading the game
And since I live in Brazil and the roblox servers are horrible, things take a long time to load, so the less things you have to load the better

but speaking of a possible browser addon, wouldn't it be possible to create a whitelist system? in which the player takes the ID of the experience he wants to bypass the web

Well, I'm not really sure what you mean by whitelist system. If you're able to elaborate further then that'd be great. The thing is, as far as I can tell, the only way to obtain the private server's access code is by getting it from Roblox's website in the first place. Private server access codes are static and don't change, so if you have it on you beforehand, you can just reuse that one link to join the server at anytime.
If you're talking about a situation where you don't have pre-defined access to a private server but someone gives you a link, to be honest, I'm not really sure how that would work.

In the case of the Whitelist, as not every game you need to quickly enter servers, you could manually put the game ID in the Whitelist, of course, if only the game ID was needed
Is there a way to make the launcher load another game or server without closing and opening again ?, this could speed up loading

Is there a way to make the launcher load another game or server without closing and opening again ?, this could speed up loading

Isn't that what the Roblox desktop app is for?

Yes and no, the Desktop is good for entering and exiting experiences, however when talking about links it doesn't even work, because then it will load the browser, close the already active launcher, open another one to load the other game/server

Well, I'm not really sure what you mean by whitelist system. If you're able to elaborate further then that'd be great. The thing is, as far as I can tell, the only way to obtain the private server's access code is by getting it from Roblox's website in the first place. Private server access codes are static and don't change, so if you have it on you beforehand, you can just reuse that one link to join the server at anytime.
If you're talking about a situation where you don't have pre-defined access to a private server but someone gives you a link, to be honest, I'm not really sure how that would work.

I can elaborate and help you make the extension

Anyway, I'm going to close this since this isn't related to Bloxstrap. You can continue talking here though.

is there any way to join without acces code just with private server link?


updates: Add update message (level=10).

The linter is hanging for me. :(

Fixed the linter error (just incorrect indentation), tested manually and merged, thanks @alya!


Server side Routes

I am writing an application that should provide some REST interface, but can't get server side routes working, not even with Picker. Am I missing something?

Hello @jamgold ,
For this purpose you need to create local package (placed above ostrio:flow-router-extra in the .meteor/packages file).
It can be easily done using webapp package only, ow extending it with solutions like express.js, connect.js and similar.
Maybe simpler will be to use simple:rest package.
It's up to you, saying that ostrio:flow-router-extra should be considered as Client-only solution

Hello @dr-dimitru ,
thank you for answering. I have figured out how to use Picker. My main confusion was that I have assets in public for which there are not routes, of course. If a request is made to something in public that does not exist, the not-found route kicks in, which unfortunately is indistinguishable from a successful request (no status 404). Do you know how to manipulate the HTTP Status with this router?

Not currently possible with Meteor. It's Meteor's "feature" and not related to FR package. It's common for most of the front-end JS solutions, and not Meteor specific.
But if you're looking 404 response codes (genuine status code) for SEO (Google, Yahoo, etc.) optimization, take a look on:

Enhanced spiderable package
Prerendering

Thank you again, will check it out.

@dr-dimitru thanks for this package... just checking if I need to serve a server side router with webapp https://docs.meteor.com/packages/webapp.html to a route like https://localhost:3000/someroutepart1/someroutepart2 Flow router is sending back a "not found"
So, I need to get in before FlowRouter and the only way to do that is add me webapp code in a package before FlowRouter gets loaded? https://github.com/VeliovGroup/flow-router/issues/23#issuecomment-331670213
it seems like if I hit https://localhost:3000/someroutepart1 webapp gets it.. .but as soon as I add some stuff ike someroutepart2 then it seems to grab it
BTW, I did post here https://forums.meteor.com/t/flowrouter-is-grabbing-my-requests-from-webapp/49888 as I know this is not a bug


6.1.1 oneof validate error

What version of Ajv are you using? Does the issue happen if you use the latest version?
6.1.1
Ajv options object

new ajv({
    allErrors: true,
    jsonPointers: true,
    useDefaults: true,
    format: "full",
    $data: true,
    errorDataPath: "property",
    removeAdditional: true,
})

JSON Schema

{
    type: "object",
    $id: "design",
    required: ["name"],
    properties: {
        name: {
            type: "string"
        },
        appType: {
            oneOf: [{
                $id: "design-app-type-1",
                type: "object",
                properties: {
                    a: {
                        type: "string",
                        title: "test-oneof-2"
                    }
                }
            }, {
                $id: "design-app-type-2",
                type: "object",
                properties: {
                    b: {
                        type: "string",
                        title: "test-oneof-2"
                    }
                }
            }]
        }
    }
}

Sample data

{
    name: "nick",
    appType: {
        a: "dkfkald"
    }
}

Your code

Validation result, data AFTER validation, error messages
[{
dataPath:"/appType",
keyword:"oneOf",
message:"should match exactly one schema in oneOf",
schemaPath:"#/properties/appType/oneOf"
}]

What results did you expect?
validate should be true
Are you going to resolve the issue?

oneOf means that only one sub-schema is valid, in your case - both are valid.

thanks.

I'm not sure I understand this.  I've run into this issue and can validate it with other validators, but not this one.  You say they are both valid?  Should it not match the first one but not the second, thus being valid?
In my case, if I change the order of the oneOf, it affects the validation. I'm not sure how to validate such a case with your tool.

TLDR: please read the JSON schema spec.
They are both valid because neither schema requires or prohibits the presence of any property. Specifically, the data is valid against the first schema, because “a” is present and matches the schema for “a”. The data is valid against the second schema, because there is no property “b”.
That’s a very common musconception to think that JSON schema describes the structure the data should have to be valid, while it just defines conditions that make data invalid if they are not met. In this way “properties” keyword does not require any property to be present.
If you have a specific example that you believe is not working according to the spec, please post it here but it would help if you checked the spec as well.

As a note about data passing validation with some other validators. The fact that the validation results differ between validators means just that - that the results are different. It obviously means that one of (oneOf:) validators is violating the spec (unless you’re in the grey zone of format validation). But it does not mean that the one that fails is incorrect. From my experience, the opposite is much more likely - i.e. the one that passes is incorrect, because the whole purpose of the validation is to recognise certain data as valid.

oneOf means that only one sub-schema is valid, in your case - both are valid. it is the correct as i was stuck for 4 day before i got this post that resolved my issue  thanks

You might add additionalProperties: false to both objects in oneOf, so they won't be valid simultaneously.


RoadRunner integration

We're using RoadRunner to run our website. It bootstraps multiple instances of your application (called workers) that wait for incoming requests. RoadRunner will pass a request object to the worker and the worker will have to pass back a response object. Anyway, you can find out more about how it works on the website. The implication is that you cannot use the globals $_GET, $_SERVER, etc. as the workers are just CLI processes.
I saw that you can pass a $request object to RequestIntegration::applyToEvent but it isn't clear to me who calls this function and how, if at all, I can pass my own request object.
I would appreciate some help, maybe this could also be added to the documentation.
Thank you!

RequestIntegration implements IntegrationInterface, and it's a default one. The integrations are registered with the Handler here:
https://github.com/getsentry/sentry-php/blob/2.3.1/src/Integration/Handler.php#L28-L50

Thanks! I'll try your suggestions and report back soon 🙂

I've tested this and it does indeed work, thank you! One annoyance is that I had to copy the entire RequestIntegration class because it is marked as final and the processEvent method is private. I could've used the applyToEvent method but I didn't since it's marked as deprecated.
There is no alternative that doesn't require copying the whole 200+ lines of code right now, correct?

There is no alternative that doesn't require copying the whole 200+ lines of code right now, correct?

As of now I don't see either any way you coukd avoid copying the whole file. However, we may accept a callable or a factory in the constructor that will take care of providing the PSR-7 server request so that how this is done is not hardcoded. WDYT?

That would be great! I'm converting the Symfony request to a PSR request anyway, I basically just need a way to pass it to Sentry.

This would need to be scheduled for version 2.4, so it's not something that will happen soon. Would you like to create a PR to address this? Branch to target is develop

FYI https://github.com/getsentry/sentry-php/pull/984


Feature/sp5 416 quickstart wayat backend prepare variable substitution

closes #416

Merge https://github.com/devonfw/hangar/pull/309 first


Side Server List

Can you increase the size of server icons on the left side, please?

They are too small.

Hey, I know this issue is pretty old, but I have found a fix to this issue, you will have to edit the .css file for this theme but you should search for the following wrapper to fix this issue.
By default, the following wrapper (28eC3z) gives the values width=48, but then follows it up with W40px;H40PX.
The following segment is an example of the corrected code that resolves the small issue, as-well as some other display issues that were caused by the mismatched size.
.wrapper-28eC3z,
[data-list-id='guildsnav'] [data-dnd-name] > div,
[data-list-id='guildsnav'] svg[width="48"] {
	width: 48px;
	height: 48px;
}```

If you still use this theme, then this worked in my case to manually patch the issue.


Consider making imgui:::Context implement Send

Is your feature request related to a problem? Please describe.
If I create imgui::Context in one thread but want to move ownership to another thread, I can't because the compiler complains:
error[E0277]: `*mut ImGuiContext` cannot be sent between threads safely
   --> src\lib.rs:205:14
    |
205 |               .spawn(move || {
    |  ______________^^^^^_-
    | |              |
    | |              `*mut ImGuiContext` cannot be sent between threads safely
    | |_____________- within this `[closure@src\lib.rs:205:20: 213:14]`
    |
    = help: within `[closure@src\lib.rs:205:20: 213:14]`, the trait `Send` is not implemented for `*mut ImGuiContext`
    = note: required because it appears within the type `imgui::Context`

error[E0277]: `Rc<RefCell<SharedFontAtlas>>` cannot be sent between threads safely
   --> src\lib.rs:205:14
    |
205 |               .spawn(move || {
    |  ______________^^^^^_-
    | |              |
    | |              `Rc<RefCell<SharedFontAtlas>>` cannot be sent between threads safely
    | |_____________- within this `[closure@src\lib.rs:205:20: 213:14]`
    |
    = help: within `[closure@src\lib.rs:205:20: 213:14]`, the trait `Send` is not implemented for `Rc<RefCell<SharedFontAtlas>>`
    = note: required because it appears within the type `std::option::Option<Rc<RefCell<SharedFontAtlas>>>`
    = note: required because it appears within the type `imgui::Context`

error[E0277]: `(dyn imgui::ClipboardBackend + 'static)` cannot be sent between threads safely
   --> src\lib.rs:205:14
    |
205 |             .spawn(move || {
    |              ^^^^^ `(dyn imgui::ClipboardBackend + 'static)` cannot be sent between threads safely
    |
    = help: the trait `Send` is not implemented for `(dyn imgui::ClipboardBackend + 'static)`
    = note: required because of the requirements on the impl of `Send` for `Unique<(dyn imgui::ClipboardBackend + 'static)>`
    = note: required because it appears within the type `Box<(dyn imgui::ClipboardBackend + 'static)>`
    = note: required because it appears within the type `imgui::clipboard::ClipboardContext`
    = note: required because of the requirements on the impl of `Send` for `Unique<imgui::clipboard::ClipboardContext>`
    = note: required because it appears within the type `Box<imgui::clipboard::ClipboardContext>`
    = note: required because it appears within the type `std::option::Option<Box<imgui::clipboard::ClipboardContext>>`
    = note: required because it appears within the type `imgui::Context`

Describe the solution you'd like
imgui::Context implements Send
Describe alternatives you've considered
I can create the imgui context on the recipient thread directly. It would be nice not to have to do this though.

ImGui is not thread safe -- so although we could implement it, the underlying library uses thread locals all over the place, so you'd be in for a bad time

Oh, I didn't know that. I'll close this then, thanks!


fix: parallel queueing race condition caused silent failure

As of the most recent llama.cpp update concurrent requests had a race condition that would result in an empty response.
This was not easy to observe since the response from the llm runner subprocess was a 200 with the error {"content":"slot unavailable"} in the response stream, which just silently closed the channel.
This change resolves this by allowing multiple slots in the llm runner subprocess. We manage the queueing ourselves so this should be ok. @dhiltgen this may be a case we need to account for in the cgo changes.

LGTM. Given https://github.com/ggerganov/llama.cpp/issues/3821, this seems like the best approach for now


Audit translation approver, and revoke translations by admins

Okay, so it's extremely easy to approve translations now since there's now a translation tab. But what to do if someone around just blindly accepts every single translation they can? Are there any ways to undo the inevitable damage they've done? Can we find out who approved the problematic translation in the first place?
Example: Someone approved the Haskell translation of this kata, which is completely pointless because Haskell has native arbitrary sized integer support and it's impossible to disable it. It should be revoked, but nobody can do that, and there are no records who approved it in the first place either (revisions page doesn't give anything useful).

The new pending translations tab is not shown to everyone. Only 20 users I explicitly granted can see it other than admins. I picked from users that was already approving translations and those that seemed to care about Codewars from past interactions/comments. I can revoke if they're abusing.
The user who approved is stored in the database so we can start to show it. Maybe on /translations page.

For undoing, it's currently not easy to do and it can cause data issues. I'm planning to add "soft" delete for comments so I can do similar for translations and prevent it from showing.

The new pending translations tab is not shown to everyone.

What? When I click "Add New" it takes me there. Also, it says that closed translations are "visible only to moderators and authors", but I can see all the translations there (no, it's not my kata).

The new pending translations tab is not shown to everyone.

What? When I click "Add New" it takes me there. Or are you talking about another tab?

That's another thing. He is talking about this:

A translation tab? That's great news! Finally we're heading towards a big next step in solving "how to get my translation approved" conundrum. Thanks!

As for technical issues for improving the quality of translation approvals:

allow to see output for 'sample tests'
allow to run tests for 'setup' code

@kazk any update on this already? I still have pending translations from 3 years ago. I don't feel the incentive to work on translations.

I still have pending translations from 3 years ago. I don't feel the incentive to work on translations.

This issue is not about translations review/approval taking too long.
Content management is a voluntary work, and PUs are not obliged to review anything. If you want people to notice your pending translations, you can try bringing attention to them by leaving a comment. In fact, you did so, but...
Make sure that your translations are well-made before saying "review and approve" (or at least be ready to fix them). Today you bumped 4 blue/purple translations. I checked the 2 of them which I had access to, and rejected both because there were 6+ months old comments about issues which had never been fixed, and here you even explicitly refused to fix anything - with such attitude I don't see how your "lack of incentive to translate anything" is bad...

I still have pending translations from 3 years ago. I don't feel the incentive to work on translations.

This issue is not about translations review/approval taking too long.

I agree, but it helps in communicating dealing with pending translations, which is a longtime issue. It's therefore VERY difficult to get translations approved.

Content management is a voluntary work, and PUs are not obliged to review anything. If you want people to notice your pending translations, you can try bringing attention to them by leaving a comment. In fact, you did so, but...

First of all: as if I don't know. Furthermore creating translations is voluntary work too, so what's your point? The point however is that pending translations often don't get noticed. For some Katas the author has longtime abandoned them. Hence my pending translations of 3 years old. This issue is about adding a translation tab which help communicating pending translations, which is great.
About your 3rd point, that is totally besides the point of this issue, I don't know why you bring that up. Still you seem to suggest that the issue might be about the quality of my translations, rather than a missing mechanism. I find your slur about the quality of my translations offensive. Contrary to what you're suggesting, the amount of my already approved translations, as well as the feedback I've got on them shows the general quality of them.
I don't mind getting feedback, but it does not mean all feedback by default is equally relevant.
The issue you explicitly refer to is a request to add random tests, which however are not in the original testcode either. I therefore think that's asking too much. The original requester did not respond to my reply anymore, so does not explicitly disagree with my reasoning.  That is not about explicitly refusing, just countering with an argument. If you think that is about an 'attitude', and hence you suggest I stop translating, well who are you to tell me that? As a matter of fact I've noticed in other issues that other commenters are actually having an issue with your attitude. Maybe it's time for some self-reflection.
As for the other translations, I've checked them, because I was not aware of any comments on them.
So instead of approving or leaving your remark, you decided to reject the 2 which had only some minor comments added, giving me a lot of unnecessary work to prepare these translation for review again. Have you made it your goal in life to annoy people? You're doing pretty well.


Border style affects resize operations

Setting the border style of the drag / resize boundary area will make the the resizing of left and top borders fail.
You can easily verify this by adding a border style in the demo app:
.area {
position: relative;
height: 200px;
width: 100%;
border-style:dotted;
border-radius: 3px;
background: rgba(211, 211, 211, 0.25);
&.large {
height: 500px;
}
}

border-style:dotted;

I've checked it. Everything works as expected.
https://github.com/user-attachments/assets/35196e30-113b-48f4-823e-5531eaefbad3
Please, try using the latest library version 1.0.3 and provide more specific details if you still have an issue.

Setting the border style of the drag / resize boundary area will make the the resizing of left and top borders fail. You can easily verify this by adding a border style in the demo app:
.area { position: relative; height: 200px; width: 100%; border-style:dotted; border-radius: 3px; background: rgba(211, 211, 211, 0.25);
&.large { height: 500px; } }

Hm, that's strange 🤔.
But you can just wrap the boundary into additional container with styled border. It will work as expected.
https://github.com/user-attachments/assets/3245d03e-885a-4083-a8c1-2c58b6e1eced


Consider one-to-one mapping of an invocation to scan result

The recent addition of the COSIGN_VULN_ATTESTATION_SPEC supports storing multiple scan results originating from a single invocation. I think it would be more intuitive if we encouraged folks to map a single invocation to a single scan result.
This would not only make it easier to construct the payload, but easier to write and evaluate policy against as well. I think we should discourage concatenating the outputs from multiple scans in a single invocation. Doing so would be inconsistent with what invocation is intended to represent as far as I can tell.
Constructing separate vulnerability attestations for different types of scanners makes it easier to write a process that evaluates one or more policies against them. For example, you would download a single, named vulnerability attestation and have some guarantee (or at least a reasonable expectation) that it will contain a result set of a particular schema.
Slack thread for reference: https://sigstore.slack.com/archives/C01PZKDL4DP/p1640878335099100

cc @developer-guy @Dentrax what do you think?

+1, that makes sense! It would be better if the the spec mapped single invocation to single scanner. In this case scanner field should not an array?

Yeah I think that's right!

Hello all, attempted to resolve this in https://github.com/sigstore/cosign/pull/1268


uproxy-lib build script for windows

setup.cmd will do the same thing as setup.sh, but on Windows.
The dependency tsd has also been updated from 0.5.7 to 0.6.3, since the new version removes an (optional) dependency on Python 2.7, which would error on vanilla Windows.
The readme has also been updated.

@trevj Added a new change to the build script – apparently it went into this branch again.

:+1:
Thanks!

Oops, never merged this in. To be honest I'm sceptical we'll keep this up to date over time but since Windows is popular and we haven't changed the build system for a while...let's try it for a bit.


Support by fire is currently a line symbol with the angle arrows as the linear stroke symbol layers; width of support by fire is a point marker symbol layer

From @Dbarnes1 on May 23, 2017 15:29
Issue originally posted in: c_api/issues/7245#issuecomment-753196
Currently the Support by Fire symbol draws with the angle arrows as two line strokes, and the line that indicates width as a shape marker. The length of the angle arrows is actually not important to the symbol according to the standard, as @joebayles has already pointed out:

The proposed solution to this would be to make the two arrow angle lines shape markers, and make the line that indicates with the line stroke. The symbol would be altered and therefore draw perpendicular to the general direction in which the fire would be moving, which is a little counter-intuitive but closer to how the standard dictates this should be implemented.
I don't think we have a fix currently for implementing the precise angle of the arrows at this time.
Copied from original issue: Esri/military-features-data#267

From @Dbarnes1 on May 25, 2017 21:2
for update 1 of runtime quartz, they are treating this as a Known Issue as it currently doesn't make sense to implement the symbol the way the standard specifies. Issue about the standard and this symbol located here: https://github.com/Esri/joint-military-symbology-xml/issues/337

We need to understand if this is an issue with APP6-B based on our new stylex file.
Currently the information we have related to this issue can be found here:
https://github.com/Esri/military-features-data/wiki/Known-Issues

From @BobBooth on April 12, 2018 17:27
APP-6B should probably match what we do with this. The point ordering is not specified in 6B.

In contrast, 2525C does specify the point order:

The Point 1 - Point 2 sequence, to define the length of the line, would make sense.

Document if this is supported in 100.2 and ArcGIS Pro 2.1. We need to understand if this is an issue with APP6 - B as well.

@kerryrobinson @mbriat this is the best example that we currently have of a challenging symbol to draw.

This issue was moved to ArcGIS/military-symbology-styles#71


Version Updated

Version Updated
Supported versions of Jira and Confluence

Jira Core and Software: 6.0 to 7.12
Jira Service Desk: 3.0.0 to 3.5.0
Confluence: 6.0.1 to 6.15.9
Confluence: 7.0.1 to 7.17.0

@NihalPerennial - please sign the CLA agreement.  https://cla.opensource.microsoft.com/MicrosoftDocs/azure-docs?pullRequest=95439 . We can't review your submission until you do. thanks.


Crash in last version svgo 1.3.0

🐛 Bug Report
When the svgo sub-dependency of @svgr/plugin-svgo has been updated to version 1.3.0, the svgs built with this library have started giving this error error while updating property 'd' of a view managed by rnsvgpath. If I block the subdependency in the previous version 1.2.2, the svg components still work.

To Reproduce
Using a svg component in a react-native application
Update version SVGR
Expected behavior
Working

Related to https://github.com/smooth-code/svgr/pull/321#issuecomment-511390022
Did you report this to svgo?

You can force the usage of SVGO 1.3 the time the bug is fixed. Unfortunately SVGO is not well maintain...

In the version SVGO 1.2.2 work I block version in this version but in the version 1.3.0 Crash. I will report this bug in SVGO @TrySound thanks alls

Other users are having this problem - https://stackoverflow.com/questions/57050691/null-for-input-string-c-error-while-updating-property-d-of-a-view-managed-b

Let's wait for SVGO to solve it.


Update repository after transfer

See https://github.com/senzing-garage/garage-roadmap/issues/3

Pull requests:

https://github.com/senzing-garage/g2-sdk-go/pull/117
https://github.com/senzing-garage/g2-sdk-go/pull/118


How to set Soap version to 1.2

Is it right, that node-soap uses only soap version 1.1?
Or ist it possible to set it to version 1.2?
With 1.2 it shall use
xmlns:soap="http://www.w3.org/2003/05/soap-envelope/"
in the soap:envelope, instead of
xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/"
I'm wondering that the soap version is mentioned nowhere.

Hello,
I'm considering providing a PR for that issue. But I'm not a soap expert (and I don't really want to become one ^_^), so I'd happily have some guidelines!
Is the soap-envelope namespace the only change that must be applied to node-soap for it to become 1.2 compliant? This sounds too easy!
When I look at the client & the server code, the envelope creation is hard-coded. Wouldn't it be possible (and better) to use the wsdl generator? Because of the rpc/document differences, maybe we still need to do it in 2 passes and put the body afterwards inside the envelope.
Any other idea?
Michel

any updates on setting soap version to 1.2 or an alternate way to make soap 1.2 request ?

@aj2050 nope.  We need a PR to add the support for it.


ceph-config: remove container_binary variable

9e7972a introduced a regression via the container_binary variable
which is undefined.
The CEPH_CONTAINER_BINARY environment variable isn't used at all.
Signed-off-by: Dimitri Savineau<EMAIL_ADDRESS>

jenkins test luminous-centos-non_container-lvm_batch

jenkins test luminous-centos-container-lvm_batch


Dataset for training prompt injection v2 model

Hello, Can you share the dataset on which you trained the prompt injection v2 model?
https://huggingface.co/hubert233/GPTFuzz
Will be great if u can add this info into the model card.

Hey @priya-dwivedi , thanks for reaching out. This dataset was generated for evaluation of this paper - repository https://github.com/sherdencooper/GPTFuzz#models


Update init.py

Fixed a url error.
Clicking the link takes you to "https://pypi.org/project/clintest." with the dot.

Thanks for your contribution. I will merge these changes. However, I am currently having problems with the automated tests. For some reason, they did not run for your pull request which is now blocking the merge.


Add option to rebase to target for git finish

Add an option to rebase to target before pushing the branch.
» .git-finish --help
Pushes branch to GitHub and opens a PR

Usage:
    git finish
  -w, --wip       Open [WIP] PR
  -r, --rebase    Rebase to master
  -h, --help      Show this message

Hehe, thanks @eqbal!
The intention of trello_flow is to automate just the essentials of our trello-based workflow, ie starting branches from trello cards, assigning members and moving cards, and using a certain naming convention for pull requests. Ie things that has to be done for every story.
Currently rebasing is not really a requirement of that flow as such. It could result in conflicts (breaking the flow), and is probably something that could be done by setting up some aliases locally if you really want it, so I think for now maybe will prefer to keep this sort of thing out of trello_flow 😉
Thanks again though! Please keep submitting ideas like this! 🙏

Thanks for the explanation @balvig. Makes perfect sense. I'll close the PR.


fixed tense and plural/singular grammar issues

fixed some singular/plural grammar issues
changed words to match the tense used
statements read as questions were rewritten as questions

Thanks for this :)


Separate out integrations docs

https://app.shortcut.com/larder/story/7999/separate-out-integrations

Can David read my thoughts ? 🤔 I was just looking at our docs for google auth (regarding the gcalendar plugin) and was thinking we could do this ^^

@iain-b S3 docs can go into this section now.