PowerToys Run Windows+S Shortcut not working

Microsoft PowerToys version
0.47.0
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Set PowerToys Run shortcut as Windows Key + S key, the default Windows Search shortcut. In previous version, 0.45.0, this shortcut worked perfectly fine and replaced Windows Search with PowerToys Run. However, with version 0.47.0, this shortcut triggers both the Windows Search and the PT Run search boxes simultaneously. You have to press the shortcut again for the Windows Search to disappear and for PT Run to show up.
✔ Expected Behavior
I have set PowerToys Run shortcut as Windows key + S key. In version 0.45.0, this shortcut worked perfectly fine. However, after updating to version 0.47.0, this shortcut triggers both the Windows Classic Search and PT Run search boxes. You have to press it again for only the PT Run search box to appear. It's incredibly annoying.
❌ Actual Behavior
PowerToys Run Windows Key + S  key shortcut is not working. Instead, both the classic Windows Start Search and PowerToys Run open at the same time. You have to press it again for it to work correctly.
Other Software
No response

+1 for this issue
Have to press the actual button instead of the shortcut. Very annoying

Thanks for reporting it.
We've changed the way the hotkey works to fix some focus issues.
We've added a fix to allow users to still use the old method through a setting: https://github.com/microsoft/PowerToys/pull/13557
This should be released in the next stable release.


Add option for adding ES vulnerabilities as findings

Context
Edgescan Vulnerabilities are unable to be ingested as Findings in Kenna.
Changes
Updated the Edgescan connector to be able to ingest Edegscan Vulnerabilities as Findings given the create_findings flag is provided.

@ShayVD I have restored the kenna sandbox account(https://edgescan.us.kennasecurity.com/). Please let me know if you can access it.

@ShayVD the PR is good to merge when you confirm the tests on your side

@annapoltieva Testing now. I seem to be getting a 401 unauthorized error when trying to upload to kenna despite a valid API token. Will keep you updated.


protobuf: 3.15.7 -> 3.15.8

Motivation for this change
https://github.com/protocolbuffers/protobuf/releases/tag/v3.15.8
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[X] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119574 at cec8914f run on aarch64-linux 1

193 packages marked as broken and skipped:

clickhouse
gnome3.gnome-books
gnome3.gnome-documents
gnunet-gtk
gtkd
libnats-c
libsForQt5.openbr
libsForQt512.akonadi-calendar
libsForQt512.akonadi-contacts
libsForQt512.akonadi-import-wizard
...

2 packages failed to build:

python38Packages.gpapi
python39Packages.gpapi

770 packages skipped due to time constraints:

MIDIVisualizer
adapta-gtk-theme
almanah
anbox
anki
ankisyncd
apache-airflow
apkid
apostrophe
aravis
...

32 packages built successfully:

criu
grpc
nanopb
protobuf (protobuf3_15)
protobufc
python38Packages.PyChromecast
python38Packages.aioesphomeapi
python38Packages.googleapis-common-protos
python38Packages.libarcus
python38Packages.mypy-protobuf
python38Packages.mysql-connector
python38Packages.proto-plus
python38Packages.protobuf
python38Packages.protobuf3-to-dict
python38Packages.pymumble
python38Packages.s2clientprotocol
python38Packages.snakebite
python38Packages.tensorboard-plugin-profile
python39Packages.PyChromecast
python39Packages.aioesphomeapi
python39Packages.googleapis-common-protos
python39Packages.libarcus
python39Packages.mypy-protobuf
python39Packages.mysql-connector
python39Packages.proto-plus
python39Packages.protobuf
python39Packages.protobuf3-to-dict
python39Packages.pymumble
python39Packages.s2clientprotocol
python39Packages.snakebite
python39Packages.tensorboard-plugin-profile
riemann_c_client

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.


Ppc64le multiarch

Draft PR, ppc64le support with multiarch images

/ok-to-test

docker buildx build cannot be simply followed up by docker push or docker run etc.; the docker daemon cannot directly access buildx build images.  If you're just building the image to see if it will build on multiple arches, then fine, but if you're planning on pushing or running the images, then you need to adapt those steps accordingly as well.
We can use neither buildx build --load (doesnt work with multiarch images), nor --push (we cant push after every build). Looks like this PR is a waste.
docker buildx and TARGETPLATFORM hacks are fine for upstream/CI but not for downstream; it doesn't seem like these dockerfiles could or would be used downstream anyway, but just checking.
No sure why.
There are a few files noted below which, once squashed, end up with only cosmetic changes; just remove those entirely.
Done
What about s390x?
That will be a separate PR from a different team.

/test all

/retest

On a strategic level, I think that it would be cleaner to add parameters to the build than to add tests about the architecture as part of the build process.
...
In conclusion, I wish we could keep arch/distro specific stuff only inside the code creating CI/CD pipelines, providing as many parameters as required to handle the specifics of each build (like the base image for the builder, for instance).

@sumitd2 , I would love to hear about your opinion on this subject, as it appears that your are pursuing in the exact same direction as before.

@ovalenti
Hi Olivier, apologies for the late reply. I am still trying to comprehend your suggestions. I am responsible for porting the code to power, and don't know much about the code or the functionality it implements. I have asked a Redhat team member (@robbycochran) to analyse your post and post a suitable reply. I am taking the same direction in the meanwhile as we are in a bit of a hurry for the first release, I just thought your suggestions could possibly be implemented in the second one. Thank you.

/test images

/test all


Update maven-shade-plugin to 3.4.1.

We have been seeing issues recently with "Error creating shaded jar: duplicate entry" errors. It may be that this is due to building twice without an intervening clean. If so, MSHADE-425 fixes this case in maven-shade-plugin 3.4.0.

Looks like @xvrl is looking into something similar in #15014 so I will close this one. I haven't figured out what the problem is.


Clippy issues

Can we fix some clippy issues (found some in nvrtc)? Maybe add some github workflow like there is in dfdx?

Yeah good ideas!

Fixed clippy warnings


feat: Support sso-apps and saml-idp

Add support for sso appllications and saml idp flags, also handle saml idp response

@orius123 @nirgur please review


fluent_syntax: Parsing error in select expression using a default variant, indented and positioned as the last variant.

What's the problem?
Select Expressions containing a last-positioned, indented default variant does not parse.
What should happen?
A select expression containing a last-positioned, indented default variant should parse.
The Fluent specifications allowed the use of indentation in variants and the use of a last-positioned default variants.
The two can lead to this special case that should be parsed.
What actually happened?
For instance, this inline_placeable failed to parse:
{ $userGender ->
[male] his stream
[female] her stream
 *[other] their stream
}

and we get this:
inline_placeable parser
- should parse a select expression *** FAILED ***
  Error(0,NonEmptyList(InRange(14,},}), InRange(79,*,*))) (core.scala:861)

Meaning the parser expected an inline expression (and want to close the bracket before the selector ->) or expected another default variant after the last variant defined. It seems the last variant is not parsed as a default variant but as a block_text following the female variant.

Additionnals notes:

This example is defined right there:
https://github.com/fluent4s/fluent4s/blob/690059bdf1262e4fb4195df41267fe06a6159998/fluent_syntax/src/test/scala/fluent_syntax/parser/core.scala#L860-L865
Tested version: 690059bdf1262e4fb4195df41267fe06a6159998

Investigation Notes:

Variant & Default Variant parsers accept uses of indentation.
variant_list parser failed to parse indented variants...

    Ftl.variant_list.parseAll("\n  [male] his stream\n  [female] her stream\n  *[other] their stream\n") match {
      case Left(e) => fail(e.toString)
      case Right(pars) => succeed
    }

variant_list parser
- should parse a list of variants (with one and only one default variant) *** FAILED ***
  Error(67,NonEmptyList(InRange(67,*,*))) (core.scala:531)

We observed the same behavior, variant_list expected another default variant...

variant parser can parse parts of others variants, see:

  it should "not parse part of another variant" in {
    Ftl.variant.parseAll("\n  [female] her stream\n  *[other] their stream") match {
      case Left(e) => succeed
      case Right(pars) => fail("parsed but should not have: "+pars.show)
    }
  }

variant parser
- should parse numeral variants
- should parse identifier variants
- should not parse part of another variant *** FAILED ***
  parsed but should not have: FVariant {
    key: IdentifierKey(value: female),
    value: FPattern {
      elements: [
        TextElement(value: her stream),
        BlockTextElement(indent: 2, value: None),
        TextElement(value: *[other] their stream)
      ]
    },
    default: false
  } (core.scala:505)

So what happened here? Pattern has expended too much.
We have the TextElement(value: her stream) we wanted, but also two extras:
First, a BlockTextElement(indent: 2, value: None),, this one parse the indentation and the newline and stop there because it does not have the right to parse an indented character like *. So after it, we have our text in a TextElement because this one did not know about indented characters... So there will be some changes to do.


Investigate the cause of the ratio between EMAILS_FETCHED and CREATED_SIGNALS

The ideal rate between EMAILS_FETCHED and CREATED_SIGNALS should be 1-4. It is almost 1-1 now.
Each fetched emails should give 4 signals

Date
Company (sender)
Subject
Action (email received)

Possible causes

Not creating all data signals for each email
Not creating data signals for all emails

Investigate the possible causes of this gap and come out with possible fixes.

@NickkTMD created new event properties to investigate this one.
This will be done just in debug mode. In release mode, we cannot work with one event for each, because it will blow our Amplitude limits. We will add a new approach to measure data signals in another ticket, and incorporate that in 0.4.2.


Need possibility to choose link onto concrete log line

Need the similar functionality as permalink in Github: https://github.com/qaprosoft/zafira/blob/4ba2edb9d98623f7e191132e031f0628a0b216e7/docker-compose.yml#L27
So /#!/tests/runs/id/info/id should be able to copy and share such link as well it provides possibility to share direct url to logs and later to screenshots as well

done


"Pointer Ballistics for Windows XP" page doesn't display

Describe the bug
The page https://www.microsoft.com/whdc/archive/pointer-bal.mspx doesn't display. Instead it says "Your request has been blocked".
To Reproduce
Steps to reproduce the behavior:

Go to https://www.microsoft.com/whdc/archive/pointer-bal.mspx
The following text is shown:

Your request has been blocked. This may be due to several reasons.

You are using a proxy that is known to send automated requests to Microsoft. Check with your network administrator if there is any proxy and what User-Agent they are sending in the request header.
Your request pattern matches an automated process. To eliminate, reduce the volume of requests over a period of time.
Reference ID: 18.4b6656b8.1607628179.4eaa4f6e

Expected behavior
A page should look like this: https://archive.is/20120907165307/http://www.microsoft.com/whdc/archive/pointer-bal.mspx
Screenshots
https://imgur.com/B7DmV4G
Desktop (please complete the following information):

