svlangserver run error in coc.nvim + gvim + win11 envirement

When use  coc.nvim + gvim in win11 system envirement, svlangserver run error, after using "npm install -g @imc-trading/svlangserver" install svlangserver, just run svlangserver in cmd, error is:
              throw new Error('Message header must separate key and value using :');
                ^
Error: Message header must separate key and value using :
    at MessageBuffer.tryReadHeaders (D:\home\windows\bin\node_modules\@imc-trading\svlangserver\node_modules\vscode-languageserver\node_modules\vscode-jsonrpc\lib\common\messageBuffer.js:87:23)
    at StreamMessageReader.onData (D:\home\windows\bin\node_modules\@imc-trading\svlangserver\node_modules\vscode-languageserver\node_modules\vscode-jsonrpc\lib\common\messageReader.js:132:45)
    at ReadStream.<anonymous> (D:\home\windows\bin\node_modules\@imc-trading\svlangserver\node_modules\vscode-languageserver\node_modules\vscode-jsonrpc\lib\common\messageReader.js:122:18)
    at ReadStream.emit (node:events:390:28)
    at addChunk (node:internal/streams/readable:315:12)
    at readableAddChunk (node:internal/streams/readable:289:9)
    at ReadStream.Readable.push (node:internal/streams/readable:228:10)
    at TTY.onStreamRead (node:internal/stream_base_commons:199:23)

open gvim.exe and using ":CocCommand workspace.showOutput" to see log, show is:
[Error  - 下午11:06:37] Starting client failed
Launching server "languageserver.svlangserver" using command svlangserver failed.
[Error  - 下午11:06:37] spawn svlangserver ENOENT

suggest fix it.

Are any other language servers working? Because from the error you have posted, it looks like the fundamental vscode json rpc is broken. That part is not specific to svlangserver. Please check if your system supports vscode-jsonrpc-6.0.0

In my win11 envirement, the coc.nvim with clangd language server is working fine. In addition, my some ohter coc.nvim's extention also no product error so far.

Can you please post the versions of coc.nvim and vscode? My guess is that those don't support version 6.0.0 for the vscode-jsonrpc

here is CocInfo print:
## versions

vim version: VIM - Vi IMproved 8.2 8023000
node version: v16.13.2
coc.nvim version: 0.0.80-b7cd84e 2022-03-17 17:24:36 +0800
coc.nvim directory: /home/zhougq/.home/.vim/plug/coc.nvim
term: tmux
platform: linux

## Log of coc.nvim

when useing svlangserver, CocInfo print is:
## versions

vim version: VIM - Vi IMproved 8.2 8023000
node version: v16.13.2
coc.nvim version: 0.0.80-b7cd84e 2022-03-17 17:24:36 +0800
coc.nvim directory: D:\home\.vim\plug\coc.nvim
term: undefined
platform: win32

## Log of coc.nvim

2022-05-22T23:41:41.209 INFO (pid:4580) [services] - registered service "languageserver.clangd"
2022-05-22T23:41:41.211 INFO (pid:4580) [services] - registered service "languageserver.svlangserver"
2022-05-22T23:41:41.214 INFO (pid:4580) [services] - svlangserver state change: stopped => starting
2022-05-22T23:41:41.257 INFO (pid:4580) [services] - registered service "highlight"
2022-05-22T23:41:41.312 INFO (pid:4580) [plugin] - coc.nvim initialized with node: v16.13.2 after 202ms
2022-05-22T23:41:41.317 INFO (pid:4580) [services] - svlangserver state change: starting => stopped
2022-05-22T23:41:41.330 ERROR (pid:4580) [services] - Server languageserver.svlangserver failed to start: Launching server "languageserver.svlangserver" using command svlangserver failed.
2022-05-22T23:41:41.330 ERROR (pid:4580) [language-client-index] - Error: spawn svlangserver ENOENT
    at Process.ChildProcess._handle.onexit (node:internal/child_process:282:19)
    at onErrorNT (node:internal/child_process:477:16)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -4058,
  code: 'ENOENT',
  syscall: 'spawn svlangserver',
  path: 'svlangserver',
  spawnargs: []
}
2022-05-22T23:41:48.403 INFO (pid:4580) [attach] - receive notification: showInfo []

I use npm install -g @imc-trading/svlangserver to install the svlangserver in windows, then try open a verilog file， then the CodInfo is:
## versions

vim version: VIM - Vi IMproved 8.2 8023000
node version: v16.13.2
coc.nvim version: 0.0.80-b7cd84e 2022-03-17 17:24:36 +0800
coc.nvim directory: D:\home\.vim\plug\coc.nvim
term: undefined
platform: win32

## Log of coc.nvim

2022-05-22T23:50:15.952 INFO (pid:11984) [services] - registered service "languageserver.clangd"
2022-05-22T23:50:15.954 INFO (pid:11984) [services] - registered service "languageserver.svlangserver"
2022-05-22T23:50:15.956 INFO (pid:11984) [services] - svlangserver state change: stopped => starting
2022-05-22T23:50:16.000 INFO (pid:11984) [services] - registered service "highlight"
2022-05-22T23:50:16.051 INFO (pid:11984) [plugin] - coc.nvim initialized with node: v16.13.2 after 193ms
2022-05-22T23:50:16.055 INFO (pid:11984) [services] - svlangserver state change: starting => stopped
2022-05-22T23:50:16.066 ERROR (pid:11984) [services] - Server languageserver.svlangserver failed to start: Launching server "languageserver.svlangserver" using command svlangserver failed.
2022-05-22T23:50:16.067 ERROR (pid:11984) [language-client-index] - Error: spawn svlangserver ENOENT
    at Process.ChildProcess._handle.onexit (node:internal/child_process:282:19)
    at onErrorNT (node:internal/child_process:477:16)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -4058,
  code: 'ENOENT',
  syscall: 'spawn svlangserver',
  path: 'svlangserver',
  spawnargs: []
}
2022-05-22T23:50:19.529 INFO (pid:11984) [attach] - receive notification: showInfo []

Can you please try building from source. I think the below steps should be done

Clone the repo
cd svlangserver && npm install
Update your .vim/coc-settings.json to point to /svlangserver/bin/main.js

If this works, then kindly post the contents of package-lock.json

I try it:

git clone https://github.com/imc-trading/svlangserver.git (clone to the D:/home/windows/coc/svlangserver dir)
cd to repo dir and npm install, it build sucess
set coc-setings.json link this:

        "svlangserver": {
            "command": "D:/home/windows/coc/svlangserver/bin/main.js",
            "filetypes": ["systemverilog", "verilog"],
            "settings": {
                "systemverilog.includeIndexing": ["**/*.{v,sv,svh}"],
                "systemverilog.excludeIndexing": ["test/**/*.sv*"],
                "systemverilog.defines" : [],
                "systemverilog.launchConfiguration": "verilator -sv -Wall --lint-only",
                "systemverilog.formatCommand": "verible-verilog-format"
            }
        }
    }

unfortunately, when open .v file:
CocInfo is:
## versions

vim version: VIM - Vi IMproved 8.2 8023000
node version: v16.13.2
coc.nvim version: 0.0.80-b7cd84e 2022-03-17 17:24:36 +0800
coc.nvim directory: D:\home\.vim\plug\coc.nvim
term: undefined
platform: win32

## Log of coc.nvim

2022-05-23T18:08:01.987 INFO (pid:9196) [services] - registered service "languageserver.clangd"
2022-05-23T18:08:01.988 INFO (pid:9196) [services] - registered service "languageserver.svlangserver"
2022-05-23T18:08:01.991 INFO (pid:9196) [services] - svlangserver state change: stopped => starting
2022-05-23T18:08:02.035 INFO (pid:9196) [services] - registered service "highlight"
2022-05-23T18:08:02.084 INFO (pid:9196) [plugin] - coc.nvim initialized with node: v16.13.2 after 199ms
2022-05-23T18:08:02.087 INFO (pid:9196) [services] - svlangserver state change: starting => stopped
2022-05-23T18:08:02.099 ERROR (pid:9196) [services] - Server languageserver.svlangserver failed to start: Error: spawn UNKNOWN
    at ChildProcess.spawn (node:internal/child_process:412:11)
    at Object.spawn (node:child_process:698:9)
    at D:\home\.vim\plug\coc.nvim\build\index.js:54060:63 {
  errno: -4094,
  code: 'UNKNOWN',
  syscall: 'spawn'
}
2022-05-23T18:08:06.050 INFO (pid:9196) [attach] - receive notification: showInfo []

the path is ok when use forward slash, if path no found, the CocInfo print other error message. by the way , backslashes is also test, print the same error as "forward slash".

Since the errors in the two cases are different, there is definitely something different between those two setups. Hence suspecting the path (based on the error message it is probably a good guess).
Unfortunately I don't have access to win11 system and hence can't test this myself.

I was able to reproduce the error on a win10 machine. I was able to get past it by using the below coc-settings.json
{
    "languageserver": {
        "svlangserver": {
            "module"<EMAIL_ADDRESS>            "args": ["--node-ipc"],
            "filetypes": ["systemverilog"],
            "settings": {
                ...
            }
        }
    }
}

the GLOBAL_NODE_MDULES_PATH should point to folder where node modules get globally installed.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 603 ms

Resolved: ClamAV performance has improved in d13c52f.


Onboard azure-spring-data-cosmos

Docs Build status updates of commit e9fe10e:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://SiteHelp and include the error report for our team to troubleshoot

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

The OPS build succeeded in a subsequent rebuild but that status doesn't appear to be reported here


disable caps lock delay broken after upgrade to 10.14.6

The complex modifications rule: disable caps lock delay (rev 1) is not functional after I upgrade OSX to 10.14.6.
I imported the rule from here
karabiner-elements version: 12.5.0
osx version: 10.14.6
logs:

[2019-08-10 12:47:54.878] [info] [console_user_server] version 12.5.0
[2019-08-10 12:47:54.947] [info] [console_user_server] karabiner_grabber_alerts.json is updated.
[2019-08-10 12:47:55.612] [info] [console_user_server] receiver is initialized
[2019-08-10 12:47:55.613] [info] [console_user_server] grabber_client is started.
[2019-08-10 12:47:55.613] [info] [console_user_server] grabber_client is connected.
[2019-08-10 12:47:55.695] [info] [grabber] karabiner_console_user_server is connected.
[2019-08-10 12:47:55.695] [info] [console_user_server] Load /Users//.config/karabiner/karabiner.json...
[2019-08-10 12:47:55.696] [info] [grabber] console_user_server_client is connected. (uid:501)
[2019-08-10 12:47:55.696] [info] [grabber] event_tap_monitor terminated
[2019-08-10 12:47:55.696] [info] [console_user_server] core_configuration is updated.
[2019-08-10 12:47:55.698] [info] [grabber] device_grabber is stopped.
[2019-08-10 12:47:55.698] [info] [grabber] device_grabber is started.
[2019-08-10 12:47:55.721] [info] [grabber] event_tap_monitor initialized
[2019-08-10 12:47:55.722] [info] [grabber] virtual_hid_device_client is started.
[2019-08-10 12:47:55.722] [info] [grabber] Load /Users//.config/karabiner/karabiner.json...
[2019-08-10 12:47:55.722] [info] [grabber] core_configuration is updated.
[2019-08-10 12:47:55.724] [info] [grabber] mouse_key_xy_scale: 100
[2019-08-10 12:47:55.771] [info] [console_user_server] Check for updates...
[2019-08-10 12:47:55.905] [info] [grabber] `system_preferences` is updated.
[2019-08-10 12:47:56.726] [warning] [grabber] virtual_hid_device_client is not connected yet. Please wait for a while.
[2019-08-10 12:47:56.733] [info] [grabber] Apple Internal Keyboard / Trackpad (device_id:4294968340) is ignored.

Similarly, Caps Lock to Escape on single press, Control on press and hold. from Modern Space Cadet (rev 3) no longer functions after update to 10.14.6.

https://github.com/tekezo/Karabiner-Elements/issues/1930 resolved my issue.


Beginner to nim/mummy - concurrency question

If I want to build an endpoint that aggregates data from 3 different data sources (lets say, HTTP APIs) that do not depend on each other - would the concurrency model used with mummy allow me to make these requests at the same time, or would I need to perform them sequentially?
In a language like nodejs or python, I would use the equivalent of Promise.all() or asyncio.gather(), but both of these are async/await type calls in their respective languages.

Right now I suggest just doing them sequentially. This is not because it is impossible to make them run concurrently, it is simply because I have not built any libs / tools to enable that yet.
The most probable solution I see right now is to add a proc to https://github.com/guzba/curly that would make use of libcurl multi requests. Then many requests could be done at once without any additional thread usage.
This is not without some implementation things to sort out -- if I use libcurl handles from the pool, I'd need to take the number needed (or some max) and then not be able to return them until all requests finished, which could cause issues if any of the requests in the multi request take a long time to complete. I'll need to think about how I want to have this work at some point.
If these are fast requests and there are only a couple, just doing them sequentially and moving on is by far the best idea.

Thanks @guzba (for the response here, and for what looks like a lot of nim packages you've provided the world)!
The example was a bit contrived - it could be 3, 10, 20, or even 100 aggregated API calls - the difference between sequential and concurrent is vast.
It sounds like there is no real way to accomplish this at the moment (at least, following mummy idioms?  as it'd seem odd to try to pull async stuff on an explicitly non-async web server?) - but I look forward to the solution you proposed!

If you are doing that many separate HTTP RPC calls in an API endpoint handler I suggest that is a bit unusual and that is unfortunately not a use-case that aligns with what Mummy is really focused on.
I know how to improve things for more HTTP requests, however nothing will prevent such an endpoint from being trivially vulnerable to abuse which would be a concern for me.
Closing this since there's no action to take in Mummy specifically and creating an issue on Curly.

The actual use case (which I was hoping to POC to my work, as an alternative to python+FastAPI) would be in a heavy microservice architecture, where a middleware/backend-for-frontend serves data tailored to a page or endpoint, which happens to be comprised of data from many different microservices.
Rate throttling and abuse potential is really a concern unrelated to a web server/framework I think - as those things tend to be locked down via other mechanisms (authentication/authorization/WAF level rules).
Thanks for making a follow up on your HTTP client lib!

Ah yeah if you're going heavy microservices then Mummy is probably not a good fit. It's not an approach to server design I use so I haven't put in the work to facilitate it better. Other options out there though so hopefully you can find something that feels cozy.


Construct Binary Tree from Preorder and Inorder Traversal

https://leetcode.com/problems/construct-binary-tree-from-preorder-and-inorder-traversal/?envType=study-plan-v2&envId=top-interview-150

https://leetcode.com/problems/managers-with-at-least-5-direct-reports/?envType=study-plan-v2&envId=top-sql-50


Missing links

For more information about available data sources, see Data Sources. For information about connecting to data, including examples and steps, see Connect to Data.
The bolded sections above, Data Sources and Connect to Data should be links.

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

ID: bd6ac628-d5b4-e722-0532-d467c8347a5a
Version Independent ID: 20ec07f6-627d-3765-ddcb-4aa738b6952a
Content: Query overview in Power BI Desktop - Power BI
Content Source: powerbi-docs/transform-model/desktop-query-overview.md
Service: powerbi
Sub-service: pbi-transform-model
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @j2associates . This issue has been assigned to the doc author for review.

Hi @j2associates ,
Thank you for your suggestions.  I've added links to the article.


How can  get data key and call id

I tried to make a payment via authorize.net payment (using visa card)
It's error E00027: The transaction was unsuccessfully
I used production mode. (development mode worked normally).
I have already compared to sample code:
https://github.com/AuthorizeNet/sample-code-php/blob/643363a7c1edad4e8029c3033902db4b0a6c07c2/VisaCheckout/create-visa-checkout-transaction.php#L37
There are 2 values which I don't know how to get them.

Data key
$op = new AnetAPI\OpaqueDataType();
$op->setDataKey(HOW_TO_GET_THIS VALUE);
Call id
$transactionRequestType = new AnetAPI\TransactionRequestType();
$transactionRequestType->setCallId(HOW_TO_GET_THIS VALUE);
Anybody can help me?

@thekingbk90, closing this thread based on the procedure provided  by @ashtru and that should resolve the reported issue.
Please feel free to re-open this, if you are still facing the issue.


Added twemojiOptions to EmojiButton(options)

This PR adds the missing twemojiOptions to src/emoji.ts (see https://github.com/joeattardi/emoji-button/issues/111) and moves them to EmojiButton constructor's options argument. This way the user can use twemoji's options like 'base' to specify where to download the images.

Thank you for the contribution. I apologize it took so long to get around to merging it!


Soft Crash on Server Join?

SP works fine, and the dedicated server works fine, however when a client tries to join a server they get immediately disconnected with the following in the client log files (the server shows nothing abnormal): https://pastebin.com/H12NWgqk

Try v1.1.1.

When will v1.1.1 be on Curse?

I will upload it as soon as you've confirmed that it solves the issue.

I will upload it to Curse when you confirm that it solves the issue.

Sorry I missed this notification. It solves the issue.


Set UA string for Openverse API requests from the frontend

Fixes

Description

Forgot I wrote this earlier in June. Adds a UA string identifying the frontend to the API in all requests. Useful for monitoring API traffic from the frontend over time.
Testing Instructions

Run locally, make a client side search and check for the user agent set in the request in devtools.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[x] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@AetherUnbound IIRC Chrome does not allow user agent to be changed. Firefox also might not allow it. In that case, this change would only be evidenced for server side requests, not browser based ones. For Openverse.org requests from the frontend, the referrer header is a reliable identifier of source rather than the UA string.

Oh I see! Does that mean that Referrer here should be the value that was changed? Or is it as-expected?

@AetherUnbound Referrer is set by the browser for client-side requests, it's the url of whatever the site is, in this case it'll be https://openverse.org in production.

@AetherUnbound IIRC Chrome does not allow user agent to be changed. Firefox also might not allow it. In that case, this change would only be evidenced for server side requests, not browser based ones. For Openverse.org requests from the frontend, the referrer header is a reliable identifier of source rather than the UA string.

Is there a way to suppress the warnings? On staging, we get instrument.js:289 Refused to set unsafe header "User-Agent" on every client request now.

Ah, we should probably only set the UA server-side then. I'll make an issue today.


I want to deposit money from another wallet to the nami wallet.

I want to deposit money from another wallet to the nami wallet.
Tokens other than ADA
How can I check the address to which Hosky and Hosky NFT will be deposited?

There is on receiving address in Nami, which takes/accepts ADA and any other token. There is no seperate address for specific tokens.


#366 Fix node-to-float conversion error if node's value is <= 0

This PR has fixed the error in converting a node's value into a native floating point value, as reported in the issue #366.
The root cause was that calling the std::min() function to get the minimum possible value of the target type: the function returns minimum possible positive normal value of a floating point value type, not the minimum possible negative value; the appropriate function to call is the  std::lowest() for that purpose.
In short, the real cause was the library coudn't convert a node value whose value is a floating point value equal to/less than 0.
So, this PR has fixed the issue by replacing the calling of std::min() with std::lowest() on node-to-float conversions.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
3971

Relevant Lines:
3971

💛  - Coveralls


DNS changes on network interface can lead to network failure

Describe the bug
During the installation the primary DNS entry of the physical network adapter is changed to <IP_ADDRESS>. The originally DNS servers are moved to the local DNS server (dnsmasq) on the KubeMaster VM. In some cases it happens sporadically that the pyhsical network adapter is not observered as "up" during cluster stop. Therefore the DNS server can't be resetted and the system looses network connectivity. DNS resolution seems instable sometimes as well.
To Reproduce
Steps to reproduce the behavior:

Install the cluster
Stop and start the cluster several times

Expected behavior
DNS resolution is stable and the DNS servers on the physical network adapter are always resetted during stop and uninstallation
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Version 21H2

Additional context
https://github.com/Siemens-Healthineers/K2s/blob/2b8a1c2d269b13a46d499f429d39438127df07b9/lib/modules/k2s/k2s.infra.module/network/windows-host-network.module.psm1#L61

@achimkist  Thanks for reporting this, probably relying on 'Up' status is not working out here. Storing the physical interface information might help, Is the interface always in 'down' state or briefly during start/stop, because a retry might help as well.


feature: add load testing to github actions

Adds load testing via locust to github actions workflow.
It run a 10 min load test with 1000 locusts (users) which have a spawn rate of 100 and wait between 5 and 15 seconds for the next request.
Test requests run against the test app's root view which returns a Http 200 with a string "Hello world!".

Thank you for your contribution.
I'd suggest to add two "heavy views", too. Maybe one which blocks ~100ms and another with ~500ms. That way we can add mixed workload cases and further equal workload cases (as of which you already got one). Our benchmarking will be closer to classical blocking Django applications and async Django applications.
On a second thought I hope we don't have too much of a resource limitation with these GitHub action runners.

I'd suggest to add two "heavy views", too. Maybe one which blocks ~100ms and another with ~500ms. That way we can add mixed workload cases and further equal workload cases (as of which you already got one). Our benchmarking will be closer to classical blocking Django applications and async Django applications.

Makes sense. I guess that could be done in a jiffy.

On a second thought I hope we don't have too much of a resource limitation with these GitHub action runners.

I researched a bit and did not find any answer concerning resource limits. However - the runner seems to have better performance than the laptop I'm working with (2.6 GHz - 6 cores i7).


features: align y-axis max & min value of the chart to compare

In analyze page, such as https://ossinsight.io/analyze/taosdata/TDengine?vs=influxdata%2Finfluxdb

Each of charts to compare are using its own Y axis which means the repos are not be compared in the same scale. It is not fair to the bigger one.
Suggestion: use the same max and min value of Y axis in one comparison chart group.

This make sense.


Connecting webservice through Data bricks

Can someone please let me know if you are able to connect webservice using Data Bricks.

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

ID: b59e754a-3a7b-0909-fc3f-c8ec75f21d07
Version Independent ID: f9a711cf-3c5a-5115-a98f-94e578029774
Content: Connect to different data sources from Azure Databricks
Content Source: articles/azure-databricks/databricks-connect-to-data-sources.md
Service: azure-databricks
GitHub Login: @nitinme
Microsoft Alias: nitinme

@balu04vn Thanks for the feedback. We are actively investigating and will get back to you soon.


Check for #224 regression

I don't think that https://github.com/microsoft/azure-pipelines-vscode/issues/224 is resolved. This should prove it.

Looks like tests are passing, @jamesmyatt. Can you explain more why you think there's been a regression?

Using this schema with check-jsonschema fails to validate in cases where the boolean is stored as a literal boolean rather than a string recognised as boolean. For example: https://github.com/python-jsonschema/check-jsonschema/pull/237
Looking at the Azure DevOps documents, they recommend using boolean true, rather than string "true": e.g. https://learn.microsoft.com/en-us/azure/devops/pipelines/yaml-schema/trigger?view=azure-pipelines#trigger-batch-branches-paths-tags
However the schema doesn't allow this (See https://github.com/microsoft/azure-pipelines-vscode/blob/main/service-schema.json#L4008) which is the same issue issue as #224 , which was closed as fixed.
I thought it would be worth updating the test cases to include something that validates against #/definitions/boolean, and I naively made a change that I thought would show this.
Fundamentally, either the schema is wrong or the documentation is wrong, since the schema says no literal booleans and the documentation says they're OK. Furthermore, the validator should fail for this change, which means that either the file isn't checked in the CI tests or the validator used here is non-compliant with the JSON Schema specification (as I understand it: I may also not understand the specification correctly).
What's the best way to proceed with this.


Host.Get with include Items return 500 error

If i comment include.Add(HostInclude.Items); then data loading succefull
 var include = new List<HostInclude>();
            include.Add(HostInclude.Items);
            include.Add(HostInclude.Interfaces);
            include.Add(HostInclude.Inventory);
            // Get Zabbix Data
            IEnumerable<Host> resp = null;
            using (var context = new Context())
            {
                var service = new HostService(context);
                resp = service.Get(new { groupids = grpIds }, include);
            }

UPD
Problem was in memory limiting...

Try change some zabbix configuration. On zabbix.conf, set:
php_value memory_limit 4096M
php_value post_max_size 128M


skywalking UI no data

Why this console has no any data?

Why do you query so large page size? This size could trigger elasticsearch error, if you want that, change elasticsearch settings.

change what?

I has changed the time but it does not work


Macros retained in memory

Lua is retaining macros modules in memory, so edits to macro files are not reflected in auto-builds without restarting neovim.
Fennel retains a package.loaded style list of macro modules it has seen, which we need to clear, exists as macro-loaded in specials.fnl, but appears to be exposed on the compiler env at some points.

    (each [k _ (pairs fennel.macro-loaded)]
      (tset fennel.macro-loaded k nil))

in compiler.compile-string fixes issue, but means every compile call will fetch and compile macro modules.

Will give slower compile performance:

make (manually) -- probably not a huge deal, the act of typing is probably slower than the build,
auto-make (on save) -- noticeable,
diagnostics (during editing) -- will extend delay when leaving text buffer ,
require (when compile needed).

Will give better UX when:

Editing files that have had macros change (eg: adding x to m1 wont show "m1 does not have x function" when importing it without restart.
auto-make (actually functional), this is probably the bigger issue as it can "say" building was all ok but the macros have not changed, so the code has not changed.

In some respects I think the performance isn't a huge deal. Compiling is already "slow" compared to anything else, so it being a bit slower (probably not to a noticeable degree unless the given files use lots of complex macros) might be a reasonable trade off for the better diagnostics and clearer-functioning auto-make.
Potentially the compile could be run in its own thread, which gets its own state. Either a fresh thread could be built per-compile-entry (probably not "per compile call"), which would incur some start up time and "config" time where hotpot is loaded again, etc (this is ~7-8ms), or a persistent thread could exist that compiles when needed, and nukes the macros-loaded table each time but the performance hit wont be as "felt".
The off-thread compile has been considered before when working on ruin.fnl which has very complex macros, and is particularly slow when editing large test files that generate 10x lua code.
Next:

See viability of only nuking when make is running. Passing an extra flag to the compiler? The check requires fennel in memory, but that could be checked "in make"? If fennel exists, nuke the table, otherwise there is no need, fennel will load in the compiler fresh anyway. Least impact on performance, adds another "quirk" to that branch.
Otherwise include the above each loop.

Then,

See viability of off-thread compilation.

adds dragons, might still be worth it.


Make transportModeList non nullable in JourneyCard

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#248
#247  👈
#246
#245
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ksharma-xyz and the rest of your teammates on  Graphite


Add option to pad the original template shape

https://ptb.discord.com/channels/915186263609454632/1139173145639653396/1140407828281045133

Added in v0.6.5.


HAWQ-1593. Vectorized execution condition check in plan tree

Replace the vectorized data types and expressions immediately after check.
2.Move the VExecInitNode to the end of the ExecInitNode.
3.Process the Parent PlanState and the vectorized flag in the VExecInitNode.

LGTM.


OSCI: handle 524 from definitions.io

Description
A 524 error from definitions.stackrox.io has shown up in at least 1 instance of the build: https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/stackrox_stackrox/1560/pull-ci-stackrox-stackrox-master-images/1530295040247599104
This PR tries to mitigate that.
Checklist

[x] Investigated and inspected CI test results

Testing Performed
CI is sufficient

/rebase

/rebase

You can /pony but not /rebase 😱

/test all


bugfix/307: Correct left spacing of hero on L2 pages

Fix #307
Test URLs:
L2

Before: https://main--hlx-test--urfuwo.hlx.page/draft/skhare/content-list-test
After: https://307-l2-page-hero-left-spacing--hlx-test--urfuwo.hlx.page/draft/skhare/content-list-test

L2, media blend

Before: https://main--hlx-test--urfuwo.hlx.live/author/juliawhite
After: https://307-l2-page-hero-left-spacing--hlx-test--urfuwo.hlx.live/author/juliawhite

L1 (Unchanged)

Before: https://main--hlx-test--urfuwo.hlx.live/draft/skhare/l1-sample-page
After: https://307-l2-page-hero-left-spacing--hlx-test--urfuwo.hlx.live/draft/skhare/l1-sample-page

L3 (Unchanged)

Before: https://main--hlx-test--urfuwo.hlx.page/news/2024/01/accelerate-innovation-with-clean-core-strategy
After: https://307-l2-page-hero-left-spacing--hlx-test--urfuwo.hlx.page/news/2024/01/accelerate-innovation-with-clean-core-strategy

@rjwtrmn: In Chrome on Mac for viewport width 505px / XS (the narrowest my computer allows me to shrink to), on the following two pages the hero text block is wider (has more padding left/right) than the elements following the hero:

https://307-l2-page-hero-left-spacing--hlx-test--urfuwo.hlx.page/draft/skhare/content-list-test
https://307-l2-page-hero-left-spacing--hlx-test--urfuwo.hlx.live/author/juliawhite

All others look fine, also the L3 page with an in-page nav: https://307-l2-page-hero-left-spacing--hlx-test--urfuwo.hlx.page/news/0000/the-article

Thank you, I have pushed a change to resolve the issue with the left padding. As regards the right side, I didn't make changes to the margin values there so it should match what is on main, but happy to change it here if main deviates from the designs (no Figma on my side as of yet though)


extra cli options

Added logical shortcuts for parameters. (NOTE: docker runner changed from -d to -r)
Added --docker, --podman, and --kubernetes shortcut
Allow the workflows to be specified as standard arguments

Before
v0.6.1

Options:
  -w, --workflow=<s>                  Path to your workflow json
  -i, --input=<s>                     JSON payload to input to the workflow (default: {})
  -c, --credentials=<s>               JSON payload with credentials (default: {})
  -d, --docker-runner=<s>             Type of runner for docker images (default: docker)
  -o, --docker-runner-options=<s+>    Options to pass to the runner
  -v, --version                       Print version and exit
  -h, --help                          Show this message

After
Usage: floe [options] workflow1 [input1 [workflow2 input2] ...]

v0.6.1

Options:
  -w, --workflow=<s>                  Path to your workflow json
  -i, --input=<s>                     JSON payload to input to the workflow (default: {})
  -c, --credentials=<s>               JSON payload with credentials (default: {})
  -r, --docker-runner=<s>             Type of runner for docker images
  -o, --docker-runner-options=<s+>    Options to pass to the runner
  -d, --docker                        Use docker to run images (short for --docker_runner=docker)
  -p, --podman                        Use podman to run images (short for --docker_runner=podman)
  -k, --kubernetes                    Use kubernetes to run images (short for --docker_runner=kubernetes)
  -v, --version                       Print version and exit
  -h, --help                          Show this message

update:

added capitals
added phrase shortcut

Todo:

move input, creds to argv
don't use input, credentials from args

Prep for multiple workflows

update:

removed argv changes (pulling into a separate PR)

update:

moved factory into a hash. used that hash for shortcuts, default, and factory
changed shortcut verbiage from #149

ok, I used the TYPES value to drive the shortcuts. is this too cryptic?

update:

lost usage in rebases, added back
updated git commit comment to reflect new --help output

ugh. sorry, got confused, usage gets changed in the other pr. (that is why it was missing...)
update:

removed usage string

ok, code was too clever. I removed hash_table.each and just hardcoded the values.

Updates:

removed TYPES from runner.
DEFAULT is a separate variable
Optimist.die is moved and simplified

I like it how it is.
But there are a few possible changes:

If we want to keep Runner::TYPES, then we can introduce set_runner(name="docker", options={}) and move the factory into there.
If we want a default for docker-runner, then we will allow short flags to override longer flags.
I keep wanting to change the code to have the options generated from TYPES, but find the code more confusing and not any shorter. So I keep tearing it back out.

update:

extracted factory / constant definition into #152
only introduce aliases.
still changes the short form of credentials file, docker runner and docker runner options (to -e, -r and -o)

Outstanding

I did like the docker_runner, :default => "docker", but we can't keep that and have long form override short form. So I dropped the default. Can have last one wins for short form (instead of first one wins)


[Windows] Update Visual Studio 2019 signature

Signature was updated.
Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2019

Signature was updated.
Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

bd838ae83bb6603d7b818ea22e01db55b29e6568

Signature was updated.
Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

BR&/;-666"999:;_-+777¢`~π∆§×✓™®<***


Updating Bureau structure and image

Updating bureau structure chart and image
Changes

Updating the bureau structure chart and image

Testing

Visit http://localhost:8000/about-us/the-bureau/bureau-structure/1 and observe the following:

LITIGATION & OVERSIGHT office should be vacant.
Last updated date should be May 3, 2016.
Bureau-structure-chart.png should be updated.

Review

@schaferjh
@jimmynotjim

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

thanks! 👍

👍


gRPC call hangs after receiving server response but before returning response to caller

Bug Report
Version
│   └── tonic v0.4.0
│   └── tonic-build v0.4.0
└── tonic v0.4.0 (*)

Platform
Running inside an alpine:3.12 Docker container
Description
I have a server exposing a Protobuf API method called RegisterTestSetupCompletion.
I have a consumer, running inside a Docker container, that will call this method. I've generated a client for the Protobuf API using tonic-build like so:
    tonic_build::configure()
        .out_dir(&output_dirpath)
        .compile(&input_proto_filepaths, &[input_dirpath])
        .context("An error occurred generating the .proto files")?;

This results in a Rust function inside a TestExecutionServiceClient struct:
        #[doc = " Registers that the testsuite has completed test setup"]
        pub async fn register_test_setup_completion(
            &mut self,
            request: impl tonic::IntoRequest<()>,
        ) -> Result<tonic::Response<()>, tonic::Status> {
            self.inner.ready().await.map_err(|e| {
                tonic::Status::new(
                    tonic::Code::Unknown,
                    format!("Service was not ready: {}", e.into()),
                )
            })?;
            let codec = tonic::codec::ProstCodec::default();
            let path = http::uri::PathAndQuery::from_static(
                "/api_container_api.TestExecutionService/RegisterTestSetupCompletion",
            );
            self.inner.unary(request.into_request(), path, codec).await
        }

The consumer constructs a Channel for talking to the server and connects it:
		let endpoint = Channel::from_shared(url)
			.context(format!("An error occurred creating the endpoint to Kurtosis API socket '{}'", &self.kurtosis_api_socket))?;

		// Sometimes the API container is still in the process of starting, so we retry the channel connection a few times
		let channel: Channel;
		let mut connection_attempts: u32 = 0;
		loop {
			if connection_attempts >= MAX_CONNECTION_ATTEMPTS {
				return Err(
					anyhow!(
						"Failed to connect to API container, even after {} retries spaced {}ms apart",
						MAX_CONNECTION_ATTEMPTS,
						TIME_BETWEEN_CONNECTION_RETRIES.as_millis(),
					)
				);
			}
			let channel_or_err = block_on(endpoint.connect());
			match channel_or_err {
				Ok(returned_channel) => {
					channel = returned_channel;
					break;
				}
				Err(err) => {
					debug!(
						"The following error occurred connecting to the API container; retrying in {}ms: {}",
						TIME_BETWEEN_CONNECTION_RETRIES.as_millis(),
						err.to_string()
					);
				}
			}
			sleep(TIME_BETWEEN_CONNECTION_RETRIES);
			connection_attempts += 1;
		}

then creates an instance of the TestExecutionServiceClient struct to do the following:
        let mut registration_client = TestExecutionServiceClient::new(channel.clone());

        // A call to registration_client.some_other_function that succeeds
        // Some work here that takes ~40 seconds

        debug!("Registering test setup completion...");
		block_on(registration_client.register_test_setup_completion(()))
			.context("An error occurred registering the test setup completion with the API container")?;
        debug!("Test setup completion registered");
        info!("Test network set up");

The oddity is that the consumer's registration_client.register_test_setup_completion never returns - Registering test setup completion... is printed, but Test setup completion registered is not - it just hangs there. It's not a problem on the server side, because the corresponding server-side both receives the register_test_setup_completion call, and exits; it seems to be after the consumer's gRPC client receives the response from the server but before register_test_setup_completion returns back to the caller.
I grabbed all the logs from Registering test setup completion... onwards (
after-setup-completion-call.log
), but I've been staring at it for hours and can't make head nor tail of what's going on. The very last logline is:
22:48:27 [TRACE] (3) h2::proto::streams::streams: [/home/rust/.cargo/registry/src/github.com-1ecc6299db9ec823/h2-0.3.0/src/proto/streams/streams.rs:1280] drop_stream_ref; stream=Stream { id: StreamId(71), state: State { inner: Closed(EndStream) }, is_counted: false, ref_count: 2, next_pending_send: None, is_pending_send: false, send_flow: FlowControl { window_size: Window(65530), available: Window(0) }, requested_send_capacity: 0, buffered_send_data: 0, send_task: Some(Waker { data: 0x254c5c0, vtable: 0xe3f0d8 }), pending_send: Deque { indices: None }, next_pending_send_capacity: None, is_pending_send_capacity: false, send_capacity_inc: true, next_open: None, is_pending_open: false, is_pending_push: false, next_pending_accept: None, is_pending_accept: false, recv_flow: FlowControl { window_size: Window(2097147), available: Window(2097147) }, in_flight_recv_data: 5, next_window_update: None, is_pending_window_update: false, reset_at: None, next_reset_expire: None, pending_recv: Deque { indices: Some(Indices { head: 1, tail: 0 }) }, recv_task: None, pending_push_promises: Queue { indices: None, _p: PhantomData }, content_length: Omitted }
22:48:27 [TRACE] (3) h2::proto::streams::counts: [/home/rust/.cargo/registry/src/github.com-1ecc6299db9ec823/h2-0.3.0/src/proto/streams/counts.rs:136] transition_after; stream=StreamId(71); state=State { inner: Closed(EndStream) }; is_closed=true; pending_send_empty=true; buffered_send_data=0; num_recv=0; num_send=0

This almost seems like the Channel/tower instance inside the TestExecutionServiceClient is getting stuck somehow, maybe due to the 40s pause between usage?

I just added debugging to see exactly which line in the generated function it's getting stuck at, and it's this last one:
self.inner.unary(request.into_request(), path, codec).await

I just read up on HTTP2 and stepped through every line in the logfile above - it's still not clear to me why this is hanging, as the behaviour seems to be what I'd expect:

The connection (which had sat idle for the 40s) is successfully resumed
The consumer sends HEADERS and DATA frames on Stream 71
The stream transitions to the HalfClosed state, indicating that the consumer is only receiving data now
The server pushes several HEADER and DATA frames on Stream 71
The server pushes a HEADER frame with flags: (0x5: END_HEADERS | END_STREAM), indicating the the stream should be closed
The stream transitions to Closed state
The consumer client calls drop_stream_ref, presumably trying to free client memory
The transition_after line is printed that indicates that Stream 71 is closed:transition_after; stream=StreamId(71); state=State { inner: Closed(EndStream) }; is_closed=true; pending_send_empty=true; buffered_send_data=0; num_recv=0; num_send=0

Yet still, the self.inner.unary(request.into_request(), path, codec).await line doesn't complete

Nothing seems out of the ordinary, so it's not clear to me why

Sounds strange indeed 🤔 I'm wondering if the bug could be in h2 and not tonic.
Are you able to provide a reproduction example that can easily be run? Then I will take a look.

Thanks for getting back @davidpdrsn ! I have the consistent repro, but unfortunately it's pretty in-depth. I'll try to see if I can get a simpler repro (I'm guessing it has to do with the dead connection). I also built a local version of tonic with some debugging lines, and the exact error seems to be this: https://github.com/hyperium/tonic/blob/fdda5ae26ace4706cb8faf56ccd35513b5de26c5/tonic/src/transport/channel/mod.rs#L192 The await that sticks it is this: https://github.com/hyperium/tonic/blob/f58fcf31d1b725416b7f457e2d12e6aaa7e6f327/tonic/src/client/grpc.rs#L185

Hm this sounds similar to https://github.com/hyperium/tonic/issues/515. Could that be it? Not sure there is a fix for it though. I remember @LucioFranco and @seanmonstar talking about it in Discord some weeks ago but don't remember the conclusion.

That sounds very similar - I'm also using grpc-go on the server side. I'd be keen to hear if @LucioFranco or @seanmonstar found a workaround; nothing I've done has been able to unblock it so far.

Just tried to get a minimal repro by simulating that same 40s delay, but no luck - it's caused by something that's not just the delay.

Closing the loop for posterity - the ultimate issue was the same as https://github.com/seanmonstar/reqwest/issues/1215 : because I'm a Rust noob I was intermixing futures::executors::block_on with #[tokio::main] on my main function, which was causing deadlocks as described by @seanmonstar here: https://github.com/seanmonstar/reqwest/issues/1215#issuecomment-796927554 . I solved this by creating a Tokio runtime via Runtime::new() as close to the async calls as possible, and using runtime.block_on() to force synchronicity on async functions.


Modifications by GP 2 (Mail Hans De Keersmaecker)

RIZIV_nursingprescription_listServiceRequested20220203_feedback2.xlsx

For reference to the xls file

superseded by #65


Allow inheritance between types and interfaces

interface Common {
    foo: String!
    bar: Int!
}

type FooBar implements Common {
    ...Common
}

included in #32


Changelog leveraging  release-toolkit itself

It would be nice to have in this repo an action dealing with releases.
The main purpose would be to maintain the changelog and to provide an example regarding how to use the actions (similarly to what we did for the e2e action).

Done in #187
Closing


fix: treat symlink directory as directory

Fixes #531

@markjm This is causing pnpm symlinks to become inconsistent if the user moves a folder. Reverting this change fixes the problem. Can you investigate? https://github.com/vercel/turborepo/issues/549

@markjm This is causing pnpm symlinks to become inconsistent/not be recognized (especially when user moves a folder). Reverting this change fixes the problem. Can you investigate https://github.com/vercel/turborepo/issues/549?
Notice in screenshot how the apps are not recognized as result of globs?

Sure. In the meantime, it seems pnpm usage is more common than the frequency of the original bug I was trying to fix - so should probably just revert this for now


Remove unneeded old JSX pragma

The peer dependencies in package.json show "react": "^0.12" as a peer dependency, which means that the JSX pragma is no longer needed for valid consumer modules.
This also fixes some deprecation error messages with tools like babel-loader.

Sorry for the late merge! Thanks!


can't find start/end way

osrm sometimes can't find out where to start/stop the route, even though the marker is right on top of a way.
http://localhost:5000/viaroute&start=55.658580063072,12.568296197487&dest=55.658654204712,12.569143775534&z=18&output=json&jsonp=showResultsRoute&geomformat=cmp&instructions=true

Fixed


Transparent Tab Bar

Operation System: macOS 10.13 (17A405)
Beaker Version: Beaker v0.7.5

Duplicate of https://github.com/beakerbrowser/beaker/issues/693


AppRegistry page is a skeleton

docs/appregistry.md is very lacking. 😢
Is there information elsewhere about the various methods of AppRegistry?
Code examples of common use cases would be extremely helpful.

Hi @aleclarson! Are you happy with the latest version of AppRegistry page or is it something that we can improve?

Closing because of lack of activity/response.


Add flag to use Stanford neural dependency parser

Adds a flag (use_nndep=True) to nltk.parse.stanford.StanfordDependencyParser to use @danqi's neural network–based dependency parser (http://nlp.stanford.edu/software/nndep.shtml). Also makes DependencyGraph correctly read CONLL representations with non-consecutive token ids (needed for correct output on one of the existing Stanford dependency parser doctests).
Note the jar requirement and speed tradeoffs in the first commit.

Thanks @futurulus. Is there a reason to prefer a flag in the initializer for selecting the parser, as compared with deriving another class? I think it would be more in keeping with the other NLTK parsers to have a one-to-one correspondence between external parsers and NLTK parser classes. In this case, both parsers could inherit from a superclass. Then tagged_parse_sents could be specified as not implemented in the subclass.

No strong preference here. Splitting it into a separate class would also probably make it cleaner.
I'm assuming we'd want all three parsers to inherit from GenericStanfordParser. (As opposed to making a common superclass for the two dependency parsers, which in turn inherits from GenericStanfordParser—I can't think of any code that would usefully go in the dependency-only superclass.)

This sounds like a good way to go. Do you think you could update your PR?

Added new commits to split off the extra class and address @dimazest's comments. I imagine it will make sense to squash some of these together, but I'll leave the history intact until you think the PR can be wrapped up.

Thanks @futurulus and @dimazest

:+1:

@futurulus – our CI server is reporting some errors here, and I'd be grateful if you could take a look please.
https://nltk.ci.cloudbees.com/job/nltk/lastCompletedBuild/TOXENV=py33-jenkins,jdk=jdk8latestOnlineInstall/testReport/nltk.parse/stanford/StanfordNeuralDependencyParser/

Ah, after I moved the doctests into the new class, they're no longer correctly skipped if the right jar isn't present. (The neural dependency class needs stanford-corenlp-x.x.x.jar and stanford-corenlp-x.x.x-models.jar from http://nlp.stanford.edu/software/corenlp.shtml , which are not the same as the parser jars.)
We could either:

Skip all stanford.py doctests if either jar is missing.
Move the neural parser doctests back to test/ and give them a fixture file.
Some other way of conditionally skipping doctests that I don't know about?

Recommendations welcome.

@futurulus – thanks for looking into this. How about adding the extra model here: https://github.com/nltk/nltk/blob/develop/jenkins.sh#L16


Socks proxy imrpovements

Category
Description
The proxy is now set via the setProxy func in Utils.java. I added a config option for the proxy and added a unit test
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[ ] I've verified that this change works as intended.

[ ] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[ ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

Coverage decreased (-0.05%) to 32.492% when pulling af7a1e7f65805f49a1587eba7a27b84945cf9967 on cyian-1756:socks-proxy into a52bbeb6d8ba46e6f47199783665c2335e4a6c83 on RipMeApp:master.

I was doing the same thing while adding HTTP Proxy support, can you wait for my pull request before merging?

@randomcommitter

I was doing the same thing while adding HTTP Proxy support, can you wait for my pull request before merging?

Sure


Silent on success

Do we want to be unixy and be silent on success? What determines whether something is successful?
I'm talking about when we create, update and delete things?

Some things take a while and I generally like to tail output of the CloudFormation stack create/update/delete


[BEAM-115,BEAM-1195] Build trigger state machine from Runner API Trigger proto directly

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

This replaces the case dispatch on the Java SDK's Trigger type with a case dispatch on the Runner API's Trigger type. This is a step towards runners/core-java executing triggers from any language.
This also attempts to mitigate unresolved discussions around "reshuffle" by translating the unofficial ReshuffleTrigger (a hack so that a runner can know that the trigger is part of a Reshuffle operation) to an official "Always" trigger, since the meaning is the same.
Once users have another way of creating an "Always" trigger it will not be safe to assume it is inside a Reshuffle. The alternative is getting rid of all special casing based on the trigger, and do something based on the transform instead. This is just a tech debt payback that I am deferring.

Coverage increased (+0.03%) to 69.213% when pulling b046d55b07bb503bade78129536d33b2684c8faf on kennknowles:trigger-proto-state-machines into 918c7fa3666e60343b86087c3450e77291b4d024 on apache:master.

Coverage increased (+0.01%) to 69.209% when pulling 2b8962a6d2e04559912f03336243a3fe2dc2427d on kennknowles:trigger-proto-state-machines into 01de255753f5417b7241e8c883ca26b51dfaf6f5 on apache:master.

retest this please

Not sure about the flakes. Rebased, though I think Jenkins builds off the merge commit.

Run Dataflow RunnableOnService

Run Flink RunnableOnService

Coverage increased (+0.02%) to 69.308% when pulling 53cf479f8cb346f6b756b9a7b3e4d5b357e46661 on kennknowles:trigger-proto-state-machines into 154c543057b54ca46975cb9e9d360a52784518d7 on apache:master.

Run Dataflow RunnableOnService

Run Dataflow RunnableOnService

(Trying to debug jenkins flakes)

So RAT is failing because TestUtils.java is "not found"? I don't really know what to make of it.

Run Dataflow RunnableOnService

Seems to be reliable.

Coverage decreased (-0.002%) to 69.298% when pulling 820d4aed1b619d2e270fd0ad38fc76140b5d925f on kennknowles:trigger-proto-state-machines into 0d639e63d635903df949a09186d1a6f2c0d9b0ae on apache:master.

Run Dataflow RunnableOnService

R: @tgroh

retest this please

HBaseIOTest seems to have regular issue with timeouts. Investigating whether it is related to this PR.

Noting that Dataflow RunnableOnService tests pass so I won't rerun them if nothing else changes.

retest this please

Run Flink RunnableOnService

Run Flink RunnableOnService

HBaseIOTest timed out. Please do review regardless. I believe the Flink ROS test run provides very high confidence in this PR, but will also investigate that timeout - which is recurring across PRs - and retest, etc.

No reason not to ship it, I think.

(Explicit LGTM)


feat: add plasgate sms provider

What kind of change does this PR introduce?
feat: add plasgate sms provider
What is the current behavior?
SMS like Twilio in SE Asia countries are expensive and currently we are using PlasGate as local SMS provider. I would to see if it can implement in Supabase.

#1181

What is the new behavior?
Add PlasGate SMS provider for local SE Asia.

Hey @Dominic-Preap,
Thanks for the PR - we're unfortunately not moving forward with this as we're going with a HTTP Hook approach -  let us know if you'd be keen to prototype the hook to allow for Plasgate support and we can circle back to ping you once we're nearing completion.
Thanks!


Configure Cloudfront for argo-site

Cloudfront CDN should increase performance of Argo-site. Additionally argo-cli binary also should be behind CDN to improve download speed in non-us regions.

Site was moved to https://applatix.com/open-source/argo which is behind MaxCDN. Closing ticket.


Update location on every login

@sanaltsk, @donygeorge As of now, we are not using any Java 1.8 specific code. But anyway should we all use the same version?

Any particular reason you are asking about this? Did you notice some issue? Using the same version would be nice.

@donygeorge pushed the changes, have incorporated the comments from previous review

Sanal, can you update timestamp when you update location? I need this to discover users who have been active within past 15 mins


deck doesn't delete "documents" entities from services

in Kong Enterprise 2.1
Reproduction Scenario:

Create a service
Create a "Document" in the service
Run deck reset

You should get an error:

Error: 400 Bad Request {"message":"an existing 'document_objects' entity references this 'services' entity","name":"foreign key violation","fields":{"@referenced_by":"document_objects"},"code":4}

Same issue as https://github.com/Kong/deck/issues/208. The problem is the new document objects entities that only exist in KE and are not managed by decK.


TestJSONSetWithInvalidJSON/Set_Invalid_JSON Integration Test failing

The following test case is failing:
TestJSONSetWithInvalidJSON/Set_Invalid_JSON

Can be assigned as a good first issue if anyone wants.

Could this be because of the error message from sonic.Unmarshal being different from that of encoding/json
https://github.com/DiceDB/dice/blob/34d5fa5ae700de8d7d87e985345db0a6c00b077f/tests/json_test.go#L173-L177
the expected field can be set to "ERR invalid JSON" since we're doing a strings.Contains() right now.
We can also check if the error message starts with the expected prefix using strings.HasPrefix(), instead of checking for the substring using strings.Contains().

https://github.com/DiceDB/dice/blob/34d5fa5ae700de8d7d87e985345db0a6c00b077f/core/eval.go#L362-L365
Also using sonicUnmarshalString can be an option here as the function definition for sonic.Unmarshal suggests below:
// Unmarshal parses the JSON-encoded data and stores the result in the value pointed to by v.
// NOTICE: This API copies given buffer by default,
// if you want to pass JSON more efficiently, use UnmarshalString instead.
func Unmarshal(buf []byte, val interface{}) error {
    return ConfigDefault.Unmarshal(buf, val)
}

I have tested out the above two and everything seems to be working well. Do you think there will be much improvement in performance during unmarshal this way? Can be a good benchmark to test out
cc: @JyotinderSingh

Could this be because of the error message from sonic.Unmarshal being different from that of encoding/json
https://github.com/DiceDB/dice/blob/34d5fa5ae700de8d7d87e985345db0a6c00b077f/tests/json_test.go#L173-L177
the expected field can be set to "ERR invalid JSON" since we're doing a strings.Contains() right now.
We can also check if the error message starts with the expected prefix using strings.HasPrefix(), instead of checking for the substring using strings.Contains().

https://github.com/DiceDB/dice/blob/34d5fa5ae700de8d7d87e985345db0a6c00b077f/core/eval.go#L362-L365
Also using sonicUnmarshalString can be an option here as the function definition for sonic.Unmarshal suggests below:

// Unmarshal parses the JSON-encoded data and stores the result in the value pointed to by v.

// NOTICE: This API copies given buffer by default,

// if you want to pass JSON more efficiently, use UnmarshalString instead.

func Unmarshal(buf []byte, val interface{}) error {

    return ConfigDefault.Unmarshal(buf, val)

}

I have tested out the above two and everything seems to be working well. Can raise a PR if approved.
Do you think there will be much improvement in performance during unmarshal this way? Can be a good benchmark to test out
cc: @JyotinderSingh

Thanks for investigating @Maveric-k07. Let's use a shorter message for asserting the error value, along with the HasPrefix and UnmarshalString APIs.
Please raise a PR for this.


Added golang build flag for massive speedup

Until now, writing to any golang file would run a go guild
command that would take up to 10 seconds during which the
editor would be essentially frozen. By using the -i flag,
any subsequent writes to that file will be almost instantaneous.

This is not needed.  It can be achieved without code chabges, by setting g:syntastic_go_go_build_args.  Patches that change defaults are not accepted; sorry, but no.


The extension popup is unstyled on Edge

I'll investigate this.

@mybearworld How did you install the extension. In this version, npm run ci must be run before the extension can be loaded into Chrome, Edge, or Opera. In this version, the extension popup uses tailwind, which is why the dependencies must be installed. Alternatively, you can try the release artifact in this Action (which has the node_modules folder): https://github.com/imadeanaccount1/wasteof.plus/actions/runs/5693794806

npm run ci errors as the script "ci" is not existent. Installing the dependencies does fix this, though.

sorry, npm ci is the right command 😅


Invalid authentication

Describe the bug
2021-07-03 20:48:44 INFO (SyncWorker_13) [custom_components.tuya_v2.config_flow] TuyaConfigFlow._try_login finish, response:, {'code': 2406, 'msg': 'skill id invalid', 'success': False, 't':<PHONE_NUMBER>595}
Expected behavior
I recreated a new account. changed pass in smartlife app. enabled all regions in IOT.
I have the Netherlands as country in the smart life app. So I use europe and country code 31
Additional context
I use version 2021.7 and 1.3 of the Tuya v2.

I have a similar issue.
My tuya app is on +61
and my dev cloud project is set to China.
"Invalid authentication"

I had the same problem with v1 addon while using an account that was made by "sign in with google". Maybe this applies here also?

I'm also having this issue...
My Dev account is set to the US region. My Tuya Smart account is AU based. I've tried logging in with +61.

@remb0 Try v1.3.1

@tsutsuku I have tried 1.3.1 but still getting the following errors
2021-07-05 12:33:12 INFO (MainThread) [custom_components.tuya_v2.config_flow] TuyaConfigFlow.async_step_user start, is_import= {'password': 'xxxxxx', 'access_id': 'xxxxxx', 'access_secret': 'xxxxxx', 'tuya_app_type': 'tuyaSmart', 'country_code': '+61', 'username': 'xxxxxx'} 2021-07-05 12:33:12 INFO (SyncWorker_13) [custom_components.tuya_v2.config_flow] TuyaConfigFlow._try_login start, user_input: {'password': 'xxxxxx', 'access_id': 'xxxxxx', 'access_secret': 'xxxxxx', 'tuya_app_type': 'tuyaSmart', 'country_code': '+61', 'username': 'xxxxxx', 'tuya_project_type': 0} 2021-07-05 12:33:17 INFO (SyncWorker_13) [custom_components.tuya_v2.config_flow] TuyaConfigFlow._try_login finish, response:, {'code': 2406, 'msg': 'skill id invalid', 'success': False, 't':<PHONE_NUMBER>444}

1.3.1 also gives me an error:
2021-07-06 01:59:08 INFO (SyncWorker_3) [custom_components.tuya_v2.config_flow] TuyaConfigFlow._try_login finish, response:, {'code': 1106, 'msg': 'permission deny', 'success': False, 't':<PHONE_NUMBER>868}

@Filip722 - Just found https://developer.tuya.com/en/docs/iot/Home_Assistant_Integration?id=Kamcjcbvk2mu8#title-5-FAQ . Have a look at the Error Code 1106 section Maybe this will help you.

Found issue and workaround. I have google created acc on Smart Life app. Managed to connect with account name instead mail. It is long starting with **gg-**and some numbers. You can find it under Me->Home Management->Your home->Home owner

I just resolved the issue by selecting myy country as china on IOT dev platform.
So server and IOT platform are on same server.
I dont know if i might have to confirm / validate a chinese number for this.
my country code was still Aus +61 on the hacs login

v1.3.1 does not need region, so how about update to v1.3.1, then try again?

unable to setup the integration with 'Invalid authentication'
getting the following in logs:
(SyncWorker_5) [custom_components.tuya_v2.config_flow] TuyaConfigFlow._try_login finish, response:, {'code': 2406, 'msg': 'skill id invalid', 'success': False, 't':<PHONE_NUMBER>435}

My issue was also resolved by linking my tuyasmart account with my iot project

Tuya login error response: {'code': 500, 'msg': 'system error,please contact the admin', 'success': False, 't':<PHONE_NUMBER>766}
If someone could héla me??

Changed the data center, unlinked my app from the old and relinked it. Worked then. Chose Western American instead of the default chinese data center in the top far right

I was getting code 2406 skill id invalid.  My country is United Arab Emirates and my Datacenter according to Mappings is Central Europe.  I changed country settings in Home Assistant Tuya account add-on UI to United Kingdom instead of United Arab Emirates and it started to work. Not sure why it's not accepting UAE.

 Sometimes it works, then it suddenly stops working, then it won't even finish the setup. SMH.

Hi @remb0, sorry for the inconvenience. This has been fixed in the Home Assistant Core integration version, which is available now with Home Assistant Core 2021.11.
You can check the Error Code and Troubleshooting to help resolve the error code issues.
As we stop maintain the Tuya v2 now, please help to create your github issues in the Home Assistant Core repository. And it's still maintained by Tuya developer team and Home Assistant Team.

Thanks, authentication works with the core integration

thanks . Happy to find back my Tuya . No way to remove iot "trying project" done without success for Tuya V2?

Dear @zlinoliver thanks for your response. Considering "Tuya v2" won't be supported anymore by the Tuya team, is there any way to integrate the Tuya Wifi door/window sensors into HA anyhow?
Thanks.


Track Expense - Track Expense option is missing on Self DM page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.80-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4601595&group_by=cases:section_id&group_order=asc&group_id=309130
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Open Global Create (FAB)
Choose Track expense
Create a manual expense for $1
Create the expense
Verify it's created in the self DM
Navigate to the Self DM
Click on the + on compose box ( maybe this should be done couple times)

Expected Result:
Track expense option should be present
Actual Result:
Track Expense option is missing on Self DM page after user creates the first Track expense from FAB
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/971277a5-c0f3-4165-81d0-51224d6ab848

View all open jobs on GitHub

@twisterdotcom I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

I think it's a BE problem, after tracking an expense, the response adds concierge as the participant.

Refreshing fixes the problem

Okay, adding Internal

Will probably just close this for now: https://expensify.slack.com/archives/C066HJM2CAZ/p1721055590330089

Issue not reproducible during KI retests. (First week)


Add "GC Optimization" For Twitchy Desktop Lite

The Official Twitch Application has custom GC code that triggers once when the window is blurred (out of focus) or minimized. We should be able to add that as well. (Manually trigger GC.)

Reference: https://github.com/electron/electron/issues/1213


iterate_ngrams is very slow

This function is used very often when decrypting vigenere, and its slowness has impacted the overall speed of vigenere. Need to look into speeding it up, perhaps with the help of itertools, or even writing a snippet of compiled C/D/Rust/whatever

Increased fairly significantly with 55a816c615192c457c54c752eda9b0431e736069
Pushing the boundary of the language I believe. Will be worth looking into further but Im happy with the results for 0.1 at this stage


Warnings in Julia v0.4

Thank you for this package.
Could you please fix a few warnings:

WARNING: module IntModN should explicitly import / from Base
WARNING: Base.Uint is deprecated, use UInt instead.
likely near /home/juliohm/.julia/v0.4/IntModN/src/IntModN.jl:109
WARNING: module IntModN should explicitly import == from Base
WARNING: module IntModN should explicitly import <= from Base
WARNING: module IntModN should explicitly import < from Base
WARNING: module IntModN should explicitly import - from Base
WARNING: module IntModN should explicitly import ^ from Base
WARNING: module IntModN should explicitly import + from Base
WARNING: module IntModN should explicitly import * from Base
WARNING: module IntModN should explicitly import & from Base
WARNING: module IntModN should explicitly import | from Base
WARNING: module IntModN should explicitly import $ from Base
WARNING: module IntModN should explicitly import % from Base
WARNING: module IntModN should explicitly import << from Base
WARNING: module IntModN should explicitly import >>> from Base
WARNING: using IntModN.factor in module Main conflicts with an existing identifier.

i'll try sometime this week.
On Wed, Apr 27, 2016 at 12:23:42PM -0700, Júlio Hoffimann Mendes wrote:

Thank you for this package.
Could you please fix a few warnings:

WARNING: module IntModN should explicitly import / from Base
WARNING: Base.Uint is deprecated, use UInt instead.
likely near /home/juliohm/.julia/v0.4/IntModN/src/IntModN.jl:109
WARNING: module IntModN should explicitly import == from Base
WARNING: module IntModN should explicitly import <= from Base
WARNING: module IntModN should explicitly import < from Base
WARNING: module IntModN should explicitly import - from Base
WARNING: module IntModN should explicitly import ^ from Base
WARNING: module IntModN should explicitly import + from Base
WARNING: module IntModN should explicitly import * from Base
WARNING: module IntModN should explicitly import & from Base
WARNING: module IntModN should explicitly import | from Base
WARNING: module IntModN should explicitly import $ from Base
WARNING: module IntModN should explicitly import % from Base
WARNING: module IntModN should explicitly import << from Base
WARNING: module IntModN should explicitly import >>> from Base
WARNING: using IntModN.factor in module Main conflicts with an existing identifier.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/andrewcooke/IntModN.jl/issues/3

ok, so this isn't so simple.  the code in git doesn't have these warnings, so you can use that.  however, some tests for the code in git fail - i think i was working on trying to simplify the structure at some point and never finished.
i don't have time to complete that work, so your best bet may be to branch the code at the last release and fix the issues yourself.
i will add a note to the front page indicating that the package has issues and i don't have time to maintain it right now.  sorry.

I will close the issue since the project is not maintained. Thanks.


Adjust frame track colors to be similar when durations are similar

When all frame times are fairly similar, we want all colors to be fairly
similar, as there is no need to point out small differences by using
a different color. We solve this by adjusting the maximum upwards a bit
when computing colors (and only then!) and when the minimum and
maximum are close to each other. Colors are still interpolated between
the real minimum and the adjusted maximum, so there will be still some
variation in color.
When the maximum is "large enough" compared with the minimum,
the behavior is not changed.
Tested: Manually tested frame track.
Bug: http://b/174224286

Screenshot before the change:

Screenshot after the change:

What do you think?

You are right, as long as we are using minimum and only the minimum, single outliers can completely mess up this scheme. I like your suggestion to base this on the average.

For your capture:
Old code (before this PR):

New code (current version, based on your suggestion):

I like it! :)
LGTM again!

Thanks for the review @karupayun


[BUG]: FusedScaleMaskSoftmax last dimension does not sum to 1

🐛 Describe the bug
I use the following code to test the softmax, but the result does not sum to one
from colossalai import kernel
import math
import torch

attention_head_size = 32

softmax = kernel.FusedScaleMaskSoftmax(input_in_fp16=True,
                                                    input_in_bf16=False,
                                                    attn_mask_type=None,
                                                    scaled_masked_softmax_fusion=True,
                                                    mask_func=lambda x, mask:x.masked_fill(mask, -50000),
                                                    softmax_in_fp32=True,
                                                    scale=1/math.sqrt(attention_head_size))

length = 200
b = 1
h = 4
hidden_states = torch.randn(b, h, length, length).half()
mask = torch.rand(1, 1, length, length)>0.5

print(softmax.is_kernel_available(mask, b, h, length, length))
output = softmax(hidden_states, mask)
print(output[0, 0, 0].sum())  # the result will be something like tensor(1.1623e-05, dtype=torch.float16), not equal 1

However, if i purposely change the head to make it not using fusion kernel, the result does sum to one.
Environment
Colossal-AI version: 0.1.13
PyTorch Version: 1.12.0
PyTorch Version required by Colossal-AI: 1.12
PyTorch version match: ✓
System CUDA Version: 11.2
CUDA Version required by PyTorch: 11.3
CUDA Version required by Colossal-AI: 11.3
CUDA Version Match: x
CUDA Extension: ✓

Thanks for your issue. We'd get back to you ASAP. :)

Finally, we found out that this code should be running on gpu.


Getting slower and slower

I took a look at our response times yesterday on heroku.
Average response time short after startup: 31ms
Average response time few hours later: 689ms
I doubt the number of requests/s increased a lot during that time, but we had a lot more traffic than we ever had before this weekend. Now that traffic decreased, response time also went down again.
I noticed this a few days ago and reverted from master to @b88db00b97e1fc0e8475296783c77c72a04709c2 - that is before the recent changes with HTTP 429 handling and caching changes. Reverting didn't help, however.
Maybe we don't have enough CPU or RAM on heroku and that slows it down when we have that many requests.

Someone mentioned to me that quality is continuing to degrade over the past few days, so I went ahead and reverted to the last build before this update. So ~2 months ago. We'll need to figure out what's causing these issues, and thoroughly test it before releasing it. @jomo

I had already reverted to @de5bad8702ec622d70fb26b780892880e96d203c from 12 days ago :stuck_out_tongue_closed_eyes:

Was still having issues :p

See also: #66.
To reproduce the slowness we had on heroku, we could probably run test/bulk.sh for some minutes (hours?) in a loop until response time gets really slow.

I'm closing this. Every now and then we get slow response times. It's just a few requests and only for a few seconds (or minutes at most). And it's always far below 0.1% of our requests, I think that's acceptable and it will most probably go away with #66 and #79 anyway.


resin-image: Generate fingerprints just before image creation

If we generate the fingerprints as a post rootfs, other functions might
change the rootfs in different ways (for example zap passwd). In this
case the fingerprints will depend on the order of this function and if
it is not the last one might include bad hashes. Moving this generation
before image create we ensure that all the other postrootfs functions
were finalized.
Signed-off-by: Andrei Gherzan<EMAIL_ADDRESS>@floion @telphan

LGTM


More Customization Please!

@martinfekete10
The App is awesome! but can you consider more customization please? on menu popover player an option remove the buttons we don't need to make it clean and how we need to. (for example i love to have art, title, artist names and seekbar only)

and last at mini player an option to lock the placement and make it wider please!!!

thank you so much!
John.

Hi John, today I released a new version of Tuneful with a new, minimalist popover option and also slightly updated the look of mini player

Hi John, today I released a new version of Tuneful with a new, minimalist popover option and also slightly updated the look of mini player

@martinfekete10
i tested and i totally!!! love it! i found a bug tho! when i select hidden or app icon at menu bar icon the sonf info details shrinks and moves even if i use 300 pixels at song info width, the only way to fix this and stay unmoved is to use album art at menu bar icon.
can you check this behaviour if you have time? beside that works great and the app feels smoother! thank you for you work!


how to install the "map"

"assumes that the map files for the conda-forge channel have also been installed"
-> What ways exist to install the "map" for the files?

conda-suggest-conda-forge is one such collection of map files, you can install it with conda install -c conda-forge conda-suggest-conda-forge

thanks
is it possible to get it without to install conda?

as a file

Yes you just need to copy a file to the $CONDA_SUGGEST_PATH, as per the readme: https://github.com/conda-incubator/conda-suggest#map-files

where can I get the file from?

You can get the file by generating it or from that other repo: https://github.com/regro/conda-suggest-conda-forge

thanks for your help and fast response


Add reference to build docker image using dotnet publish

Fixes #29276

Internal previews

📄 File
🔗 Preview link

aspnetcore/host-and-deploy/docker/building-net-docker-images.md
Docker images for ASP.NET Core

Thanks for the contribution. Be sure to check out your name as a contributor about the 3rd of next month on the What's new page in the Community contributors section at the bottom.


docs(instance.mdx): adjust the references

按照示例没有给出api声明,理解应该是上文中的rsbuild
Summary
Related Links

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

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

Thanks!


Grammar1

I committed the correct thing to my master and the wrong thing to the branch grammar1 sorry about that. So here is the correction.

Conflicts, can you repull from master and fix? Thank you!


ADO Plugin renaming and screenshots

ADO Plugin renaming for BD and screenshots

@dmamidibd @devmehtabd FYI - Changes you saw in Dev Zoomin are reflected here.


Reset idx if number overflows

Although rare, it is possible that a long running process could cause idx to overflow leading to a panic when selecting an item in conns array

Need to set p.pos atomically instead of changing idx


HoloToolkit 3D text cannot be hidden behind 3D objects

[Migrated from internal feedback 10976715]
Using HoloToolkit UI/Prefabs/3DText is not able to be hidden behind 3D objects

I'm unable to repro this. Can anybody else?

Also unable to repro.


Interesting / misleading advisories

This thread contains advisories which were found to be interesting/misleading/difficult to process in any manner whatsoever while developing VulnerableCode.

CVE-2008-2938
Although Tomcat says: Not a vulnerability in Tomcat
Reported-By: John M. Horan<EMAIL_ADDRESS>

https://nvd.nist.gov/vuln/detail/CVE-2013-2596
Although it is a real bug the ranges of what software is vulnerable is not correct and probably too broad (too many kernels reported). Also it very much depends on the configuration used to build the software.

Related: https://github.com/nexB/vulnerablecode/issues/1200

Related: https://github.com/nexB/vulnerablecode/issues/1281


IE Mobile 9 lacks support for console.info (and is not being polyfilled)

I wasn't able to find any official reference on which versions of IE Mobile support the console APIs, but I have found that a device with user agent Mozilla/5.0 (compatible; MSIE 9.0; Windows Phone OS 7.5; Trident/5.0; IEMobile/9.0; LG; LG-C900) does not have support for the console.info function. When requesting it explicitly on Polyfill.io I got back the following:
/* Polyfill service v3.12.1
 * For detailed credits and licence information see http://github.com/financial-times/polyfill-service.
 *
 * UA detected: ie_mob/9.0.0
 * Features requested: console.info
 *  */

I'm assuming console.error and most other console functions also don't work on this platform, but I have not yet been able to test this.

@rubencodes you need to add a browser target to https://github.com/Financial-Times/polyfill-service/blob/master/polyfills/console/info/config.json.
See https://polyfill.io/v2/docs/contributing/common-scenarios#adding-browsers

PR for this here: https://github.com/Financial-Times/polyfill-service/pull/914


Adding the 2024-01-01 API Version of Microsoft.HybridContainerService

ARM (Control Plane) API Specification Update Pull Request

Tip: overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

 Click here to see the details of Step 1
Breaking changes review (Diagram Step 1)
If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

 Click here to see the details of Step 2
ARM API changes review (Diagram Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

 Click here to see the diagram footnotes
Diagram footnotes
[1] ARM review queue (for merge queues, see [2])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.

Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[x] New API version. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[x] Other, please clarify:
This API Version has already been ARM signed off in the RPSaaSMaster branch in the private repo.
https://github.com/Azure/azure-rest-api-specs-pr/pull/15724
https://github.com/Azure/azure-rest-api-specs-pr/pull/16073

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the Diagram Step 2, "ARM API changes Review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

This API Version has already been ARM signed off in the RPSaaSMaster branch in the private repo. Please refer to the following PRs:

https://github.com/Azure/azure-rest-api-specs-pr/pull/15724
https://github.com/Azure/azure-rest-api-specs-pr/pull/16073

/pr RequestMerge


Docker build fails on Mac

I tried docker-compose up -d --build app on a Mac (with gcc) and it failed with
    unable to execute 'gcc': No such file or directory
    unable to execute 'gcc': No such file or directory

I don't think this is Mac-specific.  At least one of those python modules has to be compiled.  Adding this to Dockerfile at an appropriate point cured the buildproblem.
RUN apt-get update
RUN apt-get -y install build-essential libffi-dev

Thanks for spotting this @kleinfelter
I switched the Docker image to Alpine:3.8 and installed the correct packages to fix this issue


fix(sec): upgrade gopkg.in/yaml.v2 to 2.2.8

What happened？
There are 1 security vulnerabilities found in gopkg.in/yaml.v2 v2.2.4

CVE-2019-11254

What did I do？
Upgrade gopkg.in/yaml.v2 from v2.2.4 to 2.2.8 for vulnerability fix
What did you expect to happen？
Ideally, no insecure libs should be used.
How can we automate the detection of these types of issues?
By using the GitHub Actions configurations provided by murphysec, we can conduct automatic code security checks in our CI pipeline.
The specification of the pull request
PR Specification from OSCS

@dack-su Update the indirect package version in go.mod might take no effect as it is imported by third-party packages.

@dack-su Thanks for your contribution, but updates the indirect package in go.mod makes no sense.


Direct ML Compile Error

Hey there,
when compiling for windows 1809 using direct ml with cargo
[dependencies.ort]
version =  "1.15.2"
features = ["download-binaries", "directml"]

I get the following error:
error: linking with `link.exe` failed: exit code: 1120
...
  = note: libort-823b36f3ee700ecc.rlib(ort-823b36f3ee700ecc.ort.b1d99b0d4efe1b77-cgu.11.rcgu.o) : error LNK2019: unresolved external symbol OrtSessionOptionsAppendExecutionProvider_DML referenced in function _ZN3ort19execution_providers17ExecutionProvider5apply17h70344b19a4319081E

Could the reason be that direct ml is not installed in windows 1809? I tried to install it with
nuget install Microsoft.AI.DirectML -Version 1.12.1
Is there another, better way? I cannot upgrade the windows version at the momen
Thanks!

IIRC the download strategy does not have binaries compiled with DirectML support, you'd have to compile ONNX Runtime yourself and use the system strategy.

This worked! Thanks for the fast reponse :)


Add aur package for archlinux

Need to add webkit2gtk as deps then create surrealist-bin package in aur.
Can use https://github.com/marketplace/actions/publish-aur-package

I've published a package, https://aur.archlinux.org/packages/surrealist-bin. It uses the .deb file.

I've published a package, https://aur.archlinux.org/packages/surrealist-bin. It uses the .deb file.

Thanks a lot for that! ❤
@tacheometry My apologies for not knowing much about this, will this be kept up to date with the latest version?
I'll close this issue as resolved for now


Increment deployment target from iOS 12 to 13.0

This PR drops support for iOS 12, removes properties deprecated by iOS 13, and increments the deployment target to iOS 13.
[Related Issue]

[Related Issue]

That link seems to be broken.

Does the README need to be updated?

The deployment target of the two test targets should be updated too.

That link seems to be broken.

I've added you to the github ArcGIS/open-source-apps team. The link should now be accessible.

Does the README need to be updated?

We don't list iOS version information in the README, but adding a line to the Release notes might be a good idea, since we are ahead of the SDK a bit.

Does the README need to be updated?

At first glance, I didn't think I needed to. Upon further reflection (nudged by your suggestion), there are two areas where the README now reflects the iOS 13 deployment target.

Note on simulating the app on macOS Mojave
Requirements- requires a device or simulator running/simulating iOS 13.0

The deployment target of the two test targets should be updated too.

They're now updated.

unless we want to add something to the RELEASE notes.

I've updated the release notes.
@mhdostal @philium would you re-review?


Drogon missing in the Web Framework Benchmark Round 22 ??

Any idea why Drogon is missing from the most popular Web Framework Benchmark Round 22?
https://www.techempower.com/benchmarks/#hw=ph&test=fortune&section=data-r22

I see that @an-tao has updated drogon version in the benchmark code. Is this going to work now in Round 23?


Multi-label classification

My dataset contains 5 unique labels for instance:
1 0 NaN 1 0
Where 1 means it has that feature. 0 means it doesn’t and NaN means that we have no observation about that feature and it shouldn’t included in loss.
Is it possible to make ViT multi-label for a dataset like this?

@ramizdundar Hi Ramiz! Yes, just use BCEWithLogits loss instead!

@ramizdundar Hi Ramiz! Yes, just use BCEWithLogits loss instead!

I pass image through vit and get using np.argmax class label(794), how can i get class name by class label?

Hi @lucidrains ,
Please let me know how to vision transformer for multiclass-multilabel classification?
For now, I am using resnet model.
Input -> [BSx3x8x100] output -> [BSx1x5x181]
Lable -> [BSx5]
I have a total of 181 classes. And for each input 5 labels are being predicted.
I am using Cross-Entropy loss for this multilabel classification.

@lucidrains @ramizdundar  can you share the train.py i want to train it on customd dataset /??

@ramizdundar Could you please share an example code for multi-label classification using ViT?


Add public MpdClient#isConnected method

In some cases it is useful to know the current state of the client without listening to end and ready events. So I've created such method with the PR.

Why not listen to the 'end' event? 'end' signals that the client is no longer connected.

I want to pass an instance of MPD client from one application layer to another one. ready event could be already emitted when MPD client is passed in, so there is no way to check if the client is connected at the deepest layer.

Sounds like you need to pass this state as well as the MPD client instance to the other application layer.

@andrewrk, it's sounds like a dirty hack, but I've got your point.


[FEATURE REQUEST] Option to skip .webp images when converting all images in vault

First of all, I love this plugin very much. Thank you for all your hard work on it!
My feature request would be an option on the Process all vault images panel to skip processing images of a certain filetype (i.e. webp images). Right now it seems to do exactly what it says (processes all images in the vault), regardless of wether or not they have already been converted. Occassionally, I like to run this command when tidying up my vault, but it would be nice if it would only process unprocessed images.
Thanks!

Hold up, I see @KaiReichart already has a pull request submitted for this feature. Would be nice to see this implemented!  https://github.com/xRyul/obsidian-image-converter/pull/48#issue-2102523700

There are some improvements in 1.2.3 for both commands: when processing whole vault or whole note. You can explicitly specify which formats to skip too.

I am closing this for now.


Fix tiny typo in licensing section of readme

Right now it says:
"Opening a pull requests is assumed to signal agreement with these licensing terms."
It should probably instead say one of the two below:

"Opening a pull REQUEST is assumed to signal agreement with these licensing terms."
"The opening of pull requests is assumed to signal agreement with these licensing terms."

You're right, thanks!
bors r+

Keep up the good work!