Don't force hoisting

Describe the bug
dev
[vite] Internal server error: Failed to resolve import "vue" from "../../../node_modules/.pnpm/@slidev+client@0.46.3_postcss@8.4.33_typescript@5.3.3_vite@5.0.12/node_modules/@slidev/client/main.ts?v=7928d2f5". Does the file exist?`

build
[vite]: Rollup failed to resolve import "vue" from "../../../node_modules/.pnpm/@slidev+client@0.46.3_postcss@8.4.33_typescript@5.3.3_vite<EMAIL_ADDRESS>This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
`build.rollupOptions.external`

Is there workaround? We can't use pnpm hoisting please consider workaround or let it work without hoisting otherwise can't use it because project cant be outside our monorepo.
No reason for project to need hoisting this requirement should be removed.
To Reproduce
Use pnpm and start slidev in package in monorepo.
I use lot of vite based products that have lot of dependencies but vite does not have problem to find those in sub directory. They dont need to enable hoisting they work in default way.
Why it works in every product but not slidev? If you can help me understand I can try think solution.
In repo/packages/project/slideproject/node_modules I only see .bin, .vite, @slidev (with cli under it) and playwright-chromium. @slidev/cli and playwright-chromium are the only 2 dependencies I set in package.json.
Shouldn't all dependencies of slidev be in repo/packages/project/slideproject/node_modules or repo/packages/project/slideproject/node_modules/@slidev/cli/node_modules (but there is only .bin there)?

This error is strange because there is already a CI for non-hoist use case: https://github.com/slidevjs/slidev/actions/workflows/smoke.yml?query=branch%3Amain

Shouldn't all dependencies of slidev be in repo/packages/project/slideproject/node_modules or repo/packages/project/slideproject/node_modules/@slidev/cli/node_modules (but there is only .bin there)?

I think that's because you are using PNPM. All non-hoist PNPM node_modules looks like this.

We temporarily closed this due to the lack of enough information. We could not identify whether it was a bug or a userland misconfiguration with the given info.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required

@KermanX Is there demo slidev project I can use that already work with non-hoist PNPM? I can modify that to narrow down the cause.

Is there demo slidev project I can use that already work with non-hoist PNPM? I can modify that to narrow down the cause.

https://github.com/KermanX/temp-slidev-project-3-10
Works well on my Windows machine

@KermanX I tried linux and pnpm 8.15.4. dev command runs fine but when load the web page it's blank screen and this error happens in console.
4:27:32 PM [vite] Internal server error: Failed to resolve import "@vueuse/core" from "node_modules/.vite/deps/chunk-MJVIJ6GD.js?v=5130af76". Does the file exist?
  Plugin: vite:import-analysis
  File: /temp-slidev-project-3-10/slides/node_modules/.vite/deps/chunk-MJVIJ6GD.js?v=5130af76:76:99
  74 |  // ../node_modules/.pnpm/@vueuse+motion@2.1.0_vue<EMAIL_ADDRESS>  75 |  import { ref, unref, watch, computed, reactive, defineComponent, useSlots, h, resolveComponent, onUpdated, isRef } from "vue";
  76 |  import { isObject, useEventListener, useIntersectionObserver, unrefElement, useMediaQuery } from "@vueuse/core";
     |                                                                                ^
  77 |  import { tryOnUnmounted } from "@vueuse/shared";
  78 |
      at formatError (file:///temp-slidev-project-3-10/node_modules/.pnpm/vite@5.1.5/node_modules/vite/dist/node/chunks/dep-G-px366b.js:50545:46)
      at TransformContext.error (file:///temp-slidev-project-3-10/node_modules/.pnpm/vite@5.1.5/node_modules/vite/dist/node/chunks/dep-G-px366b.js:50539:19)
      at normalizeUrl (file:///temp-slidev-project-3-10/node_modules/.pnpm/vite@5.1.5/node_modules/vite/dist/node/chunks/dep-G-px366b.js:65623:33)
      at async file:///temp-slidev-project-3-10/node_modules/.pnpm/vite@5.1.5/node_modules/vite/dist/node/chunks/dep-G-px366b.js:65778:47
      at async Promise.all (index 2)
      at async TransformContext.transform (file:///temp-slidev-project-3-10/node_modules/.pnpm/vite@5.1.5/node_modules/vite/dist/node/chunks/dep-G-px366b.js:65699:13)
      at async Object.transform (file:///temp-slidev-project-3-10/node_modules/.pnpm/vite@5.1.5/node_modules/vite/dist/node/chunks/dep-G-px366b.js:50854:30)
      at async loadAndTransform (file:///temp-slidev-project-3-10/node_modules/.pnpm/vite@5.1.5/node_modules/vite/dist/node/chunks/dep-G-px366b.js:53644:29)
      at async viteTransformMiddleware (file:///temp-slidev-project-3-10/node_modules/.pnpm/vite@5.1.5/node_modules/vite/dist/node/chunks/dep-G-px366b.js:63438:32)

Exact steps
git clone https://github.com/KermanX/temp-slidev-project-3-10
cd temp-slidev-project-3-10
pnpm install
cd slides
pnpm run dev slides.md

@KermanX in earlier slidev dev and build failed but in 0.48.1 build is working. For you too?

@KermanX Is dev working for you?

Additional information:

@vueuse/motion is resolved correctly, but @vueuse/core is not, because it is excluded from optimizeDeps.
Also happens in non-monorepo

A workaround that I found is to partially hoist @vueuse/*.
Create an .npmrc file with the contents:
public-hoist-pattern[]=@vueuse/*

And re-run pnpm install. Then all @vueuse packages are symlinked in node_modules which plays nice with Vite.


bug in get_sample_cn_segments

with_chr_prefix for get_sample_cn_segments does not work as advertised.

Has been addressed in this PR.


More ppp backports

Based on https://github.com/pine64/bl_iot_sdk/pull/21#issuecomment-718221832, I am adding two commits from ppp repository. The first one (ignore received EAP messages when not doing EAP) was rebased as https://github.com/pine64/bl_iot_sdk/commit/6131dfddd8cfaf94ae694a03ec67e5cc16ebffb5#diff-2c9b83fa4781ad1823da18fd74f6f208631e4dd5f53e49934ebc2dcccb73c935R2056 is not present in pppd and I didn't try to compile it, it should be ok. The other one applies cleanly.

Thank you for your another contribution. :)
Also, great that you pushing those patches to upstream repository. At the moment, is uncertain if sending patches to upstream will be made, but I hope it will happen in future.


Update build-all.bash

Use either shasum or sha256sum whichever is available. Fixes #2065.
What does this do / why do we need it?
This allows Archlinux builds to succeed.
What should your reviewer look out for in this PR?
I did not update the changelog, because I'm no sure under what version to add a line.
Do you need help or clarification on anything?
I tested on Archlinux, works!
Which issue(s) does this PR fix?
fixes #2065

I signed it.

The travis errors seems to be caused by something else than the change I have made.
Test killed with quit: ran too long (10m0s).

A rebase may need to be required...

Can you try a rebase, the tests should be back to green!

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


Improvements for RTL

issue #618

@iriman   Thank you for your work here. It might take a little bit before we can merge this as we have a staging site (see details on https://learnosm.org/en/contribute/translator/) where we have two more rtl languages, Arabic and Hebrew, which have to be considered apart from Farsi. Additionally, we do not edit foreign language versions of our documents here but automatically pull the translations from Transifex. Any additional modifications would have to be performed by scripting during the pull process.
Getting all this under one hood needs a few preparations and checks. But I hope to get this done within this week.

Very good, please let me know if there was any problem regarding rtl.

I dig into this a bit more. We can avoid changing anything on transifex side, instead use liquid to put a condition for the rtl languages to detect rtl.  Please take a look at gh-pages...iriman:newrtl
I also renamed direction-dependent css rules to be more meaningful in both rtl and ltr.
Totally I guess this is a much cleaner way.
In case if not decided yet, I would like to suggest this approach. I can send a new PR or force push here.

As you noticed I pulled your request on the staging site. Seems to work fine. And issuing pull requests on the staging site is our preferred way as we can try things out there without disrupting the main site. I will pull these changes over to the main site this weekend. Thanks again for your work.


CAN Speed in the activeChannelComboBox CAN_L CAN_M CAN_H How does it work?

Hi! Thank you very much for this work!
Please tell me how this works at Arduino?
I can't understand!

in main.py
        if self.activeChannelComboBox.isEnabled():
            txBuf = [0x42, self.activeChannelComboBox.currentIndex()]   # TX FORWARDER
            self.serialWriterThread.write(txBuf)
            txBuf = [0x41, 1 << self.activeChannelComboBox.currentIndex()]  # RX FORWARDER
            self.serialWriterThread.write(txBuf)

Sends this
for CAN_L sent [0x42, 0] [0x41, 1] or [66, 0] [65, 1]
for CAN_M sent [0x42, 1] [0x41, 2] or [66, 1] [65, 2]
for CAN_H sent [0x42, 2] [0x41, 4] or [66, 2] [65, 4]

How does Arduino understand this? And does it work?
I thought it would work here
in canSniffer.ino
CAN.begin(CAN_SPEED)
Where CAN_SPEED can be (33E3) or (95E3) or (500E3)
Sorry for my English
Thank you very mutch!

Thanks Lelick!
That feature in the GUI does not work with the arduino code. It will never work because the arduino and the MCP2515 is connected to 1 channel at a time. The speed could be selected through that dropdown, but that's not implemented.
I've used that selector to select between the 3 can busses when I developed my own sniffer hardware that you can see in my first video on youtube. Later, I'm planning to make that hardware available and bring back this feature on the GUI.

Yes. I saw it.

I was interested. Thanks Adam for the answer. If I come up with something, I'll write here. If you can change the CAN speed without re-firmware Arduino. You may just have to add a physical switch to change the CAN Speed.
This
 or this
I will try
Sorry about my English. Again.


Configuring redis session

Ask your question
Should the working example below be the officially recommended way of configuring a nuxt-session?
Additional information
Using the example below based on the docs for configuring the redis session storage is partially broken / misleading.
// file: ~/nuxt.config.ts
export default defineNuxtConfig({
  modules: ['@sidebase/nuxt-session'],
  session: {
    session: {
      storageOptions: {
        driver: 'redis',
        options: {
          url: process.env['REDIS_URL']
        }
      }
    }
  }
})

It is working well in a local dev setup, but breaks, when the app is built into a docker container. Most probably, because at build time, we deliberately do not supply the value of the REDIS_URL env var, the storage driver falls back to its default value.
I've found, that the config is actually read via nuxt's useRuntimeConfig function and supplying the config below, makes it work in both local and prod setups.
export default defineNuxtConfig({
  modules: ['@sidebase/nuxt-session'],
  runtimeConfig: {
    session: {
      session: {
        storageOptions: {
          driver: 'redis',
          options: {
            url: () => process.env['REDIS_URL']
          }
        }
      }
    }
  }
})

Hi @gjedlicska, i tried this runtimeConfig for build production and didn't work.
My attempt commands:
yarn build
export REDIS_URL=redis://localhost:6380 (intentional changed port)
yarn preview

He don't log the error message "can't connect ..." but save nothing on redis.

@gjedlicska i created this issue two weeks ago: #82 . I can't seem to figure this out. Would you have any ideas?

@luukgruijs
Turns out, if you only set the env var NUXT_SESSION_SESSION_STORAGE_OPTIONS_OPTIONS_URL , what should be the full url of the redis instance, you do not need to configure anything to your nuxt.config.ts file.

To provide a full example for anyone still struggling with this:
{
  modules: ['@sidebase/nuxt-session'],
  // the build time config is needed, so that the nuxt module selects the right driver module during build
  session: {
    session: {
      storageOptions: {
        driver: 'redis',
        options: {
        }
      }
    }
  },
  // the runtime config is needed, to specify the shape of the config object.
  // a dummy value for the url, that can be overridden by nuxt's runtime env var system
  // https://nuxt.com/docs/guide/going-further/runtime-config#environment-variables
  runtimeConfig: {
    session: {
      session: {
        storageOptions: {
          driver: 'redis',
          options: {
            // overriden by NUXT_SESSION_SESSION_STORAGE_OPTIONS_OPTIONS_URL env var
            url: 'UNDEFINED' // you need to define the key, with a dummy value
          }
        }
      }
    }
  }
}

I'm adding a PR to update the readme with this.


Display progress hud

is it possible to use progress hud while using remote url. Until the download is finished

yes this is possible with DispatchQueue.main.async

hello, i found the delay ~ 10 second when open pdf url with large content, so i want to display progress hud while opening until the pdfcontroller is showing, can you help me know where need to put that code?


feature request: lines linking points to underlying raster

This is a super cool project, and something that I've always wanted to see in rstats. Thank you for thinking of this!
Take the following with a grain of salt and only if you are inspired by it, or see it as useful. Otherwise, feel free to close the issue.
I'm wondering how difficult it would be to implement a feature to link points or polygon centroids to the underlying raster? Something like the following, but with lines for all points:

Thanks for your kind words!
Puh. I had the same idea at some point, but my brain was to lazy to come up with a good solution for that.
Since the spatial data itself is shifted, one had to give each feature an ID with respect to its original location.
After the shifting one could come up with a line feature connecting the same ids and draw them with ggplot2?
Initial thought ... could be possible :D
Do you want to do this for single features or AOI, say a certain area shown as a rectangle?

Awesome!
I'm not sure how to implement the nitty-gritty details, but can imagine a feature like plot_tiltedmaps(map_list, connect_lines = TRUE) where by default lines are computed on-the-fly between features and the underlying raster.
Or to your point of constraining this to a set of features, perhaps plot_tiltedmaps(map_list, connect_lines = column) where column is a TRUE/FALSE column in the points or polygon dataframe that specifies if a line should be drawn for that feature.

Will tackle this as soon as there is some more time :)

I think I have it 👍 Just thinking about the interface now.

Yes, here it is cryptic. For me, a good use case would be to indicate a single or only a few points of interest, e.g. mountain peaks. I'd love to see a function and an example, maybe one illustrating it with only a few lines?


feat: Add support for Fedora

Enhancement: Add support for Fedora
Reason: SQL Server from RHEL 9 RPMs might work on latest Fedora.

[citest]

[citest]


New package: FiftyOneImageAnnotations v0.1.0

Registering package: FiftyOneImageAnnotations
Repository: https://github.com/IHPSystems/FiftyOneImageAnnotations.jl
Created by: @stemann
Version: v0.1.0
Commit: c2abd731ca0f0f0847d4e517ba07ee00de26a2a8
Git reference: HEAD

Please make sure to add a description, and some basic documentation like a usage example in the README before you register

There is some documentation in the README now @ https://github.com/IHPSystems/FiftyOneImageAnnotations.jl

Thanks for extending the README, although I still think that there's a lot of context missing here for this to be useful for a larger audience, see https://github.com/JuliaRegistries/General/pull/89020#issuecomment-1666373243

[noblock] Given that the package names are pretty unique and unlikely to cause conflicts with other registrations, if you insist, I will make my comments non-blocking. But I think you should consider either adding some substantial documentation for the packages in your organization, or using a LocalRegistry if this is intended more for internal usage.

Thanks for reviewing! And for guarding the General registry.
The intent is indeed to add documentation - to improve the general usability of the packages.
In this registration case, FiftyOneImageAnnotations, there was just a significant effort involved in wrapping the Dataset class (10,000 lines) - elaborate documentation will make more sense once there's functionality to launch the FiftyOne app. etc.
These packages will also be registered in a private registry (once the name is "final").
[noblock]


padrino rake routes cannot get anything

Hi
There has an issue about padrino rake routes task that it cannot display any route records  and only display Executing Rake routes ... when run it
This is a new project and already has lots of available controllers generated by pradrino g

I found the reason, becuz i use the following method to defind route action in controller, then i changed to :map, it is worked.
get '/' do
    'route could not be display in rake routes'
 end

get :index, :map => '/' do
    'route could be display in rake routes'
 end


adapt library for cadence1

Closes #224
Description
Various changes to support Cadence 1.0 emulator
All tests are passing
I've changed the workflow to use Cadence 1.0 CLI
This will need to be reverted later on

For contributor use:

[x] Targeted PR against master branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[x] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Hey @NtTestAlert , thanks for the PR! I will review this & try to release it today :)

Hey @NtTestAlert , thanks for the PR! I will review this & try to release it today :)

Hi, there is one more thing to be done, stable-cadence cli adds -c1 suffix (but only in some versions), and a version check in general would be good.
Will add it in soon.

Added execName param to emulator, defaulting to flow-c1
tested locally with older flow-cli, as new ones don't run on windows

I tried getting CI to work, but I'm having issues so I guess I can just run locally toverify.
I just ran the tests and it looks good, but it seems like signer parsing fails for an entitled account reference as a signer (e.g. auth(SaveValue) &Account as oppose to an unauthorized reference &Account), which is generally speaking what users will be using in their transactions.
Are we able to support this?  Seems like it might be a blocker for any real usage of the library?

The tests for flow-js-testing pass, incl those with authorized signers - it doesn't try to parse them before executing
However flow-cadut does not properly parse authorized signers as per other PR, added a fix there.
Pipeline failed because it could not parse version due to the upgrade banner, bypassed by adding --output json to "fool" flow-cli

Still need to adapt the library to work with shorthand imports
(import "ViewResolver")
In my project I have this hooked in jest and with some regexes that expand it to import ViewResolver from "./ViewResolver.cdc"
I'll try to add it to the library today


Connect iOS trait to Flutter's window brightness

Connect iOS trait to Flutter's window brightness

Any updates on this @matthew-carroll ?

This work was finished a little while ago. The setting is passed here:
https://github.com/flutter/engine/blob/master/shell/platform/darwin/ios/framework/Source/FlutterViewController.mm#L928
I'm going to close this issue.


Fix sdlConverter to export as commonjs

Exported files by sdlConverter with commonjs: true option cause ReferenceError like below.
 $  yarn start
yarn run v1.10.1
$ node lib/index.js
/Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/generated/project_server-types.js:40
    t.list.field("staffings", { type: Staffing })
                                      ^

ReferenceError: Staffing is not defined
    at Object.definition (/Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/generated/project_server-types.js:40:39)
    at SchemaBuilder.buildObjectType (/Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/node_modules/nexus/dist/builder.js:159:16)
    at SchemaBuilder.getOrBuildType (/Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/node_modules/nexus/dist/builder.js:513:29)
    at /Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/node_modules/nexus/dist/builder.js:117:45
    at Array.forEach (<anonymous>)
    at SchemaBuilder.getFinalTypeMap (/Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/node_modules/nexus/dist/builder.js:108:42)
    at buildTypes (/Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/node_modules/nexus/dist/builder.js:565:20)
    at makeSchemaInternal (/Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/node_modules/nexus/dist/builder.js:593:19)
    at Object.makeSchema (/Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/node_modules/nexus/dist/builder.js:621:18)
    at Object.<anonymous> (/Users/moriwakikento/.ghq/github.com/KentoMoriwaki/grgr/lib/index.js:7:24)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

The export file is like this. type: Staffing cannot refer exports.Staffing.
exports.Project = objectType({
  name: "Project",
  definition(t) {
    t.int("id", { nullable: true })
    t.string("title", { nullable: true })
    t.list.field("staffings", { type: Staffing })
  }
})
exports.Staffing = objectType({
  name: "Staffing",
  definition(t) {
    t.int("id", { nullable: true })
    t.string("introduction", { nullable: true })
    t.int("staffId", { nullable: true })
  }
})

So I changed to export all at the bottom of codes.
const Project = objectType({
  name: "Project",
  definition(t) {
    t.int("id", { nullable: true })
    t.string("title", { nullable: true })
    t.list.field("staffings", { type: Staffing })
  }
})
const Staffing = objectType({
  name: "Staffing",
  definition(t) {
    t.int("id", { nullable: true })
    t.string("introduction", { nullable: true })
    t.int("staffId", { nullable: true })
  }
})

exports.Project = Project;
exports.Staffing = Staffing;

it also works with commonjs: false by generating export { Project, Staffing };

Great, thanks @KentoMoriwaki!
Could we keep it so in ES export mode it does export const rather than exporting at the base?
I’ll make sure we get an option into the converter demo in the docs you can choose the type of output it generates.

Could we keep it so in ES export mode it does export const rather than exporting at the base?

OK. I fixed it.

I’ll make sure we get an option into the converter demo in the docs you can choose the type of output it generates.

Thanks!

Great, thanks @KentoMoriwaki!


Option to turn off dragging in the browser

I'm getting really weird behavior where the slide will stick to the cursor after I stop holding down my mouse button. In the following screenshot, I am not holding down my mouse button when I move back and forth

I wish there was a way to turn off dragging the image with the mouse in the browser

There is!  :-)

^ CarouselProvider component.


added municipality of Kalmar, Sweden

Swedish municipalities seem to break down further into localities (at least according to wikipedia) so I wasn't sure how much detail to add to coverage.  Let me know if more is required.
Fixes #2188

Hi, Swedish localities (tatörter) are not really an administrative division - they are defined every five years based on the density of built-up areas, and thus of course thy often are spread over many municipalities. their main purpose is statistical.
So the municipalities (kommuner) are definitely the right choice for the coverage (they are similar to US counties in scope). They are also the ones responsible for maintaining addresses, so hopefully we will see more of them publishing the data online.
Last week I talked to Lantmäteriet (Swedish Cadastre) regarding their plans of opening up the address data nationwide (they had it with a question mark in some of their presentations). Unfortunately, they said that they have not yet secured any funding for that yet, so this remains a plan without any specific date.
So, for now the best bet in Sweden will be the individual municipalities. Good news is that I have found open address data for Stockholm, will try to publish it later today.

Thanks for the background, this is super helpful :)
On Tue, Dec 6, 2016 at 9:25 AM, sergiyprotsiv<EMAIL_ADDRESS>wrote:

Hi, Swedish localities (tatörter) are not really an administrative
division - they are defined every five years based on the density of
built-up areas, and thus of course thy often are spread over many
municipalities. their main purpose is statistical.
So the municipalities (kommuner) are definitely the right choice for the
coverage (they are similar to US counties in scope). They are also the ones
responsible for maintaining addresses, so hopefully we will see more of
them publishing the data online.
Last week I talked to Lantmäteriet (Swedish Cadastre) regarding their
plans of opening up the address data nationwide (they had it with a
question mark in some of their presentations). Unfortunately, they said
that they have not yet secured any funding for that yet, so this remains a
plan without any specific date.
So, for now the best bet in Sweden will be the individual municipalities.
Good news is that I have found open address data for Stockholm, will try to
publish it later today.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/pull/2193#issuecomment-265213804,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0EOxHv7Q2PlE7ztXJl2C9TexrdFcdEks5rFZqbgaJpZM4LFgzC
.


How to bind a value for use with this SQL query ?

I want set LIMIT and OFFSET in SQL query, And code like:
let res = sqlx::query_as::<_, Todo>("SELECT * FROM todos LIMIT ? OFFSET ? ")
        .bind(opts.limit.unwrap_or(50))
        .bind(opts.offset.unwrap_or(0))
        .fetch_all(&conn)
        .await;

But I got error:
the trait bound `usize: sqlx_core::types::Type<_>` is not satisfied
the trait `sqlx_core::types::Type<_>` is not implemented for `usize`rustc(E0277)
the trait bound `usize: sqlx_core::encode::Encode<_>` is not satisfied
the trait `sqlx_core::encode::Encode<_>` is not implemented for `usize`

Who can help me , how to do this?

Change usize to u32 was worked!


client socket time out

Hi， jjenkov， I really appreciate you blog because  I've learned how to use Java NIO.And I have written a Http server based on your code.It works well when there are not much connections.But when I test it with high concurrent connection(10K connection),  almost all the connections are time out(Socket time out).  I just don't know how to optimize it now. Could you give me some advice? Thanks anyway!

Hi,
I cannot help you with that just yet. I will need to look into all that
myself (with another protocol than HTTP) next month.
Maybe I can get back to you by then. I know it's a long time to wait, but
hey, at least I am getting to it.
Oh, and if you find a solution before then, please let me know!

Jakob Jenkov

On Fri, Nov 20, 2015 at 1:46 PM, whutjs<EMAIL_ADDRESS>wrote:

Hi， jjenkov， I really appreciate you blog because I've learned how to use
Java NIO.And I have written a Http server based on your code.It works well
when there are not much connections.But when I test it with high concurrent
connection(10K connection), almost all the connections are time out(Socket
time out). I just don't know how to optimize it now. Could you give me some
advice? Thanks anyway!
—
Reply to this email directly or view it on GitHub
https://github.com/jjenkov/java-nio-server/issues/1.

--
Jakob Jenkov
<EMAIL_ADDRESS>Founder, CEO
Jenkov Aps
http://jenkov.com
http://tutorials.jenkov.com
http://feeds2.feedburner.com/jenkov-com

    Alright.Thank you anyway:-)
        在2015年11月20日 20:52，Jakob Jenkov 写道:Hi,

I cannot help you with that just yet. I will need to look into all that
myself (with another protocol than HTTP) next month.
Maybe I can get back to you by then. I know it's a long time to wait, but
hey, at least I am getting to it.
Oh, and if you find a solution before then, please let me know!

Jakob Jenkov

On Fri, Nov 20, 2015 at 1:46 PM, whutjs<EMAIL_ADDRESS>wrote:

Hi， jjenkov， I really appreciate you blog because I've learned how to use
Java NIO.And I have written a Http server based on your code.It works well
when there are not much connections.But when I test it with high concurrent
connection(10K connection), almost all the connections are time out(Socket
time out). I just don't know how to optimize it now. Could you give me some
advice? Thanks anyway!
—
Reply to this email directly or view it on GitHub
https://github.com/jjenkov/java-nio-server/issues/1.

--
Jakob Jenkov
<EMAIL_ADDRESS>Founder, CEO
Jenkov Aps
http://jenkov.com
http://tutorials.jenkov.com
http://feeds2.feedburner.com/jenkov-com
—Reply to this email directly or view it on GitHub.

I'm still studying the sources, but I have a guess: When HttpMessageReader.read() calls Message.writeToMessage(), and there is no space for the new content, and no free block to expand, then the content of the ByteBuffer will be lost. (just a guess, didn't tried)


fix: Cannot find module '@actions/core'

close #11
FYI:

If you added a .gitignore file that lists node_modules, you'll need to remove that line to commit the node_modules directory.
https://docs.github.com/en/free-pro-team@latest/actions/creating-actions/creating-a-javascript-action#commit-and-push-your-action-to-github

Another idea would be to create a release branch including node_modules, separate from the master branch.

A side note: use ncc to compile a single js file maybe another solution. That's what github-script do
https://github.com/actions/github-script/blob/315fb4f388d2c97db5d8b0617fbac774c8692670/package.json#L10

A side note: use ncc to compile a single js file maybe another solution. That's what github-script do
https://github.com/actions/github-script/blob/315fb4f388d2c97db5d8b0617fbac774c8692670/package.json#L10

A side note: use ncc to compile into a single js file maybe another solution.

Sounds good solution, I'll try it 👍
Thank you!

A side note: use ncc to compile into a single js file maybe another solution.

Sounds good solution, I'll try it 👍
Thank you!

Try and see if it works better now @yokinist Thanks for the PR by the way! 🥇

Try and see if it works better now @yokinist Thanks for the PR by the way! 🥇


Could not dumpprivkey for zcash

I am getting the following error when trying to "npm start"
[zcash] Could not dumpprivkey for zcash {"type":"offline","message":"connect ECONNREFUSED <IP_ADDRESS>:19332"}
Then later:
2017-03-21 15:15:50 [Pool]      [zcash] (Thread 1) Could not start pool, error with init batch RPC call: {"type":"offline","message":"connect ECONNREFUSED <IP_ADDRESS>:18232"}
I know my RPC service is running because I can  ./src/zcash-cli getinfo
I apologize if I'm doing something daft. I even allowed 19332 and 18232 in the firewall just in case that was my issue.

check if you're using the right rpc ports, zclassic/zcash default rpcport is 8232. Make sure you keep this port firewalled off otherwise anyone can steal the key

Closed due to inactivity

How did you solve it? I met the same problem.


Qibo for Python 3.9

Are we supporting Python 3.9? I have been told that some errors popped up when trying to install Qibo through pip in Python 3.9.
Are we supporting it and this is an error, or is it not yet implemented?
The errors were:
ERROR: Could not find a version that satisfies the requirement qibo (from versions: none)
ERROR: No matching distribution found for qibo

@scarrazza, this is the error I got on dom when I tried installing using the wheels. I think this should be easy to support if we just create a wheel for Python 3.9 which we don't have right now. The code should work as it is, in fact I think that already installing from source (clone the repository and do python setup.py install && python setup.py build) is possible with Python 3.9.

Yes, that's normal, because tf doesn't provide python 3.9 pip packages, but qibo is able to work on python3.9, if you compile tf from source with python3.9.

Related https://github.com/tensorflow/tensorflow/issues/44485


cmd/bootc-image-builder: support to use aws credential file and profiles

Support to use aws credential file and profiles
implements HMS-3305
please comment especially on the part with
		// SMELL we might want to implement a separate function
		//  like awscloud.newFromLocalProfile(awsProfile string, region string)

i guess the function would look similar to this:
// Create a new session from the locally stored AWS credentials file
// but given profile. NewDefault() above will expect "default" as profile or use environment variable AWS_PROFILE
func newFromLocalProfile(profile string, region string) (*AWS, error) {
	// Create a Session with a custom region and specified profile
	sess, err := session.NewSessionWithOptions(session.Options{
		Config: aws.Config{
			Region: aws.String(region),
		},
		Profile: profile,
	})
	if err != nil {
		return nil, err
	}

	return &AWS{
		uploader: s3manager.NewUploader(sess),
		ec2:      ec2.New(sess),
		s3:       s3.New(sess),
	}, nil
}

but would need to go to the images repository...

I'll recreate this pullrequest (also to test the syntax for jira issues) (see #72)


perf(regular_expression): Remove u16 reader, mark surrogate pairs instead

Currently, in non-unicode mode, the pattern string is handled through encode_utf16().
However,

The parsing process itself is performed only on BMP characters
Presence of surrogate pairs appears in the AST is limited
Find the corresponding Span offset is also complicated

Besides, there is no corresponding position for each surrogate pair...

From the above, can't we unify the Reader in unicode mode?
Based on the hypothesis, I'd like to try to find out.

We don't have a codspeed benchmark for it. Let me enable regex parsing in the benchmark.

Oh, really? Thank you! 🙈
Although, I was thinking of withdrawing this PR.
Not using encode_utf16() means we never iterate over the units that consists surrogate pairs in the first place.
This would result in ignoring certain specification syntax.
As a result, while test262 is currently passing (though it might just be an oversight), if there isn't a significant improvement, I think it would be better to keep behavior more spec like.

Oh, really? Thank you! 🙈
Although, I was thinking of withdrawing this PR.
Not using encode_utf16() means we never iterate over the units that consists surrogate pairs in the first place. This would result in ignoring certain specification syntax.
As a result, while test262 is currently passing (though it might just be an oversight), if there isn't a significant improvement, I think it would be better to keep behavior more spec like.

Although, I was thinking of withdrawing this PR.

I don't know about the details, feel free to make your own judgements.


[Event Request] Codeunit::"Lead-Time Management" - Function PlannedEndingDate - OnBeforePlannedEndingDate

We like to request the following Event:
Object: Codeunit::"Lead-Time Management"
Function: PlannedEndingDate
Publisher Event: OnBeforePlannedEndingDate
Reason: To implement custom business logic
begin
// Returns Ending Date calculated backward from Due Date
OnBeforePlannedEndingDate(ItemNo, LocationCode, VariantCode, DueDate, VendorNo, RefOrderType, EndingDate, IsHandled);
if IsHandled then
exit(EndingDate);
GetPlanningParameters.AtSKU(SKU, ItemNo, VariantCode, LocationCode);
Event Code:
[IntegrationEvent(false, false)]
local procedure OnBeforePlannedEndingDate(ItemNo: Code[20]; LocationCode: Code[10]; VariantCode: Code[10]; DueDate: Date; VendorNo: Code[20]; RefOrderType: Option " ",Purchase,"Prod. Order",Transfer,Assembly; var EndingDate: Date; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Editor crash when using StreamPeerMbedTLS::connect_to_stream(Ref, bool, String const&)

Godot version:
3.2.dev.custom_build. 02319dceb
OS/device including version:
Ubuntu 19.04
Issue description:
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x43f60) [0x7f67e6e85f60] (??:0)
[2] StreamPeerMbedTLS::bio_send(void*, unsigned char const*, unsigned long) (/home/rafal/Pulpit/godot/modules/mbedtls/stream_peer_mbed_tls.cpp:60 (discriminator 2))
[3] /usr/bin/godot(mbedtls_ssl_flush_output+0x15e) [0x1880860] (/home/rafal/Pulpit/godot/thirdparty/mbedtls/library/ssl_tls.c:2777)
[4] /usr/bin/godot(mbedtls_ssl_write_record+0x458) [0x1881ebf] (/home/rafal/Pulpit/godot/thirdparty/mbedtls/library/ssl_tls.c:3470 (discriminator 1))
[5] /usr/bin/godot(mbedtls_ssl_write_handshake_msg+0x477) [0x18819ff] (/home/rafal/Pulpit/godot/thirdparty/mbedtls/library/ssl_tls.c:3313)
[6] /usr/bin/godot() [0x1b543eb] (/home/rafal/Pulpit/godot/thirdparty/mbedtls/library/ssl_cli.c:1091)
[7] /usr/bin/godot(mbedtls_ssl_handshake_client_step+0x142) [0x1b5965b] (/home/rafal/Pulpit/godot/thirdparty/mbedtls/library/ssl_cli.c:3544)
[8] /usr/bin/godot(mbedtls_ssl_handshake_step+0x4e) [0x188a0db] (/home/rafal/Pulpit/godot/thirdparty/mbedtls/library/ssl_tls.c:8064)
[9] /usr/bin/godot(mbedtls_ssl_handshake+0x64) [0x188a16b] (/home/rafal/Pulpit/godot/thirdparty/mbedtls/library/ssl_tls.c:8088)
[10] StreamPeerMbedTLS::_do_handshake() (/home/rafal/Pulpit/godot/modules/mbedtls/stream_peer_mbed_tls.cpp:102)
[11] StreamPeerMbedTLS::connect_to_stream(Ref<StreamPeer>, bool, String const&) (/home/rafal/Pulpit/godot/modules/mbedtls/stream_peer_mbed_tls.cpp:157 (discriminator 4))
[12] MethodBind3R<Error, Ref<StreamPeer>, bool, String const&>::call(Object*, Variant const**, int, Variant::CallError&) (/home/rafal/Pulpit/godot/./core/method_bind.gen.inc:2505 (discriminator 16))
[13] Object::call(StringName const&, Variant const**, int, Variant::CallError&) (/home/rafal/Pulpit/godot/core/object.cpp:940 (discriminator 1))
[14] Variant::call_ptr(StringName const&, Variant const**, int, Variant*, Variant::CallError&) (/home/rafal/Pulpit/godot/core/variant_call.cpp:1073 (discriminator 1))
[15] GDScriptFunction::call(GDScriptInstance*, Variant const**, int, Variant::CallError&, GDScriptFunction::CallState*) (/home/rafal/Pulpit/godot/modules/gdscript/gdscript_function.cpp:1080)
[16] GDScriptInstance::call_multilevel(StringName const&, Variant const**, int) (/home/rafal/Pulpit/godot/modules/gdscript/gdscript.cpp:1192)
[17] Node::_notification(int) (/home/rafal/Pulpit/godot/scene/main/node.cpp:58)
[18] Node::_notificationv(int, bool) (/home/rafal/Pulpit/godot/./scene/main/node.h:46 (discriminator 14))
[19] CanvasItem::_notificationv(int, bool) (/home/rafal/Pulpit/godot/./scene/2d/canvas_item.h:166 (discriminator 3))
[20] Node2D::_notificationv(int, bool) (/home/rafal/Pulpit/godot/./scene/2d/node_2d.h:38 (discriminator 3))
[21] Object::notification(int, bool) (/home/rafal/Pulpit/godot/core/object.cpp:952)
[22] SceneTree::_notify_group_pause(StringName const&, int) (/home/rafal/Pulpit/godot/scene/main/scene_tree.cpp:975)
[23] SceneTree::idle(float) (/home/rafal/Pulpit/godot/scene/main/scene_tree.cpp:522 (discriminator 3))
[24] Main::iteration() (/home/rafal/Pulpit/godot/main/main.cpp:1922)
[25] OS_X11::run() (/home/rafal/Pulpit/godot/platform/x11/os_x11.cpp:3163)
[26] /usr/bin/godot(main+0xdf) [0x1390871] (/home/rafal/Pulpit/godot/platform/x11/godot_x11.cpp:56)
[27] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xeb) [0x7f67e6e68b6b] (??:0)
[28] /usr/bin/godot(_start+0x2a) [0x13906da] (??:?)

Steps to reproduce:

Run minimal project and wait

Minimal reproduction project:
https://github.com/qarmin/The-worst-Godot-test-project/
commit 3a8b77b5a0016faa84f91505064dc5298f6c3b98
ZIP - https://github.com/qarmin/The-worst-Godot-test-project/archive/3a8b77b5a0016faa84f91505064dc5298f6c3b98.zip

@qarmin you have opened many issues with that same reproduction project.
It runs A LOT of code (so really, it's not minimal). Some of that code might (and I suspect in fact do) compromise memory, so any successive error might be not related to the stack trace you see.
Can you reproduce this by running only the SSL part? If yes, can you provide a minimal project for that?

This is really small  project, Nodes which are hidden(all except root -
Resource and StreamPeerSSL) are deleted a little after running.
The smallest project which I can extract is here - AA.zip

@qarmin thank you for the help and instructions. I proposed a fix.


delete all todo's

delete all todo's

[x] delete all the todo's at once

The changes are approved so I will just merge the PR (Krystyna is away today)


Feat: Enable embedding of datadocs

Purpose:
This PR exposes an endpoint to enable embedding of DataDocs in external Applications. One such use case will be the ability to search and discover DataDocs through the open source project Datahub.
It is not certain that this is the method that we will employ to fully embed the DataDocs, but wanted to open up this workflow for the Querybook community if it would be useful.
External Dependencies:
Metadata modeling to ingest DataDocs to Datahub: https://datahubspace.slack.com/archives/C02R2NBJXD1/p1645047031441309
Requirements:

[ ] Update Docs
[x] Add Tests

Could I get one more review of this @czgu?


Expiremember on large htables globally affects the keydb response time

Describe the bug
KeyDB v6.0.16
When a tangible number of records with the ttl set using the expiremember is added to the htable (500k at the demo stand), the keydb practically stops responding to requests (3k rps at the demo stand), and does not even interrupt them due to timeout.
When using the usual "expire" command - there are no problems with any number of entries with ttl.
perf top shows the following information on keydb:
56.08%  keydb-server                  [.] fastlock_lock
16.63%  keydb-server                  [.] keyIsExpired
profiling graph (collected at a different moment)

Log Files
There is no information in the logs about this problem
To Reproduce

Add htable
Add entries and set ttl for each using the expiremember command
Observe the drop in the overall performance of keydb and the increase in response time from a certain number of records in the table

Fixed with PR #313


mx.utils.StringUtil not found by asdoc

Hello,
I'm using your action to automate generating asdocs but for some reason it can't find mx.utils.StringUtil which should be in the Flex SDK.
It's producing this error:
Error: Definition mx.utils:StringUtil could not be found.

import mx.utils.StringUtil;
               ^

I'm not sure if I'm doing something wrong or not. This is how I've setup the action in my workflow:
      - name: Setup Apache Flex SDK
        uses: joshtynjala/setup-apache-flex-action@v2
        with:
          flex-version: "4.16.1"
          air-version: "<IP_ADDRESS>"
          accept-air-license: true

I have the same flex and air version installed locally.
Anyways, thank you for your time and thank you for making this action.

You'll need to share the command that you use to compile your project.

I'm sorry, I should have taken a break for a day since I just realised you have to point the asdoc command to Flex's frameworks lib folder using: -library-path "${{ env.FLEX_HOME }}\frameworks\libs". It's working great now.
I'm building locally using IntelliJ IDEA which handled all of that stuff for me so I didn't know I had to specify that folder manually. I'm still pretty new to all of this but I'm slowly learning things.


Update how-to-deploy-the-app-v-51-server-using-a-script.md

Updated markdown for Notes.
Removed characters so that the tables are rendered correctly.

First of all, before I get into the messy part: Your changes look just fine, nothing appears to be wrong about them.
This PR is sort of interesting update, because now we are able to see how the HTML tables look in the rich diff preview.
So far, so good.
Have you looked at the HTML table contents yet? I have. They look pretty messy, at least when compared to each other.
I know, the HTML table contents are outside the scope of this PR, as far as I can tell.
Then again, I wonder: Should we leave that mess alone, or should we look into how much work remains to restore some order?
Never mind, I need to look at how the public page looks at the moment. I find it hard to believe it has been left alone looking like this.
I will get back to you as soon as I have looked up the public page and made up my mind about it.

Oh well. Seems the tables have looked the same as they do now, since 7 Apr 2016 and maybe even earlier. I'm too tired to even think of redesigning them or at least try to get them to make sense. 😞

commit from 4 years ago showing the same HTML table content: https://github.com/MicrosoftDocs/windows-itpro-docs/commit/11674e7a1b647c59a373a515af6e6292b007d56e (direct file link with Rich diff view: https://github.com/MicrosoftDocs/windows-itpro-docs/commit/11674e7a1b647c59a373a515af6e6292b007d56e?short_path=47be641#diff-47be6413a4670ebae6a62463d867b4e1)
commit adding the useless code block fence around the tables: https://github.com/MicrosoftDocs/windows-itpro-docs/commit/858476cf2f901f58d733f2943084bf01e11772c8 (direct file link with Rich diff view: https://github.com/MicrosoftDocs/windows-itpro-docs/commit/858476cf2f901f58d733f2943084bf01e11772c8?short_path=47be641#diff-47be6413a4670ebae6a62463d867b4e1)

Long story short: Looking at the contents in of the jumbled rectangles (1 sentence in the first column, dozens of lines in the second column, sometimes a table consists of only 1 row), it looks to me as if the content does not belong in this kind of table at all. I think the text would be more readable in ordinary bullet point lists with headings and sub-headings. 😔
I don't know if you have looked at the contents yet, but when you do, I would like to hear what you think of this oddly jumble-shaped content. 🤔

I only took a quick peek at the tables prior to submitting the PR, but took a better look now. And i have to agree with you, the information does not render itself particularly well to a table format.
I wonder if the new column format would be fitting for this?
I'll have a stab at it and see if i can manage to make sense of it. If so i'll add a commit to this PR (unless it's been closed before i'm done).

I'll open a separate PR, due to the major changes between this PR and the new one.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@baardhermansen : Are you able to resolve the merge conflict in this PR file?

My apologies, i should have closed this issue after 6280 was merged, i simply forgot :(
The differences between this PR and 6280 are too big to be resolved.
Sorry again!

Thanks for the update. Good to have a closing comment with an explanation.


Tome of Horrors - only considered equipment

The Tome of Horrors item in Mansions of Madness doesn't fall under the "Tome" category.  Instead, it is considered equipment.  Is that something that can be fixed?
Thanks!
Valkyrie Version 2.4.07

I am on it

Its not a bug, its on the card.

Interestingly enough, my copy of the card shows the "Tome" designation.  I bought my copy of this expansion late last year, so maybe the newer printings updated it to a "Tome" instead of "Equipment."

I will look into this. Maybe there is some published errata available

I made the changes https://github.com/NPBruce/valkyrie/pull/1302

Thanks, redwolf2!

Thats an odd one from FFG, I guess that was an early version printing error?

Will be in 2.4.10

I reported this in #1133 as well as a different classification issue with the Tome of Secrets card from the same expansion. Has that issue been fixed too?


⚠ SceneTime has degraded performance

In 4830991, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1031 ms

Resolved: SceneTime performance has improved in ba55c08 after 17 minutes.


integrate custom scripts

https://github.com/AUTOMATIC1111/stable-diffusion-webui/wiki/Custom-Scripts
i've already kinda implemented a couple of them in my version but for whatever reason only wildcards and seed travel seem to actually show up in the UI, plus i'm fairly sure my version is hacky as fuck
import os.path
%store -r model_storage_dir
if not os.path.exists('/notebooks/stable-diffusion-webui'):
    %cd /notebooks/
    !git clone https://github.com/AUTOMATIC1111/stable-diffusion-webui
else: # update repo if already exists
    print('stable-diffusion-webui already downloaded, updating...')
    %cd /notebooks/stable-diffusion-webui/scripts
    !wget https://raw.githubusercontent.com/jtkelm2/stable-diffusion-webui-1/master/scripts/wildcards.py
    !wget https://raw.githubusercontent.com/ThereforeGames/txt2img2img/main/scripts/txt2img2img.py
    !wget https://raw.githubusercontent.com/yownas/seed_travel/main/scripts/seed_travel.py
    !wget https://raw.githubusercontent.com/Pfaeff/sd-web-ui-scripts/main/moisaic.py
if not os.path.exists('/notebooks/stable-diffusion-webui/scripts/wildcards/'):
    !mkdir -p /notebooks/stable-diffusion-webui/scripts/wildcards
    %cd /notebooks/stable-diffusion-webui/scripts/wildcards
    !wget https://raw.githubusercontent.com/jtkelm2/stable-diffusion-webui-1/master/scripts/wildcards/adjective.txt
    !wget https://raw.githubusercontent.com/jtkelm2/stable-diffusion-webui-1/master/scripts/wildcards/artist.txt
    !wget https://raw.githubusercontent.com/jtkelm2/stable-diffusion-webui-1/master/scripts/wildcards/genre.txt
    !wget https://raw.githubusercontent.com/jtkelm2/stable-diffusion-webui-1/master/scripts/wildcards/site.txt
    !wget https://raw.githubusercontent.com/jtkelm2/stable-diffusion-webui-1/master/scripts/wildcards/style.txt
else:
    print('scripts already downloaded, updating...')
    %cd /notebooks/stable-diffusion-webui
    !git pull```

