sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: AnimationTree in inherited scene does not work
Labels: bug,topic:core,topic:animation

<!-- 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. -->
V3.2.2.stable.official (opened via steam)

**OS/device including version:**
<!-- Specify GPU model, drivers, and the backend (GLES2, GLES3, Vulkan) if graphics-related. -->
GLES2. Have tried this on GLES3, too.

**Issue description:**
<!-- What happened, and what was expected. -->
When I played my project that contains a scene that inherits from another scene which has an animation tree, that animation tree will not play any animation when called.

**Specifically**: An inherited scene's AnimationTree's parameter/state/playback seem to keep switching into a blank AnimationTreeStateMachine. This is visually shown by a reload icon appearing at that var, and as the result of the newly-created AnimationTreeStateMachine, the AnimationTree cannot change the animation state when called.

**Steps to reproduce:**
1. Create new scene (let it be called Base)
2. Add AnimationPlayer and AnimationTree and fill the appropriate details (such as adding a new AnimationTreeStateMachine to its Tree Root).
3. Add another Node for the AnimationPlayer to interact with.
4. Attach script to AnimationTree to call for its state change.
5. Create New Inherited Scene from Base (let it be called Inherited)
6. Add button to have AnimationTree change state when pressed.
7. Play Scene (Inherited)

