[Cognitive Services] Support 'usage show'

Add a new command
az cognitiveservices account usage show
I didn't modify HISTORY.rst because there is another PR pending.
(Will rebase and update when that PR get merged)

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

Ping @yangyuan. The sprint 44's code complete day is tomorrow.

@troydai I think this one should join next sprint...

close before ready to be reviewed....

Please send a PR to upgrade the dependency to the latest then.

Hi @troydai, I might have some background missing, what do you mean?
Do you mean rebase to latest code? Or you mean modify some files?

'azure-mgmt-cognitiveservices==3.0.0',

@troydai, I will do it in this PR...
I'm closing this PR temporarily, will re-open it probably later this week.

So this dependency won't be upgraded in this sprint?

Yes, better move to next sprint.
(Also I've re-opened this)

Fixes #5246. Fixes #3358.

@tjprescott about the completer, I do agree that we should implement those.
I think it should not be blocking issue for this PR as this is for adding two new commands.
How about I create a task and do them in another PR?

Code complete isn't until 9/13, so there would be ample time to get that in. However, yes you could create an issue and do completers as separate work. At a minimum though you need to supply the correct populator-command kwargs in help to direct people to the list-kinds command.

Cool, I've added that for account create.
Please review if I understand it correctly.
Also created a task for completers
https://msazure.visualstudio.com/a531312f-ba7b-4573-ab4f-d468b740a319/_workitems/edit/2978184
Let me know if you prefer a github issue.

GitHub issue is preferred because we can track it. Also, please add a populator command for the create/update commands for --sku to point to list-skus.

I've created the tasks as well as github issues for the two issues we discussed.
https://github.com/Azure/azure-cli/issues/7156
https://github.com/Azure/azure-cli/issues/7157

@tjprescott

@troydai I'm waiting for a response on a question

@tjprescott sorry didn't see that question. (it was collapsed in "outdated")
So for both issues we discussed I've created the task and issues.


Feature Request: Add translations

I love this game as I like geography and would love to let my children play it. Unfortunately their knowledge of English isn't good enough to play it in its current form.
If there was a language-support I would love to take the time to add German translations.

Hi, thanks for the suggestion. I'd definitely like to support different languages. Would translations of country names be sufficient (for now) for them to be able to play, or is the user interface being in English the bigger hurdle?

The user interface and the country names would be what's best to change. If there was language support I would love to add Spanish (México and Spain)

If translation to Portuguese(pt-br) is needed, I'm happy to help translate it too

Same for french :)

Country names would be a good start and sufficient for me. But would love to see the user interface translated, too.

I'd be happy to translate country names and UI to Italian!

I appreciate all the interest! Working on this right now.

You could use something like crowdin.

Hi everyone, I've added some info about early community translations over here: #70


add-watchguard-parser-2

Required items, please complete
Change(s):

Added Watchguard

Reason for Change(s):

New parser

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hi @oshezaf
Can you please review and provide your comments.
Thanks


Suggestion: 1 character margin to the left of ruler

