Profile argument not taken into account when calling package do_cli from python interpreter

Description
I'm using the do_cli method from the samcli.commands.package.command.py in a python interpreter to package a Serverless Function.
I provide the profile argument but it doesn't seems taken into account because I have An error occurred (AccessDenied) when calling the PutObject operation: Access Denied.
Steps to reproduce

Create a main.py and a requirements.txt (or whatever depending on the programming language)
Create a template.yaml
Have a profile not linked to the default profile
Run deploy.py in a python interpreter

main.py
def handler(event, context):
    print("Hello World")

requirements.txt
boto3

template.yaml
Resources:
  Function:
    Properties:
      CodeUri: .
      Handler: main.handler
      Runtime: python3.7
    Type: AWS::Serverless::Function

deploy.py
import os
import tempfile
from samcli.commands.build.command import do_cli as build_do_cli
from samcli.commands.package.command import do_cli as package_cli

template_path = "template.yaml"
base_dir = ""  # Todo: define here where you created the main.py and the requirements.txt
s3_bucket_name = ""  # Todo: define here your bucket
region_name = "" # Todo: define here the region
profile_name = "" # Todo: define here the profile nale

with tempfile.TemporaryDirectory() as tmp_dir_name:
    sam_build_dir = os.path.join(tmp_dir_name, "SAM_BUILD")
    sam_build_path = os.path.join(sam_build_dir, "template.yaml")
    sam_package_path = os.path.join(tmp_dir_name, "SAM_PACKAGE_TEMPLATE.yaml")

    build_do_cli(
        function_identifier="Function",
        template=template_path,
        base_dir=base_dir,
        build_dir=sam_build_dir,
        clean=True,
        use_container=True,
        manifest_path=None,
        docker_network=None,
        skip_pull_image=False,
        parameter_overrides=None,
        mode=None,
    )
    package_cli(
        template_file=sam_build_path,
        s3_bucket=s3_bucket_name,
        s3_prefix="lambdas",
        kms_key_id=None,
        output_template_file=sam_package_path,
        use_json=False,
        force_upload=False,
        metadata=None,
        region=region_name,
        profile=profile_name,
    )

Observed result
I received the following error:
An error occurred (AccessDenied) when calling the PutObject operation: Access Denied
Expected result
Package should work out of the box without passing from the cli.
Observations
Issue seems relative to this line. If profile is specified, the boto3 Session object should be created before creating the s3 client.
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)

OS: MacOs Catalina 10.15.3
sam --version: SAM CLI, version 0.41.0

@Zangror Are you using the CLI as a library?

Yes I am.

The CLI is not built in a way to do this. So it is completely not supported. We heavily rely on click and Click's Context model to setup things like profiles and regions.
Closing as this is not a supported usage.


In Pad, Matrix exponentiation doesn't behave as expected

Using the Matrix exponentiation Uiuism on Chrome on Windows 10, both up-to-date, (this is the direct Pad link, but is located at the bottom of https://www.uiua.org/isms) on the 2-D basis matrix for the Fibonacci sequence, there are computation errors for exponents greater than 78.
For reference, the 79th term of the Fibonacci sequence should be<PHONE_NUMBER>4676221, but has a zero in the ones place if run with the Uiuism. For an example of this, here is a link to a Pad with the Fibonacci basis matrix and an n-value of 78, with n+1 in the first matrix cell of the result. As sequence terms increase past 79, this continues for successive terms, and there is a loss in precision as terms increase, but the initial digits of successive terms are correct until hitting their respective limits.
Since this error only occurs above the 32-bit unsigned integer limit of<PHONE_NUMBER> and for some values under the signed 64-bit limit of<PHONE_NUMBER>854775807, this may be unintended behavior.

This is a floating-point rounding error, as most numbers in Uiua are represented as IEEE doubles.
Consider the output of this Rust program.

That rust playground link contains no rust program

@Garmelon Try this link. If that fails, here's some code and output:
Rust code:
fn main() {
    let err_num: f32 =<PHONE_NUMBER>4676221f32;
    println!("Errored number (f32): {}", err_num);
    let err_num: f64 =<PHONE_NUMBER>4676221f64; //shadowing `err_num`
    println!("Also errored number (f64): {}", err_num);
    let num: u64 =<PHONE_NUMBER>4676221u64;
    println!("Number (u64): {}", num);
    let num: u128 =<PHONE_NUMBER>4676221u128; //shadowing `num`
    println!("Also number (u128): {}", num);
}

Output:
Errored number (f32):<PHONE_NUMBER>0000000
Also errored number (f64):<PHONE_NUMBER>4676220
Number (u64):<PHONE_NUMBER>4676221
Also number (u128):<PHONE_NUMBER>4676221

In an ideal world, it might be nice to have strictly integral-type matrices that are fully integrally bounded to use integral types under the hood, but given that matrices can be float-valued, it makes sense to implement them in that domain alone, especially since one could use modulo or other algos to just remap into the appropriate range. Implementing some sort of type-switching that could do so would allow for some more flexibility, but I would imagine would be a nightmare to implement, even if possible. This isn't truly an issue at this point, especially since there are other ways of approaching this particular problem.
P.S. This same f64 (or equivalent-size IEEE double) numeric limitation applies in APL/J and BQN, so it seems fairly standard (linguistically speaking, anyhow) for operations structured like this to remain strictly float-valued.

The interpreter currently uses both an f64 array type and a u8 array type.
The u8 type is mostly for space efficiency when reading files in and sometimes for booleans.
While it would not be hard to add an integral type, it would be a lot of code both to write and for the Rust compiler to chew on, further increasing the compile times of the interpreter itself.
It is a combinatorics problem.
A new array type wouldn't necessarily be a lot to add.
A new array type that has to behave like the other number types would be a lot, as most functions on arrays have to type check, and each new number type would double the number of number-type checks. Some of these could be eliminated by doing a catch-all convert-to-f64 for some operations, but this would also defeat the purpose of adding an integral type in the first place.


unLimit delay message

Make sure set the target branch to develop
What is the purpose of the change
Support second level delay messages at any time within two months（default）
Brief changelog
Build a chronological file in seconds of delayed messages delayTimeLine, different delay messages are mapped to the corresponding time unit of the file, Messages with the same delay time are added to the delaylinklog log and written sequentially. The asynchronous time thread looks for the delay message that has expired in the current second according to the current time seconds, reads and restores the original message and puts it back into the commitlog
Verifying this change
Refer UTs, ITs and testing report
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Refer  https://note.youdao.com/s/9y0kjrvJ


How not to get an BufWinLeave  autocmd triggered when a notification closes

Title says it.
I want to trigger an autocmd on every file type but this plugin broke the setup.
Could you tell me how to modify my (simplified):
autocmd BufWinLeave * echo "autocmd triggered"
so it doesn't get triggered every time a notification timeouts?
I understand that it doesn't directly comes from nvim-notify's code, but for me it was a nasty 5h hunt for why this side-effect broke my setup, so a way to workaround this (quite counterintuitive) issue would be welcome for me, and potentially many others

That sounds like it's a pretty problematic autocmd to have at all but hey not going to question other's configs! :smile: Since I don't know what you're doing there's not much I can suggest specifically. You could change the pattern to be ?* so there has to be at least a single character in the buffer name to be triggered


postgresql extractor fixes

covering more data types
adding source.conn.database which allows you to connect to a database
that has a different name than the source schema (i.e. this is 3-part
naming convention).  Without this you cannot connect to a database
"workgroup" to pull the table "public.http_requests"

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-XXX

Description

[ ] Here are some details about my PR, including screenshots (if applicable):

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Thanks for the contri. Please also include Jira numbers while creating PRs.


BUG: download metadata function not working after metadata tree refactoring

Need to find a package that zips multiple files related to the current dataset, then download's zip. Or decide to only download dataset metadata, which needs to be correctly identified based on changes after refactoring.

Fix: decided to download dataset-level metadata; updated to account for change in metadata path specification. Issue to be closed once merged.

Done: 22f21718d958d4f6d21f3958a83d733552d0b018


uhd: <IP_ADDRESS> -> <IP_ADDRESS>

Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

LGTM.  I ran the update as documented and it looks as though it worked:
matthew@xin ~/D/nixpkgs (pkg/uhd)> nix-shell maintainers/scripts/update.nix --argstr package uhd --argstr commit true

Going to be running update for following packages:
 - uhd-<IP_ADDRESS>

Press Enter key to continue...

Running update for:
Preparing worktree (new branch 'update-tmpfftrsyp0')
Updating files: 100% (37477/37477), done.
HEAD is now at 4f03332c7031 uhd: <IP_ADDRESS> -> <IP_ADDRESS>
 - uhd-<IP_ADDRESS>: UPDATING ...
 - uhd-<IP_ADDRESS>: DONE, no changes.
Deleted branch update-tmpfftrsyp0 (was 4f03332c7031).

Packages updated!

I wasn't aware of the maintainers/scripts/update.nix script; I've been executing the update.sh scripts manually.  In the case where the UHD update script is executed 'manually' we get:
./update.sh: line 7: $1: unbound variable

Maybe there should be a better error message in that case informing the user to use the above mechanism?

I'm more looking for a review regarding the utilities for instance, which I'm pretty sure you use :). And if you use a device supported that's even better.

