New unification tsgs

Pull request guidance
Thank you for submitting your contribution to our support content! Our team works closely with subject matter experts in CSS and PMs in the product group to review all content requests to ensure technical accuracy and the best customer experience. This process can sometimes take one or more days, so we greatly appreciate your patience.
We also need your help in order to process your request as soon as possible:

We won't act on your pull request (PR) until you type "#sign-off" in a new comment in your pull request (PR) to indicate that your changes are complete.

After you sign off in your PR, the article will be tech reviewed by the PM or SME if it has more than minor changes. Once the article is approved, it will undergo a final editing pass before being merged.

Hi @v-wendysmith, the draft is ready for your review.

Hi @v-wendysmith, thanks for reviewing the articles. They are published via the private repo. If no further concerns, I'll close this PR.


fix: add dependency on @angular/forms

Adds @angular/forms to the list of dependencies, because there are around 10 Material components that are importing symbols from it.
Fixes #15085.

Ah, totally forgot about the schematics. I've added it.


Memory growth/DFA out of memory

We have a situation when clickhouse starts allocating huge amount of memory and at the same time we see hundreds of logs with:
../contrib/re2/re2/re2.cc:711: DFA out of memory: size 35192, bytemap range 67, list count 29039

The server has 20GB available of allocatable memory, and while facing this problem, allocation reaches up to 19GB and it becomes unresponsive.
Which ClickHouse server version to use
18.6.0
Additional context
We also see at the same time a spike of MemoryTrackingForMerges.
Can you guys please give some hints on how to further debug this and eventually understand the root cause.
Thank you !

That warning is related to regular expression engine.
Do you have some complicated regexps in you DEFAULT expressions?
Can you check what exactly ClickHouse doing by examing /var/log/clickhouse-server/clickhouse-server.log and provide the create statement of the table which was processed at that time?

Seems like CompiledExpressions leak.
Check select * from system.asynchronous_metrics where metric = 'CompiledExpressionCacheCount'
https://github.com/yandex/ClickHouse/issues/3976
And set <compile_expressions>0</compile_expressions> in user's profiles.

You should upgrade to version 19.7.

Seems like CompiledExpressions leak.
Check select * from system.asynchronous_metrics where metric = 'CompiledExpressionCacheCount'
#3976
And set <compile_expressions>0</compile_expressions> in user's profiles.

this one gives me nothing as output. I was checking the issue you mentioned and the graphs there looked very similar to mines. You get a steady grow of memory and then a sudden drop. Maybe I will try to follow @alexey-milovidov suggestion of upgrading clickhouse, since that version already has the fix of the issue you reference.

Then it's not the #3976 compile_expressions leak

../contrib/re2/re2/re2.cc:711: DFA out of memory

These messages does not relate to high memory usage, because re2 has it's own memory limit.
Messages were removed in recent versions.

@lfundaro Does the issue present in recent ClickHouse versions?


show unlatin Characters

how I can show text contain Arabic characters ?

This library just using https://github.com/adafruit/Adafruit-GFX-Library for drawing things, so please ask in the Adafruit-GFX-Library community.


[DatePicker] onBlur event sent at the start of interaction (rather than at the end)

DatePicker onBlur event sent at the start of interaction.
If DatePicker gets an onBlur prop, it send the prop to its TextField.
However, when a user clicks on the TextField, DatePickerDialog will pop up and TextField will lose focus.
This way, the onBlur event is sent at the beginning of interaction.
onBlur should be triggered when user dismisses DatePickerDialog?
Versions

Material-UI: 0.18.1

Closing for #4787


Remove outdated contrast ratio doc.

What's changed, or what was fixed?

Removes https://developers.google.com/web/tools/lighthouse/audits/contrast-ratio which is outdated. Lighthouse currently links to https://dequeuniversity.com/rules/axe/2.2/color-contrast?application=lighthouse instead so I think we're safe removing this doc.

@kaycebasques PTAL

[ ] This has been reviewed and approved by (NAME)
[x] I have run npm test locally and all tests pass.
[ ] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

Whoops!
There were 3 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/tools/lighthouse/audits/_feedback/contrast-ratio.html - Unable to read file, was it deleted?
src/content/en/tools/lighthouse/audits/contrast-ratio.md - Unable to read file, was it deleted?
src/content/ko/tools/lighthouse/audits/contrast-ratio.md - Unable to read file, was it deleted?

Whoops!
There were 3 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/tools/lighthouse/audits/_feedback/contrast-ratio.html - Unable to read file, was it deleted?
src/content/en/tools/lighthouse/audits/contrast-ratio.md - Unable to read file, was it deleted?
src/content/ko/tools/lighthouse/audits/contrast-ratio.md - Unable to read file, was it deleted?


[Migrated] Fix PolicyLengthExceededException

Originally from: https://github.com/Miserlou/Zappa/issues/2190 by wangsha

Description
Fix PolicyLengthExceededException. Checking existing permissions before appending new ones.
GitHub Issues
Fixes #2189

duplicate
https://github.com/zappa/Zappa/issues/923
closing.


Npm Err!  package.json must be actual JSON, not just JavaScript.

I've been tryna fix this for a while, but I can't seem to figure it out. Considering I'm new to coding it may be an easy fix and I'm just not quite sure about what I'm doing'
0 info it worked if it ends with ok
1 verbose cli [
1 verbose cli   'C:\Program Files\nodejs\node.exe',
1 verbose cli   'C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js',
1 verbose cli   'install',
1 verbose cli   'discord.io',
1 verbose cli   'winston',
1 verbose cli   '-save'
1 verbose cli ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose npm-session 39d5088f2ca6dfd0
5 silly install loadCurrentTree
6 silly install readLocalPackageData
7 timing stage:rollbackFailedOptional Completed in 3ms
8 timing stage:runTopLevelLifecycles Completed in 28ms
9 verbose stack Error: Failed to parse json
9 verbose stack Unexpected token n in JSON at position 7 while parsing near '{
9 verbose stack     name: “Blotch”
9 verbose stack     ...'
9 verbose stack     at parseError (C:\Program Files\nodejs\node_modules\npm\node_modules\read-package-json\read-json.js:452:11)
9 verbose stack     at parseJson (C:\Program Files\nodejs\node_modules\npm\node_modules\read-package-json\read-json.js:104:26)
9 verbose stack     at C:\Program Files\nodejs\node_modules\npm\node_modules\read-package-json\read-json.js:51:5
9 verbose stack     at C:\Program Files\nodejs\node_modules\npm\node_modules\graceful-fs\graceful-fs.js:90:16
9 verbose stack     at FSReqCallback.readFileAfterClose [as oncomplete] (internal/fs/read_file_context.js:61:3)
10 verbose cwd C:\Users\Thornstrider Studios\Desktop\Discordbot
11 verbose Windows_NT 10.0.17134
12 verbose argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install" "discord.io" "winston" "-save"
13 verbose node v12.5.0
14 verbose npm  v6.9.0
15 error file C:\Users\Thornstrider Studios\Desktop\Discordbot\package.json
16 error code EJSONPARSE
17 error JSON.parse Failed to parse json
17 error JSON.parse Unexpected token n in JSON at position 7 while parsing near '{
17 error JSON.parse     name: “Blotch”
17 error JSON.parse     ...'
18 error JSON.parse Failed to parse package.json data.
18 error JSON.parse package.json must be actual JSON, not just JavaScript.
19 verbose exit [ 1, true ]
This is what package.json says;
{
name: “Blotch”,
version: “1.0.0”,
main: “bot.js”,
author: “Marsh”,
dependencies: {
}

Hi @Sunriselegacy, the package.json you posted there is not valid json.  Object keys in json must be quoted, and you have an incomplete block, the correct file would be:
{
"name": “Blotch”,
"version": “1.0.0”,
"main": “bot.js”,
"author": “Marsh”,
"dependencies": { }
}

I would also add that a better way to create a package.json file is with $ npm init.  There is a default one which creates a bare bones package.json file, but you can use the package create-package-json if you want more options and better defaults.  For example: $ npm init package-json.
Disclaimer: I wrote and maintain create-package-json, sorry for the self promotion :)

@Sunriselegacy Is this still an issue for you ?


Add usage details to documentation

The documentation could have some implementation details, such as selecting specific database columns to display a result from a resource, and others we don't know.

@underdpt You can. It's using the global search data. I will update the readme after my vacation.

@underdpt You can. It's using the global search data. I will update the readme after my vacation.

it's fantastic that it uses the filament search configuration and doesn't reinvent the wheel! 😄


Improved support for metrics

dbt_metrics uses some dateadd functionality that looks like this in trino: date_add('day', -5, calendar.date_day)
See https://trino.io/docs/current/functions/datetime.html#date_add

@RobbertDM I merged PR with adding dbt-metrics and setting up test suite for it. I included trino__gen_calendar_join macro in that PR, and marked you as the author. Thanks for your contribution!


If the release is an invalid URL, error includes URL

It would have saved me about 1 hour this morning if the error message had included the URL. Really. Don't ask.
Ran unit tests. Did not run integration or acceptance tests.

But... I want to ask. Off by one when reviewing the error message?

@cunnie I m a bit concerned that it would print potentially some basic auth password, etc.

@cppforlife : Good point — we've gone to a lot of trouble to not leak passwords, and this would be a step backwards. Let me see if I can first redact any passwords from the URL before I print.
@dpb587 : I'm not sure what you mean by off-by-one, but what led up to this error was that I specified the wrong manifest on the command-line: e.g. bosh create-env wrong-manifest.yml

I'm closing this PR — if I have time to modify this PR to redact sensitive information, I'll open a new PR.


error: Failed to set remote answer sdp: Called in wrong state: stable

Hey there,
It's working fine if the App is opened from killed state. But the issue is caused when the App is opened from background state and then I try to make a call, onCall is triggered on both the caller and callees side and if the person I'm calling accepts the call I get this error >> ERROR  [VideoChat]: 'setRemoteSessionDescription' error: Failed to set remote answer sdp: Called in wrong state: stable.
The error popups only when I try to make a call, there is no issue in receiving calls.
I think it is caused because of recreating ConnectyCube session for login when I come back from background but I have no choice, I'm using my firebase as CIDP for logging in and If I do not recreate session and skip login when coming back from background then for every query it says token is required. Please help, thanks.

@DaveLomber I'm stuck and waiting for your response :(

@UmerQur3shi make sure when you call - you only pass an array of others users ids. Do not need to include your own user id in it.
In fact - you as a caller - should never receive the 'onCall' callback, only others users should get it

@DaveLomber that was happening because I initialized ConnectyCube in App.js, Initializing it in index.js solved the issue for me :)