Currently (or at least on the version I'm using), micro displays the ruler (line numbers) like this:
(separating the ruler out from everything else with |)
|1 |#include <stdio.h>
|2 |
|3 |int main(void) {
|4 |    printf("Hello world\n");
|5 |    return 0;
|6 |}

As you can see, there is a 1 character margin to the left of the ruler.
But I think that there should be another margin to the left to balance out the left and right.
| 1 |#include <stdio.h>
| 2 |
| 3 |int main(void) {
| 4 |    printf("Hello world\n");
| 5 |    return 0;
| 6 |}

I don't think this is hard to implement, and it can even be a toggleable setting, if a person is used to the old spacing and doesn't like this.

This is not a fix, since I do believe this is a bug. But if you enable diffgutter > set diffgutter true the missing margin is added, and if you don't want the coloring in the gutter you can do > set diff false.


move stage damage combo box in Scenario editor

I think we should move the stage damage combo box up a level in the Scenario editor because we should choose only 1 set of stage damage functions for a given Scenario. Currently, you can select any set for a specific impact area scenario. We need to at least add validation that the user cannot select different sets of stage damage functions for a given Scenario

I think this is now completed. @Richard do you want to take a look and see if this behaves the way you want?
These changes are in Alternative_Compute_Results branch.

I will confirm in the next couple days.
Best,
R Nugent III

On Jun 6, 2022, at 11:24 AM, codymccoy @.***> wrote:

I think this is now completed. @richard do you want to take a look and see if this behaves the way you want?
These changes are in Alternative_Compute_Results branch.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were assigned.

This does behave the way I expected. Could we move the "Add Additional Thresholds" up to the second row?

It has now been moved up:

Looks great! Thanks, @codymccoy

behaves as expected


Don’t stop daemon on migration hard failure

Also changes missing storage layer for container
RWLayer to a soft failure.
Fixes #20147
Soft failure: migration of image/container is skipped
Hard failure: migration stops
Signed-off-by: Tonis Tiigi<EMAIL_ADDRESS>

Job: Docker-PRs-WoW-TP4 FAILED:

---
 Current directory is /c/gopath/src/github.com/docker/docker
---------------------------------------------------------------------------

-----------------------------------------------
ERROR: Failed with exitcode 1 at Tue Feb 9 17:52:49 CUT 2016.
-----------------------------------------------

INFO: Tidying up at end of run
INFO: Nuking /d/CI
INFO: Zapped successfully
INFO: End of cleanup
INFO: Ended at Tue Feb  9 17:52:50 CUT 2016 (0m 2s)
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[docker] $ sh -xe D:\temp\hudson6526813037811085301.sh
+ set +e
+ set +x
INFO: End of cleanup
Notifying endpoint 'HTTP:https://leeroy.dockerproject.org/notification/jenkins'
---

Job: Docker-PRs-gccgo FAILED:

---
est.go:1030: TestDockerNetworkDisconnectFromHost.pN59_github_com_docker_docker_integration_cli.DockerNetworkSuite	1.065s
PASS: docker_cli_network_unix_test.go:641: TestDockerNetworkDriverOptions.pN59_github_com_docker_docker_integration_cli.DockerNetworkSuite	1.086s
PASS: docker_cli_network_unix_test.go:838: TestDockerNetworkDriverUngracefulRestart.pN59_github_com_docker_docker_integration_cli.DockerNetworkSuite	4.882s

----------------------------------------------------------------------
FAIL: docker_cli_network_unix_test.go:995: TestDockerNetworkHostModeUngracefulDaemonRestart.pN59_github_com_docker_docker_integration_cli.DockerNetworkSuite

[d88005000] waiting for daemon to start
[d88005000] daemon started
[d88005000] exiting daemon
[d88005000] waiting for daemon to start
[d88005000] daemon started
docker_cli_network_unix_test.go:1016:
    c.Assert(err, checker.IsNil)
... value *exec.ExitError = &exec.ExitError{ProcessState:(*os.ProcessState)(0xc20e11b880)} ("exit status 1")

[d88005000] exiting daemon

----------------------------------------------------------------------
PASS: docker_cli_network_unix_test.go:1059:
---

LGTM

Job: Docker-PRs-WoW-TP4 FAILED:

---
 Current directory is /c/gopath/src/github.com/docker/docker
---------------------------------------------------------------------------

-----------------------------------------------
ERROR: Failed with exitcode 1 at Tue Feb 9 21:10:04 CUT 2016.
-----------------------------------------------

INFO: Tidying up at end of run
INFO: Nuking /d/CI
INFO: Zapped successfully
INFO: End of cleanup
INFO: Ended at Tue Feb  9 21:10:05 CUT 2016 (0m 2s)
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[docker] $ sh -xe D:\temp\hudson2243405190160085494.sh
+ set +e
+ set +x
INFO: End of cleanup
Notifying endpoint 'HTTP:https://leeroy.dockerproject.org/notification/jenkins'
---

LGTM

LGTM


BUG: Cannot build C++ project using MSVC 2019 and OpenBLAS

On VS2019, trying to build using MSVC I get errors like the following:
[1/86] Building CXX object OpenMEEGMaths\CMakeFiles\OpenMEEGMaths.dir\src\vector.cpp.obj
FAILED: OpenMEEGMaths/CMakeFiles/OpenMEEGMaths.dir/src/vector.cpp.obj
C:\PROGRA~2\MICROS~1\2019\ENTERP~1\VC\Tools\MSVC\1429~1.301\bin\Hostx64\x64\cl.exe  /nologo /TP -DH5_BUILT_AS_DYNAMIC_LIB -DHAVE_BLAS -DHAVE_LAPACK -DHAVE_SHARED_PTR_ARRAY_SUPPORT -DOpenMEEGMaths_EXPORTS -DUSE_OPENBLAS -I%SRC_DIR%\OpenMEEGMaths\include -I%SRC_DIR%\build_Release\OpenMEEGMaths -I%SRC_DIR%\build_Release -I%SRC_DIR%\OpenMEEGMaths\OpenMEEGMaths\src -external:I%PREFIX%\Library\include -external:W0 /wd4275 /wd4101 -openmp /MD /O2 /Ob2 /DNDEBUG -std:c++17 /showIncludes /FoOpenMEEGMaths\CMakeFiles\OpenMEEGMaths.dir\src\vector.cpp.obj /FdOpenMEEGMaths\CMakeFiles\OpenMEEGMaths.dir\ /FS -c %SRC_DIR%\OpenMEEGMaths\src\vector.cpp
%PREFIX%\Library\include\lapack.h(104): error C2143: syntax error: missing ',' before '*'
%PREFIX%\Library\include\lapack.h(106): error C2143: syntax error: missing ',' before '*'
... <hundreds of these>

You can see it on conda-forge here for example, where it uses the OpenBLAS libraries that they build:
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=524975&view=logs&jobId=5be07ae1-d8ba-5406-47b6-8e3a3a12f825&j=5be07ae1-d8ba-5406-47b6-8e3a3a12f825&t=0bf03e01-0bec-5b85-5316-b1633322e895
And on GitHub actions here from earlier this week, same error -- this is using the OpenBLAS builds SciPy makes with mingw64, but I had the same problem when I tried previously just by downloading the 0.3.20 release binaries from GitHub:

Run cmake --build build  --config Release
Microsoft (R) Build Engine version 16.11.2+f32259642 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.
  Checking Build System
  Building Custom Rule D:/a/openmeeg/openmeeg/OpenMEEGMaths/CMakeLists.txt
  vector.cpp
C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\Tools\MSVC\14.16.27023\include\xlocale(319): warning C4[53](https://github.com/openmeeg/openmeeg/runs/6975816876?check_suite_focus=true#step:16:54)0: C++ exception handler used, but unwind semantics are not enabled. Specify /EHsc [D:\a\openmeeg\openmeeg\build\OpenMEEGMaths\OpenMEEGMaths.vcxproj]
d:\a\openmeeg\openmeeg\openblas\64\include\lapack.h(104): error C2143: syntax error: missing ',' before '*' [D:\a\openmeeg\openmeeg\build\OpenMEEGMaths\OpenMEEGMaths.vcxproj]
d:\a\openmeeg\openmeeg\openblas\64\include\lapack.h(106): error C2143: syntax error: missing ',' before '*' [D:\a\openmeeg\openmeeg\build\OpenMEEGMaths\OpenMEEGMaths.vcxproj]
... <hundreds of these>

I am a bit mystified by this because these lines are protected by an ifdef that should take care of this I think:
https://github.com/xianyi/OpenBLAS/blob/9283c7c0b5a9ec7bbe3b6dfc1a019b29b3e112e5/lapack-netlib/LAPACKE/include/lapack.h#L71-L104
Anyone run into this before? I didn't see any mention of anything like this in the Visual Studio wiki docs, but maybe I just missed it. Perhaps it's a cmake+MSVC+CPP issue, and there is some cmake fix...? Not sure.
An identical build configuration using mingw64 doesn't have this problem... but I think for conda-forge I might need to build with MSVC. And in any case it would be nice if it worked...

Where did you get that lapack.h from?

The include directory of the install step for OpenBLAS

Same as what is in Reference-LAPACK actually. No idea why VS2019 does not like it, I'm fairly sure earlier versions did.

Okay I opened a near-duplicate issue at https://github.com/Reference-LAPACK/lapack/issues/683

I guess you could try replacing the float _Complex in the definition of lapack_complex_float on line 39 with the Microsoft equivalent _Fcomplex (came across this when preparing the updated f2clapack that is now in the develop branch)
https://docs.microsoft.com/en-us/cpp/c-runtime-library/complex-math-support?view=msvc-170

Not sure why we have this define at https://github.com/xianyi/OpenBLAS/blob/ad4598143bfbf77acd16f4ae884462edc69b496e/lapack-netlib/LAPACKE/include/lapack.h#L7
If that is removed, I think MSVC should be able to process the header just fine.

File history suggests it has been this way ever since LAPACKE got integrated into the LAPACK package in 2011 - maybe the ifdef made sense when it was separate

Also lapacke_config.h woud probably need a define LAPACK_COMPLEX_STRUCTURE dependent on (some variant spelling of) C_MSVC like there already is for older Android. (At least I see nothing defining this elsewhere, and without it we are back at  float _Complex vs. _Fcomplex)

@larsoner  could you try @isuruf 's suggestion ?

When I take the 0.3.18 headers and comment out lines 7 and 9 (#if and #endif) I still get a ton of errors like
  vector.cpp
c:\users\larsoner\python\openmeeg\openblas\64\include\lapacke_config.h(109): error C2146: syntax error: missing ';' bef
ore identifier 'lapack_make_complex_float' [C:\Users\larsoner\python\openmeeg\build\OpenMEEGMaths\OpenMEEGMaths.vcxproj
]
c:\users\larsoner\python\openmeeg\openblas\64\include\lapacke_config.h(109): error C4430: missing type specifier - int
assumed. Note: C++ does not support default-int [C:\Users\larsoner\python\openmeeg\build\OpenMEEGMaths\OpenMEEGMaths.vc
xproj]
c:\users\larsoner\python\openmeeg\openblas\64\include\lapacke_config.h(110): error C2371: '_Complex': redefinition; dif
ferent basic types [C:\Users\larsoner\python\openmeeg\build\OpenMEEGMaths\OpenMEEGMaths.vcxproj]

One other thing that might be relevant is this in the CMAKE config of the OpenMEEG project:
CMakeLists.txt:set(CMAKE_CXX_STANDARD 17)  # use c++17

so maybe this hasn't been noticed because nobody has compiled with MSVC using the c++17 standard...?

this is supposed to be complex or struct emulating it

:scream: I hope this issue doesn't turn out to be something like that where I missed something in OpenMEEG itself
I'll check next time I'm on my Windows boot...

Okay I changed the lines @brada4 pointed out from their original form:
#if WIN32
    #define LAPACK_COMPLEX_CUSTOM
    #define lapack_complex_float float
    #define lapack_complex_double double
#endif

which show the problem on CIs in https://github.com/openmeeg/openmeeg/pull/454, to a variant of what @martin-frbg originally suggested:
#if defined(_MSC_VER)
    #include <complex.h>
    #define LAPACK_COMPLEX_CUSTOM
    #define lapack_complex_float _Fcomplex
    #define lapack_complex_double _Dcomplex
#endif

And things compile fine with https://github.com/openmeeg/openmeeg/pull/454/commits/6eb29e8df61ba2e8458ae9cbb41ca96c95f5e783 locally with MSVC 2017 at least and tests pass, though I still need to convince MSVC 2019 in CIs to accept it! 🚀
I'll close this as fixed, and I've updated the MSVC instruction page to mention this: https://github.com/xianyi/OpenBLAS/wiki/How-to-use-OpenBLAS-in-Microsoft-Visual-Studio#visual-studio-2017-c2017-standard


[metatext] link previews not showing

Using metatext, I logged in and checked the same fedibird account with a mastodon account and a takahe account.
Link previews show from mastodon, don't show from takahe
https://takahe.social/<EMAIL_ADDRESS>

Link previews are a feature supplied by the server, so this is not supposed to work (and we don't plan on implementing them yet). I'm going to close this as it's not a separate bug!


replace file

what process saves the info if i replace a file and how i can found the info, like What do I have to put in what process
xd

if i replace a file and how i can found the info

You would need to be more descriptive. For files you could search the filename or the content for the string in process memory.


fixed f string errors

When running locally on some experiments I found that I made a mistake when writing the f-strings. Specifically, I used too many single quotes in a couple of them. I went ahead and fixed them. Weird that it didn't cause a problem on my first test but I've tested everything a few different times now and it's all working.

OK, thanks for following up with a fix!


Change multiple light parameters at once?

Please explain your feature request in a one or two sentences.
Would it be possible to change multiple parameters for a light at once?  For instance, on my H6154, I have a routine which changes the color and brightness.  The plugin handles this by sending separate commands several seconds apart, one for color, one for brightness.  Is it possible to send both parameters in a single command?

Is your feature request related to a problem? Please describe.
Changing color and brightness of a lighting strip requires multiple commands over several seconds.

Any particular Govee devices that this relates to?
I have experienced the problem with my H6154 Wifi light strip, I assume it applies to all Govee lighting devices where brightness and color are controllable.

Anything else?

Ironically im not very familar with the govee app, but there must be a way to be able to make some kind of scene in the govee app with a precise brightness and colour?
If you can, you might be able to make use of this and turn this into a scene which you can then use via homekit using this guide:
https://github.com/bwp91/homebridge-govee/wiki/Scene%2C-Music%2C-DIY-Modes

Was anything done to address this issue before it was closed?
I looked into it, as far as I can tell the Govee app does not allow custom scenes of this type, at least for this light strip.  I also was unable to get the plugin to launch any scenes, just manual control.


Stm32 i2c timing option

This change offers an additional option for configuring the exact timing of I2C V2 peripherals of STM32s.
The timing computation can either be left to the simple algorithm already in place in the driver, or can be computed through SW tools like STM32CubeMX and provided as an entry in a board's Device Tree file.
This would look like:
 &i2c1 {
 	status = "okay";
	clock-frequency = <I2C_BITRATE_FAST>;
	timing = <0x10C01C20>;
 };

@pabigot thanks for your feedback. I have added a few answers to your points, and I think I would need some more guidance before deciding what to do next.

I don't like the way this silently overrides any runtime configuration by passing flags like I2C_SPEED_FAST to the device that has this flags. This seems likely to be confusing and to cause problems when different bus devices require different speeds.

Right ... the timing option allows very precise and fine-tuned configuration for a complete and definitive HW setup, but it is not dynamic at all. The use case it targets is rather a final product than a development kit.

A better approach might be to provide an array that specifies a map from a standard speed (100 kHz, 400 kHz, etc) to the desired value to use when programming that selected speed, assuming that the clock configuration is not likely to be runtime controllable.

That would handle a few more cases indeed, but still would not handle a runtime change of the clock tree.
The default algorithm should rather be used for runtime configuration.

I'm also a little concerned with the degree to which v1 and v2 STM I2C drivers diverge in functionality, but maybe that's unavoidable.
v1 and v2 are quite different. This very fine timing capability (together with programming complexity) is V2 only.

There is still interest in addressing #4039, which might be a better way to deal with this, though it requires some API redesign.

This timing option was not primarily meant to allow new frequencies, but to allow a product designer to fine-tune the I2C values to precisely match his hardware with measured fall time / rise time . Even with #4039 introduction, this aspects would probably require introduction of a timing option. But then if #4039 is addressed  ... that would become impossible to provide a table of values as suggested earlier ...
so do you think I should implement the support for 3 values (100/400/1000kHz) as suggested earlier ?

Right ... the timing option allows very precise and fine-tuned configuration for a complete and definitive HW setup, but it is not dynamic at all. The use case it targets is rather a final product than a development kit.

That's a worthwhile use case, but the limitations should be made extremely clear in the description so new folks don't shoot themselves.

That would handle a few more cases indeed, but still would not handle a runtime change of the clock tree.

How likely is such a change?  And how difficult is it to detect?

if #4039 is addressed ... that would become impossible to provide a table of values as suggested earlier ...

I would envision some sort of back door, e.g. not restricting the speed cell of the table to match a standard rate.

so do you think I should implement the support for 3 values (100/400/1000kHz) as suggested earlier ?

My preference is a table like:
timingr = <100 0x4std>,  <400 0x4fast>, <1000 0x4fast+>;

collecting a sequence of 2-cell assignments mapping bus speed in kHz to the TIMINGR register value.  There's no requirement that the table be complete, nor that the speeds be standard speeds.  Provide only the values required for the application.  If a speed requested at runtime is not present in the table then either fall back to runtime calculation or fail the configuration, depending on how robust you want your system to be.
As an extra, if it's possible to specify the clock configuration assumed in the precalculated values, and detect at runtime that it's not right, fallback to runtime or fail for that too makes sense.
I believe the table approach has more potential than the current solution for being extensible to other SoCs.  Would that be acceptable?

I believe the table approach has more potential than the current solution for being extensible to other SoCs. Would that be acceptable?

Yes I hope so. You proposal with fallback solutions seems the more appropriate. To cover most cases, I guess we would need tables with 3 values per entry: clock source frequency, I2C requested frequency,  and timing value.
timingr = <64000000 100 0x4std>,  <64000000 400 0x4fast>, <64000000 1000 0x4fast+>;
Then we can check the 2 first ones and decide to apply timing or fallback.
I have to check how to get table with dynamic lengths from device tree and how to allocate memory for storing it.

@erwango @gmarull @pabigot  I followed latest recommendations on providing a list of possible settings with their matching clock configuration. I came-up with a new proposal that seem to be flexible enough.  Let me know your feedback.

updated PR sent with fixes I could do. I left few comments open when I did not implement the change - let me know if more is needed.


Add ruby predefined objects to variable name checker

Hi, @yuki24.
I'm thinking of adding predefined objects (false, true, and nil) in the variable name checker dictionary. Their 2.4 deprecated toplevel constants are already included, but the objects themselves are not.
The constants:
a = NOL
# NameError: uninitialized constant NOL
# Did you mean?  NIL
b = FALS
# NameError: uninitialized constant FALS
# Did you mean?  FALSE
c = TREU
# NameError: uninitialized constant TREU
# Did you mean?  TRUE

but, the objects:
a = nol
# NameError: undefined local variable or method `nol' for main:Object
b = fals
# NameError: undefined local variable or method `fals' for main:Object
c = treu
# NameError: undefined local variable or method `treu' for main:Object

I need to be reminded of these objects too.
I don't know how to get the predefined objects in the nice way so I just write them in an array.
Since nil return "" when called with to_s, I replaced it with :nil.
Should I wrote all of them in symbols ([:false, :true, :nil]) to make it consistent with the others?
If this PR is merged:
a = nol
# NameError: undefined local variable or method `nol' for main:Object
# Did you mean?  nil
b = fals
# NameError: undefined local variable or method `fals' for main:Object
# Did you mean?  false
c = treu
# NameError: undefined local variable or method `treu' for main:Object
# Did you mean?  true

This is a great addition. Thanks!


Post release automated changes for orbital releases

Post release automated changes for azure-arm-orbital

API change check
API changes are not detected in this pull request.


Add with pkgs; to documentation snippet

Motivation for this change
Code snippet does not work when just adding to a configuration without a supporting with pkgs;
Things done
Small documentation change

@danieldk added this snippet recently, (merged by @mweinelt)

@danieldk added this snippet recently, (merged by @mweinelt) #95086

Thanks :+1: . Good catch! Could you also update the other examples using extraPackages in the same chapter? Could you also change the commit message to something like:
nixos/manual: add pkg to the lexical scope of opengl.extraPackages

? Makes it easier to change changes to the manual specifically :).

@Jake-Gillberg thanks for opening your first PR with nixpkgs :)

Please follow CONTRIBUTING.md and manual#submitting-changes-making-patches  and squash the fix-up commits.
This can be done without git rebase -i by doing:
git reset HEAD~1                    # move fix-up commits into unstaged
git add -- pkgs/                    # move changes into staged
git commit --amend --no-edit        # add changes to previous commit
git push ... ... --force            # modify current PR branch

However, git rebase -i is a more powerful alternative, I created a small video demonstrating it's use here. A more indepth text tutorial can be found here


Conversation setting screen shown if connection lost

Steps:
1 Create New conversation
2 Set Offline mode in network console
3 Send any request
Actual result: error is shown as a response and then conversation setting shown
Expected result: error is shown as a response, chat with error response is shown for user, Regenerate btn is available
https://github.com/epam/ai-dial-chat/assets/15156585/afe17835-abe6-437c-9996-087200e6dfeb

suggestion: try to stay input with text if we can't send message

It was verified on staging environment. Now the behaviour is the following

Create New conversation
Set Offline mode in network console
Send any prompt -> Red error appears -> DIAL is lost because of no internet connection
Turn internet connection on -> the conversation doesn't have saved sent message, no red error saved


Scopes.

Add statements to create and end scopes. I.e., NEW SCOPE creates a new scope in which only some special variables are passed/saved (something like END SCOPE to end it).
Would be much more useful once issue #7 is resolved.
Should also take the time to make special variables more noted/centralized in the code.

Add two special variables argument and return that stay in scope (so can things can be passed in and out). Multiple arguments and returns can be done with lists.


create-remix with indie stack template breaks when choosing JavaScript because prisma/seed.ts is now prisma/seed.js

What version of Remix are you using?
1.4.1
Steps to Reproduce
Follow the blog tutorial: https://remix.run/docs/en/v1/tutorials/blog and choose for JavaScript instead of TypeScript (using Node 14, though I don't know if that's relevant here)
The issue seems to be mentioned here as well: https://github.com/remix-run/remix/issues/2714#issuecomment-1106774296
Expected Behavior
The setup succeeds and I can learn how to use Remix.
Actual Behavior
The setup breaks because it tries to find a file called prisma/seed.ts, but there is only a prisma/seed.js file:
Running seed command `ts-node --require tsconfig-paths/register prisma/seed.ts` ...
internal/modules/cjs/loader.js:888
  throw err;
  ^

Error: Cannot find module './seed.ts'
Require stack:
- /home/kilian/Workspace/personal/remix/blog-tutorial2/prisma/imaginaryUncacheableRequireResolveScript
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:885:15)
    at Function.resolve (internal/modules/cjs/helpers.js:94:19)
    at requireResolveNonCached (/home/kilian/Workspace/personal/remix/blog-tutorial2/node_modules/ts-node/dist/bin.js:489:16)
    at getProjectSearchDir (/home/kilian/Workspace/personal/remix/blog-tutorial2/node_modules/ts-node/dist/bin.js:459:40)
    at phase3 (/home/kilian/Workspace/personal/remix/blog-tutorial2/node_modules/ts-node/dist/bin.js:263:27)
    at bootstrap (/home/kilian/Workspace/personal/remix/blog-tutorial2/node_modules/ts-node/dist/bin.js:44:30)
    at main (/home/kilian/Workspace/personal/remix/blog-tutorial2/node_modules/ts-node/dist/bin.js:32:12)
    at Object.<anonymous> (/home/kilian/Workspace/personal/remix/blog-tutorial2/node_modules/ts-node/dist/bin.js:519:5)
    at Module._compile (internal/modules/cjs/loader.js:1068:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1097:10) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/home/kilian/Workspace/personal/remix/blog-tutorial2/prisma/imaginaryUncacheableRequireResolveScript'
  ]
}

