sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: TypeScript
Issue: When using the MSBuild package with Blazor library extra content files are added to nupkg
Labels: Visual Studio

# Bug Report

When using the `Microsoft.Typescript.MSBuild` 4.2.1-rc to pack a Blazor Library project for nuget package, the compiled javascript files are added to the `content` and `contentFiles` folders of the nuget package, not just to `staticWebAssets` that Blazor build picks up.

`GetTypeScriptOutputForPublishing` MSBuild target seems to be the source of errors, if the content of that target is commented out, the nuget package does not have the above problem.

*UPDATE*

The real sideeffects of this coming to the surface when the build nuget package is consumed in another Blazor project and being built in VS 2019 16.x. As the JS files are showing up in the consuming project as project items and they should not. The worse thing is that if you put a tsconfig.json into the Logging package no matter if it is a `<None....` item, that will also be packed and will cause compilation failure in the consuming project as there will be no source files to compile (that's how we started to notice this problem).

### Repro

1. Clone https://github.com/BlazorExtensions/Logging
2. dotnet pack
3. Observe the contents of the nupkg file

### 🔎 Search Terms

msbuild, nuget, blazor

### 🙁 Actual behavior

See in summary above.

### 🙂 Expected behavior

Nuget package should not have those extra files.
Repository: rust
Issue: [ER] Basic bit twiddling methods
Labels: T-libs-api

A simple enhancement request. I think in line with the idea of Rust design to give low level control, to offer an ergonomic API that reduces the chances for bugs while keeping code efficient, and at the same time avoiding too many special operators, I think Rust stdlib could add few simple functions to work on bits of integral values (u8, i8, ...etc, usize, isize and char):

```rust
x.set_bit(7);
x.set_bit(1 ..= 3);
x.reset_bit(7);
x.reset_bit(1 .. 5);
x.invert_bit(7);
x.invert_bit(2 ..);
if x.is_bit_set(7) {} // For simplicity this accepts only single indexes.
if x.is_bit_reset(7) {}
```

Such functions can be implemented in user level code, in just a handful of lines, but they are twiddly and it's boring to implement them many times (especially the interval ones), so having them in the stdlib saves bugs, time and allows to focus on more interesting code :-)
Repository: transformers
Issue: [Tensor Parallelism] Megatron-LM to transformers
Labels: Tensor Parallel,WIP

# 🚀 Feature request

Splitting the discussion that started here: https://github.com/huggingface/transformers/pull/10301#issuecomment-782917393 to add the potential future feature of transformers and it's Tensor Parallelism (Horizontal Model Parallelism) - for bigger context please see [Parallelism notes](https://github.com/huggingface/transformers/issues/9766).

Let's start with important clarification: MP can mean many different things
1. Vertical MP - slice the layers vertically - one or more full layers placed on each gpu = Vertical MP - in which case VertMP is a simple version of PP with chunks=1
2. Horizontal MP - slice the layers horizontally - place a slice of a full model on each gpu - Example Megatron-LM

At the moment I think it's only Megatron-LM that implements Horizontal MP. @anthon-l has ported that model to `transformers`, except the Horizontal MP parts, since currently `transformers` doesn't yet have support for it. There is already naive Vertical MP in t5 and gpt2 thanks to @alexorona's work, I ported Bart too but it's unmerged, and there is an ongoing effort to figure out how to implement the Pipeline. All these will have to co-operate with each other and also share common tools.

@anton-l [started sharing](https://github.com/huggingface/transformers/pull/10301#issuecomment-782917393) what needs to be done to make that important feature available - and then down the road potentially make it available to other (all?) `transformers` models.

@anton-l, the floor is yours.
Repository: rust
Issue: rustdoc: Get rid of shared mutable state in `Context`
Labels: C-cleanup,T-rustdoc,E-mentor,E-medium

`Context` currently has a [`.shared`] field, which is shared mutable state.
It would be nice if we could redesign the code such that we don't have shared
mutable state.

One idea I have for accomplishing this is passing `&mut SharedContext` in
wherever it's needed, instead of getting it out of `Context`.

[`.shared`]: https://doc.rust-lang.org/nightly/nightly-rustc/rustdoc/html/render/struct.Context.html#structfield.shared


<!-- TRIAGEBOT_START -->

<!-- TRIAGEBOT_ASSIGN_START -->

<!-- TRIAGEBOT_ASSIGN_DATA_START$${"user":"Wardenfar"}$$TRIAGEBOT_ASSIGN_DATA_END -->

<!-- TRIAGEBOT_ASSIGN_END -->
<!-- TRIAGEBOT_END -->
Repository: pytorch
Issue: [TensorExpr] Add IR Verifier
Labels: module: bootcamp,triaged,NNC

## 🚀 Feature
IR Verifier could check if a TE Stmt is well formed. Examples of things we could verify:
* Each `Allocate` stmt has the `Free` counterpart and precedes it
* Variables referred to in an expression are defined in the scope of that expression
* All index expressions are inbound
* Store to an element of a buffer could not go after its Load
* There is no "bad" loop-carried dependencies: all buffer accesses either don't have inter-iteration dependencies or are reductions
Repository: pytorch
Issue: [JIT-autodiff] batch_norm isn't differentiable
Labels: oncall: jit

## Motivation

Hi, @Krovatkin ,

We have a use case of JIT training on ResNet50 to make fusion work both in forward and backward path. So far, our fusable pattern is BN_FWD + ReLU and ReLU + BN_BWD.

But batch_norm is disabled in `autodiff`. 
https://github.com/pytorch/pytorch/blob/9529ae377629f4253af0720cb01f09db6fcad4c9/torch/csrc/jit/runtime/symbolic_script.cpp#L992.

cc @gmagogsfm
Repository: rust
Issue: Account for type and trait errors due to incorrectly wrapped  types
Labels: C-enhancement,A-diagnostics,T-compiler,A-suggestion-diagnostics,D-papercut,D-terse

[Given the following code](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=fc6054ff31f0d741e529808e0a618902):

```rust
trait CanFoo {}
trait IsFoo {}
impl<T: IsFoo> CanFoo for T {}
struct Foo;
impl IsFoo for Foo {}

fn foo(x: impl CanFoo) {}
fn bar() -> Result<Foo, ()> { Err(()) }

fn main() {
    foo(bar());
}
```

The current output is:

```
error[E0277]: the trait bound `Result<Foo, ()>: IsFoo` is not satisfied
  --> src/main.rs:11:9
   |
7  | fn foo(x: impl CanFoo) {}
   |                ------ required by this bound in `foo`
...
11 |     foo(bar());
   |         ^^^^^ the trait `IsFoo` is not implemented for `Result<Foo, ()>`
   |
   = note: required because of the requirements on the impl of `CanFoo` for `Result<Foo, ()>`
```

[For the case of a type error](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=ec9f8aa7a8d7f035eff5fc262de59c05), the output is less verbose and clear enough:

```
error[E0308]: mismatched types
  --> src/main.rs:11:9
   |
11 |     foo(bar());
   |         ^^^^^ expected struct `Foo`, found enum `Result`
   |
   = note: expected struct `Foo`
                found enum `Result<Foo, ()>`
```

But ideally, for both of these cases we should be looking at available fields of all of the type's variants to detect cases where the found value is either incorrectly wrapped and could potentially be accessed, _or_ if it just needs to be wrapped with the expected new-type or an appropriate variant. For cases involving `Result` in particular we should be able to suggest `?` if the enclosing `fn` has an appropriate return type. For all other cases where fields are not directly accessible, we _could_ look at the available associated methods with a suitable return type, but we've tried doing this in the past and the suggestions weren't helpful enough. If we were to go down that route again, we should introduce an attribute to annotate `fn`s we are comfortable suggesting to minimize false positives.

_Taken from subthreads originated at https://twitter.com/mcclure111/status/1363586990944444416_
Repository: flutter
Issue: Google map on iOS does not exactly centre on the screen
Labels: platform-ios,p: maps,package,a: layout,has reproducible steps,P2,found in release: 1.22,found in release: 2.0,found in release: 2.3,team-ios,triaged-ios

Google map on iOS does not exactly centre on the screen. The screenshots below demonstrate the difference between iOS & Android. The view has been created using a `Stack` with `Google Map` and `PositionedWidget.fill` widgets. The positioned widget contains a black dot at the centre of the screen.

The red tint is the background for the PositionedWidget.fill widget, which shows its extent is full screen.

On Android there is an exact overlap. This makes it hard for implementing features like ones you see on ride hailing/food delivery applications where a user selects the centre location as their pickup/drop. Having this discrepancy will lead to different locations being selected that what the user intended to do.

Using: `google_maps_flutter: ^1.2.0`

# Screenshots
### iOS
![Simulator Screen Shot - iPhone 11 Pro Max - 2021-02-22 at 11 15 58](https://user-images.githubusercontent.com/3646627/108668722-ae9eec80-7501-11eb-8b4f-1ce54d188ae4.png)

### Android
![Screenshot_1613972802](https://user-images.githubusercontent.com/3646627/108668791-d4c48c80-7501-11eb-92df-3375b089d030.png)

### flutter doctor -v
```
[✓] Flutter (Channel stable, 1.22.6, on Mac OS X 10.15.4 19E287 darwin-x64, locale en-IN)
    • Flutter version 1.22.6 at /Users/abhaysood/flutter
    • Framework revision 9b2d32b605 (4 weeks ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/abhaysood/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /Applications/Android Studio 2.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.10.1

[✓] IntelliJ IDEA Ultimate Edition (version 2020.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin installed
    • Dart plugin version 203.6912
```
Repository: go
Issue: cmd/go: improve diagnostic and documentation for invalid //go:embed file name
Labels: NeedsFix

<!--
Please answer these questions before submitting your issue. Thanks!
For questions please use one of our forums: https://github.com/golang/go/wiki/Questions
-->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.16 linux/amd64
</pre>

### Does this issue reproduce with the latest release?
Yes

### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
$ go env
GO111MODULE="off"
GOARCH="amd64"
GOBIN="/root/go/bin"
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/root/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/root/go"
GOPRIVATE=""
GOPROXY="https://goproxy.io"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.16"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build162353995=/tmp/go-build -gno-record-gcc-switches"
</pre></details>

### What did you do?
my directory `test` tree is:
```
├── 00|00.test
└── test

0 directories, 2 files
```
but the fs.ReadDir() only returns single file: `test`.
the code is:
```
//go:embed test
var efs embed.FS

func main() {
    efps, _ := efs.ReadDir("test")
    for _, efp := range efps {
        fmt.Println(efp.Name())
    } 
}
```
<!--
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
-->



### What did you expect to see?

like os.ReadDir(), want two files return(return all files).

### What did you see instead?
Just return single file: `test`.
Repository: godot
Issue: 'ERROR: _scan_new_dir: Cannot go into subdir' and other errors when deleting and creating directory
Labels: bug,topic:editor,needs testing

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
3.2.4 RC 2


**OS/device including version:**
Windows 10


**Issue description:**
[Video of the errors](https://youtu.be/-TpkPK9OlV8)

I got spammed with these errors in the console window even after stopping the running scene:
`ERROR: _scan_new_dir: Cannot go into subdir`
`ERROR: Failed to get modified time for <PATH>`.

This occurred after calling a function which:
1. deleted all files in a directory
2. deleted the empty directory
3. asserted that the directory was deleted
4. created a new directory at the same path
5. failed the assertion:
```
				var __error = __dir.make_dir(__export_folder_path)
				assert(__error == OK)
```

The function is also supposed to generate images from a spritesheet, and save them into this newly-created folder.
I also got spammed with these error messages in the debugger:
`editor/editor_file_system.cpp:1802 - Error importing <PNG PATH>`
`Cannot open file from path <PNG IMPORT FILE PATH>`
`core/io/image_loader.cpp:56 - Error opening file <PNG PATH>`

And finally at the bottom, there were these error messages in the debugger:
`modules/gdscript/gdscript_tokenizer.cpp:1129 - Condition "tk_rb[ofs].type != TK_IDENTIFIER" is true. Returned StringName()`
`Cannot navigate to 'res://Images/Generated/Characters' as it has not been found in the file system!`

However, the directory actually **was** created, because I can see it in the `FileSystem` browser and in Windows Explorer.

**Steps to reproduce:**
Unfortunately, I can't reproduce.
I reverted to a previous version of the project, and ran the exact same code again. This time, I didn't get the error.
I then switched back to the version AFTER I got this error, and ran the code again. Still no error.
So I don't know what happened. This is actually the second time that simply reverting to an older version fixed the problem for no apparent reason, even though the code was not actually changed between versions.

But if you want to test, just run the project, and wait until it hits a breakpoint in `Main.gd`.
If there were no errors, it will begin importing the new images that were generated.
Otherwise, you should get the errors I mentioned above.
The function in question is `generate_Character_images()` in the `G.gd` singleton.

**Minimal reproduction project:**
(Can't reproduce, but here is the project file anyways.)
[_scan_new_dir Bug.zip](https://github.com/godotengine/godot/files/6020847/_scan_new_dir.Bug.zip)
Repository: create-react-app
Issue: Why is there no help or version for react-scripts binary..?
Labels: needs triage

![image](https://user-images.githubusercontent.com/31458531/108688642-267b1000-751e-11eb-92c4-f154291f9d92.png)

Why these flags are omitted from the react-scripts binary..?
Repository: pytorch
Issue: pickle is a security issue
Labels: module: pickle,module: serialization,triaged,module: hub,topic: security

## 🚀 Feature
We need to do something with it.

## Motivation

Pickle is a security issue that can be used to hide backdoors. Unfortunately lots of projects keep using `torch.save` and `torch.load`.

## Pitch

* make `pytorch.load` use pickle only as a serialization format, use an own virtual machine (https://github.com/CensoredUsername/picklemagic can be helpful) for processing pickle files that will do only allowed operations in pytorch itself in a completely controlled way instead of relying on pickle machinery.
* replace with ONNX
* deprecate `pytorch.load`, `pytorch.save`
* remove `pytorch.save`/make it save into ONNX

## Alternatives
* support pickle via a VM indefinitely.

cc @mruberry @nairbv @NicolasHug @vmoens @jdsgomes @ailzhang
Repository: pytorch
Issue: Custom scatter function for DataParallel
Labels: triaged,enhancement,module: data parallel

## 🚀 Feature
Allow for passing an optional argument `scatter_fn` to `nn.DataParallel` that defaults to `nn.parallel.scatter_gather.scatter`. This function would have a signature match `nn.parallel.scatter_gather.scatter` and would be called instead when provided.

## Motivation
I've encountered issues with the base scatter function when passing custom tensor data objects with `DataParallel`. These include:
- Passing a namedtuple of tensors: items were dispatched individually to replicas when they were intended to replicated on each.
- Passing a custom collection of tensors that is a subclass of `dict`: the constructor was not matching that of `dict`, causing bugs.

## Pitch
This change would allow for more flexibility towards custom tensor collections in use with `DataParallel`. In my experience, such custom objects become strongly desirable when working on a large-scale project.
Repository: deno
Issue: Reenable "local_sources_not_cached_in_memory" test
Labels: tests,chore

Disabled in https://github.com/denoland/deno/pull/9568 as it was quite flaky on CI. Most likely caused by race condition in worker implementation.
Repository: go
Issue: x/net/webdav: PUT handler does not support conditional requests using If-... headers
Labels: NeedsInvestigation

<!--
Please answer these questions before submitting your issue. Thanks!
For questions please use one of our forums: https://github.com/golang/go/wiki/Questions
-->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.15.6 linux/amd64
</pre>

### Does this issue reproduce with the latest release?

yes

### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/jbe/.cache/go-build"
GOENV="/home/jbe/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/jbe/go/pkg/mod"
GONOPROXY="redacted"
GONOSUMDB="redacted"
GOOS="linux"
GOPATH="/home/jbe/go"
GOPRIVATE="redacted"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build221351055=/tmp/go-build -gno-record-gcc-switches"
</pre></details>

### What did you do?

<!--
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
-->
1. Create a resource using put.
2. Overwrite the resource with another put.
3. Make a conditional GET request using a condition which is invalidated by step 2.
4. Make a conditional PUT request using the same unsatisfied condition used in step 3.

Example using `If-Unmodified-Since`:  https://play.golang.org/p/yf4YRknvXRe

Example using `If-Match`: https://play.golang.org/p/a6nl_v-R_vE


Note: It was necessary to run the examples twice on the playground, the first run timed out during compilation.


These types of conditional requests are useful to avoid the lost update problem without using WebDAV locks.

### What did you expect to see?

After running any of the provided examples I expect both GET and PUT requests failing with `412 Precondition Failed` -- producing the following output:

```
412 Precondition Failed
412 Precondition Failed
```

### What did you see instead?
The second request (PUT) succeeded:

```
412 Precondition Failed
201 Created
```
Repository: godot
Issue: h_offset and v_offset for PathFollow3D dont produce intended effect with rotation turned off
Labels: bug,topic:core

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
`3bb628d8fe26abad258a222a5e22b644e2b73a01`
<!-- Specify commit hash if using non-official build. -->


**OS/device including version:**
`Windows 10 20H2`
<!-- Specify GPU model, drivers, and the backend (GLES2, GLES3, Vulkan) if graphics-related. -->


**Issue description:**
The h_offset and v_offset for PathFollow3D node are supposed to add a displacement along and perpendicular to the path respectively. However, if we turn off the rotation, it just displaces it along the x and y axes. Also, the h_offset and v_offset give kinda weird results even when rotation is turned on.
<!-- What happened, and what was expected. -->


**Steps to reproduce:**
Add a Path3D node and PathFollow3D node as it's child. Define a Path for Path3D and try changing the h_offset and v_offset when rotation is turned off. 

**Minimal reproduction project:**
[hv_offset.zip](https://github.com/godotengine/godot/files/6021521/hv_offset.zip)

<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
Repository: node
Issue: What should we do with unknown import assertions?
Labels: esm

Soon, V8 will support [import assertions](https://github.com/tc39/proposal-import-assertions).

I'm opening this discussion so we make a decision about what Node.js is going to do when there is an `import` statement with assertions that we do not know about.

I have three cases in mind:
- `import json from "./package.json" assert { type: "json" }` where we support loading JSON but have not implemented the `type: "json"` assertion yet.
- `import json from "./package.json" assert { type: "jssssson" }` where we support loading JSON with the `type: "json"` assertion, but the user made a typo.
- `import something from "./file.js" assert { foo: "bar" }` where we just don't know what a `foo` assertion is.

I suppose we have the choice between three possibilities:
- Ignore the unknown assertion / behave the same way as if it wasn't there.
- Throw an error / fail the import
- Emit a warning

@nodejs/modules
Repository: flutter
Issue: Add a widget that always returns false from hitTest but still lets its subtree participate in hit testing
Labels: c: new feature,framework,f: scrolling,f: gestures,c: proposal,P3,team-framework,triaged-framework

## Use case

I've needed to have full screen scrollables - either PageView or CustomScrollView - as the frontmost widget in a Stack that also need to allow passing gestures through to the widgets behind, for example when there is a gap between the elements in the CustomScrollView. 

With the options currently available, this is not possible, since all scrollviews use Scrollable, which is hardcoded as `HitTestBehavior.opaque`.

Currently to make this work I've had to copy the contents of Scrollable and ScrollView or PageView to make my own version just so I could set the HitTest behavior to translucent.

## Proposal

I propose to make HitTestBehavior configurable in Scrollable as well as the built-in ScrollViews and PageViews that use it.
Repository: bitcoin
Issue: Allow for rescan using block filters with pruned node
Labels: Feature,Wallet

After #15946 it should be possible to do a rescan using block filters with a pruned node. Blocks that are not stored could be requested from peers.