Improve environment backup and restore

What was the end-user problem that led to this PR?
The problem was that the specs were removing some environment variables for every tests, but not all of them, causing problems on some environments (see #7251).
What was your diagnosis of the problem?
My diagnosis was that we don't really need to do this.
What is your fix for the problem, implemented in this PR?
My fix is to stop doing it, and fix the very few specs that failed after the change. One of the failures actually led to a bug fix (I think), see 6c27ebb.
Why did you choose this fix out of the possible options?
I chose this fix because I think it's cleaner than adding a public constant in lib/ only to fix a problem with the specs, like it's done in #7251.

@bundlerbot r=hsbt


why always error "Record not found"

Size of data is 165
JSON Response is: {
    records =     (
                {
            reason = "Record not found";
            recordName = "2/2/b5fba8fe2496ca996ebf4dbb82e134ed777fca0b";
            serverErrorCode = "NOT_FOUND";
        }
    );
}
CloudKit query for 帧享.app (2/b5fba8fe2496ca996ebf4dbb82e134ed777fca0b) failed due to "Record not found".
Could not find base64 encoded ticket in response for 2/b5fba8fe2496ca996ebf4dbb82e134ed777fca0b
The staple and validate action failed! Error 65.

error log https://gist.github.com/722433589a857599387e4e00b6c21743

sign.coffee
#!/usr/bin/env coffee

import {appBundleId} from './config.mjs'
import {notarize} from 'electron-notarize'
import {join,dirname} from 'path'
import {signAsync} from 'electron-osx-sign'
import thisdir from '@rmw/thisdir'

DIR = thisdir import.meta

do =>
  {argv,env} = process
  [app] = argv.slice(2)

  entitlements = join DIR,'i.plist'
  console.log 'sign', app
  await signAsync {
    app
    verbose:true
    entitlements
    'entitlements-inherit':entitlements
  }
  console.log 'signed'

  await notarize({
    appBundleId
    tool:'notarytool'
    appPath:app
    teamId: env.teamId
    appleId:env.appleId
    appleIdPassword:env.appleIdPassword
  })

  process.exit()

pkgutil --check-signature  ../../release/帧享-darwin-x64/帧享.app

i fixed by add hardenedRuntime:true in sign
await signAsync({
    app,
    verbose: true,
    hardenedRuntime: true,
    entitlements,
    'entitlements-inherit': entitlements
  });


Question about the group function

After I pull the bot into the group, anyone who asks a question, the bot replies "You are not authorized to use this bot.", what is the reason? How can he be used in groups?

Make sure you're leaving the TELEGRAM_ID env var empty


[Bug]

Search before asking

[X] 我在 issues 列表中搜索，没有找到类似的内容。
I searched in the issues and found nothing similar.

Pake version

System version
如图
Node.js version
v22.4.0
Minimal reproduce step
npm install -g pake-cli
What did you expect to see?
npm install -g pake-cli
What did you see instead?
npm install -g pake-cli
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

test


Terraform registry version is still 6.0.1

Description
Hello!
I would like to update this module to version 6.1.0, but the latest version available on the Terraform registry is still 6.0.1.
When I update my tf file and run terraform init, I get the following error:

There is no available version of module "registry.terraform.io/terraform-aws-modules/datadog-forwarders/aws//modules/log_forwarder" (datadog_log_forwarder.tf:5) which matches the given
version constraint. The newest available version is 6.0.1.

Could you update the version of Terraform registry?

[x] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]: 6.1.0
Terraform version: Terraform v1.9.0 on darwin_arm64
Provider version(s):

provider registry.terraform.io/hashicorp/aws v5.64.0

provider registry.terraform.io/hashicorp/null v3.2.2

provider registry.terraform.io/hashicorp/tls v4.0.5

Reproduction Code [Required]
module "datadog_forwarders_log_forwarder" {
  source  = "terraform-aws-modules/datadog-forwarders/aws//modules/log_forwarder"
  version = "6.1.0"

  runtime                   = "python3.11"
  forwarder_version         = "3.122.0"
  dd_api_key_secret_arn     = data.aws_secretsmanager_secret.dd_api_key.arn
  log_forwarder_bucket_tags = {
    Sample = "tag"
  }
}

Steps to reproduce the behavior:

Execute terraform init.

Expected behavior
Finish without an error, like the example below (this example is for terraform-aws-modules/datadog-forwarders/aws 6.0.1):
❯ terraform init
Initializing the backend...
Initializing modules...
Downloading registry.terraform.io/terraform-aws-modules/datadog-forwarders/aws 6.0.1 for datadog_forwarders_log_forwarder...
- datadog_forwarders_log_forwarder in .terraform/modules/datadog_forwarders_log_forwarder/modules/log_forwarder
Downloading registry.terraform.io/terraform-aws-modules/s3-bucket/aws 3.6.1 for datadog_forwarders_log_forwarder.this_s3_bucket...
- datadog_forwarders_log_forwarder.this_s3_bucket in .terraform/modules/datadog_forwarders_log_forwarder.this_s3_bucket
Initializing provider plugins...
- Reusing previous version of hashicorp/null from the dependency lock file
- Reusing previous version of hashicorp/aws from the dependency lock file
- Reusing previous version of hashicorp/tls from the dependency lock file
- Using previously-installed hashicorp/null v3.2.2
- Using previously-installed hashicorp/aws v5.64.0
- Using previously-installed hashicorp/tls v4.0.5

Terraform has been successfully initialized!

You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.

If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.

Actual behavior
❯ terraform init
Initializing the backend...
Initializing modules...
╷
│ Error: Unresolvable module version constraint
│
│   on datadog_log_forwarder.tf line 5:
│    5: module "datadog_forwarders_log_forwarder" {
│
│ There is no available version of module "registry.terraform.io/terraform-aws-modules/datadog-forwarders/aws//modules/log_forwarder" (datadog_log_forwarder.tf:5) which matches the given
│ version constraint. The newest available version is 6.0.1.

Additional context
I'm using terraform-aws-datadog-forwarders module from registry.terraform.io . If there is a walk around, let me know.

Fixed with magic stick :)

Thank you!


Style Formatting Getting Copied into User Query Input on Copy-Paste

When a user copies content and pastes it into the query input, the style formatting (e.g., font, color, background) is being retained.
To Reproduce
Steps to reproduce the behavior:

Go to Preview page
Copy styled content from any page and paste

Expected behavior
We want only the plain text to appear in the input box without any style formatting
Screenshots

@agberoz  Could you please assign it to me ? Also, mention the file in which the bug resides. Would save me so time.

hi @dexterousdhruv,
Assigning this issue to you, happy coding
If any assistance is required meet our team here: https://join.slack.com/t/theailounge/shared_invite/zt-2ogkrruyf-FPOHuPr5hdqXl34bDWjHjw

@agberoz  Kindly review the PR.

@sethumadhavan-k ,  Please check the PR

@sethumadhavan-k I've created a fresh PR. Please review it.


acknowledge message

I couldn't find how to acknowledge a message, suggestion ?
channel.basicAck(...)

I'm attempting to do the same thing, re issue #117
I think we are going to have to update the library itself.  It may be easier to do it with the receiver approach than the distributed approach.
I've only looked at it for a few hours, but here is what I think needs to be done

create a new ack type which requires an ack, but does not send an ack on processDelivery
save the deliveryTags
keep the channel open
Create a method to send the acks to the open channel after you have processed your data.

But let's wait to hear from some of the contributors as they may have an easier route.

This is a feature for a minimum level of reliability.
In my case, I'm sending messages to Kafka using the asynchrony batch feature, after the batch is successfully committed onto the broker, it send the ack to RabbitMQ. This process works through a callback implementation.
broker.send("id",callback())
For Spark Streaming, the callback need to be serializable. RabbitMQ don't have a feature which we can send a bulk of deliveryTags at once to the server, we just can ack all the preview tags which the context is based on the channel.
On the section "Design Patterns for using foreachRDD
" https://spark.apache.org/docs/latest/streaming-programming-guide.html
We can reuse a previews RabbitMQ's Channel object preview created to achieve the desired design, this Channel object can be preview created and used for the creation of the streaming. I'm not sure if the RabbitMQ deliveryTags are tied to a specific Channel or not.

I was able to achieve this with the receiver approach.  Basically you need to comment out the basicAck in the "processDelivery" method.

Next make the channel available via the Consumer object:

And finally ack the messages after processing the data.

The distributed approach is not reliable at all. It send the ack just after consuming the message from the queue without doing anything else. If you don't want to loose any message, you have to use the receiver approach.
@kaveh-hariri removing the ack in the receiver you are delegating the responsibility to send the ack to the spark developer. In your case its works, but you are creating an extra job just to send the acks processed by the previous job. For example, if you set the property spark.streaming.concurrentJobs to 2, It will not work correctly, because both job will run concurrently.
I think the proper way to make sure you don't lost any message is to set the property spark.streaming.receiver.writeAheadLog.enable. Using this property Spark will save the message consumed before sending the ack. Obviously, this solution will decrease the performance cause spark will have to save every message to your HDFS/S3/local filesystem....

@fjavierjimenez Hi, yes, you need the ack job to run after ingestion job is complete, so you need to leave the concurrentJobs setting at its default of 1.  From my understanding that setting is experimental anyway.
I'm currently running this job in a production environment, using a 1 minute window.  There are 3 distinct stages to the job, a repartition of the data, an ingestion, and an ack.  The ack stage for even 100k messages (roughly 35kb per message), is taking .5 seconds or less, so even though it's a separate stage, it's not taking much time/resources.
Regarding the write ahead log, two questions:

How are we sure that the ack is sent post data being written to the write ahead log?  Looking at the receiver approach, the message is acked immediately after it is stored in memory.  Not sure if there will still be zero data loss if the job is killed while consuming.
How do we make sure the next time the job is started, it reads the data from the write ahead logs?

@kaveh-hariri, At the last stage, I get this error that "getOrElse" can not apply on a row of RDD, Do you have a solution, and my next question is that, is "basicAck" method of Consumer parallelizable?


🌐 Add Japanese translation for Tutorial - Header Parameters

This PR translates the tutorial/header-params.md to Japanese.
Please review tutorial/header-params.md.
Issue #1571 generates a new directory to support Japanese.
PR #1656 generates a new directory for tutorial.
I will fix dependency after #1571 & #1656 are merged.