OS: Win 7 64
Browser Cent Browser (Chrome-based)
Version <IP_ADDRESS> (Official Build) (32-bit) (portable) (Chromium 80.0.3987.163)

@GrayFace Please see "contact options" for technical support at https://developer.microsoft.com/en-us/windows/hardware/support/

Search doesn't yield any "contact options" text on that page and I've no reason to go through the trouble of finding it and re-reporting it there anyway.


They added bosses?

Look at this: https://www.youtube.com/watch?v=xt3UOnAv90o. They added bosses?
Does your script support this? Or do I need to search for the boss manually? Thanks!

Should support it
On Sat, Jun 30, 2018, 1:24 PM Ilia Grigorev<EMAIL_ADDRESS>wrote:

Look at it: https://www.youtube.com/watch?v=xt3UOnAv90o. They added
bosses?
Does your script support this? Or do I need to search for the boss
manually? Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/meepen/salien-bot/issues/156, or mute the thread
https://github.com/notifications/unsubscribe-auth/AF5lSfxYg0nuGq8KdgqKUO-I6g4a7oYdks5uB7RmgaJpZM4U-CzB
.

Yes, it supports boss rounds. I tried a new feature this morning.

Though it's little unstable. I hope @meepen will fix it soon.

Can you make a bug report about how it's unstable as well as your version
On Sat, Jun 30, 2018, 2:51 PM Makaka2<EMAIL_ADDRESS>wrote:

Though it's little unstable. I hope @meepen https://github.com/meepen
will fix it soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/meepen/salien-bot/issues/156#issuecomment-401559459,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF5lSekM3VSo9Q2IfwCrhlr4qvS3WUsQks5uB8jHgaJpZM4U-CzB
.

