[SR-14229] [AutoDiff] conformance of synthesized TangentVector doesn't work

Previous ID
SR-14229

Radar
rdar://problem/81170168

Original Reporter
@marcrasi

Type
Bug

Additional Detail from JIRA

Votes
0

Component/s
Compiler

Labels
Bug, AutoDiff

Assignee
None

Priority
Medium

md5: 02544c29a1fa7eb644c287b7535e0342

is duplicated by:

SR-14959 [AutoDiff] Declaring protocol conformances for synthesized TangentVectors sometimes doesn't do anything

Issue Description:
Code:
public protocol P {}

public struct S: Differentiable {
  var x: Float = 0
  public init() {}

  //public struct TangentVector: AdditiveArithmetic, Differentiable {
  //  var x: Float = 0
  //}
  //public func move(along direction: TangentVector) { fatalError() }
}

extension S.TangentVector: P {}

func f<T: Differentiable>(_ t: T) where T.TangentVector: P {}

f(S())

Error:
module.swift:17:1: error: global function 'f' requires that 'S.TangentVector' conform to 'P'
f(S())
^
module.swift:15:6: note: where 'T.TangentVector' = 'S.TangentVector'
func f<T: Differentiable>(_ t: T) where T.TangentVector: P {}
     ^

This seems related to the synthesis because if you uncomment the custom TangentVector it compiles successfully.

Issue still exists on 05/24 toolchain.


Exact source of duplicate annotations

https://curation.pombase.org/dumps/latest_build/logs/log.2021-12-12-21-29-32.chado_checks.duplicate_go_annotation
I'm not sure where  these are coming from.
Midir mentioned UniProt, but I filtered some external and they are still there.
Not urgent.

I think this is OK. Most appear to duplicates in legacy and Canto. Whee the legacy annotation was not deleted when the new annotation was made in Canto.


boards: shields: Adding support for the adafruit can picowbell shield

Adding support for the adafruit can picowbell shield designed for the raspberry pi pico

Is there anything else I need to do for this merge request to be approved? :)

@erwango can you please review this one?


TM - fix xcode 11 build issue

This reverts commit ad0a303336f67770127f55aabba81a051c2b6ac4.

@laurenwilliams Without this change iOS is not building on XCode 11 but builds on Xcode 12. But Xcode 12 is still having issues with images. So we're doing this revert for now


Mising description in "Write Your First Flutter App"

In part "4. Use an external package", the const identifiers in Title and Body magically vanish with no explanations. One cannot add "child: Text(wordPair.asPascalCase)" into the body without removing the const before Body, as this throws errors. There should be a comment in the code to tell the user to remove the const before Body.

Hi @tintelle
If I understood correctly, you mentioned this paragraph.

It is clear from code which lines should be added, which should be removed.
I close the issue
if you disagree please write in the comments
and I will reopen it.
Thank you

Sorry, I should have been clearer: I'm not talking about the code in the repo, but the Codelab description. Here, the app is created using const identifiers:
https://codelabs.developers.google.com/codelabs/first-flutter-app-pt1/#2
One step later, the const's vanished without a comment, causing problems when not removing them:
https://codelabs.developers.google.com/codelabs/first-flutter-app-pt1/#3

Ah, ok, my screenshot was from this documentation.
I was looking there (and there examples were without const in mentioned places).
But now I see what do you mean
Thank you


maps: DeleteFunc fails in presence of NaN keys

What version of Go are you using (go version)?
Playground 1.21: https://go.dev/play/p/-NzVmrPm-ln
Does this issue reproduce with the latest release?
Yes: https://go.dev/play/p/-NzVmrPm-ln?v=gotip
What operating system and processor architecture are you using (go env)?
N/A
What did you do?
package main

import (
	"fmt"
	"maps"
	"math"
)

func build(n int) map[float64]int {
	m := map[float64]int{}
	for i := 0; i < n; i++ {
		m[math.NaN()] = i
		//m[float64(i)] = i
	}
	return m
}

func main() {
	m := build(10)
	for k, v := range m {
		fmt.Printf("%f %d\n", k, v)
	}
	fmt.Printf("-----\n")
	maps.DeleteFunc(m, func(k float64, v int) bool {
		return v == 7
	})
	for k, v := range m {
		fmt.Printf("%f %d\n", k, v)
	}
}

(Code based on this from @rsc)
What did you expect to see?
One element deleted from the map: the one where DeleteFunc's predicate returns true (v == 7).
What did you see instead?
No elements are deleted from the map.
However, if I replace m[math.NaN()] = i with m[float64(i)] = i, so that the map has all distinct non-NaN keys, it works.

I believe this is unfortunate but working as intended.
clear is the only way NaN keys can be removed.

clear is the only way NaN keys can be removed.

I didn't see this documented either with maps.DeleteFunc, or with map types or clear. Maybe it should be?
Aside: there's some possibly unfortunate language relating to min/max here: "assuming all NaNs are equal". NaNs are not equal - not even to themselves. But I believe that all NaNs are equivalent or fungible.

The inequality of NaNs come from:

Two floating-point values are compared as defined by the IEEE-754 standard.

Yes I understand that. I was querying the language of the golang spec which says "assuming all NaNs are equal"

We should document this behavior, if nothing else. I'll reopen for that.
It is possible we could let DeleteFunc to dive into the runtime to delete entries which have NaN keys. Since DeleteFunc  is already released with the current behavior, changing its behavior is a breaking change. Although I think it's pretty easy to justify this as a bug fix, at the same time I'm not seeing any strong reason to fix it, and compatibility argues for keeping the behavior as is.

(Compare #20660.)


GO: Version 0.5.1 and<EMAIL_ADDRESS>

Dear all,
Working with GO, my problem is that the generated Validate(all bool)  functions are not called by the<EMAIL_ADDRESS>As I understand it, that validator.go defines the validate interface with Validate() error functions, which signature differs from Validate(all bool) error.
I saw that the https://github.com/envoyproxy/protoc-gen-validate says:
“All messages generated include the new Validate(all bool) error method”
So I guess that something has changed and I might have a version mismatch.
But it seems that I do have the last version of “go-grpc-middleware" (version 1.2.2).
Thank you for any pointer regarding this problem or my misunderstanding

The change is from #455. You'll want to either use a version of PGV from before then or use a version of go-grpc-middleware that is newer (dunno if that exists yet). If you need a very rough temporary solution, you could write Validate() error functions that call Validate(true).

Thank you for your  answer.
go-grpc-middleware 1.2.2 seems currently the latest version.

I have a fix lined up here: https://github.com/grpc-ecosystem/go-grpc-middleware/pull/417

Thanks. v0.6.1 reverts the change to the Go validation API after the discussion on https://github.com/envoyproxy/protoc-gen-validate/pull/455#issuecomment-825840942. We can continue the discussion in #47 about how to implement this in Go without silently breaking consumers.


Signoff missing on old commit

Who can fix this?

Commit sha: 29e6acb, Author: Tomasz Kuzemko, Committer: GitHub; The sign-off is missing.

It creates DCO errors for PRs that include this commit.

The person pushing the commit would need to push it with -s

@tomasz-kuzemko Can you sign the above commit?

Commit has been squashed and  no longer exists.


Add GHR protocol to plug manager

After writing the discussion on https://github.com/silverbulletmd/silverbullet/discussions/47
I figured at least the protocol should be easy to do and here it is. I modified the backlinks release to conform to that one so once this is merged you can install silverbullet-backlinks by adding:
- ghr:willyfrog/silverbullet-backlinks

or specify a version
- ghr:willyfrog/silverbullet-backlinks/v0.2

I haven't properly managed what happens on unresolved plugs, as I think it is out of scope. Right now it is throwing an error the same way an unresolved https plug does
This is it in action

.

Very nice! Took me a while to figure out what ghr stood for though :D

Feel free to change the protocol to something else. I wasn’t sure what to use


Export to pdf

Add the /pdf url to export all the slides to a big pdf file.

Fixes #33

This is exactly what I wanted for KubeCon. Thank you!


xen-4.8: update changed patch hash

Motivation for this change
The patch for a Xen 4.8 seems to have changed. The new patch is signed, and looks good as far as I can tell. This fixes the build for me locally.
Things done
Please check what applies. Note that these are not hard requirements but merely serve as information for reviewers.

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] 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 nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

cc @michalpalka

The new patch says:
v2: Add QEMU_PACKED to fix handling 32-bit guests by 64-bit qemu.

and changes the following line
+                  struct blkif_response QEMU_PACKED)

LGTM.

@roberth merged, thank you.
@oxij thanks for taking a closer look


bors compatible CI config

I think the status has to match the "name" line.

bors r+
merge thyself

bors retry
Change the status to match the "job" name, not the "workflow" name.

@urschrei
we have a winner!

Amazing.


How to use ThinkingSphinx::Distributed::Index

Hi Pat,
Is there documentation on creating distributed indexes? We are moving to a distributed setup and from what I've researched it appears in addition to updating the main config, each index file will also need to specify its type (distributed) and agent addresses etc.
Something like the following:
index master
{
  type = distributed
  local = items
  agent = <IP_ADDRESS>:9312:items-2
}

I found this https://www.rubydoc.info/github/pat/thinking-sphinx/ThinkingSphinx/Distributed/Index but I'm not clear on when and where to use that class.
Thanks

Hi Chris,
At this point in time, the distributed index support in TS is only there to collect both core and delta indices. It doesn't handle Sphinx instances in other locations.
I'm certainly not against adding something more useful for you. Can you talk through how you're thinking of structuring servers and indices, so I can get a better idea of what's needed?

Thanks for the reply.  The app uses real time indexing and employs TS for filtering and search from the app's index pages. Ideally the configuration would follow the distributed model provided by Sphinx. http://sphinxsearch.com/docs/current/distributed.html
What other info can I provide that you would find helpful? Thanks!

I'm presuming in this setup you're going to have Sphinx on each of the Rails servers? Have you considered just pointing them to a single Sphinx server instead?

I haven't found a way to set them up to use a single server. My understanding is the indices are created locally regardless of where the mysql database is. Do you have any insight or docs on that?

mmm, this is something that could definitely be documented in more detail.
You'll want to have a copy of your Rails app deployed to your Sphinx server, and have address in the appropriate environment within config/thinking_sphinx.yml set to its public IP (public within the context of your Rails servers, at least). From there, make sure you invoke all TS-related rake tasks on that Sphinx server, and the address setting will be picked up on the Rails machines and they'll send all Sphinx-related requests to that.

Thanks for your help Pat, I'll continue to investigate this.


RichTextLabel's set_bbcode and get_bbcode methods are missing.

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Windows 10
Godot master
Issue description:
The recent versions of RichTextLabel don't even HAVE the get_bbcode function exposed to GDScript for some reason, even though I can see the methods being exposed in the source code. Not sure why that is happening...? We're supposed to have get_text and get_bbcode available to retrieve raw and rendered versions of bbcode_text, respectively.

Nevermind, apparently I just have a slightly older version that wasn't generating the API docs properly or something? Ignore this particular Issue.


 Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.

(DiscordCanary:398207): Gtk-WARNING **: 11:23:48.926: Failed to parse /home/mika/.config/gtk-3.0/settings.ini: Key file does not have group “Settings”
Starting app.
Starting updater.
[Modules] Modules initializing
[Modules] Distribution: remote
[Modules] Host updates: enabled
[Modules] Module updates: enabled
[Modules] Module install path: /home/mika/.config/discordcanary/0.0.119/modules
[Modules] Module installed file path: /home/mika/.config/discordcanary/0.0.119/modules/installed.json
[Modules] Module download path: /home/mika/.config/discordcanary/0.0.119/modules/pending
[Modules] No updates to install
[Modules] Checking for host updates.
[Modules] Host is up to date.
[Modules] Checking for module updates at https://discord.com/api/modules/canary/versions.json
[Modules] No module updates available.

If there's other information I can provide that would help I can provide that, not sure what do to from here.

Works after uninstalling canary and installing stable. Would be willing to troubleshoot why canary won't work.

BBD doesn't work on Canary anymore. See https://github.com/rauenzi/BetterDiscordApp/issues/442#issuecomment-768131571.

https://github.com/rauenzi/BetterDiscordApp/issues/598#issuecomment-806070645 seemed to fix it for me.

rauenzi/BetterDiscordApp#598 (comment) seemed to fix it for me.

That looks like it adds a lot of unnecessary code and has base64 encoded html inside and a bunch of other stuff for some reason?? I really wouldn't trust some random mediafire link

@therealOri This issue was pinned 3 weeks ago. That was when the update that broke BD was introduced in canary. The same update landed in stable yesterday or today. Follow the link in my earlier comment for more info and stop expecting me to spoonfeed you.

@ObserverOfTime
I don't expect to be spoonfed information.
and thanks for clarification on when it was pinned, It only showed up for me after my issue was closed.
I just expect to have at least a link that redirects me or anyone to the "duplicate" issue when you closed my main issue so I could look into it further myself. Instead of sifting through 80+ closed issues to find the not so duplicated "duplicate" issue.
It's not even that hard to do either, literally just copy the link and paste it with a reason for closing the issue.
Closed for being a duplicate: <link of OG issue here>.
that simple and saves everyone else time. AND they get an answer.
instead of "closed" and added some labels.
(i'd respond to you on my closed issue but that is not where you pinged me.)

@therealOri I had to close half a dozen duplicates so I stopped responding after the first two.
It's not even that hard to just read the pinned issues before submitting a new one.

I'm going to reopen this until we either officially deprecate betterdiscordctl or make it compatible with the new installation method.

Working with the new BetterDiscord installation is very doable.

That's good. I hope to get a good betterdiscord again!

You can use the GUI installer in the meantime.

I'm going to reopen this until we either officially deprecate betterdiscordctl or make it compatible with the new installation method.

I Hope betterdiscordctl won't get deprecated, I honestly love the CLI tool, the AppImage is fat for not much and can't be scripted.
Thanks for betterdiscordctl !

I'd really love for betterdiscordctl not to get deprecated as well, having an easy and minimal CLI alternative to the GUI application would be great.

Sorry for all the delay. betterdiscordctl v2.0.0 is in final review, and should be released within the next couple days. Early tests of #120 from anyone running Snap or Flatpak would be appreciated.

No luck for me to install on Flatpak
Flatpak 1.10.2
Discord: Stable 85712 (10a9883)
Linux 64-Bit (Fedora v33)

(Sorry for Cyrillic)
Discord 0.0.14
(node:5) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
Gtk-Message: 01:47:52.909: Failed to load module "xapp-gtk3-module"
Starting app.
Starting updater.
[Modules] Modules initializing
[Modules] Distribution: remote
[Modules] Host updates: enabled
[Modules] Module updates: enabled
[Modules] Module install path: /home/mkikets/.var/app/com.discordapp.Discord/config/discord/0.0.14/modules
[Modules] Module installed file path: /home/mkikets/.var/app/com.discordapp.Discord/config/discord/0.0.14/modules/installed.json
[Modules] Module download path: /home/mkikets/.var/app/com.discordapp.Discord/config/discord/0.0.14/modules/pending
[Modules] No updates to install
[Modules] Checking for host updates.
[Modules] Host is up to date.
[Modules] Checking for module updates at https://discord.com/api/modules/stable/versions.json
[Modules] No module updates available.

And... Discord act like a browser app...

@mkikets99 Did you use betterdiscordctl v2.0.0? (betterdiscordctl --version will tell you.)
Try running cd ~/Downloads, curl -LO "https://github.com/bb010g/betterdiscordctl/raw/beta/betterdiscordctl", and then chmod +x ./betterdiscordctl. ./betterdiscordctl --version (not betterdiscordctl --version; you want to use the version in your user's Downloads directory) should print betterdiscordctl 2.0.0. Try using ./betterdiscordctl with your Flatpak install.
If you still have problems, please provide the commands you ran (and their outputs) and the outputs of ./betterdiscordctl --version & ./betterdiscordctl status.

@mkikets99 Did you use betterdiscordctl v2.0.0? (betterdiscordctl --version will tell you.)
Try running cd ~/Downloads, curl -LO "https://github.com/bb010g/betterdiscordctl/raw/beta/betterdiscordctl", and then chmod +x ./betterdiscordctl. ./betterdiscordctl --version (not betterdiscordctl --version; you want to use the version in your user's Downloads directory) should print betterdiscordctl 2.0.0. Try using ./betterdiscordctl with your Flatpak install.
If you still have problems, please provide the commands you ran (and their outputs) and the outputs of ./betterdiscordctl --version & ./betterdiscordctl status.

Ok, so is is posible to install on standard version...
But... Did something changed in 2.0.0? I mean...
[mkikets@localhost ~]$ betterdiscordctl install --flatpak
ERROR: Unknown top-level option: --flatpak
Use `betterdiscordctl --help` for more information.
.......
[mkikets@localhost ~]$ betterdiscordctl install --d-install flatpak
ERROR: Unknown top-level option: --d-install
Use `betterdiscordctl --help` for more information.
.......
[mkikets@localhost ~]$ betterdiscordctl install -i flatpak
ERROR: Unknown top-level option: -i
Use `betterdiscordctl --help` for more information.

Did I missing something?

@mkikets99 You need to use ./betterdiscordctl --flatpak install, as --flatpak is a top-level option. The first error should have said ERROR: Unknown |install| option: --flatpak, which I'll fix.

@mkikets99 You need to use ./betterdiscordctl --flatpak install, as --flatpak is a top-level option. The first error should have said ERROR: Unknown |install| option: --flatpak, which I'll fix.

Well seems like something isn't working...
[mkikets@localhost ~]$ betterdiscordctl --version
betterdiscordctl 2.0.0
[mkikets@localhost ~]$ betterdiscordctl --flatpak install
ERROR: Unknown top-level option: --flatpak
Use `betterdiscordctl --help` for more information.

It's betterdiscordctl -i flatpak install now. Don't ask why.

Oh yeah, sorry. Forgot we changed the syntax for that.

betterdiscordctl v2.0.0 has been released! https://github.com/bb010g/betterdiscordctl/releases/v2.0.0 provides a changelog that goes over the CLI refresh. If you have problems with the new release, reporting them in new issues would be preferred so that everyone subscribed to this issue isn't notified.


[Question] Error about numpy and nevergrad version?

:question: Question
When running nndet_prep, got error about numpy has no attribute 'int' in nevergrad==0.4.2
How to fix it?
2024-01-22 03:05:20.885 | INFO     | nndet.planning.architecture.boxes.c002:_plan_anchors:258 - Filtered 5 boxes, 116 boxes remaining for anchor planning.
Traceback (most recent call last):
  File "/home/zhang-h/anaconda3/envs/nndetection/bin/nndet_prep", line 33, in <module>
    sys.exit(load_entry_point('nndet', 'console_scripts', 'nndet_prep')())
  File "/home/zhang-h/pytest/classic_models/nnDetection/nndet/utils/check.py", line 62, in wrapper
    return func(*args, **kwargs)
  File "/home/zhang-h/pytest/classic_models/nnDetection/scripts/preprocess.py", line 406, in main
    run(OmegaConf.to_container(cfg, resolve=True),
  File "/home/zhang-h/pytest/classic_models/nnDetection/scripts/preprocess.py", line 335, in run
    run_planning_and_process(
  File "/home/zhang-h/pytest/classic_models/nnDetection/scripts/preprocess.py", line 162, in run_planning_and_process
    plan_identifiers = planner.plan_experiment(
  File "/home/zhang-h/pytest/classic_models/nnDetection/nndet/planning/experiment/v001.py", line 43, in plan_experiment
    plan_3d = self.plan_base_stage(
  File "/home/zhang-h/pytest/classic_models/nnDetection/nndet/planning/experiment/base.py", line 234, in plan_base_stage
    architecture_plan = architecture_planner.plan(
  File "/home/zhang-h/pytest/classic_models/nnDetection/nndet/planning/architecture/boxes/c002.py", line 127, in plan
    res = super().plan(
  File "/home/zhang-h/pytest/classic_models/nnDetection/nndet/planning/architecture/boxes/base.py", line 352, in plan
    anchors = self._plan_anchors(
  File "/home/zhang-h/pytest/classic_models/nnDetection/nndet/planning/architecture/boxes/c002.py", line 270, in _plan_anchors
    params = self.find_anchors(boxes_torch, strides.astype(np.int32), anchor_generator)
  File "/home/zhang-h/pytest/classic_models/nnDetection/nndet/planning/architecture/boxes/base.py", line 445, in find_anchors
    import nevergrad as ng
  File "/home/zhang-h/anaconda3/envs/nndetection/lib/python3.9/site-packages/nevergrad/__init__.py", line 7, in <module>
    from .parametrization import parameter as p
  File "/home/zhang-h/anaconda3/envs/nndetection/lib/python3.9/site-packages/nevergrad/parametrization/__init__.py", line 9, in <module>
    from . import parameter as parameter
  File "/home/zhang-h/anaconda3/envs/nndetection/lib/python3.9/site-packages/nevergrad/parametrization/parameter.py", line 14, in <module>
    from .data import Array as Array
  File "/home/zhang-h/anaconda3/envs/nndetection/lib/python3.9/site-packages/nevergrad/parametrization/data.py", line 16, in <module>
    BoundValue = tp.Optional[tp.Union[float, int, np.int, np.float, np.ndarray]]
  File "/home/zhang-h/anaconda3/envs/nndetection/lib/python3.9/site-packages/numpy/__init__.py", line 324, in __getattr__
    raise AttributeError(__former_attrs__[attr])
AttributeError: module 'numpy' has no attribute 'int'.
`np.int` was a deprecated alias for the builtin `int`. To avoid this error in existing code, use `int` by itself. Doing this will not modify any behavior and is safe. When replacing `np.int`, you may wish to use e.g. `np.int64` or `np.int32` to specify the precision. If you wish to review your current use, check the release note link for additional information.
The aliases was originally deprecated in NumPy 1.20; for more details and guidance see the original release note at:
    https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations

I tried to downgrade numpy to 1.19.5, but also seems to have dependency issues.
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
contourpy 1.2.0 requires numpy<2.0,>=1.20, but you have numpy 1.19.5 which is incompatible.
matplotlib 3.8.2 requires numpy<2,>=1.21, but you have numpy 1.19.5 which is incompatible.
nibabel 5.2.0 requires numpy>=1.20, but you have numpy 1.19.5 which is incompatible.
pandas 2.1.4 requires numpy<2,>=1.22.4; python_version < "3.11", but you have numpy 1.19.5 which is incompatible.
scikit-image 0.22.0 requires numpy>=1.22, but you have numpy 1.19.5 which is incompatible.
scipy 1.11.4 requires numpy<1.28.0,>=1.21.6, but you have numpy 1.19.5 which is incompatible.
seaborn 0.13.1 requires numpy!=1.24.0,>=1.20, but you have numpy 1.19.5 which is incompatible.

Seems solved by remove version request in requirements.txt ... not sure though.

Should be fixed with #182


Automated cherry pick of #1486: fix azure set secgroup error

Cherry pick of #1486 on release/2.10.0.
#1486: fix azure set secgroup error

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@ecbf7c2). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release/2.10.0   #1492   +/-   ##
================================================
  Coverage                  ?   7.74%
================================================
  Files                     ?     409
  Lines                     ?   62161
  Branches                  ?       0
================================================
  Hits                      ?    4815
  Misses                    ?   56937
  Partials                  ?     409

Flag
Coverage Δ

#aFlag
7.74% <ø> (?)

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 ecbf7c2...7362580. Read the comment docs.

/lgtm
/approve


test: add integration test for karapace

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Checklist

[x] I have added necessary unit tests and integration tests
All passed https://buildkite.com/risingwavelabs/integration-tests/builds/667

Documentation

[x] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Karapace is a drop-in replacement for Confluent Schema Registry. Developed by Aiven, it guarantees API compatibility with Schema Registry 6.1.1  and can achieve higher performance.
Please add a note about the support in https://docs.risingwave.com/docs/current/ingest-from-kafka/#read-schemas-from-schema-registry the schema registry section.
Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

update https://github.com/risingwavelabs/risingwave/blob/main/ci/scripts/notify.py#L53? So we can receive notifications when the test fails.


Added support for extra Cassandra connection url arguments

Specifically these properties:

cluster.IgnorePeerAddr
cluster.ConnectTimeout
cluster.Timeout

FYI this repo is deprecated, and not maintained anymore.
You probably want to contribute to https://github.com/db-journey/cassandra-driver instead.
If you really need this to be merged here, I will do it once the import path is fixed.


Add host url to Environment Variables

Localhost is currently hard coded. Replace with an environment variable with 'http://localhost as a default', the environment variable should be a comma separated list of URLs.
HOST_URLS
is the variable, for example:
'http://localhost, https://localhost,'

Resolved.


Make site look decent on mobile

Some simple responsive CSS would go a long way to making the site look decent on mobile.

I'm planning to work on this.

Made many changes to make this look better on mobile in #56.
I removed the top menu, since those links are elsewhere.  We could potentially re-add them, hidden behind a button.  Otherwise they should go, because they send the demo below the fold.

Fixed by #56


Remove Multi-Table Queries from Packs

Background
Some of the queries recently added to packs involve more than one table, which may result in the pack failing to update if the customer has not ingested the corresponding log types yet.
Typically, each pack contains items focusing a single log type (or family of log types). When a customer creates a log source for those log types and ingests data, the data lake tables for those log types are also created. In some queries (namely the "Unauthorized Crowdstrike Device" queries), 2 tables are referenced: a crowdstrike table, and whichever log table the pack utilizes. If the customer hasn't previously ingested crowdstrike logs, the crowdstrike table won't exist, and the query will fail to compile (leading to the pack failing to update).
The same issue can happen if we move these queries to the Crowdstrike pack - the other tables may or may not exist. For the time being, we'll remove these queries from any packs until we can determine the best way to package them.
For more context, review this Slack thread.
Changes
Removed any Crowdstrike Unregistered Device queries (and their rules) from any packs. The items still exist in the repo itself.
Testing

make test
pat check-packs still failed, but not with any new unexpected errors

I thought I wanted feedback on: this might disable the detections for any customers who already have the rule through their pack, and added the query manually. I don't expect there to be many (if any) customers who have done this, but worth calling out. The more conservative approach would be to just remove the queries and leave the rules in - the rules will simple never execute if the query isn't present, and if it is, they will operate normally.
Thoughts?


react native app restarted when changing permission through settings

Is this a bug report?
Yes

Have you read the Bugs section of the How to Contribute guide?
Yes

Environment

react-native -v: react-native-cli: 1.0.0, react-native: 0.35.0
node -v: v6.6.0
npm -v: 3.10.3
yarn --version: --NA--

Then, specify:

Target Platform: Android(>=M) and Ios

Development Operating System: Ubuntu 14.04LTS, OS X 10.10: Yosemite

Build tools: Default

Steps to Reproduce

Take a new/existing react native project. Please make sure it have some two pages[Page 1(initial), Page 2].
Add any permissions(like Camera, Storage,.....) to AndroidManifest.xml(For Android) & info.plist(For Ios) file.
Run the app on emulator/simulator/a real device.
At initial, Page 1 will be presented to the user.
Navigate to Page 2.
Minimize the app and go to settings and change the app permissions(Camera, Stroage....).
Resume the app. It will open the Page1.

Expected Behavior

App needs to be in Page2.
Actual Behavior

App gets restarted and initial page will be displayed.
Reproducible Demo

https://github.com/pandiaraj44/ReactNativePermissionChangeBug
Video Ref: https://drive.google.com/file/d/0B9gIc_qkvdRIbm4xR3U2R292OHc

Actual Description
I have a react native app. Which is working fine in both ios & android but if change any permissions(Camera, Storage) through settings option then app get restarted.
For Example, If I am on the home-screen and I had changed the permission then the app will get restarted and takes me to the login screen.
Are there any options to maintain current app state(both ios and android)?

Any chance you can provide an up to date repro? The link you provided is for an older version of RN, o.35.

@hramos
I will create the new project and will share the link.
FYI...
I think for these kind of bug we can take react native default awesome project itself.

@hramos
Reproducible Demo Link updated.
https://github.com/pandiaraj44/ReactNativePermissionChangeBug

Hi, @hramos
If possible, Please add tags/assignees to this issue, May be it will be visible to other contributors and they might help.

Hi @hramos, Is there any updates?

Sorry, I don't have anything to add here. Let me know if you open a PR with a fix for this.

Hi, I am experiencing this issue as well, same scenario though it also appears in iOS apps. It was there in iOS 10 and it is still in iOS 11 with RN upgraded to the latest version. @hramos should we also attach the iOS label to the issue?


add maintenance container

Related issue Canasta#33
Related PRs Canasta-Maintenance#1, Canasta#266
This PR adds a separate container to run maintenance script for MediaWiki installation
changes

Added a maintenance container in the docker-compose

testing
I have tested the changes made by importing an existing wiki as well as creating a new one

PS- to locally test please edit the docker-compose.override.yml to point to the local images as there is no image in registry for the maintenance container

Closing this PR, since it was decided (almost a year ago) not to create a separate container.


请问V3 支持支持自定义的groupType吗，类似于registerGroup 这样的api

G6 Version: v3.4.0
Platform:
Mini Showcase(like screenshots):
CodePen Link:

3.4 暂时不支持。3.5 会重新设计节点分组相关，之后会支持。

When is the expected release date for  3.5 ?

v3.5 is released. Try it here:
Built-in combos: https://g6.antv.vision/en/examples/item/defaultCombos
Custom combos: https://g6.antv.vision/en/examples/item/customCombo
Combo with layout: https://g6.antv.vision/en/examples/net/comboForce
Combo behaviors: https://g6.antv.vision/en/examples/interaction/combo
For the docs, you can search 'combo' with the search bar on the top of our website.


Cleanup copy+paste UI

This updates to copy+paste UI to use styles from the new mocks
Fixes #1743

Right now there is only an entry to paste in the sharer's link, not the getter's link.  i.e. if Alice clicks setup a one-time connection to get access, then sends her link to Bob, Alice will have an entry to paste in Bob's sharing link, but Bob won't have an entry to paste in Alice's getter link (he will have to navigate to the URL to get that "Share a one-time connection" screen).  Do we have any plans to make an entry for pasting the getter's link?
Otherwise the code looks good

Completely agree with you on that, it was not in the latest version of the
mocks however and Greg thought it would be best to finish this part and
then wait for that to come later.
On Wed, Jul 29, 2015, 11:11 dborkan<EMAIL_ADDRESS>wrote:

Right now there is only an entry to paste in the sharer's link, not the
getter's link. i.e. if Alice clicks setup a one-time connection to get
access, then sends her link to Bob, Alice will have an entry to paste in
Bob's sharing link, but Bob won't have an entry to paste in Alice's getter
link (he will have to navigate to the URL to get that "Share a one-time
connection" screen). Do we have any plans to make an entry for pasting the
getter's link?
Otherwise the code looks good
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/pull/1753#issuecomment-125983811.

:+1:


clang-tidy execute error

Hi, I'm proceeding to the command below, but there's an error
make prepare
make install
cmake .. -G "Unix Makefiles"
cmake --build . -j$(nproc)
cmake --build . --target main_clangtidy

cmake --build . --target main_clangtidy
LLVM ERROR: CommonOptionsParser: failed to parse command-line arguments. [CommonOptionsParser]: clang-tidy: Unknown command line argument '--config-file=/workspace/CppProjectTemplate/.clang-tidy'.  Try: '/usr/bin/clang-tidy --help'
clang-tidy: Did you mean '--config=/workspace/CppProjectTemplate/.clang-tidy'?

Unable to run clang-tidy.
make[3]: *** [app/CMakeFiles/main_clangtidy.dir/build.make:70: app/CMakeFiles/main_clangtidy] Error 1
make[2]: *** [CMakeFiles/Makefile2:480: app/CMakeFiles/main_clangtidy.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:487: app/CMakeFiles/main_clangtidy.dir/rule] Error 2
make: *** [Makefile:260: main_clangtidy] Error

Most likely, you need to upgrade your clang tidy version

I'm doing it inside the docker container, but I searched it and it seems to be related to the docker, so could you tell me?
and I use clang-tidy version 1.10.0

Are you sure with 1.10?
There is already 14 and newer

@franneck94
https://docs.nvidia.com/deeplearning/frameworks/pytorch-release-notes/rel_22-07.html
The container I use is the same as above, but I think there is a limit to the apt... So, is it a way to build a source for clang-tidy?
Cppcheck works, but it's frustrating that it doesn't work.

I wouldnt recommend this


Jenkins & Docker into develop

Worth noting that once we deploy it may be easier to run Jenkins directly on the instance and then deploy using a docker (This way docker only is running one project)

This branch is outdated and will not be merged into develop as its contents have been included in another branch.


Opening the Godot project with a gdextension just closes Godot with no message

Godot version
4.0.3
System information
Mac OS Arm64
Issue description
I have a gdextension that uses SwiftGodot. I know that there is something with some particular scenes that are causing issues with Godot.

Install the extension
Open the Godot project
The Godot project auto-opens the scene in question
About two seconds into the project opening Godot will just quit completely
There is no error message displayed or crash log in Console. There isn't any information on why Godot decided to quit.

I can get out of this state by:

Changing the name of my gdextension file to "Extension.gdextension_"
The project now opens fine
Open a different scene and make sure to close the "bad" scene
Re enable the extension

However I still have zero feedback on why that scene in question had issues.
Steps to reproduce
Ultimately having a gdextension that causes these issues. I don't really have anything more than a whole project to show since I can't troubleshoot the root cause of why this gdextension is causing issues since Godot doesn't seem to handle it gracefully
Minimal reproduction project
I don't have one for now

@nvanfleet Please upload a minimal reproduction project to make this easier to troubleshoot.

Closing due to lack of response. Please comment if you can still reproduce this bug on the latest Godot version.
PS: Since no minimal reproduction project was included in the original bug report, please upload one as well to ease troubleshooting.


portal and CLI methods?

How about including a reference to the procedure to create a VM from the snapshot via the portal.
Also, if this can be done using the CLI, please outline that as well since that is my goto mechanism!

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

ID: 1bb0d5aa-46fa-b15d-8eec-c724b5ae468c
Version Independent ID: 1977490f-d9c6-574c-c6a4-36b0ebd484b8
Content: Create a snapshot of a VHD in Azure - Azure Windows Virtual Machines
Content Source: articles/virtual-machines/windows/snapshot-copy-managed-disk.md
Service: virtual-machines-windows
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

Here is the document to create a VM from the snapshot via Azure portal:
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/create-vm-specialized-portal

Is there any update? Did it help, let me know. Thanks. :)

Hi Karishma,
Yes, it did help.  Thanks.
But mainly, I was suggesting the document be updated to include links on alternate methods than just powershell.  Is Bash CLI also possible?
Gerald J.
?

From: Karishma Tiwari - MSFT<EMAIL_ADDRESS>Sent: December 12, 2019 6:27 PM
To: MicrosoftDocs/azure-docs
Cc: Justice, Gerald; Author
Subject: Re: [MicrosoftDocs/azure-docs] portal and CLI methods? (#44514)
Is there any update? Did it help, let me know. Thanks. :)

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/44514?email_source=notifications&email_token=AISQL76XOBDD76NF2CW4PQ3QYLXH3A5CNFSM4JZUOQCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGYWGNA#issuecomment-565273396, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AISQL7YTVXPJN24ZUR423C3QYLXH3ANCNFSM4JZUOQCA.

Thank you for the feedback. I will share this feedback to the docs team.
You can refer to this doc, samples for the Azure CLI are written for the bash shell.
We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


Add Schema Master Info to SetupAssist.ps1

Describe The Request

Add Schema Master to output, domain, site
Throw run /PrepareAd and where if Mismatch detected. Throw additional information if you aren't in Schema Admins and Enterprise Admins
Also add Exchange 2013 to the list of AD checks

#428


Component Explorer | DEV | Update name spacing

Jira
http://vjira2:8080/browse/BDS-1373
Summary
Update component generator code to be in line with the newest standards and naming conventions
Details
Update component generator code to be in line with the newest standards and naming conventions
How to test
Check the code, try to generate component and check if all test is passing.

@adamszalapski thanks for taking care of this! I tried to use the component generator recently and found it was out of date and broken, so big thanks.
I made a couple minor edits (see commit):

Use "bind" instead of "dedupe" classname package. I think "bind" is the more commonly used one.
Removed the constructor entirely from the component template. Most components don't have to do anything in the constructor anymore, so let's not assume they need it.

Let me know if you disagree.
Otherwise, good to go. Everything worked perfectly 👍

Ship it! 🚀


Use libraries from npm

Such libraries as IndexedDBShim, jquery*, resampler, sugar, zepto should definitely be removed from the repository and we should instead reference the corresponding npm packages. I am not sure about libraries/ui.js - it have no clues to real library name and where could we got it. Moreover, it seems that it was generated from something.
The same about library CSS.

fixed in 2df0827eaf


Improve GIF

[x] faster speed
[x] pause slightly before hitting enter
[x] fix missing characters
[x] make short version for readme

It is finally done!


feature request: open_on_start for "terminal" strategy

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
The toggleterm strategy has an option open_on_start which is quite useful,
it is great to add this option to the default strategy terminal.
Provide background
No response
What is the significance of this feature?
nice to have
Additional details
No response

I think if I were to redesign the toggleterm integration, I would pull some of the functionality out of the strategy and into components. This logic is something that I would expect to be in a component so it can be easily customized per-task


sc-cli: add no-beefy flag to cli config

BEEFY is soon expected to run on all substrate-based chains, make it part of sc-cli.
Note: in case of polkadot binary, CLI doesn't change. --no-beefy is moved from polkadot specific cli to sc-cli.
polkadot companion: https://github.com/paritytech/polkadot/pull/7600
cumulus companion: https://github.com/paritytech/cumulus/pull/2996

LGTM! I may also need a cumulus companion

You're right! done.

bot merge

BEEFY is soon expected to run on all substrate-based chains

That is a bold statement that isn't true...
I know that we also have there disable_grandpa, that is just some old oversight and we should not replicate it.

BEEFY is soon expected to run on all substrate-based chains

That is a bold statement that isn't true...

I hope it will prove true 😛

I know that we also have there disable_grandpa, that is just some old oversight and we should not replicate it.

So you suggest we revert to having --no-beefy as a polkadot-specific CLI flag as opposed to sc-cli flag?

So you suggest we revert to having --no-beefy as a polkadot-specific CLI flag as opposed to sc-cli flag?

Yes please. Not all Substrate chains are running grandpa either and the same will apply to beefy.


Android Embedding Migration docs page is giving page not found message.

I wanted to do some maintenance work on  an old flutter project and it had a warning that android v1 embedding is not functional anymore which my project was using.
Warning
──────────────────────────────────────────────────────────────────────────────
Your Flutter application is created using an older version of the Android
embedding. It is being deprecated in favor of Android embedding v2. Follow the
steps at

https://flutter.dev/go/android-project-migration

to migrate your project. You may also pass the --ignore-deprecation flag to
ignore this check and continue with the deprecated v1 embedding. However,
the v1 Android embedding will be removed in future versions of Flutter.
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
The detected reason was:

  /Users/srihari/Downloads/local_hero-master/example/android/app/src/main/AndroidManifest.xml uses
  `android:name="io.flutter.app.FlutterApplication"`
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

Build failed due to use of deprecated Android v1 embedding.

I followed the link it gave me in the error to view the documentation on how to migrate. But the webpage is giving me a page not found type message.
This is the link in question. I will attach a few screenshots just to be clear.

Why was this documentation removed ? And if it was removed for a reason, I think the warning/error message should be updated with the new link.
Also, if I google the migration documentation, it takes me to the same mentioned link and it gives 404 that way too.
Please look into this. Thanks.

Duplicate of https://github.com/flutter/website/issues/7267


PVRE reporting for short lived instances

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Have we verified that this template fixes our two issues that we have been having ?

I created a launch template running this CFN stack. I can validate if we can create a MNG with this LT which ideally should fix the issues. Not sure if I could validate it further

I created a launch template running this CFN stack in my dev account. If I can have permissions for perf lab, I can validate if we can create a MNG with this LT which ideally should fix the issues. Not sure if I could validate it further

I will give you access to perflab, we should verify if this task is running smoothly on experimental infra.

Overall nice work @sushimaAws.

Do not merge until internal changes are reviewed and merged as it will break current tests.


Wrong location on Apollo 5.0

Hi,
I downloaded Apollo from the following URL and used the san_francisco map.
https://github.com/ApolloAuto/apollo/tree/r5.0.0
When I am using LGSVL and Apollo 5.0, the location information cannot be synchronized, as shown in the figure below.
However, I did not encounter this situation when I simulated some maps such as borregas_ave and go_mentum.
How can I solve the problem of location synchronization? Thank you all for your answers in advance!

time sync
The error message is that the time of the IMU is not syn chronized. If it is two machines, make sure that the time is consistent.

It's the same machine and it's doesn't matter. Because I also meet this error and it also works well.

time sync
The error message is that the time of the IMU is not syn chronized. If it is two machines, make sure that the time is consistent.

It's the same machine and it's doesn't matter. Because I also meet this error and it also works well.

Hi, I also meet this problem. Did you find a solution for that?

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


chore(release): v0.9.3 🎉

:robot: I have created a release *beep* *boop*
0.9.3 (2022-12-06)
Bug Fixes

identity: enable specifying custom metadata keys for value objects (8d06f2a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/domain-objects/releases/tag/v0.9.3 :sunflower:


[BUG] Nodes dropping from cluster - possibly due to Java 21

Describe the bug
Shortly (1-2 weeks) after upgrading from 2.8.0 (with Java 11) to version 2.12.0, with Java 21:
About 1-3 data nodes occasionally cannot respond to the cluster manager (90s timeout) and are dropped from the cluster, rejoining and dropping as soon as most shards are finally allocated. When left alone this cycle lasts around 20-30 minutes.
Log before the node is dropped:
[2024-07-23T22:44:02,791][WARN ][o.o.c.c.LagDetector      ] [PROD-manager-host] node [{PROD-data-opensearch-1234}{rXOYfZXrTDmgk_mU-AM1mw}{Hwm5ETnRREeemoBoZI2Yhg}{IP_REDACTED}{IP_REDACTED:9300}{dr}{box_type=default, zone=us-east-1a, shard_indexing_pressure_enabled=true}] is lagging at cluster state version [9372496], although publication of cluster state version [9372497] completed [1.5m] ago
Observations that correlate with a node going to be dropped -

High indexing pressure
SegRep delays
right before the drop the generic thread pool spikes to the max

Temporarily fixed by downgrading to Java 11, and in some cases to Java 17 (still on OpenSearch 2.12.0).
Related component
Storage:Durability
To Reproduce
Tried so many different ways, could not reproduce this.
Expected behavior
Nodes should remain part of the cluster, and be able to answer the cluster manager in a timely manner if they are not experiencing a truly severe, or abnormal issue.
Additional Details
Plugins
discovery-ec2, repository-s3
Screenshots

Correlated spike of indexing (filtered out all nodes except the one failing and a random other one not experiencing issues):

Host/Environment (please complete the following information):

OS: Ubuntu 20.4
Version OpenSearch 2.12.0
[EDIT:]
Java version: correto <IP_ADDRESS>-1

Additional context
Why post this as a bug?

while we don't know the actual root cause, after discussing in the last triage meeting with @peternied  and @reta we thought it could at least help the community if anyone else is seeking to remedy a similar issue and has not yet tried downgrading the Java version.

@gbbafna Can you assign it to another label, if not storage who should be investigating?

Storage Triage : Attendees 1 2 3 4
@AmiStrn  : Do we know if the issue is only present in Seg Rep clusters ? Can you please provide debug logs if any ?
For now I have removed Storage label , as the issue doesn't relate directly to it .

Sorry, we do not have debug logs for this issue. And we dont have a cluster without segrep to compare this to.

@reta updated the java version in the description we were using (correto  <IP_ADDRESS>-1) in case it is related to https://github.com/opensearch-project/OpenSearch/pull/11968

@reta updated the java version in the description we were using (correto <IP_ADDRESS>-1) in case it is related to #11968

So far it seems to be mislead (although we clarified the deployment), the tests seems to be stable for 2.12.0 and correto <IP_ADDRESS>-1

A thread dump would definitely have had been useful

100% agree. this is the kind of thing you want in hindsight but during production incidents are least likely to have time to do (not saying that is correct but it is the way it goes down in most cases).


"fn lambda create-function ...." this command not found

How to create the docker image by run the command create-function? what should I do?

Hi! have you tried aws-import:
fn lambda aws-import -h
NAME:
   fn lambda aws-import - converts an existing Lambda function to an image, where the function code is downloaded to a directory in the current working directory that has the same name as the Lambda function.

USAGE:
   fn lambda aws-import [command options] <arn> <region> <image/name>

OPTIONS:
   --payload value  Payload to pass to the Lambda function. This is usually a JSON object. (default: "{}")
   --version value  Version of the function to import. (default: "$LATEST")
   --download-only  Only download the function into a directory. Will not create a Docker image.
   --config value   function configuration

this command will import the function code from the AWS region to a directory called ./user/my-function. Inside the directory, you will find the func.yml, Dockerfile. With Dockerfile you can build an image.
If that doesn't work, let me know. Meanwhile, i recommend you to read how to import lambda function to Fn: https://github.com/fnproject/fn/tree/master/docs/lambda

May this issue be closed?

This feature was removed because was not supported well, so, yeah, let’s close this.


Add a python boto3 script for creating ec2-instance in aws

I will add a boto3 python script to run instance in aws

kindly assign it to me, and put hackoctoberfest tag thanks


Import slotted SVG component from ic-ui-kit

Summary
Currently we implement our own fix for slotted SVG errors, but this ticket fixes it in the ui kit so we can remove our fix here and import the fix from @ukic/react instead.
💬 Description
Remove slotted SVG work, and import slottedSVG from @ukic/react
💰 Use value
Users will be able to see that they can also import slottedSVG if they run into the same error.

The work for this ticket has been done, it just needs the latest components from the ui-kit to pass pipeline steps.


[Bug]: Can't create an icon request if I haven't checked the "File" checkbox

Describe the bug
When I want to create an icon request, but if I select only the "Folder" option, then I can't create an issue, because there is a bug in the template (the "File" checkbox must be selected)
To Reproduce

Click "Create issue"
Select "Icon Request"
I fill out the form
I see that I cannot create an issue because I did not select the "File" checkbox

Expected behavior
Created a request for an icon without any problems
Screenshots

Computer information
No response
Additional context
No response
Code of Conduct

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

@okineadev I had to remove the validation for the checkboxes in the beginning. There's currently no syntax available from GitHub to do this. I think it's fine to have it like that for now. If GitHub provides more validation features for checkboxes in the future, then we can add it here.

@okineadev I had to remove the validation for the checkboxes in the beginning. There's currently no syntax available from GitHub to do this. I think it's fine to have it like that for now. If GitHub provides more validation features for checkboxes in the future, then we can add it here.

Why do commits have more changes than PR?
It seems to me that those changes in PR enough
In short, then I will try to figure out the template myself..


Fix compiler warnings in Swift 4.1

Thanks! Don't know how I missed these on the initial Swift 4.1 upgrade


Test grouping (test.describe) does not apply to the name of the test

We're using test.describe to provide a top-level label on parameterized tests, per this documentation: https://playwright.dev/docs/test-annotations#group-tests
The text in the test.describe is not added to the test name, which makes it impossible to know which test we're running/debugging:

The same problem applies when you're on the Testing tab and try to view in a list view:

If we add the parameter in the test call itself, it gets added to the name:

This seems like a bug, as the Playwright output lists them just fine.

Additionally, it seems the text shown (without the text in "describe") is used as the "grep" for running the test, which means that all parameterized tests get run instead of the single instance I wish to run.

I have found this. All tests will run, not just those filtered for.


⚠ Bofrost.be has degraded performance

In c1b0f80, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2279 ms

Resolved: Bofrost.be performance has improved in b26df9a after 1 hour, 51 minutes.


Having trouble running examples/possible geopy issue

I am getting the following error message when running parse_single_from_code.py:
Traceback (most recent call last):
File "/Users/annaablove/Desktop/py_test/who.py", line 53, in
extractor = MasterExtractor()
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/Giveme5W1H/extractor/extractor.py", line 60, in init
environment_extractor.EnvironmentExtractor(),
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/Giveme5W1H/extractor/extractors/environment_extractor.py", line 55, in init
self.geocoder = Nominatim(domain=host, timeout=8)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/geopy/geocoders/nominatim.py", line 103, in init
Nominatim(user_agent="my-application")
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/geopy/geocoders/nominatim.py", line 103, in init
Nominatim(user_agent="my-application")
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/geopy/geocoders/nominatim.py", line 103, in init
Nominatim(user_agent="my-application")
[Previous line repeated 487 more times]
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/geopy/geocoders/nominatim.py", line 92, in init
super().init(
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/geopy/geocoders/base.py", line 242, in init
self.adapter = adapter_factory(
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/geopy/adapters.py", line 335, in init
self.session = requests.Session()
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/requests/sessions.py", line 367, in init
self.headers = default_headers()
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/requests/utils.py", line 821, in default_headers
return CaseInsensitiveDict({
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/requests/structures.py", line 46, in init
self.update(data, **kwargs)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/_collections_abc.py", line 854, in update
if isinstance(other, Mapping):
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/abc.py", line 98, in instancecheck
return _abc_instancecheck(cls, instance)
RecursionError: maximum recursion depth exceeded in comparison
I think it has something to do with Nominatim.py, but I am inexperienced with geopy and am having trouble isolating the problem. Any help would be appreciated, thanks!

OS: MacOS 10.15.6
Python Version 3.9

TL;DR
you can probably do
pip3 uninstall geopy
pip3 install geopy==1.13.0

For repo authors, I think setup.py needs to be updated - I'm seeing that setup.py will install geopy>=1.11.0, which gives me 2.1.0. However, it seems like geopy versions > 2.0 introduces breaking changes. I saw that in requirements.txt that geopy==1.13.0 is specified. I think setup.py needs to be updated accordingly.


Source and license

Can you tell me the source and the license about this names?
Thanks for the job!

They are extracted from a so-called "Turkish Citizenship Database", which is leaked to the Web from formal sources. I don't have the full database, but it might be found on the internet.
There is no explicit license on the data itself. It is government-collected data, so it should be public domain, but it is not formally released by the government.

Thanks for the info!


Spacing in the landing page

Some spacing issues in the landing page:

Does this fix the $subject issue:
https://github.com/wso2/docs-security/pulls


Remove 'uom' from table

In today's SWG, we reviewed #3 and decided to take it in a different direction. Since vertical measurements are useless without a vertical datum, we want to encourage GPKG clients to get both the geoid of reference and the units from the SRS. While, it would improve interoperability to have everyone use same SRS, we do not wish to constrain ourselves in this way. (We have always allowed users to use alternate SRSs as long as they are defined in the SRS table.) Therefore we agreed to add EPSG::4979 to the SRS table to encourage people to use it by default.
To make this happen, we agreed to make the following modifications:

eliminate uom from the coverage ancillary table
mandate the adding of EPSG::4979 to the SRS table for GeoPackages complying with this extension
provide guidance regarding the use of the SRS table to find the geoid of reference and the unit of measure

I will produce a pull request for this change. We can review it at next week's SWG.

This may hurt GeoTools, which does poorly on 2D<->3D conversions


Removed all deprecated API from MetaModel (aaaaah)

This is a PR for the 5.x branch. I suggest getting rid of all the deprecated methods of metamodel for that release.

@kaspersorensen just wondering we want to remove the deprecated methods in 5.x branch?
Shouldn't we just do it in master?

We could do it in master, but I guess it seems nicer to only do that sort of thing when it aligns on a bigger release.

Are all the deprecations old-ish? It might be a major release, but if we can avoid throwing out stuff that has been deprecated only a few months ago, it would be best.

The most recent deprecations where introduced in MM 4.4 AFAIK. Those where related to the split of FunctionType into AggregateFunction and ScalarFunction interfaces. So the deprecations have been there since the release date of 4.4.0: 2015-10-12.

Half a year is probably reasonable when we don't have an official policy about keeping depreciations for a certain amount of time.

The only worry I have is that this change introduce API changes that could break compilation for minor release

I agree Henry, that's why I wanted to put it on the 5.0 branch so that the expectation is also managed a bit :-)

Maybe we should do shout out to dev@ list to check if anyone still in 5.0 and ok to add this breaking API changes?

I'm not sure I understand what you mean Henry? 5.0 isn't out yet so I guess no one will be depending on it yet?

Ah I am so sorry, I need more dose on caffeine =(
I apologize for the confusion.
What I meant was, would anyone in 4.0 still need it to be around for 5.0.

Aha yes... Let's ask that.


gnss driver module for parsing

Hi, Apollo team.
Apollo has two ways to receive GPS data. One is received via CAN, and the other is received by apollo/modules/drivers/gnss.
It seems to be enough to get through CAN like a radar, but why use DRIVER module for parsing?

Driver module can isolate the hardware differences and makes it easier for other modules to use GPS data.


Fix #1 - Invalid reference to entities

@franckbrignoli The master branch now is intended to support OpenAPI 3.0 version instead of Swagger 2.  I have just created another branch swagger2 and please submit your PR to that branch instead.  Bump the version to 0.1.3 too.  Thanks very much.

@kenspirit oh right! Thanks


Increase timeout for the s3-fetch pipeline with bike-rentals

Description
Periodic job finishes with 190-194 tries out of 200 which means there is not enough room in case the environment is slower, leading to false negatives. Therefore, I am raising it to 220. Other pipelines are OK.
How Has This Been Tested?
Not needed.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/skip ci/prow/test-ai-edge

/skip

/override ci/prow/test-ai-edge

/approve

/override ci/prow/test-ai-edge

/skip


Dedent

Would be cool to see a typesafe dedent function for template strings.
This was the closed I got a few months ago.
https://gist.github.com/BLamy/4b4c369ab4a0915bf61f99e46516fc86

Hey @BLamy , thank you for the suggestion!
I actually didn't even know about dedent.
It would indeed be nice to have it in the library even though I don't think it is a common scenario to have long strings at the type level. For short strings it makes more sense, eg: type Role = 'admin' | 'editor' | 'user'
That said if anyone is feeling adventurous and wants to take a shot I'd be curious to see a PR.

I use long string literals in my prompt-builder library pretty extensively.
https://blamy.gitbook.io/prompt-builder/promptbuilder

Good to know @BLamy . @jly36963 told me he will take a shot at it.

I'm gonna close this issue as it is just a feature request and it is currently in our roadmap.
Make sure you participate in the discussion if you want to see it implemented soon ;)


The example angular2 project doesn't work in ie11

Any ideas why? I managed to get it to work by including the es6-shim as a script tag but I don't think this is the correct solution.

Thanks, it was failing because the plugin used Object.assign, this should be resolved in 2.2.3.

It still doesn't run for me. I updated to 2.2.3 and I was getting errors about "Object doesn't support property or method 'keys'" in System.src.js. I tried upgrading to angular2 alpha46 and that didn't resolve it.
Furthermore, the es6-shim doesn't seem to resolve all the issues anymore. I get errors like "Object doesn't support property or method 'wtfCreateScope'" and "Unable to get property 'prototype' of undefined or null reference"

Can you re-open this issue?

I'm facing the same issue. Any ideas?

I can recreate what you are seeing but I don't know how to fix it. It is my understanding that angular2 requires es6-shim to run on IE11, and perhaps this is conflicting with the SystemJS polyfills?

I'm getting much the same issue in Edge. Complains about not finding 'detectWTF' and 'wtfCreateScope' (but doesn't give any particular file for the error). Works fine when running bundled.

Did you try using angular2 beta.0 instead of beta.1? Beta1 caused some issues for me too.

I'm getting this aswell, not finding 'detectWTF' and 'wtfCreateScope' in IE11, using angular2 beta.1.
This is without using plugin-typescript.

closing due to inactivity


Always fetch the pint 'application registry' from within functions

As a follow-up to #312, switch to an implementation where pint.get_application_registry() is always called from within functions. This allows downstream code to replace this registry.
Add a comment explaining this choice for future developers.

PR checklist

[x] ~Tests added.~ N/A
[x] ~Documentation added.~
[x] ~Release notes updated.~

The codecov/patch check failure is spurious.


Unify numerical tests

Better to unify numerical tests using a function like
check_numerical(chain, variables, exact, eps, strict) = begin
   ...
end

and use it by
check_numerical(res, [:s, :m], [49/24, 7/6], 0.1, false)

If strict is set to true @test will be used and throw error when test fails.

That makes sense. We can also do something similar to benchmark, e.g. create a special build on travis that perform numerical tests. This would allow us to separate numerical tests from common unit tests.


Add localization support

Localize all plugin's texts (also measure units).
Includes spanish localization.

All localization files are loaded at the begining. Maybe this can be improved loading only the selected localization.

Added russian locale from bekerov

Added french locale and separator localization from Nyxelio.
Added separator customization and used the localization's separator values by default.


Collect fragments while concatenating xsl-fo

Many thanks for contributing. The implementation looks ok, but it needs a test before this can be merged. There is an FOConcatenationSuite, you can add the logic to the existing success case, or you can add a third test. I'm fine with both.

Superseded by #607 which also contains a test and builds on #606 which properly deals with fragments with the same name.


Consider making PlaybackSettings::ONCE not the default or removing it

Motivation
Following up on a question posed during the "ecs-based audio" refactor in https://github.com/bevyengine/bevy/pull/8424#issuecomment-1604451931 which I don't think was ever really addressed.

This begs the question: should PlaybackMode::Once even exist? Does it even make sense to retain audio-related components, if sound is no longer playing?

As well as this comment on PlaybackMode:
https://github.com/bevyengine/bevy/blob/f67b17d80d410045f0fb0af668521e2f91569159/crates/bevy_audio/src/audio.rs#L76-L79
Motivated by a recent Discord discussion about "how to replay a sound."
What problem does this solve or what need does it fill?
PlaybackSettings currently defaults to PlaybackMode::Once, meaning the audio will play once, and then the audio entity and its components are left alone and not despawned or removed.
That sounds convenient at first. After the AudioSink is created, it can be re-used, saving you the CPU cycles of spawning, despawning, and creating the sink. Or archetype-moves adding/removing components.
Unfortunately, it really can't be reused. Once the audio in the sink's queue is played, it is gone, and there is no way for a user to refill the sink and start playback again. The only way I know of to restart audio is to remove the AudioSink component from the entity -- play_queued_audio_system will create a new one. That trick doesn't work for playing a different sound with the same sink. See #11862.
So why does this mode exist, and why is the default behavior to leak audio entities into the world and never clean them up?
What solution would you like?
Make the default behavior PlaybackMode::Remove, and add an internal system that cleans up "orphaned" entities when audio components are removed and the entity is otherwise "empty."
Leave PlaybackMode::Once around in case it is useful to someone. Add docs explaining why it's sort of useless?
What alternative(s) have you considered?

Make the default behavior PlaybackMode::Despawn. This is might be what users want most often. Leave Once in case it's useful to someone.

Same, but remove PlaybackMode::Once.

Keep current default behavior, remove the TODO comment.

Add some sort of machinery for reusing a sink by refilling it or filling it with different audio.
I naively attempted to add append to the AudioSinkPlayback trait, but it required making the trait generic, and it seemed like a mess. It's probably possible, but that route would lead to a pretty inconvenient API that seems counter to the "ecs-based audio." Using change detection on Handle<Source> seems like it would be pretty funky too.

This is useful to me at least, though bevy_audio needs some patching before it is, but being able to keep a sink on an entity and later play another souncd/disallow playing another sound. At any rate I almost exlusively use ONCE as a policy. Using the other policies is basically asking for lots of archetype table moves.

See https://github.com/bevyengine/bevy/pull/12035 which is the start of an attempt to cover such a use-case. I'd like to get in shape this release, it's POC atm as I'd like to hear how bevy_audio SME if there is one would like to handle it, my attempt was just to show how I'd like to use it.

I feel like exposing raw rodio types is probably a non-starter as a user-facing API.

See #12035 which is the start of an attempt to cover such a use-case. I'd like to get in shape this release, it's POC atm as I'd like to hear how bevy_audio SME if there is one would like to handle it, my attempt was just to show how I'd like to use it.

I feel like exposing raw rodio types is probably a non-starter as a user-facing API, but I am definitely not a subject-matter expert.

I'm agnostic to a particular implementation, but the behaviour of re-use is very wanted.

Entity commands could fit here, I think. The command would either queue the audio onto the existing Sink if already existing, therefore reusing it, or create a new AudioSink component otherwise.

Thanks for clarifying that for me. I was wondering if I could reuse an existing audio component that plays once for some events (like button clicks) or if I should just re-recreate the component to play the sound each time the event occurs (which seems a little wasteful, hence my hesitation).
I guess it is the later then.

I don't think that I agree with the premise of this issue anymore. I am not sure that entities "magically" despawning themselves or components removing themselves make for great defaults in an "ECS-based API". This behavior is likely to be just as surprising to people.
I think that the lack of sink re-usability should be treated as a bug and possibly documentation improved to reflect the current limitations if it doesn't seem like a fix is forthcoming.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2779

Questions ?
Please refer to the Backport tool documentation


Run the uncompressed game data file on the browser

Run the uncompressed game data file on the browser instead of loading an entire XP3 into the browser.
A large-scale game data file can reach more than 1GB.
It takes a long time to load all of them from the web server and it may cause lagging.
Moreover, compared with the decompressed multimedia files, the huge data files are not conducive to CDN transmission
Can you add support for running TJS scripts in the browser instead of loading the entire XP3?

Running of individual TJS scripts can be done by Module.evalTJS in a JS script.
For downloading individual files, can use Fetch or XMLHttpRequest.
See the following for more detailed documentation of the browser interface.
https://github.com/krkrsdl2/krkrsdl2/wiki/Web版吉里吉里SDL2-JS⇔TJS-インターフェースについて


operators/sample/sample.cpp file

I think the file has some blank on the first line. Can you check this issue?

Hi @yuuun, thanks for your attention to CogDL. There is some useless blank in the first line of sample.cpp file. And we will remove it later.


Can I force the mouse position on the specified monitor when hyprland starts？

Hyprland Version
Hyprland, built from branch main at commit 499df49f7b28f9e3be9b1c53843fd5c465dec60b dirty (dispatchers: fix extra monitor offset when snapping floating (3520)). Tag: v0.30.0-68-g499df49f
Bug or Regression?
Bug
Description
I'm using dual monitors, and I've set them up to go up and down. But I found that when I entered my account password into hyprland, the default mouse start position was always at the bottom of the display.
I also used xrandr to set up the above monitor as the primary monitor, but this didn't work
Can I force the mouse position on the specified monitor when hyprland starts？
my monitor.conf
monitor = ,preferred,auto,1
monitor = eDP-1,preferred,0x0,1
monitor = DP-1,preferred,0x1080,2
workspace = 1,monitor:eDP-1,default:true
workspace = 2,monitor:DP-1
exec-once = xrandr --output eDP-1 --primary

How to reproduce
No need to reproduce.
Crash reports, logs, images, videos
No response

wayland has no such thing as a primary display.
You can call hyprctl to execute a focusmonitor, technically.
Not a bug anyways


Update COD 3W and requirements

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.511%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
1754

Relevant Lines:
1896

💛  - Coveralls


Unable fetch data from unifi controller Unmarshalling Device Type: uap, usw

No description provided. is where you write what's going on.

After launching unifi-poller gives an error in the logs Unmarshalling Device type: uap, usw as shown in the screenshot

t

That's not an error?

I'm sorry... it wasn't a problem with unpoller I had trouble with scrapping from unpoller. I has fixed already


Pacing line doesn't line up between master and sub-categories

Defective ToolKit Feature Name: Budget Rows Progress Bar
Expected Behavior
Pacing line should line up across all categories.
Actual Behavior
Pacing is different on master categories and sub-categories. This began after the last update to YNAB_VERSION: v1.15388. The width of the checkbox column on a sub-category line was increased. Below are screen shots of before and after. Notice the amount of white space between the checkbox and the beginning of the shaded area.
Before:

After:

Steps to Reproduce the Behavior

Select "Pacing on name column and goals on budgeted column"

Export of Toolkit Settings (Bottom Left of Settings Page)
[paste settings here]

System Settings
ToolKit Version: 0.7.13
Operating System (Mac/Windows/Etc): Windows 10
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: 56.0.2924.87 (64-bit)
Other Extensions:

Looks like my recent bugfix #1069 also corrected this issue.

@dbaldon it looks like a change has been made to just not show this bar on the master category anymore -- do we still need this ticket?


cmd/compile: CMOV optimization allows GC to see invalid pointer

This program when run on the playground, go1.18, go1.19, or tip, crashes when the Go runtime finds an invalid pointer on the stack.
@cherrymui determined that this has to do with a conditional move optimization. We have code like:
if q != nil {
    p = &q.f
}

Which gets rewritten to a conditional move:
tmp := &q.f
p = Select q!=nil, tmp, p

Unfortunately, we can't compute &q.f before we've checked if q is nil, because if it is nil, &q.f is an invalid pointer (if f's offset is nonzero but small).
Normally this is not a problem because the tmp variable above immediately dies, and is thus not live across any safepoint. However, if later there is another &q.f computation, those two computations are CSEd, causing tmp to be used at both use points. That will extend tmp's lifetime, possibly across a call.

Simpler reproducer
package main

type ConfigSpec int

// Config is the Schema for the configs API.
type Config struct {
	x, y int
	Spec ConfigSpec
}

//go:noinline
func f1(cs *ConfigSpec) {
	grow(100000)
}

func grow(n int) { if n > 0 { grow(n-1) } }

//go:noinline
func f2(cs *ConfigSpec) {}

//go:noinline
func Foo(config *Config) {
	var cs *ConfigSpec
	if config != nil {
		cs = &config.Spec
	}
	f1(cs)
	if config != nil {
		cs = &config.Spec
		f1(cs)
	}
	f2(cs)

}

func main() {
	Foo(nil)
}

This fails since at least Go 1.11 (possibly even earlier).

Removing the release-blocker label as this is not a new regression.


Update New-OfficeWebAppsFarm.md

Branding updates

Docs Build status updates of commit 001c2f7:
:clock10: Full build: the average full build time is 12 min(s) 7 sec(s), this is based on the last 636 build(s) for this repository.

Docs Build status updates of commit 001c2f7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

officewebapps/officewebapps-ps/officewebapps/New-OfficeWebAppsFarm.md
:white_check_mark:Succeeded
View (officewebapps-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 56e87b8:
:clock10: Full build: the average full build time is 12 min(s) 3 sec(s), this is based on the last 654 build(s) for this repository.

Docs Build status updates of commit 56e87b8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

officewebapps/officewebapps-ps/officewebapps/New-OfficeWebAppsFarm.md
:white_check_mark:Succeeded
View (officewebapps-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Error while using ms.mixscape - ValueError: Input X contains NaN.

Report
Hi! Thank you for developing this tool it is very helpful! I am using mixscape to analyze my perturb-seq data. However when I arrive here
ms.mixscape(adata=mdata["rna"], control="sg_non_targeting", labels="GuidesGene",layer="X_pert")
I initally get this warning
s.py:429: PerformanceWarning: DataFrame is highly fragmented.  This is usually the result of calling frame.insert many times, which has poor performance.  Consider joining all columns at once using pd.concat(axis=1) instead. To get a de-fragmented frame, use newframe = frame.copy()
self.stats[group_name, "names"] = self.var_names[global_indices]
and then get this error:
ValueError: Input X contains NaN.
GaussianMixture does not accept missing values encoded as NaN natively. For supervised learning, you might want to consider sklearn.ensemble.HistGradientBoostingClassifier and Regressor which accept missing values encoded as NaNs natively. Alternatively, it is possible to preprocess the data, for instance by using an imputer transformer in a pipeline or drop samples with missing values. See https://scikit-learn.org/stable/modules/impute.html You can find a list of all estimators that handle NaN values at the following page: https://scikit-learn.org/stable/modules/impute.html#estimators-that-handle-nan-values
I checked and I do not have NA values in X. Any suggestions on how to solve this?
 Is there a way for you to share it?
Cheers

Hi Lucas, I am so SORRY but this email ended up in the spam not sure why! Can I still send you the data?
Again I am so sorry
Thank you so much for your help.
All the best,
Vittoria

From: Lukas Heumos @.>
Sent: 29 May 2024 09:23
To: theislab/pertpy @.>
Cc: Bocchi, Vittoria @.>; Author @.>
Subject: [EXTERNAL] Re: [theislab/pertpy] Error while using ms.mixscape - ValueError: Input X contains NaN. (Issue #617)
Hi, without looking at your dataset I can only make wild guesses. Is there a way for you to share it? Cheers — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread. Message
Hi,
without looking at your dataset I can only make wild guesses. Is there a way for you to share it?
Cheers
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/theislab/pertpy/issues/617*issuecomment-2136714495__;Iw!!KVWo1iE!SEkJA4_1VUBYVU2fq5jqNuxzpRkY14Ld3EhjijQ8ANe_Pk9o6wSZRIvY7JVgv6BiVbAalLy36iDfhOKlgY5hhAI$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A4WYJRRSNZ3OFKDLSZSA67DZEV7A7AVCNFSM6AAAAABINTCNUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZWG4YTINBZGU__;!!KVWo1iE!SEkJA4_1VUBYVU2fq5jqNuxzpRkY14Ld3EhjijQ8ANe_Pk9o6wSZRIvY7JVgv6BiVbAalLy36iDfhOKllHmrf1k$.
You are receiving this because you authored the thread.Message ID: @.***>
=====================================================================
Please note that this e-mail and any files transmitted from
Memorial Sloan Kettering Cancer Center may be privileged, confidential,
and protected from disclosure under applicable law. If the reader of
this message is not the intended recipient, or an employee or agent
responsible for delivering this message to the intended recipient,
you are hereby notified that any reading, dissemination, distribution,
copying, or other use of this communication or any of its attachments
is strictly prohibited. If you have received this communication in
error, please notify the sender immediately by replying to this message
and deleting this message, any attachments, and all copies and backups
from your computer.
Disclaimer ID:MSKCC

Yes of course!

Thank you so much!!! I will send via the box between today and tomorrow..  Thank you again for your help I really appreciate it!
Vittoria

From: Lukas Heumos @.>
Sent: 18 June 2024 21:10
To: scverse/pertpy @.>
Cc: Bocchi, Vittoria @.>; Author @.>
Subject: [EXTERNAL] Re: [scverse/pertpy] Error while using ms.mixscape - ValueError: Input X contains NaN. (Issue #617)
Yes of course! — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread. Message ID: <scverse/pertpy/issues/617/2176781480@ github. com>
Yes of course!
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/scverse/pertpy/issues/617*issuecomment-2176781480__;Iw!!KVWo1iE!WbZNl5kJQoDaYWNeZ0PW_vy8Qd4oxp-cyTTwF-MfNf3EUtg0CkKnv2sAWUg6p96XYPu1a4t8X6HDQZ8_xB0Z06o$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A4WYJRTRLTIG3HDJ4ADDCF3ZICA2JAVCNFSM6AAAAABINTCNUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZWG44DCNBYGA__;!!KVWo1iE!WbZNl5kJQoDaYWNeZ0PW_vy8Qd4oxp-cyTTwF-MfNf3EUtg0CkKnv2sAWUg6p96XYPu1a4t8X6HDQZ8_8pvHqnQ$.
You are receiving this because you authored the thread.Message ID: @.***>
=====================================================================
Please note that this e-mail and any files transmitted from
Memorial Sloan Kettering Cancer Center may be privileged, confidential,
and protected from disclosure under applicable law. If the reader of
this message is not the intended recipient, or an employee or agent
responsible for delivering this message to the intended recipient,
you are hereby notified that any reading, dissemination, distribution,
copying, or other use of this communication or any of its attachments
is strictly prohibited. If you have received this communication in
error, please notify the sender immediately by replying to this message
and deleting this message, any attachments, and all copies and backups
from your computer.
Disclaimer ID:MSKCC

I'm sorry for being slow here. I think you sent the data, but I can't find it anymore. How did you share it with me again, please?

Hi Lukas,
No worries and thanks again for your help!
Here is the link to the h5ad file https://mskcc.box.com/s/99oajfsh42znhb50hitzvt03wawkrcum
Sorry it took some to load it is quite big! hope this is ok!
Let me know if you have problems downloading the data.
Summary:
raw counts are found in adata.layers['raw_counts']
adata.X = adata.layers['raw_counts']
The adata.obs.Guides has the guide information with NT called "sg_non_targeting_*"
I then followed the pipeline to create the mudata
mdata = mu.MuData({'rna': adata})
I get the error when I reach
ms.mixscape(adata=mdata["rna"], control="sg_non_targeting", labels="GuidesGene")
(The column GuidesGene is just the Guides column with the different guided targeting the same gene replaced to the target gene and all controls summarized as sg_non_targeting)
Where I get this error ValueError: Input X contains NaN.
GaussianMixture does not accept missing values encoded as NaN natively. For supervised learning, you might want to consider sklearn.ensemble.HistGradientBoostingClassifier and Regressor which accept missing values encoded as NaNs natively. Alternatively, it is possible to preprocess the data, for instance by using an imputer transformer in a pipeline or drop samples with missing values. See https://scikit-learn.org/stable/modules/impute.html You can find a list of all estimators that handle NaN values at the following page: https://scikit-learn.org/stable/modules/impute.html#estimators-that-handle-nan-values
Thank you again so much for your help.
All the best, Vittoria

From: Lukas Heumos @.>
Sent: 28 July 2024 21:34
To: scverse/pertpy @.>
Cc: Bocchi, Vittoria @.>; Author @.>
Subject: [EXTERNAL] Re: [scverse/pertpy] Error while using ms.mixscape - ValueError: Input X contains NaN. (Issue #617)
I'm sorry for being slow here. I think you sent the data, but I can't find it anymore. How did you share it with me again, please? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored
I'm sorry for being slow here. I think you sent the data, but I can't find it anymore. How did you share it with me again, please?
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/scverse/pertpy/issues/617*issuecomment-2254621795__;Iw!!KVWo1iE!XCqGfsfUG83FduOhICpvK6TGPFrpDnNf62Bpx4DD9WIiPS29y7C2H5a9qIx7T36_QpOoumfFJiCjMu0-Fckws54$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A4WYJRRLNF7JDMLOFVVC2IDZOVBUHAVCNFSM6AAAAABINTCNUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJUGYZDCNZZGU__;!!KVWo1iE!XCqGfsfUG83FduOhICpvK6TGPFrpDnNf62Bpx4DD9WIiPS29y7C2H5a9qIx7T36_QpOoumfFJiCjMu0-lLNwmbE$.
You are receiving this because you authored the thread.Message ID: @.***>
=====================================================================
Please note that this e-mail and any files transmitted from
Memorial Sloan Kettering Cancer Center may be privileged, confidential,
and protected from disclosure under applicable law. If the reader of
this message is not the intended recipient, or an employee or agent
responsible for delivering this message to the intended recipient,
you are hereby notified that any reading, dissemination, distribution,
copying, or other use of this communication or any of its attachments
is strictly prohibited. If you have received this communication in
error, please notify the sender immediately by replying to this message
and deleting this message, any attachments, and all copies and backups
from your computer.
Disclaimer ID:MSKCC

Hi @VittoriaDBocchi!
I’ll look into this now. Unfortunately, the link you sent above seems to not be working anymore. Could you please double-check it? Also, if you prefer not to make the data public, feel free to share it with me directly via email<EMAIL_ADDRESS>Best,
Lilly

Dear Lilly,
Thank you for helping me! I have sent a new invitation to your email address? let me know if you receive it
Also this link should workhttps://mskcc.box.com/s/2uiu94no6vsom8lzhg4oohn0yt037lbz
Thankyou!
Vittoria
[https://cdn01.boxcdn.net/app-assets/preview/box-social.jpeg]https://mskcc.box.com/s/2uiu94no6vsom8lzhg4oohn0yt037lbz
BocchiPertpy.h5ad | Powered by Boxhttps://mskcc.box.com/s/2uiu94no6vsom8lzhg4oohn0yt037lbz
mskcc.box.com

From: Lilly May @.>
Sent: 30 July 2024 10:56
To: scverse/pertpy @.>
Cc: Bocchi, Vittoria @.>; Mention @.>
Subject: [EXTERNAL] Re: [scverse/pertpy] Error while using ms.mixscape - ValueError: Input X contains NaN. (Issue #617)
Hi @VittoriaDBocchi! I’ll look into this now. Unfortunately, the link you sent above seems to not be working anymore. Could you please double-check it? Also, if you prefer not to make the data public, feel free to share it with me directly via
Hi @VittoriaDBocchihttps://urldefense.com/v3/__https://github.com/VittoriaDBocchi__;!!KVWo1iE!V8gRbHJOLbs06fn-HtXv9rLM8tPXKJjkEH4UlLWz8slX0v4FR0VvMnwGss1UYOGgbmkQg-mWJxvRFxxpLOWKB9Y$!
I’ll look into this now. Unfortunately, the link you sent above seems to not be working anymore. Could you please double-check it? Also, if you prefer not to make the data public, feel free to share it with me directly via email<EMAIL_ADDRESS>Best,
Lilly
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/scverse/pertpy/issues/617*issuecomment-2257830526__;Iw!!KVWo1iE!V8gRbHJOLbs06fn-HtXv9rLM8tPXKJjkEH4UlLWz8slX0v4FR0VvMnwGss1UYOGgbmkQg-mWJxvRFxxpdiDnTSU$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A4WYJRTOHO6SA4B7AWYPGPDZO5INNAVCNFSM6AAAAABINTCNUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJXHAZTANJSGY__;!!KVWo1iE!V8gRbHJOLbs06fn-HtXv9rLM8tPXKJjkEH4UlLWz8slX0v4FR0VvMnwGss1UYOGgbmkQg-mWJxvRFxxpMAMvziE$.
You are receiving this because you were mentioned.Message ID: @.***>
=====================================================================
Please note that this e-mail and any files transmitted from
Memorial Sloan Kettering Cancer Center may be privileged, confidential,
and protected from disclosure under applicable law. If the reader of
this message is not the intended recipient, or an employee or agent
responsible for delivering this message to the intended recipient,
you are hereby notified that any reading, dissemination, distribution,
copying, or other use of this communication or any of its attachments
is strictly prohibited. If you have received this communication in
error, please notify the sender immediately by replying to this message
and deleting this message, any attachments, and all copies and backups
from your computer.
Disclaimer ID:MSKCC

Hi @VittoriaDBocchi!
Good news! I was able to reproduce your error and am pretty certain I’ve identified the bug causing it.
I’ve created a PR and am currently discussing it with the other developers, but I think it should be fixed by next week. In the meantime, if your analysis is urgent, you could remove the raw portion of your adata (del adata.raw), as that’s what’s causing the error.
I’ll get back to you as soon as the PR is merged. If you have any further problems, just let me know.

Hi Lilly!! I tried and it works!! Thank you so much!
I really appreciate it!
Vittoria

From: Lilly May @.>
Sent: 02 August 2024 09:33
To: scverse/pertpy @.>
Cc: Bocchi, Vittoria @.>; Mention @.>
Subject: [EXTERNAL] Re: [scverse/pertpy] Error while using ms.mixscape - ValueError: Input X contains NaN. (Issue #617)
Hi @VittoriaDBocchi! Good news! I was able to reproduce your error and am pretty certain I’ve identified the bug causing it. I’ve created a PR and am currently discussing it with the other developers, but I think it should be fixed by next week.
Hi @VittoriaDBocchihttps://urldefense.com/v3/__https://github.com/VittoriaDBocchi__;!!KVWo1iE!SK-IzJ0KTtJTnxl-0_ZoZgfagndd0ELLjmmP1v5uj9UHJXgZYyzG4xea_z4uTojWpQ749aiJI08W6CzGqG-4ehI$!
Good news! I was able to reproduce your error and am pretty certain I’ve identified the bug causing it.
I’ve created a PR and am currently discussing it with the other developers, but I think it should be fixed by next week. In the meantime, if your analysis is urgent, you could remove the raw portion of your adata (del adata.raw), as that’s what’s causing the error.
I’ll get back to you as soon as the PR is merged. If you have any further problems, just let me know.
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/scverse/pertpy/issues/617*issuecomment-2264757661__;Iw!!KVWo1iE!SK-IzJ0KTtJTnxl-0_ZoZgfagndd0ELLjmmP1v5uj9UHJXgZYyzG4xea_z4uTojWpQ749aiJI08W6CzGDxpPoNU$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A4WYJRUBKHNRVI63M4QYRXLZPMY5XAVCNFSM6AAAAABINTCNUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRUG42TONRWGE__;!!KVWo1iE!SK-IzJ0KTtJTnxl-0_ZoZgfagndd0ELLjmmP1v5uj9UHJXgZYyzG4xea_z4uTojWpQ749aiJI08W6CzGbJwiYGI$.
You are receiving this because you were mentioned.Message ID: @.***>
=====================================================================
Please note that this e-mail and any files transmitted from
Memorial Sloan Kettering Cancer Center may be privileged, confidential,
and protected from disclosure under applicable law. If the reader of
this message is not the intended recipient, or an employee or agent
responsible for delivering this message to the intended recipient,
you are hereby notified that any reading, dissemination, distribution,
copying, or other use of this communication or any of its attachments
is strictly prohibited. If you have received this communication in
error, please notify the sender immediately by replying to this message
and deleting this message, any attachments, and all copies and backups
from your computer.
Disclaimer ID:MSKCC

Fixed with the next release.


Fix pod status reporting by querying IoT Edge deployment status from IoT Hub

@veyalla @avranju Don't know if you are currently working to fix the issue or not. Will try to get it into the provider, but now promises on the timeline. :D

Thanks @neumanndaniel. No we aren't working on this issue at this time. I'd be happy to look at the PR when you get to it!


Support 1.1.0-1.10.3 or newer

Due to the 1.0.1-1.10.3 breaking changes, there needs to be some work on this quickstart to automate exposing the Kubernetes API.

#90 does that


Swan Authentication>Deeplink not showing permission again once canceled

Dev app v1.7.0 (292): "Authorize Hexa Wallet" doesn't show the permission again once canceled it. To view it again I need to go in-app and proceed with the swan account.

@cakesoft-swati either lets re test this. the button wont show the permission again if you deny it. User will have to go to the email again.

-Able to reproduce this issue on dev app v1.8.0(304)

Still having the same issue in dev app 1.8.5(326).
"Authorize Hexa Wallet" doesn't show the permission again once canceled it. To view it again I need to go in-app and proceed with the swan account.If I cancel and click on Open again, then I am getting this error.

This is happeing only in ios device.
Tested on iphone 7.

not able to reproduce

Verified this issue on dev app v 2.0.75(455)


chore: Update dependencies and fix OpenAPI client generation

Upgrade Spring Boot to 3.4.0
Update swagger-codegen-maven-plugin to 3.0.64
Add custom template for HttpBasicAuth
Fix mockwebserver version to 4.12.0
Fix parent version reference in opensearch-store

NOTE: Revert the boot to 3.3.6 before merging!!!

Changed the boot version to 3.3.6, rebased, squashed and merged at c05714881303e9b43ab710ec900cef11f3dbd9e4


update zh-translation content/zh/docs/tasks/configure-pod-container/configure-runasusername.md

Ref: k8smeetup#3114

/cc @tengqm
/cc @xichengliudui

@tengqm I fix it. thanks for your review!

/lgtm

/approve
@gorda good job!


cheatsheet as part of documentation?

I love this cheat sheet but I fear that updating this is going to be a pain. Every time you update it, the whole pdf will show up in the diff. That's not good for the repo as the history keeps getting larger. So, it takes longer and longer for people to clone ...
Also, I feel it's less accessible when it's there as a pdf than a webpage. If we can make it part of the docs, that would be great. Thoughts?

Also, if it's in sphinx you can crossref the rows to their relevant function documentation ...

Agreed, I'm just must faster in latex than sphinx. Anyone want to translate this?