FEATURE - Deploy to Azure

project deployment in Azure

Deployed on AWS: http://ec2-52-91-248-162.compute-1.amazonaws.com


Add a destructor to Envoy::Platform::Engine which calls terminate()

Add a destructor to Envoy::Platform::Engine which calls terminate()
Signed-off-by: Ryan Hamilton<EMAIL_ADDRESS>

👍 as long as there aren't any threading concerns here.

Thanks. Looking at how we use this, there should be no threading concerns.


Ubuntu extension manager not working properly

VSCode Version: 1.3.0
OS Version: Ubuntu 16.04

I have extensions installed, but they do not show up. I can not install them either. here are some screen shots:

I am seeing this too on Arch using the latest stable release (1.3.0):
https://az764295.vo.msecnd.net/stable/1.3.0/VSCode-linux-x64-stable.zip

#8967 has a workaround for this: delete the $HOME/.cache/Code directory after closing all instances of Visual Studio Code (killall code might be necessary if they are running in the background).

@sebadoom -- your workaround worked for me (had to do killall code). Thanks.

I'm experiencing the same issue. However command code --install-extension ms-vscode.csharp works perfectly!

That workaround fixed it!
killall code
rm -rf ~/.cache/Code

In my case, also needs to remove ~/.vscode (Ubuntu 14.04LTS)

On Arch, none of the workarounds helped for me. Not a big problem since I'm not using any for the project I'm working on right now (there are none) but it would be nice to be able to use extensions in the future.

@GregoryLundberg did you run killall code from a terminal before removing ~/.cache/Code. I'm on Arch too and this worked.

#8859 (see last comment)


Version bump

See coredump-ch/spaceapi#70

Even with more shields :wink: looks good to me :+1:

Oops, the title should have been "Added shields and bumped version" :)


New provider integration for passbolt

Describe the solution you'd like
It would be nice if passbolt  could be used as an additional provider for external secrets.
What is the added value?
Secrets could be intecrated from passbolt.
Observations (Constraints, Context, etc):
Passbolt provides a go client libraty that can be used.
https://github.com/passbolt/go-passbolt

Using a webhook and  the go-passbolt-cli source looks really a simple (generic) way to go.
https://external-secrets.io/v0.8.1/provider/webhook/
The pain in the behind is now - making this setup.

GPG key
User
Password
BasicAuth User
BasicAuth Password
https (optional)

It would be much better if passbolt had this under the hood.

@egandro I create a passbolt feature request here: https://community.passbolt.com/t/as-a-sre-id-like-to-be-able-to-use-secrets-from-passbolt-in-the-external-secrets-operator-for-kubernetes/7065

@bakito willing to jump in, if I get help.
The webhook looks nice but it's will create yet another docker image with secrets inside :) so I hope external secrets can do a direct implementation.

Any progress here? :)

Any progress? :)

Not from my side


AADSTS501491: Invalid size of Code_Challenge parameter.

Hi All, what will be possible cause for this issue.
I could not find more on this error code.
Any size limit is there for oauth 2.0 code_challenge parameter.
Please assist.

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

ID: fef99ca3-72bb-2c48-a62b-9ff9c314d72b
Version Independent ID: e58fd3a1-0af8-6e16-3d78-3dc748992593
Content: Microsoft identity platform and OAuth Authorization Code Flow
Content Source: articles/active-directory/develop/v2-oauth2-auth-code-flow.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

Could you provide us with the documentation link you were referring to ?

@mayankprayas Can you please provide us with the URL of the Azure Doc you are having issues with? We look forward to your reply.

Hi @Adam-Smith-MSFT , @SumanthMarigowda-MSFT ,
I am using oauth 2.0 in my application.
Currently we are referring below link for implementation :-
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-auth-code-flow
Due to security we have added PKCE on current solution if code_challenge length varies we will get this error.
Let me know if you need any further information on this.

@mayankprayas I apologize for the delay in getting back to you. If you are still facing the issue, can you provide the following details to investigate this issue further?

Are you getting this error when you send the code_challenge parameter to the authorize endpoint?

What exactly do you mean by "if code_challenge length varies we will get this error.". Are you changing the length of code_challenge for every request?

@mayankprayas I apologize for the delay in getting back to you. If you are still facing the issue, can you provide the following details to investigate this issue further?

Are you getting this error when you send the code_challenge parameter to the authorize endpoint?
What exactly do you mean by "if code_challenge length varies we will get this error.". Are you changing the length of code_challenge for every request?

Hi @ManojReddy-MSFT  , I am trying to integrate MS Azure Login to our application. I am using Oauth 2.0 endpoint for same.
Currently my code_challenge length is 43 character, is it possible to provide this also as random length as code verifier which we randomly choose from 43 to 128 Characters.
Response to your question
(1) Yes, when code_challenge length is more than 43 character it throws this error. Is code_challenge length is fixed to 43 character ?
(2) Currently my code_challenge is 43 character always , is it possible to change it dynamically for every new login and further request to get data. (similar to code verifier)

@mayankprayas , I ran into the same error, for me it was this: the code_verifier that you initially generate can indeed be 43 to 128 characters; then you compute base64(sha256(code_verifier)) to get your code_challenge. This code_challenge will be either 43 characters if the base64 encoder did not pad, otherwise 44 characters if the base64 encoder produced multiple-of-4 padded output but the last character in that case will always be an equal sign and should be removed for this particular application (padding is needed when concatenating base64 output, but not needed for isolated base64 output). Either way you obtain a 43-character code_challenge.

@alexanderperlis Just for my understanding, if running a sha256 online (https://xorbin.com/tools/sha256-hash-calculator) the length of result is 64, example: "e2f9604a32d9273e121237cad68266e5aaf2df53c8fa48bb5cf481a66bcaf193".
and base64 of that is properly longer, so where am i going wrong?

@pksorensen Your hexadecimal string of length 64 represents 32 bytes of data. The base64 encoding of that data has length 43.

Thanks @alexanderperlis , i discovered the same thing that doing base64 on the digest and not the hexadecimal string gave length 43 - thanks.

Did someone get fix ?

@RoyRK07 See Jun25 response above for solution.

@RoyRK07 See Jun25 response above for solution.

thanks


extensions: inotify: throw when used on darwin architecture

Inotify requirements: https://www.php.net/manual/en/inotify.requirements.php
Upstream PR: https://github.com/NixOS/nixpkgs/pull/242015
Question: Is this PR needed once the upstream PR will be merged? I don't think this PR will be needed once the upstream PR will be merged.

This is not needed anymore, it has been fixed upstream.


Add abfs scheme to hadoop support load spec list

Not sure it's enough but let's try

We need abfs schema at JobHelper. I will cherry-pick it at carrot-branch.


Test delegations with rosetta-cli

[x] run tests locally
[ ] add support for delegation transactions to rosetta-cli. See https://github.com/coinbase/rosetta-sdk-go/pull/457
[x] add tests to the CI using the rosetta-go-sdk fork until changes are merged upstream. See #12373.
[ ] Use new version of rosetta-cli after changes are merged upstream.

@Sventimir this is an issue in review. Does that mean that you believe that #12373 is sufficient to close this issue? Do we really need to wait for upstream? What is the ETA on this task?

@robinbb There's also PR ro rosetta-sdk-go, which the PR you mentioned depends on. I think we don't have to wait for Coinbase to integrate our changes, though, as we can build rosetta-cli with updated SDK for our internal testing. Actually this is what #12373 does for the moment. So I think as soon as CI issues that block merging of #12373 are fixed, we can merge and close this issue. No need to wait for the upstream indeed.

I extracted the update our tests when our changes are accepted upstream to a separate issue: #12509.


Add Item to List returning 400: Bad Request

Line 302, Default.aspx.cs file.
There is a extra closing bracket that is making the JSON body malformed, producing a 400 Bad Request response when trying to Add an Item to a List.
Fixed with just removing it like this:
 //Execute a REST request to add an item to the list.
string itemPostBody = "{'__metadata':{'type':'" + entityTypeName + "'},
                                      'Title':'" + newItemName + "'}";
Byte[] itemPostData = System.Text.Encoding.ASCII.GetBytes(itemPostBody);

Thanks. This is now fixed.


initialState sorting not working ? [v8 React Adapter]

Describe the bug
Passing initialState: { sorting: [{ id: string, desc: boolean }] } doesn't sort the column on render.
Your minimal, reproducible example
https://codesandbox.io/s/awesome-leaf-y8gx46?file=/src/main.tsx:2192-2204
Steps to reproduce

Set initialState: { sorting: [{ id: "lastName", desc: true }] } in useReactTable
Reload the page
See the column lastName not being sorted

Expected behavior
Column lastName should be sorted
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
OS: Win10 (WSL)
Browser: Chrome
react-table version
v8.1.3
TypeScript version
No response
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

You have two states with conflicting values, I'd suggest not using initialState and instead put your sorting in your useState statement at line #19.
https://codesandbox.io/s/eloquent-swirles-i633mo?file=/src/main.tsx

Your problem is that you have a state object that overwrites the initialState because in line 19, you tell useState that the initial sorting should be [] or "no sorting at all".
I see two options:

