Partition resizing not finished

RancherOS Version: (ros os version)
Ros 0.4.5
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
Virtual machines in vCloud
I am trying to resize a partition on one of my ROS machine. First, I followed the documentation and added yml #cloud-config rancher:   resize_device: /dev/sdb
To my cloud-config, rebooted, but nothing happened.
I then followed workaround in #1237 and added
rancher:
  environment:
    RESIZE_DEV: /dev/sdb
  services_include:
    https://raw.githubusercontent.com/rancher/os-services/v0.4.5/r/resize-fs.yml: true

To my cloud config instead of resize_device: /dev/sdb, and after a reboot it seems to have started doing the job.
[root@my-server-01 ~]# system-docker logs resizefs
+ RESIZE_DEV=/dev/sdb
+ STAMP=/var/log/resizefs.done
+ '[' -e /var/log/resizefs.done ']'
+ '[' -b /dev/sdb ']'
+ growpart /dev/sdb 1
CHANGED: partition=1 start=63 old: size=41929587 end=41929650 new: size=83875302,end=83875365
+ partprobe /dev/sdb
+ resize2fs /dev/sdb1
resize2fs 1.42.9 (4-Feb-2014)
Please run 'e2fsck -f /dev/sdb1' first.

+ RESIZE_DEV=/dev/sdb
+ STAMP=/var/log/resizefs.done
+ '[' -e /var/log/resizefs.done ']'
+ '[' -b /dev/sdb ']'
+ growpart /dev/sdb 1
NOCHANGE: partition 1 is size 83875302. it cannot be grown
+ :
+ partprobe /dev/sdb
+ resize2fs /dev/sdb1
resize2fs 1.42.9 (4-Feb-2014)
Please run 'e2fsck -f /dev/sdb1' first.

So on the first reboot, growpart actually grew the partition, but resize2fs failed because it wants fsck first.
The result is my partition being the right size in fdisk -l, but the file system is not extended as df still reports the old size.
Is there any way I can trigger that fsck before the resize service kicks in ?

handing this to josh for now - he know way more about this than i atm.

I managed to work my way around this by running the resize container like this:
docker run --privileged=true --rm -ti --entrypoint=/bin/bash rancher/os-resizefs:v0.4.5

Then executing the following:
root@7ec6ac2a49be:/# growpart /dev/sdb 1
CHANGED: partition=1 start=63 old: size=41929587 end=41929650 new: size=83875302,end=83875365
root@7ec6ac2a49be:/# partprobe /dev/sdb
root@7ec6ac2a49be:/# e2fsck -f /dev/sdb1
e2fsck 1.42.9 (4-Feb-2014)
Pass 1: Checking inodes, blocks, and sizes
Pass 2: Checking directory structure
Pass 3: Checking directory connectivity
Pass 4: Checking reference counts
Pass 5: Checking group summary information
/dev/sdb1: 4579/1310720 files (14.0% non-contiguous), 4975044/5241198 blocks
root@7ec6ac2a49be:/# resize2fs /dev/sdb1
resize2fs 1.42.9 (4-Feb-2014)
Resizing the filesystem on /dev/sdb1 to 10484412 (4k) blocks.
The filesystem on /dev/sdb1 is now 10484412 blocks long.
root@7ec6ac2a49be:/# exit

The problem was e2fsck -f /dev/sdb1 missing from the original entrypoint before the resize2fs


Fix #314 ResetOnReporting does nothing

The issue or feature being addressed

https://github.com/AppMetrics/AppMetrics/issues/314

Details on the issue fix or feature implementation

ToSerializableMetric now calls source.ValueProvider.GetValue(source.ResetOnReporting) instead of source.Value.Count that are not resetting the count.

Changed the IMetricValueProvider in the Counter unit test from ConstantValueProvider to DefaultCounterMetric as ConstantValueProvider has not implemented the GetValue with resetMetrics correct.

Confirm the following

[x] I have ensured that I have merged the latest changes from the dev branch
[x] I have successfully run a local build
[x] I have included unit tests for the issue/feature
[x] I have included the github issue number in my commits

Hi @alhardy, when do you plan to release the fix?

@Maximusya yes, I'll release an pre-release sometime this week

i need this change as well, thanks!

@alhardy is there some way I can help getting this released?


fix: broken project URL in setup.py

I noticed on ipyvue's project page on pypi.org that the "Homepage" link points to https://github.com/widgetto/ipyvue/ rather than https://github.com/widgetti/ipyvue/ and decided to fix it.

Closing as a duplicate of https://github.com/widgetti/ipyvue/pull/85

Thanks!


Adding Docker support.

I've added docker configuration for this exporter. Running docker is easier for me and most of my services are running on it.
Added Dockerfile + entrypoint.sh to make configuration easier. Added a docker-compose.yml example and documentation in Readme.md
There is a simple configuration and change in order to publish the container under your own name (Adding a Token and changing the package name in the documentation. ) But once done, it should build automatically once you push to master.
Let me know if this is something you would like to merge, or if you want me to change anything. Thanks for your software !

Adding Docker support


Convert WSL 1 to WSL 2 fails with 'unable to find path'

Please use the following bug reporting template to help produce issues which are actionable and reproducible, including all command-line steps necessary to induce the failure condition. Please fill out all the fields! Issues with missing or incomplete issue templates will be closed.
If this is a console issue (a problem with layout, rendering, colors, etc.), please post to the console issue tracker.
Important: Do not open GitHub issues for Windows crashes (BSODs) or security issues.  Please direct all Windows crashes and security issues to<EMAIL_ADDRESS> Ideally, please configure your machine to capture minidumps, repro the issue, and send the minidump from "C:\Windows\minidump".\
See our contributing instructions for assistance.
Please fill out the below information:

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [Version 10.0.19041.264]
What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)

PS C:\WINDOWS\system32> wsl --set-default-version 2
WSL 2 måste uppdateras enligt kernelkomponenten. Mer information finns på https://aka.ms/wsl2kernel
PS C:\WINDOWS\system32> wsl --list --verbose
NAME            STATE           VERSION
Ubuntu-18.04    Running         1
PS C:\WINDOWS\system32> wsl --set-version Ubuntu-18.04 2
Conversion in progress, this may take a few minutes...
For information on key differences with WSL 2 please visit https://aka.ms/wsl2
Det går inte att hitta sökvägen. '(unable to find path)'

What's wrong / what should be happening instead:
The Ubuntu-18.04 instance is a WSL 1 installation from Windows 1909 and installed via the server method outside of the system disk on its own disk. It has been working perfectly for a year as far as I can tell and it is working now in Windows 2004. Is the upgrade procedure using a hardcoded path? Is there a way to specify an alternative path?

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here).

For WSL launch issues, please collect detailed logs.

See #5075

See #5075

Thanks, if  I understand it right you're suggesting I should uninstall hyper-v and then reinstall it? Well, it sounds a bit dangerous but I'll try it. I just upgraded to 2004 and I used hyper-v in 1909 and it worked without issue and it's working now as well.

Just FYI
PS L:\ubuntu> wsl.exe --export Ubuntu-18.04 ./Ubuntu-18.04.tar Det går inte att hitta filen.

Can you install a new distro from the Store? Just for testing.

Still haven't tried the hyper-v thing but a good idea to try another distro. I'll do it now.

Crap
C:\Windows\System32>ubuntu2004.exe
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070002
Error: 0x80070002 Det gÕr inte att hitta filen.

Press any key to continue...

That look bad. Plenty of threads with that error with most weird solutions like in #4618.
It's a different error because you can't even export. Do you use any AV or Firewall different from Windows Security?

Yes, I have a working Symantec which I managed to bring över from 1909. I hate to touch it. I usually do a clean Windows install just to avoid these things, but I didn't want the work this time. I'm reinstalling hyper-v now and I'll try again.

Nopp, reinstall hyper-v didn't change anything other than clean up my virtual networks.

Some people fixed that bug with:
open Turn Features On or Off
untick Windows Subsystem For Linux
reboot
tick Windows Subsystem For Linux
reboot

OK, tried that but it didn't work, gives WslRegisterDistribution failed with error: 0x80070002.
My old 1909 WSL 1 instance Ubuntu 18.04 survived though and is still working.

Hi there,
I've got the exact same problem. Any luck solving this?
Sticking to WSL 1 for now but I'd really like to get my distros on WSL 2 since it's supposed to perform better.
Thanks.

Hi there,
I've got the exact same problem. Any luck solving this?
Sticking to WSL 1 for now but I'd really like to get my distros on WSL 2 since it's supposed to perform better.
Thanks.

No, I eventually gave up on my upgraded Windows and did a clean install. In the clean Windows (2004 en_US) I directly tested to install WSL 2 which succeeded without issues. This was done using a dummy install-account. I then joined my AD network and logged in as myself and tried to install a new WSL 2 instance which resulted in:
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070002
Error: 0x80070002 Det gÕr inte att hitta filen.

Logging in on my previous dummy install-account and WSL 2 has stopped to work with a similar message. So what does this mean? I think one or more of the following causes:

WSL 2 is not fully compatible with Active Directory services
WSL 2 is depended on telemetry disabled by AD policy rules

Swedish MUI was installed by the dummy install-account and worked with WSL 2, no antivirus solution was installed, no hardware driver changes, system SSD formatted, no desktop applications or store apps installs other than Chrome Edge and Google Chrome.

Confirmed after a second clean install of Windows 2004. I had a hope that the WSL 2 upgrade package was prerelease and introduced telemetry dependencies. Unfortunately this isn't the case. Even if I avoid installing the WSL 2 upgrade, I cannot create new WSL 1 or WSL 2 instances as an AD user. However, now I can continue to use WSL 1 with my local dummy install-account even after joining the computer to AD.

For completeness and how I got a semifunctional Ubuntu 20.04 WSL 1 environment with the server method:

reverted back to Windows 1903
downloaded http://cloud-images.ubuntu.com/releases/focal/release/ubuntu-20.04-server-cloudimg-amd64-wsl.rootfs.tar.gz
run wsl.exe --import Ubuntu-20.04 ubuntu-20.04-server-cloudimg-amd64-wsl.rootfs.tar.gz E:\Linux\Ubuntu-20.04
wsl.exe adduser
installed Ubuntu-20.04 from store and run it once
upgrade Windows to 2004

Still cannot install any distros in Windows 2004 and now I hit the clock_nanosleep issue.