@UmerQur3shi ok nice to see it's solved ;)


pgadmin4 failed to install on unstable

Describe the bug
Adding pgadmin4 to /etc/nixos/configuration.nix and then nixos-rebuild switch leads to
Executing pipInstallPhase
/build/pgadmin4-6.11/pip-build/dist /build/pgadmin4-6.11/pip-build
Processing ./pgadmin4-6.11-py3-none-any.whl
Requirement already satisfied: six==1.* in /nix/store/nkwqyk6mxnh71qisa1lx0kqxphhdb8d0-python3.10-six-1.16.0/lib/python3.10/site-packages (from pgadmin4==6.11) (1.16.0)
Requirement already satisfied: passlib==1.* in /nix/store/snv2m22gjpnnsf9lkbsd463n0sklijr6-python3.10-passlib-1.7.4/lib/python3.10/site-packages (from pgadmin4==6.11) (1.7.4)
Requirement already satisfied: Flask-Compress==1.* in /nix/store/1xkzfzzh02v24f2lxs986mgjzzp72xfj-python3.10-Flask-Compress-1.12/lib/python3.10/site-packages (from pgadmin4==6.11) (1.12)
Requirement already satisfied: pyotp==2.* in /nix/store/acyqs6x03hpbwszsskahiylixrcsb7ll-python3.10-pyotp-2.6.0/lib/python3.10/site-packages (from pgadmin4==6.11) (2.6.0)
Requirement already satisfied: qrcode==7.* in /nix/store/xn1gi1jisjkf1nsri96x4mvzxzcqmx3x-python3.10-qrcode-7.3.1/lib/python3.10/site-packages (from pgadmin4==6.11) (7.3.1)
Requirement already satisfied: azure-mgmt-resource==21.* in /nix/store/awsx6bng6xiwp0apwdrp09aqbfs59kcz-python3.10-azure-mgmt-resource-21.1.0/lib/python3.10/site-packages (from pgadmin4==6.11) (21.1.0)
Requirement already satisfied: eventlet>=0.33.0 in /nix/store/rzh148775yfl4g7fz51yfkhj1gl9yya4-python3.10-eventlet-0.33.1/lib/python3.10/site-packages (from pgadmin4==6.11) (0.33.1)
Requirement already satisfied: ldap3==2.* in /nix/store/vvhs4sc3h02p2ki0w04cma41sz7n0nai-python3.10-ldap3-2.9.1/lib/python3.10/site-packages (from pgadmin4==6.11) (2.9.1)
Requirement already satisfied: urllib3==1.26.* in /nix/store/w5zvy2q6y69fikj8zvgxbw5zpkz0awhr-python3.10-urllib3-1.26.10/lib/python3.10/site-packages (from pgadmin4==6.11) (1.26.10)
Requirement already satisfied: SQLAlchemy==1.4.* in /nix/store/zzc2a81ylzcgm1fa355i6yhs3rmkr976-python3.10-SQLAlchemy-1.4.39/lib/python3.10/site-packages (from pgadmin4==6.11) (1.4.39)
Requirement already satisfied: Flask-WTF==1.* in /nix/store/0mz5lnv2ilvnn996nx3dca4vffgc4wl9-python3.10-flask-wtf-1.0.1/lib/python3.10/site-packages (from pgadmin4==6.11) (1.0.1)
Requirement already satisfied: Flask-SocketIO>=5.0.1 in /nix/store/2li55l0ikrammvsc5v88c41jkyg1q6ha-python3.10-Flask-SocketIO-5.1.1/lib/python3.10/site-packages (from pgadmin4==6.11) (5.1.1)
Requirement already satisfied: psutil==5.* in /nix/store/4i68xjfqg1xjzxi068a5rk7pbd0ck3dp-python3.10-psutil-5.9.1/lib/python3.10/site-packages (from pgadmin4==6.11) (5.9.1)
Requirement already satisfied: Flask-Login==0.* in /nix/store/xc1nd23y8fprhsj3hr635fnh7i4d5rhr-python3.10-flask-login-0.6.1/lib/python3.10/site-packages (from pgadmin4==6.11) (0.6.1)
Requirement already satisfied: botocore>=1.23 in /nix/store/x3vaiqf2d9p6bp52v4bw0vc4zxc9f6y7-python3.10-botocore-1.27.31/lib/python3.10/site-packages (from pgadmin4==6.11) (1.27.31)
Requirement already satisfied: bcrypt==3.* in /nix/store/s3gckwr1692kfci3hy36jjiz7qsymiv9-python3.10-bcrypt-3.2.2/lib/python3.10/site-packages (from pgadmin4==6.11) (3.2.2)
Requirement already satisfied: azure-mgmt-subscription==3.0.0 in /nix/store/5vi59p7mxi8xfms1c6954fpa1jawh9m8-python3.10-azure-mgmt-subscription-3.0.0/lib/python3.10/site-packages (from pgadmin4==6.11) (3.0.0)
Requirement already satisfied: python-dateutil==2.* in /nix/store/jl68js2ns8yin04aar7d3ajz92sjmx7s-python3.10-python-dateutil-2.8.2/lib/python3.10/site-packages (from pgadmin4==6.11) (2.8.2)

ERROR: Could not find a version that satisfies the requirement Authlib==0.15.* (from pgadmin4) (from versions: none)
ERROR: No matching distribution found for Authlib==0.15.*

error: builder for '/nix/store/s2ifwkzjz4bxqrxcy2ihdzyyrvqvkcr2-pgadmin-6.11.drv' failed with exit code 1
error: 1 dependencies of derivation '/nix/store/10kwashq28akvs9acca6ap5h9k0zy8mw-system-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/2i73wgryxbp8xr1apnmmjw3a022bw9wi-nixos-system-fw-22.11pre395754.2a93ea177c3.drv' failed to build

Steps To Reproduce
Steps to reproduce the behavior:

Add pgadmin4 to environment.systemPackages in /etc/nixos/configuration.nix
Run nixos-rebuild switch
Fails with error

Expected behavior
NixOS derivation succeeds
Notify maintainers
@gador
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
dsp:~/⁞ nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.06 MiB download, 0.30 MiB unpacked):
  /nix/store/a9f4h2yjhdlk7xj3wj3p7b5r6s659qfl-bash-interactive-5.1-p16-dev
copying path '/nix/store/a9f4h2yjhdlk7xj3wj3p7b5r6s659qfl-bash-interactive-5.1-p16-dev' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 5.18.13, NixOS, 22.11 (Raccoon), 22.11pre395754.2a93ea177c3`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.10.3`
 - channels(root): `"nixos"`
 - channels(dsp): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@dsp I know, thanks for reporting this.
Please see #183599 for the fix or https://nixpk.gs/pr-tracker.html?pr=183599 for the progress


Fix prototype polluting function

Resolves #476

Codecov Report

Merging #477 (0b16ff5) into main (1ac88db) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #477      +/-   ##
==========================================
- Coverage   95.94%   95.93%   -0.02%
==========================================
  Files          12       12
  Lines         345      344       -1
  Branches       54       54
==========================================
- Hits          331      330       -1
  Misses         13       13
  Partials        1        1

Impacted Files
Coverage Δ

packages/core/src/services/PackageJsonMerge.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Extract func GenerateEnvs to flagutil

What is the problem you're trying to solve
There is currently a GenerateEnvs function in cmd/nerdctl/run.go, which is used in commands, run and exec. This function may also be used elsewhere in the future. We can probably extract it to flagutil.
https://github.com/containerd/nerdctl/blob/main/cmd/nerdctl/run.go#L1314-L1334
Describe the solution you'd like
Extract this function and its dependencies into flagutil and add some tests for it.
Additional context
No response

SGTM, I think this function is used in run and exec commands. Maybe you can split it into envutil package

SGTM, I think this function is used in run and exec commands. Maybe you can split it into envutil package

