Can not compile the project

Hi.
I have tried multiple times (cargo clean -> cargo c), but always got the compiler error.
Either: (signal: 11, SIGSEGV: invalid memory reference), or other error such thread 'rustc' panicked at 'index out of bounds: the len is 10766 but the index is<PHONE_NUMBER>',
Is this only happen in my end?
❯ rustc -V
rustc 1.52.1 (9bc8c42bb 2021-05-09)

async-stripe on  master [$] is 📦 v0.13.0-rc3 via 🦀 v1.52.1 took 21m35s
❯ cargo c
   Compiling num-traits v0.2.14
    Checking num-integer v0.1.44
    Checking chrono v0.4.19
    Checking async-stripe v0.13.0-rc3 (/home/user/playground/async-stripe)
thread 'rustc' panicked at 'index out of bounds: the len is 10766 but the index is<PHONE_NUMBER>', compiler/rustc_span/src/symbol.rs:1619:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

error: internal compiler error: unexpected panic

note: the compiler unexpectedly panicked. this is a bug.

note: we would appreciate a bug report: https://github.com/rust-lang/rust/issues/new?labels=C-bug%2C+I-ICE%2C+T-compiler&template=ice.md

note: rustc 1.52.1 (9bc8c42bb 2021-05-09) running on x86_64-unknown-linux-gnu

note: compiler flags: -C embed-bitcode=no -C debuginfo=2 -C incremental --crate-type lib

note: some of the compiler flags provided by cargo are hidden

query stack during panic:
end of query stack
error: could not compile `async-stripe`

Caused by:
  process didn't exit successfully: `rustc --crate-name stripe --edition=2018 src/lib.rs --error-format=json --json=diagnostic-rendered-ansi --crate-type lib --emit=dep-info,metadata -C embed-bitcode=no -C debuginfo=2 --cfg 'feature="billing"' --cfg 'feature="checkout"' --cfg 'feature="chrono"' --cfg 'feature="connect"' --cfg 'feature="default"' --cfg 'feature="events"' --cfg 'feature="fraud"' --cfg 'feature="full"' --cfg 'feature="hmac"' --cfg 'feature="issuing"' --cfg 'feature="orders"' --cfg 'feature="sha2"' --cfg 'feature="sigma"' --cfg 'feature="webhook-endpoints"' --cfg 'feature="webhook-events"' -C metadata=d384d8648156eefe -C extra-filename=-d384d8648156eefe --out-dir /home/user/playground/async-stripe/target/debug/deps -C incremental=/home/user/playground/async-stripe/target/debug/incremental -L dependency=/home/user/playground/async-stripe/target/debug/deps --extern chrono=/home/user/playground/async-stripe/target/debug/deps/libchrono-0b18a506feaf800c.rmeta --extern hmac=/home/user/playground/async-stripe/target/debug/deps/libhmac-de42eb50dff098ea.rmeta --extern serde=/home/user/playground/async-stripe/target/debug/deps/libserde-96ff4cf191e06faa.rmeta --extern serde_derive=/home/user/playground/async-stripe/target/debug/deps/libserde_derive-11e136bb03edb9ee.so --extern serde_json=/home/user/playground/async-stripe/target/debug/deps/libserde_json-9bd9a6a16456798e.rmeta --extern serde_qs=/home/user/playground/async-stripe/target/debug/deps/libserde_qs-311f90b064b42263.rmeta --extern sha2=/home/user/playground/async-stripe/target/debug/deps/libsha2-d6b53bfa93fd5ad3.rmeta --extern smol_str=/home/user/playground/async-stripe/target/debug/deps/libsmol_str-db719f92a5b5c191.rmeta --extern thiserror=/home/user/playground/async-stripe/target/debug/deps/libthiserror-310b2d26a6f7e4c8.rmeta` (signal: 11, SIGSEGV: invalid memory reference)

async-stripe on  master [$] is 📦 v0.13.0-rc3 via 🦀 v1.52.1
❯ cargo c
   Compiling proc-macro2 v1.0.27
   Compiling typenum v1.13.0
   Compiling version_check v0.9.3
   Compiling unicode-xid v0.2.2
   Compiling autocfg v1.0.1
   Compiling syn v1.0.72
   Compiling serde v1.0.126
   Compiling libc v0.2.96
   Compiling ryu v1.0.5
   Compiling serde_json v1.0.64
   Compiling serde_derive v1.0.126
    Checking subtle v2.4.0
    Checking opaque-debug v0.3.0
    Checking cpufeatures v0.1.4
    Checking cfg-if v1.0.0
    Checking itoa v0.4.7
    Checking percent-encoding v2.1.0
    Checking smol_str v0.1.17
   Compiling generic-array v0.14.4
   Compiling num-traits v0.2.14
   Compiling num-integer v0.1.44
error: could not compile `libc`

Caused by:
  process didn't exit successfully: `rustc --crate-name libc /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/libc-0.2.96/src/lib.rs --error-format=json --json=diagnostic-rendered-ansi --crate-type lib --emit=dep-info,metadata -C embed-bitcode=no -C debuginfo=2 --cfg 'feature="default"' --cfg 'feature="std"' -C metadata=6135e5f4e746ab40 -C extra-filename=-6135e5f4e746ab40 --out-dir /home/user/playground/async-stripe/target/debug/deps -L dependency=/home/user/playground/async-stripe/target/debug/deps --cap-lints allow --cfg freebsd11 --cfg libc_priv_mod_use --cfg libc_union --cfg libc_const_size_of --cfg libc_align --cfg libc_core_cvoid --cfg libc_packedN --cfg libc_cfg_target_vendor` (signal: 11, SIGSEGV: invalid memory reference)
warning: build failed, waiting for other jobs to finish...
error: build failed

After reboot, I managed to compile the project. Now I just need to figure out a fix for these errors.

    Checking serde_qs v0.8.3
    Checking async-stripe v0.13.0-rc3 (/home/azzamsya/playground/async-stripe)
error[E0432]: unresolved import `crate::config`
 --> src/params.rs:6:12
  |
6 | use crate::config::{err, ok, Client, Response};
  |            ^^^^^^ could not find `config` in the crate root

error[E0432]: unresolved import `crate::config`
 --> src/resources/generated/balance.rs:7:12
  |
7 | use crate::config::{Client, Response};
  |            ^^^^^^ could not find `config` in the crate root

I can't find any clue in contribution guide

Compiler panic is not something I've seen and should probably be reported to rustc if you can reproduce it. As for the config errors, you need to make sure you are selecting a runtime when building.
I think you're right that the contributing guide should mention this explicitly, but you'll notice running tests requires selecting a backend (in this case we are using the blocking synchronous backend).
cargo test --features runtime-blocking

I just ran rustup update and compiled this successfully on the latest rust compiler.
> rustup update
> cargo clean
> cargo build --features runtime-async-std-surf
    Compiling version_check v0.9.3
    ...
    Finished dev [unoptimized + debuginfo] target(s) in 2m 27s

Yes, I have looked to the Github workflow file multiple times. I tought it was optional.
I tried using cargo build --features runtime-async-std-surf and  cargo build --features runtime-tokio-hyper-rustls both still resulting the same error:
error[E0432]: unresolved import `crate::ids::TaxCodeId`
 --> src/resources/generated/product.rs:8:29
  |
8 | use crate::ids::{ProductId, TaxCodeId};
  |                             ^^^^^^^^^
  |                             |
  |                             no `TaxCodeId` in `ids`
  |                             help: a similar name exists in the module: `TaxIdId`

...

Strange. Can I ask you what commit you are on? CI is all green on master a few hours ago on rustc 1.52.1
https://github.com/arlyon/async-stripe/runs/2792250345
If you run the compiler in a docker container for example can you still reproduce?

I am using the latest rustup, and rust. But it so strange. If I got the error in the previous compilation. The next compilation will always fail. Similarly, I managed to solve it by rebooting my machine. I suspect cargo saves something in /tmp.
async-stripe on  master [$] is 📦 v0.13.0-rc3 via 🦀 v1.52.1
❯ cargo build --features runtime-async-std-surf
   Compiling async-stripe v0.13.0-rc3 (/home/user/playground/async-stripe)
warning: unused import: `http::url::ParseError`
 --> src/client/async_std.rs:7:12
  |
7 | use surf::{http::url::ParseError, Body, Url};
  |            ^^^^^^^^^^^^^^^^^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

warning: 1 warning emitted

    Finished dev [unoptimized + debuginfo] target(s) in 58.77s


Metadata Name questions

Hi,
I think this tool will do nicely for what we want but a couple of questions please.
Background: We have somewhere between 35,000 and 60,000 files, mostly jpgs to 'Manage'
So far windows explorer has been used but only for file and folders.
so the folder path has often been used to house extra metadata with the result that there are thousands of duplicates
for example one jpg has been filed under
\Shared_Pictures\Euroa book ALL\1985 photos
\Shared_Pictures\Newspapers\1985 photos
\Shared_Pictures\Euroa book ALL\EUROA jul 2007\1985 photos
and about 4 other places.
A bit of thinking about this and the type of metadata in common includes

Where : Place (Town, street, property etc)
Who   : Family
When  : Era

So to the questions:
Can we define new metadata items such as 'Place' or are we restricted to the pool of Properties already in existence ?
Can we change the display name for an attribute ? Already I see the metadata name an item System.Keywords but explorer shows this as 'Tags'
and/or is there any other advice anybody could give?
Thanks
JC

Hi JC
It is possible to define new properties. It involves creating a fairly complicated definition file, and then invoking a Windows API to register it. However, I've never seen a program to help you do it, or heard of anybody doing it.
There are a couple of reasons for this. Firstly, the set of predefined properties is actually pretty comprehensive. Secondly, it turns out that for searching purposes you don't really need to use specific properties: putting all the values you want in as tags (or Keywords) is typically fine. In general, meta data values don't clash that much, apart from dates, and there are already a lot of good date properties.
Search can be invoked just by typing into the search box in the top right of Explorer, and will pick up on meta data values (assuming you added your folder tree to the list of folders to be indexed). You can also narrow your search using Boolean operators like AND, or look for specific properties, for example using keywords:=whatever
As far as I know, you cannot change the display names of already defined properties.
I have encountered similar problems with my own archives, and after various experiments I settled on a simple folder hierarchy based on date taken, with all the other information in tags.
One variation retained the folder structure to help me find things, but eliminated the duplication by using hard links. These are a mechanism in NTFS that allows a single physical file to appear in multiple places in the folder structure of a single hard drive. Search works better.
Another approach rebuilt what looked like a folder structure based on the tag values i.e. files appeared in a virtual folder if and only if they had the corresponding tag. This also had an idea of folder hierarchy, so that street folders appeared within town folders, et cetera. Again, it turned out that I didn't even use this myself, so I didn't publish it.
A program that did turn out to be more useful took my old folder structure and weeded out duplicates.
In your case, something that converted your old folder structure to a new one, preserving information by converting folder names to properties such as tags might be very useful. I don't how you are at programming, but a script in a language such as Powershall should be able to handle this.
Whatever, yours is a problem close to my heart, so feel free to ask for further clarification on anything here that is obscure.
Dijji