Awesome! Thank you @SwftAlpc ! :rocket: :bow: :coffee:
And thanks @Attsun1031 and @tokusumi for the reviews! :clap: :clap: :cake:


InvalidPathException when doing nativeCompile on windows

demo (1).zip
i get the demo.zip from spring initializr. and i added these dependencies .

then i open the "x64 Native Tools Command Prompt“ and cd to the demo dir. execute gradle nativeCompile  and i got this error.

i tried it on linux and didn't found this proplem. so i thought this is a bug relate to path.

As you're using Spring Boot 3.0.0, i assume you are in the wrong issue tracker. Spring Native is for Spring Boot 2.x only. Please report that issue on the Spring Boot bug tracker at https://github.com/spring-projects/spring-boot/issues. Thanks!


I install geode but button dont show

Geode Issue

[x] I confirm that this bug is NOT related to a mod but directly to Geode Loader itself.

Platform
Windows
SDK commit
No response
Geode Version
2.0.0-beta27
Mods Installed
No response
Expected Behavior

Actual Behavior

Steps to Reproduce
I installed geode
Open GD (Steam)
Geode button dont shows itself
Additional Information
No response

2.0.0-beta.27

?

what are you doing there

what??

what??

Update Geode to version 3.7.1 (go to the installation page and reinstall it) and update GD to 2.206 if you didn't already do it. If that doesn't work try to disable mods that modify the menu (press shift while opening GD and click "Yes" to enable Safe Mode so you can recognize the mods more easily)

thanks :)


Training epochs

Thanks for sharing the codes. I wonder how many epochs the toy models are trained on noise artifacts and backdoor artifacts?
I trained a 4-layer model with backdoor artifacts and found that after 5 epochs, the percent of memorized sequence saturates to 100%, although the default epoch is 200 according to the memorization_in_toy_models.py.
By the way, the IndexedDataset class is missing in the provided source code.

Preprint: https://arxiv.org/pdf/2410.02159
RE Training epochs:

Math+Noise: 3500 epochs
Math+backdoor 500 epochs
Language+Noise: 100 epochs
Language+Backdoor: 50 epochs
To see how many epochs we trained on I refer you to Figure 6 in appendix A.1 (screenshot below.

RE Backdoor memorizing saturating to 100% in 5 epochs: We have the same behavior on our end (see Figure 6). Noise artifact memorization on the other hand takes a long time in comparison to backdoor artifacts. We designed the TinyMem model suite to have a wide variety of memorization styles/artifacts so that mitigation strategies could be tested in a wide variety of settings. Our results in Tables 1 & 2, Figure 3 in the preprint show that the backdoor artifacts are harder to mitigate compared to the noise artifacts. We speculate that this is because backdoor artifacts are easier to memorize than noise artifacts. Because we include both easy-to-memorize and hard-to-memorize artifacts in TinyMem, we found that the memorization mitigation strategies that worked best on production-grade models were those that did well in both settings.
RE  IndexedDataset: Can you point me to the line in the code where you are seeing this class? It is possible that you are seeing an outdated artifact from an earlier version of the code that got left behind.
Feel free to let me know if you have any more questions.

In memorization_in_toy_models.py, the IndexedDataset is imported in line 17 and referenced in line 157. There seems not a src.data.IndexedDataset module.

Thanks for letting me know about that import. It was indeed an old import that should have been removed.
I fixed the issue in the latest commit: https://github.com/msakarvadia/memorization/commit/fa4cc797db2e95ec0498350db58c5da12ec2d811
This should resolve the issue; please let me know if there are any further questions.


Oracle Analytics Server 6.4 public dockerfiles

Made additional fixes as per the comments in the older pull request - https://github.com/oracle/docker-images/pull/2431

Please resolve the remaining linter issues.

You'll need to make your Oracle email address publicly visible in your GitHub profile settings to stop the OCA Bot from flagging your commits. Please review the "publicizing your organization membership" documentation for instructions on how to do so.

If you update the main branch of your fork, you'll be able to use the newly added lint script to run the linters locally before you push your commits too. See the contribution guide also in the root of the repo for more info.

@Djelibeybi I have fixed the remaining lint issues and also myself & hemala have publicized our oracle membership in our github profiles.

It would also be great if you could pull the latest commits to the main branch of your fork, then rebase this branch on top of the updated main branch so that the PR picks up the updated GitHub Actions checks.

I rebased your branch for you. You'll need to run git pull to bring any local clones of the repo up-to-date though.

@Djelibeybi I have made the requested changes (haven't pushed them yet), but still when I run the lint script locally, then its failing with following two errors -
`2022-11-30 15:57:49 [INFO]   File:[/tmp/lint/OracleAnalytics/dockerfiles/6.4/Dockerfile]
2022-11-30 15:57:49 [ERROR]   Found errors in [hadolint] linter!
2022-11-30 15:57:49 [ERROR]   Error code: 1. Command output:
/tmp/lint/OracleAnalytics/dockerfiles/6.4/Dockerfile:53 DL3003 warning: Use WORKDIR to switch to a directory
2022-11-30 15:57:49 [INFO]   File:[/tmp/lint/OracleAnalytics/patches/6.4-patch/Dockerfile]
2022-11-30 15:57:49 [ERROR]   Found errors in [hadolint] linter!
2022-11-30 15:57:49 [ERROR]   Error code: 1. Command output:
/tmp/lint/OracleAnalytics/patches/6.4-patch/Dockerfile:68 DL3003 warning: Use WORKDIR to switch to a directory`
Please suggest what should be done for these errors as using WORKDIR to switch directory instead of using cd would increase number of layers (esp. for commands with many cd).

Please suggest what should be done for these errors as using WORKDIR to switch directory instead of using cd would increase number of layers (esp. for commands with many cd).

You don't need to use cd at all. Your commands can be modified to specify the right path instead of relying on being run in that path.
For example, in your main Dockerfile, use this:
RUN unzip "/u01/$BI_DISTRO_ZIP1" && \
    "$JAVA_HOME/bin/java" -jar "/u01/$BI_INSTALLER1" -silent -responseFile /u01/install.file \
      -invPtrLoc /u01/oraInst.loc -ignoreSysPrereqs -force -novalidation ORACLE_HOME="$ORACLE_HOME" \
      INSTALL_TYPE="Oracle Analytics" && \
    rm "/u01/$BI_DISTRO_ZIP1" "/u01/$BI_INSTALLER1" /u01/oraInst.loc /u01/install.file

Then, for your patching Dockerfile you can change each patch block to something like this:
RUN ...
    ...
    unzip "/u01/$PATCH_PKG_33958532" && \
    "$ORACLE_HOME/OPatch/opatch" apply -silent /u01/33958532 && \
    rm "/u01/$PATCH_PKG_33958532" && \
    ...

Given that you install unzip in your main Dockerfile, it makes sense to use it to unpack your patches in your patching Dockerfile too.

@Djelibeybi I have now made the suggested changes to rid of "cd" in the RUN commands used in Dockerfiles and tested the same.


Heapless String doesn't implement From trait

Hi, I'm fairly new to Rust, so I apologize in advance if my issue is not clear or not exact.
I've being trying to use serde-json-core with Zenoh and I get this error when I try to publish a simple struct with a String and an u64.
error[E0277]: the trait bound `Value: From<serde_json_core::heapless::String<_>>` is not satisfied
   --> src/nodes/node_example2.rs:39:27
    |
39  |             publisher.put(serde_json_core::to_string(&node_msg_example.into()).unwrap()).res_sync().unwrap();
    |                       --- ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `From<serde_json_core::heapless::String<_>>` is not implemented for `Value`
    |                       |
    |                       required by a bound introduced by this call
    |
    = help: the following other types implement trait `From<T>`:
              <Value as From<isize>>
              <Value as From<i8>>
              <Value as From<i16>>
              <Value as From<i32>>
              <Value as From<i64>>
              <Value as From<usize>>
              <Value as From<u8>>
              <Value as From<u16>>
            and 13 others
    = note: required for `serde_json_core::heapless::String<_>` to implement `Into<Value>`
note: required by a bound in `Publisher::<'a>::put`
   --> /home/ludovic/.cargo/registry/src/index.crates.io-6f17d22bba15001f/zenoh-0.10.1-rc/src/publication.rs:415:20
    |
413 |     pub fn put<IntoValue>(&self, value: IntoValue) -> Publication
    |            --- required by a bound in this associated function
414 |     where
415 |         IntoValue: Into<Value>,
    |                    ^^^^^^^^^^^ required by this bound in `Publisher::<'a>::put`

For more information about this error, try `rustc --explain E0277`.

From my understanding, the String implementation in heapless needs to satisfied the From trait. Although maybe I didn't create my
string properly or didn't give it properly to Zenoh function. If you have any insight into that, it would be much appreciated.
Here's what I'm trying to publish on Zenoh:
use serde::{Deserialize, Serialize};

#[derive(Serialize, Deserialize)]
pub struct NodeMsgExample<'a> {
    pub msg_str: &'a str,
    pub msg_random_num: u64,
}

And here's the code I'm using to publish the NodeMsgExample:
let mut config = Config::default();
config.transport.shared_memory.set_enabled(true).unwrap();
println!("Opening session...");
let session = zenoh::open(config).res_sync().unwrap();

let publisher = session
    .declare_publisher("test/node")
    .congestion_control(CongestionControl::Block)
    .res_sync()
    .unwrap();

let node_msg_example = NodeMsgExample {
    msg_str: "node_2",
    msg_random_num: self.n,
};
loop {
    println!(
        "Publish on node 2: {str} {num}",
        str = node_msg_example.msg_str,
        num = node_msg_example.msg_random_num
    );

    // publisher.put(serde_json_core::to_string(&node_msg_example).unwrap()).res_sync().unwrap();
    publisher.put(serde_json_core::to_string(&node_msg_example).unwrap()).res_sync().unwrap();
    thread::sleep(Duration::from_secs(1));
}

If you need more information from me, don't hesitate to ask.
Thanks in advance, cheers!

I was not able to reproduce this.  I put my code here: https://github.com/newAM/heapless-issue-450
I did use the latest (unreleased) git version of serde_json_core because the current release is using an older heapless (0.7), which might be why I couldn't reproduce it.
Can you share your heapless/serde/serde_json_core versions?

Hi @newAM, thanks a lot for your quick answer.
I've reused the same version as you've proposed in your repository. Which are:
heapless = { version = "0.8.0", features = ["serde"] }
serde = { version = "1.0.196", features = ["derive"] }
serde-json-core = { git = "https://github.com/rust-embedded-community/serde-json-core", rev = "9327a14e74ad3b4fd37a9ac34e72b61aa5fcc9bf" }

But I still get this error:
error[E0277]: the trait bound `Value: From<serde_json_core::heapless::String<_>>` is not satisfied
   --> src/nodes/node_example2.rs:38:27
    |
38  |             publisher.put(serde_json_core::to_string(&node_msg_example).unwrap()).res_sync().unwrap();
    |                       --- ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `From<serde_json_core::heapless::String<_>>` is not implemented for `Value`
    |                       |
    |                       required by a bound introduced by this call
    |
    = help: the following other types implement trait `From<T>`:
              <Value as From<isize>>
              <Value as From<i8>>
              <Value as From<i16>>
              <Value as From<i32>>
              <Value as From<i64>>
              <Value as From<usize>>
              <Value as From<u8>>
              <Value as From<u16>>
            and 13 others
    = note: required for `serde_json_core::heapless::String<_>` to implement `Into<Value>`
note: required by a bound in `Publisher::<'a>::put`
   --> /home/ludovic/.cargo/registry/src/index.crates.io-6f17d22bba15001f/zenoh-0.10.1-rc/src/publication.rs:415:20
    |
413 |     pub fn put<IntoValue>(&self, value: IntoValue) -> Publication
    |            --- required by a bound in this associated function
414 |     where
415 |         IntoValue: Into<Value>,
    |                    ^^^^^^^^^^^ required by this bound in `Publisher::<'a>::put`

For more information about this error, try `rustc --explain E0277`.

Maybe I'm not understanding the error correctly? Maybe the issue might be with serde-json-core?
Sorry, I'm a bit confused :confused:

Ah I see the issue now, zenoh doesn't implement conversion from heapless types for Value: https://docs.rs/zenoh/0.10.1-rc/zenoh/value/struct.Value.html
Try converting the heapless String to a str first.
I'm on my phone now so I can't easily provide an example.  If this doesn't help let me know and I'll provide a code example when I have a proper keyboard.

@newAM sadly I still get the same error.
Here's what I'm using now:
use serde::{Deserialize, Serialize};

#[derive(Serialize, Deserialize)]
pub struct NodeMsgExample<'a> {
    pub msg_str: &'a str,
    pub msg_random_num: u64,
}

