Animate the Ledger illustrations

We'd like to animate the Ledger illustrations to add more delight to the experience and also reinforce the physical interactions required to operate the device. This could be done after the feature release, but we'd like to get ahead with the design.
Scope
We currently have 5 illustrations where we could add motion. It would be worth reviewing the illustrations to make them similar in proportions so it can be easy to transition from one to the other. We have plans to add transitions between each step, to add more fluidity and further enhance the experience.

Connect: plugging the Ledger into a computer
Connecting: Could be a subtle variation of the previous one (or we re-use 1)
Success: Green checkmark
Verify: focus on pressing both buttons and checkmark
Error: Generic failed ledger

Figma

Approach
We are going to work with @eugeniadigon and a motion designer for this. I believe we haven't implemented these kinds of animations before in the Wallet, so wanted to check with @fbwoolf @kyranjamie if there are any requirements regarding the format of the animations. Is there anything you'd want to test first? I'm thinking we could do an early test to check performance and file size if needed.

References

Sonos mostly for the subtle transitions between steps and animations, which we could add at a later stage.
Ledger has a very nice onboarding full of Ledgers (of course!)

https://user-images.githubusercontent.com/239215/158643574-2b579b03-0506-4ed1-8c7e-576ed2dd256f.mov
https://user-images.githubusercontent.com/239215/158643813-f17b576f-5daf-42a4-9c13-5ca5acc78bbd.mp4

Added references! cc/ @eugeniadigon

Love this.
I don't think there's much to consider by way of constraints. We'd likely use a library like react-motion or react-spring if we have a vector-based animation.
Assets will be bundled with the extension and loaded from disk, so file size is less of an issue than were this a website.

Would about the Lottie format? I think it's widely used for motion animation (specially coming from After effects and other GUI motion software)

@eugeniadigon found an additional one we could animate, the disconnected state. This one should probably not include the (!) as part of the illustration, as I'm still refining this screen. I'd consider the ledger only as part of the illustration/animation.

How do I run the animations in the Figma link?

@markmhx @landitus @kyranjamie

Here's the figma board with the storyboard/UI reference of each animation

And I'm attaching two files, one with the mp4 preview animations, and another one with the JSON files

Json-20220329T135128Z-001.zip
Mp4-20220329T135125Z-001.zip

@kyranjamie should I be looking at helping you with this or you are doing?

Until we have updated animations there isn't anything to do yet


Stick nav bar

we have to stick the Navbar, for bater user experience and easy to access- Home, Menu, Service, Contact us, Cart
Hey
Project Admin: Khushi
Assign me this issue, i can fixed it

Assign me this issue

please assign this issue to me @khushi-joshi-05

I'm looking forward to work on this issue @khushi-joshi-05  under GSSoC'24. It would be a pleasure for me to work in this issue. Kindly assign this issue to me.


npm version missing

scrolling is actually not an npm module (as mentioned in the readme). could you publish it to npm?
npm publish

Done :+1:


Run first test method again in second test method, even though solo is finalized and activity is finished.

From<EMAIL_ADDRESS>on September 25, 2011 20:22:08
What steps will reproduce the problem? 1. setUp() method:
super.setUp();
this.activity = this.getActivity();
this.solo = new Solo(getInstrumentation(), this.activity);
tearDown() method:
try {
this.solo.finalize();
} catch (Throwable e) {
e.printStackTrace();
}
this.activity.finish();
super.tearDown();
testMethod1():
Thread t = new Thread() {
public void run() {
//It will be timed out.
doSomeTestOperations();
}
};
t.start();
t.join(20);
fail("Time out");
testMethod2():
doSomeTestOperations();

Make testMethod1() time out, then check testMethod2() can be executed well or not.

Run test for this whole class.

What is the expected output?
testMethod1 is failed by time out.
testMethod2 is passed successfully.
What do you see instead?
testMethod1 is failed by time out.
Running testMethod1 content again in testMethod2, then get NullPointerException. What version of the product are you using? On what operating system? Robotium2.2
Android2.2 Please provide any additional information below. After executing testMethod1, it is really done tearDown().
So the solo is finalized and activity is finished.
That means testMethod1 is completed done.
Why testMethod1 is executed again in testMethod2?
NullPointerException details:
java.lang.NullPointerException
E/AndroidRuntime( 2661):        at com.example.test.DemoTest$1.run(DemoTest.java:99)
W/ActivityManager(  306): Error in app com.example running instrumentation ComponentInfo{com.example.test/android.test.InstrumentationTestRunner}:
W/ActivityManager(  306):   java.lang.NullPointerException
W/ActivityManager(  306):   java.lang.NullPointerException
I/ActivityManager(  306): Force stopping package com.example uid=10118
Original issue: http://code.google.com/p/robotium/issues/detail?id=165

From<EMAIL_ADDRESS>on September 25, 2011 23:55:33
Robotium does not support being run concurrently in different threads.
Status: Invalid

From<EMAIL_ADDRESS>on September 26, 2011 01:48:49
Thanks a lot.
In fact, what I want to do is set time out for test case.
So I want to check testMethod2 can be executed well after testMethod1 timed out.
Now, I think the root cause is that thread is not closed in testMethod1, then it effects testMethod2, am I right?
I tried to use method interrupt to close thread, but no help.
Is there any other ways to close thread?
Thank you very much.


Add FIPS selftest and get_result_selftest

reference: #1260
When the LIBSPDM_FIPS_MODE is open, the test result:

When the LIBSPDM_FIPS_MODE is closed, the test result:

Signed-off-by: Wenxing Hou<EMAIL_ADDRESS>

for selftest location, please refer to https://github.com/DMTF/libspdm/discussions/1406#discussioncomment-4712827

Please add all sym, deterministic asym algo at first.
Then probabilistic asym algo with hardcoded random.

This is enough as the first patch.
I recommend, we review the design at first, then merge, then continue with rest key exchange or probabilistic algo.


Add shared preferences module

Shared preferences module and new idea module for dagger related things (lib-android-dagger)

Current coverage is 52.79%

Merging #32 into master will decrease coverage by -0.54% as of 9758c13

@@            master     #32   diff @@
======================================
  Files           11      12     +1
  Stmts          195     197     +2
  Branches        16      16
  Methods          0       0
======================================
  Hit            104     104
  Partial          7       7
- Missed          84      86     +2

Review entire Coverage Diff as of 9758c13

Uncovered Suggestions

+3.04% via ...LayoutViewPager.java#26...31
+2.53% via ...ew/ToolbarUtils.java#16...20
+2.53% via ...LayoutViewPager.java#38...42
See 7 more...

Powered by Codecov. Updated on successful CI builds.

:+1:


ldeo_magic.py

doesn't seem to work at all.

@ltauxe, for each conversion script there is a working example in _PmagPy-cli.ipynb and one in pmagpy_tests/test_imports3.py.  (At least, they work in the sense that they run and create output files.)  Let me know more specifically what you are trying that isn't working, and I'll fix it!

It generates an output file, but the method codes are all wrong so neither demag_gui nor zeq_magic work properly.  they just plot the first point.
maybe it is because there seems to be an AF step before the thermal, but that never bothered sio_magic….
lisa

On Aug 15, 2018, at 8:51 PM, Lori J<EMAIL_ADDRESS>wrote:
@ltauxe https://github.com/ltauxe, for each conversion script there is a working example in _PmagPy-cli.ipynb and one in pmagpy_tests/test_imports3.py. (At least, they work in the sense that they run and create output files.) Let me know more specifically what you are trying that isn't working, and I'll fix it!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/415#issuecomment-413416996, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKrDT2I-iIw3hq5Q3wrjFi5hZxnSGkuks5uROxCgaJpZM4V-q2U.

oh i got it... the codelist can't be blank - must be set to "AF" or something.  on the command line that is -LP AF.  I fixed both notebooks.  I'll add a doc string to make that more clear.

great — the docstrings are totally minimal and definitely need more TLC.  Also, should codelist be set to “AF” as the default in ldeo(), instead of default “"?

On Aug 15, 2018, at 9:51 PM, Lisa Tauxe<EMAIL_ADDRESS>wrote:
oh i got it... the codelist can't be blank - must be set to "AF" or something. on the command line that is -LP AF. I fixed both notebooks. I'll add a doc string to make that more clear.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/415#issuecomment-413425363, or mute the thread https://github.com/notifications/unsubscribe-auth/ABwGIsA_a_141KA61RttVR90ppDR1lZSks5uRPpvgaJpZM4V-q2U.

I added a bunch of instructions to ldeo_magic and sio_magic.  the rest have at least something although meant for command line and not the convert_2_magic function....

AHHH my bad - codelist has to be set to something, e.g., ‘AF’.

On Aug 15, 2018, at 8:51 PM, Lori J<EMAIL_ADDRESS>wrote:
@ltauxe https://github.com/ltauxe, for each conversion script there is a working example in _PmagPy-cli.ipynb and one in pmagpy_tests/test_imports3.py. (At least, they work in the sense that they run and create output files.) Let me know more specifically what you are trying that isn't working, and I'll fix it!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/415#issuecomment-413416996, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKrDT2I-iIw3hq5Q3wrjFi5hZxnSGkuks5uROxCgaJpZM4V-q2U.


Moved comments in rootston keybinding example config

The comment strings are not stripped so the commands are not currently recognised.

Thanks!


Please add details to get Digits Consumer and secret keys(for digits-web apps w/o mobile)

I am trying to build a simple node.js + web app and wanted to use Digits. I spent so much time just to FIND consumer and secret keys. Eventually I realized, I need to first have an iOS or an Android fabric app and get keys from that app! duh!
BTW, Digits is Awesome!

Thanks for the feedback on this @rajaraodv, will share with the team!

Hey @rajaraodv, you can also get them from https://apps.twitter.com/. If you are talking about a Twitter app consumer key and secret.

Also, nice to see you @rajaraodv! Been a while. :)

@crw likewise :) My issue was specifically related to digits.
Some more feedback:
My big issue is that information is across **3 different places ** (one the regular apps.twitter.com, digits portal, and this new fabric portal). Docs are not clear and detailed enough to quickly find them.
Another note: Please ask your folks to make it easy to find stuff w/in fabric portal. It's fancy but SUPER hard to find stuff things. Everything is hidden inside small icons as if we are using this portal in an iPhone! :-1:
Things that are hard:

Three different portals
Find list of my apps to manage them
Find keys for any app
Add multiple callback urls to a given app (localhost and production callback urls)
Adding Digits Login, Tweet buttons (which is trivial) in regular jQuery but is a bit hard in JS frameworks like React. Some examples would be helpful.

@rajaraodv I ran into this exact issue while following this tutorial.
https://github.com/HellerCommaA/GSW-twit-fab-web has decent instructions if you're on OSX.

 Twitter keys results in 239, Bad Guest token..

