environment_min.yml is broken

I've just installed this environment and it resulted in a broken kwant:
>>> import kwant
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/w2scratch/home/t-banij/miniconda3/envs/py36/lib/python3.6/site-packages/kwant/__init__.py", line 37, in <module>
    exec('from . import {0}'.format(module))
  File "<string>", line 1, in <module>
  File "/w2scratch/home/t-banij/miniconda3/envs/py36/lib/python3.6/site-packages/kwant/builder.py", line 20, in <module>
    from .linalg import lll
  File "/w2scratch/home/t-banij/miniconda3/envs/py36/lib/python3.6/site-packages/kwant/linalg/__init__.py", line 10, in <module>
    from . import lapack
ImportError: libgfortran.so.4: cannot open shared object file: No such file or directory

Ah, I only read these comments now https://github.com/Microsoft/qmt/blob/06e038970ba9fa63009d081d37abaecde3e146de/deployment/environment_min.yml#L1.
Should this or shouldn't it result in a correctly functioning env?

Unfortunately this also fails:
conda-env create -f environment_full_linux.yml
Collecting package metadata: done
Solving environment: failed

ResolvePackageNotFound:
  - matplotlib==2.2.2=py36_1

When you install from environment_min, we're not version locking the packages, which means that you'll get the latest packages. Since package updates happen all the time, there's no way to know if this environment would work without testing it out. the environment_full files represent the last tested set of packages that are known to work. Normally I only use environment_min when I'm doing package upgrades. I install from that file, see what's working/not working, and if everything looks good I'll overwrite the environment_full files

OK great! Then I'll close this. We've also fixed what I mentioned here by merging #116.


Configure Hercules CI

I believe herculesCI.ciSystems is mandatory to limit the building on these two systems (because that's all I have as agents).
Hercules CI will be used latter for testing #25

https://hercules-ci.com/github/srid/haskell-flake/jobs/2
Evaluation failed
Please make sure your ci.nix has at least one derivation.

Hmm, I don't think this error message is accurate? It assumes I must have a ci.nix, but that's not the case. @roberth

Looks like I need to add some derivation outputs to make HCI happy. Which I have to do anyway, eventually, when adding a check for #25
flake-parts uses a dev flake approach. Should haskell-flake do the same, or is there a better way?


Feature/set default lang

What this Pull Request (PR) does
Provide an option to set a default output language to make commands more concise.
Screenshots

Thanks for the PR. Please update Readme.

Thanks for the PR. Please update Readme.

The system will automatically set default values for you using --setup option, so there's no need to refer to the user manual.


[system-probe] allow to not use -mod option

What does this PR do?
The -mod option is always provided when building the system-probe.
Motivation
The build of system-probe with "-mod=vendor" seems to be failing because of a gobpf header that can't be found.

In which environment is it failing to build? It builds just fine in the agent VM.


Add setup troubleshooting to README

We actually already had some docs on this https://github.com/alphagov/email-alert-api/blob/master/doc/troubleshooting.md#fixing-pginsufficientprivilege-error-in-the-development-vm, but it looks like perhaps it's a bit hidden away and we might need to find a better way of making it easier to spot.
Also, the solution in that file was similar to your initial solution, if we can just run govuk_puppet and it fixes it, we could update those docs as well.

@thomasalanlee I didn't see those! I'll just add a link to the troubleshooting docs from the main README.

There are already trouble-shooting docs that cover this error.


🌹

https://github.com/github/docs/blob/main/content%2Fauthentication%2Fkeeping-your-account-and-data-secure%2Fswitching-between-accounts.md

20


Debugger crashes immediately on OS X 10.11 SeaLion with Kill 9 error

I followed instructions and was able to build and install from source and also to create a self signed cert.
All that worked. When I run the debugger it crashes with a kill 9 message. When I set a breakpoint using the runtime class and type "dlv run hello.go" it crashes with a kill 9 error

Do you mean 10.11 El Capitan? I do not have 10.11 installed on my machine, however maybe I could look into this. However, dlv run does not take an argument. It is meant to be run standalone. This is actually a common point of confusion, so I'm considering dropping run altogether and just assume the default behavior when executing dlv.

@tonyconstantinides is it possible that the binary was not signed properly? On 10.10 I was seeing an error, "unable to fork/exec" when running "dlv run".  Once I rebuilt the binary and followed the signing steps fully that stopped happening.  I did not check the exit code but I suspect it was probably SIGKILL.

Hi,
Yes I mean I running 10.11 El Capitan. I also using the 1.5 beta 1 which I know you do not support yet.
I really should of read more issues before filing this one. Feel free to ignore this issue until you have a chance to catch up with Go and OS X changes. I did try "dlv run mygoapp". I have carefully created the cert last night making sure I complete all the details so I do not  believe its a cert issue.(I may be wrong:).
I running OS X 10.11 its gone through I appStore update)
Go 1.5 beta1
Self signed cert dlv-cert like the example
2012 MacBook Pro retina with core i7 and 16gb of memory and 256 sad
GOROOT set to the 1.5beta1

Closing this as it's a result of https://github.com/golang/go/issues/11887


OS-2145; Set error message on protocol negotiation failure.

If XRDP was not passed a password or username, XRDP would disable checking for NLA, because it could not possibly do NLA without valid credentials.
However, if the device was set to force NLA, then the protocol negotiation would fail, but not set the error message. This left the error code as 0 (success) which was presented to the user.
I have also addressed another case of this if transport connection failed.

I have tested this and you now get the following error should negotiation fail:


How can I use benchmarking functions to generate data?

Steps to reproduce

How can I use benchmarking functions to generate data?___

Observed Results
*How can I use benchmarking functions to generate data?
Expected Results

How can I use benchmarking functions to generate data?

Relevant Code
How can I use benchmarking functions to generate data?
// TODO(you): code here to reproduce the problem

Hello! in the dashboard https://dl.fbaipublicfiles.com/nevergrad/allxps/list.html you can find a lot of data (all CSV files)
and you can regenerate data with commands as in
https://facebookresearch.github.io/nevergrad/benchmarks.html
e.g.
python -m nevergrad.benchmark noise --seed=12 --repetitions=10 --num_workers=10 --plot
for the "noise" benchmark with 10 cores.

Hello
Thank you for the fast response. I need list of functions in nevergrad, 24 functions in BBOB. Where can I find it? I am looking for python nevergrad function which gets input vector, math function types and generates their outputs F(x). Where can I find it? The command listed by you is less appropriate for my needs.
Thanks

If you want exactly the functions in BBOB, you might use BBOB directly ?
In Nevergrad we have YABBOB and many others.
We have many, many benchmarks in Nevergrad, probably the biggest benchmarking suites currently available with a single API. But for BBOB we have our own counterpart, which is similar to BBOB but not exactly equal.
Note that the random translations are a key component of BBOB or YABBOB --- you should not use the functions without translating them, or you will get buggy unreproducible results because the best optimization method will be "print (0,0,0,0,0)" if you do not translate the optimum from 0.
So I do not encourage you to use directly the functions in Nevergrad or in BBOB without the translations as this would lose the entire point of these frameworks. That said, if you want to do it for some reason, you can completely use e.g. https://github.com/facebookresearch/nevergrad/blob/edac4eaa65db5e5e01dc0f1ebcc5134c47b78060/nevergrad/functions/functionlib.py#L86


cdogs-sdl: 0.13.0 -> 1.3.1

Description of changes

Changelogs
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Does it make sense to add a XDG desktop item?

Does it make sense to add a XDG desktop item?

If I’m not mistaken an XDG desktop entry is already included..


Tabular: Added custom metric tutorial

Issue #, if available:
Description of changes:

Added custom metric tutorial
Sped up regression metric calculations by ~50%
Removed error_bad_lines in load_pd as it is deprecated in pandas 1.3

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-1271-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1271/1/index.html

Job PR-1271-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1271/2/index.html

Job PR-1271-3 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1271/3/index.html

Job PR-1271-4 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1271/4/index.html


Update EventSub Documentation and fix non-existent type

Hey, I have revised the documentation a bit, so that everyone can understand how to use the EventSub.
Also I fixed a type. There had been a typo in it. It should be channel.follow instead of user.follow.

I also added a fix for the EventSubSignature class. The timestamp header provided by Twitch has a special date format Y-m-d\TH:i:s.u\Z. I correct this, but cannot validate this with the real api, since Twitch has a Bug. Currently chatting with PooPooKitty from Twitch to verify the right time format.

I resolved this. Twitch switched to nanoseconds. Thanks to Dkamps18, he provied a regex for the datetime parser.

I don't feel comfortable with that much hacky regex in the getTimestamp method. Could you show an example payload you receive from twitch including the timestamps?

Wow this insane. I don't see any reason whatsoever to use a non-standard date format...
Anyways how about a more flexible expression like /^(?<pre>[\d\-:.T]+)\.(?<nano>\d{9})Z$/? (Regex 101 playground)
We would receive the following named matching groups
  "pre" => "2021-02-05T20:07:58"
  "nano" => "288209337"

Carbon::createFromFormat(
    'Y-m-d\TH:i:s.u\Z',
    sprintf('%s.%dZ', $matches['pre'], substr($matches['nano'], 0, 6))
)

They use RFC3339Nano in go lang.
This Regex looks more clear 👍

Any news on this PR ?

Yes, i updated the regex, as requested. Hope this will be merged soon :)

I now deploy this branch to Production and subscribe to 2.2 million webhooks. Let's see :D

Alright, looks good! Could you at least squash the latest commits to remove the merge commit? I'd like to keep the history clean 😄