From my understanding, this would be a statically allocated string when I create my NodeMsgExample struct (?). Which, I don't think I'm allocating anything static here? Although there might be some Rust black magic going on.
let mut config = Config::default();
config.transport.shared_memory.set_enabled(true).unwrap();
println!("Opening session...");
let session = zenoh::open(config).res_sync().unwrap();

let publisher = session
    .declare_publisher("test/node")
    .congestion_control(CongestionControl::Block)
    .res_sync()
    .unwrap();

let node_msg_example = NodeMsgExample {
    msg_str: "node_2",
    msg_random_num: self.n,
};
loop {
    println!(
        "Publish on node 2: {str} {num}",
        str = node_msg_example.msg_str,
        num = node_msg_example.msg_random_num
    );

    publisher.put(serde_json_core::to_string(&node_msg_example).unwrap()).res_sync().unwrap();
    thread::sleep(Duration::from_secs(1));
}

Again, sorry for my lack of expertise in Rust, I'm learning as I go :sweat_smile:

No worries :)
Currently the type going into publisher.put is heapless::String, but zenoh::value::Value doesn't implement From<heapless::String>, instead try serde_json_core::to_string(&node_msg_example).unwrap().as_str(), the final .as_str() changes the representation to a &str.

That's correct, the heapless types need a size for the static buffer they use, it's a little bit more clear what's going on when expanding the code:
#[derive(serde::Serialize, serde::Deserialize)]
pub struct NodeMsgExample<'a> {
    pub msg_str: &'a str,
    pub msg_random_num: u64,
}

fn main() {
    let node_msg_example = NodeMsgExample {
        msg_str: "node_2",
        msg_random_num: 123456,
    };

    let json_heapless: heapless::String<32> =
        serde_json_core::to_string(&node_msg_example).unwrap();

    let json_str: &str = json_heapless.as_str();

    let value = zenoh::value::Value::from(json_str);
}

Sorry for the delay @newAM, this works for me! Thank you very much for all your help, it is very much appreciated!


Query DirectWrite for the rendering mode to use

As an alternative to #2598, query DirectWrite for the font rendering mode to use instead of hard-coding one.
With the settings on my system this doesn't make much of a difference visually, but asking for the OS to decide the mode seems better simply because there could be different system configurations that result in different modes, or it could even be varied on a per font basis by Microsoft.

Ok, added a line to the change-log, sound alright?


Generate and access events within the enclave

The following has been changed:

sp-io and sgx-externalities now implement append() and clear_prefix().
Introduce a new pallet frame system interface (SystemPalletEventInterface) for easy event access via the Stf.
Sidechain and offchain worker now both need to reset events on a regular basis, otherwise the event storage will grow with every state change:

Sidechain: Clear event storage before proposing a new block.
Offchain: Clear event storage before executing the retrieved trusted calls of an imported parentchain block.

Medium priority because it implements important sp-io functions, which might affect pallet usage of SDK users.
closes #917
closes #630
closes #170
fixes maybe #976

[x] wait for review of PR #967
[x] rebase
[x]  Clean up helper & Stf functions
[x] Offchain worker storage clean up? Probably makes sense when syncing new parentchain blocks, before Stf execution.

After re-reading, I must admit that I was wrong. I think the PR is of an appropriate size for the feature being implemented. I was a bit confused because there were also a lot of function docs that were changed. This is the only thing that could have extracted to another PR as it is not really relevant for the change. Otherwise, very good job!


About removing abstract noun phrases in GRIT construction

Hi!
It is mentioned in the paper that

We eliminate certain abstract noun phrases that are challenging to recognize in the image, such as “time”, “love”, and “freedom”, to reduce potential noise.

So, the abstract noun phrases are eliminated manually or using spacy?
Many thanks!
### Tasks

Hi,
The abstract noun phrases were eliminated manually. More specifically, we utilized ChatGPT to generate a list of such abstract nouns as candidates, and then we manually removed them from our dataset.

Hi!
Could you share the ChatGPT prompt? I am trying to use ChatGPT to generate a list, but results are awful. ChatGPT keeps generating repeated words.
Many thanks!

Hi, @davidluciolu. Here are the abstract nouns we finally used:
abstract_nouns = ["time", "life", "love", "freedom", "happiness",
"wisdom", "peace", "justice", "hope", "courage",
"faith", "understanding", "advantage", "pursuit"]
You may use them to prompt ChatGPT to get more.

thank you!


Refactor group info extensions for MlsGroup

This PR prepares MlsGroup for custom group info extensions.
Details:

I got rid of GroupConfig, because it was only used for the ratchet tree extension
MlsGroup still has an internal state bool to track whether to use the ratchet tree extension. The reason for that is that it is not possible to treat the ratchet tree extension like any other group info extension. When a Commit is created, the corresponding Welcome message needs the ratchet tree of the provisional state. Since the provisional state is ephemeral (for now), it is not possible to generate the ratchet tree extension from outside the group. Long story short, this part hasn't changed. The alternative would be to not store the state in the group and instead pass a bool for every create_commit call, but this might get harder to track.
create_commit now accepts a Vec of group info extensions that are to be included in the Welcome message. Conversely, new_from_welcome now also return all group info extensions, except for the ratchet tree extension.
A new trait for group info extensions is introduced: GroupInfoExtension. The idea behind it is that it can be used by consumer code easily by only implementing Codec and attributing an extension number.
There is a test with a custom extension that implements GroupInfoExtension.
None of this affects the ManagedGroup so far, that will follow in another PR.

What's the plan for this one now? Shall I review or do we cancel reviews and make it a draft until we know how to proceed?

What's the state of this @raphaelrobert ?

Now that the spec changed regarding extensions and their inclusion in the group context we should not continue this PR. There is some code that can be re-used for future PRs tackling the spec changes, so I set it to draft for now. Will close when the new PRs are ready.


Clippy Timing Out

When I have run Clippy in the playground, it is timing out. It seems that it hasn't pre-checked any of the crates added by default so it is doing that and timing itself out. It's worked in the past for me so maybe something recently changed to cause this?
I tested this on the playground I was tinkering around in and then went to a new one in an incognito browser and had the same issue.
My output from Clippy:
    Checking iovec v0.1.2
    Checking num_cpus v1.10.0
    Checking net2 v0.2.33
    Checking time v0.1.42
    Checking memchr v2.2.0
    Checking rand v0.4.6
    Checking atty v0.2.11
    Checking filetime v0.2.4
    Checking xattr v0.2.2
    Checking thread-id v3.3.0
    Checking memmap v0.7.0
    Checking MacTypes-sys v2.1.0
    Checking termion v1.5.1
    Checking crossbeam-epoch v0.7.1
    Checking crossbeam-epoch v0.3.1
    Checking deflate v0.7.19
    Checking base64 v0.10.1
/root/entrypoint.sh: line 8:     7 Killed                  timeout --signal=KILL ${timeout} "$@"

Duplicate of #469

I don't know how I didn't see that, my bad 😃


Support Custom Kubernetes Namespace in AdaptDL Mode

This PR adds the support for launching trials in custom K8s namespace in AdaptDL mode. I have tested my change in our internal training cluster.
Note that other K8s based training services (Kubeflow, framework controller, etc.) still hard-code the default namespace. In long term, I think it is worth supporting this feature in all K8s training services because creating multiple namespaces in the K8s cluster is a common practice.
@SparkSnail @liuzhe-lz Please review when you have time. This should be the last feature we want to add to the AdaptDL integration before the 2.0 release.

@SparkSnail Could you close this one and review https://github.com/microsoft/nni/pull/3176 instead? Git rebase messed up with this PR...


DO NOT MERGE quay.io/nathanzsamsung/k2-tools:77.1

Testing of image for samsung-cnct/k2-tools#77

closing this.  we have CI tests for k2-tools as of, oh, 10 minutes ago :)


Bug 1762769: Prioritize APIs from same CatSrc

