TypeError: Cannot read property 'packageJson' of undefined

Affected Packages

{
  "devDependencies": {
    "@changesets/cli": "^2.23.0",
    "typescript": "4.7.3"
  }
}

node v14.19.3
pnpm v7.2.1
pc Apple M1 Pro
Problem

🦋  error TypeError: Cannot read property 'packageJson' of undefined
🦋  error     at createChangeset (/Users/fupeng/Workspace/github/silly-girl/node_modules/.pnpm/@changesets+cli@2.23.0/node_modules/@changesets/cli/dist/cli.cjs.dev.js:415:75)
🦋  error     at add (/Users/fupeng/Workspace/github/silly-girl/node_modules/.pnpm/@changesets+cli@2.23.0/node_modules/@changesets/cli/dist/cli.cjs.dev.js:507:26)
🦋  error     at processTicksAndRejections (internal/process/task_queues.js:95:5)
🦋  error     at async run$2 (/Users/fupeng/Workspace/github/silly-girl/node_modules/.pnpm/@changesets+cli@2.23.0/node_modules/@changesets/cli/dist/cli.cjs.dev.js:1254:5)

Proposed solution

The reason may be for this

Package.json file is not at the root level
If it is mono repo, package.json should be available within the each service folder

mono repo project
changesets are installed in the root directory
The rest of the subpackages also have package.json

Check do you have workspace file. I use pnpm. So i have pnpm-workspace.yaml.

I am also pnpm, workspace related declarations are normal
I was wondering if it was a problem with my nodejs version, I migrated from lerna

I had this issue, i resolved after doing the above steps. writing it again to ensure you too follow the same

have pnpm-workspace.yaml, should have references to those repos
install @changeset/cli
Added package.json inside the service repos

Thanks for your reply, I will try again
The corresponding files and dependencies should be complete

https://github.com/mahboobmonnamd/monorepo-js check this it may help you

Thanks guy

@fupengl  Has the problem been solved？

We need a repro case to look into this.

I have a repro here @Andarist https://github.com/hcharley/changesets-issue-841/runs/7570186130?check_suite_focus=true

in my root package.json has this config :  "workspaces": [ "packages/" ];  when i run " npx changeset", there has err "TypeError: Cannot read properties of undefined (reading 'packageJson');"
so, i try to change "workspaces": [ "packages/" ]  => "workspaces": [ "packages/*" ]  or delete this config, the are working!!

In my case, no packages with versions were found. The error is not very developer friendly and the issue should be easy to solve. As @Andarist said, repro steps are needed to recreate any issue. Having said that, can we have a look to the code?
Consider:
packages/cli/src/commands/add/createChangeset.ts
export default async function createChangeset(
  changedPackages: Array<string>,
  allPackages: Package[]
): Promise<{ confirmed: boolean; summary: string; releases: Array<Release> }> {
  const releases: Array<Release> = [];

  if (allPackages.length > 1) {
  // Code omitted
  } else {
    let pkg = allPackages[0];

    // Code omitted
  }
}

The source code falsely asumes pkg is a Package. It can perfectyl be undefined since allPackages can be an empty array. Consider using noUncheckedIndexedAccess.
This is particulary true to some flows where no package is eligible since
packages/cli/src/commands/add/index.ts
  const changedPackagesName = changedPackages
      .filter((pkg) => isListablePackage(config, pkg.packageJson))
      .map((pkg) => pkg.packageJson.name);
  newChangeset = await createChangeset(changedPackagesName, listablePackages);

the add function can filter out all the packages detected.
Proposed solution
For the sake of a better developer experience, what about updating createChangeset to throw a meaningfull error in case no package is detected?
packages/cli/src/commands/add/createChangeset.ts
export default async function createChangeset(
  changedPackages: Array<string>,
  allPackages: Package[]
): Promise<{ confirmed: boolean; summary: string; releases: Array<Release> }> {
  const releases: Array<Release> = [];

 if (allPackages.length === 0) {
   throw new MeaningfullError('Super duper description!');
 }

  if (allPackages.length > 1) {
  // Code omitted
  } else {
    let pkg = allPackages[0];

    // Code omitted
  }
}

This may be suficient for getting a better experience, but we could even iterate this solution to a (imo) better one. Imo, we should not change the fact createChangeset must fail if an empty array but, what if this error is catched in order to throw an even more meaningful error? As discussed earlier, the add function (at packages/cli/src/commands/add/index.ts) knows the potential listed packages, and the filtered packages. What if we catch the error thrown on createChangeset in order to throw an error informing the use the list of detected packages and a message telling that no package was eligible to add changes? This way we would defend ourselves from wrong createChangeset inputs while we deliver a developer friendly error message.
I'm more than open to go with a PR @Andarist :heart:

My repo https://github.com/yunsii/fasttext.wasm.js encounter the samle error, because root packages.json should be released, but after create beach workspace as internal package and add bench to ignore, it throw the error when I run npx changset.
Even without ignore bench, it only release bench, no root package anymore……

I found a workaround, delete pnpm-workspace.yaml before release, restore it after release completed 😂

I found a workaround, delete pnpm-workspace.yaml before release, restore it after release completed 😂

Follow the logic, I add workspace . to pnpm-workspace.yaml and restore changset ignore config, it works like a charm, refer to https://github.com/yunsii/fasttext.wasm.js/commit/5e1dd82625011175af03bdb28fb819cb1f933182

fwiw, I ran into this error because of a malformed root package.json. Since I was also using pnpm, I had missed that the workspaces entry must be an array:
{
  "workspaces": [
    "packages/*"
  ]
}

what works for me. is the name of the package that i had was not of the same account that i had the npm token of. fixing it that is chagning the name to the account that i am using npm token fixed it for me.

Follow the logic, I add workspace . to pnpm-workspace.yaml and restore changset ignore config, it works like a charm, refer to yunsii/fasttext.wasm.js@5e1dd82

This seems to be a weird workaround since the pnpm-workspace.yaml documentation states that the root package is always included.
About the workspaces property in the root package.json file, for PNPM, this is irrelevant since the pnpm-workspace.yaml file occupies this role.
If anyone has a solution that doesn't rely on a workaround or on duplication of configuration, that would be appreciated.

I fixed my issue by going through the changeset files. I had previously manually changed one of them and made a typo in the package name.

I fixed my issue by going through the changeset files. I had previously manually changed one of them and made a typo in the package name.

This worked for me!

My case:

I had a package a
Created a changeset for a
renamed the package a to b
changeset was still pointing a
Cannot destructure property 'packageJson' of 'undefined' as it is undefined

