AppendBlob seal operation fails with REST API error using JavaScript SDK

Package Name: @azure/storage-blob
Package Version: 12.8.0
Operating system:
[x ] nodejs

version: 16.14.0

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ x] SDK API docs on https://docs.microsoft.com

Describe the bug
A seal operation on a previously created append blob fails with exception, propagating underlying REST API 400 error.
The seal fails, error claiming that comp=seal query parameter for PUT operation on target blob is invalid.
To Reproduce
Steps to reproduce the behavior:

Create new append blob
append multiple times
seal the append blob

Expected behavior
Seal operation succeeds, blob is sealed and read-only after it.
Screenshots
Additional context
Code to reproduce
const {BlobServiceClient} = require('@azure/storage-blob');
let blobServiceClient;
let containerClient;
async function initBlobClient() {
let connStr = process.env["CONN_STRING"];
if(!connStr || connStr.length == 0) {
console.error("No connection string found in ENV CONN_STRING");
process.exit(1);
}
let containerName = process.env["CONTAINER_NAME"];
if(!containerName || containerName.length == 0) {
console.error("No container name found in ENV CONTAINER_NAME");
process.exit(1);
}
blobServiceClient = BlobServiceClient.fromConnectionString(connStr);
containerClient = blobServiceClient.getContainerClient(containerName);
}
async function test() {
await initBlobClient();
let fn = "2021/01/01/12.csv"
// prepare the block with multiple appends
let appendBlobClient = containerClient.getAppendBlobClient(fn);
await appendBlobClient.createIfNotExists();
let data = "some\tcsv\tdata\tline\tcontent\tfor\tillustration\tpurpose";
for(let i = 0; i < 10;i++) {
try {
await appendBlobClient.appendBlock(data+"\t"+i+"\r\n", data.length+4);
}
catch(e){
console.error("block append failed, test precondition invalid: ", e);
process.exit(1);
}
}
try {
await appendBlobClient.seal();
}
catch(e) {
console.error("appendBlob.seal failed: ", e);
}
}
test();

Hi @gr4per ,
I tried with the code you shared, it succeeded to upload and seal an append blob.
Can this issue be reproduced? If so, could you share the request id for the 400 error to me? I'll try to find some clue from storage service.
Thanks
Emma

Hi Emma,
thanks for your qucik reply,
I just ran it again, here the details:
ubuntu@ubuntu:~/nmpi$ node reproduce.js
appendBlob.seal failed:  RestError: Value for one of the query parameters specified in the request URI is invalid.
RequestId:7d82e000-901e-0074-3cc8-28a9c6000000
Time:2022-02-23T15:16:32.3103081Z
{
"name": "RestError",
"code": "InvalidQueryParameterValue",
"statusCode": 400,
"request": {
"streamResponseStatusCodes": {},
"url": "https://nmpi.blob.core.windows.net/nmpi-test/2021%2F01%2F01%2F12.csv?comp=REDACTED",
"method": "PUT",
"headers": {
"_headersMap": {
"x-ms-version": "REDACTED",
"accept": "application/xml",
"user-agent": "azsdk-js-storageblob/12.8.0 (NODE-VERSION v16.14.0; Linux 5.4.0-1052-raspi)",
"x-ms-client-request-id": "0c4bac43-b71a-40e9-bd67-6834aef188b7",
"x-ms-date": "REDACTED",
"authorization": "REDACTED",
"cookie": "REDACTED"
}
},
"withCredentials": false,
"timeout": 0,
"keepAlive": true,
"decompressResponse": false,
"requestId": "0c4bac43-b71a-40e9-bd67-6834aef188b7"
},
"details": {
"errorCode": "InvalidQueryParameterValue",
"content-length": "375",
"content-type": "application/xml",
"date": "Wed, 23 Feb 2022 15:16:32 GMT",
"server": "Microsoft-HTTPAPI/2.0",
"x-ms-client-request-id": "0c4bac43-b71a-40e9-bd67-6834aef188b7",
"x-ms-request-id": "7d82e000-901e-0074-3cc8-28a9c6000000",
"message": "Value for one of the query parameters specified in the request URI is invalid.\nRequestId:7d82e000-901e-0074-3cc8-28a9c6000000\nTime:2022-02-23T15:16:32.3103081Z",
"code": "InvalidQueryParameterValue",
"QueryParameterName": "comp",
"QueryParameterValue": "seal",
"Reason": ""
},
"message": "Value for one of the query parameters specified in the request URI is invalid.\nRequestId:7d82e000-901e-0074-3cc8-28a9c6000000\nTime:2022-02-23T15:16:32.3103081Z"
}
ubuntu@ubuntu:~/nmpi$ node -v
v16.14.0
ubuntu@ubuntu:~/nmpi$ cat package.json | grep "storage"
"@azure/storage-blob": "^12.8.0",

Hi @gr4per ,
This should not be an issue on SDK, might be issue on service. I'm asking service team to look into it. Will get back to you later.
Thanks
Emma

Thank you, Emma! I tried already raising this as a service based ticket, but the automated validation on azure portal claimed no issues with the request ID and hence didn't let me post the issue. Quite odd, as this seems unexpected behaviour so something must be wrong indeed.

Hi @gr4per ,
Service team told me that sealing an append blob is not supported on HNS enabled account.  I tried with a HNS enabled account, also encountered the same 400 error.  Could you try with an account without HNS enabled?
Thanks
Emma

Hi Emma,
thanks for getting back. I tried with non HNS account and that goes through successfully.
I believe it would be a good idea to point this important piece out in the storage API docs.
Even more, it would be a great time saver for developers if the REST error would actually indicate a conflict with the storage account setup.
Cheers,
Arne

Thanks Arne.  We've opened a task to updated the REST API references for it.

Further assistant needed, please feel free to reopen the issue.


Videoplayer - Bug report

We in makeroid receive many issues from following component:
VideoPlayer.java in line
https://github.com/mit-cml/appinventor-sources/blob/308f1f33e82a313ebd8ec0c5afaa10de30fb1678/appinventor/components/src/com/google/appinventor/components/runtime/VideoPlayer.java#L640
and in line
https://github.com/mit-cml/appinventor-sources/blob/308f1f33e82a313ebd8ec0c5afaa10de30fb1678/appinventor/components/src/com/google/appinventor/components/runtime/VideoPlayer.java#L593

OS Version: 7.1.1
Device: SM-C710F
RAM Free: 27.6%
Disk Free: 10.4%

Fatal Exception: java.lang.IllegalStateException        at android.media.MediaPlayer.getVideoWidth(MediaPlayer.java)

It looks to me that the videoplayer was null, else I have no idea why we get the IllegalStateException.

Can you first of all replicate the following issues yourself on the App Inventor platform. It's not really a good practice to post issues from a separate platform without replicating it.

If you can't replicate the issue, even though it's the same file, it's no way to ensure that the same is caused on the App Inventor. Giving crash reports from a different platform even with a lot of similarities won't help. It's better not to post all the issues from your platform. Better give something that we could look into and try to fix. For example, project files.


[GHSA-99j7-mhfh-w84p] Slack Morphism for Rust before 0.41.0 can leak Slack OAuth client information in application debug logs

Updates

References

