Refactored TestChangeCase.

Improved readability.

Codecov Report

Merging #339 into trunk will decrease coverage by <.01%.
The diff coverage is 89.28%.

@@             Coverage Diff             @@
##             trunk     #339      +/-   ##
===========================================
- Coverage     57.9%   57.89%   -0.01%
- Complexity   10023    10024       +1
===========================================
  Files         1148     1148
  Lines        73798    73773      -25
  Branches      7331     7331
===========================================
- Hits         42732    42713      -19
+ Misses       28587    28580       -7
- Partials      2479     2480       +1

Impacted Files
Coverage Δ
Complexity Δ

...rc/org/apache/jmeter/functions/TestChangeCase.java
94% <89.28%> (-2%)
13 <11> (+1)

...apache/jmeter/extractor/TestBoundaryExtractor.java
97.88% <0%> (ø)
17% <0%> (ø)
:arrow_down:

...ocol/jms/org/apache/jmeter/protocol/jms/Utils.java
61.29% <0%> (+6.45%)
15% <0%> (ø)
: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 85a9f13...019b3d5. Read the comment docs.


Cross account ssm param AMI ID updates.

image-builder-lambda-update-ssm.py is nice if you only have one account and ImageBuilder is in that account.
But ImageBuilder pipelines offer AMI distribution accross accounts which is a typical use case.
Please update the example code to distribute the AMI IDs to SSM Params in a cross-account mode to show how the update of SSM Params takes place.

 One would be to use StackSets
Another would be to use an Advanced SSM parameter and RAM share it to other accounts: https://aws.amazon.com/about-aws/whats-new/2024/02/aws-systems-manager-parameter-store-cross-account-sharing/

"ImageBuilder pipelines offer AMI distribution across accounts which is a typical use case."

"an Advanced SSM parameter" would probably be the mechanism the AWS ImageBuilder team could use to provide the complete solution.
As it stands, AMI distribution is a mess in AWS.

Another way might be to use a step function which could assume a role in the target account and call the SSM parameter API, all using service integrations.


Message Kit Documentation Fixes Changes Made

In packages/docs/pages/middleware/gpt-sdk:

commes → comes
Reason: Typo fix
Dont → Don't
Reason: Missing apostrophe

In packages/docs/pages/skills/parsing.mdx:

skill → skills
Reason: Plural form needed

In packages/docs/pages/templates/agent/prompt.mdx:

Le me know → Let me know
Reason: Typo fix
Some times → Sometimes
Reason: Correct word form

Summary by CodeRabbit

New Features

Introduced a structured memory management system for chat interactions, enhancing chat history management.

Bug Fixes

Corrected grammatical errors in documentation for improved clarity.

Documentation

Minor textual adjustments made to clarify the definition of skills in the parsing documentation.

Thank you! all fixed in the new PR


change project authors

@tdimitri @kcpevey does this look right?

In discussions with @jack-pappas and @tdimitri they would prefer a different name as author. @rpekrul did you catch what it should be?

instead of https://sig.com please use https://github.com/rtosholdings

@kcpevey ^^ Please see project author.

@mattip so sorry, but I managed to mess up the git history on your fork. Since I'm in a time crunch and these changes are minor, I moved the required changes to a new PR https://github.com/Quansight/numpy-threading-extensions/pull/63


CAPO: disable floating IP

That will prevent it using 'direct attached FIP mode'.
And the port should be removed when the cluster is destroyed.

Untested now, don't merge yet.

@MaysaMacedo tested, please review.


converting UTC

how would I take my GMT date and convert it to my local timezone?
given
gmtDate="Fri Apr 15 2016 06:21:52 GMT-0500 (Central Daylight Time)"
this usually does it
moment.utc(gmtDate).format("MM/DD/YYYY hh:mm:ss a")
I have enabled the timezone part but just didn't quite see the example that might help me out.

http://momentjs.com/timezone/


Can we use it in GitHub Action

Feature request
Is your feature request related to a problem? Please describe.
in our repo we're running CI pipeline to run unittest that depends on PostgresDB. to setup database in github action it takes time , to improve CI pipeline speed it'd be nice if we could integrate github action and database.build
Describe the solution you'd like
able to connect database.build from GitHub Action
Describe alternatives you've considered
Additional context

