Introduce ObjectHasher

Codecov Report

Merging #316 into master will decrease coverage by 0.64%.
The diff coverage is 5.46%.

@@             Coverage Diff              @@
##             master     #316      +/-   ##
============================================
- Coverage     46.57%   45.92%   -0.65%
  Complexity     2535     2535
============================================
  Files           615      616       +1
  Lines         16219    16443     +224
  Branches       1202     1205       +3
============================================
- Hits           7554     7552       -2
- Misses         8214     8440     +226
  Partials        451      451

Impacted Files
Coverage Δ
Complexity Δ

...eroic/aggregation/simple/FilterKThresholdType.java
100% <ø> (ø)
8 <0> (ø)
:arrow_down:

...potify/heroic/aggregation/AggregationInstance.java
100% <ø> (ø)
4 <0> (ø)
:arrow_down:

...rc/main/java/com/spotify/heroic/filter/Filter.java
100% <ø> (ø)
8 <0> (ø)
:arrow_down:

...m/spotify/heroic/aggregation/CollapseInstance.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...heroic/aggregation/simple/PointsBelowInstance.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

.../java/com/spotify/heroic/common/OptionalLimit.java
55.55% <ø> (ø)
2 <0> (ø)
:arrow_down:

.../com/spotify/heroic/common/ValueOptionalLimit.java
43.58% <0%> (-4.99%)
14 <0> (ø)

...com/spotify/heroic/aggregation/BucketStrategy.java
58.53% <0%> (-6.33%)
1 <0> (ø)

...main/java/com/spotify/heroic/common/DateRange.java
38.29% <0%> (-4.56%)
14 <0> (ø)

...in/java/com/spotify/heroic/filter/FalseFilter.java
33.33% <0%> (-5.13%)
5 <0> (ø)

... and 35 more

Continue to review full report at Codecov.

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


The method setSvgResource() is no use

when I invoke this method setSvgResource() to change the resourse of SVG, the SVG animator is still the playing as the old one

That is true , should be easy fix stopping the animator every time we change the SVG.
Unfortently I don't have time to do it. If you are interested in contributing , you can make a PR for it.


commiting chandana.js

Great work on trying out PRs. In the near future, you'll work on actual code changes, so we'll review it more closely.


Coong/feature/settings/language selection #1

This PR address one of task in #1
Changes including:

Add Language Selection in Settings Dialog.
Handle translation with react-18next framework.

Note: We put translation texts into separate json files in public folder and load them via the Backend plugin of i18next . This approach has a few advantages over including translation text inside the source code itself which is reduce bundle size on production build and only need to load one translation file for the target language at a time.
To add new language. just:

Add the new language code in Language.
Create translation file for that language and store it at pulic/locales/{languageCode}/translation.json.
Update the LanguageSelection component with new language.

We also need to wrap the translate function t<string>() around the e.message when we showing error messages to users via toast API since the e.message could be an ErrorCode as well.
toast.error(t<string>(e.message));


Fix auto merge conflict 11317 [skip ci]

Resolve auto-merge conflict at https://github.com/NVIDIA/spark-rapids/pull/11317.
Keep dependency version 24.10.0-SNAPSHOT in pom.xml
NOTE: this must be merged as create a merge commit

build


Benchmark

Hey,
Great job!
Do you have benchmark comparing adxe with cocos2dx 4.0? (There are some performance issues in cocos2dx 4.0 and android).
How much APK size increased comparing to cocos2dx 4.0?

APK sitze can find in the Wiki page.

A little better: Left is cocos2d-x-4.0, right is adxe

adxe can be faster: look on the windows title (its debug modus) ;)


Add a "weapon sound" chooser to equipment

Some other systems have a way to select a sound that is played with an attack. It would be cool if equipment had this option, so a mace sound could play when attacking with a mace, etc.

This can be done by the maestro module if we provide an attribute in chat cards with the item id.

This can be done by the maestro module if we provide an attribute in chat cards with the item id.

This is covered by #329


How do one use your Facebook connector in data studio

Im having a hard time finding documentation on how to actually use a custom buildt community connector, like your Facebook connector in datastudio? Can you point me in the right direction?

Good night, I developing a Facebook connection plugins with Gogle data studio, and I gave myself a problem, could you help me?


Docs for DropdownMenu component don't specify rn-primatives/portal dependency

Something that tripped me up was that the DropdownMenu component depends on @rn-primatives/portal. This could be documented :)

Opened a PR here: https://github.com/unisol1020/nativecn/pull/2
The repo is a bit light on how to contribute, so I just put forth my best effort. Please let me know if there are any changes required.


healthcheck ports messing with docker and "host" networking mode

With HOST networking mode, dynamic mapped ports are returned by mesos like in BRIDGE mode, then marathon use the portIndex of this array as the healthcheck port.
As the container doesn't have any IP address in HOST networking mode, I guess marathon should use the ports[portIndex] from the config instead.
Tested with head marathon + #2231 / mesos 0.24.1

As explained in #2336, adding "requirePorts": true fix this issue.


⚠ Bofrost.de has degraded performance

In a9ed979, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1276 ms

Resolved: Bofrost.de performance has improved in 8ef0797 after 14 minutes.


How to Publish DLLs with their Dependencies for use with this library?

So, this isn't a feature request for the library code itself, but for the documentation.
I love this library. It's amazing, but I've hit a snag.
When trying to create a DLL that has nuget dependencies, dotnet offers no obvious way to publish the DLL and aggregate all of its dependency DLLs.  This makes it a pain-in-the-but to deploy hot-loadable DLLs for use with the plugin.  I found an SO question mentioning the problem here:
https://stackoverflow.com/questions/51042647/publish-net-standard-library-with-all-its-dependencies
One of the commentor's says his solution:
"I have a very dirty way to do it, create a .net core application, reference the library and take the results except the exe, but it's totally ugly, Not sure why they did this without an option to "publish" the results."
The solution works.  But it was not obvious to me, and I bet plenty of people have run into this.  Maybe something in the basic usage documentation  explaining a technique for building DLLs for use with this lib would be helpful.  Even if it just said "Make a .net core app, reference the library, publish it."

So, having fought with this for a bit, I came up with a more elegant solution.  I tested this approach and deployed a full build with a plugin with many external dependencies to Ubuntu 18.04 and it ran perfectly.
Create a new project of type Console Application instead of Class Library.  Put all your library code files into the Console Application and all all the dependencies.  Get rid of the Class Library project (you don't need it anymore).  Finally, publish the Console Application. You will get a DLL with all of the dependencies.  You can use this DLL like any other DLL.
Elegant solution.  I suggest naming the console app project with "Library" on the end of it and adding a README just to document its not really an application even though the project is configured to build as one.

I thought the README and samples covered this already....did you try using dotnet publish?

The problem is that "dotnet publish" on a DLL does NOT publish any of the dependencies. The only thing it produces  is a .nuget package.  Its very unintuitive behavior.  If you want to publish a DLL and get all of its dependencies, you actually have to turn the DLL into an application and publish the application -or- reference the DLL in an application and publish that.  Its a very clunky dotnet design that cost me many hours of troubleshooting.  The SO link I referenced talks about this.
Its clearly a dotnet issue -- not a problem with the DotNetCorePlugins library at all. It just seems likely that a lot of people are probably running into this snag.

Are you talking about dotnet pack? The pack command will produce a nuget package, not the publish command. You should be able to dotnet publish a class library and get its dependencies just fine:
root@1eac45b3a9fa:/mylib# dotnet new classlib
The template "Class Library" was created successfully.

Processing post-creation actions...
Running 'dotnet restore' on /mylib/mylib.csproj...
  Determining projects to restore...
  Restored /mylib/mylib.csproj (in 63 ms).
Restore succeeded.

root@1eac45b3a9fa:/mylib# dotnet add package Newtonsoft.Json
  Determining projects to restore...
  Writing /tmp/tmpxO19lq.tmp
info : Adding PackageReference for package 'Newtonsoft.Json' into project '/mylib/mylib.csproj'.
info :   CACHE https://api.nuget.org/v3/registration5-gz-semver2/newtonsoft.json/index.json
info : Restoring packages for /mylib/mylib.csproj...
info : Package 'Newtonsoft.Json' is compatible with all the specified frameworks in project '/mylib/mylib.csproj'.
info : PackageReference for package 'Newtonsoft.Json' version '13.0.1' added to file '/mylib/mylib.csproj'.
info : Committing restore...
info : Writing assets file to disk. Path: /mylib/obj/project.assets.json
log  : Restored /mylib/mylib.csproj (in 76 ms).
root@1eac45b3a9fa:/mylib# dotnet publish
Microsoft (R) Build Engine version 17.0.0+c9eb9dd64 for .NET
Copyright (C) Microsoft Corporation. All rights reserved.

  Determining projects to restore...
  All projects are up-to-date for restore.
  mylib -> /mylib/bin/Debug/net6.0/mylib.dll
  mylib -> /mylib/bin/Debug/net6.0/publish/
root@1eac45b3a9fa:/mylib# ls -la ./bin/Debug/net6.0/publish/
total 708
drwxr-xr-x 2 root root   4096 Nov 15 20:05 .
drwxr-xr-x 4 root root   4096 Nov 15 20:05 ..
-rwxr--r-- 1 root root 695336 Mar 17  2021 Newtonsoft.Json.dll
-rw-r--r-- 1 root root    987 Nov 15 20:05 mylib.deps.json
-rw-r--r-- 1 root root   3584 Nov 15 20:05 mylib.dll
-rw-r--r-- 1 root root  10124 Nov 15 20:05 mylib.pdb
root@1eac45b3a9fa:/mylib#

For plugins, I've always used <CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies> in the csproj file.  I decided to use this instead of publish, as I can also use debug to launch the host application for debugging and testing.


Feature - Implemented pulling crates.io packages

Closes #23

@naveensrinivasan just a heads up, we'll want to remove the crates binary from the PR before merging.

@naveensrinivasan just a heads up, we'll want to remove the crates binary from the PR before merging.

@jordan-wright Thanks. 🤦♂

@naveensrinivasan just a heads up, we'll want to remove the crates binary from the PR before merging.

@jordan-wright Thanks. 🤦♂

Removed the crates binary


Crash with WAILA

When watching the bits of a tree failing it crashes the game with WAILA. Crash Log

Fixed by 191819fcf760ce672097a968a8cc9c253b1bb7b9


Environment setup questions

I'm having some issues trying to setup the environment to run the code so I was hoping someone could answer the following questions.

What version of python is supported?

Is this code tested for Windows?

Does this code run on a machine that doesn't have a GPU?

I'm using python 3.5 in a Linux machine and when I do 'pip install -r requirements.txt' I'm getting the following error:
Cannot uninstall 'certifi'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.

I did a workaround to uninstall certifi, but even after that I got the following error when trying to run scripts/run_beer_1.sh:

Apparently the issue with 5 was incompatibility between tensorflow and cuda. Installing tensorflow-gpu==1.12.0 through conda fixed the issue.


Serverpod version: 1.1.0 - Serializable exceptions is not catch in client side

Describe the bug
Serializable exceptions not catch in client side .
All steps are taken from the documentation  exceptions
To Reproduce
Steps to reproduce the behavior:

Create 'not_found.yaml' file in protocol folder:

exception: NotFound
fields:
  postId: int
  errorMessage: String

Run serverpod generate

Implement throw NotFound in server side:

Call test method in client Side:

Result:
"ServerpodException: Internal server error"

Thanks!

Indeed, this looks like a bug, not sure why this happens right now. Seems to be something wrong with the code generation. We will take a look at this soon.
Out of curiosity any reason for duplicating the auth module rather than using the one hosted on pub.dev?

Hi @Isakdl, Thanks for response!

Out of curiosity any reason for duplicating the auth module rather than using the one hosted on pub.dev?

Simply put, I prefer to have full control over all the code in my project. By duplicating the auth module, I have the flexibility to customize and modify it according to my specific requirements. Although there may be an existing auth module available on pub.dev, using my own duplicate version gives me the freedom to make changes and enhancements as needed without relying on external dependencies. It allows me to have a deeper understanding of the authentication logic and maintain full control over the codebase
:)
Thanks!