@ofborg build gnuradio

I'm more looking for a review regarding the utilities for instance
Ah, apologies for the misunderstanding! I can confirm that the tools work correctly.

And if you use a device supported that's even better.
Yes I have an N210 here which seem to work fine with this update.   I've tested the utilities as well as the UHD GNURadio block.


Support ReadOptions in v1beta3

Read options for lookups and queries should be disabled within transactions.  Transaction options should only be available when creating a new transaction.

Disabling read options from within a transaction is a bit challenging because the DatastoreImpl class and the Transaction class both implement the Datastore interface.  Two ways to solve this to include:

Setting read options when creating a Datastore service object.  (If users want some actions to use eventual consistency and others to use the default level of consistency, then they could create two service objects).
Including varargs read options for lookup/query methods in the Datastore interface, but throwing an exception at runtime if a user attempts to set read options from within a transaction.

Any opinions/suggestions @aozarov @pcostell?

We should also consider this option:

Add read-only to TransactionOption

Add a new ReadOptions class that only includes ReadConsistency settings.

Add only to the Datastore interface  get and run(Query) methods that accept ReadOptions

Modify Datastore implementation for runInTransaction to defer transaction creating until the first lookup or query and if none use CommitRequest with TransactionOptions

I agree with Arie's proposal --
transaction options can be made available to the user at the call to runInTransaction or newTransaction. There isn't a need to expose TransactionOptions directly on get and runQuery calls because it's effectively an optimization that runInTransaction should take advantage of to avoid the call  to beginTransaction.

Since transaction options are coming in a later release of v1beta3, I will create a separate issue for transaction options and keep this issue for read options.

Closed via #691


Manual tool calls don't work

I have a scenario where it may take a bit of time to run a tool (I need to make other model calls) so I'm trying to use manual tool calls but they don't work for a number of reasons. One issue I can work around but the other I can't without modifying the client code.
Tool registrations are overwritten
I think you might have been trying to reference this in the note but if you try to register tools before connecting to the server those registrations are overwritten when the connect happens.  That's because immediately after the connect an empty updateSession() call is made here. And that results in existing tool registrations being overwritten here. To avoid that you have to re-register your tools with another call to updateSession() after you connect.
Manual tools always throw an error
The bigger issue is that any tool without a handler will result an error because of the call here. There wont be a tool config since the tool wasn't added using addTools() and an exception will be thrown here.
To work around the issue I add this logic in my version of the library:

Relay server tries to run tools as well
To add to the confusion, when running a relay server that also tries to call tools. I couldn't understand for a while why the fix above wasn't working and I was still getting an error. That's because the relay server tries to run tools but it doesn't have any handlers so it always throws an error. This sets up a race condition between the clients tool execution and the relay servers exception.

I created a patch for your changes on the library code.
@openai+realtime-api-beta+0.0.0.patch

Install patch-package: npm i patch-package --save-dev
copy @openai+realtime-api-beta+0.0.0.patch to new root folder patches
add "postinstall": "patch-package" to scripts in package.json
run npm ci

Nice! @lx-0

@lx-0 Not all heros wear capes

@Stevenic @lx-0 You guys are legends thanks for the assistance on this blocker.
I've posted the solution here and I've also given credit to those that assisted with the bug. You guys are the real heroes!!!!!
https://www.youtube.com/playlist?list=PLkNKgOpMwm0uvvtrnFn1hYrllN-d95FVV

You are super awesome Thanks for unblocking

@bjsllc I found your video but putting the changes into the folder "patch" didn't work - had to use patches. Not sure if it was a config issue on my end or if it was specific to the weather tool - my custom tool works with the patch in ./patches/ tho :)!!

@bjsllc I found your video but putting the changes into the folder "patch" didn't work - had to use patches. Not sure if it was a config issue on my end or if it was specific to the weather tool - my custom tool works with the patch in ./patches/ tho :)!!

Thanks for the heads up @AdamBissonnette I went ahead and alerted everyone of the snafu. You are correct it should be "patches" not "patch"


Allowing users to mark submission removed

Is your feature request related to a problem? Please describe.
Some participants wanted to withdraw their submission (before deadline), but there is no other way to do so. Allowing participants to withdraw submission/mark them removed could solve it (it could be also a sanity check for bad submissions).
Describe the solution you'd like
To add an option that would allow people to mark their submissions deleted.

There is no need for removal individual submissions? I will remove the wmt23 label from this issue and we can revisit after WMT23 is in the books...

Following on this issue, teams are able to mark theirs systems withdrawn and not get evaluated, however, the flag withdrawn seems to be automatically True and only when they actively withdraw some systems, their non-withdrawn submissions lose the flag "withdrawn".
The problem:
We had a team (GTCOM) who submitted 8 primary and withdrew 6 of it, for whom the flag works. However then for a team like CUNI-DocTranslator, all systems are marked as withdrawn, despite they never marked any of them withdrawn


NoSuchMethodError: LvsPlayerError working with compose version 2024.06.00

Update compose version 2023.03.00 ->  2024.06.00
When try to start video player.

Should be fixed in a few days.

Fixed in release 1.3.0


Update Soda fountain dependencies

soda-fountain #323 FAILURE
Looks like there's a problem with this pull request

soda-fountain #324 FAILURE
Looks like there's a problem with this pull request

soda-fountain #326 FAILURE
Looks like there's a problem with this pull request

soda-fountain #328 SUCCESS
This pull request looks good


Download + install in add command

Currently the add command doesn't download + install the package.
Maybe there are good reasons to do it that way but I think I would actually prefer it to be done in the add step. From a UX perspective, I'd rather wait one longer time than two smaller times.
I guess doing the installation in add also opens up some possibilities for reducing the total wall time spent waiting, maybe by doing stuff concurrently.
What's the use case for separating the two? When would you want add to not actually download and install?

I dont see a reason why we shouldnt do that. Ill create a PR.


Download postgress extensions

Problem
Add support for downloading pg-extensions.
Summary of changes
Added functions for downloading pg-extensions.
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

// prototype of code for downloading data from an S3 bucket
use toml_edit;
use remote_storage::*;
use std::path::Path;

// read configurations from `pageserver.toml`
let cfg_file_path = Path::new("./../.neon/pageserver.toml");
let cfg_file_contents = std::fs::read_to_string(cfg_file_path).unwrap();
let toml = cfg_file_contents
    .parse::<toml_edit::Document>()
    .expect("Error parsing toml");
let remote_storage_data = toml.get("remote_storage")
    .expect("field should be present");
let remote_storage_config = RemoteStorageConfig::from_toml(remote_storage_data)
    .expect("error parsing toml")
    .expect("error parsing toml");

// query S3 bucket
let remote_storage = GenericRemoteStorage::from_config(remote_storage_config)?;
let from_path = "neon-dev-extensions/public/example-extension.control";
let remote_from_path = Some(RemotePath::new(Path::new(public_dir))?);
let data = remote_storage.download(&remote_from_path);

// Later: write `data` to a file locally

If I want to test this, do I make a new file in test_runner/regress/ that
sets up an environment with a configuration file like this:
[remote_storage]
bucket_name = 'neon-dev-extensions'
bucket_region = 'eu-central-1'
concurrency_limit = 100

AWS environment variables


Private key from entropy cannot be created without a password

What we witnessed is that when bip32PrivateKeyFromBip39Entropy is called with "" (empty string) for passphrase it is interpreted as null value when passing to ios and android bindings and because of that it errors out. To mitigate that issue, what we did is, we applied the following patch:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index c2916a5..dd51e9e 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -1130,6 +1130,10 @@ public class HaskellShelleyModule extends ReactContextBaseJavaModule {

     @ReactMethod
     public final void csl_bridge_bip32PrivateKeyFromBip39Entropy(String entropy, String password, Promise promise) {
+        if (password  == null) {
+            password = ""
+        }
+
         Native.I
             .csl_bridge_bip32PrivateKeyFromBip39Entropy(Base64.decode(entropy, Base64.DEFAULT), Base64.decode(password, Base64.DEFAULT))
             .map(RPtr::toJs)
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 0097431..6c3bc75 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -1589,8 +1589,13 @@ + (void)csl_bridge_initialize
     }] exec:selfPtr andResolve:resolve orReject:reject];
 }