Made a network trace as requested at https://github.com/microsoft/WSL/issues/5107
new logfiles for trying to install Ubuntu 18.04 in the network trace:
failed.install.Ubuntu.18.04.zip


http2 support

http2 is out in node 10.0.0.
will http 2 be supported by the library?

@budarin You can use require('http2').createSecureServer(cert, new Koa().callback()).listen() to use http2 answered by fl0w in this issue

I am not about creating and starting the server - it is simple, I am about pushing resources, methods signatures and so

While
require('http2').createSecureServer(cert, new Koa().callback()).listen()

is really neat and simple, I believe that something along the lines of
new Koa({
    http2: {
        cert: ...,
        key: ...,
        enabled: true | false    // Optional for debugging
    }
})

could be very helpfull and really not a big addition in terms of codebase and complexity.
Any thoughts on the matter?

That proposal was turned down in #1063, I added a summary in this comment

if I have to use callback syntax - then why do I need koa ???

@budarin Because that's the design of the http library. Koa's essentially just a "sugarized" requestListener.

I am about somthing like a push method in express ...
Using http2 like a usual https -  what is a profit?

@budarin Nobody said not to expand Koa support for HTTP/2, let's wait and see what happens.

what year can we see it? :)

We think about migration from express to koa in webpack-dev-server, it is only one blocker for us, can't find documentation how koa works with http2 and examples

Hey @evilebottnawi, FYI I can get koa work with http2. This is the code that I am using
const http = require('http');
const http2 = require('http2');
const Koa = require('koa');
const app = new Koa();

// response
app.use(ctx => {
  ctx.body = 'Hello Koa';
});

http.createServer(app.callback()).listen(80);

http2.createSecureServer({
  "key": certificate,
  "cert": certificate
}, app.callback()).listen(443);

i've looked at this a long time ago, and adding push support was complicated and did not provide much benefit. koa would basically just set headers in the stream, but you'd still need to pipe data to the stream yourself.
this might be better as a framework using koa than supported in koa itself
let us know if you have any API recommendations


revert #8131

This reverts the optimisation wherein pages with ssr = false and no explicit prerender = false would automatically be prerendered, as it turns out to be unsafe.

closes #9275
closes #8428

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

This needs a changeset


Does not allow to push to repo with slashes

Trying to push change to repo name with slashes gives the following error:
error: src refspec admin_tools does not match any.
error: failed to push some refs to 'ssh://my_path_to_repo.git'
'git -c push.default=simple push fork admin_tools --porcelain' exited with error code 1
The repo is "fork/admin_tools/tweaks" and appears to be dropping the "/tweaks". I can push to another repo that does not contain a "/".

Looks like a first, I will put it on the list but can promise nothing at the moment. PRs are welcome!

Same as #139.
There's a fix suggestion there.


do you use sktime?

do you use sktime
https://github.com/alan-turing-institute/sktime

nope, haven't had a chance to look at it

So what is about full example from start to end for multivariate classification case?
When data had both
Categorical and continue s values

I don't have an example of this, and I'm not sure if this library supports such a case at this time.


autoRefreshToken is not exposed in web package

Hi,
although you mention it is not recommended, I need to disable automatic refreshing of tokens.
I think that the intent was for it work like this:
<KeycloakProvider keycloak={keycloak} autoRefreshToken={false}>, because I can see that there is autoRefreshToken prop in ProviderProps in the core package (with that comment about it not being recommended). However this props is missing in ProviderProps in the web package, so TypeScript throws an error. And I was not successful in importing the provider directly from core.
So, I wonder how am I suppose to set this prop? Is this a bug or am I missing some principle of setting props to the provider? :/
Thanks!

Hi @incik ,
Thanks for your feedback, I had a quick look and it seems that prop is just missing in the TS definitions of the web package but should work as expected.
I'll update them and publish an updated version as soon as possible.

Hi @incik ,
Newly released version 2.1.0 of @react-keycloak/web package should fix your issue.
Let me know if it works so that I can close this issue.

Hi, I was finally able to test it, and it works as advertised. Thank you! :-)


add repvgg and mlpmixer

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Hello, we add repvgg and mlpmixer to mmcls, however we feel so sorry that we don't have enough time and gpus to finish the test.
BC-breaking (Optional)
Does the modification introduce changes that break the backward compatibility of the downstream repositories?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here and update the documentation.
Checklist
Before PR:

[ ] Pre-commit or other linting tools are used to fix the potential lint issues.
[ ] Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

[ ] If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects, like MMDet or MMSeg.
[ ] CLA has been signed and all committers have signed the CLA in this PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello, thanks for your contribution!
We will review this PR, please split this PR into two PRs, RepVGG and Mixer. It will be helpful for us to discuss.


discord: 0.0.18 -> 0.0.19

Description of changes
Updates Discord to version 0.0.19 on x86_64 Linux. The discord client won't open without this update.
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.

I didn't notice the other pr for this, nothing to see here.


What is 'eps' in evalutation?

eps           = 1e-5

...

precision = 1.0*correct/(proposals+eps)
recall = 1.0*correct/(total+eps)
fscore = 2.0*precision*recall/(precision+recall+eps)

I'm working on understanding the evaluation system. And having problem in understanding eps...
Why do you add '1e-5' value. And named it eps??

@richard0326 eps stands for epsilon and has a role to avoid divide-by-zero exception.

thank you


Is it possible to use DinoV2 Large or Base Model

Is it possible to use DinoV2 Large or Base Model for inference if so what changes do we need to apply?
Training from scratch
OR
Changes in code will be enough to achieve this
Kindly help

You need to re-train the model, or at least fine-tune the projection linear layer, as the output channel of different versions of DINOv2 varies.


Support for new flow import type syntax

Flow can now import type interposed with regular js import like :
import {
  MyClass
  type MyType,
} from 'mymodule';

At the moment the parser seems to return parse error. It would be good to support this syntax

This is already supported as of babylon 6.15 and commit https://github.com/babel/babylon/commit/9ea4e9dead015ebf710b58fdefd2b3bc0e50c1ca
Make sure that all your dependencies are up2date.

Looks like your example is missing a comma?

The comma was missing in the example but not in real life ;-), I edited the original post. It seems the bug was caused by the use of two babylon versions differents in the project.
I just fixed with a  yarn upgrade .


Visualise ESR indicator scores by group