This is great. I added this to the install process and also created a new block to download whatever url you give it. Feel free to reopen this issue if you encounter any issues.
ad9b953 f58b05e

everything seems to work well, the only thing i'd suggest would be mostly a bit of space saving in the code itself via
like maybe:
raw = 'https://raw.githubusercontent.com'
gist = 'https://gist.githubusercontent.com'

in the first cell and then replace as accordingly
i'll leave the above suggestion as well as any other space saving(i see a few places that similar could be done) to your discretion

hmm, i stand corrected, as i hadn't actually tried to run any prompts, yet, assuming that if it gave me the gradio link everything is fine, but it seems to be totally broken:
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.```

Ah geez I guess I shouldn't code when I'm tired. Thanks for the tip.
And for that CUDA error, it worked fine when I tested it. Can you post the full output? Maybe we can narrow it down to one script.

hmmm, i think i figured out what the issue is, and that is for whatever reason SD refusing to download
Hit:1 http://security.ubuntu.com/ubuntu focal-security InRelease
Hit:2 http://archive.ubuntu.com/ubuntu focal InRelease
Hit:3 https://deb.nodesource.com/node_16.x focal InRelease
Hit:4 http://ppa.launchpad.net/deadsnakes/ppa/ubuntu focal InRelease
Hit:5 http://archive.ubuntu.com/ubuntu focal-updates InRelease
Hit:6 http://archive.ubuntu.com/ubuntu focal-backports InRelease
Reading package lists... Done
Building dependency tree
Reading state information... Done
79 packages can be upgraded. Run 'apt list --upgradable' to see them.
Reading package lists... Done
Building dependency tree
Reading state information... Done
aria2 is already the newest version (1.35.0-1build1).
0 upgraded, 0 newly installed, 0 to remove and 79 not upgraded.
/storage/models

10/08 23:59:54 [NOTICE] Downloading 1 item(s)

10/08 23:59:54 [NOTICE] IPv4 DHT: listening on UDP port 6986

10/08 23:59:54 [NOTICE] IPv4 BitTorrent: listening on TCP port 6925

10/08 23:59:54 [ERROR] IPv6 BitTorrent: failed to bind TCP port 6925
Exception: [SocketCore.cc:312] errorCode=1 Failed to bind a socket, cause: Name or service not known
[#2f74a4 158KiB/158KiB(100%) CN:38 SD:2]        m
10/08 23:59:58 [NOTICE] Download complete: [MEMORY][METADATA]3a4a612d75ed088ea542acac52f9f45987488d1c

10/08 23:59:58 [ERROR] Exception caught
Exception: [RequestGroup.cc:435] errorCode=13 File /storage/models/sd-v1-4.ckpt exists, but a control file(*.aria2) does not exist. Download was canceled in order to prevent your file from being truncated to 0. If you are sure to download the file all over again, then delete it or add --allow-overwrite=true option and restart aria2.

10/08 23:59:58 [NOTICE] Download GID#7bf56e8f8695ad37 not complete: /storage/models/sd-v1-4.ckpt

Download Results:
gid   |stat|avg speed  |path/URI
======+====+===========+=======================================================
2f74a4|OK  |       0B/s|[MEMORY][METADATA]3a4a612d75ed088ea542acac52f9f45987488d1c
7bf56e|ERR |        n/a|/storage/models/sd-v1-4.ckpt

Status Legend:
(OK):download completed.(ERR):error occurred.

aria2 will resume download if the transfer is restarted.
If there are any errors, then see the log file. See '-l' option in help/man page for details.

Download must have failed at some point. Glad you figured it out.

there's new scripts to integrate on the wiki, also, would you like an invite to Auto's discord?

Sure, I'll take a look. Send it to Cyberes#6500

@Cyberes i've tried sending you an invite but you aren't on any of the SD discords so you have to accept the friend request to get the invite lmao


Only show SSH clone URL if signed in (#2169)

This PR targets issue #2169
For cloning via SSH a user needs to provide a public key.  Anonymous visitors hence do not need to see the SSH URL.
This is a hard-coded soliton (as voted by @lafriks), equivalent to GitHub's behavior. If a configurable solution is desired instead, I got another diofferent branch (stklcode/gitea@d7cbf5d72e15a7411ec1648037fd94872e5ed2a3) ready.
This solution introduces a config flag SSH_EXPOSE_ANONYMOUS (default true) to opt-out the current behavior.

If you have configurable solution than submit that

Done. Updated my branch and the above description.

Also integration test would be nice to check that

I hope you don't mind that I added integration tests, thanks for your first PR :+1:

Great job :+1: LGTM

LGTM


Don’t autoupgrade to PromiseKit 2.0

When PromiseKit 2 is released (soon {ish}) it would be better you check everything is good in your pod spec rather than have users get auto-upgraded. This dependency modifier will ensure your users will not upgrade before you have checked everything is good. x

You may also want to specify:
s.default_subspecs = 'Adapter'

Because otherwise users of your project will always also import PromiseKit.

CLOSING BECAUSE: FFS


Documentation: Add instructions regarding the usage of ADS CLI's

Currently, the documentations only lists out detailed instructions on how to use axondeepseg and axondeepseg_test CLI.
The documentation seems to be lacking the instructions about the remaining CLI's that we do use in the project :-

download_models
download_tests

It would be nice to have information about the remaining ADS CLI's  in the RTD for the developers, contributors or even users.
PS: Perhaps we could touch on this issue after ivadomed integration

download_tests shouldn't really have a public API, as it's only intended to be called during installation. We could clarify that.

download_tests shouldn't really have a public API, as it's only intended to be called during installation. We could clarify that.

download_tests shouldn't really have a public API documentation, as it's only intended to be called during installation. We could clarify that internally for developers.

yeah, I agree that download_tests CLI doesn't need to added in RTD. For this CLI, we could add instructions on WIKI .

download_tests shouldn't really have a public API documentation, as it's only intended to be called during installation. We could clarify that internally for developers.

yeah, I agree that download_tests CLI doesn't need to added in RTD. For this CLI, we could add instructions on WIKI .

Has been completed


Upgrade Terraform Azure provider to v3

Our TF code uses version 2 of the existing Azure TF provider. We need to upgrade to take advantage of the latest and greatest features!
This will likely require a refactor of several elements. A lower-level environment, such as dev3, can be used for the initial testing process.
Completion of this story will be required to enable Dependabot for our TF code.

Will be a HUGE refactor; required because providers are continuing to add new features that Azure V2 no longer supports.
Terraform equivalent of the Spring Upgrade :(

Per convo with Rin in slack, this ticket will also have fixing dev1 as an acceptance criteria


JAX-RS: Support custom HTTP verb annotations

JAX-RS annotations do not represent all the HTTP verbs,  for example @java.ws.rs.PATCH is missing.
A custom annotation may be created and it must be annotated with @javax.ws.rs.HTTPMethod("<VERB>") annotation.
swagger-core has its own implementation, @io.swagger.jaxrs.PATCH. However, most of us have created our own versions as well. The plugin must support them.

I have a fix for this. Will submit a PR shortly.

@who, @kongchen - submitted PR to resolve this.


@mzikherman - Show the 4-up editorial mixin when there are 4 or more articles

          1 Warning

  :warning:
  Please include a PR summary

  Generated by :no_entry_sign: danger

Danger'd!

lol, updated.


Do not allow linking to Wikipedia/Wikidata disambiguation pages

It would be useful if iD did not allow linking to these pages.

@nicolasmaia Thanks, this is #4414 🙂


Add specific errors for missing kind and version.

Partially addresses #1908

When would this happen?
IMO, we should be automatically setting kind and version by default since we have the information from the URL path.

It can happen client side with incomplete yaml files.
Filling in with defaults from paths is fine with me too.
Let's either merge this PR or close the open issue requesting this.
Brendan
On Feb 24, 2015 11:10 PM, "Brian Grant"<EMAIL_ADDRESS>wrote:

When would this happen?
IMO, we should be automatically setting kind and version by default since
we have the information from the URL path.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/4802#issuecomment-75915629
.

Ok, this is independently useful.

Comment addressed.

LGTM


Support sentry/sentry 3.5

https://github.com/getsentry/sentry-php/pull/1316 is preparing 3.5.0 for the base SDK. This includes two new options (http_connect_timeout and http_timeout) and we need to support those; also, it would be useful to notify users of the small BC on the send_attempts default value.

4.3 released, which includes #593.


podnapisi subtitles by dexter21767

Addon Manifest URL
https://2ecbbd610840-podnapisi.baby-beamup.club/manifest.json
Addon Description
Addon for getting subtitles from podnapisi, by Dexter21767.

Doesn't work. It's listed in the subtitles selection but they don't show up during the content stream.


Add the awaited attribute to CoroutineMock.

Refs #64.

@Martiusweb despite the incompatibility with 3.4 (which I will fix) what do you think?

Yes, this is something that should be implemented.
We can add the method assert_awaited(), but it can be done in an other PR.
Do you have examples of cases where an Event is useful for awaited (rather than a simple boolean)?
It can be a good idea but it can be confusing knowning that unittest.mock.Mock.called is a boolean: for instance, someone will probably do something like:
self.assertTrue(coro_mock.awaited)

which will always be ok. I think it should be renamed in awaited_event (or both attributes can be provided, awaited as a boolean, awaited_event as an Event object).
About the exception: the way you implemented it is right. Since you check that the returned result is awaitable, the exception will be raised as the result of the awaited call, not because the object is not awaitable. Thus, I think that we can safely say that the CoroutineMock has been awaited even if an exception is raised.

I cannot share the entire project, but the use case is for tests whose behavior require to do something at very specific steps of execution flow.
E.g. when a test involves 2 coroutines (client and server) and you want to send some data after server authorizes client. In that test one of server's methods is wrapped with a mock and then test awaits that wrapped mock to be awaited. That is a signal that test can proceed.

It can be a good idea but it can be confusing knowning that unittest.mock.Mock.called is a boolean

What if we replace asyncio.Event with a custom subclass:
class Event(asyncio.Event):
    def __bool__(self):
        return self.is_set()

I think it should be renamed in awaited_event

The reason I did it this way was to make it closer to called / call_count.


[Fix] close PIL images when loading images to tensor/numpy

The previous behavior opened PIL images, but didn't release their memory, causing a huge memory leak on our training pipeline.
With this PR, the PIL images will be closed immediately after the copy as numpy array or torch tensor is created.

Hi @helpmefindaname :wave:,
Tanks for the fix good catch :+1:

failing build / script jobs are expected atm so don't wonder :sweat_smile:


Improve feature

Short description of what this resolves:
just changed (Vimeo) to (Video)
i know this is a silly change but i am just a beginner
this tiniest contribution means a lot to me
Changes proposed in this pull request:

Fixes: #

I corrected a mistake in readme.md  file


Update README.md

Minor grammar correction

Really happy to see this list, great work!

thanks for spotting the mistake!


NodeJS: Use Copilot Chat and not copilot labs

@microsoft-github-policy-service agree

Thanks for the update, please fix the typo and make sure all exercises are now based on the use of Copilot Chat.

@cmendible :

I have added back the labs that are using the sample.txt that was missing
I fix the typo on language at the beginning, let me know if you have found more.

Note: my teammate @Msanzdelrio  is working on the dotnet labs

Thanks for the update, please fix the typo and make sure all exercises are now based on the use of Copilot Chat.

@cmendible :

I have added back the labs that are using the sample.txt that was missing
I fix the typo on language at the beginning, let me know if you have found more.

Note: my teammate @Msanzdelrio is working on the dotnet labs

I have already submitted my PR and has already been merged :)

@Msanzdelrio @tgrall Thanks for contributing!


Link all of the PHP Pandas sections

https://daylerees.com/php-pandas/

I have glanced at this task so many times now.
I really like Dayle's way to write, but I'm really not sure where it would fit, except as a book resource connected to a skill. But we don't do resources under skills, right?
I think that could be valuable because there are many resources that has a broader perspective than the modules we are creating. It's the same thing with php the right way.

@rockymontana Wouldn't sections like this: https://daylerees.com/php-pandas-functions/
Be good to add to the basic PHP section?

Most definitely!
We talked briefly about this a couple of days ago, but I’ll clarify my train of thoughts here.
I think we see the skill feature somewhat differently and I think that’s the main thing we stumble upon when we’ve been talking about it.
I see a skill as a higher level content type than you do I think. I see it as something that you can visit aside from the modules and tracks. That’s how I justify it having it’s own sets of resources. That way I think we cover a wider audience (meaning those that likes learning by reading entire books).
By doing that (listing learnable skills) the user would also be able to see what kinds of skills are available and could work as an on-boarding mechanism.
Does that make sense?

Hmm yah, I think "should resources attach to skills or only modules" should be a separate issue.

@mattstauffer do we want to link to each section individually as a resource? Or just link to the main site mentioned in your first comment?

@TammyTee If the whole site fits in one module, it could be a single link. If not, we may need to link them all separately.

This has been added to the Beginning PHP module.


[Global] Links open a new window without warning

Issue summary
Links which request and external resource open a new window/tab/app without warning the user. This could confuse visually impaired users who believe they are following a link in the same browser/app window which causes extra effort to switch back to the previous window/app.

Screenshots

Current code
TSX
// InfoShareView.tsx

const InfoShareItem = ({onPress, text, icon}: InfoShareItemProps) => (

// …

<TouchableOpacity onPress={onPress}>
  // …
</TouchableOpacity>

// …

<InfoShareItem onPress={onSymptomps} text={i18n.translate('Info.CheckSymptoms')} icon="icon-external-arrow" />

Steps to reproduce

Load the app into the iOS simulator
Expand the Sheet content
Enable VoiceOver screen reader on macOS
Use the Virtual Cursor to navigate through the Sheet controls

Behavior
Expected

Links which open a new window/app will be accompanied by a visible and audible warning.

Actual

Links open a new window without any warning.

Recommendation
For each link in the app which opens an external resource:

Add the accessibilityLabel prop, setting its value to the control {text} (This is required for accessibilityHint)
Add the accessibilityHint prop, setting its value to a string which warns the user audibly
If no icon is present, include a "new window" icon to convey the warning visually

Recommended code
TSX
// InfoShareView.tsx

const InfoShareItem = ({onPress, text, icon, role, hint}: InfoShareItemProps) => (

// …

<TouchableOpacity onPress={onPress} accessibilityLabel={text} accessibilityHint={hint} accessibilityRole={role}>
  // …
</TouchableOpacity>

// …

<InfoShareItem onPress={onSymptomps} text={i18n.translate('Info.CheckSymptoms')} icon="icon-external-arrow" role="link" hint="opens in a new window" />

Specifications

Component: CovidShield [Global]
WCAG Principle: Understandable
WCAG SC: 3.2.2 On Input
Severity: High
Effort: High

hi @svinkle , what should the voice over text say ? I am writing now providing the Title e.g.g ("Together, let's slow the spread of COVID-19")


Documentation is broken

https://scvi.readthedocs.io/en/master/notebooks/annotation.html

This is because we moved the notebooks from docs/ to tests/

@romain-lopez  Where did you find that link? I updated the readme to point to the new notebooks, but I excluded the annotation notebook because it's not really done yet.

I think that was referenced by Google somewhere. Maybe It'll just disappear. Ok let's stick to these new links in the readme


Should our list based results provide an entry of "error" data sets

Or perhaps a separate list.
In reviewing #865 it turns out the very first element of the NAB rating templates causes the error, thus zero results are returned.
This would make it rather difficult to fix anything. If the user was aware and had permissions they could correct the element. But all they get in that case is "not valid because X" but which rating template isn't identified at all.
This is possibly true for any other data set, while that shouldn't be able to happen, we do have a lot of historical data from before certain constraints or defaults were appropriately applied.

FYI, for awareness of the thought. @rma-psmorris @adamkorynta @RyanM-RMA @rma-rripken @rma-bryson @DanielTOsborne @krowvin

Ratings are one of the few API's that do validation of the data returned by the database before sending to the client, and this is due to the fact that it uses the hec-cwms-ratings libraries instead of basic DTO's. These API's are an outlier to me unless we decide to add more business logic to CDA.
The normal DTO required field validation is only called on POST requests to short circuit logic before hitting the database and give more meaningful responses to the client. If we add that to list based retrieval that would add overhead for each element in the list (which may be worth it?).
Last bit of context, there is some precedence for this issue in the CWMS schema for ratings. The cwms_rating.store_ratings_xml method does return a clob with descriptions of ratings that failed to store.

In the particular case where this came up, the error was well before it would've gotten to a DTO. It was nested down in the DAO.

Oh wait I see what your saying. I still don't think that's quite the issue. But likely part of it. In the case of the data in #865 it was the first rating in the result set that was incorrect and thus the user gets no data.
I'm pretty sure that data is somehow wrong in the database, but if not, the validation is wrong.

Right, that's because we are going through the hec-cwms-ratings API, where the validation takes place, which is atypical for all of our other endpoints. So it seems like this is a good idea for this particular endpoint, but we'd need to start adding more validation to other GET endpoints to make it worth adding to the others.

The particular error throw is actually in the CDA code: https://github.com/USACE/cwms-data-api/blob/fbce50d6abbdb74e15caac28a5020b4e0c24c3b7/cwms-data-api/src/main/java/cwms/cda/data/dao/RatingTemplateDao.java#L212

Oh I see! That still seems like an outlier to me, searching across the DAO classes for throw new that seems to be one of the few validations done on a GET request. Of course that doesn't count general non-thrown RuntimeExceptions but those are just CDA bugs that shouldn't be returned to the client and should be analyzed through an enterprise logging solution (or manual log analysis) where we could keep tabs on unexpected exceptions. (as a side note, I think we should lower the log level from WARNING for 404 errors so that it doesn't get reported at the same level as real bugs).
I think I'd like to see 2 or 3 other places that we would use an errors field given the current codebase before we add to a base class. Either that or if we agree to start adding more CDA validation on GET requests (we'd probably want some performance metrics on that).

Yeah, definitely an edge case. The other question is would does that particular validation even matter in the grand scheme of things.
But it general if only a single element of returned data has an error, we probably shouldn't bail on everything, but the user needs to know something is wrong; or at least an administrative user does.

That's what all our API's currently do since we generally just return the data as-is from the database, then the client decides how to return individual row failures. Maybe some of the package calls that return UDT's have validation and internal schema failures would report for all the rows.

Okay, then #865 is definitely an outlier as it isn't doing that.


Cancel swap does not work

Created an order -> waited 5 minutes -> selected cancel -> the order remained open
Additional note: The swap would cancel when I close the app but there was only a small time frame that the confirmation window was open.
Version: alpha 13
OS: win 10
Errors: none
Severity: Medium

Can reproduce this.
@jdmarlow86 The cancel behaviour in marketmaker is quite buggy, you can read more on it here:

https://github.com/jl777/SuperNET/issues/844
https://github.com/atomiclabs/hyperdex/issues/129
https://github.com/atomiclabs/hyperdex/issues/262#issuecomment-395390245
https://github.com/atomiclabs/hyperdex/issues/345

It seems temperamental and normally works, but sometimes it doesn't. We were aware the cancel behaviour was pretty buggy which was why we removed it previously, but we needed to add it back in when adding good 'till cancelled (GTC) orders.
To clarify, the bugginess is in marketmaker, not HyperDEX, we don't get reliable information back from the API to know what's happening.
In my reproduction scenario, I get this response from marketmaker when cancelling an open order:
{"result":"success","numentries":0,"status":"will stop trade negotiation, but if swap started it wont cancel"}

Which the UI treats as a cancel being issued so blanks out the cancel button.
However mm never cancels the trade internally so we don't get a cancel update message for the trade over the socket to it remains listed as "open".
The text in the message a little ambigious, it's not clear what "started" means

will stop trade negotiation, but if swap started it won't cancel

But I can see in my instance there hasn't been any network activity at all, not even any request or reserved messages, so it would appear this trades hasn't "started" and should definitely of cancelled.
We will likely have to wait for mmv2 for stable cancel functionality.
// cc @jl777 @artemii235 @ArtemGr

mm1.0 related


AcquireTokenSilent Behavior with Apollo Client setContext

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.23.0
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
1.3.2
Description
Hello,
I am using Apollo Client to manage the state of my react app.
However, when I want to acquire a token to authenticate my requests, it results in infinite loop of requests from Apollo Client.
I tried to debug the code, and it appears that this happen when I call instance.acquireTokenSilent.
But all those requests are authenticated. I just don't know why there are infinite requests, knowing that without the acquireTokenSilent call, and if I put the token directly into the authorization header, everything works fine.
MSAL Configuration
auth: {
    clientId: "clientId",
    authority:
      "https://login.microsoftonline.com/tenantId",
    // redirectUri: "http://localhost:3000/",
    navigateToLoginRequestUrl: true,
  },
  cache: {
    cacheLocation: "localStorage", // This configures where your cache will be stored
    storeAuthStateInCookie: false, // Set this to "true" if you are having issues on IE11 or Edge
  },

Relevant Code Snippets
const AsyncTokenLookup = async () => {
      const accounts = await instance.getAllAccounts();

      const account = get(accounts, "[0]");
      if (account && inProgress === "none") {
        try {
          const result = await instance.acquireTokenSilent({
            ...loginSilentRequest,
            account,
          });
          return result.accessToken;
        } catch (err) {
          if (err instanceof InteractionRequiredAuthError) {
            // fallback to interaction when silent call fails
            return instance.acquireTokenRedirect(loginRequest);
          }
        }
      }
      return;
    };
    const withToken = setContext(async (_, { headers }) => {
      const my_token = await AsyncTokenLookup();
      return {
        headers: {
          ...headers,
          Authorization: my_token ? `Bearer ${my_token}` : null,
        },
      };
    });

Identity Provider
Azure AD / MSA
Source
External (Customer)

I went back to "@azure/msal-browser": "2.14.2" and "@azure/msal-react": "1.0.0" and the issue is gone. I think the behavior of acquireTokenSilent was changed and didn't work well with Apollo

@othpwn please provide verbose MSAL logs when it loops and when also when it doesn't

Here is the log I got, it keep looping between msal:acquireTokenStart msal:acquireTokenSuccess:
[Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:handleRedirectStart
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT] : [fbea0831-737d-42db-8b24-b9ce26905408]<EMAIL_ADDRESS>: Info - handleRedirectPromise called but there is no interaction in progress, returning null.
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:handleRedirectEnd
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - MsalProvider - msal:handleRedirectStart results in setting inProgress from startup to handleRedirect
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - MsalProvider - msal:handleRedirectEnd results in setting inProgress from handleRedirect to none
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenStart
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenSuccess
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenStart
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenSuccess
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenStart
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenSuccess
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenStart
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenSuccess
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenStart
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenSuccess
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenStart
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenSuccess
auth-config.js:26 [Tue, 12 Apr 2022 17:34:05 GMT]<EMAIL_ADDRESS>: Info - Emitting event: msal:acquireTokenStart

Indeed, it works with 1.3.0 but not the latest.

Thanks for confirming, will close as a duplicate of #4701.


Unit inside the  tag

How to proceed with 92°.5?
Is it ok to embed a <measure> tag in a <num> tag?
<measure type="value"><num>92<measure type="ANGLE" unit="°">°</measure>.5</num></measure>

?

I think this should be commented and ignored for the moment. If I'm not mixing up it's one of the cases where the flat structure is not allowing us to recognise back the tagged stream.

I agree!

This can be actually added directly to the documentation, and leave the issue open ;-)

What to do with
<measure type="value"><num>126</num></measure> withdrawals out of <measure type="value"><num>162</num></measure> riders

Is "withdrawals" considered kind of a unit (then we would comment the sentence)?

I think it's more a substance, like people, I'd comment the whole sentence (or, we could also remove withdrawals and live happily)


Allow "instance" private and protected class members that don't affect assignability

Suggestion
Many classes use private and protected members for internal use only - that is, they always access them via this.field and never on other object with other.field.
For these classes it would be very useful to retain structural typing, instead of the effectively nominal typing you get with private and protected members currently.
This would be especially useful for avoiding compilation errors when programs have multiple copies of the same package installed (for the usual npm reasons) with classes that are actually compatible with each other, but aren't assignable according to TypeScript.
Could we add an instance modifier to class members that restricts the fields to only being access with this.?
Example:
class A {
  instance private _x = 0;
  get x() { return this._x; }
}

class B {
  instance private _x = 0;
  get x() { return this._x; }
}

const f = (a: A) => {}

// Should not be an error
f(new B());

🔍 Search Terms
instance private assignable
✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
Add an instance modifier that enforces that the member is only accessed with this.:
class A {
  instance private _x = 0;

  get x() {
    // fine
    return this._x;
  }

  getOtherX(o: A) {
    // error
    return o._x;
  }
}

📃 Motivating Example
Besides the common case using a class type as an interface, it's far too easy to end up in situations where users have multiple copies of a package and pass objects from one copy to another:
node_modules/foo/index.ts:
export class A {
  instance private _x = 0;
  get x() { return this._x; }
}
export const getX = (a: A) => a.x;

node_modules/bar/node_modules/foo/index.ts:
export class A {
  instance private _x = 0;
  get x() { return this._x; }
}
export const getX = (a: A) => a.x;

node_modules/bar/index.ts:
export {getX} from 'foo';

src/app.ts:
import {A} from 'foo';
import {getX} from 'bar';

// This causes a compile error but no runtime error:
getX(new A());

💻 Use Cases
Workarounds
There are some workarounds, but they are finicky and don't work completely.
Interface<T> helper:
You can define a helper to get just the instance interface of a class like:
type Interface<T> = {
  [K in keyof T]: T[K];
}

But using it isn't trivial:
export class AImpl {
  private _x = 0;
  get x() { return this._x; }
}
export const getX = (a: A) => a.x;

export type A = Interface<AImpl>;
export const A = AImpl;

Unfortunately, the exported type A doesn't fully represent the public interface of class AImpl. It doesn't have the static interface and it doesn't have the protected members which may be necessary for subclasses (and can still follow the rule that they are accessed only on the this instance).
/** @internal */
You can mark private members as /** @internal */ to exclude them from the published public types. This does not work for protected members though.

+1
Its an issue I've been struggling with for a while now due to the setup of a project I'm working on (see details below).
I thought I was misunderstanding the "structurally typed" system for TS but I am now aware this is intentional behaviour (e.g. from this), however it would be good to get some examples to clarify what issues this is avoiding when the internal members are the same shape?
Maybe there are cases where knowing an instance comes from a particular parent via its internal members but since TS private/protected members are all public at runtime, its a bit heavy-handed for this to cause errors in all cases as some code is not written in such a way that the internal members coming from a particular parent class is a factor (especially if the internal members have an identical shape/type).
A way to opt out of this on a case-by-case basis would be good, however, I think a general compiler option would also be good to opt out of this and compare classes fully by structure only.
My Use Case
I have some packages where I need to symlink package A (e.g. a common dependency) into package B (e.g. an app) to be able to develop or test changes to package A as part of a real system.
Now let's say Package A and B both have another common dependency Package C and when the symlinking is done as above Package C gets duplicated so we have Package C in the node_modules for both Package A and B. In this case TS resolves Package C from different places and treats it as separate code (even if its exactly the same) and it shows compatibility issues for the private/protected members (which are exactly the same) in some parts of the code where Package A and B overlap using types from Package C.
Currently, I am having to add a lot of // @ts-expect-error comments to get around this so the code actually compiles and I can test the actual behaviour I'm interested in, which usually has nothing to do with Package C. Then when I'm done, I re-install the latest released Package C from the package registry and TS stops complaining then I have to remove the all the expect error comments. Then repeat again for the next change to Package C.
In some cases, Package C is something used in multiple variations of packages so this pain is amplified. Currently, I've built some logic into the build system to ignore some errors which can be reliably identified as being from this behaviour automatically but there are still unreliable cases where I have to manually add some expected error comments.
So TS is making this particular workflow awkward and the reason why is not clear to me.