-RCT_EXPORT_METHOD(csl_bridge_bip32PrivateKeyFromBip39Entropy:(nonnull NSString *)entropyVal withPassword:(nonnull NSString *)passwordVal withResolve:(RCTPromiseResolveBlock)resolve andReject:(RCTPromiseRejectBlock)reject)
+RCT_EXPORT_METHOD(csl_bridge_bip32PrivateKeyFromBip39Entropy:(nonnull NSString *)entropyVal withPassword:(nullable NSString *)passwordVal withResolve:(RCTPromiseResolveBlock)resolve andReject:(RCTPromiseRejectBlock)reject)
 {
+    // Set passwordVal to an empty string if it's null
+    if (passwordVal == nil) {
+        passwordVal = @"";
+    }
+
     [[CSLCSafeOperation new:^NSString*(NSArray* params, CharPtr* error) {
         RPtr result;
         NSData* dataEntropy = [NSData fromBase64:[params objectAtIndex:0]];

To reproduce the issue, there is not much you need to do. Just execute this piece of code:
Bip32PrivateKey.from_bip39_entropy(
        Buffer.from(mnemonicToEntropy(mnemonic), "hex"),
        Buffer.from(""))

Hope this helps. Also if you feel like there is blunder in usage and that this is in fact not an issue, please let me know.

Hi @trimixlover! Could you try to pass Uint8Array.from([]) instead Buffer.from("") ? By default it should not to produce "null" on empty byte array.

Tried! Works! Closing it.


Hide notes that point to digitised files from Codebreakers

These notes are at best confusing and at worst wrong. We don't want to entirely remove them from the Sierra data, but nor do we want to display them on wc.org/works for now.
See https://wellcome.slack.com/archives/C8X9YKM5X/p1629900086107000

Here is an example of one that still shows the forbidden phrase: https://wellcomecollection.org/works/a26dt8ex
Interestingly, this record does not show the forbidden phrase: https://wellcomecollection.org/works/tb8dtsev even though it is present in the corresponding Sierra data (wdkr76jt, b32723933).  I suspect something is happening in the Merge that drops all but the first note, as this has two terms-of-use notes and one location-of-duplicates note in works-identified, but only the one appears on the website.


Add item into history events of collection

Seems we don't show item in history of collection?

related to https://github.com/kodadot/packages/issues/17


Capistrano version mismatch

Apparently, even thought we specified version ~> 3.3.5 for Capistrano in our Gemfile, version 3.4.0 is being installed in ITS testing. I'm going to switch it to an exact version identifier at 3.4.0, update the version lock in the Capfile, and push a patch to v5.3.0. This should be addressed in a matter of hours (I'll try to test on DO to make sure it's working), given that it's just version number tweaks I'll probably just merge this immediately if it runs correctly.

My apologies, I added this to the wrong repo!


Dune 2000 graphic issue

Hi,
when I use your ddraw.dll along with Dune 2000, I get this when I run it.
It is similar to what happens when I alt tab and tab back on Windows 10 (I think it also does on 8, but certainly this does not happen on W7):
And this is what it should look like.
After some testing with the Compatibility Fixes I found out I can replicate the same problem by turning on AliasDXDC. This is what it does.
Also my friend has Mac, when he uses this ddraw.dll, his Alt key stops working properly for some reason.
Can you advise what I can do to avoid these problems with your ddraw.dll?

@Shaokhan65, I have been working on a project called dxwrapper that can convert DirectDraw to Direct3D9.  The latest updates seem to work with Dune 2000.
You can download the update here: ddraw.zip
Just unzip the files into the Dune 2000 folder overwriting any existing file.
Note: in some cases you may need to reboot the computer before you can run Dune 2000.

I can't reproduce this with the latest experimental release. I assume it's been fixed already. If it's not the case. please reopen this.

I can't reproduce this with the latest experimental release. I assume it's been fixed already. If it's not the case. please reopen this.


"Become A Volunteer" page styling.

Is your feature request related to a problem? Please describe.
I want to improve the styling of the "Become A Volunteer" page. Right now it is not consistent with the rest of the site.
Describe the solution you'd like
The styling would mirror the "contact us" form on the home page.
I would also additionally add the footer to the page.
Additional context
Add any other context or screenshots about the feature request here.

hey its not required currently
Thanks


Demonstrate login with NordLEI agent/credential

We have successfully tested with NordLEI related data as a defined environment https://github.com/GLEIF-IT/reg-pilot/blob/main/signify-ts-test/test/utils/resolve-env.ts


Disable the input fields after submitting

If the fields are disabled before submitting the form, the browser will not post their values since they will have disabled attributes (see ref). Moved the disable posts logic after the submission to prevent this.
ref: https://www.w3.org/TR/html5/forms.html#constructing-the-form-data-set

Thanks!

I must have realize the submit had to be before disabling the fields as I did it on my blog.  Must not have made the changes here:
  var form = document.getElementById('commentform')
  form.action = 'https://app.damieng.com/api/PostComment'
  button.innerText = 'Posting...'
  button.disabled = true
  form.submit()
  var fields = document.getElementById('commentfields')
  fields.disabled = true


Run MongoDB compatibility tests

MongoDB has a set of integration tests that we would like to run against FerretDB to see how compatible we are: https://github.com/mongodb/mongo/tree/master/jstests
It is somewhat documented there, there, and probably in some other places.
They also have a runner of those tests in a separate repository: https://github.com/mongodb-developer/service-tests
### Tasks
- [ ] #193
- [ ] #165
- [ ] #276
- [ ] #289
- [ ] FerretDB/FerretDB#1690
- [ ] #290
- [ ] #250
- [ ] #166
- [ ] #318
- [ ] #461
- [ ] enable MOAR tests

answer FerretDB/FerretDB#33

@AlekSi what is MOAR in this context?

All of them, as much as possible. But for now, I think we better reorganize what we already have.


Clean up tests on PyPy

They're failing too often in CI.
Also there's spurious warnings.

Frequent warning:
~/.pyenv/versions/pypy3.10-7.3.16/lib/pypy3.10/asyncio/sslproto.py:320: ResourceWarning: unclosed transport <asyncio.sslproto._SSLProtocolTransport object at 0x...>

Test failure not linked to a specific test, but to the framework:
Traceback (most recent call last):
  File "/Users/myk/dev/websockets/tests/legacy/test_client_server.py", line 122, in _decorate
    return func(self, *_args, **_kwargs)
  File "/Users/myk/dev/websockets/tests/legacy/test_client_server.py", line 121, in _decorate
    with manager(self, *args, **kwargs):
  File "/Users/myk/.pyenv/versions/pypy3.10-7.3.16/lib/pypy3.10/contextlib.py", line 135, in __enter__
    return next(self.gen)
  File "/Users/myk/dev/websockets/tests/legacy/test_client_server.py", line 105, in temp_test_client
    test.start_client(*args, **kwargs)
  File "/Users/myk/dev/websockets/tests/legacy/test_client_server.py", line 331, in start_client
    super().start_client(path, **kwargs)
  File "/Users/myk/dev/websockets/tests/legacy/test_client_server.py", line 266, in start_client
    self.assertDeprecationWarnings(recorded_warnings, expected_warnings)
  File "/Users/myk/dev/websockets/tests/legacy/utils.py", line 80, in assertDeprecationWarnings
    self.assertEqual(type(recorded.message), DeprecationWarning)
AssertionError: <class 'ResourceWarning'> != <class 'DeprecationWarning'>

I doesn't make sense to spend much time fixing the test suite for the legacy implementation under PyPy, though.

This warning is a bug: https://github.com/python/cpython/issues/90476
It was fixed in Python 3.11.
I tried to remove it by adding this to tests/__init__.py:
import warnings
warnings.filterwarnings(action="ignore", category=ResourceWarning)

This filters out ResourceWarning. I could confirm by raising a ResourceWarning just after this line. However, the warnings still get raised later in the tests. Probably something changes the warning filters inbetween. I just don't know what.
Removing warnings with command line arguments (pypy3.9 -W ignore:"unclosed transport":ResourceWarning:asyncio.sslproto -m unittest) works but that's quite annoying and hard to make conditional on the Python version. Very messy.

After 1853a9b2d0247573633e2749fe1169f764abe03c tests are passing again on GitHub Actions — after a long string of non-passing tests. I couldn't get rid of the warning; I'll live with it.


Module Security to avoid RemoteSigned or Bypass ExecutionPolicy

Would be nice if the module was signed to avoid needing to modify the ExecutionPolicy to either RemoteSigned or Bypass. Need to research this further.

This is addressed by installing the module via the Gallery. This issue is no longer necessary.


Statusline virtcol display doesn't respect neovim positive count requirement

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
Statusline
Description
Hello o/.
Neovim requires positive count which is currently not respected by the virtcol display (perhaps by design?) in Mini.Statusline location section:

For the sake of this example, if you, e.g., type d0 to delete the empty line, Neovim will return an error, instead expecting d1.

One way to solve this is to set the minimum value to 1:
MiniStatusline.section_location = function(args)
  -- Use virtual column number to allow update when past last column
  if MiniStatusline.is_truncated(args.trunc_width) then return '%l│%2v' end

  -- Use `virtcol()` to correctly handle multi-byte characters
  return string.format('%%l|%%L│%%2v|%-2d', math.max(vim.fn.virtcol("$") - 1, 1))
end

Thanks for the issue!
This is indeed intentional and allows to differentiate truly empty line and a line with one character (possibly whitespace).
Closing as behavior is by design.

Neovim requires positive count which is currently not respected by the virtcol display (perhaps by design?) in Mini.Statusline location section:

I am not sure how total number of columns in the current line is related to count.

For the sake of this example, if you, e.g., type d0 to delete the empty line, Neovim will return an error, instead expecting d1.

I can not reproduce. The d0 executes d with 0 motion with no errors. The d1 requires a motion/textobject after typing it.

I can not reproduce. The d0 executes d with 0 motion with no errors. The d1 requires a motion/textobject after typing it.

I should have specified that it must be run from the command line - My bad!

I should have specified that it must be run from the command line - My bad!

Ah, then it surely doesn't have anything to do with 0 being used as number of columns in current line. Because :d operates on lines and uses {count} as number of lines.


DOC: Best practices for limiting responses to a specific source document

Issue with current documentation:
Hi, Thanks for the contribution.
I have been using your repository to train a model on a collection of books. My goal is to generate answers that are specific to a single source document, essentially using the model as an assistant that draws information from one selected book at a time (such as "cats.pdf").
Initially, I attempted to implement this by modifying the prompts, but the results were inconsistent, and the model sometimes used information from other sources. Here's an example of how I structured the prompts:

You are a helpful assistant trained to answer questions solely based on the content of book_name.pdf. Given the text in the book and a question, generate an appropriate answer. If the answer is not contained within the book, simply say that you don't know, rather than inventing an answer. The question is: What is the distance from the moon to the earth?

Seems like ingest.py adds the source path to the doc metadata. However, when a question is asked, the model retrieves the most relevant documents based on the semantic similarity between the query's embedding and the documents' embeddings, not a specific document identifier. The model does not consider the document's metadata (like its source path) during retrieval, which means it can't be instructed to refer to a specific document just by mentioning the document's name or identifier in the prompt (?).
Considering this, I'm evaluating the option of creating a dropdown menu that lists all the books I've trained the model on. When a book is selected from this menu, I would swap the databases to only include documents from the selected book when a query is made.
With that context, I have a few questions:

Is there a more efficient way to constrain the model's responses to a specific source document than by manipulating the prompt, using metadata, or swapping databases?
If I proceed with the dropdown menu and database swapping approach, are there any potential drawbacks or issues I should be aware of?
Given the potential usefulness of this feature to other users, would it be worth considering the addition of an option to limit responses to a specific source in the main repository?

Thanks for your time & I'd appreciate your insights.
PS: Adding this under docs, because it might be a result of my lack of understanding of how everything works together.
Idea or request for content:
No response

In order to enrich this discussion, it would be beneficial to be able include keywords that can aid in "directing" the response.
Keywords: Cat, Cat Book, Meals.
Please proceed with your question.

Is there a more efficient way to constrain the model's responses to a specific source document than by manipulating the prompt, using metadata, or swapping databases?

Yes. Several options:

add source metadata in prompt. Pro: very easy to implement, Con: takes tokens for each source, and relies on the right source being picked as a "relevant" source
since retrieving items from the db is very fast compared to generating the answer, we can fetch many items, and filter by their metadata before forwarding them to the model. Pro: easy to do, con: relies on the database fetcher to be good enough to retrieve the documents we want among all the documents requested

If I proceed with the dropdown menu and database swapping approach, are there any potential drawbacks or issues I should be aware of?

I'm unsure how the db is stored, but I don't think it's ordered by metadata, so if your db is big it will take a long time to filter by your document

Given the potential usefulness of this feature to other users, would it be worth considering the addition of an option to limit responses to a specific source in the main repository?

Yes that sounds like a good idea

Some delightful ideas I would have to think about, thanks.
For starters I see no reason why we could not just pipe in
if selected_document is not None:
    metadata_filter = {"key": "source_document", "value": selected_document}
    retriever = self.qdrant_langchain.as_retriever(
        search_type="mmr", metadata_filter=metadata_filter, k=n_forward_documents, fetch_k=n_retrieve_documents
    )
else:
    retriever = self.qdrant_langchain.as_retriever(search_type="mmr", k=n_forward_documents, fetch_k=n_retrieve_documents)

Where this would not perform worse than any other approach (have to check mmr compatibility tho). I think it should be possible to fetch a list of all available documents and set the filter on selection.

Nice thanks for the quick suggestions. I'd probably go with the easiest for now. https://github.com/su77ungr/CASALIOY/issues/95#issuecomment-1553825136
Another idea is to modify the ingest.py. Currently, it creates a single collection for all books, but perhaps we could adapt it so that a new collection is created for each source_document. startLLM.py will have to be updated to retrieve only the selected collection. However, I'm not sure about the potential performance implications of this approach.
The challenge here is that many users may prefer to search across all their documents, which is not my specific use case.
Regardless of the approach, will likely need a mechanism to keep track of which names have been ingested. This could potentially be achieved by creating another table in qdrant for storing global metadata.
I'd love to collaborate, but my plate is pretty full these days. I usually sneak in some time for AI projects at night. Really appreciate what you're doing here!


[limited FoV] ds000217 fieldmap issues

Poor skulls stripping of the magnitude image

Poor coregistration between magnitude and BOLD images

fieldmap corrections induces artefacts

sub-Exp1s02 all runs

Related to #23 (problems with magnitude's brain mask)

The above PR adds tests using all ds000217's magnitude images - masks can be visualized with their reportlets. Although masks are not perfect, I think this is resolved in conjunction with the B-Spline modeling.

The above PR adds tests using all ds000217's magnitude images - masks can be visualized with their reportlets. Although masks are not perfect, I think this is resolved in conjunction with the B-Spline modeling.


Ci prepare release

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


Don't delete branches/PRs without feedback

I noticed that @AmorAcebo deleted ananda's-branch without comment. Please don't do that - it's better to give feedback about any potential issues, then discuss it.

Sorry! My bad!


If-clause in manifestChecked confusing

https://github.com/BabylonJS/Babylon.js/blob/ce77425ddfaa3ac1bafa9657a7a1f6c4eba99fee/src/Loading/babylon.sceneLoader.ts#L306
Isn't it better to reverse the statement? Since we can assume that the exception is not undefined or null and we can access the property message?
onError("Failed to load scene." + (exception ? " " + exception.message : ""), exception);

Correct!
Will fix that ;)

Thank you =)


Update service map parsing results for testing

Signed-off-by: Joshua Li<EMAIL_ADDRESS>Description
Plotly.js updated the tooltip format, and tests were out of date. Updating test constants
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

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

I also changed this in my PR but we can also change it here :)

Cool, fixing this to unblock eric, shouldn't affect your PR if it's the same change


Cannot login to admin, TokenMismatchException in VerifyCsrfToken.php

I have 2 installations of Laravel version 5.3. The first one is upgraded from 5.2 and I install Voyager without any error in the development. But when I push it to production server (by git hooks), it throws error something like TokenMismatchException in VerifyCsrfToken.php when I try to login with default credential. I installed voyager via SSH on production. FYI, the server is backed by Cloudflare.
The second one is a fresh install of version 5.3. I also cannot login to the Voyager in this server. When I try to login with default credential, it throws no error, but it doesn't take me to admin page, and shows the login page instead.
Edit: It's likely caused by wrong permissions in the storage folder, but I'm not sure what to change. I have set all folders and contents inside storage folder to 777.
I have changed the folder permissions and clear cache, dump autoload; the fresh install now works well, but the upgraded one still encounters login error, although I did the same procedure.

Is this still an issue?

Yes, it's still an issue for upgraded Laravel, but now is okay for the fresh install of 5.3


Add SIS example with tauleap method

Added an example SIS model from chapter 6 of keeling & Rohani https://homepages.warwick.ac.uk/~masfz/ModelingInfectiousDiseases/Chapter6/Program_6.3/index.html

ok but add tests also at the same time, because you see that the coverage of tests decrease every time you commit code.


Support basic SQLite backend

We should add support for SQLite to allow using FerretDB without any external database process.
### Tasks
- [ ] #2495
- [ ] TBD

That depends on your definition of "embedded" :) You would still need to use a MongoDB-compatible driver.

