ICA: tests for module.go

Description
Adding a module_tests.go file for testing the channel creation callbacks. I am just testing the happy path out for these (which is mostly copy pasta from handshake.go within the keeper. If we want to get test coverage for module.go we need to add this here as well. I think it's still worthwhile keeping handshake.go within the keeper structure as it keeps the module.go file a lot cleaner and separates the logic nicely.
Also, added some minor miscellaneous types tests for keys.go
I'm planning to add tests for ack/recv/close chan in follow-up PRs.
partially closes: #279

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #424 (9e98647) into interchain-accounts (7c7bf8e) will decrease coverage by 0.37%.
The diff coverage is n/a.

@@                   Coverage Diff                   @@
##           interchain-accounts     #424      +/-   ##
=======================================================
- Coverage                77.65%   77.27%   -0.38%
=======================================================
  Files                      129      131       +2
  Lines                    10110    10214     +104
=======================================================
+ Hits                      7851     7893      +42
- Misses                    1864     1925      +61
- Partials                   395      396       +1

Impacted Files
Coverage Δ

modules/apps/27-interchain-accounts/genesis.go
27.27% <0.00%> (ø)

modules/apps/27-interchain-accounts/module.go
39.78% <0.00%> (ø)

modules/apps/27-interchain-accounts/types/keys.go
100.00% <0.00%> (+33.33%)
:arrow_up:


🚀 - New plugin to enable validation on every keystroke for composition event (IME)

Playground Link
https://stackblitz.com/edit/maskito-article-demo?file=index.ts
Description
https://github.com/taiga-family/.github/assets/2785780/6fe93a2f-b4d1-4fda-bcc8-85fad768745c
Angular version
No response
Taiga UI version
No response
Which browsers have you used?

[X] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[ ] macOS
[ ] Windows
[ ] Linux
[ ] iOS
[X] Android

@glebcha Hello!
I hope that this bug was fixed in 1.5.0 release.
I've updated Maskito's version in the stackblitz's article (1.0.0 => 1.9.0).
Could you check it again ?

@glebcha Hello!
I hope that this bug was fixed in 1.5.0 release.
I've updated Maskito's version in the stackblitz's article (1.0.0 => 1.9.0).
Could you check it again ?

But there is another sort of bug with 1.9.0.
First enter letters, then enter numbers and only then any special character.....input will be cleared of any non-digit characters.
Android 14, latest Chrome version, Samsung S23.

Anyone faced this issue, be aware of https://developer.mozilla.org/en-US/docs/Web/API/CompositionEvent.
I mean this behavior is expected when tou have autocomplete enabled and not yet selected one of proposed autocompletion variants or pressed space to follow shadowed input value.
@nsbarsukov I guess better add this to official documentation and "readme".

Problem
Some Androids always interprets their input as CompositionEvent.
Maskito will validate their input when compositionend-event is fired.
By default, Maskito does not break IME Composition (wait until it ends to calibrate value).
Maskito behaves in this way now (no changes are required).
It is default behaviour for other popular mask library too: imask, ngx-mask.
On another side, another popular legacy library – text-mask can break IME if intermediate value does not satisfy mask's constraint.
Discussion
There is not right-solution for this case.
If we validate intermediate composition's value on every keystroke => we break core concept of IME (Japanese, Chinese and etc).
If we don't validate intermediate composition's value on every keystroke => Maskito waits until it will finish => Sometimes it is not required (for example, input of numbers).
Proposed solution
By default, Maskito does not break IME Composition (wait until it ends to calibrate value).
Introduce maskitoCompositionValidationPlugin => developer can use it to enable validation of intermediate value of composition event.
import {maskitoCompositionValidationPlugin, MaskitoOptions} from '@maskito/core';

export default {
    mask: /^[0-9０-９]*$/,
    plugins: [maskitoCompositionValidationPlugin()],
} as MaskitoOptions;

What is better and more self-descriptive: maskitoCompositionValidationPlugin or maskitoStrictCompositionPlugin  ? 🤔


Add support for the attestation API?

I want to obtain the vehicle VIN, and if I understand it correct I have to first interact with the Attestation API and get the VIN VC. Is this something the Python SDK could help simplify?

@ardevd Just added this today! Newest version is up on PyPI and docs have been updated: https://docs.dimo.org/developer-platform/api-references/attestation-api/attestation-endpoints
Check it out and let me know if you run into any issues. When you're getting your privileged_token, be sure to include privilege 5, as that's the one that's required for credentials access. Example:
privileged_token_req = dimo.token_exchange.exchange(authHeader["access_token"], privileges=[1,3,4,5], token_id=<token_id>)
privileged_token = privileged_token_req['token']

Fabulous! I'll give it a try soon!


chore: option to set credentials policy in fetch request

@philsturgeon This is basic implementation of #1548. What are things needed to test in this?

Feel free to close this if not relevent or not needed.

We've merged related PR, it should be included within next release - probably on Monday/Tuesday.
Thanks for contributing!


discussion on the desgin of load the metrics definition

the config looks like https://git.musta.ch/airbnb/data/blob/master/airolap/confs/hdfs_size_silver.conf

I like the approach !

I'm ok with the approach, can we just remove direct references to AirOlap and define a Druid semantic conf instead DRUID_CONF? Is hocon the best choice here? YAML and json are much more common.
We should eventually document the feature, the docs are just getting started though.

after discussing with @mistercrunch, we will look at getting the metric definition from the segment metadata in the new druid release, closing this one


Add more loaders

Related #3047
And https://github.com/ardatan/graphql-toolkit/pull/355

@ardatan let's get this merged in graphql-toolkit, then we can merge here :)


Add basic spin bootstrap

Currently spin automatically infers rails from the gemfile and installs mysql along with setting the DATABASE_URL which breaks the env as we are using sqlite for development.
Setting .spin/svc.yml to an empty file overrides the automatically inferred setup and basically means "we do not need anything". This is enough to boot an instance, run bundle and be able to run bin/rails test unit tests. This is not enough to run system integration tests but we will tackle it separately once we have such need.

Shouldn't we focus on things like https://github.com/Shopify/maintenance_tasks/pull/544

Actually I agree
Closed in favour of https://github.com/Shopify/maintenance_tasks/pull/760


Adjoint instead of transpose in tests

This came up in a nanosoldier run related to https://github.com/JuliaLang/julia/pull/46196. While transpose(Q) can be constructed, there exist no specialized (multiplication) methods for it. In your (test) case here, you are constructing a Matrix right away, which therefore probably falls back to componentwise getindex, which is slow, whereas Matrix(Q') uses fast specializations.
My proposal is independent from the above-mentioned PR, and I'm pointing this out just in case this also occurs in code, not just in tests.

Codecov Report

Merging #5 (a43d764) into main (2d4bd52) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   60.89%   60.89%
=======================================
  Files           5        5
  Lines         381      381
=======================================
  Hits          232      232
  Misses        149      149

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks for bringing that up. I was not aware and that will certainly be useful thing to know in the future. I will think the code base here looks fine. But I will doublecheck in a related package I built (github.com/FHoltorf/LowRankIntegrators.jl) as well.
Thanks for the heads up!


Hyp 141 Reset Button

name: Hyp 141 Reset Button
about: Hyp 141 Reset Button
Summary
Because we’re storing certain things in browser/local state, need the ability to reset the demonstrator by clearing all local storage
AC:
Reset button as per design
Button clears browser/local storage and returns to /
Motivation
To help the person that's doing the demo
Describe alternatives you've considered
To add this to the left panel
Additional context

Button should only exist in the slide-out drawer, not on main page

Moved to sidepanel
https://github.com/digicatapult/sqnc-hyproof-client/assets/29942957/f1543015-f6d3-4c43-a403-79fdab3ec382


Implement the list service bindings endpoint

Is there a related GitHub Issue?
#603
What is this change about?
#603
Does this PR introduce a breaking change?
No
Acceptance Steps
#603
Tag your pair, your PM, and/or team
paired w/ @julian-hj @gnovv

Put the PR back in draft to prevent merges, but addressed all of the comments. Need to squash and do one last look-over, but we have now hit EOD and some.

Overall looks good. We filed a draft PR #708 that suggests some further improvements. Please have a look.


fix: FlutterRename when files with import is opened in another buffer

Fixes #289

@m-salman-a I need a bit more context on what this PR is trying to do

Hmm, I tested the original and changed version and I see no difference in behavior. As I understand it the reason why the import update is not applied is because the file had changes that were not saved yet. If I save changes - renaming works.

Hi, thanks for the quick reply. Upon recreating the issue, I found that you have to use the FlutterRename command more than once for the import to not be updated. I also made sure to save all the files before performing the second FlutterRename. Here is a video reproduction of the issue before and after applying the fix provided in the PR. This is the minimal config that I used.
Before
https://github.com/akinsho/flutter-tools.nvim/assets/51153904/bfb8f8b3-2fb3-4431-ab54-27d3e1eaaebf
After
https://github.com/akinsho/flutter-tools.nvim/assets/51153904/bd3ab94f-2d86-4f6f-bc81-9e4d1c90dfee
Explanation
As for the explanation, the "Buffer .... newer than edits" error comes from this check here in the vim.lsp.utils.apply_text_document_edit function which is called from vim.lsp.utils.apply_workspace_edit. I'm currently not sure why this check is failing (if anyone can help that'd be great) but applying the workSpace/willRenameFiles request before calling the textDocument/rename handler makes this check pass.

@m-salman-a thanks for the detailed explanation! Yea, when renaming more than once I am able to reproduce the issue, and your fix solves it. LGTM


"rake rspec" fails when spec/spec.opts does not exist

After running rspec-puppet-init, I'm getting this failure:
$ bundle exec rake rspec
rake aborted!
No such file or directory - spec/spec.opts
/home/me/my_module/Rakefile:6:in `read'
/home/me/my_module/Rakefile:6:in `block in <top (required)>'
/home/me/.gem/ruby/2.0.0/gems/rspec-core-2.14.7/lib/rspec/core/rake_task.rb:122:in `call'
/home/me/.gem/ruby/2.0.0/gems/rspec-core-2.14.7/lib/rspec/core/rake_task.rb:122:in `block (2 levels) in initialize'
/home/me/.gem/ruby/2.0.0/gems/rspec-core-2.14.7/lib/rspec/core/rake_task.rb:121:in `block in initialize'
Tasks: TOP => rspec
(See full trace by running task with --trace)

The problem is on line 140 of lib/rspec-puppet/setup.rb.  It's expecting File.read("spec/spec.opts") to return false if the file does not exist, but that actually raises an exception.
[1] pry(main)> File.read('/does/not/exist')
Errno::ENOENT: No such file or directory - /does/not/exist
from (pry):1:in `read'

