sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: PowerToys
Issue: Allow larger files to be previewed in File Explorer (current max 50KB)
Labels: Idea-Enhancement,Product-File Explorer,Needs-Triage

### Description of the new feature / enhancement

Ability to preview files > 50KB or at least the first 50KB of a larger file.

### Scenario when this would be used?

When using File Explorer.

### Supporting information

Attempting to view a 67KB file, I get the message: 

This file is too big to preview.
Max file size 50KB
Repository: pytorch
Issue: DISABLED test_variant_consistency_jit_linalg_lu_cuda_complex64 (__main__.TestJitCUDA)
Labels: module: cuda,triaged,module: flaky-tests,skipped,module: unknown

Platforms: linux

This test was disabled because it is failing in CI. See [recent examples](https://hud.pytorch.org/flakytest?name=test_variant_consistency_jit_linalg_lu_cuda_complex64&suite=TestJitCUDA) and the most recent trunk [workflow logs](https://github.com/pytorch/pytorch/runs/8924896021).

Over the past 3 hours, it has been determined flaky in 1 workflow(s) with 1 failures and 1 successes.

**Debugging instructions (after clicking on the recent samples link):**
DO NOT BE ALARMED IF THE CI IS GREEN. We now shield flaky tests from developers so CI will thus be green but it will be harder to parse the logs.
To find relevant log snippets:
1. Click on the workflow logs linked above
2. Click on the Test step of the job so that it is expanded. Otherwise, the grepping will not work.
3. Grep for `test_variant_consistency_jit_linalg_lu_cuda_complex64`
4. There should be several instances run (as flaky tests are rerun in CI) from which you can study the logs.


cc @ngimel
Repository: TypeScript
Issue: Truthy value not recognized by CFA as a constant condition in loops
Labels: Suggestion,Awaiting More Feedback

# Bug Report

### 🔎 Search Terms

while loop, infinite, cfa, control flow analysis

### 🕗 Version & Regression Information

This is also the 'Nightly' version in the playground: [nightly TS playground](https://www.typescriptlang.org/play?ts=4.9.0-dev.20221017#code/C4TwDgpgBAIglgcwE4EMwAsByB7AJtAXigG8AoKKXFYFALhPIqgGsIR6BnYJOAOwQDcjAL5CKYFEgi9gAfnrxkaLHghDRpUhAAeYbEmBQAZgFdeAY2BxsvKAgjAAytz4IACtXQAKXqoWJUDBx8AEpOF34AbQBdBgoAGwcocxMkKRlgiH8lINUoIl98MWSbLigJYHR8qBihRgB3dDhEqC8UtOlgTJC4pgB6PspsKF9DPnN4k3woJGxsYAByDhY2Rgo4I1aAQnb0rtUAOgk9nrImJilgVNsK9AOpADcIJA4ILxDiimE18s8jkw43l2nUyByoNAOrBAHx+wIyeSIcP2+COkk6xW+3yAA)

- This is the behavior in every version I tried

### ⏯ Playground Link

[Playground link with relevant code](https://www.typescriptlang.org/play?ts=4.8.4#code/C4TwDgpgBAIglgcwE4EMwAsByB7AJtAXigG8AoKKXFYFALhPIqgGsIR6BnYJOAOwQDcjAL5CKYFEgi9gAfnrxkaLHghDRpUhAAeYbEmBQAZgFdeAY2BxsvKAgjAAytz4IACtXQAKXqoWJUDBx8AEpOF34AbQBdBgoAGwcocxMkKRlgiH8lINUoIl98MWSbLigJYHR8qBihRgB3dDhEqC8UtOlgTJC4pgB6PspsKF9DPnN4k3woJGxsYAByDhY2Rgo4I1aAQnb0rtUAOgk9nrImJilgVNsK9AOpADcIJA4ILxDiimE18s8jkw43l2nUyByoNAOrBAHx+wIyeSIcP2+COkk6xW+3yAA)

### 💻 Code

<!-- Please post the relevant code sample here as well-->
```ts
type DigraphNode = {
  data: {
    key: string;
  };
  parent?: DigraphNode;
};

export function getStringPath(node: DigraphNode): string[] { // Function lacks ending return statement and return type does not include 'undefined'.(2366)
  let currentNode: DigraphNode = node;
  const path = [];

  while (currentNode) {
    // do not include root's key
    if (!currentNode.parent) {
      return path.reverse();
    }
    path.push(currentNode.data.key);
    currentNode = currentNode.parent;
  }
}
```

### 🙁 Actual behavior

I get an error:
> Function lacks ending return statement and return type does not include 'undefined'.(2366)

### 🙂 Expected behavior

I shouldn't get an error because `currentNode` is always truthy, this should work in the same way as `while (true)` ([TS playground](https://www.typescriptlang.org/play?ts=4.8.4#code/C4TwDgpgBAIglgcwE4EMwAsByB7AJtAXigG8AoKKXFYFALhPIqgGsIR6BnYJOAOwQDcjAL5CKYFEgi9gAfnrxkaLHghDRpUhAAeYbEmBQAZgFdeAY2BxsvKAgjAAytz4IACtXQAKXqoWJUDBx8AEpOF34AbQBdBgoAGwcocxMkKRlgiH8lINUoIl98MWSbLigJYHR8qBihRgB3dDhEqC9uEwgQuKYAeh7KbChfQz5zeJN8KCRsbGAAcg4WNkYKOCNWgEIUtOlgTIA6CXTgLrImJilgVNsK9H2pADcIJA4ILxDiimEV8s9Dkw43m2xwOVBo+1YIA+P2Bu0y1VhGVUh0ku2K32+QA)).
Repository: rust
Issue: Divergent suggestion "expressions must be enclosed in braces to be used as const generic arguments"
Labels: A-diagnostics,T-compiler

I noticed this while trying to understand #103143.

Given the following code ([playground](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2021&gist=b666ac930eb1ca293487df5e13092a50)):

```rust
fn main() {
    x::< #[a] (1 + 2) >
}
```

The current output starts with:

```
error: invalid const generic expression
 --> src/main.rs:2:15
  |
2 |     x::< #[a] (1 + 2) >
  |               ^^^^^^^
  |
help: expressions must be enclosed in braces to be used as const generic arguments
  |
2 |     x::< #[a] { (1 + 2) } >
  |               +         +
```

### Why this doesn't work

Adding the braces as suggested:

```rust
fn main() {
    x::< #[a] { (1 + 2) } >
}
```

doesn't fix the problem, and the compiler continues suggesting to add more braces indefinitely:

```
error: invalid const generic expression
 --> src/main.rs:2:15
  |
2 |     x::< #[a] { (1 + 2) } >
  |               ^^^^^^^^^^^
  |
help: expressions must be enclosed in braces to be used as const generic arguments
  |
2 |     x::< #[a] { { (1 + 2) } } >
  |               +             +
```

### Possible alternative

The suggestion could place the opening brace _before_ the `#[a]`, i.e.:

```rust
fn main() {
    x::< { #[a] (1 + 2) } >
}
```

This at least leads to a different error message:

```
error[E0658]: attributes on expressions are experimental
 --> src/main.rs:2:12
  |
2 |     x::< { #[a] (1 + 2) } >
  |            ^^^^
  |
  = note: see issue #15701 <https://github.com/rust-lang/rust/issues/15701> for more information
  = help: add `#![feature(stmt_expr_attributes)]` to the crate attributes to enable
```
Repository: PowerToys
Issue: Dynamic Layout
Labels: Idea-Enhancement,Product-FancyZones

### Description of the new feature / enhancement

Switch layout depending on the number of windows present on the screen.

### Scenario when this would be used?

First app should open centered, or full screen (or majority of the screen). Opening another window will split the screen (or depending on the layout provided). Minimizing or Closing an app will also apply. The goal is to always maximize the screen estate and reduce overlapping of windows.

### Supporting information

_No response_
Repository: stable-diffusion-webui
Issue: Add hotkey to Color Sketch
Labels: enhancement

We need hotkey dropper tool like photoshop when painting, alt
and incresse and descrese brush size "[] "or "qw"

Thanks for developing such a good tool
Repository: pytorch
Issue: Nandense layer for missing values
Labels: module: sparse,module: nn,triaged,module: masked operators

### 🚀 The feature, motivation and pitch

This paper has an interesting Tensorflow implementation of a nan aware layer so that the neural network can handle missing values natively. Code provided in the paper, might be interesting to implement in Pytorch for inclusion in this library.

https://arxiv.org/pdf/2206.01640.pdf

### Alternatives

_No response_

### Additional context

_No response_

cc @nikitaved @pearu @cpuhrsch @amjames @bhosmer @albanD @mruberry @jbschlosser @walterddr @kshitij12345 @saketh-are
Repository: stable-diffusion-webui
Issue: RuntimeError: CUDA error: an illegal memory access was encountered
Labels: asking-for-help-with-local-system-issues

**Describe the bug**
Generating images has a slight chance of crashing the app due to "illegal memory access".

**To Reproduce**
Steps to reproduce the behavior:
Put in prompts, configure settings, press generate, crash happens randomly through the image generation process.

Settings used:
Prompt: 19 tags
Negative Prompt: 34 tags
Sampling Steps 36
Sampling Method: Euler
Width/Height: 512
Batch Count: 9
CFG Scale: 11
Stop At last layers of CLIP model: 2

**Expected behavior**
Not crashing :)

**Screenshots**
Crashlog/Stacktrace: https://hastebin.com/obinazobuw.txt
TL;DR:
```
RuntimeError: CUDA error: an illegal memory access was encountered
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
```


**Desktop (please complete the following information):**
 - OS: Windows 10 build 19044.2130
 - Browser: Chrome, 106.0.5249.119
 - Commit revision: cccc5a20fce4bde9a4299f8790366790735f1d05
 - Python 3.10.6
 - HW specs: Ryzen 5800X3D; 32GB of 3000MHz ram; software on NVMe SSD; RTX 2070 Super
 - Model: NovelAI hash [925997e9]

**Additional context**
This also happened on my 3700X before upgrading; I currently have my bclk set to 100.5 (so 1000mhz = 1005mhz) on my system, no OC on my CPU, and my GPU is running at +105Mhz on the core and +1000mhz on the VRAM.
Previously was running it at 1250 but was experiencing this issue much more frequently.
Despite these OCs, I reset everything and still experience this crash.
Repository: vscode
Issue: SCM - Add sort by extension/type to Git changes
Labels: help wanted,feature-request,scm

<!-- ⚠️⚠️ Do Not Delete This! feature_request_template ⚠️⚠️ -->

Add sort by extension/type to Git changes. Currently there is only:
by Name
by Path
by Status

![gitsort](https://user-images.githubusercontent.com/2189038/196201922-2def57b3-2aba-4322-b5bf-b73d4d27e225.png)

Is it here https://github.com/microsoft/vscode/blob/5d01d3ca45946c6f5b9e25d04526a046872fc965/src/vs/workbench/contrib/scm/browser/scmViewPane.ts#L1661-L1677 ?
Repository: PowerToys
Issue: Always on Top border remains after moving window
Labels: Issue-Bug,Product-Always On Top,Needs-Triage,Needs-Team-Response

### Microsoft PowerToys version

"0.63.0"

### Installation method

Windows Store

### Running as admin

- [x] Yes

### Area(s) with issue?

Always on Top

### Steps to reproduce

I simply used the function and then moved the window.

### ✔️ Expected Behavior

While moving, the border stays with the window

### ❌ Actual Behavior

The border stays with the windows but it has created a new border.
![image](https://user-images.githubusercontent.com/29505634/196201361-e62e7165-4269-4e30-8dc9-c74879d325aa.png)

Even if i disable the always on top with the shortcut, the border stays.

I need to restart Powertoys to let the ghost border go away.

### Other Software

cmd.exe
Repository: pytorch
Issue: cppextension host compiler check ignores executable symbolic link in CUDA bin directory
Labels: module: build,module: cpp-extensions,triaged

### 🐛 Describe the bug

PyTorch 1.12 added a [host compiler version check](https://github.com/pytorch/pytorch/blob/b40f4434ac3512a21dcec91467df1b179898503f/torch/utils/cpp_extension.py#L409-L420) when building PyTorch extensions. However, one way of specifying the host compiler executable used by nvcc is to include a symbolic link to the desired compiler executable in the CUDA home directory, and it seems that the compiler version check does not take this into account. This results in an error, even though nvcc may well be setup to use the correct host compiler version. 

It may well be good to check for other ways that nvcc may be informed which compiler to use and factor these into the version check too.

It seems a simple workaround is to simply to set the ```TORCH_DONT_CHECK_COMPILER_ABI``` environment variable. But it would be nice to not rely on this, as it is useful to know when the compiler version is genuinely wrong.



### Versions

PyTorch 1.12

cc @malfet @seemethere @zou3519
Repository: go
Issue: os/exec: test timeouts on aix-ppc64
Labels: NeedsInvestigation,FrozenDueToAge,OS-AIX

```
#!watchflakes
post <- builder == "aix-ppc64" && pkg == "os/exec" && (`^Test killed: ran too long` || `panic: test timed out`)
```

Issue created automatically to collect these failures.

Example ([log](https://build.golang.org/log/611fd6464d9c020795b5d99c655ccc67b71543e0)):

    SIGQUIT: quit
    PC=0x9000000005b8290 m=2 sigcode=0
    
    r0   0xffffffffffffffff	r1   0x1102ea0f0
    r2   0xffffffffffffffff	r3   0xffffffffffffffff
    r4   0xffffffffffffffff	r5   0xffffffffffffffff
    r6   0xffffffffffffffff	r7   0xffffffffffffffff
    r8   0xffffffffffffffff	r9   0xffffffffffffffff
    r10  0xffffffffffffffff	r11  0xffffffffffffffff
    r12  0xffffffffffffffff	r13  0x1102f2800
    ...
    	/ramdisk8GB/workdir-host-aix-ppc64-osuosl/go/src/os/user/lookup.go:15 +0x24 fp=0xa0001000005ada8 sp=0xa0001000005ad88 pc=0x100145cf4
    sync.(*Once).doSlow(0x11020b9d0?, 0xa0001000005ae50?)
    	/ramdisk8GB/workdir-host-aix-ppc64-osuosl/go/src/sync/once.go:74 +0x124 fp=0xa0001000005ae10 sp=0xa0001000005ada8 pc=0x10007c064
    sync.(*Once).Do(...)
    	/ramdisk8GB/workdir-host-aix-ppc64-osuosl/go/src/sync/once.go:65
    os/user.Current()
    	/ramdisk8GB/workdir-host-aix-ppc64-osuosl/go/src/os/user/lookup.go:15 +0x50 fp=0xa0001000005ae68 sp=0xa0001000005ae10 pc=0x100144e00
    os/exec_test.TestCredentialNoSetGroups(0xa0001000022a340)
    	/ramdisk8GB/workdir-host-aix-ppc64-osuosl/go/src/os/exec/exec_posix_test.go:45 +0x38 fp=0xa0001000005af30 sp=0xa0001000005ae68 pc=0x1002b2c18
    testing.tRunner(0xa0001000022a340, 0x110085dd8)


— [watchflakes](https://go.dev/wiki/Watchflakes)
Repository: PowerToys
Issue: Keyboard manager settings in Remote Desktop connections
Labels: Idea-Enhancement,Needs-Triage

### Description of the new feature / enhancement

keyboard remap settings to pass thru RDP


### Scenario when this would be used?

i have lenovo ideapad 16, with useless media keys, but missing dedicated pg up/down and home/end buttons.
So im using PT to remap next/previous track with end/home buttons.
But suddenly figured out that these settings doesnt work in RDP session

### Supporting information

_No response_
Repository: stable-diffusion-webui
Issue: Don't want send img2img auto replace prompt and other setting
Labels: enhancement

Don't want send img2img auto replace prompt and other setting
Repository: flutter
Issue: Ink splash appears behind image when there is a `Material` above `Ink.image`
Labels: framework,f: material design,has reproducible steps,P2,found in release: 3.3,found in release: 3.4,team-design,triaged-design

In my previous issue #113524 i have a discussed the No material widget found behavior which is also discussed here #28326 that sometimes when navigating the `Material` widget not found error with red screen shows for a sec and gone i have experience this with `Ink.image` with `Hero` widget with intent to show ripple on image not behind if i wrap `Ink.image` with Material which removes the error but changes the behavior what I want solution to this to warp whole `MaterialApplication` widget with `Material` widget
Repository: stable-diffusion-webui
Issue: Feedback for skip, interrupt, and cancel
Labels: enhancement

sometimes I'm not sure if the program has actually received my input for stopping generation, it would be nice if the console or UI would indicate that it's going to cancel soon
Repository: stable-diffusion-webui
Issue: Predetermined staircase hypernetwork training for quality of life
Labels: enhancement

**Is your feature request related to a problem? Please describe.**
no

**Additional context**
Upon training hypernetworks, it would be a quality of life update to use predetermined hypernetwork training in a staircase fashion.
for example it will train:
First Staircase: 5 times on 0.05
Second: 50 on 0.005
Third: 100 on 0.0005
Fourth: 1500 on 0.00005
Fifth: 10000 on 0.000005
With a save file for every 5%(or whatever percent you want) of max training per staircase(or 5 training times, whichever is higher).
If this were to be implemented, it can be done with a plus icon for every new staircase, and the staircase preset can be saved to be default ui parameters.
Repository: neovim
Issue: can not set titlestring if TERM=wezterm when using nvim inside wezterm
Labels: bug,environment,tui,complexity:low,has:plan

### Neovim version (nvim -v)

0.8

### Vim (not Nvim) behaves the same?

N/A

### Operating system/version

macOS 12.5.1

### Terminal name/version

wezterm latest nightly

### $TERM environment variable

wezterm

### Installation

release

### How to reproduce the issue

Run the following command

```
TERM=wezterm nvim --clean -c "set title titlestring=hello"
```


### Expected behavior

The terminal title should be changed to `hello`

### Actual behavior

it is not, it just shows the full command: `TERM=wezterm nvim --clean -c "set title titlestring=hello"`.

Base on discussion on https://github.com/wez/wezterm/issues/2112, it seems that it is due to nvim matching `xterm` to decide terminal ability instead of using terminfo file.

There is workaround (change wezterm terminfo file), but it is not elegant.
Repository: TypeScript
Issue: Functions in mixins with return type 'this' are typed as 'any' in declaration files
Labels: Bug,Help Wanted

# Bug Report

### 🔎 Search Terms
mixin declaration this

### 🕗 Version & Regression Information
4.5.4, but active in 4.8.4

- This is the behavior in every version I tried

### ⏯ Playground Link
[Playground link with relevant code](https://www.typescriptlang.org/play?ts=4.8.4#code/MYGwhgzhAEDKAOAnAlgFwKbQN4ChrQDswBbTAXmgCJKBuPPaYAewIlUQFdhUnEAKIqQBc0NigIBzAJTYG+VAAtkEAHSDyhEujr4Avjn05UAT3iYAwizFceiaBQLoA7tD4r3YRBIgiwBYwDaALoyZAB82Lo6OABmHATcyCxwwGAg6AA8ACoAQpCY6AAeGAQAJjCWrOw2vGF8eRDoIrn5Mrj4iOioHIgEjOBQKWnIktBFJeXQDZjt+NAA+hCp6fbQAIzRc6JdsMvofEtpTYQcxABG6IhSIorKslvySqqLe6uH6ToPnd290LcQnz0DCiBhwOAA9OC-k9oJ0IBwQKgYCNoAByRqoXZHA57EQEU4XK6+fyo6AolSlFRInBFeC8VDQUroGJgBEMrHpPgIFAYKR0CFQ-5kmAmMylRi8TrcEDGVzonZ7HFHPEEy7XaHKVFSAUSqqiPalbloDQc-ZG3l0SFjQp0xAMpkstn6o6GpDGmhAA)

### 💻 Code
(slightly edited example from [mixins doc](https://www.typescriptlang.org/docs/handbook/mixins.html))
```ts
class Sprite {
  name = "";
 
  constructor(name: string) {
    this.name = name;
  }
}

type Constructor = new (...args: any[]) => {};
 
function Scale<TBase extends Constructor>(Base: TBase) {
  return class Scaling extends Base {
    _scale = 1;
 
    setScale(scale: number): this {
      this._scale = scale;
      return this;
    }
  };
}

// this results in 'setScale(scale: number): any' in .d.ts
export default Scale(Sprite);

// this is typed correctly ('setScale(scale: number): this')
// const scaledSprite = Scale(Sprite);
// export default scaledSprite;
```

### 🙁 Actual behavior
return type of `setScale` on the class with mixin applied is typed as `any`
```ts
declare const _default: {
    new (...args: any[]): {
        _scale: number;
        setScale(scale: number): any;
    };
} & typeof Sprite;
```

### 🙂 Expected behavior
return type of `setScale` on the class with mixin applied should be `this`
```ts
declare const _default: {
    new (...args: any[]): {
        _scale: number;
        setScale(scale: number): this;
    };
} & typeof Sprite;
```

In the playground, if `Scale(Sprite)` is assigned to a variable first before being exported, the types are correct. However, it seems that if the mixin is declared in a separate file and used in another file, the function is always typed incorrectly as `any`
Repository: stable-diffusion-webui
Issue: Add queuing (different from batching)
Labels: enhancement

**Is your feature request related to a problem? Please describe.**
Sometimes I have separate prompts I'm writing on, I'd like to be able to open multiple tabs and queue up prompt generations

**Describe the solution you'd like**
A queue, similar to that of Midjourney

**Describe alternatives you've considered**
Just waiting...

**Additional context**
N/a
Repository: stable-diffusion-webui
Issue: AssertionError: An object named 'ResNetArcFace' was already registered in 'arch' registry!
Labels: bug-report,upstream

Python 3.9.0 (tags/v3.9.0:9cf6752, Oct  5 2020, 15:34:40) [MSC v.1927 64 bit (AMD64)]
Commit hash: <none>
Installing requirements for Web UI
Launching Web UI with arguments:
Warning: caught exception 'Torch not compiled with CUDA enabled', memory monitor disabled
Traceback (most recent call last):
  File "E:\stable-diffusion-webui-master\launch.py", line 181, in <module>
    start_webui()
  File "E:\stable-diffusion-webui-master\launch.py", line 175, in start_webui
    import webui
  File "E:\stable-diffusion-webui-master\webui.py", line 14, in <module>
    import modules.extras
  File "E:\stable-diffusion-webui-master\modules\extras.py", line 12, in <module>
    import modules.gfpgan_model
  File "E:\stable-diffusion-webui-master\modules\gfpgan_model.py", line 6, in <module>
    import gfpgan
  File "C:\Users\Win10Pro\AppData\Local\Programs\Python\Python39\lib\site-packages\gfpgan\__init__.py", line 2, in <module>
    from .archs import *
  File "C:\Users\Win10Pro\AppData\Local\Programs\Python\Python39\lib\site-packages\gfpgan\archs\__init__.py", line 10, in <module>
    _arch_modules = [importlib.import_module(f'gfpgan.archs.{file_name}') for file_name in arch_filenames]
  File "C:\Users\Win10Pro\AppData\Local\Programs\Python\Python39\lib\site-packages\gfpgan\archs\__init__.py", line 10, in <listcomp>
    _arch_modules = [importlib.import_module(f'gfpgan.archs.{file_name}') for file_name in arch_filenames]
  File "C:\Users\Win10Pro\AppData\Local\Programs\Python\Python39\lib\importlib\__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "C:\Users\Win10Pro\AppData\Local\Programs\Python\Python39\lib\site-packages\gfpgan\archs\arcface_arch.py", line 172, in <module>
    class ResNetArcFace(nn.Module):
  File "E:\stable-diffusion-webui-master\repositories\CodeFormer\basicsr\utils\registry.py", line 53, in deco
    self._do_register(name, func_or_class)
  File "E:\stable-diffusion-webui-master\repositories\CodeFormer\basicsr\utils\registry.py", line 39, in _do_register
    assert (name not in self._obj_map), (f"An object named '{name}' was already registered "
AssertionError: An object named 'ResNetArcFace' was already registered in 'arch' registry!
Repository: pytorch
Issue: Bug in Histogram Observer Implementation
Labels: oncall: quantization,triaged

### 🐛 Describe the bug

It seems to me that the `HistogramObserver` implementation has a bug. In [L 1181](https://github.com/pytorch/pytorch/blob/e85dbcc9b075961ab082975348c5cf1d99b7da76/torch/ao/quantization/observer.py#L1181) the forward pass is as follows:

```
combined_histogram = self._combine_histograms(
    combined_histogram,
    self.histogram,
    self.upsample_rate,
    downsample_rate,
    start_idx,
    self.bins,
)
```
Where `_combine_histograms` is defined as:
```
def _combine_histograms(
    self,
    orig_hist: torch.Tensor,
    new_hist: torch.Tensor,
    upsample_rate: int,
    downsample_rate: int,
    start_idx: int,
    Nbins: int,
) -> torch.Tensor:
```

Therefore, ` combined_histogram` gets mapped to `orig_hist` while `self.histogram` gets mapped to `new_hist`. It seems to me like it should be the other way around given the variable names.

Can someone clarify?

Thank you!!

### Versions

latest main

cc @jerryzh168 @jianyuh @raghuramank100 @jamesr66a @vkuzo @jgong5 @Xia-Weiwen @leslie-fang-intel