FerretDB v1.5.0 is out with beta support for the SQLite backend!


api: improve health API

What problem does this PR solve?
Closes #2152.
What is changed and how it works?
This PR changes to check members' healthy by using healthy members instead of unhealthy members.
Check List
Tests

Unit test
Manual test

start three PDs and shut down one of them
use health API to check the health status

/merge


Remove version: NumeRe.NumeRe version <IP_ADDRESS>

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/numere/files/Stable Release/numere-win-v113-hf1.exe/download
OK
(Automated message - build 923)

URL: https://sourceforge.net/projects/numere/files/Stable Release/numere-win-v113-hf1.exe/download
Status Code: 200
(Automated message - build 902)

Close with reason: Duplicate;


Fix a panic caused by the case where a provider has zero prices

Description
Fixes a case where a provider has no valid prices for any denomination.
This came to be due to a fix that went in to filter out the osmosis candles before computing tvwap.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Going to add tests before marking r4r


Page component

Based on the Core Page component v3
Provide the following edit dialog options:

Main tab

Email subject (jcr:title)
Description (jcr:description, text area)
Language + language root
Tags
HTML ID
Template settings

Images

Page image
Thumbnail

Cloud configuration
Permissions
Remove any page properties not mentioned here

Provides the following "Email Header" component that is included by the page template as a mandatory but editable component:

Subject

RTE with only the Campaign personalization options enabled (no capability to create for e.g. bold text)
The value defaulting to the jcr:title defined on the page

Pre-header

RTE with the Campaign personalization fields (no capability to create for e.g. bold text)
The value defaulting to the jcr:description defined on the page

Provides the following page policy (aka design dialog) options:

Main tab

Client libraries

Styles tab

Requirements:

Extends the base or core page component.
Functions with Editable Templates.
Inlines all CSS into the HTML output on a STYLE element that is then further inlined as STYLE attributes on the individual elements by the CSS inliner rewriter.
Includes no JS from any of the included clientlibs.
Pre-header adds the entered content at the top of the page body as follows:${page.preheader}
Within the template editor, authors can configure an email header and/or footer that would be shared among all email pages resulting from this page template. In the template editor, authors can also pre-configure the design configuration of the components, like their style system.
Provides the following "Email Header" component that is included by the page template as a mandatory but editable component.

[x] Create template
[x] Create page component
[x] Update page dialog
[x] Provide page policies
[ ] Integrate CSS inliner
[ ] Exclude / Remove JS from result
[x] Include RTE Plugin with personalization fields
[x] Include Header / Footer
[ ] Include Email Header component

Estimation: M (10 days)

Added a "Test component" to showcase the CSS inliner.
The servlet for inlining the page has a selector "inline-styles".


Update vpn-gateway-highlyavailable.md

added that the vpn gateway will need a connection to each local gateway.

@josiasgarcia : Thanks for your contribution! The author, @yushwang, has been notified to review your proposed change.

#sign-off
Thanks!


Rely on cmsPage object instead of page.cmsPage

1. Why is this change necessary?
The page object is null in the route /widgets/cms/navigation/{navigationId}
Besides a product-listing we display sidebar-filter block in this routing so that we have filters refreshed every filter option change. Due to page object is null we have empty filters.
2. What does this change do, exactly?
Makes filters in /widgets/cms/navigation/{navigationId} route not empty.
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

