Change the name before v2.0.0 released?

In past Firefox add-on was named IPFS Gateway Redirect.
It is an understatement, because in addition to redirect functionality, v2.0.0alpha2  features IPFS node status monitoring, experimental dnslink support,   web+ipfs:// protocol support and quick upload options for local and web content.
I feel it may be a good time to pick visually shorter and  more broad name.
Please vote using emoji reaction below  or suggest something more elegant in comments below
(one per comment, to make it easy to vote with 👍  reactions)  🙃

👍 here → change name to IPFS Companion
👎  here → keep the old name IPFS Gateway Redirect
👍  at this comment →  change name to IPFS Station

Reasons for my change-vote:

The gateway will hopefully soon be just one of various "transports"
Redirect is an implementation detail

IPFS Station was suggested in https://github.com/ipfs/in-web-browsers/issues/54
(👍  this comment to vote for it)

Let's timebox this decision. I say, let's pick the name till Wednesday 17. That give everyone enough time to check this issue and vote

Ping -- this timebox expires on Wednesday.
Station sounds good to me IF that means the current station tool and the addon will be one thing.

Thank you @lgierth ! :)

Sounds like IPFS Gateway Redirect it is 🍾

You mean... IPFS Station ? 🙃

Oh, I meant IPFS Companion

IPFS Companion - 6 votes
IPFS Gateway Redirect - 1 vote
IPFS Station - 5 votes

Oy... that was a close one 😕
We've added "Station" after "Companion" already had 4 votes, and some votes are duplicated.
@lgierth, @RndFooBar, @verox, @jedahan, @mateon1, @sonatagreen:
Please check your votes,  decide between "Station" and Companion" and update your votes accordingly.
Make sure you vote only for one option.

Lol, seems that made IPFS Companion even stronger 🙃
Can we say it is settled? What is the next step? Repo rename to ipfs-companion? Who can do it?

Renamed :) Welcome to ipfs-companion! :tada:

Screenshot for historic purposes:


#274: improve unit test coverage in Sql namespace

This is for M2 workitem #274 to add unit test to improve code coverage in "Sql" namespace

Current coverage is 76.39%

Merging #384 into master will increase coverage by +1.05% as of c6489b4

@@            master    #384   diff @@
======================================
  Files           46      46
  Stmts         4689    4689
  Branches       419     418     -1
  Methods
======================================
+ Hit           3533    3582    +49
- Partial        179     181     +2
+ Missed         977     926    -51

Review entire Coverage Diff as of c6489b4

Powered by Codecov. Updated on successful CI builds.


b/32560113: Remove false alarm

RollbackNonBusy gets called if a tablet was externally reparented.
If so, it's normal for transactions to still be outstanding. So,
they should be simply rolled back without raising any alarms,

LGTM


fixing windows download and environment

#2
The download part is working.
I still have trouble to spawn a new shell (cmd.exe) with the right params.
Maybe another one could get this part?

thanks for the feedback!

I solved the parent process shell spawning now. I am doing some beautifying now and would be glad if my PR would get accepted afterwards :) I'll leave a note after i finished cleaning up

Hey, many thanks for the feedback and hints.
It was a great first touch with golang!
If you do not found anything further i can modify, i would call that "merge ready".
I've tested:

downloading and spawning shell in windows 10
downloading and spawning shell in ubuntu
environment variables in windows
environment variables in ubuntu

Cheers,
Patrick

@m0veax I can merge right away if don't want to participate in hacktoberfest :)

oh, wait please, i want to participate indeed!


Run OpenSearch sink integration tests against more versions of OpenDistro

Description
Data Prepper has an integration test for the OpenSearch sink. It currently runs against two OpenDistro versions. This PR adds support for testing against four additional versions. With this PR, Data Prepper integration tests verify from OpenDistro 1.6.0 / Elasticsearch 7.6.1 and beyond. Additionally, it tests against OpenDistro 1.13.3 instead of 1.13.2 since this addresses a security vulnerability.
In order to support this range of versions, the code to wipe indices between tests now uses the Get Indices API instead of the _cat/indices API.
Issues Resolved
N/A
Check List

[x] New functionality includes testing.
[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1348 (e4e690c) into main (512ffcc) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1348      +/-   ##
============================================
- Coverage     94.08%   93.99%   -0.10%
  Complexity     1156     1156
============================================
  Files           162      162
  Lines          3248     3248
  Branches        263      263
============================================
- Hits           3056     3053       -3
- Misses          138      141       +3
  Partials         54       54

Impacted Files
Coverage Δ

...java/com/amazon/dataprepper/pipeline/Pipeline.java
88.57% <0.00%> (-4.29%)
:arrow_down:

Continue to review full report at Codecov.

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


Test out BlueIris Docker Image

Docker Image: https://hub.docker.com/r/jshridha/blueiris
Thoughts:

Thinking of building a Helm Chart around it and testing out its performance.
May need to do a native VM to compare it against.
Look into attaching GPU resources to the chart?

More notes to come

Not going to happen anymore, have a unifi system now.


Check for duplicated element on the private container of ModelMap

A ModelMap is a shadow version of the TaskContainer inside the
software model. The ModelMap only check if the type of the created
element match the container. With this commit, the ModelMap will check
against the backing container to ensure there isn't an element already
present with the same name. This scenario caused different task instance
to be used because of the duplication. It is an error to create tasks
with the same name in both the TaskContainer and ModelMap.

Fixes https://github.com/gradle/gradle-private/issues/2248
Context

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

This actually already work. So the issue is creating a task inside ModelMap<Task> before creating a task inside TaskContainer. I modified the PR to do exactly that.

Well CI is green, so whoever wants to review can do so in master.


Error when loading draco compressed models while using @cesium/engine

When using the plugin in its default mode (import cesium from 'vite-plugin-cesium-build') I can load draco compressed models without any issue.
If I switch to @cesium/engine (import cesium from 'vite-plugin-cesium-build/engine'), I get the following error:
failed to asynchronously prepare wasm: CompileError: WebAssembly.instantiate(): expected magic word 00 61 73 6d, found 3c 21 64 6f @+0
(anonyme) @ chunk-TTI2SUWJ.js:817
chunk-TTI2SUWJ.js:415 Aborted(CompileError: WebAssembly.instantiate(): expected magic word 00 61 73 6d, found 3c 21 64 6f @+0)
f @ chunk-TTI2SUWJ.js:415
@cesium_engine.js?v=4aebc6e8:154021 RuntimeError
message: "Failed to load model: CesiumMilkTruck.gltf\nFailed to load glTF\nFailed to load vertex buffer\nDraco decoder could not be initialized."
...

It seems that the wasm files are missing (both in dev and build mode) for the @cesium/engine use case

Closed by #11


Improve language, add tippy

I ran everything through an LLM and synthesized its output with what I preferred, this is the result. Imho, it improves on conciseness and clearness quite a bit.
Additionally adds sphinx-tippy for sneaky peeking.
After this, I'm fine with pulling the trigger.
Still need to adjust the default README somehow though.

I'm curious - what LLM and prompt did you use? Did it preserve the MyST syntax on its own, or you had to edit the output manually?

I'm curious - what LLM and prompt did you use?

Nothing fancy, just ChatGPT 4o with the following (lazy) prompt:

I'm writing documentation around Salt extensions, which are pip-installable Python packages that can provide different modules to Salt (as in SaltStack, the automation platform).
I'm going to send you one doc page at a time. Please optimize each page for clarity and conciseness.

Did it preserve the MyST syntax on its own, or you had to edit the output manually?

It kept most of it, but sometimes dropped the roles like {path}/{question}. I still edited it manually side-by-side.
I had to break down some longer documents:

Let's break the last document I sent down into parts. The previous modification is too concise at the cost of clarity.

Sometimes, I had to focus on some specific sentences. It really tried to smoothen the following:

After the Broadcom event, the push really came to shove, and it was communicated that there won't be any deprecation period

Following the Broadcom event, it was announced that there won't be any deprecation period
(me) The change is quite fundamental and abrupt, which your suggestion does not imply. Please rephrase
After the Broadcom event, it was urgently announced that there won't be any deprecation period
(me) please rephrase again, still too soft
The Broadcom event forced an immediate shift in strategy, leading to the announcement that there won't be any deprecation period

I was really impressed with its ability to interpret and follow instructions tbh. For some lighter parts, its technical/rational tone did not match my intentions, so I had to re-prompt:

(me) Please make this less clunky, but keep the vibe:
(me) Very important: You don't need to publish inside the organization to use it!
It can be used for individual projects as well.
Important: You don’t have to publish within the organization to use it — it's available for individual projects too!


Refactoring Python Bedrock Client

Refactoring the Python bedrock client askCall method to properly wrap messages, history and reduce repeated code blocks.

fixed here - https://github.com/SEMOSS/Semoss/pull/311


Broken pipe in WSL2

Issue description
Within WSL2 (Debian), when creating an environment and running pipenv shell i receive a broken pipe when it try's to load. I dont know if this is a WSL2 thing or a pipenv thing.
Expected result
I would have expected to be put into the virtualenv
Actual result
When possible, provide the verbose output (--verbose), especially for locking and dependencies resolving issues.
$ pipenv shell --verbose
Launching subshell in virtual environment…
 . /home/jthorpe/.local/share/virtualenvs/temp-HIOIE7wf/bin/activate
--verbose
write failed 32: Broken pipe

Steps to replicate
Provide the steps to replicate (which usually at least includes the commands and the Pipfile).

[~]$ cd /tmp
[/tmp]$ mkdir temp ; cd $_
[/tmp/temp]$ echo "print('hello world')" > test.py
[/tmp/temp]$ pipenv install --three
Creating a virtualenv for this project…
Pipfile: /tmp/temp/Pipfile
Using /usr/bin/python3.7m (3.7.3) to create virtualenv…
⠴ Creating virtual environment...Using base prefix '/usr'
New python executable in /home/jthorpe/.local/share/virtualenvs/temp-HIOIE7wf/bin/python3.7m
Also creating executable in /home/jthorpe/.local/share/virtualenvs/temp-HIOIE7wf/bin/python
Installing setuptools, pkg_resources, pip, wheel...done.
Running virtualenv with interpreter /usr/bin/python3.7m

✔ Successfully created virtual environment!
Virtualenv location: /home/jthorpe/.local/share/virtualenvs/temp-HIOIE7wf
Creating a Pipfile for this project…
Pipfile.lock not found, creating…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Updated Pipfile.lock (a65489)!
Installing dependencies from Pipfile.lock (a65489)…
  🐍   ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 0/0 — 00:00:00
To activate this project's virtualenv, run pipenv shell.
Alternatively, run a command inside the virtualenv with pipenv run.
[/tmp/temp]$ pipenv shell
Launching subshell in virtual environment…
 . /home/jthorpe/.local/share/virtualenvs/temp-HIOIE7wf/bin/activate
write failed 32: Broken pipe
[/tmp/temp]$ exit
exit
[/tmp/temp]$ pipenv shell
Launching subshell in virtual environment…
 . /home/jthorpe/.local/share/virtualenvs/temp-HIOIE7wf/bin/activate
[/tmp/temp]$ which python
/usr/bin/python
[/tmp/temp]$ exit
exit
[/tmp/temp]$ pipenv run "which python"
[/tmp/temp]$

Please run $ pipenv --support, and paste the results here. Don't put backticks (`) around it! The output already contains Markdown formatting.
$ pipenv --support
Pipenv version: '2020.6.2'
Pipenv location: '/home/jthorpe/.local/lib/python3.7/site-packages/pipenv'
Python location: '/usr/bin/python3'
Python installations found:

3.7.3: /usr/bin/python3.7
3.7.3: /usr/bin/python3
3.7.3: /usr/bin/python3.7m
2.7.16: /usr/bin/python2
2.7.16: /usr/bin/python2.7

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.7.3',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '4.19.104-microsoft-standard',
 'platform_system': 'Linux',
 'platform_version': '#1 SMP Wed Feb 19 06:37:35 UTC 2020',
 'python_full_version': '3.7.3',
 'python_version': '3.7',
 'sys_platform': 'linux'}

