libretro: refactor infrastructure

This is a full refactor on how we package Libretro cores inside nixpkgs. The idea is to remove all our custom infrastructure (e.g.: update_cores.py) in favor of just having each core in their own file calling the mkLibretroCore function. This has lots of advantages:

We don't need a custom update script anymore, we can simple use the existing update scripts
Much easier to reason about each core, instead of looking at 3 different files

All cores are in their own scope (this is the reason we can't move the cores themselves to pkgs/by-name), making it much easier to override dependencies cross packages.
Other changes:

Renamed retroarchFull to retroarch-full and retroarchBare to retroarch-bare (also created aliases to the old names)
Moves retroarch and their packages to pkgs/by-name
The old pkgs/applications/emulators/retroarch directory was renamed to pkgs/applications/emulators/libretro
Reformat all files using nixfmt-rfc-style

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 357228

x86_64-linux

:white_check_mark: 9 packages built:

kodi-advanced-launchers
kodiPackages.libretro-2048
kodiPackages.libretro-fuse
kodiPackages.libretro-genplus
kodiPackages.libretro-mgba
kodiPackages.libretro-nestopia
kodiPackages.libretro-snes9x
retroarch-bare
retroarch-full

@ofborg build retroarch-full
CC @aanderse for review.

@ofborg build libretro

Ok, the number of rebuilt packages looks wrong, I will take a further look what is happening later.

I think the original PR was over ambitious. I removed all changes from RetroArch and will focus this PR in just refactoring the libretro cores. Once this is merged I will do another PR refactoring the RetroArch derivations themselves.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 357228

x86_64-linux

:white_check_mark: 7 packages built:

kodiPackages.libretro-2048
kodiPackages.libretro-fuse
kodiPackages.libretro-genplus
kodiPackages.libretro-mgba
kodiPackages.libretro-nestopia
kodiPackages.libretro-snes9x
retroarchFull

100 packages built

Finally it is working correctly.

Tested:

Gambatte
Sameboy
2048
snes9x
bsnes
PCSX-Rearmed
SwanStation

CC @aanderse for review.


Consider adding /opt to rootfs

Google chrome installs itself in /opt/google/chrome/, and sets up symbolic links in /usr/bin/.

Oz needs to be equipped to deal with applications installed this way (it assumes binary in /usr/bin/, so looks for it in /usr/bin-oz/)

For precision: /opt isn't included in the rootfs as it's pretty much considered free form and many people have the bad habit of making directories and executable user writable. I suppose that is mitigated as the rootfs bind mounts are added as read only, but definitely something to consider before doing this. At the very least, I don't think this should be a default.
See also https://github.com/subgraph/oz/blob/f34e6ce9031d99b9c7952f2b08603953ccf146a6/oz-init/init.go#L391


Web - OldDot - Secondary login is not validated

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Log in to NewDot with any account
Log into OldDot with the same account and add a Secondary Login from Account Settings e.g<EMAIL_ADDRESS>Open validate secondary log in email Verify secondary email for Expensify and copy the last part of the URL, i.e. /v/16/WSSWMNNM (Everything after the /v/)
In a text box - Formulate the URL by using the path obtained from step 3 http://staging.new.expensify.com/v/

E.g http://staging.new.expensify.com/v/12116/WSSWMNNMH

Navigate to the compiled link

Expected Result:
LHN should be displayed. Secondary login should be validated.
Actual Result:
Secondary login is not validated
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/8bc2bb96-baaa-4274-b841-7528967a4a70
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

@kbecciv - can you please re-write the reproduction steps, they are hard to follow. Are you meant to validate the secondary login and then copy the URL? In your video, the email is unvalidated and yet you are saying the result should be validated.

I cannot reproduce this with the current steps:

I logged into new and old dot with<EMAIL_ADDRESS>

I added<EMAIL_ADDRESS>as the secondary log in (I did not validate it)

I copied the link from the validation email and formatted it as suggested in the OP

When I navigate to the compiled link I am presented with the magic code, which is what I'd expect given the account is unvalidated.

Closing until I get more context from @kbecciv

@kbecciv - I need you to provide feedback on my steps here - because I am not able to reproduce it.

Waiting until I have feedback.

Hello @sophiepintoraetz! I'm reopening the issue because the QA team is still able to reproduce the problem. I reviewed your testing here, specifically on step 4. The secondary account should be validated after the user navigates to the compiled link and is presented with the magic code. I completely agree that the steps are hard to follow in TR, but if you can submit a TR update ticket and review the expected results here, that would be wonderful! Thank you and Merry Christmas
https://github.com/Expensify/App/assets/93399543/97104933-7199-472e-a755-9da37e3a8be2

Thanks @kbecciv for following up. I am going to close this issue for now - even with the clearer reproduction steps, this is:

Happening on OldDot, which we are working fast to deprecte (and therefore, not spending engineering effort on it)
There are no customer reports that I can see detailing this issue.

If there is an increase of user reports (e.g. this is one that may https://github.com/Expensify/Expensify/issues/346037 be associated) - then let's reopen it. until then, we are aware it is a bug but will not fix it.


Editor sets incorrect indent when character immediately before the break is a paren

Description
I've been having consistent issues with the  editor's automatic indents, that not only render it useless, but actively hinder in many situations. Specifically, when breaking a line, whenever the last non-whitespace character before the break is a parenthesis (which happens frequently when following the PEP8 recommendation to break before the operator), auto-indent sets an incorrect indent level for the following line, (the indent level of whatever statement on the previous line if after a close paren, or two tabs over if an open paren) and makes it very difficult to even manually fix every time—if I tab or space even one character past the desired position, backspace, shift-tab etc. will send me all the way back where I started. This is sufficiently problematic that the feature is a net loss in efficiency if left unfixed.
What steps will reproduce the problem?

Paste one of the example snippets below
Place cursor where indicated ("|")
Hit return/enter for a linebreak
Experience the joys and sorrows of trying to manually align it properly, with the auto-indent actively working against you if you slip up.

What is the expected output? What do you see instead?
Some contrived/minimal examples (cursor "|"):
Works:
When the last char before a break isn't a paren, regardless of style
Before Break:
if True:
    print(str(str("foo" |+ str("bar"))))

vs.
if True:
    print(str(str("foo" +| str("bar"))))

After Break (As Expected):
if True:
    print(str(str("foo"
                  + str("bar"))))
if True:
    print(str(str("foo" +
                  str("bar"))))

Doesn't Work
When the last character before the break is a close paren, as is much more common when following PEP8 break before operator style, it doesn't work:
Before Break:
if True:
    print(str(str(str("foo") +| "bar")))

vs.
if True:
    print(str(str(str("foo") |+ "bar")))

Expected After Break:

if True:
    print(str(str(str("foo") +
                  "bar")))

if True:
    print(str(str(str("foo")
                  + "bar")))

Actual After Break for Second Case:
if True:
    print(str(str(str("foo")
    + "bar")))

Open Paren Case
Before Break:
if True:
    some_long_variable_name = ("foo" + "bar")

Expected After Break (Yeah, not great style, but better than the actual):
if True:
    some_long_variable_name = (
        "foo" + "bar")

Actual After Break:
if True:
    some_long_variable_name = (
            "foo" + "bar")

Another (slightly less minimal/contrived) example
Before break:
if True:
    (print(str(("Long string " + "Yeah").|format(some_long_variable_name))

vs.
if True:
    (print(str(("Long string " + "Yeah")|.format(some_long_variable_name))

Expected After Break:
if True:
    (print(str(("Long string " + "Yeah").
               format(some_long_variable_name))))

if True:
    (print(str(("Long string " + "Yeah")
               .format(some_long_variable_name))))

Actual After Break for Second Case :
if True:
    (print(str(("Long string " + "Yeah")
    .format(some_long_variable_name))))

Please provide any additional information below
Experienced on multiple platforms since Spyder 3.2.2, when I started using it.
Version and main components

Spyder Version: 3.2.4
Python Version: 3.6.3
Qt Versions: 5.6.2, PyQt5 5.6 on Windows 8.1 x64

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
numpy >=1.7      :  1.13.3 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.5 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  1.1.1 (OK)
cython >=0.21    :  0.26.1 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.1.0 (OK)
pylint >=0.25    :  1.7.4 (OK)

Thanks for the detailed bug report.
@rlaverde, please take a look at this one.

I can reproduce these ispsues, but the Open Parent Case is theexcpected behavior, is the hanging indent, see :#3487
@ccordoba12 could It go to master instead of 3.x so I could also refactor indentation (using an editor extension)

is the expected behavior, is the hanging indent

Maybe we could then add an option to activate/deactivate hanging indentation?

could It go to master instead of 3.x

Sure, no problem.

Ah, okay. I wasn't so sure about that one. However, is the following example the expected behavior? I.e. not picking up the indent level of the enclosing function? Just want to confirm...
test_variable = print(long_function_name(
        "foo", "bar"))

Thanks for looking at this!

However, is the following example the expected behavior? I.e. not picking up the indent level of the enclosing function?

Yes, that's the expected behavior

Maybe we could then add an option to activate/deactivate hanging indentation?

No, there's no way to deactivate it (It'll generate pep8 issues), see: https://github.com/spyder-ide/spyder/issues/3480#issuecomment-250394229
Maybe changing two only add two levels when defining a function (one level otherwise) will be the better, see https://www.python.org/dev/peps/pep-0008/#indentation

All sounds good; thanks so much to you both for looking into that aspect. The main thing that seemed odd was that it was two levels, not one, when I wasn't doing anything special (e.g. function definition) and I couldn't find anything in PEP8 about two levels except in such special cases, so hopefully that can be fixed assuming it isn't too tricky to check for def, etc. at the beginning of the line before adding two vs one tab. The close paren behavior is a more common problem at least in my code, and seems like a more explicit bug.

@rlaverde Hey, here are some more, different cases where Spyder isn't picking up the indents properly...cursor is |
After multi-line lists in function calls, similar behavior to close paren above (whether or not close paren or comma is present):
if True:
    io.merge_files_header(["CRNS0101-05-2010-CO_Cortez_8_SE.txt",
                           "CRNS0101-05-2011-CO_Cortez_8_SE.txt"],|

Expected after linebreak:
    io.merge_files_header(["CRNS0101-05-2010-CO_Cortez_8_SE.txt",
                           "CRNS0101-05-2011-CO_Cortez_8_SE.txt"],
                          |

Actual after line break:
    io.merge_files_header(["CRNS0101-05-2010-CO_Cortez_8_SE.txt",
                           "CRNS0101-05-2011-CO_Cortez_8_SE.txt"],
    |

Picking up function-level indent in certain situations even with close paren after function call; Example:
if True:
    station_fname_suffixes = ("-" + station_data["STATE"], + "_"
                              + station_data["LOCATION"].replace(" ", "_",
                                                                 regex=True)
                              + ".txt")|

Expected after break:
if True:
    station_fname_suffixes = ("-" + station_data["STATE"], + "_"
                              + station_data["LOCATION"].replace(" ", "_",
                                                                 regex=True)
                              + ".txt")
    |

Actual after break:
if True:
    station_fname_suffixes = ("-" + station_data["STATE"], + "_"
                              + station_data["LOCATION"].replace(" ", "_",
                                                                 regex=True)
                              + ".txt")
                              |

Thanks for looking into this!

Yet another (simplified) case; similar to the first one in the previous comment except with parens instead of brackets, and different behavior again: in this case, the next line will autoindent to be two tabs from the left margin, like an open paren (nominally only one for a function or class def, but presently in Spyder for any open paren as discussed above), regardless of the actual indent level of the previous statement.
data = pd.read_fwf(io.join_path("test" + ".txt",
                                SOME_CONSTANT),|

if True:
    if True:
        if True:
            if True:
                data = pd.read_fwf(io.join_path("test" + ".txt",
                                                SOME_CONSTANT),|

Expected:
data = pd.read_fwf(io.join_path("test" + ".txt",
                                SOME_CONSTANT),
                   |

if True:
    if True:
        if True:
            if True:
                data = pd.read_fwf(io.join_path("test" + ".txt",
                                                SOME_CONSTANT),
                                   |

Actual (two tabs from left margin regardless of prior indent)
data = pd.read_fwf(io.join_path("test" + ".txt",
                                SOME_CONSTANT),
        |

if True:
    if True:
        if True:
            if True:
                data = pd.read_fwf(io.join_path("test" + ".txt",
                                                SOME_CONSTANT),
        |

Yet another diverging case, again with brackets and commas in e.g. function calls...only this time, the comma actually does make a difference, behaving incorrectly when a comma is correctly present and "correctly" when a comma is not present (when it should be for syntactic correctness).
To keep things shorter but clearer, I'll illustrate four similar but slightly different (simplified) cases; in each case, before the line break, assume the cursor was at the end of the last non-empty line. The first case produces the expected outcome for all of them (the cursor aligned one col to the right of the open paren of the function call); the others do not. Thus:
At least one arg before last one, no comma after: Produces expected indent.
 if True:
    station_data = pd.read_fwf(encoding="utf-8",
                               usecols=["UTC_DATE", "UTC_TIME",
                                        "WIND_1_5"]
                               |

At least one arg before, comma after (new case): aligns indent as if I hadn't closed the array bracket.
if True:
    station_data = pd.read_fwf(encoding="utf-8",
                               usecols=["UTC_DATE", "UTC_TIME",
                                        "WIND_1_5"],
                                        |

First arg, comma or not: Same behavior as two comments up, first case (as it is essentially the same situation)...cursor is aligned to the same level as station_data, as if starting a new statement inside the if block.


Possible invalid status error on Lead conversion

The Interactions Processor uses a SOQL query to select any status where IsConverted = true to use as the Lead Status on conversion here: https://github.com/SalesforceFoundation/Interactions-for-Student-Recruitment/blob/master/src/classes/INT_InteractionProcessor.cls#L89. If Lead Record Types are enabled with a default Record Type assigned to a profile, it's possible that this query returns an invalid status value for the assigned Lead Process.

In order to convert a Lead, it must be at a Lead Status that is marked as a "Converted" value. The code currently queries all Lead Status values and uses the first one that is marked as Converted. The work-around for this issue is to use only one Converted Status that is assigned to all Lead Record Types. If you need to use multiple Converted Statuses assigned to different Lead Record Types, significant code changes may be required.


chore: bdd-nh-import-1586287254 to 0.0.1

chore: Promote bdd-nh-import-1586287254 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Added Course on Linked Lists

What does this PR do?
Added a course on linked lists.
Description
Absolutely Free course to learn linked lists
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Please tell if any other changes are necessary before merging this PR @eshellman

Done @eshellman

please resolve conflicts.


Have all workspaces share a target directory

This has numerous benefits:

Build times are faster, because all of the workspaces share a deps directory.
cargo clean now works.
It simplifies the script used to lint the repository.

Thanks to @thomaseizinger for showing us this was possible.

Oh that is exciting!
Thank you!


ARSessionOrigin disagrees that TPD(InputSystem) is a valid pose driver

Describe the bug
TrackedPoseDriver script is missing on Main Camera object on vanilla MRTK XR Rig prefab and when running Play mode there is a warning in the console log:

Camera "Main Camera" does not use a AR Pose Driver or a Tracked Pose Driver, so its transform will not be updated by an XR device.  In order for this to be updated, please add either an AR Pose Driver or a Tracked Pose Driver.
UnityEngine.XR.ARFoundation.ARSessionOrigin:Awake () (at Library/PackageCache/com.unity.xr.arfoundation@4.2.6/Runtime/AR/ARSessionOrigin.cs:209)

Workaround:

Open MRTK XR Rig prefab then Camera Offset then select Main Camera in the Hierarchy view
Click on Add Component and manually add Tracked Pose Driver script

To reproduce
Steps to reproduce the behavior:

Create a new Unity project
Import MRTK3
Create an empty scene and remove all content
Add MRTK XR Rig prefab and MRTKInputSimulator to the scene
Hit the Play button and see the warning (Camera "Main Camera" does not use a AR Pose Driver or a Tracked Pose Driver...)  in Console log

Expected behavior
Warning should not appear in Play mode
Your setup (please complete the following information)

Unity Version 2021.3.12f1 & 2020.3.40f1
MRTK Version 3.0.0-pre.12

Target platform (please complete the following information)

HoloLens 2 (OpenXR)
PC/Desktop (OpenXR)

This is something I'm actively tracking with Unity and the ARFoundation team.
Currently there are three "pose drivers" that are usable on the camera:

ARFoundation 4.x only checks for the existence of the first two, while all three are valid for camera pose driving.
ARFoundation 5.x deprecated AR Pose Driver and recommends the Input System's TrackedPoseDriver:

Unfortunately, because Unity 2020 is currently our min version for MRTK3, we can't adopt ARFoundation 5.x (it declares itself to only be valid for Unity 2021+)

Ideally, ARFoundation 4.x will be updated to support any of the three. I'll keep this thread updated as that conversation continues.

Renaming to track the ongoing external bug.

Will be resolved by #11345 and bump to ARF 5.0 and 2021.


[Resilient HTTP apps] Add a paragraph describing how to disable retries for some HTTP methods

The PR adds description how to disable retries for some HTTP methods.
Fixes #5248

Internal previews

📄 File
🔗 Preview link

docs/core/resilience/http-resilience.md
docs/core/resilience/http-resilience

The API described in the PR will be released in dotnet/extensions 9.1.0 (https://github.com/dotnet/extensions). We want to merge the PR only after that release.

Added the DO NOT MERGE label, while this is approved and looks good, we're currently blocked as the corresponding NuGet package version isn't available yet.


Convert big numbers to spreadsheet numbers, where possible

Feature
If the whole number part of a number returned from MultiBaas is less than 2^49, the plugin should convert it to a number rather than leaving it as a string.
The implementation for this might be very simple: just ask MultiBaas to always return numbers as a number, rather than as a string. It's important to validate that this works for numbers converted as part of type conversions as well - some work on the MultiBaas side might be needed. This is probably preferable to attempting to parse numbers that come back via MultiBaas, especially in case they are being returned as arrays, multidimensional arrays, etc.
Background
Generally, what spreadsheet users care about is numbers for the sake of using numbers, not for the sake of reporting them. Google Sheets has an under-specified upper limit on the size of integer or floating point number which can be represented in a cell. The breakpoint for integers appears to be between 2^49 and 2^50:

It is much more convenient to be dealing with numbers in spreadsheets rather than strings, which is how MultiBaas will return large numbers as strings by default, if the data type (pre-decimal shifting) is larger than 48 bits. With decimal shifting in MultiBaas type conversions, and the general size of numbers in practical use (1000 tokens, etc.), the practical size of numbers in general use tends to be well within the range of a spreadsheet. In many blockchain spreadsheet use cases, the whole number part (left of the decimal point) is much more important than the fractional part (right of the decimal point).
That is, losing some precision on the fractional part is probably an acceptable trade-off vs. having to convert all numbers from strings to numbers manually.

Also some related to

@daenamkim good call. For this issue/PR, let's not worry about Event Queries, since those don't yet go through type conversion/the "normal" MultiBaas flow.

Resolved by #55, closing.

Resolved by #55, closing.


enforce linting of all js files

the testing utilities that didn't have the .spec.js extension were
not being linted.

@juttle/developers

+1


after creating a mentee profile, areas of interests are not saved correctly

created a mentee profile
selected special topics
completed application
build profile is asking again for areas of interests
profile is created with no areas of interests and first login is asking user to select it.

@SolMon9099 why you removed the field for Areas of Interests in Build Profile form?

ok it was asked in another issue to remove this field.


Cody failed to start for project inside WSL2

IDE Information
GoLand 2024.1.4
Build #GO-241.18034.61, built on June 21, 2024
Licensed to Duc Hoang
Subscription is active until September 11, 2024.
For educational use only.
Runtime version: 17.0.11+1-b1207.24 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 10.0
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 8
Registry:
debugger.new.tool.window.layout=true
ide.completion.variant.limit=500
suggest.all.run.configurations.from.context=true
ide.experimental.ui=true
go.run.processes.with.pty=false
terminal.new.ui=true
Non-Bundled Plugins:
cn.xiaoheiban.go-zero (1.1.9)
Statistic (4.2.14)
com.jetbrains.twig (241.18034.82)
com.jetbrains.space (241.18034.4)
com.sourcegraph.jetbrains (6.0.14)
Bug Description
Cody failed to start for project inside WSL2
Additional context
Stacktrace:
java.util.concurrent.ExecutionException: com.sourcegraph.cody.agent.CodyAgentException: Creating agent unsuccessful: org.eclipse.lsp4j.jsonrpc.ResponseErrorException: [UriError]: If a URI does not contain an authority component, then the path cannot begin with two slash characters ("//")

Error: [UriError]: If a URI does not contain an authority component, then the path cannot begin with two slash characters ("//")
    at f2 (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:38531:21)
    at e4.t4 (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:38546:88)
    at new e4 (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:38588:40)
    at t4.parse (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:38560:25)
    at _Uri.parse (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:94115:48)
    at C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:138478:49
    at C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:138157:50
    at handleRequest (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:131492:33)
    at handleMessage (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:131359:11)
    at processMessageQueue (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:131378:13)
    at Immediate.<anonymous> (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:131354:11)
    at process.processImmediate (node:internal/timers:478:21)
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
	at com.sourcegraph.cody.agent.CodyAgentService.startAgent$lambda$4(CodyAgentService.kt:192)
	at com.intellij.openapi.application.impl.RwLockHolder$executeOnPooledThread$1.run(RwLockHolder.kt:154)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
	at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1$1.run(Executors.java:702)
	at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1$1.run(Executors.java:699)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
	at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1.run(Executors.java:699)
	at java.base/java.lang.Thread.run(Thread.java:840)
Caused by: com.sourcegraph.cody.agent.CodyAgentException: Creating agent unsuccessful: org.eclipse.lsp4j.jsonrpc.ResponseErrorException: [UriError]: If a URI does not contain an authority component, then the path cannot begin with two slash characters ("//")

Error: [UriError]: If a URI does not contain an authority component, then the path cannot begin with two slash characters ("//")
    at f2 (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:38531:21)
    at e4.t4 (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:38546:88)
    at new e4 (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:38588:40)
    at t4.parse (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:38560:25)
    at _Uri.parse (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:94115:48)
    at C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:138478:49
    at C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:138157:50
    at handleRequest (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:131492:33)
    at handleMessage (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:131359:11)
    at processMessageQueue (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:131378:13)
    at Immediate.<anonymous> (C:\Users\test\AppData\Roaming\JetBrains\GoLand2024.1\plugins\Sourcegraph\agent\index.js:131354:11)
    at process.processImmediate (node:internal/timers:478:21)

Same error with PHPstorm:
Error: [UriError]: If a URI does not contain an authority component, then the path cannot begin with two slash characters ("//")

That is probably related to WSL2 path starting \\, e.g. \\wsl$\Ubuntu\home\user\project\file1.php.
PhpStorm 2<IP_ADDRESS>
Build #PS-242.20224.427, built on August 21, 2024
com.sourcegraph.jetbrains (6.0.34)

As a workaround, I tried to mount WSL2 as network disk and Cody start working as expected, but that is creating many other problems with IDE, files monitoring, other plugins, etc.
Really hope it will be fixed soon.

In the latest version (v7.0.13) Cody now fails completely for projects residing in WSL2 with the same error as above. Reverting to 7.0.6 makes it work again (but no autosuggestions)

Error: [UriError]: If a URI does not contain an authority component, then the path cannot begin with two slash characters ("//")

PhpStorm 2<IP_ADDRESS>
Build #PS-242.20224.427, built on August 21, 2024
com.sourcegraph.jetbrains (7.0.13)

Working fine-ish for me, at least not many errors and autosuggestion working as expected. Still can't apply suggestions, Cody trying to create new file in C:\wsl.localhost\Ubuntu\home\...\filename.php instead of doing it proper place.
Another issue is that I'm always need to have Cody sidebar open and create new chat to ask to "Edit code...", but since version 7 that is at least available for WSL2 projects


Transform fails due to IndexError in middle of parsing

Transform fails in the middle of parsing phenio.owl:
13:39:57  [KGX][owl_source.py][          load_graph] WARNING: Do not know how to handle BNode: Nd63473f9110542d9b5368b973161f582
13:44:34  [KGX][rdf_source.py][              triple] WARNING: Skipping https://w3id.org/biolink/vocab/Publication http://www.w3.org/2000/01/rdf-schema#subClassOf http://org.semanticweb.owlapi/error#Error1
13:44:34  Traceback (most recent call last):
13:44:34    File "run.py", line 85, in <module>
13:44:34      cli()
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
13:44:34      return self.main(*args, **kwargs)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/click/core.py", line 1055, in main
13:44:34      rv = self.invoke(ctx)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
13:44:34      return _process_result(sub_ctx.command.invoke(sub_ctx))
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/click/core.py", line 1404, in invoke
13:44:34      return ctx.invoke(self.callback, **ctx.params)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/click/core.py", line 760, in invoke
13:44:34      return __callback(*args, **kwargs)
13:44:34    File "run.py", line 62, in transform
13:44:34      kg_transform(*args, **kwargs)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/kg_phenio/transform.py", line 37, in transform
13:44:34      t.run()
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/kg_phenio/transform_utils/phenio/phenio_transform.py", line 49, in run
13:44:34      self.parse(k, data_file, k)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/kg_phenio/transform_utils/phenio/phenio_transform.py", line 72, in parse
13:44:34      transform(inputs=[data_file],
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/kgx/cli/cli_utils.py", line 560, in transform
13:44:34      transform_source(
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/kgx/cli/cli_utils.py", line 869, in transform_source
13:44:34      transformer.transform(input_args, output_args)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/kgx/transformer.py", line 229, in transform
13:44:34      self.process(source_generator, sink)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/kgx/transformer.py", line 332, in process
13:44:34      for rec in source:
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/kgx/source/owl_source.py", line 94, in parse
13:44:34      yield from self.load_graph(rdfgraph)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/kgx/source/owl_source.py", line 170, in load_graph
13:44:34      yield from self.triple(s, p, o)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/kgx/source/rdf_source.py", line 228, in triple
13:44:34      (element_uri, canonical_uri, predicate, property_name) = self.process_predicate(
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/kgx/source/rdf_source.py", line 591, in process_predicate
13:44:34      element = self.get_biolink_element(p)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/kgx/source/rdf_source.py", line 847, in get_biolink_element
13:44:34      mapping = toolkit.get_element_by_mapping(predicate)
13:44:34    File "/var/lib/jenkins/workspace/ledge-graph-hub_kg-phenio_master/gitrepo/venv/lib/python3.8/site-packages/bmt/toolkit.py", line 1114, in get_element_by_mapping
13:44:34      lambda s, l: s.intersection(set(l)), without_empty_lists[1:], set(without_empty_lists[0])
13:44:34  IndexError: list index out of range

Since this is saying  list index out of range it would suggest that bmt is unable to get a mapping for a given predicate. Could mean there is a disconnect between the BL model bmt is using and what phenio comes in with.
Anyway, there should be a JSON conversion before this anyway.

Wonder if it's related to this Wikidata relation that keeps following me around (and doesn't parse in robot, but that shouldn't be a showstopper):
<https://www.wikidata.org/wiki/Q2306597> <http://www.w3.org/1999/02/22-rdf-syntax-ns#type> _:genid2149661103.

Wonder if it's related to this Wikidata relation that keeps following me around (and doesn't parse in robot, but that shouldn't be a showstopper):

No I doubt it. This error is a known optical illusion and not an error..

Fixed by #72 - on to the next one.


go-toml: 2.2.0 -> 2.2.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/pelletier/go-toml/releases.
meta.description for go-toml is: Go library for the TOML language
meta.homepage for go-toml is: https://github.com/pelletier/go-toml
meta.changelog for go-toml is: https://github.com/pelletier/go-toml/releases/tag/v2.2.1
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2.1 in filename of file in /nix/store/8iasp02vv2y2gag6g8g949r3g52fq5lj-go-toml-2.2.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
go-toml

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8iasp02vv2y2gag6g8g949r3g52fq5lj-go-toml-2.2.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A go-toml https://github.com/r-ryantm/nixpkgs/archive/4d9ff3606a61b0a28509cb264115d59a95099d66.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/4d9ff3606a61b0a28509cb264115d59a95099d66#go-toml

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8iasp02vv2y2gag6g8g949r3g52fq5lj-go-toml-2.2.1
ls -la /nix/store/8iasp02vv2y2gag6g8g949r3g52fq5lj-go-toml-2.2.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

go-toml

Maintainer pings
cc @marsam for testing.

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

1 package built:

go-toml


RUST_RECURSION_COUNT: error: infinite recursion detected

Describe the bug
rustup keeps track of recursion via the RUST_RECURSION_COUNT
environment variable.
Since cargo make uses up one increment, a chain of

cargo clippy
-> cargo check
-> rustc -vV
-> ...

Uses up to rustup::env_var::RUST_RECURSION_COUNT_MAX and we get the
error message:
[cargo-make][2] INFO - Running Task: clippy
[cargo-make][2] INFO - Execute Command: "cargo" "clippy"
error: process didn't exit successfully: `rustc -vV` (exit code: 1)
--- stderr
error: infinite recursion detected

To Reproduce

a clean repo (no Cargo.lock)
cargo clippy enabled on stable
cargo make ci

Fix
Clear or set RUST_RECURSION_COUNT = 0

cargo make runs commands which are sub processes so they can't update each other or share env vars. feels like a clippy issue.

Oh, you can unset the variable inside cargo-make before calling more cargo stuff.

And it is not a clippy issue...
This patch fixes the issue for us:
https://github.com/enarx/enarx/commit/185d4daa905d4f1a7bb0073365237794e89cdc15
[env]
RUST_RECURSION_COUNT = 0

ya, I'm sure it fixes things, but doesn't explain the root cause.
what I'm trying to understand how can unrelated processes share an env which is not possible.
fact you can workaround it via setting that env, might just hide something else.
i always use all features, including clippy in both crate and workspace projects, and i rum the ci flow before push and never got that.
so the only difference i see it, is different rust/clippy versions.

We use the current versions for stable and nightly. And in only occurs on fresh projects, without target and Cargo.lock.

ya, I'm sure it fixes things, but doesn't explain the root cause.

The root cause is, that cargo make is the first RUST_RECURSION_COUNT++, and that you pass the RUST_RECURSION_COUNT env to the childs, so any job from ~/.cargo/bin/* hard linked to rustup (like cargo and rustc) starts with RUST_RECURSION_COUNT=1 instead of 0.
Running cargo clippy on a complete empty crate seems to use all allowed rustup::env_var::RUST_RECURSION_COUNT_MAX
https://github.com/rust-lang/rustup/blob/bb0fd7b5f56bf5afd030cc2a251bea0c973313ad/src/bin/rustup-init.rs#L96
and with cargo make in front, it exceeds that.

so it is a clippy issue if it maxes it out.
I'm ok with pushing that env var into cargo make internal makefile, just wanted to understand more.
i sometimes add workarounds for other rust/tools in the makefile, so thats another one to workaround clippy bug. so I'll add it specifically to the clippy task, and not in global env. makes sense?

Well, it's a rustup issue. If you call other rustup components they might call themselves rustup components. So, it's not clippy alone.
Will affect also:

clippy
llvm-tools-preview
miri
rls
rust-analysis
rust-docs
rustc
rustfmt

ya, so i did unset on global level. i never ran into this, but cant harm anything i guess...

this is now released in the new cargo-make version 0.32.2

Thanks!


Skip GitHub API checks when the data can't be fetched ARCHBOM-1310

Get around the issue fetching data from GitHub for edx-blog (and any similar problems that may crop up later in the repo health dashboard job).  I'd like to enhance this later to have the fixture report the exact problem encountered without blowing up the test suite, but for now I just want to get the new data in the dashboard for most of our repos.

I captured further work on improving error handling in the GitHub API fixture as ARCHBOM-1361.


Fix passport-trakt recursive resolve

Fixes https://github.com/zeit/node-file-trace/issues/95 which was caused by an error in the pkginfo package.json lookup inlining logic.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #98   +/-   ##
=======================================
  Coverage   90.65%   90.65%
=======================================
  Files          12       12
  Lines        1070     1070
=======================================
  Hits          970      970
  Misses        100      100

Continue to review full report at Codecov.

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


Cannot compile with OpenCL enabled

Hi, I get the following error in kinect2_bridge when I try to compile iai_kinect2 with OpenCL enabled:
/home/samarth/catkin_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘bool Kinect2Bridge::initPipeline(const string&, int32_t)’: /home/samarth/catkin_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:427:28: error: expected type-specifier        packetPipeline = new libfreenect2::OpenCLKdePacketPipeline(device);                             ^ /home/samarth/catkin_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘bool Kinect2Bridge::initPipeline(const string&, int32_t)’: /home/samarth/catkin_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:427:28: error: expected type-specifier        packetPipeline = new libfreenect2::OpenCLKdePacketPipeline(device);
Does someone know what is going wrong?
I am using ROS Kinectic, Ubuntu 16.04. I successfully compiled libfreenect2 v0.2.0. Here's the CMake output from that:
-- using stdlib as threading library -- Linking with these libraries:   /usr/lib/x86_64-linux-gnu/libusb-1.0.so  va-drm  va  /usr/lib/x86_64-linux-gnu/libjpeg.so  /usr/lib/x86_64-linux-gnu/libturbojpeg.so.0  /usr/lib/x86_64-linux-gnu/libglfw.so  /usr/lib/x86_64-linux-gnu/libGL.so  /opt/intel/opencl/libOpenCL.so -- Could NOT find Doxygen (missing:  DOXYGEN_EXECUTABLE)  -- Configurating examples -- Using in-tree freenect2 target -- Feature list: --   CUDA    disabled --   CXX11    yes --   Examples    yes --   OpenCL    yes --   OpenGL    yes --   OpenNI2    yes --   TegraJPEG    disabled --   Threading    std::thread --   TurboJPEG    yes --   VAAPI    yes --   VideoToolbox    no (Apple only) -- Configuring done -- Generating done -- Build files have been written to: /home/samarth/libraries/libfreenect2/build
I have installed OpenCL directly from Intel's website, here's the output of clinfo:
`
Number of platforms                               1
Platform Name                                   Intel(R) OpenCL
Platform Vendor                                 Intel(R) Corporation
Platform Version                                OpenCL 2.0
Platform Profile                                FULL_PROFILE
Platform Extensions                             cl_khr_3d_image_writes cl_khr_byte_addressable_store cl_khr_depth_images cl_khr_fp64 cl_khr_global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_icd cl_khr_image2d_from_buffer cl_khr_local_int32_base_atomics cl_khr_local_int32_extended_atomics cl_khr_spir
Platform Extensions function suffix             INTEL
Platform Name                                   Intel(R) OpenCL
Number of devices                                 2
Device Name                                     Intel(R) HD Graphics
Device Vendor                                   Intel(R) Corporation
Device Vendor ID                                0x8086
Device Version                                  OpenCL 2.0
Driver Version                                  r4.0.59481
Device OpenCL C Version                         OpenCL C 2.0
Device Type                                     GPU
Device Profile                                  FULL_PROFILE
Max compute units                               23
Max clock frequency                             1000MHz
Device Partition                                (core)
Max number of sub-devices                     0
Supported partition types                     by  (0x7F5E00000000)
Max work item dimensions                        3
Max work item sizes                             256x256x256
Max work group size                             256
Preferred work group size multiple              32
Preferred / native vector sizes
char                                                16 / 16
short                                                8 / 8
int                                                  4 / 4
long                                                 1 / 1
half                                                 8 / 8        (cl_khr_fp16)
float                                                1 / 1
double                                               1 / 1        (cl_khr_fp64)
`
(plus a lot more, I can share it if you want)
iai_kinect2 seems to recognize OpenCL OK. Here's the CMake output from when it was compiling kinect2_registration:
-- Using CATKIN_DEVEL_PREFIX: /home/samarth/catkin_ws/devel_isolated -- Using CMAKE_PREFIX_PATH: /home/samarth/catkin_ws/devel_isolated;/opt/ros/kinetic -- This workspace overlays: /home/samarth/catkin_ws/devel_isolated;/opt/ros/kinetic -- Using PYTHON_EXECUTABLE: /home/samarth/libraries/anaconda2/bin/python -- Using Debian Python package layout -- Using empy: /usr/bin/empy -- Using CATKIN_ENABLE_TESTING: ON -- Call enable_testing() -- Using CATKIN_TEST_RESULTS_DIR: /home/samarth/catkin_ws/build_isolated/kinect2_registration/test_results -- Found gtest sources under '/usr/src/gtest': gtests will be built -- Using Python nosetests: /usr/bin/nosetests-2.7 -- catkin 0.7.4 -- Looking for CL_VERSION_2_0 -- Looking for CL_VERSION_2_0 - found -- Found OpenCL: /opt/intel/opencl/libOpenCL.so (found version "2.0")  -- CPU based depth registration enabled -- OpenCL based depth registration enabled -- Performing Test OpenCL_C_WORKS -- Performing Test OpenCL_C_WORKS - Success -- Performing Test OpenCL_CXX_WORKS -- Performing Test OpenCL_CXX_WORKS - Success -- Configuring done -- Generating done -- Build files have been written to: /home/samarth/catkin_ws/build_isolated/kinect2_registration
But it also gave warnings:
In file included from /home/samarth/catkin_ws/src/iai_kinect2/kinect2_registration/src/depth_registration_opencl.cpp:33:0: /opt/intel/opencl/include/CL/cl.hpp: In constructor ‘cl::Image1D::Image1D(const cl::Context&, cl_mem_flags, cl::ImageFormat, size_t, void*, cl_int*)’: /opt/intel/opencl/include/CL/cl.hpp:3582:9: warning: missing braces around initializer for ‘_cl_image_desc::<anonymous union>’ [-Wmissing-braces]          };          ^ /opt/intel/opencl/include/CL/cl.hpp: In constructor ‘cl::Image1DBuffer::Image1DBuffer(const cl::Context&, cl_mem_flags, cl::ImageFormat, size_t, const cl::Buffer&, cl_int*)’: /opt/intel/opencl/include/CL/cl.hpp:3656:9: warning: missing braces around initializer for ‘_cl_image_desc::<anonymous union>’ [-Wmissing-braces]          };          ^ /opt/intel/opencl/include/CL/cl.hpp: In constructor ‘cl::Image1DArray::Image1DArray(const cl::Context&, cl_mem_flags, cl::ImageFormat, size_t, size_t, size_t, void*, cl_int*)’: /opt/intel/opencl/include/CL/cl.hpp:3717:9: warning: missing braces around initializer for ‘_cl_image_desc::<anonymous union>’ [-Wmissing-braces]          };          ^ /opt/intel/opencl/include/CL/cl.hpp: In constructor ‘cl::Image2D::Image2D(const cl::Context&, cl_mem_flags, cl::ImageFormat, size_t, size_t, size_t, void*, cl_int*)’: /opt/intel/opencl/include/CL/cl.hpp:3804:13: warning: missing braces around initializer for ‘_cl_image_desc::<anonymous union>’ [-Wmissing-braces]              };              ^ /opt/intel/opencl/include/CL/cl.hpp: In constructor ‘cl::Image2DArray::Image2DArray(const cl::Context&, cl_mem_flags, cl::ImageFormat, size_t, size_t, size_t, size_t, size_t, void*, cl_int*)’: /opt/intel/opencl/include/CL/cl.hpp:3983:9: warning: missing braces around initializer for ‘_cl_image_desc::<anonymous union>’ [-Wmissing-braces]          };          ^ /opt/intel/opencl/include/CL/cl.hpp: In constructor ‘cl::Image3D::Image3D(const cl::Context&, cl_mem_flags, cl::ImageFormat, size_t, size_t, size_t, size_t, size_t, void*, cl_int*)’: /opt/intel/opencl/include/CL/cl.hpp:4073:13: warning: missing braces around initializer for ‘_cl_image_desc::<anonymous union>’ [-Wmissing-braces]              };              ^
Thanks a lot for creating this library!

Solved: kinect2_bridge is looking for OpenCL**Kde**PacketPipeline, which is only available in the master branch of libfreenect2, not the v0.2.0 tag.
Switching to master branch and recompiling libfreenect2 solved the problem!

I have changed the branch as suggested by samarth-robot, but still having the same problem


Catch, retry and if necessary, warn on timeout errors during schema caching

Previously, when CumulusCI would try to cache information from some orgs about metadata, it would hang due to an underlying platform bug. Now it has its own timeouts and retries and will report the errors if the retries fail.

@prescod a couple of the integration tests failed

@prescod is the remaining integration test failure meaningful?


[BUG] pre-game and post-game not working

doesn't show the agents

when i run a test, it is showing something

but it shows something when i run a test

In wich region are you currently playing your matches?

I've ran a test today and I'm facing the same issue.
I'm running it in EU. Test data shows only one agent as well.

running it in NA and having similar issues. Not able to receive data from the observer tool


How to show a partial view on Dashboard

I want to show a partial view on dashboard for displaying a chart. can you guide me on how to approach this.

Check a MVC guide, there is nothing special about this in Serenity

Can i add another action result in DashboardPage.cs as under:
public ActionResult CityChart()
{
return PartialView("_CityChart");
}


Added new .yml based issue templates 🛠

🛠 Fixes Issue
Closes #150
👨💻 Changes proposed

Changes requested :

i have asked to add Issue templates in the repository instead of old .md based ones
The templates must have the same details as the old ones.

Changes made

GitHub workflow based issue templates are added.
They are of 3 default types -> docs, features, bugs
Blank issue options are true

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

Make changes in the .yml files to customize the issue templates

📷 Screenshots

this looks great!


Extending HealthService to probe pods

@msimecek / @sebader following up on the conversation we had in #98 what do you guys think about adding 1, 2 or more additional LIVENESS_ENDPOINT_x (name is tdb) the healthservice can probe additionally to what it does today.
This will of course not probe each individual pod - it will use the ClusterIP service instead.
stateDiagram
    direction LR
    [*] --> StampHealthCheck
    state FrontDoor {
    StampHealthCheck --> HealthService
    }
    state AKS {
        direction RL
        HealthService
        HealthService --> Application: Alive?
        HealthService --> BackendProcess: Alive?
    }
    HealthService --> CosmosDB: Able to query?
    HealthService --> Storage: File exists?
    HealthService --> EventHubs: Able to sent message?

Treat my implementation just a sample I've used to test that - you can probably implement it better.

@sebader / @msimecek in accordance to our discussion in team I'm going to close this PR. We came to the conclusion that querying the k8s apiserver to check the health of the application is probably the best solution. Feel free to re-use my mermaid diagram.


Until operator usage

I'd like to modify the existing example gr1.py line 112 so that the robot would wait at X0 until the park signal is released. My instinct would be to modify the line to become:
sys_safe |= {'(X (X0reach) <-> X0) || (X0reach U !park)'}

However, doing this results in a long error message list (attached below). Is this incorrect syntax for what I'm trying to achieve?
Traceback (most recent call last):
  File "single.py", line 134, in <module>
    strategy = synth.synthesize('omega', specs)
  File "/usr/local/lib/python2.7/dist-packages/tulip-1.3.0-py2.7.egg/tulip/synth.py", line 1135, in synthesize
    strategy = omega_int.synthesize_enumerated_streett(specs)
  File "/usr/local/lib/python2.7/dist-packages/tulip-1.3.0-py2.7.egg/tulip/interfaces/omega.py", line 66, in synthesize_enumerated_streett
    a = aut.build()
  File "/usr/local/lib/python2.7/dist-packages/omega/symbolic/symbolic.py", line 236, in build
    aut = _bitblast(self)
  File "/usr/local/lib/python2.7/dist-packages/omega/symbolic/symbolic.py", line 360, in _bitblast
    _bitblast_owner(aut, a, 'sys', t)
  File "/usr/local/lib/python2.7/dist-packages/omega/symbolic/symbolic.py", line 398, in _bitblast_owner
    a.action[owner] = list(map(f, aut.action[owner]))
  File "/usr/local/lib/python2.7/dist-packages/omega/symbolic/symbolic.py", line 395, in f
    return bv.bitblast(x, t)
  File "/usr/local/lib/python2.7/dist-packages/omega/logic/bitvector.py", line 40, in bitblast
    return tree.flatten(t=t)
  File "/usr/local/lib/python2.7/dist-packages/omega/logic/bitvector.py", line 455, in flatten
    x = self.operands[0].flatten(*arg, **kw)
  File "/usr/local/lib/python2.7/dist-packages/omega/logic/bitvector.py", line 456, in flatten
    y = self.operands[1].flatten(*arg, **kw)
  File "/usr/local/lib/python2.7/dist-packages/omega/logic/bitvector.py", line 455, in flatten
    x = self.operands[0].flatten(*arg, **kw)
  File "/usr/local/lib/python2.7/dist-packages/omega/logic/bitvector.py", line 456, in flatten
    y = self.operands[1].flatten(*arg, **kw)
  File "/usr/local/lib/python2.7/dist-packages/omega/logic/bitvector.py", line 460, in flatten
    op=Nodes.opmap[self.operator], x=x, y=y)
KeyError: 'U'

The "until" operator U is not in the initial predicate + action + GR(1) fragment (state machine + GR(1) for short) of linear temporal logic (action = formula that contains primed and unprimed variables, no other temporal operators), hence the error. Some LTL operators are expressible in this fragment. Not all LTL operators are expressible.
The until operator is expressible in LTL. The translation is an observer that keeps track of what is going on. The caveat is that to remember what is going on, the observer must save somewhere its observations for reading them later, so it needs "memory". The memory comes in the form of fresh variables that are added to the problem. That is the cost of expressing some temporal operators in the fragment for which we have polynomial-time synthesis algorithms.
In TLA+ those fresh memory variables are called history variables and are existentially quantified with the temporal operator \EE, so they are "internal" variables--they don't constrain the behavior. In the GR(1) literature these variables represent the current node of a deterministic Buchi automaton. They are history variables because the automaton is deterministic.
The observers are called temporal testers. The variable X0reach is the memory of a temporal tester that keeps track of satisfaction of the request-response formula [](park -> X0).
Q: What options do you have if you want to express "until" in GR(1)?
A:

first call an automated translation from LTL to the state machine + GR(1) fragment that constructs temporal testers automatically. The function omega.logic.past.translate does this translation. You will need to call it, for example as done in the function openpromela.logic.map_to_future, which calls map_translate.
Beware that the translation is sound and complete for closed systems (no environment), but in the presence of an environment is only sound, not complete. For some operators it is complete (an "until" on its own--not within a negation--is one of them).

Call the function tulip.spec.gr1_fragment.until_to_gr1, which translates the until operator to a temporal tester.

Write the translation by hand (a good exercise), then compare to the output of the previous options to check your result.

Error explanation
The function omega.logic.bitvector.bitblast takes a formula in first-order logic and returns another first-order formula, expressed in terms of fresh variables (and currently in another syntax than the input formula). Those fresh variables are usually called "bits". The main point about them is that it suffices to reason only about Boolean assignments to those fresh variables in order to reach conclusions about the generated formula, and so about the input formula too.
Q: How does the "until" operator get to the omega package from tulip?
A: The formulae are passed to omega mostly unchanged, except for replacing "string" variable values with integers.
Q: Why is "until" parsed as a binary operator?
A: The parser in omega.logic.lexyacc knows about it.
Q: Why does the parser know about "until" if it isn't going to be bitblasted?
A: Usually parsers are written once to handle more cases than the backend. The reasons are to allow extensibility later, to use the same parser for multiple backends, and because there are context-sensitivity and other properties of languages that are easier to handle in the backend, rather than the frontend (the parser).
Q: Does omega handle the "until" operator?
A: Yes, see comments above.
Regarding first-order logic: In the gr1.py example, the values of the variables that matter for deciding whether the spec is realizable and constructing a controller are Boolean, but at this layer they could have been integer values too.

@txlin did the above answer you question? Can we close this issue?

Yes, thanks for the help


Tooltip does not show up any more after render

Hi,
I want to add nice tooltips to some cells that I have in react-data-grid. So I add the required information to those cells (data-for and data-tip). When I hover on those cells, the tooltip shows up, and all is well.
In my app, I can change the layout of the datagrid by clicking a button (thus triggering a re-render of the whole component which contains the data-grid and the tooltip).
When I do this, the datagrid still contains the cells for the hover, and they still contain the data-for and data-tip. But when I hover over the cells, the tooltip does not show up any more. (The tooltip is still in the DOM, but its style information is not updated any more although I hover over the corresponding cell.)
Any help is greatly appreciated!

Inside the event handler for the button, you might try the ReactTooltip.rebuild() method, to let it re-discover the cells.

Thanks for the pointer! Actually, adding it to the event handler was too early, but I added it to componentDidUpdate(), and this works beautifully! Thank you for your quick and very helpful response.
Nicole


Documentation Update

The documentation for installing via CocoaPods should be updated. It no longer works following the instructions listed.

Can you paste in the error that you run into?

Worked fine for me. If you are having trouble importing the library, then try building your project in Xcode (⌘b).


Added one more known issue

as described in https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/2620

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@scanum it's ready for the merge

@marclauinger - please review.

@marclauinger Any updates on this PR? Thanks

@LolaJacobsen Could you please merge? Thanks!


⚠ Bofrost.lu has degraded performance

In 2bc2fbc, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2194 ms

Resolved: Bofrost.lu performance has improved in f246b2c after 6 minutes.


feat: add parameters for indivdiual rules

Fixes #807
Some SonarSource rules have parameters which determine if the rule needs to be reported or not. See forClasses for example. The changes add the ability to pass such options to a static analyzer via the CLI option - --rule-parameters. The option takes a valid JSON file with the following format:
{
    "<RULE_KEY>": {
        "<RULE_PROPERTY_NAME>": "<VALUE>"
    }
}

This PR, if ready for merge, must be merged after #815 .

I added a command-line option --rule-parameters to mine command. It takes a file object whose content is in the following format:
{
    "<RULE_KEY>": {
        "<RULE_PROPERTY_NAME>": "<VALUE>"
    }
}

I have not added this option to the repair subcommand because the rules we repair do not require configuration as of now.
Please have a look, @khaes-kth .


missing file SPMessage.h

get compile error SPMessage.h file not found in DemoModelData.h

This doesn't exist in the repo.
https://github.com/jessesquires/JSQMessagesViewController/blob/develop/JSQMessagesDemo/DemoModelData.h#L23
Must be something you've added.

This doesn't exist in the repo.
https://github.com/jessesquires/JSQMessagesViewController/blob/develop/JSQMessagesDemo/DemoModelData.h#L23
Must be something you've added.


*Playbulb example which uses gatttool non-interactive mode

*Playbulb example which uses gatttool non-interactive mode
*Added support for multiple playbulbs
*Only send messages when colour change is required

Great thanks!


Rename variable to avoid function name conflict

I was getting the following error while compiling:
netcdf-c/libnczarr/zutil.c:544:26: error: called object 'strlen' is not a function or function pointer
  544 |     if(dnamep) *dnamep = strdup(dname);
      |                          ^~~~~~
netcdf-c/libnczarr/zutil.c:533:68: note: declared here
  533 | ncz_nctype2dtype(nc_type nctype, int endianness, int purezarr, int strlen, char** dnamep)
      |                                                                ~~~~^~~~~~

My interpretation is that strdup() is implemented as a macro which calls strlen() the standard C function, and when that macro is being substituted here the call to strlen tries to "call" the integer variable named strlen.
Resolving this by renaming the integer variable to "len" instead of "strlen", avoiding a conflict with a standard C library function name.

Thanks! This was on my plate to fix this week for something over at conda-forge.  Attn: @dopplershift. Thanks!

@WardF I added this to conda-forge/libnetcdf-feedstock#154.


fix #11: add edit link

The same key can't be used for types and properties because that conflicts for some of them, like the firewall property that returns the firewall type. There's issue #6 open to add the result type to the property key that would fix that though, or you could split up the docs.source.
Also, I think the id generation doesn't match exactly what github does, but it's pretty close for now. Like I think it might be more like replace(/[^\w]+/g, '-'), with maybe a number for conflicts (but not sure on that). This is what marked does anyway, though github doesn't use that library.

I can separate out the source.
I don't think the marked implementation is completely correct either. Like you mentioned, there's a number appended and I'm not sure how it gets chosen. I was hoping to fix this later (since most sections aren't affected by this).

I made some minor changes in 87e7455ddf6f625885d173c1e97ad7d77a040a28. This is now on the site. Thanks!


Provide a way to execute an .ask file from command line, including arguments

If a user wanted to run their .ask file, they can use CLI or Playground and copy-paste the source code, which is inconvenient. Moreover, these methods don't support ask() arguments.
Another way is to run npm run ask <.ask file path>, but this command would only execute a script if user created also a file with expected return value. Moreover, if the user doesn't guess correctly the expected return value, they would get a nasty error from jest:

The task is to propose and write a command which will accept a file path to an .ask file and ask() arguments, and print the result.  It is up to discussion whether the ask() arguments should be provided from command line or rather from a separate file.

This is NOT a duplicate of https://github.com/xFAANG/askql/issues/166 or https://github.com/xFAANG/askql/issues/154

Let's fix npm run ask <.ask file path>. It should print results regardless of whether there is expected result for that file or not. The bug is in implementation of https://github.com/xFAANG/askql/blob/aefa51797af19af36c001c529b962374dddf69b2/src/test.jest.testRunner.ts#L199


Update the template with the latest Amazon ECS-optimized AMI

I think this fixes #3

Sorry for the lag, @lucacome
I've updated to the latest ECS-optimized AMIs in:
https://github.com/awslabs/ecs-refarch-service-discovery/commit/02bf7f35ba6d979813789cf322d55cb36a47e93f


Athena Verify False Option

When creating a client with boto on Athena we have a option to disable the SSL verification by setting verify False. I don't see an option to pass to the boto session. So when I am using the session with Aws wrangler I am always getting SSL errors. Please suggest if there is a way to pass verify False when using Athena with wrangler api

Hi, thanks for opening the issue. We don't recommend disabling SSL and wouldn't want to add that functionality into wrangler as it would expose our customers to MIM attacks and put their data at risk. However, I acknowledge it might simplify the development so here's a couple of things you can do:

Consider still using SSL. The only thing you'd need to do is to add Amazon CA certificates to the trust store of your system.
If the above isn't possible and you still would like to disable SSL you can do this by "monkey-patching" the client factory like below:

import boto3
import botocore
import awswrangler as wr

from awswrangler._utils import ensure_session, _get_endpoint_url, default_botocore_config, apply_configs
from typing import Optional

@apply_configs
def client(
    service_name: str, session: Optional[boto3.Session] = None, botocore_config: Optional[botocore.config.Config] = None
) -> boto3.client:
    """Create a valid boto3.client."""
    endpoint_url: Optional[str] = _get_endpoint_url(service_name=service_name)
    return ensure_session(session=session).client(
        service_name=service_name,
        endpoint_url=endpoint_url,
        use_ssl=False,
        config=default_botocore_config() if botocore_config is None else botocore_config,
    )

wr._utils.client = client  # "monkey-patch" the client

Hope this helps!

I agree we will use SSL enable this is jus to test the dev tests. Following is the issue I am getting
I am able to use below to query the athena fine
athena_client = boto3.client('athena', **credentials, verify=False, region_name="us-east-1")
When I tried
b_session=boto3.session.Session(region_name="us-east-1",**credentials,)
wr._utils.client = client  # "Monkey-patch" the client
wr.athena.read_sql_query(query,database="testdb")
I am getting A communication error occurred: "Connection refused"
Is it some thing I am missing to instantiate the client correctly fro wrangler?

Can you post the full code sample, please?

import boto3
import botocore
import awswrangler as wr
from awswrangler._utils import ensure_session, _get_endpoint_url, default_botocore_config, apply_configs
from typing import Optional
@apply_configs
def client(
service_name: str, session: Optional[boto3.Session] = None, botocore_config: Optional[botocore.config.Config] = None
) -> boto3.client:
"""Create a valid boto3.client."""
endpoint_url: Optional[str] = _get_endpoint_url(service_name=service_name)
return ensure_session(session=session).client(
service_name=service_name,
endpoint_url=endpoint_url,
use_ssl=False,  # Disable SSL
config=default_botocore_config() if botocore_config is None else botocore_config,
)
athena_client = boto3.client('athena', **credentials, verify=False, region_name="us-east-1")
b_session=boto3.session.Session(region_name="us-east-1",**credentials,)
wr._utils.client = client  # "Monkey-patch" the client
This is the full example the athena client works. The credentials we get is after the authentication and include the aws key, secret and token.

Hi, I think there's a bit of confusion - you seem to be using a separate client still. After patching you should be good to use the default client created by wrangler. Here's full code sample that:

Patches the client factory to disable SSL for all clients created by awswrangler
Executes athena query. SSL is disabled at this point because of step 1.

import boto3
import botocore
import awswrangler as wr

from awswrangler._utils import ensure_session, _get_endpoint_url, default_botocore_config, apply_configs
from typing import Optional

@apply_configs
def client(
    service_name: str, session: Optional[boto3.Session] = None, botocore_config: Optional[botocore.config.Config] = None
) -> boto3.client:
    """Create a valid boto3.client."""
    endpoint_url: Optional[str] = _get_endpoint_url(service_name=service_name)
    return ensure_session(session=session).client(
        service_name=service_name,
        endpoint_url=endpoint_url,
        use_ssl=False,  # Disable SSL
        config=default_botocore_config() if botocore_config is None else botocore_config,
    )

wr._utils.client = client  # 1 - "Monkey-patch" the client to disable SSL.

wr.athena.read_sql_query(query,database="testdb")  # 2 - Run Athena query. SSL is disabled at this point.

Please give it a try.

Closing - feel free to reopen if required.


[Monitoring V2 UI] Move Prometheus / Alertmanager from Overview to Navigation Bar

Creating a separate ticket to track this since it affects both https://github.com/rancher/dashboard/issues/1064 and https://github.com/rancher/dashboard/issues/1215.
Context
Currently, clicking on the Alertmanager or Prometheus buttons on the UI shown below takes the user to a URL that is otherwise inaccessible to the average user from the UI. It also takes you away from the monitoring-specific page back to the original Cluster Explorer page, but since Alertmanager and Prometheus are hidden here, no tab is highlighted on the left for the resource.

Instead, we want to modify the Navigation Bar (https://github.com/rancher/dashboard/issues/1215) to now show Prometheus and AlertManager CRDs on the tab for Monitoring itself; since they will be accessible on the tab, we should also remove the colored buttons from the Feature Overview (https://github.com/rancher/dashboard/issues/1064) page.

this is validated in rancher:master-head with the latest UI

Prometheus and AlertManager CRDs show as tabs on the left side
the colored buttons are removed


1.0.30 java.lang.IllegalStateException: Failed to introspect annotated methods on class reactivefeign.spring.config.ReactiveFeignClientsConfiguration

java.lang.IllegalStateException: Failed to introspect annotated methods on class reactivefeign.spring.config.ReactiveFeignClientsConfiguration
upgrade your sample project version to 1.0.30, after starting feign, you can see this error

fixed in 1.0.44


FileTransform V2 Task not documented

The FileTransform task appears to be up to version 2.0.8, but the documentation still points to FileTranform@1

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

ID: 064f48ba-1c34-6422-5fe3-c3e88d2803c4
Version Independent ID: c38f5761-4718-9aae-bc83-063257a78eb3
Content: File Transform task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/file-transform.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

I think this issue should get new awareness, as the v1 task is broken right now and it feels wrong to use an undocumented V2 as the proposed workaround:
https://github.com/microsoft/azure-pipelines-tasks/issues/16821

Thanks for letting us know. This issue is fixed in our new task reference here:
https://learn.microsoft.com/en-gb/azure/devops/pipelines/tasks/reference/file-transform-v2?view=azure-pipelines&viewFallbackFrom=azure-devops
Please check it out and let us know what you think. Thank you.


[Error] crash in only 'prod' environment: you have requested a non-existent parameter "container.build_id".

I tried to publish my site on the production server. Installed the environment in APP_ENV=prod, and the site did not start. An error is displayed:

(1/1) ParameterNotFoundException: You have requested a non-existent parameter "container.build_id". in ParameterBag.php line 100

Аfter 2 days of searching what can cause a crash, I found this bundle. I don't know why this causes an error in the 'prod' environment, but the console also stops working. In 'dev' environment all works fine!
To reproduce error, you can simply clone the Symfony Demo application, and install composer require atom-php/uploader-bundle. No additional configuration is needed - just install the bundle and delete the 'prod' Symfony cache directory.
2 methods help to fix the error:

comment out the line Atom\UploaderBundle\AtomUploaderBundle:: class => ['all' => true], but it doesn't make sense - bundle cant works.
and comment lines in // config/packages/prod/doctrine.yaml, but it works if caching is not used anywhere. The Demo application is used on post pages.

Trying few versions

"atom-php/uploader-bundle": "^1.1@dev"
"atom-php/uploader-bundle": "^1.0.8",

Used Symfony 4.1.8.

Same error with symfony 4 when I try to implement custom compiler pass. Reproduces only in prod environment
$container->addCompilerPass(new CustomPass(), PassConfig::TYPE_AFTER_REMOVING);
My workaround is just add default param in parameters.yaml
parameters:
    container.build_id: ~

Yes, this work for me too!
Thank you!


Create File System Agent in cvpysdk

Hi
Is there any method in cvpysdk that can "Create FIle System Agent" for a decoupled client that was just added?

Hi @crunozaur,
This support is not available currently.
Please raise a CMR through Commvault support for this enhancement.
Regards,
VP


[NEW] More compatible with redis-cli

"redis-cli --pipe" can not work with kvrocks.

Resolved in #452


Can't download last release build 2.1.3

I was used to downloading last build for my Ubuntu from
https://github.com/ariya/phantomjs/releases/download/2.1.3/phantomjs
But now it isn't available.
What's the new way to setup last built binaries?

Any plan for version > 2.1.1 is effectively abandoned. Please see #15344 for more details.


Add API to get user list from permissions in ACL.

My project needed to know the list of PFUsers those can read/write the PFObject.

Thanks for the pull request!
I do think that this API is useful, although I think returning a list of PFUser objects is more useful than a list of object IDs.
We'll need to discuss with the rest of the team on their thoughts, as this would need to be implemented in each of our SDKs.
I think the implementation can also be made much more performant as well.

cc @grantland @nlutsenko @hallucinogen @stanleyw Thoughts?

Hi @richardjrossiii thank you for the response.
I agree performance of the current implementation is not that good for the readability and I will commit improved one later.

There are a lot of nits in the code and block enumeration on dictionaries is actually faster...
Let's have an broad discussion first, though.
Overall, I like the idea! The API naming should be more precise though :)
Additionally, I think it would be nice to split the method into read and write accesses, since they are somewhat different and actually give more possibilities for API usage.
Something like readAccessUserObjectIds and writeAccessUserObjectIds. The only names that pop in my head sound ugly, so it's tough to come up with a good name. The important aspects here is to explicitly mention that the array being returned has objectIds not users (even though we have generic types).
@grantland @hallucinogen Thoughts?

Since @richardjrossiii suggested that it would be better if the methods returned PFUsers instead, what about usersWithReadAuthorization and usersWithWriteAuthorization?

@hhanesand Personally, I like that naming much better, and I think it translates much better as a cross-platform API.


[release/v1.7] Update Go to 1.21.5 and machine-controller to v1.57.4

What this PR does / why we need it:
Update Go to 1.21.5 and machine-controller to v1.57.4 on the release/v1.7 branch (cherry-pick of #2988).
What type of PR is this?
/kind feature
Does this PR introduce a user-facing change? Then add your Release Note here:
- KubeOne is now built with Go 1.21.5
- Update machine-controller to v1.57.4

Documentation:
NONE

/assign @kron4eg

/test pull-kubeone-e2e-aws-default-install-containerd-v1.25.11

/retest

We bumped sonobuoy in https://github.com/kubermatic/infra/pull/2501 from 0.56.16 to 0.57.1, I'm pretty sure that's the reason the jobs are starting to fail.

I think sonobuoy wait is just broken in 0.57. I see https://github.com/vmware-tanzu/sonobuoy/pull/1909 as a candidate for breaking it, but haven't validated that yet. But the same thing happens on a KKP user cluster:
$ sonobuoy run --mode=conformance-lite --dns-pod-labels "app.kubernetes.io/name=kube-dns,app=coredns" && sonobuoy wait
INFO[0000] create request issued                         name=sonobuoy namespace= resource=namespaces
INFO[0000] create request issued                         name=sonobuoy-serviceaccount namespace=sonobuoy resource=serviceaccounts
INFO[0000] create request issued                         name=sonobuoy-serviceaccount-sonobuoy namespace= resource=clusterrolebindings
INFO[0000] create request issued                         name=sonobuoy-serviceaccount-sonobuoy namespace= resource=clusterroles
INFO[0000] create request issued                         name=sonobuoy-config-cm namespace=sonobuoy resource=configmaps
INFO[0000] create request issued                         name=sonobuoy-plugins-cm namespace=sonobuoy resource=configmaps
INFO[0000] create request issued                         name=sonobuoy namespace=sonobuoy resource=pods
INFO[0000] create request issued                         name=sonobuoy-aggregator namespace=sonobuoy resource=services
ERRO[0000] error attempting to run sonobuoy: waiting for run to finish: context deadline exceeded

sonobuoy wait just fails instantly:
$ time sonobuoy wait
ERRO[0000] error attempting to run sonobuoy: waiting for run to finish: context deadline exceeded
sonobuoy wait  0.03s user 0.01s system 14% cpu 0.295 total

Okay, it seems this is intended behaviour of sonobuoy wait? You need to pass sonobuoy wait --wait to actually get that, otherwise the default value of 0 for --wait means that there will be no wait.

/lgtm
/approve

When installing kubeone_1.7.2_darwin_arm64.zip, I still see "goVersion": "go1.21.3"
Shouldn't be go1.21.5 ?


Bug when there is only two images

Hey ! It's me again :)
I've found there is a bug when there is only 2 slides in the image gallery (or 1 as well)
When trying to swipe by hand to the left, the image does not change, and stay at the position it is released.
Good luck for this. If you need more help to reproduce this bug, just ask me :)

Hi. It's not really a bug. Please see this issue: https://github.com/bqworks/slider-pro/issues/5
Cheers

Ty for answer :) Would it be possible at least to make the image come back to position instead of freezing in this particular case ?
Ty :)

You can do that by setting the loop property to false.

Ty for help, and sorry for disturbing you with this so ...

@Poyoman39 @davidghi i love you both ;) it works!


VariantSelector component: nonexistent variants for product options are showing up as isAvailable: true

What is the location of your example repository?
No response
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
2024.7.2
What version of Remix are you using?
No response
Steps to Reproduce
Issue
On hydrogen, the “VariantSelector” component from the '@shopify/hydrogen' package doesn’t seem to be working as expected.
You will see that all options under Length are available:

However, when we click on Length -> 22”, the product is actually unavailable:

You can see on the original shopify theme that these options are being disabled as expected

Observations
This is happening specifically for variants that don’t have these options under their group. For example, width of 1.2mm only has lengths 16”, 18” and 20”:

GitHub Observation
In your github package, I see that the fallback value for isAvailable on the option is set to true:
https://github.com/Shopify/hydrogen/blob/7bf255eb6fd7de83aec360dc54db0dd96623b028/packages/hydrogen/src/product/VariantSelector.ts#L115
Is there any reason why it’s set to true? I would think that if a variant is not specified for that particular option, that it should be false.
Expected Behavior
Option values should have isAvailable set to false if their corresponding variant does not exist.
Actual Behavior
Option values have isAvailable set to true even though their corresponding variant does not exist.

IMHO duplicate of #2155


Create task for alpha feature Share Process Namespace

Document a feature kubernetes/features#495 for configuring sharing a process namespace between all containers in pod. This feature will be released as an alpha feature in 1.10.

This change is

@verb I forgot to mention, you will also need to add the new doc path to the Tasks table of contents (ToC), which is this file: https://github.com/kubernetes/website/blob/master/_data/tasks.yml. Otherwise, the Travis CI check will fail.

@verb what's your schedule for finishing up this doc? Deadline for merges to 1.10 branch for the release is tomorrow (3/9)/. Let me know if I can help in any way.

Oh, also FYI to @smarterclayton, whose concerns about surprising behavior I tried to address in the "Understanding Process Namespace Sharing" section.

/assign @yujuhong


Fixed the problem that get_location_hint_from_locale does not take effect.

Agreed, this seems like a needed fix.
Not sure if that's something @acheong08 considers important, but you should clean up your commit history - there are a lot of useless and duplicate commits in there that probably shouldn't end up in the main repo.
Just run git reset upstream/master once, then re-commit your changes - just don't add a --hard to that command, otherwise your changes will be lost.

Squashed so it doesn't matter


Fix: Wiki_dpr - fix when with_embeddings is False or index_name is "no_index"

Fix the bugs noticed in #1915
There was a bug when with_embeddings=False where the configuration name was the same as if with_embeddings=True, which led the dataset builder to do bad verifications (for example it used to expect to download the embeddings for with_embeddings=False).
Another issue was that setting index_name="no_index" didn't set with_index to False.
I fixed both of them and added dummy data for those configurations for testing.

Hi @lhoestq ,
I am running into an issue now when trying to run RAG. Running  exactly as described here I get the error below. Wondering if it's due to this change.
Running Transformers 4.3.2 with datasets installed from source from master branch.
(venv) sergey_mkrtchyan datasets (master) $ python
Python 3.8.6 (v3.8.6:db455296be, Sep 23 2020, 13:31:39)
[Clang 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from transformers import RagTokenizer, RagRetriever, RagTokenForGeneration
>>> tokenizer = RagTokenizer.from_pretrained("facebook/rag-token-nq")
>>> retriever = RagRetriever.from_pretrained("facebook/rag-token-nq", index_name="exact", use_dummy_dataset=True)
Using custom data configuration dummy.psgs_w100.nq.no_index-dummy=True,with_index=False
Reusing dataset wiki_dpr (/Users/sergey_mkrtchyan/.cache/huggingface/datasets/wiki_dpr/dummy.psgs_w100.nq.no_index-dummy=True,with_index=False/0.0.0/8a97e0f4fa5bc46e179474db6a61b09d5d2419d2911835bd3f91d110c936d8bb)
Using custom data configuration dummy.psgs_w100.nq.exact-50b6cda57ff32ab4
Reusing dataset wiki_dpr (/Users/sergey_mkrtchyan/.cache/huggingface/datasets/wiki_dpr/dummy.psgs_w100.nq.exact-50b6cda57ff32ab4/0.0.0/8a97e0f4fa5bc46e179474db6a61b09d5d2419d2911835bd3f91d110c936d8bb)
  0%|                                                                                | 0/10 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/sergey_mkrtchyan/workspace/cformers/venv/lib/python3.8/site-packages/transformers/models/rag/retrieval_rag.py", line 425, in from_pretrained
    return cls(
  File "/Users/sergey_mkrtchyan/workspace/cformers/venv/lib/python3.8/site-packages/transformers/models/rag/retrieval_rag.py", line 387, in __init__
    self.init_retrieval()
  File "/Users/sergey_mkrtchyan/workspace/cformers/venv/lib/python3.8/site-packages/transformers/models/rag/retrieval_rag.py", line 458, in init_retrieval
    self.index.init_index()
  File "/Users/sergey_mkrtchyan/workspace/cformers/venv/lib/python3.8/site-packages/transformers/models/rag/retrieval_rag.py", line 284, in init_index
    self.dataset = load_dataset(
  File "/Users/sergey_mkrtchyan/workspace/huggingface/datasets/src/datasets/load.py", line 750, in load_dataset
    ds = builder_instance.as_dataset(split=split, ignore_verifications=ignore_verifications, in_memory=keep_in_memory)
  File "/Users/sergey_mkrtchyan/workspace/huggingface/datasets/src/datasets/builder.py", line 734, in as_dataset
    datasets = utils.map_nested(
  File "/Users/sergey_mkrtchyan/workspace/huggingface/datasets/src/datasets/utils/py_utils.py", line 195, in map_nested
    return function(data_struct)
  File "/Users/sergey_mkrtchyan/workspace/huggingface/datasets/src/datasets/builder.py", line 769, in _build_single_dataset
    post_processed = self._post_process(ds, resources_paths)
  File "/Users/sergey_mkrtchyan/.cache/huggingface/modules/datasets_modules/datasets/wiki_dpr/8a97e0f4fa5bc46e179474db6a61b09d5d2419d2911835bd3f91d110c936d8bb/wiki_dpr.py", line 205, in _post_process
    dataset.add_faiss_index("embeddings", custom_index=index)
  File "/Users/sergey_mkrtchyan/workspace/huggingface/datasets/src/datasets/arrow_dataset.py", line 2516, in add_faiss_index
    super().add_faiss_index(
  File "/Users/sergey_mkrtchyan/workspace/huggingface/datasets/src/datasets/search.py", line 416, in add_faiss_index
    faiss_index.add_vectors(self, column=column, train_size=train_size, faiss_verbose=faiss_verbose)
  File "/Users/sergey_mkrtchyan/workspace/huggingface/datasets/src/datasets/search.py", line 281, in add_vectors
    self.faiss_index.add(vecs)
  File "/Users/sergey_mkrtchyan/workspace/cformers/venv/lib/python3.8/site-packages/faiss/__init__.py", line 104, in replacement_add
    self.add_c(n, swig_ptr(x))
  File "/Users/sergey_mkrtchyan/workspace/cformers/venv/lib/python3.8/site-packages/faiss/swigfaiss.py", line 3263, in add
    return _swigfaiss.IndexHNSW_add(self, n, x)
RuntimeError: Error in virtual void faiss::IndexHNSW::add(faiss::Index::idx_t, const float *) at /Users/runner/work/faiss-wheels/faiss-wheels/faiss/faiss/IndexHNSW.cpp:356: Error: 'is_trained' failed
>>>

The error message is hinting that it could be related to this, but I might be wrong. Any ideas?


External Function Support

System should support typechecking and execution of external functions.
The mechanism is described in the specification, section 4.2. That is still being refined.
The implementation is in progress. Typechecking works, at least for the simple test case. Refinement expected.
Execution still in progress.

Base case works. Desired improvements/completions: automatically synthesize validation checks form signatures; deal with conversion of stream valued arguments. Deal with type/shape parameters to external functions.


400 : Bad Request

GET https://graph.microsoft.com/v1.0/me/mailFolders/AAMkAGI1ZTcyZDUyLTUwYjAtNDQ3Zi04MmJiLTk0NDYyMjRjZDg5YwAuAAAAAAAt9u-CPVJiQr1bW65ncf0rAQALZx8mRleUSpRunE2oMh9mAAAAAAEJAAA=/messages?%24select=id%2Cfrom%2CtoRecipients%2Csubject%2CisRead%2CisReadReceiptRequested%2CisDeliveryReceiptRequested%2CparentFolderId%2CsentDateTime%2CreceivedDateTime%2Cbody%2Cimportance%2CcreatedDateTime&%24filter=startswith(subject%2C'te') and startswith(from%2C'huang')&%24orderby=subject asc%2Cfrom desc
Prefer : outlook.body-content-type="text"
SdkVersion : graph-java/v2.2.0
Authorization : [PII_REDACTED]
400 : Bad Request
[...]

Hi @huangdezhou008, we're unable to help you with the information supplied.  If you're looking for the MS Graph Java SDK repo, it lives here: https://github.com/microsoftgraph/msgraph-sdk-java/
I'm going to close out this issue, but feel free to reactivate it with more information or create a new issue if that is easier.  Thanks


AnimatableLabel slideFade(in,right) is not working in TableViewCell in iOS 10

There is an issue when to use slideFade or slide animation inside a TableViewCell in iOS 10.
There are not problems if we don't use those animations within a TableViewCell . Or there are not problems in iOS 9.
------Origin message---------------------
I am currently migrating to version 3.0 and observed that AnimatableLabel with _animationType = "slideFade(in,right)" not working when I have configured in IB but pop(1) is working. Could you please advise.

I just try it in the demo app, it seems to be working as expected for me. Here my setup:

Few questions in order to reduce the potential errors:

What's not working? Is it the Fade? The slide? No animation at all?
Did you try to use slide(in,right) instead?

No animation at all.
I have tried slide(in,right) even this and not working.
Which screen of Demo app you have tried? Let me try as well.

I just added an AnimatableLabel without any customisation in the root controller of the Main.storyboard, then added your setup. Let me know how it goes for you!

One more thing, are you able to reproduce that in a sample project? If so, can you send it to us?

Cocoapods and but it was working nicely with same constraints before.
Sure I will create sample project soon and send you soon.

Which screen of Demo app you have tried?

Just added a label on the top of the first screen, not sure if there's an example using slideFade directly in the interface builder.

I can see working on the home screen. But when I add label into TableViewCell not working. Please see the attached project.
IBAnimatableLabelTest.zip

@dhamukrish I have tried out your demo code, it is not working inside the table view cell. But Pop animation is working fine, I can see the x was right to be 375 (on iPhone 6), but the animation doesn't run. I am not sure it is an issue of iOS 10 or not. Will have more investigations. On the other hand, if you have a static number of cell item, you can use stack view like the demo app does. "Demo Application" -> "Create Account" then you will see the effect.

@JakeLin Thanks for your update. I would rather wait for your fix at this moment.

@dhamukrish thanks, I have tried it in iOS 9, it is working as expected.

I don't know it is a bug of iOS 10 or Apple intended to change the implementation for UIView.animate.
UIView.animate(withDuration: duration, delay: delay, usingSpringWithDamping: damping, initialSpringVelocity: velocity, options: [],
      animations: {
        self.transform = CGAffineTransform.identity
        self.alpha = alpha
      },
      completion: { completed in
        if completed {
          completion?()
        }
    })

But it is still working fine when we do not put it in a table view cell. I am busy on preparing a talk for YOW Connected in about 10 days. Probably, I have to focus on the talk and try to fix this issue after that, sorry for the delay.
I have some idea if you would like to give it a go. We may need to use the new API UIViewPropertyAnimator for iOS 10, I have played around it before in https://github.com/JakeLin/SaveTheDot project. Please let me know if you have any questions, thanks.

Thanks for your update and can wait for your fix.

Can confirm this is still persisting, at least in my app. Did anyone work on it?

@sambing sorry, I haven't got time to look at this issue. Can you confirm this issue only happen in iOS 10 not iOS 9? It looks like an iOS 10 specific issue. Thanks.

I have just tested and I can see still see the same issue on iOS 10 but not on iOS9. Basically the issue is that when I add AnimateLabel inside UITableViewCell with animation type slideFade(in,right).

@dhamukrish I investigated this issue before and found out it is an iOS 10 specific issue. It's not easy to find out why. Apple may change the lifecycle for UITableViewCell. I suggest you use StackView for slideFade(in,right) animation for that moment.

Ok sure

Ok sure

@JakeLin iOS 10 only. Before i read your comment i simply animated the views programmatically and that works fine.

@dhamukrish Would you please try @sambing 's approach. Thanks for your feedback @sambing .

@JakeLin Thank you.

If we look at animate stack call if auto = true (animate on layoutSubviews)
on ios10 we see

UIView.layoutSubviews
UIView.performWithoutAnimation
UIView.layoutSublayers

on ios9

UIView.layoutSubviews
UIView.layoutSublayers

Just to try, enqueueing the animation to main queue allow to play the animation
 func autoRunAnimation() {
    if autoRun {
        autoRun = false
        DispatchQueue.main.async { [unowned self] in
            self.animate()
        }
    }
   }

but the translation to set view at start position of animation is visible
So maybe there is another moment to launch the animation automatically
As a workaround I call an animation in my
 func tableView(_ tableView: UITableView, willDisplay cell: UITableViewCell, forRowAt indexPath: IndexPath) {
the stack of this function has _performWithCachedTraitCollection but not UIView.performWithoutAnimation

@phimage thanks for your workaround/solution for that issue. Do you know why iOS 10 changed the behaviour of that for adding 'UIView.performWithoutAnimation'? Because autoRunAnimation() method is used for everywhere, I think it is too big to change it to have  DispatchQueue.main.async inside. We may suggest the user use your solution for the animation within table view cell.

Closing this now since it's been a while, let us know if you still have any issues.
By the way, I do think we should drop iOS10 support with next release.


Display icons for grsisort histograms in GRootBrowser

As the title says, it would be nice if the GRootBrowser were to show the icons for 1D, 2D, or 3D histograms, even if they are grsisort histograms.
This might mean making a new class inheriting from TRootGuiBuilder, which has a helper method BuildShutter() which apparently is used to create a TGShutter widget (and is the only place the icons h1_s.xpm, h2_s.xpm and so on are used).

I remember looking into this a while back, and I couldn't really find where the icons are selected. So while this might be nice to have, I don't think this is feasible.


[bnb] fix 4 bit SFT

As per title

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.