Unable to purchase - API purchase experienced an error, trying puppeteer purchase

As far as I know this was working fine up until today?
It doesn't ever actually purcahse the game - any idea why?
Setting timezone: America/New_York
Run on startup: true
Setting timezone: America/New_York
Run on startup: true
[2021-12-16 19:57:45.726 -0500] INFO (44 on a354c2211981): Checking free games for<EMAIL_ADDRESS>[2021-12-16 19:57:54.633 -0500] INFO (44 on a354c2211981): Successfully refreshed login
user<EMAIL_ADDRESS>[2021-12-16 19:57:54.636 -0500] INFO (44 on a354c2211981): searchStrategy is `all`: searching for weekly and promotional games
user<EMAIL_ADDRESS>[2021-12-16 19:57:54.633 -0500] INFO (44 on a354c2211981): Successfully refreshed login
user<EMAIL_ADDRESS>[2021-12-16 19:57:54.636 -0500] INFO (44 on a354c2211981): searchStrategy is `all`: searching for weekly and promotional games
user<EMAIL_ADDRESS>[2021-12-16 19:58:09.749 -0500] INFO (44 on a354c2211981): Available free games
user<EMAIL_ADDRESS>availableGames: [
"Shenmue III",
"Shenmue III - Standard Edition"
]
[2021-12-16 19:58:09.968 -0500] INFO (44 on a354c2211981): Unpurchased free games
user<EMAIL_ADDRESS>purchasableGames: [
"Shenmue III"
]
[2021-12-16 19:58:09.969 -0500] INFO (44 on a354c2211981): Purchasing Shenmue III
[2021-12-16 19:58:11.728 -0500] ERROR (44 on a354c2211981): There was a problem processing your request. Please try again later.

user<EMAIL_ADDRESS>[2021-12-16 19:58:11.731 -0500] WARN (44 on a354c2211981): Cannot read property 'map' of undefined
TypeError: Cannot read property 'map' of undefined
at Purchase.confirmOrder (/usr/app/src/purchase.ts:37:82)
at Purchase.purchase (/usr/app/src/purchase.ts:155:16)
at processTicksAndRejections (internal/process/task_queues.js:95:5)
at redeemAccount (/usr/app/src/index.ts:31:11)
at async Promise.all (index 0)
at main (/usr/app/src/index.ts:58:5)
[2021-12-16 19:58:11.796 -0500] WARN (44 on a354c2211981): API purchase experienced an error, trying puppeteer purchase

[2021-12-16 19:58:17.999 -0500] WARN (44 on a354c2211981): Error from hCaptcha get_cookie request, continuing without hCaptcha accessibility cookies

status: 401
errorBody: {
"status": "error"

}
[2021-12-16 19:58:30.769 -0500] WARN (44 on a354c2211981): Could not find talon_frame_checkout_free_prod
Error: Could not find talon_frame_checkout_free_prod

at PuppetPurchase.waitForHCaptcha (/usr/app/src/puppet/purchase.ts:113:31)
at processTicksAndRejections (internal/process/task_queues.js:95:5)
at PuppetPurchase.purchaseShort (/usr/app/src/puppet/purchase.ts:174:29)
at redeemAccount (/usr/app/src/index.ts:39:9)
at async Promise.all (index 0)
at main (/usr/app/src/index.ts:58:5)
[2021-12-16 19:58:31.371 -0500] INFO (44 on a354c2211981): Done purchasing Shenmue III
Run once: false
Setting cron schedule as 17 01 * * *

same here,  no difference on the lastest image

Yeah Epic changed a lot. I'll have to figure out what to update and what to ignore.

Yeah Epic changed a lot. I'll have to figure out what to update and what to ignore.

I guess it DID eventually purhcase the game once I let it sit there long enough cycling through errors, about 20 mins later I got my email receipt (which may have been behind due to influx of store orders today)

Okay, there's a lot of stuff happening right now.