#176 is already merged, this can be closed


[bug] one single line of code will screw the entire html

int[][] dirs = new int[][]{{-1, 0}, {1, 0}, {0, -1}, {0, 1}};
This above line of code, if added to post, will stop the entire post from displaying.

I tested it in markdown, whether it is direct writing, inline code or block level code, it's ok


docs: Update for new pr-* workflows

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Update documentation to excise mentions of Jenkins and replace with brew pr-pull or brew pr-publish as needed.

Thanks! I've removed the Jenkins assets and fixed the maintainer docs

Thanks for the reviews!


Feature/views

Dear reviewer, it is a pleasure to request a review of my Controller specs feature before merging into dev.
Below are the changes made

Implement the design from the sneak peek wireframes

Thank you very much for the review sir @mwanawabangona , but won't that be an error from your local database? It appears you have the "Blogapp_development" database already and trying to create the migration on top. How about you delete that "Blogapp_development" database on your local database and try again?

@flemton can we do a zoom call so i show you?

Alright thanks @peterrobert , I understand now. Can you please pull and try again?

Hi @flemton  ,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request

[ ] You have submitted a project in a used PR , make sure you create a new PR  and the ask again for a review

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @iambenkis in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


This data link is invalid

Could you please send me the data zip, my email is<EMAIL_ADDRESS>Thanks very much!

try this link https://seljuk.me/upload/CVRR_dataset_trajectory_clustering.zip

It works! Thanks a lot!
从 Windows 版邮件发送
发件人: Selçuk Gülcan
发送时间: 2022年5月28日 17:12
收件人: seljukgulcan/comparing-trajectory-clustering-methods
抄送: River-Zhang; Author
主题: Re: [seljukgulcan/comparing-trajectory-clustering-methods] This datalink is invalid (Issue #7)
try this link https://seljuk.me/upload/CVRR_dataset_trajectory_clustering.zip
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


[Task]: Fix build scripts for Windows

assetgen.sh

Windows has a native convert command so the ImageMagick command is not executed, the convert commands need to be changed to magick convert
Windows needs the directories to be have backslash instead of slash

I think for windows should be needed to install mingw64 + bash, or WSL. Or maybe make it with nodejs, wdyt?

I think for windows should be needed to install mingw64 + bash, or WSL. Or maybe make it with nodejs, wdyt?

I think it would be better to use NodeJS, because it is the tech that most of the team is familiarized with


Add information how to use a debugger with the cms

Motivation

Using a debugger with the cms is not trivial, since our project structure does not match the default django project structure (Running python integreat_cms/integreat-cms-cli runserver fails)
Proposed Solution

Add documentation how to properly configure a debugger for the cms using various editors.
Alternatives

Don't and let people figure it out by themselves or use print-debugging
User Story

As a developer I want to use a debugger with the cms
Additional Context

Launch config for vscode: https://chat.tuerantuer.org/digitalfabrik/pl/4s6rsr9hobd7jfbyja1x45bdsy

I'll take a shot at documenting the setup with neovim 👍🏼

Here's my debugging setup for pycharm:

Enable django support in the settings and configure it like this:
Create a django run configuration and configure it like this:
The environment variables are: DJANGO_SETTINGS_MODULE=integreat_cms.core.docker_settings;PYTHONUNBUFFERED=1;INTEGREAT_CMS_DEBUG=True
Make sure to call ./tools/run.sh at least once before debugging to make sure that the database and assets are all initialized
Click on the debug icon for this run configuration

NIce, thank you @david-venhoff! Would you mind adding this to the existing PR, or creating a new one after #3030 is merged?
I have no experience with VSCode and PyCharm, so I am hesitant to add the description myself.

Yes, I can do that :) (Although I also don't have a lot of experience with vscode)
I am not sure when I'll do it though, so it probably makes most sense if I do it as a separate pr.

Yes, I can do that :) (Although I also don't have a lot of experience with vscode)
I am not sure when I'll do it though, so it probably makes most sense if I do it as a separate pr.

Then just do PyCharm! 😄
I'm sure someone with knowledge of VSCode will eventually add it :)


Review and change statuses for operations

Currently, during an upgrade operation, the statuses follow:

not_deployed
deploying
action_succeeded

None of these statuses are ideal and do not really represent what is going on. Same for deleting. Statuses need to reflect what the operation is intended to do.
Following this, the UI will need a small update to add successStates and failedStates etc.

The following statuses are proposed:

--
Deployment
Upgrade ("update")
Custom Action
Uninistall ("delete")

Before Starting
awaiting_deployment
awaiting_update
awaiting_action
awaiting_uninstall

In Progress
deploying
updating
performing_action
uninstalling

Success
deployment_succeeded
update_succeeded
action_succeeded
uninstall_succeeded

Failure
deployment_failed
update_failed
action_failed
uninstall_failed


Error  apurl service is not defined

I change Document Service Location
api url : service is not defined
OS version:  CentOS Linux release 8.4.2105
Software version :
[root@localhost ~]# rpm -qa | grep onlyoffice
onlyoffice-xmppserver-11.0.0-107.noarch
onlyoffice-documentserver-6.3.1-32.x86_64
onlyoffice-communityserver-11.5.2-460.noarch
onlyoffice-repo-1.0.0-11.noarch
onlyoffice-controlpanel-3.0.0-381.noarch

Moved this issue to CommunityServer repo, since problem is not for Docker image

I'm trying to connect groups to documentserver and following the provided YT video from the docs. However the fields look completely different. I'm getting the same API url error.

Hello @chris-aeviator Please describe in more details - what is the address of the community server and document server? And what addresses do you specify in the setting Document Service? Did communityserver can resolve address of document server, and vice versa?

The documentation is just a little out of date. The video shows how to add the urls with paths, while the most recent version of groups->settings-> documentserver only want's the plain URLs
I managed to get it up and running by providing in this order

https://my-document-server.my-domain.tld
https://my-document-server.my-domain.tld
https://my-community-server.my-domain.tld

Okay, it's look good. Did you configured the JWT function? You can find some information in this issue.

it's commented out for me ATM - is this for having login functionality/pw protection to my document server?

We recommend to use it. More info you can find here.

thanks, this looks useful, I will work on enabling it!


Error when make

just use make
make[3]: Leaving directory '/home/atn4517/KeyDB-0.9.2/deps/jemalloc'
make[2]: Leaving directory '/home/atn4517/KeyDB-0.9.2/deps'
    CC adlist.o
    CC quicklist.o
    CC ae.o
    CC anet.o
    CC dict.o
    CC server.o
In file included from server.cpp:31:0:
server.h:2331:68: error: declaration of ‘void* calloc(size_t, size_t)’ has a different exception specifier
 void *calloc(size_t count, size_t size) __attribute__ ((deprecated));
                                                                    ^
In file included from server.h:42:0,
                 from server.cpp:31:
/usr/include/stdlib.h:468:14: error: from previous declaration ‘void* calloc(size_t, size_t) throw ()’
 extern void *calloc (size_t __nmemb, size_t __size)
              ^
In file included from server.cpp:31:0:
server.h:2332:49: error: declaration of ‘void free(void*)’ has a different exception specifier
 void free(void *ptr) __attribute__ ((deprecated));
                                                 ^
In file included from server.h:42:0,
                 from server.cpp:31:
/usr/include/stdlib.h:483:13: error: from previous declaration ‘void free(void*) throw ()’
 extern void free (void *__ptr) __THROW;
             ^
In file included from server.cpp:31:0:
server.h:2333:54: error: declaration of ‘void* malloc(size_t)’ has a different exception specifier
 void *malloc(size_t size) __attribute__ ((deprecated));
                                                      ^
In file included from server.h:42:0,
                 from server.cpp:31:
/usr/include/stdlib.h:466:14: error: from previous declaration ‘void* malloc(size_t) throw ()’
 extern void *malloc (size_t __size) __THROW __attribute_malloc__ __wur;
              ^
In file included from server.cpp:31:0:
server.h:2334:66: error: declaration of ‘void* realloc(void*, size_t)’ has a different exception specifier
 void *realloc(void *ptr, size_t size) __attribute__ ((deprecated));
                                                                  ^
In file included from server.h:42:0,
                 from server.cpp:31:
/usr/include/stdlib.h:480:14: error: from previous declaration ‘void* realloc(void*, size_t) throw ()’
 extern void *realloc (void *__ptr, size_t __size)
              ^
Makefile:278: recipe for target 'server.o' failed
make[1]: *** [server.o] Error 1
make[1]: Leaving directory '/home/user/KeyDB-0.9.2/src'
Makefile:6: recipe for target 'all' failed
make: *** [all] Error 2

Hi can you let me know what operating system you are using?  Mac or Linux (and if linux which distro?
It looks like this is a conflict with a system header.  I'll try my best to reproduce it.

I haven't been able to reproduce this issue - but I've committed a fix which I think should resolve it: a9e525c8
I'm creating a new issue to track fixing the underlying cause once I can reproduce the issue.

I use this version: https://github.com/JohnSully/KeyDB/archive/v0.9.2.tar.gz
and this is my environment:
$ gcc --version
gcc (Debian 4.9.2-10+deb8u2) 4.9.2
Copyright (C) 2014 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

$ g++ --version
g++ (Debian 4.9.2-10+deb8u2) 4.9.2
Copyright (C) 2014 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

$ cat /etc/issue
Debian GNU/Linux 8 \n \l

Thanks I will try Debian tomorrow.
In the meantime the unstable branch should compile for you with my temporary fix. I have not tried with a GCC that old but it does support C++14 so should work.

I've verified Debian 8 (jessie) and confirmed it works.
It did require one additional fix to ifdef out socket CPU pinning which is not available on older kernels.


(UX/UI) Clarify help menu

Currently the help menu is labeled as "Keyboard shortcuts", with keyboard shortcuts on top and "useful links" (blog/guides/issue) below (those are even hidden below scroll viewport depending on screen height):

I think a better flow would be:
help (ex: question mark) icon > useful links on top > keyboard shortcuts below
Why?

users generally expect a "help" menu providing all kind of help
keyboard shortcuts nicely fit in a "help" menu in my opinion

I could try to PR for this.

I like the idea!

I like the idea!

@inwardmovement should I assign it to you?

@inwardmovement should I assign it to you?

@inwardmovement should I assign it to you?

Ok!
Can you just point out the files where I can find the code related to the keyboard-shortcuts icon and keyboard-shortcuts modal window? I haven't had a look to the codebase yet.

@inwardmovement should I assign it to you?

Ok!
Can you just point out the files where I can find the code related to the keyboard-shortcuts icon and keyboard-shortcuts modal window? I haven't had a look to the codebase yet.

Awesome.
This is a good start...

https://github.com/excalidraw/excalidraw/blob/master/src/components/HelpIcon.tsx
https://github.com/excalidraw/excalidraw/blob/master/src/components/ShortcutsDialog.tsx

This should eventually change to HelpDialog and we can continue giving feedback on your PR. So as soon as you have something ready.. open a draft and we can take it from there.

Awesome.
This is a good start...

https://github.com/excalidraw/excalidraw/blob/master/src/components/HelpIcon.tsx
https://github.com/excalidraw/excalidraw/blob/master/src/components/ShortcutsDialog.tsx

This should eventually change to HelpDialog and we can continue giving feedback on your PR. So as soon as you have something ready.. open a draft and we can take it from there.


Property 'isUIFeedbackEnabled' not found on object of type 'KVNProgressConfiguration

Howdy
After updating to the latest version, we are now getting this error in our Xcode 10, Swift 4.2 project
Property 'isUIFeedbackEnabled' not found on object of type 'KVNProgressConfiguration
It looks like its happening after this commit
https://github.com/AssistoLab/KVNProgress/commit/519ac3f9d9b0484643a5620c4bb1a68d10d4af3a

After we explicity set the podfile to master  pod 'KVNProgress', git: 'https://github.com/AssistoLab/KVNProgress', branch: 'master' we got the right code.
but when we had it as just pod 'KVNProgress', we for some reason were not getting this updated bool flag. When we looked at the code in the develop branch:https://github.com/AssistoLab/KVNProgress/blob/develop/KVNProgress/Classes/KVNProgressConfiguration.h
it matched this header file.
Doing a full clean and pod repo update didn't fix it until we explicitly set to master...any suggestions as to why this might have occurred?


Maven integration

I just took a look at asciidoc and how to integrate document generation to PDF/HTML via the asciidoctor maven plugin: http://asciidoctor.org/docs/asciidoctor-maven-plugin/.
Then I stumbled over asciidocfx and I'm impressed what powerful extensions it brings (charts, UML diagrams etc.). Of course now I'm wondering, whether any maven integration is planned for automatically generate different documentation from the asciidoc - including the extensions. It looks like asciidocfx has everything it needs for auto-generating the documentation.

Thank you guy, yes we can write a maven-plugin for asciidocfx

That would be awesome :)!

i second that! :-)
:+1:

:+1:

👍

:thumbsup:

Think support for uml & plantuml already means that the existing plugins would work:
https://github.com/asciidoctor/asciidoctor-maven-examples/tree/master/asciidoctor-diagram-example
eg. [uml,my.png] & [plantuml,my,png]
<plugin>           <groupId>org.asciidoctor</groupId>           <artifactId>asciidoctor-maven-plugin</artifactId>           <version>${asciidoctor.version}</version>           <dependencies>             <dependency>               <groupId>org.asciidoctor</groupId>               <artifactId>asciidoctorj-diagram</artifactId>               <version>${asciidoctorj-diagram.version}</version>             </dependency>


[ZEPPELIN-5171]. Support flink command in flink interpreter

What is this PR for?
This PR is to support %flink.cmd to flink command so that user can run flink jar jobs in zeppelin. This is a sub interpreter of flink. Internally it runs shell interpreter for flink command.
What type of PR is it?
[Feature ]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5171

How should this be tested?

CI pass

Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Will merge if no more comment

Will merge if no more comment


Release CI/CD pipeline

We need to create a release CI/CD pipeline for this project.
Use these documents as a reference:

How to release the first Eclipse version of projects
JSONP release job

Jobs:

https://jenkins.eclipse.org/metro/job/policy-master-build/
https://jenkins.eclipse.org/metro/job/policy-ee4j8-build/

Release job: https://jenkins.eclipse.org/metro/job/policy-release/
tested with master and EE4J_8 branches


Does you consider support enum?

for example:
type struct Example {
    Language string `valid:required;enum(goalng,c,cpp,python,rust)`
}

or you just implement with regular expression?

sorry, I find in tag


Slow?

Many people say that sv-ttk is really slow on their system. I'm not experiencing any lag, but it might be a good idea to use gif for some widget images.

Or it would be interesting how the SVG version performs on Windows. I wanna convert this to SVG anyways.

Any new info on this? I really wanna use sv-ttk for my Windows app

I'm not really working on this project any more, only some occasional commits. I also have no idea how it could be made any faster, other that telling people not to use Windows :P
This theme is probably as fast as it gets, even if that is quite slow on Windows. However I can say that it is one of the fastest tkinter themes, and it is multiple times faster than customtkinter

I've found that using certain versions of packages with it (e.g. pywinstyles) can affect the speed, that's maybe something to look into.

In what way? Does it make things faster?

Resizing the window becomes much slower, at least for that package (when using the windows acrylic theming).
But that hooks into windows api's, so I'm not sure it's something you really can fix speed on.

Yeah. I mean, if pywinstyles makes the window laggier, then it's probably an issue with that package, and not with sv-ttk. It could also be that those Windows APIs are only optimized for Microsoft's own GUI toolkits.


feat(scope): custom vcs scope resolution

Some users do not use git for a VCS. The git scope resolution should implement a way of letting users customize what the "commit sha" suffix of a scope would be in those instances.
Add Grapple.GitResolver

Expand git scope resolution to allow for a git commit resolver
A git commit resolver has the type fun(): string

Update README with relevant details

Update configuration section
Update Tag Scopes section

This has been made available to users with scope resolvers. See: https://github.com/cbochs/grapple.nvim#project-scopes


Custom Deployment -> Response status code does not indicate success: 403 (Forbidden).

I have setup a Custom Deployment using the ARM template in Azure Devops and all is running fine, except for when a Luis call is made in the Emulator. It then responds with :
Response status code does not indicate success: 403 (Forbidden).
What is causing this? I have nicely both the normal and authoring Luis endpoints and both keys are in the appSettings.json and are correct. Is there any other setup that I am missing for Luis during development that I should include in the deployment?

Could you check your settings in appsettings.deployment.json? We are using this configuration for a deployed bot to avoid the mixing up between local and deployed environment.

@luhan2017 I don't have an appsettings.deployment.json. I do see one in another skill that has a custom runtime, but this one does not.
It would be really helpful if one could explain how Luis is deployed, published and used in the local vs azure setup.

For local, the step is

Fill in the luis settings in the publish luis page. (those settings will be saved in botfolder/settings/appsettings.json)
Call lubuild, output is the generated folder.
Start botruntime.

For azure, the step is

Run the provision steps, which will provisioned two luis services for you(luis authoring and luis prediction), those settings are outputted as a publish profile.
Add this publish profile in the publish page, composer will take the keys from the profile and put it in appsettings.deployment.json
Deploy the bot, the deployment script, will call lubuild and use the authoring key to generate luis sevices, and then call another API to bind the service with the prediction key. (So in a deployment scenario, you will use the endpointKey which is come from the luis prediction service).

So for your question, if you want to update the luis settings for deployment, you could just update it in your publish profile. Because the appsettings.deployment.json will be deleted after the deployment is done.

@luhan2017 Thanks for the information. I have the same issue on a normally deployed bot, so I guess that it is not related to the pipeline itself vs the normal deployment. Thanks for the info that is really helpful though.


Hue 4.7 Can't login using LDAP credentials

Describe the bug: I've setup LDAP integration, but it doesn't work. It's copy paste from old version of Hue to newer and it was working fine.
When I provide my credentials I'm receiving on yellow background error like this:

Request Method:
POST

http://HOSTNAME:8888/hue/accounts/login

1.11.29

TransactionManagementError

An error occurred in the current transaction. You can't execute queries until the end of the 'atomic' block.

/usr/local/hue/build/env/lib/python2.7/site-packages/Django-1.11.29-py2.7.egg/django/db/models/sql/compiler.py in execute_sql, line 899

/usr/local/hue/build/env/bin/python2.7

2.7.5

Any ideas?
Steps to reproduce it? Yes, everytime when I want to login using LDAP.
Hue version or source? Hue 4.7

Could you share how you fixed this error?


[WIP] Supavisor V2

In this PR, Supavisor has been adapted for Erlang 27 and Elixir 1.17. The metrics backend has been changed from PromEx to Peep, and the most significant change is the introduction of ProxyHandler for processing transaction mode.
The ProxyHandler first authenticates the incoming connection through auth_query. If there is no connection pool, it creates one and also launches a Ranch on a free port, saving the port and host in the corresponding Syn record.
If the pool is on the same node as the client connection, we locally change the owner of the TCP socket for each transaction to the direct connection with the database. After the transaction is completed, the client process returns ownership of the TCP socket to the DbHandler.
If the pool is on a different node, the client process connects to the Ranch of the corresponding pool on the other node, switches to session mode, and forwards everything (works like proxy). The local Ranch operates the same way as when the client connection is on the same node as the connection pool.
With this handler, erldist is used only for synchronizing Syn. Forwarding query data between nodes is done via TCP sockets (Ranch)

Hey, @supabase/dashbit, we have prepared improvements for Supavisor and would love to hear your feedback. There are a lot of changes, so feel free to review whatever you like. We are particularly interested in your opinion on the ProxyHandler. The corresponding implementation is here:

lib/supavisor/handlers/proxy/handler.ex
lib/supavisor/handlers/proxy/client.ex
lib/supavisor/handlers/proxy/db.ex

Exciting work! Do you have any numbers or results into the performance of this new version?

After the transaction is completed, the client process returns ownership of the TCP socket to the DbHandler.

I haven't reviewed the code yet, so I will try to verify this as well, but remember that, if the client crashes for some reason, you most likely can't assert for certain the state of the socket: for example, was something only partially written to it? So you may need to drop the connection altogether or do some sort of cleanup (if possible), like make sure to abort transactions and other things. I am not sure how much state can be carried around.

