sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: AudioStreamOGGVorbis ignores the loop property and uses the import setting due to the saved resource not being able to store metadata
Labels: bug,topic:editor,confirmed,topic:audio,topic:import

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
<!-- Specify commit hash if using non-official build. -->
3.2.3.stable

**OS/device including version:**
<!-- Specify GPU model, drivers, and the backend (GLES2, GLES3, Vulkan) if graphics-related. -->
Manjaro Linux, 5.8.11-1-MANJARO

**Issue description:**
<!-- What happened, and what was expected. -->
When using a .ogg, the `loop` property on AudioSreamOGGVorbis is only acknowledge by the editor, and the game uses instead the loop option set when importing the file.

**Steps to reproduce:**
Add a AudioStreamPlayer with a .ogg file.

Play the audio on the editor. It will loop depending on the AudioSreamOGGVorbis `loop` property.

Play the audio on the game. It will loop depending on the `loop` value in the import settings.

**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
[OGGLoopPreferenceBug.zip](https://github.com/godotengine/godot/files/5354897/OGGLoopPreferenceBug.zip)


**Related to:**
 #40694

**May be related to:**
#40359
Repository: godot
Issue: Audio is stopped when changing the properties of AudioStream
Labels: bug,topic:gdscript

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
<!-- Specify commit hash if using non-official build. -->
3.2.3.stable

**OS/device including version:**
<!-- Specify GPU model, drivers, and the backend (GLES2, GLES3, Vulkan) if graphics-related. -->
Manjaro Linux, 5.8.11-1-MANJARO

**Issue description:**
<!-- What happened, and what was expected. -->
If an AudioStreamPlayer is playing and you change any of the properties of the stream the audio stops.

**Steps to reproduce:**
* Add an AudioStreamPlayer with any sound.
* Change any of the properties of the AudioStream.
* The audio stops.

**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
[AudioStreamPropertiesStopsAudio.zip](https://github.com/godotengine/godot/files/5355429/AudioStreamPropertiesStopsAudio.zip)
Repository: go
Issue: time: create ticker with instant first tick - rebooted
Labels: NeedsInvestigation,FeatureRequest

I have the same issue as https://github.com/golang/go/issues/17601, but the issue was closed and locked [with this reason](https://github.com/golang/go/issues/17601#issuecomment-307906597):

> This doesn't seem to rise to the level of new API. You don't have to "copy the section of code underneath the first <-t.C", you just have to put the code in its own function and call that function once at the start and once after each timeout interval. That seems clear enough.

However, "call the function once at the start" does not work when you are not in control of when the function is called.

Consider this clumsy implementation of a rate limiting in `rateLimited`, vs. the much more concise and elegant one in `rateLimitedImmediate`:

```Go

package main

import (
	"fmt"
	"sync"
	"time"
)

var interval = time.Second

type rateLimited struct {
	rateLimiter <-chan time.Time
	f           func() string
}

func (r *rateLimited) do() string {
	<-r.rateLimiter
	return r.f()
}

type rateLimitedImmediate struct {
	mu          sync.Mutex
	rateLimiter <-chan time.Time
	f           func() string
}

func (r *rateLimitedImmediate) do() string {
	r.mu.Lock()
	defer r.mu.Unlock()
	defer func() { r.rateLimiter = time.After(interval) }()
	<-r.rateLimiter
	return r.f()
}

func main() {
	r := &rateLimited{
		rateLimiter: time.Tick(interval),
		f:           func() string { return "result" },
	}

	fmt.Println(r.do())
	fmt.Println(r.do())
	fmt.Println(r.do())

	r2 := &rateLimitedImmediate{
		rateLimiter: time.After(0), // start immediately
		f:           func() string { return "result" },
	}

	fmt.Println(r2.do())
	fmt.Println(r2.do())
	fmt.Println(r2.do())
}
```

Just that the nice implementation in `rateLimited` cannot start immediately.

I am not in control of calling the function myself, the library client does that.

So I second the need for a stdlib solution like proposed in the original issue, and hope the issue can be re-opened (or discussed here).
Repository: flutter
Issue: flutter upgrade command is not working as expected
Labels: tool,a: quality,P3,team-tool,triaged-tool

first things first, thanks for this awesome framework,
My flutter version: 
```fl --version
Flutter 1.22.1 • channel beta • https://github.com/flutter/flutter.git
Framework • revision f30b7f4db9 (22 hours ago) • 2020-10-08 10:06:30 -0700
Engine • revision 75bef9f6c8
Tools • Dart 2.10.1
```
i'm using macbook air, when i issue flutter upgrade command it initiated the upgrade process, at same time i issues flutter create app_name in my vscode terminal, vscode terminal prompt asked me to wait until the flutter startup lock is removed,
all good so far.
when it came to upgrade engine it redirected its output to vscode terminal which should not happen
![Screenshot 2020-10-09 at 7 57 41 PM](https://user-images.githubusercontent.com/60929919/95597331-17faf300-0a6c-11eb-9c91-075558edc7db.png)
![Screenshot 2020-10-09 at 7 58 00 PM](https://user-images.githubusercontent.com/60929919/95597359-234e1e80-0a6c-11eb-9cac-48abfed4866d.png)
after this in vscode terminal 
it throwed me the error like this
![Screenshot 2020-10-09 at 7 59 32 PM](https://user-images.githubusercontent.com/60929919/95597424-3a8d0c00-0a6c-11eb-9e8e-d29ccc59cd69.png)

finally after more or less 10 minutes in my regular terminal upgrade process is continued
![Screenshot 2020-10-09 at 8 08 32 PM](https://user-images.githubusercontent.com/60929919/95597666-8cce2d00-0a6c-11eb-846b-02baa72af935.png)

 and finished without any error.
but what i expected is output of upgrade process should stay in one terminal and it should redirect its output to some other terminal
Repository: pytorch
Issue: Spawn wrapper should catch BaseException, not Exception
Labels: module: multiprocessing,triaged,better-engineering

https://github.com/pytorch/pytorch/blob/9fb8e33a5bdbe36084bb6182e9f72e1aac14e464/torch/multiprocessing/spawn.py#L22

Not all exceptions inherit from `Exception`, for example, pytest's `Skipped` exception. Throwing `Skipped` in a subprocess isn't caught and rethrown by the multiprocessing module.
Repository: PowerToys
Issue: Documentation for debugging modules
Labels: Area-Runner,Issue-Docs,Area-Build

<!-- Briefly describe which document needs to be corrected and why. -->

## 📝 How to debug (C#) modules such that symbols are loaded correctly

Folks who are familiar with C# WPF applications may want to contribute to a module like Fancy Zones, but out of the box it's not clear how to run the module with symbols loaded for debugging. If you try to run Fancy Zones by itself you're shown a message that it should not be run as a standalone application.

Though with some playing around you might find you can start up the `runner` project and open the module through settings. However at this point, by default, no symbols are loaded for breakpoints within Fancy Zones.

It would be a great to have some documentation around the right way to debug this (and other) modules.

**Note:** I am currently in the boat of having not figured out how to do this yet, so I'll take a quick version of how to accomplish this if one exists!
Repository: flutter
Issue: [Flutter Driver] tap on a Sliver throws an exception
Labels: c: crash,tool,f: scrolling,t: flutter driver,has reproducible steps,P2,found in release: 3.3,found in release: 3.6,team-tool,triaged-tool

Flutter driver throwing the error:
`await driver.tap(find.byType('RenderSliverStickyHeader'));`

Error thrown:
```
DriverError: Error in Flutter application: Uncaught extension error while executing tap: type 'RenderSliverStickyHeader' is not a subtype of type 'RenderBox' in type cast
  #0      _HitTestableFinder.filter.<anonymous closure> (package:flutter_test/src/finders.dart:504:52)
  #1      _SyncIterator.moveNext (dart:core-patch/core_patch.dart:165:25)
  #2      Iterable.isEmpty (dart:core/iterable.dart:440:33)
  #3      Iterable.isNotEmpty (dart:core/iterable.dart:447:27)
  #4      FlutterDriverExtension._waitForElement.<anonymous closure> (package:flutter_driver/src/extension/extension.dart:278:51)
  #5      FlutterDriverExtension._waitUntilFrame (package:flutter_driver/src/extension/extension.dart:263:19)
  #6      FlutterDriverExtension._waitForElement (package:flutter_driver/src/extension/extension.dart:278:11)
  <asynchronous suspension>
  #7      FlutterDriverExtension._tap (package:flutter_driver/src/extension/extension.dart:385:41)
  #8      FlutterDriverExtension.call (package:flutter_driver/src/extension/extension.dart:216:53)
  #9      BindingBase.registerServiceExtension.<anonymous closure> (package:flutter/src/foundation/binding.dart:547:32)
  <asynchronous suspension>
  #10     BindingBase.registerServiceExtension.<anonymous closure> (package:flutter/src/foundation/binding.dart)
  #11     _runExtension (dart:developer-patch/developer.dart:84:23)
```

The code from package:flutter_test/src/finders.dart that results in the error:
```
@override
  Iterable<Element> filter(Iterable<Element> parentCandidates) sync* {
    for (final Element candidate in parentCandidates) {
      final RenderBox box = candidate.renderObject as RenderBox;
      assert(box != null);
      final Offset absoluteOffset = box.localToGlobal(alignment.alongSize(box.size));
      final HitTestResult hitResult = HitTestResult();
      WidgetsBinding.instance!.hitTest(hitResult, absoluteOffset);
      for (final HitTestEntry entry in hitResult.path) {
        if (entry.target == candidate.renderObject) {
          yield candidate;
          break;
        }
      }
    }
  }
```

As this Widget is a Sliver, it cannot be cast to a RenderBox. I did not work much with Slivers, I am therefore unsure how this is correctly implemented for them. I know that they work with geometry, but I don't know how to get the absoluteOffset for the calculation.

If someone could point me to the right direction, I would create a PR for this
Repository: go
Issue: crypto/x509: add ability to reload root certificates
Labels: NeedsDecision,FeatureRequest,Proposal-Crypto

**Problem description:**

As it can be seen on this line, root certificates loaded only once during the lifetime of the application - 
https://github.com/golang/go/blob/9e5b1367cb45931b3730027f30afc03bda224583/src/crypto/x509/root.go#L16

This creates a problem when new root certificates are added. In our case, it happens on a regular basis when clients add intermediate/root certificates to the system via a separate component and then all other components that run in separate processes are expected to make use of them. This is currently not possible.

**Workaround:**

We are currently re-implemented Root Certificate loading logic by cutting and pasting the code from this library into our codebase and create our own certPool() for every request that requires the ca-chain refresh.

**Proposed solutions:**

1. Add ReloadRootCertificates() call and call it as needed - backward compatible but still the source of gotchas.
2. Do not use the global state and singleton and parse the certificate chain on each request - may have an unexpected performance impact.
3. Make the method for creating the root certificate pool public.
4. Add root certificate cache expiry timeout that can go all the way to 0 triggering cert chain reload on each request.
Repository: pytorch
Issue: test_cat_cuda (__main__.TestTensorExprFuser) fails
Labels: module: tests,triaged,shadow review

## 🐛 Bug

`test_cat_cuda (__main__.TestTensorExprFuser)` fails

## To Reproduce

Steps to reproduce the behavior:

```
python test/test_tensorexpr.py TestTensorExprFuser.test_cat_cuda
```

```
======================================================================
ERROR: test_cat_cuda (__main__.TestTensorExprFuser)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/pytorch/pytorch/torch/testing/_internal/common_utils.py", line 828, in wrapper
    method(*args, **kwargs)
  File "test_tensorexpr.py", line 1048, in test_cat_cuda
    self._test_cat('cuda')
  File "test_tensorexpr.py", line 1036, in _test_cat
    x = warmup_and_run_forward(traced, *values)
  File "test_tensorexpr.py", line 42, in warmup_and_run_forward
    results = f(*args)
  File "/opt/pytorch/pytorch/torch/testing/_internal/common_utils.py", line 127, in prof_func_call
    return prof_callable(func_call, *args, **kwargs)
  File "/opt/pytorch/pytorch/torch/testing/_internal/common_utils.py", line 124, in prof_callable
    return callable(*args, **kwargs)
RuntimeError: UNSUPPORTED DTYPE
----------------------------------------------------------------------
```
## Expected behavior

no fail

## Environment

1.8.0a0+ 59414b3

## Additional context

A new failure found last night. https://github.com/pytorch/pytorch/pull/45788 might be related @eellison 


cc @mruberry @VitalyFedyunin @ezyang
Repository: go
Issue: cmd/gofmt: unexpected indentation in chained calls
Labels: NeedsInvestigation

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.15.2 darwin/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=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/cbandy/Library/Caches/go-build"
GOENV="/Users/cbandy/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/cbandy/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/cbandy/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/cbandy/.local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/cbandy/.local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
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 -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/bw/_dxdzsmx6h1ffcgp6rvfh55c0000gn/T/go-build600613638=/tmp/go-build -gno-record-gcc-switches -fno-common"
</pre></details>

### What did you do?

Ran `gofmt -s -w` on the following code: https://play.golang.org/p/CXEb-91Y6Mm

```go
package main

func f(a interface{}) func(b interface{}) {
	return func(interface{}) {}
}

type s struct{}

func (s) m(a interface{}) func(b interface{}) {
	return func(interface{}) {}
}

func main() {
	f("a")("b")
	f("a")(
		"b",
	)
	f("a")([]string{
		"b",
	})
	f(
		"a",
	)(
		"b",
	)

	s{}.
		m("a")("b")
	s{}.
		m("a")(
			"b",
		)
	s{}.
		m(
			"a",
		)(
			"b",
		)
	s{}.
		m("a")([]string{
			"b",
		})
}
```

### What did you expect to see?

(I expected no changes to the indentation because) I expected closing brackets to be indented the same amount as the line of their matching open brackets.

### What did you see instead?

On some of the chained method calls, the closing brackets are indented one level less. Starting on L29, the output is:

```go
	s{}.
		m("a")(
		"b",
	)
	s{}.
		m(
			"a",
		)(
		"b",
	)
	s{}.
		m("a")([]string{
		"b",
	})
```

```diff
--- main.go.orig
+++ main.go
@@ -28,16 +28,16 @@
 		m("a")("b")
 	s{}.
 		m("a")(
-			"b",
-		)
+		"b",
+	)
 	s{}.
 		m(
 			"a",
 		)(
-			"b",
-		)
+		"b",
+	)
 	s{}.
 		m("a")([]string{
-			"b",
-		})
+		"b",
+	})
 }
```
Repository: node
Issue: node only using half of the available memory
Labels: memory

* **Version**: 12.19.0
* **Platform**: Debian Buster
* **Subsystem**:

### What steps will reproduce the bug?
1. Start a container with limited memory (something like `docker run --rm -it --memory=1G --memory-swap=1G node:12.19.0-buster-slim bash`
2. Inspect heap size (`node -e 'const v8 = require("v8"); console.log(v8.getHeapStatistics());'`)

### How often does it reproduce? Is there a required condition?
100%

### What is the expected behavior?
`node` would use all available memory

### What do you see instead?
`node` is only using half of the memory, so we either need to give the container twice as much memory as we really want it to use (which is problematic) or also specify `--max-old-space-size` in addition to controlling the container which means we have to keep 2 different settings in sync and that's a big pain

### Additional information
It appears that it was [changed here](https://github.com/nodejs/node/pull/25576#issuecomment-455737693), and only using half of memory makes sense when running on a system with multiple process, but when running in a docker container, using all of the memory allocated to the container makes more sense
Repository: go
Issue: testing: run Cleanup if SIGINT occurs
Labels: NeedsInvestigation

Currently, `testing.T.Cleanup` and `testing.B.Cleanup` are only run if the test and all of its subtests have finished running. This means it won't run if the test is stopped with a SIGINT (e.g. Ctrl+C). There are several things that may be in such a `Cleanup` function that would be helpful to run regardless of whether or not the test fully completed.

This will be especially unfortunate in the case of fuzzing (http://golang.org/s/draft-fuzzing-design). Take for example:

```
func FuzzFoo(f *testing.F) {
	f.Cleanup(func() {
		// Check for things like stray go routines, etc
		// Delete tmp resources that may be left behind
	})
	f.Fuzz(func(t *testing.T, b []byte) {
		// some code being fuzzed
	})
}

```

It may be a very common use case that someone could run `go test -fuzz FuzzFoo` and give a SIGINT after a few hours. If no crash occurs, there would be no way to run any cleanup code or do any checks after fuzzing completed since the fuzzing runs in an infinite loop.

I would like to expand the behavior of Cleanup to not only run if tests finished, but also run in the case of a SIGINT. This could of course be special cased to fuzzing, but I can see good reasons why it should be a part of unit and benchmark testing as well (e.g. if the test hangs for some reason).

/cc @jayconrod @bcmills @FiloSottile
Repository: godot
Issue: [Bullet] HingeJoint deprecation message prints even for default property setup
Labels: bug,topic:editor,confirmed,topic:physics

**Godot version:**
Godot 3.2.3 Stable, Windows 10.

**OS/device including version:**
Windows 10, 64-bit.

**Issue description:**
When a HingeJoint node is created and one of the two nodes for the joint are setup, the following deprecation warning is always printed:

    set_param: The HingeJoint parameter 0 is deprecated

The code for this deprecation print can be found [here](https://github.com/godotengine/godot/blob/3.2/modules/bullet/hinge_joint_bullet.cpp#L120) for Godot 3.2.

This issue was first discovered by Dominic on the Godot Forums: [Can I disable the "HingeJoint parameter deprecated" warning?](https://godotforums.org/discussion/24231/can-i-disable-the-hingejoint-parameter-deprecated-warning#latest)

**Steps to reproduce:**

Make a HingeJoint node and assign one of its two Node slots to a valid node. This will cause the message to be printed, even if no properties on the joint were changed.

**Minimal reproduction project:**
[HingeJoint_DeprecationWarning_Project.zip](https://github.com/godotengine/godot/files/5357267/HingeJoint_DeprecationWarning_Project.zip)
Repository: youtube-dl
Issue: Add support for crowdcast.io
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.20. 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.20**
- [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.crowdcast.io/e/gresham-mars


## Description

Requires registration, which is free.

I assume this will be fairly simple to automate. Clicking on the play button in a browser with Chrome developer tools turned on, I can currently search the list of URLs that gets spat out for 'm3u8', copy the URL to a command like `ffmpeg -i https://blahblahblah Gresham-Mars.mp4` and it just Does The Right Thing.
Repository: rust
Issue: Code bloat from monomorphization of methods applied to arrays
Labels: C-enhancement,I-heavy,A-fmt,A-array

The following talks about `Debug::fmt`, but is likely true of a lot more methods applied to arrays.
Take the following code:
```
fn main() {
    let a = [1, 2, 3];
    let b = [1, 2, 3, 4];
    let c = [1, 2, 3, 4, 5];
    let d = [1, 2, 3, 4, 5, 6];
    println!("{:?}", a);
    println!("{:?}", b);
    println!("{:?}", c);
    println!("{:?}", d);
}
```
Compiled with --release, you end up with 4 different functions for `core::array::<impl core::fmt::Debug for [T; N]>::fmt` (BTW, shouldn't they each have their N replaced by the actual number?). To add insult to injury, each of these have their loops unrolled, so the longer the array, the larger the function, up to 59 (!).
Almost ironically, building with `-C opt-level=1` generates 4 functions that create a slice and a separate, common, `fmt` function for it.
This is one of these cases where you'd probably want a `#[inline(never)]` at the call site in `core::array::<impl core::fmt::Debug for [T; N]>::fmt` if that were possible.