Anil, see my post above. Twitter still hasn't fixed this.
https://github.com/HellerCommaA/GSW-twit-fab-web

Thank you @HellerCommaA , your post was very helpful. I am not from mobile app development experience and it was very difficult to setup xcode & android studio with working app to just get keys. Finally i solved using Android Studio. With XCode i saw many errors while building the app with DigitalKit.

Send me a PR with your instructions for Android studio! (if not, I'll get
to it this weekend)
Adam
On Sat, May 14, 2016, 09:39 Anil Sagar<EMAIL_ADDRESS>wrote:

Thank you @HellerCommaA https://github.com/HellerCommaA , your post was
very helpful. I am not from mobile app development experience and it was
very difficult to setup xcode & android studio with working app to just get
keys. Finally i solved using Android Studio. With XCode i saw many errors
while building the app with DigitalKit.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/twitterdev/cannonball-web/issues/1#issuecomment-219220914

Thanks for the feedback both, I know our web documentation isn't ideal if you're trying to just use Digits for Web without a corresponding mobile app. While you can use the web onboarding at https://fabric.io/kits/ios/digits, you would still need to create a mobile app and configure the URL on the Fabric Digits dashboard. This is in order to help prevent man-in-middle attacks by confirming that the Digits SDK is being hosted on a site with the correct API key.


Secure grpc start clean code

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: hzxuzhonghu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu]
pkg/OWNERS
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I like the simplification of this PR.
However, I'm wondering the background we introduce retry in the first place. Could it be due to the racing condition? Namely Citadel hasn't prepared key cert for Pilot (or k8s does not mount secrets into Pilot volume yet) when Pilot tries to use the key & cert to start grpc server.
Might be helpful to dig some archaeology a little bit :)
/cc @myidpt

Namely Citadel hasn't prepared key cert for Pilot

Fail this sounds reasonable.

(or k8s does not mount secrets into Pilot volume yet) when Pilot tries to use the key & cert to start grpc server.

It is impossible. K8S/docker  guarantee secret must be mounted before run the process. If secret is not exist, the pod would not run.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu, rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu,rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
ecd865c0b14d862e2abe912fb94f73f052678f7f
link
/test istio-unit-tests

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

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
ecd865c0b14d862e2abe912fb94f73f052678f7f
link
/test istio-unit-tests

prow/e2e-mixer-no_auth-mcp.sh
ecd865c0b14d862e2abe912fb94f73f052678f7f
link
/test e2e-mixer-no_auth-mcp

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

/test istio-unit-tests
unrelated ut TestFull flake


[rom_ext] Select address tranlation based on link address

Select address translation based on the link address of the .text section.  If the link address is in EFLASH, then address translation is false, else true.
Remove various manifest_virtual targets from BUILD files in favor of using the automatic selection in (1).

Signed-off-by: Chris Frantz<EMAIL_ADDRESS>(cherry picked from commit 4ccb0a366bedc0324c7b98e6ea336f2fab5d40cc)

Submitted with #23308.


make flash failed for esp32 with esp32-who (GIT) & esp-idf 3.2.2 (IDFGH-1460)

I need a new toolchain for ESP-WHO, which is compatible with the new MSYS2.org  (msys2-x86_64-20190524.exe) toolchain. Please help (more info below)
Environment

Development Kit:      ESP32-DevKitC
Kit version (for WroverKit/PicoKit/DevKitC): ?
Module or chip used:  ESP32-WROOM-32  (esp32-cam)
IDF version (run git describe --tags to find it):
// v3.2.2
Build System:         Make
Compiler version (run xtensa-esp32-elf-gcc --version to find it):
xtensa-esp32-elf-gcc.exe (crosstool-NG esp32-2019r1) 8.2.0
Operating System:     Windows
Power Supply:         USB