Dijji
Thanks for the feedback; we must be 'attuned' as I think I have ended up intuiting and/or adopting all of your suggestions except the links.
Once I found an actual list of the properties that I could search I settled on the following existing attributes

Place      System.Keywords    'Tags'
Era        System.Media.Year
Family     System.Contact.LastName
Topic      System.Category  eg 'Sport', 'Soldiers', 'Books', 'Cemetery'
Business/Property   System.Company
as well as generic ones
System.Comment
System.Title
Seeing the collection is of local history I wanted to allow for the separation of 'Family' and 'Place' for example - plenty of the early settlers ended up with localities or streets named after them for instance. So I choose not to just use 'tags' for everything.

There was also a bit of trial and error seeing some attributes are editable in explorer and some are not and I haven't found a lst of these.
eg  System.ItemParticipants and System.Photo.PeopleNames do not seem to be editable but System.Contact.LastName is
I am choosing to have (aiming for) all the files to be uniquely named in the ONE folder.
This will necessitate a fair bit of process/ habit change as now they cant just dump a batch of IMG_* files in a folder where ALL the context is hidden in the folder path.
For migration of the existing, as you say, programatically generating the metadata from existing folder names into appropriate attributes as above.
Its ambitious but I am pretty much there.  It took 3 aborted strategies before I have one looks like it will work.
From the starting 81,221 files in 1,478 folders I am down to 57,050 uniquely named files.  There are still 7,800 duplicates here but they can wait until stage 2.
The tool set ended up using some bits that are generally beyond the home user