System environment variables:

SHELL
LESS
WSL_DISTRO_NAME
WT_SESSION
PIPENV_ACTIVE
NAME
PWD
LOGNAME
PIP_PYTHON_PATH
HOME
LANG
WSL_INTEROP
LS_COLORS
TERM
LESSOPEN
USER
PIP_DISABLE_PIP_VERSION_CHECK
DISPLAY
SHLVL
PYTHONDONTWRITEBYTECODE
WSLENV
PATH
HOSTTYPE
WT_PROFILE_ID
OLDPWD
_
PIP_SHIMS_BASE_MODULE
PYTHONFINDER_IGNORE_UNSUPPORTED

Pipenv–specific environment variables:

PIPENV_ACTIVE: 1

Debug–specific environment variables:

PATH: /home/jthorpe/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/mnt/c/Windows/System32:/mnt/c/Windows:/mnt/c/Windows/System32/wbem:/mnt/c/Windows/System32/WindowsPowerShell/v1.0/:/mnt/c/Windows/System32/OpenSSH/:/mnt/c/Program Files (x86)/GnuPG/bin:/mnt/c/Android:/mnt/c/Windows/System32:/mnt/c/Program Files/dotnet/:/mnt/c/Users/Joe/AppData/Local/Microsoft/WindowsApps:/mnt/c/Users/Joe/Documents/opt/emacs26/bin:/mnt/c/Users/Joe/AppData/Local/hyper/app-3.0.2/resources/bin:/mnt/c/Users/Joe/AppData/Local/Programs/Microsoft VS Code/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games:/home/jthorpe/.local/bin:/mnt/c/Windows/System32/:/mnt/c/Windows/SysWOW64/:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games:/home/jthorpe/.local/bin:/mnt/c/Windows/System32/:/mnt/c/Windows/SysWOW64/
SHELL: /bin/bash
LANG: en_US.UTF-8
PWD: /tmp/temp

Contents of Pipfile ('/tmp/temp/Pipfile'):
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[dev-packages]

[packages]

[requires]
python_version = "3.7"

Contents of Pipfile.lock ('/tmp/temp/Pipfile.lock'):
{
    "_meta": {
        "hash": {
            "sha256": "7e7ef69da7248742e869378f8421880cf8f0017f96d94d086813baa518a65489"
        },
        "pipfile-spec": 6,
        "requires": {
            "python_version": "3.7"
        },
        "sources": [
            {
                "name": "pypi",
                "url": "https://pypi.org/simple",
                "verify_ssl": true
            }
        ]
    },
    "default": {},
    "develop": {}
}

I just tried to replicate this on my WSL2 Debian with Python 3.8.5 and had no issues doing so:
Linux DESKTOP-OFFICE 4.19.104-microsoft-standard #1 SMP Wed Feb 19 06:37:35 UTC 2020 x86_64 GNU/Linux

Thank you, did you install python 3.8.5 from source? as i cant seem to install it from apt
$ sudo apt install python3.8
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package python3.8
E: Couldn't find any package by glob 'python3.8'
E: Couldn't find any package by regex 'python3.8'

But python 3.7 is there
$ sudo apt install python3.7
Reading package lists... Done
Building dependency tree
Reading state information... Done
python3.7 is already the newest version (3.7.3-2+deb10u2).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

Pretty sure I'd installed it using apt. I didn't install it specifically to test this though - I already had it in place.

Ignore the message I just deleted. This was Debian not Ubuntu :) - I think I built from source according to this: https://linuxize.com/post/how-to-install-python-3-8-on-debian-10/ (though with 3.8.5 not 3.8.2).

For info, here's my pipenv --support output.
mike@DESKTOP-OFFICE:~/workspaces/temp/pipenv$ pipenv shell
Launching subshell in virtual environment…
 . /home/mike/.local/share/virtualenvs/pipenv-3mqHYeJE/bin/activate
       _,met$$$$$gg.          mike@DESKTOP-OFFICE
    ,g$$$$$$$$$$$$$$$P.       -------------------
  ,g$$P"     """Y$$.".        OS: Debian GNU/Linux 10 (buster) on Windows 10 x86_64
 ,$$P'              `$$$.     Kernel: 4.19.104-microsoft-standard