Problem Description
//Detailed problem description goes here.
"make flash"  failed
Expected Behavior
No error
Actual Behavior
make flash
Toolchain path: /opt/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc
WARNING: Toolchain version is not supported: esp32-2019r1
Expected to see version: crosstool-ng-1.22.0-80-g6c4433a
Please check ESP-IDF setup instructions and update the toolchain, or proceed at your own risk.
WARNING: Compiler version is not supported: 8.2.0
Expected to see version(s): 5.2.0
Please check ESP-IDF setup instructions and update the toolchain, or proceed at your own risk.
WARNING: Toolchain version is not supported: esp32-2019r1
Expected to see version: crosstool-ng-1.22.0-80-g6c4433a
Please check ESP-IDF setup instructions and update the toolchain, or proceed at your own risk.
WARNING: Compiler version is not supported: 8.2.0
Expected to see version(s): 5.2.0
Please check ESP-IDF setup instructions and update the toolchain, or proceed at your own risk.
CC build/bootloader/bootloader_support/src/bootloader_random.o
CC build/bootloader/bootloader_support/src/flash_encrypt.o
....
....
....
....
CC build/app_update/esp_ota_ops.o
AR build/app_update/libapp_update.a
CXX build/asio/asio/asio/src/asio.o
In file included from C:/Users/tuan/Documents/Arduino/esp-who/esp-idf/components/lwip/lwip/src/include/lwip/opt.h:51,
from C:/Users/tuan/Documents/Arduino/esp-who/esp-idf/components/lwip/lwip/src/include/lwip/sockets.h:42,
from C:/Users/tuan/Documents/Arduino/esp-who/esp-idf/components/asio/asio/asio/include/asio/error_code.hpp:21,
from C:/Users/tuan/Documents/Arduino/esp-who/esp-idf/components/asio/asio/asio/include/asio/error.hpp:19,
from C:/Users/tuan/Documents/Arduino/esp-who/esp-idf/components/asio/asio/asio/include/asio/impl/error.ipp:20,
from C:/Users/tuan/Documents/Arduino/esp-who/esp-idf/components/asio/asio/asio/include/asio/impl/src.hpp:22,
from C:/Users/tuan/Documents/Arduino/esp-who/esp-idf/components/asio/asio/asio/src/asio.cpp:11:
C:/Users/tuan/Documents/Arduino/esp-who/esp-idf/components/lwip/port/esp32/include/lwipopts.h:40:10: fatal error: sys/ioctl.h: No such file or directory
#include <sys/ioctl.h>
^~~~~~~~~~~~~
compilation terminated.
make[1]: *** [/c/Users/tuan/Documents/Arduino/esp-who/esp-idf/make/component_wrapper.mk:286: asio/asio/src/asio.o] Error 1
make: *** [/c/Users/tuan/Documents/Arduino/esp-who/esp-idf/make/project.mk:530: component-asio-build] Error 2
tuan@nhatrang MINGW32 /c/Users/tuan/Documents/Arduino/esp-who/examples/single_chip/camera_web_server
Steps to repropduce
Error during "make flash",  run from mingw32.exe (http://www.msys2.org/)
see below
Code to reproduce this issue
I use your code "camera_web_server" from https://github.com/espressif/esp-who
c:\Users\tuan\Documents\Arduino\esp-who\examples\single_chip\camera_web_server
Guide: https://docs.espressif.com/projects/esp-idf/en/stable/get-started/index.html and https://robotzero.one/esp-who-recognition-with-names/  and
http://www.msys2.org/
make menuconfig
make flash
Debug Logs
see errors from "make flash" above
Other items if possible
www.chezmoi.dk/sdkconfig
www.chezmoi.dk/bootloader.elf

[ ] sdkconfig file (attach the sdkconfig file from your project folder)
[ ] elf file in the build folder (note this may contain all the code details and symbols of your project.)
[ ] coredump (This provides stacks of tasks.)

please close the case, the issue can be read here (the post from David, 5 July 2019) , an old toolchain inside your esp-who, I need to git clone esp-idf as well.
https://robotzero.one/esp32-camera-module   ; step by step by robotzero
Thanks

see above, the compile is ok now


Error during automatic sleep staging

Hi. I was trying to get the automatically extracted sleep stages, but, while executing sls.predict() command, Python (Jupyter Lab) returned the following error.

I used an edf file loaded via mne.io.read_raw_edf function(...preload=True...): 4 channels (2 EEG, 2 EMG) with the same sf (200 Hz), then I executed sls = yasa.SleepStaging(raw, eeg_name="EEG1", emg_name="EMG1") which worked without errors for ~5 seconds, though the file is 26 hours long. I use the newest version of yasa (0.4.1), lightGBM (3.2.1) and antropy (0.1.4) are also installed. Seems like it cannot find the files containing pretrained classifiers. What could it be due to? Because manually I could find the files int the file system.

Thanks in advance.

I added a print() in the staging.py and figured out that path is correct but the classifiers files names are not. As can be seen from the OP post second screenshot, files names also contain a version before the extention, whereas print() indicated that clf_name is just, for example, "clf_eeg+emg", so it doesn't contain package version and cannot be found.

Hi @cyberfag,
Good catch -- I'll make sure to add a classifier for eeg+emg in the next release to avoid such errors in the future.
Thanks!
Raphael

Fixed in release 0.5.0.


[WelcomeUX] When overlay appears, can we get a blank editor behind it?

Testing #18091 on OS X El Capitan
The text on the overlay fights for attention with the text under the overlay.
Usually when websites have overlays, the background is greyed out/blurred and the text on the overlay can be clearly read.

We haven't implemented the new design for this, but it essentially does what you're describing.  It's coming :)


using automake to customize installation

This  would be a cleaner way to do the installation currently. Check out this SO post for info.

Superceded by #64 as both resolve the same issue, but CMake is a more modern way of resolving it.

Implemented CMake in PR #91 making this issue obsolete.


Trying to fix set is not ordered errors by using OrderedDict in place…

… of dict
Errors are found in this travis build which was caused by merging a few bugfixes from master into develop (not sure how, maybe updating the dockerfile? Does travis even use that file?)

Codecov Report

Merging #116 into develop will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #116   +/-   ##
========================================
  Coverage    96.17%   96.17%
========================================
  Files           17       17
  Lines         2299     2300    +1
========================================
+ Hits          2211     2212    +1
  Misses          88       88

Impacted Files
Coverage Δ

trtools/mergeSTR/mergeSTR.py
95.89% <100.00%> (+0.02%)
:arrow_up:

trtools/qcSTR/qcSTR.py
99.01% <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 8cd22e1...e7c6867. Read the comment docs.

I wasn't able to reproduce this locally (here's some specific version of Pandas that caused the issue and I couldn't figure out the version) so I didn't add any new tests around this, but it's now passing.


Updates definition of install_sources

Updates the definition of the install_sources's behaviour to be up to the implementor of the API, and details mechanisms that developers have to tailor the app's distribution to their needs.

Chatted offline. WRT my comment around browsers behaving differently, Diego pointed out that devs will already need to handle error cases, so for the scenario described above, the store can decide that, after a certain number of failed installations, it should prompt the user to navigate to the site and try to install it themself. Plus, if most browsers decide to allow by default, then this is a non-issue

Abandoning in favor of #793


Perform a performance sweep over Stage3D

There are some performance regressions (mainly on Intel Graphics) in the Starling benchmark demo since it was first announced on http://www.openfl.org/blog/2016/10/13/improved-support-for-starling/, it would be good to try the current OpenFL release with the Stage3D layer rolled back to an older version, then step forward and determine which consistency fix may be related to the performance difference

I don't think this applies since the Stage3D rewrite in OpenFL 8


Swagger example for GET /_api/user/{user}/database/ is not useful

The example shown in the above API call does not contain any results.
Also there is no documentation about the response body.
So only trying it out shows what the result looks like.

Should be fixed now in devel.

ArangoDB 3.1.13 is available for download.


Issues with HBO Max when using Widevine

I'm looking into https://github.com/nativefier/nativefier/issues/1153 and trying to determine the cause. We're packaging an Electron app based on your distribution (trying both 12.0.4 and 12.0.5), that contains a BrowserWindow pointed to https://play.hbomax.com/.
When trying to view content in this app, the video does not load, but instead displays:

Can't Play Title
Make sure your browser is not in private mode.

In the background the only event that fires is widevine-ready with version = '4.10.2209.1' and lastVersion = null
We are also listening for widevine-update-pending and widevine-error but neither of those are triggered.
Right before this occurs, the following request fails in the developer tools:
POST https://comet.api.hbo.com/drm/license/widevine?keygen=playready&drmKeyVersion=2 400
Interestingly enough, that is preceded by 1 successful call and followed with 1 successful call so I'm not sure if it's related:

Let me know if there is any more info I can provide.

There are a number of things that could be the culprit here, so first a few questions:

On what platform does this happen? Multiple platforms?
Are you waiting for the widevine-ready event before opening the HBO Max URL in a browser window?
Are you using EVS to VMP re-sign ECS for production use?
Do other commercial services work correctly?

Assuming the VMP signature is a correct production signature there are still some services that enforce other constraints, e.g. by checking the User-Agent string, and HBO Max has been mentioned in at least one other ticket, relating to this (see #80,  #91 might also be relevant). Actual verification that changing the User-Agent works in the case of HBO Max does not yet appear to be in any of the threads, but it might be worth a try.

I think we've got it worked out now, but to document for future use:

MacOS
Yes
I am now, and this seems to fix it.
Yes

As well I did try a few different User Agents, and this did not seem to have an affect. After signing, our default user agent works fine.


All extensions should be grouped under [extensions]

In examples/hil.cfg.dev-no-hardware  the [hil.ext.network_allocators.vlan_pool] is defined in the middle of extensions. The authentication extensions are defined after it.
Because of this when the [hil.ext.network_allocators.vlan_pool] is uncommented, running hil-admin db create fails, giving the following error:
$ hil-admin db create
ERROR: No authentication/authorization backend registered; make sure your hil.cfg loads an extension which provides the auth backend.
the authentication extensions are defined

Make sure that all the extensions are defined together under  [hil.ext.network_allocators.vlan_pool] fixes the issue.

Ah, I see what you mean. The error code is due to the fact that we are calling the network pool allocator before registering any auth backend.


Seems no way to tell whether a value is from cli input or default

In my app, the precedence of the value being used is like this: cli input > config file value > cli default value. Now if I get the value which happens to be the default cli value, I need to differentiate whether it's from cli input explicitly by the user, or it is from the cli default implicitly. This matters because it decides whether to use the happen-to-be default value or the value in the config file. So is there a way differentiate them? Thanks.

Essentially, when a user input option happens to be the same as the default value, there's no way to tell whether the value if from user input or the default value. They have different priorities.

Hi @elgs,
I'd need to look to double check how this plays with the new altsrc package, but you should be able to use context.IsSet to check if the user specified the flag.

Thanks @jszwedko! context.IsSet does the trick.


Evaluation Metrics

Look into the potential options to use for evaluating a LLM:
https://python.langchain.com/docs/guides/evaluation/
Research on cost evaluation for LLMs could also be useful if we want to compare things like runtime or resource usage in the future.
This task could also involve comparing the existing pre-trained models on Langchain and evaluating which ones are most appropriate for us to use:
https://python.langchain.com/docs/integrations/components

Just a heads up that the links under "LangSmith Evaluation" in the first link are broken, but further reading on LangSmith is available here: https://docs.smith.langchain.com/ (the "Evaluation" section here has "How-To Guides", including "How to use off-the-shelf LangChain evaluators")

"For simple queries, such as filtering by project, run time, name, or run ID's, you can directly use keyword arguments in the list_runs method."
https://docs.smith.langchain.com/tracing/faq/querying_traces#using-keyword-arguments

Have accomplished this task using Langfuse, which is automatically tracking all of the questions asked to the model and scoring every question/answer based on answer relevancy and faithfulness.


MRG: Refactor _get_ps_kwarg

This PR refactors _3d.py and removes a dependency to surfer. It should help with the following item of #7162 :

thx @GuillaumeFavelier


error in real_time_object_counting.py

when i ran this in colab i got like this.
-0.227492094039917
list index out of range
what is the problem?

Helo @navya-techv I'm beginner using google colab, can you give me some instructions to use this object counting in google colab? Thanks a lot

@navya-techv can you share stack trace?

@navya-techv can you share stack trace?


Update contact block information

Update contact infromation and avatar with own's

Its necesary to update de avatar file


Add a SyftPath class that standardizes all Path operations

Needs discussion
We can add a SyftPath that handles absolute, relative, and datasite path so we don't have to do that manually everywhere. In the sync plugin we're manually passing file paths, datasite roots, sync folder roots around which gets pretty confusing, SyftPath should solve that problem.
Not sure about the interface yet, but something like this:
class SyftPath(Pathlib.Path):
    def __init__(self, client: Client, path: Path | str | Any):
        path: Path = self.validate(path) # ensure this is a Path and relative to sync folder

        # these could be @property methods, depends on implementation
        self.root_dir: Path = client.sync_folder.resolve().expanduser()
        self.relative: Path = path.relative_to(self.root_dir)
        self.absolute: Path = self.root_dir / self.relative
        self.datasite: str = self.relative.parts[0]

    def to_json(self) -> str: # pydantic serialize
        return str(relative_path)

my_file =<EMAIL_ADDRESS>abs_path = my_file.absolute
...

This fixes the problem that the developer has to remember if a path is absolute, relative, validated, etc

I think it solves something different @yashgorana , we still have the issue that we're using both abs paths (for local operations) and relative paths (for server-client communication). I think something like this would be nice to make this clear


asynchronized or not, the document should clarify it.

I'm really confused that all API are asynchronized ? But after I dig into the simple-chaining.js example, I see the Statment#prepare API is synchronized at least, but all of them has a node-style callbacks. so the following code
    var stmt = db.prepare("INSERT INTO lorem VALUES (?)");
    stmt.run("lorem content");
    stmt.finalize();

may be identical to the following:
    var stmt = db.prepare("INSERT INTO lorem VALUES (?)", function(){
        console.log("second");
        stmt.run("lorem content");
        stmt.finalize();
    });
    console.log("first");

the output is as expected, printing 'first' followed by 'second', but I'm thinking the "key logical" inside to be synchronized ( the same as Statement#finallize and Statement#bind ? ), it behaves like a asynchronized one just because the callback is excuted in the next event loop, is that the truth ?

check my code, the async code is in the done callback, the codes should be like this:
stmt.prepare('INSERT INTO t1 VALUES (?)', function() {
  stmt.run('value1');
  stmt.finalize(done);
});

db.serialize(function(){
  // one by one
  db.wait(done);
});

The documentation does explain this in detail:

https://github.com/mapbox/node-sqlite3/wiki/Control-Flow
https://github.com/mapbox/node-sqlite3/wiki/API


How to get datasets

I haven't found the link or processing tool to get the dataset yet

The adopted datasets are the same as SIGIR21 SURGE. You can obtain the datasets at this repo

We have provided a Google Drive link to the adopted dataset. You can check data.


BUG: special cupy import guard

Fixes #21174

Don't fail the SciPy testsuite when the wrong CuPy/NumPy combination is installed, since CuPy is an optional dep.

Precedent is scipy/stats/tests/test_morestats.py, it catches the generic Exception for matplotlib import, which is even more aggressive.

[skip ci]

Thanks @tylerjereddy for fixing this. My intention was to only run these tests if there is a working CuPy and SCIPY_XSLOW=1. The problem is that I just copied the import guard for mpmath from test_mpmath.py.
https://github.com/scipy/scipy/blob/d48a13af42c10145b91031ba7a737ac9a439c29e/scipy/special/tests/test_mpmath.py#L25-L28
and didn't think things through and consider that importing CuPy could fail in ways beyond an ImportError. At the moment these tests are only meant to be run locally, though at some point they might run in a weekly job on GitHub actions. I'm +1 for being more aggressive and catching the generic exception.


Updating output files path for eQTL analysis

Results from the SAIGE-QTL pipeline ended up in an awkwardly called path with repeated output_files directory, e.g., gs://cpg-bioheart-main/saige-qtl/bioheart_n990_and_tob_n1055/output_files/v3/output_files/, so fixing that here.
While I am at it, also split results by chromosome and cell type to make them easier to navigate.
Because of the too-small genotype files in test cannot test this but hoping it is a small enough change that it is not a problem!

running to test new paths: https://batch.hail.populationgenomics.org.au/batches/483775
https://batch.hail.populationgenomics.org.au/batches/483776


Correct info between concurrency and ENI limits

Concurrency and memory size define ENI requirements.
Issue #, if available:
Inaccurate documentaion
Description of changes:
Clarify relationship between ENIs, concurrency, and memory size.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This topic now links to the VPC configuration page, which has this information.
Thanks,
-Michael


🐛 [firebase_remote_config] flutter pub giving error on getting 0.8.0-dev.0 with core 0.7.0

Bug report
Remote config with firebase_core 0.7.0 gives the following error on pub get:
Because firebase_remote_config >=0.8.0-dev.0 depends on firebase_remote_config_platform_interface ^0.1.0-dev.0 which depends on firebase_core ^0.5.2, firebase_remote_config >=0.8.0-dev.0 requires firebase_core ^0.5.2.

So, because **** depends on both firebase_core ^0.7.0 and firebase_remote_config ^0.8.0-dev.0, version solving failed.
pub get failed (1; So, because **** depends on both firebase_core ^0.7.0 and firebase_remote_config ^0.8.0-dev.0, version solving failed.)
exit code 1

Steps to reproduce
Steps to reproduce the behavior:

Add firebase_remote_config: ^0.8.0-dev.0 and firebase_core: ^0.7.0 to pubspec
Run a flutter pub get

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 1.22.5, on Microsoft Windows [versÃ£o 10.0.19042.746], locale pt-BR)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[!] Android Studio (version 4.1.0)
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
[√] VS Code, 64-bit edition (version 1.52.1)
[√] Connected device (1 available)

! Doctor found issues in 1 category.

Was 0.7.0 supposed to be a pre-release?

Was 0.7.0 supposed to be a pre-release?

Was 0.7.0 supposed to be a pre-release?

No, 0.7.0 is the latest release of core and needed by others updated firebase plugins, only remote_config is giving this error

Ah, I should've specified that I was asking about firebase_remote_config 0.7.0. There's a pre-release of 0.8.0-dev.0 and it's changelog contains notes for 0.7.0 but the latest resolvable version is 0.6.0
Anyway, the issue you're facing should be fixed via https://github.com/FirebaseExtended/flutterfire/pull/4689, they just need to publish a new release.

Was 0.7.0 supposed to be a pre-release?

No, 0.7.0 is the latest release of core and needed by others updated firebase plugins, only remote_config is giving this error

Ah, I should've specified that I was asking about firebase_remote_config 0.7.0. There's a pre-release of 0.8.0-dev.0 and it's changelog contains notes for 0.7.0 but the latest resolvable version is 0.6.0
Anyway, the issue you're facing should be fixed via https://github.com/FirebaseExtended/flutterfire/pull/4689, they just need to publish a new release.

Anyone have a workaround for this? I'm trying to go over to the "better packages" but I can't even get the package on my PC 😅

Anyone have a workaround for this? I'm trying to go over to the "better packages" but I can't even get the package on my PC 😅

Has this been fixed?  Makes this package almost impossible to use right now.

Has this been fixed?  Makes this package almost impossible to use right now.

@pinkfish no, they need to launch an update... For now, you can try dependency override, put this on your pubspec:
dependency_overrides:
  firebase_core: ^0.7.0

@pinkfish no, they need to launch an update... For now, you can try dependency override, put this on your pubspec:
dependency_overrides:
  firebase_core: ^0.7.0

Follow this Migration Guide to update the packages, but got:
Because firebase_remote_config >=0.8.0-dev.0 depends on firebase_remote_config_platform_interface ^0.1.0-dev.0 which depends on firebase_core ^0.5.2, firebase_remote_config >=0.8.0-dev.0 requires firebase_core ^0.5.2.

Any update??

Follow this Migration Guide to update the packages, but got:
Because firebase_remote_config >=0.8.0-dev.0 depends on firebase_remote_config_platform_interface ^0.1.0-dev.0 which depends on firebase_core ^0.5.2, firebase_remote_config >=0.8.0-dev.0 requires firebase_core ^0.5.2.

Any update??

The same for me...
Because firebase_remote_config >=0.8.0-dev.0 depends on firebase_remote_config_platform_interface ^0.1.0-dev.0 which depends on firebase_core ^0.5.2, firebase_remote_config >=0.8.0-dev.0 requires firebase_core ^0.5.2.

Have anyone found workaround?

The same for me...
Because firebase_remote_config >=0.8.0-dev.0 depends on firebase_remote_config_platform_interface ^0.1.0-dev.0 which depends on firebase_core ^0.5.2, firebase_remote_config >=0.8.0-dev.0 requires firebase_core ^0.5.2.

Have anyone found workaround?

Same problem here..

Same problem here..

Please thumbs up the original issue comment instead of commenting that you have the same issue unless you have anything unmentioned to contribute.
To be clear here, the problem is just with the very early and unfinished prerelease firebase_remote_config: ^0.8.0-dev.0. You can just use the latest firebase_remote_config: ^0.6.0 instead.

Using  the migration guide on FlutterFire with firebase_remote_config: ^0.6.0 doesn't work either

What issue are you specifically having with that version? @SeanZom

This should have been resolved since this issue has been made, please try the latest release.


[SPARK-9876] [BRANCH-2.0] Revert "[SPARK-9876][SQL] Update Parquet to 1.8.1."

What changes were proposed in this pull request?
Since we are pretty late in the 2.0 release cycle, it is not clear if this upgrade can be tested thoroughly and if we can resolve the regression issue that we observed before. This PR reverts #13280 from branch 2.0.
How was this patch tested?

Test build #59764 has started for PR 13450 at commit 0c08f31.

@rdblue  @liancheng Can you review this PR? This is for branch 2.0.

Test build #59766 has started for PR 13450 at commit 1d52cfb.

+1, looks fine to me assuming tests pass.

LGTM

Test build #59764 has finished for PR 13450 at commit 0c08f31.

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

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59764/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59766/
Test PASSed.

Merged build finished. Test PASSed.

Merging to branch-2.0.


Update plugin Request - Ghinja

I would like to ask for an update of my plugin Ghinja with the new release:
https://github.com/Martyx00/ghinja/releases/tag/1.7
PS: not sure if this is s till the way to request this :)

