[S3] How to use spring.cloud.aws.s3.path-style-access-enabled (or is it broken?)

Type: Feature
Is your feature request related to a problem? Please describe.
The documentation at https://docs.awspring.io/spring-cloud-aws/docs/3.0.2/reference/html/appendix.html describes the property spring.cloud.aws.s3.path-style-access-enabled as

Option to enable using path style access for accessing S3 objects instead of DNS style access. DNS style access is preferred as it will result in better load balancing when accessing S3.

My understanding is that an entry like this would switch from DNS style access to path style access:
spring.cloud.aws.s3.path-style-access-enabled=true

In my Java Spring Boot 2.x application I have a local test setup that uses a minio docker container to emulate S3. My pom.xml contains
<dependency>
	<groupId>io.awspring.cloud</groupId>
	<artifactId>spring-cloud-aws-starter-s3</artifactId>
	<version>3.0.2</version>
</dependency>

My bean for the S3Client looks like this
  @Bean
  public S3Client s3Client(AwsClientBuilderConfigurer awsClientBuilderConfigurer, S3Properties s3Properties) {
    return awsClientBuilderConfigurer
        .configure(S3Client.builder(), s3Properties, null)
        .build();
  }

When I then test my app and want to write something to the "bucketname" bucket of my local minio S3, I get an exception with the following cause:

Caused by: java.net.UnknownHostException: bucketname.localhost

Maybe here's my misunderstanding, I thought path-style-access-enabled=true would make the SDK not use the DNS style access.
I get the desired result by changing my Bean like this:
  @Bean
  public S3Client s3Client(AwsClientBuilderConfigurer awsClientBuilderConfigurer, S3Properties s3Properties) {
    S3ClientBuilder builder = awsClientBuilderConfigurer
        .configure(S3Client.builder(), s3Properties, null);
    if (Boolean.TRUE.equals(s3Properties.getPathStyleAccessEnabled())) {
      builder.forcePathStyle(true);
    }
    return builder.build();
  }

Describe the solution you'd like
I could imagine two possible solutions:

If the existing property is supposed to make S3 calls use path style access, I think it's broken and should be fixed
Add a new property force-path-style=true and its handling in the AwsClientBuilderConfigurer that would make my explicit if-statement plus builder.forcePathStyle(true) obsolete

Describe alternatives you've considered
For now I'm using the above code that basically maps a property name of spring-cloud-aws to the "real" property name of the AWS SDK v2.

@IngoStrauch2020 sorry for late reply. This is happening because you create custom S3Client bean. If you rely on autoconfigured one, all properties are applied to S3ClientBuilder. If you need to create custom S3Client you may also want to rely on autoconfigured builder bean:
@Bean
public S3Client s3Client(S3ClientBuilder builder) {
    // ...
    return builer.build();
}

I don't think there's anything missing from the Spring Cloud AWS side?


Is there any best practice to render the quill to read only on flutter?

Hi, im using flutter-quill for a simple article reading app. I successfully render the quill delta on the flutter with the help of QuillEditor.basic() or setting up the QuillEditor via readOnly parameter.
But somehow, i figure that _getIndentWidth on the EditableTextBlock class always returning 32px left padding. This behaviour resulting in our app to have look like this.

Is there any way for me to override this behaviour? Or maybe you guys have a way to render the quill delta just for reading purpose?
Thanks.

Are u talking about image?

Check out https://github.com/singerdmx/flutter-quill/blob/master/app/lib/main.dart

Is this web or mobile?

This is on mobile, and im talking about the TextBlock

I think it is more of an issue with image
We have an ongoing discussion on this in slack group
https://bulletjournal1024.slack.com/archives/C0189N1LF24/p1613156828005800

Currently the image takes full screen. I believe that is the issue, right?

We fixed this issue for web.
To confirm, this issue is about web right? May I resolve it?

@dimassrio Can you pull the latest code on master and try again? Please but the ss if you see any abnormalities.

Closing due to inactivity.


Missing DDPM

Hello, thank you, for this nice repo !
I tried to launch training with DDPM, but the file {data_root}/c{num_classes}_ddpm.npz' is missing for CIFAR10. Can you provide a way to download it please?

@blaisedelattre Sorry for the late reply. You can download c10_ddpm.npz here: https://drive.google.com/file/d/11beyNYWny24c_dHJPZT0MiKBwZ2Uz6N-/view?usp=sharing
Will update other files soon.

Thank you !

Hi, sorry for re-opening this issue, but I am also interested in having access to this file. But the link is dead, could you update it, please?