Job story
When I am viewing a single country (#1) ...
... I want to be able to explore the underlying indicator scores visually for each sub-group ...
... so I can better understand the implications for population groups.
Additional considerations

Currently only scores for all groups are presented (#4)
Possible group breakdowns are by sex, ethnicity, urban/rural status, income quantile
At this stage we will not have data disaggregated by multiple breakdowns
Sub-group scores could have different group-specific immediate targets
Consider mobile support #13
Consider accessibility #15
Consider general design considerations #14
Consider shareability #27
TBD

Tasks

Consider chart options
Consider chart interactions

go back to aggregated indicator score (#4)
drill down to explore underlying raw sub-group indicator data (#24)

Job story
When I am viewing a single country (#1) ...
... I want to be able to explore the underlying indicator scores visually for each sub-group ...
... so I can better understand the implications for population groups.
Additional considerations

Currently only scores for all groups are presented (#4)
Possible group breakdowns are by sex, ethnicity, urban/rural status, income quantile
At this stage we will not have data disaggregated by multiple breakdowns
Sub-group scores could have different group-specific immediate targets

Tasks

Consider chart options
Consider chart interactions

go back to aggregated indicator score (#4)
drill down to explore underlying raw sub-group indicator data (#24)

See also #24


added get started with UI

this is an _index file for query with SQL section

This index file is no longer needed. Closing this PR.


Adding files in FreeRTOS SVN but not in GitHub

Description
As subject. Source of truth: SVN 2720.
Once this PR is merged, to be checked in Jenkins that croutine.{c, h} are not causing issue.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

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

/bot run checks

/bot run checks

/bot run checks

/bot run checks


Dapr should require app-port to be set if a pub/sub component is loaded/used

In what area(s)?
Use of pub/sub components should be prevented if no app-port is set on dapr when it is started, which is needed for dapr to successfully call the subscription endpoints.
When debugging an issue on pub/sub the subscriber service for the topics did not have an **app-port set, so dapr was receiving the topic events successfully from the publisher but could not send these to the listening app. It was hard to find this issue since the components loaded successfully and there were no errors in the logs. There needs to be check and a error message written to find this condition and even fail dapr from successfully starting.

/area runtime
/area pubsub

What version of Dapr?
1.0.1
Expected Behavior
There needs to be check and a error message written to find this condition and even fail dapr from successfully starting.
Actual Behavior
No error and no failure on this condition.
Steps to Reproduce the Problem
Run the pub/sub sample and remove the --app-port CLI setting from the subscriber. Everything starts ok, but the sample does not work.
Release Note

RELEASE NOTE:
Require app-port to be set if a pub/sub component is loaded/used

There needs to be check and a error message written to find this condition and even fail dapr from successfully starting.

I think this would cause problems - the set of component defined is typically accessible to all daprized apps running the same domain. There no way to know which apps actually want to use pub/sub, and this would require all of the to define a listening point if any of them are using pubsub.

I think the spirit of this makes sense - making a connection between the definition of a listening port and the features that require one.
If there were log statements like:
app does not have a listening port, skipping pubsub
app does not have a listening port, skipping input bindings

That would go a long way.
Going even farther would be something like the health check or dashboard showing a detailed status of each component/building-block.

We cannot require an app to listen on a port to use pub/sub.
This is because publishers that send messages do not need to open a port or receive messages at all, and doing what is suggested here would break these scenarios.
I agree with Ryan's comment that more verbose logging in this case is the way to go.


data_path() twice

data_path() is redefined in Hernquist.jl (already defined in CB73.jl)
may cause trouble at prefactors filling.
If name change, to do in every kwargs

Good catch. Fixed with recent push; need to have a unique data_path() for any bases that use tables.


scheduler(cdc), tests: fix TestNoFinishOperationBeforeSyncIsReceived and TestClientSendAnomalies

What problem does this PR solve?

Issue Number: close #4975
What is changed and how it works?

Loosened TestClientSendAnomalies. We accept apparent successes of sending messages after the client is closed.
Replaced require.Never in TestNoFinishOperationBeforeSyncIsReceived, as require.Never will cause data races.

Check List
Tests

Unit test

Related changes

Need to cherry-pick to the release branch

Release note
None

/run-leak-tests

Codecov Report

Merging #5011 (1d25f4f) into master (687e248) will decrease coverage by 0.3267%.
The diff coverage is 52.0814%.

Flag
Coverage Δ

cdc
61.0903% <44.4565%> (+0.4916%)
:arrow_up:

dm
51.5564% <85.6459%> (-0.9131%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #5011        +/-   ##
================================================
- Coverage   56.1240%   55.7973%   -0.3268%
================================================
  Files           522        523         +1
  Lines         65325      66600      +1275
================================================
+ Hits          36663      37161       +498
- Misses        25094      25767       +673
- Partials       3568       3672       +104

/check-issue-triage-complete
/run-verify

/merge


[Laravel] Handle SetUpClassMethodNodeManipulator service not found

SetUpClassMethodNodeManipulator is removed at https://github.com/rectorphp/rector-phpunit/pull/90.
This PR handle it.

All checks have passed 🎉 @TomasVotruba it is ready for review.

Thank you :+1:


Is it possible to loop a video that is playing?

HI,
I am looking to repeat my video once it finishes. Is that possible?

I never found a working solution, but I did find that in order to generate a loop, it requires a new AVAsset every loop. My method never seemed to work unfortunately, but if anyone wants to see my work adding to Movie-input, let me know.

Yes, you must create a new AVAssetReader every loop.

Limon did you ever figure out how to loop on movie input? I am guessing I am going to need to look up AVPlayerItem and how to use that with AVAssets. If I figure it out I will happily add it to the code.

public func start() {
    self.assetReader = generateAssetReader(asset: asset, readAudio: readAudio, videoOutputSettings: videoOutputSettings, audioOutputSettings: audioOutputSettings)

.....
.....
.....

    if assetReader.status == .completed {
        assetReader.cancelReading()
        self.assetReader = nil
        if self.loop {
            self.start()
        } else {
            self.endProcessing()
        }
    }
}

So just to be clear you made a function called generateAssetReader somewhere in your code? Also on my screen I can not assign assign self.assetReader = nil. I think I am probably missing something vital.

I found a way of doing it. The only issue is it isn't as pretty as it is when I don't loop it or only have it loop once. Would it be OK if I emailed you the code and we could optimize it together @Limon-O-O and put in a pull request?

Did you guys found a solution to this? I'm in the same problem.

Yes! I made it! And it is smooth as a snowflake LOL.
So I made the famous generateAssetReader() @SoundandStuff  was asking for, this is my final code.
`public func createReader() -> AVAssetReader
{
var assetRead:AVAssetReader!
do{
assetRead = try AVAssetReader.init(asset: self.asset)
        let outputSettings:[String:AnyObject] = [(kCVPixelBufferPixelFormatTypeKey as String):NSNumber(value:Int32(kCVPixelFormatType_420YpCbCr8BiPlanarFullRange))]
        let readerVideoTrackOutput = AVAssetReaderTrackOutput(track:self.asset.tracks(withMediaType: AVMediaTypeVideo)[0], outputSettings:outputSettings)
        readerVideoTrackOutput.alwaysCopiesSampleData = false

        assetRead.add(readerVideoTrackOutput)
    }catch{

    }

    return assetRead
}

public func start() {
    self.assetReader = createReader()
    asset.loadValuesAsynchronously(forKeys:["tracks"], completionHandler:{
        DispatchQueue.global(priority:DispatchQueue.GlobalQueuePriority.default).async(execute: {
            guard (self.asset.statusOfValue(forKey: "tracks", error:nil) == .loaded) else { return }

            guard self.assetReader.startReading() else {
                print("Couldn't start reading")
                return
            }

            var readerVideoTrackOutput:AVAssetReaderOutput? = nil;

            for output in self.assetReader.outputs {
                if(output.mediaType == AVMediaTypeVideo) {
                    readerVideoTrackOutput = output;
                }
            }

            while (self.assetReader.status == .reading) {
                self.readNextVideoFrame(from:readerVideoTrackOutput!)
            }

            if (self.assetReader.status == .completed) {
                self.assetReader.cancelReading()

                if (self.loop) {
                    // TODO: Restart movie processing
                    self.start()
                } else {
                    self.endProcessing()
                }
            }
        })
    })
}`

Lol wow you solved the problem! You're solution runs much more smoothly than mine.

Your*


Add a web service to download content offline

Add a web service to download content offline
MSDN has a web service which allows downloading content offline, which can then be integrated into the Visual Studio Help Viewer through tools like Package This. This is very helpful if I want instant help on API which is not pre-installed with Visual Studio, such as ASP.NET MVC or Entity Framework, or any of the Azure documentation, or any API published after MSDocs was launched.
It would be great if such a web service will be available for docs.microsoft.com, so future documentation can also be integrated into Help Viewer the same way.
Issue Details
What is needed is a web service which will allow

browsing the content tree.
downloading content in a format which can be viewed in Help Viewer.

I know SOAP is out of fashion, and REST is all the rage now, but it may be easier to update existing applications that use the existing SOAP service from MSDN, if you just change the endpoint and maybe something else, and not change the application protocol completely.
Oh, and PDF is not it. I can't press F1 in Visual Studio into a PDF.

@dastaffo can you please follow-up on this item?

Adding @cillroy here.

Hello - reviewing and cleaning up some older feedback/issues. Thanks!


Question on paper result and contribution claim

Hi
In your work, you indicate"
In sum, we make three key claims: LIO-EKF
(i) is on par with the state-of-the-art LIO systems in terms of estimation accuracy,
(ii) can provide an accurate pose estimate in different environments and vehicle motion profiles,
(iii) can provide pose estimates at close-to-IMU frame rate.
 Why are you not comparing with KISS ICP  or CT ICP for the NEWER COLLEGE DATASET?
Since you submitted it to ICRA 2024, at least I would expect an open-source SOTA from ICRA 2023 comparison like DLIO, which happens to be also run on the Newer Colledge dataset.  Why was it not included? Why other people can run LIO SAM in the same dataset and you can not?   Why other people can get 0.3-0.4 for fast lio2 and you get 5-6m? Is the measurement have some trick? like using end to end point measurement and not ATE? Did you align the sequence?

Above is author claimed result

Above is the DLIO claimed result

Above is the LOG-LIO Claimed result.

This is result from Fast and Robust Lidar-Inertial Odometry by Tightly-Coupled Iterated Kalman Smoother and Robocentric Voxels
If all of them consistently getting better result than you for the baseline methods, that means something is wrong with your estimation

The main contribution of the paper is an implementation that is more accurate and fast, so I'd recommend you include more state of the art LIO methods: DLIO, SLICT, Traj-LIO, CLIC in your experiments. Besides, some LO methods such as KISS-ICP, CT-ICP can also be included to reinforce the advantage of LIO over LO.

Hey, thanks for your comments.

Yes. We have compared with KISS-ICP in the old version. Here is the table. We removed this because we think it is unfair since KISS-ICP and other LO systems don't have an IMU. In addition, the advantages of adding IMU into a LO are apparent. Therefore, we didn't report it.

For the metrics, we had the same findings. We used the metric implemented in KISS-ICP to get the numbers. We also tried evo, and the numbers were smaller for all systems. Here is an example.

We didn't pay too much for selecting the metric since we thought using the same one for every system was fair. In addition, it doesn't affect our claim that it achieves accuracy similar to the others.

We failed to run LIO-SAM with the newerCollege dataset because it requires attitude information from the IMU, which is not the case in the dataset. Therefore, we didn't get the results. We have explained this in the paper.

This work wants to show that with a simple design, LIO-EKF can achieve higher efficiency while not losing much accuracy. We focused more on computation efficiency instead of challenging all the SOTA in terms of accuracy. Therefore, we selected the two most representative LIO systems. For example, in the DLIO tables you showed, it was also only compared with these two LIO systems.

Hope this helps.

The difference between the two evaluation metrics is due to the additional multiplication of the rotation error to the reference position.
As shown in Equation (23) of the paper A Tutorial on Quantitative Trajectory Evaluation for Visual(-Inertial) Odometry:

As also mentioned in this issue: https://github.com/uzh-rpg/rpg_trajectory_evaluation/issues/58
Evo and other evaluation libraries use the one without the multiplication of the rotation error while kiss-icp and lio-ekf uses the one with the multiplication of the rotation error.

Thanks for the prompt response.
OK, I think I get the difference. EVO shows better results by giving additional scale adjustment  EVO whereas KISSICP  does not
For speed wise, i do concur with you

Since for EVO, it has similar result with others. then i guess that clears the cloud.
Then, I guess it did similar contribution as FLOAM.


Add support of disabled options

{ name: 'Alexandria' },
{ name: 'Alexandria' },
{ name: 'Alhambra', disabled: true },
{ name: 'Aliso Viejo' },
{ name: 'Allen' },
{ name: 'Allentown' },

Thanks for the feature, @denisborovikov. Looks great.

1.3.0 just released with this feature :)

Thank you for so quick response!


Searching Deeper Nesting

In the steps outlined here, https://github.com/lord/slate/wiki/Deeper-Nesting, I believe it is missing a couple steps in regards to enabling searching of these deeper nested elements.
The missing steps include:

./source/javascripts/app/_search.js:23 Add 'h3' to the jQuery selector
./source/javascripts/app/_search.js:25 Add 'h3' to the jQuery selector

Hey! As mentioned elsewhere, I unfortunately don't have that much time to improve documentation, but if you've already done this recently, you should have all the permissions necessary to edit it!

added it to the docs: https://github.com/lord/slate/wiki/Deeper-Nesting

Sweet, thanks so much!


毕业指南原文疑似错误

在翻译毕业指南时，发现原文的一个疑似错误。
原文链接： http://incubator.apache.org/guides/graduation.html#whether_to_graduate_to_subproject_or_to_top_level_project
第二段：
Graduation as a subproject is only possible if the subproject still falls within the scope of the project and requires the consent of the project PMC. Graduation as a project requires the formation of the new project by the Board.
此处如果将Graduation as a project改为Graduation as a top level project则符合上下文。

我再JIRA上创建了一个issue

已经合并


Don't merge borders if any of the values is 'initial'

E.g. border: initial initial initial; would not be valid CSS.

Thanks! 👍
By the way, the tests are failing because appveyor is not configured on master yet.


Swift sample application fails with incorrect details

I've modified the API key within AppDelegate.swift class SumUpSDK.setup(withAPIKey: "affiliated_key") however trying to login with the same credentials which works for the official app returns a "server error incorrect password/email combination" error message.
Am I missing a step?

For those facing the same issue: Make sure that you update the API key located within SumUpSDKSampleApp -> SUSAppDelegate.m file as well as modifying the Swift directory.


[HUDI-1427] Throw a FileAlreadyExistsException when set HOODIE_AUTO_C…

…OMMIT_PROP to true
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
Fix FileAlreadyExistsException when set HOODIE_AUTO_COMMIT_PROP to true
Brief change log
I just remove the HOODIE_AUTO_COMMIT_PROP from the parameters in HoodieSparkSqlWriter because it will do the commit in HoodieSparkSqlWriter#commitAndPerformPostOperations, which will repeat to that in BaseSparkCommitActionExecutor#commit when HOODIE_AUTO_COMMIT_PROP set to true.
Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #2295 (c806a3e) into master (62b392b) will decrease coverage by 43.14%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #2295       +/-   ##
=============================================
- Coverage     53.50%   10.35%   -43.15%
+ Complexity     2788       48     -2740
=============================================
  Files           355       51      -304
  Lines         16169     1786    -14383
  Branches       1650      213     -1437
=============================================
- Hits           8651      185     -8466
+ Misses         6818     1588     -5230
+ Partials        700       13      -687

Flag
Coverage Δ
Complexity Δ

hudicli
?
?

hudiclient
?
?

hudicommon
?
?

hudihadoopmr
?
?

hudispark
?
?

huditimelineservice
?
?

hudiutilities
10.35% <ø> (-59.75%)
0.00 <ø> (ø)

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


Chronograf: Fix spelling of --htpasswd command line option

Chronograf documentation provides two ways to write command line option --htpasswd wrong. Using the provided example opens a security hole by not enabling any authentication upon startup. Command line option --htpasswd is written first as -htpasswd and then as —-htpasswd.
Relevant URLs

https://docs.influxdata.com/chronograf/v1.8/administration/managing-security/#configure-chronograf-to-authenticate-with-a-username-and-password

Thanks, @janilahti! Fixing this now.


Enable TCP states and ThreadPool stats

This commit adds the ability to enable TCP and ThreadPool
statistics to be collected when making a request, exposing
these statistics on the IApiCallDetails on the response, as
well as including in the debug information.
Add documentation explaining how to include these statistics
either on a global or per request basis.

Thoughts on including this in 7.9.1, or 7.10.0, @Mpdreamz ?

I wonder if it might be good to create include these types under https://github.com/elastic/elasticsearch-net/tree/master/src/Elasticsearch.Net/Diagnostics and in the Diagnostics namespace?

I was initially thinking of putting them there, then wondered if that should strictly be for System.Diagnostics related types. I'll move them there 👍

Failures are related to transforms; opened https://github.com/elastic/elasticsearch-net/issues/5004 to address


Use Promise.allSettled when loading timeline resources

Coverage: 93.701% (+0.0004%) from 93.7% when pulling 18c814b3a65fcebff404ef33799d094a97bbf73f on cody-resource-timeline-access-denied into 5a5a27d59d32e71eedded3a96d78fda9c1f6b474 on main.


Where can I find the packages required for executing the code?

Hello, sir!
When i reading this book and want to execute these codes, i can not find code python package version requirement.
It will generate so many error!
So, Please provide the version of the Python package that executes the relevant code!
Thank you very much!

Hello,
Thank you very much for your message.
Here is what I suggest :

Go to the README page which contains the list of notebooks.

Open any notebook with the Google Colab icon. You need a free Google account.

3.Go to the top of the notebook.

Add a new code  cell.

5.Finally, copy and run the following code:
!python --version
Voilà ! The Python version used is displayed and you have the information!
Please keep me informed. 😊
Denis

Hello, sir! Thank you very much for your patient reply and detailed guidance.
And, I Following your guidance, I operated step by step and the code could run correctly in Google Colab. succès!!
But, I want to run the code in my location computer.
!python --version could only show the python version information，It does not include the versions of other packages such as numpy, transformer, and pytorch！
You know, if there are no versions of these packages, it's very easy to make mistakes during runtime.
So， if it's not too much trouble, could you please provide the version information of the relevant packages so that I can run them on my local computer.
Merci beaucoup!
在 2024-09-19 01:10:41，"Denis Rothman" @.***> 写道：
Hello,
Thank you very much for your message.
Here is what I suggest :
Go to the README page which contains the list of notebooks.
Open any notebook with the Google Colab icon. You need a free Google account.
3.Go to the top of the notebook.
Add a new code cell.
5.Finally, copy and run the following code:
!python --version
Voilà ! The Python version used is displayed and you have the information!
Please keep me informed.
Denis
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hello,
To obtain the versions of the packages installed during the installation of
each notebook in the repository:
a) open a notebook in Google Colab free version with a free Google Account
b) run each cell that contains  a pip install instruction
c) the output shows the list of the packages installed with their version
numbers
Voilà!
Denis
On Thu, Sep 19, 2024 at 3:46 AM XiaoluJiayou @.***>
wrote:

Hello, sir! Thank you very much for your patient reply and detailed
guidance.
And, I Following your guidance, I operated step by step and the code could
run correctly in Google Colab. succès!!
But, I want to run the code in my location computer.
!python --version could only show the python version information，It does
not include the versions of other packages such as numpy, transformer, and
pytorch！
You know, if there are no versions of these packages, it's very easy to
make mistakes during runtime.
So， if it's not too much trouble, could you please provide the version
information of the relevant packages so that I can run them on my local
computer.
Merci beaucoup!
在 2024-09-19 01:10:41，"Denis Rothman" @.***> 写道：
Hello,
Thank you very much for your message.
Here is what I suggest :
Go to the README page which contains the list of notebooks.
Open any notebook with the Google Colab icon. You need a free Google
account.
3.Go to the top of the notebook.
Add a new code cell.
5.Finally, copy and run the following code:
!python --version
Voilà ! The Python version used is displayed and you have the information!
Please keep me informed.
Denis
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID:
@.***>
—
Reply to this email directly, view it on GitHub
https://github.com/Denis2054/Transformers-for-NLP-2nd-Edition/issues/11#issuecomment-2359802108,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHLCIVT5XQ33PIB75SW7ZDDZXIUIBAVCNFSM6AAAAABOM7IKL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJZHAYDEMJQHA
.
You are receiving this because you commented.Message ID:
@.***
com>

Hello，sir!
Thank you for your reply, the problem has been solved.
This is the best book I have read recently about transformer and NLP practice.
Thank you for writing such a book.
I have benefited a lot from it.
merci beaucoup！
At 2024-09-19 15:59:31, "Denis Rothman" @.***> wrote:
Hello,
To obtain the versions of the packages installed during the installation of
each notebook in the repository:
a) open a notebook in Google Colab free version with a free Google Account
b) run each cell that contains a pip install instruction
c) the output shows the list of the packages installed with their version
numbers
Voilà!
Denis
On Thu, Sep 19, 2024 at 3:46 AM XiaoluJiayou @.***>
wrote:

Hello, sir! Thank you very much for your patient reply and detailed
guidance.
And, I Following your guidance, I operated step by step and the code could
run correctly in Google Colab. succès!!
But, I want to run the code in my location computer.
!python --version could only show the python version information，It does
not include the versions of other packages such as numpy, transformer, and
pytorch！
You know, if there are no versions of these packages, it's very easy to
make mistakes during runtime.
So， if it's not too much trouble, could you please provide the version
information of the relevant packages so that I can run them on my local
computer.
Merci beaucoup!
在 2024-09-19 01:10:41，"Denis Rothman" @.***> 写道：
Hello,
Thank you very much for your message.
Here is what I suggest :
Go to the README page which contains the list of notebooks.
Open any notebook with the Google Colab icon. You need a free Google
account.
3.Go to the top of the notebook.
Add a new code cell.
5.Finally, copy and run the following code:
!python --version
Voilà ! The Python version used is displayed and you have the information!
Please keep me informed.
Denis
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID:
@.***>
—
Reply to this email directly, view it on GitHub
https://github.com/Denis2054/Transformers-for-NLP-2nd-Edition/issues/11#issuecomment-2359802108,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHLCIVT5XQ33PIB75SW7ZDDZXIUIBAVCNFSM6AAAAABOM7IKL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJZHAYDEMJQHA
.
You are receiving this because you commented.Message ID:
@.***
com>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Path Bot is running out of Pokeballs

Expected Behavior
Running on path an spinning all nearby forts.
Actual Behavior
If the bot is running on path mode he isn't spinning all nearby forts and running so out of pokeballs.
If I set move_to_spin=true he actually want follow the path anymore.
Steps to Reproduce
Run on
"navigator": {
"type": "path",
"path_mode": "loop",
"path_file": "configs/path.json"
},
"forts": {
"spin": true,
"move_to_spin": false,
"avoid_circles": true,
"max_circle_size": 1
},

Reason it is invalid is this is how it should be.


Bump git2 dep to fix SSL paths

This commit includes alexcrichton/git2-rs@a8f4a7faa which switches the order of
initialization of libgit2. That commit ensures that the relevant env vars which
a statically linked OpenSSL needs to function are set before libgit2 is
initialized to ensure that libgit2 uses them.
This was regressed accidentally in alexcrichton/git2-rs@071902aa when
initialization was tweaked.
Closes #3340

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 52fa741 has been approved by alexcrichton

:hourglass: Testing commit 52fa741 with merge 930bdaa...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing 930bdaab2ec7517c16edc82d77f8553f74803f1c to master...


Investigate container resizing strategy for diverse data sets

For feature requests, please provide the following:
Description
This is at the top of roaring.go:
// manual allocation size tuned to our average client data
manualAlloc = 524288

This is a parameter for linear array resizing behavior, tuned to a specific expected index width. We should be aware of its impact on other data sets.
Success criteria (What criteria will consider this ticket closeable?)
Support configuring this parameter, or other resizing options (exponential?); or explicitly accept this parameter as sufficient for general use.

By default, Go slices grow by doubling until size 1024, after which they grow by 25% each time.  The manual  Go slice sizing scheme implemented currently was based on the average audience size of the original use case of pilosa in an attempt to decrease the number of times a costly  Go slice reallocation occurs. It should be noted that this is resizing of the go slice that maintains a collection of containers and not a specific container itself. This was an oversight on my part, and now that I have a better understanding of the roaring container's purpose, I believe we could achieve a more efficient method with a small initial capacity on bitmap creation.

Sounds like someone is going to need to dive into this and investigate - let's move to backlog, and we can open up to wider discussion of tradeoffs when there is a better understanding of what's going on, and someone has enough understanding to make a recommendation.

It appears that this was removed in d7272f46a231d289a81b006b1a21e58e0ce875c4 and 28d590c5751f2342ae5906d947a7aca3f650e473.
Is there any reason to keep this open @alanbernstein?

this is probably obsolete as of the btree work anyway

It's not clear to me why this was removed in d7272f4, but since it's gone we can close this. Agreed this is probably obsolete with btree, but if that ends up being an enterprise feature then we may need to revisit this.


Test conneg process to get direct access to some representations

If a /id/ is request in a mime-type that is not one of the 4 mime type supported (or not an hypermedia), and if there is only 1 representation in that format, the conneg can 303 directly to that resource.
eg: /id/ resource is available in .pdf and only one representation if pdf is available, the conneg process will 303 directly to this .pdf.  Otherwise, it will return 404, UNLESS the accept header also has one of the aforementioned hypermedia in the preferred list.

does not work since we don't know what format the mir will be if request format in ambiguous


corrmap titles are bad again

I don't know why yet, but something changed that results in corrmap again having bad axis titles.
I'll eventually take care of it.

Closing because the fig and subplot titles look fine to me here (and there are no axis titles): https://mne.tools/dev/auto_tutorials/preprocessing/plot_40_artifact_correction_ica.html#selecting-ica-components-using-template-matching
@jona-sassenhagen feel free to re-open if you disagree, and explain how you think they should look.


Workaround for iproute v5.1 issue with creating veth pairs

As described in issue #884 iproute2 introduced a  bug in v5.1 that throws an error message while creating veth pairs even when the operation was successful. I can confirm this behavior on openSUSE Tumbleweed.
As this issues breaks the core functionality of mininet, I propose the workaround seen in this PR.

Hmm, that is irritating. I don't think that a blanket check ignoring error messages is a good idea though.

If we want a workaround for the iproute2 bug, this is probably the best way to have it. Otherwise, mininet won't work at all and what makes it worse, it won't even tell you why! This modification silently ignores the faulty error message, but at least it gives a notification about this when setting the log level to "debug".

My preference would to check for the error and report it. Is it possible to do that?

surely this was a transient issue that has been fixed on any proper machine now ? not sure that deserves a mininet specialcase.

I can confirm that the issue was solved in recent versions of iproute2 (just tested it with v5.2-1.1 on openSUSE Tumbleweed) and therefore close this pull request.


[FR] API for python scripts

It would be cool to have a description/implementation to use your project from other scripts.

ugly and hacky work around
import sys
import tempfile
from termgraph.termgraph import main

def plot(data):
    with tempfile.NamedTemporaryFile(mode='a+') as f:
        # Save data in temporary file
        for row in data:
            f.write('\t'.join(map(str, row)) + '\n')

        # Move cursor in order to make sure that script will
        # start reading file from the beggining.
        f.seek(0)

        # Overwrite args in case if there were some other
        # arguments passed to the main script
        #
        # Additional arguments can be passed in the same way.
        original_argv = sys.argv
        sys.argv = [sys.argv[0], f.name]
        main()

        # Revert back changes to the original arguemnts
        sys.argv = original_argv

plot([['a', 3], ['b', 8], ['c', 6]])

a: ▇▇▇ 3.00
b: ▇▇▇▇▇▇▇▇ 8.00
c: ▇▇▇▇▇▇ 6.00

how can i make a  Multi-variable ?

can you elaborate more @songmuyi?

this is snippet for example 1
>>> labels = ['2007', '2008', '2009', '2010', '2011', '2012', '2014']
>>> data = [[183.32], [231.23], [16.43], [50.21], [508.97], [212.05], [1.0]]
>>> args = {
>>>     'stacked': False, 'width': 50, 'no_labels': False, 'format': '{:<5.2f}',
>>>     'suffix': '', "vertical": False
>>> }
>>> from termgraph.termgraph import chart
>>> chart(colors=[], data=data, args=args, labels=labels)

Needed a modification from code above, added few param, got exceptions without them.
Here is a working code:
labels = ["2007", "2008", "2009", "2010", "2011", "2012", "2013"]
data = [[183.32], [231.23], [16.43], [50.21], [508.97], [212.05], [1.0]]
args = {
    "stacked": False,
    "width": 50,
    "no_labels": False,
    "format": "{:<5.2f}",
    "suffix": "",
    "vertical": False,
    "histogram": False,
    "no_values": False,
}
from termgraph.termgraph import chart

chart(colors=[], data=data, args=args, labels=labels)

Here is a function, FWIW:
def tprint(
    labels,
    values,
    args={
        "stacked": False,
        "width": 50,
        "no_labels": False,
        "format": "{:<5.2f}",
        "suffix": "",
        "vertical": False,
        "histogram": False,
        "no_values": False,
    },
):
    from termgraph.termgraph import chart

    data = [[x] for x in values]
    chart(colors=[], data=data, args=args, labels=labels)

labels = ["2007", "2008", "2009", "2010", "2011", "2012", "2013"]
values = [183.32, 231.23, 16.43, 50.21, 508.97, 212.05, 1.0]
tprint(labels, values)

How can I use this for 2 Categories?

How to define a color in this case.


Inability to subscribe metdata-change, device-added and device-removed events

I am trying to subscribe to these events (in the title) like following:
events, err := l.SubscribeEvents(ctx, libvirt.DomainEventIDDeviceAdded, []libvirt.Domain{})
if err != nil {
	log.Fatal(err)
}

and then reading from that channel, but there no event comes no matter what I do..
I can see all these in virsh event --all --loop:
event 'lifecycle' for domain instance-000005ad: Defined Added
event 'lifecycle' for domain instance-000005ad: Started Booted
event 'lifecycle' for domain instance-000005ad: Suspended Paused
event 'lifecycle' for domain instance-000005ad: Resumed Unpaused
event 'device-added' for domain instance-000005ad: net1
event 'metdata-change' for domain instance-000005ad: element http://openstack.org/xmlns/libvirt/nova/1.1

but only lifecycle events work fine and can be subscribe/read from:
events, err := l.SubscribeEvents(ctx, libvirt.DomainEventIDLifecycle, []libvirt.Domain{})
if err != nil {
	log.Fatal(err)
}

any chance I am just missing something? Btw trying to use this on a openstack host with libvirt 5:
# virsh version
Compiled against library: libvirt 5.0.0
Using library: libvirt 5.0.0
Using API: QEMU 5.0.0
Running hypervisor: QEMU 2.11.1

Does it still behave this way if you populate the third argument slice with Domains to subscribe to?
events, err := l.SubscribeEvents(ctx, libvirt.DomainEventIDDeviceAdded, []libvirt.Domain{aDomain, anotherDomain})
if err != nil {
	log.Fatal(err)
}

fyi with a different library github.com/libvirt/libvirt-go-module it works just fine :/
with that I am subscribing to the events like this:
_, err := conn.DomainEventMetadataChangeRegister(nil, p.cbMetadadaChanged)

Thanks, that's good information.
Would you be willing to share a stack trace of your reproducer program?
Just let it run, find its PID, and do a kill -s SIGABRT <PID> and paste it here or link to a pastebin somewhere?


Test failure: Interactive Window Debugging @debugger

https://github.com/microsoft/vscode-jupyter/actions/runs/3446206002/jobs/5750829824
https://github.com/microsoft/vscode-jupyter/actions/runs/3441902799/jobs/5741935471
  1) Interactive Window Debugging @debugger
       Debugging with VSCodePythonDebugger @debugger
         Debug a cell from a python file @mandatory:

      Error: Cell 2 did not complete successfully, State = 1
      + expected - actual

      at Timeout.<anonymous> (src/test/common.ts:65:20)
      at listOnTimeout (node:internal/timers:559:17)
      at processTimers (node:internal/timers:502:7)

https://github.com/microsoft/vscode-jupyter/actions/runs/3441902799/jobs/5741935072
is similar
  1) Interactive window execution @iw
       Execute cell from Python file @mandatory:

      Error: Cell 2 did not complete successfully, State = 1
      + expected - actual

      at Timeout.<anonymous> (src/test/common.ts:65:20)
      at listOnTimeout (node:internal/timers:559:17)
      at processTimers (node:internal/timers:502:7)

