Put dev secrets into chamber

Description
NOTE: This does not replace .envrc.local. It's supposed to be a no-op for those that don't have access to chamber.
Many of our newer developers don't have access to GDrive. This means they start the project and cannot get the secrets for their .envrc.local that they need. This change puts all those secrets into chamber so they can be accessed by anyone with AWS access. It also runs an eval statement to grab all those secrets so that users no longer have to fill them into an .envrc.local.
Here are the secrets added:
chamber list app-devlocal

Reviewer Notes
What is the user experience like with this?
Setup
This should remove the need for much of the .envrc.local file (but probably not all of it). So try getting rid of it as a starting point to see how this works. You'll want to bring back parts of it you want after you try things out.
mv .envrc.local .envrc.local.bkp
direnv allow
touch .envrc.local
# Now add back the things you want

Code Review Verification Steps

[x] Request review from a member of a different team.
[x] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

I've added a change that makes this work nicely if you either 1) don't have access to AWS SSM, 2) incorrectly enter your MFA token.  It now just warns you instead of trying to eval the error statement.
Here is the output of such a call coupled with missing variables:
direnv allow
direnv: loading .envrc
Enter token for arn:aws:iam::923914045601:mfa/cgilmer: 1234567
direnv: Unable to access app-devlocal variables with chamber.
direnv: Login to chamber with 'chamber list app-devlocal'.
direnv: loading .envrc.local
direnv: loading ~/Projects/transcom/ppp-infra/transcom-ppp/.envrc
direnv: LOGIN_GOV_SECRET_KEY is not set: See https://docs.google.com/document/d/148RzqgaQbhOxXd4z_xuj5Jz8JNETThrn7RVFmMqXFvk
direnv: MOVE_MIL_DOD_TLS_CERT is not set: See https://docs.google.com/document/d/1nvLXLQYz5ax3Ds4n2Y5OeANJhs0AbHtjkrKzI0gN3_o
direnv: MOVE_MIL_DOD_TLS_KEY is not set: See https://docs.google.com/document/d/1nvLXLQYz5ax3Ds4n2Y5OeANJhs0AbHtjkrKzI0gN3_o
direnv: HERE_MAPS_APP_ID is not set: See https://docs.google.com/document/d/16ZomLuR6BPEIK4enfMcqu31oiJYZWNDe9Znyf9e88dg
direnv: HERE_MAPS_APP_CODE is not set: See https://docs.google.com/document/d/16ZomLuR6BPEIK4enfMcqu31oiJYZWNDe9Znyf9e88dg
direnv: GEX_BASIC_AUTH_PASSWORD is not set: See https://docs.google.com/document/d/1nvLXLQYz5ax3Ds4n2Y5OeANJhs0AbHtjkrKzI0gN3_o
direnv: DPS_AUTH_SECRET_KEY is not set: https://docs.google.com/document/d/1HAD9tu9WahzVEam5FFWrgywdMm4aTfVW-Mp3rL7idAo
direnv: DPS_AUTH_COOKIE_SECRET_KEY is not set: https://docs.google.com/document/d/1HAD9tu9WahzVEam5FFWrgywdMm4aTfVW-Mp3rL7idAo
direnv: CSRF_AUTH_KEY is not set: See https://docs.google.com/document/d/1DuWXZLFaW7FXvqh-PStqjZI40niEavXWS5PPtWPlK3w
direnv: EIA_KEY is not set: https://docs.google.com/document/d/1K1-xlYcZaS518PQiaB39gSvqz2tTo0W8eM0wImB7TcI
direnv: Your environment is missing some variables!
direnv: Set the above variables in .envrc.local and try again.
direnv: export +AWS_PROFILE +AWS_S3_BUCKET_NAME +AWS_S3_KEY_NAMESPACE +AWS_S3_REGION +AWS_SDK_LOAD_CONFIG +AWS_SES_DOMAIN +AWS_SES_REGION +AWS_VAULT_KEYCHAIN_NAME +CHAMBER_KMS_KEY_ALIAS +CHAMBER_RETRIES +CHAMBER_USE_PATHS +CLIENT_AUTH_SECRET_KEY +DANGEROUSLY_DISABLE_HOST_CHECK +DB_HOST +DB_NAME +DB_NAME_DEV +DB_NAME_PROD_MIGRATIONS +DB_NAME_TEST +DB_PASSWORD +DB_PORT +DB_PORT_PROD_MIGRATIONS +DB_PORT_TEST +DB_SSL_MODE +DB_USER +DEVLOCAL_CA +DOD_CA_PACKAGE +DPS_COOKIE_EXPIRES_IN_MINUTES +DPS_COOKIE_NAME +DPS_REDIRECT_URL +EIA_URL +GEX_BASIC_AUTH_USERNAME +GEX_URL +GO111MODULE +HERE_MAPS_GEOCODE_ENDPOINT +HERE_MAPS_ROUTING_ENDPOINT +HONEYCOMB_API_HOST +HONEYCOMB_API_KEY +HONEYCOMB_DATASET +HONEYCOMB_DEBUG +HONEYCOMB_ENABLED +HTTP_SDDC_PORT +HTTP_SDDC_PROTOCOL +IWS_RBS_HOST +LOGIN_GOV_CALLBACK_PORT +LOGIN_GOV_CALLBACK_PROTOCOL +LOGIN_GOV_HOSTNAME +LOGIN_GOV_MY_CLIENT_ID +LOGIN_GOV_OFFICE_CLIENT_ID +LOGIN_GOV_TSP_CLIENT_ID +MOVE_MIL_DOD_CA_CERT +MYMOVE_DIR +NO_SESSION_TIMEOUT +PPP_INFRA_PATH +SECURE_MIGRATION_DIR +SERVE_SWAGGER_UI +TZ ~PATH

direnv allow won't work unless I switch from fish to bash. Since several of us use fish, we need to figure out what the issue is. Maybe @jim can help?

direnv allow won't work unless I switch from fish to bash. Since several of us use fish, we need to figure out what the issue is. Maybe @jim can help?

Does it just not work at all? Because if it acts as a no-op in the fish shell then that's fine and we can work out alternatives later. But if it stops direnv allow from working at all then I need a new solution. It's supposed to fail-over as a no-op, not disable direnv from working.
Personally I'd also like to know how direnv works a little more because our .envrc is bash script itself and even points to /usr/bin/env bash at the top. So I'd expect it to do whatever it does in bash and then for direnv to act appropriately with that information in your desired shell.  I find it odd that the output of that file gets swallowed by fish, but honestly I don't know that shell at all. I'll probably dig into this later.
I also want to re-iterate that this doesn't replace .envrc.local, it's just supposed to make life easier for people and otherwise be a no-op where the fallback plan is to use the GDrive links and our old copy-paste method.
The alternative is that I can update the docs and just tell people to put this line of code in their .envrc.local .
In the meantime I'm going to give you and @akostibas access to our chamber secrets and I'll be curious if that changes the behavior at all.  You can try chamber list app-devlocal && direnv allow to see if that fixes it later today.

@tinyels @jim - can you try this branch with the command direnv export fish | source instead of direnv allow?  It looks to me that direnv allow output is swallowed by this function which is why you can't see anything:
https://github.com/direnv/direnv/blob/master/shell_fish.go#L12-L16
So the solution here would be making a script named scripts/direnv-fish with the contents direnv export fish | source.
Btw, I did download the fish shell and this worked for me so I just want to confirm.

@tinyels - this is now an opt-in feature that you enable by copying the template over.

I am now able to run without using chamber and with using chamber. (Though will need to remember to call direnv export fish | source instead of direnv allow)


H6089 needs to be added to no scale brightness

What issue do you have? Please be as thorough and explicit as possible.
The brightness gets set to 39% in HomeKit because it is being divided by 2.54.
...
Details of your setup.
Do you use Homebridge or HOOBS?
Homebridge
...
Which version of Homebridge/HOOBS do you have?
1.1.6
...
Which version of this plugin (homebridge-govee) do you have?
2.0.1
...
Please paste any relevant logs below. It helps if you can turn debug and debugReqRes in the package settings for more thorough logging.

If you are posting an error then it is helpful for me to also see the previous few lines as this can show the cause of the error.
[Main Light] could not be updated and its status will be reverted so on. Error: Request failed with status code 400.
...

All done. Please update to v2.0.2 and this should be fixed.
Thanks for alerting me of this!


lockfile is tied to python version

Describe the bug
my project is supposed to work on python versions 3.8 to 3.12, however it doesn't seem to be possible to generate a lockfile that works on multiple python versions:
ERROR: Ignored the following versions that require a different python version: 0.1.0 Requires-Python >= 3.10; 0.2.0 Requires-Python >= 3.10; 0.3.0 Requires-Python >= 3.10; 0.4.0 Requires-Python >= 3.10; 0.5.0 Requires-Python >= 3.10; 0.6.0 Requires-Python >= 3.10; 0.6.1 Requires-Python >= 3.10; 0.7.0 Requires-Python >= 3.10; 0.8.0 Requires-Python >= 3.10
ERROR: Could not find a version that satisfies the requirement truststore==0.8.0 (from versions: none)
ERROR: No matching distribution found for truststore==0.8.0

Notice:  A new release of pip is available: 23.3.1 -> 24.0
Notice:  To update, run: python -m pip install --upgrade pip
PYPROJECTX ERROR: installation of 'pdm' failed with exit code 1

To Reproduce

put pdm in your pyprojectx dependencies:# pyproject.toml
[tool.pyprojectx]
pdm = "pdm>=2.12.3"

run ./pw --lock using python 3.12
run ./pw pdm --version using python 3.8

Expected behavior
no error: generated lockfile should contain dependency information for all supported python versions
Desktop (please complete the following information):

Python  version: 3.8, 3.12
OS: windows, linux, macos

Additional context
full output: https://github.com/DetachHead/pytest-robotframework/actions/runs/7999782115/job/21848170675?pr=217

I added a clarification in the docs:

Supporting multiple python versions:
Make sure to run px --lock using the minimum supported python version for your project. This prevents the lock file from containing dependencies that are incompatible with older python versions.

Although not perfect, locking does avoid broken toolchains.
This could be improved by using a better dependency locking mechanism than just plain pip freeze.
I considered using Pdm's or Poetry's dependency locking libs, but that's not planned in the near future.

Is this resolved in 3.0?

Indeed resolved in 3.0 by using uv


update crypto libs to RustCrypto for M1 compatibility

The tests are passing. I'm going to work on some additional improvements tomorrow This should be ready for review :)

Looks great! Thanks a lot! :)


upload ARM packages

Upload ARM apckages.
Hold until the "upload" operation supports multiple files uploading.

/hold

/assign @hopkiw @Quintonamore


FYI

Go 1.9 ships with os/user.LookupId, os/user.Lookup, os/user.LookupGroup, os/user.LookupGroupId that work without cgo. If you compile without cgo enabled, those functions will read /etc/passwd and /etc/group respectively to find answers to the questions you ask. Which may be good enough for your use case.
(I wrote the functions in question, and I'm curious how well they work for folks. I would appreciate feedback on how well they work or don't. If they don't work, I'd love to help fix them. Thanks!)

@kevinburke: for some reason I got user: Current not implemented on linux/amd64 (compiled using go version go1.11.1 linux/amd64). I'm running inside a container, if that makes a difference.
Here's my code:
usr, err := user.Current()
if err != nil {
	panic(err)
}

fmt.Printf("homedir = %s\n", usr.HomeDir)

This /etc/passwd parsing is total overkill, since all I want is a cross-platform way to resolve user's homedir via the ENV variables. That's why I'm having to resort into using this external dependency.

@joonas-fi Are you using FROM scratch perhaps? Do you got the Dockerfile of the image available somewhere to see?

here's the logic in go 1.11 that leads to an error
	u := &User{
		Uid:      currentUID(),
		Gid:      currentGID(),
		Username: os.Getenv("USER"),
		Name:     "", // ignored
		HomeDir:  os.Getenv("HOME"),
	}
	// On NaCL and Android, return a dummy user instead of failing.
	switch runtime.GOOS {
	case "nacl":
		if u.Uid == "" {
			u.Uid = "1"
		}
		if u.Username == "" {
			u.Username = "nacl"
		}
		if u.HomeDir == "" {
			u.HomeDir = "/"
		}
	case "android":
		if u.Uid == "" {
			u.Uid = "1"
		}
		if u.Username == "" {
			u.Username = "android"
		}
		if u.HomeDir == "" {
			u.HomeDir = "/sdcard"
		}
	}
	// cgo isn't available, but if we found the minimum information
	// without it, use it:
	if u.Uid != "" && u.Username != "" && u.HomeDir != "" {
		return u, nil
	}
	return u, fmt.Errorf("user: Current not implemented on %s/%s", runtime.GOOS, runtime.GOARCH)

note there is a patch to this logic incoming in go1.12, the patch is here: https://github.com/golang/go/commit/3a18f0ecb57
logic now looks like:
	uid := currentUID()
	// $USER and /etc/passwd may disagree; prefer the latter if we can get it.
	// See issue 27524 for more information.
	u, err := lookupUserId(uid)
	if err == nil {
		return u, nil
	}
	u = &User{
		Uid:      uid,
		Gid:      currentGID(),
		Username: os.Getenv("USER"),
		Name:     "", // ignored
		HomeDir:  os.Getenv("HOME"),
	}
	// On NaCL and Android, return a dummy user instead of failing.
	switch runtime.GOOS {
	case "nacl":
		if u.Uid == "" {
			u.Uid = "1"
		}
		if u.Username == "" {
			u.Username = "nacl"
		}
		if u.HomeDir == "" {
			u.HomeDir = "/"
		}
	case "android":
		if u.Uid == "" {
			u.Uid = "1"
		}
		if u.Username == "" {
			u.Username = "android"
		}
		if u.HomeDir == "" {
			u.HomeDir = "/sdcard"
		}
	}
	// cgo isn't available, but if we found the minimum information
	// without it, use it:
	if u.Uid != "" && u.Username != "" && u.HomeDir != "" {
		return u, nil
	}
	return u, fmt.Errorf("user: Current not implemented on %s/%s", runtime.GOOS, runtime.GOARCH)

@Ilyes512 unfortunately my image is not public, but my image's first public parent is FROM golang:1.11.1
@kevinburke thanks for responding so quickly, and with the code. The issue seems to be here:
$ whoami
root
$ env | grep USER
$ env | grep HOME
HOME=/root

The code you shared seems to require that USER env is defined. I am not sure if this is common behaviour inside Docker containers.
The upcoming patch doesn't seem to mitigate this exact case.

Go 1.12 supports os.UserHomeDir().

I noted this somewhere else but can't find it... but the Go stdlib implementation is quite simplistic. It only checks env vars. This will work in most cases (90+%?) so if it works for you that's great and you can remove a dep!
In the nearly decade (!!!) we've had some of our software like Vagrant, there has been enough situations where that is not enough so we have extra logic here in go-homedir to try other ways to detect the home directory. We haven't had a bug reported in home directory detection in a long time...
So if you want something more robust then keep using this lib, otherwise you can use the stdlib!

Correct me if I'm mistaken, but having an HOME env var is required for POSIX compliance. If it is unset for whatever reason, that is an issue which needs to be fixed.
@mitchellh can you point to a specific case where the HOME env var is missing?

No, its non-standard. Just the realities of delivering software to many people... you hit some truly “wtf” environments.

The bit I hit just now, @mitchellh, is the difference between %HOME% and %USERPROFILE% on windows. AWS appears to specifically use %USERPROFILE% which is fine in most cases, but if %HOME% is defined as well, go-homedir prefers it.

Hey @jhulten. Its been long enough that I don't have full context here, but if there is a well established practice that USERPROFILE is preferred, then we should/can switch to that as a preference for Windosws.

Probably would be a breaking change for a lot of people, but the ability to provide lookup order instead of it being hardcoded would be a plus.


Using JSX inside ui.component breaks compilation

This seems like a regression, and I'm having trouble pinning it down but was able to reproduce in the starter https://github.com/tinacms/tina-cloud-starter/pull/310, though I couldn't reproduce in the monorepo.
Any ui.component prop with a function fails the build with a syntax error:
{
  name: 'title',
  type: 'string',
  ui: {
    component: () =>
Hello

  }
}

This appears to be due to the presence of jsx: "preserve" in the user's tsconfig.json. We use esbuild under the hood which will pick that config up as of 0.14.51 (see related issue), that would explain the difficulty reproducing this in the monorepo since it's pinned to an earlier version.
For now, I think the best course of action is to instruct users who run into this to change their tsconfig.json to not have this property or to specify "react" instead. I expect that the users who've run into this only had "preserve" in there because for some reason our starter has that config (I've updated that). This feels like a pretty niche scenario, but if we feel like it's worthwhile we can try to grab the user's tsconfig.json and analyze it ourselves to make sure this isn't configured. Unfortunately this argument seems to be overridden by the tsconfig setting.
The remaining question is whether or not we should respect the user's tsconfig.json. I think ideally we do respect it, but if it causes other problems perhaps we should ignore.
Thoughts @logan-anderson ?


Demo doesn't work in IE9

Hi,
I have noticed that your library's demo doesnt work in IE9.
I'm testing this using the IE11 dev tools to emulate IE9.
The page displays: "If you see this then something is wrong."
and the console says: SCRIPT5007: Object expected sample.js (18774,5)
Could it have something to do with MatchMedia not being supported in IE9?
And if I want to use your library in IE9, what polyfills do you recommend I use?
Thanks.

https://github.com/contra/react-responsive#browser-support


Move common dependency versions into properties

Common dependency versions, such as those used by the Android support library dependencies, should be moved into the project.ext for reuse.
Includes:

Android support
Dagger 2
AutoValue
AutoValue Parcelable
Firebase
Stetho

This is relevant: https://segunfamisa.com/posts/android-gradle-extra-properties


Can output_kafka consume event from input_zeromq

I have one server has input_zeromq and output_elasticsearch, and it works well.
However I just added output_kafka, but it seems not work since I can't find any event from Kafka client.
So I'm wondering if output_kafka can consume event from input_zeromq
Version: logstash 2.3.2
Operating System: Ubuntu 14.04
Config File (if you have sensitive info, please remove it): I have 1 input config file and 2 output config files as below:
input_zeromq:
input {
zeromq {
topology => 'pushpull'
type => 'pull-input'
}
}
output_kafka:
output {
kafka {
topic_id => 'logstash_logs'
bootstrap_servers => 'aaaa:9092'
}
}
output_elasticsearch:
output {
elasticsearch {
hosts => ['bbbb:9200']
}
}

There should not be any specific reason that the kafka output would deal with input_zeromq messages any differently. are you sure you can send to Kafka with this configuration outside of logstash? or with a different input?

thanks. I'll check my Kafka settings.


Drop Python 3.4 support

I would like to consider dropping official support for Python 3.4 after the next release.  It is easy to update from 3.4 to 3.5 and we haven't seen any recent users using 3.4 (which seems generally unpopular).

In general I'm for it. Note that both scikit-image and x-array still support 3.4, and are downstream projects. I'd be fine dropping testing for 3.4 though, as I think it's unlikely that we'll have any major version issues, and I don't think it gets much use (as you said).

cc @shoyer @stefanv

I would do a query for dask and Python 3.4 a la:
http://www.randalolson.com/2016/09/03/python-2-7-still-reigns-supreme-in-pip-installs/

https://gist.github.com/mrocklin/341c9b93c46dbc147f755ddf22394ac7
I'm also trying to get access to conda downloads.  I suspect that these are
more 3.x focused
On Fri, Aug 25, 2017 at 1:32 PM, Stefan van der Walt <
<EMAIL_ADDRESS>wrote:

I would do a query for dask and Python 3.4 a la:
http://www.randalolson.com/2016/09/03/python-2-7-still-
reigns-supreme-in-pip-installs/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/2626#issuecomment-324987238, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszPxQ45xFza4VS6cU17GGhV09tYtAks5sbwU9gaJpZM4PC7qA
.

I'm currently looking at using Dask in a 3.4 project due to a PySide dependency. Just as a reason for why upgrading might not be a option for some.

The cluster of computers I am working on are mainly python 3.4.2 (old Debians) but we are implementing docker containers to avoid multiple versions in different machines. So, dropping support 3.4 won't affect us much.

In #2697 I've turned off testing for 3.4 on PRs (still run when pushing/merging to master). For now I think this will ensure 3.4 still works without slowing down our tests too much.


1.7.2 & 1.7.4 crash

InvalidInjectionException: Critical injection failure: @Inject annotation on outputTitleState could not find any targets matching Lnet/minecraft/class_624;method_1033()V in TitleScreen. Using refmap state-output-refmap.json [PREINJECT Applicator Phase -> TitleScreenMixin from mod state-output -> Prepare Injections ->  -> handler$zdb000$state-output$outputTitleState(Lorg/spongepowered/asm/mixin/injection/callback/CallbackInfo;)V -> Parse]
works on 1.6.4 & 1.7.10

Screen#tick intermediary mismatch :/

should just fix this in a 1.1.7 patch release for 1.6-1.7.10


HI! The PDF does not display the contents

demo:

this file :
003211-240320 monarch诊断推文.pdf
pdfjs version:  4.1.376
pdf version: 1.6 Adobe Acrobat 20.13 Image Conversion Plug-in / Adobe Acrobat 20.13

I can reproduce the issue on Arch Linux with Firefox 124.0.2. What could be related here, and certainly doesn't help the rendering time of the document, is that it contains a large 4688 x 37363 image XObject; please see https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#what-types-of-pdf-files-are-slow-in-pdfjs-can-i-optimize-a-pdf-file-to-make-pdfjs-faster for rendering time optimization tips.
I don't think at the moment that it's related to the image itself rendering partially though.


command palette focus isn't respected when standing on PR-page

Extension version: 0.27.1
VSCode Version: 1.57.1
OS: Mac OS

Version: 1.57.1
Commit: 507ce72a4466fbb27b715c3722558bb15afa9f48
Date: 2021-06-17T13:28:32.912Z (1 wk ago)
Electron: 12.0.7
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0
Steps to Reproduce:

Be on PR page, focus commentbox.
Cmd + p, type something, then cmd + v and it will paste in commentbox instead of in command palette.

@mjbvz do you know if this is something that needs to be solved at webview level?

@mjbvz do you know anything about this?

What more info do you need? @alexr00

@alexr00 I think webviews may be overriding the copy/paste keyboard when they are active (not just focused): https://github.com/microsoft/vscode/blob/08ef40bd522d5a1309d7b54fb5ba9066d4cf5983/src/vs/workbench/contrib/webview/browser/webview.contribution.ts#L20
We already have a workaround in that code for context menus but not for quick picks


serialisers: data cite should read all affiliation in one go

Affiliations are being read one by one:
    def _read_affiliation(self, id_):
        """Retrieve affiliation record using service."""
        affiliations_service = (
            current_service_registry.get("rdm-affiliations")
        )
        return affiliations_service.read(id_, system_identity)

Instead, we should either

Dereference all the needed information into the record JSON so it's available,
or
In the schema pre_dump collect all affiliation ID's and make a single read_many query to get all affiliations in one go.

This was part of https://github.com/inveniosoftware/invenio-rdm-records/pull/704, which didn't get done yet. There was a card in the ToDo (I'm deleting it in favour of this issue.


Add support for jest.clearAllMocks and jest.resetAllMocks

This PR aims to add support for jest.clearAllMocks and jest.resetAllMocks.
Fixes #18

[x] Show proper error message instead of the existing error log TypeError: Cannot read properties of undefined (reading 'length'). Now shows TypeError: jest.clearAllMocks is not a function. Fixed in 14336c3d42e23705d77778e7a198ba12121a8905
[x] Support jest.clearAllMocks
[x] Support jest.resetAllMocks
[x] Testing

Need a small help with last 2 tasks
Thanks for hinting me where to change. I tried this, and I'm facing a small issue.
I could have done like this in the global.setup.js:
// excerpt
globalThis.jest = {
  fn: mock.fn,
  spyOn: mock.spyOn,
  clearAllMocks: mock.JestMock.clearAllMocks.bind(mock.JestMock),
  // ...
};

But the jest-mock package seem not exporting the JestMock instance. So I can't do mock.JestMock.<thing>.
https://github.com/facebook/jest/blob/d2f3dc694d72c0208f47fc2e2ca101e26a9a6d68/packages/jest-mock/src/index.ts#L1239-L1243
Could someone help?

We could instantiate our own ModuleMocker and use it to grab also fn and spyOn.

@vajahath This looks good; let me know when you have verified that it works in your app!

Yes I just verified with my project. :smile_cat:

Thanks!


Support for constructor optional dependency injection

Hello, after digging into some TP code and docs I couldn't find a way to achieve that but pardon me if it's possible.
I want to do optional injection in the constructor with TP, just like Spring does here using the Optional class: https://dzone.com/articles/optional-dependency-injection-with-spring
Is it possible, does it make sense for having that in Toothpick, can this be achieved with Kotlin Nullables and/or Java's Optional in a future release?
Currently I'm using property injection for this scenario, works as expected: if TP can provide me an object in the scope it will fill the property, otherwise will leave it as null. (using @JvmField + @Inject in Kotlin code).

To be honest, I am surprised TP can leave a field as null. TP should never
return a null value.
I don't think we are gonna add this feature: TP's motto is to remain
simple. And in this case, I would advise to create a new entity where the
dependency is not optional, or to use TP.getInstance in code where needed.
Le mer. 8 mai 2019 à 15:07, Matheus Villela<EMAIL_ADDRESS>a
écrit :

