sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: PowerToys
Issue: white bar on screen
Labels: Issue-Bug,Product-PowerToys Run

### Microsoft PowerToys version

0.78.0

### Installation method

PowerToys auto-update

### Running as admin

Yes

### Area(s) with issue?

General

### Steps to reproduce

![whitebar](https://github.com/microsoft/PowerToys/assets/158061232/f7a1c2b4-23d2-4f08-af31-4d7ddce9c86a)
I have a hdmi KVM switch to switch between different machines.

When even I switch away from the machine that has powertoys installed. and I return. I get this white bar on the screen.
It is laying on top of all applications, and can only be removed if I close powertoys.

### ✔️ Expected Behavior

No white bar in the middle of my screen

### ❌ Actual Behavior

I have a hdmi KVM switch to switch between different machines.

When even I switch away from the machine that has powertoys installed. and I return. I get this white bar on the screen.
It is laying on top of all applications, and can only be removed if I close powertoys.

### Other Software

_No response_
Repository: pytorch
Issue: MODULE.bazel support
Labels: module: build,triaged,module: bazel

### 🚀 The feature, motivation and pitch

With Bazel 7 defaulting to Bzlmod, it would be great if Pytorch would also support MODULE.bazel instead / in addition to WORKSPACE.

### Alternatives

_No response_

### Additional context

This makes dependency version resolution a lot easier and WORKSPACE functionality will be removed in the future.

cc @malfet @seemethere
Repository: go
Issue: x/build: missing LUCI TryBot coverage for x/pkgsite
Labels: Builders,NeedsInvestigation

### Go version

6f44cc88f5f94253096ceed16b8e0fdb117cdd06

### Output of `go env` in your module/workspace:

```shell
N/A
```


### What did you do?

See #65545.

### What did you see happen?

https://go.dev/cl/559615 introduced a regression on `windows`, but the CL had a `TryBots-Pass+1` vote.

I examined the `Checks` tab to see which builders had run, and saw:
-  tricium
- x_pkgsite-go1.20-linux-amd64
- x_pkgsite-go1.21-linux-amd64
- x_pkgsite-go1.22-linux-amd64
- x_pkgsite-gotip-linux-amd64
- x_pkgsite-gotip-linux-amd64-boringcrypto
- x_pkgsite-gotip-linux-amd64-misccompile
- x_pkgsite-gotip-linux-amd64-misccompile-test_only
- x_pkgsite-gotip-linux-amd64-race
- x_pkgsite-gotip-windows-amd64


### What did you expect to see?

Since `x/pkgsite/cmd/pkgsite` is intended to be usable as a replacement for `x/tools/cmd/godoc` (#59347, #40371, #59056), it should have pre-commit testing on at least all of the [first class ports](https://go.dev/wiki/PortingPolicy#first-class-ports) that have scalable builders. Namely:
- `linux/386`
- `linux/amd64`
- `linux/arm`
- `linux/arm64`
- `windows/386`
- `windows/amd64`

In addition, it is surprising that the release-branch and `-race` variants were only run for `linux`, since all of those variants also exist for `windows` too.

Adding the release-branch builders, `386` architecture, and `-race` configuration would have increased the chances of catching the regression by a factor of 6.
Repository: PowerToys
Issue: Process-specific Settings to Lock Window Coordinates + Size
Labels: Idea-New PowerToy,Needs-Triage

### Description of the new feature / enhancement

The Fancy Zones editor should allow individual programs, specified by process name dropdown box or manual browsing to *.exe location, to lock their window placement & size.

### Scenario when this would be used?

When Windows decides to rearrange window sizes, and PowerToys users need to then reshape their desktop, Fancy Zones would have already recognized their processes. Fancy Zones would have immediately fitted app windows to their pre-specified location and size.



### Supporting information

Especially in 21:9 and 32:9 monitors, Fancy Zones often acts as only a temporary zone placement utility. Windows and other operations can detach windows from their last-known zone. The result is an unreliable window placement, which doesn't stay in the same place as where the user left it.

On my 49" 32:9 monitor, after rebooting Win10, I'll typically find a window shrunk and placed erroneously in the upper-right corner of my monitor. These random placements are nowhere near their last FZ placement.

Please implement an extra measurement of making sure intended program windows stay as placed/sized.

I couldn't find a matching feature request open, so maybe this needs a combination with [#463](https://github.com/microsoft/PowerToys/issues/463) or something else…
If you need any extra clarification, I have used the Free & Premium versions of this program:
https://www.actualtools.com/windowmanager/screenshots/

That program serves different purposes than PowerToys, but its Window Settings are exactly what FZ needs, adapted to its own benefit.
Repository: pytorch
Issue: Support for dynamic DataLoader (DataLoader.__getitem__)
Labels: module: dataloader,triaged

### 🚀 The feature, motivation and pitch

## Pitch

I want to be able to access multiple times my dataset without knowing in advance the indices queried.

I propose to build a `DynamicLoader` inheriting the `DataLoader` class using a `DynamicSampler`. I am thus requesting some support for the `__getitem__` method.

## Implementation

```python
class DynamicSampler(Sampler[int]):
    """
    Samples elements dynamically, always in the same order.
    """

    def __init__(self) -> None:
        self.indices = []

    def __iter__(self) -> Iterator[int]:
        return iter(self.indices)

    def __len__(self) -> int:
        return len(self.indices)
    
    def set_indices(self, indices):
        self.indices = indices

class DynamicLoader(DataLoader):
    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        if not isinstance(self.sampler, DynamicSampler):
            raise ValueError("Sampler must be an DynamicSampler")
    
    @contextmanager
    def __getitem__(self, idx):
        self.sampler.set_indices(idx)
        yield super().__iter__()
        self.sampler.set_indices([])
```

## Use Case

The use case would be:

```python
with loader[1, 2, 3] as it:
    for batch in it:
        ...
```

## Disclaimer

This is only the best solution *I* could think of, but I remain open to comments or improvements.


### Alternatives

The alternatives could be to write a custom getter function, for example, using multi-threading, but it is less efficient than just using dataloader multi-processing capabilities.

### Additional context

_No response_

cc @SsnL @VitalyFedyunin @ejguan @dzhulgakov
Repository: go
Issue: x/tools/go/analysis/unitchecker: test fail with "illegal byte sequence" (EILSEQ)
Labels: Builders,NeedsInvestigation,Tools

```
#!watchflakes
default <- pkg == "golang.org/x/tools/go/analysis/unitchecker" && `illegal byte sequence`
```

Issue created automatically to collect these failures.

Example ([log](https://ci.chromium.org/ui/b/8757079028780343489)):

    === RUN   TestExampleSeparateAnalysis
        invoke.go:268: 9.798123ms for GOROOT= GOPATH=/Users/swarming/.swarming/w/ir/x/w/gopath GO111MODULE=off GOPROXY=off PWD=/Users/swarming/.swarming/w/ir/x/t/TestExampleSeparateAnalysis3299946371/001 go list -e -f {{context.ReleaseTags}} -- unsafe
        invoke.go:268: 95.863536ms for GOROOT= GOPATH=/Users/swarming/.swarming/w/ir/x/w/gopath GO111MODULE= GOPROXY=off PWD=/Users/swarming/.swarming/w/ir/x/t/TestExampleSeparateAnalysis3299946371/001 go list -e -json=Name,ImportPath,Error,Dir,GoFiles,IgnoredGoFiles,IgnoredOtherFiles,CFiles,CgoFiles,CXXFiles,MFiles,HFiles,FFiles,SFiles,SwigFiles,SwigCXXFiles,SysoFiles,CompiledGoFiles,Export,DepOnly,Imports,ImportMap,Module -compiled=true -test=false -export=false -deps=true -find=false -pgo=off -- separate/main
    unitchecker.test: failed to export type information: write /Users/swarming/.swarming/w/ir/x/t/TestExampleSeparateAnalysis3299946371/001/30.types: illegal byte sequence
        separate_test.go:188: exit status 1
    --- FAIL: TestExampleSeparateAnalysis (1.38s)


— [watchflakes](https://go.dev/wiki/Watchflakes)
Repository: kubernetes
Issue: Declarative field deprecation & deprecated field usage tracking
Labels: sig/api-machinery,kind/feature,triage/accepted

### What would you like to be added?

I'd like a mechanism to denote that a field is deprecated in CRDs (as well as core/built-in types) and be able to track deprecated field usage via similar mechanisms to how we track deprecated API version usage.

Ideally, this would be an extra field in the AuditEvent structure, similar to:

```
deprecatedFieldUsages:
- spec.myDeprecatedField
```

Depending on how markers for deprecated fields are implemented, we *may* be able to include some kind of details on what version a field was deprecated (or possibly even removed) but I'd leave that TBD after further discussion with sig-apimachinery.

### Why is this needed?

As Kubernetes and third-party CRDs evolve, fields may be deprecated and (depending on the policies of the projects authoring these CRDs) may even be removed at a future date.

Currently, the only way to do this would be with a validating/mutating webhook of some sort, that could inspect the request body in POST/PUT requests and emit audit annotations.

By supporting this as a native feature, it'll ease platform operators managing upgrades and migrations of both Kubernetes and related ecosystem tools that utilise CRDs.

/sig apimachinery
Repository: tauri
Issue: [feat] enhance capability context
Labels: type: feature request

### Describe the problem

The [Capability struct](https://docs.rs/tauri-utils/2.0.0-beta.1/tauri_utils/acl/capability/struct.Capability.html) lets you associate the list of permissions with a [CapabilityContext](https://docs.rs/tauri-utils/2.0.0-beta.1/tauri_utils/acl/capability/enum.CapabilityContext.html) that either links the permissions with local or remote URLs. Currently it is really complicated to define a capability that applies to both local **AND** remote URLs.

Additionally, the remote context only validates the URL domain, and we should change it to also check scheme and port, similar to the [v1 implementation](https://docs.rs/tauri-utils/1.5.2/tauri_utils/config/struct.RemoteDomainAccessScope.html).

### Describe the solution you'd like

Change the CapabilityContext to also allow an array of contexts (weird approach that has less impact on the codebase and is not a breaking change).
Alternatively, we could split the local/remote options to their own fields **(breaking change)**:
```rust
struct Capability {
  local: bool,
  remote: Vec<...>,
 ...
}
```

Change `CapabilityContext::Remote { domains: Vec<glob::Pattern> }` to `CapabilityContext::Remote { urls: Vec<glob::Pattern> }` so you can validate the entire app URL instead of just the domain **(breaking change)**

### Alternatives considered

_No response_

### Additional context

_No response_
Repository: transformers
Issue: BertTokenizer and BertTokenizerFast have different behavior when requested "return_overflowing_tokens"
Labels: Good Second Issue,bug

### System Info

- `transformers` version: 4.37.2
- Platform: Linux-6.5.5-arch1-1-x86_64-with-glibc2.38
- Python version: 3.11.5
- Huggingface_hub version: 0.20.3
- Safetensors version: 0.4.2
- Accelerate version: not installed
- Accelerate config: not found
- PyTorch version (GPU?): 2.2.0+cu121 (True)
- Tensorflow version (GPU?): not installed (NA)
- Flax version (CPU?/GPU?/TPU?): not installed (NA)
- Jax version: not installed
- JaxLib version: not installed



### Who can help?

@ArthurZucker 


### Information

- [ ] The official example scripts
- [x] My own modified scripts

### Tasks

- [ ] An officially supported task in the `examples` folder (such as GLUE/SQuAD, ...)
- [x] My own task or dataset (give details below)

### Reproduction

```python
from transformers import BertTokenizer, BertTokenizerFast, BatchEncoding
n_tok = BertTokenizer.from_pretrained("bert-base-uncased")
f_tok = BertTokenizerFast.from_pretrained("bert-base-uncased")

text = "hello my name is nikola and i debug transformers now"

n_inputs: BatchEncoding = n_tok(text=text, add_special_tokens=True, max_length=6, truncation=True, padding='max_length', return_overflowing_tokens=True)
o = n_inputs.get("overflowing_tokens")
print(f'Overflowing {o}')
n_inputs['input_ids']


f_inputs: BatchEncoding = f_tok(text=text, add_special_tokens=True, max_length=6, truncation=True, padding='max_length', return_overflowing_tokens=True)
o = f_inputs.get("overflowing_tokens")
print(f'Overflowing {o}')
f_inputs['input_ids']
```

### Expected behavior

For the `n_inputs['input_ids']` we get `[101, 7592, 2026, 2171, 2003, 102]`, and
for the `f_inputs['input_ids']` we get `[[101, 7592, 2026, 2171, 2003, 102], [101, 24794, 1998, 1045, 2139, 102], [101, 8569, 2290, 19081, 2085, 102]]`.
Outputs should be the same.
Repository: material-ui
Issue: [material-ui][Autocomplete] Shouldn't close popper if disableCloseOnSelect and the click is within the popper/paper
Labels: new feature,waiting for 👍,package: material-ui,component: autocomplete

### Steps to reproduce

Link to live example: https://codesandbox.io/p/sandbox/autocomplete-dontcloseonselect-hlj9mp

Steps:
1. Open dropdown
2. Click outside of the `ul` but within the Popper or within the Paper.

### Current behavior

The popper closes.

### Expected behavior

The popper shouldn't close.

### Context

Check this real life example: https://codesandbox.io/p/sandbox/autocomplete-multiple-popper-7w9tn6?file=%2Fsrc%2FAutocompleteMultiCustom.tsx%3A23%2C11-23%2C31

Not closing on Paper/Popper click would allow to easily implement things like SelectAll / DeselectAll and other stuff. In this example clicking on Select All closes the popper and doesn't even trigger SelectAll button click.

I'm not clear how the `disableCloseOnSelect` is implemented, but seems that it does not close the popper when the click is within the `ul` element. Because even though I customize `ListboxComponent` it still closes the popper.

(Alternatively another props can be added: `disableCloseOnClickWithinPopper`, `disableCloseOnClickWithinPaper`)

### Your environment

_No response_

**Search keywords**: autocomplete disableCloseOnSelect popper close
Repository: angular
Issue: Style guide updates
Labels: P2,area: docs

### Describe the problem that you experienced

This is an umbrella issue to report improvements that have been suggested for the style guide. 

* #54341
* #54277 
* #54220
* #52561
* #49261
* #46420
* #55703

PRs 

* #54084
Repository: go
Issue: x/tools/gopls: extract func/method arg(s) to struct
Labels: help wanted,FeatureRequest,gopls,Tools,Refactoring

Given something like this, where `timeout` is an option:

```go
func search(query string, timeout time.Duration) {}
```

When it's time to add another option,  one might extract `timeout` to an options struct so it's then easy to add more things:

```go
type searchOptions struct {
  timeout time.Duration
}

func search(query string, opts searchOptions) {}
```

A code action for this kind of extraction would be very helpful!
Repository: TypeScript
Issue: Path completions from subpath exports containing `/` have broken replacement range
Labels: Bug,Help Wanted,Domain: Completion Lists

Fourslash test:

```ts
/// <reference path="fourslash.ts" />

// @module: preserve
// @moduleResolution: bundler
// @allowJs: true

// @Filename: /node_modules/repo/package.json
//// {
////   "name": "repo",
////   "exports": {
////     "./foo/bar": "./src/bar.js"
////   }
//// }

// @Filename: /node_modules/repo/src/bar.js
//// export {};

// @Filename: /main.js
//// import { } from "repo/[|f/**/|]";

verify.completions({
  marker: "",
  isNewIdentifierLocation: true,
  exact: [
    { name: "foo/bar", kind: "script", kindModifiers: "", replacementSpan: test.ranges()[0] },
  ],
});
```

The actual replacement range is the full module specifier. (VS Code shows nothing since the text of the replacement range doesn’t match the insertion text.) The root cause is https://github.com/microsoft/TypeScript/pull/41412. At the time, only full module specifiers from `compilerOptions.paths` were able to return slashes (I guess?).

The replacement range logic is really overcomplicated and based on weird heuristics like this. I think it would be simpler and more reliable for path completions to always return entries with replacement ranges equal to the entire string literal (and with text starting from the beginning of the string literal), instead of _sometimes_ returning fragments intended to be inserted after the last slash. However, the existing code makes this really difficult, as nearly all functions directly mutate the set of results without having any knowledge of what’s already been typed in the string literal.

Originally reported at https://github.com/microsoft/TypeScript/issues/56412#issuecomment-1875586765
Repository: ollama
Issue: install process forces to copy `ollama` executable into only specific folders
Labels: feature request,macos

I understand why `ollama` executable might be added to $PATH. For me it's convenience that particular executable will be found in user's `$PATH` folder. The downsides of the process are locations are predefined, they can't be changed or step entirely skipped (under users' conscious choice).

Could you please add this option and an ability to repeat the step if needed.
Repository: pytorch
Issue: Optimizer step pre/post hook tests should try harder to ascertain the hook happens before/after the step call
Labels: module: optimizer,triaged,better-engineering,actionable

### 🚀 The feature, motivation and pitch

The step hook tests after I migrate them to OptimizerInfo live in test/test_optim.py under the names `test_step_pre_hook`, `test_step_post_hook`, and `test_step_all_hooks`. We can improve these tests by:

1. For the pre/post hook tests, modify them so it is more clear the hooks are firing before/after the step function as expected. One idea is to mock the optimizer step to affect some common state.
2. Add more tests that separately test the global hooks similar to the pre/post hooks today.

### Alternatives

Do nothing

### Additional context

see https://github.com/pytorch/pytorch/pull/119288#discussion_r1480405509

cc @vincentqb @jbschlosser @albanD @crcrpar
Repository: PowerToys
Issue: Unable to completely disable the Windows key. (Keyboard Manager)
Labels: Issue-Bug,Needs-Triage

### Microsoft PowerToys version

0.78.0

### Installation method

Microsoft Store

### Running as admin

Yes

### Area(s) with issue?

Keyboard Manager

### Steps to reproduce

![image](https://github.com/microsoft/PowerToys/assets/159171746/8a21b8f8-2ff8-4fd6-9514-a0fe98e97184)
![image](https://github.com/microsoft/PowerToys/assets/159171746/e6f529d4-54bd-4be4-af83-85440d02d0b3)
![image](https://github.com/microsoft/PowerToys/assets/159171746/6ca2c82d-9487-465d-920a-f672a683ff7d)


### ✔️ Expected Behavior

I wanted to disable the Windows key because my physical keyboard is malfunctioning, constantly pressing the key without my input. Therefore, I attempted the Win + disable commands to prevent it from functioning and activating on its own. I expected the typical Windows commands that activate the lock screen or open various menus to be disabled as well.

### ❌ Actual Behavior

The commands continue to activate; for example, when I press the L key, my computer locks as if executing the Win + L command, or if I press the G key, the Xbox Game Bar window opens as if executing the Win + G command. In summary, the Windows key continues to function as if pressed despite the modifications made with the keyboard manager.

### Other Software

_No response_
Repository: vscode
Issue: Option to show intellisense only after certain number of characters typed
Labels: feature-request,suggest

<!-- ⚠️⚠️ Do Not Delete This! feature_request_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- Please search existing issues to avoid creating duplicates. -->

<!-- Describe the feature you'd like. -->
Is there a way to only show intellisense/autocomplete suggestions after a certain number of characters have been typed? This has also been asked about four years ago (cf.  [stackoverflow](https://stackoverflow.com/q/61083829/13147660)}.

My use case is the following: I have a snippet and want to type exactly one character at a specific tabstop position. Then I want to use `Tab` to go to the next tabstop. Instead some unrelated suggestion gets executed. I think one character suggestions are often not helpful anyway because one cannot infer much from a single character.
Repository: PowerToys
Issue: Batch unzip archive files
Labels: Needs-Triage

### Description of the new feature / enhancement

Select multiple archive files eg .zip etc. right click, and in the context menu, select batch extract (for example) to extract all files to one or many folders eg named after each archive file

### Scenario when this would be used?

If you have many archive files and want to extract them all at once, instead of current windows behaviour, where you have to go through each on individually

### Supporting information

_No response_
Repository: Web-Dev-For-Beginners
Issue: ASSIGNMENTS QUERY
Labels: no-issue-activity

I don't see where we submit assignments for feedback
Repository: flutter
Issue: [Impeller] merge `impeller_golden_tests` into `impeller_unittests`
Labels: c: proposal,P3,e: impeller,team-engine,triaged-engine

Right now they are 2 different executables.  I think we should make them one executable where by default you get golden images, if you specify `--playground` you get an interactive window.

This will make the testing setup a bit more clear since there is large overlap between `impeller_golden_tests` and `impeller_unittests`

Some caveats:
- `impeller_golden_tests` needs a directory to put golden images into.
- `impeller_golden_tests` is only macOS now, we need to make sure we fail gracefully at runtime on other platforms.
- The behavior that decides between `impeller_golden_tests` and `impeller_unittests` today is a compile time decision.  We'd need to make it a runtime decision.

The benefits are just clarity and less duplicate work on CI.

cc @chinmaygarde
Repository: PowerToys
Issue: PT Run not displaying Shortcut Application for shortcuts using the obsidian protocol
Labels: Issue-Bug,Needs-Triage

### Microsoft PowerToys version

0.78

### Installation method

GitHub

### Running as admin

None

### Area(s) with issue?

PowerToys Run

### Steps to reproduce

Create a windows shortcut like: obsidian://open?path=C:%2Fmy%2Fpath

Put shortcut in an indexable directory.


### ✔️ Expected Behavior

The "Shortcut Application" should show up in the list with the obsidian icon.
It should be visible when you prefix your search with '.'

### ❌ Actual Behavior

Shortcut does not show up as a "Shortcut Application". 
Does not show up at all when prefixing search with a '.'

### Other Software

_No response_