I have succesfuly signed the license/cla three times but status is still pending :(

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-13997
Please use this issue to track the state of your pull request.

Hey @bystro, thanks for your contribution 💙


RN 0.59.8: postMessage result from iOS WebView is double-encoded

The data returned to onMessage WebView callback on iOS is double-percent-encoded.
React Native version: 0.59.8
Steps To Reproduce

Inject a JS into WebView that calls e.g. window.postMessage('https://test.com')
The onMessage handler will receive double-encoded value:

<WebView onMessage={onMessage} ... />

function onMessage(event) {
   console.log(event.nativeEvent.data);  // "https%253A%252F%252Ftest.com"
}

This was introduced in https://github.com/facebook/react-native/commit/61ca11965046f75e7500e5152c5f2b60df2a2cd5#diff-17ddfccc3e39071c0aa51f82677e6d68 :

Correct method to use instead of stringByReplacingPercentEscapesUsingEncoding is stringByRemovingPercentEncoding.
cc @pvinis

React Native's WebView was deprecated please migrate to Community version

me too

I had the same error with Expo SDK 33 and iOS.
So I have tried to use react-native-webview from community, and it works to me!
In this change, if you use postMessage function, you should change windows.postMessage() to window.ReactNativeWebView.postMessage().
However, it shows error message as belows when I inject this code
Error evaluating injectedJavaScript: This is possibly due to an unsupported return type. Try adding true to the end of your injectedJavaScript string..
If you try this and solve, please notice me, too!

me too

me too

改 OC原生可以避免它

Platform.OS === 'ios' ? decodeURIComponent(decodeURIComponent(event.nativeEvent.data))
: event.nativeEvent.data

Platform.OS === 'ios' ? decodeURIComponent(decodeURIComponent(event.nativeEvent.data)) : event.nativeEvent.data

I tried your method, but it still went wrong.

I solve it with
let string_data = m.nativeEvent.data;
if(Platform.OS === "ios"){
	// IOS returns the data url encoded/percent-encoding twice
	// unescape('%257B') -> %7B
	// unescape(%7B) -> {
	string_data = unescape(unescape(string_data));
}

somehow adding useWebKit={true} fix the issue

it's working for me, thanks a lot!
(using expo sdk 33 on iOS)


[REQUEST] Yakuza 3-4-6 and Like a dragon [STEAM]

like a dragon +1

like a dragon +

like a dragon + 1


fix: large chat history

Description

Fixes chat history for users with a long chat history
Cause: https://www.prisma.io/docs/accelerate/limitations#response-size-limit

5mb limit was being exceeded because so much data (every chat 'output') was being retrieved from database

https://www.notion.so/oaknationalacademy/Fix-s-chat-history-7ea2831b3b954a10ab38a3d3b0f9a72b?pvs=4
How to test

history should still work
... once merged @hannah-bethclark should check history

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


⚠ Processors has degraded performance

In 000699d, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 56 ms

Resolved: Processors performance has improved in f899dd3.


feat: 添加简单的NFT Demo

基于openzeppelin编写的合约，包含创建和铸造NFT

Thanks for your pr ,yet we have this task here.  https://github.com/Dapp-Learning-DAO/Dapp-Learning/tree/main/basic/10-hardhat-ipfs-erc721


Make an install directory changable

I want to choose an install directory. For example,
curl -s https://raw.githubusercontent.com/dawsonbotsford/swim/master/install | env usrDir=${HOME}/bin bash

Looks awesome, thanks @kawakami-o3! Can you say a sentence or two here about how to make this install directory easily configurable because of your changes? I will add it to the readme :+1:

My change use the default value of bash. The variable, usrDir will be "/usr/local/bin" if usrDir is empty. When you give usrDir a value, it is assigned to usrDir. To do it, I use "env" command. "env" allows you to execute a command with a temporally environment variable.

That's great, I was not aware usrDir works that way. So here is what I am planning on adding to the readme for, do you agree?
===
Custom Executable location
The default install location of the swim executable is /usr/local/bin, but if you would like a custom location, simply set the environmental variable named usrDir.
Example: export usrDir="/usr/sbin"

Yes, I am. It is the good readme.

Great, should be updated now and I included a new version bump to 0.4.0! Thanks again


Errant comma in config

Hi @ind1go, trailing commas are valid in JS but not valid in JSON
See http://ecma262-5.com/ELS5_HTML.htm#Section_11.1.5


java -jar  does not autocomplete

I get a very strange behaviour.
Attempting to run my custom file using some java.jar program fails to autocomplete filename even if I provide full path.
I was trying to unpack an .apk file using apktool (http://ibotpeaches.github.io/Apktool/), but autocomplete simply doesn't work

Hi! How did you figure this out?

I forgot to add a parameter to the call.
I was writing java -jar apkTool.jar my.apk instead of java -jar apkTool.jar d my.apk.

Oh nice, thanks for the info :+1:


Regression parsing array slice expressions (foo[1:2]) in 0.46

As reported by @tisonkun  in https://github.com/sqlparser-rs/sqlparser-rs/pull/1215#issuecomment-2119108808
This query ran in version 0.45.0
select make_array(1, 2, 3)[1:2], make_array(1.0, 2.0, 3.0)[2:3], make_array('h', 'e', 'l', 'l', 'o')[2:4];

The query fails in version 0.46.0
External error: query failed: DataFusion error: SQL error: ParserError("Expected variant object key name, found: 2")

It appears to have been introduced in https://github.com/sqlparser-rs/sqlparser-rs/pull/1215
@jmhain 's analysis is https://github.com/sqlparser-rs/sqlparser-rs/pull/1215#issuecomment-2125141230

This was fixed in 0.48: https://github.com/sqlparser-rs/sqlparser-rs/pull/1290


[BUG/ISSUE] Chocolatey Uninstall cancel doesn't stop uninstalling process

Before submitting the issue

[x] I have checked that the bug is not a duplicate or, if it is, it needs to be reopened
[x] I have checked that I am running the latest version of WingetUI
[x] I have tried resetting and reinstalling WingetUI and the issue persisted or happened again
[x] I have checked the FAQ section for solutions
[x] I have read the contributing guidelines and I agree with the Code of Conduct

Describe the bug
If you start an uninstalling process of a Chocolatey package and shortly after you stop it by clicking cancel, it doesn't really stop the uninstalling process, that will continue in background and eventually uninstall the package.
To Reproduce
Steps to reproduce the behaviour:

Go to Installed packages
Right click on Chocolatey package and select Uninstall or Uninstall as administrator, confirm
Click cancel
Uninstalling process will continue in background and uninstall the package. You can eventually see ps1 consoles flashing on the screen.

Expected behaviour
Stop uninstalling process

I canceled the process in the 20s timeout that chocolatey prompts when you execute the process without admin permissions: see #659


Update Docker Compose scripts to resync with DSpace/DSpace

The Docker Compose scripts in DSpace/dspace-angular have become slightly out of sync with those in DSpace/DSpace.
This PR syncs them back up, namely by updating the following:

Porting https://github.com/DSpace/DSpace/pull/3225 to dspace-angular
Updating dspace-angular compose scripts to use Solr 8.x, as we forgot to modify the scripts in this repository after https://github.com/DSpace/DSpace/pull/3020 was merged.

The best test for these changes is the GitHub CI.  I'll also run manual testing via Docker to verify it's working properly & report back in the comments.

I've fully tested these updates using the instructions at https://wiki.lyrasis.org/display/DSPACE/Try+out+DSpace+7#TryoutDSpace7-InstallviaDocker (as those instructions reference dspace-angular's Docker scripts).  All instructions now work (one last bug fix was pushed up).
Once GitHub CI approves, this will be merged.


Regarding issue #458: Zoom-In Maximum

This question is closely related to issue #458.
Is there a calculation that can be done to figure out how far in zoom should work before exceeding the canvas limits?
If yes, then I could set my slider max accordingly, right?
I'm not sure how to attack this problem, so any direction would be helpful.
In Chrome, I noticed that the waveform fails to render when peaks.length is 72840, and it succeeds in rendering when peaks.length is 60000. That seems like a lead, but it also seems wasteful to calculate peaks for several zoom levels as soon as the file is loaded. Any alternate suggestions are welcome. Thank you.

Would an alternate solution be to only render the part of the waveform showing along the horizontal scroll? Would that be too slow for playback as it scrolls?

The limitation is imposed on the Canvas size in pixels. The limit differs across the browsers. Here's a list from 2012: http://stackoverflow.com/a/11585939/352796
Thus, to calculate the maximum zoom you would multiply wavesurfer.getDuration() * wavesurfer.params.minPxPerSec * wavesurfer.params.pixelRatio (width of the waveform) and wavesurfer.params.height * wavesurfer.params.pixelRatio (height of the waveform), where wavesurfer.params.minPxPerSec is the zoom level.

Would an alternate solution be to only render the part of the waveform showing along the horizontal scroll?
Yes, that would be the solution. Peaks.js, for instance, does that.

Thank you for your response. It has been very helpful.


Added an example of using PlanOut for assignments with Mendel

First of all: Thank you very much for your contribution! Your example is super easy to understand and ran smoothly for me. I love that adding ?reset=true and refreshing you can clearly see the assignments changed by seeing the background color changes.
I will be merging it soon. Before that, could be so kind to address a few small adjustments I'll be suggest directly to the lines of code. In case you are time constraint I'll do it as soon and I merge.
Cheers!

CLA is valid!

I forgot something. There is a CLA to sign https://yahoocla.herokuapp.com/
It was supposed to be automated -- and I am double-checking why it was not triggering, it should be pretty simple to sign.

Thank you very much for the changes!


Sync Request timeout bug

Syncronous requests gets the timeout of the given retry policy when available. Retry policy requires the timeouts to be in milliseconds while the below line adds the timeout in Seconds.
https://github.com/orhanobut/wasp/blob/master/wasp/src/main/java/com/orhanobut/wasp/VolleyNetworkStack.java#L64
When I set 30000 milliseconds for the request to wait for 30 seconds, it will wait about 8 hours. :smile:

Can you tell me why did you closed it?

Oh I see. You manually merged it.


[WIP] Refactor/base class

Proposed Changes
Streamline Encoders
By introducing a BaseEncoder and a Un/SupervisedTransformerMixin that almost (c.f. below) all encoders inherit from the code is much more streamlined. This removes the boilerplate when implementing new encoders and hence makes it easier for new contributor.
Following DRY Principle
Common components are reused:

When fitting: input conversion, selecting cols, handle_missing='error', setting self.feature_names, determine and handle invariant columns
When transforming: input conversion, checking if fitted, checking correct input dimension, handle_missing='error'
get_feature_names function

No Breaking Changes
Note that the API does not change. At least I did not have to change a single test and all our tested cases still work. Since the library is rather well tested, there is reason to be optimistic that such a big change does not break anything.
Minor Changes

Rename drop_cols to invariant_cols since it is more precise to state the (only) reason why these columns are dropped.
Apply refit with different columns from LOO encoder to all others.
Introduce a metadata flag if an ordinal pre-encoder is fitted. This might be useful for further abstraction.
Started to add type annotations in some places
Streamline OneHotEncoder to return the input without converting if no categorical columns are present. All other encoders have this behaviour. Probably setting return_df = False and passing an numpy.ndarray would even throw an error in the current implementation.

Fixes
Major
#166 At least a big step towards that goal. @janmotl suggests adding metadata to encoders via base classes. I'm introducing the base classes here, but only with a minimum of metadata. More can be added very easily.
Minor
#317  Broken link
#237 Warning if transform does not do anything (see discussion)
#122 This is also partially addressed. The fix for LOO-Encoder is deployed to all other encoders. However this only ensures that the cols parameter is treated correctly when calling fit twice. Not the other params. The solution would be probably to define a fitting-params object that cleared on every call to fit (but keeping some hyperparmeters like regularisation etc).
Further ToDo's
This PR is work in progress. I just want to line out my ideas to the community as soon as possible and enter discussion as soon as possible.
When going through basically the whole code base I noticed some possible improvements and / or bugs related as well as unrelated to the restructuring:

The _score function that applies the category mapping and applies random noise to training data is copied 4 times. This can be moved to a mixin
BaseNEncoder and SummaryEncoder are based on other encoders and do not fit into the schema proposed. Probably they can be re-wirtten to use _fit and _transform functions of the underlying encoders rather than their fit and transform. This way the main encoder would still be in charge of handling inputs, dropping invariants etc. I'll look into this.
Docstrings: While most docstrings were copy-paste, some do contain valuable information. Probably the easiest is to just create a function that overwrites the base-class function, add the docstring and only do return super().function()
Docstrings (II): In transform functions often the output shape is given, I'm not sure if this is correct for all encoders since there are some copy-paste error. However this information would be useful to have (e.g. if one input column is transformed to 1 or N output columns). I'd move this to a separate issue

NB
Since this is a rather big change I'm not comfortable to do this on my own and I'd like to get some feedback on it from the community. I understand @wdm0006 does not have a lot of time but I'd be great if you could give your two cents on the idea of the change (without working through all the details - which would of course also be welcome).
I'd also be interested in feedback from recent commiters and people interested in the project, e.g. @cmougan @bmreiniger

I had also been working on this.  I focused more on the sklearn tagging system suggested in #166, but I think in the tests it would be more natural to create the list of supervised encoders by checking inheritance (in my version, just recycling TransformerWithTargetMixin as a supervised encoder class).  (Tags may be useful for more instance-dependent tags, like allow_nan which would depend on handle_missing.)
I do think it might be nicer to leave unsupervised encoders without their own class: inheriting from the base class alone should be enough?
The other main difference (in a first look) in my implementation is that I kept each encoder's fit method and just refactored (or planned to) chunks into separate methods of the base encoder(s).
It looks like python 3.5 doesn't understand the type annotations.

Thanks for sharing you code. I've read through it. I think we haven't done too much work twice, since I started with the structure and you with adding the metadata and parametrising tests.
We should decide on a way to go forward. I very much like the idea of tagging encoders and parametrising tests using the tags.
Regarding the structure and flexibility: In general I'm a fan of "Composition over Inheritance"  and hence using a flat rather than a deep inheritance structure with MixIns. As far as I know (but I'm not an expert) this is also what sklearn does. I agree that the structure does not seem very flexible, however I was planning to split the big MixIns into smaller ones. This would probably end up somewhere close to what you describe.
Regarding this

I do think it might be nicer to leave unsupervised encoders without their own class: inheriting from the base class alone should be enough?

I'm not quite sure: The way I implemented it (which is definitely up for discussion) is moving the transform method to the MixIns rather than the BaseEncoder. This did not feel quite right in the first place, but it works really well. So there has to be something if the BaseEncoder does not come with a transform method. Also why would you want to have an extra class for supervised but not for unsupervised? Shouldn't it be symmetric? The only difference is the signatures of some methods.
In general I think we're heading into a similar direction and merging our approaches will be fruitful.
And yes, the python 3.5 is really annoying :D I've check it supports type hints, but probably does not yet support the synthax for declaring abstract class variables.

Also why would you want to have an extra class for supervised but not for unsupervised? Shouldn't it be symmetric? The only difference is the signatures of some methods.

An unsupervised encoder basically is just an encoder.  It's the supervised ones that need some additional handling.  Your unsupervised transform overlaps a lot with the supervised one, and refactoring them into the base encoder class makes sense to me; and after doing that, there's nothing really left for the unsupervised class to do.

This looks like a great PR! 1500 lines of code removed already, should help a lot with maintinability.


Possible causes of '[Vue warn]: data functions should return an object:'