https://github.com/microsoft/vscode-jupyter/issues/11917


Feature Request: Built-in placeholder to label functionality

I was wondering if placeholder to label functionality is already a built-in feature? If it isn't, is there a way I could help implement it?

Hello !
Nah, we force the label to shrink because it's not the same animation because of the flag position.


Would it be possible to set camera parameters?

I'd like to set the camera location, depth of view, etc. via code, but I cannot find any way to do this. Is it possible to add this as a feature? Is there a workaround I could use in the meantime?
Thanks, and thanks for all the great work!

I guess we only have those properties so far.
But the list of camera parameters could be extended here and have dash-vtk expose the latest library with any add-on parameters.

Is there a way to do that via js so I can get something working until a release of dash-vtk that makes it possible in python? (I'm a novice with react and js so sorry if that's a silly question.)

I don't think that you have much control on the JS side from Dash but I might be wrong.
If you can make a JS call on the react VtkView component, you can get your hand on the camera by calling componentRef.renderer.getActiveCamera().set({ ... })

Thanks. I've no idea how to get that view component, yet, but I'll look into it.

Even if I could find a way to make a JS call on the React-vtk-js View component, I don't think it'd work. I realise now that the resetCamera method is called here, which moves the camera position anyway.
So, I think I need to make a custom React class, based on the View component of React-vtk-js, and turn it into a Dash component using the dash-component-boilerplate. Then I think I can get the functionality I need. And, I think I can use clientside callbacks to make the camera updates fast (e.g. for animation). Hopefully that makes sense and will work. And hopefully it's not too hard to do!
Thanks again for your comments. I'll close this issue as it doesn't seem like it's a dash-vtk issue to me now.

