uv run pytest does not work

Ten words or less
uv run pytest does not work in my project.
Description
I have a very small project trying our uv and the next-gen tooling provided by astral. (I ❤ the work you guys are doing). I am coming from a poetry background and trying to convince my workplace to transition from poetry|conda|nothing to uv.
You can find the entire, source code of the project here which looks something like this,
Structure
.
├── LICENSE
├── README.md
├── pyproject.toml
├── releaseer
│   ├── __init__.py
│   ├── app.py
│   └── names.yaml
├── tests
│   ├── conftest.py
│   └── test_base.py
└── uv.lock

# Contents of releaseer/app.py
from typing import Dict, List
from litestar import Litestar, get
from yaml import load, Loader

@get("/")
async def hello() -> Dict[str, str]:
    return {"message": "Hello from releaseer!"}

@get("/names")
async def names() -> Dict[str, List[str]]:
    with open("names.yaml") as f:
        names = load(f, Loader=Loader)
    return names

@get("/healthy")
async def healthy() -> bool:
    return True

app = Litestar(route_handlers=[hello, names, healthy])

# Contents of tests/conftest.py

from typing import TYPE_CHECKING
from collections.abc import AsyncIterator

import pytest

from litestar.testing import AsyncTestClient

from releaseer.app import app

if TYPE_CHECKING:
    from litestar import Litestar

@pytest.fixture(scope="function")
async def client() -> AsyncIterator[AsyncTestClient[Litestar]]:
    async with AsyncTestClient(app) as client:
        yield client

The Error
uv sync --verbose
shinybrar@saturn releaseer % uv sync --verbose
DEBUG uv 0.4.6 (Homebrew 2024-09-05)
DEBUG Found project root: `/Users/shinybrar/Workspace/releaseer`
DEBUG No workspace root found, using project root
DEBUG Reading requests from `/Users/shinybrar/Workspace/releaseer/.python-version`
DEBUG The virtual environment's Python version satisfies `Python 3.12`
DEBUG Using request timeout of 30s
DEBUG Acquired lock for `/Users/shinybrar/.cache/uv/built-wheels-v3/path/fbac97741b8da1f8`
DEBUG Found static `pyproject.toml` for: releaseer @ file:///Users/shinybrar/Workspace/releaseer
DEBUG No workspace root found, using project root
DEBUG Released lock at `/Users/shinybrar/.cache/uv/built-wheels-v3/path/fbac97741b8da1f8/.lock`
DEBUG Existing `uv.lock` satisfies workspace requirements
Resolved 58 packages in 1ms
DEBUG Using request timeout of 30s
DEBUG Requirement already installed: annotated-types==0.7.0
DEBUG Requirement already installed: anyio==4.4.0
DEBUG Requirement already installed: argcomplete==3.5.0
DEBUG Requirement already installed: certifi==2024.8.30
DEBUG Requirement already installed: charset-normalizer==3.3.2
DEBUG Requirement already installed: click==8.1.7
DEBUG Requirement already installed: colorama==0.4.6
DEBUG Requirement already installed: commitizen==3.29.0
DEBUG Requirement already installed: decli==0.6.2
DEBUG Requirement already installed: editorconfig==0.12.4
DEBUG Requirement already installed: faker==28.4.1
DEBUG Requirement already installed: fast-query-parsers==1.0.3
DEBUG Requirement already installed: h11==0.14.0
DEBUG Requirement already installed: httpcore==1.0.5
DEBUG Requirement already installed: httptools==0.6.1
DEBUG Requirement already installed: httpx==0.27.2
DEBUG Requirement already installed: idna==3.8
DEBUG Requirement already installed: iniconfig==2.0.0
DEBUG Requirement already installed: jinja2==3.1.4
DEBUG Requirement already installed: jsbeautifier==1.15.1
DEBUG Requirement already installed: litestar==2.11.0
DEBUG Requirement already installed: markdown-it-py==3.0.0
DEBUG Requirement already installed: markupsafe==2.1.5
DEBUG Requirement already installed: mdurl==0.1.2
DEBUG Requirement already installed: msgspec==0.18.6
DEBUG Requirement already installed: multidict==6.0.5
DEBUG Requirement already installed: packaging==24.1
DEBUG Requirement already installed: pluggy==1.5.0
DEBUG Requirement already installed: polyfactory==2.16.2
DEBUG Requirement already installed: prompt-toolkit==3.0.36
DEBUG Requirement already installed: pydantic==2.9.0
DEBUG Requirement already installed: pydantic-core==2.23.2
DEBUG Requirement already installed: pygments==2.18.0
DEBUG Requirement already installed: pytest==8.3.2
DEBUG Requirement already installed: python-dateutil==2.9.0.post0
DEBUG Requirement already installed: python-dotenv==1.0.1
DEBUG Requirement already installed: pyyaml==6.0.2
DEBUG Requirement already installed: questionary==2.0.1
DEBUG Requirement already installed: rich==13.8.0
DEBUG Requirement already installed: rich-click==1.8.3
DEBUG Requirement already installed: ruff==0.6.4
DEBUG Requirement already installed: six==1.16.0
DEBUG Requirement already installed: sniffio==1.3.1
DEBUG Requirement already installed: termcolor==2.4.0
DEBUG Requirement already installed: tomlkit==0.13.2
DEBUG Requirement already installed: typing-extensions==4.12.2
DEBUG Requirement already installed: tzdata==2024.1
DEBUG Requirement already installed: uvicorn==0.30.6
DEBUG Requirement already installed: uvloop==0.20.0
DEBUG Requirement already installed: watchfiles==0.24.0
DEBUG Requirement already installed: wcwidth==0.2.13
DEBUG Requirement already installed: websockets==13.0.1
Audited 52 packages in 0.37ms

shinybrar@saturn releaseer % uv run pytest --verbose
ImportError while loading conftest '/Users/shinybrar/Workspace/releaseer/tests/conftest.py'.
tests/conftest.py:8: in <module>
    from releaseer.app import app
E   ModuleNotFoundError: No module named 'releaseer'

I am hoping, I am missing something obvious and this is something really trivial.

Interesting. I seem to get the same error when I run directly after activating the virtualenv, though:
❯ pytest
ImportError while loading conftest '/Users/crmarsh/workspace/puffin/releaseer/tests/conftest.py'.
tests/conftest.py:8: in <module>
    from releaseer.app import app
E   ModuleNotFoundError: No module named 'releaseer'

I'll have to take a look at how this typically works.

Adding this to pyproject.toml resolves it for me:
[tool.pytest.ini_options]
pythonpath = ["."]

Creating tests/__init__.py also resolves it, which I think is the right fix based on https://docs.pytest.org/en/stable/explanation/pythonpath.html.

Thank you so much @charliermarsh -- that was a simple enough fix. i.e. The correct way would be to use tests/__init__.py. Thank you so much for the help!

No problem, I learned something too!

I had the same problem, the project was missing [build-system] section

Same problem here: I ran uv run pytest as suggested by the docs but it failed to import a package installed in the virtual environment.
I've added the following to an __init__.py placed in the folder of the file with the failing import:
import sys
print(sys.path)