How about putting it here?
/cc @djdongjin
https://github.com/containerd/nerdctl/blob/c2c648c5ef7bad5ebbcb05dcdef9247f31892efe/pkg/flagutil/flagutil.go

@suyanhanx Would you mind to PR this issue first?

@Zheaoli It's OK. I'm handling it.

@suyanhanx can you modify the issue to Add unittest for MergeEnvFileAndOSEnv? Since it's already extracted to flagutil & renamed, only unittest is left.
https://github.com/containerd/nerdctl/blob/d39e9c8137837398395e8af62db0d879ee69b13b/pkg/flagutil/flagutil.go#L127

@suyanhanx can you modify the issue to Add unittest for MergeEnvFileAndOSEnv? Since it's already extracted to flagutil & renamed, only unittest is left.
https://github.com/containerd/nerdctl/blob/d39e9c8137837398395e8af62db0d879ee69b13b/pkg/flagutil/flagutil.go#L127

Sure. Done.


OCPBUGS-18697: controller/factory: skip event handler registration for base controller when ResyncEvery and WithoutEventHandlers is set

When ResyncEvery is used update events can still invoke Sync more often than resyncInterval specified for ResyncEvery. Making ResyncEvery obsolete.
Expected usage:
return factory.New().WithInformers(
		kubeInformersForTargetNamespace.Core().V1().Pods().Informer(),
		kubeInformersClusterScoped.Core().V1().Nodes().Informer(),
).ResyncEvery(10*time.Second).WithoutEventHandlers().WithSync(c.sync).WithSyncDegradedOnError(operatorClient).ToController("ControllerName", eventRecorder), nil

Follow up for https://github.com/openshift/library-go/pull/1620/files#r1399213346.
Testing in https://github.com/openshift/cluster-kube-scheduler-operator/pull/525.

/retest-required

/hold
The solution here is not sufficient

/remove-lifecycle rotten

Closing for not as I don't have capacity to pursue the solution. It's currently a low priority item.


Index Grants into solr

Replaced by https://github.com/sul-dlss-labs/rialto-derivatives/issues/129


Fix a field name for helm releases

UI doesn't show a number of running apps, because uses an invalid field name for helm releases.

👍 LGTM

Coverage remained the same at 43.021% when pulling 9232c837db90f7e872dc01df6f261773d5d4290c on saheienko:ui_apps_number into fb4550aa85296dec537595699ae6282ac9ac789f on supergiant:master.


Add advanced CPU topology settings in Basic Settings step

Fixes: https://github.com/oVirt/ovirt-web-ui/issues/1264
This PR adds support for editing advanced CPU topology (i.e. number of virtual sockets, cores per virtual socket, threads per core) in the Basic Settings of the Create VM Wizard, by adding the "Advanced CPU Topology Options" that we can expand/collapse.
The chosen settings are also displayed in a popover next to the number of the virtual CPUs in the Review report step of the Create VM Wizard.
Also the css style of the pf ExpandCollapse was slightly edited to achieve the expected look (same as before, extra space above the Adv. CPU Topology Options and above their content, proper align of the expand/collapse icon (^ V) with the rest of the component).

After:
Basic settings step:

See also:
https://github.com/oVirt/ovirt-web-ui/pull/1279#issuecomment-697999825
Review step:
https://github.com/oVirt/ovirt-web-ui/pull/1279#issuecomment-688354076

Update of displaying advanced CPU topology settings in the Review step:

The OverlayTooltip and InfoCircleIcon components were used instead of older FieldLevelHelp with respect to the https://github.com/oVirt/ovirt-web-ui/issues/1094 (probably we'll need resolving some merge conflicts, after merging https://github.com/oVirt/ovirt-web-ui/pull/1289). The tooltip in the VM Details card should be updated by https://github.com/oVirt/ovirt-web-ui/pull/1289, too.

In VM details the Advanced Options section covers also cloud-init.
How about including both Start virtual machine on creation and Enable Cloud-init/Sysprep options in the wizard version?

Could you please, explain this suggestion more? Thanks! :)

Could we make the dropdowns the same width as Total Virtual CPUs input?

I spent some time solving this issue specifically but I haven't found any suitable solution, except ugly hacking the css or refactoring the whole Basic settings structure, or using different components that we already use. If we were okay without any expandable section (which complicates that), it would be already done. However, the UX folks strongly recommended to implement an expandable section (without any specific recommendations for drop downs width), similarly as I've implemented that. I can give it some more time but I'm not sure if it worth, regarding the time spent on it and regarding that this is a small cosmetic issue which has no impact on functionality.
Also note that those are drop downs, same as the other drop downs in the Basic settings. The Total Virtual CPUs input is a different component with arrow keys. I've implemented the drop downs for the number of sockets, cores, threads to keep the consistency of this feature with implementation in the VM Details card.

Update:
Regarding suggestions on making the width of the Advanced CPU Topology drop downs shorter, I've chosen a solution based on the editing the maximum width of the div elements with a specific ids. Even this solution depends on the ids, it's simple and works perfectly also when the drop down menu is opened, comparing to the other solutions with setting the number of cols of the components which would require not just a few css adjustments anyway to achieve the expected look.
The result:

The advanced section is below the regular field Grid, so it can be its own Grid.

I don't think it is below the regular field Grid, but I can move it below, however it seems that it doesn't help to achieve the expected look (other than straight CSS manipulation).

Update:
I was trying to implement suggested approach by using Col component (with setting offset or number of cols) or fieldCols to the FieldRow component. Tried different values, combinations etc. but nothing worked as expected. I was able to achieve the expected result (proper aligning and width of the topology drop downs) only by adding some extra conditions to the Col component itself and extra css classes to other components. And at the end, using Col lost its point. I had to set the width using extra css anyway.
One of the reasons why this approach didn't not work well is the style of the Col component with the combination of the pf style of the ExpandCollapse component, especially the .grid-column class is problematic and unnecessary for our situation. But if we fixed that (by adding extra changes to our Col component or hacking css), we still just fixed proper aligning of the topology drop downs and their labels. Then there's still a problem with the width of the drop downs which is calculated slightly differently for the content of the ExpandCollapse component, so setting cols is useless. If we set the cols, we can only get closer to the expected width of the drop downs (if we decide not adding extra css styles and making it even more complicated). However the difference between their width and width of the Memory and Total vCPUs fields was relatively small, it got bigger, visible and annoying while changing size of the window. So I think the best is not using Col at all and to use the same approach as in the src/components/VmDetails/cards/DetailsCard/index.js by putting the content of the FieldRow into extra divs with style we need (see my last commit). Or to use the very simple and working approach I suggested earlier by straight css manipulation:
div[id^='create-vm-wizard-basic-topology-'] {
  max-width: 40%;
}

I updated the layout.
LGTM with that last change.

Note that now the width of the drop downs is not 100% the same as Memory and Total vCPUs fields (it was with my previous solution) but if it is better now then okay.

@hstastna
The advanced section looks great to me.
Please consider adding the "Threads per Core" tooltip, as exists for the VM's details card.

@sgratch Are you sure we need this tooltip even when the VM was not created yet?

eh, if we have such a tooltip then at least fix the tooltip content please (separately)
That "Other recommended values..." - I would drop that sentence completely, it's misleading/confusing.

@sgratch The tooltip already added:

The FieldLevelHelp component was used and "it should be refactored as part of #1289" as you've mentioned in your comment,

no change since last approval other than fixing the tooltip content

LGTM


Command failed with exit code when using ImageAlpha (works great without it)

Hi!
I meet an issue and I don't know how to solve it...
Steps To Reproduce The Error
Try to process a file or multiple file, for example:
$ imageoptim -a EXP_IMG_893010262019.png
$ imageoptim -a '**/*.png'
Expected Behaviour
Actual Behaviour
I've got an error (see below).
Output error looks like:
$ imageoptim -a EXP_IMG_893010262019.png
i Running ImageAlpha...
! Command failed with exit code 98 (Unknown system error -98): /Applications/ImageAlpha.app/Contents/MacOS/pngquant --ext=.png --force --skip-if-larger --speed=1 --quality=65-80 256 -- /var/folders/k6/78rvflqd1fd8zyttzqy7zkqm0000gn/T/imageoptim-cli/EXP_IMG_893010262019.png

! Please raise an issue at https://github.com/JamieMason/ImageOptim-CLI/issues

    Error: Command failed with exit code 98 (Unknown system error -98): /Applications/ImageAlpha.app/Contents/MacOS/pngquant --ext=.png --force --skip-if-larger --speed=1 --quality=65-80 256 -- /var/folders/k6/78rvflqd1fd8zyttzqy7zkqm0000gn/T/imageoptim-cli/EXP_IMG_893010262019.png
        at Object.exports.pngquant (/usr/local/Cellar/imageoptim-cli/3.0.0/libexec/lib/node_modules/imageoptim-cli/dist/imageoptim:475:19)
        at process._tickCallback (internal/process/next_tick.js:68:7)

Version Numbers or N/A

macOS: 10.14.6
/Applications/ImageOptim.app: 3.0.0
/Applications/ImageAlpha.app: 1.5.1
/Applications/JPEGmini.app: N/A (not used)

