Can't import data from multiple events

When trying to import data using the python api assigned to multiple events (such as patient id combined with capdis_date) I get the following error:
requests.exceptions.RequestException: {'error': '"5006","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5006","capdis_outcomedate","1997-01-21","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5008","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5008","capdis_outcomedate","2020-10-27","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5010","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5010","capdis_outcomedate","2012-08-01","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5012","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5012","capdis_outcomedate","1995-03-23","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5014","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5014","capdis_outcomedate","2018-04-20","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5016","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5016","capdis_outcomedate","2015-09-07","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5018","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5018","capdis_outcomedate","2010-02-17","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5020","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5020","capdis_outcomedate","2011-07-14","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5022","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5022","capdis_outcomedate","1999-07-20","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5024","capdis_date","1","This field (\'capdis_date\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."\n"5024","capdis_outcomedate","2001-08-28","This field (\'capdis_outcomedate\') exists on an instrument that is not designated for the event named \'Baseline CAPACITY and ISARIC\'. You are not allowed to import data for this field into this event."'}

I don't know whether this is a limitation of the python api or redcap in general. Possibly we have to switch to raw python http requests. Data probably needs to be split up per event but I don't know how these events will then be linked.

Will be solved in PR #10


[Remote Cluster State] Remote state interfaces

Description
Creating generic interface to upload or download a generic object to a configurable remote storage.
Related Issues
#13791
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] API changes companion pull request created.
[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

If the Gradle check fails with error , please merge/rebase the PR from upstream to ensure the post_merge_action field is part of the Gradle check setup. More details in the PR https://github.com/opensearch-project/OpenSearch/pull/13786.


Mkv

.mkv videos format are not working

Yep, because they're not an HTML5-compatible format.
This is worth a read:
https://developer.mozilla.org/en-US/docs/Web/Media/Formats/Video_codecs


plot_orientation: support directed graph bearings

closes https://github.com/gboeing/osmnx/issues/1137
for v2 / https://github.com/gboeing/osmnx/pull/1106
TODOs:

[x] update changelog
[x] add tests / code example
[x] consider directed bearing support in orientation_entropy
[x] ensure tests passing on CI (workflow approval required)

@gboeing looks like I need another workflow approval. If the test passes then this PR is good on my end absent any additional suggestions.

hoping 9cb9c99c7bea8e8913e24881c90f43d53c9a723a will fix test.
unrelated to PR, when min_length=0, it might make sense to skip the data["length"] such that setting length is not required:
https://github.com/gboeing/osmnx/blob/9cb9c99c7bea8e8913e24881c90f43d53c9a723a/osmnx/bearing.py#L212

The first pre-release OSMnx v2 beta has been released. Testers needed! See #1123 for details.


Need to add Juneteenth to US Holidays

Juneteenth National Independence Day
https://en.wikipedia.org/wiki/Juneteenth#Legal_observance
"According to the bill, federal government employees will now get to take the day off every year on June 19, or should the date fall on a Saturday or Sunday, they will get the Monday or Friday closest to the Saturday or Sunday on which the date falls."

I created a PR with the changes applied to the PBIX. The DAX is attached to this comment.
DAX Date Template.txt

I implemented it in v1.15 through a more generic technique that can be applied to any similar holiday. The replaced holiday appears as "(in lieu of Juneteenth)" as for other similar holiday replacements.

I knew there was a more elegant solution--thanks. The only issue is that Juneteenth started in 2021 yet this code will show Juneteenth as a holiday in any year.  Do we need to add a parameter to HolidayParameters for "start year", -1 for all? As time marches on it will be less important, but especially for 2022 it will be noticed in all "prior year" measures that take working days into account.

This is now fixed in 1.18. Thanks!


feat: add WideMul128 hint

Estimated time: 0.5d
Lifespan: 1d

Implement the WideMul128 hint.
Add unit tests, testing the parsing and the logic of the hint.
Add a Cairo program using the hint, used as an integration test .

How-to implement a hint on the Cairo VM TS available here
Rust implementation reference

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am Daniel Barrantes, a fullstack software developer passionate about learning new technologies. I have recently entered the world of web3 and I would like to complete my first issue.
How I plan on tackling this issue
-I'll read the code carefully to understand the task
-I'll work hard to make the code work properly
-I'll implement unit tests and integration tests to verify their functionality

Hello @zmalatrax, can I work on this issue? I would like to help

@coxmars re-assigned

Thanks sir, I will be working on it asap 🫡

Hi @zmalatrax , please is this open for contribution?


Missing support for<EMAIL_ADDRESS>

Used in this configuration: https://github.com/mgrenier/remote-ftp/issues/445

This should be resolved now with the ssh2 rewrite. For more details see here.


Dynamic GE Creation

Received PreReplicatedRemove with no UGameplayEffect def.
Not sure why but I get this error when I create a GE dynamically. This is my code:
        UGameplayEffect* GE_StatusEffect = NewObject<UGameplayEffect>(GetTransientPackage(),
	                                                              FName(TEXT("StatusEffect")));
	GE_StatusEffect->DurationPolicy = EGameplayEffectDurationType::HasDuration;
	GE_StatusEffect->DurationMagnitude = FGameplayEffectModifierMagnitude(FScalableFloat(Duration));

	// Add one new modifier to the list.
	const int32 Idx = GE_StatusEffect->Modifiers.Num();
	GE_StatusEffect->Modifiers.SetNum(Idx + 1);
	FGameplayModifierInfo& ModifierInfo = GE_StatusEffect->Modifiers[Idx];
	GE_StatusEffect->InheritableOwnedTagsContainer.AddTag(FGameplayTag::RequestGameplayTag(FName("Status.Slowed")));
	ModifierInfo.ModifierMagnitude = FScalableFloat(Magnitude);
	ModifierInfo.ModifierOp = ModOp;
        ModifierInfo.Attribute = UMy_AttributeSet::GetMovementSpeedAttribute();

I'm wondering if like, the gameplay effect spec or the gameplay effect is being garbage collected after function call, so it's not being shown. I'm going to try and store them in a TArray marked with UPROPERTY() to see if that fixes it.

Didn't seem to do anything sadly.

Hello, these issues are only for bugs in the sample project or issues directly with the readme. For technical help, I'd suggest trying the Unreal Slackers Discord Server, the official UE Forums, or the UE AnswerHub.

Any solution to this ?

Nope.

FGameplayEffectContextHandle EffectContext = AbilitySystemComp->MakeEffectContext();
EffectContext.AddSourceObject(AbilitySystemComp->GetOwnerActor());
const FGameplayEffectSpec Spec = FGameplayEffectSpec(GEDynamicEffect, EffectContext,1);
FActiveGameplayEffectHandle OutHandle = AbilitySystemComp->ApplyGameplayEffectSpecToSelf(Spec, InPredictionKey);

My man!


RC12 header overflow

Versions

nuxt: rc 12
node: 16 or 18

Reproduction
Iv been struggling to reproduce this on smaller projects, but its definitely only a problem on RC 12,  RC 11 works fine.
Steps to reproduce
I noticed on larger projects that when requesting the router page nuxt RC12 produces a header overflow error on h3. which then causes the entire dev server to crash
What is Expected?
This shouldnt break the server
What is actually happening?

Please use https://github.com/nuxt/framework from Nuxt 3 issues ☺
Also see https://github.com/nuxt/framework/issues/8306


Fix nested logging error

Original thread

I have a data loader code that is running completely fine when running the code within edit window. However, the pipeline is not running when I am running it via schedule. I am getting infinite "" in the VS Code terminal. Not sure why the same code is having two different behaviours in edit window and actual pipeline run. There is only one block in the pipeline that is the data loader.
Setup : AWS EC2

Slack Message

Close it since it's not reproducible so far


Change animations.py to avoid "import * only allowed at module level" error

Also added a .gitignore file.

Thanks for the improvements and fixes!
Regarding .gitignore - it already exists, but I have (foolishly) ignored itself too.
Also, now I've seen I have neglected this repo - I have some uncommited changes I need to push :)


Add Predicate Push Down

WIP: https://github.com/rupurt/odbc-scanner-duckdb-extension/compare/add-predicate-push-down


Nullspace, sign change

Thanks for the work and code repo. I have a query here (maybe a minor one), what is the effect of changing sign based on first basis value (w_basis = w_basis * np.sign(w_basis[0][0])).

Hi, and sorry for the late response.
Scipy's orth uses SVD, and there is an inherent sign ambiguity associated with SVD, see here. However, since the rowspace is is the same regardless of the sign of its basis vectors, we can safely remove this line from the code.

Thanks for the reference :)


visual studio code full screen

basically i was using visual studio code. and some how it was showing buttons for maximize, minimize and close. i clicked on maximize or full screen and from that point its not honoring layouts of any sort. tried everything. any idea?

Some apps use client side decorating title bar so they render themselves the buttons like google chrome or visual studio code. In both you can disable that to remove the titlebar.
In visual studio code for exemple there is an option called Window: Title Bar Style you can set it to native
Is there a little plane next to the app now?
Closing it and reopening it don't make it back to default?

nope closing didn't work. so what i did lougout and log back in. that solves the problem of honoring tiled layout.

I added some code to prevent that coming so often. Tell me if it's help

where is that code?


SDK assumed to be in PATH for "Run Dart Analyzer" command

Running the "Run Dart Analyzer" command produces "[Errno 2] No such file or directory: 'dartanalyzer'" when the Dart SDK is not in the PATH. It seems the problem is that the commands in docgen.py should be using path_to_analyzer (and path_to_docgen) from the SDK class and they are not.

Dart analyzer has been obsoleted. Closing.


Controlling Tool Output while using a newer version of the Robotiq Gripper URCap is not possible