And I got ['~/Projects/MyProject', '~/.local/share/uv/tools/pytest/bin', '~/.local/share/uv/python/cpython-3.13.0-macos-aarch64-none/lib/python313.zip', '~/.local/share/uv/python/cpython-3.13.0-macos-aarch64-none/lib/python3.13', '~/.local/share/uv/python/cpython-3.13.0-macos-aarch64-none/lib/python3.13/lib-dynload', '~/.local/share/uv/tools/pytest/lib/python3.13/site-packages']
(NOTE: I've replaced my home folder with ~ in this comment, the result were absolute paths).
I can see '~/Projects/MyProject' which I configured by adding
[tool.pytest.ini_options]
pythonpath = ["."]
addopts = ["--import-mode=importlib", ]

but no virtual env.
However, if I run uv run python the sys.path in the console does include the virtual-environment folder '~/Projects/MyProject/.venv/lib/python3.13/site-packages' as last item.
I fixed the issue adding the virtual-env folder under [tool.pytest.ini_options] as well.
VERSIONS:
uv 0.5.1 (dac8c41af 2024-11-09) (built locally with cargo)
uv run python --version
Python 3.13.0

I shouldn't need to add the virtual env folder under the tools settings, isn't it?
Do you have any idea or request?

I'm hitting a similar issue and none of the above suggestions work. I tried adding __init__.py to my tests module, and adding pythonpath == "." to pyproject.toml. Either way, the tests pick up the project's default venv, ignoring the one-time venv uv run creates to satisfy --with options.

uv add pytest --dev fixed my problem.
I found that when running uv run pytest without pytest in dependencies, it somehow does not use the venv uv has set up.


minor refactor

move specaug.py out of asr folder since its a general operation compatible with all the downstream tasks
from dictionary import Dictionary --> from .dictionary import Dictionary
make train/dev/test sets configurable
add model (torchaudio.models.wav2letter)
default 12 workers for _build_dictionary

when dev we wish to use both dev-clean and dev-other, while when testing we might want to separate test-clean & test-other


Change log for 1.0.0-beta5

beta4 is also not very informative.

Hey, working on something:
In the meantime, the new version includes these three PRs:

#185
#183
#178


Roundtrip failures for some of the fonts I'm working with.

Attached is a report generated by roundtrip.py when I run it on a ttf:
report.txt
Something is getting messed up in the kern subtables, but I don't understand font internals well enough to know if it's something wrong with the ttf I was given or a bug in fonttools.

@jarrodlombardo can you send me the font via email?

Had to wait for some approvals about sharing the font. I'll email now.

closing in favour of #1089


445 kitra710 support

Hi there, just a heads up – we’ll be moving support for free users to the community forums
this Monday, March 20. The main difference is that your questions and our developers' answers will be public instead of private, building a knowledge base for everyone.
You can take this question to the forums now, I’ll meet you there to answer it. If not, I’ll help you out here, but know that future questions should be directed to the forums.
P.S. if you’d like to continue accessing private support after the change, we can upgrade you to a paid plan and you won’t see any difference.

You can view, comment on, or merge this pull request online at:

https://github.com/resin-io/docs/pull/447

Commit Summary
initial support for kitra710>add terminal image

File Changes
M> config/dictionaries/device.json> (9)> M> config/navigation.txt> (1)> M> shared/getDeviceOnDash/artik710.md> (18)> A> shared/getDeviceOnDash/kitra710.md> (29)> A> shared/whatYouNeed/kitra710.md> (12)> A> static/img/common/webterminal/terminal-kitra710.png> (0)> A> static/img/device/kitra710.svg> (1)> A> static/img/kitra/710.png> (0)

Patch Links:
https://github.com/resin-io/docs/pull/447.patch>https://github.com/resin-io/docs/pull/447.diff

—

You are receiving this because you are subscribed to this thread.

Reply to this email directly, view it on GitHub, or mute the thread.


unable to run node report

$ faraday --version
faraday version 0.2.0-alpha commit=v0.2.0-alpha-69-g5bb8bc008fdbaecfec43e76bab117a75ca2588a1

$ lnd --version
lnd version 0.11.0-beta commit=v0.11.0-beta-30-gc3821e5ad19e374ef2454cc9c7bf81b74408b8ca

$ cat ~/.lnd/lnd.conf | grep bitcoin.node
bitcoin.node=neutrino

$ frcli nodereport
[frcli] rpc error: code = Unknown desc = bitcoin node required

The README has just been updated to document this new requirement.
The NodeReport endpoint requires a connection to a bitcoind or btcd node to fully report on on chain fees. This is required for sweeps and channel closes where the fees for the transaction are taken from existing inputs which lnd's wallet does not know belong to us. Fees report as 0, when we've actually used some of our input as fees. For reliable accounting, we need to account for these fees, so we require a bitcoin node to lookup the values of these inputs and report on their fees.
Is connecting faraday to a bitcoin node going to be an issue for you? I've been considering adding a flag to the report --nonode which allows you to generate best-effort reports, accepting that they will not have fees reported for sweeps/channel closes. Ideally, we will fix this in lnd's wallet and then remove the need for a node, but this could be a workaround in the meantime?

I saw the readme changes when tracking down the cause of the error after the most recent commits.
It would be nice to be able to run the command via --nonode or whatever solution is best for neutrino nodes.

Updated to the latest commit. Getting a new error:
$ frcli --version
frcli version 0.2.0-alpha commit=v0.2.0-alpha-90-g2415b5d866838880f7ddc0f57a79e3c9e589dbda
$ frcli nodereport
[frcli] open : no such file or directory

[frcli] open : no such file or directory

Are you running on testnet/regtest? We've recently added tls, so you'll need to specify --network=regtest/testnet when using frcli. But this should have a better error (since the addition of TLS is new) so I will update that).
If you're running on mainnet however, it should just work.

@githorray the latest merge should allow you to run with neutrino, the report will exclude fees for sweep/ close tx fees but the txids will be logged so you know exactly what's missing.
Feel free to reopen the issue if you don't come right, or ping me on the lnd slack!

I am running on mainnet.


Expose file metadata (or stream length)?

I have a situation where I am using FileStream and I want to check the file length but I don't want to create a new File just to read the meta data. We could:

Expose &File in FileStream
Expose self.file.metadata() in FileStream
Support a len() method on the Stream trait

My preference would be for 3) but I wonder what you think?
I can imagine a situation where there may be streams that don't have a known length so if we do support a len() function maybe it should look like:
fn len(&self) -> Option<usize>;

But we do know the length at the moment for MemoryStream and FileStream so maybe cross that bridge later.

I have submitted #32 to address this, hope you think this is worthwhile too 🙏


.github: Fix concurrency group comment triggers

The first commit fixes the concurrency groups to avoid random PR comments cancelling ongoing end-to-end tests. The second commit enables skipping the paths-filter job when possible. See commit descriptions for details.
Fixes: https://github.com/cilium/cilium/pull/16199.

Note the second commit will also have the side effect of avoiding running the paths-filter job on forks, which if you updated your fork recently, you may have noticed is currently happening.

Removing the label for backport, the relevant parts of the GitHub actions are not in 1.10.

@qmonnet If they are not, I think they should be added. Maybe a PR is missing its needs-backport/1.10 label?

I can have a look at it tomorrow, and backport in a later PR if necessary.


MODOAIPMH-72 Update inventory interface version

Purpose
MODOAIPMH-72 Update inventory interface version
Approach

Adding 7.0 dependency along with 6.0 for instance-storage interface (it is safe because the logic does not work with notes)
Just in case removing notes from mock files

I noticed that some mock files still have empty notes arrays... was this intentional?

Taking into account that notes are going to be still an array (but element type is changing from string to object) I decided that having empty arrays is just fine (and it reduces number of files in the PR so it is easier to review :) )


Codebase cleanup

[ ] Remove all "using namespace std" from any header files

[ ] Add License file for freeBSD license

[ ] Create a generic file comment header and add to every source file.

[ ] Audit all the decisions to use class vs struct

[ ] Go through all 'TODO's in code and resolve or move to a github issue

[ ] Change code to use References rather than pointers when possible

[ ] Add const when I can to the api.

[ ] Remove all using namespace xxx from any header files
is probably more along the lines of what we want.

Some of these are fixed by #200


Not an issue - question on applicability/ future research with music scores / music glyphs

I use an OCR software to recognize music https://www.neuratron.com/photoscore.htm
it fails miserably if the music sheet is a little off due to bad scanning - works perfectly on computer generated pdfs

I understand that there is music glyphs - https://www.notationcentral.com/product/musglyphs/
so I wonder if this glyph technique / or new research could be done to rebuild scanned pdfs into fresh pdf documents.
in this scenario - text is not so important -  it would be image to image - but I need a base model that is suitably trained on music scores.    Resolutions are a big problem though.

Good point! We would like to see what should we do to help you on generating high-quality music glyphs.

related
https://www.kaggle.com/competitions/denoising-dirty-documents/code


Sandra kriemann profile add

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

advocates/media/profiles/sandra-kriemann.png
:white_check_mark:Succeeded
View

advocates/sandra-kriemann.yml
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

advocates/media/profiles/sandra-kriemann.png
:white_check_mark:Succeeded
View

advocates/sandra-kriemann.yml
:white_check_mark:Succeeded
View

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


xml_node::select_node(s) will act on the entire document

pugi::xml_node::select_nodes (and related) will act on the entire document rather than the node it was called on.
The following code
	pugi::xml_document doc;
	if (auto result = doc.load_file("testcase"); !result) {
		std::cerr << "Failed to parse file: " << result.description() << " at " << result.offset << " near\n";
		return;
	}

	auto container = doc.select_node("//div[@class=\"container\"]").node();
	auto tags = container.select_nodes("//div[@class=\"alpha\"]");
	int i = 0;
	std::array<std::string, 3> want{ "foo", "bar", "foobar" };
	for (auto&& tag : tags) {
		auto node = tag.node();
		std::cout << "=== Processing node:\n";
		node.print(std::cout);
		std::cout << "\n\n";
		std::string info = node.select_node("//div[@class=\"info\"]").node().text().as_string();
		std::cout << "Have: " << info << " Want: " << want[i] << '\n';
		assert(info == want[i++]);
	}

with testcase having the following content:

unrelated stuff

cool shit

foo

bar

foobar

should not fail the assertion but does. In fact it will always get the unrelated stuff part.
Line 14 will in fact print out the correct node that the XPath should be applied to, but isn't.
This is a very much simplified example showing the issue.

This is correct as per XPath spec, see https://www.w3.org/TR/1999/REC-xpath-19991116/:
//para selects all the para descendants of the document root and thus selects all para elements in the same document as the context node
.//para selects the para element descendants of the context node

In other words, you probably want to use .//div instead.

Not very intuitive to have a node/object that a function is executed on, go to an entirely different object to run on...
Anyways I guess there is some obscure situation where someone somewhere would want the current one so

This is how XPath works - queries that start with // apply to the entire document that the context node belongs to. This isn't something that pugixml chooses as an API behavior, this is simply how every conformant XPath implementation must work.

Yeah, it was just not really intuitive. In any case it can be worked with. Thanks for the explanation.