This bug is fixed with https://github.com/serverpod/serverpod/pull/935
For the next version 1.1.1 this issue should be fixed.

@Isakdl Thanks!!!


Release0.0.4 (for real)

Updating changelog and preparing for release on crate.io.

For contributor use:

[ ] Unit tests written
[ ] Added test to CI if applicable
[ ] Updated CHANGELOG_PENDING.md
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #319 into master will increase coverage by 23.8%.
The diff coverage is 61.3%.

@@            Coverage Diff            @@
##           master    #319      +/-   ##
=========================================
+ Coverage    13.6%   37.5%   +23.8%
=========================================
  Files          69     124      +55
  Lines        3752    7989    +4237
  Branches     1374    2783    +1409
=========================================
+ Hits          513    2997    +2484
- Misses       2618    4753    +2135
+ Partials      621     239     -382

Impacted Files
Coverage Δ

modules/src/events.rs
0.0% <0.0%> (ø)

modules/src/ics02_client/events.rs
0.0% <ø> (ø)

modules/src/ics02_client/raw.rs
0.0% <0.0%> (ø)

modules/src/ics03_connection/context.rs
0.0% <0.0%> (ø)

modules/src/ics03_connection/error.rs
15.0% <0.0%> (-18.4%)
:arrow_down:

modules/src/ics04_channel/error.rs
75.0% <ø> (+50.0%)
:arrow_up:

modules/src/ics04_channel/packet.rs
0.0% <0.0%> (ø)

modules/src/ics07_tendermint/client_def.rs
0.0% <0.0%> (ø)

modules/src/ics07_tendermint/error.rs
75.0% <ø> (+75.0%)
:arrow_up:

modules/src/ics18_relayer/error.rs
0.0% <0.0%> (ø)

... and 226 more

Continue to review full report at Codecov.

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


RuntimeError: CUDA error (10): invalid device ordinal

when i train on my GTX1060(ONLY 1 GPU),I get this problem,i konw maybe that is a gpus number problem,but how to fix this problem,please help me and i would preciate that.
/home/vincent/venv/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
return f(*args, **kwds)
Traceback (most recent call last):
File "main.py", line 29, in
p = Processor(sys.argv[2:])
File "/home/vincent/st-gcn/processor/processor.py", line 29, in init
self.init_environment()
File "/home/vincent/st-gcn/processor/processor.py", line 38, in init_environment
super().init_environment()
File "/home/vincent/st-gcn/processor/io.py", line 62, in init_environment
torchlight.occupy_gpu(gpus)
File "/home/vincent/venv/lib/python3.6/site-packages/torchlight-1.0-py3.6.egg/torchlight/gpu.py", line 35, in occupy_gpu
RuntimeError: CUDA error (10): invalid device ordinal

that is a easy problem but still take my some time to fix this ,you can just set your gpus to do this ,
just like this : -- device 0

@Vincent630 I am also facing the same error. Can you please tell what changes did you do to resolve it? Where do you write --device 0?

@Vincent630 I am also facing the same error. Can you please tell what changes did you do to resolve it? Where do you write --device 0?

you can appoint gpus by --device, for example: python main.py recognition -c config/st_gcn/kinetics-skeleton/train.yaml --device 0


presentaion-api article is shown in the wrong sidebar section

https://developers.google.com/web/updates/2015/11/presentation-api should be in November's archive but is shown in December's archive in the left sidebar.

After checking auto-generated 2015/_toc.yaml, I see a lot of files that seems to be in a different archive than the expected archive based on the month in the URLs. Maybe, It's an intended behavior or there's an issue with the build system!

It doesnt have the correct meta data - https://github.com/google/WebFundamentals/blame/master/src/content/en/updates/2015/11/presentation-api.md#L6
Not sure it's worth fixing now as it will likely lead to RSS feed changes etc.

Could I close this or you want it to be open in case you decide to fix it?


Plumber busts task out of gulp.watch

When using gulp.watch, here’s what happens:
$ gulp watch
[11:36:53] Starting 'scripts'...
[11:36:53] Finished 'scripts' after 197 ms
[11:36:53] Starting 'styles'...
[11:36:53] Plumber found unhandled error:
 Error in plugin 'gulp-less'
Message:
    Unrecognised input. Possibly missing something in ...
Details:
    ...
# Changed some files
[11:37:15] Starting 'scripts'...
[11:37:15] Finished 'scripts' after 179 ms
# Changed some files again
[11:39:10] Starting 'scripts'...
[11:39:10] Finished 'scripts' after 179 ms

After an error has been caught in scripts, the task is never resumed. Is this the intended behavior?

Hard to say with out gulpfile to reproduce.

gulp.watch issues belongs here - https://github.com/gulpjs/gulp/issues

I thought it’s a problem with the way plumber handles these errors.
Anyway, thanks for the link – cheers!

@tomekwi oops, sorry, I thought that issue were in gulp-watch repo. Could you provide gulpfile to figure this out?

@floatdrop I followed your suggestion and tried to replace gulp.watch with gulp-watch – regarded as more robust. But the issue remains.
Our gulpfile is quite complex, but I remembering encountering the same with simple builds. I’ll try to reproduce the thing with a simpler build and post it here – I’ll be right back!

Hey, I’ve tried to reproduce the error, even written a more complex gulpfile for that: https://gist.github.com/tomekwi/0d1818c89b6453917a6c . But it doesn’t look bad after all.
My original project definitely has the issue, and it has been there for a long time already. But it looks like it lies somewhere else.
Sorry for the false alarm.

This should probably be reopened; specifically, it seems the issue is caused by this. A change to the default error handler should fix this, I think.

@floatdrop I'm having the same problem. The article linked by @joepie91 fixed it -- could this be added to gulp plumber?


Styling tooltips on surveyjs questions

Are you requesting a feature, reporting a bug or ask a question?
Question/ feature request
What is the current behavior?
Tooltips achieved on survey questions using onAfterRenderQuestion event.
What is the expected behavior?
How to create styles to those tooltips on survey questions
How would you reproduce the current behavior (if this is a bug)?
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
your_code_here

Specify your

browser:
browser version:
surveyjs platform (angular or react or jquery or knockout or vue):
surveyjs version:

@ashygk What do you mean? It is a custom html code, just put any css or change as you wish for you app.
Thank you,
Andrew

i do not see any demo for tooltips using custom html code

Here the working example: https://plnkr.co/edit/0vXbRM?p=preview based on this example: https://surveyjs.io/Examples/Library/?id=survey-afterrender&platform=jQuery&theme=default


Running Join-AzStorageAccount Permissions