No need to request it at all anymore! We now automatically track and then manually update each plugin on our side. I'll push the update first thing tomorrow.

That sounds amazing :) Closing the issue.


Reply queue message not ack'ed on req-resp pattern

On request-response pattern, the reply queue has messages piled up not .ack ().  Saw previous issue where this bug should had been fixed.  But it is appear on my end.
Can anyone shed some light on this?  Thanks.
I am using

wascally 0.2.7,
node.js 0.10.36
rabbitMQ 3.5.4

server.post ('/test', function (req, res, next) {
  var body = req.body;
  rabbit.configure (local).then (function () {
    rabbit.request (exchange, {
      routingKey:"",
      type: "reqres.type",
      body: body
    })
    .then (function (response) {
      console.log ('received response...'.success);
      console.log (response);
      response.ack ();
      res.send ();
      return next ();
    })
    .then (undefined, function (err) {
      next (new restify.errors.BadRequestError (err));
      handleError (err);
    });

    rabbit.handle("reqres.type", function(message) {
      try {
        console.log ('');
        console.log ('receiving msg..'.success);
        console.log(message);

        message.reply({
          foo: "replied bar"
        });
      } catch(err) {
        message.nack();
      }
    });

    rabbit.startSubscription ('user.create');
  })
  .then (undefined, handleError);
});

receiving msg..
{ fields:
   { consumerTag: 'amq.ctag-Xb3S6BUP9Kos6geuTPWMhw',
     deliveryTag: 1,
     redelivered: false,
     exchange: 'user',
     routingKey: '' },
  properties:
   { contentType: 'application/json',
     contentEncoding: 'utf8',
     headers: {},
     deliveryMode: undefined,
     priority: undefined,
     correlationId: '',
     replyTo: '82547150-3758-11e5-a17f-493c531269a7.response.queue',
     expiration: undefined,
     messageId: 'ed3d85b0-3758-11e5-a17f-493c531269a7',
     timestamp: undefined,
     type: 'reqres.type',
     userId: undefined,
     appId: '',
     clusterId: undefined },
  content: <Buffer...>,
  body: { foo: "bar"},
  ack: [Function],
  nack: [Function],
  reject: [Function],
  reply: [Function],
  type: 'reqres.type' }

received response...
{ fields:
   { consumerTag: 'amq.ctag-LWBka6t32Q5rbIPkQ26CtQ',
     deliveryTag: 37,
     redelivered: false,
     exchange: '',
     routingKey: '82547150-3758-11e5-a17f-493c531269a7.response.queue' },
  properties:
   { contentType: 'application/json',
     contentEncoding: 'utf8',
     headers: { sequence_end: true },
     deliveryMode: undefined,
     priority: undefined,
     correlationId: 'ed3d85b0-3758-11e5-a17f-493c531269a7',
     replyTo: '82547150-3758-11e5-a17f-493c531269a7.response.queue',
     expiration: undefined,
     messageId: undefined,
     timestamp: undefined,
     type: 'reqres.type.reply',
     userId: undefined,
     appId: undefined,
     clusterId: undefined },
  content: <Buffer ...>,
  body: { foo: 'replied bar' },
  ack: [Function],
  nack: [Function],
  reject: [Function],
  reply: [Function],
  type: 'reqres.type.reply' }

@pocheng - I don't think the example code you have will work, and I'd like to suggest some improvements as well.
Move your configuration out of the request block
I would move configuring the topology into a function that happens as your service starts up and only start the HTTP server after it's in place. This is the pattern we use for our services and it works well. If connectivity is lost, wascally will automatically re-establish the entire topology for you when it re-connects. There is no need to re-call configure.
Bug in the response handler?
You have the line: console.log ('received response...'.success); which might throw an exception. I usually recommend wrapping message handling blocks in try/catch so that you consistently ack or reject messages.
Let me know if you are continuing to see this issue. I haven't been able to reproduce it but there may be other differences in how I've set up my topology.


How can I build the pkg in windows?

How can I build the pkg in windows?
Can you provide a compiled python installation package for tensorflow-gpu 2.0.0rc1?  Just like warp-rnnt
thanks!

I'm not working on Windows. I'd suggest you to install anaconda, and manage related tools using conda.

I'm not working on Windows. I'd suggest you to install anaconda, and manage related tools using conda.

fine, I will try it on linux,Thanks


Dialog doesn't handle with async callback

The onCancel and onConfirm function could be async too.

mentioned in merge request team-tecnologia/planfy!151


[router] implement addLiquidityFor(), removeLiquidity(), addRelayerFee()

seems like it's not required in phase 0


Add National Open University of Nigeria (noun.edu.ng)

This pull request adds the National Open University of Nigeria to the list of educational institutions.
Official website: http://www.nou.edu.ng
Proof of domain: The institution uses the domain noun.edu.ng for its official email addresses.
IT-related course: [Link to a long-term IT-related course offered by the university](https://nou.edu.ng/programmes/

)

@codertjay I am sorry, the domain noun.edu.ng is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Prompt_tone does not stay disabled

HA core version
2024.11.1
Current integration version
0.6.5
Last known working integration version
georgezhao2010 version
Device type and model
Air Conditioner 00000Q17 (0)
Working mobile app
SmartHome
The detail description of bug/problem
I am trying to disable prompt_tone so my units never beep. In the old georgezhao2010 integration, I was able to change the default from FALSE to TRUE in the Python code itself. I can't find a place to do that in the current code. I tried adding a custom parameter like this using both "false" and "0". The temp step gets set correctly, but the prompt tone turns back on with every reboot of Home Assistant.

I am able to disable it using the "Extra controls" toggle, but this only lasts until rebooting.
The logs
I think this is the relevant debug log section. If not, please let me know.
Midea log.txt