That would preclude the use of correct (in this case) XPath expression “.//node”.
While use of // in a non-document context is certainly unintuitive, it can’t be treated as an error as that would violate XPath spec, and there’s no warning mechanism that can be used.


Loosen type

@visr @c42f

I'll squash and rebase after approval

oops sorry wasn't aware you were already onto it @visr.

No worries I had only made a small start, you've made all changes I had in mind as well. Apart from adding doc/.ipynb_checkpoints/RasterioGuide-checkpoint.ipynb.


Add a TODO file.

The TODO tracks open issues and tasks to be done.  The intent is that it should be updated as tasks are completed.

Instead of a TODO.txt file in the git repository, I've added the task list as issues in the github issue tracker.


Question: Redux in background

Initially I apologies for not follow proper conduct but I have exhausted all other avenues to find an answer. I have an issue where when the app is in the background or transitioning to the background my updates to the redux store do not happen. This is happening at the moment due to events triggered by React-Native-CallKeep when the app is in the background in which I need to update my store on the status of calls. Any pointers in the general direction of an answer or a work around would be greatly appreciated.

System:
OS: macOS 10.15.5
CPU: (4) x64 Intel(R) Core(TM) i5-8210Y CPU @ 1.60GHz
Memory: 54.81 MB / 8.00 GB
Shell: 5.7.1 - /bin/zsh
Binaries:
Node: 13.7.0 - /usr/local/bin/node
Yarn: 1.21.1 - /usr/local/bin/yarn
npm: 6.14.6 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.9.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 13.6, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
Android SDK:
API Levels: 28, 29
Build Tools: 28.0.3, 29.0.2
System Images: android-28 | Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 3.5 AI-191.8<IP_ADDRESS>77832
Xcode: 11.6/11E708 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_232 - /usr/bin/javac
Python: 2.7.16 - /usr/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.11.0 => 16.11.0
react-native: 0.62.2 => 0.62.2
npmGlobalPackages:
react-native: Not Found


[Bug]: 'Insufficient Savings' error displays when attempting to pay a LN invoice with no spending balance

Describe the bug
Using a wallet with no spending balance, when attempting to pay a LN invoice the error displayed is 'Insufficient Savings' and the total amount of the invoice is displayed for '₿X more needed to pay the Bitcoin invoice'. This error message should refer to 'Spending'.
Reproduce

Using a wallet that has onchain funds but no LN connection
Attempt to pay an LN invoice (it was a ₿35_000 cjit channel open on a different wallet)
Observe 'Insufficient Savings' error

Screenshots / Recording
https://github.com/synonymdev/bitkit/assets/74595920/09c04fa9-ee30-4da2-8726-dbc02e318e54
Operating system
Android 13 TKQ1.221114.001
Bitkit version
a9a83572aebf75ec5d217d0b3fa124b2b4a34fad
Log output
No response

Verified in v124
'Insufficient Spending Balance' toast displayed


Project file improvements

Remove meaningless and deprecated items
Use slnf file to split solutions
Introduced in VS2019. Rider also supports it (verified on 2019.2.2).
Avoids potential inconsistency between solutions.
Simplify duplicated properties.
Based on my experience in UWP, the legacy project system should handle it correctly.
Utilize shared props files.
Rider says legacy project system doesn't import Directory.Build.props at all, but I remember that it should do so.
Use local dotnet tools manifest
This minimizes influences to global environment, and stores tool version in one place.
Build Xamarin with VS2019
Since desktop requires 2019, developer will be unhappy to have both version installed.
Moving to separated PR.

Verified compatibility

Building .NET core projects in VS and Rider.
Loading projects based on filter in VS and Rider.
InspectCode knows slnf.
It seems supporting Xamarin with VS2019. Leave the behavior as-is to analysis desktop projects only.
CI is happy.

To do

Verify dotSettings are recognized correctly.
Verify compatibility with VS for Mac. I expect it to behave the same with VS as a first party stuff.

Breaking

VS2017 doesn't know slnf. Since it's already dropped in fact by .NET Core 3.0, it's safe to drop it in other places.
dotnet build doesn't know slnf. This breaks dotnet build .sln, but that only works for desktop projects today because dotnet CLI can't build Xamarin projects. dotnet build on single .NET Core csproj won't get broken.
This will require some change and workaround in build script.

Just gonna mention that VS for mac 2019 doesn't support slnf's yet, probably something we'll want to report.


Migrations for Topics, Modules, Questions, Feedback and Users

Description
Created Knex migrations files which consist of DB tables for all Topics, Modules, Questions, Feedback and Users - with team Yellow includes (@1620prabn , @Elia-Git , @yukafjaponesa )
Fixes #63
How to test?
in text editor.
Checklist

[ x] I have performed a self-review of my own code
[ x] I have commented my code, particularly in hard-to-understand areas
[ x] I have made corresponding changes to the documentation
[ x] This PR is ready to be merged and not breaking any other functionality

If this PR has 4 migrations files for 4 different tables why do you call it Db migration modules ?

Also the description of the PR only mentions Modules when you have 3 tables more. Why is that?

If this PR has 4 migrations files for 4 different tables why do you call it Db migration modules ?

titile is changed

But to not sound too negative, this is a 90% great job. You were almost at the finish line! 👍


G1GC Collections with two sets of parenthesized parameters are not matched

Describe the bug
Given a GC event similar to the following:
[6.837s][info][gc] GC(3) Pause Young (Normal) (G1 Evacuation Pause) 79M->27M(1024M) 2.929ms

The UnifiedG1GCPatterns.G1_COLLECTION GCParseRule does not seem to take the second parameter (G1 Evacuation Pause) into account, and these events are missed.
There is no other similar GCParseRules to handle this case that I can find.  I'm not entirely clear on where the (G1 Evacuation Pause) text is coming from, though a "default" GC log with the Corretto JDK seems to use the same format, so this may be a Corretto thing?
No error is logged when running with the debug flag.
To Reproduce
Steps to reproduce the behavior:

Attempt to analyze the provided file with the library. (Sample app can be used)
gc.out.4.zip

Expected behavior

Events matching the above format are processed by the library.
Sample output is non-zero as events have been parsed.

Desktop (please complete the following information):

OS: Windows 11
Java Version: Amazon Corretto 11.0.19_7

From looking at some of the source code of OpenJDK vs Corretto, I've spotted a couple of differences where this might come into play.  So this appears to be specific to Corretto.
On that note, if I were to take a stab at fixing this myself, would the preferred fix be to update the G1_COLLECTION rule with another conditional capture group, or to create a specific rule for these Corretto lines?  (Do we need to capture the second parentheses? Might change indexes elsewhere...)

This should be picked up by the pattern for UnifiedG1GCPatterns.YOUNG_DETAILS.

@dsgrieve You're right.  Not sure how I missed that.
It has led me to what seems to be the problem though.  I'll close this thread off and post a new issue.


LoginPath override doesn't work for CookieAuthentication

I have a MVC 6 beta 8 web application based on default MVC template. I have renamed default AccountController.cs to Auth.cs and then I've tried to replace login path to /Auth/Login, but unsuccessfully. I've tried two different cases: override CookieAuthenticationOptions.LoginPath in ConfigureServices or in Configure, but there's no effect: [Authorize] attribute redirects me to /Account/Logic and I've received 404 error page.
Code example 1:
services.Configure<CookieAuthenticationOptions>(options =>
{
    options.LoginPath = new PathString("/Auth/Login");
});

Code example 2:
app.UseCookieAuthentication(options =>
{
    options.LoginPath = new PathString("/Auth/Login");
});

If you are using the default templates, the application cookie is using the options configured via Identity, you can access these via IdentityOptions.Cookies.ApplicationCookie


more newbie-friendly README

Presenting this as "a better ack" is completely opaque for anyone who isn't coming from ack. I highly suggest changing this to just describe what it does and then later mention how it relates to other options / history.

I have to agree with the OP.
I think it's bad marketing tool to promote such a great tool, comparing to others.
I think the silver searcher can stand by itself.
I would mention ack as I would mention other searching tools when comparing speed, but that's it.


Update CAMARA-ICM-examples.md

Fixed purpose value in examples to comply to w3C vocabulary and added examples on http responses and token request
What type of PR is this?

correction
enhancement/feature
documentation

What this PR does / why we need it:
Some typing error was included in an example. The PR fixes this example.
Which issue(s) this PR fixes:
The problem is named in a comment to pull request 121, no separate issue is raised.
Special notes for reviewers:
Please review the extensions to the examples adding the HTTP responses as well as the POST request for the access token,
Changelog input
 release-note

Additional documentation
This section can be blank.
docs

Sorry @Elisabeth-Ericsson , I was traveling because May 1st is a holiday here and did not see this PR. I changed the file and now there are merge conflicts.
Although, I am not sure about the HTTP "decoration". I left those out because I wanted to focus on the purpose parts.
And I want to remove all two-purpose examples for now, because I think those lead to discussions I currently don't want to focus on.
I am astonished that I missed the missing sub field, because I copied the examples from some spec, thinking that they must be correct.
Not sure, how to proceed with this PR.

close it and create smaller ones, that e.g. add sub?
resolve merge conflicts and merge?

WDYT?