Hello, after digging into some TP code and docs I couldn't find a way to
achieve that but pardon me if it's possible.
I want to do optional injection in the constructor with TP, just like
Spring does here using the Optional class:
https://dzone.com/articles/optional-dependency-injection-with-spring
Is it possible, does it make sense for having that in Toothpick, can this
be achieved with Kotlin Nullables and/or Java's Optional in a future
release?
Currently I'm using property injection for this scenario, works as
expected: if TP can provide me an object in the scope it will fill the
property, otherwise will leave it as null. (using @JvmField + @Inject
https://github.com/Inject in Kotlin code).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stephanenicolas/toothpick/issues/331, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAN7PXKLCSEYYBKBJZSDU2LPUNFLVANCNFSM4HLVTD5A
.

@villela could you provide an example of how you use @JvmField + @Inject to achieve that for field injection?
Regarding the request itself, could you give us some use cases where you might need optional dependencies?

@stephanenicolas @dlemures I'm sure I could achieve that behaviour with property injection but I can't reproduce it again, maybe it was just some scenario I've got with kapt, having the property but not the member injector generated, or maybe I just made a mistake. Doing the steps again TP always raises an exception if it can't retrieve an instance, just as you expected.
It's fine if you don't want to include it, I really like the minimalist approach from TP. I'll probably solve it by wrapping it over a retriever class that knows the correct scope.
Regarding the use case: it's a scenario were the object is responsible for providing a dependency for its "reincarnation", in the case it's just simple parcelables that represents state.  Some pseudo code to give an idea:
class Reincarnable(val args: MyInitializationArgs, val state : Optional<MyPersistableState>) {   init {     stateToSaveOnKilledAndWillReincarnate {        return createObjectOfMyPersistableStateTypeToInjectOnMyReincarnation()     }   } }
I used "val state : Optional" in the example instead MyPersistableState? just to not add another complicated issue on the table.
Think about an activity that shows paged lists of foods, Reincarnable is my Presenter or ViewModel, my args may be the id of a kind of food like fruits. Them I'm on page 7, retrieved from a web api and open the details of apple in another activity. My device has low memory and the activity with the pages is destroyed, ofc I won't save the items of the list in the bundle but at least the state that I was on page 7 I want saved so I can reload the page 7 from web when the activity and its Presenter/ViewModel reincarnates. This example can happen even on single activity apps without fragments (my scenario) when we launch intents to open stuff like camera or even when the app is killer on background.
In the pseudo code example when the system decides to destroy the activity the navigation stack with its arguments and states are saved in the bundle for future recreation.
Not sure if I'm clear enough here, I'm doing a Navigator implementation for a fragmentless single activity app  very coupled with TP so it may like a bit strange.

So you are injecting a configuration that can be there or not. On my side, I'd wrap the logic inside a configuration provider and inject that one instead, but that's just a design choice.
Let us discuss about it. On the one hand, TP is used to inject dependencies and here is used as an parameter provider, but on the other, there might be other situations when it makes sense and we want to provide that flexibility if needed.

@dlemures @stephanenicolas just to give a bit more context on this, what I was trying to do with the optional injection was to achieve the same thing Google provided with the recent SavedStateHandle.
Of course a container, or a bundle like they did, solves the issue. But I really feel like the optional injection a more elegant solution that also has other usages.


822 - Syntactic Sugar for Importing files if exist

#822
Changed
<Import Project="$(MSBuildExtensionsPath)\Foo\Bar.targets"
    Condition="Exists('$(MSBuildExtensionsPath)\Foo\Bar.targets')"
/>

to
<ImportIfExists Project="$(MSBuildExtensionsPath)\Foo\Bar.targets" />`

with
@stuleelight
Stuart Lightfoot
This is our first time opening a pull request in this project!

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

Thanks for the contribution! However, I don't think we'll be able to accept it at this time.
The ImportIfExists element is not currently supported. #822 proposes adding support for it, at which time a PR like this could be made to adopt it.
If you're interested in adding support for the element, we should talk through its design and implications in #822 before starting to code it up. Historically, we've been hesitant to add new elements, since using a new element means you can no longer load a project in an older Visual Studio or MSBuild. That's not necessarily reason to not do it, though--we have to start somewhere! But we should talk about the timing.

Rainer,
I super appreciate the reply. If I am being honest, this was part of a Hackathon, and I believe our prompt was a touch off. It seemed too easy and my partner and I asked several times about it.
However, I’m decent with C# most through game development, but I’m obviously a touch green still. Where or how would you start this?
Am I understanding correctly we are looking to create a type of function that checks ifExists first then proceed with the import?
Any help is appreciated.
Sincerely,
Stuart

On Nov 4, 2019, at 10:36 AM, Rainer Sigwald<EMAIL_ADDRESS>wrote:

Thanks for the contribution! However, I don't think we'll be able to accept it at this time.
The ImportIfExists element is not currently supported. #822 proposes adding support for it, at which time a PR like this could be made to adopt it.
If you're interested in adding support for the element, we should talk through its design and implications in #822 before starting to code it up. Historically, we've been hesitant to add new elements, since using a new element means you can no longer load a project in an older Visual Studio or MSBuild. That's not necessarily reason to not do it, though--we have to start somewhere! But we should talk about the timing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Update VM create to query using the resources API instead of network

Because I'm sick of re-recording all of VM when Network bumps its API version...

It turns out that the resources API is not sufficient for the networking calls the VM module needs. Instead, I will allow pinning to a supported API version when using profile "latest"


refactor!: make the entry package @ckb-lumos/lumos more convenient

Description
Resolves #585
Go to the migration for more details
Type of change

[x] Refactor (non-breaking change)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation or Website
[x] Example

How Has This Been Tested?

[x] Use Verdaccio to publish the test pkgs

@zhangyouxin can you help to check this PR?


hcs: Fail operations when the HCS service crashes

If the HCS service crashes, then all processes and compute systems get
notified of this. The code was previously ignoring this notification,
meaning outstanding waits would never be satisfied. Now the notification
causes all pending waits to complete with an error.

This is a funny change. I both fixed, over complicated, and regressed this code all at once. Thanks for fixing.


Updated NPM env varibale

This env varibale is the correct way to publish packages via CI

:robot:
Changes in .github/workflows/release.yml:

Line 103: The environment variable NPM_TOKEN has been updated to NPM_AUTH_TOKEN.

Optimization suggestion:

No optimization is required as the change is simply updating the name of the environment variable.

Potential problems:

There are no potential problems identified in the modified lines.

> gpt-3.5-turbo-16k


[Bug] Plugin not working in RedHat 7.9 (Plugin unavailable)

Describe the bug
Adding the infinity datasource result saving error message.
Screenshots

** Details**
Plugin can be used from panels but
A) default JSON URL query fails https://jsonplaceholder.typicode.com/users
{"status":500,"statusText":"Internal Server Error","data":{"message":"Metric request error"},"config":{"url":"api/ds/query","method":"POST","data":{"queries":[{"refId":"A","datasource":{"uid":"Hq68vHLnk","type":"yesoreyeram-infinity-datasource"},"type":"json","source":"url","format":"table","url":"https://jsonplaceholder.typicode.com/users","url_options":{"method":"GET","data":""},"root_selector":"","columns":[],"filters":[],"global_query_id":"","datasourceId":34,"intervalMs":5000,"maxDataPoints":2981}],"range":{"from":"2022-03-02T08:47:44.290Z","to":"2022-03-02T14:47:44.290Z","raw":{"from":"now-6h","to":"now"}},"from":"1646210864290","to":"1646232464290"},"requestId":"Q174","retry":0,"headers":{"X-Grafana-Org-Id":1},"hideFromInspector":false},"message":"Metric request error"}

JSON inlile query fails
{
"name": "job_teamname_app:system_cpu_usage:count",
"dnsname": "xxx-monitor-v1",
"job": "prod"
}

{"status":500,"statusText":"Internal Server Error","data":{"message":"Metric request error"},"config":{"url":"api/ds/query","method":"POST","data":{"queries":[{"refId":"A","datasource":{"uid":"Hq68vHLnk","type":"yesoreyeram-infinity-datasource"},"type":"json","source":"inline","format":"table","url":"https://jsonplaceholder.typicode.com/users","url_options":{"method":"GET","data":""},"root_selector":"","columns":[],"filters":[],"global_query_id":"","data":"{\n"name": "job_teamname_app:system_cpu_usage:count",\n"dnsname": "xxx-monitor-v1",\n"job": "prod"\n}","datasourceId":34,"intervalMs":5000,"maxDataPoints":2981}],"range":{"from":"2022-03-02T08:49:28.779Z","to":"2022-03-02T14:49:28.779Z","raw":{"from":"now-6h","to":"now"}},"from":"1646210968779","to":"1646232568779"},"requestId":"Q180","retry":0,"headers":{"X-Grafana-Org-Id":1},"hideFromInspector":false},"message":"Metric request error"}
Version Details:

Grafana version : 8.3.4
Plugin version : 0.8.3

Additional context
In Windows10 Grafana 8.4.2 and Plugin 0.8.3 works

Does the previous version of the plugin used to work in RH for you?

I have not tried that. Which version I should try ?
I have also Grafana 7.5.7 in one another Redhat machine --> can try if 0.8.3 works in that.

Still fails in Grafana 8.4.2 or when plugin downgraded to (0.8.0, lower does not have Node Graph).
Zabbix had similar error when plugin was not compiled to bsd. But Redhad should work with linux_amd64.
https://github.com/alexanderzobnin/grafana-zabbix/issues/1054

lvl=eror msg="Could not start plugin" pluginId=yesoreyeram-infinity-datasource err="fork/exec /var/lib/grafana/plugins/yesoreyeram-infini
ty-datasource/gpx_infinity_linux_amd64: permission denied"
Giving more permissions solved the issue


vercel dev still results to release build of rust

Seems the debug build only be enabled if VERCEL_BUILDER_DEBUG was set. But I think vercel cli won't set this. We need other way to know which mode we are running.
https://github.com/vercel-community/rust/blob/954bdba290266de5831f7606122909667fc64249/src/index.ts#L36
https://github.com/vercel-community/rust/blob/954bdba290266de5831f7606122909667fc64249/src/index.ts#L71-L81
Set this env manully won't be good idea as it produces lot of unnecessary log.

Shipped in 4.0.8

Please keep this open file a while, as I think the cache mechanism for the dev may be also corrupted. I will dive into it.


Update availability-modes-always-on-availability-groups.md

Fix typo

Docs Build status updates of commit a46bf1f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/database-engine/availability-groups/windows/availability-modes-always-on-availability-groups.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Great catch! Thanks @0x7FFFFFFFFFFFFFFF :) #sign-off


powertoysrun: opening a file or folder should also work if the string is encapsulated in double quotes

Microsoft PowerToys version
0.70.0
Installation method
GitHub
Running as admin
No
Area(s) with issue?
PowerToys Run
Steps to reproduce
have explorer open. shift-rightclick on some file and "copy as path".
then hit the hotkey to open Powertoysrun.
paste. hit enter.
✔ Expected Behavior
the file opens in the associated progam
❌ Actual Behavior
nothing happens. in earlier versions I also saw a websearch being opened for the string.
if I remove the double quotes (that the "copy as path" function adds by itself, they were not added by me!), opening the file via Powertoysrun does work, so it really is the double quotes that are causing this. Given that enclosing a string that potentially contains whitespaces in double quotes is such a normal thing, Powertoysrun should be able to handle that.
Other Software
No response

This appears to be fixed! Going to close :)


Moving hostdiscovery under build tag

In order to vendor-in libnetwork to docker, we need to remove the swarm
dependency even though it is used as library. using this PR, a new build
flag libnetwork_discovery is introduced in order to avoid pulling in the
unused hostdiscovery functionality into docker.
We are working with the Swarm project to see if we can modularize the
discovery package to become independent so that we can include them as a
vendor-in package in docker.

LGTM

LGTM


CPU information for zOS

Need to figure a command that can be executed from Groovy

https://www-03.ibm.com/systems/z/os/zos/features/unix/bpxa1ty2.html
oeconsol "D M=CPU" | grep ".. + " | wc -l
To get memory...
oeconsol "D M=STOR"

oeconsol isn't currently on the zOS machines but it looks like we could add it quite easily


circle.yml: Fix ui upload step so it doesn't build twice

There are two source checkouts in circle, and we were building in the wrong one
(ie. not the one where the earlier build happened). This was wasteful at best
and buggy at worst due to build differences.
Fixes https://github.com/weaveworks/scope/issues/1918

LGTM after circle passes!


[Fix] Tracking initialization

[Fix] Tracking initialization

Se agrega como default la Batch Strategy para trackear
Se agrega un constructor que obliga inicializar con public key y contexto.

Codecov Report

Merging #634 into develop will decrease coverage by 0.01%.
The diff coverage is 34.48%.

@@             Coverage Diff             @@
##           develop     #634      +/-   ##
===========================================
- Coverage    27.45%   27.43%   -0.02%
===========================================
  Files          231      231
  Lines        16964    16927      -37
  Branches      1477     1474       -3
===========================================
- Hits          4657     4644      -13
+ Misses       11807    11786      -21
+ Partials       500      497       -3

Impacted Files
Coverage Δ

...c/main/java/com/mercadopago/RejectionActivity.java
0% <0%> (ø)
:arrow_up:

...rc/main/java/com/mercadopago/CongratsActivity.java
0% <0%> (ø)
:arrow_up:

...src/main/java/com/mercadopago/PendingActivity.java
0% <0%> (ø)
:arrow_up:

...rc/main/java/com/mercadopago/CheckoutActivity.java
0% <0%> (ø)
:arrow_up:

...c/main/java/com/mercadopago/BankDealsActivity.java
0% <0%> (ø)
:arrow_up:

...ain/java/com/mercadopago/PaymentTypesActivity.java
0% <0%> (ø)
:arrow_up:

...ain/java/com/mercadopago/InstructionsActivity.java
0% <0%> (ø)
:arrow_up:

...java/com/mercadopago/CallForAuthorizeActivity.java
0% <0%> (ø)
:arrow_up:

...java/com/mercadopago/ReviewAndConfirmActivity.java
0% <0%> (ø)
:arrow_up:

...ain/java/com/mercadopago/SecurityCodeActivity.java
49.13% <100%> (-0.44%)
:arrow_down:

... and 7 more

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 e13fa66...35c5374. Read the comment docs.


lpp rosdep workflow fails

I have lpp installed in my regular catkin workspace
catkin_ws
-- src
-- -- lpp
-- -- another package depending on lpp in package.xml
My problem is that I get the following error when trying to install the dependencies using rosdep
For example
rosdep install omav_hovery_core
ERROR: the following packages/stacks could not have their rosdep keys resolved
to system dependencies:
omav_hovery_core: Cannot locate rosdep definition for [lpp]

My guess is there is some export or install missing in LPP's CMakeLists or I need to tell rosdep somehow to also look in the sources workspace to find dependencies. Thus rodep does not know that lpp is within catkin workspace. Do you know what to do @4c3y ?

AFAIK rosdep is only designed to install system dependencies.  It is not designed to install other ROS packages needed by omav_hovery_core

rosdep can install both ROS packages as well as system dependencies. Which is which is defined in this repository which is queried when one runs rosdep update. The yaml files in this repository map entries in package.xml files to packages for the respective package managers on different OS's and distros. If a dependency is not listed there it will try to install it as a ros package (ros-$ROSDISTRO-package-name). Since lpp is already present in the workspace and should not be considered one can add the -i or --ignore-src flag which will prevent rosdep to install packages already present in the workspace.


ARROW-3489: [Gandiva][C++] Added support for IN expressions.

Gandiva now supports processing in expressions.

We have an alternative to in-expression : a in (c1, c2, c3, .., cn) can also be written as (a == c1 || a == c2 || .. || a == cn). Can you please add a test in the microbenchmarks to compare the perf of the two ?

we'll know the threshold n at which "in expression" outperforms the ||
it'll test the efficiency of the boost variant (i'm sceptical about it's perf in this use case)

Codecov Report

Merging #2773 into master will increase coverage by 0.99%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2773      +/-   ##
==========================================
+ Coverage   87.56%   88.56%   +0.99%
==========================================
  Files         403      342      -61
  Lines       62113    58331    -3782
==========================================
- Hits        54389    51660    -2729
+ Misses       7654     6671     -983
+ Partials       70        0      -70

Impacted Files
Coverage Δ

rust/src/record_batch.rs

go/arrow/datatype_nested.go

rust/src/util/bit_util.rs

go/arrow/math/uint64_amd64.go

go/arrow/internal/testing/tools/bool.go

go/arrow/internal/bitutil/bitutil.go

go/arrow/memory/memory_avx2_amd64.go

go/arrow/array/null.go

rust/src/lib.rs

rust/src/array.rs

... and 53 more

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 aaf70a4...8a8a448. Read the comment docs.

Travis has passed, unrelated failure in Rust due to throttling by Travis.

@pitrou I've addressed all of the comments and the PR is good to go.
I am getting unrelated python code style errors in CI. Do you want to fix them as part of this PR?

The Python errors are ARROW-3606. To work around them (so that we have a CI run) please apply https://github.com/apache/arrow/pull/2812/commits/1adbe8a2bd81665ef5f182ad6a99f79299509293 (and undo your Python changes) :-)

@pitrou can you please merge if this looks ok to you..got a clean run on CI for everything except R(which is getting throttled)..

@pitrou - thanks for the thorough review :) can you please merge if this looks ok now?

thanks @pitrou


Add dbsize method

Please support redis dbsize method

I believe this issue should be closed now, since it was fixed in  #76

You are correct. Closing this issue.


Unexpected behavior when sending reaction in groups

When using self.send_reaction in DMs, it seems to work normally. However, when in groups, only the target author can see the reaction that the bot puts on their post. This is not the behavior i experience as a user of Signal Android or Desktop. Not sure if this is a bug in the Signal class, or an rpc error, or something in the underlying signal-cli.

this seems like it might be a bug with signal-cli, we should test signal-cli sendReaction directly w/o jsonrpc

https://github.com/mobilecoinofficial/forest/blob/main/forest/core.py#L496
lol we only ever send it as a DM, clients just magically figure out what message it's actually reacting too...

should be closed now


Feature/rpcholesky

PR Type

Feature
Documentation content changes
Tests

Description

The RPCHOLESKY algorithm, Algorithm 1 in this paper, which produces a low-rank approximation to a given matrix. This can be used for coresets (given Gram matrix) or quadrature (given kernel function). For the latter -- which is not implemented in this PR -- see this paper.
How Has This Been Tested?

Test A: RPCholesky tests added to tests/unit/test_coresubset.py
Test B: RPCholesky test added to tests/performance/test_coresubset.py
Test C: RPCholesky examples/tests added to examples and picked up in tests/integration/*
Checklist before requesting a review

[x] I have made sure that my PR is not a duplicate.
[x] My code follows the style guidelines of this project.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have performed a self-review of my code.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] New and existing unit tests pass locally with my changes.
[x] Any dependent changes have been merged and published in downstream modules.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.tl82649 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the changes @tl82649 - 2 very minor final requests. When you have done those, can you merge develop into this branch to resolve the conflicts and I'll approve?

Thanks @pc532627. Do you mean main instead of develop? I can't seem to see the latter in the remote branches.

Sorry, I did mean develop yes - but after approving all changes (thanks for making those) there are no conflicts so it's ready to merge - I'll do that now and ensure the related tickets are closed


[Feature Request] Enable file editing with azd

Bicep enables me the ability to read a JSON file during a build. This is great, as I can use a JSON file with settings that need to be applied to the deployment. However, it lacks the ability to edit a JSON file, so for some scenarios, I can't configure my app during an AZD deployment exercise.
For example, I have a Blazor WASM project I want to deploy. BWASM apps don't have the ability to read from environment variables. Instead, BWASM apps load the wwwroot\appsettings.json file in the project during run-time. I'd like to be able to set values in this file during an AZD deployment.

@bradygaster - I am guessing that what you want to do here is flow some values (which could be outputs of your bicep module) into wwwroot\appsettings.json or are the values you need to insert there coming from a different location?
If that's the case, and If I squint, it feels like our logic that we have today to take bicep output and put them into the dotnet secret store is sort of doing the same thing here.  I wonder if our existing logic could be generalized in some way.  That might also be nice since it feels a little wrong to me that we are using dotnet user-secret add on every bicep output instead of just ones that represent secrets like API keys.  I wonder if for normal values we should be adding them to some appsettings.json file.
If not, perhaps this a place where a pre-deployment script may be useful.  It could modify wwwroot\appsettings.json before it was packaged into the application bundle that we actually deploy.

If not, perhaps this a place where a pre-deployment script may be useful. It could modify wwwroot\appsettings.json before it was packaged into the application bundle that we actually deploy.

I do like this idea.


Download data from aws s3

Dear people in Tabula muris,
Thank you for your contribution in generating this beautiful data set.
I am trying to download data with the following command:
aws s3 cp s3://czbiohub-tabula-muris/TM_droplet_mat.h5ad .
However, I encounter the following error:
warning: Skipping file s3://czbiohub-tabula-muris/TM_droplet_mat.h5ad. Object is of storage class GLACIER. Unable to perform download operations on GLACIER objects. You must restore the object to be able to perform the operation. See aws s3 download help for additional parameter options to ignore or force these transfers.
I then tried to download data with the following command (plus --storage-class STANDARD --force-glacier-transfer):
aws s3 cp s3://czbiohub-tabula-muris/TM_droplet_mat.h5ad . --storage-class STANDARD --force-glacier-transfer
But I still have this error:
download failed: s3://czbiohub-tabula-muris/TM_droplet_mat.h5ad to ./TM_droplet_mat.h5ad An error occurred (InvalidObjectState) when calling the GetObject operation: The operation is not valid for the object's storage class
Could you provide some command lines for downloading data from you S3 buckets?
Best,
Tsunghan Hsieh (post doc in Riken)

Hi @pocession, the data is located here -- if you update your paths that should work!

Thanks.  It works now.

Hey, sorry if this is a silly question but isn't the link you posted to tabula-muris-senis and not tabula-muris data? I wanted to check out the data for the original tabula muris paper, but I'm getting this and some other S3 errors while attempting to download it. For example,
$ aws s3 cp s3://czb-tabula-muris/TM_droplet_metadata.csv .
fatal error: An error occurred (403) when calling the HeadObject operation: Forbidden

aws s3 ls works though. I'm not so familiar with S3, but it looks like some files have been compressed via Glacier and others we just don't have access to.

Hi, you are having same issues as I had before. To get the files by using AWS CLI, you have to apply for an account from Amazon S3 and get the keys. Follow these steps:

Apply for Amazon S3 account (You need a credit card). You can follow Amazon S3's manual.
Get the Amazon S3's access key ID and secret access key. You can follow AWS Account and Access Keys
Install Amazon S3 command line interface.
In your terminal, type aws configure.
Type your Amazon S3's access key ID and secret access key.
Go to Tabula muris S3 storage.
Copy the S3 URL of the data you want.
In your terminal, type aws s3 cp S3-URL your-destination-path

It seems more than three people (plus you and me) confused the downloading URL and the usage of Amazon S3. You may consider to open a new issue so that Tabula muris people may notice this.
Best

Thanks for the reply @pocession! You linked to tabula-muris-senis, which works for me. But tabula-muris doesn't work. I had configured the CLI already, like you mentioned. And the link I posted above is the S3 URI. For example, if you click on TM_droplet_metadata.csv here and then copy S3 URI, you'd get the same URI I posted above:
$ aws s3 cp s3://czb-tabula-muris/TM_droplet_metadata.csv .
fatal error: An error occurred (403) when calling the HeadObject operation: Forbidden

Could you please try this specific file and let me know if it works for you?

Maybe @aopisco, can you help?

@warrenalphonso No, I also could not download from czb-tabula-muris. But I guess the czb-tabula-muris-senis is the official repository?

Thanks for trying! I thought tabula-muris and tabula-muris-senis were two different datasets, since they're released in two separate papers around two years apart.

Ah I see. Maybe tabula-muris-senis is for single cell RNAseq and tabula-muris is for bulk RNAseq.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Warren Alphonso @.>
Sent: Wednesday, August 31, 2022 1:52:00 AM
To: czbiohub/tabula-muris @.>
Cc: Tsung-Han Hsieh @.>; Mention @.>
Subject: Re: [czbiohub/tabula-muris] Download data from aws s3 (Issue #245)
Thanks for trying! I thought tabula-muris and tabula-muris-senis were two different datasets, since they're released in two separate papers around two years apart.
—
Reply to this email directly, view it on GitHubhttps://github.com/czbiohub/tabula-muris/issues/245#issuecomment-1231920712, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGEW44BSY6LUWR5FI53VACLV3Y33BANCNFSM5677S6NQ.
You are receiving this because you were mentioned.Message ID: @.***>

Hi all, Tabula Muris Senis contains Tabula Muris. Both datasets are single cell. Tabula Muris is the 3 month timepoint. Tabula Muris Senis is 1 month, 3 months (Tabula Muris), 18 months, 21 months, 24 months, and 30 months. The reason why I sent the instructions for Tabula Muris Senis is because that dataset is more accessible and it does contain the Tabula Muris. I hope this helps.

Perfect, Thanks for the clarification!
From: aopisco @.>
Date: Wednesday, August 31, 2022 9:59
To: czbiohub/tabula-muris @.>
Cc: Tsung-Han Hsieh @.>, Mention @.>
Subject: Re: [czbiohub/tabula-muris] Download data from aws s3 (Issue #245)
Hi all, Tabula Muris Senis contains Tabula Muris. Both datasets are single cell. Tabula Muris is the 3 month timepoint. Tabula Muris Senis is 1 month, 3 months (Tabula Muris), 18 months, 21 months, 24 months, and 30 months. The reason why I sent the instructions for Tabula Muris Senis is because that dataset is more accessible and it does contain the Tabula Muris. I hope this helps.
—
Reply to this email directly, view it on GitHubhttps://github.com/czbiohub/tabula-muris/issues/245#issuecomment-1232327452, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGEW44CIUU7XAQUNZDAKKI3V32U53ANCNFSM5677S6NQ.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks for clarifying!


Fix when trying to convert an mma<!tt.ptr> into blocked

Hit by @apgoucher. Putting this up to unblock him, but I find absolutely
bizarre that we end up with a tensor of pointers in mma format. I
haven't investigated where this is created though. We should probably
look into it.

If the element type is the pointer type, the bitwidth is still 16?

Aren't gmem pointers 64-bit?

Right, so I think we should hit false and do not use stmatrix if the element type is !tt.ptr<f32> already with the existing code, right?

Right, so I think we should hit false and do not use stmatrix if the element type is !tt.ptr<f32> already with the existing code, right?

@Jokeren No, the problem isn't that it returns true; it's that the bitwidth is unimplemented in LLVM for pointer types so it throws an exception when evaluating the condition.

Oh, I see it now! Thanks for the explanation!


KIALI-362 Add pod status for each service listed

Adding pod status to every service in the service list.

There are no test for this Service List, so I couldn't update them. I am planning to add a subsequent PR adding them ( I think tests are out of KIALI-362 scope).

@lucasponce no it's fine, I was just experimenting but I'll consolidate over this work


coverage calculation fix

@kibertoad can you revert it?
Actually I saw no differences, and dig it deeper:
https://github.com/istanbuljs/schema/blob/master/default-exclude.js
defaults already excludes tests

Sure!


show-names - Avoid showing undefined on Enterprise

For #7699
const names = await api.v4(
	[...usernames].map(user =>
		api.escapeKey(user) + `: user(login: "${user}") {name}`,
	).join(','),
);

refined-github.js:3397 Uncaught (in promise) Error: GraphQL:
Could not resolve to a User with the login of 'undefined'.
    at Object.v4uncached (refined-github.js:3397:60)
    at async show_names_updateLink (refined-github.js:6239:21)

The reason cause this error is user is undefined, it means usernames set contains undefined data.

@busches it'd be great to have a proper fix at some point. I'm gonna merge this PR this week as a temporary fix.

I'll take a look ideally tonight, I haven't had any issues, but they upgraded GHE this weekend and maybe now there are. 😂

Never mind, the user said it was a conflict with their other extension


padding not working as expected in progress bar

Tried the widget page of agartest, testing with negative and positive values of padding in my style sheet.
I expected the whole progress bar be resized, but only the "external box" is, not the blue bar:
Examples
I'm attaching some screenshots
Agartest default

Negative value of padding
Note that percent value is not visible

Positive "large" value of padding

I modified the progress bar code. Now it almost works, but I am not sure if the solution is the right one, because I have not studied AGAR's widget structure thoroughly. Also I still have some doubt that there may be  ± 1 px problems.
I attach the diff so that you can take a look at it.
--- progress_bar.c.orig
+++ progress_bar.c
@@ -230,58 +230,56 @@
 static void
 Draw(void *_Nonnull obj)
 {
        AG_ProgressBar *pb = obj;
        AG_Rect rd = WIDGET(pb)->r;
        const int paddingLeft   = WIDGET(pb)->paddingLeft;
        const int paddingRight  = WIDGET(pb)->paddingRight;
        const int paddingTop    = WIDGET(pb)->paddingTop;
        const int paddingBottom = WIDGET(pb)->paddingBottom;
        int min, max, val;

        min = AG_GetInt(pb, "min");
        max = AG_GetInt(pb, "max");
        val = AG_GetInt(pb, "value");
        if (val < min) { val = min; }
        if (val > max) { val = max; }

        AG_DrawBoxSunk(pb, &rd, &WCOLOR(pb,BG_COLOR));

        if ((max - min) <= 0)
                return;

-       rd.x = paddingLeft;
-       rd.y = paddingTop;
+       rd.x = 0;
+       rd.y = 0;

        switch (pb->type) {
        case AG_PROGRESS_BAR_VERT:
-               rd.h -= (paddingTop + paddingBottom);
-               rd.y += rd.h - (val-min)*(rd.h)/(max-min) - 1;
-               rd.h = HEIGHT(pb) - paddingTop - rd.y + 1;
-               rd.w -= (paddingLeft + paddingRight);
+                rd.y += rd.h - (val-min)*(rd.h)/(max-min) - 1;
+                rd.h = HEIGHT(pb) - rd.y;
+               rd.w -= 1;
                break;
        case AG_PROGRESS_BAR_HORIZ:
        default:
-               rd.w = (val-min)*(rd.w - paddingLeft - paddingRight)/(max-min);
-               rd.h -= (paddingTop + paddingBottom);
+               rd.w = (val-min)*(rd.w - 1)/(max-min);
                break;
        }
        AG_DrawRect(pb, &rd, &WCOLOR(pb,SELECTION_COLOR));

        if (pb->flags & AG_PROGRESS_BAR_SHOW_PCT)
                DrawPercentText(pb);
 }

 AG_WidgetClass agProgressBarClass = {
        {
                "Agar(Widget:ProgressBar)",
                sizeof(AG_ProgressBar),
                { 1,0, AGC_PROGRESSBAR, 0xE032 },
                Init,
                NULL,           /* reset */
                Destroy,
                NULL,           /* load */
                NULL,           /* save */
                NULL            /* edit */
        },
        Draw,
        SizeRequest,

If you think I should submit a push request I will try to clone the repository and do it.

Looks good to me! Committed to: https://github.com/JulNadeauCA/libagar/commit/1140aa6aeb3b99ce76c890fe1fdd817bc7763801. Thank you very much!


TypeError: getRealCarData() takes 1 positional argument but 2 were given

I am finding some trouble after the initial data fetch with an Enyaq, which results in the following output
########## 09:54:23 AM - NEW LOOP ##########
TMBJC7NY1MFxxxxxx Charging time left - (charging_time_left)
	str_state: 0 h - state: 0
	attributes: {}

TMBJC7NY1MFxxxxxx Electric range - (electric_range)
	str_state: 344 km - state: 344
	attributes: {}

TMBJC7NY1MFxxxxxx Battery level - (battery_level)
	str_state: 78 % - state: 78
	attributes: {}

TMBJC7NY1MFxxxxxx External power - (external_power)
	str_state: On - state: True
	attributes: {}

TMBJC7NY1MFxxxxxx Charging cable connected - (charging_cable_connected)
	str_state: Charging - state: True
	attributes: {}

TMBJC7NY1MFxxxxxx Charging - (charging)
	str_state: Off - state: 0
	attributes: {'last_result': ''}

TMBJC7NY1MFxxxxxx Charging cable locked - (charging_cable_locked)
	str_state: Locked - state: None
	attributes: {}

Sleeping for 20 seconds
Updating ALL values from Skoda Connect...
Success!
Updating INDIVIDUAL values for TMBJC7NY1MFxxxxxx...
Updating carportdata...
Updating charger data...
Updating climater data...
Updating position data...
Updating pre-heater data...
Updating realcardata...
Traceback (most recent call last):
  File "skoda.py", line 315, in <module>
    loop.run_until_complete(main())
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "skoda.py", line 285, in main
    await vehicle.get_realcardata()
  File "xxxxxxx/skodaconnect/vehicle.py", line 154, in get_realcardata
    data = await self._connection.getRealCarData(self.vin)
TypeError: getRealCarData() takes 1 positional argument but 2 were given

Updating realcardata...
Traceback (most recent call last):
File "skoda.py", line 315, in
loop.run_until_complete(main())
File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
return future.result()
File "skoda.py", line 285, in main
await vehicle.get_realcardata()
File "xxxxxxx/skodaconnect/vehicle.py", line 154, in get_realcardata
data = await self._connection.getRealCarData(self.vin)
TypeError: getRealCarData() takes 1 positional argument but 2 were given

Thanks, I'll fix this in next commit.

Thanks @Farfar This fix resulted in the following error:
Sleeping for 20 seconds
Updating ALL values from Skoda Connect...
Success!
Updating INDIVIDUAL values for TMBJC7NY1MFxxxxxx...
Updating carportdata...
Updating charger data...
Updating climater data...
Updating position data...
Updating pre-heater data...
Updating realcardata...
Traceback (most recent call last):
  File "skoda.py", line 315, in <module>
    loop.run_until_complete(main())
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "skoda.py", line 285, in main
    await vehicle.get_realcardata()
  File "xxxxxx/skodaconnect/vehicle.py", line 156, in get_realcardata
    self._states.update(data)
ValueError: dictionary update sequence element #0 has length 7; 2 is required

Best regards

Thanks @Farfar This fix resulted in the following error:
Sleeping for 20 seconds
Updating ALL values from Skoda Connect...
Success!
Updating INDIVIDUAL values for TMBJC7NY1MFxxxxxx...
Updating carportdata...
Updating charger data...
Updating climater data...
Updating position data...
Updating pre-heater data...
Updating realcardata...
Traceback (most recent call last):
  File "skoda.py", line 315, in <module>
    loop.run_until_complete(main())
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "skoda.py", line 285, in main
    await vehicle.get_realcardata()
  File "xxxxxx/skodaconnect/vehicle.py", line 156, in get_realcardata
    self._states.update(data)
ValueError: dictionary update sequence element #0 has length 7; 2 is required

Best regards

Seems like it's more places in example code that the changes have broken stuff. I will look through once more.

Fixed in latest commit and release 1.0.46rc6

Hi @Farfar
Now we have a different issue, looks like it triggers before the others
Sleeping for 20 seconds
Updating ALL values from Skoda Connect...
Success!
Updating INDIVIDUAL values for TMBJC7NY1MFxxxxxx...
Updating carportdata...
Traceback (most recent call last):
  File "skoda.py", line 315, in <module>
    loop.run_until_complete(main())
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "skoda.py", line 275, in main
    await vehicle.get_carportdata()
AttributeError: 'Vehicle' object has no attribute 'get_carportdata'

Kind regards

Hi @Farfar
Now we have a different issue, looks like it triggers before the others
Sleeping for 20 seconds
Updating ALL values from Skoda Connect...
Success!
Updating INDIVIDUAL values for TMBJC7NY1MFxxxxxx...
Updating carportdata...
Traceback (most recent call last):
  File "skoda.py", line 315, in <module>
    loop.run_until_complete(main())
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "skoda.py", line 275, in main
    await vehicle.get_carportdata()
AttributeError: 'Vehicle' object has no attribute 'get_carportdata'

Kind regards

Make sure you are using latest example code and library.
The carportdata function has been removed since Skoda has migrated the information to new API endpoints.

Absolutely agree with your point. I did not update the code in my script.
I did a brand new git clone to a new directory but got similar results:
########################################
#    Updating all values from Skoda    #
########################################
Updating ALL values from Skoda Connect...
DEBUG:skodaconnect.connection:Going to call vehicle updates
DEBUG:skodaconnect.connection:Adding TMBJC7NY1MFxxxxxx for data refresh
DEBUG:skodaconnect.connection:Calling update function for all vehicles
DEBUG:skodaconnect.connection:HTTP GET "https://api.connect.skoda-auto.cz/api/v1/charging/TMBJC7NY1MFxxxxxx/status"
DEBUG:skodaconnect.connection:Request for "https://api.connect.skoda-auto.cz/api/v1/charging/TMBJC7NY1MFxxxxxx/status" returned with status code [200], response: {'plug': {'connectionState': 'Connected', 'lockState': 'Locked'}, 'charging': {'state': 'ReadyForCharging', 'remainingToCompleteInSeconds': 0, 'chargingPowerInWatts': 4000.0, 'chargingRateInKilometersPerHour': 0.0, 'chargingType': 'Invalid', 'chargeMode': 'OFF'}, 'battery': {'cruisingRangeElectricInMeters': 304000, 'stateOfChargeInPercent': 66}}
DEBUG:skodaconnect.connection:Got vehicle data {'plug': {'connectionState': 'Connected', 'lockState': 'Locked'}, 'charging': {'state': 'ReadyForCharging', 'remainingToCompleteInSeconds': 0, 'chargingPowerInWatts': 4000.0, 'chargingRateInKilometersPerHour': 0.0, 'chargingType': 'Invalid', 'chargeMode': 'OFF'}, 'battery': {'cruisingRangeElectricInMeters': 304000, 'stateOfChargeInPercent': 66}}
Success!

Sleeping for 20 seconds

########################################
#          Update carportdata          #
###########TMBJC7NY1MFxxxxxx############
Traceback (most recent call last):
  File "example.py", line 282, in <module>
    loop.run_until_complete(main())
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "example.py", line 206, in main
    await vehicle.get_carportdata()
AttributeError: 'Vehicle' object has no attribute 'get_carportdata'

Absolutely agree with your point. I did not update the code in my script.
I did a brand new git clone to a new directory but got similar results:
########################################
#    Updating all values from Skoda    #
########################################
Updating ALL values from Skoda Connect...
DEBUG:skodaconnect.connection:Going to call vehicle updates
DEBUG:skodaconnect.connection:Adding TMBJC7NY1MFxxxxxx for data refresh
DEBUG:skodaconnect.connection:Calling update function for all vehicles
DEBUG:skodaconnect.connection:HTTP GET "https://api.connect.skoda-auto.cz/api/v1/charging/TMBJC7NY1MFxxxxxx/status"
DEBUG:skodaconnect.connection:Request for "https://api.connect.skoda-auto.cz/api/v1/charging/TMBJC7NY1MFxxxxxx/status" returned with status code [200], response: {'plug': {'connectionState': 'Connected', 'lockState': 'Locked'}, 'charging': {'state': 'ReadyForCharging', 'remainingToCompleteInSeconds': 0, 'chargingPowerInWatts': 4000.0, 'chargingRateInKilometersPerHour': 0.0, 'chargingType': 'Invalid', 'chargeMode': 'OFF'}, 'battery': {'cruisingRangeElectricInMeters': 304000, 'stateOfChargeInPercent': 66}}
DEBUG:skodaconnect.connection:Got vehicle data {'plug': {'connectionState': 'Connected', 'lockState': 'Locked'}, 'charging': {'state': 'ReadyForCharging', 'remainingToCompleteInSeconds': 0, 'chargingPowerInWatts': 4000.0, 'chargingRateInKilometersPerHour': 0.0, 'chargingType': 'Invalid', 'chargeMode': 'OFF'}, 'battery': {'cruisingRangeElectricInMeters': 304000, 'stateOfChargeInPercent': 66}}
Success!

Sleeping for 20 seconds

########################################
#          Update carportdata          #
###########TMBJC7NY1MFxxxxxx############
Traceback (most recent call last):
  File "example.py", line 282, in <module>
    loop.run_until_complete(main())
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "example.py", line 206, in main
    await vehicle.get_carportdata()
AttributeError: 'Vehicle' object has no attribute 'get_carportdata'

That's my bad. I forgot to actually remove the lines from the example. You can remove lines 202-206 or pull latest commit.

Now it is working just fine. Thanks!


[Bug Report] import gymnasium causes TypeError: expected string or bytes-like object

Describe the bug
Importing gymnasium causes a python exception to be raised.
$ python3 -c 'import gymnasium as gym'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/martin/.local/lib/python3.9/site-packages/gymnasium/__init__.py", line 12, in <module>
    from gymnasium.envs.registration import make, spec, register, registry, pprint_registry
  File "/home/martin/.local/lib/python3.9/site-packages/gymnasium/envs/__init__.py", line 364, in <module>
    load_env_plugins()
  File "/home/martin/.local/lib/python3.9/site-packages/gymnasium/envs/registration.py", line 273, in load_env_plugins
    for plugin in metadata.entry_points(group=entry_point):
  File "/home/martin/.local/lib/python3.9/site-packages/importlib_metadata/__init__.py", line 856, in entry_points
    return EntryPoints(eps).select(**params)
  File "/home/martin/.local/lib/python3.9/site-packages/importlib_metadata/__init__.py", line 853, in <genexpr>
    eps = itertools.chain.from_iterable(
  File "/home/martin/.local/lib/python3.9/site-packages/importlib_metadata/_itertools.py", line 16, in unique_everseen
    k = key(element)
  File "/home/martin/.local/lib/python3.9/site-packages/importlib_metadata/_py39compat.py", line 18, in normalized_name
    return dist._normalized_name
  File "/home/martin/.local/lib/python3.9/site-packages/importlib_metadata/__init__.py", line 778, in _normalized_name
    or super()._normalized_name
  File "/home/martin/.local/lib/python3.9/site-packages/importlib_metadata/__init__.py", line 445, in _normalized_name
    return Prepared.normalize(self.name)
  File "/home/martin/.local/lib/python3.9/site-packages/importlib_metadata/__init__.py", line 700, in normalize
    return re.sub(r"[-_.]+", "-", name).lower().replace('-', '_')
  File "/usr/lib/python3.9/re.py", line 210, in sub
    return _compile(pattern, flags).sub(repl, string, count)
TypeError: expected string or bytes-like object

Code example
No response
System info
$ pip3 install 'gymnasium[atari]'
Collecting gymnasium[atari]
  Downloading gymnasium-0.27.1-py3-none-any.whl (883 kB)
     |████████████████████████████████| 883 kB 775 kB/s
Collecting typing-extensions>=4.3.0
  Downloading typing_extensions-4.5.0-py3-none-any.whl (27 kB)
Collecting cloudpickle>=1.2.0
  Downloading cloudpickle-2.2.1-py3-none-any.whl (25 kB)
Collecting jax-jumpy>=0.2.0
  Downloading jax_jumpy-0.2.0-py3-none-any.whl (11 kB)
Collecting numpy>=1.21.0
  Downloading numpy-1.24.2-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (17.3 MB)
     |████████████████████████████████| 17.3 MB 1.4 MB/s
Collecting gymnasium-notices>=0.0.1
  Downloading gymnasium_notices-0.0.1-py3-none-any.whl (2.8 kB)
Collecting importlib-metadata>=4.8.0
  Downloading importlib_metadata-6.0.0-py3-none-any.whl (21 kB)
Collecting shimmy[atari]<1.0,>=0.1.0
  Downloading Shimmy-0.2.1-py3-none-any.whl (25 kB)
Requirement already satisfied: zipp>=0.5 in /usr/lib/python3/dist-packages (from importlib-metadata>=4.8.0->gymnasium[atari]) (1.0.0)
Collecting ale-py~=0.8.1
  Downloading ale_py-0.8.1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (1.7 MB)
     |████████████████████████████████| 1.7 MB 1.4 MB/s
Collecting importlib-resources
  Downloading importlib_resources-5.12.0-py3-none-any.whl (36 kB)
Collecting zipp>=0.5
  Downloading zipp-3.15.0-py3-none-any.whl (6.8 kB)
Installing collected packages: zipp, numpy, typing-extensions, jax-jumpy, importlib-metadata, gymnasium-notices, cloudpickle, importlib-resources, gymnasium, shimmy, ale-py
  WARNING: The scripts f2py, f2py3 and f2py3.9 are installed in '/home/martin/.local/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
  WARNING: The script ale-import-roms is installed in '/home/martin/.local/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
Successfully installed ale-py-0.8.1 cloudpickle-2.2.1 gymnasium-0.27.1 gymnasium-notices-0.0.1 importlib-metadata-6.0.0 importlib-resources-5.12.0 jax-jumpy-0.2.0 numpy-1.24.2 shimmy-0.2.1 typing-extensions-4.5.0 zipp-3.15.0

$ python3 --version
Python 3.9.2

I'm using Debian 11.
Additional context
Debugging the import code, I can see that in the following method:
@property
def _normalized_name(self):
    """Return a normalized version of the name."""
    print(self.name)
    return Prepared.normalize(self.name)

The value of self.name is None.
I verified that my python installation works correctly and imports of other modules don't generate this error.
Checklist

[X] I have checked that there is no similar issue in the repo

Thanks for the fast reaction!
After further investigation, I was able to fix this.
I was testing on my local system. I was seeing the exception when running with a regular user but not with root. After some debugging, I figured out there is an egg file (kubernetes-25.3.0-py3.9.egg) which seems to interfere with gymnasium. Removing the file fixes the issue.
Given that it worked with root, I'm suspecting the problem had something to do with permissions. I have no idea how exactly kubernetes causes problems specifically with gymnasium and not with other modules like numpy or matplotlib. I'm not sure if importing gymnasium triggers some deep chain of dependencies. In any case the problem seems to be somewhere else so this bug should probably be closed.
I'm providing the output of freeze in case somebody finds this helpful.
$ pip freeze
attrs==20.3.0
awscli==1.19.1
beautifulsoup4==4.9.3
botocore==1.20.0
cached-property==1.5.2
cachetools==5.1.0
cajarename==20.1.26
camset==0.0.19
certifi==2020.6.20
chardet==3.0.4
charset-normalizer==2.0.12
click==8.1.3
click-default-group==1.2.2
cloudpickle==2.2.1
cloup==0.13.1
colorama==0.4.4
colour==0.1.5
commonmark==0.9.1
configobj==5.0.6
coverage==7.0.5
crcmod==1.7
cryptography==3.3.2
cycler==0.10.0
dbus-python==1.2.16
decorator==5.1.1
distro==1.5.0
distro-info==1.0
docker==4.1.0
docker-compose==1.25.0
dockerpty==0.4.1
docopt==0.6.2
docutils==0.16
evdev==1.4.0
glcontext==2.3.6
google-api-core==2.7.3
google-api-python-client==2.47.0
google-auth==2.6.6
google-auth-httplib2==0.1.0
google-auth-oauthlib==0.5.1
googleapis-common-protos==1.56.1
gpg===1.14.0-unknown
gymnasium==0.27.1
gymnasium-notices==0.0.1
gyp==0.1
html5lib==1.1
httplib2==0.18.1
idna==2.7
importlib-metadata==6.0.0
iniconfig==0.0.0
isosurfaces==0.1.0
jax-jumpy==0.2.0
jmespath==0.10.0
joblib==1.1.0
jsonschema==3.2.0
kiwisolver==1.3.1
kubernetes==25.3.0
kubernetes-client==0.1.7
lxml==4.6.3
manim==0.15.2
manimlib==0.2.0
ManimPango==0.4.1
mapbox-earcut==0.12.11
mate-menu==20.4.1
mate-tweak==21.4.2
matplotlib==3.3.4
moderngl==5.6.4
moderngl-window==2.4.1
more-itertools==4.2.0
mpmath==1.2.1
multidict==6.0.2
multipledispatch==0.6.0
mypy==0.991
mypy-extensions==0.4.3
networkx==2.8.2
nonconformist==2.1.0
numpy==1.24.2
oauthlib==3.2.0
olefile==0.46
opencv-python==<IP_ADDRESS>
packaging==20.9
pandas==1.4.1
Pillow==9.1.1
pluggy==0.13.0
progressbar==2.5
protobuf==3.20.1
psutil==5.8.0
pulsar-client==2.9.1
py==1.10.0
pyasn1==0.4.8
pyasn1-modules==0.2.8
pyC8==0.16.9
pycairo==1.21.0
pycurl==<IP_ADDRESS>
pydantic==1.9.0
pydub==0.25.1
pygame==1.9.6
pyglet==1.5.26
Pygments==2.12.0
PyGObject==3.38.0
PyICU==2.5
PyOpenGL==3.1.5
pyparsing==2.4.7
pyrr==0.10.3
pyrsistent==0.15.5
pystache==0.5.4
pytest==6.0.2
pytest-cov==4.0.0
python-apt==2.2.1
python-dateutil==2.8.1
python-debian==0.1.39
python-magic==0.4.20
python-xlib==0.29
pytz==2021.3
pyxdg==0.27
PyYAML==6.0
requests==2.27.1
requests-oauthlib==1.3.1
rich==12.4.4
roman==2.0.0
rsa==4.0
s3transfer==0.3.4
scikit-learn==1.1.1
scipy==1.8.1
screeninfo==0.8
semver==2.10.2
setproctitle==1.2.1
six==1.16.0
skia-pathops==0.7.2
sklearn==0.0
soupsieve==2.2.1
srt==3.5.2
sympy==1.10.1
table-logger==0.3.6
texttable==1.6.3
threadpoolctl==3.1.0
toml==0.10.1
tomli==2.0.1
tqdm==4.64.0
typing_extensions==4.5.0
unattended-upgrades==0.1
uritemplate==4.1.1
urllib3==1.24.3
watchdog==2.1.8
webencodings==0.5.1
websocket-client==0.57.0
xdg==5
XlsxWriter==3.0.2
xlwt==1.3.0
zipp==1.0.0

I just saw that this is the same error as https://github.com/Farama-Foundation/Gymnasium/issues/428


Adding conda environment (anaconda.org) segment.

See https://github.com/necaris/conda.el.

Would love to colour it though, but it is not as straight-forward as I wish ;-)

Hummm, for some reason it breaks the buffer-position and the hud segments, whenever its condition is not true ...

I'm not familiar with Conda or Anaconda 😅
What does conda-env-current-name look like?
Also, in your predicate, won't conda-env-current-name be nil if there isn't one?
And how does it break the buffer-position/hud segments?

What does conda-env-current-name look like?

It is defined as such:
(defvar conda-env-current-name nil "Name of current conda env.")

Also, in your predicate, won't conda-env-current-name be nil if there isn't one?

(length nil) ;; => 0

So I guess it is fine ?
As for the buffer-position/hud segments, they only both appear when the :when condition of this segment holds.
I have then tried to use the vanilla spaceline-all-the-icons, but I can't these two to work together.


fixed wording

You should make changes upstream.


API: PeriodIndex.values now return array of Period objects

[x] split from #13941 (comment)
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

This change itself needs more tests / perf testing, split from the original PR.

Current coverage is 85.27% (diff: 94.66%)

Merging #13988 into master will decrease coverage by <.01%

@@             master     #13988   diff @@
==========================================
  Files           139        139
  Lines         50206      50237    +31
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42818      42841    +23
- Misses         7388       7396     +8
  Partials          0          0

Powered by Codecov. Last update 29d9e24...f1984b4

this looks fine. pls add a small section for showing the changes in whatsnew (though if this should be merged after #13941 can just add on there (and vice-versa)

OK, let #13941 be first, because this PR includes some workaround which can be removed once we have period dtype.

@sinhrks pls rebase. This will be a welcome change!

Needs a new rebase ;) (for your fixed of the tests)

Can you also add a whatsnew notice?

Added whatsnew, now ready for review.

thanks!


Issue with code highlighting of numbers with underscores.

If you look at chapter-1 #integer-rules or #floats
The part explaining you you can use underscores has an example:

In which the colouring of the numbers stops as soon as an underscore is hit.
I assume this is an easy fix, but I have no idea how this site generates it's code highlighting.

Just looked at the repo, it's just the markdown files?
Where exactly is the site stored or is it all auto-generated?
If it is how/what is used to generate should be mentioned in the README.

The site itself uses a build of https://github.com/gohugoio/hugo, which uses https://github.com/alecthomas/chroma for highlighting. The zig support was added in https://github.com/alecthomas/chroma/pull/364, and has this bug.
I don't plan on adding info on how it's generated, because it shouldn't need anything special to be generated - plain markdown is wanted.