Affected ROS2 Driver version(s)
Prob. all
Used ROS distribution.
Iron
Which combination of platform is the ROS driver running on.
Ubuntu Linux with standard kernel
How is the UR ROS2 Driver installed.
Build both the ROS driver and UR Client Library from source
Which robot platform is the driver connected to.
UR E-series robot
Robot SW / URSim version(s)
<IP_ADDRESS>463
How is the ROS driver used.
Headless without using the teach pendant
Issue details
We have a UR16e with a Robotiq 2f-85 connected to the wrist.
We currently use the Robotiq Gripper URCap on the arm in order to control the gripper. The URCap opens a TCP Socket which allows to remotely control the gripper as well as controller the gripper with the Teachpanel, what is quite convenient.
But when upgrading the gripper URCap from  UCG-<IP_ADDRESS>52 to anything higher we can not select in the Installation tab -> ToolIO -> Controlled by: User, as the URCap will then refuse the interact with the gripper.
I contacted the Robotiq support and UR support. Basically the answer was. Yes this is intended (but they didn't tell me why). BUT: When selecting: ToolIO -> Controlled by:  Robotiq Gripper UrCap it is still possible to control the IO via the script command: set_tool_digital_out (Confirmed by the UR support. I didn' test it myself to be honest).
When taking a look at the external control urscript I can not find this command being used to set the tool io. (But we definitely can via the ROS2 Driver). I guess the IOs are set via a different control method.
So my request is: Support setting the tool IO via the urscript in order to workaround with issues in combination with the Robotiq Gripper UrCap. Perhaps you have a better contact to UR and can figure out why Robotiq was forced to include this check...
Relevant log output
No response
Accept Public visibility

[X] I agree to make this context public

I don't quite understand how this relates to our ROS 2 driver. We use the RTDE protocol to read the robot state, move the arm and set i/os. This did not change.
What exactly do you want to achieve? Why do you need to be able to directly set tool i/os if the robotiq urcap is used?

I see, that makes sense.
We are aware that using RTDE for IOs can lead to problems when integrating with additional equipment (also e.g. if you try to control the UR IOs from a PLC while controlling the joints through ROS). Enabling IO control through a different interface is on our roadmap, but we don't have a timeline for that yet.
@urrsk do you maybe have some more information on this specific conflict with robotiq?

@firesurfer you have a good point and we are aware about it. In general, we are missing a good way to integrate URCaps with the use of the ROS drivers.
On the short term, regarding the digital and analog outputs, we have talked about expending the script_commands thread in the Universal_Robots_Client_Library/resources/external_control.urscript#L496 to also control the outputs as an alternative to use RTDE.

@urrsk
Could you perhaps elaborate why it should make a difference to control an IO via RTDE or via a urscript? In both cases one would need to accept that one can interfere with the settings from an installed URcap (or it has to be blocked). Unfortunately I did't get any useful answer from the UR support why this specific behavior has been changed (That the robotiq urcap may only control the gripper if the IOs are controlled by the urcap) .

@firesurfer Our current way of managing the Tool Connector have it limitations. The intend with selecting one owner is that they are in control. In the past we have seen cases with other tool manufactures tools have been disturbed by the presence of Robotiqs URCap, even when they were selected as owner of the tool connector. That is what Robotiq have fixed here.
Is it an option to use the UCG-<IP_ADDRESS>52 version of the URCap?

@EbbeFuglsang  Thanks for the info.

Is it an option to use the UCG-<IP_ADDRESS>52 version of the URCap?

Thats what we are currently doing. But given that there were some necessary updates in the past I would like being able to run the newest version of the URCap to avoid bad surprises in the future.
(E.g. at some point the combination of the URCap and a newer Polyscope version lead to the gripper shutting down during startup because of an over current condition)


Winget downloads incorrect locale version of SSMS

Brief description of your issue
I have an "ENU" version of SSMS installed on my Dutch locale Windows 11. When I tried to update it with WinGet it downloaded a "DEU" version (German / Deutsch) which then failed to install.
Steps to reproduce
winget upgrade --id Microsoft.SQLServerManagementStudio
Expected behavior
Download correct version (ENU, not DEU)
Actual behavior

In (relevant) text:
winget upgrade --id Microsoft.SQLServerManagementStudio
Found Microsoft SQL Server Management Studio [Microsoft.SQLServerManagementStudio] Version 19.0.2
Downloading https://go.microsoft.com/fwlink/?linkid=2226343&clcid=0x407
Successfully verified installer hash
Installer failed with exit code: 1626
Installer log is available at: C:\Users\Rob\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir\WinGet-Microsoft.SQLServerManagementStudio.19.0.2-2023-04-14-10-54-02.025.log

The problem appears to be in the clcid=0x407 part of the URL WinGet uses to download:
https://go.microsoft.com/fwlink/?linkid=2226343&clcid=0x407
Which results in the DEU version downloaded. When I strip that querystring parameter from the link the correct version (ENU) is downloaded:
https://go.microsoft.com/fwlink/?linkid=2226343
Environment
winget --info
Windows Package Manager (Preview) v1.5.441-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.1555
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.20.441.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

User Settings: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

Other software:
Microsoft SQL Server Management Studio 19.0.1 / 19.0.2

@denelon - This one can probably be moved to pkgs; Confirmed the issue is with the URL
PS D:\Git\winget-pkgs> Get-UrlResponse 'https://go.microsoft.com/fwlink/?linkid=2226343&clcid=0x407'

Name                           Value
----                           -----
ResponseCode                   200
ResponseUrl                    https://download.microsoft.com/download/9/f/8/9f8197f4-0f71-42a3-8717-b2817c77b820/SSMS-Setup-DEU.exe
Headers                        {Server, ETag, Last-Modified, x-ms-blob-content-md5...}
Response                       https://download.microsoft.com/download/9/f/8/9f8197f4-0f71-42a3-8717-b2817c77b820/SSMS-Setup-DEU.exe
Url                            https://go.microsoft.com/fwlink/?linkid=2226343&clcid=0x407
ContentLength                  676960496

PS D:\Git\winget-pkgs> Get-UrlResponse 'https://go.microsoft.com/fwlink/?linkid=2226343'

Name                           Value
----                           -----
ResponseCode                   200
ResponseUrl                    https://download.microsoft.com/download/9/f/8/9f8197f4-0f71-42a3-8717-b2817c77b820/SSMS-Setup-ENU.exe
Headers                        {Server, ETag, Last-Modified, x-ms-blob-content-md5...}
Response                       https://download.microsoft.com/download/9/f/8/9f8197f4-0f71-42a3-8717-b2817c77b820/SSMS-Setup-ENU.exe
Url                            https://go.microsoft.com/fwlink/?linkid=2226343
ContentLength                  659373600


Create a working group for Bento

Likely a temporary working group until it's ready to be merged into an existing working group. But, anticipating aa wide-ranging set of design documents and issues this will benefit from a working group at this time.
Facilitator: @dvoytenko

This was approved in the 2019-11-06 TSC meeting

I'll create the repo and team & update docs.

I have created the wg-bento repo, team and Slack channel.
I also have a PR to add wg-bento to the working-groups README.

Thanks so much, @mrjoro!


Upgrade api to v1.37.0

Description & motivation 💭
Screenshots (if applicable) 📸
Design Considerations 🎨
Testing 🧪
How was this tested 👻

[ ] Manual testing
[ ] E2E tests added
[ ] Unit tests added

Steps for others to test: 🚶🏽♂🚶🏽♀
Checklists
Draft Checklist
Merge Checklist
Issue(s) closed
Docs
Any docs updates needed?

Looks like there's no need to eagerly update the dependency


I18n country params

Testing:

/index.html, /, /give.html, /stats.html all load the correct English data
Selecting a new locale will toggle the English / pseudo translated content
Selecting a country will navigate to page filtered to that country and its donation locations. For France it will correctly load data-fr.json


FileTypeIcon Errors Out and Crashes Page

Thank you for reporting an issue, suggesting an enhancement, or asking a question. We appreciate your feedback - to help the team understand your
needs please complete the below template to ensure we have the details to help. Thanks!
Please check out the documentation to see if your question is already addressed there. This will help us ensure our documentation is up to date.
Category

[ ] Enhancement
[x] Bug
[ ] Question

Version
Please specify what version of the library you are using: v3.18.1
If you are not using the latest release, please update and see if the issue is resolved before submitting an issue.
Expected / Desired Behavior / Question
After upgrading to v3.18.1 I'm getting this error when trying to use the FileTypeIcon component.
TypeError: Cannot add property onClick, object is not extensible
    at FileTypeIcon.render

Observed Behavior
I expected it to render the file icons based on the given path, but it crashes the page.
Steps to Reproduce
Using SPFX 1.19.0, import and use the FileTypIcon component:
  <FileTypeIcon type={IconType.image} path={file.Filename} />

This seems to be related to the additional props that were added to this component in v3.18.0? Pull Request
Thanks!

I rolled back to SPFX 1.18.2 and it also has the same error and crashes the page.


Problem create sticker

i successed send sticker.. but sticker crash in android, but in whatsapp web is working.. how to fix it?

i successed send sticker.. but sticker crash in android, but in whatsapp web is working.. how to fix it?

same problem with me

Works fine here. Show your code.

Works fine here. Show your code.

`stickers,err := os.Open(filename)
if err != nil {
fmt.Fprintf(os.Stderr, "error opening file: %v\n", err)
}
_, err = client.wac.Send(whatsapp.StickerMessage{
Info: whatsapp.MessageInfo{
RemoteJid: message.Info.RemoteJid,
},
Type: "image/webp",
Content: stickers,
})
if err != nil {
fmt.Fprintf(os.Stderr, "error sending stickers: %v\n", err)
}`

the solution is to make a variable, for example i = 0 and when in the handler or play it is given i ++, if the number of the trial reading session is> 10 os.Exit ()
success this way


Add Hbmartin's ruleset to Marketplace

Warnings

:warning:
This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

  Generated by :no_entry_sign: dangerJS against 8b8cb7572c9aa4a4d792afbf0ca322210a4d9b86


Promote shell commands audit to global cop

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

