File not openning

When I double click my file (it's a .json file, though not sure that matters) VSCode opens, but I get an 'untitled' document and not the file I double clicked.

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-09T00:26:45.394Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

Steps to Reproduce:

@dancvogel completely uninstalling and reinstalling VS Code will help you!

@dancvogel Does it work after restarting and/or resetting VS Code to be your default editor?

First, it may be worth noting that I was recently forced to update from a beta version to the release version. I'm not sure what version I had before, but one day I opened Code and was told I had to update.
Ok so, I tried uninstalling and reinstalling. It seems to have fixed the issue. However, then from the Azure Explorer I double clicked a file (Something I could do in the older version) and I once again got the 'Untitled' instead of the file I had opened. (Note: when I double click open a file from Azure Explorer it opens a temporary version of the file) Now every time after that even opening local files I get the 'Untitled' situation.
I've tried the uninstall/reinstall again and it doesn't help. Though during the reinstall it says the folder it's trying to install into already exists.

Interesting. Can you check the HKEY_CURRENT_USER\SOFTWARE\Classes\*\shell keys in your Registry?
It should look something like this:

Have you tried resetting VS Code as the editor for those given file types?

Can you use something like Process Hacker? I'm interesting in knowning the full command line arguments of the Code instance that was opened when you double clicked the file.

"C:\Program Files (x86)\Microsoft VS Code\Code.exe" --open-url "C:\xxx-xxx\main.cpp"

Awesome, that's exactly what I wanted to see.
@theultrazone Unfortunately you'll have to uninstall VS Code and reinstall it once again. Once reinstalled, re-configure the file associations with it. It will work.

@joaomoreno I've uninstalled/reinstalled again. This time making sure that the ProgramFiles folder for VS Code was deleted. Also, this time I did not select the "Register for supported file types" during install. Once up I right-clicked, selected "Open With" and reset Code as the editor for json files. I don't what of that actually helped but it looks to be working again. Thanks for the help.


[AutoRelease] t2-azurearcdata-2022-11-18-83474(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3389
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=2955

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


[KOGITO-3311] Tests: Update tests to reflect task endpoint change

Signed-off-by: Karel Suta<EMAIL_ADDRESS>https://issues.redhat.com/browse/KOGITO-3311
Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster

tested manually

/jenkins test

/jenkins test

Can be merged, it is fixing the task related tests.


Enable cross os caching

Pre-merge checklist
Performance:

[ ] Compare zig benchmark on linux before and after this pr.# benchmark results before
...

# benchmark results after
...

OR

[x] I am very sure this PR could not affect performance.

Context
https://github.com/actions/cache/blob/main/tips-and-workarounds.md#cross-os-cache
Seems update of this action broke our go builds.
bors r+

Cache not found for input keys: go-binaries-f3d96d93c2e8bd64073b536fb770aa77e348fc4d, go-binaries-f3d96d93c2e8bd64073b536fb770aa77e348fc4d

bors retry

Well now I don't see clients/go in the list at all.
bors cancel


current_date in docs is shown with braces

https://prestosql.io/docs/current/functions/datetime.html#current_date:

current_date()
Returns the current date as of the start of the query.

but in SQL this must be used without braces:
presto> SELECT current_date;
   _col0
------------
 2020-03-12

presto> SELECT current_date();
Query 20200312_100603_00004_kvt2r failed: line 1:20: mismatched input '('.

The page has this note (session is also the same), but many people might miss it unless scrolling.


cmake

Issue Type: Bug
doctor flutter says that i should install cmake altough it is already installed. What can i do?
VS Code version: Code 1.69.1 (b06ae3b2d2dbfe28bca3134cc6be65935cdfea6a, 2022-07-12T08:21:24.514Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

@Celeness can you please share more details about this issue, if it is related to VS Code
Please create your issue here https://github.com/microsoft/vscode/issues

Since we haven't heard back from you in a while, we assume the issue is resolved. Feel free to create a new ticket if you have any other issues.


Provide storage class options in cluster backup spec

Currently, we only allow GCE, AWS as input and set up storage class for users. This is simple and cloud based for most users, especially new ones.
For the future, if there are user requests, we will add storage class field in backup spec. This issue to track it.

We use glasterfs as persistence provider in Kubernetes. To make operator save backups to PV created by glusterfs we manually created storageClass named 'etcd-backup-gce-pd' that use glusterfs provider. Now we can use '--pv-provisioner=kubernetes.io/gce-pd' and PV is created in glusterfs cluster and operator works all right.

implemented by #1358. Thanks @jamiehannaford!


fix:fix the database conection

fix the database

[ ] connection and improve the security connection by SSL

no error there


patch: add support for --batch and --force options for prerequisites

--batch being an entirely new option altogether which we did not previously support! It seems from reading the man page of patch that --batch is used for more than just this, but this should serve as a good initial go at it.

Codecov Report
Base: 69.00% // Head: 69.16% // Increases project coverage by +0.16% :tada:

Coverage data is based on head (e91db51) compared to base (dcf480e).
Patch coverage: 71.42% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
+ Coverage   69.00%   69.16%   +0.16%
==========================================
  Files          17       17
  Lines        1910     1920      +10
  Branches      844      847       +3
==========================================
+ Hits         1318     1328      +10
  Misses        108      108
  Partials      484      484

Impacted Files
Coverage Δ

src/patch.cpp
55.72% <63.63%> (+0.98%)
:arrow_up:

src/cmdline.cpp
78.97% <100.00%> (+0.29%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


event: use enhanced testbarrier in stream_test.go

This test required the new behaviour where the testbarrier
never waits once the barrier has been lifted.

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

#105  👈
#104
#103
#102
#101
#100
#99
main

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

Join @jespert and the rest of your teammates on  Graphite


Please add a cmake or make build system for the newest branch

^

The one called "goodbye cmake"? :)

Please don't file bugs for development branches, they are broken by definition.

Okay,I'm sorry I just want to get my hands on the "new" Polycode and start making/adding stuff !


Seed files do not correctly display Danish characters æ, ø, å when exported to MSSQL

UTF-8 encoded csv files with æøå, are displayed as weird characters Ã¦, Ã¸ etc. in the database when using MSSQL.
I believe the issue is due to use of the pymssql's bulk_copy() method at: https://github.com/TobikoData/sqlmesh/blob/main/sqlmesh/core/engine_adapter/mssql.py#L180
How to reproduce
-- Assumes a table exist like:
CREATE TABLE [sqlmesh__portman].[test_bulk_copy](
       [transaction_type_key] [bigint] NULL,
       [transaction_code] [varchar](max) NULL,
       [transaction_type] [varchar](max) NULL,
       [alternative_transaction_subtype] [varchar](max) NULL,
       [investment_activity_type] [varchar](max) NULL
) ON [PRIMARY] TEXTIMAGE_ON [PRIMARY]

import pymssql

mssql_user = "user"
mssql_password = "password"
host = "<IP_ADDRESS>"
port = 1435

conn = pymssql.connect(
    host=host,
    port=port,
    user=mssql_user,
    password=mssql_password,
    database="datafoundations_test",
    charset="UTF-8"  # charset=None neither works
)

rows = [
    (19, 'RG', 'Værdiregulering', ' ', None),
    (21, '00', 'Køb af aktier', None, None)
]

conn.bulk_copy('sqlmesh__portman.test_bulk_copy', rows)
conn.commit()
conn.close()

A workaround
I can get bulk_copy() to work if charset="cp1252" or charset="ISO-8859-1" is set on the connection:
import pymssql

mssql_user = "user"
mssql_password = "password"
host = "<IP_ADDRESS>"
port = 1435

conn = pymssql.connect(
host=host,
port=port,
user=mssql_user,
password=mssql_password,
database="datafoundations_test",
charset="cp1252" or charset="ISO-8859-1",
)

rows = [
    (19, 'RG', 'Værdiregulering', ' ', None),
    (21, '00', 'Køb af aktier', None, None),
]

conn.bulk_copy('sqlmesh__portman.test_bulk_copy', rows)
conn.commit()
conn.close()

For charset="UTF-8" I haven't found a solution, this might be due to bulk_copy problems:
https://github.com/pymssql/pymssql/issues/725
Background
There seems to be multiple configuration options to consider:

charset arg for pymssql.connection()
column datatype
column collation

I'm not completely sure that they are all relevant, as I haven't found any solutions with UTF-8 charset.
Similarly, specifying either "cp1252" or charset="ISO-8859-1" works independently of the datatype and collation type.
Nevertheless the following concerning MSSQL datatype encoding and collation for MSSQL might be relevant:
I believe SQLmesh converts columns of text to varchar columns in MSSQL. Such does not support unicode unless with an UTF-8 enabled collation:
https://learn.microsoft.com/en-us/sql/relational-databases/collations/collation-and-unicode-support?view=sql-server-ver16#unicode-data-types
I have tested the charset="UTF-8" with all combinations of the following with no luck:

utf-8 enabled colation column and Danish_Norwegian_CI_AS
nvarchar and varchar column

Possible solutions (https://github.com/pymssql/pymssql/issues/725):

Using .execute() method instead (Seems to works independent of pymssql charset and table colation and varchar/nvarchar)
Uploading bytes and converting from there

We use the Danish_Norwegian_CI_AS on our server, so all new tables and columns will be created with non-unicode supporting varchar types by default.
So a solution that works independently of collation and with varchar might be optimal.

Hello - could you try specifying the charset key in your connection configuration to one of the working character sets? https://sqlmesh.readthedocs.io/en/stable/integrations/engines/mssql/

Oh wow, there's a config for that! :)
I had it hardcoded here in my local files for a temp workaround: https://github.com/TobikoData/sqlmesh/blob/main/sqlmesh/core/config/connection.py#L980
I can confirm that setting the charset = "cp1252" in the gateway config works as well.
So I guess there's actually not a bug, and this issue can be closed.
Although, initially I would never had known that to fix the issue I just had to specify a little thing in the config, took me quite some time to figure out what was going on.
So maybe some kind of warning or hint somewhere in the documentation for windows mssql users that specifying charset might be necessary?


Clarified setup steps

For dummies like me

This closed because I deleted the branch three. Only master is relevant, now.


feat: upgrade native sdk 4.2.2

Update native sdk 4.2.2 dependencies

This pull request is trigger by bot, you can checkout this branch and update it.

Examples built:

https://download.agora.io/demo/test/agora_rtc_engine_android_6.2.2-sp.422.a_20230807032713.zip
https://download.agora.io/demo/test/agora_rtc_engine_ios_6.2.2-sp.422.a_20230807033022.zip
https://download.agora.io/demo/test/agora_rtc_engine_macos_6.2.2-sp.422.a_20230807032252.zip
https://download.agora.io/demo/test/agora_rtc_engine_windows_6.2.2-sp.422.a_20230807032145.zip

This comment is commented by bot, do not edit it directly


python312Packages.nitrokey: 0.2.0 -> 0.2.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.nitrokey is: Python SDK for Nitrokey devices
meta.homepage for python312Packages.nitrokey is: https://github.com/Nitrokey/nitrokey-sdk-py
meta.changelog for python312Packages.nitrokey is: https://github.com/Nitrokey/nitrokey-sdk-py/releases/tag/v0.2.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.1 with grep in /nix/store/ry8hqdsj47iappgjf8dgn1751b6gj5sy-python3.12-nitrokey-0.2.1
found 0.2.1 in filename of file in /nix/store/ry8hqdsj47iappgjf8dgn1751b6gj5sy-python3.12-nitrokey-0.2.1

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

8 total rebuild path(s)

8 package rebuild(s)

First fifty rebuilds by attrpath

nitrokey-app2
nitrokey-fido2-firmware
pynitrokey
python311Packages.nitrokey
python311Packages.pynitrokey
python312Packages.nitrokey
python312Packages.pynitrokey

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/49ff1184089cca6a77c4a471988cd9aaabf3dd89#python312Packages.nitrokey

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ry8hqdsj47iappgjf8dgn1751b6gj5sy-python3.12-nitrokey-0.2.1
ls -la /nix/store/ry8hqdsj47iappgjf8dgn1751b6gj5sy-python3.12-nitrokey-0.2.1/bin

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

x86_64-linux

:white_check_mark: 11 packages built:

nitrokey-app2
nitrokey-app2.dist
nitrokey-fido2-firmware
pynitrokey (python312Packages.pynitrokey)
pynitrokey.dist (python312Packages.pynitrokey.dist)
python311Packages.nitrokey
python311Packages.nitrokey.dist
python311Packages.pynitrokey
python311Packages.pynitrokey.dist
python312Packages.nitrokey
python312Packages.nitrokey.dist

Maintainer pings
cc @panicgh for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


feature/refactor

Removing duplicate code from :
JedisShardInfo.java
BinaryClient.java

@OgmaJ tests are failing.

Please check tests and resubmit


feat: synchronise Readme from Snyk Docs [ROAD-775]

Description
Workflow that synchronises readme.md from Snyk Docs.
Checklist

[ ] Tests added and all succeed
[ ] Linted
[ ] CHANGELOG.md updated
[ ] README.md updated, if user-facing

@aldanchenko feel free to comment on some code lines with questions, happy to answer and improve the understanding. We should share more knowledge within the team 👍


dev superfluid WIP

Description
I'm adding more docs / code changes as I go through the delegate flow and what happens during it.
We need to change the code to allow:

1 superfluid delegation per (addr, denom) pair
many superfluid unbondings per (addr, denom) pair

Need to understand what happens on delegate, if the validator is:

jailed
unbonding
tombstoned

I suspect nothing can cause a chain halt, worst case the tx reverts? If so, then we can just cacheCtx wrap it

For contributor use:

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #894 (69c1d0b) into main (b990199) will increase coverage by 0.10%.
The diff coverage is 70.37%.

@@            Coverage Diff             @@
##             main     #894      +/-   ##
==========================================
+ Coverage   20.38%   20.49%   +0.10%
==========================================
  Files         192      192
  Lines       24783    24803      +20
==========================================
+ Hits         5053     5083      +30
+ Misses      18798    18791       -7
+ Partials      932      929       -3

Impacted Files
Coverage Δ

x/superfluid/keeper/distribution.go
85.71% <ø> (ø)

x/superfluid/keeper/hooks.go
56.25% <ø> (ø)

x/superfluid/keeper/stake.go
63.50% <57.89%> (+6.82%)
:arrow_up:

x/superfluid/keeper/slash.go
84.61% <100.00%> (+17.94%)
:arrow_up:

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 b990199...69c1d0b. Read the comment docs.

Going to merge this, as soon we will have far more substantial changes to synth locks & regular locks


DO NOT MERGE - M3 samples

Add new M3 samples:

[X] conversation-update-es6
[X] echobot-console-es6
[X] single-prompt-bot-es6
[X] multiple-prompts-bot-es6
[X] rich-cards-es6
[X] dispatch-es6
[ ] luis-bot-es6
[ ] qnamaker-bot-es6
[ ] browser-adapter-bot-es6
[ ] contoso-bot-es6
[ ] contoso-bot-v3-v4-es6
[ ] vsix/azure-echo-bot-es6  (bot that has commented out state management providers, e.g. Table storage, CosmosDB , Blob Storage)

Cleanup:

[ ] Remove preexisting samples

Pull Request Test Coverage Report for Build 302

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
257 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-11.5%) to 68.752%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-ai/src/luisRecognizer.ts
85
3.09%

libraries/botbuilder-ai/src/languageTranslator.ts
172
2.18%

Totals

Change from base Build 295:
-11.5%

Covered Lines:
1554

Relevant Lines:
2188

💛  - Coveralls

@CezaryMarcjan and @DeniseMak, can you take a look at the dispatch sample and let me know if you have any feedback regarding it?
Per the original email thread that outlines the samples to be added to the repo, I was planning on going back in and re-adding the QnAMaker KBID and Subscription Key for quick spin-up as QnAMaker does not have a "public" setting at this time.


Avoid a lot of NameError exceptions

Purpose
Greatly improves rendering time by avoiding the generation of NameError exception
Context
Rendering a view with 966 inputs (ok I admit it's a mess) allowed me to see that this portion of code is not optimized at all.
I used https://github.com/tmm1/stackprof on my view to understand via the flamegraph that ~ 70% of the time is dedicated to handle NameError exceptions.
Changes
Instead of trying to obtain a constant directly, it is verified that it exists first.
Results
development environment :
Flamegraph  :arrow_left: before and after :arrow_right: fix :

As you can see before the fix, a lot of time was spent in the SimpleForm::FormBuilder#attempt_maping.
In the development environment the view rendering speed up is ~=> X 5.0
production environment : ( edit: not reproduced afterwards )
The same view goes from :
Completed 200 OK in 2658ms (Views: 2637.2ms | ActiveRecord: 12.0ms | Allocations: 1240207)
to
Completed 200 OK in 1503ms (Views: 1482.9ms | ActiveRecord: 13.3ms | Allocations: 1244907)
In the production environment the view rendering speed up is ~=> (2637 / 1482) = X 1.78
And many thanks for your great work 😙

I tested it in development and production modes but generated flamgraphs come from development mode
Rails log give me these speed up

X 5.0 for development
X 1.78 for production

I use the default configuration for cache_discovery.
config/initializers/simple_form.rb
...
  # Cache SimpleForm inputs discovery
  # config.cache_discovery = !Rails.env.development?
...

In development cache_discovery is off
[3] pry(main)> Rails.env
=> "development"
[4] pry(main)> SimpleForm.cache_discovery
=> false

In production cache_discovery is on
irb(main):001:0> Rails.env
=> "production"
irb(main):002:0> SimpleForm.cache_discovery
=> true

I will generate 2 flamegraphs without and with the fix in production mode

I tested it in development and production modes but generated flamgraphs come from development mode
Rails log give me these speed up

X 5.0 for development
X 1.78 for production

I use the default configuration for cache_discovery.
config/initializers/simple_form.rb
...
  # Cache SimpleForm inputs discovery
  # config.cache_discovery = !Rails.env.development?
...

In development cache_discovery is off
[3] pry(main)> Rails.env
=> "development"
[4] pry(main)> SimpleForm.cache_discovery
=> false

In production cache_discovery is on
irb(main):001:0> Rails.env
=> "production"
irb(main):002:0> SimpleForm.cache_discovery
=> true

I will generate 2 flamegraphs without and with the fix in production mode

Hello,
In production mode with and without the patch of this PR:

the rendering durations are identical,
the generated flamgraphs are identical.

I can no longer reproduce my performance improvement in production with only this patch. Which seems logical compared to what you say. It must have been due to another correction of my view (Oops).
So this improvement is only valid for the development mode.

Thanks @kriom, I believe that in production that discovery cache option takes place and helps prevent the many NameError rescues you were seeing in development.
I hope to circle back on this for review in the next few days.

@carlosantoniodasilva I came across this issue while researching a similar simple_form rendering time issue and I found in our configuration, we had the default generated config that has the discovery cache option (that turns it off in non-development) commented out. Would a PR to change that so the option is not commented out in the configuration file be accepted?
I suspect many people are running simple_form in production with cache_discovery enabled and are not aware. The change I suggest above won't help them but at least new users won't experience that.
I was also surprised that I couldn't find any documentation for the discovery cache configuration setting. Maybe the impact isn't as dramatic in production but when I toggled it on in development, the admittedly complex form we are rendering took a bit less than 50% of the original time to render.

Thanks for your work on this one @kriom, and my apologies for not circling back on it back then. We got a similar change in https://github.com/heartcombo/simple_form/pull/1810, so I went ahead and merged that, credited both of you in the changelog.
@cymen I'm afraid that enabling that by default could have side-effects with code reloading in dev, i.e. if you make changes to input code it could remove the previous cached constant for example... this is off the top of my head, untested, but sounds like it would be possible. If you would like to give that a try and doesn't run into any issues, we could consider enabling that on by default in dev, but otherwise I think this should be fine as is. We could of course improve documentation on that config. (it should be on by default in production though.)
Thanks all!

I'm reverting the merged change, as it caused a regression. See #1824 for more info.
Happy to review something else that fixes the original problem without the regression, but for now we'll have to live with it.


Add support of root path  to ReadComposite Operation.

This aims to implements : #989.
This is based on #1520.

I added support in leshan-server-demo too (simple than expected), mainly because of #1537.

This is now integrated in master, commits :

4fa2c8004205dc9c96a25fb5e7a24ab738d3a19e
83dca9158171ffe5870248e688bc19f32200a4eb
a9aeb8988b3c6e724769ffc0d639d5aeb63cfb53
b56287c4e3e9019dbbb12907867946577e9c9c6a
09f87e757f317c5d37da2a5072350ca281370e86

This will be available in next release (2.0.0-M14).


[monitoring] Virt-operator and virt-controller readiness metrics being calculated inaccurately

/kind bug
What happened:
Virt-operator and virt-controller expose a Gauge metric called kubevirt_virt_operator_ready and kubevirt_virt_controller_ready. That metric should reflect the readiness of the app.
Currently this metric is not correct. As can be seen in the handler of the /healthz endpoint, the readyGauge doesn't been updated by the endpoint handler, thus making the metric inconsistent compared to the readiness probe status (The gauge value keeps 1 while the app is unhealthy and thus not ready). As a result of that, the following alerts are being affected:

LowReadyVirtControllersCount
NoReadyVirtController
LowReadyVirtOperatorsCount
NoReadyVirtOperator

What you expected to happen:
The Gauge metric should be set to 0 as soon as the app considered unhealthy, because then it's not ready for the leader election.
How to reproduce it (as minimally and precisely as possible):
Imitate connectivity issue between a virt-controller pod to the Kubernetes API, make the /healthz return unhealthy status and observe the kubevirt_virt_controller_ready
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version): latest
Kubernetes version (use kubectl version): 1.20, 1.21, 1.22
VM or VMI specifications: N/A
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/triage accepted

@enp0s3 This should still stay open right?

@sradco Hi, yes.

/remove-lifecycle rotten

if we know that this does not solve itself, let's add
/lifecycle frozen

/close
My understanding of kubevirt_virt_controller_ready and kubevirt_virt_operator_ready was incorrect. The aim of these metrics is to indicate whether the application is ready to be elected as a leader in the leader election process. It doesn't related to the readiness probes, thus it should not be coupled with the /healthz endpoint logic.


Stream could not be attached for user

2017-01-22 15:42:08.377 error Stream could not be attached for user %s: %s @today_freedom:matrix.org User has no attached timeline room. This is probably a bug.                                   Unhandled rejection TypeError: Cannot read property 'formatting' of undefined                                                         at _storage.get_profile_by_id.then (/root/matrix-appservice-twitter/src/twitter/Twitter.js:218:24)

Last minute bug caused by a change that will be fixed in #77. Just awaiting review of the PR and then this should be fixed.

This was presumed fixed so I'll close this.


Fix findUsersOnleave Function

Query was checking status field instead of state field to find active users.

Wow, those tests should have failed

Sometimes a wondering human eye performs better than automated tests :)


Running ConvertTo-DSC multiple times in a session causes issues

ConvertTo-DSC being run multiple times in the same PowerShell session causes unpredictable behavior with the file generation. Mostly commonly causing dynamically generated parameters to not appear.
I suspect this is due to the script scope variables not being reset between runs. A helper function should be created that "factory resets" these script scope variables that is called at the start of ConvertTo-DSC to see if this resolves the issue.

The non-static script scope variables need their default values assigned in a helper function that is called at the start of convertto-dsc
$script:BenchmarkRecommendations
$script:StaticCorrections
[string[]]$script:UsedResourceTitles
[int]$script:ServiceSection
[int]$script:UserSection
$script:DSCConfigurationParameters
[System.Collections.ArrayList]$script:ScaffoldingBlocks
https://github.com/techservicesillinois/SecOps-Powershell-CISDSC/blob/afa9966bbd4b051324e55450e268ca6a4d1937f3/src/CISDSCResourceGeneration/CISDSCResourceGeneration.psm1#L244


use recent static commit hash for iced dep

This is from a recent commit on the @hecrj /iced repo (Nov 19, 2020).

Yeah this looks good 👍


Update Docusaurus to v3

Docusaurus v3 has been released bringing some slight Markdown changes. Namely, curly braces may need escaping within markdown and <! -- --> comments are not supported

I have done some looking through the deployment and I believe everything is alright. If anyone can have a look through that would be appreciated :smile:

3.0 is out now. Looking at the changes between this pr and the actual release it should just be a matter of updating the packages.
The primary change is an update from MDX v2 to the newly released MDX 3
This sounds pretty major (i.e. v1 -> v2), but its actually just a few very small changes, hence why it was added at the very end. We don't inferface with MDX directly hence no changes are needed. It's only new features

Yep! Got the notification too. I marked it as draft when the RC came out. I’ll be looking to update in the coming days around Uni work. Someone poke me if i forget 😝

I had missed this initially but the patch for @docusaurus/core can also now be removed because loading typescript config files is now natively supported https://github.com/facebook/docusaurus/pull/9317

Okey, all patches are updated. Ready for reviewal


[Feature] Add tables about stacking bars

What problem does this feature solve?
I wonder if I can reproduce this drawing using echarts

What does the proposed API look like?
Give some methods or examples，Link!

see #19250 for solution
📌 please close issue if problem solved.

No, it is not quite the same, mine is stacked bar chart + X-axis for the table, but it seems that the space between the columns does not need to be so troublesome, difficult in how to integrate into this composite chart @helgasoft

This is not supported by default. You may use graphic.elements to draw lines and text.


error crashing companion

I am runing companion v2 build 2034 at the moment and this module makes companion crash when i enter the ip of the pc i want to control with pptrsc. I have both firewalls off and the RSC test client is working fine on both the pc i am trying to control.
Here is the screenshot of the message i get.
Any ideas on how to fix it please ?

I may add that the instance shows OK and if i set up a command it works ... ( Companion is usable between two error messages)

Ok I'll check it there's an update on this module might mist something

I tried both versions (before and after the recent update) it's still the same.

Ok tnx for that, that I know it's not the update but something else

Thank you for having a look .
And by the way thanks a lot for all this amazing works you're all doing with companion, it's fantastic !!

I could not recreate it from source, I'll check it with a build

Thank you for searching. The error shows as soon as i enter th ip of the pc with pptrsc, if i enter a diferent ip there is no error ... If it helps ...

I just tried from a mac running MacOs 10.13.6 and the module works fine ... It's just the windows version that have the issue ... ( i tried with same version : 2061 )

Just to say, i just tried with the latest build (2244) and i get the exact same error as before...
The updated "index.js" didn't fix it ...
Hope someone can find a solution.
Cheers and good luck to everyone in this troubled times !

This error is still occuring (from a user). Also while testing their setup with v.2.1.0 47823d0-2493 I had this error:

Any chance you could look at the ping code @haakonnessjoen @PerRoine
TIA


Add playbooks for ci-full-rhel

When running Ursula with ci-full-rhel started from ursula/test/setup, and found that we missed the playbook of ci-full-rhel so that setup is failed. So here I added the playbook of ci-full-rhel referred to ci-full-centos playbook.
Also I modified test/common to fit ci-full-rhel running.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test

@nirajdp76 Hi Niraj, Thanks for your comments, I updated the PR, please help review again.

@paulczar I deleted the un-needed files as your comments and updated "setup" script, please help review again.

ok to test

retest

retest

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test


[PLAT-8619] Capture userInfo from NSExceptions

Goal
Capture userInfo from Objective-C exceptions, both uncaught and those provided to notify().
NSExceptions raised by Apple's frameworks don't usually have a userInfo, but those raised in user or 3rd party code might.
Changeset
Overhauls BugsnagClient notify: methods to reduce code duplication, make implementation easier to follow, and capture userInfo from exceptions. Metadata is added in a structure that matches what is already included in crash events.
Updates BSG_KSCrashSentry_NSException.m to capture the userInfo, and BSG_KSCrashReport.c to include it in crash reports.
Updates -[BugsnagEvent initWithKSCrashReport:] to read exception info (including userInfo) from crash reports and include it in the nsexception metadata.
Testing
Amends barebones test scenarios to verify nserror and error metadata.

Metadata structure changes have been reverted to avoid breaking customers' existing filters.


Wallet signer fixes for registerNodeTx and claimTx: draft for discussion

Take a look at comments in wallet/chain/p/camino_visitor.go
Currently, in code there are not implemented backend funcs so it won't compile

We need to extend the signing functionality of the wallet in order to properly include existing alias signers in multisigAliasTx (when updating an existing definition) -> https://github.com/chain4travel/caminogo/pull/246


Change Request: General SDK Guidelines should require support for redirects

The General guidelines should be updated to require proper handling of 3xx Redirect type responses.
The Basics

Which design guideline is affected?

This should be stated in the General guidelines, to make it clear that all languages must conform.
It probably belongs in the "Azure Core" topic:
https://azure.github.io/azure-sdk/general_azurecore.html

Which target languages are affected?

All

Describe the change:

By default, the SDK should transparently follow 301 and 303 redirects.  The SDK may allow this behavior to be altered with configuration settings.
Reasoning
We should make clear how the SDKs should handle redirects.

@lmazuel cooked up this query that suggests that all the SDKs do currently handle redirect:
https://azure.grep.app/search?q=redirect&words=true&filter[path][0]=sdk/core/
So I think this issue is just to clearly document this.


Handle the issue with FNB's multiple elements on a single line which …

…throws 'RuntimeException: Failed to initialize DateTime for string:'

Hmm, not currently sure how to make this more clever regarding the various ways the OFXs are generated. Ideally, we need to conform to spec (we'd need to read up on it), and if an OFX doesn't follow spec, then it wouldn't be supported...

@jacques what are the differences here in the 2.00 format please? Does it actually warrant a second class? Also missing tests for this new class :/


Fix source clock not correctly getting started after a decoupled transition

Cause is pretty obvious...

Initially, this fix bothered me because I was worried that this lastSeekFailed flag could be wrong in some other scenario and prevent a start, too. But after some experimentation, I'm probably okay with it.
For the record, I tried to see if I can apply something like
diff --git a/osu.Framework/Timing/DecouplingFramedClock.cs b/osu.Framework/Timing/DecouplingFramedClock.cs
index e4d248f4b..e7d6304ed 100644
--- a/osu.Framework/Timing/DecouplingFramedClock.cs
+++ b/osu.Framework/Timing/DecouplingFramedClock.cs
@@ -62,10 +62,11 @@ public sealed class DecouplingFramedClock : ISourceChangeableClock, IAdjustableC
         private double? lastReferenceTime;

         /// <summary>
-        /// Whether the last <see cref="Seek"/> operation failed.
+        /// The last time instant given to <see cref="Seek"/> to which the source clock failed to seek.
         /// This denotes that we need to <see cref="Start"/> in decoupled mode (if possible).
         /// </summary>
-        private bool lastSeekFailed;
+        private double? lastFailedSeekPosition;
+        //private bool lastSeekFailed;

         /// <summary>
         /// This clock is used when we are decoupling from the source.
@@ -89,6 +90,12 @@ public void ProcessFrame()

             try
             {
+                if (lastFailedSeekPosition != null && lastFailedSeekPosition != Source.CurrentTime)
+                {
+                    // source moved, reset last failed seek flag.
+                    lastFailedSeekPosition = null;
+                }
+
                 // If the source is running, there is never a need for any decoupling logic.
                 if (Source.IsRunning)
                 {
@@ -125,7 +132,6 @@ public void ProcessFrame()
                 {
                     adjustableSourceClock.Seek(currentTime);
                     adjustableSourceClock.Start();
-                    lastSeekFailed = false;

                     // Don't use the source's time until next frame, as our decoupled time is likely more accurate
                     // (starting a clock, especially a TrackBass may have slight discrepancies).
@@ -164,7 +170,7 @@ public void Reset()
         {
             adjustableSourceClock.Reset();
             isRunning = false;
-            lastSeekFailed = false;
+            lastFailedSeekPosition = null;
             currentTime = 0;
         }

@@ -175,7 +181,7 @@ public void Start()

             // If the previous seek failed, avoid calling `Start` on the source clock.
             // Doing so would potentially cause it to start from an incorrect location (ie. 0 in the case where we are tracking negative time).
-            if (lastSeekFailed && AllowDecoupling)
+            if (lastFailedSeekPosition != null && AllowDecoupling)
             {
                 isRunning = true;
                 return;
@@ -193,7 +199,7 @@ public void Stop()

         public bool Seek(double position)
         {
-            lastSeekFailed = !adjustableSourceClock.Seek(position);
+            bool lastSeekFailed = !adjustableSourceClock.Seek(position);

             if (!lastSeekFailed)
             {
@@ -204,6 +210,8 @@ public bool Seek(double position)
             }
             else
             {
+                lastFailedSeekPosition = position;
+
                 if (!AllowDecoupling)
                     return false;

i.e. rather than track failed seeks as a binary, track the instant to which the seek has failed and clear it if the source clock has since moved. But I dunno, I'm not sure it's that much clearer / better than what you have.
As a total aside, can we rename the internal isRunning flag to something like shouldBeRunning or isRunningInternal or something? I had a total :exploding_head: moment when attempting to track isRunning and IsRunning and (re-)discovering that one is not just a front / backing field for the other.

As a total aside, can we rename the internal isRunning flag to something like shouldBeRunning or

Done


Is there a safe procedure for C++ interaction

Is there a safe way of wrapping frameworks using CException with C++? I realize C jumps subvert stack unwinding, causing undefined behaviour. Still, the project I'm working on will most certainly be wrapped in C++ by users.
What I'm interested in is if there's a pattern that ensures safe integration with C++. Ideally, there could be some conversion from CExceptions to C++ exceptions.
I imagine no C++ class instances would be allowed to instantiate or destruct themselves in a Try block. I also imagine it would be absolutely necessary to ensure no CExceptions escape upwards into C++ land, which means there would need to be very tightly scoped wrapping. I also imagine it would be illegal to leverage CException at the C++ level as a stand-in for C++ exceptions. Am I on the right track with this line of thought?

I apologize to anyone waiting on this question. This project almost never gets Issues posted and I didn't notice the notification when it did.
This is an excellent question, but I'm not sure I have a clean answer for you.
I think I would start with making sure I had CEXCEPTION_NO_CATCH_HANDLER specified. This will make it so that any uncaught exceptions are channeled somewhere. This function handler could then throw a C++ exception, call a callback, or do whatever interaction you're hoping for.
I agree with most of your points.
The last point isn't necessarily true. I know people have used CException as a replacement for C++ Exceptions and had their compiler disable C++ Exceptions completely, because CException is so much lower overhead. This is almost universally for embedded targets. You'd have to be incredibly careful doing this sort of thing, primarily because of the object creation rules mentioned above... but these types of applications are also the same applications that often require no dynamic memory, so all the objects are defined up-front. This makes it a decent match.


Crash on loading Native\AssetPackages

Crashes on loading Native\AssetPackages on Bannerlord 1.6.0.

+1
Would greatly appreciate an update to 1.6.0

Same, i am afraid to try and debug this myself :(

Doesn't look like there's been any movement here. @szszss is this program still being supported?

 Myself and several modders I know use it without any issues, and have never reported any issues with malware being detected.

@Areldir just download antivirus like Norton or MacAfee and you'll see the result

thats pretty rich considering I didn't add more dependencies or anything of the sort. Only ever changed cs files that were already present in the original repo. If you can't decide false positives for yourself, probably modding (or any advanced computer usage) is not for you.

@hunharibo yes I totally agree, I should stick with the original repo as I just tried the lib on version 1.5.8 beta and it's working as intended. so I guess all I needed is to revert back to a previous version that the lib actually support, and the antivirus won't give me alerts anymore, so basically just stick with the original repo.


Disable major agent update from v2 to v3

Reason:
Prevent update to a major version with FF
Description:
If an incoming update message with server agent version 3 and FF is enabled it has to skip updating
Checks:

[ ] TBD

Was implemented outside the agent


Allow users to ignore fs.watch "rename" events

Hi,
I have a log file that I want to watch and which triggers "rename" events every now and then, without being actually renamed, truncated or deleted. If I do a normal tail -f logfile, I get the updates as expected. When using node-tail, however, the internal fs.watch treats some of the updates as rename events rather than change events. This might be a platform issue (I'm using Mac OSX) or an issue with the process that creates the logfile (I have no idea) - but when I force node-tail to use fs.watchFile, the problem goes away.
Since rename events reset the interal position and have a 1 second reload-delay, I'd like there to be a way to ignore these rename events and treat them as normal change events. Because change handles logfiles that are suddenly a lot shorter (ie. when actually renaming a file and replacing it with an empty one) perfectly fine, I don't see any harm in adding an option that allows the user to override the default behevaiour.
What do you think?
I also updated the docs to outline how these options could be used, along with the other options that were already there but not yet documented.
Cheers,
Steffen

Old PR . Archive.


[ADD] osustech to list of schools

Add Ondo State University of Science and Technology to the list of schools in the codebase.
The school is a valid school with the school website:
https://www.osustech.edu.ng
here is the link to ptume application one of the school application portal
https://app.osustech.edu.ng/
and here is my official school mail<EMAIL_ADDRESS>

@willypelz Pull request merged. Thank you!


Extracting large svo files (25GB)  get stuck halfway

I am extracting a large svo files which has about 600k frames inside at 1280 x 720.
I used the code provided in here and was able to extract 50% of the svo content. After 50%, the progress got stuck and I do not know what is the reason and kind of stuck. Any suggestions to debug this? I do not think there is a limitation in my storage space or the file codec format so the issue should be something else.

You should try to display the code returned by the grab function.
You can print the number of the frame in the SVO and where it is currently and enable the verbose mode in the init parameters to see if there are any messages.


ubuntu 22.04 copy and paste is unusual

Windows Terminal version
1.12.10733.0
Windows build number
windows 11  22000.613
Other Software
ubuntu 22.04
Steps to reproduce
try to copy text by selecting text
Expected Behavior
Expected that the text is copied and can paste as usual from windows clipboard.

Actual Behavior
when pasted ,the text has a white background.

Huh. Any chance this might be relevant?
https://askubuntu.com/questions/1334205/pasted-text-in-gnome-terminal-in-21-04-is-always-highlighted

Thanks.
I do not like the new  bash "feature".
"set enable-bracketed-paste off" works for me.

Glad to hear this one wasn't us!


Add exclude_unset_none arg

Change Summary
Add exclude_unset_none arg to BaseModel.dict and BaseModel.json. This argument excludes all fields that were unset and whose value is None.
Related issue number
No issue, but resolves this feature request:
https://github.com/samuelcolvin/pydantic/discussions/2837
Checklist

[X] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Please review


Issues #304:

modify ISearchLoop.interrupt(), add new parameter 'avoidable' to qualify the strength of the interruption

Are lines 92 and 93 of LargeNeighborhoodSearch.java necessary?

sounds like "true" means "strong", which is in contradiction with the word "voidable", right?

Not correct: set voidable to false (ie, strong) set alive to false and stop the process. Moreover, canBeResumed() will then return false which is the expected behavior.


Fix ARP flow not installed issue in networkpolicy-only mode

This PR fixes #1572
arpResponderStaticFlow is changed to be installed only in IPv4 mode by #1518, since 'ARP uses broadcast, but IPv6 doesn't support broadcast' https://github.com/vmware-tanzu/antrea/pull/1272#discussion_r508149290.
However, IsIPv4Enabled() is determined by c.nodeConfig.PodIPv4CIDR != nil (pkg/agent/openflow/client.go#L827), and in pka/agent/agent.go, when networkpolicy-only mode is true, setting nodeConfig.PodIPv4CIDR is skipped:
if i.networkConfig.TrafficEncapMode.IsNetworkPolicyOnly() {
		return nil
	}

	// Parse all PodCIDRs first, so that we could support IPv4/IPv6 dual-stack configurations.
	if node.Spec.PodCIDRs != nil {
		for _, podCIDR := range node.Spec.PodCIDRs {
			_, localSubnet, err := net.ParseCIDR(podCIDR)
			if err != nil {
				klog.Errorf("Failed to parse subnet from CIDR string %s: %v", node.Spec.PodCIDR, err)
				return err
			}
			if localSubnet.IP.To4() != nil {
				if i.nodeConfig.PodIPv4CIDR != nil {
					klog.Warningf("One IPv4 PodCIDR is already configured on this Node, ignore the IPv4 Subnet CIDR %s", localSubnet.String())
				}

As a result, the arpResponderStaticFlow is never installed for IPv4 networkpolicy-only mode, which causes pods to not able to communicate to apiserver:
 kubectl exec -it antrea-agent-vg6rf -n kube-system -c antrea-ovs -- ovs-ofctl dump-flows br-int table=20
 cookie=0x1000000000000, duration=2756.860s, table=20, n_packets=0, n_bytes=0, priority=190,arp actions=NORMAL
 cookie=0x1000000000000, duration=2756.861s, table=20, n_packets=0, n_bytes=0, priority=0 actions=drop

sonobuoy logs
namespace="sonobuoy" pod="sonobuoy" container="kube-sonobuoy"
time="2020-11-18T22:04:29Z" level=info msg="Scanning plugins in ./plugins.d (pwd: /)"
time="2020-11-18T22:04:29Z" level=info msg="Scanning plugins in /etc/sonobuoy/plugins.d (pwd: /)"
time="2020-11-18T22:04:29Z" level=info msg="Directory (/etc/sonobuoy/plugins.d) does not exist"
time="2020-11-18T22:04:29Z" level=info msg="Scanning plugins in ~/sonobuoy/plugins.d (pwd: /)"
time="2020-11-18T22:04:29Z" level=info msg="Directory (~/sonobuoy/plugins.d) does not exist"
time="2020-11-18T22:04:59Z" level=error msg="could not get api group resources: Get \"https://<IP_ADDRESS>:443/api?timeout=32s\": dial tcp <IP_ADDRESS>:443: i/o timeout"
time="2020-11-18T22:04:59Z" level=info msg="no-exit was specified, sonobuoy is now blocking"

This PR fixes this issue by always installing the arpResponderStaticFlow. In IPv6 cases, this flow will simply not get hit.

Codecov Report

Merging #1575 (0a87501) into master (9d3d10b) will decrease coverage by 22.64%.
The diff coverage is 10.53%.

@@             Coverage Diff             @@
##           master    #1575       +/-   ##
===========================================
- Coverage   63.31%   40.67%   -22.65%
===========================================
  Files         170      106       -64
  Lines       14250    12746     -1504
===========================================
- Hits         9023     5184     -3839
- Misses       4292     7093     +2801
+ Partials      935      469      -466

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
40.67% <10.53%> (-0.61%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/antrea-agent/agent.go
0.00% <0.00%> (ø)

cmd/antrea-agent/options.go
21.69% <0.00%> (+0.97%)
:arrow_up:

pkg/agent/agent.go
12.05% <0.00%> (-36.67%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <ø> (-100.00%)
:arrow_down:

.../agent/flowexporter/connections/conntrack_linux.go
28.37% <0.00%> (-0.79%)
:arrow_down:

pkg/agent/openflow/pipeline.go
28.17% <0.00%> (-41.61%)
:arrow_down:

pkg/agent/stats/collector.go
51.13% <ø> (-46.60%)
:arrow_down:

pkg/apiserver/certificate/cacert_controller.go
10.81% <0.00%> (-45.58%)
:arrow_down:

pkg/controller/networkpolicy/mutate.go
0.00% <0.00%> (ø)

... and 153 more

/test-all

/test-e2e


UtestShell::setCrashOnFail() does not work for Deallocating non-allocated memory failures

If I call UtestShell::setCrashOnFail() and then force a 'Deallocating non-allocated memory' failure in my test thus:
char *pBlock = (char*)malloc(10);
free(pBlock+1);

I do not get the expected crash; instead I just get a warning.
Debugging this, the problem seems to come from the use of TestTerminatorWithoutExceptions() in the MemoryLeakWarningReporter class.
The fix I have found is to change it to:
class MemoryLeakWarningReporter: public MemoryLeakFailure
{
public:
    virtual ~MemoryLeakWarningReporter() _destructor_override
    {
    }

    virtual void fail(char* fail_string) _override
    {
        UtestShell* currentTest = UtestShell::getCurrent();
        currentTest->failWith(FailFailure(currentTest, currentTest->getName().asCharString(), currentTest->getLineNumber(), fail_string), UtestShell::getCurrentTestTerminator());
    } // LCOV_EXCL_LINE
};

I'm happy to pick this up but I think that it first needs PR #1430 to be delivered, which has been stalled for the last 2 years. Calling UtestShell::getCurrentTestTerminator() will return an appropriate terminator, but which may use exceptions. While this works for you, I assume that it may not work in all environments.
@basvodde Any chance that we could get #1430 completed so that I can pick this up?


IndexError: list index out of range ERROR:root:ERROR: 500: {'exc_info': ...

I have a bunch of experiments (18 to be exact) running on various machines connected to one visdom server.  About this time, the server machine stops accepting ssh connections.
When I restart the server machine and the visdom server, I get this error on the server when I navigate to the visdom server IP:
INFO:tornado.access:101 GET /vis_socket (<MY IP>) 0.42ms
INFO:root:Opened visdom socket from ip: <MY IP>
INFO:tornado.access:101 GET /socket (<MY IP>) 0.56ms
INFO:root:Opened new socket from ip: <MY IP>
INFO:root:comparing envs
ERROR:tornado.application:Uncaught exception POST /compare/DIM_rkhs_1024_dimz_192_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_192_pen_0.1_do_pt041+DIM_rkhs_1024_dimz_512_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_512_pen_0.1_do_pt041+DIM_rkhs_1024_dimz_64_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_64_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_192_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_192_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_512_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_512_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_64_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_64_pen_0.1_do_pt041+DIM_rkhs_512_dimz_192_pen_0.0_do_pt041+DIM_rkhs_512_dimz_192_pen_0.1_do_pt041+DIM_rkhs_512_dimz_512_pen_0.0_do_pt041+DIM_rkhs_512_dimz_512_pen_0.1_do_pt041+DIM_rkhs_512_dimz_64_pen_0.0_do_pt041+DIM_rkhs_512_dimz_64_pen_0.1_do_pt041 (<MY IP>)
HTTPServerRequest(protocol='http', host='<VIZ_IP>', method='POST', uri='/compare/DIM_rkhs_1024_dimz_192_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_192_pen_0.1_do_pt041+DIM_rkhs_1024_dimz_512_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_512_pen_0.1_do_pt041+DIM_rkhs_1024_dimz_64_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_64_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_192_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_192_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_512_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_512_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_64_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_64_pen_0.1_do_pt041+DIM_rkhs_512_dimz_192_pen_0.0_do_pt041+DIM_rkhs_512_dimz_192_pen_0.1_do_pt041+DIM_rkhs_512_dimz_512_pen_0.0_do_pt041+DIM_rkhs_512_dimz_512_pen_0.1_do_pt041+DIM_rkhs_512_dimz_64_pen_0.0_do_pt041+DIM_rkhs_512_dimz_64_pen_0.1_do_pt041', version='HTTP/1.1', remote_ip='<MY IP>')
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/tornado/web.py", line 1590, in _execute
    result = method(*self.path_args, **self.path_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/visdom/server.py", line 51, in _check_auth
    f(self, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/visdom/server.py", line 1001, in post
    self.env_path)
  File "/usr/local/lib/python2.7/dist-packages/visdom/server.py", line 835, in compare_envs
    res = copy.deepcopy(envs[list(envs.keys())[0]])
IndexError: list index out of range
ERROR:root:ERROR: 500: {'exc_info': (<type 'exceptions.IndexError'>, IndexError('list index out of range',), <traceback object at 0x7f2d51c9e950>)}

What is going on here?

I have a bunch of experiments (18 to be exact) running on various machines connected to one visdom server.  About this time, the server machine stops accepting ssh connections.
When I restart the server machine and the visdom server, I get this error on the server when I navigate to the visdom server IP:
INFO:tornado.access:101 GET /vis_socket (<MY IP>) 0.42ms
INFO:root:Opened visdom socket from ip: <MY IP>
INFO:tornado.access:101 GET /socket (<MY IP>) 0.56ms
INFO:root:Opened new socket from ip: <MY IP>
INFO:root:comparing envs
ERROR:tornado.application:Uncaught exception POST /compare/DIM_rkhs_1024_dimz_192_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_192_pen_0.1_do_pt041+DIM_rkhs_1024_dimz_512_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_512_pen_0.1_do_pt041+DIM_rkhs_1024_dimz_64_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_64_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_192_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_192_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_512_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_512_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_64_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_64_pen_0.1_do_pt041+DIM_rkhs_512_dimz_192_pen_0.0_do_pt041+DIM_rkhs_512_dimz_192_pen_0.1_do_pt041+DIM_rkhs_512_dimz_512_pen_0.0_do_pt041+DIM_rkhs_512_dimz_512_pen_0.1_do_pt041+DIM_rkhs_512_dimz_64_pen_0.0_do_pt041+DIM_rkhs_512_dimz_64_pen_0.1_do_pt041 (<MY IP>)
HTTPServerRequest(protocol='http', host='<VIZ_IP>', method='POST', uri='/compare/DIM_rkhs_1024_dimz_192_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_192_pen_0.1_do_pt041+DIM_rkhs_1024_dimz_512_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_512_pen_0.1_do_pt041+DIM_rkhs_1024_dimz_64_pen_0.0_do_pt041+DIM_rkhs_1024_dimz_64_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_192_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_192_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_512_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_512_pen_0.1_do_pt041+DIM_rkhs_2048_dimz_64_pen_0.0_do_pt041+DIM_rkhs_2048_dimz_64_pen_0.1_do_pt041+DIM_rkhs_512_dimz_192_pen_0.0_do_pt041+DIM_rkhs_512_dimz_192_pen_0.1_do_pt041+DIM_rkhs_512_dimz_512_pen_0.0_do_pt041+DIM_rkhs_512_dimz_512_pen_0.1_do_pt041+DIM_rkhs_512_dimz_64_pen_0.0_do_pt041+DIM_rkhs_512_dimz_64_pen_0.1_do_pt041', version='HTTP/1.1', remote_ip='<MY IP>')
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/tornado/web.py", line 1590, in _execute
    result = method(*self.path_args, **self.path_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/visdom/server.py", line 51, in _check_auth
    f(self, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/visdom/server.py", line 1001, in post
    self.env_path)
  File "/usr/local/lib/python2.7/dist-packages/visdom/server.py", line 835, in compare_envs
    res = copy.deepcopy(envs[list(envs.keys())[0]])
IndexError: list index out of range
ERROR:root:ERROR: 500: {'exc_info': (<type 'exceptions.IndexError'>, IndexError('list index out of range',), <traceback object at 0x7f2d51c9e950>)}

What is going on here?

I'm not sure quite why but for some reason there are no envs in envs at this moment here: https://github.com/facebookresearch/visdom/blob/e34c6d5abb23ed1e3333515af714c70b570e86c1/py/visdom/server.py#L913. Are all of those experiments existent at the moment? It'd be useful to check what's in envs, eids, and state right before that line.

Yeh, I'm not sure. What is confusing is this is after restarting the server, even if the jobs have stopped. Which means there some files somewhere that visdom is reading that are broken.

Are you able to hit the home page of the server? And if so do those envs appear in the list in the main bar?

So i can visit the page, but only the "main" env appears. On the server side, that error I posted appears.
Note also that after restarting the jobs and server, things work fine after a few steps, so whatever was broken get overwritten. However, if I try to view too many envs at once, things crash with the same issues.
Temp solution right now is just not to view too many envs at once.

The index error hits on the server side from landing on the home page? That would imply a different callsite runs into the same problem, which I wasn't expecting for the home page.
Definitely agreed that it might be an issue with comparing too many envs, seems like the right area for me to investigate when I get a chance to go into this one.


[ENH] Add new event  to 'yii\db\Command' class

I think it will be useful for different situations.
class Command extends Component
{

const EVENT_AFTER_EXECUTE = 'afterExecute';

public function execute()
    {
        $sql = $this->getSql();
        list($profile, $rawSql) = $this->logQuery(__METHOD__);

        if ($sql == '') {
            return 0;
        }

        $this->prepare(false);

        try {
            $profile and Yii::beginProfile($rawSql, __METHOD__);

            $this->pdoStatement->execute();
            $n = $this->pdoStatement->rowCount();

            $profile and Yii::endProfile($rawSql, __METHOD__);

	    $this->trigger(self::EVENT_AFTER_EXECUTE, new AfterExecuteEvent([
		'command' => $this,
	    ]));

            $this->refreshTableSchema();

            return $n;
        } catch (\Exception $e) {
            $profile and Yii::endProfile($rawSql, __METHOD__);
            throw $this->db->getSchema()->convertException($e, $rawSql ?: $this->getRawSql());
        }
    }

}

As for me I use this event for automatic generation raw .sql files with migrations.

Was reverted as wontfix.


support to ok.ru

Hi,
please, can you help me with example from ok.ru URL.
your app is awesome

The only way that you could make this work using an ok.ru URL is to add a proxy that is both launched on the streamlink side, and on the end user's side.
Or that you launch streamlink locally, on your computer.
There's not much I can do sadly.


Update module from antrea.io/antrea to antrea.io/antrea/v2

Describe the bug
The antrea module in go.mod should be updated from antrea.io/antrea/v2 since now we are on v2.x
Additional context
If we try to require this in some other repository then it will generate error because go.mod will not parse until the module is antrea.io/antrea/v2.

cc: @antoninbas @tnqn

That's a real issue, but I think there is a deeper issue that should be addressed at the same time: antrea.io/antrea is not really meant to be imported as a module (like k8s.io/kubernetes is not meant to be imported for example). Only a small part of antrea.io/antrea is meant to be imported: APIs and generated clientsets. By forcing consumer projects of the Antrea APIs to import all of antrea.io/antrea, we can create issues in dependency management for these other projects.
In my view, there are 2 solutions that are worth considering:

move Antrea APIs and generated clients to a separate repository / module (e.g., antrea-io/apis), for which we would not have any semantic versioning. Calico has taken that approach.
leverage workspace support introduced in Go 1.18. K8s currently uses a Go workspace: https://github.com/kubernetes/kubernetes/blob/master/go.work

A few characteristics of each approach:
Approach 1:

when making an update to an API, antrea-io/apis needs to be updated first, separately from antrea-io/antrea. For minor changes / fixes or for chores (code generation version updates), this may be a bit of a nuisance. For major changes such as the introduction of a new API, we anyway recommend opening separate PRs even today (one for the API change, one for the implementation)
we don't actually need to update the package name (and all imports) from antrea-io/antrea to antrea-io/antrea/v2

Approach 2:

everything stays in the same repo, so we keep the contribution model the same
we still need to update the package name (and all imports) from antrea-io/antrea to antrea-io/antrea/v2
APIs need to be imported as antrea-io/antrea/v2/apis, assuming we use apis as the module for APIs inside the workspace

In both cases, some code reorganization is necessary. I don't have a strong preference at the moment. Approach 1 is pretty straightforward, while I am not as familiar with Approach 2 and what it entails. Probably something worth discussing at the next community meeting.

@antoninbas Approach 1 seems good because only apis are used by consumers, but in general approach2 seems to be more uniform and maintainable in long run, because we need not maintain separate versions and this makes the code(test code as well) more maintainable.

After reviewing the 2 solutions above, I have concluded that none of them are really satisfactory:

Approach 1 is superior IMO, but does require extra repos (at least one for APIs, and potentially one for the generated clientset?) and will complicate the development process.
Approach 2 introduces more complexity, and while workspaces can be used to build "monorepos", that doesn't seem to be the primary use case (it is actually recommended for go.work not to be checked in). I am sure this is a good solution for a project like K8s, but doesn't seem appropriate at our scale.

Therefore, I'd like to suggest the following 2-step approach:

Keep the status quo for projects importing antrea-io/antrea, and update the module name to antrea.io/antrea/v2 to match our major version number
In order to streamline imports, consider introducing new "sub-modules" as part of the same repo (https://github.com/antrea-io/antrea) that can be imported by downstream projects in a convenient fashion. For example, we could have antrea.io/antrea/apis for APIs, and antrea.io/antrea/client for generated clientsets.

Step 1 does require modifying most Go files in this repo (as well as a few others) to update imports from antrea.io/antrea/... to antrea.io/antrea/v2/.... For Go files, a tool such as https://github.com/marwan-at-work/mod works pretty well. Here is the size of the diff:
 906 files changed, 3078 insertions(+), 3078 deletions(-)

Step 2 does also have some caveats:

Necessary to generate separate additional tags for each Antrea release (see https://go.dev/doc/modules/managing-source#multiple-module-source)
Need to manage replace directives in go.mod files (in particular, in the "top-level" go.mod file, for the antrea.io/antrea module)

The advantages of step 2 still exist, but may not be as important as I initially thought / presented. For example, I wrote above:

we can create issues in dependency management for these other projects

While this is somewhat true, and fewer dependencies is always better, especially for an "API" module, dependency pruning (introduced in go 1.17) means that not all Antrea dependencies are added to downstream projects. This is the go.mod I get when I create a module that import antrea.io/antrea, but only consumes APIs:
go 1.23.0

require antrea.io/antrea v1.15.2

require (
	github.com/go-logr/logr v1.4.1 // indirect
	github.com/gogo/protobuf v1.3.2 // indirect
	github.com/google/gofuzz v1.2.0 // indirect
	github.com/json-iterator/go v1.1.12 // indirect
	github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd // indirect
	github.com/modern-go/reflect2 v1.0.2 // indirect
	golang.org/x/net v0.20.0 // indirect
	golang.org/x/text v0.14.0 // indirect
	gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c // indirect
	gopkg.in/inf.v0 v0.9.1 // indirect
	gopkg.in/yaml.v2 v2.4.0 // indirect
	k8s.io/api v0.26.4 // indirect
	k8s.io/apimachinery v0.26.4 // indirect
	k8s.io/klog/v2 v2.100.1 // indirect
	k8s.io/utils v0.0.0-20230209194617-a36077c30491 // indirect
	sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd // indirect
	sigs.k8s.io/structured-merge-diff/v4 v4.2.3 // indirect
)

These dependencies would presumably stay the same if we were to move Antrea APIs to a separate module.
So in practice, the advantages of step 2 would be:

Better code organization and separation of APIs, with a clear picture of the API-specific dependencies
The ability to set a lower go version directive for the API module compared to the main Antrea module, hence enabling import by projects using an older go version

See https://github.com/vmware-tanzu/vm-operator for an example of a project which uses a separate module for APIs (but in the same source repository).


Language server is not supported in web environments

Since we are using a bundled JAR for the language server, it can only run in desktop environments that have Java installed. It would be great to get a version that works with web-assembly so it can run in the browser.

Using WebAssembly for Extension Development


[macOS] Update window visible state on deminiaturize.

Seems like windowDidChangeOcclusionState sometime arrive too early and window stay with is_visible = false (rendering disabled) after restoring it from minimized state.
Fixes https://github.com/godotengine/godot/issues/87309

Thanks!


Bumped Gradle to 4.9

Done via https://github.com/spring-cloud/spring-cloud-pipelines/commit/6dc0dede1874f2868b601a02eca6a8476b818832


Fix inittialization order issue.

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

Why are you removing examples? If it's because of check_spelling, then you can exclude examples/wasm/envoy_filter_http_wasm_example.wat by adding it to tools/spelling_skip_files.txt.

re-added those files and added them to tools/spelling_skip_files.txt


Identify Resources that Require CredSSP in the Documentation

For future consideration, just like we now identify the category of resource (e.g. Distributed, Utility, etc.), we could identify right from the main list of resources which one require CredSSP. I know for sure that will help many of my customers.

Great idea! Lets add that once we have some time after the release of v3.0 :-)


Add keyboard shortcuts for calendar navigation: Browser Extension

The idea that I have so far for the commands:

Command
Purpose

Open calendar extension
Ctrl+p (p for patro)

Show today's date
Ctrl+t (t for today)

Go to previous month
Ctrl+h

Go to next month
Ctrl+l

Go to previous day
Ctrl+j

Go to next day
Ctrl+k

The h,j,k,l are inspired by vim bindings.


configuration

Hi,
thanks for the code, it helped me a lot. My problem is, with your configuration file I cannot detect objects with a distance of more than about 3 meters. With the default settings in the mmwave demo visualizer, I can detect the object but when I copy this settings in the config-file for the python-programm, its not running. When I increase just the ramptime (and decrease the slope), I can detect the object but then I detect non-existing objects in front of the sensor too.
It would be nice if you can help me, Im new to the topic.

Hi,
Yes, the configuration is defined for short range. You should be able to use the same configuration file as in the demo visualizer. Just to make sure, which mmWaveSDK are you using? One possible issue could be this the program only works for the mmWave SDK 2 (I have other repositories for the other SDK versions), but since you are able to run with my configuration I guess that you are using the SDK2.
Did you also modify the range of the plot? Another option is that the objects are outside the range of the plot so that is why nothing is being shown.
p.setXRange(-0.5,0.5)
p.setYRange(0,1.5)

As for the noise near the sensor, do you get the same results in the demo visualizer? In that case, you might want to increase the CFAR threshold, or simply add some lines of codes to my program to remove the points that are too close to the radar.

Thanks for the respond,
yes i am using the 2.1 SDK. Its's not running if I copy the value for ,  (ProfileCfg) but that shouldn't make a diffrence, right? Also, i cannot copy the value for CQRxSatMonitor and CQSigImgMonitor. I dont understand what these values are good for but it doesnt sound like its effecting the range. I already modified the plot. In the demo visualizer the object in front of the sensor is appearing and disappearing all the time but in the python programm i have always to objects about 10 cm in front of the sensor.
I added code to ignore the first 10 cm but it would be nice if I could oberve this area. The CFAR value didnt really help, since I either loose the object or get some new ones. I guess there is no other possibilty to not detect the objects in front of the sensor?
Thanks for your help.

just saw it deleted something in my text: Its's not running if I copy the value for numAdcSamples and digOutSampleRate, (ProfileCfg) but that shouldn't make a diffrence, right?

I cannot check the about the configuration, I will try to answer more in detail later. For the chirp configuration, you should probably ask in the Texas Instruments' forum, probably they can help you better than I do (I am not sure if it is the same but maybe this can help you https://e2e.ti.com/support/sensors/f/1023/t/810631).
As for the objects in front of the radar, I will have to look at it, but it should be the same results as in the demo visualizer, the code just parses the radar data and extracts the position of the points. One possible problem is that the program uses the values in the configuration to get the X, Y, Z of the points. So, if the configuration is not parsed correctly or in case the configuration is complex (I am not sure if there are any other ways to extract the 3D position in case of different configurations) there might be problem extracting the 3D position of the point cloud.
If you have the AWR1642 ES2.0, then I would recommend you to test the mmWave SDK 3, in that version the X,Y, Z is extracted directly from the radar raw data without using the values in the configuration.

Hi @FlorianMaier123,
were you able to get the configuration file to work correctly?

I stoped trying because I decided to get the AWR1843 (i want the z coordinate). When it arrives I will use your python script for it and see if it works. Im wondering if I can use it with the subframe configuration example I found in the doxygen. Im not sure if its possible to read each subframe or if I just get the detected objects for the whole frame.
Thanks for asking.

I have no experience with the advanced subframe configuration but even with the AWR1843 you might have to modify the code slightly. The program reads the subframe number, so then you can use that value to separate the data from the different subframes I guess. If you have any question with that please let me know and I will try to help you.
Ibai


Add dynspread transfer function

dynspread dynamically spreads pixels in an image based on a density heuristic. This is useful for maintaining visibility of points when zooming in.
The heuristic is defined as the normalized mean number of non-empty adjacent pixels for each non-empty pixel. This gives a simple to compute tuning parameter in [0, 1].

This looks great to me; just what I was imagining!  No method like this will ever be perfect, but it seems to work well in both sparse and dense regions.  I'll merge it now and we can tweak it over time.  Thanks!


Modify Controller

I want to add @setting, how to change the controller?

Where do you get settings from, and how do you normally assign it? You can set it in a plug in your router pipeline, calling Plug.Conn.assign/3, but depends what @setting is.

setting are obtained this way

In this case I would load it in a custom plug:
defmodule MyAppWeb.LoadSettingPlug do
  def init(config), do: config

  def call(conn, config) do
    setting = Adjustments.get_setting_first!()

    Plug.Conn.assign(conn, :setting, setting)
  end
end

Then you can add it to your browser pipeline in your router:
defmodule MyAppWeb.Router do
  use MyAppWeb, :router

  pipeline :browser do
    plug :accepts, ["html"]
    plug :fetch_session
    plug :fetch_flash
    plug Phoenix.LiveView.Flash
    plug :protect_from_forgery
    plug :put_secure_browser_headers
    plug MyAppWeb.LoadSettingPlug
  end

  # ...
end

Now setting will be loaded for all pages.


OSError: Error no file named config.json found in directory ./pretrained_model/stable-diffusion-v1-5.

Hello,
I have installed as mentioned in Readme.
I have not made any changes in the yaml file and trying to run using the default config  and getting this error.
There is no config.json file
https://huggingface.co/runwayml/stable-diffusion-v1-5
(venv) C:\sd\AniPortrait>python -m scripts.pose2vid --config ./configs/prompts/animation.yaml -W 512 -H 512 -L 64
Traceback (most recent call last):
  File "C:\Program Files\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Program Files\Python310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "C:\sd\AniPortrait\scripts\pose2vid.py", line 199, in <module>
    main()
  File "C:\sd\AniPortrait\scripts\pose2vid.py", line 60, in main
    reference_unet = UNet2DConditionModel.from_pretrained(
  File "C:\sd\AniPortrait\venv\lib\site-packages\diffusers\models\modeling_utils.py", line 712, in from_pretrained
    config, unused_kwargs, commit_hash = cls.load_config(
  File "C:\sd\AniPortrait\venv\lib\site-packages\diffusers\configuration_utils.py", line 365, in load_config
    raise EnvironmentError(
OSError: Error no file named config.json found in directory ./pretrained_model/stable-diffusion-v1-5.

While downloading the config files from HugginFace it prepend the config file names with folder name.
Renamced the same and it worked.


Implement Contact wrench Cone in math component

The math is taken from Stability of surface contacts for humanoid robots: Closed-form formulae of the Contact Wrench Cone for rectangular support areas
The PR is required to implement the Feasibility wrench constraint in TSID
cc @Giulero

I would add some test cases where you test the constraint with values you are sure are inside, and values for which you are sure are outside. I have seen you have tested with respect to what you have computed in Matlab, but you could have done the same mistake twice. Also, if you need to edit the constraint by adding or removing lines, that part would break.

I agree on that @S-Dafarra, I'm working on this 😃

Here I improved the test https://github.com/dic-iit/bipedal-locomotion-framework/pull/352/commits/50e1f44afc2dc94166b51983914ab6ed61b3c00d

@S-Dafarra and @prashanthr05 let me know if you're fine with the modification that I did``


GetReferenceAssemblyPaths needs GetFrameworkPaths

GetReferenceAssemblyPaths references
@(_CombinedTargetFrameworkDirectoriesItem) and other properties/items
that get defined (for .NET projects) in GetFrameworkPaths, but did
not depend on it. Dependencies were enforced by listing
GetFrameworkPaths first whenever dependencies existed, but that is hard
to enforce, and when missed caused dotnet/sdk#1730.
I'm adding this explicitly rather than appending to the already-existent
but undefined $(GetReferenceAssemblyPathsDependsOn) because a GitHub
search found a few projects (mostly Mono-related) that override rather
than append to GetReferenceAssemblyPathsDependsOn. That was safe because
no released MSBuild ever defined anything in that property, but could
introduce problems if we started defining it.
Fixes #3196.

Another possiblity would be appending to $(GetReferenceAssemblyPathsDependsOn) in Microsoft.NETFramework.CurrentVersion.targets where GetFrameworkPaths is defined. That wouldn't be a comprehensive fix--it'd still be susceptible to complete-overrides like the Mono things I found above--but it'd have somewhat better scoping.
And since almost everything has the double-dependencies set up (how it's been working for a long time) the possiblity of overriding to lose this new link doesn't seem totally catastrophic.
Preferences on approach?
cc @dsplaisted

@dotnet-bot test Ubuntu16.04 Build for CoreCLR please


White screen when starting Brackets

When starting Brackets it will occasionally start with a white screen. The window will become unresponsive and have to be killed from terminal.
As per #5381 this can be temporarily fixed by clearing cef_data and restarting.

Stale, haven't experienced this issue in more recent versions of brackets.

I faced this issue yesterday. Don't know why.

When I open brackets all I get is a black screen.  Why is this?


Play symbol not rendering on Android

The play symbol on the Playground website https://www.portals-project.org/playground/ does not render on Android phones. The underlying issue seems to be that the symbol ⏵ is not supported in android fonts, and it is also not covered in the fonts used on the website https://www.fileformat.info/info/unicode/char/23f5/fontsupport.htm.

[ ] Suggest some alternative play symbols (that you've checked work)
[ ] Integrate the new play symbol

https://www.compart.com/en/unicode/U+25B6
This character appears to be working as well on my phone (Samsung A52, Android) and on all of my browsers (Firefox, Google Chrome, Edge)


ITSM SKILL (Experimental Skill of Microsoft)

Except experimental skills all skills uses the Bot Framework Solutions Beta Release 0.8.
As per this release ActivityHandlerDialog.cs has been deprecated and replaced with a Waterfall Dialog implementation in the MainDialog of the Virtual Assistant and Skill templates.
More details you can verify in below links:
https://microsoft.github.io/botframework-solutions/overview/whats-new/0.8-beta/Summary/
https://microsoft.github.io/botframework-solutions/overview/whats-new/0.8-beta/maindialog-updates/
https://microsoft.github.io/botframework-solutions/virtual-assistant/handbook/activity-flow/
Some of the experimental skills still uses the ActivityHandlerDialog(Now deprecated)  in its MainDialog and some experimental skills uses the RouterDialog(Now deprecated) in its MainDialog.
[Note: RouterDialog replace with ActivityHandlerDialog and now ActivityHandlerDialog replace with Waterfall Dialog.
https://github.com/microsoft/botframework-solutions/issues/3092
https://github.com/microsoft/botframework-solutions/pull/2585]
So  experimental skills (e.g ITSM Skill) should work properly if ActivityHandlerDialog replaced with a Waterfall Dialog implementation in the MainDialog. Due to this reason ITSM skill is not working as per expected even properly setup on ServiceNow and successfully deployed on Azure Portal.
When ITSM skill will be available with latest code on Github?

@SavitaKumari-NTT It seems you used ITSM skill directly. The sources of these packages are configured in this file https://github.com/microsoft/botframework-skills/blob/master/skills/csharp/Directory.Build.props in the solutions project.
You could add them manually to your sources:

https://botbuilder.myget.org/F/aitemplates/api/v3/index.json
https://botbuilder.myget.org/F/botbuilder-v4-dotnet-daily/api/v3/index.json
https://www.myget.org/F/botbuilder-community-dotnet/api/v3/index.json

@SavitaKumari-NTT , please reactivate this issue if you have more question.


Can I customize the charset of the database?

Firstly, thanks for your work on this project! The image is convenient for me.
And then my issue is how I can customize the charset of the database.
The default charset of Oracle is WE8MSWIN1252.
'Cause I need insert data in Chinese, I should change the charset to AL32UTF8 or ZHS16GBK.
But that simply execute 'ALTER DATABASE character set INTERNAL_USE ZHS16GBK;' will break the em down.
Can I configure the charset before the database is initialized?

My issue has been solved in previous issue  #53


Unable to sync projects  in my android studio

This is the error . Please help me to resolve this.. I want to be in sync with the course. I downloaded the repository from the github(for the udacity's sunshine app) and pasted it in the android project directory.

Import project from version control right there from top left corner. and then use git logs to directly move from lesson to lesson


(Request) Tutorial reading images directory

Hi
Any chance you could make a (simple) tutorial on how to read an image directory and make an image clickable instead of uploading them? So an "Insert image" button which just shows all images which are clickable for inserting?
Thanks!

Hi @Rverm are you looking to build the media manager yourself or are you thinking about integrating an external media manager (such as KCFInder). The reason I ask is that @concept-core is currently looking at creating a tutorial to cover KCFinder integration which supports selecting from an existing folder/list of images.

@Rverm Jup that is completely true.
If you use php I will be able to help you with what you need, but I do use KCFinder. ( keep that in mind )

@anthonyjb I would prefer building my own media manager, but I've looked at KCFinder and it also looks cool.
@concept-core That's exactly what I'm looking for! Looking forward to it!

@Rverm I Will try to create the tutorial this weekend.
There is only one thing I need to know for you specific.
Do you use a framework by any change ?
If so which one.

@concept-core I'm using jQuery, and Code Igniter for the php part. Thanks again!

@Rverm the newer version of codeigniter?
Or the old one when hosted by Ellis lab ?

@concept-core the latest version, 3.0.6 :)

@Rverm i am going to Wright the tutorial Today.
If you wish you can message me.

yes i want to make a tutorial on how to integrate KCFinder inside contenttools. It will really be appreciable and make contenttools more usable and effectable.
regards.


Fix the no soma h5 morphologies

Fix the no soma morphologies. The single-neurite.h5 is :
HDF5 "single-neurite.h5" {
GROUP "/" {
   DATASET "points" {
      DATATYPE  H5T_IEEE_F32LE
      DATASPACE  SIMPLE { ( 3, 4 ) / ( 3, 4 ) }
      DATA {
      (0,0): 4, 0, 0, 2,
      (1,0): 5, 0, 0, 2,
      (2,0): 6, 0, 0, 2
      }
   }
   DATASET "structure" {
      DATATYPE  H5T_STD_I32LE
      DATASPACE  SIMPLE { ( 1, 3 ) / ( 1, 3 ) }
      DATA {
      (0,0): 0, 2, 0
      }
   }
}
}

Could be useful to check for soma types in the new added tests, but it's up to you

Good idea. I will do it. I need to rebase on your test PR anyway.

What test PR? Do I have one?

I meant your pytest PR

Rah not PR (we merged it already) commit.


fix(esm): add explicit extensions to imports/exports

I'm unable to run the tests and the build seems broken. :/
I think it's because the project is doing nonconventional things like requiring ts files from js files. 😅
I added an ESLint rule to ensure that extensions are set in imports/exports.

What:
Add extensions to all imports/exports to be fully compatible with ESM modules
Why:
Resolves #14
How:
Checklist:

[ ] Tests
[ ] TypeScript definitions updated
[ ] Ready to be merged

@jgoux can you rebase from main if possible? (force pushing is OK
I ran npm run format on main to hopefully make the diffing easier

@jgoux can you rebase from main if possible? (force pushing is OK)
I ran npm run format on main to hopefully make the diffing easier

Done! Could you look into the build and tests? I think the most reasonable move would be to have everything in TS.

Could you look into the build and tests?

Yup yup!

I think the most reasonable move would be to have everything in TS.

Definitely not opposed to this, but I'm curious how "upstream" handles this, given that this mixture of both JS and TS come from the same codebase structure as there.

After playing a bit with the types generated by your package I no longer think the issue is with the explicit file extensions (even if it's a best practice 😄).
It seems that the type RenderResult is incorrect in the type definitions. Is it something you maintain by hand?
This line is incorrect: https://github.com/crutchcorn/cli-testing-library/blob/main/types/pure.d.ts#L4
queries is a value, not a type (and this value doesn't exist in ./get-queries-for-instance.
I think doing should fix everything!
import * as queries from './queries'
import type { BoundFunction } from './get-queries-for-instance'


Update safe-url-list.md

Corrected referenced Event ID to 3701

@RKauf00 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Pavithir is a PM and approved the update, so we'll merge the PR. Thanks for the review.

Thanks!

From: Kristine Toliver<EMAIL_ADDRESS>Sent: Wednesday, July 22, 2020, 17:06
To: MicrosoftDocs/azure-docs
Cc: Rick; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Update safe-url-list.md (#59536)
@Pavithirhttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPavithir&data=02|01||69cc79c7089441aec86108d82e831d30|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637310487942227078&sdata=QAA9q8PbV3%2B7PdIyy9y8nc3HtS6iUAYP0vhVMonUOQ4%3D&reserved=0 is a PM and approved the update, so we'll merge the PR. Thanks for the review.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F59536%23issuecomment-662696580&data=02|01||69cc79c7089441aec86108d82e831d30|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637310487942237073&sdata=ykeG3Mnx%2FsYsWM%2F%2FLtXTAmS4E45PfHcKAMUxIpvmNxg%3D&reserved=0, or unsubscribehttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGIFLUBHN4PF4AI37MDAYVTR45IFTANCNFSM4PFAXFHA&data=02|01||69cc79c7089441aec86108d82e831d30|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637310487942237073&sdata=kk4Gw8C9dUzBcGfzTcePxwc6wlSHAiX4i9RpHGDWLac%3D&reserved=0.


Allow adding custom swap recipient

[x] create prototype on Swap
[x] extract to CustomAddressInput to separate component
[x] separate to DEFAULT and EDITABLE states
[x] add i18n
[x] enable editing
[x] enable copy button
[x] get target chain from asset to provide for opening externally
[x] enable open external
[x] styling adjustments
[x] adjust state management
[x] extract props
[x] add types
[x] add story
[x] add tests
[x] extract address link handler into hook
[x] address validation

Closes #1683

Pls fix background color of the input (dark theme). Check SendXYZForms to get color values.

Pls disable Swap button whenever EditableAddress has an editable mode. That's needed to make sure an user has entered + saved a new address before swapping.

Mobile view needs to be tweaked a little bit - it can be the same position as we have in desktop mode

Also, let's show shorten address in mobile mode only. In desktop mode there is good space to show full address.


Request for apiserver network policy document

This is a Feature Request

What would you like to be added

The replacement of SSH tunnel solution is apiserver network policy, and it has been alpha in v1.16, I think it's time to setup document about this part basically, since there are many requests for deprecating ssh tunnel and related options support, see 49402, 75330.
Why is this needed
The master-node communication is not as easy as that we thought, it became even more complicated while we are facing multi-cloud environment and vpn based networking, therefore, the apiserver network policy which intend to build a more extendable and isolate control network has been proposed. As it became alpha stage in v1.16, I think it's reasonable to setup related document about this feature.
Comments

I'm not sure about whether we have been setup related pages about this, for me, it's NO.
TBH, I'm also not clear about the KEP, would this be the full replacement of ssh tunnel, and shall our users customize it for suitable to their complex environment, just like what CNI did?
BTW, api server policy indicates generated proxy urls for builtin cluster component services, see this section, maybe we need to find another terminology for our control panel networking proxy.
Thanks.

The target page should be part of this, we may also need to setup separate page for explaining the new proposal about apiserver network proxy.
/sig network
/language en
/priority backlog


V1175:  Broken fwlinks

Hello, @mcleblanc,
Localization team has reported source content issue that causes localized version to have broken links on target versions.
Please, help to check my proposed file change into the article and help to merge if you agree with fix. If not, please, let me know either if you would like me to fix it in another way within this PR, if you prefer to fix it in another PR or if I should close this PR as by-design. In case of using another PR, please, let me know of your PR number, so we can confirm and close this PR.
Many thanks in advance.
Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Thanks for pointing this out, @carolinacmoravia. Since you've pointed out the issue without actually fixing it, I'm going to close this PR. I've opened Issue #4931 to address this issue, since it will require some investigation before we can address it adequately.


use IHttpActionResult

There are good arguements for standardizing on IHttpActionResult raised by @darrelmiller.
See the discussion beginning here:
https://gitter.im/mspnp/azure-guidance?at=553035100e3138bb6be858e5
https://msdn.microsoft.com/en-us/library/system.web.http.ihttpactionresult(v=vs.118).aspx
http://www.bizcoder.com/composing-api-responses-for-maximum-reuse

The code samples will be updated in the near future