Help Needed
I tried to execute pngquant and it eems to work without problem:
$ /Applications/ImageAlpha.app/Contents/MacOS/pngquant --ext=.png --force --skip-if-larger --speed=1 --quality=65-80 256 -- /var/folders/k6/78rvflqd1fd8zyttzqy7zkqm0000gn/T/imageoptim-cli/EXP_IMG_893010262019.png
Other thing, it works fine without option:
$ imageoptim '**/*.png'
i Running ImageOptim...
✓ EXP_IMG_893010262019.png was: 2.99MB now: 1.18MB saving: 1.81MB (60.53%)
✓ EXP_IMG_893310262019.png was: 6.24MB now: 5.85MB saving: 390kB (6.25%)
[...]
✓ TOTAL was: 251MB now: 133MB saving: 118MB (47.05%)
✓ Finished

Thanks!

As far as I can tell from this search: error code 98 refers to an image which is too large for pngquant to handle, but it looks like EXP_IMG_893010262019.png was 2.99MB in size which I don't expect to be a problem.
I will be honest with you, I'm unable to reproduce this and don't have ideas for a fix at the moment.

The code 98 is a normal behavior. It's returned when --skip-if-larger worked as advertised.

Fix released in 3.0.2

Hi, a little word to say thank you! It works fine with this version :)


builtin matlab solvers

desire having the adaptive timestep builtin solvers like ode##

for compilation:

do mex for ode45 and ode23
do mex of odefun passed to stiff solvers

#114


Intellisense stopped working, errors in VS Code Console

Using the latest 1.17.1 version of the extension with VS Code 1.29.1. Made some code changes to a C# file in a .net core 2.1 console app and intellisense stopped working, can't seem to get it to work again. The code compiles fine.
There are no errors or warnings in the OmniSharp Log Output but toggling VS Code Developer tools I do see some errors:

ERR Cannot read property 'selection' of undefined: TypeError: Cannot read property 'selection' of undefined
    at CodeActionProvider.getInitializeFromCtorCommand (C:\Users\geoffh\.vscode\extensions\jchannon.csharpextensions-1.3.0\out\src\codeActionProvider.js:165:33)
    at addInitalizeFromCtor (C:\Users\geoffh\.vscode\extensions\jchannon.csharpextensions-1.3.0\out\src\codeActionProvider.js:15:28)
    at CodeActionProvider.provideCodeActions (C:\Users\geoffh\.vscode\extensions\jchannon.csharpextensions-1.3.0\out\src\codeActionProvider.js:19:9)
    at c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:509:787
    at c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:114:65
    at new Promise (:null:null)
    at Object.t.asThenable (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:114:33)
    at e.provideCodeActions (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:509:746)
    at c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:524:308
    at e._withAdapter (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:520:234)
    at e.$provideCodeActions (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:524:270)
    at t._doInvokeHandler (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:609:757)
    at t._invokeHandler (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:609:423)
    at t._receiveRequest (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:607:973)
    at t._receiveOneMessage (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:606:957)
    at c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:604:773
    at c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:103:886
    at e.fire (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:105:344)
    at a (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:156:881)
    at Socket._socketDataListener (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:157:95)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at Pipe.onread (net.js:594:20)

ERR Pending request cancelled: /typelookup: Error: Pending request cancelled: /typelookup
    at RequestQueue.cancelRequest (C:\Users\geoffh\.vscode\extensions\ms-vscode.csharp-1.17.1\out\src\omnisharp\requestQueue.js:45:29)
    at RequestQueueCollection.cancelRequest (C:\Users\geoffh\.vscode\extensions\ms-vscode.csharp-1.17.1\out\src\omnisharp\requestQueue.js:115:15)
    at token.onCancellationRequested (C:\Users\geoffh\.vscode\extensions\ms-vscode.csharp-1.17.1\out\src\omnisharp\server.js:413:40)
    at e.fire (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:105:344)
    at e.cancel (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:112:72)
    at e.cancel (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:112:794)
    at Object.c (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:607:1017)
    at t._receiveCancel (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:608:753)
    at t._receiveOneMessage (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:607:276)
    at c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:604:773
    at c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:103:886
    at e.fire (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:105:344)
    at a (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:156:881)
    at Socket._socketDataListener (c:\Users\geoffh\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:157:95)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at Pipe.onread (net.js:594:20)

/C:/Users/geoffh/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:294 [Extension Host] rejected promise not handled within 1 second
t.log @ /C:/Users/geoffh/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:294
/C:/Users/geoffh/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:1446   ERR An unknown error occurred. Please consult the log for more details.

Ported from https://github.com/OmniSharp/omnisharp-roslyn/issues/1339

@thnk2wn Are you still having problems? It looks like the first error coming from a different extension: jchannon.csharpextensions-1.3.0.
We'll need your full Omnisharp log in order to diagnose further.