This promotes the shell commands formula audit to a cop that can be applied to all Homebrew repositories, for security reasons.
Multiple arguments is the safer thing to do as it prevents shell injection exploits.
For example, the safety of Utils.popen_read("git -C \"#{resource_path}\" rev-parse -q --verify HEAD") depends solely on whether resource_path is trusted input - which I don't think it is as it is based on the download URL basename. One could potentially redirect a URL to one containing malicious shell code and trick a maintainer into running brew bump-formula-pr. Whether this is actually possible in practice is something I have not fully investigated - it's just best to make the code safe rather than depending on some behaviour in another place, and prevent new code from being exploitable.
Note that calls using shell metacharacters are currently allowed (as it was before with the audit), such as Utils.popen_read("git tag --list | sort -rV"). We should probably however prefer processing the stdin & stdout in Ruby instead.

Review period will end on 2021-03-18 at 15:53:00 UTC.

Note that calls using shell metacharacters are currently allowed (as it was before with the audit), such as Utils.popen_read("git tag --list | sort -rV"). We should probably however prefer processing the stdin & stdout in Ruby instead.

Another common-case that's less easily possible in Ruby is 2>&1 and friends.

Another common-case that's less easily possible in Ruby is 2>&1 and friends.

Maybe this? Utils.popen_read("something", :err => :out)

Sounds like this should work for that scenario: Utils.popen_read("something", :err => :out)

🎉

Review period ended.


Improper syntax highlighting of shell aliases in 1.89.0

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.89.0
OS Version: Sonoma 14.4.1, and also reproduces with vscode.dev

Steps to Reproduce:

Navigate to https://vscode.dev/ or open local VSCode
Create a new file
Set the file Language Mode to "Shell Script"
Copy/paste these two alias definitions:alias ll='ls -alF'
alias ..='cd ..'

Note that syntax highlighting for the second alias is broken

should report to the upstream grammar repo https://github.com/jeff-hykin/better-shell-syntax

Thanks. Reported: https://github.com/jeff-hykin/better-shell-syntax/issues/87

FYI @alexr00


Remove unused code

Coverage increased (+0.4%) to 95.272% when pulling 5f8d6fbcae435d15ea98d76838aa73db75d5c46b on remove-unused-code into a5b953de498b87e8f9382a39c7d878dbf751289d on master.


Untweet and Retweet weren't passing config to authenticatedTwit

Sorry, can you provide further info as to why this config is needed? Thanks.

I remember now – thanks!


[BUG] PDF module does not work out of the box

Describe the feature
The pdf module should call pdf-tool-install during the installation of the module as advised here. Otherwise PDFs wont open properly in doom.

Hm, I just saw https://github.com/hlissner/doom-emacs/commit/4f7bf8b976f88d08a62c40e5b605a51876676fb9
So is there a workaround? Maybe doom doc could warn the user that he/she has to run pdf-tool-install manually?

After some experimenting, I found a way to reliably pull it off. As of fccafff epdfinfo is once again built when pdf-tools is first loaded, which should resolve this issue. Let me know if that isn't the case and I'll reopen it.
Thanks for bringing it to my attention!

Sorry to open an old thread but this issue is relevant, after updating I needed to rebuild epdfinfo each time with with the warning to build it. I ran pdf-tools-install which permanently built it, but each time I open a pdf I still get the warning to build it (it loads instantly as it is already built, just gives a warning every time).

@AethanFoot fixed in f11979e

Awesome, works great.


Error while processing route: home Unexpected token o in JSON at position 1

