sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: EditorPlugin: undo method is not called if object parameter can't be casted to Resource
Labels: bug,topic:editor,confirmed,topic:plugin

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
 v3.2.2.stable.official


**OS/device including version:**
Windows 10 x64, two different systems tested


**Issue description:**
After adding a undo method for a `UndoRedo` action, it will only be called on undoing the action if the object is a builtin type, or extends `EditorPlugin`

I'm wondering if that is intended behaviour, in that case we should probably add something to the documentation.

Example:
`undo_test.gd`:
```gdscript
tool
extends EditorPlugin

func _input(event):
	var just_pressed = event.is_pressed() and not event.is_echo()
	
	if Input.is_key_pressed(KEY_F12) and just_pressed:
		test_action()
		return true

func test_action():
	var undo = get_undo_redo()
	
	undo.create_action("test action")
	
	var action_ref = ActionRef.new()
	var action_plugin = ActionPlugin.new()
	
	undo.add_do_method(action_ref, "output", "do method in ActionRef")
	undo.add_do_method(action_plugin, "output", "do method in ActionPlugin")
	undo.add_undo_method(action_ref, "output", "UNDO method in ActionRef")
	undo.add_undo_method(action_plugin, "output", "UNDO method in ActionPlugin")
	
	undo.commit_action()
```
`action_plugin.gd`:
```gdscript
tool

class_name ActionPlugin
extends EditorPlugin

func output(message):
	print(message)
```
`action_ref.gd`:
```gdscript
tool

class_name ActionRef
extends Reference

func output(message):
	print(message)
```

Then, when executing the action (press F12), both output methods are called. When undoing the action, only the undo method for the instance which extends `EditorPlugin` is called.