Comments
Adding the patch link for v0.41.0: https://github.com/abdolence/slack-morphism-rust/commit/4923fb7d458ed28c0302244c54cb4df0acee7ee6
This is the merge for 0.41.0 referenced in the original pull (https://github.com/abdolence/slack-morphism-rust/pull/133).

Hi there @abdolence! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


Ability to save book picture

Along with this issue we also need to figure out where we will display the image for the book.
Will the user click on the name of the book to go to the 'view' page for this book? Or will we display just a small version of the book image inline in the JQuery Table we currently have?

the easiest would be to add a picture column, something that would be better would be to remove the jquery table, and have a catalogue of book covers, then when you click on each book, you get more info, as well as the ability to edit or delete. However, I don't think we will have time for this so a picture column would have to do for now

Fixed in #98


Set up Send Message section

Since the website is hosted through GitHub as backend, PHP files can't be processed. Set up a 3rd party app to send emails.

Fixed via commit: 63ce40c6a9c377eb8e877a58dea073f47e4cff03


[5.5] Check if ability is defined using resource method

This PR fixes bug with policies which doesn't work properly with abilities registered using "Resource Gates" due to special format of it (eg. posts.view).
Mentioned bug occurs because "callable check" always returns false when ability is defined by resource.
if (! is_callable([$policy, $this->formatAbilityToMethod($ability)])) {
   return false;
}

For example, let's assume that ability is posts.view, this statement checks if posts.view method is callable instead of checking method associated with registered ability (which makes sense). Method resolvePolicyCallback returns false, rest of code isn't executed and as a result before method can't be called.
Solution
Gate has to check type of ability, properly get method name and then make sure if method is callable.

Can someone explain this more clearly?

@taylorotwell Generally it refers to docs (upgrade guide).

The before method of a policy class will not be called if the class doesn't contain a method with name matching the name of the ability being checked.

Gate::resource('name', 'App\Policies\ExamplePolicy', [
   'ability' => 'method',
]);

So when we register ability using resource method it will have format  name.ability. As we all know we aren't able to create function named name.ability due PHP limitations.

A valid function name starts with a letter or underscore, followed by any number of letters, numbers, or underscores.

Steps To Reproduce
First create model php artisan make:model Example
Define abilities in AuthServiceProvider (boot method)
Gate::define('ability', 'App\Policies\ExamplePolicy@ability');

// It will be tested later
//Gate::resource('name', 'App\Policies\ExamplePolicy', [
//   'ability' => 'ability',
//]);

Map policy in policies array
protected $policies = [
   'App\Example' => 'App\Policies\ExamplePolicy',
];

Create ability and before methods in ExamplePolicy class
public function ability(User $user, \App\Example $example)
{
   dd('ability is called');
}

public function before(User $user, $ability)
{
   dd('before is called');
}

In any controller (e.g. HomeController)
$ability = Gate::allows('ability', new \App\Example);

After all, you should see before is called in browser. Let's switch to resource method.
//Gate::define('ability', 'App\Policies\ExamplePolicy@ability');

// It will be tested later
Gate::resource('name', 'App\Policies\ExamplePolicy', [
   'ability' => 'ability',
]);

Now you have to change ability in allows method
$ability = Gate::allows('name.ability', new \App\Example);

After refresh you'll see ability is called instead of before is called.
Solution
I had to change first if-statement in resolvePolicyCallback method because it's not working with resource abilities (is_callable([App\Policies\ExamplePolicy::class, 'name.ability']) will always return false).
if (! is_callable([$policy, $this->formatAbilityToMethod($ability)]))

In this PR Gate checks "type of ability" (registered with resource or define methods). If ability is registered using resource - $callback will be set to $this->abilities[$ability] (each item in array contains callback so it can be used to check if it's callable, dd() this array will give you more details).
If you have any questions just ask.

@themsaid So this PR didn't entirely fix policies, the problem persists... I have to find better way.
I'll try to fix this in next weekend because I'm a little bit busy. Have I to create new PR or pull changes here?

@mputkowski a new PR targeting master would be better imo.


Fixed a link to an LF topic (again...)

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav-app/ui-extensions.md
:white_check_mark:Succeeded

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


error running repl: Fatal Error: MeTTa repl requires HyperonPy version matching '>=0.1.0, <0.2.0'

I added path to hyperonpy to PYTHONPATH and then run metta from build directory.
I got this error
Fatal Error: MeTTa repl requires HyperonPy version matching '>=0.1.0, <0.2.0'.  Found version: '0.0.0'

Anatoly, could you please paste what python -c 'import hyperon; print(hyperon.__version__)' command returns.
It looks like your PYTHONPATH contains very old version of Python hyperon library.

indeed it's 0.0.0

I installed with python3 setup.py install, if i install hyperon with pip install . the version is correct.


Problem with compilation

Environment

arch: x86_64
os: GNU / Linux Debian
version: testing

Description
It fails to compile:
/home/stefanos/development/sources/lua-eco/eco.c: In function ‘eco_resume’:
/home/stefanos/development/sources/lua-eco/eco.c:100:18: error: too few arguments to function ‘lua_resume’
  100 |     int status = lua_resume(co, L, narg);
      |                  ^~~~~~~~~~
In file included from /usr/include/lua5.4/lualib.h:11,
                 from /home/stefanos/development/sources/lua-eco/eco.c:26:
/usr/include/lua5.4/lua.h:300:15: note: declared here
  300 | LUA_API int  (lua_resume)     (lua_State *L, lua_State *from, int narg,
      |               ^~~~~~~~~~
make[2]: *** [CMakeFiles/eco.dir/build.make:76: CMakeFiles/eco.dir/eco.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:122: CMakeFiles/eco.dir/all] Error 2
make: *** [Makefile:136: all] Error 2

Yes, the error points at Lua 5.4, but I have tested it with 5.3 and produces the same error.
Also, I get this warning during cmake ..:
CMake Warning at CMakeLists.txt:108 (message):
  Not found libubus.  Skip build eco.ubus

Fixed.

New problem now:
stefanos@debian:build [master]$ make
[  3%] Building C object CMakeFiles/eco.dir/eco.c.o
[  7%] Linking C executable eco
[  7%] Built target eco
[ 11%] Building C object CMakeFiles/time.dir/time.c.o
[ 14%] Linking C shared module time.so
[ 14%] Built target time
[ 18%] Building C object CMakeFiles/bufio.dir/bufio.c.o
[ 22%] Linking C shared module bufio.so
[ 22%] Built target bufio
[ 25%] Building C object CMakeFiles/sys.dir/sys.c.o
[ 29%] Linking C shared module sys.so
[ 29%] Built target sys
[ 33%] Building C object CMakeFiles/file.dir/file.c.o
[ 37%] Linking C shared module file.so
[ 37%] Built target file
[ 40%] Building C object CMakeFiles/socket.dir/socket.c.o
[ 44%] Linking C shared module socket.so
[ 44%] Built target socket
[ 48%] Building C object CMakeFiles/termios.dir/termios.c.o
[ 51%] Linking C shared module termios.so
[ 51%] Built target termios
[ 55%] Building C object CMakeFiles/log.dir/log.c.o
/home/stefanos/development/sources/lua-eco/log.c: In function ‘lua_log_set_level’:
/home/stefanos/development/sources/lua-eco/log.c:15:5: error: implicit declaration of function ‘set_log_level’; did you mean ‘log_level’? [-Werror=implicit-function-declaration]
   15 |     set_log_level(level);
      |     ^~~~~~~~~~~~~
      |     log_level
cc1: all warnings being treated as errors
make[2]: *** [CMakeFiles/log.dir/build.make:76: CMakeFiles/log.dir/log.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:304: CMakeFiles/log.dir/all] Error 2
make: *** [Makefile:136: all] Error 2

Can you also create a .gitignore? We need to exclude build to avoid causing problems with future git pull requests.

Can you also create a .gitignore? We need to exclude build to avoid causing problems with future git pull requests.

git submodule update

git submodule update

OK, this worked.
Can we add it in README to clarify it for newcomers?
Also, can we add instructions how to use it without having to install it as super-user?


[idea] [ios] Alert Controller

Why?
AlertControllers (https://developer.apple.com/documentation/uikit/uialertcontroller?language=objc) are a way of presenting and requesting information to the user.
They are useful in Forms and other events.
How?
This is just a sample syntax. Considering that this element only will be presented on a boolean condition only.
This syntax is inspired by https://surface-ui.org/
<alert :if={show_alert} title="My Title" message="My Message">
  <button style="cancel">Cancel</button>
  <button>OK</button>
</alert>

This appears to be part of UIKit and would be out of scope for the switui client.
I do think this is important to cover though and probably should be a part of a project that covers UIKit components that you can add as a dependency

It's also available in SwiftUI as a presentation modifier
https://developer.apple.com/documentation/swiftui/view-presentation
https://developer.apple.com/documentation/swiftui/view/alert(_:ispresented:actions:)-1bkka

Alerts will definitely be needed, but the infrastructure isn't currently there to implement them. It's blocked by a couple things:

There needs to be a mechanism for turning DOM elements into modifiers that apply to the parent element
There needs to be a way of creating two-way bindings to assigns on the backend that are accessible from the client. Alerts in SwiftUI require a Binding to a boolean that controls whether the alert's presented or not, and it needs to be writable from the client (otherwise the alert can't be dismissed)

There needs to be a mechanism for turning DOM elements into modifiers that apply to the parent element

What do you think of the same syntax as liveview slots, i.e. child element with prefixed tag name?
<view>
  // view content
  <:alert title="Attention!" is-presented="false">
    // alert content
  </:alert>
</view>

Duplicates #636 . Rejecting


vmware_exporter bin file version

How can I get the package version?
Can we promote something like:
/usr/local/bin/vmware_exporter -v , --version
to get package version like in node_exporter

PR https://github.com/pryorda/vmware_exporter/pull/280

This feature was added in release  v0.17.1.

Closed


Remove global styles from dapp and api connection error handling

Summary of changes

Reference issue to close (if applicable)

Closes

Code Checklist

[ ] Tested
[ ] Documented

@nathan looks good just that Polkadot call if we can use the types


Update munit-cats-effect-3 to 1.0.0

Updates org.typelevel:munit-cats-effect-3 from 0.13.1 to 1.0.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (0.13.1).
You might want to review and update them manually.
website/src/hugo/content/changelog.md

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "munit-cats-effect-3" } ]

labels: library-update, semver-major, old-version-remains

Toss this in with #4665.


Black Screen with Hints

When viewing Gossip Stone Hints it just shows black screen when trying to view the Gossip Stone Hint.
Using Dolphin [makepkg] 5.0-19870 on Manjaro Linux

My guess is you're referring to the videos on the Sheikah Stone near the sparring hall. Those videos were removed to trim the file size of the randomized iso. Gossip stone hints refer to the gossip stones you would raise with the harp in vanilla. For example, there is one on the Lumpy Pumpkin.

Those stones display no text but shows a text box when interacting with them.

Which hint distribution were you using?

@alkaris2 Please upload the spoiler log here and also describe a way to reproduce this issue, currently we don't have enough information to reproduce this.
Which gossip stone exactly did you talk to that causes the issue? Does "black screen" mean some kind of crash or softlock or do you just mean the textbox is empty? Cause the empty textbox is known

Probably fixed by #502
I'll close this since there isn't enough information, please open a new issue with more infos if this bug persists


Updating CSB script to run the conv_ops_test_gpu standalone.

The //tensorflow/python/kernel_tests:conv_ops_test_gpu often times out when run as part of the CSN testsuite, which leads to the Nightly CSB run failing.
The timeout in that test seems to happen only when it is run in parallel with other tests. The timeout value is 900s, and even in cases where the test does not timeout, it only does so on the 2nd or 3rd run (note that tests that either fail or timeout are rerun upto 3 times).
A likely theory is that

the first run which is almost always guranteed to run in parallel wih other tests, and hence always timesout.
if the 2nd or 3rd run of the test happens, once other tests have finished, then it will succeed
else all 3 attempts timeout

When running in a standalone mode, the same test runs in under 300s (as opposed to taking more than 900s and timing out, when run in parallel).

/cc @whchung @jerryyin

jenkins: retest rocm please

the sole rocm failure was //tensorflow/python:convert_to_constants_test. that test seems very flaky...have seen it fail like this before too. will file a separate PR to put the no_rocm tag back on it :(
link to failing rocm log :
http://ml-ci.amd.com:21096/job/develop-upstream-unit-tests/job/tensorflow-upstream-unit-tests/1564/


online demo please :)?

for react.rocks... Github a pages is good host. Thanks!

:+1:
I will try to do this today


Wasm build getrandom dependency.

Im trying to build for wasm and getrandom dependency is making it difficult. I'm curious what its used for or if there is a way to exclude it from the build?

@taylorh140
Thank you for contacting us.
getrandom is used to set passwords for files.
If you do not need this feature, you can exclude it.

Ya i figured out that I can use a feature to prevent some issues. particularly:
cargo.toml
getrandom = { version = "0.2.12", features = ["custom"] }

and
use getrandom::register_custom_getrandom;
use getrandom::Error;

// Some application-specific error code
pub fn always_fail(_buf: &mut [u8]) -> Result<(), Error> {
    Ok(())
}

register_custom_getrandom!(always_fail);

@MathNya

@taylorh140 Thank you for contacting us. getrandom is used to set passwords for files. If you do not need this feature, you can exclude it.

Is there a easier way to exclude this?

getrandom is only used in src.helper.crypt.rs.
I think we can remove this class.
I would also like to address the problem with getrandom when building for wasm.
I will replace it with an alternative package in the next version.


Track only valid requests and responses

This Commit,

Adds Debug logs to slack and matter most bot to log the valid request sent to the bot and the corresponding responses generated.
removes the logging to all events, to avoid capture of sensitive data.

ISSUE TYPE

Feature Pull Request
Bug fix Pull Request

SUMMARY

Created this as a separate PR, as it not connected with Discord Integration.
added these features there as well.

Hello @codenio ,
Thanks again for the PR!
Thank you for contributing to BotKube. Could you please fill out this form, so we can send the well deserved awesome swags :slightly_smiling_face:
Team BotKube


Built docs should use relative paths

There are a few places where we assume that the docs will live inside /KaTeX.  This works when deploying to khan.github.io b/c the site lives in khan.github.io/KaTeX, but this won't work when deploying to netlify since we want index.html to live at katex.netlify.com instead of katex.netlify.com/KaTeX.

Relatedly, I found it confusing that the docusaurus server run via npm run start would not respond to http://localhost:3000/ (error message Cannot GET /), but rather required http://localhost/KaTeX.  I expected everything to live at the root level of the test server (which would require relative linking), but barring that, we should at least improve the error message.

