Fix recursive ORM parsing error

The ORM mode doesn't parse dictionaries if it's not explicitly called.
Change Summary
Prepend the orm_mode checking to the dict's.
Related issue number

Checklist

[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)

Example
from pydantic import BaseModel
from pydantic.utils import GetterDict

class Getter(GetterDict):
    # not relevant, just to show the behavior
    def get(self, key, default):
        key = key + key
        try:
            return self._obj[key]
        except TypeError:
            return getattr(self._obj, key, default)
        except KeyError:
            return default

class Model(BaseModel):
    class Config:
        orm_mode = True
        getter_dict = Getter

class ModelA(Model):
    a: int

class ModelB(Model):
    b: ModelA

from types import SimpleNamespace

x = dict(bb=SimpleNamespace(aa=1))
ModelB.from_orm(x)
#> b=ModelA(a=1)

y = dict(bb=dict(aa=1))
ModelB.from_orm(y)
#> ...
#> b -> a
#>   field required (type=value_error.missing)

@PrettyWood Should I create a new test case or would you prefer I adapt test_custom_getter_dict_derived_model_class to check this condition?
https://github.com/samuelcolvin/pydantic/blob/c489445cea40c4962ecfa9e342eb30a0ba819129/tests/test_orm_mode.py#L271-L304

Thank you!

This change actually prevents models (that are ORM-enabled) from being initialized the normal way from dictionaries. Prior to this change, the following code worked:
from types import SimpleNamespace

from pydantic import BaseModel

class User(BaseModel):
    first_name: str
    last_name: str

    class Config:
        orm_mode = True

class State(BaseModel):
    user: User

    class Config:
        orm_mode = True

# Pass an "orm instance"
state = State.from_orm(SimpleNamespace(user=SimpleNamespace(first_name='John', last_name='Appleseed')))

# Pass dictionary data directly
state = State(**{'user': {'first_name': 'John', 'last_name': 'Appleseed'}})

After this change, the final line throws a validation error:
Traceback (most recent call last):
  File "/Users/luke/Developer/Clients/doctorschoice/flow/orm_issue.py", line 26, in <module>
    state = State(**{'user': {'first_name': 'John', 'last_name': 'Appleseed'}})
  File "/Users/luke/.pyenv/versions/flow/lib/python3.9/site-packages/pydantic/main.py", line 330, in __init__
    raise validation_error
pydantic.error_wrappers.ValidationError: 2 validation errors for State
user -> first_name
  field required (type=value_error.missing)
user -> last_name
  field required (type=value_error.missing)

This is actually a bit of an issue given the way FastAPI handles response models. During the processing of a request, it serializes the data, and then passes it through a response model (if one was provided) to make sure the response matches the desired output schema. If you're using an ORM-mode model for convenience (like one generated from Tortoise-ORM's pydantic_model_creator), it now crashes with a validation error.


Support proxy configuration

Closed environments often require a proxy configuration. This currently causes an error as fresh rules cannot be downloaded.
Replace file_get_contents in CachedDownload with cURL, which has simple proxy support using environment variables.

Implemented by https://github.com/lightswitch05/php-version-audit/pull/6


The backup-documents function should test and split the archive

Is your feature request related to a problem? Please describe.
The function itself works well, but the additional steps are required every time. At first, the archive with the documents is not being tested. We always need to check if it's not corrupted. Also, the archive can be quite big, and it cannot be copied to the flash drive with fat32. We need to split it manually.
Describe the solution you'd like
It'll be good to run split --bytes=1G <archive name.zip> at the end of the function if the archive is bigger than 4GB. It doesn't require any interactions with the user. Also, unzip -t will check the archive, but it requires user input (password). Need to think how to avoid it and to not leave anything sensitive in the shell history.
Describe alternatives you've considered
For now all these commands can be executed manually.

Fixed.


winpr-makecert implementation

Implemented self-signed TLS certificate generation for RDP usage with winpr-makecert
winpr-makecert -rdp generates a .crt and .key file readily usable by the FreeRDP servers.

I am only able to create .key.
.crt file is not created.


Getting exception when I reset the store

I'm not quite sure how to debug this. I'm getting an exception in this code in QueryManager:
forOwn(this.queryListeners, function (listeners, queryId) {
    listeners.forEach(function (listener) {
        if (listener) {
            var queryStoreValue = queries[queryId];
            listener(queryStoreValue);
        }
    });
});

Specifically, the 'listeners' variable is undefined. This occurs immediately after calling resetStore() - I'm doing a logout.
Any thoughts as to what might be causing this or how I might go about tracking it down?

I have same issue with latest<EMAIL_ADDRESS>I can simulate it always with refetchQueries.

My this.queryListeners looks like this: Object {0: Array[1], 8: Array[1]}
but in forEach I will get 3 cycles:
QueryManager.js:629 listeners [function] 0
QueryManager.js:629 listeners [function] 8
QueryManager.js:629 listeners undefined 1

It looks like a bug in the forOwn because Object.keys(this.queryListeners) will return ["0", "8"]
or function listener on the line https://github.com/apollostack/apollo-client/blob/master/src/core/QueryManager.ts#L931 will change this.queryListeners. The second option is more likely

@viridia @seeden thanks for reporting this and digging a bit deeper. I will set aside some time to figure out what's really going on and hopefully fixing the bug.
I'm pretty sure there's nothing wrong with forOwn, it might just be that the thing we're iterating over changes while we're iterating.

@viridia I am using temporar fix for this issue because I was not able to continue in my work without it.
Add next code
if (!listeners) {
  return;
}

into the https://github.com/apollostack/apollo-client/blob/master/src/core/QueryManager.ts#L986

This issue is still here in the 0.5.0-1

It was OK in the version 0.4.20 but from 0.4.21 I can see this issue

@seeden Yeah, it's getting hard to work as I am hitting this repeatedly.

@stubailo can you add it into the 0.5 milestone?

@seeden @viridia can either of you try to make a minimal reproduction or a failing test case for this? That would make it a lot easier for me to track it down.

I will try to create test case today

I put in a patch for this for now so it doesn't crash things any more. We'll track it down post 0.5

OK I updated to the latest version I'll let you know if I see any more of these.

Since updating to 0.5.0 I'm not seeing this anymore.

Closing this since it seems to have been solved. If anyone runs into a similar issue, please also make sure that you're properly binding all the method calls to your Apollo Client instance where necessary, e.g. client.methodX.bind(client) .


ngRepeat:dupes error for multi select and remote data source

Hi,
I found a race condition in the latest version v0.11.2, for multi select.
If the data source of the select is loaded after the selection the widget raise the error ngRepeat:dupes and create duplicates item in the drop down.
But if the data source is loaded before the selection everything works as expected.
Here is a plunker to demonstrate the issue [http://plnkr.co/edit/juqoNOt1z1Gb349XabQ2?p=preview]
In this plunker if the variable objectLoadTime (line 6) is lower than dataSourceLoadTime (line 5) everything works
If the variable objectLoadTime is greater than dataSourceLoadTime duplicates are created.
Thanks

+1

+1， I cant find the demo code. But as you say, I prepare the data before ui-select render and it works!

+1. Has anyone found a workaround?

Closing as a duplicate of several ng-repeat errors.


sqlc: adopt

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Splitting your changes into separate commits would be great too:
https://github.com/NixOS/nixpkgs/blob/86c19dd026cd371140b6a5b9f7e6188830ee2621/CONTRIBUTING.md?plain=1#L497
For example:

sqlc: migrate to pkgs/by-name
sqlc: nixfmt-rfc-style formatting
.git-blame-ignore-revs: add "sqlc: nixfmt-rfc-style formatting"
sqlc: install shell completions
sqlc: "add testers.testVersion test" or "use versionCheckHook"

This PR is more like a rewrite. How about making the PR description more detailed?

Closing in favor of https://github.com/NixOS/nixpkgs/pull/351796

@zi3m5f
I’m really sorry for how I handled that PR. I didn’t mean to overlook your time and effort—your input was valuable and made a real difference. I think that’s also why the new PR could be merged so quickly. Thanks for your understanding and for your contributions! :)


Pitch a Project page

Add page for pitching a project.
Figma design is here.

I'll start working on this today @natebass. Thanks


kodiPackages.jellyfin: 0.7.12 -> 1.0.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jellyfin/jellyfin-kodi/releases.
meta.description for kodiPackages.jellyfin is: A whole new way to manage and view your media library
meta.homepage for kodiPackages.jellyfin is: https://jellyfin.org/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.2 with grep in /nix/store/5wgranj0vfdnngl3bj2zq20cbifx1cak-kodi-jellyfin-1.0.2
found 1.0.2 in filename of file in /nix/store/5wgranj0vfdnngl3bj2zq20cbifx1cak-kodi-jellyfin-1.0.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
kodiPackages.jellyfin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5wgranj0vfdnngl3bj2zq20cbifx1cak-kodi-jellyfin-1.0.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A kodiPackages.jellyfin https://github.com/r-ryantm/nixpkgs/archive/e7ab7b41b2aad18a9bdde126bb1622181bc9092d.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e7ab7b41b2aad18a9bdde126bb1622181bc9092d#kodiPackages.jellyfin

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5wgranj0vfdnngl3bj2zq20cbifx1cak-kodi-jellyfin-1.0.2
ls -la /nix/store/5wgranj0vfdnngl3bj2zq20cbifx1cak-kodi-jellyfin-1.0.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

kodiPackages.jellyfin

Maintainer pings
cc @aanderse @cpages @dschrempf @edwtjo @minijackson @peterhoeg @sephalon for testing.

As far as I can see, this is covered by https://github.com/NixOS/nixpkgs/pull/303497. Please reopen if this is not the case.


Running formatters with non-stdin option to respect "ignore patterns"

Neovim version (nvim -v)
0.9.4
Operating system/version
macOS, Linux
Add the debug logs