Hi @mlabieniec ,
I've tried this as per the blog, twice, and am getting this error on the /home route.
Uncaught (in promise) WorkboxError: An error was thrown by workbox with error code: ;'request-not-cached' with extras: '{"url":"http://localhost:4200/assets/failed.png","error":{"name":"invalid-response-for-caching","message":"The fetched response could not be cached due to an invalid response code."}}'
    at http://localhost:4200/workbox-sw.prod.v2.1.0.js:160:1358
    at Generator.throw (<anonymous>)
    at step (http://localhost:4200/workbox-sw.prod.v2.1.0.js:83:30)
    at http://localhost:4200/workbox-sw.prod.v2.1.0.js:96:13

and then
Error while processing route: home Unexpected token o in JSON at position 1 SyntaxError: Unexpected token o in JSON at position 1
    at JSON.parse (<anonymous>)
    at Module.callback (http://localhost:4200/assets/ember-mobilehub-tester.js:14:28)
    at Module.exports (http://localhost:4200/assets/vendor.js:111:32)
    at requireModule (http://localhost:4200/assets/vendor.js:32:18)
    at Class._extractDefaultExport (http://localhost:4200/assets/vendor.js:66242:20)
    at Class.resolveOther (http://localhost:4200/assets/vendor.js:65960:32)
    at Class.superWrapper [as resolveOther] (http://localhost:4200/assets/vendor.js:53562:22)
    at Class.resolve (http://localhost:4200/assets/vendor.js:17906:35)
    at resolve (http://localhost:4200/assets/vendor.js:15303:36)
    at Registry.resolve (http://localhost:4200/assets/vendor.js:14764:21)

and
ember.debug.js:19840 SyntaxError: Unexpected token o in JSON at position 1
    at JSON.parse (<anonymous>)
    at Module.callback (note.js:10)
    at Module.exports (loader.js:106)
    at requireModule (loader.js:27)
    at Class._extractDefaultExport (index.js:391)
    at Class.resolveOther (index.js:109)
    at Class.superWrapper [as resolveOther] (ember.debug.js:42856)
    at Class.resolve (ember.debug.js:7200)
    at resolve (ember.debug.js:4597)
    at Registry.resolve (ember.debug.js:4058)

authentication seems fine for the /home route, and it displays the session expiration, and all jwt id and access fields.
any ideas?
ref:
ember-cli: 2.13.1
node: 8.9.3
os: darwin x64

thanks.. marty

resolution via https://github.com/aws-samples/aws-mobilehub-ember/pull/4


Feature Request - Change Private link DNS zones resource group

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
When deploying private_link_by_service DNS zones, they deploy to a pre-defined resource group as {prefix}-dns. It appears that the resource_group_name cannot be changed in the same way that other connectivity resources can be changed using advanced settings.

Describe the solution you'd like
using the advanced = {} block to pass in custom resource group names to override the default prefix-dns resource groups
Additional context

You can use the advance setting in your connectivity configuration block in the local.tf file. using below snippet.
NOTE: Use of the advanced setting is currently undocumented and experimental. Please be aware that using this setting may result in future breaking changes.
This works perfect for now
... other connectivity configuration

#Advance settings you need to fix the issue.
locals {
  configure_connectivity_resources = {
    advanced = {
      custom_settings_by_resource_type = {
        azurerm_resource_group = {
          dns = {
            ("location1 name") = {
              name = "customize resource group name"
            },
            #Remove location 2 if not needed.
            ("location2 name") = {
              name = "customize resource group name"
            }
          }
        }
      }
    }
  }
}


fix: Adds indexers unit test back and fixes them

Description
For some unknown reason, @bfc/indexers test were not running, this PR adds them back and fixes the tests.
Task Item
closes #8329

Coverage decreased (-0.1%) to 56.306% when pulling fa6debadb4355a9caabec6b4408f481943714085 on sorgh/8329 into 7340f23da884bad454c3c259668e5a65f4cb48b8 on main.


remove arbitrary 600 pixel limit on quick input widget

Add maximumWidth and relativeWidth settings to customize width of quickInput widget rather than the hardcoded minimum of 62% of available width or 600 pixel max width

This PR fixes #85374

@dscherger Wouldn't this make it super-wide on wide screens? I think there is value in having it "in the middle" for easier reading and visual appeal.

Hi @chrmarti thanks for looking at this!
It does make the quick input widget 75% of the width of the vscode window, still "in the middle" of the window but wider than it is now.  Perhaps 75% is too much but for projects with the unfortunate combination of long (40-50 chars), non-unique file names and with paths that have the distinguishing element somewhere beyond the 30 character mark there's currently no way to see which variant of a file to open.
I'm sure there other are changes that could be made to solve this problem but it has been reported several times as #38225, #85374, #90948, #100214 and it would be nice to have a workaround in the meantime.
Other Solutions:

display the full path of the currently selected item somewhere else (tool tip, bottom of window, etc.) without length constraints
move the ... from the end of the displayed path to the front, so that trailing distinguishing elements can be displayed (may fail depending on where the unique element is)
remove display of the filename from the front of the path and display the full path (may still fail for long paths)
allow left/right scrolling of the content within the quick picker to help with navigation & selection (probably the best solution for very long paths)
allow the quick picker to resize dynamically to fit the full path of the currently available elements (may still fail for very long paths)

Improvements to this Solution:
We could preserve the current look by default and add settings to control both the max width (default to 600px but allow unlimited) and the scale factor (defaulted to 62% but allow for 100%). This would allow people who need it to deal with the problem and not affect anyone who doesn't. Of course this may still fail for really long paths but it will handle less extreme situations a little better than they are now.

Thank you so much for this!
Although the settings are a solution, I don't think they're the best solution for this task. I'd prefer to see the quick input dynamically expand with the size of the content or allow it to be resized.
I think we'll want to revisit this when we look at the new "omnibar" experience detailed in https://github.com/microsoft/vscode/issues/115641

This is a fairly minor change that replaces the customize-ui workaround for many users (with all its security implications). The merge broadens the user-base that will give feedback, maybe also usable for the future omnibar.

@isidorn Please push this through. This was my case for using Customize UI. And since 1.74 it's no longer viable https://github.com/iocave/customize-ui/issues/156

@chrmarti Aesthetics are important but not to the degree that they get in the way of functionality. If I can't use the tool well, it doesn't matter to me how pretty it is.
@dscherger Thank you very much for the PR, I look forward to it being merged! The only thing I mildly disagree with is the equally arbitrary 2000 maximum limit. Just let me set it to what I want. If it breaks because the number's too big, the settings can always be changed back.

@chrmarti Aesthetics are important but not to the degree that they get in the way of functionality. If I can't use the tool well, it doesn't matter to me how pretty it is.
@dscherger Thank you very much for the PR, I look forward to it being merged! The only thing I mildly disagree with is the equally arbitrary 2000 maximum limit. Just let me set it to what I want. If it breaks because the number's too big, the settings can always be changed back.

@chrmarti @Moghul
This is definitely a "perfect is the enemy of the good" type of situation.  It's been over 2 years since I put up the original fix and I'd be really curious to know how many people would have benefitted from it over that time while we're waiting for  omnibar perfection.  I haven't read the latest comments in https://github.com/microsoft/vscode/issues/115641 and don't have any sense of whether this is close to landing or if it will ever land.
As far as the arbitrary 2000 character limit goes, I'm not sure if this is a required value in the config or whether it can be left out entirely. It looks like I need to go resolve some conflicts and I'll have a look at the limit while rebasing.

Adding a screenshot here as well to make clear what this is about. The paths in the project I’m working on are not too deep but still line 3 and 4 are indistinguishable from each other. (One is an interface and the other the implementation). I run into this few times a day.

Here's an example why this enhancement is desperately needed

What exactly has to happen for this to be merged and released? It seemed like such a slam dunk, but it's been 3 years.

@aiday-mar @jrieken @amunger @meganrogge @connor4312 What needs to be done to get this merged? Pinging you because you are the top 5 contributors in the last month, we haven't heard back from maintainers in over 3 years, & there is overwhelming community desire for this. FYI, this small PR would solve my #1 gripe with VSCode: I cannot quick search my files because I cannot see the full path 99% of the time.

I would desperately love this as well 🙏

The quick input is really useless in my repo for finding files without this change since everything is so deeply nested and I can see the paths. VA Code core team—please merge this!!

I've been bumping into this myself a lot recently and it's really made me sad that this has never been merged. I've updated the branch, fixed some conflicts and re-tested and it still seems to work fine. This has been waiting now for almost 4 years and it would be great to see it merged even as a temporary solution. I'd be more than happy to see this deleted once a better approach is available but in the meantime, please can we get this merged?
@Tyriar @meganrogge @aiday-mar @jrieken @TylerLeonhardt what do we need to do to get this approved?

This is desperately needed. When working in a large monorepo with deeply nested files, the 600px limit is very restrictive.

This would be a huge QOL improvement! 😀

@Tyriar are we ready to rock on this?

@protoEvangelion spoke with some people in the team and we're a little concerned about adding a setting that will just get deleted. I think the way forward here is to allow the user to resize this by using a Sash on the right side of the quick pick.
You can see a similar interaction in "quick chat" if you have copilot installed:

We would then just want it to be horizontal. How this gets persisted is a bit of an unknown to me still, but my guess is maybe we want to store it in the percentage-based value and do away with the pixel one all together? So most users will configure it purely through the mouse interaction.

I desperately need this. If we can push out a temporary fix, it would be such a QOL improvement. I've been dealing with this pain in my large monorepo for over 3 years. Having come from Visual Studio IDE (which allows resizing of the command palette), this has been a tough adjustment.

Is there any way we can help push this forward? This issue is currently very frustrating for our projects. From what I understand, the only remaining step is obtaining Community PR Approvals. Could you explain the process for this? Is there any way to expedite these approvals?

Is this still relevant?

Is there any reason this wouldn't be relevant? Did a feature come out to make this irrelevant? Have all monorepos been deprecated?


TimeOut Exception not being throw

Hello loentar,
Using staff as a SOAP client I found that when a connection timeout occurs,  the failed operations throws an AssertException instead of the expected TimeoutException.  Debugging this bug, I found the source of the problem in the next line of the ServiceClient Invoker:
if (m_pEnv->error->status_code == AXIS2_ERROR_RESPONSE_TIMED_OUT)
{
  STAFF_THROW(TimeoutException, "Axis2/C client send receive failed (Timeout). error: " + GetLastErrorStr())
}

The bug is the condition of the if statement due that it checks the error code for a time-out with the status code, while it should have been validated using the error_code. I mean:
if (m_pEnv->error->error_code == AXIS2_ERROR_RESPONSE_TIMED_OUT)
{
  STAFF_THROW(TimeoutException, "Axis2/C client send receive failed (Timeout). error: " + GetLastErrorStr())
}

Making this change and rebuilding the staff lib I could check that the TimeoutException is thrown when this error is detected. Please, whenever you can run your own tests in order to verify and apply this fix.
Regards,
Marco
P.S: My apologies for not creating a branch, but I think that this little bug don't  deserves one...

I cannot check it now, but hope it works. Thanks for the tip :)


Add AUTh schools

Closes #117

I went to https://www.auth.gr/search/people and put a common name and the department I want.
I think the way the emails in AUTh work is that you select a username when you create your account, and your mail is<EMAIL_ADDRESS>But it seems that any mails sent to<EMAIL_ADDRESS>will also be delivered normally.

Oh very nice, good job!

If you could also share it with anyone you might know at auth or with any fb groups it would be perfect :)

ok :smiley:


Enhancement: Show callback and type definitions on the navigation bar

Neither callbacks nor type definitions are listed on the navigation bar. (using JSDoc 3.3.0-beta1).

I'd prefer not to add these to the navbar, because I think that would just create extra noise for large codebases.
You should be able to add these to the navbar for your project by modifying the default template.

Thank you for the lead on modifying the template.
About the noise argument, I would argue that it depends on the nature of the project rather than the size. To document an API, callback and type definitions are really important.


💡 [REQUEST] - Test Suite Folder Structure

Start Date
No response
Implementation PR
No response
Reference Issues
No response
Summary
The folder structure seems ok, but the user flow is not.
Basic Example
You can try creating new test suite, new folder and new test. It should be as close as possible to Visual Studio Code's folder structure.
Drawbacks
Bad UX for the first commers. These functions should work properly and smooth like Visual Studio Code or Postman to compete with them.
Unresolved questions
No response

I mean these would be better. Between New Folder and New File buttons, we can add New Suite button.

https://github.com/sprintrest/support/issues/46


rubyPackages: Add back rails

Description of changes
d8358154694 (rubyPackages: add dip, 2022-04-13) commented this out, for reasons I can't entirely figure out - adding it back seems to work perfectly fine, so this does that.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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

[x] Fits CONTRIBUTING.md.

I can't entirely figure out - adding it back seems to work perfectly fine, so this does that.

I'm sorry I should have left a better comment. I removed it because Rails and CocoaPods depend on ActiveRecord, and at that time, both required a conflicting version. This caused to the Bundler resolver to hang forever, and prevent us from updating the gems. For example if CocoaPods requires AR<6, and Rails AR>7, Bundler will not find a valid version.
Anyways, I think we can add Rails again. I just wanted to mention that it is a bit brittle.
( I think the previous update broke .gpgme https://github.com/NixOS/nixpkgs/pull/216647, let's see if it is still failing )
@GrahamcOfBorg build rubyPackages.gpgme

@GrahamcOfBorg build rubyPackages.solargraph rubyPackages.jekyll rubyPackages.rails rubyPackages.gpgme rubyPackages.rubocop


Update master branch dependencies and associated bugfixes

Jumped forward to gdnative 0.10.0 thanks to @bromeon 's changes in #32 I also incorporated the necessary changes to work with egui-themes.
Probably still some more minor things to wrap up before we can do a 0.2.0 release, but any additional testing that folks can do would be greatly appreciated.
Note: At the time of opening this PR, the egui-theme and egui-stylist crates have not yet been published. I'm going to do that shortly.

I've started testing this and am having some trouble with the fonts... Seems to me like partial texture updates are not working properly? I have some buttons with emojis on them, e.g. "👎" and they don't show up at all. If I display egui_demo_lib::DemoWindows and then show my emoji-buttons, all text suddenly becomes vertically stretched. Opening the font book fixes it, maybe because that triggers a full texture update.
Another issue: enlarging the font size in the font book causes a silent crash.

@derivator Thank you for the info. I'll see if I can't test that out.

@jacobsky I can confirm that the texture gets messed up on a partial update. I don't understand the existing code there, but even just getting the texture and setting it again (unchanged) messes it up:
let delta_image = Image::new();
delta_image.create_from_data(
    delta.image.width() as _,
    delta.image.height() as _,
    false,
    Image::FORMAT_RGBA8,
    pixel_delta
);

if let Some(pos) = texture_pos {
    let old_image = texture.get_data().expect("this must exist");
    let old_image = unsafe { old_image.assume_safe() };
    let blit_rect = Rect2 { position: Vector2::ZERO, size: delta_image.get_size() };

    // this should update the texture, but we leave it out for now:
    //old_image.blit_rect(delta_image, blit_rect, Vector2::new(pos[0] as _, pos[1] as _));

    // the next line messes up the texture:
    texture.cast::<ImageTexture>().expect("`ImageTexture` is subclass of `Texture`").set_data(old_image);
    godot_print!("Partial update - broken")
} else {
    // this works fine:
    texture.cast::<ImageTexture>().expect("`ImageTexture` is subclass of `Texture`").set_data(delta_image);
    godot_print!("Full update - works fine")
};

It seems to me like texture.get_data() is returning faulty data?

@jacobsky I can confirm that the texture gets messed up on a partial update.

I've found the issue and will submit a patch later today.

@jacobsky I can confirm that the texture gets messed up on a partial update.
Another issue: enlarging the font size in the font book causes a silent crash.

Both fixed in https://github.com/jacobsky/godot-egui/pull/2.

@derivator Thank you so much! I will get that merged in and then wrap up this PR in the next couple of days.
Sorry for the delay on this, I got caught up in other stuff.


Fix checkstyle issues

Purpose
Fix checkstyle issues in intellij plugin.

Closing the PR since the intellij module was removed from this repo.


Persistence needs to support IAM based DB access

Summary
What change you think needs making.
Motivation
Please give examples of your use case, e.g. when would you use this.
Proposal
How do you think this should be implemented?

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.
- [ ] https://github.com/argoproj/argo-workflows/issues/3569

This would be good to have because managing password is difficult to manage. Also, If role based access is possible then it will easy to access db via trusting role.

Why not to support this:
https://forrestbrazeal.com/2019/02/18/cloud-irregular-iam-is-the-real-cloud-lock-in/

It's better not to introduce vendor-specific code to persistence.
One verified solution for anyone on GCP: use CloudSQL Proxy. Basically you can start your proxy using your IAM user and then specify localhost and empty password "" in your persistence config.


Use the ontology labels for fields that are faceted

In general, values in .text fields should not be used as facets because these are free text fields in the ontology schemas that users can enter anything they wish. The standardized string for these fields is in the .ontology_label  field and these are the fields that should be used to facet.
I understand that as of now, the .ontology_label field is not available for most ontologized fields, but until we get the ontology lookup service incorporated into metadata validation, faceting on .text will continue to show multiple strings for the same value.
In the below example, "10x v2", "10X v2 sequencing", and "Chromium 3' Single Cell v2" are all referring to the same Library Construction Approach. Also, "Illumina NextSeq 500" and "Illumina NextSeq500" are the same Instrument. Users need a unified way to refer to a single value and select it from a list of values.

┆Issue Number: HCAB-272

See also https://github.com/HumanCellAtlas/metadata-api/issues/31
@malloryfreeberg, the indexer extracts the .text field because it is usually the only one that is required. Both, .ontology and .ontology_label are often optional which is why we favored the .text field. Would the following algorithm be preferrable?

Use ontology_label if present
Otherwise use ontology if present and lookup the label using the ontology service
Otherwise use text if present
Otherwise fail

Since we are not able to implement ontology service lookup in the short term, would the following algorithm be a sufficient fall-back in the interim?

Use ontology_label if present
Otherwise use text if present
Otherwise use ontology if present
Otherwise fail

I will let our resident ontologist @daniwelter give her opinion, but I think the second (short-term) approach seems reasonable at this time. It is certainly reasonable to facet on the .text field now since it's the only required field. I'm thinking how to incrementally improve the Browser, though, and doing an ifelse on the various ontology fields is a good next step.
There hopefully will be an ontology breakout at our F2F where maybe this topic can be discussed.

I agree with @malloryfreeberg - as nice as it would be to immediately go for the full ontology look-up, I appreciate that that takes time to implement, so starting with version 2 of the algorithm would be a good starting point. We will try to submit as many datasets as possible with complete ontology mappings and I've actually started adding ontology_label columns to our priority datasets.

@daniwelter, @malloryfreeberg
If we wanted to implement either of the above algorithms generically for all fields for which an ontology exists, would it be safe to apply it to every property whose name is text? Are there textual fields that are not called text but that have sibling ontology_label and ontology?

Great question!
I just had a scan through the metadata schemas, and it's true that the text field only exists in the ontology schemas where they have the sibling fields ontology and ontology_label.
However,
I would not be comfortable making this assumption in general about the metadata schemas. The safer option is to introspect the schemas to determine which fields are ontologized (e.g. a field that references an ontology schema).
In order to enable the algorithm to function generally for all ontologized field in the short term, I think it would be ok to apply it to all properties named text.
I'll also note (and I didn't know this until today) that this topic will be heavily discussed with @daniwelter and @simonjupp at the F2F, so it would be great if you could be at that sessions, @hannes-ucsc :)

+1 on everything @malloryfreeberg said.

Solved by humancellatlas/metadata-api#31.


Security Mechanism's Vulnerabilities

According to ROSE, only Object at Risk and Risk Enablers have Vulnerabilty. However, Security Mechanism also have Vulnerability.

In principle, Object at Risk, Security Mechanism, and Risk Enablers are not disjoint types. So, a Security Mechanism can also be an Object at Risk.


chat: mark-as-reads out of sync

Similar to #1172.
Log from @joemfb: https://pastebin.com/qmHYPYQW

@jamesacklin this looks more like the issue I put up #1245. I don't see anything about mark as reads from this log.


Add more logging to the results.

Such that logs are part of the results.

dont merge. Not sure if worked.

Worked


feat: generative user interface

https://github.com/miurla/morphic
https://morphic.sh/
Morphic is a "An AI-powered answer engine with a generative UI." This would be a cool addition to Open-webui.
Describe the solution you'd like
A toggle switch to enable/disable this feature.
Additional context

So like Perplexity.AI? Or if we're strictly talking something open-sourced like you have mentioned, there's also this I've stumbled across these: ClarityAI & Omniplex


http: fallback to Eyre on failed cache scry

Haven't tested this, pushing early to get feedback on the approach. Basically, we put [~ ~] scries into the cache just like any other noun, and if _http_cache_respond sees [~ ~], it falls back to _http_req_dispatch.
Resolves https://github.com/urbit/urbit/issues/6582

Updated: now, instead of falling back to Eyre, vere returns a 404, matching Eyre's (current) behavior. We discussed modifying this behavior to support truly "unbinding" a URL, as currently it's not possible to make a URL dynamic again after it's been bound in the cache. We also discussed changing Eyre's %cache scry API to properly distinguish between unbound and never-bound paths. Both of these would have to happen in a future kelvin.
Also incorporated the auth checking from https://github.com/urbit/vere/pull/199, so authorization actually works on cache entries now.

@lukechampine I pushed a first draft of state-tracking for these requests to jb/eyre-cache-state. Haven't had a chance to test it yet.


Search By Description & Various Fixes

This PR adds the ability to search for transactions via their description.
This PR also includes some fixes for applying custom currency in account settings and group filters.
Search By Description
GNOME
Account Open

Search

WinUI
Account Open

Search

Closes #203
Closes #225
Closes #226

Ok this PR is ready for review.

Ready.

Also made the total string match the way transaction/group amounts are displayed with +/-:


Update to meteor 1.10

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Replaces https://github.com/nrkno/tv-automation-server-core/pull/151
Since Meteor 1.10 has now been released, it's time for us to upgrade the platform to it.
This gives us several improvements, including Node 12 and Typescript 3.8

What is the current behavior? (You can also link to an open issue here)
This is should not affect any behaviour of the system.
However, when upgrading a system to this, mongodb will have to be updated.

Key changes

meteor run no longer reports any typescript errors. Instead there are check-types and watch-types npm scripts which invoke tsc to check the typings. This is run as part of ci and will cause a failure if it does not pass.

When meteor starts it no longer loads every file under the server or client folder. Files must be imported from the main.ts/main.tsx file. When adding some code that needs to be invoked at startup, it must be referenced explicitly of those files, to ensure it gets loaded. TODO: should we restructure the files to make this more obvious and clearer?

This bumps the version of mongo significantly. Meteor makes no attempt at startup to upgrade the data and will instead crash as mongo will fail to start. The easy fix is to take a system snapshot before upgrading, delete your local .meteor/local folder to wipe the db and then reimport the snapshot. See the upgrading section below for full steps

To ensure that your node_modules is all updated correctly, it is recommended to delete node_modules, run meteor --version then meteor npm install. This ensures that npm and all the dependencies are installed with the correct version of node and npm.
If you get npm ERR! Unsupported URL Type "npm:"<EMAIL_ADDRESS>this means that it is using an old version of npm

Upgrading
Production
As mongodb needs upgrading, this might be possible to do online if you have a replicaset with more than one node, which is best described by the mongo documentation. I do not know how meteor will handle the mongo version changing, especially over such a big jump so we recommend doing an offline upgrade, but hopefully in future the jumps will be small enough for it to be possible while online

Stop sofie core
Upgrade mongodb. You need to first upgrade to 3.6, then on to 4.0, and finally to 4.2
Startup the new sofie

Development
The easiest option is to do a clean checkout of sofie and either import a system snapshot, or to reconfigure from scratch.
If you want to upgrade in place (after this, the db will not work in older meteor):

With a meteor 1.6 branch, do meteor update --release 1.7
Start sofie. This is expected to fail to startup, but will do the mongo upgrade to 3.6
Once mongo is running, in a mongo shell execute db.adminCommand( { setFeatureCompatibilityVersion: "3.6" } )
do meteor update --release 1.8.1
Start sofie. This is expected to fail to startup, but will do the mongo upgrade to 4.0
Once mongo is running, in a mongo shell execute db.adminCommand( { setFeatureCompatibilityVersion: "4.0" } )
checkout the meteor 1.10 branch
meteor --version to get the latest meteor tools setup
Delete your node_modules folder
meteor npm install to install dependencies
Startup sofie. This time it should be successful

Known bugs

[ ] Rundown view pieces missing and adlibs colours
[ ]

Status

[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3d438b4). Click here to learn what that means.
The diff coverage is 70.00%.

@@            Coverage Diff            @@
##             master     #239   +/-   ##
=========================================
  Coverage          ?   63.00%
=========================================
  Files             ?      156
  Lines             ?    10720
  Branches          ?     2369
=========================================
  Hits              ?     6754
  Misses            ?     3916
  Partials          ?       50

Impacted Files
Coverage Δ

meteor/lib/api/methods.ts
88.46% <ø> (ø)

meteor/lib/api/rundownLayouts.ts
66.66% <ø> (ø)

meteor/lib/collections/PartInstances.ts
100.00% <ø> (ø)

meteor/lib/collections/Parts.ts
47.36% <ø> (ø)

meteor/lib/collections/RundownPlaylists.ts
59.01% <ø> (ø)

meteor/server/DatabaseCaches.ts
97.79% <ø> (ø)

meteor/server/api/ingest/rundownInput.ts
68.03% <ø> (ø)

meteor/server/api/playout/adlib.ts
38.80% <ø> (ø)

meteor/server/api/rundown.ts
47.93% <ø> (ø)

meteor/server/coreSystem.ts
56.66% <0.00%> (ø)

... and 192 more

Continue to review full report at Codecov.

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

@jstarpl What was the motivation to use legally here? After a quick look at it myself I am not impressed with it.
Firstly, after installing and running  yarn legally I get 1: Syntax error: "(" unexpected. I have to invoke it with yarn node ./node_modules/.bin/legally for it to work. Its missing the #!/usr/bin/env node line that scripts need to be runnable like that on *nix. That I could live with though
Secondly, if a license doesn't meet the criteria it does report that, but then it exits with OK rather than an error, so circleci will never fail.
eg:
yarn node ./node_modules/.bin/legally -r --filter GPL --plain --border ascii
yarn node v1.22.4
Working on "node_modules". It might take a while...

   +------------------------------------------------------------------------+
   |                                                                        |
   |                                Reports                                 |
   |                                                                        |
   +------------------------------------------------------------------------+
   | 100% of the dependencies are not "GPL" (951/951)                       |
   +------------------------------------------------------------------------+

Done in 4.62s.

vs
yarn node-license-validator -p -d --allow-licenses MIT
yarn run v1.22.4
$ /home/julus/Projects/sofietv/circle-test/node_modules/.bin/node-license-validator -p -d --allow-licenses MIT
Identified licenses: 0BSD, MIT
Invalid license<EMAIL_ADDRESS>0BSD
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Hmm... that's weird, I thought that it was something wrong with node on my system that made the script broken. The motivation was that node-license-validator is dead and needs to be replaced with something that is still maintained. I guess I'll wrap legally in a wrapper script, that will resolve both the execution problem and the exit code problem.

that sounds like a good plan, we do need to do something about that issue eventually.
That wrapper sounds like something we could later bundle in the eslint & tsconfig config package we want to make.

Merged into https://github.com/nrkno/tv-automation-server-core/pull/245


Static test method in abstract class runs repeatedly

TestNG Version
6.11
Expected behavior
When I want to test static methods in an abstract class, and also provide heritable test cases for the instance methods, I can create an abstract test class with static @Test methods to test the static methods, instance @Test methods to test the instance methods, and an abstract method that instantiates the class under test. The instance methods are run once in each concrete subclass of the test class; the static methods are run only once.
Actual behavior
The static test methods are run as many times as the abstract test class has concrete subclasses.
Is the issue reproductible on runner?

[ ] Shell
[X] Maven
[ ] Gradle
[ ] Ant
[ ] Eclipse
[ ] IntelliJ
[ ] NetBeans

Test case sample

Please, share the test case (as small as possible) which shows the issue

public abstract class Testable {
  public static String staticMethod() {
    return "Hello from a static method";
  }
  public abstract String instanceMethod();
}
public class RedTestable extends Testable {
  @Override
  public String instanceMethod() {
    return "Hello in red";
  }
}
public class BlueTestable extends Testable {
  @Override
  public String instanceMethod() {
    return "Hello in blue";
  }
}
public abstract class TestableTest {
  public abstract Testable createTestable();
  @Test
  public static void testStaticMethod() {
    Assert.assertTrue(Testable.staticMethod().startsWith("Hello"));
  }
  @Test
  public void testInstanceMethod() {
    Assert.assertTrue(createTestable().instanceMethod().startsWith("Hello"));
  }
}
public class RedTestableTest extends TestableTest {
  @Override
  public Testable createTestable() {
    return new RedTestable();
  }
}
public class BlueTestableTest extends TestableTest {
  @Override
  public Testable createTestable() {
    return new BlueTestable();
  }
}

A workaround I'm using at https://github.com/Pr0methean/BetterRandom/blob/dead28df1f482e73681b7c3c14a17a4c856e84c4/betterrandom/src/test/java/io/github/pr0methean/betterrandom/prng/BaseRandomStaticTest.java is to create a second test class for each affected abstract class under test, make it concrete and move the static-method tests there.

@Pr0methean - I am not sure if this is a problem with TestNG. Here's why.

TestNG makes use of Reflection to determine the methods that are to be run.
When it comes to reflection, it doesn't make any sense for TestNG to distinguish between static methods and non static methods, because TestNG is always going to create an instance of your test class (the class that houses your @Test method) and then run the methods within it using reflection.
If TestNG were to distinguish between static and non-static methods, then TestNG has to provide different execution mechanisms for both and am guessing that is going to cause an avalanche of changes (which am not sure is worth the effort)

Couple of questions for you :

Why would one want to define static methods and brand them as static test methods ?
Why can't these methods be instance methods which perhaps internally still test static implementations ?

IMO, this is not a problem with TestNG.
ping @juherr @cbeust - Please chime in with your thoughts. Based on your comments, we can proceed further with this issue.

The point is that instance methods should be tested in every subclass, since they can be overridden or can call overridden methods, but static methods should generally only be tested once since they won't change. If it is indeed essential for TestNG to instantiate every test class whose methods it runs, even when it only runs static ones, then my two-test-class workaround may be the best solution for everyone affected (although it should probably be mentioned in the documentation).

but static methods should generally only be tested once

Agreed and makes sense. But the question is, against which class should it be run ? The child classes to which it has visibility (or) should the parent class be taken into consideration ? On the whole it creates a lot of confusion IMO.

ping @juherr - IMHO we can close this issue. WDYT ?

@cbeust WDYT? What is the expected behavior for static tests involved in inheritance?

@cbeust - Please advise so that we may proceed further accordingly.

I think arguments can be made both ways, whether static methods should be run once or multiple times, so TestNG is doing the right thing.
If you want to add this semantic to the meaning of static in the context of tests, you could add a boolean that makes sure these test methods only run once.

Closing this issue based on the feedback provided by @cbeust


Window freezes when pasting (when there is a pending update)

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.80.0
OS Version: Darwin arm64 22.5.0 / macOS Ventura 13.4.1

Disclaimer: This issue is very much from my own experience. I am unable to fully identify (and thus prove) the source of the issue, but I am creating it here just in case there is a solution.
I run into this issue sometimes where trying to paste into a file (no matter the length of the clipboard contents) will cause the window to freeze and hang for a few seconds (sometimes up to 20 or 30 seconds). During this time I will be trying to do other things, such as clicking around the file or in other files, with no response. Finally, once the hang stops, the cursor will go to where I last clicked (or where I last navigated to with the arrow keys) (basically, the window will update), but the content will not be pasted where I originally directed it to. I will then try again and it will immediately work.
This happens seemingly randomly, and I have been dealing with it for over a year now. I notice that it only seems to happen when there is a pending update in VS Code (i.e., "Restart to Update" prompts). I had previously looked up this issue and saw a suggestion to first update VS Code to the latest version (obviously), then report back if the issue persisted. (I believe the answer was on Stack Overflow for a slightly different problem.) This is clearly the natural thing to try if there are any problems, and this always fixes the issue, at least for a bit. However, I'm not sure why a pending update might cause this hanging behavior, and it would be nice for it not to happen while I wait for a good time to update VS Code. It also always comes back, which is annoying.
A similar issue occurs when trying to save a file too. I thought it might be a format-on-save thing, but I was working in some files which don't have formatters and the window still froze (and I also disabled all extensions, so I wasn't getting formatting anyway).
I don't really have any steps to reproduce, as it does happen randomly on my end too, but thought I could bring it to someone's attention to try to see if there's a solution other than forcing to update to the latest version (which I'm going to do eventually anyway, but am holding off on for now because of my currently open projects and editors).
Thanks for your help!

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.81.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Can you follow the steps over here whenever the issue hits again? https://github.com/microsoft/vscode/wiki/Performance-Issues Thanks

Thanks, I will update if it comes up again. I am now on version 1.81.1.

Hi, the issue is happening again, but as I mentioned before it happens randomly and it's difficult to reproduce. Thus I wasn't really able to follow the steps you linked. Honestly I'll probably just keep VS Code up to date because that's really the only solution that's worked and there isn't a good reason for me not to update. Thank you for helping me with this.


Release azure-cli-cognitiveservices 0.1.4

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Started releasing azure-cli-cognitiveservices 0.1.4...

Release of 'azure-cli-cognitiveservices' with version '0.1.4' successful. View at https://pypi.python.org/pypi/azure-cli-cognitiveservices/0.1.4.

GitHub release created. https://github.com/Azure/azure-cli/releases.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3680   +/-   ##
=======================================
  Coverage   72.23%   72.23%
=======================================
  Files         422      422
  Lines       26357    26357
  Branches     3991     3991
=======================================
  Hits        19040    19040
+ Misses       6089     6086    -3
- Partials     1228     1231    +3

Impacted Files
Coverage Δ

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
72.1% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Design improvement ideas

Clean looks:

https://jezhumble.net/
https://blog.jessfraz.com/

Loving the years-as-headings at https://varunksaini.com/

Loving the typography and color scheme at https://julian.digital/2020/02/23/my-quantified-self-setup/ :

Loving the structure + projects separation

Source: https://themes.gohugo.io/theme/hugo-theme-cactus/

Timeline approach is visually appealing, albeit informationally sparse:

https://themes.gohugo.io/theme/hugo-spectre-pixel-theme/

I like the TOC-on-the-side approach:

https://michael.stapelberg.ch/posts/2020-11-30-scs-processing-microcontroller/


ImportError: cannot import name 'compat_cfg' from 'mmfewshot.utils'

Hello! I have encountered the same problem as you. Have you solved this problem?

同问，请问楼主解决了吗？


🐞 [Bug]: The Governance link in the dashboard footer doesn't work properly

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?

When I try to click on the Governance link in the dashboard footer, It redirects me to a broken link.

https://github.com/user-attachments/assets/4748be20-181c-4aef-898e-ef2b1c9e4421
Steps To Reproduce
No response
which network/s did you face the problem on?
Dev
version
038404b
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records
n/a
Relevant log output
n/a

Blocked on: https://github.com/threefoldtech/info_grid/issues/660

Work Completed:
Fix broken governance link

Verified on Dev,
Version: 56ca263

https://github.com/user-attachments/assets/da7c83b2-e299-4866-90c3-400327f1716a


Can`t get route segment in custom middleware

After initialized mvc in startup.cs it is not possible get route segment in custom middleware.
Related issue aspnet/Localization#122

@Codenator81 routing happens after middleware runs, so I think this ordering is incorrect. We are looking to enable middleware to run as a result of routing in https://github.com/aspnet/Routing/issues/224, and I think that would enable you to do this.

@Eilon Great news! This will be very helpful for many scenarios


Butterknife Gradle Plugin + AGP 3.3 Alpha 12 fails due to R.jar generation

Using AGP 3.3 alpha 12 with android.enableSeparateRClassCompilation = true and Butterknife's Gradle plugin 9.0.0-SNAPSHOT causes a build error.
On my machine, it generates the following while compiling a module:
Failed to capture fingerprint of input files for task ':august-commons:generateDebugR2' property '$1' during up-to-date check.
> Could not get file details of /Users/parthpadgaonkar/luna/august-commons/build/intermediates/compile_only_not_namespaced_r_class_jar/debug/generateDebugRFile/R.jar/com/august/luna/commons/R.java: could not stat file (errno 20: Not a directory)

I suspect this is due to 3.3a12's new R.jar generation, since if you set enableSeparateRClassCompilation=false, everything compiles just fine.
cc @kageiit

The same problem with 9.0.0-Rc1

I have the same problem with AS 3.3 Beta2 and Butterknife 9.0.0-rc1.


access token can not be null, and display still showing connecting.

Access token can not be null sounds like you are not retrieving and setting your access token that should be generated from firebase.
See the example
Line 58 makes the call for the access token from firebase
Line 66-67 sets the access token to Twilio

Closing due top lack of responses, feedback or activity.


Firmalife failed to load correctly with the latest version of TFC

Here are some logs for the crash with only TFC and firmalife installed
firmalife_tfc_update_crash_log.txt
firmalife_tfc_update_crash-latest.log

Had the same issue as you, if you are on the latest version of terrafirmcraft (1.20.1 - 3.2.0) rolling it back to the previous version (1.20.1 - 3.1.9) seems to fix the issue.

Had the same issue as you, if you are on the latest version of terrafirmcraft (1.20.1 - 3.2.0) rolling it back to the previous version (1.20.1 - 3.1.9) seems to fix the issue.

Fix worked for me. Thanks


Feature request: enforce multiple thresholds

Hello,
Just an idea that I had today when I ran into the commit -n scenario (this is the N time with N > 50), when trying stuff in a project and commit a partial advance.  I want to have a different coverage threshold on certain files (jsx in this case) because of a decision (let's say 50%). But I want 90% on the rest of my code. And I think that it may apply also, as a second scenario, for environment dependent thresholds.
I'm thinking in something like:
"nyc": {
  "thresholds": {
    "dev": [
        {
          "include": ["**/*.js"],
          "exclude": ["**/*.jsx"],
          "lines": 90,
          "statements": 90,
          "functions": 90,
          "branches": 90
        },
       {
          "include": ["**/*.jsx"],
          "exclude": ["**/*.js"],
          "lines": 50,
          "statements": 50,
          "functions": 50,
          "branches": 50
        }
    ],
   "prod": [
        {
          "include": ["**/*"],
          "exclude": [],
          "lines": 90,
          "statements": 90,
          "functions": 90,
          "branches": 90
        }
    ]
  }
}

This way, when I run the CLI with a --threshold=dev argument it will enforce that threshold.
Expected Behavior
Support multiple enforce threshold configurations with include/exclude globs. As I said this can be also useful if we have different threshold among different environments. You may want to stop using -n in development branches when doing a git commit that is just a checkpoint for the day, because you can play in a wider range. And, if that didn't brake the test, it is conceptually ok because you know that your tech debt on that environment is higher. And also, it is annoying to waste some time waiting for that, and even maybe receive unuseful alerts from the CI server.
Observed Behavior
Currently it works with a global threshold enforcement configuration.
Bonus Points! Code (or Repository) that Reproduces Issue
Since this is not an issue but an idea, it just the regular behavior. I will avoid the rest of the template because of this.
Thanks for even reading!

I just realized I can override those configurations via CLI args. Closing this.

@nicosommi 👍 awesome; did this approach work for you?


fix PushPredicateThroughNonJoin rule not work with column alias

we should rewrite alias column to real expr, such as:
select t.a from (select * from t1 where a > 1) t where t.b > 7;

already fixed by https://github.com/Fedomn/sql-query-engine-rs/pull/80
at src/optimizer/rules/pushdown_predicates.rs https://github.com/Fedomn/sql-query-engine-rs/pull/80/files#diff-5acf4bbc2911e286d71dbb5120ec47f6431bc6d4b15f72c249c12687c2d27bbcR219


Properly handle nullable type

actual is a nullable type but the K1 compiler ignores that when referencing actual in functions in the class. This is a compilation failure in K2. Backporting the fix as the new behaviour is correct.

Warnings

:warning:
It looks like this PR contains functional changes without a corresponding test.

  Generated by :no_entry_sign: dangerJS against 5ab1777ab53d0bd8ee0cefc0d7dfd92e1ad50951

@3flex I'm re-adding this to the 2.0.0 milestone. Was there as a specific reason why you added and then removed it?

Not really... It's housekeeping with no external impact so I'm not really fussed if it's in the changelog.

Yup you're right. I haven't realised this is only /test/ source code 👍


ChildView crash in MojoShell when embedding a "heavy" app

Steps to Reproduce

Create an app that uses a lot of resources
Embed that app in another app via ChildView
CRASH

No such issue when embedding another app with a very light UI.
Flutter Doctor
$ flutter doctor
[✓] Flutter (on Mac OS, channel unknown)
• Flutter at /Users/pylaligand/Work/turquoise/sysui/src/third_party/flutter
• Framework revision 7e8274e24f (17 hours ago), engine revision 93e9328662
[x] Android toolchain - develop for Android devices
x Android Studio / Android SDK not found. Download from https://developer.android.com/sdk/
(or visit https://flutter.io/setup/#android-setup for detailed instructions).
[✓] iOS toolchain - develop for iOS devices (Xcode 7.3.1)
• XCode at /Applications/Xcode.app/Contents/Developer
• Xcode 7.3.1, Build version 7D1014
x ideviceinstaller not available; this is used to discover connected iOS devices.
Install via 'brew install ideviceinstaller'.
x ios-deploy not available; this is used to deploy to connected iOS devices.
Install via 'brew install ios-deploy'.
[✓] Atom - a lightweight development environment for Flutter
• flutter plugin version 0.2.3
• dartlang plugin version 0.6.26
Logs and Crash Reports
[0616/164730:FATAL:view_registry.cc(562)] Check failed: false.

(https://github.com/domokit/mojo/blob/dec9013739bd49292096072912197341fa056795/services/ui/view_manager/view_registry.cc#L562)
Note that this used to work when synced to Flutter at 0d21e69b1dd88db7f0c50de81bfba703e7905cc8. Here's the diff:
flutter/flutter@7e8274e Update engine.version (#4580)
flutter/flutter@5766cda Define LayoutWidgetBuilder in terms of BoxConstraints instead of Size (#4578)
flutter/flutter@661740d updates from previous review (#4577)
flutter/flutter@fc2dbde Remove unused consts in floating action button code (#4574)
flutter/flutter@e2299fd Remove unused consts in floating action button code. FAB segue animation is done in scaffold and the consts are there.
flutter/flutter@fc47909 send application files to the observatory's devfs (#4572)
flutter/flutter@3ee2f6f Fix the switch to match the Material Design spec.
/cc @abarth

view_registry.cc is part of Mozart, so we should /cc @j9brown

Ah, you hit an unimplemented case during view transfer.  Is this blocking you?

I've neutralized the code path that leads to this error locally, and this is a part of the codebase I'm pretty much the only one using right now. So, not P0 but probably P1.

+Michael Jurka<EMAIL_ADDRESS>It's basically a race.  If you add a view then quickly remove it, and
re-add it somewhere else before the view has had a chance to transfer its
token then we hit this unimplemented case.  This is probably something Mike
Jurka could fix pretty easily.  It's even marked with a TODO in the code.
Jeff.
On Thu, Jun 16, 2016 at 10:13 AM P.Y. Laligand<EMAIL_ADDRESS>wrote:

I've neutralized the code path that leads to this error locally, and this
is a part of the codebase I'm pretty much the only one using right now. So,
not P0 but probably P1.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/4586#issuecomment-226551371,
or mute the thread
https://github.com/notifications/unsubscribe/AAGPhMZ4KSD8dvjOfMfYdDG2v0FIZLeTks5qMYQ9gaJpZM4I3eBE
.

Filed bug here: https://github.com/domokit/mojo/issues/801

I believe mojoshell is no longer a thing and this is probably obsolete.

It was fixed in Mozart in https://github.com/domokit/mojo/commit/a264fc69c1e682eb642ac1c29dc891b33d04a9e6


How to Install and Run Netopeer2GUI

Hi,
How to Install and Run Netopeer2GUI on an Ubuntu PC?

Resolved in #19 .


Parameterize kubelet.service.j2 config file for kube-perf-tests

Non Breaking change to parameterize Kubelet Config file to enable passing different kubelet config file required for kube-perf-tests execution.
Change has been tested and runs fine.
This PR will be followed by another PR for kubetest2-plugins repo once the approach is finalized.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: piyushgupta1551
To complete the pull request process, please assign mkumatag after the PR has been reviewed.
You can assign the PR to them by writing /assign @mkumatag in a comment when ready.
The full list of commands accepted by this bot can be found 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

/cc @mkumatag

Closing PR as trying new approach suggested.


Unreal Engine is configuring libcurl for HTTP 1.1

There are most likely speed advantages from using HTTP 2.0, or even 3.0
From CurlHttp.cpp..
// HTTP2 is linked in for newer libcurl builds and the library will use it by default.
// There have been issues found with it use in production on long lived servers with heavy HTTP usage, for
// that reason we're disabling its use by default in the general purpose curl request wrapper and only
// allowing use of HTTP2 from other curl wrappers like the DerivedDataCache one.
// Note that CURL_HTTP_VERSION_1_1 was the default for libcurl version before 7.62.0
curl_easy_setopt(EasyHandle, CURLOPT_HTTP_VERSION, CURL_HTTP_VERSION_1_1);

Setting a breakpoint confirms that cesium-unreal executes this line, and there's no apparent way to configure it differently.
Thanks @kring for catching this.

any updates on this?

From what I can tell, this code is unchanged (and still an issue) as of Unreal Engine 5.4.1.


Merge Release/1.11 into main

Checklist

[ ] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[ ] Make sure mvn -P integration-tests clean verify runs for the whole project and, if you touched any code for a service in the services folder, ensure it can be run with spring-boot:run

Description

@roesslerj / @mibrasap - may I ask you to review this PR and check why we have difference in regards to SSL-hostname verification to main?

@roesslerj / @mibrasap - may I ask you to review this PR and check why we have difference in regards to SSL-hostname verification to main?

replaced by #1161

replaced by #1161


Docstrings for set_aog_theme! and aog_theme

What it says on the box.

Thanks for the PR! I've added some comments mostly about wording / style. For CI to pass, the docstrings will need to be included in the documentation website. I imagine the most reasonable place for them would be a Theming subsection here. They could probably also be referenced here, but that can be done in a separate PR.

Nice! Your suggestions were all improvements in my eyes too, thanks.
It will be some time before I find the time to add a theming section, and I do not have all that much to say about it. Is it something that you would be okay with implementing?


Phantom js not installed

Type of problem

Runtime problem after the build succeeds

Summary
Im not sure if I understood that phantom js it's being installed with this buildpack, if so Im getting the error that is not. Is something I've miss?
Contents of .meteor/release
<EMAIL_ADDRESS>Buildpack version
https://github.com/AdmitHub/meteor-buildpack-horse.git
Logs
spiderable: phantomjs not installed. Download and install from http://phantomjs.org/
Deployment target
heroku

I did set the buildpack to phantomjs branch and now it works


[Feature Request]: Implement IBM MQ Sink Connector for Python SDK

What would you like to happen?
I am proposing the implementation of an IBM MQ sink connector within the Apache Beam Python SDK. This feature would enable Apache Beam users to easily send data to IBM MQ, which is a widely used and robust messaging middleware. The absence of this connector in the Python SDK limits the integration capabilities for users working with IBM MQ in their data processing pipelines.
Issue Priority
Priority: 3 (nice-to-have improvement)
Issue Components

[X] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam YAML
[X] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[X] Component: Google Cloud Dataflow Runner

Note that Beam Java SDK has a JmsIO that can be used to connect to IBM MQ. For Python it's possible to write a cross-language wrapper

Do we need to write one with the Python autowrapper?


Add support for salting blocking

See this discussion
In a nutshell, we want the settings object to have an optional key of salt_partitions, defaulting to None (no salting).
If set, we want the blocking_rules_to_generate_predictions, (and blocking_rule_for_training in the EMTrainingSession) to be transformed as follows.
First, we need a new columns in __splink__df_concat_with_tf called (e.g.) __splink_salt, which is populated with a uniformly distributed integer between [1, salt_partitions.
Supposing
blocking_rules_to_generate_predictions = [
"l.col_a = r.col_a",
"l.col_b = r.col_b and l.col_c and r.col_c"
]

then we want this to be translated into:
blocking_rules_to_generate_predictions = [
"l.col_a = r.col_a and l.__splink_salt = 1",
"l.col_a = r.col_a and l.__splink_salt = 2",
...
"l.col_a = r.col_a and l.__splink_salt = salt_partitions",

"l.col_b = r.col_b and l.col_c and r.col_c and l.__splink_salt = 1"
"l.col_b = r.col_b and l.col_c and r.col_c and l.__splink_salt = 2"
...
"l.col_b = r.col_b and l.col_c and r.col_c and l.__splink_salt = salt_partitions"
]

Then everything else can proceed as normal.
Tricky parts:

Generating the splink salt.  Syntax likely to be different between backends.  Probably need a similar approach to how we take a random sample e.g. here
Need to check this doesn't interact in a weird way with other parts of Splink e.g. the real time linking functions

Why do we need this?

Why might we want this?
Some of the most expensive computation in Splink are:

Blocking data, because it creates a large number of comparisons
Computing the value of the comparison vectors, because some of the case when statements are expensive

We want to evenly distribute these computations across the cluster i.e. each machine does a similar number of computations
A fundamental problem in with data linking is that Spark is not very good at distributing blocking rules, because they are often skewed.
This is the ‘john smith’ problem - that 50k comparisons may be generated within the cartesian product of ‘john smiths’, even though only 10 comparisons are generated within Robin Linacre
So blocking serves to severely accentuate skew (because original skew becomes n^2 skew)
Spark hasn’t traditionally been very good at dealing with skew, but with adaptive query execution (ADE) it’s getting better.
The problem is that ADE is designed to work with ‘normal’ skew (n), not the type of n^2 skew that comes out of blocking.
So turning on ADE might give a small speed boost but it’s not a magic bullet
So we have to manually intervene, and tell Splink to re-distrbute the workload between machines at the earliest opportunity (i.e. redistribute the 50k john smiths between machines)
In practice, there are two places we can do it:
1 After blocking, but before computing the comparison vectors
2 After computing the comparison vectors
The problem with (1) is you potentially have to checkpoint (persist to disk) twice, which is expensive
The problem with (2) is that if you don’t redistribute (repartition) after blocking, then all the john smith comparison vectors end up being computed on a single machine (edited)
There’s isn’t an obvious best solution, I’m currently experimenting with (1) and (2) to see what gives the fastest result in practice
Note this is particularly an issue for EM iterations, because in EM we want to computer the comparison vectors once, and iteratively re-use them for EM
It may be that we want to build salting into Splink, to try and avoid skew in the first place, which would mean we wouldn’t need to worry so much about (1) skew after blocking and could focus on (2)

On this discussion, can we not improve the load balancing further by evaluating how many comparisons will be created by our various blocking rules and then differing our salting allocations accordingly?
So, if we had say:
blocking_rules_to_generate_predictions = [
"l.col_a = r.col_a",
"l.col_b = r.col_b and l.col_c and r.col_c"
]

and we knew that "l.col_a = r.col_a" created 50,000 comparisons, whereas "l.col_b = r.col_b and l.col_c and r.col_c" only created 10,000, then we could distribute our salt partitions like so, to balance the load:
"l.col_a = r.col_a and __splink_salt = 1",
...
"l.col_a = r.col_a and __splink_salt = 10",
"l.col_b = r.col_b and l.col_c and r.col_c and __splink_salt = 1",
"l.col_b = r.col_b and l.col_c and r.col_c and __splink_salt = 2"

On this discussion, can we not improve the load balancing further by evaluating how many comparisons will be created by our various blocking rules and then differing our salting allocations accordingly?
So, if we had say:
blocking_rules_to_generate_predictions = [
"l.col_a = r.col_a",
"l.col_b = r.col_b and l.col_c and r.col_c"
]

and we knew that "l.col_a = r.col_a" created 50,000 comparisons, whereas "l.col_b = r.col_b and l.col_c and r.col_c" only created 10,000, then we could distribute our salt partitions like so, to balance the load:
"l.col_a = r.col_a and __splink_salt = 1",
...
"l.col_a = r.col_a and __splink_salt = 10",
"l.col_b = r.col_b and l.col_c and r.col_c and __splink_salt = 1",
"l.col_b = r.col_b and l.col_c and r.col_c and __splink_salt = 2"

For what it's worth, I have implemented exactly this (where N salts varies by rule) manually in my own work using large datasets (100s of millions of rows) and it works well.

Closed by #278


chore(main): release 1.2.8

:robot: I have created a release beep boop
1.2.8 (2024-01-06)
Bug Fixes

allow images in attachments (#43) (bdad3cc)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cedricziel/mattermost-php/releases/tag/v1.2.8 :sunflower:


Remove old task to delete expired sessions

Does not apply any more with new session management

Fixes https://github.com/inventree/InvenTree/issues/2429

@matmair any ideas why the checks are failing here? It seems to be suggesting that the django_q app is not there, but I can run the same command manually and it works ok


Graph is moving while dragging

Hi Team,
According to the project requirement, the user should be able to move Crosshair while dragging across the graph. This feature is functioning well. But, when the Crosshair is absent, when the user drags across the graph, it moves which should not be happening. This is the present issue. Is there any solution for the above issue.
https://user-images.githubusercontent.com/70312717/123745835-72740200-d8ce-11eb-8eb5-aade0ff53fdf.mov

Hi! You should use HandleScrollOptions. You can pass it to handleScroll field in applyOptions method of ChartApi

Okay, thank you


Migration of old frontend

Only added CSS.
To-do:

Add old JS and convert jQuery to vanilla JS when applicable
Omit obsolete styles and scripts

Old JS (mainly pjax navigation) will be added on a later PR.


How to check which version is installed?

Is there a way to check which version is installed?  The help output does not reveal it, e.g.
$ imgcat --help
usage: imgcat [flags] filename
  -height string
        height (e.g. 100px, 10%, or auto)
  -p    preserve aspect ratio
  -size string
        width,height in pixels (e.g. 1024px,768px or 3,3)
  -width string
        width (e.g. 100px, 10%, or auto)

$ imgls --help
usage: imgls [flags] filename
  -height string
        height (e.g. 100px, 10%, or auto)
  -size string
        width,height in pixels (e.g. 1024px,768px or 3,3)
  -width string
        width (e.g. 100px, 10%, or auto)

No, there's no version yet. What are you trying to achieve with it? There have been no backwards compatibility issues that I know from.

Whoops. I see that I tagged it with 1.0 in the latest releases. Well, if I find the time, I can do it. Will re-open.