well. the headless mode didnt do anything on the boss planet. :(

Ok. I'm using the latest version, headless, re-downloaded it just 1 hour ago. Firstly, the percentage of the boss HP is broken. Secondly, when the fight ends, game freezing (but scores are counting anyway) and to continue you must reload the script.

I can second the report by @Makaka2 . Just fought a boss on an easy tile on Time Manipulation Planet. Percentage HP was like 0.XX% all the way, and headless failed to move on to a new zone after the fight but scores of the boss fight were recorded. Regrettably I forgot to add the --log switch after updating my script version to 2.1.5, so I can't provide you with logs to debug.

@darkl1ch You must use a special command besides primary. Add an "-e" supplement to usual "node headless" command.

I had the newest version at the time of the boss fight and my headless also stopped picking new zones after boss fight ended. It didn't even show the updated score. I waited few minutes (At least 5mins) and it was still looping without picking new zone or showing the new score. After restarting the app it showed the updated score and picked new zone.

In my case, whenever there is a boss fight, I must manually accept the battle as this script does not do it automatically. Am I missing something? :(

@darkl1ch you should update the script because boss-support is enabled by default without having to specify -e since commit [11f7e73fae266d8c0a495307c032eef075ef42ec] (https://github.com/meepen/salien-bot/commit/11f7e73fae266d8c0a495307c032eef075ef42ec) 17 hours ago.

@Spartanul if you are running the headless, maybe you didn't give your script enough time to notice the fight or the servers were hammered with join requests from other users and couldn't get through before you manually zoned to the fight?

@ahakola - I'm just running the simple "Userscript Mode".

@Makaka2 probably serverside issues, that might be also why there was hickups after the boss fight.

thx for the advices, but i had the latest version which i could, but during the boss level, the "headless" bot didnt do anything.

I've been checking https://wrya.net/tmp/salien-status/all.html , yet so far I haven't seen any red triangle that represents the Boss battle. Add it to the bot so it automatically starts the correct planet with the correct "Boss-battle" for maximum XP?

That hung up after the boss fight is probably caused by error in the network.js #162

Just fought a Boss, gaining 2.5 million XP
I joined https://discordapp.com/channels/459435127932715008/462403694122631169
Boss is added manually!!
This script helped me, but at the end it didn't press the button, but I did in time before the script relaunched the game!

Am running: index.user.js (version 29),  i dont see the script currently supporting the new skill that appears during the boss battle.

I'm using index.user.js version 29.  I've been manually clicking on the 'Fight!' button to start the boss fight.  I have also been manually hitting the '6' key on my keyboard to activate the healing ability.  I'm looking through the attacks.js file in the developer console to try and kludge together a HealAttack like FreezeAttack and try it out the next time I encounter a boss.

thanks @hptwo feel free to make a pr to add that! the issues with headless are fixed.


wallet failed to load

Hi, I can't get into my wallet. See screenshot below, and I have attached the logs. I have been using it ok for past 5 or so months. When I logged into the laptop browser 2 weeks ago, it requested an update. I then made a transaction, which worked fine. Today I initially forgot my password. So I hit forgot password, and thats when I started getting the screenshot message below. I did attempt to export wallet state into a new browser. But that didn't work either. Unsure if I executed that correctly though. I'm not great at this tech. Any assistance would be much appreciated. It won't work on my phone. I keep getting the screenshot below as well.

mutiny-logs.txt

If you forgot your password then just do a normal restore from seed words.

Tried that. Entered my seed words and that didn’t work. I get the same screenshot I sent.
Sent from Proton Mail for iOS
On Wed, Jan 31, 2024 at 7:34 pm, Tony Giorgio @.***(mailto:On Wed, Jan 31, 2024 at 7:34 pm, Tony Giorgio < wrote:

If you forgot your password then just do a normal restore from seed words.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

This error normally means you're unable to access mempool.space
Possibly your internet is shotty?

Not seeing a problem in the downloaded logs. Try opening the console logs of your browser and seeing what errors show up there.

I haven’t had any issues with my internet. I’ve been using Mutiny for 4-5 months on my existing wifi, with no problems until today
Sent from Proton Mail for iOS
On Wed, Jan 31, 2024 at 9:21 pm, benthecarman @.***(mailto:On Wed, Jan 31, 2024 at 9:21 pm, benthecarman < wrote:

This error normally means you're unable to access mempool.space
Possibly your internet is shotty?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

How do I access those logs if I can’t login to my wallet please?
Sent from Proton Mail for iOS
On Wed, Jan 31, 2024 at 9:28 pm, Tony Giorgio @.***(mailto:On Wed, Jan 31, 2024 at 9:28 pm, Tony Giorgio < wrote:

Not seeing a problem in the downloaded logs. Try opening the console logs of your browser and seeing what errors show up there.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

I haven’t had any issues with my internet. I’ve been using Mutiny for 4-5 months on my existing wifi, with no problems until today

your log file indicates slow/bad internet

How do I access those logs if I can’t login to my wallet please?

press f12

Any updates here?

It’s working ok now. Thanks
Sent from Proton Mail for iOS
On Tue, Feb 6, 2024 at 9:08 pm, benthecarman @.***(mailto:On Tue, Feb 6, 2024 at 9:08 pm, benthecarman < wrote:

Any updates here?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Set corner radius

Can you add the possibility to change the value of the corder radius please?

I'll add the property asap. Thanks for the suggestion.

You can now change the cornerRadius property. :+1:


golang support 1.21+

I haven't actually tested go 1.22 yet since it hasn't been released. I did confirm that this works for 1.21.
I completely forgot about the "trim trailing whitespace" option. If it is too much I will try to work up the motivation to fix it.

I completely forgot about the "trim trailing whitespace" option. If it is too much I will try to work up the motivation to fix it.

When your PR is pulled in, our process will stomp on your whitespace changes in the license header comment block and put the whitespace back.
It will be a bit noisy in the commit for no good purpose.

Ok it should be cleaner now and I fixed the problems.
However, since I like to live dangerously, I'm not even bothering to rebuild Ghidra 😝


Proposed internal bridge refactor

Description
This is a proposed new API by @jackzampolin

closing in favor of https://github.com/cosmos/gravity-bridge/pull/287


Update accessibility-report.js

Added a container class and set max width to limit the HTML report width.
Using this approach will ensure a consistent look across various screen sizes, in particular ultra-wide monitors.

James (@w4dd325), I think this is a good idea, but how about making the max-width: 1920px; instead of  max-width: 1200px;?
The most common monitor resolution these days is 1920 x 1080 for desktop monitors and laptops, and I believe these reports would be used when debugging an issue in a desktop or laptop, so the user might want to have as much resolution as possible, specially for small DOM elements more difficult to spot.

I'm open to suggestions.
1200px was an example from another reporter that I used as a guide.
I have just tested 1920px and it looks fine, so I have changed the PR to reflect this.


User management form

It should be possible to rename/edit and delete users
Bugs from TEI, 08.10.15 - n.14

Related to issue #110


Fix a wait for input-event

This PR will fix #670.
Problem
On GUI MacVim, when channel-I/O event is detected in run-loop, it interrupts a wait for mapped-key sequence.
Repro steps
sample.py
import time
while True:
    print("hello")
    time.sleep(1.0)

sample.vim
set timeoutlen=10000
inoremap jk <Esc>
let job = job_start(['python3', '-u', 'sample.py'], {'callback': {c, m -> 0}})

Start GUI MacVim:
/path/to/MacVim.app/Contentes/MacOS/Vim --clean -S sample.vim -g
Type j and no other input (also mouse), then:
Expected: Vim waits for mapped-key sequence until a time of timeoutlen (10sec) passes
Actual: Wait-state is interrupted within 1sec, and text j is determined
Cause
[MMBackend waitForInput:] can return by also channel-I/O event (dispatch-source event).
Thus, in that case, Vim decides there is no key input and interrupts a wait for mapped-key sequence.
Solution

Ignore channel-I/O event in run-loop in [MMBackend waitForInput:]

OMG I think this has been rusing me for a few months now and I thought it was a bug in a plugin... thanks for this!


Realtek ASIO wont be removed

Realtek added support for ASIO. DDU doesn't remove ASIO related entries from Realtek:

Windows Registry Editor Version 5.00
[HKEY_LOCAL_MACHINE\SOFTWARE\ASIO\Realtek ASIO]
"CLSID"="{A80362FF-CE76-4DD9-874A-704C57BF0D6A}"
"Description"="Realtek ASIO"

Thanks I'll check this, DDU still need some work for realtek.
Le dim. 9 févr. 2020 10 h 02, Andreas<EMAIL_ADDRESS>a écrit :

Realtek added support for ASIO. DDU doesn't remove ASIO related entries
from Realtek:
Windows Registry Editor Version 5.00
[HKEY_LOCAL_MACHINE\SOFTWARE\ASIO\Realtek ASIO]
"CLSID"="{A80362FF-CE76-4DD9-874A-704C57BF0D6A}"
"Description"="Realtek ASIO"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Wagnard/display-drivers-uninstaller/issues/48?email_source=notifications&email_token=ACXWDGLWEFW7EDV5CY7F3RLRCALIRA5CNFSM4KSCGNB2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IMCKUUQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACXWDGNV3IVMWAH33HG5ALTRCALIRANCNFSM4KSCGNBQ
.

Added with commit : 185888152ea9c9f4c7e7e2175aa183e1965b9f45


NodeJS DOM Emulation

Various parts of Qooxdoo depend on there being global objects for window and document, and also that these behave like their browser DOM equivalents to one degree or another.  In many circumstances, a very basic emulation is acceptable but it is possible to incorporate code which requires a much more complete implementation.
The best (worst) example of this is qx.bom.Selector which does lots of feature-detection tests during class initialisation that break badly on NodeJS; this class is depended on by lots of other classes, so even if not used at runtime it causes an issue.  If you want to manipulate DOM-like structures on the server (eg you want to use Qooxdoo's upcoming JSX support ;) ) you may want to use qx.bom.Selector anyway.
This PR adds conditional support for JSDOM to be used if it is available - i.e. if you do not add JSDOM via npm i jsdom then the previous behaviour is unchanged.

closinfg because the commit history is messed up


Support GHC 9.6

Fixes #6

Hi @ocharles. Is your PR blocked?

Nope, just tight on time at the moment :) I'll get this merged soon.

No worries. Let me know if there is any way I could help.


Send ANSI colors to non-tty output

Awesome tool!
It would be nice if there was an option to force ANSI colors to non-tty output.  In particular, I'd like to use the tool like this:
command | gnomon --force-ansi --medium=1.0 | tee /tmp/log
I'm fine with having some ANSI jibberish in the log file.

After playing with gnomon some more, it looks like it's doing more than just sending ANSI control characters to update the time.  Perhaps a better solution would be to have an option such as:
command | gnomon --medium 1.0 --save-log=/tmp/log
Where standard out would be updated in real time, and then the non real time log would be saved to /tmp/log.

Would be nice if we have colors +1

That would be a great feature.


fix: improve searchbar colors

brighten the search box so that colored images don't affect text contrast.

Before:

After:

Before:

After:

@abvthecity do you think it may be easier to just make it opaque?


Invalidate cachedZoomParameters when updating configuration

The cachedZoomParameters don't get invalidated when updating the configuration. That's why running this code:
fotoapparat.updateConfiguration(UpdateConfiguration(flashMode = torch()))
fotoapparat.setZoom(0.5f)

results in the torch being off. By clearing the cachedZoomParameters when calling updateConfiguration(), this piece of code results in the torch turned on (as intended).

Thanks for finding this issue @garnele007 !  Actually I realised we don't need to cache the zoom parameters. We should directly use the camera.parameters.


Requirements for Production setup.

${\color{red} Please\ Do\ Not\ Create\ the\ Resources }$
Stating the required things for the production setup of Feature flag service on AWS.

Tables in Dynanmodb
FeatureFlag

Id (string) Partition key
name (string) (GSI)
description (string)
createdAt (number) ->  dynamodbType
createdBy (string)
updatedAt (number)
updatedBy (string)
status (string)

FeatureFlagUserMapping

userId (string) Global Secondary Index
flagId (string) Partition key
status (string)
createdAt (number)
createdBy (string)
updatedAt (number)
updatedBy (string)

Indexes on the tables

Feature Flag table - name (GSI)
FeatureFlagUserMapping - userId (GSI)

For the IAM roles and users, we have used Cloudformation to create programmatically, sharing the template below.
Cloudformation template =
CF-template.txt
REF: https://aws.amazon.com/blogs/mt/share-reusable-infrastructure-code-aws-cloudformation-modules-and-stacksets/
The env variables for main branch

ARTIFACTS_BUCKET (generated by Cloudformation)
AWS_ACCESS_KEY_ID (Pipeline user access key)
AWS_SECRET_ACCESS_KEY (Pipeline user secret key)
CLOUDFORMATION_EXECUTION_ROLE (Cloudformation template execution role)
PIPELINE_EXECUTION_ROLE (Pipeline user execution role)
REGION
STACK_NAME (Cloudformation stack name)

In the lambda execution role, we need to add permissions of Dynamodb.

CreateFeatureFlag = Dynamodb.PutItem
CreateUserFeatureFlag = Dynamodb.PutItem
UpdateFeatureFlag = Dynamodb.UpdateItem
UpdateUserFeatureFlag = Dynamodb.UpdateItem
GetFeatureFlag = Dynamodb.GetItem
GetUserFeatureFlag = Dynamodb.GetItem
GetAllFeatureFlags = Dynamodb.Scan
GetUserFeatureFlags = Dynamodb.Query

Update in Tables schema in DynamoDB
Tables in DynanmoDB
FeatureFlag
Partition Key - Id (string)
FeatureFlagUserMapping
Partition Key - userId (string)
Sort Key - flagId (string)

Note
Indexes are not required

@vikhyat187 Why does one table have Pascal case for keynames, while the other is camelCased? Could we have consistency in the field and table names please?

Updated @ankushdharkar, now we have all the keynames in camelCase

We need to create service level rate limiter, as lambdas can scale up infinitely and shoot up the AWS costs. So we need to create a POC for service based rate limiter.
Ticket : #66

@ankushdharkar this change was done in this PR #78

This is deployed on Prod account, so closing this


Update go major updates

This PR contains the following updates:

Package
Type
Update
Change

github.com/gofrs/uuid
indirect
major
v3.3.0+incompatible -> v4.4.0+incompatible

github.com/golang/glog
indirect
major
v0.0.0-20160126235308-23def4e6c14b -> v1.1.2

github.com/kevinburke/ssh_config
indirect
major
v0.0.0-20190725054713-01f96b0aa0cd -> v1.2.0

github.com/pulumi/pulumi-azure-native-sdk
indirect
major
v1.104.0 -> v2.4.0

github.com/pulumi/pulumi-azure-native-sdk/resources
require
major
v1.104.0 -> v2.4.0

github.com/pulumi/pulumi-azure-native-sdk/storage
require
major
v1.104.0 -> v2.4.0

github.com/texttheater/golang-levenshtein
indirect
major
v0.0.0-20191208221605-eb6844b05fc6 -> v1.0.1

gopkg.in/yaml.v2
indirect
major
v2.4.0 -> v3.0.1

Release Notes

gofrs/uuid (github.com/gofrs/uuid)
v4.4.0+incompatible
Compare Source
v4.3.1+incompatible
Compare Source
v4.3.0+incompatible
Compare Source
v4.2.0+incompatible
Compare Source
v4.1.0+incompatible
Compare Source
v4.0.0+incompatible
Compare Source
v3.4.0+incompatible
Compare Source

golang/glog (github.com/golang/glog)
v1.1.2
Compare Source
Bugfix release.
What's Changed

glog: populate symlinks -log_link directory by @chressie in https://github.com/golang/glog/pull/64

Full Changelog: https://github.com/golang/glog/compare/v1.1.1...v1.1.2
v1.1.1
Compare Source
Bugfixes since the larger v1.1.0, which have been addressed.
v1.1.0
Compare Source
Tagging v1.1.0 after syncing glog with internal changes

kevinburke/ssh_config (github.com/kevinburke/ssh_config)
v1.2.0
Compare Source

pulumi/pulumi-azure-native-sdk (github.com/pulumi/pulumi-azure-native-sdk)
v2.4.0
Compare Source
v2.3.0
Compare Source
v2.2.0
Compare Source
v2.1.1
Compare Source
v2.1.0
Compare Source
v2.0.0
Compare Source

texttheater/golang-levenshtein (github.com/texttheater/golang-levenshtein)
v1.0.1
Compare Source

go-yaml/yaml (gopkg.in/yaml.v2)
v3.0.1
Compare Source
v3.0.0
Compare Source

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

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

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: go.sum
Command failed: docker run --rm --name=renovate_sidecar --label=renovate_child -v "/tmp/renovate/repos/github/renovate-reproductions/23818":"/tmp/renovate/repos/github/renovate-reproductions/23818" -v "/tmp/renovate/cache":"/tmp/renovate/cache" -e GOPATH -e GOFLAGS -e CGO_ENABLED -e GIT_CONFIG_KEY_0 -e GIT_CONFIG_VALUE_0 -e GIT_CONFIG_KEY_1 -e GIT_CONFIG_VALUE_1 -e GIT_CONFIG_KEY_2 -e GIT_CONFIG_VALUE_2 -e GIT_CONFIG_COUNT -e CONTAINERBASE_CACHE_DIR -w "/tmp/renovate/repos/github/renovate-reproductions/23818" ghcr.io/containerbase/sidecar:9.19.4 bash -l -c "install-tool golang 1.21.0 && go get -d -t ./... && go install github.com/marwan-at-work/mod/cmd/mod@latest && mod upgrade --mod-name=github.com/pulumi/pulumi-azure-native-sdk -t=2 && mod upgrade --mod-name=github.com/pulumi/pulumi-azure-native-sdk/resources -t=2 && mod upgrade --mod-name=github.com/pulumi/pulumi-azure-native-sdk/storage -t=2 && mod upgrade --mod-name=gopkg.in/yaml.v2 -t=3 && go mod tidy && go mod tidy"
go: errors parsing go.mod:
go.mod:6:2: require github.com/pulumi/pulumi-azure-native-sdk/resources: version "v2.4.0" invalid: should be v0 or v1, not v2
go.mod:7:2: require github.com/pulumi/pulumi-azure-native-sdk/storage: version "v2.4.0" invalid: should be v0 or v1, not v2


[Feature Request] Add Support for LESS' Nested Selectors

Let's say I have some rectangles in Sketch, which I named .rect .bordered. I then add some ovals, which I name '.oval .bordered`. I would like to be able to do this - which is perfectly valid LESS:
.bordered {
    border: 3px solid red;

    .rect { background-color: fade(red,75%);
    .oval { background-color: fade(blue,55%);
}

Currently, the nested selectors - .rect and .oval - get totally ignored, and all you get is a red border, and no background. To get what we want, right now it is necessary to repeat ourselves, and do:
.bordered.rect {
    border: 3px solid red;
    background-color: fade(red,75%);
}

.bordered.oval {
    border: 3px solid red;
    background-color: fade(blue,55%);
}

As you can imagine, for complex layouts with hundreds of nested objects, being able to use LESS' nested selectors is vital!

This would be amazing. Willing to pitch in on the feature... not sure how to get started.

@luxlogica, @andrewmbacon: this does work, just your syntax is a bit wrong
You have this:
.bordered {
    border: 3px solid red;

    .rect { background-color: fade(red,75%);
    .oval { background-color: fade(blue,55%);
}

This will match a layer setup like this: .bordered > .rect or .bordered > .oval, so basically a group with .rect and inside a layer with .oval or .rect
To match a layer named .rect .bordered change your LESS to:
.bordered {
    border: 3px solid red;

    &.rect { background-color: fade(red,75%);
    &.oval { background-color: fade(blue,55%);
}

@JohnCoates Doh!
That's awesome! 👍


Fix english wording

Once again corrected to proper English like last it was done last November in former blocknewsletter.php: https://github.com/PrestaShop/blocknewsletter/commit/bed58cdbf656b2e9734daad37ebf22d0ba7992b4
And why did you change this distinct module name "newsletter" to an unclear "ps_emailsubscription"? Do you really think someone would confuse the former unique name with documents sent by post? I mean, welcome in the 21. century. :)

Thank you.
@AlexEven can you tell us if it's okay for you?

@maximebiloe yes you can merge.
@Nobodaddy: we changed the name to better depict what the module is about. "Newsletter" can be interpreted as "I want to send newsletters" for example.
Thanks!

Thank you @Nobodaddy!


🐛 Qualify js globals with globalThis to prevent name pollution.

If you import a module and call it Array, the array type pattern can break because it calls Array.isArray which may be shadowed or undefined by the import.

Closed by https://github.com/ren-lang/compiler/commit/52ca3ecac67a1b2a7bbac712ba0f6d375f219c89.


[cxx-interop][rebranch] Do not crash for static operator()

This fixes an assertion failure:
Assertion failed: (isInstance() && "No 'this' for static methods!"), function getThisType, file DeclCXX.cpp, line 2636.

Clang changed it's handling of member call expressions in https://github.com/llvm/llvm-project/commit/af4751738db89a142a8880c782d12d4201b222a8 causing the assertion to fail when synthesizing a forwarding function declaration for static operator().
rdar://133257179

@swift-ci please smoke test


fix(package): add basic and full css output for patternfly

Closes #306

Deploy preview for pf-next ready!
Built with commit 81eca5d29ad41c70f15702f14b5ddc5ab1b26c98
https://deploy-preview-315--pf-next.netlify.com


Create a manually started workflow for bulk run of integration tests

Description
We've been looking to increase test reliability and that has lead to many runs and re-runs of the integration test in GitHub actions.  Creating a manually started workflow that runs the integration tests on both jdk11 and jdk17 10x times.
Issues Resolved

Related #1917

Testing
See an example of this run here.
Check List

[ ] New functionality includes testing
[X] New functionality has been documented
[X] Commits are signed per the DCO using --signoff

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

The changes look good to me. Looks like there's a minor lint issue with the integration-tests.yml file. Clever use of matrix!

Codecov Report

Merging #1937 (d86326a) into main (be876c0) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1937      +/-   ##
============================================
- Coverage     61.00%   61.00%   -0.01%
  Complexity     3233     3233
============================================
  Files           256      256
  Lines         18088    18087       -1
  Branches       3224     3224
============================================
- Hits          11035    11034       -1
  Misses         5470     5470
  Partials       1583     1583

Impacted Files
Coverage Δ

...ecurity/ssl/rest/SecuritySSLReloadCertsAction.java
84.78% <0.00%> (-0.33%)
:arrow_down:

Continue to review full report at Codecov.

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

Testing
See an example of this run here.

@cliu123 Here is an execution of this PR on my branch, 20 out of 20 passed, but you can drill in to see failures that were not blocking like our normal workflow.

Where is the button triggering this workflow? I mean I don't see the button in the Checks page. How was the example fun triggered?


Disallow build failures on JDK 12 [ECR-3133]:

The flaky test no longer runs at all (see abf2d850).
This reverts commit 8bdd31bd
Overview

See: https://jira.bf.local/browse/ECR-XYZ
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage remained the same at 85.398% when pulling 947dc85b3462773a4b6000f8bdef16685216d1f6 on dmitry-timofeev:disallow-failures-on-12 into 598fe20cffa5b22bd3d31c5fdcd14b77754b9189 on exonum:master.


serverType to 1 ,serverType=3 means clan channel in 2017 client

serverType=3 means clan channel in 2017 client

thanks


[discussion] Moving away from euclid in favor of local types

On multiple occasions, there were users complaining on GitHub and Discord about how euclid's typed interface, with its distinction between Points, Vectors, Sizes and Scales, gets in the way when the Godot API itself is untyped. While there is the simple solution of conversion (among euclid types, or through mint to types from another library), new users often aren't aware of them, and the method calls can get quite verbose at times. There is also the issue with porting, where someone will attempt to port some GDScript code to Rust, and want to access the Godot versions of vector methods. We're currently handling those with extension traits, but local types can do that a little bit better.
The use of euclid types in the public interface also caused version conflict problems in user code, leading to recurring PRs to bump them: #216, #523, #588
Therefore, I think it might be valuable to eventually replace euclid types with local newtypes in our public interface instead. This is not a suggestion that we should reinvent the wheel: we can just pick any vector math library we want to use as the underlying implementation, as long as its types have the correct layout. For conversion to types from other libraries, we can add (optional?) mint support.
This is obviously too late for inclusion in 0.9, but we can consider it for 0.10.

Has anyone started working on this yet? If not, I'm willing to give it a go.

Thanks for offering help! Before delving straight into an implementation, we should probably discuss the

This is not a suggestion that we should reinvent the wheel: we can just pick any vector math library we want to use as the underlying implementation, as long as its types have the correct layout.

part 🙂

True. I think glam can be an option given that it directly advertises repr(C): https://docs.rs/glam/0.13.0/glam/index.html, but I haven't explored its API enough yet. I'm not sure if it suits our use cases enough.

Just quickly made a test: glam has no required dependencies, a cargo new crate with it compiles in 9s.
nalgebra has 13 indirect dependencies, compiles in 1m 20s.
Huge difference, and I'm definitely a fan of lightweight libraries 🙂 it also seems to be specifically for game development, not a general-purpose linear algebra package.

What would the general approach be?

We would offer "front-end" vector types
 #[repr(C)]
 pub struct Vector3 {
     pub x: f32,
     pub y: f32,
     pub z: f32,
 }

 impl Vector3 {
     // map Godot API 1:1
 }

and then "memcpy" them to glam::Vec3 everywhere and back.
Could even use mint::Vector3<f32>, but not sure if the generics contribute a lot (maybe feature-gate).

The local newtypes you mentioned
pub struct Vector3(glam::Vec3);

impl Vector3 {
    // map Godot API 1:1
}

This would avoid conversion, but there's no more direct access to the fields.

Use selected glam types directly in the API. Would be easiest for people already using glam, but repeats the problems with euclid (versioning, different function names).

Indeed, I wasn't considering compile time in my previous comment. If we take that into consideration then glam is definitely more competitive here.

offer "front-end" vector types

Good idea! Direct field access is definitely something good to have. We can have private methods for value and reference conversion to glam types and delegate any appropriate calls to the library. rustc should be able to optimize all of them out.

Use selected glam types directly in the API. Would be easiest for people already using glam, but repeats the problems with euclid (versioning, different function names).

Indeed. That's what we're trying not to do here.

Sounds good! Since vector types are used across the entire library, such a refactoring is quite an endeavour. Maybe it would make sense to split the efforts into steps, with intermediate working versions?
I could imagine something like this:

Only change the definitions, not the usages.
This would mean adding glam, removing euclid and defining our own type, with methods that have the same names as the euclid ones. We could even leave the Vector*Godot traits for the time being.
Could be done just by starting with an struct with zero methods, read compile errors and implement the missing methods using glam conversions.

Rename methods
After everything still works, all methods can be renamed to fit the Godot counterparts.
There might be a few manual interventions where the signature differs, but for the most part I'd expect a good IDE to do all the work here.

Port missing Godot methods
There are probably methods which are part of GDScript's API, but not Rust's.
They can be implemented.

Also, the above parts could be done first for Vector2 as a proof-of-concept, and then for Vector3 (or vice versa).
Then @Demindiro would also be more free to choose if he still wants to work on it, and if yes on what parts.

Thinking about this a bit more, if we provide our own types, we still need to deal with a lot of code duplication just for delegating (just from the operators alone). It would be almost as much as providing our own implementation, since most vector operations are one-liners. But if it helps for clarity and consistency with Godot, it's probably worth it.
Regarding back-end math library, there are a few contenders:

euclid (current one)
glam
ultraviolet -- like glam, focus on minimalism and performance

There's also nalgebra, nalgebra_glm, vek and quite a few others, but on first sight they don't seem to have a "simplicity first" philosophy, and a lot of the available crates get lost in advanced mathematics irrelevant to gamedev.
Also, arewegameyet provides an overview of crates, and mathbench-rs some benchmarks. I think the API and "bloatedness" is more decisive for our use case, but having good performance is definitely a nice to have.

ultraviolet seems to be focused on SIMD, which is a bit removed from our use case. It's also not clear whether the types are repr(C) without extra alignment, a deal-breaker in our case.
Since Demindiro has made a PR based on glam I think it's fine to go with it for now. We can switch to another implementation any time later.

Thinking about this a bit more, if we provide our own types, we still need to deal with a lot of code duplication just for delegating (just from the operators alone). It would be almost as much as providing our own implementation, since most vector operations are one-liners. But if it helps for clarity and consistency with Godot, it's probably worth it.

While I still prefer nominal type systems for most of the use cases, this is one of the niches where a structural typing system would really shine. Without a "standard" implementation widely agreed upon, we're bound to repeat the old problem if we want to provide at least some ease of use. If we ignore that though, there's the ggez way of only providing mint types (that don't support any operations on their own).


gaurantee the order of writes when writes come from the same session

Due to https://github.com/risingwavelabs/risingwave/blob/main/src/source/src/table.rs#L86,
we may not know which writes will win when there are multiple ones.
(pk conflict, and due to https://github.com/risingwavelabs/rfcs/pull/4/files#diff-73ca4924ee3c22e62b50a34acb513c5252bc3d5c44075d5b0398a2387f6d5c4dR32, the first writes win and the rest will be ignored)?
from the same session seems to be a reasonable setting

A quick but working workaround might be always scheduling the DML to a fixed node and choosing a fixed parallelism here. 🥵

A quick but working workaround might be always scheduling the DML to a fixed node and choosing a fixed parallelism here. 🥵

https://github.com/risingwavelabs/rfcs/pull/4/files#diff-73ca4924ee3c22e62b50a34acb513c5252bc3d5c44075d5b0398a2387f6d5c4dR54 seems there was a plan

close in favor of https://github.com/risingwavelabs/risingwave/issues/7213


Build static site to upload to GitHub Pages

This way we don't have to deal with Vercel limits.

i will do this if vercel doesn't accept my limit increase.

switched hosting provider


Problem with deleting actors/sensors intertwining with segmentation faults

Hello,
I want to sample a bunch of snapshots of a Carla world and then reset the world and move on sampling.
   def reset(self):
        self.client.apply_batch([carla.command.DestroyActor(x) for x in self.actor_list])

        self.actor_list = []
        self.collision_hist = []

        # Spawn vehicle
        if self.random_spawn: transform = random.choice(self.spawn_points)
        else: transform = self.spawn_points[1]
        self.vehicle = self.world.spawn_actor(self.vehicle_bp, transform)
        self.vehicle.set_autopilot(True)
        self.actor_list.append(self.vehicle)

        # Attach and listen to image sensor (RGB)
        self.ss_cam = self.world.spawn_actor(self.ss_camera_bp, self.ss_cam_transform, attach_to=self.vehicle, attachment_type=carla.AttachmentType.Rigid)
        weak_self = weakref.ref(self)
        self.actor_list.append(self.ss_cam)
        self.ss_cam.listen(lambda data: self.__process_sensor_data(data, weak_self))

        # Attach and listen to image sensor (Semantic Seg)
        self.ss_cam_seg = self.world.spawn_actor(self.ss_camera_bp_sg, self.ss_cam_transform, attach_to=self.vehicle, attachment_type=carla.AttachmentType.Rigid)
        self.actor_list.append(self.ss_cam_seg)
        self.ss_cam_seg.listen(lambda data: self.__process_sensor_data_Seg(data, weak_self))

        time.sleep(RESET_SLEEP_TIME)
...

My reset() Method therefore destroys all actors/sensors before resetting.
As suggested here I use
self.client.apply_batch([carla.command.DestroyActor(x) for x in self.actor_list])

however, I then get tons of warning-messages in the simulator like:
ERROR: Invalid session: no stream available with id 856

which (after accumulating for a while) kill the simulator, with a "segmentation fault (core dumped)"
Thus, I replaced the destroy method with
for actor in self.actor_list:
       actor.destroy()

which throws the error-message in the simulator:
terminate called after throwing an instance of 'std::runtime_error'
  what():  trying to operate on a destroyed actor; an actor's function was called, but the actor is already destroyed.
Aborted (core dumped)

As proposed here  I also use weakrefs for my sensors as one can see in my code snippet.
Both ways of destroying actors/sensors do not work for me. The first way runs for a while but will crash at some point and the second way crashes when destroying the actor for the first time.
What am I doing wrong here?

I have worked it out.
After you are done with an environment, run:
for actor in self.actor_list:
     actor.destroy()

Afterwards, the environment object is still alive.
When you then create a new environment, the old environment will get destroyed. This calls the __del__ method that (in my opinion) contains some code that calls actor.destroy() again.
Thus overwrite the method like so:
def __del__(self):
      print("__del__ called")

It worked for me, however my explanation might be wrong.


Performance drop when using hug vs native Falcon

I was wondering if we had tried to quantify the cost of using Hug.
To my surprise, there seems to be a 40% performance drop (23297 from 35911 request/second)  when using hug in a simple hello world scenario.
Is there a plan to profile and explain the performance drop?
Scenarios
Falcon
import falcon

class ThingsResource(object):
    def on_get(self, req, resp):
        resp.status = falcon.HTTP_200  # This is the default status
        resp.body = 'Hello World !'

app = falcon.API()
things = ThingsResource()
app.add_route('/', things)

Hug
import hug

@hug.get('/')
def hello():
    return "hello world"

Server
Served with gunicornand using the meinheldworker
gunicorn <falcon_test:app | hello:__hug_wsgi__> --workers=2 --worker-class=meinheld.gmeinheld.MeinheldWorker
Benchmark
Running wrkwith the following options:
wrk -t10 -c200 -d30s http://<IP_ADDRESS>:8000/
Results
Falcon
Running 30s test @ http://<IP_ADDRESS>:8000/
  10 threads and 200 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     5.56ms  820.43us  30.78ms   93.29%
    Req/Sec     3.61k   293.58     7.09k    92.01%
  1081057 requests in 30.10s, 189.70MB read
  Socket errors: connect 0, read 56, write 0, timeout 0
Requests/sec:  35911.67
Transfer/sec:      6.30MB

Hug
Running 30s test @ http://<IP_ADDRESS>:8000/
  10 threads and 200 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     8.58ms  842.25us  64.73ms   96.27%
    Req/Sec     2.34k    90.01     3.78k    89.50%
  699435 requests in 30.02s, 112.73MB read
  Socket errors: connect 0, read 61, write 0, timeout 0
Requests/sec:  23297.93
Transfer/sec:      3.75MB

I'm pretty impressed by the speed gain meinheld provides.
My test results:
Falcon
Running 30s test @ http://<IP_ADDRESS>:8000/
  10 threads and 200 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     7.28ms    4.06ms  87.17ms   95.86%
    Req/Sec     2.88k   599.14     3.73k    77.53%
  860060 requests in 30.04s, 150.92MB read
  Socket errors: connect 0, read 54, write 0, timeout 0
Requests/sec:  28631.16
Transfer/sec:      5.02MB

Hug
Running 30s test @ http://<IP_ADDRESS>:8000/
  10 threads and 200 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     9.61ms    3.76ms  97.02ms   96.99%
    Req/Sec     2.14k   290.98     2.37k    92.83%
  640232 requests in 30.03s, 103.19MB read
  Socket errors: connect 0, read 61, write 0, timeout 0
Requests/sec:  21322.90
Transfer/sec:      3.44MB

@rodcloutier, there has been some work here: http://pycnic.nullism.com/images/pycnic-bench.png and I'm working to create benchmarks to the repo and to the test suite to ensure performance regressions don't happen. However, I can say very confidently that 1:1 hug performs no where near 40% slower then Falcon, but instead around 5-10% at worst case. As one of many examples of where your test is not a 1:1, your using costly json output for hug but simple no processing text output for falcon. This probably alone explains the benchmarking difference.

@rodcloutier I've added benchmarks here: https://github.com/timothycrosley/hug/tree/develop/benchmarks/http that provide a 1:1 comparison feel free to update them to add additional frameworks or use cases, currently the tests show a very small and expected performance difference then writing Falcon by hand.


Allow mcycle to be shared between harts

The commentary says this is allowed, but the normative text doesn't.  I don't like this behavior, but the manual should at least be self-consistent.

Spell check is your friend, Mr. mcountintihib


Fix: mosaic-onnx sample for ONNX InferenceService

Closes #1256
What this PR does / why we need it:
This PR fixes mosaic-onnx.ipynb to run seamlessly from the procedure in README.md.
Also, I fixed the bugs on mosaic-onnx.ipynb.
The main fixed points are as follows.

Use INGRESS_HOST and INGRESS_PORT instead of CLUSTER_IP that make us easy to work from the procedures in README.md.
Change model name to default not to raise 500 error.

c.f. https://github.com/microsoft/onnxruntime/issues/2442

Change arguments of image.resize which does not match model's input shape.

/lgtm
/approve


rename requirements.lowest.txt

dependency management tools always try to modify requirements.lowest.txt
this file should not be altered.
unfortunately there is no way to exclude this file from dependabot and others.
but if the file had another name, then it might work ...

Solved the same here: https://github.com/CycloneDX/cyclonedx-python/issues/512
Solution: rename to deps.lowest.r  -- the file extension seamed to be an issue, which cased analysis in the first place.


Careful which region you choose

In step 4 of section "Deploy Blockchain Workbench", the screenshot shows region "East US" as an example, and it is a good choice because currently Azure Blockchain Data Manager is only available in East US and West Europe. I went through the whole Azure Blockchain Workbench setup in West US 2, only to then see that Blockchain Data Manager was disabled because of my region. It would be nice to note that in this document.

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

ID: 3eb2e7c5-e153-4067-ae8c-45eb64add7da
Version Independent ID: 3c87633d-44b7-525d-0e29-75f2d4bcbf2b
Content: Deploy Azure Blockchain Workbench Preview - Azure Blockchain
Content Source: articles/blockchain/workbench/deploy.md
Service: azure-blockchain
GitHub Login: @PatAltimore
Microsoft Alias: patricka

@HarlowBurgess Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Hi @HarlowBurgess,
Thanks for the feedback. Good point. I'll add a backlog item to add the information to the article.
Thanks,
Pat
#in-progress

I've added the feedback to the article.
Thanks,
Pat
#please-close


[Docs] Use the regular path for the import

Fix https://github.com/callemall/material-ui/issues/2803.

Thanks :+1:


PySAML vulnerable to signature stripping by default

The the default value for WantAssertionsSigned is true in metadata, the default value for want_assertions_signed is false. While the default value of false is documented, the discrepancy gives a false sense of security.
Additionally and most importantly, false is an exceptionally terrible default. Allowing unsigned assertions is the type of issue that costs companies real money when found in a bug bounty and is much more costly if found by a real attacker.

There is also "authn_requests_signed", "logout_requests_signed" and maybe others?!. What about them? What's the default and are they documented?

https://github.com/rohe/pysaml2/blob/master/doc/howto/config.rst#logout-requests-signed
→ default: False
https://github.com/rohe/pysaml2/blob/master/doc/howto/config.rst#authn-requests-signed
→ default: True

The the default value for WantAssertionsSigned is true in metadata, the default value for want_assertions_signed is false. While the default value of false is documented, the discrepancy gives a false sense of security.

The default value of WantAssertionsSigned in metadata is False. That is if the optional element WantAssertsionsSigned is not included in the metadata, then the IdP should assume it is False ( https://docs.oasis-open.org/security/saml/v2.0/saml-metadata-2.0-os.pdf  line 743 )
want_assertions_signed controls the WantAssertionsSigned element in the SP metadata and its default is False, as per the the spec. Now in metadata.py the default value is True which will create SP metadata that have WantAssertionsSigned=True .
While I agree there is a discrepancy and needs to be cleared out in documentation, why would you say that this causes a false sense of security ? If something it forces WantAssertionsSigned=True which will make a conforming IdP to sign the Assertion in the SAML response it will return to this SP.
Regarding the default, now, assertions can be unsigned, as long as the SAML request is signed. See the current version of the spec

For the purposes of the profile, either the  or the  element(s) in the
 MUST be signed, if the HTTP POST binding is used, and MAY be signed if the HTTP-Artifact binding is used. If an  element is present and a CBC-mode algorithm is used, then the SHOULD be signed to ensure the ciphertext is integrity protected (see section 6.2 of [SAMLCore]).

So, if we are to do something, it would be probably better to change sign_responses to True by default instead, at https://github.com/rohe/pysaml2/blob/master/src/saml2/server.py . On the other hand, if we take pysaml2 for what it is ( a SAML2 implementation in python and not a standalone SAML2 Service Provider implementation that conforms to a profile), I don't know how much sense it makes to enforce defaults in code.

I stand corrected. want_assertions_signed is also used as a flag for whether unsigned assertions are allowed to be consumed by the SP.
Furthermore, there is the undocumented want_response_signed parameter ( https://github.com/rohe/pysaml2/blob/master/src/saml2/config.py#L66 ) which is alse False by default and controls whether an SP will accept unsigned SAML responses.
The current best practice is to sign SAML responses and not the assertions so I suggest we make this explicit in the documentation and include the want_response_signed . Regarding changing this default value, I am pro setting this to true in all examples and mark it as important in the docs, but I am not entirely sure on what we should do with the default value in pysaml2 src

Thank you for your response, @jkakavas!

used as a flag for whether unsigned assertions are allowed to be consumed by the SP.

This is part of what I meant to convey with my original ticket, thank you for adding & clarifying that!
I do want to push for changing the default value to True in code though, for a number of reasons.

For users already using pysaml2, the changes to documentation might easily be missed and the issue will persist for them. Users using pysaml==4.4.x might not even notice that this change is made.
If the documentation marks it important and the default value is True, I think that's a strong sign that the default value in code should be true.
I think it's correct to say that "A value of False means that users authenticating through SAML can lie about who they are or what privileges they have". For this reason, I do not believe False is a correct default value.

After some additional thought, I fully agree with @terite that this should be set by default. I will add a PR to address this asap


Catch iso639.language.LanguageNotFoundError

The iso639 module occasionally doesn't recognizing ISO language codes. Updated the code to catch this error and display a warning.

[X] Updated Unreleased in CHANGELOG.md to reflect the changes in code or data.

Fixing checks rn.

Should be good now


Add search capability for tickets

Instead of scrolling what could be a potential large list of tickets.  We want to be able to search a specific ticket and return that to the UI.

This is mostly fixed however, we need to make sure the correct ticket is selected before viewing it's description.


Powershell scripts are not recognised as a 'Program' for 'Open with' screen. Only .cmd, and .bat are.

Windows 10, build 6.2.9200.0.
Try and associate a file type with a Powershell script.
Right click a file, pick 'Open With'.  .bat and .cmd files are included as 'Programs'. ps1 is not.

Close as stale issue. Feel free to continue discussion.

Close as stale issue. Feel free to continue discussion.

@iSazonov still occurs in Windows 10 build 210108. What should I do to get it fixed?

@iSazonov still occurs in Windows 10 build 210108. What should I do to get it fixed?

@mikemaccana I guess it is Windows security design. You will also not find VBS on this list. You could discuss this in a more appropriate security forum. (I'm not even sure if this can be configured.)

@mikemaccana I guess it is Windows security design. You will also not find VBS on this list. You could discuss this in a more appropriate security forum. (I'm not even sure if this can be configured.)


Namadillo: Side menu opens and closes when switching to small screen view

When switching from the desktop layout (which has the side menu always visible on the left) to the mobile layout (which has the side menu hidden and a burger button to open it), the menu opens and closes very quickly but it shouldn't open at all.

Won't fix for now. See comments in #913.


Roll to v0.8.5-SNAPSHOT

Coverage decreased (-0.01%) to 32.86% when pulling 20f27b49ffab069801ff7b0718b3576e719c725e on 0.8.5-SNAPSHOT into 5bd34bc8d3dbf71323f9cb3aa0f2d79e5b59eab6 on master.

@chrisbennight Does this snapshot include the large polygon optimization?
@panrufio, @djohnson729

@drew-bower @panrufio @djohnson729
The 0.8.4 Tag has it (so does this).  Need to add it to the main page, but here's our changelog:
http://ngageoint.github.io/geowave/changelog.html
The item is #285 - Exception on input format with large / complex multi-polygon


Fix typos model-viewer article

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2019/05/model-viewer-ar.md#L59 - '


Fix nightly publish script on typedoc-site

It's been failing for the past couple days, haven't looked at it closely, but I think ts-node might just need to be updated.

Yep you were right 🚀


help error

Traceback (most recent call last):
File "/home/dureasd/TikTok-ViewBot/test.py", line 199, in
obj = zefoy()
File "/home/dureasd/TikTok-ViewBot/test.py", line 22, in init
self.driver      = uc.Chrome()
File "/home/dureasd/.local/lib/python3.10/site-packages/undetected_chromedriver/init.py", line 466, in init
super(Chrome, self).init(
File "/home/dureasd/.local/lib/python3.10/site-packages/selenium/webdriver/chrome/webdriver.py", line 45, in init
super().init(
File "/home/dureasd/.local/lib/python3.10/site-packages/selenium/webdriver/chromium/webdriver.py", line 56, in init
super().init(
File "/home/dureasd/.local/lib/python3.10/site-packages/selenium/webdriver/remote/webdriver.py", line 206, in init
self.start_session(capabilities)
File "/home/dureasd/.local/lib/python3.10/site-packages/undetected_chromedriver/init.py", line 724, in start_session
super(selenium.webdriver.chrome.webdriver.WebDriver, self).start_session(
File "/home/dureasd/.local/lib/python3.10/site-packages/selenium/webdriver/remote/webdriver.py", line 290, in start_session
response = self.execute(Command.NEW_SESSION, caps)["value"]
File "/home/dureasd/.local/lib/python3.10/site-packages/selenium/webdriver/remote/webdriver.py", line 345, in execute
self.error_handler.check_response(response)
File "/home/dureasd/.local/lib/python3.10/site-packages/selenium/webdriver/remote/errorhandler.py", line 229, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: unknown error: cannot connect to chrome at <IP_ADDRESS>:38899
from chrome not reachable
Stacktrace:
#0 0x559d0bdede93
#1 0x559d0bae5b37
#2 0x559d0bad0a13
#3 0x559d0bb1f601
#4 0x559d0bb1616b
#5 0x559d0bb5f71c
#6 0x559d0bb52c53
#7 0x559d0bb23db3
#8 0x559d0bb2477e
#9 0x559d0bdb37cb
#10 0x559d0bdb77e5
#11 0x559d0bda10e1
#12 0x559d0bdb8372
#13 0x559d0bd851bf
#14 0x559d0bddc488
#15 0x559d0bddc683
#16 0x559d0bded044
#17 0x7f15ee2e2ac3

I have the same issues


fix(nfs): handle export mode and mount permissions

/retest

Cuhu
On Sun, Oct 16, 2022, 10:15 AM krestomatio-cibot @.***>
wrote:

Merged #134
https://github.com/krestomatio/ansible-collection-k8s/pull/134 into
master.
—
Reply to this email directly, view it on GitHub
https://github.com/krestomatio/ansible-collection-k8s/pull/134#event-7596586927,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL4XCMAW7LWYJ2BAEILGU3WDQS3TANCNFSM6AAAAAARGM7XKM
.
You are receiving this because you were mentioned.Message ID:
@.***
.com>


Parameter Server

Hi all,
How to read parameter server code ? Do you guys have documentary files ? what is the last generation of the parameter server ? I read the paper for generation 3rd and I found it interesting.

@mu li

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


[Draft] Return methods in a lexicographical order from ReflectUtils.getMethods()

Fixes #30276. Returning methods in lexicographical order will prevent non-deterministic schema inference.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

assign set of reviewers

How does it work for update compatibility if I add a new field to my POJO? I added Reuven because he knows the most about this.

Dataflow keeps track of the old and new schema, and will match up fields to
ensure that the encoding position remains the same even if the field orders
are different. This is the reason why schemas have an encoding position in
addition to a field number.
Sorting the fields is not needed for update, and in fact is not a solution
since Dataflow supports adding new fields to a schema via update (if we
relied on sorting the new field would end up in the middle of the list, and
would break update).
On Mon, Apr 29, 2024 at 7:53 AM Kenn Knowles @.***>
wrote:

How does it work for update compatibility if I add a new field to my POJO?
I added Reuven because he knows the most about this.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/30279#issuecomment-2082958359, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAYJVOTCZO2RRBFYYXKJZLY7ZNHNAVCNFSM6AAAAABDBXMF4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSHE2TQMZVHE
.
You are receiving this because your review was requested.Message ID:
@.***>

Excellent. I was worried this would cause update incompatibility just as you described. But it sounds like it is fine. 👍

stop reviewer notifications

After talking to @reuvenlax we concluded that this might not be the best way to solve this issue. We'll try to reproduce this problem first and then try to address the root cause. I'll close this PR for now.


Fix Monolingual version for OS X Yosemite

(latest version for yosemite is 1.6.7, see https://ingmarstein.github.io/Monolingual/)

Thank you for the contribution. It needed some fixes, so they were made in 213f482e74ea05690e625c6a6d3bae07cb97ff6e. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


feat(icons-list): Adding a page displaying all the available icons - OEL-1684

Cool!! But can we align those icons..? It looks a bit messy right now :)

@planctus done, thx


Polymorphic queries currently not supported

public interface Permission { }

@Entity
public class UserPermission implements Permission { }

@Entity
public class UserGroupPermission implements Permission { }

"SELECT p FROM Permission p"

Hibernate generates two separate SQL queries for the above JPQL query. We currently do not handle this case in any way. Problems might arise wherever we deal with SQL directly:

Set Operations (UNION etc)
CTEs
Returning Support
Special cases of limit

Maybe we could also support polymorphic subqueries.
Tasks

[ ] Implement query per subtype strategy
[ ] Implement UNION strategy
[ ] Documentation

Just querying them like you showed might not be a big problem, but how do you expect to be able to filter properties? Implicit downcasting was removed in 1.2.0-Alpha2 and I am not really planning to revive it.
If you want to allow doing further operations like filtering, grouping etc. on a permission we have the following options

Construct a metamodel for the interfaces which contains all the commonalities of the subtypes implementing it. Then we can use that metamodel for building the query
Introduce implicit downcasting just for JoinNodes involving interface types

Apart from that, we have to decide how we implement this behind the scenes. Queries that don't select relations with polymorphic types could actually be implemented by using a UNION operator, because in that case, the generated SQL select clauses for the subtypes will have matching types. When polymorphic types are involved the column count and types can vary, so for such queries we'd have to execute a query for every subtype.
Do we want to allow an interface just as query root or also as cross/entity join like "SELECT ... FROM Permission p, Permission p2"? I think the strategy which executes queries per subtype will break with such queries, so I'd say we either disallow cross/entity joins as a whole or only support the case when the query can be formulated as UNION query.
How about supporting the interface type in nested queries like subqueries, CTEs or as SET operand? This obviously can only work when we can transform the query into a UNION query.
If that simple query is all you want to do, I'd rather not support this feature at all.
I'd like to hear more about the use case and how you would expect blaze-persistence to behave or to support that use case.

@beikov has there been any movement on this functionality? Looks like it hasn't been touched in a long time.

Hi @jcputney, would you mind sharing your use case? I'm kind of unsure what this would be useful for in Blaze-Persistence since people can just use plain HQL for these simple things.


[Feature Request] Render Markdown and HTML tags

Hello @trey-wallis !!!!
Would be amazing if we could render **Bold** *Italic* ==Highlights== and HTML tags like <u>Underline</u> in Reading and Live Preview Modes :)
This way We could make our tables even more beautiful :)

Thanks for reading this !!!!
Have a great day!!!!!

@FelipeRearden That is a great suggestion! I will look into adding this.

@FelipeRearden That is a great suggestion! I will look into adding this.

Thanks you very much @trey-wallis !!!!

@FelipeRearden I plan on adding this in the next release. :)

@FelipeRearden I plan on adding this in the next release. :)

Thank you very much @trey-wallis !!!!!!

@FelipeRearden Added in 3.4.0. Check it out! :)

@FelipeRearden Added in 3.4.0. Check it out! :)

This is absolutely fantastic @trey-wallis !!!!
Working perfect on version 3.4.1 !!!!!
Thank you and have a great day 🌞


Health check leaks file descriptors when DNS servers are unavailable

The problem
Styx 0.7 (possibly 1.n too) health check functionality leaks file descriptors when no domain name servers are available.
This is easy to reproduce in Vagrant or in Docker.

Configure origin(s) with health checks.
Watch the file descriptor count increase
Start styx
Modify name server IP address in /etc/resolv.conf to something non-existent.

Detailed description
When no name servers are available, InetAddress.getByName() appears to block for 20 seconds, blocking the Netty epoll event loop. In the meanwhile the health check monitor keeps creating more connections subsequent polls. They get queued up in Netty executor queue behind the slow name resolution.
"Health-Check-Monitor-app-Client-Worker-0-Thread@5018" prio=5 tid=0x13 nid=NA runnable
  java.lang.Thread.State: RUNNABLE
      at java.net.Inet4AddressImpl.lookupAllHostAddr(Inet4AddressImpl.java:-1)
      at java.net.InetAddress$2.lookupAllHostAddr(InetAddress.java:929)
      at java.net.InetAddress.getAddressesFromNameService(InetAddress.java:1324)
      at java.net.InetAddress.getAllByName0(InetAddress.java:1277)
      at java.net.InetAddress.getAllByName(InetAddress.java:1193)
      at java.net.InetAddress.getAllByName(InetAddress.java:1127)
      at java.net.InetAddress.getByName(InetAddress.java:1077)
      at io.netty.util.internal.SocketUtils$8.run(SocketUtils.java:146)
      at io.netty.util.internal.SocketUtils$8.run(SocketUtils.java:143)
      at java.security.AccessController.doPrivileged(AccessController.java:-1)
      at io.netty.util.internal.SocketUtils.addressByName(SocketUtils.java:143)
      at io.netty.resolver.DefaultNameResolver.doResolve(DefaultNameResolver.java:43)
      at io.netty.resolver.SimpleNameResolver.resolve(SimpleNameResolver.java:63)
      at io.netty.resolver.SimpleNameResolver.resolve(SimpleNameResolver.java:55)
      at io.netty.resolver.InetSocketAddressResolver.doResolve(InetSocketAddressResolver.java:57)
      at io.netty.resolver.InetSocketAddressResolver.doResolve(InetSocketAddressResolver.java:32)
      at io.netty.resolver.AbstractAddressResolver.resolve(AbstractAddressResolver.java:108)
      at io.netty.bootstrap.Bootstrap.doResolveAndConnect0(Bootstrap.java:208)
      at io.netty.bootstrap.Bootstrap.access$000(Bootstrap.java:49)
      at io.netty.bootstrap.Bootstrap$1.operationComplete(Bootstrap.java:188)
      at io.netty.bootstrap.Bootstrap$1.operationComplete(Bootstrap.java:174)
      at io.netty.util.concurrent.DefaultPromise.notifyListener0(DefaultPromise.java:507)
      at io.netty.util.concurrent.DefaultPromise.notifyListenersNow(DefaultPromise.java:481)
      at io.netty.util.concurrent.DefaultPromise.notifyListeners(DefaultPromise.java:420)
      at io.netty.util.concurrent.DefaultPromise.trySuccess(DefaultPromise.java:104)
      at io.netty.channel.DefaultChannelPromise.trySuccess(DefaultChannelPromise.java:82)
      at io.netty.channel.AbstractChannel$AbstractUnsafe.safeSetSuccess(AbstractChannel.java:915)
      at io.netty.channel.AbstractChannel$AbstractUnsafe.register0(AbstractChannel.java:512)
      at io.netty.channel.AbstractChannel$AbstractUnsafe.access$200(AbstractChannel.java:423)
      at io.netty.channel.AbstractChannel$AbstractUnsafe$1.run(AbstractChannel.java:482)
      at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:163)
      at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:403)
      at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:309)
      at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:858)
      at java.lang.Thread.run(Thread.java:748)

Acceptance criteria
File descriptor count should remain flat.

Thank you @mikkokar for creating this issue.
A couple of other observations:

It appeared to leak file handles only when there are more than one origins for the application.
Shorten DNS timeout and retry attempts helped alleviate the issue.
I had no luck reproducing it on MacOS, not sure if this is OS specific as well.

Thanks,
Xiuwen

Revisit this issue after Netty upgrade: PR #484.

Closing all issues over 3 years old. New issues can be created if problems are still occurring.


Add youtube-dlc app manifest

Not adding to https://github.com/ScoopInstaller/Main because I don't consider it to be "a reasonably well-known and widely used developer tool".
Youtube-dlc is at least as of now more actively developed than youtube-dl (over 300 commits ahead) and seems to have fixes for extractors that are buggy in the original project.

https://github.com/lukesampson/scoop-extras/pull/4756#issuecomment-691927490


The documentation for hipMemcpyToArray / hipMemcpy2DToArray is wrong

The documentation for hipMemcpyToArray / hipMemcpy2DToArray is seemingly copied from immediately above, and is wrong.
Note that it references dpitch instead of wOffset and hOffset.
https://github.com/ROCm-Developer-Tools/HIP/blob/0b257068fad6c91332a1ee931ee67b62ea201e48/include/hip/hcc_detail/hip_runtime_api.h#L2054-L2090

I believe -Wdocumentation should warn for these issues.


Datum not shown in utxo

I send lovelace to an address with datum like:
  tx.sendLovelace(
    {
      address: 'addr_test1vq8ncnj4xszdpy2ew0fzh8mudkwywqyz4fv8fkqh3jn20lsfkxtvt',
      datum: {
        value: {
          alternative: 0,
          fields: [49]
        },
        inline: true
      },
    },
    '1111111'
  );

The datum shows in the utxo as expected.
But when there are multiple objects in the fields like fields: [49,42] or  fields: [49,'ABC'] the transaction is sent but no datum in the utxo. Also, the inline: true doesn't seem to make any difference.

Hi @adam-young-dev, can you please give the TX Hash of one of the transactions you submitted?

Just try it again. Same code. But it works now! This transaction/datum.
This is very likely on my end. Sorry.


Default policy

Add default policy feature that applies to all messages sent using slack_message_via_policy() but when policy is None. Policy instance should be created while running migrations.

Instead of messaging via policy, it could be applied to slack_message() to apply global simple messaging policy. For policy, #63 might fit better.


[Qt] Incorrect error message with Start Mixing and Passphrase entry

This is an easy to duplicate issue (on the current develop branch - have not checked other versions) resulting in a misleading error message that could lead someone to think their wallet was locked when it actually is not.

Click the Start Mixing button while wallet is locked
Uncheck the Only for mixing via PrivateSend checkbox

Enter correct passphrase and click OK
The wallet will report an incorrect error message as shown:

The wallet is now unlocked (contrary to the error message). Also, mixing is not started.
I think the issue is due to the requestUnlock parameter here being decoupled from the actual state of the checkbox (the fForMixingOnly parameter is always true, not based on the checkbox state). This then results in fMixingUnlockFailed to be true (if the user supplied the correct password) because the new status of the wallet is Unlocked, not UnlockedForMixingOnly -
reference.

Should be fixed by #1941 - no more such option on mixing, no issues :D
Speaking seriously though, letting users to unlock wallet for sending when starting mixing was probably not a good idea in the first place. With #1941 wallet will be unlocked for mixing only when user hits that button. Wallet still can be fully unlocked via menu (or rpc) for infinite amount of time or it will be unlocked temporary for sending via Send tab (and switch back to mixing only mode right after tx is sent if it was mixing atm) but in both cases it will ask for the passphrase to do so. This seems to be the optimal flow, pls test and confirm if it's working as expected now.

I agree - this seems to be the safest and most intuitive way to do this.

Ok, closing this then :)


[Question]: How do I change the OpenAI api key?

What is your question?
I've searched discussions.  How do I change the OpenAI api key on windows11 device?

Run --setup again

ran setup and i'm getting this warning:  RuntimeWarning: Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work

thanks for your help, this has been difficult for me as i am not able to do any of this but trying to fail fast and pick up just enough to get this installed somehow.
So, i was able to add api keys for openai, claude, youtube and then run fabric --help, which shows me the options but also shows me this warning...which I'm sure I need to correct before moving forward:  RuntimeWarning: Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work
warn("Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work", RuntimeWarning)
instructions tell me to goto here: https://www.ffmpeg.org/download.html#build-windows
I can download sourcecode or download the windows exe files build, I got both just in case.  SHould i add one of these to the fabric folder on my computer?

I'm facing the fame issue currently on an M1 Pro w/ Sonoma 14.3.1

So, i was able to add api keys for openai, claude, youtube and then run fabric --help, which shows me the options but also shows me this warning...which I'm sure I need to correct before moving forward: RuntimeWarning: Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work warn("Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work", RuntimeWarning)

Are you running this in wsl? I just installed ffmpeg (and its dependencies) with sudo apt install ffmpeg and that resolved that error. I can't imagine what it actually would use ffmpeg for though, I would guess that regular usage of fabric wouldn't run into any potential issues that not having ffmpeg would cause.
If you're not using wsl, then you would just need to install ffmpeg however you want and ensure that the directory that the executable is in is on your path.

I downloaded ffmpeg to a folder and then added the location to the PATH in environmental values the restarted the terminal, (in my case also restarted Visual Studio) However I still cannot get the connection to OpenAI to work see my Issue Post.
Editing Path

Where I put ffmpeg


No access to GPU while building dockerfile

I need to build a package inside dockerfile that uses CUDA. Is there a way to pass --gpus params to docker build on sdkv2?

Thanks for reaching out.
Could you share more information like which SDK/service do you want to use?

I'm using azure machine learning to create endpoints that will use AKS (with AML extension) to process the workloads.
While creating the environment, I have to compile some custom pytorch modules and CUDA is required for compiling with the right micro operations.
Is there a way to have the GPU available while building the dockerfile under the AML environment creation?


You’d have to run a registry locally (could be docker’s registry, Trow, Harbor, etc.), and make sure to build and tag your image accordingly:

for registry (docker’s default registry implementation) => just omit the registry name ie. docker build -t myimage:0.1.0

in the absence of a prefix, docker run etc. will first try to find the image locally, then on Docker Hub

for other registries (eg. Trow, Harbor, etc.) => explicitly set that registry’s IP as a prefix ie. docker build -t localhost:5000/myimage:0.1.0

Thank you. That really helps!


Guild profiles module title is never printed

https://github.com/lautaroangelico/WebEngine/blob/97a91e500f4666a42e1cb727189be001e1fdf3c4/modules/profile/guild.php#L15

Fixed in 1.2.0


Adding templates, similar to kn eventing

/assign @houshengbo
templates like we have on kn-eventing

the template fails with:
.github/pull-request-template.md:21: MD014 Dollar signs used before commands without showing output
.github/pull-request-template.md:15: MD036 Emphasis used instead of a header

A detailed description of the rules is available at https://github.com/markdownlint/markdownlint/blob/master/docs/RULES.md

However, that is the template we use on other projects too ... :confused:  /cc @mattmoor


Add flare chain

Deployed the contracts using this PR: https://github.com/safe-global/safe-singleton-factory/pull/77/files
Thanks :)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
253 unchanged lines in 9 files lost coverage.
Overall coverage decreased (-6.04%) to 84.508%

Files with Coverage Reduction
New Missed Lines
%

gnosis/safe/safe.py
1
88.43%

gnosis/util/util.py
3
66.67%

gnosis/eth/multicall.py
9
88.35%

gnosis/eth/oracles/superfluid.py
9
55.0%

gnosis/eth/oracles/kyber.py
11
59.18%

gnosis/eth/oracles/cowswap.py
22
35.14%

gnosis/eth/oracles/uniswap_v3.py
28
56.34%

gnosis/eth/ethereum_client.py
39
81.38%

gnosis/eth/oracles/oracles.py
131
50.46%

Totals

Change from base Build<PHONE_NUMBER>:
-6.04%

Covered Lines:
3524

Relevant Lines:
4170

💛  - Coveralls


Testing the Simon Says light detector boards

The Simon Says light detector boards are ready to be implemented in software. There may be some inconsistencies in the readings, due to the current mount. make sure the sensors are in the correct position to detect the light from the light sources.

There is a micro switch on the back side of the stepper motor mount that will allow us to fine a home position. This will reduces staring errors, but may not be needed. Testing will tell if this is needed

The light detection circuit worked like a charm.


tailwind starter template fails dependencies installation

Is there an existing issue for this?

[X] I have searched the existing issues

Do you want this issue prioritized?

[ ] Yes, I have sponsored
[X] Not urgent

Current Behavior
The command npx create-solito-app@latest my-solito-app -t with-tailwind fails during yarn install with the following error message:
      throw new UsageError(`Unsupported package manager specification (${raw})`);
            ^

UsageError: Unsupported package manager specification<EMAIL_ADDRESS>    at parseSpec (C:\Users\AntoninoGargiulo\scoop\apps\nodejs-lts\20.15.1\node_modules\corepack\dist\lib\corepack.cjs:23258:13)
    at loadSpec (C:\Users\AntoninoGargiulo\scoop\apps\nodejs-lts\20.15.1\node_modules\corepack\dist\lib\corepack.cjs:23316:11)
    at async Engine.findProjectSpec (C:\Users\AntoninoGargiulo\scoop\apps\nodejs-lts\20.15.1\node_modules\corepack\dist\lib\corepack.cjs:23490:22)
    at async Engine.executePackageManagerRequest (C:\Users\AntoninoGargiulo\scoop\apps\nodejs-lts\20.15.1\node_modules\corepack\dist\lib\corepack.cjs:23542:24)
    at async Object.runMain (C:\Users\AntoninoGargiulo\scoop\apps\nodejs-lts\20.15.1\node_modules\corepack\dist\lib\corepack.cjs:24235:5) {
  clipanion: { type: 'usage' }
}

Node.js v20.15.1
[solito] error installing node_modules with yarn
 yarnpkg exited with non-zero code: 1

Expected Behavior
The scaffolding should complete without errors and the both yarn dev and yarn native should run without getting errors.
Steps To Reproduce
Run the command to scaffold the tailwind template project npx create-solito-app@latest my-solito-app -t with-tailwind
Versions
- Solito: installation failed
- Next.js: 14.2.5
- Expo: 51.0.23
- React Native: 0.72.4

Screenshots
No response
Reproduction
No response

I think this was fixed


Implemented a new optional file manager called Resource Manager

This new file manager allows us to no longer have any code that relies on Telerik libraries.
This does not replace the current file manager automatically but allows those who want to replace it. Doing so with this release means (providing no other 3rd party modules use Telerik) that the Telerik libraries and bindings can be completely removed.

Yes we are!


Binding attachment before a clusterInvoker invoke.

Binding attachments into invocation so that a rpc invoke have a same invocation
Fix a bug that the ForkingClusterInvoker don't clear attachment when a rpc invoke is over.

Codecov Report

Merging #2024 into 2.6.x will increase coverage by 0.03%.
The diff coverage is 76.47%.

@@             Coverage Diff             @@
##             2.6.x    #2024      +/-   ##
===========================================
+ Coverage     42.5%   42.53%   +0.03%
- Complexity    4703     4707       +4
===========================================
  Files          646      646
  Lines        30657    30661       +4
  Branches      5378     5379       +1
===========================================
+ Hits         13031    13042      +11
+ Misses       15609    15605       -4
+ Partials      2017     2014       -3

Impacted Files
Coverage Δ
Complexity Δ

...om/alibaba/dubbo/rpc/protocol/AbstractInvoker.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...bo/rpc/cluster/support/AbstractClusterInvoker.java
69.52% <33.33%> (-1.07%)
41 <0> (ø)

...bbo/rpc/cluster/support/ForkingClusterInvoker.java
85.71% <83.33%> (+0.42%)
7 <6> (ø)
:arrow_down:

.../com/alibaba/dubbo/monitor/dubbo/DubboMonitor.java
87.85% <0%> (-1.87%)
15% <0%> (ø)

...baba/dubbo/remoting/transport/mina/MinaClient.java
59.37% <0%> (+1.56%)
9% <0%> (+1%)
:arrow_up:

...rpc/protocol/dubbo/telnet/InvokeTelnetHandler.java
55.69% <0%> (+3.79%)
13% <0%> (+2%)
:arrow_up:

...va/com/alibaba/dubbo/config/DubboShutdownHook.java
73.91% <0%> (+30.43%)
6% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

fix problem:
https://github.com/apache/incubator-dubbo/issues/1978

I think you can attach a testcase for this

add some unit test to this case.

Fix:
https://github.com/apache/incubator-dubbo/issues/2023

@kimmking
fix code style
pls review

good job

LGTM, &READY-TO-MERGE&

@carryxyh Does this PR need to merge into master? I think so.

need another pr to master ?
@diecui1202

I will merge this PR. I think to wrap Cluster Invoker with Filter chain maybe another solution. Currently, the Filter chain only works with the underlying Invoker.


chore(integration): improve build failure output in tests

Display the actual contents of stderr when things fail. I preserved the commented-out debug stuff, as that can be useful for seeing what's happening when things don't fail.
Testing Strategy:

Ran all integration tests, they passed, and no output changed
Made a test fail and confirmed I saw the new error message and useful output :)

✅  Done!


Prometheus: Make server installation optional

This change allows to have the installation of a prometheus server
optional. This is useful for when you want to install an HA cluster of
alert managers that don't need a prometheus server.

I have tested this on my installation successfully. Please let me know if you'd like anything changed.
Thank you very much.

I don't like this. The whole point of this chat is to install Prometheus. I'd be good if the whole chart was split up into separate chart. I'm not sure what the futuren of this chart is anyways.
You may look here:
https://github.com/coreos/prometheus-operator/tree/master/helm
These charts are more actively maintained now.
/cc @mgoodness

Hi @unguiculus ,
So, you suggest moving over to that Coreos helm chart rather than using this one?
I have also seen there was an effort to migrate this chart to prometheus2 (or rather, create a new one as prom2 changes lots of things).
Are there any plans to incorporate that coreos chart into this default repo?
(My whole setup is based at the moment on this chart, that's why I was adding this PR)
Thanks.

/ok-to-test


Fixing bugs in MultiWorkUnit

Since the MultiWorkUnit class extends WorkUnit it is possible for a user to execute the following code:
WorkUnit workUnit = new MultiWorkUnit(Lists.newArrayList(new WorkUnit(), new WorkUnit());
workUnit.setProp("key1", "value1");

In the current implementation of MultiWorkUnit the setProp("key1", "value1") method will add the k, v pair to the internal properties object of the MultiWorkUnit, but it won't add the property to the WorkUnits in the internal list.
The current implementation / usage has a few issues:
1: The k, v pair gets dropped when a MultiWorkUnit gets serialized.
2: The JobLauncherUtils.flattenWorkUnits method will also drop all these properties.
The approach I took to remedy this is to override the setProp method to invoke super.setProp() and to invoke setProp() on each individual WorkUnit in the internal list. Also, to fix the serialization bug.
I find it a little confusing that now a MultiWorkUnit can hold state in addition to a list of WorkUnits so I am open to suggestions for a better approach.
I consider (and briefly talked to Yinan about this today), just overriding setProp and getProp with implementations that throw UnsupportedOperationException, but realized that would require overriding at least 5 methods. It would also be against Object Oriented Design, as MultiWorkUnit extends State and thus should be capable of storing and retrieving k, v pairs.
@liyinan926 and @ibuenros any comments or suggestions?

Travis build failed because line 408 workUnit.setProp(ConfigurationKeys.TASK_ID_KEY, multiTaskId); caused all wrapped WorkUnits to have the same task ID, given the changes in this PR. So you need to fix that. You can remove that line and use workUnit.getId() in line 412 instead of reading the value of ConfigurationKeys.TASK_ID_KEY.

I removed lines 405-411, I'm not sure if setting a property on a MultiWorkUnit actually does anything in the original implementation, since the MultiWorkUnit is serialized right in the next code block, the 'workUnit.setProp(ConfigurationKeys.TASK_ID_KEY, multiTaskId);' property will get dropped during serialization.

Ignore the most recent changes please

I messed up the commit history, will make a new PR.