To fix I just remove the changeset (since I'll be starting over the publish)


ui: Login page

PR: https://github.com/cockroachdb/cockroach/pull/46783
From release notes:

Updated the Login page design. [#46783][#46783] {% comment %}doc{% endcomment %}

Closing because we don't have a login page screenshot (and I don't think we need one unless the design & content are not likely to change again).


Argument Exception: Username and password required on .UseWatchLog()

ASP.NET 6 API Project
Project builds successfully but I get this exception at startup.
Watchdog 1.0.1 installed

Hi @AltaherEmhemed
From the screenshot of your code, I can see that you are setting the password twice.
Meaning you have no username set up.
Correct that and you should be good to go

We are marking this as closed.. Cheers

@IzyPro
I fixed the username but now I get a DirectoryNotFound exception

Hi @AltaherEmhemed
This has been fixed and an updated version (1.1.0) has been published.


Fixes to RenderAllCells test

For #8483
Wait for messages instead of waiting for a specific number of renders.

Codecov Report

Merging #8717 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8717      +/-   ##
==========================================
+ Coverage   58.21%   58.22%   +<.01%
==========================================
  Files         526      526
  Lines       24572    24572
  Branches     3974     3974
==========================================
+ Hits        14304    14306       +2
+ Misses       9325     9322       -3
- Partials      943      944       +1

Impacted Files
Coverage Δ

src/client/linters/pydocstyle.ts
84.61% <0%> (ø)
:arrow_up:

src/client/common/process/proc.ts
13.28% <0%> (ø)
:arrow_up:

src/client/datascience/debugLocationTracker.ts
79.24% <0%> (ø)
:arrow_up:

src/client/common/utils/platform.ts
64.7% <0%> (ø)
:arrow_up:

src/client/linters/pylint.ts
98.18% <0%> (+3.63%)
:arrow_up:

Continue to review full report at Codecov.

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


Test: New terminal title and description variables

Refs: #233091

[x] macOS @ulugbekna
[x] linux @justschen
[x] windows @chrisdias

Complexity: 2
Create Issue

The terminal.integrated.tabs.title and terminal.integrated.tabs.description settings now have access to 3 more variables: ${shellType}, ${shellCommand} and ${shellPromptInput}. Please consult the setting descriptions and make sure they work as described.

LGTM


NASA Challenge_[@AndrejOrsula]_[Parallel ProcGen Environments]

Hello there!
This PR adds a spaceros_procgen_envs package that offers a collection of procedurally generated environments designed to bridge the gap between simulated robotics and the vast unpredictability of space.
Key Features:

On-Demand Procedural Generation with Blender
Highly-Parallelized Simulation with NVIDIA Isaac Sim
Compatibility with Gymnasium API
Integration with Space ROS
Agnostic Interfaces
Diverse Demonstrations

Video Examples (click to expand)
https://github.com/user-attachments/assets/1adf2853-dd62-4819-a60a-d633ce14b7a5
https://github.com/user-attachments/assets/a757cbdb-3209-41b6-8567-ccc3bd051061
https://github.com/user-attachments/assets/8ab43dc7-765b-436f-83d2-0e128a9d1b99
https://github.com/user-attachments/assets/d8975865-bba6-4b65-aba1-70c09757cce8
https://github.com/user-attachments/assets/bbbe8c2b-e1b4-4ca1-b8b1-069b288ffdc6
https://github.com/user-attachments/assets/8f83465e-a647-4211-8b1b-7561babcf50c
https://github.com/user-attachments/assets/1a3b3706-8855-46bc-9c5d-14892cc9a4ea

Challenge Name:   NASA Space ROS Sim Summer Sprint Challenge
Freelancer Name:  AndrejOrsula
Submission Title: Parallel ProcGen Environments

Closes #71


headerTruncatedBackTitle option doesn`t work

Current Behavior
I found a header options headerTruncatedBackTitle https://github.com/react-community/react-navigation/blob/be1da2c4c97a97aefc81c7e7497913e440423bdb/docs/api/navigators/StackNavigator.md#headertruncatedbacktitle
But truncated back title leave Back.

My code
class AboutScreen extends Component {
  static navigationOptions = () => ({
    headerTitle: 'About',
    headerStyle: styleHeader.common,
    headerTitleStyle: styleHeader.title,
    headerTruncatedBackTitle: 'Return', // this is not working
  })

  render() {
    return (
      <Container>
        <Content>
          <Text>some text</Text>
        </Content>
      </Container>
    );
  }
}

Expected Behavior
I want change back title to Return or disabled title at all, leave only arrow back
Your Environment

software
version

react-navigation
1.0.0-beta.11

react-native
0.47.1

node
6.11.2

npm or yarn
3.10.10

Can you provide a demo of this issue? Preferably on Snack

The headerTruncatedBackTitle is never showing because headerBackTitle is not set. headerTruncatedBackTitle is a back-up when headerBackTitle is too long to render properly.

@matthamil Thanks, it works for me.

Pinging OP @tavriaforever since this issue has not been active for a while, and it's related to an old version of the lib.
Please let me know if you want this to remain open; if I get no answer in the next 7 days I will close it.

Let`s close it! Thanks for help!

pr to docs welcome: http://github.com/react-navigation/website

can continue discussion on the issue linked to above ^


File.createTempDirectory permission

Hi,
I try to use File.createTempDirectory to create a temporary directory and a file inside it.
But both on Mac OS et PC the problem is the same. The new directory is on read only.
var temp:File = File.createTempDirectory();
 var file:File = new File(temp.nativePath + "/" + "lib.txt");

So in my exemple, lib.txt is not created.
Could you help me to change permission before create new file.
Best regards

Please post in a forum for Adobe AIR. This is not a vscode-as3mxml issue.


Seperate Exception for authentication error in an extractor

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.08.12. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[ x] I've verified and I assure that I'm running youtube-dl 2016.08.12

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

Can we have a seperate exception for authentication errors in extractors so that while using it as a python module we can identify login errors.

Good idea!


[Loc] update to azurecore xlf 11-24-2022

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.004%) to 41.96%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-bindings/src/common/azureFunctionsUtils.ts
1
80.65%

extensions/notebook/src/dialog/configurePython/configurePathPage.ts
1
57.02%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
28886

Relevant Lines:
63969

💛  - Coveralls


Get an exception when trying to convert a Word document to Html

Hi,
When trying to run WordToHtmlConverter.Process(path)
I get
Inheritance security rules violated by type: 'NPOI.HWPF.HWPFDocumentCore'. Derived types must either match the security accessibility of the base type or be less accessible.
NPOI version: 2.5.2,  NPOI.HWPF version: 2.3.0, SharpZipLib version: 1.3.1
Could you please tell what is wrong?

HWPF is not included in the official release. Where did you get this release?

My bad, didn't pay attention that it is a different source. NuGet, https://github.com/ziyunhx/npoi
Could you please tell, do you support Word to Html conversion?

HWPF is not officially supported because there are too many bugs and it's kind of early implementation. there is a story about HWPF in POI. After the major contributor of HWPF joins Microsoft, he cannot contribute anymore. Then HWPF module almost stopped.
For docx, yes, we do support that as I remember. But doc, no.

Thank you so much!
Could you please tell me where can I find this functionality for docx, as I actually need it for docx? It seems like not in there
https://github.com/nissl-lab/npoi/tree/master/ooxml/XWPF.

I double check XWPF namespace. WordToHtmlConverter is not implemented but WordExtractor is implemented.


[bugpoint] Few improvements

This PR contains few improvements for bugpoint:

New mutator that tries to move instructions to the entry block (closes #1540)
Missing conversion to constants for vector and reference types (closes #1633)
Aliases are now resolved after reduction, not before (some mutators may create aliases)

r? @abrown

New mutator that tries to move instructions to the entry block (closes #1540)

Thank you for adding this!

Can you keep this one. I think it can help prevent duplication in some cases.

Sure. I added also check if function still crashes after resolving them, just to be sure.


Imviz tries to create directories upon import

In the top-level scope of common.py:
https://github.com/joruof/imviz/blob/1ee67380aa9677da12aea6b118c4a438304e24e2/imviz/common.py#L138-L146
This seems undesirable when just importing and not yet running any library functions.
Also, wd might be a non-writable (systems) directory, in which case the import just fails with a PermissionError.

This behavior is a leftover of some earlier experiments with including latex rendering in the library.
I agree that this is undesirable. The respective functions are now removed on the develop branch.


Improve pre-commit usage documentation

Documentation for usage with pre-commit should be improved. In particular noting how to use alongside optional dependencies, if the user is using things such as pyproject.toml for configuration.
See: https://github.com/timothycrosley/isort/pull/1076

This is fixed in develop for release 5.0.0 by virtue of removing any and all magic dependencies


Update dependency net.fabricmc:yarn to v1.21.3+build.2 - abandoned

This PR contains the following updates:

Package
Type
Update
Change

net.fabricmc:yarn
dependencies
patch
1.21.2+build.1 -> 1.21.3+build.2

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.

Autoclosing Skipped
This PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.


[Bug]: If you enter an incorrect password to log in, a message is incorrect

ODC version
4.3.3
OB version
no related
What happened?

If you enter an incorrect password and click Login, the login will be successful, and the interface will stay on the login page
What did you expect to happen?
Click Login with the wrong password and you should be prompted correctly
How can we reproduce it (as minimally and precisely as possible)?
Enter the wrong password and click to log in, and observe the prompt copy
Anything else we need to know?
No response

pass


CVU-75 Temp Fix Google Drive download issue

Temporary fix for the downloading issue in cvu.utils.google_utils.gdrive_download

Looks good to me.


Sm/external nuts update for plugin migration

What does this PR do?
adjust the external nuts for changes related to plugin-source and PDR (some commands and some nuts are in different places)
run pdr nuts now that it's "ga"
[skip-validate-pr]

closed in favor of #917


NPM Package - console.log

Again, thanks a lot for fixing the previous issue. It even resolved an other issue we had. Now it fulfills all requirements we have!
Might it be possible to pump the newest version to npm. The current npm version includes a console.log in the ui router code...
get permissions

Thanks!

done :)


What is the GNU social protocol specification?

Hello,
In AndStatus' description I read:

It supports multiple protocols, including GNU social

So I wanted to implement the GNU social protocol on my own.
However, the GNU social homepage says:

It is social communication software

I haven't found any links to a protocol specification either. My question is:
Is there a specification for the GNU social protocol, or is the interface undocumented, reverse-engineered by you, and possibly subject to change often?

Hi @dejj This is the only live link to GNU Social API,  which I know:
http://www.skilledtests.com/wiki/Twitter-compatible_API
Unfortunately, it refers to the Twitter Rest API v.10, which is not available now.
As AndStatus supported Twitter earlier, I had to add the "StatusNet" specific. See https://github.com/andstatus/andstatus/blob/master/app/src/main/java/org/andstatus/app/net/social/ConnectionTwitterGnuSocial.java
If you will find Twitter Rest API 1.0 docs, please post here.


Auth redirection user when not connected doesn't work

Hi again!
Auth module doesn't redirect to login page in case of LoggedIn: false
auth: {
    globalMiddleware: true,
    redirect: {
      login: "login", // redirect user when not connected
      callback: "/", // redirect user after login
      logout: "/search", // redirect user after logout',
      home: "/search",
    },
    strategies: {
      // @ts-ignore
      auth0: {
        scheme: "oauth2",
        endpoints: {
          authorization: `https://${process.env.AUTH0_DOMAIN}/authorize`,
          userInfo: `https://${process.env.AUTH0_DOMAIN}/userinfo`,
          token: `https://${process.env.AUTH0_DOMAIN}/oauth/token`,
          logout: `https://${process.env.AUTH0_DOMAIN}/v2/logout`,
        },
        responseType: "token",
        grantType: "authorization_code",
        clientId: process.env.AUTH0_CLIENT_ID,
        domain: process.env.AUTH0_DOMAIN,
        scope: ["openid", "profile", "email", "offline_access"],
        codeChallengeMethod: "S256",
        autoLogout: true,
        resetOnError: true,
      },
    },
  },

I created redirect logic in my main layout and it works, but I don't like this solution
const router = useRouter();
const nuxtApp = useNuxtApp();

if (!nuxtApp.$auth.loggedIn) {
  router.push("/login");
} else {
  fetchData...
}

Is this when you (as a user) logout or when you visit a page (as a guest that has never logged in)?

When I am logged out.


Slimmer docker image. Also uses PyPI instead of GitHub for the package

Based on the official Python-on-debian images instead of Ubuntu base.
Unfortunately is a little less readable than the existing one due to having two stages: a build stage based on the thick debian buster Python image and a runtime stage based on the slim debian buster Python image.
Takes the final image size down from 485 MB to 165 MB.

Open to your feedback, @cluelessperson and @stakauskas as I know you've worked with electrumx in container environments before.

Going to go ahead and merge this. Feedback still very welcome.


Running SATURN with CPU

Dear Authors,
I understand that SATURN was made for GPU. However, with a server that operates only with CPU, I am attempting to run it on a CPU by modifying the command as follows:
!python ../../train-saturn.py
...
--device_num=-1
I am reaching out to confirm whether it is feasible to run the code on a CPU. Currently, the code is running without issues during the pretraining step, but taking significantly longer. My main concern is whether using a CPU instead of a GPU could cause problems in later stages of the integration process.

Thank you for your guidance!

I also want to run SATURN with CPU ！！！！


Read files lazily to help checksumming files which can't fit in RAM

Reading files which are are too large to fit in RAM can cause the Python interpreter to be killed in some managed environments.

Beta 2 addresseses this. Waiting on feedback from @seanblau before closing.

Confirmed fixed in Beta 2.


Link to EULA for Docker containers

Hi,
if you really want to use uploaded reports from the docker container version it may be a good idea to link to the EULA in the readme of this repository.
Link: https://kube-hunter.aquasec.com/eula.html

Customer hereby agrees that Aqua may upload samples of such results to its servers for its internal informational and research purposes

Kind regards,
Matthias

Good idea, thank you @ramshazar


listkey props missing in FlatList  0.59.5 version

we have added manually listkey props for now , please update it next release , so we can sync .

Please provide environment info and demo code so we can reproduce the issue. Will reopen the issue once you provide it. Thank you


YouTube Player

Create a method to filter out portion of the YouTube url to use for YouTube embedded player.

Used a ruby gem that will translate any of the YouTube urls. See here for more info, https://github.com/thibaudgg/video_info.
Added a 'video_info' method to artist model


Integration with Cargo-Leptos

Hello, I was wondering if there was any plans to get this library working with cargo-leptos? I was looking to make a template for The axum template with leptonic. What work is needed to make that happen? i'd be willing to help out on that feature if given some context.

I added initial SSR support in the main branch. Would you like to check out the current state?

I also need to use SSR support with cargo leptos support without trunk, are there any instructions in the documentation? I can't find anything about it

@ccmvn Have you figured it out by any chance?

@ccmvn Have you figured it out by any chance?

Unfortunately not, I also wrote an email to the repo owner but no answer came back

@ccmvn Have you figured it out by any chance?

Unfortunately not, I also wrote an email to the repo owner but no answer came back

Thank you for coming back.
After I asked here and was not able to make Leptonic work, we switched to Thaw, which got better documentation regarding the SSR thing, and I actually made it work. After many hours spent reading Leptos and Thaw docs, I actually understood the whole thing, and I made Leptonic work with cargo leptos in a matter of 30 minutes. I made a public repo template that you can check https://github.com/dady8889/leptos-ssr-leptonic.
Explanation if anyone stumbles here and is lost like me.

Cargo leptos builds your project as lib once with features = ssr and once with features = hydrate by default.
With features = hydrate, the WASM library gets compiled, but with features = ssr, only the server's stuff gets compiled
Therefore, when you try to run a frontend component like <Button> in a code that gets compiled with features = ssr, you get an error.
To make this work, library maintainers must provide also a crate with "features = ssr/hydrate" that you can conditionally compile, and in the SSR mode, you basically do nothing, its just an empty function.

For Thaw, that was all I needed to do. But for Leptonic, there was also a few more things, and you can see everything in this commit: https://github.com/dady8889/leptos-ssr-leptonic/commit/36e87253a7eeeb72cf4bdf98ce6a36d93036f5b1

Can confirm this appears to work.


Bump iohk nix and fix platform passing

I think this is doing the right thing!
Also a few other minor fixes.

Don't merge yet, this has somehow broken package set generation:
Using hackage snapshot from 2018-11-08T09:58:14Z

Ensuring git version is >= 2 ...
Ensuring cabal version is >= 2 ...
stack2nix: Failed with exit code 1...
"truncated tar archive\n"

CallStack (from HasCallStack):
  error, called at src/Stack2nix/External/Util.hs:21:3 in stack2nix-0.2.1-E9bCspWRYXk5eV9b6ny8cA:Stack2nix.External.Util

Scratch that, it happens to me on master too, so that's a separate issue...

(The problem was a corrupted .cabal, but note that this is generated in pkgs not your home directory, so that's what you need to clear out)

Gonna merge this to unbreak things, but I'd still like a look over at some point.


Dart migrate positions its markers not correctly in FireFox and Safari

Hi,
while trying to migrate my get_it_mixin package on my MBA M1 I found this effect here

´´´
[✓] Flutter (Channel master, 1.26.0-2.0.pre.17, on macOS 11.1 20C69 darwin-arm, locale de-DE)
• Flutter version 1.26.0-2.0.pre.17 at /Users/escmoteur/Entwicklung/flutter
• Framework revision e74712c4d6 (4 weeks ago), 2020-12-15 12:48:05 -0500
• Engine revision cca6ab654e
• Dart version 2.12.0 (build 2.12.0-153.0.dev)
´´´
I have no Chrome installed here so that I can't tell you if it would work there
What I realized is that the tool adds a lot of type information where before only a ´var´ was and for that type you can't set any hint
Also it would be helpful if it where possible also to set ´late´hints

@srawlins would you mind having a look at this?  I suspect there's probably a fairly easy CSS fix

@srawlins would you mind having a look at this?  I suspect there's probably a fairly easy CSS fix

You can set late hints (either /*late*/ or /*late final*/ but the tool does not have buttons for those yet.

You can set late hints (either /*late*/ or /*late final*/ but the tool does not have buttons for those yet.

I can reproduce the visual issue in Firefox

I can reproduce the visual issue in Firefox


NGINX as HTTP proxy

I want to use NGINX as a forward proxy with my public ip . Using this configuration i managed to proxy using my local ip but i want it to be a public proxy using my public ip address. How can i do that ? Do i need port forwarding or some changes in my configuration ?
here is my nginx.conf:
worker_processes 1;
events {
worker_connections 1024;
}
http {
include mime.types;
default_type application/octet-stream;
sendfile on;
keepalive_timeout 65;
gzip on;
server {
listen 8080;
location / {
resolver <IP_ADDRESS>;
proxy_pass http://$http_host$uri$is_args$args;
}
}
}

Merged with #55


Set rules for outline entries

Description
I'm not sure whether to classify this as a bug, but as a typical user, I expected:
show outline.entry.where(level: 1): set outline.entry(fill: none)
to work. Alas, due to the parameters in outline.entry being entirely positional, it does not. My request is that they be made keyword parameters.
Use Case
This would make it way easier and quicker to style a large variety of outlines.

Another way of giving people a work around for all such cases would be to have e.g. show outline.entry.where(level: 1): set outline.entry(_, _, _, _, none) or set outline.entry(,,,, none) retain the pre-set value at the underscore/empty parameters.

If outline.entry isn’t possible to construct from the Typst side, then we could make its fields named without breaking changes.

It is possible, but according to people on the Discord server it's highly discouraged/not supposed to be done.

Outline.entry was created before show-set rules, in their current form, existed, so styling wasn't factored into their design. The proposal seems reasonable to me.

I'm interested in implementing this. A simple change to a function's interface seem like it would be a decent first issue. If there are no objections, I'll give it a shot.

That wouldn't be the only change needed. You'd also need to change the outline implementation in such a way that it doesn't specify entry properties directly, allowing them to be overridden. I'm not sure what'd be the best way to do that, but one could investigate.

This morning I decided to check it out. I've got it working for
show outline.entry.where(level: 1): set outline.entry(fill: none)

and
set outline.entry(fill: none)

but
show outline.entry.where(level: 1): it => {
    set outline.entry(fill: none)

    strong(it)
}

doesn't work. I have some intuition as to why, but I feel like as an end user I would like this to work (re-constructing it using outline.entry() would lead to an infinitely recursive show rule). Can anyone give me some comments on this? Should it? If yes, some hints as to how to get there?
Another question I have: The interface I decided to try for (with a look to consistency with what's already there) is to have fill be Smart<Option<Content>>, where Smart::Auto means it inherits fill from the nearest surrounding outline element, none and content behave as currently. My question is this: is """best practice""" to crawl the ancestors of the outline element using engine and retrieve the fill from an outline element if the field is set to Smart::Auto, OR should I instead go top-down and have outline reassign the fill if it finds that it is set to the default (Auto)? What are the guidelines on a choice like that? Regardless, it would help me to have one or two quick hints as to how to execute the last option.

As for the last show rule, it is expected that it doesn't work. It works for no element like this. This was an intentional design after a longer design process because there was no good way to make it work without sacrificing other desirable properties. See #3311 for details.
The proper solution is to split this rule into two: One show-set rule and one normal show rule.

Yes, I thought so.

What about the second question? Any insights?

Another question I have: The interface I decided to try for (with a look to consistency with what's already there) is to have fill be Smart<Option<Content>>, where Smart::Auto means it inherits fill from the nearest surrounding outline element, none and content behave as currently. My question is this: is """best practice""" to crawl the ancestors of the outline entry using engine and retrieve the fill from an outline element if the field is set to Smart::Auto, OR should I instead go top-down and have outline reassign the fill if it finds that it is set to the default (Auto)? What are the guidelines on a choice like that? Regardless, it would help me to have one or two quick hints as to how to execute the last option.

A top-down approach would be necessary, and would probably look similar to how table and table.cell work.

I think the outline.fill parameter becomes rather unnecessary if set outline.entry(fill: ..) is possible.

Any updates on this? I've been looking for this as well, precisely for fill as it would make styling not require re-making the entry.

No updates. I'm honestly not sure whether it's straightforward or not. The fact that the outline(fill: _) parameter populates the fields might make it a little tricky.

No updates. I'm honestly not sure whether it's straightforward or not. The fact that the outline(fill: _) parameter populates the fields might make it a little tricky.

Is it tricky in the sense that it's not trivial per-se and could lead to issues if not careful or more that it would need some grander refactoring? Also, another idea maybe: Allow outline.fill to take an array of content similar to list.marker where it depends on the level such that I could do ([none], repeat[.]) for example. Not sure if that is less or more idiomatic but just a thought I had.

Is it tricky in the sense that it's not trivial per-se and could lead to issues if not careful or more that it would need some grander refactoring?

The careful thing. One would need to make sure that all the interactions between set rules and prepopulation from outline are correct. Maybe it just happens to be, maybe not. I'm not sure.

Hi all, I just want to say that it is straightforward and that I did get it working when I last sent any messages here. Without the requirement of backwards compatibility, I can basically make a PR tomorrow.

I don't think backwards compatibility is too much of a concern. Field access on outline.entry will continue to work as usual, so only people that manually created an outline entry would be affected.


Update response.py

modification serialize_headers

font{
    line-height: 1.6;
}
ul,ol{
    padding-left: 20px;
    list-style-position: inside;
}

    Thank you.

                            qinling8980

<EMAIL_ADDRESS>

    签名由
    网易邮箱大师
    定制

On 11/7/2019 15:52，Mariusz<EMAIL_ADDRESS>wrote：

Closed #12043.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.


feat(v6): provide gas price for gentx

Closes https://github.com/strangelove-ventures/interchaintest/issues/1050
Testing
I'm no longer observing the error in https://github.com/strangelove-ventures/interchaintest/issues/1050.

@mergifyio backport v7


Renders blank on 0.29.0-rc.0

Might be related to flex layout changes, nothing is rendered, no errors/issues in the console. Tested on iOS. Same code works w/o a single warning on 0.26/0.27 (haven't tested 0.28)

Maybe this PR in ART can fix this.
I`ll try to fix this later.


pytest hangs

In running through the full test suite a macbook I experience a reproducible hang at:

dask/array/tests/test_array_function.py::test_array_function_dask[2]

I am running the with the following setup:

py.test dask --runslow dask

I believe @pentschev has also seen a similar hangs other linux based machines.  @crusaderky have you seen this issue ?

I actually think I found the culprit, which is specifically test_setitem_extended_API. On my test machine (an NVIDIA DGX-1 with 40 cores/80 threads) it ends up creating more than 20000 threads, eventually making it impossible to create any new processes.

        It must be called at most once per thread object. It arranges for the
        object's run() method to be invoked in a separate thread of control.

        This method will raise a RuntimeError if called more than once on the
        same thread object.

        """
        if not self._initialized:
            raise RuntimeError("thread.__init__() not called")

        if self._started.is_set():
            raise RuntimeError("threads can only be started once")
        with _active_limbo_lock:
            _limbo[self] = self
        try:
>           _start_new_thread(self._bootstrap, ())
E           RuntimeError: can't start new thread

../../miniconda3/envs/rn-102-0.19.210304/lib/python3.8/threading.py:852: RuntimeError

I actually think I found the culprit, which is specifically test_setitem_extended_API. On my test machine (an NVIDIA DGX-1 with 40 cores/80 threads) it ends up creating more than 20000 threads, eventually making it impossible to create any new processes.

E           RuntimeError: can't start new thread

Yes I've seen it before. I've observed on very large AWS Ubuntu VMs (64 cores) that the Linux kernel just gives up after 20k-something threads. This is regardless of ulimit, and clearly a major kernel bug.
I've just run test_setitem_extended_API and I can confirm it spawns 29k threads on my 16 cores / 32 threads local Ubuntu box. I'm not affected by the kernel bug so it doesn't crash.
Clearly, dask should not go anywhere close that number of threads.
test_array_function_dask doesn't give me any problems on my ubuntu box, nor it does on the MacOSX CI. I don't think the two issues are related in any way?

@quasiben the first thing I'd check is that the global dask scheduler is 'threads' and not a distributed scheduler started by some unrelated test, not properly cleaned up, and halfway through its shutdown.

For context the issue with test_setitem_extended_API was reported over in https://github.com/dask/dask/issues/7261

test_array_function_dask doesn't give me any problems on my ubuntu box, nor it does on the MacOSX CI. I don't think the two issues are related in any way?

I actually think the hang was more of a side-effect to test_setitem_extended_API, probably at that point the process had already too many threads spawned, and later on test_array_function_dask[2] hangs when trying to obtain some now exhausted resource. Therefore, I don't think we have anything here besides the test_setitem_extended_API issue, so I think it's fine closing this and continuing discussion on https://github.com/dask/dask/issues/7261. Thanks both for looking and linking to the other bug report. 🙂

Thanks @jrbourbeau and @crusaderky closing and continuing the discussion in #7261


isinstance fails nondeterministically with envs / approaches

likely because of the use of load_module in envs/__init__.py and approaches/__init__.py, related to this: https://stackoverflow.com/questions/15876395/isinstance-behavior-with-module-reload

I can't figure out any good workaround, and I don't expect this to really cause issues down the line, but in case it does, we'll have this as a record at least.

@amburger66 encountered a version of this issue independently, so I no longer feel like we can ignore it. FYI @ronuchit. We can discuss how to proceed...


Fix click-to-play delay for standalone videos

The click-to-play delay, introduced with the options page, didn't apply to videos which are on their own page. This is a potential fix.

Nice catch. I'll be away from my computer for the day, but when I get back I'll push out a quick patch.

There we go.


Wednesday 15-05-2024 roll call

Roll Call!
Leave a comment with these two things:

A summary of your study using only emojis
Something you'd like to share, anything goes! (within respect)

emoji cheat sheet

hi

Good morning!

salut

Bonjour!

🍋

hi

Hello!

hello

Bonjour !

good morning

Hey

Good Morning

gud morning...

✌

Good morning


NoMethodError: undefined method `instantiate_observers' for ActiveResource::Base:Class

My Gemfile contains the following:
gem 'activeresource', github: 'rails/activeresource'
gem 'rails-observers', github: 'rails/rails-observers'

I found that a similar issue was reported on the activeresource issue tracker, which seems to have been resolved in the master branch, but using rails-observers master brings back the same error.

Could you provide more information about the error? When it happened? what is the stack trace?

It happens when trying to start the server. Here's the full stack trace.
rails aborted!
NoMethodError: undefined method `instantiate_observers' for ActiveResource::Base:Class
Did you mean?  instance_variables
/Users/schristm/.rvm/gems/ruby-2.3.1/bundler/gems/rails-observers-f1e4eaa14539/lib/rails/observers/railtie.rb:38:in `block (2 levels) in <class:Railtie>'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/lazy_load_hooks.rb:38:in `instance_eval'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/lazy_load_hooks.rb:38:in `execute_hook'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/lazy_load_hooks.rb:28:in `block in on_load'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/lazy_load_hooks.rb:27:in `each'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/lazy_load_hooks.rb:27:in `on_load'
/Users/schristm/.rvm/gems/ruby-2.3.1/bundler/gems/rails-observers-f1e4eaa14539/lib/rails/observers/railtie.rb:37:in `block in <class:Railtie>'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/lazy_load_hooks.rb:36:in `execute_hook'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/lazy_load_hooks.rb:45:in `block in run_load_hooks'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/lazy_load_hooks.rb:44:in `each'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/lazy_load_hooks.rb:44:in `run_load_hooks'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/application/finisher.rb:65:in `block in <module:Finisher>'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/initializable.rb:30:in `instance_exec'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/initializable.rb:30:in `run'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/initializable.rb:55:in `block in run_initializers'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/initializable.rb:54:in `run_initializers'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/application.rb:352:in `initialize!'
/Users/schristm/Code/panel/config/environment.rb:5:in `<top (required)>'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/dependencies.rb:293:in `require'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/dependencies.rb:293:in `block in require'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/dependencies.rb:259:in `load_dependency'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/activesupport-5.0.0/lib/active_support/dependencies.rb:293:in `require'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/application.rb:328:in `require_environment!'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/application.rb:448:in `block in run_tasks_blocks'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/commands/rake_proxy.rb:13:in `block in run_rake_task'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/commands/rake_proxy.rb:10:in `run_rake_task'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/commands/commands_tasks.rb:51:in `run_command!'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/railties-5.0.0/lib/rails/commands.rb:18:in `<top (required)>'
/Users/schristm/Code/panel/bin/rails:9:in `require'
/Users/schristm/Code/panel/bin/rails:9:in `<top (required)>'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/spring-1.7.2/lib/spring/client/rails.rb:28:in `load'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/spring-1.7.2/lib/spring/client/rails.rb:28:in `call'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/spring-1.7.2/lib/spring/client/command.rb:7:in `call'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/spring-1.7.2/lib/spring/client.rb:30:in `run'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/spring-1.7.2/bin/spring:49:in `<top (required)>'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/spring-1.7.2/lib/spring/binstub.rb:11:in `load'
/Users/schristm/.rvm/gems/ruby-2.3.1/gems/spring-1.7.2/lib/spring/binstub.rb:11:in `<top (required)>'
/Users/schristm/Code/panel/bin/spring:13:in `require'
/Users/schristm/Code/panel/bin/spring:13:in `<top (required)>'
bin/rails:3:in `load'
bin/rails:3:in `<main>'
Tasks: TOP => db:migrate => environment
(See full trace by running task with --trace)

Thanks! Just fixed it.


Routing Resources Custom Matcher

Testing resources in rails via routing specs in RSpec can be verbose.
For an entire resource, that means tests for all 7 actions:
it 'get to show' do
  expect({:get=>"admin/posts/1"}).to route_to({:controller=>"admin/posts", :action=>"show", :id=>"1"})
end
it 'get to index' do
  expect({:get=>"admin/posts"}).to route_to({:controller=>"admin/posts", :action=>"index"})
end
it 'get to new' do
  expect({:get=>"admin/posts/new"}).to route_to({:controller=>"admin/posts", :action=>"new"})
end
it 'get to edit' do
  expect({:get=>"admin/posts/1/edit"}).to route_to({:controller=>"admin/posts", :action=>"edit", :id=>"1"})
end
it 'post to create' do
  expect({:post=>"admin/posts"}).to route_to({:controller=>"admin/posts", :action=>"create"})
end
it 'delete to destroy' do
  expect({:delete=>"admin/posts/1"}).to route_to({:controller=>"admin/posts", :action=>"destroy", :id=>"1"})
end
it 'patch to update' do
  expect({:patch=>"admin/posts/1"}).to route_to({:controller=>"admin/posts", :action=>"update", :id=>"1"})
end

I have a few custom matchers to help with this, currently the DSL is:
expect("admin/posts").to have_resourceful_routes

or
expect("users").to have_resourceful_routes(:shallow)

Would the RSpec team be interested in those matchers or is the use case here too specific?

You could implement and share these as a gem if you found others were interested, however it's not something we'd bring into rspec-rails. rspec-rails is a pretty thin wrapper around the test helpers rails provides, if they were to bring this into their helpers (probably as assert_resourceful_routes) we'd similarly then expose it.
As an aside however this isn't particularly useful testing, you're asserting that rails is doing it's job with 20 LOC in your tests for 1 LOC in your router, generally things like that don't need to be continually tested, it just adds to your build time with little chance of going wrong after it's committed.


Backscrape sc_u and scctapp_u

These are new scrapers, but their sources have data available back from 2003, we can backscrape them using the following commands
manage.py cl_back_scrape_opinions --courts juriscraper.opinions.united_states.state.sc_u  --verbosity 3
manage.py cl_back_scrape_opinions --courts juriscraper.opinions.united_states.state.scctapp_u  --verbosity 3

Part of #929

We now have 401 unpublished sc opinions.
However, from year 2003 up to May 2012, unpublished opinions are in HTML (ex1 ; ex2 ), which will need updating the scraper to support 'text/html' content type

After another run, with support for HTML files, we got 851 files for sc_u, from back to 2004

Older scctapp_u files are triggering the error described in this issue. Most recently https://github.com/freelawproject/courtlistener/issues/4570#issuecomment-2412474257
Last one was:
{
blocked_statuses: False,
case_dates: datetime.date(2007, 11, 7),
case_name_shorts: "Wilson",
case_names: "Peerless Insurance Company v. Wilson",
date_filed_is_approximate: False,
docket_numbers: "2007-UP-514",
download_urls: "https://www.sccourts.org/media/opinions/unpublishedopinions/HTMLFiles/COA/2007-UP-514.htm",
precedential_statuses: "Unpublished",
summaries: ""
}

Until that is solved,  I will avoid backscraping October, 2007 to July 2012, which consists mostly of HTML files


Constants for status codes, lookup for reason phrases.

#174

Looks lovely :shipit:

:shipit:

My 2 cents: the [Status] + code + name pattern is a bit hard to read and I'm not sure it's really useful: if you know the status code, you don't need to rely on a constant to use it. And if you wanna use a constant, that's often because you don't want to play with the status code directly.

@PinpointTownes has a point, should me model it after the crazy System.Net enum?

The enum naming assumed you (and everyone reading your code) were deeply familiar with the reason phrases and what the resulting status codes would be. If you just use reason phrases people have to guess the matching status code, where if you just use integers they have to guess the matching reason phrase.
I picked the Status404NotFound pattern because it lets you see both parts together for validation. It also lets you use intellisense like a lookup table where you can type in either the status code or the reason phrase and it immediately shows you the other half. It has Status on there because you can't start with digits.
Would you prefer NotFound404? I find that a little odd because it's the opposite of how it shows up in the response and spec. How about S404NotFound?

I like it just fine the way it is. I view the point of these as being to eliminate what appear to be magic numbers from the code. And I think this does that beautifully.

And for the record, I strongly dislike the System.Net StatusCode enum because it adds no value at all. It just takes one hard to understand piece of information and hides it behind another hard to understand piece of information.


Fixes:  the Content-Type requested by the client is not supported: multipart/form-data; boundary=--

Fixes #[derive(Multipart)] for struct, so it will work with #[derive(ApiRequest)]:
#[derive(ApiRequest)]
enum MyRequest {
    Json(Json<Data>),
    File(DataUpload),
}

#[derive(Multipart)]
struct DataUpload {
    file: Upload,
}

MyRequest will now correctly recognize content type multipart/form-data; boudary=--.... as part of MyRequest::File

Thanks! 🙂


Update Jenkinsfile to reduce chances of hung or failed builds

Based on a suggestion made by David, this updates the Jenkinsfile to effect two changes:

Disable concurrent builds of the same tag or commit
Add a timeout (90 minutes) as a safeguard against hung builds

These options are used in many other repos in our org to good effect.

Already incorporated in a different PR.


Support HTTP proxy

I've implemented this a few months ago and have been using this in a private project ever since.
Given I have no immediate use case besides this, I'll just dump this here for now and will postpone this until anybody sees fit.
https://github.com/clue-labs/http-client/compare/compat...clue-labs:proxy?expand=1

Thanks for this commit, it's great, but it's still lacking something.
I want to go thru a http proxy where authentication is required as user and pass, any way to add user and password ?
Thanks,
Alex

I want to go thru a http proxy where authentication is required as user and pass, any way to add user and password ?

You should be able to pass the Proxy-Authorization header yourself to each call, e.g. http://stackoverflow.com/questions/10023636/http-spec-proxy-authorization-and-authorization-headers
Eventually, we should parse the given proxy server URI and use the user name and password to pass the header automatically (also refs #4).

Thanks for the info, great commit!


stax transaction 'frozen' for many hours

Transaction ID
0x8adfa585f746537b0fd52dcb2ed763a1f58d78eecde8aab6cf25c35648bc3afe

This is not an Explorer issue, please ask for help in Discord


feat: change module download directory to reside at context root

This PR changes the module download and caching functionality to reside at the top level of multi-project repositories. This departs from the old logic of using a .infracost folder in each sub-project.
This change is significant as it departs from Terraform logic in favour of speed and efficiency. This change should speed up most Infracost runs by some measure, as we will no longer need to download multiple copies of modules per project or per multi-project context.
I've also added concurrency to the module loader to further boost download performance.
Changes

Infracosts .infracost/manifest.json renamed to .infracost/manifest.v2.json to accomodate the new structure. This means that we won't blow up previous Infracost CLIs that use the old manifest.json

The .infracost directory now lies at the top level of the Infracost working directory this means, with a project like such:
    dev/
    prod/
    modules/
    infracost.yml

running infracost breakdown --path . or infracost breakdown --config-file infracost.yml would place a .infracost directory:
    .infracost/ <== here
    dev/
    prod/
    modules/
    infracost.yml

as opposed to dev/.infracost and prod/.infracost

the manifest.json file changes all local module paths to be relative to the working directory. e.g. using a folder structure like so:
    .infracost/
       manifest.json
    dev/
       main.tf
    prod/
    modules/
    infracost.yml

Prior to this changedev/main.tf file including a module with source ../modules/local-module would output a manifest entry like:
    {
      "Key": "local-module",
      "Source": "../modules/local-module",
      "Dir": "../modules/local-module"
    }

it now is changed to hold a Dir of modules/local-module as the path is relative to the working directory.

when running in a multi-project environment the .infracost folder separates project manifests by a hash of it's name, so:
    dev/
    prod/
    modules/
    infracost.yml

would have two manifest.json files corresponding to both prod and dev

The .infracost download paths are now a hash of the module address and version. This means we don't have to clean or copy the download folder when more than one module uses the same source.

Various structs and functions like Loader, Cache and Disco have been modified to lock operation based on address/hostname.

The test failures look unrelated to these changes, so please ignore

As discussed

I've changed the version to reside in the file https://github.com/infracost/infracost/pull/2208/commits/c0a95d8f55fcc4b63b1afe717ffa39d8df4bd2c6
we're leaving the manifest per project change, as it's a bit too involved to have a singular manifest.json


Added a selection for the axon shape in FSLeyes

Fixes #432
Should be implemented when #399 is merged

@Stoyan-I-A we have merged PR #399. Just letting you know so you can work on this PR.
Thanks!

I would like https://github.com/neuropoly/axondeepseg/pull/507 to be merged before adding more options in the settings menu.

FYI PR #507 has been merged.

I'm thinking of creating a new PR that resolves #432 because this one is pretty old and I have made changes to the plugin file in between.

I created a new PR for this: https://github.com/neuropoly/axondeepseg/pull/541


ENH: integrate.nsum: elementwise evaluation of finite or infinite series

Reference issue
NA
What does this implement/fix?
_nsum is a private function for evaluating a (convergent) finite or infinite series.
This PR renames the private function to nsum and makes it public.
Features:

vectorized for array summation limits, step sizes, and arguments
evaluates the log of a sum from the log of the integrand (e.g. useful for logcdf of discrete distributions)
supports absolute and relative tolerances
provides an error estimate

Planned features:

Array API support.
method argument to allow, for example, series acceleration methods
Inclusion of the error of the partial sum (i.e. due to roundoff error of np.sum/scipy.special.logsumexp) in the error estimate

Additional information
Currently, when the number of terms is very large or infinite, a partial sum is computed and the remainder is approximated as an integral. This allows for an error bound on the true value of the sum using the theory behind the integral test for convergence.[^1]
Example 1:

gh-20648 noted that yulesimon.kurtosis was incorrect, and our tests didn't catch it because the generic implementation of discrete distribution moments was inaccurate for the case in question. nsum computes the moment accurately.
import numpy as np
from scipy import stats
from scipy.integrate import nsum

a = 5
res = stats.yulesimon._munp(4, a)
ref = stats.yulesimon.moment(4, a)
np.testing.assert_allclose(res, ref)

AssertionError:
# Not equal to tolerance rtol=1e-07, atol=0
# Mismatched elements: 1 / 1 (100%)
# Max absolute difference: 0.58139004
# Max relative difference: 0.01116269
#  x: array(51.501943)
#  y: array(52.083333)

rtol = 1e-12
res = nsum(lambda x: stats.yulesimon._pmf(x, a)*x**4, 1, np.inf, rtol=rtol)
np.testing.assert_allclose(res.sum, ref, rtol=rtol)

Example 2:

gh-8424 noted that stats.poisson.logcdf suffers from overflow and underflow issues. The first example given there is easy even just using logsumexp, but it's slightly more natural with nsum.
import numpy as np
from scipy import stats, integrate, special
stats.poisson.logcdf(5, 1000)  # -inf
# Using logsumexp
special.logsumexp(stats.poisson(1000).logpmf(np.arange(6)))  # -970.2437078462409
# Using nsum
integrate.nsum(stats.poisson(1000).logpmf, 0, 5, log=True)
# success: True
#  status: 0
#     sum: -0.48457218951277037
#   error: -36.52822557862992
#    nfev: 7

As noted further below (https://github.com/scipy/scipy/issues/8424#issuecomment-378035141), logsumexp has trouble when the (logarithmic) result is very close to -0.
stats.poisson.logcdf(250, 5)  # 0.0
# This is *better*, but read on...
special.logsumexp(stats.poisson(5).logpmf(np.arange(250)))  # -6.661338147750939e-16

The relative tolerance in non-log-space is very good - the CDF is very close to 1.0. But in log-space, the relative tolerance is poor: the true result is around -2.32210854e-322. nsum suffers from the same problem as logsumexp when the logcdf is evaluated from the left, but it also allows computation of the logsf:
res = integrate.nsum(stats.poisson._logpmf, 251, np.inf, args=(5,), log=True)
res.sum  # -740.5817295772861

If desired, a much better logcdf can be computed with log1p (or log1mexp gh-19021 to avoid having to think):
np.log1p(-np.exp(res.sum))  # -2.3e-322

Of course, at that point we're into the subnormal numbers, so perhaps I should have chosen a slightly different example, but I think it still makes the point.
[^1]: However, this only works if the function is positive and monotonically decreasing, so the function is documented as though the function must satisfy these requirements. When a method argument is added, this overally requirement will be relaxed, but each method will have its own requirements.

Example 3:

gh-4548 requested a wrapped normal distribution. nsum can be used to implement the wrapped version of any distribution pretty accurately, even if the tails are much longer than the normal distribution.
import numpy as np
import matplotlib.pyplot as plt
from scipy import stats
from scipy.integrate import nsum

dist = stats.laplace

def f(x):
    # `logpdf` can be implemented directly in terms of `dist._logpdf` without precision loss
    # `cdf`, `logcdf`, `ccdf`, and `logccdf` are just as simple
    def _f(k, x):
        return dist._pdf(x + k) + dist._pdf(x - k)

    step = 2*np.pi
    res = nsum(_f, step, np.inf, step=step, args=(x,))
    return res.sum + dist._pdf(x)

x = np.linspace(-np.pi, np.pi, 300)
y = f(x)
plt.plot(x, y, '-')
plt.show()

That might work. How about making that an immediate follow-up PR?

Why don't we revert the last commit, squash merge, and do an immediate follow-up with the renames?

Why don't we revert the last commit, squash merge, and do an immediate follow-up with the renames? Seems easier to get right.

Yeah. Sounds good to me.

👍 reverted

👍 reverted

Thanks. Now awaiting follow up


Enable caching of Nuget packages on AppVeyor

Enabling caching of the Nuget "packages" folder should result in less network traffic, and hopefully faster builds.

This seems like a really good idea. Thank you for looking into it.
-Brett


Add 9p kernel support

@PLangowski please, at least describe here why we are adding this configuration. Ideally it would be if this goes into commit message as a second input (so it is not displayed e.g. in git log --oneline format).

@TomaszAIR Done 16a8114c8655580eac63b10a0b4fb8de16744317


Upgrading MySQL version?

Is it possible via config.yml to upgrade the version of MySQL?
I am looking to upgrade to at least 5.7.7.. which still appears to be listed as a dev release and not a GA release.
http://dev.mysql.com/downloads/mysql/
The reason I am asking is I want to explore possibilities of using JSON data types in Drupal 8, without having to install Postgres. I initially did some R&D with Postgres, but it is quite a behemoth compared to MySQL.
Ref: http://mysqlserverteam.com/json-labs-release-overview/

Anything concerning upgrade / changing versions of MySQL should be addressed in upstream repos, like my geerlingguy.mysql role; otherwise, if there's a simple way to get later versions (e.g. adding a separate apt repository for the latest GA release), then that could be accepted—that's how I do PHP releases.
However, I don't plan on supporting non-stable versions of MySQL explicitly; PHP yes, MySQL probably not, unless there's some simple way.


Live Server reload keeping the scroll position

How can I make Live Server reload my page, but keep the scroll position on Chrome. For example, if I'm working on the footer, and the page reloads, it scrolls right up and I have to scroll back down every time.
I'm using version 70.0.3538.77 of Chrome and version 1.28.2 of VS Code.
Thanks a lot!

Can you produce something?

It's happening with me also, you can check the code I'm working on in my repo here.
I also noticed that this only happens when I update the CSS file. If I save the HTML being served it doesn't scroll back.
OS: Ubuntu 18.04.1 LTS
Chrome version: Version 70.0.3538.110 (Official Build) (64-bit)
VS Code version: 1.29.1

im facing the same issue too

Me either unfortunately, it's really annoying, I've tried set liveServer.settings.fullReload to true, but it didn't work too. Like gassiss wrote, the issue is happening with CSS files, HTML is fine.
OS: Windows 8.1
VS Code version: 1.30.1
Chrome version: 71.0.3578.98
Happy New Year!

https://github.com/tapio/live-server/issues/273#issuecomment-471229763

Thanks a lot for this @fanfare!  Thought there would be more complaints about this issue. Frustrating on every single refresh.
To apply to VS Code extension, navigate to:
~/.vscode/extensions/ritwickdey.liveserver-5.6.1/.vscode/extensions/ritwickdey.liveserver-5.6.1/node_modules/live-server/injected.html

@fanfare thanks, it works for me too.

Brackets preview is very good! Got a similar viewer for vscode?

Hey I got a solution! .... imagine in HTML you have a link-tag which import your style.css ,

set the full Reload in VSCode for LiveServer to true/enable it (If i am not clear enough search it)
under you css link tag open a style-tag and make your rest of your style sheet implementation there, now it is not jumping up everytime , and then cut paste these changes to your real css file.

I'm also really annoyed buy this.

I'm having the same problem and unticking the "Full reload" option doesn't work for me (on Mac Chrome).

I'm having the same problem and unticking the "Full reload" option doesn't work for me (on Mac Chrome).

I know this is kinda old but it might be useful for someone. Basically what I did is set body to min-height: 100% instead of height: 100%

2021 this problem still exists, why no fix

2021 this problem still exists, why dont they fix it by default

This is probably a problem with your website. Try checking if you can get scroll position using javascript.

2021 this problem still exists, why dont they fix it by default

This is probably a problem with your website. Try checking if you can get scroll position using javascript.

I think my issue is with SASS live extention

2021 this problem still exists, why dont they fix it by default

This is probably a problem with your website. Try checking if you can get scroll position using javascript.

I think my issue is with SASS live extention

Why did you put a thumb down on my post? Didn't my solution work for you?

How can I make Live Server reload my page, but keep the scroll position on Chrome. For example, if I'm working on the footer, and the page reloads, it scrolls right up and I have to scroll back down every time.
I'm using version 70.0.3538.77 of Chrome and version 1.28.2 of VS Code.
Thanks a lot!

What's  work for me is
Add this script to your html head section

Using --no-css-inject command line option when running live-server solves this issue for me. It reloads page instead of injecting CSS and my browser (Firefox) keeps scroll position the same during reloads.

body to min-height

DUDE!!
Thank you!!
This worked for me, when nothing else was
Thank you soo much.

I use sass and live sass compiler and have same problem. Then I activate css map option and page scroll don't reset on refresh.

I'm also really annoyed buy this.
EDIT:
I just had to tick the box in settings that says Settings: Full Reload. Do not need to hack it more. In Visual Code.

its really good advice its work for me thank u so much. 😇😇😇

Me either unfortunately, it's really annoying, I've tried set liveServer.settings.fullReload to true, but it didn't work too. Like gassiss wrote, the issue is happening with CSS files, HTML and JS are fine. OS: Windows 8.1 VS Code version: 1.30.1 Chrome version: 71.0.3578.98
Happy New Year!

Make sure to close and reopen live server for the effects to take place.

Using --no-css-inject command line option when running live-server solves this issue for me. It reloads page instead of injecting CSS and my browser (Firefox) keeps scroll position the same during reloads.

This works for me


Ignore case when detecting colors for interpolation.

Fixes #2359.

Do we need a /i flag on the rgb/hsl regular expression, too?

Yes, good point.  I’ve updated the patch to support case-insensitive RGB/HSL functional notation as defined in the CSS spec.


Move test into build job

The test job is just rebuilding the project for itself so might as well live in the same job as build

When we're ready to merge this I'll update the branch protection settings to look for CI/build-and-test rather than CI/build-app and CI/test-virtual


Second Screen

Continue the creation of the second screen

@nardkeza working through this now

@nardkeza Pull Request #5 has addressed this multiple pages issue and linked with the navigation on the side panel. Let me know what you think


account: update grants in-place instead of remove/add method

Terraform supports updating a resource in-place if a Terraform schema
key is not marked as ForceNew and an Update method is provided. In
fact, it is implemented for 2 resources already:

https://github.com/chanzuckerberg/terraform-provider-snowflake/blob/main/pkg/resources/schema_grant.go#L90
https://github.com/chanzuckerberg/terraform-provider-snowflake/blob/main/pkg/resources/role_grants.go#L18

This PR aims to add in-place support for account grants.
We're testing this method at @peak for a while. This is critical because
'force delete/add' method makes services hiccup while trying to access a
Snowflake resource.
I plan to add more of this for:

database grants
table grants
view grants
warehouse grants

also update:

schema grants
role grants

to have the same structure as the others if this PR looks good to you.
Updates #137

/ok-to-test sha=018d623


[Bug][SDK] Go SDK load balance logic not perfect, causing consumption to stop suddenly

What happened
consumption suddenly stop

What you expected to happen

If the old code fails to register to Master again, there will be no further actions. It needs to be changed to continue retrying indefinitely upon failure.

OfferEventAndNotify is responsible for receiving rebalance events, while OfferEventResult handles the results of those events. Therefore, the former should definitely not operate on rebalanceResults, nor should it reprocess the header of rebalanceResults as if it were a newly received event. Otherwise, if the result of the previous event has not been reported yet when the second event arrives, the subsequent processRebalanceEvent will end up processing the first event again, causing the second received event to be discarded.

ip_selector directly returns the domain name, while the connection pool maintains one connection per address, and it will not reconnect as long as the packet sending and receiving are normal. If there is a cluster switch, as long as the old master is sending and receiving packets normally, even if the registration fails, the TCP connection will not reconnect. Therefore, it is necessary to resolve the domain name into an IP address before returning it. During a cluster switch, a new IP can be returned by re-selecting.

Here, there is no check to see if rsp's success is true. If a cluster change causes the master to no longer recognize the client, it will continue to send heartbeats without re-registering.

How to reproduce
when master changes
Environment
No response
InLong version
master
InLong Component
InLong TubeMQ
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

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

Thanks @yuexihan


Cursor shows briefly when capturing mouse

Tested versions
Reproducible in 4.3, 4.0, 3.5.2
System information
Godot v4.3.stable.mono - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3070 (NVIDIA; <IP_ADDRESS>81) - Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 Threads)
Issue description
When setting mouse mode to captured, the cursor can be briefly seen at the center of the window.

It doesn't seem to matter when mouse mode is changed (_init, _ready, process).
Happens more consistently when cursor is outside the window, but still very often when inside.
Setting mouse mode to hidden before setting it to captured seems to fix it for when the mouse is inside the window, but not when outside

Steps to reproduce

Run minimal reproduction project
Place cursor either inside or outside window, but away from the center
Press ui_left to capture mouse
See that the cursor briefly shows at the center of the window

Minimal reproduction project (MRP)
capture-mouse-flicker.zip

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

DO NOT DOWNLOAD ANYTHING I HAVE POSTED AS A COMMENT MY ACCOUNT WAS COMPROMISED AND BEING USED TO COMPROMISE OTHERS
I AM WORKING ON DELETING AND POSTING THIS ON ALL THREADS IN WHICH THEY USED MY ACCOUNT TO POST THIS

Is this really a bug or a general limitation? Judging by the tested versions, it seems to be very long-standing. How do other OSes handle this, I wonder? Could use some testing.

It's not doing this on macOS. Seem like it's caused the order of operations, in case of Windows it's setting position and capture before hiding cursor.


how does Course class work

Summary
Course Class
Is the course name the key and assessment the value?
then how does it work when:
new Course("CS2040")
does this mean im creating a map with a key without value
And when i do this
new Course("CS2040").put(new Assessment("Lab1", "B")).put(new Assessment("Lab2","A+"))
does it mean im creating a map with 2 values?

For the first two levels, the Course class contain its name and an ImmutableMap for the Assessments.
So when initially creating a Course("CS2040") it will be just an empty ImmutableMap. To get the result output by this function call, you can use the name attribute of the Course class.
When you new Course("CS2040").put(new Assessment("Lab1", "B")).put(new Assessment("Lab2","A+")), it should create a map with two key and two values, one for each respectively. This will return you a Course which stores all the assessments.
Keyable only has getKey() method, I read it as a hint that the ImmutableMap should take the getKey() from Assessment as the key and the entire Assessment object as the value of the respective key.

Hello I'm also a bit lost for this part.

Course class contain its name and an ImmutableMap for the Assessments.

Does this mean Course is not an ImmutableMap but it contains an ImmutableMap of Assessments? Thank you.

jshell> new Course("CS2040").put(new Assessment("Lab1", "B"))
$76 ==> CS2040: {CS2040={Lab1:B}}
This is my current output, how do I fix my code so I can just extract the value and not the entire immutableMap?

implement your own toString(), dont need to extend immutableMap

A Course object should contain the name of the course followed by a map of assessments associated with it. When you create a new course, you just assign the name of the course. For example, when creating new Course("CS2030"):
{
    courseName: "CS2030",
    assessmentMap: {}
}

When you call the put method to add a new assessment, it would add the Assessment object into the ImmutableMap (via the put method of the ImmutableMap class).
After calling new Course("CS2040").put(new Assessment("Lab1", "B")).put(new Assessment("Lab2","A+")), the course object would look something like this:
{
    courseName: "CS2030",
    assessmentMap: {
        "Lab1": "B",
        "Lab2": "A+",
    }
}

Does this mean Course is not an ImmutableMap but it contains an ImmutableMap of Assessments? Thank you.

Yes, for the first 2 parts of Lab 4. But for level 3 and beyond, a Course would become a KeyableMap

This is my current output, how do I fix my code so I can just extract the value and not the entire immutableMap?

Override the toString() method for the Course class to show the desired output as shown in the lab 4

when u do new Course("2040"), u are creating an object w name course and and an empty immutable map . When u put assessments, u are adding new entries into your immutable map in your course class


using custom view for sign up

hi,
how can i create my own view for sign up, and does not use the library view??
is it avilable?

Hi @walaagomaa18 Currently its unavailable. But you can tweak the page as you like by adding the complete project as a module instead of adding aar file.
Hope that helps! I have plans to improve with custom layout options.

@walaagomaa18 The library is updated to version 1.x and can now you can have your own custom UI for login page. Check out README for more details. Project Wiki will be ready soon.


Fix up USFM pretranslations for Parallel corpus

#522

This change is

Codecov Report
Attention: Patch coverage is 76.92308% with 3 lines in your changes missing coverage. Please review.

Project coverage is 56.93%. Comparing base (6e9b736) to head (46b128e).

Files with missing lines
Patch %
Lines

...rval.Translation/Services/PretranslationService.cs
76.92%
2 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #529      +/-   ##
==========================================
+ Coverage   56.91%   56.93%   +0.01%
==========================================
  Files         299      299
  Lines       15706    15715       +9
  Branches     2168     2169       +1
==========================================
+ Hits         8939     8947       +8
- Misses       6119     6120       +1
  Partials      648      648

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

src/Serval/src/Serval.Translation/Services/PretranslationService.cs line 55 at r1 (raw file):
Previously, Enkidu93 (Eli C. Lowry) wrote…
What about a situation in which we're using a parallel corpus that has two source (or target) corpora: one that includes MAT & MRK and another that includes MRK & JHN. Won't be run into issues if we try and insert pretranslations into the USFM for JHN from a project that isn't the one that contains JHN?

Resolved through the meetings we had.


How does Gluon-ts deal with NaN value?

Q1.
Hi, I'm looking for processing NaN value in your DeepAR code.
Can you let me know the mechanism or location of code?
Q2.
What is lags_seq in DeepAR? I cannot find details in DeepAR paper.
According to my understanding, it seems to be similar with data augmentation by lagging target data.
Thanks a lot.

Hello @HanSangJun,
there is no persistent strategy that models use. I.e., each model can decide how it wants to support NaN-values, if it even does so.
You can find the used default-lags here, which are derived from the frequency of the data: https://github.com/awslabs/gluon-ts/blob/fe065b42051225a8df317db7354d41e266adc1ea/src/gluonts/time_feature/lag.py#L30-L123
The lags_seq parameter allows to specify custom lag-values, which (as far as I can see) are not currently used in the code-base.


[Improvement] Internationalization

Insomnia Version: ___
Operating System: ___

Details

I think internationalization is a great idea as Insomnia gets more popular.
Can you elaborate on what parts of the app should be covered first?


Horizontal scrolling is not working

Is the bug present when using the default Obsidian theme?
No
Is the bug present when snippets and plugins are disabled?
Yes
Minimal theme version
7.0.5
Describe the bug
After the recent updates, I cannot see the horizontal scrolling bar on wide pages, which used to appear at the bottom of Obsidian, like it does in the Default theme.
This makes it pretty uncomfortable to operate on pages with wide Markdown tables in "Source mode" and “Live Preview”, as there is no horizontal scroll bar. Moreover, using the right arrow key does not make Obsidian to follow the cursor, same as selecting the text to the outside of the visible editing area.
The only solution that works is to e.g. use the arrow key to move further away in a table, and then write any character so that the focus shifts to the position of the cursor.
Here is a sample of a wide Markdown table for testing:
| aaa1                                                                                | aaa2                                                                                | aaa3                                  | aaa4                                  | aaa5                                  | aaa6                                  |
| -------------------------------------------------------------------------------------------------------------------------- | -------------------------------------------------------------------------------------------------------------------------- | ------------------------------------- | ------------------------------------- | ------------------------------------- | ------------------------------------- |
| aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa |
| aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa | aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa |

Debug info
SYSTEM INFO:
	Obsidian version: v1.3.5
	Installer version: v1.3.5
	Operating system: Windows 10 Pro 10.0.22621
	Login status: logged in
	Catalyst license: vip
	Insider build toggle: off
	Live preview: on
	Legacy editor: off
	Base theme: dark
	Community theme: Minimal v7.0.5
	Snippets enabled: 0
	Restricted mode: off
	Plugins installed: 18
	Plugins enabled: 18
		1: Text Snippets v0.1.2
		2: Templater v1.16.0
		3: Style Settings v1.0.6
		4: Minimal Theme Settings v6.3.1
		5: Latex Suite v1.7.3
		6: Icon Folder v2.1.2
		7: Execute Code v1.8.1
		8: Emoji Shortcodes v2.2.0
		9: Copy Block Link v1.0.4
		10: Filename Heading Sync v1.9.0
		11: Natural Language Dates v0.6.1
		12: Find orphaned files and broken links v1.9.0
		13: Recent Files v1.3.6
		14: Linter v1.16.0
		15: Smart Typography v1.0.18
		16: Paste URL into selection v1.7.0
		17: Advanced Tables v0.18.1
		18: Calendar v1.5.10

RECOMMENDATIONS:
	Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.
	Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.

I have also noticed that sometimes the left side of the page is being cut a bit (as in the screenshot below), and then the solution is to reopen the tab/page:

Maybe it is related, but if not, I may open a separate issue.