**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
[inherited_animtree.zip](https://github.com/godotengine/godot/files/4877619/inherited_animtree.zip)

Note: There is no Main Scene; Base and/or Inherited have to opened from the FileSystem dock.
Note2: I have added print statements in AnimationTree script to show that the function is called, but doesn't change state.
Repository: youtube-dl
Issue: please add support for tv.lonelyplanet.com
Labels: site-support-request

## Checklist

- [X] I'm reporting a new site support request
- [X] I've verified that I'm running youtube-dl version **2020.06.16.1**
- [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

- Single video: https://tv.lonelyplanet.com/play/#/tv/837/season/17/episode/1/1.-building-england:-part-2
- Single video: https://tv.lonelyplanet.com/tv/planet-food/4/#episode-1
- Playlist: https://tv.lonelyplanet.com/tv/globe-trekker/17/


## Description

tv.lonelyplanet.com lets you watch copies of their own show (Lonely Planet/Globe Trekker) and some other travel related programmes. 
Some of the shows may be watched just by registering an email address, others require an account and payment but all the links given above can be watched just by registering your email address to create a free account.
Repository: TypeScript
Issue: Bundling typescript using webpack: the request of a dependency is an expression (+possible fix)
Labels: Suggestion,Experience Enhancement

**TypeScript Version:**  3.9.6 and 4.0.0-dev.20200706

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:**
webpack request of a dependency expression

**Code**

```ts
import ts from 'typescript';
console.log(ts)
```

**Expected behavior:**
Bundle successfully, and without warnings, using webpack.

**Actual behavior:**
Bundles successfully, but a warning is shown:
```
WARNING in ./node_modules/typescript/lib/typescript.js 5710:41-60
Critical dependency: the request of a dependency is an expression
```
**Playground Link:** <!-- A link to a TypeScript Playground "Share" link which demonstrates this behavior -->
N/A

**Related Issues:** <!-- Did you find other bugs that looked similar? -->
Couldn't find any that talked about this issue

**Suggested fix:**

The warning is printed due to the following dynamic require call:
```ts
require: function (baseDir, moduleName) {
    try {
        var modulePath = ts.resolveJSModule(moduleName, baseDir, nodeSystem);
        return { module: require(modulePath), modulePath: modulePath, error: undefined };
    }
    catch (error) {
        return { module: undefined, modulePath: undefined, error: error };
    }
}
```
Rather than calling require directly, create a helper function:
```ts
function requireModule(requestingModule, specifier) {
    return requestingModule.require(specifier)
}
```
And call it the following way:
```ts
return { module: requireModule(module, modulePath), modulePath: modulePath, error: undefined };
```

The resulting code will behave 1:1 in Node, while not triggering any warnings in bundlers trying to resolve these dynamic calls.
Repository: rust
Issue: Parameter usage is not properly propagated through trait aliases
Labels: T-compiler,C-bug,F-trait_alias,requires-nightly

It seems to me that both `S1` and `S2` should compile:

```rust
#![feature(trait_alias)]

// Works
struct S1<'a, T: 'a, I: Iterator<Item = &'a T> + Clone> {
    field: I,
}

trait SpecialIterator<'a, T: 'a> = Iterator<Item = &'a T> + Clone;
// Doesn't work
struct S2<'a, T: 'a, I: SpecialIterator<'a, T>> {
    field: I,
}
```
[Playground](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=27cb8b2de9ddc65a62f440e1b907ec39). On `1.46.0-nightly (2020-07-05 2753fab7ce3647033146)`.
Repository: pytorch
Issue: Simplify Adam Optimizer
Labels: module: optimizer,triaged,enhancement

<!-- A clear and concise description of the feature proposal -->
At the moment in the Adam optimizer the exponential moving averages are decoupled from the bias correction, as per the original paper. However, it is possible to combine these operations into a single update step.

An unbiased exponential moving average with parameter `b` can be computed using the following update rule:

`b_t = 1-(1-b)/(1-b^t)`
`u_t = b_t * u_(t-1) + (1-b_t)*x_t`

This eliminates the bias correction operation for the second-moment estimate, so should be slightly faster and use less memory, and would be a simple change.

I'm happy to implement this.


cc @vincentqb
Repository: TypeScript
Issue: Object member completions are offered from all union constituents even after union has been discriminated
Labels: Bug,Domain: Completion Lists

<!-- 🚨 STOP 🚨 STOP 🚨 STOP 🚨

Half of all issues filed here are duplicates, answered in the FAQ, or not appropriate for the bug tracker. Even if you think you've found a *bug*, please read the FAQ first, especially the Common "Bugs" That Aren't Bugs section!

Please help us by doing the following steps before logging an issue:
  * Search: https://github.com/Microsoft/TypeScript/search?type=Issues
  * Read the FAQ: https://github.com/Microsoft/TypeScript/wiki/FAQ

Please fill in the *entire* template below.
-->

<!--
Please try to reproduce the issue with the latest published version. It may have already been fixed.
For npm: `typescript@next`
This is also the 'Nightly' version in the playground: http://www.typescriptlang.org/play/?ts=Nightly
-->
**TypeScript Version:**  3.9.5 (probably more)

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:**

**Code**
```ts
type Choice = {
    kind: number, x: 42
} | {
    kind: undefined, y: 42
}

const c: Choice = {kind: undefined, X_OR_Y: 42} // X_OR_Y is a placeholder
```

**Expected behavior:**
When you use autocompletion to replace `X_OR_Y` by `x` or `y`, the editor should suggest `y`, only:
<img width="305" alt="Screenshot 2020-07-06 at 13 08 01" src="https://user-images.githubusercontent.com/1295054/86587535-152ec480-bf8a-11ea-96fa-23212c5e40a1.png">


**Actual behavior:**
The editor suggests both `x` or `y`:
<img width="315" alt="Screenshot 2020-07-06 at 13 11 32" src="https://user-images.githubusercontent.com/1295054/86587624-40b1af00-bf8a-11ea-9e70-b97e0cef4b8c.png">

Of course, if you choose `x`, the compiler will then complain about the wrongly typed assignment.

**Playground Link:** <!-- A link to a TypeScript Playground "Share" link which demonstrates this behavior -->
[Playground](https://www.typescriptlang.org/play?#code/C4TwDgpgBAwgFgewJYGNoF4oG8BQV9QDWSAdgCYBcUJArgLYBGEATgDRQAeVALAEw4BfKAB9seAsXJUa5CADNSEMuxA9+AnDhQISAZ2BQUVeMjRRMWSZSgyy8xcqgANAPoB5AEouAmmoFA)

**Related Issues:** <!-- Did you find other bugs that looked similar? -->
Repository: youtube-dl
Issue: Site support request: livesets.online
Labels: site-support-request

## Checklist
- [x] I'm reporting a new site support request
- [x] I've verified that I'm running youtube-dl version **2020.06.16.1**
- [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 audio: https://www.livesets.online/markus_schulz_presents_-_global_dj_broadcast_(29_may_2014).shtml (has a link to the site below)
- Single audio: https://www.livesets.online/stream/markus_schulz_presents_-_global_dj_broadcast_(29_may_2014).html (plays the stream, even if takes 10-20s to load)
- Playlist: n/a
## Description

I believe this could be fairly easy. the first site links to the second, the second contains

    <script>start_set('https://stream.livesets.stream/playlist/1b82e26f1e7430d285343d966b0a2349/markus_schulz_presents_-_global_dj_broadcast_(29_may_2014)/0/stream.m3u8','256kbps cbr - markus schulz presents   global dj broadcast 29 may 2014')</script>

and

    youtube-dl https://stream.livesets.stream/playlist/1b82e26f1e7430d285343d966b0a2349/markus_schulz_presents_-_global_dj_broadcast_(29_may_2014)/0/stream.m3u8

works fine already.

Main tasks would be finding the playlist URL and extracting a proper file name from its URL.
Repository: node
Issue: readable[Symbol.asyncIterator]().next() on stream causes immediate exit (no error)
Labels: readline,stream

* **v14.5.0**:
* **Darwin helmholtz 19.5.0 Darwin Kernel Version 19.5.0: Tue May 26 20:41:44 PDT 2020; root:xnu-6153.121.2~2/RELEASE_X86_64 x86_64**:
* **Subsystem**:

### What steps will reproduce the bug?

When `next()` is called in the program below, nothing else gets run. list.txt and list2.txt are just two line files `line1\nline2` and `line3\nline4`

This repl.it shows the issue.
https://repl.it/repls/FrightenedPastLogin#index.js

```
const fs = require("fs");
const readline = require("readline");

async function main() {
  const linereader = readline.createInterface({
    input: fs.createReadStream("./list.txt"),
  });

  for await (const s of fs.createReadStream("./list2.txt")){}
  console.log(await linereader[Symbol.asyncIterator]().next())

  // nothing below here gets run
  console.log("=============");
  for await (let s of linereader) {
    console.log(s);
  }
  console.log("=============");

  return "test";
}

main()
  .then((t) => console.log(t))
  .catch((e) => console.log(e));

```

### How often does it reproduce? Is there a required condition?
Always

### What is the expected behavior?
The rest of the program should execute. The following should print

```
node test.js
{ value: 'line1', done: false }
=============
line2
=============
test
```


### What do you see instead?
Nothing:
```
node test.js

```

Possibly related to #33792  #34035?
Repository: vscode
Issue: Allow DAP progress notifications to be shown more prominently than the status bar
Labels: feature-request,debug

Extracting this from https://github.com/microsoft/vscode/issues/101405#issuecomment-652358374 since it's not really the same issue:

I changed Dart/Flutter over to using DAP progress notifications (from previous custom messages that showed full notifications for progress). I've had a few comments suggesting the status bar notifications may not be obvious enough requesting the full notifications are brought back.

I think progress messages tend to fall into two fairly distinct categories:

- things like "Analyzing...", where we want to show something is happening, but it's not directly related to what the user is doing
- things like hot reload/launching/building - something specifically started by the user and that they are waiting for to complete before doing anything else

The second type I think could be better shown as more obvious notifications (it's something I'd like myself - sometimes when I press F5 to launch my VS Code extension code I wonder why nothing is happening - the only indication is "Building..." in the status bar - I'd rather have an expanded notification).

Requests for this:

- https://github.com/Dart-Code/Dart-Code/issues/2597#issuecomment-652204865
- https://github.com/Dart-Code/Dart-Code/issues/2601#issuecomment-653356813
- https://github.com/flutter/flutter/issues/60439 / https://github.com/Dart-Code/Dart-Code/issues/2628
Repository: flutter
Issue: ReorderableListView animate insertion/removal of items
Labels: c: new feature,framework,f: material design,P3,team-design,triaged-design

<!-- Thank you for using Flutter!

     If you are looking for support, please check out our documentation
     or consider asking a question on Stack Overflow:
      * https://flutter.dev/
      * https://api.flutter.dev/
      * https://stackoverflow.com/questions/tagged/flutter?sort=frequent

     If you have found a bug or if our documentation doesn't have an answer
     to what you're looking for, then fill our the template below. Please read
     our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports
-->

## Use case

<!--
     Please tell us the problem you are running into that led to you wanting
     a new feature.

     Is your feature request related to a problem? Please give a clear and
     concise description of what the problem is.

     Describe alternative solutions you've considered. Is there a package
     on pub.dev/flutter that already solves this?
-->

## Proposal

<!--
     Briefly but precisely describe what you would like Flutter to be able to do.

     Consider attaching images showing what you are imagining.

     Does this have to be provided by Flutter directly, or can it be provided
     by a package on pub.dev/flutter? If so, maybe consider implementing and
     publishing such a package rather than filing a bug.
-->
Repository: TypeScript
Issue: Write typescript as modules, but output js without modules
Labels: Suggestion,Awaiting More Feedback

## Search Terms

Searched the whole internet for anything like it, only webpack and browserify related 'solutions' are proposed.

## Suggestion

Split module options in tsconfig so typescript modules could be set separately from output target modules.

## Use Cases

Our usecase:
- We have a huge codebase, and we want to slowly move to typescript.
- We have a sophisticated build system that we don't want to change - we want typescript to transpile to js and to take js and continue to use it as we were.
- We don't need nor want to use modules in js. We never used modules - we have our system of imports, minification, 
 versioning, caching, etc.
- The only reason why we need to use modules in typescript is tsc incremental watch build. With no modules, it builds complete codebase on each change. We want to build only file changed and dependent files.

So currently we have two bad options - don't use modules (unacceptably slow build time) or use modules (get the output that we don't want). Currently we 'mitigate' the issue by removing parts of js tsc has output (removing stuff we recognize as related to exports and require), but that's just terrible.

Even worse, even if we were to use modules, require / exports are inlined even in output we want to inline to index.html as critical JS...

## Examples

I guess I explained it all above.

## Checklist

My suggestion meets these guidelines:

* [ * ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
* [ * ] This wouldn't change the runtime behavior of existing JavaScript code
* [ * ] This could be implemented without emitting different JS based on the types of the expressions
* [ * ] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
* [ * ] This feature would agree with the rest of [TypeScript's Design Goals](https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals).
Repository: flutter
Issue: Google_maps_flutter KML support
Labels: c: new feature,p: maps,package,c: proposal,team-ecosystem,P3,triaged-ecosystem

## Use case  

I have a app where I want to load approximately 1000+ streets data(polylines) to draw on initState but It is taking too much time to load and blocking a UI when I try to do setState to place polylines on map. So I think if I can load the kml on it then it will be less expensive computation. So my app will be a faster to load the lines or any other data as well. Alongside with it I have other 2 apps in which I wanted to use KML but for them I don't had that much data to load on map which is taking too much time to load but for this  

There is no plugin/package on pub.dev which is providing a facility for loading a KML on map right now. 

## Proposal

So , if google_maps_flutter package can have the feature of adding KML layer on map then It will be good for many developers as many are waiting and wondering for this feature. There was already a feature request on this thread https://github.com/flutter/flutter/issues/33563  but probably there was no response from flutter team and probably he closed it and probably at that time no one was concerned about this feature but as of now the flutter community is growing there are many developers who are looking for official feature from goole_maps_flutter. I will be waiting for your response.
Repository: go
Issue: html/template: unexpected semantic differences compared to text/template
Labels: NeedsInvestigation

I ported the text/template tests over to html/template and discovered some semantic differences between the two that I did not expect.

In html/template:

 - Missing values format as empty strings, while in text/template they format as `<no value>`. For example:

       - 	{"map .NO", "{{.MSI.NO}}", "<no value>", tVal, true},
       + 	{"map .NO", "{{.MSI.NO}}", "", tVal, true}, // NOTE: <no value> in text/template

       - 	{"empty nil", "{{.Empty0}}", "<no value>", tVal, true},
       + 	{"empty nil", "{{.Empty0}}", "", tVal, true}, // NOTE: <no value> in text/template
       
       and so on

 - Missing values passed explicitly to the html escaper format as `<nil>` instead of `<no value>`:
 
       - 	{"html untyped nil", `{{html .Empty0}}`, "&lt;no value&gt;", tVal, true},
       + 	{"html untyped nil", `{{html .Empty0}}`, "&lt;nil&gt;", tVal, true}, // NOTE: "&lt;no value&gt;" in text/template

 - Given an addressable struct field with a pointer-receiver String method, the String method is not used (in text/template it is):
 
       - 	{"V{6666}.String()", "-{{.V0}}-", "-<6666>-", tVal, true},
       + 	{"V{6666}.String()", "-{{.V0}}-", "-{6666}-", tVal, true}, //  NOTE: -<6666>- in text/template
	
 - Given an addressable struct field with a pointer-receiver Error method, the Error method is not used (in text/template it is):

       - 	{"W{888}.Error()", "-{{.W0}}-", "-[888]-", tVal, true},
       + 	{"W{888}.Error()", "-{{.W0}}-", "-{888}-", tVal, true}, // NOTE: -[888] in text/template

 - TestExecuteOnNewTemplate crashes on all the cases from #11379.
 
 - TestMessageForExecuteEmpty shows a different error message:

       - template: empty: "empty" is an incomplete or empty template
       + template: "empty" is an incomplete or empty template

 - TestIssue31810 fails (“check that a parenthesized first argument behaves properly”, for #31810).

 - TestTemplateLookUp (for #10910, #10926) fails. Lookup returns non-nil values for undefined templates.

 - Calling `Parse("")` does not overwrite the text of a previous template parse, as it does in text/template.
   See TestEmptyTemplate, specifically the last case in the table:

       	{[]string{"{{.}}", ""}, "twice", ""},

The `<no value>` change looks like it may possibly be intentional. All the others look like bugs.

/cc @robpike
Repository: terminal
Issue: Epic: Settings UI Completion
Labels: Product-Terminal,Issue-Scenario,Area-SettingsUI

_written and maintained by @carlos-zamora_

Settings UI Issue: #1564
Settings UI PR: #8048
Settings UI Spec & Design: #6720

Now that it has merged to main, there are a few follow-up items to work on. This scenario tracks follow-up tasks from that PR, and other features we are committing to for Windows Terminal v2.

## Features
- [x] **Profile/ColorScheme should have an Add/Rename/Delete option**
  - PR #8403
- [x] **SettingsContainer**
  - ~Implement #8269 (tracked in #8804 )~ PR #8919
  - Add hyperlinks (navigate to source)
  - Add hyperlinks for read-only pages (i.e. profile generators)
- [x] **Terminal Appearance Preview** (#9122)
  - Embed a `TermControl` in the "Appearance" tab of a Profile page. This `TermControl` should update as you make changes to your `Profile`
  - A similar thing should be done to the ColorSchemes page
- [x] **FontFace as a ~TextBox~ ComboBox**
  - FontFace would be easier to select if it was a ComboBox of supported fonts. The list should only include (or at least denote) monospace fonts.
- [x] #9121 "Add a profile" page

## Styling
- [X] **Custom Enum Ordering** (PR #8784 )
  - Some enum lists (i.e. Font Weight), are not intuitively ordered.

## Tasks
- [x] Document how to add a setting ("AddASetting.md")
- [ ] Deduplicate Browse button logic
- [ ] #8452 Polish `TabBase`
- [ ] Remove visibility hacks (i.e. acrylic opacity slider), and use ProfileViewModel instead
- [x] #8746: Finalize localization changes (#8777)
- [x] #8764: Polish Profile page
- [x] #8765: Polish ColorSchemes page
- [x] #8899: Automation Properties
- [x] #9075: initialPosition, centerOnLaunch

## Bugs
- [X] JSON change --> crash
  - ~open SUI --> open JSON --> edit retro effects in JSON --> save file --> cry because the app crashed~
  - No longer getting a repro on main (6b2ae625). 990e06b4 's changes to the refresh mechanism may have fixed this?
- [x] #8747: delete new profile --> crash
- [x] #8748: delete newly (un)hidden profile --> crash
- [x] #8756: Rename color scheme should update profile
- [x] #13451


## Backlog
- [ ] Settings UI reformats settings.json in an hard to read way #8991
- [ ] **Home Page** #9120 
  - Add a Home page that can present release notes, docs, and any other helpful information for Windows Terminal
- [ ] #8639 Import Color Scheme
- [ ] **Search**
  - #7370 has a rough implementation of Search in MainPage.xaml ([link](https://github.com/microsoft/terminal/blob/a305c0d9689a87140a1fdd69520068e25d5fee80/src/cascadia/TerminalSettingsEditor/MainPage.xaml#L29))
  - Needs to be localizable
- [ ] **Detect unsaved changes**
  - Update the warning label and Discard/Apply buttons appropriately
  - #12424