Requirements contain conflicting indexes for package torch error when using extra markers in both the project and a dependency

I have the following setup in a monorepo:

projects/lib -> helper library which wraps PyTorch, with two extras torch-v1 and torch-v2
projects/app -> app which uses the lib and also PyTorch directly, with the same two extras torch-v1 and torch-v2

I tried to set up both projects with the markers, and the app -> lib dependency is also only encoded in the optional dependencies of app. E.g. in app/pyproject.toml:
[project.optional-dependencies]
torch-v1 = [
    "torch==1.12.1",
    "torchaudio==0.12.1",
    "lib[torch-v1]",
]
torch-v2 = [
    "torch~=2.3.1",
    "torchaudio~=2.3.1",
    "lib[torch-v2]",
]
[tool.uv]
conflicts = [
    [
      { extra = "torch-v1" },
      { extra = "torch-v2" },
    ],
]

[[tool.uv.index]]
name = "pytorch-cu113"
url = "https://download.pytorch.org/whl/cu113/"
explicit = true

[[tool.uv.index]]
name = "pytorch-cu121"
url = "https://download.pytorch.org/whl/cu121/"
explicit = true

[tool.uv.sources]
lib = { path = "../lib", editable = true }
torch = [
  { index = "pytorch-cu113", extra = "torch-v1", marker = "platform_system != 'Darwin'"},
  { index = "pytorch-cu121", extra = "torch-v2", marker = "platform_system != 'Darwin'"},
]
torchaudio = [
  { index = "pytorch-cu113", extra = "torch-v1", marker = "platform_system != 'Darwin'"},
  { index = "pytorch-cu121", extra = "torch-v2", marker = "platform_system != 'Darwin'"},
]

lib/pyproject.toml looks identical except for the lib dependency.
When I then try to sync app, I get:
$ uv sync --extra torch-v1
error: Requirements contain conflicting indexes for package `torch` in split `platform_system != 'Darwin'`:
- https://download.pytorch.org/whl/cu113/
- https://download.pytorch.org/whl/cu121/

I've uploaded a repro here: https://github.com/reuben/uv-pytorch-dep-bug-repro

Does it work if you remove the dependency on lib[torch-v1] and lib[torch-v2]? I think you have to also mark those as conflicting here but @BurntSushi would know.

Isn't it already marked as conflicting? The lib dep only appears inside the optional extras which are marked as conflicting. In the docs I understand that only extras and groups can be marked as conflicting, not dependencies directly.

Oh, sorry, forgot to answer the first question: yes, it does work without the lib dependency.

Does it work if you add this to your pyproject.toml for lib as well? (In addition to what you have in your root pyproject.toml.)
conflicts = [
    [
      { extra = "torch-v1" },
      { extra = "torch-v2" },
    ],
]

Basically, if you have those extras defined on lib as well, then those are separate extras from what's in your root pyproject.toml, and you need to tell uv that those conflict as well.
It's hard to say without an MRE though. Another thought is more of a hammer (and I guess this uses an undocumented feature), but to put this in your root pyproject.toml (and only this):
conflicts = [
    [
      { extra = "torch-v1" },
      { extra = "torch-v2" },
      { package = "lib", extra = "torch-v1" },
      { package = "lib", extra = "torch-v2" },
    ],
]

@BurntSushi there's no "root" pyproject.toml, I'm not using a workspace, just two projects that reference each other as path dependencies. lib already has the conflicts declared, see here: https://github.com/reuben/uv-pytorch-dep-bug-repro/blob/main/projects/lib/pyproject.toml
I'll try with the package conflict and report back.

With the package conflict declared I get a different error:
$ uv sync --extra torch-v1
error: Found conflicting extra `torch-v1` unconditionally enabled in `lib[torch-v1] @ file:///Users/reubenn/dev/personal/uv-pytorch-dep-bug-repro/projects/lib ; extra == 'torch-v1'`

This repo should be a MRE btw:  https://github.com/reuben/uv-pytorch-dep-bug-repro

Hmmm okay, but from what I can tell, you have the pyproject.toml that you shared and also another pyproject.toml for lib right? And both of those define torch-v1 and torch-v2 extras right? Those extras are technically distinct and uv needs to be taught that they are conflicting.
Also, the "hammer" I mentioned in my previous comment should be this (not what I wrote, which is wrong):
conflicts = [
    [
      { extra = "torch-v1" },
      { extra = "torch-v2" },
    ],
    [,
      { package = "lib", extra = "torch-v1" },
      { package = "lib", extra = "torch-v2" },
    ],
    [
      { extra = "torch-v1" },
      { package = "lib", extra = "torch-v2" },
    ],
    [
      { extra = "torch-v2" },
      { package = "lib", extra = "torch-v1" },
    ],
]

Basically, you have to tell uv about all of the different conflicting interactions. For example, without the above, uv doesn't know that app[torch-v1] and lib[torch-v2] can't be combined.

With the corrected hammer I still get the new error:
$ uv sync --extra torch-v2
error: Found conflicting extra `torch-v1` unconditionally enabled in `lib[torch-v1] @ file:///Users/reubenn/dev/personal/uv-pytorch-dep-bug-repro/projects/lib ; extra == 'torch-v1'`

I can always re-export torch from lib, so I've reduced the test case even further by eliminating the app -> torch dependency. It's pushed to this branch here: https://github.com/reuben/uv-pytorch-dep-bug-repro/tree/more-minimaler
This is what I have:
.
├── README.md
└── projects
    ├── app
    │   └── pyproject.toml
    └── lib
        └── pyproject.toml

app/pyproject.toml is:
[project]
name = "app"
version = "0.1.0"
description = "Add your description here"
readme = "README.md"
requires-python = ">=3.10.15"
dependencies = []

[project.optional-dependencies]
torch-v1 = [
    "lib[torch-v1]",
]
torch-v2 = [
    "lib[torch-v2]",
]

[tool.uv]
conflicts = [
    [
      { extra = "torch-v1" },
      { extra = "torch-v2" },
    ],
]
[tool.uv.sources]
lib = { path = "../lib", editable = true }

and lib/pyproject.toml is:
[project]
name = "lib"
version = "0.1.0"
description = "Add your description here"
readme = "README.md"
requires-python = ">=3.10.15"
dependencies = []

[project.optional-dependencies]
torch-v1 = [
    "torch==1.12.1",
    "torchaudio==0.12.1",
]
torch-v2 = [
    "torch~=2.3.1",
    "torchaudio~=2.3.1",
]

[tool.uv]
conflicts = [
    [
      { extra = "torch-v1" },
      { extra = "torch-v2" },
    ],
]

[[tool.uv.index]]
name = "pytorch-cu113"
url = "https://download.pytorch.org/whl/cu113/"
explicit = true

[[tool.uv.index]]
name = "pytorch-cu121"
url = "https://download.pytorch.org/whl/cu121/"
explicit = true

[tool.uv.sources]
torch = [
  { index = "pytorch-cu113", extra = "torch-v1", marker = "platform_system != 'Darwin'"},
  { index = "pytorch-cu121", extra = "torch-v2", marker = "platform_system != 'Darwin'"},
]
torchaudio = [
  { index = "pytorch-cu113", extra = "torch-v1", marker = "platform_system != 'Darwin'"},
  { index = "pytorch-cu121", extra = "torch-v2", marker = "platform_system != 'Darwin'"},
]