',$$P       ,ggs.     `$$b:   Uptime: 1 hour, 14 mins
`d$$'     ,$P"'   .    $$$    Packages: 602 (dpkg)
 $$P      d$'     ,    $$P    Shell: bash 5.0.3
 $$:      $$.   -    ,d$$'    Terminal: pipenv
 $$;      Y$b._   _,d$P'      CPU: Intel i7-7700 (8) @ 3.599GHz
 Y$$.    `.`"Y$$$$P"'         Memory: 2040MiB / 19175MiB
 `$$b      "-.__
  `Y$$
   `Y$$.
     `$$b.
       `Y$$b.
          `"Y$b._
              `"""

mike@DESKTOP-OFFICE:~$  . /home/mike/.local/share/virtualenvs/pipenv-3mqHYeJE/bin/activate
(pipenv) mike@DESKTOP-OFFICE:~$ pipenv --support
<details><summary>$ pipenv --support</summary>

Pipenv version: `'2020.6.2'`

Pipenv location: `'/home/mike/.local/lib/python3.8/site-packages/pipenv'`

Python location: `'/usr/local/bin/python3.8'`

Python installations found:

  - `3.8.5`: `/home/mike/.local/share/virtualenvs/pipenv-3mqHYeJE/bin/python3.8`
  - `3.8.5`: `/home/mike/.local/share/virtualenvs/pipenv-3mqHYeJE/bin/python3`
  - `3.8.5`: `/home/mike/.local/share/virtualenvs/pipenv-3mqHYeJE/bin/python3.8`
  - `3.8.5`: `/home/mike/.local/share/virtualenvs/pipenv-3mqHYeJE/bin/python3`
  - `3.8.5`: `/usr/local/bin/python3.8`

PEP 508 Information:

{'implementation_name': 'cpython',
'implementation_version': '3.8.5',
'os_name': 'posix',
'platform_machine': 'x86_64',
'platform_python_implementation': 'CPython',
'platform_release': '4.19.104-microsoft-standard',
'platform_system': 'Linux',
'platform_version': '#1 SMP Wed Feb 19 06:37:35 UTC 2020',
'python_full_version': '3.8.5',
'python_version': '3.8',
'sys_platform': 'linux'}

System environment variables:

  - `SHELL`
  - `WSL_DISTRO_NAME`
  - `WT_SESSION`
  - `PIPENV_ACTIVE`
  - `NAME`
  - `PWD`
  - `LOGNAME`
  - `PIP_PYTHON_PATH`
  - `HOME`
  - `LANG`
  - `WSL_INTEROP`
  - `LS_COLORS`
  - `VIRTUAL_ENV`
  - `TERM`
  - `USER`
  - `PIP_DISABLE_PIP_VERSION_CHECK`
  - `SHLVL`
  - `PYTHONDONTWRITEBYTECODE`
  - `PS1`
  - `WSLENV`
  - `PATH`
  - `HOSTTYPE`
  - `WT_PROFILE_ID`
  - `OLDPWD`
  - `_`
  - `PIP_SHIMS_BASE_MODULE`
  - `PYTHONFINDER_IGNORE_UNSUPPORTED`

Pipenv–specific environment variables:

 - `PIPENV_ACTIVE`: `1`

Debug–specific environment variables:

  - `PATH`: `/home/mike/.local/share/virtualenvs/pipenv-3mqHYeJE/bin:/home/mike/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/mnt/d/Python/Python38/Scripts/:/mnt/d/Python/Python38/:/mnt/c/Python/Python38/Scripts/:/mnt/c/Python/Python38/:/mnt/c/Program Files (x86)/Common Files/Oracle/Java/javapath:/mnt/c/Program Files/Java/jdk-10.0.1/bin:/mnt/c/Windows/system32:/mnt/c/Windows:/mnt/c/Windows/System32/Wbem:/mnt/c/Windows/System32/WindowsPowerShell/v1.0/:/mnt/c/WINDOWS/system32:/mnt/c/WINDOWS:/mnt/c/WINDOWS/System32/Wbem:/mnt/c/WINDOWS/System32/WindowsPowerShell/v1.0/:/mnt/d/Program Files/PuTTY/:/mnt/d/Program Files/Geth:/mnt/c/Program Files (x86)/Windows Kits/8.1/Windows Performance Toolkit/:/mnt/c/Program Files/Calibre2/:/mnt/c/WINDOWS/System32/OpenSSH/:/mnt/c/Android:/mnt/c/Program Files/SourceGear/Common/DiffMerge/:/mnt/c/Program Files (x86)/NVIDIA Corporation/PhysX/Common:/mnt/c/Program Files/NVIDIA Corporation/NVIDIA NvDLISR:/mnt/d/Program Files/Apache/apache-maven-3.6.0/bin:/mnt/d/Program Files/Microsoft VS Code/bin:/mnt/c/WINDOWS/system32:/mnt/c/WINDOWS:/mnt/c/WINDOWS/System32/Wbem:/mnt/c/WINDOWS/System32/WindowsPowerShell/v1.0/:/mnt/c/WINDOWS/System32/OpenSSH/:/mnt/d/Program Files/nodejs/:/mnt/d/Program Files/Git/cmd:/mnt/c/Program Files/dotnet/:/mnt/c/Program Files/Docker/Docker/resources/bin:/mnt/c/ProgramData/DockerDesktop/version-bin:/mnt/c/Program Files (x86)/Elm/0.19.1/bin:/mnt/c/Users/mjnic/AppData/Local/Microsoft/WindowsApps:/mnt/c/Users/mjnic/AppData/Local/GitHubDesktop/bin:/mnt/c/Users/mjnic/AppData/Local/atom/bin:/mnt/d/Cryptocurrency/Wallets:/mnt/c/Users/mjnic/AppData/Local/GitHubDesktop/app-1.0.1/resources/app/git/cmd:/mnt/c/Users/mjnic/AppData/Local/Microsoft/WindowsApps:/mnt/d/Anaconda/Library/bin:/mnt/d/Program Files/cmder:/mnt/d/Program Files/cmder/bin:/mnt/d/Program Files (x86)/Nmap:/mnt/c/Users/mjnic/AppData/Roaming/npm:/mnt/d/src/flutter/bin`
  - `SHELL`: `/bin/bash`
  - `LANG`: `en_US.UTF-8`
  - `PWD`: `/home/mike`
  - `VIRTUAL_ENV`: `/home/mike/.local/share/virtualenvs/pipenv-3mqHYeJE`

---------------------------

</details>

@jthorpe6 Has this been resolved?

Hey, I'm unable to test this, as I don't use WSL anymore. I'm happy for it to be closed.

Not sure if there's any interest in revisiting this, but I've been running into the same issue as described above. If it helps, I also get the exact behaviour using Poetry's poetry shell command. I wonder if WSL doesn't play nice with pexpect

@ividito What commands were you running/what Pipfile or other useful context?  I have a friend at work that runs pipenv in WSL and I haven't heard of any complaints from him, so I would like to ask for more details to be able to reproduce it.

@ividito What commands were you running/what Pipfile or other useful context? I have a friend at work that runs pipenv in WSL and I haven't heard of any complaints from him, so I would like to ask for more details to be able to reproduce it.

I've dropped pipenv since I commented, but here's my Pipfile from when I last observed this issue:

[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[dev-packages]

[packages]

[requires]
python_version = "3.7"

I ran into the issue while running pipenv shell . I do still run into this issue with poetry shell , and I've generally adapted my workflow to compensate rather than continue to wrestle with it. If there are specific resources I can provide or troubleshooting steps I can take that would help with reproducing it, let me know.


A11y_VisualStudioCodeClient_Terminal_ScreenReader: NVDA/JAWS Announces Blank when Using Arrow Keys on Terminal Commands

"Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility or the Divisional Driver".
GitHub bugs
#A11ySev2; #A11yTCS; #A11yMAS; #DesktopApp; #Visual Studio Code Client; #WCAG1.3.1; #DesktopApp; #BM-VisualStudioCodeClient-Win32-Aug2022; #NVDA; #Win32; #Win11; #A11yeDAD; #ScreenReader; #FTP; #NVDA; #AINotInScope;
Actual:
Upon using the left or right arrow key to navigate the characters in the Terminal command, NVDA announces a blank instead of correctly providing the character feedback.
Expected:
As the arrow keys are pressed, NVDA should announce the characters in the command being navigated.
Environment Details:
Application Name: Visual Studio Code
Version: 1.80.0-insider (user setup)
V8: <IP_ADDRESS>-electron.0
Windows 11 22H2 OS build 25869.1000
Screen Reader: NVDA (2023.1)
Repro Steps:
1.Launch Terminal in Visual studio Code.
2.Turn on NVDA.
3.In terminal, type any command.
4.Use left/ right arrow key to move NVDA focus within the written command.
5.Check NVDA behavior
User Impact:
The incorrect feedback provided by NVDA/JAWS affects the accessibility and usability of the Terminal application for users who rely on screen readers. This behavior hinders their ability to navigate and interact with command-line interfaces effectively.
"Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier."
Attachments:
https://github.com/microsoft/vscode/assets/105198219/8e2b2bad-e04e-4424-ba57-5414199a3f5f

@meganrogge not sure if we're tracking this but we're talked about the fix I had in mind for this which was to synchronize xterm.js' textarea content with the help of shell integration.

This also happened to me when I was trying to edit a recent command in the integrated terminal (weirdly, arrowing on the newly typed command doesn't seem to be an issue).
This makes integrated terminal nearly unuseable with the long command (in which typo is very much possible).
I have also created the issue here: #185163

@meganrogge
Verified the issue in insiders 1.82.0
The issue is still repro.
Please tell me in which version the issue is fixed.

#Regressed:08-21-23; #PendingInput;

This works as shown here

As the arrow keys are pressed, NVDA should announce the characters in the command being navigated.

That is, until the terminal loses and then regains focus. That's a separate issue IMO and thus can be handled as a different bug.

@meganrogge
I verified the issue in below version of Visual Code Insiders.
Version: 1.82.0-insider (user setup)
Issue is still reproing.
Please tell me in which version of Visual code Insiders the issue is fixed.

@ManishaC1 when reporting insiders versions you need to go to Help > About and copy out all the version details. In particular the commit and date are needed to identify the insiders version.

This was verified by @jooyoungseo on both windows and mac.
@rperez030 is having issues with it so we're going to meet about that.

@rperez030 and I met and he showed it not working on windows. I'm guessing this is a conpty issue

what were version of windows are you on @pawin35 ?

@meganrogge  I use windows 11 Pro Version 22H2 (OS Build 22621.2215)

I'm also able to repro on the latest Insiders on Windows using NVDA.

This is working on Windows for me.  (10.0.22621 Build 22621) @rzhao271 which version do you have?

In System > About I see version 22621.2134

@Tyriar
Below is the Environment Details of the latest Visual Studio Insider installed in my system:
Version: 1.82.0-insider (user setup)
Commit: 35be9bf683eace09796e59d54f1f225bbc3a7866
Date: 2023-08-30T05:33:47.972Z
Electron: 25.7.0
ElectronBuildId: 23434598
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621

Yeah I can reproduce now. Seems to have regressed

currentCommand.commandStartX is undefined for Windows

@meganrogge it may also in the wrong position on Windows, if I remember it's typically at 0. This will get fixed eventually in conpty.

what's odd to me is that it was working for me before
https://github.com/microsoft/vscode/issues/186904#issuecomment-1688550710

I'm not sure when it would be undefined, I guess it may be inconsistent? Command executed does not ensure command started had occurred:
https://github.com/microsoft/vscode/blob/d89cac9ac10d1aa4310e827c458f2bea597685ed/src/vs/platform/terminal/common/capabilities/commandDetectionCapability.ts#L441-L450

We need info on the command line before command execution @Tyriar and for windows, we don't have that currently.

As with a lot of things on Windows ptys you'll need to add some heuristics here. It might work fine to just use commandStartX ?? 0. You could also try add some basic prompt detection for standard pwsh/powershell.exe

To verify, enable terminal.integrated.experimental.syncTextArea. Then follow the steps here. https://github.com/microsoft/vscode/issues/186904#issuecomment-1699945147

@meganrogge
I did as you said. But still the issue is still reproducing. Please help

Environment details:
Version: 1.82.0-insider (user setup)
Commit: e7756c8870ee1df7360e6624e220534174039b02
Date: 2023-08-31T05:33:59.698Z
Electron: 25.7.0
ElectronBuildId: 23434598
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
NVDA version: 2023.1

you need to update @ManishaC1 you're using an old version

Verified the issue in below environment.
Version: 1.82.0-insider (user setup)
Commit: dd112ec0243c4b42bb3106e64df1688e242a6559
Date: 2023-09-01T05:33:39.919Z
Electron: 25.7.0
ElectronBuildId: 23434598
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Issue is fixed.
#Closed; #Regressed:09-04-23;
https://github.com/microsoft/vscode/assets/105198219/3bf9cc14-f8c9-4f0d-91e3-efe785232895


Video dome 180

closing https://github.com/BabylonJS/Babylon.js/issues/6624
The video dome will now support 180 VR videos. the parameter can be set dynamically or using the options (options.halfDomeMode) .

The second sphere is a black sphere with no material. It's just there to hide the back of the sphere. Nothing is rendered on it.
So, the main mesh (sphere) always renders the video. In 180 mode there is an extra black mesh to hide the back.
There is no way around it without a second mesh sadly.  I could use a sphere with slice 0.5, but then I need to change the default uv coordinates of the sphere, which, I thought, was an overkill for this.

Ok I misunderstood :) so all good for me!


Fix/virtual scroll stacking

Pull request checklist
Please check if your PR fulfills the following requirements:

[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Build (npm run build) was run locally and any changes were pushed
[ ] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type
Please check the type of change your PR introduces:

[X] Bugfix
[X] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
If you have and observable that throws data at virtual-scroll, you will have an overlapping issue.
Issue Number: 18409
What is the new behavior?
You can optionally set new parameter that prevents content stacking if needed.
Does this introduce a breaking change?

[ ] Yes
[X] No

Other information
One of the many issues .
Pls check if the fix works.

Can someone Implement the same feature without any type errors? We need optional minimumItemHeight parameter to fix stacking content bugs.


Details of KNN classifier for ImageNet-LT

Thanks for your interesting work!
I would like to know some details of how you perform kNN classifier on the ImageNet-LT dataset, including the number of K and the temperature used in exp(cos_sim / t). Besides, which set of the dataset (train, validation or both) is used to be the support set to search for kNN? Thanks!

Hi, for KNN classifier on ImageNet-LT, we used the support set from training set and use the same number of K and temperature as in the experiment on ImageNet. Sorry for the late reply


console.log in a plugin causes a "EPERM: operation not permitted, write"

Prerequisites

[ ] Can you reproduce the problem in safe mode?
[x] Are you running the latest version of Atom?
[x] Did you check the debugging guide?
[x] Did you check the FAQs on Discuss?
[x] Are you reporting to the correct repository?
[x] Did you perform a cursory search to see if your bug or enhancement is already reported?

Description
The bug has since been reported in several places. The current use case that bothers is that console.log is used in the eslint-plugin-react which is used in the widely adopted eslint-config-airbnb which is in turn consumed by eslint and AtomLinter/linter-eslint.

https://github.com/AtomLinter/linter-eslint/issues/579 (closed because the AtomLinter team cannot do anything with that)
https://github.com/yannickcr/eslint-plugin-react/issues/416#issuecomment-177119027 (closed because the issue was not stated properly, or simply comments on closed issues were ignored)
https://github.com/airbnb/javascript/issues/978#issuecomment-241332995 (can't be fixed in eslint-config-airbnb yet because of other dependencies)
https://github.com/yannickcr/eslint-plugin-react/issues/783#issuecomment-241778161 (looks like a won't fix because eslint-plugin-react "is a Node module, so using console is valid")

Here's the exception that pops up in Atom:
Error: EPERM: operation not permitted, write
    at Error (native)
    at Object.fs.writeSync (fs.js:706:20)
    at SyncWriteStream.write (fs.js:2064:6)
    at Console.log (console.js:39:16)
    at EventEmitter.Program (__CENSORED__\node_modules\eslint-plugin-react\lib\rules\require-extension.js:103:17)
    at emitOne (events.js:95:20)
    at EventEmitter.emit (events.js:182:7)
    at NodeEventGenerator.enterNode (__CENSORED__\node_modules\eslint\lib\util\node-event-generator.js:40:22)
    at CodePathAnalyzer.enterNode (__CENSORED__\node_modules\eslint\lib\code-path-analysis\code-path-analyzer.js:608:23)
    at CommentEventGenerator.enterNode (__CENSORED__\node_modules\eslint\lib\util\comment-event-generator.js:97:23)

Here's one of the faulty lines:
https://github.com/yannickcr/eslint-plugin-react/blob/v3.16.1/lib/rules/jsx-quotes.js#L52
      /* eslint-disable no-console */
      console.log('The react/jsx-quotes rule is deprecated. Please use the jsx-quotes rule instead.');
      /* eslint-enable no-console */
      isWarnedForDeprecation = true;

There are more occurences of this pattern in eslint-plugin-react:
https://github.com/yannickcr/eslint-plugin-react/blob/e84af68a3bf48c9a34f14e73a8b4dc1ed7488bab/lib/rules/require-extension.js#L103
        /* eslint-disable no-console */
        console.log('The react/require-extension rule is deprecated. Please ' +
                    'use the import/extensions rule from eslint-plugin-import instead.');
        /* eslint-enable no-console */
        isWarnedForDeprecation = true;

https://github.com/yannickcr/eslint-plugin-react/search?utf8=✓&q=console.log&type=Code
Steps to Reproduce

Install atom-linter, atom-linter-eslint plugins, eslint and eslint-plugin-react modules.
Use a .eslintrc that uses one of the eslint-plugin-react's deprecated rules; for example, the eslint-config-airbnb base config.
Run the linting on a JS file.

Expected behavior: No error.
Actual behavior: The exception mentioned above.
Versions
Atom 1.9.9 on Windows 7 64bit
atom --version
Atom    : 1.9.9
Electron: 0.37.8
Chrome  : 49.0.2623.75
Node    : 5.10.0

apm --version
apm  1.10.0
npm  2.13.3
node 0.10.40
python 3.5.0
git 1.9.4.msysgit.1
visual studio

+1 Just ran into this today

+1 same thing here

Had the same issue in Atom. An interim solution was to use these eslint versions:
"eslint": "^2.13.1",
"eslint-config-airbnb": "^9.0.1",
"eslint-plugin-import": "^1.14.0",
"eslint-plugin-jsx-a11y": "^1.5.5",
"eslint-plugin-react": "^5.2.2",

Same issue here with atom-typescript. I'm using a custom version of the compiler in atom-typescript that uses console.warn and I get the same error. I noticed that this issue happens only when using a child process for the compiler (atom-typescript uses child_process if you don't need debug on it). At the beginning I thought that it was a runInContext issue, but it works correctly when I disable child process creation. Hope this helps to fix it.

Yup, this one is super annoying.

This is super annoying. But in windows, running the Atom as an administrator solves the issue for me.

Annoying. And running Atom as administrator didn't helped.

@50Wliu Were you able to reproduce this? I haven't ever run into this when using console.log in any of my packages (though I haven't used it recently).

I've only ever seen this when the console.log is in a Node.js process launched from the package.

@lee-dohm What is needs-reproduction about this?

Install Atom on Windows
Install the AtomLinter plugin
Set up a project that makes a console.log call inside eslint:

npm init -y
npm install --save-dev<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>echo '{ "extends": "airbnb" }' > .eslintrc
echo "'foo';" > foo.js
$(npm bin)/eslint .
# Which gives you this on stderr:
# The react/require-extension rule is deprecated. Please use the import/extensions rule from eslint-plugin-import instead.

I cannot confirm this on Atom 1.12.0-dev-18089df, Windows 10 x64.

I reproduce it 15,000 times a day in Atom 1.10.2, Windows 7 and Windows 10.  Maybe it's fixed in dev?

Same, it's definitely still broken in 1.10.2.

If anyone interested in a workaround, it's here

Same here.. Really annoying issue, should really be acknowledged.

Has anyone been able to reproduce this in the 1.11 betas or 1.12-dev?

@50Wliu Has the version of the embedded node changed between 1.10.2 and 1.11? Also make sure that ESLint is generating the warning message on the CLI.

It has in 1.12.0; I can't recall whether or not it has in 1.11.0.
I tried reproducing this again just to be sure using @vdh's example.  However, I do not get any logging about react/require-extension; rather, I get the following error:
Configuration for rule "eol-last" is invalid:
        Value "always" must be an enum value.

@50Wliu Apologies, it looks like it needs eslint-config-airbnb-base locked to an older version to re-create the scenario:
npm install --save-dev<EMAIL_ADDRESS>

Ok, thanks for the update.  I still can't reproduce this on Atom 1.12.0-dev-fcddf83.  The warning gets printed to the dev tools console as expected.

Can reproduce on 1.11 beta but not on 1.12.0-dev-fcddf83.

Assuming fixed in Atom 1.12.0 or whenever Atom starts using Electron 1.4.1.

I fixed this problem by adding
"react/require-extension": 0,
to .eslintrc.js . and my atom works fine now

@Edge00 You don't need to do that anymore, later versions of eslint-config-airbnb have already solved the warning that triggers this Atom issue (by updating to a later version of eslint-plugin-react).

@vdh  thank you , got it

@mikecousins I want to know the name of your atom's fant-family, it's very good-looking

It's called Operator Mono.
http://www.typography.com/blog/introducing-operator
On Dec 4, 2016 6:38 PM<EMAIL_ADDRESS>wrote:
@mikecousinshttps://github.com/mikecousins I want to know the name of your atom's fant-family, it's very good-looking

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/atom/atom/issues/12504#issuecomment-264749506, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAsSbq7t5Biur3T13pM7qGKXVb8qOmT8ks5rE2sXgaJpZM4JrLFv.

@mikecousins  Thank you!


Proposal: removing "_helpers.tpl"

Hello
we currently introduce helm to our development teams and constantly see people stumble over the "_helpers.tpl" file.
What we understood from the docs and some talks is that the "values.yaml" file is sort of the API for a chart and everything that needs to be configurable is put in there. Either due to multiple deployments of the chart in the same cluster (Chart release name in tiller) or mutliple deployments of the chart in the same namespace - so resources don't conflict and the right ones communicate with each other.
This has some complexity for charts that work standalone or as sub-chart (requirement).
Since the "values.yaml" is the API of the chart everything should be in there. I understand there is a nameOverride in the default chart - but it seems cumbersome to tweak the _helpers file and the values file to end up with the desired resource names in kubernetes.
Currently if using sub-charts and I want to change some names I copy-paste the template section of that sub-charts _heplers.yaml into my own _helpers.tpl and use the global scope of those defines to override them. And hoping the sub-chart does not change too much.
So my proposal would be to abolish "_helpers.tlp" from the default chart created by "helm create" - (maybe not entirely the whole mechansim with the .tpl files - if someone needs to put functions or utils somewhere). But trying to mostly or only rely on values.yaml.
I'm not complety sure this is possible or what the reason for introducing the _helpers.tpl file is and if the values.yaml behaves well enough in these cases when charts are used standalone or as sub-charts. But there is a global section in values.yaml as well. So it seems possible to achieve.
Let me know your thoughts :)

We do not plan on removing _helpers.tpl from helm create as it's a convenient standard for chart authors as a placeholder for helper functions. If, however, you do not like the format that helm create provides, you can provide your own set of templates via the "starter chart" feature of helm create. See https://docs.helm.sh/developing_charts/#chart-starter-packs for more information on starter charts.
Thanks!


Request Contribution

Hello @mustapha-ghlissi  is this package open for contributing to

Yes it is with requirements.


chore(deps): update common-dev-assets digest to d66ebf9

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
89a62dd -> d66ebf9

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

GitHub release
v3.0.0

Your semantic-release bot :package::rocket:


[libxlsxwriter] upgrade to 0.8.7

Upgrade to the latest version of libxlsxwriter.
Closes #6982

Thanks for the update!


Higlight currently active app

Background:
The currently active app is the last one on which RUN was clicked. When saving/updating files in the folder of this app, it automatically reloads.
In some situations it can be confusing which app currently is active, so the idea is to highlight the active app in the Workbench UI.
Furthermore, when clicking COPY or NEW, the UI presents the new app, but it does not become active until RUN is clicked. If one forgets to click RUN, live reload on file save will not work for the new app, which can be confusing. A solution is to set the active app to the new app when doing COPY or NEW.
Tasks:

Display the active app highlighted in the UI
When COPY or NEW is clicked, make the new app the active one (set current base directory and monitor files, update UI highlight)

It turns out to be tricky to automatically select the active app on NEW and COPY. The problem is that sending RELOAD (reloading the active app on save) reloads the app from the current URL. A new URL needs to be set in the Viewer for the new app, and this is used by clicking RUN.
To make it clear that you should click RUN, instructions have been updated to clearly say this, in dialog boxes for NEW and COPY, and on the Getting Started screen.
Also, the active app is now highlighted in the UI, the remind of which app is currently active.
Updates are found in these commits:
https://github.com/evothings/evothings-studio/commit/f31386b8a8be8e19ad0e5e19b69da429d03057eb
https://github.com/evothings/evothings-studio/commit/8788cfa0d74c2ab8c63f43241102097efb04e8de


New package: Bmlt v0.1.0

Registering package: Bmlt
Repository: https://github.com/sylvaticus/Bmlt.jl
Created by: @sylvaticus
Version: v0.1.0
Commit: 568ae062a29f692607ca02c6135929f435fe5036
Reviewed by: @sylvaticus
Reference: https://github.com/sylvaticus/Bmlt.jl/issues/1
Release notes:

First registered release of Bmlt, the Beta Machine Learning Toolkit

It seems I am 1 character lost for auto merging.
However I can't see many other names for the "Beta Machine Learning Toolkit"... BetaMLT ? I believe Bmlt is quite a unique combination of letters. Also I have submodules, so the user have already to type using Bmlt.Nn or using Bmlt.Perceptron...

How about BetaML?
I think the Toolkit is implied, given that it is indeed a package.
I think library, package, and toolkit are synonymous

@oxinabox  I agree with you and I am eventually ok for that name, still I would have preference for the original Bmlt name if possible. The reasons is that (1) I will refer to the package always as a "toolkit", so I believe "t" is somehow expected in the name (a bit like "SciKit"), and (2) I may want in the future to change the meaning of the leading "B" from "Beta" (the name of my university lab) to a more universal  (and perhaps more appropriate) "Basic" meaning...
So, if possible I would stay with the original name Bmlt, otherwise no pbm, I'll use the suggested "BetaML", thank you.

@oxinabox could you please give your ok with the name (even if you would prefer BetaML) or should I rather change it ?
Thank you..

I am not in  position to make requirements of you.
I personally think Bmlt is a almost incomprehensible set of random letters.
At no opoint looking at it will I know what it means.
Only other things I can think of that exist in the wild like that at NLTK, HTK, HFST.
All of which i also don't like.
But I am neither the author of the package, nor the maintainer of the registry.
So I am in no position to do anything more than give suggestions.
Its not my game.
You need to talk to @StefanKarpinski , @DilumAluthge  or one of the other maintainers of the registry

It's ultimately your call, but I agree that this is not a great name and it's very much not in the naming style of the ecosystem. Whether we want them to or not, names matter; if you want this to have a chance at success, I would suggest a name that is more harmonious with the Julian style. BetaML, as Lyndon suggested, seems like a good name, for example. If you're going to stick with Bmlt then you should at least capitalize all the letters to conform to the Julia convention of capitalizing acronyms, so the package would be BMLT instead.

ok, gonna change the name, thank you everyone...

Please close this pull request --> pull request under the new name: https://github.com/JuliaRegistries/General/pull/16047


macOs 'Internal Error: Unable to follow symbol for "Record"' when rollupTypes set to true.

Thank you for this plugin.
Like in this [issue](Internal Error: Unable to follow symbol for "Record"), suddenly I have this error.
I can't provide a reproduction link as this is a pretty complex library code. But, like in the mentioned issue, it disappears when rollupTypes: false.
Note that I use swc to compile.
I am on:

macOs 12.7.3
node 20.15.0
vite 5.4.6
vite-plugin-dts 4.2.3
rollup-plugin-swc 0.2.1

Thank you.

I ran into the same issue.
I noticed there is a FAQ about it, https://github.com/qmhc/vite-plugin-dts?tab=readme-ov-file#internal-error-occurs-when-using-rolluptypes-true, indicating it is a limitation of the @microsoft/api-extractor.
In my case I was able to resolve the issue by correcting the import to be more exact.

I also encountered this issue when the typescript was upgraded from 5.6.3 to 5.7.2 in https://github.com/lawvs/zod-compare/pull/42, and I did not specify baseUrl in tsconfig.json.
  plugins: [
    dts({
      rollupTypes: true,
      include: ["src/**/*.ts", "src/**/*.d.ts"],
      exclude: ["src/**/__tests__/*"],
    }),
  ],

I also encountered the same issue After I downgraded TypeScript to 5.4.2 the error disappeared

For me it was because of ts version mismatch between api-extractor and local package.json. You can override it and it works fine:
  "pnpm": {
    "overrides": {
      "typescript": "^5.7.2"
    }
  }

Thank you @brc-dd, that works!


teleport command doesn't teleport you to the correct location

Using the tp command
Screencast from 08-21-2024 10:25:52 AM.webm
Nuke disk transform component

Using the loc command

The actual position of the disk is around ~tp 12.2 41.5 3

It works perfectly on reach for some reason

on omega too


Test: ESLint fix all command

[x] Windows @kieferm
[x] OS X @kieferrm
[x] Linux @kieferrm

Steps:

install eslint
see https://github.com/Microsoft/vscode/issues/5654 how to setup eslint to get fixes
F1>ESLint
you should get a command to fix all auto-fixable problems
execute it
ensure that all problems are fixed.

I believe I was tagged on this accidentally.  @kieferrm  is similar to @kieferm


Switching offline repo to online sets branch to "master"

Needs detection of default branch like "new online repo".

ref https://github.com/abapGit/abapGit/pull/3983/files

so, this uses the old poup
I'll use this opportunity to suggest removing the switch offline <-> online feature :)

I'm adding the switch to the new repo settings page and it will do a remove and recreate keeping the settings. We can remove switch, attach, and detach, afterwards.

Close by #4705


How to search from 20M vectors and How much Memory would it take to save 20M vectors of size 2048?

I followed your work and got the idea that how it's done. I have Embeddings from ResNet which is a 2048 length vector. I got an approximation that it within a search span of 17M vectors, with default configuration, it takes less than 1 second. That is pretty decent for me. But I want to know what would be the size of this data structure? How would it work? IF I save 20M vectors in my memory, it'll blow.
I tried something:
dim = 2048
num_elements = 10000

# Generating sample dat
data = np.float32(np.random.random((num_elements, dim)))

data1 = data[:num_elements // 2]
data2 = data[num_elements // 2:]

# Declaring index
p = hnswlib.Index(space='cosine', dim=dim)  # possible options are l2, cosine or ip
p.init_index(max_elements=num_elements//2, ef_construction=100, M=26)
p.set_ef(10)
p.set_num_threads(2)
p.add_items(data1)

p.save_index('part1.bin')

Then I used sys.getsizeof(p) and I got 52bytes. That is unreal. Secondly by checking the size of part1.bin, it gives me 40MB. So  in that sense, if I make batches and save every structure like this,

How can I search from these parts?
How much memory would it exactly take?

Hi @deshwalmahesh,
I am not sure. Are you talking about ram or disk?

Hi @deshwalmahesh,
I am not sure. Are you talking about ram or disk?

I'm talking about RAM. index has to be in memory at the server side to give the results. right?

I'm talking about RAM. index has to be in memory at the server side to give the results. right?

RAM consumption after the index is created is roughly: the size of the data (4DN for float32) + size of the links (~ 2MN)+some overhead that depends on the number of threads (say, 20-30%).
And as the code is done in C++ there is not easy way to measure the RAM consumption without modifying the code.
Note that when you call add_items(data1) you will have two copies of your data (one in data1 and another in the index), so for large indices you would like to load data in small portions (e.g. load 1Mb add to index, load another 1Mb ...)

RAM consumption after the index is created is roughly: the size of the data (4DN for float32) + size of the links (~ 2MN)+some overhead that depends on the number of threads (say, 20-30%).
And as the code is done in C++ there is not easy way to measure the RAM consumption without modifying the code.
Note that when you call add_items(data1) you will have two copies of your data (one in data1 and another in the index), so for large indices you would like to load data in small portions (e.g. load 1Mb add to index, load another 1Mb ...)


Idea: access parent cursor in doc helpers

That would be pretty useful to be able to access the parent cursor from a document returned to the helpers method, and in particuler its previous / next docs in the cursor.
That would make things like grouping elements by an attribute very clean. Examples: for a chat app where you would need to insert a separator for each day, a contacts list where you need to group people by the first letter of their names, etc.
The code could look like this:
Messages.helpers({
  '$newDay': funtion(prev, next) {
    return (!prev || (this.day !== prev.day)) ? this.day : null;
  }
});

Haven't been able to dig into minimongo yet - but @dburles do you think this is something that can be done?

Interesting idea @gwendall. I haven't looked into anything like this yet.


Cleanup warnings in tests

Description
closes #1805 by clearing up warnings again.  This time, I've added the filterwarning "error:::napari"... which will turn all warnings originating from within napari into errors unless explicitly allowed using with pytest.warns(...).

This is very interesting, our pre-test is now failing because of what looks like some numpy changes deprecating something! We've also got a magicgui segfault unrelated to this PR and another fail. We'll probably have to #2045 fail too if I rerun.
Overall I'm good with this PR, we don't want uncaught warnings floating around, but we'll probably want to get the CI back up to 100% passing before final review and merge

but we'll probably want to get the CI back up to 100% passing before final review and merge

yeah, i agree.  sounds good

I've just updated from master, but it looks like we've got a couple tests now failing specifically on this PR. Havn't looked at details yet

they seem to be mostly just our own warnings that don't normally get triggered.  should be easily fixed, but will take a look

you know, an alternative to turning all of our "unasserted" internal warnings into errors (i.e. being explicit about testing our warnings), is simply auto-ignoring them all (i.e. saying, it's always fine if we emit warnings ourself... its the ones that we didn't expect that we want to find).  thoughts?

you know, an alternative to turning all of our "unasserted" internal warnings into errors (i.e. being explicit about testing our warnings), is simply auto-ignoring them all (i.e. saying, it's always fine if we emit warnings ourself... its the ones that we didn't expect that we want to find). thoughts?

I'm open to that, but I also didn't mind turning ours into errors. We add a lot of warnings now during deprecations and so don't want to be accidentally using those methods/ properties still internally. This could help catch that, which I think was the original motivation. I'd defer to @jni if he had a strong opinion though, I don't really mind either way

ok, will leave them in and just "allow" the warnings that come from really early versions of stuff.  the "--pre" warnings are more interesting, and I need to look at them

ready again @sofroniewn.  main changes from last time:

fixing all of the deprecation warnings from numpy 1.20 (e.g. np.float -> float)
warnings are only errors if they come from a napari. module (which excludes plugin napari_* modules)
any vendored modules within napari are exempt
whitelisted the "update your dask" warning triggered in the MIN_REQs check


Fixed PropTypes and React.createClass deprecation warnings

Basically solution for this stuck PR https://github.com/JedWatson/react-hammerjs/pull/67

Can anyone pls review this and possibly accept and push to npm?

@JedWatson :)

@JedWatson I think this is a high priority, can you see it? Only merge && publish

@JedWatson could you please cut this? Happy to help with the release if you want.

@JedWatson bump! (please 😄 )

This just became slightly more pressing, now that React 16 has been released.

We are using hammer and are eager to update to react 16! Any movement on this?

No longer relevant.

Why is this no longer relevant? Is there a different PR that adds React 16 support? Or does this just mean go find a different component?

Got it. Thanks!


add darwin/osx as another option

saw that the format string for OSX was really much the same as linux, hope this helps, i love running it on the machines here at home.

This is brilliant @kinabalu, thankyou, I was kinda hoping someone would add MacOS :smile:
Would you be able to add an example output to the README as well? Can be as an extra commit/PR if you like.

I already added it but just hadn't gotten around to pushing it up, looks like I can add it to this PR.  Let me know if you need anything else!

Excellent, thanks again @kinabalu :tada:


calendar does not open when clicking calendar symbol

it works when clicking the field but not the actual symbol.at least not in FF and chrome

@smax3 Thanks your feedback, I will fix it in next version.


fix: use correct array ops

Tests?

There are already tests for all of this. Changing from "UnsupportedOp" to an op from "prelude" does change the codepath, but it still passes.

Can we do an execution test now? (If execute-llvm doesn't lower array ops then fair enough but otherwise it might only be a few lines and would be good to have!)


Ban Appeal attempt number 2: LimitlessJoker

After providing new evidence to Ras and refurther explaining the situation and having abuse displayed publicly by another staff member about the first appeal, I would very much appreciate another opportunity to be unbanned from the server, I wish the situation would ease and drop but sadly since the ban it has only provided more issues. I'm kindly asking to be unbanned even in the case of a punishment has to be made in response for me getting unbanned for example a reset as I am very open to my mistakes which I'm very up mostly sorry for yes I will admit that I didn't mean to store abused items and for that I am truly sorry and it was not my intention but I am definitely most truly sorry for not reporting the issue to a lead mod staff member sooner rather than later. I will try to learn from this mistake in future if I ever become staff on a different pixelmon based server. But after some reflection time I would kindly at minimum like my access back onto the server I care about very much. And if starting from scratch is the case as punishment. So be it. I also would like my staff role back but I'm happy to just remain a normal bystander on the server like everyone else if that's what it takes.

Considering recent developments, I am going to go ahead and deny this appeal as well. Please do not remake them freely, as if there is a chance at an unban, it'll come from me personally.
Should you have questions, feel free to reach out. Some of the reasons to keep your ban are extremely private, and I feel they should not be publicized in such a way.


Define base structure for registration and login api

As ZITADEL developers we want to have the basic knowledge how we want to implement the login and register API, so we can start with the implementation afterwards.
Acceptance Criteria

[ ] Output: Document as base for the following implementations
[ ] How is the api designed, services?
[ ] Basic structure of resources
[ ] Basic structure of endpoints
[ ] How do we handle the context? Administrator, self service, etc
[ ] How do we handle backwards compatibility?

@livio-a @peintnermax can you please describe the outcome of the spike here? thanks

Basic decisions (see also Jamboard esp. page 4):

resource centric endpoints:

/users

/_search
/ (<- create; auth methods need to be added / managed on separate resources)
/id  (Get User with all attributes incl. auth methods (based on permission))

/passwords

/ (<-create with a userID in body)
/_check

Password will be checked for a user for an existing session (if provided by a session token in the body) or a new session created
a new session token will be returned

/passkeys
/sessions

/_search
/id

/terminate

/polices

/login
/domain
...

IDPS
...

context (org / instance) in body or query params (GET)

GetLoginPolicy:
/policies/login?instance=true
/policies/login?org_id=id
/policies/login?org_domain=test.com
SetLoginPolicy

{
  "org_id": "id",
  ...
}

no My* endpoints, e.g. MyUser, MyOrg, ...

By having instance wide endpoints (e.g. UserByID) and without My* endpoints we need to check if a user is able to access a resource and what data within a resource after calling the database (compare AuthContext (Token / Session) to resource):

"authenticated" (self service): authzCtx.UserID == resource.ID
GetUserByID:

User within same organisation: all data / PII
User from other organisations: only id, display name, ?

SDKs (ongoing discussions):

provide ability to build your own login (and generally use it for api calls)
languages / frameworks

node.js
Android
iOS

two SDKs:

frontend (ability to create UI components; will require / include backend)
backend

create login flow with parameters
includes api calls
proxy OIDC / SAML endpoints?

idea: start with node sdk to be able to use it in the customer portal


sdml-core requires rustc 1.75

Compiling the code since recent refactor fc02d43 with rustc 1.70 (the version of rustc available in Debian unstable), fails like this:
error[E0562]: `impl Trait` only allowed in function and inherent method return types, not in trait method return types
  --> sdml-core/src/load.rs:41:28
   |
41 |     fn resolver(&self) -> &impl ModuleResolver;
   |                            ^^^^^^^^^^^^^^^^^^^
   |
   = note: see issue #91611 <https://github.com/rust-lang/rust/issues/91611> for more information

For more information about this error, try `rustc --explain E0562`.
error: could not compile `sdml-core` due to previous error

Seemingly (according to some chatter), that syntax is supported only since rustc 1.75.
I am currently working on packaging sdml officially for Debian.
If possible, please consider rewriting that code to support rustc 1.70, as that will help getting sdml into Debian now, rather than needing to wait until Debian succeeds in upgrading its included rustc compiler.
Alternatively, if the current code is considered more beneficial (e.g. for performance reasons), then it would be helpful to me if you could suggest a patch that I could apply locally for the Debian package - I have tried and failed to wrap my head around the use of #[async_trait] which is mentioned at some places as a workaround.

no longer needed (for Debian, at least).

Sorry I didn't get to that.


Remove type generalizations from snark work

Snark work is defined at a completely polymorphic level of type generalization. This was to be able to talk about the type without requiring any functor dependencies. Now that much of the code is being defunctored, this work should be undone, and the types of snark work should not be polymorphic at all.

Pointer to code?

will do this as part of the refactor for verifying snarks when adding them to the snark pool

I actually only removed one of the type parameters. It was (hopefully at least) the hardest one to do, but there is still some work left.

Oh, sorry, I thought it was fully completed. My bad.


[WIP] Seen View

Opening a WIP pull request for the Seen View.
It's not quite ready to merge it in yet since it only displays the title of a seen entry and whether or not it was local to a task.

I recommend pulling latest dev as I made some changes to seen api, mainly performance ones. Not sure if something I did should break anything though

Also, you can push this into webui branch yourself i think... that's what it was created for, so you wouldn't have to create those PRs i think.


bug on model splicing + Flatten

Repro Colab: https://colab.research.google.com/drive/18Fna4m_WDGr0lVjFeGlqOUX5Id2a_ljb
Repro step: uncomment line #y = keras.layers.Flatten(y) then run the Colab
Error message:
ValueError: The data_format argument must be one of {'channels_first', 'channels_last'}. Received: data_format=<kerastensor shape=(none, none, none, 12), dtype=float32, sparse=false, name=keras_tensor_43>

I have replicated the issue with inclusion of line y = keras.layers.Flatten(y). Attaching gist for reference.
Need to look into the issue.

Hi @martin-gorner ,
We missed this part. We need to call the flatten layer like this y = keras.layers.Flatten()(y). What we have done actually we are passing the output tensor from Conv2d layer to the Flatten layer constructor and hence the error. Please refer modified gist.
Thanks!


Provide way to disable auto save for .Batch Rename.txt

It's just plain text so I could not add per-language settings like below:
  "[batch-renamer]": {
    "files.autoSave": "off"
  },

Upvote for this feature.

Maybe, instead of using onDidSaveTextDocument here, you could use onDidCloseTextDocument, so that only when the document is closed, the rename takes action.
Currently I have to do a quick edit, or edit the content in a new file, and then copy the changes back to the rename document. Otherwise it closes on its own and apply whatever changes I managed to cramp within a second.
I haven't tried this suggestion, let me know if I can help with it


extract activity schema

@apdavison we need your help here. Should the Activity hold all the properties that are currently in there or should it only be "input" and "output" ??

looks good in general, a couple of minor things, and a discussion needed about what to put in input/output

@apdavison just to be sure: so besides input, output, description and lookup label, activity should have also the properties: study target and semantically anchored to, as well as parameter set?

looks good in general, a couple of minor things, and a discussion needed about what to put in input/output

@apdavison just to be sure: so besides input, output, description and lookup label, activity should have also the properties: study target and semantically anchored to, as well as parameter set?

yes, I think so.

What do you think about adding fields "startedAtTime", "endedAtTime"? We add these in Computation, and for the ephys extension we would like to add them to subclasses of ProtocolExecution, so maybe having them in Activity, as the base class of all these, is the best place.

What do you think about adding fields "startedAtTime", "endedAtTime"? We add these in Computation, and for the ephys extension we would like to add them to subclasses of ProtocolExecution, so maybe having them in Activity, as the base class of all these, is the best place.

@apdavison I don't have any objections having them in the activity base class. They should remain optional though, because in many cases these metadata will not be available for a protocol execution.

@apdavison just to be sure: we said now we'll remove the semantically anchored to from the ProtocolExecution / Activity schema correct? And reintroduce it in the TissueSample(Collection) schemas? And in the files?? I think this we need to discuss again (where to reintroduce a user assessed anatomical location of their data).
Summary of necessary corrections:
Activity input / output should be reduced to File / FileBundle.
Activity semanticallyAnchoredTo should be removed.
Activity should get startTime / endTime properties (with time and datetime as formats).

@apdavison I'm merging this PR now, but it will cause some changes to be made in openMINDS_computation. I will go through them and either make a PR there or raise an issue as soon as I can.


Horizontal ScrollView Resets Scroll Position (Android, RTL)

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

This was reproduced locally with 55 & 56 and on expo (53)

Environment
  React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.3
      CPU: x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
      Memory: 43.51 MB / 16.00 GB
      Shell: 4.4.12 - /usr/local/bin/bash
    Binaries:
      Node: 9.3.0 - /usr/local/bin/node
      Yarn: 1.3.2 - /usr/local/bin/yarn
      npm: 5.6.0 - /usr/local/bin/npm
      Watchman: 4.7.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 11.2, macOS 10.13, tvOS 11.2, watchOS 4.2
      Android SDK:
        Build Tools: 23.0.1, 23.0.3, 25.0.0, 25.0.2, 25.0.3, 26.0.0, 26.0.1, 26.0.2, 27.0.3
        API Levels: 19, 21, 22, 23, 24, 25, 26, 27
    IDEs:
      Android Studio: 2.3 AI-162.4069837
      Xcode: 9.2/9C40b - /usr/bin/xcodebuild
    npmPackages:
      react: 16.3.1 => 16.3.1
      react-native: 0.56.0-rc.4 => 0.56.0-rc.4
    npmGlobalPackages:
      react-native-cli: 2.0.1

Description
Horizontal ScrollViews reset their scroll position when their content size changes on Android in RTL.
That description is a mouthful so here is a gif showing the problem:

Reproducible Demo
Here is a snack from expo made by @megantracy93 https://snack.expo.io/@meganltracy/scrollview-scrolltoend-rtl-issue
Steps to Reproduce:

Create a UI with a horizontal scroll view
Introduce an event handler that adds content to the scroll view (eg a button that adds views to the scroll view.
Change the device locale to RTL (Arabic) OR force rtl on the ui:

 componentWillMount() {
    I18nManager.forceRTL(true);
  }

Scroll the horizontal scroll view to its end (left)
Trigger the event handler causing the contents of the scroll view to grow

Observed

The horizontal scroll position is set to the start (right side) of the view.

Expected

The horizontal scroll position should be maintained

Additional Info
From some debugging, we think ReactHorizontalScrollContainerView is likely the class which is responsible for positioning the horizontal ScrollView onLayout. That method gets called whenever the size of the contents change and resets the scroll position of the view. Instead, it should be setting it back to where it was before.

Same problem with a ScrollView that has inverted property set.

Hello there 👋
I tried this on 0.59.x with the code you provided and it seems to be fixed. But please, if it's actually still an issue with 0.59 please comment below and we can reopen it 😊

I have the same problem
in horizontal scroll view content  sticks to the left side even when set contentContainerStyle={{ flexDirection: "row-reverse" }}

#23233

No I did not fix that!


Why is every version 2.x.x release marked as "beta"?

The format of the 2.x.x tags seem to indicate they're stable versions. And perhaps, seeing as the beta is several months old now (and working fine in my experience), that is indeed the case now.
If so, is it time to create a v2 tag? This would allow folks to use @v2, reducing noise from dependabot and similar services.

@staabm, I have raised a ticket to GitHub Support with a request to configure Dependabot in such a way that it will ignore pre-releases of actions, and here is their reply:

There is an open Dependabot Issue to address this functionality, and I've gone ahead and added your case as a +1 to it. We aren't able to say when this bug will be fixed, but you can keep an eye on the GitHub blog for information on when and if this makes it into the product.

Hello everyone!
We have recently released stable actions/setup-node@v2 that points to v2.1.4 that is marked as stable too.
I am closing this issue but please let me know if you have any concerns.


don't use unreferenced var in debug message

fixes #SLACK-1J
https://sentry.io/organizations/cobot/issues/762487561/

@DeraLuce you can't use it in the else statement, because the else statement is only reached when user is nil.

@DeraLuce you can't use it in the else statement, because the else statement is only reached when user is nil.

Ah makes sense!


Fix scrollbar's position and pointers-event

fix #16

Thanks @oldergod for the awesome work.
There’s only one issue (that I ran into as well while trying to whip up a quick fix) when the browser window gets small:

Do you have an idea on how to fix this?

@surma I cannot repro this. I tried on Chrome/Firefox(Ubuntu), Safari/Chrome(iPhone6S), Chrome(Nexus5)
Here is what I see in Chrome/Firefox on Ubuntu:

Am I missing something?
Since I see a ruler, I guess you are using Chrome? on Mac? I'll be able to try this on Mac at the end of the week.

I could reproduce what you saw this way:

toggle device mode
set device type to Desktop
refresh and scroll a bit till header expands
set device type to Mobile

But the werid thing is : if I start with a device type set to Mobile, I cannot reproduce it. I tested on iPhone6s and Nexus5 without problem so I would rather think it is a chrome thing?

Alright, if it’s a chrome-thing only, then feel free to ignore it.
Is this ready to merge in your opinion?

IMO it is, yes. If ever a bug were to show up, I would promptly fix it anyway!

@oldergod Forgive me for how long it took me to get back to you! I/O and the subsequent jetlag knocked me out ;)
Finally gave this the last test run. LGTM. Merged. Awesome 🎉


fix to only update rule table if rule proc is deployed

to: @chunyong-lin
cc: @airbnb/streamalert-maintainers
size: small
Background
The rule table should only really be updated if the rule processor is being deployed (not, say, if only the alert processor is deployed).
Changes

Ensuring rule processor is being deployed before updating rules tables.

Coverage remained the same at 96.508% when pulling<PHONE_NUMBER>264e3d4ae6790df5151afd2bc72b5e on ryandeivert-rule-table-update-on-deploy into ffea623f3477361285971a1c9ff83c6c18e06ac8 on master.

that gif is incredible


Is it possible to detect if any subscriber received my message?

After publishing a message i wanted to know if any subscriber received it, is it possible?

@ericholiveira the only way for you to know if someone got it would be to send the message as a request/reply.
If you needed to know that at least one client got the message, you would:
// Request with Auto-Unsubscribe. Will unsubscribe after the first response is received via {'max':1}
nats.request('help', null, {'max':1}, function(response) {
  console.log('Got a response for help: ' + response);
});

The clients for the request are normal subscribers. They simply use the replyTo inbox subject to send the response:
// Replies
nats.subscribe('help', function(request, replyTo) {
  nats.publish(replyTo, 'I can help!');
});

Hope it helps.


PODS-8821 - Add Event 24

Is this code change required for the ticket? What impact does it have on frontend?

It does seem to be needed, if you run the FE locally without this change then you'll get a 500 error after the Member's details page where it cannot find Event 24

Is this code change required for the ticket? What impact does it have on frontend?

It does seem to be needed, if you run the FE locally without this change then you'll get a 500 error after the Member's details page where it cannot find Event 24

Sounds good.


WIP: Tests

Description
Tests #9
Motivation and Context
We'll be confident :D
Component list
Feel free to add a component to the list.

[x] Avatar
[x] AvatarGroup
[ ] Box
[ ] PseudoBox
[x] Button
[x] ButtonGroup
[x] CloseButton
[x] Divider
[x] Icon
[ ] IconButton
[ ] Text
[x] Alert
[x] Badge
[ ] Toast
[x] Flex
[x] Heading
[ ] ControlBox
[ ] Stack
[ ] Modal
[ ] Drawer
[ ] AlertDialog
[ ] PseudoBox
[x] BreadCrumb
[ ] Tooltip
[x] CircularProgress
[x] Code
[x] Grid
[ ] Progress
[ ] Menu
[ ] Popover
[x] Collapse
[x] Accordion
[ ] AspectRatio
[ ] Image
[ ] Input
[x] FormControl
[ ] Radio
[ ] RadioGroup
[ ] Tag
[x] Switch
[ ] Slider
[x] Checkbox
[ ] CheckboxGroup
[x] Fragment

For the simple components like Flex or Box etc. we could test the css like using jest-emotion serializer. But it'll add so much css style outputs to all the snapshots...
Hmm how should we go about it? 🤔 cc @codebender828


Issues using the program "recode" (character encoding conversion utility) on Windows filesystems

Version
Microsoft Windows [Version 10.0.19043.1586]
WSL Version

[ ] WSL 2
[X] WSL 1

Kernel Version
4.4.0-19041-Microsoft
Distro Version
Ubuntu 20.04.4 LTS
Other Software
recode 3.7.12
https://github.com/rrthomas/recode/
Repro Steps
recode is a utility for converting text files to different encodings
when run from WSL on a file stores on a Windows-type filesystem (NTFS / FAT32 / exFAT) some serious issues are encountered
I filed this bug yesterday on the recode repo so ref it for more info:
https://github.com/rrthomas/recode/issues/39
I was initially using recode 3.6, the version packaged by Ubuntu, and it broke with exFAT and NTFS but worked properly with FAT32
I then tried current version 3.7, which broke on exFAT and FAT32 but worked properly on NTFS
Replication is easy, just run recode on a file stored on a mounted Windows filesystem such as
recode utf8..utf16 /mnt/g/test.txt

A temporary file will be created with the desired conversion done properly, and the original file will be deleted, however, it will somehow fail to rename the temporary file back to the original filename
Expected Behavior
Program should create a temporary file with the requested conversion done and then rename the temporary file back to the original filename
Actual Behavior
Original file is deleted but the temporary file is not renamed back to the original filename
The data technically isn't lost but all affected files will end up with meaningless *.tmp filenames, and if you didn't keep track of the original filenames getting them renamed back properly can be difficult or impossible, especially if you converted a lot of files and can't easily identify which .tmp file correlates with which original file (the error message in 3.7 helps with this somewhat assuming you still have access to the output)
Diagnostic Logs
:/mnt/d$ recode --version
recode 3.7.12
Written by François Pinard<EMAIL_ADDRESS>
Copyright (C) 1990-2022 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
:/mnt/d$ echo HELLO > testing.txt
:/mnt/d$ file testing.txt
testing.txt: ASCII text
:/mnt/d$ uchardet testing.txt
ASCII
:/mnt/d$ recode utf8..utf16 testing.txt
/home/username/bin/.libs/recode: rename (/mnt/d/recode-ALN8Z9.tmp, /mnt/d/testing.txt): No such file or directory
:/mnt/d$ ls -l testing.txt
ls: cannot access 'testing.txt': No such file or directory
:/mnt/d$ ls -l *.tmp
-rwxrwxrwx 1 root root    60 Mar 28 09:02 rec10721.tmp
-rwxrwxrwx 1 root root 47307 Mar 28 01:27 rec5097.tmp
-rwxrwxrwx 1 root root    14 Mar 28 09:52 recode-ALN8Z9.tmp
:/mnt/d$ file recode-ALN8Z9.tmp
recode-ALN8Z9.tmp: Big-endian UTF-16 Unicode text
:/mnt/d$ uchardet recode-ALN8Z9.tmp
UTF-16
:/mnt/d$

This apparently only happens under WSL, not when accessing a FAT32 filesystem under a native Linux OS.


Remove obsolete version attribute from docker-compose.yml

Remove obsolete version attribute from docker-compose.yml

Closing as a duplicate of #13


Update hygiene filter to fix CI build

Fix #26799

Just found that @joaomoreno have pushed a fix for this to master branch.


FIX: render multiline legend items without overlapping (#10532)

Co-authored-by: Nirav Chavda<EMAIL_ADDRESS>Fixes: this issue
Explanation
Previously all the labels were considered to have a constant height, we changed it so that it calculates the height in case of multi-line labels
Screenshot

Input for the above screenshot: -
var myChart = new Chart(ctx, {
  type: 'doughnut',
  data: {
	labels: [
	      'Some Text',
	      [
	        'ABCDEFG',
	        'ABCDEFG',
	      ],
	      [
	        'Hello World',
	        'Lorem Ipsum',
	        'Multi Line Text',
	      ],
	      'Some Text',
	      'Some Text',
	      'Some Text',
	],
	datasets: [
	      {
	        label: 'test',
	        data: [
		  73.42,
		  18.13,
		  7.54,
		  0.9,
		  0.0025,
		  1.8e-5,
	        ],
	        backgroundColor: [
		  '#0078C2',
		  '#56CAF5',
		  '#B1E3F9',
		  '#FBBC8D',
		  '#F6A3BE',
		  '#4EC2C1',
	        ],
	      },
	],
  },
  options: {
	plugins: {
	      legend: {
	        labels: {
		  usePointStyle: true,
		  pointStyle: 'rect',
	        },
	        position: 'right',
	        align: 'center',
	        maxWidth: 860,
	      },
	},
	aspectRatio: 3,
  },
});

Looks like the size limit needs to be increased slightly to accommodate this change. @kartik-madhak can you up that and then we'll merge this?
It's this file https://github.com/chartjs/Chart.js/blob/master/.size-limit.cjs and the failing GH action logs https://github.com/chartjs/Chart.js/blob/master/.size-limit.cjs#L37 should be updated. I think a good new size is "27.5KB"


ssb-patchwork crashes when trying to change profile picture

Describe the bug
When clicking on "Profile > Edit your profile > Choose profile image", ssb-patchwork crashes.
To Reproduce
Steps to reproduce the behavior:

nix-env -iA nixpkgs.ssb-patchwork
Profile > Edit your profile > Choose profile image
:boom:

Expected behavior
A file browser window shows up
Screenshots
x.
Additional context

I'm running i3.
ssb-patchwork 3.17.1
This is not a regression, it has been broken for a wihle

Metadata

system: "x86_64-linux"
host os: Linux 4.19.87, NixOS, 19.09.1484.84586a4514d (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3
channels(root): "nixos-19.09.1484.84586a4514d, nixos-hardware"
channels(asymmetric): "home-manager-19.09"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

See https://github.com/NixOS/nixpkgs/issues/76743

@mkf, let's keep the conversation in a single issue.

@NinjaTrappeur The way I obtained the error message was by launching it from a terminal. It clearly doesn't do that for me. I use awesomewm and xterm.

:( That's to be expected, this workaround probably depends on some impurities coming from my host system.
Maybe https://github.com/NixOS/nixpkgs/pull/83410 could fix your issue?

Maybe #83410 could fix your issue?

resolving this diff with today's master requires quite some time and understanding from me, i'm afraid i will be forced to just leave it for now and stay without an avatar due to being in trouble with time recently

You're mostly interested by these 3 lines: https://github.com/NixOS/nixpkgs/pull/83410/files#diff-2505cd8ade7d2013fb5d195c710b8ccb8ea6db1fc7a63855830919165e9dfc46R38-R41

.


Inbound Vs Outbound Explanation in Context of Firewall

Hello Microsoft,
I infer that the article here would benefit from clarifying INBOUND vs OUTBOUND in context of Firewall.
https://learn.microsoft.com/en-us/azure/firewall/policy-rule-sets#rules

To put it simply, an inbound firewall rule protects your network from threats that originate from outside your network(traffic sourced from Internet) and attempt to infiltrate your network inwardly.
An outbound firewall rule protects against nefarious traffic that originates internally(traffic sourced from private IP within Azure) and travels outwardly. This is usually traffic from within Azure resources being redirected via Firewall before reaching a destination.
Based on above understanding,rules are applied as described here:
Inbound processing
Outbound processing
2.I also wish to ask if there is any reason for INBOUND vs OUTBOUND segregation not being done on Azure portal in context of firewall(done for NSGs though)?
I mean, DNAT and Network Rule Collection would fall under INBOUND category.
Network and Application Rule Collection would fall under OUTBOUND category.
Regards,
Aditya Garg
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 08324cd3-7f61-81f6-a7e8-3f2111304d20
Version Independent ID: 348ef905-d8d2-3dc0-08d9-961b1ba5f7ed
Content: Azure Firewall policy rule sets
Content Source: articles/firewall/policy-rule-sets.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

@adi-garg
Thanks for your feedback! We will investigate and update as appropriate.

@adi-garg ,
With respect to Azure Firewall, we do not segregate the rules as Inbound and Outbound at platform level.
This could be mainly because Network Rules fall under both of these.
This is how the platform is designed.
The InBound and Outbound Rules documentations are a subset of Firewall documentations, and I can see it's listed under Next Steps of the main document.
So users should be able to navigate easily.
Please let me know if you need more details on this.

Hello Kapil, Thanks for the response.
Kind of disagree here.
-The subsections INBOUND and OUTBOUND call out a different processing logic. This implies there is a differentiation/one must look at direction before trying to apply firewall rules.
Here is my broader understanding,open to feedback:

-Network rules are processed under both INBOUND and OUTBOUND, BUT Application Rules are processed for OUTBOUND traffic only and DNAT rules are processed for INBOUND traffic only.
-Further, given that the article talks about INBOUND and OUTBOUND, Azure readers would benefit getting clarity on what INBOUND and OUTBOUND means as inferred in my diagram(earlier post).
Regards,
Aditya Garg

Hi @adi-garg
Sure. Thanks for the detailed follow-up.
I shall assign the case to the author @vhorne for further review on this.

added new Inbound vs. Outbound section to article.
To suggest Azure feature changes, go to https://developercommunity.visualstudio.com/AzureDevOps
#please-close


  Problem 1
    - laravel/envoy v1.0.27 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.26 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.25 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.24 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.23 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.22 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.21 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.20 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.19 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.18 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.17 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.16 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.15 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.14 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.13 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.12 requires symfony/console ~2.3 -> no matching package found.
    - laravel/envoy v1.0.11 requires symfony/console 2.4.* -> no matching package found.
    - laravel/envoy 1.0.5 requires symfony/console 2.4.* -> no matching package found.
    - laravel/envoy 1.0.3 requires symfony/console 2.4.* -> no matching package found.
    - laravel/envoy 1.0.2 requires symfony/console 2.4.* -> no matching package found.
    - laravel/envoy 1.0.0 requires symfony/console 2.4.* -> no matching package found.
    - Installation request for laravel/envoy ^1.0 -> satisfiable by laravel/envoy[1.0.0, 1.0.2, 1.0.3, 1.0.5, v1.0.11, v1.0.12, v1.0.13, v1.0.14, v1.0.15, v1.0.16, v1.0.17, v1.0.18, v1.0.19, v1.0.20, v1.0.21, v1.0.22, v1.0.23, v1.0.24, v1.0.25, v1.0.26, v1.0.27].

Potential causes:
 - A typo in the package name
 - The package is not available in a stable-enough version according to your minimum-stability setting
   see <https://getcomposer.org/doc/04-schema.md#minimum-stability> for more details.

Read <https://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.

Installation failed, reverting ./composer.json to its original content.

Looks like it's conflicting with another globally installed package.


Plugin crashes on hover

Repo with example
Hover over Map.add to crash the server with
>> Fatal error: Longident.flat
Fatal error: exception Misc.Fatal_error

Seems this issue will be resolved once https://github.com/rescript-lang/syntax/pull/449 is merged.

@Lomand thanks for the report!
This has indeed the same cause as https://github.com/rescript-lang/syntax/pull/449, however it crashes in the extension's codebase, rather than while calling the formatter.

Thanks a lot for the fix, Cristiano!
Really glad to hear that this issue could be resolved without relying on external repos. 🎉


Apple Clone Website

Description of  the change
I'd like to change the Apple clone website .
Domain of Docs

[] Readme
[] Solidity , Html, Css , JavaScript , Web3.js / Ether.js
[] Solidity , ReactJs /NextJs , Web3.js /Ether.js
[] Solidity , Python3 , Web3.py
[] Other , feel free to mention below

Are you contributing under any open-source program ?
YES under SSOC'23

@Vikash-8090-Yadav Could you assign this to me?

@Saloni020 please specify the details of the changes in this project.


Line does not compile

line 63
https://github.com/mpilquist/udp-replay/blob/main/udp-replay.scala#L63
.through(CaptureFile.udpDatagrams.toPipeByte)
It says .udpDatagrams does not exist on this class.
If I am using
val fs2Io = "co.fs2" % "fs2-io_3" % "3.2.11"
val fs2Protocols = "co.fs2" % "fs2-protocols_3" % "3.2.11"

Can you tell me how to fix?

The current code depends on an unreleased snapshot of fs2 which includes this PR: https://github.com/typelevel/fs2/pull/2954. This change was made in this commit: https://github.com/mpilquist/udp-replay/commit/e2b6694ffd1e60dbb78a7596570fde44e13c1add
You could either checkout that branch and publish fs2 locally or you could revert back a few commits on this repo -- something like git checkout e2b6694ffd1e60dbb78a7596570fde44e13c1add~1.
Let me know how it goes. I'm likely going to write up this example app in a blog post.


chore(yarn): add yarn support

Right now, I'm opening this against the upgrade-deps PR. Before we merge this, let's get that other one merged, then we'll merge this into master.
Also, I think something may be broken, so I'll have to come back to this if the build breaks.

Any reason why you didn't move to yarn? Is it not stable enough?

Yarn looks interesting, I'm hoping to try it out soon.
On 9 November 2016 at 14:12, Tommy<EMAIL_ADDRESS>wrote:

Any reason why you didn't move to yarn? Is it not stable enough?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kentcdodds/p-s/pull/72#issuecomment-259423411, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB4Pj9WzQyVPYhgobAH0RjwTEPKyZkjZks5q8dTQgaJpZM4KWeLQ
.

Yeah, I'm still planning on doing this eventually, but open source modules should not commit the yarn.lock file (to make sure they're always tested with the latest versions of modules which users will be downloading when they install your package). So I was just cleaning things up :)