Remove the custom state and onSortChange handler and only use the initialState (probably easier if you don't want to do anything with the state) (fork)
Remove the initialState and set the sorting in line 19 where you currently set your sorting to [] (better if you need more flexibility but more complex) (fork)

Thank you so much, I got it now !


Improve handling of undefined visitor data for Somaliland region

Describe the Bug
Description:
When viewing the visitor map, hovering over the Somaliland region displays "undefined: X visitors" which appears to be a raw JavaScript undefined value being rendered. While Somaliland's status as an unrecognized state may explain the lack of data, the current display looks like a technical error rather than an intentional handling of this edge case.
Suggested Solution:
Replace the "undefined" text with a more user-friendly message such as:

"Unknown region: X visitors"
"Unrecognized region: X visitors"
"No data available: X visitors"

This would make it clearer that the lack of data is handled intentionally rather than showing what appears to be a JavaScript error to end users.

Database
PostgreSQL
Relevant log output
No response
Which Umami version are you using? (if relevant)
v2.13.2
Which browser are you using? (if relevant)
Chrome
How are you deploying your application? (if relevant)
Docker

That makes sense. A PR would be welcome.

We should handle this with i18n in mind. We could either add new translations for all languages, or simply reuse the existing label.unknown from the locale JSON. With label.unknown it would be simpler since we wouldn't need to create new translations.
Example image:

PR made 3060

Fixed in v2.14.0.


Add solution for Challenge 2023 02 Feb

Hi, @MaldoAlberto.
As far as I can see it's solution is already under my username folder

and in general I was trying to follow the recommendation below

Also I left the original notebook intact and I have created a new one and I have updated PR with some comments and references. Thank you for your feedback 🤓

Thank you so much for your quickly reply @TerraVenil ! Looks good, congratulations on solve those task, I hope you like this kind of problem and continue to solve this monthly challenges or the mentorship program :grinning:


spotify and chrome don't seem to show up in the plugin

playing any kind of content in both spotify (flatpak) and google chrome (flatpak) doesn't show up in this plugin, the plugin show no available players

Hm both Spotify and Chrome show up correctly in the media controls for me. Please make sure you run the flatpaks through game mode (they should show up only if they are running or in the background) and are running on the latest decky version (v2.0.5-pre13).
If this doesn't work, can you try running printenv | grep DBUS_SESSION_BUS_ADDRESS in your console and verify that its set to unix:path=/run/user/1000/bus?

yes they're running through game mode, should i do the print thing in game mode?

ideally if you can yes (you can use PuTTY to ssh in and execute the command there

(deck@steamdeck ~)$ printenv | grep DBUS_SESSION_BUS_ADDRESS
DBUS_SESSION_BUS_ADDRESS=unix:path=/run/user/1000/bus

While having spotify open in game mode, running the following command:
dbus-send --print-reply --dest=org.freedesktop.DBus /org/freedesktop/DBus org.freedesktop.DBus.ListNames
should show org.mpris.MediaPlayer2.spotify.. is that the case?

Also in the case that you are running on the preview steam deck branch, install version 1.0.2 of MusicControl from the decky plugin store and reboot to see if this has any effect

Also in the case that you are running on the preview steam deck branch, install version 1.0.2 of MusicControl from the decky plugin store and reboot to see if this has any effect

i am not on the preview branch, i'm on the default stable branch, and i'm already running version 1.0.2, should i be running a different version?

While having spotify open in game mode, running the following command: dbus-send --print-reply --dest=org.freedesktop.DBus /org/freedesktop/DBus org.freedesktop.DBus.ListNames should show org.mpris.MediaPlayer2.spotify.. is that the case?

(1)(deck@steamdeck ~)$ dbus-send --print-reply --dest=org.freedesktop.DBus /org/freedesktop/DBus org.freedesktop.DBus.ListNames
method return time=1660853998.897871 sender=org.freedesktop.DBus -> destination=:1.1139 serial=3 reply_serial=2
   array [
      string "org.freedesktop.DBus"
      string ":1.1093"
      string "org.kio5.kio_http_cache_cleaner"
      string "org.freedesktop.portal.IBus"
      string "org.freedesktop.ReserveDevice1.Audio0"
      string "org.freedesktop.portal.Desktop"
      string "org.freedesktop.ReserveDevice1.Audio1"
      string "com.feralinteractive.GameMode"
      string ":1.945"
      string "org.a11y.Bus"
      string "org.freedesktop.systemd1"
      string ":1.64"
      string ":1.1121"
      string "org.freedesktop.impl.portal.PermissionStore"
      string ":1.65"
      string ":1.1133"
      string ":1.1122"
      string ":1.11"
      string ":1.66"
      string ":1.1134"
      string "org.mpris.MediaPlayer2.spotify"
      string ":1.67"
      string ":1.1135"
      string ":1.0"
      string "org.freedesktop.Flatpak"
      string ":1.1136"
      string ":1.1"
      string "org.pulseaudio.Server"
      string ":1.1137"
      string ":1.1138"
      string ":1.1127"
      string ":1.3"
      string ":1.1139"
      string ":1.4"
      string "org.freedesktop.impl.portal.desktop.kde"
      string ":1.1091"
      string ":1.5"
      string "ca.desrt.dconf"
      string "org.freedesktop.portal.Documents"
      string ":1.1092"
      string "org.freedesktop.IBus"
      string ":1.149"
   ]

yep it's there

Issue was fixed by rebooting the steam deck, closing the issue, feel free to reopen if the problem reoccurs (but most likely will be a decky loader issue if that happens)

seems a reboot of the steam deck fixed it somehow


No ability to resize the scannable window

Hi,
I wanted to point out that the following variables do not appear to be used in the ZXingSurfaceView.cs file:
private const int MIN_FRAME_WIDTH = 240;
private const int MIN_FRAME_HEIGHT = 240;
private const int MAX_FRAME_WIDTH = 600;
private const int MAX_FRAME_HEIGHT = 400;
I have managed to resize the overlay in half inside of the function GetFramingRect(Canvas canvas) of ZXingOverlayView.cs by reducing the height value, but there is not ability (that I can see) to do this without updating the source. Even so, this reduces the clear window on the scanner, but has no impact on reducing where a barcode can be scanned. Upon holding the barcode outside the window, i can still pickup a result.
Next, I went into ZXingSurfaceView.cs and edited the function OnPreviewFrame(byte[] bytes, Android.Hardware.Camera camera).
I reduced the height variable, which is declared as:
var height = cameraParameters.PreviewSize.Height;
On my local device, the height being picked up was 960. If I reduce this a little bit, it still works, but if I divide by 2 (effectively halving the scannable size, which matches up with how much I reduced the overlay), the scanner won't work.
I created a loop which iteratively calls the barcodeReader.decode function (once it has already found a result) passing in a lower height each time with the same bytes until it fails. Sometimes this fails at around 700 height, sometimes gets down to 570. Any idea why this would be failing? It seems there is more support in the java version than the C# version.
Thank you

So if I understand correctly you're trying to make the scanner use a smaller part of the preview image to look for barcodes in.
To do this, you really can't just change the height variable, you would actually need to crop the image data that comes out of the 'OnPreviewFrame'.
This is something I've considered doing, however it may add some memory/cpu overhead to do so.  At this point, it's just not supported, sorry.


"error": "Missing query q parameter."

The url link gives me this message.

Hi @hshi420, thank you for reaching out 🙂

Could you please provide a full traceback or a screenshot of the error?
Could you please provide a code snippet of your search params, or a screenshot? By search params I meant this:

params = {
  "api_key": "...",
  "engine": "google",
  "q": "Coffee",
  "gl": "us",
  "hl": "en"
}

What API do you use? Google Search/Google Maps, etc?

I'll reopen this issue as we're not helped you yet 🙂

@hshi420 Got it! That's great that the issue was solved 🙂 Thank you for clarifying, I'll close this issue then. Feel free to open if you need to.

I have the same issue! Even when I copy the code from the playground into my script I get this error.
I wrote the following:
params = {
"api_key": personal_key,
"engine": "google_maps",
"type": "search",
"google_domain": "google.com",
"q": "Coffee",
"hl": "en",
"ll": "@52.092876,5.104480,15.1z"
}
search = GoogleSearch(params)
results = search.get_dict()

The error is then the following:   "error": "Missing query q parameter."
}

Hey, @myrthebuckens 🙂
A few obvious questions for the sake of clarity:

Did you pip install google-search-results?
Did you import it? Didn't see in your code snippet.
personal_key is a global variable? Have you tried passing your SerpApi API key directly to "api_key" under params? ("api_key": "1111adsad131")

Looking forward to your reply ✨


Remove unnecessary jvm opts when starting worker

What changes are proposed in this pull request?
fix https://github.com/Alluxio/alluxio/issues/14994
Why are the changes needed?
Remove redundent JVM options for worker.
See https://github.com/Alluxio/alluxio/blob/master/bin/launch-process#L77
When running launch-process script, it had initialized the config. It is unnecessary to pass the worker JVM opts in alluxio-start.sh
Does this PR introduce any user facing changes?
No

alluxio-bot, merge this please


Can not run

I run the example in your README and get this error: TypeError: cannot unpack non-iterable NoneType object

can you upload the exact files you used? i cant seem to reproduce the issue.
On Sat, 2 Mar 2024, 13:13 Nguyễn Bình Thành, @.***>
wrote:

I run the example in your README and get this error: TypeError: cannot
unpack non-iterable NoneType object
—
Reply to this email directly, view it on GitHub
https://github.com/crdoconnor/orji/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOJKNLZJDJMUSIFRU7MOR3YWHF6RAVCNFSM6AAAAABEDDAZUGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3DIOBQGMYDQNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

There might be something slightly different about the whitespace in your
files.
On Sat, 2 Mar 2024, 16:19 Colm O'Connor, @.***> wrote:

can you upload the exact files you used? i cant seem to reproduce the
issue.
On Sat, 2 Mar 2024, 13:13 Nguyễn Bình Thành, @.***>
wrote:

I run the example in your README and get this error: TypeError: cannot
unpack non-iterable NoneType object
—
Reply to this email directly, view it on GitHub
https://github.com/crdoconnor/orji/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOJKNLZJDJMUSIFRU7MOR3YWHF6RAVCNFSM6AAAAABEDDAZUGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3DIOBQGMYDQNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Here you are. I don't know what specific white space I use. I just copy your example to vscode on Windows 11, save the file, and run on WSL

Your "latexed" function may not convert orgmode format to latex.
Example: I write bold, italic in orgmode note, but Orji not auto convert it to latex format, even when I use "latexed" after "body"
Please fix this, your project have inspired me a lot. I use both orgmode and latex heavily in my daily job

I can't verify why exactly you were getting the above error for the default examples, but I've been getting it too for files without any actual notes like this:
This is the preface.\n\n

Your "latexed" function may not convert orgmode format to latex.

Yep, I'm afraid I did a very minimal version that worked for my purposes. I only cared about ampersands and URLs.
I've raised this as a separate issue here: https://github.com/crdoconnor/orji/issues/2

This issue should be fixed on the main branch of orgmunge. It's not yet released yet though.


Grammatical error

Under "Deployment" section... the statement "... Device Management Admin Console are now are Microsoft Endpoint Manager" is incorrect.

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

ID: 16a7f713-d59f-01cf-bd6b-d41e8889cfc8
Version Independent ID: dc8641da-26b5-1657-cc25-d5dd00ae84d7
Content: What's new in Windows 10, version 1909 - What's new in Windows
Content Source: windows/whats-new/whats-new-windows-10-version-1909.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@gene0486.  Device Management Admin Console is renamed to Microsoft Endpoint Manager
will you accept the above statement?

There's no need to change it very much. There is just a single word ("are") that is repeated and needs to be deleted.

Closing this as I am correcting  it in the private repo.


fixes country names

fixes  #644
I have changed the countries name.

Fixes https://github.com/github/covid19-dashboard/issues/643

@KamalSinghKhanna It looks like you tried to add yourself as an additional author, but it was not rendered properly.  You can test if the final webpage renders your name properly by running Fastpages/covid19-dashboard on your local machine. This requires installing Docker successfully and then running make server in the root of the git repository.
If you create a Pull Request before testing your code, the results are usually unexpected.  If there are errors or serious problems with the code change, the reviewer may also become frustrated that you did not bother to test your code before creating a Pull Request.

very sorry for the trouble @anomal.
I have installed docker successfully, But how to run make server in the root of the git repository ?


Add depth to directories search

I've added the depth option to use it with Symfony/Finder. I want to mutate the files that diff from master (master -> my_branch) and I need this feature because I can have multiple multiple files with same name in the directory tree. Ex:
#git diff --name-only origin/master
src/Data/GetData.php

My only diff is src/Data/GetData.php but my directory tree looks like:
src/Data/GetData.php
src/Data/Request/GetData.php
src/Estimate/Infrastructure/Persistence/DAL/GetData.php
In Symfony/Finder I can't add full path to file so I need to limit the depth of searching files in directories to 0 to mutate only src/Data/GetData.php.
So my config would like like
{
    "timeout": 10,
    "source": {
        "directories": [
            "src/Data"
        ]
    },
    "logs": {
        "text": "humbuglog.txt",
        "json": "humbuglog.json"
    }
}

and the command :
humbug --file=GetData.php
This way I can assure that only src/Data/GetData.php is analyzed

Thanks for the PR @holysleeper! Much appreciated.

I want to mutate the files that diff from master (master -> my_branch)

So maybe another way would be to be able to pass a list of files to mutate, so what you want is the intersection of the files found by the Humbug config and the files you're providing (intersection as some of those files from the diff may be out of the scope)?

Hello,
Thanks for the suggestion. I've created another PR with a new option that will filter the files.

Closed in favour of #264


Extend documentation

Hi Liz!
This PR adds some quick installation and development docs so that newcomers can quickly see how to install the module and anyone who wants to help with development can see how to install dependencies and run the test suite.  I've also bumped the copyright year to match the current year (assuming that this PR gets submitted ;-) ).  I also noticed that the name of the main program had been changed and I've updated the docs to reflect this.
As usual, if you want anything changed with this PR, please just let me know and I'll be more than happy to fix things and resubmit.
Say hello to Wendy from me!
Cheers,
Paul

@paultcochrane Thanks for the PR.  The effort is appreciated!
Note that for all my modules, the README.md is generated from the pod by App::Mi6.  The pod either lives in the source file if it is a smallish module, or in a separate file in the doc directory.  So any changes should be done there, and if you have App::Mi6 installed, a mi6 build will re-generate the README.md.
Also, I usually do not mention the installation boilerplate.  This maybe lazy of me, but the installation boiler plate has changed over the years, and very well may change in the future as well.  So I'd rather assume that any user of the module will find that boiler plate.  Or maybe a site such as raku.land could have a generic boilerplate.

So, if you could remove the boilerplate, and do the changes in the pod, that would be brill!
Also, perhaps better to do 2021, 2024 as nothing happened to that source in 2022, 2023.

Hi @lizmat. Sorry, I should have realised that the pod is automatically generated. We've been here before :-/  I'll remove the installation boilerplate, and fix up the other commits that's no problem :-)  From a user perspective, I've always found such information useful directly in the README, hence why I often propose such changes.
Just wondering: would perhaps a link to an appropriate documentation page be a better idea?  I mean, I've been quite familiar with Raku in the past and it took me a while to find the correct invocations again; a link to a definitive installation source might reduce the friction for newcomers and mean that you don't have to add installation boilerplate to all of your modules.  Just an idea!

I would welcome a link for installation instructions.  But I wouldn't know where to link to?

Also, perhaps better to do 2021, 2024 as nothing happened to that source in 2022, 2023.

Looking at the git history, there are commits from 2022 and 2023.  There were also releases in 2022.  Thus, I think it's fair to say that the copyright spans 2021 up to and including 2024 (assuming this patch gets merged).  I'll set the copyright years how you want, but I think the range is still accurate.  If you change your mind, let me know and I'll update the PR :-)

I've updated the commits and have force-pushed the feature branch.  Let me know what you think!

Ok, 2021 - 2024 it is then  :-)