Problem: When OLM installs an operator that requires dependencies that
are provided via multiple operators in different CatalogSources, the API
is pulled from any of the CatalogSources that provide the API.
Solution: This commit introduces a change so that OLM will generate a
list of operators that depend on the API, randomly select one of their
sources, and prioritize checking in that CatalogSource for the API prior
to checking the remaining CatalogSource for the API.
Reviewer Checklist

[x] Implementation matches the proposed design, or proposal is updated to match implementation
[x] Sufficient unit test coverage
[x] Sufficient end-to-end test coverage
[x] Docs updated or added to /docs
[x] Commit messages sensible and descriptive

Fixes #1076

Where's the Definition of Done checklist in your PR description? 😉

@evan

Where's the Definition of Done checklist in your PR description? wink

Added! The PR updates functionality to match behavior defined in dependency-resolution.md

/lgtm
great work!

/retest

/retest

/retest

/retest

/retest
On Sat, Oct 19, 2019 at 9:38 AM OpenShift CI Robot<EMAIL_ADDRESS>wrote:

@awgreene https://github.com/awgreene: The following test failed, say
/retest to rerun them all:
Test name Commit Details Rerun command
ci/prow/e2e-aws-olm b49bee6
https://github.com/operator-framework/operator-lifecycle-manager/commit/b49bee67b296ce5d64a30c394eb6ea8dac95b1f2
link
https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/operator-framework_operator-lifecycle-manager/1080/pull-ci-operator-framework-operator-lifecycle-manager-master-e2e-aws-olm/3168 /test
e2e-aws-olm
Full PR test history
https://prow.svc.ci.openshift.org/pr-history?org=operator-framework&repo=operator-lifecycle-manager&pr=1080.
Your PR dashboard
https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:awgreene.
Please help us cut down on flakes by linking to
https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests
an open issue
https://github.com/operator-framework/operator-lifecycle-manager/issues?q=is:issue+is:open
when you hit one in your PR.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/operator-framework/operator-lifecycle-manager/pull/1080?email_source=notifications&email_token=AAAOFR3MC5Y45PFS3DFYBI3QPMEVTA5CNFSM4JB5GFR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBXP2ZQ#issuecomment-544144742,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOFR4ABGU4LKEXXBGQ7XTQPMEVTANCNFSM4JB5GFRQ
.

/retest

/retest

/retest

/retest

/retest

/retest

/cherry-pick release-4.2


storage crate diff

Current dependencies on/for this PR:

master

PR #10   👈

This comment was auto-generated by Graphite.


cod1.c: remove old dead code

No description provided.

Please provide some more context, e.g. no longer need to handle OPparam b/c parameters are now flattened into an array beforehand.

Auto-merge toggled on

noice


add any_image_fallback option

Now almost exactly 3 years later, I forgot my fork existed and ended up implementing it in a much nicer way.
Closes #1
Changes isValidCoverart to return nil for an extension whitelist fail, it doesn't break existing conditions and doesn't introduce extra burden as it's still falsy.
We keep track of the first extension-valid file (which returned false) in directory/playlist, and use it if nothing else matched.
Placeholder picture still works if no matching extensions are found.

You asked in the original issue:

Do you have some use-cases that this feature would solve?

This is meant to load the proper coverart image when it's correctly named, and fall back to whatever image it can find in the directory, even if there's a risk it'll be a booklet scan.
Setting names to an empty string would risk loading unrelated images even when there is a well named cover.

Sorry for the delay, this looks good but I have a few small stylistic suggestions I'd like to make before it gets merged. I'm busy now but I'll try to do a proper review with my suggestions soon. If I have not done so within a week, please feel free to ping me in this thread as a reminder.

@CogentRedTester ping (:


Stuck at boot screen does not boots

V86 doesn't support a lot of the hardware requirements that new OS's require.
The newest version of windows that might work is like vista

V86 doesn't support a lot of the hardware requirements that new OS's require.
The newest version of windows that might work is like vista

Windows 7 works, too.
How I know, Windows 11 is only x64, but v86 supports only x86

It only supports x86 and does NOT support PAE emulation which is a requirement for any of the recent versions of windows. (I think starting with windows 8)
There are probably many other CPU instructions that would be missing as well, but without PAE there's really no hope unless you somehow manage to patch the OS

If you send me the iso I might have a look. That said, it's probably better to use a thread with the corresponding Windows version.

See #571


[stable/velero] Add Carlisia and Ashish as chart maintainers

What this PR does / why we need it:
This PR adds two new maintainers in order to help with responsiveness and help with staff changes.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@prydonius

/assign prydonius

/lgtm 👍

/lgtm


Added white listed email domains

So that nobody outside of those domains can be invited to the
application.
Right now it's only allowing: (hmcts.gsi|digital.justice).gov.uk.

Looks good to go, once the tests pass :+1:


feat(description-list): Added bordered param to description-list - OEL-1777

can you just check the indentation in the template..? it's a bit off, as usual try to get html tags correctly indented, twig code is less important..

@planctus done.


FLINK-30577 unable to access flink artifacts dir

What is the purpose of the change
In openshift when the operator is installed into a restricted namespace the operator is unable to create the /opt/flink/artifacts dir in order to download remote jars for jobs
Brief change log

Modified the flink-operator.yaml to add an emptyDir volume mount for /opt/flink/artifacts

Verifying this change
This change can be verified as follows:

On an opensshift cluster create a new namespace
Install the helm chart
Apply the examples/basic-session-deployment-and-job.yaml
Verify the jobs are running correctly

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 changes to the CustomResourceDescriptors: no
Core observer or reconciler logic that is regularly executed: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

I tested on an OpenShift 4.10.46 Cluster, both with the helm install method, as well as building a bundle and installing in multiple namespaces.  It looked good in all cases.  The operator logs looked like this:
oc logs -f flink-kubernetes-operator-5b6d584c7c-jgttf |grep artifacts
2023-01-24 23:21:01,713 o.a.f.k.o.a.ArtifactManager    [INFO ][flink/basic-session-job-only-example] Created dir: /opt/flink/artifacts/flink/basic-session-deployment-only-example/basic-session-job-only-example
2023-01-24 23:21:24,427 o.a.f.k.o.a.ArtifactManager    [INFO ][jim/basic-session-job-only-example] Created dir: /opt/flink/artifacts/jim/basic-session-deployment-only-example/basic-session-job-only-example

Did a twistlock scan of the image created from the Dockerfile, and no known vulnerabilities.
I think it looks good, the artifacts folders can be written now both in default and non-default namespaces on OpenShift.


disable auto data volume extension with DATA_SIZE is specified

We are still doing auto data volume extension when user specifies DATA_SIZE. Say DATA_SIZE=+10%FREE, then every reboot we will extend data volume by 10% of free space.
Given that we are moving towards relying on lvm to extend data volume for pool based on threshold, not sure how many people need this functionality.
There are two ways to handle it.

Get rid of auto extension for pool data volume completely. Rely on lvm. This might be little surprise over upgrade for an existing DATA_SIZE user. Having said that we don't publicize in man page that DATA_SIZE can take arguments which suit to "lvmextend -l". We have maintained that pass in arguments suitable to "lvextend -L". So chances of somebody relying on auto data extension on every reboot are slim.

Retain auto growth capability but make it conditional. Say AUTO_DATA_LV_GROW=1. By default turn it off in /etc/sysconfig/docker-storage-setup. For users upgrading d-s-s, they will have turned it on by default until they pass in  AUTO_DATA_LV_GROW=0.

Right now I am planning to send patches for first approach. If that's not acceptable, I can work on second approach.

Patch merged. Closing issue.


Impove Line Number handling

User feedback:
Line numbering seems to be always relative to 96-line buffer breaks. For example, if I delete 20 lines from the first buffer, then go down until the second buffer loads, the line numbers go from 75 to 96 in one step. In this case also, if I use the ^G go command to go to line 80, it says “line not found”. Maybe this is by design but it makes the line numbers not very useful.
If a command line ^G go or ^F find is pressed and then not needed and then return is pressed without entering anything, the command aborts, but the status line still shows the prompt which might be a little confusing.

Need to set the refresh bit after the prompt, so that it's restored if the command is canceled.


feat: Provide scaler for Amazon managed service for Prometheus

Provide a description of what has been changed
Checklist

[X] Commits are signed with Developer Certificate of Origin (DCO)

Fixes #
https://github.com/kedacore/keda/issues/2214

Perfect!


getting exception when merge guest cart with customer cart with configurable item

Bug report
Issue Description
Getting error when guest cart is going to added in login customer cart.
Bagisto version

[X] master branch
[ ] 1.3.1
[ ] 1.3.0
[ ] 1.2.0

Steps to reproduce

Login as customer and add configurable item in cart
Logout and add same item as guest user
Login again with same user

Expected result
guest user cart item should be added in customer cart
Actual result

I am reopening this issue as PR #4961 was causing issues with bundle product #6134 so I have to revert previous changes.

@jitendra-webkul
Not fixed, getting the error 500 after login in as a user. Please find the below screenshot.

@jitendra-webkul
Thanks, now this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here.for future reference.


Trying to understand Collections...

Hi,
I am parsing the STIX from this web page:
https://www.cisa.gov/sites/default/files/2023-09/AA23-201A Threat Actors Exploiting Citrix CVE 2023-3519 to Implant Webshells - Update.stix_.json
And I am trying to extract all the file hashes from the STIX document.   However, the collection Files() does not have anything in it.
What am I doing wrong here?

The bundle does not have any file objects. It looks like the hashes you are looking for are stored in the indicator objects as STIX patterns.


Packagist still on v1.3.1 - can you update it, or setup auto update

My Environment (version of the project, operating system, or hardware)
...replace me...

My phpda.yml:
...replace me...

When I run this command:
...replace me...

Actual behavior:
...replace me...

Expected behavior:
...replace me...

Hi, i second that : requirements are out of synchro between :
Github
https://github.com/mamuz/PhpDependencyAnalysis/blob/master/composer.json
Packagist
https://packagist.org/packages/mamuz/php-dependency-analysis#dev-master
Note that i don't know how the 'auto-update" mentionned by @lsv works so i can not help :/

Thanks for hint. I juat need to refresh webhook token for that. Now automated integration is working again.
@dallgoot https://packagist.org/about explains how "Auto-Update" works.


Allow overriding SSL library in Erlang

Motivation for this change
Erlang uses OpenSSL only as a primitives provider and without ability to
replace library we are cut out of using newer primitives (for example
Ed25519 is not available when built with OpenSSL 1.0.2).
Things done

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

[ ] NixOS
[x] macOS
[ ] other Linux distributions

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

It's not really necessary, you can just as well pass in a different version of OpenSSL in the openssl parameter. I don't think it's good to rename it, since Erlang probably expects something OpenSSL compatible.

@JohnAZoidberg I can pass it, but not as a override value. Instead I need to use callErlang and full path to R22.nix.

Ohh, it's not the first argument, I see.
Then you can make OpenSSL optional in the first one: openssl ? null.

@JohnAZoidberg I am not much experienced with Nix yet, but AFAIK this will still not work, as override works only on the derivation itself, not on it's environment (aka wrapping function).

What I mean is:
Keep it as is, just modify the second line like:
- , libxml2, libxslt, ncurses, openssl, perl, autoconf
+ , libxml2, libxslt, ncurses, openssl ? null, perl, autoconf

Because if you override ssl to use a different package than openssl you don't need to pass openssl to the wrapping function.

I'd like to see this merged too. Anything holding it back? @hauleth

I have no idea. I have implemented all suggestions and checks are all green. It is question to the maintainers.

Sorry, lost track of this.
Did you test whether this works?
How are you overriding this? Like this?
pkgs.erlang.override { openssl = pkgs.openssl_1_1 };

Could you please squash the commits together?

just to say, I started another PR with the exact same change here
https://github.com/NixOS/nixpkgs/pull/112989
This is just in order to get this merge. I will happily close this anytime if you want to get a contributions.

Closes in favor of https://github.com/NixOS/nixpkgs/pull/112989


Support reading from UNIX socket

From this fine article

The stats page is great for a quick, human-readable view of HAProxy. However, there are a couple of downsides:

static: the page must be refreshed to be updated
ephemeral: old statistics are lost on each refresh If you plan on scraping HAproxy’s metrics for a script or otherwise, communicating over the socket interface is a much more practical method.

I'm open to support unix socket as well, but please be aware that these arguments are nonsense. Whether to fetch the CSV formatted stats via HTTP or unix socket doesn't make any difference. It's just different protocols. I guess the point they're trying to make is "just looking at the stats page every couple of minutes is not enough". And no shit, that's why we wrote Prometheus and store the results.

There are some additional stats that'd be useful that are only available via the unix socket.

@grobie I'm not sure (because I haven't bothered to try yet), but we're running HAProxy through consul-template (similar tohttps://github.com/SirIle/miniboxes/tree/master/minihaproxy) , which restarts HAProxy rather often (using the -sf command line option), and stats on the http stats page are zeroed every time.