I'm trying to login and get authenticated to an admin dashboard. When I click Login I get the token back as expected, but then, on console, I see: [Vue warn]: data functions should return an object:, and a list of red lines like this: [Vue warn]: Property "visible" must be accessed with "$data.visible" because properties starting with "$" or "_" are not proxied in the Vue instance to prevent conflicts with Vue internal. (found in ).
And 8 more like this one.
I understand why data needs to be a function and not an object in Vuejs, I searched in Google and I just found clarification on this. But I double checked my code and data is in fact a function returning an object. I don't know then, if all the other warnings below are there because of it. Also, the login, if successful, should redirect to the dashboard page. It actually does (the url on the browser changes), but the view does not update.
Are all these things related?
I am now using a real api, before, with a fake one, everything was working fine. Sorry if for someone this can sound a stupid question, yes I am new to Vue and programming in general, but I don't know what to look at on Google to solve it.
Thank you.
I'm running on localhost, the apis point to a server with a different address (I avoided CORS running Chrome without it). I checked all the data() in my code to be sure there wasn't any of them declared as objects but everything looks fine.

[Vue warn]: The "data" option should be a function that return...
Reference to official documents or #data-Must-Be-a-Function
Restriction: Only accepts Function when used in a component definition.

[Vue warn]: Property "visible" must be accessed with "$data.visible" because properties starting with "$" or "_" a...
VUE source code
  const warnReservedPrefix = (target, key) => {
    warn(
      `Property "${key}" must be accessed with "$data.${key}" because ` +
      'properties starting with "$" or "_" are not proxied in the Vue instance to ' +
      'prevent conflicts with Vue internals' +
      'See: https://vuejs.org/v2/api/#data',
      target
    )
  }

官方文档

After the instance is created, the original data object can be accessed as vm.$data. The Vue instance also proxies all the properties found on the data object, so vm.a will be equivalent to vm.$data.a.

Properties that start with _ or $ will not be proxied on the Vue instance because they may conflict with Vue’s internal properties and API methods. You will have to access them as vm.$data._property.

Message.error(error || 'Has Error') 换成 Message.error('Has Error')

Any solution to this?  I'm getting this warning with a property named "card_on_file" - clearly not starting with $ or _, which feels very mysterious.
Vue 2.6.11


Embedding link to Other referred essential information

The 'Set up a Service Bus Connection' refers to earlier step Obtain the credentials. Embedding a link to the relevant page which gives details on 'Obtaining Credentials' would make it easy for users to gather all related information.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 87507e10-00e2-250a-2074-bb26bbea5a03
Version Independent ID: f21aee07-f75d-093e-cdae-cd97c52d34cf
Content: Use Azure Service Bus topics with azure/service-bus Node.js package
Content Source: articles/service-bus-messaging/service-bus-nodejs-how-to-use-topics-subscriptions.md
Service: service-bus-messaging
GitHub Login: @axisc
Microsoft Alias: aschhab

@HPatki Thanks for the feedback! We are currently investigating and will update you shortly.

It's part of the prerequisite now. You get the connection string by following steps here:
https://docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-quickstart-topics-subscriptions-portal#get-the-connection-string

@mike-urnun-msft - please let me know if you want me to make the update or you can. Thanks.

Thanks, @spelluru! re-assigning to you, please make the update :)

Done. Updated.
#please-close


Remove network images from cache on any exception during loading

Description
Makes network image resolution evict from the cache on any exception - previously, we'd miss various things like socket exceptions during an http request and get stuck with a bad image in the cache.
This means that we'd also refuse to keep a 0 length image in the cache even if everything else succeeds, which seems fine to me - it seems likely that is either a network error or a cheap thing to retry, and we still report the error.
Related Issues
Fixes #47270 again
Tests
I added the following tests:
Test covering eviction of the image from the cache on a socket exception
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

why every time rethrow when handle 404 image url error, can anyone guide me how to solve this issue...


feat: add cloudquery/cloudquery

https://github.com/aquaproj/aqua-registry/pull/5993 cloudquery/cloudquery: The open-source data integration platform for security and infrastructure teams
$ aqua g -i cloudquery/cloudquery

Thanks.

cli/v0.32.5
cli_Darwin_arm64.zip
Asset name format was changed.
I guess we have to use version_overrides.

v0.32.3

cloudquery_Darwin_arm64.zip

cli/v0.32.5

cli_Darwin_arm64.zip

cli/v0.32.8

cloudquery_Darwin_arm64

It is a little complicated. Let me try to fix.

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.47.0


set_mouse_mode behavior

Setting mouse mode to MOUSE_MODE_CAPTURED or MOUSE_MODE_CONFINED systematically warps the cursor to the center of the screen (at least on x11). I think that should not happen when the cursor is already inside the window. Clamping might also be more appropriate.

(Since Noshyaar keeps closing this, I am going to continue to post it until one thread remains visible for discussion, no need to censor)

Lead developer sent this message after a user was giving his honest feedback via this reddit thread.
They also said (I kid you not) -- giving feedback is "bullying",  a "professional whiner", and "ranting". Reduz's passive aggressive tone and insulting behavior is rampant there.  I already terminated my patreon account because of this, and not saying you should either (FLOSS projects) are great, however, for the lead developer to act like this towards people giving feedback.. is not okay!
Let me re-iterate, not saying that you should stop giving feedback, but I just want this message to be seen across github and reach as far as possible. Because this kind of communication and how the lead developer treats a regular Godot developer is not right at all. I've never seen this behavior from a FLOSS project lead developer.  It's a shame, because the guy giving feedback is just a random person anyway, and the way the project managers and lead developers took it so personally like it was an attack (when it wasn't), means the lead developers/contributors are possibly not viewing your issues with the right mindset.
Also, after reading the last paragraph from this post by akien

From here on, we will avoid responding to xxx's threads on the various Godot communities, and focus instead on the huge amount of constructive feedback and bug reports that we get from other users. This will likely overlap with some of the issues mentioned above, which will thus be fixed eventually, but we won't accept being bullied into working on them now when we have different priorities. Users are free to disagree with our priorities, but in the end the contributors are the ones making the call.

He's saying they will now favor/prioritize issues based on the user reporting it, and not the issue itself. (Extreme prejudice), which will not be good for the project because the issue should be irrelevant to the user reporting it. Just because if a user who has a snotty attitude that is reporting a huge client crash error, doesn't mean that client crash error should not be taken lightly.
He's also stating that anyone giving hard feedback is now "bullying" them. Which is not true whatsoever.
Just wanting to post this to get the word out there, hopefully some things change. @reduz @akien-mga  This is just unbelievable :(

With MOUSE_MODE_CAPTURED we need to be able to get the relative movement still with each motion event, this relative movement could be returned as zero after the mouse has traveled to the edges of the window if we allowed that. If there is a way to do this and avoid that problem then that's fine otherwise I don't think there's an issue with the current behavior of that mouse mode.

I don't think this is the case any more on master 584ca0f156cec64c259382895e105cf27566a987.
@letheed could you check again?

Fixed in 7d6364d431a16e619db7cd1092192ea647dc11b8.
This issue can be closed now.

@NilsIrl I'll close it then. Thanks for checking the issue's status :slightly_smiling_face:


feat: add func of getting specified type from workspace genericConfig

What type of PR is this?
/kind feature
What this PR does / why we need it:
add func of getting specified type from workspace genericConfig.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

54 of 59 (91.53%)  changed or added relevant lines in 2 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.2%) to 74.647%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/workspace/util.go
51
56
91.07%

Files with Coverage Reduction
New Missed Lines
%

pkg/workspace/util.go
1
51.72%

pkg/engine/operation/parser/parser.go
3
79.27%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
7240

Relevant Lines:
9699

💛  - Coveralls


reset value or set value

I want to reset a Value or set the Value to the first option by click on a Link
my Code:
$('#myButton').click(function(){
	$('select option[value="--"]').attr('selected',true);
	$('select').dropkick('refresh');
});

In the select element the option is selected but the dropkick dropdown element doesn't show this option.
I also tried it with 'reset' but then dropkick displays the value which was selected at page loading and not the '--'

You need to use the dropkick methods to affect the dropkick element. Our new docs are still WIP, but the method you're looking for is the #select() method.
This would be the equivalent with jQuery:
$('select').dropkick('select', '--');

#reset() seems to be working as it should - "Resets the Dropkick and select to it's original selected options." Unless of course your option is the first one and you're using the method with the optional argument.

thank you for the fast responce.
The problem ist that .dropkick('reset',true); does the same as .dropkick('reset');
It selects the original option. But i need the first option selected

If you do do $("#select").dropkick("reset"); it will reset to the originally selected option. If you do $("#select").dropkick("reset", true); it will reset to the first option in the select.
Checkout the JSBin below to see an example :)
http://jsbin.com/jizuhihobu/1/edit

@strigga, if you're still having trouble would you mind providing a JSBin recreating your issue? Everything seems to be working as expected with our tests.

here i have set it up at JSBin: http://jsbin.com/yewimuyibe/1/edit?html,css,js,output

hello,
i now use the JS and the CSS from the hosted CDN and now it works. i think i had an old version.
thank you!

Glad everything was sorted out. 👍


load issue on macOS