@AxelNennker
I believe the RFC9101 request object examples should have the following claims, as per RFC7519:
exp: Expiration Time Claim
nbf: Not Before Claim
iat: Issued At Claim
jti: JWT ID Claim


Flatten crashing CLI

Description
This is a potential fix for #17381, and may need to be iterated on.
I updated avtFlattenQuery to only set the XmlResult in the QueryAttributes object if there is a valid results vector.
Type of change

[x] Bug fix
[ ] New feature
[ ] Documentation update
[ ] Other

How Has This Been Tested?
I applied the change and rebuilt and retested on my system (Ubuntu 18.04).
I do not currently have access to a Windows 10 environment to test this on.
Checklist:

[x] I have followed the style guidelines of this project.~~
[x] I have performed a self-review of my own code.~~
[] I have commented my code where applicable.
[ ] I have updated the release notes.
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] I have confirmed new and existing unit tests pass locally with my changes.~~
[ ] I have added new baselines for any new tests to the repo.
[ ] I have NOT made any changes to protocol or public interfaces in an RC branch.
[ ] I have assigned reviewers (see VisIt's PR procedures for more information).~~

We should get this in.


Allow for custom sort order with dict output_type

I would like to pass my own custom function for sorting the dict when using the output_type='dict' option. Currently it sorts by the first character of each key which is fine for some cases, but not all. It would be nice if there was a parameter, say sort_function that I could pass in and replace the lambda x : x[0] function.

Nice suggestion, I will add that parameter these days :D

new parameter for output object is updated
You can update reprint to 0.5.0 and use it :D
Check out the README or the random_sort.py demo

Sweet, I'll check it out next Monday. Thanks!


Android - Warning: Can't perform a React state update on an unmounted component

On Android I am getting the following error:

Hi @joey-harward if possible, do you have a small reproduction repo you could provide for this issue?


Using 3 fingers to pan model on touchscreen

A feature that is in v1.0 of the heart app, but not in v3.0. On a touchscreen (e.g. ipad), dragging 3 fingers across the screen is used to pan (translate) the model.

The old heart app used orbitcontrols to control the model (zoom, rotate, pan) and it was also re-redesigned by the old heart app developers. There are over 1000 lines of code and I need time to read them and am not sure if it still works now in current threejs environment.
Also I checked the threejs official documentation, the Orbitcontrols only have two types of gestures on  touchscreen can be used. One finger: rotate, Two fingers: pan and zoom. We cannot do three fingers now. Then I test it in heart app, it is very hard to use: 1. the rotate angle is very strange, it’ll move in directions you can’t imagine when you use one finger to rotate it on touchscreen. 2. it will have a dead end and will not be able to move again once he has rotated to that angle. This is why we did not initially choose this Orbitcontrols for use in our new heart application.
Currently, we use the Trackballcontrols in our heart applications, which has the advantage of being easy to rotate and having no dead space. But it hasn’t have pan gesture on touchscreen.

My solution is to develop our own three fingers touch event on the touch screen to pan the model by TrackballControls.
For example: when three fingers are on the touchscreen, we can allow the user to pan, to better performance we can disable zoom and rotate when we pan the model.

Now on the touchscreen, we can use one finger - rotate, two fingers-zoom,pan,rotate, three-fingers-pan.


Nodes LoadProperty Parent where no parent is available throws an error

Description
If you load the parent property from a node with no parent $svc.Core.LoadProperty($child, 'Parent') then the system throws an error.

Error details
Error
Exception calling "LoadProperty" with "2" argument(s): "NotFound"
At line:1 char:1
+ $svc.Core.LoadProperty($child, 'Parent')
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : DataServiceClientException

Exception
Error[0].Exception
Exception calling "LoadProperty" with "2" argument(s): "NotFound"

Inner exception
$Error[0].Exception.InnerException
NotFound

Inner inner exception > nothing shows up
$Error[0].Exception.InnerException.InnerException

resolved :+1:


Implement Full Place of Birth Search

Title changed for clarification: this story applies to all aspects of searching for a place of birth, from country level to hamlet or ship level.
There are 507,191 distinct (Unique) values in the verbatim birth field as of 25 Nov 2021.
There are 421,137 distinct (Unique) values in the birth field as of 25 Nov 2021.

It is not clear to me how we can structure a meaningful birth place constraint given that there are over 500,000 distinct place names in the records. Clearly a drop down list for selection is going to contain too many entries.

Might need an initial basic search, with enhanced tools.
Basic search: select different country, if not United Kingdom, then state (uk gives the four home countries plus Ireland?) then county, then lat and long is found from drop down list of standard places?
Enhanced: final step (or first step?) is a free text search using gazateer to get lat and long
Alternative: third party tool is used to identify lat and long from a map (seach by place name or by pointing at map)?

Unfortunately a large number of the distinct place names for the birth place do not have a location hence their birth place record is inaccessible.

Sorry, I thought all identifiable Places of Birth were in the gazetteer, and every place in the gazetteer was georeferenced.
Would the Ordnace Survey Place data be sufficient to geocode the rest? It is Open Data.  https://www.ordnancesurvey.co.uk/business-government/products/open-map-names
There is an API: https://osdatahub.os.uk/docs/places/gettingStarted or we can download as CSV.

POB is validated for CSVProc but we have 40 million VLD POB values that were only validated against whatever the transcriber validator used

We cannot get a reliable result for POB searching until the vld places of birth have been aligned with the Gazetteer. Once this is done then we can search using the Gazetteer geolocation capabilities.
In turn, this cannot be done until the vld data set i stable (i.e. no more additions).
Geoff


if a user's roles change, the user must sign back in to get updated roles

This is an interesting problem because I don't think there is a straight forward solution. A user is given a set of roles. On sign in, this server fetches the user's session and turns it into a token. The problem arises when a user's roles change later on in the life of the token. The token does not get the updated set of roles and therefore will continue to access the database on the old set of roles. This creates two problems: users not having enough permission and getting denied (role was added) or the user has more permission than they should (role was removed).
Currently, the only way to solve this is to have the user sign out and sign in again, but this requires their username and password which may not always be available.
One solution may be to have the server request a new session on each token renewal. This has its downsides, however: this would require the database to be configured with public user profiles or for a super admin user and password to be set.

Another solution is to use the JWT token passed to the renewal to grab a copy of the user's document and build a new session from that. This has a few downsides:

couchdb_jwt_auth must be installed in the authentication database, which is not currently a requirement. #8 suggests using a JWT instead of credentials for the store, so maybe this isn't a huge deal? Either way though, the auth database and the normal database must share the same token secret.
user document has to be pulled down on every renewal. I guess that's the price to pay when needing a refresh on information.


What's is remoteConfigUrl?

hello,friend
I get function getRemoteConfig() ,is this get remote webrtc config(ice/stun/turn) ,how I modify var remoteConfigUrl to my iceserver
I  use coturn in my server , and  how to use my coturn server
Thank u and God bless u

I find this remoteConfigUrl is a http ask and get a json like {"iceserver":["urls" :{}]},
you can get the result by var remoteConfigUrl = 'https://instant.io/_rtcConfig',
you can get some info ,so I write an api get result like remoteConfigUrl.
If u get this, u can get everywhere whatever innernet or internet.

It is not resolved


Custom attributes / json import & export

Not sure if you will want to merge this, these are changes I need for a personal project and so it may not be very useful to others - I am generating an interactive image map from json, and I needed an easy way to export map data as json. I also needed more properties than just href, alt, and title so I added the ability to add/import custom attributes. The commit messages should explain the other changes I made.
Also, there are conflicts with merging to master because I started from the gh-pages branch, I had issues with the current master branch (couldn't create polygons).

Hi,
thank you for your pull-request.
I created this project 4 years ago, and my old code is very bad.
Now, before I will add some features to editor, I am making big refactoring of this code.
And, after this, I will divide interface parts from editor (https://github.com/summerstyle/app.js), and work with areas - to separate library too.
Some commits is alredy in master branch (you are right, there are bug with polygons), and other changes I don't push to master yet. And I can't merge your pull-request because code of gh-pages is old.
But I fixed some bugs you finded, and  I will add import/export to json (this your idea is very good).

Looking back at old code is always fun.
That makes sense, I wasn't really expecting it to get merged.
Sounds good, the main thing I added besides json import/export was the ability to add attributes other than href, alt, and title, you may want to consider adding that.


Docker image doesn't contain any of /client/ assets

I spent a good half an hour scratching my head, thinking I'd done something wrong. I followed the docs in the README pretty closely but couldn't work out for the life of me why I was getting 404 on the /client/ paths.
I started with a simple container, orchestrated with Docker Compose as such:
services:
  [...]
  slidingsync:
    image: ghcr.io/matrix-org/sliding-sync
    environment:
    - SYNCV3_SECRET=<snip>
    - SYNCV3_SERVER=https://<snip>
    - SYNCV3_DB=host=db dbname=syncv3 sslmode=disable user=syncv3 password=<snip>
    - SYNCV3_BINDADDR=:8008
    depends_on:
    - db
    networks:
      backend:
    restart: on-failure

and after futzing with postgres permissions, syncv3 started up, but browsing to the container IP at /client/ I was getting 404
Sync v3 [0.99.0] (613cbd9a670d2cf166a71a182183a5b179a4347a)
10:13:59 INF creating handler
10:13:59 INF retrieved global snapshot from database
10:13:59 INF listening on [::]:8008
10:13:59 INF StartV2Pollers num_devices=0 num_fail_decrypt=0
10:13:59 INF StartV2Pollers finished
10:14:18 INF  duration=0.027891 path=/ size=19 status=404
10:14:19 INF  duration=0.048513 path=/favicon.ico size=19 status=404
10:14:22 INF  duration=0.039538 path=/client size=19 status=404
10:14:35 INF  duration=0.308996 path=/client/ size=19 status=404
10:16:10 INF  duration=0.056527 path=/client/ size=19 status=404

After poking in the code, I realised that the client directory is simply a static directory of files, loaded from the current working directory https://github.com/matrix-org/sliding-sync/blob/613cbd9a670d2cf166a71a182183a5b179a4347a/v3.go#L131 and checking inside the container, sure enough, there was no client directory to be found anywhere.
I cloned this repo and bind-mounted the /client/ directory into the container, and magically everything seems to work now.
Should the client files be included into the container image?

The client files are test files for development purposes. They aren't intended to be used by end users. Use a sliding sync enabled client like Element X or Element Web.

Ah, that explains my confusion. The readme makes it sound like the usage instructions are for "normal" usage and not just development

I got bitten by this too - trying to hit /clients/ to see if I've configured the reverse-proxy correctly, only to see it 404 every time. Even without a reverse-proxy (hitting :8008 directly), I was observing the same, which was causing me to scratch my head.
I suppose the README should be improved.


XHalfLap doesnt run outside Rhino?

Tried writing unittests for the half lap joint and couldn't get it to run just with code (CPython)
Inside Rhino it seems to be running fine.
assembly = TimberAssembly()
beam_a = Beam.from_endpoints(Point(0.5, 0, 0), Point(0.5, 1, 0), 0.2, 0.2, z_vector=Vector(0, 0, 1))
beam_b = Beam.from_endpoints(Point(0, 0.5, 0), Point(1, 0.5, 0), 0.2, 0.2, z_vector=Vector(0, 0, 1))
assembly.add_beam(beam_a)
assembly.add_beam(beam_b)

from compas_timber.connections import XHalfLapJoint
joint = XHalfLapJoint.create(assembly, beam_a, beam_b)

tests\compas_timber\test_joint.py:163: in test_joint_create_kwargs_passthrough_xhalflap
    joint = XHalfLapJoint.create(assembly, beam_a, beam_b)
src\compas_timber\connections\joint.py:130: in create
    joint.add_features()
src\compas_timber\connections\x_halflap.py:172: in add_features
    negative_brep_beam_a, negative_brep_beam_b = self._create_negative_volumes()
src\compas_timber\connections\x_halflap.py:144: in _create_negative_volumes
    planes_a = self._sort_beam_planes(self.beam_a, plane_cut_vector_a)
src\compas_timber\connections\x_halflap.py:101: in _sort_beam_planes
    planes_angles, planes = zip(*sorted(zip(planes_angles, planes)))
E   TypeError: '<' not supported between instances of 'Plane' and 'Plane'

could be sorted with the zip there is behaving differently in CPython vs. IronPython (or just 2.7 vs. 3)

this function only returns the planes anyways, so we can just sort planes with planes_angles as key. I think that's how I did it in the other laps. I'll change it real quick

@chenkasirer maybe you can test it real quick. I just changed it

yup seems to have worked. I wrote a few unittests you can find them in tests/compas_timber/
you run them using invoke test and they should all pass.
When opening a PR or when pushing new commits to it, it'll run them automatically as Github Actions.
You can click on Details next to an action that failed to checkout why it failed. All checks should be green before we can merge.


Failing to fit an lstm model.fit(x=g,) with generator g, custom loss function and run_eagerly is False

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
v2.10.0-rc3-6-g359c3cdfc5f 2.10.0
Custom Code
Yes
OS Platform and Distribution
Linux xxxxxx 5.10.0-19-amd64   SMP Debian 5.10.149-1 (2022-10-17) x86_64 GNU/Linux
Mobile device
No response
Python version
python  3.9.13   h9a8a25e_0_cpython    conda-forge
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
cudatoolkit  11.2.2  hbe64b41_10  conda-forge,  cudnn  <IP_ADDRESS>  h90431f1_0  conda-forge
GPU model and memory
Quadro RTX 4000, 8GB
Current Behaviour?
I was trying to rerun some earlier work, that used to work on TF 2.6.0 I think (not sure).
In a nutshell, extracted for a repro gist thereafter:
def create_and_fit(x, run_eagerly):
    model = new_model()
    model.compile(optimizer=Adam(learning_rate=1e-4), loss=mean_absolute_error_na, metrics=[some_metric], run_eagerly=run_eagerly)
    return model.fit(
        x,
        steps_per_epoch=steps_per_epoch,
        epochs=epochs,
        validation_data=None
    )

if x is a <generator object train_generator_batch> and run_eagerly is False, it seems that invalid ground truth observations y_true is passed to the custom loss function, which reports:
Epoch 1/5
y_true = Tensor("mean_absolute_error_na/remove_squeezable_dimensions/Squeeze:0", shape=(None, None), dtype=float32)
y_pred = Tensor("model_1/dense_1/BiasAdd:0", shape=(16, 1), dtype=float32)
etc etc stack trace...
    ValueError: y_true and y_pred do not have compatible shapes: (None, None) and (16, 1). Respective types are <class 'tensorflow.python.framework.ops.Tensor'> and <class 'tensorflow.python.framework.ops.Tensor'>

Model fitting works however if run_eagerly is True, or if x is passed a dataset created with tf.data.Dataset.from_generator.
Note that I run the jupyter python kernel launched via optirun to access the GPU, in case this has relevance. The bug anyway occurs also if forcing the execution on the CPU.
Standalone code to reproduce the issue
A self-contained notebook (python file obtained with jupytext) is available at this gist
Relevant log output
Epoch 1/5
y_true = Tensor("mean_absolute_error_na/remove_squeezable_dimensions/Squeeze:0", shape=(None, None), dtype=float32)
y_pred = Tensor("model_1/dense_1/BiasAdd:0", shape=(16, 1), dtype=float32)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In [40], line 1
----> 1 create_and_fit(train_gen, run_eagerly = False)

Cell In [37], line 4, in create_and_fit(x, run_eagerly)
      2 model = new_model()
      3 model.compile(optimizer=Adam(learning_rate=1e-4), loss=mean_absolute_error_na, metrics=[some_metric], run_eagerly=run_eagerly)
----> 4 return model.fit(
      5     x,
      6     steps_per_epoch=steps_per_epoch,
      7     epochs=epochs,
      8     validation_data=None
      9 )

File ~/miniconda/envs/my_env_name/lib/python3.9/site-packages/keras/utils/traceback_utils.py:70, in filter_traceback.<locals>.error_handler(*args, **kwargs)
     67     filtered_tb = _process_traceback_frames(e.__traceback__)
     68     # To get the full stack trace, call:
     69     # `tf.debugging.disable_traceback_filtering()`
---> 70     raise e.with_traceback(filtered_tb) from None
     71 finally:
     72     del filtered_tb

File /tmp/__autograph_generated_file7kl24yk4.py:15, in outer_factory.<locals>.inner_factory.<locals>.tf__train_function(iterator)
     13 try:
     14     do_return = True
---> 15     retval_ = ag__.converted_call(ag__.ld(step_function), (ag__.ld(self), ag__.ld(iterator)), None, fscope)
     16 except:
     17     do_return = False

File /tmp/__autograph_generated_filexp4m9ubh.py:11, in outer_factory.<locals>.inner_factory.<locals>.tf__mean_absolute_error_na(y_true, y_pred)
      9 do_return = False
     10 retval_ = ag__.UndefinedReturnValue()
---> 11 (y_true_m, y_pred_m) = ag__.converted_call(ag__.ld(remove_items_missing_observations), (ag__.ld(y_true), ag__.ld(y_pred)), None, fscope)
     12 try:
     13     do_return = True

File /tmp/__autograph_generated_filei0twifpa.py:24, in outer_factory.<locals>.inner_factory.<locals>.tf__remove_items_missing_observations(y_true, y_pred)
     22 def else_body():
     23     pass
---> 24 ag__.if_stmt(ag__.ld(y_true).shape != ag__.ld(y_pred).shape, if_body, else_body, get_state, set_state, (), 0)
     25 try:
     26     do_return = True

File /tmp/__autograph_generated_filei0twifpa.py:20, in outer_factory.<locals>.inner_factory.<locals>.tf__remove_items_missing_observations.<locals>.if_body()
     18 ag__.ld(print)(f'y_true = {ag__.ld(y_true)}')
     19 ag__.ld(print)(f'y_pred = {ag__.ld(y_pred)}')
---> 20 raise ag__.converted_call(ag__.ld(ValueError), (f'y_true and y_pred do not have compatible shapes: {ag__.ld(y_true).shape} and {ag__.ld(y_pred).shape}. Respective types are {ag__.converted_call(ag__.ld(type), (ag__.ld(y_true),), None, fscope)} and {ag__.converted_call(ag__.ld(type), (ag__.ld(y_pred),), None, fscope)}',), None, fscope)

ValueError: in user code:

    File "/home/xxxyyy/miniconda/envs/my_env_name/lib/python3.9/site-packages/keras/engine/training.py", line 1160, in train_function  *
        return step_function(self, iterator)
    File "/tmp/ipykernel_36702/144245855.py", line 13, in mean_absolute_error_na  *
        y_true_m, y_pred_m = remove_items_missing_observations(y_true, y_pred)
    File "/tmp/ipykernel_36702/144245855.py", line 8, in remove_items_missing_observations  *
        raise ValueError(f"y_true and y_pred do not have compatible shapes: {y_true.shape} and {y_pred.shape}. Respective types are {type(y_true)} and {type(y_pred)}")

    ValueError: y_true and y_pred do not have compatible shapes: (None, None) and (16, 1). Respective types are <class 'tensorflow.python.framework.ops.Tensor'> and <class 'tensorflow.python.framework.ops.Tensor'>

@jmp75,
To work in Graph mode, Please remove or comment out the raise ValueError() and code works without any issue.  Under the  Graph mode TF automatically creates placeholders for the None shapes and it is an intended behaviour.
Please refer to attached gist here.  Thank you!

Thank you for this, I'll study this and communicate to colleagues.


result problem

Hello, I used the 274 test sets in the sabio database mentioned in your paper"Deep learning allows genome-scale prediction of Michaelis constants from structural features"。sing the model provided here, and the results were：'pearson': tensor(0.6948), 'rmse': tensor(0.8430), 'r2': tensor(0.4667)
Is the result due to a difference in the use of the model, or is it due to a problem with the input? Here is the input code：
import pandas as pd
compare_df = pd.read_csv('sabio_df_checkpoint3_V2.csv', index_col=False)
subs = []
sequs = []
for ind in compare_df.index:
subs.append(compare_df['KEGG ID'][ind])
sequs.append(compare_df['Sequence'][ind])
KM = KM_predicton(substrate_list=subs, enzyme_list=sequs)
KM.to_csv('test1.csv')

Thanks

Dear jiejiangtao,
to me it looks like you have done everything right. The test metrics that you achieved are very similar to the ones that we  achieved in the manuscript: (R2: 0.47 vs. 0.49) and (MSE: 0.71 vs. 0.67). This indicates that you produce meaningful results. Please note that the prediction function in this repository is not completely identical to the function from our manuscript or the following GitHub repository: https://github.com/AlexanderKroll/KM_prediction
I stated the reason for that in the Readme of this repository:
"Please note that the provided model is not identical to the one presented in the paper: Here, we used enzyme representations that are slightly different. Instead of the UniRep model, here we are using the ESM-1b model to create the enzyme representations. It was shown that the ESM-1b model outperforms the UniRep model as it is trained with a more up-to-date model for natural language processing (with a transformer network instead of a LSTM)."


HIVE-27716: Precommit: Save log files for first 10 failures

What changes were proposed in this pull request?
Save some tests logs in case of failing tests.
Why are the changes needed?
Because currently the artifact is pretty useless, it doesn't contain more info than what's already visible in jenkins.
Does this PR introduce any user-facing change?
No, it's precommit related.
Is the change a dependency upgrade?
No.
How was this patch tested?
Jenkins is supposed to pick up Jenkinsfile from the patch. Need to verify at:
http://ci.hive.apache.org/job/hive-precommit/job/PR-4732/1/
verified, first commit:
Obtained Jenkinsfile from 2d2ca54feb59480bbc960c5ffd45aa6134e76f2c

ironically there were test failures, and the file contains the saved logs:
f "*TestCrudCompactorOnTez*"
./split-01/itests/hive-unit/target/surefire-reports/org.apache.hadoop.hive.ql.txn.compactor.TestCrudCompactorOnTez-output-save.txt
./split-01/itests/hive-unit/target/surefire-reports/TEST-org.apache.hadoop.hive.ql.txn.compactor.TestCrudCompactorOnTez.xml
./split-01/itests/hive-unit/target/surefire-reports/org.apache.hadoop.hive.ql.txn.compactor.TestCrudCompactorOnTez.txt

so this seems to work

@ayushtkn , can you please review? failure are not related, infra code seems to work

I have a question here, like two queries failed for some CLI driver, will this overwrite the output of the prev, combine the output or each output would be present separately

hm, I guess simply overwrites the previous but with the same file :) this happens after all the tests finished, so you have a driver output file containing both 2 errors, and you write it twice


Add support for keyof operator

This is a pre-req for supporting mapped types since many of the utility types defined using mapped types make use of keyof.

Codecov Report

Merging #280 (f5e7e9e) into main (6a09d3d) will increase coverage by 0.04%.
The diff coverage is 76.66%.

@@            Coverage Diff             @@
##             main     #280      +/-   ##
==========================================
+ Coverage   90.07%   90.11%   +0.04%
==========================================
  Files          31       32       +1
  Lines        4552     4635      +83
==========================================
+ Hits         4100     4177      +77
- Misses        452      458       +6

Impacted Files
Coverage Δ

crates/crochet_dts/src/util.rs
67.41% <0.00%> (-0.77%)
:arrow_down:

crates/crochet_infer/src/infer_expr.rs
93.98% <0.00%> (-0.24%)
:arrow_down:

crates/crochet_types/src/keyword.rs
50.00% <0.00%> (-10.00%)
:arrow_down:

crates/crochet_infer/src/unify.rs
90.41% <42.85%> (-1.02%)
:arrow_down:

crates/crochet_codegen/src/d_ts.rs
74.30% <50.00%> (-0.30%)
:arrow_down:

crates/crochet_infer/src/substitutable.rs
94.73% <50.00%> (-0.80%)
:arrow_down:

crates/crochet_infer/src/util.rs
96.98% <80.00%> (+1.50%)
:arrow_up:

crates/crochet_infer/src/key_of.rs
80.24% <80.24%> (ø)

crates/crochet_infer/src/lib.rs
98.99% <87.50%> (-0.14%)
:arrow_down:

crates/crochet_infer/src/infer_type_ann.rs
98.52% <100.00%> (+15.41%)
:arrow_up:

... and 7 more

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


vaultwarden-vault: 2.25.0 -> 2.25.1b

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/dani-garcia/bw_web_builds/releases.
meta.description for vaultwarden-vault is: "Integrates the web vault into vaultwarden"
meta.homepage for vaultwarden-vault is: "https://github.com/dani-garcia/bw_web_builds"
meta.changelog for vaultwarden-vault is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.25.1b with grep in /nix/store/nzcd11yifl5mlj2vbas6nmqc7lq2w126-vaultwarden-vault-2.25.1b

found 2.25.1b in filename of file in /nix/store/nzcd11yifl5mlj2vbas6nmqc7lq2w126-vaultwarden-vault-2.25.1b

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

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
vaultwarden-vault

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nzcd11yifl5mlj2vbas6nmqc7lq2w126-vaultwarden-vault-2.25.1b \
  --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 vaultwarden-vault https://github.com/r-ryantm/nixpkgs/archive/c9d60cc7cd11e24784ae3995a5814cc2bb152a38.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nzcd11yifl5mlj2vbas6nmqc7lq2w126-vaultwarden-vault-2.25.1b
ls -la /nix/store/nzcd11yifl5mlj2vbas6nmqc7lq2w126-vaultwarden-vault-2.25.1b/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 run on x86_64-linux 1

1 package built:

vaultwarden-vault

Maintainer pings
cc @msteen @mic92 for testing.

Result of nixpkgs-review pr 159827 run on x86_64-linux 1

1 package built:

vaultwarden-vault

Does b stand for beta? https://github.com/dani-garcia/bw_web_builds/releases/tag/v2.25.1b
https://github.com/dani-garcia/bw_web_builds/releases/tag/v2.26.1 might be the better choice.

https://github.com/dani-garcia/bw_web_builds/releases/tag/v2.27.0
2.27.0 is out, any progress here?


Modules routes not getting added to web...

After installing modular and creating several modules, It would appear that the web routes are not getting added properly. "route:list" from the command-line shows the module and routes correctly, but attempts to visit that same route via web fail. If I add the route group in RouteServiceProvider.php like so:
            Route::middleware('web')
                ->group(base_path('app-modules/core/routes/core-routes.php'));

the route does work via web. I did not see anything in the install/usage instructions that required adding modules to the RouteServiceProvider. Has anyone run into this issue? Does anyone have an idea what might be going on?
PHP 8.1.2-1ubuntu2.17
Composer version 2.7.4 2024-04-22 21:17:03
Laravel Framework 10.48.10
"internachi/modular": "^2.2",

I have the same problem.

Maybe related to this; in my base layout I use the Auth facade to check if some menu items should be shown to the user based on role.
If I define the routes in the module, this gives an error, because the auth -> user is null.
Adding the group similar to micah-quinn's comment fixes that.

Thank you! This solved my issue related to https://github.com/InterNACHI/modular/issues/78.
    // My routes here
});