This will be the same when using a unix socket. HAProxy does not support
config reloading. Changing the config will always result in spawning a new
process and terminating the existing one. Stats are kept in memory per
process (when running with nbproc > 1 you'll need one exporter per process)
and not shared in any form.
On Fri, Apr 29, 2016 at 12:56 AM, Sune Keller<EMAIL_ADDRESS>wrote:

@grobie https://github.com/grobie I'm not sure (because I haven't
bothered to try yet), but we're running HAProxy through consul-template
(similar tohttps://github.com/SirIle/miniboxes/tree/master/minihaproxy) ,
which restarts HAProxy rather often (using the -sf command line option),
and stats on the http stats page are zeroed every time.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/prometheus/haproxy_exporter/issues/53#issuecomment-215626917

@brian-brazil I guess you mean show info? I've wished for these information quite a few times indeed. Show pools maybe as well.

Implemented in #58 (by accident, actually)


IOU - Request money is not successful after refreshing in confirm page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to any report
Click on + button and request money
Upload an image for receipt
Refresh the page
Press the request button

Expected Result:
Money should be requested and it should show scanning
Actual Result:
Nothing happens
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.74-2
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/81e77d81-7fd8-4fab-90d4-fd349c9cf4c2
https://github.com/Expensify/App/assets/78819774/ca400e31-186f-4f31-b392-86e05b68e910
Expensify/Expensify Issue URL:
Issue reported by: @alitoshmatov
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1694717028781839
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Request money is not successful after refreshing in confirm page
What is the root cause of that problem?
https://github.com/Expensify/App/blob/446f29b7d47b3dc60dc0b16d7a4adddd0127b58c/src/pages/iou/ReceiptSelector/index.js#L84
In the web, we are creating a new Object URL to save the file. But when we reload the page that object URL is removed (reference for this point: https://stackoverflow.com/questions/13966186/how-long-does-a-blob-persist)
What changes do you think we should make in order to solve the problem?
Let's see another place where we also use a new Object URL to save the file
https://github.com/Expensify/App/assets/141406735/604a1fa1-c0c6-447a-a64d-18c0a0a569fe
In adding an attachment, when reloading we hide the selected attachment modal. So, I suggest that we check if the url Object is removed, we will redirect to ReceiptSelector page. To do that, in here
https://github.com/Expensify/App/blob/144e932c2ea863bffafbd26e9cc7602e89b1e352/src/pages/iou/steps/MoneyRequestConfirmPage.js#L75
Add this code to check objectURL is valid or not
if (props.iou.receiptPath && props.iou.receiptSource) {
            FileUtils.readFileAsync(props.iou.receiptPath, props.iou.receiptSource).then((receipt) => {
                if (!receipt) {
                    const fallback = ROUTES.getMoneyRequestRoute(iouType.current, reportID.current);
                    Navigation.goBack(fallback);
                } else {
                    setReceiptState(receipt)
                }
            });
        }

And then in here
https://github.com/Expensify/App/blob/144e932c2ea863bffafbd26e9cc7602e89b1e352/src/pages/iou/steps/MoneyRequestConfirmPage.js#L205
we only need to check receiptState and request money
if (receiptState) {
                requestMoney(selectedParticipants, trimmedComment, receiptState);
                return;
            }

What alternative solutions did you explore? (Optional)
We can save uploaded files to localStorage

@DylanDylann Does your alternative solution use Onyx to save the URL object? The better fix should make the submit still processable.

@mollfpr I can't reproduce this bug when reloading the App redirects to the initial page. Do you want to fix as suggestion here

The better fix should make the submit still processable.

In adding an attachment, when reloading we hide the selected attachment modal. So, I suggest that we check if the url Object is removed, we will redirect to ReceiptSelector page.

@DylanDylann The expected result should keep the user on the same page after reloading and the submit request is successful. From your RCA the issue is because the object URL is removed after reloading, so I'm thinking your alternative solution is to keep the object URL and set it back.

We need to wait for this reverted PR https://github.com/Expensify/App/pull/28410 to be merged. So we can move forward this bug

@DylanDylann Where's the revert issue?

@mollfpr We had a blocker that blocked this issue on the main yesterday. But now It is resolved. Please ignore the above comment

@mollfpr

The expected result should keep the user on the same page after reloading and the submit request is successful.

To do that, I have 2 suggestions:

Created new API to update receipt image to server when user uploading image
Using FileSystem API  you'll need to request PERSISTENT storage, and the user will have to agree to let you store data on their machine permanently.

I understand the benefit from your viewpoint But both ways are complicated. I see It's a bit overkill for this issue. Note that

when the user uploads the receipt image and close modal we also remove the uploaded image
When the user uploads attachment then reloading, this attachment also be removed

Anyways, Let's confirm the expected before moving forward.
@shawnborton Please help to give some thought from UX perspective on this issue.

Hmm I am a bit confused here about this one. @alitoshmatov since you reported this one, can you walk me through when exactly you think a user would experience this? I am having a hard time understanding why a user would refresh the app after they upload a receipt.

Thanks @DylanDylann. It seems reasonable to redirect the user back.
@luacmartins I think you already agree with the expected result, and the proposal looks good!
🎀 👀 🎀 C+ reviewed!

Thanks for the link @aimane-chnaif. We'll refactor navigation for multi-step flows, but I still think the changes to readFileAsync mentioned in this proposal are good and shouldn't overlap with that refactor, so I think we can move on with this issue.

@mollfpr The PR is ready for review

Waiting to issue payment on 2023-10-25

[@luacmartins] The PR that introduced the bug has been identified. Link to the PR:

https://github.com/Expensify/App/pull/23770

[@luacmartins] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

https://github.com/Expensify/App/pull/23770/files#r1371795681

[@luacmartins] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

The regression step should be enough.

[@mollfpr / @DylanDylann] Determine if we should create a regression test for this bug.
[@mollfpr / @DylanDylann] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Only testable in Web, mWeb, and Desktop.

Clicking + (FAB) Button
Clicking on the scan request
Upload an image for receipt
Go to a confirmation page
Refresh the page
Verify that the App navigates back to the Scan Request Page
👍 or 👎

@johncschuster Could you give us the payment summary? I'll request the payment in NewDot, thanks!

Thanks for filling the checklist. 👍 to the regression tets

@johncschuster Could you give us the payment summary? Thank you!

Just pending payment

On it!

Payment Summary:
External issue reporter: @alitoshmatov - $50 - paid via Upwork
Contributor that fixed the issue: @DylanDylann - $500 - paid via Upwork
Contributor+ that helped on the issue and/or PR: @mollfpr - $500 - paid via manual request

Payment has been issued via Upwork

$500 payment approved for @mollfpr based on summary above.


use built-in project var in e2e module

This PR will:

replace the _PROJECT substitution variable with the built-in PROJECT_ID variable in the e2e testing module cloud build config

/fyi this might conflict with #759.


Spike adding arrow-datafusion & arrow-rust

Closing, as this proof of concept was moved to the arrow-datafusion & arrow-rs repos.

https://github.com/apache/arrow-datafusion/pull/1791
https://github.com/apache/arrow-rs/pull/1289


Created a personal page for snook1.eth (0xeb50...)

Fix #27
Created a page.tsx and directory for my address under the builders directory in the next.js app.
Also created a styles.modules.css file in that directory, as well as added a small .png for my avatar.

@NJarosz do you want to complete this PR? Or should I close it?


[WIP] Algorithm Addition

Adding Use-Case Algorithms (Hhl and qaoa) to Quokka

Please use black to fix the code style (see readme)


Problem referencing Microsoft.Web.Infrastructure

That package is very peculiar. I believe it is a dependency for SimpleInjector.Integration.Web. In my project file, I get this :
<Choose>
    <When Condition="$(TargetFrameworkIdentifier) == '.NETFramework' And ($(TargetFrameworkVersion) == 'v4.0' Or $(TargetFrameworkVersion) == 'v4.5' Or $(TargetFrameworkVersion) == 'v4.5.1' Or $(TargetFrameworkVersion) == 'v4.5.2' Or $(TargetFrameworkVersion) == 'v4.5.3' Or $(TargetFrameworkVersion) == 'v4.6' Or $(TargetFrameworkVersion) == 'v4.6.1')">
      <ItemGroup>
        <Reference Include="SimpleInjector.Integration.Web">
          <HintPath>..\packages\SimpleInjector.Integration.Web\lib\net40\SimpleInjector.Integration.Web.dll</HintPath>
          <Private>True</Private>
          <Paket>True</Paket>
        </Reference>
        <Reference Include="Microsoft.Web.Infrastructure">
          <Paket>True</Paket>
        </Reference>
      </ItemGroup>
    </When>
  </Choose>

But in VS, if I look at the references, that one isn't linked to any assembly (path is empty) and there is the yellow triangle next to the reference. I also get the warnings in the Error List pane:
The referenced component 'Microsoft.Web.Infrastructure' could not be found.
The package / assembly (Microsoft.Web.Infrastructure.dll) exists in packages\Microsoft.Web.Infrastructure\lib\net40.
When I run the paket update command, if im on Nuget v2 api it tries to download
Downloading Microsoft.Web.Infrastructure <IP_ADDRESS>
However if im on v3, it
Downloading Microsoft.Web.Infrastructure 1.0.0
(not sure if it's a hint or not).
Even if I manually paket add Microsoft.Web.Infrastructure to my projects, it does the same things. It does add the Microsoft.Web.Infrastructure (1.0.0) in the lock file after the update command. Do you think it's paket related, I'm pretty sure I managed to add that reference using Nuget before ?

the behaviour is as specified in the nuspec:
<?xml version="1.0"?>
<package xmlns="http://schemas.microsoft.com/packaging/2011/08/nuspec.xsd">
  <metadata>
    <version>3.1.2</version>
    <authors>Simple Injector Contributors</authors>
    <owners>Simple Injector Contributors</owners>
    <licenseUrl>https://simpleinjector.codeplex.com/license</licenseUrl>
    <projectUrl>https://simpleinjector.readthedocs.org/en/latest/lifetimes.html#per-web-request</projectUrl>
    <iconUrl>https://simpleinjector.org/simpleinjector.ico</iconUrl>
    <dependencies>
      <dependency id="SimpleInjector" version="3.1.2" />
    </dependencies>
    <frameworkAssemblies>
      <frameworkAssembly assemblyName="Microsoft.Web.Infrastructure" targetFramework=".NETFramework4.0" />
    </frameworkAssemblies>
    <id>SimpleInjector.Integration.Web</id>
    <title>Simple Injector ASP.NET Integration</title>
    <requireLicenseAcceptance>false</requireLicenseAcceptance>
    <description>The Simple Injector ASP.NET Integration package adds a lifestyle to the Simple Injector called 'Per Web Request', which allows instances to live within a single HTTP request and get disposed when the request ends.</description>
    <summary>ASP.NET integration for Simple Injector. General integration package for web applications such as Web Forms, MVC and IIS hosted Web API projects.</summary>
    <copyright>Simple Injector Contributors 2012 - 2014</copyright>
    <language>en-US</language>
    <tags>simpleinjector architecture dependency-injection DI easy easy-to-use Inversion-of-Control IOC per-web-request web integration</tags>
  </metadata>
</package>

I believe Microsoft.Web.Infrastructure was previously installed in GAC from "something", but is now a Nuget package. Does it make sense ?

Yep, but the package is not respecting that changes. You need to contact
the package author
On Jan 15, 2016 16:05, "dcorriveau-omniscient"<EMAIL_ADDRESS>wrote:

I believe Microsoft.Web.Infrastructure was previously installed in GAC
from "something", but is now a Nuget package. Does it make sense ?
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/1393#issuecomment-171985032.

As stated previously, this is not an issue with Paket so you can close this ticket.
However, if anybody stumbles on this page, one possible solution is to install the Assembly in GAC. Here's how you can list the Assemblies registered in GAC:

Run the following command from a Developer Command Prompt (run as admin):

gacutil -l > c:\gac.txt

If you don't see a line that looks like this in the output file (gac.txt) then it means it's not registered:
Microsoft.Web.Infrastructure, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL

To register it, find the file Microsoft.Web.Infrastructure.dll on your computer (you might need to install some components to get it, you can search the Web to see what installers heve it). Then run this:
gacutil -i Microsoft.Web.Infrastructure.dll


Add microk8s to foundational.md

Adding microk8s as credible and stable alternative to get started with Kubernetes on a local machine. This is especially attractive for those not wanting to incur the overhead of running a VM for a local cluster.

/assign @mistyhacks

@mistyhacks - thanks. I've implemented your comments. LMK if this works?

I added a small copyedit to make the list items more active (start with a verb).
/lgtm
/approve


change flip mode in recyclerview

when I use flipview inside recyclerview some bugs happen
when scrolling, reused view that I need to change from back to front (or opposite), I tried to use flipTheView method withAnimate=false - but the result is seeing the back for second and then the front
I tried to use setFlipDuration(0) before, but the result of that is the animation stays with duration 0 for every next flipTheView call withAnimation=true

We have already compiled a demo of recyclerview with flipview in each item. Can you check whether that works for you or not?  Let me know so that i can work accordingly.


cephadm-adopt: Add --networks parameter support to ceph orch apply rgw

When radosgw_address_block was defined, it was not taken into account during rgw adoption process
depends on: https://tracker.ceph.com/issues/62185
fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2224351

jenkins test centos-container-cephadm_adopt

jenkins test centos-container-cephadm_adopt


CheckResponse: Allow reuse of response body

I tried using CheckResponse in a custom transport to identify certain errors and implement conditional retries and I kept bumping into a weird error
http: ContentLength=157 with Body length 0

from another transport I use to log all HTTP requests & responses (incl. bodies).
I noticed (after some debugging) that ioutil.ReadAll is in use, which closes the stream and prevents other readers from reading the body.
Attached is a test reproducing described behaviour.

I'm not convinced this is a safe change.
In https://godoc.org/net/http it says: The client must close the response body when finished with it.
I've instrumented this package many times to dump out full requests and responses without having to use this technique.
Have you taken a look at the methods found in https://godoc.org/net/http/httputil ? I have found those to be incredibly handy.
Additionally, you can make yourself a RoundTripper described here: https://godoc.org/net/http#RoundTripper
that can intercept each request and dump any info you might need before and after the request.
I think these alternatives are much safer than this change.
I'll leave this PR open for a few days in case there is any more discussion, but unless I hear a strong argument for why this is needed, I don't think we want to go this route as it can potentially cause a lot of heartache if clients do not close the body of the responses.

I've instrumented this package many times to dump out full requests and responses without having to use this technique.

So have I. It worked just fine until I called CheckResponse explicitly. Have you looked at the attached test which demonstrates the problem? Just remove that one line and you should see the test failing.

Additionally, you can make yourself a RoundTripper described here:  https://godoc.org/net/http#RoundTripper
that can intercept each request and dump any info you might need before and after the request.

That is exactly what I did as mentioned in my initial post 😉
While I see your point, httputil.DumpResponse does exactly what I did here - reads the response body and then resets it w/ ioutil.NopCloser, so either we're misinterpreting the comment The client must close the response body when finished with it or these two packages are contradicting each other. 🤔

Oh, I see what you are saying now.  Thanks for the detailed explanation.
OK, I would like to get some discussion going on how users of this package address this situation.
This is an open invitation to all users and contributors to add your thoughts to this PR. Thanks.

There is no discussion on this PR, so I'll take that as no interest in it.
Closing, but we can reopen if there is demand.


Multiple modals and their background

Hi,
I have just started using this library and so far so good, but I have some visual problems with modals.
On my page I'm using multiple modals (sometimes I even switch from one modal to other). For example: customer is able to add a row into table which represents one objects that holds data about vendor, product and so on. To add new line user is presented with first modal. There they can search for existing vendors in database and they can also make new one which opens another dialog.
Problem here now is that background works perfectly on first modal, but when second opens it disappears and it doesn't show back up even when switching from second modal to first one.
Maybe I'm only using this library wrong and this is not bug, but I would like to fix it to give our customers good UX with animated modals and lots of feedback. I also have modal for saving action (VFRemoting, modal is visible until I get response feedback from controller).
import statements are:
<apex:includeScript value="{!URLFOR($Resource.aljs, '/jquery/jquery.aljs-init.min.js')}"/>         <apex:includeScript value="{!URLFOR($Resource.aljs, '/jquery/jquery.aljs-modal.min.js')}"/>
javacript init is:
`
$.aljsInit({
scoped: true
});
    $(document).ready(function() {
        $('[data-aljs="modal"]').modal();
    });`

and then in html I added data-aljs-dismiss="modal" on things that closes modal and I'm showing it with: data-aljs="modal" data-aljs-show="savingDialog"
If I wish to programmatically open modal I use:
$('#savingDialog').modal('dismiss'); and $('#savingDialog').modal('show');
Am I doing anything wrong here?

Hi,
Thanks for trying our library out! I'd just like to clarify that you are running code to dismiss one modal, then show another. Such as:
function a() {
$('#modal-1').modal('dismiss');
$('#modal-2').modal('show');
}
Is that correct?
If so, then this is a bug with our plugin where programmatically showing/dismissing modals in succession is not happening synchronously. We will log this and update you with a fix very soon.
Cheers!
Victor

Hi,
I have tried both ways:
$('#dialog-1').modal('dismiss');             $('#dialog-2').modal('show'); //and this one:             $('[data-aljs-show="dialog-2"]').modal('trigger');
I suspect that problem is somewhere with removing some CSS class and it is not attached back to the new dialog. I have multiple pages where I use "nested modals", one is opened from another or there is saving or loading modal in between to show user status of operation.
Do you have any approximate ETA when this might get fixed so I can forward it to my customer when they can expect those fixes?

I matej, we have implemented a fix for this and will release a new version Friday (Nov 13).

Hi,
I updated aljs but background on dialogs still doesn't work correctly if I close first one and open second one.

Hi, we have tested using  $('#dialog-1').modal('dismiss'); $('#dialog-2').modal('show'); and it works in our code examples. Can you provide more information about your use case? Specifically about how you are launching your "nested modals". Are you running the show/dismiss code in the callback functions?

Hi,
looks like I made a big mistake and forgot to properly save my project file to refence new aljs package I have uploaded into static resources. Sorry for that, it works now perfectly. Thank you for fix :)


Documentation adjustment

The demo nor any of the documentation shows how to detect what the current state of permissions is.  I had to dig around in the Push.js source code to find a function.
Push.Permission.get() appears to be the method that retrieves the cross-browser permission level.  Since the function doesn't appear to be called internally by Push.js, I can only assume that it is meant to be used by application code and calling the functions does appear to work as expected.  However, assumptions aren't as good as documentation.

Hey there! I apologize for not getting around to this until now. Just added this info to the README with commit 9716db297b669bcd8b3471eeb2e54607bbe6dd0b. Not sure why I didn't include it originally. Happy holidays!

The additions look great.  Thanks!


🧹 Standard Codebase Format

clang-format through CI
This PR will introduce a consistent format across the codebase.
PR Changes

[ ] .clang-format has reached an agreed-upon format among etterna developers and contributors
[x] format target has been added to CMake to allow for easy updates of code format
[ ] Github Actions is updated to verify format is correct
[ ] .editorconfig updated to match .clang-format config

The above force-push is a rebase on develop.


[ENHANCEMENT] add webhook validation of the http proxy config

Is your enhancement related to a problem? Please describe.
If you configure httpProxy and httpsProxy, you must also put Harvester node's CIDR into noProxy, otherwise, the Harvester cluster will be broken. It would be nice to add proper validation to prevent misconfiguration.
Describe the solution you'd like
We can add a webhook to validate the user input; it should reject if the node IP is not included in the no_proxy CIDR range.
Describe alternatives you've considered
Add the caution text to the UI before implementing the webhook.
Additional context
doc ref: https://docs.harvesterhci.io/v1.1/advanced/index/#http-proxy

If user configure cluster-registration-url setting, user must also put the host of  cluster-registration-url into noProxy, otherwise, the Harvester cluster cannot be accessed from upstream Rancher

@votdev thanks for this enhancment :smile: :+1:
This looks good on  Version: v1.4.0-dev-20240830
With just the added context about the open issues/enhancments:

https://github.com/harvester/harvester/issues/6539
https://github.com/harvester/harvester/issues/6440

Everything looks good :smile: from all test-cases.
Some small artifacts:

I'll go ahead and close this out :+1:


Add evm rpc endpoint debug_traceCall

Describe your changes and provide context
This PR add a new EVM rpc endpoint debug_traceCall: https://docs.chainstack.com/reference/ethereum-tracecall
Testing performed to validate your change
Added unit test

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 60.86%. Comparing base (c92dc5f) to head (1235403).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1692      +/-   ##
==========================================
+ Coverage   60.81%   60.86%   +0.05%
==========================================
  Files         368      368
  Lines       26857    26865       +8
==========================================
+ Hits        16332    16351      +19
+ Misses       9411     9401      -10
+ Partials     1114     1113       -1

Files
Coverage Δ

evmrpc/tracers.go
100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes


Turn off terminal beep at ly start

Hi all, when ly starts it makes a beeping noise, how can I turn this off? I cannot find anything in the configuration files nor in past issues about this.
Thanks in advance!

I ran into the same problem

I actually think this might be related to gnome and it's horrible terminal stuff.
Just a wild guess though - I might just be completely wrong

I solved it like this:
edit src/main.c
add to c #include <stdlib.h> ...... int main(int argc, char** argv) { system("rmmod pcspkr");

Likely related to this https://wiki.archlinux.org/title/PC_speaker#Beep

@UltraBlackLinux I'm on i3-gaps and this is in a tty, so GNOME should have nothing to do with it anyway

It's most likely related to Unicode characters behaving weirdly on certain machines. Might be due to the Unicode characters on the termbox. I don't think this is easily fixable though, but I may be on the wrong track too.

You can try this, as a root user:
echo "blacklist pcspkr" > /etc/modprobe.d/nobeep.conf

You can try this, as a root user:
echo "blacklist pcspkr" > /etc/modprobe.d/nobeep.conf

Hi , that's not work for me. I'm running archlinux and Ly it's the only one that beep in all my system.
I tried that and also to put  this on /etc/input.rc :
set bell-style none
and that here /etc/profile:
setterm -blength 0
The annoying beep it's still there

Same thing here on Debian 12


Custom domain name (android)

I set up custom domain name in Branch dashboard, but it doesn't redirect me into my app, only to the Play store, but if I use default one, it opens my app. I'm trying to catch URI Scheme myscheme://
and
<data
                android:scheme="http"
                android:host="my.site.com"
                android:pathPrefix="/" />

also I can add any other filters to try to solve this issue, but my question is should it work in the way I described above?
Thanks for any help.

Hi eGorets,
Here's an example of a functional intent filter for both basic URI scheme deep linking (the first one you'll see in the file) and App Link deep linking (the second one): https://github.com/BranchMetrics/Branch-Example-Deep-Linking-Branchster-Android/blob/master/app/src/main/AndroidManifest.xml
It looks like you're trying to setup App Links, in this case:
 <data android:scheme="https" android:host="bnc.lt" android:pathPrefix="/xhsd" />
bnc.lt should be replaced w/ your custom domain (assuming you aren't using bnc.lt). And "/xhsd" should be the pathPrefix value copied from the Link Settings page of your Branch Dashboard. If you are on Branch's app.link domain, you will not need to include the pathPrefix portion. Which means that your statement would come out to: <data android:scheme="https" android:host="my.site.com" />
If this does not help, please e-mail<EMAIL_ADDRESS>and I'll be able to offer more involved assistance.

@EvangelosG thanks, I've just sent more details to email

@eGorets When we use our own custom domain, does that mean we need to set up the assetlinks.json and all of that on our own servers? Or is providing our SHA fingerprints on the Branch dashboard enough?

@stephen-workpop I think SHA fingerprints are enough, but you also have to update CNAME on your server


Refactor VGA driver code to iteratively assign internal registers from main RAM

Currently, all registers in the driver cog which need to be populated with values calculated in the hub are done so one at a time in what's essentially an unrolled loop. Need to determine if there is a way to set them all in a single tight loop.

It's a shame really. Those things have been figured out a long time ago ...
Check out this link, it's a generic loop transferring a long per hub cycle in either direction. There is also a counter based version pre-dating this one which - while OK for me - wasn't generally accepted (and sometimes you don't have a spare counter). Anyway, use (one of) them if you want/need speed ...

I apologize for the mess Parallax made during forum transition(s) which is the reason I stopped participating.

@konimaru Thanks, looks like just the thing I need. Hadn't even gotten around to Googling for a solution yet so thanks for saving me a step! Only thing is that the procedure looks almost as nasty (instruction-wise and visually) as explicitly setting variables, no matter how you do it.

Your call. I - obviously - use this in most of my drivers (where applicable). And the setup is only required once.

@konimaru True story! I'm definitely riding the assembly-timing learning curve into outer space. I'm actually going to have to do a refactor of my VGA driver to populate a line buffer during the visible porch/sync regions, instead of racing to grab a new tile every time I need it. As-is the driver works fine up to 15 horizontal 16x16 tiles, or 16 horizontal 8x8 tiles. Past that, de-sync.

Satisfied by commit a2ff0f2167f0a29744ab3d61e357886df1cfc68d

For the record, I just realised you only wanted a one-off load not something repeating (where speed might be important).

@konimaru no problem! There's an entirely realistic chance the driver will change in the future to make that necessary, and being aware of the execution-time conscious approach is useful.


Not painting

90% paints failing

And ads started again if you removed update it
Thanks 👍🏻

Ban

Yes, you have been banned
I didn't understand, what do you mean "And ads started again if you removed update it"


Proactively send messages to customers from the customer details page

Problem
Users want to be able to proactively send messages to their customers.
Now that we have a customer details page, let's add this feature there!
Solution
Add a modal (or some UI) that enables sending a message to a customer that hasn't started a conversation yet.
Additional context
At the moment this feature only exists within the Live Session viewer (with our Storytime feature).
Testing
This will require a bit of manual testing.
Questions, or need help getting started?
Feel free to ask below, or ping us on Slack :)
(You can also check out our CONTRIBUTING.md)

might be better to just use setTimeout() and pop a chat ANYWAY  with a pre-written message in case of user idling, so that it engages the user
(FYI: im the one who requested this feature on slack chat with Alex)

How do you enable this feature? im using the hosted version of papercups

@taewookim we'll be deploying some updates today :) but here's the general flow:


chore: rename hermes-inspector-msggen to @react-native/hermes-inspector-msggen

Summary
This PR renames  hermes-inspector-msggen to @react-native/hermes-inspector-msggen, moves it to  packages/hermes-inspector-msggen and makes the package private  as part of RFC480.
Related to https://github.com/facebook/react-native/issues/34692
Changelog
[General] [Changed] - Move and rename hermes-inspector-msggen to @react-native/hermes-inspector-msggen
Test Plan
run sh ReactCommon/hermes/inspector/tools/run_msggen

Can you remove the yarn.lock file?

Sure, done


No State to Show - Augury doesn't track state

Augury: 1.23.0
Date: Tue, 02 Jul 2019 18:15:54 GMT
OS: Win32
Description of issue:
When attempting to run an angular app locally using ng serve from the base cli project startup, state is not showing in the panel for any selected components. If I close and reopen dev-tools I receive  an error only when Augury is installed.
Uncaught (in promise) DOMException: Failed to execute 'postMessage' on 'Window': An object could not be cloned.
I think these 2 issues may be connected, but either way Augury is unusable. A separate but possibly related note is that when attempting to select different components it causes chrome to slow down drastically and hang for roughly 5 seconds on larger component loads that it didn't used to do in the past.
Angular version (required): 7.2.15

Experiencing this in Angular version 8.1.1 as well.


maintainers: Add hanemile

Motivation for this change
Adding myself as a maintainer for adding future packages.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
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

[ ] Fits CONTRIBUTING.md.

Please cherry-pick the commit into the PR where you are adding yourself as a maintainer to a package. This makes sure all maintainers entries are actually used and we do not collect stale ones.


More examples

Use plorth-cli shebang for examples.
Add an example for demonstrating the use of loops.
Add an example demonstrating recursion.

Examples are always good.


The ability to provide a custom index file name

Synopsis
The ability to give LocalIndex class a second parameter of "index file name".
Details
Basically, I needed a way to customize the name of the index file to be loaded. We break up our embedding indexes by classification name. I tried to stay consistent with the patterns you had already established in your code base.
Closing
We very much enjoy using this package for our simple AI experimentations. Thank you very much!

@ispyhumanfly sorry I missed this could I get you to merge your changes with main? I'll get this merged in this week

@ispyhumanfly sorry I missed this could I get you to merge your changes with main? I'll get this merged in this week

Sure!

Thanks for the contribution!


ABBREVIATION:

Description
kindly add abbreviation WYTB which is Wish You the Best.

@TreciaKS
Assign this to me. I will surely complete this.

@Njong392 will assign you :)

@Yasir-Khan22 Take it away!


I wish /examples/guestbook-go/ I wish this page existed.

I wish /examples/guestbook-go/ I wish this page existed.
Because it was linked here: https://tectonic.com/starter/docs/latest/quickstart.html

Fixed by adding new 404.md functionality that knows to forward users to examples/ on kubernetes/kubernetes.


Still work ?

Hi,
I'll need a way to authenticate a user thanks to his gamer tag. I would like to know if your module still work ? (any API update needed ?)
Thanks

I'm also interested

There is no way to authenticate against an xbox account, however depending on your use case it could be possible to link a user/session to an XBL account. I have not tried it lately, but it does have a test suite, try starting there. Be forewarned, the endpoints which expose this kind of data have always been flaky and unreliable(fails one second, claims invalid account, or returns data randomly) and you'll have to code around this behaviour.

+1

Now working without any APIs required, API support is coming.


Ollama Function Calling Feedback

Ollama function calling has been implemented in Witsy 1.26. Please post any feedback, questions, concerns here.
Not all models support tools. Please check https://ollama.com/library
From @senovr (https://github.com/nbonamy/witsy/issues/4):
Just tried - worked, sometimes weird, sometimes great.
As an example, during my simple test described above - sometimes it simply
outputs the name, sometimes -gives tons of text about the election process,

Agreed. Not sure how much is because of the model or Ollama itself. Witsy does not do much and as far as I have looked at, there is not a lot of things I could have done wrong. Will try with bigger models (llama 3.1 70b and qwq 32b) and post results here

Things are happening on the Ollama front


Request: New Tabs always open in default location

New Issue Checklist (tick off like this: [x])

[x] I checked existing issues (open and closed) for possible duplicates
[x] I can reproduce the problem on the latest stable version (not nightly!)

What is your browser?

[x] Firefox
[ ] Chrome

What is your operating system?

[ ] Linux
[ ] Mac
[x] Windows

I want new tabs to always open in the default location.  Can we get an option page to set that?  I know that Ctrl-Y (Ctrl-Shift-Y on Windows) can be used, but most people are conditioned to use Ctrl-T or click the button.

Won't do it because Ctrl-Y is a simple way to achieve this and everything else will add too much complexity.


fix: Fix CCA initialization by exporting function instead of immediate execution

What changed? Why?

Fixed the CCA (Client Analytics) initialization in initCCA.ts by modifying the export statement
Previously, the initialization function was being executed immediately on import, which could cause timing issues with analytics initialization
Now exporting the function itself instead of its execution result, allowing proper initialization timing control

Notes to reviewers

The change is minimal but important for analytics functionality
Only the export statement was modified, the initialization logic remains unchanged
This follows best practices for module initialization, allowing the importing code to control when the initialization occurs
The fix aligns with the CCA team's recommendations for library usage

How has it been tested?

Verified that analytics initialization still works correctly in development environment
Checked that page view tracking continues to function as expected
Confirmed that device ID management and tracking preferences are maintained
Tested with both allowed and disallowed tracking preferences

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


config: support lable ok-to-test on tidb & tidb-operator

Because the number of members in pingcap org has decreased, many people are no longer under org. This change is made to allow self-service addition of ok-to-test to trigger CI.
/label ok-to-test

/lgtm
/approve


This is already implemented.


:arrow_up: Fix vulnerabilities by upgrading lodash

Fix vulnerabilities by upgrading lodash #208

Before fix :
found 1072 vulnerabilities (64 low, 1 moderate, 1007 high)

After fix :
found 65 vulnerabilities (64 low, 1 moderate)


WIP - Keyboard Traversal for Autocomplete

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
List which issues are fixed by this PR. You must list at least one issue.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I signed the CLA.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

closing for now, talked offline about alternative approaches, so i'm going to try those out

closing for now, talked offline about alternative approaches, so i'm going to try those out


MAAP CMR and .ops updates

Tutorials so far that work and can be updated to remove .ops and/or MAAP CMR. Technical and science tutorials - pushing technical first, science will follow.
Other technical tutorial issues are listed here
Other science tutorial updates listed here

Note about HLS tutorial: seems to work with the .ops update, but it does like to crash on me when it pulls all of the different bands (I think it even did this before I removed it). Made note of it in the individual issue for HLS when we do VEDA updates, but I'm going to go ahead and push it here

I'll leave a comment when I finish updating the science tutorials with any more cmr/ops changes that I can make, which should be tomorrow afternoon. After that, I'll start looking and making other edits to each individual notebook and do separate PRs

Was it ready for review ? I did a quick review @smk0033, apart from the above comment it looks ok to me.
In addition, could you reference the issue this PR is solving ? My guess is that it's https://github.com/orgs/NASA-IMPACT/projects/29?pane=issue&itemId=27143805 but I am not sure. Thanks !

Sure thing! I'll have the rest of the science tutorials that can be pushed soon, but thanks for looking at what I've gotten so far @emileten! I also plan to fix the links when we go through the individual tutorials after this PR, my biggest thing with this ticket was fixing cmr/.ops in the tutorials that allowed it (and that don't seem to have anything else broken besides links)

I think that's all of the science tutorials I can go ahead and update just for this ticket. Other issues have been noted in the individual tickets under VEDA guidelines

I think I've fixed all the links in the tutorials provided so far for this PR (please let me know if I missed any!). I also added searching_compiling_list_of_granule_ids on here, it didn't have any cmr/ops changes, but I accidentally updated it and just went with it since it was only a link and text fix. I was also able to rerun the HLS tutorial and it seemed like it processed with no issues


Send camera positions less frequently when animating the map marker

Currently, we sent a new camera position update for each of the locations that we're animating. This might not be optimal because if the locations are close to each other the camera movement breaks the smoothness of the map marker movement. Therefore, we should try out different frequencies of updating camera positions to see what works best.
┆Issue is synchronized with this Jira Task by Unito

We can try and adopt the solution from AAT JS where the camera is updated each time a few locations are animated instead of each time a single location is animated. We can even make it configurable and allow the users to specify how often the camera position should be updated.

This will be done as a part of https://github.com/ably/ably-asset-tracking-android/issues/685


[Profiler] Only map implicit getters for lazy vars

This is the only implicit getter case we care about, the rest do not contain user-written code. This means that we now otherwise only ever map explicit decls, which we need to SILGen anyway. As such I believe this should fix rdar://39332957.
We do still potentially map implicit expressions, but should only do so for explicit property initializers. There is one remaining bug where we can map an implicit property wrapper backing initializer, which I'm tracking in rdar://99931619.

@swift-ci please test


Does your Dataset class cope with empty frames?

I often use this repo as a point of reference.
boxes = []
boxes = torch.as_tensor(boxes, dtype=torch.float32)

I don't believe the above would create the required shape to be provided to Faster RCNN in the case of an empty frame. See https://stackoverflow.com/questions/66063046/how-to-train-faster-rcnn-on-dataset-including-negative-data-in-pytorch .
Does your dataset / model cope with empty frames in some other way? Just trying to find what's considered sensible practice.
https://github.com/pytorch/vision/issues/1598

@GeorgePearse
Yes, it handles empty frames. This part in datasets.py handles it.


why class's name change in production mode ?

Environment:

Angular CLI: 1.7.3
Node: 9.8.0
OS: darwin x64
Angular: 5.2.9
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, platform-server, router
... service-worker
@angular/cdk: 5.2.4
@angular/cli: 1.7.3
@angular-devkit/build-optimizer: 0.3.2
@angular-devkit/core: 0.3.2
@angular-devkit/schematics: 0.3.2
@ngtools/json-schema: 1.2.0
@ngtools/webpack: 1.10.2
@schematics/angular: 0.3.2
@schematics/package-update: 0.3.2
typescript: 2.4.2
webpack: 3.11.0

this is my class , a Array Extenion

export class TypeArray extends Array {
constructor(type: any, items = []) {
super(...items);
this.GENERIC_MODEL = type
Object.setPrototypeOf(this, Object.create(TypeArray.prototype));
}
GENERIC_MODEL: T
public getType(): T {
return this.GENERIC_MODEL
}
}

when i construct the object :
i want to get object's property and its name,for example

let properties = Object.getOwnPropertyNames(obj)
//check the type and build the output type
let property_types = properties.map(p => {
return {p: p, creator: (obj[p] || {}).constructor, name: (obj[p] || {}).constructor.name}
})

but in dev environment:
The object's properties are:

0: {p: "id", creator: ƒ, name: "Object"}1: {p: "name", creator: ƒ, name: "Object"}2: {p: "desc", creator: ƒ, name: "Object"}creator: ƒ Object()name: "Object"p: "desc"Symbol(): (...)Symbol(): (...)proto: Object

3: {p: "templates", creator: ƒ, name: "TypeArray"}
the property 'templates'  name is "TypeArray"
But In Prod Mode:
4) [{…}, {…}, {…}, {…}]0: {p: "id", creator: ƒ, name: "Object"}1: {p: "name", creator: ƒ, name: "Object"}2: {p: "desc", creator: ƒ, name: "Object"}
3: {p: "templates", creator: ƒ, name: "t"}creator: ƒ t(e,l)name: "t"
why this happened ? anyone can help me ?

read about uglify plugin for webpack, simple it just optimization

Hello, we reviewed this issue and determined that it doesn't fall into the bug report or feature request category. This issue tracker is not suitable for support requests, please repost your issue on StackOverflow using tag angular-cli.
If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.