chore: more session replay metadata

:bulb: Motivation and Context
Ideas for https://posthog.slack.com/archives/C043VJ93L3B/p1725259136850079
:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed the submitted code.
[ ] I added tests to verify the changes.
[ ] I updated the docs if needed.
[ ] No breaking change or entry added to the changelog.

we are optimizing the screenshot snapshots with better encoding and some other things, closing this for now.
we already send $lib, so also do not need to do anything else.


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.2.4
Time: Thu May 26 2016 11:44:38 GMT 0200 (W. Europe Summer Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\jimkum\AppData\Local\GPMDP_3\app-3.2.4\resources\app\build\inject\GPMWebView\interface\customNavigation\mouseButtonNavigation.js:27:9.
    at BrowserWindow.callIntoRenderer (C:\Users\jimkum\AppData\Local\GPMDP_3\app-3.2.4\resources\electron.asar\browser\rpc-server.js:193:19)
    at emitTwo (events.js:105:20)
    at BrowserWindow.emit (events.js:185:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #1045


Device discovered twice

For some reason, the plugin is finding my Apple TV twice, and then causing an error that makes me unable to connect to homebridge
[2019-4-28 15:07:25] [Apple TV HB] [INFO] Discovered Apple TV (Living Room) at <IP_ADDRESS>
[2019-4-28 15:07:25] [Apple TV HB] [INFO] Added pair switch for Apple TV (Living Room) at <IP_ADDRESS>
[2019-4-28 15:07:25] [Apple TV HB] [INFO] Discovered Apple TV (Living Room) at <IP_ADDRESS>
[2019-4-28 15:07:25] [Apple TV HB] [INFO] Added pair switch for Apple TV (Living Room) at <IP_ADDRESS>
[2019-4-28 15:07:25] [Apple TV HB] No accessories have been added to the "Apple TV HB" platform config.
[2019-4-28 15:07:25] [Apple TV HB] Initializing platform accessory 'Pair Living Room'...
[2019-4-28 15:07:25] [Apple TV HB] Initializing platform accessory 'Pair Living Room'...
(node:26258) UnhandledPromiseRejectionWarning: Error: Cannot add a bridged Accessory with the same UUID as another bridged Accessory: <UUID of my Apple TV>
    at Bridge.Accessory.addBridgedAccessory (/usr/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/Accessory.js:268:13)
    at Server.<anonymous> (/usr/lib/node_modules/homebridge/lib/server.js:416:24)
    at /usr/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/util/once.js:16:19
    at AppleTVPlatform.accessories (/usr/lib/node_modules/homebridge-apple-tv/node_modules/homebridge-platform-helper/platform.js:42:14)
    at <anonymous>:null:null

(node:26258) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:26258) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

config.json platform setup:
{
     "platform": "AppleTV",
     "name": "Apple TV HB",
     "devices": [ ],
     "accessories": [
     ]
}

Just discovered this is a known issue


Timeline qwidget: Zoom

We can zoom in/out on the event section, but the time section remains the same.  Is it possible to connect these two sections so that when we zoom in the event section, the time section is also zoomed in.

Issue fixed: https://apps.audacy.space/trac/ticket/9485


There should be required security step

If someone doesn't change the secret key in the config file, there is security risk. So i propose that in the backend there is automatic generator of random keys and it generates a new key on the startup. Or so that if the key is set to the default vaule as shown in the config file, the backend would not start and will request a change of the key.

We could generate a hash of some random numbers and dump them into the config file if the value is default.

yep

@HyScript7 i have added that feature but how will we commit when the secret key will change on every dev start? and wouldnt be the default. is there a way to do it with github actions or should we use something else instead?

if we change the key on every startup so there can be anything in the config file then the development would be uhh because you would need new token every time you make a change and that is um..

@thesvec ist your now xd

@HyScript7 i have added that feature but how will we commit when the secret key will change on every dev start? and wouldnt be the default. is there a way to do it with github actions or should we use something else instead?

The secret key really doesn't matter if it changes, since it's only used for cookie signing. If the server reboots, everyone will just login again. No problem.

Also I am not sure what you mean by "How will we commit"

well if it was in the config.json then we would leak our keys. i think that the env will be the best option

Also I am not sure what you mean by "How will we commit"

well if it was in the config.json then we would leak our keys. i think that the env will be the best option

If you're talking about the app writing a new default token on startup, we can make disable it during testing. Make an environment flag called is_test_env, and if it's true, the script won't write the session secret into the config file. Also it's not that hard to just not stage the config file when committing.

Well the env would be the best because it wouldn't be in config.json so github would be okay. And the program will just check if the env is set or not and if not it will make one. This way when you first start it up there will be no env and it will create on. If you then restart for some reason, the env will still be there so it would not need a new one = users don't have to login everytime the app is restarted.


[Feature Request] Ability to close the panel window when no media is playing

I use a theme with a floating panel, as in, it has gaps like this:

However, when no media is playing, rather than disappearing, the panel window just becomes blank. Like this:

Would it be possible to fix this so that it disappears when no media is playing?

For me it disappears when all supported not blacklisted players are closed.
Maybe a problem of the theme ?


Add Outsourcing

https://github.com/morenoh149/awesome-outsourcing
The business practice of hiring a party outside a company to perform services and create goods. It should be added because it provides valuable information if you are thinking about outsourcing software development. Where do you start, how does this process work, etc.
I reviewed #1817 , #1816 and #1813 .
I confirm I've read and complied with the awesome requirements
(I made a new pr because I accidentally deleted the first fork, re https://github.com/sindresorhus/awesome/pull/1825)

unicorn


[BUG] -Azure Service Fabric OpenSSL version <IP_ADDRESS> vulnerability

Describe the bug
Azure Service Fabric appears to be using OpenSSL version <IP_ADDRESS> which has a vulnerability as detailed in CVE-2023-2650.
Area/Component:
Azure Service Fabric OpenSSL version <IP_ADDRESS>
To Reproduce
Steps to reproduce the behavior:

Create a new Service Fabric cluster from the Azure portal. Use WindowsServer 2019-Datacenter for the Operation System. Service Fabric version used was 10.1.1541.9590
After the cluster is deployed, login to one of the deployed virtual machines
Check the version information on these files, they both show Version: <IP_ADDRESS>.

c:\program files\microsoft service fabric\bin\fabric\fabric.code\libcrypto-3-x64.dll
c:\program files\microsoft service fabric\bin\fabric\fabric.code\libssl-3-x64.dll

Expected behavior
Update needed to OpenSSL version used by Azure Service Fabric
Observed  behavior:
Check the version information on these files, they both show Version: <IP_ADDRESS>.

c:\program files\microsoft service fabric\bin\fabric\fabric.code\libcrypto-3-x64.dll
c:\program files\microsoft service fabric\bin\fabric\fabric.code\libssl-3-x64.dll

Screenshots

Service Fabric Runtime Version:
10.1.1541.9590
10.0.1949.9590
Environment:

Azure
OS: Windows Server 2019
Version 10.1.1541.9590

If this is a regression, which version did it regress from?
Additional context
I reported this through MSRC and they just closed the case.

Assignees: /cc @microsoft/service-fabric-triage

The same library versions are present in latest 10.1.2175.9590 runtime release


Fix create Lookup Table

Fix Create Lookup table with incorrect estimation on 1 unit of usage amount
Issue: when I create a Lookup table based on input usage csv, but after multiply value on usage csv with estimation value on Lookup table. It didn't match the value from API /footprint
Maybe this is something like the issue on ISSUE-839
Checklist

[x] yarn test passes
[x] yarn lint has been run

Notes
Additional information relevant to the changes
© 2021 Thoughtworks, Inc.

@kakazaro Thanks for opening this PR and apologies for the time it took to review!
If I am understanding your implementation correctly, it seems you are adding support to pass a custom usage amount and cost as columns in the Lookup Table Input CSV so that it can conditionally calculate estimates according to that usage instead of always giving a unit of 1.
Feel free to correct me if I'm wrong in that understanding. I could see that being a useful feature in a general sense. However, if my understanding is correct and if you are truly seeing discrepancies between the API estimations and the Lookup Table calculations prior to this, could you explain how this approach would fix the issue of those discrepancies?
Wouldn't passing in a usage amount in your input data, having CCF calculate according to that usage, and then outputting the result accordingly still give you the same calculations as manually multiplying the co2e/energy output by your usage?

@4upz we at ReqPOOL worked on this fix together with @kakazaro from TechStackApps since we came across a bug while working with the LUT-approach for one of our customers. The reason for using a LUT is because the customer does not permit their actual cloud usage and cost data to leave their environment, so we need to generate a LUT and then multiply the values in a separate application within their environment.
What we've discovered is that for unknown usage types, the LUT-approach creates different results than the API with direct AWS integration does. This is because kWh for unknown usage types are estimated based on the cost/usageAmount ratio, which is information that you don't have (by design) when you create the LUT. This bug #839 is already know for quite a while if I look at its history, but nobody ever got around to fixing it.
The solution we found is that it is enough to pass the correct ratios of cost/usageAmount in the input CSV for create-lookup-table. so you can work with falsified/randomized data by multiplying all the values created from the Athena query by a random factor (thus keeping the original ratios while falsifying the true values). @kakazaro's fix allows us to do that by actually taking cost and usageAmount into consideration when generating the LUT. If these values are not present, they default back to 1 as was the case before.
One problem I've found after these changes is that create-lookup-table now throws an error when usageAmount and cost  are not present, you might need to look into that before accepting the PR.
Hope this clears things up and we made a valuable contribution for anybody using the LUT-approach in the future! Let me know if you have any more questions.


Improve yasb->xws maintenance

Currently manually hacked for cards that didn't map from my collection.
This could be automated along lines of this, I think: https://github.com/awsaba/xwing/blob/awsaba/xws-content-dumper/coffeescripts/content/dump-content.coffee

we really ought not to do this, in my opinion. There should be a plan/scheme for xws id generation (currently, some cards, especially SL cards, don't have a unique name from the current rules) that we follow across the board, not rely on yasb's internal ids for cards.
Some examples of issues here are the ~50 "hansolo" that we have in listfortress, which were exporting by YASB during the time squad2xws was broken. "hansolo" refers to scum (customized falcon) pilot, but the vast majority are rebel han's which have invalid names, because I assume YASB's internal "hansolo" refers to rebel falcon pilot

apologies, I got sent here because of a PR that swapped to this branch instead of the default xwingdata2 branch!


500 Response, no jobs returned

Overview
Receiving 500 HTTP response from Bullhorn API. Using the WordPress plugin, though the behavior is the same as the "regular" install.
Similar issues found on this repo mention that this is due to the career portal not being enabled on the Bullhorn account. My client has been in contact with Bullhorn support, and the career portal should be enabled.
Attempted Remedies

Confirmed that the app.json generated by the WordPress plugin looks correct
The client contacted their Bullhorn support representative and confirmed the swimlane & corp token.
The client also ensured that career portal access was enabled for their account.

Relevant Issues
#29 #72 #167

Hey @LakenH As mentioned previously (thanks for your diligence in researching this btw).  This is still the same behavior that companies with the public api disabled exhibit.
Can you share with me the company's name so I can make sure they are taken care of?
Thank you

Sure - my client's company name is Healthcare Support

I took a look at this and fixed the issue preventing the jobs from loading they should work now.  A user was disabled that shouldn't have been.

Thank you, working great now.


Modified chpst to su in init file

chpst does not load secondary groups, which makes granting the service permissions for files (e.g. client.pem) very hard
su exists on virtually all Linux systems, so should be OK

@reset any chance to merge this please?

@BackSlasher thanks for the contrib

Happy that master is one step closer to my environment :)


Added undefined check to resolve the getButtonText undefind issue. (https://github.com/dotansimha/angularjs-dropdown-multiselect/issues/22)

https://github.com/dotansimha/angularjs-dropdown-multiselect/issues/22
'ngDropdownMultiselect' directive is getting executed before the scope binding, so the $scope.selectedModel is undefined at that time while executing the $scope.getButtonText() function, so calling the length on undefind is causing this issue.

@dotansimha Please review the changes. Thanks.

(Y)

@dotansimha Would still be excellent if this were merged...

+1


Convert from using jscs(depricated) in favor of ESLint

node-jscs has hit EOL and is no longer being developed en lieu of team joing ESLint.
jscs is full of security vulnerabilities that need to be patched but being a sunset product it will never be updated. Teh grunt jscs library is actively being utilized by jquery-ui which is consumed by thousands of products.

Huh, this has been discontinued as well... And it is exactly a library to provide an entry point into JSCS from Grunt 🤔
If it's a problem for jquery-ui, then it should be dropped there?
I bet that there already is a grunt-eslint library somewhere that will do the job better?

@gustavohenke if that's the case then I recommend archiving this repository and dropping a note in the ReadME


Windows 10 Latest Not Supported

Is this version of Windows 10 not supported? It doesn't work....

W10HOME 17763.379
I also tried to uninstall and reboot. I tried rewriting INI 292 → 379, but it does not work.
I tried initializing the OS and installing it.1.6.2 installation, termservice stop, replace termservice.dll, INI rewrite, restart.
Is the procedure wrong? I want you to help.

I have the same problem.

Same problem.
My Windows 10 version is profession 1809 17763.379

Same problem. please fix it!

Same problem.
Please fix it!

sorry, same for me with a regular Win 10 LTSC 1809

Use the following file :
Open a command prompt windows in admin mode  then :
net stop termservice
Copy the file from the zip provided  into C:\Program files\RDP Wrapper
Return to your command prompt windows (still in admin mode) then :
net start termservice.
If it is not working, simply reboot the computer.
rdpwrap 10.0.17763.292.zip

I have had this problem but it was solved by the following. Use "administrator rights" as necessary.

uninstall rdpwrap including deleting the rdpwrap.ini and rdpwrap.dll files in <Program Files/RDP Wrapper>
reboot the computer
install rdpwrap.
Check RDPConf and note the version number. The screenshot above shows .292
Check the version of termserv.dll and compare to that of RDPConf.
Use CMD to "net stop termservice"
edit the rdpwrap.ini file in <program Files/RDP Wrapper> to include the snippets relative to the versions above. save it in documents if you can't save it the same spot. Or you can use one of the files mentioned above.
Rename rdpwrap.ini and move the edited one into <Program File/RDP Wrapper>.
8., Use CMD to "net start termservice".
Reboot computer.
run RDPConf to confirm it is listening and supported.
A comment on the above screenshots. RDP is used by hackers to install ransomware as it is easily hacked. Maintaining maximum security is very important and 2 of the settings above do not do that.  "Hide users on logon screen" and "Network level Authentication"  should be checked. In addition Windows security should limit the number of failed password attempts and your users and passwords should not be short nor simple or common. I made these mistakes and have lost all my work and memories from the last 20 years.
If I've made a mistake in my directions above please post the correction.

Use the following file :
Open a command prompt windows in admin mode then :
net stop termservice
Copy the file from the zip provided into C:\Program files\RDP Wrapper
Return to your command prompt windows (still in admin mode) then :
net start termservice.
If it is not working, simply reboot the computer.
rdpwrap 10.0.17763.292.zip

Thank you very much @thaguitarist, this fixed the problem for me

Maybe somebody could do a merge request for https://github.com/stascorp/rdpwrap/blob/master/res/rdpwrap.ini ? Then the installer would just download the right version on install, I guess.

Hello good morning friends, and try the version 10.0.17763.292 and works perfectly in the version 10.0.17763.379 ... They must install the RDP 1.6.2 version, if they already have it installed it is just running the patch and it is ready and it is working.
I hope you help them who have not yet been able to solve the error.

Download Patch

Hello good morning friends, and try the version 10.0.17763.292 and works perfectly in the version 10.0.17763.379 ... They must install the RDP 1.6.2 version, if they already have it installed it is just running the patch and it is ready and it is working.
I hope you help them who have not yet been able to solve the error.
Download Patch

THE REAL MVP
insertgif THANKYOU!

你好早上好朋友，并尝试10.0.17763.292版本，并在10.0.17763.379版本中完美运行...他们必须安装RDP 1.6.2版本，如果他们已经安装它只是运行补丁它已经准备好了它正在发挥作用。
我希望你帮助那些尚未解决错误的人。

下载补丁

这个是有效的，谢谢

Use the following file :
Open a command prompt windows in admin mode then :
net stop termservice
Copy the file from the zip provided into C:\Program files\RDP Wrapper
Return to your command prompt windows (still in admin mode) then :
net start termservice.
If it is not working, simply reboot the computer.
rdpwrap 10.0.17763.292.zip

This works on me. ver. 10.0.17763.292

Use the following file :
Open a command prompt windows in admin mode then :
net stop termservice
Copy the file from the zip provided into C:\Program files\RDP Wrapper
Return to your command prompt windows (still in admin mode) then :
net start termservice.
If it is not working, simply reboot the computer.
rdpwrap 10.0.17763.292.zip

it does work on 292

new build and same problem.
can you write the "how-to" steps to solve the problem?

::::::::::::::
[10.0.17763.437]
::::::::::::::
[10.0.17763.437]
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=77A41
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=3E520
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=18025
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=1ACDC
SLInitFunc.x64=New_CSLQuery_Initialize
bInitialized.x64 =ECAB0
bServerSku.x64 =ECAB4
lMaxUserSessions.x64 =ECAB8
bAppServerAllowed.x64 =ECAC0
bRemoteConnAllowed.x64=ECAC4
bMultimonAllowed.x64 =ECAC8
ulMaxDebugSessions.x64=ECACC
bFUSEnabled.x64 =ECAD0
::::::::::::::
[10.0.17763.437-SLInit]
::::::::::::::
[10.0.17763.437-SLInit]
bInitialized.x86 =CD798
bServerSku.x86 =CD79C
lMaxUserSessions.x86 =CD7A0
bAppServerAllowed.x86 =CD7A8
bRemoteConnAllowed.x86=CD7AC
bMultimonAllowed.x86 =CD7B0
ulMaxDebugSessions.x86=CD7B4
bFUSEnabled.x86 =CD7B8
bInitialized.x64 =ECAB0
bServerSku.x64 =ECAB4
lMaxUserSessions.x64 =ECAB8
bAppServerAllowed.x64 =ECAC0
bRemoteConnAllowed.x64=ECAC4
bMultimonAllowed.x64 =ECAC8
ulMaxDebugSessions.x64=ECACC
bFUSEnabled.x64 =ECAD0

Closed as not actual


Generated schema for eslint/max-len can be optimized

The generated file for eslint/max-len can be optimized.
The rule has 3 times the same pattern.
Instead of generating a config and option type, we can reuse the setting type.

Could also be done using an AST
However I wonder if it is worth it. Do you think it's really worth it to optimize this? These are auto-generated files not really made for reading, and for the end user, it doesn't change anything, he will always have the auto-completion of his types despite the interface is duplicated several times.
By introducing a method with AST to remove duplicates, it makes us more code to maintain and we risk introducing bugs.
That said, it's an interesting thing to do.
What do you think about it?

What do you think about it?

:arrow_down:

That said, it's an interesting thing to do.

:smile:


feat: Camera Visor: force icons to always be white

Hi everyone,
In some cases, the icons in the visor may be black.
Here is a simple fix to ensure it's always white.

Codecov Report

Merging #4340 (d950844) into develop (5b376d0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #4340   +/-   ##
========================================
  Coverage    10.54%   10.54%
========================================
  Files          290      290
  Lines        14294    14294
========================================
  Hits          1507     1507
  Misses       12787    12787

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


APPEXECLINK reparsepoints do not show target and link type

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
No target shown for APPEXECLINK type reparse points:
See visual below
Expected behavior
Can see target of the APPEXECLINK reparsepoint

Actual behavior
No target shown

Error details
No error

Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0-preview.3
PSEdition                      Core
GitCommitId                    7.4.0-preview.3
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

While I agree that this would be helpful, please note the following comment from the .NET team, after they had talked to the Windows team; from https://github.com/dotnet/runtime/pull/58233#issuecomment-911971904

We had a conversation with the Windows team that owns AppExecLinks. They explained that we are not supposed to care about the target path. AppExecLinks are meant to work seamlessly as any other executable. Hence why they never exposed the APIs that we were adding in this PR, and which PowerShell is also calling.

...

We also had a conversation with @SteveL-MSFT , @rjmholt, @daxian-dbw from the PowerShell team to let them know what the Windows people told us, so they make adjustments to their code if needed.

This led to the following (already merged) PR:

#16044

So it sounds like simply removing the -> from the Name column for AppX reparse points is called for (the l in the Mode would still indicate the presence of a reparse point; that, combined with not having a target displayed would then imply that the file at hand is an AppX reparse point).

@daxian-dbw, I think there's still something actionable here:

Fix the formatting data to omit the -> suffix for AppX reparse points - so that user won't wonder why there's nothing to the right of the arrow.

Amend the documentation to explain why AppX reparse points have l in their Mode output column, yet do not list a target path.

Hence why they never exposed the APIs that we were adding in this PR, and which PowerShell is also calling.

fsutil.exe shows target:
fsutil reparsePoint query .\AppData\Local\Microsoft\WindowsApps\notepad.exe

And this output can be parsed as string

@237dmitry, the point isn't whether the target can be determined, the point is that it shouldn't be attempted, because it isn't officially supported, and the underlying data structure may change without notice, at least hypothetically.
Clearly, how to do it is known, via unsupported APIs that were explicitly removed when the fact that they're unsupported came to light.
It's fair to assume that this code will not be put back in, for the reasons stated.
Those who - despite the caveats - want to implement their own target-finding code nonetheless, can adapt the removed code or, less efficiently, can indeed parse the output from fsutil reparsePoint query.
The latter approach is used by a custom Resolve-AppXExePath function shown in this Stack Overflow answer.
Note, however, that for some AppX reparse points the target listed in the fsutil reparsePoint query output is a generic launcher: e.g., for MicrosoftEdge.exe the target is reported as C:\Windows\system32\SystemUWPLauncher.exe, even though the true ultimate target is C:\Program Files (x86)\Microsoft\Edge\Application\msedge.exe.

Opened #19794 to track the formatting change proposal.

Amend the documentation to explain why AppX reparse points have l in their Mode output column, yet do not list a target path.

@mklement0 Can you please open a doc issue for this?

@daxian-dbw, I've opened the following:

https://github.com/MicrosoftDocs/PowerShell-Docs/issues/10168

@237dmitry, I think the decision by the Windows team is unfortunate, as it is helpful to be able to determine the path of the executable that is eventually launched, but this decision isn't related to open-source vs. closed-source (which Windows itself is).


Retains mods except annotations

Resolves https://github.com/scalameta/paradise/issues/8
Ready for review

Thanks for addressing the feedback! The final step is signing the Scala CLA (https://www.lightbend.com/contribute/cla/scala), and then I can merge.

Done

Thanks for your patience! I'll be publishing 3.0.0-SNAPSHOT momentarily.

Thanks a lot.


KOGITO-7606 Custom dashboards

JIRA:
https://issues.redhat.com/browse/KOGITO-7606

Description:

Please make sure that your PR meets the following requirements:

[ ] You have read the contributions doc
[ ] Pull Request title is properly formatted: KOGITO-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting main: [0.9.x] KOGITO-XYZ Subject
[ ] The nav.adoc file has a link to this guide in the proper category
[ ] The index.adoc file has a card to this guide in the proper category, with a meaningful description

How to setup automatic cherry-pick to another branch ?

The cherry-pick action allows to setup automatic cherry-pick from main to a specific branch.
To allow it, you will need to add the corresponding label with pattern: backport-{RELEASE_BRANCH}.
For example, if a backport to branch 1.26.x is needed, then the label should be backport-1.26.x.
Once the PR is merged, the action will retrieve the commit and cherry-pick it to the desired branch.
NOTE: You can still add label after the merge and it should still be cherry-picked.

Moving to DP4. @radtriste are you ok with this PR?

sorry Marian took over that one so I was not looking.
One QE approval should be enough ;)

@domhanak you wanna take a look to? :)

@bxf12315 @radtriste do we need to backport this?

backport to ?

to another branch

@ricardozanini I think we could backport it to 1.34.


Add zlib (de)compression support to File class

Title. zlib is already packaged within Godot, and NetworkedMultiplayerENet already has a zlib compression mode, so I wonder why File can't have the same.
If it helps, my use case involves PoolByteArray.decompress(), which itself uses the compression mode enum from File (and thus I presume it uses File somewhere within).

Duplicate of #3701.

@akien-mga While I can understand the assessment, I'm not sure I agree with it being a duplicate. My use case involves exposing zlib compression as an option for use with PoolByteArray.decompress.

While I agree it's not strictly speaking the same thing (#3701 was focusing on making Zip archives, while this issue requires Zlib compression), the two are strongly linked with each other and should be implemented together IMO, so it makes sense to have one single issue to keep track of that.

Okay, that's fair. Thank you for explaining.

That said, if you can put some details about the Zlib features you'd expect in #3701, that would be good. I renamed the issue to match what I described above, but the comments themselves don't speak much of Zlib yet, so that should be added so that potential contributors willing to work on it take both into account.

I'll do that shortly. Thanks.

In second thought, I think I'm wrong in my above assessment. Both topics are linked and should likely be handled together, but it still makes sense to have dedicated issues for each.

zlib is already implemented, File.COMPRESSION_DEFLATE means it's using zlib.

Then shouldn't it be File.COMPRESSION_ZLIB if it's actually zlib? I know Zlib implements Deflate as an option, but deflate isn't necessarily zlib decompression.

AFAIK the only thing special about zlib is the special header. The compression algorithm is the same (deflate).

To my knowledge, zlib can use more than deflate. I might be wrong, however, and would appreciate if someone could clarify. https://tools.ietf.org/html/rfc1950

One of the bits in the header specifies the compression method (https://tools.ietf.org/html/rfc1950#section-2, see section 2.2, under "CM (Compression method)"). That's why I'm confused.
Either way, it would be nice if the docs cleared up this distinction, or if a alias named File.COMPRESSION_ZLIB was added.

Somewhat unrelated (except to my use case), but since this is an issue about (de)compression, can I ask why PoolByteArray.decompress() requires me to specify the size of the uncompressed data? It just seems really odd that it asks for that...

Somewhat unrelated, but since it relates to the use case I described for this issue... PoolByteArray.decompress() requires me to specify the maximum size of how big I want the resulting array to be. Why? Wouldn't it sort of be possible to figure out the size automatically?

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


An exception occurred during the decompilation of the 'OS/2' table

When I try to get some attributes of the OS/2 table with Mona.ttf, I got this error
An exception occurred during the decompilation of the 'OS/2' table
Traceback (most recent call last):
  File "C:\Users\Admin\AppData\Local\Programs\Python\Python38\lib\site-packages\fonttools-4.29.1-py3.8.egg\fontTools\ttLib\ttFont.py", line 409, in _readTable
    table.decompile(data, self)
  File "C:\Users\Admin\AppData\Local\Programs\Python\Python38\lib\site-packages\fonttools-4.29.1-py3.8.egg\fontTools\ttLib\tables\O_S_2f_2.py", line 113, in decompile
    dummy, data = sstruct.unpack2(OS2_format_2_addition, data, self)
  File "C:\Users\Admin\AppData\Local\Programs\Python\Python38\lib\site-packages\fonttools-4.29.1-py3.8.egg\fontTools\misc\sstruct.py", line 103, in unpack2
    return unpack(fmt, data[:length], obj), data[length:]
  File "C:\Users\Admin\AppData\Local\Programs\Python\Python38\lib\site-packages\fonttools-4.29.1-py3.8.egg\fontTools\misc\sstruct.py", line 86, in unpack
    elements = struct.unpack(formatstring, data)
struct.error: unpack requires a buffer of 18 bytes

Here is an sample code:
    font = ttLib.TTFont("Mona.ttf", fontNumber=0, ignoreDecompileErrors=True)

    isItalic = font["OS/2"].fsSelection & 0b1 > 0
    weight = font['OS/2'].usWeightClass

According to ots, that font is broken in more than one way:
$ python -m ots /Users/just/Downloads/Mona.ttf
WARNING: OS/2: Bad version number, setting it to 1: 2
ERROR: hmtx: Failed to read side bearing 7225
ERROR: hmtx: Failed to parse table
Failed to sanitize file!

According to ttx, that font is broken in several ways, too:
$ ttx /Users/just/Downloads/Mona.ttf
Dumping "/Users/just/Downloads/Mona.ttf" to "/Users/just/Downloads/Mona.ttx"...
Dumping 'GlyphOrder' table...
WARNING: 'created' timestamp seems very low; regarding as unix timestamp
WARNING: 'modified' timestamp seems very low; regarding as unix timestamp
Dumping 'head' table...
Dumping 'hhea' table...
Dumping 'maxp' table...
ERROR: An exception occurred during the decompilation of the 'OS/2' table
Traceback (most recent call last):
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/ttFont.py", line 417, in _readTable
    table.decompile(data, self)
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/tables/O_S_2f_2.py", line 113, in decompile
    dummy, data = sstruct.unpack2(OS2_format_2_addition, data, self)
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/misc/sstruct.py", line 103, in unpack2
    return unpack(fmt, data[:length], obj), data[length:]
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/misc/sstruct.py", line 86, in unpack
    elements = struct.unpack(formatstring, data)
struct.error: unpack requires a buffer of 18 bytes
Dumping 'OS/2' table...
ERROR: An exception occurred during the decompilation of the 'hmtx' table
Traceback (most recent call last):
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/ttFont.py", line 417, in _readTable
    table.decompile(data, self)
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/tables/_h_m_t_x.py", line 51, in decompile
    glyphName = glyphOrder[i]
IndexError: list index out of range
Dumping 'hmtx' table...
Dumping 'cmap' table...
Dumping 'fpgm' table...
Dumping 'cvt ' table...
WARNING: corrupt 'loca' table, or wrong numGlyphs in 'maxp': 7225 7226
Dumping 'loca' table...
WARNING: 6967 glyphs have no name
Dumping 'glyf' table...
Dumping 'name' table...
Dumping 'post' table...
Dumping 'gasp' table...
ERROR: An exception occurred during the decompilation of the 'EBLC' table
Traceback (most recent call last):
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/ttFont.py", line 417, in _readTable
    table.decompile(data, self)
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/tables/E_B_L_C_.py", line 116, in decompile
    indexSubTable.decompile() # https://github.com/fonttools/fonttools/issues/317
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/tables/E_B_L_C_.py", line 438, in decompile
    offsetArray = [struct.unpack(dataFormat, self.data[slice(*loc)])[0] for loc in indexingLocations]
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/tables/E_B_L_C_.py", line 438, in <listcomp>
    offsetArray = [struct.unpack(dataFormat, self.data[slice(*loc)])[0] for loc in indexingLocations]
struct.error: unpack requires a buffer of 4 bytes
ERROR: An exception occurred during the decompilation of the 'EBDT' table
Traceback (most recent call last):
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/ttFont.py", line 417, in _readTable
    table.decompile(data, self)
  File "/Users/just/code/git/fontra/venv/lib/python3.10/site-packages/fontTools/ttLib/tables/E_B_D_T_.py", line 52, in decompile
    for curStrike in locator.strikes:
AttributeError: 'DefaultTable' object has no attribute 'strikes'
Dumping 'EBDT' table...
Dumping 'EBLC' table...

Yes, the font is broken.
It would be nice if I write something like that to not have the log print like I show in my first message:
try:
    font = ttLib.TTFont("Mona.ttf", fontNumber=0, ignoreDecompileErrors=True)
    isItalic = font["OS/2"].fsSelection & 0b1 > 0
    weight = font['OS/2'].usWeightClass
except:
    # Do XYZ

I'm afraid I don't understand what you are asking for.

What I want
Don't print the exception when it is surrounded by an try except.
Example where it would only print: Warning: The file "XXX.YYY" does not have an OS/2 table. Default style will be applied to the font)
try:
    font = ttLib.TTFont("Mona.ttf", fontNumber=0, ignoreDecompileErrors=True)
    isItalic = font["OS/2"].fsSelection & 0b1 > 0
    weight = font['OS/2'].usWeightClass
except:
        print(f"Warning: The file \"{fontPath}\" does not have an OS/2 table. Default style will be applied to the font.")

Something like this, perhaps?
from fontTools.ttLib import TTFont

fontPath = "Mona.ttf"

font = TTFont(fontPath, fontNumber=0)
try:
    os2 = font["OS/2"]
except Exception as e:
    print(repr(e))
    print(f"Warning: The file \"{fontPath}\" does not have an OS/2 table. Default style will be applied to the font.")
else:
    isItalic = os2.fsSelection & 0b1 > 0
    weight = os2.usWeightClass

The exception is only print when I try to access to attributes of font["OS/2"]
So, there is no exception when you write: `os2 = font["OS/2"]
Even something like this will still print the exception
    try:
        os2 = font["OS/2"]
        os2.fsSelection & 0b1 > 0
    except:
        isItalic = False
        weight = 400
        print(Fore.LIGHTRED_EX + f"Warning: The file \"{fontPath}\" does not have an OS/2 table. This can lead to minor errors." + Fore.WHITE)
    else:
        isItalic = os2.fsSelection & 0b1 > 0
        weight = os2.usWeightClass

Oh, I just understand.
I had this before: font = ttLib.TTFont("Mona.ttf", fontNumber=0, ignoreDecompileErrors=True)
If I remove the ignoreDecompileErrors, it work. I have no idea why because I thought it will ignore it when it was true, but it seems to do something else.
    font = TTFont(fontPath, fontNumber=0)
    try:
        isItalic = font["OS/2"].fsSelection & 0b1 > 0

        weight = font['OS/2'].usWeightClass
    except:
        print(Fore.LIGHTRED_EX + f"Warning: The file \"{fontPath}\" does not have an OS/2 table. This can lead to minor errors." + Fore.WHITE)


feat(gitlab): init

🎉 Theme for Gitlab 🎉
Gitlab is a online service to manage git repos with devops tools
💬 Additional Comments 💬
🗒 Checklist 🗒

[x] I have read and followed Catppuccin's submission guidelines.
[x] I have made a new directory underneath /styles/<name-of-website> containing the contents of the /template directory.

[x] I have ensured that the new directory is in lower-kebab-case.
[x] I have followed the template and kept the preprocessor as LESS.

[x] I have made sure to update the
userstyles.yml
file with information about the new userstyle.
[x] I have included the following files:

[x] catppuccin.user.css - all the CSS for the userstyle, based on the
template.
[x] preview.webp - composite image of all four individual flavor screenshots (taken with the default accent color of mauve) stitched together, generated via Catwalk.

Lot of things still unthemed, please mark it as ready when you have fully themed the entire application - settings menus, modals, dropdown/selects/inputs, profile pages, READMEs, etc. A good trick is to use GitLab dark mode, but set the Catppuccin flavor to latte. Very easy to spot unthemed areas.

Yes, still a couple of things to work on.
I just started working on this today, I'm making the pr to save my work, and mark this as work in progress

I'm happy to help with this userstyle too if necessary

I'm happy to help with this userstyle too if necessary

I will be more than happy to have some help, how can we manage this ?

I'm happy to help with this userstyle too if necessary

I will be more than happy to have some help, how can we manage this ?

Are you in the discord server? Communicating there could make things a bit easier

I'm happy to help with this userstyle too if necessary

same here, i'll help fix bugs for this userstyle for a little bit

What editor are you using where the file isn't recognized as Less?

What editor are you using where the file isn't recognized as Less?

Webstorm. And if I manually reassign it to less it doesn't persist

Webstorm. And if I manually reassign it to less it doesn't persist

@thelooter try going to settings (or preferences) > editor > file types and you should be able to change it there

To add on to that, if there is a way to set that in a project local settings file (like .vscode/settings.json) we would be happy to accept a PR that introduces it here.

To add on to that, if there is a way to set that in a project local settings file (like .vscode/settings.json) we would be happy to accept a PR that introduces it here.

i've tried searching webstorm's docs and installing webstorm to try and to this but unfortunately, theres no way to do that

Hi everyone, I've been using this branch for a couple of weeks now and I thinks its ok.
It might be good to talk about reviewing the PR.

Colors on Security Dashboard:

Compliance Center Status:

Readability on issue labels:

Header on Security View in Pipeline:

Repository > CI/CD Settings > Runner:

Repository > General Settings > Service Desk

Instance Runner View:

Instance Runner Fleet Dashboard:

Search Popup (commands text bottom right)

Great, Thanks for the review
I will work on theses issues in the next days

Unfortunately I might not have access to all these pages :

Colors on Security Dashboard
Compliance Center Status
Header on Security View in Pipeline
Repository > CI/CD Settings > Runner
Repository > General Settings > Service Desk
Instance Runner View
Instance Runner Fleet Dashboard

This is now done

Search Popup :heavy_check_mark:
Readability on issue labels: :heavy_check_mark:

Sorry for the mass review requests, I just wanted to update the branch

Sorry for the mass review requests, I just wanted to update the branch

the review requests got triggered because you pushed changes to other userstyles. you probably accidentally added some formatting changes. you should be able to revert the commit and uncommit the changes.

do you want to convert this to a draft and work on it further?
you'll be able to convert it to a full pr and request a review when you feel it's ready <3


Use xcode 9.4

Codecov Report

Merging #69 into master will decrease coverage by 1.27%.
The diff coverage is 33.33%.

@@            Coverage Diff            @@
##           master     #69      +/-   ##
=========================================
- Coverage   86.78%   85.5%   -1.28%
=========================================
  Files          81      81
  Lines        2935    3518     +583
=========================================
+ Hits         2547    3008     +461
- Misses        388     510     +122

Impacted Files
Coverage Δ

Sources/Bitcoin/BitcoinTransactionSigner.swift
83.47% <33.33%> (+11.24%)
:arrow_up:

Sources/Ethereum/RLP.swift
60.09% <0%> (-17.41%)
:arrow_down:

Sources/Bitcoin/WitnessProgram.swift
76.31% <0%> (-13.69%)
:arrow_down:

Sources/Blockchain.swift
11.76% <0%> (-5.89%)
:arrow_down:

Sources/Ethereum/TransactionSigning.swift
95.45% <0%> (-4.55%)
:arrow_down:

...ces/Ethereum/Solidity/EIP712/EIP712TypedData.swift
92.85% <0%> (-3.81%)
:arrow_down:

Sources/Bitcoin/BitcoinScript+Parsing.swift
50% <0%> (-2.39%)
:arrow_down:

Sources/PrivateKey.swift
76.92% <0%> (-1.93%)
:arrow_down:

Sources/Bitcoin/PublicKey+Bitcoin.swift
100% <0%> (ø)
:arrow_up:

Sources/BigIntEncoding.swift
100% <0%> (ø)
:arrow_up:

... and 10 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 de1386b...4d6ec5b. Read the comment docs.


Accepting autocomplete suggestions in a notebook duplicates already typed text

Environment data

OS: XUbuntu 20.04, Linux x64 5.8.0-53-generic
VS Code: 1.56.2 (and also previous versions)
Commit: 054a9295330880ed74ceaedda236253b4f39a335
Date: 2021-05-12T16:45:26.313Z
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0

Expected behavior
Selecting autocomplete suggestions (e.g. hitting Tab) should add only the missing chars of the suggested expression
Actual behavior
(Sometimes) manually-typed characters are duplicated, and the completion is appended after that

Steps to reproduce
Unknown; the issue ocurrs sporradically and with no apparent reason. I've had this start happening in the exact same cell where it worked as expected mere seconds before that (and without the cell even losing focus).
P.S. this never happens in regular files; only in notebooks.

Dupe #5956


Change key derivation path to be correct for Dogecoin

As noted in #2489: needs a solution for the installed base - any specific reason to put this in 1.14.x and not 1.21.x?

@patricklodder All good for backward compatibility.
Old wallet will continue using the old derivation aka m/0'/0' and any new wallet will use m/0'/3'

How did I test it:
1/ Create a brand new wallet with 1.14.4 version
2/ ./src/dogecoin-cli getnewaddress ""
3/  ./src/dogecoin-cli validateaddress xxx
"hdkeypath": "m/0'/0'/1'"
4/ Compile with the new derivation
5/ ./src/dogecoin-cli getnewaddress ""
6/  ./src/dogecoin-cli validateaddress xxx
"hdkeypath": "m/0'/0'/2'"

Needs a rebase as #2487 fixed the (unrelated) whitespace issue already. After that I can approve & merge (have tested)

Done!


Support capture screen

:Release Notes:
Support capture in ares-device
:Detailed Notes:
Implement capture screen feature to developer watch target display

Add ares-device --capture(-c)
Add capture option description in ares-device help
Update positive/negative TC for capture option

:Testing Performed:

Pass unit test on ose and auto
Pass eslint
Verify with CLI commands

Check capture file generated with targetname__display0__time.png
$ ./ares-device.js -c
Create rpi4_display0_20210115095912.png to /home/ye0607kim/project/github/ares-cli/bin
Success

Give display option and capture file name should be targetname__display1__time.png
$./ares-device.js -c -dp 1
Create rpi4_display1_20210115095826.png to /home/ye0607kim/project/github/ares-cli/bin
Success

Give directory path with display option
$ ./ares-device.js -c webOSCap -dp 1
Create rpi4_display1_20210115095944.png to /home/ye0607kim/project/github/ares-cli/bin/webOSCap
Success

Give file name and format(png, bmp, jpg) and check the generated file name & format
$ ./ares-device.js -c webOSCap/test1.bmp [or test1.jpg or test1.png]
Create test1.bmp  [or test1.jpg or test1.png] to /home/ye0607kim/project/github/ares-cli/bin/webOSCap
Success

Check /tmp/capture has been removed from device target after command working completed.

:Issues Addressed:
[PLAT-125564] Implement ares-device --capture

Code 👍
Verified 👍

Code 👍
Verified 👍

Code :+1:
Verified :+1:
@ssuminahn , Let's have a discussion about "unresolved conversation" s.

Code :+1:
Verified :+1:
@ssuminahn , Let's have a discussion about "unresolved conversation" s.

Code :-1:
Verified :+1:
I'm still working for apply code review.

Code :-1:
Verified :+1:
I'm still working for apply code review.

Code :+1:
Verified :+1:

Code :+1:
Verified :+1:

code :+1:
verify :+1:

code :+1:
verify :+1:


[AutoPR sql/resource-manager] [DO NOT MERGE]

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3627

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/2470


Improve the efficiency of exporting data by reducing number of database hits

To do this, the export method has to be reworked to pull out data by variable (column) instead of by subject (row).
This way, the variable responses can be processed and formatted at once per variable (instead of once per subject per variable). Since this will export each variable as a row, the resulting matrix has to then be transposed to match the current format.

[x] Sheet Export
[x] Grid Export
[x] Fix overall progress counter
[x] Remove Checkbox "aggregate" columns, these may be combined via SAS or R at a later point

@mrueschman I've finished the updating the exports. Can you test the SAS code generated on staging to see if everything is working as expected? Thanks!

I went through a couple exports and, on their face, everything looked intact. The SAS imports completed successfully and all the variable label/formats seemed to be in place.

Great thanks, I also ran the R import script and it appeared to work. We'll have to get an R expert to make sure it's working as intended though. Closing this issue then, if new errors come up, we can open new issues for those. Thanks!


In safari the time selector layout on the transition pop-up is broken

You mean this one?

Give me more info about the issue, please.

Yep, IMO should be better to have it left aligned in the same line.
Also I'd add an indicator saying that those are seconds!

Hahaha, ok, but that's not "broken".

I thought that was broken (sorry)

No problem :)
Is this fine?

Updated after our last conversation:

Ossom
On Jan 14, 2015 12:34 PM, "Javier Arce"<EMAIL_ADDRESS>wrote:

Updated after our last conversation:
[image: screen shot 2015-01-14 at 12 34 13]
https://cloud.githubusercontent.com/assets/4933/5738184/b49c0fac-9be9-11e4-8e87-176ff2393403.png
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb/issues/1719#issuecomment-69904009.


publish .jar

Hey, could you publish a built .jar? Just so this can be used more easily by others.
Ideally, could this use the non-deprecated event for future-proofing?

Added https://github.com/Redned235/HideCommands/commit/94ddb9eccf9a9f4b34eb0f27d504020bcc053550.


Pressing "Restart bot" and then "Test in emulator" opens the new instance of emulator every time

Version
MASTER - 10/30 bugbash
Describe the bug
Pressing "Restart bot" and then "Test in emulator" opens the new instance of emulator every time instead of restarting the current test in emulator.
To Reproduce
Steps to reproduce the behavior:

Work on a bot
Press Restart bot
Press Test in emulator.   Emulator opens.  Test the bot.
Work on the bot more in Composer
Press Restart bot
Press Test in Emulator.

ISSUE: instead of restarting the conversation, entirely new window of Emulator opens.  After a few minutes, you have 20+ emulator windows open.
Expected behavior
EXPECTED INSTEAD:  If you press Restart and then Test in Emulator, I expect the same instance of emulator to restart - not an entirely new instance to open.    Fallback: maybe open a new tab in the existing emulator instance if you want to preserve the past debugging session.
Screenshots

This is not the behavior on my Mac. @hibrenda can you repro on Windows?

Yes. this is reproduced on Windows.

This is the issue at Emulator side. This is an enhancement for Emulator. You could check the enchancement updates here


"az role definition create" throws trace, unhashable type: 'slice'

Describe the bug
Executing az role definition create is throwing error TypeError: unhashable type: 'slice', the command is broken.
praveen@Azure:~$ az role definition create --role-definition prav_devops_provisioning.json unhashable type: 'slice' Traceback (most recent call last):   File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 197, in invoke     cmd_result = self.invocation.execute(args)   File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 336, in execute     six.reraise(*sys.exc_info())   File "/opt/az/lib/python3.6/site-packages/six.py", line 693, in reraise     raise value   File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 310, in execute     result = cmd(params)   File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 171, in __call__     return super(AzCliCommand, self).__call__(*args, **kwargs)   File "/opt/az/lib/python3.6/site-packages/knack/commands.py", line 109, in __call__     return self.handler(*args, **kwargs)   File "/opt/az/lib/python3.6/site-packages/azure/cli/core/__init__.py", line 420, in default_command_handler     result = op(**command_args)   File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/role/custom.py", line 50, in create_role_definition     return _create_update_role_definition(cmd.cli_ctx, role_definition, for_update=False)   File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/role/custom.py", line 65, in _create_update_role_definition     names = [p for p in role_definition if p[:1].isupper()]   File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/role/custom.py", line 65, in <listcomp>     names = [p for p in role_definition if p[:1].isupper()] TypeError: unhashable type: 'slice'
To Reproduce
Execute az role definition create --role-definition prav_devops_provisioning.json
Expected behavior
Should create a role or throw proper error message indicating the issue in config.
Environment summary
Azure CLI on azure portal
Additional context
$ cat prav_devops_provisioning.json
[   {     "assignableScopes": [       "/subscriptions/111-222-333-444/resourceGroups/prav-*/providers/Microsoft.ContainerRegistry/registries/prav-*/providers/Microsoft.Authorization/roleAssignments/*"     ],     "description": "DevOps",     "name": "prav-DevOps",     "id": null,     "iscustom": true,     "permissions": [       {         "actions": [           "Microsoft.Authorization/roleAssignments/write",           "Microsoft.Authorization/roleAssignments/delete"         ],         "dataActions": [],         "notActions": [           "Microsoft.Authorization/*/Delete",           "Microsoft.Authorization/*/Write",           "Microsoft.Authorization/elevateAccess/Action"         ],         "notDataActions": []       }     ],     "roleName": "prav-DevOps"   } ]

This is also an issue for az role definition update, not just create.

@praveendhac, please remove the outside []  so to input an object instead of an array of such objects.
If you get such samples json from azure documents, please let me know.

At the same time, I will update CLI to avoid this bad error.

Seeing below error after removing []
No registered resource provider found for location 'eastus' and API version '2018-01-01-preview' for type 'registries'. The supported api-versions are '2016-06-27-preview, 2017-03-01, 2017-10-01, 2017-06-01-preview'. The supported locations are 'westus, eastus, southcentralus, westeurope, northeurope, uksouth, ukwest, australiaeast, australiasoutheast, centralindia, eastasia, japaneast, japanwest, southeastasia, southindia, brazilsouth, canadaeast, canadacentral, centralus, eastus2, northcentralus, westcentralus, westus2'.```

@praveendhac, could you please paste here the definition artifacts particularly the registries mentioned in the error? I don't believe the original definition you provided would cause this error.

Please reactivate with more information


financial settlements

hi. where do you handle financial settlements with partners in oscar?
how to you checkout with different partners?

Hi there. We use Github only for managing bugs and feature requests, and not for support. We recommend that you ask for help on Stack Overflow using the django-oscar tag, or joining our slack workspace or mailing list and asking a question there. Thanks.

but I think it's a bug. there is no way to checkout between partners. if we have multi sellers we can't handle financial settlements with partners

@RasoulRostami if you are able to write up in more detail what (a) you are trying to achieve and (b) how you think Oscar should be supporting this, then it is something we'd be able to review and consider. Unfortunately the one-line description you've provided on this issue is not really enough for us to understand either your objectives or where you think there is an issue with Oscar.


tugraph图数据库镜像下载

以下是参考地址
https://hub.docker.com/r/tugraph/tugraph-runtime-centos7

https://github.com/TuGraph-family/tugraph-db

源码仓库呢

https://github.com/TuGraph-family/tugraph-db


Collapse type/data family instances by default

Fixes #474. Here is what this code:
{-# LANGUAGE DeriveGeneric #-}
{-# LANGUAGE GADTs #-}
{-# LANGUAGE TypeFamilies #-}
module Example where

import GHC.Generics

data Letter = A | B | C deriving Generic

type family TFam a
type instance TFam a = Int

data family DFam a b
data instance DFam Int b
  = DFam1 { dfam1 :: Int }
  | DFam2 { dfam2 :: Int }
data instance DFam Char b where
    DFam3 :: { dfam3 :: Char } -> DFam Char Char
    DFam4 :: { dfam4 :: Char } -> DFam Char Char

now renders as, with some family instances expanded to show how they are rendered:

Looks good to me. Thanks!


It seems need a long progressing after i open xcodegr8

every time i open xcodegr8 type some code and make a save(cmd+s or autosave) there must be a long progress (On the SSD need 3 to 5 minutes, ordinary hard disk need 20 to 25 minutes )
i have two mac,both of them has this problem.
my system is macOS 10.12
By the way, macOS 10.11 do not have this long progress on save.but have this problem when i create a new file(class xib...)
so.. way this problem?
is it a bug or necessary?
how can i deal with?

I think I had the same problem. Turns out there's a process called tccd that's working really had to verify the validity of your unsigned Xcode while sucking up all of your CPU. You can stop it from doing this by disabling Xcode's access to Contacts in the Security and Privacy settings.

Thanks for pointing it out, @chrisellsworth!
@AMEstudio does the solution above solve your problem?

@chrisellsworth  @fpg1503
At first, thank you for reply me.Thank you for your efforts on this plugin.
This is a great plugin, however I use other method to resign now.
this method is noticed from The author of XVim
here is an example.
https://github.com/XVimProject/XVim/blob/master/INSTALL_Xcode8.md
Although I don't understand the principle, but I believe that you can try to put this principle to your plugin.
Let's make development easier : )

Woah! That's really awesome, I'll try to automate the XVim solution and add it to MakeXcodeGr8Again since it's working so well!

@fpg1503
That sounds good! Look forward to your efforts! Thank you very much : )

Moving this to #28


Added error handler

What
I have added the error handler.
How to review
run a couple tests sending false data
Who can review
?

Need to resolve conflicts by merging base branch into this one, give me (or anyone) a shout if you have any questions about resolving git conflicts as they can be a bit of a pain sometimes


Integration Test - Release - 2018/06/07 03:52:44

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest3 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


fix(api): Tailscale v1.32.3 compatibility

fix local API endpoint for tailscaled >= v1.32.3
https://github.com/tailscale/tailscale/compare/v1.32.2...v1.32.3

update dependencies

Can confirm that this restores compatibility with tailscaled 1.42.0, too!

Thirding that this solves the localapi connection issue I'm having; I'd love to see this put into the docker image!


[Move] Add freeze/unfreeze events in assoc account

Motivation
To emit events on freeze and unfreeze invocations
Test Plan
cargo test event emission in functional test

@bors-libra r=sblackshear

:pushpin: Commit 757919f has been approved by sblackshear

:hourglass: Testing commit 757919ffa3892c4eadf083443f0646788c4dca84 with merge d21fda1c3bd522b4e3479291777b59f23bbbbc05...

:broken_heart: Test failed - checks-actions_land_blocking_test

@bors-libra retry

:hourglass: Testing commit 757919ffa3892c4eadf083443f0646788c4dca84 with merge 8ac31f522edad2ea4777cd2ae1c96c8ba1fb0040...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit 757919ffa3892c4eadf083443f0646788c4dca84 with merge 71150328cd22f46bfdb677aa7d6f1e9df08ccaf2...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

@bors-libra r=sblackshear

:pushpin: Commit 21b4bba has been approved by sblackshear

:hourglass: Testing commit 21b4bbaa4526bbade81455b64740c07ecb910ed1 with merge 30aa86ec019d3408d46698a0ef8b3820c56a2d74...

@bors-libra r=sblackshear

:pushpin: Commit 5e5ba86 has been approved by sblackshear

:hourglass: Testing commit 5e5ba8684dcae34f65856b411ef92b6abafa6512 with merge b6a24fded0eb54a3a8f4e3f00d0a6eb46ce459c9...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: sblackshear
Pushing b6a24fded0eb54a3a8f4e3f00d0a6eb46ce459c9 to master...


Add Crystal::System::Process to split out system-specific implementations

Note that most of this change is just extracting parts of process.cr, mostly unchanged.
To see that for yourself, make sure to view the patch as
git diff -w master:src/process.cr src/crystal/system/unix/process.cr
This is based on https://github.com/RX14/crystal/commit/ed14ad5ee4b0422200a18f63350ab83d90153f61
The Windows counterpart of this
Closes #8536

Chat conversation about types to use at the boundaries of Crystal::System::Process

From my side this is ready, only as squash-merge.

Add Crystal::System::Process to split out system-specific implementations (#9035)
Note that most of this change is just extracting parts of process.cr, mostly unchanged.
To see that for yourself, make sure to view the patch as `git diff -w HEAD~:src/process.cr src/crystal/system/unix/process.cr`

I've already added it to the milestone (which effectively signals it's accepted). Just waiting for CI to finish.

Thanks!
I just always feel like I need to clarify how it should be merged.

Thanks
Wow this seems like a big step to windows support.


Add another text editor to body in bread

Version information

Laravel: v5.8.0
Voyager: v1.3.0
PHP: 7.3
Database: PostgreSQL 12

Description:
I want to user be able to chose text editor for page's body creation. So I want to add grapesjs editor so user could select which editor to use for body creation
Additional context
I want to get body editor selectable, for example:
"default": "option1",
"options": {
"option1": "Rich Text Box",
"option2": "GrapesJs"
}

No one is holding you back from doing so.
Everything else here


Allow *ClientBuilder.path() even when created with a URI

Currently, the path() call in the following code will throw an exception:
WebClient.builder("http://<IP_ADDRESS>:8080")
         .path("/foo")

.. because path() is currently allowed only when a builder is created with an Endpoint.
However, I feel like we don't really need to prohibit path() but could just concatenate, e.g.
WebClient.builder("http://<IP_ADDRESS>:8080")
         .path("/foo") // URI becomes http://<IP_ADDRESS>:8080/foo

WebClient.builder("http://<IP_ADDRESS>:8080/foo/")
         .path("/bar") // URI becomes http://<IP_ADDRESS>:8080/foo/bar
``

We need to define the case of multiple `.path()` calls though:

```java
// /foo/baz or /foo/bar/baz ?
WebClient.builder("http://<IP_ADDRESS>:8080/foo/")
         .path("/bar")
         .path("/baz")

I'm somewhat in favor of concatenating multiple .path() calls, inserting/deduplicating slashes between them automatically. Thoughts?
/cc @mauhiz

Concatenating paths seems a bit weird to me.
I'd vote for removing the path method and adding it as a parameter next to the factories that accept Endpoint.
It also brings a larger question as to us having several places where we do a runtime check even though it's not impossible to do a compile one, though may for example complicate inheritance or something (have several different types of builders for a type). I wonder if it's worth fixing this one location if we have the similar practice throughout the API.

I understand the point of passing a hostname or endpoint separately from the rest of URL, but as most kind of clients will call varying paths, is there a point in passing a path or an URL at ClientBuilder level?

@mauhiz I guess thrift clients are a common one that only access one path - without setting the path at the ClientBuilder level, thrift wouldn't work.

I see. Trying to make same API for REST client and Thrift client ... hits this kind of conundrum

Concatenating two paths that are set using path(...) seems weird to me too.
Retrofit supports concatenating paths set from the constructor and path() method but the logic is a little bit hard to understand. https://github.com/square/retrofit/blob/master/retrofit/src/main/java/retrofit2/Retrofit.java#L496-L543
For example,
WebClient.builder("http://<IP_ADDRESS>:8080/foo/")
         .path("/bar") // URI becomes http://<IP_ADDRESS>:8080/bar

WebClient.builder("http://<IP_ADDRESS>:8080/foo/")
         .path("bar") // URI becomes http://<IP_ADDRESS>:8080/foo/bar

So I think if it's possible to remove path(...) method from the builder, I would go for it. 😄

Removing `path(...) from builder sounds nice to me. 👍
WebClient.builder(SessionProtocol.HTTP, Endpoint.of("<IP_ADDRESS>", 8080),  "/foo/" )
WebClient.builder(SessionProtocol.HTTP, Endpoint.of("<IP_ADDRESS>", 8080))
WebClient.builder("http://<IP_ADDRESS>:8080/foo/")

Optionally requiring a path when constructing a builder with an Endpoint and removing builder.path() sound good to me, although it will make the builder() methods somewhat crowded. Any other ideas?

I'd personally lean towards longer factories with compile time safety vs shorter ones with runtime safety in general. +1 to the crowded methods


Use current gas price invalid filter

use current standard gas price for computing cost of trade and claiming for invalid filter

@Mergifyio backport dev


feat: allow SET with CIRCLE

Hey there 👋
While SETHOOK allows for CIRCLE as an input object, SET is currently missing this functionality (and some others too). So Tile38 already kind of allow users to create objects as CIRCLE, just not everywhere. If this is not the way you wanted to use it, then just close. 😄
This adds CIRCLE lat lng meters to the SET command. Given meters 0, it will fall back to create a point instead.
Wdyt?

I don't love the CIRCLE type. The problem I have with it is that it's not really a geometry per se. This will just a point and a radius. I don't like the default circle <-> geojson translation. But, I can see in your PR that you are calling Primitive when the radius > 0.  That's seems like a reasonable solution.

@tidwall always up to you. users can always use third party libraries to create objects of all kinds. now I remember that we had a similar slack thread about buffers recently. Yet I thought that if you provide the functionality to one command it makes sense for it to be consistent in the others as well. The geometry of a SETHOOK CIRCLE will probably be different from a turf.circle (depending on the steps on the circle).
But I also understand why you wouldn't want to do it, because where do you draw the line considering there are a lot of different shapes (think h3, sector, S2, ...).


Use Submodule

And integrate with tulul/tulul-quotes

I don't think this is necessary. From my experience, submodules complicate development. We can just treat tulul-quotes as an external resource.
It's a subjective opinion, though.

So if that is the case, how should we implement the /quote command?
I have two proposals:

Use submodule: By default, Heroku will recursively pull the submodule too. And since we have all of the data locally, we can simply read the yml from file.
Use Github API to retrieve that yml file from master branch, cache it at start (or every 5 minutes) to reduce network overhead.

What do you think? So you don't prefer first option?

You can just wget https://raw.githubusercontent.com/tulul/tulul-quotes/master/quote.yaml.

+1 @fushar 's idea. If the quote rarely changes, we can even load it once and store it on memory; restarting the app whenever a new quote is added. I think it's simpler.

Thanks. Settled. So no submodule for now. If something is changing in the future, feel free to reopen this.


chore: convert all .eslintrc files to JS

ESLint has a new flat config feature (which is going to be the default in v9 or v10), which requires a JS config file
https://eslint.org/blog/2022/08/new-config-system-part-1
https://eslint.org/blog/2022/08/new-config-system-part-2
https://eslint.org/blog/2022/08/new-config-system-part-3

it also renames the config to eslint.config.js, so i think we can wait until we fully support the flat config

@mcansh This change makes that people only need to change the name of the file, no need for extra formatting anymore (as automatic formatting isn't included in these templates)


DeepLinking and host filtering in react-router-native

I was having trouble to use DeepLinking in android.
In my android application, I set a DeepLinking object and added in my Manifest file in the MainActivity part :
<intent-filter>
<action android:name="android.intent.action.VIEW" />
<category android:name="android.intent.category.DEFAULT" />
<category android:name="android.intent.category.BROWSABLE" />
<data
android:scheme="http"
android:host="myhost"
android:pathPrefix="/threads" />
</intent-filter>
Then i start the application using an Intent containing the data "http://myhost/threads/threadId"
I would have expect the DeepLinking component to push the /threads/threadId route.
I solved my problem redefining a ne DeepLinking component with a new "push" method that removes the "myhost" from the url
push (url) {
const pathname = url.replace(regex, '').replace('myhost', '');
this.context.router.history.push(pathname);
}
The request would be to add a prop 'filterOut' to DeepLinking that could be used to filter out some part of the url generated by the RN Linking module.

This is a bug tracker, not a support system. For usage questions, please use Stack Overflow or Reactiflux where there are a lot more people ready to help you out. Thanks!


Fixed #25759 -- Added keyword arguments to various Expressions.

Takes over from #6147 which got a little confusing. I've removed **kwargs from most as_sql methods, and added explicit key word parameters where it made sense. The changes I've made have been rebased onto the original commit made by @kaifeldhoff.

The main doc edit I have is "key-word" -> "keyword". I'll send more when the patch is closer to completion. To simplify this patch, could we merge these changes separately: https://github.com/django/django/pull/6303?

Rebased and updated in https://github.com/django/django/pull/6338.


Covid-19 Timeseries Analysis with multiple model

Covid-19 third wave analysis using multiple models to test which one suits the best among the all . Basically to find the best model.
I would like to work on this under GSSOC'22
Using multiple model help in comparing between models .
Please assign this project to me .

Covid-19 third wave analysis victimization multiple models to check that one suits the most effective among all. to seek out the most effective model.
I would prefer to work on this beneath GSSOC'22
Using multiple models facilitates examination between models.
Please assign this project to Maine.

@shivam-s16 Hey, pls start working ...


Special characters are unsupported

If you try to use special characters like ä or Ö in a FlutterMath widget, it will not work.
It seems like the KaTeX fonts need to support that as it works in KaTeX. What do you think @znjameswu 😃

Yes, I'm planning to support that. I'll work on it in next release.

Support for unicode accents is added in v0.1.8. Welcome to check out the online demo!

But one thing needs to mention is that i, j based unicode accent looks very unnatural (e.g. ì). This is inherited from KaTeX (see flutter_tex's output) and MathJaX does this job even worse. If you have any idea to fix this, please let me know.

@znjameswu Very awesome!
No, I have no idea. However, after looking into it, I think that it is not too bad. I mean the characters do look unnatural as you said, but they still fit into the rendered output generally. Before the change, they would not even have the correct metrics, so it is a huge improvement 🚀


Remove references to the input types still not well-supported in current browsers?

The Inputs section on the CSS page of the documentation really makes it sound like the new input types in HTML5, like the color input type, are well-supported.

Most common form control, text-based input fields. Includes support for all HTML5 types: text, password, datetime, datetime-local, date, month, time, week, number, email, url, search, tel, and color.

Searching through the issues, the only discussions I can find relating to the newer input types like color are specifically in reference to the customizer, not the core Bootstrap scripts and styles.
I know that the reason there aren't any specific styles for these newer input types is that there just isn't wide enough browser support to do so yet, I just found it strange that the documentation seems to indicate these elements have some sort of support.

Rather than just removing the second sentence in that paragraph from docs/_includes/css/forms.html and submitting a pull request, I thought I'd raise the issue here first to see if I was reading that part wrong.
Basically, there is no special support whatsoever for the HTML5 input types like color, so I suggest we remove the sentence that suggests that there is support from the documentation, or add some sort of disclaimer.
Here is a fiddle so that you can see how the color input type looks for you.

Depends how you define "support" I guess. With the exception of the problem with the color picker one you noted, all other input types receive the correct basic rounded corners, glowing focus indication, etc. and can be used with additions like the optional icons, validation states, etc.

Yeah, that section of the docs is meant to say you can use the .form-control class on any textual input, including the ones listed. It's not to say we provide special support for all those controls' interactions, just their initial styling.


🔥 Add easy way to contribute a language

Add easy way to contribute a language with good documentation

Added easy way to contribute a language in this PR


Integration Test - Release - 2019/03/07 22:13:04

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest2 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


Possible rounding issue: unnecessary line break for certain widths

Apologies for not having a readily reproducible example since this only happens when the character count and font choice produce an unlucky width, but I believe there's a slight precision or rounding issue that's causing text that can normally fit in a single line to be broken when certain ratios are applied
Given two different headlngs:
Heading Example One

Container Width
Ratio
Result

226.484
1
No break

226.398
.8
Break

Heading Example Two

Container Width
Ratio
Result

209.406px
1
No break

209.406px
.9
No Break

209.344px
.7
Break

Sorry this might have come from an unrelated issue.


How to transfer the result of the grasp pose to ROS

Dear Author
If the GPD retrieval process has been executed, in which ROS topic will the grasping pose exported to the arm be stored?

You can figure that out with ros topic list. Also have a look at this line: https://github.com/atenpas/gpd_ros/blob/1e15b25a7280851ef8e9ab34e5e4ed54860fa673/src/gpd_ros/grasp_detection_node.cpp#L68 to see the topic name (clustered_grasps).


Revert #224474, git completion command patching

Description of changes
This PR reverts #224474, which patches the command ls in Bash scripts used by Git and patches the Bash completion script.
This prevents the command patching from breaking the definition of __git_complete_revlist_file where there's a local varible named ls, and prevents __gitcomp_builtin grep (for git grep) to be patched as the grep command.
This reverts commits b5d052d438df99d5a80e7c126551656d5ba74f75 and 820dc2a56057bcc396dbddbff6be91e1cec73f6f.

Things done

Built on platform(s)

[ ] 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
[ ] 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

[ ] Fits CONTRIBUTING.md.

@ofborg build gitMinimal gitMinimal.passthru.tests

The other commit b5d052d438df99d5a80e7c126551656d5ba74f75 from #224474 should be reverted too, because this diff is wrong:
 _git_grep ()
 {
 	__git_has_doubledash && return

 	case "$cur" in
 	--*)
-		__gitcomp_builtin grep
+		__gitcomp_builtin /nix/store/4sd4qx367yqczyrwfvr0p7cd0p5357sf-gnugrep-3.7/bin/grep
 		return
 		;;
 	esac

The grep argument here was referring to the git grep subcommand, not the GNU grep binary.

@andersk
Got to implement them again in yet another PR after revert.
Is there a way to test them all?

I dunno, I found this one by reading the diff between the source file and the built file.

Thank you for the tip.

staging branch more likely, but I can have a look based on rebuild state at the moment when this PR gets ready to merge.   It's really large rebuild; ofBorg says 106k.

If we revert both commits (as I think we need to) instead of just the one, Hydra might have most of the builds cached from before?

No, it won't.  That PR was merged three weeks ago.  We normally have multiple mass rebuilds per week.

The data partition on my laptop is corrupted, and I haven't got a useable setup to build and test the execution. The derivation doesn't build on NixOnDroid, but it is more likely a NixOnDroid issue.
Could someone test it for me?

Can we merge this? I don't think a revert needs much testing since we're just back were we started...

@ofborg build gitMinimal gitMinimal.passthru.tests git git.passthru.tests gitFull gitFull.passthru.tests


WIP: Add field "image_name" to metadata

Closes #820

It's still a draft, but it's definitely in the direction that I imagined. Thanks very much! :green_heart:


Clear doesn't work for extension contributed tasks

Already brought this change into master, just need to merge it into release.

Fixes #62474


[SPARK-36918][SQL] Ignore types when comparing structs for unionByName

What changes were proposed in this pull request?

Rather than using DataType.sameType for comparing structs when unioning by name, only compare the names and orders of the fields, since all we are doing is determining if we need to recreate the struct in a different order. After ResolveUnion is done, the normal union will handle if the types are incompatible or not.
Additionally, adds a check to the recursive struct handling as well, so we don't have to recreate a nested struct if only one of its parents or siblings are different.
Why are the changes needed?

Performance improvement, especially with nested nullable structs, which get wrapped in an If(IsNull()). Unioning three or more structs can explode the plan due to the multiple structs created extracting values from If(IsNull()) values when the projections are collapsed.
Does this PR introduce any user-facing change?

No, just a performance improvement.
How was this patch tested?

New unit test for the helper method, and existing tests still pass.

Can one of the admins verify this patch?

ok to test

cc @viirya FYI

Test build #143829 has started for PR 34166 at commit 6bb6ecb.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48342/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48342/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/48342/

Test build #143829 has finished for PR 34166 at commit 6bb6ecb.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143829/

Test build #143853 has started for PR 34166 at commit d527e38.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48366/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48366/

Test build #143853 has finished for PR 34166 at commit d527e38.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143853/

Thanks! Merging to master.

Can one of the admins verify this patch?

Test build #143920 has started for PR 34166 at commit 43f14d7.

Test build #143953 has started for PR 34166 at commit 43f14d7.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143920/

Can one of the admins verify this patch?

Test build #143979 has started for PR 34166 at commit 43f14d7.


client authentication failed  on mendeley website

I have filled the application ID, and when I login, the Mendeley site blocked me,  and reads :
"An invalid request was made by a third-party app
Client authentication failed"

I get exactly the same error.

Hi @flyinskybtx and @mschepers188,
please specify what product are you using, Document Server or Desktop Editors.

My apologies, my problem is for the Desktop version. Every time I'm redirected to the browser for authentication I get the above error.

Duplicate of #33


systemfunc child process error

Hello all,
an error is occurring randomly, with the following error message:

NCL ERROR MESSAGE: fatal:systemfunc: cannot create child process:[errno=12]

The error stem from the use of "systemfunc" in the function "write_references", in interface script "messaging.ncl":
systemfunc("wc -c < " + output_refs)
The "write_references" function is called at the beginning of many diagnostic scripts.
I tried to understand the origin of this error, and the only explanation I found is related to an overload of the system, which prevent to further open any additional file. Indeed, once this error occurs, the only way to fix it is to reboot the computer. I checked that my computer has a reasonable amount of ram memory, and usual values of shell resources (ulimit -v is unlimited (virtual memory), ulimit -n is 1024 (open file descriptors), ulimit -s is 8192 (maximum stack size)).
The explanation of the error could be due to an improper memory management by ESMValTool, for instance if too many files are kept open while they could be closed.
Does anyone encounter the same issue?
And would you have any suggestion or updated version to fix it?
Best wishes,
Josué Bock

Yes, the error is well known.
As you said, it is due to an overload of the system.
In v2.0 we have completely revised and improved the memory management, so it should not occur anymore. Specifically to the write_references function, it is also planned to replace it by a more general python module that can be applied to all diagnostics, regardless of language (see #278).

Alright, thank you for your answer.
Best wishes


Add Sentry to simplify debugging

Will be very useful to debug errors in staging and prod environments.

Summary by CodeRabbit

New Features

Introduced a GlobalError component for enhanced error handling using Sentry.
Added Sentry integration for improved error tracking and monitoring across client, server, and edge environments.

Chores

Updated .gitignore to include .env.sentry-build-plugin.
Added @sentry/nextjs package to dependencies.

Configuration

Updated next.config.mjs for Sentry integration with various configuration options.
Configured Sentry initialization for client, server, and edge environments.

Not directly, AFAIK it’s just a Sentry config.

On Jun 26, 2024, at 8:39 AM, Michael Poyatt @.***> wrote:
@mvpoyatt commented on this pull request.
On sentry.client.config.ts https://github.com/polymerdao/ibc-explorer/pull/143#discussion_r1655112400:
Does this file get used anywhere?
—
Reply to this email directly, view it on GitHub https://github.com/polymerdao/ibc-explorer/pull/143#pullrequestreview-2142259737, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAC3LOYOPVTM7GEMDUC2HMTZJLOEJAVCNFSM6AAAAABJ4X5WKSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCNBSGI2TSNZTG4.
You are receiving this because you were mentioned.


Reduce bootloader memory usage

By not requiring that the provided buffer must be able to contain one "erase-size" bytes.
This PR is the successor of #1314 and replaces it entirely.

feel free to merge when green
bors d+

bors r+ 🤞

bors r+


MSID: 91359 Version: 1 DOI: 2<IP_ADDRESS>7326

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=91359)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/91359/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>7326)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/91359/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/91359)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/91359

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/91359

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/91359/v1/91359-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/91359)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

(Issues to resolve)


Get an existing Load Balancer based on tags or other means.

Edit: Could someone move this to AWSX/Kubernetes as its related to the load balancers created with EKS.
Looking at the documentation for  GetLoadBalancer I can't tell if GetLoadBalancerArgstakes optional values since both arn and name are defined as string as opposed to say string? or optional<string>.
That being said there is a tags key, so I was wondering if it's possible to fetch a load balancer using just tags?
This is so something like this can be used.
const www = new aws.route53.Record("www", {
    zoneId: aws_route53_zone.primary.zone_id,
    name: "www.example.com",
    type: "A",
    ttl: "300",
    records: [aws_eip.lb.public_ip],
});

or
const www = new aws.route53.Record("www", {
    zoneId: aws_route53_zone.primary.zone_id,
    name: "example.com",
    type: "A",
    aliases: [{
        name: main.dnsName,
        zoneId: main.zoneId,
        evaluateTargetHealth: true,
    }],`

Can be possible when the Load Balancer is created using Kubernetes, though I am open for suggestions on how to create a load balancer using crosswalk/awsx via Pulimi and make it work with EKS.

That being said there is a tags key, so I was wondering if it's possible to fetch a load balancer using just tags?

Unfortunately, it is not possible to fetch a load balancer using tags alone. It might be possible to use the AWS SDK to search for the load balancer using its tags, then transform the result into an ID that is appropriate for use with GetLoadBalancer.

@pgavlin Would you be open to a pull request adding this feature?
I don't see how this isn't a bigger issue, I'd almost say it's a bug.
Not being able to access part of the infrastructure necessary to setup a EKS cluster, is a bit of a concern.
I'd say ALB is almost necessary as both NLB and ELB would use instance type load balancing, where some nodes might not run a specific pod and fail health checks.

@seivan What @pgavlin is pointing out is that the GetLoadBalancer invocation doesn't support retrieving via only tags. This is how the call is designed to work (to use either an ARN or a name). You could, as he suggests, use the AWS SDK to perform the search you want to do, so there's nothing preventing you from accessing the part of the infrastructure you want to use.

@leezen So why is tags part of the function definition?
Just to be clear, it's either name OR ARN, correct?

Correct, name or ARN. In some cases, additional parameters are available in arguments for filtering purposes to narrow down the resource.


Don't show the 'Choose payment request' screen if there is just one payment request

In normal use there is just one matching tx for an MBHD payment request.
(In theory a single tx can pay off multiple MBHD payment requests).
In the transaction details wizard:
To make the UI simpler and more consistent with how BIP70 payment requests are shown, remove the 'Choose matching payment request' screen if there is just one matching payment request.

Pushed some code that skips the 'Choose payment request' (MBHD payment requests) if there is just one payment request.
Also updated FEST test.
To see the 'choose payment request' screen you now have to create two payment requests and then go to somewhere like BCI and send a tx that pays both (use the Custom option on the left hand side). I've tested this ok. It's quite good fun to do.
Awaiting review and closing.

Verified working. Closing.


feat(@angular/cli): Update generate & new to use schematics

This feature is related to #6593

I've been following this feature and addon support for a while now. This looks fantastic!  Can't wait to try this out. :thumbsup: :thumbsup:

I'm going to have to add comments about schematics themselves here, unfortunately.
New e2e are generated with MyProjectPage instead of AppPage. We want to at least document that change in the commit and the release notes.
@Brocco Any reason the baseUrl value in tsconfig.app.json and tsconfig.spec.json changed?

And help remains a big factor. We need to support help right away.

@hansl I don't see any baseUrl change in this commit, but maybe you mean some relative changes. And as you mention this, I would like to point your attention to this : baseUrl has already been changed twice. Each time, issues emerged (see #5875 for the first change, and #7135).
baseUrl can't change everytime, it should be stable.

@cyrilletuzi that's because that change is in the schematics, not in this PR.


feat: add quasar preset

add here the vuex preset? maybe I can add also pinia preset

Using vuex with the Vue composition api requires

In fact for the composition api, the idiom used is that all plugins is that they provide a "useXXX" function so that components can write
const xxx = useXXX()

in the setup function.


Update Generic Pages “privacy-notice”

Automatically generated by Netlify CMS

👷 Deploy Preview for tech-works-for-us processing.

Name
Link

🔨 Latest commit
f74aaef49868938fc8917431c842d436b6dc899e

🔍 Latest deploy log
https://app.netlify.com/sites/tech-works-for-us/deploys/63da4f32a97e7500083673e0


FormHelper::input() is deprecated: edited README accordingly.

Description
Changed all four occurences of Form->input to Form->control. Also added the "type" option (=checkbox) on the "published" examples
fixes #187

This seems to break the tests in master.

How is a readme change gonna break tests? :)

True, I was just seeing the master being red after this merge.
Then it must be coming from the recent 3.4.2 release.