As regards to this, is it possible to change the routes file as via stub file? It doen't seem to be supported, I guess.
I would like to edit it based on some middleware I have to apply as default.
Thanks.


Add GET /safe-apps client_url param to swagger, align chainId param/variable naming

This PR fixes:

In my previous PR for access control for Safe Apps I forgot to include client_url param to swagger parameters
(nit) Aligns naming between chainId param and the variable naming

@fmrsabino I just realized that chainId and client_url use different naming conventions. Should I also align them to use camelCase or something else?

@fmrsabino I just realized that chainId and client_url use different naming conventions. Should I also align them to use camelCase or something else?
EDIT: Ok, it looks like we use camelCase in responses, so it makes sense to stick to camelCase in params as well. I just pushed a commit to align client_url

Yes 👍 Even though we use _ in the python parameters, it goes through a camel case middleware which Is installed then converts it to camelCase for the clients

Yes 👍 Even though we use _ in the python parameters, it goes through a camel case middleware which Is installed then converts it to camelCase for the clients

It is only true for responses, isn't it?

Yes 👍 Even though we use _ in the python parameters, it goes through a camel case middleware which Is installed then converts it to camelCase for the clients

It is only true for responses, isn't it?

Yes. For the query parameters we do it as you just did in this PR


Add multilib to the NonStop configuration definitions.