Thanks!


Add ability to run evaluation against existing database

DESCRIPTION
Evaluator provides and option to export database containing issues without running evaluation. It's suggested to introduce an option to run evaluation against database rather than file with issues.
This option might be useful when different types of analysis can be performed after errors were gathered.

Fixed already


WIP - Transfer user controller to json

Closes #85
Thought I'd throw this up to track some progress.
Also, getting a weird error when I try to run:
mix test test/controllers/user_controller_test.exs:8

...

** (ArgumentError) Postgrex expected an integer in -2147483648..2147483647, got <<14, 127, 114, 184, 132, 121, 75, 108, 137, 99, 238, 177, 93, 36, 193, 201>>. Please make sure the value you are passing matches the definition in your table or in your query or convert the value accordingly.

Based on the error, I would assume I'm not passing an integer somewhere, but I can't tell where. I can investigate again later, but if you could run it once locally to let me know if you get the same error, that would be helpful.

Could you hop on this branch, run
MIX_ENV=test mix ecto.reset
and then
mix test test/controllers/user_controller_test.exs:8
and see if you get the same error as I do in the PR description?

Yeah, I'll take a look tonight

Ok, I just updated the plugs you wrote to simply be an Auth plug, where the main one dealing with the session is a module plug, but the rest are simply function plugs. Let me know what you think, and if you are ok with it, then I can delete the unused files.
I also commented out all the auto generated tests. Right now, all the tests that I wrote are passing, which means I might set up Travis next.

Also, I'm thinking we should get rid of the admin controllers and just collapse it into the normal ones.  Seems odd to have to hit a different endpoint if you are an admin even though admins and members will be basically be using the same app. I have an idea for a simple permission system that should work.

I don't believe the admin controllers constitute a different endpoint, they are just different controllers to handle the routes differently for admins. Check router.ex to see how they're being used. Let me know if I'm misunderstanding something.

I don't think it's possible to have one route lead to two different actions. Every route can only belong to one action. There might be a way we can hijack that, but I'm not sure we want to. So if there are specific actions that only admins can do e.g., destroy, then that can live in an admin controller that will run through admin auth plugs to verify. But outside of that, if the admin controller has the same action like index as the normal user controller, /users can only lead to one of those controllers.
That being said, we can also just attach those plugs to a specific action in the user controller like in user_controller.ex:6, and I think that is the simpler approach.

Not sure what you mean, that's literally how it was set up when it was serving views. It's more than one "route", but for a single endpoint (e.g. /users is accessible as a "user", or with extra features as an "admin").
We may be talking about the same thing here, but you seem to think we weren't already doing this? Dunno. Message me on Slack and we can talk in more detail.

Yeah, each action can only exist once per route, but that's what the router.ex does, it tells Phoenix which routes to serve depending on the plugs that resolve. So if the authorized user is an "admin" the router points /user/1/edit to the admin/user_controller.ex controller instead of the normal user_controller for that route, which provides edit, create, update, etc, that only admins can do. Authed non-admin users can hit the non-admin update to update themselves only.

I recommend rebasing on master, I added the Heroku app.json, which is needed for "review apps".

Ok, rebased


Monkey Patch Xformers use past_key_value but use_cache can't be True?

I'm confused with this script longchat/train/monkey_patch/llama_xformer_monkey_patch.py. I thought by not allowing use_cache parameter to True means past_key_value will not be used and not return. But, somehow past_key_value is still used but attn_weights is always None. Did you mean to set
assert not output_attentions, "xformer does not support output_attentions=True"

instead of
assert not use_cache, "xformer does not support use_cache=True"

@fahadh4ilyas Sorry for the confusion. The codes are intended to be used with use_cache=False, output_attentions=False. We will clean the use_cache logic in the next refactoring. Thanks for letting us know!

So, past_key_value is not supported in xformers?

Right, it is intended for Training purpose, where past_key_value is always not provided.

Okay, that makes sense. Which one is faster? Using flash attention or xformers?

In my benchmark during experiments, they are the same. Xformer supports V100, but flash does not.

What about for inference? Doesn't inference need past_key_value? Does this mean xformers can't be used for inference?

Yes, xformer and flash attention are not intended for inference in their most naive usage. We have more advanced system supporting this, and some models already been supported by this system in FastChat. LongChat is the next todo.

Okay right... Thank you for the answer...

@DachengLi1 Hi, so if one need support v100 training, it's better using xformers?
BTW, does it work if full params finetune with xformers or flashattn?


Experimental changes to allow relational-data-sink implementation to work with different database backends

Added test code to scope out the work needed to support the following variants:

SQLite
MariaDB
MySQL
Microsoft SQL (MSSQL)

