fix: Icon button click should not de-select row in table widget

Description
Fix row deselection in table widget on icon button click
Fixes #7959
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Added cypress test

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
// Code coverage diff between base branch:release and head branch: fix/table-icon-click

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
app/client/src/widgets/TableWidget/widget/index.tsx
26.67 (0.07)
5.79 (0)
26.92 (0)
26.42 (0.08)

@vicky-primathon Tested the deploy preview and working fine.


Create a GUI for this application!

You are free to design it according to your preference, but make sure all of the functionality is still there.
Worth 300 points but the TAs/Professor will award 75 bonus points if they feel you went above and beyond.

Since much of this is repetitive work, enable functionality for only 15 buildings of your choosing and comment which buildings you have decided to do when you finish this issue.

done


"RemoteApp Marker Window" appears in remoteapp session in WS2019, both taskbar and black rectangle window

Describe the bug
Launching remoteapp in windows server 2019 a black window refering to "RemoteApp Marker Window" appears, this window is the process rdpinit.exe in the remote session user.

Application details
This is FreeRDP version 3.0.0-dev (d31a98336)
/u:test /app:"||mozilla" /title:"Test" /d:test.test /smartcard /p:"test" /v:localhost:3389 /size:100% /log-level:TRACE /cert-ignore
Output of xfreerdp /buildconfig
This is FreeRDP version 3.0.0-dev (d31a98336)
Build configuration: BUILD_TESTING=OFF BUILTIN_CHANNELS=ON HAVE_AIO_H=1 HAVE_EXECINFO_H=1 HAVE_FCNTL_H=1 HAVE_INTTYPES_H=1 HAVE_MATH_C99_LONG_DOUBLE=1 HAVE_POLL_H=1 HAVE_PTHREAD_MUTEX_TIMEDLOCK=ON HAVE_PTHREAD_MUTEX_TIMEDLOCK_LIB=1 HAVE_PTHREAD_MUTEX_TIMEDLOCK_SYMBOL= HAVE_SYSLOG_H=1 HAVE_SYS_EVENTFD_H=1 HAVE_SYS_FILIO_H= HAVE_SYS_MODEM_H= HAVE_SYS_SELECT_H=1 HAVE_SYS_SOCKIO_H= HAVE_SYS_STRTIO_H= HAVE_SYS_TIMERFD_H=1 HAVE_TM_GMTOFF=1 HAVE_UNISTD_H=1 HAVE_XI_TOUCH_CLASS=1 WITH_ALSA=OFF WITH_CAIRO=OFF WITH_CCACHE=ON WITH_CHANNELS=ON WITH_CLANG_FORMAT=ON WITH_CLIENT=ON WITH_CLIENT_AVAILABLE=1 WITH_CLIENT_CHANNELS=ON WITH_CLIENT_CHANNELS_AVAILABLE=1 WITH_CLIENT_COMMON=ON WITH_CLIENT_INTERFACE=OFF WITH_CUPS=OFF WITH_DEBUG_ALL=OFF WITH_DEBUG_CAPABILITIES=OFF WITH_DEBUG_CERTIFICATE=OFF WITH_DEBUG_CHANNELS=OFF WITH_DEBUG_CLIPRDR=OFF WITH_DEBUG_DVC=OFF WITH_DEBUG_KBD=OFF WITH_DEBUG_LICENSE=OFF WITH_DEBUG_MUTEX=OFF WITH_DEBUG_NEGO=OFF WITH_DEBUG_NLA=OFF WITH_DEBUG_NTLM=OFF WITH_DEBUG_RAIL=OFF WITH_DEBUG_RDP=OFF WITH_DEBUG_RDPDR=OFF WITH_DEBUG_RDPEI=OFF WITH_DEBUG_RDPGFX=OFF WITH_DEBUG_REDIR=OFF WITH_DEBUG_RFX=OFF WITH_DEBUG_RINGBUFFER=OFF WITH_DEBUG_SCARD=OFF WITH_DEBUG_SND=OFF WITH_DEBUG_SVC=OFF WITH_DEBUG_SYMBOLS=OFF WITH_DEBUG_THREADS=OFF WITH_DEBUG_TIMEZONE=OFF WITH_DEBUG_TRANSPORT=OFF WITH_DEBUG_TSG=OFF WITH_DEBUG_TSMF=OFF WITH_DEBUG_TSMF=OFF WITH_DEBUG_TSMF_AVAILABLE=0 WITH_DEBUG_WND=OFF WITH_DEBUG_X11=OFF WITH_DEBUG_X11_CLIPRDR=OFF WITH_DEBUG_X11_LOCAL_MOVESIZE=OFF WITH_DEBUG_XV=OFF WITH_DSP_EXPERIMENTAL=OFF WITH_DSP_FFMPEG=OFF WITH_EVENTFD_READ_WRITE=1 WITH_FAAC=OFF WITH_FAAD2=OFF WITH_FFMPEG=TRUE WITH_FFMPEG=TRUE WITH_GFX_H264=ON WITH_GPROF=OFF WITH_GSM=OFF WITH_GSSAPI=OFF WITH_ICU=OFF WITH_IPP=OFF WITH_JPEG=OFF WITH_LAME=OFF WITH_LIBRARY_VERSIONING=ON WITH_LIBSYSTEMD=OFF WITH_MACAUDIO=OFF WITH_MACAUDIO=OFF WITH_MACAUDIO_AVAILABLE=0 WITH_MANPAGES=ON WITH_MBEDTLS=OFF WITH_OPENCL=OFF WITH_OPENH264=OFF WITH_OPENSLES=OFF WITH_OPENSSL=ON WITH_OSS=ON WITH_PCSC=ON WITH_PROFILER=OFF WITH_PULSE=OFF WITH_SAMPLE=OFF WITH_SANITIZE_ADDRESS=OFF WITH_SANITIZE_ADDRESS_AVAILABLE=1 WITH_SANITIZE_MEMORY=OFF WITH_SANITIZE_MEMORY_AVAILABLE=1 WITH_SANITIZE_THREAD=OFF WITH_SANITIZE_THREAD_AVAILABLE=1 WITH_SERVER=OFF WITH_SERVER_INTERFACE=ON WITH_SMARTCARD_INSPECT=OFF WITH_SOXR=OFF WITH_SSE2=ON WITH_SWSCALE=OFF WITH_THIRD_PARTY=OFF WITH_VAAPI=OFF WITH_VALGRIND_MEMCHECK=OFF WITH_VALGRIND_MEMCHECK_AVAILABLE=1 WITH_WAYLAND=ON WITH_WINPR_TOOLS=ON WITH_X11=ON WITH_X264=OFF WITH_XCURSOR=ON WITH_XEXT=ON WITH_XFIXES=ON WITH_XI=ON WITH_XINERAMA=ON WITH_XKBFILE=ON WITH_XRANDR=ON WITH_XRENDER=ON WITH_XSHM=ON WITH_XV=OFF WITH_ZLIB=ON
Build type: Debug
CFLAGS: -fPIC -Wall -Wno-unused-result -Wno-unused-but-set-variable -Wno-deprecated-declarations -fvisibility=hidden -Wimplicit-function-declaration -Wredundant-decls -g -DWINPR_DLL
Compiler: GNU, 8.3.0
Target architecture: x64
OS version connecting to (server side)
Windows Server 2019
Environment (please complete the following information):
OS: Fedora
Version/Distribution: 33 with openbox
Architecture: amd64

@FutureKidBuu  and the bug being? FreeRDP has no control over which windows the remote opens.

I thought that it could be a misinterpretation in the freerdp code since this does not happen in previous versions of windows server. Maybe it could be fixed by configuration in windows server¿?


Small black line in picture's bottom after compression

I use this library to compress images to a max file size of 70kb before uploading it to AWS S3. I do it twice: one time for my main higher quality image and another time to generate a thumbnail of the same image.
In some cases, randomly and after conversion, the images comes out with a small black line in the bottom of the image. There's one example in here: https://s3-sa-east-1.amazonaws.com/carpediemxp-test-0000001/partners-pictures-thumb/5b528fc884f90700148ec69a
In most cases this does not happen, but when it does, it's really annoying, because images are the main attractions of my web service.
I suspect this may happen because of image dimensions floating point approximations while converting the image from one size to another, but I don't know how to fix this.
Is there any way to fix this problem and keep my images without those black lines?
Thanks in advance!

Fixed by #47


Generated 2022 week 4 schema

Key changes:

Better nullability for parameters
Cleaned up LockstepApi file

More changes:

Switched the build process to use rollup
Set up imports in the index.ts file so that all objects are included
Fixed an issue where the type "File" was not valid - switched it to a Uint8Array.
Corrected imports of OS and URL using NodeJS types
Compiled the program down to a single file - lockstep-api.js
Included a sourcemap and a .d.ts file
Created a proper "npm run build" task
Fixed some configuration options in tsconfig.json
Switcfhed ErrorResult to be a class rather than a type
Code generator now emits the new patterns
According to ESLint, we prefer doublequotes rather than singlequotes, so switched to those

Reviewed with Ted on the use of Rollup.js.

Per @dhric 's approval, landing!


carbon.xml entries move to backup

Description:

I am using wso2 api 3.0, what I observe that is entries to cabon.xml are being restored  to default when I start the apim and changed entries moved to backup folder
Steps to reproduce:
Go to wso2 home/repository/conf/carbon.xml
host123
host123
Affected Product Version:

wso2 3.0
Environment details (with versions):

OS: windows 10 x 64
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Hi @rajputvishwas,
With APIM-3.0 we have introduced a new configuration model. The changes to configuration files must now be done in repository/conf/deployment.toml.
You can follow the configuration catalog document [1] for more information about the configurations.
[1] - https://apim.docs.wso2.com/en/latest/reference/config-catalog/

Thanks


Use cached source tarball if it exists instead of redownloading every time.

Changes line 58, source.rb, from remote_file nginx_url do to remote_file src_filepath do.
This change causes chef to first check the file cache for a valid source tarball before attempting to download the nginx source. Previously this line forced a download of source tarball even if there was a checksum-valid file at src_filepath as this was never checked, only written to.
Several of our servers have already been hellbanned by the nginx download server (download connection is reset by remote after a few bytes are read) due to constantly re-downloading this file every time chef-client is run.

Are you sure it actually downloads the file? Usually chef does conditional requests with ETag.


upgrade nodejs-engine buildpack to v0.6.0 and nodejs-typescript buildpack to v0.1.0

Changes
nodejs-engine
Added