@romanzipp can you check it again. :pray:


Stop stirring when OD measurement is taken

An idea is to completely stop stirring the moment the OD reading is taken.
Hypothesis: there is typically enough rotational momentum to keep cells suspended, and the signal (to the eye) looks much smoother. This would also prevent microbubbles (from a bubbler) from causing more noise.
Implementation: this becomes part of stirring job's sneak_action_between_readings - basically, we turn duty_cycle/rpm to 0 when reading, and high inbetween.

This didn't really work. In fact, it looked like faster stirring produced the best measurements:
Using a static wort. First 10m are with constant stirring throughout. Next 10m are reduced stirring speed, still constant. Next 10m are with stirring turned off during reading. Comparing the variance of the time series.

12:45:07 reduced stirring to 26 (from 38)
12:56:22 turned back to 38, stopping during reading


onBackgroundMessage not fired and app  not opening on notification click

Hi,
I have two problems.
The onBackgroundMessage Event is not triggered, i only receive the onMessage when the App is in foreground while i receive a message.
At first I also didn't get the onMessage but then i added this to my config.xml:
      <config-file target="AndroidManifest.xml" parent="/manifest/application" >         <service android:name="by.chemerisuk.cordova.firebase.FirebaseMessagingPluginService">           <intent-filter>               <action android:name="com.google.firebase.MESSAGING_EVENT"/>           </intent-filter>         </service>       </config-file>
Is there anything I have to add that i get the onBackgroundMessage to?
The second problem I have is, when i recevie a notification in the background, the push notification appears but when i click on it the app won't come in the foreground :(
Does anyone have a idea what I'm doing wrong?

Hello @pilz97 ,
I'm also facing same issue.
Do you got the solution for it??

Facing the same issue in Android as well as iOS.

Same issue :(

Facing the same issue on Android. Work on iOS

Hello,
I found out that removing the click_action attribute from the notification config array ( i wrote my backend in PHP), resolves the issue. Apparently, the firebase plugin takes charge of creating the appropriate intent and launches the app on its own. No need to add a click_action filter name, because it confuses firebase (?) and leads to canceling the intent all together.

Same issue. «cordova.plugins.firebase.messaging.onBackgroundMessage» not Work on Android.

Hello,
I found out that removing the click_action attribute from the notification config array ( i wrote my backend in PHP), resolves the issue. Apparently, the firebase plugin takes charge of creating the appropriate intent and launches the app on its own. No need to add a click_action filter name, because it confuses firebase (?) and leads to canceling the intent all together.

Getting rid of click_action does solve the issue if you don't have a custom data payload, or if the data payload is just title and body.
When I send just title and body, or custom data elements which are empty in a notification payload and the app is in the background. If i click the notification, it opens up the app hitting the .onBackgroundMessage function.
example payload:
{
"notification": {
"title": "test title",
"body": "test body"
},
"data": {
"title": "test title",
"body": "test body",
"link": ""
}
}
However when I send some custom data In my case a url link such as www.example.com, If I click the notification it does not open up the app, and it does not hit the .onBackgroundMessage function.
example payload:
{
"notification": {
"title": "test title",
"body": "test body"
},
"data": {
"title": "test title",
"body": "test body",
"link": "www.example.com"
}
}
If the app is in the foreground any and all payloads hit the .onMessage

I am facing the same issue. Does anyone have any solution for a workaround ?

Hi, just curious if this issue is already fixed, especially with function onBackgroundMessage not fired?

@helloworld123456780 read notes section in https://github.com/chemerisuk/cordova-plugin-firebase-messaging#onbackgroundmessagecallback

The probleM is that on FirebaseMessagingPlugin.sendNotification() there is a check "if (instance != null)" that simply ignores push notifications is the app is not initialize INSTEAD OF INITIALIZE IT. @chemerisuk can this plugin perform this initialization in order to enable push notification when the app is closed?


Ability to filter service APIs for controller auto generation

Ability to configure set of Service APIs so that ACK controller auto generation logic can take that into account.
This would enable multi phase, incremental support for a given service API from ACK.

Note: the GeneratorConfig struct contains instructions to the code generator on how to handle a particular API:
https://github.com/aws/aws-controllers-k8s/blob/mvp/pkg/model/generator_config.go
The solution to this particular feature request is to add a field to GeneratorConfig that would instruct the generator to ignore certain top-level resources or API operations.
Perhaps something like this would work:
type GeneratorConfig struct {
...
    Ignore IgnoreSpec `json:"ignore"`
}

type IgnoreSpec struct {
    // Set of operation IDs/names that should be ignored by the
    // generator when constructing SDK linkage
    Operations []string `json:"operations"`
    // Set of resource names that should be ignored by the
    // generator
    ResourceNames []string `json:"resource_names"`
}

and then in the pkg/model.Helper.GetCRDs() method, we could filter out various operations or resource names by inspecting the GeneratorConfig.Ignore field.


Image Viewer using pyvips?

How would you recommend using pyvips to perform visualization instead of using the C version of vips? I am asking this because I couldn't find the vips_sink_screen function implemented in any of the Classes in pyvips.
Also, if I want to build a visualizer using libvips, is the C implementation my only option?

Hi @prasadbandodkar,
I think it just needs binding, it shouldn't be too hard.
sink_screen() used to be very difficult to bind since callbacks came from worker threads.You would have needed a layer to serialise these things and work around the GIL.
sink_screen() was refactored a few years ago and is now really simple: all callbacks always come from the main thread, so there's no serialisation required, and there are no GIL issues to work around.
Absent a binding for sink screen, you can use fetch to pull tiles from a pipeline and draw the screen. It won't be parallelised though.


MAJOR: weird character / encoding issue in lib/uid.js

Error
There is an invalid 0xc2 character in lib/uid.js as you can see in the hexdump I just retrieved from master:
000001f0  72 74 73 2e 72 65 73 65  74 20 3d 20 66 75 6e 63  |rts.reset = func|
00000200  74 69 6f 6e 28 29 c2 a0  7b 0a 20 20 20 20 63 6f  |tion()..{.    co|
00000210  75 6e 74 65 72 20 3d 20  30 0a 7d                 |unter = 0.}|

After running it through webpack I get this:
	exports.reset = function()Â {
	    counter = 0
	}

Background
I'm trying out jss with webpack and the resulting javascript is invalid. At first I was blaming my webpack config so I removed everything to find the cause. Eventually I ended up with a close to empty webpack config. Before blaming webpack I looked at the orignal uid.js and found the error.
I also scanned the rest of the js files and it is only in this file and the non minified dist file.
Workaround
Minifying with UglifyJS removes the invalid character.

I am working an a PR

yeah its  a bad whitespace again, will add a linter soon.


Adding new async API for DMA

Hi, I was working on a PR for embedded_graphics that will enable a AsyncDrawTarget that we could use in the driver in order to provide an async API. For example, this could enable the driver to use the DMA async implementation from embassy.
The implementation of this trait in the driver could be similar to DrawTarget but using AsyncWriteOnlyDataCommand generic trait bound instead of WriteOnlyDataCommand. This could be relatively easy once AsyncDrawTarget will be available publicly and could bring us some better performances.

I have done a quick implementation on my fork to add this feature, but there is a typical issue of code duplication with sync and async code. I can't see a good way of avoiding it since this is a language current issue :/

We don't need to wait for embedded_graphics necessarily. We can already offer the async api to users that don't need the high level drawing methods.
About the duplication: yeah, not great. I looked at your fork and indeed it's pretty much copy-paste minus the .await and the method names having the _async suffix.
There are macros like https://github.com/fMeow/maybe-async-rs (but I think others too) to avoid duplication, but I am not sure if they are practical to use.

I can confirm that maybe_async_cfg is used in multiple crates for embedded development. Practicality-wise, well, it's fairly straightforward to set up and helps avoid code duplication.


Make defense less vulnerable to lures / landmines

Phase 1: dedicated reflect 3 / thrusters 3 miners lured out the tanks.

Phase 2: 2 of the same, plus 2 reflect 3 / melee 2 teleporters. Once rangers engaged the miners were pretty useless.

Phase 3: laser 3 / reflect 3.

Ideas:

lasers on defense?
if tanks are damaged and beyond halfway point, pull back.

Try adding laser dodging code when there is one enemy and it's out of charge range.

Perhaps limit how far out bots will chase enemies. This will not only make it harder to get mined but also protect against lure attacks.
By running all the way back when this happens, we also avoid getting shot by artillery micro.

Copied from #52:
We can also make the area clearing less stringent. Too stringent and we risk getting lured and the CPU killed. We can probably clear if either condition is satisfied:

<= 3 spaces away with sensors on
<= 1 space away with no sensors on

Also, bots should not follow enemies more than halfway across the map (just delete enemy target location). This will both cause them to be less sensitive to mines, and get back to the CPU faster in case of lures. Because it will return and not just sit at the edge, it won't be vulnerable to artillery micro.

Another idea: if the enemy is more than X distance away don't chase, just return to CPU.

Idea: "recycle" units with lower health toward the back, with higher evade threshold if they get damaged.


pythonPackages.isort: fix tests

Motivation for this change
noticed it was broken on hydra due to timeout https://hydra.nixos.org/build/104639574
I believe something with the sub shell was causing it to hang
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

looks good for me

LGTM. Compiles, tests, runs for me.

They seem to be passing now https://hydra.nixos.org/eval/1555603?filter=isort&compare=1555543&full=#tabs-still-succeed
I might just close

closing for now, if hydra hangs again, I'll re-open


"CREATE ... WHERE NOT EXISTS MATCH..." caused the server crash

When I was just getting into AGE, I wrote a mixed SQL statement. It caused the server crash. But I don't know why.
Here, I write the SQL statement into a sql file. You can execute the file using psql command line.
CREATE SCHEMA test01;

SET search_path = test01, ag_catalog, "$user";

SELECT create_graph('mygraph');
SELECT create_vlabel('mygraph', 'node');

SELECT * FROM cypher('mygraph', $$
    CREATE (:node {name: 'a'})
$$) AS (a agtype) WHERE NOT EXISTS (
    SELECT * FROM cypher('mygraph', $$
        MATCH (v:node {name: 'a'})
        RETURN v
    $$) AS (b agtype)
);

SELECT drop_graph('mygraph', true);

DROP SCHEMA test01 CASCADE;

The output is
...

psql:xx.sql:17: server closed the connection unexpectedly
        This probably means the server terminated abnormally
        before or while processing the request.
psql:xx.sql:17: fatal: connection to server was lost

...

Does my SQL file has an illegal SQL or OpenCypher grammar?
PS:
Now, I know MERGE clause can create an entity when it does not exist. But, it is still difficult to write when the condition is complicated.
I've tested the sql file in PostgreSQL 11 and AGE commit id 12edd3e .

I don't see how the following is more complicated than what was put above -
MERGE (u:node {name: 'a'}) RETURN u;
I checked and, neither Neo4j nor AGE currently allow a syntax like (or similar in function to) the command that crashed above, except as MERGE. It is true that we will need to look and correct that crash. But, what you defined above is exactly MERGE, without all of the extra SQL.
The WHERE clause, on the other hand, is not currently supported in conjunction with CREATE or MERGE in AGE. We will need to look into that.
Hope this is helpful

I have used MERGE to replace CREATE ... WHERE. After all, psql did not complain that the grammar was illegalI. So I opened this issue to indicate that. It would be better to tell the user via an exception that the WHERE clause  is not supported for CREATE now.

As I stated above, we will need to address that crash.
Your command isn't syntactically incorrect, but the usage of some command like CREATE (blah) WHERE NOT EXISTS (MATCH (blah)) is not valid in either AGE or Neo4j - it will generate a syntax error - at least as far as I could tell.
The difference here is that you wrapped the two parts (create & match) into two separate cypher queries and glued them together with SQL's WHERE NOT EXISTS. That is legal and the two command separately are also legal. However, for whatever reason - and we will need to address it - they don't play well together.
Unfortunately, making that generate a syntax error isn't necessarily easy because they are separately valid commands AND we have no access to the PG parser.

Yes. It is difficult for AGE to detect the weird usage as the two cypher statements are in different parts of a single SQL.


Keep all image versions in a single JSON file.

This should make release engineering and automated updates of
image versions easier.
Also, bump versions for:

cert-manager
external-dns

Yes, it is similar to your previous take at this and we need something in place in order to be able to automate part of the release engineering when new assets are released (e.g. due to security issues).
Do you think that it is not easy to parse and manipulate JSON? There are libraries for Go and Python and Java. You can even manipulate JSON on the command-line using jq.

@falfaro sure re jq and friends, tho I was thinking on even simpler
ways (e.g. being able to do so via sed and friends).
Nevertheless, this is better than existing, LGTM'ing.

Why we are not doing it for all the images at once?

I am going to do this for all images, and keep the versions unchanged (0-diff).

Done. PTAL.

The logic is so simple that I think no helper functions are needed, but if necessary I could do that.


Add AsyncIterator tracking on async-replace directive

It solves the problem commented on the issue Polymer/lit-html#484

Thanks a lot @ManRueda!


Update Dockerfile

Coverage remained the same at 8.26% when pulling 73f236993fb0a226d075cf938d3efe9722173084 on starskim:patch-1 into 598ab1d2d163739b6eb604587478cfeb4e8f5a9d on orvice:master.


Virtual touchscreen in not active

After install is I got a Virtual touchscreen in not active message and the touchscreen does not work. Any suggestions?

Yes, forgot set android pi ip


Fix text expressions in conditionals

Problem:
Normally we hide expression elements from the navigator when they only return text, e.g.
<div>{1+1}</div>
is shown as:

The reason is that the expression results in text content, and normally we don't treat text content as elefants.
However, when text content is in a conditional branch, we show a navigator line for that text content (otherwise it the conditional branch would be empty).
The bug was that the conditional branch in the navigator was really empty when there was an expression which returned text content:
{true ? 1 + 1 : null}
resulted in

Fix:
I made sure we don't filter out text only expressions when their parent is a conditional.

I think I'm weeks behind on the conversation / design. In the spring, the idea was that we only show conditionals in the navigator if at least one branch is JSXElement or if both branches are null.
So we would not show conditionals in the navigator if their content is {true ? text : text } or {true ? text : null} or {true ? null  text}
Did this thinking evolve? Is the null slot more important now? Or did we change the design because the rest of the navigator changed (with the code blocks now showing up as Elefants)?


[InputLabel] Fix shrunk label click not focusing input

Reverts #13040 but should not regress from #12997
Fixes clicking shrunk labels not focusing their associated input: https://deploy-preview-16215--material-ui.netlify.com/components/text-fields/#outlined
I think the issue with #12997 was that the label was not associated with the input. pointer-events: none fixed the apparent issue but hides a deeper issue with the a11y of the text field.
We should optimize for proper a11y setups in which a click on a shrunk label would not focus the text field. Depending on the layout pointer-events: none could prevent the default label-input UX.
Would like some feedback about the new cursor logic. I'd be fine without cursor: text trickery. Problem without cursor: text the cursor behaves different on outlined vs standard text fields. But with it you get a short flash when clicking on shrunk labels.
TODO:

[ ] Need to figure out hover styles.

Google uses the same tradeoff (z-index + pointer-events: none) when logging into gmail. I guess this is fine to loose some clickable pixels.
The autofill background color hack might be useful when we tackle #14427


Add ref to props

Currently TS disallows adding a ref to the Quill component. This allows a ref to be passed.

Refs shouldn't be added to props - they are a "special" prop type that applies to all components, and aren't accessible on the props object.

Hmm I just tested again with a codesandbox and it was working as expected. I'll close this for now, and I'll check with my team to make sure everything is as expected.


[Bug][github plugin] Unexpected "Bad Credentials" when accessing github connection proxy

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Hi! Hope all is well over there :)
I am not sure why this happens and can't tell the exact behavior, but we're seeing unexpected "Bad credentials" errors when accessing a specific Github plugin endpoint, like this: https://devlake/api/plugins/github/connections/3/proxy/rest/repos/my-org/my-repo
We use this endpoint when orchestrating the creationg of blueprints/projects from an API of ours (to abstract that away from the end user).
This seemed  to work fine in version 0.17
This is the response we get
{
  "message": "Bad credentials",
  "documentation_url": "https://docs.github.com/rest"
}