An error occured while running the seed command:
Error: Command failed with exit code 1: ts-node --require tsconfig-paths/register prisma/seed.ts

When choosing the TypeScript variant, things work as ecpected.

Duplicate of #2781


whitelist sync broken since 8.9.0

Description
If I sync my whitelist with the normal sync only the users on the last page are recognized while all others are not. This breaks Whitelist/Blacklist Highlighter.
Steps to Reproduce

Go to https://www.steamgifts.com/account/settings/profile?esgst=sync
Mark whitelist for sync
Click on the "Sync" button and confirm the warning
If not already activated activate Whitelist/Blacklist Highlighter
Check users from your whitelist which aren't on your last whitelist page

Expected Behavior
Sync all whitelist users properly.
Console Errors
nothing
System (please complete the following information):

ESGST Version: Extension v8.9.0
Browser + Version: Firefox 78.14.0esr

Additional Context
Blacklist might also be affected but my blacklist is empty so I can't check.

It's not just that only the last page is recognised, but it also clears ESGST's whitelist data of all other pages.

@at46 This bug has been fixed in SquishedPotatoe's fork. https://github.com/SquishedPotatoe/esgst


Broker with multiple devices

Hello,
First of all, congrats for this project : cool, light, good doc....
Btw, i'm struguling with some stuff :