jwt_authn: implementation of www-authenticate header (#16216) (#17441)

Co-authored-by: Lizan Zhou<EMAIL_ADDRESS>Signed-off-by: Ryan Sutton<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

Hi @cyran-ryan. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test


feat: export poseidon2_permutation and add to foundation/crypto

#5707
#5706  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fcarreiro and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

note_history_trial_decrypting_time_in_ms (5): 115 (+207%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
772
772
772

l1_rollup_calldata_gas
6,796
6,796
6,808

l1_rollup_execution_gas
587,277
587,277
587,289

l2_block_processing_time_in_ms
1,326 (+2%)
4,771 (+1%)
9,408 (+4%)

note_successful_decrypting_time_in_ms
165 (+4%)
514 (+5%)
904 (-5%)

note_trial_decrypting_time_in_ms
67.5 (+10%)
24.4 (-75%)
23.5 (-66%)

l2_block_building_time_in_ms
10,541
37,603
75,015 (+1%)

l2_block_rollup_simulation_time_in_ms
10,424
37,209
74,248 (+1%)

l2_block_public_tx_process_time_in_ms
5,027 (-1%)
19,666 (-1%)
39,479

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,744 (+6%)
25,681 (-8%)

note_history_successful_decrypting_time_in_ms
1,185 (+4%)
2,223 (+4%)

note_history_trial_decrypting_time_in_ms
:warning: 115 (+207%)
72.8 (+1%)

node_database_size_in_bytes
18,735,184
34,938,960 (-1%)

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
145
44,561
26,424

private-kernel-inner
155 (-1%)
72,188
26,424

private-kernel-ordering
109 (+1%)
51,090
39,297

base-parity
3,906
128
311

root-parity
1,142 (+8%)
1,244
311

base-rollup
13,548 (+2%)
110,820
957

root-rollup
49.5 (+1%)
4,551
821

public-kernel-app-logic
83.1
48,151
40,933

public-kernel-tail
123 (+1%)
53,644
7,385

merge-rollup
10.8 (+5%)
2,760
957

public-kernel-teardown
78.3
48,151
40,933

public-kernel-setup
77.3 (+1%)
48,151
40,933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1 (+1%)
16.0
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.590 (+1%)
0.495
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
46.3 (+1%)
73.7 (+2%)
231
452
871
1,761 (+2%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.475 (+1%)
0.455 (+2%)
0.420
0.422
0.414
0.421 (+2%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.9 (+1%)
110 (+3%)
337 (-1%)
682 (+3%)
1,310
2,606
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.485 (+1%)
0.494 (+2%)
0.459 (-1%)
0.469 (+3%)
0.454
0.453
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.7 (+1%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.539 (+1%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
41,005
528,889

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
905
1,161
1,377

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,147 (+2%)
766
3,348 (-2%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
441 (+1%)
296
756
385 (+1%)
1,231
453


resolve string entity

Fixes #2957
Proposed Changes

Testing

Would be good to fix the generic ones as well, but this works for now.


Select window by handle

I need to select window by hwnd parameter, it is more convenient to test multiple windows with the same titles

Hello @igaisin, I agree this is definitely a valid feature request. I'm afraid I cannot get this done before next month, unfortunately I'm quite busy on other tasks currently.
I would be happy to take pull request if you are eager to get this done asap. I can make a new release once the feature is merged.

I did something similar, but noticed a memory leak when reading windows handle or title, run in an infinite loop.
a 2mb memory jump occurs when an action is performed. Everything is fine during downtime.
I noticed Java application memory leak too, but not so fast.
Java application is third party.
Could the JAB be leaking from a third party application. Or is it a JAB bug?
Later I will prepare and share code examples

I did something similar, but noticed a memory leak when reading windows handle or title, run in an infinite loop.
a 10-15 MB memory jump occurs when an action is performed. Everything is fine during downtime.
I noticed Java application memory leak too, but not so fast.
Java application is third party.
Could the JAB be leaking from a third party application. Or is it a JAB bug?
Later I will prepare and share code examples

Solved a problem, after each window, before the new definition, you need to write jab_wrapper.shutdown() for clear memory


Bootstrap stack already at version '13'. Not downgrading it to version '10'

Having this error when running sst start --increase-timeout --stage $ENV. We didn't update anything but it just breaks on Monday while still working fine on Friday.
Package.json
    "@aws-sdk/client-ssm": "^3.205.0",
    "@serverless-stack/cli": "^0.69.6",
    "@serverless-stack/resources": "^0.69.6",

One of our team member find a way to solve this By running these 2 commands following orders :
yarn sst update

yarn sst bootstrap

I hope this would help someone in the future if they encounter the same error


Live Test for Testing BlobClient.uploadWithResponse

Description
Testing issue #38991

API change check
API changes are not detected in this pull request.

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/check-enforcer override


Fails to run for Ep. 7+

Related to #9  - but not the same issue.

I fixed redis to run on expected port 8000 https://github.com/htm-community/nupic-history-server/issues/9
there I also had to update the dependencies
you've used some local build 0.5.7.dev0 and 0.4.13.dev0, which are not available
tested with both: stable (0.5.7 and 0.4.13), and latest (just nupic, nupic.bindings in requirements)

And I get the same error:
(schools) mmm@mmm-U2442:/mnt/store/devel/HTM/htm_schools/htm-school-viz$ npm start 8080

><EMAIL_ADDRESS>start /mnt/store/devel/HTM/htm_schools/htm-school-viz
> node index.js "8080"

htm-school-viz running on http://localhost:8080
Error: Parse Error
    at Error (native)
    at Socket.socketOnData (_http_client.js:304:20)
    at emitOne (events.js:77:13)
    at Socket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:146:16)
    at Socket.Readable.push (_stream_readable.js:110:10)
    at TCP.onread (net.js:523:20)

The screen is stuck on a "loading" circle

This is on both Chromium, Firefox, Ununtu 16.04.
Note: I get a warning $ npm install npm WARN optional Skipping failed optional dependency /chokidar/fsevents: npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>but I guess it's really just optional.

Yeah, the npm error is likely unrelated

It seems you have a few ports confused

htm-school-viz runs on 8080 (or any other you provide by the command line)
it expects nupic-history-server to run on 8000 which is NOT configurable
nupic-history-server expects redis to run on 6379, which is also NOT configurable

Please review the 2 PRs, @rikkertkoppes was right that I had the servers confused; but I think there still was an error with the ports.
Now it's:

redis (as default on 6379)
nupic-history-server listens on 8080 (it actually listnes there, not configurable)
htm-schools-viz expects "history" now correctly on 8080, and it itself runs on 8001 (now)

Is this correct? :scream_cat:

I'd still be getting an error
(schools) mmm@mmm-U2442:/mnt/store/devel/HTM/htm_schools/nupic-history-server$ jobs
[1]+  Running                 redis-server &
(schools) mmm@mmm-U2442:/mnt/store/devel/HTM/htm_schools/nupic-history-server$ python webserver.py
http://<IP_ADDRESS>:8080/
Traceback (most recent call last):
  File "/mnt/store/devel/HTM/htm_schools/schools/local/lib/python2.7/site-packages/web/application.py", line 239, in process
    return self.handle()
  File "/mnt/store/devel/HTM/htm_schools/schools/local/lib/python2.7/site-packages/web/application.py", line 230, in handle
    return self._delegate(fn, self.fvars, args)
  File "/mnt/store/devel/HTM/htm_schools/schools/local/lib/python2.7/site-packages/web/application.py", line 462, in _delegate
    return handle_class(cls)
  File "/mnt/store/devel/HTM/htm_schools/schools/local/lib/python2.7/site-packages/web/application.py", line 438, in handle_class
    return tocall(*args)
  File "webserver.py", line 404, in DELETE
    ioClient.nuke()
  File "/mnt/store/devel/HTM/htm_schools/nupic-history-server/nupic_history/io_client.py", line 154, in nuke
    for f in os.listdir(folder):
OSError: [Errno 2] No such file or directory: './working'

<IP_ADDRESS>:54632 - - [06/Apr/2017 16:58:50] "HTTP/1.1 DELETE /_flush/" - 500 Internal Server Error

...but I think it's a move in the right direction :thought_balloon:

And I was right :wink:
Added a commit that fixes err above, now we face a new one:
(schools) mmm@mmm-U2442:/mnt/store/devel/HTM/htm_schools/nupic-history-server$ python webserver.py
http://<IP_ADDRESS>:8080/
<IP_ADDRESS>:54852 - - [06/Apr/2017 17:13:52] "HTTP/1.1 DELETE /_flush/" - 200 OK
{u'boostStrength': 1,
 u'columnDimensions': [2048],
 u'dutyCyclePeriod': 1000,
 u'globalInhibition': True,
 u'inputDimensions': [631],
 u'localAreaDensity': 0,
 u'minPctOverlapDutyCycle': 0.001,
 u'numActiveColumnsPerInhArea': 40,
 u'potentialPct': 0.85,
 u'potentialRadius': 631,
 u'stimulusThreshold': 1,
 u'synPermActiveInc': 0.05,
 u'synPermConnected': 0.1,
 u'synPermInactiveDec': 0.008,
 u'wrapAround': True}
Traceback (most recent call last):
  File "/mnt/store/devel/HTM/htm_schools/schools/local/lib/python2.7/site-packages/web/application.py", line 239, in process
    return self.handle()
  File "/mnt/store/devel/HTM/htm_schools/schools/local/lib/python2.7/site-packages/web/application.py", line 230, in handle
    return self._delegate(fn, self.fvars, args)
  File "/mnt/store/devel/HTM/htm_schools/schools/local/lib/python2.7/site-packages/web/application.py", line 462, in _delegate
    return handle_class(cls)
  File "/mnt/store/devel/HTM/htm_schools/schools/local/lib/python2.7/site-packages/web/application.py", line 438, in handle_class
    return tocall(*args)
  File "webserver.py", line 75, in POST
    sp = SpFacade(SP(**params), ioClient)
TypeError: __init__() got an unexpected keyword argument 'boostStrength'

<IP_ADDRESS>:54854 - - [06/Apr/2017 17:13:52] "HTTP/1.1 POST /_sp/" - 500 Internal Server Error

Which should be easy to fix. It's related to the recent renaming of the "boosting parameter", the decorator must match with the release used.
@rhyolight can you please match and update the requirements to the latest release?

nupic-history-server should listen on 8000, start the server with python webserver.py 8000 to specify that port

Ok, that would also work, I did it the other way around: I let the history to run on :8080 and changed the visualizer to expect history running on 8080, so it really is correct now. ....We should decide what is a better approach, imho these PRs, as there are no parameters required and it runs out of the box (?)
I'm facing the last problem with the facade: https://github.com/htm-community/htm-school-viz/issues/19#issuecomment-292207090
PS: to validate this approach: you must review both the PRs linked here.

Thanks you guys!

@rhyolight do you have an idea about the last problem above, TypeError: __init__() got an unexpected keyword argument 'boostStrength' ?
I checked, and in SP it's called the same. So I don't know where this issue comes from: facade, capnp, web hooks,... ?
The backtrace doesn't tell much to me, can you help us debugging, or point in some direction?

@breznak The only time I've ever seen this error is when I have an older version of NuPIC or nupic.core installed. You might check to see that there are no old versions installed in your environment.

Awesome, a good word comes a long way ;) I had the 0.5.7/0.4.13 older recommended versions installed, updated to the latest v0.6.0 and it's running!!
Btw, this is an awesome project, a sweet candy to play with :heart_eyes:


Organize final research/notes (Google Drive)

Date entered - 11/25/2024
Last date updated action step entered -1/8/2024
Actively updating notes and organizing notes. Once organized from steps 1-10, A-Z I will print out the hard copy for (SOP- Book and Business Model Structure)
Next is to organize Hardcopy structure on google drive (putting all research in order)

One thing at a time now. @Bossiana


pwrite64() is missing for uclibc targets

When I upgraded to a recent toolchain (2021-03-25) and updated some crates to today's latest as a result, I started getting errors about missing libc::pwrite64() for my uclibc-based target.  I haven't tracked it all the way back to see when the error first started happening, but I think the issue is that the standard library source requires pwrite64() for non-Android Linux:
https://github.com/rust-lang/rust/blob/48691ea6e639640f110b43e33d4aba1f07e7415c/library/std/src/sys/unix/fd.rs#L167-L183
#[cfg(target_os = "linux")]
use libc::pwrite64;
cvt(pwrite64(fd, buf, count, offset))

Whereas pwrite64() is disallowed for uclibc in the libc crate:
https://github.com/rust-lang/libc/blob/f913fc53a682c7b836bdf2ac0da601f2d8334b32/src/unix/linux_like/mod.rs#L1671-L1685
However, the companion function pread64() is permitted for uclibc:
https://github.com/rust-lang/libc/blob/f913fc53a682c7b836bdf2ac0da601f2d8334b32/src/unix/linux_like/mod.rs#L1543-L1548
The actual pwrite64() function is available on my uclibc target.  Considering that libc contains a SYS_pwrite64 value for uclibc targets, I suspect this is just an oversight in the libc crate, and that pwrite64() was not intended to be caveated by cfg(not(target_env = "uclibc")).  Unless there is some other reason for disallowing pwrite64() on uclibc-based Linux targets, could this function be promoted to work for uclibc variants?

Thanks for reporting, yeah, I think there's no reason to disable pwrite64 on uclibc. Submitted #2131 as a fix.

I tested with a custom copy of std using this new libc and it works great.  Thanks for the lightning-fast fix.

Anytime :)

@drmikehenry Could you please share an example of how you consumed this fix? I tried playing with xargo and the patch feature, but no luck 😟
I created a PR to rust to update the libc dep, but want to consume it in the meanwhile.
Thanks

@zvirja I'd be happy to post the steps I followed once I retrieve them from the office early next week.  Mostly I just followed somebody's tutorial on re-compiling the Rust standard library, but there are indeed a number of steps involved and without notes I'd be lost :-)

@zvirja I'm having trouble replicating this change with my home setup, but it appears that your pull request has gone through already, so perhaps you can use a new nightly toolchain to get the libc updates.

@zvirja You probably won't need this if you use the very latest Rust sources; commit e41f378f825488a537b024fc3ed599d9c12fda96 bumped the libc version dependency to 0.2.93.  But in case you want to use an older toolchain with the new libc, here are the steps I used to test libc-0.2.92.
First, install a nightly toolchain and make it the default:
rustup install nightly-2021-03-01
rustup default nightly-2021-03-01

Clone Rust sources and enter rust directory:
mkdir -p ~/projects/custom-std
cd ~/projects/custom-std
git clone https://github.com/rust-lang/rust
cd rust

Optionally, checkout a branch or commit hash as the desired starting point, perhaps using the hash from the active compiler:
rustc -vV | grep commit-hash

with output:
commit-hash: e37a13cc3594004663738bd18d8100e6db9666cf

Checkout this commit:
git checkout e37a13cc3594004663738bd18d8100e6db9666cf

Initialize and update the submodules:
git submodule init
git submodule update

For use with Cargo's build-std feature, the Rust sources need to live in the expected location below ~/.rustup.  Using this example toolchain (nightly-2021-03-01), enter the toolchain's rustlib directory, create src (if necessary), and enter src:
cd ~/.rustup/toolchains/nightly-2021-03-01-x86_64-unknown-linux-gnu/lib/rustlib
mkdir -p src
cd src

If there's already a rust/ directory, it must be deleted or renamed out of the way:
mv rust rust.orig

Create a symlink to the cloned Rust sources:
ln -s ~/projects/custom-std/rust rust

Create an example project to test:
cd ~/projects
cargo new hello-custom-std
cd hello-custom-std

Add the following to Cargo.toml:
[profile.release]
panic = "abort"

At this point, Cargo's build-std feature should work:
cargo build \
  -Z build-std=panic_abort,std \
  -Z build-std-features=panic_immediate_abort \
  --target x86_64-unknown-linux-gnu \
  --release

Building with Xargo should work as well, if a Xargo.toml is created, e.g.:
[dependencies]
std = {default-features=false, features=["panic_immediate_abort"]}

Then install Xargo and build:
cargo install xargo
xargo build \
  --target x86_64-unknown-linux-gnu \
  --release

With Xargo, it's not necessary to place the sources below ~/.rustup; instead, an environment variable can be set to point directly to the cloned sources, e.g.:
export XARGO_RUST_SRC=$HOME/projects/custom-std/rust/library

You might need to remove ~/.xargo to force a rebuild of everything.
To use libc-0.2.92, clone the libc source and checkout the 0.2.92 branch:
cd ~/projects
git clone https://github.com/rust-lang/libc
cd libc
git checkout 0.2.92

Edit the Rust standard library's Cargo.toml file to use this new version; the changes are shown via this command:
cd ~/projects/custom-std/rust
git diff library/std/Cargo.toml

with sample output:
diff --git a/library/std/Cargo.toml b/library/std/Cargo.toml
index 275fcc4c292..8a82c66b250 100644
--- a/library/std/Cargo.toml
+++ b/library/std/Cargo.toml
@@ -16,7 +16,7 @@ cfg-if = { version = "0.1.8", features = ['rustc-dep-of-std'] }
 panic_unwind = { path = "../panic_unwind", optional = true }
 panic_abort = { path = "../panic_abort" }
 core = { path = "../core" }
-libc = { version = "0.2.85", default-features = false, features = ['rustc-dep-of-std'] }
+libc = { version = "=0.2.92", default-features = false, features = ['rustc-dep-of-std'], path = "../../../../libc" }
 compiler_builtins = { version = "0.1.39" }
 profiler_builtins = { path = "../profiler_builtins", optional = true }
 unwind = { path = "../unwind" }

The change involves requesting version = "=0.2.92" (matching the version in libc/Cargo.toml) and setting the path to point to the libc working copy (note that relative paths are taken relative to the directory containing library/std/Cargo.toml).
It may be necessary to clean the workspace and remove the ~/.xargo tree before rebuilding::
rm -rf ~/.xargo
cargo clean
xargo build \
  --target x86_64-unknown-linux-gnu \
  --release

The indication that the local copies are being used is in these lines of output:
   Compiling libc v0.2.92 (/home/USERNAME/projects/libc)
   Compiling std v0.0.0 (/home/USERNAME/projects/custom-std/rust/library/std)

The reason I couldn't replicate this earlier is that I'd been using version = "0.2.92" to request version 0.2.92.  I'd mistakenly thought this would request the version exactly, but it turns out that this is equivalent to a "caret requirement" "^0.2.92", which asks Cargo to use the newest version of the form 0.2.x for x >= 92.  The day I initially tried this, the newest version on crates.io was 0.2.92, so that's the version Cargo selected.  Since I'd placed version 0.2.92 at the path location, Cargo used my local working copy.  But when I later tried to replicate this, libc-0.2.93 had been published.  Cargo selected 0.2.93 as the best version, then skipped over my local working copy of libc because it contained the wrong version (0.2.92).


style: "io/ioutil" has been deprecated since Go 1.16

I have read the CLA Document and I hereby sign the CLA

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 16 (87.5%)  changed or added relevant lines in 8 files are covered.
14 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.1%) to 72.014%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/scaffold/templates.go
5
6
83.33%

pkg/util/kfile/file.go
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

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

pkg/engine/operation/parser/spec_parser.go
4
66.04%

pkg/kusionctl/cmd/apply/options.go
7
70.27%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
4601

Relevant Lines:
6389

💛  - Coveralls


nginx deployment pod fails after move migration

After performing a move operation during migration noticed the following
root@ip-172-31-61-189: ~/go/src/github.com/fusor/mig-operator # oc logs pods/nginx-deployment-669d967995-z2bvg -n nginx-example
nginx: [alert] could not open error log file: open() "/var/log/nginx/error.log" failed (13: Permission denied)
2019/07/18 14:40:13 [emerg] 1#1: open() "/var/log/nginx/error.log" failed (13: Permission denied)
root@ip-172-31-61-189: ~/go/src/github.com/fusor/mig-operator # oc get pv
NAME   CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS   CLAIM                      STORAGECLASS   REASON   AGE
pv9    100Gi      RWO,ROX,RWX    Retain           Bound    nginx-example/nginx-logs                           5m16s

I hit this issue while testing EBS -> NFS copy migration as well.

I believe this has been resolved by https://github.com/konveyor/openshift-migration-plugin/pull/8


Cannot Share using RNFetchBlob

I am trying to share an image after downloading from an URL.
the code to reproduce -
const dirs = RNFetchBlob.fs.dirs
    RNFetchBlob
      .config({
        fileCache: true,
        appendExt: 'jpg'
      })
      .fetch('GET', this.props.file)
      .then((res) => {
        Share.share({
          url: res.path()
        })
      })

But this approach works with iOS, but it sends a blank url in Android.
i tried -
Share.share({
          url: 'file://'+res.path()
        })

But it still sends null in Android
maybe related to #517

According to https://facebook.github.io/react-native/docs/share.html sharing a URL is an iOS thing.

i will not open i guess its only iOS thing? but any ideas how to share with android?

Sorry, no. You could ask on StackOverflow.

Actually... have you tried sending the URL in the text message? After all, when you enter stuff on a social media site they automatically parse URLs form the text. Maybe that already is enough.

ya that is working good, let me try passing with a native module


Remove composer.phar and add short installation instruction

Close frozen PR https://github.com/domnikl/DesignPatternsPHP/pull/127 from @TomasVotruba

@bocharsky-bw :+1:

@domnikl Thanks, I'm a bit rephrased installation section and fixed typo.

That's better, thanks! So I am merging it.


What MQTT library is being used in the paho iot hub twin device sample

I am trying to integrate twin device properties into my device code but I have found that this sample uses a different MQTT library and I couldn't find the file in the repo. I am using platformIO and vscode and was wondering where I could find the .h file to import into my project.
(https://github.com/Azure/azure-sdk-for-c/blob/main/sdk/samples/iot/paho_iot_hub_twin_sample.c)

Hi @boura0013, this sample uses the paho mqtt library which is obtained via vcpkg. You can take a look at our sample instructions to see how we do it: https://github.com/Azure/azure-sdk-for-c/tree/main/sdk/samples/iot.

Hi @OhmV-IR, closing this issue because it has been 3 weeks since the last response.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 08:33:29 on 2023.05.04
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 3, errors out: 0
Original problem ID: 20269
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=20269

Problem has been resolved in 6m 0s at 08:39:29 on 2023.05.04
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 20269


PNG compression when use palette will lost some color

What are you trying to achieve?
png file compression
Have you searched for similar questions?
no
Are you able to provide a minimal, standalone code sample that demonstrates this question?
yes
const fs = require('fs')
const sharp = require('sharp')
const path = require('path')
const uuidV4 = require('uuid').v4
;(async () => {
  const tmpDir = `${Date.now()}-${uuidV4().replace(/-/g, '')}`
  await fs.promises.mkdir(path.resolve(__dirname, `./result/${tmpDir}`), { recursive: true })
  const buffers = []
  const dirs = await fs.promises.readdir(path.resolve(__dirname, './assets'))
  for (const dirName of dirs) {
    console.log(dirName)
    const files = await fs.promises.readdir(path.resolve(__dirname, `./assets/${dirName}`))

    for (const fileName of files) {
      if (path.extname(fileName) === '.png') buffers.push({
        fileName: `${dirName}-${fileName}`,
        buffer: await fs.promises.readFile(path.resolve(__dirname, `./assets/${dirName}/${fileName}`))
      })
    }
  }

  for (const buf of buffers.slice(0, 1)) {
    const newBufferPng = await sharp(buf.buffer).toFormat('png', {  compressionLevel: 0  }) .toBuffer()
    // const newBufferJpeg = await sharp(buf.buffer).jpeg({ quality: 100, palette: true }).toBuffer()
    await fs.promises.writeFile(path.resolve(__dirname, `./result/${tmpDir}/${buf.fileName}`), newBufferPng)
    console.log('done')
    // await fs.promises.writeFile(path.resolve(__dirname, `./result/${tmpDir}/${buf.fileName}`.replace('png', 'jpg')), newBufferJpeg)
  }

})().then(console.log).catch(console.error)

Are you able to provide a sample image that helps explain the question?
without png option quality or palette

with  png option quality or palette

Yes, this is expected, the use of palette in this example is reducing a 32-bit per pixel "full colour" RGBA image down to an 8-bit per pixel indexed image, which is a "lossy" operation.

@lovell Which effected method or option I can use to reduce PNG file size, but not lose color info.
I tried option compressionLevel but in the test, the size didn't change linerly.
In some cases compressionLevel possibly make size larger than before comrpession
thx

For full-colour PNGs, compressionLevel and adaptiveFiltering affect output file size. You'll need to experiment to find the right setting for your scenario.
https://sharp.pixelplumbing.com/api-output#png

For full-colour PNGs, compressionLevel and adaptiveFiltering affect output file size. You'll need to experiment to find the right setting for your scenario.

ok, thanks☻


Eastleigh Parklets

Would parklets work in Eastleigh? See twitter thread for examples:
https://twitter.com/GroundworkGM/status/1279008288735182849

Another example:
https://twitter.com/adamtranter/status/1358384081386557443

I like what they've done with the Winchester high street which has partially gone down this route. Getting the buses out of the high street was a genius move. Three issues I see with the kinds of designs in those tweets. 1. cleanliness. I see in some of the pictures people sitting with their feet on some of the other seating area - apparently it is designed with that use case specifically in mind. it'd just get too dirty to sit on. 2. maintenance, especially in winter. 3. finding practical space to build those kinds of structures without consuming walking or cycling space. It'd be interesting to see if you could come up with a pedestrianised area, but with the Swan Centre being the dominant retail, I can't see any street that makes sense apart from as a gesture.


How to get .txt for bbox in test.py

I get this error on running test.py
OSError: my_vid_file.txt not found.

bbox loads a file (.txt or .face, etc) that records face position of every frame. You need to put this file into the same directory with the tested video.

@fengfengfen ，but the txt file is empty when I only run the  facepad-test.py, and  through a errorindexerror too many indices for array, do you know what happend

Did you solve this problem ? I meet the same.

@fengfengfen ，but the txt file is empty when I only run the facepad-test.py, and through a errorindexerror too many indices for array, do you know what happend

You have to provide a text file which has coordinates for the bounding boxes for the faces. Then it is loaded into bbox.


Docs: Featurevisor SDK usage with NextJS

Write documentation for using Featureivor with Next.js
Other existing relevant docs:

JavaScript SDK: https://featurevisor.com/docs/sdks/
React.js bindings: https://featurevisor.com/docs/react/

Where to add it?

In docs/frameworks/next.md
Add the link in docs/sidebarNavigation.json under "Framework guides" section

Available here: https://featurevisor.com/docs/frameworks/nextjs/


(demo)-collector: update prom endpoint

Updates the prometheus endpoint to use the port number only.
The prior reference to an environment variable for this property was incorrect, using env. instead of env:. Removing the IP portion of the endpoint altogether makes this configuration easier to maintain.

I will close this in favor of a move to the OTLP write receiver built into Prometheus instead.


Dont show service credential keys not in state create succeeded

Basically we want not to show any service key whos state is not in
create succeeded. This is due to all other states cannot be certain that
the key still exists/already exists and orphane mitigation hasnt cleaned
it up even after a initial failed delete.
For more information see https://github.com/cloudfoundry/cloud_controller_ng/issues/2637

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[x] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

The commit 5dc3636bcf593f9d9aa895071931dc2c63195026 will vanish once https://github.com/cloudfoundry/cloud_controller_ng/pull/2682 is merged :)


Production issue fixes.

Handling abort and timeout in multiple states.
Handling s1setup reconnect from enb.
Abort fix in s1 release handling.
Discard handling of s1release of enb ue s1ap id doesn't match

retest this please


Add single-point implementation of per-object flow vectors, working t…

…owards #10
Does anyone remember how this works?

Looks exactly the same.


object deserialization

I have an object that has an object inside like this
public class customer
{
public string name {get;set;}
public object someProperty {get;set;}
}
when I serialize and deseralize the object
var obj = JSON.Deserialize(serializedString)
I don;t get back the object somproperty I only get the json string...how can I get this to work
thanks

This is because the default Deserialize method is highly optimized, it needs to know all possible types of a field before hand. To get the functionality you need, you should use the DeserializeDynamic method.
Example:
var obj = JSON.DeserializeDynamic(jsonString);

Console.WriteLine(obj);
Console.WriteLine(obj.name);
Console.WriteLine(obj.someProperty);

I tried that before but it gives me the same result...if I change the object type to type of T it works during serialization and deserialization although I have to instantiate the class using that type T which I didn't want
here is what I am trying to do
I have 2 sample classes here the embedded one can handle that class or any other class depending on whats passed to it
public class embeddedCust
{
public string Name { get; set; }
public int Age { get; set; }
public string MyProperty { get; set; }
public object Customer { get; set; }
}
public class Customer
{
    public string name { get; set; }
    public int age { get; set; }

}

then I have this
var data = new embeddedCust { Age = 11, Customer = new Customer { age = 12, name = "name" }, MyProperty = "myProperty", Name = "someName" };
var serializedString = JSON.Serialize(data);
var obj = JSON.Deserialize(serializedString);
I don't get the customer at all...
thanks for the help

What you want to do is impossible. There is no way Jil knows what type of class will be used in Customer because it is of type object.
What you should use is SerializeDynamic and DeserializeDynamic.
Like so:
var data = new embeddedCust {
    Age = 11,
    Customer = new Customer { age = 12, name = "name" }
};
var serializedString = JSON.SerializeDynamic(data)
var dynObj = JSON.DeserializeDynamic(serializedString);

// dynObj won't be of type embeddedCust, but you can map it yourself
// if you know what kind of data it will be, for example:
var obj = new embeddedCust {
    Age = (int)dynObj.Age,
    Customer = new Customer {
        age = (int)dynObj.Customer.age,
        name = (string)dynObj.Customer.name
    }
}

As a side note, I really recommend you use the Microsoft Naming Conventions, it makes your code a lot easier to read.

Wazner is correct.


✨[Feature Request]: want to add real colour hover effect in social icons

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
when we hover on social icon it will change in its real colour
Use Case
improve the ui of project
Benefits
looks accractive to the users
Add ScreenShots
before :
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/assets/159682348/4d2e0900-6941-416d-b643-30b6780f5ec0
want to make it like this :
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/assets/159682348/aea5a32f-3482-4d7f-80e8-68d2fcae70d3
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Anishkagupta04 please assign me this issue under gssoc labels I am sure users will like this feature .

@varshith257 please assign me this issue under gssoc labels


protein mislocalized to the vacuolar membrane

ntr: protein mislocalized to the vacuolar membrane
standard def
PMID:22484924

also what term to use for a  normally resident vacuolar protein which is secreted?
protein mislocalized from vacuole to extracellular region?

also
normal endosome to Golgi transport
Note that, as all of these are proteins, for this one I started from 'normal protein transport' and ended up under  'intracellular protein transport', which is a different branch.
This always worries me. This is because  veiscles can  presumable carry other things. However, I think every assay we have curated refers to proteins. I wonder if we can make our terms
normal protein transport endosome to Golgi transport etc...and add x-p to intracellular protein transport?
Then, if we come across an instance which does not refer to protein, we just create a new parent?

also what term to use for a normally resident vacuolar protein which is secreted?

How about 'abolished [or decreased, if some makes it to the vacuole] maintenance of protein localization in vacuole, with protein secreted'? I can include friendlier synonyms.

Yep, do both terms...there are lots of papers which do this assay (secretion of CPY) and they might be both.IIRC this one is abolished but I'll need to double check.

by 'both' you mean 'decreased yada' and 'abolished yada'? if so, no problem

on to the other issue ...

This is because vesicles can presumable carry other things.

That's correct; the FYPO structure mirrors GO's.

I wonder if we can make our terms normal protein transport endosome to Golgi transport etc...and add x-p to intracellular protein transport?

How many terms are affected?
I am always very leery of narrowing the definitions of existing terms. I've only done it on rare occasions, for single terms that have 1 or 2 (or in one case, zero) annotations, and usually where the term either didn't have a logical definition, or where the xp could be changed cleanly.
Doing a batch would make me twitchy, and no, I don't think simply adding intracellular protein transport would work (or at least, it's not obvious how to build it in). I also doubt that GO would add protein-specific terms (e.g. normal endosome to Golgi protein transport) unless there are cases where different mechanisms operate or different gene products are involved in moving proteins as opposed to other substances.
Unless you're only talking about one or two terms, I would prefer to:

add narrow synonyms that include 'protein';
add comments to explain that proteins or other stuff can be transported, and that if transport of a protein was assayed, put it in an extension;
add to the extension config to allow (but not require) a gene ID to represent an assayed protein.

This also has the advantage of being generic - I can do the same for the term in https://github.com/pombase/curation/issues/968 and any new terms in this area.

yep

Go with your preferred solution. I haven't come across an e.g where  the gp in question does not transport proteins (ever).
It would be great if these could be XP-d to  intracellular protein transport, but I have always been aware of this sub-opitimality? in GO so I often IC to "intracellular protein transport" for enrichment purposes.

new terms
protein mislocalized to vacuolar membrane FYPO:0005489
abnormal protein localization to vacuole FYPO:0005490
decreased maintenance of protein localization in vacuole, with protein secreted FYPO:0005491
abolished maintenance of protein localization in vacuole, with protein secreted FYPO:0005492
normal endosome to Golgi transport FYPO:0005493
synonyms & comments added as described, and extension config updated (new ticket if you spot any I missed)
edit file: 87bbd5a48383ecf64dfdfde1fc48c7f8b8c05389
release: a1a90675d29890871922540ab49849611cfd8bc0


Run as a systemd service

This way the tool can just run on a server and log to journald and stuff.

I wrote some files for this but I have no idea if they work or not so lol? I'll give it a try sometime this week.

The sample service and timer files are now essentially my functioning install, so I know they work! At least, on Fedora 32 at time of writing.


File tree root is not added to object references

The file-tree root object is not added to object references and will therefore not be transported by fetch, and might be removed by git's garbage collection.

Fixed by PR #48


Chore:hidding the carousel when we select any category

JIRA:https://newrelic.atlassian.net/browse/CXPUI-26
Description:When user selects a category in left navigation pane carousel of most popular and featured are supposed to be hidden
OLD VIEW:
when All option selected in left panel

when any category is selected

NEW VIEW:
when All option selected in left panel

when any category is selected

Quick close/reopen to retrigger that build


how to run main.py

How to preprocess data, set the data directory, and training commands and testing commands.

First you need to download the dataset
From here
https://drive.google.com/drive/mobile/folders/1ZOYpTUa82_jCcxIdTmyr0LXQfvaM9vIy
After that setup path to your dataset in this file datautils.py
the train and the test you can do it with this line in main file
parser.add_argument('--is_train', type=int, default=1, help='training the model')

I forgot tell you
1 mean train model
0 mean test model
parser.add_argument('--is_train', type=int, default=1, help='training the model')

I set the path of 6 datasets in datautils.py.
Then, do you mean executing a command like "python main.py --is_train" on the terminal window for training?
However, I got the error below. Do you know how to solve this error?


Change endpoint status Online/Offline to push mode

In current implementation, every client /negotiate request enumerates all the endpoint online status and creates a new array.
User has requirements to monitor the endpoint status when it is changed

I'm trying to set up multiple service instances base on Resiliency and disaster recovery
To test it, I deleted the primary service resource and experienced the mentioned issue. I implemented a workaround to resolve it. This solved my client side connection to the secondary/online signalr service. But when I try to invoke any method on the server side in the hub (connection.send("addtogroup", groupName)) I get an error:
Connection disconnected with error 'Error: Server returned an error on close: No app server is currently connected to the Azure service.
Question: Will fix for the endpoint status also solve the issue with the connection on the server side?
Question: Should I create a separated issue for it?
Question: Is there any better way to ping the service, w/o preview package?
Workaround: I used the preview package Azure SignalR Service Management SDK where I added my own router where I try if an endpoint is working:
` private bool CheckEndpoint(string connectionString)
{
try
{
var _serviceManager = new ServiceManagerBuilder()
.WithOptions(option =>
{
option.ConnectionString = connectionString;
option.ServiceTransportType = ServiceTransportType.Transient;
}).Build();
            IServiceHubContext _hubContext = _serviceManager.CreateHubContextAsync("Ping", null).ConfigureAwait(false).GetAwaiter().GetResult();
            _hubContext.Clients.All.SendCoreAsync("ping", new object[0]).ConfigureAwait(false).GetAwaiter().GetResult();

            return true;
        }
        catch (Exception ex)
        {
            Logger.LogError(ex, $"Error with negotiate endpoint: {connectionString}");
            return false;
        }
    }`

I'm using:

ASP.NET Core v2.2
Microsoft.Azure.SignalR v1.0.10
Microsoft.Azure.SignalR.Management v1.0.0-preview1-10420

Hi @nikolasur

To test it, I deleted the primary service resource

As Resiliency and disaster recovery stated, the disaster recovery uses active-active pattern, which means, for different regions, there are both "app server" -> "azure service" pairs. The connection between "app server" and "azure service" within one region should be always primary, and connections cross region is secondary in case disaster happens.
No app server is currently connected to the Azure service. happens when there is no primary connections connected.

Let's create an issue to discuss it if you still have any.

Hi @vicancy, Thank you for the response, for now there is no other issues/questions. I will set both endpoint as primary, and return the endpoint in same region by default in custom router.


Multiple funding addresses for TokenType1

Hi there,
As already suggested at https://github.com/Bitcoin-com/slp-sdk/issues/30, TokenType1 operations currently only support a single funding address for some operations. The problem is there are scenarios where supporting multiple funding addresses is necessary. As an example:
Suppose a client wallet wants to implement https://github.com/bitcoin/bips/blob/master/bip-0044.mediawiki to support BCH at the derivation path m/44'/145' and SLP tokens at the derivation path m/44'/245'. A TokenType1.mint operation will need some "BCH
utxos" (from path m/44'/145') and the "baton utxo" (from path m/44'/245'), but currently, slp-sdk will try to get all utxos from a single funding address.
This PR implements multiple funding addresses to all TokenType1 operations, although maybe not all operations may need it. What do you think? It makes sense to implement multiple funding addresses in all operations or the above scenario should be done using something like https://github.com/simpleledger/slpjs directly?

@thr0wn I think slp-sdk was intended to support these types of multi-input transactions, but was erroring out due to a bug. I submitted a patch earlier here in #89
fundingAddress and fundingWif should be able to accept an array or a single address
Was this the functionality targeted, or is this PR also extending to other cases past #89?

@josephroyking, this PR is also extending to other cases. As a exemple, the line  https://github.com/Bitcoin-com/slp-sdk/blob/5b72a659ca48c95d7f6792b7db291cfa8e565803/src/TokenType1.ts#L107 does not handle multiple funding wifs (one for each funding address), so if an array of funding addresses and wifs is passed, the code will break in some point trying to use a invalid wif.

Thank you for the PR @thr0wn, it's a bit big for us to consider right now though.
There may be some more single -> array cases that can be solved with small changes like the ones in #90 , for more elaborate changes or more advanced use cases it may be best to use SLPJS.


Make Operadics constraint looser in SwiftPM

Algebra needs at least 0.2.3 now, so this change is needed to use
Algebra and Swiftx in the same project.

There is now a dash in (12.9-inch) for some ungodly reason.  Update the travis script accordingly.


Adding building block game wrt #4056

PR Description 📜
Adding Building block game
Fixes #4056
Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Thank you soo much for contributing to our repository 💗

Thank you @aasthakourav20 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @kunjgit , I have already added the README file kindly review .

Hey @kunjgit , I have already added the README file in my newly created folder & updated Repository's main README file , kindly review the changes.

Thank you  @aasthakourav20 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @aasthakourav20,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


Add Repository Tour page to docs for review.  Not yet added to index.

Codecov Report

Merging #40 (778c6ba) into main (b063180) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
- Coverage   77.31%   77.17%   -0.15%
==========================================
  Files         162      162
  Lines        2896     2878      -18
  Branches       70       67       -3
==========================================
- Hits         2239     2221      -18
  Misses        657      657

Impacted Files
Coverage Δ

.../surge/internal/kafka/KafkaProducerActorImpl.scala
87.01% <0.00%> (-0.90%)
:arrow_down:

...src/main/scala/surge/core/KafkaProducerActor.scala
87.50% <0.00%> (-0.38%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b063180...778c6ba. Read the comment docs.


config: replace use of protobuf UnpackTo with MessageUtil::unpackTo.

Previously, we were missing a number of checks on the return of
UnpackTo(). This was dangerous, since Any conversion might fail, with
the result being that later config validations returned confusing error
messages (e.g. due to an empty message).
This PR switches all uses of UnpackTo() to a checked
MessageUtil::unpackTo(), which also provides helpful type information in
the exception.
Risk level: Low
Testing: Unit tests, also fixed above observed issue in v3 integration
tests I'm working on.
Signed-off-by: Harvey Tuch<EMAIL_ADDRESS>

/azp run


Look into how to update the README and License on our npm releases

Worth looking at : https://github.com/release-it/release-it but we'd have to see how it works with nx / GitHub Actions


Makefile: run deps-update when go.mod changes

Whenever go.mod is changed, it is required that deps-update is run.
Change the Makefile to make it automatic.
Signed-off-by: Raghavendra Talur<EMAIL_ADDRESS>

Can you explain a bit about how this works for those of us who don't understand the make magic?

As discussed offline, this change is wrong.

With deps-update phony, it will always be executed, and the added dependency to go.mod does not have any effect. (Wouldn't have an effect without phony either, unless you create a file named deps-update...)
since nothing depends on the deps-update target, the pr also has no immediate effect. the right thing would be to have build depend on deps-update.

I have made these changes now:

Build depends on deps-update
deps-update is PHONY
Fixed the PHONY list in a new commit by fixing a bug in the Makefile syntax

@umangachapagain @obnoxxx @jarrpa The Makefile has some lines in it before the all target (which is the first target in the file) because of which just running make does not build all target.
This is the output when make is run. I see that only two targets are built, operator-sdk and update-generated.
$ make --dry-run
echo "Ensuring operator-sdk"
hack/ensure-operator-sdk.sh
Using existing controller-gen from "_output/tools/bin/controller-gen"
Using existing yq from "_output/tools/bin/yq"
'_output/tools/bin/controller-gen' schemapatch:manifests="./deploy/crds" paths="./pkg/apis/ocs/v1" output:dir="./deploy/crds"
echo Updating generated files
hack/generate-k8s-openapi.sh
echo Running update-codegen-crds
make update-codegen-crds

If I comment those lines, then the target all becomes the default target. Here is the output:
$ make --dry-run
echo "Running deps-update"
go mod tidy && go mod vendor
echo "Building the ocs-operator binary"
mkdir -p build/_output/bin
env GOOS=linux GOARCH=amd64 go build -i -ldflags="-s -w" -mod=vendor -o build/_output/bin/ocs-operator ./cmd/manager
echo "Building the ocs-operator image"
hack/build-operator.sh
echo "Building ocs-registry image in appregistry format"
hack/build-appregistry.sh
echo "Building the ocs-must-gather image"
hack/build-must-gather.sh

I have a commit at the top of this branch to show the difference but I understand that it is not the right fix. If you guide me on the expected behavior of PR #509 , I will modify the commit.

@raghavendra-talur : See #550 for a fix for the all target. And an unrelated fix to PHONY...

Oh, I see that you already spotted the missing newline escape in PHONY definition here. I can remove it from #550. ;-)

And for the record, the fix for the all target is not required for this PR. The main thing is that build triggers deps-update, not that all does it.
FWIW, the actual change to have build depend on deps-update looks good.
But I would actually split it into 3 patches:

build depends on deps-update
make deps-update phony
add the echo output
I know that not everyone would agree to that. ;-)

And for the record, the fix for the all target is not required for this PR. The main thing is that build triggers deps-update, not that all does it.

I have removed the fixes for missing escape and the test patch for all target. Let us fix that in PR #550

FWIW, the actual change to have build depend on deps-update looks good.
But I would actually split it into 3 patches:

build depends on deps-update
make deps-update phony
add the echo output

I know that not everyone would agree to that. ;-)

Have split the original commit into 3 commits as per the suggestion.


Editor scroll position jumps up when returning to editor

Type: Bug
When navigating away from and back to an editor containing expanded PR comments, the scroll position of the editor is not preserved when focus returns to it. Instead, the editor's scroll position jumps higher up in the file.
Expected behavior
An editor is open in an editor group and the user navigates to a different editor in that group. When the user navigates back to the first editor, it opens at the same scroll position it was in when the user navigated away from it.
Actual behavior
An editor (containing one or more expanded PR comments) is open in an editor group and the user navigates to a different editor in that group. When the user navigates back to the first editor, it momentarily appears in the same scroll position it was in when the user navigated away from it. A moment later, however, the scroll position jumps higher up in the file.
Steps to reproduce:

Open two or more editors in an editor group. At least one of the editors must contain PR comments.
Navigate to one of the files containing PR comments, and expand at least one of the comments. Note the scroll position of the editor, for example, the line number of the first line visible at the top of the editor.
Navigate to another editor in the group.
Navigate back to the first editor. Note the scroll position immediately upon returning to the file; it should be the same as when you left.
Wait a moment (seems to be no more than one second). You should now have seen the scroll position of the editor jump up. Now, the line number of the first visible line should be less than that of the first visible line previously.

Other notes:

The magnitude of the jump in scroll position seems to vary proportionally to the amount of vertical space occupied by PR comments. For example, in one test I did, I collapsed all but one of the comments, scrolled elsewhere in the file, then navigated away from the editor. When I returned, the scroll position jumped up by about 25 lines. Then, I expanded a few more of the PR comments and repeated the same sequence of actions. This time, the jump was equivalent to 75 lines.
As a corollary to the first point, this issue does not present when all PR comments in a file are collapsed. Of course, this makes sense in terms of the first point because, when the vertical space occupied by PR comments is zero, the magnitude of the scroll position jump is zero.
I first noticed this issue immediately after updating to the latest VS Code version (1.85). I did not notice whether or not the version of the extension also changed at that time.

Extension version: 0.79.2023121409
VS Code version: Code 1.85.1 (0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2, 2023-12-13T09:48:06.308Z)
OS version: Darwin x64 23.1.0
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
3, 7, 15

Memory (System)
32.00GB (2.44GB free)

Process Argv
--crash-reporter-id 3622bda1-f8a6-4c12-b164-9fe14b832e0f

Screen Reader
yes

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
py29gd2263:30899288
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
3biah626:30602489
f6dab269:30613381
2i9eh265:30646982
showlangstatbar:30737416
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
welcomedialog:30910333
pythonnosmt12:30797651
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
dsvsc013:30795093
dsvsc014:30804076
dsvsc015:30845448
pythontestfixt:30902429
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
2e7ec940:30885897
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
aa_t_chat:30882232

To verify:

Open a file with a comment and make sure the comment is expanded. Scroll to the bottom.
Switch to another file in the same editor group.
Switch back to the first file.
Verify that the scroll position only ever jumps by 2 lines and not by an amount proportional to the size of the comment thread widget.


Misleading error message for super.instanceProperty

Take the following code:
export abstract class YaddaBase {
    protected roots = "hi";
}

export class DerivedYadda extends YaddaBase {
    public get rootTests() {
        return super.roots;
    }
}

TypeScript nightly (5.3.0-dev.20230926) reports:

Class field 'roots' defined by the parent class is not accessible in the child class via super.

The error message here is confusing - one might believe that this has to do with accessing a member with a protected modifier, but it is really about accessing an instance field with super. A better error message would be:

'roots' is defined as an instance property and must be accessed through 'this', not 'super'.

We can also provide a quick fix for this error message.

https://github.com/microsoft/TypeScript/pull/54056

How is

Class field 'roots' is defined by the super class and must be accessed through 'this', not 'super'.

I don't understand the advantage of calling 'roots' a class field vs instance property, so maybe "Instance property 'roots' is ..." is better.
I definitely think 'super class' is more accurate than 'parent class' and more a propos than 'base class' since super is the mistaken keyword in question.

Because it's not really about class fields, it's about instance properties.
Though for what it's worth, we don't issue the error in JavaScript for properties inferred from assignments.
// @ts-check

class YaddaBase {
    constructor() {
        this.roots = "hi";
    }
}

class DerivedYadda extends YaddaBase {
    get rootTests() {
        return this.roots; // no error?
    }
}

console.log(new DerivedYadda().rootTests); // prints undefined!

I prefer “instance property”, as that is the important distinction: it’s not accessible through super precisely because it’s a property of the instance and not the prototype.

Wait, why does that print undefined?  Shouldn’t the base class ctor run before the getter?

Sorry, fixed now to make it super.roots again.


Maximization & Minimization functions collecting list input from third argument

added another else statement if the expression is a list, to break apart variables and coefficients

it's trivial, but for my gui and anyone else creating gui lp solvers, it's a nice feature to have variables and coefficients mapped for objective function. Please see my repository [for the pywraplp.py][1] file where the max and min functions are altered and the [simplex_class.py] file line's 60:69 for further clarification and explicit detail

This looks very stale, I'll take a look over the winter holidays; someone @google will need to officially assign this to me.

Sorry, I do not get the point, or the added value over python iterations.

No worries then, I'm not sure there's "added" value in the sense of making this easier internally @google, so much as making the interface "friendlier";


server: Add ttl compaction filter

What problem does this PR solve?
part of https://github.com/tikv/tikv/pull/9595
What is changed and how it works?
Add TTL compaction filter
Related changes

Need to cherry-pick to the release branch

Check List
Tests

Unit test

Release note

None

/test

/merge

/merge


Adds state artifact fetcher

PART OF #49

Summary by CodeRabbit

New Features

Introduced new environment variables for better configuration: GROUP_NAME and STATE_ARTIFACT_NAME.
Added functionality for managing and fetching state artifacts from a registry.
Enhanced input handling to support fetching state artifacts based on user-provided URLs.
Implemented a new scheduling system for managing processes effectively.
Introduced a notification system for improved event handling.
Restructured JSON data format for artifacts, providing more detailed representation.

Bug Fixes

Updated registry URL configuration to a more general endpoint for improved accessibility.

Documentation

Updated configuration loading to utilize new environment variables for better clarity and management.

I wanted to share a snippet of Go code that demonstrates how to extract artifacts from a state artifact image created using crane.Image() rather than ORAS. I ran into an issue where extracting artifacts using ORAS returned an empty artifact, but this method solved it. You can use this approach to pull the image, export its content, and extract the artifacts.json from state artifact. hope this helps

import (
	"archive/tar"
	"bytes"
	"encoding/json"
	"fmt"
	"io"
	"log"
	"os"

	"github.com/google/go-containerregistry/pkg/crane"
)

// Define the structure of your JSON artifacts
type Result struct {
	Registry  string     `json:"registry"`
	Artifacts []Artifact `json:"artifacts"`
}

type Artifact struct {
	Repository string   `json:"repository"`
	Tag        []string `json:"tag"`
	Labels     []string `json:"labels"`
	Type       string   `json:"type"`
	Digest     string   `json:"digest"`
	Deleted    bool     `json:"deleted"`
}

func main() {
	// Image reference (replace with the actual image URL)
	imgRef := "localhost:80/satellite/group1/state:latest"

	// Pull the image from the registry
	img, err := crane.Pull(imgRef)
	if err != nil {
		log.Fatalf("Failed to pull image: %v", err)
	}

	// Export the content of the image (as a tar file)
	tarContent := new(bytes.Buffer)
	if err := crane.Export(img, tarContent); err != nil {
		log.Fatalf("Failed to export image: %v", err)
	}

	// Parse the tar content to extract `artifacts.json`
	tr := tar.NewReader(tarContent)
	var artifactsJSON []byte
	for {
		hdr, err := tr.Next()
		if err == io.EOF {
			break // End of tar archive
		}
		if err != nil {
			log.Fatalf("Error reading tar archive: %v", err)
		}

		if hdr.Name == "artifacts.json" {
			// Found `artifacts.json`, read the content
			artifactsJSON, err = io.ReadAll(tr)
			if err != nil {
				log.Fatalf("Failed to read artifacts.json: %v", err)
			}
			break
		}
	}

	if artifactsJSON == nil {
		log.Fatal("artifacts.json not found in image")
	}

	// Unmarshal the JSON content into the Result struct
	var result Result
	err = json.Unmarshal(artifactsJSON, &result)
	if err != nil {
		log.Fatalf("Failed to unmarshal artifacts.json: %v", err)
	}

	file, err := os.Create("state.json")
	jsn, err := json.MarshalIndent(result, "", "  ")
	file.Write(jsn)

	// Print the extracted data
	fmt.Printf("Registry: %s\n", result.Registry)
	for _, artifact := range result.Artifacts {
		fmt.Printf("Artifact: %s, Digest: %s\n", artifact.Repository, artifact.Digest)
	}
}


fix(ai-proxy): add analytics for chat route_type of anthropic

Summary
fix a bug that the analystics is not included in the response of the chat route_type of anthropic
Checklist

[x] The Pull Request has tests
[x] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference
FTI-5769

The failure of test cases should probably be fixed after #12699.


"bash: [: true: integer expression expected" inside path_module

Bash complains about a test comparison inside path_module with incorrect operator introduced in commit 1045000d00b03041878e9b4c47653d19f593f52d
Printing the $3 value when inside path_module shows that this variable holds a boolean value instead of an integer one. Don't know if this has to be that way though.

Reverting the changes of said commit fixes bash complaints about the correct use of the operator inside the test.
Bash version:
GNU bash, version 4.4.19(1)-release (x86_64-unknown-linux-gnu)
I'd gladly provide more information if needed.
EDIT: Strangely, I have not been able to reproduce this on a friend's machine. I fear that this might not be a pureline bug.

Closing this because I was too dumb to not realize I had my config file outdated after pulling some changes.
Funny enough I find a little and harmless file named '1', now I get it why those changed had to be made.
Sorry for this useless issue.


pgbadger   only create index.html file and also doesnt create any directory (in incremental mode))

Hi;
I  didnt understand that but pgbadger doesn not create the report . I did all config any other pg server and it create a report even with directory like 2019 .. week etc... But on production  server it doesn't. It just generate a file - index.html. and  this report  shows nothing.

my parameters:
log_destination = 'stderr'  (it request -f option also...)
log_checkpoints = on
log_connections = on
log_disconnections = on
log_duration = off
log_error_verbosity = dafault           # terse, default, or verbose messages
log_line_prefix = '%t [%p]: user=%u,db=%d,app=%a,client=%h '
log_lock_waits = on
#log_statement = 'none'                 # none, ddl, mod, all
#log_replication_commands = off
log_temp_files = 0
log_autovacuum_min_duration = 0
my command;
/usr/bin/pgbadger -I  /pgV11Data/data/log/postgresql-14.log -O /var/lib/pgsql/11/backups/
How can I handle it?
thanx in advence

It seems that log_min_duration_statement = 0 is missing in your settings?

I have set it like this:
log_min_duration_statement = 10000

Well I guess that there is no statement or information in your log that is parsable by pgBadger. If this is not the case please post a log line sample here.

now I have checked again and I saw that reports was generated. your guess is correct I think.  I will keep checking if somethigs wrong I will write here again. Thank u so much.


Anoncreds library still referencing dev11 pre-release

ACA-Py 0.12.0 is still using anoncreds= { version = "0.2.0.dev11", optional = true, allow-prereleases = true } as a dependency but anoncreds back in Feb looks to have a real release that ACA-Py could be referencing instead that came out shortly after that one.
https://pypi.org/project/anoncreds/#history
Doesn't look like much difference between 0.2.0dev11 and 0.2.0 proper.

Fixed in #2917


orderwebhook not work

when I use orderedwebhook on openshift, on_chat_message not work. but when using deprecated message_loop this function run and work just fine.

Can you let me see how you used OrderedWebhook? Thanks.

of course.
not working example
from flask import Flask, request
import telepot
from telepot.loop import MessageLoop
from telepot.loop import OrderedWebhook
try:
    from Queue import Queue
except ImportError:
    from queue import Queue

def on_chat_message(msg):
	#not working
    print("something")

def on_callback_query(msg):
	#not working
    print("something")

application = Flask(__name__)
bot = telepot.Bot(token)
webhook = OrderedWebhook(bot, {'chat': on_chat_message,
                               'callback_query': on_callback_query})

@application.route('/')
def hello_world():
    return b'Hello World!'

@application.route('/webhook', methods=['GET', 'POST'])
def pass_update():
	print("something") #working
    webhook.feed(request.data)
	print("something else") #working
    return 'OK'

if __name__ == '__main__':
	try:
		bot.setWebhook("https://py-tgbot.7e14.starter-us-west-2.openshiftapps.com/webhook")
	except telepot.exception.TooManyRequestsError:
		pass
    webhook.run_as_thread()
    application.run()

working example
from flask import Flask, request
import traceback
import telepot

try:
    from Queue import Queue
except ImportError:
    from queue import Queue

def on_chat_message(msg):
	#working
    print("something")

def on_callback_query(msg):
	#working
    print("something")

application = Flask(__name__)
bot = telepot.Bot(token)
update_queue = Queue()

bot.message_loop({'chat': on_chat_message,
                  'callback_query': on_callback_query},source=update_queue,ordered=False)

@application.route('/')
def hello_world():
    return b'Hello World!'

@application.route('/webhook', methods=['GET', 'POST'])
def pass_update():
    update_queue.put(request.data)  # pass update to bot
    return 'OK'

if __name__ == '__main__':
    application.run()

@javad94, sorry for the delay. I have been busy and lazy. Expect some answers within 2 days.

no problem. thanks

I slightly modified your program and it works on my webserver:
import sys
from flask import Flask, request
import telepot
from telepot.loop import OrderedWebhook
try:
    from Queue import Queue
except ImportError:
    from queue import Queue

def on_chat_message(msg):
    print("On chat message")

TOKEN = sys.argv[1]
PORT = int(sys.argv[2])
URL = sys.argv[3]

application = Flask(__name__)
bot = telepot.Bot(TOKEN)
webhook = OrderedWebhook(bot, {'chat': on_chat_message})

@application.route('/')
def hello_world():
    return b'Hello World!'

@application.route('/webhook', methods=['GET', 'POST'])
def pass_update():
    print("something") #working
    webhook.feed(request.data)
    print("something else") #working
    return 'OK'

if __name__ == '__main__':
    try:
        print('Starting ...')
        bot.setWebhook(URL)
    except telepot.exception.TooManyRequestsError:
        pass
    webhook.run_as_thread()
    application.run(port=PORT, debug=True)

When I send it a text message, on_chat_message() gets called appropriately.
Run it with this command line:
python3.5 <file>.py <token> <listening port> <webhook URL>

Would you please double-check?

great. thanks
sorry for the delay.


fix(client-go): defer deferred closing of body until after error check.

If the request aborted due to an error the body didn't exist and
the deferred function would throw a nil pointer exception.
Reported by yebyen on IRC.
Example panic: http://pastebin.com/NsjfvEWf

LGTM

Code LGTM


Allow manual adjustment/reset of BuildID

In VSTS please allow this. I see many posts and I have had several challenges in how people aren't able to seamlessly continue versioning when they move to VSTS. But other systems like TeamCity allow you to manually reset/set a BuildID. And if VSTS uses this in some deep internal way, then provide another way to set and then auto-increment a variable. Having a hard coded "seed" adjustment to the BuildID seems .. dirty.

Just created i little helper for myself
https://github.com/FrankMue/AzureDevopsBuildIdIncrementer

We have the counter() variable expression available for this.

@moswald as this is the only place where I see the counter() var expression mentioned besides the docs:
Is there any way to use this function in classic pipelines?

counter() is valid as a pipeline variable in both designer and yaml pipelines.


chore: disable unneeded api calls on deploy

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#334
#333  👈
#330
#329
#327
#326
#323
#322
#320
#328
#325
#324
#318
#317
#316
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


GPL-816 Modify LTHR-384 pipeline to pool into 1 tube

User story
As R&D I would like the Heron LTHR-384 pipeline to multiplex all 384 samples into just one tube at the end of the workflow.
Currently, if you start with 4 LTHR Cherrypick plates, and go down the 'LTHR 384 Novaseq' workflow, it creates one submission for each of the 4 plates and therefore ends up pooling the samples into 4 different tubes.
Who are the primary contacts for this story
Ben F, Scott T, (James G - did original story)
Acceptance criteria
To be considered successful the solution must allow:

[ ] When entering the LTHR-384 pipeline in Limber with 4 LTHR Cherrypick plates, I should end up with 1 multiplexed tube at the end with all the samples pooled in it (rather than 4).
[ ] Update the Integration Suite test.

Additional context
I assume this should be done by creating 1 submission for all 4 plates, rather than 4 separate ones.
However, it could be done by changing the pooling behaviour at the end of the pipeline.

End of the pipeline might actually be easier, both for us, and the user. Although we'd need to remove the sequencing request from. the initial submission and generate it on the final tube instead.

End of the pipeline might actually be easier, both for us, and the user. Although we'd need to remove the sequencing request from. the initial submission and generate it on the final tube instead.

Yep, thinking about this, I think this could be just config only if we don't try and change the submission to include all 4 plates.

Yep, thinking about this, I think this could be just config only if we don't try and change the submission to include all 4 plates.

Ahh, noticed they still want to be able to support the separate pooling, in which case changing the submisison is the right way to go.

This will also require changing quad plate creation to validate that we have all plates of the submission in the quad
OR
Changing quad plate creation to find all plates automatically

Ahh, noticed they still want to be able to support the separate pooling, in which case changing the submisison is the right way to go.

This will also require changing quad plate creation to validate that we have all plates of the submission in the quad
OR
Changing quad plate creation to find all plates automatically


Breadcrumbs Component

Backwards Compatibility Implications
None
New Features

new breadcrumb (with title) component

Bug Fixes
None

This is ready for use, however, it is missing the animation that Antoine would like.  I'll endeavour to add animation before release although this could be added later...


chore: commitlint configuration

:tada: This PR is included in version 0.1.0 :tada:
The release is available on:

GitHub release
v0.1.0

Your semantic-release bot :package::rocket:


Update enhancement

Hello,
When an update is executed through WDT it updates all the model's elements even though they are the same on the actual domain, which is up and running.
Is it possible to change this in this way?

Let us say I have a model used to create a domain.
After some time I decided to add a new server on the domain so, I modify the model by adding a new server.
WDT update is executed and it queries the actual domain up and running, compares it against the model and updates just things (in this case to create a new server) that are different.

Kindly let me know whether this could be considered as an enhancement or not.
Best regards,
TAM

Hello there,
Just a question, is it possible to implement this? This is going to help us a lot and we are heavy user of Weblogic and SOA/OSB/BPM domains.
Thanks a lot in advance.
Best regards,
TAM

I can try to automate your idea. However, it would be nice to have this functionality out of the box.

@textanalyticsman From the start, WDT has always been a collection of "single-purpose" tools (that operate on a common data model) that can be chained together to meet different use case requirements.  For example, cloning an existing domain requires a combination of using the Discover Domain and Create Domain tools.   One could make an argument that the ability to chain together different tool invocations that operate on the same data (i.e., the model files) is out of the box functionality.  It certainly was a key design requirement when we started this project!
As Richard points out, it is possible to combine the use of the Compare Model and Update Domain tools to achieve your use case.   However, we will discuss whether or not it is possible to improve the Update Domain tool to make it more efficient.


sf package version create report command fails with "sObject type 'Package2VersionCreateRequest' is not supported"

Summary
One of our CI workflow runs had a timeout failure while running the sf package version create command on an unlocked package. This resulted in the following output:
{ "name": "PollingClientTimeout", "message": "The client has timed out. Run 'sf package version create report -i 08cWx0000000MyDIAU' to check the status.", "exitCode": 1, "context": "PackageVersionCreateCommand", "data": { "VersionCreateRequestId": "08cWx0000000MyDIAU" }, "stack": "PollingClientTimeout: The client has timed out. Run 'sf package version create report -i 08cWx0000000MyDIAU' to check the status...

Trusting the recommendation from the above error message, I ran the suggested command:
sf package version create report -i 08cWx0000000MyDIAU

This errored out with the following message:
Error (1):
Package2Version.ConvertedFromVersionId , AsyncValidation FROM Package2VersionCreateRequest WHERE
                                                              ^
ERROR at Row:1:Column:460
sObject type 'Package2VersionCreateRequest' is not supported. If you are attempting to use a custom object, be sure to append the '__c' after the entity name. Please reference your WSDL or the describe call for the appropriate names.

Steps To Reproduce
I do not know how to reproduce the issue but I guess that the sf package version create report doesn't behave as expected.
Expected result
I expect to see a report that mentions the original error or some success.
Actual result
Unexpected SOQL error as show above.
System Information
{
  "architecture": "darwin-arm64",
  "cliVersion"<EMAIL_ADDRESS>  "nodeVersion": "node-v18.17.0",
  "osVersion": "Darwin 23.6.0",
  "rootPath": "/Users/pozil/.volta/tools/image/packages/@salesforce/cli/lib/node_modules/@salesforce/cli",
  "shell": "zsh",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 3.2.10 (core)",
    "@oclif/plugin-commands 4.1.10 (core)",
    "@oclif/plugin-help 6.2.18 (core)",
    "@oclif/plugin-not-found 3.2.28 (core)",
    "@oclif/plugin-plugins 5.4.17 (core)",
    "@oclif/plugin-search 1.2.16 (core)",
    "@oclif/plugin-update 4.6.13 (core)",
    "@oclif/plugin-version 2.2.16 (core)",
    "@oclif/plugin-warn-if-update-available 3.1.23 (core)",
    "@oclif/plugin-which 3.2.19 (core)",
    "@salesforce/cli 2.68.6 (core)",
    "apex 3.6.3 (core)",
    "api 1.3.2 (core)",
    "auth 3.6.75 (core)",
    "community 3.3.2 (user) published 33 days ago (Sat Nov 02 2024) (latest is 3.3.6)",
    "data 3.11.4 (core)",
    "deploy-retrieve 3.15.13 (core)",
    "info 3.4.21 (core)",
    "lightning-dev 1.9.3 (user) published 30 days ago (Mon Nov 04 2024) (latest is 1.9.4)",
    "limits 3.3.40 (core)",
    "marketplace 1.3.6 (core)",
    "org 5.2.4 (core)",
    "packaging 2.9.3 (core)",
    "schema 3.3.42 (core)",
    "settings 2.4.6 (core)",
    "sobject 1.4.46 (core)",
    "telemetry 3.6.23 (core)",
    "templates 56.3.30 (core)",
    "trust 3.7.43 (core)",
    "user 3.6.3 (core)"
  ]
}

@pozil Can you rerun the failing command with the --dev-debug flag? That will give us some debug logs to work with. Thanks!

@pozil - I'm curious if setting the --api-version to something lower than the default API version changes anything.  E.g., sf package version create report --api-version 61.0 or 60.0.  That query uses the tooling API and we haven't changed that code in a long time so maybe something changed server side recently.

Hi both and thanks for the quick response.
Something happened on the server side because the command now works. I only realized this after trying both of your suggestions and retrying the original command so I don't think that they impacted the result. There must have been been an "unstable" state during the package creation process.
I now have the expected report followed by the list of packaging errors (as expected).
sf package version create report -i 08cWx0000000MyDIAU
Package Version Create Request
┌───────────────────────────────┬────────────────────┐
│ Name                          │ Value              │
├───────────────────────────────┼────────────────────┤
│ ID                            │ 08cWx0000000MyDIAU │
│ Status                        │ Error              │
│ Package Id                    │ 0HoWx00000000mPKAQ │
│ Package Version Id            │                    │
│ Subscriber Package Version Id │                    │
│ Tag                           │                    │
│ Branch                        │                    │
│ Created Date                  │ 2024-12-05 15:01   │
│ Installation URL              │                    │
│ Created By                    │ 005Wx000000RInFIAW │
│ Converted From Version Id     │ ID Unavailable     │
└───────────────────────────────┴────────────────────┘


Please remove platform/debian/qemu-arm-static

Please remove platform/debian/qemu-arm-static from the git archive or the release tar.gz .
A binary file should not be in a source archive.
If someone wants to use the armhf docker file, he/she should supply the qemu binary.

See also: https://robinwinslow.uk/2013/06/11/dont-ever-commit-binary-files-to-git


Create Cloud_database_connectivity_instructions.txt

Provide the proper instructions about how to integrate this within the project

Thank you For your Valuable Contribution


Install to both partitions in one call

You can mount the second partition temporarily and make the necessary changes to enable the service manually, since it's just adding a file link.

Hmm, mounting seems easy enough, but what of the service file
How would I tell systemd to enable it, unless I dig for the location of the config and manually change it...Let me look into how I could do it

Hmm, mounting seems easy enough, but what of the service file How would I tell systemd to enable it, unless I dig for the location of the config and manually change it...Let me look into how I could do it

You would need to manually create the symlink that systemctl creates for it. When running systemctl enable ... you'll see that it outputs that it creates a symlink. That's what you'd need to do manually.

Solved for the most part in the https://github.com/FouzR/rM_dualboot/tree/Improved branch
Will either merge with main and have the current code in main for a singular partition be it's own branch for people who only want to enable it in the current partition. Might make a 3rd branch for the code to only install to the fallback. Plans for the future I suppose


Improved Explicit Upgrade Message

Closes #492  This solves issue #492 by adding explanation as to why a package with a newer release is having its upgrade skipped, and why a package is not being reinstalled when it is the requested version
Edit: corrected error number

Ok, I have fixed it


[depth_from_video_in_the_wild]How to get the learned camera intrinsic?

Thanks for your work and sharing.
I tested your pre-trained models on my own image sequence.
It seems the trajectory_inference.py only provided the relative position.
The paper said your work can learn the camera intrinsic but I can't find any part about it in this project.

If I'm not mistaken, in order to retrieve intrinsics, you need to modify the model itself. Under modify I mean and intrinsic as a property to the model class, take the returned values from motion_prediction_net.motion_field_net, and assign it later as self.intrinsics = 0.5 * (mat + inv_mat).