The one thing I know is wrong and fixed is the weekly or all search strategy returning a version of Shenmue by a seller called "Epic Dev Test Account". Just had to do some extra offer mapping to fix it.
The purchase browser automation is fairly new, so there are some dialog boxes it may miss (#195)
The API-based purchase broke a couple days ago. This is either due to an API change, or more likely, because captchas were added. (#192, #195)
The Epic Games store has been super unstable today, so it's hard to know if an issue is my fault or not.

From my testing, it appears it is working decently well now. If there's any issues with the purchase browser automation, please open an issue. There's minute differences between every region, so it's tough to catch sometimes.

Pulled the latest docker image, still getting this error. Thought posting here again would make the most sense, hope that's fine...
Log:
`
[2021-12-17 18:12:01.606 +0000] INFO  (44 on 268fd6464cc5): Checking free games for<EMAIL_ADDRESS>[2021-12-17 18:12:07.408 +0000] INFO  (44 on 268fd6464cc5): Successfully refreshed login
user<EMAIL_ADDRESS>
[2021-12-17 18:12:07.408 +0000] INFO  (44 on 268fd6464cc5): searchStrategy is all: searching for weekly and promotional games
user<EMAIL_ADDRESS>
[2021-12-17 18:12:24.764 +0000] INFO  (44 on 268fd6464cc5): Available free games
user<EMAIL_ADDRESS>
availableGames: [

  "Neon Abyss"

]

[2021-12-17 18:12:25.098 +0000] INFO  (44 on 268fd6464cc5): Unpurchased free games
user<EMAIL_ADDRESS>
purchasableGames: [

  "Neon Abyss"

]

[2021-12-17 18:12:25.099 +0000] INFO  (44 on 268fd6464cc5): Purchasing Neon Abyss
[2021-12-17 18:12:27.191 +0000] ERROR (44 on 268fd6464cc5): Error encountered during purchase
user<EMAIL_ADDRESS>
errorBody: {

  "error": true,

  "message": "We could not confirm your purchase at this time.  Please try again later.",

  "order": null,

  "syncToken": "37d9a864-4130-4b33-abd0-3a114debb2bc"

}

code: 400

[2021-12-17 18:12:27.191 +0000] WARN  (HTTPError/44 on 268fd6464cc5): Response code 400 (Bad Request)
HTTPError: Response code 400 (Bad Request)

    at Request.<anonymous> (/usr/app/node_modules/got/dist/source/as-promise/index.js:117:42)

    at processTicksAndRejections (internal/process/task_queues.js:95:5)

[2021-12-17 18:12:27.192 +0000] WARN  (44 on 268fd6464cc5): API purchase experienced an error, trying puppeteer purchase
[2021-12-17 18:12:31.476 +0000] WARN  (44 on 268fd6464cc5): Error from hCaptcha get_cookie request, continuing without hCaptcha accessibility cookies
status: 401

errorBody: {

  "status": "error"

}

[2021-12-17 18:12:32.199 +0000] INFO  (44 on 268fd6464cc5): Loading purchase page
user<EMAIL_ADDRESS>
purchaseUrl: "https://www.epicgames.com/store/purchase?highlightColor=0078f2&offers=1-b0ebefb11a9145488af78f6d2488afff-1ad9b7fca007477ca454d2048d019d2f&orderId&purchaseToken&showNavigation=true"

`
The image it created on the .config folder:

Would the "Accept All Cookies" banner block the bot from claiming the game in any way? This is just a guess but I assume it might try to click the claim button and it doesn't find it?
If I can provide any additional information let me know.

Pulled the latest docker image, still getting this error. Thought posting here again would make the most sense, hope that's fine...

This is identical to #194. Please continue this there. There should have been an HTML file output by the error as well. I'll need that to create the query to click the accept cookies button.


fix(docs): Remove unrelated docs entry

Fixed the docs, removes unrelated entry from an earlier function from the docs.

Thanks!


Tools_v0.8.4 _24: Small pop up screen is drag able even after pinning it.

SEVERITY: Major
MODULE: Tools page
Platform: Windows 7 / Chrome 48.0
STEPS TO REPRODUCE:

Navigate to URL: http://dev-tools.gapminderdev.org:3001/
Tools page is loaded.
Click the “Options” button, a small pop up screen will be displayed
Click “Pin” icon which is available on the small pop up screen
Hold the hand icon and drag the small pop up screen
Note that hand icon is working even after we enable the pin functionality

ACTUAL RESULTS:
Hand icon is working and small pop up screen is drag able even after we enable the pin functionality
EXPECTED RESULTS:
Small pop up screen should not be drag able after enabling the pin functionality
SCREENSHOT:

it's not a bug, it's a feature. see #728


Python crashes when importing HttpApiClient

Just wanted to play a little bit with this python binance chain library on macOS Catalina (10.15.4) using Python 3.7.6 but i can’t get it running. Keep getting an error that my python crashed with error code 134 and the system logs show:
Clients should not load the unversioned libcrypto dylib as it does not have a stable ABI.

Seems like a macOS 10.15 issue: https://stackoverflow.com/a/58445755/1599962


SSH keys configured in config.sh file are added to config.json, do not replace it

Currently, the update script explicitly adds any new keys specified in the config.sh file to the os.sshKeys entry of config.json.  While it seems this is the desired behavior, it has undesirable effects.

It becomes very difficult to keep a reference config.sh for the group of field devices
It becomes very difficult to deprecate and replace ssh keys

To reproduce issue:

check the config.json file on the target device.  Check that the value for key os.sshKeys is empty.
Add an ssh key in config.sh script, e.g.
SSHKEYS=("ssh-rsa AAAA ..... testKey")
Configure the key for the test unit in batch
run the run.sh script.
View the config.json file on the target device.  It will now include the specified key
Change the key in config.sh
Remove config.log and run run.sh again
View the config.json file on the target device.  It will now include two copies of the specified key

issue addressed in https://github.com/etcetc/configizer


(2023-09-28) v. 3.7.2: Documentation Update; Changed README.md (Grammatical, Structural, and Informational Update)

Checklist
Common

[x] I ran the app before creating PR
[x] I ran all tests before creating PR

UI

[x] I ran the app for visual checks.

Logic

[x] I tested basic app functionality.

Changes
This update to the documentation made substantial changes to the ordering of the document. Fixed structural and grammatical mistakes  and added details where needed.

First; Added a table of contents that better organized the documentation. This addition makes it a lot easier to find information in a timely manner.
Second; The layout of the documentation was rearranged. More important features that I believe users and contributors should know about such as "What is this app like" and "How to run the app" have been moved to the top of the page for convenience.
Third; The "About" section was reworded to include a more holistic view of the app, its purpose, and its use.
Fourth; The "How to Use" section was edited to include a lot more details on the documentation itself instead of fully relying on an external link. Different use case for the app were outlined in this section for ease of use
Fifth; Wording added to the "Supporting Languages" and "Videos" sections

Comments
Small grammatical changes were made to titles to make them stand out more

I don't have time to support the application, so the edits are unfortunately not relevant.


Refactor interpreter into working over a separate HIR and AST

Having been working with the interpreter more it's clear that the current design is pretty painful to work with in almost every way. It currently works by doing a pass over the html5ever tokenstream that stores a pretty minimal amount of state while computing the elements for output on the fly. In theory this is very efficient because we can stream our output and we retain a pretty minimal amount of past state, but it also translates to a soup of conditionals that are incredibly difficult to refactor without regressing other behavior.
This also expresses itself where most of the bugs for things lie in not properly keeping track of enough history. Take for instance this markdown snippet
1. First list item

    Some nested text

2. Second list item

This should render as follows:

First list item
Some nested text

Second list item

but the actual HTML for this is pretty complex

First list item

Some nested text

Second list item

There's quite a lot involved in rendering this right: you've got to keep track of which list item you're in, the first element in the list item (
) gets the line with the list prefix while the rest get indented further on separate lines, etc. Unfortunately this shows as we do a pretty sloppy job of rendering most things that are nested even semi-complexly
I think the better way forward would be to first convert the raw HTML to an HIR (higher intermediate representation or HTML intermediate representation :grin:) which is just a nice full structured format describing the things we care about in the HTML doc (basically a more complete version of Element). Then from there we can more easily do the analysis we need to correctly render things above into our output AST which represents how thing are laid out in inlyne's eyes

So what things would we need to do?
And what would the HIR contain?

So for starters the AST would likely remain a VecDeque<crate::Element>. Each element in the HIR would contain all of the crate::interpreter::html::Attrs that we recognize along with the inner runs of text and other elements. For something like

In a paragraph https://example.org

In a list https://example.org

would get represented as something like
use crate::interpreter::html::{attr::Align, Attr, TagName};

enum HirOrText {
    Hir(Hir),
    Text(String),
}

Hir {
    tag: TagName::Paragraph,
    attrs: vec![Attr::Align(Align::Center)],
    contents: vec![
        HirOrText::Text("In a paragraph ".into()),
        HirOrText::Hir(Hir {
            tag: TagName::Anchor,
            attrs: vec![Attr::Href("https://example.org".into())],
            contents: ...
        }),
        ...
    ],
}

That would be the main change as the existing work on the AST can be re-written to run over the HIR without having to change much of the logic there (although it opens the door to more freely change things in the future)

@kokoISnoTarget is currently working on things with #318, so I'll leave the immediate direction of where things go up to them, but yes I am very interested in sharing more of the nitty gritty HTML bits

(once stylo-dioxus is a bit more developed you could also straight-up build inlyne of it / help contribute if that's an approach you were interested in)
It isn't ready, so nothing I would decide now. But sound promising.

We've just landed inline / inline-block support in Blitz (https://github.com/DioxusLabs/blitz). It's still quite early / immature, but it's now in a state that you guys could look into using / contributing to / building on it if you were interested. This is a screenshot of blitz rendering it's own README (the markdown example in the repository):

We have a full HTML style/layout engine so this example is using comrak to compile the README.md file to HTML and applying a stylesheet from https://github.com/sindresorhus/github-markdown-css (theming is just CSS).

Thanks for the update! Things are looking very nice
I think my only big concern with migrating to using something like blitz is that I don't know if there will be a nice layer of abstraction to describe the parts that inlyne specifically needs in terms of both 1) the low-level of control we need, and 2) the loss of control over a large chunk of our dependencies
Both of those are pretty crucial to trying to maintain being a lightweight markdown viewer (although that's still more of a future goal than the current state). Maybe it's better at this point for us to steal different things from each other as features land. It's also totally possible that blitz's development will far outpace ours and maybe in the future it'd be silly for us to not use it :shrug:


Provide direct encryptor stream + some minor improvements

Updated the PR as per your (excellent) comments.


hotwire-livereload.js was not found

Propshaft::MissingAssetError in Home#show
Showing /Users/danieldocki/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/hotwire-livereload-1.2.3/app/views/hotwire/livereload/_head_action_cable.html.erb where line #2 raised:

The asset 'hotwire-livereload.js' was not found in the load path.

16:49:30 web.1      | ActionView::Template::Error (The asset 'hotwire-livereload.js' was not found in the load path.):
16:49:30 web.1      |     1: <% if Rails.env.development? %>
16:49:30 web.1      |     2:   <%= javascript_include_tag('hotwire-livereload', defer: true) %>
16:49:30 web.1      |     3: <% end %>

What can it be?

Sorry, I ran RAILS_ENV=production bundle exec rake assets:precompile before bin/dev 😞


Not working with Vercel CLI 23.0.1

Thanks a lot for this tool, I would like to use it with the latest vercel CLI. I am getting the following error:
Error! Environment Variable was not found.
Error! The Environment "API_URL" is invalid. It must be one of: <production | preview | development>.

Looking at the code, the issue come from a change in the vercel cli. Older CLI has:
 Vercel CLI 21.0.0
 vercel env [options] <command>
 add     [type] [name] [environment]     Add an Environment Variable (see examples below)

New CLI has:
Vercel CLI 23.0.1
vercel env [options] <command>
add     [name] [environment] [gitbranch]  Add an Environment Variable (see examples below)

which means [type]has been removed ([gitbranch]has been added).
Could you please update this project to the latest CLI version?

Fixed in v3.0.0

Fixed in v3.0.0


[FEAT]: Select Excel Reading Starting coordinates

Description
I want to use a template that provides a lot of information. On this template, the actual table only starts at the coordinates (C,7).
Unfortunately, when uploading the xlsx file, the plugin doesn't recognize where the table is.

That is a valid request.
Currently working on another feature, will take some time.


Project Page - Custom Field View - Role Required

Problem -

Previously "Report" was Role name that provides Role requirement to this functionality
Previous design Menu name was Report for All Custom Field View for Contact , Company, Project so Role name was Report
After "Report" role removal, Project Custom Field List Role is missing
so New User can not assign Role for This page
Company and Contact Custom Fields are aligned with Company / Contact Edit Role, so no problem for them, They also previously had Report Role, but now using Edit Role

Solution-
Bring back Report Role for Project only, as Company / Contact works correctly, they do not require any further change
Technical Details-

New Role / API changed required to accommodate this

initiated by @mitulpanch
verified by @vinpahilwani

Yes Role is Required

Yes Role is Required

Yes Role is Required

Please give suggestion for Role name and Sub type?
I suggest this should be independent Role -
ie. Project Custom Field -> Read , Edit
Please provide your View, Placement should be just below Project Role

Matlab?

Project Custom Field -> Read , Edit
ye role bana du?

Yes aisa hi

Showing All Unfinished Projects,
So any body with Project Custom Field Read / Edit Role can process custom field page
No Project Member . Manager etc role required

Project Page roles do not apply here,
This page have 2 role,
Project Custom Field => read
Project custom Field => Edit

Project Page roles do not apply here,
This page have 2 role,
Project Custom Field => read
Project custom Field => Edit

Describe the bug

User has read and edit project custom field role but user can not edit project custom field
create button also not display

https://www.berrycast.com/conversations/16ce6c63-5592-5e56-a905-391d81d20625

User has not customfield role so it is not working


dropbox access failed

Dear SCAFE team,
Check your dropbox files, please.
It seems that the files that need to be downloaded through the scafe.download.demo.input command are not available.

Thanks for reporting the issue. I just tried it and was able to download the demo input. Would you mind to try it again, and post the error message if there is?

This is an error that I've getting after run scafe.download.demo.input:
Singularity> scafe.download.demo.input
--2022-06-28 08:09:27--  https://www.dropbox.com/s/iz3fqv6gy0ljrtp/input.tar.gz Resolving www.dropbox.com (www.dropbox.com)... <IP_ADDRESS>, 2620:100:6022:18::a27d:4212 Connecting to www.dropbox.com (www.dropbox.com)|<IP_ADDRESS>|:443... connected. HTTP request sent, awaiting response... 301 Moved Permanently
Location: /s/raw/iz3fqv6gy0ljrtp/input.tar.gz [following]
--2022-06-28 08:09:28--  https://www.dropbox.com/s/raw/iz3fqv6gy0ljrtp/input.tar.gz Reusing existing connection to www.dropbox.com:443. HTTP request sent, awaiting response... 404 Not Found
2022-06-28 08:09:28 ERROR 404: Not Found.

Hi, you might still be using the previous beta version. Could you update to the current release and try again? Thanks.


[docs] Fix typo in typography.md

Fix typo in typography.md and some typography-*.md.
occassion -> occasion

[x] I have followed (at least) the PR section of the contributing guide.

@hexium310 It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!


How to enable tls 1.1

Describe the bug
I use set_min_proto_version to set min proto version tls1.1. Then the log of tls_settings.min_proto_version() is SslVersion(770).
However, When I set --tls-max 1.1 for curl, tls handshake fail.
curl 'https://charts.npmtrend.com/stats' -v --tls-max 1.1
* Host charts.npmtrend.com:443 was resolved.
* IPv6: (none)
* IPv4: <IP_ADDRESS>
*   Trying <IP_ADDRESS>:443...
* Connected to charts.npmtrend.com (<IP_ADDRESS>) port 443
* ALPN: curl offers h2,http/1.1
* (304) (OUT), TLS handshake, Client hello (1):
*  CAfile: /etc/ssl/cert.pem
*  CApath: none
* LibreSSL/3.3.6: error:1404B42E:SSL routines:ST_CONNECT:tlsv1 alert protocol version
* Closing connection
curl: (35) LibreSSL/3.3.6: error:1404B42E:SSL routines:ST_CONNECT:tlsv1 alert protocol version

And the pingora debug log:
DEBUG 2024-06-13T21:17:41.626730947+08:00 Setting tcp keepalive
DEBUG 2024-06-13T21:17:41.626793712+08:00 new ssl session
DEBUG 2024-06-13T21:17:41.642064484+08:00 Dropping socket BufStream { inner: BufReader { reader: BufWriter { writer: Tcp(PollEvented { io: Some(TcpStream { addr: <IP_ADDRESS>:443, peer: <IP_ADDRESS>:21051, fd: 45 }) }), buffer: 0/1460, written: 0 }, buffer: 0/65536 } }
ERROR 2024-06-13T21:17:41.642136755+08:00 Downstream handshake error  TLSHandshakeFailure context: TLS accept() failed: error:0A000102:SSL routines:tls_early_post_process_client_hello:unsupported protocol:ssl/statem/statem_srvr.c:1755:

Pingora info
Please include the following information about your environment:
Pingora version:  31d7b63ed7e3a1595903bca3680e130fe90e05a0
Rust version: i.e. cargo 1.78.0 (54d8815d0 2024-03-26)
Operating system version:  messense/rust-musl-cross:x86_64-musl
Steps to reproduce
Please provide step-by-step instructions to reproduce the issue. Include any relevant code
snippets.
Expected results
What were you expecting to happen?
Observed results
What actually happened?
Additional context
What other information would you like to provide? e.g. screenshots, how you're working around the
issue, or other clues you think could be helpful to identify the root cause.

By default https://docs.rs/pingora/0.2.0/pingora/listeners/struct.TlsSettings.html creates a TLS server setting that does not support TLSv1.1. According to https://wiki.mozilla.org/Security/Server_Side_TLS, you have to adjust your TLS settings to allow TLSv1.1, including the cipher, security level, ssl options as well as tmp_dh. It is cumbersome but using TLSv1.1 is discourage.
The following works
    // generated from openssl dhparam 1024
    const DH: &str = "
-----BEGIN DH PARAMETERS-----
MIGHAoGBAOEz2IYhQ3IuU28X51BBS+o/s01zOdEaYCbIuiHOQTlviuKwWDiIPFqz
uxt2N265LnDYf1/vSO2E/m7XP1H5UEA4gtJ0J6FhyH9bgF0UHbAyzrwyFR4CboCn
Yskm+g1ZVWDyRs8UO2niPbp7LrmtN6tdWK0RXeqwcxVEJOwijK6XAgEC
-----END DH PARAMETERS-----";
    tls_settings.set_security_level(0);
    tls_settings.clear_options(pingora::tls::ssl::SslOptions::NO_TLSV1_1);
    // we only put TLSv1 cipher here to test
    tls_settings
        .set_cipher_list("ECDHE-ECDSA-AES128-SHA:ECDHE-RSA-AES128-SHA")
        .unwrap();
    let dh = openssl::dh::Dh::params_from_pem(DH.as_bytes()).unwrap();
    tls_settings.set_tmp_dh(&dh).unwrap();
    tls_settings
        .set_min_proto_version(Some(pingora::tls::ssl::SslVersion::TLS1_1))
        .unwrap();

Thanks


Logcat client improvements

You can now pass a CancellationToken to the RunLogService method, which allows you to stop the logcat client
You can now call .ToString() on logcat messages and receive a message in the adb logcat output format.

/cc @bartsaintgermain


Start and stop commands

Is it possible to stop a server without rebooting? And starting it later using a different call? Eg. for scaling extra servers using an API?

@barryvdh can you explain more?

There is only action now: reboot. But you can't shutdown and boot droplets/servers, right?
For example in AWS, you don't pay for an instance that is shut down, so could be preferred to shut something down when it's not needed, and boot it when extra capacity is preferred.

So basically:
https://developers.linode.com/api/v4#operation/bootLinodeInstance
https://developers.linode.com/api/v4#operation/shutdownLinodeInstance
https://developers.digitalocean.com/documentation/v2/#power-on-a-droplet
https://developers.digitalocean.com/documentation/v2/#shutdown-a-droplet
https://docs.aws.amazon.com/cli/latest/reference/ec2/start-instances.html
https://docs.aws.amazon.com/cli/latest/reference/ec2/stop-instances.html

@barryvdh thanks. So is this the workflow?

Stop server using AWS, Linode, or DigOcean API.
Start server using AWS, Linode, or DigOcean API.
Reboot server using Forge API

How does Forge know the new IP address if the stopping and starting changes it?

For load-balanced servers, you only need the internal IP, right? And workers also don't need public IPs

Regardless the changed IP is metadata Forge needs, right?
Did you solve this problem for yourself? If so, would you mind sharing pseudo-code to help explain what works?

Not yet, sorry. I think for scaling workload in workers/queue, I'm going for the Serverless route (waiting for Laracon for that ;) )
For webservers I'm not sure yet. I think it  it would be something like this:

Start (pre-configured) server
Trigger deployment to server to match current state of other servers
Health check
Enable load balancing to that server with Forge API
Scaling down would just be disabling load balancing to that server and stopping it.

But it would be probably not really flexible and if it's a big load then containers/kubernetes etc would be much better.
(Also, you could assign an elastic IP to that instance I think)

Closing this issue because it's inactive, already solved, old or not relevant anymore. Feel free to reply if you're still experiencing this issue and we'll re-open this issue.


[FEATURE] Add Windows support

Should be able to use MSBuild/Visual Studio to build and run the project

Will need to be done parallel with #40


Create binarySearch.java

Related Issue

Info about the related issue

Closes: #[issue number that will be closed through this PR]
Describe the changes you've made
Give a clear description what modifications you have made
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots(optional)

Original
Updated

original screenshot
updated screenshot

@tosankrisna This issue has been assigned to someone already!
so please create an issue before Creating a PR


LUA error when login to character

Every time i login the a character (in version 7.4.1) i got that LUA error:
2x CraftSim/Util/Frames.lua:12: CraftSim Error: Frame not found: 5
[string "=[C]"]: in function `error'
[string "@CraftSim/Util/Frames.lua"]:12: in function `GetFrame'
[string "@CraftSim/Main/CraftSim.lua"]:317: in function `HandleCollapsedFrameSave'
[string "@CraftSim/Main/CraftSim.lua"]:398: in function `?'
[string "@CraftSim/Main/CraftSim.lua"]:4: in function <CraftSim/Main/CraftSim.lua:4>

Locals:
(*temporary) = "CraftSim Error: Frame not found: 5"

you can try deleting the wtf data of craftsim

I deleted and it fixed. Thanks


sol-netctl: Adding the api version of netctl

Signed-off-by: Wu Zheng<EMAIL_ADDRESS>

@ceolin
I  submited the new patch to fix the api_version issue.
Please check it.

merged, thanks !


stop testing Nim pre-1.6 in CI

I do wonder if we should start expressing version constraints more specifically in nimble files for nim - ie we have random versions sprinkled all over the place, but when we discover that a particular library is (significantly) broken with a particular Nim version, the correct thing to do would be to write that down as a version constraint in its nimble file, thus communicating it to users in general

For the record, a couple of repros for the CI issue here; it seems faststreams/outputs related.
repro.txt
repro2.txt

I do wonder if we should start expressing version constraints more specifically in nimble files for nim - ie we have random versions sprinkled all over the place, but when we discover that a particular library is (significantly) broken with a particular Nim version, the correct thing to do would be to write that down as a version constraint in its nimble file, thus communicating it to users in general

https://github.com/status-im/nim-ssz-serialization/pull/45/commits/ecf42b0a9b6d351e3f6287059d580743ed9640e5 seems to hit all the current devel/version-2-0 conditions.
In general, I don't disagree, but here, it's specifically devel, not a released version, which it's incompatible with (in ORC specifically). It's reasonable enough to count incompatible-with-default-memory-manager as incompatible, but devel is not a targetable version as such, so I don't think this concern applies here.
It does, somehow, work under version-2-0, so despite preferring to work within the .nimble file, this adds the CI-side workaround, because from the .nimble perspective, and I just checked a couple hours ago, both devel and version-2-0 are Nim version 1.9.3.
Once either Nim 2.0 acquires its final (NimMajor, NimMinor) tuple values or this devel + ORC + 64-bit target (and no other combination I've found) bug is fixed, then one can remove the .github/workflows/ci.yml workaround.

it's specifically devel,

indeed, supporting devel remains best-effort - the version tracking would note only released nim versions.

Maybe it's https://github.com/status-im/nim-faststreams/issues/39 ?


php74.extensions.iconv: fix error signalling

Motivation for this change
The configure script checks whether iconv supports errno. Unfortunately, on PHP < 8, the test program includes $PHP_ICONV_H_PATH, which defaults to FHS path so it fails to build:
conftest.c:13:10: fatal error: /usr/include/iconv.h: No such file or directory
   13 | #include </usr/include/iconv.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~

That causes the feature check to report a false negative, leading PHP to use a degraded code that returns PHP_ICONV_ERR_UNKNOWN when error occurs, breaking granular error handling in applications.
To prevent this, let’s just include <iconv.h>.
PHP 8 just uses include path so the detection works there: https://github.com/php/php-src/commit/7bd1d703411e1e4b1f663f0cb06af619eea04b42
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 124075 at 024243ba run on x86_64-linux 1

3 packages marked as broken and skipped:

hhvm
libsForQt512.kcachegrind
php73Packages.phpmd

47 packages built successfully:

adminer
php (apacheHttpdPackages.php ,php74)
arcanist
bookstack
drush
icingaweb2
libsForQt5.kcachegrind (libsForQt515.kcachegrind ,plasma5Packages.kcachegrind)
libsForQt514.kcachegrind
lsp-plugins
matomo
matomo-beta
nagios
nextcloud-news-updater
phoronix-test-suite
php73
php73Extensions.iconv
php73Packages.box
php73Packages.composer
php73Packages.composer1
php73Packages.deployer
php73Packages.php-cs-fixer
php73Packages.php-parallel-lint
php73Packages.phpcbf
php73Packages.phpcs
php73Packages.phpstan
php73Packages.psalm
php73Packages.psysh
php74Extensions.iconv
php74Packages.box
php74Packages.composer
php74Packages.composer1
php74Packages.deployer
php74Packages.php-cs-fixer
php74Packages.php-parallel-lint
php74Packages.phpcbf
php74Packages.phpcs
php74Packages.phpmd
php74Packages.phpstan
php74Packages.psalm
php74Packages.psysh
pulseeffects-legacy
pulseeffects-pw
qcachegrind
rss-bridge-cli
unit
wp-cli
yle-dl

6 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/top-level/php-packages.nix:240:7:
    |
240 |       installPhase = ''
    |       ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/top-level/php-packages.nix:416:11:
    |
416 |           (pkgs.writeText "zend_file_cache_config.patch" ''
    |           ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/top-level/php-packages.nix:345:11:
    |
345 |           (fetchpatch {
    |           ^

warning: unused-argument
Unused argument: fetchgit.
Near pkgs/top-level/php-packages.nix:1:22:
  |
1 | { stdenv, lib, pkgs, fetchgit, phpPackage, autoconf, pkg-config, re2c
  |                      ^

warning: missing-phase-hooks
checkPhase should probably contain runHook preCheck and runHook postCheck.
Near pkgs/top-level/php-packages.nix:238:7:
    |
238 |       checkPhase = "echo n | make test";
    |       ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/top-level/php-packages.nix:313:11:
    |
313 |           (fetchpatch {
    |           ^

By the way, I also noticed iconv supports //IGNORE... no. Not sure if it is a problem.
Isolated test:
with import <nixpkgs> { };
with pkgs;

let
  # From PHP: https://github.com/php/php-src/blob/59522ba968d9e736c8c1bc80aa96c5d336b4f979/ext/iconv/config.m4#L128-L143
  testSrc = writeText "test.c" ''
    /* confdefs.h */
    #define PACKAGE_NAME ""
    #define PACKAGE_TARNAME ""
    #define PACKAGE_VERSION ""
    #define PACKAGE_STRING ""
    #define PACKAGE_BUGREPORT ""
    #define PACKAGE_URL ""
    #define HAVE_ICONV 1
    #define HAVE_GLIBC_ICONV 1
    #define PHP_ICONV_IMPL "glibc"
    #define ICONV_SUPPORTS_ERRNO 1
    /* end confdefs.h.  */

    #include <iconv.h>
    #include <stdlib.h>

    int main() {
      iconv_t cd = iconv_open( "UTF-8//IGNORE", "UTF-8" );
      char *in_p = "\xC3\xC3\xC3\xB8";
      size_t in_left = 4, out_left = 4096;
      char *out = malloc(out_left);
      char *out_p = out;
      size_t result = iconv(cd, (char **) &in_p, &in_left, (char **) &out_p, &out_left);
      if(result == (size_t)-1) {
        return 1;
      }
      return 0;
    }

  '';
in runCommand "iconv-errno" {
    buildInputs = [ gcc ];
  } ''
    cc \
      ${testSrc} -o test

    ./test

    echo status=$?
    touch $out
  ''

Yeah, I noticed that too. It seems like it enables a workaround for it when it's broken in the iconv implementation - see http://git.php.net/?p=php-src.git;a=commit;h=473ec539a1c3d242c8b171dd6a5a98fa17e05c13. It's broken in the glibc implementation, but works if libiconv is used instead, by passing "--with-iconv=${libiconvReal}". Maybe it would be cleaner to always use libiconv, which should give us the same implementation on all platforms, but at least it doesn't seem to imply any broken functionality.

Since the work around has been merged as described in the commit and the accompanying issue, I would probably prefer to keep using it – my main goal is to have an environment that is similar to other Linux distros for CI purposes.


add intrinsics for uint64

UInt64::op_add, UInt64::op_sub, UInt64::op_mul, UInt64::op_div, UInt64::op_mod
UInt64::compare, UInt64::op_equal
UInt64::to_int64, Int64::to_uint64

Pull Request Test Coverage Report for Build 1981
Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 88.787%

Totals

Change from base Build 1977:
0.06%

Covered Lines:
3500

Relevant Lines:
3942

💛  - Coveralls

Pull Request Test Coverage Report for Build 1982
Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 88.787%

Totals

Change from base Build 1977:
0.06%

Covered Lines:
3500

Relevant Lines:
3942

💛  - Coveralls

Pull Request Test Coverage Report for Build 1983
Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 88.787%

Totals

Change from base Build 1977:
0.06%

Covered Lines:
3500

Relevant Lines:
3942

💛  - Coveralls


Help: Add a chapter for the cat command

Please fill out this template
Add a chapter for the cat command:
https://github.com/bobbyiliev/101-linux-commands-ebook/blob/main/ebook/en/content/003-the-cat-tac-command.md
Additional context
Make sure to check out the Guidelines.md file on how to contribute!

Can you assign this part to me?

I have forked the repository and want to work on it.

Absolutely! Feel free to go ahead and submit a PR


Add action for when a release is published

We need to automate the process of creating a new version as much as possible, because there's a bunch of parts and its easy to miss one. This PR is an attempt to reduce user workload to solely creating a new release tag on GitHub.
Things that need to happen each time we tag a new released version on GitHub:

[x] Update speechbrain/version.txt
[x] Push new version.txt to main
[ ] Push new version.txt to develop
[x] Build distribution files
[x] Upload to PyPI

So far this PR just updates version.txt and pushes to main and builds and uploads to PyPI.
I think that since Mirco owns the PyPI account he's the one that has to create the PyPI API token and add it as a secret token on GitHub i.e. PYPI_API_KEY.
Also, I'm not sure exactly how we test this without just merging it and creating a new tag.

THe documentation is already generated automatically.

Regarding the PyPI build: Does it matter which version of Python runs the build? Should we specify a particular one?
Regarding the flow here: This all starts when a new release is made on GitHub. Should and could making a release be conditional on passing all the tests? Or should we just make sure the main branch passes the tests otherwise?
Documentation: yeah the documentation stuff runs on readthedocs, it's built there and it automatically follows what happens here on GitHub. I guess @TParcollet owns that account? (I think I have access too.)
Release notes: Can we somehow get an easy view of all the changes that have been incorporated since the last release? This is could be some external tool.
How to see if this works: Yeah I don't know I guess we simply gotta try it and see. We can experiment in a test repo first, but the final workflow here we just gotta do live I suppose.
We should meta-document this whole process, who controls which accounts, and how and even when to do this.

Working on this, but one question I had:

THe documentation is already generated automatically.

I looked just now and Mirco tagged a new release (0.5.7) but I don't see this version on Read the Docs. The only versions I see listed are latest and stable.

In my latest changes, I added some documentation to the end of "Contributing". Also, I added a new action that runs when commits get pushed to main, using this: https://github.com/marketplace/actions/automatic-releases
Seems like a useful way to create a draft release with the changes that have been made. This is my attempt to auto-generate the "Release Notes".

I checked ReadTheDocs (indeed I have admin rights, nice), and added an automation there that builds and shows (Activates in RTD lingo) documentation for any new SemVer-style tag. Additionally, RTD has a "latest" build pointing to the develop branch and as far as I can tell "stable" pointing to the main branch. @TParcollet are those built automatically? There doesn't seem to be an explicit automation rule for them but maybe their automation is handled separately?

It is handled with github hooks.

Thank you  @pplantinga, let me try it!


[AST] NFC: Simplify AnyFunctionRef::isKnownNoEscape()

This removes a defer body hack and an assumption that non-defer functions are always escaping.

@swift-ci please test

This doesn’t remove it so much as shuffle it into the type checker.  Can you document that?

Sure. For my own edification and a better comment in the code, why is it still a hack if the type is set correctly from the start?

why is it still a hack if the type is set correctly from the start?

It’s a hack, but less of a hack now that it’s inconfigureInterfaceType which has become Too Big To Fail and needs to be simplified and broken up.  One way or another I have a working theory that there’s a better way to do 90% of the things in there elsewhere.

I think I'm missing something. What you wrote seems like general feedback about configureInterfaceType and not specific feedback about my change. As you requested, I'd like to document this pull request. Can you please be more specific about how/where you want documentation added? Thanks

Thanks for the feedback @CodaFi.
@swift-ci please smoke test and merge.

@swift-ci please smoke test and merge

@davezarzycki I'm not sure what your interest in isKnownNoEscape is, but FYI I've got a branch that lowers all function types not declared @escaping as @noescape SILFunctionTypes. That's going to enable stack allocation of closures. I  just need to work through a long list of test failures before pushing it.

Hi @atrick! That's cool! For whatever it may be worth, I was doing some hacking/experimentation that tripped over isKnownNoEscape assuming that all function decls (other than defer statements) were @escaping. I figured I might as well push the cleanup upstream. :-)


function locateGradleAndDownloadIfRequired errors in download tool if version has been downloaded but not installed

function locateGradleAndDownloadIfRequired checks if the requested version is installed but if it is not installed immediately attempts to download without checking if the requested version has been downloaded
If the zip file exists in gradle-installations/downloads/ , the download tool throws Error: Destination file path ... already exists
the function should check for existence of the .zip file in between checking for the installation and initiating the download.

Duplicate of #376


CE Match similar to CO Match

Feature or enhancement
I would like to be able to match Collecting Events similar to the CO match. At least for starters be able to copy and past batches of CEs to see if they're in the database then perhaps in the future make batch updates to CEs.
Location
Task - CE Match
Screenshot, napkin sketch of interface, or conceptual description
No response
Your role
No response

@AntWeb-org When you say "copy and paste CEs" what format would that text look like?

Copy and paste a list of text such as the following CEs:
ANTC48955
ANTC48956
ANTC48957
ANTC48958
ANTC48960
ANTC48961
ANTC48962
ANTC48963
ANTC48964
ANTC48965
ANTC48966
ANTC48967
ANTC48968
ANTC48969

You can do that now by proxy.  Paste them in Filter CO, then navigate to Filter CE.

Ah- but maybe those are field numbers, not CatalogNumbers, right?

Yes, they are field numbers/trip identifiers for the CE record.

By the way, where can you copy and paste COs into the CO Filter? I pasted a number of catalog numbers into the Matching Identifiers field and received zero results.

Forget my last comment, I figured it out. The list has to be more like a paragraph with identifiers separated by a comma.

If you want to use one per line you can use \n as the delimiter (see the note below).

I tried that before and it didn't seem to work. I am copying and pasting the list of identifiers from a column in excel so the delimiters are paragraph marks.

Again, forget my last comment - I got it :)

This presently available on the CE filter, I don't anticipate adding an individual match task, but rather add downstream reports based on the match as needed.


Decreased accuracy of results

when i convert h5 model to ncnn model
process with the training data,  the ncnn model result is worse than h5 result
I  check the input data and find nothing different
i want to checkout if  it is a problem with model conversion
but there is error with debug mode

please help~
Thankyou very much

the model Link
https://drive.google.com/file/d/1RyN9Ccfp1g8NJldr-CR7rr_UWAfYf9ln/view?usp=sharing

See it, I will take a look, but in practical terms, it is pretty difficult to solve the misalignment of accuracy.

please help me  to use the debug mode.
it is so cool to show the convert result
Give my best wish to you and your family

ok,thanks. it is solved by update the relu6 bug report


Api End Point : Add Data to Data product ( Presigned Url)

There is an implementation for presigned url. The scope of the ticket would be to  implement path parameter  based on the document below [ Data operation - Add data to a data product]
https://dsdmoj.atlassian.net/wiki/spaces/DataPlatform/pages/4511991882/Initial+APIs
The end point should be /{data-product-name}/ingest
Versioning should be part of the data product as well - {data-product-name}/{version}/{api-method}
Checklist

[ ] Changes to terraform module for api infrastructure change
[ ] Changes to lambda to consume the values from path instead of query string

Definition of Done

[ ] File upload functionality is working and could see the file landed in landing bucket
[ ] unit test changed or implemented as per new change
[ ] smoke test implemented ( if possible)

#1594


Add ability to invert the horizontal and vertical axis

Resolves #63
Adds -iv and -ih (with long-names --invert-vertical and --invert-horizontal respectively) that flips the displayed image - for calibration, still the "un-flipped" frame is passed

LGTM!  Thanks @VanDavv


[SGx Mode] Bugs in Teaclave deployment on azure

I built teaclave on azure Standard DC2s v2 VM instance, following the instruction from https://github.com/apache/incubator-teaclave/blob/master/docs/azure-confidential-computing.md
When I tried to launch Teaclave services, I encountered the following error.
$ (cd docker && ./run-teaclave-services.sh -d)
/var/run/aesmd/aesm.socket socket detected.
Error: No such volume: aesmd-socket
Cannot find a valid sgx device.

I checked properly that SGx  driver availability with below results on Azure VM instance.
$ ls /dev/sgx
enclave  provision

Resolved the issue by installing ISGX driver to azure VM.
wget https://download.01.org/intel-sgx/sgx-linux/2.11/distro/ubuntu18.04-server/sgx_linux_x64_driver_2.6.0_b0a445b.bin
Azure VM by default setting up only SGX drivers. So ISGX will be needed to use intel auth services.
Also docker volume "aesmd-socket" needs to be created manually if not added as default at the time of installing SGX driver.

@hiteshvpatel256, this could be an issue of detecting sgx devices on Azure in the run-teaclave-services.sh script. Let me setup a new one on Azure and go through the instruction to see the root cause again.

@mssun That is the same problem that I had and we commented previously on discord (https://discord.com/channels/891067964743376906/891068072503410738/944151562765361192)

Hi Torvel, thank you for bringing up the issue. We have an internal version of Teaclave, and it works perfectly on Azure Confidential Cloud. The open-source deployment scripts have some problems to be fixed. All the core developers of Teaclave are experiencing some qualifying life events. I will work on the issue once I have access to Azure Confidential VMs.


iOS swiper can't load Image after pop

swiper has 3 image loading from url, and I set the autoplay prop, then I push to the next View(using navigator animated) , wait few seconds and pop back 。。And the image just go away ,it will show after next autoplayTimeout . I don't know how to fix it~~

I have the same problem, set autoplay time quickly ?

Swiper  add  removeClippedSubviews = {false}


Fix Hotplug disk to VM adds RestartRequired condition

What this PR does
Before this PR:
The RestartRequired condition was always set on the VM if the VM rollout strategy wasn't live update and an hotplug volume was added
After this PR:
Fix the wrong set of the require condition. Hotplugging a volume doesn't set the RestartRequired condition anymore

Fixes https://github.com/kubevirt/kubevirt/issues/13222
NONE

/cc @mhenriks

/cc

@mhenriks the problematic part for hotplug is to understand which volumes that has been modified are part of the hotplug request.
Right now, this works in a way that we patch the volume request in the VM status, the in the next iteration of the controller we update the VM spec base on the request and we remove the request from the status. Consequentially, this triggers again the controller which has the volumes set in the spec change, the problem is that we need to understand which change is valid, but we lost information the status.
I really hope I can get it from the VMI, otherwise, I have no idea how the get the information.

@mhenriks I think, I found the trick.
So, we should compare the latestVM with the VMI volumes. The VMI is the runtime source of true. In the case of imperative hotplug, we first patch the VM status with the request AND the VMI spec, then we patch the VM spec. So, comparing the VMI spec and the last seen VM spec is correct. I also tested using virtctl and patching the volume declarative, In the first case, the condition isn't set while for the latter is.
In the case of the live uptable fields, the flow is the opposite, first we patch the VM spec, we verify if the update is valid and then we have the change in the VMI spec.
I will clean-up the code and implement the test, and the push the code.

@mhenriks PTAL

/lgtm

/lgtm

/cherrypick release-1.4
/cherrypick release-1.3

/cherrypick release-1.4


Update tests

Even though the current tests cover 100% of the code, they should be rewritten in a specific style.

Done.


EACCESS Denied error on start

Looking at the docker logs i'm getting an EACCESS denied error and googling it looks to be an npm error, but i can't seem to get it to work.

Im getting the same error too

If it helps, what you're seeing is a permissions issue. The /app/config directory specified in the run file needs to be accessible by whichever account you're running the docker image with. If you correct that, your issue should go away.

Following what @SoSJames said should fix the issue


Don't throw an Exception when Attribute specified in Config does not exist

We have a common config for all our applications. In this case the config is for bout the Kafka Producer and Kafka Consumer. The application using this config actually might not be a KafKa producer or consume at all, or is just one of them.
We are now getting exceptions in the log when the jmx attribute is missing while we have specified it in the config. I think it would be better to not start collection and just omit this metric.
The reference.conf we are using: reference.zip
11:41:47.898 TKD [api-push-test-akka.actor.default-dispatcher-8] ERROR o.a.kafka.common.metrics.JmxReporter - Error getting JMX attribute:
javax.management.AttributeNotFoundException: Could not find attribute request-latency-max
	at org.apache.kafka.common.metrics.JmxReporter$KafkaMbean.getAttribute(JmxReporter.java:189)
	at org.apache.kafka.common.metrics.JmxReporter$KafkaMbean.getAttributes(JmxReporter.java:197)
	at com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.getAttributes(DefaultMBeanServerInterceptor.java:709)
	at com.sun.jmx.mbeanserver.JmxMBeanServer.getAttributes(JmxMBeanServer.java:705)
	at com.segence.commons.jmx.collector.JmxCollector.lambda$null$1(JmxCollector.java:20)
	at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
	at java.util.HashMap$KeySpliterator.forEachRemaining(HashMap.java:1553)
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
	at java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:151)
	at java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:174)
	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:418)
	at java.util.stream.ReferencePipeline$7$1.accept(ReferencePipeline.java:270)
	at java.util.Iterator.forEachRemaining(Iterator.java:116)
	at scala.collection.convert.Wrappers$MapWrapper$$anon$1$$anon$5.forEachRemaining(Wrappers.scala:181)
	at java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
	at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)
	at com.segence.commons.jmx.collector.JmxCollector.queryAsSet(JmxCollector.java:33)
	at kamon.jmx.collector.MetricCollector$.$anonfun$generateMetrics$1(MetricCollector.scala:198)
	at kamon.jmx.collector.MetricCollector$.collectMetrics(MetricCollector.scala:90)
	at kamon.jmx.collector.MetricCollector$.generateMetrics(MetricCollector.scala:198)
	at kamon.jmx.collector.JmxMetricCollectorActor$$anonfun$kamon$jmx$collector$JmxMetricCollectorActor$$waiting$1.applyOrElse(JmxMetricCollectorActor.scala:26)
	at akka.actor.Actor.aroundReceive(Actor.scala:502)
	at akka.actor.Actor.aroundReceive$(Actor.scala:500)
	at kamon.jmx.collector.JmxMetricCollectorActor.aroundReceive(JmxMetricCollectorActor.scala:8)
	at akka.actor.ActorCell.receiveMessage(ActorCell.scala:526)
	at akka.actor.ActorCell.invoke(ActorCell.scala:495)
	at akka.dispatch.Mailbox.processMailbox(Mailbox.scala:257)
	at akka.dispatch.Mailbox.run(Mailbox.scala:224)
	at akka.dispatch.Mailbox.exec(Mailbox.scala:234)
	at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
	at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
	at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
	at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:157)

I saw the same issue with an application I built. It uses Kafka Consumer 1.0.1 version.
Looks like the Kafka library does the logging on its own: https://github.com/apache/kafka/blob/adec6d6e47e18882ba012ae4f1fa8793c18bd0e1/clients/src/main/java/org/apache/kafka/common/metrics/JmxReporter.java#L205
Maybe you're using an older version that didn't use warning level but logged as error?
Looks like the solution is to override the log level for the org.apache.kafka.common.metrics package.

I added this line in my log4j2.xml file (I'm bypassing Kafka's Slf4j implementation):
<Logger name="org.apache.kafka.common.metrics" level="ERROR" />

Closing this issue because error reporting is to do with the underlying library that generates the JMX MBeans and properties therefore there are only 2 solution to prevent these kind of errors:

either remove the query for non-existent metrics from the application.conf file
or silence them by log configuration, depending on your logging library


Typings: add number type for set and push methods

I think number should be a valid type for the value parameter of the set and push methods. I'm getting an error when trying to do the following:

And the following code works well (using vanilla js):
const quickdb = require("quick.db");
quickdb.set("key", 3);
console.log(typeof quickdb.get("key")); // number

Thank you for contributing!


Pad tags with space at the end

Right now if one wants to extract the CFF table using ttx, they have to request -t"CFF ".  The space at the end is implied and we shouldn't require it explicitly.

I'm not sure at which level to fix it.  Maybe just the TTX level.

and "cvt " too

Yes, I think we should fix that from TTX end, and leave the unambiguous four byte tag in the TTFont keys.

Sure, I wasn't implying that we special-case 'CFF' ;)

This might be opening a can of worms.  pyftsubset also has commandline args that take tags, etc...

I see...
Well, the specs allow one to specify arbitrary hew table tags, the only requirement being 4-byte long. So even if one wished to create a table tagged 'CFF' with three bytes, it wouldn't be possible.
I guess we can put a check in the TTFont setitem: pad the "key" to 4 bytes if len(key) <= 4, and raise an exception of > 4. Similarly, the getitem would first pad the key with spaces up length 4 before attempting to get the table.

That's what I had in mind initially.  But it opens door to a case where font[tag1] == font[tag2] when tag1 != tag2...

That sounds like recipe for headache.  I'll take a look inside and see if we can have a normalize tag or something.

Yeah, I think that's what we want.

that's weird I know, but despite the appearance it is actually "True" that
font['CFF'] == font['CFF ']

it could not be otherwise given the 4-byte requirement for table tags in the OT spec.

Ok, if there's water under the bridge already we might as well go full in. :)

maybe it's easier to not open the can of worms -- e.g. font['CFF'] == font['CFF '] -- and simply fix this at the TTX or simlar tools' level.


Improve README

Make the README file more user friendly (for non technical users)
Add screen shots or screen grabs to show step by step process

#15
Screenshots need to be added


Documentation for fatal: Could not read from remote repository.  Please make sure you have the correct access rights

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
Authentication/connection with ssh
there should be a documentation for fatal: Could not read from remote repository.
Please make sure you have the correct access rights.
even after saving ssh public key its showing error mentioned above.
there must be section for it in documentation
What part(s) of the article would you like to see updated?

Additional information
No response

@rakeshfoxberry We're sorry you experienced this issue! This would best be handled by our wonderful support team. Please reach out to them, and they'll be happy to assist you 💛


Avoid to pass --cmd to non-vim-s vi-s

Hello to the entire hub community,
AFAIK not every vi-s support the --cmd option and the current arguments
appended in openTextEditor().
At least with vi(1) on NetBSD doing hub pull-request fails with:
% hub pull-request
vi: unknown option -- -
usage: ex [-eFRrSsv] [-c command] [-t tag] [-w size] [file ...]
usage: vi [-eFlRrSv] [-c command] [-t tag] [-w size] [file ...]
error using text editor for pull request message

At the moment we've patched that in the pkgsrc net/hub package changing the regular expression in openTextEditor() to not match vi (i.e. \b(?:[gm]?vim)(?:\.exe)?$).
Could such patch be acceptable?
Please let me know and I will try to fill a pull request if needed.
Thank you!

We would accept a patch to stop matching vi and just match vim instead! Thank you


Verify Email with Auth

Laravel Version 8 and Admin LTE as from the repository.
###Current behavior
Unable to send emails for reset password or user registration.
Expected Behavior
Sending emails and Verified At timestamp populated to users table.

Changes done to RegisterController

Changes to user model:
class User extends Authenticatable implements MustVerifyEmail
{
    use HasFactory, Notifiable;

Changes to routes, web.php
Auth::routes(['verify' => true]);

Please create an issue on the Laravel project. This is not an issue on this package.


Add Need Help page

There is no need help page on the website.
As it is there on the Netflix site

I would like to work on this issue
if you find it a valid issue please assign it to me

assigned! go ahead


Update approve-to-run-ci.yml

Description
This PR fixes #474
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

If we're going to move forward with commands like this, all repos should support this behavior universally and consistently.
Command should be prepended with / so as to avoid clashing with commonly used comments like the one that @Aisuko gave above.
Please raise this PR and this question in the discuss.layer5.io forum as a proposal for change across all repos.

Also, I generally agree with @Aisuko in terms of this PR: LGTM. Let's review as a broader proposal, though.

If we're going to move forward with commands like this, all repos should support this behavior universally and consistently.
Command should be prepended with / so as to avoid clashing with commonly used comments like the one that @Aisuko gave above.
Please raise this PR and this question in the discuss.layer5.io forum as a proposal for change across all repos.

Will do.

Hi @duckling69 , May you help check the why the Build CI was not triggered? I believe it is not too much useful. And we can change to the simple one which is only adding need key word for the Build job.

Hi @duckling69 , May you help check the why the Build CI was not triggered? I believe it is not too much useful. And we can change to the simple one which is only adding need key word for the Build job.

Will have a look into it

Hi @duckling69
If it is not working so well, let's add a need: [job1, job2] condition for the build job. And we can get a quick merge.

Hi @duckling69 , May you help check the why the Build CI was not triggered? I believe it is not too much useful. And we can change to the simple one which is only adding need key word for the Build job.

Will have a look into it

Hi @duckling69 If it is not working so well, let's add a need: [job1, job2] condition for the build job. And we can get a quick merge.

Sure done


Enable starting a Node without using the bootstrap peers nor mdns (for controlled tests)

important to note:
15:40 <+daviddias> whyrusleeping: does go-ipfs have a command to start a daemon and avoid using mdns and bootstrap nodes or is that all env vars?
15:42 <@whyrusleeping> daviddias: uhm....
15:42 <@whyrusleeping> we were going to have a setting so you could pass arbitrary config values to init
15:42 <@whyrusleeping> but i dont know if that ever happened
15:43 <@whyrusleeping> you'll have to run init, make the changes, and then run the daemon

done on #25

maybe would be good to add  --discovery none and --routing none options

cc @whyrusleeping


The operator "in" is not permitted

I need to use "in" operator inside leftJoin.
knex.select('*').from('users').leftJoin('accounts', function() {
    this.on('accounts.id', 'in', '(1,2,3,4,5,6)');
});

I got error: 'The operator "in" is not permitted'
For inner joins, as a workaround I may move conditions to "where" conditions. However for left joins it is difficult and error prone to move it to where condition.
Of course my actual queries are much more complex than this.
Is there a way to use "in" operator on "on"?

For now you might want to do it with a raw query in the on statement, but I'll look into adding support for this.

I second the request.

@Rush
Use onIn
knex.select('*').from('users').leftJoin('accounts', () => this.onIn('accounts.id', [1, 2, 3]))

Closing, resolved by using onIn


How to consume messages faster

Hello,
Could you please suggest how can i consume messages faster?

if you are more specific, we can give specific advice.

Ask is to consume messages at the rate of 40 millions per hour. Producer (written by another vendor ) is able to produce messages with that speed on 40 partitions. We want to consume that many messages with that speed. I tried to use parallel.Invoke but with no luck.

You don't say how you're trying to use Parallel.Invoke.. (you almost certainly shouldn't be using this). Have a look at the AdvancedConsumer example for how to consume messages.
A single consumer on my dev machine can read at about 400000 messages per second (100 byte messages). 40 million messages per hour is a small number.

Hi Matt . could you please link the advance consumer example code page. I can not find it in the examples directory.


move bounties to closed folder, add README

Move closed bounties to closed_bounties folder, add explanation in README.md and rename bounties to open_bounties.

Please also modify the README to remove the ones that are fully closed, update the links to point to the new locations, and ensure that we refer to the open_bounty folder correctly :)

@JST5000 done

Please also modify the main README to remove the ones that are fully closed, update the links to point to the new locations, and ensure that we refer to the open_bounty folder correctly :)


Make distrib.sh POSIX compliant

This was picked up by pkgsrc:
===> Configuring for hypatia-0.3.3
=> Checking for portability problems in extracted files
ERROR: [check-portability.awk] => Found test ... == ...:
ERROR: [check-portability.awk] distrib.sh: if [ "$1" == "live" ]; then
ERROR: [check-portability.awk] distrib.sh: elif [ "$1" == "test" ]; then

Explanation:
===========================================================================
The "test" command, as well as the "[" command, are not required to know
the "==" operator. Only a few implementations like bash and some
versions of ksh support it.

When you run "test foo == foo" on a platform that does not support the
"==" operator, the result will be "false" instead of "true". This can
lead to unexpected behavior.

There are two ways to fix this error message. If the file that contains
the "test ==" is needed for building the package, you should create a
patch for it, replacing the "==" operator with "=". If the file is not
needed, add its name to the CHECK_PORTABILITY_SKIP variable in the
package Makefile.
===========================================================================

*** Error code 1

Stop.
make[1]: stopped in /lastkaj/pkgsrc/wip/hypatia

Any reason this was closed without merge? Or do we not expect people to be using older shells?

This question is mostly so I know how to go forward with the pkgsrc port.

Oh sorry, this was closed without my knowledge, IDK how that happened!

I will test this and then merge it depending on the results in zsh

Resolved, thanks for pointing this out! Sorry this got lost in the clutter.


Core dumped when load xgboost model in  ARM CPU(C++)

Environment info
Operating System:
Ubuntu 14.04
CPU: Nvida4-Plus-1 ARM Cortex-A15(The miniPC producted by DJI, Manifold)
Package used (python/R/jvm/C++): C++
xgboost version used:
The commit hash :c2b647f26e3b9a870969043fbe2100008d4fb7e1
Steps to reproduce

Train model in Python
Import model in C++
Compile the program
It works well in ubuntu with Intel CPU, but failed in ARM CPU

Here follows the exception infomation

       [12:42:48] dmlc-core/include/dmlc/././logging.h:300: [12:42:48] dmlc-
        core/include/dmlc/./registry.h:78: Check failed: fmap_.count(name) == 0U (1 vs. 0) raw already
      registered

      Stack trace returned 3 entries:
     [bt] (0) ./BigBuff(_ZN4dmlc15LogMessageFatalD1Ev+0x21) [0x5bfd6]
    [bt] (1)

./BigBuff(_ZN4dmlc8RegistryIN7xgboost4data19SparsePageFormatRegEE12__REGISTER__ERKSs+0x40b) [0x80474]
[bt] (2) ./BigBuff() [0x49dea]

     terminate called after throwing an instance of 'dmlc::Error'
     what():  [12:42:48] dmlc-core/include/dmlc/./registry.h:78: Check failed: fmap_.count(name) == 0U (1 vs. 0) raw already registered

      Stack trace returned 3 entries:
       [bt] (0) ./BigBuff(_ZN4dmlc15LogMessageFatalD1Ev+0x21) [0x5bfd6]

[bt] (1) ./BigBuff(_ZN4dmlc8RegistryIN7xgboost4data19SparsePageFormatRegEE12__REGISTER__ERKSs+0x40b) [0x80474]
[bt] (2) ./BigBuff() [0x49dea]

The import code is():

    pFile=fopen("/home/ubuntu/BigBuff/model/xgb.model","rb");
    fseek (pFile , 0 , SEEK_END);
    lSize = ftell (pFile);
    rewind (pFile);
    buffer = (char*) malloc (sizeof(char)*lSize);
    fread (buffer,1,lSize,pFile);

   const float *f;
   bst_ulong out_len;
  DMatrixHandle h_test[1];
  int sample_rows = 1;
  XGDMatrixCreateFromMat((float *) ptr, sample_rows, 784, -1, &h_test[0]);
  XGBoosterCreate(h_test, 1, &h_booster);
  XGBoosterLoadModelFromBuffer(h_booster,buffer, static_cast<bst_ulong>(lSize));
  XGBoosterPredict(h_booster, h_test[0], 0, 0, &out_len, &f);

//    XGDMatrixFree(h_test[0]);
XGBoosterFree(h_booster);
The cmake info is:

cmake ..
-- The C compiler identification is GNU 4.8.5
-- The CXX compiler identification is GNU 4.8.5
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Found CUDA: /usr/local/cuda-6.5 (found suitable exact version "6.5")
-- Found OpenCV: /usr/local (found version "3.4.0")
-- Try OpenMP C flag = [-fopenmp]
-- Performing Test OpenMP_FLAG_DETECTED
-- Performing Test OpenMP_FLAG_DETECTED - Success
-- Try OpenMP CXX flag = [-fopenmp]
-- Performing Test OpenMP_FLAG_DETECTED
-- Performing Test OpenMP_FLAG_DETECTED - Success
-- Found OpenMP: -fopenmp
-- Configuring done
-- Generating done
-- Build files have been written to: /home/ubuntu/BigBuff/build

What have you tried?
I have disturbed by this problem for serveral days...

Can you compile the whole program with debugging flag (-g) enabled and run it inside gdb to produce the full backtrace?

Closing this for the lack of follow-up in the last 90 days. Feel free to re-open.


Added  Blog UI Cards

Added some interesting UI card for blog posts

Thanks for the PR,
Is the code yours?
If not, it is always a good idea to mention credits!
Improvisations -: create a seperate folder with README.MD for your code in the path.
I am marking this as hacktoberfest accepted 👍

Sorry I learn from Youtube: https://www.youtube.com/watch?v=Bst-EZHpuD8&ab_channel=AngelaDelise. I did thank her at the bottom of my index.html but I can add a better credit note

I can remove it if a problem

Its not any problem, I have marked it as Hacktoberfest-accepted !
What you can do is maybe credit the youtube/codepen url of the creator


Avoid endlessly re-connecting in case a connection was already estalished

This can happen in scenarios where the connection is still being established, but the dialer ran into a timeout already.
The dialer then restarts, but when asking the endpoint for connection information it is connected at that point - it then stops and restarts endlessly (until we lose connection which could be never).
resolves https://github.com/itchysats/itchysats/issues/2316

bors r+

bors r+


python312Packages.django-admin-sortable2: 2.2.2 -> 2.2.3

Diff: https://github.com/jrief/django-admin-sortable2/compare/refs/tags/2.2.2...2.2.3
Changelog: https://github.com/jrief/django-admin-sortable2/blob/2.2.3/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

4 packages built:

python311Packages.django-admin-sortable2
python311Packages.django-admin-sortable2.dist
python312Packages.django-admin-sortable2
python312Packages.django-admin-sortable2.dist


Visual bug with APY numbers

This looks to be an older version of the app. The current app design does not look like this


[Improvement] Check if neo4j is running and if not print user friendly error

@kipliklotrika yes, it makes sense, will work on that idea!


I want to use Mongo Operators in pagination

Hi,
I want to improve my queries with parameters. I have to send an array through parameters and query accordingly. I did it myself manually, but I wonder how to do this without breaking the structure.
As an example, what I want to do is filter a content that has an array. I have to shoot content that has the values I have specified in this way. The query I made is below
endpoint?limit=10&page=1&tags[]=foo&tags[]=bar
When I change the codes in the paginate plugin as follows, what I want to do is exactly. How can I make my query in a simple and beautiful way?

 schema.statics.paginate = async function(filter, options) {
  ...
    const countPromise = this.countDocuments(filter).exec();
    const docsPromise = this.find({tags: { $in : ['foo', 'bar'] }} ).populate(options.relations).sort(sort).skip(skip).limit(limit).exec();

Thanks.

@kvlknctk if I understand you correctly, you want to do filtering using the paginate plugin. The filter argument is where you would supply the mongo filter.


report visibility of logs to other github actions beta users

I have confirmed:

if a user is a repo contributor or owner, they can see the actions tab and logs, even when they're not on the beta (as per their personal user account)
if a user is an org member, and the org is on the beta, then the user can see the actions tab and logs for all repos of that org.
even if a user is on the beta (as per their personal user account) but neither member of the org nor the repo, then they can't see the actions tab or the logs.

this will pretty much limit the visibility of whatever actions are doing for TDD, because most contributors won't be members of the repos and/or r-lib/tidyverse orgs.

For example, I can't see the actions tab for https://github.com/actions/docker, even though I am on the beta, and actions/docker appears to be using actions (there's a github/main.workflow).

emailed github support about this.


Remove extraneous items from imaging schema

e.g. genomics terms are showing up in imaging schema probably due to dependencies:
isStranded | libraryPrep | nucleicAcidSource | readLength | readPairOrientation | readStrandOrigin | runType

Closing old issue that hasn't been an issue since 2022...


[SPARK-24412] [SQL] Adding docs about automagical type casting in isin and isInCollection APIs

What changes were proposed in this pull request?
Update documentation for isInCollection API to clealy explain the "auto-casting" of elements if their types are different.
How was this patch tested?
No-Op

add to whitelist

Merged into master as it's only document changes. Congratulations on first PR! Thanks!


Docs leftovers

Proposed changes:

Rename "Rasa Enterprise" to "Rasa X/Enterprise"
Update SQL queries
Update recommendation for data lakes

Generally lgtm too. Just one rewording suggested - not super important


Windows trampoline that works on stable rust

Our current windows launcher require nightly rust and a lot of unsafe just to avoid panics. Puffin on the other hand works on stable rust and doesn't use any unsafe. It would be great to have a stable rust mostly-safe trampoline, even if the resulting binaries are slightly larger.

See #1126

@konstin - I think you linked to the wrong issue.


Remove Cairo

We should think how to do the same without Cairo

I agree. Unfortunately, this issue may not help anyone if no one knows how to do this or can contribute an alternative.

@jonathantneal I made this issue to have a link from our Cult of Martians

@jonathantneal @ai Hello everyone. Saw this issue on the Cult website and spent this weekend working on it. I achieved some results but now I need your input here.
I initially went with node-pureimage library but it quickly became clear that it just won't cut it. Their implementation is way different with what browsers have to offer, unfortunately. Generated images were simply incorrect no matter how hard you try.
There is though a library named Skia which seems to be used under the hood in Chrome and they are currently working on an official WASM version of the library (thanks to inspiration from @Zubnix who initially ported a subset of the lib).
I found 2 modules they have: PathKit and CanvasKit and implemented a solution using both of them. But before I make a PR we should choose one. So here's the deal.
PathKit
Pros:

Has a published npm module, so can be easily used as a dependency
Is somewhat documented and left the experimental stage a while ago

Cons:

Not a canvas replacement, can only generate SVG path commands in node.
The resulting SVG is HUGE! The 400x400 hue wheel svg is almost a 100KB. I am not sure if that is an issue since the original @LeaVerou polyfill generated SVG was about 80KB but it feels like a lot for a single declaration. Will it be fine if there are lots of them?

CanvasKit
Pros:

Is a fully featured drop-in replacement for canvas
From p1 -> can generate a real image and after some changes to bindings even with a format selection: PNG or JPEG
400x400 PNG is 50KB, JPEG with 50 quality argument is 6KB

Cons:

Experimental thing, literally resides under experiments folder, doesn't have an npm module
Required some manual changes to make it work for us

Now, the CanvasKit is really cool but I had to make changes to the Emscripten bindings - the arc function wasn't exposed and the exposed overload for encodeToData was the one without parameters. After switching the overload I was able to render JPEG with a quality parameter and finally got the hue wheel as a 6KB picture, although with visible compression artefacts. You can take a look yourself: https://codepen.io/alx-khovansky/pen/rQMQXN (this is a pasted output from the plugin)
We can add a format & quality option to the plugin though so that people can decide for themselves what is more important.
So, for now the CanvasKit seems like the best option. But it implies that we have to maintain a fork with changes we need + precompile JS/WASM and ship it along with the plugin until they release a public version.
What do you think?
P.S. If none of the proposals seem good enough I can release it as a separate plugin.

@jonathantneal I had this thought too but after researching there doesn't seem to exist anything lightweight and capable enough. Mainly because to transform this you have to implement an arraybuffer-like canvas mock and all the graphics related things and we are back to square one.
svg-to-img mentions on their npm page:

Note: When you install svg-to-img, it downloads a recent version of Chromium (~170Mb Mac, ~282Mb Linux, ~280Mb Win) that is guaranteed to work with the library.

Having to download so much for a simple transformation kind of defeats the purpose.
Plus, again from the description:

If you're planning on running svg-to-img on Debian, you will need to manually install the following dependencies: <a huge list of dependencies>

We will add even more dependency headache for people who build their frontend on CI/CD.

I think CanvasKit is a good option.

Ask CanvasKit to release it.
If they will not release it on npm, put in git repository as git submodule and take it to our npm package

Ok. I will have to talk to people who work on Skia and try to submit a patch with our required changes there first.
I'll keep you updated.

Alright, landed the changes to Skia master. Kudos to @kjlubick for an instant review! As he said, npm module is on the roadmap but not ready just yet.
I will work on incorporating canvaskit as a submodule for now. We can switch to a proper npm lib as soon as it releases.

So, can we make the move?

@jonathantneal Yes. I'm sorry, had a lot going on lately. I will make the PR with canvaskit on the coming weekend.

@jonathantneal PR got delayed but it's finally here. Would love to hear your thoughts on it: #11. Very sleepy but hopefully the description is readable 😄
I think that we might think about a ground-up rewrite (1.0) to address some things:

There is an updated polyfill algorithm available now. I initially implemented changes into the PR but decided to ditch them as unrelated
We now mix things from 2 worlds: old node and new one. Async/await alongside with plain old var. Can upgrade the codebase
We now have options that can be implemented. Not only width/height but also image format and quality for JPEG
Things can be separated into modules for increased readability and maintainability
Not only readability, but for the sake of testability. Currently we have a sanity test that compares a huge pre-existing base64 string. I feel like we can check plugin sanity with a small 1x1 generated image and check plugin correctness by comparing a set of images pre-generated with a browser and our output using something like https://github.com/mapbox/pixelmatch. To do that we have to extract code that generates the image.

If that is something that you think would benefit the plugin then I'd love to work on those proposals after we finish with the originally planned transition.

@khovansky-al, I have moved the updated plugin to https://github.com/jonathantneal/postcss-conic-gradient/tree/next
I hope this will encourage you, as we can now use modern JavaScript to write the plugin. I’ve only started by addressing some of the concerns you mentioned with the old PostCSS syntax, and by breaking out the unique functions (one looks more like a class) into their own files.


add serde support

This PR adds serde support for PublicKey, SecretKey, and Signature values for both min_pk and min_sig. It also adds a simple roundtrip test.
This support is gated by the serde feature flag as requested in discussion of #148. With the serde flag, public keys and signatures are serialized in uncompressed form.
I also added a serde_compressed flag that works the same way, but it serializes compressed points instead. Since the deserialize() methods on PublicKey and Signature can accept both compressed and uncompressed points, there's no interoperability issues with having both compressed and uncompressed serialization. (As you can see, the deserialize function is the same in both cases!)
If it would make sense to add some documentation for these features, please let me know. I'm not quite sure where it should be added, so a pointer would be appreciated!

closes #148

Just in case, if you don't have time to fix it up, I can do it as I merge...

Again, as for the secret key serialization. As already mentioned, I'm reluctant to provide the interfaces beyond the bare minimum in general, and more so if it leaves redundant secret key copies behind. For example, as ser.serialize_bytes(&self.serialize()) would. Wouldn't it be appropriate to zeroize the intermediate buffer? Yeah, I know, I know...  But the core takes every precaution, so why would this part get a pass? [Actually, I was expecting it would be more elaborate. One can even argue that application could just as well throw few lines int...]

I also added a serde_compressed flag that works the same way, but it serializes compressed points instead

This is a very bad idea. Cargo unifies features, so features can get enabled if some other crate needs them. As the result you may get different serialization results depending on what dependencies you have. That is basically a disaster waiting to happen.
BTW you can create a crate that will wrap data structure from this crate with serialization support.

This is a very bad idea. Cargo unifies features, so features can get enabled if some other crate needs them. As the result you may get different serialization results depending on what dependencies you have. That is basically a disaster waiting to happen.

This is a great point that I didn't consider. I will remove it.
This leaves the question: are compressed or uncompressed points preferred for serialization? I think it makes sense to go with uncompressed to avoid paying the modular square root on deserialization, but it's context specific so hard to say.
By the way: while I agree that getting different serialization results based on dependencies is spooky, I'm not sure I agree that it's a disaster: the deserialization routine can consume the result of either of the serialization routines. So it would be weird, but it wouldn't break anything as far as I can tell. (Still: spooky is bad enough, and I'm not arguing against removing the feature.)

BTW you can create a crate that will wrap data structure from this crate with serialization support.

I'm aware of this. But it's code that lots of people will presumably use, which means it's a good candidate for inclusion in the library. Asking lots of different people to write the same code is begging for bugs, incompatibility, etc. And having a separate crate (blst_serde) isn't as discoverable as a feature and, from a dependency standpoint, is morally equivalent. So I do not see any argument for a separate crate.

Again, as for the secret key serialization. As already mentioned, I'm reluctant to provide the interfaces beyond the bare minimum in general, and more so if it leaves redundant secret key copies behind. For example, as ser.serialize_bytes(&self.serialize()) would. Wouldn't it be appropriate to zeroize the intermediate buffer? Yeah, I know, I know... But the core takes every precaution, so why would this part get a pass? [Actually, I was expecting it would be more elaborate. One can even argue that application could just as well throw few lines int...]

Apologies that I didn't address the zeroizing issue after you specifically mentioned it in #148---completely a slip of the mind. Thank you for catching it. I addressed in my latest commit via the very handy Zeroizing wrapper.
Agreed that in principle people can implement this themselves. But on the other hand: beyond interoperability, making sure that the serialization routines get the details right seems like a worthy goal. (For example, I knew I was supposed to zeroize and simply forgot!)

This leaves the question: are compressed or uncompressed points preferred for serialization? I think it makes sense to go with uncompressed to avoid paying the modular square root on deserialization, but it's context specific so hard to say.

Customarily it's argued that compressed is preferred because it halves the bulk storage requirement. But it doesn't have to prevent anybody from using uncompressed in live communications. Besides, when we're talking about serde, we're probably not talking about bulk storage. I mean if one actually cares about space, JSON and alikes won't be an actual choice, right?

I mean if one actually cares about space, JSON and alikes won't be an actual choice, right?

To be fair serde is not just about JSON. It supports various efficient formats like bincode and even zero-copy deserialization.

As one data point, it looks like Eth2 impls use compressed keys everywhere. So apparently no one is worrying so much about the cost of the modular square roots, which I'm sure are fast anyway because y'all are wizards.
@dot-asm do you have a number in mind for the modular square root cost? It's got to be tens of microseconds at worst given the speed of signing---no?

@dot-asm do you have a number in mind for the modular square root cost? It's got to be tens of microseconds at worst given the speed of signing---no?

Does "less than 15" qualify as "tens"? Well, this is for the base field. Double that for the extension field, which should qualify:-) Either way, it's ~1/6 of the sign, but it's more meaningful to compare it to the group-check. It's ~1/3 for the base field and ~1/2 for the extension one.
Again, the Eth2's choice is said to be driven by the [bulk] storage requirements, but I for one argue that the rationale is twisted. Ultimately serialization-deserialization is about communications and it would arguably be appropriate to trade bandwidth for that expensive calculations. In other words it makes more sense to compress only as you store the data for a long term than to use it everywhere...

The secret-key discussion actually asks for the following question. As already mentioned, serialization-deserialization is really about communications. While a secret key is not something one would communicate, there is a non-secret communication-worthy counterpart, blst_scalar. Well, SecretKey is blst_scalar in disguise, but one can say that it's effectively a marker to distinguish scalars that are used specifically as secret keys from the "communication-worthy" ones. In other words, a suggestion would be to add serde interface to blst_scalar. Thoughts? [It might be sensible to add a type alias for the blst_scalar...]

Does that solve the problem? Basically we're telling users just to "declassify" their secret key by turning it into a blst_scalar, but an adversary doesn't care what type I call the bytes ;)
I could be missing your point, of course! Please correct me if I've missed something important here.

Does that solve the problem?

Suggestion to serde blst_scalar is independent of SecretKey. Point is that there a legitimate need to communicate with blst_scalar-s, e.g. to challenge a commitment. The original suggestion was to serde-ify at least EC points, public keys and signatures, but formally speaking we should raise the "at least" bar to cover even blst_scalar.

Basically we're telling users just to "declassify" their secret key by turning it into a blst_scalar,

No, suggestion isn't to facilitate "declassification." There is no direct path from SecretKey to blst_scalar, and adding serde interface to blst_scalar doesn't change that. Naturally it won't stop dedicated developer exercising usafe powers or cross-deserializing the two, but there is nothing we can do about that in either case :-)


Storage data from RTS-GMLC

Hi,
It seems that the data imported from the RTS-GMLC data set is wrong for HydroEnergyReservoir and GenericBattery instances. I could identify two issues but there may be also the third one (see my inflow/initial-storage related constraints).

Scale: the storage.csv lists volumes in GWh. After import, the capacity values appear to be too small by a factor of 1000.
Base power: This issue affects mostly the hydro instances. On import, the storage.csv>Rating MVA is used to do the per-unit conversion for attributes coming from the storage file. For attributes coming from the gen.csv, the gen.csv>Base MVA column is used. When building the JuMP model the later is used for all. The result is that all the parameters that originate from the storage.csv are wrong in the JuMP model, by about 6% as the Rating MVA is usually 50 while the Base MVA 53.

Maybe an example can better demonstrate:
The 222_Hydro_1 plant has a max power of 50MW and a reservoir capacity of 1 GWh
The two constraints now look like:
P__HydroEnergyReservoir_{222_HYDRO_1, 1} ≤ 0.5 This seems to be correct
E__HydroEnergyReservoir_{222_HYDRO_1, 1} ≤ 0.0106 The right side here should be 10. The magnitude problem is what I listed as issue 1 the inaccuracy problem seems to be issue 2
E__HydroEnergyReservoir_{222_HYDRO_1, 2} - E__HydroEnergyReservoir_{222_HYDRO_1, 1} + Sp__HydroEnergyReservoir_{222_HYDRO_1, 2} + P__HydroEnergyReservoir_{222_HYDRO_1, 2} = 0.0007238423518714864 The right side here should be the inflow (0.093) but I haven't figured it out where this goes wrong.
It would be great if your could look into these problems. Please let me know if I can help with anything.
Thanks,
Tamas

Hi @tamas-borbath, are you running a particular script? Can you point me to it, or post it? I need to see which data file you are relying on, the parsing configuration, and the simulation configuration to take a deeper look.
Thanks for posting!

@claytonpbarrows we should move this issue to PowerSystems.jl


--ignore-

For what it's worth, I found that this causes some weird issues when working with promisify.

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Error installing via yay on Manjaro Linux

➜  ~  yay emoji-keyboard-git
==> Starting package()...
Failed to list sessions: Access denied
Traceback (most recent call last):
  File "setup.py", line 5, in <module>
    from lib.emoji_shared import version
  File "/home/james/.cache/yay/emoji-keyboard-git/src/emoji-keyboard/lib/emoji_shared.py", line 102, in <module>
    wayland = check_wayland()
  File "/home/james/.cache/yay/emoji-keyboard-git/src/emoji-keyboard/lib/emoji_shared.py", line 87, in check_wayland
    ['loginctl', 'show-session', session, '-p', 'Type'],
UnboundLocalError: local variable 'session' referenced before assignment
==> ERROR: A failure occurred in package().
    Aborting...
Error making: emoji-keyboard-git

I thought that this might have something to do with just installing XFCE on the same partition as Gnome, but after removing XFCE again, the issue persisted. I have also been able to do a yay update.
System info (using X.Org not Wayland):
➜  ~ inxi -Fxz
System:    Host: james-dl Kernel: 4.14.71-1-MANJARO x86_64 bits: 64 compiler: gcc v: 8.2.1
           Desktop: Gnome 3.30.0 Distro: Manjaro Linux
Machine:   Type: Laptop System: Dell product: Latitude E6410 v: 0001 serial: <filter>
           Mobo: Dell model: 0HNGW4 serial: <filter> BIOS: Dell v: A16 date: 12/05/2013
Battery:   ID-1: BAT0 charge: 48.8 Wh condition: 39.4/48.8 Wh (81%) model: Samsung SDI DELL PT43699
           status: Full
CPU:       Topology: Dual Core model: Intel Core i5 M 560 bits: 64 type: MT MCP arch: Nehalem rev: 5
           L2 cache: 3072 KiB
           flags: lm nx pae sse sse2 sse3 sse4_1 sse4_2 ssse3 vmx bogomips: 21288
           Speed: 1278 MHz min/max: 1199/2667 MHz Core speeds (MHz): 1: 2019 2: 1259 3: 1656 4: 1356
Graphics:  Device-1: Intel Core Processor Integrated Graphics driver: i915 v: kernel bus ID: 00:02.0
           Display: x11 server: X.Org 1.20.1 driver: i915 resolution: 1440x900~60Hz, 1920x1080~60Hz
           OpenGL: renderer: Mesa DRI Intel Ironlake Mobile v: 2.1 Mesa 18.2.2 direct render: Yes
Audio:     Device-1: Intel 5 Series/3400 Series High Definition Audio driver: snd_hda_intel v: kernel
           bus ID: 00:1b.0
           Sound Server: ALSA v: k4.14.71-1-MANJARO
Network:   Device-1: Intel 82577LM Gigabit Network driver: e1000e v: 3.2.6-k port: 7040 bus ID: 00:19.0
           IF: eno1 state: down mac: <filter>
           Device-2: Broadcom Limited BCM43224 802.11a/b/g/n driver: wl v: kernel port: 7000 bus ID: 02:00.0
           IF: wlp2s0 state: up mac: <filter>
Drives:    Local Storage: total: 1.20 TiB used: 513.87 GiB (41.8%)
           ID-1: /dev/sda vendor: Seagate model: ST9320423AS size: 298.09 GiB
           ID-2: /dev/sdb type: USB vendor: Seagate model: Expansion size: 931.51 GiB
Partition: ID-1: / size: 105.24 GiB used: 67.01 GiB (63.7%) fs: ext4 dev: /dev/sda4
Sensors:   System Temperatures: cpu: 46.0 C mobo: N/A sodimm: 43.0 C
           Fan Speeds (RPM): cpu: 2880
Info:      Processes: 205 Uptime: 2h 38m Memory: 3.72 GiB used: 2.50 GiB (67.1%) Init: systemd Compilers:
           gcc: 8.2.1 Shell: zsh v: 5.6.2 inxi: 3.0.26

I can't install pip3 via pacman or yay or find it on Octopi in the official or AUR repositories.

Duplicate of #31

New version, no installation, it's an AppImage, use that.


Fixing Service Name References When Using fullnameOverride For Masters

Description
If using fullnameOverride when creating the masterService, the StatefulSet and Ingress would target a Service that did not exist. This fixes that by creating a new serviceName helper that uses the same conditional logic that was being used in the Service manifests, so that it can be reused in the Ingress and StatefulSet manifests.
NOTE: serviceName in a StatefulSet is an immutable field so this would be a breaking change to any existing installations that used fullnameOverride for the masters. They would have to delete the StatefulSet (but not the PVCs!) and then have helm recreate it in order to upgrade and fix the broken serviceName ref.
Issues Resolved
Closes #92
Check List

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

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

Hi @hfuss sorry for the late response.
Could you possibly create an issue and link to it in this PR?
This would help us track the issues later on.
Thanks.

@peterzhuamazon @hfuss @TheAlgo I have made the necessary changes to the branch and CHANGELOG. Please review.

Conflicts needs to be resolved @hfuss thanks.

@peterzhuamazon @DandyDeveloper I have resolved the conflicts here

@DandyDeveloper @peterzhuamazon I feel we should make update minor version here.

@TheAlgo Done.

This needs to make it to 1.5.4 as #162 consumes 1.5.3.
Also @DandyDeveloper you forget to sign DCO.
Thanks.

Use version 1.5.6.

Hi @hfuss I try to rebase your branch but there are so many commits and conflicts and hard to resolve one by one, since there are several PRs merged after you open this.
Would you mind open a new PR with the exact same change and reference this PR? We can merge very quick since it is already approved in this one.
Or I am willing to open it for you if you are busy with things.
Thanks,
Peter

Hi @hfuss since this PR is really old and have a lot of issues, I have cherry pick the changes to https://github.com/opensearch-project/helm-charts/pull/185.
Please check.
@DandyDeveloper @TheAlgo the changes are exactly the same between this PR and #185.
Will close this one in favor of #185.
Thanks.


1,001.9 xel error

when its more xel then 1 k like 1,001 platform looks like its thinks that you have 1.001
.

Can you circle the problem spot you are talking about? I read this over and over and am not spotting what you are talking about.

Look at the balance. You have a " . " and a " , " it seems that the software or the OS it running from thinks it is not thousand but 1 with some decimals.

yes thats a point .