rcc_f0_f1_f3_common has two values with the same name

In rcc_f0_f1_f3_common.yaml, the RCC.CFGR.PLLMUL patch contains Mul16 twice but with different values.  As a single enumeratedValue can't have multiple values, the second one ends up overriding the first.  This means that if the user would manually set PLLMUL to the first bit pattern, they would get back PLLMULR::_Reserved when reading back the value.
https://github.com/stm32-rs/stm32-rs/blob/f11c937464cf6782d0d19b27c26854ed2cdc0665/peripherals/rcc/rcc_f0_f1_f3_common.yaml#L21-L22
This is sort of an annoying situation.  I think that preferably the SVD format would support specifying multiple bit patterns for one enumeration, and then just using the first of them when writing.  But without changing the file format, the only work-around I can think of is naming the second one something like Mul16_2, and then seeing if svd2rust could be modified to detect this and merge them together for reads, so that the consumer doesn't have to care what the actual bit pattern was.

Futhermore f101-107 have differ PLLMUL:

Only F105 and F107, which make up the connectivity line of F1s, have that PLLMUL, whereas all other F1 series devices have the PLLMUL values currently specified.

Is #259 better?

Closed by #259.


Don't show "No changes!" notification when there are local plugins

I think that the "Plugins Update > No Changes!" notification shouldn't be shown when there are local plugins because it's not necessarily true and for me it actually blocks an error notification about the plugin I'm working on. May I submit a PR to change this conditional?
https://github.com/zeit/hyperterm/blob/f175268912d01ce13bf853fa5afe5dda2c3ae0aa/app/plugins.js#L101

Probably best to submit a PR and then have it reviewed so ZEIT knows what your intentions are. This is the easiest way for the change to happen. 😄

Yea, I need to do some further investigation actually, because what I really want is for the notifications to stack, instead of overlapping.

Hey there, please try again with the new Hyper v2 release and open a new issue if the problem stills exists. Thank you!


Model output error and execution failure when increasing sensitivity and threshold.

Hello, I hope everyone is well.
I have two problems when I'm using the BirdNET GUI.
The first is that my custom model's output does not save when I select "Raven" or "Both" as the output format. And it's important to note that this only occurs when I don't use autotune
The second point is that the detection process is not executed when I increase the sensitivity and threshold.

Please provide more details. I was not able to reproduce you problem on the current version.
Are you using the Windows installer? You can also have a look at the error_log.txt or logs.txt for further information if an error occured.

No activity on this issue, so I'm closing it.


Incomplete command in Appendix 1

Hi @ijrussell
I'm currently reading your book to learn about F#. It's quite easy to digest and keeps me interested to know more about the language features.
I've arrived in chapter 4 and have issue following the appendix 1 to setup the project.
This line is not working on my end (.NET 6.0)
https://github.com/ijrussell/essential-fsharp/blob/bef2ea6e79dd1d26b53d5270d7152cf1a6b8f319/Chapters/99b - Appendix1.md?plain=1#L21
[09:37 | ch-3]$ dotnet sln add tests/MyProjectTests/MyProjectTests.fsproj
Specified solution file temp/learn/fsharp/ch-3/ does not exist, or there is no solution file in the directory.

So I looked at the help command and I should write the command in the following format
Usage:
  dotnet [options] sln <SLN_FILE> add [<PROJECT_PATH>...]

I modify the command from the book a bit to
[09:37 | ch-3]$ dotnet sln MySolution/ add tests/MyProjectTests/MyProjectTests.fsproj
Project `../tests/MyProjectTests/MyProjectTests.fsproj` added to the solution.

I guess we need to update the example command to dotnet sln MySolution/ add tests/MyProjectTests/MyProjectTests.fsproj  so others can follow through

Never mind, I found that if we put the MyProject inside the MySolution directory then the command works.
I was putting MyProject and MySolution in separate directory hence the command does not work.
I'll close this issue


add localstack container

Official docker image

LocalStack Docs : https://docs.localstack.cloud/overview/
LocalStack Github : https://github.com/localstack


[Console APP]Summary of user store addition missing some fields

Describe the issue:
When adding user store through console app, the summary of the user store doesn't contain connection url and some other details.
How to reproduce:

Create a user store through the console app.
Check the summary tab you will see below details which missing some details.

Expected behavior:
All details should be filled
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [IS-5.11.0-beta2-SNAPSHOT]
OS: [Linux]
Database: [H2]
Userstore: [LDAP]

Similar issue: https://github.com/wso2/product-is/issues/9881
Duplicate issue: https://github.com/wso2/product-is/issues/9780
Fixed by https://github.com/wso2/identity-apps/pull/1250


Update source data

Hello,
I would like to add a few new volumes and spot sets to the platy browser. The registration procedure from my raw data to something registered with the volume in the platy browser is still work in progress, so I wonder if there's a way to update the source data without deleting and creating the source again?

The registration procedure from my raw data to something registered with the volume in the platy browser is still work in progress, so I wonder if there's a way to update the source data without deleting and creating the source again?

If you know what you're doing you can just replace the corresponding image data or tabular data. You can read the corresponding  information about a source from the metadata like this:
from mobie.metadata import read_dataset_metadata

dataset_folder = "/path/to/mobie/project/dataset"

ds_metadata = read_dataset_metadata(dataset_folder)
source_metadata = ["source_name"]

This will contain all info stored within MoBIE about the source (that is not part of the actual data), including the (relative) path to where the data is stored.
You could then replace the data directly instead of removing and adding the source again.
NOTE: if you do this for image data you need to take care to update all resolution levels.


Entering accented characters

There are some symbols in OS X which are outside of keyboard. So you can hold some key and in combination with a number get another letter, this is called accented characters.
When I do this texture append accented character after the cursor, but still the backspace will remove it (e.g. it will remove what is after the cursor):

It is not necessary to be outside the keyboard, for the Spanish case and with the accent keys, it also doesn't work.


Adding language preference page 186713028

Adding language preference page
PT#186713028

There are some acceptance criteria for this screen that aren't happening:

When the page first loads, the default language for both reading and writing, and speaking should be set to the current language of the application.
If a client changes the language that they prefer to read and write to a different language than the current language of the application, then site language should be immediately set to the selected language.
Tracking: Once the onboarding-language-pref screen is completed, the pref_written_language and pref_spoken_language properties should be set with the client's chosen language and applied to all future events.

You could either include them in the PR or accept this PR and continue the work in another PR.


use browserify pack latest es6-promise

I'm looking for a solution for using Promise to provide plugin interfaces. I think use a plugin providing common Promise is a cool idea.
I find this plugin use old 1.0.0 es6-promise. Version 2.3.0 include performance improvements and bug bugfixes.
Also size of offical es6-promise.min.js is 53.3 KiB. Using browserify with uglifyjs can reduce its size to 8.0 KiB.
I write a script to pack them together. It can pack latest version.

Thanks for bringing this PR forward! Having a nicer way to release the plugin in sync with es6-promise is really needed.
I've checked the es6-promise.min.js in the dist directory and it appears to have 18 kB. I always thought that size is not an issue for Cordova/PhoneGap apps as the dependencies are bundled in the app, instead of being downloaded...

My Cordova app using a remote URL to load pages. That's the reason why I do not pack a bluebird to my plugin: the size is over 180 KiB. Too huge for a syntactic sugar.


Doc: update README.md

Feature Description:
we need a better, up to date and more user friendly readme
Expected Behavior:

up to date and relevant README.md
make it cool and appalling

I would like to work on this but can you give me information on what needs to be updated and what the new information would be

Hello!
Thanks for your interest in updating our README.md! We’re aiming to make it more engaging and informative. Here’s what we’re looking for:

Introduction: Let's make the introduction catchier. Perhaps add a quick example of how Cogo simplifies command tasks.
Installation Instructions: More details about prerequisites and troubleshooting tips could smooth out the setup process for new users.
usage: perhaps upgrading the usage section can be nice, providing more details about what each command do and possible the right flag usage for each command.
Visuals: Adding some logos or badges could make the README visually appealing.
Contribution Section: Let’s warm this up by explaining how contributions are integrated and what types of contributions are most helpful.

Feel free to tackle any part that interests you, or if you have new ideas, let's hear them! You can start by forking the project and creating a feature branch. If you have questions or need more specifics as you go, don’t hesitate to reach out.
Looking forward to seeing what you come up with!


Interact event fire twice

Hi! i using command with interact event and its fires twice on use
Also did you fixed bug with placing heads?

The latest version (1.3.3) fixes the interact-event issue, you can get it from the Spigot page under the version "<IP_ADDRESS>" because I needed to fix a small bug with releasing 1.3.3.
The issue with placing heads is in the priority 1 section on my todo list and will be in the next update, thank you for waiting this long :)


No option to create integration in vscode extension

In vscode extension there is no option to create new integration. Previously it used to have plus icon.

I got it. No button in Karavan view


[Performance]: Show a warning when an operation is going to take time

Currently users are not aware when an action is leading to the execution of expensive operations. For eg. editing a token with the setting 'Apply to Document' and 'Update on Change'.
Adding a dialog box that will inform the user when an expensive operation is going to take place.

Definition of an expensive operation:

Edit token (Apply to Document with > 1000 nodes)
Edit token (Apply to Page with >1000 nodes)
Switching theme (Apply to Document/Page with > 1000 nodes)

With the performance improvements I would consider this obsolete. We were able to bring down this number by a huge factor, so I don't think we should do this at this point.


build fails with just 1.13.0

$ uname -sm
Darwin arm64
$ just --version
just 1.13.0
$ just compile
error: Expected backtick, identifier, '(', ')', '/', or string, but found '{'
   |