Here is how I used to change it in the old version:

I'm sorry, I don't know why you can't add prompt tone as a extra control, then you can control it without any setting.
please check the detail.
if it still can't work for your device, could you share me your device SN and provided your device info in midea_ac_lan device detail web UI.
below is my AC device :

once it done, I can control it without any setting.
so we need to confirm your device protocol/type/subtype/etc to confirm your issue.

I CAN add prompt tone an extra control and it works fine. I'm trying to permanently disable prompt tone for all units, but it turns back on every time Home Assistant restarts.
I spent hours looking at the code and can't figure out where the current version is getting this default state of "true" (enabled) from.
Where that attribute is declared in the code for an AC class device, I tried adding "default: False", but even that does not work.

The default value probably comes from the midea-local lib, see screenshot.

Thanks, that's what I was looking for! But where is this library stored? It doesn't seem to be where the integration is installed, inside custom_components/midea_ac_lan. I installed this with HACS if that matters.

You can find out the path of midea-local as follows, see: https://github.com/wuwentao/midea_ac_lan/issues/319#issuecomment-2371003648

Thank you! I will use that fix for now.
Let me know if you need anything else from me to get the value in the "customize" box working in the future.

thanks @erikgieseler to help for this issue.
@BenJewell  it's a special feather requirement: try to change the device attribute default value.
in addition, I have a holiday for some weeks now, and current github feature support issue will have some delay.
anyway, I will check with the details to confirm with it after some weeks.

Thanks @wuwentao for following up on this!
I disagree with it being a special requirement for two reasons:

It defies the typical behavior of the Midea hardware in that the AC devices remember not to beep if you turn off beeping from the remote or the official app, and simply running the Home Assistant integration breaks this existing functionality. This seems to be model specific. For example the wall unit and ceiling cassette mini splits from the same series behave differently.
This one is more of an opinion but beeping is loud and useless so the vast majority would probably not want it. There aren't even guides online on how to break the pizio electric speaker.

That being said, I respect that you are volunteering your time for this effort and are free to prioritize things however you want. I always appreciate any contributions!

Thanks @wuwentao for following up on this!
I disagree with it being a special requirement for two reasons:

It defies the typical behavior of the Midea hardware in that the AC devices remember not to beep if you turn off beeping from the remote or the official app, and simply running the Home Assistant integration breaks this existing functionality. This seems to be model specific. For example the wall unit and ceiling cassette mini splits from the same series behave differently.
This one is more of an opinion but beeping is loud and useless so the vast majority would probably not want it. There aren't even guides online on how to break the pizio electric speaker.

That being said, I respect that you are volunteering your time for this effort and are free to prioritize things however you want. I always appreciate any contributions!

midea device can storage default value setting,  if we want to have the same feature, we also need to storage these attribute and latest value to HA storage, and not only for prompt_tone, most of these attributes and values should be saved for future.
in addition, from HA integration side and midea_ac_lan, it's two different software and solution, so it should be a new feature support, and we can try to add the same software feature in HA like midea hardware device, there will be too many feature need to be confirm, for example, we need to save these attributes setting and try to load it when device reboot.

Okay thanks! I didn't realize it was that complex. I see what you mean about not being able to change the default on the Midea side since it seems to be hard coded into the unit firmware basically.


[Automated PR] Resynced OWNERS file for backplane-2.2

Updated OWNERS to match source of truth.

/lgtm


add option to define logging level

added
  -l {DEBUG,INFO,WARNING,ERROR,CRITICAL}, --logging_level {DEBUG,INFO,WARNING,ERROR,CRITICAL}
                        logging level

@pgehring can you please review :slightly_smiling_face:


Implement Issue Reminder Workflow🚀

Description: Create a GitHub Actions workflow that sends reminders on open issues without updates. The workflow should post a comment prompting assignees to take action.
Expected Benefits:
Improved responsiveness and prioritization of tasks.
Increased overall productivity.
please assign me with level3 label @DeadmanAbir

Description: Create a GitHub Actions workflow that sends reminders on open issues without updates. The workflow should post a comment prompting assignees to take action.
Expected Benefits:
Improved responsiveness and prioritization of tasks. Increased overall productivity.
please assign me with level3 label @DeadmanAbir

attach screenshots of examples


"cache" scale colors and styles

because visual patterns are important to Lanttern, we use ordinal values and numeric scales start and stop bg_color/text_color a lot. the way we are doing it right now is not efficient, because we repeat the same process (described below) for the same scales multiple times per session.