Fixes: #16373
Co-authored-by: Randall S. Becker<EMAIL_ADDRESS>

Alternative to #16389

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master. Thank you.


osrf_pycommon: 2.1.1-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository osrf_pycommon to 2.1.1-1:

upstream repository: https://github.com/osrf/osrf_pycommon.git
release repository: https://github.com/ros2-gbp/osrf_pycommon-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 2.0.2-2

osrf_pycommon
* Declare test dependencies in [test] extra (#86 <https://github.com/osrf/osrf_pycommon/issues/86>)
* Contributors: Scott K Logan

I'm going to close this PR and we'll make a release later (not for the upcoming Humble patch).


edited backend dockerfile to add python to path. edited scss for remo…

…ve spin button from 2FA passcode form.

🗣 Description

💭 Motivation and context

🧪 Testing

✅ Pre-approval checklist

[ ] This PR has an informative and human-readable title.
[ ] Changes are limited to a single goal - eschew scope creep!
[ ] All future TODOs are captured in issues, which are referenced
in code comments.
[ ] All relevant type-of-change labels have been added.
[ ] I have read the CONTRIBUTING document.
[ ] These code changes follow cisagov code standards.
[ ] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[ ] Tests have been added and/or modified to cover the changes in this PR.
[ ] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

Edited backend dockerfile to add python to path.
Edited scss for remove spin button from 2FA passcode form.


fix ut fail by upgrade gomock to v1.5.5

What this PR does:
Just fix the ut for xds_test.go
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

waiting https://github.com/dubbo-go-pixiu/samples/pull/6 merged, the ci will be pass.

Codecov Report

Merging #428 (c117ad3) into develop (bf77bc3) will increase coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #428      +/-   ##
===========================================
+ Coverage    37.58%   37.64%   +0.06%
===========================================
  Files           55       55
  Lines         3720     3743      +23
===========================================
+ Hits          1398     1409      +11
- Misses        2166     2176      +10
- Partials       156      158       +2

Impacted Files
Coverage Δ

pkg/client/dubbo/dubbo.go
18.83% <ø> (ø)

pkg/server/adapter_manager.go
63.33% <0.00%> (-7.04%)
:arrow_down:

pkg/server/cluster_manager.go
38.29% <0.00%> (+2.76%)
: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 4dec4fb...c117ad3. Read the comment docs.


Add project name "ci" to "docker-compose down" command

To shutdown and remove the containers which are created by "docker-compose up" as expected.

To simplify git-log, I have done the rebasing and the local test and CI passed.

Thanks for your contribution :-)