47 | _build features="": (compile {{features}})
   |

I noticed the issue because I tried tilt up, which uses just
$ tilt up
Tilt started on http://localhost:10350/
v0.32.0, built 2023-03-13

(space) to open the browser
(s) to stream logs (--stream=true)
(t) to open legacy terminal mode (--legacy=true)
(ctrl-c) to exit
Tilt started on http://localhost:10350/
v0.32.0, built 2023-03-13

Initial Build
Loading Tiltfile at: /Users/mkm/Build/controller-rs/Tiltfile
compiling with features:
Successfully loaded Tiltfile (6.091125ms)
WARNING: You are running Kind without a local image registry.
Tilt can use the local registry to speed up builds.
Instructions: https://github.com/tilt-dev/kind-local
      compile │
      compile │ Initial Build
      compile │ Running cmd: just compile
      compile │ error: Expected backtick, identifier, '(', ')', '/', or string, but found '{'
      compile │    |
      compile │ 47 | _build features="": (compile {{features}})
      compile │    |                              ^
      compile │ ERROR: just compile exited with exit code 1
      compile │ ERROR: Build Failed: Command "just compile" failed: exit status

Regression introduced in #47

Ah, thanks. I must have missed some quotes in there. Did those last end bits rather quickly and the justfile isn't used in CI.


[suggestion] Simplify logger's behaviour and configuration

Description
I think the current logger's behaviour & configuration are a little overcomplicated, and I see a way to make its UX more straightforward and useful.
Current implementation
Iroha prints logs into standard output in default way and optionally into a file in JSON format.
There are these options:

level: logging verbosity
compact_mode: flag to make logs in the standard output less verbose in terms of format
terminal_colors: flag to control ANSI colouring in the standard output
log_file_path: enables JSON logging into a file, in a addition to standard output

log_file_path seems not useful for me here. It is the only way to get JSON logs, and if you need JSON logs, than you most probably don't need any other logs. However, stdout is always busy with default logging.
Apart from that, I've created a separate issue for removal of terminal_colors from the logger configuration:

https://github.com/hyperledger/iroha/issues/4030

Proposal