[X] I have set log_level = vim.log.levels.DEBUG and pasted the log contents below.

Log file
          21:29:59[INFO] Run yapf on /path/to/some/project/mypackage/envs.py
          21:29:59[DEBUG] Run command: { "yapf", "--quiet" }
          21:29:59[DEBUG] Run CWD: /path/to/some/project/envs.py
          21:30:00[DEBUG] yapf exited with code 0

Describe the bug
It looks like many of the formatters run the external formatter program through stdin/stdout PIPE. One drawback of this is that the filename is lost, so the formatting does not respect "ignore files".
For example, yapf 0.32+ supports .yapfignore or [tool.yapfignore] in pyproject.toml, but this setting is ignored because conform.nvim executes yapf by feeding the buffer contents into stdin.
I know the options stdin = false and args = { ..., "$FILENAME "}, but is this something that can be improved by default (also for many formatters not only for yapf) or are users supposed to take care of this manually?
Steps To Reproduce
Put .yapfignore in the project root:
# Ignore everything
**/*.py

Expected Behavior
Ignore files are respected.
Minimal example file
(Any files are affected)
Minimal init.lua
Skipped, not needed.

Additional context
No response

I think this is going to be handled on a case-by-case basis for formatters. They all have different config files and logic for allowlisting/blocklisting certain files from formatting rules. If you can come up with a configuration that respects the config file by default, I'll take a PR to add it.

Fewer moving parts, fewer things that can go wrong, slightly better performance. But it's a mild preference, so if there's a solid reason why we should prefer the file-based formatting we shouldn't be shy about using that instead.

I see, thanks for the input. I also see another benefit: it can be also useful when dealing with range formatting.
For YAPF, actually some upstream fix is needed. I will come back again with a PR if I have a reasonable solution to deal with this specific use case.

ref: https://github.com/bazelbuild/buildtools/issues/1268


Work around BZ 1233329

https://bugzilla.redhat.com/show_bug.cgi?id=1233329
http://tracker.ceph.com/issues/12005

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1566/
Tests passed for this pull request.

LGTM.


feat: add crn parser submodule

Description
Add the crn parser submodule and initial module cleanup from template
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Add CRN parser submodule
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

@ocofaigh one thing I thought about while fixing a test failure, do we need the CRA scan since this is just a local? I presume any utilities we'd add in the future would be similar in that they don't need the provider so might be better to just pull it from the module

/run pipeline

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix an OIP compatibility issue

Overview
Fix a class name collision with Oculus Integration Package.

Insane that this is needed. Ugh.


Why is it borderline impossible to downgrade WSL2?

Is your feature request related to a problem? Please describe.
There have been multiple times already when a WSL2 update has changed and broken some networking settings, and it will probably keep happening - no one is perfect, not even WSL2 developers.
Thus it should be possible to install a specific WSL version (any of those from the releases page), without having to go through hoops. Previously there was a wsl --update --rollback option, which wasn't perfect (didn't allow you to specify a specific version), but at least rolled back a version, but then you removed it.
There was also a very hidden (took me ages to find this through google, why is it so hidden!) way to install a specific release like this:
powershell Add-AppxPackage -Path .\Microsoft.WSL_<IP_ADDRESS>_x64_ARM64.msixbundle -ForceUpdateFromAnyVersion -ForceApplicationShutdown
But as we tried this on a colleague's machine, we got this error:
error 0x80073D27: While processing the request, the system failed to register the windows.service extension due to the following error: The package installation failed because a version of the service exists outside of APPX packaging. Please contact your software vendor.
So apparently that's no longer supported either? Or at the very least it expects you to fully uninstall WSL2 (and lose your distro setup) before doing the downgrade.
Being able to downgrade is an important troubleshooting step, even if you think WSL2 releases are perfect and bugfree (have they ever been?), so this should be made as easy as possible!
Describe the solution you'd like
It should be easy to find docs on how to downgrade and it should be easy to do it too. There should just be a CLI flag to install a specific version: wsl --downgrade 1.2.3
Please for the love of god introduce this feature so troubleshooting WSL2 environments is less painful
Describe alternatives you've considered
There is no alternative to downgrading when you're troubleshooting issues. Sometimes WSL2 just has a buggy version out and all you can do is either downgrade or wait for the issue to be resolved (which could take weeks or months depending on how complicated the issue is).

/feature

Thank you @fabis94.
Automatically downgrading to older versions of WSL is unfortunately very difficult to achieve at the technical level, and is a relatively rare scenario.
In your case though since you appear to be downgrading from >= 2.0.0 to 1.X, here's what you were missing:
Running:
wsl --uninstall

Before installing  a version < 2.0.0 should resolve the issue.

@OneBlue Fair enough. Could this information about why downgrading is not made possible and instructions on how to rollback be made more visible, tho? That's the other problem - it is very difficult to find any resources explaining what is the recommended procedure for rolling back versions, and a lot of it is outdated (pointing towards wsl --update --rollback).


[fix][flaky-test] Fix RawReaderTest.testFlowControlBatch

Fixes #17365
Modifications
The batchingMaxPublishDelay default value is 1ms, it is too short, so change the batchingMaxPublishDelay value to 1min to ensure no individual message was sent.
https://github.com/apache/pulsar/blob/c48a3243287c7d775459b6437d9f4b24ed44cf4c/pulsar-broker/src/test/java/org/apache/pulsar/client/impl/RawReaderTest.java#L295-L306
Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

@codelipenghui PTAL


darwinkit: fill out placeholder structs

We don't yet generate structs. Part of the reason for this is we need to match the data alignment of the structs they represent, and we have everything but what order the fields are. In order to get current frameworks to generate, I have created placeholder structs in the <framework>_structs.go file for frameworks that are empty of fields. Somebody needs to go and check out their actual full declaration in the framework header files and port their fields to Go fields. Perhaps in the process find a way to automate this.

done in https://github.com/progrium/macdriver/commit/9eee6811773651c1fd93b4df84137fbfa02bfab8


feat: add support for clipboard access

I want to test clipboard interaction and the test will fail because it requires user authorization.
I think it would be a nice feature to add for this plugin.
  it('copies word on click', () => {
    cy.get('.CopyButton').click();

    cy.window()
      .then(({ navigator }) => navigator.clipboard.readText())
      .then((text) => expect(text).to.match(/hello world/));

  });

Fails with NotAllowedError: Read permission denied.

Okay I did a bit of digging, it's not as simple as adding clipboard to the list - the permission is not listed in Chromium's profile.managed_default_content_settings.
Do you think there's another way to allow this?

I just added "clipboard": "allow" into cypress.json and it works. But I found also some other solution that may help somebody.
  "env": {
    "browserPermissions": {
      "clipboard": "allow"
    }
  }

Somehow I ended up here but the solution by @miso-belica didn't work or the one he linked. Instead, adding this before the clipboard invoking part worked https://stackoverflow.com/questions/60174546/how-grant-cypress-test-application-some-permissions

I just tried @miso-belica's { "clipboard": "allow" } solution and it worked like a charm -- I wonder if it's related to specific browser versions?
This is using

<EMAIL_ADDRESS><EMAIL_ADDRESS>
Running in Electron 106, Ubuntu linux using<EMAIL_ADDRESS>on ubuntu-latest runner (hoping this helps as a point of comparison for others who might have run into similar issues)

Very interesting, I'll need to see what I can do to add this natively. Thanks for all the info!


⚠ File access has degraded performance

In bf25373, File access (https://fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 5808 ms

Resolved: File access performance has improved in afb1176 after 8 minutes.


initial commit of README

Beginning to document the repository to help people, including possible developers, understand how the process of creating the Machine OS works.

/ok-to-test
/lgtm

/ok-to-test
/lgtm

/cherrypick release-4.6

/cherrypick release-4.6

/retest

/retest

/retest
/approve

/retest
/approve


Need to Show The Browse Button

the browse button get replaced with file name and 3 selected files type text , so can you provide me a way so that i keep visible that browse span tag visible all the time.

Currently there isn't a way to do this in file-upload-with-preview. You could add some text to the labels explaining to click the input to upload again.
Another option is forking the library and adjusting the code. But you'll have to decide how to show the name of the file and the browse button together. Should be fairly straightforward to use an ellipsis to cut of the file name.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
visithink, @charles-allen, @wdunicornpro, @LEAFERx, @yingmsky, @kathgironpe, @alan-w-255, @xchen437, @MantasMiksys, @lutfuahmet, @rrubo, @0-o-0, @medson10, @zhangsoledad, @pschfr, @yizhiru, @fzudust, @VirrageS, @morsamatias, @Jatin-8898


[Azure, AWS] Peer created VNET/VPC to another specified VNET

Add to cluster-info document information about VNET to which newly created VNET should peer.

@seriva this looks like point from list we were reviwing yesterday:

Virtual Network peering to SecOps cluster

Closing this for now as with the new security team this will probably look a bit different.


Changing required asterisks on register form to read "required" for b…

Changing required asterisks on register form to read "required" for better accessibility on screen readers. Also removed the "* Fields Required" since it was now redundant.

To keep the coloring and size difference on the (required) I just removed the aria-hidden. Let me know if this works!


Fetch a link from an existing Element

I'd like to get the self URL for the new created Element. According to the spec you should send a Location-Header with the new URL. If I parse the element I get the new self::URL in JSON. How to get the URL without parsing the json and extracting the Url.
While debugging I've seen the most methods wich would be helpfull are protected in the DocumentWriter. Is there a best practice solution?

I'd like to get the self URL for the new created Element.

This code might answer your question. It receives a newly created object, URL of the request to create one and a schemas.  The schemas is used to get a schema for the object and BaseSchema::getSelfLink is used to create the URL.
There are multuple ways to get the schemas. Encoder recieves them as an input (so when you create one you might remember it) or you can use getter though it's protected and you will need to do something with it.

Hi, I'm closing the issue because I've got a feeling that the question has been answered. Please don't hesitate to ask any further questions.


init的react脚手架，为什么刷新时会渲染两次

Describe the bug
代码：

输出结果：

可以看出加载时，打印了两次结果
Reproduction
none
System Info
win10、Chrome

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This kind of question is more suitable to ask on stackoverflow.


php artisan route:list error

PHP Fatal error:  Call to a member function make() on null in /Users/mymachine/Sites/connfa-integration-server/bootstrap/cache/compiled.php on line 5544
[Symfony\Component\Debug\Exception\FatalErrorException]
Call to a member function make() on null

@coldmachine
We found a similar problem https://github.com/laravel/framework/issues/10908
We are investigating this issue and we will fix it as soon as possible.


Add checking for dirty git tree after manifest and generate are run

This action should fail on it's first run now because there are modifications in the tree that aren't committed.

It failed. Now, I will add all the changes and this check should be passing.

Yay 🎉

Excellent, that KS for adding this 🎉

@phoban01 Thanks! :) Can you put a +1 on it, please? :)


simplify the way we get subdomain and url_prefix at blueprints

Here is an subtle improvement of simplifying the way we get subdomain and url_prefix at blueprints.py

Thanks for looking at this. We generally don't accept these types of PRs as they only shuffle code around rather than improving it significantly. Additionally, x if condition else y is preferred over the old or pattern.

Okay, thanks for your reply


UE5

this fork should fix the issue of not being able to compile the plugin for your project in UE5, if there is any issue just dm me on discord fluffy#4290 or open an issue

fixed smth rq

should now work in UE5


Move local sounder drive from key into the key handling loop to avoid…

… delay. Fix start-up issue with mkob class restructure and auto-connect.

My separate key & sounder interface is packed away, so I could only test this with key & virtual sounder (computer audio), but that works well, so I think the actual sounder will as well.
The other fix is for a bug that was introduced with the refactor/reorganization of all the MKOB modules into true classes. The problem occurs during start-up when the 'auto-connect' option is enabled.

That fixed the problem. Thanks, Ed!
~Les
On Wed, Dec 27, 2023 at 4:37 PM Ed Silky @.***> wrote:

Merged #291 https://github.com/MorseKOB/PyKOB/pull/291 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/MorseKOB/PyKOB/pull/291#event-11347774371, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APHISTJKVWGAWZD6OMSUWBLYLS5MNAVCNFSM6AAAAABBE4LOLCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGM2DONZXGQZTOMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Navigate through button

how to navigate through button to go next page in portia

+1

Need this feature for me too.  I want to fill a form and go to next page then scrape.

+1

+1


[android] Adds formatJava task for android project

Change-Id: If15efb1ad2a744b1bfadb7bf89a09a265a81082b
Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #1871 (e03be3b) into master (bb5073f) will decrease coverage by 2.36%.
The diff coverage is 64.69%.

@@             Coverage Diff              @@
##             master    #1871      +/-   ##
============================================
- Coverage     72.08%   69.72%   -2.37%
- Complexity     5126     5543     +417
============================================
  Files           473      527      +54
  Lines         21970    24474    +2504
  Branches       2351     2667     +316
============================================
+ Hits          15838    17065    +1227
- Misses         4925     6099    +1174
- Partials       1207     1310     +103

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

.../cv/translator/StyleTransferTranslatorFactory.java
40.00% <ø> (ø)

... and 422 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Website lacks many Security- and Privacy Features

Disclaimer: I am not sure if the website issues for https://www.coronawarn.app should be post here. So please feel free to forward this to whom is responsible for)
Though the website claims to be privacy-friendly, there are still these issues:

Unsafe TLS Encryption Configuration
TLS 1.3 is not enabled, TLS 1.0 is still active (!)
Please check Ciphers aswell.
CSP (Content Security Policy) is missing
"no-referrer" tag is missing. Referers are told to 3rd parties.
Several X-Headers (X-Content-Type, X-Frame, X-XSS-Protection) are missing

Please use common test suites to check website for security and privacy:

https://observatory.mozilla.org/
https://webbkoll.dataskydd.net/de
https://www.ssllabs.com/ssltest/

The following resource can´t be found
https://www.coronawarn.app/assets/css/ajax-loader.gif

@Julian-B90 I'll create a new issue from your last comment

Sorry that it took a while, but finally all mentioned issues should be fixed. See also the results on:

https://observatory.mozilla.org/analyze/coronawarn.app
https://webbkoll.dataskydd.net/de/results?url=http%3A%2F%2Fcoronawarn.app
https://www.ssllabs.com/ssltest/analyze.html?d=coronawarn.app
https://www.ssllabs.com/ssltest/analyze.html?d=www.coronawarn.app

Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team


Implement API for opportunity changes wallet password.

https://app.zeplin.io/project/5ba9be48f029d520c456f395/screen/5c0fd36b77c41daf9b95e562

checked


1000+ row search

It become a bit slow at first one or two letter search when I have a json data with more than ten thousand rows. (sorry not one thousand plus rows)
How can it make faster?

Past a certain point, you can't. If it needs to index and search a ton of documents, the time will increase with the more documents that are added.
I suggest taking a look at js-worker-search instead as a possible alternative. Less flexible API but should perform better.


[stable/prometheus-redis-exporter] Adding acondrat to prometheus-redis-exporter owners

Signed-off-by: Arcadie Condrat<EMAIL_ADDRESS>What this PR does / why we need it:
Adding an owners file
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[ ] Chart Version bumped - N/A
[ ] Variables are documented in the README.md - N/A
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @davidkarlsen

/lgtm
thanks!

/lgtm


Add general word-classes to crk.altlabel.tsv

In order to make the presentation of various labels based on explicit linguistic definitions rather than deduced by coding, I would add the general word class to the relabeling scheme in crk.altlabel.tsv.
Original	General	Short linguistic	Long linguistic	Plain English	Plain nêhiyawêwin	Emoji
TA	V	Transitive Animate		like: wîcihêw, itêw	tâpiskôc: wîcihêw, itêw	🧑🏽➡🧑🏽
VTA-1	V	VTA-1	transitive animate verb – class 1: regular	like: wîcihêw	tâpiskôc: wîcihêw	-

This likely means that the code that makes use of crk.altlabel.tsv might need to be revised accordingly. References to general word-classes can the be relabeled according to their own respective relabeling scheme based on user selection / default setting, so V --> Verb / Action word / ispayin-itwêwin, etc.

This should become redundant with how the new to-be-imported dictionary data is specified and created (in: https://github.com/UAlbertaALTLab/crk-db).

@dwhieb Isn't this redundant now?

@aarppe yes, closing this now.


Support for generic write-in characters directly in the text event

From Discord: "Request support for just generic write-in characters directly in the text event.
For when a character only have a temporary role, and don't need a character file. It gets messy when a character who only appears with two lines in one scene needs to be defined as a character."
should be real simple to implement, with simply surrounding it in special character that can't be part of a filename. Something like this:
*Temporary*: Hi, I'm just a guest here
Mostly just depends on which special character works best with the regex without rewriting it (if possible)

Do we need to specify it is temporary? what if we make it show on the name label as if it existed when it doesn't find the character associated to it? That way it would just be
Temporary: Hi, I'm just a guest here

for performance reasons its better to have an actual designiator IMO, so it knows ahead of time not to bother searching for the character because it's not going to exist

I wonder if people would want to also specify a color for a temporary character... And how that could look in the text format.

I would not go into more hypotheticals there, I think just a name should be fine 😅  if some of you want the feature for a game you are working on it can be implemented, otherwise it will have some technical debt associated with that that we shouldn't risk at the moment.
Recognizing non-character names even if it comes at a performance cost I think is worth it as a "fallback". But I would love to see how slow it is before trying to optimize for lookup speed.


Re-freezes Kerbalism 2.2.0 (hopefully)

lol i forgot to downgrade version file

The bot will keep re-indexing it as long as it's still on SpaceDock.

Yes but the versioning is based on KSP-AVC. Hmmm.

Oh wait... KSP-AVC is coming from SpaceDock. Hm.

I see, my freeze has not worked. @HebaruSan, wouldn't it make sense to tell the bot to skip frozen ckans too and don't create new ones?
@steamp0rt, I think the only way to solve this right now ist to freeze the .netkan too, to keep the bot from indexing.

Why is it still on SpaceDock?

... because i'm dumb and just realized i have to change the default version.

:man_facepalming:

i feel so embarassed

OK, I think whatever you changed has helped. Running netkan locally, I get 2.1.2 now, and the bot has already updated it, which should mean that 2.2.0 won't get regenerated. Makes sense to merge now.


refactor(styles): remove scss from LoadingBolt component

Description:
Refactor LoadingBolt component to use styled-components and remove old scss file.
Motivation and Context:
Finish porting app to styled-components.
How Has This Been Tested?
Manually - UI should look and work pretty much as before (minor style changes possible as moved to standardised components).
Types of changes:
Refactor
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage increased (+0.05%) to 19.151% when pulling 0757d69ff1aadbea15b3c90f2990c3bad7a7394c on mrfelton:refactor/scss-loadingbolt into 4bdf573480244f4eca00c6c7f5b42b02c4f8dae4 on LN-Zap:master.

Coverage increased (+0.05%) to 19.151% when pulling 0757d69ff1aadbea15b3c90f2990c3bad7a7394c on mrfelton:refactor/scss-loadingbolt into 4bdf573480244f4eca00c6c7f5b42b02c4f8dae4 on LN-Zap:master.

Coverage increased (+0.05%) to 19.151% when pulling 0757d69ff1aadbea15b3c90f2990c3bad7a7394c on mrfelton:refactor/scss-loadingbolt into 4bdf573480244f4eca00c6c7f5b42b02c4f8dae4 on LN-Zap:master.


'fontFace' not working for any profile after system reboot.

Environment

Microsoft Windows [Version 10.0.18362.418]
Windows Terminal (Preview) Version: 0.5.2762.0
Ubuntu subsystem with Oh My Zsh using the agnoster theme with Powerline Fonts installed.

Steps to reproduce

Setup Windows Terminal (With Ubuntu subsystem)
Set the fontFace for various profiles (Tested with powershell, cmd and Ubuntu WSL)
Note, that everything was working as intended up to here.
Restart Windows.

Expected behavior
The font setting being applied:

Actual behavior
After restarting Windows fontFace is being ignored:

Additional Info
My current profiles.json: profiles.txt

I assume, issue #3250 is the same issue. Just wanted to create a new issue since that one is lacking any information.

Which font specifically are you using? "Powerline Fonts" isn't terribly descriptive.
Does the font work in other applications after rebooting? Can you share a screenshot of it working in Windows Terminal before rebooting? Thanks.

Which font specifically are you using? "Powerline Fonts" isn't terribly descriptive.

Roboto Mono for Powerline at first, then Noto Mono for Powerline, DejaVu Sans Mono for Powerline and even with Courier New. It seems that every profile gets the fallback to the default font, no matter the fontFace-setting.

Does the font work in other applications after rebooting?

Outside of the terminal (Using the conhost setting dialogue the fonts work excluding Roboto Mono which is bugged atm (See #295, respectively google/fonts#1832), so that ones off the table for me).

Can you share a screenshot of it working in Windows Terminal before rebooting? Thanks.

Unfortunately I can't provide a screenshot from before restarting Windows. But I'll go check, if restarting Windows and/or reinstalling Terminal helps.

Ok, uninstalling and or restarting didn't work.
What 'seems' to do the trick was deleting all newly installed Powerline fonts and reinstall them:

To reinstall all of them I've cloned the repo https://github.com/powerline/fonts with GitHub Desktop and ran the install.ps1 in powershell (Used Set-ExecutionPolicy Bypass before running that script).
Next will be another restart or two to see, if it breaks again.

Ok, uninstalling and or restarting didn't work.
What 'seems' to do the trick was deleting all newly installed Powerline fonts and reinstall them:

To reinstall all of them I've cloned the repo https://github.com/powerline/fonts with GitHub Desktop and ran the install.ps1 in powershell (Used Set-ExecutionPolicy Bypass before running that script).
Next will be another restart or two to see, if it breaks again.

Ok, same issue after a restart. I've tried to install only a single set: DejaVu Sans Mono for Powerline. It worked, till I've restarted Windows.
In short: Reinstalling the font fixes it and restarting Windows breaks it again. Currently I'm too tired to hunt that issue down any more.

This is really fascinating! I wonder if it reproduces on v0.6? Sorry, I've never seen anything like this before, but we did make some font changes in that release.

Also, if you install the font "for all users" instead of just plain installing it, does it work after a reboot? This might be something fun for the font team to look at.

@DHowett-MSFT wrote:

This is really fascinating! I wonder if it reproduces on v0.6? Sorry, I've never seen anything like this before, but we did make some font changes in that release.

Yes, it reproduces with: Windows Terminal (Preview) Version: 0.6.2951.0

Also, if you install the font "for all users" instead of just plain installing it, does it work after a reboot? This might be something fun for the font team to look at.

Yes, this works fine after a reboot.
When installing it locally (What the install.ps1 actually does) it breaks after a restart, unless:

You disable the Windows FontCache Service or
You restart the Windows FontCache Service

Did a couple more restarts and installing and uninstalling of fonts, what I've found out so far:
I've installed Roboto Mono for Powerline locally and DejaVu Sans Mono for Powerline "for all users".
I've switched between those two a couple times and everytime I hit save I saw Roboto broken and DejaVu working thanks to Terminal immediately updating.
So the steps to reproduce should be:
Steps to reproduce

Setup oh-my-zsh with the agnoster theme (I guess oh-my-posh should show the same error).
Now install one Powerline-font locally (for example Roboto Mono for Powerline) aka mark all font files → Right click → Install.
Install one Powerline-font for all users (for example DejaVu Sans Mono for Powerline) aka mark all font files → Right click → Install for all users.
Make sure the Windows FontCache service is running at set to start automatically (should be the default).
Setup the profile to use the locally installed font and check, if it still works.
Restart Windows.
Restart Windows Terminal.
Switch the fonts in the config back and forth between those two.

Expected behaviour
Both fonts always work.
Actual behavior
The globally installed (aka for all users) font works fine, the locally installed font breaks/won't load unless you restart the FontCache or disable the service before restarting Windows.

Oh and btw: I wonder, if its the FontCache to blame or how Windows Terminal uses it. I guess a bit of both, since the issue with the FontCache described above only exists in the Terminal.

So, Terminal isn't doing anything special with the font cache. We're using DirectWrite, the text rendering API, like most other apps on the system are. Now, I've got one last crazy idea before I escalate this to the fonts team:
If you introduce a delay (say, 30-60 seconds) between steps 6 and 7 ... does it work for you?
Sorry to make you jump through so many hoops. ☹

I doubt it does since I usually do a bunch of other thing before restarting the terminal, but I can test it one last time. Gimme a min or ten.

Nope, that didn't WFM. I've waited ~10 mins between restarting Windows and (re)starting Windows Terminal.
Just for funsies I've recorded and cut a brief video, that shows the issue:

PS:

Sorry to make you jump through so many hoops. ☹

I've already planned to test with FontCache disabled 'n stuff. Thanks for the hint with the globally installed fonts, bzw.
And well: Through most of those hoops I've jumped voluntarily or otherwise it would leave me restless. 😉

:grin:
Thanks for testing for me. I was concerned that Terminal was starting up before the font cache was, and the font cache was somehow not simply a cache, but you've handily blown that theory away.
I'm wrapping this all up to escalate to the team who owns font handling on Windows, since it looks like we're doing "everything we can" here.

btw: I doubt, that its limited to powerline fonts. I assume, it would break with other locally installed fonts as well, but the extra glyphs of those fonts make it easier to visualize the issue.

Just appeared in 0.7 for me (was ok in 0.6)
I first updated Win10 to 10933.1 Fast Insider (i think i had 19030 before)
Then updated the terminal from the store
font installed Delugia Nerd Font Complete,
https://github.com/adam7/delugia-code/releases/download/v1911.21/Delugia.Nerd.Font.Complete.ttf
https://github.com/adam7/delugia-code/releases/tag/v1911.21
When i boot i have to manually install the font even if the system tells me that it's already installed (i attempted cancellation, won't work)
That until Cascadia Code renders PowerLines font (OmyZsh + OMyPosh)
For now Cascadia Coder render empty squares ;)

When i do boot up Deluagia does not render the same "thing" it looks like a Square but not empty, there's like 4 character within

The re-installing the font and restarting the terminal fixes it until the next reboot.
Worth noting I believe the font also doesn't appear in the Windows 10 Fonts when doing a search until you re-install.
Running Terminal version 0.9.433.0. Font: Meslo LGM.

@suavelizard
one of the workaround is to install the font System Wide and not for the user only.
It should properly persist then
(this is not optimal but it has fixed this behavior for me)

Yes, installing font system wide fixes the issue.

I came across this issue while experiencing these exact same symptoms. I was using the Nerd Font patched Cascadia Code font (Delugia Nerd Font) and after every reboot the font face would not be recognized in VSCode (I had not opened Windows Terminal to see if it was the same but I suspect it would be).
If I went to Start -> Settings -> Personalisation -> Fonts and searched for the font it would not show in the list, but if I went to C:\Windows\Fonts\ I would see the font file there. I will see how it goes after performing a full reboot now that I have installed the font  system wide.

I was using the Nerd Font patched Cascadia Code font (Delugia Nerd Font) and after every reboot the font face would not be recognized in VSCode (I had not opened Windows Terminal to see if it was the same but I suspect it would be).

Good to know, that this is not a Terminal only issue. But I'm still curious, if this is 100% a font cache issue or if this can be fixed in Windows Terminal.

I have fonts installed (for all) in C:\Windows\Fonts directory, still Windows Terminal can't use it. Other system fonts can be recognized by the WT though.
Using WT Preview from MS store.

I had same issue and here is the fix. At least works for me.
Install font by right click it and select "install for all users".
More details with findings:

C:\Windows\Fonts folder from explorer does not access "real" file system. It contains both system and user fonts
System fonts are installed in Windows\Fonts folder, and it shows in [HKEY_LOCAL_MACHINE\Software\Microsoft\Windows NT\CurrentVersion\Fonts]
User fonts are installed in AppData\Local\Microsoft\Windows\Fonts\ as I can see. It shows in [HKEY_CURRENT_USER\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Fonts]
Double click will install font into user's font folder.

However, still not idea why font installed in local user folder stopped working after restarting.
Hope this useful.

Hi.I install fonts using "install for all users" option after right clicking the fonts. It shows up in regedit, but can’t use it at places such as VS Code, MS Terminal etc. I opted in for alacritty terminal. And I patched fonts myself. And I installed it using “all users” menu and it is working. Regards,Aasutosh Jha From: JTSent: Friday, July 24, 2020 10:35 PMTo: microsoft/terminalCc: Aasutosh Jha; CommentSubject: Re: [microsoft/terminal] 'fontFace' not working for any profile after system reboot. (#3257) I had same issue and here is the fix. At least works for me.Install font by right click it and select "install for all users".More details with findings:C:\Windows\Fonts folder from explorer does not access "real" file system. It contains both system and user fontsSystem fonts are installed in Windows\Fonts folder, and it shows in [HKEY_LOCAL_MACHINE\Software\Microsoft\Windows NT\CurrentVersion\Fonts]User fonts are installed in AppData\Local\Microsoft\Windows\Fonts\ as I can see. It shows in [HKEY_CURRENT_USER\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Fonts]Double click will install font into user's font folder.However, still not idea why font installed in local user folder stopped working after restarting.Hope this useful.—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.

@aasutossh your message was incomprehensible because of the email formatting, so I had to remove it.

What I previously said was, I installed the fonts using the right click option "install for all users", and they were present in the regedit too. But I still couldn't use them (they were nerd fonts).
So I patched the fonts myself and now I can use it. I stopped using ms terminal, I use alacritty now.

OMG! 😱😱
It was not just me!
I have the exact same issue than @Riiskyy and @tebeco
But I use WSL on VSCode
After reboot I always have to reinstall my fonts (MesloLGS NF and CaskaydiaCove NF) to work in Windows Terminal/VScode Terminal.
If not, they behave as if the fonts have not been installed.
PS: I will test Font System Wide

I don't believe it..
It really fixes the problem when i right-click the .ttf and "Install for all users"
Simple as that, after reboot all terminals looking fine with my OMZ powerlevel10k in Windows Terminal or VScode

This is for anyone else who Googles up this issue. Using "Install for all users" was the solution for me as well.
However, go to Start -> Settings -> Personalisation -> Fonts and search for the font. The name there is the one you want to use. As for me Ubuntu Mono Nerd Font Complete Mono Windows Compatible did not work, but in the Fonts directory it shows as UbuntuMono NF and this has correctly worked between reboots and shutdowns.
Cheers @luizwhite @Riiskyy @tebeco big ups. I almost resorted to going to page 2 on Google.

Holy crap... 1 year have been passed for the issue. I can't set the Cascadia Mono ExtraLight font face. Only can set default Cascadia Mono.
But in cmd.exe it works well....
This project is a mess....

I experienced this same issue as well. It's especially confusing because the fonts are visible in Word and work fine there, but putting the same font name in Terminal appears to not work. "Install for all users" did resolve the problem for me as well.

I experienced this same issue as well. It's especially confusing because the fonts are visible in Word and work fine there, but putting the same font name in Terminal appears to not work. "Install for all users" did resolve the problem for me as well.

This is for anyone else who Googles up this issue. Using "Install for all users" was the solution for me as well.
However, go to Start -> Settings -> Personalisation -> Fonts and search for the font. The name there is the one you want to use. As for me Ubuntu Mono Nerd Font Complete Mono Windows Compatible did not work, but in the Fonts directory it shows as UbuntuMono NF and this has correctly worked between reboots and shutdowns.
Cheers @luizwhite @Riiskyy @tebeco big ups. I almost resorted to going to page 2 on Google.

Oops!It works! "go to Start -> Settings -> Personalisation -> Fonts and search for the font " which can get the real name,so the icons won't just be the "□"

Hi, I'm trying to reproduce this issue to see if I can figure out why the fonts are disappearing, and hopefully fix the issue. So far, I haven't been able to get it to happen. I tried cloning the https://github.com/powerline/fonts repo and running the ./install.ps1 script at @Stadler76 described above. For me, the fonts are all installed successfully and remain installed after rebooting. They also remain installed after installing Terminal Preview and rebooting.
Speculating as to what might be happening, I can think of two possibilities. First, let me say that installing a font file basically should do three things:

Copy the font file to a fonts folder
Add the font file to the current Windows session
Add a registry value that references the file

I suspect that either registry values are not being added in the first place, or they're being added and later removed (e.g., on reboot). Either one would explain the observed behavior. The font initially works because it's been added to the current session, but no longer appears after reboot because it's not in the registry.
I'm wondering if someone who can reproduce this would be willing to help me narrow it down by doing an experiment. Here is what I would like you to try if you are willing:

Install the font or fonts the same way as before
Check if registry values exist for the fonts (see below)
Check if the font(s) appear in Settings / Personalization / Fonts
Reboot
Repeat steps 2-3

The registry values for per-user fonts are under "HKCU\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Fonts". You can export the key to a file using the following command:
reg.exe export "HKCU\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Fonts" fonts.reg

Where fonts.reg is the output file and can be any file name you like. E.g., it could be after-install.reg the first time you run the command after after-reboot the second time.
Thanks,
--Nick

@niklasb-ms: Are you by any chance testing this on Windows 11? Because I narrowed it down to be an issue with Windows Font Cache. As you see in the video I recorded, it can be resolved by restarting the Font Cache or by disabling it.

I Changed the font in Windows Terminal > Settings > Default > Appearance > Font face, and i saw it concurrently change the "fontFace" object in settings.json to a "font": {"face": "{YOUR_FONT_HERE}"} object, I think the wsl2 has changed the code a bit resulting the fontFace to be deprecated.

Pardon my ignorance, because this is still an issue for me. How do I install a font system wide? I installed the fonts to C:\Windows\Fonts but after rebooting, Windows Terminal says it can no longer find the fonts. I did not have this issue before installed 2H22.

Pardon my ignorance, because this is still an issue for me. How do I install a font system wide?

Right click -> Install for all users

I'm wondering if someone who can reproduce this would be willing to help me narrow it down by doing an experiment. Here is what I would like you to try if you are willing:

Install the font or fonts the same way as before
Check if registry values exist for the fonts (see below)
Check if the font(s) appear in Settings / Personalization / Fonts
Reboot
Repeat steps 2-3

@niklasb-ms
This issue never affected me until W11 Update 22H2. Now it happens every time I reboot.

I clone https://github.com/ryanoasis/nerd-fonts.
Fonts show here.
Reboot

2.a. Registry values exist. I ran a diff on both output files and they are exactly the same.
2.b. Note also that font files exist in "C:\Users\simsrw73\AppData\Local\Microsoft\Windows\Fonts", both before and after reboot.
3. Font does not appear in Settings.
Reinstalling the fonts corrects the issue until next reboot. And as reported above by @Stadler76, stopping the Windows Font Cache Service and restarting Terminal corrects the issue until next reboot.
This issue is driving me crazy. I will run any diagnostic you need, incl allowing remote access to see this resolved.
Output of the two commands after clean install of fonts (manually deleted all previous references to the font in registry and font files) and after reboot:

reg.exe export "HKCU\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Fonts" <file>
fd --glob *meslo* C:\Users\simsrw73\AppData\Local\Microsoft\Windows\Fonts > <file>

In both commands, output of before & after reboot are identical, but fonts no longer show in settings and are not available to use until reinstalled or font cache service is restarted.
font-listings.zip

Writing this here as well, since I accidentall wrote my reply in the closed #12587 issue, which looks like a duplicate.
For me it also started after the Window 11 22H2 update. It seems like it looses all user installed fonts.
The old control panel still shows the fonts, but they are missing from the new set settings Personalization/Fonts and Windows programs.
Like above, restarting the Windows Font Cache Service, seems to make it work again.

Could anyone find any workaround yet? I'm having the same issue. I have Windows 10 Enterprise. My windows version is 22H2 tried installing powerline fonts from the repo above as the author did and exported the registry. I can see all the fonts are installed correctly everywhere, but the issue persists.

Could anyone find any workaround yet?

They don't seem to have any interest at all in fixing the issue. The only workaround is to restart the FontCache service. I setup task scheduler to do this at startup

Run taskschd
Action menu -> Create Basic Task
Name = Restart Font Cache
Next
Run task "When I log on"
Next
Action to perform = Start a program
Next
Program = powershell
Add arguments = -WindowStyle Hidden -Command "Restart-Service -Name 'FontCache'"
Next
Check "Open the Properties dialog..."
Finish
On General tab, under Security options, check "Run with highest privileges"
On the Triggers tab, Edit the "At log on" trigger, and change it to "Any user"
Click OK

Apparently, reinstalling Windows also fixes this. Just upgraded my GPU from Nvidia to AMD and Windows started crashing periodically. Even after using GPU driver removal tool and reinstalling AMD, so I tried reinstalling Windows. Both issues seem to be fixed. No crashing and No font problems in wt. No idea what causes the issue but, in my case, reinstalling Windows seems have resolved it. That is a horrible solution. I wish someone could have helped debug the cause of the issue.

Currently in the camp where none of the proposed workarounds remedy the issue for me. Anyone manage to resolve this issue with any of the Iosevka Nerd Fonts in a way not listed above?

+1 Having this w/ multiple fonts. Victor Mono as of today.

I seem to have fixed this issue on Windows 11 latest updates as of 7/4/2023:
windows key -> services -> services (local) -> Windows Font Cache Service
In this window, select tab "Log On", and use "Local System Account", also check "Allow service to interact ..."
Click "Ok"
Then stop and start service

Okay so housecleaning time. Internally, this is tracked by MSFT:42103854.
After the last couple PRs we did in this area, we're pretty confident there's nothing more that we (the Terminal team) can do on our side here to mitigate this.
This is pretty clearly still an issue. I'm bumping the internal thread to see how we can get this fixed in the OS.
I've decided that even though there's nothing more we (the Terminal team) can do to fix this on our side, I'm going to leave this one open. Folks have a hard enough time searching for existing open issues, let alone closed ones. This is super painful when it does hit, so I think it warrants being left open regardless of if there's any work we can do to fix it.
I have no idea when this'll be fixed, but I'll make sure to share any relevant internal updates here in this thread.

I seem to have fixed this issue on Windows 11 latest updates as of 7/4/2023:
windows key -> services -> services (local) -> Windows Font Cache Service
In this window, select tab "Log On", and use "Local System Account", also check "Allow service to interact ..."
Click "Ok"
Then stop and start service

This worked for me too! after months of issues with the nerd fonts in Terminal and VSCode. Thank you!

I seem to have fixed this issue on Windows 11 latest updates as of 7/4/2023:
windows key -> services -> services (local) -> Windows Font Cache Service
In this window, select tab "Log On", and use "Local System Account", also check "Allow service to interact ..."
Click "Ok"
Then stop and start service

Workaround confirming to still work as of today (10/23/23)

I seem to have fixed this issue on Windows 11 latest updates as of 7/4/2023:
windows key -> services -> services (local) -> Windows Font Cache Service
In this window, select tab "Log On", and use "Local System Account", also check "Allow service to interact ..."
Click "Ok"
Then stop and start service

Yoppe seems to work.

work for me same as of today (24-Oct-2023) with windows 11 latest version

Please use upvotes/reactions if you agree with a post. A lot of people are subscribed to this thread and each confirmation pings everyone every time, so please avoid doing that. I'm collapsing the above comments as off-topic.

Pardon my ignorance, because this is still an issue for me. How do I install a font system wide?

Right click -> Install for all users

sorry...right click on what?

sorry...right click on what?
On the font you want to install?

Can confirm I am still having this issue with fonts in Windows Terminal on Windows 11 Pro.

This is driving me insane.

This bug is super annoying! Please fix it.

This superuser answer worked for me on windows 11

This superuser answer worked for me on windows 11

Interesting. In my case, even if I reinstall the fonts in Settings -> Personalization -> Fonts, I still have the error when I try to use the font in the terminal.
It did seem to work however after I restarted the Windows Font Cache Service (Win-R -> Open: services.msc -> Windows Font Cache Service -> right click -> Restart). After rebooting, I reinstalled the Nerd fonts in Settings -> Personalization -> Fonts, and at that point I was able to set the fonts in my terminal. Sadly, the Fonts vanished again at the next reboot.

This superuser answer worked for me on windows 11

Interesting. In my case, even if I reinstall the fonts in Settings -> Personalization -> Fonts, I still have the error when I try to use the font in the terminal.
What did work for me was to remove the fonts, clear the Windows Font Cache Service, and reinstall the fonts for all:

in Control Panel -> Appearance and Personalization -> Fonts, select the Nerd Fonts, Click "Delete" and confirm
Win-R -> Open: services.msc -> Windows Font Cache Service -> right click -> Restart
In the file explorer, select the fonts -> Right click -> Show More Options -> Install for all Users (requires admin privileges)

I verified that after rebooting or shutting down, all the nerd fonts are still available in Settings -> Personalization -> Fonts, and correctly set in my Terminal.

This works for me. Thanks!

I was able to resolve this issue by installing the font for all users instead of a install, even after a restart I had no problems. It appear to me that the bug has something to do with how Windows 11 handles the location of  user installed fonts. Hope this helps!


Eagerly computing results limits potential for optimization

Problem
By eagerly computing, e.g., when calling .head(), we limit the potential for optimizations. For example, it shouldn't matter whether I call df.head(n)[["a", "b"]] or df[["a", "b"]].head(n). With eager computations, we lose the opportunity to push the column projection before the head selection.
Apart from that, in my opinion that eager computation comes as a surprise and makes it harder to argue about when Dask actually computes things.
Proposed solution(s)
In general, limit eager computation as much as possible. For this particular example: Deprecate compute=True as default and switch to compute=False in the future.

With "eagerly computing" you are referring to stuff like head, tail, Len, ...., correct? Just to avoid confusion with the set_index space

IMHO it depends.
head() is meant as a quick and dirty development tool. I don't see a problem with it being eager.
An eager len() is, on the other hand, catastrophic as there's a strong user expectation for it to be always trivial.

Why would head be considered a "quick and dirty" development tool? IMO, there are plenty of use cases where I just want to compute the top K of something.
Honestly, I don't care too much about len in the context of optimization because I don't see any use cases where it would prohibit further optimization even if it's executed eagerly.

I guess that's just my coding habits. Yes, if head() is used in production code it should definitely be lazy by default.


Consider Log4Cats for Logging

It allows us to abstract out of the logging game entirely. We could have people provide loggers, and we could just use it throughout.
Its relative immaturity holds me back from putting it out there strongly.
However I was wondering if people had opinions. As it would get us out of telling people what they are logging with.

The external liability of a dependency is API churn times adoption.  typesafelogging was high in both factors and got dumped.  log4s has low churn which has made increasing adoption painless (and contributed to increasing adoption!).  log4cats doesn't have the same track record, though we are generally fond of the author's work and one of us has his home address. :wink:
The external upside of a dependency is its use in public API times adoption.  Since it doesn't appear in API, the external upside of any choice is zero.
The internal upside of a dependency is that it makes the code more maintainable, and the external upside to that is more maintainable code lets us focus on HTTP instead of logging.  log4cats scores highly here if we commit to pure logging and is an obstacle if we don't.
So:

It should come with a backend that doesn't require log4s or similar, or the liability is additive.
Consider that blaze uses log4s.  The liability may be additive for a majority of our users anyway.
The API needs to be rock solid, or else you need to go out of your way to scare off other adopters.
It requires every logging statement in an effect.  It's the FP thing to do, but not yet universal in our code.  Are we sure there are no places we want to opt out of that behavior?

I think we should not move on this today, but very strongly consider this as it matures, if it has a good story for the above.

I'm pro log4cats in a lot of places that aren't performance sensitive (so basically everywhere outside of blaze).
I'm neutral on this. Logging is one of the few things I don't really care at all about side effecting much (It's almost never a side effect that bites you from my experience).

So, ember uses log4cats, and is the one that will stay in this repository.  Blaze pretty much has to use log4s, but will be moving out of this repository.
I'd prefer to have just one in this repository, so it makes sense to me to replace log4s with log4cats outside of blaze.

The only compelling case for a logger in core is in multipart, if we can't clean up the temp files.  It would be nice to eliminate that entirely.

👍 log4s has shimmed the slf4j package on Scala.js which is dubious.


Autoregressivity

I had a question about Figure 2 and equation 3 from the paper. How does the last token of each chunk C_u being able to attend to the retrieved content E_u not break autoregressivity?

so basically you have to make sure past tokens never see a future token. basically the last token is the most far future token, so it can safely attend to all of E_u without violating that rule

@sdpmas the same trick was actually used here https://arxiv.org/abs/2110.13711 (i think deepmind probably read this paper and got some inspiration tbh)

I see, thanks a lot for the explanation!


Don't construct an AnnotationExcluder unnecessarily

The creation of objects would a be waste of memory.
The changes made comply with the implementation in other rules.

Closed since the one scenario can't be changed that easily, as noted in the comments. The rest of the changes only concern the code arrangement.


Cli crashes when no name is provided

HI, nice tool ! Thanks for sharing it.
I've noticed a small bug while using the cli:
If I don't provide a chart name, it crashes without providing much info:
> candlestick-chart -m json-file -f candles.json
Traceback (most recent call last):
  File "/home/jose/charts/.venv/bin/candlestick-chart", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/home/jose/charts/.venv/lib/python3.11/site-packages/candlestick_chart/__main__.py", line 59, in main
    chart.draw()
  File "/home/jose/charts/.venv/lib/python3.11/site-packages/candlestick_chart/chart.py", line 57, in draw
    print(self._render())
          ^^^^^^^^^^^^^^
  File "/home/jose/charts/.venv/lib/python3.11/site-packages/candlestick_chart/chart.py", line 53, in _render
    return self.renderer.render(self)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jose/charts/.venv/lib/python3.11/site-packages/candlestick_chart/chart_renderer.py", line 104, in render
    output.append(chart.info_bar.render(chart_data.main_candle_set, chart_data.width))
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jose/charts/.venv/lib/python3.11/site-packages/candlestick_chart/info_bar.py", line 60, in render
    " | ".join(
TypeError: object of type 'NoneType' has no len()
>

It works if a name is provided:
> candlestick-chart -m json-file -f candles.json --chart-name __NAME__

and even with an empty name:
> candlestick-chart -m json-file -f candles.json --chart-name ''

It looks like the default value for the argument should be a string, changing this line seems to fix it
https://github.com/BoboTiG/py-candlestick-chart/blob/e6a2a0bb8709830ae35f66675fbd2312cf8cc07a/src/candlestick_chart/__main__.py#L33
    parser.add_argument("--chart-name", help="Sets the chart name.", default="")

I'm using the latest version from PyPI:
> candlestick-chart --version
candlestick-chart 2.6.0

Thanks for the report @JoseKilo :)
Would you like to open a PR?

I've some free time, I will work on it, and add tests at the same time.

v2.7.0 released with the fix 🎉

Amazing ! Thanks a lot !!


60fps limit?

I'm seeing my fps limited to 60fps during the encode.flush(device) step (with the GL backend); is there a way to remove that restriction?

Have you disabled vsync on window creation?
(gfx's Application enables it)

I'm not using gfx::Application (using SDL2 instead).

That is, I don't explicitly enable it; it is probably implicit somewhere, but I don't see a place to configure it.

This is probably what you are looking for: https://wiki.libsdl.org/SDL_GL_SetSwapInterval

It is already set to 0 and if I comment the encoder.flush(device) line, it is no longer limited to 60fps, (even with no draw commands inserted into it).

How many FPS are you getting if you don't call encoder.flush doesn't matter, since you get nothing on screen. gfx-rs core/render modules are completely unaware of your swap chain, which is responsible for vsync, and which is controlled by SDL only.
Care to show the source?

If I comment it out, I get either 0 or 1ms per loop (SDL's timer resolution).
Yeah, I'll upload the source tonight.

Hmm. I guess it could be optimizing out the swap if no commands were given…

Is this still a problem? Please re-open if true.

Yeah, I'm still seeing this. The code is here and the loop is in this directory, mainloop.rs and video.rs for the flush calls.

It appears I do not have permission to reopen.

I got the game running after a few tweaks. Where do you see the FPS?

Ah, sorry. Just pushed by FPS output hunk to this branch.

Out of curiosity, what tweaks were required?

Ah, found the missing error_chain import.

I had to tweak the path to utils as well as the external crate define for error_chain.
I can repro now and see that I'm also limited to 60 fps. Here is what I see in the capture:
glXSwapIntervalMESA(interval = 0) = 0, meaning the successful return. Although, the spec says it's not actually guaranteed to disable vsync:

glXSwapIntervalMESA specifies the minimum number of video frame periods per buffer swap.

So basically the implementation is free to ignore value 0 and pretend that nothing happened...
The whole issue seems to be SDL/driver related, since we do issue the proper calls (as indicated in the capture).

OK, thanks for the info. I'll try and see if I can't try it out on another machine with a different video card.

@mathstuf if you're running on linux try running the application as: vblank_mode=0 target/.../yourapp vblank_mode=0 environment variable disables the vsync for me.

Ah, thanks, I'll use that.
I see I never said my original goal: I'm porting a game from D to Rust and the D code runs at 90+ fps. I'm interested in the relative speed of the Rust code (and the better pipelined OpenGL code too) and 60 fps lock kind of defeats the purpose there :) . The workaround is sufficient for me.


Fixing local Fusion object when retrieving from local storage

tests corrected and docs improved. Please, review again.


Build interfaces around our postgres access

We should make sure that users could bring their own SQL databases into Armada.  Currently we only allow postgres as our db but we should create an interface and potentially allow other databases.
This could help with test improvement as we can add mocks for SQL calls and allow us to run unit tests without an active database.
┆Issue is synchronized with this Jira Task by Unito

Which other databases other than postgres are we expecting Armada to deal with?

At the moment our main user will be using Postgres.
Generally our goal with this is to allow other relation databases so some that come to mind are MSSQL (Microsoft SQL Server), SQLLite, MySQL and Postgres.
I don't think we should directly support these in Armada until there is a request to support them.  But what I would like to do is to start changing the code so that we could plug and match other databases.

I am interested too in this sir

https://developers.google.com/open-source/gsoc/timeline
I suggest getting familiar with our project.
I've created some other issues for people to play around with.  We are excited to have this much interest but I ask that you don't take this project unless you are officially allowed to do so via GSOC application.

I would say abstracting as necessary so that SQLite or Postgres can work interchangeably is a good concrete starting point, as we could then use SQLite for running unit tests instead of requiring Postgres. The existing tests with SQLite db provider would provide an indicator of success.

We're using postgres-specific SQL calls for performance (specifically, the postygres-specific copy wire protocol).
In my opinion, adding interfaces around SQL calls would be counter-productive for this reason.
I.e., I don't think we should do it.

@severinson what's your suggestion for how to solve the underlying issue (we need PG to run unit tests)?  :)

I think there are two benefits -- less external dependency for unit tests and more openness in the system which might aid adoption. As long as we keep the postgresy performance stuff in that provider, I don't think it's counter-productive.

Does this mean this is no longer a gsoc project?

No, this project is still valid, please proceed!
On Fri, Feb 24, 2023 at 12:57 PM Maureen Ononiwu @.***>
wrote:

Does this mean this is no longer a gsoc project?
—
Reply to this email directly, view it on GitHub
https://github.com/armadaproject/armada/issues/2121#issuecomment-1444145204,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEO3Y3W6UW77XUQXF6KIL3WZDZAZANCNFSM6AAAAAAU4DDELQ
.
You are receiving this because you commented.Message ID:
@.***>

Is this project valid then can I proceed to learn more about it?

Hello @kannon92 @suprjinx ,
I work to work on this project.
Where can I discuss about the same?

Is this project going to be a part of LFX summer this year?

Is this project going to be a part of LFX summer this year?

Yes. https://github.com/cncf/mentoring/blob/main/programs/lfx-mentorship/2023/02-Jun-Aug/project_ideas.md#armada
Applications open up on May 10th.

Hey @kannon92. I want to work on this project under LFX summer mentorship. I am a new contributor to this project. How should I get started? Is there any good-first bug that needs attention, I would be happy to help.

Hello everyone, I am also interested in this issue and would like to join this as LFX mentorship. Can you help me to get started with this project and issue?

Yeah sure @i-am-yuvi. what kind of help do you need?

Hey @satyampsoni. If you are already working into this issue. Will you mind if you could share some resources to get started with this project.

First of all, I would suggest you understand what is Armada and the problem it is trying to solve, You can read the readme it explains it well and talks about the resources are mentioned in the readme and I am mentioning some links here too.

https://www.youtube.com/watch?v=FT8pXYciD9A
https://www.youtube.com/watch?v=B3WPxw3OUl4

I am also looking forward to working on it during LFX and am also researching more about it. I'll be sharing the resources in the Slack channel as I progress with the issue, I just came to know about this issue last day.
Again pls join the Slack channel people there will definitely help you.

Thanks for this @satyampsoni, I'll definitely join the slack channel

Hey, I would love to work on this project. I am familiar with Golang and have a good understanding of MySQL and SQL. I am also highly interested in applying for the LFX mentorship program during the months of June and August.

I am also interested in this can somebody paste the slack channel link here.

I am interested in this as I have knowledge of Go and SQL databases.
How can I go from here as I would like to join and also apply for the LFX mentorship program

I think this is the link to our Slack -- https://cloud-native.slack.com/archives/C03T9CBCEMC. If that doesn't work, find #armada channel once signed in.

/assign


Passing absolute path to output fails

Regarding the "output" argument:

Path processing does not play nice with Windows paths.
Path processing assumes path is relative to current working directory. Passing an absolute path fails.

Path processing should use the python 3 pathlib library to become OS independent.
https://docs.python.org/3/library/pathlib.html
Passing an absolute path on windows causes issues rooted at the following lines:
HTMLTestRunner/runner.py:319
        self.html_report_file_name = f'./{self.output}/{self.report_name}_{time.strftime("%d-%m-%y %I-%M-%S")}.html'
        os.makedirs(os.path.dirname(self.html_report_file_name), exist_ok=True)
        self.log_file = ''
        if log:
            self.log_file = f'./{self.output}/{self.report_name}_{time.strftime("%d-%m-%y %I-%M-%S")}_log.txt'

Okay. I will check

@jalapenopuzzle
Thanks for notify the issue.
Issue resolved


Internal Server Error when opening any Sales/Build or Purchase order

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
Hi Team,
after updating my ubuntu machin  am now at rev 0.12.0. great job addinthe adres field option thank you so much.
but unfortunatly i am now running into an isseu when i try to Build order / Sales Ordr or Purchase orders.
i can see all my orders but when clicking an order i get an error screen.

log file attached below
Steps to Reproduce

Goto sales order
click on nay sales order
i get the error

Expected behaviour
not this
Deployment Method

[ ] Docker
[X] Bare metal

Version Information
Version Information:
InvenTree-Version: 0.12.0
Django Version: 3.2.19
Commit Hash: 1d18b487
Commit Date: None
Commit Branch: master
stable
Database: sqlite3
Debug-Mode: False
Deployed using Docker: False
Platform: Linux-5.15.0-75-generic-x86_64-with-glibc2.29
Installer: PKG
Target: ubuntu:20.04
Active plugins: [{'name': 'InvenTreeBarcode', 'slug': 'inventreebarcode', 'version': '2.0.0'}, {'name': 'InvenTreeCoreNotificationsPlugin', 'slug': 'inventreecorenotificationsplugin', 'version': '1.0.0'}]
Please verify if you can reproduce this bug on the demo site.

[ ] I can reproduce this bug on the demo site.

Relevant log output
Change Error
Error object (138)
[HISTORY](http://<IP_ADDRESS>/admin/error_report/error/138/history/)

Type:
AttributeError
Path:
http://<IP_ADDRESS>/order/sales-order/10/
Info:
'str' object has no attribute 'removesuffix'
When:
July 2, 2023, 6:34 p.m.
Data:
Traceback (most recent call last):

File "/opt/inventree/env/lib/python3.8/site-packages/django/core/handlers/base.py", line 204, in _get_response
response = response.render()

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/response.py", line 105, in render
self.content = self.rendered_content

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/response.py", line 83, in rendered_content
return template.render(context, self._request)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/backends/django.py", line 61, in render
return self.template.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 170, in render
return self._render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 162, in _render
return self.nodelist.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/loader_tags.py", line 150, in render
return compiled_parent._render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 162, in _render
return self.nodelist.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/loader_tags.py", line 150, in render
return compiled_parent._render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 162, in _render
return self.nodelist.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/loader_tags.py", line 150, in render
return compiled_parent._render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 162, in _render
return self.nodelist.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/loader_tags.py", line 62, in render
result = block.nodelist.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/loader_tags.py", line 62, in render
result = block.nodelist.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/loader_tags.py", line 62, in render
result = block.nodelist.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/loader_tags.py", line 62, in render
result = block.nodelist.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 938, in render
bit = node.render_annotated(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/base.py", line 905, in render_annotated
return self.render(context)

File "/opt/inventree/env/lib/python3.8/site-packages/django/template/library.py", line 192, in render
output = self.func(*resolved_args, **resolved_kwargs)

File "/opt/inventree/InvenTree/generic/states/tags.py", line 14, in status_label
state = {cls.tag(): cls for cls in inheritors(StatusCode)}.get(typ, None)

File "/opt/inventree/InvenTree/generic/states/tags.py", line 14, in <dictcomp>
state = {cls.tag(): cls for cls in inheritors(StatusCode)}.get(typ, None)

File "/opt/inventree/InvenTree/generic/states/states.py", line 108, in tag
ref_name = cls.__name__.removesuffix('Status')

AttributeError: 'str' object has no attribute 'removesuffix'

Hello @mabroens ,
the str.removesuffix function is only available in python 3.9+ which is also the min version you need to run InvenTree. From your log I can see you run python 3.8, so you need to update your python version to fix that issue.

Hello @wolflu05 ,
thanks for the hint.
I have installed pyhthon 3.9 and alsa ran as python was still set for 3.8
sudo apt install python-is-python3
then restarted my ubuntu.
but now it seems like nothing is working, i now get 502 Bad Gateway
will try more tomorrow but if you have any hints please.

@mabroens you'll want to look at the server logs to determine what is going on. No doubt updating your python version means that all your library references are broken

@SchrodingersGat ,
thank you for the feedback, but sorry to ask where can i find 'the server logs' you are refering to? do you mean logfiles from ubuntuu or from inventree?
~/inventree$ ls -al
total 348
drwxr-xr-x 13 root     root       4096 apr  7 09:08 .
drwxr-xr-x 23 spectrum spectrum   4096 jul  3 10:49 ..
drwxr-xr-x  2 root     root       4096 apr  7 09:08 ci
drwxr-xr-x  4 root     root       4096 apr  7 09:08 contrib
-rw-r--r--  1 root     root       8071 apr  7 09:08 CONTRIBUTING.md
-rw-r--r--  1 root     root        257 apr  7 09:08 crowdin.yml
drwxr-xr-x  2 root     root       4096 apr  7 09:08 deploy
drwxr-xr-x  2 root     root       4096 apr  7 09:08 .devcontainer
drwxr-xr-x  2 root     root       4096 apr  7 09:08 .devops
drwxr-xr-x  3 root     root       4096 apr  7 09:08 docker
-rw-r--r--  1 root     root       2472 apr  7 09:08 docker-compose.yml
-rw-r--r--  1 root     root        553 apr  7 09:08 docker.dev.env
-rw-r--r--  1 root     root       4664 apr  7 09:08 Dockerfile
-rw-r--r--  1 root     root        404 apr  7 09:08 .eslintrc.yml
drwxr-xr-x  8 root     root       4096 apr  7 09:08 .git
-rw-r--r--  1 root     root        118 apr  7 09:08 .gitattributes
drwxr-xr-x  5 root     root       4096 apr  7 09:08 .github
-rw-r--r--  1 root     root       1397 apr  7 09:08 .gitignore
-rw-r--r--  1 root     root       1354 apr  7 09:08 .gitpod.yml
drwxr-xr-x  3 root     root       4096 apr  7 09:08 images
drwxr-xr-x 17 root     root       4096 apr  7 09:08 InvenTree
-rw-r--r--  1 root     root       1071 apr  7 09:08 LICENSE
-rw-r--r--  1 root     root        119 apr  7 09:08 package.json
-rw-r--r--  1 root     root     149310 apr  7 09:08 package-lock.json
-rw-r--r--  1 root     root        916 apr  7 09:08 .pkgr.yml
-rw-r--r--  1 root     root       1205 apr  7 09:08 .pre-commit-config.yaml
-rw-r--r--  1 root     root        194 apr  7 09:08 Procfile
-rw-r--r--  1 root     root       7726 apr  7 09:08 README.md
-rw-r--r--  1 root     root        746 apr  7 09:08 RELEASE.md
-rw-r--r--  1 root     root        936 apr  7 09:08 requirements-dev.in
-rw-r--r--  1 root     root      23082 apr  7 09:08 requirements-dev.txt
-rw-r--r--  1 root     root       3177 apr  7 09:08 requirements.in
-rw-r--r--  1 root     root       5966 apr  7 09:08 requirements.txt
-rw-r--r--  1 root     root         14 apr  7 09:08 runtime.txt
-rw-r--r--  1 root     root        790 apr  7 09:08 SECURITY.md
-rw-r--r--  1 root     root       1047 apr  7 09:08 setup.cfg
-rw-r--r--  1 root     root      17973 apr  7 09:08 tasks.py
drwxr-xr-x  2 root     root       4096 apr  7 09:08 .vscode

From whatever service is giving you the 502:

Do you have a proxy server running in front of InvenTree?
What URL are you trying to access?
What are the connection logs from your browser?

Im connecting to local host:

not running a proxy server
but the same happens when i connect from an othe machine to the IP of that machine (Which btw is how i alway connected to InvenTree)

does this help you?
Br Martijn

Ok, so probably the web server is not running, and nginx is not able to connect you to the server.
First port of call - check the nginx logs to see if there is any useful information there.
Second, @matmair can you advise on how to view inventree logs when installed via pkg?

Brief update. mu Yubuntu was still at 20.01lts and since it appeared somethig was horebly broken i decide to upgrade to 22.04.
this did me bring closer to home. i.e. i no longer get the ngnix error but now the <IP_ADDRESS> reports Unable to connect. an error occurd during the connection to <IP_ADDRESS>
seems to me like InvenTree is not starting its web server??
but perhaps we knwo more once we know where the InvenTree log files are located?
Br Martijn

Depends on your log level/where the error is if anything is logged for this error. Check inventree logs to see if anything was logged.
I think that your install is not "horribly broken", there is no need for large changes in the environment - that will make debugging just harder for us.
If you have already upgraded your os you will need to uninstall the inventree package and run the installer again (your settings and data remain) through this if you use the package installer. 22.03 and 20.01 have different packaging sources and 22.04 needs several extra packages for inventree to run.
 Let's check first if uninstalling and running the quick install again fixes it - then we will try to check the env.

@matmair thnx that sounds hopefull.
when i look in  /opt/inventree/log -> /var/log/inventree
ls -al
drwxrwxr-x  2 inventree inventree  4096 jul  5 00:00 .
drwxrwxr-x 17 root      syslog     4096 jul  5 00:00 ..
-rw-r--r--  1 inventree inventree     0 apr  9 00:00 setup_2023-04-07_09_01_45.log
-rw-r--r--  1 inventree inventree 66727 apr  9 00:00 setup_2023-04-07_09_01_45.log.1
-rw-r--r--  1 inventree inventree     0 apr 27 00:00 setup_2023-04-25_15_30_41.log
-rw-r--r--  1 inventree inventree 41051 apr 27 00:00 setup_2023-04-25_15_30_41.log.1
-rw-r--r--  1 inventree inventree     0 mei 27 00:00 setup_2023-05-25_21_00_40.log
-rw-r--r--  1 inventree inventree 25335 mei 27 00:00 setup_2023-05-25_21_00_40.log.1
-rw-r--r--  1 inventree inventree     0 jul  4 00:00 setup_2023-07-02_20_27_31.log
-rw-r--r--  1 inventree inventree 41338 jul  4 00:00 setup_2023-07-02_20_27_31.log.1
-rw-r--r--  1 root      root          0 jul  5 00:00 setup_2023-07-03_09_09_36.log
-rw-r--r--  1 root      root       2144 jul  5 00:00 setup_2023-07-03_09_09_36.log.1
so the most recent log is the one from the 3d.
setup_2023-07-03_09_09_36.log.1.log
hope this help you or els epoint me to the file we are looking for.
wrt to reinstalling InvenTree.
invoking the install again is mostlikly again:
wget -qO install.sh https://get.inventree.org && bash install.sh
but removing it... how do o do this?

Hi All,
sorry to ask but any hints how to proceed?
@matmair did the above help you in anyway?
If its safe to remove Inventree and reinstall that i'm fine with that as long as i get it operational again.
I do think that the update mechanism need some updating to avoid this for others. somethings about a dependency check or automatically install the required pyhton version.

You can uninstall inventree with apt remove inventree like any other package on debian/ubuntu https://linuxhint.com/uninstall-package-ubuntu/. It is safe to uninstall and install again - which is why i suggested it.
We are happy to review PRs regarding improvements. There are hosted offerings of InvenTree for ppl. without sysadmin knowledge, I recommend those for users relying on InvenTree for their buisness.

@matmair thanks for confirming and that solved the issue.
Uninstall was not succesfful though as there was no Inventree installed it reported. but just running wget -qO install.sh https://get.inventree.org && bash install.sh solved it and made it work again.
many thanks.


chore: Send query text when executing query over RPC

{
  "database_id": "00000000-0000-0000-0000-000000000000",
  "connection_id": "00000000-0000-0000-0000-000000000000",
  "query_text": "SELECT * FROM my_pg.public.lineitem WHERE l_shipdate <= DATE '1998-12-01' - INTERVAL '90' LIMIT 5",
  "telemetry_tag": "unknown",
  "execution_status": "success",
  "error_message": null,
  "elapsed_compute_ns": 11627950,
  "output_rows": 5,
  "bytes_read": 15466360,
  "bytes_written": 0
}

I didn't make many changes to incorporate an operation_id right now since there are going to be many changes as we're working on distributed exec but getting the query text in was quick and easy enough.

this is fine in it's current form, getting the operation ID is definitely needed for distributed execution, and I expect that we'll want to pass around an operation context object that will have all of this request-related metadata rather than passing each one in.


Add a changelog with stubs for older releases

For now, just annotate the past two releases. Fixes #499

Good idea!
Sometimes I forget to mention stuff in the tags, it will help to document our progress in the changelog as we go.


Should the README.md refer to the spling_bee_moderator app for releases?

So the https://github.com/splunk/spling_bee_moderator releases includes the spling_bee app, and considering there are no releases on this github perhaps the README should mention to refer to that github location to get a copy of the required apps?

I will get a release added to the repo now and clear up the README

Release added


Better support interactive VisualizationPage - keep PlotlyManager

When using interactive components on a VisualizationPage, the callback will need to get access to the PlotlyManager.
We need a mechanism to retain the original PlotlyManager, so we don't have to re-generate.
Work-around:

In the get_layout_children' store the pm: self.pm=pm`
In the callbacks, you can now get it back: pm = self.pm

Define a VisualizationPage.get_plotly_manager() -> Optional[PlotlyManager]
And store the pm in the VisualizationPage when the pm gets created in VisualizationPage.get_layout`

In v<IP_ADDRESS>b, added self.pm to VisualizationPage.get_layout.
So later we can get it back using self.pm.


Fix CI OpenSSL Issue

CI builds error with failed to run custom build command for `openssl v0.9.24

Resolved by d237407a594ad4c203c5f71dd67c5f22f603f761


fix(list): zindex, height 수정

:tada: This PR is included in version 1.4.17 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: