all telescope windows empty using lunarvim

Describe the bug
When I try to run any action / generator inside lunarvim the telescopes windows are empty and commands to dont work.
To Reproduce
Steps to reproduce the behavior:

Create a new nx repo with an application
Install extension inside lunarvim using packer
Run :Telescope nx generators -> empty window shows up
type in anything and hit enter -> error displayed

E5108: Error executing lua ...arvim/site/pack/lazy/opt/nx.nvim/lua/nx/multirunners.lua:124: attempt to index a nil value
stack traceback:
        ...arvim/site/pack/lazy/opt/nx.nvim/lua/nx/multirunners.lua:124: in function 'run_replace_or_original'
        ...ack/lazy/opt/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'key_func'
        .../pack/lazy/opt/telescope.nvim/lua/telescope/mappings.lua:350: in function 'execute_keymap'
        [string ":lua"]:1: in main chunk

Expected behavior
Available generators beeing displayed
Screenshots

Desktop (please complete the following information):

OS: [WSL Ubuntu 20.04]
Version: [Commit c4d2243]
lunarvim: [1.3]
neovim: [0.9]

Additional context
config.lua
lvim.plugins = {
 ...
  {
    'Equilibris/nx.nvim',
    dependencies = {
      'nvim-telescope/telescope.nvim',
    },
    config = function()
      require("nx").setup {}
    end
  }
}

nx is installed globaly using npm and the command is working inside my terminal.

This is a piece of the nx.log file:
[2023-10-03 20:37:52 LOG] ===================================
[2023-10-03 20:37:52 LOG] ========== Setting up NX ==========
[2023-10-03 20:37:52 LOG] ===================================
[2023-10-03 20:37:52 LOG] {
    ["projects"] = {
    },
    ["graph_file_name"] = /tmp/lvim.jan/VCuqH9/0.json,
    ["log"] = [2023-10-03 20:37:52 LOG] ===================================
[2023-10-03 20:37:52 LOG] ========== Setting up NX ==========
[2023-10-03 20:37:52 LOG] ===================================
,
    ["command_runner"] = function: 0x7f8cd9ce18f8,
    ["nx_cmd_root"] = nx,
    ["cache"] = {
      ["targets"] = {
      },
      ["actions"] = {
      },
    },
    ["form_renderer"] = function: 0x7f8cd9ca94d8,
    ["generators"] = {
      ["external"] = {
      },
      ["workspace"] = {
      },
    },
  }
[2023-10-03 20:37:52 LOG] Starting reading
[2023-10-03 20:37:52 LOG] ----------------
[2023-10-03 20:37:52 LOG] Reading ./nx.json
[2023-10-03 20:37:52 LOG] Read nx.json completed.
[2023-10-03 20:53:45 LOG] ===================================
[2023-10-03 20:53:45 LOG] ========== Setting up NX ==========
[2023-10-03 20:53:45 LOG] ===================================
[2023-10-03 20:53:45 LOG] {
    ["log"] = [2023-10-03 20:53:45 LOG] ===================================
[2023-10-03 20:53:45 LOG] ========== Setting up NX ==========
[2023-10-03 20:53:45 LOG] ===================================
,
    ["cache"] = {
      ["targets"] = {
      },
      ["actions"] = {
      },
    },
    ["generators"] = {
      ["external"] = {
      },
      ["workspace"] = {
      },
    },
    ["nx_cmd_root"] = nx,
    ["graph_file_name"] = /tmp/lvim.jan/5AS3Qc/0.json,
    ["form_renderer"] = function: 0x7fb655509b60,
    ["projects"] = {
    },
    ["command_runner"] = function: 0x7fb655502088,
  }
[2023-10-03 20:53:45 LOG] Starting reading
[2023-10-03 20:53:45 LOG] ----------------
[2023-10-03 20:53:45 LOG] Reading ./nx.json
[2023-10-03 20:53:45 LOG] Read nx.json completed.

For some reason both config and default_config are empty inside the setup function in the init.lua file of the plugin.
I added this code to my init.lua ( inside .config/nvim ):
require('nx').setup({nx_cmd_root='nx'})

This solved the problem for me: ;)


chore(dependencies): bump typescript and related dependency versions …

chore(dependencies): bump typescript and related dependency versions to support string literals
the current cli does not support string literals and is using an beta7. Bumping versions for typescript as well as the beta.
Fixes #279

lgtm


Attack time grows unexpectedly while latency is low

I am running some tests with Vegeta and, while at lower rates everything was going fine, I am hitting something weird when increasing the rate. The duration of the test is increasing a lot, but latencies are still relatively low.
See the results below for example for a 20 second test that eventually took nearly 2 minutes:
echo "GET http://api-server/path" | vegeta attack -rate=6000 -duration=20s -timeout=5s | tee results.bin | vegeta report
Requests	[total]				120000
Duration	[total, attack, wait]		1m57.971208919s, 1m57.922599203s, 48.609716ms
Latencies	[mean, 50, 95, 99, max]		28.419263ms, 39.337944ms, 43.668078ms, 206.692678ms, 206.692678ms
Bytes In	[total, mean]			6940836, 57.84
Bytes Out	[total, mean]			0, 0.00
Success		[ratio]				100.00%
Status Codes	[code:count]			200:120000
Error Set:

Btw, Vegeta is running in a 4 core machine and none of the cores goes beyond 50% utilization throughout the test.
Any advice or insight into what might be happening here will be welcomed!

Hello @vdel26,
Thanks for reporting. Would you please provide me with the intermediate results for me to debug your case? You can obtain them by redirecting vegeta attack to a file.

Oh, I see you did this. Could you send me your results.bin?

Here is my results.bin file: https://transfer.sh/Ts2se/results.bin
Thanks for your help!

@vdel26: You're welcome. What were the attack parameters for the results.bin file you sent me?

They were these:
vegeta attack -rate=8000 -duration=20s -timeout=5s

@vdel26: Which version of vegeta are you running? On which OS? 64 or 32 bits?

The OS is Ubuntu 14.04.2 LTS, 64 bits. More info below:
$ vegeta -version
v5.6.2

$ uname -a
Linux ip-172-30-0-218 3.13.0-48-generic #80-Ubuntu SMP Thu Mar 12 11:16:15 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

I'm working on a bunch of performance improvements which will allow a single instance of vegeta to drive higher rates.

Sounds great :) I'll be looking forward to try that when it is ready.

@vdel26: https://github.com/tsenart/vegeta/releases/tag/v5.6.3

@tsenart :+1: the changes work great, I was able to reach much higher rates than before on the same server.
I am not sure I understand the purpose of the new connections option? Is it to set an upper limit to avoid reaching the max number of open sockets?
Thanks again for the super quick turnaround on this issue.

The -connections flag defines how many open idle connections can exist per target host. When using HTTP keepalive, TCP connections are reused. However, due to the high throughput that Vegeta can generate, one connection isn't enough and more are created. We don't want to tear down idle connections before the end of the test because that would create large latency spikes if they're needed again, reducing the effective rate of the attack.


Fix some warnings on GCC 7.5

When building with GCC 7.5 and the following flags -Wsign-conversion -Wconversion these lines produced warnings.
For some reason GCC didn't like the UL literal only in multiplications...
#151

I don't understand why you can't cast char * to uint32_t - what does the error say?
As to the constant suffix, it is better to use UINT32_C(value) since U and UL are platform specific.

The header needs
#ifndef UINT8_MAX
#include <stdint.h>
#endif

in order to use UINT32_C, although it is already included via flatcc_flatbuffers.h via flatcc_types.h, but that depends on the order of include files.

Regarding the char to uint32_t conversion, I actually found out an open bug in the GCC project for this type of conversions: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49626
As for the literal suffix, I'll probably fix them tomorrow.
Thanks for the quick turnaround :)

I'll try make a commit based on clangs -Wsign-conversion warnings.
I'm not sure I want to deal with GCC bugs from 2011, that are still open. If you can't cast properly, all bets are off.

I agree, it's probably safe to suppress this warning on GCC.

For reference, the proposed changes were committed directly to master branch along with many others. See also related issue #151.


docs: Clarifies use of default archetypes when themes are active

I previously thought that, when you ran something like hugo new notes/test.md, hugo would automatically look in themes/THEMENAME/archetypes/deafult.md if archetypes/ did not exist, but that is not the case.
You need to specify the theme for this to work, ala: hugo new --theme=THEMENAME notes/test.md.  This tweaks the documentation to remind users about that argument.

--theme is no longer a valid flag with hugo new.  I'm now just specifying the theme in my config.toml and the archetype is correctly picked up when I run hugo new.


Get-SafeEvents

Get-SafeEvents Output in the "Data" Property can sometimes be a multiline xml value which gets cast into multiple (incorrect) object properties and invalidates subsequent objects output by the function.

Work in progress resolution implemented in #20


[Bug] Users with private email on GitHub cannot login

Bug Description:
Users who have their email address visibility set to hidden on GitHub are facing issues logging in currently. The reason for this is because the OAuth endpoint is being called with default no scope, but for private email addresses, the "user:email" scope is required.
Steps To Reproduce:
Steps to reproduce the bug behavior:

Check that your GitHub email is private
Click on login button on gallery website
Authorize the gallery application
See error

Expected behavior:
Users should be able to login just fine.
Additional Context:
The backend test server has already been updated to cater for this fix, frontend needs to redirect to GitHub OAuth endpoint with the scope "user:email". It might be a good idea to include the "read:user" scope as well for more clarity, even though it seems to achieve the same as the default no scope - worth exploring.

As we're using the recommended GitHub Application instead of OAuth Application, scope isn't required. Configuration is done directly with the application in settings.


SPFx Add-In for Word?

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Target SharePoint environment
SharePoint Online
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
SPFx plusbeta
Issue description
Any estimation of when will there be Office Add-In support for Microsoft Word/PowerPoint? Current plusbeta only supports Outlook, I am very keen to test the integration with Word/PP.
This youtube video below demonstrates that we can load the webpart on word and interact, mind sharing the manifest.xml that handles this for Word taskpane?
https://www.youtube.com/watch?v=3da-ApeHtV8
Building Office Addin would be much better and easier if can be done through SPFX as I no longer have to setup the SSO and auth just to call into sharepoint API using Node server. I really hope microsoft will have a timeline to extend and release this to GA on upcoming versions.

@VesaJuvonen this is truly unfortunate as a lot of our clients now had to stick with old deployment to azure/web app method to enable SSO instead of easy sharepoint way. I guess this might be more of an economical decision than technical.
Thanks for the update, much appreaciated.


[PINOT-4885] Changed in-memory byte buffers in realtime to use native order

Also, removed unused variables from classes

Codecov Report

Merging #1394 into master will decrease coverage by 0.02%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master    #1394      +/-   ##
==========================================
- Coverage   66.99%   66.96%   -0.03%
==========================================
  Files         781      781
  Lines       38243    38236       -7
  Branches     5027     5026       -1
==========================================
- Hits        25619    25604      -15
- Misses      10952    10955       +3
- Partials     1672     1677       +5

Impacted Files
Coverage Δ

.../pinot/core/realtime/impl/RealtimeSegmentImpl.java
71.19% <ø> (ø)
:arrow_up:

...riter/impl/FixedByteSingleValueMultiColWriter.java
85.41% <ø> (-0.59%)
:arrow_down:

...ter/impl/FixedBitSingleColumnMultiValueWriter.java
82.92% <ø> (ø)
:arrow_up:

...din/pinot/core/segment/memory/PinotDataBuffer.java
55.55% <ø> (ø)
:arrow_up:

...e/io/writer/impl/v1/FixedByteMultiValueWriter.java
56.92% <ø> (ø)
:arrow_up:

...ore/segment/index/column/ColumnIndexContainer.java
87.5% <ø> (ø)
:arrow_up:

...ot/core/data/readers/PinotSegmentRecordReader.java
61.01% <ø> (ø)
:arrow_up:

...in/pinot/core/segment/memory/PinotLByteBuffer.java
0% <0%> (ø)
:arrow_up:

.../io/reader/impl/v1/FixedByteSingleValueReader.java
0% <0%> (ø)
:arrow_up:

...e/io/reader/impl/v1/FixedByteMultiValueReader.java
50% <100%> (ø)
:arrow_up:

... and 26 more

Continue to review full report at Codecov.

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


SA1623 don't require "a value indicating"

To avoid unnecessary verbiage, I'd like it if SA1623 would respect this:

Gets or sets whether blah blah

Instead of marking it as an error and recommending this:

Gets or sets a value indicating whether blah blah

Same goes for the individual get and set descriptions. Essentially, it's okay if I omit the text "a value indicating" because it's totally unambiguous and I don't want to type it all out. I think that would be a better rule.

It could be an option to make this configurable, with the default being the current text. Downright changing the expected text will most likely result in problems for lots of our users.

Better yet. Very few rules are configurable right now, but it seems like they could be.

@JoshuaLamusga StyleCop Analyzers is an opinionated formatter. Configurability is not generally desirable, but it's easy to turn rules off.

I don't want to turn the rule off, or fork the entire project and maintain it forever (that's a popular suggestion people give here, apparently). But it's fine to just say "we don't want to" and I'll forget it.
I think a few 'best practices' floating around are really assertions, and in this case "configurability is not generally desirable" is one of those. I believe the opinion here is that having it confuses the 'styling standards' between projects that use StyleCop. But then, if projects can use other analyzers (or none) then there isn't really a styling standard in C#, so no reason I can see not to support configurable rules. I don't think that opinion has been validated in a hard-and-fast way (the industry hasn't complained about tslint yet). This wouldn't be problematic if there were a lot of decent analyzers to use, but there isn't really.

... and I don't want to type it all out ...

What if we improved the experience specifically for this part? For example, what if we recognized 'Gets or sets whether ...' and the code fix could automatically change it to 'Gets or sets a value indicating whether ...'? We could even take it one step further and recognize 'whether ...' and put the entire proper prefix on it.

Totally agreed sharwell. That would be a nice and neat solution.


Document server-side use of Python TUF

Review the capabilities of Python TUF and document how we'll want to invoke it as part of operating project infrastructure.
Procedures will need to cover several things:

[ ] Initial root key setup
[ ] Use with YubiHSM root keys
[ ] Rotation ceremonies
[ ] How infrastructure should be deployed (across message queues, etc.)

Changes to the repo for this may involves various utility scripts.

Probably useful to leverage the fixture creation scripts in #3 to create this.

Python TUF CLI documentation: https://github.com/theupdateframework/tuf/blob/develop/docs/CLI.mda

For reference Trail of Bits developed a runbook for the PSF to use in key-generation and signing ceremonies for TUF on PyPI: https://github.com/trailofbits/psf-tuf-runbook

This is a non-issue. PHP-TUF itself no longer uses Python, and the server side (Rugged), although built in Python, isn't our concern.


Github API compatibility of GFM

Nowadays I am developing a github_issue_backup utility, which uses parsedown for parsing data.
For this, I use Github Issues API to download data and then parse it with parsedown and present it with the CSS which Github uses, so that the user gets the same feeling as he gets in github issues. e.g. original github issue -> locally backed up github issue.
While doing so,
I have come across a limitation of parsedown while handling the data as given by Github API.
Typical line containing the body of a Github issue looks like the following.
  "body": "# GitHub Flavored Markdown\r\n\r\nGitHub uses \"GitHub Flavored Markdown,\" or GFM, across the site--in issues, comments, and pull requests. It differs from standard Markdown (SM) in a few significant ways, and adds some additional functionality.\r\n\r\nIf you're not already familiar with Markdown, take a look at Markdown Basics. If you'd like to know more about features that are available in issues, comments, and pull request descriptions, such as task lists, read Writing on GitHub.\r\n\r\n## Differences from traditional Markdown\r\n### Multiple underscores in words\r\n\r\nWhere Markdown transforms underscores (_) into italics, GFM ignores underscores in words, like this:\r\n> wow_great_stuff\r\n> do_this_and_do_that_and_another_thing.\r\n\r\nThis allows code and names with multiple underscores to render properly. To emphasize a portion of a word, use asterisks (`*`).\r\n\r\n### URL autolinking\r\n\r\nGFM will autolink standard URLs, so if you want to link to a URL (instead of setting link text), you can simply enter the URL and it will be turned into a link to that URL.\r\n\r\n`http://example.com`\r\n\r\nbecomes\r\n\r\nhttp://example.com\r\n\r\n### Strikethrough\r\n\r\nGFM adds syntax to create strikethrough text, which is missing from standard Markdown.\r\n\r\n`~~Mistaken text.~~`\r\n\r\nbecomes\r\n\r\n~~Mistaken text~~.\r\n\r\n### Fenced code blocks\r\n\r\nStandard Markdown converts text with four spaces at the beginning of each line into a code block; GFM also supports fenced blocks. Just wrap your code in triple backtics (as shown below) and you won't need to indent it by four spaces. Note that although fenced code blocks don't have to be preceded by a blank line—unlike indented code blocks—we recommend placing a blank line before them to make the raw Markdown easier to read.\r\n\r\nHere's an example:\r\n\r\n```\r\nfunction test() {\r\n  console.log(\"notice the blank line before this function?\");\r\n}\r\n```\r\n\r\nKeep in mind that, within lists, you must indent non-fenced code blocks eight spaces to render them properly.\r\n\r\n### Syntax highlighting\r\n\r\nCode blocks can be taken a step further by adding syntax highlighting. In your fenced block, add an optional language identifier and we'll run it through syntax highlighting. For example, to syntax highlight Ruby code:\r\n\r\n```ruby\r\nrequire 'redcarpet'\r\nmarkdown = Redcarpet.new(\"Hello World!\")\r\nputs markdown.to_html\r\n```\r\n\r\nWe use [Linguist](https://github.com/github/linguist) to perform language detection and syntax highlighting. You can find out which keywords are valid by perusing the languages YAML file.\r\n\r\n### Tables\r\n\r\nYou can create tables by assembling a list of words and dividing them with hyphens `-` (for the first row), and then separating each column with a pipe `|`:\r\n\r\nFirst Header  | Second Header\r\n------------- | -------------\r\nContent Cell  | Content Cell\r\nContent Cell  | Content Cell\r\n\r\nFor aesthetic purposes, you can also add extra pipes on the ends:\r\n\r\n| First Header  | Second Header |\r\n| ------------- | ------------- |\r\n| Content Cell  | Content Cell  |\r\n| Content Cell  | Content Cell  |\r\n\r\nNote that the dashes at the top don't need to match the length of the header text exactly:\r\n\r\n| Name | Description          |\r\n| ------------- | ----------- |\r\n| Help      | Display the help window.|\r\n| Close     | Closes a window     |\r\n\r\nYou can also include inline Markdown such as links, bold, italics, or strikethrough:\r\n\r\n| Name | Description          |\r\n| ------------- | ----------- |\r\n| Help      | ~~Display the~~ help window.|\r\n| Close     | _Closes_ a window     |\r\n\r\nFinally, by including colons `:` within the header row, you can define text to be left-aligned, right-aligned, or center-aligned:\r\n\r\n| Left-Aligned  | Center Aligned  | Right Aligned |\r\n| :------------ |:---------------:| -----:|\r\n| col 3 is      | some wordy text | $1600 |\r\n| col 2 is      | centered        |   $12 |\r\n| zebra stripes | are neat        |    $1 |\r\n\r\nA colon on the **left-most** side indicates a left-aligned column; a colon on the **right-most** side indicates a right-aligned column; a colon on **both sides** indicates a center-aligned column.\r\n\r\n# HTML\r\n\r\nYou can use a subset of HTML within your READMEs, issues, and pull requests.\r\n\r\nA full list of our supported tags and attributes can be found in the [github/markup repository](https://github.com/github/markup).",

Right now parsedown doesn't parse this data properly. (I will of course remove the  "body": " at the start and  ", at the end.)
This is a major problems while using parsedown with data retrieved from github API.

Parsedown treats this line as a single line, and ignores line breaks \r\n. Therefore it applied h1 tag to the whole chunk, and none of the features applied.

@erusev
To give a visual feel of what I want,
please see
Expected output for GFM
Actual output for GFM
also see the difference between basic markdown
Expected output for basic markdown
Actual output for basic markdown

Although it is possible to write a wrapper around parsedown for handling Github API, I think it should be a part of native parsedown.php because GFM would be most probably derived from Github. For most programming purpose fetching data would be via Github API. It would be better if you can provide native support to the data fetched via Github API.
Please have a look and let me know whether this makes sense.

the string you get from github is json encoded, did you call json_decode on it? that should give you proper newlines as far as I see.

Great.
It helped.
The json_decode worked like a charm.
Closing the issue.


speed of sageattention is same as sdp

Running CogVideox1.5 i2v, compare sdpa and sageattention for several times, almost same speed. Is there something wrong?
btw, torch compile is off, because it always throws an error.
system info
gpu: 3090
torch: 2.5.1+cu124
triton: 3.1.0
sageattention: 1.0.5

Ah almost forgot to mention, running on windows 10


Switches Elasticsearch to only write single-type indexes

Multi-type indexes are no longer supported starting with Elasticsearch
6. We added support for single-type writes in Zipkin 1.29 as an opt-in
functionality via ES_EXPERIMENTAL_SPAN2=true. This makes single-
type writes the only means to ingest spans, eventhough we still read
both models.
To turn off dual-reads, set ES_LEGACY_READS_ENABLED=false once your
collectors have been updated to Zipkin 1.31 and old indexes are no longer
valuable.

cc @openzipkin/elasticsearch

here's the double-conversion which is easier to obviate when we know how storage happens by type https://github.com/openzipkin/zipkin/pull/1700


CAN FW update questions

Dear all,
I am trying to upgrade the CAN FW update on iCub 2.5 (Eth base). I am able to connect to see CAN 1 and CAN 2. However I am not sure where to find the FW for each one one of CAN 1 and 2. I have updated CAN1 with the EMS.hex. and I got the following:
[DEBUG]  BUS:id   type     version
[DEBUG] CAN1:4       8     2.1.-1
[DEBUG] CAN1:1       8     2.1.-1
[DEBUG] CAN1:2       8     2.1.-1
[DEBUG] CAN1:3       8     2.1.-1
[DEBUG] CAN2:13       6     3.0.4
However the version is lower. I think the type is now correct, but I lost the S/N of strain and it now says PROTOCOL N/A.
Therefore, I may be using the wrong FW version. should I use MCP2PLUS and/or MCP4PLUS? If so what file is to be downloaded to CAN1 and CAN2?
Many thanks in advance.
Pedro

@marcoaccame can you help Pedro?
@pedrombmachado  can you please specify the name of your robot?

The robot should be iCubNottingham01 🤖

Hi @pattacini ,
Thanks for adding the robot name :).

Hi @pedrombmachado ,
as a general rule, instructions on how to update FW on ETH robots are on the icub wiki: http://wiki.icub.org/wiki/Firmware#Updating_the_FW_of_the_ETH_boards.
The link tells how to update the FW of both ETH and CAN boards.
If you read the wiki I am sure most of your doubts will be answered.
About your particular case, I can see that you are receiving support for how to perform FW udate for your robot iCubNottingham01  in another issue: https://github.com/robotology/icub-support/issues/407.
I would recommend to keep any request about you need of FW update in the the above issue.
But if it can help, I can already tell you that no FW update of CAN boards is required on your robot, because iCubNottingham01  was delivered around August 2016 and since then we did not release any fundamental FW for CAN boards.
If by mistake you wrongly mis-programmed some board I am available to help you to restore the status quo ante inside the above icub-support issue.

Hi @pedrombmachado, I close the issue for now and I will help you for any specific problem you may have in the relevant issue http://wiki.icub.org/wiki/Firmware#Updating_the_FW_of_the_ETH_boards.


Entity cannot be added a second time to an entity platform

In recent days, some devices connected to the Xiaomi multi-mode gateway through ZigBee have reported the above errors. Please help to find out the reason.
Every time homeassistant restarts, the devices (such as temperature and humidity sensors and door magnetic sensors) connected to Xiaomi multi-mode gateway through BLE are displayed as unavailable. I wonder whether it is possible to record the status of the sensors before homeassistant restarts and assign values after restart?
My English is not very good, the expression may not be very clear and accurate, please understand!
Looking forward to your reply and solution. Thank you so much!

I had some problems with my BLE devices about a month ago. After the reboot, the devices are unavailable. Then I kind of deleted all the devices and registered again. I noticed that most of the devices have changed the token! Out of 10 devices, 8 received a new token. It looks like Xiaomi changed the tokens. Only 2 devices connected to Xiaomi last had the same token.
I hope this will be useful to you

大约一个月前，我的 BLE 设备出现了一些问题。重启后，设备不可用。然后我删除了所有设备并重新注册。我注意到大多数设备都更改了令牌！在 10 台设备中，有 8 台收到了新令牌。看起来小米改变了代币。最后连接到小米的只有 2 台设备具有相同的令牌。 我希望这对你有用

Actually, when state of ble devices changed, it will shows new state on HA, but it will always show unavailable/unknown until state changed after HA restart

Are you using supported gateway firmware?

Please reopen the issue if it is still relevant

I  recently bought a new gateway, the previous one is the multimode gateway (1.5.4_xxxx) and the new one is multimode gateway 2 (1.0.3_xxxx) , then I added the new one to HA.
The issue is all entities in multimode gateway 2 turned to unavailable in a few minutes, or if I try reload the integration, the entities became unavailable  immediately. Then I tried  to remove the new one from HA, but the problem then occurred in old gateway.
Here is the sample error logs.

2023-12-27 16:28:01
stdout
homeassistant.exceptions.HomeAssistantError: Entity 'sensor.balcony_humidity_gw3' cannot be added a second time to an entity platform [0m

2023-12-27 16:28:01
stdout
raise HomeAssistantError(

2023-12-27 16:28:01
stdout
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 993, in add_to_platform_start

2023-12-27 16:28:01
stdout
super().add_to_platform_start(hass, platform, parallel_updates)

2023-12-27 16:28:01
stdout
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 215, in add_to_platform_start

2023-12-27 16:28:01
stdout
entity.add_to_platform_start(

2023-12-27 16:28:01
stdout
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 565, in _async_add_entity

2023-12-27 16:28:01
stdout
await asyncio.gather(*tasks)

2023-12-27 16:28:01
stdout
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 509, in async_add_entities

2023-12-27 16:28:01
stdout
Traceback (most recent call last):

2023-12-27 16:28:01
stdout
[31m2023-12-28 00:28:01.565 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved

2023-12-27 16:28:01
stdout
homeassistant.exceptions.HomeAssistantError: Entity 'sensor.balcony_temperature_gw3' cannot be added a second time to an entity platform [0m

After a HA restart, the entities are back but will be gone in few minutes.

Logger: homeassistant.components.sensor
Source: helpers/entity_platform.py:533
Integration: Сенсор (documentation, issues)
First occurred: 9 февраля 2024 г. в 19:26:38 (40 occurrences)
Last logged: 19:47:02
Error adding entities for domain sensor with platform xiaomi_gateway3
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 533, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 589, in _async_add_entity
entity.add_to_platform_start(
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 240, in add_to_platform_start
super().add_to_platform_start(hass, platform, parallel_updates)
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1293, in add_to_platform_start
raise HomeAssistantError(
homeassistant.exceptions.HomeAssistantError: Entity 'sensor.a4c138b9eecb_temperature' cannot be added a second time to an entity platform

#1090

Logger: homeassistant.components.binary_sensor
Source: helpers/entity_platform.py:533
Integration: 二元传感器 (documentation, issues)
First occurred: 19:36:31 (6 occurrences)
Last logged: 19:36:32
Error adding entities for domain binary_sensor with platform xiaomi_gateway3
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 533, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 589, in _async_add_entity
entity.add_to_platform_start(
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1293, in add_to_platform_start
raise HomeAssistantError(
homeassistant.exceptions.HomeAssistantError: Entity 'binary_sensor.xxxxx_gateway' cannot be added a second time to an entity platform

In the normal state, I disabled the entity first and then enabled it. At this time, the gateway and all entities on it are unavailable, and you will see the log above.

Fixed https://github.com/AlexxIT/XiaomiGateway3/releases/tag/v4.0.0


Improved image SRC regex.

Previously even things like <m c="a would match, with the "SRC" being "a". This fixes a lot of the mistakes, but it's not perfect (shouldn't be parsing HTML with regex...).

Thanks


TIKA-3963

Thanks for your contribution to Apache Tika! Your help is appreciated!
Before opening the pull request, please verify that

there is an open issue on the Tika issue tracker which describes the problem or the improvement. We cannot accept pull requests without an issue because the change wouldn't be listed in the release notes.
the issue ID (TIKA-XXXX)

is referenced in the title of the pull request
and placed in front of your commit messages surrounded by square brackets ([TIKA-XXXX] Issue or pull request title)

commits are squashed into a single one (or few commits for larger changes)
Tika is successfully built and unit tests pass by running mvn clean test
there should be no conflicts when merging the pull request branch into the recent main branch. If there are conflicts, please try to rebase the pull request branch on top of a freshly pulled main branch
if you add new module that downstream users will depend upon add it to relevant group in tika-bom/pom.xml.

We will be able to faster integrate your pull request if these conditions are met. If you have any questions how to fix your problem or about using Tika in general, please sign up for the Tika mailing list. Thanks!

Y, good point.  I just added processing to prefer the  element over the title attribute.  If this is backwards, we can flip the logic, but we at least have a sensitivity to this issue in our code and a unit test. :D


Check for conversion compiler warnings

Exclude them from Werror on CI runs though, cause the ubuntu bison/flex versions do not generate warning free code

Looks fine.  Want to run it on fedora before merging, but I'm on my Ubuntu box right now.


Data plotted outside plot area

Not sure if this is a Ferret issue or LAS Ferret scripts in SOCAT issue.
Salinity exists for only the first few points in the SOCAT dataset 33BI20160827.
33BI20160827.nc.zip
With the latest SOCAT LAS using the latest Ferret (v7.3.1), the salinity thumbnail, as well as the correlation viewer plot, has a point that is plotted outside the plot area.

In older SOCAT LAS using Ferret 7.3, the salinity plot is within the plot and actually has some extra whitespace around the plot.

Just realized that most of the new thumbnail plots do not have Y-axis labels.  (Some not shown here do have the fCO2_recommended label.)  Not sure if that was intentional or not.  Since the hover help explains the plot, figuring the lack of labels is not a major problem.

This is all in the Ferret scripts for LAS - not particularly specific to SOCAT, but just in setting up thumbnail and property-property plots with time as the horizontal axis.
The symbol getting plotted outside the plot box is due to an extra short time axis, as the Salinity variable for this trajectory only has a few valid data points; the time axis being defined for an underlay plot that draws the axes was not quite right.
The vertical axes should be labeled, just as they are for non-time plots - the thumbnails have one plot that's fco2_rec as a function of temperature, and that has labels.  I have added those labels back to the plots.
All of this is checked in to git, so will need to be gotten over to svn and the socat_latest branch.


[WFLY-18316] Implement canonical links for WildFly documentation

https://issues.redhat.com/browse/WFLY-18316
To implement support for a link of the form docs.wildlfy.com/latest/, I've updated the Maven build with a Python script. This script will handle the necessary renaming and reorganization of files when upgrading to a new release. More details on how the script works are included in the docs README.

Hello, jessicarod7. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.


Syntax of the if statements on the sh scripts

I'm having problems running the pipeline because of the syntax of the if statements.
I run this command:
sh /home/eric/CUT-RUNTools-2.0/run_bulkModule.sh /home/eric/CUT-RUNTools-2.0/config/bulk-config.json PRO_2-4_IgG_S6_L001
And I get several warnings like this ones:
/home/eric/CUT-RUNTools-2.0/run_bulkModule.sh: 42: [: Truseq: unexpected operator
/home/eric/CUT-RUNTools-2.0/run_bulkModule.sh: 46: [: Truseq: unexpected operator

This does not stop the execution, but it ignores a lot of the options specified in the config file. The problem seems to be with the syntax of the if statements, that gives an error and evaluates as False, ignoring the config file.
I solved this by rewriting the if statements. Right now they are written as:
if [ "$experiment_type" == "CUT&Tag" ]

Instead, I used (without the double =):
if [ "$experiment_type" = "CUT&Tag" ]

Alternatively I think that this would work as well (double [ and no $):
if [[ $experiment_type == "CUT&Tag" ]]

Updating all the if statements of run_bulkModule.sh and src/bulk/bulk-pipeline.sh solved the issue for me. I don't know if I triggered this issue by doing something wrong, but thought I would post it in case you think it's worth to update the code.

Thank you so much for reporting the bug and sharing the solution!!
I will keep this in case other users run into the same issue, thank you!


Login button into Navbar for mobile

I like the sign-out button but I think the sign-in button shouldn't be centered at the bottom of the list. Maybe put it at the top next to menu or move it to the left?

I think it looks fine. The top part of the navbar looks a little awkward but it isn't a big deal. Also I'm not a huge fan of the plane icon for sign-out!


Exception : No MaterialLocalizations found. Trying to display AlertDialog

Trying to display Alert Dialog by calling a function from the parent widget.
Here's the code :
void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: RaisedButton(
            onPressed: () {
              popUp(context);
            },
            child: Text("Click"),
          ),
        ),
      ),
    );
  }

  void popUp(BuildContext context) {
    showDialog(
      context: context,
      builder: (BuildContext context) {
        return AlertDialog(title: Text("ok"),);
      },
    );
  }
}

But getting the following Exception :
I/flutter ( 4041): ══╡ EXCEPTION CAUGHT BY GESTURE ╞═══════════════════════════════════════════════════════════════════
I/flutter ( 4041): The following assertion was thrown while handling a gesture:
I/flutter ( 4041): No MaterialLocalizations found.
I/flutter ( 4041): MyApp widgets require MaterialLocalizations to be provided by a Localizations widget ancestor.
I/flutter ( 4041): Localizations are used to generate many different messages, labels,and abbreviations which are used
I/flutter ( 4041): by the material library.
I/flutter ( 4041): To introduce a MaterialLocalizations, either use a  MaterialApp at the root of your application to
I/flutter ( 4041): include them automatically, or add a Localization widget with a MaterialLocalizations delegate.
I/flutter ( 4041): The specific widget that could not find a MaterialLocalizations ancestor was:
I/flutter ( 4041):   MyApp
I/flutter ( 4041): The ancestors of this widget were:
I/flutter ( 4041):   [root]

It reads "use a  MaterialApp at the root of your application" which I have already done. What am I doing wrong here?

same problem, how to solve?


(PDB-5128) Decrease round trips and upsert to improve cmd processing

Might want to look at the commits individually.


Update publish-jekyll.md

This PR implement #68123 and solve #65318 , #65867 and #67838

@ncarandini : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ncarandini : Thanks for your contribution! The author(s) have been notified to review your proposed change.

This PR now has a merge conflict due to another PR making the same change. I am going to #please-close this PR now.

This PR now has a merge conflict due to another PR making the same change. I am going to #please-close this PR now.


[BUG] Delay around 0.4s with GestureDetector in PhotoView.customChild

Describe the bug
GestureDetector inside PhotoView.customChild detects taps with delay of around 0.4s which is quite noticeable. I try to use Listener instead of GestureDetector and it seems it works good, but gestures is useful also for my needs.
To Reproduce
Paste this code in the default flutter project in main.dart, tap on the screen and look at the output.
I tested it in web and windows builds.
import 'package:flutter/material.dart';
import 'package:photo_view/photo_view.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);
  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    DateTime timeGestureStart;
    DateTime timeListenerStart;
    Duration diff;

    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: PhotoView.customChild(
        child: GestureDetector(
          onTap: () {
            timeGestureStart = DateTime.now();
            print('GestureDetector');
            diff = timeGestureStart.difference(timeListenerStart);
            print('diff ' + diff.toString());
          },
          child: Listener(
            onPointerDown: (PointerDownEvent event) {
              timeListenerStart = DateTime.now();
              print('Listener');
            },
            child: Container(
              color: Colors.greenAccent,
              child: Text('Sample text'),
            ),
          ),
        ),
      ),
    );
  }
}

What is the current behavior?
Listener and GestureDetector works with delay between each other.
Here is calculated difference in this code as a Duration:
diff 0:00:00.388052
Expected behavior
Expected to see no delay in general and working of the GestureDetector and Listener at the same time.
Screenshots

Additional context
It seems it's similar to this issue #233
Maybe there is a sort of gesture disambiguation problem.
Logs
photo_view: ^0.9.2
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel master, 1.19.0-2.0.pre.193, on Microsoft Windows [Version 10.0.17763.1217], locale ru-RU)
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
X Android license status unknown.
Try re-installing or updating your Android SDK Manager.
See https://developer.android.com/studio/#downloads or visit visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions.
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio CommunityВ 2019 16.4.2)
[√] Android Studio (version 3.6)
[√] VS Code (version 1.43.2)
[√] Connected device (3 available)

I think this is due to the fact that the package expects a double tap, but the double tap function cannot be disabled. I did not find how to do it

We push the image to full screen on tap and the delay is noticeable. Hopefully this can be solved in the package itself somehow, instead of finding some hack to fix it :)

@renancaraujo
When will this issue can be fixed？
Can this problem be solved by turning off the double click event by parameter?  Double-click is generally not used. When viewing full screen, it is often used to close the page with a single click.

Can this problem be solved by turning off the double click event by parameter
That would be a great way to approach this. But I am afraid that photoview is growing to receive too many paremeters.

We could group all params related to double tap into one that would describe the double tap behavior with and option to "not accept double taps". Then the delays would be gone. What do you think?

When will it be update? @renancaraujo

Any updates here?

@renancaraujo

@renancaraujo

@renancaraujo


Test HandleVnodeOperation

This incorporates

Updates by @pmj to allow parameter testing
Tests for HandleVnodeOperation

Also highlighting two areas we should unit test in the future

CurrentProcessWasSpawnedByRegularUser
UseMainForkIfNamedStream

/azp run GitHub VFSForGit Mac Functional Tests


Stops syncing

Installed mynode via Virtual Machine OVA  community edition 0.2.16 and constantly chain stop syncing, after restarting node starts sync but after a while stops again. Did not change anything just installed on Oracle VM Virtual Box. There is no error.

Does the bitcoin log on the status page show any helpful information about why it may be stopping? No peers?
You can toggle tor as well. That could help. I would also suggest upgrading to the latest release.

Oigo


Distinguish between tool call generations and final output streams with Anthropic chat models

Currently, there's no good guidance on distinguishing between an Anthropic model's generated tool call vs. a plain text output.
This makes e.g. streaming back final agent responses difficult/hacky.
See: https://github.com/langchain-ai/langgraphjs/issues/310

Hi @jacoblee93, Is there any progress on this bug?
I spent some time checking the code and noticed that the behavior of the OpenAI(@langchain/openai), Anthropic(@langchain/anthropic), and some other integration packages seems to differ. Could you help clarify what the standard behavior should be?

In the OpenAI package, when handleLLMNewToken is called, a chunk is passed, but in the Anthropic and other packages, a chunk is not passed.
https://github.com/langchain-ai/langchainjs/blob/42631e4e4d83f0f5c2f9fed9893e9fd081eaca30/libs/langchain-openai/src/chat_models.ts#L1169-L1177
https://github.com/langchain-ai/langchainjs/blob/42631e4e4d83f0f5c2f9fed9893e9fd081eaca30/libs/langchain-anthropic/src/chat_models.ts#L821

In the OpenAI package, when the model streams out the tool_call parameter, the token received by handleLLMNewToken is an empty string, whereas in the Anthropic package, handleLLMNewToken does receive the tool_call parameter token.

When receiving the tool_call parameter, should the on_chat_model_stream event be triggered?

Any update @jacoblee93 ? The @renxinyan suggestion would solve the problem when using BaseCallbackHandler

Workaround with a Monkey Patch on _streamResponseChunks
// 3rd's
import { ChatAnthropicMessages } from '@langchain/anthropic';
import { BaseMessage } from '@langchain/core/messages';
import { ChatGenerationChunk } from '@langchain/core/outputs';
import { CallbackManagerForLLMRun } from '@langchain/core/callbacks/manager';

const originalStreamResponseChunks = ChatAnthropicMessages.prototype._streamResponseChunks;

ChatAnthropicMessages.prototype._streamResponseChunks = async function* (
  messages: BaseMessage[],
  options: ChatAnthropicMessages["ParsedCallOptions"],
  runManager?: CallbackManagerForLLMRun
): AsyncGenerator<ChatGenerationChunk> {
  const generator = originalStreamResponseChunks.call(this, messages, options, runManager);

  for await (const chunk of generator) {
    await runManager?.handleLLMNewToken(
      chunk.token,
      undefined,
      undefined,
      undefined,
      undefined,
      { chunk }
    );
    yield chunk;
  }
};

To implement it you can use something like this as the CallbackHandler:
// 3rd's
import { BaseCallbackHandler } from "@langchain/core/callbacks/base";
import { NewTokenIndices } from "@langchain/core/callbacks/base";
import { HandleLLMNewTokenCallbackFields } from "@langchain/core/callbacks/base";

export class StreamCallbackHandler extends BaseCallbackHandler {

  name = "stream_callback_handler"

  constructor() {
    super();
  }

  public async handleLLMNewToken?(
    token: string,
    idx: NewTokenIndices,
    runId: string,
    parentRunId?: string,
    tags?: string[],
    fields?: HandleLLMNewTokenCallbackFields
    ) {
    const isOriginalImplementation = !fields,
      // @ts-ignore
      isToolCall = fields && fields.chunk?.message?.tool_call_chunks?.length > 0;
    if (isOriginalImplementation || isToolCall) return;

    return fields.chunk.text;
  }
}

And then:
const stream = await chain.stream(
  { input: "Hi! How are you?" },
  { callbacks: [new StreamCallbackHandler()] }
);

Leaving open


Error installing on linux VPS

Hi, I have a problem:
`npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node deps/install
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2018-03-29T15_00_54_668Z-debug.log`
I can give you the full log if needed
None of the methods at troubleshooting guide helped me (I think)
If you know what can I do to fix this you can answer here or at discord: i do read sometimes#7594
Thanks!

I'm experiencing the same issue.
node: v6.12.3
npm: v3.10.10

@dndrbots - Not sure if you're still having issues, but running the install with --verbose helped me better identify the issue.  It turned out that I wasn't setting the environment LZZ_COMPAT=1 correctly.


Refactor Rack dependency

Grape is in many parts depentend from Rack. Like this post show's Racks future is uncertain. In my point of view grape should plan ahead and decouple from Rack as much as possible.

I like the progress http://hanamirb.org has been making, would love to compare, benchmark and potentially support both Rack and Hanami.

@dblock I thought Hanami is a Rack-based framework. Are they providing an alternative low-level abstraction as well?

You're right @ch1c0t, I meant https://github.com/hanami/router.

AFAIK, hanami router has depended on http_router, and it's thread unsafe.
So I've implemented new router by using the Mustermann gem.
https://github.com/namusyaka/pendragon
This has been merged into Padrino Framework.

@namusyaka :+1: you should give it a try in Grape - replacing the legacy unmaintained libraries with faster newer better implementations is high on my list.

Yeah, I'm going to replace the router.

We replaced the router in https://github.com/ruby-grape/grape/pull/1276. It's a step forward!


Weird data sync problem

While I am developing custom app, I faced some weird behavior of data sync. I found it might be related to object property names, like "next".  Because it works with other names.
I was able to modify hello world to demonstrate this. Next program syncs first character but then stops. What can be the reason?
<!DOCTYPE html>
<style>html, body, textarea { width: 100%; height: 100%; padding: 0; margin: 0; }</style>
<textarea id="paste" placeholder="paste here!"></textarea>
<script src="https://cdn.jsdelivr.net/npm/gun/gun.js"></script><script>
  gun = Gun(['https://gun-manhattan.herokuapp.com/gun']);
  copy = gun.get('test').get('paste947');
  paste.oninput = () => { copy.put({serving:paste.value, next:'test'}) };
  copy.on((data) => { paste.value = data.serving });
</script>

Thanks.

this is the official repository of the documentation platform, the problem you report seems to be directly related to Gun. https://github.com/amark/gun


added diagnostics and removed code lens

The Vs-code extension now uses the diagnostics feature in order to display the outputs and suggestions, this code works on existing API response :

Sample response :
{
  "style_guide" {
    "issues" : [
      {
        "line": 1,
        "message": "IndentationError: unexpected indent.  At least one space needed after 'def' keyword."
      },
      {
        "line": 2,
        "message": "IndentationError: unexpected indent.  Docstring content should start at 4 spaces indentation."
      },
      {
        "line": 4,
        "message": "IndentationError: unexpected indent.  Code inside function should be indented by 4 spaces."
      },
      {
        "line": 6,
        "message": "IndentationError: unindent does not match any outer indentation level."
      }
    ]
  }
}

[x] Need to update the API response so that it only highlights the specific keyword not the line

The highlighting of specific keywords works fine, but the the start_char and end_char seems not to be accurate. Due to this it's highlighting wrong places


[loc][Query][DevDiv] Pelase clarify "Balance"

Hi, we have a query from our translator. Can you help pleasE? Thanks!
Source:
Emmet: Balance (inward)
File Name:
src\vs\workbench\parts\emmet\node\actions\balance.nls.resx
Resource ID:
".resx", 0; "balanceInward"
Query:
What exactly is meant by "balance"?  	DE_Translator2	7/21/2016 9:53:34 PM

Balancing is meant in the sense that after executing the action you have a matching tag on both sides of the selection.
This is the full explanation.
http://docs.emmet.io/actions/match-pair/


CrtHttpEngine leaks connections on cancellations

Describe the bug
CrtHttpEngine currently tries to guarantee a connection is always closed with an invokeOnCompletion call, which tells the ResponseHandler to complete, which in turn closes the connection:
val conn = withTimeoutOrNull(config.connectionAcquireTimeout) {
            manager.acquireConnection()
} ?: throw ClientException("timed out waiting for an HTTP connection to be acquired from the pool")

val respHandler = SdkStreamResponseHandler(conn)
callContext.job.invokeOnCompletion {
    logger.trace { "completing handler; cause=$it" }
    // ensures the stream is driven to completion regardless of what the downstream consumer does
    respHandler.complete()
}

This implementation would be fine if there weren't any suspension points which could interrupt the coroutine between manager.acquireConnection() and job.invokeOnCompletion. But if you look closely at the definition of manager.acquireConnection():
public actual suspend fun acquireConnection(): HttpClientConnection {
        val connFuture = jniManager.acquireConnection()
        val jniConn = connFuture.await()
        return HttpClientConnectionJVM(jniConn)
}

The connection request is made at jniManager.acquireConnection(). This is a hot future, whether the connection is taken or not it has been already been requested from the pool.
The problem is that awaiting for the connection itself IS a suspending point, and that allows the coroutine itself to be cancelled before the invokeOnCompletion hook is installed.
Expected behavior
Canceling an AWS SDK request when it's waiting for connections should always return the connection.
Current behavior
Cancelling coroutines running AWS requests may leak the connection and never return it to the pool.
Further requests after such leaks fail with
aws.sdk.kotlin.runtime.ClientException: timed out waiting for an HTTP connection to be acquired from the pool
 at aws.sdk.kotlin.runtime.http.engine.crt.CrtHttpEngine.roundTrip(CrtHttpEngine.kt:85)
  at aws.sdk.kotlin.runtime.http.engine.crt.CrtHttpEngine$roundTrip$1.invokeSuspend(CrtHttpEngine.kt)
  at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
  at kotlinx.coroutines.internal.ScopeCoroutine.afterResume(Scopes.kt:33)
  at kotlinx.coroutines.AbstractCoroutine.resumeWith(AbstractCoroutine.kt:102)
  at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:46)
  at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:104)
  at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:571)
  at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:750)
  at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:678)
  at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:665)

Steps to Reproduce
In a simple Spring Boot project:
@Configuration
class DynamoDbConfig {
    @Bean fun ddbClient() = DynamoDbClient {}
}

@Component
class SomeDao(val dynamo: DynamoDbClient) {
   val random = Random()
    suspend fun putItem() {
        dynamo.putItem {
            tableName = "some-table"
            item = mapOf("SomeKey" to AttributeValue.S("some-value-${random.nextInt()}" ))
        }
    }
}

@RestController
@RequestMapping("/api/put-items")
class SomeResource(val someDao: SomeDao) {
    @PostMapping
    suspend fun putItems() {
        log.info("Persisting items...")
        coroutineScope { // This block will await untill all items are persisted
            repeat(100) {
                log.info("Launching persistItem $it...")
                launch { someDao.putItem() }
                log.info("Launched persistItem $it.")
                throw RuntimeException("Some random exception which will cancel all awaiting coroutines in scope")
            }
            log.info("Awaiting for persistItem results...")
        }
        log.info("All items persisted")
    }
}

Steps to reproduce:

Request POST /api/put-items once.
This will launch 100 PutItem requests to the dynamo client and await for them all.
The first finishing request will throw an exception. The coroutineScope will cancel all awaiting operations (due to Structured Concurrency
Request POST /api/put-items once again (or twice more)
The request will block waiting for connections until it times out. No connection should be available as they weren't returned to the pool.

Possible Solution
The easiest solution I see is wrapping the whole block that acquires connections in a withContext(NonCancellable) { ... } block. But this makes every request now required to at least wait for the connection before returning.
Preferably, a better way to notify the jniManager the connection wasn't used needs to be found.
Context
This makes the use of Structured Concurrency very hard.
I want to be able to put multiple items in Dynamo at once, so launching it all in a scope seems like the obvious choice. But now that makes me worried one exception in a single coroutine could make all the others awaiting leak their connections.
AWS Kotlin SDK version used
0.11.0-beta
Platform (JVM/JS/Native)
JVM
Operating System and version
macOS BigSur 11.6.2

@tavomaciel Thanks for the detailed report and explanation! We will take a look at getting this fixed. Using NonCancellable is (as you pointed out) probably not the right solution.


Bug: Elasticsearch doesn't initialize with --tail mode

When passing --tail in config to import for Elasticsearch, it doesn't initialize with --tail mode.

This has been fixe in the 0.6.6 release.


Cloud Services

is there anyway to make it so then there are cloud services? You could probably do this by using scratch accounts that nobody wants or needs and you could use these accounts to add mobile cloud expierience. This would also help on games that save on the cloud or multiplayer games, such as griffpatch's Scratcheria series.

I meant to put the title as 'Cloud Services'

I also just thought of something else, if the project contains a 'Username Block' couldn't you make it so then it puts your Public IP instead because then you could host basicly a small mini server on your computer if your the first one on

But you would need to port forward before this

You can get cloud data from a project in JSON using::
https://scratch.mit.edu/varserver/[project id]

What about using the cloud variables that Blue uses and put the project id in front of each request making each unique


Fix lazy hydration for Dict, Queue, Volume, NetworkFileSystem

This should fix most, if not all, hydration issues that users have had lately.
Looking into other object types too, and they should be pretty straight forward imo
As a shortcut I just inject the Client.from_env() client where a client was currently missing and not readily accessible, which feels a bit like a crutch. Preferably, we should find a nice way to inject a client directly in from_name() (mostly to make tests cleaner)

Great, ship it!!
I briefly looked at the same thing a couple of weeks ago and ran into the same issue with the client dependency. I think we should add an optional client to .from_name. This would decouple hydration from setting the client which maybe changes a few things a bit? However I think it would make tests a bit cleaner.

Btw it will be a bit more annoying to solve this for Cls than other object types because of how the constructor needs to be synchronous. We'll have to figure this out.


How is the Progress Indicator calculated?

Progress on psvtrophyisgood, absolutely everything works trophys sync online and everything, however theres an issue and thats that the progress bar on the trophy set doesnt update to respect the new trophys.
for example, it can show 0% and have all trophys unlocked when you click on it,
PSN doesnt care about this and will sync it anyway, but id like to fix it regardless.
i found the offset in TRPTITLE that controls this data, but i have no idea how it is calculated besides that
FF FF FF FF is 100%.
How is this calculated on ps3?

I found some code in my project.
unlock
trophyListInfo.AchievementRate[id / 32] |= (uint)(1 << id).ChangeEndian();
AchievementRate[id / 32] |= (uint)(1 << id);

lock
trophyListInfo.AchievementRate[id / 32] &= 0xFFFFFFFF ^ (uint)(1 << id).ChangeEndian();
AchievementRate[id / 32] &= 0xFFFFFFFF ^ (uint)(1 << id);

Hope this helps.

thanks, the changeEndain part isnt needed for me . but aside from that this is helpfull


Errors Reported by Jon Pearce

[x] page 80: Broken links "[JSON][linkjson]" and "[BNF][link-bnf]" in Section <IP_ADDRESS> JSON
[x] page 86: "Empty()" should be replaced with "End()"
[x] page 214: "benefited from fold as we remove the duplication" should read "benefited from fold as we remove[d] the duplication"
[x] page 100: "final case object EmptyA extends Maybe[A]" should read "final case class EmptyA extends Maybe[A]"; this correction has been made in the subsequent code listing
[x] page 100: Two instances of broken links/italics at the very begining of Section 5.5.3 in "[ˆmonads]"
[x] page 101: In "val list: LinkedList[Int] = Pair(1, Pair(2, Pair(3, Empty)))", Empty should be "EndInt"
[x] Page 121: The code listing at the top of the page contains two duplicate lines i.e. one instance of each of "var mutable = 1" and "mutable = 2" should be deleted
[x] Page 124: In the definition of getOrElse for None, the compiler reports the error " error: not found: type A"
[x] Page 143: In "method fromLessThan defined the companion object", add "in" before "the companion object"
[x] Page 151: Chanage "mne" to "men" in "implemnetation"
[x] Page 162: Identifier "res4" isn't consistent with the name ("res") shown in the preceding output
[x] Page 162: Five instances of "val" may be deleted from the case class definitions of "User" and "Anonymous"
[x] Page 181: In B.6.2.1, one of the two instances of "import scala.collection.mutable" may be deleted
[x] Page 222: "Some(123).isEMpty" should read "Some(123).isEmpty"
[x] Page 225: In "allFilms.foldLeft(0)((a, b) => a.imdbRating + b.imdbRating) / films.length", "0" should be replaced with "0.0" (Double) and "a.imdbRating" should be replaced with "a" (Double, not Film)
[x] Page 226: "We these three pieces we can solve the problem" should be "With these three pieces..."
[ ] Page 229: In C.5.24, one may need to replace "${director.name}" with "${director.firstName} ${director.lastName}"; in addition, the compiler complains with "error: illegal start of simple pattern" if the comprehension is not assigned to a val (which gets type Unit)
[x] Page 244: Something is wrong with the mark-up tags of the code given for "object Positive"

I've addressed all of these except pave 229 in a soon-to-be-merged branch (feature/bugfixes). I believe the page 229 one is because I defined two different versions of Director and Film. The second version has def name on it but readers might end using the first version for this exercise. I think there's a bigger problem here than just the name of that method, but I'm going to merge the low hanging fruit here and factor that one issue out into its own thing.


[KYUUBI #526][BRANCH-1.1] Support kyuubi.operation.scheduler.pool for queries

backport #528 for branch-1.1

travis passed. Merging to branch-1.1.


styles are still broken on project cards

Description

Gitea Version
trunk
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
docker
Database
None

Which commit hash are you on exactly? https://github.com/go-gitea/gitea/pull/30687 should have fixed it.

564102c

Link to try.gitea.io with repro?

https://try.gitea.io/stuzer05/test-release/projects/5401

->  Fix issue card layout #30800


Why isn't Node.js a stack option in General Settings?

There are just these:

However, creating a Node.js app is one of the quickstarts

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

ID: 1e25899c-1ed8-7cfc-2c37-73bb010a87f0
Version Independent ID: 8b366c42-43f8-d5eb-b619-b356e75a9916
Content: Configure apps in the portal - Azure App Service
Content Source: articles/app-service/configure-common.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@v-kydela Thanks for the feedback! We are currently investigating and will update you shortly.

@v-kydela, Sorry for the delay Here. We have heard from our internal team that For Windows Apps, the node version is set in the 'Application Settings' tab (as an App Setting - 'WEBSITE_NODE_DEFAULT_VERSION') rather than the stacks dropdown in the 'General Settings' tab. And there is a plan in the future to remove this confusion.
I hope this answers your question. So closing this thread for now, If there are any further questions regarding this matter, please tag me in your reply and I will be glad to continue the discussion with you.

@MonikaReddy-MSFT - Can you clarify that a bit? I wasn't asking how to set the Node version, I was asking how to select Node as a stack. Are you saying if a Node version is present in the Application Settings tab then Node is automatically selected as the stack and the stack dropdown in the general settings is ignored?

@v-kydela , I am sorry for not clarifying it clearly, and yes you are correct. Once you select the Node in Runtime Stack while creating the WebApp, it should be automatically selected as the stack and the actual stack dropdown is ignored for now. As I mentioned earlier, the Product team is aware of it and they are planning to clear this confusion in the future.

I hope this clarifies your question.  Let me know if you have any further questions regarding this matter.

This was mark as Resolved even though the issue is still here and running Node on Windows machines is a nightmare...

I've just experienced similar confusion. In particulare as I am trying to configure Application Insights but the runtime for Node.js is not being picked up.

I am confused by this page then.
https://docs.microsoft.com/en-us/azure/azure-monitor/app/azure-web-apps-nodejs?tabs=windows

this is still an issue when setting up CI with Github in the deployment slot.
I'm trying to set up a Github Action for an auto-deployment on commit, but the detected stack is .Net, even though I have a WEBSITE_NODE_DEFAULT_VERSION variable set.
As a result, the only option is to write the workflow script manually.


wacs.exe error saving certificate to store access deny

Before you post
First make sure that your question is not already answered by the documentation.
Manual: https://www.win-acme.com/manual/getting-started
Reference: https://www.win-acme.com/reference/cli
Then make sure to search through the issue list to check if others have asked a similar question already. If you find an existing question without a satisfying answer, make your voice heard there.
Why
What are you trying to achieve and for what reason?
How
What is your intuition about the solution?
Documentation improvements
Please suggest which part of the documentation could be added or improved.

Please provide some log output at least.


Ignore result data

#406 would unnecessarily reload pages if show= was toggled.
The checking function compares pages but the new page will be loading while the old one has results already. Hence also ignore `resultz state.
Sorry @ncfavier to have another PR on the same issue

Yes ill have a look later and merge it myself then 👍🏼

Actually I just realised how annoying this issue is 😅 merging now

Yes 😅
Thanks
Happy new year

Yes 😅
Thanks
Happy new year


Feature request: A new setting to sort JSON keys when formatting a JSON file with "Format Document"

Hey,
I would like to request a new setting like "json.format.sort_keys" (false by default).  When set to true, it should make the formatter built into VS Code (triggered by the "Format Document" command in a file with Language Mode set to JSON) sort JSON keys alphabetically.
Illustration
Original JSON:
{"b":2,"a":1}

After "Format Document", current behavior (*subject to indentation preferences):
{
    "b": 2,
    "a": 1
}

Proposed behavior with "json.format.sort_keys": true:
{
    "a": 1,
    "b": 2
}

Additional considerations

Keys of nested objects should be sorted, too.

Use case
I sometimes use JSON formatting to normalize JSON configurations obtained from several installations of a particular software, in order to diff them. It would tremendously help if keys were sorted right away. Now I have to use an external tool for that.
References:
Version information:

VS Code 1.68.0

My settings.json:
{
  "[json]": {
    "editor.defaultFormatter": "vscode.json-language-features"
  }
}

In the use case I gave as an example, key order is arbitrary in the original files. It makes diffs unreadable. When keys are sorted, the structure of the file is consistent and it is easy to see which keys were added, which were removed and which had respective values modified.
Personally I never needed to sort properties by value. Also, once VS Code allows to sort properties by keys OR values, I assume people would want to sort some objects by key and other objects by value etc. In my opinion, this will make things unnecessary complicated. If you ask me, I would advise to start with sorting by keys only. Other kinds of sorting can be added later if needed.

Well, I see, in that particular use case, I can see the advantage of sorting keys.  Once sorting keys has been implemented then it's a matter of time before more devs want to sort both. I think that the best course of action for a single feature would be the sorting of both keys and their respective values combined as an individual value.

Honestly, I don't see a point of combining keys with values because normally (in 99.999% cases, I presume) one doesn't have duplicate keys.

They might be related information from the key to its value, for instance in a recent project that I have been working on they is an error message key that contains its message as a value. If I wanted to sort that JSON data, It would not be helpful to split that key from its value

I assume the result should be
{
    "a": 1,
    "b": 2
}

@aeschli , Yes, of course, thank you for noticing. I edited the description.

IMO a formatter should only add or remove whitespaces (space, new lines), and not remove, add or reorder tokens.
But the requested functionality makes sense and can be provided by a separate command.
There's already an extension that does it: https://marketplace.visualstudio.com/items?itemName=richie5um2.vscode-sort-json
Given that I don't think this should be part of the formatter, I close the issue as won't fix


Removing gctheme.css from splash styles

@ahmad-shahid Same unsigned commit problem   (weird that it only warns us AFTER I approve the changes)

Good to go!


Data download and setting environment

@gwaybio Hey Greg! This pull request is now ready for review.

i see the .py file, but I don't see the updates in the environment.yml. I'll take a look at the .py file now!


npm run clean : windows support

This command works:
DEL /s /q dist
Is there anyway to change package.json so the script uses DEL or rm depending on platform?

there are some common solutions to this problem, e.g. rimraf package which I would prefer rather that implementing own solution

I used the solution suggested by @piotrwitek :

run yarn add rimraf --dev
change the package.json to replace the "clean" scripts element with "clean": "rimraf dist"

This seems to make things work on Windows. The only small annoyance is that an extra Window showing the inner node process output shows up. Haven't been able to track down why yet...

Instructions Unclear, rimraf-ed my entire hard drive.
Seriously now, this seems like an improvement to the template with no downsides.


Enable gocritic linter

This is a proposition, not sure we should merge it.
It catches errors like capitalized variable name and switch with default case only.
On the other hand, it forces us to rewrite some if-else statements with switchs.

/lgtm


apps/cmp.c and APP_HTTP_TLS_INFO: Fix use-after-free and add proper free() func

This came up while preparing the HTTP client for an implementation of #14004.

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.


Remove internal comment so as to not populate end user code

Context: https://weightsandbiases.slack.com/archives/C072S236A75/p1733264848842839

Closing per Internal Slack: https://weightsandbiases.slack.com/archives/C072S236A75/p1733758304928649?thread_ts=1733264848.842839&cid=C072S236A75


How do I make a warning note?

Describe your problem here.

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
I've added a few custom characters and some songs

use hank accelerant notes from psych engine port

how do I make the notes bigger?

draw the image and note get bigger or if you know how to use XML make the X and the Y bigger number


Add docker based nginx deployment guide to docs

This guide shows how to build a docker image to run a vue-cli based application with nginx. It has an example nginx config that does the appropriate pushState() based routing fallbacks for 404's.

The process looks good to me. 👍

@cregetycreg I don't understand the part with the 50x errors, if we don't have a server how is it possible to have a 50x error ?


HDDS-3263. Fix TestCloseContainerByPipeline.java.

What changes were proposed in this pull request?
Enabled the test as it seems to be stable with latest ratis upgrade.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3263
How was this patch tested?
Manually verified.

I think it was a bit too early to merge this after a single run.  TestCloseContainerByPipeline failed on master and in #1120 since then.


Crawl vs List

1stly - Cool!
SSM rocks.
Think this is the first time I've said this.
So does this hit a single page, crawl a whole site or just run through a list?
It isn't clear to to me how much needs to be defined in your Lighthouse scan.
I'm also unclear how multi-page content is agregated.
Sorry for not RTFMing this.

Thanks for your interest @mgifford .  I don't think there's a manual to read, so you're forgiven, 😄
Each website builds it's own list of pages to crawl using a buildConfig file, located in the sites folder.  Here's the one for SaultSteMarie.ca.
import { writeConfig } from "../../utils.js";

(async () => {

  await writeConfig([
    "https://saultstemarie.ca/",
    "https://saultstemarie.ca/Search.aspx?searchtext=parks",
    "https://saultstemarie.ca/webapps/meetingMinutes.asp?type=council",
    "https://saultstemarie.ca/webapps/corporateCalendar.asp?e=true",
    "https://saultstemarie.ca/webapps/parabusCalendar.asp",
    "https://saultstemarie.ca/webapps/parksAndPlaygrounds.asp"
  ], [
    "https://saultstemarie.ca/"
  ],
    "saultstemarie");
})();

So there's two sections of URLs.  The first list are pages that may not appear when crawling the website.  The second list are pages that should be crawled.
The depth of the crawl is defined in the global config file.
In the end, after combining the list of crawled URLs with the list in the build file, a random selection of URLs is picked, based on the limit set in the config file.  This ensures that the GitHub action can complete before the time limit.  For example, there are hundreds of pages on the City website.  It takes too long to scan them all.
So to do a scan on SaultSteMarie.ca, after installing the project, two scripts are run.
npm run build:website:saultstemarie
npm run test:website:saultstemarie

The build script uses the config files to build a fresh lighthouserc.json file.  The test script runs Lighthouse tests on that lighthouserc.json file using lighthouse-ci.
A GitHub Action runs for each website daily.  It builds and tests.
If any page in the lighthouserc.json file doesn't meet the thresholds, the GitHub Action is marked as failed.  I use badges from shields.io to show the results of the last run.
Does all that make sense?


Update sr.coffee

I have translated some actions to Serbian language.
Greetings

Hello, @Ivanmateja! Thank you for contributing. Can you sign our CLA so that I can merge your pull request?
Can you also uncomment (remove the # symbol from) the lines you translated?

I hope I did everything as it should be done............


Simple MLP graph

Hi @gstoica27,
How are you?
Can you please provide a graph example for stacked linear layers (i.e. MLP) with activation functions?
Graph sketch: Linear->ReLU->Linear->ReLU->Linear->ReLU (3 linear layers as an example).
Cheers,

@dbolya Maybe you can help with this, please?

Hey,
The graph would look something like this:
(Node 0, INPUT) -> (Node 1, Linear) -> (Node 2, ReLU) -> (Node 3, POSTFIX) -> (Node 4, Linear) -> (Node 5, ReLU) -> (Node6, POSTFIX) -> (Node 6, Linear) -> (Node 7, ReLU) -> OUTPUT
There may also optionally be a POSTFIX after node 7 depending on if you place a classification/head layer after the last ReLU activation.
However to get a better sense of the graph structure, I would strongly encourage you to run the already made implemented graphs in the "graphs" directory of ResNet, SinGAN, and VGG (most like the network you're describing) models. You can run the files by following the instructions in the readme and easily generate graph visualizations to then inspect.
Thanks!

Also please see here: https://github.com/gstoica27/ZipIt/issues/6 (bottom) for a visualization of the resnet graph from running graphs.resnet_graph.py file :) - Hope this helps!

Hi @gstoica27,
Thank you for responding!
I think you misunderstood me, I didn't mean to graphic graph, I'm looking for a code snip that constructs BIGGraph suitable for MLP architecture.
I really appreciate any help you can provide.

Hi Aviv,
If you comment out line 27, you can actually already use the VGG graph code to construct an MLP. To construct an MLP as you've described you can just set the architecture parameter to be [Linear1Dim, Linear2Dim Linear3Dim] (Assuming your MLP is an nn.Sequential assigned to a variable called "self.features").
To be clear though, all BIGGraph does is encode the computational flow of an actual model architecture you would place under "models/*". models/ and graphs/ go in tandem with one another, as described in the README. So to do what you're asking you will have to create both a graph and a model architecture.
Here is an example (pseudocode):
# Model
#imports....
class MLP(nn.Module):
    def __init__(self):
        super(MLP, self).__init__()
        self.features = nn.Sequential(
            nn.Linear(10, 32),
            nn.ReLU(),
            nn.Linear(32,32),
            nn.ReLU(),
            nn.Linear(32, 32),
            nn.ReLU()
        )
        self.classifier = nn.Linear(32, 10) # CIFAR10
    def forward(self, x):
        return self.classifier(self.features(x))

# Graph
#imports....

class MLPGraph(BIGGraph):

    def __init__(self, model):
        super().__init__(model)

    def graphify(self):
        input_node = self.create_node(node_type=NodeType.INPUT)
        node_insert = NodeType.PREFIX
        graph = [f'features.{graph_idx}' for graph_idx in range(6)] + ['classifier', NodeType.OUTPUT]
        self.add_nodes_from_sequence('', graph, input_node, sep='')
        return self

Please see the repo, graphs and models READMEs for more information :).

Now it is clear! @gstoica27 Thank you very much!
One final question, suppose I have two trained networks both having the architecture you wrote in the code snip above. Which script / function do I need to run to align them?
Thanks in advance.


[Bug] Thank you very much ; product seems great but i have an issue starting the docker container i get just an image with 3 bullets and nothing more

Bug Description
Thank you very much ; product seems great but i have an issue starting the docker container i get just an image with 3 bullets and nothing more
Steps to Reproduce
Access the page on port 3000

Expected Behavior
[Bug] Thank you very much ; product seems great but i have an issue starting the docker container i get just an image with 3 bullets and nothing more
Screenshots
No response
LLM Model ID
No response
Desktop Browser
No response
Desktop Browser Version
No response
Smartphone OS
No response
Smartphone Browser
No response
Smartphone Browser Version
No response
Additional Logs
No response

Could you provide any console output from the backend and chrome devtools to help locating the issue?


Week1 Assignment (ques 1 and 2) Prafful Gupta

screen shots of armstrong number and html page

Good job 🎉

Good job, closing the pull request as your changes have been approved.


tests fail with Rust 1.71+

We're packaging the bytecheck crate for Fedora Linux as a dependency of rkyv and other crates and run tests during the build. I noticed that doctests have started to fail with Rust 1.71.0 - this is also reproducible by checking out this git repo and just running "cargo test":
   Doc-tests bytecheck

running 1 test
test src/lib.rs - (line 23) ... FAILED

failures:

---- src/lib.rs - (line 23) stdout ----
Test executable failed (exit status: 101).

stderr:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: StructCheckError { field_name: "c", inner: CharCheckError { invalid_value: 51646506 } }', src/lib.rs:45:7
stack backtrace:
(actual backtrace excluded, since it's not helpful)

failures:
    src/lib.rs - (line 23)

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.09s

I delete three test code example in lib.rs, and the cargo test run ok, it might be the error with these three test code example.
https://github.com/rkyv/bytecheck/blob/273880316e79a0ffa3978e3bdf3708cdda1904d3/bytecheck/src/lib.rs#L57~L80
@djkoloski
Would you take a look at it?


Time data type returns the right time but the Date is the unix epoch (1970-01-01)

Environment details:

Database: Postgres
prisma : "@prisma/cli": "^2.0.0", "@prisma/client": "^2.0.0", "@nexus/schema": "0.13.1","nexus-prisma": "0.13.0"
I am using introspect to generate schema from DB.

Issue: Prisma2 datetime fields always consider default("1970-01-01T07:13:17.486Z").
Description: I have two field  "grade_created_at" and  "grade_updated_at" and I am assigning this value using new Date() object. request and arguments looks good to me but its store in db default date time.
Please find my steps are as follows:

Resolver code snippet

  let mydate=new  Date();
   args.grade_updated_at =_mydate_;
    console.log("arguments ",args)
     return ctx.prisma.grade.update({
               where: {
                    grade_id: args.grade_id,
                },
                data: args,
            });

Result: console output snippet as follows and its should store value in DB "2020-06-13T00:15:08.421Z"
Current date 2020-06-13T00:15:08.421Z
arguments  {
  grade_id: 27,
  grade_name: 'UpdatedByHCM',
  grade_updated_at: 2020-06-13T00:15:08.421Z
}

PlayGround Request and response

mutation{
  updateGrade(grade_id:27, grade_name:"UpdatedByHCM"){
    grade_id
    grade_created_at
    grade_updated_at
  }
}

Database value:

grade_created_at:"07:13:17.486000+00:00"
grade_update_at:" 00:15:08.421000+00:00"

response:
{
  "data": {
    "updateGrade": {
      "grade_id": 27,
      "grade_created_at": "1970-01-01T07:13:17.486Z",
      "grade_updated_at": "1970-01-01T00:15:08.421Z"
    }
  }
}

SQL to generate Grade Table are as follow:
CREATE TABLE "Grade" (
	"grade_id" serial NOT NULL,
    "grade_name" varchar(255) NOT NULL,
	"grade_start_date" time with time zone,
	"grade_end_date" time with time zone,
	"grade_created_at" time with time zone NOT NULL,
	"grade_updated_at" time with time zone NOT NULL,
	CONSTRAINT "Grade_pk" PRIMARY KEY ("grade_id")
) WITH (
  OIDS=FALSE
);

I can reproduce this, thanks for reporting:

Here is the script that I used for reproduction:
import { PrismaClient } from "@prisma/client";

async function main() {
  const client = new PrismaClient();

  const currentDate = new Date().toISOString();

  const data = await client.grade.create({
    data: {
      grade_name: "test",
      grade_created_at: currentDate,
      grade_updated_at: currentDate,
      grade_end_date: currentDate,
      grade_start_date: currentDate,
    },
  });

  console.dir(data, { depth: Infinity });

  client.disconnect();
}

main();

I used the SQL posted above and introspected it to get the schema.
CREATE TABLE "Grade" (
	"grade_id" serial NOT NULL,
    "grade_name" varchar(255) NOT NULL,
	"grade_start_date" time with time zone,
	"grade_end_date" time with time zone,
	"grade_created_at" time with time zone NOT NULL,
	"grade_updated_at" time with time zone NOT NULL,
	CONSTRAINT "Grade_pk" PRIMARY KEY ("grade_id")
) WITH (
  OIDS=FALSE
);

model Grade {
  grade_created_at DateTime
  grade_end_date   DateTime?
  grade_id         Int       @default(autoincrement()) @id
  grade_name       String
  grade_start_date DateTime?
  grade_updated_at DateTime
}

This is still reproducible with 2.21.0-dev.37 even with native types information in the schema now:
model Grade {
  grade_id         Int       @id @default(autoincrement())
  grade_name       String    @db.VarChar(255)
  grade_start_date DateTime? @db.Timetz(6)
  grade_end_date   DateTime? @db.Timetz(6)
  grade_created_at DateTime  @db.Timetz(6)
  grade_updated_at DateTime  @db.Timetz(6)
}

This is more of a product question. It feels like we'll want a special Time type like we have with Decimal.

Here is the other direction where only the date part is supposed to be stored, but time is also stored and returned: https://github.com/prisma/prisma/issues/5769


Upgrade node dependencies and migrate to node 18

Github Ticket: https://github.com/ConsenSys/dx-team/issues/1511
Metamask Ticket: https://github.com/MetaMask/website/issues/236

We have acknowledged some layout bugs on Safari, we are working on it.

We have acknowledged some layout bugs on Safari, we are working on it.

This has been fixed.


[v2] Automatically bind all factory methods

Description

background
Before we dive into the solution, let's have a quick recap of the this keyword in javascript.
If you have an object and you call the method on the object, this keyword refers to the object itself.
const test = {
  prop: 42,
  func: function() {
    return this.prop;
  },
};

console.log(test.func());
// 42

However, if you don't call the method on the object, but call the method itself, this will be undefined
const test = {
  prop: 42,
  func: function() {
    return this.prop;
  },
};

const myFunc = test.func

console.log(myFunc());
// undefined

problem
Our server factory exposes public APIs like render, serveServiceWorker. Users can use the methods in the following ways:
app.get('*', runtime.render)
// "this" is undefined because the render method is invoked directly

Or
app.get('*', (req, res) => runtime.render(req, res))
// "this" is defined because we call the render on the runtime object

To support both, we need to bind this to the server factory itself to make sure this always work as intended.
The solution is to always bind this for every single method on the server factory singleton object.
Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

(change1)

How to Test-Drive This PR

(step1)

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[ ] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

This looks like an extension of the work done by @vmarta in #596.
I don't think there are any issues with using both approaches.
In #596, we also updated build-remote-server.js. Should we apply this change to build-remote-server.js as well?

Is anyone able to run the dev server for like the Retail React App? My server quits each time I run it.


Play Playlist by ID from shell

It would be nice to start a Playlist by parsing the Spotify ID by Argument.

+1 for this feature


dependency name directory content disclosure (ssl)

An attacker is in control of the projects or a sub-dependencies bower.json and puts a malicious dependency names in the configuration file.
{
  "name": "malicious",
  "dependencies": {
    "ssl": "../../../../../../../../../../etc/ssl"
  }
}

so that contents of /etc/ssl are copied to ./bower_components/ssl/

I don't think this is security issue as Bower should not be executed as root user

Because Bower by design allows to install packages from any directory on disk, and we cannot possible blacklist all sensitive directories, it's only reasonable to close this issue and recommend to not run Bower as a root

Because Bower by design allows to install packages from any directory on disk, and we cannot possible blacklist all sensitive directories, it's only reasonable to close this issue and recommend to not run Bower as a root


Allow to override isDifferent

Could you explain the purpose of this change?

I had the problem what timestamps where not compared correctly and, if the file existed already, it would never get updated. This method allows me to make files always overwrite if found in both sides.

Take a look at the latest commits. Timestamp comparison has finally been added.


[SPARK-32309][PYSPARK] Import missing sys import

While seeing if we can use mypy for checking the Python types, I've stumbled across this missing import.

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

cc @HyukjinKwon

Thanks @Fokko


Wellform

This helper function make sure there is no variable shadowing in the AST.
Stuff left to be done:

[ ] write some test
[ ] Should we check for weak uniqueness (current form) or strong uniqueness (each var can only be bind once)? It require a single line of change (in particular, deleting the destructor for Insert), so it is more of a design question. We can also support both though.

We should check the strong uniqueness, I think it makes sense to have one function that checks all the things

I agree with @tqchen I think strong uniqueness makes since, in debug mode we can check this more frequently to aid in development and bugs. As the pass infrastructure evolves it would make sense to add a sanity mode, which does the checks aggressively to suss out bugs.

I left some first pass review, could you also add tests of various scenarios? We could use C++ tests or expose and test in Python? @tqchen what do you think?

I also lean favor of strong uniqueness mode. It also free us from using RAII (which is uglier then just a function invocation)
Do you two want me to write a copy function, which, given an expr, generate a structurally equivalent expression with new var? It will come handy in optimization.

I don't think we need a copy function yet unless I'm missing something.

Yes, we dont need it yet. I will open a separate pr for it.

In the interest of making development faster, we can focus on getting most test cases in python and have minimum c++ test cases that covers data structure correctness

@jroesch please https://docs.tvm.ai/contribute/code_review.html#approve-and-request-changes-explicitly

Thanks @MarisaKirisame @jroesch this is now merged