Testing has been done on a MacBook Pro M3 system for the first three; the corresponding log file is attached for reference. The code for MSSQL remains to be tested, so consider it as a place holder for now.
sqlite.log
mariadb.log
mysql.log

Just pushed back the change for MS-SQL. The testing result is captured in
mssql.log


Fade to black

Ran black on the code base. The code is now correctly formatted.
(also fixes one SyntaxError on Python 3)

How about adding black --check --diff to the CI?
That would of course also require adding Python 3.6 or 3.7 to the CI, as Black needs 3.6+.

That's a good idea about adding the check, and also a good reminder about the required version. Due to the delays it will take to add that check (and its interactions with the rest of the suite), I'm going to defer that until later.


Add expand and shrink option for table

The expand button adds all columns to the display, it is "<>".
The shrink button removes all columns that were added by the expand button, it is "><".
The function of the shrink button means that with a table consisting of the columns A, B, and C and A and B being the current display,
after the expand button was pressed, A, B, and C are shown.
If you remove B now you are left with A and C.
If you now shrink the table back, you are left with A because the shrink button disables all columns that were enabled through
the expand button.
Design:

Behaviour:
default

expand (license added)

remove a column except license (eg Uncolored)

shrink (License removed, rest is the same)

Use glyph icons, then this can be merged.


RSSAgent should record which feed errored and keep going on the other ones

@dsander @knu Do you agree with the change that the RSSAgent, when given multiple feeds, should log errors for failed ones but keep going on the others?
Fixes #849

@cantino Agreed!


Add agent instance register and heartbeat report

Relate #183

@ascrutae You can do the UI work first. I and @pengys5 will review this again after we merge #215 .

For naming things, I still didn't like the heartbeat, prefer ping for record. @pengys5 thoughts?


[joy-ui][Accordion] The expand button acts as type="submit" by default

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Basic working code:
<form>
  <AccordionGroup>
    <Accordion>
      <AccordionSummary>Title</AccordionSummary>
      <AccordionDetails>Content</AccordionDetails>
    </Accordion>
  </AccordionGroup>
</form>

Steps:

Add a mui-joy accodion within a form tag
Click the expand button

Current behavior 😯
It works the same as using AccordionSummary's slot prop for button type='Submit'.
<form>
  <AccordionGroup>
    <Accordion>
      <AccordionSummary slotProps={{ button: { type: 'submit' } }}>
        Title
      </AccordionSummary>
      <AccordionDetails>Content</AccordionDetails>
    </Accordion>
  </AccordionGroup>
</form>

Expected behavior 🤔
It should work the same as using the slotprop of AccordionSummary for button type='button'.
<form>
  <AccordionGroup>
    <Accordion>
      <AccordionSummary slotProps={{ button: { type: 'button' } }}>
        Title
      </AccordionSummary>
      <AccordionDetails>Content</AccordionDetails>
    </Accordion>
  </AccordionGroup>
</form>

Context 🔦
I created a settings page for the user. Settings page information is contained within form elements, and accordion elements are used in the middle of the page to provide detailed guide information. When I used the accordion component, I thought the accordion component would not interact with the form elements. But it was.

  Output from `npx @mui/envinfo` goes here.

  System:
    OS: macOS 14.0
  Binaries:
    Node: 18.16.1 - ~/.nvm/versions/node/v18.16.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.16.1/bin/yarn
    npm: 9.5.1 - ~/.nvm/versions/node/v18.16.1/bin/npm
  Browsers:
    Chrome: 118.0.5993.88
    Edge: 114.0.1823.37
    Safari: 17.0
  npmPackages:
    @emotion/react: ^11.11.1 => 11.11.1
    @emotion/styled: ^11.11.0 => 11.11.0
    @mui/base:  5.0.0-beta.10
    @mui/core-downloads-tracker:  5.14.14
    @mui/icons-material: ^5.14.3 => 5.14.3
    @mui/joy: ^5.0.0-beta.0 => 5.0.0-beta.11
    @mui/material: ^5.14.4 => 5.14.4
    @mui/private-theming:  5.14.14
    @mui/styled-engine:  5.14.12
    @mui/system:  5.14.14
    @mui/types:  7.2.5
    @mui/utils:  5.14.14
    @types/react: 18.2.16 => 18.2.16
    react: 18.2.0 => 18.2.0
    react-dom: 18.2.0 => 18.2.0
    typescript: 5.1.6 => 5.1.6

Safari 17.0

@maakcode assing me this issue

@maakcode I have started working on this issue. Please assign this to me.

@maakcode i have come first and already started working

Same class of bug as #39262


Shale cannot deal with default XML namespaces

Shale as it stands cannot deal with default namespaces, e.g. <metanorma-collection xmlns="http://www.metanorma.org">
If I specify
xml do
      root "metanorma-collection"
      namespace "http://metanorma.org", "m"

I get <m:metanorma-collection>, and namespacing for elements mapping to strings, but not elements generated through supplied methods.
If I specify
xml do
      root "metanorma-collection"
      namespace "http://metanorma.org", ""

I get <: metanorma-collection>
If I specify
xml do
      root "metanorma-collection"
      namespace "http://metanorma.org", nil

The namespace is completely ignored.
This is screwing me over in my effort to adopt Shale in Metanorma and refactor it, which is awful enough already without me having to deal with this as well.

From @andrew2net https://github.com/metanorma/metanorma/issues/383#issuecomment-2120719515
Yes, it needs to monkey patch the add_namespace method
https://github.com/kgiszczak/shale/blob/8ec3be64b96d2d9bc77ddfb6638a6b3d3587a063/lib/shale/adapter/nokogiri/document.rb#L65-L67

The desired behaviour is,
xml do
      root "metanorma-collection"
      namespace "http://metanorma.org", nil

(a) gives me <metanorma-collection xmlns="http://metanorma.org">
(b) works in both from_xml and to_xml. I note with dismay that Shale's tests appear to test only the latter in their rspec. The former breaks: if I introduce namespaces manually in the root, I think it no longer finds child elements.
If we are to be good corporate citizens and feed these changes back to Shale, we should be doing so for the other two XML adapters supplied as well.

@HassanAkbar I am being forced into some very unpleasant workarounds because I cannot do this. @ronaldtse clearly has not communicated this, but this is not a nice-to-have, it is holding up work I am doing right now.

@opoudjis I'm looking into it.

@opoudjis From what I understand, for the following input
class Address < Shale::Mapper
  attribute :city, Shale::Type::String
  attribute :street, Shale::Type::String
  attribute :zip, Shale::Type::String

  xml do
    root "address"
    namespace "https://foobar.com", "foo"

    map_element "city", to: :city
    map_element "street", to: :street
    map_element "zip", to: :zip
  end
end

address = Address.from_xml(<<~DATA)
  <address>
    <city>London</city>
    <street>Oxford Street</street>
    <zip>E1 6AN</zip>
  </address>
DATA

address.to_xml

we need to generate the following output
<foo:address xmlns:foo="https://foobar.com">
  <foo:city>London</foo:city>
  <foo:street>Oxford Street</foo:street>
  <foo:zip>E1 6AN</foo:zip>
</foo:address>

I just want to confirm what the output should be in this case? When the prefix is nil for the namespace
class Address < Shale::Mapper
  attribute :city, Shale::Type::String
  attribute :street, Shale::Type::String
  attribute :zip, Shale::Type::String

  xml do
    root "address"
    namespace "https://foobar.com", nil

    map_element "city", to: :city
    map_element "street", to: :street
    map_element "zip", to: :zip
  end
end

address = Address.from_xml(<<~DATA)
  <address>
    <city>London</city>
    <street>Oxford Street</street>
    <zip>E1 6AN</zip>
  </address>
DATA

address.to_xml

@HassanAkbar I think you need to fix this method:
# gems/ruby-3.2.0/gems/shale-1.1.0/lib/shale/adapter/nokogiri/document.rb
...
        def add_namespace(prefix, namespace)
          @namespaces[prefix] = namespace if prefix && namespace
        end
...

It blocks namespaces with nil prefixes. The solution could be:
...
        def add_namespace(prefix, namespace)
          @namespaces[prefix] = namespace if namespace
        end
...

I've tried to monkey-patch this and it works for me, but it needs to be tested.

@andrew2net Thank you for pointing this out, I can change this for sure.
But I still don't know what to test after changing this.
What do we really want to achieve here ?

@HassanAkbar isn't the described desired behavior enough?

What we both want is:
```ruby
class Address < Shale::Mapper
  attribute :city, Shale::Type::String
  attribute :street, Shale::Type::String
  attribute :zip, Shale::Type::String

  xml do
    root "address"
    namespace "https://foobar.com", nil

    map_element "city", to: :city
    map_element "street", to: :street
    map_element "zip", to: :zip
  end
end

address = Address.from_xml(<<~DATA)
  <address xmlns="https://foobar.com">
    <city>London</city>
    <street>Oxford Street</street>
    <zip>E1 6AN</zip>
  </address>
DATA

address.to_xml
```

We don't want to switch all our code to `<foo:xxx>`.

And we do want to confirm that XML both reads in and outputs with this namespacing; the testing like I said is currently only in one direction.


Idea: Allow for assigning an MP3/OGG file to play on stream when a character/player is on stream

Allow the user to assign an audio file to a character or player for when said character / player shows up on stream. (Either as Player 1 or Player 2)
The Player audio would potentially override the character audio
Maybe avoid storing MP3s in the Assets repo to avoid copyright issues and leave it as an optional user feature though...

A side effect of the current asset pack implementation is that you can create an asset pack with MP3 files, so the feature technically already exists ^^


Microsoft Learn: sometimes a translated learning path is in progress even if it is completed

Describe the bug
Sometimes a translated learning path is in progress even if it is completed.
To Reproduce
Take Learn the business value of Microsoft Dynamics 365 and the Power Platform as an example:

the steps to take for an English speaker:

Navigate to the learning path: Learn the business value of Microsoft Dynamics 365 and the Power Platform
Complete all the modules of the learning path except the last one: Dynamics 365 AI - Insights-driven business applications
Complete an additional module: Microsoft runs on trust
Go to Profile
Switch the language setting to Simplified Chinese | Substitute en-us in the address bar to zh-cn