Remove terminal_colors (https://github.com/hyperledger/iroha/issues/4030), log_file_path and compact_mode
Use stdout and a single output destination. It is easy to redirect it wherever, e.g. into a file.
Introduce format option to change the format of logger output. I think we can just adopt the formats supported by tracing_subscriber out of the box:

full: default format now
compact: what is now enabled with compact_mode
pretty: I would personally very appreciate having this option, it might be very helpful  during debugging of Iroha and development of SDK
json: JSON machine-readable output

So, after all there will be only two options to configure: level and format.

Closed by #4100


nixos/repart: add loop module to initrd

Description of changes
Systemd-repart will use loopback devices for partition creation if it is able to, and will fallback to doing "offline" partition creation writing data directly to files. From what I see looking at the repart code, there are specific features that cannot be taken advantage of when not using loopback devices (e.g. no BTRFS subvolumes in systemd v255) and in certain places they have to perform some manual re-sizing work that can otherwise be avoided.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1386


Normalize resource path for displaying S3 objects

Currently Hue errors out after refreshing the browser when
displaying a file. When refreshing the page it errors out
with "Not a file: ..." It looks like Django is appending a slash
to the resource path, and s3fs is assuming that this object
is a dir because of the appended slash.
To fix this, I propose that the resource path is normalized
in parts of s3fs. This provides consistency with how "view"
calls "stats" (which normalizes path) when routing "display" and
"listdir_paged" views. After this change, something like s3a://foo/bar/
will be mapped to s3a://foo/bar.

@dpvelasquez, Can you try with master branch? We've made some changes to our handling of paths in the URL and I believe this issue should be fixed without this change.


Title: New Game Touch-No-Fire Game

Assign under GSSOC
Touch-No-Fire-Game
Enjoy The Game!!!

Description 📃

This is a Normal crossword game that we can play in virtually and blow I have print some tips for the beginners as well!
The project will contain HTML, CSS and JavaScript files. The HTML file adds structure to the game followed by styling using CSS. JavaScript adds functionality to the game.

How to play? 🕹

Controls :

Game Rules!

Screenshots 📸

Working video 📹
https://github.com/GameSphere-MultiPlayer/GameSphere/assets/154777864/a88f676a-69c8-4cc9-b6e6-98e5fca1aa59

@Anjaliavv51 Thank you for providing your insight but it is not required as of now


Added refreshSecret method to ApiKey model

refs #9865
This will be used for when we PUT to the apikey resource to update the secret in the case that it's been comprimised.

I would keep the comment specific to what the code is doing which is adding a refreshSecret method to the model.

Yeah that's a good point, I'll update the message when squashing it into master :+1:


use COPY --chown=hubot:hubot over ADD

COPY should be used over ADD and use "--chown=hubot:hubot" to get around the permissions hacks of the past.
cc: @Sing-Li @geekgonecrazy

Thanks for contributing that.  We have a PR open here https://github.com/RocketChat/hubot-rocketchat/pull/289 that is related and completely revamps things


New version incorrectly suggested

I just received a new version notification even though there's nothing newer than 1.25.1 and I'm on that version:

Not seeing it here.
It gets the version number from: http://markdownedit.com/version.txt

Not seeing it now either. I remember there were network problems at that time so maybe the request returned something like 500 ... and it translated to version number 500 and prompted for update, or something like that.

It should just ignore a bad network connection or 500 response.
https://github.com/mike-ward/Markdown-Edit/blob/master/src/MarkdownEdit/Models/Version.cs

Ah so maybe if the shopping mall WiFi returned a login HTML instead of 1.25.1 (as those public WiFi's commonly do), that method returned false and I was prompted for update.

That's a possibility. Guess I'll have to add some sanity checks check the response (numbers and periods only for instance).


Fixing deadlock, fixing typo

We had a deadlock that in the mutex lock, the bug locked the destruction of the objects created and it was aggravated with the new PUSH_SINGLE_PACKETS macro, but also it was likely to happen without the macro, 48 times less likely (the rate of datapackets per revolution) but still affected. This PR should fix it. Also a typo was corrected in the CMakeLists files.

Thank you for your contributions!


The channel is has been closed When we use more that 1 parallel process

Hello. I tried to use this code for getting some data  from url array
Loop::run(function ()  {
    $uris = [
        'http://local.test/sleep.php?sec=1',
        'http://local.test/sleep.php?sec=5',
    ];

    $threads = [];
    foreach ($uris as $key => $uri) {

        $threads[] = Thread::spawn(function ($uri) {
            $result = file_get_contents($uri);
            return $result;
        }, $uri);
    }

    foreach ($threads as $thread) {
        $result[] = yield $thread->join();
    }
});

Code from sleep.php : <?php sleep($_GET['sec']);
And it is works correctly, but if i change the line order of urls from
'http://local.test/sleep.php?sec=1',
'http://local.test/sleep.php?sec=5',
to
'http://local.test/sleep.php?sec=5',
'http://local.test/sleep.php?sec=1',
Every time I've error:
Fatal error: Uncaught Amp\Parallel\Sync\ChannelException: The channel is has been closed in /home/anton/www/test/php/parallel/vendor/amphp/parallel/lib/Sync/ChannelledSocket.php:251

Could you tell me how I can solve such problem?

This was an issue I created in 9cc70c1f86bf8ffa8ca744e40702502410b733f0, but should now have been fixed with a31f3491ca25db50b5a253b30fee2ade2ae6bd9f. Pull the latest master and try again please.

Thanks a lot, it works!


driver/AWG70000A broadbean compatibility

To make the AWG70000A .seqx file generation play nicely with broadbean, some code restructuring and slight API changes are needed in the AWG70000A driver. This PR introduces those.
Changes proposed in this pull request:

[x] make makeSEQXFile take numpy arrays rather than .wfmx files as input
[ ] add more tests of file generation
[ ] update the documentation to reflect the changes

There is a parallel PR in broadbean about this: https://github.com/QCoDeS/broadbean/pull/47.
@jenshnielsen

Apart from the commented out line that should be removed this looks fine


SingleItemRecovery default value incorrect

Hello,
The default value for parameter -SingleItemRecoveryEnabled is incorrect in the documentation:

Please fix it
Thank you.
Ribo

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

ID: c7f835af-5905-368a-df68-6646782ad807
Version Independent ID: 153ce2d2-dab5-9c8f-c511-ee3bc8ee2dbc
Content: Set-Mailbox (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Set-Mailbox.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@riohedfield, the problem you are describing is apparently a well-known and ancient Windows PowerShell issue (product independent): no 'reflection' utility that attempts to dump cmdlets to cmdlet reference topics (we use platyPS) is able to correctly read the state of the 'Default value' property on a parameter, even when the parameter really does have a default value (many do not).
Of the ~1410 Exchange-related cmdlet reference topics, only 15 have 'Default value' entries that contain something other than 'None'. Or, to put it another way, of the ~14,316 parameters in cmdlet reference topics, only 42 of them have a 'Default value' entry that contains something other than 'None'.
We've made no effort to globally populate the 'Default value' values by hand. Sometimes, people take it upon themselves to manually fix those entries themselves.


[BUG] Segment_N file cleanup not happening on replicas for indexes with segrep using remote store replication

Describe the bug
Observed this issue on replica shards where the segment_N file is not getting removed even after the commits (flushes).
To Reproduce
Create index with segment replication using remote store replication. Index docs, refresh, & then flush. On the primary, the segment_N+1 would appear and segment_N would be deleted, but on replica both segment_N and segment_N+1 file shows up. If you repeat the (Index docs, refresh, & then flush), then more and more segment_N files shows up on replica.
Expected behavior
There should be only 1 segment_N file in steady state.
Plugins
Standard
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: [e.g. iOS]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

This is happening due to changes introduced by #8050 which removes replica commits and as a side-effect the cleanup of these files as part of the commit process is being skipped.

This will be fixed with https://github.com/opensearch-project/OpenSearch/pull/9111


updated to Roslyn 3.4.0-beta3-19516-01

We have fallen behind the schedule a bit (especially with .NET Core 3.1 Preview 1 out already). We had consumed 3.4.0-beta1-19460-02 over a month ago but even Nuget publicly has a newer version now.
This PR pulls in the latest build intended for VS 16.4.

@david-driscoll are you ok with merging this?

Yeah ship it
On Thu, Oct 17, 2019 at 4:01 PM Filip W<EMAIL_ADDRESS>wrote:

@david-driscoll https://github.com/david-driscoll are you ok with
merging this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OmniSharp/omnisharp-roslyn/pull/1634?email_source=notifications&email_token=AAJV3JM4S7WEK6BTAMF6YMDQPDACHA5CNFSM4JBLYAB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBRK6VI#issuecomment-543338325,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJV3JK5SHNU7SFUZFAQIB3QPDACHANCNFSM4JBLYABQ
.


Use --auto-iam-authn with IAM based user does not work with sqlalchemy python

Description
I like to use alloydb-auth-proxy instead of python language connnector.
psql client is working when I authenticate using service account and not specifying PGPASSWORD. But sqlalchemy is throwing error when I'm not specifying password:
failed: fe_sendauth: no password supplied

Potential Solution
Can you give us some guidance for using sqlalchemy with IAM based user and alloydb-auth-proxy?
Additional Details
No response

Here's what I'd expect to see:
Start the Auth Proxy like this:
# starts a listener on port 5432 by default
# also assumes private IP, use the `--public-ip` or `--psc` flag otherwise
./alloydb-auth-proxy <INSTANCE_URI> --auto-iam-authn

Connect from SQL Alchemy like this:
import sqlalchemy

engine = sqlalchemy.create_engine(
        "postgresql+pg8000://localhost/postgres", # or whatever database you want to connect to
)

# use engine as usual

Thank you so much for replying. I tried that already:
          args:
            - --auto-iam-authn
            - --public-ip
            - --address=<IP_ADDRESS>
            - --http-address=<IP_ADDRESS>
            - --health-check
            - --prometheus
            - --run-connection-test
            - --debug
            - --quitquitquit
            - --structured-logs

But still got error:
failed: fe_sendauth: no password supplied

oh, do you mean no need to put IAM user? Right now I'm using this user and use empty password like this:
postgresql+asyncpg://mv-svc\%40my-project.iam@localhost:5432/mydb

oh it seems working now. Thank you so much.

You can check your Auth Proxy logs to confirm -- it will report on connections being established and so-on. Glad it's working. Feel free to re-open if you have any other questions.

For others who face similar issue. I had to put PGUSER to match the alloydb-auth-proxy user for it to work.

      # Still need to set PGUSER to match service account alloydb-auth-proxy is using
      -<EMAIL_ADDRESS>      # Should not have user or password, which will be handled by alloydb-auth-proxy
      - DATABASE_URL='postgresql+asyncpg://localhost/mydb'

This is a common misunderstanding -- for Auto IAM AuthN to work the database user must match the Proxy IAM user.

You can put that IAM user in the connection string, but the at sign @ will confuse some drivers and might need escaping.

Sounds good. I'll try to experiment with this and see what kind of escaping has to happen to make it work. FWIW other languages struggle with the @ too because using Postgres drivers interpret that as the start of the host (instead of part of the user).

I already escaped that @ like this: https://github.com/GoogleCloudPlatform/alloydb-auth-proxy/issues/724#issuecomment-2489049049
but still have same issue failed: fe_sendauth: no password supplied. I don't know why though.

notice that I also needed to escape \% , otherwise the sqlalchemy will report string interpolation is not correct.

To escape the @ symbol in the username, we need to use %40. This is stated in the sqlalchemy documentation: https://docs.sqlalchemy.org/en/20/core/engines.html#escaping-special-characters-such-as-signs-in-passwords.
If my username is<EMAIL_ADDRESS>the following worked for me: "postgresql+pg8000://user%40example.com@localhost/postgres".


FIX Devise::Models::SecureValidatable#email_not_equal_password_validation

Devise::Models::SecureValidatable#email_not_equal_password_validation crashes* with a nil email, whereas in case of blank email this validation should be skip (password validation context)
NoMethodError:
NoMethodError: undefined method `downcase' for nil:NilClass
from ~/.asdf/installs/ruby/2.6.7/lib/ruby/gems/2.6.0/gems/devise-security-0.16.0/lib/devise-security/models/secure_validatable.rb:82:in `block in email_not_equal_password_validation'

Fix looks good. Please add a test to cover this case!

Hmm looks like there's a problem with the library + ruby head. I'll look into it.


UX: Terminal Tab - Pin feature request.

Description of the new feature/enhancement
Could you add terminal tab pinning feature like in the browsers?
Think, you're running "batch scripts" in a terminal page and it will be end at 2 hour later. If i can pin this tab it's not confused me when i'm using other pages.
Proposed technical implementation details (optional)
If Tab state is pinned, shrinks , comes to front order and can be prevents accidentally close.
Like bellow;

We have #5969 for this. I'd say it's indeed a good idea.

Thanks for the suggestion! Please make sure to search for duplicates when filing new issues. This is already being tracked by another issue on our repo - please refer to #5969 for more discussion.
/dup #5969


style: body content text color

We used to have a different color for proposal content, currently it's using the text color variable, it's too dark unfortunately:

We should going back to earlier version of SX UI to find out proper color for the body content and change to it.

Fixed in https://github.com/snapshot-labs/sx-monorepo/commit/a2b79c290298b1223f7c5fb8b109e1f24a5bf706


How to increase quality?

Hey,  @gitmylo, great work on this repo.
If I want to increase the quality what's the best way to go about that?
I imagine the number of steps that are used during both training and inference must be stored in some variable somewhere. Can you point me to it?
Or maybe there's another obvious solution?

inference steps? i don't know what you're talking about?
this is not an iterative model, it just takes an input, and spits out an output. in one step.
quality is mostly held back by bark's inconsistency, bark can switch speakers on its own, bark can slowly fade to another speaker as well. this can be unpredictable.
because of that, quality greatly depends on how the audio ends, the quality of the audio, the words spoken, and more.

Oh, I see. I just assumed it was iterative. Thanks for the info.
If I want to train it for TTS purposes to have more consistent quality, with cloned voices of arbitrary sample quality -- is that possible with this model?
Like, would training the core model improve the likelihood of high quality output despite low quality voice cloning samples -- something akin to how 11Labs works?

no, this model cannot be trained on a specific voice, as what it does doesn't really interact much with the voice.
more info can be found here: https://github.com/gitmylo/audio-webui/wiki/how-bark-works

Interesting. Are you aware of this HG space?
https://huggingface.co/spaces/kevinwang676/Bark-with-Voice-Cloning
It seems to be be based on this repo, and succeeds at voice cloning.

I was not aware of it until now, but yes, the code used in that repo is exactly the same as my example code
In fact, these files are still the originals, including the original credit.
The official ways to run are:

audio-webui

audio-webui colab

official colab notebook
regular ipynb notebook
raw code
official huggingface space


Only 3 Presets are visible in HomeKit

I only have 3 presets available to select from in the HomeKit app although I have entered 6 in the HomeBridge Config. I've tried this multiple times even with a different ESP board for WLED but no luck. I only ever get to chose the first 3 presets.
Plugin Config:
        {
            "name": "Jeriel's LED Strip",
            "ip": "<IP_ADDRESS>",
            "presetNames": [
                {
                    "presetNb": 1,
                    "name": "Fireworks"
                },
                {
                    "presetNb": 2,
                    "name": "Peachy"
                },
                {
                    "presetNb": 3,
                    "name": "Ice Fire"
                },
                {
                    "presetNb": 4,
                    "name": "Fire"
                },
                {
                    "presetNb": 5,
                    "name": "Aurora"
                }
            ],
            "platform": "WledPreset"
        }

Screenshots:

Environment:

Plugin Version: v0.1.1
Homebridge Version: v1.3.8
Node.js Version: v14.18.0
NPM Version: v6.14.15
Operating System: Docker

Hi Jeriel,
This repo has been on standby for quite some times as I moved my leds in some other place in the house before completing this.
At the moment, only 3 presets can be added because it is "hard-coded" (see these lines).
I should be able to start working again on the project during December. I will keep this issue open as a reminder, but that was defnitely something I wanted to implement. I will comment back here when I will have implemented changes.
Adrien

Hi Jeriel,
I just pushed version 0.2.0, when configuring the plugin you will be now asked how many presets are configured on your WLED interface. The number you indicate assumes that your presets ID in WLED are incrementing 1 by 1 starting from 1 (eg. if you have 2 presets with ID 2 and 40, you will need to enter 40).
Hope that make sense.
Please note: At the moment the plugin only allows to use 1 WLED/ESP board.
Feel free to reach out if you have any feedback/requests.
Adrien

Hey Adrien,
Thanks for the update! The numbering makes sense to me.
I just gave it a try, I have 9 presets set up, so I configured that appropriately on the plugin settings. However I still only see the roller dial for the first 3 presets. The first three presets within the roller dial in HomeKit do work in accordance with the WLED Preset numbers. Differently to before, I now see these other 5 tiles for each preset beyond preset 3, however I cannot interact with them (as in turn them on or off)
My config is now as follows:
Plugin Config:
        {
            "name": "LED Strip",
            "ip": "<IP_ADDRESS>",
            "presetsNb": 9,
            "platform": "WledPreset"
        }

I've also tried adding the preset names to the config as below, but no difference.
        {
            "name": "LED Strip",
            "ip": "<IP_ADDRESS>",
            "presetsNb": 9,
            "presetNames": [
                            {
                                "presetNb": 1,
                                "name": "Fireworks"
                            },
                            {
                                "presetNb": 2,
                                "name": "Peachy"
                            },
                            {
                                "presetNb": 3,
                                "name": "Ice Fire"
                            },
                            {
                                "presetNb": 4,
                                "name": "Fire"
                            },
                            {
                                "presetNb": 5,
                                "name": "Aurora"
                            }
                        ],
            "platform": "WledPreset"
        }

Screenshots:

Hi,
The first config is the way to go now, I removed the presetNames for now (planning to implement it back in the future, it is just a bit more complicated to implement now as I use a loop).
To fix the tiles, I'd recommend to go to your Homebridge UI and use the "Remove Single Cached Accessory" feature. I faced the same behavior as well. I think I am doing something wrong somewhere with the cached accessories. I will try to figure this out in the future.

Hello,
Any way you can fix this issue (I assume it has something to do with these changes to allow for more presets) to allow for more than 1 WLED instance?  If I add a second WLED instance, HomeBridge keeps crashing and goes into a restart loop.  If I need to open a new 'issue' I can.

@osxster Yes we can definitely open another issue for this one. I will try to get this one solved ASAP as it annoys me as well.

Sounds great!  Just started deploying WLED and wanted to figure out a way to get these into my HomeKit environment.  So I'm obviously new to your plugin.  I also was trying to play with Playlists, was hoping that a playlist would show up as a preset.  However if I create a playlist, and execute the playlist from WLED, the presets no longer work in WLED Preset.  The power button works, but the playlist keeps playing.  I have to go into WLED and manually select a Preset for WLED Preset to start allowing you to select a preset again.

Closing this issue as v0.3.0 allows to add as many presets as desired.


NAV-204: Landserver-to-BBB waypoint test script

Can you just rebase and squash your commits

Should be good now!


Save and load authored blockly code

This is a small addition to save the authored blockly xml, using the available LARA features, and piggybacking off the student save-data model.
I was going to include the authoring support for the rest of the model parameters, but that is becoming a much bigger commit so I thought it wise to get the small one out first.

@sfentress what's the easiest way to test these changes?


Obtain football-data.org token

Description of Problem:
When I am developing code, I want to be able to access the football-data.org API, so I can learn the API and figure out how to meet the bot's requirements.
Potential Solutions:
Register as a football-data.org developer, and obtain a token.

I now have a token.


Fix devcontainer on Codespaces

Resolves #3606
What is being addressed
Fixing build of devcontainer on codespaces by removing sshd extension.

/test-force-approve


Committing to binded field

Hey there,
I have a bit of a weird one.
I have a form which is bound to my store using this plugin - it works awesome! Thank you.
The only thing I want to do is on mount, to automatically assign something to one of the form fields.
For example, I have a select field with options ['red','green','blue];
There is a v-model on this select field which is mapping my 'colour' field in my store.
I can't put the colour as default in my store, I need to query the database and get the default colour and somehow update the default colour on mount, in my mount function if I try to assign the dynamically retrieved colour - nothing happens. It also doesn't work when I try to do a mutation of that field.
Is there any other way, perhaps something I'm missing?
Cheers

Provide a reduced test case or a live example.

– https://github.com/maoberlehner/vuex-map-fields/blob/master/CONTRIBUTING.md#reporting-issues

@mrdigitalau Most likely this is a race condition that is causing the problem. I.e. you are probably doing the request and assignment asynchronously. You need to ensure that your assignment occurs only once your request has completed, in other words you need to look into Promise handling and / or async / await.
I'm closing this as there is no further feedback from OP. If you still have an issue, please feel free to reply and I'll re-open it.


Avoiding request other pages than the first one when a search is submitted

This was fixed in the dashboard search and now it is fixed in the create dialog search.
REVIEWER: @ethervoid :dancer:
Fixes #4291.

Frontend tests were OK :+1: (details)

@ethervoid ping!

retest this please

I was waiting the test to pass. LGTM :)

If frontend tests pass, it is ready to go :).

Frontend tests were OK :+1: (details)


Upload Data backend and frontend

@andrewbovbel we had a recent PR to clean up the frontend, it looks like you have a few merge conflicts with it. The fixes were mainly to the text and colours, lmk if you need help resolving them.

should be a-ok
On Sat, Nov 25, 2023 at 6:11 PM Rawan Mahdi @.***>
wrote:

@andrewbovbel https://github.com/andrewbovbel we had a recent PR to
clean up the frontend, it looks like you have a few merge conflicts with
it. The fixes were mainly to the text and colours, lmk if you need help
resolving them.
—
Reply to this email directly, view it on GitHub
https://github.com/DSC-McMaster-U/Auto-ML/pull/153#issuecomment-1826439542,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG2V4C7RSROOGCHH4CXMC5DYGJ3KNAVCNFSM6AAAAAA7ZZVNCGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRWGQZTSNJUGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Executing arbitrary shell command

What's the best way to execute a command in shell?
Let's say I want to run a custom task and execute some command in the current PyB folder?

from subprocess import check_call

check_call(["/opt/myprocess", "arg1", "arg2"])

good. thanks.

i guess the exec plugin doesn't allow for a command hook on the 'post-build source dist'?

Also, any way to get the 'expanded'/'effective' values of properties set on the project?
The ones which have stuff like $dir_dist/blah/blah/blah but with $dir_dist substituted with the referenced property value?

project.expand_path("$dir_dist/blah/blah/blah")

project.expand_path("$dir_dist/blah/blah/blah")

Perfect!
Did I miss it in the docs?

One more indelicate request: is there any way to know what Python version PyBuilder is being run with?
Or, maybe 'pip3' is always available as a command?
I hard-coded the PIP name (pip3) to install local dependencies for AWS Lambda packaging but not sure if I can discover the name of that command instead...

import sys
print(sys.version_info)

import sys
print(sys.version_info)

Thanks! Real helpful.


Add cabify as adopter

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

Thank you @maaaikoool ! Welcome to the official adopters :)


sidecar egress mtls configuration override ROOTCA to cause internal mtls failure

Bug description
I found after adding client cert/key/CA in pod annotation (sidecar.istio.io/userVolume and sidecar.istio.io/userVolumeMount), the CA will override the ROOTCA in envoy proxy configuration, and then internal mtls communication will failed even external mtls is ok.
The client cert/key/CA will be used to connect to external service for MTLS communication ( I use sidecar directly instead of through egress gateway). The annotation will auto add volume/volumeMount to the pod when pod creating, and sidecar can use it later which cert/key/CA path is also identified in DR, and then I add SE and DR to try to enable external mtls communication.
After that, the external mtls is ok, but internal mtls is failed for ROOTCA body is replaced by the new CA in annotation. I don't find the same problem in egress gateway.
More detail can be found in the reproduce steps later.
Affected product area (please put an X in all that apply)
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[x] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Expected behavior
After configuring sidecar egress mtls (annotation + SE +DR(with cert/key/CA path), internal mtls ROOTCA is not be override and also can work.
Steps to reproduce the bug
0.  Deploy client to connect to external mtls server
1.1 deploy sleep with sidecar and client cert/key/CA in NS hobby
kubectl -n hobby apply -f <(istioctl kube-inject -f samples/sleep/sleep.yaml)

1.2. Add annotation to assign client cert/key/CA
Add the following annotation in sleep deployment to add new volume for cert/key/CA which will be used to communicate with external mtls service later.
         sidecar.istio.io/userVolume: '{"egress-secret":{"secret":{"secretName":"client-certs","optional":true}},"egress-ca-secret":{"secret":{"secretName":"client-certs-cacert"}}}'
        sidecar.istio.io/userVolumeMount: '{"egress-secret":{"mountPath":"/etc/istio/egress-certs/","readOnly":true},"egress-ca-secret":{"mountPath":"/etc/istio/egress-ca-certs/","readOnly":true}}'

2. Add external service mtls configuration (SE + DR) in NS hobby
kubectl apply -f - <<EOF
apiVersion: networking.istio.io/v1beta1
kind: ServiceEntry
metadata:
  name: first-service-entry
  namespace: hobby
spec:
  exportTo:
  - .
  hosts:
  - my-nginx.mesh-external.svc.cluster.local
  location: MESH_EXTERNAL
  ports:
  - name: http-port-for-tls-origination
    number: 31443
    protocol: HTTP
  resolution: NONE
---
apiVersion: networking.istio.io/v1beta1
kind: DestinationRule
metadata:
  name: first-destination-rule
  namespace: hobby
spec:
  exportTo:
  - .
  host: my-nginx.mesh-external.svc.cluster.local
  trafficPolicy:
    loadBalancer:
      simple: ROUND_ROBIN
    portLevelSettings:
    - port:
        number: 31443
      tls:
        caCertificates: /etc/istio/egress-ca-certs/ca-chain.cert.pem
        clientCertificate: /etc/istio/egress-certs/tls.crt
        mode: MUTUAL
        privateKey: /etc/istio/egress-certs/tls.key
        sni: my-nginx.mesh-external.svc.cluster.local
EOF

3. Add external mtls service (ngnix)
kubectl create namespace mesh-external
kubectl create -n mesh-external secret tls nginx-server-certs --key my-nginx.mesh-external.svc.cluster.local.key --cert my-nginx.mesh-external.svc.cluster.local.crt
kubectl create -n mesh-external secret generic nginx-ca-certs --from-file=example.com.crt

$ cat <<\EOF > ./nginx.conf
events {
}

http {
  log_format main '$remote_addr - $remote_user [$time_local]  $status '
  '"$request" $body_bytes_sent "$http_referer" '
  '"$http_user_agent" "$http_x_forwarded_for"';
  access_log /var/log/nginx/access.log main;
  error_log  /var/log/nginx/error.log;

  server {
    listen 443 ssl;

    root /usr/share/nginx/html;
    index index.html;

    server_name my-nginx.mesh-external.svc.cluster.local;
    ssl_certificate /etc/nginx-server-certs/tls.crt;
    ssl_certificate_key /etc/nginx-server-certs/tls.key;
    ssl_client_certificate /etc/nginx-ca-certs/example.com.crt;
    ssl_verify_client on; # In mutual TLS, server verifies client's certificate
  }
}
EOF

kubectl create configmap nginx-configmap -n mesh-external --from-file=nginx.conf=./nginx.conf

kubectl apply -f - <<EOF
apiVersion: v1
kind: Service
metadata:
  name: my-nginx
  namespace: mesh-external
  labels:
    run: my-nginx
spec:
  ports:
  - port: 31443
    protocol: TCP
    targetPort: 443
  selector:
    run: my-nginx
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: my-nginx
  namespace: mesh-external
spec:
  selector:
    matchLabels:
      run: my-nginx
  replicas: 1
  template:
    metadata:
      labels:
        run: my-nginx
    spec:
      containers:
      - name: my-nginx
        image: nginx
        ports:
        - containerPort: 443
        volumeMounts:
        - name: nginx-config
          mountPath: /etc/nginx
          readOnly: true
        - name: nginx-server-certs
          mountPath: /etc/nginx-server-certs
          readOnly: true
        - name: nginx-ca-certs
          mountPath: /etc/nginx-ca-certs
          readOnly: true
      volumes:
      - name: nginx-config
        configMap:
          name: nginx-configmap
      - name: nginx-server-certs
        secret:
          secretName: nginx-server-certs
      - name: nginx-ca-certs
        secret:
          secretName: nginx-ca-certs
EOF

4. Verification Steps
The external mtls communication work correctly as below screenshot, but internal mtls can not work for ROOTCA is replaced by new CA assigned in sleep annotation.
$ kubectl -n hobby exec "$(kubectl -n hobby get pod -l app=sleep -o jsonpath={.items..metadata.name})" -c sleep -- curl -s http://my-nginx.mesh-external.svc.cluster.local:31443
<!DOCTYPE html>

<title>Welcome to nginx!</title>

Welcome to nginx!

...

The secret of ROOTCA is replaced by ca-chain.cert.pem , you can find the "NOT AFTER" and "NOT BEFORE" are the same now
$ istioctl pc secret sleep-747dcdbd6f-rbpkx.hobby
RESOURCE NAME                                                                 TYPE           STATUS     VALID CERT     SERIAL NUMBER                                        NOT AFTER                NOT BEFORE
file-cert:/etc/istio/egress-certs/tls.crt~/etc/istio/egress-certs/tls.key     Cert Chain     ACTIVE     true           1                                                    2021-10-20T02:49:14Z     2020-10-20T02:49:14Z
default                                                                       Cert Chain     ACTIVE     true<PHONE_NUMBER>28178672933431284968727571656              2020-10-21T07:57:35Z     2020-10-20T07:57:35Z
file-root:/etc/istio/egress-ca-certs/ca-chain.cert.pem                        CA             ACTIVE     true<PHONE_NUMBER>75612486678315663074644359785194352408     2021-10-20T02:48:52Z     2020-10-20T02:48:52Z
ROOTCA                                                                        CA             ACTIVE     true<PHONE_NUMBER>75612486678315663074644359785194352408     2021-10-20T02:48:52Z     2020-10-20T02:48:52Z

Version (include the output of istioctl version --remote and kubectl version --short and helm version if you used Helm)
$ istioctl version --remote
client version: 1.7.3
control plane version: 1.7.3
data plane version: 1.7.3 (4 proxies)
How was Istio installed?
istioctl install --profile=demo
Environment where bug was observed (cloud vendor, OS, etc)
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

And the cluster configuration is as below.
$ istioctl  pc cluster sleep-747dcdbd6f-rbpkx.hobby --port 31443 -ojson
[
    {
        "name": "outbound|31443||my-nginx.mesh-external.svc.cluster.local",
        "type": "ORIGINAL_DST",
        "connectTimeout": "10s",
        "lbPolicy": "CLUSTER_PROVIDED",
        "circuitBreakers": {
            "thresholds": [
                {
                    "maxConnections":<PHONE_NUMBER>,
                    "maxPendingRequests":<PHONE_NUMBER>,
                    "maxRequests":<PHONE_NUMBER>,
                    "maxRetries":<PHONE_NUMBER>
                }
            ]
        },
        "transportSocket": {
            "name": "envoy.transport_sockets.tls",
            "typedConfig": {
                "@type": "type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.UpstreamTlsContext",
                "commonTlsContext": {
                    "tlsCertificateSdsSecretConfigs": [
                        {
                            "name": "file-cert:/etc/istio/egress-certs/tls.crt~/etc/istio/egress-certs/tls.key",
                            "sdsConfig": {
                                "apiConfigSource": {
                                    "apiType": "GRPC",
                                    "transportApiVersion": "V3",
                                    "grpcServices": [
                                        {
                                            "envoyGrpc": {
                                                "clusterName": "sds-grpc"
                                            }
                                        }
                                    ]
                                },
                                "resourceApiVersion": "V3"
                            }
                        }
                    ],
                    "combinedValidationContext": {
                        "defaultValidationContext": {},
                        "validationContextSdsSecretConfig": {
                            "name": "file-root:/etc/istio/egress-ca-certs/ca-chain.cert.pem",
                            "sdsConfig": {
                                "apiConfigSource": {
                                    "apiType": "GRPC",
                                    "transportApiVersion": "V3",
                                    "grpcServices": [
                                        {
                                            "envoyGrpc": {
                                                "clusterName": "sds-grpc"
                                            }
                                        }
                                    ]
                                },
                                "resourceApiVersion": "V3"
                            }
                        }
                    }
                },
                "sni": "my-nginx.mesh-external.svc.cluster.local"
            }
        },
...

@howardjohn  @ramaraochavali @JimmyCYJ do you think if my issue is the same with https://github.com/istio/istio/issues/27218? Thanks

It seems so. Can u verify if that fixed your issue?

@ramaraochavali I have tried istio-1.8.0-alpha.0, and this issue gone, so I think #27218 fixed it. Thanks,

@hobbytp Thanks for the update. I will close this issue then.


[Improvement][API] Support to check if the worker group has been used by any tasks or schedulers when users delete or rename it.

Purpose of the pull request
This PR will close #14864 .

Brief change log
Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally.

The test results are as follows:

Codecov Report

Merging #14893 (910538b) into dev (121c981) will decrease coverage by 0.05%.
The diff coverage is 36.66%.

:exclamation: Current head 910538b differs from pull request most recent head 55ecf6a. Consider uploading reports for the commit 55ecf6a to get more accurate results

@@             Coverage Diff              @@
##                dev   #14893      +/-   ##
============================================
- Coverage     38.82%   38.78%   -0.05%
+ Complexity     4588     4585       -3
============================================
  Files          1232     1232
  Lines         43318    43283      -35
  Branches       4783     4771      -12
============================================
- Hits          16820    16789      -31
+ Misses        24639    24632       -7
- Partials       1859     1862       +3

Files Changed
Coverage Δ

...duler/api/service/impl/WorkerGroupServiceImpl.java
75.13% <29.62%> (-6.74%)
:arrow_down:

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <100.00%> (ø)

... and 49 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I see SonarCloud check and OWASP did not pass. Should I add more unit test? I think  OWASP is not about my pr.


However, trying to define a string with quotes instead of double quotes doesn't work

Hi,
I'm trying to use LARK as a search pattern engine for the given code and have to redefine a string with Quotes "'".
I'm using parser=lalr and lexer=contextual but I am not able to create a quoted separated string containing all rules to be ignored;
although the string is recognized, it is parsed, and a violation of a rule is recognized.
How can I define a quoted string so that all rules contained in it are ignored?
Thanks, Fried

Sorry, but I don't understand at all what you're asking. Try to include a code example.

If I understand correctly, a string terminal in Lark is defined with double quotes.
However, my Lark-built code handler for a given grammar requires the string as a quote definition;
any characters and rules should be ignored in '....bla.bla...rule(1)..rule(n)..' and
i.e. everything must be ignored in quotes-separated strings (no lexing & parsing).
I am going to build an extracted example for this where in my string definition Lark parses the string content.
Thanks, Fried

I have in my example to use the newly defined quoted-string '. any char or any rules ..'
to ignore the included content means no parsing of the quoted string e.g.
see in the example line 59 :
x:= '#+*()/&!"§$%&/()=?  select a into ' ;
that brings me to:
raise UnexpectedToken(token, expected, state=self, interactive_parser=None) lark.exceptions.UnexpectedToken: Unexpected token Token('INTO', 'into') at line 4, column 54. Expected one of:  	* QUOTE 	* __ANON_0 	* SEMICOLON 	* SYMBOLS__
How can I define the quoted string so that its content is not parsed?
Thanks, Fried

Look at how strings are defined in the python grammar: https://github.com/lark-parser/lark/blob/master/lark/grammars/python.lark#L284C26-L284C26
Any reason that you can't use this approach?

Great!


Seqvars Effects Filters missing fields

all the commented out entries are present in the design, but missing in the API. Manuel confirmed that these are still WIP.
Most of them seem to be present here:
https://github.com/varfish-org/varfish-server/blob/8d39e44c3d0d0265d353a5f4f17b52fd175755c8/backend/seqvars/models.py#L109-L208

Effect groups I also could not find in the API yet

duplicate of #1834


Step name not displayed correctly in log viewer when using node-version

Hi there! I have a basic workflow defined as below but within the Actions log viewer the step name is not correctly formatted when trying to substitute the node-version into it.
If this isn't the correct place to post such issues feel free to point me in the right direction. Really liking GitHub Actions so far though!!

name: continuous-integration

on: [push]

jobs:
  build:
    runs-on: ubuntu-latest
    strategy:
      matrix:
        node-version: [10.x]

    steps:
      - name: Checkout source code
        uses: actions/checkout@v1

      - name: Setup Node.js ${{ matrix.node-version }}
        uses: actions/setup-node@v1
        with:
          node-version: ${{ matrix.node-version }}

Duplicate of #46

Thanks @okuryu


Replace doesn't work with Enumerables

This code fails:
> Replace.Invoke { NonEmptyList.Head =  1; Tail =  [2] } { NonEmptyList.Head =  1; Tail =  [] } { NonEmptyList.Head =  100; Tail =  [200] } ;;

  Replace.Invoke { NonEmptyList.Head =  1; Tail =  [2] } { NonEmptyList.Head =  1; Tail =  [] } { NonEmptyList.Head =  100; Tail =  [200] } ;;
  ---------------^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

stdin(3,16): error FS0001: This expression was expected to have type
    'NonEmptyList<'a>'
but here has type
     'seq<'a>'

Note: NonEmptyList from F#+

Actually it will never work with any Enumerable but that's the correct behavior, because it requires also that the IEnumerable converts back from Seq for instance with the OfSeq method.
The NonEmptyList however can be fixed by adding a specific Replace instance which uses the knowledge that the resulting string will never be empty because the newValue parameter is also NonEmpty.
So the only thing to do in FsControl is to add a method with the right signature which operates using ToSeq and OfSeq.


Release v1.2.3

Description:
Sync with osm 1.2.3

Testing done:
DONE

Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Is this a breaking change? no

Has documentation corresponding to this change been updated in the osm-edge-docs repo (if applicable)? no

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-v1.2@7343384). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@               Coverage Diff               @@
##             release-v1.2     #135   +/-   ##
===============================================
  Coverage                ?   63.27%
===============================================
  Files                   ?      216
  Lines                   ?    16391
  Branches                ?        0
===============================================
  Hits                    ?    10371
  Misses                  ?     5968
  Partials                ?       52

Flag
Coverage Δ

unittests
63.27% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Incorrect lMask

Either I'm misunderstanding the masking process that happens in stereo_pprc, or the lMask that's being produced is incorrect. Can you confirm that this is not the intended behavior, and suggest what I might be doing wrong, or whether this is a bug?
*lMask.tif and *-L.tif screenshot from stereo_gui (seems correct):

*lMask.tif and *-L.tif screenshot from stereo_gui (seems incorrect):

As a result, I get spurious correlation in the output where there wasn't any input data (this is the DRG output from running point2dem with --orthoimage:

To reproduce:

Download M1210626920RE  and M1134059748RE
run spiceinit, lronaccal, lronacecho, cam2map4stereo.py on both images
run stereo on M1210626920RE and M1134059748RE with default settings

This does not look good but it is not unexpected. After the correlation happens, each pixel P in the left image is associated with a pixel Q in the right image. They both will be within their masks. However, when Q is the wrong point to associate with P, at triangulation stage you will get a 3D location that, when visualized, as you do, appears to be outside of the region covered by the images (and will have wrong height as well, but that you can't see in this picture).
We have some filters for dealing with this. point2dem should filter out the biggest triangulation errors (you can look at its params for doing that). There is also --erode-max-size in stereo filtering, and a bunch more options for that command. And in stereo_tri there is --near-universe-radius and --far-universe-radius.
You can also try the MGM algorithm --stereo-algorithm 2 (with parallel_stereo), which uses semi-global matching which should improve things, or at least the default --stereo-algorithm 0 with --subpixel-mode 3 with should do a better job at refinement.
You can also mapproject your images onto a smoother version of this DEM and redo stereo per the mapprojection chapter in the book. (Use parallel_stereo for that, and you can try with smaller clips.) That operation will unwarp some of the distortions due to the terrain being steep and will result in less wrong points.
I do agree that we could filter more even based on the picture you see, but it won't be just mask-based filtering, since, again, each of those 3D wrong points actually should project in both images, those points look sideways because point2dem "flattens" the cloud and a point that is deep down and a little sideways but still projecting into the cameas now looks just being to the side.

Thanks so much for the detailed reply!
I took a close look at my settings and realized that I was using alignmentmethod affineepipolar. Since I had already map-projected the images beforehand using cam2map4stereo.py, I figured I would try setting alignmentmethod to none. My output now has no stray points to speak of!
With alignmentmethod none, my left mask is blank... I guess that's normal?
I'm going to close this since the the resolution doesn't require any changes to the software -- feel free to reopen if you disagree.

I am glad it works. I think we could indeed improve our filtering. With alignment method none and mapprojection at some point I put some logic that does extra filtering (https://github.com/NeoGeographyToolkit/StereoPipeline/blob/master/src/asp/Tools/stereo_pprc.cc#L317) but I don't think that should have kicked in for you as you used cam2map rather than our own mapprojection.
Anyhow, noise is always an issue and we could do more to deal with it. Not sure what to say about the left mask. Maybe blank here means that all the pixels are valid. I don't recall now how that logic works.

After switching to MGM and looking into this further, I'm increasingly convinced that there's a problem in the implementation of masking.
My understanding is:

Masking happens before any correlation (and in my case, I'm not using ASP for alignment / projection)
The idea is to mask out pixels from one image that do not have corresponding pixels in the other image

My right mask makes sense, but my left mask is entirely blank (all 0 values).
Here's a video to show this, in QGIS. Sorry about the jerky cursor.

For now, I have implemented my own masking using image footprint shapefiles and gdalwarp -crop_to_cutline , as a workaround.
I think this might actually be a good first issue for me to try and fix this myself. I haven't been able to successfully compile ASP yet but I've gotten pretty close... maybe this weekend.

Seems the problem is in the normalization step. NoData in my input image becomes 0.0278 (greater than zero) in my stereo-L.tif file, whereas my NoData in my stereo-R.tif file the nodata becomes -0.091265 .

Adding the --individually-normalize option makes the problem go away.

The mask not necessarily masks pixels in one image that are missing in the second image. In fact, our masking is very light-weight, sometimes it does nothing at all, so accepts all pixels as valid. I don't recall the details now.
It is odd that your mask has only zeros only. I'd think it should have only ones sometimes, or at least mostly ones and some zeros.  Maybe if the --nodata-value in stereo is not set well it blanks the whole image. I am not sure. You can try running stereo with raw images, without cam2map, and see how it looks then. Anyhow, I don't recall running into empty masks before.

I think the problem is that pair normalization is failing to preserve nodata in some edge cases. I'll submit a PR.

Maybe it's the same as https://github.com/NeoGeographyToolkit/StereoPipeline/issues/290 ? By the way I'm trying to get ASP to compile... running into problems with boost when it's trying to link las2las from libLAS I think.

Compiling ASP is a pain. You can just turn off liblas and the tools in ASP
which depend on it, which consist, I hope, only of point2dem, point2las,
and pc_align. Hope you have no other problems. Likely you will.
On Mon, Apr 6, 2020 at 10:09 PM Aaron Curtis<EMAIL_ADDRESS>wrote:

Maybe it's the same as #290
https://github.com/NeoGeographyToolkit/StereoPipeline/issues/290 ? By
the way I'm trying to get ASP to compile... running into problems with
boost when it's trying to link las2las from libLAS I think.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NeoGeographyToolkit/StereoPipeline/issues/302#issuecomment-610176612,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKDU3E2MVRZM7KWIN2TP6TRLKYOXANCNFSM4LSL7M4Q
.

Haha ok. I'll probably just try the virtual machine approach.


When get the secrets if the secret contain a dollar (example$rest) sign only retrieve the secret until the dollar sign (example)

When I get a secret from the Vault Server using hiera_vault if the secret contains a dollar sign (exampe$more-characters) the value returned is until the dollar sign (example), I use a lot to render ini/config files.
I use puppet 6.x as code and make a lab with Puppet Server Open Source.
I try a lot of things like escaped the dollar signs but doesn't work!

Can you give me some example code of how this looks?

Hello Peter,
Sorry, I finally found a solution, the problem does not go through your
module, it was from a previous step that involved third-party encryption
software.
You can rule out this problem!
Thanks for following up on this issue!
Regards,
Fernando
El vie, 23 de jul. de 2021 a la(s) 10:40, Peter Souter (
@.***) escribió:

Can you give me some example code of how this looks?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/petems/petems-hiera_vault/issues/67#issuecomment-885490390,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFBW7RH6SFA6RNGKLN4VHOLTZETI3ANCNFSM45P64KJQ
.


Output file flag is not parssed correctly

On running executable while defining output file flag I do get error. Main branch. MacOS.
open_proxies -i ./socks.txt -o ./live.txt
error: The argument '--input <FILENAME>' cannot be used with '--out <FILENAME>'

Usage: open_proxies --input <FILENAME>

For more information try '--help'

open_proxies -i ./socks.txt -o ./live.txt

Thank you for rising this issue, I fixed it right away.
Fix: code version 0.1.1


add link to preprint in readme

The preprint link is found in the introduction of the readme (as a hyperlink). Let me know if it's too hard to find, or if you have ideas on making it more visible.


[WIP] kubernetes provider: add cross linking with azure

Adding an option to identify nodes hosted on Azure to try and cross link to their hosting VM.

@miq-bot add_label wip

Hi @enoodle
Can you update us on the status of this PR? Thanks.

@bronaghs I am trying to get access to an Azure provider to verify that this still works. I tested it with preliminary versions of Openshift  v1.3 but never fully so I am waiting on that to un-wip this. Sorry for the delay.

@enoodle - No problem. if you need to set up an account on the Azure portal the following instructions should help: https://github.com/ManageIQ/azure-armrest/wiki/Adding-a-Service-Principal

@loicavenel this seems an RFE (or anyway enhancement) that we never discussed.
The code is trivial but I suppose it's taking a long time because we don't have access to Azure and we don't have a test plan for QE.
What should we do about this? Purse (with the required resources) or abandon?

@simon3z ultimately, we need to have all supported CloudForms providers covered Amazon, GCE, Azure, Wmware, Openstack and Rhev. I think Azure is important to get in, we can plan it for 5.0, not getting it does not break the product.


A local .servicex file does not seem to be over-riding the .servicex file found in the home directory

If I put "localhost:5000" in the local file, it doesn't seem to pick it up - rather, it is found in the login directory.

This should be fixed by other testing. I think this was a pre- xAOD and uproot type thing in servicex.


[scope] Adding scope_exit

Adding https://github.com/davisking/dlib/discussions/2874

Weird the macos-latest runner keeps stalling...

Nice, that's great.


dlrs:Too many SOQL queries: 101

Error received which is hindering processing of other flows and not able to create opportunities.
org has 145 DLRS, what the best way to process so that we aren't running into this error? Should I set all to Scheduled and do I just need to create one scheduled job with referencing the apex class RollupJob?  currently in the org there are about 45 scheduled jobs all referencing the apex class RollupCalculateJobSchedule... Thanks!
BTW already read this but doesn't fully answer our issue https://github.com/afawcett/declarative-lookup-rollup-summaries/wiki/What-you-need-to-know-about-Scheduling-Rollups

Yes, the best way is to split some or all into schedule mode. The tool does try to reuse queries between rollups, but with that number and likely variance in criteria (which prevents the query reuse from working effectively) its hard for its optimization to be completely successful in all honesty.
I have updated the wiki page to clarify that schedule entries using the class RollupCalculateJobSchedulable relfect the use of Scheduled Full Calculate for a given rollup. You can review the Job name to determine the ID of the Rollup then enter this in the URL (later versions of the tools did make this is easier to reconcile by rollup unique name if specified).
If you already have a lot of scheduled jobs, yes, moving to Scheduled Full Calculate will not only use up a lot more of these slots, but (depending on the use of criteria) is quite expensive resource wise. I would consider using the Scheduled Incremental Calculate, as this uses only one slot and only recalcs based on the child records changing between runs.  Hope this helps.

Quick question of clarification!  So when you say variance in criteria you
only mean the relationship criteria (the WHERE in the SOQL it builds) not
in the aggregated field correct?
On Sun, Feb 4, 2018 at 9:23 AM, Andrew Fawcett<EMAIL_ADDRESS>wrote:

Yes, the best way is to split some or all into schedule mode. The tool
does try to reuse queries between rollups, but with that number and likely
variance in criteria (which prevents the query reuse from working
effectively) its hard for its optimization to be completely successful in
all honesty.
I have updated the wiki page to clarify that schedule entries using the
class RollupCalculateJobSchedulable relfect the use of Scheduled Full
Calculate
https://github.com/afawcett/declarative-lookup-rollup-summaries/wiki/What-you-need-to-know-about-Scheduling-Rollups#scheduled-full-calculate
for a given rollup. You can review the Job name to determine the ID of the
Rollup then enter this in the URL (later versions of the tools did make
this is easier to reconcile by rollup unique name if specified).
If you already have a lot of scheduled jobs, yes, moving to Scheduled
Full Calculate will not only use up a lot more of these slots, but
(depending on the use of criteria) is quite expensive resource wise. I
would consider using the Scheduled Incremental Calculate
https://github.com/afawcett/declarative-lookup-rollup-summaries/wiki/What-you-need-to-know-about-Scheduling-Rollups#scheduled-incremental-calculate,
as this uses only one slot and only recalcs based on the child records
changing between runs. Hope this helps.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/629#issuecomment-362923558,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APXKqku78OUHviKXurc0zrdNEWtUXiUHks5tReePgaJpZM4R2L2z
.

@DanDonin yes... mostly.... though combos like COUNT rollup, plus a CONCAT rollup will generate a new query as the code uses to entirely different approaches, since CONCAT is not native to SOQL. However, if the criteria match between a roll-up having COUNT and another having SUM the tool attempts to accomplish this with one query.

Thanks Andrew! For my client that has 154 DLRS with most child on the opportunity I have set everything to calculation mode scheduled and we aren’t getting SOQL errors anymore.
Best Regards,
Ashley

On Feb 4, 2018, at 9:46 AM, DanDonin<EMAIL_ADDRESS>wrote:
Fantastic thanks!
On Feb 4, 2018 9:45 AM, "Andrew Fawcett"<EMAIL_ADDRESS>wrote:

This information is used to match compatible rollups into one query...
https://github.com/afawcett/declarative-lookup-rollup-
summaries/blob/master/rolluptool/src/classes/RollupService.cls#L1259
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/629#issuecomment-362925083,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APXKqr5y8Ee6VYUKIqAGQkj8BLty6YeAks5tRezGgaJpZM4R2L2z
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/629#issuecomment-362925155, or mute the thread https://github.com/notifications/unsubscribe-auth/AIkhCHioUPkaDA9xxqidyTBrIvsQw8jyks5tRe0EgaJpZM4R2L2z.

@battarro  is it possible you have some recursion here? DLRS will do minimal queries for rollups in scheduled mode, just to check the mode of the rollup, hence the selectActiveByChildObject code is run. This runs once for each DLRS trigger invocation... if you grab the debug logs i would wager you see a lot of DLRS trigger calls... maybe due to recursion and/or non-bulkified inserts to the child object DLRS is monitoring? Maybe?

@ashleyleifer let me know if you want to re-open, will close for now.


anomaly score

Hi,
I'm using C++.
My question is, what is the return value of Score function? how can I detect a sample is anomaly or not based on the Score return value?
Thank you in advance.

in the original paper by the algorithm's creator it is quoted that:

Using the anomaly score s, we are able to make the following assessment:
• (a) if instances return s very close to 1, then they are deﬁnitely anomalies,
• (b)ifinstanceshavesmuchsmallerthan0.5,thenthey are quite safe to be regarded as normal instances, and
• (c) if all the instances return s ≈ 0.5, then the entire sample does not really have any distinct anomaly.

so I need something like this.

in the original paper by the algorithm's creator it is quoted that:

Using the anomaly score s, we are able to make the following assessment:
• (a) if instances return s very close to 1, then they are deﬁnitely anomalies,
• (b)ifinstanceshavesmuchsmallerthan0.5,thenthey are quite safe to be regarded as normal instances, and
• (c) if all the instances return s ≈ 0.5, then the entire sample does not really have any distinct anomaly.

so I need something like this.

I need to add this. Right now it's just returning the average path length.

So now it is not possible to detect whether a data is normal or it is abnormal?
if it is not, when do you think you can add this feature?

Right now, you'd have to draw an arbitrary line based on the path length. However, I can probably add this in the next few days.

Thanks a lot that will be great.
I'm waiting for it.

Added a NormalizedScore() method (in python, rust, and cpp).

Released in v.1.1 as "NormalizedScore".


Updated tooltip for Current Page button

This resolves #15967
Added a more descriptive tooltip for the "View URL from Current Page" button.

@Snuffleupagus oh sorry mb this is my first PR here so I messed up. I had run the tests and linting that was mentioned in the contribution guide but this sentence was not provided by the UX team. I just tested the function myself and came up with the sentence based on what I understood. Again, sorry for the trouble.

Thanks for the patch; however let's close this for now since it's not currently clear if/how this should be changed.


Bump build number

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.
Hoping to get linux builds this time...

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

The problem I'm ultimately hoping to solve is getting support for some version of cibots with linux and python 3.7.  None currently exists.

@conda-forge/cibots, could you merge this once CI has completed?  That should create the missing linux packages.


Remove publish rights for normal user

I noticed that normal users in a project have the "Publish to NFDI4Health GCHSH"-option. I think this option should be limited to project administrators and (maybe) asset gatekeepers.

Please specify what a "normal User" is for you. Now: If you have "edit" rights within project you are allowed to publish.

I am using the terminology from the user guide. Therefore, it is someone with edit rights.
I thought a bit further and noticed what is missing: a sole passive project member.
From my point of view, it makes sense if all relevant project members are added to the project as a person. This improves the networking of people. At the same time, it is not necessarily desirable for everyone to be able to edit the metadata.
In this context, people without an account also represent a “risk”. If they do register with the LDH, they have direct editing and publication rights.
I therefore have three suggestions:

Creation of the role of the passive project member.
If the NFDI4Health PI currently has the rights of a normal user, change these to passive, since it is a representative role (Issue #65).
The project admin could get access to a project-internal rights management. This could be used, for example, to set whether normal users are allowed to publish or whether changes made by normal users must first be approved by an asset gatekeeper before publication.

For now I found a work around to create passive users: Add them and set them as inactive.
However, this makes it look like the member has left the project.


Flask 1.1.2 accidentally ships with itsdangerous v2, hence ignores simplejson

Description
A deployment of our app suddenly stopped being able to (de)serialize Decimals.
An installation of Flask~=1.1.2 suddenly ships with itsdangerous version 2, which has been released yesterday.
However, this introduces the breaking change of not using the simplejson anymore, even if it's present.
This is most likely not intended, because the v1.1.2 docs of flask.json.dumps clearly dictates that simplejson will be used if present:
>>> from flask.json import dumps
>>> help(dumps)
Help on function dumps in module flask.json:

dumps(obj, app=None, **kwargs)
    Serialize ``obj`` to a JSON-formatted string. If there is an
    app context pushed, use the current app's configured encoder
    (:attr:`~flask.Flask.json_encoder`), or fall back to the default
    :class:`JSONEncoder`.

    Takes the same arguments as the built-in :func:`json.dumps`, and
    does some extra configuration based on the application. If the
    simplejson package is installed, it is preferred.

    :param obj: Object to serialize to JSON.
    :param app: App instance to use to configure the JSON encoder.
        Uses ``current_app`` if not given, and falls back to the default
        encoder when not in an app context.
    :param kwargs: Extra arguments passed to :func:`json.dumps`.

    .. versionchanged:: 1.0.3

        ``app`` can be passed directly, rather than requiring an app
        context for configuration.

Replication

install  Flask~=1.1.2 and simplejson~=3.11.1 (or any other version, really)
Try to deserialize a Decimal:

>>> from flask import Flask, jsonify
>>> from decimal import Decimal
>>> with Flask('test').app_context():
...     print(jsonify(foo=Decimal('20.00')))

Environment

Python version: 3.9.5
Flask version: 1.1.2
simplejson~=3.11.1 in requirements.txt

Fix
Change itsdangerous>=0.24 to itsdangerous>=0.24, <2.0 in the setup.py.

You should pin transitive requirements in your application. Tools like pip-compile make this task rather easy.

Sorry about that, I forgot that requirements.txt was meant to be the output of pip freeze. I didn't know about pip-compile.
However, the fact remains that the v1.12 docstring of flask.json.dumps now contains incorrect information.
If the itsdangerous requirement shall remains to be a lower bound, then the docstring should be fixed, shouldn't it?

Also, may I ask whether there is a specific reason against adding the upper bound? Just because I did something suboptimally does not mean it would be a bad thing for flask to assist me with slightly more conservative requirements.
If users of Flask~=1.12 for whatever reason desperately miss the improvements of itsdangerous~=2.0, then they can always upgrade to Flask 2.
I do realize it's additional work to commit things and add a patch release though, I'm just wondering whether that's the only reason.
Thanks for the quick response so far.

I do realize it's additional work to commit things and add a patch release though, I'm just wondering whether that's the only reason.

IIRC that's the main reason

This is the type of incompatibility you could get from any library upgrade, potentially. Which is why applications need to pin their transitive dependencies in general.

As with #4027, this means that all Flask users need to either upgrade to the newest major release within a few hours or know your internal policies on which transitive dependencies work correctly when they're following the (very important!) best practice of routinely updating their dependencies.
This level of implementation detail knowledge is probably unwise to rely on
It would be really great to get a Flask 1.1.3 that sets upper bounds on all of its dependencies.

#4043


Fix KINFree call

Pass [kmem] so KINFree gets pointer to pointer

Can we add this to the tests, so that we can be sure this doesn't crash again?

Bump - someone who knows these APIs better than me should review and merge.

Should be Ok. It is the same approach as in #29.


[Docs] add Chinese version of 2_new_data_model

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
This PR adds Chinese version of 2_new_data_model
Modification
This PR adds Chinese version of 2_new_data_model

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

There is something wrong about lint. Please refer to CONTRIBUTING.md and install the pre-commit to fix the code style.

[INFO] Installing environment for https://github.com/jumanjihouse/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: CalledProcessError: command: ('C:\\Ruby30-x64\\bin\\gem.CMD', 'install', '--no-document', '--no-format-executable', 'fake_gem__-0.0.0.gem')
return code: 1
expected return code: 0
stdout: (none)
stderr:
    ERROR:  While executing gem ... (Gem::DependencyResolutionError)
        conflicting dependencies rubocop (>= 0.68.1) and rubocop (= 0.85.1)
      Activated rubocop-0.85.1
      which does not match conflicting dependency (>= 0.68.1)

      Conflicting dependency chains:
        fake_gem__ (= 0.0.0), 0.0.0 activated, depends on
        rubocop (= 0.85.1), 0.85.1 activated

      versus:
        fake_gem__ (= 0.0.0), 0.0.0 activated, depends on
        rubocop-rspec (= 1.40.0), 1.40.0 activated, depends on
        rubocop (>= 0.68.1)

      Gems matching rubocop (>= 0.68.1):
        rubocop-1.18.3

Check the log at C:\Users\Administrator.DESKTOP-EASJP9H\.cache\pre-commit\pre-commit.log

I use the pre-commit on win10. And I install the config into hook. When i use the git commit, this error occurs. how can I  fix this problem?
many thanks!

[INFO] Installing environment for https://github.com/jumanjihouse/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: CalledProcessError: command: ('C:\\Ruby30-x64\\bin\\gem.CMD', 'install', '--no-document', '--no-format-executable', 'fake_gem__-0.0.0.gem')
return code: 1
expected return code: 0
stdout: (none)
stderr:
    ERROR:  While executing gem ... (Gem::DependencyResolutionError)
        conflicting dependencies rubocop (>= 0.68.1) and rubocop (= 0.85.1)
      Activated rubocop-0.85.1
      which does not match conflicting dependency (>= 0.68.1)

      Conflicting dependency chains:
        fake_gem__ (= 0.0.0), 0.0.0 activated, depends on
        rubocop (= 0.85.1), 0.85.1 activated

      versus:
        fake_gem__ (= 0.0.0), 0.0.0 activated, depends on
        rubocop-rspec (= 1.40.0), 1.40.0 activated, depends on
        rubocop (>= 0.68.1)

      Gems matching rubocop (>= 0.68.1):
        rubocop-1.18.3

Check the log at C:\Users\Administrator.DESKTOP-EASJP9H\.cache\pre-commit\pre-commit.log

I use the pre-commit on win10. And I install the config into hook. When i use the git commit, this error occurs. how can I fix this problem?
many thanks!

This is because the installation of ruby does not succeed. You might need to refer to this doc to install ruby, there is a section about it in the end. For windows environment, you might need fix more issue about it, you may try conda install -c conda-forge ruby first.
There are also two workarounds:

Use markdown lint tool in your IDE, for example, VSCode support markdown lint tool and you can use it to modify it in your IDE.
add '-n' in your command to skip the pre-commit hook and fix the issues in CI, which is not recommended as it waste the resources in CI.

[INFO] Installing environment for https://github.com/jumanjihouse/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: CalledProcessError: command: ('C:\\Ruby30-x64\\bin\\gem.CMD', 'install', '--no-document', '--no-format-executable', 'fake_gem__-0.0.0.gem')
return code: 1
expected return code: 0
stdout: (none)
stderr:
    ERROR:  While executing gem ... (Gem::DependencyResolutionError)
        conflicting dependencies rubocop (>= 0.68.1) and rubocop (= 0.85.1)
      Activated rubocop-0.85.1
      which does not match conflicting dependency (>= 0.68.1)

      Conflicting dependency chains:
        fake_gem__ (= 0.0.0), 0.0.0 activated, depends on
        rubocop (= 0.85.1), 0.85.1 activated

      versus:
        fake_gem__ (= 0.0.0), 0.0.0 activated, depends on
        rubocop-rspec (= 1.40.0), 1.40.0 activated, depends on
        rubocop (>= 0.68.1)

      Gems matching rubocop (>= 0.68.1):
        rubocop-1.18.3

Check the log at C:\Users\Administrator.DESKTOP-EASJP9H\.cache\pre-commit\pre-commit.log

I use the pre-commit on win10. And I install the config into hook. When i use the git commit, this error occurs. how can I fix this problem?
many thanks!

This is because the installation of ruby does not succeed. You might need to refer to this doc to install ruby, there is a section about it in the end. For windows environment, you might need fix more issue about it, you may try conda install -c conda-forge ruby first.
There are also two workarounds:

Use markdown lint tool in your IDE, for example, VSCode support markdown lint tool and you can use it to modify it in your IDE.
add '-n' in your command to skip the pre-commit hook and fix the issues in CI, which is not recommended as it waste the resources in CI.

Thanks for your advice. I fix it degrade the ruby3 to ruby2.

Thanks for your patience review. You guys really makes my day. Appreciate it!


chore: delete log file

:tada: This PR is included in version 2.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update docs for minifying chevrotain parsers with a serialized grammar

This is the work so far but it’s failing examples/parser tests for two reasons:

the tests use a shared cache and are (probably?) run in parallel, so clearing the cache in one test will cause other tests to fail.
the tests use the npm release of chevrotain and I can’t link my local package b/c the test command itself calls yarn.

I’ll pick it up again later, I just ran out of time for now.

I think the basic example for de-serialization and serialization should not be part of the minification example
because:

Minification  can be solved in other much easier ways (name mangling options)
Reduce complexity of a single solution
serialization can be used other purposes such as improving cold start performance.

@bd82 created a new example that only implements serialization/deserialization.

@brainkim Thanks! I will have a look tomorrow

You need to format the code using prettier...

This looks much better, a few small fixes and it would be ready for merge.

An optional bonus would be to commit the serialized JSON to source control so it can easily be inspected by users.

Thanks !