Sequence of Triggers needs to be revised for reliable extension behavior

Tests has indicated the Page level trigger is running in correct sequence. The correct sequence of Validation Trigger should be as follows from the first one to last one as follows.

Page - OnBeforeValidate Trigger
Table - OnValidate Trigger
Page - OnAfterValidate Trigger
Page - OnModifyRecord/OnInsertRecord Trigger
Table - OnModify/OnInsert Trigger

Can this order be implemented so that we can reliably use these triggers in logically. It will also really useful to add additional parameter to the event subscription such as "Skip Trigger" as VAR. Once "Skip Trigger" is set to TRUE in the event subscription function, system will not run any further trigger of it's kind.
For example if "Skip Trigger" is set to TRUE in Page - OnBeforeValidate Trigger, Table - OnValidate Trigger and Page - OnAfterValidate Trigger will not run, but page and table level Modify/Insert trigger will run. But the value needs to be set without running the trigger.
If "Skip Trigger" is set to TRUE in OnModifyRecord/OnInsertRecord Trigger in Page any of the existing OnModifyRecord/OnInsertRecord Trigger in page as well as OnModify/OnInsert Trigger in Table will not run. But record needs to be inserted/modified without running the trigger.
If "Skip Trigger" is set to TRUE in OnModify/OnInsert Trigger in Table is set to True any of the existing OnModify/OnInsert Trigger in Table will not run. But record needs to be inserted or modified without running the trigger.

Hi! Please take a minute to read the guidelines for this repository https://github.com/microsoft/AL and our collection of frequently asked questions https://github.com/Microsoft/AL/wiki/frequently-Asked-Questions .
As this repository is only for issues related to the AL compiler in latest developer preview environment for Dynamics 365 Business Central. We suggest that you open a support case or file a bug in Collaborate, to ensure that all Business Central users benefit from your catch as soon as possible.
To open a support case, you can:

Open a support request to CSS through the PartnerSource portal
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company

If you file the bug in Collaborate, remember to include steps to reproduce the issue, and the Business Central build number and country version you're using.


Charts: pop-up for sync refresh icon

Description:
Related to card: https://github.com/ruuvi/com.ruuvi.station/issues/675
Show refresh icon always even if GATT-sync is not available or when it's cloud sensor and cloud mode is being used. Bring icon visible immediately after sync is completed.
We will show popup every time when user taps on sync icon.
Title: Bluetooth download
Text: Local sensor data can be downloaded, when you're within its Bluetooth range.
Buttons: Cancel, Download
Title FI: Bluetooth-lataus
Text FI: Voit ladata anturin sisäisen historian ollessasi Bluetooth-kantaman sisällä.
Buttons FI: Peruuta, Lataa

QA passed


(discussion) Tasks: Configure Tasks vs. Customize Tasks

There are currently two ways to customize/configure tasks when the user has no tasks.json:

select a detected task and select the Customize action for this tasks. This creates a customized entry for the selected task in the task.json and the tasks.json file is opened.
select Task>Configure Tasks. This presents the user with a pick list for templates and a task.json is generated from a template. If there is already a tasks.json then it is opened directly.

We should try to unify the Customize and Configure flow. For example:

always use the term "customize"
Task>Configure/Customize Tasks shows a quick pick with two sections: 'templates' and 'detected tasks'. If there already is a task.json then the 'templates' section is not shown.
Run Task: when there is no tasks.json then it also offers the use to create a tasks.json from a template. Prompt the user with a prompt "create a task.json from a template/customize a detected tasks"  [not happy with this one yet]

I like the idea to make the wording more consistent a lot. Let me do this for June since this is easy. I currently only offer to create a tasks.json when you either explicitly ask for it or if you try to run a task and none is found. I will test the flow again for June.

For June I used configure in terms of naming:

I will look into merging the templates into the task selection dialog in July. It was simply to many code changes.

Moving to August due to vacation time.

Still outstanding... Worked on multi root workspaces in August.

The term we use consistently is no configure.
The task template picking got merged into the picker for 1.17. Closing the issue.


Memory Leak in VorbisTranscoder::Decode() @269

Hi,
TempFloatBuffer  doesn't need to be allocated here at all, it's assigned without being used in the subsequence function. Simply setting it to nullptr should fix it.
	// Transcoding int16 to float format
	{
		float* TempFloatBuffer = static_cast<float*>(FMemory::Malloc(DecodedData.PCMInfo.PCMNumOfFrames * NumOfChannels * 2 * sizeof(float)));
		int32 TempFloatSize;

		RAWTranscoder::TranscodeRAWData<int16, float>(Int16RAWBuffer, TempPCMDataSize, TempFloatBuffer, TempFloatSize);
		DecodedData.PCMInfo.PCMData = FBulkDataBuffer<uint8>(reinterpret_cast<uint8*>(TempFloatBuffer), TempFloatSize);
	}

Hi. Thanks a lot for letting me know! Will be fixed soon.

Fixed in 5b6d33202066e1b34c195c25cb1bd7ea23e0611c.


Add .NET 8 Instructions

Please describe the feature.
.NET 8 will be released in November 2023. Once generally available, update our instructions appropriately.

As of November 3, .NET 8 RC2 is the current version. Let's wait until .NET 8 is final to minimize potential rework.
Classifying as medium priority until .NET Conf 2023 starts November 14. Then it may become high priority.


Delete account roles - classic ROSA

Add the classic specification to the messages of the command.
Related: SDA-8599

@hunterkepley @gdbranco PTAL


dataset issue

Hi,
https://www.kaggle.com/datasets/jeanmidev/smart-meters-in-london?select=daily_dataset

But the data looks different in the code?

where does 'LCL-FullData\CC_LCL-FullData.csv' come from?
BR

The actual original dataset is based on https://data.london.gov.uk/dataset/smartmeter-energy-use-data-in-london-households (this is the original source of dataset), the dataset in kaggle is derived and worked upon from this original source.


Add option to specify the attrs over the static (such as link_section) in the newly added make_static macro