In order for this to work properly in many customer environments (all that I've seen), the user executing the Join-AzStorageAccount command needs to be a domain admin. I think this would be helpful to add to the docs as the error doesn't clearly indicate it is a permissions issue.

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

ID: 7a1a9f3d-5733-012a-a0d7-c00ec21b9232
Version Independent ID: 1e70ab95-9488-5aa3-b8ef-e58b3e0067bb
Content: Enable AD DS authentication to Azure file shares
Content Source: articles/storage/files/storage-files-identity-ad-ds-enable.md
Service: storage
Sub-service: files
GitHub Login: @khdownie
Microsoft Alias: kendownie

@denver-reed
Thanks for your feedback! We will investigate and update as appropriate.

@denver-reed
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.
@khdownie
Please review .

@denver-reed we provide the following guidance in the doc. Do you think more clarification is needed?
Install and execute the module on a device that is domain joined to on-premises AD DS with AD DS credentials that have permissions to create a service logon account or a computer account in the target AD. You must run the script using an on-premises AD DS credential that is synced to your Azure AD. The on-premises AD DS credential must have either Owner or Contributor Azure role on the storage account.

@khdownie Perhaps further testing can be done on your lab environment for this. In the customer environments I have assisted with, the command only works if they are a Domain Admin of the target domain. When the customer tried to run the command after confirming they had permissions to create a computer account, it failed. Only resolution we found was having a domain admin do it. If that's the case, I think we should change it to "...with domain administrator AD DS credentials."

@denver-reed thanks - I will look into this.

@denver-reed I check with devs and the user needs to use a credential which has permission to create a computer account. Domain admin has this permission.

Confirmed with devs that we have proper permissions documented.

#please-close


[C++] Drop gtest to 1.10 / Raise Arrow to 6.0.1

I had to add UseAsync back in, as its removal was based on 7.0.0, which is a touch premature as 7.0.0 is not in conda yet.  It was ok to be removed in 5.0.0 but in 6.0.1 we don't support non-async dataset writes and so the dataset write example was failing.

One bonus is that we can remove the download & build gtest step from the C++ build since conda has gtest 1.10 binaries.

Great, thank you!


If the DW can be applied to softmax classifier?

In the code,  DW use sigmoid classifier by default. And I apply it to logo detection(num of categories is 352）. However, I find that there are a lot of FP( same position, different category). I wonder if it is due the use of sigmoid classifier. How could change the DW to use softmax classifier?

The majority of current detectors use sigmoid-based classifiers.  Since the formulations of the two classifiers are different,  it is not direct to apply DW on softmax classifiers. I suggest you can use sigmoid classifiers.

But isn't the principle of the two the same? They both can give different weights to positive and negative samples.

The two almost have no difference in the final AP if no weighting scheme is used. Suppose the score and logit of a positive anchor with label j is s_j and z_j,  s_j has an influence on the gradient with respect to z_j in both classifiers and s_j has (no) influence on the gradient with respect to z_k(k!=j) in softmax(sigmoid) classifiers. When we change the weight loss of a positive anchor in softmax, the gradient with respect to all classes will be changed which is not what we want in DW.  Sigmoid-based classifier is more flexible to do sample weighting in a finer level.

But isn't the principle of the two the same? They both can give different weights to positive and negative samples.

The two almost have no difference in the final AP if no weighting scheme is used. Suppose the score and logit of a positive anchor with label j is s_j and z_j, s_j has an influence on the gradient with respect to z_j in both classifiers and s_j has (no) influence on the gradient with respect to z_k(k!=j) in softmax(sigmoid) classifiers. When we change the weight loss of a positive anchor in softmax, the gradient with respect to all classes will be changed which is not what we want in DW. Sigmoid-based classifier is more flexible to do sample weighting in a finer level.

Thanks! So do you have any suggestion to remove these occurrences of FP(same postition, different categories)？

Sorry, I'm little confused about the caculation process of cls Loss of DW, can you give a specific example？For example, if a positive anchor's predicted cls score is [0.3,0.5], the IoU is 0.6, what's the cls loss?


when i send a image from cameraStream to native(java),the app will Lost connection to device

Steps to Reproduce
I develop with dart&java, when i send a image Data from camera stream, to native(java),the app will stop, actually it's dark screen and pause there. And the log show "Lost connection to device". it's happend at  camera ResolutionPreset is medium and high . low it's working but cant use because image it's too small and low quailty.
at last thanks for help.  :)
Logs
  E/flutter ( 6983): [ERROR:flutter/lib/ui/ui_dart_state.cc(148)] Unhandled Exception: CameraException(No camera is streaming images, stopImageStream was called when no camera is streaming images.)
E/flutter ( 6983): #0      CameraController.stopImageStream (package:camera/camera.dart:397:7)
E/flutter ( 6983): <asynchronous suspension>
E/flutter ( 6983): #1      _MyHomePageState.onCameraSelected.<anonymous closure>.<anonymous closure> (package:flutter_decode_qr_demo_1/main.dart:127:24)
E/flutter ( 6983): #2      _rootRun (dart:async/zone.dart:1120:38)
E/flutter ( 6983): #3      _CustomZone.run (dart:async/zone.dart:1021:19)
E/flutter ( 6983): #4      _FutureListener.handleWhenComplete (dart:async/future_impl.dart:150:18)
E/flutter ( 6983): #5      Future._propagateToListeners.handleWhenCompleteCallback (dart:async/future_impl.dart:609:39)
E/flutter ( 6983): #6      Future._propagateToListeners (dart:async/future_impl.dart:665:37)
E/flutter ( 6983): #7      Future._complete (dart:async/future_impl.dart:473:7)
E/flutter ( 6983): #8      new Future.delayed.<anonymous closure> (dart:async/future.dart:313:16)
E/flutter ( 6983): #9      _rootRun (dart:async/zone.dart:1120:38)
E/flutter ( 6983): #10     _CustomZone.run (dart:async/zone.dart:1021:19)
E/flutter ( 6983): #11     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
E/flutter ( 6983): #12     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
E/flutter ( 6983): #13     _rootRun (dart:async/zone.dart:1124:13)
E/flutter ( 6983): #14     _CustomZone.run (dart:async/zone.dart:1021:19)
E/flutter ( 6983): #15     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
E/flutter ( 6983): #16     Timer._createTimer.<anonymous closure> (dart:async/runtime/libtimer_patch.dart:21:15)
E/flutter ( 6983): #17     _Timer._runTimers (dart:isolate/runtime/libtimer_impl.dart:382:19)
E/flutter ( 6983): #18     _Timer._handleMessage (dart:isolate/runtime/libtimer_impl.dart:416:5)
E/flutter ( 6983): #19     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)
E/flutter ( 6983):
Lost connection to device.

It's there any size limit of methodChannel for send data? :)

@bladeofgod Could you please check it with current stable channel and current camera version?

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of ' flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


ACM-2463  create mce-gather script to make it easy to sync scripts wi…

Signed-off-by: Erin Murphy<EMAIL_ADDRESS>issue:  https://issues.redhat.com/browse/ACM-2463

/lgtm


Release 095

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.13 out of 14 committers have signed the CLA.:white_check_mark: cxfeng1:white_check_mark: kfeagle:white_check_mark: acton393:white_check_mark: boboning:white_check_mark: sospartan:white_check_mark: zshshr:white_check_mark: Hanks10100:white_check_mark: atomtong:white_check_mark: xkli:white_check_mark: YorkShen:white_check_mark: yuanyan:white_check_mark: misakuo:white_check_mark: yangshengtao:x: lybeenyangshengtao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.


Move battery related parameters out of rmf_traffic::agv::Configuration

Feature request
Description
Parameters battery_system, motion_sink, ambient_sink and planner are specific to agents yet they are passed as general parameters to rmf_traffic::agv::Configuration. These are only used when generating ChargeBattery requests and hence it is appropriate that they are coupled with the respective agents.

Implementation considerations

Move these into a rmf_task::agv::Parameters class that gets bundled with State and Constraints.

Addressed by https://github.com/open-rmf/rmf_task/pull/15


[BUG]: Suggested terms should not be empty

Describe the bug
When suggested terms returns an empty list, do not display the suggested terms.
Screenshots

Closing for now, cannot reproduce


Font Awesome 5 support

Font Awesome 5 is available! Please update your library

hey
How can I update my pod to support FA 5 ?
Please help me.
Thanks.


Add enchanting jewelry to the idle notification plugin

Requested in discord, it uses animation 931.

Mind if I take a look at this one?


[6.0] Hide parameter and return value documentation from properties. (#968)

Explanation: This fixes a special case for hiding parameter and return value documentation where a C function that is represented by a property in Swift would display the parameter and return value documentation on the Swift page because the property doesn't have any "function signature" data.
Scope: Parameter and return value documentation for C functions that are refined as properties in Swift.
Issue: rdar://130750134
Risk: Low.
Testing: New tests verify that a Swift property representation of a C function hides the parameter and return value documentation. Manually tested with a few projects that reproduce this issue.
Reviewer: @anferbui
Original PR: #968

@swift-ci please test

@swift-ci please test


Fixed messageSerialization test in GenericMessageTest

messageSerialization test within GenericMessageTest.java can fail based on the order of keys in the JSON output. It was expecting a <{"key":"value","foo":"bar"}> but actually returned: <{"foo":"bar","key":"value"}>.
To fix this test, the comparison of the raw serialized strings was replaced by parsing the JSON and comparing the actual content of the serialized metadata with the use of a map. The HashMap of the expectedMetaData and the Map of the actualMetaData was made and then compared for equality, regardless of key order. A throws IOException was added to handle the deserialization process.
The change/addition was confirmed by running the NonDex tool.
To reproduce the problem, you can run the NonDex tool, using this command:
mvn -pl messaging edu.illinois:nondex-maven-plugin:2.1.7:nondex -Dtest=org.axonframework.messaging.GenericMessageTest#messageSerialization

Thank you for the suggestions! I have taken them in account and have made the changes within a new commit. Please let me know what you think.

Thanks again for providing another fix, @wtrzas2!

No problem, thank you @smcvb!


Readme update to test

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 5.412%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
16

Relevant Lines:
201

💛  - Coveralls


[c#] Support defining AliasConveters in any assembly

This will allow the user to create a BondTypeAliasConverters for types like System.TimeSpan once,
and use in any generated code (independent of the generated type namespace).
In my case, I want to distribute a library with common converters, which is currently not doable in C# (I can do this in C++)
The logic is simple: try to locate converter class in the same assembly of the target type, if not found, try to locate in any assembly.
related to issue #279

Thanks for your review.
As you pointed out, it might not work.
I'm sorry I didn't test my assumptions first...
I suppose I'll figure some other mechanism that will also adress your other concern regarding conflicting/racing definitions from different assemblies.
I'm closing for now and will open another PR  (with proper UTs this time) when I have time to rethink and implement.
Thanks again!

Sounds good & looking forward to it.


Fix Rocky Linux optimized name/family.

Goes with https://github.com/GoogleCloudPlatform/guest-test-infra/pull/497


[SPARK-36839][INFRA][FOLLOW-UP] Respect Hadoop version configured in scheduled GitHub Actions jobs

What changes were proposed in this pull request?
This is a followup of #34091 and https://github.com/apache/spark/pull/34217.
We should respect the default Hadoop 2.7 (not only in Hive's tests).
Why are the changes needed?
In order to run the daily job with Hadoop 2 for all tests.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
It will be tested once it's merged. It won't break any build in any event.

Test build #144049 has started for PR 34230 at commit 84f39a5.

The job configuration logic was tested in https://github.com/HyukjinKwon/spark/actions/runs/1322887874. Let me get this in as it's scheduled time soon, and we can test it together after being merged.
It won't affect anything in developement anyway.

Merged to master.

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48526/

Test build #144049 has finished for PR 34230 at commit 84f39a5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.


Error when running TorchRuntimeClient

Environment

Qiskit Machine Learning version: 0.4.0 and 0.5.0
Python version: 3.7.15
Operating system: Google Colab

What is happening?
When using TorchRuntimeClient, it fails with the following error (taken from the log of the job):
2022-11-16T07:37:37.332393231Z --- Logging error ---
2022-11-16T07:37:37.335383631Z Traceback (most recent call last):
2022-11-16T07:37:37.335438588Z   File "/provider/programruntime/program_starter_wrapper.py", line 88, in execute
2022-11-16T07:37:37.335466993Z     final_result = self.main(backend, self.messenger, **self.user_params)
2022-11-16T07:37:37.335499686Z   File "/code/./program.py", line 502, in main
2022-11-16T07:37:37.335520910Z     ret = trainer.train()
2022-11-16T07:37:37.335545397Z   File "/code/./program.py", line 221, in train
2022-11-16T07:37:37.335625843Z     loss, t_fwd, t_bckwd = self.train_step(train_batch)
2022-11-16T07:37:37.335644144Z   File "/code/./program.py", line 293, in train_step
2022-11-16T07:37:37.335662319Z     self.optimizer.step()
2022-11-16T07:37:37.335679267Z   File "/opt/app-root/lib64/python3.8/site-packages/torch/optim/optimizer.py", line 140, in wrapper
2022-11-16T07:37:37.335744542Z     out = func(*args, **kwargs)
2022-11-16T07:37:37.335790758Z   File "/opt/app-root/lib64/python3.8/site-packages/torch/optim/optimizer.py", line 23, in _use_grad
2022-11-16T07:37:37.335813894Z     ret = func(self, *args, **kwargs)
2022-11-16T07:37:37.335837831Z   File "/opt/app-root/lib64/python3.8/site-packages/torch/optim/adam.py", line 214, in step
2022-11-16T07:37:37.335863581Z     if self.defaults['capturable'] or self.defaults['fused']
2022-11-16T07:37:37.335885565Z KeyError: 'fused'
2022-11-16T07:37:37.335929568Z
2022-11-16T07:37:37.335946623Z During handling of the above exception, another exception occurred:
2022-11-16T07:37:37.335963172Z
2022-11-16T07:37:37.335991995Z Traceback (most recent call last):
2022-11-16T07:37:37.336010325Z   File "/usr/lib64/python3.8/logging/init.py", line 1085, in emit
2022-11-16T07:37:37.336049729Z     msg = self.format(record)
2022-11-16T07:37:37.336065098Z   File "/usr/lib64/python3.8/logging/init.py", line 929, in format
2022-11-16T07:37:37.336082152Z     return fmt.format(record)
2022-11-16T07:37:37.336198160Z   File "/usr/lib64/python3.8/logging/init.py", line 668, in format
2022-11-16T07:37:37.336366939Z     record.message = record.getMessage()
2022-11-16T07:37:37.336391059Z   File "/usr/lib64/python3.8/logging/init.py", line 373, in getMessage
2022-11-16T07:37:37.336410227Z     msg = msg % self.args
2022-11-16T07:37:37.336459186Z TypeError: not all arguments converted during string formatting
2022-11-16T07:37:37.336477951Z Call stack:
2022-11-16T07:37:37.336596920Z   File "/usr/lib64/python3.8/threading.py", line 890, in _bootstrap
2022-11-16T07:37:37.336679861Z     self._bootstrap_inner()
2022-11-16T07:37:37.336700948Z   File "/usr/lib64/python3.8/threading.py", line 932, in _bootstrap_inner
2022-11-16T07:37:37.336725819Z     self.run()
2022-11-16T07:37:37.336744522Z   File "/opt/app-root/lib64/python3.8/site-packages/anyio/_backends/_asyncio.py", line 867, in run
2022-11-16T07:37:37.336762596Z     result = context.run(func, *args)
2022-11-16T07:37:37.336780238Z   File "/provider/server/main.py", line 75, in execute_program
2022-11-16T07:37:37.336797858Z     starter.execute()
2022-11-16T07:37:37.336816243Z   File "/provider/programruntime/program_starter_wrapper.py", line 92, in execute
2022-11-16T07:37:37.336833844Z     logger.error(
2022-11-16T07:37:37.336851598Z Message: 'Error executing program job cdq96djcgdedhj0apec0'
2022-11-16T07:37:37.336873987Z Arguments: (KeyError('fused'),)
2022-11-16T07:37:37.336940229Z webserver-starter - ERROR Failed to execute program: 'fused'
2022-11-16T07:37:37.336959480Z Traceback (most recent call last):
2022-11-16T07:37:37.336977111Z   File "/provider/server/main.py", line 75, in execute_program
2022-11-16T07:37:37.336998211Z     starter.execute()
2022-11-16T07:37:37.337016918Z   File "/provider/programruntime/program_starter_wrapper.py", line 96, in execute
2022-11-16T07:37:37.337035462Z     raise ex
2022-11-16T07:37:37.337057385Z   File "/provider/programruntime/program_starter_wrapper.py", line 88, in execute
2022-11-16T07:37:37.337075587Z     final_result = self.main(backend, self.messenger, **self.user_params)
2022-11-16T07:37:37.337093184Z   File "/code/./program.py", line 502, in main
2022-11-16T07:37:37.337108520Z     ret = trainer.train()
2022-11-16T07:37:37.337127089Z   File "/code/./program.py", line 221, in train
2022-11-16T07:37:37.337143146Z     loss, t_fwd, t_bckwd = self.train_step(train_batch)
2022-11-16T07:37:37.337160520Z   File "/code/./program.py", line 293, in train_step
2022-11-16T07:37:37.337176835Z     self.optimizer.step()
2022-11-16T07:37:37.337197548Z   File "/opt/app-root/lib64/python3.8/site-packages/torch/optim/optimizer.py", line 140, in wrapper
2022-11-16T07:37:37.337214349Z     out = func(*args, **kwargs)
2022-11-16T07:37:37.337232286Z   File "/opt/app-root/lib64/python3.8/site-packages/torch/optim/optimizer.py", line 23, in _use_grad
2022-11-16T07:37:37.337544979Z     ret = func(self, *args, **kwargs)
2022-11-16T07:37:37.337579685Z   File "/opt/app-root/lib64/python3.8/site-packages/torch/optim/adam.py", line 214, in step
2022-11-16T07:37:37.337602418Z     if self.defaults['capturable'] or self.defaults['fused']
2022-11-16T07:37:37.337627948Z KeyError: 'fused'
2022-11-16T07:37:37.352319665Z jaeger_tracing - INFO Span publisher exited
2022-11-16T07:37:37.352881613Z webserver-starter - DEBUG Writing job status to termination marker file: '{"status": "Failed"}'
How can we reproduce the issue?
I have used the code of the TorchRuntimeClient that used to be on Qiskit's website (it fails both with ibmq_qasm_simulator and all real devices I have tried):
import numpy as np
import matplotlib.pyplot as plt
from torch import Tensor, manual_seed, is_tensor
from torch.nn import Linear, CrossEntropyLoss, MSELoss
from torch.optim import LBFGS, Adam
from torch.utils.data import Dataset, DataLoader
from qiskit import Aer, QuantumCircuit
from qiskit.utils import QuantumInstance, algorithm_globals
from qiskit.opflow import AerPauliExpectation
from qiskit.circuit import Parameter
from qiskit.circuit.library import RealAmplitudes, ZZFeatureMap
from qiskit_machine_learning.neural_networks import CircuitQNN, TwoLayerQNN
from qiskit_machine_learning.connectors import TorchConnector
from qiskit_machine_learning.runtime import TorchRuntimeClient, TorchRuntimeResult
seed = 42
manual_seed(seed)
algorithm_globals.random_seed = seed
Generate random dataset
Select dataset dimension (num_inputs) and size (num_samples)
num_inputs = 2
num_samples = 20
Generate random input coordinates (X) and binary labels (y)
X = 2 * algorithm_globals.random.random([num_samples, num_inputs]) - 1
y01 = 1 * (np.sum(X, axis=1) >= 0)  # in { 0,  1}, y01 will be used for CircuitQNN example
y = 2 * y01 - 1  # in {-1, +1}, y will be used for OplowQNN example
Convert to torch Tensors
X_ = Tensor(X)
y01_ = Tensor(y01).reshape(len(y)).long()
y_ = Tensor(y).reshape(len(y), 1)
Plot dataset
for x, y_target in zip(X, y):
if y_target == 1:
plt.plot(x[0], x[1], "bo")
else:
plt.plot(x[0], x[1], "go")
plt.plot([-1, 1], [1, -1], "--", color="black")
plt.show()
Create custom torch dataset class
class TorchDataset(Dataset):
"""Map-style dataset"""
def __init__(self, X, y):
    self.X = Tensor(X)
    self.y = Tensor(y)

def __len__(self):
    return len(self.X)

def __getitem__(self, idx):
    import torch

    if torch.is_tensor(idx):
        idx = idx.tolist()

    X_i = self.X[idx]
    y_i = self.y[idx]

    # important: the dataset item must be returned as data,target
    return X_i, y_i

y = y.reshape(20, 1)
train_set = TorchDataset(X, y)
train_loader = DataLoader(train_set, batch_size=1, shuffle=False)
Set up QNN
qnn = TwoLayerQNN(num_qubits=num_inputs)
print(qnn.operator)
Set up PyTorch module
initial_weights = 0.1 * (2 * algorithm_globals.random.random(qnn.num_weights) - 1)
model = TorchConnector(qnn, initial_weights=initial_weights)
print("Initial weights: ", initial_weights)
Define optimizer and loss
optimizer = Adam(model.parameters(), lr=0.1)
loss_func = MSELoss(reduction="sum")
from qiskit.providers.ibmq import *
token="*****" # Token deleted for privacy
IBMQ.save_account(token)
provider = IBMQ.load_account()
backend=provider.get_backend("ibmq_qasm_simulator")
torch_runtime_client = TorchRuntimeClient(
provider=provider,
model=model,
optimizer=optimizer,
loss_func=loss_func,
epochs=5,
backend=backend,
)
fit_result = torch_runtime_client.fit(train_loader=train_loader, seed=42)
print("id: ", fit_result.job_id)
print("execution time: ", fit_result.execution_time)
What should happen?
Last week (iirc) this code worked with no problems.
Any suggestions?
No response

This is a server side problem and I don't think it can be fixed here. @woodsp-ibm can you transfer it to the qiskit-ibm-runtime repo?

This is a server side problem and I don't think it can be fixed here. @woodsp-ibm can you transfer it to the qiskit-ibm-runtime repo?

I do not have the necessary permission on qiskit-ibm-runtime- maybe @mtreinish has and can oblige?

Closing as the author opened an issue directly on qiskit runtime Qiskit/qiskit-ibm-runtime#631


Can't able to persist existing data when i update docker image of database

I am using PostgreSQL  docker container , But when i upgrade its version by changing its docker image my existing databases are lost

you need to store your data for the container in a persistant volume.
this aint no problem with the watchtower, check information about volumes and persistance here.


[BEAM-6350] Reuse PCollectionView when created in translators

If for LeftJoin is used BroadcastHashJoinTranslator then from right side is created PCollectionView (as sideInput).
If we use right side in multiple joins then PCollectionView  is created multiple times which is not optimal behavior.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.


naoqi_libqi: 2.9.7-0 in 'noetic/distribution.yaml' [bloom]

Release naoqi_libqi for noetic, in version 2.9.7-0:
upstream repository: https://github.com/ros-naoqi/libqi.git
release repository: https://github.com/ros-naoqi/libqi-release.git
distro file: noetic/distribution.yaml
bloom version: 0.11.2
previous version for package: null

I'll open another PR


Update README.md

I translate FRILLER's introduction from Raspberry Pi official magazine the MagPi into simplified Chinese, for Chinese makers to learn more about this phenomenal robots and try to make it better.

Wow Tommy, that is pretty neat.  Thanks for doing that. Cheers!


Images for step 10 and 11 are too small.

When you see Provisioning completed, select the CustomLearningAdministration tab as shown in the following image:

[Too small to see] Image of the M365 learning pathways solution tab

Select Complete as shown in the following image:

[Too small to see] Image of the M365 learning pathways solution complete button
I would reconsider how we instruct the user to return to the administration page. You might simply say:

Close the provisioning site to return to the learning pathways administration page.
If you have closed the administration page, you can reopen it now.

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

ID: d5c747f8-17cb-209a-a503-a9caa9da8cd8
Version Independent ID: 0a7ae298-5bd2-1327-5933-7adfd7c0975a
Content: Provision the SharePoint Success Site - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/provision-sss.md
Service: sharepoint-online
GitHub Login: @Holland-ODSP
Microsoft Alias: hokavian

In other words, I'm not sure that images will be helpful here.

@jsuddsjr Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hello @jsuddsjr John, how are you?
Document has been updated after you opened this ticket with bigger images.
Thanks
@scanum please close

@jsuddsjr Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Fixed height TextView inside ScrollView is not scrollable

Environment

Nativescript-Playgroud: 5.0.2

Describe the bug
when TextView  has fixed height and it is placed inside the ScrollView at the bottom of the content then when we type more line than the height it does scroll to bottom automatically. but to see the top content there is no way other than erasing the text. scrolling inside TextView doesn't work.
To Reproduce
https://play.nativescript.org/?template=play-ng&id=kwJNyQ
Expected behavior
we should be able to scroll the typed text inside the TextView to see the top text.
Sample project
https://play.nativescript.org/?template=play-ng&id=kwJNyQ

It seems that the issue is reproducible with NativeScript Core project (not related to Angular) - here is a Playground demo with Core project. Moving the issue to the main modules repository.

This issue was moved to NativeScript/NativeScript#7040


Example Leads to Empty Rendering

While my experiments with the human loop worked well, there is something I am missing about the Python API. When I try to run the code example I come up with code like this:
from machine_common_sense import MCS

import matplotlib.pyplot as plt
import numpy as np
import time

from PIL import Image

config_json_file_path='/home/fplk/mitibm/MCS/python_api/scenes/playroom.json'
unity_app_file_path = '/home/fplk/mitibm/Downloads/AI2Thor_MCS/MCS-AI2-THOR-Unity-App-v0.0.1.x86_64'

count = 0

def select_action(output):
    global count
    if count < 3:
        count += 1
        return 'Pass', {}
    return '', {}

# Either load the config data dict from an MCS config JSON file or create your own.
# We will give you the training config JSON files and the format to make your own.
config_data = MCS.load_config_json_file(config_json_file_path)

# We will give you the Unity app file.
controller = MCS.create_controller(unity_app_file_path)

output = controller.start_scene(config_data)

test = plt.imshow(np.asarray((output.image_list[0])))
plt.show()

# Use your machine learning algorithm to select your next action based on the scene
# output (goal, actions, images, metadata, etc.) from your previous action.
action, params = select_action(output)

# Continue to select actions until your algorithm decides to stop.
while action != '':
    step_return = controller.step(action)
    plt.imshow(np.asarray((step_return.image_list[0])))
    plt.show()
    action, params = select_action(output)
    time.sleep(2)

# For interaction-based goals, your series of selected actions will be scored.
# For observation-based goals, you will pass a classification and a confidence
# to the end_scene function here.
controller.end_scene('classification', 'confidence')

While I see all objects in the initial data structure config_data, the object_list fields in all outputs are empty and the rendered images I show only contain backgrounds. I thus have three questions:
~a) Why is that and what can I do to render the full scene like I get in the human loop? So far I have not found the fundamental difference of the code above to what you do in python_api/machine_common_sense/run_mcs_human_input.py. Why does one work and the other does not?~
b) Could I be added to the AI2Thor / TA2 Slack channel to ask questions like this more dynamically? This would be great.
c) I see the depth maps as well in the data structures, but no segmentation maps. Should I segment myself (obviously more error-prone, since it involves another CV task) or is there a way to get segmentation maps directly? It seems that in the underlying AI2Thor this is accessible via instance_segmentation_frame, cmp. https://ai2thor.allenai.org/ithor/documentation/metadata/ - can I access the underlying system and extract this or what is the best way to obtain segmentation maps?
Thank you very much in advance.

Regarding (a): Oh, I see. I added , status in front of the load_config_json_file call and I see objects now. It silently failed when it got handed a tuple. Good thing I read your code :)
Would still be great to get answers regarding (b) and (c). Thanks in advance.

B. You should be able to add yourself to the ta2 and ta2-api channels.
C. The instance_segmentation_frame should be available in the object_mask_list of the MCS_Step_Output object.

Excellent. Thank you.


angular2

is this available in angular2 yet?

No, see #327 and close this issue (it's an unnecessary duplicate of #327).

@Koslun thx man


Create loading screen/text and hide charts while loading

Fits with 5.7, I think.

Superseded by #165


refactor: create samplers directory

Codecov Report

Merging #27 (b49e12f) into main (c51008f) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
+ Coverage   94.89%   94.91%   +0.02%
==========================================
  Files          15       16       +1
  Lines        1390     1397       +7
==========================================
+ Hits         1319     1326       +7
  Misses         71       71

Impacted Files
Coverage Δ

seqal/samplers/base.py
95.52% <ø> (ø)

tests/test_samplers.py
98.31% <ø> (-0.01%)
:arrow_down:

seqal/samplers/__init__.py
100.00% <100.00%> (ø)

seqal/samplers/samplers.py
99.34% <100.00%> (ø)

tests/test_base_sampler.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Clarify that the full config enables everything that can be tested together

Document what config.py full (formerly config.pl full) is supposed to enable:

The goal of the full configuration is to have everything that can be tested
together. This includes deprecated or insecure options. It excludes:

Options that require additional build dependencies or unusual hardware.
Options that make testing less effective.
Options that are incompatible with other options.
Options that remove features.
Options that are variants, so that we need to test both with and without.

Annotate each excluded option to explain why it is excluded.
Stop excluding options that had no justification for being excluded:

SSLv3 — it was excluded due to being deprecated, but that's no reason.
MBEDTLS_X509_ALLOW_EXTENSIONS_NON_V3. It was not tested at all.
MBEDTLS_PSA_CRYPTO_SE_C — it was originally excluded from full because it could break programs, but that's no longer been the case for a while.
Finally fix that long-standing bug whereby MBEDTLS_PLATFORM_xxx_ALT was treated differently if xxx contains an underscore (keeping MBEDTLS_PLATFORM_SETUP_TEARDOWN_ALT as the one explicit exception that needs different treatment).

Add a config.py full_no_deprecated and rework the way MBEDTLS_DEPRECATED_WARNING, MBEDTLS_DEPRECATED_REMOVED and MBEDTLS_TEST_DEPRECATED are handled in all.sh.
Make psa_crypto_se.c build with -pedantic. While I'm at it, make everything build with -pedantic and test it (before, we were only testing in some “exotic” configurations, and some of the code had minor, easily fixed deviations). Arguably -pedantic should be on by default at least in CMake builds with GCC or Clang, but I'd prefer to avoid extensive changes in this PR which has already grown pretty large.
2.16 backport: #3286. 2.7 backport: incoming.

I fully agree with the design, but I think there's still a point we should clarify: "Options that are variants, so that we need to test both with and without." - which variant should be in full?
IMO since we'll use config.py full for test coverage reports, it would make sense to prefer the variant that's also in the default config, so that the report is closer to what's most likely to be used.
I noticed because MBEDTLS_X509_ALLOW_EXTENSIONS_NON_V3 is an example of it IMO: you might argue it's a feature, but since it's a feature that also breaks standard compliance (v1 and v2 standard have no extensions, and since they require strict parsing, they require use to reject v1/v2 certs with junk at the place where v3 introduced extensions), I'd argue it's really a variant: do you prefer to be strict about compliance (on this point) of the certs you receive or not?
So I was surprised you added it to full (but thanks for noticing that it wasn't tested and improving the tests, I obviously agree it needs to be tested), and when arguing that IMO it shouldn't be included, I noticed that we probably want to have some general rule about variants, because otherwise for each variant toggle we can argue what its setting should be.

Hmmm. In a sense, everything is a variant. For example, MBEDTLS_SHA256_C is a variant: we should test how the X509 module behaves if it's asked to verify a certificate that embeds a SHA-256 hash when MBEDTLS_SHA256_C is disabled.
So maybe “variant” isn't a useful concept. What definitely needs to be tested separately from full is things that have an impact on other parts of the code. For example MBEDTLS_CTR_DRBG_USE_128_BIT_KEY and MBEDTLS_ENTROPY_FORCE_SHA256 interact with each other, so we test all four combinations with each of them on/off. MBEDTLS_X509_ALLOW_EXTENSIONS_NON_V3 doesn't interact with anything else so all that really matters is that we test with and without it (each time with MBEDTLS_X509_CRT_PARSE_C, otherwise it doesn't count), and under this criterion it's still ok to enable it in full.
Under this reasoning, more things might be ok to enable in full. But in practice I think it's only MBEDTLS_X509_ALLOW_UNSUPPORTED_CRITICAL_EXTENSION. I'm going to think about this a little more and welcome other opinions.

Hmmm. In a sense, everything is a variant.

I think some things are still more a variant than others in a relatively objective way.
For example, with MBEDTLS_SHA256_C there are parts of the code that will be built only when it's enabled, and no (relevant) part of the code that will be built only when it's disabled. (I added "relevant" because there's a dummy version of mbedtls_ecjpake_self_test() that's only built when SHA-256 is disabled, I think we can agree to disregard it).
For contrast, with MBEDTLS_RSA_NO_CRT there are parts of the code that are only built when it's enabled, and parts of the code that are only built when it's disabled.
I think config.py full should be Pareto-optimal in that it's not possible to included more code in the build with excluding some other code (while respecting other constraints such as external dependencies and hardware), and a variant would be a switch such that the config is Pareto-optimal regardless of its state, and then we need some other guiding principle for what its value should be.
(Note: I'm mostly thinking out loud. Intuitively the concept of "variant" seemed to make sense to me and I'm trying to clarify what sense.)
Now I checked with this definition MBEDTLS_X509_ALLOW_EXTENSIONS_NON_V3 would not be a variant, as there is a line of code that's only included if it's disabled, and no line of code that's only included if it's enabled. So apparently my attempt at a more formal definition of "variant" doesn't quite match my intuition on this example. Perhaps it's because there are two different things here:

being a variant in that both values will disable some code in the build
being a variant in that both values really deserve testing (and of course when it comes to testing as you say an important question is whether this interacts with other options)

I clearly need to think about it more. I also welcome other opinions!

I think config.py full should be Pareto-optimal in that it's not possible to included more code in the build with excluding some other code

Or hurting testing in some way (e.g. dependency). That matches my original intuition too, and then as you note MBEDTLS_X509_ALLOW_EXTENSIONS_NON_V3 should be included and MBEDTLS_RSA_NO_CRT should be excluded.
But there's another way to approach this. We can't test every single combination because there are just too many of them. So we go for conditional compilation coverage: for every #if in the code, we should test every branch at least once. Doesn't matter if it's a defined test on xxx_C, a defined-test on something else, or a more complex expression. The full config is just a convenient way to lump many boolean options that don't interact with each other.

One thing my PR still doesn't do, and which I'm not keen to add because I think it's big enough already, is to try building with full but without the MBEDTLS_PLATFORM_xxx_ALT.

CI is passing except for a Windows job which tries to compile C99 code as C89. Fix in the CI job: https://github.com/ARMmbed/mbedtls-test/pull/137 (private link).

I did a quick second pass, and I'm still happy with the design. I'll finish reviewing the details of the code tomorrow.
A question on PR structure/scoping though: was there a reason to include passing -pedantic in this PR, as opposed to a distinct PR, or was it an accident? (Regardless of the answer, not asking for any change at this point, I don't think it would be worth the effort.)

We were already running a couple of jobs with -pedantic. One of these jobs broke when I added MBEDTLS_PSA_CRYPTO_SE_C to config.py full. For this reason, and also for MBEDTLS_PSA_CRYPTO_SE_C to work with Visual Studio, I had to fix the gccisms in psa_crypto_se.c. I didn't have to make other builds pass with -pedantic, but I decided to do it because it wasn't too difficult and it'll make the testing more robust.

By the way, I'm still on the fence about adding -std=c99 -pedantic to CMAKE_C_FLAGS for GCC and Clang in CMakeLists.txt. Is there any reason why we didn't do so before?
I tested locally that the build passes with -std=c99 -pedantic with gcc-5, gcc-7, gcc-9, clang-3.8, clang-6.0 and clang-10.

I think the only reason why we didn't use -pedantic everywhere before is because some parts didn't pass, and I guess we weren't very motivated to fix things like overlong string literals in example programs. So IIRC the compromise was to add some tests building the library only (not tests or programs) with -pedantic.
But now that you've done the work of having everything build with -pedantic and tested this with several compiler versions, I think it's worth adding it to our default CFLAGS in CMakeLists.txt and Makefiles (unless there's a particular reason you're only mentioning CMake?). That's the best way to ensure it keeps passing in the future.

I was thinking cmake and not make because in cmake, we know what the compiler is. In make, we assume that the compiler supports gcc-like options, but we're more tolerant of compiler variation: for example, we don't pass -Werror, which has always bothered me, but makes sense because one particular version of gcc might end up emitting a spurious warning.

I was thinking cmake and not make because in cmake, we know what the compiler is. In make, we assume that the compiler supports gcc-like options, but we're more tolerant of compiler variation: for example, we don't pass -Werror, which has always bothered me, but makes sense because one particular version of gcc might end up emitting a spurious warning.

@gilles-peskine-arm Since I'm thoroughly satisfied with the design, should we remove the "needs design approval" label or would you like a second opinion?

There was a merge conflict in tests/scripts/all.sh. Rather than rebase the branch, which would have made reviewing harder, I've removed the commit "all.sh: test ssl3 in the full-config build". This commit is not necessary, and I have second thoughts about it. It does two things:

Run the ssl3 compat.sh with the legacy version of GnuTLS rather than the default version. This was misrepresented in the commit message, which incorrectly stated that we didn't run compat.sh for ssl3 with GnuTLS. Our reference default version of GnuTLS does support SSLv3 so this change is not necessary.
Group SSL3 testing into the same component as other testing, which simplifies all.sh, but may hurt performance when all.sh is run in parallel because it lengthens one of the longest components.

I force-pushed to fix typos in commit messages. Content changes will be in new commits.

@gilles-peskine-arm Right now it's unclear to me whether you're planning further changes or the PR is ready for re-review. Can you re-request a review from me when you're done, so that it show on my list? Thanks!

@mpg At this point I don't intend any more changes unless motivated by review feedback, so please review.
CI passed except for an issue in merge jobs due to a temporary problem on development.

While making the backports, I realized that I'd managed to confuse myself as to what the various *deprecated* components of all.sh were doing. So I added a bit of documentation.


Getting relations using filtering strategy

Project.find('1').then(project => {
    project.tasks().all();
});

This code produces query: /projects/1/tasks
But I need to utilize filters instead:
/tasks?filter[project][type]=Project&filter[project][id]=1
Is there is an easy way to do it?

project.tasks().where({ project: { type: 'Project', id: '1' }})
Though why, if you are requesting /projects/1, do you need to specify the project id? Or the type for that matter?

I don't need to have /projects/1 in my URI.
My URI: http://localhost/tasks?filter[project][type]=Project&filter[project][id]=1


Setup a remote dev workflow for devs to run code and unit tests remotely

should we merge this with the ducklake proj? Both needs a server connection, and will read from the serial port. Both will need a webpage front end, and we might as well include a simple login to avoid random people visiting the site and taking over the arduino.

Yes and no.
Yes, because of all the reasons you mentioned
No, because DuckLake if things pan out wouldn't end up requiring a server, that whole setup was a little too much, PlatformIO has some remote development workflows we might end up going with that rather than building some infra ground up. (although would love to do that) now, we'd probably end up making some sort of credential manager to avoid people running random code on our remote dev servers.
We basically have three projects as of now:

Project Name
Description
Priority

Duck Lake
Visualise Rocket Data
Low

Esterhazy
Remote dev/ run tests
Moderate

Delisle
Software in the Loop tester
High

We'll work on Delisle to get the whole software in the loop system going and then work on Esterhazy to use the whole system remotely, we might be able to do some Esterhazy work and then add on more features later, and Duck Lake will be very low on our list, as I cannot find someone to write some React code as of now.

Thats great news! I just googled platformio remote dev and it seems super easy to setup (fingers crossed)

Moved to the Esterhazy project


on reconnect always get myTeams

Summary
On a previous PR we remove the getMyTeams from the WS reconnection, this adds it back as well as calling getTeams to be sure we are getting the open teams from the server

closing and re-opening to trigger jenkins


KeyError: 'render_modes'

Just tried to run the maze_2d_q_learning.py example but it returns the error:
KeyError: 'render_modes'

You can try "pip install gym==0.24.1" to change the gym version.


.ground

so, w/ generic Unlocker, mouseover.ground does not "click" the spell after bringing up the reticle, and then hangs the CR.
but target.ground does not, it works as mouseover.ground should.

fixed


hash (txid) isn't correctly generated; perhaps because nTime isn't deserialized properly.

Localise with debugger; this happens before/during OutputTxJson(const CTransaction & tx). Probably at CMutableTransaction/CTransaction in main.h.
nTime is there during parsing the hex, but not when outputting the CTransaction.

Correct.
Next issue: figure out why bitcoin doesn't (de)serialize them.


Added support for pathType, ingressClassName

Description
Kubernetes 1.18 brought a few changes in API for ingress:

A new pathType field that can specify how Ingress paths should be matched.
A new IngressClass resource that can specify how Ingresses should be implemented by controllers.
Support for wildcards in hostnames.
This PR covers the first two (I guess, the third one should already work, haven't checked it).

pathType could be one of the following: ImplementationSpecific, Exact, Prefix (defaults to ImplementationSpecific).
API gives controllers freedom to decide what ImplementationSpecific means to them. - They could treat it as a separate PathType or identically to Prefix or Exact path types (docs).
To give some examples, for nginx ImplementationSpecific = Prefix, for Istio - Exact (meaning, you can't access subpaths in URI, by default). That's why it's important to support this feature in grafana-operator.
Note: Since k8s defines an exact set of pathTypes and offers a default value, I decided to let operator fall back to ImplementationSpecific in case of incorrect input. I can change that if it doesn't fit your view of how opinionated on the inputs the operator should be.
Note: I tried to implement pathType using *v1beta1.PathType, though it didn't work very well (not at all, tbh :D). I haven't seen any issues with string (*v1beta1.PathType is actually a string const), so stayed with it.

IngressClass. In modern Kubernetes environments, the kubernetes.io/ingress.class annotation is deprecated in favour of IngressClass. I guess it doesn't require any additional comments :)

Relevant issues/tickets
None
Type of change

[X] New feature (non-breaking change which adds functionality)

Checklist

[X] This change requires a documentation update (included in PR)
[ ] I have added tests that prove my fix is effective or that my feature works
[X] I have added a test case that will be used to verify my changes
[X] Verified independently on a cluster by reviewer

Verification steps
Deploy grafana with similar set of settings to the one below:
    ingress:
      enabled: true
      hostname: grafana.microk8s.localhost
      ingressClassName: nginx
      path: /
      pathType: Prefix
      tlsEnabled: true
      tlsSecretName: grafana-tls

For k8s 1.18+, you should see both ingressClassName and pathType in the resulting ingress:
~ k get ingress grafana-ingress -o yaml
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
[...]
spec:
  ingressClassName: nginx
  rules:
  - host: grafana.microk8s.localhost
    http:
      paths:
      - backend:
          serviceName: grafana-service
          servicePort: 3000
        path: /
        pathType: Prefix

For k8s <1.18, both ingressClassName and pathType will be absent in the resulting ingress (underlying lib handles it well):
~ k get ingress grafana-ingress -o yaml
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
[...]
spec:
  rules:
  - host: grafana.microk8s.localhost
    http:
      paths:
      - backend:
          serviceName: grafana-service
          servicePort: 3000
        path: /

One more note: by looking at preexisting pieces of code, I wasn't really sure whether new helpers should be exported or not, so stayed with the former option.

Hi @weisdd , Thanks for your PR , trying to verify it, I'm running into an issue when upgrading from v3.9.0 -> your branch, using a normal deployment from deploy/examples/Grafana.yaml
{"level":"debug","ts":1613601922.845546,"logger":"action-runner","msg":"(    4)     FAILED create grafana ingress"}
{"level":"debug","ts":1613601922.8456655,"logger":"controller-runtime.manager.events","msg":"Warning","object":{"kind":"Grafana","namespace":"grafana","name":"example-grafana","uid":"3487782a-ecb4-45b7-adb0-264dbf7c1ed0","apiVersion":"integreatly.org/v1alpha1","resourceVersion":"156423"},"reason":"ProcessingError","message":"Ingress.extensions \"grafana-ingress\" is invalid: spec.ingressClassName: Invalid value: \"\": a lowercase RFC 1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')"}

I think you need to add some upgrade-proof logic (shouldn't be a massive change). I think the easiest way of doing that would be to define some defaults in constants, then use them if the fields you have added that don't have values assigned to them in the cr.
I assume that not every field requires a value, so it's fine if you just add those defaults for fields that need them.
Also , if you have a better approach to the problem I described, then feel free to use it :+1:

Hi @HubertStefanski,
Thanks for pointing that out! Do you, by chance, have a doc somewhere with a standard set of test cases for the operator? :)
It turned out that it's better to return a nil pointer by default, so the end spec of ingressClassName is nullified on client side and, as a consequence, defaulted on the API server side.
With this change in place, I'm not seeing the issue you described anymore.

@weisdd Perfect, thanks for addressing that, I'll have a look again shortly.
We don't have a standard set of test cases, but generally when we verify a PR we try to account for the following:

Is the upgrade seamless? (many of our users run the operator from OLM, and that has a known issue where it will upgrade the operator without users permission) so we need to ensure that when we push new changes in a release they won't break existing deployments.
Do the changes work as intended?
Is there anything that was missed?

Other than that we rely on our e2e tests, they're basic for now, but we'd like to expand them in the near future.

It turned out that it's better to return a nil pointer by default, so the end spec of ingressClassName is nullified on client side and, >as a consequence, defaulted on the API server side.

Even better than what I had in mind :+1:

/lgtm

@HubertStefanski thanks for sharing the details about testing, I'll keep the list in mind next time :)


How did the user switch from Azure portal to the web app (non-swagger UI)

Hi,
I was able to deploy successfully but haven't been able to understand how do i get to the webapp UI page now (non swagger view). I have an existing AAD app registration which I can use.
-Deepika

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

ID: 3eb2e7c5-e153-4067-ae8c-45eb64add7da
Version Independent ID: 3c87633d-44b7-525d-0e29-75f2d4bcbf2b
Content: Deploy Azure Blockchain Workbench Preview
Content Source: articles/blockchain/workbench/deploy.md
Service: azure-blockchain
GitHub Login: @PatAltimore
Microsoft Alias: patricka

@shdeepika , Thank you for sharing the query. We are looking into this and would get back to you soon.

Hi,
I was able to open the launch cloudshell webapp UI by clicking on the second app service plan 2. The documentation doesn’t call this out though and I had to figure it out on my own. Suggestion to add this 😊
-Deepika
From: souravmishra-msft<EMAIL_ADDRESS>Sent: Monday, October 7, 2019 3:25 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Deepika Sharma<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How did the user switch from Azure portal to the web app (non-swagger UI) (#40219)
@shdeepikahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fshdeepika&data=02|01|Deepika.Sharma%40microsoft.com|1a3ed64b9e3544762a8808d74b0c5eb9|72f988bf86f141af91ab2d7cd011db47|1|0|637060388822662654&sdata=D%2BmNeIpmkGQ%2BjyOzIOwvZ%2FrUn9U50hH48JSNHvsrrUc%3D&reserved=0 , Thank you for sharing the query. We are looking into this and would get back to you soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F40219%3Femail_source%3Dnotifications%26email_token%3DANNFV4Y3DZBVCVU5IP7J3RDQNMBN5A5CNFSM4I6B7MO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAPV6ZA%23issuecomment-538926948&data=02|01|Deepika.Sharma%40microsoft.com|1a3ed64b9e3544762a8808d74b0c5eb9|72f988bf86f141af91ab2d7cd011db47|1|0|637060388822672649&sdata=vuMIoWv7ONrfN1KQhrXIxVElrwE79iamGEkoP1U71v0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANNFV46AX7P45NALDITVBJTQNMBN5ANCNFSM4I6B7MOQ&data=02|01|Deepika.Sharma%40microsoft.com|1a3ed64b9e3544762a8808d74b0c5eb9|72f988bf86f141af91ab2d7cd011db47|1|0|637060388822682645&sdata=BhWZ6mh%2B%2FiXca5Ms2UPNuTY4KUYB%2Fl1ov09R%2BQEWWV4%3D&reserved=0.

@shdeepika, Thanks for the feedback. I'm adding a section on how to get the API endpoint to the "use api" article. https://docs.microsoft.com/en-us/azure/blockchain/workbench/use-api. It will go out with another update soon.
Thanks,
Pat
#in-progress

I added a section on how to get the API endpoint in the "Use API" article. The update has been published.
Pat
#please-close


Public Cloud web content reviews

Describe the Issue
New public cloud content and web page review with web designer
Additional Context
Web content specialist will produce content from initial conversations and produce draft web content for review
Acceptance Criteria

Meet with web designer
Work through understanding of service we provide and what is public cloud
Review web content drafts
Help finalize new website content

.
.
.
.
.
.
.
.
.
.
.
Extra Template Info
Definition of Ready
These set of conditions will need to be met in order to bring a ticket into the sprint and start work. Protects the team from unclear requirements.
Issues (Task/Story/Spike) aligned to an EPIC and linked appropriately.
Assigned to the appropriate CPF MEMBER (and is not left blank in ZenHub) at the start of sprint (1st day of the sprint)
Acceptance has been defined for the issue and has been reviewed with CPF team and approved by the necessary approver.
Has been sized and estimated by the delivery team.
Detailed breakdown of the steps required to complete the story in the additional context
Any additional specifications have been documented, reviewed with CPF and approved by the necessary approver.
Acceptance Criteria
A set of pre-defined requirement that need to be met in order to mark the user story as ?done?.
It should be testable with no room for interpretation
It should be either ?pass? or ?fail?
It should be clear enough for business stake holders to understand
As part of the user story, it should be written from the user perspective
A well written acceptance criteria is great for

managing expectations
defining scope
reducing ambiguity
establishing testing criteria for QA
Given (Pre-condition)
When (Action)
Then (Outcome)

Definition of Done
These set of conditions are met for work to be considered as complete on an issue.
All acceptance criteria(s) have been validated.
All the necessary documentation for the issue has been uploaded to Teams.
Functionality has been tested when applicable.
Functionality has been demonstrated to the relevant stakeholders (where applicable).
Story has been scheduled for a Sprint Demo/community update and impacted teams invited to the Demo.
Stories accepted by PO and documented for potential sharing with impacted users.

Documented = Captured in Community Update (CU) deck
Release notes = aspirational goal of providing a bulleted list of features and changes that our users can touch. Was going to store them in our Cloud Pathfinder repo. Can also display to users in the CU

Review completed.


[Dependency Scanning] Build libSwiftScan with Swift Modules required for parsing regex

Otherwise scanning actions which encounter regex literals will fail.
Resolves rdar://108321860

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test Windows platform

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test macOS platform

@artemcm, since you added this, I see an incorrect runpath added to this library on linux:
> readelf -d swift-DEVELOPMENT-SNAPSHOT-2023-05-06-a-ubuntu20.04/usr/lib/swift/host/lib_InternalSwiftScan.so | ag runpath
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN:$ORIGIN/./swift/linux]

The issue is that you've triggered this CMake function to automatically add the Swift runpath, which fails because it expects all host libraries to be placed in usr/lib/.
You'll want to fix this, else this library won't always work on linux.


kube3d 1.7.0 -> 3.0.0

Things done
nix-review ... - fails with the following error:
builder for '/nix/store/74pn0qlf2gfg36mxsq2xfwdbvqvra422-kube3d-3.0.0.drv' failed with exit code 1; last 10 log lines:
  Building subPackage ./cmd/util
  Building subPackage ./pkg/cluster
  Building subPackage ./pkg/runtimes
  Building subPackage ./pkg/runtimes/containerd
  Building subPackage ./pkg/runtimes/docker
  Building subPackage ./pkg/tools
  Building subPackage ./pkg/types
  Building subPackage ./pkg/util
  Building subPackage ./tools
  go: directory tools is outside main module

What do?

\marvin opt-in

/marvin opt-in

Result of nixpkgs-review pr 93680 1

1 package built:
- kube3d

[nix-shell:~/.cache/nixpkgs-review/pr-93680-1]$ ./results/kube3d/bin/k3d --helphttps://k3d.io/
k3d is a wrapper CLI that helps you to easily create k3s clusters inside docker.
Nodes of a k3d cluster are docker containers running a k3s image.
All Nodes of a k3d cluster are part of the same docker network.

Usage:
  k3d [flags]
  k3d [command]

Available Commands:
  cluster     Manage cluster(s)
  completion  Generate completion scripts for [bash, zsh, powershell | psh]
  help        Help about any command
  image       Handle container images.
  kubeconfig  Manage kubeconfig(s)
  node        Manage node(s)
  version     Show k3d and default k3s version

Flags:
  -h, --help      help for k3d
      --verbose   Enable verbose output (debug logging)
      --version   Show k3d and default k3s version

Use "k3d [command] --help" for more information about a command.

Validated k3d --version is set (introduced v3 suffice to the module path)

[x] k3d cluster create successfully creates a cluster

Skaffold has an optional dependency on k3d. Yesterday compatibility with k3d v3 was merged and will be the only supported version going forward, so ahortly after skaffold cuts a releas I'll update skaffold as well.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/231

Could we merge this?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/191

LGTM!

/status needs_merger

You probably want to set your PR to needs_reviewer after opting-in @blaggacao :) That way you could theoretically get an assigned reviewer (though there currently aren't enough of those).

Thanks @blaggacao @jlesquembre!


Should we ever restore events?

We back up but don't ever restore nodes. Should we consider doing the same with events?
cc @lypht @jrnt30 @jbeda @skriss @nrb @paulbouwer

Meh -- I have a hard time seeing that being a good idea. A lot of times they'll have details that don't apply to the current cluster. I could see it causing more confusion than helping.

That's my thinking too

Yeah, I almost always exclude them from restores if I remember to - often I think they don't even restore properly.

Add events to nonRestorableResources in pkg/controller/restore_controller.go

I'd like to give this a try over the weekend if it's not super urgent

@sanketjpatel that would be fantastic! Please do!


Exclamation Marks not rendering correctly

Exclamation marks render as a lowercase 'l' when sent
when typing, the exclamation mark renders normally, but when the message is sent it turns into a lowercase 'l'

DiscordOS: [vencord and openasar in discord canary]

Will not be fixing unless it becomes a problem on stable. I do not provide support for discord canary

strangely enough, I switched to stable and it was fixed for a bit, but it is now an issue on normal/stable discord as well... might just be an issue with my computer though

strangely enough, I switched to stable and it was fixed for a bit, but it is now an issue on normal/stable discord as well... might just be an issue with my computer though.
edit: I fixed the issue by changing the font in the theme editor

Yeah I'm pretty sure it's with your computer as I do not have the problem on stable


Formatting support

I am interested in getting textDocument/formatting support working. There was some talk in the #kakoune IRC about it a few days ago.

That would be great! Would you like to go with implementation?

Wow this is just awesome, thanks for implementing @ul 🙂

Re-opening because it's broken, text edit is not implemented properly.


Non-Bezier edges connecting a node to itself are invisible

Describe the Bug
Using latest version of ReactFlow, trying to connect a node to itself results with invisible edge when not using bezier and not always visible when using bezier.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
import React, { StrictMode } from "react";
import ReactDOM from "react-dom";
import ReactFlow from "react-flow-renderer";
import "./styles.css";

const initialNodes = [
  {
    id: "n1",
    data: { label: "n1" },
    position: { x: 200, y: 0 }
  },
];

const initialEdges = [
  {
    id: "e1",
    source: "n1",
    target: "n1",
    type: "step"
  }
];

const EdgesFlow = () => (<ReactFlow nodes={initialNodes} edges={initialEdges} fitView />);

const rootElement = document.getElementById("root");
ReactDOM.render(
  <StrictMode>
    <EdgesFlow />
  </StrictMode>,
  rootElement
);

Expected behavior
It is expected that the edge will be at least visible, and that it will be possible to select it for adjustments or removal needs.
Screenshots or Videos

Platform

OS: Any
Browser: Any
Version: 10.x

Additional context
There is also an issue related to this one, where if we have a step with outputs on the left and right and we connect the left output to a step located to the right, the edge will go up/down and to the left, through the source node. This will cause a visual confusion where it seems as the edge is connected to the right output while it is in fact connected to the left.

React Flow doesn't support self connecting edges. If you need it, you need to implement a custom edge. Example: https://codesandbox.io/s/self-connecting-node-2nbh7v?file=/src/CustomEdge.tsx

Above CustomEdge.tsx is a horizontal (input at right and output at left) self-connected node.
Here is a vertical (input at top and output at bottom) version:
import { BezierEdge, EdgeProps } from "reactflow";

export default function CustomEdge(props: EdgeProps) {
  if (props.source !== props.target) {
    const newProps = { ...props,
      sourceX: props.sourceX - 2,
      targetX: props.targetX - 2,
    };
    return <BezierEdge {...newProps} />;
  }

  const { sourceX, sourceY, targetX, targetY, id, markerEnd } = props;
  const radiusX = 50;
  const radiusY = (sourceY - targetY) * 0.6;
  // notice here
  const edgePath = `M ${sourceX - 2} ${sourceY - 2} A ${radiusX} ${radiusY} 0 1 0 ${ targetX - 2 } ${targetY + 2}`;

  return (
    <path
      id={id}
      className="react-flow__edge-path"
      d={edgePath}
      markerEnd={markerEnd}
    />
  );
}


How do I get latest direct messages?

I'm building an application that requires access to direct messages. My application has the necessary permissions to access dm's. There will always be only a single dm in the inbox but I do not know what the ID of that message will be (at least initially).
I've tried this code:
direct_messages = api.direct_messages(0,1,5,True)
print direct_messages
But it's returning empty even though I know that there is indeed a direct message sitting there waiting. How do I handle this? Is there a way to grab a dm when you don't know its id?

Hi, use this pytohn lib:
https://python-twitter.readthedocs.io/en/latest/twitter.html#module-twitter.api
There are DM's.

Twitter has now changed its Direct Message API endpoints.
I've made a branch (https://github.com/tweepy/tweepy/tree/dm-migration) and draft PR (#1228) that should fully support these changes.
Feel free to test it and report any issues you encounter.
With it, you should now be able to use the direct_messages/events/list endpoint using  API.list_direct_messages to do what you want.
Note though, that only Direct Messages within the last 30 days can be accessed with this endpoint.
For code block usage, see https://help.github.com/articles/creating-and-highlighting-code-blocks/.

is this still available?

#1228 has been merged and released as part of Tweepy v3.8.0.


Rename the Server-Timing header property

And use true as the default value.
README changes in another PR, they need to be merged after release.

docs will get updated when we release, I assume?

Yes, exactly that


fix: Multi add select review fixes

Contributes to #2367
addresses all the points in the above issue with the exception of the accessibility tests are failing and i think they might be related to carbon, so more investigation is necessary.

@elycheea thanks for the feedback! yeah i'm going to work on fixing that test in a follow up PR.


Fix the build process

The build script was trying to run things from the build/ directory, which is empty. etsc is used to build that directory out, but currently that's only run if you do npm run ts or npm run nodemon and then quit that process once built.
This adds a new build:etsc script and runs that as part of dev, start, and build scripts.

Run etsc as part of the npm build script
Use run-s instead of npm run, for consistency and Yarn support
Removes the redundant build:type-check script — that's what build:ts does

faf464d makes it even more clear.


Import dataset YAML in GUI

Is your feature request related to a problem? Please describe.
I can happily export/download complex dataset-configurations through the GUI as a YAML files:

However, I cannot import these configuration in a similar manner.
Describe the solution you'd like
I would like to upload/import dataset yaml files through the GUI.
Describe alternatives you've considered
I'm using a docker setup, so using the commandline tool requires significantly more effort like additional volumes etc.

+1, this is a very confusing UI to have an export button but not an import one.
If the feature is not completed then it's better to just disable both.


GET for /{db}/_partition returns 404 instead of 400

Description
I'm getting a 404 not_found when calling a GET to /{db}/_partition endpoint but expect a 400 bad_request similar to Cloudant given that the database exists.
Steps to Reproduce
$ curl http://localhost:5984/partitiondb/_partition
{"error":"not_found","reason":"missing"}

Expected Behaviour
I was expecting the response to be the same or similar to Cloudant:
curl https://ACCOUNT.cloudant.com/partitiondb/_partition
{"error":"bad_request","reason":"invalid_partition_req"}

Your Environment
I'm using the ibmcom/couchdb3 image.
{"couchdb":"Welcome","version":"3.0.0-a8f3ad9","git_sha":"a8f3ad9","uuid":"5d37b58c79457e5e44b52792ff9b45b8","features":["search","partitioned","pluggable-storage-engines","reshard","scheduler"],"vendor":{"name":"The Apache Software Foundation"}}

CouchDB Version used: 3.0.0-a8f3ad9
Browser name and version: using curl
Operating System and version: MacOS 10.15.1 (19B88)

Additional context

@garrensmith FYI


Feature Request - Make todo-trigger optional on Daily Notes Page

I'd love to see an option for the todo-trigger extension to be disabled on the daily notes page.
For my use case, on my daily page, I have lots of tasks that come in via SmartBlock (routines, etc.) but when ticking them off I obviously already know what day they belong to so it's a bit redundant since my trigger includes today's date. I love the use of the trigger elsewhere in Roam to keep track of what I did that day so something like this would be perfect.
Let me know if there are any questions about this use case and thanks for all of the awesome work @dvargas92495!

implemented with new{now:skip dnp} template option


bitRange support

cc @Emilgardis

We can also edit the range if LSB and MSB are edited separately by reading the current range and recalculating it. But it is better to do this logic in svd-rs

We can also edit the range if LSB and MSB are edited separately by reading the current range and recalculating it. But it is better to do this logic in svd-rs

It is more complex as can't be done fn make_field. Only in modify_field.


0.80.0b1 LIFX Component Won't Load

Home Assistant release with the issue:
0.80.0b1
Last working Home Assistant release (if known):
0.79.3
Operating environment (Hass.io/Docker/Windows/etc.):
Linux Debian (venv)
Component/platform:
LIFX
Description of problem:
LIFX component won't initiate. No lights added, errors in legs as shown below
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
light:
  - platform: lifx

Traceback (if applicable):
2018-10-06 11:25:08 ERROR (MainThread) [homeassistant.components.light] Error while setting up platform lifx
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/lib/python3.5/asyncio/tasks.py", line 400, in wait_for
    return fut.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/asyncio/tasks.py", line 239, in _step
    result = coro.send(None)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/light/lifx.py", line 148, in async_setup_platform
    lifx_manager = LIFXManager(hass, async_add_entities)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/light/lifx.py", line 215, in __init__
    self.effects_conductor = aiolifx_effects().Conductor(loop=hass.loop)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/light/lifx.py", line 133, in aiolifx_effects
    import aiolifx_effects as aiolifx_effects_module
  File "/srv/homeassistant/lib/python3.5/site-packages/aiolifx_effects/__init__.py", line 1, in <module>
    from .aiolifx_effects import Conductor, EffectPulse, EffectColorloop
  File "/srv/homeassistant/lib/python3.5/site-packages/aiolifx_effects/aiolifx_effects.py", line 207
    yield None
    ^
SyntaxError: 'yield' inside async function

Additional information:
No changes to config between versions.

Confirmed working again after manually changing aiolifx_effects back to 0.1.2 in lifx.py

I don't have Python 3.5 available, can you test aiolifx_effects 0.2.1 for me?


Handwritten digits recognition using ANN

Is your feature request related to a problem? Please describe.
I want to add this project of deep learning
Describe the solution you'd like
I will create the ANN to recognize the handwritten digits from the dataset with good accuracy.
Please assign his issue to me.
Thank You
Dhruv Saxena
Describe alternatives you've considered
No response
Additional context
No response
Code of Conduct

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

Please assign this project to me

Hi, we have many similar issues. Hence, we cannot assign you this one.


wandering-trapper

Link to the mod page
https://www.curseforge.com/minecraft/mc-mods/wandering-trapper
An estimate of the number of mobs introduced by the mod
2
Game version
1.18.1

has own support


Signed in Subscription id is not shown in plugin

Environment: Windows/IntelliJ14
As per the story board the plug in should show the signed in id info while creating the VM.
PFB the screen shot.

The Plugin is not able to show the signed in id.
PFB the screen shot.

This user info will show only when log in as user AAD credentials

In the latest build plugin  is not able to show the user info even if he has logged in through AD authentication.

Will be displayed only when the user uses the AAD to sign.

This issue is fixed and functionality is working as expected in the latest build.


Create spells pages

Summary
This adds the spell index and show pages.

Related Issues
Fixes #765

Checklist

[x] This pull request has a meaningful title and description.
[x] I have read and followed the Contributing guidelines.
[x] My code follows the conventions and coding style of this project.
[x] All tests pass and I have added tests for my changes
[x] I have updated necessary documentation

Screenshots (if applicable)

Spells Index Page:

Spell Show Page:

Footer:

Additional information
The list and details skeleton components have been moved and reused in character, book and spell pages

@sushma1031
Thanks for contributing! Looks pretty nice from the pictures so far.
Before I go more detailed in the review, could you please undo the pull / rebase from the master branch? Those changes are not relevant to your PR and should not be added. Thank you.

All right, I'll do rhat.

@danielschuster-muc I've made the changes. Please let me know if anything else must be modified.

Sure, sorry about forgetting to remove those comments, I'll remove them now.
Also, to clarify, as of this PR, there should be seperate skeleton components for each page, is that correct?

@danielschuster-muc I've made the required changes, let me know if there's anything else!

@danielschuster-muc fixed the responsiveness issue and resolved the merge conflict