Thanks for your contribution :-)


[WinUI] pressing enter when focused on TextBox with CommandExtensions.Command attached crash the app

Current behavior

Error thrown
Exception {"Element not found. (0x80070490)"}

Callstack
CommandExtensionsCrash.Windows.dll!CommandExtensionsCrash.App.InitializeComponent.AnonymousMethod__7_1(object sender = {CommandExtensionsCrash.App}, Microsoft.UI.Xaml.UnhandledExceptionEventArgs e = {Microsoft.UI.Xaml.UnhandledExceptionEventArgs}) Line 68
	at C:\p\CommandExtensionsCrash\CommandExtensionsCrash.Windows\obj\x86\Debug\net6.0-windows10.0.19041.0\win10-x86\App.g.i.cs(68)
Microsoft.WinUI.dll!WinRT._EventSource_global__Microsoft_UI_Xaml_UnhandledExceptionEventHandler.EventState.GetEventInvoke.AnonymousMethod__1_0(object sender, Microsoft.UI.Xaml.UnhandledExceptionEventArgs e)
Microsoft.WinUI.dll!ABI.Microsoft.UI.Xaml.UnhandledExceptionEventHandler.Do_Abi_Invoke(System.IntPtr thisPtr, System.IntPtr sender, System.IntPtr e)
[Native to Managed Transition]
[Managed to Native Transition]
Microsoft.WinUI.dll!ABI.Microsoft.UI.Xaml.IApplicationStaticsMethods.Start(WinRT.IObjectReference _obj, Microsoft.UI.Xaml.ApplicationInitializationCallback callback)
Microsoft.WinUI.dll!Microsoft.UI.Xaml.Application.Start(Microsoft.UI.Xaml.ApplicationInitializationCallback callback)
CommandExtensionsCrash.Windows.dll!CommandExtensionsCrash.Program.Main(string[] args = {string[0]}) Line 31
	at C:\p\CommandExtensionsCrash\CommandExtensionsCrash.Windows\obj\x86\Debug\net6.0-windows10.0.19041.0\win10-x86\App.g.i.cs(31)

Expected behavior
The command should be executed without crashing the app.

How to reproduce it (as minimally and precisely as possible)

Download this sample CommandExtensionsCrash.zip
Deploy on WinUI
Make sure to focus on the textbox and press enter

Environment

Nuget Package: Uno.Toolkit.WinUI
Package Version(s): 2.5.0-dev.8
Affected platform(s):

[ ] iOS
[ ] Android
[ ] WebAssembly
[ ] WebAssembly renders for Xamarin.Forms
[x] Windows
[ ] Build tasks

Visual Studio:

[x] 2022 (version: 17.4.3)
[ ] 2019 (version: )
[ ] for Mac (version: )

Relevant plugins:

[ ] Resharper (version: )

Anything else we need to know?

fixed in #440 already

@Xiaoy312 do we need to backport #440 ?

Hum the weird part is that I tested with the latest version available 2.5.0-dev.8. Although, the PR seems to have been merged yesterday. Maybe there are no build available with this fix yet

The bug is indeed fix in 2.5.0-dev.10


Put the docs warning png in a known place

Github's linking through redirects seems broken, and I need this in a known place without relative paths.
@dchen1107

LGTM


Quality declaration statement for rclc_paramter

is fixed in:

Rolling: https://github.com/ros2/rclc/pull/144
Galactic: https://github.com/ros2/rclc/pull/181
Foxy https://github.com/ros2/rclc/pull/182 (only backport of Windows compatability)


[BACKLOG-35958] Added pop-up dialog to delete the file/folder, when t…

…he user taps on X button

:white_check_mark: Build finished in 5m 25s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
plugins/file-open-save-new/core

:exclamation: No tests found!
:information_source:  This is an automatic message


Add "Master" track

Get/set/listen properties in the master track:

tracks
song (track_data)
view (select track)

The master track is simply added tot the track list, so if you have 8 tracks going from [0...7], then master is number 8.

Hey @markmarijnissen, apologies for the slow reply on this!
Thanks for your changes. My main concern is that indexing the master track based on the number of tracks is brittle to changes in the number of tracks in the set. A more robust approach would be to replace the integer track ID with the string "Master", which could be used in place of the numeric ID — e.g.:
/live/track/set/volume "Master" 0.5

Would this work in your situation?

As live/song/get/num_tracks only counts the midi/audio tracks (no return tracks, no master) I agree that the master does not belong in the track list as additional track. How is the master actually accessed in the Live API (and how about the return tracks)? Should be done the same way in AbletonOSC.

@Coupe70 Re namespace for send tracks:
I've just looked at the Live API docs, and here's how it works internally. The Song object has three properties:

.tracks contains a list of all MIDI/Audio tracks
.master_track contains the master track
.send_tracks contains a list of sends

One approach to doing the same in the AbletonOSC namespace:

/live/track/set/volume 0 0.5 to set the volume of track 0
/live/track/send/set/volume 0 0.5 to set the volume of send 0 ("A")
/live/track/master/set/volume 0.5 to set the volume of the master track

This is OK, although it does mean that the master get/set take one less parameter. It also adds slight confusion between /live/track/set/send (to set the send/return level of a track) and /live/track/send/set/.. (to set properties of a given send track).
Another approach:

/live/track/set/volume 0 0.5 to set the volume of track 0
/live/track/set/volume A 0.5 to set the volume of send 0 ("A")
/live/track/set/volume master 0.5 to set the volume of the master track