This could be related to recent tag on GLAbstraction METADATA.jl/pull/15225 , but:
julia> using Makie
INFO: Precompiling module Makie.
WARNING: Method definition center!(Any) in module GLAbstraction at /Users/abradley/.julia/v0.6/GLAbstraction/src/GLCamera.jl:588 overwritten in module Makie at /Users/abradley/.julia/v0.6/Makie/src/plotsbase/scene.jl:177.
ERROR: LoadError: LoadError: TypeError: UnionAll: expected Type, got Int64
Stacktrace:
 [1] include_from_node1(::String) at ./loading.jl:576
 [2] include(::String) at /Applications/Julia-0.6.app/Contents/Resources/julia/lib/julia/sys.dylib:?
 [3] include_from_node1(::String) at /Applications/Julia-0.6.app/Contents/Resources/julia/lib/julia/sys.dylib:?
 [4] include(::String) at /Applications/Julia-0.6.app/Contents/Resources/julia/lib/julia/sys.dylib:?
 [5] anonymous at ./<missing>:2
while loading /Users/abradley/.julia/v0.6/Makie/src/plotsbase/conversions.jl, in expression starting on line 40
while loading /Users/abradley/.julia/v0.6/Makie/src/Makie.jl, in expression starting on line 17
ERROR: Failed to precompile Makie to /Users/abradley/.julia/lib/v0.6/Makie.ji.
Stacktrace:
 [1] compilecache(::String) at /Applications/Julia-0.6.app/Contents/Resources/julia/lib/julia/sys.dylib:?
 [2] _require(::Symbol) at /Applications/Julia-0.6.app/Contents/Resources/julia/lib/julia/sys.dylib:?
 [3] require(::Symbol) at /Applications/Julia-0.6.app/Contents/Resources/julia/lib/julia/sys.dylib:?

You'll need the newest version of makie:
99bb74ad39ab6d40b19efe8e1abbc26548eede8c

ah, thanks, and sorry for the noise


compute-vm: Secure tags with "For use with network firewall"  can't be used and creates crash

Describe the bug
If a secure tag, where "For use with network firewall" has been set, is used in variable tag_bindings it creates  a crash.
Result
Error: Error waiting for instance to create: Internal error. Please try again or contact Google Support. (Code: 'Internal error')

Tags of that kind can not be added to disks.

This sounds more like a provider issue.
Can you share your code? Please provide all the details requested when you open a bug.

I don't think the code itself would be any helpful. Let me instead explain how to replicate the error:

Create a secure tag via console and set the "For use with network firewall" flag.
Create a VM via the compute-vm terraform module and set the tag_bindings variable like:
tag_bindings = {
"tagKeys/281483402711742" = "tagValues/281479473111820"
}
deploy the VM. It will crash with:
Error: Error waiting for instance to create: Internal error. Please try again or contact Google Support. (Code: 'Internal error')

On the one side it is a provider or even API issue.
On the other side (and that is the reason I opened here this issue) it is obvious that a secure tag which used in firewall rules makes no sense to e attached to a disk. I saw it also stated somewhere in the documentation but can not find the place anymore.
But it makes perfect sense to attach a tag which is use in firewall rules to an instance. The modules in the current form does not allow to attach these kind of tag solely to the instance and not to the boot disk. The module always attaches the tags to the instance AND the boot disk. This is the issue.

From your description it seems the issue comes from this line
@ludoo wdyt if we keep the existing tag_bindings only for instance-level tags? If needed we can later introduce another variable (or field) for disk-level tags.

I think this might be a short example for that:
module "org" {
  source          = "./fabric/modules/organization"
  organization_id = var.organization_id
  tags = {
    environment = {
      description = "Environment specification."
      iam         = null
      values = {
        dev  = null
        prod = null
      }
    }
  }
}

module "simple-vm-example" {
  source     = "./fabric/modules/compute-vm"
  project_id = var.project_id
  zone       = "${var.region}-b"
  name       = "test"
  network_interfaces = [{
    network    = var.vpc.self_link
    subnetwork = var.subnet.self_link
  }]
  tag_bindings = {
    (module.org.tag_keys["environment"].id) = module.org.tag_values["environment/prod"].id
  }
}

I tried this one, and actually no failures occured.

From your description it seems the issue comes from this line
@ludoo wdyt if we keep the existing tag_bindings only for instance-level tags? If needed we can later introduce another variable (or field) for disk-level tags.

yep it makes sense, I also remember discussing some weirdness and/or incomplete support with instances and tags internally not too long ago, maybe we should resurface that discussion and attach it to a bug here

Scrap my comment, disks and instances need the same tag-level support or IAM conditions won't work.
What Wiktor mentions above is what I remember: tag-based support is for resource management tags, not VPC-scoped tags.

So, do I understand correctly, that we have two separate use cases which use Resource Manager tags here:

For IAM conditions, which requires the tags to be set on instance and disk level (preferably the same)
For firewall policies, which requires VPC-scoped tags assigned only on instance level (that's how I understood this docs)

So to support both of the use cases we need to allow to set the tags only on instance level, and explain to the users, which to use when, or have a separate field vpc_scoped_tag_bindings (not pretty) which goes only to the instance level.

Yes. But for network scoped tags there's some complications in the provider, or there were up to a few weeks ago: from my recollection you can set them in the instance resource, but then any change triggers recreation. While the external binding resource is not available, or did not work last time I tried it. I need to fetch the chat history for the discussion.

This is still true and is documented here

From your description it seems the issue comes from this line
@ludoo wdyt if we keep the existing tag_bindings only for instance-level tags? If needed we can later introduce another variable (or field) for disk-level tags.

yes

Nope. 😀 The current interface is used across modules for resource management scoped tags. We need to define a new one for network scoped tags.

network_tag_bindings?

network_tag_bindings?

Or actually firewall_tag_bindings to not confuse with the old school network tags.


Panasonic PT-VZ580 wont accept password

@haakonnessjoen I've now tested the recent password fix on my Panasonic PT-VZ580, and unfortunately the issue remains.
Since companion doesn't output much debugging info, it's hard to see what happens.
I've tested with a generic pjlink client, and that works fine with my projector.
I did try to uninstall companion before installing the latest version, to make sure that I have the latest version of the pjlink plugin.

Is this issue still valid?


Fixed warning during compile for community devices

Description of changes
When compiling the sources for community devices a warning is generated.
This PR fixes the warnings.

Should this not be using the F() macro?

Oh yes.


Adds DTLS1.3 to ssl protocol to text structs

Another dtls 1.3 specific update. It is dependent on https://github.com/openssl/openssl/pull/22259

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 61 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 92 days ago

I don't believe the CI failure is relevant.

@mattcaswell Please reconfirm

This pull request is ready to merge

Pushed to the feature/dtls-1.3 branch.


gpio: intel: fix reg address for Apollo Lake

On Apollo Lake, each GPIO controller has more than 32 pins.
But Zephyr API can only manipulate 32 pins per controller.
So the workaround is to divide each hardware GPIO controller
into 32-pin blocks so each block has a GPIO driver instance.
Compounding to the issue is that there cannot be two device
tree nodes with same register address. So another workaround
is to increment the register addresses by 1 for each block.
So when mapping the address, the lowest 8-bit needs to be
masked to get the actual hardware address.
Fixes #28551
Signed-off-by: Daniel Leung<EMAIL_ADDRESS>

it seems that this patch didn't fix issue #28551, I tested commit 7295c19 on up_squared, the issue still existed. I will track it using #35970 .
*** Booting Zephyr OS build v2.6.0-rc1-200-g7295c19893b2  ***
ERROR: cannot set HAT pin 35 to OUT (-22)

it seems that this patch didn't fix issue #28551

I am sorry I made a mistake, I forgot to set up gpio configuration, #28551 was fixed by this PR.


Adding mlflow tests

@agola11 + @hwchase17 + @dev2049  thanks for the review and pushing the PR over the finish line!
I'd like to add a few more tests in a follow up - are you willing to add mlflow + azureml-mlflow packages as test dependencies or should we explore some other options?
Originally posted by @akshaya-a in https://github.com/hwchase17/langchain/issues/4150#issuecomment-1540983848

test dependencies should be ok!

Sorry for not following up in this thread, for anyone that ends up here later, there are tests being added for the most important part (serialization) here, so decided not to duplicate efforts in this repo for now while things are moving quickly: https://github.com/mlflow/mlflow/tree/master/tests/langchain


Empty file uploads leads to error

When trying to upload an empty file (ex: error logs... in lucky cases), i'm getting error:
 File "/var/www/venv/local/lib/python2.7/site-packages/slacker/__init__.py", line 427, in upload
   files={'file': f})
 File "/var/www/venv/local/lib/python2.7/site-packages/slacker/__init__.py", line 70, in post
   return self._request(requests.post, api, **kwargs)
 File "/var/www/venv/local/lib/python2.7/site-packages/slacker/__init__.py", line 62, in _request
   raise Error(response.error)
Error: no_file

Just tried to put a symbol in file and error disappeared.
Thank you.

The error code relates to something at Slack's end and isn't related to the library so it would be up to you to choose how to handle it.