Legit dies on 'sync' command

Got this traceback while running 'legit sync' -- rather disconcerting... :-/
Pulling commits from the server.
Traceback (most recent call last):
  File "/usr/local/bin/legit", line 8, in <module>
    load_entry_point('legit==0.0.9', 'console_scripts', 'legit')()
  File "/Library/Python/2.6/site-packages/legit/cli.py", line 34, in main
    cmd_map.get(arg).__call__(args)
  File "/Library/Python/2.6/site-packages/legit/cli.py", line 131, in cmd_sync
    status_log(smart_pull, 'Pulling commits from the server.')
  File "/Library/Python/2.6/site-packages/legit/cli.py", line 58, in status_log
    log = func(*args, **kwargs)
  File "/Library/Python/2.6/site-packages/legit/scm.py", line 83, in smart_pull
    return repo.git.execute(['git', verb, '{0}/{1}'.format(remote, branch)])
  File "/Library/Python/2.6/site-packages/git/cmd.py", line 377, in execute
    raise GitCommandError(command, status, stderr_value)
git.exc.GitCommandError: 'git merge origin/develop' returned exit status 1:
Exception TypeError: "'NoneType' object is not callable" in <bound method WindowCursor.__del__ of <smmap.mman.WindowCursor object at 0x1017c9f18>> ignored
Exception TypeError: "'NoneType' object is not callable" in <bound method WindowCursor.__del__ of <smmap.mman.WindowCursor object at 0x1007c4890>> ignored```

so this can be closed now, thanks to https://github.com/Byron/smmap/commit/24a35052108e7cd15a040348d151328a2d528921 ?

\o/


log to memory in integration tests

This alleviates the problem that flaky tests are often not reproducible when running the integration tests with logging enabled.
100 MB is quite a large buffer for a log file, but it's only run once (the buffer is reused between tests).

Codecov Report

Merging #1843 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1843      +/-   ##
==========================================
- Coverage   87.14%   87.11%   -0.03%
==========================================
  Files          95       95
  Lines        6507     6507
==========================================
- Hits         5670     5668       -2
- Misses        591      593       +2
  Partials      246      246

Impacted Files
Coverage Δ

session.go
76.24% <0%> (-0.27%)
: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 358fce2...9f2bfe9. Read the comment docs.


Harrison/openai functions

only two small things

not sure i see the 'force function_call' argument
https://github.com/jxnl/openai_function_call/blob/main/example_segmentation.py#L61

doing function_call={"name": ...} on Completion.create
the api to force the usage, maybe abstracted away and already there, not sure

In the docs its worth calling out that the docstring will be part of the schema 'prompt' and cna contain additional rules and information.

https://github.com/jxnl/openai_function_call/blob/main/example_segmentation.py#L36-L42https://github.com/jxnl/openai_function_call/blob/main/example_segmentation.py#L36-L42

I don't see any validation after the json is returned when pydantic is being used (does it go back to return json.loads)? see:
https://github.com/jxnl/openai_function_call/blob/main/openai_function_call.py#L63


Run tests on GitHub Actions

Disable one failing test to make this work.

You can see this passing in https://github.com/foolip/reffy/commit/c8453b7994d6632e0af15fc08b0532a03ba7fd4c/checks?check_suite_id=383068399.

It worked! https://github.com/tidoust/reffy/runs/377140873


Filter page using fields of other tables

Is there any option to filter a page list using a field or data from another table? For example, can I do something to get items with a specific work center in the routing?

Hello,
You cannot do that in the client.
By developping an extension, you can do this with queries (and then expose query result in a page) or by overloading the OnFindRecord and OnNextRecord triggers on page. Please note this last capability can lead to very bad performances.

Hello,
You cannot do that in the client.
By developping an extension, you can do this with queries (and then expose query result in a page) or by overloading the OnFindRecord and OnNextRecord triggers on page. Please note this last capability can lead to very bad performances.

Thanks, I will do a query to get it if there is no option to do this in filters.

Closing as this issue seems to have been answered. Please note that this repository no longer handles general questions and is only dedicated to bug reports for the latest preview version of the AL compiler and developer tools.
To get help for your question please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group


An error occured after running command "roslaunch object_tracking test.launch"

Hello Mr.Huang,
thanks for your codes with explain. I have met a problem when I try to run the command "roslaunch object_tracking test.launch", which looks like this:

@Huleyun 请问这个问题你解决了吗？我也出现了这样的问题。


Updated Mulitstage Image

Hello @johanbrandhorst,
I am working on a leaner update of your generator image and would be happy, if you could test it - and if you are happy, merge the changes into your repo :)

this is an updated version of jfbrandhorst/grpc-web-generators : https://github.com/johanbrandhorst/grpc-web-generators
it is a multistage build, reducing to 1/5 (!)
node:18.16.1-bullseye-slim as base image
ts-protoc-gen has been replaced by grpc-protoc-gen
bullesye / bullseye-slim are using python3.9

I use the github container registry, so you can directly try out the image:
https://github.com/LARAsuite/grpc-web-generators/pkgs/container/grpc_generators

Hi, thanks for this, it looks like you spent a lot of time getting this updated. Unfortunately, I'm not sure merging this is the right thing to do. I created this project before there were better alternatives, but nowadays almost all the problems this project set out to solve are solved better by Buf's tooling. I would recommend against using docker images for protobuf generation when you can simply use remote plugins on the Buf Schema registry: https://buf.build/plugins. I hope you understand.

Thanks a lot @johanbrandhorst,
for the hint to the buf - workflows. This I was not aware of. Nonetheless, for a local compilations, protoc compilers in a docker would still have some use-cases (that's why I created the docker in the first place).
If you are planning to keep this repo, I would at least merge the updated multistage docker image into some branch for people looking for a local solution and do not want to undergo the protoc installation hazzle (which can be quite painful, if you want to have multiple compilers on your system) and write a hint into the README.


feat(extras): add GNOME terminal

This MR adds Tokyo Night themes for the GNOME terminal. An instruction on how to easily import the generated themes is included in the files itself.
I tested creating and importing every theme on my machine, which worked fine. Let me know if there's anything I can improve.

ty!


Fix crash when using formats with bits per pixel size not divisible by 8

Two separate fixes actually:

get_bytes_per_frame assumed that a bits per pixel size that is not divisible by 8 means that it can't retrieve the frame size because that will yield a frame buffer with a non full-byte size. This is not entirely true since for instance YUV formats typically have size restrictions such that their raw frame buffers don't run into this problem. ➡ The check got changed to the frame buffer's size
don't panic when get_bytes_per_frame can't return a size and instead report the error and exit

The former fixes using yuv formats for me, the later is a bit of a general requirement for me since I deal with arbitrary user data: not being able to handle data is sad, but crashing over it goes too far :)

if the bytes per frame can't be calculated, it allocates a zero-size buffer and then the iterator constantly produces OutputFrame events with zero size and never finishes

oops. I didn't notice that I only exit from the lambda that's passed to map and not the entire thread - I still wanted to handle this as a fatal error as before, just more gracefully.
Thank you so much for following up on this!


[DatePicker] Improve runtime

Opening the date picker on desktop and mobile have a perceptible delay, it's even more noticeable with the range picker. As a UI library, we have a 100ms delay budget before the users start to notice.

0 to 100ms | Respond to user actions within this time window and users feel like the result is immediate. Any longer and the connection between action and reaction is broken.

https://developers.google.com/web/fundamentals/performance/rail

0.1 second is about the limit for having the user feel that the system is reacting instantaneously, meaning that no special feedback is necessary except to display the result.

https://www.nngroup.com/articles/response-times-3-important-limits/

For instance, using the desktop version as a baseline: https://next.material-ui-pickers.dev/demo/daterangepicker#responsiveness. On my high-end $3,000 MacBook, it takes about 190ms to get visual feedback. It's x2 too much, it's frustrating/feels wrong:

To be compared with:

the 180ms of Kendo-UI
the 140ms of react-dates
the 80ms of react-datepicker
the 40ms of react-day-picker

I think that anything below 80ms would be great 🔥(react-datepicker, relative to my machine). For comparison, in the Autocomplete, it takes 70ms to open the popup with 100 items. 40ms is mind-blowing 🤯 (react-day-picker, relative to my machine).

Regarding delivering on better runtime performance

The first thing I would look at is the usage of the ButtonBase for the days, I suspect an easy win by replacing them with a simple <button>, it seems to be an obvious win, we don't need the ripple there.

For instance, in dev mode, we get from 400ms to 250ms with this diff:
diff --git a/lib/src/DateRangePicker/DateRangePickerDay.tsx b/lib/src/DateRangePicker/DateRangePickerDay.tsx
index 9f54cb31..05ea0182 100644
--- a/lib/src/DateRangePicker/DateRangePickerDay.tsx
+++ b/lib/src/DateRangePicker/DateRangePickerDay.tsx
@@ -135,13 +135,7 @@ export const PureDateRangeDay = ({
           [classes.rangeIntervalDayPreviewStart]: isStartOfPreviewing || isStartOfMonth,
         })}
       >
-        <Day
-          allowKeyboardControl={false}
-          disableMargin
-          {...other}
-          day={day}
-          selected={selected}
-          inCurrentMonth={inCurrentMonth}
+        <button
           data-mui-test="DateRangeDay"
           className={clsx(
             classes.day,
@@ -152,7 +146,9 @@ export const PureDateRangeDay = ({
             },
             className
           )}
-        />
+        >
+          1
+        </button>
       </div>
     </div>
   );

The second thing I would look at is how "loaded" the DOM is. For instance, react-day-picker has a single DOM node per cell vs x for us. replacing div > div > button with button we reduce the runtime from 250ms to 220ms.

I believe we could do the same with the year picker. div > h6 seems wasteful (as well as breaking the accessibility rule of headers continuity. It should be div > div at the very least, and ideally only div.

Removing the Grow animation gets us down from 220ms to 130ms, still in dev mode. Something seems wrong with the implementation, it double renders. In this condition, I think there we are better off without. We would need to investigate, on what's wrong. We might only need a memo on the component just below Grow. Hopefully, we will be able to make the other components benefit from it.
I haven't noticed another opportunity but it's probably not necessary to look deeper. from 400ms to 130ms in dev mode, if we extrapolate to the 190ms baseline, it means 62ms in production. This is perfect.

Environment

Tech
Version

@material-ui/lab
v5.0.0-alpha.16

I haven't noticed another opportunity but it's probably not necessary to look deeper. from 400ms to 130ms in dev mode, if we extrapolate to the 190ms baseline, it means 62ms in production.

Just in case others are reading this: Do not extrapolate from dev to prod. And in general: Do not meassure in development.

On my high-end $3,000 MacBook,

These are not expensive because of their performance.
Current results for opening a picker by keypress:
Datepicker: 50ms
Timepicker: 30ms
DateRangePicker: 70ms
We can definitely improve the performance and there are some "easy" wins within reach. But let's be accurate in our measurements so that we can make lasting changes. Not measuring did lead to some unnecessary work.

I haven't noticed another opportunity but it's probably not necessary to look deeper. from 400ms to 130ms in dev mode, if we extrapolate to the 190ms baseline, it means 62ms in production.

Just in case others are reading this: Do not extrapolate from dev to prod. And in general: Do not meassure in development.

On my high-end $3,000 MacBook,

These are not expensive because of their performance.
Current results for opening a picker by keypress:
Datepicker: 50ms
Timepicker: 30ms
DateRangePicker: 70ms
We can definitely improve the performance and there are some "easy" wins within reach. But let's be accurate in our measurements so that we can make lasting changes. Not measuring did lead to some unnecessary work.


"Add a cosignatory" button missing on Multisig page after toggling from co-signer to not co-signer via upper menu

Steps to reproduce:

Select a co-signer account using upper menu or accounts page
Go to Multisig page
Toggle to another account (not co-signer) via upper menu

Expected result: A user is able to click "Add a cosignatory" button to add a co-signer and convert an account to Multisig under not co-signer account
Actual result: There is no "Add a cosignatory" button on Multisig page after toggling from co-signer account to not co-signer account using upper menu
Video

Verified


[1.1] Lots of path requests at once causes lag and error.

A bunch of people tried to start paths at once and it caused a ton of lag, and the following error.
(It also caused DB locked errors which is a known issue)
[17:07:28 WARN]: [Journey] Scope from plugin Journey is using an invalid sub-scope id: ITZWBL_. Please notify the plugin owner.
[17:07:28 WARN]: [Journey] Scope from plugin Journey is using an invalid sub-scope id: The3ditor_. Please notify the plugin owner.
[17:07:28 WARN]: [Journey] Scope from plugin Journey is using an invalid sub-scope id: 4DrewsEh. Please notify the plugin owner.
[17:07:28 WARN]: [Journey] Scope from plugin Journey is using an invalid sub-scope id: 2xcl. Please notify the plugin owner.
[17:07:28 WARN]: [Journey] Scope from plugin Journey is using an invalid sub-scope id: ITZWBL_. Please notify the plugin owner.
[17:07:28 WARN]: [Journey] Scope from plugin Journey is using an invalid sub-scope id: The3ditor_. Please notify the plugin owner.
[17:07:28 WARN]: [Journey] Scope from plugin Journey is using an invalid sub-scope id: 4DrewsEh. Please notify the plugin owner.
[17:07:28 WARN]: [Journey] Scope from plugin Journey is using an invalid sub-scope id: 2xcl. Please notify the plugin owner.

Relevant Spark:
https://spark.lucko.me/q8f2EVVCGM

Some debug info
[17:32:13 INFO]: [Journey] [Destination Graph Goal Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: 4, y: 98, z: 5, domain: 0}, destination: {x: -1468, y: 63, z: -4281, domain: 1}, caller: (PLAYER) 00d477dc-6c86-473f-97c0-6434b2fba066, state: idle}: scheduling search
[17:32:13 INFO]: [Journey] [Itinerary Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: 4, y: 98, z: 5, domain: 0}, paths: 2, path searches: 0, state: idle, changed graph: false}: itinerary trial started
[17:32:13 INFO]: [Journey] [Itinerary Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: 4, y: 98, z: 5, domain: 0}, paths: 2, path searches: 0, state: running, changed graph: false}: itinerary trial needs to calculate 2 paths
[17:32:13 INFO]: [Journey] [Destination Path Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: 4, y: 98, z: 5, domain: 0}, destination: {x: 10, y: 100, z: -40, domain: 0}, cycles: 1, state: idle, visited: 0, from cache: false}: path trial beginning
[17:32:13 INFO]: [Journey] [Destination Path Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: -3993, y: 120, z: -3798, domain: 1}, destination: {x: -1468, y: 63, z: -4281, domain: 1}, cycles: 1, state: idle, visited: 0, from cache: false}: path trial beginning
[17:32:13 INFO]: [Journey] [Destination Path Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: 4, y: 98, z: 5, domain: 0}, destination: {x: 10, y: 100, z: -40, domain: 0}, cycles: 1, state: running, visited: 317, from cache: false}: succeeded
[17:32:14 INFO]: TheDeafCreeper issued server command: /tps
[17:32:14 INFO]: [Journey] [Chunk Cache] {131}: added: 131, removed: 0
[17:32:16 INFO]: [Journey] [Chunk Cache] {258}: added: 127, removed: 0
[17:32:18 INFO]: Artar05 lost connection: You have been kicked for idling more than 10 minutes.
[17:32:18 INFO]: [voicechat] Disconnecting client Artar05
[17:32:18 INFO]: [Journey] [Chunk Cache] {392}: added: 134, removed: 0
[17:32:20 INFO]: [Journey] [Chunk Cache] {537}: added: 145, removed: 0
[17:32:21 INFO]: [Journey] [Destination Path Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: -3993, y: 120, z: -3798, domain: 1}, destination: {x: -1468, y: 63, z: -4281, domain: 1}, cycles: 16, state: running, visited: 15299, from cache: false}: succeeded
[17:32:21 INFO]: [Journey] [Itinerary Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: 4, y: 98, z: 5, domain: 0}, paths: 2, path searches: 2, state: stopped (successful), changed graph: true}: itinerary trial succeeded
[17:32:21 INFO]: [Journey] [Destination Graph Goal Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: 4, y: 98, z: 5, domain: 0}, destination: {x: -1468, y: 63, z: -4281, domain: 1}, caller: (PLAYER) 00d477dc-6c86-473f-97c0-6434b2fba066, state: stopped (successful)}: search complete
[17:32:21 INFO]: [Journey] [Destination Path Search] {session: 632b623a-fb13-4bb1-abad-fe81b256bf72, origin: {x: -3993, y: 120, z: -3798, domain: 1}, destination: {x: -1468, y: 63, z: -4281, domain: 1}, cycles: 16, state: stopped (successful), visited: 15299, from cache: false}: caching path in database
[17:32:21 INFO]: [Journey] [Chunk Cache] {668}: added: 131, removed: 0
[17:32:23 INFO]: [Journey] [Chunk Cache] {638}: added: 0, removed: 30
[17:32:24 INFO]: [Journey] [Chunk Cache] {555}: added: 0, removed: 83
[17:32:25 INFO]: [Journey] [Chunk Cache] {503}: added: 0, removed: 52
[17:32:25 WARN]: Can't keep up! Is the server overloaded? Running 12796ms or 255 ticks behind
[17:32:26 INFO]: [Journey] [Chunk Cache] {425}: added: 0, removed: 78
[17:32:27 INFO]: [Journey] [Chunk Cache] {353}: added: 0, removed: 72
[17:32:28 INFO]: [Journey] [Chunk Cache] {232}: added: 0, removed: 121
[17:32:29 INFO]: [Not Secure] JustBins: I cant believe that worked
[17:32:30 INFO]: [Journey] [Chunk Cache] {131}: added: 0, removed: 101
[17:32:31 INFO]: [Journey] [Chunk Cache] {0}: added: 0, removed: 131

Fixed with #55


🐛 lint/correctness/noUndeclaredVariables doesn't recognize type Uppercase

Environment information
CLI:
  Version:                      1.0.0
  Color support:                true

Platform:
  CPU Architecture:             x86_64
  OS:                           windows

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         unset
  JS_RUNTIME_VERSION:           "v20.3.1"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "npm/9.6.7"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    false
  VCS disabled:                 true

Workspace:
  Open Documents:               0

Discovering running Biome servers...

Incompatible Biome Server: ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

i Rage discovered this running server using an incompatible version of Biome.

Server:
  Version:                      <=10.0.0

What happened?
// The Uppercase variable is undeclared lint/correctness/noUndeclaredVariables
export type AnyStringToUpper<T extends string> = Uppercase<T>;

playground
Expected result
No error
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

Uppercase is a TypeScript built-in type.
Strangely, the type is well declared as global. Testing some other globals, the bug seems also affect the next type URIErrorConstructor.


Dotnet SDK generation failing for Azure.ResourceManager.MobileNetwork

I am following the steps mentioned here https://eng.ms/docs/products/azure-developer-experience/develop/sdk-generate-mgmt-dotnet#generate-sdks-from-api-specs and “dotnet build /t:GenerateCode” throws below error:
fatal   | The ResourceCollection ExtendedUeInfoCollection (RequestPath:/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.MobileNetwork/packetCoreControlPlanes/{packetCoreControlPlaneName}/ues/{ueId}/extendedInformation/default) does not have a GetAll method  fatal   | Process() cancelled due to failure EXEC : error   | error : Plugin csharpgen reported failure. [C:\Repos\azure-sdk-for-net\sdk\mobilenetwork\Azure.ResourceManager.MobileNetwork\src\Azure.Res ourceManager.MobileNetwork.csproj]  error   | Autorest completed with an error. If you think the error message is unclear, or is a bug, please declare an issues at[https://github.com/Azure/](https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2F&data=05%7C02%7Ctarunkhattar%40microsoft.com%7C42329b84b1a947290bca08dc34ace724%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C638443164451774468%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=9XJHuXOsuDIbKayi4LPCwRWgAz9HslmSq7oa2LLuhNQ%3D&reserved=0)  autorest/issues with the error message you are seeing. C:\Users\tarunkhattar\.nuget\packages\microsoft.azure.autorest.csharp\3.0.0-beta.20240222.3\buildMultiTargeting\Microsoft.Azure.AutoRest.CSharp.targets(57, 5): error MSB3073: The command "npx autorest@ --max-memory-size=8192 --skip-csproj --skip-upgrade-check --version=3.9.7 C:\Repos\azure-sdk-for-net\sdk\mobi lenetwork\Azure.ResourceManager.MobileNetwork\src/autorest.md  --use=C:\Users\tarunkhattar\.nuget\packages\microsoft.azure.autorest.csharp\3.0.0-beta.20240 222.3\buildMultiTargeting\../tools/net7.0/any/ --clear-output-folder=true --shared-source-folders="C:\Repos\azure-sdk-for-net\eng\/../sdk/core/Azure.Core/s rc/Shared/;C:\Users\tarunkhattar\.nuget\packages\microsoft.azure.autorest.csharp\3.0.0-beta.20240222.3\buildMultiTargeting\../content/Generator.Shared/" -- output-folder=C:\Repos\azure-sdk-for-net\sdk\mobilenetwork\Azure.ResourceManager.MobileNetwork\src/Generated --namespace=Azure.ResourceManager.MobileNetwor k" exited with code 1. [C:\Repos\azure-sdk-for-net\sdk\mobilenetwork\Azure.ResourceManager.MobileNetwork\src\Azure.ResourceManager.MobileNetwork.csproj] Build FAILED.
I need to build SDK for new API release ([Hub Generated] Publish private branch 'tarunkhattar/2024-02-01' by khattartarun · Pull Request #27579 · Azure/azure-rest-api-specs (github.com)) which introduces new proxy resource UES and it's nested resource EXTENDEDINFORMATION where former have only list (GetAll) and later have only Get operations supported.

UES  /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.MobileNetwork/packetCoreControlPlanes/{packetCoreControlPlaneName}/ues/
No Get operation
ExtendedInformation  /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.MobileNetwork/packetCoreControlPlanes/{packetCoreControlPlaneName}/ues/{ueId}/extendedInformation/default)
No List operaton.

Do let me know how to handle this error.

The GetAll method check has been removed from the generator.
Since every time it was raised, we just added a configuration to bypass it.
@khattartarun Please try it again with the latest autorest.charp, or get the latest main in azure-sdk-for-net, the error should be gone.


Connection pool performance

Some documentary ticket.
Some time ago rethinkdb can't do parallel queries in a single connection. Then query token was introduced but go driver was using it wrong until v6.
v5 has old connection pool over fatih/pool.v2, but that pool is suitable only for one connection - one query pattern. If there are more queries than pool capacity, then a new connection is created and destroyed after query is done.
If you service is doing many queries at time, you can easy run out of sockets. (a simple benchmark on v5 does this)
v6 has a rethinked new connection pool with connection reusage.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Disable / enable automation

It's a bit confusing when pumps start dosing or lights turn on / off. We should explicitly enable or disable automation.

This is fixed with web-UI now, correct?


Sorting not possible

I have this note in my TestVault. All plugins and css are disabled:
## 1 Tabelle mit erster Spalte

|   a | b   | c   |   d |
| --- | --- | --- | --- |
|   1 | 6   | A   |   F |
|   2 | 5   | B   |   E |
|   3 | 4   | C   |   D |
|   4 | 3   | D   |   C |
|   5 | 2   | E   |   B |
|   6 | 1   | F   |   A |

SYSTEM INFO:
Obsidian version: v1.5.12
Installer version: v1.5.12
Operating system: Windows 10 Home 10.0.19045
Login status: not logged in
Insider build toggle: off
Live preview: off
Base theme: dark
Community theme: none
Snippets enabled: 3
Restricted mode: off
Plugins installed: 13
Plugins enabled: 1
1: Table Sorting v1.4.0
I reinstalled it, without success.

For me, sorting is not possible. What can I do to get it run again?

No entries:

Clicking with the left or right mouse button does not change the sorting.

I believe it, but it does not work for me:

Restarted the vault.
(I know, your plugin was running some weeks before.)

When I active sortable plugin, task, dataview,  I see some console entries:

I seem to have the problem "Did not find the contextmenu..."

I seem to have the problem "Did not find the contextmenu..."

Thank you for your contribution. With that error message I already suspect the cause in the plugin. I will get back to you when it's solved.
Out of curiosity though, what hardware and how many plugins do you have?

 Before integrating these changes into the main branch, I need you to test that this indeed fixed your issue.
I'm uploading the necessary files to this issue thread. Just replace the files in your obsidian-table-sorting plugin folder and restart your Obsidian to apply the changes.
obsidian-table-sorting_contextmenu-fix.zip

No. I still have issues; I cannot see the context menu.
Here is the error msg:
Uncaught TypeError: Cannot read properties of undefined (reading 'contains')
    at eval (plugin:table-sorting:431:37)
    at Array.forEach (<anonymous>)
    at MutationObserver.eval (plugin:table-sorting:426:15)

And here is my SYSTEM Info:
SYSTEM INFO:
Obsidian version: v1.6.7
Installer version: v1.6.3
Operating system: Windows 10 Enterprise 10.0.22631
Login status: not logged in
Insider build toggle: off
Live preview: on
Base theme: light
Community theme: Blue Topaz v2024071502
Snippets enabled: 4
Restricted mode: off
Plugins installed: 40
Plugins enabled: 24
1: Checklist v2.2.13
2: Chem v0.4.0
3: Dataview v0.5.67
4: Dictionary v2.22.0
5: Editor Syntax Highlight v0.1.3
6: Footnote Shortcut v0.1.3
7: Frontmatter Tag Suggest v0.4.1
8: Full Calendar v0.10.7
9: Kanban v2.0.51
10: Latex Environments v0.4.2
11: Outliner v4.8.1
12: Recent Files v1.4.1
13: Tag Wrangler v0.6.1
14: Tasks v7.9.0
15: Charts v3.9.0
16: Metadata Menu v0.8.7
17: LanguageTool Integration v0.3.7
18: Templater v2.4.2
19: Regex Find/Replace v1.2.0
20: Find and replace in selection v1.0.1
21: Linter v1.25.0
22: Mind Map v1.1.0
23: Sortable v0.3.1
24: Table Sorting v1.4.0
I do see the impact of the plug-in such that the table rows are highlighted when I move the cursor over a table, suggesting the plug in is loaded.

@whytong
Even though I tried to trigger that error message, I could not for the life of me reproduce it.

As for the general issue at hand, I'll have to try out a completely different approach that is shipped by the Obsidian API. Initially, I've skipped this solution because of some limitations that arose with my specific use case. Nevertheless, I'm giving it a try to see if it changes something for you.
I'm sure you are already aware but testing the plugin in a fresh vault with only this plugin installed would help me greatly. It would eliminate the need for checking any dependencies and external influences.

New vault, all plugins are deactivated, Windows 10 Obsidian

No sorting possible, not with 1.4.0, not with your zip-package.

@whytong Even though I tried to trigger that error message, I could not for the life of me reproduce it.
But as a side note, sortable and this plugin were not compatible with each other because of some event prevention on sortables end. Can't say if this has changed since then.
As for the general issue at hand, I'll have to try out a completely different approach that is shipped by the Obsidian API. Initially, I've skipped this solution because of some limitations that arose with my specific use case. Nevertheless, I'm giving it a try to see if it changes something for you.
I'm sure you are already aware but testing the plugin in a fresh vault with only this plugin installed would help me greatly. It would eliminate the need for checking any dependencies and external influences.

Sorry if I jumped into the thread, not sure whether merlinuwe and I have the same issue.
But I updated the installer to 1.6.7 and tried it on a new vault.

Table Sorting works in a brand new vault.
Table Sorting does not work in my old vault despite that I disabled all other plugins and CSS snippets. The error message says:

Uncaught TypeError: Cannot read properties of undefined (reading 'contains')        plugin:table-sorting:431
    at eval (plugin:table-sorting:431:37)
    at Array.forEach (<anonymous>)
    at MutationObserver.eval (plugin:table-sorting:426:15)

Not sure what else I can test.

Complete new vault. Only Table sorting plugin installed.

These settings

SYSTEM INFO:
Obsidian version: v1.6.7
Installer version: v1.6.7
Operating system: Windows 10 Home 10.0.19045
Login status: not logged in
Insider build toggle: off
Live preview: off
Base theme: adapt to system
Community theme: none
Snippets enabled: 0
Restricted mode: off
Plugins installed: 1
Plugins enabled: 1
1: Table Sorting v1.4.0
RECOMMENDATIONS:
Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.
This is your new *vault*.

Make a note of something, [[create a link]], or try [the Importer](https://help.obsidian.md/Plugins/Importer)!

When you're ready, delete this note and make the vault your own.

| a     | b     | c     | d     |
| ----- | ----- | ----- | ----- |
| 1     | a     | 4     | d     |
| 2     | b     | 3     | c     |
| 3     | c     | 2     | b     |
| 4     | d     | 1     | a     |

No sorting possible with 1.4.0
Error messages in console

It's progressively getting harder to debug this issue. Glad, we've got it working on an empty vault though. Wouldn't have thought that an issue of that persistence could arise.
Nevertheless, I want to know if you get any sort of hints that you've clicked a table when right clicking. Usually, there should be a little message popping up on the screen that you've clicked a table with its table ID. With dev mode enabled you'd get an additional log in the console.
If you came from a previous version of this plugin, the handling has changed significantly. Read through the changelogs here if you are still applying the previous method of sorting which would be left clicking the table headers.
@whytong
As for you, I'd recommend completely removing the sortable folder from the plugin folder for now. I've found that the plugin has persistent changes that affect the vault even after the plugin was disabled. It is marked as deprecated by now if you didn't know.

No, there is no context menu visible.
The console shows this after activating in the settings of the plugin.

Okay, wow. I've just noticed that you use the preview/reading mode where not even the default context menu appears. It's just something I never used in my entire Obsidian lifecycle.
When you switch to editing mode, I assume, you should be able to see and sort your table.
That just means that I have to clean up the remaining reading mode styling which currently indicates a possible interaction. I also have to find a way to make it work with the reading view. I think we've come a good bit closer to a solution.

No, I cannot sort in editing mode.

Related to the issue #26 where you can't sort in the source editing mode. Make sure to have this toggle enabled.

Interesting, your plugin works, but only in Live-preview mode. What do you think about mentioning this in the documentation?

You are right, thank you for the suggestion. That'll clear up some of the confusion.
So to summarize this issue you are having:
Since you are using the preview mode and source edit mode, you can't use the sorting plugin. This issue you are having is otherwise not related to any other plugins or configurations in your vault. It does work though when you are using the live preview edit mode.


Add a retry in run_envoy_docker

CI fails fairly frequently due to bad dockerhub endpoint:
Generating script.
Script contents:
ci/run_envoy_docker.sh 'ci/upload_gcs_artifact.sh /source/generated/fuzz_coverage fuzz_coverage'
========================== Starting Command Output ===========================
/bin/bash --noprofile --norc /srv/azure-pipelines/_work/_temp/8788096c-b5e6-4a3b-8f8a-3c150786c997.sh
Error response from daemon: Get https://registry-1.docker.io/v2/: EOF

https://dev.azure.com/cncf/envoy/_build/results?buildId=113688&view=logs&j=7b6d60aa-b5de-5f9b-678c-063d31881e76&t=0c8bcd19-14ac-5aba-0af8-facf9399c298&l=14
as this seems to be a transient issue that resolves quickly, we could probably mitigate this with a retry in run_envoy_docker

we could also remove any unnecessary use of it - eg in the artefact upload jobs

cc @lizan

Another idea is that we host image tarball at our side if run in CI env, load it when use since CI image won't change often.

Another idea is that we host image tarball at our side

its not that simple - the docker image is huge and the cache is not much faster than the pulling directly from dockerhub
essentially, it can be faster doing this but iff there are no additonal steps required (a big iff)
see #22093 for an implementation

This problem seems to be happening across images and randomly but fairly frequently so ive opened an upstream ticket here https://github.com/docker/hub-feedback/issues/2262

@phlax seems we need to add retry step to fix flake tests.

@phlax seems we need to add retry step to fix flake tests.

the problem with adding a retry - apart from being an imperfect solution that may/not work, is that the envoy-build image is one of many cases where we are pulling images, so just adding a retry on that is not going to resolve the problem

one observation is that the bigger image that its pulling, the more likely to fail - which i think is why we see this more frequently with the envoy-build image and skywalking, jaeger, etc
another point of note here is that while the dockerhub issue is by far the biggest flake atm for verify examples - this can also flake on debian/ubuntu updates and pip downloads
i think shifting the verify tests to incremental testing would not only reduce the workload/transfer that we are placing on dockerhub, but would also mitigate some of these other transients also


DL: Remove keras dependency

JIRA: MADLIB-1438
Remove keras dependency to use the keras from the tensorflow package.
Remove the channels first test since it only works on GPUs.

[ ] Add the module name, JIRA# to PR/commit and description.
[ ] Add tests for the change.

LGTM!


Fixes compiling for watchOS only and Digital Crown bug

Adds conditional compile time checks for watchOS only.
When compiling mixed projects, the Package gets picked up by `xcodebuild`, for example: for iOS, and the compiler then throws errors about not being able to find `WatchKit`.
This is the unfortunate state of SPM packages at the moment using the latest toolchains, and this PR works around that.

Fixes a bug occurring on watchOS 9.1 when building with Xcode 14.1 RC 2 where the time input would crash on appear due to invalid values setup on the Digital Crown event modifier.

Many thanks for this PR!
Are these conditionals truly necessary? I thought this mixed projects issue had been remedied already by using @available. If not, I’ll add the #ifs. Please open another issue if this is still relevant.
I’m glad you could help others out with a fix for the watchOS 9 crash while I was away and I’ve added a tweaked version of your solution to the latest version of Watch Date Picker. If you have any idea how to completely hide the crown indicator, please let me know! It didn’t used to appear like this in watchOS 8.
The dynamic styling changes I won’t be accepting. They make the appearance diverge too much from the system’s time input view. For example the buttons are now too small.

System
Watch Date Picker
This PR

Are these conditionals truly necessary? I thought this mixed projects issue had been remedied already by using @available.

Xcode 14.x would refuse to compile a mixed-target or multi-platform project without the conditionals. A standard iOS project  would be fine (sample app)

I’m glad you could help others out with a fix for the watchOS 9 crash while I was away and I’ve added a tweaked version of your solution to the latest version of Watch Date Picker. If you have any idea how to completely hide the crown indicator, please let me know! It didn’t used to appear like this in watchOS 8.

I'll keep an eye out for this one, thank you.


✨[Feature Request]: ADD BACK TO TOP BUTTON ON FIND DOCTORS PAGE

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
ADD BACK TO TOP BUTTON ON FIND DOCTORS PAGE
Use Case
BACK TO TOP BUTTON ON FIND DOCTORS PAGE
Benefits
CAN GO DIRECTLY TO TOP OF PAGE BY PRESSING THIS BUTTON
Add ScreenShots
No response
Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@varshith257 PLEASE ASSIGN ME THIS ISSUE I WANT TO WORK ON IT


[bug] Conda Store auth redirects via http not https

When logging into Conda Store, the redirect back from Keycloak seems to get confused between HTTP and HTTPS.
This can cause blocked content in the browser (in particular, within Cypress tests).
This is the flow:
https://qhub.mycompany.com/auth/realms/qhub/login-actions/authenticate?session_code=...
Returns a 302 to https://qhub.mycompany.com/conda-store/oauth_callback?state=...
Which returns a 308 to http://qhub.mycompany.com/conda-store/oauth_callback/?state=... (note http)
Which returns a 308 to http://qhub.mycompany.com/conda-store/oauth_callback/?state=...
For some reason this loads https://qhub.mycompany.com/conda-store/oauth_callback/?state=... (instead of http)
Which returns a 302 to http://qhub.mycompany.com/conda-store/user/
Which returns a 302 to http://qhub.mycompany.com/conda-store/user/
For some reason this loads https://qhub.mycompany.com/conda-store/user/

Where I say "for some reason this loads https", I think that's related to Upgrade-Insecure-Requests provided by the browser's request headers, and the CSP response - however, I suspect that will all be irrelevant if we work to ensure no HTTP is involved in the first place.
I'm not sure if this is a problem inside Conda Store or within QHub's integration of Conda Store.

Ah yes I remember this issue. It was fixed by https://github.com/Quansight/conda-store/pull/230 in the 0.3.10 conda-store-server release. I've bumped the conda-store version in https://github.com/Quansight/qhub/pull/1118 which should resolve this issue. I'll make that PR close that issue with an appropriate test to make sure it is working.

The PR in question didn't quite work. Another one in progress.

I think some of this has been fixed (and I'm not sure if Cypress will still fail) but I still see an http creep in during the OAuth2 login process:
https://qhub.mycompany.com/conda-store/oauth_callback?state=...
returns a 302 to
http://qhub.mycompany.com/conda-store/user/

When it is available, upgrade QHub's conda-store to 0.3.11 or higher.

This needs to be tested and see if it disappears but I believe this should be resolved.

@costrouc I believe this has been resolved but feel free to open this if you think this is still an issue.


Formatting != operator with the non-compact formatter

The != operator seems to be not correctly formatted when the Use compact formatter option is disabled in settings.
Before formatting
if (0 != 0){}

After
if (0 ! = 0) {}

And there is an error message in the output tab: ERROR line 1    if      syntax error
Thank you for this extension!

@EnixCoda issue has been resolved in latest version 0.2.12. Thank you for your submission.


Fix issue #742

Fix issue #742

Please elaborate more in the commit message about the solution.

Is #1367 related to this PR?

@LaszloLango : No, it It appears not, but I will also check that issue.


WIP make symbolication async

I was curious how bad this was and then suddenly I was "done" lol
This compiles and correctly handles amd64.
TODO:

[x] copy-paste changes to other stackwalking backends (and re-enable them)
[x] make tests work with async (with #[tokio::test]?)
[x] make sure the symbolic-syms shim still compiles
[ ] performance analysis (socc-pair --bench)
[ ] test #[tokio::main(flavor = "current_thread")]

implements #327

Me: what if I add Send + Sync everywhere. Surely that won't work. There's like, borrows and stuff right?
rustc: seems fine
Me: well, this is the entire purpose for your existence so I'm going to assume this is correct..?

I feel like I'm probably not full leveraging the power of async in fetch_symbol_file and save_contents, but that's probably better left to the sentry folks who actually use async all the time.

Me: what if I add Send + Sync everywhere. Surely that won't work. There's like, borrows and stuff right?

The minidump data is only ever borrowed immutably. :) At one point I pondered what a multithreaded process_minidump implementation would look like but I discarded that idea because the actual unwinding takes no time at all. If anything, per the Discord discussion we had today, doing all the unwinding in one pass, fetching all the symbol files needed to symbolicate in parallel, and then symbolicating the stacks in one fell swoop would be the most clever approach. (The only downside being that the current heuristics for stack scanning rely on symbols to determine if a return address is inside a known function.)

Given the CI errors here, you're probably going to need to pull in #[tokio::test].

As far as I know, this is a complete implementation. Still not gonna land it anytime this ~month ideally but it's here if anyone wants to build on it.

cc @nornagon in case this is at all still relevant to you

Oh cool! Is the conversion to the new symbolizer done too?

Unfortunately the folks at sentry decided to focus on something else, and I had higher priorities to take care of first. Also in local testing we've still found this implementation is still significantly faster than breakpad, which is nice.
Now that rust-minidump is ready for production, I checked in with sentry and they cited the lack of async symbolication support as an issue. So this is in some sense "preparing" for the eventual future where they're the backend. But for the short-to-midterm breakpad-symbols is gonna be The Backend.
Note that mozilla is in the middle of transitioning our infrastructure to rust-minidump, so I'm holding off on landing this patch until probably january when everyone's back from the holidays, to minimize risk. In the meantime, this branch is here for you to build on top of if you want (I imagine you still need to add a bit more async to make minidump loading work for you, and to add a bunch of wasm cfgs). If you'd like to touch base on what electron needs from this, I'm happy to discuss further details on whatever your preferred platform is.

Rebased, but this is effectively reverting the functionality of #359 right now. I need to rework the code to use Response::chunks which is the "async" version of Read for reqwest. This is slightly annoying because File i/o and what not does not use chunks. Might just copy-paste the code, idk.

Async streaming parsing now works? Kinda sloppy but seems legit?
miniump-stackwalk runtime: 00m:06s:885ms (6885ms)
miniump-stackwalk max memory (rss): 255MB (268423168 bytes)

I think that's a wrap for me in 2021, see y'all in January.

Ok one last note before I forget:
In this design, there's kind of a split async personality:

When reading from a file, once you start reading you're in a blocking loop that will run to completion.
When downloading a file, the process is fully async, and will await any time we need more data from the network.

This is intuitively the right balance since async file I/O is a Lie, but I don't know enough about what people "want" from async to know for sure.
Although I could imagine this "version" being friendlier for compile-to-wasm? So maybe we do end up keeping both paths around. shrug

I added a --mock-server mode to socc-pair, so I can now bench the server path without actually hitting the network:
socc-pair
--api-token=<snip>
--run-local=C:\Users\ninte\dev\rust-minidump\minidump-stackwalk
--crash-id=9520523c-45ff-4d6c-a186-4fec80211216
--output-file=output.diff
--mock-server
--bench=30

Summary of Results
This Change

average runtime: 1602ms -> 1643ms (1.02x runtime, probably within margin of error)
average max-memory: 253MB -> 254MB (no significant change)

Remeasurement of streaming parsing change

average runtime: 1864ms -> 1602ms (0.86x runtime, nice little win but easily dwarfed by real download times)
average max-memory: 383MB -> 253MB (0.66x memory usage!)

Individual Results:
async (this PR) - 30 iterations, cold cache (local static file server):
benchmark results (ms):
  1630, 1642, 1604, 1645, 1645, 1655, 1649, 1626, 1684, 1641, 1633, 1652, 1628, 1624, 1622, 1660, 1640, 1663, 1599, 1717, 1648, 1674, 1671, 1627, 1626, 1666, 1624, 1624, 1643, 1642,
average runtime: 00m:01s:643ms (1643ms)
median runtime: 00m:01s:642ms (1642ms)
min runtime: 00m:01s:599ms (1599ms)
max runtime: 00m:01s:717ms (1717ms)

max memory (rss) results (bytes):
  266653696, 266702848, 266485760, 265920512, 267214848, 266309632, 267427840, 266227712, 266633216, 265838592, 266575872, 266588160, 265940992, 266084352, 266993664, 266498048, 265887744, 266072064, 266149888, 266407936, 266211328, 266932224, 266964992, 267255808, 266276864, 266936320, 266256384, 267132928, 265351168, 267161600,
average max-memory: 254MB (266503099 bytes)
median max-memory: 254MB (266498048 bytes)
min max-memory: 253MB (265351168 bytes)
max max-memory: 255MB (267427840 bytes)

## master - 30 iterations, cold cache (local static file server):

  1610, 1612, 1574, 1612, 1600, 1568, 1583, 1594, 1590, 1607, 1625, 1581, 1610, 1799, 1599, 1592, 1606, 1594, 1597, 1601, 1571, 1586, 1590, 1596, 1592, 1589, 1603, 1595, 1577, 1631,
average runtime: 00m:01s:602ms (1602ms)
median runtime: 00m:01s:596ms (1596ms)
min runtime: 00m:01s:568ms (1568ms)
max runtime: 00m:01s:799ms (1799ms)

max memory (rss) results (bytes):
  265908224, 266510336, 265248768, 265646080, 265555968, 265035776, 266092544, 265515008, 265510912, 265961472, 266174464, 264835072, 265707520, 265990144, 266002432, 266915840, 266076160, 265310208, 265334784, 266268672, 265588736, 265003008, 265302016, 265940992, 265338880, 265875456, 265576448, 265646080, 265846784, 265408512,
average max-memory: 253MB (265704243 bytes)
median max-memory: 253MB (265646080 bytes)
min max-memory: 252MB (264835072 bytes)
max max-memory: 254MB (266915840 bytes)

before streaming parser (2dc7ecc384f8ed123387cace30157c5288) - 30 iterations, cold cache (local static file server):
benchmark results (ms):
  1771, 1897, 1815, 1888, 2049, 2035, 1733, 1932, 1736, 1751, 1890, 1742, 1912, 1790, 1881, 2069, 1911, 1724, 1762, 1855, 1832, 2029, 1915, 1932, 1863, 1872, 1830, 1762, 1854, 1903,
average runtime: 00m:01s:864ms (1864ms)
median runtime: 00m:01s:872ms (1872ms)
min runtime: 00m:01s:724ms (1724ms)
max runtime: 00m:02s:069ms (2069ms)

max memory (rss) results (bytes):
  402206720, 401584128, 402771968, 400928768, 401539072, 402026496, 401924096, 402530304, 402178048, 402096128, 402280448, 401825792, 401575936, 402894848, 402010112, 401981440, 401850368, 402341888, 401457152, 402329600, 401281024, 401469440, 401362944, 402178048, 402026496, 402587648, 402583552, 402243584, 401514496, 402354176,
average max-memory: 383MB (401997824 bytes)
median max-memory: 383MB (402026496 bytes)
min max-memory: 382MB (400928768 bytes)
max max-memory: 384MB (402894848 bytes)

@jan-auer I'm inclined to land this so it stops being a looming specter that I keep needing to rebase. Any final objections / arguments against it?

No objections, landing as soon as CI passes...

I AM FREE


Possible misalignment with standard

I might be misreading this, but it seems like this is allowing a bracketed rest parameter like this:
function (a, b, [c, d]) { }
which seems to be invalid syntax from what I can tell.
Am I wrong here?

Nevermind, I see my mistake....the line before it will catch that


Set minSteerSpeed for Honda

I don't think this is necessary as we're removing that information from the car docs.

What about to determine if we can steer to zero or not?


Hi, Do you provide the sourcecode of training scripts now?

You've done a excellent job!

Yes @luannd @ctymoszek excellent job, and yes the training code would be nice to see how it does with different datasets.

@gogo03 @isgursoy The researchers who worked on this project have since moved on and are no longer maintaining the software. I have updated the accompanying documentation to reflect this. Feel free to build upon or modify it for your own purposes, but please understand that it is provided as-is with no additional warranty.


Update TrinaryTest.php

My solution should not have passed all the tests. For 311 it gives 31, but the correct answer is 0
function toDecimal(string $number): int
{
return array_reduce(
str_split(strrev($number)),
fn (array $acc,int $digit): array => array(
'power' => $acc['power'] + 1,
'valid' => $acc['valid'] && in_array($digit,range(0,2)),
'sum' => ($digit * 3 ** $acc['power'] + $acc['sum']) * $acc['valid'],
),
array(
'power' => 0,
'sum' => 0,
'valid' => 1
)
)['sum'];
}

This is a legacy exercise and has been deprecated.  So I don't think we should add this test case.  I would consider a PR to mark this exercise as such if you are interested.
To do this, we need to add the "status": "deprecated" to the exercise's entry in the ./config.json file.

As a rule though, we ask that when people encounter missing test cases, it first be discussed as an issue in the problem-specifications repo (https://github.com/exercism/problem-specifications), so that it can be discussed and subsequently distributed to all of the different exercism tracks.

I'm going to close this PR, buy as I commented previous if you'd like to open another with the deprecated status, I'll merge it.


[DOCS] Add documentation main page for links examples

We have many examples in the repo, but the documentation doesn't contain any. It would be good to add the walk-throughs from the github repo.
This page should link to the notebooks in the repo. Separate issues will be created for each example.

I'd like to work on this issue. Could you please assign it to me ?

@devin-petersohn We have a "getting started" section in the github README with links to notebooks and examples. I'm going to close this issue.


[RFC] optional plugin to provide table of contents in help files in popup menu

Usually, there is a table of contents at the top of a help file.  But it might not be detailed enough.  And having to jump to the top of the file makes us lose the current context.  This PR adds an optional helptoc plugin which lets the user access an interactive popup menu showing a table of contents for the current help file, from any position in the latter.
Among other features, the menu can be collapsed or expanded to control the type of overview one wants to see (e.g. only the main sections, or everything down to every tag).  Also, it can be fuzzy searched to find an arbitrary entry from some text input by the user.
Here is a short demo:
https://i.imgur.com/23RwN1K.mp4
It also works in man pages, markdown files, and terminal buffers.  I added a short documentation at :help help-TOC.
I think it could be helpful to people frequently using Vim to read some documentation.  If I'm wrong, feel free to close the PR.

I like this. From a quick glance, I noticed it supports standard help tags. How does it deal with the help headers as I saw in the video? If there is a "standard" way to support helptoc, documenting it as such would mean 3rd-party help files could be written to support it as well.

I like this. From a quick glance, I noticed it supports standard help tags. How does it deal with the help headers as I saw in the video? If there is a "standard" way to support helptoc, documenting it as such would mean 3rd-party help files could be written to support it as well.

Sorry for the late response, I couldn't answer earlier.
Help files are tricky to handle, because there doesn't seem to be a widely adopted convention.  At least one which is accurate enough.  It seems everybody uses various types of headers however they like, or based on how they look, not much on what their type mean.  Anyway, the plugin tries to add these lines as entries in the TOC:

*01.1* some text
1. some text
1.2 some text
1.2.3 some text
some header ~
SOME HEADLINE

The tricky part is:

not conflating 1. with a simple list item (like at :help registers)
assigning the correct level to each entry (the tags are always assigned the highest level, since their scope is always the smallest one)

I don't remember the exact logic.  I just tested the algorithm on as many help files I had (builtin + third-party) and checked the end result looked as much as possible to what the user would expect.

I tried it out and it is quite nice. Thanks for this!

I'll add the :Helptoc tag. I leave the rest for a followup PR

A first step: https://github.com/vim/vim/pull/16097.


Error is not precise when 2 database connexions use the same display name

Bug description
Hello
with Superset 3.1.1, Error is not precise when 2 database connexions use the same display name

Regards
How to reproduce the bug
Just create 2 database connexion with the same display name
Screenshots/recordings
No response
Superset version
master / latest-dev
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[X] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

Do you mean to say you'd like it to:

Throw an error saying "Database name _____ is in use. Please give your database connection a unique name"
Let you name databases with the same name?

I hope you mean #1, as #2 would cause a lot of confusion/problems

Hello,
I mean (1).
In this use case, with a error message more precise (like your proposition "Please give your database connection a unique name"), the user would immediately know that he must change his connection display name.
Best regards


[WIP] HybridTopologyFactory-geometry compatibility

add old-new electrostatics exceptions.  removing electrostatic and steric exceptions from old-new Nonbonded
Force (into CustomBondForce).  addInteractionGroup between old-old and
new-new.  removing old-old and new-new Nonbonded Force exception.  add
angle exceptions and _logger comments

https://github.com/choderalab/perses/blob/07f4607c97a68c6221db7041f272b1bfccbbd51f/perses/annihilation/new_relative.py#L1261-L1285
as this is currently implemented in master, it looks like that for every atom pair of unique_old-unique_old, if there is a pairwise exception, the exception is being placed such that the custom sterics is being deleted and the electrostatics/steric exceptions are being placed in the 'standard_nonbonded_force' (without an exception offset).  otherwise, if it is not an exception, the electrostatics and sterics are being placed into the 'standard nonbonded force' without a parameter offset.  If I am reading this correctly, wouldn't this mean that the old-old and new-new atom pairs have nonbonded interactions for the entire lambda protocol?  if this is the case, then it doesn't correspond to the way the geometry.py placed/deletes unique_new/unique_old atoms.


ValidationRenderer and MaterialDesignLite

Hi all. I have such issue:
We use MaterialDesignLite library in out project.
We created ValidationRenderer, and in the method "render(error, target)" it's adds a span element with the error message and sets class 'is-invalid' to '.mdl-textfield' to the control group. As validateTrigger we use 'blur' value.
All work fine except one situation: When you start changing something in mdl textfield, mdl automaticaly removes class 'is-invalid' from that textfield. But, if you after that returns value of this textfield to the previous value before 'blur' trigger, ValidationRenderer won't run method "render(error, target)".
Is it possible somhow to fire the method "render(error, target)" after each exit from the control, even if the value of that control haven't changed?

@crabulik we have a new release with changes to the ValidationRenderer API. Give the new release a try and let us know if your issue is resolved.
If you still have an issue, please put together a gist that reproduces the issue and post a question to the aurelia validation gitter channel or stackoverflow and someone will help you with a fix.
https://gist.github.com/jdanyow/3625244afe4f188785190b23f2e7f472


Different Game Over For A Specific Song

Describe your problem here.
so like i want to make it so in a specific song i want an image to appear when you die instead of bf's skeleton
and i dont know how to do this myself
please help me
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No i didnt edit anything

there is a example on the wiki lmao (lua api section)

o ok

which one 💀

which one 💀

https://github.com/ShadowMario/FNF-PsychEngine/wiki/Lua-Script-API#example-scripts
the one who changes the game over shit

o ok

thanks


bump Scala versions and enable Scalafix

and fix new 2.13.3 warnings

tweaked comment in 55fc269eb2a5f3107459fec33c9d6ae1045ecb97:
-// scalafix; run with `scalafixEnable` followed by `scalafix` and/or `Test/scalafix`
+// scalafix; run with `scalafixEnable` followed by `scalafixAll`


chore(ci): deprecate deployent configs

Replaces deprecated Deployment Configs with Deployments. Also removes image streams and uses a randomly generated envar to ensure consistent rollouts.
Unfortunately there will be some downtime when this deploys.
Steps:

Delete deployment configs and services for TEST database and backend
Delete deployment configs and services for PROD database and backend
Merge this PR
Let deployments to TEST and PROD complete

Thanks for the PR!
Deployments, as required, will be available below:

Java
Py
Go

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

lets update this to remove these unused files , since we have HELM for this repo already

@mishraomp Lol!  Good thing I've been doing this so often lately it's happening entirely on autopilot.  :P


Crash in TableViewSectionDataSource when showing and hiding a section

In my project I have a chat detail with a list of messages. I use the RxTableViewSectionedAnimatedDataSource with two sections.
In most cases, only the section with the messages is shown but when the user scrolls to the top of the table view, I set the isLoading property so an additional section at the top is shown with just 1 cell showing a spinner.
let data = Observable.combineLatest(viewModel.messages.asObservable(), viewModel.isLoading.asObservable()) { (messages: [Message], isLoading: Bool) -> [AnimatableLoaderSectionModel] in
        var sections: [AnimatableLoaderSectionModel] = []
        if isLoading {
            sections.append(.LoaderSection)
        }
        sections.append(.ItemsSection(items: messages.map({ .Item(item: $0) })))
        return sections
}

After loading the older messages is done isLoading is set back to false and the new messages appear in viewModel.messages.
This works fine most of the times, but when I scroll to top really fast doing it again and again I get a crash in https://github.com/RxSwiftCommunity/RxDataSources/blob/master/Sources/RxDataSources/TableViewSectionedDataSource.swift#L190.
When I do not append the .LoaderSection when constructing data then it does not happen, ever.
To be more exact, in the method
open func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        precondition(indexPath.item < _sectionModels[indexPath.section].items.count)

The indexPath is for example section=1, row=6 but the _sectionModels only have 1 item, resulting in an out fo bound crash basically.
It looks like the data source is trying to update some item and still thinks the
.LoaderSection exists (so .LoaderSection is section=0 and the update is for section=1) but the _sectionModels does not contain the .LoaderSection anymore.

I guess I'm going through the same situation, do you have any solution?

I'd gladly try to check this but do you have some kind of reduced example?

My issue was with other not disposable sequence, not related with TableViewSectionDataSource, thanks for asking @mkko

I have same issue. In my case my tableview continuously append and remove cell using accept data to datasource.


Aperio GT 450 SVS files don't populate any ImageDescription tag for label and macro IFDs

Looking there for "label" and "macro" strings would break on those images. Instead remove IFDs with TIFF_SUBFILETYPEs  1 and 9 instead (the real images will be type 0).

@fiendish Are you sure this is a problem?  There is code to fall back to the SubfileType if the ImageDescription doesn't contain the associated image key.  If so, can you provide a sample file?

Try redacting the macro image from (warning large download) https://slides.cldavis.org/slides/slideupload/NEVPC2020_8.svs

Ah, you're right but...
I tried redacting the macro image from (warning 1.3GB download) (also warning, this is not my file) https://slides.cldavis.org/slides/slideupload/NEVPC2020_8.svs and it didn't work for me and the macro image remained. Maybe I did something wrong in my testing, but I went and looked inside process.py and saw that you're internally using openslide which has an open bug report regarding identification of label and macro images so I was wondering if perhaps the macro image string isn't getting into my image redaction list despite your local key setting.


Stuck at "Locating topologies" after launching on CentOS

OS: CentOS 7 (lastest)
Arch: x86_64

@calvintam236 any more info on this?

seems to be Debian comes with older Linux kerne (3.x) that scope cannot run.

seems to be Debian comes with older Linux kernel (3.x) that scope cannot run.

Scope should run just fine on kernels >=3.10. Though, I would certainly recommend moving to 4.4 or higher.

I have already switched back to Ubuntu when scope doesn't launch.
You should able to reproduce the error on the latest version of CentOS even in the virtual machine.
I might attach logs if I have time to install Debian as virtual machine once I have time.. (kinda busy recently)

I have just run CentOS through Vagrant ( https://app.vagrantup.com/centos/boxes/7 ) and works just fine:

Just put the following content in a file called Vagranfile:
# -*- mode: ruby -*-
# vi: set ft=ruby :

Vagrant.configure("2") do |config|

  # on fedora-like, you must use the downloadable package of vagrant
  # available on hashi corp website.
  # The per distrib provided package wont provide winrm support..
  # see https://www.vagrantup.com/downloads.html

  config.vm.define :win2016 do |win|
    win.vm.box = "centos/7"
    win.vm.network "private_network", ip: "<IP_ADDRESS>"
  end

end

And run the following commands:
$ vagrant up
$ vagrant ssh
[vagrant@ip-10-0-2-15 ~]$ sudo yum install docker
[vagrant@ip-10-0-2-15 ~]$ sudo systemctl start docker
[vagrant@ip-10-0-2-15 ~]$ sudo curl -L git.io/scope -o /usr/local/bin/scope
[vagrant@ip-10-0-2-15 ~]$ sudo chmod a+x /usr/local/bin/scope
[vagrant@ip-10-0-2-15 ~]$ sudo /usr/local/bin/scope launch


Synopsys: Automated PR: Update express/4.17.1 to 4.21.0

Vulnerabilities associated with express/4.17.1
BDSA-2024-6188 (HIGH): Express.js web framework is vulnerable to Cross-Site Scripting (XSS) due to the improper handling of user input in the response.redirect() function. This could allow an attacker to execute JavaScript code on the users browser.
Note The attacker must be in control of the input to response.redirect() and the user must click before the redirect occurs.
BDSA-2024-0722 (MEDIUM): Express.js contains an open redirect vulnerability due to insufficient validation of user supplied URLs. An attacker could exploit this by providing a crafted URL when performing a redirect bypassing the redirect allow list leading to an open redirect.
Click Here To See More Details On Server

Automated PR Comment From BlackDuck

❌ Found dependencies violating policy!

Policies Violated
Dependency
License(s)
Vulnerabilities
Short Term Recommended Upgrade
Long Term Recommended Upgrade
Resolved / Filtered Out

No High Severity Vulns
PostgreSQL JDBC Driver (pgjdbc)
N/A
❌   CVE-2022-31197 HIGH CVSS 8.0
9.4-1206-jdbc42 (2 known vulnerabilities)
42.7.4 (0 known vulnerabilities)
:grey_exclamation:

No High Severity Vulns
tough-cookie
N/A
❌   BDSA-2023-1661 HIGH CVSS 8.8

5.0.0 (0 known vulnerabilities)
:grey_exclamation:

No Medium Severity Vulns
iText, a JAVA-PDF library
N/A
❌   CVE-2022-24197 MEDIUM CVSS 6.5
<IP_ADDRESS> (2 known vulnerabilities)
<IP_ADDRESS> (2 known vulnerabilities)
:grey_exclamation:


Improvements on SplitChunks article

Besides the fact that the document might need some gardening and organization, there are some stuff that I couldn't completely get from the codebase or the config so there are some requests and gaps that might need to be filled.
Specially these:

include the different things that you can pass to runtimeChunk. It can be a boolean, 'single', 'multiple' or an object that has property name whose value can be either a string or a function of signature (entrypoint: Chunk) => string.
a paragraph explaining better how enforces works and combines with other options. The enforce allows the special cache group to ignore the minSize, minChunks, maxAsyncRequests and maxInitialRequests set in the overall splitChunks config and purely use test to decide what goes into the chunk. What I'm unclear on is how that fits in when used with reuseExistingChunk.
cacheGroups can receive a method. What are the arguments and what value needs to be returned.
function signature description for name option

Article pull request is https://github.com/webpack/webpack.js.org/pull/1955

As always, I can help, but need the knowledge 😄

The magic value true automatically chooses a name based on chunks and cache group key, otherwise a string or function can be passed.

What is the signature for this function?

@glen-84 that's a missing thing right now.

Would this be a good place to request documenting how to pass a function to splitChunks.cacheGroups[xxx].test or should that be its own ticket?  The documentation only mentions it in passing, and I haven't been able to reverse-engineer what the callback parameters are on my own at this time.

Actually I'm not 100% clear what the regex version of test is testing against, and I'm sure that's complicated by my build chain where I start with Typescript and wind up with name-shorthash.ext.  Maybe just a little more detail on test generally?  Also: trackback to #2103.

Fixed by #2183? Or are there still some parts that are unclear?
@glen-84

What is the signature for this function?

relevant section
@thw0rted

Would this be a good place to request documenting how to pass a function to splitChunks.cacheGroups[xxx].test

relevant section

@chrisdothtml do you feel you've covered what was mentioned in my initial description? If that's the case we can close it for sure.

cacheGroups can receive a method. What are the arguments and what value needs to be returned.
function signature description for name option

I think these ones were covered by my PR

include the different things that you can pass to runtimeChunk. It can be a boolean, 'single', 'multiple' or an object that has property name whose value can be either a string or a function of signature (entrypoint: Chunk) => string.

Looks like this mostly already exists on the optimization page. Only thing I can see missing is the function signature

a paragraph explaining better how enforces works and combines with other options. The enforce allows the special cache group to ignore the minSize, minChunks, maxAsyncRequests and maxInitialRequests set in the overall splitChunks config and purely use test to decide what goes into the chunk. What I'm unclear on is how that fits in when used with reuseExistingChunk.

Don't think this exists anywhere currently. Perhaps it could be provided in a tip somewhere on the splitChunks page?

Thanks @chrisdothtml , it's a bit more fleshed out, but I'm not sure it's done.
The updated docs say that the function takes one argument, "chunks" (without any description of the type of that argument!). In my testing (webpack 4.8.3) it looks like the callback is actually invoked with two arguments.  I logged the prototype of each argument and I see output like NormalModule, [Chunk], where the first argument could also be a DelegatedModule, CssModule, MultiModule, etc.
So, what I'd like to see is:

double-check the arguments to the callback, I'm guessing the signature is (Module, Chunk[])
Link to docs about the type of each argument (is the API for NormalModule, Chunk, etc documented somewhere?)
a slightly expanded example that actually uses some part of the arguments to make a simple decision that would make sense in a real-world project, e.g. I use test: (mod) => mod && mod.resource && mod.resource.endsWith(".special.js") to put .special.js files in their own chunk.

As I said on another issue, my main focus is helping users who either don't have experience with code splitting or didn't fully grok how it worked in the older plugin (like me).

I think the documentation could still be improved.

The documentation (https://webpack.js.org/plugins/split-chunks-plugin/#splitchunks-cachegroups-test) shows a code example defining splitChunks.test (which I dont believe exists?) instead of splitChunks.cacheGroups[yourGroup].test.

The example code only illustrates that you can specify a function, it doesn't show any example functionality. It would be beneficial to show use of the test function arguments. It would also be useful to have info about the function parameter types and their properties. Maybe a basic example of what properties one might match against.

The example code shows test(chunks) as the function signature yet (from my naive attempts) it seems like the example code in the post by @thw0rted (above) where the signature is test(module) is more correct? Though I am fairly ignorant with regards to the webpack object types - again, type definitions/specs would be useful.

Not entirely convinced that this was solved. As @thw0rted mentioned, the knowledge required to effectively configure splitChunks extends beyond knowing function signatures, and into Webpack's OO model for representing modules, dependencies, and other code encapsulation and organization abstractions.
There are several resources online (vid, slides) that try to fill in the gaps, although short of reading the source code, wouldn't it be worthwhile, considering that through the config we can access some of these objects, to add a section in the documentation that covers Webpack's internals?


Feature Request: Add module support for modules like Dragging tokens (Token Ruler)

Good afternoon. At the moment, the system works well with the basic character speed value. But when choosing a maneuver in battle, the character's speed in the character sheet changes, but not the speed of the token. Can you add support for changing the speed so that modules such as Elevation Ruler or Drag Ruler work?

It works fine on the system version 0.17.13 and Elevation ruler module version 0.10.14


Can't push Windows based images to private registry >= 2.7.0

Hi everyone,
I am using a private registry on Ubuntu 18.04.1 LTS (with insecure access from Windows Server 2016).
Uploading Windows images from Windows Server 2016 works fine with registry version <= 2.6.2.
But if I use registry version >= 2.7.0, I get following error on the Windows console during docker push:
received unexpected HTTP status: 500 Internal Server Error
When I look into the logs of the registry container, I see messages like these:
manifest blob unknown: blob unknown to registry
I tested these configurations with different registry versions:
docker run -d -p 5000:5000 --restart always --name registry registry:2
docker run -d -p 5555:5000 --restart always --name registry registry:2
I attached the logs for registry 2.6.2 (good case) and registry 2.7.1 (bad case):
Logs_Registry_2.6.2.txt
Logs_Registry_2.7.1.txt
What goes wrong here?
Tested with my own Docker image as well as with the hello-world image (both based on nanoserver).
The push of the Linux based hello-world image works fine, even with registry >= 2.7.0.
Thanks in advance for your replies

This is the docker version output on Ubuntu:
Client:
 Version:           18.09.1
 API version:       1.39
 Go version:        go1.10.6
 Git commit:        4c52b90
 Built:             Wed Jan  9 19:35:31 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          18.09.1
  API version:      1.39 (minimum version 1.12)
  Go version:       go1.10.6
  Git commit:       4c52b90
  Built:            Wed Jan  9 19:02:44 2019
  OS/Arch:          linux/amd64
  Experimental:     true

This is the docker version output on Windows Server:
Client:
 Version:           18.09.1
 API version:       1.39
 Go version:        go1.10.6
 Git commit:        20b67756d0
 Built:             unknown-buildtime
 OS/Arch:           windows/amd64
 Experimental:      false

Server:
 Engine:
  Version:          18.09.1
  API version:      1.39 (minimum version 1.24)
  Go version:       go1.10.6
  Git commit:       20b67756d0
  Built:            01/09/2019 17:09:57
  OS/Arch:          windows/amd64
  Experimental:     false

I found out, that the problem does not occur with registry 2.7, if I set the allow-nondistributable-artifacts at the Docker client (in C:\ProgramData\docker\config\daemon.json).
Of course this is only a workaround, not a real solution.

I can confirm this, for me the workaround is to stick on 2.6.2... :|
I can't deploy that setting on all computers that pushing images.

I confirm issue with registry > 2.6.2 running on Debian stretch, from Windows server 2016 10.0.14393, with deprecated images microsoft/windowsservercore:latest and newest images mcr.microsoft.com/windows/servercore:ltsc2016	.

do we have a solution to push windows container images to registry > 2.7.0 version?

@stevvooe / @dmcgowan / @aaronlehmann: Can one of you provide some hints on how to solve this issue, so someone can pick it up from the community (I'll do it if it's not too difficult 😃).
Could potentially be a GoodFirstIssue.

I had the same problem, I fixed it by disabling validation.
docker run -d -p 5000:5000 --name registry -e REGISTRY_VALIDATION_DISABLED=true registry:2

Any progress on this? I'm getting the same issue with:
    - docker build -t $CI_REGISTRY/app/docker-images/appclient .
    - docker push $CI_REGISTRY/app/docker-images/appclient


Fix the URL parameters for querying prices for given connectionIds

Hi there, I'm testing this repository and it turns out that when calling the prices API, it no long accept the connectionIds in the query string like this:
https://tickets.oebb.at/api/offer/v1/prices?connectionIds=d5a3f2c68a85be4cb3c16be01186ee093873fb48be01b8c0e732d2021fc56f3b,785b73dfe8c85ee44509a38e21afab5ecb92f5c497e5205342bf31435e7a48d3,5fbfe048f5d486b8be0ecf839664dfa7170c9002d0

but this works
https://tickets.oebb.at/api/offer/v1/prices?connectionIds[]=d5a3f2c68a85be4cb3c16be01186ee093873fb48be01b8c0e732d2021fc56f3b&connectionIds[]=785b73dfe8c85ee44509a38e21afab5ecb92f5c497e5205342bf31435e7a48d3&connectionIds[]=5fbfe048f5d486b8be0ecf839664dfa7170c9002d0

Thank you very much for your contribution! Will review later today 🙂

Thanks for the reply, currently only my first two commits follows your original API interface, the rest changes are only for my own usage, and you might not need it.
da2da5c
5e69896

Cough I finally had time to fix the library, but I had completely forgotten about your PR. If you have the time, feel free to check out the current status of master (which works again, finally), and tell me if you miss any information.
Again - Thank you very very much for the efforts, I'm really sorry that I totally forgot about your contribution here 🤦♂

Please re-open if you're still interested in merging some feature / support for some field that I left out in the general re-write.


Tutorial - add new loss

The detailed steps how to add a new loss
add add_loss.md in /tutorials
add link to the wiki Tutorial Index

for custom functionalities, i suggest to label them as post-r1

this should be a simple tutorial to write and completely independent to code. I marked it as post-r1, but if we have time, we should add this.


Repo access blocked

<bound method Scraper.parse_json_output of <__main__.Scraper instance at 0x7f3bf4dae560>> returned in 403 Repository access blocked
reinh/dm

File "scraper.py", line 130, in parse_json_output
error_handler(e, self.parse_json_output, data)
File "/home/tinyl_server/GithubTODOScraper/scraper/utils.py", line 103, in error_handler
func(*args)
File "scraper.py", line 114, in parse_json_output
owner, repo), repo)
File "/usr/local/lib/python2.7/dist-packages/github3/github.py", line 1063, in repository
json = self._json(self._get(url), 200)
File "/usr/local/lib/python2.7/dist-packages/github3/models.py", line 130, in _get
return self._session.get(url, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests-2.8.1-py2.7.egg/requests/sessions.py", line 480, in get
return self.request('GET', url, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/github3/session.py", line 81, in request
response = super(GitHubSession, self).request(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests-2.8.1-py2.7.egg/requests/sessions.py", line 468, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests-2.8.1-py2.7.egg/requests/sessions.py", line 576, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests-2.8.1-py2.7.egg/requests/adapters.py", line 439, in send
return self.build_response(request, resp)
File "/usr/local/lib/python2.7/dist-packages/requests-2.8.1-py2.7.egg/requests/adapters.py", line 209, in build_response
response = Response()
File "/usr/local/lib/python2.7/dist-packages/requests-2.8.1-py2.7.egg/requests/models.py", line 547, in init
self.headers = CaseInsensitiveDict()
File "/usr/local/lib/python2.7/dist-packages/requests-2.8.1-py2.7.egg/requests/structures.py", line 46, in init
self.update(data, **kwargs)
File "/usr/lib/python2.7/_abcoll.py", line 540, in update
if isinstance(other, Mapping):
File "/usr/lib/python2.7/abc.py", line 132, in instancecheck
if subclass is not None and subclass in cls._abc_cache:
File "/usr/lib/python2.7/_weakrefset.py", line 75, in contains
return wr in self.data
RuntimeError: maximum recursion depth exceeded in cmp

Sent a ticket into github and 7d2d99b7b121e68602583d9cd4dca13a301ca558 added a catch-all exception.


Bug you can't avoid

This is an information to you: There will be a major problem with importing Playstation games to GG.Deals and you can do nothing to avoid it.
Problem:
If you own any of the Playstation Plus plans (in my case, Premium) and you try a game from the catalogue for a limited time (download or stream doesn't matter) or just play a game within the provided catalogue, Sony "adds" that game to your games as "owned", although you don't own it.
This is 100% reproducable, since Sony even sends you an Email with "Your transaction has been successful. You have been charged 0,00 Dollar/Euro/Yen."
So, in reality, you never bought that game and if the limited time is over or you cancel Playstation Plus, you are no longer able to play the game, since you don't really own it.
This leads to severe problems, because Sony reports such games as owned and Playnite, logically, picks up those games as owned. Thus your extension also picks up those games as owned and reports it back to GG.Deals as owned. While, in reality, you don't own it and maybe want to buy it and shop for deals. Which you will no longer see at GG.Deals.
Microsoft handles the "owning" the same way for games you try out via Gamepass, but I have to do further testing if the problem still persists if you uncheck "XCloud" within the settings of your extension.
The "Sony" Extension for Playnite doesn't differentiate between really owned games and cloud games. I guess this is because Sony itself doesn't differentiate between them and sends no divided data that can be progressed by any extension.
As I have pointed out, this is a major problem for GG.Deals and I see no way for you to work around this. You would have to contact Josef Nemec, the Playnite dev, or at least the dev of the "Sony" Playnite-Extension to work together with them. But as long as Sony doesn't change any of their practices or at least sends divided data, I see no way to make this work.

Yeah, not much we can do about it.
You can always remove/hide those games in Playnite, but it's not going to be practical when you have thousands of games, as well, it will not work on automatic sync.
I'll see, if there's anything I could do about it, but probably not :/

This applies to GamePass as well correct? In my limited experience it seems to be pulling everything down, both owned games and games that I have subscription-based access to from GP.

I suggest switching off automatic sync in addon settings. Then you can curate your Playnite library before syncing with GG.deals.
Probably gonna make automatic syncing off by default too.

Yeah I just have the console game syncs off on Playnite (Xbox, PS, Nintendo) for now as I had already manually added everything and the times that I buy things on console these days is rare.
The real value for with this tool is to scoop up all those games outside of GOG and Steam. Great job @SparrowBrain !!!

From version v2.1.1 the auto-sync for new games is not automatically enabled (affects new users). This is so new users can clean up their libraries before syncing with GG.deals.

just ad excusions to what is synced based on library and / or platform. I do not want to clean my library as adding again ton of emulated games takes lots of time. And handling emulated games by playnite is headache on its own.

@C4rl0s79 filtering by library is configurable via GG.deals settings. Emulated games should be exluded by default (Playnite library).


implement create method for mongdb-store

Evidently mongo doesn't support transactions unless you use a replica set and it's kind of a pain to set this up for testing.
If any mongo store users read this and would like this to be updated, I would appreciate your input.

I think setting up a replica set locally isn't that complicated, as you can have a single-node replica set. With proper documentation, it
should be sufficient. I'm really interested in updating these dependencies because my project is using MongoDB as the database and I'm
currently experiencing compatibility issues with tower-sessions, tower-sessions-mongodb-store, and MongoDB. What do you think would be a
solution to move forward on this as soon as possible? Thank you in advance.

@davidbayo10 why don't you implement SessionStore yourself?

Because this package solves all I need. It just needs to be adapted to the new tower-sessions API requirements

Because this package solves all I need. It just needs to be adapted to the new tower-sessions API requirements

@davidbayo10 it's a small interface which you can adapt directly if you like. It makes no sense to publish a broken implementation as I mentioned yesterday. If you'd like to address that problem then that's fine but if you want to fix your immediate problem I think you have all the tools to do so.


AdoptedResource for KMS Alias doesn't work

Describe the bug
AdoptedResources of group "kms.services.k8s.aws" and type "Alias" get stuck in a resource found error
Steps to reproduce
1- Deploy the KMS controller
2- Create a KMS key with an Alias
3- Create in the same cluster an AdoptedResource of type Alias and set the alias name in the nameOrID parameter.
Expected outcome
I expected the alias to be adopted but instead I see the AdoptedResource in this state:
    Message:  resource not found
    Status:   False

And the following logs in the controller:
2023-03-22T13:37:14.590Z	INFO	adoption.adopted-reconciler	starting adoption reconciliation	{"target_group": "kms.services.k8s.aws", "target_kind": "Alias", "namespace": "cloud-application-platform", "name": "oamapp-0022-test-kms-mig-alias", "generation": 8}
2023-03-22T13:37:14.646Z	ERROR	controller.adoptedresource	Reconciler error	{"reconciler group": "services.k8s.aws", "reconciler kind": "AdoptedResource", "name": "oamapp-0022-test-kms-mig-alias", "namespace": "cloud-application-platform", "error": "resource not found"}

Environment
Kubernetes version: 1.24
Using EKS: 1.24
AWS service targeted: KMS

I think adoption for aliases is a bit different to other resources because the AWS Alias resource uses the TargetKeyId field as the primary key, and AliasName as a secondary index. In retrospect, I think aliases should have been a property of the Key resource.
Regardless, I would try setting the alias key ID in the nameOrID field, and then under additionalKeys set name: <your-alias-name>. This should tell the controller to search for the alias within that target key's list.
The reason we need the key ID as the primary key is because we can have the same alias across different keys, so we can't uniquely identify an alias by its name.

Upon further review of the sdkFind method - which the AdoptedResource reconciler relies on to determine the spec of an object - the only field that it relies on is Spec.Name. Alias names must be unique within an account, so it's not possible for a reconciler to find two aliases with the same name but different target key IDs. In that case, we should update the generator.yaml to move the is_primary_key: true attribute from TargetKeyId to Name.

@RedbackThomson It's working now, thanks!


Usage of the "joystick_disabled_frame" and "auto_enabled_label" in GUI

What are the usage of "joystick_disabled_frame" and "auto_enabled_label" in GUI? I could not find any functions related to them.

The information is displayed when rovers are in autonomous mode.


Add 3D Animated Accordion

Prerequisites

[X] I am an Open Source Contributor
[X] I checked to make sure that this issue has not already been filed

Description
Hello, I'd like to add a 3d animated accordion please assign it to me under SWOC'24.
Screenshots
No response
Code of Conduct

[X] I follow Contributing Guidelines of this project.

Go ahead @rishicds


[[FIX]] Disallow dups in non-simple parameter list

Re-use the (hasSimpleParams) property of the funct object (recently
introduced to implement ES2016 restrictions on the "use strict"
directive) to honor the early error for duplicate parameters.

Coverage increased (+0.0003%) to 98.555% when pulling 3cf580453ff5b3d6030cd8e6b7d92dc66ef68228 on jugglinmike:non-simple-dups into e7ac6b2e167298fe069908308c9449618ff0bca7 on jshint:v2.10.0.


fix typo "requres_grad" in zero-3

@HeyangQin Looks like this is failing on formatting. Running the pre-commit command will fix this: https://github.com/microsoft/DeepSpeed/blob/master/CONTRIBUTING.md

Close it as it has been fixed by https://github.com/microsoft/DeepSpeed/pull/2750


[Bug]: email verification not working

Expected Behavior
email verification link should work and verify the email
Actual Behavior
the email gets sent out, but the link does not work:
Page not found
We are sorry that we can’t find the page, please go back or try again
the link is https://someurl.com/verify?token=sometoken
Possible Solution
should be a minor thing to add this one route
Steps to reproduce
No response
What version of vendure integration are you using?
1.1.0
What version of Node.js are you using?
12.x
What browser (and version) are you using?
chrome
What operating system (and version) are you using?
macos
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hey @macrozone
Are you sure that this is a frontend issue? Seems more like Vendure to me. Also VSF by default does not support email verification.
Maybe @michaelbromley could help as well.

@Baroshem it is definitly a frontend problem, because the verification email sends the user to a certain route on the frontend: https://someurl.com/verify?token=sometoken
the frontend should honor this route and call the corresponding api (depending on the shop-backend) and show some message or redirect the user somewhere.

I think it's just not implemented in the VSF integration yet.
Indeed, this is explicitly stated in the docs:

So I think this is more of a feature request on supporting the verification flow.

Closing this as it is out of the scope of VSF functionality.


Training a model with only normal data, gettting "RuntimeError: Found 0 abnormal images"

Describe the bug
When training a model with only normal images, getting "RuntimeError: Found 0 abnormal images "

The problem can be fixed by removing the abnormal dir in make_dataset in the folder module:
dirs = {"normal": normal_dir}
instead of
dirs = {"normal": normal_dir, "abnormal": abnormal_dir}
If I do this, training finishes successfully and I get a good model.
Should the abnormal_dir be made Optional, and the abnormal_dir left out of dirs when it is not specified? Or can this be due to issues in config/setup?
To Reproduce
Train a PatchCore model without any abnormal images.
Expected behavior
A successful training run.
Hardware and Software Configuration

OS: Ubuntu
anomalib: 0.3.7

Duplicate of #277.
@toon-vc, this is something we are currently working on. It will be available here once we merge this feature branch to main.


refactor(p2wdb): Refactored canAppend()

PR related to this trello task --> https://trello.com/c/xE47118I

:tada: This PR is included in version 1.11.0 :tada:
Your semantic-release bot :package::rocket:


"Pull from..." command no longer works

Issue Type: Bug
Steps to reproduce:

Execute command Git: Pull from...

Prompted to choose a remote; list is displayed.

On choosing upstrean, the prompt is again displayed without a list

The same behavior happens when the command is executed from the command palette, and from the More Actions context menu in the SCM pane.
VS Code version: Code 1.26.1 (493869ee8e8a846b0855873886fc79d480d342de, 2018-08-16T18:38:57.434Z)
OS version: Windows_NT x64 10.0.17134
I've seen the same behvaior on VS Code running on Linux (Fedora 28), same VS Code version.

System Info

Item
Value

CPUs
Intel(R) Pentium(R) CPU B980 @ 2.40GHz (2 x 2395)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: disabled_offnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
3.90GB (1.23GB free)

Process Argv
C:\Users\win8\AppData\Local\Programs\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions: none

Not sure why vscodebot has tagged this with integrated-terminal.

Does it work if you pick origin? Seems to work fine for me.

@joaomoreno origin seems to work fine.

So, what branches are there in the upstream remote?
git branch -r | grep upstream

No grep; I'm on Windows.
But git branch -r doesn't return any upstream branches, even though git remote -v returns upstream entries.

So, upstream has no branches, that's why quick open appears empty.


net-emissions-token-network - npm ERR! code ENOLOCAL

To run this project Prerequisite  need to defined.
npm ERR! code ENOLOCAL
npm ERR! Could not install from "node_modules/ganache-core/node_modules/web3-provider-engine/node_modules/eth-sig-util/ethereumjs-abi@git+https:/github.com/ethereumjs/ethereumjs-abi.git" as it does not contain a package.json file.
npm ERR!      /Users/kamleshnagware/.npm/_logs/2021-05-03T07_55_28_859Z-debug.log

How did you get to this error?  Did you follow the steps from https://github.com/hyperledger-labs/blockchain-carbon-accounting/blob/main/net-emissions-token-network/docs/using-the-contracts.md?


Init integration test framework

Init integration test framework and add test case

really a huge PR ! can we separate it into two? one for integration framework, the other one is test for RQA?

sure, will separate it into two PRs later. one for vendor update(PR is huge) and the other for integration test(PR is small) for RQA.

Close this PR to separate it into two


[ART-5257] Trigger publish-rpms for multi-arch

Requires https://github.com/openshift/aos-cd-jobs/pull/3316/

/lgtm