sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: Potential use after free on GDScript callable methods; methods do not capture self object when used as callables.
Labels: bug,topic:gdscript,needs testing

### Tested versions

- Reproducible in 4.4.dev2, 4.3.stable, 4.2.2.stable, 4.1.4.stable, and 4.0.4.stable 

### System information

Godot v4.3.stable (77dcf97d8) - NixOS #1-NixOS SMP PREEMPT_DYNAMIC Wed Sep 18 17:24:10 UTC 2024 - X11 - GLES3 (Compatibility)

### Issue description

Consider the following code:
```gdscript
extends Node

class Test:
	func print_hi():
		print("hi")

func other() -> Callable:
	return Test.new().print_hi

func _ready():
	other().call()
```
The expected result would be that it prints "hi" to the console; however, this instead pushes the following error:
> Attempt to call function 'null::print_hi (Callable)' on a null instance.

Calling `print_hi()` from within `other()` works fine, but the self object seems to be cleaned up after the callable is returned, resulting in calling it on a null instance. I believe the callable should capture its self object when bound like this.

### Steps to reproduce

1. Create a new function.
2. Create new object inside of that function and return a bound callable method from it.
3. Call the function from some other code, and call `.call()` or `.callv()` on the returned callable.
4. Observe the editor error, which says that the initial object is null.

### Minimal reproduction project (MRP)