![image](https://user-images.githubusercontent.com/59559612/92466273-28e7e880-f1d0-11ea-8303-36d455c24c11.png)


**Steps to reproduce:**
- create a `UndoRedo` action
- create a new instance `object` that is neither builtin nor `extends EditorPlugin`
- add undo method via `undo.add_undo_method(object, method, params)`
- commit the `UndoRedo` action
- when undoing the action, the registered undo method on the `object` instance will not be called

**Minimal reproduction project:**
https://github.com/lolligerjoj/undo_test.git
Repository: rust
Issue: [ER] catch some wrong copy_from_slice statically
Labels: C-enhancement,A-lints

I think we should catch some `slice::copy_from_slice` length mismatch in basic cases like:

```rust
fn main() {
    let mut a = [0_u32; 10];
    let b = [0_u32; 20];
    a.copy_from_slice(&b);
}
```
Repository: ant-design
Issue: Cascader 级联选择器希望增加virtual选项
Labels: 💡 Feature Request,Inactive

- [ ] I have searched the [issues](https://github.com/ant-design/ant-design/issues) of this repository and believe that this is not a duplicate.

### What problem does this feature solve?
当数据量比较大时，打开级联选择器还是很卡，希望像select组件一样，可以增加virtual选项

### What does the proposed API look like?
virtual={true}


<!-- generated by ant-design-issue-helper. DO NOT REMOVE -->
Repository: youtube-dl
Issue: Request for site support: kan.org.il
Labels: site-support-request

<!--

######################################################################
  WARNING!
  IGNORING THE FOLLOWING TEMPLATE WILL RESULT IN ISSUE CLOSED AS INCOMPLETE
######################################################################

-->


## Checklist

<!--
Carefully read and work through this check list in order to prevent the most common mistakes and misuse of youtube-dl:
- First of, make sure you are using the latest version of youtube-dl. Run `youtube-dl --version` and ensure your version is 2020.09.06. If it's not, see https://yt-dl.org/update on how to update. Issues with outdated version will be REJECTED.
- Make sure that all provided video/audio/playlist URLs (if any) are alive and playable in a browser.
- Make sure that site you are requesting is not dedicated to copyright infringement, see https://yt-dl.org/copyright-infringement. youtube-dl does not support such sites. In order for site support request to be accepted all provided example URLs should not violate any copyrights.
- Search the bugtracker for similar site support requests: http://yt-dl.org/search-issues. DO NOT post duplicates.
- Finally, put x into all relevant boxes (like this [x])
-->

- [x] I'm reporting a new site support request
- [x] I've verified that I'm running youtube-dl version **2020.09.06**
- [x] I've checked that all provided URLs are alive and playable in a browser
- [x] I've checked that none of provided URLs violate any copyrights
- [x] I've searched the bugtracker for similar site support requests including closed ones


## Example URLs

<!--
Provide all kinds of example URLs support for which should be included. Replace following example URLs by yours.
-->

- Single video: https://www.kan.org.il/Item/?itemId=74658
- Single video: https://www.kan.org.il/Item/?itemId=74657
- Playlist: https://www.kan.org.il/program/?catId=1636

## Description

<!--
Provide any additional information.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

[kan.org.il](https://www.kan.org.il/) is the official website of Kan11, an Israeli free-to-air state media and news television channel.
Downloading from them doesn't violate copyright since everything there is available for free, and it isn't against their tos.
Repository: godot
Issue: Can't download assets from the asset library if their Content-Length hasn't been declared (like for GitHub's uncached ZIP archives)
Labels: bug,topic:editor,confirmed,topic:assetlib,topic:network

*Bugsquad edit (keywords for easier searching): HTTP chunk too big*

**OS/device including version:**
<!-- Specify GPU model and drivers if graphics-related. -->
Windows 10

**Issue description:**
<!-- What happened, what was expected, and what went wrong. -->
Went to the templates in Godot 3.2.2 stable, and tried to download the TPS demo, it downloaded a little over 60 mb and the download progress hit 100%, shortly after that it just said that there was an error as seen on the screen shot...

I've tried plenty of other templates and no such issues were found with downloading those so seems like something specific with this one to me.. However the console error might suggest the issue is with the engine, but as it seems only this one is affected I'm reporting here as it could very well be an issue with the demo itself doing sth wrong.


**Screenshots of issue:**
<!--
This section is optional.
Drag in an image, or post an image with a link in the form of:
![Alt Text Here](https://pbs.twimg.com/media/DW5AJnZVAAM1805?format=jpg)
-->
![TPS](https://user-images.githubusercontent.com/50302479/92474853-c8ab7380-f1dc-11ea-84b4-c9908af395c0.png)
![TPS error](https://user-images.githubusercontent.com/50302479/92475673-11176100-f1de-11ea-84f7-70969589628f.png)
Repository: ant-design
Issue: Slider Maker position
Labels: Inactive

- [x] I have searched the [issues](https://github.com/ant-design/ant-design/issues) of this repository and believe that this is not a duplicate.

### What problem does this feature solve?
In case of horizontal slider while the user is using the slider the marks gets hidden below the cursor. This can create confusion in some cases. This can be solved by having marks above the slider.
Similarly, In case of vertical slider there might be some use case to have slider on the left side of the line. However as per the current API there is no may to change the position of marker in horizontal it is always on the bottom in vertical slider it is always on the right side

### What does the proposed API look like?
Slider component should have a props to allow markers to be on the other side of the line
This can be achieved the in the following way
1. Using type marksPosition = 'left' | 'right' | 'above' | 'below', right, left only work in case of vertical slider and above below for horizontal marker
2. Using a boolean flag alternateMarkPosition true will move marker to alternate position


<!-- generated by ant-design-issue-helper. DO NOT REMOVE -->
Repository: terminal
Issue: Change pointer when hovering over a link
Labels: Area-UserInterface,Product-Terminal,Issue-Task

<!-- 
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨

I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:
1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

# Description of the new feature/enhancement
When the user hovers over a link and presses ctrl, change the pointer to the 👆 icon
<!-- 
A clear and concise description of what the problem is that the new feature would solve.
Describe why and how a user would use this new functionality (if applicable).
-->
Repository: terminal
Issue: Add hyperlink support to conhost
Labels: Product-Conhost,Area-Interaction,Issue-Task

<!-- 
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨

I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:
1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

# Description of the new feature/enhancement
Currently, conhost supports OSC 8 sequences but does not support clicking/opening links. We should have conhost support this. 

<!-- 
A clear and concise description of what the problem is that the new feature would solve.
Describe why and how a user would use this new functionality (if applicable).
-->
Repository: terminal
Issue: Include hyperlinks in HTML copy
Labels: Area-Output,Product-Terminal,Issue-Task

<!-- 
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨

I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:
1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

# Description of the new feature/enhancement
When a user does a HTML copy of text to the clipboard, hyperlink information should be copied over as well. 
<!-- 
A clear and concise description of what the problem is that the new feature would solve.
Describe why and how a user would use this new functionality (if applicable).
-->
Repository: PowerToys
Issue: Frame Draw and 60Hz+ Displays
Labels: Idea-New PowerToy,Product-Tweak UI Design

There's seems to be and rendering method that limits the redraw to 30hz (determined by debugger). Windows 10 will honor the Display Adapter's setting regardless of the configuration (120hz in my display's case). This is likely done to save battery, but would like an option for desktops to draw at the system's configured resolution and refresh rate.
Repository: rust
Issue: Overly specific PartialEq impl for Option and Result
Labels: T-libs-api,C-bug

#### I tried this code:

```rust
// This works:
let s1: String = "foo".to_owned();
let s2: &str = "foo";
s1 == s2;

// But these don't:
let o1: Option<String> = Some("foo".to_owned());
let o2: Option<&str> = Some("foo");
o1 == o2;

let r1: Result<String, ()> = Ok("foo".to_owned());
let r2: Result<&str, ()> = Ok("foo");
r1 == r2;
```

([playground](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=1eb53b21200dd1036a1febd42d5cd8f7))

This is because `PartialEq` impls are `#[derive]`d for `Option` and `Result`, which gives them conservative bounds.

Instead of current

```rust
impl<T> PartialEq<Option<T>> for Option<T>
where
    T: PartialEq<T>
```

I'd expect

```rust
impl<A, B> PartialEq<Option<B>> for Option<A>
where
    A: PartialEq<B>
```

and likewise, for `Result`

```rust
impl<T1, T2, E1, E2> PartialEq<Result<T2, E2>> for Result<T1, E1>
where
    T1: PartialEq<T2>,
    E1: PartialEq<E2>,
```

Note that this is similar to what `Vec<T>` already implements:

```rust
impl<A, B> PartialEq<Vec<B>> for Vec<A>
where
    A: PartialEq<B>
```

### Meta

`rustc --version --verbose`:
```
rustc 1.45.2 (d3fb005a3 2020-07-31)
binary: rustc
commit-hash: d3fb005a39e62501b8b0b356166e515ae24e2e54
commit-date: 2020-07-31
host: x86_64-unknown-linux-gnu
release: 1.45.2
LLVM version: 10.0
```
Repository: flutter
Issue: Flutter tools on iOS should show a message for untrusted personal certificate apps
Labels: platform-ios,tool,a: quality,t: xcode,P3,team-ios,triaged-ios

Flutter running with a personal account just shows a generic 

```
Could not build the precompiled application for the device.

It appears that your application still contains the default signing identifier.
Try replacing 'com.example' with your signing id in Xcode:
  open ios/Runner.xcworkspace

Error launching application on iPhone.
```

Though behind the scenes, in verbose, the issue is

`
error: The operation couldn’t be completed. Unable to launch com.example.io.flutter.demo.gallery because it has an invalid code signature, inadequate entitlements or its profile has not been explicitly
trusted by the user.
`

We should catch and show a summary message in those cases. Works after setting Settings->General->Profiles & Device Management to trusting the new certificate.
Repository: flutter
Issue: Linux build exposes user's build tree path (which may contain username, etc.) in the rpath of `lib*_plugin.so`
Labels: tool,platform-linux,a: desktop,a: build,P2,team-linux,triaged-linux

## Problem description
When building a Flutter app on Linux with plugins that need to create a dynamic library (e.g. `url_launcher`), flutter will create a `lib<plugin_name>_plugin.so` under `build/linux/release/bundle/lib`. However, `readelf -d lib<plugin_name>_plugin.so` shows that its rpath (RUNPATH) includes the user's local build tree path.

## Impact
The build tree path may contain username and other information that a user might not want to share. This may cause privacy concerns of the developer when distributing the app to public and it also adds unnecessary and incorrect rpath to the shared lib.

It will not affect running the app on Linux though, since the rpath of the app main binary is correct and the `lib<plugin_name>_plugin.so` will be loaded by the main binary.

## Problem cause
During building, CMake will set the rpath of the binary/shared lib to some path under the build tree. And it will remove it (by default) or set it to CMAKE_INSTALL_RPATH when installing. However this resetting of rpath will only be done if the binary is installed as a target. 

In `linux/CMakeLists.txt`, the plugin libraries are installed as files by
```
if(PLUGIN_BUNDLED_LIBRARIES)
    install(FILES "${PLUGIN_BUNDLED_LIBRARIES}"
    DESTINATION "${INSTALL_BUNDLE_LIB_DIR}"
    COMPONENT Runtime)
endif()
```
Therefore rpath of those libraries will not be reset. 

## A proof of concept fix 
* Modify `FILES` above into `TARGETS`
* In the generated `linux/flutter/generated_plugins.cmake` file, within the loop of `foreach(plugin ${FLUTTER_PLUGIN_LIST})`
  * Change `list(APPEND PLUGIN_BUNDLED_LIBRARIES $<TARGET_FILE:${plugin}_plugin>)` into `list(APPEND PLUGIN_BUNDLED_LIBRARIES ${plugin}_plugin)`.
  * Add  `set_target_properties(${plugin}_plugin PROPERTIES INSTALL_RPATH "")`
* Run `cmake` and `ninja` manually (to avoid overriding generated files) 

This will fix the rpath in the `lib<plugin_name>_plugin.so`.

## Temp Workaround
One can use `chrpath` or `patchelf` to change the rpath by hand.
Repository: terminal
Issue: Add auto-suggestion to the Command-Palette wt arguments
Labels: Issue-Feature,Area-UserInterface,Product-Terminal,Priority-3,Area-CmdPal

<!-- 
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨

I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:
1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

# Description of the new feature/enhancement
Starting with https://github.com/microsoft/terminal/pull/7293, users can type out wt commands into the Command Palette. Please provide auto-suggestion to users as they type out commands in the Command-Palette (the wt commands). This makes it easier for users to start using the wt command arguments and also speeds up the process for users to execute an action within Windows Terminal.
<!-- 
A clear and concise description of what the problem is that the new feature would solve.
Describe why and how a user would use this new functionality (if applicable).
-->

# Proposed technical implementation details (optional)
Extend the current auto-suggestions to include wt arguments.
<!-- 
A clear and concise description of what you want to happen.
-->
Repository: pytorch
Issue: `Undefined symbol: torch::kNearest` when building App iOS
Labels: triaged,oncall: mobile,module: ios

## 🐛 Bug

XCode clang linker produces errors such as

`Undefined symbol: torch::kNearest`

when using

`torch::nn::functional::InterpolateFuncOptions( )`

## To Reproduce

Steps to reproduce the behavior:

1. clone https://github.com/pytorch/ios-demo-app and build
2. add this code somewhere in [here](https://github.com/pytorch/ios-demo-app/blob/master/HelloWorld/HelloWorld/HelloWorld/TorchBridge/TorchModule.mm)


```cpp
auto options = torch::nn::functional::InterpolateFuncOptions( );
```
3. `#import <torch/nn/functional.h>`
4. add "HEADER SEARCH PATHS" in XCode Build Settings:
`"$(PODS_ROOT)/LibTorch/install/include/torch/csrc/api/include/`

linker will complain about `torch::kNearest`

## Expected behavior

Linker must have no errors using torch::nn::functional classes and methods.

## Environment

 - PyTorch Version 1.6.0
 - OS macOS 10.15 Catalina
 - How you installed PyTorch (`conda`, `pip`, source): cocoa pods
 - Build command you used (if compiling from source): XCode

## Additional context

I have found only this temporary workaround.

In file
https://github.com/pytorch/pytorch/blob/master/torch/csrc/api/include/torch/nn/options/upsampling.h#L71

change 
`TORCH_ARG(mode_t, mode) = torch::kNearest;`
to
`TORCH_ARG(mode_t, mode) = torch::enumtype::kNearest;`

and it will work.
Repository: pytorch
Issue: [FX] Schema normalization tooling 
Labels: module: internals,triaged,module: fx

When a PyTorch program is represented in FX, callsites to ops, modules, and methods appear simply as a tuple of (reference to target, args, kwargs). The representation itself is agnostic to the schema of the op/module/method or overloads of such. It would be nice to have infrastructure that takes the aforementioned tuple, matches it against the known schemas in the system, and returns a normalized representation of the callsite (e.g. default args populated) for the purpose of transformation passes (in Python)

cc @ezyang @bhosmer @smessmer @ljk53 @bdhirsh @SherlockNoMad @ailzhang
Repository: rust
Issue: format_args! is slow
Labels: I-slow,C-feature-request,T-libs,A-fmt

Like molasses in the Antarctic.

As a consequence, so is any method which depends on its Arguments, like `{fmt, io}::Write::write_fmt`. The microbenchmarks in [this issue about `write!`'s speed][slow_write_macro] demonstrate that merely running the same arguments through `format_args!` and then `write_fmt`, even if it's just a plain string literal without any formatting required, produces a massive slowdown next to just feeding the same through `fmt::Write::write_str` or `io::Write::write_all`.

Unfortunately, `write!`, `format!`, `println!`, and other such macros are a common feature of fluent Rust code. Rust promises a lot of zero-cost abstractions, and on a scale from "even better than you could handwrite the asm" to "technically, booting an entire virtual machine is zero cost if you define the expression as booting a virtual machine..." this is currently "not very". Validating and formatting strings correctly can be surprisingly complex, which is going to increase with features like [implicit named arguments in format_args!][implicit_args], so we can expect increasing speed here may be challenging. However, this should be possible, even if it might require extensive redesign.

### Multiple Problems, Multiple Solutions
- `format_args!`'s internal machinery in the Rust compiler can likely be improved.
- Consumers of Arguments, such as `fmt::{format, write}` and `{fmt, io}::Write::write_fmt`, can be reviewed for runtime performance.
- Macros downstream of `format_args!` often are invoked to do something simple that does not require extensive formatting and can [use the pattern-matching feature of `macro_rules!` to special-case simple patterns][special_case_format] to side-step `format_args!` when it's not needed. This will increase the complexity of those macros and risks breakage if done incautiously, but could be a big gain in itself.

Unfortunately some of these cases may run up against complex situations with types, trait bounds, and method resolutions, because e.g. both `io::Write` and `fmt::Write` both exist and `write!` needs to "serve" both. Fortunately, this is exactly the sort of thing that can benefit from the recent advances in const generics, since it's a lot of compile-time evaluation that could benefit from interacting with types (as opposed to being purely syntactic like macros), and in the future generic associated types and specialization may be able to minimize breakage from type issues as those features come online, so it's a good time to begin reviewing this code.

### Related issues and PRs
- #75742 (and #75894)
- #75301 (and #75358)
- #52804
- #10761 

[slow_write_macro]: https://github.com/rust-lang/rust/issues/10761
[implicit_args]: https://rust-lang.github.io/rfcs/2795-format-args-implicit-identifiers.html
[special_case_format]: https://github.com/rust-lang/rust/issues/52804#issuecomment-408628574