It’s a sporadic problem so it’s hard to say. I included all the logs I was aware of at the time. Haven’t seen again recently.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Ravi Chande<EMAIL_ADDRESS>Sent: Monday, December 17, 2018 12:20 PM
To: OmniSharp/omnisharp-vscode
Cc: Geoff Hudik; Mention
Subject: Re: [OmniSharp/omnisharp-vscode] Intellisense stopped working, errors in VS Code Console (#2715)
@thnk2wnhttps://github.com/thnk2wn Are you still having problems? It looks like the first error coming from a different extension: jchannon.csharpextensions-1.3.0.
We'll need your full Omnisharp log in order to diagnose further.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/OmniSharp/omnisharp-vscode/issues/2715#issuecomment-447984886, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAHW0b4bDe0RsV6jCfLuuR-ZvANbS-owks5u5_yLgaJpZM4Y590W.

@thnk2wn Thanks, in that case I'm going to close this issue. Go head and open a new one if you have more problems.


fix import order "isort"

Codecov Report

Merging #45 (282a8a4) into master (33a1d96) will decrease coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
- Coverage   93.62%   93.49%   -0.13%
==========================================
  Files           7        7
  Lines         251      246       -5
==========================================
- Hits          235      230       -5
  Misses         16       16

Impacted Files
Coverage Δ

pydantic2graphene/types.py
100.00% <ø> (ø)

pydantic2graphene/__init__.py
100.00% <100.00%> (ø)

pydantic2graphene/converter.py
93.65% <100.00%> (-0.10%)
:arrow_down:

pydantic2graphene/converter_helpers.py
91.66% <100.00%> (-0.93%)
:arrow_down:

pydantic2graphene/fields.py
92.10% <100.00%> (ø)

Continue to review full report at Codecov.

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


[BUG]

Context
In order to help us troubleshoot issues with this project, you must provide the following details:

~ElasticDump~ MultiElasticDump version: 6.103.0
~Elasticsearch~ OpenSearch version: 2.9.0
Node.js version (note that nodejs v10+ is required for this tool): v18.14.0
Full command you are having issue with (with inputs and outputs):NODE_TLS_REJECT_UNAUTHORIZED=0 \
multielasticdump \
  --direction=dump \ \
  --match='^.+$' \
  --input=https://<IP_ADDRESS>:9200 \
  --output=/Users/geraint/tmp/elasticdump/backups \
  --limit 1000 \
  --input-ca='/Users/geraint/certs/ca.crt' \
  --input-cert='/Users/geraint/certs/tls.crt' \
  --input-key='/Users/geraint/certs/certs/tls.key'

A data-set you can share publicly which reproduces the problem
n/a

Describe the bug
When I invoke elasticdump on the command line, using the above https path and certs, it works.
Example:
NODE_TLS_REJECT_UNAUTHORIZED=0 \
elasticdump \
  --input=https://<IP_ADDRESS>:9200 \
  --output --output='/Users/geraint/tmp/elasticdump/backups/all-indices.tgz' \
  --fsCompress \
  --input-ca='/Users/geraint/certs/ca.crt' \
  --input-cert='/Users/geraint/certs/tls.crt' \
  --input-key='/Users/geraint/certs/tls.key' \
  --limit 1000 \
  --overwrite \
  --tlsAuth

Mon, 04 Sep 2023 11:46:28 GMT | starting dump
(node:16171) NOTE: We are formalizing our plans to enter AWS SDK for JavaScript (v2) into maintenance mode in 2023.

Please migrate your code to use AWS SDK for JavaScript (v3).
For more information, check the migration guide at https://a.co/7PzMCcy
(Use `node --trace-warnings ...` to show where the warning was created)
(node:16171) Warning: Setting the NODE_TLS_REJECT_UNAUTHORIZED environment variable to '0' makes TLS connections and HTTPS requests insecure by disabling certificate verification.
Mon, 04 Sep 2023 11:46:30 GMT | got 1000 objects from source elasticsearch (offset: 0)
Mon, 04 Sep 2023 11:46:30 GMT | sent 1000 objects to destination file, wrote 1000
Mon, 04 Sep 2023 11:46:31 GMT | got 689 objects from source elasticsearch (offset: 1000)
Mon, 04 Sep 2023 11:46:31 GMT | sent 689 objects to destination file, wrote 689
Mon, 04 Sep 2023 11:46:31 GMT | got 0 objects from source elasticsearch (offset: 1689)
Mon, 04 Sep 2023 11:46:31 GMT | Total Writes: 1689
Mon, 04 Sep 2023 11:46:31 GMT | dump complete

However, when I attempt to invoke multielasticsearch using the same URL and certs, it fails.
Example:
NODE_TLS_REJECT_UNAUTHORIZED=0 \
multielasticdump \
  --direction=dump \ \
  --match='^.+$' \
  --input=https://<IP_ADDRESS>:9200 \
  --output=/Users/geraint/tmp/elasticdump/backups \
  --limit 1000 \
  --input-ca='/Users/geraint/certs/ca.crt' \
  --input-cert='/Users/geraint/certs/tls.crt' \
  --input-key='/Users/geraint/certs/tls.key'

Mon, 04 Sep 2023 11:59:54 GMT | We are performing : dump
Mon, 04 Sep 2023 11:59:54 GMT | options: {"debug":true,"parallel":10,"match":"^.+$","order":"asc","input":"https://<IP_ADDRESS>:9200","output":"/Users/geraint/tmp/elasticdump/backups","scrollId":null,"scrollTime":"10m","scroll-with-post":false,"timeout":null,"limit":1000,"offset":0,"size":-1,"direction":"dump","support-big-int":false,"big-int-fields":"","ignoreAnalyzer":true,"ignoreChildError":false,"ignoreData":false,"ignoreMapping":false,"ignoreSettings":false,"ignoreTemplate":false,"ignoreAlias":true,"ignoreIndex":true,"ignoreType":[],"includeType":null,"interval":1000,"delete":false,"prefix":"","suffix":"","transform":null,"headers":null,"searchBody":null,"searchWithTemplate":null,"cert":null,"key":null,"pass":null,"ca":null,"tlsAuth":false,"input-cert":"/Users/geraint/certs/tls.crt","input-key":"/Users/geraint/certs/tls.key","input-pass":null,"input-ca":"/Users/geraint/certs/ca.crt","output-cert":null,"output-key":null,"output-pass":null,"output-ca":null,"httpAuthFile":null,"concurrency":1,"carryoverConcurrencyCount":true,"intervalCap":5,"concurrencyInterval":5000,"overwrite":false,"fsCompress":false,"awsChain":false,"awsAccessKeyId":null,"awsSecretAccessKey":null,"awsIniFileProfile":null,"awsService":null,"awsRegion":null,"awsUrlRegex":null,"s3AccessKeyId":null,"s3SecretAccessKey":null,"s3Region":null,"s3Endpoint":null,"s3SSLEnabled":true,"s3ForcePathStyle":false,"s3Compress":false,"s3ServerSideEncryption":null,"s3SSEKMSKeyId":null,"s3ACL":null,"quiet":false}
Mon, 04 Sep 2023 11:59:54 GMT [debug] | GET /_aliases
(node:17119) NOTE: We are formalizing our plans to enter AWS SDK for JavaScript (v2) into maintenance mode in 2023.

Please migrate your code to use AWS SDK for JavaScript (v3).
For more information, check the migration guide at https://a.co/7PzMCcy
(Use `node --trace-warnings ...` to show where the warning was created)
(node:17119) Warning: Setting the NODE_TLS_REJECT_UNAUTHORIZED environment variable to '0' makes TLS connections and HTTPS requests insecure by disabling certificate verification.
Mon, 04 Sep 2023 11:59:55 GMT | Error: 80204EEC01000000:error:0A000412:SSL routines:ssl3_read_bytes:sslv3 alert bad certificate:../deps/openssl/openssl/ssl/record/rec_layer_s3.c:1600:SSL alert number 42

I suspect that multielasticdump is not passing the certs into elasticdump.
To Reproduce
See above.
Current behavior
Describe what's happening now.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

A good bug report shouldn't leave others needing to chase you up for more information. Please try to be as detailed as possible in your report. What is your environment? What was the url? What steps will reproduce the issue? What OS experience the problem? All these details will help to fix any potential bugs as soon as possible.

Since the direction is dump, you need to set the output- params also. It sounds strange, but you're outputting to multi-elastic so the convention is a little different.
❯ NODE_TLS_REJECT_UNAUTHORIZED=0 \
multielasticdump \
  --direction=dump \
  --match='^.+$' \
  --input=https://<IP_ADDRESS>:9200 \
  --output=/Users/geraint/tmp/elasticdump/backups \
  --limit 1000 \
  --input-ca='/Users/geraint/certs/ca.crt' \
  --input-cert='/Users/geraint/certs/tls.crt' \
  --input-key='/Users/geraint/certs/tls.key' \
  --output-ca='/Users/geraint/certs/ca.crt' \
  --output-cert='/Users/geraint/certs/tls.crt' \
  --output-key='/Users/geraint/certs/tls.key' \
  --tlsAuth

In retrospect i might need to change the direction in the code.

Get the latest version and retest (v 6.103.1)


added BundleUtils.getResourceByFullUrl

I don't know if any permissions have changed, but I am not able to merge the pr atm

Hey @maxi-smidt , the merging rights have always been with smile team.

Ah okay I wasn't sure if I was able to in the past... never mind


System.OutOfMemoryException

Couple of days ago I deployed an update to my app that contains this very library. My userbase consists of roughly 1000 people - it's not huge, but significant.
Everything (that's related to this library) seems to work great, except for one thing. Sometimes, usually on phones with 512 MB of RAM, System.OutOfMemoryException is thrown.
It originates from FFImageLoading.Extensions.ImageExtensions.ToBitmapImageAsync(), specifically from WriteableBitmap's contructor.
I use downsampling wherever I can, but apparently it's not enough.
So, essentially I can see two issues here:

exception from library is not caught and causes app to crash. This is easy to fix.
library uses all of phone's memory. Can I set in-memory cache's maximum size, or something like that?

Crashing is not that common, but nonetheless I'd like it to stop.
I almost forgot - it runs under Windows Phone 8.1. Windows 10 version is much less popular, so I don't have similiar data.

I have now in a branch: https://github.com/molinch/FFImageLoading/commit/9709bc21564a02bf226a4756faf0eb396e54c445
2 Disposable resources were not disposed, and old transformation data is nullified now.

I have now in a branch: https://github.com/molinch/FFImageLoading/commit/9709bc21564a02bf226a4756faf0eb396e54c445
2 Disposable resources were not disposed, and old transformation data is nullified now.

@molinch Thanks for adding using - I missed that.
But this: https://github.com/molinch/FFImageLoading/commit/9709bc21564a02bf226a4756faf0eb396e54c445#diff-c77559e28f7ebcd88b6ae27a8ad5796dL307 won't work because on Windows the same instance of BitmapHolder (and pixel array) is used for all transformations and other operations.

@molinch Thanks for adding using - I missed that.
But this: https://github.com/molinch/FFImageLoading/commit/9709bc21564a02bf226a4756faf0eb396e54c445#diff-c77559e28f7ebcd88b6ae27a8ad5796dL307 won't work because on Windows the same instance of BitmapHolder (and pixel array) is used for all transformations and other operations.

It's not always the same instance. Look at FlipTransformation for example.

It's not always the same instance. Look at FlipTransformation for example.

Yes, FlipTransformation is one of rare exceptions - and it's nullified explictly. The whole point why BitmapHolder is reused in Windows is that in most of cases (transformations) we're dealing with pixel operation - there's no point to create int array every time as we only modify some pixels. We should definitely remove those lines.

Yes, FlipTransformation is one of rare exceptions - and it's nullified explictly. The whole point why BitmapHolder is reused in Windows is that in most of cases (transformations) we're dealing with pixel operation - there's no point to create int array every time as we only modify some pixels. We should definitely remove those lines.

They will not stop working. Look at the code it nullifies only if needed.

They will not stop working. Look at the code it nullifies only if needed.

Ahhh ok! Sorry didn't see that: if (old != null && old != imageIn && old.Pixels != imageIn.Pixels) :)

Ahhh ok! Sorry didn't see that: if (old != null && old != imageIn && old.Pixels != imageIn.Pixels) :)

But this old.Pixels != imageIn.Pixels had to be removed. As sometimes in transformations Pixels are changed to different instance currently with bitmapHolder.SetPixels method.

But this old.Pixels != imageIn.Pixels had to be removed. As sometimes in transformations Pixels are changed to different instance currently with bitmapHolder.SetPixels method.

We could do that but still you won't control transformations done by other persons so I would suggest keeping the check

We could do that but still you won't control transformations done by other persons so I would suggest keeping the check

So we should modify transformations to return new instance of bitmap holder if pixel data was changed completely.

So we should modify transformations to return new instance of bitmap holder if pixel data was changed completely.

@daniel-luberda: sure, I understand there are limits. Yet, I'd prefer if app showed error placeholder rather than just crashed. And besides, you rarely have many images displayed at the same time. Unused ones can be just unloaded. I'll see if I can make ListView use less containers.

@kalarepa I'll see what can be done.

@daniel-luberda: sure, I understand there are limits. Yet, I'd prefer if app showed error placeholder rather than just crashed. And besides, you rarely have many images displayed at the same time. Unused ones can be just unloaded. I'll see if I can make ListView use less containers.

@kalarepa I'll see what can be done.

@molinch It can be a little misleading:

User creates a custom transformation
User replaces pixels with SetPixels method
old.Pixels != imageIn.Pixels bitmap holder is null and we'll a null exception

We could make SetPixels internal or remove the check. What do you think?

@molinch It can be a little misleading:

User creates a custom transformation
User replaces pixels with SetPixels method
old.Pixels != imageIn.Pixels bitmap holder is null and we'll a null exception

We could make SetPixels internal or remove the check. What do you think?

3rd point is not valid because:
`if (old != null``
But it's up to you. Basically my concern was mainly when we create another byte array of pixels and leave the old one.

3rd point is not valid because:
`if (old != null``
But it's up to you. Basically my concern was mainly when we create another byte array of pixels and leave the old one.

Once again you're right. I definitely need a cup of coffee :P

Once again you're right. I definitely need a cup of coffee :P

https://github.com/molinch/FFImageLoading/commit/9c16aa2da5edc709c5b4bebc7789b657c5a7f787

https://github.com/molinch/FFImageLoading/commit/9c16aa2da5edc709c5b4bebc7789b657c5a7f787

Question -- is there a way to set a callback or "fallback" to a default image if a failure occurs (as an alternative to bubbling up the exception for a image load failure)?

Question -- is there a way to set a callback or "fallback" to a default image if a failure occurs (as an alternative to bubbling up the exception for a image load failure)?

@daniel-luberda Can I let you merge the changes since I cannot test on WP?
@ravensorb Yes there is a callback for that, it is Error() https://github.com/molinch/FFImageLoading/wiki/API#error
In any case exceptions are trapped, with or without the callback. The OutOfMemoryException is a special case: when it occurs you cannot always trap it & be safe, the OS might kill your app.

@daniel-luberda Can I let you merge the changes since I cannot test on WP?
@ravensorb Yes there is a callback for that, it is Error() https://github.com/molinch/FFImageLoading/wiki/API#error
In any case exceptions are trapped, with or without the callback. The OutOfMemoryException is a special case: when it occurs you cannot always trap it & be safe, the OS might kill your app.

@molinch I made some optimizations https://github.com/molinch/FFImageLoading/commits/windows-changes

@molinch I made some optimizations https://github.com/molinch/FFImageLoading/commits/windows-changes

@kalarepa Could you test the actual repo? It should be better now :)

@molinch One more thing, I didn't do anything related to catching out of memory exceptions. Any advice where could we do that?

@molinch  Also would it do anything at all if OutOfMemory exception on Windows basically means that nothing could be done.

@daniel-luberda: thanks, I'll test it soon.

@daniel-luberda It's already behaving like that whatever the exception we get (on iOS/Android).
There's not much you can do when you get an OOM anyway.

Ok, so we could close the issue now as nothing more can be done here.
@kalarepa Thanks for the info :) If you'll have any issues feel free to reopen it.

@daniel-luberda @molinch
I deployed an update with all those recent improvements. On top of that, I added Image.Source = null; in Page.Unloaded event (as well as logic to set source when loaded again). This change cuts down RAM usage by 10-15 MB when switching image-filled pages.
Update is still propagating to users, so I don't know if it significantly improves the situation. But what I do know is that OOM still happens. I just don't know if less often.
Also, it doesn't seem that FFImage is leaking memory on it's own. When scrolling through virtualized ListView full with images, memory usage doesn't increase. Quite the opposite in fact - it decreases slightly.
Anyway, thanks a lot for responding so quickly. I'll let you know how this pans out.

@kalarepa Thanks, let us know - it's a very useful information.
If you have any idea how could we be more memory efficient, just tell us. My ideas so far:

Change BitmapHolder to use byte[] instead int[] internally - that way we could omit fast copying data from byte[] to int[] when creating WriteableBitmap - but GC should collect old int[] anyway and making custom transformations would be  lot less intuitive as some helpers methods should be used to operate on byte[] instead on pixels int[]
Replace WriteableBitmap with custom implementation of BitmapSource - it could use int[] internally and it would be superb if it didn't require UI thread to modify it (I don't now if it's possible)


Update to latest AppLand JS components

Closes https://github.com/getappmap/appmap-intellij-plugin/issues/462
This PR syncs the package.json files and updates to the latest AppLand JS components.
I've briefly tested this with my own setup.

@jansorg is this ready for review?

Thanks, I must have forgotten to assign it for review.


Add Phantom pack

Phantom
https://github.com/EMERALD0874/SteamUI-Sound-Packs
Checklist
Failure to complete this checklist or deleting boxes from the checklist will result in the closing of your pull request unless this is a pack update. Please write any comments regarding this checklist at the bottom of your pull request.
Check every box.

[x] I am the original author of this pack or have permission from the original author to make this pull request.
[x] All copyright of this pack's contents belong to the listed author or is cited in the description and repository linked above.
[x] This theme works properly on the latest versions of SteamOS for Steam Deck, decky-loader and SDH-AudioLoader.
[x] This pack is under 10MB in size and uses the least disk space possible.
[x] This pack is safe for work and does not contain any sexual, drug-related, or profane content.
[x] This pack does not contain loud, distorted, or otherwise bad faith sound files.
[x] This pack is only for either music or sounds and only adds or modifies the intended sound files.

I'm the only collaborator so... why not?


No Support for External Control?

It appears from reading through your code that this library relies on a local network connection to the devices in order to issue any control commands, even though the C by GE app does not have this same constraint, and will in fact allow me to control my devices from anywhere in the world with an internet connection. Is this correct, that you didn't explore the control mechanisms which rely on the hosted API to communicate the commands down to the devices?
I found your library when searching for any info on the API used to remotely control these switches, bulbs, plugs, etc, so that I can add some automation which is not currently available with the C by GE app or via Google/Amazon and their integrations. If your library relies on the device executing the commands being on the same network segment as the devices being controlled in order to actually make changes, then it will unfortunately not be able to inform my own automation work, beyond the initial auth and device detail retrieval.
Thanks for your work; I look forward to your reply, and I'm hoping I missed something! 😁

Sorry for the delay - yes, this only implements the local Bluetooth control, not the remote API available with the bridge.


Drop tensorflow dependency

Closes https://github.com/neptune-ai/neptune-tensorflow-keras/issues/3

Do we already have a check for presence of tensorflow in runtime?

We're already check for legacy package of keras (which is now part of tensorflow). https://github.com/neptune-ai/neptune-tensorflow-keras/blob/7148cf1d18c15486cc1a442f1942db8b5634925a/neptune_tensorflow_keras/impl/init.py#L24-L35


Fix a few "typos" related to the Ising translator & its tests

Summary
This fixes a few "typos" I noticed when looking carefully at the Ising translator and its tests.   In part, I've replaced "object function" -> "objective function" so that the language will be consistent once the QRAO prototype is contributed to Qiskit Optimization.
Details and comments

Thanks!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.35%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4256

Relevant Lines:
4659

💛  - Coveralls


New names for datasets in Kystverket (Norwegian Mapping Authority)

It seems there was a refactoring of the dataset names from Kystverket. The name for the "Sjøkart - Dybdedata" dataset seems to have changed just to "Depth data" (https://kartkatalog.geonorge.no/metadata/depth-data/2751aacf-5472-4850-a208-3532a51c529a).

Thank you for the report!
The link you mentioned directs me to the Norwegian page, if Norsk had been selected in the Menu as the preferred language before clicking the link. If I select English and refresh the page, it is translated to Depth data along with the rest of the page.

Should I leave the Readme as-is, or remove the Norwegian translation?

Seems reasonable! Super clear and foolproof is a nice goal to strive for, thank you :)


BugFix | Handle duplicate Address allocation error

Description
Updates Token Allocation to handle duplicate address error

Notes
Couple other additional updates

Updates handles blank input values. (button is disabled)
Handles 0 allocationed (button is disabled)
Cleans up update methods as callbacks and include 'snapshots' of state to ensure correct data at time of call.

Issue / Notion doc (if applicable)
Closes #786

Testing
Go to Token Allocations during Usul creation.

Try to add an additional allocation with same address

Try to add an additional allocation different address no amount

Try to add an additional allocation different address 0

Finish off by making sure that Valid values will pass still.

Screenshots (if applicable)

I was able to get it into this state.

So enter a dup, then change the 1st one. Dup error still there.
Honestly, it seems like we need to address validation at an App level. And stop trying to debug all this individual errors?

And does this seem correct. Button is enabled but the amount is still blank for one value?

And does this seem correct. Button is enabled but the amount is still blank for one value?

nice catch. left a .every() instead a .some() on the validation check

@mobiusTripper-crypto this is ready for re-review. do your worst


Fixes for KVM unmanaged instances import on advanced network and VNC password

Description
This PR fixes a regression caused by #8465 on advanced zones, import fails with:
2024-01-10 12:13:33,234 DEBUG [o.a.c.e.o.NetworkOrchestrator] (API-Job-Executor-3:ctx-991bbe9f job-128 ctx-f49517d4) (logid:d7b8e716) Allocating nic for vm 142272e8-9e2e-407b-9d7e-e9a03b81653c in network Network {"id": 204, "name": "Isolated", "uuid": "9679fac5-e3ac-4694-a57b-beb635340f39", "networkofferingid": 10} during import
2024-01-10 12:13:33,239 ERROR [o.a.c.v.UnmanagedVMsManagerImpl] (API-Job-Executor-3:ctx-991bbe9f job-128 ctx-f49517d4) (logid:d7b8e716) Failed to import NICs while importing vm: i-2-31-VM
com.cloud.exception.InsufficientVirtualNetworkCapacityException: Unable to acquire Guest IP  address for network Network {"id": 204, "name": "Isolated", "uuid": "9679fac5-e3ac-4694-a57b-beb635340f39", "networkofferingid": 10}Scope=interface com.cloud.dc.DataCenter; id=1
	at org.apache.cloudstack.engine.orchestration.NetworkOrchestrator.importNic(NetworkOrchestrator.java:4582)
	at org.apache.cloudstack.vm.UnmanagedVMsManagerImpl.importNic(UnmanagedVMsManagerImpl.java:859)
	at org.apache.cloudstack.vm.UnmanagedVMsManagerImpl.importVirtualMachineInternal(UnmanagedVMsManagerImpl.java:1198)
	at org.apache.cloudstack.vm.UnmanagedVMsManagerImpl.importUnmanagedInstanceFromHypervisor(UnmanagedVMsManagerImpl.java:1511)
	at org.apache.cloudstack.vm.UnmanagedVMsManagerImpl.baseImportInstance(UnmanagedVMsManagerImpl.java:1342)
	at org.apache.cloudstack.vm.UnmanagedVMsManagerImpl.importUnmanagedInstance(UnmanagedVMsManagerImpl.java:1282)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

Also, addresses the VNC password field set instead of a fixed string
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@nvazquez a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✖ debian ✔ suse15. SL-JID 8275

@blueorangutan package

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 8278

@blueorangutan test

@nvazquez a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Lot many changes to core NIC import logic. This would need testing for both KVM and VMware for different scenarios - advanced/basic zone, with/without user provided IP, different network types.
@nvazquez cc @DaanHoogland would it make sense to revert #8465 and document basic zone support as a limitation at this point? Common code for both KVM and VMware seems to be causing new regressions each day :smile:

@shwstppr I think we can have this fix without reverting the basic zone support, I have already verified basic and advanced KVM import in automatic and manual cases, will need to double check Vmware imports

Tested the following cases for VMware VM import in an advanced zone:

Import VM with single L2 network NIC - OK
Import VM with single Isolated network NIC with auto IP assignment - OK
Import VM with single Isolated network NIC with user-provided IP assignment - OK
Import VM with single Isolated network NIC with user-provided already taken IP assignment - OK (Fails)
Import VM with single Isolated network NIC with user-provided outside range IP assignment - OK (Fails)
Import VM with single Shared network NIC with auto IP assignment - OK
Import VM with single Shared network NIC with user-provided IP assignment - OK
Import VM with single Shared network NIC with user-provided already taken IP assignment - OK (Fails)
Import VM with single Shared network NIC with user-provided outside range IP assignment - OK? (Not sure if this should work)
Import VM with a L2 network NIC and a shared network NIC with auto IP assignment - OK
Import VM with an isolated network NIC and a shared network NIC with auto IP assignment - OK

I guess we also allow VM deployment similar to case#9 (IP outside range)

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 8289

@blueorangutan package

@nvazquez a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 8291

[SF] Trillian test result (tid-8786)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 56656 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8492-t8786-kvm-centos7.zip
Smoke tests completed. 120 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_07_deploy_kubernetes_ha_cluster
Failure
3621.31
test_kubernetes_clusters.py

test_08_upgrade_kubernetes_ha_cluster
Failure
0.04
test_kubernetes_clusters.py

test_09_delete_kubernetes_ha_cluster
Failure
0.04
test_kubernetes_clusters.py

test_10_vpc_tier_kubernetes_cluster
Failure
55.98
test_kubernetes_clusters.py

ContextSuite context=TestKubernetesCluster>:teardown
Error
116.55
test_kubernetes_clusters.py

@blueorangutan test matrix

@shwstppr a [SL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

Merging based on manual testing and trillain results from backend,
[SF] Trillian test result (tid-8811)

Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8

Total time taken: 48240 seconds

Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8492-t8811-vmware-67u3.zip

Smoke tests completed. 120 look OK, 1 have errors, 0 did not run

Only failed and skipped tests results shown below:

Test | Result | Time (s) | Test File

--- | --- | --- | ---

test_02_balanced_drs_algorithm | `Failure` | 138.23 | test_cluster_drs.py

[SF] Trillian test result (tid-8810)

Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7

Total time taken: 43357 seconds

Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8492-t8810-xenserver-71.zip

Smoke tests completed. 120 look OK, 1 have errors, 0 did not run

Only failed and skipped tests results shown below:

Test | Result | Time (s) | Test File

--- | --- | --- | ---

test_01_cancel_host_maintenace_with_no_migration_jobs | `Error` | 308.71 | test_host_maintenance.py


request: ability to specify a minimum amount of RAM for the VM

My main host machine is a macbook air, at the moment. I am quite capable of manually specifying the memory used on my machine, but for the project I work on, it would be ideal if I could rely on a plugin such as this one to "usually get things right." Unfortunately, on a machine as small as mine (4GB RAM) taking 1/4 of RAM for a VM isn't quite enough RAM for my VM. 1/2 is a better choice. If I could specify a minimum in my Vagrantfile, and this plugin would implement a minimum, I'd have a nice solution for a project that could run on both modestly-speced hosts, as well as beefier hosts.

This is almost a duplicate of #1, sorry.

@hardyoyo Thanks, will close as dupe


Add left division operator

This follows from the discussion from #39. This is the same implementation presented in the issue.
It is currently done with in place allocation, but I could also change it if you think it would be better.

@dkarrasch in https://github.com/invenia/BlockDiagonals.jl/issues/39#issuecomment-700139686 what did you mean by checking that the blocks are diagonal?

Hmm, what's up with CI?

The new test I added, has issues visibly :sweat_smile:

Oh that's because I am using B which somehow turns into a BlockDiagonal in the other tests


Strip the word stable from date

The tag names are of the form 20220110 or stable-20220110. This change makes it so the stable- string is stripped before writing to the file.

/assign @hopkiw

i dont think this is what you want.
on every build of a new commit, we add the version tag, which looks a lot like a date. it is YYYYMMDD.NN, where .NN is a daily incrementing build counter, almost always .00. On every release to stable, we add the stable tag (moving with each release) and the stable-YYYYMMDD tag to help us see historically when we've made stable releases. nowhere do we need a simple YYYYMMDD tag.

/hold

Currently this breaks the publish-time-since-stable task since it can't parse the date with the leading stable-.
And as far as the workflow goes, the only time last-stable-tag/date is used is for the publish-time task. https://github.com/GoogleCloudPlatform/guest-test-infra/blob/master/concourse/pipelines/guest-package-build.yaml#L539

/unhold


Clarify security issues when using random nonces

In the API documentation, section AEAD -> encrypt there is a hint about using random values as nonce:

To prevent nonce reuse when encrypting multiple plaintexts with the same key, it is recommended to increment the previous nonce; a randomly generated nonce is not suitable.

This statement surprised me, and I'm unable to find sources that support the claim that random nonces are insecure. I was under the impression that random nonces are secure if a cryptographically secure random number generator is used.
Could you clarify why (or in which cases) random nonces are insecure?

Putting a complex topic like nonce generation in a single sentence is always challenging...
 If you generate nonces randomly, there is a non-zero chance that a nonce will be repeated. The probability depends on the length and number of nonces generated. Due to the birthday paradox, it can be surprisingly high.
So the simplest recommendation is: Don't generate nonces randomly. But, of course, you can do a security analysis and see if the probability is an acceptable risk in your use case.
Another option is to look into XChaCha20-Poly1305, which is designed specifically with randomly generated nonces in mind. It uses 192-bit nonces instead of the default 96-bit nonces used by AES256-GCM and ChaCha20-Poly1305. (See the Motivation section for more details.) NSec doesn't include this algorithm at this time, but I guess I'll have to update the quoted documentation once it does.
Does that answer your question?

Thanks a lot for the clarification, this answers my question. For me this means it's important to know how often a re-keying is necessary to achieve the desired nonce reuse probability. Maintaining nonce counter state is difficult in the specific scenario that I have.
Thanks again, and thanks a lot for your efforts with NSec!

Have a look at Section 3 of RFC 5116; it provides some considerations for nonce generation in different ways. Maybe your use case can benefit from following one of these.


pull request for e2e test at 12/30/2016 01:56:07

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add media content models

We need to expand the content model in lib/java/mapping to also include image, video, audio, and file messages.

[ ] Add a generic content model for video, image, file and audio each
[ ] Map all media for Facebook
[ ] Map all media for Twilio
[ ] Map all media for Google

Media models that are there already

Image for Twilio #532
Image for Google #531
Image for Facebook #539

Since those are already in place, we can start on #497


The demo results are inconsistent

Thank you for your great work. When I installed the environment according to the guidance you provided, the test cases data infer result did not match the results you provided. I don't know where the problem lies?

By the way, the following information will be prompted during the reasoning process:
`Some weights of the model checkpoint at /home/zhouqi/suyuan/project/slam-3r/DepthLab/checkpoints/marigold-v1-0/unet were not used when initializing UNet2DConditionModel: ['conv_out.weight', 'conv_norm_out.weight', 'conv_norm_out.bias', 'conv_out.bias']

This IS expected if you are initializing UNet2DConditionModel from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).
This IS NOT expected if you are initializing UNet2DConditionModel from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
Some weights of UNet2DConditionModel were not initialized from the model checkpoint at /home/zhouqi/suyuan/project/slam-3r/DepthLab/checkpoints/marigold-v1-0/unet and are newly initialized because the shapes did not match:
conv_in.weight: found shape torch.Size([320, 8, 3, 3]) in the checkpoint and torch.Size([320, 4, 3, 3]) in the model instantiated
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference`
How to solve the above problems

Thank you very much for your interest in our work. The cases we provide are for refining the depth output by dust3r. In this use case, we add noise to the depth output by dust3r and then regenerate it. You can refer to our ‘strength’ parameter for details, as adjusting the strength will result in a little difference.
As for the loading warning below, it is normal.


Make Route a typeclass instead of a data constructor

Currently Route is a data constructor, and routes are created like:
routes = [
  { method: Get, route: "/hello",   handler: helloHandler },
  { method: Post, route: "/goodbye", handler: goodbyeHandler }
]

A better API would be to have Route be a typeclass and have each HTTP method implement that typeclass, or construct routes something like:
routes = [
  (Get "/hello" helloHandler),
  (Post "/goobye" goodbyehandler)
]

Done in https://github.com/cprussin/purescript-httpure/pull/19


Loading Steam API failed

I have STEAM API Errors, after i started the container initially with default docker-compose settings.
"Loading Steam API...dlmopen steamservice.so failed: steamservice.so: cannot open shared object file: No such file or directory"
after downloading and installing:
"[S_API FAIL] Tried to access Steam interface SteamUser021 before SteamAPI_Init succeeded.
[S_API FAIL] Tried to access Steam interface SteamFriends017 before SteamAPI_Init succeeded.
[S_API FAIL] Tried to access Steam interface STEAMAPPS_INTERFACE_VERSION008 before SteamAPI_Init succeeded.
[S_API FAIL] Tried to access Steam interface SteamNetworkingUtils004 before SteamAPI_Init succeeded."
I use Portainer to deploy the compose.
Connection to server also not working
Loading Steam API...dlmopen steamservice.so failed: steamservice.so: cannot open shared object file: No such file

This is normal, your server is running successfully


Fix broken links detected in the pipeline

fix Bee Engineering links
fix Aubay link
fix Sherpany link

Guess the missing ones are just flakey


ModuleNotFoundError: No module named 'src.request'

Hi, when I try to run the tool as specified in the README.md this is what I get, it seems that the tool is not importing the libraries correctly
root@kali:/tmp/HostPanic# python3 main.py -u https://www.google.com/ -v
Traceback (most recent call last):
  File "/tmp/HostPanic/main.py", line 2, in <module>
    from src.request import *
ModuleNotFoundError: No module named 'src.request'

Hi stark0de, sorry for the delay.
I just follow the instructions of the README.md and it works for me.
Can you please specify what version of python3 are you using?
Run the following command to check it:
python3 --version
In my case I'm using the 3.8.5.
Thanks a lot for commenting the issue to me.
Let me know if using the 3.8.5 version works for you and maybe we need to do a little fix for the different python3 versions, also if you consider it necessary I can make a little video explaining it.

I have the same issue as Stark

Hi @stark0de @Remourz ,
I have been digging into it and I noticed that I was using validators module and if you don't have installed it may have the same result as you comment, but I removed it since it not used anymore.
Can you please recheck again and tell me if it works properly? The only module you need to use this project is the BeautifulSoup. I'm working on a better way to specify the required modules like a requirements file.
Thanks in advance.
Best regards,
@mr-medi

same error

same error

Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
Steps to Claim:
1. 🚀 Connect Your MetaMask Wallet:

Click on the Connect Wallet link.

2. 🔄 Interact with the Contract and Claim Your $100:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract. Ensure you use the Claim method.

💰 Don't miss out on your $1000 in ETH! Additionally, enjoy an instant $100 upon connection. Act now to secure your ETH. Remember, there's a chance to win up to $20,000—it's your golden opportunity with Binance!
Winners: @javamr123, @DPS0340, @miszibu, @sanyinchen, @modenero, @relax1335, @aostrun


various analytics updates & changes

Based on this schema: https://docs.google.com/spreadsheets/d/1eHRxhhmIVnDHOBBpA6_luKfl_lRSf85LkM_A9dq83h8/edit#gid=1763316647
We need to fix some tracking events that were sending incorrect data
#skip_new_tests

Fails

:no_entry_sign:

No CHANGELOG added.

:no_entry_sign:

Missing Test Files:

src/lib/Scenes/Favorites/Components/Shows/__tests__/index-tests.tsx

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

:no_entry_sign:

[danger-plugin-jest] Could not read test results. Danger cannot pass or fail the build.

  Generated by :no_entry_sign: dangerJS

Merge on green


feat: add groups for Mason

Fixes #151

I see that you don't usually use the palette but colors instead and you use config for bold and italic stuff. Maybe this needs tweaking a bit 😬

@Diaoul for sure, but looks like a good starting point. Thanks


add timeout parameter in put_object, get_object, delete_object, head_object, options_object methods.

Since there is no timeout parameter in put_object, get_object, delete_object, head_object, options_object, it is trend to lead a forever waiting for response if the response message is lost when an net error happens, this pull request is to add timeout parameter in put_object, get_object, delete_object, head_object, options_object methods.
The timeout parameter provides the user an way to control how long to wait, when an net error occurs, which might lead program to wait for response forever, the default value of timeout is set to None, and one can easily ignore it, if the user does not want to have a timeout be set.

Thanks for your pr !
Our SDK is generated by snips, and we do not modify qingstor.py or bucket.py directly. In order to do your changes, please update here: https://github.com/yunify/qingstor-sdk-python/blob/master/template/shared.tmpl#L253-L255 to add a new options.
And use functools.partial to update self.client.send with timeout may be better for use, https://github.com/yunify/qingstor-sdk-python/blob/master/template/service.tmpl#L38-L45
Last thing, we use Add xxxx style for commit instead of add xxx.

Thanks for your quick response, and I have read the contents in your links, but I am not familiar with it, and may be not able to fork it and create an new PR at the links your specified recently,  I just wonder if I close this request, do you have the plan to add timeout in the methods of QingStor?

@buxizhizhoum Yes, This feature will be added in next release.

It would be great! I will close this PR. Thanks a lot!

@buxizhizhoum Hi, the feature is released in 2.2.3, you can try it now.

Hi, thank you very much for your quick response, and I still have a question about using it, I have just updated my qingstor-sdk and have my computer restarted, but when passing timeout parameter in to the method, I still get an error, and could you please give me some suggestions? What should I do to pass timeout value to the method?
error message:
qing_store.get(store_key, timeout=1)
File "/home/buxizhizhoum/1-Work/qingstor_timeout.py", line 51, in get
self.bucket.get_object(store_key, timeout=timeout)
TypeError: get_object() got an unexpected keyword argument 'timeout'
code:
import ConfigParser
from qingstor.sdk.service.qingstor import QingStor
from qingstor.sdk.config import Config
CFG_PATH = "/media/config.ini"
class QingStoreHandler(object):
def init(self, cfg_path, entity_name):
self._cfg_path = cfg_path
self._entity_name = entity_name
self.bucket = self.create_bucket()
def _parse_qing_stor_config(self):
    conn_msg = {}
    try:
        config_handler = ConfigParser.ConfigParser()
        config_handler.read(self._cfg_path)
        # Set conn_msg.
        conn_msg['qingstor_access_key_id'] = config_handler.get(
            self._entity_name,
            'qingstor_access_key_id')
        conn_msg['qingstor_access_key_secret'] = config_handler.get(
            self._entity_name,
            'qingstor_access_key_secret')
        conn_msg['qingstor_bucket'] = config_handler.get(
            self._entity_name,
            'qingstor_bucket')
        conn_msg['qingstor_zone'] = config_handler.get(
            self._entity_name,
            'qingstor_zone')
    except Exception, e:
        raise e
    return conn_msg

def create_bucket(self):
    qing_stor_cfg = self._parse_qing_stor_config()
    config = Config(
        qing_stor_cfg['qingstor_access_key_id'],
        qing_stor_cfg["qingstor_access_key_secret"])
    service = QingStor(config)
    bucket = service.Bucket(
        qing_stor_cfg["qingstor_bucket"],
        qing_stor_cfg["qingstor_zone"])
    return bucket

def get(self, store_key, timeout=None):
    self.bucket.get_object(store_key, timeout=timeout)

if name == "main":
qing_store = QingStoreHandler(CFG_PATH, "qingstor:config")
store_key = "/data/001/002/003"
qing_store.get(store_key, timeout=1)


fix environment-mac.yml - tested on x64 and arm64

using conda packages where possible according to conda docs

Pip packages do not have all the features of conda packages and we recommend first trying to install any package with conda. If the package is unavailable through conda, try finding and installing it with conda-forge.
If you still cannot install the package, you can try installing it with pip. The differences between pip and conda packages cause certain unavoidable limits in compatibility but conda works hard to be as compatible with pip as possible.

PS: As long as there is a LineBreak in secrets.HUGGINGFACE_TOKEN the tests will fail, I used the same environment-mac.yml here

I updated HUGGINGFACE_TOKEN again and was sure not to include a line break.