A database - using SqlServer because it was already installed
an ETL tool - using Pentaho Data Integrator because I was already using it for a real client
'Duplicate Cleaner' (https://www.digitalvolcano.co.uk/duplicatecleaner.html)

I have details of all the specific steps and things to think about if anybody else is deep in the same situation.
Thanks again
JC


Hardsuit balance

About the PR

This Pr changes several of the stats for multiple of the hardsuits found in game.
 Sec having heat protection will help them a lot when they have to deal with emagged borgs, lazer raptors, behonkers, and every other threat on, and off of the station.
Technical details

All I changed, and added were stat values.
Media

[x ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: All Hardsuits have more armor!
tweak: Sec Hardsuits now have heat protection.
tweak: Brigmedic Hardsuit is much quicker with less protection.
tweak: Warden's Hardsuit is much slower with much more protection!
tweak: Salvage Hardsuits have exchanged armor for quicker mobility!
tweak: Wizard Hardsuits have much more protection
tweak: Pirate Hardsuits have more mobility!
tweak Captains hardsuit is more resistant to radiation, and shock.
tweak: CMO's Hardsuit has more protection against Chem explosions!

CL doesn't need to describe every exact change, just generally "Most hardsuit values have been changed." is enough, players can inspect it in-game themselves.

Do clowns/mimes really need that 70% rad resist?

I don't really get the point of balancing 5 - 10 - 15 - 20 - 25 - 30 as different hardsuit values, it just seems like unnecessary changes for the point of change instead of standardising properly and making it much easier to balance well.

Do clowns/mimes really need that 70% rad resist?

Both Clowns and Mimes deal and work with Bananium which is a radioactive material, and is the reasoning behind this

we do not want hardsuits to have this much armor, the values are the way they are for a reason

agree with sloth that most of these 5-10-15 point changes in stuff are really meaningless and arbitrary as well, I just don't really see the point

opting to close for the reasons listed above

The 5% makes it a lot easier to balance the suits around each other. For example the Engineer Hardsuits should have less combat capability than the basic standard Hardsuit, but more environmental protection. When it comes down to movement speed that 5% really does make a noticeable difference. Which changes in general do you not like? just the 5%? All of the suits getting more combat armor? certain ones? etc.

I don't really get the point of balancing 5 - 10 - 15 - 20 - 25 - 30 as different hardsuit values, it just seems like unnecessary changes for the point of change instead of standardising properly and making it much easier to balance well.

agree with sloth that most of these 5-10-15 point changes in stuff are really meaningless and arbitrary as well, I just don't really see the point

If I show you a video of 10% vs 15% movespeed you're not going to notice a difference unless it's side-by-side.

If I show you a video of 10% vs 15% movespeed you're not going to notice a difference unless it's side-by-side.

I tested them in game, but sure if you want I will check it out.

only maybe worthwhile change is making the brigmedic suit faster which I'm not even sure is used right now. if anything most of the values being buffed here should be going down, not up

only maybe worthwhile change is making the brigmedic suit faster which I'm not even sure is used right now. if anything most of the values being buffed here should be going down, not up

You think Pirates Hardsuits are in a good place? Sec doesn't deserve any heat protection at all? Salv should have more piecing protection than sec? Engineer and atmos dont deserve any basic combat protection at the cost of 30% slower movement speed? Also yes Brigmedic isnt used currently but it also needs some love, and I'd love to see it get reimplemented into the game one day. Why do you think these value should be going down instead of up a lot of these value are in fact going down with new one going up instead.

Pirates aren't even in the game lmao.

Pirates aren't even in the game lmao.

 I see them a lot in admemes which are always fun, and people always really enjoy them.

they are used for events, and pirates are underpowered as they currently are, a hardsuit buff for them would be nice

the issues preventing pirates from being a base rule and being unbalanced do not include "their hardsuit should be slightly faster"

You think Pirates Hardsuits are in a good place?

they are not in the game naturally

Sec doesn't deserve any heat protection at all?

in a hardsuit no they do not

Salv should have more piecing protection than sec?

i don't think this matters at all

Engineer and atmos dont deserve any basic combat protection at the cost of 30% slower movement speed?

no they do not

Also yes Brigmedic isnt used currently but it also needs some love, and I'd love to see it get reimplemented into the game one day.

then it can get rebalanced if they get readded to the game and we can actually see whether it matters or not

Pirate EVA (Pirates should be designed for quick attacks where they get in and out with loot)
-Blunt 20% --> 35%
-Slash 20% --> 35%
-Heat 60% --> 50%
-piercing 10% --> 35%
-Caustic 25%
-Radiation 0%
-Shock 0%
-Explosive 30%
-movement walk 40% slower --> 5%
-movement run 40% slower --> 5%
Pirate Captain (there is only one pirate captain and they would be able to afford a good hardsuit)
-Blunt 30% --> 50%
-Slash 20% --> 50%
-Heat 60%
-piercing 15% --> 60%
-Caustic 25% -->30%
-Radiation 0%
-Shock 0%
-Explosive 40% --> 50%
-movement walk 20% slower --> 10%
-movement run 20% slower --> 10%

the issues preventing pirates from being a base rule and being unbalanced do not include "their hardsuit should be slightly faster"

It's not just "slighty faster" I changed a lot of other values as well. Pirates are designed for getting in, stealing loot, and getting out alive.  And even if they are "just a week away" this would make pirate admemes even more enjoyable for everyone.

If I show you a video of 10% vs 15% movespeed you're not going to notice a difference unless it's side-by-side.

While 0.9 vs 0.85 might not seem like much, relative difference of 0.9 vs 1 and 0.85 vs 1 is way more noticeable and outside of spaced areas it can be a huge deal.


Above the Fold CSS (Critical CSS)

Is there an established way to have (criticalCSS) above the fold CSS placed inline in the html file and the rest in a separate CSS file?
Isomorphic does this for templating done in React templating. But what if templating is done outside of javascript?
I could run the output of extract-text-webpack-plugin through another build tool, but was wondering if it exists within the webpack ecosystem.

Same question as https://github.com/webpack/extract-text-webpack-plugin/issues/187 ?

Let's close in favor of #187. Better discuss there.


@W-14325484@ Update sample integration to support MDG

What does this PR do?
This PR is just to merge MDG (multi delivery group) related changes which was pushed to p4/main before.
What issues does this PR fix or reference?
@W-14325484@
Functionality Before
MDG wasn't supported before this change.
Functionality After
MDG will be supported after this change.
How to Test/Testing Effort
The functionality has already been tested before and pushed into p4/main. This is only for merging the same changes here.

Here's the original change which couldn't be pushed at that time because of Heroku decommissioning: https://github.com/forcedotcom/commerce-on-lightning/pull/262/files


Sudoku Solver using Vanilla JS

WebMasterLog Repository (Proposing new issue)
:red_circle: Project Title : Sudoku Solver
:red_circle: Aim : Build a website which can solve a Sudoku by entering a numbers of the problem of the sudoku.
:red_circle: Tech Stack Used : HTML, CSS, JS
:red_circle: Category :  Vanilla-JS

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be all the main components.

Cover image of the application, if project will be deployed in future, with the name of the folder in Kebab-Case (hypen-separated).
Main files in formatted state so that code will be readable.
Other Images - In seprate folder named Images used to form the application.

Inside the Project folder, the README.md file must be filled up properly, with proper visualizations and descriptions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
What is your participant role? (Mention the Open Source Program name. Eg. GSOC, GSSOC, SSOC, JWOC, etc.)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

I Would like to Work on this issue.
Please assign it to me under JWOC'2024.

sir , please assign it to me.

Please assign me under JWOC 2024.
Name = Sohan RoyChowdhury.
Github = https://github.com/SohanRC
Role = Under JWOC 2024.

Assign me under JWOC
Name : Aftab Mankapure
Github: https://github.com/AftabMankapure
Role: Under JWOC

Please assign me under JWOC 2024.
Name = Sohan RoyChowdhury. Github = https://github.com/SohanRC Role = Under JWOC 2024.

Please assign me this issue under JWOC 2024.

@SohanRC Assigned to you. Follow the contributing guidelines carefully.


update generated files for prometheus operator v0.39.0

Updating to prometheus-operator v0.39 and then merging #519 caused the generated files to become out of sync.  This is change is just the result of a "make generate" against current HEAD

I wonder why it wasn't regenerated when I run make all for https://github.com/coreos/kube-prometheus/pull/524 :thinking:

@paulfantom I think it was an unlucky combination of merging #519 without first rebasing it onto #524


#1425 #1422 #1441 Link to repo in package details should be clickable

#1422 Package Manager UI: Library in details page should be clickable
#1441 Packaga manager: Move list of authors to the bottom
#1425 Link to repo in package details should be clickable

:+1:


是否需要配置ssl自签名证书？对比electrumx-proxy有什么优势？

rt

If you want to share your proxy with others, using an SSL certificate is optional and has nothing to do with elex-proxy. Unlike electrumx-proxy, elex-proxy is much faster and way easier to set up and use.

If you want to share your proxy with others, using an SSL certificate is optional and has nothing to do with elex-proxy. Unlike electrumx-proxy, elex-proxy is much faster and way easier to set up and use.

electrumx-proxy只支持tcp，elex-proxy支持tcp+ssl吗？

Elex-proxy is like a revamped version of electrumx-proxy, but it's all about using the WebSocket protocol for swapping data. Anyway, it's not related to what you were talking about.

Elex-proxy is like a revamped version of electrumx-proxy, but it's all about using the WebSocket protocol for swapping data. Anyway, it's not related to what you were talking about.

I downloaded and unzipped elex-proxy-Linux-x86_64-musl.tar.gz on ubuntu and got elex-proxy, but I don't know how to run it

You can directly execute the script below. If you need additional configuration, please follow the instructions in the README.
./elex-proxy


Icon Fonts

For a while, icon sets were distributed primarily as font files on the web, hence the term icon font. While many designers and developers are shifting to SVGs instead, they clearly constitute a user case for symbolic characters.
Almost all icon fonts rely on PUA code points exclusively, although many of their symbols have obvious or at least approximate Unicode equivalents. The prime reason for this is that some of the symbols would be drawn from default system fonts otherwise, the native emoji font in particular. Many designers apparently hate that and instead accept that there is no fallback at all when the font fails to load. Amusingly, some icon fonts use the same PUA code points that Apple originally used for their emoji set before it became part of Unicode – and still supports to remain compatible with legacy content – which means that a failing icon font may result in unrelated emojis showing up. That is just one of the many problems of icon fonts.
However, as for the contents of the most popular icon fonts, much is already covered by Unicode, much is inapplicable (i.e. logos) and much of it is redundant (e.g. various weights, circled and squared variants of the same symbol without canonical semantic difference) or questionable (e.g. representable by a short sequence of existing characters). The rest should be collected here, then be compiled into a formal proposal.
After all, Font Awesome alone has probably seen more use on the web than Wingdings 1--3 and Webdings -- not to mention Zapf Dingbats -- ever have.
Specific References

Font Awesome Fort Awesome

Font Strap / Font Awesome More

Material Design Icons Google
Octicons Github
Ico Moon
Typicons
Entypo
Ionicons
Ico Font
Zondicons
MFG Labs
Captain Icon
Devicons
Themify Icons
Foundation Icons
Ionicons
Linecons
Open Iconic
Just Vector Icons
SVG Icons
Glyphincons
Iconic
Foundicons
Sosa
Raphael Icon Set
Line Awesome
Dashicons Wordpress
Linea Iconset
iOS7 Vector Icons
Genericons
Stroke 7, Stroke 7 Icon Font, Stroke Icons
Icon-Works
Solid Icons
77 Essential Icons
Tonicons
Elegant Link Icon Font
Dripicons
Metrize Icons
Line-Styled Icon Pack
Ionicons
Batch Icons
Ligature Symbols
Open Web Free Icon Font
PW Drawn Icon Font
Premium Pixels
Fontelico
Icons V.3
Open Iconic
Linecons
Outlined Icons
Icony
Stackicons
Map Icons
Weather Icons
Meteocons
Icono
Feather Icon Set
Round Icons
Icon Sweets
Smart Icons
Ego Icons
Elusive Icons
Modern Pictograms
Maki
Web Symbols
Openmoji Emoji set with some extensions
Rivoliconos
Mmmicons
Prometheus
Interface Icons
Evil Icons
Jam
Bicon
Swifticons
Robicons
Epic Outline Icons
Weather Icon Font
Helium
Leksico
Iconia
Fonticon
Minimal

Missing Links

One Div
Goodies

Standard References

How to propose normal additions or changes to Unicode (required form, email address: docsubmit＠unicode.org)
Unicode document submission form

SAP UI5 Icons, see also #407.


[BUG] ClassCastException in AWS transport error handling

What is the bug?
The cast from Throwable to Exception in AwsSdk2Transport misses the case when a Throwable does not extend Error or Exception.
Unfortunately I don't have any other information other than that the class of the casted Throwable is simply Throwable (as opposed to some other inheritor of Throwable that is not Error or Exception).
My guess is that the HC5 transport is also vulnerable to this bug.
How can one reproduce the bug?
So far it's been a one-off.
What is the expected behavior?
Exception handling should not fail!
What is your host/environment?
Linux. OpenSearch 2.12.0.
Do you have any screenshots?
Nope.
Do you have any additional context?
This is part of the error handling code to wrap exceptions whose stack traces are unrelated to the caller due to the fact that the AWS transport is asynchronous.

[Catch All Triage - 1, 2, 3, 4]


Price data for native tokens

Some tokens will show pricing and then not show it at another time. From speaking with the dev I think he is pinging the individual pool API which is not updated as often or only updated when trades actually take place (could be more than 24 hours for lesser used coins). The correct API link for real time trade/pricing is this:
https://api.ergodex.io/v1/amm/markets?from=1653075262280
Special thanks to Gazza for helping me with this info when I was building my app.
Here is a link to how I parse out the dump for the coins I collect info off of ergodex (spectrum) if it helps at all (prob not as it's in perl): https://github.com/andrehafner/my.ergoport.dev/blob/main/priceupdater.pl

Thank you, checked and the prices seem to be good. Next version will use the magic from parameter.


Add first two frames to application

Please look at below two screenshots and implement it in Bulma
[

](url)

Adding Icon files shown in mockups here.
[

](url)


Alias structs don't have any fields in Vulkan

For example, BufferMemoryRequirementsInfo2KHR has no fields whereas the struct of which it's an alias, BufferMemoryRequirementsInfo2, has fields.

Closed by #137


Add breaking tests to test Element.equals

Related to #614, #561, and #554, this introduces two tests to assert that selectors return an expected number of elements.

Note that the failing checks are on purpose, pending a resolution to #614. I can make the code changes and/or update tests on resolution of #614.

Thanks - fixed by changes for #614


Rails/BulkChangeTable docs sentence

There's a sentence which doesn't make sense:
"When use this method, make combinable alter queries a bulk alter query."
In the docs for Rails/BulkChangeTable cop... (in this repo)

I could have a go at rewriting it myself:
"This option causes the migration to generate a single ALTER TABLE statement combining multiple column alterations."
...but I'm not sure if that's correct, or using correct terminology.

Thanks @harry-wood !
Your proposal seems to be a better than the sentence I wrote. Would you open a pull request?


Move getCircularReplacer() to common-utils package

I'd appreciate your help here :)

am I moving out the extraction under error?.type === "TransportError" condition? or the whole thing including the if clause before and else clause after

@chensixx  is there still work here? if so can you move to the jan milesone, otherwise close it


DataQueryRequest enricher

Something like this as an alt to https://github.com/grafana/scenes/pull/303
Closes https://github.com/grafana/scenes/issues/298

Closing in favor of next iteration: https://github.com/grafana/scenes/pull/311


[Discussion] Public Release Version 2.0.2

Hello here, and thanks for all that stuff ! really usefull !
I'm going to update the Archlinux AUR package of spleeter and I need a public release at 2.0.2 version.
Is it possible to provide such a release ?
Thanks a lot in advance.

Hi, thanks for maintaining Spleeter on Arch. It would be better to wait just a little as we are about to release 2.1.0 in coming days. I would let you know at this time on this issue :)

Hi, thanks for maintaining Spleeter on Arch. It would be better to wait just a little as we are about to release 2.1.0 in coming days. I would let you know at this time on this issue :)

Ok for me to wait for the 2.1.0 release. Thank you.

Ok for me to wait for the 2.1.0 release. Thank you.

Alright,  we release the new version and apply some patches, current version 2.1.2 should be good for you to go, the package is available on PyPi: https://pypi.org/project/spleeter. Please note that we do not have CPU / GPU package anymore but only one that works on both CPU and GPU.

Alright,  we release the new version and apply some patches, current version 2.1.2 should be good for you to go, the package is available on PyPi: https://pypi.org/project/spleeter. Please note that we do not have CPU / GPU package anymore but only one that works on both CPU and GPU.

Thanks a lot @Faylixe ! I'm building the package... :-) Will let you know if it's working properly on Archlinux.

Thanks a lot @Faylixe ! I'm building the package... :-) Will let you know if it's working properly on Archlinux.

I have have a great working version of the original Spleeter 1.43 release with modifications for 16 khz & even 22.4 khz stem separations that run fine on my Windows 10 computer. I have even wrote bin files to do multiple song separations that work fine also. Is there any reason to upgrade to a newer version? Has the Stem separations been improved on?
I was told there would be a new Spleeter version released that would provide a slight improvement -- is this 2.02 version it & that a Pro version with a much larger training base has been sold to a US company that should be making an announcement about it but heard nothing about either -- any more details?
Thanks, Rog
 On Monday, January 11, 2021, 05:52:32 AM CST, Félix Voituret<EMAIL_ADDRESS>wrote:

Alright, we release the new version and apply some patches, current version 2.1.2 should be good for you to go, the package is available on PyPi: https://pypi.org/project/spleeter. Please note that we do not have CPU / GPU package anymore but only one that works on both CPU and GPU.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I have have a great working version of the original Spleeter 1.43 release with modifications for 16 khz & even 22.4 khz stem separations that run fine on my Windows 10 computer. I have even wrote bin files to do multiple song separations that work fine also. Is there any reason to upgrade to a newer version? Has the Stem separations been improved on?
I was told there would be a new Spleeter version released that would provide a slight improvement -- is this 2.02 version it & that a Pro version with a much larger training base has been sold to a US company that should be making an announcement about it but heard nothing about either -- any more details?
Thanks, Rog
 On Monday, January 11, 2021, 05:52:32 AM CST, Félix Voituret<EMAIL_ADDRESS>wrote:

Alright, we release the new version and apply some patches, current version 2.1.2 should be good for you to go, the package is available on PyPi: https://pypi.org/project/spleeter. Please note that we do not have CPU / GPU package anymore but only one that works on both CPU and GPU.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Remove deprecated mkfs-ext4-parameters setting

https://github.com/longhorn/longhorn/issues/5849

I've tried a number of times to test these changes in Jenkins, but I can't currently get a working version of master-head deployed (https://ci.longhorn.io/job/private/job/longhorn-tests-regression/3719/). I have a dev cluster running with a longhorn-manager build from my 4914-remove-mkfs-ext4-parameters-setting, and select test cases (including the one called out in https://github.com/longhorn/longhorn/issues/5849) are passing.
eweber@laptop:~/longhorn-tests/manager/integration> kubectl logs -f longhorn-test
Defaulted container "longhorn-test" out of: longhorn-test, longhorn-test-report
============================= test session starts ==============================
platform linux -- Python 3.9.16, pytest-5.3.1, py-1.11.0, pluggy-0.13.1 -- /usr/bin/python3.9
cachedir: .pytest_cache
rootdir: /integration, inifile: pytest.ini
plugins: repeat-0.9.1, order-1.0.1
collecting ... collected 395 items / 394 deselected / 1 selected

test_node.py::test_node_disk_update PASSED                               [100%]

================ 1 passed, 394 deselected in 370.19s (0:06:10) =================
eweber@laptop:~/longhorn-tests/manager/integration>

I ran a suite of tests against master-head in my local cluster. After fixing an environment issue, these four tests are failing:
test_basic.py::test_expansion_canceling FAILED                           [ 50%]
test_basic.py::test_expansion_with_scheduling_failure FAILED             [ 52%]
test_migration.py::test_migration_with_rebuilding_replica FAILED         [ 86%]
test_migration.py::test_migration_with_restore_volume[s3] FAILED         [ 88%]

The top two are explained by https://github.com/longhorn/longhorn/issues/5860. I am still looking for an explanation for the bottom two.

Merge until @ejweber clarify the failed cases.

The two unexplained failures seem to have been flakes. I am having no trouble getting them to pass in my cluster now. I suggest we merge.
test_migration.py::test_migration_with_failed_replica PASSED             [100%]
test_migration.py::test_migration_with_rebuilding_replica PASSED         [100%]


"Color by column" now works again

I just checked, in the spatial query notebook I get this:

While this was what we got in the past:

Is this expected?

Do you mean that both now use blue? I'm rather confused why the old one had two colors

Yes. You are right, the all blue plot is actually the correct one.


 This isolates tests
        that affect the contents of the CWD to prevent them from
        interfering with each other.

        :param temp_dir: Create the temporary directory under this
            directory. If given, the created directory is not removed
            when exiting.

        .. versionchanged:: 8.0
            Added the ``temp_dir`` parameter.
        """
>       cwd = os.getcwd()
E       FileNotFoundError: [Errno 2] No such file or directory

/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/click/testing.py:466: FileNotFoundError

Not sure why it only affected the "Calculate test coverage" one.

It's this test here:
https://github.com/simonw/datasette/blob/1ad92a1d87d79084ebe524ed186c900ff042328c/tests/test_cli.py#L181-L189
Added in:

#2033

Got the same failure again for a recent commit: https://github.com/simonw/datasette/actions/runs/4368239376/jobs/7640567282


Error when uploading to GitHub release: "Error from GitHub: Validation Failed"

Hey folks, I'm running prebuild on CI after not having used it in a while and I'm getting this error when prebuild tries to upload assets to a GitHub release:
prebuild ERR! build Error: Error from GitHub: Validation Failed
prebuild ERR! build     at responseHandler (C:\projects\node-pty-prebuilt\node_modules\ghutils\ghutils.js:25:23)
prebuild ERR! build     at BufferList._callback (C:\projects\node-pty-prebuilt\node_modules\jsonist\jsonist.js:61:7)
prebuild ERR! build     at BufferList.end (C:\projects\node-pty-prebuilt\node_modules\bl\bl.js:105:10)
prebuild ERR! build     at DuplexWrapper.onend (C:\projects\node-pty-prebuilt\node_modules\hyperquest\node_modules\readable-stream\lib\_stream_readable.js:537:10)
prebuild ERR! build     at Object.onceWrapper (events.js:273:13)
prebuild ERR! build     at DuplexWrapper.emit (events.js:187:15)
prebuild ERR! build     at C:\projects\node-pty-prebuilt\node_modules\hyperquest\node_modules\readable-stream\lib\_stream_readable.js:934:16
prebuild ERR! build     at process._tickCallback (internal/process/next_tick.js:61:11)
Command exited with code 2

I've recreated the GitHub token I'm using in CI to upload the releases so it shouldn't be an authentication issue.  Any idea what might be causing this?

Also, I'm using Node 10 on the CI machine so that I can make prebuilds for that version, dunno if that is contributing to the issue.  Strangely the issue went away on its own on TravisCI after a rebuild but it fails consistently on Appveyor.

@vweevers Any thoughts?

@daviwil We could start by tweaking ghutils to give a better error message. We might get a more meaningful error message. I'll dig a bit.

We could start by tweaking ghutils to give a better error message. We might get a more meaningful error message. I'll dig a bit.

@ralphtheninja isn't "Validation failed" an error returned by GitHub? Not much we can do.

It is, but there might also be a .errors array with extra information that we can tack on. If present. https://developer.github.com/v3/#client-errors

https://github.com/rvagg/ghutils/pull/16

@daviwil You could try AppVeyor again to see if we get any better messages.

Thanks @ralphtheninja!  Definitely clearer now:
Error from GitHub: Validation Failed ([{"resource":"ReleaseAsset","code":"already_exists","field":"name"}])

Looks like there are some partially uploaded assets from a previous build:

Just cleared those out and running another build to see if it resolves the issue.

Failed again for the same reason...  I'll try debugging it locally on my machine to see what's going on.

@daviwil Did you manage to get any more information by debugging locally?

Might be a bug in https://github.com/ralphtheninja/ghreleases , but strange that we haven't noticed it before. I'll try to reproduce with leveldown.

@daviwil I went through your setup for node-pty-prebuilt and it's quite different from how we do it on leveldown. We use prebuild-ci (by @juliangruber) and multiple node versions in e.g. .travis.yml/appveyor.yml, i.e. we always build using the corresponding node version, which means we never use node 10 to build binaries for node 9 etc.
I can explain in greater detail if you like.

I'm wondering if there are multiple builds running on e.g. AppVeyor that try to build and upload the same binaries.

Thanks for looking into this @ralphtheninja!  I hadn't come across prebuild-ci before, looks like that might solve this problem.  I'll give it a shot!

I can confirm that I get dupes uploaded in a single call to my upload npm script and that no other builds produce the same files. "prebuild-upload": "prebuild --all --strip --verbose"

So all it takes to reproduce is a npm run prebuild-upload in node-serialport? Which version of node? I'd like to reproduce this.

This problem seems to be in prebuild rather than any dependency.

Only the runtime and the abi fields are used to build the filename and prebuild package name, which results in one file for drivelist-v6.4.0-electron-v57-win32-x64.tar.gz, while the upload function tries to upload once per target which means that electron-v57 will always be uploaded twice.

Awesome find! I'll start by patching ghreleases as well.

https://github.com/ralphtheninja/ghreleases/commit/81470ee2fe1960d4d80afea587873f4c64b6adb4

@ralphtheninja I have submitted a PR to fix this issue on the prebuild side also: #226
Thanks! It's up to you if you want to keep a GIGO policy or implement a filter for this, because I think it is quite an invalid input to ghreleases to call it with duplicate entries.


Print usage when fails to parse subcommand.

Print usage of subcommand when subcommand fails to parse.
For example for
forge.exe new projext
the output is
Bad or missing parameters.
   Available parameters:

        project: Creates new project
        file: Creates new file
        solution: Creates new solution

        --help [-h|/h|/help|/?]: display this list of options.

instead of
Unhandled error:
The input list was empty.

Great. Thanks a lot!


discord: 0.0.56 -> 0.0.57

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for discord is: All-in-one cross-platform voice and text chat for gamers
meta.homepage for discord is: https://discordapp.com/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.0.57 with grep in /nix/store/y3wm79ihkpkcdba6a86y75pym76d2r5q-discord-0.0.57
found 0.0.57 in filename of file in /nix/store/y3wm79ihkpkcdba6a86y75pym76d2r5q-discord-0.0.57

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

discord

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/y3wm79ihkpkcdba6a86y75pym76d2r5q-discord-0.0.57 \
  --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 discord https://github.com/r-ryantm/nixpkgs/archive/0c2ca99648b98472eb3f1da82ba7a47ce05de9de.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0c2ca99648b98472eb3f1da82ba7a47ce05de9de#discord

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/y3wm79ihkpkcdba6a86y75pym76d2r5q-discord-0.0.57
ls -la /nix/store/y3wm79ihkpkcdba6a86y75pym76d2r5q-discord-0.0.57/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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

discord

Maintainer pings
cc @Scrumplex @Artturin @Infinidoge @jopejoe1 for testing.

Done in #322978.


AWS DEFAULT REGION REMOVAL

@wombat can you please resolve conflicts?

@arun-gupta I think we don´t need this PR anymore as you already deleted the references


Make minor typo fixes in README (#2353) [skip ci]

(intentionally stripping away the checklist as this is a trivial PR)

thank you


FileNotFoundError when writing plot fits

Hi John,
I've got a minor bug to report here:
Versions
I am using PopPUNK v2.3.0 with pp-sketchlib 1.6.2, as provided by a conda environment built with
conda create -n poppunk230 -c defaults -c conda-forge -c bioconda poppunk==2.3.0 pp-sketchlib==1.6.2

Command used and output returned
I ruan the command:
poppunk --create-db --r-files /lustre/scratch118/infgen/team216/fl4/poppunk_7kVc/phylocoregenome_clade_genome_fasta_list.tab \
--output /lustre/scratch118/infgen/team216/fl4/poppunk_7kVc/6kVcPGc --threads 8 \
--min-k 15 --max-k 35 --plot-fit 5  --qc-filter prune --length-range 3500000 4500000 --max-a-dist 1

I get this output (combined stdout and stderr streams):
Sketching 7628 genomes using 8 thread(s)
Writing sketches to file
Calculating random match chances using Monte Carlo
Calculating distances using 8 thread(s)
PopPUNK (POPulation Partitioning Using Nucleotide Kmers)
        (with backend: sketchlib v1.6.2
         sketchlib: /lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/pp_sketchlib.cpython-38-x86_64-linux-gnu.so)

Graph-tools OpenMP parallelisation enabled: with 8 threads
Mode: Building new database from input sequences
0092_L008_R1.contigs_velvet failed QC
...
19787_7#60.contigs_spades failed QC
Traceback (most recent call last):
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/bin/poppunk", line 10, in <module>
    sys.exit(main())
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/PopPUNK/__main__.py", line 278, in main
    refList, queryList, distMat = queryDatabase(rNames = rNames,
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/PopPUNK/sketchlib.py", line 521, in queryDatabase
    plot_fit(klist,
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/PopPUNK/plot.py", line 127, in plot_fit
    plt.savefig(out_prefix + ".pdf", bbox_inches='tight')
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/matplotlib/pyplot.py", line 859, in savefig
    res = fig.savefig(*args, **kwargs)
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/matplotlib/figure.py", line 2311, in savefig
    self.canvas.print_figure(fname, **kwargs)
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/matplotlib/backend_bases.py", line 2210, in print_figure
    result = print_method(
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/matplotlib/backend_bases.py", line 1639, in wrapper
    return func(*args, **kwargs)
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/matplotlib/backends/backend_pdf.py", line 2586, in print_pdf
    file = PdfFile(filename, metadata=metadata)
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/matplotlib/backends/backend_pdf.py", line 562, in __init__
    fh, opened = cbook.to_filehandle(filename, "wb", return_opened=True)
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/matplotlib/cbook/__init__.py", line 408, in to_filehandle
    fh = open(fname, flag, encoding=encoding)
FileNotFoundError: [Errno 2] No such file or directory: '/lustre/scratch118/infgen/team216/fl4/poppunk_7kVc/6kVcPGc//lustre/scratch118/infgen/team216/fl4/poppunk_7kVc/6kVcPGc_fit_example_1.pdf'

in the specified output folder /lustre/scratch118/infgen/team216/fl4/poppunk_7kVc/6kVcPGc I can find the database, but not the plots:
ls -ltr 6kVcPGc/
total 790700
-rw-r--r-- 1 me mygroup      6184 Mar  5 18:50 6kVcPGc_qcreport.txt
-rw-r--r-- 1 me mygroup 809662250 Mar  5 18:53 6kVcPGc.h5

Describe the bug
it seems that the plot function makes the path of ouput folder by pasting its specified dirname and full path name.
It used to work OK if I executed the command in the folder above and gave a single-level relative path as value for --output.
The rest of the --create-db command runs fine though.
Cheers,
Florent

I'll add that subsequently running the --fit-model command:
--fit-model bgmm --ref-db /lustre/scratch118/infgen/team216/fl4/poppunk_7kVc/6kVcPGc --output /lustre/scratch118/infgen/team216/fl4/poppunk_7kVc/6kVcPGc --threads 8
  --qc-filter prune --length-range 3500000 4500000 --max-a-dist 1 --K 3

it leads to the same issue as it returns:
Graph-tools OpenMP parallelisation enabled: with 8 threads
PopPUNK: visualise
Traceback (most recent call last):
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/bin/poppunk_visualise", line 10, in <module>
    sys.exit(main())
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/PopPUNK/visualise.py", line 340, in main
    generate_visualisations(args.query_db,
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/PopPUNK/visualise.py", line 193, in generate_visualisations
    rlist, qlist, self, complete_distMat = readPickle(distances)
  File "/lustre/scratch118/infgen/team216/fl4/miniconda3/envs/poppunk230/lib/python3.8/site-packages/PopPUNK/utils.py", line 129, in readPickle
    with open(pklName + ".pkl", 'rb') as pickle_file:
FileNotFoundError: [Errno 2] No such file or directory: '6kVcPGc//lustre/scratch118/infgen/team216/fl4/poppunk_7kVc/6kVcPGc.dists.pkl'

Thanks for pointing this out @flass - I think this should be fixed on the mst_dev branch we are currently working on (see c1f4c5a23fecc5d099529c5948642dfa8569ed4c), but this hasn't been merged in yet because I'm being slow, apologies!

HI Nick,
thanks for the tip.
I have edited the code in the __main__.py and assign.py modules from my conda environment by hand to replicate the fix in c1f4c5a; as I was not sure I removed the corresponding *pyc files in __pycache__.
I also had to do a similar fix in visualise.py for when calling poppunk_visualise:
187:            distances = os.path.basename(ref_db) + "/" + ref_db + ".dists"
189:            distances = os.path.basename(query_db) + "/" + query_db + ".dists"

it seems to have done the trick.
Cheers,
Florent


Bug in subscribe with MQTTQOS1

MQTTQOS1 is defined as (1 << 1) (that is 2)
now look at this function:
boolean PubSubClient::subscribe(const char* topic, uint8_t qos) {
if (qos < 0 || qos > 1) {  // qos (2) > 1
return false;  // it will always return false so subscribe will fail
}
It is too obvious... or am I missing somethigs?

Hmm... interesting. Think you're on to something there.

It's not a bug. Only should not be defined as MQTTQOS1.
MQTTQOS1 is used to make the first 4 bits of subscribe/unsubscribe packets (return write(MQTTSUBSCRIBE|MQTTQOS1,buffer,length-5);) and MUST be 1<<1 regardless of qos (http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc398718064).
So DO NOT use MQTTQOSx defines to choose qos, just use 0 for qos 0 and 1 for qos 1


Improving test coverage

It will be nice to have more refactoring and test coverage. It would be great if we can get some help on more testing of components.

I will close this card. We are doing it a lot anyway.


Use gliderlabs/herokuish docker image

Currently the project is using a self maintained herokuish container based on a self maintained heroku cedar14 image.
gliderlabs/herokuish published a container image based on the official heroku cedar14 image.
Maybe use them to reduce the maintenance, if this is possible

"E: Unable to locate package docker-engine"
Can you retrigger this build?

Need to get this on CircleCI ... I can't really use wercker.
Anyway, the plan for buildstep is to be replaced by buildkit which is more general. The usecase of buildstep now can pretty much be replaced entirely by just using herokuish. I'm pretty sure that's why this hasn't been done here yet. Dokku just uses Herokuish. Am I right @josegonzalez ?

Yeah, we just use herokuish. Seems to work, was mostly a drop-in replacement for buildstep.

have you a link for buildkit ?


Fix: Rename

This pull request

[x] renames php-cs-fixer.asc to php-cs-fixer.phar.asc

Follows #2.

Thank you, @keradus!


Calling DetectIntent with v2beta1 gives error 401 with same Authentication process used for v2

Hi,
I was using DetectIntent API in v2 and it was working fine with Access token signed using JWT
one I migrated to v2beta1 in order to use Knowledge I receive for the exact same POST request

"code": 401,
"message": "Request had invalid authentication credentials. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.",
"status":

"UNAUTHENTICATED"

here is the whole POST request :
const options = {
    method: 'POST',
    host: 'dialogflow.googleapis.com',
    path: '/v2beta1/projects/'+ google_project_id +'/agent/environments/draft/users/6542423/sessions/124567:detectIntent',
    headers: {
      'Authorization': 'Bearer ' + google_access_token,
      'Content-Type': 'application/json',
      'Accept': 'application/json'

the only change is in the path... Any idea if this is bug or a mistake from you ?

I'm not sure if this is the right place for your question, this repo is for the gRPC client and it looks like you're manually building REST requests? You might have better luck over at https://github.com/googleapis/google-auth-library-nodejs

I'm not sure if this is the right place for your question, this repo is for the gRPC client and it looks like you're manually building REST requests? You might have better luck over at https://github.com/googleapis/google-auth-library-nodejs

I'll check it . Thank you so much for your reply


run state on remote not syncing to local?

Perhaps I'm misunderstanding something in how I'm using guild, but I might've encountered a bug. I was running a batch run on an ec2 remote consisting of 4 trials. At some point, I used guild pull while it was running on the remote so that I could look at the tensorboard. Later on, realizing that I had screwed something up, I used guild stop with the --remote flag in order to stop those runs on the remote. Checking the runs on the remote via guild runs --remote [myec2instance], it appears that the batch run and the first of 4 runs had indeed been terminated as expected (the remaining runs are still listed as staging).
I then used guild pull to get my local runs up to date. It looks like it went about downloading my model artifacts. However, guild runs still displays the local representation of the runs as running. I tried to also use guild sync but this did nothing to fix that.
Perhaps this is already a known bug or I am doing something wrong? Let me know if I can provide any additional information.

Thanks for the report. This sounds like a bug. The remote runs, when terminated, should reflect as terminated when copied to other systems. I'll see if I can recreate and update here.

Thanks for the report. This sounds like a bug. The remote runs, when terminated, should reflect as terminated when copied to other systems. I'll see if I can recreate and update here.

Also, I'm pretty confident this wouldn't be related at all, but just for full transparency, I did mess around a tiny bit with the terraform code inside of ec2.py (adding a subnet id, changing vpc id, adding an instance name).

Also, I'm pretty confident this wouldn't be related at all, but just for full transparency, I did mess around a tiny bit with the terraform code inside of ec2.py (adding a subnet id, changing vpc id, adding an instance name).

Okay, I think I've recreated this. I suspect the problem is that the <run dir>/.guild/LOCK file is being transferred from the remote system to the local system. This will signal to the local system that the process is running if there's a process ID matching the value in LOCK.
You can confirm this by running the following command on each system:
$ guild cat -p .guild/LOCK <run ID> [-r <remote>]

The output should be the same value. The local system should have a running process (not related to Guild - this is a coincidence) with that ID. The remote system should not.
The bug, I believe, is that the remote sync should not copy the LOCK file across systems.

Okay, I think I've recreated this. I suspect the problem is that the <run dir>/.guild/LOCK file is being transferred from the remote system to the local system. This will signal to the local system that the process is running if there's a process ID matching the value in LOCK.
You can confirm this by running the following command on each system:
$ guild cat -p .guild/LOCK <run ID> [-r <remote>]

The output should be the same value. The local system should have a running process (not related to Guild - this is a coincidence) with that ID. The remote system should not.
The bug, I believe, is that the remote sync should not copy the LOCK file across systems.

I think this theory is wrong ^
Your local LOCK should have a value 0. This is the bug. 0 is a valid PID and is likely alive on your local system.

I think this theory is wrong ^
Your local LOCK should have a value 0. This is the bug. 0 is a valid PID and is likely alive on your local system.

Haha ya was just about to say that: I'm getting 0 locally and something else non-locally (I checked manually as well via shell).

Haha ya was just about to say that: I'm getting 0 locally and something else non-locally (I checked manually as well via shell).

Okay great - this will be fixed in the next pre-release of Guild (will be 0.7.3.dev2). In the meantime, you can manually delete LOCK files that have this 0 value to fix the running status.
$ LOCK=`guild ls -fp .guild/LOCK <local run ID to fix>`; test ! -z $LOCK && test `cat $LOCK` == "0" && rm $LOCK

The rather absurd command just makes sure you're not deleting a LOCK for an actual running process. Feel free to tamper as needed.

Okay great - this will be fixed in the next pre-release of Guild (will be 0.7.3.dev2). In the meantime, you can manually delete LOCK files that have this 0 value to fix the running status.
$ LOCK=`guild ls -fp .guild/LOCK <local run ID to fix>`; test ! -z $LOCK && test `cat $LOCK` == "0" && rm $LOCK

The rather absurd command just makes sure you're not deleting a LOCK for an actual running process. Feel free to tamper as needed.

Thanks, @gar1t !
When I was looking around at different experiment tracking tools, a common theme was how responsive you are. Glad I got to experience it firsthand!

Thanks, @gar1t !
When I was looking around at different experiment tracking tools, a common theme was how responsive you are. Glad I got to experience it firsthand!

Some extra feedback in case it's useful:
So I tried removing the LOCK for one of the trial runs. This changed the run status to "error". Taking a brief look at the source code, it looks like this happens if exit_status isn't available.
Given lack of exit status, it seems then that the run status is fully reliant on the LOCK file. As a workaround, instead of deleting the lock file I'm just editing the pid to something greater than my system's max.
Also, I'm not sure if this is another unrelated bug (or just a matter of perception - if it's a bug we can make new issue), but I found it surprising on the remote instance that the later trials stayed as staged even though I used stop to terminate the batch run and the 1st trial that was in progress. If I were to later ssh into remote and setup a queue or something, then would those staged runs get picked up even though I had terminated the batch run?

Some extra feedback in case it's useful:
So I tried removing the LOCK for one of the trial runs. This changed the run status to "error". Taking a brief look at the source code, it looks like this happens if exit_status isn't available.
Given lack of exit status, it seems then that the run status is fully reliant on the LOCK file. As a workaround, instead of deleting the lock file I'm just editing the pid to something greater than my system's max.
Also, I'm not sure if this is another unrelated bug (or just a matter of perception - if it's a bug we can make new issue), but I found it surprising on the remote instance that the later trials stayed as staged even though I used stop to terminate the batch run and the 1st trial that was in progress. If I were to later ssh into remote and setup a queue or something, then would those staged runs get picked up even though I had terminated the batch run?

Okay, this turned into a bit more work. There are two issues in play here. I believe the original issue is that the terminated runs were not stopped cleanly. If they have a LOCK on the remote server but the process is not alive, it means the run processes died either without a Guild parent process (likely) or another abnormal termination (e.g. the system crashed - not at all likely from what you describe). In any case, the correct status for that type of run is error - not terminated. Guild was incorrectly listing those as terminated.
This doesn't mean that there's a problem with the run - it may contain useful checkpoints, saved models, etc. It just means that it didn't terminate with control.
The sync process should accurately reflect the server status.
I just commit a number of changes that should address this.

Guild uses error status for runs that were not terminated with control (i.e. their exit was detected by a Guild process that writes the exit code and deleted the LOCK file).
Guild now terminates the active trial when a batch is stopped.
Guild no longer uses a fake LOCK (e.g. with value 0 or some other invented process ID) to trick itself into thinking a local run is terminated. LOCK files are not copied from remote runs or otherwise simulated.

If you want to manually cleanup your runs to reflect the correct terminated status, delete the LOCK files and write -15 to .guild/attrs/exit_status. This is what a correctly terminated should will look like.

Okay, this turned into a bit more work. There are two issues in play here. I believe the original issue is that the terminated runs were not stopped cleanly. If they have a LOCK on the remote server but the process is not alive, it means the run processes died either without a Guild parent process (likely) or another abnormal termination (e.g. the system crashed - not at all likely from what you describe). In any case, the correct status for that type of run is error - not terminated. Guild was incorrectly listing those as terminated.
This doesn't mean that there's a problem with the run - it may contain useful checkpoints, saved models, etc. It just means that it didn't terminate with control.
The sync process should accurately reflect the server status.
I just commit a number of changes that should address this.

Guild uses error status for runs that were not terminated with control (i.e. their exit was detected by a Guild process that writes the exit code and deleted the LOCK file).
Guild now terminates the active trial when a batch is stopped.
Guild no longer uses a fake LOCK (e.g. with value 0 or some other invented process ID) to trick itself into thinking a local run is terminated. LOCK files are not copied from remote runs or otherwise simulated.

If you want to manually cleanup your runs to reflect the correct terminated status, delete the LOCK files and write -15 to .guild/attrs/exit_status. This is what a correctly terminated should will look like.

Hmm, if that's the case I'm not sure why the terminated runs were not stopped cleanly. Did you suspect there's a bug with how guild stop --remote <remote> works?
IIRC, I believe I stopped the run that was inside the batch (didn't try directly stopping the batch run itself, though this terminated as well as expected).

In that case Guild may have stopped the batch first, leaving the trial runs without a Guild parent to handle their exit. This would leave the trials in an error state. But Guild was masking this by showing terminated (an example of how defensive programming undermines correct behavior). With the new behavior it doesn't matter what order the runs are stopped - they should correctly show terminated and this status should correctly be reflected when the run is copied to another system.


Return batch with offset throws validation error

ACH Version: v1.15.0
What were you trying to do?
Add an offset to a return batch. This was done by calling
batch.WithOffset(&ach.Offset{
	RoutingNumber: "DUMMY",
	AccountNumber: "DUMMY",
	AccountType: "checking",
	Description: "S",
})
batch.Create()

What did you expect to see?
Successful creation
What did you see?
Error:
batch #1 (WEB) Addenda99 is a mandatory field and has a default value

While debugging and stepping through the code, we see that the error is being thrown by the OFFSET entry that gets added during the build step.
The offset entry is created internally as a last step in the build method that is called by batch.Create() and is given the same category as the first entry in the batch, which in this case is CategoryReturn. However, since it's an OFFSET record, it doesn't get an Addenda99 record.
When Validate() gets called on the batch, it loops through all the entries (including the OFFSET record) and errors out on the addendaFieldInclusionReturn when validating the OFFSET record.
How can we reproduce the problem?
Very simple example with dummy data.
package main

import (
	"fmt"
	"time"

	"github.com/moov-io/ach"
)

func main() {
	// Batch Header
	batchHeader := ach.NewBatchHeader()
	batchHeader.ServiceClassCode = 200
	batchHeader.CompanyName = "COMPANY"
	batchHeader.CompanyIdentification = "IDENTIFICATION"
	batchHeader.StandardEntryClassCode = ach.WEB
	batchHeader.CompanyEntryDescription = "DESCRIPTION"
	batchHeader.CompanyDescriptiveDate = time.Now().Format("060102")
	batchHeader.EffectiveEntryDate = time.Now().Add(time.Hour * 24).Format("060102")
	batchHeader.ODFIIdentification = "1"

	// Addenda99
	addenda99 := ach.NewAddenda99()
	addenda99.ReturnCode = "R07"
	addenda99.OriginalTrace = "99912340000015"
	addenda99.AddendaInformation = "Authorization Revoked"
	addenda99.OriginalDFI = "9101298"

	// Entry
	entry := ach.NewEntryDetail()
	entry.TransactionCode = 22
	entry.RDFIIdentification = "1"
	entry.CheckDigit = "3"
	entry.DFIAccountNumber = "DUMMY"
	entry.Amount = 1000
	entry.IndividualName = "DOE, JOHN"
	entry.Category = ach.CategoryReturn
	entry.Addenda99 = addenda99
	entry.AddendaRecordIndicator = 1

	// Batches
	batch1 := ach.NewBatchWEB(batchHeader)
	batch1.AddEntry(entry)

	// Prints nil
	fmt.Printf("batch1 error: %v\n", batch1.Create())

	batch2 := ach.NewBatchWEB(batchHeader)
	batch2.AddEntry(entry)
	batch2.WithOffset(&ach.Offset{
		RoutingNumber: "100000007",
		AccountNumber: "DUMMY",
		AccountType:   "checking",
		Description:   "S",
	})

	// Prints error
	fmt.Printf("batch2 error: %v\n", batch2.Create())
}

Go playground link: https://go.dev/play/p/EwWCywnMq_1

Thanks for the report! I never thought about adding OFFSET records to returns... Do you expect the same 99 addenda on the OFFSET EntryDetail? I would think it should be a Forward, but your use-case is new to me.

I've made a change to copy over those addenda records which should work for you, but I want to chat with some ACH folks to get their thoughts on this.
I assume you can manually add the Addenda99 record to work around this?

Hello! Thank you very much for looking into this.
Well, actually we cannot add an Addenda99 record to the OFFSET record. All of the below that I described in the issue happens inside of the batch.Create() method after calling the WithOffset method on the batch (see example):

The offset entry is created internally as a last step in the build method that is called by batch.Create() and is given the same category as the first entry in the batch, which in this case is CategoryReturn. However, since it's an OFFSET record, it doesn't get an Addenda99 record.
When Validate() gets called on the batch, it loops through all the entries (including the OFFSET record) and errors out on the addendaFieldInclusionReturn when validating the OFFSET record.

Also, since there is a single OFFSET record per batch, and a single batch can contain multiple transactions, each with its own Addenda99 record, it doesn't make much sense to add an Addenda99 record.
Thinking through how to solve this, I would probably suggest adding a validationOption option to BypassAddendaFieldInclusion and setting that to true when generating the Offset entry.

Thinking through how to solve this, I would probably suggest adding a validationOption option to BypassAddendaFieldInclusion and setting that to true when generating the Offset entry.

Yea. I'll take a look at doing something like that. We can probably fix the math internally as well.

@jcgarciaram I've updated https://github.com/moov-io/ach/pull/1011 to skip that addendaFieldInclusion check for OFFSET records. That should resolve the issue you're seeing.

Can you try go get github.com/moov-io/ach@master and verify your issue is resolved? I'll release a bugfix if it works for you.

@jcgarciaram have you had a chance to verify if this fix works?

I'm releasing v1.16.0 with this change included. Please let us know if you continue to run into issues. Thanks!

Apologies for the delay in answering. YES! This change accomplishes what we needed. Thank you very much!


Unwrapping on an Err() causes a program to panic

Description
If a file descriptor provided to get_filedescriptor() is out of the allowed range: 0 to 1024(excluded), the method returns Err(), and unwrapping on the returned Err() value later on causes a program to panic.
Cases:
Any syscall, like fcntl or ioctl, that involves file descriptors needs to access a file descriptor table using get_filedescriptor(). The pattern is always the same and involves the following line:
let checkedfd = self.get_filedescriptor(fd).unwrap();
Why this behavior?
By definition, an unwrap method for the Result type should panic when provided with an Err() result.
How is this tested?
Unit tests calling fcntl syscall with out-of-range file descriptors, e.g. negative values or values greater than or equal to 1024, failed with called Result::unwrap() on an Err.
References
https://web.mit.edu/rust-lang_v1.25/arch/amd64_ubuntu1404/share/doc/rust/html/book/first-edition/error-handling.html#unwrapping-explained

Should a match condition be written to handle the Err() value before unwrapping it?


SMBACK-6098 copy old values for defaultTeamRole and entryPoint

Default role and entry point are resetting by sdclient.edit_team method
When customer calls sdcclient.remove_memberships (that calls edit_team)
two team attributes are resetting:default team, entry point

Thank you for the PR @IvanNik, all works fine! I just merged it and will be available in the next release.


Makefile: install-ocb target -> add ~/bin directory creation before curl output…

… targets it
Description
Problem: "install-ocb" makefile target assumes existence of ~/bin directory - fails if doesn't exist
Fix: add non-destructive creation of ~/bin directory before curl tries to output there
Checklist

[NA] Created tests which fail without the change (if possible)
[NA] Extended the README / documentation, if necessary

Closing due to incorrect workflow


More examples

1: this doesn't align with the Github pages build and as such will not be displayed on the site,
2: there is an issue with the first example where it just explodes and
3: it doesn't integrate with the docs which are at unle.js.org, which would benefit with these examples

Sure I'll do that.
I'll also implement mobile zooming.
On Tue, 20 June 2023, 17:22 Lachlan Jowett, @.***>
wrote:

@.**** requested changes on this pull request.
Possibly make const K a value in a constants.js file to avoid issues when
changing the value in the future?
—
Reply to this email directly, view it on GitHub
https://github.com/lochyj/UNLE/pull/17#pullrequestreview-1487482334, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWIYOVFMGARSHZ4UQPDW7NLXMFFU7ANCNFSM6AAAAAAZC4RF5A
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add option of append-to-body?

Same problem with #3 , could you provide an option of "append to body" like ui-bootstrap? That would be very helpful.

Duplicate #107


Fix Code Re-Struture Project #26

Please Review and Merge @ZapeeoSheikh, @arjun26122002
#26

@madhur87  Resolve Conflicts

@ZapeeoSheikh Conflicts Resolve

Can you Merge please @ZapeeoSheikh

Is there any particular reason, why you not merged the MR. @ZapeeoSheikh


do full security audit

What type of audits are required?

i don't know too much about the topic, but some ideas -
-block unused ports
-block non-localhost ip addr access?
-audit our adapter, meter, relay Dockerfiles and build procedures
-what kind of schedule to rebuild docker images at client installations? monthly? automate eventually
-passwords - we tend to re-use pws across a client install, and for root access(?) - better way?
-postgres - make a user, give permissions to access certain tables/views
...

I am not quite sure if this issue should be labelled as enhancement or bug, as it is related to both.


how to add an external javascript to load all the time in the application

is it possible?

This issue looks like a question that would be best asked on StackOverflow.
StackOverflow is amazing for Q&A: it has a reputation system, voting, the ability to mark a question as answered. Because of the reputation system it is likely the community will see and answer your question there. This also helps us use the GitHub bug tracker for bugs only.
Will close this as this is really a question that should be asked on StackOverflow.


Issue with image.fillPath with some shapes

I noticed I was able to call fillPath on rectangles that are taller than longer, but not rectangles that are longer than taller. It may just be a misconception on my part.
Here's a runnable example:
import pixie

const
  pointsA = [
    vec2(0, 0),
    vec2(0, 1120),
    vec2(32, 1120),
    vec2(32, 0)
  ]
  pointsB = [
    vec2(0, 0),
    vec2(1120, 0),
    vec2(1120, 32),
    vec2(0, 32)
  ]

proc renderPoints(points: openArray[Vec2]): Image =
  let
    image = newImage(1920, 1080)
    ctx = newContext(image)

  ctx.fillStyle = rgba(255, 0, 0, 255)
  var path: Path
  path.moveTo(points[0])
  for v in points:
    path.lineTo(v)

  ctx.image.fillPath(path, ctx.fillStyle)
  return image

when isMainModule:
  # Works
  let imageA = renderPoints(pointsA)
  imageA.writeFile("shapeA.png")

  # Does not work
  let imageB = renderPoints(pointsB)
  imageB.writeFile("shapeB.png")

You'll notice shapeA.png will have a vertical red rectangle, but shapeB.png appears to be empty.

I see why this is happening. If you walk the path in your head (or on paper), you'll see you only have 1 vertical line in the 2nd case, but have 2 vertical lines in the first (working) case.  If you close the path (or lineTo the first point at the end), they'll both produce closed paths and render as expected.
What's interesting is JS canvas renders the 2nd path, even though it is not closed. It appears JS is auto-closing paths or something. I'll need to look into what JS canvas/context is doing to match that behavior.

Thanks for the insight - for now I'll return back to points[0] as a work around after I've looped over the array

Found some docs for SVG that clear this up for me. Quite interesting:

The fill operation fills open subpaths by performing the fill operation as if an additional "closepath" command were added to the path to connect the last point of the subpath with the first point of the subpath. Thus, fill operations apply to both open subpaths within ‘path’ elements (i.e., subpaths without a closepath command) and ‘polyline’ elements.

https://svgwg.org/svg2-draft/painting.html
I will adopt this behavior.


Add Test  to Verify git plugin

Signed-off-by: Pranav Suryawanshi<EMAIL_ADDRESS>

https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/510/console
Result : Passed

@pranavsuryaw could you try creating a keyword with arguments to contain these lines:
    ${status} =    Verify Libraries In Base Image    tensorflow    ${additional_libs}
    Append To List    ${status_list}    ${status}
    Run Keyword If    '${status}' == 'FAIL'    Fail    Shown and installed libraries for tensorflow image do not match```

   By leveraging the arguments you coul re-use the kw in the different TCs

This tests is missing PolarionID and TierX tag

https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/547/console
Result : Passed

Merging this, @pranavsuryaw make sure to update the Polarion status to Automated


build error in ubuntu 16.04

commit c1c1e426b32794d5e84134ee81bf895ff0228fe5
system : ubuntu 16.04
there is an errror on lstmrecoginzer.cpp and other files.
the infamous 'isnan()' error
one way to fix NOW it is replacing  isnan() with std::isnan()
go into lstm dir and execute

sed -i 's/isnan/std::isnan/g' grep "isnan" -rl

have fun

You can sent pull requests...

PR #491 should fix this build problem (which also breaks the continuous integration tests).

@blairq, could you please repeat your test with the latest version from git?

This issue should be closed.


MAINT: transition from __array__ to from_dlpack

Continues gh-21890. Blocked on https://github.com/data-apis/array-api-strict/pull/108.
Around our codebase that has been converted to support standard arrays, we still widely rely on __array__, but it is not in the standard. Therefore, array-api-strict would like to remove it. @mdhaber wrote this xp_asarray which delegates to from_dlpack under the hood for library-interchange.
Here I get all tests passing for constants and cluster.
An important consideration from @mdhaber in the PR this continues:

Outputs will typically be read-only after this step. This seems like a big disadvantage compared to xp.asarray using the array protocol. I don't think we want to return read-only results from special functions. Is there a general solution that is efficient (e.g. not just copy the output before returning it) or do we have to be very careful about using from_dlpack only when necessary?

Perhaps we could attempt to use __array__ and only fall-back to from_dlpack (and thus copying read-only outputs) if that fails. But I think that would still require an array-level mutability check such as that described in https://github.com/data-apis/array-api/issues/845.

scores on the doors:
6923 failed, 46725 passed, 7957 skipped, 16195 deselected, 886 xfailed, 14 xpassed in 1635.07s (0:27:15)

Did you want me to close gh-21890 or is the intent to merge that first?

Around our codebase that has been converted to support standard arrays, we still widely rely on array,

This may just be an issue of semantics, but we don't explicitly rely on array (almost?) anywhere. We use np.asarray, which tries array is a fallback method - but that's not quite the same thing. The actual thing we relied on can be more accurately phrased like this I think: "np.asarray works for the array types we are trying to support".

I agree

Outputs will typically be read-only after this step.

I don't think that is true. Could you explain why you think this is the case? Or is the claim more like "we should aim to treat the returned arrays as read-only"?

In [1]: import array_api_strict as xp

In [2]: import numpy as np

In [3]: x = xp.arange(3)

In [4]: y = np.from_dlpack(x)

In [5]: y[0] += 1
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[5], line 1
----> 1 y[0] += 1

ValueError: assignment destination is read-only

In [6]:

In [6]: x = np.arange(4)

In [7]: y = xp.from_dlpack(x)

In [8]: y[0] += 1
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[8], line 1
----> 1 y[0] += 1

File ~/programming/array-api-extra/.pixi/envs/dev/lib/python3.13/site-packages/array_api_strict/_array_object.py:907, in Array.__setitem__(self, key, value)
    904 if isinstance(key, Array):
    905     # Indexing self._array with array_api_strict arrays can be erroneous
    906     key = key._array
--> 907 self._array.__setitem__(key, asarray(value)._array)

ValueError: assignment destination is read-only

I also commented on the array-api-strict issue about a possible alternative solution (implement the buffer protocol there).

Are we interested in supporting libraries which don't implement the buffer protocol?


Add pdosqlsrv options that make it behave "normal"

Found the root cause & fix (source) related to mssql acting out: it's pdo driver by default stringifies everything; including numerics.
This causes every relation to fail (as for some reason the primary key IS getting returned as an int; causing a type mismatch between the relation; causing laravel-enso to break.
This should fix this issue (instead of mitigating it with casts everywhere)

@jlsjonas Apparently the app does not boot when using PDO class constants. I removed the options for now until we find a workaround.


Referencing source dependencies lead to shadowed dependency error

Issue description
According to https://github.com/Gabriel439/haskell-nix/blob/master/project1/README.md#source-dependencies and https://nixos.org/nixpkgs/manual/#how-to-build-projects-that-depend-on-each-other it's possible to reference a local source dependency via release.nix.
However, sometimes it leads to dependency shadowing (if I'm getting this error right):
$> nix-shell --command zsh
these derivations will be built:
  /nix/store/2ddm8i6my675a7lncm4frkcasprf5gxl-liba-<IP_ADDRESS>.drv
  /nix/store/7wmsgz49759pff7ikky2nksxn8cckr63-ghc-8.4.3-with-packages.drv
building '/nix/store/2ddm8i6my675a7lncm4frkcasprf5gxl-liba-<IP_ADDRESS>.drv'...
setupCompilerEnvironmentPhase
Build with /nix/store/nv4h00ma4lmwj9bnc8dhf9nzyj28d6ds-ghc-8.4.3.
ignoring (possibly broken) abi-depends field for packages
ignoring (possibly broken) abi-depends field for packages
unpacking sources
unpacking source archive /nix/store/nf0yf8xsi7l92ark5q08ddhrkh2syy9b-hi
source root is hi
patching sources
compileBuildDriverPhase
setupCompileFlags: -package-db=/private/tmp/nix-build-liba-<IP_ADDRESS>.drv-0/setup-package.conf.d -j1 -threaded
Loaded package environment from /private/tmp/nix-build-liba-<IP_ADDRESS>.drv-0/hi/.ghc.environment.x86_64-darwin-8.4.3
<command line>: cannot satisfy -package-id liba-<IP_ADDRESS>-inplace:
    liba-<IP_ADDRESS>-inplace is unusable due to missing dependencies:
      aeson-<IP_ADDRESS>-KPiiXcK6dELrtdAOauWhV
    (use -v for more information)
builder for '/nix/store/2ddm8i6my675a7lncm4frkcasprf5gxl-liba-<IP_ADDRESS>.drv' failed with exit code 1
cannot build derivation '/nix/store/7wmsgz49759pff7ikky2nksxn8cckr63-ghc-8.4.3-with-packages.drv': 1 dependencies couldn't be built
error: build of '/nix/store/7wmsgz49759pff7ikky2nksxn8cckr63-ghc-8.4.3-with-packages.drv' failed

Related issue: https://github.com/Gabriel439/haskell-nix/issues/56
Steps to reproduce
I've tried to reproduce it in a minimal repo - https://github.com/dmalikov/repro-shadowed-deps
$> cd libb
$> nix-shell --command zsh

Technical details
$> nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-darwin"`
 - host os: `Darwin 16.7.0, macOS 10.12.6`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.0.2`
 - channels(m): `""`
 - channels(root): `"nixpkgs-18.09pre144939.14a9ca27e69, nixpkgs-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

>> cabal --version
cabal-install version <IP_ADDRESS>
compiled using version <IP_ADDRESS> of the Cabal library

This is because of the .ghc.environment.* files that cabal generates. If you remove those files, or filter them from Nix, I believe this error will go away.

@domenkozar could you please link a related issue that was already fixed in staged-master?

@dmalikov this is irrelevant to that issue, which is about a linker error from GCC.

@ElvishJerricco it helped, thank you.
Is it a bug, or is it supposed to remove them or keep ignored all the time?

Nix doesn't have any opinion about those files, but it is intended and documented behavior that GHC would pay attention to them. They're supposed to make it so you can invoke ghc directly after using cabal and still get all the installed libraries available. Obviously these are installed in your users home directory, and the file itself is not relocatable, so it's an impurity if GHC tries to use these files (which it will, if they exist).

Closing as, if I understand correctly, there's nothing actionable for nixpkgs here.


Channels: Why IReadableChannel.ReadAsync is not an extension method?

@stephentoub
Looks like it can be implemented using just TryRead and WaitToReadAsync:
public static class ChannelExtensions
{
    public static async Task<T> ReadAsync<T>(IReadableChannel<T> channel, CancellationToken cancellationToken = default(CancellationToken))
    {
        T item;
        do if (channnel.TryRead(out item))
        {
            return item;
        }
        while (await channnel.WaitToReadAsync(cancellationToken));

        throw new Exception("Channel closed?");
    }
}

Then why is it a part of IReadableChannel? Is it because of performance or some implicit ordering/fairness guarantee?

There's certainly a performance aspect to it, in that channel implementations can typically implement this more efficiently than you can in an extension method (even just avoiding the overhead of extra interface calls).
But there's also a functional reason.  Your approach to ReadAsync could also be done for WriteAsync, e.g. pseudo-code:
public static async Task WriteAsync<T>(this IWriteableChannel<T> channel, T value)
{
    while (true)
    {
        if (channel.TryWrite(value)) return;
        await channel.WiteToWriteAsync();
    }
}

but certain kinds of channels will not function well if you have such read/write implementations.  Consider a reader and writer trying to coordinate on an UnbufferedChannel; they'll never successfully join with such implementations as they'll never actually be waiting on the channel at the same time.

I thought about separate interfaces for ReadAsync/WriteAsync but it then realized it would complicate the matters.

@stephentoub
Then another question.
Are there any implicit ordering/fairness guarantees in ReadAsync/WriteAsync?
I am asking because I see that all the channels do FIFO not only for data but for waiting activities as well.

Are there any implicit ordering/fairness guarantees in ReadAsync/WriteAsync?

No, it's up to the implementation.

see that all the channels do FIFO not only for data but for waiting activities as well.

That's not strictly true, as Monitor isn't necessarily fair.

Thanks @stephentoub. Now it is more than less clear )


please fix this

Status:
Account 1: Potential Points: 0, Countdown: Calculating...
Account 2: Potential Points: 25, Countdown: Calculating, it might take a minute before starting...
Account 3: Potential Points: 25, Countdown: Calculating, it might take a minute before starting...
Account 4: Potential Points: 25, Countdown: Calculating, it might take a minute before starting...
Account 5: Potential Points: 25, Countdown: Calculating, it might take a minute before starting...
Account 6: Potential Points: 0, Countdown: Calculating...
Account 7: Potential Points: 25, Countdown: Calculating, it might take a minute before starting...
WebSocket error for Account 7: ErrorEvent {
[Symbol(kTarget)]: WebSocket {
_events: [Object: null prototype] {
open: [Function],
message: [Function],
close: [Function],
error: [Function]
},
_eventsCount: 4,
_maxListeners: undefined,
_binaryType: 'nodebuffer',
_closeCode: 1006,
_closeFrameReceived: false,
_closeFrameSent: false,
_closeMessage: ,
_closeTimer: null,
_errorEmitted: true,
_extensions: {},
_paused: false,
_protocol: '',
_readyState: 2,
_receiver: null,
_sender: null,
_socket: null,
_bufferedAmount: 0,
_isServer: false,
_redirects: 0,
_autoPong: true,
_url: 'wss://secure.ws.teneo.pro/websocket?userId=cj2e8ebf-997b-4205-b9f0-6d4e3203314b&version=v0.2',
_req: null,
[Symbol(shapeMode)]: false,
[Symbol(kCapture)]: false
},
[Symbol(kType)]: 'error',
[Symbol(kError)]: Error: write EPROTO F4500000:error:0A000410:SSL routines:ssl3_read_bytes:sslv3 alert handshake failure:c:\ws\deps\openssl\openssl\ssl\record\rec_layer_s3.c:1605:SSL alert number 40
  at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:87:19) {
errno: -4046,
code: 'EPROTO',
syscall: 'write'

},
[Symbol(kMessage)]: 'write EPROTO F4500000:error:0A000410:SSL routines:ssl3_read_bytes:sslv3 alert handshake failure:c:\ws\deps\openssl\openssl\ssl\record\rec_layer_s3.c:1605:SSL alert number 40\n'
}
Account 7 Disconnected
WebSocket restarted for index: 6

if you already confirm the email verification and have problem, please pull the repo again. use the new script.


Fix demos to include avionics.js from CDN

We should use unpkg version of avionics in demos.

Closed by commit d7fa3829e6


When will we be allowed to contribute templates again?

I'm dying to create/submit a .NET Core template, but I'm not sure if it is really wanted at this point.

We currently have no plans for integrating new workspace types in the immediate future, however we are always working on improving the IDE with new features and appreciate all suggestions and feedback like this.
Keep track of updates on our blog (https://c9.io/blog) and Twitter (https://twitter.com/cloud9ide).


Python extension isn't showing up in the extension view search

This is scrolled to the top, the way I found the python extension was to find an extension pack and click the python ext dependency

Thanks for reporting. This is fixed. Was a problem with the Insiders search service.


Built-in controls on mobile platforms

I have seen a lot of people on the forums asking about how to use the built-in control components for mobile platforms (e.g. most recent forum post),
I think it would be nice to add some kind of key emulating component or handle that in controls themselves (iff on mobile with respect to multitouch setting) . In both cases it should be configurable.

Preliminary idea is to detect (with addEvent to Crafty.stage.elem) where mouse/finger clicked/touch the playing area relative to the controlled entity. Build vector from controlled entity to mouse/touchDown position. Determine angle of vector and categorize it into desired directions (e.g. simulate up direction if angle between 135° and 45°). Translate these directions to input for Multiway and Jumper.

It would also make sense to change these controls to listen for direction presses / changes, rather than actual keys. The required DirectionControl component would take care of translating incoming mouse clicks, (multi)touch events and keydowns to appropriate direction presses / changes. This would allow to programmatically trigger a direction also (in case there is a dedicated onscreen D-Pad for movement on mobile platforms). The directions emitted could be part of the re-purposed NewDirection event and arbitrary in value between -90° and 90°. Joystick and gamepad could then control Twoway and Multiway (see crafty-joystick and crafty-gamepad). This would not only serve controls, but also to determine whether to flip a sprite (already possible with existing NewDirection event), or to rotate entities (see MouseFace).

tracked in #1049 now


[ci] add yaml files linter to pre-commit hook

We have a lot of yaml files across the repo, so I thinks it'll useful to lint them.

The diff of this PR looks quite large. I think it'll be better to split this PR into several smaller ones similarly to how we did with other linting PRs.
Opening this PR to get some initial comments from reviewers.

/gha run r-valgrind

I'm personally a fan of prettier

I'd prefer yamllint here in this project, because as far as I understand from https://prettier.io/docs/en/precommit.html, prettier requires npm install. This isn't a JavaScript project and I wouldn't expect the average person trying to contribute here to have npm installed.
I think prettier is pretty heavy for this limited purpose.

@jameslamb

and that fact that (correct me if I'm wrong) yamllint does not do auto-fixes and these changes were made manually.

TBH, I just didn't checked possibility of auto-fixes. I wanted to fix all manually because during that process I checked which yamllint rules will be best for this project.
Just checked this right now: seems you're right and there is no auto-fix functionality.

I'll prepare the first one of smaller PRs later today, so we'll be able to discuss specific decisions there.


Synapse uses datetime2 for timestamp

review "vyreseno" tady https://keboola.slack.com/archives/CFVRE56UA/p1658309652918049


Update service types supported by Aiven Operator

Fixes #1496
Clickhouse (https://github.com/aiven/aiven-operator/pull/179) and Opensearch (https://github.com/aiven/aiven-operator/pull/174) support has been added recently.

Hi @jelmer thanks for this contribution. According to API reference, I believe that Aiven for Redis is also supported. Happy to make that change if you 👍🏽

Thanks for the review, Dewan. I've updated it to include Redis as well.


Don't show numerator breakdown for raw prescribing measures

Fixes #2135.

Closing this -- after discussion with Brian we still want to show the breakdown for semagglutidequantity, even though it's potentially misleading.

Will do the seven-day prescribing separately later.


Pure keypair transactions is never fully signed

Only phantom signer checks if the transaction is fully signed.

It is, but the signal is casted while calling sign which makes sense. awaiting fully signed should only be called when using a phantom signer


Template variable service failed

Just re-built now, and in the browser UI it no longer renders stats. Instead there's  a pop-up error saying:

:warning: Templating [fromdomain]
Error updating options: Support for Elasticsearch versions after their end-of-life (currently versions < 7.10) was removed.

Hi @casperdcl grafana docker img was updated last week thats why now there are some deps conflicts. To make it work again just specify an older grafana version. I have tested the 8.5.2 and its working. (grafana/Dockerfile -> FROM grafana/grafana:8.5.2)

thx; confirmed working with 8.5.4 (latest before 9.0.0) too.
Updated #24 to include both this fix and #29 too :)


Laravel Passport for multiple auth security problem

Laravel Version: 9.19
PHP Version: 8.1.0
Laravel/passport: ^11.2

Description:
Laravel Passport returns true for several different guards with different tables and the same ID.
For example, if an admin has ID 1 in the admins table and a user has ID 1 in the users table, the user can identify himself as an admin during authentication.
If a normal user logs into the system with ID 1 and gets an access token, he can access the admin user account number 1! And in the same way, user number 2 can access the account of admin number 2!!!!!
Is this a bug or do we have to use only one table for all users when using Laravel Passport?

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


Duplicate key when running make setup

One of the step ran by the "make setup" is to populate the territories
table with the content of the "territories.cvs" files. However, if the
database is already populates by some previous failed "make setup" run,
The command fails. To fix that, before start the process of build
container images and start the process, a docker-compose call is done to
remove all the container images and volume used by the services. Thus,
the "make setup" will always rebuild everything fine.
Signed-off-by: José Guilherme Vanz<EMAIL_ADDRESS>

@jvanz sorry I can't test it right now I will try it later. But from what I understood from the docs, make destroy would erase the images, containers and networks too, right? Do you think that in this case, when running make setup it should just clean the existing volumes instead? To avoid having to rebuild everything?

@jvanz sorry I can't test it right now I will try it later. But from what I understood from the docs, make destroy would erase the images, containers and networks too, right? Do you think that in this case, when running make setup it should clean everything but leave the images instead? To avoid having to rebuild everything?

Hum, the goal of the make setup is to build the images. But I understand your point. I`ll add a config variable with the default value to not delete the images.

Tested it. Worked well. Kudos! (it even doesn't delete the image if other containers use it)