All urls are generated using baseUrl in siteConfig.js:
https://github.com/Khan/KaTeX/blob/edede0867dde7250d2aab07bb54312fe256f86f5/website/siteConfig.js#L25

Doesn't running npm start automatically open a browser with http://localhost:3000/KaTeX?

I'm running npm start via ssh into a Linux machine, and in that setting, it doesn't.  (Cool to hear that it does that when running locally though.)

https://github.com/facebook/Docusaurus/issues/448 seems to be @kevinbarabash's original issue.

One place where we have to use absolute paths is docs, e.g., in #1520:

https://github.com/Khan/KaTeX/pull/1520/files#diff-4c5caa110a8eefbc2536f38bd144c8fcR5

<link rel="stylesheet" href="/KaTeX/static/katex.min.css"/><!-- must use absolute path -->

I'm not sure how to make it work using the environment variable.

Why does that path have to be absolute? Does ../static/katex.min.css not work?

@edemaine If versioning is enabled(#1521) or translations are enabled, the path may be nested one or two levels further.

I've opened #1526.

Closing as #1526 is merged and it's an upstream issue.


🎨 support config mail domain setting

closes https://github.com/TryGhost/Ghost/issues/11414

This enabled support for PR email-config-domain in Ghost repository.

members settings changes from the blog domain to config mail setting

Got some code for us? Awesome 🎊!
Please include a description of your change & check your PR against this list, thanks!

[ ] There's a clear use-case for this code change
[ ] Commit message has a short title & references relevant issues
[ ] The build will pass (run ember test from the repo root - will be core/client if working from the submodule in Ghost).

More info can be found by clicking the "guidelines for contributing" link above.

Reason same as here https://github.com/TryGhost/Ghost/pull/11474#issuecomment-572461194


Why Zesty - rocket league case study

Make background grey

Let's have the image still span the the whole way like in the original, just make the left side grey (and line up the top and bottom with the image)

Can the image line up with the top and bottom of the grey so there's no padding above or below the image?


Merge decoupling controller

This PR will merge the changes to decouple the controller from the main cycle simulator.
Fixes #3
does it fix others too?
@andresjarami , I have renamed the branch so that it better reflects the work (not the person working on it)
I created the PR because the commits seems to have been for a few days, I was waiting for you to create the PR so I can check it, but I suppose this is work to be checked already right? Whenever you finish a certain amount of work that is ready to merge, create a PR so we can check it and discuss it. Basically we want to do this

@andresjarami I couldn't do a full review with request of changes because I am the author of the PR. Next time, once you think the changes are ready for submission/review, you need to create the PR against OUR main branch (be careful creating the PR in the original repo which we forked) so that I can do a proper review


Could you please tell how to set the hyparameters of the GLUE?

Hello, I noticed that you gave a search space of hyparameters on GLUE dataset. I am confused that how you search the hyparameters. Did you train each task on each of the hyparameters with different seeds? There are about fifty combinations of parameters. Did you finetune GLUE with each of the parameters? Thank you.

The speach space refers to the search space in Should You Mask 15% in Masked Language Modeling?

Yes, we indeed run a large number of fine-tuning runs. We have a grid of hyperparameters for each task (see Appendix A.2) and run three random seeds per run.


Databases should be able to have their own defaults

At the moment defaults are set in config files/ command options.
But databases might have their specific needs in defaults and changing some default server-wide might introduce an issue in a database, if for example journal size is changed in the global  scope, it might affect a specific database's collection journal.
Same might happen with wait_for_sync or other default values that are inherited further down the database level.
Therefore it would be good for Databases to have their own default values, that can be picked up by any database objects. Only in the event where a database default is not set, it would mean to use the server default.
What do you think?

at the time being there are defaults for replication factor and sharding on per database basis.


Contact details Doctype

client module -->2'nd doctype [Contact Details Doctype]

Unnecessary changes in this PR. Closing


Docs2

@rishig

Thanks @Lyla-Fischer! Closing, since this article is now deleted / merged into the general "joining and inviting" doc.


Extra quotes appearing in string values

Hi,
Thanks for all your help recently.  I'm just doing some testing.  Since v<IP_ADDRESS>6-alpha, I am getting some odd results with some string values ... having extra quotes. For example, if I run this example:
var personV = new PersonVertex
{
    Ages = new[] { 4, 6, 23 },
    Id = "person-12345",
    Label = "some-label",
    Name = "my name"
};
var test = g.AddV(personV).As("person");
var resp = await graphClient.QueryAsync(test);

The code works and the Vertex is created, but I can no longer query for it in the Azure Portal (Data Explorer).  Even if I simply run g.V() then it gives this error:
Failed to execute query: g.V(""person-12345""): Script eval error:  ExceptionType : GraphCompileException ExceptionMessage :  Gremlin Query Compilation Error: Script compile error: Unsupported groovy language rule: 'commandExpression' text: 'g.V(""person-12345"")' @ line 1, column 1.
I noticed the extra quotes in that message, so I looked at the data using g.V().valueMap() instead, which gives this:
[
  {
    "Ages": [
      "4",
      "6",
      "23"
    ],
    "Name": [
      "\"my name\""
    ]
  }
]

Which is where I also see extra quotes in the Name property.  I'm guessing that's the problem and it's possibly doing the same with the id field.
Thanks again for your help.  And sorry about this.

Yeah, checking the data with a SQL query, the id also comes out with extra quotes, like this:
"id": "\"person-12345\"",
...which really seems to confuse the Azure Portal!  Nice!

Well that's embarrassing! That's what I get for not having unit tests, I guess. I'll take a look. Sorry for the hassle!

New release up. Sorry again! Proceed with caution, I guess.

No worries, thanks for your help!


Xiaomi Smart Plug WIFI

Device model / 设备型号
lumi.plug.v1
Component version / 插件版本
0.7.19
HA core version / HA版本
2024.7.3
Integrated mode / 集成方式
Automatic (自动模式)
The problem / 问题详情
Xiaomi Smart Plug integration fails to upgrade the power usage after Home Assistant Core-2024.2.5.
Entity attributes / 实体属性
select.lumi_v1_aed5_default_power_on_state - Power On/Off state and control works fine. However, the power usage fails to update
sensor.lumi_v1_aed5_electric_power
Home Assistant Logs / 系统日志
2024-07-22 10:39:53.339 ERROR (MainThread) [homeassistant.components.sensor] Platform xiaomi_miot does not generate unique IDs. ID lumi.158d0001716085-3ed40554ecb02d0e5abc9b993ba7b067-power_consumption-3.electric_power-2 is already used by sensor.lumi_v1_6085_electric_power - ignoring sensor.lumi_v1_6085_electric_power
2024-07-22 10:39:53.340 ERROR (MainThread) [homeassistant.components.sensor] Platform xiaomi_miot does not generate unique IDs. ID lumi.158d000183aed5-3ed40554ecb02d0e5abc9b993ba7b067-power_consumption-3.electric_power-2 is already used by sensor.lumi_v1_aed5_electric_power - ignoring sensor.lumi_v1_aed5_electric_power
2024-07-22 10:39:53.340 ERROR (MainThread) [homeassistant.components.sensor] Platform xiaomi_miot does not generate unique IDs. ID lumi.158d00015e7906-3ed40554ecb02d0e5abc9b993ba7b067-power_consumption-3.electric_power-2 is already used by sensor.lumi_v1_7906_electric_power - ignoring sensor.lumi_v1_7906_electric_power
2024-07-22 10:39:53.346 ERROR (MainThread) [homeassistant.components.sensor] Platform xiaomi_miot does not generate unique IDs. ID lumi.158d000114c023-3ed40554ecb02d0e5abc9b993ba7b067-power_consumption-3.electric_power-2 is already used by sensor.lumi_v1_c023_electric_power - ignoring sensor.lumi_v1_c023_electric_power
2024-07-22 10:39:53.366 ERROR (MainThread) [homeassistant.components.sensor] Platform xiaomi_miot does not generate unique IDs. ID lumi.158d000183b02b-3ed40554ecb02d0e5abc9b993ba7b067-power_consumption-3.electric_power-2 is already used by sensor.lumi_v1_b02b_electric_power - ignoring sensor.lumi_v1_b02b_electric_power
2024-07-22 10:39:53.382 ERROR (MainThread) [homeassistant.components.sensor] Platform xiaomi_miot does not generate unique IDs. ID lumi.158d000246aa8c-3ed40554ecb02d0e5abc9b993ba7b067-power_consumption-3.electric_power-2 is already used by sensor.lumi_v1_aa8c_electric_power - ignoring sensor.lumi_v1_aa8c_electric_power

The power consumption updates if I reload the miot. Once reloaded, it will never update.

Hi @sunnyiiii, do you have more than one smart plug?
I was seeing the same logs, and in my case, I removed one of the smart plugs from both HA and the Xiaomi app (cloud). The first plug started reporting parameters normally. After that, I re-added the second smart plug to both the Xiaomi app and HA, and it worked for me.

@rjaros87, Yes, I do have a few of them. I have tried adding new plugs to Xiaomi App. I have the same problem with the newly added plugs.

Fixed in beta version.

https://github.com/al-one/hass-xiaomi-miot/releases/tag/v1.0.0b3

Hi @al-one, thank you for fixing the Lumi power plugs. But the cuco.plug.v3 is not working.
2024-11-23 00:43:07.517 INFO (MainThread) [custom_components.xiaomi_miot.core.device.cuco.plug.v3] Device updated: {'indicator_light-3.mode': None, 'indicator_light.on': True, 'electric_power-11-2': 1, 'physical_controls_locked': False, 'switch.on': True}
2024-11-23 00:43:07.517 DEBUG (MainThread) [custom_components.xiaomi_miot.core.hass_entity] sensor.cuco_v3_9093_electric_power: Entity state updated: 1
2024-11-23 00:43:07.518 DEBUG (MainThread) [custom_components.xiaomi_miot.core.hass_entity] switch.cuco_v3_9093_switch: Entity state updated: True
2024-11-23 00:43:07.518 DEBUG (MainThread) [custom_components.xiaomi_miot.core.hass_entity] switch.cuco_v3_9093_physical_control_locked: Entity state updated: False
2024-11-23 00:43:07.519 DEBUG (MainThread) [custom_components.xiaomi_miot.core.hass_entity] light.cuco_v3_9093_indicator_light: Entity state updated: True
2024-11-23 00:43:07.519 DEBUG (MainThread) [custom_components.xiaomi_miot.core.hass_entity] button.cuco_v3_9093_info: Entity state updated: 29 - Washing Machine PM
2024-11-23 00:43:07.520 DEBUG (MainThread) [custom_components.xiaomi_miot.core.device.cuco.plug.v3] Finished fetching 68:ab:bc:af:90:93-3ed40554ecb02d0e5abc9b993ba7b067-update_miot_status data in 1.444 seconds (success: True)
2024-11-23 00:43:07.520 DEBUG (MainThread) [custom_components.xiaomi_miot.core.coordinator] 29 - Washing Machine PM(cuco.plug.v3): Coordinator updated: ['68:ab:bc:af:90:93-3ed40554ecb02d0e5abc9b993ba7b067-update_miot_status', <custom_components.xiaomi_miot.core.miot_spec.MiotResults object at 0x7f4896d22120>]

This is fixed.

Hi @al-one, I have downloaded the latest release v0.7.25, and I don't think it included the fixes in v1.0.0b3. Can you please take a look at this when you have time? In case, you missed it.

Thank you @al-one. Working flawlessly with [v1.0.1]


cmd/go: when initializing a go.mod from other formats, verify dependency module paths before saving?

The repo is loki, after use dep ensure at master branch:
[root@dx-app2 loki]# GO111MODULE=on go mod vendor
go: creating new go.mod: module github.com/grafana/loki
go: copying requirements from Gopkg.lock
go: converting Gopkg.lock: stat github.com/weaveworks/common@6ebd07f752e2a8d51d3ac919014f972350d25b39: unknown revision 6ebd07f752e2a8d51d3ac919014f972350d25b39
go<EMAIL_ADDRESS>go.mod has post-v0 module path "github.com/cespare/xxhash/v2" at revision 3b82fb7d1867
go: error loading module requirements
[root@dx-app2 loki]# go version
go version go1.11 linux/amd64
[root@dx-app2 loki]# gometalinter --version
gometalinter version 3.0.0 built from df395bfa67c5d0630d936c0044cf07ff05086655 on 2019-01-29T22:44:16Z
[root@dx-app2 loki]# go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/deploy/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/lib/golang"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build947627841=/tmp/go-build -gno-record-gcc-switches"

I do not know how to fix this.

Just to be sure that this isn't an issue that has already been fixed, try updating to the newest version of Go and trying again.
/cc @bcmills

I was able to reproduce this with a newer version of Go.
This project doesn't have a go.mod file, so when it's built with GO111MODULE=on, we generate one from Gopkg.lock. This isn't specific to go mod vendor; any build will do this.
Gopkg.lock has a dependency on github.com/cespare/xxhash at 3b82fb7d186719faeedd0c2864f868c74fbf79a1. We import this as the pseudoversion v0.0.0-20190104012619-3b82fb7d1867. However, xxhash is tagged as v2.0.0 at that commit, and it has a go.mod file, so it should be required as github.com/cespare/xxhash/v2 v2.0.0. After manually changing the requirement to this, go mod vendor completes successfully.
@bcmills I think this could be fixed in internal/modconv/convert.go, but I want to make sure I'm not going off into the weeds before starting. WDYT?
Related: this Gopkg.lock file has some case-sensitivity problems. For example, we end up with github.com/sirupsen/logrus v1.3.0 // indirect instead of github.com/Sirupsen/logrus.

Assuming that you don't actually need the v2 features, you should be able to fix your build by running go get<EMAIL_ADDRESS>

Since github.com/cespare/xxhash at commit 3b82fb7d1867 has a go.mod file specifying a /v2 path, go mod vendor really is correct to reject it: the go.mod file defines the expected import path, which doesn't match the import path under which your build is attempting to use it.
Probably the best we can do in terms of a general fix would be to download the go.mod file for each module in the converted go.mod file after conversion, check its module declaration, and remove any entries that don't match. We would presumable re-resolve those modules to latest whenever we need to access a package from them, and hopefully that would pick up something compatible.

Following that strategy, we'd lose some fidelity when importing from dep and other tools, but it seems preferable to writing out a broken go.mod file.
Russ pointed out that go get in GOPATH-mode will try fetching from the go1 tag before master. Projects should add a go1 tag when adding a go.mod with v2 or later. go-release should probably give advice on this.
I'm not sure if vendoring tools handle this tag. It's documented in go help get, but I hadn't heard of it before today. This also still relies on downstream projects doing the right thing when they aren't warned about doing the wrong thing.

Is there any doc that describe difference between v1 and v2, alse the different way go mod would use?
I am confused about it, control vendor is very hard to me, mod file is a good choice, but now, migrate from vendor cost me too much time.

@daixiang0 go help modules gives an introduction and reference on modules. You can also find this on the web at https://golang.org/cmd/go/#hdr-Modules__module_versions__and_more. The Modules page on the wiki has more detail and a lot of FAQs. Specficially Semantic Import Versioning.
To answer this specifically though, when a module is versioned at v2.0.0 or greater, its path must have the suffix v2. That means you need to import packages github.com/cespare/xxhash/v2, not github.com/cespare/xxhash.
In this project, there isn't a direct dependency on github.com/cespare/xxhash/v2; it comes through another project (I think it was one of the prometheus projects). So loki isn't actually doing anything wrong, but something upstream is importing the project with the wrong path, and neither dep nor go get in GOPATH-mode is reporting an error.
Concretely, you may be able to get this to work by depending on github.com/cespare/xxhash at an older version. You can do that simply by deleting the requirement on the module from your go.mod file. It will be added back automatically on your next build (go mod tidy will also add it). This assumes you don't depend on a feature or API in v2.0.0 though.

@daixiang0 I just wanted to briefly expand on some of the points from @jayconrod, as well as ask a few questions.
The first is I was wondering if your intent is to convert this loki repository over to using a go.mod file?
If so, it would be more conventional I think to first run go mod init, and to do that prior to running go mod vendor.
As @jayconrod said, the root of the problem seems to be github.com/cespare/xxhash. This quick test seemed to work for me at least (where in my case, I first ran go mod init to have a clean conversion step):
git clone https://github.com/grafana/loki /tmp/scratchpad/loki
cd /tmp/scratchpad/loki
go mod init github.com/grafana/loki
sed -i '/github.com\/cespare\/xxhash/d' go.mod
go list -m all
go mod vendor

I also then ran make, which seemed to complete successfully as well, but I am not familiar with the details of the loki project, so I don't know if make completing is meaningful or not.
In terms of where the v2.0.0 requirement for github.com/cespare/xxhash came from, there might be some nuance there. According to go mod graph | grep github.com/cespare/xxhash (run from within loki after the steps above), it seems loki's overall dependency on github.com/cespare/xxhash is coming via two of loki's dependencies:

github.com/prometheus/prometheus 718344434c52
github.com/prometheus/tsdb 6d489a1004dc

Both of those dependencies themselves seem to already have a go.mod, which both seem to require:

github.com/cespare/xxhash v1.1.0

However, if you run dep ensure update (or some other dep ensure variations) from inside of loki, at that point I think dep has no knowledge of the go.mod files that live in github.com/prometheus/prometheus and github.com/prometheus/tsdb.  I suspect at that point, dep needs to pick a version of xxhash for the loki Gopkg.lock file, and under those circumstances I think dep might pick the latest available semver version of xxhash, which happens to be v2.0.0.  In other words, I think when dep is run inside of loki, dep made a guess that github.com/prometheus/prometheus and/or github.com/prometheus/tsdb want the v2.0.0 version of xxhash, even though their go.mod files actually ask for the v1.1.0 version of xxhash.
Sorry if any of that is confusing or if I missed something here, but if correct, I think it would both explain where the v2.0.0 for xxhash came from, as well as underscore the validity of the original advice from @jayconrod to remove xxhash from the the loki go.mod file (because in fact, no one actually seems to be declaring their need for that v2.0.0 version).

@jayconrod @thepudds Thanks very much. Now i learn something about how go mod work.
The guess about why go mod vendor fail after dep ensure may be close to fact, i have some ideas about this:

if there is no go.mod, refuse to call go mod vendor and tip user to call go mod init could be better?
if init failed, do not generate go.mod since a broken file may be a trap right?
do not recommend run dep before go mod xxx

I am happy to contribute to docs about above ideas if possible. Thanks again for your patient explanation。

Then i call go mod tidy, still not find, is it normal?

go.mod will list requirements that are either direct dependencies by your module, or indirectly required by another module without a specific version (e.g., because the other module doesn't have a go.mod file). An indirect requirement will not be listed in go.mod if it is required at a specific version in another module's go.mod. This seems to be the case here.

if there is no go.mod, refuse to call go mod vendor and tip user to call go mod init could be better?

Whether you call go mod vendor depends on whether you need a vendor directory. It's supposed to work either way.

if init failed, do not generate go.mod since a broken file may be a trap right?

I don't think we should bail out entirely if there's an error importing from dep, especially if it's an error in a dependency the users can't directly address. In this case, there's a workaround (deleting problematic requirements and re-running go mod tidy or other module commands). But I think cmd/go should confirm imported requirements work before saving them to go.mod.

do not recommend run dep before go mod xxx

It can be useful to import versions from dep if specific versions are needed. Not always necessary, but frequently useful.

Note that as of #29433, you will not be able to run go mod vendor until after an explicit go mod init anyway.

It can be useful to import versions from dep if specific versions are needed. Not always necessary, but frequently useful.
@jayconrod Where should i add comment?

@daixiang0 I'm not sure more documentation is needed for this, but the Modules wiki is probably the right place if any details are missing.
Specifically, I have a problem with a complex dependency that has not opted in to modules. Can I use information from its current dependency manager? and FAQs — Possible Problems.


Implement simple application with iCub3 model to test FixedBaseTSID

This is needed to test https://github.com/dic-iit/bipedal-locomotion-framework/issues/251

The application has been implemented in https://github.com/isorrentino/iCub-torque-control/commit/15d3489eee32f723a9c1aa58cac07de31120e327. I still need to add the logging of all the variables to understand what happens while running. I'm saying that as if I try to set 4 contacts which are set starting from the initial end-effector position and moving the end-effector of 0.05 m along the x axis the motors go into hardware fault in simulation.
Here the video of what I described.
https://user-images.githubusercontent.com/43743081/116539640-63ca9800-a8e9-11eb-9c15-ccc7c0834f1f.mp4

I found a bug in the initialization of the kinDyn object. Now the behaviour is:
https://user-images.githubusercontent.com/43743081/117114913-b3e3a780-ad8c-11eb-90bc-37a1f6409499.mp4
and the cartesian position tracking error:


sowm: Fixed Borders

* Borders for windows

Made with code from #85.
Improved from #82.
Patch: https://patch-diff.githubusercontent.com/raw/dylanaraps/sowm/pull/91.patch

the sowm-handlebar.patch does not cover the border ? any fix


Use thiserror instead of eyre

I used eyre to get something up and running, but before I get too much further I should migrate over to custom error types using thiserror.
I'll probably still use eyre for main.rs since it has nice error formatting.
Going forward, using something like miette would be really cool, but that will probably take significant structural changes to the parser and such to be able to make high quality error messages.

Eyre has been removed from the dependency list. There are probably a few places where we are leaking miette Report or Error types in public APIs, but I'm less concerned about that given that miette is going to be used for regular user-facing diagnostics.


Can't find DAOTestRule.java  at dropwizard-testing maven 1.0.5 version jar

Can't find DAOTestRule.java  at dropwizard-testing maven 1.0.5 version jar.

@mutaherul That's because it's only available in Dropwizard 1.1.x and later.
https://github.com/dropwizard/dropwizard/commits/master/dropwizard-testing/src/main/java/io/dropwizard/testing/junit/DAOTestRule.java

Thanks.


UCP: TiKV memory control(5)

Description
Control memory in TiKV more precisely

Set and adjust memory capacity for coprocessor module dynamically

Difficulty

TBD

Score

TBD

Mentor(s)

@zhangjinpeng1987
@BusyJay

Recommended Skills

jemalloc
Rust

Duplicated with https://github.com/tikv/tikv/issues/6720


Additional recipient resources

adds slack, pagerduty, and webhook types
Am still unclear if any effort to DRY this out would be worth it, but will ponder (and am open to thoughts/suggestions)

Codecov Report

Merging #188 (5254691) into main (7152fa6) will decrease coverage by 1.57%.
The diff coverage is 63.67%.

@@            Coverage Diff             @@
##             main     #188      +/-   ##
==========================================
- Coverage   82.09%   80.52%   -1.58%
==========================================
  Files          34       37       +3
  Lines        2458     2690     +232
==========================================
+ Hits         2018     2166     +148
- Misses        352      421      +69
- Partials       88      103      +15

Impacted Files
Coverage Δ

honeycombio/provider.go
92.50% <ø> (ø)

honeycombio/resource_slack_recipient.go
58.82% <58.82%> (ø)

honeycombio/resource_pagerduty_recipient.go
62.16% <62.16%> (ø)

honeycombio/resource_webhook_recipient.go
65.43% <65.43%> (ø)

honeycombio/type_helpers.go
97.07% <90.00%> (+0.16%)
:arrow_up:

honeycombio/resource_email_recipient.go
55.07% <100.00%> (ø)

Continue to review full report at Codecov.

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


plugin "image_picker", tap the bottom of the screen when taking a photo and a keyboard will pop up

import 'package:flutter/material.dart';
import 'package:image_picker/image_picker.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);
  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  static final picker = ImagePicker();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Column(
        children: [
          Expanded(
            child: GestureDetector(
              behavior: HitTestBehavior.opaque,
              onTap: () {
                FocusScope.of(context).requestFocus(FocusNode());
              },
              child: Container(),
            ),
          ),
          Container(
            child: TextField(
              keyboardType: TextInputType.multiline,
              decoration: InputDecoration(
                border: InputBorder.none,
                isDense: true,
                contentPadding: const EdgeInsets.symmetric(horizontal: 15, vertical: 8.0),
                hintText: 'Please input here',
                hintStyle: TextStyle(
                  fontSize: 16,
                  fontWeight: FontWeight.normal,
                ),
              ),
            ),
          ),
        ],
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () async {
          final pickedFile = await picker.getImage(
            source: ImageSource.camera,
            maxHeight: 1000,
            maxWidth: 1000,
            imageQuality: 75,
          );
          print(pickedFile);
        },
        tooltip: 'Camere',
        child: Icon(Icons.photo),
      ),
    );
  }
}