Two additional questions for now:

I also remember we discussed improvements to poolboy or your fork to poolboy. I don't remember what they were exactly though. I assume those may already be in place?

Is it correct that this pull request still preserves the old code for talking to the database? Is the plan to remove it in the future or are there issues in unifying everything through the proxy handler?

Two more:

The ProxyHandler first authenticates the incoming connection through auth_query. If there is no connection pool, it creates one and also launches a Ranch on a free port, saving the port and host in the corresponding Syn record.

Is the maximum number of ports (65k) a potential concern here? If so, could we use a single Ranch port but pass some other information (such as a preamble when connecting via gen_tcp) to choose with pool to use?

If the pool is on a different node, the client process connects to the Ranch of the corresponding pool on the other node, switches to session mode, and forwards everything (works like proxy).

How much do you handoff to the other node? From looking at the code, if it is on a different node, you are doing both auth and ssl handshakes. Does it mean you are doing them twice? On in the current node and then once you have to proxy? I wonder if it is worth doing a "special handshake" between those nodes, so when you join this special Ranch port (which should not be exposed), you can assume auth and ssl have been handled in the first node and you don't have to redo it. Maybe that's already the case, but I wanted to double check.

Exciting work!

:blush::blush::blush:

Do you have any numbers or results into the performance of this new version?

We are still measuring performance for different scenarios. When the pooler is on the same machine as the database, Supavisor outperforms or performs the same as PgBouncer. However, it is slightly slower when a Supavisor cluster of two nodes is in the same AWS zone as the database.

I haven't reviewed the code yet, so I will try to verify this as well, but remember that, if the client crashes for some reason, you most likely can't assert for certain the state of the socket: for example, was something only partially written to it? So you may need to drop the connection altogether or do some sort of cleanup (if possible), like make sure to abort transactions and other things. I am not sure how much state can be carried around.

We keep a linked DbHandler which originally created the socket, and if the client goes down, DbHandler will be respawned with a new direct connection.

I also remember we discussed improvements to poolboy or your fork to poolboy. I don't remember what they were exactly though. I assume those may already be in place?

I spent some time finding bottlenecks, and the biggest one was metrics collection (shout out to @hauleth who fixed that) and the synchronous calling between ClientHandler and DbHandler to avoid overwhelming the mailboxes. So far, we haven't added anything new to Poolboy, except our previous changes for idle timeout.

Is it correct that this pull request still preserves the old code for talking to the database? Is the plan to remove it in the future or are there issues in unifying everything through the proxy handler?

DbHandler is still responsible for establishing direct connections in transaction mode, reconnecting in idle state, and it has been extended with handler_call to change ownership of the TCP socket. However, the connection logic for talking with the database during transactions is located in ProxyDb. In the future, we will remove the old module.

Is the maximum number of ports (65k) a potential concern here? If so, could we use a single Ranch port but pass some other information (such as a preamble when connecting via gen_tcp) to choose with pool to use?

Not at the moment, but it could potentially become a problem. The update is already quite complex, so I don't want to add further complications. We can always add this feature later

How much do you handoff to the other node? From looking at the code, if it is on a different node, you are doing both auth and ssl handshakes. Does it mean you are doing them twice? On in the current node and then once you have to proxy? I wonder if it is worth doing a "special handshake" between those nodes, so when you join this special Ranch port (which should not be exposed), you can assume auth and ssl have been handled in the first node and you don't have to redo it. Maybe that's already the case, but I wanted to double check.

Hmm, it should not use SSL for proxying to another node. The main reason for authentication was to provide an additional security guard. It's not very resource-intensive but can prevent accidental connections if the client has some issues with authentication.

Fantastic. I have already reviewed the code and I will do another pass later. My only suggestion so far would be to have a custom tcp handshake for the proxy, so you have only a single special port instead of a pool of them and there is no need for additional auth. For security, you can put a 20-40 bytes random token on Syn instead of the Ranch port, and the new handshake uses a key plus this token for connecting. But as you said, I don't think it is a necessary change at this moment.

We are still measuring performance for different scenarios. When the pooler is on the same machine as the database, Supavisor outperforms or performs the same as PgBouncer. However, it is slightly slower when a Supavisor cluster of two nodes is in the same AWS zone as the database.

This is great! As far as I know, no other "bouncer" offers clustering anyway, and slightly slower would certainly be expected.

My only suggestion so far would be to have a custom tcp handshake for the proxy, so you have only a single special port instead of a pool of them and there is no need for additional auth. For security, you can put a 20-40 bytes random token on Syn instead of the Ranch port, and the new handshake uses a key plus this token for connecting

a good call, thanks!


gapindex version of the GOA Split Fractions Table

Extending the conversation from afsc-gap-products/data-requests#76, putting in a request to use the gapindex R package to recreate the tables in GOA.SPLIT_FRACTIONS.

@Ned-Laman-NOAA
Just committed an R script that creates the GOA.SPLIT_FRACTIONS table using gapindex. In this folder of the repo, you'll see the R script that creates a csv file which shows the differences between the biomass output from the current SPLIT_FRACTIONS table (2019-2023) vs those created from gapindex.

Thank you. This looks great. Where will we serve these data from?

I would consider this a non-standard data request and to not support this table in GAP_PRODUCTS going forward. Now that we have an R script that produces the same output using gapindex and GAP_PRODUCTS, it should be very easy to fulfil a request via the data-requests repo by simply changing the arguments in the gapindex::get_data() function OR provide the requesters that R script so they can do it themselves. Do you think either of those avenues are feasible going forward?

In the future, split fractions from GOA will not be necessary because of restratification. Historically, we provided split fractions as a standard product for SAs and AKFIN. With those two bits of information in mind, and given that the requests I was fielding this year were to adjust groupings of rockfishes, I agree that going forward this can be a data request rather than a standard product and thus does not need to be formalized in GAP_PRODUCTS.


AzCopy for MacOS ?

Hi,
Is there AzCopy available for MacOS?
We see for Windows and Linux here
https://docs.microsoft.com/en-us/azure/storage/common/storage-use-azcopy-linux?toc=%2Fazure%2Fstorage%2Ffiles%2Ftoc.json, but nothing for Mac.
Considering that we run powershell on Macs it would be nice to have this tool compiled for Mac.
Thanks

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

ID: e224df65-311e-166c-caff-918f47ad2f57
Version Independent ID: 21914f58-1720-39d9-acb7-ed8475cab9d4
Content: Copy or move data to Azure Storage with AzCopy on Linux
Content Source: articles/storage/common/storage-use-azcopy-linux.md
Service: storage
GitHub Login: @seguler
Microsoft Alias: seguler

@123dev Thanks for your feedback! We will investigate and update as appropriate.

@123dev There is a fairly new project still in preview allowing support for MacOS: https://github.com/Azure/azure-storage-azcopy
You can check it on the provided link.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

Hi Adam-Smith
Many thanks for a prompt follow up.
Checked the link, which suggested that there should be links for 3 downloads

Download the AzCopy executable using one of the following links:
Windows x64
Linux x64
MacOS x64

But there are no links, so I attempted to build it myself.
Have no experience with GO, installed Go and added one by one the respective dependencies
until I encounter the following error.
common/credCache_darwin.go:28:2: cannot find package "github.com/jiacfan/keychain" in any of:
        /usr/local/opt/go/libexec/src/github.com/jiacfan/keychain (from $GOROOT)
        /Users/hrant/go-workspace/src/github.com/jiacfan/keychain (from $GOPATH)
cmd/cancel.go:28:2: cannot find package "github.com/spf13/cobra" in any of:
        /usr/local/opt/go/libexec/src/github.com/spf13/cobra (from $GOROOT)
        /Users/hrant/go-workspace/src/github.com/spf13/cobra (from $GOPATH)
common/credCache_linux.go:26:2: cannot find package "github.com/jiacfan/keyctl" in any of:
        /usr/local/opt/go/libexec/src/github.com/jiacfan/keyctl (from $GOROOT)
        /Users/hrant/go-workspace/src/github.com/jiacfan/keyctl (from $GOPATH)
cmd/cancel.go:28:2: cannot find package "github.com/spf13/cobra" in any of:
        /usr/local/opt/go/libexec/src/github.com/spf13/cobra (from $GOROOT)
        /Users/hrant/go-workspace/src/github.com/spf13/cobra (from $GOPATH)
cmd/cancel.go:28:2: cannot find package "github.com/spf13/cobra" in any of:
        /usr/local/opt/go/libexec/src/github.com/spf13/cobra (from $GOROOT)
        /Users/hrant/go-workspace/src/github.com/spf13/cobra (from $GOPATH)

I can't find github.com/jiacfan/keychain to add that dependency.
If you prefer I move this as an issue into that repo, I'll gladly do that.

Thanks

@123dev I'd highly recommend moving it to an issue within the Azcopy repo

Thanks Adam-Smith
Done


News item 50% response rate

Title
Community curation  response rate reaches 50%
800/1600 publications assigned to community members for curation are finished.  A big thank you to everyone who has participated so far.

Do we have a canto logo  thumbnail size? I guess we could use that here?

Hi KIm, if you can size me down a logo, I can make the news item later.
v

I've added a Canto thumbnail. You should be able to use it with this in your news item:
 <!-- newsfeed_thumbnail: canto.png -->

I committed this one

That's live now.


Feature request: preserve order in value

Currently, value is based on BTree map, which loses the original order of ron document. I think it makes sense to add an option to preserve order. I'd even say that if ron's primary purpose is to be a human-editable format, then order should be preserved by default.  https://crates.io/crates/indexmap might come in handy do actually implement this.
Here's my specific use-case for ordering: I use ron to generate the AST for Rust. As such, I group related things, like types or expressions, together, and ideally I'd love to see the same order in the generated code. Alas, as I use a map for some part of my input, it doesn't work. I can swtich to a list of pairs for this case, but a mapping makes more sense.

@matklad this sounds very reasonable

If support is added for an ordered map parsing mode, how would you like this functionality to be enabled? A macro? An object-format flag? One frustrating thing that could occur now is that the serialized format doesn't show whether it is meant to be parsed in "preserved order mode" or "ordered key mode." This could be problematic if we ever want to provide syntax highlighting support for RON. Should we consider schemas for this?

@VermillionAzure as I understand this issue, it's not about parsing, it's about serialization only. The parser should work regardless, and no new extensions are needed. I think it can just be an option of the serializer (or a compile feature that enables it and adds dependency on indexmap).

@VermillionAzure Are you still interested in fixing this issue?

Hello, does this will be implemented. I am not sure if i have the same issue, but when I Deserizlise, i do not have the same structure order.
I am using hashmap as
(
tasks: {
    "debug message": Dbg(
      msg: "test message. some text after it."
    ),
    "shell command": Shell(
      command: "ls",
      args: Some([
        "-l",
        "-h",
      ]),
      ch_dir: Some("/")
    ),
}
)

and it does not preserve order of it, so that sometimes shell command goes first.

@lykhouzov if your tasks is a HashMap, then yes, you are hitting the same issue.
I don't know if you need to have a hashmap there though, might easily do tasks: [...] as a list.

Hi, @kvark. So yes, it is HashMap.
I already was thinking about use Vec instead, but this construction looks more readable for me.
Porbably i will convert it to something like following if this issue is not gonna be fixed
[
    ("debug message", Dbg(
      msg: "test message. some text after it."
    )),
    ("shell command",Shell(
      command: "ls",
      args: Some([
        "-l",
        "-h",
      ]),
      ch_dir: Some("/")
    ))
]

or maybe move a name of a task to task itself.
Thanks

@lykhouzov You could also just replace the HashMap type by IndexMap.

Blocked by https://github.com/bluss/indexmap/issues/91


[BUG] invalid_unless_false generates a warning and raises an exception even when config is valid

setuptools version
67.7.0
Python version
3.7
OS
Ubuntu Linux
Additional environment information
No response
Description
Installing a package that sets the use_2to3 command to false results in a warning and an exception being raised.
Expected behavior
No exception should be raised because calling invalid_unless_false with false is valid.
How to Reproduce

Upgrade setuptools to version 67.7.0
git clone git@github.com:pyvisa/pyvisa-sim.git && cd pyvisa-sim
Checkout version that uses use_2to3 flag with  git checkout 0.3
Install using pip install .

Output
jamhernandezq@JHQ jam % git clone git@github.com:pyvisa/pyvisa-sim.git
Cloning into 'pyvisa-sim'...
remote: Enumerating objects: 967, done.
remote: Counting objects: 100% (242/242), done.
remote: Compressing objects: 100% (83/83), done.
remote: Total 967 (delta 190), reused 170 (delta 159), pack-reused 725
Receiving objects: 100% (967/967), 254.20 KiB | 2.27 MiB/s, done.
Resolving deltas: 100% (604/604), done.
jamhernandezq@JHQ jam % cd pyvisa-sim
jamhernandezq@JHQ pyvisa-sim % git checkout 0.3
Note: switching to '0.3'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at f7c4158 Preparing release 0.3
jamhernandezq@JHQ pyvisa-sim % ls
AUTHORS		CHANGES		LICENSE		MANIFEST.in	README		docs		pyvisa-sim	setup.cfg	setup.py
jamhernandezq@JHQ-XG2HY37XND pyvisa-sim % pip install setuptools --upgrade
Requirement already satisfied: setuptools in /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages (65.5.0)
Collecting setuptools
  Downloading setuptools-67.7.0-py3-none-any.whl (1.1 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.1/1.1 MB 6.9 MB/s eta 0:00:00
Installing collected packages: setuptools
  Attempting uninstall: setuptools
    Found existing installation: setuptools 65.5.0
    Uninstalling setuptools-65.5.0:
      Successfully uninstalled setuptools-65.5.0
pip instaSuccessfully installed setuptools-67.7.0
l%
jamhernandezq@JHQ pyvisa-sim % pip install .
Processing /Users/.../jam/pyvisa-sim
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [3 lines of output]
      /private/var/folders/4z/djhjm20x6b925sp6ljz2bhwr0000gn/T/pip-build-env-q_8awvro/overlay/lib/python3.11/site-packages/setuptools/dist.py:954: DistDeprecationWarning: use_2to3 is ignored.
        ep.load()(self, ep.name, value)
      error in PyVISA-sim setup command: use_2to3 is invalid.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.
jamhernandezq@JHQ pyvisa-sim %

Closing in favor of #3898


Add a nicer way to set features in spirv-builder

It's pretty opaque setting features using spirv-builder:
https://github.com/Rust-GPU/rust-gpu/discussions/144
We should have a function on SpirvBuilder that makes it easy, as this is a potential common use-case.
Here are where the rustflags are set in the builder: https://github.com/Rust-GPU/rust-gpu/blob/0da80f8a61867590a0824873fa45dc8983e49da8/crates/spirv-builder/src/lib.rs#L663

Oh weird, missed that in the docs.


Google pay button is not showed

The Google pay button is not shown. I'm have integrated the stripe google pay button in my react app. And set the react app url in my mobile app webview, But the google pay button not showed. The chrome and other browser I call the react app URL it's working Good

If it is about "Google Pay not working" it might make sense as Google Pay is not compatible with WebView. You need to use native bridge.
Here below the notes from Google (source: https://developers.google.com/pay/api/web/support/faq)
Does the Google Pay API integration work in WebViews?
For Android apps which use WebViews, you must invoke platform specific Android Google Pay APIs. See [Binding JavaScript code to Android code](https://developer.android.com/guide/webapps/webview#BindingJavaScript) for examples.

Facing same issue with google pay.


[SPARK-28549][BUILD][CORE][SQL][test-hadoop3.2][test-maven] Use text.StringEscapeUtils instead lang3.StringEscapeUtils

What changes were proposed in this pull request?
org.apache.commons.lang3.StringEscapeUtils was deprecated over two years ago at LANG-1316. There is no bug fixes after that.
/**
 *
Escapes and unescapes {@code String}s for
 * Java, Java Script, HTML and XML.

 *
 *
#ThreadSafe#

 * @since 2.0
 * @deprecated as of 3.6, use commons-text
 *
 * StringEscapeUtils instead
 */
@Deprecated
public class StringEscapeUtils {

This PR aims to use the latest one from commons-text module which has more bug fixes like
TEXT-100, TEXT-118 and TEXT-120 like the following replacement.
-import org.apache.commons.lang3.StringEscapeUtils
+import org.apache.commons.text.StringEscapeUtils

This will add a new dependency to hadoop-2.7 profile distribution. In hadoop-3.2 profile, we already have it.
+commons-text-1.6.jar

How was this patch tested?
Pass the Jenkins with the existing tests.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13381/
Test PASSed.

Test build #108281 has started for PR 25281 at commit 4f680c8.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13382/
Test PASSed.

Test build #108282 has started for PR 25281 at commit 4f680c8.

Test build #108281 has finished for PR 25281 at commit 4f680c8.

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

Merged build finished. Test PASSed.

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

Test build #108282 has finished for PR 25281 at commit 4f680c8.

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

Merged build finished. Test PASSed.

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

I checked them by myself and the fix looks fine to me;
(base) maropu@~/Repositories/spark/spark-master: (master $%=)$find . -type f | grep -e "\.scala$" | xargs grep org.apache.commons.lang3.StringEscapeUtils
./core/src/main/scala/org/apache/spark/ui/HttpSecurityFilter.scala:import org.apache.commons.lang3.StringEscapeUtils
./core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala:import org.apache.commons.lang3.StringEscapeUtils
./core/src/main/scala/org/apache/spark/ui/jobs/JobPage.scala:import org.apache.commons.lang3.StringEscapeUtils
./core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala:import org.apache.commons.lang3.StringEscapeUtils
./core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala:import org.apache.commons.lang3.StringEscapeUtils
./core/src/main/scala/org/apache/spark/ui/scope/RDDOperationGraph.scala:import org.apache.commons.lang3.StringEscapeUtils
./repl/src/test/scala/org/apache/spark/repl/SingletonReplSuite.scala:import org.apache.commons.lang3.StringEscapeUtils
./sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala:import org.apache.commons.lang3.StringEscapeUtils
./sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala:import org.apache.commons.lang3.StringEscapeUtils
./sql/core/src/main/scala/org/apache/spark/sql/execution/ui/SparkPlanGraph.scala:import org.apache.commons.lang3.StringEscapeUtils
./sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/ui/ThriftServerPage.scala:import org.apache.commons.lang3.StringEscapeUtils
./sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/ui/ThriftServerSessionPage.scala:import org.apache.commons.lang3.StringEscapeUtils
./streaming/src/main/scala/org/apache/spark/streaming/ui/BatchPage.scala:import org.apache.commons.lang3.StringEscapeUtils
./streaming/src/main/scala/org/apache/spark/streaming/ui/UIUtils.scala:import org.apache.commons.lang3.StringEscapeUtils

cc: @srowen

Merged to master.

Thank you, @maropu and @HyukjinKwon !


Add Support for Google Downloadable Fonts?

Been using calligraphy for a while now. However, theres an issue i would like to share. The only way to use calligraphy is via including your own .ttf files. (afaik ;_; correct me if im wrong) Since, .ttf files can get large for most fonts, they contribute heavily to the overall app size. Therefore, Google downloadable fonts are a much light weight alternative for custom fonts in android now.
It would be great if we could set a google downloadable font as the calligraphy global font. I dont know if this is possible in the current implementation (if at all) but will really appreciate this feature.

I would love to see a similar feature to be able to download ttf at runtime

I would love to see a similar feature to be able to download ttf at runtime

Probably unlikely as that would mean we would have to include something like OkHttp in the project and we try to keep things lightweight.

What might be enough is to provide a way to set the Typeface object for the font, and not just the static path. Because the result of getting the downloadable font is the Typeface.

True, I don't really have time look right now. But between ViewPump and Calligraphy should be possible, happy to look and shape up PR's if you have time to submit one


fix: nexted object keys and single quote strings

@marcusrbrown not super happy with how I am detecting the end of a key / object.
Maybe we need a begin / end match for key-objects vs single line keys.
I could not get object as a standalone type to function correctly, so I nested the logic into the keys and optional keys.
Thoughts?

Example from vscode-pbtxt repo:
https://raw.githubusercontent.com/thesofakillers/vscode-pbtxt/master/.vscode-test/test.pbtxt

Ok, I think this might be the ticket now.

 Haha

The changes look good - anything else needed for this PR? I'd recommend trying a feature where we generate from YAML in a similar vein as the official vscode extensions. That process may offer a bit of clarity to help handle those funky edges, as well as open up another testing pipeline. I'm happy to get the ball rolling and get you a draft PR.

The changes look good - anything else needed for this PR? I'd recommend trying a feature where we generate from YAML in a similar vein as the official vscode extensions. That process may offer a bit of clarity to help handle those funky edges, as well as open up another testing pipeline. I'm happy to get the ball rolling and get you a draft PR.

Would love to see this. Just to be sure I understand.

Create a YAML file for the grammar
Generate JSON from YAML
Get more GH Stars!?


OTG Test - RT-2.2: IS-IS LSP Updates

OTG conversion of RT-2.2: IS-IS LSP Updates
Output log found here
Fixes #620

@octpetre : The base adjacencies test needs an update for the modified ConfigISIS function.

I will update the file in the #735. Here is not needed

@jasdeep-hundal could you please approve after the latest ygnmi API changes

/gcbrun


Optimize decoding of varint CSR

What do these changes do?

add back the offset array for correct degree
use v8dec/nec for varint encoding
implements batch decoding in CompactNbr

Related issue number
Part of #1373

🐮


⚠ AsianCinema has degraded performance

In 8522319, AsianCinema ($ACM) experienced degraded performance:

HTTP code: 200
Response time: 16396 ms

Resolved: AsianCinema performance has improved in e5fa8cf.


Remove DependencyInfoBlock

We find that there is a DependencyInfoBlock in your APK.
It's a Signing block added by AGP and encrypted with the Google public key so it can't be read by anyone else except Google. You can read more about it here and here.
While this was added a while ago, we were only enforcing it for new apps, and recently we started scanning updates too.
Could you please disable it with the following code?
android {
    dependenciesInfo {
        // Disables dependency metadata when building APKs.
        includeInApk = false
        // Disables dependency metadata when building Android App Bundles.
        includeInBundle = false
    }
}

Thanks!

Done: https://github.com/aj3423/SpamBlocker/blob/ada4e44692b525bb3d085ba18c381e402e80596c/app/build.gradle.kts#L10-L15

Thanks!


[Feature request] show tracker information in serial console log

I think it would be helpful for providing tech support to just print the basic configuration of the firmware at startup, so for example the board, the imus, the pins etc. This would make identifying incorrectly flashed firmware much simpler.

The firmwarte is capable of recognize GET INFO and GET CONFIG serial commands:
https://github.com/SlimeVR/SlimeVR-Tracker-ESP/blob/main/src/serial/serialcommands.cpp#L328
https://github.com/SlimeVR/SlimeVR-Tracker-ESP/blob/main/src/serial/serialcommands.cpp#L154-L161
Isn't this more request for GUI to add Get Config button to sertial console commands?

Ehh, i mean, kinda, but i also just think its convenient if somebody asks for help and shows the serial console log that i dont have to specifically ask for get info or stuff like that. I get how its not that hard to ask for get info though, so i wouldnt exactly call this high priority


Add disabled state for colored buttons. Move all disabled to bottom.

Adding disable state for colored buttons to fix issue #907. While i was here, I moved all the disabled states down to the bottom to ensure maximum specificity for the component.

LGTM in all browsers. Thanks, @Garbee :)


Update simple request example

Hello,
New guy using Rust here.
This quarantine morning I was trying to run the current example of simple request. I just was expecting  to copy and paste it and see how it work, but first thing I got was:
$ cargo build
   Compiling tokio v0.2.18
   Compiling tokio-util v0.3.1
   Compiling tokio-tls v0.3.0
   Compiling h2 v0.2.4
   Compiling hyper v0.13.5
   Compiling hyper-tls v0.4.1
   Compiling reqwest v0.10.4
   Compiling restget v0.1.0 (/Users/omar/code/rust/restget)
error[E0107]: wrong number of type arguments: expected 2, found 1
 --> src/main.rs:5:14
  |
5 | fn main() -> Result<()> {
  |              ^^^^^^^^^^ expected 2 type arguments

With the help of @onielfa he manage to fix the example , there is something we are not catching form the original example or it wasn't working with current version of rust:
$ rustc --version
rustc 1.42.0 (b8cedc004 2020-03-09)

and modules:
reqwest = { version = "0.10"}
tokio = { version = "0.2", features = ["full"] }

Also will be nice to have the Cargo.toml requirements into the documentation, specially for people which is starting with Rust and they go directly to the example.
I don't know if this is OK or NOT , but maybe could help.

Nice, very similar to what I'm introducing in #575
What I find is when I update the reqwest version to .10, 15 recipes break.  I can't accept a PR that breaks 14 other examples.
The workaround is to use Tokio 0.8 as show inside the Cargo.toml file (but isn't spelled out in every recipe).  The solution is to update 15 recipes.

Oh I see , maybe I could try thanks for the feedback @AndyGauge and I didn't see #575 🙀 sorry!

Id be happy to accept a PR that addresses all the recipes.  I'm unlikely to finish #575 on my own.

@AndyGauge I make work some code into the examples but I don't know why test are still breaking, I was checking https://github.com/budziq/rust-skeptic but for me everything is ok :S, any clue why the code is still breaking? As I told you I'm new with rust in fact this is my second day 🙀
Thanks

The scope is a bit extreme.  Maybe we should concentrate on one recipe at a time.

Looks like there are less test to fix:
I started with 13:
    basic_sect_download_a_file_to_a_temporary_directory_line_12
    broken_sect_check_a_webpage_for_broken_links_line_13
    extract_links_sect_extract_all_links_from_a_webpage_html_line_11
    get_sect_make_a_http_get_request_line_10
    paginated_sect_consume_a_paginated_restful_api_line_9
    partial_sect_make_a_partial_download_with_http_range_headers_line_12
    post_file_sect_post_a_file_to_paste_rs_line_12
    request_sect_parse_the_mime_type_of_a_http_response_line_14
    rest_get_sect_query_the_github_api_line_8
    rest_head_sect_check_if_an_api_resource_exists_line_11
    rest_post_sect_create_and_delete_gist_with_github_api_line_14
    retain_sect_avoid_discarding_errors_during_error_conversions_line_15
    unique_sect_extract_all_unique_links_from_a_mediawiki_markup_line_11

and right now are 7 :
    basic_sect_download_a_file_to_a_temporary_directory_line_12
    broken_sect_check_a_webpage_for_broken_links_line_13
    paginated_sect_consume_a_paginated_restful_api_line_9
    partial_sect_make_a_partial_download_with_http_range_headers_line_12
    post_file_sect_post_a_file_to_paste_rs_line_12
    request_sect_parse_the_mime_type_of_a_http_response_line_14
    retain_sect_avoid_discarding_errors_during_error_conversions_line_15

Was hard to me to read the test output :S , right now is more clear thanks for your examples @AndyGauge and support.
I will try to follow on this during the next weekend

Just 3 left
    broken_sect_check_a_webpage_for_broken_links_line_13
    paginated_sect_consume_a_paginated_restful_api_line_9
    retain_sect_avoid_discarding_errors_during_error_conversions_line_15

Two remaining:
 paginated_sect_consume_a_paginated_restful_api_line_9
 retain_sect_avoid_discarding_errors_during_error_conversions_line_15

I'm having troubles with the error types there , we will see 🧷

Your work made it into the cookbook.  Thank you for your contribution!


Arbitrary configuration loading

Hi
Thanks for this deployer addon, really helpful!
I just set it up in a local project and wanted to put the jobs.yaml into the root directory instead of cluttering the project setup with an etc/* folder that wouldn't be used otherwise – most of the configs, like the deployer configuration files, are in the project root anyways.
I discovered that the deployment always failed because deployer-cron tried to load any file from the configured directory; .sql, .md, other .yml files, etc. I think it would be a better solution to either have a fixed filename or some sort of resolver for files that is restricting to certain file types. Loading SQL files wouldn't be a use case in the real world anyway.

Yeah, this is right. Would be better to make that configurable somehow. Can you come up with a PR? :)

I'll check if I find some time in between restructuring all the chaos projects I've been assigned to.. ^^

Perfect, thanks :)

I just released an alpha version that supports v7. This should also fix this issue


OpenBSD build, no waitid(2)

Hi,
I'm trying to build Criterion on OpenBSD and I'm stuck on build of BoxFort because there is no waitid(2) on OpenBSD
https://github.com/diacritic/BoxFort/blob/master/src/sandbox-posix.c#L161
https://github.com/diacritic/BoxFort/blob/master/src/sandbox-posix.c#L173
Is there any workaround I could try to do the same things without this syscall ? If you can explain me what this is supposed to do I can try to find something.
Regards

waitid(2) is mainly use to check whether a child is ready to be reaped without reaping it now (and leaving that up to a later wait syscall). It could be emulated using waitpid and a queue, queueing the element for a later call.


Display routing/trxn fee in transactions list

Task
We store the transaction fee for each payment.
We should display the fee for successful transactions in the transaction list area for successful transactions.

@tomsmith8 Can i pick this up?

@Nightrider0098 to be fair they should assign me since i identified the issue :(

We need backend changes for this ticket.


Missing secDevLabs logo from the Home page

I've noticed that while the secDevLabs logo is available at Projetos, it is missing from the Home page, where it was replaced with an uncentralized H2 tag. Was it intended? If not, I would like to try to fix it!

Hey @LFRusso, in Home Page we have a dark background and maybe some logos doesn't contrast very well. If it's the case, probably you'll have to invert the logo color.
But yes, you can fix it 👍

So I might be wrong (I'm pretty new to Gatsby), but it seems like I can't really change the projects images directly as they are being loaded here. Also, I've tested locally and the original colors are indeed a little off in the dark background

So tried inverting the black and white parts:

Would adding the inverted version to /static/images (as I've done here) be enough?

hey @LFRusso the inverted version seems a little bit weird, i think it's better just centralize the h2 tag.


Update value type of contact attributes.

Issue #, if available:
https://github.com/aws/amazon-connect-streams/issues/102
Description of changes:
Update value type of contact attributes in TypeScript type definition file.
Old type: string
New type: { name: string, value: string }
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you!


Add 'PYTHON|3.11' to runtimeStack input, in AzureFunctionApp@2 documentation

Python 3.11 is supported by the AzureFunctionApp@2 task, but it's missing in the documentation.

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

ID: 87b190ff-4ca4-a7ca-381d-e97185d05df1
Version Independent ID: 87b190ff-4ca4-a7ca-381d-e97185d05df1
Content: AzureFunctionApp@2 - Azure Functions Deploy v2 task
Content Source: task-reference/azure-function-app-v2.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-tasks
GitHub Login: @steved0x
Microsoft Alias: sdanie

Does the runtimeStack work for updating the language version?

Thank you for your feedback. We are migrating this issue to our new documentation feedback system, and closing this issue.
As we phase out GitHub issues for this repository, we encourage you to propose changes using public PRs or submit feedback through our updated control at the top of the page. For more information on our feedback systems, see https://aka.ms/contentuserfeedback.


Pre trained model

Could you share a pretrained models to be able to test without need to train?

Hi @deli4iled
Here you go!
https://1drv.ms/u/s!ApbTjxa06z9CgRh9Klo4K1xZgHF9

Thank you very much!


Pass through socket options

Currently the code attempts to map options to the TCPSocket. This PR adds a tcpSocket property to the options object that is provided to TCPSocket.connect as its options. This allows the features of the underlying shims to evolve without needing to modify BrowserBox.
This PR was motivated by my need to set the ws.url for a socket.io proxy running on a host other than the one serving the client code.

Ack! wrong branch. meant for v2.


RIO-51 feat(CollectionActor): Support PreUpdate Hook

Can one of the admins verify this patch?

ok to test

ok to test

Refer to this link for build results (access rights to CI server needed):
http://ci.really.io//job/really-pr-validator/61/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
http://ci.really.io//job/really-pr-validator/62/
Test FAILed.

retest this please

Refer to this link for build results (access rights to CI server needed):
http://ci.really.io//job/really-pr-validator/63/
Test FAILed.

@HatemAlSum Tests are failing

Refer to this link for build results (access rights to CI server needed):
http://ci.really.io//job/really-pr-validator/84/
Test PASSed.


detect links only w/o profiles

Hi! I'm wondering if it's possible to detect social links only, not profiles. For example:
const detectSocialLink = (url: string) =>
  networks.reduce<[network: typeof networks[number] | 'url', url: string]>(
    (prev, network) => {
      try {
        if (socialLinks.isValid(network, url)) {
          return [network, socialLinks.sanitize(network, url)];
        }
      } catch (err) {
        return prev;
      }

      return prev;
    },
    ['url', url],
  );

expect(detectSocialLink('https://instagram.com/vinspee')).toBe(['instagram', 'https://instagram.com/vinspee'])
expect(detectSocialLink('vinspee').toBe(['url', 'vinspee'])

i'd like to take user input and determine which network was detected. When "profile" names match, essentially everything becomes a match until it looks like a URL.

Good point.
I added detectProfile in v1.7.0
I can also automatically detect profile in following functions when profileName is omitted.
getProfileId
getLink
isValid
sanitize
What do you think about this @VinSpee ?

I think that nails it. Looks good! Thanks.


nvm.sj domain is throwing a 404

When you try to go to nvm.sh, it redirects to:
https://github.com/creationix/nvm/blob/master/README.markdown
which throws a 404. :)

Fixed; may take up to an hour to update.


Fix UInt docs

Description

Fixes some issues in the docs, and also makes them compile on nightly.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (main)
[ ] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[ ] Wrote unit tests
[ ] Updated relevant documentation in the code
[ ] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

(a reference implementation is in https://github.com/arkworks-rs/r1cs-std/tree/fix-doc-attr, which only fixes the errors but not improves the documentation system)
Feel free to delete that branch when this one is merged.


Color Picker: Use Raycast Swift Macro

Description
Migrate the extension to use the Raycast Swift Macro instead of manually compiling the Swift code into a binary.
There should be no changes feature-wise.
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi, I'm working on #8534 and encountered a swift compile problem
npm run build

> build
> ray build -e dist

info  -  entry points [src/pick-color.ts src/menu-bar.tsx src/organize-colors.tsx]
error -  compiling swift package
build failed (src/pick-color.ts:5:26 import { pickColor } from "swift:../swift/color-picker";): failed to compile swift:
exit status 1

If this is unexpected, try to delete the color-picker/swift/color-picker/.raycast-swift-build folder and run this command again

After the switch to Raycast Swift Macro, I'm not able to figure out how to build it, I've read the official docs but get no clue, can you give some help on this?

I tried to upgrade it to 1.0.0 which uses @raycast instead of #exportFunction and #handleFunctionCall, however I can't build it via build script.
I don't have background knowledge of swift development, do I need to open XCode to compile this extension, or I could just get to sucess by simply using npm run build?

@otakustay Thinking about it, it may be a bit complex to do this migration, especially since the Swift Macro is quite new. I'll migrate it and get back to you 🙂


Fix ui tests for latest rust stable

Just tested it locally, the paritytech/ci-linux:staging image passes test-linux-stable and check-web-wasm tests. Can be merged together with https://github.com/paritytech/substrate/pull/6223


Update hooks-intro.md

What changed
Made link to Writing Custom Hooks lead to the top of the page rather than to Using a Custom Hook
Before the change
When you clicked the link to Writing Custom Hooks from hooks-intro.html:

You would be taken to /docs/hooks-custom.html#using-a-custom-hook:

After the change
Now when you click the link, you are taken to the top of the page, /docs/hooks-custom.html, as you would expect:

Thanks!


cmd windows not closing on eclipse startup

Category: Bug or enhancement
Bug
Severity: Low , Medium, High, Critical
High
Expected behaviour (In case of bug)
The initiating cmd window as well as the command window of eclipse.exe should vanish if eclipse is launched.
Actual/current behaviour (In case of bug)
executing eclipse-main.exe results in the two windows keep open until you close eclipse again. This is messing up the windows on each machine.

Steps to reproduce (In case of bug)
run eclipse-main.exe, two windows are opened and eclipse.
Description of issue / expected enhacement /Comments
Your environment - windows/Linux, Devonfw version , component
win7

I assume it is the call command:
https://github.com/devonfw/devon-ide/blob/ed65ea578340107b1a6ae53192bd40bb9c26c4d7/scripts/src/main/resources/scripts/functions#L854
Can some windows guy play around with the eclipse-*.bat script and let me know it it works if we just remove the "call" in front of the devon command?

Was autoclosed by github due commit comment.
Manual retest with beta14 on windows confirmed it is working for

windows CMD
git-bash (MINGW)

However it fails on cygwin:
$ devon eclipse
ERROR: DEVON_IDE_HOME does not exist!
/cygdrive/c/projects/beta5
It seems as if you original installation has been moved.
Unknown command eclipse

And trying to test it in powershell reveals that with new windows 10 also PowerShell integration is broken. I get a popup:
This file does not have a program associated with it for performing this action. Please install a program or, if already installed, create an association in the Default Programs control panel.

Both latter bugs have nothing to do with this fix. I will file new separate issues for that.


Can deploy QAT_Engine on only few servers

And other servers can send handshake data to the QAT_Engine, and get the decrypt data
How?

Hi @crasyangel,
I'm not sure I completely understand your question.
Are you hitting a limit on how many servers you can enable for Quickassist?
If that is your issue I will need details of the following:

Which QuickAssist product are you using?
How many servers (processes) are you running and after how many processes do you hit the limits?
How are your QAT Driver config files configured in the [SHIM] section?

Kind Regards,
Steve.

Hi @stevelinsell

Hi @stevelinsell

Which QuickAssist product are you using?
icp_dh895xcc_dev0

What is the question?
We have hundreds of nodes, and hundreds of machines within each node. So plug acc card in every machine is not possible, quite expensive, and not necessary. We want plug some machine with one node, and all machines can use those ability which have acceleration card, and offload ssl key exchange calculation by acceleration card

This sounds like an SSL termination use case. It is possible to do that via something like the HAProxy "engine" (and "async") feature:
https://cbonte.github.io/haproxy-dconv/1.8/configuration.html#ssl-engine
https://cbonte.github.io/haproxy-dconv/1.8/configuration.html#ssl-mode-async

Hi @crasyangel,
So are you effectively trying to use one machine fitted with QuickAssist as an HSM that you offload the handshake processing to from your other machines?
Something functionally similar to Amazon's AWS CloudHSM:
https://docs.aws.amazon.com/cloudhsm/latest/userguide/introduction.html
but something you completely own all the infrastructure to?
Kind Regards,
Steve.

Closing through inactivity.


Invalid response type of deleteMessage()

The response type of the deleteMessage() function should be Call<Response<Boolean>>,
not Call<Response<Message>>
Expected behavior
No errors when call deleteMessage()
Actual behavior
Responce error when call deleteMessage()
To Reproduce
deleteMessage(c_id, m_id)
Screenshots/Traceback
Response:
nov 04, 2020 2:38:04 AM okhttp3.internal.platform.Platform log
INFO: {"ok":true,"result":true}
Possible solutions
Change <Message> to <Boolean>

From Telegram API:
"Returns True on success."

Thanks for the report @reopold! I'll take a look to it

Fixed in https://github.com/kotlin-telegram-bot/kotlin-telegram-bot/pull/122 @reopold!


fixes chinese character hex mismatch

Fixes #10

IssueHunt Summary

Referenced issues
This pull request has been submitted to:

#10: MD5 results are different in node and browser environments

IssueHunt has been backed by the following sponsors. Become a sponsor

This needs a test.

Done.


Accept float value of the foreign currency

Issue: The CLI was not able to forex c 1.1 USD
Please input a valid number for amount.
exit status 1
Solution: Use strconv.ParseFloat to parse the input amount

Thanks :+1:


Failed to set locale category

macOS Catalina 10.15.1 on MacBook Pro
After installing lightline with vim-plug, I get this warning:

Warning: Failed to set locale category LC_NUMERIC to en_KR.
Warning: Failed to set locale category LC_TIME to en_KR.
Warning: Failed to set locale category LC_COLLATE to en_KR.
Warning: Failed to set locale category LC_MONETARY to en_KR.
Warning: Failed to set locale category LC_MESSAGES to en_KR.

After adding export LC_ALL=en_KR.UTF-8 to .zshrc, I get the same warning twice. If I remove that line from .zshrc, I get one from lightline.

Adding export LC_ALL=en_US.UTF-8 to .zshrc fixes the issue (see this)? I believe the warning has nothing to do with this plugin. Please write the precise steps to reproduce the warnings. When you get the messages from what command? While starting Vim or Neovim?

Sorry I checked again and it was a different plugin.


[stable/airflow] adding podLabels to web, scheduler, flower and worker pods

What this PR does / why we need it:
Adds the ability to set labels on pods for the web, scheduler, flower deployments and worker statefulSet.
Copied the same format already used for setting podAnnotations.
Also cleaned up a duplicate "workers.podAnnotations" in the README
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @gsemet

@gsemet tried to assign to you as instructed but it wouldn't allow

look fine, just update the minor version and that will be good

/lgtm


[Feature] Add addEvents method

This pull request adds a new method for quickly adding multiple events at once..
I find myself having to loop though an array of objects so i find this more useful.

Hey Alex!
Thank you for getting back to me.. 👍
I'll impliment your changes when i find some time.. :)

Changes implemented

Hey @morloderex - sorry again for taking so long, life is hectic at the moment.
I've made a small performance tweak to the code, using array_filter and then array_merge effectively means you loop the input array twice (albeit in engine space), whereas in this instance just a foreach and a call to addEvent does the trick. Keeping event adding in addEvent also allows us to change things in one place in the future if needs be. Does that make sense? Happy with that change? (I felt guilty being quiet for ages and then asking for more changes, so did it myself!)
I've also tweaked the TravisCI file to get the build passing.
If you're happy with the above, I'll get this merged and tagged today. Thanks again for your patience!

Seems fine.. :)
If you really want to do it in that way :)
i'm not that great with conditionals inside a foreach loop anymore :P


LE Audio ensure unicast client is central

The unicast client shall be the central. Ensure that control point operations and discovery is only done as the central.

Review take 1. This PR is huge! - will continue from bap.c another day :-)

This PR includes all the commits from my other PRs, and is marked as a draft for that very reason :)

This PR includes all the commits from my other PRs, and is marked as a draft for that very reason :) @Casper-Bonde-Bose

One PR to rule them all! I get it @Thalley ;-)
As it have been a while since I looked at the code I actually prefer one large PR - it kind of sums up what I missed. Hope it is OK I'm doing review on this PR - rather than the individual once?

This PR includes all the commits from my other PRs, and is marked as a draft for that very reason :) @Casper-Bonde-Bose

One PR to rule them all! I get it @Thalley ;-) As it have been a while since I looked at the code I actually prefer one large PR - it kind of sums up what I missed. Hope it is OK I'm doing review on this PR - rather than the individual once?

I'd prefer it in the others, so I don't have to update the code in multiple PRs :) Would also make it faster to get the other PRs merged if you approve those instead of this, but I'll update the code regardless :)

I'd prefer it in the others, so I don't have to update the code in multiple PRs :) Would also make it faster to get the other PRs merged if you approve those instead of this, but I'll update the code regardless :)

@Thalley I'll try the code and then do review of the other PR's. most of the comments I had were questions rather than change requests :-)

@Casper-Bonde-Bose Hopefully processed your comments now, and updates the commits in all the PRs, or otherwise responded to your comments.
One thing to notice, is that most of the changes here are removing the capabilities implementation from the BAPS implementation, but does not otherwise fix other issues with the implementation, which has intentionally left as is. Hopefully once this is merged, fixing those things should be a lot easier than it previously have been (mostly due to the fact that I've removed the high coupling between the different roles).

LGTM When doing the review I was wondering if it would be possible to limit the number of checks by preventing the user of the API to use the functions for the wrong role? For example if possible failing an early function call that returns something that needs to be passed into the functions with the added checks?

Possibly.
The stream API may also be used for the unicast server in the future (the server may also change the state of the streams/endpoints), and if so the functions may be called with two different roles. We may also want to add a separate API for the unicast server, and then we can possibly optimize the checks as well.


Create My Therapist

Create a chatbot that can offer mental health support and counseling to individuals. It should be able to understand and empathize with users and help them cope
📝 Description
Add a brief description.
💣 Is this a breaking change (Yes/No):

[x] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

@nandinisahu407 Hello, your promoto seems to have some problems. If you need to use variables, please use {{}} instead of {}

@AruSeito updated suggested changes,kindly merge it

@nandinisahu407 Hello, your Prompt seems to have some problems. If you need to use variables, please use {{}} instead of {}

@AruSeito updated suggested changes, let me know if any more changes need to be done

@wenzhuo2022 kindly merge this


[Bug] When drawing to a canvas, fast circles result in frayed lines

Hello,
Thanks for your work on this amazing library. I want to use it in combination with a canvas but I have the following phenomenon
https://user-images.githubusercontent.com/596986/138428669-5f05170b-6df7-4ad1-b85d-9836542fcdef.mp4
As you can see, the first straight and slow lines work perfectly, but as soon as I start drawing quick circles the lines start to fray. Is there anything I can do to circumvent that? Let me know if you need any additional informations.

Hey, it looks like you’re not clearing the canvas before drawing the new line.
The easiest way to fix this is to call clearRect at the beginning of each frame (or whenever you want to update the image) in order to clear the canvas, then repaint all of the old lines, and then paint the new one too. You can make this more efficient by using two canvases, one for lines that are finished and another for the current line, so that you don’t have to repaint the old lines as often.
Why the jagged lines? This is something I call “sawblading” and it’s a result of the streamlining that the algorithm applies to your points. Basically the last point will try to be very close to your pointer, but other points will be placed closer to their previous point; so that a point will change positions depending on whether it’s the last one or not. You can reduce this effect by setting the last option to false until the line is complete.

Closing the issue but happy to discuss more in Discussions!

Thanks for the tip, works perfectly!


Spark 3.1 support

Databricks has introduced the 8 series runtimes which are build uppon Spark 3.1.1, as shown in the image below.
The com.microsoft.azure:spark-mssql-connector_2.12_3.0:1.0.0-alpha is perfectly working on Spark 3.0.x but unfortunately not working on Spark 3.1.x.
If possible it would be great if the Spark 3 connector could work with all Spark 3.x.x
If required updates for Spark 3.1 are not backward compatible with Spark 3.0 it would be great if a Spark 3.1.x compatible connector could be introduced
I would be happy to help but in this case I have no clue where to start.
error logs on Databricks Runtime 8.1 ML
Py4JJavaError: An error occurred while calling o483.save.
: java.lang.NoSuchMethodError: org.apache.spark.sql.execution.datasources.jdbc.JdbcUtils$.schemaString(Lorg/apache/spark/sql/Dataset;Ljava/lang/String;Lscala/Option;)Ljava/lang/String;
	at com.microsoft.sqlserver.jdbc.spark.BulkCopyUtils$.mssqlCreateTable(BulkCopyUtils.scala:506)
	at com.microsoft.sqlserver.jdbc.spark.SingleInstanceConnector$.createTable(SingleInstanceConnector.scala:46)
	at com.microsoft.sqlserver.jdbc.spark.Connector.write(Connector.scala:73)
	at com.microsoft.sqlserver.jdbc.spark.DefaultSource.createRelation(DefaultSource.scala:64)
	at org.apache.spark.sql.execution.datasources.SaveIntoDataSourceCommand.run(SaveIntoDataSourceCommand.scala:48)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult$lzycompute(commands.scala:73)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult(commands.scala:71)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.doExecute(commands.scala:94)
	at org.apache.spark.sql.execution.SparkPlan.$anonfun$execute$1(SparkPlan.scala:196)
	at org.apache.spark.sql.execution.SparkPlan.$anonfun$executeQuery$1(SparkPlan.scala:240)
	at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:165)
	at org.apache.spark.sql.execution.SparkPlan.executeQuery(SparkPlan.scala:236)
	at org.apache.spark.sql.execution.SparkPlan.execute(SparkPlan.scala:192)
	at org.apache.spark.sql.execution.QueryExecution.toRdd$lzycompute(QueryExecution.scala:163)
	at org.apache.spark.sql.execution.QueryExecution.toRdd(QueryExecution.scala:162)
	at org.apache.spark.sql.DataFrameWriter.$anonfun$runCommand$1(DataFrameWriter.scala:1079)
	at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withCustomExecutionEnv$5(SQLExecution.scala:126)
	at org.apache.spark.sql.execution.SQLExecution$.withSQLConfPropagated(SQLExecution.scala:267)
	at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withCustomExecutionEnv$1(SQLExecution.scala:104)
	at org.apache.spark.sql.SparkSession.withActive(SparkSession.scala:843)
	at org.apache.spark.sql.execution.SQLExecution$.withCustomExecutionEnv(SQLExecution.scala:77)
	at org.apache.spark.sql.execution.SQLExecution$.withNewExecutionId(SQLExecution.scala:217)
	at org.apache.spark.sql.DataFrameWriter.runCommand(DataFrameWriter.scala:1079)
	at org.apache.spark.sql.DataFrameWriter.saveToV1Source(DataFrameWriter.scala:468)
	at org.apache.spark.sql.DataFrameWriter.saveInternal(DataFrameWriter.scala:438)
	at org.apache.spark.sql.DataFrameWriter.save(DataFrameWriter.scala:311)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at py4j.reflection.MethodInvoker.invoke(MethodInvoker.java:244)
	at py4j.reflection.ReflectionEngine.invoke(ReflectionEngine.java:380)
	at py4j.Gateway.invoke(Gateway.java:295)
	at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)
	at py4j.commands.CallCommand.execute(CallCommand.java:79)
	at py4j.GatewayConnection.run(GatewayConnection.java:251)
	at java.lang.Thread.run(Thread.java:748)

supported-databricks-runtime-releases-and-support-schedule

Would be highly appreciated!

Any progress on this?

Yes please! As time goes on, this compatibility becomes more important.

This change is must required, As we are not able to use this lib on DataBricks 8.X runtime version.

waiting for the new release of this connector to work with the new DataBricks runtime version at least  8.4 onwards.

Hi all,
Is there a way to make a build locally based on PR-134 that would be able to run on Databricks runtime 8 versions ?
A bit the same as could be done before when spark 3.0 versions were not supported yet  and is described here .

+1, even an alpha release is very helpful, at least unblock us from using it in spark 3.1.2

@shivsood Thanks a lot for this new version! Has it been published to Maven already?

Thanks for the fix.
I couldn't the new version in Maven as of now, can you advise the timelines on the new release as we have hold the Databricks runtime version upgrade to 8.x ?