if i have many devices on a node, how can i make different brokers for each ? (let's say, i have a zigbee device and a z-wave one : how can i make a broker for zigbee, and one different for z-wave ?) -> as far as i know, in helm you can define only one udev config 🤔
In the documentation, you talk about ID hash for names : which hash function do you use ? (If i want this name to be "predictive", i think i need to know how to hash and compare 🤔 )
Talking about brokerPod, this could be awesome if we can make a deployment, to deploy more k8s objects in reaction of device discovery (I don't see for now how we can pop a bunch of pods/services/configmaps etc when a device is discovered 🤔 )

Many thanks !!!!
(sorry if it's not the good place)

Hi !
To answer your questions from my understanding:

You can create multiple HELM deployments, one for the core components, and then one for each discoveries you want to have so multiple udev config in your use case
the related code is here https://github.com/project-akri/akri/blob/18f7dd4bd0b1a1aeb90db91abeda6516ae4af27a/agent/src/util/discovery_operator.rs#L1049 but I guess it's easier to just get the list of environment variables, filter with the prefix without the hash
not possible from what I know about AKRI but these components can live outside of AKRI management

Hi, I'll add a bit more information:

The integrated configuration item in the helm chart is a convenience one, so you can easily have a simple configuration when you start. You can also directly create the akri.sh/Configuration resource (either through kubectl, another helm chart or anything you want), there you will be able to create multiple Configuration resource for the same discovery handler. There is no documentation about that right now, but you can take examples from these tests files (from PRs) debugEchoConfiguration.yaml and udevDevNullConfiguration.yaml
For the ID hash for device names, this is indeed "hard" to predict them as for example in the case of a udev device, it will be a hash of the node hostname and the sys path of the device, the best is to not need to predict them, if you explain a bit more why you want that hash beforehand, we may find a way without it :sweat_smile:
About the ability to deploy other k8s objects, there is already an issue about that (#542) and a proposal in that direction (project-akri/akri-docs#75), but this is still in planning phase.

Awesome !
Thanks for your quick answers, i will go forward with my POC ;)
For the context, i wanted to explore some ways :

On a node, access 1 or more devices (as i said, like zigbee or z-wave or wathever)
in udev config, i put this kind or udevRules :

- 'KERNEL=="ttyACM[0-9]*", DRIVERS=="usb", ATTRS{idVendor}=="1a86", ATTRS{idProduct}=="55d4"'
- 'KERNEL=="ttyACM[0-9]*", DRIVERS=="usb", ATTRS{idVendor}=="1xxxxx", ATTRS{idProduct}=="5765"' # whatever

the brokerPod and service attached :

      brokerPod:
        image:
          repository: tetragramato/akri-ser2net
          tag: 0.1.0
          pullPolicy: Always
      createInstanceServices: true
      createConfigurationService: false
      instanceService:
        portName: tcp
        port: 20108
        targetPort: 20108

it works fine, and i think it's a good way to consume devices on my own code/stuff (with ser2net, etc).
Btw, it should pop 2 brokerPod and 2 services (and it's fine and good, it's what we want !), but i need to run zigbee2mqtt for the zigbee device, and another software for z-wave device 🤔  In the same way, i think it could be awesome to have zigbee2mqtt disappear if the device disappear 🤔
So i wonder if it's not better if i try to have zigbee2mqtt as a brokerPod, but for that i need to know exactly what kind of device i mount 🤔 (that why i asked if we could have a list of udev config )
That's why i asked here, to have your opinion and be sure i'm in "state of the art" !
BTW you gave me some clues, thanks !


pub roller PRs should not require human approval provided that presubmits pass

Skia autoroll PRs are allowed to auto-merge provided presubmits all pass. This should be enabled for pub autoroll PRs as well.

[x] created new GitHub account: https://github.com/flutter-pub-roller-bot
[x] updated cloudKMS key to a new PAT for the new account
[x] update recipe to stop passing deprecated CLI argument https://flutter-review.googlesource.com/c/recipes/+/46722
[x] https://github.com/flutter/flutter/pull/129844
[x] public documentation https://github.com/flutter/flutter
[x] https://github.com/flutter/.github/pull/25
[x] registor bot with go/github-robot-accounts
[x] get CLAbot approval b/290661269
[x] mark .ci.yaml target as non-bring-up https://github.com/flutter/flutter/pull/130088
[x] internal setup documentation cl/547246229

Done


Automated cherry pick of #6123: Install Multicast related iptables rules only on

Cherry pick of #6123 on release-1.14.
#6123: Install Multicast related iptables rules only on
For details on the cherry pick process, see the cherry pick requests page.

/test-all

/test-ipv6-e2e
/test-ipv6-only-conformance
/test-multicluster-e2e


Remove metadata from snapshot/clone/parent RBD image

Describe what this PR does
Remove metadata on images for various scenarios:

Unset the PVC metadata on the rbd image created for the snapshot
Unset the snapshot metadata from the rbd image created from the snapshot
Unset the parent PVC metadata on the temp clone rbd image
Also add various testcases to validate unset of metadata

Related issues
Fixes: #2970

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)
/retest all: run this in case the CentOS CI failed to start/report any test
progress or results

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/k8s-e2e-external-storage/1.22

/retest ci/centos/mini-e2e-helm/k8s-1.22

@Mergifyio rebase

@Mergifyio rebase

@Mergifyio rebase

/retest ci/centos/mini-e2e-helm/k8s-1.22

/retest ci/centos/k8s-e2e-external-storage/1.21

/retest ci/centos/mini-e2e-helm/k8s-1.21

/retest ci/centos/mini-e2e-helm/k8s-1.22

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.23

@Madhu-1 @nixpanic @ceph/ceph-csi-contributors please review this changes. Thanks!

/retest all

/retest ci/centos/k8s-e2e-external-storage/1.22

/retest ci/centos/mini-e2e/k8s-1.21

@nixpanic addressed your review comments now, PTAL. Thanks!

/retest ci/centos/mini-e2e-helm/k8s-1.21

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.22

@Mergifyio rebase

/retest ci/centos/mini-e2e-helm/k8s-1.23

@Mergify rebase

@Madhu-1 @nixpanic PTAL

@Madhu-1 @nixpanic PTAL

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/mini-e2e-helm/k8s-1.21

@Mergifyio rebase

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/mini-e2e-helm/k8s-1.21

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

This can be rebased with Mergify, just have CI finish testing and merging other PRs.

@mergifyio rebase

Diff:
V3: https://github.com/ceph/ceph-csi/pull/3005#discussion_r866707138 is not working, so replaced errno with string comparison for now. Added a TODO, will address in future PRs

/retest ci/centos/mini-e2e-helm/k8s-1.21
/retest ci/centos/mini-e2e-helm/k8s-1.23
May 11 10:33:45.243: FAIL: backend images not matching kubernetes resource count,image count 23 kubernetes resource count 21

/retest ci/centos/mini-e2e-helm/k8s-1.21

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest all

/retest all

It seems the CI jobs got stuck somehow. This PR is the 1st in the queue to be merged, hopefully things pass quickly.

@mergifyio unqueue
#3070 has passed CI, and should be able to get merged immediately. Let's try the new dequeue command

@mergifyio rebase

@mergifyio requeue
The other PR got merged, so this one is next now. Restarted CI jobs after rebasing.

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.23

Failed with:
    May 12 09:46:29.815: backend images not matching kubernetes resource count,image count 23 kubernetes resource count 21

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.23

Failed with:
May 12 10:50:53.103: FAIL: backend images not matching kubernetes resource count,image count 12 kubernetes resource count 11

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.23

Failed with:
May 12 12:14:55.114: FAIL: backend images not matching kubernetes resource count,image count 22 kubernetes resource count 21


Implement Error Bingo mapping for nextsteps generation

Adds a troubleshooting lookup keyword that can "suggest" nextsteps based on a error bingo board, called RWNextSteps.Suggest
Adds RW.NextSteps.Format keyword to allow for quick injection of live object values/names into nextsteps message before issue submission to reduce robot bloat
Board is based on a maintained static yaml mapping file, specific to each system/platform (eg: 1 for Kubernetes, 1 for ECS, 1 for Google Cloudrun, etc)
Adds Run Bash File keyword

See https://github.com/runwhen-contrib/rw-cli-codecollection/blob/err-bingo/codebundles/cmd-test/runbook.robot for samples

Example walkthrough of troubleshooting a postgres issue based on API logs:

using kubectl get logs in a previous line, a author could get OperationalError: FATAL: connection limit exceeded from their API logs
They then provide this to the Suggest keyword like so:

    ${next_steps}=    RW.NextSteps.Suggest    OperationalError: FATAL: connection limit exceeded for non-superusers

This performs a lookup in the troubleshooting mapping for Kubernetes, using a fuzzy match (levenshtein distance) this will match to https://github.com/runwhen-contrib/rw-cli-codecollection/blob/err-bingo/libraries/RW/NextSteps/Kubernetes/mapping.yaml#L44 which is an alias for https://github.com/runwhen-contrib/rw-cli-codecollection/blob/err-bingo/libraries/RW/NextSteps/Kubernetes/mapping.yaml#L33 producing those tasks+object entries

The produce next step will be organized to the following string:
Get Standard Postgres Resource Information, Describe Postgres Custom Resources, Get Postgres Pod Logs & Events, Get Postgres Pod Resource Utilization, Get Running Postgres Configuration, Get Patroni Output, Run DB Debug Queries

StatefulSet:$postgres_name

Either referencing codebundle configuration or pulling the object name with a CLI command, we can inject the database name into the above message like so:

    ${next_steps}=    RW.NextSteps.Format    ${next_steps}
    ...    postgres_name=${db_name.stdout}

The ${next_steps} string now adheres to the format requested by the platform team and is ready for submission in an issue.
This should:

reduce robot code bloat and repeated string format calls
leaves the next_steps formatting up to a keyword which we can update to match the platform's needs if needed in the future
allows platform troubleshooting experts to update mappings over time while other authors simply pull from the mapping with error messages
multiple mappings can be intertwined so troubleshooting next_steps can be generated for up the stack or down the stack relative to the current error being investigated


On Update part

For Update part, there is a mistake:
fmt.Printf("Deleted %d row(s) of data.\n", rowCount)

Should be
fmt.Printf("Updated %d row(s) of data.\n", rowCount)

@proyb6, Thank you for pointing this out. It would be great if you can share the document where we see this mistake so that we can fix that soon. Please do share the doc.

Under Update Data
https://docs.microsoft.com/en-sg/azure/mysql/connect-go

@proyb6 Thank you for pointing out the issue, I have updated the doc accordingly.
The changes should be live EOD or tomorrow. We will close this issue now.
In future, if you see any minor bug fixes in the doc, feel free to click on the edit button and submit a PR to update. We love taking contributions from the community.


Configurable tab bar and add "Album Artist" tab

I think we need to be able to edit the tab bar in a way that you could exclude some tabs from view and rearrange them. e.g. I want to only see "Queue | Artist" in the tab bar.
Also I would like to use an "Album Artist" tab because not always tracks are properly tagged using Artist.
I'm loving this project so far btw!

Hi, I am glad you are enjoying rmpc!
I am currently working on a feature that will allow you to do just that. I want to make tabs completely configurable, including merging two or more tabs into one. I am still figuring out how to do things the right way so it may take some time to finish.
For the Album Artist, I want to do that as well, just not sure what the right way to go about this is. Should we make it somehow configurable, add a new tab since they are going to be configurable anyway or just simply swap artist for albumartist. Simply swapping them should be fine, MPD  falls back to artist if albumartist is not present anyway, but I am leaning towards adding a new tab.

Hi, this is resolved with the merge of https://github.com/mierak/rmpc/pull/61. You can get your desired behavior by configuring tabs. It is now possible to reorder, rename and remove tabs as well as combine more panes(what were previously tabs are now called panes) into one.
I have also added an AlbumArtists pane to address your second point.
This config should get you what you want:
tabs: [
    (
        name: "Queue",
        border_type: None,
        pane: Split(
            direction: Horizontal,
            panes: [
                (
                    size: "40%",
                    pane: Pane(AlbumArt),
                ),
                (
                    size: "60%",
                    pane: Pane(Queue),
                ),
            ],
        ),
    ),
    (
        name: "Artists",
        border_type: None,
        pane: Pane(AlbumArtists),
    ),
],

Please let me know if you have any issues!


feat: split out rsx into its own crate

This PR moves separates out the rsx macro and its infrastructure to allow use in an rsx autoformatter and accompanying vscode extension.
This will be part of our 0.1.8 release train alongside the router, optional props, better docs, and more.
Right now, the crate is not released on crates.io but is being relied upon in the fork of the CLI

... more to come


feat: add log mr slog

add charm log with slog support
this depends on golang.org/x/exp/slog
this also changes the module path from github.com/charmbracelet/log to github.com/charmbracelet/x/exp/log

fwiw slog apparently will be in go1.21 👀

Log now supports log/slog


Deleting images from raw data breaks everything

Don't do it

Now deleting a file is disabled. For deleting a file the user needs to specify this action in Edit elaboration page and then start an elaboration.


Bugfix/fix e2e tests

Related issues
https://github.com/DivanteLtd/vue-storefront/issues/1860
Short description and why it's useful
It fixes e2e tests. Few informations about changes:

ReturnIcon component has had a copy - ReturnToMainIcon. Only difference was that second one redirected to home page. I removed this component and added optional props which works same as router-link.
ProductBundleOptions broke page on SSR case. I corrected it by adding computed error messages. But i left a todo to make ProductOption as child component in bundle because there is a complex logic. Haven't done that in this PR because it is about e2e tests.
there were missing imports in MinimalHeader which shows on error page. You'll see difference in screens.
the rest was just improvements in actual tests, if any of them is unclear just ask me to elaborate that case

Screenshots of visual changes before/after (if there are any)

Screenshot of passed e2e tests (if you are using our standard setup as a backend)

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility)
[ ] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

Contribution and curently important rules acceptance

[x] I read and followed contribution rules
[x] I read the TypeScript Action Plan and adjusted my PR according to it
[x] I read about Vue Storefront Modules and refactoring plan for them

Thanks! Could You please make the separate issue for: 'ProductBundleOptions broke page on SSR case. I corrected it by adding computed error messages. But i left a todo to make ProductOption as child component in bundle because there is a complex logic. Haven't done that in this PR because it is about e2e tests.'


Create a View to Register Users

Since, we will be making a login authentication system, we need a view to register users to the users app.
Tasks

[ ] Use the built-in UsersCreationForm to register users in Register View in users/views.py
[ ] create a register.html template in  users/templates/users/ folder.
[ ] render the form - created in the register view - in this html template
[ ] add the url path 'register/' for this view in users/views.py

i will do it

Sure @ayush1999guptA , I will assign this issue to you as soon as you accept the invitation through email.


buckets: remove transform tags from create bucket

I'm unsure what the clippy failures are :thinking:  - I don't seem to be reproducing them locally


Make it possible to inline assets into the resulting index.html

What
This PR adds a CLI-option -i Inline all assets in the resulting html-file, making it heavier, but more portable, passing this in will inline images in the resulting index.html
Why
When e.g. using this in a CI it's very useful to be able to have a single artifact that can be downloaded/stored in some way, this enables that.
We're e.g. using CircleCI, where it's possible to upload artifacts that can later be accessed as a webpage, having to upload the whole directory makes this very slow, and it also seems to occasionally drop links to images.

@felginep Would you mind taking another look at this?

@felginep Thanks, but I guess @TitouanVanBelle needs to approve it as well?

I’m not sure how to proceed with this, I can’t merge it, so I guess some maintainer needs to do that

Could you merge @TitouanVanBelle ?

@cheif Wait, it seems to me there is an issue when the .xcresult is in a deeper folder, for example xchtmlreport -r output/test_report/UI_Tests.xcresult -i

@TitouanVanBelle should be good now

@chrisballinger maybe ? (sorry for the notification, but it's hard to find anyone who can merge on this repo)

@cheif In the meantime, you can use our fork if you want the inlining feature: https://github.com/applidium/XCTestHTMLReport/releases/tag/2.1.0
Thanks again for the PR.


Python 2 EOL

When would be a good time to remove Python 2 support? It's been EOLed for a while.
Thank you!

Yes, we plan to remove Python 2 support in the next release (likely versioned as 1.0.0).

Sounds good, thank you!

v1.0.0 has dropped Python 2 support.

Great, thank you!

Is six still necessary with Python 2 support removed?

These will be cleaned up in future updates. Also internally we have a few very rare cases that still use parts of the library in Python 2, those will need cleanups too.


How to quantize the model?

Currently having issues attempting to quantize, save, then load the model using HF Transformers.
Is there any known working method for quantizing Aria (preferably to 4bit)?

@iamthemulti Quantizing the Aria model presents challenges due to its use of grouped-gemm for efficient inference and training with bfloat16, rather than standard nn.Linear layers. The grouped-gemm implementation can be found in the Aria repository:
https://github.com/rhymes-ai/Aria/blob/719ff4e52b727443cba3793b0e27fe64e0244fe1/aria/model/moe_lm.py#L444-L482
I'm currently working on a custom solution to address this quantization issue.

@iamthemulti I've uploaded a fork of aria model that replaces the grouped gemm with a sequential mlp, in which each expert is implemented as a torch.nn.Linear layer executed in sequence. This adjustment simplifies quantization with current open source libraries that are optimized for nn.Linear layers.
If you want to quantize an aria model, please use rhymes-ai/Aria-sequential_mlp
I am also trying to use some open-source tools to quantize the Aria model, but I'm encountering some issues on the H100. Currently, I don't have access to other GPUs for quantization.

Any updates on quants would be highly valuable @aria-hacker! Please keep us posted about your progress

I got a BitsAndBytes NF4 quant working here, requires less than 16 GB of VRAM and runs on an RTX 3090

I've uploaded an int8 weight-only model that has been quantized using torchao. It's also compatible with grouped-gemm. Feel free to try it out if you're interested!

Anyone else getting [ERROR|vllm_server.py:212:3614300] 2024-11-04 11:21:12,223 >> KeyError: 'language_model.layers.27.mlp.experts.experts.61.down_proj.weight’ while loading the MLP model via VLLM?

We have an HQQ  4-bit version working well with just 15GB of VRAM: https://github.com/mobiusml/hqq/blob/master/examples/hf/aria_multimodal.py


node-sass和sass-loader版本不对 导致npm install 报错

修改：

"node-sass": "^7.0.1",
"sass-loader": "^7.0.0"

报错详情：
npm ERR! gyp verb which failed Error: not found: python2
npm ERR! gyp verb which failed     at getNotFoundError (D:\code\zmcode\study-vue\mall-admin-web\node_modules\which\which.js:13:12)
npm ERR! gyp verb which failed     at F (D:\code\zmcode\study-vue\mall-admin-web\node_modules\which\which.js:68:19)
npm ERR! gyp verb which failed     at E (D:\code\zmcode\study-vue\mall-admin-web\node_modules\which\which.js:80:29)
npm ERR! gyp verb which failed     at D:\code\zmcode\study-vue\mall-admin-web\node_modules\which\which.js:89:16
npm ERR! gyp verb which failed     at D:\code\zmcode\study-vue\mall-admin-web\node_modules\isexe\index.js:42:5
npm ERR! gyp verb which failed     at D:\code\zmcode\study-vue\mall-admin-web\node_modules\isexe\windows.js:36:5
npm ERR! gyp verb which failed     at FSReqCallback.oncomplete (node:fs:198:21)
npm ERR! gyp verb which failed  python2 Error: not found: python2
npm ERR! gyp verb which failed     at getNotFoundError (D:\code\zmcode\study-vue\mall-admin-web\node_modules\which\which.js:13:12)
npm ERR! gyp verb which failed     at F (D:\code\zmcode\study-vue\mall-admin-web\node_modules\which\which.js:68:19)
npm ERR! gyp verb which failed     at E (D:\code\zmcode\study-vue\mall-admin-web\node_modules\which\which.js:80:29)
npm ERR! gyp verb which failed     at D:\code\zmcode\study-vue\mall-admin-web\node_modules\which\which.js:89:16
npm ERR! gyp verb which failed     at D:\code\zmcode\study-vue\mall-admin-web\node_modules\isexe\index.js:42:5
npm ERR! gyp verb which failed     at D:\code\zmcode\study-vue\mall-admin-web\node_modules\isexe\windows.js:36:5
npm ERR! gyp verb which failed     at FSReqCallback.oncomplete (node:fs:198:21) {
npm ERR! gyp verb which failed   code: 'ENOENT'
npm ERR! gyp verb which failed }
npm ERR! gyp verb check python checking for Python executable "python" in the PATH
npm ERR! gyp verb which succeeded python C:\Users\Administrator\AppData\Local\Programs\Python\Python38\python.EXE
npm ERR! gyp ERR! configure error

感谢

我改了一些依赖，把node-sass换成sass了。用的环境是os:win11, nodejs版本：18.20.4，装了vs2019
"dependencies": {
"axios": "^0.18.0",
"echarts": "^4.2.0-rc.2",
"element-ui": "^2.3.7",
"js-cookie": "^2.2.0",
"normalize.css": "^8.0.1",
"nprogress": "^0.2.0",
"v-charts": "^1.19.0",
"v-distpicker": "^1.0.20",
"vue": "^2.5.2",
"vue-router": "^3.0.1",
"vuex": "^3.0.1"
},
"devDependencies": {
"autoprefixer": "^7.1.2",
"babel-core": "^6.26.3",
"babel-helper-vue-jsx-merge-props": "^2.0.3",
"babel-loader": "^7.1.1",
"babel-plugin-syntax-jsx": "^6.18.0",
"babel-plugin-transform-runtime": "^6.22.0",
"babel-plugin-transform-vue-jsx": "^3.5.0",
"babel-preset-env": "^1.3.2",
"babel-preset-stage-2": "^6.22.0",
"chalk": "^2.0.1",
"copy-webpack-plugin": "^4.0.1",
"css-loader": "^0.28.10",
"extract-text-webpack-plugin": "^3.0.2",
"file-loader": "^1.1.4",
"friendly-errors-webpack-plugin": "^1.7.0",
"html-webpack-plugin": "^3.2.0",
"node-notifier": "^5.1.2",
"sass": "^1.56.1",
"optimize-css-assets-webpack-plugin": "^3.2.0",
"ora": "^1.2.0",
"portfinder": "^1.0.32",
"postcss-import": "^11.0.0",
"postcss-loader": "^2.0.8",
"postcss-url": "^7.2.1",
"rimraf": "^2.6.0",
"sass-loader": "^7.0.1",
"semver": "^5.3.0",
"shelljs": "^0.8.5",
"svg-sprite-loader": "^3.7.3",
"uglifyjs-webpack-plugin": "^1.1.1",
"url-loader": "^0.5.8",
"vue-loader": "^14.2.4",
"vue-style-loader": "^3.0.1",
"vue-template-compiler": "^2.5.2",
"webpack": "^3.12.0",
"webpack-bundle-analyzer": "^2.9.0",
"webpack-dev-server": "^2.11.5",
"webpack-merge": "^4.1.0"
},


PhotonVision and pose estimation

See discord discussion on multitag performance issues: https://discord.com/channels/176186766946992128/368993897495527424/1087615047875956786
PhotonPoseEstimator/multitag is potentially very slow and may create loop overruns. We should put the pose estimation code in a separate thread. Maybe we should borrow 7028's implementation:
https://github.com/STMARobotics/frc-7028-2023/blob/main/src/main/java/frc/robot/subsystems/PhotonRunnable.java
https://github.com/STMARobotics/frc-7028-2023/blob/main/src/main/java/frc/robot/subsystems/PoseEstimatorSubsystem.java
We should also take a look at their std deviations because ours are probably wrong.
Additional PhotonVision links
Pose estimation: https://docs.photonvision.org/en/latest/docs/programming/photonlib/robot-pose-estimator.html
Setting to driver mode (if we want to use camera for driving):
https://docs.photonvision.org/en/latest/docs/programming/photonlib/driver-mode-pipeline-index.html
Do we have NetworkTables server on/off in PhotonVision? It's recommended that we turn this off for PhotonLib to work properly: https://docs.photonvision.org/en/latest/docs/programming/nt-api.html
Are we using 3D mode? Is our camera calibrated properly? We should do these things and retune our pipeline: https://docs.photonvision.org/en/latest/docs/integration/aprilTagStrategies.html
Calibrating camera: https://docs.photonvision.org/en/latest/docs/getting-started/pipeline-tuning/calibration.html PhotonVision docs recommend using calibdb and 960x720 resolution with 3D mode.
Exporting config: https://docs.photonvision.org/en/latest/docs/hardware/config.html#directory-structure
Logs: https://docs.photonvision.org/en/latest/docs/troubleshooting/logging.html Logs can be viewed live by pressing the ` key

We should make sure our network setup looks like this to prevent connection issues:

https://www.chiefdelphi.com/t/why-you-probably-shouldnt-use-the-second-port-on-your-openmesh-om5p-radio-and-embrace-using-an-ethernet-switch-instead/406374/24?u=amoryg


Add instructions and easy batch script to make it easier for newbies

Hey Daniel,
I don't know if it would be useful at all but I made a Batch Script and added some instructions right at the bottom as I've seen a couple of issues opened as they don't know how to use it.
Let me know if any adjustments need to be made, I think it would be pretty helpful for those who are new and want to try it out.

Cool, but not everyone has python installed
Others do not know how to install it or maybe the PC is not theirs and they are not allowed to install it
Could it be make in C in an .exe?
For example, I use
https://github.com/danielkrupinski/curiuminjector-csgo
and I wish I could add it in there
Regards

Cool, but not everyone has python installed
Others do not know how to install it or maybe the PC is not theirs and they are not allowed to install it
Could it be make in C in an .exe?
For example, I use
https://github.com/danielkrupinski/curiuminjector-csgo
and I wish I could add it in there
Regards

You can work with the implementations Daniel added in the readme to work that out

Cool, but not everyone has python installed
Others do not know how to install it or maybe the PC is not theirs and they are not allowed to install it
Could it be make in C in an .exe?
For example, I use
https://github.com/danielkrupinski/curiuminjector-csgo
and I wish I could add it in there
Regards

You can work with the implementations Daniel added in the readme to work that out

I have tried but it gives me error. I am new at this


[SPARK-44623][BUILD] Upgrade commons-lang3 to 3.13.0

What changes were proposed in this pull request?
This PR aims to upgrade commons-lang3 to 3.13.0.
Why are the changes needed?
commons-lang3 added Java 21 support at 3.13.0.

https://commons.apache.org/proper/commons-lang/changes-report.html#a3.13.0

https://github.com/apache/commons-lang/blob/9d85b0a11e5dbadd5da20865c3dd3f8ef4668c7d/src/main/java/org/apache/commons/lang3/JavaVersion.java#L167-L172

    /**
     * Java 21.
     *
     * @since 3.13.0
     */
    JAVA_21(21, "21"),

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs.

Thank you, @HyukjinKwon .

Merged to master for Apache Spark 4.0.


use trueHeading instead of magneticHeading

Agree that they should match. Don't agree that this should be reverted. Android should be migrated to provide true heading

I actually agree. I should have added a new field for trueHeading instead of changing the existing one.


AttributeError: 'module' object has no attribute 'Consumer'

I tried to use oauth library for my application, that will parse tweets from an account, used instructions from here https://dev.twitter.com/oauth/overview/single-user:
def oauth_req(url, key, secret, http_method="GET", post_body=None, http_headers=None):
    consumer = oauth.Consumer(key=key, secret=secret)
    token = oauth.Token(key=key, secret=secret)
    client = oauth.Client(consumer, token)
    resp, content = client.request( url, method=http_method, body=post_body, headers=http_headers, force_auth_header=True )
    return content

def get_tweets():
    timeline = oauth_req('https://api.twitter.com/1.1/statuses/user_timeline.json?screen_name=climagic', '3WTohSoXhJrmmk61oOwmJBwQp', 'FlqER9N3qoYl89bPTZ8EP9CWXBtttkikaYeE9pqIyCwX7GSNYA' )
    print(timeline)

But it doesn't work because of absence of attribute 'Consumer'.

Consumer is a class not an attribute.
Can you update with the error you get

Consumer is a class not an attribute and it does exist:
https://github.com/simplegeo/python-oauth2/blob/master/oauth2/__init__.py#L175
You probably just need to import oauth2
import oauth2 as oauth

I got an error
$ python3 script.py
Traceback (most recent call last):
  File "script.py", line 1, in <module>
    import clone
  File "/Users/artemdremov/Documents/grabber/proglib/clone.py", line 7, in <module>
    import oauth2 as oauth
  File "<frozen importlib._bootstrap>", line 2237, in _find_and_load
  File "<frozen importlib._bootstrap>", line 2222, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 2164, in _find_spec
  File "<frozen importlib._bootstrap>", line 1940, in find_spec
  File "<frozen importlib._bootstrap>", line 1916, in _get_spec
  File "<frozen importlib._bootstrap>", line 1897, in _legacy_get_spec
  File "<frozen importlib._bootstrap>", line 863, in spec_from_loader
  File "<frozen importlib._bootstrap>", line 904, in spec_from_file_location
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/oauth2-1.5.211-py3.4.egg/oauth2/__init__.py", line 105
    except UnicodeDecodeError, le:
                             ^
SyntaxError: invalid syntax

I got the same issue:
AttributeError: 'module' object has no attribute 'Consumer'

But I imported it:
import oauth2 as oauth

I installed it earlier on through
sudo pip install python-oauth2

I also checked with print 'oauth2' in sys.modules that the module is imported. Then by using print oauth I got the path to the module and realized, that it is a totally different one, than this one, which you refered to. Mine is only 230 lines.
Is it possible, that I somehow installed something wrong?

yeah, seems to work. I downloaded the __init__.py and replaced mine with the new one. Then I commented these two lines out, since I didn't find any further usage of it
#import _version
#__version__ = _verion.__version__

And now it worked. HURRAY!

@xamiax if you're issue is resolved can you please close the issue? Thanks

I can't, since I didn't opened this issue...

@xamiax I have followed your solution and my application now working fine. Thanks

glad my comment was helpful to someone! :D

I have the similar issue with Anaconda and Python 3.5. at my terminal, when i type pip3 install oauth2 - i could not find the repository. and i try to specify the exact wheel by downloading it from pypl wheel and install it (though it says python 3.4)...when i run the twitter sample code, it still gives me the same error message:

consumer = oauth2.Consumer(key=CONSUMER_KEY, secret=CONSUMER_SECRET)
AttributeError: module 'oauth2' has no attribute 'Consumer'

pip install oauth2 solved the problem for me.

@xamiax Thank you so much. Your solution help me save so much time.

i've downloaded it and replaced it, but it says ModuleNotFoundError: No module named 'oauth2._compat'

n I commented these two lines out, since I didn't find any further usage of it

[xamiax] @xamiax your solution worked for me too. (I can't believe it worked... but yeah.


[Bug] Can't initialize windows workspace under the "Research-GPC" account in Production

I am not able to initialize windows workspace under project "Research-GPC-Main-Project", either associating with a study or not:

Note that Research-GPC-Main-Project is associated with the external account "Research-GPC" using CFN template following the steps described in: https://github.com/awslabs/service-workbench-on-aws/blob/mainline/docs/Service_Workbench_User_Guide.pdf (page 9). There may be some fundamental healthy issue with either that external account or how the association was made in the first place, given multiple failure incidents recently (related to #15).

when we have multiple accounts where SWB is deploying workspaces, like Research-GPC, we have to remember to update all accounts with role policy customizations we make, not just the main account.  I had not added the policy customization that allows the EC2 auto-stop pieces in the template to work, under Research-GPC, and Aaron pointed it out to me.  That's the cause of the newly failing workspace launches against that account.  I've got the policy customization added now, and am testing Windows workspace launch against Research-GPC.

Hmm, also looks like I need to update the policy under the Research-GPC account to explicitly allow "cloudwatch:PutMetricAlarm" ... so something differs from how it's working under the primary SWB account too.  We should have tested this under App-Dev against Research-GPC before pushing it into App-Prod :(

This may take longer to figure out... still getting this error in the stack despite definitely providing that permission in the role policy...
API: cloudwatch:PutMetricAlarm User: arn:aws:sts::994262031937:assumed-role/prod-oh-umbmi-swb-LaunchConstraint/servicecatalog is not authorized to perform: cloudwatch:PutMetricAlarm on resource: arn:aws:cloudwatch:us-east-2:994262031937:alarm:SC-994262031937-pp-kuallhi75cpxa-IdleCPUAlarm-WI0930LJF9H6 because no identity-based policy allows the cloudwatch:PutMetricAlarm action

Double-checking the resource targets on the policy...

Discovered the AddCloudWatchAlarm inline policy on the LaunchConstraint role is getting changed back to the one from the Application-Prod account role (i.e. replacing the Research-GPC project ID with the Application-Prod project ID in the resource specifications).  I think I know how to work around this, implementing the policy as an attached one, and making sure there is no policy name collision.  Implementing and testing now.

Confirmed that because the AddCloudWatchAlarm inline policy name was the same in both the Application-Prod and Research-GPC accounts, when the workspace template launches it was overwriting the policy, so naming it distinctly for the account it is applicable to should fix the issue. Testing again.

Well, dangit, that didn't work either.  The separately named inline policy on the role was removed upon workspace launch... it doesn't like that role drift, it feels like.  Hmm.  I will need to think on this and perhaps consult with our AWS-side experts.

Should we temporarily remove the Research-GPC project/index from that account, and recreate it against the main account, so that people can continue to launch workspaces without unexpected errors (from their perspective), in the meantime? @sxinger

Wait, possible eureka...

FIXED IT!
Made the requisite policy change to include the Research-GPC account, on the inline policy for the LaunchConstraint role under Application-Prod, and it carried over into the role under Research-GPC this time 😊

There are only two studies under Research-GPC account: GROUSE-MUIRB2057285-Obesity , GROUSE-MUIRB2057285-ALS (both with me involved). I tested it out by creating a workspace under each study and did the following usability tests:

saving data to S3 bucket - worked
snowflake db connectivity - worked
same workspace associated with multiple studies - worked
getting data from S3 bucket - worked

@fergusonsha-umkc one little caveat I noticed was that, does rclone only sync once at workspace creation? If I manually upload something into the corresponding S3 study-data bucket, it doesn't sync right way or even after I stop and re-start the workspace:

But if I create a brand new workspace, the uploaded file is synced.
PS: Please don't change the original class of the issue (i.e., Bug, Feature, or Misc). Closing it with comment would suggest fixing of the issue.

That's not an rclone workspace, it's an s3-synchronizer workspace.  Rclone is only fully implemented in dev still, while we stress test it.  Also, did you mean to post rclone stuff under this issue, which is unrelated to rclone?
I wasn't able to change the status of this bug-fix to "In Validation," like I am able to with other issues, thus I updated the title to indicate original problem resolution and was monitoring it before closure.  I left the original class of the issue intact, as Bug, and removed only the "Urgent" status, as it was no longer so.

I can tell it's not an rclone workspace because D: is showing itself as a physical disk in the File Explorer, and does not have the study label, FYI.

Since the workspace launching against Research-GPC is fixed, I'm closing this issue.


Add #ifdef conditions to CFPlatform.c for WASI

WASI doesn't support multi-threading and process info or environment variables. Here we #ifdef relevant code out, or replace with stubs that allow Foundation to be compiled for WASI.

@swift-ci please test


Fix tests with Julia 0.5

Looks like this also fixed tests for 0.3, no idea why...

Are there any tests for ordering using a vector of different functions? Would Vector{S}, S<:Function declaration still work in that case?

Are there any tests for ordering using a vector of different functions? Would Vector{S}, S<:Function declaration still work in that case?

I don't know whether it's correctly tested, but it should work:

julia> f{T<:Function}(x::AbstractVector{T}) = x
f([meg (generic function with 1 method)

julia> f([mean,sum])
2-element Array{Function,1}:
 mean
 sum

@nalimilan Ah, right. So the updated declaration would potentially optimize the code when all functions are the same.

@nalimilan Ah, right. So the updated declaration would potentially optimize the code when all functions are the same.

The declaration wouldn't change anything AFAIK (we could also completely get rid of type assertions), but indeed the new Julia makes passing functions more efficient. I'm not sure whether the fact that all functions are the same would make a difference or not.

I thought in 0.5 each function (e.g. mean) would have its own unique type, so if T/S is resolved to this type, julia might do some call optimizations.

Yes, but that might also happen even when an array contains different functions, if we pass each function individually to another one, so that type inference is applied separately. We'd have to check that.

OK to merge? This makes all PRs fail the tests.

LGTM


Translation also translated the module names

Have looked at the German translation of settings. Have seen that the module names were translated on some translations. This leads to interesting names and inconsistencies:
Example

Original
Translsted into German
Comment

About FacyZones
Info zu FancyZones

About Color Picker
Info zum Farbwähler

About Image Resizer
Info zur Bildgrößenänderung
Translated text says "Information on how to change image size" and not "Info about Image resizer tool"

About PowerToys Run
Info zum PowerToys-Startprogramm

About File Explorer
Info zum Datei-Explorer
Translation says "Information about File Explorer". What about renaming into "File Explorer Addons"?

Originally posted by @htcfreek in https://github.com/microsoft/PowerToys/issues/199#issuecomment-709582005

The problem is that we get different namings of the PT modules and the translation leads to incorrect meanings.

@htcfreek
can you provide some screenshots of the places where they got translated?

@enricogior
I have checked all translation files I could find. Here is a list with files that contain translated or not translated module names.

Settings translation (German)
PT Run translation (German)
Color Picker translation (German)
FancyZones-Editor translation (German)
Image Resizer translation (German)

Screenshots of some translations
coming soon
I hope that I don't have missed something.

Yes, actual names should not be translated. It's too weird.

Yes, actual names should not be translated. It's too weird.

I personally would find it strange if the names get translated in the future.

@enricogior
How is this prioritized? I think it should be fixed before releasing 0.25.0!

@crutkas
have we decided which modules names should be translated? (none/some/all)
If some/all will be translated can we document which one so we will know if we need to put them in the resx files or not.
Problem is, some names cannot be translated literally otherwise they would become hilarious (FancyZones in particular), who will come up with acceptable/meaningful name?

@enricogior, @crutkas
Any updates on this issue?
@crutkas
For which date do you planning to release 0.25? This month or next month?

One more example: the text from Power Rename in the shell context menu

@enricogior no module name should be translated

The actually top issue translation matches the English.  The root of don't translate certain items we need to address


byref sometimes just returns 'R' as encoding

Ran into some interesting cases where byref was not returning the expected values
and OCMTypeWithoutQualifiers was running into issues and was actually returning the
next type in a string.

Thank you. Interesting catch! I've obviously merged this PR but I did want to say that I reduced the test to the bare necessities. I like my tests damp, not dry), and I think the test should only cover what's in the code; instead of confirming some other assumptions about the runtime, too.


Fix message formatting

Fix message formatting when it includes a title.  The message is indented by a single space, which looks a little untidy.

PR updated to include changes to improve file upload process

This should resolve: https://github.com/t0mer/green-api-custom-notifier/issues


chore: Remove unused Amplify managed table & RDS transformer & generator

Description of changes
Removes the SQL and Amplify-managed DDB table logic from @aws-amplify/graphql-model-transformer:

RDS & Amplify-managed generators
SQL lambdas
RDS-related E2E tests

This PR is not technically a prerequisite for it, but I'm doing this as part of the work for https://github.com/aws-amplify/amplify-category-api/issues/3006.
Although the functional changes in this PR are limited to @aws-amplify/graphql-model-transformer, it has broad-reaching implications for tests elsewhere, since we make heavy use of snapshot tests and the generated resources will no longer contain SQL or Amplify-managed resolvers. Thus, this PR touches or deletes 91(!) files. This PR simply removes RDS & Amplify-managed DDB tests.
Finally, removing the RDS tests reduced coverage in some cases; those coverage thresholds have been adjusted down accordingly.
Issue #, if available
https://github.com/aws-amplify/amplify-category-api/issues/3006
Description of how you validated changes

Local tests pass
E2Es pass

Checklist

[x] PR description included
[x] yarn test passes
[ ] E2E test run passes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Gen2 RDS tests still use Gen 1 RDS functionality to provision the stack. Once we get all of our tests no longer using Gen 1 CLI, we can revisit this effort.


Unable to deploy Kubernetes cluster to existing VNet

Is this a request for help?:
YES
Is this an ISSUE or FEATURE REQUEST? (choose one):
ISSUE
What version of acs-engine?:
HEAD

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes
What happened:
Deployment failed. Correlation ID: a35dedad-094c-4897-a311-2857581c00c8. {
"status": "Failed",
"error": {
"code": "ResourceDeploymentFailure",
"message": "The resource operation completed with terminal provisioning state 'Failed'.",
"details": [
{
"code": "VMExtensionProvisioningError",
"message": "VM has reported a failure when processing extension 'cse1'. Error message: "Enable failed: failed to execute command: command terminated with exit status=5\n[stdout]\n\n[stderr]\nstat: cannot stat '/opt/azure/containers/provision.complete': No such file or directory\n"."
}
]
}
}
What you expected to happen:
Cluster deployed in existing VNet
How to reproduce it (as minimally and precisely as possible):

Create VNet

Create subnets

build acs-engine according to instructions found here:
https://github.com/Azure/acs-engine/blob/master/docs/acsengine.md

Generate ARM templates:
./bin/acs-engine generate config.json
see config.json below.

create cluster
az group deployment create -g RES_GROUP --name "ClusterDeployment" --template-file azuredeploy.json --parameters<EMAIL_ADDRESS>

config.json
{
"apiVersion": "vlabs",
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes"
},
"masterProfile": {
"count": 3,
"dnsPrefix": "test123",
"vmSize": "Standard_D2_v2",
"vnetSubnetId": "/subscriptions/SUB_ID/resourceGroups/RES_GROUP_ID/providers/Microsoft.Network/virtualNetworks/VNET_NAME/subnets/SUBNET_NAME",
"firstConsecutiveStaticIP": "<IP_ADDRESS>"
},
"agentPoolProfiles": [
{
"name": "agentprivate",
"count": 3,
"vmSize": "Standard_D2_v2",
"availabilityProfile": "AvailabilitySet",
"vnetSubnetId": "/subscriptions/SUB_ID/resourceGroups/RES_GROUP_ID/providers/Microsoft.Network/virtualNetworks/VNET_NAME/subnets/SUBNET_NAME"
}
],
"linuxProfile": {
"adminUsername": "azureuser",
"ssh": {
"publicKeys": [
{
"keyData": "ssh-rsa<EMAIL_ADDRESS>}
]
}
},
"servicePrincipalProfile": {
"clientId": "CLIENT_ID",
"secret": "PASSWORD"
}
}
}
Anything else we need to know:

Try adding vnetCidr into the masterProfile section with your master subnet cidr. From memory i had a similar issue and found that the subnet was used in the provisioning scripts and was required.

Never resolved. We decided to abandon the unmanaged container service. Eventually we abandoned azure itself. Went with AWS.

@CecileRobertMichon Is there any workaround / solution for this? I am hitting this issue:
    "code": "VMExtensionProvisioningError",
    "message": "VM has reported a failure when processing extension 'cse-master-0'. Error message: \"Enable failed: failed to execute command: command terminated with exit status=4\n[stdout]\n\n[stderr]\n\"."

@ashvindeodhar a lot has changed since this issue was opened. Could please follow the steps in https://github.com/Azure/acs-engine/blob/master/docs/kubernetes/troubleshooting.md#vmextensionprovisioningerror-or-vmextensionprovisioningtimeout and open a new issue? I'll close this one.

@CecileRobertMichon - opened a new issue: https://github.com/Azure/acs-engine/issues/3730


[Tasks] — Release 1.2 preparations:

Description
Preparing to release 1.2, those are the current tasks for the code freeze:
Checklist

[ ] add to Wiki: AZ – @adarovadya
[ ] Fix main node object doc https://github.com/valkey-io/valkey-glide/issues/2729
[ ] fix and release python rc @adarovadya
[ ] Release rc for Java - https://github.com/valkey-io/valkey-glide/issues/2534 @acarbonetto
[ ] fix multi key bug https://github.com/valkey-io/valkey-glide/issues/2714 @eifrah-aws
[ ] add to Wiki: password update - @avifenesh  https://github.com/valkey-io/valkey-glide/issues/2692
[ ] https://github.com/valkey-io/valkey-glide/issues/2377 API doc autogen @eifrah-aws
[ ] add to wiki - active connections https://github.com/valkey-io/valkey-glide/issues/2376 - @eifrah-aws
[ ] Improve setup glide instruction - https://github.com/valkey-io/valkey-glide/issues/2371
[ ] Wiki - add import statements to examples https://github.com/valkey-io/valkey-glide/issues/2309
[ ] Wiki - add timeout config to all example and add recommendation to always set
[ ] Docs for error handling - https://github.com/valkey-io/valkey-glide/issues/1746
[ ] Wiki - add modules
[ ] Update docs for config get and config set - https://github.com/valkey-io/valkey-glide/issues/2192 - @Yury-Fridlyand
[ ] Fix tests - https://github.com/valkey-io/valkey-glide/issues?q=is%3Aopen+is%3Aissue+label%3A1_2_candidate+label%3A1_2_candidate+label%3Atesting%2CFlaky-tests
[ ] Prepare release notes - Whats new: Features, optimization, bug fixes, Other, breaking changes, Team: contributors, first contributions, Docks: Links to blogs, wiki etc
[ ] Backport to main - @ikolomi

Additional Notes
No response

@avifenesh , why is - https://github.com/valkey-io/valkey-glide/issues?q=is%3Aopen+is%3Aissue+label%3A1_2_candidate+label%3A1_2_candidate+label%3Atesting%2CFlaky-tests in here.
We should take it as port release effort

@asafpamzn you have to pass all tests before release, doesn't make sense otherwise.


State class variable is not shareing across pages and tabs

Describe the bug
The state variable is not shared between the pages.
To Reproduce
The string variable is declared as a string in the state class.

Code/Link to Repo:
code attached here as a python file code.txt

Expected behavior

The class variable should be shared across the pages and tabs.

Screenshots

Specifics (please complete the following information):

Python Version: Python 3.11.3
Reflex Version: 0.2.8
OS: Microsoft Windows 10 Pro
Browser (Optional): Google Chrome

Additional context
Your effort on this library is literally awesome. I'm a big fan of your Reflex library.
but Your examples in the main documentation of the Reflex web page are not sufficient. If you provide more examples with different use cases for each component would be greatly helpful.
code.txt

In reflex, each tab gets its own state. So it's "expected" that opening the app in two different tabs will result in each tab storing colorState independently.
However, within a single tab when I navigate between page1 and page2, I do see that the state is consistent; going to page1 and clicking the button makes it red, then on page2 I still see red.
There is not a trivial way to link 2 states together... however you may consider the use of rx.LocalStorage or rx.Cookie which leverages the client browser to syncronize values across tabs.
Changing the colorState definition to colorState: str = rx.Cookie("green") gets a little closer to the behavior you want... however it still doesn't live update across tabs; but opening a new tab retains the value set on a previous tab and changing the value on the new tab then refreshing the previous tab will show the value from the new tab.
There is not currently a simple mechanism in reflex to keep the state of two or more tabs in sync live, because the reflex backend doesn't really know that any two states belong to the same user.
If this is something that you want, then you can

set a cookie in the client browser
associate all "active" sessions with that cookie on initial page load
use app.modify_state to explicitly update all associated states when any one state changes.
track disconnects and update session tracking dicts accordingly

I should stress that this is kind of a hack, but it does at least use public APIs.
Here is a bit of sample code to help if you want to go down this road:
import reflex as rx

# Keep track of tokens associated with the same client browser ("shared" sessions)
shared_sessions_by_token: dict[str, set[str]] = {}

# Keep track of (shared_token, state_token) pairs for each websocket connection (sid)
tokens_by_sid: dict[str, tuple[str, str]] = {}

class State(rx.State):
    # The clientToken is saved in the browser and identifies "shared" sessions
    clientToken: str = rx.Cookie("")

    # The colorState is a special variable that is shared among all sessions with the same clientToken
    colorState: str = "green"

    def ChangePage1Color(self):
        self.colorState = "red"

        # Apply changes to all other shared sessions
        return State.set_color_state_for_shared_sessions

    def ChangePage2Color(self):
        self.colorState = "yellow"

        # Apply changes to all other shared sessions
        return State.set_color_state_for_shared_sessions

    async def set_color_state_for_shared_sessions(self):
        """Iterate through all shared sessions and update them with the new colorState."""
        if not self.clientToken:
            self.set_client_token()

        print(f"{self.clientToken} -> {shared_sessions_by_token[self.clientToken]}")

        for token in shared_sessions_by_token.get(self.clientToken, set()):
            if token != self.get_token():
                async with app.modify_state(token) as other_state:
                    other_state.colorState = self.colorState

    async def set_color_state_for_new_session(self):
        """When a new session is created, copy the colorState from another shared session."""
        for token in shared_sessions_by_token.get(self.clientToken, set()):
            if token != self.get_token():
                async with app.modify_state(token) as other_state:
                    self.colorState = other_state.colorState
                    return

    def set_client_token(self):
        """Page on_load handler uses the clientToken cookie to identify shared sessions."""
        if not self.clientToken:
            self.clientToken = self.get_token()

        # Mark this state's token as belonging to the clientToken
        shared_sessions_by_token.setdefault(self.clientToken, set()).add(self.get_token())

        # Mark this state's websocket id as belonging to the clientToken and state token
        tokens_by_sid[self.get_sid()] = (self.clientToken, self.get_token())

        # Set the colorState for the new session from existing shared sessions (if any)
        return State.set_color_state_for_new_session

def page1() -> rx.Component:
    return rx.vstack(
        rx.button(
            "Click 1",
            on_click=State.ChangePage1Color,
        ),
        rx.text(
            State.colorState
        )
    )

def page2() -> rx.Component:
    return rx.vstack(
        rx.button(
            "Click 2",
            on_click=State.ChangePage2Color,
        ),
        rx.text(
            State.colorState
        ),
    )

app = rx.App()
app.add_page(page1, on_load=State.set_client_token)
app.add_page(page2, on_load=State.set_client_token)
app.compile()

# Handle websocket disconnect events to avoid memory leaks when sessions are closed
orig_disconnect = app.event_namespace.on_disconnect

def disconnect_handler(sid):
    orig_disconnect(sid)

    clientToken, token = tokens_by_sid.get(sid, (None, None))
    print(f"Disconnect event received for {sid}. Removing {token} from shared {clientToken}")

    shared_sessions_by_token.get(clientToken, set()).discard(token)
    tokens_by_sid.pop(sid, None)

app.event_namespace.on_disconnect = disconnect_handler

https://github.com/reflex-dev/reflex/assets/1524005/e672b1b5-60d4-4958-93f4-6f43a322f05f


[Feature Request] Sort dict keys

Would be nice and I did not find this feature already being requested.
ESLint has this feature: sort-keys. Should have options like case-sensitivity, minimum keys, natural, asc/desc. # noqa: ... should disable the requirement and stop the auto-fixer.

Thanks for the feature request! For reference, this is similar to a few issues that have been opened before:

https://github.com/astral-sh/ruff/issues/6175
https://github.com/astral-sh/ruff/issues/7929
#7220

However, I think we should keep this open. The first two of those issues were closed in favour of #1198, which was closed out without implementing this specific feature. The third is different enough to this that it could be considered separate.
I'm not sure I'd want to turn on this rule in my Python projects, since Python dictionaries maintain insertion order when you iterate through them (they have been guaranteed to do so by the language spec since Python 3.7), and I've written code in the past that's relied on that useful property. That means that the autofix proposed here has the potential to break working code. The only Python builtin collections where it would be safe to autosort the collection are sets and frozensets, since those are the only two that do not maintain insertion order.
Nonetheless, it does seem like there's a fair amount of user demand for this feature, and I gather the ESLint rule also has the potential to change the iteration order of a JavaScript object. So I think we should consider having a rule like this. If we're sorting dictionary keys, though, we may as well also sort list, tuples and sets, since we'd already be doing something pretty unsafe by sorting dictionary keys.
One open question is whether enabling this rule should sort all collections? isort has a feature where collections that are marked with an # isort: unique-list comment are deduplicated and sorted -- we could possibly consider doing something similar.

The advantage is that it neatly works around the safety issues (we'd only be sorting collections where the user has explicitly asked us to do so).
The disadvantages are:

It's another pragma comment for users to have to remember/more configuration is really confusing
It's more verbose to have to add a comment next to every collection you want to be sorted like this.

we may as well also sort list, tuples and sets, since we'd already be doing something pretty unsafe by sorting dictionary keys.

It would be worth having different rules, at least for lists. I've used object sorting in very large code bases in JS with very little need to suppress the rules because it's uncommon that the ordering matters (at least in the projects I worked). However, that's different for lists where ordering is more likely to be semantically meaningful (or I would have used a set).

I would like this specifically for set (I realize that's not what the issue is about). Sets don't have insertion ordering, so unlike with dicts or lists, there isn't as much of a concern about changing behavior. For example, it would be nice if Ruff sorted this set so it's easier to keep track of which letters are in it.

I would like this specifically for set (I realize that's not what the issue is about). Sets don't have insertion ordering, so unlike with dicts or lists, there isn't as much of a concern about changing behavior. For example, it would be nice if Ruff sorted this set so it's easier to keep track of which letters are in it.

Yeah, I agree that it makes much more sense for sets than any other data structure. I wonder if this should be multiple rules — or one, configurable rule.


add jiffy

Hello
I've been playing with fiar to learn me some erlang, and I noticed I got a cowboy error registering.
I debugged it and it looks like jiffy is not on the application path, even when I tried to call jiffy:decode/1 from the console it failed. I changed that and its letting me create users.
(now I'm having trouble logging in, but thats another story...)

Cool thanks! If you have more problems, you can ask me in the open source chat room, maybe me or someone else can help you.


Add install pytorch operator steps for the doc

Ref: https://github.com/kubeflow/kubeflow/issues/1257

/assign @nickchase

/ok-to-test
/lgtm
/approve


Support building Ambassador with a custom Envoy binary

Please describe your use case / problem.
Since Ambassador only orchestrates and glues services using k8s and envoy, it doesn't actually handle any request processing. Envoy is a very fast reverse proxy, and to keep that advantage for latency-critical services, the ability to write envoy filters, and build a custom envoy binary that Ambassador will use instead of the default is a must.
Without it, very simple use cases like headers/request body manipulation need to be processed by an external service, that will present serious performance degradation.
In my specific use case, I need to have a request enter through an API gateway, apply some header manipulations based on custom data that is loaded from an external data store (and stored in-memory as part of the filter), and send the result to an upstream service with a response, all of this should be done under 20ms of overhead + ~25ms from the upstream (I need RTT to be under 50ms).
Describe the solution you'd like
Support for building Ambassador with a custom envoy proxy, preferably with some kind of validation tool that will check that the envoy binary contains the needed features for it to be compatible, and an easy way to build that Ambassador image (perhaps through a Dockerfile).
Describe alternatives you've considered
None, this is a must to allow for flexibility.
Thanks, you're awesome! 😄

I'm going to close this, since I believe that the BUILDING.md file on the master branch actually describes how to do this. If you find that it doesn't, please reopen!

@kflynn can we we still do it? I am unable to find this file


DO NOT APPROVE - bug bounty test - sorry for spamming :-)    vaeabo

DO NOT APPROVE - bug bounty test - sorry for spamming :-)    vaeabo

AppVeyor (DocuSign Rooms) build and test pass.


adds preferred_name

@peintnermax Does it need the scope both files?

profile scope for the user should be enough, you don't need a scope on jwt sign as it is for the service user