For ESP32 development, often I want to offload the large static buffers to PSRAM, instead of internal memory. The macro is really handy but doesn't work in these cases. Adding a base variant like
macro_rules! make_static {
    ($val:expr, $(#[$m:meta])*) => {{
        type T = impl ::core::marker::Sized;
        $(#[$m])*
        static STATIC_CELL: $crate::StaticCell<T> = $crate::StaticCell::new();
        let (x,) = unsafe { STATIC_CELL.uninit().write(($val,)) };
        x
    }};
}

would allow to specify the link_section to store it in external RAM bss and would solve this issue. What do you think?

@Dirbaio unfortunately ($(#[$m:meta])* $val:expr) causes a parsing ambiguity: local ambiguity when calling macro make_static: multiple parsing options: built-in NTs expr ('val') or 1 other option.
I could think of a couple of options to keep this interface:
#[macro_export]
macro_rules! make_static {
    ($(#[$m:meta])+ $val:expr) => {{
        type T = impl ::core::marker::Sized;
        $(#[$m])+
        static STATIC_CELL: $crate::StaticCell<T> = $crate::StaticCell::new();
        let (x,) = unsafe { STATIC_CELL.uninit().write(($val,)) };
        x
    }};
    ($val:expr) => {{
        type T = impl ::core::marker::Sized;
        static STATIC_CELL: $crate::StaticCell<T> = $crate::StaticCell::new();
        let (x,) = unsafe { STATIC_CELL.uninit().write(($val,)) };
        x
    }};
}

or
#[macro_export(local_inner_macros)]
macro_rules! make_static {
    ($(#[$m:meta])+ $val:expr) => (make_static!($val, $(#[$m])+));
    ($val:expr) => (make_static!($val, ));
    ($val:expr, $(#[$m:meta])*) => {{
        type T = impl ::core::marker::Sized;
        $(#[$m])*
        static STATIC_CELL: $crate::StaticCell<T> = $crate::StaticCell::new();
        let (x,) = unsafe { STATIC_CELL.uninit().write(($val,)) };
        x
    }};
}

or
#[macro_export]
macro_rules! make_static {
    ($(#[$m:meta])+ $val:expr) => ($crate::__priv_make_static!($val, $(#[$m])+));
    ($val:expr) => ($crate::__priv_make_static!($val, ));
}

#[macro_export]
macro_rules! __priv_make_static {
    ($val:expr, $(#[$m:meta])*) => {{
        type T = impl ::core::marker::Sized;
        $(#[$m])*
        static STATIC_CELL: $crate::StaticCell<T> = $crate::StaticCell::new();
        let (x,) = unsafe { STATIC_CELL.uninit().write(($val,)) };
        x
    }};
}

I don't like the code duplication in #1, and I don't like the "private" variant/macro in #2/#3. Could there be a better way?

oof... oh, well. That's not great! No strong preference on which is best, up to you.
Also, check that they work with 2 or more attrs. I suspect that using + instead of * resolves the ambiguity for 1 attr only.

perhaps putting the attrs after the expr wasn't so bad, after al... Up to you, i'm fine with either now, seeing how troublesome it is to put them before :)


taoup: 1.1.23 -> 1.21

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/taoup/versions.
meta.description for taoup is: Tao of Unix Programming (Ruby-powered ANSI colored fortunes)
meta.homepage for taoup is: https://github.com/globalcitizen/taoup
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.21 in filename of file in /nix/store/xl5ih7sbd50fjzgz3z367rw8403rhyab-taoup-1.21

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

taoup

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xl5ih7sbd50fjzgz3z367rw8403rhyab-taoup-1.21 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A taoup https://github.com/r-ryantm/nixpkgs/archive/600326df55ae56f37ce814e8336960c471691878.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/600326df55ae56f37ce814e8336960c471691878#taoup

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xl5ih7sbd50fjzgz3z367rw8403rhyab-taoup-1.21
ls -la /nix/store/xl5ih7sbd50fjzgz3z367rw8403rhyab-taoup-1.21/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

taoup

Maintainer pings
cc @zakame for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Rejected, as 1.1.23 is the actual latest version - see https://github.com/NixOS/nixpkgs/pull/350998


Checkout better enums

http://aantron.github.io/better-enums/
e.g. for keycodes

closed with #208


Update Bootstrap nav in index.html

Still need to format hamburger menu correctly.

index.html bootstrap added


man failure

Updated mandb man-db-2.11.2 today and now get this: The top image is what happens with a return and the bottom after entering q:

Tried with less, cp, and git with same results.  z/OS 2.5 under TN3270 OMVS entered from ISPF.

investigating

still investigating, but i deleted the bad builds on github, so if you a) delete all your man* files and then issue a new zopen install man-db you should be back in business.

done - after doing the delete had to update .bootenv to remove the for man as .bootenv broke not finding it.
Should .bootenv be updated when zopen install installs something?
And now I get this:

This is still an issue - but I can use man on the IBM provided man pages

this is via TN3270

It works fine via SSH


Login on windows is not working: non-zero custom status code considered fatal

What happened?
I'm developing a flutter app that uses realm with device sync and atlas user management using anonymous login or custom function credentials.
It is working perfectly on iOS, Android, MacOS (with exactly the same code) but it is failing for Windows. This is the error I get upon calling the login function (with either credentials):
AppException (AppException: non-zero custom status code considered fatal, link to server logs: null)

There is nothing in the logs of the Atlas app. Could you help me narrow down what the issue might be? Since I don't know what the underlying error is, it is almost impossible for me to start debugging.
Repro steps
Call App.logIn with anonymous credentials or custom function credentials on Windows 10.
Version
1.3.0
What Atlas Services are you using?
Both Atlas Device Sync and Atlas App Services
What type of application is this?
Flutter Application
Client OS and version
Windows 10
Code snippets
Either:
final app = App(AppConfiguration("<app>"));
final credentials = Credentials.anonymous();
final user = await app.logIn(credentials);

or:
final app = App(AppConfiguration("<app>"));
final credentials = Credentials.function(jsonEncode({
  "email": email,
  "password": password
}));
final user = await app.logIn(credentials);

Stacktrace of the exception/crash you're getting
Empty stack trace: _StringStackTrace

Relevant log output
flutter: 2023-08-05T17:55:13.721611: [ERROR] Realm: App: log_in_with_credentials failed: 0 message: non-zero custom status code considered fatal

AppException (AppException: non-zero custom status code considered fatal, link to server logs: null) with status code 999

Can I ask you to put a breakpoint on
        responseRef.custom_status_code = _CustomErrorCode.unknown.code;

in realm_core.dart (If you are using v1.3.0 it is line 1730).and report back what the exact exception it is that causes the unknown error code to be set.

Update:
I added a global interceptor for bad certificates (badCertificateCallback) with the flutter http client using HttpOverrides. The failing host is:
eu-central-1.aws.realm.mongodb.com

I then went on to verify the ssl certificate used by the host with this result (using https://www.geocerts.com/ssl-checker):

There seems to be a configuration error concerning the certificate chain. See also this thread talking about the same problem but not related to flutter:
https://community.letsencrypt.org/t/handshake-verification-error/61614

On macos 13.5 it resolves fine:
❯ gnutls-cli eu-central-1.aws.realm.mongodb.com
Processed 138 CA certificate(s).
Resolving 'eu-central-1.aws.realm.mongodb.com:443'...
Connecting to '<IP_ADDRESS>:443'...
- Certificate type: X.509
- Got a certificate list of 3 certificates.
- Certificate[0] info:
 - subject `CN=*.aws.realm.mongodb.com', issuer `CN=R3,O=Let's Encrypt,C=US', serial 0x0415a544e6686b98a3a4f1068226dbb907fe, RSA key 2048 bits, signed using RSA-SHA256, activated `2023-07-26 11:31:23 UTC', expires `2023-10-24 11:31:22 UTC', pin-sha256="CVaXq2/zpuX8BaHTwVmlMV728SayahkassVuF1dxIYQ="
        Public Key ID:
                sha1:16a8f104ea29796f4eda59186070c3f0c0cfc254
                sha256:095697ab6ff3a6e5fc05a1d3c159a5315ef6f126b26a191ab2c56e1757712184
        Public Key PIN:
                pin-sha256:CVaXq2/zpuX8BaHTwVmlMV728SayahkassVuF1dxIYQ=

- Certificate[1] info:
 - subject `CN=R3,O=Let's Encrypt,C=US', issuer `CN=ISRG Root X1,O=Internet Security Research Group,C=US', serial 0x00912b084acf0c18a753f6d62e25a75f5a, RSA key 2048 bits, signed using RSA-SHA256, activated `2020-09-04 00:00:00 UTC', expires `2025-09-15 16:00:00 UTC', pin-sha256="jQJTbIh0grw0/1TkHSumWb+Fs0Ggogr621gT3PvPKG0="
- Certificate[2] info:
 - subject `CN=ISRG Root X1,O=Internet Security Research Group,C=US', issuer `CN=DST Root CA X3,O=Digital Signature Trust Co.', serial 0x4001772137d4e942b8ee76aa3c640ab7, RSA key 4096 bits, signed using RSA-SHA256, activated `2021-01-20 19:14:03 UTC', expires `2024-09-30 18:14:03 UTC', pin-sha256="C5+lpZ7tcVwmwQIMcRtPbsQtWLABXhQzejna0wHFr8M="
- Status: The certificate is trusted.
- Description: (TLS1.3-X.509)-(ECDHE-X25519)-(RSA-PSS-RSAE-SHA256)-(AES-256-GCM)
- Session ID: FC:98:D3:39:7D:81:9A:E9:C8:78:FB:87:21:92:39:C5:5B:0F:B1:35:30:4A:96:2B:91:67:98:56:6B:1E:70:9E
- Options:
- Handshake was completed

Will investigate

Could you try loading https://letsencrypt.org/ in a browser on the windows box in question? It is signed with the same root certificate ISRG Root X1.
Windows does some shenanigans downloading installing certificates lazily, but I speculate that boringssl does not.

https://github.com/realm/realm-dart/issues/1087 relates to a somewhat similar issue with Android 7 and earlier, and how you can work around it.
The PR #1088 never landed, but it illustrates a work-around.

I think the most failsafe approach is to install the ISRG Root X1 root certificate on the SecurityContext of the HttpClient and try not to forget to update it before 2035. A lot can happen with the certificate chains before that anyway.
Other than that I guess you need to look into why and when Windows will automatically download a missing root certificate.

We will likely add it automatically on all platforms in the next version. Stay tuned.

I think the most failsafe approach is to install the ISRG Root X1 root certificate on the SecurityContext of the HttpClient and try not to forget to update it before 2035. A lot can happen with the certificate chains before that anyway.

This is exactly what I did now. Yes, I set a reminder to repeatedly check the situation about the certificates.
Thanks for your help.

Just checking to see how #1378 is progressing?

@fealebenpae that's more in your alley - can you take a look and see if the approach in #1378 is what we want to do?

I encountered this error after i did a password reset using bcrypt coded in nextjs and tried logging on windows on flutter side. But logging with plain passwords work.


[FLINK-8331][core] FieldParser do not correctly set EMPT_COLUMN error state.

Thank you very much for contributing to Apache Flink - we are happy that you want to help us improve Flink. To help the community review your contribution in the best possible way, please go through the checklist below, which will get the contribution into a shape in which it can be best reviewed.
Please understand that we do not do this to make contributions to Flink a hassle. In order to uphold a high standard of quality for code contributions, while at the same time managing a large number of contributions, we need contributors to prepare the contributions well, and give reviewers enough contextual information for the review. Please also understand that contributions that do not follow this guide will take longer to review and thus typically be picked up with lower priority by the community.
What is the purpose of the change
(In this PR fixed FieldParser do not correctly set EMPT_COLUMN error state bug, i.e. All FieldParser correctly return the EMPTY_COLUMN error state in case of an empty field.)
Brief change log

*Change nextStringEndPos method set EMPT_COLUMN error state logic.

Verifying this change
This change added tests and can be verified as follows:

Add a testNextStringEndPos test case in FieldParserTest.
Add a testNullValueWithoutTrailingDelimiter test casle in RowCsvInputFormatTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@fhueske Thanks for the review.
I have updated the PR according your comments. And I have extended the RowCsvInputFormatTest.testTailingEmptyFields method for DOUBLE/FLOAT/BIGINT/BIGDEC/SQL_DATE/SQL_TIME/SQL_TIMESTAMP type. Please have a look at the change, And feel free to comments if there is any inappropriate logic.
Thanks, Jincheng

@fhueske thanks for your explanation, make sense to me. I updated the PR. The changes as follows:

Revert the change of RowCsvInputFormatTest.testTailingEmptyFields.
Unity parseField behavior when startPos = limit, except StringParser,  that is StringParser set error state EMPTY_COLUMN and set empty string to result,return limit value as index, and other set error state EMPTY_COLUMN, return -1 as index;

Best, Jincheng


Cannot find module '@ngneat/elf-ng-router-store'

Is this a regression?
No
Description
Hi Iam new to state management with Elf and would like to use your plugin because it perfectly fits my requirements of reacting to changes in the router
Steps to reproduce:

create a new Angular project using angular cli
install elf using npx npx @ngneat/elf-cli install

I installed @ngneat/elf, @ngneat/elf-entities and @ngneat/elf-devtools. No external packages

install elf-ng-router-store by running npm install @ngneat/elf-ng-router-store
open app.module and add the following line

import { ElfNgRouterStoreModule } from '@elf/ng-router-store'; (from documentation)
this gives me the error: Cannot find module '@elf/ng-router-store' or its corresponding type declarations.

I think the import statement is not correct (so to say the reference in the documentation is not correct) but when I try to change the import to import { ElfNgRouterStoreModule } from '@ngneat/elf-ng-router-store'; (which I think is the correct import) typescript still cant find the module (same error as given above)
When I inspect my node_modules under @ngneat/elf-ng-router-store it displays only a package.json which could be the mistake.

Is this a bug or am I doing something wrong?
Nonetheless thanks for your work
Please provide a link to a minimal reproduction of the bug
see description above
Please provide the exception or error you saw
Cannot find module '@elf/ng-router-store' or its corresponding type declarations.

Please provide the environment you discovered this bug in
Angular 12
Typescript 4.2.3
@ngneat/elf-ng-router-store 1.0.1

Anything else?
No response
Do you want to create a pull request?
No

@yackinn can you please check it?

Yes, I'll check.

@ForzaMark
You're correct. The import should include elf.
import { RouterRepository } from '@ngneat/elf-ng-router-store'

The build seems to be broken and the NPM install command is only installing package.json and nothing else. That's why the below error is coming up;
Cannot find module '@elf/ng-router-store' or its corresponding type declarations._
Thanks for checking this.

The build process seems to be failing and the npm install is only installing an empty folder with just package.json. Nothing else is installed.
That's why the import is not finding the repository
import { RouterRepository } from '@ngneat/elf-ng-router-store'
Could you please check the build process? Thanks

Try the latest version

All good with the new version. Thanks a ton.


Add all plugins to MultiContextDeserializer

I've added two new plugins: Hadoop and GraphChi, and I've also uncommented a previously mentioned plugin. Since I'm new to open source, please let me know if I've missed anything.

Also, the headers are mandatory to all files so you should not have removed it :)

Hi, I apologize for my impatience. I'll make sure to address the bugs and mistakes I've introduced, and I'll be more mindful of testing in the future. Thank you for your patience and assistance.

Hey @Aryant01,
Great to see you here!
We know it's overwhelming to start working on open source projects, we've all been there! That's why we've put together information to make it easier to contribute => https://wayang.apache.org/docs/community/contribute
We also encourage contributors (yes, you're one now 👍🏾) to join the public mailing lists => https://wayang.apache.org/docs/community/mailinglist.
Keep up the great work and thank you,
@2pk03

Hey @2pk03 ,
Thankyou for your concern. I really appreciate it. I will follow the contribution guide and will be joining the mailing lists.
Thankyou.

Hi @Aryant01, thank you for your question. I believe it's missing some implementation so that Flink works with the multicontext. Could you try uncommenting that line and check if everything runs correctly?


Fix for JSON parser being unable to handle some floats.

I encountered this problem: A JSON file was created with a value in it like 0.6666666666666666. This results in an exception because some of the intermediate values cannot be stored as floats. I propose that the best solution for reliably parsing floats that works in ALL cases is to use existing code for parsing floats. I think that the attached code will fix this (plus it adds a test case to illustrate the original error).

Can one of the admins verify this patch? (see README)

This probably also fixes https://github.com/ceylon/ceylon-sdk/issues/296

I'll leave this for @chochos to merge

Huh, I get this failure now in JS tests:
[ceylon-test-js] test.ceylon.json::testParse - failure (ceylon.test.engine::AssertionComparisonError "assertion failed: expected <0.66666666666666> but was <0.6666666666666666>")
(two extra sixes)

Looks like parseFloat has reduced precision on JS due to its smaller maxIntegerValue.
Maybe parseFloat should be native? Or perhaps it would be safe to just set maximumIntegerExponent to 17 to recover some of the precision?

You were supposed to to test it before merging ;)

I think making maximumIntegerExponent native should be enough; it only works for JVM

Wait: I really doubt we got that wrong after all the attention we put into it. Remember that on JS you can see values that look like exact integers but actually  aren't!
Sent from my iPhone

On 28 Mar 2016, at 20:58, Enrique Zamudio<EMAIL_ADDRESS>wrote:
I think making maximumIntegerExponent native should be enough; it only works for JVM
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

I.e. The last couple of digits can be subject to rounding error.
Sent from my iPhone

On 28 Mar 2016, at 20:58, Enrique Zamudio<EMAIL_ADDRESS>wrote:
I think making maximumIntegerExponent native should be enough; it only works for JVM
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

On JS, the determination of maximumIntegerExponent seems overly conservative. Just because values > 2^53 can't all be exactly represented, it doesn't mean that they're useless.

@jvasileff well the problem is that the process of parsing them could produce a different value to the value represented by the string being parsed.

If we truncate the least significant digits, we will, say, replace a 7 with a 0. Without less aggressive truncation, the 7 may become a 6, but that's a lot better than 0. I'd be surprised if less aggressive truncation ever produced worse results.
However... the modified parseFloat on master is broken, since parseFloat uses parseInteger, and parseInteger returns null on overflow.
As a solution, I still think we should use a native parsing function. I'm guessing it will be more accurate and orders of magnitude faster on JS.

However... the modified parseFloat on master is broken,

WDYM broken?

You mean: refuses to do something that's not well-defined, don't you?

It returns null for 0.999999999999999999, now that maximumIntegerExponent is 17.

An IEEE double has 15–17 decimal digits of precision. Realistically, you can only count on 15.

I don't see your point. My points are:

we shouldn't return null for that input
we should strive for maximum precision/best results
if there is a way to have better precision and much better performance, we should take it.

Realistically, you can only count on 15.

So the test is broken then? We shouldn't expect 17 digits to work?

I don't see why we should expect/accept platform differences. On both, the conversion is from a String to a 64-bit IEEE 754 value.


Memory leak due to reloading config in DB-less mode

Summary
I am currently using kong in db-less mode with kong-ingress-controller. there is a limit of 1.5GB of ram (not sure if it is enough) for each kong proxy. there are 3 in total (I have also tried with a single replica and with 2, with the same result).
I think it is related to https://github.com/Kong/kong/issues/6055, this change is not found in master. https://github.com/Kong/kong/commit/0eb15e65e0290967341108e3052e69462f1f0b81
it always starts with 180-200mb and with every update from ingress-controller it increases constantly.

I have configured these parameters in kong-proxy.
- name: KONG_NGINX_MAIN_WORKER_RLIMIT_NOFILE
  value: '4096'
- name: KONG_NGINX_EVENTS_WORKER_CONNECTIONS
  value: '4096'
- name: KONG_WORKER_STATE_UPDATE_FREQUENCY
  value: '3600'
- name: KONG_WORKER_CONSISTENCY
  value: eventual

in ingress-controller I don't know if they work, but the behavior has improved.
- name: CONTROLLER_SYNC_PERIOD
  value: 3600s
- name: CONTROLLER_SYNC_RATE_LIMIT
  value: '0.9'

now it takes 24 hours to reach the memory limit(before it took 12 hours)
Steps To Reproduce
1.run kong in db-less mode
docker run --rm --name kong --net=host \
-e "KONG_ADMIN_ACCESS_LOG=/dev/stdout" \
-e "KONG_ADMIN_ERROR_LOG=/dev/stderr" \
-e "KONG_ADMIN_GUI_ACCESS_LOG=/dev/stdout" \
-e "KONG_ADMIN_GUI_ERROR_LOG=/dev/stderr" \
-e "KONG_ADMIN_LISTEN=<IP_ADDRESS>:8444 http2 ssl" \
-e "KONG_CLUSTER_LISTEN=off" \
-e "KONG_DATABASE=off" \
-e "KONG_KIC=on" \
-e "KONG_LUA_PACKAGE_PATH=/opt/?.lua;/opt/?/init.lua;;" \
-e "KONG_NGINX_WORKER_PROCESSES=1" \
-e "KONG_PLUGINS=bundled" \
-e "KONG_PORTAL_API_ACCESS_LOG=/dev/stdout" \
-e "KONG_PORTAL_API_ERROR_LOG=/dev/stderr" \
-e "KONG_PORT_MAPS=80:8000, 443:8443" \
-e "KONG_PROXY_ACCESS_LOG=/dev/stdout" \
-e "KONG_PROXY_ERROR_LOG=/dev/stderr" \
-e "KONG_PROXY_LISTEN=<IP_ADDRESS>:8000, <IP_ADDRESS>:8443 http2 ssl" \
-e "KONG_STATUS_LISTEN=<IP_ADDRESS>:8100" \
-e "KONG_STREAM_LISTEN=off" \
-e "KONG_LOG_LEVEL=debug" \
-e "KONG_NGINX_DAEMON=off" \
-e "KONG_TRUSTED_IPS=<IP_ADDRESS>/0,::/0" \
kong:2.2.0

Create a kong.yaml with the content of this gist. (this configuration is taken from https://localhost:8444/config in prod, just change usernames and api-keys, namespaces and domain)
run this command

for i in $(seq 1 1000)
do
   curl -k --http1.1 -s -o /dev/null  -X POST 'https://localhost:8444/config' --form<EMAIL_ADDRESS>   date && docker exec kong ps -o pid,rss,comm,args
   sleep 10
done

result with empty config
date && docker exec kong ps -o pid,rss,comm,args

Thu 05 Nov 2020 02:37:41 PM -05
PID   RSS  COMMAND          COMMAND
    1  30m nginx            nginx: master process /usr/local/openresty/nginx/sbin/nginx -p /usr/local/kong -c nginx.conf
   21  86m nginx            nginx: worker process
   22    4 ps               ps -o pid,rss,comm,args

first request with config
Thu 05 Nov 2020 02:38:06 PM -05
PID   RSS  COMMAND          COMMAND
    1  30m nginx            nginx: master process /usr/local/openresty/nginx/sbin/nginx -p /usr/local/kong -c nginx.conf
   21 168m nginx            nginx: worker process
   27    4 ps               ps -o pid,rss,comm,args

1 hour after
Thu 05 Nov 2020 03:38:50 PM -05
PID   RSS  COMMAND          COMMAND
    1  29m nginx            nginx: master process /usr/local/openresty/nginx/sbin/nginx -p /usr/local/kong -c nginx.conf
   21 1.6g nginx            nginx: worker process
 1675    4 ps               ps -o pid,rss,comm,args

I only make this request, I do not make any request to port 8000.
request to status
curl -k --http1.1 -vvv  'https://localhost:8444/status'

response: kong_status.json
"workers_lua_vms":[
         {
            "http_allocated_gc":"71.22 MiB",
            "pid":21
         }
      ]

Additional Details & Logs

Kong version 2.2.0
Kong debug-level startup logs kong.logs
Kong error logs
no errors
Kong configuration kong.json
Kong k8s deployment kong_deployment.yaml
kong status 1 hour after starting: kong_status.json
Operating system
linux mint 20(ubuntu 20.04).  in production it runs on amazon EKS. kubernetes 1.17.
but the behavior is similar in local, I have also tried 2.1.0, 2.1.1, 2.1.2, 2.1.4 and the same behavior.

Yes, I can reproduce this with latest next using:
for i in $(seq 1 1000)
do
  curl -k -s -o /dev/null --http1.1 -X POST 'https://localhost:8444/config' --form<EMAIL_ADDRESS>  ps -Ao pid,rss,comm,args | grep nginx
done

And I started my local Kong with:
KONG_NGINX_MAIN_WORKER_RLIMIT_NOFILE=4096 \
KONG_NGINX_EVENTS_WORKER_CONNECTIONS=4096 \
KONG_WORKER_STATE_UPDATE_FREQUENCY=3600 \
KONG_WORKER_CONSISTENCY=eventual \
KONG_ADMIN_LISTEN="<IP_ADDRESS>:8444 http2 ssl" \
KONG_PROXY_LISTEN="<IP_ADDRESS>:8000, <IP_ADDRESS>:8443 http2 ssl" \
KONG_STATUS_LISTEN="<IP_ADDRESS>:8100" \
KONG_CLUSTER_LISTEN=off \
KONG_STREAM_LISTEN=off \
KONG_DATABASE=off \
KONG_NGINX_MAIN_WORKER_PROCESSES=1 \
KONG_TRUSTED_IPS="<IP_ADDRESS>/0,::/0" \
KONG_PLUGINS=bundled \
./bin/kong start

Initial investigation shows that pending timers grow. This is probably a leak in timers for DNS client and/or healthchecks.

Yes, I can reproduce this with latest next using:
for i in $(seq 1 1000)
do
  curl -k -s -o /dev/null --http1.1 -X POST 'https://localhost:8444/config' --form<EMAIL_ADDRESS>  ps -Ao pid,rss,comm,args | grep nginx
done

And I started my local Kong with:
KONG_NGINX_MAIN_WORKER_RLIMIT_NOFILE=4096 \
KONG_NGINX_EVENTS_WORKER_CONNECTIONS=4096 \
KONG_WORKER_STATE_UPDATE_FREQUENCY=3600 \
KONG_WORKER_CONSISTENCY=eventual \
KONG_ADMIN_LISTEN="<IP_ADDRESS>:8444 http2 ssl" \
KONG_PROXY_LISTEN="<IP_ADDRESS>:8000, <IP_ADDRESS>:8443 http2 ssl" \
KONG_STATUS_LISTEN="<IP_ADDRESS>:8100" \
KONG_CLUSTER_LISTEN=off \
KONG_STREAM_LISTEN=off \
KONG_DATABASE=off \
KONG_NGINX_MAIN_WORKER_PROCESSES=1 \
KONG_TRUSTED_IPS="<IP_ADDRESS>/0,::/0" \
KONG_PLUGINS=bundled \
./bin/kong start

Initial investigation shows that pending timers grow. This is probably a leak in timers for DNS client and/or healthchecks.

The timer leak is gone if I remove upstreams and targets from yaml, but memory leak seems still be present even then.

The timer leak is gone if I remove upstreams and targets from yaml, but memory leak seems still be present even then.

I have found the place where this occurs, and this is the line:
https://github.com/Kong/kong/blob/next/kong/db/schema/init.lua#L886
The starting point of the leak is this:
https://github.com/Kong/kong/blob/next/kong/db/schema/others/declarative_config.lua#L628

I have found the place where this occurs, and this is the line:
https://github.com/Kong/kong/blob/next/kong/db/schema/init.lua#L886
The starting point of the leak is this:
https://github.com/Kong/kong/blob/next/kong/db/schema/others/declarative_config.lua#L628


Add ResponseBody factory overload for ByteString

We shouldn't punish people with a roundtrip through byte[] for using a superior type!

We already have a RequestBody one for symmetry. Phew!

Yep. That was the first thing I checked!
On Thu, Jul 5, 2018 at 8:44 PM Jesse Wilson<EMAIL_ADDRESS>wrote:

We already have a RequestBody one for symmetry. Phew!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/pull/4115#issuecomment-402891850, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEEESc2i9TZtKeyZnu1fNLki6RbOMrfks5uDrLmgaJpZM4VEtwr
.


Hot Reload Authentication settings

The authorization settings are configured during startup. This means that simply refreshing the RuntimeConfig to reflect saved changes is not enough to update the authorization settings that the service will be using. Instead, those changes that need to be made must be done so in a way that refreshes the settings that were configured during startup.

Moving this + hot reload related tasks to post GA -> tracking via 1.2rc milestone.

Startup.cs ConfigureAuthentication() function requires refactoring to fulfill this task.
All possible authentication schemes (JWT, SWA, AppService, Simulator) need to be configured regardless of what contents exist in the runtime configuration.
e.g.
services.AddAuthentication(defaultScheme: EasyAuthType.StaticWebApp)
     .AddJwtBearer(options =>
     {
          options.Property = "<IOptionsMonitor?>"
      });
    .AddEasyAuthAuthentication(easyAuthAuthenticationProvider: EasyAuthType.StaticWebApp)
    .AddEasyAuthAuthentication(easyAuthAuthenticationProvider: EasyAuthType.AppService);
    // condition Simulator auth for development mode ONLY
    .AddSimulatorAuthentication();

Requirements to figure out:

AddJwtBearer supports IOptionsMonitor, need to figure out how to signal changes due to hot reload because hot reloading different jwt providers requires refreshing the jwtprovider options.
Authenticate requests using the currently configured authentication configuration, this requires modifications the ClientRoleHeaderAuthenticationMiddleware

    public async Task InvokeAsync(HttpContext httpContext)
    {
        // authNResult will be one of:
        // 1. Succeeded - Authenticated
        // 2. Failure - Token issue
        // 3. None - No token provided, no auth result.
        AuthenticateResult authNResult = await httpContext.AuthenticateAsync(); // no param to AuthenticateAsync() uses default authentication scheme (which per above config is set as: services.AddAuthentication(defaultScheme: EasyAuthType.StaticWebApp))

The following method signature should be used instead:
` ``csharp
public static Task AuthenticateAsync(this HttpContext context, string? scheme);
Depending on the configured authentication provider, supply the appropriate scheme to `AuthenticateAsync()`.


feat(payment): PAYPAL-3447 Allow editing of Billing Address for eWallet Buttons flow

What?
Allow editing of Billing Address for eWallet Buttons flow
Why?
Because of PayPal is not returning all required billing information for buyer accounts and hence upon order placement shoppers receive an error as they can’t return to the billing step and edit the billing information
Testing / Proof

@bigcommerce/team-checkout

PayPal is not returning all required billing information for buyer accounts and hence upon order placement shoppers receive an error as they can’t return to the billing step and edit the billing information

Can we upload a video to show the PayPal button e2e workflow after the change?

PayPal is not returning all required billing information for buyer accounts and hence upon order placement shoppers receive an error as they can’t return to the billing step and edit the billing information

Can we upload a video to show the PayPal button e2e workflow after the change?

@bc-peng attached screen recording of the PayPal button workflow:
https://github.com/bigcommerce/checkout-js/assets/99336044/b824c18f-87fb-42a3-9c33-623546b0d95c


Keys remain held down when using remap a shortcut to paste as plain text feature

Microsoft PowerToys version
0.79.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce

Enable Paste As Plain Text keyboard shortcut
Keyboard Manager > Shortcuts > Remap a shortcut
Map a key in an app specific context to the paste as plain text shortcut from step 1 (ex. ctrl + shift + f becomes Win + Ctrl + alt + v in Outlook)
Do the mapped keyboard shortcut in the customized app (ex. ctrl + shift + f in outlook)
Type after the paste happens

✔ Expected Behavior
Content is pasted and plain text and all shortcut keys are released.
❌ Actual Behavior
Some of the shortcut keys (ex. ctrl, shift, alt) remain held down. I've tried different shortcut mappings and it seems like it repros 100% of the time whenever a remapped shortcut maps to the global paste as plain text shortcut.
Other Software
N/A

I came here to report this same issue, however I have not remapped the shortcut for Paste as Plain Text. It is still Win-Ctrl-Alt-V for me and after I paste, it appears that the Alt key is still software-locked on, and stays on until I hit it two or three times.


Fix #6459

Fixes #6459

I'm sceptical of this change - it feels like too much special casing.
It'd be nice if the PR title was more descriptive (e.g. this one could be "fix reference counting for manually called Py_DECREF" (just so it's obvious what it is when looking at a list of PRs/notifications)

I think the question should be "why does a pointer cast to object need to go through a temp?" (Although noting that a non-pointer cast to object probably does)

I'm sceptical of this change - it feels like too much special casing.

Calling INCREF and DECREF is pretty common in Cython though. And I don't see anything else this could apply to, so it couldn't be generalized.

It'd be nice if the PR title was more descriptive (e.g. this one could be "fix reference counting for manually called Py_DECREF" (just so it's obvious what it is when looking at a list of PRs/notifications)

Done.

A few more comments having had a look into this:
The argument is coerced into a temp because it fails this check
https://github.com/cython/cython/blob/a3086ff75f5b33639b867ab15922d928e14e99bf/Cython/Compiler/ExprNodes.py#L6250-L6260.
The reason it fails the "non-locally immutable" check is because it's an attribute lookup rather a pure local variable. i.e in your example
local = node.items[itr].value
Py_DECREF(<object>local)

wouldn't generate the extra code.
Coercing it into a temp and then adding a special mechanism to not reference count that temp is definitely the wrong thing to do.  A better fix would skip the coercion to temp.
The current mechanism definitely isn't getting everything right. And is very very specific to the SimpleCallNode class only (so misses similar errors in other cases). However, in this case I think it's doing as well as it can do without being able to see into Py_DECREF.

My view remains that a hard-coded list of function names is too messy and too much effort for a minor optimization failure that generates the right overall behaviour.

Or possibly handle it in Optimize.py which is where we tend to special-case functions? Probably just by stripping the arguement out of the coerce-to-temp


Cleanup includes.

This Cl cleans up the include paths to be realtive to the top level
directory. Various include-what-you-use fixes have been added.

Looks like the android makefile is right and cmake is wrong, but I don't know how ot remove the include/ directory from the -I lines in cmake? I'll fixup the issues found by android when I get a chance.

Actually, I think going the other way makes sense and leaving include/ in the -I's.  Will go that route instead.

Done, let's see if that works better on Android.


using Counters.Counter

Hi,
When using OpenZeppelin's counters, scribble is unable to understand that for example, current() is a function and says Counters.Counter doesn't have property current.
using Counters for Counters.Counter;
Counters.Counter private _tokenIdCounter;

For example:
///#if_succeeds old(tokenCounter.current()) + 1 == tokenCounter.current();
Can we look into this?
Thank you

Hello there. First of all, I guess tokenCounter should be _tokenIdCounter in the annotation.
I managed to reproduced the issue anyway with the following complete sample:
library Counters {
    struct Counter {
        uint256 _value;
    }

    function current(Counter storage counter) internal view returns (uint256) {
        return counter._value;
    }

    function increment(Counter storage counter) internal {
        unchecked {
            counter._value += 1;
        }
    }
}

contract Test {
    using Counters for Counters.Counter;

    Counters.Counter private _tokenIdCounter;

    ///#if_succeeds old(_tokenIdCounter.current()) + 1 == _tokenIdCounter.current();
    function doSomething() public {}
}

Error message is following:
sample.sol:22:24 TypeError: Struct Counters.Counter doesn't have a field current
sample.sol:22:24:
    ///#if_succeeds old(_tokenIdCounter.current()) + 1 == _tokenIdCounter.current();
                        ^^^^^^^^^^^^^^^^^^^^^^^^^

I've created PR to fix the issue: #177

@efecarranza Please, check new version of Scribble: https://github.com/ConsenSys/scribble/releases/tag/v0.6.10. The bug should be fixed. Thanks for the collaboration.


Synchronise maps

If possible, synchronize the two Leaflet maps

Due to the layout changes I don't think this would be required anymore.


Query - Is contextual-ai library shap and lime tested on GPU environment ?

Hi ,
Is contextual-ai library shap and lime tested on GPU environment ?  If yes, adding more GPU's will it improve the performance

Hi @manduripramodh, for now the shap and lime are mainly ran on CPU only as the library doesn't provide GPU implementation yet. Feel free to folk and contribute a GPU implementation or any parallel-based implementation to boost the performance.


Gecko integration broken after Home Assistant patch

Home assistant:
Core 2024.7.4
Supervisor 2024.06.2
Operating System 12.4
Frontend 20240710.0
Gecko 0.1.9
Log:
Logger: homeassistant.util.package
Source: util/package.py:123
First occurred: 8:02:44 PM (3 occurrences)
Last logged: 8:02:51 PM
Unable to install package geckolib==0.4.8: WARNING: Ignoring version 0.4.8 of geckolib since it has invalid metadata: Requested geckolib==0.4.8 from https://files.pythonhosted.org/packages/95/b5/6b1293493ddbcef8c576a68adf05cdcc40b9387619507179d3b4607beabd/geckolib-0.4.8-py3-none-any.whl has invalid metadata: Expected matching RIGHT_PARENTHESIS for LEFT_PARENTHESIS, after version specifier python-version (>="3.7") ~^ Please use pip<24.1 if you need to use this version. ERROR: Could not find a version that satisfies the requirement geckolib==0.4.8 (from versions: 0.3.10, 0.3.11, 0.3.12, 0.3.13, 0.3.14, 0.3.15, 0.3.16, 0.3.17, 0.3.18, 0.3.19, 0.3.20, 0.3.22, 0.3.23, 0.3.24, 0.4.0, 0.4.1, 0.4.2, 0.4.4, 0.4.5, 0.4.6, 0.4.7, 0.4.8, 0.4.9, 0.4.15) ERROR: No matching distribution found for geckolib==0.4.8

I have problems too!
I have tried with HA core 2024.7.4 and 2024.7.3
I have tried to reinstall the Gecko integration.
No success!
This is what I read in the logfiles
ogger: homeassistant.util.loop
Source: util/loop.py:77
First occurred: 10:38:43 (3 occurrences)
Last logged: 10:38:43
Detected blocking call to import_module with args ('geckolib.driver.packs.inyt',) in /usr/local/lib/python3.12/site-packages/geckolib/async_spa.py, line 270: GeckoPack = importlib.import_module(pack_module_name).GeckoPack inside the event loop; This is causing stability issues. Please create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue For developers, please see https://developers.home-assistant.io/docs/asyncio_blocking_operations/#import_module Traceback (most recent call last): File "", line 198, in _run_module_as_main File "", line 88, in _run_code File "/usr/src/homeassistant/homeassistant/main.py", line 223, in  sys.exit(main()) File "/usr/src/homeassistant/homeassistant/main.py", line 209, in main exit_code = runner.run(runtime_conf) File "/usr/src/homeassistant/homeassistant/runner.py", line 190, in run return loop.run_until_complete(setup_and_run_hass(runtime_config)) File "/usr/local/lib/python3.12/asyncio/base_events.py", line 674, in run_until_complete self.run_forever() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 641, in run_forever self._run_once() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1990, in _run_once handle._run() File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run self._context.run(self._callback, *self._args) File "/usr/local/lib/python3.12/site-packages/geckolib/async_spa_manager.py", line 543, in _sequence_pump await self.async_connect(self._spa_identifier, self._spa_address) File "/usr/local/lib/python3.12/site-packages/geckolib/async_spa_manager.py", line 334, in async_connect return await self.async_connect_to_spa(spa_descriptors[0]) File "/usr/local/lib/python3.12/site-packages/geckolib/async_spa_manager.py", line 300, in async_connect_to_spa await self._spa.connect() File "/usr/local/lib/python3.12/site-packages/geckolib/async_spa.py", line 367, in connect await self._connect() File "/usr/local/lib/python3.12/site-packages/geckolib/async_spa.py", line 270, in _connect GeckoPack = importlib.import_module(pack_module_name).GeckoPack
Is this due to a intouch 2 update?

hmm, i am on 2024.7.4 too, but gecko is running with his bugs
First installation was 2-3 month ago
Question
is gecko death, because there are no response here since ???

I am on 2024.7.3 with Gecko Integration working good, I use it every day to read values & send commands to my pool.
I do however also have the above mentioned warnings in the HA Core log.
Can't comment if it would be able to install on a fresh system since I use it already since 12/2023.

as i wrote i am on 2024.7.4 (HA) and Gecko 0.1.11
sometimes the connection struggle, but after "Deactivate" Reboot and "Activate" the integration are working.

Solved!
I have tried many things... reinstall gecko, removed and added device, restored an old backup. Nothing helped. I started to have a look in the in.touch2 app and noticed that the spa bath did not have any profile selected( economy, standard, away) I enabled standard mode and then it all worked fine. I am sorry for bothering you guys, but I started to get frustrated.

For the original reported issue:
Unable to install package geckolib==0.4.8: WARNING: Ignoring version 0.4.8 of geckolib since it has invalid metadata: Requested geckolib==0.4.8 from https://files.pythonhosted.org/packages/95/b5/6b1293493ddbcef8c576a68adf05cdcc40b9387619507179d3b4607beabd/geckolib-0.4.8-py3-none-any.whl has invalid metadata: Expected matching RIGHT_PARENTHESIS for LEFT_PARENTHESIS, after version specifier python-version (>="3.7") ~^ Please use pip<24.1 if you need to use this version. ERROR: Could not find a version that satisfies the requirement geckolib==0.4.8 (from versions: 0.3.10, 0.3.11, 0.3.12, 0.3.13, 0.3.14, 0.3.15, 0.3.16, 0.3.17, 0.3.18, 0.3.19, 0.3.20, 0.3.22, 0.3.23, 0.3.24, 0.4.0, 0.4.1, 0.4.2, 0.4.4, 0.4.5, 0.4.6, 0.4.7, 0.4.8, 0.4.9, 0.4.15) ERROR: No matching distribution found for geckolib==0.4.8

This appears fixed in the latest version of this addon(v0.1.11)

Any news to this issue?

Solved! I have tried many things... reinstall gecko, removed and added device, restored an old backup. Nothing helped. I started to have a look in the in.touch2 app and noticed that the spa bath did not have any profile selected( economy, standard, away) I enabled standard mode and then it all worked fine. I am sorry for bothering you guys, but I started to get frustrated.

Mine broke too, but after Disabling the Gecko Integration, restarting HA, and re-enabling the integration, everything was back to normal. (HA 2024.8.3)

2024.10.3 and seeing this issue. Disable/restart/enable/restart does not seem to resolve this issue.
Any tips?


Mobile Menu Peer to peer code review

Please feel free to comment

Not completed. Also ESLint error running the code base


Convert anonymous to arrow functions

Heu Adrian, Marjin .. as you're doing all this restructuring work, maybe have a quick look at my project here https://github.com/kofifus/New ... IMO a convenient way to create javascript 'classes' which is better than the standard 'class' way ... anyway great work and thx

Hi @kofifus, we're aiming for a plain, widely understood code style here. Also, your approach (which looks a lot like the thing Crockford has been advocating) will allocate a lot of unneccesary function objects and throw off the compiler's inlining behavior.

Merged as 9ad40ee76ff
(Feel free to merge in uncontroversial stuff yourself, by the way.)

Thanks. I think I'll need a few more weeks before I allow myself to consider something uncontroversial.


Cst rewrite

WIP
https://github.com/purescript-codegen/purescript-ps-ast/issues/1

Hey thanks a lot for your pull request. Could you keep a summary in the first post of what you are changing? Right now it's a lot of changes and i'm not sure what to look for.

hi @flip111, I have changed types to the types from purescript-cst
http://hackage.haskell.org/package/purescript-0.13.6/docs/src/Language.PureScript.CST.Types.html#Declaration
all types from there are added
but not all are printed http://hackage.haskell.org/package/purescript-0.13.6/docs/src/Language.PureScript.CST.Types.html#Declaration
the Expected.txt was added to some tests, but these tests are not yet implemented
tests are working https://travis-ci.org/github/srghma/purescript-ps-ast/builds/685609928#L3307
in readme you can find
Objectives:
- types are copy of purescript-cst
- parentheses are added automatically, there is no need to use ExprParens or TypeParens (they are added by purescript parser on parsing phase)

the only difference from purescript-cst is the comments field in Declarations

I think everything that I needed is implemented and I'm going to test how it works in https://github.com/srghma/purescript-fernet

the functions in Imports module are commented now, this module was adding imports from fully qualified names in type and value level during printing (and was making fully qualified names in type and value level non-qualified / was simplifying it)
this is cool feature, maybe in future we will re-add it

Looks great so far, give us a notice when you think it's finished and we can do a proper review.

but not all are printed

Do you mean there is no printer code the stringify some types? I know the purescript compiler doesn't have a full pretty printer. There is this project https://gitlab.com/joneshf/purty which has a pretty printer, but i'm not sure how much it is compatible with the code from the purescript compiler.

the only difference from purescript-cst is the comments field in Declarations

What is the change in the comments field? I think to be able to print comments would be very useful because in case not all typescript definition information (this repo is supposed to work together with purescript-dts later) can be transformed into purescript, it can then be put in some comments at least.

@srghma Thanks a lot for your huge contribution. I'm not sure if it is the best place to ask but we have decided that it would be really cool if you join our independent purescript-codegen org team, so I can assure you that you are very welcome :-)
Would you like to join the team?

tests are working https://travis-ci.org/github/srghma/purescript-ps-ast/builds/685609928#L3307

This is huge!
P.S. For sure we have to cleanup some warnings :-P

I'm not using purescript-matryoshka (maybe this is bad)

From my perspective it is really good choice. I think that it is better to preserve consistency with the original library than to reach for fancier API in this case.

I think everything that I need is implemented and I'm going to test how it works in https://github.com/srghma/purescript-fernet

I'm not sure when I find time but I want to cleanup read-dts once again and rewrite codegen for MUI as well using this new API of yours.

@flip111

Do you mean there is no printer code the stringify some types?

yes
sorry, wrong links, updated 1, 2

the only difference from purescript-cst is the comments field in Declarations

What is the change in the comments field?

I have added comments for now only to the declarations, but in purescript-cst every element can have comments, before or after the element
e.g. SourceToken contains comment

@paluh

Would you like to join the team?

yes, of course! finally)

P.S. For sure we have to cleanup some warnings :-P

yes, ok, will do

I'm not using purescript-matryoshka (maybe this is bad)

From my perspective it is really good choice. I think that it is better to preserve consistency with the original library than to reach for fancier API in this case.

:+1:

but in purescript-cst every element can have comments, before or after the element

I don't know if this makes sense in this context. But i once wrote an AST that had a terminal type that had support for the piece of source code + a comment. That way i didn't need to implement the comment for each type separately.

@flip111 terminal type? could you make an example?

I dont think it makes sense to add comments for every type
yes, it could be useful to be able to write someday
foo = bar {- COMMENT1 -} {- COMMENT2 -} foo {- COMMENT3 -}

but I dont need this for now
for now I need comments only at Declaration

@paluh @flip111 @jvliwanag I think it's ready, could you please make a review?
should we make another repo called purescript-ps-cst OR merge to this repo?

@flip111 terminal type? could you make an example?

In the grammer you have non-terminal rules, and terminal rules (tokens). Your tokens can be a String or so. But you can also make it a record of String for the token and a Maybe String for a comment.

should we make another repo called purescript-ps-cst OR merge to this repo?

We can merge here and then rename in to purescript-ps-cst IMO

@srghma Do you think we can do the following:
Generate random AST with quickcheck
Print it into source code
Read source code with parser
Verify that AST from parser matches the AST we put in

In the grammer you have non-terminal rules, and terminal rules (tokens). Your tokens can be a String or so. But you can also make it a record of String for the token and a Maybe String for a comment.

that's basically Haskell types, with Wrapped and everything else

Generate random AST with quickcheck
Print it into source code
Read source code with parser
Verify that AST from parser matches the AST we put in

we don't have parser implemented in purescript

(...) I think it's ready, could you please make a review?

@srghma this is great! Please give me "a moment" - I hope to do the review on the next few days... I'm just not sure if I'm able to add anything valuable to this PR ;-)

@paluh thanks for kind words, did as for myself
review notes are resolved)

created https://github.com/purescript-codegen/purescript-ps-cst


Requesting different model Blackbird 4K 8x8 HDMI Matrix

Product ID is 24180
https://www.monoprice.com/product?p_id=24180&utm_term=&utm_campaign=Sales-Performance+Max-3&utm_source=google&utm_medium=cpc&hsa_acc=6614305189&hsa_cam=15514364592&hsa_grp=&hsa_ad=&hsa_src=x&hsa_tgt=&hsa_kw=&hsa_mt=&hsa_net=adwords&hsa_ver=3&gclid=Cj0KCQjw06-oBhC6ARIsAGuzdw36h6aF2elFWPSyRnEpwXnjnu23Wv-JBRhkWfccX4_8fy_EM_ElGt0aAqPsEALw_wcB

I also have a need there...

I have the same issue.
I wonder if a previous firmware or hardware revision did not require the login credentials by default, as my 24180 does in the GUI, since there is not a field for this in the connection settings.
Using a serial adapter, the serial port on the blackbird and the 'tsp Generic' module configured to that serial port gets me functionality using the control commands in the included instruction manual. I use the 'tcp-udp generic module' pointed at the port I set up in the 'tsp Generic' module. No line-end character for configured tcp commands.

I also have a need there... Although you write that it works with PN 24180, I get "error" in Companion. What can I do?

It doesn’t work with PN24180 unfortunately

Also have a need here! i bought the unit as i found an article on here that says it works with the 24180. now im stuck with a matrix and remote haha!
do i just need to do what gregatjhu is doing and "Using a serial adapter, the serial port on the blackbird and the 'tsp Generic' module configured to that serial port gets me functionality using the control commands in the included instruction manual. I use the 'tcp-udp generic module' pointed at the port I set up in the 'tsp Generic' module. No line-end character for configured tcp commands."

Hi Jarrad91
Yes it works. I did it with e common USB - Serial Adapter.

Just adding to this thread that I also have a need for this module. I get the same "error" message as described above. Will try using prsmvt's module (thanks for that btw!)

Same as last poster. would love an update for this to v3 and for it to work with the 24180 modle


fix: build new JsonLDObject of JsonLDObject with non-empty DEFAULT_JS…

The changes in the pom file are required to run the unit tests (kotlin). Feel free to only merge the source changes and keep the project pure java.
Without the correction the test createDerivedJsonLDObjectWithDefaultContexts fails, because the property contexts of the resulting jsonLDObject contains a listOf(URI, listOf(URI)) instead of a flat listOf(URI, URI)

Sorry, my fault. Intellij automatically ignored all formating changes so I wasn't aware of the mess. I have removed all formatting changes and unit tests. Is there any reason that you don't accept kotlin. It's really hard for me to fall back to plain Java. I did it for the existing sources, but would prefer to add new separated code using kotlin. Nevertheless, for the time being I removed the unit tests.

Thanks, please check if you are a member of DIF and the Identifiers&Discovery Working Group, so we can accept the contribution: https://identity.foundation/join/


Add CLI support for orgs and Auth tokens

Fix #4576

Motivation and context

How has this been tested?

Checklist

[X] I submit my changes into the develop branch
[X] I have added a description of my changes into CHANGELOG file
[X] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[X] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[X] I have updated the license header for each file (see an example below)

# Copyright (C) 2022 Intel Corporation
#
# SPDX-License-Identifier: MIT

@dreaquil , thanks for the PR. Please give us some time to review the patch. Unfortunately right now some delays are possible from our side.

Hi, @dreaquil, thanks for the contribution! Are you still going to contribute to us? If so, could you please resolve conflicts in this PR?

I will close the PR for now. Please, do not hesitate reopen and contact us if you are ready to contribute


full virtual machine requires public sshkey

on a new account

it should tell me what to do or where to go to have that sorted

Already exists


Add preference reset confirmation dialog

Screenshot:

I think it's better if we also use our own SmoothAlertDialog here so that it remains consistent. Otherwise it looks good

I personally prefer the look and feel of stock material widgets. Let's merge as-is for now? Tim and I are planning to do a style refinement iteration later.


Nuclide run-through #3

[ ] Trailing space for types

[ ] Trailing whitespace for argument type

[ ] {} should be on their own line

[ ] Doesn't break #146

[ ] Missing parenthesis for await #164

[x] We have an unwritten rule where we use " if it means not adding an escape. Not sure about this one but this may be nice.

[x] Special case for (;;) {

[ ] Should hardcode \0 to stay as is, \u0000 while valid is weird.

[ ] Weird insertion of commas

[ ] Overly aggressive at breaking #130

I've got one here as well, probably similar to the last one from @vjeux's comment above:

#130 is tracking the issue you mention @knowbody

thank you @vjeux

Closing this one, most of those are now either done or have separate issues.


added theme-switch

Description
This PR Fixes #37
Changes Made
Added a theme toggle button that enables user to switch between light and dark modes

Starred 👍


feat: URL filtering with createFilter

The createFilter is a popular util from Rollup to filter system paths when transforming modules. This PR introduces a port of that util, but for URL paths.
Why?
I'm working on a Nuxt module that allows end users to filter paths that it should run on (nuxt-delay-hydration v1).
How
The API is pretty much the same, supporting regex and some globbing patterns. The globbing isn't as full-featured, as we want to minimize the shipped code (basically whatever radix3 supports).
Implementation is inspired from the nitro rule matching.
This adds an inline dependency of radix3. This can be moved to a regular dependency if you think that makes more sense.
It may be the case that the ufo package may not be the best place for this, instead, it could live in radix3. However, keeping it in this package would improve its discoverability and I believe it falls under the scope of the ufo package.
Interesting to hear your thoughts
Thanks!

Thanks for PR @harlan-zw. This seems an interesting feature. I think we can add it as radix3/utils subpath util from same package.


Does eloquent require pivot models to have a unique non-composite id?

The documentation page has the following requirement:
Eloquent requires each model to have at least one uniquely identifying "ID" that can serve as its primary key. "Composite" primary keys are not supported by Eloquent models. However, you are free to add additional multi-column, unique indexes to your database tables in addition to the table's uniquely identifying primary key.
Does this requirement apply to the Pivot models?

The documentation page also states the following:

Laravel's GitHub issue trackers are not intended to provide Laravel help or support. Instead, use one of the following channels

For not letting you without an answer, the requirement aforementioned does not apply to pivot models. You cans see below it handles the expected composite key:
https://github.com/laravel/framework/blob/1814e8066a6ba658ad42a662de9d3263b9637f9c/src/Illuminate/Database/Eloquent/Relations/Concerns/AsPivot.php#L86-L116
By expected I mean the foreign keys to a pivot related to two tables. If you have a pivot related to more tables, or if you will need to manipulate custom fields on the pivot by retrieving pivot models directly, I would consider adding a dedicated primary key.
Just don't forget to set its $incrementing property back to true if you are using an incrementing ID.
Next time, please use one of the support channels listed on the docs for this kind of questions. Thanks!


orient_to_camera

It appears that mlabtex does not have the orient_to_camera option that is the default in mlab.text3d so I guess this is a feature request, though possibly a difficult one. The feature is important to retain visibility when interactively rotating a scene, such as for latex labels to identify solids or sets of coordinate axes. Thanks.

Pull requests are welcome. mlabtex mainly provides an image viewer. The TeX viewer only generates an image file from the given TeX code... The text3d provides more of an overlay.


Add grafana VPA dashboard

How to categorize this PR?
/area monitoring
/kind cleanup
/kind enhancement
/priority normal
What this PR does / why we need it:
Add grafana dashboard for VPA when the shoot VPA feature is enabled.
Special notes for your reviewer:
follow up to #2478
Release note:
Add grafana dashboard for VPA when the shoot VPA feature is enabled

fixed the time setting in grafana to use the past 30m


Reset DIV2 with other flags when setting waveform

@RobTillaart - I hope you're open to PRs. I think the DIV2 bit should be reset in the control word when the frequency is set, so that you can switch from SQUARE1 to SQUARE2.

I'm open for PR's,
will review later tonight as I have several other tasks at the moment.

Thanks for the PR.
Not planned a new release soon, but your fix will definitely be in the next one.
(created an issue about a detailed API for the control register, for next release too)

No problem. Thanks for your efforts!


Can Mxnet fit a regression LSTM model?

http://stackoverflow.com/questions/39595084/can-mxnet-fit-a-regression-lstm-model-in-r
Is LSTM in Mxnet applicable to a regression problem (such as sentence sentiment classification)?
Thanks!

have written it