query the scale with the ordinal values preloaded
run a Enum.map/2 to create styles based on the scale (something like "background-color: #{ov.bg_color}; color: #{ov.text_color}"

maybe it would be great to have some kind of structure pre-built to optimize resources (mat view? ETS? other solution?).
for ordinal values, we could have "style maps" with id as keys and styles as values in the format:
style_map = %{
  1: "background-color: #000000; color: #ffffff",
  2: "background-color: #000000; color: #ffffff",
  ...
}

<div<EMAIL_ADDRESS>/>

for numeric scales, we would need a function to calculate the color "on the fly" based on start and stop values. something like
# given a scale from 0 to 10, with
# start_bg_color = #000000 and stop_bg_color = #808080
# start_text_color = stop_text_color = #ffffff
get_score_style(5)
# > "background-color: #404040; color: #ffffff"

one minor improvement on this issue was the creation of Assessments.EntryParticleComponent in #203
it doesn't solve the numeric scale color issues, but improves the ordinal values preload. instead of preloading ordinal values in assessment point entries (we can have thousands of entries in the same page/view — e.g. report card students assessment tracking), we use the live component update_many/1 callback to map the distinct ordinal values ids needed, fetch them, and build the style maps


H7021 - Can't change color / brightness

What issue do you have? Please be as thorough and explicit as possible.
When trying to change light color it always reverts to "rgb 120 46 3"
Changing brightness results in incorrect brightness. When changing the brightness from within the Govee app, homebridge spits out a much different brightness percentage. For example: 50% brightness in govee app = 20% in home bridge, 100% in govee = 39% in homebridge.
Details of your setup.
Homebridge v1.6.0 - homebridge-config-ui-x v4.50.2
homebridge-govee v7.9.3 - just installed, have not updated.
Docker on Raspberry Pi 4
Govee LED String Lights - H7021
Please paste any relevant logs below.
[3/12/2023, 4:41:40 PM] [Govee] [LED Bulb String Lights] current colour [rgb 0 255 0].
[3/12/2023, 4:41:42 PM] [Govee] [LED Bulb String Lights] current colour [rgb 120 46 3].
[3/12/2023, 4:41:46 PM] [Govee] [LED Bulb String Lights] current colour [rgb 79 103 255].
[3/12/2023, 4:41:48 PM] [Govee] [LED Bulb String Lights] current colour [rgb 120 46 3].
[3/12/2023, 4:41:50 PM] [Govee] [LED Bulb String Lights] current colour [rgb 204 219 255].
[3/12/2023, 4:41:52 PM] [Govee] [LED Bulb String Lights] current colour [rgb 120 46 3].
[3/12/2023, 4:41:55 PM] [Govee] [LED Bulb String Lights] current brightness [63%].
[3/12/2023, 4:41:56 PM] [Govee] [LED Bulb String Lights] current brightness [63%].
[3/12/2023, 4:41:58 PM] [Govee] [LED Bulb String Lights] current brightness [45%].
[3/12/2023, 4:42:00 PM] [Govee] [LED Bulb String Lights] current brightness [45%].
[3/12/2023, 4:42:03 PM] [Govee] [LED Bulb String Lights] current brightness [30%].
[3/12/2023, 4:42:04 PM] [Govee] [LED Bulb String Lights] current brightness [30%].

v8.0.0-beta.4 fixed the issue.


upstream connect error when cluster address is localhost

Hi! I started microservice without docker and i want get this from envoy. Envoy started with docker. And this is not working. Microservice without envoy is available and works.
My config:
  access_log_path: /tmp/admin_access.log
  address:
    socket_address: { address: <IP_ADDRESS>, port_value: 9901 }
static_resources:
  listeners:
    - name: listener_0
      address:
        socket_address: { address: <IP_ADDRESS>, port_value: 80 }
      filter_chains:
        - filters:
            - name: envoy.http_connection_manager
              config:
                stat_prefix: ingress_http
                route_config:
                  name: local_route
                  virtual_hosts:
                    - name: local_service
                      domains: ["*"]
                      routes:
                        - match: { prefix: "/" }
                          route: { cluster: my_service }
                http_filters:
                  - name: envoy.router
  clusters:
    - name: my_service
      connect_timeout: 0.25s
      type: LOGICAL_DNS
      lb_policy: ROUND_ROBIN
      hosts: [{ socket_address: { address: localhost, port_value: 8102 }}]

Envoy logs:
[2020-06-03 10:38:35.828][17][debug][http] [source/common/http/conn_manager_impl.cc:268] [C0] new stream
[2020-06-03 10:38:35.828][17][debug][http] [source/common/http/conn_manager_impl.cc:781] [C0][S3580059227218846830] request headers complete (end_stream=true):
':authority', 'localhost:8000'
':path', '/app/ping'
':method', 'GET'
'user-agent', 'curl/7.68.0'
'accept', '*/*'

[2020-06-03 10:38:35.828][17][debug][http] [source/common/http/conn_manager_impl.cc:1333] [C0][S3580059227218846830] request end stream
[2020-06-03 10:38:35.828][17][debug][router] [source/common/router/router.cc:477] [C0][S3580059227218846830] cluster 'my_service' match for URL '/app/ping'
[2020-06-03 10:38:35.828][17][debug][router] [source/common/router/router.cc:634] [C0][S3580059227218846830] router decoding headers:
':authority', 'localhost:8000'
':path', '/app/ping'
':method', 'GET'
':scheme', 'https'
'user-agent', 'curl/7.68.0'
'accept', '*/*'
'x-forwarded-proto', 'http'
'x-request-id', 'b7aacc7c-f391-4c6a-95f8-7b3df0593c46'
'x-envoy-expected-rq-timeout-ms', '15000'

[2020-06-03 10:38:35.828][17][debug][pool] [source/common/http/conn_pool_base.cc:337] queueing request due to no available connections
[2020-06-03 10:38:35.828][17][debug][pool] [source/common/http/conn_pool_base.cc:47] creating a new connection
[2020-06-03 10:38:35.828][17][debug][client] [source/common/http/codec_client.cc:34] [C1] connecting
[2020-06-03 10:38:35.828][17][debug][connection] [source/common/network/connection_impl.cc:727] [C1] connecting to [::1]:8102
[2020-06-03 10:38:35.828][17][debug][connection] [source/common/network/connection_impl.cc:740] [C1] immediate connection error: 99
[2020-06-03 10:38:35.828][17][debug][connection] [source/common/network/connection_impl.cc:504] [C1] raising immediate error
[2020-06-03 10:38:35.828][17][debug][connection] [source/common/network/connection_impl.cc:200] [C1] closing socket: 0
[2020-06-03 10:38:35.828][17][debug][client] [source/common/http/codec_client.cc:91] [C1] disconnect. resetting 0 pending requests
[2020-06-03 10:38:35.828][17][debug][pool] [source/common/http/conn_pool_base.cc:265] [C1] client disconnected, failure reason:
[2020-06-03 10:38:35.828][17][debug][router] [source/common/router/router.cc:1018] [C0][S3580059227218846830] upstream reset: reset reason connection failure
[2020-06-03 10:38:35.829][17][debug][http] [source/common/http/conn_manager_impl.cc:1475] [C0][S3580059227218846830] Sending local reply with details upstream_reset_before_response_started{connection failure}
[2020-06-03 10:38:35.829][17][debug][http] [source/common/http/conn_manager_impl.cc:1706] [C0][S3580059227218846830] encoding headers via codec (end_stream=false):
':status', '503'
'content-length', '91'
'content-type', 'text/plain'
'date', 'Wed, 03 Jun 2020 10:38:35 GMT'
'server', 'envoy'

[2020-06-03 10:38:35.829][17][debug][connection] [source/common/network/connection_impl.cc:558] [C0] remote close
[2020-06-03 10:38:35.829][17][debug][connection] [source/common/network/connection_impl.cc:200] [C0] closing socket: 0
[2020-06-03 10:38:35.829][17][debug][conn_handler] [source/server/connection_handler_impl.cc:86] [C0] adding to cleanup list
[2020-06-03 10:38:40.202][1][debug][upstream] [source/common/upstream/logical_dns_cluster.cc:100] starting async DNS resolution for localhost
[2020-06-03 10:38:40.202][1][debug][upstream] [source/common/upstream/logical_dns_cluster.cc:108] async DNS resolution complete for localhost
[2020-06-03 10:38:40.202][1][debug][upstream] [source/common/upstream/logical_dns_cluster.cc:153] DNS refresh rate reset for localhost, refresh rate 5000 ms
[2020-06-03 10:38:40.211][1][debug][main] [source/server/server.cc:177] flushing stats

I don't know what I'm doing wrong

The trouble was docker's networking

@AlexDaSoul Could you provide more details on this issue? Is there any workaround for this? Thanks

@AlexDaSoul Could you provide more details on this issue? Is there any workaround for this? Thanks

The problem was that the specified hosts are inside the docker network and they are not the same as the addresses on your local computer.


Add ability to output in plaintext

This creates the files and outputs in plain text instead of JSON strings. This is helpful for passing into other functions via command line (e.g. linters).
Note that I manually edited the .js files because I couldn't get tsc working.

Hey @dorner thank you so much for your contribution.
I love the idea of having this option I'll work on getting it merged.

Thank you very much for the contribution.  I am cleaning up the repo and will go over contribution there and that should resolve #3 and provide easier deployment.

Wow, that was fast! 😮 great job, will test this out!

@trilom this isn't tagged yet so I can't try it out...?


Remove brackets in title causing localizing issue

Story 1510143
Content Update - Localization fix: L81041: Unescaped bracket in the title causes issues with localization

#sign-off

Thank you, @dksimpson!


Heading: add new component

Describe the feature you would like to see added
I have a project where we dynamically include piece of content where the level of the HTML headings is variable. So I suggest to add a heading component with a level attribute. This would allow you to do:
<pe:heading level="2">Hello world</pe:heading>

to generate

Hello world

See also:
https://github.com/primefaces/primefaces/issues/9527
Is your feature request related to a problem?
No response
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

If level is over 6 we can fallback to a paragraph with a class:
<pe:heading level="7">Hello world</pe:heading>

generates

Hello world

Following will throw an exception:
<pe:heading level="0">Hello world</pe:heading>
<pe:heading level="#{null}">Hello world</pe:heading>
<pe:heading level="wrong">Hello world</pe:heading> (not sure if we want the type to be string though)

I like that idea for anything > 6 !

I'm not too sure if we should add this component. Should be fairly easy to write as a custom tag.


Show descriptive component selection reasons in dependencyInsight report

With richer component selection reasons available in the ResolutionResult API, these reasons can easily be displayed in the dependencyInsight report.

This is implemented as part of #3830


Implement Warning Dialog on attempted navigation site wide for MeasureGroups dirty state

Implement 4491
Remove React context.
Fix dirty state issues from updating form fields with useEffects.
Refactor tests.
Implement routehandler and dialog.
MADiE PR
Jira Ticket: MAT-4491
(Optional) Related Tickets:
Summary
All Submissions

[x] This PR has the JIRA linked.
[x] Required tests are included.
[x] No extemporaneous files are included (i.e Complied files or testing results).
[x] This PR is merging into the correct branch.
[x] All Documentation needed for this PR is Complete (or noted in a TODO or other Ticket).
[x] Any breaking changes or failing automations are noted by placing a comment on this PR.

DevSecOps
If there is a question if this PR has a security or infrastructure impact, please contact the Security or DevOps engineer assigned to this project to discuss it further.

[ ] This PR has NO significant security impact (i.e Changing auth methods, Adding a new user type, Adding a required but vulnerable package).
[ ] All CDN/Web dependencies are hosted internally (i.e MADiE-Root Repo).

Reviewers
By Approving this PR you are attesting to the following:

Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose.
The tests appropriately test the new code, including edge cases.
If you have any concerns they are brought up either to the developer assigned, security engineer, or leads.

Git leaks secrets scanner is a false positive on a fake credentials for a test


fix(Modal): prevent unnecessary renders

Fixes #969
We continuously check the height of the Modal to determine if it should be scrolling or not. When we made Modal autocontrolled, it broke the check we were using to determine if we should call setState.
This fixes the check and makes it faster and more future-proof so if more keys are added to state it won't unexpectedly break again.

Thanks much!

Released in<EMAIL_ADDRESS>


fix: converting Pinecone metadata fields from float back to int

Proposed Changes:
Pinecone converts all meta numbers in the meta field to float (https://docs.pinecone.io/guides/data/filter-with-metadata). This causes the SentenceWindowRetriever to crash completely.
This PR checks if the metadata values are floats and convert them back to integers making Pinecone supported by the SentenceWindowRetriever
How did you test it?

manual verification, unit tests and integration tests

Checklist

I have read the contributors guidelines and the code of conduct
I have updated the related issue with new insights and changes
I added unit tests and updated the docstrings
I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.

Do you think an extra unit test is necessary? It's a private function, and testing its functionality is already covered by other test

Do you think an extra unit test is necessary? It's a private function, and testing its functionality is already covered by other test

I'll leave it up to you; I'd like to have it.

Do you think an extra unit test is necessary? It's a private function, and testing its functionality is already covered by other test

I'll leave it up to you; I'd like to have it.


MINOR: Refactor cleanupGroupMetadata

Refactoring avoids the need to call this method with a infinity as current time to remove all group offsets (when manually deleting the group).
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@hachikuji I took a first crack at the refactoring you suggested on cleanupGroupMetadata. It seems to me a simple Map[GroupMetadata, Seq[TopicPartition]] does the job for the newly introduced method signature. But for now I went with your suggested signature. For the remaining parts I tried to move the common code into its own method.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-test-coverage/236/

@hachikuji thanks for the nice improvement suggestion, and sorry for the late response. I submitted an update based your feedback. Test failures do not seem to be related.


Added hashCode and equals methods to persistence query objects.

@clicktravel-basharat Review complete, one comment raised.

@clicktravel-steffan Made hashCode and equals methods for AttributeQuery use super()


fix GridItem calcXY error calculation

Fixed: #918
Issue:
GridItem calcXY calculation is wrong.
Solution:
Change the calculation of the left and top value. The new calculation is left = colWidth * x + margin * x + containerPadding

Indeed #918 is a real bug. If we can get codesandbox working on PRs it would be great to try this out, otherwise we can pull & compare.

Verified this fixes the issue locally with the example grid and:
          containerPadding={[0, 0]}
          margin={[100, 10]}

However we need a regression test.

@STRML Sorry, I don't know how to write a regression test for this fix. Do you mean add a example in the test directory?

Yes, something in test/spec that fails under the old code but succeeds under the new.

Yes, something in test/spec that fails under the old code but succeeds under the new.

Sorry, I don't know what fails under the old code. I didn't change test/spec and my pr has passed the test.

The intention here is to write a new test that ensures that your fix doesn't get removed in the future and to ensure that the new behavior we've just identified as "good" stays good.

@STRML  I find test/spec only test function, but GridItem.jsx is a react component. This is my first time to commit pr, so I don't know how to add a test for react component. Could you give an example?

Yeah, we do it in https://github.com/mzabriskie/react-draggable/blob/master/specs/draggable.spec.jsx using react-dom/test-utils & react-test-renderer. Enzyme is also an option.

Ok, I will look at it first and then spend time adding the test

@STRML I have added a test case for the fix.But it show Run actions/labeler@v2 failed,

##[error]HttpError: Resource not accessible by integration
##[error]Resource not accessible by integration
##[error]Node run failed with exit code 1

what does it mean?

I put up an updated PR (#2059) that fixes this and includes updated tests.


[AprilHorizon] Created basic UI for interacting with backend

Trello task: Create a basic UI
This pull request introduces a user interface for our project so that we can prevent non-systematic behavior. An iOS app has been developed for this purpose.
The UI app makes HTTP calls to our existing codebase and displays&manipulates the data.
The first screen displays the list of accounts. Users should be able to create accounts.
The detail screen allows deposit, withdrawal, and update name actions. This screen is navigated by selecting an account from the list.
This also provides a UI that can be helpful to do a demo during the presentation.
Updated super-linter job in the workflow to only check account-management directory since the UI is not written with Java.
See the reference for including and excluding paths in the super-linter. This allows the definition of the environment variable for the desired directory.
https://user-images.githubusercontent.com/29355355/169840796-9e8ba9c7-fdd3-454e-a7d6-98c835eb951d.mov

 Create a basic UI


Pure python rewrite

Here's the big one!
This PR would thank @funkey's C++ code for its service and move to a pure python interface where we dispatch to gurobipy or PySCIPOpt for the backend implementations (both of which are available on pypi and conda!).
This makes ilpy (and therefore motile) dramatically easier to distribute, even just with pypi (no conda needed at all).  End users can pip install gurobipy or conda install -c gurobipy if they have a license and want to bring it in.
all tests are currently passing locally with essentially no changes to the test suite.  So i think the public API should be essentially untouched.
cc @cmalinmayor, @funkey
closes #68
closes #69

still need to reimplement callbacks


Update channel improvements

Also documented two methods for voice connections. (pause and resume)

I wanted to change all function(args){ to function(args) { but wasn't sure if I should. :P


Some class from sample app should be in the library

While testing MvRx it happened I did a lot of code copy/paste from the sample app. Mainly theses classes : MvRxEpoxyController, BaseFragment (minus the androidx.navigation part), MvRxViewModel. I think it would really help to have theses class in the library. Especially the epoxy related classes. May be in an optional library when we combine MvRx with Epoxy. To add on this, some of the class from sample app are referenced in the doc, it is a bit confusing to not see them in the main library (ex: https://github.com/airbnb/MvRx/wiki/Advanced-Concepts#mvrxviewmodel). Thanks!

I think it's reasonable to provide some of this in a module that we ship.
The downside is that so much of the fragment implementation is up to how you choose to use MvRx.
Playing devil's advocate, how hard is it to copy 1 or 2 files? We could update the documentation to be more clear about where the files live.

@eboudrant @elihart There was a reason for each one:

MvRxViewModel: it requires you to pass debugMode into super. You should use the BuildConfig constant from your application package which we can't do for you.
BaseMvRxFragment. You should also have a BaseFragment in your app in which you can put shared code. We recommend having your app's BaseFragment extend BaseMvRxFragment and do not recommend using BaseMvRxFragment directly.

I get that this is slightly confusing so I added a few sentences to this wiki page explaining this.


Multi GPU training

Hi, I ran into some problems while trying to launch training on multi gpu (one machine):

First, there were problems with DistributedDataParallel missing some fields in gigagan_pytorch.py: self.D.multiscale_input_resolutions in line 2159 and self.G.input_image_size in 2095. For a quick fix I added module before referencing a field and moved on.
Then I got a following error: RuntimeError: Inplace update to inference tensor outside InferenceMode is not allowed.You can make a clone to get a normal tensor before doing inplace update originating from gigagan_pytorch.py line 2514 (2479). I tried to clone the generator output for a quick fix (as well as sample function output), but it didn't help.

I use accelerate for the first time, so no prior experience - while running on single GPU everything works fine.
Since multi gpu training is marked in your todo list as done, I suppose the issue is on my side - but maybe you have some idea what causes those problems? I use torch 2.0.1+cu117, accelerate 0.22.0.
Anyway, thanks for great work!

@dprze thanks Dominika!
took care of the first bullet point, but wasn't sure if you gave me the right line for the second, as i don't see any inplace updates
did you mean this line? https://github.com/lucidrains/gigagan-pytorch/blob/main/gigagan_pytorch/gigagan_pytorch.py#L2479

@dprze spotted a few issues and put in a few fixes
i don't have access to my deep learning rig (w/ multi-gpus), so let me know if it works or not

@lucidrains thank you, now I'm able to run multi gpu training - although for some reason one iteration takes longer than on single gpu ;)
sorry for the line mix-up, it was 2479 indeed, but I was confused as I couldn't find the troublesome inplace operation myself

no problem! happy training!
I spent a year a long time ago training GANs and would not wish it on my worst enemy 🤣


Ad in loop forbid to iniciate the video

Script version
Remove Adblock Thing ver. 3.2
Describe the bug
Some ads rewind in a loop without to play the video.
Expected behavior
The ad appears quickly and the video start.
Desktop

OS: Windows 10
Browser Chrome
Version 120.0.6099.217

Additional context
It's not a persistent issue, but only occurs in the start of the video. During the video don't appears any ad, only sometimes when I start a video the issue happens.

Same issue here. Also screen flickering.

same, the only way to get rid of them is to refresh the page like 5 times -.-. In my case, the ads appear during the video as well.

Same here, with opera gx
Opera V: LVL4 (core: 97.0.4719.84)

Check out 3.3, This should now be fixed.
(Closed as a duplicate issue)


After Restart PM still doesn't appear in 'Packages'

After the update and subsequent restarts of the program Project Manager doesn't appear :(

Is it under the install tab or under packages?
If it's installed try opening the console and run apm uninstall project-manager and then apm install project-manager

I uninstalled Atom for Sublime Text 3 again. But thanks for the comment.


Add demo?

Would be nice to have a README.md to see what it does with an image included :>

readme + gif demo video added


OpenShift CI : Flaky openshift clusters with login failures

[kind/bug]

What versions of software are you using?

Run tests on OpenShift CI
Actual behavior
Randomly fails with unauthorized at various points in test run, even after successful login
Expected behavior
CI Should not be failing on login especially after a successful login
Any logs, error output, etc?
++ sleep 3
++ for i in '{1..40}'
++ oc login -u developer -p developer
++ '[' 0 -eq 0 ']'
++ OC_LOGIN_SUCCESS=true
++ break
++ '[' -z true ']'
++ oc whoami
error: You must be logged in to the server (Unauthorized)
++ oc new-project myproject
Now using project "myproject" on server "https://api.ci-op-t5jndf67-5186a.origin-ci-int-aws.dev.rhcloud.com:6443".

You can add applications to this project with the 'new-app' command. For example, try:

    oc new-app centos/ruby-25-centos7~https://github.com/sclorg/ruby-ex.git

Note Logs above representative as they are from one instance, but multiple tests fail with
Result:
 stdout:
 stderr: ✗  Please log in to the cluster

on multiple instances
/kind test-flake
/kind flake
/area testing

/priority high

Example : https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_odo/1636/pull-ci-openshift-odo-master-e2e-core/637

Conversation with auth team https://coreos.slack.com/archives/CEKNRGF25/p1556020997133000
Most likely cause https://bugzilla.redhat.com/show_bug.cgi?id=1694878

Frequency of hitting the the issue has increased in recent days  - https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_odo/1612/pull-ci-openshift-odo-master-e2e-core/838

/cc @openshift/openshift-team-developer-productivity-test-platform @openshift/openshift-team-auth

Is there a final determination of what is wrong from auth?

They were pointing it to the fapping api server and it definetly looks like it. It looks like it is happening more often again

I believe "flapping" is the word we used :smile:
There were fixes to this in authn operator, not sure about apiserver.
https://github.com/openshift/cluster-authentication-operator/pull/125 (May 4)
https://github.com/openshift/cluster-authentication-operator/pull/120 (Apr 29 - only if you're setting up IdPs in your test cluster)
https://github.com/openshift/cluster-authentication-operator/pull/118 (Apr 27 - quite important, would have caused "Unauthorized" issues as well)

OK, please keep me up to date on any needs you have from DPTP.

It looks like this should be resolved. The Bugzilla issue was closed

Closing the issue as it has been fixed in https://bugzilla.redhat.com/show_bug.cgi?id=1694878


Milestone 2 Feedback

Congratulations on finishing milestone 2! We can see you put a lot of work into this project, nice work!
Below we list some specific feedback you can use to improve your project.
We provide tick boxes for you to use in the future as you address these concerns to improve the final grade of your project.
If anything is unclear, please feel free to ask questions in this issue thread.

1. Write test cases and code iteratively
rubric={accuracy:20,quality:10,mechanics:10}
Great job!
2. R package checklist
1. Create project structure for the R project
rubric={mechanics:20}

[ ] The code of conduct is missing - mechanics -2.

Comments:
It is recommended to include sections in README.md rather than one section for everything.
3. R package function specifications
rubric={quality:10}

[ ] The documentation (3 out of 4 functions) has no examples. - quality -2

4. Specific expectations for this milestone are:
rubric={mechanics:10}
Well done!
5. Submission instructions
rubric={mechanics:10}
Well done!

Hi @negsadr,
Regarding the conduct file, I can see it on the 0.1.0 version of this repo, it is called CONDUCT.md.
Also, regarding examples in the documentation; we're a little confused by the expectation there. In the previous week for the Python package, we were not expected to provide function examples before writing the function itself because we weren't sure about the outputs/ if changes would be made. Since the expectation for this week was basic docstrings only it didn't make sense to provide an example for each since that's something that comes from a fully realized function -- our Milestone 3 version has examples now.

Hi Samson
Thanks for your message. I've modified your grade for the code of conduct.
Thanks for adding examples in your milestone 3. Please note that you are expected to follow the standards for documentations in every step, including milestone 1.
Feel free to reach out if you have any other questions.


Added next-*, thunderstorm and possible-thunderstorm to Dutch language

Fixes #107

This looks good. Thanks for the patch!


Add KMP Algorithm

Hey @aaryahjolia I want to add KMP Algorithm to your repo. Please assign me this issue. Thank you!

Hello, I am Dhinesh. Actually I have solved  KMP algorithm in my college. So sign me up for this issue

@dk172923 i am already solving this issue. Please raise another issue. Thank you.


Using pre-packaged runtimes, outside of NuGet.

See #87, #116

So far, I got the CI to use our packages.
Next step, implement the C# needed to auto download and install them.


App breaks when creating a new namespace from a workload

Setup

Rancher version: 2.7.2
Browser type & version: Any

Describe the bug
While creating a Namespace from any Workload editor, the application breaks.
To Reproduce

Access to Cluster explorer
Open any Workload section list, e.g. Jobs
Create a new Job
Create a new namespace instead of picking one

Result
The application breaks.
Expected Result
TBD
Screenshots
https://user-images.githubusercontent.com/5009481/230131810-e43aee96-71c2-43b2-9c3e-9fb7c4e9e77d.mp4
Additional context
More insight from the mapped traceroute log:
TypeError: this.namespaces.find is not a function
    at VueComponent.updateNamespace (NameNsDescription.vue:333:1)
    at VueComponent.namespace (NameNsDescription.vue:304:1)

This issue has been caused while moving the namespace filtering logic to the existing getter, then extending it. https://github.com/rancher/dashboard/issues/6094

find() method is for Array and not Object, so when creating a namespace here this should already break before my changes. Otherwise there's something wildly odd.

Engineering to write an automated test in #8806 to help with QA.

We already have a test there https://github.com/rancher/dashboard/pull/8638/files#diff-2b5e2a92db7594b42709ac907f4475c8ee31cb630213a03af52222903c7ffb3e


Data missing when extracting cfg3 mp4 data as image

task_0041_user_0010_scene_0004_cfg_0003_human/cam_036422060909
and task_0041_user_0010_scene_0004_cfg_0003_human/cam_104422070011
missing color.mp4, also has more missing data

Cameras sometimes did not record data during data collection due to hardware issues. Just ignore these camera folders where color.mp4 does not exist.


Infix to Postfix

Type of change

[x] Added new project

Changes proposed in this pull request

Project name : Infix to Postfix

Short description :  Given any Infix expression, it gives the resultant Postfix expression

Checklist

[x] The code is Python 3 compatible
[ ] I have added relevant comments in my file to make code understandable
[x] I have added my project in a folder which includes all the necessary files

"Handtracking Volume" is good but "Infix to Postfix" project is not related to development , so kindly please remove that folder.

Removed the Infix to Postfix folder, and kept the Handtracking Project. Please see to it


Add CIs for pushing docker images on dockerhub

Two workflows are set up for pushing Docker images to Docker Hub:
One for dev branches, tagged as dev-snapshot-{branch}-{distro}
One for release tags, tagged as {tag}-{distro}

Where can I find these images on docker hub? https://hub.docker.com/r/bitnine/agensgraph - this is no longer updated, right?
Also, would you be publishing images for released versions v2.13.2 and v2.14.1 of AgensGraph to dockerhub?

@ns-skaranth We're currently working on fixing this. Hopefully, they will be there in the next day or 2.


[AppService] Creating a function app should only set AzureWebJobsDashboard if AppInsights is disabled

hi @gzuber could you pls share info about az cli version you're using, and command to help trouble shooting? thanks

Sorry @yungezz I'm on the AppService/Functions team, this is a tracking issue for a work item we need to fix the az functionapp create command. I've linked the related Function's github issue and the PR I just finished.

Got it. thanks @gzuber for explanation.


chore: Remove generalize'

Shall we remove generalize' altogether?

Shall we remove generalize' altogether?

Yes.

I'm okay with this, but it's not really solving the problem, just sidestepping it.
maintainer merge

!bench

bors merge


hijack netrw instead of replacing

I recently was trying to figure out why I am not able to use open URL under cursor default neovim functionality. At some point I realized that I have set the option replace_netrw = true.  When setting back to false the gx mapping functionality is as expected.
Does ranger.nvim fully disable netrw when set to replace it? Does it replace all netrw functionality? It doesn't seem so, gx mapping doesn't error, it just does nothing.
Maybe it would be possible for ranger.nvim to also replace this functionality of netrw (also hook up rifle for opening files)? Or maybe not disable netrw? It should be possible because plugins like neo-tree.nvim replace netrw directory browsing functionality without disabling it.
I actually really like this replace_netrw option, so I would not prefer to set it to false, but I still want gx mapping functionality.

I found the code responsible for disabling netrw. When commenting it out it seems fine (gx mapping works), but when actually opening a folder it spawns both netrw and ranger (on top). Maybe just quickly close netrw window instead of fully disabling it? Again it seems ranger.nvim doesn't replace gx functionality so fully disabling netrw doesn't seem right.

I believe the most optimal solution would be to hijack netrw FileExplorer instead of fully disabling it. You may find useful this reddit thread. I am not a lua or vimscript dev, so sorry I can't even make that work because its in vimscript.

I tried to convert some of that code into lua and I got the functionality I expect.
vim.api.nvim_create_autocmd("VimEnter", {
  pattern = { "*" },
  command = "silent! autocmd! FileExplorer",
})

Essentially while vim.g.loaded_netrw and vim.g.loaded_netrwPlugin are not set this deletes the FileExplorer autocmd for netrw on every VimEnter. This disables netrw FileExplorer functionality, so allows only ranger.nvim to start, and it does not disable gx file opening netrw functionality.
The only thing is that idk how to convert autocmd deletion to lua. I am also not sure if it works in all the cases?

diff --git a/lua/ranger-nvim.lua b/lua/ranger-nvim.lua
index 3b18880..2e3bdb1 100644
--- a/lua/ranger-nvim.lua
+++ b/lua/ranger-nvim.lua
@@ -213,11 +213,12 @@ end

 ---Disable and replace netrw with ranger.
 local function replace_netrw()
-       vim.g.loaded_netrw = 1
-       vim.g.loaded_netrwPlugin = 1
        vim.api.nvim_create_autocmd("VimEnter", {
                pattern = "*",
                callback = function()
+                       if vim.fn.exists("#FileExplorer") then
+                               vim.api.nvim_create_augroup("FileExplorer", { clear = true })
+                       end
                        if vim.fn.isdirectory(vim.fn.argv(0)) == 1 then
                                M.open(false)
                        end

There you go. Got from https://github.com/stevearc/oil.nvim/issues/182#issuecomment-1915334258
Add this patch and issue is fixed, however you may later revert it because neovim 0.10 adds implements gx mapping separate from netrw, so it will also be fixed when neovim 0.10 is released.


Style' is not a member type of 'UIActivityIndicatorView' Error

I am getting these errors on installation of pod

Same issue in Xcode 9.4

Got the same error at Xcode 9, and it's gone after updating to Xcode 10

The only thing you have to change is "UIActivityIndicatorView.Style" to "UIActivityIndicatorViewStyle"

I fixed this in master so it should now work in both Xcode 9 and Xcode 10 correctly.


Need a clear and thorough tutorial for writing custom operators in c++ with nnvm

This page gives a very simple tutorial for creating operators in c++. After reading, however, I'm still unclear how to write a complicated operator in c++.
Compared to other frameworks (like caffe, pytorch), learning how to create operators in c++ is indeed much harder for mxnet/nnvm. A lot of new concepts, classes and macros are involved, and not explained in detail:

For Attribute Parser, what's NodeAttrs? What's the type of attrs->parsed? What field types are supported by DMLC_DECLARE_FIELD and how? Why I cannot find ParamParser and other classes in the C++ API?
What does it mean by "Outputs can be made invisible to other operators"?
For FGradient, what's NodePtr and NodeEntry? Why the prototype is so different from FCompute? No example is given for how to write a FGradient function and how to access NodePtr and NodeEntry.
For FCompute, what's TBlob? What is the difference of TBlob and NDArray/mshadow::Tensor? How can I know which input blobs need gradient and how can I tell which output blobs need gradient?
The existence of req argument in FCompute makes writing custom operators more difficult, although it may have some benefit for performance. Is it possible that I only handle the case of kWriteTo and the framework handle the other cases for me?
What's auxiliary states and how to use that?
How to declare backward dependency and request resource like the legacy operator interface?
For python, what's the lifetime of an instance of operator? Will an instance of operator be called more than once in a forward pass of a graph? For c++, how can I implement a stateful operator that has some internal states (possibly a very complicated c++ data structure in cpu memory) that are accessed across many forward and backward passes (but do not need to be saved)? And how to initialize those states?

Overall, the nnvm c++ API for creating custom operators is really complicated, especially compared to caffe. A thorough tutorial is required. Or better, simplify the API and make it easier to understand.
Thank you.

It's easier to use the legacy interface for complex operators.
@reminisce Let's update the tutorial later

Yeah, I'm working on the tutorial pr right now. It is also expected to serve as a prerequisite tutorial for contributing sparse operators.

@piiswrong but the operator interface is legacy. Will it be deprecated and removed in near future?

@x10000year I have submitted a PR for the tutorial of implementing operators in MXNet backend. Hope it could answer some your questions. Your comments and suggestions are welcome on the PR. Thanks. https://github.com/apache/incubator-mxnet/pull/7828

Hi, I'm writing a custom stateful op in c++ using the new interface, but I can't find how to share the state of a forward op with its gradient partner. Any tips for solving this? Thanks

@reminisce @cjolivier01

@zheng-da Can you describe how batch norm caches its aux values (running mean and variance) between forward and backward passes (see above)?

just saw the comment. I'll update the tutorial

@zheng-da can we close out this issue now? thanks

@sandeep-krishnamurthy @yzhliu this issue was resolved as part of https://github.com/apache/incubator-mxnet/pull/7828. Good to close, thanks!


Spelling error

Issue in ./help/cmp-resources-and-contact-information.md
In the table under "Current recorded Target sessions include:" there is a spelling error of Tyou instead of You

Thanks. Fixed and publishing now.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 216 ms

Resolved: ClamAV performance has improved in cb44312 after 5 minutes.


SAK-46268 Samigo : button alignment

Jira refers to Samigo but other places might be affected too. Apparently this was added for the Conversations tool
(https://github.com/sakaiproject/sakai/commit/4b3c9eba2fce1c0365c6e611e3f71a00d419fa47#diff-faa29a65c68a13ad617cc3fe3bd77df8428756a63f98da8f0388d7b157cba6f6R408), @adrianfish could you please take a look and see if it's actually needed or if it can be done differently? Thanks!

@bgarciaentornos can you add this to override for samigo only? as it looks needed by other tool(s)

Shouldn't have been the other way around? It was the previous modification on Conversations the one that broke other tools.

I think we can close this patch.


fixdate_add function should accept date type as its param

Proposed changes
Issue Number: close #xxx

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

run buildall


5 establishing connection with database

Read me
Establishment with database works fine.
For tests only sections are still there, so you can check if everything works fine on your devices.
Health check is made by catch method.
NOTE: The program seeks database called test with table students in it. Change it to desired one or create temporary database with specified table (of course remember to fill this table with some data). Program seeks for int, string, and int again.

Not ready for merging.
WIP.

Works!


Not able to run Load on Demand Example

I was trying to use Load On Demand example but if I copy paste the entire example code provided on the link, I get an infinite scrollable list with costant additions of new posts without even scrolling which is not really the behavior shown on the website which works perfectly. I even tried to clone this repository so that I can check the code you might have used for website but no luck. Click here to see the output if we copy the code from the website. There was only one change that I did, I copied the useFakerseed code from the repository in the example.

1.) This website is built with tailwindcss. h-full class means height: 100%. Without tailwind it is useless.
2.) List's desire to stretch vertically should be limited somehow. Some options:

provide explicit height to the list
provide explicit height to list parent and use height: 100% on the list. Or use flex/grid instead of providing full height.
use position: relative on list parent and position: absolute; inset: 0; on the list.

Hi @nowaalex, I implemented your suggestions in the same sandbox and it is working as expected now. Thanks.


Improve alembic

These commits add kansha-admin with alembic related commands:
Usage: kansha-admin <command>

with <command>:
 - alembic-current  : Display alembic's current revision.
 - alembic-downgrade: Revert to a previous version.
 - alembic-revision : Create a new alembic revision file.
 - alembic-stamp    : 'stamp' alembic's revision table with the given revision; don't run any migrations.
 - alembic-upgrade  : Upgrade to a later alembic version.

These commands use Kansha's configuration file to get the database URI used in alembic commands so alembic.ini is not needed anymore.

Thank you. Can you complete the documentation in the doc/install.rst  too? In particular, update sections quickstart, configuration file and a new section about version upgrading would be great!

Please rebase and complete the docs:

configuration file,
quickstart,
developper setup,
ugrading (in production and quickstart)
contribute, in the developper part, add a section about how to change the schema.

Thank you.

Looks like this PR is orphaned. I've pulled it on a local branch to redact the missing documentation.

@bcroq  How do you pass an alternate configuration file? The -c option is not recognized.