➜  pick_image_issue flutter doctor -v
[✓] Flutter (Channel stable, 1.22.5, on Mac OS X 10.15.6 19G2021 darwin-x64, locale zh-Hans-CN)
• Flutter version 1.22.5 at /Users/macbook/Documents/flutter
• Framework revision<PHONE_NUMBER> (5 weeks ago), 2020-12-10 11:54:40 -0800
• Engine revision ae90085a84
• Dart version 2.10.4
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/macbook/Library/Android/sdk
• Platform android-29, build-tools 29.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.2, Build version 12B45b
• CocoaPods version 1.10.0
[✓] Android Studio (version 4.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin installed
• Dart plugin version 201.9002
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
[✓] VS Code (version 1.52.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.18.1
[✓] Connected device (2 available)
• xxxxx (mobile)    • xxxxxx • ios • iOS 13.3
• iPhone 6s (mobile) • 446368C7-C29F-458E-A7FD-72FECCAB11D9     • ios • com.apple.CoreSimulator.SimRuntime.iOS-12-4 (simulator)
• No issues found!

➜  pick_image_issue flutter doctor -v
[✓] Flutter (Channel stable, 1.22.5, on Mac OS X 10.15.6 19G2021 darwin-x64, locale zh-Hans-CN)
• Flutter version 1.22.5 at /Users/macbook/Documents/flutter
• Framework revision<PHONE_NUMBER> (5 weeks ago), 2020-12-10 11:54:40 -0800
• Engine revision ae90085a84
• Dart version 2.10.4
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/macbook/Library/Android/sdk
• Platform android-29, build-tools 29.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.2, Build version 12B45b
• CocoaPods version 1.10.0
[✓] Android Studio (version 4.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin installed
• Dart plugin version 201.9002
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
[✓] VS Code (version 1.52.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.18.1
[✓] Connected device (2 available)
• xxxxx (mobile)    • xxxxxx • ios • iOS 13.3
• iPhone 6s (mobile) • 446368C7-C29F-458E-A7FD-72FECCAB11D9     • ios • com.apple.CoreSimulator.SimRuntime.iOS-12-4 (simulator)
• No issues found!

@ashamp
There's similar open issue describing your case, https://github.com/flutter/flutter/issues/42417
Please follow-up there for updates.
Closing this as duplicate.

@ashamp
There's similar open issue describing your case, https://github.com/flutter/flutter/issues/42417
Please follow-up there for updates.
Closing this as duplicate.


[Bug] We are unable to take screenshot after the commit 18ca4ea

Hello Rui,
I hope you are doing well.
After the below commit 18ca4ea we are unable to take screenshots.
We are getting the error below:
background.js:83769 Uncaught Error: binary: expected base64-encoded buffer, got object
    at tBinary (background.js:83769:11)
    at background.js:83808:22
    at CrxConnection.dispatch (background.js:118150:26)
    at CrxConnection.dispatch (background.js:118434:18)
    at background.js:135631:83
    at run (background.js:312:9)
    at runIfPresent (background.js:336:11)
    at channel.port1.onmessage (background.js:383:7)
tBinary @ background.js:83769
(anonymous) @ background.js:83808
dispatch @ background.js:118150
dispatch @ background.js:118434
(anonymous) @ background.js:135631
run @ background.js:312
runIfPresent @ background.js:336
channel.port1.onmessage @ background.js:383

It was working on the commit d808a75
The issue is not reproducible via unit tests,
for example, the below code takes the screenshot:
test("test screenshot", async ({ page }) => {
  await page.goto("https://example.com");
  const screenshot = await page.screenshot();
  expect(screenshot).not.toBeNull();
});

To reproduce the issue we have to load it as an extension to chrome.
We can simply add before createTodos
.
.
.
await page.screenshot();
await createTodos(page);

I believe the change was the playwright's version. Since the difference is huge and the change caused the issue is not on our side I don't know where to look for the fix.
I would love to help if you can give me a direction.
Thanks in advance,
Best regards,
Ekin

Hmm, you're right. As reference, here's the test I used:
test("should take screenshot", async ({ runCrxTest }) => {
  await runCrxTest(async ({ expect, page, server }) => {
    await page.goto(server.EMPTY_PAGE);
    await page.setContent('
Hello World!
');
    const screenshot = await page.screenshot();
    expect(screenshot).not.toBeNull();
  });
});

and the stacktrace with sourcemap:
validatorPrimitives.ts:68 Uncaught Error: binary: expected base64-encoded buffer, got object
    at tBinary (validatorPrimitives.ts:68:11)
    at validatorPrimitives.ts:112:21
    at CrxConnection.dispatch (connection.ts:161:26)
    at CrxConnection.dispatch (crxConnection.ts:51:18)
    at index.ts:46:81
    at run (setImmediate.js:40:13)
    at runIfPresent (setImmediate.js:69:21)
    at channel.port1.onmessage (setImmediate.js:128:13)

I already fixed the bug, and I'm going to release a new version today or tomorrow with playwright 1.42.0

I just released 0.4.0

Thank you very much!


AggregatedFullHttpMessage toString() behaviour

This is related to #3019, the previous problem persists:
Prior to v4.0.24-FINAL you could call .toString on the output of HttpObjectAggregator regardless of whether the resulting HttpMessage has been released or not, from v.4.0.24-FINAL onwards you will encounter an error similar to:
io.netty.util.IllegalReferenceCountException: refCnt: 0
	at io.netty.buffer.DefaultByteBufHolder.content(DefaultByteBufHolder.java:39) ~[netty-all-4.0.27.Final.jar:4.0.27.Final]
	at io.netty.handler.codec.http.HttpMessageUtil.appendFullCommon(HttpMessageUtil.java:79) ~[netty-all-4.0.27.Final.jar:4.0.27.Final]
	at io.netty.handler.codec.http.HttpMessageUtil.appendFullRequest(HttpMessageUtil.java:55) ~[netty-all-4.0.27.Final.jar:4.0.27.Final]

This is due to the fact that AggregatedFullHttpMessage overrides .copy, .duplicate, and .retain from the ByteBufHolder interface but delegates .content to the DefaultByteBufHolder superclass which  checks the ref count every time you access content:
@Override
public ByteBuf content() {
    if (data.refCnt() <= 0) {
        throw new IllegalReferenceCountException(data.refCnt());
    }
    return data;
}

whereas something like DefaultHttpContent does not check that count:
@Override
public ByteBuf content() {
    return content;
}

It appears the AggregateFullHttpMessage only takes the benefit of .copy and .release from that type hierarchy, so I offer a PR where that is removed and the impl is similar internally to DefaultHttpContent, apologies if I've missed something obvious.
context
I have a Clojure networking framework which combines netty, clojure/core.async, and stuartsierra/component in an attempt at an async, event-driven architecture.
Basically you can combine components using core.async channels and macros, netty is plugged in as the network layer which translates input/output events to/from requests/responses. Any netty codec would work but in reality I use a fairly standard HTTP channel pipeline most often, and HttpObjectAggregator features in that.
I can manage the lifecycle of pooled bytebuf cleanly, or for some applications when each request is decoded and converted to an event I copy the bytebuf into memory as a single java.util.ByteBuffer and release the original at that point. In that case I still place the request alongside the ByteBuffer on the event, making it available for further processing - i.e. decoding the query-string or http-headers.
On occasion events are logged to disk, which calls toString, which errors if the msg has been released, where previously it did not.
There are multiple solutions to my problem, I could always manage pooled buffers and not memory copy, I could always configure my server to use Unpooled buffers, but all the other HttpMessage impl seem to be fine with .toString when released and the original behaviour has changed so I thought I'd chip in again.
Ta,
Derek

Fixed by #3638


Secondary WAN incorrect status [Bug]

The problem
Hi,
I'm using Asus RT-AX88U with Dual Wan enabled (second wan is via usb lte dongle). As you can see in the screenshot below entity "Secondary WAN" is set as "Connected" although it is not in reality (look at the secondary WAN IP):

On the router configuration page the status of the secondary wan is "Cold - Standby":

Question - is it possible to get the value "Disconnected" in this integration given the fact that secondary wan is actually disconnected?
Thanks!
Your device model
RT-AX88U
Firmware type
Merlin
Firmware version
3004.388.7
Integration version with the issue
0.31
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
Core 2024.5.3
Is there anything useful in the logs?
No response
Diagnostics information
No response
Additional information
No response

Can you please show what does the attribute link look like? Unfortunately, I don't have LTE dongles, but when tested with phone and tethering, it works as expected. Might be, USB modems report differently

Here:

I do not observe any issues - defect to be closed


PE-460: Bulk bundles

This PR uses the new uploadAllEntities method from Core for all upload paths. The expected results for a bundled multi file upload have been changed to expect one combined bundle. New BATS test cases have been added for a bulk folder

Nit: there's a conflict at the README.md

I like how the changes at CORE made this code simpler by using that method which doesn't care about the entities' privacy, but instead uses a generic type 👏

We needed to abstract the privacy away from the core method here because CSV uploads can be sent up to different destination folders with potentially different privacy levels


Heap & asm instance pooling

Fix #154 (Improving Firefox performance). Also, bring back sha*.bytes() convenience functions.

Will this get merged?

Hey :wave: There is an updated implementation of this, here: https://github.com/openpgpjs/asmcrypto.js/commit/6e5fe3f23fc58d57ae33330473d887b0a5193e74. If there's interest from the maintainers I'd be happy to update this PR to that; alternatively you can also use that fork in the meantime.

@alippai any chances that this can get picked back up?


Follow redirects for GH issue creation

I tracked this down related to a CX escalation. Issue creation URLs for
at least some repos are currently returning 307 statuses: GH's docs
make it fairly clear that we should not be changing the URL we start
with, but we should be following these. I was locally able to confirm
this fixed issue creation for an affected repo.

Nice catch. LGTM modulo CC issues.

Our travis config seems to have gotten broken somehow. I've switched to Circle & turned off Travis rather than investigate that breakage too much, since we've been moving most of our repos to Circle.


Windows periodic job: Add creation timestamp to RG

When creating a resource group in Azure for the Windows Periodic Job, we need to tag it with a timestamp so that the automatic cleanup tool will not delete the RG mid run.
Signed-off-by: Adelina Tuvenie<EMAIL_ADDRESS>

Fixes: https://github.com/containerd/containerd/issues/5549


stream does not represent a PKCS12 key store

Hi, I have a this problem. Can you help me? I'm not solution other issues

Hi, you need to use a valid cert for android. It could be taken from .swan file.

I had the same issue today and to fix that you need to use a correct certificate & password.
Check the other issue in this repo


Fix govendor for x/crypto curve25519

Fixes build on non amd64 arhitecture

Codecov Report

Merging #2925 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2925      +/-   ##
==========================================
- Coverage   27.26%   27.24%   -0.02%
==========================================
  Files          89       89
  Lines       17640    17640
==========================================
- Hits         4809     4806       -3
- Misses      12144    12146       +2
- Partials      687      688       +1

Impacted Files
Coverage Δ

modules/process/manager.go
69.56% <0%> (-4.35%)
:arrow_down:

Continue to review full report at Codecov.

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

LGTM

LGTM


chore: add ignore_piece_content_status url parameter to disable piece content status checks

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

developer feature

What is the current behavior? (You can also link to an open issue here)

in a setup with no working media-manager/package-manager clip missing warnings are produced for every clip in the rundown, which hides any potential real warnings that are produced

What is the new behavior (if this is a feature change)?

adds a url 'config' parameter, to bypass the piece content checks

Other information:

@jstarpl I am unsure if doing the localStorage checks will have a performance penalty and if the value should be cached?
Targetting r37 for now, that can be changed to r38 if wanted
Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report

Merging #583 (4fb67a4) into release37 (693a78e) will decrease coverage by 0.29%.
The diff coverage is 20.00%.

@@              Coverage Diff              @@
##           release37     #583      +/-   ##
=============================================
- Coverage      67.29%   66.99%   -0.30%
=============================================
  Files            261      262       +1
  Lines          17894    18259     +365
  Branches        3838     3877      +39
=============================================
+ Hits           12042    12233     +191
- Misses          5811     5985     +174
  Partials          41       41

Impacted Files
Coverage Δ

meteor/client/lib/localStorage.ts
22.03% <20.00%> (-0.19%)
:arrow_down:

meteor/server/api/packageManager.ts
36.00% <0.00%> (-3.29%)
:arrow_down:

meteor/server/api/integration/expectedPackages.ts
14.35% <0.00%> (-1.99%)
:arrow_down:

meteor/server/api/peripheralDevice.ts
69.43% <0.00%> (-1.57%)
:arrow_down:

meteor/lib/api/pubsub.ts
91.52% <0.00%> (-1.34%)
:arrow_down:

meteor/lib/api/peripheralDevice.ts
92.70% <0.00%> (-0.82%)
:arrow_down:

meteor/lib/api/userActions.ts
100.00% <0.00%> (ø)

...server-core-integration/src/lib/corePeripherals.ts
100.00% <0.00%> (ø)

meteor/lib/collections/PackageContainerStatus.ts
77.77% <0.00%> (ø)

meteor/lib/lib.ts
79.76% <0.00%> (+0.19%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


the default model class is ErrorViewModel, not ErrorModel

You can see the correct name is already used the the code sample two sentences down

Hello @johnmwright ... This passage refers to a Razor Pages app. It refers to the template code of the ErrorModel ...
namespace Sample.Pages
{
    public class ErrorModel : PageModel
    {
        public string RequestId { get; set; }

        public bool ShowRequestId => !string.IsNullOrEmpty(RequestId);

        [ResponseCache(Duration = 0, Location = ResponseCacheLocation.None, NoStore = true)]
        public void OnGet()
        {
            RequestId = Activity.Current?.Id ?? HttpContext.TraceIdentifier;
        }
    }
}

The example below the next paragraph refers to an MVC app.

btw - It wouldn't be terrible to have that code on display in the text. However, the engineers change the content of the ErrorModel. They changed it between 2.0 and 2.1. If we were to include the code, it would require versioning in the topic, and it would be fragile going forward. We'd likely miss a change at some point in the future and end up with a stale code example when the engineers make additional changes.

@Rick-Anderson Do u think we should maintain example code for the ErrorModel? I'm not really feel'in it based on the template churn. This page model was changed between 2.0 and 2.1 ... and it's changing again for 2.2 (click to see the 2.2 version). We might be fighting a losing battle if we go down this road. ⚠

See my commit, that should solve the problem.

@scottaddie  another WIP bug. No WIP in commit title or label.

No WIP link either


every flutter app seems jank on my android devices

I experience the janky ui on my any Android device.(Xiaomi 6，Xiaomi 8，Huawei p30，), but some other devices run well.
no matter scroll, switch tab or other simple interaction, the app is lower 60 fps.
I tested some flutter app, including offical flutter app, they all lead to the same experience.
please tell me what's the problem with this situation.is it flutter not compatible with some android device?
here's the preview
https://drive.google.com/open?id=1StG18kxAMqaJ98lEGBw0--j1U4tV6FQz
my updated flutter doctor -v
[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Mac OS X 10.15.1 19B88, locale
zh-Hans-CN)
• Flutter version 1.9.1+hotfix.6 at /Users/jason/Downloads/flutter
• Framework revision 68587a0916 (9 weeks ago), 2019-09-13 19:46:58 -0700
• Engine revision b863200c37
• Dart version 2.5.0
[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /Users/jason/Library/Android/sdk
• Android NDK at /Users/jason/Library/Android/sdk/ndk-bundle
• Platform android-29, build-tools 28.0.3
• Java binary at: /Applications/Android
Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build
1.8.0_202-release-1483-b49-5587405)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor
--android-licenses
[!] Xcode - develop for iOS and macOS (Xcode 11.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.2, Build version 11B52
✗ CocoaPods installed but not working.
You appear to have CocoaPods installed but it is not working.
This can happen if the version of Ruby that CocoaPods was installed with is
different from the one being used to invoke it.
This can usually be fixed by re-installing CocoaPods. For more info, see
https://github.com/flutter/flutter/issues/14293.
To re-install CocoaPods, run:
sudo gem install cocoapods
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 41.0.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build
1.8.0_202-release-1483-b49-5587405)
[✓] Connected device (1 available)
• MI 6 • e96002b7 • android-arm64 • Android 9 (API 28)
! Doctor found issues in 2 categories.

Hi @lawliet467
could you please provide your updated  flutter doctor -v
and a minimal code sample to reproduce the problem?
Thank you

Hi @lawliet467
could you please provide your updated flutter doctor -v
and a minimal code sample to reproduce the problem?
Thank you

I dont't have a flutter project., even I install any other flutter apps, it happens always.

Hi again @lawliet467
I'm not sure to understand,
how are you experiencing this issue without a Flutter project.
Can you please describe the issue more in dept?
thank you

it means that some my android devices are in the same janky experience which installed the offical flutter example apps.
---Original---
From: "Francecsco<EMAIL_ADDRESS>Date: Fri, Nov 15, 2019 19:55 PM
To<EMAIL_ADDRESS>Cc: "Jason"<532744984<EMAIL_ADDRESS>Subject: Re: [flutter/flutter] every flutter app seems jank on my android devices (#44901)
Hi again @lawliet467
I'm not sure to understand,
how are you experiencing this issue without a Flutter project.
Can you please describe the issue more in dept?
thank you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi @lawliet467
without additional information this issue is not actionable;
unfortunately "janky" (quote) is not a sufficient description.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


cmake + toolchain: Unique source file ID

This PR introduces a mechanism to add a unique ID for each source file. This mechanism is then used to generate unique section names that do not contain any \/. characters.
This fixes an issue where it was impossible to relocate variables/functions using the cmake zephyr_code_relocate function if they were placed in a unique section.

This fixes an issue where it was impossible to relocate variables/functions using the cmake zephyr_code_relocate function if they were placed in a unique section.

would be very nice if you can open an issue which precisely describes the problem you're trying to solve.


COVID-19 Case Data Schema Documentation

Making the Schema documentation for various COVID-19 data sources (for cases) available as a markdown document.  This is currently a draft document.

Thanks @olynch .  Yes, I can certainly do that.  Committing another version in a few minutes.

Hi @olynch .  Can you please review the latest changes?
Thanks,


skforecast version

Roll back from 0.8.1? Causes a bug with exogenous inputs

skforecast version 0.8.1 is supported in buildings_bench release v1.1.0.


DIDCommMessaging Service Endpoint is in the wrong format

This issue of credo-ts also exists in ACA-Py.

It's been observed that generating DIDs for did:peer:2 with a service endpoint type of DIDCommMessaging, the serviceEndpoint follows the format of the did-communication endpoint type instead of the DIDCommMessaging endpoint type.

ACA-Py is following the did-communication convention right now. This convention is defined here: https://github.com/hyperledger/aries-rfcs/blob/main/features/0067-didcomm-diddoc-conventions/README.md#service-conventions
ACA-Py is correctly adhering to this convention and has not yet adopted the DIDCommMessaging service type.
Credo, by comparison, is using the DIDCommMessaging service type but using the old did-communication conventions i.e. the serviceEndpoint is a string when it should be an object with the accept and routingKeys fields.
I'm not sure I see how this issue applies to ACA-Py at the moment. Could you elaborate?

I filed this issue to notice that there's a compatibility issue among peer-did libraries as listed here: https://github.com/decentralized-identity/peer-did-method-spec/issues/64
At first, I thought using did-communication type services in peer-did was not conforming to the spec. Now I have learned that the spec does not specify which type to use. And maybe this is the source of the compatibility issue.


Documentation: commands: add image extract/render/show

Some initial thoughts to close out #633 and #645.

cc @jonboulle @bcwaldon thoughts?

This is not really rkt specific -- it's just ACI manipulation. Should it be in actool instead? If we want it to work with rkt cas, it would need to be in rkt, of course.

@eyakubovich It is exporting stuff out of the CAS, it shouldn't fetch I don't think.

@eyakubovich see previous discussion in #633, this should only be Rocket-specific stuff

How is everyone feeling on this proposal turning into code at this point? @eyakubovich @jonboulle @bcwaldon ?

Seems reasonable. Another useful sub-command would be to list contents. When working with tarballs, rpms, debs, I often just want to list the filenames in the package.

cc @robszumski We need to think through the various subcommands in rkt a bit. Any thoughts on adding this sort of stuff?

Subcommands aren't as discoverable but at some point you have so many commands that there isn't really a choice. A lot of tools pick tricky naming like foo list-images but you can easily remember it as foo image-list and it gets frustrating quickly. It might make sense to create a new issue that lists out all the current and planned commands to see how they work work as regular commands vs with subcommands.

As the cas can contain multiple images with the same name the user should be able to choose them in a more precise way.
I propose to also standardize the URL like image format. Ex. coreos.com/etcd:v2.0.5,os=linux,arch=amd64
So on the implementation side cas.GetACI can be directly used to find the best image.
Additionally the cas can also contain images with completely identical name and labels. They can be distinguished by 2 criteria:

ImageID
UrlLike image and import time (assuming no images are imported at the same time)

While the first is obviously just ready to get rendered the second will need a new function to find it.
I think the doc and future implementation should also handle these options.

@sgotti it sounds like you want to formalise something like this? https://github.com/appc/spec/issues/4

@jonboulle Yeah as it's probably going to be used by various commands.

Do we expect users to type coreos.com/etcd:v2.0.5,os=linux,arch=amd64? It seems that coreos.com/etcd:v2.0.5 should be enough to get the right image for the current os and arch.

I agree with @robszumski that it would be nice to consider the number of subcommands we have and how they fit together.

@philips Does this solve the docker images experience as well? Over time I will fetch many ACIs and it would be nice to be able to run something like rkt images to get a list of those ACIs.

Do we expect users to type coreos.com/etcd:v2.0.5,os=linux,arch=amd64? It seems that coreos.com/etcd:v2.0.5 should be enough to get the right image for the current os and arch.

@kelseyhightower os/arch would continue to be substituted in with defaults as they are today

On Sun, Apr 12, 2015 at 6:32 PM, Kelsey Hightower
<EMAIL_ADDRESS>wrote:

@philips Does this solve the docker images experience as well? Over time I will fetch many ACIs and it would be nice to be able to run something like rkt images to get a list of those ACIs.

I think so. I don't know exactly what problem you are thinking of
here. Can you be more specific @kelseyhightower.

Does this solve the docker images experience as well? Over time I will fetch many ACIs and it would be nice to be able to run something like rkt images to get a list of those ACIs.

@kelseyhightower For image listing I opened #587. By now its an "images" command but it should become something like image list.
In #603 there's image removal and in comment https://github.com/coreos/rkt/pull/603#discussion_r27068364 @philips proposed to call it image remove.

I am going to give rkt image cat-file a try because I will need to use it to get image manifest for rkt in k8s. Anything more you guys want to talk about this?

I had the same concern as bcwaldon about the semantic, how about cat-manifest and cat-file (which does not require rootfs/ prefix) ?

@jonboulle SGTM. Actually in the implementation, the manifest can be read from the store without rendering the ACI. And I think the manifest will be cat more frequently than some binary files in rootfs ...

We'll handle this in #1098

woot woot!
On Wed, Jul 8, 2015 at 3:07 AM, Iago López Galeiras <
<EMAIL_ADDRESS>wrote:

Closed #649 https://github.com/coreos/rkt/pull/649.
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/rkt/pull/649#event-350625899.


Code quality: Introduced SidebarContext

Details
This PR introduces SidebarContext which will be needed to implement RichCommand for sidebar items (such as OpenInNewPane, OpenInNewTab, UnpinItem, ...)
@cinqmilleans I'm once again asking for your review and help
Validation
How did you test these changes?

[x] Built and ran the app
[ ] Tested the changes for accessibility

The goal is for actions to behave differently when a sidebar context menu is open. RightClickedItemChanged is only triggered when the context menu opens, not when it closes, so you have to notify when closed
(RightClickedItemChanged should announce that the item becomes null again).
Booleans can easily be replaced by classes.
public bool IsDriveItem => rightClickedItem is DriveItem;
public DriveItem? OpenDriveItem => rightClickedItem as DriveItem;
This makes IsDriveItemPinned useless.

The goal is for actions to behave differently when a sidebar context menu is open. RightClickedItemChanged is only triggered when the context menu opens, not when it closes, so you have to notify when closed (RightClickedItemChanged should announce that the item becomes null again).

I'll implement this also in WidgetsContext

@ferrariofilippo
There is a structural problem with this context. The actions will have to test 2 contexts each time. For example, Properties can apply to the parent folder, selection, or sidebar. The action does not need to know where the item is. We could further simplify the contexts.
ContentPageContext can return active items, be it selection, parent folder or sidebar. Most actions will no longer need to double check as currently (if HasSelection use this else use that). It will take an enum to know where the applied element is located (for example, one can only rename a selected element). But this will simplify the code most of the time.
What do you think ?

The actions will have to test 2 contexts each time.

Actually it would be three, we would have also WidgetsContext

ContentPageContext can return active items, be it selection, parent folder or sidebar. Most actions will no longer need to double check as currently (if HasSelection use this else use that). It will take an enum to know where the applied element is located (for example, one can only rename a selected element). But this will simplify the code most of the time.
What do you think ?

Yes, that seems reasonable. As I mentioned earlier, we should also merge WidgetsContext into our main context. Should I start working on this or would you rather do it on your own?
I'll mark this and #11657 as Drafts so that you can easily access them if you need to. Once the final PR is merged, I'll close them

@ferrariofilippo The problem that these elements are of different classes. Let's think about the best solution before intervening.

We can access Item property of the widget which is a INavigationControlItem

@ferrariofilippo @yaira2 This solution will not work because it has a flaw. For example Properties is disabled on the homepage. If when we open the context menu of a favorite, the action updates according to this element, the toolbar button will be activated. This behavior is not desirable. For Paste, we used 2 commands, but this complicates the commands in the palette.
A solution may be to use temporary commands for some menus. The menu creation code instantiates an action with a parameter (a specific unobservable class). This class is not visible in the palette. Then, we use CommandManager to get an instance of IRichCommand from this action. It may be a simpler implementation as no need to update it. The menu uses it and then throws it away.
These actions won't need any context, everything will be passed in the constructor. As these are rich commands, we can use the builder to generate the menu.

I have a better idea. I will implement it for the tab menu, it will be more understandable.

What's the latest on this?

I'll close this as soon as @cinqmilleans ends their work

I have thought a lot about this problem.  I think I have a practical solution.  I will open an issue to validate it before implementation.

@cinqmilleans what's the latest on this?


Add Zephyr post on Automation API

Description
This PR adds an Automation API-focused post to the Zephyr IaC Recommended Practices series.
Closes https://github.com/pulumi/devrel-team/issues/537
Still TBD (but not blocking reviews and feedback):

[x] Add links to specific sections of the Automation API program on GitHub when referenced in the text

Checklist:

[x] I have reviewed the style guide.
[x] If blogging, I have reviewed the blogging guide.
[x] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.
[ ] If making css changes, I rebuilt the bundle.

@cnunciato @usrbinkat I believe I've incorporated all your feedback and will be merging this tomorrow AM. Feel free to give it a final review, and I'll do my best to address any last-minute comments.


feat: option twoslash to disable twoslash

Resolve https://github.com/slidevjs/slidev/issues/1428

Maybe the docs should be updated too.


Make global-reference-to-import faster

Based on a facile profile of transforms, global-reference-to-import accounts for ~60% of esify's runtime.
Most likely, this is caused by continually searching the codebase with ag/ack.  Adding a cache may significantly speed this up.

Timing based on a full transform of shopify/shopify/**/*.coffee, with ag.

Transform
Time (minutes)

shopify-codemod/global-reference-to-import
11.65

js-codemod/no-vars
1.54

shopify-codemod/mocha-context-to-closure
0.35

shopify-codemod/global-identifier-to-import
0.34

shopify-codemod/coffeescript-soak-to-condition
0.32

shopify-codemod/function-to-arrow
0.32

shopify-codemod/remove-useless-return-from-test
0.31

shopify-codemod/ternary-statement-to-if-statement
0.31

js-codemod/unquote-properties
0.27

shopify-codemod/coffeescript-range-output-to-helper
0.27

shopify-codemod/conditional-assign-to-if-statement
0.27

shopify-codemod/remove-addeventlistener-returns
0.27

shopify-codemod/constant-function-expression-to-statement
0.26

js-codemod/template-literals
0.25

shopify-codemod/strip-template-literal-parenthesis
0.25

shopify-codemod/remove-trailing-else-undefined-return
0.25

js-codemod/object-shorthand
0.25

shopify-codemod/remove-empty-returns
0.24

shopify-codemod/mocha-context-to-global-reference
0.17

shopify-codemod/global-assignment-to-default-export
0.16

js-codemod/arrow-function
0.14

shopify-codemod/assert/call-count-equals-to-assert-called
0.14

shopify-codemod/assert/called-method-to-assert-called
0.13

shopify-codemod/assert/move-literals-to-expected-argument
0.12

shopify-codemod/assert/called-with-methods-to-assert-called-with
0.12

shopify-codemod/assert/negated-assert-ok-to-assert-not-ok
0.12

shopify-codemod/assert/assert-to-assert-ok
0.12

shopify-codemod/assert/assert-false-to-assert-fail
0.12

shopify-codemod/assert/called-equals-boolean-to-assert-called
0.12

shopify-codemod/assert/equality-comparisons-to-assertions
0.12

shopify-codemod/assert/falsy-called-method-to-assert-not-called
0.11

Honestly, I thought about this and think we shouldn't care. It's a one time transform, and even if we do it 500 times it won't make sense to spend 2 hours trying to make it faster

I've experienced the full transform many times over, and it makes sense to make it faster.  Sadly, our transforms aren't perfect, and regularly 🎩 the full output uncovers serious bugs.

I see. It could be made faster by doing a single pass over all the files and keeping track of what every file contains, but unless we do that in parallel it's not going to be faster, and tuning it might be quite an undertaking


Usage returning "Internal Server Error".  (5m billing)

My script to fetch usage data is returning an "Internal Server Error" today.
https://api.amber.com.au/v1/sites/XXXXX/usage?startDate=2024-11-10&endDate=2024-12-09' -H 'accept: application/json' -H 'Authorization: Bearer psk_XXXXX'
I'm on the 5m beta, but this api call has been working ok with mixed 30m and 5m data.

You're hitting an internal payload limit...
Short term: don't pull a full month's worth of data.
Longer term: We'll setup paging.

Ah, so it's the volume of data this time.   More datapoints in the same timeframe has now exceeded some byte count.
I'll reduce the number of days.
Thanks.

We're going to clamp this and the price API to 7-days at a time.

Ah, so it's the volume of data this time. More datapoints in the same timeframe has now exceeded some byte count.
I'll reduce the number of days.
Thanks.

Yeah, as the days since your 5-minute cutover is getting bigger, the amount of data required to fulfil the request has gotten bigger.
Why are you fetching such large data ranges continuously?

I also like to fetch older data because that data might get corrected, e.g. initially missing and populated with filler data.

Keep in mind that some may want to fetch this data at irregular times and need to do a bulk catchup.
I sometimes go more than a week between calls, and want to catch every day I've missed.

You can do that by adjusting the startDate and endDate - it just means you'll need to make multiple queries to get the same amount of data.

Yup.    1 month used to work, so that's what I went with to catch as many data updates as possible.


Add a prelude

Closed by commit https://github.com/OxidizedLoop/HashMapSettings/commit/11b0520ad15ed09f298782ee70d2a3de5f3c3ea2


Added popToTop functionality

Hi,
In my app there are a few tabs, and the state is persisted if you toggle between them (thanks to PR #390). If you are already inside the tab however, I would like the state to be reset to the base of the tab for easy navigation.
I've added an action called popToTop (name based on https://facebook.github.io/react-native/docs/navigator.html#content).
I can now do this in my reducer:
const reducerCreate = params=> {
  const defaultReducer = Reducer(params);
  return (state, action)=>{
    if (state) {
      let currentTabIndex = state.children[0].index;
      if (state.children[0].children[currentTabIndex].name === action.key && action.type === 'jump') {
        return defaultReducer(state, {...action, ...{type:'popToTop'}});
      }
    }
    return defaultReducer(state, action);
  }
};

I figured I'd do a pull to see if you'd like to add it to the repo. I couldnt find where in the docs it would be added but if you can point me to it I can add it there too.
Regards

@AlexDM0 I tried that solution but it doesn't work 100%. I have to click on the tab icon twice in order for the page to show. I'm not really good with reducers, do I need to kick off two actions? One for the parent tab and one for the child?

Hi,
I explicitly wanted it to be done at the second tap. The first tap is just to switch.
Regards

On 06 Apr 2016, at 17:05, rgovindji<EMAIL_ADDRESS>wrote:
@AlexDM0 I tried that solution but it doesn't work 100%. I have to click on the tab icon twice in order for the page to show. I'm not really good with reducers, do I need to kick off two actions? One for the parent tab and one for the child?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Gotcha, instead i just reset the child component when I navigated away to another page. Thanks for the help!


Right limit label position error

Steps to reproduce

Display slider with visible limits
Resize browser window to larger area (larger width)

Expected behaviour
Right limit label should be always rendererd at the right end of the slider.
Actual behaviour
If you resize browser window to larger area (larger width) right limit label is not renered at the right end of
the slider.

Thanks for reporting. I'm gonna fix it right now! Sorry for the inconvenience.

Fix released under 5.8.2


Stream resize error

Hi! There is an issue with frame fetching from a youtube video when using different resolutions.
Code to reproduce:
from vidgear.gears.helper import dimensions_to_resolutions
from vidgear.gears import VideoGear

import pafy
import cv2

source = "https://youtu.be/j1GLs_fMn1s"

video = pafy.new(source)
valid_streams = dimensions_to_resolutions(
    [stream.resolution for stream in video.allstreams]
)   # ['144p', '240p', '360p', '480p', '720p', '1080p']

options = {
    "STREAM_RESOLUTION": valid_streams[1],  # '240p'
    "STREAM_PARAMS": {"nocheckcertificate": True}
}

stream = VideoGear(
    source=source,
    stream_mode=True,
    **options
).start()

while True:
    frame = stream.read()

    if frame is None:
        break

    cv2.imwrite('frame.jpg', frame)

    break

stream.stop()

Fetched image w/ 240p resolution (happens with  480p as well):

Fetched image w/ 144p resolution:

Logs
10:17:21 :: CamGear :: INFO :: Using Youtube-dl Backend
[ WARN:0] global /opt/opencv-4.5.1/modules/videoio/src/cap_gstreamer.cpp (924) open OpenCV | GStreamer warning: unable to query duration of stream
[ WARN:0] global /opt/opencv-4.5.1/modules/videoio/src/cap_gstreamer.cpp (961) open OpenCV | GStreamer warning: Cannot query video position: status=1, value=449, duration=-1
10:17:22 :: CamGear :: DEBUG :: Setting backend `1800` for this source.

@ioangatop Yes bug is reproducible. But bug is with OpenCV's GSTREAMER backend and not vidgear. You can test this by:

First clone vidgear:

  # clone the repository and get inside
  git clone https://github.com/abhiTronix/vidgear.git && cd vidgear

Then, commenting this line, so as to disable enforce cv2.CAP_GSTREAMER backend.

https://github.com/abhiTronix/vidgear/blob/eb19b9ce96061c09b5fe4c3ee63d84c940a19af2/vidgear/gears/camgear.py#L136

Then install vidgear as:

  # install normally
  pip install .

Running your code again with CAP_FFMPEG backend:

from vidgear.gears.helper import dimensions_to_resolutions
from vidgear.gears import VideoGear

import pafy
import cv2

source = "https://youtu.be/j1GLs_fMn1s"

video = pafy.new(source)
valid_streams = dimensions_to_resolutions(
    [stream.resolution for stream in video.allstreams]
)  # ['144p', '240p', '360p', '480p', '720p', '1080p']

options = {
    "STREAM_RESOLUTION": valid_streams[1],  # '240p'
    "STREAM_PARAMS": {"nocheckcertificate": True},
}

stream = VideoGear(
    source=source, stream_mode=True, backend="CAP_FFMPEG", **options
).start()

while True:
    frame = stream.read()

    if frame is None:
        break

    cv2.imwrite("frame.jpg", frame)

    break

stream.stop()

@ioangatop Can you confirm your OpenCV version by excuting this command:
python3 -c "import cv2; print(cv2.__version__)"

@abhiTronix thanks for the quick response. My OpenCV version is 4.5.1.

@abhiTronix thanks for the quick response. My OpenCV version is 4.5.1.

@ioangatop Thanks, I'm looking into OpenCV issues for clue.

I guess there isn't an easier way to change / choose the backend, right?

I can make that simpler, but there's a bug with FFmpeg, which made me to enforce Gstreamer backend in the first place.

@ioangatop OpenCV is crappy in doing VideoIO even with backends. I'm thinking to switch FFmpeg decoding completely with issue #148. But unfortunately, It will going to happen next year due to busy 2021.
I think the best resolution to this problem for now is to  use 360p or 720p and then reduce size manually. You can use reducer() for that purpose.

I'm closing this issue for now. I'll look into it in later versions.

@abhiTronix thanks! I believe that worse and best in general don't have a problem, but it's good to have an option to fetch frames with a specific resolution, as it is a nice trade-off of speed and quality.
I'm not sure but I hope that in general, only these resolutions have problems, and its not for this specific stream (meaning if I put another stream url, 480p will act properly but not the 720p)

@ioangatop I'm thinking this is to do with GStreamer https://github.com/opencv/opencv/issues/19025#issuecomment-740183729. As we can see errors in logs:
[ WARN:0] global /home/abhishek/Downloads/opencv-master/modules/videoio/src/cap_gstreamer.cpp (898) open OpenCV | GStreamer warning: unable to query duration of stream
[ WARN:0] global /home/abhishek/Downloads/opencv-master/modules/videoio/src/cap_gstreamer.cpp (935) open OpenCV | GStreamer warning: Cannot query video position: status=1, value=450, duration=-1

and its not for this specific stream (meaning if I put another stream url, 480p will act properly but not the 720p)

@ioangatop This errors is only for Youtube livestreams and not general videos. General videos will works out of the box for any resolution.

@ioangatop I think #274 will resolve this issue too, can you check it as follows:
Clone and Install development branch:
  # clone the repository and get inside
  git clone https://github.com/abhiTronix/vidgear.git && cd vidgear

  # checkout the latest testing branch
  git checkout development

  # install normally
  pip install .[core]

  # OR install with asyncio support
  pip install .[asyncio]

Test your code:
from vidgear.gears.helper import dimensions_to_resolutions
from vidgear.gears import VideoGear

import pafy
import cv2

source = "https://youtu.be/j1GLs_fMn1s"

options = {
    "STREAM_RESOLUTION": "240p",
    "STREAM_PARAMS": {"nocheckcertificate": True},
}

stream = VideoGear(
    source=source, stream_mode=True, logging=True, **options
).start()

while True:
    frame = stream.read()

    if frame is None:
        break

    cv2.imwrite("frame.jpg", frame)

    break

stream.stop()

Successfully resolved and merged in commit: https://github.com/abhiTronix/vidgear/commit/dc26c00bcb4ec6e40645b4ad1aa1aedf4f0c6b43
Usage docs are available at: https://abhitronix.github.io/vidgear/v0.2.4-dev/gears/camgear/usage/#using-camgear-with-streaming-websites


Support CTAS for Hive transactional tables v2

Squashed and rebased version https://github.com/prestosql/presto/pull/4516

CI green except UI checks which didn't start at all.


Update to fix typo with property bag keys

Fixed typo based on issue 1296.

Hi @patrick-rodgers, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


#9 code refactoring for PSR-2 Code Standart

If you would like to merge it please add #hacktoberfest tag

@iGusev Would you mind fixing the merge conflicts? I would like to avoid using of tools like phpcs for now in lieu of manual updates since this project still going through changes. Thanks

Changes Unknown when pulling 881a7fa5039d7ae9149297d3a3a7b89ff8bf35f9 on iGusev:issue-9 into ** on jabranr:master**.

Changes Unknown when pulling 881a7fa5039d7ae9149297d3a3a7b89ff8bf35f9 on iGusev:issue-9 into ** on jabranr:master**.