Roll engine c02dd8f935a4..cebde437a1b1 (15 commits)

https://github.com/flutter/engine/compare/c02dd8f935a4...cebde437a1b1
git log c02dd8f935a4..cebde437a1b1 --no-merges --oneline
cebde437a Guard the service protocol's global handlers list with a reader/writer lock. (flutter/engine#6900)
b6e93759f Revert "Guard the service protocol's global handlers list with a reader/writer lock (#6888) #6895" (flutter/engine#6899)
b44b94eea Roll src/third_party/skia 66b91e4afb88..def9bcecd994 (1 commits) (flutter/engine#6898)
25db6e082 Roll src/third_party/skia d017e5138c57..66b91e4afb88 (1 commits) (flutter/engine#6897)
fd0911cc0 Guard the service protocol's global handlers list with a reader/writer lock (#6888) (flutter/engine#6895)
fffcce47f Roll src/third_party/skia 1bfdd1adfa22..d017e5138c57 (1 commits) (flutter/engine#6896)
a01c82e9d Roll src/third_party/skia 7e9ea391cfce..1bfdd1adfa22 (1 commits) (flutter/engine#6894)
9ba556156 Revert "Guard the service protocol's global handlers list with a reader/writer lock (#6888)" (flutter/engine#6893)
517e52548 Roll src/third_party/skia 73b4a1f57254..7e9ea391cfce (1 commits) (flutter/engine#6892)
9352360c8 Guard the service protocol's global handlers list with a reader/writer lock (flutter/engine#6888)
9296d80c8 Roll src/third_party/skia 7fb30f11a055..73b4a1f57254 (3 commits) (flutter/engine#6891)
93cbf60b6 Roll src/third_party/skia 281224020a57..7fb30f11a055 (1 commits) (flutter/engine#6889)
aa9793f73 Roll src/third_party/skia a2187bf7629a..281224020a57 (5 commits) (flutter/engine#6887)
3978f0753 Keep a copy of each engine's description that can be accessed outside the engine's UI thread (flutter/engine#6885)
0870e3782 Roll src/third_party/skia 3b155a77c370..a2187bf7629a (11 commits) (flutter/engine#6884)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Auto-roller completed checks. About to merge.


Localize the Flash On/Off button would be better

The Flash On/Off button text is hard coded now, localize it would be better.
https://github.com/apptreesoftware/flutter_barcode_reader/blob/5d498e39a76466b03579486475ec777fba33ed2c/android/src/main/kotlin/com/apptreesoftware/barcodescan/BarcodeScannerActivity.kt#L39
https://github.com/apptreesoftware/flutter_barcode_reader/blob/5d498e39a76466b03579486475ec777fba33ed2c/android/src/main/kotlin/com/apptreesoftware/barcodescan/BarcodeScannerActivity.kt#L43
https://github.com/apptreesoftware/flutter_barcode_reader/blob/5e360fc9dad8f3525e75aaaf6e7d64490759eda2/ios/Classes/BarcodeScannerViewController.m#L93
https://github.com/apptreesoftware/flutter_barcode_reader/blob/5e360fc9dad8f3525e75aaaf6e7d64490759eda2/ios/Classes/BarcodeScannerViewController.m#L97

+1 on the above. I would even suggest an easier IMO approach - rather than localize the button, it can be replaced with icon pair Flash on / off.

would be really awesome!!

+1

+1

+1

+1

+1 on the above. I would even suggest an easier IMO approach - rather than localize the button, it can be replaced with icon pair Flash on / off.

I replaced flash toggle title by two icons in my fork.

As mentioned here https://github.com/mintware-de/flutter_barcode_reader/issues/131#issuecomment-588521486 I need to rewrite the platform communication. After that it's possible to use custom strings.

Soon :-)

Feel free to test: https://github.com/mintware-de/flutter_barcode_reader/pull/185

https://pub.dev/packages/barcode_scan/versions/3.0.0-dev.1


Improve installation

The current way of installation by running ./setup.sh has a couple of issues:

It assumes that a macOS user already has homebrew installed. This is not always true - for example, I use macports and not homebrew This means that the installation fails for me.

It also installs all the python packages in the main python environment. This is not ideal since it might cause conflicts with package versions that might be there for a reason.

I would recommend that you revamp your installation instructions to explicitly list out the various setups that are in ./setup.sh to make it easier for users to see what they are doing, to allow using macports instead of homebrew, and to include a step to create a virtual environment before installing the various python packages.
If you can do the latter using conda, that would be even better since the conda-forge channel there might even include some of the non-python dependencies (ffmpeg, pkg-config) which means the user does not need to worry about causing conflicts there either and those can be contained in the virtual environment as well.

I believe this is completed now. Thanks!


Pinephone/Pinephone Pro Support

Just saw librem5 support added so figured i would mention bringing in support for the Pine64 mobile phone devices. There is currently support for the Pinephone and there is a PR for Pinephone Pro in the mobile-nixos repo but it might be better to bring in support to the official nixos hardware instead of just leaving it there.
One of the big benefits i see in their repo is they have built their own installer so its possible to build an image and install it on to the sd card and the installer will prompt for all the information and flash it onto the emmc.

Mobile NixOS is a bit more than that,
What Mobile NixOS brings to your everyday system is those three main things:

hardware-specific knowledge (which could interchangeably live anywhere)
bootloader/generations problems workarounds
touch-based stage-1 implementation

The second point includes, for the Pinephones (and would help with the Librem 5) a generation selection mechanism for stage-2 that is usable without requiring access to serial.
Why would you want a touch-based stage-1 implementation? For LUKS passphrase input for FDE.
Though, I don't think anyone should see this a preventing them to try and do things in other ways. They are only one solution to different problems.
Though note that some parts can be used independently. For example, I use the touch-based stage-1 implementation on my Steam Deck so I can input the FDE passphrase on boot without requiring a physical keyboard.

Thats all good information, i love the installer you have created since it makes it easier to get started on a mobile device like the PP. All three of those benefits that mobile NixOS sound great and seems like a great reason for Mobile NixOS to exist and continue existing but im not sure if hardware configurations should exist there permanently atleast for the more linux focused devices such as the PP/PPP and/or librem devices.
We can add the configurations on here and then point to Mobile NixOS for documentation/instructions for getting an image with the installer and with all the other benefits that Mobile NixOS provides that you mentioned.


syscall: cannot use s.Rdev (type uint32) as type uint64 in field value on mips

What version of Go are you using (go version)?
go1.7.rc2

What operating system and processor architecture are you using (go env)?
[root@localhost ~]# go env
GOARCH="mips64le"
GOBIN=""
GOEXE=""
GOHOSTARCH="mips64le"
GOHOSTOS="linux"
GOOS="linux"
GOPATH=""
GORACE=""
GOROOT="/usr/lib/golang"
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_mips64le"
CC="gcc"
GOGCCFLAGS="-fPIC -mabi=64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build685506292=/tmp/go-build"
CXX="g++"
CGO_ENABLED="1"

What did you do?
My PC is a mips64le pc.   When I compile the docker, I encountered a problem:
this is the error message:
docker/pkg/system/stat_linux.go:13: cannot use s.Rdev (type uint32) as type uint64 in field value

3.1 In go/src/syscall/syscall_linux_mips64x.go, I see that
type stat_t struct {
    Dev        uint32

3.2 On my pc RUN: man 2 stat, I see that
struct stat {
    dev_t     st_dev;

3.3 Then I wrote a test program.
#include <stdio.h>
#include <sys/types.h>

int main()
{
    printf("sizeof(dev_t) = %d\n", sizeof(dev_t));

    return 0;
}
---------------------------
sizeof(dev_t) = 8

Weird. @cherrymui, how did you generate that file?

The kernel's layout is (e.g. https://github.com/torvalds/linux/blob/master/arch/mips/include/uapi/asm/stat.h#L92)
#if _MIPS_SIM == _MIPS_SIM_ABI64

/* The memory layout is the same as of struct stat64 of the 32-bit kernel.  */
struct stat {
	unsigned int		st_dev;

So I believe uint32 is correct. Go invokes syscalls directly to the kernel, so we should use kernel's layout instead of libc's.

@cherrymui ， thx， I see.

@ka1em  How did you fix the problem at last?  Now I am building  kubernetes os mips64el/Fedora 21  and getting the same error shown as follows. Any suggestions for me? Many thanks.
k8s.io/kubernetes/vendor/github.com/docker/docker/pkg/system
k8s.io/kubernetes/vendor/github.com/google/cadvisor/fs
k8s.io/kubernetes/vendor/github.com/google/certificate-transparency/go/x509
# k8s.io/kubernetes/vendor/github.com/docker/docker/pkg/system
vendor/github.com/docker/docker/pkg/system/stat_linux.go:13: **cannot use s.Rdev (type uint32) as type uint64 in field value**
# k8s.io/kubernetes/vendor/github.com/google/cadvisor/fs
vendor/github.com/google/cadvisor/fs/fs.go:422: cannot use buf.Dev (type uint32) as type uint64 in argument to major
vendor/github.com/google/cadvisor/fs/fs.go:423: cannot use buf.Dev (type uint32) as type uint64 in argument to minor
# k8s.io/kubernetes/vendor/github.com/google/certificate-transparency/go/x509
vendor/github.com/google/certificate-transparency/go/x509/x509.go:342: undefined: elliptic.P224
vendor/github.com/google/certificate-transparency/go/x509/x509.go:355: undefined: elliptic.P224
vendor/github.com/google/certificate-transparency/go/x509/x509.go:1461: undefined: elliptic.P224
Makefile:79: recipe for target 'all' failed
make: *** [all] Error 1

Platform info:
[root@dscn2 kubernetes]# uname -a
Linux dscn2 4.4.21+ #3 SMP PREEMPT Wed Oct 12 09:14:23 CST 2016 mips64 mips64 mips64 GNU/Linux
[root@dscn2 kubernetes]# go env
GOARCH="mips64le"
GOBIN=""
GOEXE=""
GOHOSTARCH="mips64le"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/root/mygo"
GORACE=""
GOROOT="/usr/lib/golang"
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_mips64le"
CC="gcc"
GOGCCFLAGS="-fPIC -mabi=64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build797767121=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"


Feature request: Blacklisting

Currently we have a whitelist, but I want to use this plugin the other way around; I would like an inverse activation mode.
In other words, I would like to have the plugin active at all times, but be able to blacklist sites.
The sites on the black list would go dark mode as specified by the extension.
Example: Enable extension > all sites unmodified > blacklist pesky ultra bright white site > pesky ultra bright white site becomes legible thanks to the extension but all other sites remain normal unless blacklisted too.
Which is the opposite of how all of these extensions work.
I would like the option to invert and toggle between these two modes (whitelisting and blacklisting).
Thanks.

There is already a toggle that works exactly as you described:

Just uncheck it.


automate release process

Description
Automate the release process using GitHub Actions.
Upon pushing a tag beginning with v to napari/napari, the CI will:

copy the release notes to the tag
build the distribution
upload the distribution to the appropriate release on GitHub
upload the distribution to PyPI

@jni When I made this, I didn't realize that we stopped uploading wheels to PyPI. Just because I'm curious, what's the advantage of this? Could you make a suggestion to this PR so that it does not build the wheels? Sorry, I'm not super experienced at packaging! :P

huh? We should definitely upload wheels, though they should be noarch. I can't remember how to limit wheels to certain Python versions, but that should be done also.

huh? We should definitely upload wheels, though they should be noarch

Maybe I've been messing up there, my bad - glad the new process will fix that. I'm very excited about this!!


Check dummy inputs order and correct it with wrapper

Related issue: #18 and https://github.com/aws-neuron/aws-neuron-sdk/issues/642
Context
torch_neuronx.trace accepts tuples as inputs for the tracing, whereas we need to ensure that the dummy inputs tuple corresponds to the order of model.forward().
This PR will fix the previous unusual behavior of Flaubert and XLM models (where token_type_ids was mistaken as langs during the compilation).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Peer Dependency issues with React18

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

should install on npm i
Current Behavior

Steps to Reproduce (for bugs)

install react 18 (latest)
install react-swipeable-views

Context

Your Environment

Tech
Version

react-swipeable-views
0.14.0

React
18

platform
VS code

etc

Use npm i react-swipeable-views --legacy-peer-deps.

This is hardly acceptable solution, we have to change whole CI pipeline to enforce --legacy-peer-deps flag for npm ci and npm i commands, change documentation for running project, seems like very undesirable workaround.


Use conda-lock

https://conda.github.io/conda-lock/

conda-lock now supports preprocessing selectors for doing OS-specific locking files.

But, conda-lock doesn't work particularly great with pip dependencies: https://conda.github.io/conda-lock/pip/
...
  File "C:\Users\schmiedt\miniconda3\envs\neuro-conda-2023b\lib\site-packages\conda_lock\_vendor\poetry\puzzle\solver.py", line 65, in solve
    packages, depths = self._solve(use_latest=use_latest)
  File "C:\Users\schmiedt\miniconda3\envs\neuro-conda-2023b\lib\site-packages\conda_lock\_vendor\poetry\puzzle\solver.py", line 241, in _solve
    raise SolverProblemError(e)
conda_lock._vendor.poetry.puzzle.exceptions.SolverProblemError: Because -dummy-package- depends on nitime (=0.10.*) which doesn't match any versions, version solving failed.
(neuro-conda-2023b) PS C:\Code\neuro-conda>


Create ability to add a file back in after excluded from .gitignore

From the discussion on this issue: https://github.com/sdras/project-explorer/issues/8, we need to have a way to add files back in that are excluded in gitignore.

fixed!


"EMBED" and "DEMO_EMBED" mis-compatibilites

Differences can be noted in gif above.
Issue is present in all three widget types.

This can be resolved by using the squatch-container-popup class instead of the squatch-container-embed class to do the popup styling. For instance here: https://github.com/saasquatch/dollar-credit-theme/blob/ab52a5ab01299bdc3a8a734e6d553482c84cfef2/assets/css/_theme.less#L84-L86


repeated wording

Description
The wording appears to be repeated and does not make sense.
Pages to Update
Add a service with an existing certificate to API ML on z/OS
Screenshots
Ensure that your service has its own certificate. If it was signed by intermediate CA all intermediate CA certificates ensure that all certificates are in its keystore.
Expected behavior

I'm not sure what is meant; please confirm with original writer.
Additional context

@IgorCATech can you check in with your team on this one? Thanks!

Hey, guys. I must have overlooked that because I was on vacation back then. Thanks for reminding me, Ashley! Will take a look at this today.

@janan07 and @IgorCATech
hi Andrew/Igor - any chance that we can address the issue for the 1.5 release?  I tried to find similar content in the EE space but came up empty.
The topic is:  https://docs.zowe.org/stable/extend/extend-apiml/api-mediation-security.html#add-a-service-with-an-existing-certificate-to-api-ml-on-z-os
Perhaps we can change:
FROM
Ensure that your service has its own certificate. If it was signed by intermediate CA all intermediate CA certificates ensure that all certificates are in its keystore.
TO
Ensure that your service has its own certificate. If your certificates are signed, intermediate, CA certificates, ensure that all of your certificates reside in its keystore.
???????
thx,
jb

I think that the suggested rewrite changes the meaning of the sentence. The following would be more accurate, but I would still want an engineer to confirm that it's correct -
"Ensure that your service has its own certificate. If your certificates are signed by  by an intermediate Certificate Authority (CA), ensure that all of your certificates reside in the keystore for that CA."

@janan07 Maybe, you can work with devs on this issue? I've addressed it a few months ago but it didn't get any reaction.
The way that sentence was formulated confuses me. I'd rather talk to devs asking what they meant and rewrite it from scratch

Not my field, I'm afraid.  Perhaps @plavjanik could identify who's best to advise us.

The original text has been rewritten.
I would replace it with: Import the public certificate of the service to the APIML truststore.
We need to review if the section is still correct and the same error message are still displayed. I will work with @VitekVlcek-Broadcom and @jandadav to sort this out.

@jandadav I think this still needs improvement. Can you please take a look at it?

Checked that this has been addressed. See https://docs.zowe.org/stable/extend/extend-apiml/api-mediation-security/#add-a-service-with-an-existing-certificate-to-api-ml-on-zos.


blocksize and persist has impact on result

What happened:
When running read_csv in combination with persist and blocksize, results change on re-evaluation.
I am using a 10 worker dask-cluster with version 2.29.0 (I skip the cluster setup part of the script).
import dask.dataframe as dd
from distributed import wait

kwargs = dict(
    dtype={
        "payment_type": "UInt8",
        "VendorID": "UInt8",
        "passenger_count": "UInt8",
        "RatecodeID": "UInt8",
    },
    storage_options={"anon": True}
)

# First read it with 16 MiB blocksize
df = dd.read_csv(
    "s3://nyc-tlc/trip data/yellow_tripdata_2019-01*.csv",
    blocksize="16 MiB",
    **kwargs
)
df = df.persist()
wait(df)
print(df.shape[0].compute()) # -> gives 7667792

# Then with default blocksize
df = dd.read_csv(
    "s3://nyc-tlc/trip data/yellow_tripdata_2019-01*.csv",
    **kwargs
)
df = df.persist()
wait(df)
print(df.shape[0].compute()) # -> still 7667792

# Then with 16 MiB again
df = dd.read_csv(
    "s3://nyc-tlc/trip data/yellow_tripdata_2019-01*.csv",
    blocksize="16 MiB",
    **kwargs
)
df = df.persist()
wait(df)
print(df.shape[0].compute()) # -> suddenly gives 12204124

What you expected to happen:
The same result every time :-)
Environment:

Dask version: 2.29.0
Python version: 3.8.6
Operating System: coiled cluster
Install method (conda, pip, source): coiled cluster

Yikes. I can reproduce on dask/distributed master although I get a different final number. I suspect that there might be some issue with tasks having the same name even if the blocksize is different.


Tab completion with multiple selection works only on last input element

Here's an explanation gif: http://i.imgur.com/f9FCDPn.gif
It works with other tags.
Also there's a bug (may be intended?) when you press cmd+z to undo autocompletion, multiple selection resets to the first line only.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/15032022-tab-completion-with-multiple-selection-works-only-on-last-input-element?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github).

This also happens with <a> tags.


Please use a crc library that provides a pure Python implementation

Right now, Apycula cannot be used with Pyodide:
>>> await micropip.install('Apycula==0.10.0')
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/lib/python3.11/site-packages/micropip/_commands/install.py", line 142, in install
    await transaction.gather_requirements(requirements)
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 204, in gather_requirements
    await asyncio.gather(*requirement_promises)
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 211, in add_requirement
    return await self.add_requirement_inner(Requirement(req))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 300, in add_requirement_inner
    await self._add_requirement_from_package_index(req)
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 347, in _add_requirement_from_package_index
    await self.add_wheel(wheel, req.extras, specifier=str(req.specifier))
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 385, in add_wheel
    await self.gather_requirements(wheel.requires(extras))
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 204, in gather_requirements
    await asyncio.gather(*requirement_promises)
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 208, in add_requirement
    return await self.add_requirement_inner(req)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 300, in add_requirement_inner
    await self._add_requirement_from_package_index(req)
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 339, in _add_requirement_from_package_index
    wheel = find_wheel(metadata, req)
            ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/lib/python3.11/site-packages/micropip/transaction.py", line 431, in find_wheel
    raise ValueError(
ValueError: Can't find a pure Python 3 wheel for 'crcmod'.
See: https://pyodide.org/en/stable/usage/faq.html#why-can-t-micropip-find-a-pure-python-wheel-for-a-package
You can use `await micropip.install(..., keep_going=True)` to get a list of all packages with missing wheels.

This is the only problematic dependency AFAICT (numpy is supported).

Any suggestions for such a library?

https://pypi.org/project/crc/

It's only used here so the change should be simple.
https://github.com/YosysHQ/apicula/blob/fedab4cf9871184c41e10ee6cdc6b306e3eb6a21/apycula/bslib.py#L5
I'm guessing we'll need these parameters, that's the main tricky part

>>> import crcmod.predefined
>>> crc16 = crcmod.predefined.mkPredefinedCrcFun('crc-16')
>>> crc16(b"abcd")
14743
>>> calc = crc.Calculator(crc.Configuration(width=16, polynomial=0x8005, reverse_input=True, reverse_output=True))
>>> calc.checksum(b"abcd")
14743

Thanks!


Adds set_response_header; tunes read_request_header and send_response

This packs ok|value_len as the result and also doesn't copy unless the buf_limit is large enough. This also changes param names to match conventions in WASI. Doing so has some surprising benefits and prevents us having to define another function for things like determining size before reading.

re-using a buffer for multiple header reads (buf).
growing a buffer only when needed (retry with larger buf_limit).
avoiding copying invalidly large header values (buf_limit).
determining if a header exists, or its size, without copying it (buf_limit=0).

This also formalizes setting status code as actually sending the response. The handler state machine should either delegate to the next handler or send a response.
This also switches the wasm module to "http-handler" as mentioned in https://github.com/http-wasm/http-wasm-abi/pull/6

completed the Proof of Concept ABI in this PR #3 is now dependent on this


@webext-core/fake-browser don't mock the webextension-polyfill with vitest

Description
The mocking functionality no longer works.
The following is the error we encountered in the vitest.
Error: This script should only be loaded in a browser extension.
 ❯ node_modules/webextension-polyfill/dist/browser-polyfill.js:10:9
 ❯ node_modules/webextension-polyfill/dist/browser-polyfill.js:5:5
 ❯ Object.<anonymous> node_modules/webextension-polyfill/dist/browser-polyfill.js:13:3

Dependencies

"@types/webextension-polyfill": "^0.10.5"
"@webext-core/fake-browser": "^1.2.2"
"typescript": "^5.2.2"
"vite": "^4.4.9"
"vitest": "^0.34.5"
"@webext-core/storage": "^1.1.3",
"webextension-polyfill": "^0.10.0"

Steps To Reproduce
I follow the documentation with a minimal setup.
Also, provide the repository
https://github.com/kayac-chang/webext-core-fake-browser-test

"@types/webextension-polyfill": "^0.10.5"
"@webext-core/fake-browser": "^1.2.2"
"typescript": "^5.2.2"
"vite": "^4.4.9"
"vitest": "^0.34.5"
"@webext-core/storage": "^1.1.3",
"webextension-polyfill": "^0.10.0"

Can you add your lockfile to the reproduction? Otherwise I might be using different versions of these dependencies.
I haven't updated the fake-browser package for a while, so it's likely something Vitest broke, but let me check it out.

Related to https://github.com/vitest-dev/vitest/issues/3936#issuecomment-1739836844.
diff --git a/vitest.config.ts b/vitest.config.ts
index 7f3ee78..6c7b4f1 100644
--- a/vitest.config.ts
+++ b/vitest.config.ts
@@ -7,5 +7,10 @@ export default defineConfig({
   test: {
     globals: true,
     setupFiles: ["vitest.setup.ts"],
+    server: {
+      deps: {
+        inline: ["@webext-core/storage"],
+      },
+    },
   },
 });

I've updated the docs. Read them here: https://webext-core.aklinker1.io/guide/fake-browser/testing-frameworks.html#vitest
https://github.com/aklinker1/webext-core/commit/3f7187d75ad0cbf557b6d5484e367dac83670854

It works now! Thank you @aklinker1.


chore rename clone request field version

clone request field 'committed_version'  is not right,  it's visible version.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Missing C# statement?

https://github.com/ardalis/OrganizingAspNetCore/blob/f15f2cf07fde606e3037c1f9642963a267e89145/WithRazorPages/Startup.cs#L118
this will output an error on the next line (logger.LogWarning...), but it seems something is missing in the line above it

Thanks! Sorry, that was an errant merge.


COMBINE API deployment not reading shared S3 bucket configuration

This causes multiple things to fail

Creation of COMBINE archives
Export of visualizations to COMBINE archives

Here's the trace from the dev deployment. The problem appears to be that secret.env isn't mounted.
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.9/site-packages/flask_cors/extension.py", line 165, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.9/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python3.9/site-packages/connexion/decorators/decorator.py", line 68, in wrapper
    response = function(request)
  File "/usr/local/lib/python3.9/site-packages/connexion/decorators/uri_parsing.py", line 149, in wrapper
    response = function(request)
  File "/usr/local/lib/python3.9/site-packages/connexion/decorators/validation.py", line 193, in wrapper
    response = function(request)
  File "/usr/local/lib/python3.9/site-packages/connexion/decorators/parameter.py", line 115, in wrapper
    return function(**kwargs)
  File "/app/src/handlers/combine/create.py", line 161, in handler
    archive_url = src.utils.save_file_to_s3_bucket(archive_filename, public=True)
  File "/app/src/utils.py", line 80, in save_file_to_s3_bucket
    s3_bucket = get_s3_bucket()
  File "/app/src/utils.py", line 67, in get_s3_bucket
    s3_bucket = S3Bucket()
  File "/app/src/s3.py", line 12, in __init__
    self.validate_configuration(config)
  File "/app/src/s3.py", line 69, in validate_configuration
    raise ValueError('The configuration for the S3 bucket is not valid:\n  {}'.format('\n  '.join(errors)))
ValueError: The configuration for the S3 bucket is not valid:
  Storage endpoint (`STORAGE_ENDPOINT`) must be set, not `None`.
  Public storage access (`STORAGE_PUBLIC_ENDPOINT`) must be set, not `None`.
  Storage bucket (`STORAGE_BUCKET`) key must be set, not `None`.

I think I figured it out.

I removed these configuration variables from the configuration for the COMBINE API because they're redundant with the shared configuration.
The COMBINE API wasn't reading the shared configuration.
The configuration files are correctly getting mounted. We know this because it is correctly reading the bucket token and secret.
I think my confusion stemmed from thinking I had tested our recent changes, but didn't because they hadn't actually been deployed.
I changed the COMBINE API to also read the shared configuration file.

Fixed


Dependency error on Python 3.12

$ pip install git+https://github.com/OrderlyNetwork/orderly-evm-connector-

Error
python.git@794072bdf91abdfdcd373cc2d86909b7687ce63e
Collecting git+https://github.com/OrderlyNetwork/orderly-evm-connector-python.git@794072bdf91abdfdcd373cc2d86909b7687ce63e
  Cloning https://github.com/OrderlyNetwork/orderly-evm-connector-python.git (to revision 794072bdf91abdfdcd373cc2d86909b7687ce63e) to /tmp/pip-req-build-7u5lwuax
  Running command git clone --filter=blob:none --quiet https://github.com/OrderlyNetwork/orderly-evm-connector-python.git /tmp/pip-req-build-7u5lwuax
  Running command git rev-parse -q --verify 'sha^794072bdf91abdfdcd373cc2d86909b7687ce63e'
  Running command git fetch -q https://github.com/OrderlyNetwork/orderly-evm-connector-python.git 794072bdf91abdfdcd373cc2d86909b7687ce63e
  Resolved https://github.com/OrderlyNetwork/orderly-evm-connector-python.git to commit 794072bdf91abdfdcd373cc2d86909b7687ce63e
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Requirement already satisfied: aiohttp<4.0.0,>=3.9.5 in ./.venv/lib/python3.12/site-packages (from orderly-evm-connector==0.1.1) (3.10.5)
Collecting asgiref<4.0.0,>=3.8.1 (from orderly-evm-connector==0.1.1)
  Using cached asgiref-3.8.1-py3-none-any.whl.metadata (9.3 kB)
Collecting base58<3.0.0,>=2.1.1 (from orderly-evm-connector==0.1.1)
  Using cached base58-2.1.1-py3-none-any.whl.metadata (3.1 kB)
Collecting cryptography<43.0.0,>=42.0.8 (from orderly-evm-connector==0.1.1)
  Using cached cryptography-42.0.8-cp39-abi3-manylinux_2_28_x86_64.whl.metadata (5.3 kB)
Collecting ed25519<2.0,>=1.5 (from orderly-evm-connector==0.1.1)
  Using cached ed25519-1.5.tar.gz (868 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [31 lines of output]
      /tmp/pip-install-1n2gkhyf/ed25519_a7e8b5e24f114b7999637e1ad61a3fe2/versioneer.py:467: SyntaxWarning: invalid escape sequence '\s'
        LONG_VERSION_PY['git'] = '''
      Traceback (most recent call last):
        File "/root/liquidity-sniper/.venv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/root/liquidity-sniper/.venv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/root/liquidity-sniper/.venv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-wbgd1l9w/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 332, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=[])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-wbgd1l9w/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 302, in _get_build_requires
          self.run_setup()
        File "/tmp/pip-build-env-wbgd1l9w/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 503, in run_setup
          super().run_setup(setup_script=setup_script)
        File "/tmp/pip-build-env-wbgd1l9w/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 318, in run_setup
          exec(code, locals())
        File "<string>", line 115, in <module>
        File "/tmp/pip-install-1n2gkhyf/ed25519_a7e8b5e24f114b7999637e1ad61a3fe2/versioneer.py", line 1405, in get_version
          return get_versions()["version"]
                 ^^^^^^^^^^^^^^
        File "/tmp/pip-install-1n2gkhyf/ed25519_a7e8b5e24f114b7999637e1ad61a3fe2/versioneer.py", line 1339, in get_versions
          cfg = get_config_from_root(root)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-install-1n2gkhyf/ed25519_a7e8b5e24f114b7999637e1ad61a3fe2/versioneer.py", line 399, in get_config_from_root
          parser = configparser.SafeConfigParser()
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      AttributeError: module 'configparser' has no attribute 'SafeConfigParser'. Did you mean: 'RawConfigParser'?
      [end of output]

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

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

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

could u try to use pip to install, instead of from github

I am getting same error. This might be related: https://github.com/pydata/pandas-datareader/issues/969

did u try to use poetry shell?
maybe ur env issue


optimize distance restraint calculations

Optimizing distance restraint calculations for both explicit and implicit solvent simulations.

Codecov Report

Merging #122 into master will decrease coverage by 0.19%.
The diff coverage is 57.14%.


exporting projection layers of joiner separately for onnx

Sherpa-onnx requires projection layers separately. Corresponding issue is here.
"blank_id/unk_id/context_size" parameters are stored in all_in_one.onnx internally anymore.

onnx_check argument problem in actions is fixed.

onnx_check argument problem in actions is fixed.

Thanks! Will look into it later today.

Thanks!


Multiple copies of "tput: No value for $TERM and no -T specified"

I recently updated my project to Rails 3.2.0 (Cap 2.9.0). Now, with every command issued to the remote server via Cap, I get multiple (10) copies of ": tput: No value for $TERM and no -T specified". For example...

  * executing "which git"
    servers: ["<server>"]
    [<server>] executing command
<server>: tput: No value for $TERM and no -T specified

<server>: tput: No value for $TERM and no -T specified

<server>: tput: No value for $TERM and no -T specified

<server>: tput: No value for $TERM and no -T specified

<server>: tput: No value for $TERM and no -T specified

<server>: tput: No value for $TERM and no -T specified

<server>: tput: No value for $TERM and no -T specified

<server>: tput: No value for $TERM and no -T specified

<server>: tput: No value for $TERM and no -T specified

<server>: tput: No value for $TERM and no -T specified

    command finished in 528ms

This does not happen when manually issuing ssh commands, for example...
$ ssh <server> "which git"
/usr/bin/git
$ ssh -t <server> "which git"
/usr/bin/git
Connection to <server> closed.
$ ssh -T <server>: "which git"
/usr/bin/git

It does appear that the commands are successfully executed. My project DID deploy.

Control,
I had the same issue, it looks like a serverside rvm issue. Try rvm get head on your deploy targets.
--Mike

Beautiful!  Thanks much, Mike!


Fix version variables evaluation in github build

Using github variables to provide version and commit sha variables for build.
Follow-up to https://github.com/konveyor/kantra/pull/154

@eemcmullan @djzager Sorry for complications, missed not evaluating variables in github build process.


Disable Lidar point rendering when Camera sensor frames are being rendered

When a Lidar sensor is attached and visualized, the point cloud is visible in the Camera sensor's rendered frames. They shouldn't.

Moved issue to internal ticketing system.

Moved issue to internal ticketing system.


The filters are not visible in the shop by category section.in mobile view

Bagisto Version: Master
Steps To reproduce--
1. Log in as a customer in the mobile view.
2. Shop by selecting any category.
3. Try to apply per page filter.
Expected result: The filters should be visible in the shop by category section in the mobile view.
Actual result: The filters are not visible in the shop by category section.in mobile view. Please find the screenshot Here

Hi I wanted to work on this. Can you please assign me ? Thanks

Hi, I wanted to work on this issue. Can you please assign me @Sanjaybhattwebkul ?

Hi @Sanjaybhattwebkul
I made changes for filters can you please confirm it?
Before:

After

Let me know if any changes are required.

Hi @Sanjaybhattwebkul
I have some doubts while pushing this Ui changes. I downloaded project through composer. I made some Ui changes in file /bagisto/resources/themes/velocity/views/products/list/toolbar.blade.php
while pushing this toolbar.blade.php file on git it says The following paths are ignored by one of your .gitignore files:
So i am not able to commit it. Please guide me how to push changes or If I made changes in wrong files. Thanks.

ANY UPDATED @Sanjaybhattwebkul @saurav-webkul

Hi @Sanjaybhattwebkul
I have some doubts while pushing this Ui changes. I downloaded project through composer. I made some Ui changes in file /bagisto/resources/themes/velocity/views/products/list/toolbar.blade.php
while pushing this toolbar.blade.php file on git it says The following paths are ignored by one of your .gitignore files:
So i am not able to commit it. Please guide me how to push changes or If I made changes in wrong files. Thanks.

Hi @Sanjaybhattwebkul can you please guide me in this scenario ?

Hi there,
Please  follow the given steps to push the changes
1. Fork the Bagisto repository.
2. clone the Bagisto from your repository
3. Create a new branch [ git checkout -b branchName ]
4. Do the changes
5. git add .
6. git commit -m "issue #ID fixed"
7. git push -u origin branchName
8.  Create a Pull Request.

Thanks @Sanjaybhattwebkul
I forked & cloned latest project.
In local for me all filters showing are working fine in mobile view.

Is there any changes are required in this filters - mobile view ? @Sanjaybhattwebkul @saurav-webkul

Issue fixed:  https://prnt.sc/6k55-YhXMwcc


Add religion and health facilities data

@Bezzy1999
Your work on health facilities is now live on https://www.nepalmap.org
Thank you for adding the data.


[MAJOR BUG] Create breaks all resource packs (that I've tested)

Describe the Bug
Create stops all resource packs from being loaded
Here's a relevant line in the logs: Caused by: java.io.IOException: Couldn't compile vertex program (Fabric Mods, create:glowing_shader) : 0(29) : error C1503: undefined variable "cylindrical_distance"
Reproduction Steps
Download create
Try to load any resource pack
Expected Result
The resource pack should load
Screenshots and Videos
No response
Crash Report or Log
https://mclo.gs/LrCXObo
Operating System
Windows 10
Mod Version
0.4.0d
Minecraft Version
1.18.1
Other Mods
No response
Additional Context
No response

That's a really old build, try build 445 (that's the one I use).

https://mclo.gs/ANdzHeI
I tried build 463 and it crashes on startup. I'll keep running down the builds until it works

https://mclo.gs/Ahh1SF8
I tried it with the latest build. The bug is still present

latest builds are unusable, test 441 or 445

This was the one I used, and everything else worked fine. It was usable. I will try it with 445 though just to be sure.

Just tested build 445, no difference

Tested on 448 with this resource pack: https://resourcepack.net/wayukian-resource-pack/
Everything loads fine and I don't get any error.
@redstonerti could you please link the pack you used?

can't reproduce

Featuring the almighty H Pack

I just tested it in build 445 with your resource pack, and it works. What doesn't work however, is this resource pack.
fourniv6.zip
Which is very weird to be honest. All I did was upscale some textures. Everything is a multiple of two and some small details have been added. I'm going to try and test it later with more resource packs and see what happens but this is all I have for now.

Also yeah the same error happened as before. You could say that this is my fault, that I made a bad texture pack, but this error might cause problems with other packs as well.

Yeah sorry, I should have changed the title myself. Were you able to reproduce this?

Can you test this on forge too

Can you test this on forge too

We were playing on forge with this resource pack before switching to fabric. I guess I could test it again on the latest version but I am 99.99% sure that it works on forge.

I've figured out that if a resource pack defines core shaders, flywheel shaders will fail to load. This example pack seems to be made exceptionally terribly, and seems to bundle duplicates of vanilla resources, including shaders.

further investigation: the pack has a messed up include/fog.glsl shader. The fog_distance function is just missing.

to my understanding this is just a flaw in the pack.


Question: CUI (from include of .min.js) with intellisense in VSCode

I have been struggling to get intellisense working (autocomplete) in VSCode when using the construct-ui.min.js as an include in my HTML file, while using modern modules in the rest off the application.
For example, when in the code below you type: 'CUI.' the autocomplete should prompt options, like Button, ButtonGroup etc.
file: /modules/myView.js
const myView = {
    view: () => {
        return m(CUI.Button, { label: 'My button label' })
    }
}
export default myView;

This works as expected when using import * as CUI from "construct-ui"
But this then fails in the browser, since it can't resolve NodeJS module references like this and you would need a compiler (like parceljs, rollup, babel etc.) which we do not want (which will bloat the module file (that are conditionally imported by the browser)
Any thoughts on this? how to setup jsconfig.json and/or package.json and/or tsconfig.json and/or globals.d.ts or anything... cannot seem to figure this one out.

I have bundled a single typescript declaration file in v0.3.4.
Include the path to the declaration file in your jsconfig.json:
{
  "compilerOptions": {
    "target": "ES6",
    "module": "commonjs",
  },
  "include": [
    "src",
    "node_modules/construct-ui/lib/cui.d.ts" <------
  ]
}

Tkan you @vrimar  very much, this works out great! Much appriciated


Improving the scroll back to top button

@goodchai0 please assign me the task i want to work on it

If this issue has not been solved yet, can you assign it to me.


Stopping web services on default port before install.

Any chance we can disable services on the default port (8080) before installing, or switching to the desired port "{{ jenkins_http_port }}" before installation?
I know it's small, but this particular problem has been giving me headaches a while now.
My particular case is where either nginx or apache are started when trying to install jenkins.
There is a work around, but it required running the provisioning process twice in addition to manually changing the service port between running the script.
I would like to say, please, pretty please can we have this - I'm sure there's a demand for it.
Console output (for evidence):
TASK [geerlingguy.jenkins : Install our specific version of Jenkins.] **********
fatal: [20160623143214.loc.gbuild.net]: FAILED! => {"changed": false, "failed": true, "msg": "dpkg --force-confdef --force-confold -i /tmp/jenkins.deb failed", "stderr": "dirname: missing operand\nTry `dirname --help' for more information.\ndirname: missing operand\nTry `dirname --help' for more information.\nchown: missing operand\nTry `chown --help' for more information.\nInvalid --pidfile argument: '' (Must be an absolute file path)\nusage: daemon [options] [--] [cmd arg...]\noptions:\n\n      -h, --help              - Print a help message then exit\n      -V, --version           - Print a version message then exit\n      -v, --verbose[=level]   - Set the verbosity level\n      -d, --debug[=level]     - Set the debugging level\n\n      -C, --config=path       - Specify the configuration file\n      -N, --noconfig          - Bypass the system configuration file\n      -n, --name=name         - Guarantee a single named instance\n      -X, --command=cmd       - Specify the client command as an option\n      -P, --pidfiles=/dir     - Override standard pidfile location\n      -F, --pidfile=/path     - Override standard pidfile name and location\n\n      -u, --user=user[:group] - Run the client as user[:group]\n      -R, --chroot=path       - Run the client with path as root\n      -D, --chdir=path        - Run the client in directory path\n      -m, --umask=umask       - Run the client with the given umask\n      -e, --env=\"var=val\"     - Set a client environment variable\n      -i, --inherit           - Inherit environment variables\n      -U, --unsafe            - Allow execution of unsafe executable\n      -S, --safe              - Deny execution of unsafe executable\n      -c, --core              - Allow core file generation\n\n      -r, --respawn           - Respawn the client when it terminates\n      -a, --acceptable=#      - Minimum acceptable client duration (seconds)\n      -A, --attempts=#        - Respawn # times on error before delay\n      -L, --delay=#           - Delay between spawn attempt bursts (seconds)\n      -M, --limit=#           - Maximum number of spawn attempt bursts\n          --idiot             - Idiot mode (trust root with the above)\n\n      -f, --foreground        - Run the client in the foreground\n      -p, --pty[=noecho]      - Allocate a pseudo terminal for the client\n\n      -l, --errlog=spec       - Send daemon's error output to syslog or file\n      -b, --dbglog=spec       - Send daemon's debug output to syslog or file\n      -o, --output=spec       - Send client's output to syslog or file\n      -O, --stdout=spec       - Send client's stdout to syslog or file\n      -E, --stderr=spec       - Send client's stderr to syslog or file\n\n          --running           - Check if a named daemon is running\n          --restart           - Restart a named daemon client\n          --stop              - Terminate a named daemon process\ninvoke-rc.d: initscript jenkins, action \"start\" failed.\ndpkg: error processing jenkins (--install):\n subprocess installed post-installation script returned error exit status 7\nErrors were encountered while processing:\n jenkins\n", "stdout": "Selecting previously unselected package jenkins.\n(Reading database ... 189309 files and directories currently installed.)\nUnpacking jenkins (from /tmp/jenkins.deb) ...\nSetting up jenkins (2.9) ...\nInstalling new version of config file /etc/init.d/jenkins ...\n * Starting Jenkins Continuous Integration Server jenkins\nThe selected http port (8080) seems to be in use by another program \nPlease select another port to use for jenkins\n   ...fail!\nProcessing triggers for ureadahead ...\n", "stdout_lines": ["Selecting previously unselected package jenkins.", "(Reading database ... 189309 files and directories currently installed.)", "Unpacking jenkins (from /tmp/jenkins.deb) ...", "Setting up jenkins (2.9) ...", "Installing new version of config file /etc/init.d/jenkins ...", " * Starting Jenkins Continuous Integration Server jenkins", "The selected http port (8080) seems to be in use by another program ", "Please select another port to use for jenkins", "   ...fail!", "Processing triggers for ureadahead ..."]}

I've created PR #88 is association with this task.

Jenkins by default uses port 8080—and in the majority of use cases, it's either running on it's own server (where running on 8080 or 80 is fine), or it's running behind a balancer/proxy (e.g. balancer on 80, jenkins on 8080).
I'd rather not try to add complicated logic to detect running servers on the same port as Jenkins, stop/disable them in this role, install Jenkins, etc., as that seems really fragile to me.
Rather, I would either recommend running Jenkins on a different port (unless... is the issue that Jenkins' install on Debian boots it on a certain port before we get to configure it??), or running Jenkins on a separate server.

The problem was that configuration was happening on install, but so be it.
I'll add in the tasks just before the role gets called, but at least that way it'll work on the same server (in my case this was the point).

Yeah, unfortunately in that case I'd just recommend switching to CentOS/RHEL, where default package installs don't do weird things like boot up services that haven't even been configured (e.g. when you install MySQL on Debian/Ubuntu, by default, it starts it up and gives free access to a test user, has root access open to the world with a default password... it's a mess!).

But that's just my snarky opinion :D

Interesting, might do that when I start on personal projects - but we're still using 12.04 where I'm working as our Acquia servers are running on that also.


Extract method

Implements the "extract function" refactor
Given a user selection, this produces a set of "extractions" the user can select from. For example, with an expression in a class, we can refactor into a peer method, a function in an enclosing namespace, or a function in the same file (these are referred to as "scopes" in the code):

Not all selections will produce the same set of extractions. For example, if we refer to a property of this in the selection body, then we won't be able to extract to the enclosing namespace or file:

Still writing some tests so expect a few commits with bugfixes to come in, but it's pushing 2,000 lines of implementation already so jump in
Most code courtesy @vladima so I may not be able to perfectly answer all questions

🔔

Does it work with JSX? Can I select some JSX markup and extract it to stateless component?

Probably don't want to insert private in JS files. 😉

Refactoring introduces break:
Extract the RHS of let x: 1 = 1;.  The resulting function returns number.

Refactoring introduces break:
Extract the default value of a parameter into the method that declares that parameter.

Curiosity:
The lightbulb menu appears to merge entries with the same text.  This can be an issue if multiple enclosing functions have the same name.

Refactoring introduces break:
Extracting a declaration into a function moves it into an inaccessible scope.  Unextracted references break.

It seems like you can't extract an expression (even one with no side-effects or dependencies) from within a lambda?

The compiler asserts while formatting the extracted method when the input range is the second lambda in (x => x)(x => x)(1);.

@aozgaa can you do another pass?

Should it be impossible to extract an import statement into a function?

I tried to extract this:
/// <reference path="./file2.ts"/>

newFunction_2();

I got this:
function /// <reference path="./file2.ts"/>
    newFunction_4() { /// <reference path="./file2.ts"/>
    /// <reference path="./file2.ts"/>
    newFunction_2();
}

The first and second methods extracted into a class are both called "newFunction"

Refactoring breaks code:
class C {
    constructor(q : string = "hello") {
   }
}

Extract the string literal into a method in the class: 'this' cannot be referenced in constructor arguments

Should it be impossible to extract a super call from a constructor?

Should it be impossible to extract the block of a function declaration?
e.g. { return 1; } from function newFunction_4() { return 1; }

function /// <reference path="./file2.ts"/>
    newFunction_4() { /// <reference path="./file2.ts"/>
    /// <reference path="./file2.ts"/>
    newFunction_2();
}

@rbuckton can you help explain this behavior? Something's going wrong in the emit pipeline but it's pretty hard to debug why

Extracting i++ clobbers the side-effect:
function foo() {

    for (let i = 1; i < 10; i++) {

    }
}

function foo() {

    for (let i = 1; i < 10; newFunction(i)) {

    }

    function newFunction(i: number) { return i++; }
}

(FWIW, it appears to work if the increment is its own statement.)

What's expected to happen when I extract an invocation of a user-defined type guard?

It should probably be impossible to extract the string literal from an import statement.

It should probably be impossible to extract the LHS of a destructuring assignment.

It should probably be impossible to extract an expression like ...x.

Extraction breaks compilation:
When the return statement below is extracted into a global function, the resulting function isn't generic.
function identity<T>(arg: T): T {
    return arg;
}

It should probably be impossible to extract the initializer of a const enum member.

This case involves an error, so it's probably acceptable.  I mention it mostly because it's curious.
interface Person {
    name: string;
    age: number;
}
let person: Person = {
    name: 'Jarid',
    age: 35
};
let strings: string[] = pluck(person, ['age', 'unknown']); // error

If you extract 'age' into a function, the type of that function is, for some reason, K.

Another curiosity:
function* newFunction_3() {
    yield* [1, 2, 3];
}

If you extract yield* [1, 2, 3] (no semicolon), you get function* newFunction_4() { return yield* [1, 2, 3]; }.
However, if you extract yield* [1, 2, 3]; (with semicolon), you get function* newFunction_4() { yield* [1, 2, 3]; }.
Maybe the distinction matters when it's a subexpression?

What's expected to happen when I extract an invocation of a user-defined type guard?

I don't think we can do anything good here. For any non-trivial extraction we're going to lose track of the guardness unless we can somehow figure out that the overall expression only returns true when the type guard does. This is to say nothing of compound things like if (isFoo(x) && isBar(y)) which have no corresponding signature (there's no f(x, y): x is Foo && y is Bar)
function f(x: any): x is string { return true; }
function g() {
    var m: number | string;
    if (f(m) && m !== "hello") {
        ~~~~~~~~~~~~~~~~~~~~~
        m.toLocaleLowerCase();
    }
}

Destructuring assignment when there's only one variable: https://github.com/minestarks/TypeScript/commit/e5b01b43d7ac639aa573c12b0296597ebbec3a34#diff-c767c1f059d44d7e5b3acf921b43362b

Name collision across scopes: https://github.com/minestarks/TypeScript/commit/e5b01b43d7ac639aa573c12b0296597ebbec3a34#diff-ee8f80be166d12fe0768d29ffff334aa

Wrong results when extracting x++ without semicolon at the end: https://github.com/minestarks/TypeScript/commit/65d457b2f06e3132e72779c7fc7fc120aa7c8068#diff-99ed8c6aaf7810869177b35009b54306

I don't even know what this is: https://github.com/minestarks/TypeScript/commit/65d457b2f06e3132e72779c7fc7fc120aa7c8068#diff-00270fa4df2e932afa4830195ff6cfc2

Extracting "use strict"  dubious: https://github.com/minestarks/TypeScript/commit/dfd1ccfc381d219b48ef7fc6215fef543e31cf16#diff-960a1971e82e08844cbaa557bbb268f1

Extraction results in const getting reassigned:  https://github.com/minestarks/TypeScript/commit/3da2a45bb43bf4267e80798b0efb1eb6ae56098e#diff-d5ce3c92231ddfa8f09d3a38f0d1db6e

Extraction results in readonly property being written to from new method (not sure how to get out of this, maybe just disallow) https://github.com/minestarks/TypeScript/commit/9485db783a8244ec983509f5b0fa3b12ca3165d7#diff-d77af8d0b644cf4fd3b9d5174a9d2606

Extracting from static method creates instance method: https://github.com/minestarks/TypeScript/commit/d7e779b8b492b33790f7ea2a69602ad5fde00aa4#diff-056822ad5ab933201dcd8fba568db126

Decorators don't have this, so anything extracted from a decorator has to go outside all enclosing class decorations.

Extraction in ambient contexts shouldn't be possible: https://github.com/minestarks/TypeScript/commit/b049bb013ebe1dfb781ef423f89531dac43359fa#diff-6b5d8f3130a5b0249e28bbc73771f739

Extracting export statements shouldn't be possible? https://github.com/minestarks/TypeScript/commit/b049bb013ebe1dfb781ef423f89531dac43359fa#diff-8d6bca2fc13e4725aa3387454594774a

It seems to be impossible to extract a single identifier into a method.  Is that intentional?  e.g. I might want to wrap access to a variable in a function so that I can bracket it with EH or logging.

Type is referenced from context where it's not visible: https://github.com/minestarks/TypeScript/commit/d1fa06e2f3050ffaadcb200392c1e33a136590a7#diff-92a31e9489e4badbd019ff88953fe79f

function M() {
    let a = [1, 2, 3]
    let x = 0
    a[x++]++
}

Extract a[x++]++ into the containing scope.
function M() {
    let a = [1, 2, 3]
    let x = 0
        ({ a, x } = newFunction(a, x));
}
function newFunction(a: number[], x: number) {
    a[x++]++;
    return { a, x };
}

There's a syntax error because of the missing semicolon between 0 and (.
AFAIK, there's no reason to assign to a.

I suspect @minestarks or I may already have reported this, but if you extract a[x] into a global scope, x is not passed as a parameter and fails to bind.
function M() {
    let a = [1,2,3];
    let x = 0;
    a[x];
}

Fresh sheet of paper at #17625


Application workflow prototype: declining offers

Context
Candidates can decline offers.
Changes proposed in this pull request
Spec and implementation of declining.

@benilovj have rebased this for you, merging :shipit:


⚠ Processors has degraded performance

In a1cb3c0, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 103 ms

Resolved: Processors performance has improved in 985560f.


Swift 3 EXC_BAD on custom date transformation

Custom date transformer was working fine with the swift 2.2 but not with 3.0.
I also put break points on TransformOf closures but it never comes in.
Here is the mapping function and response:
func mapping(map: Map) {

        parent <- map["sub_profile_is_parent"]

        userId <- map["user_id"]
        subProfileId <- map["sub_profile_id"]
        adult <- map["sub_profile_is_adult"]
        fullName <- map["sub_profile_name"]
        emailId <- map["email"]
        gender <- map["sub_profile_gender"]

        let dateTransform = TransformOf<Date, String>(fromJSON: { (value: String?) -> Date? in
            let dateFormatter = DateFormatter()
            dateFormatter.dateFormat = serverDateFormat2 //yyyy-MM-dd
            return dateFormatter.date(from: value!)
            }, toJSON: { (value: Date?) -> String? in
                let dateFormatter = DateFormatter()
                dateFormatter.dateFormat = serverDateFormat2
                return dateFormatter.string(from: value!)
        })

        dob <- (map["sub_profile_date_of_birth"], dateTransform)
    }

{
  "response": {
    "code": 200,
    "data": {
      "user_id": 20,
      "name": "Test",
      "email"<EMAIL_ADDRESS>      "role_id": 3,
      "sub_profile_id": 37,
      "sub_profile_name": "Turab",
      "sub_profile_gender": "male",
      "sub_profile_date_of_birth": "1982-06-25",
      "sub_profile_is_parent": 1,
      "sub_profile_is_adult": 1
    }
  }
}

I have did the following to move on for now:

Created an other temporary string and map the date value into it.
Use my custom transformer to convert it to Date.

Here it is:
dobString <- map["sub_profile_date_of_birth"]
dob = Helper.dateTransformer2().transformFromJSON(dobString)!

@mzeeshanid, which error message did you get?

I am getting EXC_BAD without any other error message.

That's odd, I'm not sure why this would crash.
Can you try the following:
CustomDateFormatTransform(formatString: "yyyy-MM-dd")

Closing this until further reply


Changed FSM variable

to resolve #62

@isi-avbulimen - Can you expand on either the work item or the pull request what this change is regarding in the following format:
Background
...
Steps Taken
...
Similarly your build test is failing due to the unit test using old data for checking the filter function is working correctly for FSM.


add notification to inform user of URL change

this still works because I pushed a copy to a duplicate account after updating my username, need add a notify_once to tell people to update the URL to
https://github.com/purarue/gitsigns-yadm.nvim

if anyone has questions on how to change the URL, feel free to post your config here, will leave this open for a few months

@seanbreckenridge does it mean that this repo will eventually be archived and all the development going to happen in new repo ?

yes, thats the plan
but will probably remain around for a year or so, to let people upgrade/migrate


⚠ Bofrost.de has degraded performance

In b1e093c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1079 ms

Resolved: Bofrost.de performance has improved in f4cbc08 after 7 minutes.


Some Apps did not install due to an internal error

Dev Home version
0.401.226.0
Windows build number
10.0.22631.2338
Other software
OS Build Version: 10.0.22631.2338.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.20
Steps to reproduce the bug
Installing all apps
Expected result
All to install without issue
Actual result
Docker, Intelj and Unity did not install
Included System Information
Physical Memory: 63.92GB (51.01GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_0.401.226.0_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_0.400.226.0_x64__8wekyb3d8bbwe

Can you reproduce the errors and share log files?

/logs

Dont know if its the same error
Just followed this steps

Machine Configuration
Install Applications
Notepad++ (tried several at once before same result)
Next
Agree -> Setup
Error

CoreWidgetProvider.zip

Hi Sorry for the late reply, hope these help, I could not get several to
install so I installed separately and all went OK,I have since ran
devhome and it allowed some installs over existing,
Thanks
-------------------------------------*--
email:  @.***
------------------------------------*--
On Tue, 19 Sept 2023 at 18:46, microsoft-github-policy-service[bot] <
@.***> wrote:

Hi @KarlSheldon https://github.com/KarlSheldon. It would be helpful for
us if you could share your Dev Home logs. These logs can be found at
%LOCALAPPDATA%\Packages\Microsoft.Windows.DevHome_8wekyb3d8bbwe\TempState
and
%LOCALAPPDATA%\Packages\Microsoft.Windows.DevHomeGitHubExtension_8wekyb3d8bbwe\TempState.
You can share these folders via a OneDrive link or zip them and attach them
to a comment here. If you share this way, you may want to look through the
logs in case there are any details included that you would like to remove
(for example, private repo names). Alternatively, you can open a Feedback
Hub issue and attach them there. If you use Feedback Hub, please paste the
URL at the bottom of the report here so we can easily find it.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/devhome/issues/1562#issuecomment-1726214562,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATQWNVTZCOR2CSP3I6ACPPTX3HK7BANCNFSM6AAAAAA43C46SI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@KarlSheldon, sorry for the delayed response. I've recently refactored WinGet's integration in the PRs below. Would it possible to validate if the issue still repros in latest Dev Home version?

https://github.com/microsoft/devhome/pull/2089

Added recovery when RPC exceptions occur.

https://github.com/microsoft/devhome/pull/3134

Elevated process now uses the same WinGet logic as Dev Home's main process (enhanced implementation overall + recovery)

https://github.com/microsoft/devhome/pull/3307

Mitigated an issue that occurs when Dev Home's elevated process attempts to install multiple packages at the same time.

I am happy to report that I have now used devhome to install on 4 machines
without issues. Tried to trip it up with some apps already installed but it
now works as intended,
Thanks
On Sat, 6 Jul 2024, 03:21 AmirMS, @.***> wrote:

@KarlSheldon https://github.com/KarlSheldon, sorry for the delayed
response. I've recently refactored WinGet's integration in the PRs below.
Would it possible to validate if the issue still repros in latest Dev Home
version?

#2089 https://github.com/microsoft/devhome/pull/2089

Added recovery when RPC exceptions occur.

#3134 https://github.com/microsoft/devhome/pull/3134

Elevated process now uses the same WinGet logic as Dev Home's
main process (enhanced implementation overall + recovery)

#3307 https://github.com/microsoft/devhome/pull/3307

Mitigated an issue that occurs when Dev Home's elevated process
attempts to install multiple packages at the same time.

—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/devhome/issues/1562#issuecomment-2211590836,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATQWNVRLWPYBMKLHWPQGWUDZK5ICVAVCNFSM6AAAAAA43C46SKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJRGU4TAOBTGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


lxd/request/const: Add CtxTrusted

Adds a request context key informing us if this request is trusted by us.

Closing as it's not actually used within LXD so doesn't make sense to have in here.
Instead the downstream project using this package can probably use CtxAccess with its own struct.


Sink Strategy parameter

Hi,
Could you please advice if Sink Strategy parameter would be implemented in JDBC importer, or it's just part of deprecated River plugin and wouldn't be supported?
Thanks.

I don't understand. Codebase for importer is exactly the same as for river implementation. It just runs outside Elasticsearch in a separate JVM.

I would be very appreciate for any clue or example how to set up the export of data from mssql with this parameter. Thanks.

It is not a parameter, it's for developers who want to write different sinks. Just use the default sink for mssql to Elasticsearch if you don't know what it is.


Added Steve Jobs Quote from 2005 Address.

Changes

Added Steve Jobs Quote.
One of the greatest minds who has revolutionized technology, only appropriate we have it here.

Source:
YouTube Link to Speech

Hey! I hope you consider this quote.
Thanks!

Thanks!


Dadhania tilak patch 1

This is my first open source contribution.

Thank you for your valuable contribution!
Make sure to leave a star ⭐ on this repo


Modified PER test

This PR is for a reliable PER test.

Results:
http://<IP_ADDRESS>:8000/per/pdf/?pdf=msdk-2023-02-16_15-26-00_max32665_EvKit_V1.pdf
http://<IP_ADDRESS>:8000/per/pdf/?pdf=msdk-2023-02-16_15-44-13_max32690_EvKit_V1.pdf
http://<IP_ADDRESS>:8000/per/pdf/?pdf=msdk-2023-02-16_15-09-18_max32655_EvKit_V1.pdf

The running workflow code is the new one in the PR.
https://github.com/yc-adi/msdk_open/blob/dfbef3c6940461f9975d40642c98eaf9e2492087/.github/workflows/simple_per_test.yml#L219
But the log shows the script the new CI called is still the current one.
board_per_test.sh 1_MSDK 2_BRD1 3_BRD2 4_CURR_TIME 5_CURR_JOB_FILE 6_CURR_LOG 7_all_in_one 8PKG 9PHY 10STEP
The new script in the PR is:
https://github.com/yc-adi/msdk_open/blob/dfbef3c6940461f9975d40642c98eaf9e2492087/.github/workflows/scripts/board_per_test.sh#L5
I tested the new PR in my repo. It worked.
https://github.com/yc-adi/msdk_open/actions/runs/4219687342

Ok I see. The automated run being executed here is using an older version.


Fix #37 - Authenticated User Routes

Fix #37: Pull code for POST '/users' out from authenticed routes to allow new users to sign up without already being users (this is more realistic).  GET '/users' is still protected.  Change content-type to Content-Type in server.js
WebStorm trimmed some whitespace too.

I see the value in this, but I think that this would better be solved by creating an auth section of the API to handle registrations.
The users API will still need the authenticated POST /users route for admins that want to create users from the dashboard.
I'd recommend creating unauthenticated routes in a new app/routes/api.auth.js or app/routes/auth.js :
/api/auth/login
/api/auth/register

I think admin users may be outside of the scope of this chapter. Also, breaking the routes out into different files may cause confusion for some of those following along in the book.  Is the book updated alongside the repo?  If it is, I think you're right about creating a separate file for registrations.  Otherwise I think that might make the code in the repo too different from the code in the book for beginners.  What do you think?

Yea I want to keep the code in the repo the same as the book. This is a great exercise to go beyond the book curriculum, but you're right; it's better for learning to keep it the way it is.


PNG Image Links are Not Processed

Obsidian Version 0.12.19
Local Images Version 0.14.2
When I run this plugin it works wonderfully for many image types but seems to ignore PNG image links. I've tried with several PNG image URLs and the plugin skips them.

Same problem here. I have a link like this, which isn't processed:

I have the same issue. png localization does not work automatically. This includes running the command to download images manually.

Update: It is very dependent on the original link.
For example, this link
<img src="https://ceblog.s3.amazonaws.com/wp-content/uploads/2017/11/10002521/random-people-on-the-internet-buy.png" alt="https://ceblog.s3.amazonaws.com/wp-content/uploads/2017/11/10002521/random-people-on-the-internet-buy.png">
did not localize.
This link:
https://ceblog.s3.amazonaws.com/wp-content/uploads/2017/11/10002521/random-people-on-the-internet-buy.png
also did not localize.
But, with it set to localize on paste, copying the image and pasting the image did localize.
Also, on this same web page, i used the 'pluck' plugin, and this plugin Did localize the images, including .png.
Hopefully this can help the author.
Thank you


Add link to version number normalisation page

Description Of Changes
Add link to the Version Number Normalization page
Motivation and Context
Relates to this section of the docs
Testing

[x] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[x] Minor documentation fix (typos etc.).
[ ] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[ ] New documentation page added.
[ ] The change I have made should have a video added, and I have raised an issue for this.

Issue #

Change Checklist

[ ] Requires a change to menu structure (top or left-hand side)/
[ ] Menu structure has been updated

Related Issue
Fixes #1082

@flcdrg awesome work! Thanks for your contribution!

Just need someone to approve this :-)

Just need someone to approve this :-)

I think Paul intended to approve it and instead just enabled the auto merge. I've approved it. Thanks for the contribution @flcdrg 👍

I did. I usually remind myself to go back and check it happened. Must have missed this one. Apologies.


BeforeLeaveEventArgs.retry with Router.base navigates to incorrect location

Describe the bug
When using the base prop on the Router component, invoking BeforeLeaveEventArgs.retry within useBeforeLeave  navigates to an incorrect location.
Your Example Website or App
https://stackblitz.com/~/github.com/vveisard/bug-solidjs-router-base-with-use-before-leave
Steps to Reproduce the Bug or Issue

install dependencies
Run "dev" package.json script
Navigate to https://localhost:3000/base
Click "to /bar" button
Observe resolved url to be /base/base/bar

(I recommend opening the StackBlitz preview in new tab, because you can see the resolved location much easier.)
Expected behavior
As a user, I expected BeforeLeaveEventArgs.retry to navigate to the proper location (ie, same as Navigator).
Screenshots or Videos
No response
Platform

OS: Manjaro Linux
Browser: Brave
Engine: pnpm, Bun
Version: Latest

Additional context
No response

Thanks for the great reproduction.  I've submitted a PR for a fix that seems to resolve this.


Diagnose and re-enable WebScenarioTests

The following WebScenarioTests started failing with a SDK dependency flow.
These were disabled to unblock the dependency flow.  The failures need investigation.
System.InvalidOperationException : Failed to execute /vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/.dotnet/dotnet build /bl:/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/logs/WebScenarioTests_Mvc_FSharp-build.binlog
Exit code: 1
MSBuild version 17.6.0-preview-23115-01+dfd8f413a for .NET
Determining projects to restore...
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_Mvc_FSharp/WebScenarioTests_Mvc_FSharp.fsproj : error NU1102: Unable to find package Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation with version (>= 8.0.0)
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_Mvc_FSharp/WebScenarioTests_Mvc_FSharp.fsproj : error NU1102:   - Found 617 version(s) in dotnet8 [ Nearest version: 8.0.0-preview.2.23115.12 ]
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_Mvc_FSharp/WebScenarioTests_Mvc_FSharp.fsproj : error NU1102:   - Found 112 version(s) in dotnet-public [ Nearest version: 7.0.3 ]
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_Mvc_FSharp/WebScenarioTests_Mvc_FSharp.fsproj : error NU1102:   - Found 0 version(s) in /vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/.dotnet/sdk/8.0.100-preview.2.23116.1/FSharp/library-packs
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_Mvc_FSharp/WebScenarioTests_Mvc_FSharp.fsproj : error NU1102:   - Found 0 version(s) in dotnet8-transport
Failed to restore /vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_Mvc_FSharp/WebScenarioTests_Mvc_FSharp.fsproj (in 535 ms).

Build FAILED.

System.InvalidOperationException : Failed to execute /vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/.dotnet/dotnet build /bl:/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/logs/WebScenarioTests_WebApi_CSharp-build.binlog
Exit code: 1
MSBuild version 17.6.0-preview-23115-01+dfd8f413a for .NET
Determining projects to restore...
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_WebApi_CSharp/WebScenarioTests_WebApi_CSharp.csproj : error NU1102: Unable to find package Microsoft.AspNetCore.OpenApi with version (>= 8.0.0)
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_WebApi_CSharp/WebScenarioTests_WebApi_CSharp.csproj : error NU1102:   - Found 617 version(s) in dotnet8 [ Nearest version: 8.0.0-preview.2.23115.12 ]
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_WebApi_CSharp/WebScenarioTests_WebApi_CSharp.csproj : error NU1102:   - Found 10 version(s) in dotnet-public [ Nearest version: 7.0.3 ]
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_WebApi_CSharp/WebScenarioTests_WebApi_CSharp.csproj : error NU1102:   - Found 0 version(s) in dotnet8-transport
Failed to restore /vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_WebApi_CSharp/WebScenarioTests_WebApi_CSharp.csproj (in 476 ms).

Build FAILED.

System.InvalidOperationException : Failed to execute /vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/.dotnet/dotnet build /bl:/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/logs/WebScenarioTests_BlazorWasm_CSharp-build.binlog
Exit code: 1
MSBuild version 17.6.0-preview-23115-01+dfd8f413a for .NET
Determining projects to restore...
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_BlazorWasm_CSharp/WebScenarioTests_BlazorWasm_CSharp.csproj : error NU1102: Unable to find package Microsoft.AspNetCore.Components.WebAssembly with version (>= 8.0.0)
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_BlazorWasm_CSharp/WebScenarioTests_BlazorWasm_CSharp.csproj : error NU1102:   - Found 617 version(s) in dotnet8 [ Nearest version: 8.0.0-preview.2.23115.12 ]
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_BlazorWasm_CSharp/WebScenarioTests_BlazorWasm_CSharp.csproj : error NU1102:   - Found 68 version(s) in dotnet-public [ Nearest version: 7.0.3 ]
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_BlazorWasm_CSharp/WebScenarioTests_BlazorWasm_CSharp.csproj : error NU1102:   - Found 0 version(s) in dotnet8-transport
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_BlazorWasm_CSharp/WebScenarioTests_BlazorWasm_CSharp.csproj : error NU1102: Unable to find package Microsoft.AspNetCore.Components.WebAssembly.DevServer with version (>= 8.0.0)
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_BlazorWasm_CSharp/WebScenarioTests_BlazorWasm_CSharp.csproj : error NU1102:   - Found 617 version(s) in dotnet8 [ Nearest version: 8.0.0-preview.2.23115.12 ]
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_BlazorWasm_CSharp/WebScenarioTests_BlazorWasm_CSharp.csproj : error NU1102:   - Found 68 version(s) in dotnet-public [ Nearest version: 7.0.3 ]
/vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_BlazorWasm_CSharp/WebScenarioTests_BlazorWasm_CSharp.csproj : error NU1102:   - Found 0 version(s) in dotnet8-transport
Failed to restore /vmr/test/Microsoft.DotNet.SourceBuild.SmokeTests/bin/Release/net8.0/projects-202302161225127650/WebScenarioTests_BlazorWasm_CSharp/WebScenarioTests_BlazorWasm_CSharp.csproj (in 510 ms).

Build FAILED.

[Triage] It looks like there are some missing packages that the tests depend on.


Build failed with an exception - Execution failed for task ':app:transformDexArchiveWithExternalLibsDexMergerForDebug'.

I'm working on a fresh install of Flutter. All I did was adding contact services to the pubspec.yaml file and when I try to run it throws build error.
pubspec.yaml
dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^0.1.2
  simple_permissions: ^0.1.9
  contacts_service: ^0.2.4

Error
Launching lib/main.dart on SM G950W in debug mode...
Initializing gradle...
Resolving dependencies...
Running Gradle task 'assembleDebug'...
D8: Program type already present: android.support.v4.os.ResultReceiver$MyResultReceiver

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:transformDexArchiveWithExternalLibsDexMergerForDebug'.
> com.android.builder.dexing.DexArchiveMergerException: Error while merging dex archives: /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/28.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/5.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/40.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/58.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/49.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/34.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/64.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/9.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/61.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/37.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/55.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/43.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/25.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/31.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/67.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/46.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/22.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/52.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/15.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/10.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/6.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/7.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/11.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/14.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/3.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/8.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/12.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/4.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/16.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/2.jar, /home/mithunnath/IdeaProjects/fluttercontactsapp/build/app/intermediates/transforms/dexBuilder/debug/13.jar
  Learn how to resolve the issue at https://developer.android.com/studio/build/dependencies#duplicate_classes.
  Program type already present: android.support.v4.os.ResultReceiver$MyResultReceiver

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 2s
*******************************************************************************************
The Gradle failure may have been because of AndroidX incompatibilities in this Flutter app.
See https://goo.gl/CP92wY for more information on the problem and how to fix it.
*******************************************************************************************
Finished with error: Gradle task assembleDebug failed with exit code 1

Can you post the output of flutter doctor -v?

I'm also getting this for the following class:
android.support.v4.media.MediaBrowserCompat$ConnectionCallback$ConnectionCallbackInternal
Flutter Doctor -v
[✓] Flutter (Channel stable, v1.2.1, on Mac OS X 10.14.3 18D109, locale en-GB)
    • Flutter version 1.2.1 at /Applications/Flutter-SDK/flutter
    • Framework revision 8661d8aecd (6 weeks ago), 2019-02-14 19:19:53 -0800
    • Engine revision 3757390fa4
    • Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Applications/Android-SDK/
    • Android NDK location not configured (optional; useful for native profiling
      support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Applications/Android-SDK/
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 32.0.1
    • Dart plugin version 182.5124
    • Java version OpenJDK Runtime Environment (build
      1.8.0_152-release-1248-b01)

[✓] VS Code (version 1.32.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.24.0

[✓] Connected device (1 available)
    • Redmi 6 • 05687ad57d28 • android-arm • Android 8.1.0 (API 27)

• No issues found!

debug run well,build faild

[✓] Flutter (Channel stable, v1.7.8+hotfix.3, on Mac OS X 10.14.5 18F132, locale zh-Hans-CN)
    • Flutter version 1.7.8+hotfix.3 at /Users/chenhailong/development/flutter
    • Framework revision b712a172f9 (4 weeks ago), 2019-07-09 13:14:38 -0700
    • Engine revision 54ad777fd2
    • Dart version 2.4.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/chenhailong/Library/Android/sdk
    • Android NDK at /Users/chenhailong/Library/Android/sdk/ndk-bundle
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Users/chenhailong/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 10.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.3, Build version 10G8
    • CocoaPods version 1.7.5

[✓] iOS tools - develop for iOS devices
    • ios-deploy 1.9.4

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 25.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2019.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 21.2.2
    • Dart plugin version 191.7830

[✓] Connected device (1 available)
    • iPhone X • 67E3F035-80A4-41BA-9008-FC43EBE8672C • ios • com.apple.CoreSimulator.SimRuntime.iOS-12-2 (simulator)

• No issues found!

Hi, could you please confirm if this problem still exists with the latest versions on stable or master?
Thanks a lot

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Fitness example app should track weight

Initially, use file system to store data between sessions. We can implement cloud sync later

It's possible to enter weight now, closing bug

we haven't implemented writing to disk yet, but it is possible to enter the weight. :)

This works now.


MPIR-268: Mirror Dependency Information to Maven Central

Add 'Gradle' to Grails as they are the same matching maven central.
Move SBT up in order and add 'Scala' to match maven central.

Any status on this PR?

Any status on this PR?

You should have linked to the JIRA issue: https://issues.apache.org/jira/browse/MPIR-268. Request to reopen the issue, squash your commit and bring it uptodate with current trunk. I will review and pull in.

@michael-o I have marked the commit with jira ticket number as well as PR.  I requested reopen and squashed commits as you asked.  Thanks for checking in on this.

Running ITs...

Damn, can't run any of the tests at work because the test mojo executor ignores my settings.xml. Have to rerun when I get back to my home machine.

Cannot cleanly apply your patch:
[INFO] There are 10 errors reported by Checkstyle 6.1.1 with config/maven_checks.xml ruleset.
[ERROR] src/main/java/org/apache/maven/report/projectinfo/DependencyInformationReport.java[168] (sizes) LineLength: Zeile länger als 120 Zeichen
[INFO] Ignored 9 errors, 1 violation remaining.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 44.638 s (Wall Clock)
[INFO] Finished at: 2015-10-01T13:10:26+02:00
[INFO] Final Memory: 34M/407M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.15:check (checkstyle-check) on project maven-project-info-reports-plugin: You have 1 Checkstyle violation. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.

The line is too long.

I'll look at this later tonight.
Sent by Outlook for Android
On Thu, Oct 1, 2015 at 4:13 AM -0700, "Michael Osipov"<EMAIL_ADDRESS>wrote:
Cannot cleanly apply your patch:
[INFO] There are 10 errors reported by Checkstyle 6.1.1 with config/maven_checks.xml ruleset.
[ERROR] src/main/java/org/apache/maven/report/projectinfo/DependencyInformationReport.java[168] (sizes) LineLength: Zeile länger als 120 Zeichen
[INFO] Ignored 9 errors, 1 violation remaining.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 44.638 s (Wall Clock)
[INFO] Finished at: 2015-10-01T13:10:26+02:00
[INFO] Final Memory: 34M/407M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.15:check (checkstyle-check) on project maven-project-info-reports-plugin: You have 1 Checkstyle violation. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.

The line is too long.

Reply to this email directly or view it on GitHub:
https://github.com/apache/maven-plugins/pull/44#issuecomment-144699457

@michael-o Should be good now.  I ran a test build locally and it passes.

I just rebased.

Thanks, will check next week.

Patch is in test...

Thanks
Sent by Outlook for Android
On Tue, Oct 6, 2015 at 1:30 AM -0700, "Michael Osipov"<EMAIL_ADDRESS>wrote:
Patch is in test...

Reply to this email directly or view it on GitHub:
https://github.com/apache/maven-plugins/pull/44#issuecomment-145782061

Where you able to run mvn clean verify -Prun-its -Dinvoker.mergeUserSettings and mvn -Preporting site? I receive:
[INFO] Generating "Invoker Build Results" report --- maven-invoker-plugin:1.9:report
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 10:02 min
[INFO] Finished at: 2015-10-06T19:03:25+02:00
[INFO] Final Memory: 58M/191M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.4:site (default-site) on project maven-project-info-reports-plugin: Error generating maven-invoker-plugin:1.9:report: Failed to read report file: D:\Projekte\maven-project-info-reports-plugin\target\invoker-reports\invoker-summary.txt: input contained no data -> [Help 1]

The invoker summary is not written. I will try the latest tag first.

I think, I have found the problem. This comes from a newly introduced changed in Plugins Parent 28-SNAPSHOT.

Patch applied. Please close the PR. In future, please avoid adding unrelated changes like in .gitignore in one PR.

Thanks @michael-o I thought to remove that as well.  I had it originally as a separate commit but squashed when asked to perform rebase / squash.  Anyway, I see it's not included and probably not worth the effort to include the ignore file.  Anyway, thanks for picking this PR up.  I'm closing the request as noted.


Support anonymous config value

This change helps jwt-claims-validate play nice when the jwt plugin's config.anonymous option is used as described here.

I'm guessing this will never be merged...closing it to get it off of my pull request page 😆


Streamlit caching and session state store fixes

This PR does the following things

Lowers the TTL for query result cache to 1 minute (old was 5 minutes),
Avoids storing schema in streamlit's session state store.

Issue: https://github.com/dlt-hub/dlt/issues/1264

@rudolfix yeah, I also agree with this, it rather better to have a single file instead of later trying to tackle the side effects.


Serenity homepage and open-source option

Hello @volkanceylan
I see that Serenity's homepage has new UI, it's nice UI !
But there is a small problem, I introduced Serenity to many people and they always ask me is if it's free, because when they look at Serenity homepage, all options of Pricing have a price.
So can you add one more options that show that Serenity also has a free version (open source) too.
I think that small change will help them spend more time to research Serenity framework

I think it would be a good idea.

Hi @volkanceylan ,
on your new serenity.is page, both the "Demo" and the "buy now" button do currently (2020-04-13, 14:10) not work. tested both with Chromium-based browser and Edge.


net: Conn.Write() does not detect dropped TCP connection on the first attempt

What version of Go are you using (go version)?
go version go1.8.3 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/kvs/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build565879858=/tmp/go-bui
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
What did you do?
A dropped TCP connection does not get detected on the first Write(), but detected on subsequent ones.
Steps to reproduce:

Compile Go client - https://gist.github.com/vbox/d6b8d3f431fd5195846929ee9659ed25

Open TCP socket with nc:
nc -kl 5555

Start client binary, and immediately after client has established connection, kill nc. The client will attempt to write to a dropped connection, but no error is generated.

What did you expect to see?
$ ./client
connection established
Write to server failed: write tcp <IP_ADDRESS>:48176-><IP_ADDRESS>:5555: write: broken pipe
What did you see instead?
$ ./client
connection established
writen to server =  Halo
Write to server failed: write tcp <IP_ADDRESS>:48176-><IP_ADDRESS>:5555: write: broken pipe

This'll be Nagel
On Fri, 2 Jun 2017, 15:45 vbox<EMAIL_ADDRESS>wrote:

What version of Go are you using (go version)?
go version go1.8.3 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/kvs/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0
-fdebug-prefix-map=/tmp/go-build565879858=/tmp/go-bui
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
What did you do?
A dropped TCP connection does not get detected on the first Write(), but
detected on subsequent ones.
Steps to reproduce:

Compile Go client -
https://gist.github.com/vbox/d6b8d3f431fd5195846929ee9659ed25
2.
Open TCP socket with nc:
nc -kl 5555
3.
Start client binary, and immediately after client has established
connection, kill nc. The client will attempt to write to a dropped
connection, but no error is generated.
What did you expect to see?
$ ./client
connection established
Write to server failed: write tcp <IP_ADDRESS>:48176-><IP_ADDRESS>:5555: write:
broken pipe
What did you see instead?
$ ./client
connection established
writen to server = Halo
Write to server failed: write tcp <IP_ADDRESS>:48176-><IP_ADDRESS>:5555: write:
broken pipe
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/20553, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAcAzgesSxDQV-hDzefmX7tOcUX40Knks5r_6FwgaJpZM4Nt2vv
.

Your test code doesn't guarantee that the first write call that invokes some functionality inside the kernel has got a closed state of the underlying connection before it returns to the test code. On linux net/ipv4/tcp.c and tcp_output.c describe how the write system call interacts with the TCP implementation.