Unifies the namespace, and means that all track getters/setters are handled with the same parameter sequence (track_id, ...). It does mean that the first argument is sometimes an int and sometimes a string, and iterating the characters corresponding to multiple sends ("A", "B", "C", ...) is a bit gross.
I think I come down on the side of the latter. Other suggestions welcomed.

@ideoforms
I was hoping to find some inspiration in the OSC list of LiveOSC.
https://github.com/dinchak/LiveOSC/blob/master/OSCAPI.txt
LiveOSC does not make a difference between set/get. If there are parameters, it's set, if there are no parameters, it's get.
But that's a different story.
LiveOSC does not put all kinds of tracks under "track".
A "track" is just a "normal" audio/midi track, others are "send", "return", "master".
So that would be /live/master/set...
I also noticed a mistake in your approach to set a send.
`/live/track/send/set/volume 0 0.5`

This would work for a return track volume, but is missing a third parameter for sends and should be
/live/track/send/set/volume trackID sendID value

Yeah, I did think about "action-less" setters when first designing the API, but I wanted symmetry/consistency between get, set, start_listen, stop_listen (plus there are other actions like /live/clip/add/notes), so I concluded that having an explicit get/set was overall more consistent.
I don't think there was an error in my example, but maybe an interpretation issue: between send tracks and send levels. (In the Live API terminology, a return track is always referred to as a send track, so I'm keen to keep this terminology in AbletonOSC)
There's already a getter/setter for the send levels of an ordinary (audio) track:
/live/track/set/send trackID sendID value

"volume" is really used to explicitly set the output volume of a track (not for send levels).
The example that I'm illustrating below is setting the output volume level of the first send track (i.e., Send A):
/live/track/send/set/volume 0 0.5

I wonder whether questions like this would crop up a lot if we did go with the namespace of `/live/track/send/set/volume` to refer to send tracks...  `/live/track/send/set/volume` (to set the output level of the first send track) is extremely easily confused with `/live/track/set/send` (to set the send level of a given audio track)!

Ok, I understand.
It's not the send knob, it's the volume fader of the "FX track".
I understand the problem with the terminology as with a hardware mixer you have clearly definied the send -> outboard equipment -> return while in Live you are straight SENDing into a track which contains the FX.
While I understand this, "send track" is still very confusing to me - as you said the term "send" appears at two different spots.
So here are my arguments no matter how it is called in the API:
Even if you might call it a send track in a DAW (you would never do this with a hardware mixer) the volume fader is AFTER the FX, so it is the volume RETURNING to the mix. But my main point, not only applying to the fader: When you open the info view in the lower left corner of Live and hover over the "FX track" the info view says RETURN track in german and english language setting. And the german manual calls it RETURN track, too.
I think calling it 'return' in AbletonOSC would avoid a lot of confusion.

Hello Folks
is there finally an OSC-command to control the Master-Volume ?? I tried the three different ways you where talking about... but they seem not to work !!?? or did I oversee something ??
thanks in advance... :)

All of us are waiting for this, so please .....


Thread safety fixes

I was using this library to write a local "mock" API server, and starting/stopping it between test cases in setUp and tearDown.  I noticed I would occasionally get EXC_BAD_ACCESS when a client socket was being removed, and upon closer inspection, noticed the server's internal fields weren't being read & written from multiple threads.
I was able to reproduce the crash in the unit test I added, and was able to get it to pass consistently with the changes you see here.  In short, a serial queue was added to protect thesockets set and the state variable was protected by using libatomic primitives in the getter & setter.

Some more work might need to be done to make this Linux-compatible and resolve conflicts with the master branch if it should be merged there instead.

I ran the tests a few more times, and I'm also seeing occasional crashes in HttpServer.findHandler, I'm guessing because it mutates the routing nodes as it routes requests.  Don't think I have time to resolve that just yet, but would suggest either making the routes immutable so they can be safely read from multiple requests threads/queues.

👍 This PR is excellent.

@bgerstle


Use ruby 2.2 parser until 2.3 is supported in ruby_parser.

Fixes errors on ruby 2.3.0-preview2 of the variety:
  1) ManageIQ::Providers::Kubernetes::ContainerManager::Refresher will perform a full refresh on k8s
     Failure/Error: @ems = FactoryGirl.create(:ems_kubernetes, :hostname => "<IP_ADDRESS>",
     RuntimeError:
       unrecognized RUBY_VERSION 2.3.0
      ./lib/extensions/descendant_loader.rb:74:in `classes_in'
      ./lib/extensions/descendant_loader.rb:193:in `classes_in'
      ./lib/extensions/descendant_loader.rb:205:in `block in class_inheritance_relationships'
      ./lib/extensions/descendant_loader.rb:204:in `glob'
      ./lib/extensions/descendant_loader.rb:204:in `class_inheritance_relationships'
      ./lib/extensions/descendant_loader.rb:228:in `load_subclasses'
      ./lib/extensions/descendant_loader.rb:250:in `descendants'
      ./spec/models/manageiq/providers/kubernetes/container_manager/refresher_spec.rb:7:in `block (2 levels) in <top (required)>'

See the request to support 2.3 features such as &. "safe navigation operator"
https://github.com/seattlerb/ruby_parser, issue: 201

@matthewd Please review.  :bow:


Unable to get parameters of current view in a local pdf

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version: Firefox Nightly 105.0a1
Operating system and its version: Debian 11
PDF.js version: 2.16.71
Is a browser extension: No

Steps to reproduce the problem:

Download a pdf.
Open the pdf in Firefox.
Click on current view.
What is the expected behavior? (add screenshot)
The url should update.

What went wrong? (add screenshot)
The url does not get updated. Security Error in the console.

This is, for all intents and purposes, a duplicate of e.g. https://bugzilla.mozilla.org/show_bug.cgi?id=1678914


Testing Android CI

Trying to debug #1765 .

r? @JohnTitor
(rust_highfive has picked a reviewer for you, use r? to override)

Note that we don't have Android builders on PRs, but on bors.

You cannot do because bors permission is restricted to the maintainers. I could delegate you but I think the ways I mentioned in https://github.com/rust-lang/libc/issues/1765#issuecomment-710249540 are more suitable.

Since the issue is fixed now, I'm going to close this. Thanks for contributing anyway!


debug print

LGTM, how about printing total time tests taken?

Not enought changes to merge in my opinion. I can merge it with pleasure if you add more metrics about tests. Thanks your effort ^^


Remove glyph id's outside the range of valid glyphs.

OTS does not like invalid glyph ids in a camp table.
Fixes issues with warning "Range glyph reference too high"
Fixes #5564, #6648, #3483, #5702, #6343, #5690. Maybe more, we should look for people talking about boxes appearing.

/botio-linux preview

Really nice patch!
r=me, provided that the tests pass, and with the PDF file added to .gitignore.

Nice indeed! It also fixes #6312.

/botio test

Awesome work, thank you!

/botio makeref

I use the latest version and this issue still there.


Add support for passing of Publication Name to Google News Sitemap & ensure title  is encoded correctly.

Add the ability to pass in Publication name instead of just getting it off of the site product.
Add xmlEntities encoding on Title, ImageTitle & ImageCaption fields in the googleNewsSitemap xml

Can we wait until tomorrow to discuss over the phone before merging?


Re-organize terra-form component documentation

Feature Request
Description
The terra-form documentation needs a re-organization.
The goals are:

Better form grouping within components Tab to strictly identify what components are form elements.
Encourage use of the "Field" variant of the form element over the "raw" variant which should only be used in special occasions.

Update the "Field" variants to have more appropriate examples.

Isolate the raw variants under and "advanced" folder.
Add accessibility guides for both best practices for the Field variants and how to correctly compose the 'raw' variants.
Create guides outlining the proper usage of terra-forms.

Additional Context / Screenshots

@ Mentions

Putting this on hold until terra-dev-site issue #311 is fixed.


pick pick #45210

What problem does this PR solve?
fix Disable Jemalloc Hook #45210
fix:
*** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1733734087" if you are using GNU date ***
*** Current BE git commitID: 43f06a5e26 ***
*** SIGSEGV address not mapped to object (@0x0) received by PID 1671420 (TID 1671420 OR 0x7f4f35f74ac0) from PID 0; stack trace: ***
 0# doris::signal::(anonymous namespace)::FailureSignalHandler(int, siginfo_t*, void*) at /home/zcp/repo_center/doris_release/doris/be/src/common/signal_handler.h:421
 1# PosixSignals::chained_handler(int, siginfo*, void*) [clone .part.0] in /usr/local/jdk-17.0.2/lib/server/libjvm.so
 2# JVM_handle_linux_signal in /usr/local/jdk-17.0.2/lib/server/libjvm.so
 3# 0x00007F4F34340400 in /lib64/libc.so.6
 4# je_arena_dalloc_promoted at ../src/arena.c:1277
 5# je_free_default at ../src/jemalloc.c:3014
 6# __pthread_create_2_1 in /lib64/libpthread.so.0
 7# os::create_thread(Thread*, os::ThreadType, unsigned long) in /usr/local/jdk-17.0.2/lib/server/libjvm.so
 8# JVM_StartThread in /usr/local/jdk-17.0.2/lib/server/libjvm.so
 9# 0x00007F4F0A96C918

Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

run buildall

run buildall