[test.zip](https://github.com/user-attachments/files/17158845/test.zip)
Repository: opencv
Issue: Test_Model.Segmentation failes with the new DNN engine
Labels: bug,category: dnn

Reference: https://github.com/opencv/opencv/pull/26056

### System Information

Platform: any

### Detailed description

```
 [ RUN      ] Test_Model.Segmentation/0, where GetParam() = OCV/CPU
/home/ci/opencv/modules/dnn/test/test_common.impl.hpp:83: Failure
Expected: (normL1) <= (l1), actual: 0.0603638 vs 0
  |ref| = 7
/home/ci/opencv/modules/dnn/test/test_common.impl.hpp:86: Failure
Expected: (normInf) <= (lInf), actual: 7 vs 0
  |ref| = 7
[  FAILED  ] Test_Model.Segmentation/0, where GetParam() = OCV/CPU (447 ms)
```

### Steps to reproduce

-

### Issue submission checklist

- [X] I report the issue, it's not a question
- [X] I checked the problem with documentation, FAQ, open issues, forum.opencv.org, Stack Overflow, etc and have not found any solution
- [X] I updated to the latest OpenCV version and the issue is still there
- [X] There is reproducer code and related data files (videos, images, onnx, etc)
Repository: opencv
Issue: Test_Model.TextDetectionByDB fails with the new engine
Labels: bug,category: dnn

### System Information

Platform: any
Reference: https://github.com/opencv/opencv/pull/26056

### Detailed description

```
[ RUN      ] Test_Model.TextDetectionByDB/0, where GetParam() = OCV/CPU
unknown file: Failure
C++ exception with description "cannot create std::vector larger than max_size()" thrown in the test body.
[  FAILED  ] Test_Model.TextDetectionByDB/0, where GetParam() = OCV/CPU (113 ms)
```

### Steps to reproduce

-

### Issue submission checklist

- [X] I report the issue, it's not a question
- [X] I checked the problem with documentation, FAQ, open issues, forum.opencv.org, Stack Overflow, etc and have not found any solution
- [X] I updated to the latest OpenCV version and the issue is still there
- [X] There is reproducer code and related data files (videos, images, onnx, etc)
Repository: transformers
Issue: Add support for TimesFM
Labels: New model,Time Series

### Model description

**TimesFM** (Time Series Foundation Model) is a pretrained time-series foundation model developed by Google Research for time-series forecasting.

### Open source status

- [X] The model implementation is available
- [X] The model weights are available

### Provide useful links for the implementation

- Research Paper: https://arxiv.org/abs/2310.10688

- Authors: [Abhimanyu Das](https://arxiv.org/search/cs?searchtype=author&query=Das,+A), [Weihao Kong](https://arxiv.org/search/cs?searchtype=author&query=Kong,+W), [Rajat Sen](https://arxiv.org/search/cs?searchtype=author&query=Sen,+R), [Yichen Zhou](https://arxiv.org/search/cs?searchtype=author&query=Zhou,+Y)

- Implementation: [google-research/timesfm](https://github.com/google-research/timesfm)
  - The linked repository contains code for implementation in `jax` as well `pytorch`. To implement this in `huggingface` the `pytorch` specific code can be found at [src/timesfm/pytorch_patched_decoder.py](https://github.com/google-research/timesfm/blob/master/src/timesfm/pytorch_patched_decoder.py)

- Models Weights: [google/timesfm-1.0-200m-pytorch](https://huggingface.co/google/timesfm-1.0-200m-pytorch)
  - Although there are weights given in the repository, yet there are missing config files that are to be completed to ensure smooth loading of weights.
Repository: flutter
Issue: [go_router_builder] RouteExtension._fromState should be public to prevent non-const widgets from previous routes are rebuilding
Labels: c: new feature,package,c: proposal,P3,p: go_router_builder,team-go_router,triaged-go_router

### Use case

https://github.com/flutter/flutter/issues/144511
We have issue `Non-const widgets from previous routes are rebuilding even though new route was pushed.` and
> Since go_router is using pages api, there isn't a way for navigator to know whether page content has changed or not without rebuilding the page.

### Proposal

We can workaround this by make `pageBuilder` always return `const` and use `RouteExtension._fromState` for get route parameter instead of passthrough in `pageBuilder`.
But current `RouteExtension._fromState` is private so I always need to override, it is unnecessary, so `RouteExtension._fromState` should be public?

**Old code which lead to issue page always rebuild when routing other page because `buildPage` result is not const.**
```
@TypedGoRoute<HomeTabRoute>(path: '/home')
class HomeTabRoute extends GoRouteData {
  const HomeTabRoute({
    this.fromLogin = false,
  });

  final bool fromLogin;

  @override
  Page buildPage(BuildContext context, GoRouterState state) {
    return NoTransitionPage(
      child: HomeTabPage(fromLogin: fromLogin),
    );
  }
}
```

**New code working as expected but need to write unnecessary code**
```
@TypedGoRoute<HomeTabRoute>(path: '/home')
class HomeTabRoute extends GoRouteData {
  const HomeTabRoute({
    this.fromLogin = false,
  });

  final bool fromLogin;

  // this is unnecessary
  static HomeTabRoute fromState(GoRouterState state) =>
      $HomeTabRouteExtension._fromState(state);

  @override
  Page buildPage(BuildContext context, GoRouterState state) {
    return const NoTransitionPage(
      child: HomeTabPage(),
    );
  }
}

class HomeTabPage extends StatefulHookConsumerWidget {
  const HomeTabPage({super.key});

  @override
  ConsumerState<HomeTabPage> createState() => _HomeTabPageState();
}

class _HomeTabPageState extends ConsumerState<HomeTabPage> {
  // get parameters instead of passthrough constructor
  late final _route = HomeTabRoute.fromState(GoRouterState.of(context));
  
  @override
  Widget build(BuildContext context) {
    return Text(_route.fromLogin.toString());
  }
}
```
Repository: opencv
Issue: Misleading "'Release' build type is used by default" message
Labels: bug,category: build/install

### System Information

OpenCV version: 4.11-pre, 5.0-pre
Operating System: macOS or Windows
Compiler & compiler version: whatever VS or Xcode supplies
Python version: N/A

### Detailed description

The first thing that developers sees when trying to configure OpenCV with default parameters using CMake is the following message:

```
'Release' build type is used by default. Use CMAKE_BUILD_TYPE to specify build type (Release or Debug)
```

this is true on Linux and on Mac when 'Unix Makefiles' generator is used, but not true on Windows and MacOS when generators for Visual Studio or Xcode, respectively, are used. In projects generated by CMake for those IDEs 'Debug' binaries will be generated by default. I think, generator type should be checked for Xcode or VS before printing this message.


### Steps to reproduce

run CMake from command line or using GUI, specify "Xcode" or "Visual Studio ..." generator

### Issue submission checklist

- [X] I report the issue, it's not a question
- [X] I checked the problem with documentation, FAQ, open issues, forum.opencv.org, Stack Overflow, etc and have not found any solution
- [X] I updated to the latest OpenCV version and the issue is still there
- [ ] There is reproducer code and related data files (videos, images, onnx, etc)
Repository: kubernetes
Issue: Allowing 'watch' clients to request watch bookmarks (or optionally increasing frequency of bookmarks)
Labels: sig/api-machinery,kind/feature,triage/accepted

### What would you like to be added?

It should be possible for a client of the apiserver that is watching a resource type to 'request' a watch bookmark be sent immediately, similar to how etcd allows requesting progress of a watch (https://github.com/etcd-io/etcd/issues/9855).

Alternatively, having some kind of configurable 'higher frequency' (than the current default 1 minute) for bookmarks may be sufficient. This configuration wouldn't be needed for 99% of watch requests, so I'd propose exposing something like `?watchBookmarkFrequency=100ms&allowWatchBookmarks=true`. We'd need to expose some kind of configurable 'floor' for this value, and also look at whether we need to authorise its usage somehow too (once an analysis on scalability is completed and we've got an idea of the relative extra cost of higher-frequency bookmarks).

### Why is this needed?

When building Kubernetes apiserver proxies, to support consistent reads being served from a proxy layer that maintains its own in-memory cache (i.e. another instantiation of cacher.Cacher), we need some way to either request on demand a watch bookmark, or otherwise 'bound' the amount of time the apiserver must wait for the next bookmark to some semi-predictable frequency.

This would allow a caching proxy to serve consistent reads from its own cache, with similar assurances to what the WatchList feature provides today (e.g. "we can serve a consistent read within 100ms+(overhead)").

More generally, this also brings a closer alignment between the behaviour of etcd (sending frequent(ish) bookmarks) and watch clients, which is advantageous for proxies as we begin to use/lean on this feature (and others) more within the cacher.
Repository: excalidraw
Issue: Whitelist https://web.dev to be embedded
Labels: Embeddable

Im in process of presenting CLS issues, and i wanted to embed this video url https://web.dev/static/articles/cls/video/web-dev-assets/layout-instability-api/layout-instability2.webm

Could you whitelist it?
Repository: excalidraw
Issue: Add or remove selected objects to/from a specific(selected) group
Labels: enhancement

- There's no way to remove a selected object from a group without ungrouping all the members(destroying the group) and grouping again with the new object. If the group is a child of another group, the freed objects will get in the parent and there's no way to create a child group out of some objects of an entire group.
- We can't add an object to an entire group unless grouping the object and the group together that creates a new bigger group involving both, that is not the answer.

- The best state is to have a hierarchical structure of the groups(or all the canvas like figma) to logically move objects between groups.
- Suggested solution:
1. defining a Group type with having its individual paragraph in the JSON.
2. assigning any object to only one group instead of an array of groups
3. defining a parent ID for groups

This structure improves the process of working with groups gets us simpler and cleaner code.



```[tasklist]
### Tasks
```
Repository: react
Issue: [DevTools Bug] Cannot add node "20468" because a node with that id is already in the Store.
Labels: Type: Bug,Status: Unconfirmed,Component: Developer Tools

### Website or app

https://demo.ragflow.io/

### Repro steps

Login into the site and checkout components at any page. The error will pop up.

### How often does this bug happen?

Every time

### DevTools package (automated)

react-devtools-extensions

### DevTools version (automated)

5.3.1-ccb20cb88b

### Error message (automated)

Cannot add node "20468" because a node with that id is already in the Store.

### Error call stack (automated)

```text
at chrome-extension://fmkadmapgofadopljbjfkapdkoienihi/build/main.js:1:1172435
    at v.emit (chrome-extension://fmkadmapgofadopljbjfkapdkoienihi/build/main.js:1:1141877)
    at chrome-extension://fmkadmapgofadopljbjfkapdkoienihi/build/main.js:1:1143565
    at bridgeListener (chrome-extension://fmkadmapgofadopljbjfkapdkoienihi/build/main.js:1:1551564)
```


### Error component stack (automated)

_No response_

### GitHub query string (automated)

```text
https://api.github.com/search/issues?q=Cannot add node  because a node with that id is already in the Store. in:title is:issue is:open is:public label:"Component: Developer Tools" repo:facebook/react
```
Repository: vscode
Issue: SCM Graph - allow to multi select for e.g. cherry-pick
Labels: feature-request,scm

Would really like to use the graph view to perform bulk-operations, such as cherry-picking 🙏 

![Image](https://github.com/user-attachments/assets/e056ad28-2cf4-4e1f-86f6-549d8a08beb3)
Repository: godot
Issue: [3.x] `_input` reports `is_action_just_pressed()` multiple times per frame 
Labels: discussion,documentation,topic:input,regression

### Tested versions

Reproducible in 3.6 stable
Not reproducible in 3.5.3

### System information

All

### Issue description

If `_input()` is called multiple times per frame, then `is_action_just_pressed("some_action")` will return true every time until the next frame.

I'm not yet sure whether this affects other actions aside from mouse wheel.

UPDATE:
* It does not occur with regular mouse buttons for me, only mouse wheel
* Doesn't occur for key presses

It seems to occur with mousewheel because this creates a `press` followed by `release` in rapid succession, whereas regular mouse click is unlikely to press and release on the same frame.

I saw reported on reddit:
https://www.reddit.com/r/godot/comments/1fgi328/36_registers_mouse_wheel_event_more_than_once_35/
But wanted to create an issue for reference.

The change in behaviour is introduced by my PR #77040 which fixes #73339 .

This is a similar confusion as #80158 in master (which likely would also occur in 3.x).

### Steps to reproduce

```
func _input(_event):
	print("input frame: ", Engine.get_idle_frames())
	
	if Input.is_action_just_pressed("wheel_up"):
		print("wheel_up")
	if Input.is_action_just_pressed("wheel_down"):
		print("wheel_down")
```

Results in 3.6:
```
input frame: 6
input frame: 7
input frame: 42
wheel_down
input frame: 42
wheel_down
input frame: 57
wheel_down
input frame: 57
wheel_down
input frame: 197
input frame: 198
input frame: 199
```
Results in 3.5.3:
```
input frame: 6
input frame: 7
input frame: 42
wheel_down
input frame: 42
input frame: 57
wheel_down
input frame: 57
input frame: 197
input frame: 198
```
i.e. the `is_action_just_pressed()` would only occur once.

### Minimal reproduction project (MRP)

[Double_Input.zip](https://github.com/user-attachments/files/17160265/Double_Input.zip)

## Discussion

I am hesitating to call this a "bug", but it is a change in behaviour.

To some extent this has always been the case, because if multiple `_input()` come in on a single frame for any action that is just pressed, it will, in most cases report the same action as true multiple times. That is correct, because any action pressed *IS* considered pressed throughout the frame. 

What did happen previously though was an anomaly: on a mouse button press the press created the input with the action just pressed, and the depress created an input with the action just pressed set to false (on the same frame).

This resulted in "expected" behaviour for handling input via `_input()`, however created the bug #73339 . Fixing the bug can result in less "expected" behaviour via `_input()`, and can make input handling more complex in this case.

To make life easier we could look at an option to simply auto-cancel the action once it had been read once with `is_action_just_pressed()`, which would be friendly to both solutions to reading input.
Repository: rust
Issue: Update min supported musl to 1.2.5
Labels: O-musl,T-libs

The currently min supported musl version for the x86_64-unknown-linux-musl target is 1.2.3. The statx syscall is used to determine the file creation time. statx wasn't added to musl until version 1.2.5, so querying the file creation time leads to a `creation time is not available on this platform currently` error. For more background see the discussion.

- https://users.rust-lang.org/t/musl-and-file-creation-time/111559
Repository: vscode
Issue: Test: filter file to test could be more readable
Labels: polish,testing,test-coverage

![Image](https://github.com/user-attachments/assets/fce5c7ef-0457-469e-be1e-76401aaf5686)

I would put the file name first and description second, like we do in file picker:

![Image](https://github.com/user-attachments/assets/4979da97-949e-4236-9c50-f61dd2b7b03c)
Repository: vscode
Issue: Editor Sticky Scroll doesn't rerender when token colouring comes in after widget has been rendered
Labels: bug,editor-sticky-scroll

I'm using the tree sitter colorization which currently is slower on start up than the sticky scroll rendering. The Sticky Scroll widget does not seem to listen on tokenization colouring updates.

<img width="1301" alt="image" src="https://github.com/user-attachments/assets/6d1f089d-57e9-49b8-82c7-64f0f9c07cfa">
Repository: TypeScript
Issue: autoImportSpecifierExcludeRegexes needs a window refresh for changes to take effect
Labels: Needs Investigation

### 🔎 Search Terms

autoImportSpecifierExcludeRegexes

### 🕗 Version & Regression Information

- This changed between versions ______ and _______
- This changed in commit or PR _______
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
- I was unable to test this on prior versions because _______


### ⏯ Playground Link

_No response_

### 💻 Code

.vscode/settings.json
```json
{
  "<language>.preferences.autoImportSpecifierExcludeRegexes": ["<regex>"]
}
```

### 🙁 Actual behavior

Changes only take effect after a window reload.

### 🙂 Expected behavior

Changes to take effect immediately, same way that `autoImportFileExcludePatterns` does.

### Additional information about the issue

I mentioned this earlier on another issue (https://github.com/microsoft/TypeScript/issues/35395#issuecomment-2371010892) and I initially thought that the update I got fixed the problem, turns out it was just because that forced the window to refresh, as I later encountered the same problem once I tried adding new regexes to it.
Repository: vscode
Issue: Source Control Graph - "Go to file" opens full path and does not update explorer
Labels: feature-request,git,multi-diff-editor

Type: <b>Bug</b>

View a commit in the SCG, then on the diffs tab click on "Go to file" icon. The file opens correctly but it uses the full path and the Explorer pane does not get updated with the file location.

VS Code version: Code 1.93.1 (Universal) (38c31bc77e0dd6ae88a4e9cc93428cc27a56ba40, 2024-09-11T17:20:05.685Z)
OS version: Darwin arm64 23.6.0
Modes:

<details>
<summary>System Info</summary>

|Item|Value|
|---|---|
|CPUs|Apple M1 Pro (10 x 2400)|
|GPU Status|2d_canvas: enabled<br>canvas_oop_rasterization: enabled_on<br>direct_rendering_display_compositor: disabled_off_ok<br>gpu_compositing: enabled<br>multiple_raster_threads: enabled_on<br>opengl: enabled_on<br>rasterization: enabled<br>raw_draw: disabled_off_ok<br>skia_graphite: disabled_off<br>video_decode: enabled<br>video_encode: enabled<br>webgl: enabled<br>webgl2: enabled<br>webgpu: enabled<br>webnn: disabled_off|
|Load (avg)|2, 2, 2|
|Memory (System)|32.00GB (0.78GB free)|
|Process Argv|--crash-reporter-id f1725485-c9d4-4a1e-b3f7-bcbac394420d|
|Screen Reader|no|
|VM|0%|
</details><details><summary>Extensions (12)</summary>

Extension|Author (truncated)|Version
---|---|---
vscode-eslint|dba|3.0.10
githistory|don|0.6.20
prettier-vscode|esb|11.0.0
vscode-docker|ms-|1.29.3
remote-wsl|ms-|0.88.4
powershell|ms-|2024.2.2
vscode-thunder-client|ran|2.25.8
vscode-yaml|red|1.15.0
code-spell-checker|str|3.0.1
code-spell-checker-spanish|str|2.3.4
vscode-gradle|vsc|3.16.4
volar|Vue|2.1.6


</details><details>
<summary>A/B Experiments</summary>

```
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
9c06g630:31013171
a69g1124:31058053
dvdeprecation:31068756
dwnewjupyter:31046869
impr_priority:31102340
nativerepl1:31139838
refactort:31108082
pythonrstrctxt:31112756
flightc:31134773
wkspc-onlycs-t:31132770
wkspc-ranged-t:31125599
cf971741:31144450
pme_test_c:31118331
fje88620:31121564
iacca1:31144502

```

</details>

<!-- generated by issue reporter -->

![Screenshot 2024-09-27 at 11 27 45](https://github.com/user-attachments/assets/623cbc2f-578a-4891-896a-6691d1acd38b)