If your code in JS, you should import react-vtk-js directly. dash-vtk is just exposing for Python/R/Julia react-vtk-js.

Ah, so I just make react-vtk-js a dependency in the package.json and do something like `import VtkView from 'react-vtk-js' in the JS file? And it won't matter that my component doesn't know about dash_vtk?

yes exactly. You can double check how dash-vtk do its import as you are basically doing the same thing yourself.

Great, thanks again!


Physical Properties: Magnetic Permeability References

References in the magnetic permeability section need to be added to the references.bib and cited using :cite:. For an example, take a look at the Mt. Isa Setup

Fixed links, references etc...


Home button of Footer not working

Description
on event tab by clicking on this home button it opens 404 not found page, could you assign this to me under gssoc'23

Yes @trisharaj06 , pls go ahead and fix this!


VR V3 don't deploy.

Visual Recognition don't deploy on my Bluemix. Generates error on the final of the process when was trying to upload something.

What errors is it generating?  It would be best if you could paste in the commands you ran and the output it generated.
Thanks.

I will generate the error again and send you. Thanks.

Follows attached complete log of the deploy.
log_bluemix.txt

Then Pittsburgh... Some idea?

I don't have any idea yet.  I can't deploy it as a normal user either.  @germanattanasio
I'm still investigating.

Did you try doing cf logs visual-recognition-nodejs-visrecmor-1848 --recent afterwards? Any useful info there?

I will check.

I just ran that,  it looks like it's a problem with babelify
Error: Cannot find module 'babelify' from '/home/vcap/app'

I think I've got it sorted.  PR in process.

Ok,  I know what the problem is but the easy fix isn't going to work.
Here is the workaround:
checkout commit 476bce622b2e204ec20d5967b5e57613dda15367 and
use that commit.
$ git checkout 476bce622b2e204ec20d5967b5e57613dda15367
HEAD is now at 476bce6... Updates the README to describe common uses
$ cf push

and that will get you running with the demo for now.  I'm going to fix the other issue, which is a problem with too much disk space being used by some of the node modules.
Thank you.

 It's working perfectly.


index.html is not redirecting to route

Hello,
My site https://shamdasani.org (github) is not redirecting the blog post urls on refresh.
here is an url that goes back to the homepage, but does not redirect: https://shamdasani.org/hello-world
all there is a ?p=/hello-world in the domain
Would appreciate any help. Thanks!

You have to add this script to index.html, https://github.com/rafrex/spa-github-pages/blob/gh-pages/index.html#L58


Add Quill example page to https://yorkie.dev/examples

Description:
Add Quill example to https://yorkie.dev/examples
We have a Quill example, but we haven't created a page on the homepage.
Let's create a page that displays the example.

CodeMirror Page: https://github.com/yorkie-team/yorkie-team.github.io/blob/main/pages/examples/codemirror.tsx
Quill Example in JS SDK: https://github.com/yorkie-team/yorkie-js-sdk/tree/main/examples/vanilla-quill

Why:

Can I try this issue? 😊

@g2hhh2ee Sure. :) If you have any questions, please feel free to ask.

@hackerwins
Hi,
I was working on this issue and realized that I don't have a thumbnail image file for the quill example. Is there an image file for the quill example?

We can capture thumbnail images from the site below.

https://yorkie.dev/yorkie-js-sdk/examples/vanilla-quill/

This is fixed by https://github.com/yorkie-team/yorkie-team.github.io/pull/89


Numbro cannot find language files for a node app launched with 'forever'

I'm using forever (https://github.com/foreverjs/forever) to launch a nodejs application.
When launched with forever I get this message when settting the culture : "Error: Unknown culture : fr-FR"
It's because the process.title value is not 'node', but this : '/home/user/.nvm/versions/node/v4.1.2/bin/node'
So the function inNodejsRuntime() returns false. And the languages are thus not loaded.
I guess there is a more reliable method to detect a nodejs environement ? The process.title === 'node' is not correct in this particular case. Checking process.versions sounds like a better idea.
https://nodejs.org/api/process.html#process_process_versions

Sorry wrong issue tracker! :)


fixing react router dom version and switching to absolute routing whe…

Fixes #3830
This PR fixes the following:

Nails down the react-dom-router dependency to version 6.21.1.
Updates the routing path to be more absolute when creating an airlock request.

Thanks @yahya130
@tamirkamara any idea why flag_external_pr is failing .
Could it be:

This org has transitioned to a read-only GITHUB_TOKEN for GitHub Action workflows. This does break your builds. See https://aka.ms/github-token-perms-changes for how to resolve.

Thanks @yahya130
@tamirkamara any idea why flag_external_pr is failing .
Could it be:

This org has transitioned to a read-only GITHUB_TOKEN for GitHub Action workflows. This does break your builds. See https://aka.ms/github-token-perms-changes for how to resolve.

Actually can see the GitHub Scripts action uses GITHUB_TOKEN.  Will have to look to het the workflows fixed.

@marrobi it is this. I'm adding a permissions block in a pr.

any idea why flag_external_pr is failing .
Could it be:

This org has transitioned to a read-only GITHUB_TOKEN for GitHub Action workflows. This does break your builds. See https://aka.ms/github-token-perms-changes for how to resolve.

Actually can see the GitHub Scripts action uses GITHUB_TOKEN. Will have to look to het the workflows fixed.

Random (ish) stranger here! 👋
Just a suggestion - it's normally good practice to add the update the updated package-lock.json whenever the package.json is updated.
And actually changes to dependency versions in the package.json might not even take place until the lockfile is updated too

Random (ish) stranger here! 👋 Just a suggestion - it's normally good practice to add the update the updated package-lock.json whenever the package.json is updated. And actually changes to dependency versions in the package.json might not even take place until the lockfile is updated too

Hi @benc-uk  👋, not random-ish at all 😄 great suggestion and well noted, just to add more context, the app uses yarn as the package manager and the package.json file has been updated to reflect what it is in the yarn.lock file.

@yahya130 @benc-uk should there be an updated yarn.lock file then? There is one in the repo, but not this PR.

@yahya130 @benc-uk should there be an updated yarn.lock file then? There is one in the repo, but not this PR.

It looks ok:
https://github.com/microsoft/AzureTRE/blob/31960894e15266f6ae76a4b7c1028cf9cc6bc459/ui/app/yarn.lock#L9395-L9408
@benc-uk what would be your opinion on this?

Totally missed you're using yarn
Yep that looks good!

/test

@tamirkamara looks like the bot is fixed. Thanks.
@yahya130 As this is UI and doesn't have tests, but will let it run through so we can ensure our builds are fixed, then merge. Thanks!

/test b000986

/test 7e1506f

/test caea73e

/test 7660005

Looks like our tests are blocked by - https://github.com/getporter/porter/issues/2987
Will force approval given this only touches UI. But the bot seems to be fixed.

/test-force-approve 7660005

@tamirkamara seems still an issue, tests aren't being marked as complete. Looks like granted the right permissions here - https://github.com/microsoft/AzureTRE/blob/3835e5ad89209d55c131eaa8e6b4211fbcc6c9ff/.github/workflows/pr_comment_bot.yml#L21 , can't see any errors either.
Any ideas?

/test-force-approve 7660005


Bug located in ...

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug

Trying to create a plane, it seems that everything work properly but the definition of point vector.
📝 Steps to reproduce

Launch Case Session
Load .cas and .dat
Execute root command

💻 Which operating system are you using?
 Tried setting it to values via API that can be set by other means in the UI but via API it always stays at the old value.

bug 726205 in Fluent

Fixed in Fluent 2023R2, not yet released.

@prmukherj if you have a bit of spare time, please could you check this defect?

@cj-hodgson perhaps one for JM to check?

@millerj97 Please could you check this.

@cj-hodgson tested in pyfluent and pyconsole, confirmed fixed
Steps:

launch pyfluent and load case/data
create plane using below commands

tester = solver.results.surfaces.plane_surface.create(name = 'test_plane')
tester.method = 'point-and-normal'
tester.print_state()

Defaults to;
point_vector 0.0, 0.0, 0.0
point_normal 1, 0, 0

Modify point values

tester.point_vector = (2.12, -0.377, 0.23)
tester.point_normal = (0, 1.34, 0)
tester.print_state()

Modify point_vector and point_normal to multiple values and print state

tester.point_vector = (2.8, 0.1, 0)

Load case/data in pyconsole and create plane in point-normal mode with UI
Edit the created plane using pyconsole commands

tester.point_vector = [2.8, 0.1, 0]

Point_vector settings confirmed to update as required
Repeat above steps creating point-normal plane and editing, using console commands


[WEB] Text colour in a Post

The text colour in an input text needs to be in sync with the design.

However, the current implementation uses some other colour. Please see the screenshot below
Sample text: Lorem Ipsum.

This change needs to be for both the tabs, "Offering Help" and "Requesting Help"
Please reach out to me in case of questions.

@snowiesuet @anniegiang I think the design we have here is outdated right? If it is, there would be no point in me changing the text color for an old design, we should just fix that when we redesign it
Just want to make sure, I'm not certain I found the right design for this

@Yu-Alexa-Zhao can you confirm this please?

I just corrected the design bugs in the above mobile design. The text color for input text should be #282828, when there's no input text, the color is #939393.

Yea I actually noticed the text color differences from the PR related this but it was taking a while fix revisions so I decided to merge it anyways to speed up progress


found 1 low severity vulnerability   run npm audit fix to fix them, or npm audit for details. I dont know why this occurs everytime.

PS C:\Users\ACER\Desktop\MyVegoAdmin> npx create-react-app vego-admin
Creating a new React app in C:\Users\ACER\Desktop\MyVegoAdmin\vego-admin.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts with cra-template...

<EMAIL_ADDRESS>postinstall C:\Users\ACER\Desktop\MyVegoAdmin\vego-admin\node_modules\babel-runtime\node_modules\core-js
node -e "try{require('./postinstall')}catch(e){}"

<EMAIL_ADDRESS>postinstall C:\Users\ACER\Desktop\MyVegoAdmin\vego-admin\node_modules\core-js
node -e "try{require('./postinstall')}catch(e){}"

<EMAIL_ADDRESS>postinstall C:\Users\ACER\Desktop\MyVegoAdmin\vego-admin\node_modules\core-js-pure
node -e "try{require('./postinstall')}catch(e){}"

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>added 1626 packages from 751 contributors and audited 1630 packages in 422.406s

59 packages are looking for funding
run npm fund for details
found 1 low severity vulnerability
run npm audit fix to fix them, or npm audit for details

I have same problem.......how to fix it

That's not an error, but is core functionality of npm.  The message occurs every time because the vulnerability hasn't been fixed.  The message tells you to run npm audit fix to try and fix the vulnerability.  Sometimes npm is not able to fix it since the problem lies in a dependency.
In short, these messages are warnings and not errors.  They won't affect the results of your build.


I think PPO deterministic is implemented incorrect

https://github.com/google/brax/blob/ff3ff641097699703087e1dc0a7b6e8305d78270/brax/training/agents/ppo/networks.py#L46
I noticed something weird when comparing the mean of many sampled actions vs the deterministic action. I would expect from a distribution that the deterministic value would be almost equal to the mean of samples.
I found that the action created at line 46 should also be postprocessed if I am not mistaken.
However this sadly still doesn't give the expected result. I created a test plot to show the difference:
Tanh(x) is the post processing step which still doesn't end up being the same as the mean.

So I think something is incorrect but I haven't found the fix yet.

Nevermind, it only had to something with my policies model.
The code in brax is correct, I noticed that the postprocessing is done in the method that outputs the deterministic action.


Button doesn't stay on top of Modal component

Hello, thanks for the great component.
I wish to create an action button that stays on the screen even after I render Modal component from RN, but zIndex doesn't work here.
Would that be possible?

hey @avishayil,
this should most likely be possible. just make sure RNAB is higher in your component hierarchy than the modal component. didn't test it, though


feat: enabling eth_signTypedData_v4

As part of the Matic integration, we need to allow eth_signTypedData_v4 messages. This is needed to bridge between the Eth and Matic chains.
@HPrivakos has a library in place that exposes this integration, but in order for it to work, we need to allow eth_signTypedData_v4 messages.
This is how it is used from the said library:
requestManager.provider.sendAsync(
      {
        method: "eth_signTypedData_v4",
        params: [fromAddress, dataToSign],
        jsonrpc: "2.0",
        id: 1,
      } as RPCSendableMessage
)

It comes from Metamask address, like a normal web3 call
Current web3 calls doesn't show any UI before opening metamask

Hey @eordano !
First, as @HPrivakos said, the ephemeral key isn't used for these types of operations, so we should be ok on that side.
Then, regarding the pop-up, I agree that it would be nice to add a a message that explains that the scene is trying to perform a blockchain operation. Now, just to add some context, right now a lot of scenes do these operations without a decentraland pop-up, and only the Metamask one. If we were to add one, we should try to make it consistent.
Also, said pop-up should have the option to "Don't show again on this scene", so we don't show both the decentraland and Metamask pop-up all the time. However, this feature on itself would require a few changes:

We realised that we don't have a good way to identify a scene. The entity id changes with each deployment, and the parcels where it was deployed might change (for example a 3x3 scene might be reduced to a 3x2).
Where should we store this preference? On the profile? On the local storage?

I think it would be best to enable the Matic integration, and work on this pop-up or underlaying permission system on another issue.
Do you think that makes sense?
cc @marcosnc


Fix Incorrect DNS Zone API Specifications

The spec for the /getDnsZone and /getDnsZones API calls say that they return JSON where the value for key "domain" is a string.  However, these calls return a JSON object for the key "domain".
I ran into this problem when trying to use the go client library and this inconsistency with spec and implementation means that JSON parsing the response fails.
This issue can be reproduced with the following curl commands:
/getDnsZones
curl -H "Authorization: Bearer <personal_access_token>" https://api.netlify.com/api/v1/dns_zones/

/getDnsZone
curl -H "Authorization: Bearer <personal_access_token>" https://api.netlify.com/api/v1/dns_zones/<zone_id>/

The response I receive from these two API calls has the following type of JSON object for the "domain" key:
  "domain": {
    "id": string
    "name": string
    "user_id": string
    "created_at": string
    "updated_at": string
    "expires_at": string
    "registered_at": string
    "renewal_price": number
    "auto_renew": bool
    "auth_code": ??? (I got null),
    "transferred_at": ??? (I got null)
    "auto_renew_at": string
  }

yes


Drop SQLite development dependency

On CI, we run using Postgres and so this shouldn't be necessary. It
should fix this failure:
    sqlite3-1.7.1-x86_64-linux requires ruby version < 3.4.dev, >= 3.0, which is
    incompatible with the current version, ruby 2.7.0p0

Now we're failing with:
nokogiri-1.16.0-x86_64-linux requires ruby version < 3.4.dev, >= 3.0, which is
incompatible with the current version, ruby 2.7.0p0

Finally time to go back to #67!


sequencer and renderer fixes

Added a sequencer! Bare bones but fully functional. Could use more editing features for better flow.
Also a few improvements to the music function, the format is the same.

fix popping if first notes are empty due to undefined values
allow negative values to stop notes (currently only used for -1)
fix pop at end of song where final note is cut off to allow for better looping

Looks good! Not sure if need to something else to merge in your changes before you pull.

Thanks Frank. I'll rebuild the minified version, update readme and merge the changes.

This PR has been rolled into #11.


Add Github action to optimize all images

Copied from https://github.com/galaxyproject/galaxy-hub/pull/733, thanks @davelopez!
This should make shallow clones faster in the future.

Seems good, we should watch it for a bit since it has many triggers.

@martenson Sorry I didn't notice earlier the repo name in one of the conditions needs to match this repo. I made a PR with the change #2539.
In case you want to try it, you can trigger it manually. If there are many images in the repo, the first PR will contain all of them, but after that, the bot will create a PR only if there are new unoptimized images.


ModuleNotFoundError: No module named 'enchant'

I get this error on trying to use the module:
>>> import splitter
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/dcollier/dev/ds/ds-databricks/src/notebooks/intentris/venv/lib/python3.7/site-packages/splitter/__init__.py", line 1, in <module>
    from .compound_word_splitter import split
  File "/Users/dcollier/dev/ds/ds-databricks/src/notebooks/intentris/venv/lib/python3.7/site-packages/splitter/compound_word_splitter.py", line 3, in <module>
    import enchant, sys
ModuleNotFoundError: No module named 'enchant'

▶ pip install pyenchant
does seem to fix it, if this is a dependency can that be made more explicit?
I had to google around to find why pip install enchant actually is now pyenchant
https://superuser.com/questions/1501146/in-python-i-get-the-error-no-module-named-enchant

but it seems pyenchant is not maintained, I came across hunspell as an alternative which is used by chrome, so should be maintained.
https://github.com/blatinier/pyhunspell
although elsewhere I've used  SpellChecker

Thanks for reporting this. I think it makes sense to switch.


Enable/Fix multi-arch build for DOCKER_V2_BUILDER

When DOCKER_V2_BUILDER=true, the build on arm64 would get an error:
./tools/docker: line 29: out/linux_amd64/docker-builder: cannot execute
binary file: Exec format error
which is due to lack of arch and os information in tools/docker when
building with common/scripts/gobuild.sh(so it would lead to an amd64
binary on arm64 platform).
Now we set the right OS and Arch to enable and fix the building both
on arm64 and amd64.
Signed-off-by: trevor.tao<EMAIL_ADDRESS>Please provide a description of this PR:

Hi @TrevorTaoARM. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@TrevorTaoARM: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
482146221966aad897465642e1cd18ad71073c59
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Setting he release-notes-none label as no release notes needed.

/test release-notes_istio

It looks like this has broken the docker targets on the Mac.


Pooling operator expects kernel size even when 'global_pool' is set to True

Pooling operator expects kernel size even when 'global_pool' is set to True. If 'global_pool' is true, kernel should automatically be set to (height, width). User shouldn't have to provide kernel size.
Environment info
Operating System: Ubuntu
Compiler: gcc
Package used (Python/R/Scala/Julia): Python
MXNet version: 0.10
Or if installed from source:
MXNet commit hash (git rev-parse HEAD):
If you are using python package, please provide
Python version and distribution:
Python 2.7.12
If you are using R package, please provide
R sessionInfo():
Error Message:
Traceback (most recent call last):
  File "resnet50.py", line 965, in <module>
    name='pool5')
  File "/usr/local/lib/python2.7/dist-packages/mxnet-0.10.0-py2.7.egg/mxnet/_ctypes/symbol.py", line 181, in creator
    ctypes.byref(sym_handle)))
  File "/usr/local/lib/python2.7/dist-packages/mxnet-0.10.0-py2.7.egg/mxnet/base.py", line 84, in check_call
    raise MXNetError(py_str(_LIB.MXGetLastError()))
mxnet.base.MXNetError: Required parameter kernel of Shape(tuple) is not presented, in operator Pooling(name="", global_pool="True", pool_type="avg")

Minimum reproducible example
pool5 = mx.symbol.Pooling(data=data,
    pool_type='avg',
    global_pool=True,
    #kernel=(1,1),
    name='pool')

Steps to reproduce
or if you are running standard examples, please provide the commands you have run that lead to the error.

Create a pooling layer with 'global_pool' set to True and kernel size not set (like shown above)
Run the code.

'kernel=(1,1)' is a good setting and It's convenient for FCN. Please refer to https://stackoverflow.com/questions/42070528/what-does-global-pooling-do

The kernel parameter is required to determine whether the Pooling Operation is 1d or 2d or 3d based on the shape of the kernel. This is also used to determine the shape of stride. The size of each dimension doesn't matter when the global_pool is set to True.
https://github.com/apache/incubator-mxnet/blob/master/src/operator/pooling-inl.h#L104
https://github.com/apache/incubator-mxnet/blob/master/src/operator/pooling-inl.h#L107

Since the documentation was not clear to indhub@, is there an opportunity here to clarify/improve the API documentation? What is we update the document to say:
from:
global_pool (boolean, optional, default=False) – Ignore kernel size, do global pooling based on current input feature map.
to: (something like this)
global_pool (boolean, optional, default=False) – Ignore kernel size, but the kernel must be specified to determine the shape of the stride and whether the Pooling Operation is 1d or 2d or 3d based on the shape of the kernel. Do global pooling based on current input feature map.

@indhub  this does not seem necessary anymore. Please close:

pool5 = mx.symbol.Pooling(data=mx.sym.var('data'),
    pool_type='avg',
    global_pool=True,
    name='pool')

mx.test_utils.simple_forward(pool5, data=np.ones((1,3,224,224)))
array([[[[ 1.]],

        [[ 1.]],

        [[ 1.]]]], dtype=float32)


Alpha - Changing Tile format Messes up States

So I had a tiled progress bar, that i originally setup as 1 column, 2 rows, but i really wanted it left to right and so i changed it to a tile setup of 2 columns and 1 row. As soon as i did that, the states for the original set were removed, but only 1 state for the new one was created.
Will need to validate if this is a code issue or something with requests being made into Touch Portal and it not being handled correctly there.

That's a bit strange.  The fact that there is now a "col 2" state (and no "row 2") suggests it's working correctly in terms of removing the old ones... and new states are created the same way each time as long as DynamicIcon.stateCreated is false.  Which it must have been, otherwise that "col 2" state wouldn't have been created.
Try it again?  I have seen TP miss a createState message if it comes soon after a remove... very occasionally.  Seems unlikely that would happen here, but maybe since the remove/create messages are so close together.
Looking at the code, only other thing I can think of offhand is that the TP Client threw an exception on one of the createState() calls... but that would be logged I think.
The part where it deletes the previous states when changing tiling parameters is at index.ts:276.  Though that part doesn't seem to be the issue.
🤨

Seems to be a TP thing... I can recreate the issue but only randomly.  The missing state is usually col1,row1 because it always exists so it's deleted and then created rapidly. My guess is some timing in TP deletes it after the creation message comes in.
I added this delay and now can't recreate the issue:
 278:                    clearIconStates(icon);
 279:                    // set the tile property after clearing out any old states.
 280:                    Point.set(icon.tile, tile);
+281:                    await new Promise(resolve => setTimeout(resolve, 50));
 282:                }

Kinda ugly but maybe a workaround.

Yeah I figured as much, it has to be related to the rapid remove and create. Because I also saw if you increase one of the parameters, like row to 3 from 2 it has the same effect. It only ended up showing the row 3 state because the first two were removed. Has to be an issue with speed and TP. I don't love the idea of having to put in those sleep type statements.  But ... unsure how else it would work. Only other way is determine only the new states that are needed and don't remove the old ones if there is overlap...

Only other way is determine only the new states that are needed and don't remove the old ones if there is overlap

Yea, that was the way to go.... the old way was simple (lazy?) but the needing a delay is unpalatable.


Crash when choosing “Share” or “Collaborate”

I’m getting a crash when choosing either “Share” or “Collaborate”.
This is only happening on iPad. The same outline on iPhone does not cause the crash.
This is iPadOS 15.4, Zavala 2.0.6 (build 49).

Thanks for the bug report. This has been fixed and will be in the next release.


Init fails in deployments job

I tried to switch job which runs Init to deployment job but it's failing. Do I miss something in the context of deployment jobs or it's knowed limitation?
Failing deployment job:
jobs:
  - deployment: RunTerraform
    displayName: 'Run Terraform'
    environment: ${{ parameters.environment }}
    strategy:
      runOnce:
        deploy:
          steps:
            - task: TerraformInstaller@0
              displayName: 'Install Terraform'
              inputs:
                terraformVersion: 'latest'

            - task: TerraformCLI@0
              displayName: 'Init Terraform'
              inputs:
                command: 'init'
                workingDirectory: $(terraformWorkingDir)
                backendServiceArm: 'deployer'
                backendType: 'azurerm'
                backendAzureRmResourceGroupName: 'tfstate'
                backendAzureRmStorageAccountName: 'gwtfstate'
                backendAzureRmContainerName: 'tfstate'
                backendAzureRmKey: '${{ parameters.environment }}.tfstate'

Regular job which runs successfully
jobs:
  - job: RunTerraform
    displayName: 'Run Terraform'
    steps:
      - task: TerraformInstaller@0
        displayName: 'Install Terraform'
        inputs:
          terraformVersion: 'latest'

      - task: TerraformCLI@0
        displayName: 'Init Terraform'
        inputs:
          command: 'init'
          workingDirectory: $(terraformWorkingDir)
          backendServiceArm: 'deployer'
          backendType: 'azurerm'
          backendAzureRmResourceGroupName: 'tfstate'
          backendAzureRmStorageAccountName: 'gwtfstate'
          backendAzureRmContainerName: 'tfstate'
          backendAzureRmKey: '${{ parameters.environment }}.tfstate'

Failing Pipeline Logs
##[debug]Agent.ProxyUrl=undefined
##[debug]command=init
##[debug]command=init
##[debug]workingDirectory=/home/vsts/work/1/s/.deploy/terraform/environments/dev
##[debug]which 'terraform'
##[debug]found: '/opt/hostedtoolcache/terraform/0.15.0/x64/terraform'
##[debug]which '/opt/hostedtoolcache/terraform/0.15.0/x64/terraform'
##[debug]found: '/opt/hostedtoolcache/terraform/0.15.0/x64/terraform'
##[debug]/opt/hostedtoolcache/terraform/0.15.0/x64/terraform arg: version
##[debug]exec tool: /opt/hostedtoolcache/terraform/0.15.0/x64/terraform
##[debug]arguments:
##[debug]   version
/opt/hostedtoolcache/terraform/0.15.0/x64/terraform version
##[debug]allowTelemetryCollection=true
##[error]Error: There was an error when attempting to execute the process '/opt/hostedtoolcache/terraform/0.15.0/x64/terraform'. This may indicate the process failed to start. Error: spawn /opt/hostedtoolcache/terraform/0.15.0/x64/terraform ENOENT
##[debug]Processed: ##vso[task.issue type=error;]Error: There was an error when attempting to execute the process '/opt/hostedtoolcache/terraform/0.15.0/x64/terraform'. This may indicate the process failed to start. Error: spawn /opt/hostedtoolcache/terraform/0.15.0/x64/terraform ENOENT
##[debug]allowTelemetryCollection=true
##[debug]command=init
##[debug]command=init
##[debug]command=init
##[debug]executed command 'init'
##[debug]set TERRAFORM_LAST_EXITCODE=1
##[debug]Processed: ##vso[task.setvariable variable=TERRAFORM_LAST_EXITCODE;isOutput=false;issecret=false;]1
##[debug]task result: Failed
##[error]Error: There was an error when attempting to execute the process '/opt/hostedtoolcache/terraform/0.15.0/x64/terraform'. This may indicate the process failed to start. Error: spawn /opt/hostedtoolcache/terraform/0.15.0/x64/terraform ENOENT
##[debug]Processed: ##vso[task.issue type=error;]Error: There was an error when attempting to execute the process '/opt/hostedtoolcache/terraform/0.15.0/x64/terraform'. This may indicate the process failed to start. Error: spawn /opt/hostedtoolcache/terraform/0.15.0/x64/terraform ENOENT
##[debug]Processed: ##vso[task.complete result=Failed;]Error: There was an error when attempting to execute the process '/opt/hostedtoolcache/terraform/0.15.0/x64/terraform'. This may indicate the process failed to start. Error: spawn /opt/hostedtoolcache/terraform/0.15.0/x64/terraform ENOENT

Agent Configuration

OS - ubuntu
Hosted
Terraform version used - 0.15.0
AzureCLI version used - default for hosted agents

Removing bug label as this task was never intended to run as a deployment job nor has it been tested. Can you explain why you need to run this as a deployment job as opposed to the standard job?

The key reason that I want to associate a job with a DevOps Environments where I can set up approvals and checks.

@favoretti curious if you guys have setup approvals in pipelines that use this extension? Are you also using the environments feature in pipelines for this?

Closing this as it will be replaced this with #271.

@charleszipp yeah, we use Environments to set up approvals, everything seems to work fine.

  - stage: applyStage_${{ replace(replace(parameters.TFEnvironment, '/', '_'), '-', '_') }}
    displayName: "[${{ parameters.TFEnvironment }}] Apply"
    condition: and(succeeded(), eq(variables['Build.SourceBranch'], '${{ parameters.defaultBranchName }}'), eq(dependencies.planStage_${{ replace(replace(parameters.TFEnvironment, '/', '_'), '-', '_') }}.outputs['planJob.terraformPlanStatus.hasChanges'], 'true'))
    jobs:
      - deployment: applyJob
        timeoutInMinutes: 180
        displayName: 'Terraform apply'
        environment: ${{ parameters.deploymentEnvironment }}
        strategy:
          runOnce:
            deploy:
              steps:
                - download: none
                - task: DownloadPipelineArtifact@2
                  displayName: 'Download plan'
                  condition: and(succeeded(), eq(variables['Build.SourceBranch'], '${{ parameters.defaultBranchName }}'))
                  inputs:
                    source: 'current'
                    artifact: terraform_${{ replace(replace(parameters.TFEnvironment, '/', '_'), '-', '_') }}_plan
                    path: $(System.DefaultWorkingDirectory)/apply/${{ parameters.TFEnvironment }}

                - template: ./init-steps.yaml
                  parameters:
                    terraformVersion: '${{ parameters.terraformVersion }}'
                    workingDirectory: '$(System.DefaultWorkingDirectory)/apply/${{ parameters.TFEnvironment }}/${{ parameters.TFEnvironment }}'
                    secureVarsFile: ${{ parameters.secureVarsFile }}
                    sshKeySecureFileName: ${{ parameters.sshKeySecureFileName }}
                    sshKeyPassphrase: ${{ parameters.sshKeyPassphrase }}

                - task: TerraformCLI@0
                  displayName: 'Apply ${{ parameters.TFEnvironment }}'
                  condition: and(succeeded(),  eq(variables['Build.SourceBranch'], '${{ parameters.defaultBranchName }}'))
                  inputs:
                    command: 'apply'
                    commandOptions: '-input=false -parallelism=${{ parameters.parallelism }} -auto-approve tfplan'
                    workingDirectory: '$(System.DefaultWorkingDirectory)/apply/${{ parameters.TFEnvironment }}/${{ parameters.TFEnvironment }}'
                    secureVarsFile: ${{ parameters.secureVarsFile }}

@favoretti any ideas what can be wrong in my case as I'm using same tasks with same parameters for regular and deployment jobs?
Do you have any magic around terraform init in your template?

@pbakshy the error in the logs is indicating that the agent cant even run the terraform executable. Are you using a self-hosted agent? If so is the correct terraform executable for the OS installed? I dont think this is so much a problem with the yaml config as it is with the agent not being able to execute the terraform

Also if you look at the log, it cant even run terraform version. Its not even getting to the init part. Which tells me the agent cant run terraform at all.

We use Windows-hosted agents. I've got same error when previously specified wrong working directory, but in this case it doesn't change between normal and deploy jobs, and based or logs working directory is correct


Avoid dispatcher busy wait

The dispatcher was unable tell the difference between "queue empty"
and "timeout while dequeuing" leading to it looping continuously
(taking up 100% CPU in many cases).
By removing the arbitrary timeout, we know that when dequeue() returns
false, the queue is empty.  In order to support no timeout, we need to
fully empty the queue with a while loop when we are clearing before
checking _is_alive again.
This also has the effect of fully draining the queue on stop().
This is not fix for #5783, but it addresses some problems I noticed while debugging it.

I'm giving on this for now.