the original steps:

Switch the language setting to Simplified Chinese | Substitute en-us in the address bar to zh-cn
Navigate to the translated learning path: Learn the business value of Microsoft Dynamics 365 and the Power Platform
Complete all the modules of the learning path
Go to Profile

the common steps:

Check that the translated learning path is still listed under in progress
Click on the translated learning path to navigate to it
Check that the translated learning path is shown as completed
Check that the last module is not included in the translated learning path

Expected behavior

the translated learning path is not listed under in progress, or
the (untranslated, I guess) module is included in the translated learning path

Screenshots
N/A
Desktop

OS: Windows 10
Browser: Microsoft Edge
Version: latest

Additional context
The issue results from the difference in the modules:

for the English learning path, the last module is Dynamics 365 AI - Insights-driven business applications
for the Simplified Chinese learning path, the last module is Microsoft runs on trust

@DanteMustCode Thank you for submitting this issue. This has been submitted to our internal ticketing system.


re-enable disabled job after the worker api change

Merge after the latest build / deploy - revert a temp workaround for production sidekiq jobs

LGTM. I'll leave it to you to merge when the deploy is done.


add amount paid detection

detection of under/over paid

For now it is possible to accept underpaid invoices via underpaid_percentage setting

For now it is possible to accept underpaid invoices via underpaid_percentage setting