Add profile.d script to set WEB_CONCURRENCY to app (#53)
Set NODE_ENV to production at runtime (#54)
Set NODE_ENV to production or set variable in build environment (#55)

nodejs-typescript
Added

set custom tsconfig or use default tsconfig.json (#10)

@danielleadams I'm seeing
cp: cannot stat '/cnb/buildpacks/heroku_nodejs-engine/0.6.0/profile/WEB_CONCURRENCY.sh': No such file or directory
It looks like that file maybe didn't get added to the zip/CNB package.

@jabrown85 re-released and just restarted the test, so we should be good to go once tests pass.

nvm, I rebuilt it with the new script and it still didn't work 😑  .. need to investigate tomorrow


gf gen dao 生成的文件与gf-demo-user 项目目录结构不一致

1. What version of Go and system type/arch are you using?
go version go1.18.1 darwin/arm64
2. What version of GoFrame are you using?
2.0.4
3. Can this issue be re-produced with the latest release?
yes
4. What did you do?
执行 gf gen dao， 生成的文件与 gf-demo-user   项目目录结构不一致
gf-demo-user  目录结构为
internal/model/entity/user.go
internal/service/internal/do/user.go
internal/service/internal/dao/user.go
internal/service/internal/dao/internal/user.go
部分文件在 service中， 而执行gf gen dao生成的文件不在 service/internal 中
5. What did you expect to see?
6. What did you see instead?
$: gf gen dao
generated: internal/dao/internal/user.go
generated: internal/model/do/user.go
generated: internal/model/entity/user.go
done!

最新版本的gf生成的目录，dao 目录从 service 下移到项目目录，也可以自定义，do 目录移到了 model 目录


Add pilot integration tests using new framework

This is the trivial PR moving pilot tests from examples to the outside directory.
This is my first shoot and will gradually add more e2e/integration test cases using the new framework.
Also, it would be great if I can find some guidances/examples about how to use the new test framework.
cc @nmittler @ozevren @hklai

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ozevren
If they are not already assigned, you can assign the PR to them by writing /assign @ozevren 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:

tests/integration2/OWNERS

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

I don't think we want to do this.  The example is there as a showcase of the test framework. I suggest leaving it where it is for now and just adding new tests.
Also, it should be noted that we can't currently port any tests until we sort out the deployment story.  @ozevren is currently working on something to push a simple Istio deployment.

@nmittler
Please let me know when deployment is ready so that we can start this project.

@nmittler I suppose we do want to create a pilot sub package for all the pilot tests eventually. We can leave some tests in the example package, but I guess it will become less meaningful when people can use the real tests as examples.


CMake support

What do you think about adding cmake support?
We're really interested in incorporating a svg dom type library into our library, but we use cmake, and support Mac, Linux and Windows. So, best way is if svgdom were all packaged into a single git repo, and we could just add it as a git submodule.
I'd be willing to write the CMake build system.

Hi, thanks for the interest in my lib.
I know that CMake is quite popular nowadays, but I'm still considering whether it is worth to support and maintan CMake build scripts. I personally have small CMAke experience.
Note, that all my libraries are packaged to several packet systems and are cross-platform.
Installing, managing and updating libraries using packet systems is much more convenient than using submodules or copying code to your codebase.
The supported OS list is as following:

Linux
Windows (7+)
Mac OS X
Android
iOS

Supported packet systems list is:

deb (debian / ubuntu/ raspbian)
Msys2 pacman (Windows)
Nuget (Windows Visual Studio)
homebrew (Mac OS X)
cocoapods (iOS, Xcode, Mac OS X)

Would you consider if you can use those already supported distribution options?
By the way, if you are interested in rasterizing SVGs, I have the svgren library for that, while svgdom is only about reading/writing/manipulating SVG files.

@AndySomogyi I have corrected the installation instructions on the wiki page https://github.com/igagis/svgdom/blob/master/wiki/MainPage.adoc

@AndySomogyi rolling out your own CMake file to build svgdom and svgren is fairly easy, since all the dependencies are found on @igagis repos. Here's an example CMakeLists that I posted a while ago: https://github.com/igagis/svgren/issues/43#issuecomment-354893026


Create a copycat of the Spotify Wrapped 2022 graphics or something similar

We need to do a mockup of the design

@404answernotfound I think is the wrong issue


Docker Hub repository name
appignition/app-ignition
Terms of services

[X] I accept the term of services

Hi @garrett9! Sorry for the late response here, the team was on PTO last week. Congrats on your extension, it looks amazing! 🎉


Make the plot sizing sliders sticky

Part of #2585
https://user-images.githubusercontent.com/3683420/224411102-9f9a5802-0296-4721-a834-de9b737e8250.mov

I think sliders should be part of the header with the section name (is it sticky now?). We should be trying to optimize the space here. Same as we do with buttons that control the section.

I think sliders should be part of the header with the section name (is it sticky now?). We should be trying to optimize the space here. Same as we do with buttons that control the section.

They have to be separate as the sliders cannot be in the header when the section is collapsed. But they can behave the same way. I'll make the section header sticky as well in a follow-up.

This is an improvement. I agree with what Ivan said. The section header should be condensed and sticky.
Side note: Stories/snapshots in Chromatic look really strange with the slider moved to the LHS.

Can't seem to figure out why. It looks ok in Storybook and the slider was on the left on main screenshots as well. I'll try to fix this when merging inside the vertical sizer as this requires the CI to test.

They have to be separate as the sliders cannot be in the header when the section is collapsed. But they can behave the same way. I'll make the section header sticky as well in a follow-up.

We can hide the controls when it's collapsed. I still think it's better to take less space. It's hard to justify 3 sticky rows (ribbon + header + controls).

They have to be separate as the sliders cannot be in the header when the section is collapsed. But they can behave the same way. I'll make the section header sticky as well in a follow-up.

We can hide the controls when it's collapsed. I still think it's better to take less space. It's hard to justify 3 sticky rows (ribbon + header + controls).

I'll try to move them inside the header while making the headers sticky.


FID Results on CIFAR10 are worse.

Thank you for your GREAT work and for providing the reproduction scripts. However, based on my reproduction, I found that the FID on the CIFAR10 is much worse than 11 in the paper(https://arxiv.org/pdf/2302.00482.pdf), and I actually got 30 (with TargetConditionalFlowMatcher). Any idea about this results?

Hi, thanks for your interest!
There must be something wrong with the parameters when I copied over. Sorry about that. I will check against what I have. We are also planning on releasing code + model weights (hopefully this week) for CIFAR with improved parameters which give FID ~5-6.

I would appreciate it if it could be open sourced! Looking forward to
further work. Thanks.
Alexander Tong @.***> 于2023年10月16日周一 20:16写道：

Hi, thanks for your interest!
There must be something wrong with the parameters when I copied over.
Sorry about that. I will check against what I have. We are also planning on
releasing code + model weights (hopefully this week) for CIFAR with
improved parameters which give FID ~5-6.
—
Reply to this email directly, view it on GitHub
https://github.com/atong01/conditional-flow-matching/issues/63#issuecomment-1764349615,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK73Y5FVX3AI75YCOEX5BJDX7UQTVAVCNFSM6AAAAAA6BS2PEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRUGM2DSNRRGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I checked your commit over my initial one @atong01. I do not see how your modifications could have impacted the current script.
@zhuyu-cs do you mind running the exact OT instead of original FM? I have no idea how FM should work with this script as I have never tested.

It should work. I just need to check wandb to see differences.

I checked your commit over my initial one @atong01. I do not see how your modifications could have impacted the current script. @zhuyu-cs do you mind running the exact OT instead of original FM? I have no idea how FM should work with this script as I have never tested.

yeah, I'm running the exact OT at this time. I'll update the results here when it finished.

@atong01 Yeah I agree, the code is the same as for
Thank you @zhuyu-cs.
As Alex mentioned, we have planned to make a new release this week. Once we have done it, we would be really grateful to know if you are able to reproduce our results (FID-5/6). We will let you know once the update has been done.

We should also release the FID computation. FYI I used the clean-FID library to compute (with tensorflow). https://github.com/GaParmar/clean-fid. I will release a script to compute the FID with clean-FID.

Last question, can you point me to which code you used? Is it the cifar10 code from the example folder or is it the cifar10 code from the runner folder? Thank you.

it is example folder.
On Mon, Oct 16, 2023 at 23:52 Kilian @.***> wrote:

Last question, can you point me to which code you used? Is it the cifar10
code from the example folder or is it the cifar10 code from the runner
folder? Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/atong01/conditional-flow-matching/issues/63#issuecomment-1764785961,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK73Y5BCL4NK4TOP27SGPGTX7VJ3FAVCNFSM6AAAAAA6BS2PEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRUG44DKOJWGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi, can you share the training hyperparameters? I am also trying to reproduce it myself. If possible, I'd like to enable a result comparison between the results of my own codebase and your soon-to-be open-sourced code, the difference might only be the training process. Thanks! :)

The training hyper-parameters do not really change. The optimizer will as well as some components of the neural networks. I am currently running the new code to check that everything works fine. I hope to share it on Monday with the inference code.

Thanks!

Hello!
We can confirm that our new script works well and reach a FID close to 4.5. The current draft release can be found at https://github.com/atong01/conditional-flow-matching/tree/cifar_10_FID_5/examples/cifar10 with the new script and the FID computation. Please, reinstall the whole dependencies from scratch as we have added some new dependencies. The default parameters should give you an FID close to 4.5/5.
We are making the last checks and cleaning before making our official release. If everything goes well, this should happen by the end of this week.

Got it! Thanks for your work! I'll run it with your code and my distributed training code for a double check.

Hello, I have officially added the PR. I need to clean some elements in the README but otherwise, it is ready to use. Would you mind trying to reproduce our results please? We are very close to merge the PR :)

yeah, I'm running it, but it still not finished. I will update the FID here if finished.

Thank you! I also would like to point out that we have updated the integration method at inference. We are now using dopri5 to get a FID of 3.5-ish. To get similar results with Euler integrations, you should use 500 steps instead of a 100 (you would get 3.8-ish). With a 100 steps, you should be around 4.5-ish.

Great work! Thanks!

Thanks for your great work, I'm already in the process of reproducing it. Although not quite finished, the test based on the EMA model training 40w iters has FID=22.5. I wonder if this is consistent with your experience. If it's different, it could be a difference in environment configuration. My environment is as follows, please refer to it:
'''
python==3.9.18
torch==2.0.1
torchvision==0.15.2
torchdiffeq==0.2.3
matplotlib==3.8.0
numpy==1.26.1
scipy==1.11.3
scikit-learn==1.3.2
timm==0.9.8
torchdyn==1.0.6
pot==0.9.1
torchdiffeq==0.2.3
absl-py==2.0.0
clean-fid==0.1.35
'''

How many iterations is 40w? How do you compute the FID?
with the 100 Euler integration steps:
after a 100000 iterations, the FID should be around 7.
After 200000 iterations, it should be around 6.
After 300000 iterations it should be 5

It is 400000 iterations.
Actually, I'm using a code for accelerated testing which takes about only 30 mins to sample 50000 images (Referenced from https://github.com/NVlabs/edm). The training process is exactly as you provided for single card GPU training and also I use a 2080Ti. And the sampler used is torchdiffeq's dopri5. But the reference feature statistics used for cifar, mean and std, are the clean-fid supplied 'cifar10_legacy_tensorflow_train_32.npz '. So the sampling part is the same as your strategy, just parallelized. Also, the code you provided to test the FID is still running and it takes about 3 hours to finish the sampling.
Thanks for the reference experience! I'll continue to debug the code and update results here.

Unfortunately, I do not know the code you are using and I cannot ensure its faithfulness. The fact that you get such a large FID after 400k iterations is not normal and make me think that the problem is how you compute the FID.
Please from now on, only communicate the results that you are getting from clean-FID. We know it works correctly as it is the code we have used to test our code.

OK, thanks for your help!

@zhuyu-cs would you like to share some sample images with worse FID?

Here is a sample of generated images after our training with a 100 euler iterations(FID 4.5-ish).

Hi, @kilianFatras ,you are right! I use the clean-fid to compute the FID and the FID have reduced to 3.87. That's an interesting finding.


add namespace label/tag to non-deprecated throttle metrics

Changes
Back when implementing https://github.com/tektoncd/pipeline/pull/6744 for https://github.com/tektoncd/pipeline/issues/6631 we failed to realize that with k8s quota policies being namespace scoped, knowing which namespace the throttled items were in could have some diagnostic value.
Now that we have been using the metric added for a bit, this realization is now very apparent.
This changes introduces the namespace tag.  Also, since last touching this space, the original metric was deprecated and a new one with a shorter name was added.  This change only updates the non-deprecated metric with the new label.
Lastly, the default behavior is preserved, and use of the new label only occurs when explicitly enabled in observability config map.
Fixes https://github.com/tektoncd/pipeline/issues/7878
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[/ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[/ ] Has Tests included if any functionality added or changed
[ /] pre-commit Passed
[ /] Follows the commit message standard
[ /] Meets the Tekton contributor standards (including functionality, content, code)
[ /] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ /] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ /] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Add 'namespace' label/tag to the 'tekton_pipelines_controller_running_taskruns_throttled_by_quota' and 'tekton_pipelines_controller_running_taskruns_throttled_by_node' metrics, as kubernetes quota definitions are namespace scoped, hence certain namespaces may be more susceptible to quota throttling than others, and in a multi-node environment, not all namespaces are necessarily on the same node.

To enable this new label/tag, set 'metrics.taskrun.throttle.enable-namespace' to 'true' in the 'config-observability' ConfigMap

ping @khrm - can you take a look when you get the chance?
also, see my question in the description wrt if adding labels to existing experimental metrics is "OK"

We can add this behind a flag also. Like we do for count.reason

Ah you mean https://github.com/tektoncd/pipeline/blob/main/docs/metrics.md#configuring-metrics-using-config-observability-configmap
Before I make the change, how about an early review on the name .... say
metrics.taskrun.throttle.enable-namespace
for the new flag's name @khrm ?
Thanks for the pointer.

ok @khrm new flag to enable the namespace label on the throttle metric is added ... PTAL

Thanks @gabemontero /lgtm

likewise thanks @afrittoli
question to you and @khrm - any suggestion on who I should reach out to in order to get the approve label (minimally assuming @khrm you are fine with my updates from your comments) ?

Yes, this is fine.

unrelated flake on alpha integration
/test pull-tekton-pipeline-alpha-integration-tests

@afrittoli @khrm PTAL at the updates from @khrm 's last review - thanks

/lgtm


http://eclairjs.cloudet.xyz/ welcome page contains wrong date

Currently the "last updated" date welcome notebook needs to be manually.  This should somehow be automated to the date the docker image is built, and should also include the milestone number

fixed


Model importer regression with Collada file

Describe the bug
Loading Collada models in Stride <IP_ADDRESS>1 did work but fails in <IP_ADDRESS>2
To Reproduce

Open game studio
Import this collada file: astroBoy_walk.zip without changes to settings in the "Import from model" dialog
Ignore the fact that textures are missing
Drag the model into the scene

How it looks in <IP_ADDRESS>1:

How it looks in <IP_ADDRESS>2:

@Noah7071 is that something you could be looking at?


where is normalization for image feature?

Figure 3 in Fukui et al. (2016) depicts that L2 normalization is applied after CNN, however, I cannot locate the normalization in the code.
Am I missed somewhere?

L2 normalization of the image feature is located here: https://github.com/akirafukui/vqa-mcb/blob/master/train/multi_att_2_glove/vqa_data_provider_layer.py#L230

@daylen Thanks! Though I'm not familiar with caffe, this code l2-normalizes over 2048x14x14 or for each spatial location, i.e. 2048-dim vectors?

Yes.


Don't allow to rewrite "MONGO_URL" application setting if already exist

Now if you add new mongo connection it silently rewrites "MONGO_URL" app setting. I think that some notification like  "MONGO_URL" application setting is going to be rewritten, are you sure you want to do it?Yes|No could help here.

This will overwrite your existing MONGO_URL (or COSMOS_ENDPOINT... etc.) Application Setting(s).

Yes | Cancel


Added example/test for knockout-sortable, fix #623 #585 #557 #622

This should fix #623 #585 #557 #622

@srosengren Done.


Data logging class

More generic way to log data.
Separate mounting of fat_fs file system and actual data_logging.

Still some issue to solve with data logging,
sometimes it costs140ms of delay for stabilization loop, which is far too much !!
There are also some adjustment to be done on the project itself => blame @ndousse
I will let you know tomorrow the details of that.
Since, everything is tested and works at least as good as before (since those 140ms where already there before this pull request)
I suggest to merge and create and issue to be solved regarding that too long delay.
That I will focus on later on !!
Do you agree @jlecoeur ?
Best

Ok!


Feature: add Azure to the list of available filesystems

SDK
Rust
Description
The vectordb::connection::ConnectOptions struct describes valid URIs for S3 and GCS but does not describe how to use Azure cloud storage in any way.

News on this one?


Upload and query documents in SAP HANA Express Edition

Tutorial URL: https://developers.sap.com/tutorials/hxe-docstore-02-query-jsondocs.html
Please specify the step you are referring to 4
The message "Technical Error" is shown no matter my answer. Could you check it out ?
Best regards,

That issue was sorted out. I have already got through it.
Anyway, Thanks.


Missing Card

Hello,
I'm trying to import my hunter deck from heartstats, but it seems Eaglehorn Bow is missing. It's listed in cards.json, but doesn't appear in my interface. Any idea?
Thanks.

You may have managed to delete a card from your deck manager, something I have managed to do as well. Ended up deleting and reinstalling to get the assets back.

Yes that was it! Thank you.


Feat: Design of "Connect with a Reader" and "Send Now" buttons is not suitable in dark mode

Describe the feature
@anuragverma108 @RitiChandak
I saw your project, it's really superb! Although I found an issue that "Connect with a Reader" and "Send now" buttons are not looking good according to the dark mode.
Add ScreenShots

I want to work on this issue to enhance the design of these buttons so that they will be suitable according to dark mode.
Please add labels on this issue and then assign it to me.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@RitiChandak sure


Revert http schemas

Revert https://schemas back to http://schemas, since it is a plain text, but not clickable link.

Docs Build status updates of commit d2bc904:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Docs/Specifications/playready-header-specification.md
:warning:Warning

Details

Docs/Packaging/content-encryption-modes.md
:bulb:Suggestion

Details

Docs/Overview/key-and-key-ids-kids.md
:white_check_mark:Succeeded

Docs/Overview/license-server.md
:white_check_mark:Succeeded

Docs/Packaging/how-to-generate-playready-header.md
:white_check_mark:Succeeded

Docs/Packaging/how-to-package-mp4-based.md
:white_check_mark:Succeeded

Docs/Packaging/mp4-based-formats-supported-by-playready-clients.md
:white_check_mark:Succeeded

Docs/Specifications/playready-header-specification.md

Line 160, Column 0: [Warning-insecure-link] Link 'http://schemas.microsoft.com/DRM/2007/03/PlayReady' is insecure. Links to Microsoft sites must use 'https' instead of 'http'.
Line 165, Column 0: [Warning-insecure-link] Link 'http://schemas.microsoft.com/DRM/' is insecure. Links to Microsoft sites must use 'https' instead of 'http'.

Docs/Packaging/content-encryption-modes.md

Line 35, Column 0: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text. NOTE: This Suggestion will become a Warning in late September, 2020.
Line 57, Column 0: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text. NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Feature Request: add support for Middy.js 6.x

Expected Behavior
You can install v6.x of @middy/core and @aws-lambda-powertools/logger
Current Behavior
When installing v6.x of @middy/core you get this error
 npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@middy/core
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peerOptional<EMAIL_ADDRESS>|| 5.x" from<EMAIL_ADDRESS>npm ERR! node_modules/@aws-lambda-powertools/logger
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/@middy/core
npm ERR!   peerOptional<EMAIL_ADDRESS>|| 5.x" from<EMAIL_ADDRESS>npm ERR!   node_modules/@aws-lambda-powertools/logger
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Code snippet
in package.json
  "dependencies": {
    "@aws-lambda-powertools/logger": "^2.11.0",
    "@middy/core": "^6.0.0",

Steps to Reproduce
Install @middy/core v6 and @aws-lambda-powertools/logger
Possible Solution
change peerDepenedencies in @aws-lambda-powertools/logger to include v6 of @middy/core
Powertools for AWS Lambda (TypeScript) version
latest
AWS Lambda function runtime
22.x
Packaging format used
npm
Execution logs

Hi, thank you for opening this issue.
This is intentional, and also not a bug.
Powertools for AWS is officially compatible only with v4.x and v5.x as of now, as stated in our docs. Middy.js v6.x was released recently and we haven't yet added automated tests to ensure it's compatible, this is why the peer dependency version range doesn't include the new version.
I'm going to convert this to a feature request and put it in the backlog.

Thanks for the update. I have created a PR that runs test for middy 4, 5, 6 but I appreciate that you might want further tests or investigations on it
https://github.com/aws-powertools/powertools-lambda-typescript/pull/3368

Hey @WtfJoke, sorry for the delay.
The team has been busy for re:Invent and we skipped a release week last week.
I am planning on making a release early next week on Monday/Tuesday at the latest.
Thank you for your patience, and sorry for the inconvenience.

No worries! Take your time for the release. Thank you for the fast answer.

@dreamorosi and team, thank you for the release 🙌♥


Add CLI version to t2 version command output (#604)

Fixes #604
As discussed in #604, the initial issue was resolved with a separate PR, but this information might be handy as well.
If this is not wanted on the CLI, that's fine, we can just close the PR. But, it is up here for discussion.

Just one minor change and this is good to go

Updated as per our convo @rwaldron. Also rebased onto latest master.

Good to go! Thanks @wyze!


When using monthsShown allow startDate to be in the firstMonth

Describe the bug
I'm using the range date picker with two months displayed. The startDate of the range always appears in the second month. Therefore it's not possible to see directly a range between one and two months.
To Reproduce
const [defaultStartDate, setStartDate] = React.useState(new Date());
const [defaultEndDate, setEndDate] = React.useState(new Date());
const handleOnChange = (e: Date | [Date, Date] | null) => {
if (Array.isArray(e)) {
const [start, end] = e;
setStartDate(start);
setEndDate(end);
}
};
<DatePicker
selectsRange
selected={defaultStartDate}
startDate={defaultStartDate}
endDate={defaultEndDate}
onChange={(e) => handleOnChange(e)}
inline={true}
locale={translator.getLocale()}
monthsShown={2}
onClickOutside={() => setShowDatePicker(false)}
Expected behavior
The first month should be as such the startDate of the selected range is displayed in it.
Screenshots
If applicable, add screenshots to help explain your problem.

First month should by may and second month should be june

I'm dealing with this EXACT issue.
When I open the date picker for the first time, it starts in the current month.
I select the start date, it starts in the current month.
I select the end date, it starts in the current month.
I reopen the picker once both start/end dates have been selected, it shows last month as the start date.
AAAAAAAAHHH.
I've tried using the openToDate prop but that doesn't help the situation

Try adding focusSelectedMonth={true} to your DatePicker

Also you can manually render per month using monthsShown={1}

Try adding focusSelectedMonth={true} to your DatePicker

Adding focusSelectedMonth worked to me
Thanks!

Its worked for me too! Many thanks!


Live Videos not working

Tried to play several live videos.... without success....freezing by "Downloading MPD manifest"
INFO     2020-05-29 21:46:50,681 [859:YouTubeBackend-3] mopidy_youtube
youtube LibraryProvider.lookup "yt:https://www.youtube.com/watch?v=IiCpDP0zclY"
DEBUG    2020-05-29 21:46:50,718 [859:Thread-28] urllib3.connectionpool
Resetting dropped connection: www.youtube.com[youtube] IiCpDP0zclY: Downloading webpage
DEBUG    2020-05-29 21:46:51,056 [859:Thread-28] urllib3.connectionpool
https://www.youtube.com:443 "GET /results?search_query=%22IiCpDP0zclY%22&sp=EgIQAQ%253D%253D&app=desktop&persist_app=1 HTTP/1.1" 200 None
ERROR    2020-05-29 21:46:51,605 [859:Thread-28] mopidy_youtube
list_videos error "'NoneType' object has no attribute 'text'"
INFO     2020-05-29 21:46:51,608 [859:YouTubeBackend-3] pykka
Exception returned from YouTubeBackend (urn:uuid:57f12d38-5774-4320-ba52-775eeb11d50c) to caller:
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/pykka/_actor.py", line 193, in _actor_loop
response = self._handle_receive(envelope.message)
File "/usr/local/lib/python3.7/dist-packages/pykka/_actor.py", line 299, in _handle_receive
return callee(*message.args, **message.kwargs)
File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/backend.py", line 193, in lookup
name=video.title.get().replace(";", ""),
AttributeError: 'NoneType' object has no attribute 'replace'
ERROR    2020-05-29 21:46:51,610 [859:Core-11] mopidy.core.library
YouTubeBackend backend caused an exception.
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/mopidy/core/library.py", line 17, in _backend_error_handling
yield
File "/usr/lib/python3/dist-packages/mopidy/core/library.py", line 217, in lookup
result = future.get()
File "/usr/local/lib/python3.7/dist-packages/pykka/_threading.py", line 45, in get
_compat.reraise(*self._data['exc_info'])
File "/usr/local/lib/python3.7/dist-packages/pykka/_compat/init.py", line 29, in reraise
raise value
File "/usr/local/lib/python3.7/dist-packages/pykka/_actor.py", line 193, in _actor_loop
response = self._handle_receive(envelope.message)
File "/usr/local/lib/python3.7/dist-packages/pykka/_actor.py", line 299, in _handle_receive
return callee(*message.args, **message.kwargs)
File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/backend.py", line 193, in lookup
name=video.title.get().replace(";", ""),
AttributeError: 'NoneType' object has no attribute 'replace'
DEBUG    2020-05-29 21:46:51,614 [859:HttpServer] mopidy.http.handlers
Sent WebSocket message to <IP_ADDRESS>: '{"jsonrpc": "2.0", "id": 47, "result": []}'
[youtube] IiCpDP0zclY: Downloading m3u8 information
[youtube] IiCpDP0zclY: Downloading MPD manifest

Sorry @tryinit6 - didn't mean to close that!  Looks like a problem caused by live videos not having any time.  I pushed something to develop that works around that; let me know if it helps.
With the changes, I was able to play IiCpDP0zclY (Café del Mar• Chillout Radio & Ibiza Webcam • Live Stream 24/7)

Sorry @tryinit6 - didn't mean to close that!  Looks like a problem caused by live videos not having any time.  I pushed something to develop that works around that; let me know if it helps.
With the changes, I was able to play IiCpDP0zclY (Café del Mar• Chillout Radio & Ibiza Webcam • Live Stream 24/7)

Yes, perfect! I can confirm this solves the issue, tested with some live stations around. Thanks a lot!

Great - thanks for raising the issue and for the feedback.

Hello, I encountered an issue when I try to listen the same live video. (IiCpDP0zclY).
So, when I play that youtube link, the song is played only for 5 seconds, then i have this error: Jun 14 17:25:34 pimusic.local mopidy[539]: OverflowError: Python int too large to convert to C long.
My config file is standard, not changed.
[youtube]
enabled = true
I will attach the log with the error.
mopidy-live-video-log.txt

This does not fix the issue and still causes mopidy to crash.
I installed the repo via
sudo python3 -m pip install --upgrade https://github.com/natumbri/mopidy-youtube/archive/develop.zip

Install logs
agneev@pi4:~ $ sudo python3 -m pip install --upgrade https://github.com/natumbri/mopidy-youtube/archive/fix/issue-150.zip
Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
Collecting https://github.com/natumbri/mopidy-youtube/archive/fix/issue-150.zip
  Downloading https://github.com/natumbri/mopidy-youtube/archive/fix/issue-150.zip
     | 1.8 MB 348 kB/s
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
    Preparing wheel metadata ... done
Requirement already satisfied, skipping upgrade: Mopidy>=3.0 in /usr/lib/python3/dist-packages (from Mopidy-YouTube==3.0) (3.0.2)
Requirement already satisfied, skipping upgrade: requests in /usr/local/lib/python3.7/dist-packages (from Mopidy-YouTube==3.0) (2.20.1)
Requirement already satisfied, skipping upgrade: beautifulsoup4 in /usr/lib/python3/dist-packages (from Mopidy-YouTube==3.0) (4.7.1)
Requirement already satisfied, skipping upgrade: Pykka>=2.0.1 in /usr/lib/python3/dist-packages (from Mopidy-YouTube==3.0) (2.0.2)
Requirement already satisfied, skipping upgrade: cachetools in /usr/local/lib/python3.7/dist-packages (from Mopidy-YouTube==3.0) (4.1.0)
Requirement already satisfied, skipping upgrade: setuptools in /usr/local/lib/python3.7/dist-packages (from Mopidy-YouTube==3.0) (49.6.0)
Requirement already satisfied, skipping upgrade: youtube-dl>=2020.5.3 in /usr/local/lib/python3.7/dist-packages (from Mopidy-YouTube==3.0) (2020.5.29)
Requirement already satisfied, skipping upgrade: urllib3<1.25,>=1.21.1 in /usr/local/lib/python3.7/dist-packages (from requests->Mopidy-YouTube==3.0) (1.24.3)
Requirement already satisfied, skipping upgrade: chardet<3.1.0,>=3.0.2 in /usr/lib/python3/dist-packages (from requests->Mopidy-YouTube==3.0) (3.0.4)
Requirement already satisfied, skipping upgrade: idna<2.8,>=2.5 in /usr/local/lib/python3.7/dist-packages (from requests->Mopidy-YouTube==3.0) (2.7)
Requirement already satisfied, skipping upgrade: certifi>=2017.4.17 in /usr/local/lib/python3.7/dist-packages (from requests->Mopidy-YouTube==3.0) (2019.9.11)
Building wheels for collected packages: Mopidy-YouTube
  Building wheel for Mopidy-YouTube (PEP 517) ... done
  Created wheel for Mopidy-YouTube: filename=Mopidy_YouTube-3.0-py3-none-any.whl size=26496 sha256=5745bbd2268c334004678790b6494cac7a23de8dc93c0e556ca64fddf645744e
  Stored in directory: /root/.cache/pip/wheels/df/39/be/50ca706b25181d65718e048da24ae7e1101fef579bdcc8ea63
Successfully built Mopidy-YouTube
Installing collected packages: Mopidy-YouTube
  Attempting uninstall: Mopidy-YouTube
    Found existing installation: Mopidy-YouTube 3.0
    Uninstalling Mopidy-YouTube-3.0:
      Successfully uninstalled Mopidy-YouTube-3.0
Successfully installed Mopidy-YouTube-3.0
agneev@pi4:~ $ sudo systemctl restart mopidy

Crash logs
Sep  2 18:09:27 pi4 systemd[1]: Starting Mopidy music server...
Sep  2 18:09:27 pi4 systemd[1]: Started Mopidy music server.
Sep  2 18:09:28 pi4 mopidy[6292]: INFO     [MainThread] mopidy.__main__ Starting Mopidy 3.0.2
Sep  2 18:09:28 pi4 mopidy[6292]: INFO     [MainThread] mopidy.config Loading config from builtin defaults
Sep  2 18:09:28 pi4 mopidy[6292]: INFO     [MainThread] mopidy.config Loading config from file:///usr/share/mopidy/conf.d/mopidy.conf
Sep  2 18:09:28 pi4 mopidy[6292]: INFO     [MainThread] mopidy.config Loading config from file:///etc/mopidy/mopidy.conf
Sep  2 18:09:28 pi4 mopidy[6292]: INFO     [MainThread] mopidy.config Loading config from command line options
Sep  2 18:09:28 pi4 mopidy[6292]: INFO     [MainThread] mopidy.__main__ Enabled extensions: http, file, iris, stream, mpd, m3u, youtube, softwaremixer
Sep  2 18:09:28 pi4 mopidy[6292]: INFO     [MainThread] mopidy.__main__ Disabled extensions: none
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [MainThread] mopidy.commands Starting Mopidy mixer: SoftwareMixer
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [MainThread] mopidy.commands Starting Mopidy audio
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [MainThread] mopidy.commands Starting Mopidy backends: YouTubeBackend, FileBackend, M3UBackend, StreamBackend
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [YouTubeBackend-3] mopidy_youtube using bs4API
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [Audio-2] mopidy.audio.actor Audio output set to "alsasink"
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [MainThread] mopidy.commands Starting Mopidy core
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [MainThread] mopidy.commands Starting Mopidy frontends: YouTubeAutoplayer, IrisFrontend, HttpFrontend, MpdFrontend
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [IrisFrontend-9] mopidy_iris.core Starting Iris 3.52.4
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [HttpFrontend-11] mopidy.http.actor HTTP server running at [::ffff:<IP_ADDRESS>]:6680
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [MainThread] mopidy_mpd.actor MPD server running at [::ffff:<IP_ADDRESS>]:6600
Sep  2 18:09:29 pi4 mopidy[6292]: INFO     [MainThread] mopidy.commands Starting GLib mainloop
Sep  2 18:09:31 pi4 mopidy[6292]: ERROR    [HttpServer] mopidy_iris.core Connection "0KDWK0EX8HRO" not found
Sep  2 18:09:43 pi4 mopidy[6292]: INFO     [MpdSession-13] mopidy_mpd.session New MPD connection from [::ffff:<IP_ADDRESS>]:53480
Sep  2 18:09:47 pi4 mopidy[6292]: INFO     [YouTubeBackend-3] mopidy_youtube youtube LibraryProvider.lookup "yt:http://www.youtube.com/watch?v=5qap5aO4i9A"
Sep  2 18:09:47 pi4 mopidy[6292]: INFO     [Thread-15] mopidy_youtube session.get triggered: list_videos (experimental)
Sep  2 18:09:48 pi4 mopidy[6292]: [youtube] 5qap5aO4i9A: Downloading webpage
Sep  2 18:09:48 pi4 mopidy[6292]: INFO     [Thread-15] mopidy_youtube nothing in the soup, trying japi
Sep  2 18:09:49 pi4 mopidy[6292]: [youtube] 5qap5aO4i9A: Downloading m3u8 information
Sep  2 18:09:49 pi4 mopidy[6292]: [youtube] 5qap5aO4i9A: Downloading MPD manifest
Sep  2 18:09:53 pi4 mopidy[6292]: --- Logging error ---
Sep  2 18:09:53 pi4 mopidy[6292]: Traceback (most recent call last):
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/apis/youtube_japi.py", line 105, in json_to_items
Sep  2 18:09:53 pi4 mopidy[6292]:     duration_text = content[base]["lengthText"]["simpleText"]
Sep  2 18:09:53 pi4 mopidy[6292]: KeyError: 'lengthText'
Sep  2 18:09:53 pi4 mopidy[6292]: During handling of the above exception, another exception occurred:
Sep  2 18:09:53 pi4 mopidy[6292]: Traceback (most recent call last):
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/lib/python3.7/logging/__init__.py", line 1034, in emit
Sep  2 18:09:53 pi4 mopidy[6292]:     msg = self.format(record)
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/lib/python3.7/logging/__init__.py", line 880, in format
Sep  2 18:09:53 pi4 mopidy[6292]:     return fmt.format(record)
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/lib/python3.7/logging/__init__.py", line 619, in format
Sep  2 18:09:53 pi4 mopidy[6292]:     record.message = record.getMessage()
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/lib/python3.7/logging/__init__.py", line 380, in getMessage
Sep  2 18:09:53 pi4 mopidy[6292]:     msg = msg % self.args
Sep  2 18:09:53 pi4 mopidy[6292]: TypeError: not all arguments converted during string formatting
Sep  2 18:09:53 pi4 mopidy[6292]: Call stack:
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/lib/python3.7/threading.py", line 885, in _bootstrap
Sep  2 18:09:53 pi4 mopidy[6292]:     self._bootstrap_inner()
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/lib/python3.7/threading.py", line 917, in _bootstrap_inner
Sep  2 18:09:53 pi4 mopidy[6292]:     self.run()
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/lib/python3.7/threading.py", line 865, in run
Sep  2 18:09:53 pi4 mopidy[6292]:     self._target(*self._args, **self._kwargs)
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/youtube.py", line 558, in worker
Sep  2 18:09:53 pi4 mopidy[6292]:     f(*args)
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/youtube.py", line 212, in job
Sep  2 18:09:53 pi4 mopidy[6292]:     data = cls.api.list_videos([x.id for x in sublist])
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/apis/youtube_bs4api.py", line 307, in list_videos
Sep  2 18:09:53 pi4 mopidy[6292]:     for r in rs
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/apis/youtube_bs4api.py", line 308, in <listcomp>
Sep  2 18:09:53 pi4 mopidy[6292]:     for result in cls.run_search(r)
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/apis/youtube_bs4api.py", line 66, in run_search
Sep  2 18:09:53 pi4 mopidy[6292]:     return jAPI.json_to_items(cls, result_json)
Sep  2 18:09:53 pi4 mopidy[6292]:   File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/apis/youtube_japi.py", line 111, in json_to_items
Sep  2 18:09:53 pi4 mopidy[6292]:     logger.warn("no video-time, possibly live: ", e)
Sep  2 18:09:53 pi4 mopidy[6292]: Message: 'no video-time, possibly live: '
Sep  2 18:09:53 pi4 mopidy[6292]: Arguments: (KeyError('lengthText'),)
Sep  2 18:09:53 pi4 mopidy[6292]: INFO     [YouTubeBackend-3] mopidy_youtube youtube PlaybackProvider.translate_uri "youtube:video/lofi hip hop radio - beats to relaxstudy to.5qap5aO4i9A"
Sep  2 18:09:55 pi4 systemd[1]: mopidy.service: Main process exited, code=killed, status=11/SEGV
Sep  2 18:09:55 pi4 systemd[1]: mopidy.service: Failed with result 'signal'.

Thanks @agneevX
Did you install with this:
sudo python3 -m pip install --upgrade https://github.com/natumbri/mopidy-youtube/archive/develop.zip

or this:
agneev@pi4:~ $ sudo python3 -m pip install --upgrade https://github.com/natumbri/mopidy-youtube/archive/fix/issue-150.zip

If it is the second one, are you able to post the file that is generated and saved to /tmp/ and post it to a gist or similar?

Hey, thanks for getting back.
Well, I installed both. First the "fix", then the whole repo. Both have the same issue.
In that case, I re-upgraded to issue-150.zip.
Install logs
agneev@pi4:~ $ sudo python3 -m pip install --upgrade https://github.com/natumbri/mopidy-youtube/archive/fix/issue-150.zip
Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
Collecting https://github.com/natumbri/mopidy-youtube/archive/fix/issue-150.zip
  Using cached https://github.com/natumbri/mopidy-youtube/archive/fix/issue-150.zip
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
    Preparing wheel metadata ... done
Requirement already satisfied, skipping upgrade: youtube-dl>=2020.5.3 in /usr/local/lib/python3.7/dist-packages (from Mopidy-YouTube==3.0) (2020.5.29)
Requirement already satisfied, skipping upgrade: setuptools in /usr/local/lib/python3.7/dist-packages (from Mopidy-YouTube==3.0) (49.6.0)
Requirement already satisfied, skipping upgrade: Mopidy>=3.0 in /usr/lib/python3/dist-packages (from Mopidy-YouTube==3.0) (3.0.2)
Requirement already satisfied, skipping upgrade: beautifulsoup4 in /usr/lib/python3/dist-packages (from Mopidy-YouTube==3.0) (4.7.1)
Requirement already satisfied, skipping upgrade: cachetools in /usr/local/lib/python3.7/dist-packages (from Mopidy-YouTube==3.0) (4.1.0)
Requirement already satisfied, skipping upgrade: Pykka>=2.0.1 in /usr/lib/python3/dist-packages (from Mopidy-YouTube==3.0) (2.0.2)
Requirement already satisfied, skipping upgrade: requests in /usr/local/lib/python3.7/dist-packages (from Mopidy-YouTube==3.0) (2.20.1)
Requirement already satisfied, skipping upgrade: urllib3<1.25,>=1.21.1 in /usr/local/lib/python3.7/dist-packages (from requests->Mopidy-YouTube==3.0) (1.24.3)
Requirement already satisfied, skipping upgrade: chardet<3.1.0,>=3.0.2 in /usr/lib/python3/dist-packages (from requests->Mopidy-YouTube==3.0) (3.0.4)
Requirement already satisfied, skipping upgrade: certifi>=2017.4.17 in /usr/local/lib/python3.7/dist-packages (from requests->Mopidy-YouTube==3.0) (2019.9.11)
Requirement already satisfied, skipping upgrade: idna<2.8,>=2.5 in /usr/local/lib/python3.7/dist-packages (from requests->Mopidy-YouTube==3.0) (2.7)
Building wheels for collected packages: Mopidy-YouTube
  Building wheel for Mopidy-YouTube (PEP 517) ... done
  Created wheel for Mopidy-YouTube: filename=Mopidy_YouTube-3.0-py3-none-any.whl size=26496 sha256=095ef214eb7338934823657aa0bae99f72468cdb067ce57635cb7cf21a4c0367
  Stored in directory: /root/.cache/pip/wheels/df/39/be/50ca706b25181d65718e048da24ae7e1101fef579bdcc8ea63
Successfully built Mopidy-YouTube
Installing collected packages: Mopidy-YouTube
  Attempting uninstall: Mopidy-YouTube
    Found existing installation: Mopidy-YouTube 3.1
    Uninstalling Mopidy-YouTube-3.1:
      Successfully uninstalled Mopidy-YouTube-3.1
Successfully installed Mopidy-YouTube-3.0

Gist: https://gist.github.com/agneevX/b59f64d8039db75b7ab3f246c9110df7

Ok - thanks! I'll investigate and get back to you. Might take a few days - I'm a bit flat out at the moment.
Have you tried the YouTube Music fork at all? Might not be relevant to this, but maybe have a look at it, if you have the time/inclination.

Have you tried the YouTube Music fork at all?

Wasn't sure there was one. Think it's helpful for live videos?

I haven't really looked into it - uses a different API, so might make a difference. It's still pretty beta, but it seems to basically work.

Yeah that also exhibits similar or same behavior..
Part of the log...
Sep 05 21:20:18 pi4 mopidy[30021]:     for result in cls.run_search(r)
Sep 05 21:20:18 pi4 mopidy[30021]:   File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/apis/youtube_bs4api.py", line 60, in run_search
Sep 05 21:20:18 pi4 mopidy[30021]:     return jAPI.json_to_items(cls, result_json)
Sep 05 21:20:18 pi4 mopidy[30021]:   File "/usr/local/lib/python3.7/dist-packages/mopidy_youtube/apis/youtube_japi.py", line 111, in json_to_items
Sep 05 21:20:18 pi4 mopidy[30021]:     logger.warn("no video-time, possibly live: ", e)
Sep 05 21:20:18 pi4 mopidy[30021]: Message: 'no video-time, possibly live: '
Sep 05 21:20:18 pi4 mopidy[30021]: Arguments: (KeyError('lengthText'),)
Sep 05 21:20:18 pi4 mopidy[30021]: INFO     [YouTubeBackend-3] mopidy_youtube youtube PlaybackProvider.translate_uri "youtube:video/lofi hip hop r
Sep 05 21:20:21 pi4 systemd[1]: mopidy.service: Main process exited, code=killed, status=11/SEGV
Sep 05 21:20:21 pi4 systemd[1]: mopidy.service: Failed with result 'signal'.

IiCpDP0zclY is NLA on youtube.  If there are other videos that still have this issue with the current version, please let me know.


Zero copy deserialization

Explore adding a zero-copy deserialization feature that can be opted into instead of using borsh.

Not having this is the main reason I can think of that someone wouldn't use Anchor. Bumping to P1.

Addressed by #202.

Is that zero copy based on structs references as API to state or traits?  Is it borsh compatible in binary format? Does it works on non x86 cpus? how it works with growing stuctrues?  and just wire this to other stuff https://github.com/solana-labs/solana/issues/13391#issuecomment-815553048

Zero copy here directly re-interprets the bytes from the AccountInfo's &mut [u8] as a Ref or RefMut of the zero-copy data structure, where all fields are POD types.
More specifically the account struct must implement pub trait ZeroCopy: Discriminator + Copy + Clone + bytemuck::Zeroable + bytemuck::Pod {}.
Growing structures aren't currently supporrted, and it's not necessarily borsh compatible in binary format (though some subset of things conveniently are, like arrays and structs), and there's no expectation it works on non x86 cpus.


fix: custom manual fee not used in Send

Two different NetworkFeesCubit instances where used between the SendPage and the SendCubit: the SendPage components used a local NetworkFeesCubit to store the fee state, but the global default NetworkFeesCubit was injected in the SendCubit and was used to get the fees on confirmation from, not having the locally set state.
This fixes https://github.com/SatoshiPortal/bullbitcoin-mobile/issues/331.

@i5hi I accidentally closed the previous PR without it being merged and am unable to reopen it, so this isn't merged to main yet. Can you do that with this PR, please?
Sorry for the confusion.

Nice catch! Cheers @kumulynja !


feat: Add extend property

Fixes https://github.com/TypeStrong/typedoc/issues/493

Is this intended to be in draft status?

@Gerrit0 Probably not. I made it and forgot it. I'll finish up the PR soon.

@Gerrit0 How should we detect circularity?

The simplest way is probably to just keep a Set<string> of the resolved paths that have been loaded, and then error if we see the same path twice. This would technically have some false positives (e.g. extends: ['./a.json', './a.json'], but that's likely a mistake in the first place.

Hi @Gerrit0, is this PR awaiting any further corrections?

@maneetgoyal The last thing that needs to be implemented is circularity detection.

Hi @Richienb, thanks for picking up this PR. It seems very useful. Any time estimate on this will be of great help.

@maneetgoyal I've just finished the last part of the PR. After a bit of testing, it will be ready.

Logic looks good to me :+1: I should be able to pull this in this weekend.

Ah, of course this was built before the options refactor - I'll pull the changes in to the refactored code.


[WIP] Add Elixir.load function

Implements #262

@bryanjos I'm not sure how to test this. How would you like to go about this?

@stephanvd good question. I think the easiest thing to test is the translation, testing that the JavaScript you expect to be generated, is. The tests in /tests/translator have examples of how to do so. assert_translation will check that the string given is in the generated JavaScript. I think that should be enough for this.
As for testing if it works when executed, I've usually tested things like this in an example project. I would like a test setup that would run the generated js in a js test suite, but not sure of the best way to do so.

@bryanjos Added the specs and fixed your feedback.
In the last commit I've also added a simple node app to test the generated js with chai. Is this something you want?

@stephanvd I was trying to figure out a good way to test the generated js in the standard library and I think this is a good way to do it! Thanks!
My only feedback is that what is in the package.json you created was moved to the package.json file in the root of the project. It has mocha and chai as well, but probably older versions that should be updated. If you have the time to change that, it would be helpful, otherwise I can just merge this and update it myself.
Either way, this is awesome!

@bryanjos Thanks. Now using the root package.json
Only need to figure out why travis is failing. Do you see what's wrong? I can look into it somewhere this weekend.

@stephanvd I see the problem. It's because I forgot to add a step to run npm run build in the travis ci script to build the bootstrap js file. I can merge this and add that to it.


feat: Allow to use workload federation principals in state bucket permissions

To use workload federation this validation needs to be extended;
More information on federated principals
It seems like this validation is the only part that was blocking.

Attempted to load application default credentials since neither credentials nor access_token was set in the provider block.
Unit tests failing, but it doesn't look related to PR contents.


Beef up logging from imports

Plan to:
-- Use ImportRecord as a record of the overall import run, as it is currently.
-- Add ImportRecordDetail as a record of each importer (or job) that the import runs. This will have number of rows processed, rejected, and deleted. For most importers only the rows processed will be populated. For the new sections related importers, they will have the number of rejected rows. Eventually, they will also have the number of deleted rows once I change the way stale data is deleted from student_section_assignment and educator_section_assignment.
-- Add low level performance data for generating the searchbar json since we suspect that is causing the timing issue. (Turned on with an environment variable, so we can profile in production).

Hey, this sounds great @jhilde. Really thorough and well-thought-out.
I had a branch in my local codebase that tackles this issue in a quick-and-dirty way: https://github.com/studentinsights/studentinsights/pull/1123
It stores some JSON in the database about import timing and prints it in the ErrorReport email.
If you don't mind, we could merge this first for some quick-and-dirty easier-to-access info about timing and then you can implement your more comprehensive solution, which I think will serve us really well especially as we add more districts. Let me know what you think!

Sounds great!
On Sun, Oct 01, 2017 at 4:40 PM Alex Soble
<
mailto:Alex Soble<EMAIL_ADDRESS>
wrote:

a, pre, code, a:link, body { word-wrap: break-word !important; }
Hey, this sounds great
https://github.com/jhilde
. Really thorough and well-thought-out.
I had a branch in my local codebase that tackles this issue in a quick-and-dirty way:
https://github.com/studentinsights/studentinsights/pull/1123
It stores some JSON in the database about import timing and prints it in the ErrorReport email.
If you don't mind, we could merge this first for some quick-and-dirty easier-to-access info about timing and then you can implement your more comprehensive solution, which I think will serve us really well especially as we add more districts. Let me know what you think!
—
You are receiving this because you were mentioned.
Reply to this email directly,
https://github.com/studentinsights/studentinsights/issues/1121#issuecomment-333405117
, or
https://github.com/notifications/unsubscribe-auth/ACeZuwzcU84GltBrNdOPuDA1AisjPiuJks5sn_i7gaJpZM4PpKGr
.


docs: fix coinflip recursive example

Fix the resursive example to call the coinflip template, which was NOT recursive.

Thank you! Could you run this an attach the result please?

Name:                coinflip-recursive-9gzmm
Namespace:           argo
ServiceAccount:      unset (will run with the default ServiceAccount)
Status:              Succeeded
Conditions:
 PodRunning          False
 Completed           True
Created:             Tue May 10 13:44:54 -0400 (40 seconds ago)
Started:             Tue May 10 13:44:54 -0400 (40 seconds ago)
Finished:            Tue May 10 13:45:34 -0400 (now)
Duration:            40 seconds
Progress:            4/4
ResourcesDuration:   5s*(1 cpu),1s*(100Mi memory)

STEP                         TEMPLATE   PODNAME                              DURATION  MESSAGE
 ✔ coinflip-recursive-9gzmm  coinflip
 ├───✔ flip-coin             flip-coin  coinflip-recursive-9gzmm-3059564890  3s
 └─┬─○ heads                 heads                                                     when 'tails == heads' evaluated false
   └─✔ tails                 coinflip
     ├───✔ flip-coin         flip-coin  coinflip-recursive-9gzmm-608008902   2s
     └─┬─○ heads             heads                                                     when 'tails == heads' evaluated false
       └─✔ tails             coinflip
         ├───✔ flip-coin     flip-coin  coinflip-recursive-9gzmm-2566434138  3s
         └─┬─✔ heads         heads      coinflip-recursive-9gzmm-3038940703  3s
           └─○ tails         coinflip                                                  when 'heads == tails' evaluated false

This workflow does not have security context set. You can run your workflow pods more securely by setting it.
Learn more at https://argoproj.github.io/argo-workflows/workflow-pod-security-context/


Refactor the listener gracefulstop and immediate stop test cases

Description:

$Subject

Describe your task(s)
We could use workers to send response and then stop the listener

The existing test cases are disabled in the above PRs since they were failing intermittently. The tests should be rewritten with this task.


NFT Blockchain

@mwmwmw mostly looks good, seems like world tokens won't load without opensea api, please review this:
https://user-images.githubusercontent.com/18633264/177024112-7c179792-bf8d-4714-b996-c1f85a2f02a5.mp4

@supernftier @mwmwmw please review
I deleted OpenSea API key, so I can now see the old user inventory window (white window on the right side)

I have some NFTs which you can see. However they don't show up in my player inventory (tab)

Lastly, there is no way to log out or change wallets. The "logout" button in settings doesn't worry, and it feels like there should be an option near the login.

Here is a video clicking through all of it.
https://user-images.githubusercontent.com/18633264/177024503-7410176b-fd48-48e1-8d4f-e855923a1133.mp4
I think we need to rid of the wallet on the right side entirely, and move inventory into the player inventory (tab menu)

Closing in favor of https://github.com/webaverse/app/pull/3203


A bug where the login page title changes unexpectedly when multi-factor authentication is set

When multi-factor authentication is set up, the title of the current application is displayed on the login homepage, but after entering the multi-factor authentication page, the title will change to the title of the build-in.
This bug does not occur when logging into the user page (e.g. https://www.mycasdoor.com/login/test),
but does occur during the normal authorization process (e.g. https://www.mycasdoor.com/login/oauth/authorize?response_type=code&client_id=xxxxxxxxxxxxx&redirect_uri=https://my-domain.com/callback&scope=openid+email+profile&code_challenge=xxxxxxxxxxxxxxxx&code_challenge_method=S256)

I failed to reproduce this bug , can you reproduce this bug in demo site or provide more detail ?

@chung1912 use English screenshots, this issue will be deleted shortly


Update the swagger for promotion of the read replica clusters.

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[x] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

We are aware of breaking change but it is a private preview. This error will be fixed before GA.

@sirmaridvan could you fix the errors of all the CI checks , also adding API in the existing version, you need to go through the breaking change review process, please follow the action in https://github.com/Azure/azure-rest-api-specs/pull/21245#issuecomment-1290490164

@jianyexi this is a privatepreview api, we should not be needing to go through the breaking change process. Additionally, the lint failures are not newly introduced by this pr. As I said, they will be fixed with a subsequent PR. However, it makes it easier for us to handle the fixes after this property is merged in.

I am not sure why your private preview version exists here, basically, this repo should not contain any private preview api version, you can send PR to the private repo https://github.com/Azure/azure-rest-api-specs-pr for reviewing private preview api version


tf.pow(x, y) edge case with negative x (Bug)

I am using tf.pow for my project, but my losses are 'nan', so I setup the test cases as shown below.
I found that whenever x is negative, tf.pow seems to output nan instead of the correct answer.

r = tf.pow(0.4,0.4)
r2 = tf.pow(-0.4,-0.4)
r3 = tf.pow(0.4,-0.4)
r4 = tf.pow(-0.4,0.4)
sess.run(r)
0.69314486
sess.run(r2)
nan
sess.run(r3)
1.4426999
sess.run(r4)
nan

I appreciate for anyone of the community who can address this issue.
Respectfully,

But tensorflow does support complex number, right?
I can even create variable type as this:<tf.Tensor 'Exp:0' shape=() dtype=complex128>
sess.run(y3)
(1.8369701987210297e-16+3j)
I guess my question now is since tensorflow does support complex number, why does it output NaN?
Some operations in tf does support, and some does not? If possible I wish this issue to still be open, so people can work on the operations that does not support complex numbers.

@stefanjuang The numbers you are using are float, not complex. define them as complex in order to get correct results.

I think there are still some not expected results. For instance, the cubic root of a negative number exists always and is negative. However, tf.pow(-27., 1/3) will output nan. To get the expected result for any float number x, you need to write tf.sign(x)*tf.pow(tf.abs(x), 1/3)


Numbering was found to decrease along the sequence

I get an error that says "numbering was found to decrease along the sequence" when using the command line tool but do not get the same error when i use the same sequence on the python tool. Have you ever seen this issue before/know a solution?

I also encountered the same problem. What caused this? Is there a solution?

This is an old issue now, probably already resolved in the latest version of ANARCI.
Unless you replicate the same problem while providing more details I'll have to close the issue imminently.

I installed ANARCI (v2021.02.04) via conda.
Does this version address this issue?
By the way, this problem often occurs with the light chain, and almost not with the heavy chain.

I got the same error:
 File "/xxxxx/anarci/anarci.py", line 147, in validate_numbering
    assert index >= last, "Numbering was found to decrease along the sequence %s. Please report."%name
AssertionError: Numbering was found to decrease along the sequence Input sequence. Please report.

I have also encountered this error (using the latest github version):
$ ANARCI --scheme chothia -i NRFSGVPDRFSGSGSGTDFTLKISRVEAEDVGVYYCMQSIQLPLTFGGGT
Warning: Non IG chains cannot be numbered with the chothia scheme. These will be ignored.
Numbering was found to decrease along the sequence Input sequence. Please report.
Error:  Numbering was found to decrease along the sequence Input sequence. Please report.

The sequence (partly) comes from https://www.imgt.org/3Dstructure-DB/cgi/collier_perles_allele.cgi?alleleid=3086&type=V&dom=1


Custom dialog view - automatically set default margins

Material Dialogs version: <IP_ADDRESS>
API: 26
Device Manufacturer: Android
Device Name: Samsung S6

I would like to have following small extension:
    new MaterialDialog.Builder(getActivity())
		.customView(view, false, <boolean>)

Or an alternative extra method for keeping the backwards compatibility. The additional parameter should have the effect that the default margins are set to the custom view. So that I don't have to do following manually:
    View view = new CustomView(context);
    MaterialDialog dlg = new MaterialDialog.Builder(getActivity())
		.title(R.string.info)
		.customView(view, false)
		.positiveText(android.R.string.ok)
		.show();

    // this should be done automatically if desired
	int horizontalMargin = (int)getActivity().getResources().getDimension(R.dimen.md_dialog_frame_margin);
	((ViewGroup.MarginLayoutParams)view.getLayoutParams()).leftMargin = horizontalMargin;
	((ViewGroup.MarginLayoutParams)view.getLayoutParams()).rightMargin = horizontalMargin;

Will no longer apply to version 2


Understanding Diffie Helman and Elgamal Asymmetric Encryption

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #3722

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Plagiarism check done ✅
I think this article is light on substance - What do you think? @lalith1403

@ahmadmardeni1 I just went through the article. The article explains the concepts well. However, I agree there is lack of content. The content explained is really good but not enough.
@njerigitahi I would suggest you to add more content. Some possibilities are, where does this algorithm fail, in-depth case study of this algorithm. The python code snippets need to be formatted better for easier understanding. Check out the published articles to know more.

@lalith1403
I have done the revision, explained the reason for use of asymmetric over the symmetric algorithm, formatted the code, and added some examples and images for clarity.

@njerigitahi, please remember to follow the Topic suggestion this closes instructions in the PR template correctly to link your topic suggestion to your PR. This helps ensure a swift payout.
You need to remove the backticks from that line for it to take effect.

@louisefindlay23  I have made the changes

Yes. It is due for another round of review.

@njerigitahi
We will be closing this PR due to the amount of revisions occuring - please work on this PR to polish it up and re-submit it (only) when it is ready to be published. (meaning no glaring errors).
👍
Be sure to check out our resources to help guide you as well.
🚀
As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published.
Our team of peer reviewers do not provide revisions services at this time in order to review them all in a timely manner.
All revisions and edits should be completed on your own forked repo (as to not take up room in the queue).
Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.


piano keys, and tone generator parameters

Adding a piano keys to make ease of determining pitch value of musical notes.
I also had corrected the parameters of the tone generator into which I consider much more appropriate.
preview: https://kouzeru.github.io/Octo/

I disagree with a number of aspects of this design:

showing the piano keys at the top is extremely confusing; the most important elements of the UI should be the pattern and the pitch. the piano keys are logically part of the tone generator.
it is no longer possible to directly approximate a desired frequency; pitch and pattern are totally disconnected.

In addition, the piano keys themselves provide no audio feedback; it's awkward to have to hit "play" after every change

I'm putting the piano key on the middle-
And my interest is mostly into music, and I usually don't give mind to frequency values...
The only matter is frequency interval, only to grasp with given distance between one note to another.
Here's the only reason why I used number cycles (harmonics) to determine frequency of pattern in the tone generator.
Combining interval 4 : 5 : 6, creating a C major chord.
https://user-images.githubusercontent.com/46141631/133897202-40156d9e-4157-4c36-8efd-2d4b0eac5969.mp4

I'm adding the frequency monitor to show the relation between frequency and pattern below the tone generator.
And I guess that's enough, and I'm already satisfied with the existing functionalities.

I like the proposed inputs for the tone generator, having a duty % and target number of pulses is much more intuitive in my mind than needing to specify a bitwidth and how much of that should be active. Also allowing inverting and reversing the pattern makes it faster to do blends. And relabelling the Random button to Noise makes sense to me.
The bits I found confusing were the play buttons felt disconnected from the buffer that they actually use. Having play below the keyboard while it uses the buffer in Audio Pattern had me pressing it and getting no output till I worked out what the go was. I had a quick play around with moving controls and ended up with this UI which I thought made the relationship a bit clearer (moving the play buttons alongside the representation of the buffer they use):

I also changed the position of the Pitch input in that screenshot but I'm not particularly happy with that. The aim there was to show that the keyboard visualisation is another method of selecting a pitch value so having those controls next to each other seems appropriate. On that note I prefer how this implementation behaves in that the selected pitch is applied to both the tone generator buffer and the audio pattern buffer. In the old editor changing the frequency changes the pitch used for the tone generator, but you can't see what the actual pitch value is until you click Generate, overwriting the active pattern. The relationship could be made clearer but I don't have the UX skills to know how to do that effectively.
The live frequency display is great, one of the things I found most confusing about the existing Octo audio editor is the relationship between the frequency input for the tone generator and the resulting pattern and pitch. This lets me see how changing the number of cycles and the pitch adjusts the tone (and also highlights that duty% is more like a volume control). The preview frequency doesn't handle out of range values nicely though as trying to set a cycle count higher than 64 shows an overly high frequency until you click play for the tone generator. This is an issue with the current implementation anyway, there's no input validation.

Thank you for the lovely feedback! It's nice to know something I didn't even realize about being self-explanatory, I'm quickly applying the things that you had suggested here, and fixing what you issued about. :)

By the way, I'm looking forward for the blending stuff, since I couldn't abuse the audio blending to do the sprite blending, now it's reintroduced in Sprite Editor on the last changes here, which is also abusable to blend audio patterns :P
Also, I'm adding the sprite invert, flip and rotate, since there was no easy way to do it.

The piano key is now clickable to play pattern directly from it...
Left-click plays pattern from the pattern editor, right-click plays pattern from tone generator.

( also thought forward for someone's who's lefty, hence I don't need to specify which and which )
As the side changes of adding the mouse-release to the handler on drawOnCanvas() in order to achieve this, I've done some optimizations upon the sprite and audio editors, which only refresh the text box when mouse is released. I had to do this because the DOM refresh is slow, hence editing sprite and audio was lagging.
The other minor changes is also readjusting the audioScriptProcessor buffer size, to make it dynamic depending on the environment instead of being stuck at some constant.

It seem lot of changes packed here 😓
But, I'm still wondering if there is any further thoughts-

I had tested it with three browsers (Chrome, Firefox, and Safari), It's ready!
Lemme know if there's something need to add or remove, and
I'm going to make a bit of manual of using the interface effectively.


fix(index): prevent render without results

The index assumes that the lastResults prop is set which is not always the case. The issue arises only when the stalled render kicks in. The previous implementation (without index) fixed this issue by turning the default value of _isSearchStalled to true to avoid extra render until we have results (https://github.com/algolia/instantsearch.js/pull/2623). This fix does not work anymore since we don't have a single Helper.
This PR revert the change to a default value of false which makes more sense (it's not stalled by default). Only the index is aware that we have results or not, then the condition belongs to its render function.
Before

After

The issue with <mark> in your screencast, is that related or #3830?

The issue with <mark> in your screencast, is that related or #3830?

Yes, the branch is not up to date with develop.


MCP2517FD interface with TM4C123gxl

Hello everyone,
Is anyone did interfacing MCP2517FD module with TM4C123gxl(Tiva launchpad) micro controller ?
Please help me if anyone has done this project.

hello,
No, I did not interface a MCP2517FD modulewith a TM4C123gxl micro-controller.

Le 22 nov. 2019 à 10:56, Neelakar544<EMAIL_ADDRESS>a écrit :
Hello everyone,
Is anyone did interfacing MCP2517FD module with TM4C123gxl(Tiva launchpad) micro controller ?
Please help me if anyone has done this project.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/pierremolinaro/acan2517/issues/8?email_source=notifications&email_token=AEWKZVHUU3IRCK2ZJUT25V3QU6UGLA5CNFSM4JQOLTV2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H3K4GSA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEWKZVBB4GAMFZW3HKHDCH3QU6UGLANCNFSM4JQOLTVQ.


ros_control_boilerplate: 0.1.0-1 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository PACKAGE to 0.1.0-1:

upstream repository: https://github.com/davetcoleman/ros_control_boilerplate.git
release repository: https://github.com/davetcoleman/ros_control_boilerplate-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.20
previous version for package: null

ros_control_boilerplate
* Initial release of ros_control_boilerplate
* Contributors: Dave Coleman

Is this a bug? I ran bloom release again this time with the "Repository Name:" specified. Still it calls my repo "PACKAGE"

Ok, I manually fixed the commit. I think bloom-release might have a bug where it always says "PACKAGE"

Are you sure? You using 0.5.20 version, right? Actually, it's fine to me like the link below...
https://github.com/ros/rosdistro/pull/9881

Yeah, I haven't seen anyone else run into this as a bug.

Ok. I even tried deleting the whole -release repo and doing it from scratch. But its likely I had an old version of bloom release, I only update when I see a need. I've just updated my Ubuntu. I'll see if I run into the same problem again for a different repo. Thanks.
This is ready to merge because I manually edited and rebased the commit.

The CI is still failing. I think it would be better for you to rerun bloom after updating.

Also, you are using 0.5.20 (it says it in the release repository and in this pr's description), so you're using the correct version.

Ok, I'll re-bloom.
Also, this package requires https://github.com/ros/cmake_modules/pull/38, will it still be able to build even though you haven't released cmake_modules yet? I assume not now that I think about it.

It will not. Also I wasn't planning on releasing that change into Indigo. It's currently only on the 0.4-devel branch which is Jade only right now.


Make content/design changes as needed to facilitate a better Okta migration for users

Problem statement
Design and Content have created changes that need to be made to the website and Okta flows to facilitate a positive transition for users to the new Okta instance.
What you need to know
This will be based on the change requests coming from Design and Content in these tickets:

(#10537) Revise content of initial email to users
(#10538) Revise content of login page to have better instructions for users
(#10539) Revise design (if needed) of login page to have better instructions for users
(#10541) Identify any changes/needs to the design/branding on rest of Okta instance needed for initial migration

Summary from Jillian:

User flow with design and content updates
Cheat sheet with global design styles

Acceptance criteria

[ ] Content and components implemented and published
[x] Update the asset logo in the Okta theme (get from Jillian)
[x] Update the theme panel with new colors
[ ] Move legal notice from the alert to the body of the page Login Page
[ ] Repurpose the alert at the top of the page with the new content on the Login Page (DO NOT DEPLOY UNTIL 9/28 Deploy - captured in ticket #11340)
[x] Update the content on the password reset alternate flow email
[x] Update the content on the password reset normal flow email
[x] Update the content of the automated activation email
[x] Update content in the "Unable to Sign in" alert in the widget
[x] Update remaining content per Figma file
[ ] Update the ReportStream support page per Content update (#11069)

To do

[ ] ...

@audreykwr @jillian-hammer I do not see in the given materials anything showing the desired look of the legal notice when moved out of the alert.

engineer reference of highlights of improvements while preparing frontend for migration: https://docs.google.com/document/d/1FXj4QDLXD4Ym0xYo7OoFxsarec3_g3URGkiURt_jYhc


added support for other java.time types

LocalDateTime, LocalDate, LocalTime and Instant
all support granularity.
Instant granularity is a bit of a fudge (see updated docs)
I think I fixed a bug in GenDateTime where by normalizing the output you might push the returned value outside of the range - e.g. range 03:00 1/1/17 - 07:00 2/1/7 might generate a value on 1/1/17 that when normalized at year granularity would round it down to 00:00 1/1/17. Please check logic
Full tests provided and docs updated

Codecov Report

Merging #42 into master will decrease coverage by -32.19%.
The diff coverage is 31.29%.

@@             Coverage Diff             @@
##           master      #42       +/-   ##
===========================================
- Coverage   80.43%   48.24%   -32.19%
===========================================
  Files          10       15        +5
  Lines          92      228      +136
  Branches        0        3        +3
===========================================
+ Hits           74      110       +36
- Misses         18      118      +100

Impacted Files
Coverage Δ

...datetime/jdk8/granularity/InstantGranularity.scala
0% <ø> (ø)

...me/jdk8/granularity/ZonedDateTimeGranularity.scala
0% <ø> (ø)

...me/jdk8/granularity/LocalDateTimeGranularity.scala
0% <ø> (ø)

...tetime/jdk8/granularity/LocalTimeGranularity.scala
0% <ø> (ø)

...tetime/jdk8/granularity/LocalDateGranularity.scala
0% <ø> (ø)

...ortysevendeg/scalacheck/datetime/GenDateTime.scala
100% <100%> (ø)
:white_check_mark:

...ysevendeg/scalacheck/datetime/instances/joda.scala
100% <100%> (ø)
:white_check_mark:

...rtysevendeg/scalacheck/datetime/jdk8/GenJdk8.scala
44.18% <40%> (-55.82%)
:x:

...ysevendeg/scalacheck/datetime/instances/jdk8.scala
52% <59.09%> (+52%)
:white_check_mark:

... and 4 more

Continue to review full report at Codecov.

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

Hi Tim,
I'm so so sorry, but I never received a Github notification for this, so it has gone unnoticed for a couple of weeks. I'm going to take a look at this ASAP.
Thanks
Noel

Hi,
No problem. Interested to know whether it works for you guys in terms of
how I did it.
Tim
On 2 March 2017 at 12:53, Noel Markham<EMAIL_ADDRESS>wrote:

Hi Tim,
I'm so so sorry, but I never received a Github notification for this, so
it has gone unnoticed for a couple of weeks. I'm going to take a look at
this ASAP.
Thanks
Noel
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/47deg/scalacheck-toolbox/pull/42#issuecomment-283645790,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNjp6LckL24tZvaENWgnefMEylYu1IKks5rhrvLgaJpZM4MC6uz
.

--
Tim Pigden
Optrak Distribution Software Limited
+44 (0)1992 517100
http://www.linkedin.com/in/timpigden
http://optrak.com
Optrak Distribution Software Ltd is a limited company registered in England
and Wales.
Company Registration No. 2327613 Registered Offices: Suite 6,The Maltings,
Hoe Lane, Ware, SG12 9LR England
This email and any attachments to it may be confidential and are intended
solely for the use of the individual to whom it is addressed. Any views or
opinions expressed are solely those of the author and do not necessarily
represent those of Optrak Distribution Software Ltd. If you are not the
intended recipient of this email, you must neither take any action based
upon its contents, nor copy or show it to anyone. Please contact the sender
if you believe you have received this email in error.

Thanks very much for this PR, and is definitely of value. Overall the code seems fine, and most of my review points are cosmetic. It's also highlighted a few things I need to do, such as put in a plugin like Scodec to handle the code formatting (wasn't a concern until now as it's only been me committing on the project!), and have a clear document about copyright/copying of the project.
Probably the biggest things here are considering what to do about the large blocks of duplicated code. Feel free to have a go at condensing those, or this is something I should think about too, as it would be good to get this right before a 1.0 release.
Thanks again

I've put this PR on a branch in the 47deg repo, to continue to work on this: https://github.com/47deg/scalacheck-toolbox/tree/nm-41-other-java-time-types
I'm going to close this PR as I won't be merging this, but will merge from the other branch. @TimPigden - I will copy you in on the PR when I do it if you have any interest, but I'm not too sure when exactly I'll be finishing this up.
Thanks again for the PR.


Fix Docker image deletion when using remove_images: true

Hi all!
This PR resolves #360
My fix is based on the idea of deleting the target Docker image only if it's not being used by an existing container, in other cases Kitchen will print an INFO message.
Note: I first merged the fix of https://github.com/test-kitchen/kitchen-docker/pull/356 from the corresponding fork branch: paulcalabro:fix-ip-address-issue.
Kr,
Rshad

Hi @rshad - Do you have any update to this PR based on the previous feedback? Do we still need these updates or can this PR be closed?

Hi @rshad - Do you have any update to this PR based on the previous feedback? Do we still need these updates or can this PR be closed?
Also, another PR was merged to fix #356, so this change would have to be removed from this PR. Thanks.

Hi @jeffreycoe, sorry I didn't have time to add the changes you requested before, but I think this issue is still persisting.
Kind regards,

@rshad - Thanks for the response. Will you be updating this PR with these changes, or can this PR be closed out and be addressed at a later time?

@rshad - Thanks for the response. Will you be updating this PR with these changes, or can this PR be closed out and be addressed at a later time?

Sorry, I am not working with kitchen since many months ago and I don't think that I'll be adding the related changes soon.
Kind regards,
Rshad

@rshad Ok - I'll close this PR for now, and we can revisit this at a later time. Thanks!


add aws iam authentication for mysql

Building off of https://github.com/authzed/spicedb/pull/1858, adds similar support for MySQL
Marking this as a draft since I haven't validated on AWS yet
Validated with MySQL Community on AWS

Please rebase as well


TCP::Socket implement connect

We've only got passive connect (TCP::Socket::onConnect) as of v0.6.3

#370 needs testing, been unable to do so in my current environment. Will update when tested.

Fixed by @AndreasAakesson somewhere before #407