Other Github connections seem unnaffected, and the authentication method we use for all of them is the same - github app. And it's also the same github app for all of them (which are just different orgs)
Apparently if I re-save the connection settings with the same token, things start working again.. I'm wondering if there's some sort of expired token being used here.. like it doesn't refresh it for some reason? Would you be able to help us understand this please?
one thing to note.. we've noticed our instance crashes quite often (about 3-4 times a day, haven investigated it very much, but apparently with OOM).. maybe this contributes to an unknown state, so wanted to share it
as a workaround, we are looking to perhaps trigger the connection settings update every 30 minutes or so
Thanks!
What do you expect to happen
We would expect the token to refresh properly (assuming we are using an expired token), and then we would not get Bad Credentials
How to reproduce
Not sure yet..
Anything else
It seems to occur every day, quite frequently.. this is the rate for the last 3 days:

Version
v0.21.0-rc1@ac4f7fe
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

this functionality is affected as well (it is not able to list repositories)

Hi there,
Thanks for reaching out! Let's address the two issues you mentioned:

Expired Token:

Unfortunately, due to GitHub's design, tokens cannot be refreshed.  The best course of action is to regenerate a new token with the appropriate permissions for your needs.

OOM (Out-of-Memory) Error:

To help diagnose the OOM error, could you please provide some additional information:
Crash Logs: Were there any logs indicating the crash, such as "panic" messages, around the time of the error?
Container Resources: How much memory and CPU have you allocated to the container running the process?
Pipeline Context: Did the OOM occur while running a pipeline? If so, were there any specific workflows or tasks in progress during the breakdown?

Hi @klesh, thanks for reaching back :D
I think we can focus solely on problem 1 on this issue if that's okay!

Unfortunately, due to GitHub's design, tokens cannot be refreshed. The best course of action is to regenerate a new token with the appropriate permissions for your needs.

We are using a github app, so I think the tokens should be re-generated with the app's secret key, which does not expire, right? These tokens only live for 1 hour if I am not mistaken.. so we just wanted to make sure they are being refreshed.. from what it looks, we might have caught a situation where it's not 😥
thanks!

@pimguilherme
I apologize for the misunderstanding. I wasn't aware of the GitHub App authorization process and initially thought you were referring to a Personal Access Token (PAT).
I've reviewed the code, and it appears the access token is regenerated at the start of each collection. This logic hasn't changed since version 0.17.
You mentioned the token expiring after an hour, which could be the issue. It's possible that previously collections completed within that timeframe. However, with growing data volumes, that might no longer be the case.