Hey @kubosuke thanks for the issue. To confirm, are you hoping to point your GitHub action to a temporary Postgres DB hosted on database.build? The difficulty with this is that database.build runs Postgres databases directly in your browser via PGlite. This means that in order to interact with the database, you need an actual browser running (which is technically possible in a CI environment but probably not what you're looking for).
Instead you may be interested in connecting directly to PGlite itself within your runner which I've heard other folks doing. You can either connect directly to PGlite (via JS interface), or combine it with pg-gateway to make it behave like a real TCP server.
Let me know if this helps!

Thanks, sounds great, I'll look into them ~ 🙏


Updated post build event to work if solution folder contains space

Added quotes to post build event. It fails if the solution folder contains a space.

Thanks @dsparkplug for the PR and nice catch


More formulas in equations

The algebraic formulas section is missing some formulas.
Would like to add them under GSSOC' 21.
Thanks!

Go ahead
Deadline 4 April,2021 (11:59 PM IST)


Seminar 15

add seminar 15 summary
TBD: libo weixin openAI introduction PPT

@windmemory Good catch, just added ski bot plan in the summary.


Image based on alpine

Can we have an image based on alpine for scala-sbt.

The problem with more builds is that they take longer and it's more likely that one of them will fail. But let's try and see what happens :+1:


[AutoRelease] t2-network-2023-07-28-38538(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/2985
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1031
BuildTargetingString
azure-mgmt-network
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/2985


Fixed redis dependency.

Please see #19 for details.

Thanks for the report and PR.

The release is available on rubygems v0.3.6


[AutoPR datamigration/resource-manager] Remove TaskOutput base object.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2760

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2071

Codecov Report

Merging #2288 into restapi_auto_datamigration/resource-manager will not change coverage.
The diff coverage is n/a.

@@                             Coverage Diff                              @@
##           restapi_auto_datamigration/resource-manager    #2288   +/-   ##
============================================================================
  Coverage                                        52.12%   52.12%
============================================================================
  Files                                             5627     5627
  Lines                                           134591   134591
============================================================================
  Hits                                             70157    70157
  Misses                                           64434    64434

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 a2c6bc5...7fa4066. Read the comment docs.


Add ability to insert and remove items from the list

Restructured internals so that the controller does most of the work, as opposed to the widget
Move controller and list view to separate files

Codecov Report

Merging #1 into master will decrease coverage by 6.73%.
The diff coverage is 91.75%.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
- Coverage   98.48%   91.75%   -6.74%
==========================================
  Files           1        2       +1
  Lines          66       97      +31
==========================================
+ Hits           65       89      +24
- Misses          1        8       +7

Impacted Files
Coverage Δ

lib/src/expandable_sliver_list_controller.dart
89.47% <89.47%> (ø)

lib/src/expandable_sliver_list.dart
100.00% <100.00%> (ø)

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 eb1e6cf...a6d6b2c. Read the comment docs.


miri reporting UB

When trying to run my program with miri, it reports UB, however with the note that "the Stacked Borrows rules it violated are still experimental". This minimal example UBs according to miri. I don't know how to make it more minimal unfortunately without investing too much time into it. I'm not using circular directly, it's a transitive dependency through rc-zip.
use rc_zip::prelude::*;

fn main() -> Result<(), Box<dyn std::error::Error>> {
    let f = &include_bytes!("../tests/data/Test.docx")[..];
    let a = f.read_zip_with_size(f.len() as u64).unwrap();
    let entry = a.by_name("word/document.xml").unwrap();
    let mut doc = xml::EventReader::new(entry.reader());
    let _ = doc.next();
    Ok(())
}

Cargo.toml:
[package]
name = "ziptest"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
rc-zip = "2.0.1"
xml-rs = "0.8.16"

Is this a valid concern or a false positive?
Test.docx (but any Word document should do I guess)
Output:

Full output: output.txt

Running the test suite of this crate with miri also produces errors:


Consistently seeing more rows being dropped in minhash_spark.py compared to minhash.py

Hi @ChenghaoMou ,
I have been using minhash_spark.py via GCP dataproc (removed all the code present for bigcode) for deduplicating my multi-lingual dataset. To get an understanding on the reproducibility of the result, I also deduped the same multi-lingual dataset using minhash.py.
Currently, deduplication is performed on one individual language at a time.
When I ran this for the first language, I witnessed that minhash.py retained around 15-20% more documents as compared to minhash_spark.py.
minhash_spark.py output had around ~12M documents and minhash.py output had around ~14.5M documents.
In #28 , you have mentioned that for same algorithm, although the documents being removed is random, the number of documents being removed is same. But, I am witnessing different behaviour.
To validate my experience, I ran the deduplication over rest of the language subsets and found more documents were being dropped in minhash_spark.py.
It would be great if you can help me better understand this by answering a few question:

Does connected components used in minhash_spark.py create different clusters than union-find used in minhash.py?
If the number of clusters are same then shouldn't the number of samples in the outputs for both the scripts be same?
Is running the scripts on different machines responsible for this behaviour? If yes, what is the reason for this behaviour.

I would be grateful if you can share any info apart from the above questions which can help me troubleshoot this behaviour!
Thanks.

Hi @prikmm,
Thanks for creating this PR. One thing that might explain the disparity — the num_perm is slightly different in the two scripts (256 vs 250), though only b*r permutations are being used in all settings. This creates different PERMUTATIONS and results as a consequence. It should be fixed in the lastest commit. Let me know if the issue persists.

Hi @ChenghaoMou
Thanks for the looking into the issue.
But, regarding num-perm. I did handle that.
I set num-perm to 256 for running both minhash.
I forgot to mention this during issue creation.
Infact all the 4 parameters are same:
length: 5
n-gram: 5
threshold: 0.7
num-perm: 256

I see. Could you provide any example data for me to reproduce the issue? Could you also share the exact command you use to run the scripts?

Hi @ChenghaoMou , I'm facing the same problem using another local minhash deduplication, which removes significantly less documents than spark implementation. See https://github.com/huggingface/datatrove/issues/107

@jordane95
Can you share more details? Like the command or the log output?
I took a look at the dataset you shared. The immediate observation is that that particular dataset might not be suitable with near deduplication, especially when the Q is significantly longer than the answers. You would remove them as a result of Q overshadowing A with dominate ngrams, even though diverse answers to the same question might not be considered duplicates in reality. You might find it more helpful to do hierarchical deduplication: grouping similar Qs together then dedup based on As only within each group.
With regarding to your question, the text preprocessing before ngrams are significantly different in two repos. Different hashing functions can also lead to different performance. I don't think I can expect the same results. In fact, you can run the normal minhash script in this repo, which is based on UF, and compare the results more directly.

I have tried another implementation of starcoder, which produces nearly same deduplication rate as datatrove implementation. Is there any reason why bigcode didn't use the graphframe implementation in this code but re-implement it using self-defined functions?

We moved away from union find to spark implementation and then graphframe. Graphframe is used in the latest V2 (to be released).


workaround-removal-biased-locking

In JDK 20, JVM option -XX:-UseBiasedLocking fails the build.
One pragmatic solution to use -XX:+IgnoreUnrecognizedVMOptions - instead of creating different sbt configs for different JDKs.

@raboof @jrudolph @mdedetrich any objection to just removing -XX:-UseBiasedLocking from the sbtopt file?

No objection

Yep, just drop it, probably premature optimization anyway...

All for it as well, in fact in general I wanted to remove the more esoteric JVM options (-J-XX:+UnlockExperimentalVMOptions -J-XX:+UseJVMCICompiler is another one I would like to remove, I have seen it in other Pekko projects and while it may provide improvements to compile speed I have noticed on many occasions it also throws spurious exceptions/warnings).


kNN extension [rls/daal-2020-mnt]

Description
Extensions for k-Nearest Neighbors algorithm
Changes proposed in this pull request:

brute-force method of algorithm
kNN search

Merged copy of https://github.com/oneapi-src/oneDAL/pull/765 and https://github.com/oneapi-src/oneDAL/pull/776

http://intel-ci.intel.com/eaf46877-4925-f198-9a01-8cdcd4b723a1

https://github.com/oneapi-src/oneDAL/pull/924 - constructor change from another PU is applied

/intelci: run

/azp run

@averbukh  Do we want to see your changes https://github.com/oneapi-src/oneDAL/pull/928 in the release 2020u3?

/intelci: restart
set infra_branch=dev/pyakovle-change-ipldt-file

/intelci: restart
set infra_branch=dev/pyakovle-knn-remove-pydaal

http://intel-ci.intel.com/eaf6b4ec-de01-f1a2-b434-8cdcd4b723a1?

@averbukh Do we want to see your changes #928 in the release 2020u3?

Fix of #928 is added here.

CI of last fixes: http://intel-ci.intel.com/eaf6c578-510f-f1e1-924f-8cdcd4b723a1

@averbukh Do we want to see your changes #928 in the release 2020u3?

Yes


running grunt on atom-typescript-> JSX-related issues and more

I'm trying to build atom-typescript locally.
Did I miss some steps in configuring my environment or the local repo?
Things I did:

npm install
npm install grunt (why wasn't this in the package.json as a dev dependency?)
npm install grunt-ts (ditto?)

$ grunt
Running "ts:dev" (ts) task
Watching all TypeScript / Html files under : lib
Compiling...
### Fast Compile >>lib/globals.ts
### Fast Compile >>lib/linter.ts
     <lots of Fast Compiles snipped>
### Fast Compile >>lib/worker/parent.ts
### Fast Compile >>lib/worker/queryParent.ts
Using the custom compiler : ./node_modules/ntypescript/bin/tsc
lib/main/atom/views/rView.tsx(34,16): error TS17004: Cannot use JSX unless the '--jsx' flag is provided.
lib/main/atom/views/semanticView.tsx(101,16): error TS17004: Cannot use JSX unless the '--jsx' flag is provided.
lib/main/atom/views/semanticView.tsx(107,16): error TS17004: Cannot use JSX unless the '--jsx' flag is provided.
lib/main/atom/views/semanticView.tsx(109,13): error TS17004: Cannot use JSX unless the '--jsx' flag is provided.
lib/main/lang/transformers/transformer.ts(2,28): error TS2307: Cannot find module 'source-map'.
lib/main/tsconfig/tsconfig.ts(210,27): error TS2307: Cannot find module 'tsconfig'.

npm install grunt (why wasn't this in the package.json as a dev dependency?)

Because it broke atom package installer at some point : https://github.com/TypeStrong/atom-typescript/issues/600
Don't use grunt. Sadly you need atom-typescript to build atom-typescript as is documented here : https://github.com/TypeStrong/atom-typescript/blob/master/CONTRIBUTING.md#workflow Probably should be more up in the guide :-/ I'll move it around and then close this issue :rose:


Refactoring endpoint and request body

Codecov Report

Merging #8 (d28a050) into main (ebe00e8) will decrease coverage by 5.28%.
The diff coverage is 16.42%.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   53.77%   48.48%   -5.29%
==========================================
  Files          18       18
  Lines         848      957     +109
==========================================
+ Hits          456      464       +8
- Misses        392      493     +101

Impacted Files
Coverage Δ

src/posts/reader.rs
93.33% <0.00%> (ø)

src/server/mod.rs
0.00% <0.00%> (ø)

src/server/route/posts.rs
0.00% <0.00%> (ø)

src/server/route/search.rs
0.00% <0.00%> (ø)

src/text_engine/query.rs
0.00% <0.00%> (ø)

src/text_engine/schema.rs
62.35% <6.34%> (-30.92%)
:arrow_down:

src/markdown/frontmatter.rs
89.14% <90.47%> (-0.56%)
: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 ebe00e8...d28a050. Read the comment docs.


0.13.2 - failed to run custom build command for libudev-sys v0.1.4 (fedora)

Bevy version
0.13.2
Relevant system information
OS: Fedora Linux 39 (Workstation Editi
Host: 82RB Legion 5 15IAH7H
Kernel: 6.8.4-200.fc39.x86_64
Uptime: 4 hours, 41 mins
Packages: 2241 (rpm), 55 (flatpak), 11
Shell: bash 5.2.26
Resolution: 1920x1080, 2560x1440
DE: GNOME 45.5
WM: Mutter
WM Theme: Adwaita
Theme: Adwaita [GTK2/3]
Icons: Papirus-Dark [GTK2/3]
Terminal: gnome-terminal
CPU: 12th Gen Intel i7-12700H (20) @ 4
GPU: NVIDIA Geforce RTX 3070 Ti Laptop
Rust Version: cargo 1.77.0 (3fe68eabf 2024-02-29)

What you did

I installed all required dependencies for fedora. (That went. But libudev I have installed. For some of them it tells me that their installation would remove systemd (that's why it won't let me do it))
I added bevy to dependencies list in my cargo.toml
Run cargo run

What went wrong
An error occurred after "cargo run"
Additional information
error: failed to run custom build command for `libudev-sys v0.1.4`

Caused by:
  process didn't exit successfully: `/mnt/data/.dev/java/millionscraft/target/debug/build/libudev-sys-5ce38d591eac6f9c/build-script-build` (exit status: 101)
  --- stdout
  cargo:rerun-if-env-changed=LIBUDEV_NO_PKG_CONFIG
  cargo:rerun-if-env-changed=PKG_CONFIG_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=PKG_CONFIG_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=HOST_PKG_CONFIG
  cargo:rerun-if-env-changed=PKG_CONFIG
  cargo:rerun-if-env-changed=LIBUDEV_STATIC
  cargo:rerun-if-env-changed=LIBUDEV_DYNAMIC
  cargo:rerun-if-env-changed=PKG_CONFIG_ALL_STATIC
  cargo:rerun-if-env-changed=PKG_CONFIG_ALL_DYNAMIC
  cargo:rerun-if-env-changed=PKG_CONFIG_PATH_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=PKG_CONFIG_PATH_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=HOST_PKG_CONFIG_PATH
  cargo:rerun-if-env-changed=PKG_CONFIG_PATH
  cargo:rerun-if-env-changed=PKG_CONFIG_LIBDIR_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=PKG_CONFIG_LIBDIR_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=HOST_PKG_CONFIG_LIBDIR
  cargo:rerun-if-env-changed=PKG_CONFIG_LIBDIR
  cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=HOST_PKG_CONFIG_SYSROOT_DIR
  cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR

  --- stderr
  thread 'main' panicked at /home/gggedr/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libudev-sys-0.1.4/build.rs:38:41:
  called `Result::unwrap()` on an `Err` value: "\npkg-config exited with status code 1\n> PKG_CONFIG_ALLOW_SYSTEM_LIBS=1 PKG_CONFIG_ALLOW_SYSTEM_CFLAGS=1 pkg-config --libs --cflags libudev\n\nThe system library `libudev` required by crate `libudev-sys` was not found.\nThe file `libudev.pc` needs to be installed and the PKG_CONFIG_PATH environment variable must contain its parent directory.\nThe PKG_CONFIG_PATH environment variable is not set.\n\nHINT: if you have installed the library, try setting PKG_CONFIG_PATH to the directory containing `libudev.pc`.\n"
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...

My Cargo.toml
[package]
name = "millionscraft"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
bevy = "0.13.2"

Well, systemd-devel is the package you are missing. Could you post the error that comes up when installing it?

I ran into the same issue and managed to resolve it by installing libudev-dev on my machine.
sudo apt install libudev-dev

That's for Debian based distros, the issue talks about Fedora.

Well, systemd-devel is the package you are missing. Could you post the error that comes up when installing it?

 root  /  home  gggedr  dnf install systemd-devel
ookla_speedtest-cli                             833  B/s | 951  B     00:01
ookla_speedtest-cli-source                      781  B/s | 951  B     00:01
Error:
 Problem: The operation would result in removing the following protected packages: systemd, systemd-udev
(try to add '--skip-broken' to skip uninstallable packages)

@Vladimir-Urik  I found this thread with a solution: https://discussion.fedoraproject.org/t/the-operation-would-result-in-removing-the-following-protected-packages-systemd-systemd-udev-installation-package-alien/70276.

This issue can be closed, as it's not an issue with bevy but rather an issue entirely with the user's system.


High amount of errors when using 'copy blob container'

Storage Explorer Version: 1.3.1:
Platform/OS Version: Windows 10 Builkd 20180711.1
Architecture: ia32
Regression From: Unknown
Steps to Reproduce:

Copy container from azure to storage emulator using the 'Paste Blob Container' right click option

Expected Experience:

No or few error
Actual Experience:

I'm encountering a lot of errors.  They all read something like:
Server encountered an internal error. Please try again after some time. RequestId:6b307f74-15b9-4b94-a2d6-f3dc502bcfda Time:2018-07-23T11:49:42.7596018Z
Server encountered an internal error. Please try again after some time. RequestId:54787e78-6519-4bf0-abdb-197a14b3c2aa Time:2018-07-23T11:53:19.8344014Z

I don't think the issue is Azure side as we have no issues with the application and the connection I'm using is rock steady.

What emulator are you using? It's quite possible the issue is with the emulator, since the "server encountered an internal error".


Configuring beam heating

The docs currently do not have a description of the parameters to configure a beam (that I can find), although pydrad does have the functionality to do so.  (They just list the boolean to turn it on.)
Additionally, the template is slightly confusing:
https://github.com/rice-solar-physics/pydrad/blob/dd41f6adf9f6a55e0a4b921eadfa0cf4cf3aaaa4/pydrad/configure/templates/heating.beam.cfg#L5
If the beam is configured as time-independent, the first parameter beam.time_start is actually the duration of the beam (starting at time zero).  I'd suggest this parameter needs to be renamed or this should be clarified.

This is certainly confusing. The first time I went to use this, I was confused by the labeling of it as start time.
That being said, I think this is mostly a documentation issue as most users are not (or at least I hope not!) going to resort to reading the templates to figure out what is going on.
We could rename it something time_start_or_duration though that seems a bit clumsy. Maybe just a clarifying comment in addition to better docs would suffice?

We could probably be a bit more clever about this.  Since it's looking for a python dictionary, allow it to be named either duration or time_start, depending on what the user defines.  The length parameter at the top of the template dictates which should be used anyway.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove moot version property from bower.json

Per bower/bower.json-spec@a325da3

We may use this for tooling or other future uses.

Interesting choice. Not sure why you'd want to rely on something which they say to leave out and isn't even used, but to each their own. Best of luck

Also their maintainer says they probably won't ever use it: http://stackoverflow.com/questions/24844901/bowers-bower-json-file-version-property

We use it in our build process, so I'd prefer it it stays there and is maintained.


Caching serialized components for broadcasting (2+ Receivers)

Ideas:

store an Arc in the ComponentProtocol (but we'd like to avoid that for Copy types that can be cheaply copied)
serialize early into Bytes, that can then be copied cheaply


Fix for incorrect regex for module name

fixes #507

Codecov Report

Merging #560 into master will decrease coverage by 0.23%.
The diff coverage is n/a.

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


Admin API V2: swagger.json misses full declaration of /definitions/OutputStream

Please answer the following questions before submitting your doc issue. Thanks!
What issue do you find in Pulsar docs?
The declaration of /definitions/OutputStream is just a generic JSON object:
"OutputStream" : {       "type" : "object"     }
What is your suggestion?
Exactly define the schema of the JSON object.
Do you have any references?
Most of the definitions (if not all) have the schema correctly defined, this one is missed.
Would you like to fix this issue?
No.

@edgebroker sorry for the late response. Can you pls help address this issue by providing the link to the docs?


Using grpc 0.8.0 again.

#439 fixes the problems we had with closing connections in gRPC 0.8.0.  Using 0.8.0 again.

LGTM


(Bug) Button "Finalize" is active after finalization

Closes #975

The field is_finalized  now takes into account whether it is a mintedcapped or dutch crowdsale
Added variable to disable the button

See images
Crowdsale with all the tokens sold

Allow to finalize crowdsale before finish because all the tokens were sold

Disable button after finish

@mariano-aguero
Button finalize  disabled if crowdsale time over and not all tokens were sold

@dennis00010011b I made some changes, can you test again the same "case" ? Thank you


PandasDataset slow at creating when many large DataFrames are given

Description
The PandasDataset class is slow at constructing when several large DataFrames are given. It appears like this check is to be blamed.
To Reproduce
The following snippet takes something like 14 seconds to run on my machine:
import pandas as pd
from gluonts.dataset.pandas import PandasDataset

df = pd.DataFrame(
    {
        k: [1.0] * 5000
        for k in range(200)
    },
    index=pd.period_range("2005-01-01", periods=5000, freq="2H")
)

dataset = PandasDataset(dict(df))

What I tried
Changing the definition of is_uniform to
def is_uniform(index: pd.PeriodIndex) -> bool:
    ts_index = index.to_timestamp()
    return (ts_index[1:] - ts_index[:-1] == index.freq).all()

drastically reduces the runtime. However, this doesn't work with irregular offsets like MonthEnd (in fact, a test using 3M frequency fails): turning MonthEnd periods to timestamp makes their difference become irregular in terms of days:
import pandas as pd
pi = pd.period_range("2012-01", periods=3, freq="M")
print(pi[1:] - pi[:-1])  # Index([<MonthEnd>, <MonthEnd>], dtype='object')
dti = pi.to_timestamp()
print(dti[1:] - dti[:-1])  # TimedeltaIndex(['31 days', '29 days'], dtype='timedelta64[ns]', freq=None)

my suggestion is to just use the index even if it is regular or irregular without converting it to period and then back to time ranges....  as far as I can tell with pandas dataframe one already has the index for each time point so you can use the irregular time series approach...

@kashif that would require #1973 right?

Another fix would be to have a constructor option that disables the check, and have an alternative constructor from_wide_dataframe that does the check just once.
My example above is really about constructing a PandasDataset from a wide DataFrame: the solution relying on pd.melt (turning it a long DataFrame and then invoke from_long_dataframe) appears very slow when a lot of data is involved.
cc @rsnirwan

@lostella yes... I believe so... so just use the index directly and as you can see it all works without going back to period and date range again...


debug_mesh pipeline validation error.

With core checks of validation layers, running Chapter 6 produces following error:
...
Parsing GPU Technique debug
temp.shader

temp.shader

UNASSIGNED-CoreValidation-Shader-InputNotProduced(ERROR / SPEC): msgNum: 602160055 - Validation Error: [ UNASSIGNED-CoreValidation-Shader-InputNotProduced ] Object 0: handle = 0x5131e00000000419, name = debug_mesh, type = VK_OBJECT_TYPE_SHADER_MODULE; | MessageID = 0x23e43bb7 | Vertex shader consumes input at location 0 but not provided
    Objects: 1
       [0]  0x5131e00000000419, type: 15, name: debug_mesh
A breakpoint instruction (__debugbreak() statement or a similar call) was executed in RaptorEngine_Chapter6_Debug.exe.

Better late than never :D Turns out we did some refactoring for the pipeline definition and missed a couple of files. This should now be fixed.

Closing this for now, feel free to reopen if the issue is not fixed.


AsyncValidate triggering on change instead of on blur

This is a bug introduced in 7.2.0 (works fine in 7.1.2)
What is the current behavior?
Currently asyncValidate triggers onChange (and not only on the fields added to asyncBlurFields, but any field)
What is the expected behavior?
asyncValidate should only be triggered by submitting the form or blur happening on asyncBlurFields
Sandbox Link
https://www.webpackbin.com/bins/-L-636BVM6SM1Eky2PQQ
What's your environment?
redux-form 7.2.0
react ^16.0.0
Other information

@erikras - any idea when this would be included in a release?


USD Currency

Any guide/suggestion on how to set the currency to USD, by default Yuan (¥) is used

@inxunxa sure, we will add it recently

@inxunxa more currency has been add into fortune-sheet！


Typo in Verifier code

Found a typo here from a recent commit: https://github.com/pact-foundation/pact-node/blob/a640ebb31fd1748539c1e7084419a74cdcb27331/src/verifier.ts#L33
customProviderHeaderu should be customProviderHeaders

Thanks for reporting this! Fixed in 9.0.2.


style: Trim trailing whitespaces

This is a formatting commit that trims trailing whitespaces in all Python files.
First commit of a set of formatting commits to slowly get the project format to the style that black uses to have a consistent styling throughout the project ^^

Preparing for the final version of Spectre and the repo being set to read only, prs will be closed. Refer to Gecko's fork for the version that will receive updates past the final one.


Got a "SyntaxError: Unexpected identifier"

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
I got "Error invoking remote method 'send-http-request': SyntaxError: Unexpected identifier" when I try to make any request on my Bruno.
.bru file to reproduce the bug
meta {
name: test
type: http
seq: 2
}
get {
url: https://catfact.ninja/fact
body: none
auth: none
}
Screenshots/Live demo link

It's fixed in version 1.27.0, thanks 🙌🏽


Add The Ending

The game is going to need an ending, which we have been preparing for. In the map you will notice 3 regions, the gym, a grassy field, and a black void with a ramp leading up to a glowing white crag.
The idea is once you reach 1000-2000 exercise points, the screen fades to white with a staicky noise effect before suddenly cutting to black. A noise similar in structure to the hard cut from terry to blackness in the intro (this can be done via UI panels). As you fade back in, you are using a new pawn which uses the ThirdPersonController and ThirsPersonCamera that comes stock with s&box.
Instead of the default Terry in his gym clothes, we will switch to the buff handsome version of Terry with the speedo. As we fade in the music for this part of the game plays, then after a short period, the rant Mungus made plays.
The player will walk up to the crag then the screen will fade to white with a softer white noise more evocative of wind than static. The screen fades and the player, still in the third person controller, appears in a grassy field filled with trees. The sound of wind rushing through nature plays, the music has gone silent, we just have a moment where the player catches their breath. In the center of the field is our exercise equipment, walking up to it causes us to enter the exercise states again with the old pawn. The player can play infinitely from this point forward.

@Jac0xb or @taek-dev if you wanna work on this talk to me so we can work out timings.

Pretty much done


Hide GetConn

This makes the future implementation private. This is a braking change, but:

It is unlikely anyone was using GetConn directly instead of just awaiting it.

This opens the way for replacing the manual implementation with an async fn.

Ping. I have a patch that makes get_conn a plain async fn, but it depends on this.

ping. I have rebased it on top of current master.

I have rebased it once more.


Harvest should provide a FIPS compliant image

Use boringcrypto

https://kupczynski.info/posts/fips-golang/
Cross Compile with Zig
zig targets | jq .libc
CC="zig cc -target x86_64-linux-gnu" GOOS=linux GOARCH=amd64 CGO_ENABLED=1 GOEXPERIMENT=boringcrypto go build -trimpath -o bin -ldflags="-X 'github.com/netapp/harvest/v2/cmd/harvest/version.VERSION=24.07.30' -X 'github.com/netapp/harvest/v2/cmd/harvest/version.Release=1' -X 'github.com/netapp/harvest/v2/cmd/harvest/version.Commit=e896fa7a' -X 'github.com/netapp/harvest/v2/cmd/harvest/version.BuildDate=`date +%FT%T%z`'" ./cmd/harvest ./cmd/poller


update-drafts: delete completed specs

update-drafts ought to clean up specs that are finished—that is, specs that have no more keys to assign to features. This PR fixes that deficiency.
Although a change made in https://github.com/web-platform-dx/web-features/pull/1694 satisfies this, it counteracts the benefit of https://github.com/web-platform-dx/web-features/pull/1742, which was to avoid needless diffs (see https://github.com/web-platform-dx/web-features/pull/1782 for an example).

@Elchi3 Thank you! I ended up doing the PR from scratch in https://github.com/web-platform-dx/web-features/pull/1796. This worked really well and I think I may up the frequency for these updates as a result.


ctas task succeed after drop table

Steps to reproduce the behavior (Required)
1. create ctas task
2. drop table
3. check task

Expected behavior (Required)
task failed
Real behavior (Required)
task succeed
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

mysql> select current_version();
+----------------------------+
| current_version()          |
+----------------------------+
| BRANCH-2.3-RELEASE b202d93 |
+----------------------------+

This does not need to be repaired. At present, our ctas does not guarantee a relatively strong consistency. We will wait until users have this demand.


More user-friendly means of posting images

Right now users can only post images using the awkward markdown syntax (e.g.):
![Sorry guys](http://my.setale.me/image/052u2w2q3u3w/Schermata 2015-04-30 alle 19.09.21.png)
What's worse is that the above actually doesn't work.
This issue is to brainstorm and ideally decide on an alternative (or, that the current functionality is fine).

Idea: how about auto-converting image URLs to inlined images? It means straying from pure markdown, but I'm not sure how sold folks are on keeping that as a strict requirement.

using the awkward markdown syntax

omg, lol. markdown is standard around the community.

markdown is ok, but it is anoying to type
![Sorry guys](http://my.setale.me/image/052u2w2q3u3w/Schermata 2015-04-30 alle 19.09.21.png)
instead just copy - paste image url

Perhaps we can handle image URLs similar to how Slack manages them, by leaving the image URL in the message but displaying the image below?  We can still use markdown for inline messages.

displaying the image in chat would be wrong way, imho

@tunnckoCore Markdown is well known to developers but I don't think we necessarily want to make something just for developers. Let's try to be friendly about it.
I think @pokemonmegaman has a point -- we can just allow users to toggle whether or not they want images, videos, etc to be embedded. IRC Cloud does this like so:

There's room for allowing both.

@ngoldman sgtm, good point. :+1:

So i tried adding image preview from message containing links here's what i got :

any feedback is welcome :+1:

looks good @SkonTeam! I think it's best to keep the original link visible, and maybe removed the rounded corners too. feel free to open a PR when you think it's ready.


Can not load some of Google fonts

Web font loader can not load some of Google web fonts like
Bokor, UnifrakturCook, Taprom, Suwannaphum
I used below code for checking fonts loading and saw that above fonts have errors in loading.
Also I should say that I did not tested all of Google fonts.
WebFont.load({
			google: {
				families: [ fontName ]
			},
			fontactive: function( familyName, fvd ) {
				// Activation codes  #here.
			},
			fontinactive : function( familyName, fvd ) {
				console.log( 'font inactive', familyName, fvd );
			}
		});

Excuse me this issue occurred because I used these fonts with latin texts, and I saw that these fonts are not for latin texts.


License mishap

You left the Luke's License section in your repository.
The section says that donation links are welcome, but you made it clear earlier that donation links are not welcome, and only name and website link are allowed.
Forking and creating a PR to fix this is pointless, so I just created an issue.

Cheers.


No mimi-type for .ICO files

My browser always requests /favico.ico. The mime-type is not in the mime-types directory.
the file mime-types/type-ico should contain return "image/x-icon"

Solved in #3 . Thanks


Using ## causes the word to go to the center of the screen

I don't know why this is happening. Could someone help me out

This is probably because the output (on the right) has been styled so that h2 elements are centred. Something like:
h2 { text-align: center; }

Hope that helps.


gopls: 0.16.0 -> 0.16.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/gopls/versions.
meta.description for gopls is: Official language server for the Go language
meta.homepage for gopls is: https://github.com/golang/tools/tree/master/gopls
meta.changelog for gopls is: https://github.com/golang/tools/releases/tag/gopls/v0.16.1
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.16.1 with grep in /nix/store/rn52ijbpsss8xhh0fs1zy52anvffrfg4-gopls-0.16.1
found 0.16.1 in filename of file in /nix/store/rn52ijbpsss8xhh0fs1zy52anvffrfg4-gopls-0.16.1

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

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

emacsPackages.lsp-bridge
gopls
vimPlugins.vim-go

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/a7327a8b45f6589d5d7688391bd5e8b15ae0e446#gopls

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

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

3 packages built:

emacsPackages.lsp-bridge
gopls
vimPlugins.vim-go

Maintainer pings
cc @Mic92 @rski @SuperSandro2000 @zimbatm for testing.

Result of nixpkgs-review pr 324222 run on x86_64-linux 1

3 packages built:

emacsPackages.lsp-bridge
gopls
vimPlugins.vim-go


Remove Client.subscribe

Description:
The current implementation of Document.subscribe provides sufficient event support corresponding to Channel or Room. To simplify the usage of the Client as a simple Connection object, the Client.subscribe method should be removed.
Why:
Removing Client.subscribe, the system can be streamlined for better usability and consistent functionality.

The current ClientEvent includes four events: StatusChangedEvent, DocumentChangedEvent, StreamConnectionStatusChangedEvent, and DocumentSyncedEvent. Upon reconsideration, it appears some places may still require these events. What do you think about the following modifications?

StatusChangedEvent = activated | deactivated

Since it's not being used, it can be removed for now and added back when needed.

DocumentChangedEvent

This event can be used to detect changes in the document when performing manual sync. It can be moved to document.subscribe().

StreamConnectionStatusChangedEvent = connected | disconnected

Since the watch stream is established for each document, it can be moved to document.subscribe().

DocumentSyncedEvent = synced | sync-failed

It can be moved to document.subscribe().

Thank you for the detailed investigation. Since there is currently no use case for Client.Subscribe, it may be a good idea to remove it and add it later as needed.


Download as doc button visibility

The download as document button is visible even when there is no text in the list. It should be hidden like the CSV button

@akhilkumarmengani Resolved this issue. Please check and close

Changes look Perfect. Closing the issue.


Tidy up scif branches in soil mask repos

[x] Remove unused files

packages.txt
Dockerfile.template
generate_docker.py
configuration.py (move to recipe.scif)

[x] Use Pipenv or Conda environment.yml for dependencies instead of requirements.txt
[ ] Add metadata in recipe.scif (see configuration.py)
[x] Fix/remove container references in .scif files
[ ] Update README.md with new instructions

Close. Might move README & metadata sub-tasks to next sprint.
Not using ODM transformer repo.

Remainder of tasks moved to: https://github.com/AgPipeline/issues-and-projects/issues/147


validate row: TIMESTAMP WITH TIME ZONE fails for SQL Server and Teradata validations

Row validations of SQL Server to BigQuery and Teradata BigQuery fail.
See integrations test test_row_validation_core_types_to_bigquery in test_sql_server.py and test_teradata.py.
If you add col_tstz to the above tests and run them you will reproduce the error.

Hi,
For a datetime column with timezone, DVT generates incorrect sql

For teradata TO_CHAR(col_tstz, 'YYYY-MM-DD HH24:MI:SS') - instead DVT should generate TO_CHAR(col_tstz at time zone 'gmt', 'YYYY-MM-DD HH24:MI:SS')
For sqlserver convert(VARCHAR(32), col_tstz, 20) - instead DVT should generate `convert(VARCHAR(19), col_tstz at time zone 'utc', 20)

Generating the correct SQL will fix these issues.
Sundar Mudupalli

I had a quick look at Teradata support and believe the following edits would be a good start.
third_party/ibis/ibis_teradata/datatypes.py:
@@ -165,7 +165,7 @@ class TeradataTypeTranslator(object):
     @classmethod
     def to_ibis_from_TS(cls, col_data, return_ibis_type=True):
         if return_ibis_type:
-            return dt.timestamp(nullable=cls._col_data_nullable(col_data))
+            return dt.timestamp(nullable=cls._col_data_nullable(col_data), timezone="UTC")

         return "TIMESTAMP"

third_party/ibis/ibis_teradata/registry.py:
@@ -115,6 +115,12 @@ def _table_column(t, op):
     if alias is not None:
         quoted_name = f"{alias}.{quoted_name}"

+    if op.output_dtype.is_timestamp():
+        timezone = op.output_dtype.timezone
+        if timezone is not None:
+            timezone = "GMT" if timezone == "UTC" else timezone
+            quoted_name = f"{quoted_name} AT TIME ZONE '{timezone}'"
+
     return quoted_name

To get the correct point-in-time on SQL Server I suggest these changes:
third_party/ibis/ibis_addon/operations.py
@@ -64,10 +64,12 @@ from ibis.expr.operations import (
     Strftime,
     StringJoin,
     Value,
+    TableColumn,
 )
 from ibis.expr.types import BinaryValue, NumericValue, TemporalValue

 import third_party.ibis.ibis_mysql.compiler
+from third_party.ibis.ibis_mssql.registry import mssql_table_column
 import third_party.ibis.ibis_postgres.client
 from third_party.ibis.ibis_cloud_spanner.compiler import SpannerExprTranslator
 from third_party.ibis.ibis_redshift.compiler import RedShiftExprTranslator
@@ -644,6 +646,7 @@ MsSqlExprTranslator._registry[RandomScalar] = sa_format_new_id
 MsSqlExprTranslator._registry[Strftime] = strftime_mssql
 MsSqlExprTranslator._registry[Cast] = sa_cast_mssql
 MsSqlExprTranslator._registry[BinaryLength] = sa_format_binary_length_mssql
+MsSqlExprTranslator._registry[TableColumn] = mssql_table_column

third_party/ibis/ibis_mssql/registry.py (new file):
# Copyright 2024 Google Inc.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

import sqlalchemy as sa

from ibis.backends.base.sql.alchemy import (
    get_sqla_table,
)
from ibis.backends.base.sql.alchemy.registry import get_col

def mssql_table_column(t, op):
    ctx = t.context
    table = op.table

    sa_table = get_sqla_table(ctx, table)

    out_expr = get_col(sa_table, op)
    out_expr.quote = t._quote_column_names

    if op.output_dtype.is_timestamp():
        timezone = op.output_dtype.timezone
        breakpoint()
        if timezone is not None:
            # Using literal_column on Oracle because the time zone string cannot be a bind.
            out_expr = sa.literal_column(
                f"{out_expr.name} AT TIME ZONE '{timezone}'"
            ).label(op.name)

    # If the column does not originate from the table set in the current SELECT
    # context, we should format as a subquery
    if t.permit_subquery and ctx.is_foreign_expr(table):
        try:
            subq = sa_table.subquery()
        except AttributeError:
            subq = sa_table
        return sa.select(subq.c[out_expr.name])

    return out_expr

Note that this is only getting us to the correct point-in-time, we still have a format mismatch. But at it's a good step closer.

Hi Neil,
Thanks for the input and working code. I agree with the proposed change for Teradata and I have used it.
I tried the proposed change for MS SQL - I am not too happy with it. Basically with the change - when it is a field with a timestamp, I get a composite Label object - which is hard to work with in strftime_mssql. If we keep the existing code, I can check isinstance(arg.type, sa.dialects.mssql.base.DATETIMEOFFSET) in this function and cut the size of the time string to 19 characters to ensure it matches what is done in BQ. Let me know what you think.
Thank you.
Sundar Mudupalli

Because the length is VARCHAR then the precision should be irrelevant, the resulting value will just be as long as the convert_style makes it.
Therefore I don't think you need to change strftime_mssql at all and can revert it back to it's develop implementation, or did I miss something?

Neil,
Yes - the problem is that convert(VARCHAR, col_tstz at time zone 'utc', 20) prints 1970-01-03 03:00:03 +00:00, whereas BQ expects 1970-01-03 03:00:03 - hence the usage of convert(VARCHAR(19), col_tstz at time zone 'utc', 20) which limits it to the first 19 characters.
Of course,  I could limit it to 19 characters for both types (w and w/o timezone) - which would not work for the timezone naive types where we print milliseconds - but we don't do that now.
I am not seeing the benefit to creating a new ExprTranslator for mssql. I see a small benefit in keeping it as it is - the code is clear that I do something special with the DATETIMEOFFSET type.
We are at the mercy of the idiosyncracies of the different database engines - but that is why DVT is needed.
Thanks.
Sundar Mudupalli

Perhaps you could use arg_type.timezone to understand if the data type is time zone aware (like in strftime_bigquery()) instead of isinstance. Then we can try to keep some consistency regarding where the AT TIME ZONE clause kicks in?


import/order group by prefix

I import from all over my app, with a mix of relative and root imports. The current order rule covers most of my cases, and with #629 fixing #389 (sorting alphabetically), that will cover more. But one case that won't be covered is that I prefer to have all my global functions imported first, which come from a /lib directory, followed by my components which come from a /client/modules directory.
Sorting alphabetically, I would have to do
import someComponent from '/client/modules/someComponent';
import someFunction from '/lib/someFunction';

But I'd rather do
import someFunction from '/lib/someFunction';
import someComponent from '/client/modules/someComponent';

Within the rule itself, there's one way I could see it being defined
Use the existing groups option, and anything that isn't currently an option (builtins, sibling, etc) would be considered a prefix
    "import/order": ["error", {"groups": ["builtins", "external", "/lib", "/client"]}]
This would also handle the concern in this comment that people will start to request a bunch of order types and exceptions to them

If we allow any string there, then adding any new options goes from being a semver-minor to a semver-major, and there'd be no easy way to tell people when they have to update their configs.

What if the prefixes were done in a second option prefixGroups, like so?
"import/order": ["error", {
  "groups": ["builtins", "external"]
  "prefixGroups": [
    {
      "prefix": "/lib",
      "before": "external"
    },
    {
      "prefix": "/client",
      "after": "external"
    },
}]

The before and after arguments allow for ordering alongside the base groups that you have.

That would avoid the conflict I specified.
Also, instead of a prefix, we'd probably want to use globs - ie, /lib/** would be for a prefix, *_test.js for a suffix, etc (call it "pattern")

Globs would work as well. And I like the addition of the suffix.
What's the difficulty of doing something like this? I'd love to contribute this myself if possible. Do you have any suggested resources (aside from the code itself) on how to get started with writing part of a linter plugin?

I am also interested in this. Are there any plans to implement this?

Yea I would also like to see support for this.
Is it posible to also declare imports based on imported file extension?
For example I want to have .css and .md imports at the bottom of the list.

See my comment in https://github.com/benmosher/eslint-plugin-import/issues/807#issuecomment-415358830
I think in times of yarn/lerna workspaces, users need some degree of control about e.g. what is "internal". Having namespaced packages should actually help the situation.

Would love to see some action on this feature request.
Using babel-plugin-root-import, I wish import/order could interpret all my ~/ imports as one group.

Any news on this? :(


enable runner

Closes: #38

mmm if I start mess-up one-line PRs, maybe I should call it a day for now :sleeping:

@lucaferranti You'll need to add a test_runner.average_run_time field that estimates the amount of time the test runner takes (as an integer representing seconds). See https://exercism.org/docs/building/tracks/config-json#h-metadata
Judging by the speed running the tests of the test runner locally, it should probably be set to about 10 or so.
FYI I've opened a PR that we need to merge to allow deploying the test runner to production: https://github.com/exercism/terraform/pull/109

cool, I've set the average time. Just double-checking this is indeed average time, not maximum time threshold to timeout? Anyway, once the test-runner is deployed, I can test on the website an exercise wiht a lot of tests and see if it causes problems

@ErikSchierboom fixed the missing field. Let me know if anything else is missing


Netstream library relies on pac for error messaging

https://github.com/StyledStrike/gmod-gpaint/blob/master/lua/autorun/netstream.lua#L38 uses two instances of pac.Message for error messaging, meaning it would error out if PAC isn't installed and netstream errors out.

This pushed me to finally do it: I'm currently working on a refactor for the whole addon, and a replacement for netstream is one of the objectives too. I'll get back to ya once it's done.


remove vertical scroll from the  pipeline run graph

Fixes

https://issues.redhat.com/browse/HAC-3290
Description

Vertical and horizontal scrollers should not be shown for a pipeline with single task. Add a min height for the graph to avoid the scrollers.
Type of change

[x] Bugfix

Screen shots / Gifs for design review

Before:

After:

How to test or reproduce?

Browser conformance:

[x] Chrome
[x] Firefox
[x] Safari
[ ] Edge

cc: @MariaLeonova

/retest

cc: @jeff-phillips-18

/retest


Minor readability changes in Configuration File.md

Minor markdown changes to improve backgroundPreparationMode section readability.

@swift-ci please test


Minor DPrint fixes

See commit messages

Passing CI: https://github.com/tenstorrent-metal/tt-metal/actions/runs/7762608154
(Re-runs from connection stability issues with the CI runners)


REPL cache commands/configuration not working

I have a configuration file that specifies an in memory cacher (tried both cacher true and with options), yet the service broker never has any cacher active.
info shows that the config loaded is correct, i.e. cacher is true, but it also shows in the broker section that cacher is none, what am I missing here? Obviously the cache commands then complain about there being no cacher.
    Node version             : v12.22.8
    Moleculer version        : 0.14.19
    Protocol version         : 4

---

    Namespace                : <None>
    Node ID                  : cli-desktop-po9k7n1-32464
    Services                 : 1
    Actions                  : 7
    Events                   : 0

    Strategy                 : RoundRobinStrategy
    Cacher                   : <None>
    Nodes                    : 2

---

    default
      namespace              : ""
      metadata
      logger
        type                 : "Console"
      logLevel               : "info"
      cacher                 : true
      serializer             : "JSON"
      requestTimeout         : 10000
      .....

Hi @bbmarkus can you create a repro repo please?

It looks like your moleculer.config.js is wrong. There is no "default" property in the config file.

@icebob the config is passed through typescript, so the default is probably because of that, and it does work for the actual node running the microservices just fine the config gets picked up correctly there 🤔

In case someone bumps into similar issue, the fix was simply changing export default brokerConfig; to export = brokerConfig; in the typescript file.
I guess the config was never working to begin with, although we certainly never noticed this before 🤦.

Turns out moleculer-runner has support for export default in typescript files, so made a patch to make the file loading behaviour same between the two components @icebob


Really strange behavior...anyone else experience?

Note: If your question is regarding the AWS Amplify Console service, please log it in the
official AWS Amplify Console forum

Which Category is your question related to?
AWS Amplify CLI
Amplify CLI Version
You can use amplify -v to check the amplify cli version on your system
4.21.0
What AWS Services are you utilizing?
Amplify, appsync, cognito, s3, etc
Provide additional details e.g. code snippets
Two days ago updated to latest amplify-cli and sdk, now using expressvpn vpn as I have done for a few years, I can't reach endpoints in the US or Asia from the CLI.  Don't know how to explain as I am not that savvy at DNS issues.  If others have any ideas, much appreciated.

@weisisheng, which regions in Asia and the US are unable to be reached?

They are ap.southeast.1 and us.west.2.  I have done some scouring around the last few days and still can't find anything related to this type of issue + VPNs.  Using lantern or outline, both work although I have to monkey around with setting proxies on the CLI.

Hi @weisisheng the unavailability is outside of the scope of the CLI, and unfortunately we do not support using VPNs at this time so we can't provide any insight as to why expressvpn stopped working with those regions.

Sure, I understand. It was suggested in one of the discord groups that I post here, so gave it a shot.  Cheers.


Advanced connection options are showing in notebook drop-down

Azure Data Studio Version: main
OS Version:

Steps to Reproduce:

Open SQL notebook and connect to server

Expected: Connection drop-down is concise and easily readable.
Actual: Drop-down contains advanced connection options unnecessarily.  This is a change in behavior from April release.

@smartguest this appears to be a regression from your recent change regarding connection options.  Could you please take a look?

Sort of related - this "unnecessary info" is showing everywhere. Status in the bottom bar, titles, etc.
https://github.com/microsoft/azuredatastudio/issues/22802
So two things that should be looked at when fixing this:

Fixing the issue where we're showing extra information, even when it's unnecessary (for example, why is applicationName showing up at all?)
Making sure that we're only showing this additional information where it makes sense. We should explicitly go through places that display connection names and deciding if we want to actually show the "full" name there or keep it as the shortened version

@smartguest Regarding application name:

Treat 'azdata' prefixed names as default application names, but show otherwise (as we are going to support customization of app name soon and current app names will be suffixed)

Regarding other issues, only advanced options that are not defaults should be appended (even if option value is specified and its same as default, don't treat it differently)
But showing advanced options specified by user is a requirement (comes from an open issue) that can be handled either on hover or if not possible, goes in tab title content.
Status bars can show the same labels as before

@kburtram showing advanced options in dropdown is the only way to identify connection that has any advanced property enabled which would be useful for users, is there a different way to achieve this?
Considering we remove default azdata application name and treat it as default.

In most implementations of the EditorInput I've seen in ADS including table designer and query editor, the tab title is literally the same as the tooltip or the header name. I'm trying to find out how to make it only appear via tooltip, but without any example so far, it is left onto the tab title.

Closing issue as new connection string was removed

Reopening as new connection string is being reintroduced.

Closing as discussed offline, not introducing the same behavior again - will check-in with fixed design.


I get an error: electrumx.server.block_processor.ChainError: UTXO xxx / 0 not found in "h" table

It had been working fine before. Restarting after an unexpected power outage failed.
On Ubuntu server 22.04 LTS, thanks.
The Message is:
Atomicals envelope and operation found: dmt
Atomicals envelope payload: a16461726773a46474696d651a65698926656e6f6e63651a0011c2a468626974776f726b6367313834303030306b6d696e745f7469636b657268656c656374726f6e
INFO:BlockProcessor:advance_txs: atomicals_operations_found_at_inputs operation_found=dmt, operation_input_index=0, size_payload=99, tx_hash=3d20a1f64e6caed8b4ff7d8c3703ba72d0c19a8922f8d3b6429e418dc9c8d190, commit_txid=18400002920163dfd6eec9fce79b813ca980129ec1c91a69f1ea95846add659a, commit_index=0, reveal_location_txid=3d20a1f64e6caed8b4ff7d8c3703ba72d0c19a8922f8d3b6429e418dc9c8d190, reveal_location_index=0
INFO:BlockProcessor:get_atomicals_id_mint_info atomical_id=00000000536737aadfaffa17233bca342be2571e14916f6a29003ff4766d515283e68e90
INFO:BlockProcessor:get_atomicals_id_mint_info general_data_cache atomical_id=b'\x90\x8e\xe6\x83RQmv\xf4?\x00)jo\x91\x14\x1eW\xe2+4\xca;#\x17\xfa\xaf\xdf\xaa7gS\x00\x00\x00\x00'
INFO:BlockProcessor:atomical_id=b'\x90\x8e\xe6\x83RQmv\xf4?\x00)jo\x91\x14\x1eW\xe2+4\xca;#\x17\xfa\xaf\xdf\xaa7gS\x00\x00\x00\x00'
INFO:BlockProcessor:get_atomicals_id_mint_info atomical_id=00000000536737aadfaffa17233bca342be2571e14916f6a29003ff4766d515283e68e90
INFO:BlockProcessor:create_or_delete_decentralized_mint_outputs: found mint request in 3d20a1f64e6caed8b4ff7d8c3703ba72d0c19a8922f8d3b6429e418dc9c8d190 for electron. Checking for any POW in distributed mint record...
INFO:BlockProcessor:create_or_delete_decentralized_mint_outputs: has VALID mint_bitworkc 1840000 for<PHONE_NUMBER>0163dfd6eec9fce79b813ca980129ec1c91a69f1ea95846add659a for electron. Continuing to mint...
INFO:BlockProcessor:put_atomicals_utxo: atomical_id=536737aadfaffa17233bca342be2571e14916f6a29003ff4766d515283e68e90i0, location_id=3d20a1f64e6caed8b4ff7d8c3703ba72d0c19a8922f8d3b6429e418dc9c8d190i0, value=b83d51e56828a5cc09fd7a5ac0bf366ed6e67e2767e347492fe1596ce5e7c3a401bcb99cc94a4244cba31748d5000000000000878a573700
INFO:BlockProcessor:put_decentralized_mint_data: atomical_id=908ee68352516d76f43f00296a6f91141e57e22b34ca3b2317faafdfaa37675300000000, location_id=90d1c8c98d419e42b6d3f822899ac1d072ba03378c7dffb4d8ae6c4ef6a1203d00000000, value=5ac0bf366ed6e67e2767e347492fe1596ce5e7c3a401bcb99cc94a4244cba31748d5000000000000
INFO:BlockProcessor:advance_txs: create_or_delete_decentralized_mint_output:atomical_id_of_distmint - atomical_id=908ee68352516d76f43f00296a6f91141e57e22b34ca3b2317faafdfaa37675300000000, tx_hash=3d20a1f64e6caed8b4ff7d8c3703ba72d0c19a8922f8d3b6429e418dc9c8d190
INFO:BlockProcessor:advance_txs: create_or_delete_pow_records tx_hash=3d20a1f64e6caed8b4ff7d8c3703ba72d0c19a8922f8d3b6429e418dc9c8d190
INFO:BlockProcessor:advance_txs: has_at_least_one_valid_atomicals_operation tx_hash=3d20a1f64e6caed8b4ff7d8c3703ba72d0c19a8922f8d3b6429e418dc9c8d190
INFO:Prefetcher:cancelled; prefetcher stopping
INFO:Controller:shutting down
INFO:Controller:shutdown complete
ERROR:electrumx:ElectrumX server terminated abnormally
Traceback (most recent call last):
  File "/electrumx/electrumx_server", line 35, in main
    asyncio.run(controller.run())
  File "/usr/local/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "uvloop/loop.pyx", line 1517, in uvloop.loop.Loop.run_until_complete
  File "/electrumx/electrumx/lib/server_base.py", line 129, in run
    await server_task
  File "/electrumx/electrumx/lib/server_base.py", line 102, in serve
    await self.serve(shutdown_event)
  File "/electrumx/electrumx/server/controller.py", line 137, in serve
    await group.spawn(wait_for_catchup())
  File "/usr/local/lib/python3.9/site-packages/aiorpcx/curio.py", line 297, in __aexit__
    await self.join()
  File "/electrumx/electrumx/lib/util.py", line 370, in join
    task.result()
  File "/electrumx/electrumx/server/block_processor.py", line 3501, in fetch_and_process_blocks
    await group.spawn(self._process_prefetched_blocks())
  File "/usr/local/lib/python3.9/site-packages/aiorpcx/curio.py", line 297, in __aexit__
    await self.join()
  File "/electrumx/electrumx/lib/util.py", line 370, in join
    task.result()
  File "/electrumx/electrumx/server/block_processor.py", line 3461, in _process_prefetched_blocks
    await self.check_and_advance_blocks(blocks)
  File "/electrumx/electrumx/server/block_processor.py", line 310, in check_and_advance_blocks
    await self.run_in_thread_with_lock(self.advance_blocks, blocks)
  File "/electrumx/electrumx/server/block_processor.py", line 293, in run_in_thread_with_lock
    return await asyncio.shield(run_in_thread_locked())
  File "/electrumx/electrumx/server/block_processor.py", line 292, in run_in_thread_locked
    return await run_in_thread(func, *args)
  File "/usr/local/lib/python3.9/site-packages/aiorpcx/curio.py", line 57, in run_in_thread
    return await get_event_loop().run_in_executor(None, func, *args)
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/electrumx/electrumx/server/block_processor.py", line 494, in advance_blocks
    undo_info, atomicals_undo_info = self.advance_txs(block.transactions, is_unspendable, block.header, height)
  File "/electrumx/electrumx/server/block_processor.py", line 2726, in advance_txs
    cache_value = spend_utxo(txin.prev_hash, txin.prev_idx)
  File "/electrumx/electrumx/server/block_processor.py", line 3444, in spend_utxo
    raise ChainError(f'UTXO {hash_to_hex_str(tx_hash)} / {tx_idx:,d} not '
electrumx.server.block_processor.ChainError: UTXO 20d655006763e020e88dcaea8cc254fb6d25bca52b6ad31dcdfb08b1ca0ae3c0 / 0 not found in "h" table

unexpected power outage

The DB got corrupted. You should resync electrumx from genesis. :/
similar to e.g. https://github.com/spesmilo/electrumx/issues/216

OK Thanks


[Docs] fix defaultBitcoinMinHTLCInMSat comment

Comment says default value is 0, when it is in fact 1.

Default was changed here: https://github.com/lightningnetwork/lnd/pull/3697#discussion_r355104437 but comment was not updated to reflect new value.


Example code leads to memory leak

In your example code you're never disposing of the logger, which is going to lead to a OutOfMemoryException eventually.

Hi Cam,
Thanks for your message, the example code now displays the Using pattern.


CI: Archive coverage reports on failures

Since we require 100% covered, there is no reason to use time on archiving the reports unless it failed.

I merged this PR too fast.
Coverage is uploaded as an artifact so that the code climate workflow can retrieve it and upload it.
I've reverted the commit in the main branch to fix code climate (I didn't know there's a "revert" button in here).


Add support for .net 4.8

This would significantly ease the migration of existing applications using Microsoft.SqlServer.MessageBox since they currently only run on classic .NET Framework.

Thanks.  I'm in a similar boat too but I haven't gotten around to adding the additional framework support yet.  This will get me started.

And FYI - just uploaded this change along with net472 and net6.0 updates to nuget under version 1.0.1.


Small fixes in log messages to help debugging the file upload

This patch contains some small not important but useful fix:

change variable name in Multipart class destructor: the variable holds the m_uploadKeepFiles value, not the opposite
when unlink() (remove temporary files) returns with 0, the message was "Failed to delete file", which is wrong
when temporary file was created, not the used create mode showed

Hi @airween,
In the comments, you mentioned -

change variable name in Multipart class destructor: the variable holds the m_uploadKeepFiles value, not the opposite

The name of the variable seems to be unchanged. However, there is a message change. That is really what you meant to do?

Hi @zimmerle,

The name of the variable seems to be unchanged. However, there is a message change. That is really what you meant to do?

sorry, I wasn't accurate when I wrote the comment. I meant about SecUploadKeepFiles variable, and not for the name but for the meaning of this variable. If this configuration directive is set to On, then the engine will log:
Multipart: Cleanup started (remove files True)
but the IMHO the expected message should:
Multipart: Cleanup started (keep files set to True)
You're right, I have to change the comment - do you have any idea, what can I write that's more accurate?

Thank you @airween. The commit is Ok. The comment that you made here was not accurate, leading the impression that you have forgotten a commit or something similar. Now it is clarified. No need for you to do anything. I will proceed with the merging.

Merged! Thank you!


Color.value is deprecated with message "Use component accessors like .r or .g."

We probably need to use this method toARGB32 method- Color class - dart:ui library - Dart API.
Related to:

[packages] Color.value is deprecated · Issue #159739 · flutter/flutter
[Proposal]Provide a alternative to deprecated Color.value · Issue #160184 · flutter/flutter


Migrate image tiling and renaming functions into toffy

Is your feature request related to a problem? Please describe.
Erin has a number of helper functions to stitch and organize individual tiled images back together. We should add these to toffy, baking in additional validation and probably some refactoring.

The goal of this function is slightly different than our existing tiling scripts. The one we have currently in toffy takes a run and stitches it based on acquisition order. What we want is one that puts the image back together again based on the original shape it was acquired from. This assumes it was generated using the create_tile_image jupyter notebook.
What this means is that it will be an R1C1, RNCM image. We want to reassemble it back into that same original shape.
In addition, we want to allow for missing FOVs. I think we can probably write it from scratch, and it will require a design doc


Executor

related to https://github.com/106-inc/sim2022/issues/28

#25 and #26 also fixed.

Closes #25
Closes #26


[docs] make bold+italic work without escaping _

one can't grep for params in the docs because of escaped _ as in  ***loss\_scale\_window***, which makes it

difficult to find docs
allows more mistakes as in https://github.com/microsoft/DeepSpeed/pull/773 where a param is renamed but the search-n-replace misses the escaped versions.

so this PR uses an alternative bold+italics syntan which doesn't require escaping. **A_B** test: A_B
Also looking at diff some ***A_B*** weren't escaped in first place.
This was done with:
find . -name "*md" -exec perl -pi -e 'BEGIN{ sub a{$_=$_[0]; ~s|\\||g; $_}} s|\*\*\*(\S+)\*\*\*|"**".a($1)."**"|msge' {} \;

I used it for all ***'s inputs, but can change to have only for the ones containing _ - I just thought it'd be more consistent that way. Especially when new entries are added and copy-n-paste is done, so ***foo*** won't be copied and edited as ***foo_bar*** (i.e. escape missed because it wasn't in the original) as it's in a few places in the docs now.
But it's your call.

@stas00  Thanks so much for these significant docs improvements. They are greatly appreciated.

Actually, I'm not sure you needed to escape _ in the first place, if you look at reduce_bucket_size prior to this PR - it's not escaped:
***reduce_bucket_size***: [boolean]

| Description                                                                                | Default |
| ------------------------------------------------------------------------------------------------------------------- | ------- |
| Number of elements reduced/allreduced at a time. Limits the memory required for the allgather for large model sizes | `5e8`   |

Yet, it renders just fine:

Is it possible that just removing \ is enough and leaving *** is just fine?
In any case why do you even need bold+italic - italic fonts almost never aid readability. Most projects use fixed font for variables without bold or italic, so it reads even better.


Content revalidate fails on updating Server update bits

Once you get content revalidation SQL Table right (needs documented), there is a failure when submitting related to Set up_pending which seems to be log insert problem.
Who's tm_user = 0 ?
Traffic Ops fatal error occurred while processing your request.
Error at line 215 (    my $insert = $self->db->resultset('Log')->create()
DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception: DBD::mysql::st execute failed: Cannot add or update a child row: a foreign key constraint fails (`traffic_ops_db`.`log`, CONSTRAINT `fk_log_1` FOREIGN KEY (`tm_user`) REFERENCES `tm_user` (`id`) ON DELETE NO ACTION ON UPDATE NO ACTION) [for Statement "INSERT INTO log ( level, message, tm_user) VALUES ( ?, ?, ? )" with ParamValues: 0='CODEBIG', 1='Set upd_pending = 1 for all applicable caches', 2=0] at /opt/traffic_ops/app/lib/UI/Utils.pm line 215

Merged PR #820


Ducati rental V1.0

Ducati Rental V1.0
This application allows the user to reserve one of our motorcycle models once they have logged in. Also, the user is able to add another model to our database and reserve that specific model to get a test ride. Finally, the user can delete one of the motorcycle models if it is not available anymore.
To built this app we followed the following requirements:
Basics

Followed the layout provided by Murat Korkmaz on behance.

Core Features

Added a login/signup page for the user to register.
Implemented a navigation menu where the user can access the following sections:

Models.
Reserve.
My reservations.
Add motorcycle.
Delete motorcycle.
Log out.

Added a list of the motorcycle models on the main page.
Added a "details" page to see the specifications of each motorcycle.
Integrate a motorcycle form to add a new model to the database.
Added a "delete" page where the user can delete a motorcycle model.
Implemented a "reservations" page where the user can rent a bike model.
Added a "my-reservations" page where the user can see its rental list.
Built the mobile and desktop versions of the app.
The App is connected to an API built with Ruby on Rails & PostgreSQL: Back-end repository.

Hi @jevazquezb,
Wow, you did it 🎉

Thank you for the changes implemented 💪 🥇 ㊗
Unless you want to add more features, go ahead to your final presentation ⏩ ⏩ ⏩
You are about to finish the Microverse program.  You have come a long way!!!
Good luck in the software industry!!  I'll see you there. ✨
Congratulations!!!!!!  🎉

To highlight

Good job!!! ✔

Cheers and Happy coding!👏👏👏


Compound Types with Examples

A type system should handle compound types like Quantity
Not sure how to make a length, weight, speed quantity with dry-types
Expected behavior
Dry::Types.load_extensions(:length)
module Types
include Dry.Types()
end
class Equipment < Dry::Struct
attribute :height, Types::Params::Length.inches()
attribute :width, Types::Params::Length.inches()
end
radio = Equipment.new( height: "3", width: "4")
puts radio.height  => "3 inches"

Pls forward questions or feature requests to our forum https://discourse.dry-rb.org/ (see CONTRIBUTING.md)


Sc 40039/keepers benchmark test

Codecov Report

Merging #342 (849677e) into main (c4ff01b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #342   +/-   ##
=======================================
  Coverage   28.32%   28.32%
=======================================
  Files          16       16
  Lines        1762     1762
=======================================
  Hits          499      499
  Misses       1218     1218
  Partials       45       45

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 c4ff01b...849677e. Read the comment docs.

To run
CHAINLINK_IMAGE="795953128386.dkr.ecr.us-west-2.amazonaws.com/chainlink" CHAINLINK_VERSION="develop-nonroot" make test_benchmark_keeper

Closing this since the code had to be moved to the chainlink repo.