Finally done via 16f151ff3fa86a5ba737ceaf568e4fe8ce5c179d (:


Unable to access the created ITables after defineData

Description
When trying to access the tables after constructing them, no tables are defined:
import { defineBackend } from '@aws-amplify/backend';
import { auth } from './auth/resource.js';
import { data } from './data/resource.js';

const backend = defineBackend({
  auth,
  data,
});
console.log('tables', backend.data.resources.tables);
// tables {}

The error only occurred when using Amplify. When I tried with CDK directly, everything was okay Reference
Related threads:

https://github.com/aws-amplify/amplify-category-api/issues/2137
https://discord.com/channels/705853757799399426/1191670590319362078
https://discord.com/channels/705853757799399426/1190515212323926076/1190515212323926076

Environment information
Binaries:
    Node: 18.18.2 - ~/.nvm/versions/node/v18.18.2/bin/node
    Yarn: 1.22.21 - ~/.nvm/versions/node/v18.18.2/bin/yarn
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.2/bin/npm
    pnpm: 8.9.2 - ~/Library/pnpm/pnpm
  npmPackages:
    @aws-amplify/backend: ^0.7.0 => 0.7.0
    @aws-amplify/backend-cli: ^0.9.2 => 0.9.2
    aws-amplify: ^6.0.10 => 6.0.9
    aws-cdk: 2.110.1 => 2.110.1
    aws-cdk-lib: 2.110.1 => 2.110.1
    typescript: ^5.3.3 => 5.3.3

Hey @marnutux, We are looking into this and let you know if there is any workaround.

The table name is here: backend.data.resources.amplifyDynamoDbTables.<model>.resource._cfnProperties.tableName. However resource is private currently. I see no issue changing this to public or adding an accessor method though.

Closing this issue as we will continue tracking it at https://github.com/aws-amplify/amplify-category-api/issues/2137. Please subscribe to the linked issue for further updates.


with 4-width tabs set in settings, pressing tab inserts 2 spaces

I was able to reproduce this with a fresh bin, but, oddly, not with the editor preview on the settings page.

Remy, is this on someone's to do list? To answer your 2014 question: Even with no spaces anywhere in this bin, pressing tab still inserts a space.

It's not on anyone's todo. Currently, it's just me on jsbin, and I'm prioritising either the very interesting parts or sans-UI elements. I need to do a push to get contributors and I'd love to see you have some commits in the project given your long history with file issues (thank you) on the project.
Quickly trying, I can't replicate - it gives me two spaces both when I tab and shift+tab. I'm controlling my settings exclusively in the preferences now - I assume you are?


fixed method arguments in Mechanical

Codecov Report
Attention: Patch coverage is 86.88525% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 81.50%. Comparing base (85bd251) to head (1ee6df2).
Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4545   +/-   ##
=======================================
  Coverage   81.50%   81.50%
=======================================
  Files         110      110
  Lines       53731    53731
=======================================
+ Hits        43791    43794    +3
+ Misses       9940     9937    -3


When I run mvn install the test fails. It's most likely an x509 issue.

Question
SecurityTest fails during testing step during mvn installation.
"Running org.eclipse.leshan.integration.tests.SecurityTest"
Among the error phrases,
"org.eclipse.leshan.integration.tests.SecurityTest.registered_device_with_x509cert_to_server_with_x509cert_rootca_certificate_usage_taa_intca_given(Protocol, String, String)[1]"'
stands out.
I just read the README and ran mvn install normally from the root path of leshan-master.
Is there any way to fix this error?
This question was created using Google Translate.
Sorry if there is a problem with communication. I can not speak english well.

I build Leshan almost every day and it is also built by github or jenkins without issue. (all run are on Linux)
So this is maybe a issue with your environment.
Looking at https://github.com/eclipse-leshan/leshan/issues/1510, it seems you are using :

maven 3.6.0
java 17 (from Oracle)
Windows 10

I'm not able to test in this environment.
I understand you try to build master right ? (eventually share the exact commit id)

Is there any way to fix this error?

I don't know exactly what is the issue, but as a workaround you can skip the tests with :  mvn clean install -DskipTests
You tried to share the output of your build but :

you share image instead of text ... See why it's bad idea https://github.com/eclipse-leshan/leshan/issues/1510#issuecomment-1720864309 ...
the output is not complete, please share the full log as text not image.

@sbernard31
Thanks for your reply.
Sorry for attaching an image.
Can I know sbernard31’s development environment?
I would like to try changing to Linux.
Please tell us your development environment, such as Maven, Java, or Linux.
We will proceed with reference to this.

Can you tell me the development environment of sbernard31?

>> mvn --version
Apache Maven 3.6.3
Maven home: /usr/share/maven
Java version: 11.0.20, vendor: Debian, runtime: /usr/lib/jvm/java-11-openjdk-amd64
Default locale: fr_FR, platform encoding: UTF-8
OS name: "linux", version: "5.10.0-23-amd64", arch: "amd64", family: "unix"

I also just tested with : Java version: 17.0.7, vendor: Debian, runtime: /usr/lib/jvm/java-17-openjdk-amd64   and it works for me too.
But I"m on Debian 11 but I will probably upgrade to Debian 12 soon so it will change.
Anyway, I guess this should work on Windows 10  too, so It would have been great to (at least) understand this issue ?  :thinking:
Could you share full build output ?
And also if you could test : https://github.com/eclipse-leshan/leshan/issues/1461#issuecomment-1720873438, this could help ?


bug

======== Exception caught by widgets library =======================================================
The following assertion was thrown building TBExceptionRoute(state: _TBExceptionRouteState#e73c8):
'package:flutter/src/rendering/object.dart': Failed assertion: line 1609 pos 14: '_debugSubtreeRelayoutRootAlreadyMarkedNeedsLayout()': is not true.
Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
https://github.com/flutter/flutter/issues/new?template=2_bug.md
The relevant error-causing widget was:
TBExceptionRoute TBExceptionRoute:file:///Users/songwenshan/baidu/app/app_all/lib/main.dart:227:12
When the exception was thrown, this was the stack:
#2      RenderObject.markNeedsLayout (package:flutter/src/rendering/object.dart:1609:14)
#3      RenderBox.markNeedsLayout (package:flutter/src/rendering/box.dart:2320:11)
#4      RenderObject.adoptChild (package:flutter/src/rendering/object.dart:1334:5)
#5      ContainerRenderObjectMixin.insert (package:flutter/src/rendering/object.dart:3312:5)
#6      MultiChildRenderObjectElement.insertRenderObjectChild (package:flutter/src/widgets/framework.dart:6273:18)
#7      RenderObjectElement.attachRenderObject (package:flutter/src/widgets/framework.dart:5870:35)
#8      RenderObjectElement.mount (package:flutter/src/widgets/framework.dart:5544:5)
#9      SingleChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:6194:11)
...     Normal element mounting (1209 frames)
#1218   Element.inflateWidget (package:flutter/src/widgets/framework.dart:3673:14)
#1219   Element.updateChild (package:flutter/src/widgets/framework.dart:3425:18)
#1220   ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:4703:16)
#1221   StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:4840:11)
#1222   Element.rebuild (package:flutter/src/widgets/framework.dart:4355:5)
#1223   BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2620:33)
#1224   WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:882:21)
#1225   RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:319:5)
#1226   SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1143:15)
#1227   SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1080:9)
#1228   SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:996:5)
#1232   _invoke (dart:ui/hooks.dart:166:10)
#1233   PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:270:5)
#1234   _drawFrame (dart:ui/hooks.dart:129:31)
(elided 5 frames from class _AssertionError and dart:async)
Another exception was thrown: 'package:flutter/src/rendering/object.dart': Failed assertion: line 1609 pos 14: '_debugSubtreeRelayoutRootAlreadyMarkedNeedsLayout()': is not true.

Hi, please follow the template for filing bugs:
**Is there an existing issue for this?
Steps to reproduce
Expected results
Actual results
Code sample
Logs
Flutter Doctor output**

@247078143 When opening an issue, please try to provide all information indicated in the template so that we can address this issue properly (See bug issue template here)
Closing the issue. Once you update the issue with more information in the issue template, please write in comments and I will reopen this for further investigation. Thanks!


UI and Layer Updates

Implemented new vector base layer for plurality (50 states)
Updated to show "peers of" text when peers selected
Added links to peer institution names to pull up same map with their LAR count
Repaired Print view to print affiliates and peers when options selected.

Coverage remained the same at 82.61% when pulling 6388263dac4078de4a56fd66b5429b80dc23bd1d on sephcoster:chrome-changes into 20009ddfb97a6efe8ac8410e5a19715c2a17398d on cfpb:master.

Damn. That is a way to start the day. Nice work.

Coverage remained the same at 82.61% when pulling a2574dbefa49b0348f245ecbd2c752992e43cab2 on sephcoster:chrome-changes into 20009ddfb97a6efe8ac8410e5a19715c2a17398d on cfpb:master.

:+1:


Add playbooks for ci-ceph-redhat

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test

retest

retest

URSULA_TES_PR CI keeps failing with below error
00:50:31 fatal: [controller-1]: FAILED! => {"changed": false, "failed": true, "invocation": {"module_args": {"allow_unauthenticated": false, "autoremove": false, "cache_valid_time": 3600, "deb": null, "default_release": null, "dpkg_options": "force-confdef,force-confold", "force": false, "install_recommends": null, "name": "python-pip", "only_upgrade": false, "package": ["python-pip"], "purge": false, "state": "present", "update_cache": true, "upgrade": null}}, "msg": "No package matching 'python-pip' is available"}
00:50:32 fatal: [controller-0]: FAILED! => {"changed": false, "failed": true, "invocation": {"module_args": {"allow_unauthenticated": false, "autoremove": false, "cache_valid_time": 3600, "deb": null, "default_release": null, "dpkg_options": "force-confdef,force-confold", "force": false, "install_recommends": null, "name": "python-pip", "only_upgrade": false, "package": ["python-pip"], "purge": false, "state": "present", "update_cache": true, "upgrade": null}}, "msg": "No package matching 'python-pip' is available"}
00:50:32 fatal: [compute-0]: FAILED! => {"changed": false, "failed": true, "invocation": {"module_args": {"allow_unauthenticated": false, "autoremove": false, "cache_valid_time": 3600, "deb": null, "default_release": null, "dpkg_options": "force-confdef,force-confold", "force": false, "install_recommends": null, "name": "python-pip", "only_upgrade": false, "package": ["python-pip"], "purge": false, "state": "present", "update_cache": true, "upgrade": null}}, "msg": "No package matching 'python-pip' is available"}
00:50:32

retest

retest

retest

https://jenkins.ci.blueboxgrid.com/job/Ursula_Test_centos_PRs/446/ and https://jenkins.ci.blueboxgrid.com/job/Ursula_Test_PRs/868/ both are passed for checking.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist


Documentation updates related to the Provider Review

WIP document targeting to update the OPCT documentation with inputs from the Provider Review.
Related cards:

Document filters: https://issues.redhat.com/browse/SPLAT-831
Create Support Guide: https://issues.redhat.com/browse/SPLAT-832
Create Cluster Review Guide: https://issues.redhat.com/browse/SPLAT-833

Indirect related (mentioned on the documentation):

process command: https://issues.redhat.com/browse/SPLAT-834

/hold

@mtulio I'm done with my review

@rvanderp3 ptal too?

@bostrt PR reviewed. I left open questions unresolved from your last comments. Thanks for reviewing it. ptal?

Looks good to me :)
@mtulio what do we need to merge?

@mtulio what do we need to merge?

I have suggested some changes here https://github.com/mtulio/provider-certification-tool/pull/1 but after that I think it's nearly good-to-go!

/unhold
/lgtm


Healthcheck for graph-node docker image

Description
The docker image for the graph-node doesn't contain a health check. It would be useful to have a simple way of checking the health of a graph node in the docker image automatically.
Docker health checks are meant to provide a very simple check if the relevant application is running correctly. They should be quick and cheap so that they can execute frequently. In the graph-node case this would be a check if the graph node is up and running and not a check for a particular subgraph.
Proposal
As graph-node doesn't have a native health check, the fairly simple command to check if the node is up and running would be a GraphQL request:
query getVersion {
  version {
    version
  }
}

should return the version of the node. To run this as a shell command, we could use:
curl -s http://localhost:8030/graphql -H 'Content-Type: application/json'  -d '{"query": "query getVersion { version { version } }" }' -f -m 2

The command runs inside the docker container. However, the default docker image does not provide curl. So the proposal is two-fold:

Add a curl dependency to the docker image (i.e. apt-get -y install curl in the docker file)
Add a health check command to the docker file.

If there is interest in this, I'm happy to create a PR for it.
Are you aware of any blockers that must be resolved before implementing this feature? If so, which? Link to any relevant GitHub issues.
No
Some information to help us out

[X] Tick this box if you plan on implementing this feature yourself.
[X] I have searched the issue tracker to make sure this issue is not a duplicate.

HI, are you planning to create a PR for this change? It would be very useful, actually

we don't even need a special graphql query. We can just have a route for /health and /ready  so we can define liveness and readiness probes


📝 Refresh six classic guides

Solving #338 🤝

Hi @JFWooten4! The bounty is over but we'll still accept this PR whenever it's ready!! Thanks for all your work on this one!

Understood @briwylde08—appreciate you support. I'd also like to thank @silence48, @earrietadev, and @ducksquaddd for their helpful perspectives around native protocol AMMs. Further shoutout to @orbitlens who was a vocal early advocate of developing the most efficient system.

Hi @silence48, thank you for the thoughtful review of that last js example draft! It's been an inspiring week per some stellar feedback from regulators and community developments. Working on those likely till Sunday, and I'll be back to respond to the specifics early next week. 🤝📅
As I'm writing this, I see someone out my window mowing yards on a riding lawn mower which must cost at least $5,000. Indeed, this professional works only to mow lawns and offer other such yardwork in the area. It's this longstanding specialization that enables exponential social progress, progress which can too easily be halted by monopolization.
It's in this context that I ask if we might further advocate for specialized open-source work? Namely, it's our understanding that true sustainability for community members like yourself to just do stellar work and receive fair compensation. It's in this context that I'd appreciate the opportunity to share some thoughts on decentralizing our documentation, the very tool which widely introduced me to Stellar in the first place.[^cb]
[^cb]: After watching the inaugural Lumenaut videos through Coinbase Earn in 2017, making some trades, and reading the whitepaper.
1. Initial Reactions
Your brief review points bring up some material background which seems relevant in the context of this PR and other discussions or issues previously referencing it. Accordingly, might I share my overarching thoughts about the specific items you brought up? 💬 Namely, I am not and do not consider myself to be a coder.
All of the production applications I've written are in Python, which is why I mapped out the basic premise like that first. After that, I'm onto ChatGPT and such to preliminarily translate the info as requested. 👩💻 I think there are some challenges on that side because it doesn't seem to understand how Go account address resolution works—something I understand you've actually trained a model around for more accurate results in all languages.
Like other community members, I am much more of a writer and strategist, outside of my true call in asset management. And I believe these educational skillsets are crucial to widespread adoption of the network. Given the sunsetting of SCF education grants by my present understanding, might I share some more thoughts on the documentation, governance, and progress of Stellar? 🌌
2. An Abundant Network Opportunity
I think you more than anyone have seen firsthand just how creative our community can be given adequate resources. Particularly, it's my understanding that many projects solicit your consultation because of your active, generous, and timely support on Discord. 💜 The incredible reality of this work is that it enables specialization, a tool so crucial to efficient capitalist society.[^xdr]
2.1 Fostering Directness
Indeed, most of the challenges in this particular example come from an unfinished draft incorporating the AI response into the page. However, I ran out of time that night, and just 'saved' my progress with an incomplete commit, per the added line spacing separation. 💾 In fact, I was quite struggling with the inheritance part, as that's something I've always had a tough time with in JavaScript.
I think we build better when everyone can learn, reference, and iterate on prior research. There doesn't seem to be any reason in my mind that composability ought disappear when we enter the realm of decision making, as is so often the case in mammoth policy releases from a select few middlemen. If we agree that anyone anywhere ought to be able to develop their problem-solving skills, might we develop a collective strategy to make our choices more open to public review? 👁
Namely, it can be challenging to comprehend the operations of some stellar production applications based on their source code alone. Indeed, I intrinsically believe that the only way to end "back-room deals" is for all stakeholders to apply the "cleansing sunlight of transparency" praised by President Barack Obama. And in the sense of open source free communities, well, that happens to be all of humanity. 🌍
2.2 Efficiency for the Impoverished
Low net worth is a temporary status, and I'm obsessed with how we might empower masses of people to change that state via Stellar.[^long-term-thinking] A couple days ago, I was looking to schedule some flights back home/to family for holidays—Thanksgiving in particular. I needed to confirm some times with my partner and such, so I waited till this morning to buy the tickets since she was at work.
But when I went back to secure my flight yesterday, the price for one leg increased by 80% just this morning. Comparatively, aviation used to be regulated down to a specific cost per mile, so as to promote equal access opportunities.[^air] Without an efficient American rail system, I'll indeed be at the whims of the oligopolies of how much of my rent savings go towards this particular trip, albeit at my fault partially for not confirming some family event dates earlier.
Thankfully, our diverse, brilliant, and hardworking community seems to fall on the more considerate side of this scale. 🫱🏿🫲🏻 Indeed, it's the dramatic efficiency baked into native operations which so effortlessly empower the masses to employ the financial system, not the other way around via a select few middlemen.[^wallst] I find this particularly inspirational given the precedent of blockchain rent-extractors copying major innovations for the sake of short-term profits at the expense of long-term sustainability.
[^wallst]: See also generally discussions on the quandaries of central intermediaries in Stack Exchange, Bitcoin Forum, and Core Protocol. 💬
[^air]: Compare mandated approval for airline routes upon the contingency that they accepted less popular destination with the present shareholder-extraction model which prioritizes profits so much that they extort cities for service.[^censorship]
2.3 Shared, Nonproprietary Market Upgrades
I wholeheartedly believe that only together do we stand a chance at freeing our markets from Wall Street's tight oligopolistic grasp. And if we elect to act in unison, it seems like we might just give investors the saving grace they didn't know they needed. Might we act collectively amidst this great turning point so that the next great financial system doesn't face the quandaries of today's?
Indeed, if we are all sincerely "driven by technology that helps people to truly own what is theirs," then it seems natural that the network's documentation itself ought live at the whim of the community. Do you find this perspective particularly material, given:

99% of regular people don’t know that they don’t own a piece of all the companies they’re paying for.
First they have to understand that, then they have to decide if they prefer convenience or ownership.
— 6days1week

3. Ownership for All
As inspired by our community and deeply evident in the Network's design, might we continue advocating for the foundational promise of usable self-custody? It's my view that this mission starts at the core of our open-source advocacy, advancements, and accessibility. Namely, it's only when changes are clicks away that the dispersed, impoverished, and developing citizens spread across this vast world might coherently collaborate towards our grander mission. 🚀
Do you think it's possible and worthwhile to engineer these free markets? Might the crowdsourcing ethos so evident in protocol development itself spread to the key responsibility of widespread education through the docs themselves? 📖 This is what I think of when contemplating your generous, unsolicited, and pertinent feedback, Silence.

We're tackling an unprecedented common enemy with a unique ability to buy their way out of every past catastrophe. As someone that didn't attend Meridian due to cost, I believe so wholeheartedly that the unique community consensus methods employed by Stellar can sincerely replace their broker monopolies. Might we contemplate further how employing a regulation-aware development strategy could best serve us?
[^xdr]: See, e.g., initial instinct to learn the specific xdr implementations affecting transparent protocol amendments when starting these thoughts. This puts specific understanding further up on my timeline despite the business reality that I will end up down this rabbit hole regarding discussions, defenses, and dialogue across industry. But don't you think the true division of labor fostering stellar protocol developments should and does come from remarkable community members like you?
[^censorship]: Since first joining the community, it has been abundantly clear that the "token listing standards" loosely associated with ledger spam (in a system which favors usability) revolve around the quandary of centralized trust in a select few middlemen who "approve" or "disapprove" what is or is not an acceptable asset, potentially based on the liveness of such entity. Quite trivially, the extension of the argument presents itself if a number of pending litigations involving the U.S. Securities and Exchange Commission. If we might agree that upgradable smart contracts are so requisite to software innovation, do you think we should sacrifice this pillar of innovation for the limited application of democratizing market-making opportunities?
[^long-term-thinking]: See, e.g., a limited case presently facing masses of people which introduces, e.g., the conundrums of taking shortcuts supra § 2.3, as expressed on a permissionless basis. This innovative perspective was introduced to me by someone whom I do not know the real name of (we only know each other through online platforms, and they prefer pseudonymity).
Might we venture on the path less traveled? 🛤

Merge overrode txResponse parse as discussed—thanks for the collaboration! 🫱🏼🫲🏻

Re past few weeks, it's been quite a process understanding everyone's perspective to now get back into writing the existing SDEX section's code examples. I'd like to especially thank the above-mentioned community members for both assisting in clarifying the present system and its implications. Upon further outreach I sincerely appreciate @tomerweller, @emir/PENDING, @adam/PENDING, @ddombrowsky, @noot/PENDING, and @Row-Bear whom were instrumental in shaping my current understanding through many generous, collaborative, and thoughtful Discord discussions.

I'm removing the maker/taker diction since it doesn't apply to the DEX. Some of this got added to contextualize Stellar's functions in terms more akin to traditional markets. But these concepts are both outdated and likely unknown for most new readers—and hopefully we can keep it like that if we win. 💡

ddd80c6257c4519e65eacce4fe415809da3cda44 overrides 32455bb92664a7b4dea090d64285ebd7ff4b416b in deference to DocCardLists over MethodTables. As I understand it, most the API pages use the latter method, but it seems the implemented list system offers a more stylistically-appealing display. 🧑🏻🎨 I want to investigate how to change the emojis next to each title card, and potentially update the remaining legacy pages to this more descriptive summary page toolkit.


Panic if ticker not available

I am requesting quotes client.GetQuotes([]symbols) for a bunch of tickers and get panic if a ticker is not available in FinViz, e.g. ticker SHAC is not..
error received while scraping quote for 'SHAC': error getting url: 'https://finviz.com/quote.ashx?t=SHAC&ty=c&p=d&b=1', status code: '404'
I could catch the error, remove the ticker and pull again, but that could be much overhead, specially from FinViz point of view. Could this be fixed internally so the program does not panic but throws warning instead that a ticker is not available?

Read about the changes here.


Importer failure of test class in documentation build only

This has been going on for weeks, and I'm not certain why it started or how to fix it - see error at end.
There's test class here that is used entirely for examples in the documentation.
It now fails to import only during the documentation phase.  I've tried from inside a Python interpreter and it works perfectly well.
I feel it's something to do with the fact that we don't install the test files (as we shouldn't) but the PYTHONPATH does point to the root of the BiblioPixel directory so it shouldn't be an issue.
Interestingly enough a line towards the end seems to indicate that I'm somewhere loading bibliopixel.animation.test. I've definitely made that mistake before - it should be bibliopixel.animation.tests - but I can't find anywhere that this mistake is today in the codebase.
Worst case, I'll just move this one file somewhere obscure in the main hierarchy and use it from there, but I hate to do that.
ERROR - extract-gifs - Error in project doc/tutorial/2-example-6.gif
Traceback (most recent call last):
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 7, in imp
    return importlib.import_module(tn)
  File "/development/env/blips/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 953, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'test.bibliopixel'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 7, in imp
    return importlib.import_module(tn)
  File "/development/env/blips/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 953, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'test.bibliopixel'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 7, in imp
    return importlib.import_module(tn)
  File "/development/env/blips/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 953, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'test.bibliopixel'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 7, in imp
    return importlib.import_module(tn)
  File "/development/env/blips/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 953, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'test.bibliopixel'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 26, in _import
    return name, imp(name)
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 14, in imp
    namespace = imp('.'.join(parts))
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 14, in imp
    namespace = imp('.'.join(parts))
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 14, in imp
    namespace = imp('.'.join(parts))
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 15, in imp
    return getattr(namespace, module)
AttributeError: module 'test' has no attribute 'bibliopixel'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/development/env/blips/lib/python3.6/site-packages/loady/importer.py", line 7, in imp
    return importlib.import_module(tn)
  File "/development/env/blips/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 953, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'bibliopixel.animation.test'

Ah, I've figured out where 'bibliopixel.animation.test' comes from and it's a coincidence.  Developing.

Got it!
Very interesting - something earlier (and new) in the sys.path was matching test. so test.bibliopixel didn't resolve.  Solved by inserting the root directory at the start of sys.path.


Option for Ctrl-R to use current command-line content as a search pattern [idea]

Very often I reuse parts of commit messages from previous commands (history) in command line. zsh-autosuggestions helps me with that. For example, I type hg ci and it suggests hg ci -m "RT123456 Some previous commit message". However, sometimes it doesn't suggest the right thing. That's when I press Ctrl-R.
I thought that when my command line already contains hg ci and I press Ctrl-R to search in the history, it would be handy to use what I've already typed as initial search pattern to avoid typing it again.
I have a feeling this might be useful. Do you think you could add an option to enable such behavior? I'm using zsh. Thank you.

I think that should be doable by using the --query flag.

I like the idea, though I think it will be only possible on zsh, or maybe fish.. but I'm not really into that and growing weary of dealing with its non-standard stuff.
I'll look into it when I get some time. (I'm a bash user and honestly not really familiar with zsh stuff) Feel free to send me a PR if you feel like it.

# CTRL-R - Paste the selected command from history into the command line
fzf-history-widget() {
  local selected
  if selected=$(fc -l 1 | fzf +s --tac +m -n2..,.. --query "$LBUFFER"); then
    num=$(echo "$selected" | head -1 | awk '{print $1}' | sed 's/[^0-9]//')
    LBUFFER=!$num
    zle expand-or-complete
  fi
  zle redisplay
}
zle     -N   fzf-history-widget
bindkey '^R' fzf-history-widget

Does this work? I tried to fix #142 as well.

Committed the code. Please upgrade fzf and let me know if you see any issue.

Yes, it's awesome! Thank you very much! I'll start using it right away.


Added output param to all cli commands except coverage

Resolves #53 Add parameter to remove the verboseness of the deploy API command

Codecov Report

Merging #80 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #80   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          17     17
  Lines         306    308    +2
=====================================
+ Hits          306    308    +2

Impacted Files
Coverage Δ

deployed-contract/etherlime-ganache-wrapper.js
100% <100%> (ø)
:arrow_up:

deployed-contract/deployed-contract-wrapper.js
100% <100%> (ø)
: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 31383da...59952a4. Read the comment docs.


TextEdit: Correct typo that broke custom selected font color

text_edit's cache was trying to get "font_selected_color" but the colors name as defined in default_theme.cpp is "font_color_selected"
Fixes #27760.

Try to use the word fix/fixes/close/closes/resolve/resolves before a issue number so it's tagged automatically for closing when the commit is merged.

Looks good, but there are some more occurrences of trying to set "font_selected_color" which should be fixed too:
$ rg font_selected
scene/gui/text_edit.cpp
1181:                                                           drawer.draw_char(ci, Point2i(char_ofs + char_margin + ofs_x, yofs + ascent), '_', str[j + 1], in_selection && override_selected_font_color ? cache.font_selected_color : color);
1189:                                                           drawer.draw_char(ci, Point2i(char_ofs + char_margin + ofs_x, yofs + ascent), '_', str[j + 1], in_selection && override_selected_font_color ? cache.font_selected_color : color);
1261:                                                   int w = drawer.draw_char(ci, Point2i(char_ofs + char_margin + ofs_x, yofs + ascent), str[j], str[j + 1], in_selection && override_selected_font_color ? cache.font_selected_color : color);
1268:                                                           draw_rect(Rect2(char_ofs + char_margin + ofs_x, yofs + ascent + 2, w, line_width), in_selection && override_selected_font_color ? cache.font_selected_color : color);
1272:                                                   cache.tab_icon->draw(ci, Point2(char_ofs + char_margin + ofs_x, ofs_y + yofs), in_selection && override_selected_font_color ? cache.font_selected_color : color);
1277:                                                   cache.space_icon->draw(ci, Point2(char_ofs + char_margin + ofs_x, ofs_y + yofs), in_selection && override_selected_font_color ? cache.font_selected_color : color);
4557:   cache.font_selected_color = get_color("font_selected_color");

scene/gui/text_edit.h
187:            Color font_selected_color;

editor/plugins/text_editor.cpp
119:    text_edit->add_color_override("font_selected_color", text_selected_color);

editor/plugins/shader_editor_plugin.cpp
127:    get_text_edit()->add_color_override("font_selected_color", text_selected_color);

editor/plugins/script_text_editor.cpp
240:    text_edit->add_color_override("font_selected_color", text_selected_color);

I would suggest to also rename the variables used internally like text_selected_color and cache.font_selected_color to text_color_selected and cache.font_color_selected to avoid the confusion and further errors.

I decided not to change text_selected_color because that really is the name of the setting.
Its really just font_color_selected that strangely doesn't end with _color like everything else.
Maybe font_color_selected should be the one changing? What do you think?

Its really just font_color_selected that strangely doesn't end with _color like everything else.
Maybe font_color_selected should be the one changing? What do you think?

Ideally yes, but that would break compatibility with themes made with earlier Godot version, so it can't be done now. We can rename it in 4.0 though, I'll add a mention to it in #16863.
Changes look good, could you squash commits together?

Thanks, and congrats for your first Godot PR :)