Hi @klesh, thanks for the feedback! I apologize for the delay
We do have issues with large repositories that fail because they need to access github after the token has already expired :(
And I am not sure this is the same mechanics, but we also have problems loading data from the Github connection proxy (ie https://devlake/api/plugins/github/connections/3/proxy/rest/repos/my-org/my-repo), how does that token get refreshed?
Thanks again!!

@pimguilherme Yup, you are correct, the token for Github App should be refreshed when expires automatically.
The proxy endpoint is the perfect proof of the problem because the token is requested once when ApiClient gets instantiated and the proxy helper caches it  so the token expires one hour later.
It might take a while to fix.

Do you have the status code of the Bad credentials response?

Any updates on this issue, we got the same issue. Is it because of any rate limits from Github?
error decoding response from https://api.github.com/user/orgs?page=1&per_page=100: raw response: {"message":"Bad credentials","documentation_url":"https://docs.github.com/rest","status":"401"} (400) Wraps: (2) json: cannot unmarshal object into Go value of type []api.org Wraps: (3) json: cannot unmarshal object into Go value of type []api.org Error types: (1) *hintdetail.withDetail (2) *hintdetail.withDetail (3) *json.UnmarshalTypeError

No, no one is working on the issue AFAIK.


Empty results

A lot of empty stuff for Anime and Manga source etc
would be good if it can detect the type of sauce like AnimeSource, MangaSource, PixivSource, TwitterSource etc. and provide results according to that

For empty stuff, can you provide an example? The only sauce I've tested so far is pixiv illustrations.
And for sauce detection, I don't think SauceNAO web ui show them, and if they do it'll be just a little text unlike their API which does more.

For empty stuff, can you provide an example? The only sauce I've tested so far is pixiv illustrations.
And for sauce detection, I don't think SauceNAO web ui show them, and if they do it'll be just a little text unlike their API which does more.

you can use for manga panels like this - https://telegra.ph/file/d2a677fe623e88b915a5a.jpg
and for anime a gif like this - https://telegra.ph/file/213bfae4c02f23a3aefad.mp4 (need to convert into .gif)
and not even saucenao api does sauce detection i just wanted to tell if you could do it in the library

you can use for manga panels like this - https://telegra.ph/file/d2a677fe623e88b915a5a.jpg

It does show the result in SauceNAO with the sauce "MangaDex" and "MangaUpdates", I'll try to do that when I'm free but for now you'll have to do something like this:
misc_info = SaucererSync().search("https://telegra.ph/file/d2a677fe623e88b915a5a.jpg").sauces[0].misc_info
is_manga = any([x for x in ["mangadex"] if x in ["mangadex", "mangaupdates"]]) # should be true


Please make the components "Accessibility Insights for Web" compliant

Feature request type
Enhance component
Component name
No response
Is your feature request related to a problem?
Hi,
I am using almost every component of MudBlazor in my application. They are super awesome. Thanks a lot for providing such a library. Our UX teams is not giving clearence for a production deployment of our application because aria-labels are missing on various buttons. They are using browser extension "Accessibility Insights for Web" to check such things.
Please provide aria-label for buttons in MudDataGridPager, FilterHeaderCell, Sort buttons on MudDataGrid.
One more suggestion is, please make all private methods of MudDataGrid and other components as protected virtual methods so that we can subclass them.
Thank you.
Describe the solution you'd like
Please provide aria-label for buttons in MudDataGridPager, FilterHeaderCell, Sort buttons on MudDataGrid.
One more suggestion is, please make all private methods of MudDataGrid and other components as protected virtual methods so that we can subclass them.
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[x] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Submitted a PullRequest https://github.com/MudBlazor/MudBlazor/pull/5719

This should be much improved in version 7. If you have the time to check it out please let us know what you think!


Add a basic check for if current user can upload files

Summary
Add very check to file uploads. Related to #196 and #1468
This check is to be doubly sure that user is shown an error if they can not upload files.

E2E tests failed because of a React error: https://reactjs.org/docs/error-decoder.html/?invariant=200
Merging latest master to see if that resolves the issue.

E2E tests failed because of a React error: https://reactjs.org/docs/error-decoder.html/?invariant=200
Merging latest master to see if that resolves the issue.

I can't replicate this locally.


SWAPI Quality Flag

Change Summary
Overview
Uses Quality flags class to set SWAPI quality flags. Closes #707
Updated Files

imap_processing/cdf/config/imap_swapi_global_cdf_attrs.yaml

update descriptor attrs

imap_processing/cdf/config/imap_swapi_variable_attrs.yaml

update flags attrs

imap_processing/quality_flags.py

Added SWAPI flags class

imap_processing/swapi/l1/swapi_l1.py

update L1 code to use new flag class
update uncertainty variables

imap_processing/swapi/l2/swapi_l2.py

updates based on l1 updates

pyproject.toml

too many argument error

Testing
update test based on above changes

Please update this page when you have a chance:
https://lasp.colorado.edu/galaxy/display/IMAP/Quality+Flags

I am going to merge this since there was not change request.


Md5sum for binaries

Hello,
There is no md5sum for binaries in release could you provide that for the next ?
And/or add prometheus-postgresql-adapter to your ppa repository to have one only trusted source ?
This should be better for continuous deployment
thank you

@RobAtticus @blagojts  something we can take a look at?


Compilation: Add detection for native deps for Libuv

Add these too as a stopgap until we can do full native dependency detection.
Fixes #120.
Temporarily Don't Merge so I can test on Mac.

Removing Don't Merge, tested on Mac and this part at least works.

build


BLE_URLBeacon: Use extern c to import nordic sdk files written in c

This is to prevent examples being broken in the pending update to ble-nrf51822.

@andresag01

+1


Redirect to session detail from logout link

Fixes https://github.com/matrix-org/matrix-authentication-service/issues/1550
Redirects to session detail when query param action=session_end is encountered.
https://github.com/matrix-org/matrix-authentication-service/assets/3055605/e1ec182e-7310-4127-8434-bad217811b19

Alright, I:

refactored a bit the router
did the backend part of the redirect so that params are kept through the login
implemented the other action defined in the MSC on the frontend


Attempts to address my missing CSS

Hello @thismatters,
Please check this out and consider merging this, or maybe some feedback if you think some things need to be changed. I'm a little skeptical whether this is appropriate in every case -- since /build/bundle.css shouldn't be available unless it's requested using the parent path /static/build/bundle.css.

Sorry for the really late response here! I am reworking the way CSS is discussed in the docs and handled in the package to hopefully address the issue that motivates this PR.
Since you've got one known CSS file to bring in to the template it seems reasonable to just link it in the template directly as a static file, no?


sqlite3.OperationalError: database is locked

Provide a general summary of the issue in the Title above.

Expected behavior

Tell us what should happen.

Current behavior

Tell us what happens instead of the expected behavior.

Screenshot [ Optional ]

Make a screenshot to help us understand the context more clearly.

Possible reasons and solutions [ Optional ]

Suggest a fix/reason for the bug.

Context

How has this issue affected you? How it appeared? What are you trying to accomplish?
Providing context helps us come up with a solution that is most useful in the real world.

Custom values [Optional]

NO

Your environment
*ARM DEBIAN 11
Stack trace

Include stack trace of the logs/console.

Checklist

Go over all the following points, and put an x in all the boxes that apply.

[ ] I searched the current issues and the issue is not known.
[ ] I searched on Google, Steam, YouTube, Reddit and there's no fix.
[ ] I am mostly sure that the problem is related to the project and isn't my fault.

Do in main dir of YukkiMusic
rm YukkiMusicBot.session
rm YukkiMusicBot.session-journal
And start the bot !

Do in main dir of YukkiMusic
rm YukkiMusicBot.session rm YukkiMusicBot.session-journal
And start the bot !

But the error still exists

Do in main dir of YukkiMusic

rm YukkiMusicBot.session rm YukkiMusicBot.session-journal

And start the bot !

But the error still exists

Delete current directory (for clearing old saved sessions which cause lock of database) and re clone it.

Do in main dir of YukkiMusic

rm YukkiMusicBot.session rm YukkiMusicBot.session-journal

And start the bot !

But the error still exists

Delete current directory (for clearing old saved sessions which cause lock of database) and re clone it.
OR
Do command 'ls' in main directory, if you find any '.session' or '.session-journal' files, remove it.

Thanks,I have done that,but the error still exists, is there something wrong with the python version(3.9.2)?

Do in main dir of YukkiMusic

rm YukkiMusicBot.session rm YukkiMusicBot.session-journal

And start the bot !

But the error still exists

Delete current directory (for clearing old saved sessions which cause lock of database) and re clone it.

OR

Do command 'ls' in main directory, if you find any '.session' or '.session-journal' files, remove it.

Thanks,I have done that,but the error still exists, is there something wrong with the python version(3.9.2)?

Nope ig 🙂, try upgrading python if you believe in it.

Do in main dir of YukkiMusic

rm YukkiMusicBot.session rm YukkiMusicBot.session-journal

And start the bot !

But the error still exists

Delete current directory (for clearing old saved sessions which cause lock of database) and re clone it.

OR

Do command 'ls' in main directory, if you find any '.session' or '.session-journal' files, remove it.

Thanks,I have done that,but the error still exists, is there something wrong with the python version(3.9.2)?

Nope ig 🙂, try upgrading python if you believe in it.

Thank you,I'll try that later!

Try
sudo pkill -9 python3
rm -r *.session
Than tmux and bash start

marking this as closed, as seems like it has been solved

ave done that,but the error still exists, is there something wrong with the python version(3

the error still exists


Custom Tags per TaggedItemBase not connected to objects in admin/taggit/

Hello everyone.
So I created a model with Custom Tags:
class ArticleTaggedItemBase(TaggedItemBase):
    content_object = models.ForeignKey('Article')

class Article(models.Model):
    title = models.CharField(max_length=300)
    comment = models.CharField(max_length=600, blank=True)

    tags = TaggableManager(through=ArticleTaggedItemBase)

Works fine from what I understand but the problem is that you can't see the tags in /admin/taggit:

Here's one tagged model object with this tag:

So as you can see the tags show up fine within the tagged model object but the connection doesn't show up in the /admin/taggit view.
I guess that Custom Tags don't work with that view properly?
If so, and if there's no simple way to correct this, how do I turn off the /admin/taggit link altogether?

I'm having the same issue.

Same issue if primary keys are UUIDs. django-taggit creates a new table in the database which I am not happy about. It does it as I created a class derived from GenericUUIDTaggedItemBase in my django app. Taggit should have a setting if your primary keys are UUID.

Yep, as people have identified this is due to custom tag models using different tables etc. Seems very doable for us to improve this (or at least document some workarounds)

Any update on this issue @rtpg?

@ggiller-collective no progress has been made on this specific point.
Generally if you make a custom tag, you should be able to add that model to the admin like any other model (generally in some admin.py file, see the Django docs)
As to admin/taggit appearing at all in that case.... I have not experimented with this, but you might be able to use AdminSite.unregister to remove taggit.models.TaggedItemBase or otherwise remove admin/taggit entirely from the admin views.  We might want to add an option to do this or something.
This is all working "as designed", in that derived models won't share the same admin pages as the base models because they're not the same model! But we can clearly make improvements given people coming here and asking this


segfault cudf re-use of data with many trials.

segfault_xgb140.zip
When applying this work-around: https://github.com/h2oai/xgboost/commit/8e319d48411617f221fc3c84cf3769b9f91034fb
I get a segfault eventually after some number of trials of the loop, say 40 out of 100.  Sometimes second trial.
This is just re-using the cudf data in the loop.  I'm using rapids 0.14, but haven't tried new rapids yet.  Perhaps someone can help try or explain why the work-around would lead to a segfault.
In real use I'm using Optuna with different hyperparameters, but even same parameters every fit and avoid re-using same model instance still segfaults sometimes.

Backtrace:
/home/jon/minicondadai/lib/python3.6/site-packages/xgboost/lib/libxgboost.so(+0x32c643)[0x7f2a55451643]
/home/jon/minicondadai/lib/python3.6/site-packages/xgboost/lib/libxgboost.so(+0x32c643)[0x7f2a55451643]
/home/jon/minicondadai/lib/python3.6/site-packages/numpy/core/../../../.././libgomp.so.1(+0x14685)[0x7f2ca3a62685]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x76db)[0x7f2ca6df26db]
/lib/x86_64-linux-gnu/libc.so.6(clone+0x3f)[0x7f2ca6b1ba3f]

Backtrace:
/home/jon/minicondadai/lib/python3.6/site-packages/xgboost/lib/libxgboost.so(+0x32c643)[0x7f2a55451643]
/home/jon/minicondadai/lib/python3.6/site-packages/xgboost/lib/libxgboost.so(+0x32c643)[0x7f2a55451643]
/home/jon/minicondadai/lib/python3.6/site-packages/numpy/core/../../../.././libgomp.so.1(+0x14685)[0x7f2ca3a62685]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x76db)[0x7f2ca6df26db]
/lib/x86_64-linux-gnu/libc.so.6(clone+0x3f)[0x7f2ca6b1ba3f]

The mentioned work-around was bad, but shouldn't have caused segfault.

The mentioned work-around was bad, but shouldn't have caused segfault.


Wfix

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Non noarch packages should have python requirement without any version constraints.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Non noarch packages should have python requirement without any version constraints.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The build section contained an unexpected subsection name. python is not a valid subsection name.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

@ocefpaf Would you be able to help figure out what to do about this package? Please ignore the embarrassing mess of things I tried in this pull request. Also tagging @mhearne-usgs since he would probably like to follow this conversation.
The main issue is that the noarch version of this package was not able to install in Windows (details in issue #27) so I wanted to modify the recipe to use build.sh and bld.bat scripts for the different architectures, hoping that I could eventually resolve the Windows issue with the bld.bat script. It is odd that the package installs via pip in Windows, and this conda package is built from pypi package.
The current status is:

Linux and OSX builds are not able to solve for the dependencies (OSX did solve at one point, but I relaxed the dep versions hoping it would resolve the linux deps and that ended up breaking the OSX deps)
Windows builds for python 3.6 and 3.7 give UnboundLocalError: local variable 'out' error, and there's something about unicode before that but I am not able to make sense of it.
Windows builds for python 3.8 don't have the UnboundLocalError but are not able to resolve dependencies.

Any guidance you have would be much appreciated!

@ocefpaf Would you be able to help figure out what to do about this package? Please ignore the embarrassing mess of things I tried in this pull request. Also tagging @mhearne-usgs since he would probably like to follow this conversation.

I can try to look into this PR tomorrow but I just tried this on a Winddows machine:
conda create --name TEST openquake.engine

with this .condarc:
show_channel_urls: true
channel_priority: strict
channels:
  - conda-forge
  - defaults

Everything worked.

The main issue is that the noarch version of this package was not able to install in Windows (details in issue #27) so I wanted to modify the recipe to use build.sh and bld.bat scripts for the different architectures, hoping that I could eventually resolve the Windows issue with the bld.bat script. It is odd that the package installs via pip in Windows, and this conda package is built from pypi package.

I'm not sure that is the issue. noarch packages are is installable using the new env and the condarc above.

The current status is:

Linux and OSX builds are not able to solve for the dependencies (OSX did solve at one point, but I relaxed the dep versions hoping it would resolve the linux deps and that ended up breaking the OSX deps)
Windows builds for python 3.6 and 3.7 give UnboundLocalError: local variable 'out' error, and there's something about unicode before that but I am not able to make sense of it.
Windows builds for python 3.8 don't have the UnboundLocalError but are not able to resolve dependencies.

Any guidance you have would be much appreciated!

I'll send an updating PR and test all platform in a moment.


libcouchbase: 2.7.2 -> 2.7.6

Motivation for this change
Version Update of libcouchbase. Needed for a following PR when adding php couchbase extension to nixpkgs.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

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

Thank you


Proposals list - merge Homebase proposals with Lite proposals (change lite proposals schema)

Standardize proposal data structures across off-chain and on-chain proposals.

Hey team! Please add your planning poker estimate with Zenhub @EightRice @fabiolalombardim @Man-Jain


Modbus switch register support

Description:
Added support for modbus switch based on registers.
Related issue (if applicable): fixes #10432
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
switch:
  platform: modbus
  slave: 1
  coils:
    - name: Switch1
      slave: 2
      coil: 14
  registers:
    - name: Register1
      slave: 1
      register: 11
      command_on: 1
      command_off: 0
      verify_state: True
      verify_register: 12
      register_type: holding
      state_on: 6
      state_off: 7

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Does someone have suggestion about how to fix this build?


Fallback to old dashboard view if monitoring is not healthy

Proposed changes

When MonitoringEnabled  === True, monitoring is ready.
Types of changes

Bugfix (non-breaking change which fixes an issue)

Linked Issues
https://github.com/rancher/rancher/issues/19197

Further comments

Need to merge UI and Backend PR at the same time.
Backend PR for master branch is https://github.com/rancher/rancher/pull/19259

LGTM


Update ESP_Mail_Client.h

Make getEncodingFromCharset(), decodeLatin1_UTF8(), decodeTIS620_UTF8() and RFC2047Decoder public.
Description:
This makes getEncodingFromCharset(), decodeLatin1_UTF8(), decodeTIS620_UTF8() and RFC2047Decoder public (see discussion #288 ).
Type of change:

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Testing:

[ ] merged with the current development branch (before testing)
[ ] CI build finished without issues
[x] Tested on real hardware (List board here)
[x] Changes are backward compatible

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Please add ENABLE_IMAP guard over those moving functions and class.


Add initial setup

Hello code reviewer,
I have added the actions below:

Added linters
Addded gitflow
Added react initial setup.

Hi @jlberbesi,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because the README file is unprofessional and lacks all the relevant headers. For a more professional README file visit this template
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

@addod19 A new README file was added. Thanks for your feedback.


Foreach statement makes EXO cmdlets much slower

When foreach statement is used, Get-MailboxPermission is way faster than Get-ExoMailboxPermission. 50min vs 3h.
We have about 4000 mailboxes.  Does anyone know why this happens as according to the document, EXO cmdlets should be faster than the old ones.
Sample code below:
# If Type is EXO, the EXO Cmdlet will run. $type = 'EXO' $global:AllUserMailboxInfo = Get-EXOMailbox -Filter {RecipientTypeDetails -eq 'UserMailBox'} -PropertySets Minimum -ResultSize unlimited $permissions = foreach ($user in $AllUserMailboxInfo) {     if ($type -eq 'EXO')     {$user | Get-EXOMailboxPermission -verbose:$false}     else     {$user | Get-MailboxPermission -verbose:$false} }

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: de2836a0-0af3-c147-2d4a-78220ac4e0f1
Version Independent ID: 2cde9735-6281-bb9f-bf07-b30b2f6c7f37
Content: About the Exchange Online PowerShell V2 module and V3 module
Content Source: exchange/docs-conceptual/exchange-online-powershell-v2.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@yukiiis, if you're using v3.1.0 of the ExchangeOnlineManagement module, try using the latest Preview version to see if that helps (v3.2.0-Preview3). Or if you're using a Preview version of the module, try switching to v3.1.0.
You can try sending details to exocmdletpreview[at]service[dot]microsoft[dot]com to see if they have any suggestions.

Thanks @chrisda ! I was using v3.1.0, Have just installed the latest Preview version (v3.2.0-Preview3), but still the same. EXO cmdlets took much longer to complete.

@yukiiis Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


ERR getState | {"code":-10002}

ERR getState | {"code":-10002}

Hey! I've been looking around about that code {"code":-10002}, and it looks like it means access denied. Everywhere I could find references to it, they claimed it's because it's a Chinese model not allowed to work in Europe (or vice-versa). They mention there's the possibility to flash it with a firmware that doesn't apply the regional block (https://forum.xda-developers.com/android/apps-games/app-xvacuum-firmware-xiaomi-vacuum-t3896526/page3)
Does it make sense?

Hey! I've been looking around about that code {"code":-10002}, and it looks like it means access denied. Everywhere I could find references to it, they claimed it's because it's a Chinese model not allowed to work in Europe (or vice-versa). They mention there's the possibility to flash it with a firmware that doesn't apply the regional block (https://forum.xda-developers.com/android/apps-games/app-xvacuum-firmware-xiaomi-vacuum-t3896526/page3)
Does it make sense?

I'll try that and answer later. Thx

Can we get the around this region block somehow in home bridge? AFAIK, using Xiaomi Mi Home App on a phone which has Chinese SIM card and having UTC+8 can bypass the region block.
Can we somehow simulate this behaviour via home bridge?

Maybe changing the TZ of the host where homebridge is running? 🤔

but how does the device know the TZ of the homebridge host? I guess there is a way to hijack the miio communication? not very familiar on the miio level, any idea?

If you run homebridge with the environment variable DEBUG=* you'll see all the underlying messages in the miio protocol.

I'll go ahead and close this issue for now. Please, feel free to reopen it if you think it's worth investing more time in it (and can provide the requested debug logs)


useExperiment() port to react-use lib proposal.

Do you think useExperiment hook would fit react-use lib?
Usage:
const AddToCartButtonExperiment = () => {
  const experimentConfig = {
    id: "3143106091",
    name: "add-to-cart-green",
    variants: [{ name: "control", weight: 50 }, { name: "test", weight: 50 }]
  };

  const { variant: { name } } = useExperiment(experimentConfig)

  if (name === "control") {
     return <button class="black">Add to cart</button>
  } else if (name === "test") {
     return <button class="green">Add to cart</button>
  }
}

Codesendbox demo: https://codesandbox.io/embed/ab-react-hook-playground-4crjn

Feels like useExperiment should be plain JavaScript function. Why make it a hook?


Speaker id design tweaks

Consider moving the speaker identification icon so that it is between the status icon and the start of the text.
Consider reworking the design of the speaker id icon from silhouette and plus sign to silhouette and talk bubble (or another option)
Update the speaker id colors from a color safe palette.

updated colors, moved speakers to the right of the status icon


c_api.h uses undefined THREAD_LOCAL macro

Environment info
Operating System: x86_64-linux-gnu
CPU: Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz
C++/Python/R version: gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.9)
Error message
/usr/local/include/LightGBM/c_api.h: In function ‘char* LastErrorMsg()’:
/usr/local/include/LightGBM/c_api.h:765:38: error: ‘THREAD_LOCAL’ does not name a type
static char* LastErrorMsg() { static THREAD_LOCAL char err_msg[512] = "Everything is fine"; return err_msg; }
^
/usr/local/include/LightGBM/c_api.h:765:100: error: ‘err_msg’ was not declared in this scope
static char* LastErrorMsg() { static THREAD_LOCAL char err_msg[512] = "Everything is fine"; return err_msg; }
Reproducible examples
In any source file, including <LightGBM/c_api.h>
results in compile error about undefined THREAD_LOCAL
The only workaround is to include <LightGBM/utils/log.h> before including <LightGBM/c_api.h>. Moving the definition of THREAD_LOCAL to c_api.h (or export.h) would fix the issue.

that is very strange, we have CI to check the build in linux. So it should not have the compile issue.

You need to include <LightGBM/c_api.h> first thing in c_api.cpp to actually catch this issue. This is also a good practice in general to ensure that header files are self-sufficient:
https://google.github.io/styleguide/cppguide.html#Self_contained_Headers
https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes

The library build works fine.
When calling the library from a c program, I only include c_api.h.
My work around was to #define THREAD_LOCAL thread_local for non-widows.


Using DefaultDbModelStore throws exception

I posted this here before: https://github.com/aspnet/EntityFramework6/issues/275
But this issue is closed already and maybe I should create a seperate issue anyway so here we go.
The new feature from 6.2 "Reduce start up time by loading finished code first models from a persistent cache" is not working for me.
What I did is very simple and I think exactly the way it's supposed to be done:
I saved the model using
EdmxWriter.WriteEdmx(context, writer);
In my DbConfiguration implementation I set
SetModelStore(new DbModelStore());
DbModelStore is my implementation of DefaultDbModelStore where I simply overwrite GetFilePath().
When making the first call to EntitySet I get
InvalidOperationException: The entity type EdmMetadata is not part of the model for the current context.
System.Data.Entity.Internal.InternalContext.UpdateEntitySetMappingsForType(Type entityType) +273
System.Data.Entity.Internal.InternalContext.GetEntitySetAndBaseTypeForType(Type entityType) +31
System.Data.Entity.Internal.Linq.InternalSet1.Initialize() +77 System.Data.Entity.Internal.Linq.InternalSet1.AsNoTracking() +29
System.Data.Entity.Infrastructure.DbQuery1.AsNoTracking() +75 System.Data.Entity.Internal.EdmMetadataRepository.QueryForModelHash(Func2 createContext) +159
System.Data.Entity.Internal.ModelCompatibilityChecker.CompatibleWithModel(InternalContext internalContext, ModelHashCalculator modelHashCalculator, Boolean throwIfNoMetadata, DatabaseExistenceState existenceState) +120
System.Data.Entity.Internal.InternalContext.CompatibleWithModel(Boolean throwIfNoMetadata, DatabaseExistenceState existenceState) +77
System.Data.Entity.CreateDatabaseIfNotExists1.InitializeDatabase(TContext context) +155 System.Data.Entity.Internal.InternalContext.PerformInitializationAction(Action action) +71 System.Data.Entity.Internal.InternalContext.PerformDatabaseInitialization() +482 System.Data.Entity.Internal.RetryAction1.PerformAction(TInput input) +174
System.Data.Entity.Internal.LazyInternalContext.InitializeDatabaseAction(Action1 action) +273 System.Data.Entity.Internal.InternalContext.GetEntitySetAndBaseTypeForType(Type entityType) +20 System.Data.Entity.Internal.Linq.InternalSet1.Initialize() +77
System.Data.Entity.Internal.Linq.InternalSet1.AsNoTracking() +29 System.Data.Entity.Infrastructure.DbQuery1.AsNoTracking() +75
[my call to EntitySet]
I have been able to get it to work when I create a DbCompiledModel instead:
var model = EdmxReader.Read(reader, "dbo");
I can then pass the model to the constructor of the context. However this is much more complicated mostly because of the stupid null check of the model in the constructor (!) so I have to make sure to call the right constructor overload depending on whether I already have a saved model or not.
A fix for the null check or even better an explanation why the DefaultDbModelStore is not working would be appreciated. At the moment I consider both to be bugs.

@srudin I don't believe that the feature is intended to be used in that way. You should not need to read/write EDMX files manually. Instead you should just need to configure the model store using something like this:
public class MyDbConfiguration : DbConfiguration
{
    public MyDbConfiguration() : base()
    {
        this.SetModelStore(new DefaultDbModelStore(Directory.GetCurrentDirectory()));
    }
}

(Taken from https://blogs.msdn.microsoft.com/dotnet/2017/05/23/announcing-ef-6-2-beta-1/)

Hm yes, your are right. Using it like this works. However I don't understand the difference between this and my solution which is basically nothing else but a different file path. The generated files are binary identical.
I was using it this way because I followed this blog post: https://www.fusonic.net/en/blog/ef-cache-deployment/


parallelize unit tests

We can make the unit tests go faster, by running all of the python versions in parallel.
Per this doc, nox allows specifying python version on the command line, e.g.: python3 -m nox --python=3.7
We can use Cloud Build's parallel steps to test py 3.6, 3.7, and 3.8 (as specified in the noxfile) simultaneously.

Per my testing, parallelizing didn't make it much faster (~7m50s --> ~7m25s). I don't think it's worth the [minor] extra complexity for that [minor] degree of improvement. Closing this.

Per my testing, parallelizing didn't make it much faster (~7m50s --> ~7m25s). I don't think it's worth the [minor] extra complexity for that [minor] degree of improvement. Closing this.


.dockerignore and Dockerfile ignore

Docker 1.5 introduced the ability to ignore both the .dockerignore and the Dockerfile (in the .dockerignore file), but this is processed on the server since and only for dealing with the build cache (ie, when someone does ADD ., Dockerfile changes can be ignored).
Currently with the docker-compose 1.2 RC2 build, if Dockerfile is in .dockerignore, Dockerfile won't be sent to the server, so the server can't be built.
Basically, .dockerignore, and Dockerfile should be treated as special case entries in the .dockerignore so that they still get sent up to the server.

I suspect this support will be entirely in docker-py and we won't need to make any code changes to compose. compose doesn't deal with .dockerignore at all.
Should we move this issue to the docker-py project?

That makes sense.


bug(python): if the search text includes "&" or "-", the query will fail, seems the tantivy lib problem.

LanceDB version
v0.5.5
What happened?
if the search text includes "&" or "-", the query will fail, seems the tantivy lib problem.
Are there known steps to reproduce?
No response

Try quoting the query. If that doesn't work, you can call the .phrase_query() method.

@prrao87 and I have added more documentation to clarify this: https://github.com/lancedb/lancedb/pull/982


"synchronous transformer must export a "process" function" when using processAsync

To Reproduce
module.exports = {
	async processAsync(src, filepath, config) {
		// ..
	},
}

When running on CI I get this error:
FAIL ./test.js
  ● Test suite failed to run

    Jest: synchronous transformer /home/runner/work/jest-transformer-mdx/jest-transformer-mdx/index.js must export a "process" function.

      at invariant (node_modules/@jest/transform/build/ScriptTransformer.js:1091:11)

You can see the CI result here, and you can view the transformer here.
Expected behavior
It matches what happens when I run the tests locally:
><EMAIL_ADDRESS>test:index
> jest --config jest.config.js

 PASS  ./test.js
  jest transformer md
    ✓ that frontMatter is correct (1 ms)
    ✓ that it still renders in react (21 ms)

Test Suites: 1 passed, 1 total
Tests:       2 passed, 2 total
Snapshots:   1 passed, 1 total
Time:        0.863 s, estimated 1 s
Ran all test suites.

Link to repl or repo (highly encouraged)
All the files are in the public repo: https://github.com/bitttttten/jest-transformer-mdx
envinfo
CI
  System:
    OS: Linux 5.4 Ubuntu 20.04.2 LTS (Focal Fossa)
    CPU: (2) x64 Intel(R) Xeon(R) CPU E5-2673 v4 @ 2.30GHz
  Binaries:
    Node: 14.17.0 - /opt/hostedtoolcache/node/14.17.0/x64/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.13 - /opt/hostedtoolcache/node/14.17.0/x64/bin/npm
  npmPackages:
    jest: ^27.0.1 => 27.0.1

Local machine
  System:
    OS: macOS 11.2.1
  Binaries:
    Node: 14.15.4 - ~/.nvm/versions/node/v14.15.4/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.10 - ~/.nvm/versions/node/v14.15.4/bin/npm
  npmPackages:
    jest: ^27.0.1 => 27.0.1

I see in https://github.com/facebook/jest/issues/11226 that async transformations are only supported in ESM mode. So is there any way to use an async transformer without relying on users of the transformer to run jest in ESM mode?
My use case is that I am using mdx, which is async by default. So it's not really possible to write this transformer synchronously 🤔

I am trying to emulate the setup in your tests following this file.
// jest.config.js
transform: {
		"^.+\\.js$": "babel-jest",
		"^.+\\.(md|mdx)$": [
			path.resolve(__dirname, "mdx-transformer.cjs"),
			{
				mdxOptions: {
					rehypePlugins: [rehypeSlug],
				},
			},
		],
	},

// mdx-transformer.cjs
"use strict"

const mdx = require("@mdx-js/mdx")
const { process: babelProcess } = require("babel-jest")

module.exports = {
	async processAsync(src, filepath, config) {
		const jsx = await mdx(withFrontMatter, { ...config?.transformerConfig?.mdxOptions, filepath })
		const toTransform = `import {mdx} from '@mdx-js/react';${jsx}`
		return babelProcess(toTransform, filepath, config).code
	},
}

I then run it with NODE_OPTIONS=--experimental-vm-modules npx jest --no-cache, and I still hit the Jest: synchronous transformer jest-transformer-mdx/mdx-transformer.cjs must export a "process" function.
I also played around with different ways of running Jest as per the ECMAScript Modules docs like node --experimental-vm-modules node_modules/.bin/jest etc. and I can see it is enabled:
(node:30693) ExperimentalWarning: VM Modules is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
 FAIL  ./test.mdx-options.js
  ● Test suite failed to run

    Jest: synchronous transformer jest-transformer-mdx/mdx-options.cjs must export a "process" function.

I feel like I am missing something because I am really trying to copy what is in your tests.
Also I understand if this goes out of scope of this issue :)

process is a fallback for processAsync so that’s why process is always required IIRC.

Do you mean to stub process? I had tried that with:
module.exports = {
	process() {
		return ""
	},
	async processAsync(src, filepath, config) {
		// omitted
	},
}

Which sadly doesn't work. It's using the return value from process 🤔

Not sure if .cjs is the issue, what if you use .js for your transformer?

I did come across extensionsToTreatAsEsm in the docs, however it noted that  ".mjs" files were picked up already, so I tried renaming my transformer to mdx-transformer.mjs and refactoring it to support ESM.
import path from "path"
import matter from "gray-matter"
import stringifyObject from "stringify-object"
import mdx from "@mdx-js/mdx"
import babelJest from "babel-jest"

async function processAsync(src, filepath, config) {
	// omitted
}

export default { processAsync }

However that still gave me Jest: synchronous transformer jest-transformer-mdx/mdx-transformer.mjs must export a "process" function.. I tried including .md in the array also and got:
 FAIL  ./test.mdx-options.js
  ● Test suite failed to run

    Must use import to load ES Module: jest-transformer-mdx/test.md

      1 | import React from "react"
    > 2 | import Component, { frontMatter } from "./test.md"
        | ^

I think I am getting very stuck, I have tried:

Transformer written in CommonJS, always gets must export a "process" function with or without VM Modules feature enabled, and without extensionsToTreatAsEsm: [".md"],

Transformer written in CommonJS with extensionsToTreatAsEsm: [".md"], gets Must use import to load ES Module: jest-transformer-mdx/test.md

Transformer written with ESM with extensionsToTreatAsEsm: [".md"] gets Must use import to load ES Module.

Transformer written with ESM without extensionsToTreatAsEsm: [".md"] gets must export a "process" function.

Even though I know these combinations don't work, I just wanted to explore to see if I got any different errors. On paper, it should work with only exporting processAsync, in mdx-transformer.cjs, with VM Modules enabled as that is like you have it in your test files. I tried to follow the examples in your test files, although I hit similar issues too.

The transformer can be either ESM or CJS, extensionsToTreatAsEsm is for the files that are transformed - i.e. you want extensionsToTreatAsEsm: ['md', 'mdx'] as you have.
You get Must use import to load ES Module since you use require to load it - use import or import()

What do you mean with "since you use require to load it"? Do you mean inside my test file? As in my test I import the file with import, like Import * as Test from "./test.md". Maybe I am missing something here, so sorry if I don't understand that :)
So I tried taking this step by step, and wrote the transformer with ESM. I could only get it running without getting Cannot use import statement outside a module errors unless the transformer file extension is .mjs. I can't seem to get this error to go away without renaming the file.
No problem though, now the transformer is written with ESM, with the file extension mjs. However now I still get the error from my original issue.  I am kind of put off by this since it would mean users of the transformer would need to do that too.
With CJS, it runs fine but I also hit the same error from my original issue, synchronous transformer mdx-options.js must export a "process" function.

@bitttttten You might want to try adding "type": "module" to your package.json - this is what worked for me 😅

I ran into this when trying to port a cascading transform from Jest 26 to Jest 27 because the ScriptTransformer export was removed from @jest/transform, and the package.json exports prevents the easy workaround. The old codepath would use a new ScriptTransformer to create a synchronous transformer, but as only the async createScriptTransformer is available now, it's no longer possible to chain transformers, at least not in the way this transformer (https://github.com/storybookjs/storybook/blob/next/addons/storyshots/storyshots-core/injectFileName.js) did it, so it seems I cannot (obviously) port it to Jest 27.
"it will be easy, I just need to change , config, { instrument } to , { config, instrument }" 😆

I see in #11226 that async transformations are only supported in ESM mode. So is there any way to use an async transformer without relying on users of the transformer to run jest in ESM mode?
My use case is that I am using mdx, which is async by default. So it's not really possible to write this transformer synchronously 🤔

require is synchronous, so async transform is only supported when using ESM.
As for your following error, I assume it's because by default jest loads most files as CJS, you can use extensionsToTreatAsEsm as mentioned in the docs to tell jest that markdown files will be ESM.
You shouldn't need to define process if you transformer is only used asynchronously (i.e. in ESM mode)

But this is just the module resolution, not the execution of the function. If this is true, then no one using a CJS library would be able to use an async function. Which I don't think is true.

@Jessidhia We discussed some sort of async "setup" function, but ended up with "ESM is enough" (https://github.com/facebook/jest/issues/11081#issuecomment-791259034). Not sure if that would help in your case... Supporting some way of composing multiple transforms would be pretty nice. Could you open up a new issue with that use case?

@sebastianrothe I don't really follow what you're saying. Jest implementing JIT transforms, so require needs to remain synchronous - this has nothing to do with resolution at all. And you can import CJS code via import, so you should never be blocked by your libraries being CJS

@SimenB I was wondering, if an exclusive (processAsync only) asyncTransformer in CJS is possible. We maintain https://github.com/mihar-22/svelte-jester and replaced processwith processAsync in our transformer. We also switched the code to ESM, but now the users of the library have a lot of problems with the experimental flags required for ESM.

That's not possible unfortunately as CJS cannot be asynchronous

That's not possible unfortunately as CJS cannot be asynchronous

But then this test https://github.com/facebook/jest/blob/master/e2e/transform/async-transformer/my-transform.cjs gets transpiled to ESM?

I have been waiting months to see whether this issue would be addressed.  When I update to Babel/Jest 27 all of my almost 300 test suites fail with "Jest: a transform must export a process function."

I'll close this as the OP answered themselves in the first comment.

So is there any way to use an async transformer without relying on users of the transformer to run jest in ESM mode?

No, when a user calls require, the result must be returned synchronously, there is no way for that to ever by async. People will need to use ESM.
I'll add that you can use import() from within CJS, which is async, but the file that is imported will still need to be native ESM. But the use case if the OP (MDX) that can probably work - just transform to ESM rather than CJS (please see docs, you still need the runtime flag to node and extensionsToTreatAsEsm option).

@CourtHive feel free to open up a new issue with minimal reproduction steps.


add arm template to test release pipeline, change the branch of js-mgmt-ci and js-mgmt-pr, change release pipeline for mgmt

This PR contains 3 parts:

changes of release pipeline for track2 mgmt sdk, which is same as: https://github.com/Azure/azure-sdk-for-js/pull/15975
change the trigger in mgmt-ci.yml and mgmt-pr.yml to disable it in main branch (we will release track1 sdk in branch release/track1)
add arm template to sdk/template to test release pipeline

@dw511214992 I recall that in our last sync, we agreed that there was no need for a template package for mgmt plane. Data plane and mgmt plane packages in Track 2 should be similar for all intents and purposes, so we shouldnt need a separate template package. Can you share if anything changed since then that we now need this?

hi @ramya-rao-a I have compared the generated codes of track2 JS SDK to the dataplane sdk, and I think they are different:

the package name's format and sdk-type is different. So we need to test it in updated release pipeline to make sure it works.
dataplane has more commands/codes, such as tests, samples and so on. We need to make sure the pipeline works when mgmt plane sdk doesn't have.
The package @azure/arm-template‘s file structure and contents is the same as track2 JS sdk, which is totally different from @azure/template. I believe it's very important to add this package.
Add package @azure/arm-template to directory sdk/template can test the scenario: a rp contains both dataplane sdk and mgmt sdk. We need to make sure the release pipeline can handle this scenario.

Thanks

I am still not convinced that we need an arm-template package..
If the intention is to test the release pipelines, then can we not do it using the system that publishes nightly builds on one of the real RPs?
The intention of the template package was to document best practices around folder structure, usage of scripts and different configuration for when package owners hand write SDKs. Mgmt plane packages are being planned to be purely auto generated with very little human interference.

@ramya-rao-a okay, get it. I will close this pr and create a new pr https://github.com/Azure/azure-sdk-for-js/pull/16026 to disable the original pipeline for mgmt. About supporting track2 mgmt plane sdk, we will track it in https://github.com/Azure/azure-sdk-for-js/pull/15975
thanks


Next step, ECMAScript 2015 and Meteor 1.3.1

Use ECMAScript 2015 and Meteor 1.3.1 to rewrite this package.
TODO

ECMAScript 2015 class support
Unit tests
Auto generate documents

published


Windows release?

Hello, can You please add a Windows release so one can download it at https://github.com/djey47/tdumt/releases and just run or install on Windows?
I am only aware about v. 1.16.3 downloadable from https://turboduck.net/forums/topic/6702-djey-tdu-modding-tools-1163

Hi,
just created a release here. That's the same archive one could find via Turboduck or sharemods.
Regards.


GhoVariableDebtToken V2 used for update on GHO and Aave V3 Ethereum Pool

Code used for https://app.aave.com/governance/proposal/308/

Creating this PR just for reference.


ZH-trans: In v1.12 version update install-kubeadm

ref: https://github.com/kubernetes/website/pull/11731/files

/assign @markthink  @xiangpengzhao

/lgtm
/approve
Thanks for the cleanup.


Winget doesn't work when run from elevated cmd

Brief description of your issue
Winget works fine from non-elevated cmd. From elevated cmd it gives "Failed when searching source; results will not be included: winget" error on almost every command
Expected behavior
Expected to be able to use winget on elevated prompt without the need to enter admin password to install software
Actual behavior
If I run exactly same command on elevated and non-elevated prompts I get different behaviours

Source update doesn't make any difference

Comment
I think I have read though all the solutions I could find, most of them point to the issue with PATH, but in my case I can run the software, but it doesn't work as expected. Please advise.

Which version of winget is this?
The output from winget --info helps to troubleshoot.
Does winget run in administrator mode from "C:\Users\Administrator>"?
Try running winget source reset --force in the administrator prompt.

Hi,
See below. No change

Getting same issue from Administrator command prompt on Win 10:

The following fixed this issue for me (running from admin powershell), I installed the Microsoft.Winget.Source app with the following:
Invoke-WebRequest 'https://cdn.winget.microsoft.com/cache/source.msix' -OutFile 'c:\downloads\Microsoft.Winget.Source.msix' -UseBasicParsing
Push-Location 'c:\downloads';
Add-AppxPackage -Path 'Microsoft.Winget.Source.msix';
Pop-Location;

I am having the same issue; I can run as local user but not as admin from cmd and PowerShell.
The invoke command from above did not help in my case.

I am having the same issue; I can run as local user but not as admin from cmd and PowerShell.
The invoke command from above did not help in my case.

For me it's fixed in 1.5.101-preview


rustfmt makes a mess of unsafe

In:
fn main() {
    unsafe {
        let foo = 1u32;
    }
}

Out:
fn main() {
    {nsafe {
        let foo = 1u32;
    }
}

The extra initial blank line added by rustfmt also seems odd.

Thanks for the bug report. I'll have a look at this.


Node reference which do not contain non-dynamic path parts generate Regex errors.

Expected Behaviour
Node with defs like this should not generate errors
"someNode": { "type": "array", "sync": "{$path:/_links/someOtherNode}" }
Actual Behaviour
These nodes generate errors due to a string indexing miscalculation.
Reproduce Scenario (including but not limited to)
Steps to Reproduce
call buildValidModelInstance() on a ModelObject which consumes a schema which contains a node similar to the above.
Platform and Version
All
Sample Code that illustrates the problem
Shown Above
Logs taken while reproducing problem

Fixed


Re-publish to pursuit

See https://github.com/purescript/pursuit/issues/213

Done!
https://pursuit.purescript.org/packages/purescript-simple-moment/0.3.0


use of randomElementModQ when encrypting

Right now, we have two calls to group.randomElementModQ() in Encryptor.kt. These cause the encryption process to be non-deterministic. Ideally, encryption should have the option to be deterministic, so if you know the seed nonce(s), then you can derive all of the other nonces, all the way down.
We could have a flag when the Encryptor object is created that says "go ahead and be non-deterministic", which might be useful in some circumstances.

In Encryptor.kt, we now have two methofs:
fun encrypt(
      ballots: Iterable<PlaintextBallot>,
      codeSeed: ElementModQ
   ): List<CiphertextBallot>

and
fun encryptWithFixedNonces(
        ballots: Iterable<PlaintextBallot>,
        codeSeed: ElementModQ,
        masterNonce: ElementModQ
    ): List<CiphertextBallot>

which I think? handles this issue.
We do need to add deterministic option for the ContestData encryption, see issue #164

All done


Can not see price pr hour and cost pr hour in staff page

I have the role of price/cost hour but I can not see them into the staff


test: benchmark e2e install scenario

The scenario will install default TAS, sign and verify one image and delete TAS.
Example of command to install TAS 10x:
go test -tags=integration -benchtime=10x -timeout=1h -bench . ./test/e2e/benchmark/...

Example of command to run test for 1h:
go test -tags=integration -benchtime=1h -timeout=2h -bench . ./test/e2e/benchmark/...

/lgtm


Create sonar reports for dolphin polymer

SonarQube integration with Travis CI build

Add sonar-project.properties containing sonarqube details
Add soneqube plugin and sonar-scanner in travis.yml

LGTM


Remove label when variable is filled

Overview
I made so the label is not shown when you fill a variable
connected issues and PRs:
GN-4824
How to test/reproduce
You can insert one of each variables and fill it, appreciate that the label is indeed hidden, in some variables you can go back to an unfilled status, and the variable should reappear, but on others this is just not possible because of how the variable UI is made
Checks PR readiness

[x] UI: works on smaller screen sizes
[x] UI: feedback for any loading/error states
[x] Check if dummy app is correctly updated
[x] Check cancel/go-back flows
[x] changelog
[x] npm lint
[x] no new deprecations

The thinner version works with all of them, and I think is prefered, what I did now is to make it consistent, as some variable were 25px and some 29px, so I set all of them at 30px.
So we will have with label 35px and without 30px, I don't know is this is correct or I should make them all 35. Maybe @abeforgit can throw some light

The thinner version works with all of them, and I think is prefered, what I did now is to make it consistent, as some variable were 25px and some 29px, so I set all of them at 30px. So we will have with label 35px and without 30px, I don't know is this is correct or I should make them all 35. Maybe @abeforgit can throw some light

IMO we should probably use the same height whether a label is shown or not. This shows the user that they are handling the same type of data.

For me, I think thinner is better, as it reduces the impact of adding a variable to the rest of the layout. I think the most important thing is that they don't change size though, as it means the layout changes just by adding content, which to me would indicate that the type of node is changing.

You can review this again


0.3.0 patch

Merge fixes done on release-0.3.0 branch to master

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "0.3.0-patch" git@github.com:chungers/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354144896
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.