uv sync in lib works fine, but when I try to uv sync --extra torch-v1 in app, I get the original error:
$ uv sync --extra torch-v1
error: Requirements contain conflicting indexes for package `torch` in split `platform_system != 'Darwin'`:
- https://download.pytorch.org/whl/cu113/
- https://download.pytorch.org/whl/cu121/
- ```

OK I was finally able to get something working by using the global PyTorch index and using local version specifiers to narrow down to a specific CUDA version, and limiting resolution to Linux environments only. It's not super clean but it works:
lib/pyproject.toml:
[project]
name = "lib"
version = "0.1.0"
description = "Add your description here"
readme = "README.md"
requires-python = ">=3.10.15"
dependencies = []

[project.optional-dependencies]
torch-v1 = [
    "torch==1.12.1+cu113",
    "torchaudio==0.12.1+cu113",
]
torch-v2 = [
    "torch==2.3.1+cu121",
    "torchaudio==2.3.1+cu121",
]

[tool.uv]
environments = ["platform_system == 'Linux'"]
conflicts = [
    [
      { extra = "torch-v1" },
      { extra = "torch-v2" },
    ],
]

[[tool.uv.index]]
name = "pytorch-all"
url = "https://download.pytorch.org/whl/"
explicit = true

[tool.uv.sources]
torch = [
  { index = "pytorch-all" },
]
torchaudio = [
  { index = "pytorch-all" },
]

app/pyproject.toml:
[project]
name = "app"
version = "0.1.0"
description = "Add your description here"
readme = "README.md"
requires-python = ">=3.10.15"
dependencies = []

[project.optional-dependencies]
torch-v1 = [
    "lib[torch-v1]",
    "torch==1.12.1+cu113",
    "torchaudio==0.12.1+cu113",
]
torch-v2 = [
    "lib[torch-v2]",
    "torch==2.3.1+cu121",
    "torchaudio==2.3.1+cu121",
]

[tool.uv]
environments = ["platform_system == 'Linux'"]
conflicts = [
    [
      { extra = "torch-v1" },
      { extra = "torch-v2" },
    ],
]

[[tool.uv.index]]
name = "pytorch-all"
url = "https://download.pytorch.org/whl/"
explicit = true

[tool.uv.sources]
lib = { path = "../lib", editable = true }
torch = [
  { index = "pytorch-all" },
]
torchaudio = [
  { index = "pytorch-all" },
]

Agree that there's a bug here.

Maybe multiple bugs? Not sure.


pastis patch for honggfuzz

This is not my patch, it is functionality added by Quarkslab for their fuzzer orchestration.
It adds two optional features:

dynamic input directory (e.g. a libafl/AFL/libfuzzer corpus directory) from which periodically new inputs are read from
a statistic file that prints some data that can be parsed by scripts

IMHO these are very useful features that would improve fuzzing with honggfuzz, so I would like to see them added.
The current method without such a patch is to stop honggfuzz, copy the honggfuzz corpus and the other fuzzer corpus directory together, minimize and restart with that as new inputs. which cost a lot of time and reduces effectiveness in a fuzzing campaign and hence why especially this feature is so useful.
If necessary I can add text to the README etc.

Hi! I implemented those features for PASTIS (dynamic-input and stats file). I did not made a PR because I was having compiling issues on Honggfuzz master (issue #497).
I could make the PR if these change are going to be merged.

Nice!

@cnheitman Would you like to submit something on top of this PR, or instead (or modifying) this PR?

@robertswiecki I would like to keep the authorship of the PR since I wrote those feature for Quarkslab.
Can we do something about it? Like revert an make the PR from my fork?

@cnheitman - my mistake, reverted

Thanks! I'll make the PR now.

Done. Thanks @robertswiecki. And thanks @vanhauser-thc for bringing this patch up.

@robertswiecki rewriting history is not nice to people :)

@donpiekarz Correct, I'll do better in the future!


ARROW-726: [C++] Fix segfault caused when passing non-buffer object to arrow::py::PyBuffer

This leads to calling Py_DECREF on a null pointer

rebased

triggered build


Spacing between window boarder and text.

Is there a setting where I could make the boarder between the window and the start of the text smaller?
See screen shot:

There seem to be a line or so around the text which makes the window a little bigger than it needs to be.

https://github.com/gnotclub/xst/blob/master/doc/Xresources#L12

@actionless Thank you. Curse Dyslexia!


Upgrade to 4.2.0

Upgrade of camel to 4.2.0 using staging repo.

The failure in the example project is expected until Quarkus releases with an upgraded k8s client. I'll disable the native profile in that project after this PR is merged.

Quarkus 3.6.0.CR1 will be out later today. It'll be cleaner if we tack that upgrade onto this PR (which I can do).

Actually, I'll follow up with the Quarkus upgrade in another PR as there's potentially some native memory usage issues with the k8s client to work through.


Overview

What
Added two changes to the overview:

an indicator to show if there are pending updates and if there are, how many; and
the  local network IP address of the Raspibolt.

Why
The updates indicator is important, so that it is clear if there are updates pending, which should be applied asap. The second change is a convenience change.
How
Updated the bash file.
Scope
[ ] significant change to core configuration
[x] independent bonus guide
[ ] simple bug fix
Fixes # (link issue)
Test & maintenance
Both changes are visible in the overview. It should not require maintenance, but if so I'm willing to do it.
Animated GIF (optional)

The current system overview script is now this one:
https://github.com/raspibolt/raspibolt-pulse

@Mynima Yes, it is possible to use set it up without sudo. A restricted user van be created who can only run specific commands using sudo without needing a password for those commands. This is achieved by editing the sudoers file. This does make the setup more complicated.
BTW I have made this change and others to the new script as remarked by @zynos (https://github.com/raspibolt/raspibolt-pulse).


Incompatible with Iris

When shaders are enabled, particles don't render at all.

I can't replicate this:
https://github.com/fzzyhmstrs/pc/assets/72876796/86581867-9590-47ce-a309-c0d862f1a260

I'm using the iris distant horozons beta, maybe it's just an iris issue?

Fixed on 1.21


GWT - Calculate MD5 hash while copying file to avoid errors on very large files

For GWT, we calculate an MD5 hash of the file contents of each asset to assist browsers with caching. The current implementation reads the file fully into a byte array, calculates the MD5, adds it to the destination file name, then copies the source file to the output path. While this is fine for small files (albeit inefficient as we read each file multiple times), it can cause problems with large files.
This proposed implementation instead calculates the MD5 while reading the source InputStream for the copy operation, then renames (moves) the file to include the MD5 hash.
Refer to #4314 for the original addition, and #6848 for the problem this solves.

@MobiDevelop amazing! Have you noticed any gain on speed during GWT build?

A note on the build failure, oddly it has started picking up the emulation of File which doesn't have the getPath() method. Weird that it never did that while I was coding it, and now it is. I wouldn't think the compile time generator would be using the emulation classes - but GWT magic is the darkest of magic.

Thank you for the PR @MobiDevelop! I will try to test this new logic with the asset I was trying before and check how it performs.


Add Missing Java EEE translation to Moment's "ddd"

According to the Java Doc for SimpleDateFormat, we should support EEE for day of week. This translates to "ddd" in moment.

thanks a lot @stephenliang, this seems like a valuable addition :+1: I'll leave it open for @MadMG to  merge the PR tough, because he's also the one who can release a new version over at npm.

thx  @stephenliang! as @herom said, it is a valuable addition!


Anerdw: Watch udyam.webinfinix.in

Anerdw requests the watch of the watch_keyword udyam\.webinfinix\.in. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

udyam\.webinfinix\.in has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by Dragonrage in Charcoal HQ. Reason: 'watching domain instead of subdomain. -> webinfinix.in'


Projekt DataImports and Monte Carlo Method

This is the result of our Projekt.

Pull Request Test Coverage Report for Build 457

103 of 124 (83.06%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 88.1%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

viroconcom/dataNDBC.py
48
51
94.12%

viroconcom/dataECMWF.py
5
10
50.0%

viroconcom/distributions.py
22
35
62.86%

Totals

Change from base Build 455:
-0.4%

Covered Lines:
1377

Relevant Lines:
1563

💛  - Coveralls

Thanks for the pull request @adrdrew ! As this is a pretty big pull request with lots of functionality, I'll have a first discussion with @kawindme about it. Congrats on your project!

Moved to branch  direct_sampling_and_imports, as that makes it easier to modify.


Fix wasmer url

Summary

Fix the suggestion for wasmer installation
Closes #6

Thanks for the fix @maekawatoshiki .
Merging!


Compact penalty maybe overwhelmed elsewhere in render pipeline

From measurements at https://aisleriot.jovial.com/, it looks like any overhead on the Dart side is being overwhelmed by things happening elsewhere in the graphics pipeline, at least on slower hardware (my tablet and phone).  I suspect, therefore, that the time penalty of the compact representation may be insignificant, most of the time.
The spec should have some informative documentation saying so.

Release 1.0.5


disable flycheck

I didn't see an obvious way to configure rustic to disable flycheck -- is there a way to do that? (If so, perhaps it should be documented.) I've noticed that editing is fairly laggy and I (perhaps incorrectly) attribute this to flycheck.

Yeah I have to remove flycheck finally. I wanted to wait until I have something that also works with rust-analyzer, in case somebody is actually using the flycheck related code in rustic(that's actually mostly the code from flycheck-rust.
I will remove it completely the next days and replace it with updated code if I have something that works.
For anybody who is curious, or even better motivated to help me =) -> https://github.com/emacs-lsp/lsp-mode/pull/1179
lsp-mode currently isn't working with rust-analyzer and flycheck(it is for rls).

@brotzeit any tips how I could remove flycheck in the meantime as a workaround?

Oh, sorry. This should work:
(require 'rustic)
(remove-hook 'rustic-mode-hook 'flycheck-mode)

I just noticed that I forgot to turn off flymake in my config. Maybe you also have flymake activated ? I don't know if flycheck is the reason for the lags you are experiencing, but it's possible that it is related to some other package(s).

I'm not even sure what flymake is :)

but it does seem to be enabled, or at least it appears in the mode line

@brotzeit how do I remove flymake mode?

You can't, it's a builtin mode.

@brotzeit ok but there must be some way to make it stop executing? I added (flymake-mode 0) into my rustic hook, but that doesn't seem to do the trick. I can't quite figure out why it is even starting in the first place.

Ah, I think eglot is starting it.

Yeah, I think this was turned on because eglot works better with flymake(the author of eglot is also the maintainer of flymake).
You can disable flymake this way (add-hook 'eglot--managed-mode-hook (lambda () (flymake-mode -1))) (https://github.com/joaotavora/eglot/issues/123#issuecomment-444104870)
I will document it in rustic.

Is it possible to use rustic with the built-in flymake and eglot instead of flycheck?

Last time I checked it still worked https://github.com/brotzeit/rustic#client but wouldn't be the first time that stuff suddenly stops working. But in this case I assume there would already be some complaints.

See this thread about what I experienced trying to use rustic-mode with eglot and flymake-mode turned on: https://github.com/rust-lang/rust-mode/issues/484#issuecomment-1372525271
Let me know if you'd like to see my emacs config. I can share it here with what I have configured. I just have to push my latest dots to my git repo today:
https://git.sr.ht/~whereiseveryone/confetti


Better check in getObjectProperty()

Added two more checks to ensure that obj is not undefined or null. If either are the case, the function will error without these checks.

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Throwing an error is part of the API. This change will break things that expect the exception to be thrown - you can see the unit tests are failing as a result of this PR.
If anything, the docs for the function need updating to explain how/why the exception will be thrown and what it signifies.

Got it. Yeah - I think the docs could use some work, all around. Otherwise Node-RED is awesome! Thanks for taking the time to look at this.

we take PRs to docs as well ;-)

Where is that at? On https://nodered.org/docs/?

The main docs site is https://github.com/node-red/node-red.github.io/tree/master/docs

Right, but this function isn't documented on the website. I was referring to the doc in the source code for this function.


samsung-grandma: Add UCM configuration for Samsung Galaxy Grand Max

Use case configuration for Samsung Galaxy Grand Max. No SecondaryMic.
Speaker connected to external codec (TFA9895)
[   25.104571] qcom-apq8016-sbc 7702000.sound: ASoC: driver name too long 'samsung-grandmax' -> 'samsung-grandma'

I guess we need a Linux device tree patch for this as well?

I prefer samsung-grandmax instead of samsung-grandma, so I would like to wait for patches for qcom-apq8016-sbc.

AFAIK this is a limitation of the ASoC/ALSA API in the Linux kernel and there is no trivial way to fix this. If you don't like Samsung's grandma then you'd need to pick another name that is shorter :D


signald: use jre_headless

There is no GUI component

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 125900 at 2aa09d08 run on aarch64-linux 1

1 package failed to build:

signald

2 suggestions:

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild and runHook postBuild.
Near pkgs/applications/networking/instant-messengers/signald/default.nix:54:3:
   |
54 |   buildPhase = ''
   |   ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/networking/instant-messengers/signald/default.nix:63:3:
   |
63 |   installPhase = ''
   |   ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

Thanks!
On Wed, 9 Jun 2021, 6:43 pm Sandro, @.***> wrote:

Merged #125900 https://github.com/NixOS/nixpkgs/pull/125900 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/125900#event-4864186664, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRJXD7W24AWD55W2O6B4TTR5AWXANCNFSM46FMG4XQ
.


[v1] useForm is unstable

Describe the bug and the expected behavior
const [form, fields] = useForm({});

useEffect(() => {
  console.log('form');
}, [form])

useEffect(() => {
  console.log('fields');
}, [fields])

const [foo, setFoo] = useState(1);

useEffect(() => {
  setInterval(() => {
    setFoo((foo) => foo + 1);
  }, 1000)
}, [])

Every time foo is updated, form and fields values mutate as well.
Conform version
v1.0.0
Steps to Reproduce the Bug or Issue
N/A
What browsers are you seeing the problem on?
No response
Screenshots or Videos
No response
Additional context
No response

This is no different from v0. The form and fields object are never stable and is unlikely to happen due to mixed sources (across networks / formdata). Feel free to share if you have a specific use case.

Hm. Not even sure how to solve this.
My use case is an article generator.
The article is generated in a <textarea />.
As it is being generated, I use SSE to poll updates.
It turns out, this means that every time there is a new SSE event, the entire form is re-rendered.
So it seems there is no way to update then form value without causing the form to re-render.
I will need to think of other ways to implement this.

Would it be possible to at least make the useInputControl stable?
const articleOutlineMarkdownControl = useInputControl(articleOutlineMarkdown);

const [outlineValue, setOutlineValue] = useState(
  articleProject.articleOutlineMarkdown,
);

useEffect(() => {
  articleOutlineMarkdownControl.change(outlineValue);
}, [articleOutlineMarkdownControl, outlineValue]);

At the moment, I cannot seem to figure out how to even implement this pattern given that the articleOutlineMarkdownControl updates on every cycle.

I realized I can just skip articleOutlineMarkdownControl from useEffect dependencies.
useEffect(() => {
  articleOutlineMarkdownControl.change(outlineValue);
// `articleOutlineMarkdownControl` is not stable.
// https://github.com/edmundhung/conform/issues/377#issuecomment-1889524894
}, [outlineValue]);

It is technically wrong, but it will work and will achieve the desired result.

The control object won't be stable because the value will keep changing. But control.change is stable. So feel free to put it to the dependency of useEffect.


[VM - FFI] Optimize Pointer<T>.asTypedList()

This change refactors _asExternalTypedData into multiple functions,
which are data type specific, e.g _asExternalTypedDataInt8.
These functions are implemented as recognized methods.
Argument checks have previously been performed in the removed
runtime entry implementation of _asExternalTypedData. These are
now handled in Dart.
Closes https://github.com/dart-lang/sdk/issues/39843

Review in https://dart-review.googlesource.com/c/sdk/+/221360

@mit-mit
I'm waiting on this CL, so I can rebase and push my changes which address the feedback.

Got it, looks like that just got merged.


Tutorial steps are incorrect in 'Publish a Jekyll site to Azure Static Web Apps' page

After the line - uses: actions/checkout@v2 add the following configuration block.

If this step is followed exactly, then the resulting YAML file is invalid. Since the text should be inserted after the entire YAML block.
 - uses: actions/checkout@v2
   // the instructions tell uers to insert the text block here
    with:
      submodules: true
   // but it should be here

The YAML should look like this:
      - uses: actions/checkout@v2
        with:
            submodules: true
      - name: Set up Ruby
        uses: ruby/setup-ruby@ec106b438a1ff6ff109590de34ddc62c540232e0
        with:
          ruby-version: 2.6
      - name: Install dependencies
        run: bundle install
      - name: Jekyll build
        run: jekyll build

But users end up with this (invalid):
      - uses: actions/checkout@v2
      - name: Set up Ruby
        uses: ruby/setup-ruby@ec106b438a1ff6ff109590de34ddc62c540232e0
        with:
          ruby-version: 2.6
      - name: Install dependencies
        run: bundle install
      - name: Jekyll build
        run: jekyll build
        with:
            submodules: true

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d1532547-d8ef-bdea-4164-4f617a249eb5
Version Independent ID: f8a21eb1-662f-13f3-dd9d-9ed2962e740d
Content: Tutorial: Publish a Jekyll site to Azure Static Web Apps
Content Source: articles/static-web-apps/publish-jekyll.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Hello @alexweininger, Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

Please note that uses: ruby/setup-ruby@ec106b438a1ff6ff109590de34ddc62c540232e0 is obsolete, see #68123

Please note that uses: ruby/setup-ruby@ec106b438a1ff6ff109590de34ddc62c540232e0 is obsolete, see #68123

thanks, @alexweininger - This article has changed a few times since this issues was crated. I just went through each of the steps and verified they work. I am going to move to #please-close this issue for now, but please feel free to reopen this issue in the future if you think more changes are required.


fix: feature flags are passed correctly

This fixes how the feature flags are passed into the trigger queueing logic, which should help with the failing test

cc @yijie-04


Changing date is one month off

For instance, if the date is currently 03/10/1992, and I change it to February 20, 1990 by clicking in the datepicker, it becomes 01/20/1990.

It's actually only if it's a date before 1993. Anything post '93 works.

Mistake on my end, sorry!


Manage dark theme in Qwik

For now there is not any reference in docs that how to add support of dark theme. Here I add that part.
Overview

What is it?

[x] Feature / enhancement
[ ] Bug
[ ] Docs / tests / types / typos

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Added dark theme support

Thanks for this great contribution 👍
This scenario fits better in the cookbook section
Can you move your example there please?


Update on test

Hello,
I would like to know if this project is dead :/
I updated the dependencies and made two  separated tests for the native and mongoose drivers.
There was also a bug on the next function.
There only a single test failing for the mongoose driver although I don't think it is crucial.
Best regards,
Andre

Thank you so much, I will implement it in my project .


Update stopwords.txt

Description
In brazillian portuguese the conjuntion "em(preposition)+<a|o|as|os>(article)" take the form "na, nas, no, nos" being commom stop words.
For some reason the "nas" conjunction appear twice and the "no" is nowhere to be found, I think it was probably a mistake.
This pull request add the word to the list and remove the duplication.
For reference the same word can be found on the Portuguese stopwords.txt on line 34
Fix #14065

While its a simple change, it does change the analysis chain. I wonder if it should stick to Lucene 11 (admittedly, that will not be shipped for a LONG time).
I wonder what others think on if we should backport to 10.2?

Is there something I can investigate further? I tried parsing the error and it seems to relate to a mn_MN dictionary that refers to Mongolian 🤔
While checking /home/runner/work/lucene/lucene/lucene/analysis/common/build/hunspell-regressions/libreoffice/mn_MN/mn_MN.aff:
  2> java.lang.NumberFormatException: For input string: "ээв$"
  2> 	at java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:67)
  2> 	at java.base/java.lang.Integer.parseInt(Integer.java:662)
  2> 	at java.base/java.lang.Integer.parseInt(Integer.java:778)
  2> 	at org.apache.lucene.analysis.hunspell.Dictionary.parseNum(Dictionary.java:594)
  2> 	at org.apache.lucene.analysis.hunspell.Dictionary.readAffixFile(Dictionary.java:408)
  2> 	at org.apache.lucene.analysis.hunspell.Dictionary.<init>(Dictionary.java:264)
  2> 	at org.apache.lucene.analysis.hunspell.TestAllDictionaries$1.<init>(TestAllDictionaries.java:75)
  2> 	at org.apache.lucene.analysis.hunspell.TestAllDictionaries.loadDictionary(TestAllDictionaries.java:75)
  2> 	at org.apache.lucene.analysis.hunspell.TestAllDictionaries.lambda$testDictionariesLoadSuccessfully$7(TestAllDictionaries.java:165)
  2> 	at org.apache.lucene.analysis.hunspell.TestAllDictionaries.lambda$testDictionariesLoadSuccessfully$8(TestAllDictionaries.java:187)
  2> 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:317)
  2> 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144)
  2> 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642)
  2> 	at java.base/java.lang.Thread.run(Thread.java:1583)

I am unable to understand how my change could impact the analysis on the Mongolian language.

I am unable to understand how my change could impact the analysis on the Mongolian language.

I don't understand the connection to your change either, but it looks to me as if Mongolian is coming in via the randomized test framework, which sets the active Java Locale to something random. If you were to capture the full "reproduce with" command line that is output by the test framework when there is a failure, we could tell if that's what's going on. Are you able to reproduce the failure?

It fails again. maybe problem comes from https://github.com/LibreOffice/dictionaries/commit/d1696029d8923ae697cb2d6d4d7d69791b1943f2 ?

And yeah i see the commit date, but that's not the push date. So I suspect this issue has nothing to do with your PR and may fail all PRs until we address it.

If you were to capture the full "reproduce with" command line that is output by the test framework

You mean this?
Reproduce with: gradlew :lucene:analysis:common:test --tests "org.apache.lucene.analysis.hunspell.TestAllDictionaries.testDictionariesLoadSuccessfully" -Ptests.jvms=2 -Ptests.jvmargs= -Ptests.seed=18B87D355DA5CF0F -Ptests.gui=true -Ptests.file.encoding=ISO-8859-1 -Ptests.vectorsize=default -Ptests.forceintegervectors=false

I'm happy to try to debug this but it might be a few days. Issue may be with REP rules in the referenced commit.
the way these rules work are:
REP 3619
REP a а
REP c с
... (3617 more times)

if the count (3619) is incorrect, then parser might try to parse a rule (such as REP ээв$ ээ_вэ) as a another count, leading to the NumberFormatException. maybe the C hunspell parser is lenient, so nobody has yet noticed. In my experiences the dictionaries often have such problems, hence the testing here.
It could also be some other bug in the code, this is just what it looks like to me.

Yes, that's it. the REP 3619 should be changed to REP 3621. I guess we could send the PR to libreoffice, since the "upstream" dictionary looks totally different here: https://github.com/bataak/dict-mn/blob/main/mn_MN/mn_MN.aff#L49

I will send them a one-liner PR explaining the situation, we can take it from there.
We may want to separately try to be more lenient about this part of the parsing. Have not looked at the C implementation, but I'm guessing the numbers might be used as more of a "hint" to size hashtables or something, and not really a hard rule?

https://github.com/LibreOffice/dictionaries/pull/46

https://github.com/LibreOffice/dictionaries/pull/46

Their contribution process seem to be elsewhere, but I could not understand it fully 😅
I tried sending the issue to their IRC channel, hope someone sees it.

Thank you @eusousu
I tried to make some progress, for now at least I have an open bug report: https://bugs.documentfoundation.org/show_bug.cgi?id=164366

I took a stab at hacking around this on our side as well: https://github.com/apache/lucene/pull/14079

See https://github.com/LibreOffice/dictionaries/commit/c3ff53711dcac4bdec24f23a2c1f9712a0833b67
Sorry for all the unrelated noise here on your PR!

I have an open bug report

@rmuir they asked me about a license statement on the IRC chat that I think is necessary for them to accept your contribution

@eusousu I sent the email. I will also try to chase down the issue in the upstream bataak repo later, but it should not be an  issue anymore. Thanks for the help and patience!

@eusousu if you want, add entry to lucene/CHANGES.txt for your fix. Otherwise I'm happy to do it for you, you've done plenty to get this fix in, I want to merge it.

@rmuir sorry I was unaware of this requirement. Added the entry, I really hope I got the pattern right 😅
I really appreciate your work and patience, thank you 😄


feat: abstract modules for reuse

currently chain modules are serviced only.
see how I changed hermes modules to be reusable not only in serviced.
so can make abstract module config per existing chain, and just reuse in serviced.
so can configure later shells/devnets/hermes/testnets/microvm/process-compose/dockers etc - whatever can be derived.
so request is to split chain modules in repo in 2 options parts, one is abstract for process/node, and other is wrapper for systemd.
so that can build modules on top - using lowest layer as basic config abstraction.

I will close for now. We have example of hermes as module for serviced vs just to generate toml. if somebody will want to do chains - can do it. So I am using process-compose directly, so not up to that in nearest time. New task can be opened or this reopend.
May be with cosmosd-sdk 1.0 release when all is settled.

could start making more sense if start sharing oci containers https://github.com/informalsystems/cosmos.nix/pull/231#issuecomment-2004615830


is there any way to generate document?

I always use schema and generate types from that file but for client-side type generation in graphql-code-generator it needs document file. https://graphql-code-generator.com/docs/getting-started/documents-field
what I technically need are all possible queries, mutations, and subscriptions for client-side use.

I think that might not be the purpose of this lib. The code generator works very well for me.

IIUC gql code generator refers to graphql SDL files as “documents”. Nexus can generate one if you want. Look under outputs config of nexus.
Sent with GitHawk

could you please show me the link to that part of the documentation?
there is nothing related to output in Type Generation or makeSchema

Sorry, I think @jasonkuhrt is mistaken, the "documents" graphql code generator is looking for are the client side queries & mutations for your client-side, contained either in .graphql files or in tagged template literals - Nexus has no opinions on how and where these are created, it's up to your client-side application.

I found a library that do the job gql-generator

@tgriesser Thanks for the correction


Convert maven_jar to jvm_maven_import_external

Saw it’s in draft so I’m waiting your ping to review
On Mon, 2 Dec 2019 at 19:59 Jin<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/bazelbuild/rules_scala/pull/889
Commit Summary

Convert maven_jar to jvm_maven_import_external

File Changes

M WORKSPACE
https://github.com/bazelbuild/rules_scala/pull/889/files#diff-0 (32)
M test/src/main/scala/scalarules/test/scala_import/BUILD
https://github.com/bazelbuild/rules_scala/pull/889/files#diff-1 (20)
M test_expect_failure/scala_import/BUILD
https://github.com/bazelbuild/rules_scala/pull/889/files#diff-2 (8)

Patch Links:

https://github.com/bazelbuild/rules_scala/pull/889.patch
https://github.com/bazelbuild/rules_scala/pull/889.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/889?email_source=notifications&email_token=AAKQQF4AS5YJSRTSIBYEVWTQWVEI5A5CNFSM4JTZTNM2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H5MBJLA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKQQF6FQSF462DR7NRHL23QWVEI5ANCNFSM4JTZTNMQ
.

--
Ittai Zeidman
Cell:<PHONE_NUMBER>
40 Hanamal street, Tel Aviv, Israel
http://www.wix.com

Ready for review.


Cannot sort by #

@danielfm Is it just me or it doesn't allow to order by track number after listing an album?

The track number isn't sortable at this moment, but this can be easily changed. I'll add this to #37

That'd be great! Thanks Daniel!


Supermium is not virus, just browser. Same issue will happends to you when you use any other browser to watch porn during school.
Btw, where do you live If these days you are going to school?

wtf this is spam
though it did give me a good chuckle lol

Trolling/spam post. Go try more suited websites for this kind of stuff, kid.

Great post

This issue tracker has enough real issues as it is, so I won't allow it be clogged with this junk. But still better than the impersonators.

 Still really funny to read tho 😂


[Feature] Terraform: add widget_type validation support for widget_log_table

This PR adds "logger.log-table-widget" / "widget_log_table" to the WidgetTypes map.
The validation for widget types is missing the widget log table type and as such, when converting a dashboard that contains this widget type to a terraform file, it fails with the following error:
FATAL unrecognized widget type "logger.log-table-widget"
Support for this widget type has been added recently to terraform-provider-newrelic here: https://github.com/newrelic/terraform-provider-newrelic/issues/1831
Solves #1328

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Looks good 👍 thanks a lot for your PR @nunopi
We can merge this in once you sign our CLA: https://github.com/newrelic/newrelic-cli/pull/1339#issuecomment-1195099391

thanks @kidk I've signed the CLA


Pragmatic Microservices talk by Ondro at CodeOne 2019

Conference: Oracle Code One 2019
Conference website and social media links:
https://www.oracle.com/code-one/
@OracleCodeOne
Talk name: Pragmatic Microservices in Kubernetes with MicroProfile
Talk time: Tuesday, September 17, 08:45 AM
Talk Link: https://events.rainfocus.com/widget/oracle/oow19/catalogcodeone19?search=TUT2500
Speakers: Ondro Mihalyi, @OndroMih, Payara Senior Engineer
Tweet: Join @OndroMih at a tutorial about turning a monolith to #Microservices running in #Kubernetes with @MicroProfile @OracleCodeOne: https://events.rainfocus.com/widget/oracle/oow19/catalogcodeone19?search=TUT2500 #Payara #opensource #BreakNewGround
Pictures of the speaker: https://blog.payara.fish/hubfs/Ondrej-1.jpg

Pictures of speakers:

Ondro: https://blog.payara.fish/hubfs/Ondrej-1.jpg
Fabio: https://blog.payara.fish/hubfs/Fabio.jpg

@OndroMih,
you need to submit each session by 1 ticket. re-do  by copy paste. hug you super soon, tonight!

OK, I'll do, @aeiras, thanks!

I kept only one session here and moved other sessions to #198, #199 and #200


Leaks from Shaders on shutdown

Environment

OS Version: Ubuntu 20.04
Source or binary build?
main

Steps to reproduce
Running INTEGRATION_camera in the Vulkan branch reveals the following leaks:
ERROR: [Validation] Code 0 : Validation Error: [ VUID-vkDestroyDevice-device-00378 ] Object 0: handle = 0x55f71e464fa0, type = VK_OBJECT_TYPE_DEVICE; Object 1: handle = 0x74d977000000004a, name = HeatSignatureVS_VK, type = VK_OBJECT_TYPE_SHADER_MODULE; | MessageID = 0x71500fba | OBJ ERROR : For VkDevice 0x55f71e464fa0[], VkShaderModule 0x74d977000000004a[HeatSignatureVS_VK] has not been destroyed. The Vulkan spec states: All child objects created on device must have been destroyed prior to destroying device (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/linux/1.2-extensions/vkspec.html#VUID-vkDestroyDevice-device-00378)
16:27:55: ERROR: [Validation] Code 0 : Validation Error: [ VUID-vkDestroyDevice-device-00378 ] Object 0: handle = 0x55f71e464fa0, type = VK_OBJECT_TYPE_DEVICE; Object 1: handle = 0x67022e000000004b, name = HeatSignatureFS_VK, type = VK_OBJECT_TYPE_SHADER_MODULE; | MessageID = 0x71500fba | OBJ ERROR : For VkDevice 0x55f71e464fa0[], VkShaderModule 0x67022e000000004b[HeatSignatureFS_VK] has not been destroyed. The Vulkan spec states: All child objects created on device must have been destroyed prior to destroying device (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/linux/1.2-extensions/vkspec.html#VUID-vkDestroyDevice-device-00378)
16:27:55: ERROR: [Validation] Code 0 : Validation Error: [ VUID-vkDestroyDevice-device-00378 ] Object 0: handle = 0x55f71e464fa0, type = VK_OBJECT_TYPE_DEVICE; Object 1: handle = 0x74ea280000000054, name = selection_buffer_vs_VK, type = VK_OBJECT_TYPE_SHADER_MODULE; | MessageID = 0x71500fba | OBJ ERROR : For VkDevice 0x55f71e464fa0[], VkShaderModule 0x74ea280000000054[selection_buffer_vs_VK] has not been destroyed. The Vulkan spec states: All child objects created on device must have been destroyed prior to destroying device (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/linux/1.2-extensions/vkspec.html#VUID-vkDestroyDevice-device-00378)
16:27:55: ERROR: [Validation] Code 0 : Validation Error: [ VUID-vkDestroyDevice-device-00378 ] Object 0: handle = 0x55f71e464fa0, type = VK_OBJECT_TYPE_DEVICE; Object 1: handle = 0x6871490000000055, name = selection_buffer_fs_VK, type = VK_OBJECT_TYPE_SHADER_MODULE; | MessageID = 0x71500fba | OBJ ERROR : For VkDevice 0x55f71e464fa0[], VkShaderModule 0x6871490000000055[selection_buffer_fs_VK] has not been destroyed. The Vulkan spec states: All child objects created on device must have been destroyed prior to destroying device (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/linux/1.2-extensions/vkspec.html#VUID-vkDestroyDevice-device-00378)
16:27:55: ERROR: [Validation] Code 0 : Validation Error: [ VUID-vkDestroyDevice-device-00378 ] Object 0: handle = 0x55f71e464fa0, type = VK_OBJECT_TYPE_DEVICE; Object 1: handle = 0x603cd90000000044, name = ThermalCameraVS_VK, type = VK_OBJECT_TYPE_SHADER_MODULE; | MessageID = 0x71500fba | OBJ ERROR : For VkDevice 0x55f71e464fa0[], VkShaderModule 0x603cd90000000044[ThermalCameraVS_VK] has not been destroyed. The Vulkan spec states: All child objects created on device must have been destroyed prior to destroying device (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/linux/1.2-extensions/vkspec.html#VUID-vkDestroyDevice-device-00378)
16:27:55: ERROR: [Validation] Code 0 : Validation Error: [ VUID-vkDestroyDevice-device-00378 ] Object 0: handle = 0x55f71e464fa0, type = VK_OBJECT_TYPE_DEVICE; Object 1: handle = 0x8cc1d80000000045, name = ThermalCameraFS_VK, type = VK_OBJECT_TYPE_SHADER_MODULE; | MessageID = 0x71500fba | OBJ ERROR : For VkDevice 0x55f71e464fa0[], VkShaderModule 0x8cc1d80000000045[ThermalCameraFS_VK] has not been destroyed. The Vulkan spec states: All child objects created on device must have been destroyed prior to destroying device (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/linux/1.2-extensions/vkspec.html#VUID-vkDestroyDevice-device-00378)

Although it would look like Vulkan errors; these errors are very likely happening because Ignition is holding to a SharedPtr after OgreNext has been shutdown; and it would affects all APIs (it's just happens that Vulkan layers caught it).
These modules all come from low level shaders, so it's most likely coming from either Ogre2Material or the Cameras (e.g. ThermalCamera, etc)
I will track this bug myself. I'm opening the ticket so I don't forget.

@iche033 I tracked down the problem to the following in camera.cc:
Old code:
void CameraTest::ShaderSelection(const std::string &_renderEngine)
{
  auto thermalCamera = scene->CreateThermalCamera("ThermalCamera");
  //...
  engine->DestroyScene(scene);
  rendering::unloadEngine(engine->Name());
}

Fixed code:
void CameraTest::ShaderSelection(const std::string &_renderEngine)
{
  {
    auto thermalCamera = scene->CreateThermalCamera("ThermalCamera");
    //...
    root->RemoveChild(thermalCamera);
    engine->DestroyScene(scene);
  }
  rendering::unloadEngine(engine->Name());
}

The extra scope is something I was expecting (i.e. smart_ptr should be reset before rendering::unloadEngine). However I noticed the call root->RemoveChild(thermalCamera); is required.
This looks like a bug; as I think this is something that should be happening as part of DestroyScene() call (because the Root Visual afaik belongs to the Scene)
@iche033 : Is this behaviour a bug or intended?

Btw this can be easily debugged with:
fprintf( stderr, "%i\n", thermalCamera.use_count() );
After engine->DestroyScene(scene); the value of thermalCamera.use_count() should be 1.
But when root->RemoveChild(thermalCamera); isn't called, the count is 2 instead.

Nope that's not intended. Definitely an issue, which should hopefully be fixed by https://github.com/ignitionrobotics/ign-rendering/pull/617


should Migrate from legacy HTTP to HTTP v1

Apps using the FCM legacy HTTP API should consider migrating to the HTTP v1 API
For details:
https://firebase.google.com/docs/cloud-messaging/migrate-v1

Beware of the fact HTTP v1 does not support devices groups, nor multicast messages yet, wich would cause a big breaking change.
We can use both, legacy for those features, and V1 to the rest, but i don't think it a good idea.

Is there anything we could help to bring this over the finishing line?

Out of date. Please try with the new version. closing for now.

Hi there, thanks for providing this gem 👍 .
Just checking if this issue is on the road map at all? There are features in the HTTP v1 API that would be great to utilise (analytics_label, for example).
OR has it been integrated already?
Line 48 of the README implies yes, but checking the code it still looks like this gem is using the legacy api /fcm/send.  The readme#48 implies all of those options are available, but after some testing it would appear not to be the case.  Am I doing something wrong?  Here's my options block:
{
  'data': {
    'type': 'page',
    'path': @notification.path
  },
  'notification': {
    'click_action': 'FCM_PLUGIN_ACTIVITY',
    'title': @notification.title,
    'body': @notification.body,
    'image': @notification.image
  },
  'android': {
    'fcm_options': {
      'analytics_label': @notification.label
    }
  },
  'apns': {
    'fcm_options': {
      'analytics_label': @notification.label
    }
  }
}

@ostigley you are correct it seems we are still using the old API. I was under the impression @prcongithub 's PR addressed that but I think it didn't. I am reopening this. Will add it to the road map

Hello everyone, are there any updates on this issue?

Hi all I'll need some help with this. I am open to having a paired programming session with someone who is using this day to day to help me close this. If you know the relevant portion of the FCM docs we can have a paired programming session in the coming days. Just send me a meeting request at https://calendly.com/sabman

All I'd like to make some time to pair with someone interested in migrating from the legacy API. to this end I've created a doodle to try to find a time that people interested can remotely pair. If you are interested in helping please let me know your availability here: https://doodle.com/poll/7gmqmyk62qq6z7sm

So unfortunately after upgrading to Flutter 2.0 and the latest flutterfire messages being sent from the legacy HTTP API is no longer coming through unless there is a notification attached. That's a bummer as we aren't able to use this gem any more. Since we are just needing to push one very specific type of message to topics we just wrote a few lines code for that very specific use case as a work-around but I wanted to share it here in hopes someone has the interest / time to pick up the torch here or if someone else finds themselves needing a work-around to get them to HTTP v1 in a bind.
Here is the gist: https://gist.github.com/f3z0/a09ae96b55ba95e117a596fb7e67e279
Feel free to ping me with any questions. Thank you 🙏

@f3z0 thanks for this looks useful. I presume we will need to cut a new major version if do this since it will break existing functionality unless people have created their credentials correctly and made them available to the library.
I'll make a start on it next week if anyone has any further thoughts beside's the gist from @f3z0 please let me know below.

Here is the gist: https://gist.github.com/f3z0/a09ae96b55ba95e117a596fb7e67e279

Hi everyone, I've incorporated gist from above to FCM gem here: https://github.com/decision-labs/fcm/pull/91
Happy to discuss what else needs to be done if that approach will work
Vadim

Hello guys, when will this feature be available ?

@iamdeveloper-lopez I am going to release a new version which includes the send_v1 method for HTTP v1 shortly. However, we will need to refactor this in the future to use name spaces as this is a little untidy.

https://github.com/decision-labs/fcm/releases/tag/v1.0.5


Report correlationId to Ladybug when using Request Reply on EsbJmsSender

Related Problem or Business Case
See #7648, I'm investigating using Playwright for testing adapters across multiple applications using the FF / Ladybug API.
In order to successfully correlate a message published on a queue to a report on the receiving system, it's essential to use the correlationId however this value is not reported to Ladybug when using RR pattern.
When using an EsbJmsSender with a FF pattern, the correlationId is returned by the sender to the pipeline. When using RR pattern, the reply is returned by the sender however the correlationId cannot be found anywhere in the report.
Describe the Feature
Infopoint that contains the correlationId for the message published on the queue thus it can be correlated to another Ladybug report in another IBIS application
Collaboration with External Companies
No, community resolution only
Anything else?
7.9.X friendly would be very appreciated !

It's not the correlationId that's being returned so be careful, the messageId of the 'message to be sent' is returned, which in this case luckily matches your correlation id. See this article about the message id and correlation id patterns.
Since the #2746 we keep track for the message's metadata or context if you will. You can find the information your looking for in the logging, as we've not yet implemented a way to show this in the ladybug.


[ERROR] errors with "sh trans" command with cygwin

I got these errors with "sh trans" command with cygwin :
/usr/bin/trans: trans: line 5630: syntax error near unexpected token `('
/usr/bin/trans: trans: line 5630: `gawk -f <(echo -E "$TRANS_PROGRAM") - "$@"'

You need to run it through bash, not sh.


Fixed #24474 -- Make empty_value_diplay a property in admin

Add param to ModelAdmin to set the empty value display:
ModelAdmin.empty_value_display.
Add property on ModelAdmin custom fields to set
empty_value_display for individual fields.
Add method to set empty value display on admin site level:
AdminSite.set_empty_value_display().

Please start with the commit I reverted as I made some edits while doing a final review, thanks.

Closing in absence of follow up.


HDFWriterPart now checks XML layout file is valid during configure

Added a check in configure to assert :XMLValid_RBV or otherwise report the status message.
If this check is not in place then any invalid XML layout file (path or formatting) will stop the images from being written and the scan will still succeed even though there is no data.

Codecov Report

Merging #311 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #311   +/-   ##
=======================================
  Coverage   92.34%   92.34%
=======================================
  Files         212      212
  Lines       10890    10893    +3
=======================================
+ Hits        10056    10059    +3
  Misses        834      834

Impacted Files
Coverage Δ

malcolm/modules/ADCore/parts/hdfwriterpart.py
95.07% <100.00%> (+0.07%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9043739...a640632. Read the comment docs.


Add further detail about automatically downloaded artifacts.

https://docs.microsoft.com/en-us/azure/devops/pipelines/artifacts/pipeline-artifacts?view=azure-devops&tabs=yaml#artifacts-in-release-and-deployment-jobs

Add further detail about which lifecycle hooks of deployment jobs automatically download artifacts. If they all do, then state so, if some do then make it explicit.
Add further detail about whether agentless jobs within the deployment lifecycle hooks automatically download artifacts, or not (or link to documentation that already states the outcome).

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 861ce7a5-4fa4-55f1-2c97-46a9cc6a2f66
Version Independent ID: bdf954b9-9bbe-8b34-b674-71ad8153519d
Content: Publish and consume artifacts in pipelines - Azure Pipelines and TFS
Content Source: docs/pipelines/artifacts/pipeline-artifacts.md
Product: devops
Technology: devops-cicd-artifacts
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@adriantorrie Thank you for your feedback. The download artifact task will be auto injected only when using the deploy lifecycle hook. See lifecycle hooks to learn about the other types of hooks.
We only support the tasks below for agentless jobs but you can use extensions to add other agentless tasks:

Delay task
Invoke Azure Function task
Invoke REST API task
Publish To Azure Service Bus task
Query Azure Monitor Alerts task
Query Work Items task


Does not run under Wine

This is not an actual issue, just here for curiosity and book keeping purposes.
AutoDDU_CLI.zip
https://pastebin.com/VEhzaq3J

Works in Arch


Improved return-response tests

This PR adds more tests to validate if methods return Response instead of void when configured.
This is related to: https://github.com/quarkiverse/quarkus-openapi-generator/issues/424

Closing this as we're going to change to return Response by default for void methods in https://github.com/quarkiverse/quarkus-openapi-generator/pull/426.


troubles when running multiple springboot projects

I tried to run 2 springboot projects and got the next error:
Could not get unknown property 'classesDirs' for main classes of type org.gradle.api.internal.tasks.DefaultSourceSetOutput.

plugin is applied in parent and child projects and also there is such configuration in build.gradle:
Parent project:
gretty {
  springBoot = true
}

farms {
  farm 'TestFarm', {
    servletContainer = 'tomcat7'
    webapp ':webapps1'
    webapp ':webapps2'
  }
}

Child projects:
gretty {
    springBoot = true
}

Gradle version is 3.4

https://discuss.gradle.org/t/could-not-find-property-classesdir-on-source-set-main/1645
This can help you


Wiki Improvements - Includes External PR

I want to modify the wiki but the current state does not allow this. Instead I cloned the wiki locally and pushed to a fork.
Includes:

Additional metadata options
[Migrated] Document missing events (sort of a fix for IdentityModel/oidc-client-js#1139)
Organized UserManager methods
Added missing UserManager methods (Fix IdentityModel/oidc-client-js#1253, Fix )
Add link to samples
Updated logging to include DEBUG (Fix IdentityModel/oidc-client-js#356)
Add refresh_token to User properties
[Migrated] Added UserManagerSetting stateStore

External PR:
coolhome/oidc-client-js#1

If the github wiki system doesn't feel right, maybe it could just move to a regular md file in the git repo that could accept PRs?


[Event Request] codeunit 7312 "Create Pick" DINOnFindPickBinCheck

Hello,
Is it possible to add the following Event:
        TotalAvailQtyToPickBase :=
            CalcTotalAvailQtyToPick(
                LocationCode, ItemNo, VariantCode, TempWhseItemTrackingLine,
                SourceType, SourceSubType, SourceNo, SourceLineNo, SourceSubLineNo, TotalQtytoPickBase, false);
        if TotalAvailQtyToPickBase < 0 then
            TotalAvailQtyToPickBase := 0;

            // DINNL <<
            DINOnFindPickBinCheck(TempWhseItemTrackingLine, TotalAvailQtyToPickBase);
            // DINNL >>
        repeat
            BinIsForPick := UseForPick(FromBinContent) and (WhseSource <> WhseSource::"Movement Worksheet");
            BinIsForReplenishment := UseForReplenishment(FromBinContent) and (WhseSource = WhseSource::"Movement Worksheet");
            if FromBinContent."Bin Code" <> ToBinCode then

[IntegrationEvent(false, false)]
local procedure DINOnFindPickBinCheck(TempWhseItemTrackingLine: Record "Whse. Item Tracking Line" temporary; var TotalAvailQtyToPickBase: Decimal)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Remove duplicate target from ci/build.sh

I'm almost certain this is not intentional

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @gnzlbg (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+

:pushpin: Commit 55189f6aa60d9f08b4d9baa5b138d4049b406673 has been approved by gnzlbg

:hourglass: Testing commit 55189f6aa60d9f08b4d9baa5b138d4049b406673 with merge 0d0163380ad0b1a848ca17eddecb454648a52088...

:sunny: Test successful - checks-cirrus, checks-travis, status-appveyor
Approved by: gnzlbg
Pushing 0d0163380ad0b1a848ca17eddecb454648a52088 to master...


BP-1442 Bancontact mobile: only QR visible no button

@harli91 please have a look at the unit test that fails (as a separate task): https://github.com/buckaroo-it/Magento2/runs/5645170663?check_suite_focus=true


Cannot use turbofish generic syntax

Take the following code:
script;

pub enum Option<T> {
    None: (),
    Some: T,
}

impl<T> Option<T> {
    pub fn none(self) -> Option<T> {
        Option::<T>::None
    }
}

fn main() {
    Option::Some(5).none();
}

Works fine in Rust but gives the following error in Sway:
error
  --> /sway/test-case/src/main.sw:10:17
   |
 8 |
 9 |     pub fn none(self) -> Option<T> {
10 |         Option::<T>::None
   |                 ^^^ generics are not supported here
11 |     }
12 | }
   |
____

I think we support Option::None::<T>?

Hmm yeah but this works Option::None::<T>() and we don't want this to work as per https://github.com/FuelLabs/sway/issues/3375 so this is certainly an issue.

With the changes of #3743 we will support Option::None::<T> without the parenthesis.
Nontheless Rust still supports the code below:
let n = Option::<u64>::None;

We still doesn't support it. Do we want to support it?

The answer usually is 'if Rust does it, then we should do it too'. 🙂  I think it's the more idiomatic form too.


add table with models comparison to the docs

google if there is an opportunity to import csv to the documentation and add csv from 02_models_comparison.ipynb notebook if so

copy the csv text otherwise (

.


Add support for build secrets by catching up with the 18.09 Docker CE release

Catch up with the 18.09 Docker CE release to provide its build secrets feature:

The new --secret flag for docker build allows the user to pass secret information to be used in the Dockerfile for building docker images in a safe way that will not end up stored in the final image. [...] For example:

# syntax = docker/dockerfile:1.0-experimental
FROM alpine
RUN --mount=type=secret,id=mysecret cat /run/secrets/mysecret # shows secret from default secret location
RUN --mount=type=secret,id=mysecret,dst=/foobar cat /foobar # shows secret from custom secret location

Note for anyone browsing and finding this ticket: balena has also an alternative (beta) secrets implementation. It's not currently compatible with the Docker CE feature, but compatibility work is planned. See also issue: https://github.com/balena-io/balena-cli/issues/1063

Closing as we are based on docker 18.09 with #151 being merged


unique usernames

[x] issue #105
[x] follows contribution guide
[x] code change includes tests
[x] breaking change

@aweolumidedavid fo the build/ check to pass,
cd into onchain directory and
run scarb fmt, and push the changes,

Thank you for making those changes!


add deluge (re-upload of #380)

What this PR does / why we need it:
Adds Deluge bit torrent web client.
Which issue (if any) this PR fixes:
Fixes #380 issues I made by not creating a separate branch on my fork.
Any other useful info:
I set deluge's downloads volume to be mounted at /root/downloads instead of /downloads because the default config of the client have the downloads folder pointed to that directory.

Awesome! 👍  Thanks for the contribution.

Awesome! 👍  Thanks for the contribution.


⚠ Bofrost.de has degraded performance

In 0cd06d5, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1737 ms

Resolved: Bofrost.de performance has improved in 23b82af.


Datamodel change

Removed getFragmentScanNumbers() from Scan
Precursor information for ion mobility stored in ImsMsMsInfo for Frames with mslevel = 2
IMSRawDataFile.getScan returns the frames (Bit of a dirty hack atm, can be fixed in the future)

old modules should be able to work with these average spectra
getFrame[..] methods have not been removed!

MobilityMassSpectrum added

StorableMobilityMassSpectrum only stores its frame (ptr), highest data point (ptr + 2 doubles), number (int), storageId(int) in ram and caches a Range if it was accessed once.

the numDp are accessed via this hack: return ((IMSRawDataFileImpl)getFrame().getDataFile()).getDataPointsLengths().get(storageId);
RT accessed via frame
MobilitySpectrum number <-> mobility is stored once in the IMSRawDataFile per segment => millions of doubles less

Just a cosmetic detail: I think MobilityMassSpectrum should be called MobilityScan, because it represents the result of instrument scanning the mass range.

I didn't want to use the word 'scan' because it could imply it implemented 'Scan'. So should i call it MobilityScan?
I want to add one more method before merging, though.

￼I think MobilityScan implements MassSpectrum is fine.

Just a cosmetic detail: I think MobilityMassSpectrum should be called MobilityScan, because it represents the result of instrument scanning the mass range.

I am very much in favor of MobilityScan!

@robinschmid what do you think about the MobilityScan <-> MobilityMassSpectrum situation?

Note: this breaks project loading for ims raw data files since mobility scans are not stored. However, this does not limit us from proceeding

I will have to add masslist + detection support. Might take another hour or so

About the masslist support etc
Maybe the best option to add broad support to access the scans is to change the ScanSelectionParameter
We typically access the scans from there with getMatchingScans or something. You can then automatically check if we are looking at Frames and select all sub scans

I will add getMatchingScan methods for mobility spectra (have that in some branch already), but returning all subscans when a frame is inserted will lead to non-ims modules to retrieve the mobility sub-scans, which is not a good idea in my opinion, because it will break almost everything.

seems to work.

Ansgar and i would need a review + merge so we can continue properly, please.

It’s fine for now!
Overall the PR looks good to me.
On Sun, Dec 20, 2020 at 11:40, Steffen Heuckeroth<EMAIL_ADDRESS>wrote:

@SteffenHeu commented on this pull request.
In
src/main/java/io/github/mzmine/modules/io/projectload/version_3_0/RawDataFileOpenHandler_3_0.java
https://github.com/mzmine/mzmine3/pull/118#discussion_r546357937:

@@ -327,9 +327,9 @@ public void endElement(String namespaceURI, String sName, String qName) throws S

 if (qName.equals(RawDataElementName_3_0.FRAME.getElementName())) {

 final StorableFrame storableScan = new StorableFrame(newRawDataFile, currentStorageID,

 /*final StorableFrame storableScan = new StorableFrame(newRawDataFile, currentStorageID,

Due to the change in data model, the project import for ims is broken, and
will need to be fixed in the future. Currently constructing modules for
processing is more important to continue than having a project import
(imo). But i will fix the project load later on.
Or maybe we should switch to a new one, directly? What do you think?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mzmine/mzmine3/pull/118#discussion_r546357937, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AANWUVABEMXBXUNRXHWHTHTSVXIBZANCNFSM4VBZ7URQ
.


[Bug]: Report 1; All Counties in the World missing columns

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Name
Bobby Walters
Description of the bug?
A country report requires the following columns:
Code.
Name.
Continent.
Region.
Population.
Capital.
Only Name, region and population listed. Columns headings need to be formatted correctly. Also should headline all counties in the world.
Version
Develop Branch
Steps to Reproduce the Error
Select your report number or q to quit!
1
Countries In A Region By Population Size:
"name                                Region                                   population
China                               Eastern Asia<PHONE_NUMBER>
India                               Southern and Central Asia<PHONE_NUMBER>

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Column Headers should be in CAPS.


Use JavaScript Array.slice in 'getslice' when applicable

For step=1, the common case, performing a native array slice should be faster than copying the elements ourselves.
I'm not sure how to profile to tell whether it actually is, but it seems like it could be a good idea.
Let me know what you think, and whether there's anything else I can do in this PR. I'm not 100% familiar with the structure nowadays and I'm not sure whether anything needs to be done for pre-ES6 compatibility here?

Hi
ES5 compatibility was dropped, somewhat anticipating on a bright future..., ES6 modules are needed anyhow for 3.7.
So I think this PR is OK.
Maybe a testcase can be added to the autotest (if it isn't there yet, I'm not sure).
I'll pull this one now already, since it matches the current head.
Glad to hear from you again!
Jacques

Glad to contribute! I have a few patches that I realized I'd not actually made PRs for to check whether they'd be useful.
I can make tests if that would be helpful, I'm just not entirely sure how the testing framework works and haven't gotten the tests running on my computer yet.


TaggedItemBase related_name not filling in %(app_label)s_%(class)s_items since Django 4.2 LTS?

I upgraded a Wagtail site recently and because of that ended up on Django 4.2 LTS.
I noticed it detected changes in my models when it shouldn't have. Upon closer inspection the issue was caused by the tag field in TaggedItemBase in models.py. The tag field has related_name="%(app_label)s_%(class)s_items" and in my initial migration of my app this gets resolved to related_name='blog_blogpostpagetag_items'.
However in the new version of Django 4.2 LTS, it seems to no longer be filling in %(app_label)s and %(class)s and it wants to change related_name='blog_blogpostpagetag_items' to related_name="%(app_label)s_%(class)s_items" which doesn't seem right to me.
What I'm going to do in the mean time at least is hardcode the related_name until this is resolved.
I have not yet reported this with Wagtail as it is in this project really, not Wagtail.

I'm just wanting to know if this is actually a problem.  I have a feeling it isn't actually a problem but the migrations just need updating.
I guess one test would be to check if that backref is still working, and if it is then it's probably just a matter of updating the migrations.

I tested if the backref is still working and it is still working just fine using the old name.
I tried some_tag.blog_blogpostpagetag_items.all() and it worked the same, so this is a non-issue.
So perhaps this is just how Django now represents this in migrations and I will update my migrations, considering this is a non-issue.
I'm happy to close this issue, or perhaps someone else will run into this.

Thanks for the report! This indeed sounds like an issue with django-taggit. Going to attempt a reproduction of this issue

I don't think there is anything you can do about it other than to update the migration.
It's a non-issue as per my last comment and it isn't Taggit, it's Django 4.2+ that does this

I'm happy to close this issue as I long moved on by just updating my initial migration.

At the very least this is something where I would have wanted to put in documentation to confirm that this was indeed a thing that happens. Having to manage migrations on library upgrades, I'm often having to go in and figure out why something has changed or not.
It seems like this change happened in the 3.2 -> 4.0 move (how I figured this out: installing various Djangos, then running PYTHON_PATH=$(pwd) DJANGO_SETTINGS='tests' django-admin makemigrations --dry-run --check to find where we started to add a bunch of stuff. You can also see the difference by checking Through2.tag.field.deconstruct() and seeing that related_name is filled in in 3.2, and is an untouched template in 4.0).
The related Django change is in the 4.0 release notes:

As a side-effect, running makemigrations might generate no-op AlterField operations for ManyToManyField and ForeignKey fields in some cases.

Reading through this though, in practice it sounds like something where we would notice the issue while upgrading to Django 4 on the "real" project.
I'm going to set up a tests that tests for changes to the test file migrations, so we can at least signal this sort of stuff in the release notes (even if it's redundant)


How to calculate floating-point operations(FLOPS) in mxnet?

Are there any tools to calculate floating-point operations(FLOPS) in mxnet? If not, then how to do this manually? Any suggestions would be appreciated.

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!

@szha can we reopen this issue?
In one of my big data assignments, I was tasked to find "number of floating point computations taking place when one single data is passed through the network"
Is there a way to find this directly in mxnet?

@ChaiBapchya there's no existing feature in mxnet for calculating flops and people usually estimate this based on manual calculation on different (mostly gemm type) operators.

@mxnet-label-bot add [Feature request]

I was able to count total number of parameters for a model in Pytorch using this command
pytorch_total_params = sum(p.numel() for p in model.parameters() if p.requires_grad)

Source - https://discuss.pytorch.org/t/how-do-i-check-the-number-of-parameters-of-a-model/4325/9?u=chaibapchya

@ChaiBapchya I wrote a tool to calculate flops and model size of a given mxnet symbol, you can have a try.
https://github.com/Ldpe2G/DeepLearningForFun/tree/master/MXNet-Python/CalculateFlopsTool


Release 0.2.0

Changing the file to include in the github release

@yatharthranjan The travis.yml was still pointing to old distribution files with tomcat.


feat: add a basic example of text detection

A script to extract text from files.

Hi @ianardee 👋,
thanks for the PR :)
Some minor things:

could we add also the xml output ?
missing (minor/lightweight) CI test

@charlesmindee I think there are some missing things (mentioned above) wdyt ? 🤔

@felixdittrich92 I'll make another PR to answer some of your concerns.

@ianardee top 👍

Hi @ianardee :wave: , any updates about the refactor PR ? :)


SoundofRaindrops not a bot

Hello. I came from the ** raid. I have been banned from random streams since I have been put on a bot list. I am not a bot. I have had this account since 2015 and is not a new one. Unfortunately for me even after I explained that to a few mods the appeal has been rejected but I guess I can't do much about that.

might have been your use of the channel points miner
https://github.com/rdavydov/Twitch-Channel-Points-Miner-v2
however i dont see your name on the twitchinsights
https://twitchinsights.net/bots
so...ill remove you from the list.
good luck


[Bug] app crashs when requesting for optimized settings if no game is selected

Describe the bug
choosing the option to "look up for the optimized settings..." if no game is selected crashes the app
To Reproduce
Steps to reproduce the behavior:
open the app for the first time
go to settings
press the cloud icon for downloading the optimized settings
Expected behavior
i feel a small pop up sating choose a game first would be a good way
Environment Information

OS: windows 11
App Version: 2.0.0 experimental

Log File
Log-20241011.txt
Additional context
while in a normal situation the user will have a game in their library before attempting to download a game prefix its a bug nevertheless, i could not see a way to change global settings in the new app but if it will be added in the future it will be a problem if the user accidentally press the cloud icon when in the global setting.

Please for the future issues for v2.0.0 use the dedicated discussions.
I have disabled the global settings changing because of the way Xenia Manager v2.0.0 loads the configuration files (Symbolic Links instead of "--config" argument). Good reminder that I need to disable the access to settings if there are no installed games.

thanks for the fast reply :)
and sorry didn't know there was a separate discussion for it and thanks for the great work


👔 Add identification logic of parameters in Spring

Hi @ksg97031 👋🏼
I've added some code to the SpringAnalyzer for #232. This code identifies well-known parameter processing logic and adds it as a parameter to the last added Endpoint during file content analysis. I've written it lightly for now, but please review and let me know if this method seems good!

If this direction is correct, I think it would be good to continue expanding the pattern together in the future. However, I'm slightly worried because developers have such a wide variety of ways to handle parameters.

@hahwul
Hi~ I've implemented a simple lexer and parser for the Java language.
Now we can gather parameters for Spring MVC annotations and endpoints more accurately.
Since it took longer than expected, I decided to implement it first for Java then Kotlin later.
Let me know if you encounter any suspicious issues while using it!

@ksg97031
Cool! Thank you for the great work!
I'll make sure to try to include the changes you've applied in the next release (maybe 0.13.0).
Have a wonderful refresh :D


Diagnostic log in Key Vault does not support categoryGroup

Description of the issue
Azure.KeyVault.Logs rule fails while using categoryGroup to configure diagnostic log.
To Reproduce

Create bicep file keyvault.test.bicep with below code.

@description('location')
param location string = resourceGroup().location

@description('Analytics workspace name')
param logWorkspaceName string = 'log-iac-test-dev'

resource log 'Microsoft.OperationalInsights/workspaces@2022-10-01' existing = {
  name: logWorkspaceName
}

resource keyVault 'Microsoft.KeyVault/vaults@2022-07-01' = {
    name: 'kv-iac-test-1-dev'
    location: location
    tags: {
      Environment: 'Test'
    }
    properties: {
      sku: {
        family: 'A'
        name: 'standard'
      }
      tenantId: subscription().tenantId
      accessPolicies: []
      enabledForTemplateDeployment: true
      enableSoftDelete: true
      enablePurgeProtection: true
    }
  }

  resource logService 'Microsoft.Insights/diagnosticSettings@2021-05-01-preview' = {
    scope: keyVault
    name: 'key-vault-diagnostic'
    properties: {
      workspaceId: log.id
      logs: [
        {
          categoryGroup: 'audit'
          enabled: true
        }
        {
          categoryGroup: 'allLogs'
          enabled: true
        }
      ]
    }
  }

Run below comlet:

invoke-PSRule -InputPath '.' -Module 'PSRule.Rules.Azure' -Outcome Fail

Expected behaviour
There should be no Fail outcome.
Error output

RuleName                            Outcome    Recommendation
--------                            -------    --------------
Azure.KeyVault.Logs                 Fail       Consider configuring diagnostic settings to log access for Key Vaul…

Module in use and version:

Module: PSRule.Rules.Azure
Version: 1.21.2

Captured output from $PSVersionTable:
Name                           Value
----                           -----
PSVersion                      7.3.0
PSEdition                      Core
GitCommitId                    7.3.0
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Additional context

@yzhu228
Looks like the rule is missing support for category groups (audit, allLogs) and only supports categories property at the moment.
Lets add support for category groups that should solve this.
@BernieWhite Would you mind if I created a PR for this if my theory is correct?

@yzhu228 Thanks for reporting the issue.
@bengeset96 I think you are correct. Yes if you want to open a PR that would be great.


Step missing to add the FHIR resource application onto second tenant

An extra step needs to be added to Part 2 of the tutorial for Register Public Client Application to add the Azure API for FHIR onto the user's second tenant. A second tenant is required when using FHIR to allow data and resource control plane authorization.
Problem:
After I create an app registration on my second tenant, I am unable to add the Azure Healthcare APIs under API permissions. After I select the button 'Add a Permission' under API permissions, and search 'APIs my organizations uses' I am unable to find the Azure Healthcare APIs. This is because the resource application is not included in my tenant on default.
What I did to fix this:

I logged into my second tenant on the Azure CLI az login -t $tenantID --allow-no-subscriptions
I manually added the  Azure API for FHIR resource app on the Azure CLI: az ad sp create --id 4f6778d8-5aef-43dc-a1ff-b073724b9495
Note: For Azure Powershell, to add the resource app: New-AzADServicePrincipal -ApplicationId 4f6778d8-5aef-43dc-a1ff-b073724b9495

Then I was able to successfully add permissions for the Azure API for FHIR.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 74d0dae2-ed92-bad6-1a97-4fdcda016c07
Version Independent ID: ec5ca2c6-319a-1af5-eda3-95e84c7fac8d
Content: Web App Tutorial - Test connecting to Azure API for FHIR
Content Source: articles/healthcare-apis/tutorial-web-app-test-postman.md
Service: healthcare-apis
Sub-service: fhir
GitHub Login: @CaitlinV39
Microsoft Alias: cavoeg

This tutorial assumes that the FHIR Service, the Azure AD application, and Azure AD users are all in the same Azure AD tenant. You don't have to have them in different tenants necessarily. We mention this as a note on the first step of the tutorial that you will need to take extra steps if you don't have them in the same tenant. We don't plan to add additional steps to this tutorial to keep it a simple straightforward path. If there are other details that we could add to one of the how-to guides please let us know!

Thanks for the quick reply Caitlin! The reason I ran into this was because I needed a second tenant on Step 3 to get the OAuth token from Postman. When I created everything in one tenant, I didn't have the proper admin access needed in order to successfully get the OAuth token to perform a GET on the Patients endpoint!

@melrush13 are you using your Microsoft account? That will cause the limitation. If you use a personal Azure account it should work.

@CaitlinV39 Yes, that is correct I was using my Microsoft account! That explains the limitation. Feel free to close the issue if you'd like!

@CaitlinV39 Thanks for pitching in and providing the necessary information.
@melonrush13 We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


How to resolve the error - sh: 1: craco: Permission denied

I set up a react app on Windows platform ((react app + typescript) + craco + tailwindcss). Then I copied the app folder on Ubuntu platform (which contains vscode, node, npm). When I run "npm start" command on vscode terminal, the error sh: 1: craco: Permission denied shows up.
I want to know what the problem is, and how to resolve it?
Please help.

Same issue. But this works fine on my SSD, which runs the OS. But I have a secondary hard drive in the system. I get this issue when I try to run a project in the secondary hard drive.

I set up a react app on Windows platform ((react app + typescript) + craco + tailwindcss). Then I copied the app folder on Ubuntu platform (which contains vscode, node, npm). When I run "npm start" command on vscode terminal, the error sh: 1: craco: Permission denied shows up.
I want to know what the problem is, and how to resolve it?
Please help.

Same is the case when a raect-app set on Ubuntu is copied to Windows platform. Does anyone have some explanation?

have you tried completely removing the node_modules folder and reinstalling?

Yes. I tried it just now, deleted node_modules folder, and ran command npm start. Then everything works fine.

Error: Command "npm run build" exited with 126

Did your issue got solved , I am also getting same error while vercel deployment.


Add CLI args for debug, nodeVersion, and npmVersion

Support npmVersion, nodeVersion, and debug flags being sent on deploy
Closes #137
Closes #78
References #106

👍

👍

Should we update the README for these changes in this PR?

Yea. Making those changes now

@tzmanics changes made

👍


KFserving Porting Tensorflow Object Detection models

Hi  Team,
Do we have any recommendations for porting Tensorflow Object Detection models in KFServing
https://github.com/kubeflow/examples/tree/master/object_detection
There are custom libraries used.
https://github.com/kubeflow/examples/tree/master/object_detection/serving_script
Has anyone tried porting it, Any pointers appreciated

@siva2k16 would you be interested in contributing porting the example using KFServing? Looks like we follow the tensorflow example.

I am getting started on this. I would need sometime to figure out details.

I am getting started on this. I would need sometime to figure out details.

Great! Feel free to reach out on kfserving slack or here if you need any help.


draw not compatible with latest blocks

The current draw code is again incompatible with the latest version of blocks (currently 49a12a3). This seems to mainly be due to the API change in mila-udem/blocks#725 and I have a fix for this in my blocks_fix branch. Unfortunately, this change also invalidates all previous pickled models, so I'm going to hold off on a merge request until I can train a new models for the README update.

@dribnet thanks so much. it works now.

#28 (@dibnets fix; thanks!) and #29 should fix it for now.


Configurable behavior on critical error: message on Error stream or pop-up dialog

Summary of the new feature / enhancement
When critical errors such as "cannot start EXE due to missing DLL" occur in File Explorer or CMD, an explanatory error dialog is displayed:

When the same kind of error occurs in PowerShell, no error message is displayed, only $LASTEXITCODE is set:

It would be desirable to make this behavior configurable, so that especially in interactive scenarios the user is supplied with information about what's happening.
This inspired by MKS Korn Shell's set -X option (@BrucePay will remember...):
From set KornShell flags and positional parameters:

Of course, that exception handler is not desirable for scripted / unattended execution, so it should be off by default.
It doesn't have to be a GUI-based one, outputting the explanatory text to the Error stream would also be fine.
@corbob has collected repro steps on various shells.
Proposed technical implementation details (optional)
No response

Thanks for opening the issue @sba923. I've created a simple repro and have verified behaviour in Windows PowerShell 2 on Windows 7, Windows PowerShell 5.1, and PowerShell 7.2. I have also checked the behaviour of cmd, the run dialog box, and git-bash.
To expand on what is listed in above: PowerShell behaves the same as cmd and the run dialog when you use Start-Process. In all other cases I tested (running adb, . adb, & adb) the error is not emitted outside of $LASTEXITCODE. In git-bash an error is emitted, but it is not extremely helpful as it just indicates it couldn't load a file and gives a different exit code:

The repro steps I've used (from an administrative PowerShell window):
choco install adb
Remove-Item "$env:ChocolateyInstall\lib\adb\tools\platform-tools\AdbWinApi.dll"
adb

Note Chocolatey is not  the only way to get adb installed, but it is a pretty quick and painless way to do it.

Is there any solution for the required "pop up window" when running .exe files missing .dll files?

This should be reopened if you ask me 😜@StevenBucher98 @SteveL-MSFT


[New Feature]: Add DSWx-S1 test cases into integration test

Checked for duplicates
Yes - I've already checked
Alternatives considered
No - I haven't considered
Related problems
No response
Describe the feature request
An unit test of RTC data retrieval and DSWx-S1 PGE execution needs to be added into the integration test.

Hi @hhlee445, as discussed I'd like to propose (based on conversations with @chrisjrd and @collinss-jpl):

Edge case tests for DSWx-S1 defined here need to be run nightly / automatically either:

Within the test_integration.py script (which runs every day)
As part of @collinss-jpl 's nightly tests of the PGE

Resolve https://github.com/nasa/opera-sds-pcm/issues/725 so that we’re also testing our assumptions for how the MGRS DB is structured
Add unit tests for @chrisjrd's DSWx-S1 evaluator script to account for a given set of bursts matching multiple mgrs_set_ids (as well as also checking existing 100%, 0% and partial coverage constraints)

See also this more general ticket: https://github.com/nasa/opera-sds-pcm/issues/726

https://github.com/nasa/opera-sds/blob/riverma-edge-case-table-fix/processing_request_datasets/edge_cases/RTC-S1_granules_for_edge_cases.md

Hey @hhlee445 - just wanted to bring in @philipjyoon and his DISP-S1 testing framework which he mentioned could be repurposed for DSWx-S1. This framework has a really easy to modify json testing scenario file approach that separates the test details from the underlying execution of the test. Down-the-line it might be a good idea converge on a single integration / system testing framework. Let's chat more.

PCM team discussed about DISP-S1 testing framework and @chrisjrd is looking at it.

Hi @hhlee445, as discussed I'd like to propose (based on conversations with @chrisjrd and @collinss-jpl):

Implement these tests into the existing test_integration.py script. We should be checking for metadata of the finished jobs to ensure they match the expected mgrs_tile_set ids listed (to ensure the right jobs were kicked off).

Edge case tests for DSWx-S1 defined here need to be run nightly / automatically either:

Within the test_integration.py script (which runs every day)
As part of @collinss-jpl 's nightly tests of the PGE

Resolve [New Feature]: MGRS Tile Set DB Integration Tests #725 so that we’re also testing our assumptions for how the MGRS DB is structured

Add unit tests for @chrisjrd's DSWx-S1 evaluator script to account for a given set of bursts matching multiple mgrs_set_ids (as well as also checking existing 100%, 0% and partial coverage constraints)

Hey @hhlee445 - since you closed this ticket, could you clarify which of the above were resolved? I see the (3) bullet point ticket is still open.


Nodes created manually never get to Ready Mode

This is a...

[ ] Feature Request
[ x] Bug Report

Problem:
Following the steps https://kubernetes.io/docs/getting-started-guides/fedora/fedora_manual_config/
Node stayed in the NotReady status.
Proposed Solution:
Remove the manual node creation using the JSON file.  Nodes will self register and come up properly.
Page to Update:
http://kubernetes.io/docs/getting-started-guides/fedora/fedora_manual_config/

/remove-lifecycle stale

Hello guys,
unfortunately, this guide [1] still does NOT work. The node never gets ready. I double checked several times. Still the same result. Any suggestions?
Cheers.
[1] https://kubernetes.io/docs/getting-started-guides/fedora/fedora_manual_config/