Cherry-picked for 3.1.2.


Some tests failed to pass

Hi,
While preparing this project to include to Guix package index I've faced with
some unit tests are failed:
--- FAIL: Test (0.00s)
    --- FAIL: Test/escape.json (0.00s)
        jfmt_test.go:36:
            --- have
            +++ want
            @@ -1,6 +1,6 @@
                  {
            -have     "backtab \u0008 backtab \u0008": 1,
            -have     "formfeed \u000c formfeed \u000c": 1,
            +want     "backtab \b backtab \b":   1,
            +want     "formfeed \f formfeed \f": 1,
                      "newline \n newline \n":   1,
                      "return \r return \r":     1,
                      "tab \t tab \t":           1

    --- FAIL: Test/toml-test-key-escapes.json (0.00s)
        jfmt_test.go:36:
            --- have
            +++ want
            @@ -1,8 +1,8 @@
                  {
            -have     "\u0008":     {"type": "string", "value": "bell"},
            +want     "\b":         {"type": "string", "value": "bell"},
                      "\n":         {"type": "string", "value": "newline"},
                      "\"":         {"type": "string", "value": "just a quote"},
            -have     "backsp\u0008\u0008": {},
            +want     "backsp\b\b": {},
                      "À":          {"type": "string", "value": "latin capital letter A with grave"},
                      "\"quoted\"": {
                          "quote": {"type": "bool", "value": "true"}

    --- FAIL: Test/toml-test-string-quoted-unicode.json (0.00s)
        jfmt_test.go:36:
            --- have
            +++ want
            @@ -1,6 +1,6 @@
                  {
                      "basic_string":   {"type": "string", "value": "~  ÿ ퟿   𐀀 "},
            -have     "escaped_string": {"type": "string", "value": "\u0000 \u0008 \u000c A   ÿ ퟿   𐀀 "},
            +want     "escaped_string": {"type": "string", "value": "\u0000 \b \f A   ÿ ퟿   𐀀 "},
                      "literal_string": {"type": "string", "value": "~  ÿ ퟿   𐀀 "},
                      "not_escaped_string": {
                          "type":  "string",

FAIL
FAIL    zgo.at/jfmt     0.007s
FAIL

go version 1.21.5

jfmt version 0.0.0-20240531161922-a97493b8db3c

inputs:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I wouldn't really recommend packaging this. It's extremely experimental and has known bugs that I just haven't had the time to fix.
I assume you ended up here due to it being a dependency for toml-test, but I also wouldn't really recommend packaging that, as I mentioned in the issue you created there. Or if you must package toml-test: don't provide a package for this (although I don't know what the guix policy on that).


Register Dialog Issue

Rocket.Chat.Android version: 1.0.16 (also seen on live version 1.0.17 didn't check on 1.0.18)
When registering an account, if you set a username already in use you get an error message but you still get "logging" and the account is created anyway. And on that kind of accoutn you can't see anything or send a message to anyone.

Good finding, thanks @cheval-2!


📚 docs: Adjust badge spacing

@all-contributors please add @nathonius for code

@all-contributors check


Updating to remove set-output

Per Github set-output is now deprecated and should be removed.

@nicleary sorry for the delay
Thanks

@unfor19 Thanks for merging this one! When do you think this update will be available on the GitHub Actions Marketplace? https://github.com/marketplace/actions/install-aws-cli-action

@unfor19 Thanks for merging this one! When do you think this update will be available on the GitHub Actions Marketplace? https://github.com/marketplace/actions/install-aws-cli-action

Hi @maxlapides thanks for the reminder, I was out for a long time :)
I will create a new release later on today
Replies/inputs like this ticket are so valuable, thank you everyone for assisting with keeping this action up to date

Hey thanks @unfor19! Really appreciate it. Quick question, I am discovering that when I update my workflow to use<EMAIL_ADDRESS>the workflow runs with the latest version of this action. But, when I use unfor19/install-aws-cli-action@v1, the workflow runs with an older version of this action. Is there some way to update the v1 tag also? I'm not sure how releasing to GitHub Marketplace works exactly!

Thanks @maxlapides
Apparently, you're right; setting v1 doesn't mean you're using the latest v1 (1.0.4), which is what I would expect.
Thanks for bringing it up, I'm reading about it and I'll see if I can tweak the version to avoid modifying end-users workflows.


Create VM failed with error 'Cannot Create File'

Hi Expert,
Anyone can help looking into this failure? We randomly run into this failure and we'd like to know how to avoid it or fix it. Thanks a lot!
The method we used to create virtual machine as below:
task_info = vm_data['vmFolder'].CreateVM_Task(config=config, pool=vm_data['resourcePool'], host=vm_data['esxiHost'])
Below is the run log and exception:
10/26/2016 21:36:29 INFO     Initiate Create VM Task
10/26/2016 21:36:29 INFO     Task State: running
10/26/2016 21:36:34 ERROR    Task State: error
10/26/2016 21:36:34 ERROR    error.....:
(vim.fault.CannotCreateFile) {
dynamicType = ,
dynamicProperty = (vmodl.DynamicProperty) [],
msg = 'Cannot complete file creation operation.',
faultCause = ,
faultMessage = (vmodl.LocalizableMessage) [
(vmodl.LocalizableMessage) {
dynamicType = ,
dynamicProperty = (vmodl.DynamicProperty) [],
key = 'com.vmware.esx.hostctl.default',
arg = (vmodl.KeyAnyValue) [
(vmodl.KeyAnyValue) {
dynamicType = ,
dynamicProperty = (vmodl.DynamicProperty) [],
key = 'reason',
value = 'Hostsvc::osfs::CreateDirectory : Failed to create directory CycloneVirtualMachineTest (Cannot Create File)'
}
],
message = 'Operation failed, diagnostics report: Hostsvc::osfs::CreateDirectory : Failed to create directory CycloneVirtualMachineTest (Cannot Create File)'
}
],
file = 'Hostsvc::osfs::CreateDirectory : Failed to create directory VirtualMachineTest (Cannot Create File)'

same issue with me also


fix dataclass containing Any

Change Summary
remove Any types that contributed to usages of the dataclass decorator causing mypy errors when using the disallow-any-expr rule

Related issue number

fixes #4355
Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review

This is a fix since V1.9, so really needs to be included before we release v1.10.

Notice
See twitter :bird:, I've you'd like this to be included in V1.10, please fix it and request a review TODAY.
Or if you need this in V1.10 but don't have time to complete it (or aren't the author), please comment here and on #4324.

@samuelcolvin all outstanding conversations are resolved, unless there's something i'm missing? i think it's ready to merge

Okay, I'll look.
In future, best if you can say "please review" so the label changes and reviewers get assigned. I basically only look at PRs with the "ready for review" label.

my bad, i didn't realize i had to comment it again after making changes, i assumed clicking "re-request review" would've done it

my bad, i didn't realize i had to comment it again after making changes, i assumed clicking "re-request review" would've done it

we might be able to somehow extend hooky to detect "re-request review" if there's a webhook for it. But it doesn't work at present.

This is great, thanks so much.


undefined is not an object (evaluating 'RNFIRMessaging.getFCMToken')]

I am using expo detached project. I am able to print
console.log(FCM) successfully, but when I call any of the functions I get undefined error. Sorry, but I am trying some expo project for the first time

I got the same error. Have any solution everyone?

This is a duplicate of #1029. The library is not compiled and linked correctly. Follow the installation steps in https://github.com/evollu/react-native-fcm#android-configuration and https://github.com/evollu/react-native-fcm#ios-configuration


feat: add support for tagging to Connection API

Adds support for tagging in the Connection API. This is needed to support tagging in the JDBC driver.
This is a binary breaking change, as it adds 4 methods to the Connection interface. That interface is marked as @InternalApi and warns that it may make breaking changes without prior notice.

@thiagotnunes Would you mind taking a final look before I merge? I've rebased on master and realigned with the last choice that were made for tagging in the client library. Also: This is a binary breaking change as it adds methods to the Connection interface. Do we need a version bump for this?


Accessibility related updates.

Removed all title attribute for font icons. Replaced with hidden text. title attribute are not reliably supported on all screen readers.
Added aria hidden to all font icons
Every section should have a heading - added a heading to the intro. Hid it so it’s only available to screen reader users
Downgraded existing headings because you only want one  on the page.
Added a  at the top of the page. Feel free to add class=“sr-only” if you want to visually hide it.
Added foreground image with alt text for “open new windows” instead of relying on title attribute. Clearer for people who don’t rely on assistive technology

Thank you! Will sort out conflicts and merge over lunch! <3