Improve the experience with GitHub's tar.gz source archives

GitHub prominently displays tar.gz files of source code in various places:

But they fail to build!
$ ./build.sh
Found bootstrap SDK 3.1.102, bootstrap Arcade 1.0.0-beta.19359.6 fatal: Not a git repository (or any of the parent directories): .git

Instead, you need to actually have a git clone and check out the commit to build.
We should try to improve this experience. Options:

Disable these links. (I haven't seen a way.)

Make the GitHub-provided source archive work. May involve fixes to Arcade scripts.

Detect the situation and give actionable instructions to fix the problem.

Detect the situation and fix it up with a workaround. (Maybe it's enough to git init, even without any source-build repo data?)

Thanks for filing this.  The easiest quick solution is just to have a better error message explaining the need to use a git repository, and not the tarball.  git init might work, but only if there were no attempt to git pull, since there wont be any remote.  If it were possible to get the github built tarball to include the .git folder, all might be well, but I don't know if that's possible.

Yeah, I think git init might be the simplest. I don't know why we would actually need any git data in there. Removing the unnecessary-seeming dependency on a .git dir would be the cleanest, but might need a bit more work.
Another thing we should do is actually provide the tarball output of ./build-source-tarball.sh and indicate it's what you should use to run an offline build. Related:

End-to-end: "create linux-x64 compatible tarball for each release", https://github.com/dotnet/source-build/issues/298
Reasons we potentially can't do this yet due to distro incompatibility: https://github.com/dotnet/source-build/issues/1412 (might be fixed already)
Improving publish infra to reduce manual per-release work to get this done: https://github.com/dotnet/source-build/issues/1321

The experience for 3.1 and 2.1 will be different here, since 2.1 is still using submodules which won't be in the archive.

Another thing we should do is actually provide the tarball output of ./build-source-tarball.sh and indicate it's what you should use to run an offline build.

Yes please. https://github.com/dotnet/sdk/issues/11795#issuecomment-668263844

Another thing we should do is actually provide the tarball output of ./build-source-tarball.sh and indicate it's what you should use to run an offline build.

Yes please. dotnet/sdk#11795 (comment)

You are actually able to do this yourself: you can run ./build-source-tarball.sh <target directory to create> from a Git clone, then that target directory contains the sources/prebuilts you need to do an offline build. That's what's submitted to distros. It would mostly just save time for us to provide this as a release artifact.
Bit more info here:
https://github.com/dotnet/source-build/tree/release/3.1/Documentation#building-net-core-sdk-and-a-source-tarball
https://github.com/dotnet/source-build/blob/release/3.1/Documentation/packaging-installation.md

@dagood, I'm not running a build, I'm only fetching source and performing a compile and install. I don't have CI/CD tooling available, as I'm operating within the confines of a Homebrew formula. I'm able to point to a .tar.gz file that is automatically extracted for me, and then operate on the extracted result.
Of course, I can write whatever Ruby code I can to orchestrate the universe, but that goes sharply against Homebrew's architecture and will most likely just end up with the formula pull request being rejected.
It feels like a very small change to your CI/CD pipeline to make the result of ./build-source-tarball.sh available as a release artifact so I can point the Homebrew formula to it instead of the one GitHub bundles up automatically.

Understood, thanks for expanding on that. And FWIW I can understand the underlying problem that producing and hosting a tar.gz yourself and feeding it into Homebrew (rather than using some artifact directly provided by this repo) seems like it'd be rejected as well for provenance reasons. (If it's even possible to feed it in that way.)

It feels like a very small change to your CI/CD pipeline to make the result of ./build-source-tarball.sh available as a release artifact

We actually don't have CI/CD for GitHub Releases. We have automation for some things, but nothing's published in a way that we could link it in each GitHub Release page. But, this is the purpose of those issues I linked to--to make this possible. I don't think these are hard things, but we're significantly pressed for time, and getting to a better and more sustainable place with source-build infra (in particular for 5.0) as well as continuing to service 2.1 and 3.1 are currently the highest priorities (vs. implementing new infra on existing servicing release trains to help new distro maintainers).

4. Detect the situation and fix it up with a workaround. (Maybe it's enough to git init, even without any source-build repo data?)

I'm going to give this a quick try. It would be trivial if possible, and it has essentially zero chance of adverse side effects because when .git doesn't exist, the build has no chance of doing anything useful anyway.

Understood, thanks for expanding on that. And FWIW I can understand the underlying problem that producing and hosting a tar.gz yourself and feeding it into Homebrew (rather than using some artifact directly provided by this repo) seems like it'd be rejected as well for provenance reasons. (If it's even possible to feed it in that way.)

Indeed.

It feels like a very small change to your CI/CD pipeline to make the result of ./build-source-tarball.sh available as a release artifact

We actually don't have CI/CD for GitHub Releases. We have automation for some things, but nothing's published in a way that we could link it in each GitHub Release page.

I have no idea how to set this up with AzDO, but with GitHub Actions, this is just 10 lines of YAML. I would have submitted a pull request if you would consider sprinkling some GitHub Actions on top of your existing AzDO pipeline.

Detect the situation and fix it up with a workaround. (Maybe it's enough to git init, even without any source-build repo data?)

I'm going to give this a quick try. It would be trivial if possible, and it has essentially zero chance of adverse side effects because when .git doesn't exist, the build has no chance of doing anything useful anyway.

That would be awesome. I'll be the first one to test and let you know how it works out. 👍

I have no idea how to set this up with AzDO, but with GitHub Actions, this is just 10 lines of YAML. I would have submitted a pull request if you would consider sprinkling some GitHub Actions on top of your existing AzDO pipeline.

We won't accept GitHub Actions, but the uploading itself is pretty easy on AzDO too. There's a deeper problem though: we build internally for many servicing releases (e.g. if they have security fixes) so we need to have some new CI thing that pulls artifacts from private -> public storage, or rebuild the entire release but with a flag set to publish those artifacts to public storage. Rebuilding is the easy path here for dev time but wastes CI machines. (There could also potentially be problems with nondeterminism, but we're inflicting that on everyone whether they build from a ./build-source-tarball.sh output or from our source code, so that isn't a consideration here.)
In any case... nothing insurmountable but someone needs to spend some time on it.

That would be awesome. I'll be the first one to test and let you know how it works out. 👍

I posted a PR at https://github.com/dotnet/source-build/pull/1695, seems to work locally although I've only let it run as far as corefx so far. I included links to zip/tar.gz with the workaround included. 😄

https://github.com/dotnet/source-build/pull/1695 with the git init approach is merged into release/3.1, and local build with the GitHub tar.gz worked for me. Closing.


Pneumatic cannon used as an automatic crossbow

Description
Pneumatic cannons can be use to rapid fire arrows
Reproduction
Load PC with gas tank and arrows
fire
unload a volley of arrows (probably coated with all sorts of funny liquid)
Screenshots

Additional context

feature

arrow damage should depend on velocity but otherwise why not

eh.... someone would like to say something

shouldnt do full damage since its not firing from a bow but having an arrow thrown at you should still hurt somewhat

shouldnt do full damage since its not firing from a bow but having an arrow thrown at you should still hurt somewhat

i'll be honest, the arrow damage isn't the main issue, it's the 5 units of chemicals EACH ARROW will inject you

straight up shouldnt inject anything if not shot by a boe to have enough speed
this would also mean you could use your ipecac arrows in an overclocked cannon as a metal bow that stuns you :trollface:

resolved by #21067


🚸 enhance portfolio output

Generated summary (powered by Graphite)

Pull Request Description
TL;DR
This pull request refactors the portfolio display function in the pyrb application. It separates the portfolio table and summary into two distinct functions for better readability and maintainability. It also adds a new feature to display the cash balance of the portfolio.
What changed

Refactored the portfolio function into _print_portfolio_table and _print_portfolio_summary functions.
Added a new feature to display the cash balance of the portfolio.
Added a new abstract property cash_balance in the Portfolio base class and implemented it in the EbestPortfolio class.

How to test

Checkout to this branch.
Run the pyrb application.
Use the portfolio command to display the portfolio table and summary.
Verify that the portfolio table and summary are displayed correctly.
Verify that the cash balance of the portfolio is displayed correctly.

Why make this change

The refactoring improves the readability and maintainability of the code.
The new feature to display the cash balance of the portfolio provides more information to the user, which can be useful for making investment decisions.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #61

PR #62   👈

This stack of pull requests is managed by Graphite.

Merge activity

Dec 29, 11:49 AM: @mingi3314 started a stack merge that includes this pull request via Graphite.


Interval During Question

I have created an interval and am seeing an odd finding. Hoping someone can help clarify.
My interval is this:
Interval[2177-10-18T16:30:00, 2177-10-19T16:30:00]
My target event occurs at this time:
DateTimeType[2177-10-19T00:00:00+00:00]
When I run a query in this clunky way, it returns results correctly:
define "Diagnoses":
[Condition: Code in "My Diagnoses"] Dx
    with "Risk Window" P
    such that Dx.onset after start of P
    and Dx.onset before end of P

But when I run a query using the during syntax it returns empty :
define "Diagnoses":
[Condition: Code in "My Diagnoses"] Dx
    with "Risk Window"  P
    such that Dx.onset during P

It would seem that the two queries are equivalent, but am I missing something?
Thanks!

Those two queries are almost equivalent. The "during" (or "included in") operator has a precision element. When that precision element is not specified, the precision defaults to millisecond precision. Since your interval DateTime boundaries are specified to second, the comparison is uncertain, which will typically evaluate to null.
Try the following query:
define "Diagnoses":
[Condition: Code in "My Diagnoses"] Dx
    with "Risk Window"  P
    such that Dx.onset during day of P

Here are some links to further reading on this subject:
IncludedIn operator
CQL Precision based timing and Uncertainty
Hope that helps!

Thank you so much @c-schuler!  Makes perfect sense.


Allow generic tickers

Would it be possible to make
https://github.com/tkennon/ticker/blob/4a945402cbfbd76c16bda2480f71dd2831068d50/ticker.go#L41
and subsequently
https://github.com/tkennon/ticker/blob/4a945402cbfbd76c16bda2480f71dd2831068d50/ticker.go#L16-L18
public? I want to implement a custom ticker and would rather not vendor the whole library.

Yep, of course. I'll try and find some time this weekend.


[bug] [2018.2] filter-changed event not firing if filter was added from another sheet

It looks like indirectly applied filters are not triggering the filterchanged event on sheets.

Sheet 1 is the sheet i'm listening to filter-changed events on.
filter 1 (sheet 1) was created by adding a filter from sheet 1 and 1 (sheet 2) was created by adding it from sheet 2.
By adding from I mean using this button on the respective sheet:

EDIT:
just to be clear: both filters are set to "apply to worksheets > all using this datasource"

Any updates?

Hi. Thanks for the report. Is this happening on desktop or server/online, or everywhere? I am currently working on a related issue (https://github.com/tableau/extensions-api/issues/179), but that is a server/online issue.

Was tested on desktop... I don't remember testing it on server so I'm not sure. Ill try it out tomorrow ;)

Tested on the tableau online developer preview (2019.1 beta) and it also happens there, so I think it's safe to assume this is happening everywhere.
Wild speculation: this seems to be caused by the extension only receiving filter changed events that are sourced from the worksheet the event listener is attached to, instead of all filter changed events targeting that worksheet.

@johnDance This is still happening in 2019.2 (on Server and Desktop):

Extensions (tables on the right) are reading data (and listening to filter-changed events) from the sheet on their left.
Filter on Segment was added to the dashboard from the dropdown menu of the top left sheet (Sheet 3).
Filter on Segment was set to "filter all using this datasource"
Extension on the top re-reads the summarydata because it received a filter changed event, extension on the bottom does not.

The good news is that we have fixed this internally. It will be part of 2019.3!

I downloaded the 2019.3 prerelease and can confirm it fixes this issue. Thanks

Hi @johnDance
You may have to re-open this ticket/issue since Tableau Server (Tableau Online) does not register the dashboard action when you use the GoTo Dashboard action before it redirects to the new dashboard/sheet. We store the event of the dashboard action but it never gets stored because the GoTo action will have the user leave the dashboard and the extension does not get the important event of the dashboard action.
I can confirm it works in Tableau Desktop (2019.3 and higher) but on tableau Server the registration of the selection doe snot take place. I tested in all versions up to 2020.2 (Tableau Online) and i cannot get the dashboard action to be registered before the GoTo sheet/dashboard action will redirect to the new dashboard/sheet.
This is an urgent issue for us since one of our clients has indicated not to continue their subscription on the extension if this will not be fixed.


Facing ValueError: Non-consecutive added token '<extra_id_99>' found. Should have index 32100 but has index 32000 in saved vocabulary

Hi team,
This is something I have been experimenting with for last 2 days. Really appreciate the work! I am hoping to use this model in my RAG work.
Now, while running your example code on my Linux machine ( intel chip + Nividia RTX 3060 GPU), I am getting the following exception.

ValueError                                Traceback (most recent call last)
Cell In[6], line 9
      7 model_name = "chentong00/propositionizer-wiki-flan-t5-large"
      8 device = "cuda" if torch.cuda.is_available() else "cpu"
----> 9 tokenizer = AutoTokenizer.from_pretrained(model_name)
     10 model = AutoModelForSeq2SeqLM.from_pretrained(model_name).to(device)

File ~/software/anaconda3/lib/python3.11/site-packages/transformers/models/auto/tokenization_auto.py:727, in AutoTokenizer.from_pretrained(cls, pretrained_model_name_or_path, *inputs, **kwargs)
    723     if tokenizer_class is None:
    724         raise ValueError(
    725             f"Tokenizer class {tokenizer_class_candidate} does not exist or is not currently imported."
    726         )
--> 727     return tokenizer_class.from_pretrained(pretrained_model_name_or_path, *inputs, **kwargs)
    729 # Otherwise we have to be creative.
    730 # if model is an encoder decoder, the encoder tokenizer class is used by default
    731 if isinstance(config, EncoderDecoderConfig):

File ~/software/anaconda3/lib/python3.11/site-packages/transformers/tokenization_utils_base.py:1854, in PreTrainedTokenizerBase.from_pretrained(cls, pretrained_model_name_or_path, cache_dir, force_download, local_files_only, token, revision, *init_inputs, **kwargs)
   1851     else:
   1852         logger.info(f"loading file {file_path} from cache at {resolved_vocab_files[file_id]}")
-> 1854 return cls._from_pretrained(
   1855     resolved_vocab_files,
   1856     pretrained_model_name_or_path,
   1857     init_configuration,
   1858     *init_inputs,
   1859     token=token,
   1860     cache_dir=cache_dir,
   1861     local_files_only=local_files_only,
   1862     _commit_hash=commit_hash,
   1863     _is_local=is_local,
   1864     **kwargs,
   1865 )

File ~/software/anaconda3/lib/python3.11/site-packages/transformers/tokenization_utils_base.py:1886, in PreTrainedTokenizerBase._from_pretrained(cls, resolved_vocab_files, pretrained_model_name_or_path, init_configuration, token, cache_dir, local_files_only, _commit_hash, _is_local, *init_inputs, **kwargs)
   1884 has_tokenizer_file = resolved_vocab_files.get("tokenizer_file", None) is not None
   1885 if (from_slow or not has_tokenizer_file) and cls.slow_tokenizer_class is not None:
-> 1886     slow_tokenizer = (cls.slow_tokenizer_class)._from_pretrained(
   1887         copy.deepcopy(resolved_vocab_files),
   1888         pretrained_model_name_or_path,
   1889         copy.deepcopy(init_configuration),
   1890         *init_inputs,
   1891         token=token,
   1892         cache_dir=cache_dir,
   1893         local_files_only=local_files_only,
   1894         _commit_hash=_commit_hash,
   1895         **(copy.deepcopy(kwargs)),
   1896     )
   1897 else:
   1898     slow_tokenizer = None

File ~/software/anaconda3/lib/python3.11/site-packages/transformers/tokenization_utils_base.py:2073, in PreTrainedTokenizerBase._from_pretrained(cls, resolved_vocab_files, pretrained_model_name_or_path, init_configuration, token, cache_dir, local_files_only, _commit_hash, _is_local, *init_inputs, **kwargs)
   2066     raise ValueError(
   2067         f"Wrong index found for {token}: should be {tokenizer.convert_tokens_to_ids(token)} but found "
   2068         f"{index}."
   2069     )
   2070 elif not has_tokenizer_file and index != current_index:
   2071     # Tokenizer slow: added token cannot already be in the vocabulary so its index needs to be the
   2072     # current length of the tokenizer.
-> 2073     raise ValueError(
   2074         f"Non-consecutive added token '{token}' found. "
   2075         f"Should have index {current_index} but has index {index} in saved vocabulary."
   2076     )
   2078 is_special = bool(token in special_tokens)
   2079 if is_last_special is None or is_last_special == is_special:

ValueError: Non-consecutive added token '<extra_id_99>' found. Should have index 32100 but has index 32000 in saved vocabulary.

I try the same code on mac M2 pro(16GB), it works. Though I get the following warning here too,
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.

I am pretty new to this LLM, RAG stuffs. It will be a real help if you can suggest me how to resolve the issue.
Thanks.

My bad, I was running Linux with Python 3.11.5. When changed it to 3.10.12, same as in mac, the error is gone. Only the warning remains.
I am closing the issue.


ANOMALY: Use of REX.w is meaningless (default operand size is 64)

I get this error message for each instance of attempting to hook an API. This appears to be the same issue as #11.
[0x7FFE0A04B210] ANOMALY: use of REX.w is meaningless (default operand size is 64)
[0x7FFE0A04BC10] ANOMALY: use of REX.w is meaningless (default operand size is 64)
[0x7FFE0A04B960] ANOMALY: use of REX.w is meaningless (default operand size is 64)

The memory at the failing location appears to be an indirect jump:
48 FF 25 71 9B 05 00    jmp qword ptr [KERNELBASE.VirtualAlloc]

Throwing that sequence of bytes into ODA gives me this:
rex.W jmp QWORD PTR [rip+0x59b71] # 0x00059b78

Which I suspect is the culprit. OSDev's documentation for REX says that the prefix is valid but ignored for the JMP. I do not know why the Microsoft compiler emits this particular prefix, but it's there.
I tried (rather naively) commenting out the check at this line: https://github.com/martona/mhook/blob/master/disasm-lib/disasm_x86.c#L1745
Unfortunately this only makes the message go away, and doesn't result in a successful hook.
I've added a test case for you to try out: testcase.zip

Fixed in pull request #17.

Was this fixed by https://github.com/apriorit/mhook/pull/20 ?


Add support to increase PVC size dynamically via distributed redis cluster CRD object update

Size of PVC bound to the redis pods should get updated (increased) when updating (increasing) the value of 'storage.size' field in  CRD spec.

Dynamically increasing the PVC size requires the volume plugin or CSI driver for your volume to support dynamic expansion.


ipv6 - Adaptive Server is unavailable or does not exist” error connecting to SQL Server

Hello,
      On ipv4 network my application working good, But on ipv6 fetching the error **Adaptive Server is unavailable or does not exist” error connecting to SQL Server**.

for ipv6 network is there any changes, please let me know if i missing something.

Thanks for the reply,
I am fetching  problem in iOS, there is no option to change connection string.
[client connect:OTP_GD_DB_IP username:OTP_GD_DB_USER_NAME password:OTP_GD_DB_PASSWORD database:@"OTP_DB_NAME" completion:^(BOOL success) {
if (success)
{
}
}

Please see #37.


Make clean fails, leaves buildkitd container running

After running make deploy/make clean (from a clean/default/current env), a buildkitd container is left running.
[shipyard]$ [cluster2] return 0
Successfully remade target file 'deploy'.
[shipyard]$ make --debug=b deploy
[~/go/src/submariner-io/shipyard]$ docker ps
CONTAINER ID   IMAGE                           COMMAND                  CREATED          STATUS          PORTS                       NAMES
1e1e407f6336   kindest/node:v1.20.7            "/usr/local/bin/entr…"   22 minutes ago   Up 22 minutes                               cluster1-worker
ba90b0542162   kindest/node:v1.20.7            "/usr/local/bin/entr…"   22 minutes ago   Up 22 minutes   <IP_ADDRESS>:43503->6443/tcp   cluster1-control-plane
12abcc8c08aa   kindest/node:v1.20.7            "/usr/local/bin/entr…"   22 minutes ago   Up 22 minutes                               cluster2-worker
559de661177f   kindest/node:v1.20.7            "/usr/local/bin/entr…"   22 minutes ago   Up 22 minutes   <IP_ADDRESS>:36469->6443/tcp   cluster2-control-plane
e66461ea50b1   registry:2                      "/entrypoint.sh /etc…"   30 minutes ago   Up 30 minutes   <IP_ADDRESS>:5000->5000/tcp    kind-registry
fa0c03da140a   moby/buildkit:buildx-stable-1   "buildkitd"              41 minutes ago   Up 41 minutes                               buildx_buildkit_buildx_builder0
[~/go/src/submariner-io/shipyard]$ make clean
$ docker ps
CONTAINER ID   IMAGE                           COMMAND       CREATED          STATUS          PORTS     NAMES
fa0c03da140a   moby/buildkit:buildx-stable-1   "buildkitd"   42 minutes ago   Up 42 minutes             buildx_buildkit_buildx_builder0

If I show more debugging info (via https://github.com/dfarrell07/shipyard/pull/30) for make clean, I see:
docker buildx version && \
docker buildx use buildx_builder && \
docker buildx rm
github.com/docker/buildx v0.5.1 11057da37336192bfc57d81e02359ba7ba848e4a
error: failed to find instance "buildx_builder": open /root/.docker/buildx/instances/buildx_builder: no such file or directory
make: *** [Makefile.inc:94: clean-buildx] Error 1

So https://github.com/submariner-io/shipyard/issues/608 isn’t really fixed...

@dfarrell07 @skitt I marked the priority as low as it's not affecting any functionality and it's like a stale resource running on the host. Feel free to modify the priority if required. Thank you.

+1 to prio:low, this is more of an annoyance than a problem.

I confirmed this is still happening:
$ docker ps
# none
$ make deploy
$ docker ps
CONTAINER ID   IMAGE                           COMMAND                  CREATED         STATUS         PORTS                       NAMES
7f3bc7d05167   kindest/node:v1.20.7            "/usr/local/bin/entr…"   6 minutes ago   Up 6 minutes                               cluster1-worker
602232d52a1d   kindest/node:v1.20.7            "/usr/local/bin/entr…"   6 minutes ago   Up 6 minutes   <IP_ADDRESS>:46761->6443/tcp   cluster2-control-plane
b4938a69521a   kindest/node:v1.20.7            "/usr/local/bin/entr…"   6 minutes ago   Up 6 minutes                               cluster2-worker
b880eab03e72   kindest/node:v1.20.7            "/usr/local/bin/entr…"   6 minutes ago   Up 6 minutes   <IP_ADDRESS>:46249->6443/tcp   cluster1-control-plane
32652763a61d   registry:2                      "/entrypoint.sh /etc…"   6 minutes ago   Up 6 minutes   <IP_ADDRESS>:5000->5000/tcp    kind-registry
84429d84c797   moby/buildkit:buildx-stable-1   "buildkitd"              4 weeks ago     Up 7 minutes                               buildx_buildkit_buildx_builder0
$ make clean
$ docker ps
CONTAINER ID   IMAGE                           COMMAND       CREATED       STATUS         PORTS     NAMES
84429d84c797   moby/buildkit:buildx-stable-1   "buildkitd"   4 weeks ago   Up 8 minutes             buildx_buildkit_buildx_builder0

Doesn't reproduce for me, please reopen if still relevant with reproduction steps.

I can confirm this is no longer reproducible. We don’t use buildx for regular images builds, so most people should never see this anyway (unless we start using buildx again); and multi-arch image builds result in a setup where make clean does the right thing.


Migrate to GitHub actions

This PR migrates our CI/CD from Travis to Github Actions.
Instead of one deployment, I split into three workflows:

Run the test suite on every push, in every branch
Deploy to pypi when the default branch (master in our case) receives a new commit and has a diepvries/* tag
Build and deploy docs to Github Pages when the default branch receives a new commit and has a diepvries/* tag

Additionally, because of a pylint update, I also had to change the formatting of the pylint settings in tox.ini
To enable this, I changed the following repository settings:

Add TWINE_PASSWORD to the repository secrets
Change Github Pages deployment source from the gh-pages branch to Github Actions

Fancy! What is the best way to test this myself in a fork?

Quite a lot of small changes. In your fork:
Upload to TestPyPI:

Add a TWINE_PASSWORD to the repository secrets (Settings > Secrets > Actions > New repository secret). You can fill in a token that you get from https://test.pypi.org/, so you don't push to the real pypi.
In tox.ini, change python -m twine upload dist/* to python -m twine upload --repository testpypi dist/*.
TestPyPI would not allow me to upload a package named diepvries. In setup.cfg I changed the name diepvries to deeperfreezer (see: https://test.pypi.org/project/deeperfreezer/0.6.3.dev8/). You can think of some other nice test name (e.g. shallowheater).

Enable Github Pages deployment from Actions:

Ensure that the Github Pages deployment source is Github Actions (Settings > Pages > Source)
In  Settings > Environments, there should be a github-pages environment. If not, create it, and click Add deployment branch rule and add your local branch name. In my case, that's test-migrate-to-github-actions.

Enable test runs of workflows:

Modify the run condition of pages-build-and-deploy.yml and pypi-build-and-deploy.yml to on: [push], which makes it run on every push.

run-test-suite already runs on every push, so it should run automagically.
You can find an overview of the code changes here: https://github.com/michael-the1/diepvries/commit/c09d05617b67c724908c66cfa959a0981fa96099

@dlouseiro Unfortunately, currently there's no option to filter on both branches and tags on a push event.
https://github.com/orgs/community/discussions/25244#discussioncomment-3247065 (old thread but still valid)
The best way to work around this is to start using Releases. Then we can change these workflows to run whenever a new release is posted.

@matthieucan Yes, let's merge the https://github.com/PicnicSupermarket/diepvries/pull/34 first.

start using Releases. Then we can change these workflows to run whenever a new release is posted.

Using release sounds like a good idea!

Ready for merging, only needs a rebase. Once this is done, let's do a patch-level release @michael-the1, to verify everything works end-to-end?


github actions now check for lock file changes

Current dependencies on/for this PR:

develop

PR #31   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: tCEFOsXmIVR1PvpWOGXr


Fix Traefik chart versioning scheme

Fixes #999
cc @c-knowles

Hold off on any consideration of merging this. There might actually be a problem because anywhere the chart version is included in a label, k8s is going to not like the "+" character.

@k8s-bot ok to test


test/blobs: add explicit test for de-duplication

Adds a passing test for de-duplication of two blobs with identical content & mime type, but different filenames.

Test originally written as part of https://github.com/getodk/central-backend/pull/1073.
This test passes, whereas the other tests (which demonstrate https://github.com/getodk/central-backend/issues/1219) require more complex changes.


add PeerConnection.GetRemoteDTLSCertificate

Description
Previously it was not possible to get DTLS certificate of the remote
peer from PeerConnection.
Refs https://github.com/libp2p/specs/pull/412#discussion_r912612288
Copy of https://github.com/webrtc-rs/webrtc/pull/221

Let me know if you think this makes sense or not. Thanks 🙏

This looks fantastic @melekes nice work!
I am going to improve the test a bit (assert that Certificates equal) then merge :)

hi @melekes
Thanks for all the work you have done on this, but unfortunately this doesn't match the WebRTC API. GetRemoteCertificates doesn't exist on the PeerConnection in the WebRTC API. You can get this information today like this
package main

import (
    "fmt"

    "github.com/pion/webrtc/v3"
)

func main() {
    offerer, err := webrtc.NewPeerConnection(webrtc.Configuration{})
    if err != nil {
        panic(err)
    }

    answerer, err := webrtc.NewPeerConnection(webrtc.Configuration{})
    if err != nil {
        panic(err)
    }

    if _, err = offerer.CreateDataChannel("foo", nil); err != nil {
        panic(err)
    }

    offer, err := offerer.CreateOffer(nil)
    if err != nil {
        panic(err)
    }

    gatheringStatePromise := webrtc.GatheringCompletePromise(offerer)
    if err = offerer.SetLocalDescription(offer); err != nil {
        panic(err)
    }
    <-gatheringStatePromise

    if err = answerer.SetRemoteDescription(*offerer.LocalDescription()); err != nil {
        panic(err)
    }

    answer, err := answerer.CreateAnswer(nil)
    if err != nil {
        panic(err)
    }

    gatheringStatePromise = webrtc.GatheringCompletePromise(answerer)
    if err = answerer.SetLocalDescription(answer); err != nil {
        panic(err)
    }
    <-gatheringStatePromise

    if err = offerer.SetRemoteDescription(*answerer.LocalDescription()); err != nil {
        panic(err)
    }

    connectionStateChan := make(chan webrtc.PeerConnectionState, 100)
    offerer.OnConnectionStateChange(func(p webrtc.PeerConnectionState) { connectionStateChan <- p })
    answerer.OnConnectionStateChange(func(p webrtc.PeerConnectionState) { connectionStateChan <- p })

    connectedCount := 0
    for c := range connectionStateChan {
        if c == webrtc.PeerConnectionStateConnected {
            connectedCount++
        }
        if connectedCount == 2 {
            break
        }
    }

    fmt.Println("Answerer Certificate (As seen by Offerer)")
    fmt.Println(offerer.SCTP().Transport().GetRemoteCertificate())

    fmt.Println("Offerer Certificate (As seen by Answerer)")
    fmt.Println(answerer.SCTP().Transport().GetRemoteCertificate())
}

Ah, thanks! I saw that PeerConnection gives access to SCTP, but didn't look further.


Move About Us to a new page

The current about us is fine but I think it'll be better if we move it to another page.

Kindly add margin below the page to allow a bit of breathing room. It feels a bit cramped.

Moved Issue to "In Review" as this is only under review. Only remaining change is for the said requested change from the last comment and this issue will be closed.

Awaiting implementation of dynamic information via Issue #43, which was just finished recently. Moving back to "In Progress" category


[FEATURE REQUEST] Add "group by month/quarter/year" option in the reports with a date range modal

Is your feature request related to a problem? Please describe.
I have been using YNAB for almost 5 years and I like to see trends in reports over time but it is starting to get really crowded and difficult to compare. It would be awesome to be able to group the x coordinate by month/quarter/year. I am trying to get some answers like how much did my NW increase in 2021 vs 2022 for instance (or even per quarter)

Describe the solution you'd like
In the date range modal, add a dropdown to select a "group by" option with month/quarter/year. It would month by default so it can be backward compatible.
Describe alternatives you've considered
No alternative options are available as far as I know

Just to confirm, this would be under the page Reports > Inflow/Outflow, right?

It is for all reports in general. It would be great to see this options for Reports > Inflow/Outflow and Reports > Net Worth. Thanks


[Blocked] Run e2e tests in Spartacus model T / SPA-1414

New server is: https://storefront.c39j2-walkersde1-d4-public.model-t.cc.commerce.ondemand.com

Model T is down and I need help on how to troubleshoot it


help

Can I run it as windows with .exe
and how can i install it on cpanel hosting.

Can I run it as windows with .exe

use waifu2x-caffe or waifu2x-ncnn-vulkan.
https://github.com/lltcggie/waifu2x-caffe
https://github.com/nihui/waifu2x-ncnn-vulkan

how can i install it on cpanel hosting.

I don't know.


Docs missing info

Custom formatting (Format.register) is not explained in the docs.

Repeater widget is used but never explained in the docs. It could be covered under General purpose widgets.

Format.register is now documented. Repeater is documented in Data Views.


Can't seem to get it to work in Ubuntu 22.04

 pip install certbot-dns-cloudns --break-system-packages
Defaulting to user installation because normal site-packages is not writeable
Requirement already satisfied: certbot-dns-cloudns in ./.local/lib/python3.12/site-packages (0.6.0)
Requirement already satisfied: acme<2.8.0,>=2.7.4 in ./.local/lib/python3.12/site-packages (from certbot-dns-cloudns) (2.7.4)
Requirement already satisfied: certbot<2.8.0,>=2.7.4 in ./.local/lib/python3.12/site-packages (from certbot-dns-cloudns) (2.7.4)
Requirement already satisfied: cloudns-api==0.9.3 in ./.local/lib/python3.12/site-packages (from certbot-dns-cloudns) (0.9.3)
Requirement already satisfied: dnspython<3.0.0,>=2.3.0 in ./.local/lib/python3.12/site-packages (from certbot-dns-cloudns) (2.6.1)
Requirement already satisfied: zope.interface>=5.4.0 in /usr/lib/python3/dist-packages (from certbot-dns-cloudns) (6.1)
Requirement already satisfied: requests in /usr/lib/python3/dist-packages (from cloudns-api==0.9.3->certbot-dns-cloudns) (2.31.0)
Requirement already satisfied: cryptography>=3.2.1 in /usr/lib/python3/dist-packages (from acme<2.8.0,>=2.7.4->certbot-dns-cloudns) (41.0.7)
Requirement already satisfied: josepy>=1.13.0 in /usr/lib/python3/dist-packages (from acme<2.8.0,>=2.7.4->certbot-dns-cloudns) (1.14.0)
Requirement already satisfied: PyOpenSSL!=23.1.0,>=17.5.0 in /usr/lib/python3/dist-packages (from acme<2.8.0,>=2.7.4->certbot-dns-cloudns) (23.2.0)
Requirement already satisfied: pyrfc3339 in /usr/lib/python3/dist-packages (from acme<2.8.0,>=2.7.4->certbot-dns-cloudns) (1.1)
Requirement already satisfied: pytz>=2019.3 in /usr/lib/python3/dist-packages (from acme<2.8.0,>=2.7.4->certbot-dns-cloudns) (2024.1)
Requirement already satisfied: setuptools>=41.6.0 in /usr/lib/python3/dist-packages (from acme<2.8.0,>=2.7.4->certbot-dns-cloudns) (68.1.2)
Requirement already satisfied: ConfigArgParse>=1.5.3 in /usr/lib/python3/dist-packages (from certbot<2.8.0,>=2.7.4->certbot-dns-cloudns) (1.7)
Requirement already satisfied: configobj>=5.0.6 in /usr/lib/python3/dist-packages (from certbot<2.8.0,>=2.7.4->certbot-dns-cloudns) (5.0.8)
Requirement already satisfied: distro>=1.0.1 in /usr/lib/python3/dist-packages (from certbot<2.8.0,>=2.7.4->certbot-dns-cloudns) (1.9.0)
Requirement already satisfied: parsedatetime>=2.4 in /usr/lib/python3/dist-packages (from certbot<2.8.0,>=2.7.4->certbot-dns-cloudns) (2.6)
johnny@proxy:~$ sudo certbot certonly --authenticator dns-cloudns
Saving debug log to /var/log/letsencrypt/letsencrypt.log
The requested dns-cloudns plugin does not appear to be installed
Ask for help or search for solutions at https://community.letsencrypt.org. See the logfile /var/log/letsencrypt/letsencrypt.log or re-run Certbot with -v for more details.

any recommendation or what I am doing wrong?

Hi,
It looks like the python version running certbot might not match the one you're using to install the authenticator plugin, or at least it can't find the package installed in ./.local/lib/python3.12/site-packages. Try running certbot with the -v option to get a bit more info. Feel free to report back with the new output and the contents of /var/log/letsencrypt/letsencrypt.log and I will have another look.

I ran pip install certbot-dns-cloudns with sudo and now it works...
But now I am getting this error.
 sudo certbot certonly --authenticator dns-cloudns --dns-cloudns-credentials ~/.secrets/certbot/cloudns.ini -d misery.dns-dynamic.netSaving debug log to /var/log/letsencrypt/letsencrypt.logEnter email address (used for urgent renewal and security notices) (Enter 'c' to cancel): @.***

Looks like you have a ClouDNS free tier account, which isn't eligible to use the HTTP API. See https://www.cloudns.net/premium


Test apple_support CI toolchain issues

Hello @BalestraPatrick, modules you maintain (apple_support) have been updated in this PR. Please review the changes.

you can ignore, just testing


Build Failure

command “mvn clean spring-boot:run” failing for . ../java-artefacts/data-holder with the error:
An exception occurred while running. null: InvocationTargetException: java.io.CharConversionException: Invalid UTF-32 character 0x42746431 (above 0x0010ffff) at char #1, byte #7) -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.springframework.boot:spring-boot-maven-plugin:2.1.5.RELEASE:run (default-cli) on project data-holder: An exception occurred while running. null
As raised by @anzbankau:
https://github.com/ConsumerDataStandardsAustralia/engineering/issues/52#issuecomment-539832429

Cannot reproduce.
Even though the artifacts are deployed in the maven central repository, it's a good idea to run mvn clean install in the parent (java-artefacts) directory once before running mvn spring-boot:run in the data-holder directory. No need to clean in the data-holder directory afterwards.

@anzbankau, I cannot reproduce it either. This exception should have been thrown by jackson when reading data from files under payloads/products folder within data-holder module. Please check if there is any change to the files. Meanwhile I have upgraded jackson version from 2.9.9 to 2.9.10 to see if any bug fix in the new version can solve this issue.
The change has been made in codegen-attempt branch and will be ready in 1.0.0-RC2 release.

hi @anzbankau
We've just released 1.0.0-RC2. Can you please checkout the latest code in master and see if the issue persists?


Add meta tags for the index page and update it according to the current challenge page

Changes proposed in this pull request:

Added default meta tags in index.html

Updated these meta tags for the particular challenge with the challenge title, challenge logo and it's url and also with the short description of the challenge

Link to live demo: http://pr-176-evalai.surge.sh

Codecov Report

Merging #176 into master will increase coverage by 0.11%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #176      +/-   ##
==========================================
+ Coverage   52.32%   52.43%   +0.11%
==========================================
  Files          58       58
  Lines        2968     2975       +7
  Branches      333      333
==========================================
+ Hits         1553     1560       +7
  Misses       1332     1332
  Partials       83       83

Impacted Files
Coverage Δ

...rc/app/components/challenge/challenge.component.ts
98.93% <100%> (+0.08%)
:arrow_up:

Impacted Files
Coverage Δ

...rc/app/components/challenge/challenge.component.ts
98.93% <100%> (+0.08%)
: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 2f56ef0...eeada21. Read the comment docs.

@lunayach @RishabhJain2018 @Shekharrajak @galipremsagar
Please review


Suppport for innerText

Adds support for innerText unless the superior textContent is used. Thanks!

lgtm.

are you using the serialize function as a toHtmlString() for real dom elements ?

Yep here, should I not be yet?

@shama it's reasonable; seems a bit heavy.

So far I've found require('min-document/serialize') to work the best when serializing vtrees, do you know of a better module for it? The goal I'm after is pre-rendering my vtrees to static HTML and then having the client side scripts take over once loaded.

@shama https://github.com/Matt-Esch/virtual-dom/wiki#vdom-to-html

Thanks! Using that now (pending a PR to it).


expireat loss timezone info

https://github.com/aio-libs/aioredis-py/blob/89b63b14878746370ba7084d58762c524471c7f4/aioredis/client.py#L1817-L1824
Timezone info loss in line 1823, is it a better way to use timestamp?
if isinstance(when, datetime.datetime):
    when = int(when.timestamp())

ref: https://github.com/andymccurdy/redis-py/issues/598
Not fixed in either repository. The fix? Maybe (?): https://github.com/MobPro/redis-py/commit/aa0e789d0fe60267dc56ce164bf72ebb84e43314
Concern: we must maintain compatibility with the previous implementation... and maybe we should stay in touch with redis-py API such that migration is not a surprise to anyone?


Add ubuntu source for installing new version unconv

Add ubuntu source to upgrade new version of unoconv. Debian jessie still uses old version
Related PR in tfk-api-unoconv https://github.com/zrrrzzt/tfk-api-unoconv/pull/23

thx @vincentkwok 👍


server: fix resource reservation leakage

Description
Some entries are left over in resource_reservation table if there is a breach in resource limits or if the management server crashes.
This PR fixes that and also adds additional information in resource_reservation table to track when was a reservation created and who created it.
We also cleanup any stale entries (older than resource.reservation.cleanup.delay, defaults to 3600 seconds) as part of the recalculate resource count method.

Steps to reproduce:

Define 2 resource limit host tags - t1, t2
Set overall user_vm limit to some higher value - 10
Set t1 user_vm limit to a higher value - 5
Set t2 user_vm limit to 1Now you'll be able to deploy one VM with t2 tag. When you will try to deploy another VM with t2 tag it should leave some leftover reservations for null and t1 tag

Now you'll be able to deploy one VM with t2 tag. When you will try to deploy another VM with t2 tag it should leave some leftover reservations for null and t1 tag

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@vishesh92 a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@vishesh92 a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 9775

@blueorangutan test

@vishesh92 a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-10369)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 47752 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9169-t10369-kvm-centos7.zip
Smoke tests completed. 132 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File


TODO: Pool accepts None as processes argument

Case if parallel and threads not None: in tsfeatures not needed.

Solved,


DO NOT MERGE: Testing c/image mtrmac:dependencies

Signed-off-by: Miloslav Trmač<EMAIL_ADDRESS>

/retest

Apparently the rdoproject.org/github-check failure is expected (??!).

Apparently the rdoproject.org/github-check failure is expected (??!).

Unfortunately yes.


Adjust font size

From user feedback, it would be good to let users increase or decrease the font size for the content of the textbook. I'm adding design as a label because there is currently no UI pattern for this.

This repository is no longer maintained. It has been superseded by IBM Quantum Learning.


Benchmark CPU metric values

Description of changes

Fix logic bug for IsKnown function
minor linting and formatting for files
Fix issue since Base was dropped from the struct causing struct to not implement an interface
Introduced Bounds struct to define the acceptable upper and lower bounds that a metric value should be from our testing. This is compared against the average of the values for the given metric.
Added benchmark values for CPU plugin

License
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Tests
https://github.com/SaxyPandaBear/amazon-cloudwatch-agent/actions/runs/3809070507/jobs/6480241725
null_resource.integration_test (remote-exec): 2022/12/30 22:01:49 ==============CPU==============
null_resource.integration_test (remote-exec): 2022/12/30 22:01:49 ==============Successful==============
null_resource.integration_test (remote-exec): cpu_usage_guest        [0 0]                                         Successful
null_resource.integration_test (remote-exec): cpu_usage_guest_nice   [0 0]                                         Successful
null_resource.integration_test (remote-exec): cpu_usage_nice         [0 0]                                         Successful
null_resource.integration_test (remote-exec): cpu_time_nice          [4.11 4.11 4.11]                              Successful
null_resource.integration_test (remote-exec): cpu_usage_active       [0.27076564192279584 0.2541666666667197]      Successful
null_resource.integration_test (remote-exec): cpu_time_iowait        [24.01 23.96 23.92]                           Successful
null_resource.integration_test (remote-exec): cpu_time_steal         [18.83 18.68 18.53]                           Successful
null_resource.integration_test (remote-exec): cpu_time_system        [46.02 45.72 45.48]                           Successful
null_resource.integration_test (remote-exec): cpu_time_user          [194.4 194.25 194.07]                         Successful
null_resource.integration_test (remote-exec): cpu_usage_irq          [0 0]                                         Successful
null_resource.integration_test (remote-exec): cpu_usage_steal        [0.06248437890527311 0.06249999999999941]     Successful
null_resource.integration_test (remote-exec): cpu_time_guest         [0 0 0]                                       Successful
null_resource.integration_test (remote-exec): cpu_time_idle          [2125.96 1886.55 1647.16]                     Successful
null_resource.integration_test (remote-exec): cpu_time_active        [289.5<PHONE_NUMBER>0000014 288.24]             Successful
null_resource.integration_test (remote-exec): cpu_time_guest_nice    [0 0 0]                                       Successful
null_resource.integration_test (remote-exec): cpu_usage_idle         [99.7292343580772 99.74583333333327]          Successful
null_resource.integration_test (remote-exec): cpu_usage_iowait       [0.020828126301758194 0.016666666666666313]   Successful
null_resource.integration_test (remote-exec): cpu_usage_softirq      [0 0]                                         Successful
null_resource.integration_test (remote-exec): cpu_usage_system       [0.12496875781054917 0.10000000000000082]     Successful
null_resource.integration_test (remote-exec): cpu_usage_user         [0.06248437890527606 0.07500000000000284]     Successful
null_resource.integration_test (remote-exec): cpu_time_irq           [0 0 0]                                       Successful
null_resource.integration_test (remote-exec): cpu_time_softirq       [2.13 2.13 2.13]                              Successful

Changing how I want to approach this, so this is irrelevant now


[Question] how can I use inFipsJvm option in gradle for enabling the FIPS 140 mode for the build?

Is your feature request related to a problem? Please describe
I tried -Porg.opensearch.gradle.info.BuildParams.inFipsJvm=true and -PinFipsJvm but the build logs still say In FIPS 140 mode: false as false, so what can we do to enable FIPS build?
Describe the solution you'd like
I just wanted to ask a question
Related component
Build
Describe alternatives you've considered
No response
Additional context
No response

[Triage - attendees 1 2 3]
@developer-guy Thanks for creating this issue - @peternied can follow up with an SME or insight into what to do next.

Hi @peternied, thanks so much, I would also love to know your thoughts on it!


Real 1.0.1

Spam request. I get the feeling people are using the PetClnic as some toy project to learn spring and github!


Issue with PersistentTabView

I am new to this package and I have been trying to set up a bottom_navigation component which can be used on selected pages within my app.
Below is my bottom_navigation.dart:
import 'package:mobile_app/utils/library/my_app.dart';

class BottomNavBar extends StatelessWidget {
  final PersistentTabController controller;

  const BottomNavBar({super.key, required this.controller});

  List<PersistentTabConfig> _tabs() => [
    PersistentTabConfig(
      screen: const HomePage(),
      item: ItemConfig(
        icon: const Icon(Icons.home),
        title: "Home",
      ),
    ),
    PersistentTabConfig(
      screen: const ProfileViewPage(),
      item: ItemConfig(
        icon: const Icon(Icons.message),
        title: "Messages",
      ),
    ),
    PersistentTabConfig(
      screen: const ProfileViewPage(),
      item: ItemConfig(
        icon: const Icon(Icons.settings),
        title: "Settings",
      ),
    ),
  ];

  @override
  Widget build(BuildContext context) {
    return PersistentTabView(
      navBarOverlap: const NavBarOverlap.full(),
      controller: controller,
      tabs: _tabs(),
      navBarBuilder: (navBarConfig) => Style4BottomNavBar(
        navBarConfig: navBarConfig,
        navBarDecoration: const NavBarDecoration(
          //boxShadow: const [BoxShadow(color: Colors.grey, blurRadius: 5)],
          // color: bottombackgroundColor,
        ),
      ),
      backgroundColor: AppColors.primary,
      stateManagement: true,
      popAllScreensOnTapAnyTabs: true,
      popActionScreens: PopActionScreensType.all,
      screenTransitionAnimation: const ScreenTransitionAnimation(
        curve: Curves.ease,
        duration: Duration(milliseconds: 200),
      ),
    );
  }
}

This for instance is my homepage:
import 'package:mobile_app/utils/library/my_app.dart';

class HomePage extends StatefulWidget {
  const HomePage({super.key});

  @override
  State<HomePage> createState() => _HomePageState();
}

class _HomePageState extends State<HomePage> {
  final PersistentTabController _controller = PersistentTabController(initialIndex: 0);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Home Page'),
      ),
      body: const Text("Welcome home"),
      bottomNavigationBar: BottomNavBar(controller: _controller),
    );
  }
}

However, I get an issue with the navbar being populated and repeated across the entire page.

I'd appreciate the help in the right direction. Thank you.

Hi, yes there is a misconception about what the PersistentTabView does. In essence, the PersistentTabView is like a  Scaffold that holds your pages and the bottom navigation bar. So you need to remove the BottomNavBar from the Scaffold in your HomePage, ProfileViewPage and so on

Hey @jb3rndt, thanks I was able to figure out what the issue was.


Accessibility Issues:

Detailed description

Describe in detail the issue you're having.

The <ibm-combo-box> component fails to pass the accessibility specifications outlined by IBM. There are two violations found for the v3 <ibm-combo-box> and issues appear to persist in the v4 Combobox as well. The label created doesn't point to either of the elements given roles; these being the actual <ibm-combo-box> given the role "combobox" and an internal div given the role "listbox".
In some sandbox testing, there appears to be a potential for the id property to be duplicated as well, but this could not be verified in an active codebase.

Is this a feature request (new component, new icon), a bug, or a general issue?

Bug

Is this issue related to a specific component?

<ibm-combo-box>

What did you expect to happen? What happened instead? What would you like to see changed?

Expected for <ibm-combo-box> to pass accessibility scans. The Combobox component failed the following rules:

Form control with "combobox" role has no associated label
Form control with "listbox" role has no associated label

What browser are you working in?

Chrome

What version of the Carbon Design System are you using?

v3. However, some issues appear to persist in v4.

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

ibm.com marketplace checkout

Steps to reproduce the issue

Install IBM accessibility plugin.
Visit an ibm product page - example.
a. click the "Continue to checkout" button.
b. sign in to w3.
Start an accessibility scan as described on the accessibility plugin page.
Expanding the violations dropdown.
b. expand the "Each form control must have an associated label" dropdown.
Clicking on one of the violations now presented should highlight that portion in the code, showing the <ibm-combo-box>

Possible fixes
Adding aria properties to some of the elements removed some of the problems.
For the main Combobox, adding the property aria-label and a label string can fix the issue. Also, adding the aria-labelledby property to the Combobox and a corresponding id to the label can also fix the issue.
For the nested div, these can also fix the issue where the div doesn't have a corresponding label.

Closing since this doesn't seem to be an issue in v4 and #1761 takes care of it for v3


Migrate to transparency-dev/merkle

Use the new merkle repo for doing proof verification.
The new repo is a much lighter-weight dependency.

Seems to be an issue with the linter, I've opened #75 which addresses it.

lgtm, but I do not understand why leaves_pr action is failing:
error: Could not access 'origin/master'
Running the action locally does not raise that error. Do you have any hint?

lgtm, but I do not understand why leaves_pr action is failing: error: Could not access 'origin/master' Running the action locally does not raise that error. Do you have any hint?

I've had a quick look but I'm not sure what's going on with the leaves_pr fail; nothing's changed with the action or the config for it on this repo that I can see, so presumably must be some upstream change in one of the supporting actions (checkout?) or the GHA environment?
Should be safe to ignore for this PR at least as we're not modifying anything under log/.


SAP spool administration: OS system commands

Our company is currently trying to manage SAP printouts via CUPS.
The installation of CUPS went without problems.
But during test prints in SAP with access method E (external output mangement), we encountered the problem that the OS commands of the CUPS LOMS (logical output management system) are not defined in SAP. Here is an example from the currently used OMS (VPSX):

Where can we find the commands for CUPS or do we have to define them ourselves?
CUPS runs on a linux server.
Thank you in advance for any feedback.

@Moezd2203 Unfortunately, I don't think you'll find anyone on the OpenPrinting team with SAP experience. You might ask the SAP folks - after all, CUPS is the standard printing system on all Linux systems, and has been for almost 20 years, so I'm shocked that they don't already have something in place for this!


Attempting to view vehicle info on a scheduled route shows "Error connecting"

Expected behavior
Either no ability to look at Vehicle info if it's not available, or a clearer message that vehicle info isn't available with scheduled rather than actual data.
Actual behavior
Vehicle Info screen appears, showing a title of either "Error connecting" or "Unknown error" and "Updating..." in the view.
Steps to reproduce the behavior

Open a stop with several route arrivals
Choose an arrival showing "scheduled arrival" instead of an actual live time
Tap Vehicle Info at the bottom of the route list screen
Notice the Vehicle Info screen...

Screenshot of the issue (if applicable)

I recall seeing it for quite a while, so if there are no substantive complaints it can totally wait.

cool, 2.6.1 it is


Issue Loading Data

Hello,
I have been unable to unzip this data.zip file. Is it possibly broken?
Additionally, is there pre-processed Seq2Seq available for download? (uploading as a Huggingface dataset would be awesome)
Thanks!

Hi Joseph,
the problem with the data.zip file might come from an incorrectly set up Git LFS. Did you run git lfs install and try pulling again? A telltale sign is usually that the file is only 1kb in size (or similarly small)
If it still doesn't work, feel free to let me know, ideally by posting the exact error message you're getting.
As for the data, unfortunately I do not think that we have the necessary rights to publish it publicly, so unfortunately I can only point you to the script we provide that generates weakly labeled data with heideltime (here).
Best,
Dennis


Add a Makefile and a Dockerfile which automates the build and release steps

The following build commands are supported:
Build:
make
Run tests:
make test
Make a release:
make release
Make a release and build the docker image:
make image
Publish the docker image:
make image-push

Strongly against Docker image publishing via docker push vs. using Automated Builds from source on Docker Hub as an official Docker release method.
Automated Builds create image tags from git tags & branches and provide code traceability, opposed to docker push'ing.
See #372 and #460.


Missing license

I would like to use some code of yours. Could you add a license file.

Hi @stnby,
Thank you for your interest and for reaching out regarding the license. I apologize for the delayed response. I've added a MIT License file to the repository, so feel free to use the code in your projects!
Contributions are much appreciated. If you have any suggestions, improvements, or features you'd like to see, please don't hesitate to let me know or submit a pull request. 🙂

Thank you!


BannerStyle adaptive has no button padding

Hi, thanks for the package. Is there a way to customize the button?

Hi, could you be more explicit please. What are you trying to do ? What is blocking you in the package API ?

I tested the web running on macos, and the button styling are off since it's a macOS (different from iOS). That's not handled yet.

Yeah currently the package only implement the specs for Android and iOS. I'm not sure how the banner should be designed with a desktop view 🤔

Probably add detection for .macOS, should just use the iOS?
Ive done this though in userland

Sorry for the wait @sawirricardo, I've published a fix on the repo to detect when the browser is running on the MacOS platform and display the iOS style banner.
I'll publish an update of the package as soon as I can.


create all tmpdirs inside build_dir

CLOUDBLD-7743

Signed-off-by: Harsh Modi<EMAIL_ADDRESS>Maintainers will complete the following section

[ ] Commit messages are descriptive enough
[ ] Code coverage from testing does not decrease and new code is covered
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Changes to metadata also update the documentation for the metadata
[ ] Pull request has a link to an osbs-docs PR for user documentation updates
[ ] New feature can be disabled from a configuration file

Noticed one more: https://github.com/containerbuildsystem/atomic-reactor/blob/eb2f4efe480d458ccd663282d8f369636de871c6/atomic_reactor/plugins/post_rpmqa.py#L93
What about NamedTemporaryFiles? I guess those can just stay in /tmp?

Noticed one more:
https://github.com/containerbuildsystem/atomic-reactor/blob/eb2f4efe480d458ccd663282d8f369636de871c6/atomic_reactor/plugins/post_rpmqa.py#L93
What about NamedTemporaryFiles? I guess those can just stay in /tmp?

mmm actually there is also plugins/post_rpmqa.py:        with tempfile.TemporaryDirectory() as rpmdb_dir:
I'd say all should be in that workdir

Just throwing this idea out there - would it be better to put all the temporary directories and files in a "temp" subdirectory of the build_dir rather than directly in the build_dir root?
The build_dir already contains the cloned repo and I'm also planning to keep all the per-architecture build directories in it. While naming conflicts between tempfiles and per-arch build dirs are very unlikely, I would still feel calmer if I knew everything temporary is hidden away in a subdir

Just throwing this idea out there - would it be better to put all the temporary directories and files in a "temp" subdirectory of the build_dir rather than directly in the build_dir root?
The build_dir already contains the cloned repo and I'm also planning to keep all the per-architecture build directories in it. While naming conflicts between tempfiles and per-arch build dirs are very unlikely, I would still feel calmer if I knew everything temporary is hidden away in a subdir

sounds good to me

@hjmodi; I see os.path.join(workflow.source.workdir, 'temp') a lot - couldn't it be stashed in the workflow somehow (e.g. workflow.tmp_dir or something)?

@hjmodi; I see os.path.join(workflow.source.workdir, 'temp') a lot - couldn't it be stashed in the workflow somehow (e.g. workflow.tmp_dir or something)?

we're using different names now :D


Info > Pygame FAQ link is broken

https://www.pygame.org/wiki/FrequentlyAskedQuestions returns an Internal Server Error.

Thanks.


client/Windows/wf_cliprdr.c: handle malloc error

I believe the root issue you wanted to fix is a size == 0 case? that is also not handled properly - no, I caught that error from cppcheck. I'll have a look at suggestions as well

@freerdp-bot test please


Virtualbox fails on Unstable to build

Issue description
I have VBox and Extensions in my configuration.nix. It fails to build now on unstable
checking for getc_unlocked... yes
checking for getcwd... yes
checking for getegid... yes
checking for geteuid... yes
checking for getgid... yes
checking for getuid... yes
checking for mempcpy... yes
checking for munmap... yes
checking for putenv... yes
checking for setenv... yes
checking for setlocale... yes
checking for stpcpy... yes
checking for strcasecmp... yes
checking for strdup... yes
checking for strtoul... yes
checking for tsearch... yes
checking for __argz_count... yes
checking for __argz_stringify... yes
checking for __argz_next... yes
checking for __fsetlocking... yes
checking for iconv... yes
checking for iconv declaration... install-shextern size_t iconv (iconv_t cd, char * *inbuf, size_t *inbytesleft, char * *outbuf, size_t *outbytesleft);
checking for nl_langinfo and CODESET... tar: Skipping to next header

bzip2: Data integrity error when decompressing.
        Input file = (stdin), output file = (stdout)

It is possible that the compressed file(s) have become corrupted.
You can use the -tvv option to test integrity of such files.

You can use the `bzip2recover' program to attempt to recover
data from undamaged sections of corrupted files.

tar: Child returned status 2
tar: Error is not recoverable: exiting now
do not know how to unpack source archive /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2
builder for ‘/nix/store/0ipjp7m8758c517g661a17fssvffxwpb-virtualbox-5.1.26.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/nwz38h1zm19gm3rr27mf7i66v2v3hda5-nixos-system-subi-18.03pre116572.9824ca6975.drv’: 1 dependencies couldn't be built

Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)

18.03pre116347.272c0bf88c (Impala)

Nix version: (run nix-env --version)

nix-env (Nix) 1.11.15

Nixpkgs version: (run nix-instantiate --eval "<nixpkgs>" -A lib.nixpkgsVersion)

"18.03pre116572.9824ca6975"

Sandboxing enabled: (run grep build-use-sandbox /etc/nix/nix.conf)

build-use-sandbox = false

Does it fail if you try to build again?

yes, tried about 10x or so.... even wiped everything and started over...

What is the output of this commands?
ls -l /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2
sha256sum /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2

root@subi:~# ls -l /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2 -r--r--r-- 1 root root 115915813  1. Jan 1970  /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2 root@subi:~# sha256sum /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2 ec31ee7d8f31b74daa680b2dc93de081a8c383877db3d041f56a1ffc4322b0ee  /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2

Does this print anything?
nix-store --verify-path /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2

root@subi:~# nix-store --verify-path /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2
path ‘/nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2’ was modified! expected hash ‘a0efe0b41a32c4e6623e22eabd62feb116a2b13faf2f6a5048d712f6ca59bc1b’, got ‘98f9d2a9c048e5cf292f233cc93833e8962ba3a1ef6af8faf04db0f05b837db9’
root@subi:~#

Please remove this file and build again:
sudo nix-store --delete --ignore-liveness /nix/store/r9pdz70s6rjn1w8zm6sncgg2l6mwys6p-VirtualBox-5.1.26.tar.bz2

VirtualBox sources have not been updated since July. The size of your file is correct, but the hash sum is wrong. My guess is that you had that file cached for a while and then it got corrupted without any action from nix tools. You may also want to check health status of your disk and integrity of the file systems.

ssd is fine as is the filesystem.
But deleteing seems to have worked...


New regex breaks url's

Since regex removes everything after //, url's are also removed

Hi @Nuffic,
thanks for your contribution, but new regexp also removes comments from html which is not good.
I've made some test with added cases here: https://regex101.com/r/ijYPvS/1, and all seems good except one case. So if you want, you may add that regex, adapt test cases and I'll merge your request.


Create new text/Edit text's texts fields don't resize horizontally.

These resize vertically but not horizontally.

it can be set to resize in both axes by adding this rule to custom styles:
#translation {
resize: both;
}

it can be set to resize in both axes by adding this rule to custom styles: #translation { resize: both; }

I just tried this which fixes it on all the pages:
textarea { resize: both; }

Right we should make this part of the default, I guess.  I can't see any reason not to have it in defaults.  @webofpies thoughts?  Anything weird this will do?

@jzohrab it seems there's nothing weird. it'll go back to default size after each term form load.

Having testing this out a bit, it works well enough with 'textarea { resize: both; }'. It can deform the rest if you expand it too much ,but it's useful in general.
You can only test it for yourself, @jzohrab. See how it feels.

Cheers will do!  If useful, great, quick addition. 👋
El El mié, feb. 14, 2024 a la(s) 7:43 a. m., M-Biggles <
@.***> escribió:

Having testing this out a bit, it works well enough with 'textarea {
resize: both; }'. It can deform the rest if you expand it too much ,but
it's useful in general.
You can only test it for yourself, @jzohrab https://github.com/jzohrab.
See how it feels.
—
Reply to this email directly, view it on GitHub
https://github.com/jzohrab/lute-v3/issues/223#issuecomment-1942913613,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMPWDMEJ6COWTW3ZQ7FUEDYTQCEFAVCNFSM6AAAAABDBJZQO2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBSHEYTGNRRGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Changed in develop, thanks.

Launched in 3.2.1.


Feature request: compatibility with VR goggles

Awesome app! Congrats!
It would be great to be able to split the screen in order to use some VR headset like the Google cardboard. That way another person can espectate with goggles too, and it shouldn't be difficult to implement.
Cheers!

We would need to first setup a proper UI, but I like the idea :)

I haven't touched my Occulus in a while, but you should just be able to run the .apk in "TV mode" on the Quest.

I haven't touched my Occulus in a while, but you should just be able to run the .apk in "TV mode" on the Quest.

Well I didn't mean a proper VR headset like the Oculus, I meant a cheap Cardboard headset where you put your phone inside.

This would be just perfect! You could use any cheap vr headset for phones!
And indeed just split te image, nothing fancy.

@jlucidar I'd like to take a stab at adding this feature if you think that's cool.  Have you heard of anyone else attempting this yet?  I bought a VR headset phone holder that I can test with.

@jlucidar I'd like to take a stab at adding this feature if you think that's cool. Have you heard of anyone else attempting this yet? I bought a VR headset phone holder that I can test with.

Go for it! No one has been working on this yet as there are higher-priority issues 🙂

Sure, go ahead @CollinTripActions :) fork dev branch and go 👍

@jlucidar I have a branch ready to push to a PR, but when I try to push it says I don't have permission.  Do you know what the problem might be?  Also, FYI I switch to this account for adding this feature as the other one was my work account.

@jlucidar I have a branch ready to push to a PR, but when I try to push it says I don't have permission. Do you know what the problem might be?

You are not a member of this repo, so you need to fork it and change your remote url then push there and open a PR from your own forked repo.

@jlucidar I have a branch ready to push to a PR, but when I try to push it says I don't have permission. Do you know what the problem might be? Also, FYI I switch to this account for adding this feature as the other one was my work account.
here's a quick video showing switching from normal view to VR view via the prefs:
vrdemo.mov

Looks just perfect! Can't wait to try it out! Did you add a zoom function? Some phone/vr goggle combination need too zoom out a bit for it to fit on the screen while using the goggles.
This app is a good example: https://play.google.com/store/apps/details?id=com.vertile.fpv3d&hl=en&gl=US
You can zoom out, makes the 2 screens smaller. So it looks good while using the goggles.

@ybjlus thanks - yes I plan to add that after some in the field testing.  at the moment it's only zoomed in the way that you see in my video.  using it that way made me really queezy actually and was difficult to get the images lined up right in the goggles.  will look into your suggestion, thanks!

@0TheRain0 your work is really appreciated! Hope you get it working. If you need any help testing, I'm happy to help out.


Same code in android and iOS producing different results?

Hi
We're trying to verify signature, and then decrypt the encrypted text from server. iOS wasn't getting successful key verification, and decryption also wasn't successful.
Upon trying to debug using the same set of keys, found that verification and decryption passed in android, but failed in iOS?
I'm posting the iOS and android code for reference.
iOS:
func verifySignature(key: String, encrypted: String, signature: String) -> Bool{
        var isVerified = false

        do {
            guard let keyData = Data(base64Encoded: key) else {return false}
            guard let encryptedData = Data(base64Encoded: encrypted) else {return false}
            guard let signatureData = Data(base64Encoded: signature) else {return false}

            let config = try TINKSignatureConfig.init()
            try! TINKConfig.register(config)
            let reader = try TINKBinaryKeysetReader.init(serializedKeyset: keyData)
            let handle = try TINKKeysetHandle.init(cleartextKeysetHandleWith: reader)
            let verifyFactory = try TINKPublicKeyVerifyFactory.primitive(with: handle)

            try verifyFactory.verifySignature(signatureData, for: encryptedData)
            isVerified = true
        } catch  {
            isVerified = false
        }
        return isVerified
    }

android :
boolean verifySignature(String data, String signature, String publicKey) throws Exception{
        SignatureConfig.register();
        boolean isVerified = false;
        try{
            byte[] signatureBytes = Base64.getDecoder().decode(signature);
            byte[] keyBytes = Base64.getDecoder().decode(publicKey);
            KeysetReader keysetReader = BinaryKeysetReader.withBytes(keyBytes);
            KeysetHandle handle = CleartextKeysetHandle.read(keysetReader);
            PublicKeyVerify verifier = handle.getPrimitive(PublicKeyVerify.class);
            verifier.verify(signatureBytes, data.getBytes());
            isVerified = true;
        }catch (Exception e){
            System.out.println(e);
        }
        return isVerified;
    }

Why would signature verification fail in iOS but pass in android for the same set of keys?
(The code seems the same to me.. please can someone help out? )

Sorry!
There was a mistake which wasn't obvious to me. I was taking in the encrypted string as base64 string.. while it was not!
So changing this line :


try to load fbx models

the jmonkey fbx loader is merged now. see here https://github.com/jMonkeyEngine/jmonkeyengine/pull/488
go to the unity asset store, load a free fbx model and try to load it.
what is the result?

We can also try to use assets from 0 A.D.
The are not the newest but a whole package for a game at once
http://svn.wildfiregames.com/public/ps/trunk/binaries/data/mods/public/art/

0.A.D. looking really cool!!!

http://zero-k.info/Forum/Thread/11899

https://jmonkeyengine.org/203418/3ds-and-collada-import/
https://hub.jmonkeyengine.org/t/3ds-collada-import-through-blender-in-sdk/25113/13

https://springrts.com/wiki/Games

loading s3o in blender is possible with https://github.com/sanguinariojoe/blender_s3o_import
save as blend and migrate to j3o. after this the texture is broken

http://www.indiedb.com/games?filter=t&kw=&released=&genre=15&theme=&indie=&players=&sort=ranktoday-asc


Returning 0 from a helper displays nothing

Not sure if this has been reported already, but I think returning the number 0 from a helper should display the 0. It currently displays nothing. I'm guessing there is a truthy check on the helper's output.
// when bar is 0
{{ bar }} // prints "0"
{{ myHelper bar }} // blank (myHelper simply returns the value())

http://jsbin.com/helasubejo/edit?html,js,output

There is a truthy check here: https://github.com/bitovi/canjs/blob/master/view/stache/mustache_core.js#L535
We need to make sure this wouldn't be a breaking change as some authors might return 0 with the intent of give me an empty string.

This is a bug.  Returning 0 to get an empty string wouldn't make sense and its not documented that way so it's ok to change prior to 3.0.

This would be the stache equivalent to this mustache issue: https://github.com/bitovi/canjs/issues/1109


Stackoverflow when using the copy constructor of an OpenApiSchema.

Using version 1.4.4 of Microsoft.OpenApi.
Describe the bug
When using the copy constructor of OpenApiSchema it comes in a recursive call and end with a stackoverflow exception.
To Reproduce
    OpenApiSchema baseSchema = new OpenApiSchema()
    {
        Type = "string",
        Format = "date"
    };

    OpenApiSchema actualSchema = new OpenApiSchema(baseSchema)
    {
        Nullable = true
    };

The "perpetrator" is the property "AdditionalProperties", this in itself is also a schema but is "copied" with AdditionalProperties = new(schema?.AdditionalProperties); so if null the constructor is again called the original parameter 'schema' and when not null it calls the constructor with AdditionalProperties as a parameter, and this is repeated untill it encounters a null and crashes.
Expected behavior
I expected to get a new OpenApiSchema with the properties from the baseSchema and the explicitly set new properties.

Thank you for reporting this. This is definitely a bug caused by an oversight on our part.

Update: this is now fixed in 1.4.5


Make 'x/x-optparse' public

I'm trying to build icicle but x/x-optparse is not public so I can't.

It is now. It should of been before, not sure what happen.

(yell if there issues with commits etc... I didn't check that it actually built, but I don't think there should be a problem now that it is public.


Fix scope of server variable

change window.server to server to fix the error shown below

Thanks for catching this! Much appreciated :)


Future support for attention bias or masking

Hi, I noticed this plan for customized attention bias was recently removed. Do you still have this in plan anytime? I feel Flash-attention is such a great project, and having this feature will make it perfect 😄
https://github.com/HazyResearch/flash-attention/commit/40a25c8ee7465cf547b929cfa2937034e37bfce9#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5L80

As mentioned in the README, we have an experimental implementation in Triton that support attention
bias (e.g. ALiBi):
https://github.com/HazyResearch/flash-attention/blob/main/flash_attn/flash_attn_triton.py

Thank you! I'll give it a try

are there plans to implement gradients for the bias? i.e for learnt attention biases. Or how difficult would this be too implement do you think?

I'm not planning to work on it as I don't use attention bias in my work. Implementing it in Triton is probably not hard, all the necessary ingredients are there in the Triton backward pass implementation. I suspect one just has to add some code to save the gradient.

Hi, I find flashattn of triton version is very experimental, only support 64 head dimension size. Does this condition have any changes？


New jsonsetget

An alternative to #13177:

set and get operations written as serializablefunctions, the same way we did for other resp operations.

this  solution works well with marshaller and externalizer
Wdyt @wburns @jabolina ?

I added a new commit with:

legacy path
fixed  json path config to always return a list of results


0.5.0 dev install issues

I've run into problems trying to install 0.5.0 out the box, with no changes.
However its apparently a problem on my local machine.
I tried installing it on a "new" windows 10 machine, which required that I first install docker.
After installing docker, which included WSL2, and update to WSL2, and reboot, I unpacked Inventree 0.5.0 and ran
cd inventree/docker
docker-compose -f docker-compose.dev.yml build
docker-compose -f docker-compose.dev.yml run inventree-dev-server invoke update
which produced the error
[+] Running 2/2

Network docker_default  Created                                                                                0.8s
Container inventree_db  Created                                                                                0.3s
[+] Running 1/1
Container inventree_db  Started                                                                                1.7s
/bin/bash: ../docker/init.sh: No such file or directory

For various reasons I suspected it was not a inventree code issue, but something to do with my installation on this specific machine.
To prove this, I unpacked the same Inventree 0.5.0 on a different Windows 10 machine that already has a version of Inventree running under docker.  Follow the same instructions as above, all worked as expected.
Can anyone provide guidance on my machine setup that would cause this issue, apparently docker related on Windows?
I tried uninstalling and reinstalling docker (latest version for Windows).

Might be an issue with folder permissions?
Can you run the following command (on the system which is failing) and provide the output:
docker volume inspect docker_src

Should provide something like :

I ran this command.  The only difference in my result is that my compose version is 2.0.0
On my system that does work, my compose version is the same as yours.
Is this the issue?  Is there a way to install an older docker that has the 1.29.2 version?

I used the docker version command on both systems and the output is identical
Client:
Cloud integration: 1.0.17
API version:       1.41
Go version:        go1.16.6
Git commit:        3967b7d
Built:             Fri Jul 30 19:58:50 2021
OS/Arch:           windows/amd64
Context:           default
Experimental:      true
Server: Docker Engine - Community
Engine:
Version:          20.10.8
API version:      1.41 (minimum version 1.12)
Go version:       go1.16.6
Git commit:       75249d8
Built:            Fri Jul 30 19:52:31 2021
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.4.9
GitCommit:        e25210fe30a0a703442421b0f60afac609f950a3
runc:
Version:          1.0.1
GitCommit:        v1.0.1-0-g4144b63
docker-init:
Version:          0.19.0
GitCommit:        de40ad0

I uninstalled docker desktop,  and reinstalled a version from 2.5 months ago.  I deleted all images and volumes and rebuilt, but still the same problem.  Inspection of the new volume still shows compose version 2.0.0 even though the docker version is considerably older, as shown by the version command.
Client:
Cloud integration: 1.0.17
Version:           20.10.7
API version:       1.41
Go version:        go1.16.4
Git commit:        f0df350
Built:             Wed Jun  2 12:00:56 2021
OS/Arch:           windows/amd64
Context:           default
Experimental:      true
Server: Docker Engine - Community
Engine:
Version:          20.10.7
API version:      1.41 (minimum version 1.12)
Go version:       go1.13.15
Git commit:       b0f5bc3
Built:            Wed Jun  2 11:54:58 2021
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.4.6
GitCommit:        d71fcd7d8303cbf684402823e425e9dd2e99285d
runc:
Version:          1.0.0-rc95
GitCommit:        b9ee9c6314599f1b4a7f497e1f1f856fe433d3b7
docker-init:
Version:          0.19.0
GitCommit:        de40ad0

Ok, with lot of work, I was able to downgrade my docker-compose to 1.29 and it seems to have fixed the problem.

@bgrant85 that's very strange. What was the process you had to follow to downgrade?

Actually it wasn't hard to do.  Just figuring out how was a bit of a challenge.
First download the new executable.  I found the link for this on the docs page for installing docker on windows server.
https://docs.docker.com/compose/install/
PS> Invoke-WebRequest "https://github.com/docker/compose/releases/download/1.29.2/docker-compose-Windows-x86_64.exe" -UseBasicParsing -OutFile $Env:ProgramFiles\Docker\docker-compose.exe
This command puts the target in the wrong place.  Probably could adjust the above command to put the target in the correct place, but I simply copied it to the correct place after running the above command.
Above command puts the target at c:\Program Files\Docker
You can find the correct place with the command:
PS> Get-command docker-compose
for me the correct place was  c:\Program Files\Docker\Docker\resources\bin
Test the result with the command:
PS>  docker-compose --version

@bgrant85 so can we close this issue? 😇

yes, thanks

Hi guys,
I'm, having exactly the same problems on Linux Mint 20. I really want to try the program, it looks great.
I have followed every step, and when running
"docker-compose -f docker-compose.dev.yml run inventree-dev-server invoke update"
The message I get is:
"  /bin/bash: can't open './docker/init.sh': No such file or directory  "
I'll appreciate your help.
Thank you guys!!
Patricio

did you check your version of docker-compose?  My issue was fixed by going back to 1.29.  Version 2.x did not work for me.
docker-compose --version
Not sure if that issue is fixed or was just my problem.


Undefined variables

Holaa;

ebpf_exporter_version
ebpf_exporter_version

Maybe worth sticking this in defaults/main.yml with [] and an explanation.
This issue will be updated with a full list of vars that are undefined as I continue to play with the role.

Fixed. :smile:


compiler-rt: apply armv6l patches to llvm >= 15 where applicable

Fixes armv6 build of compiler-rt.
the patches were written for
https://github.com/NixOS/nixpkgs/pull/205176 but not kept for llvm >= 15
cc @lopsided98 as the author of https://github.com/NixOS/nixpkgs/pull/205176
tested by cross compiling only.
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Are all the patch files copies of each other?

no there are 3 versions of it: the original for llvm <= 14, an intermediary one for 15 16 17 18 and the latest one for 19 and 20.

intermediary one for 15 16 17 18 and the latest one for 19 and 20.

Wdym by intermediary? Are the ones for 15+ the same exact file?

I mean nothing special by intermediary.
There are three distinct patches.
$  md5sum pkgs/development/compilers/llvm/**/armv6-scudo-libatomic.patch
e179fdad601bc6cb60cf7a2587354b6b  pkgs/development/compilers/llvm/15/compiler-rt/armv6-scudo-libatomic.patch
e179fdad601bc6cb60cf7a2587354b6b  pkgs/development/compilers/llvm/16/compiler-rt/armv6-scudo-libatomic.patch
e179fdad601bc6cb60cf7a2587354b6b  pkgs/development/compilers/llvm/17/compiler-rt/armv6-scudo-libatomic.patch
e179fdad601bc6cb60cf7a2587354b6b  pkgs/development/compilers/llvm/18/compiler-rt/armv6-scudo-libatomic.patch
ba8b03aef2c271c484275408c1a30b33  pkgs/development/compilers/llvm/19/compiler-rt/armv6-scudo-libatomic.patch
f2e6764468ceca1f4d5b29fa16b92553  pkgs/development/compilers/llvm/common/compiler-rt/armv6-scudo-libatomic.patch
ba8b03aef2c271c484275408c1a30b33  pkgs/development/compilers/llvm/git/compiler-rt/armv6-scudo-libatomic.patch

Yeah, I was just wondering because there's work to slim down our patches by removing duplicates and changing getVersionFile to branch based on the file path and version.

ping @RossComputerGuy

Please update getVersionFile to handle duplicates and remove duplicate patches. The PR which implemented that was merged so you'll have to rebase.

done


ApiGatewayResolver root routes and strip_prefixes

I ran into an issue when using a route with the rule "/" along with strip_prefixes feature.  This rule is unable to trigger due to the path variable being reduced down to an empty string instead of a forward slash.
from aws_lambda_powertools.event_handler.api_gateway import ApiGatewayResolver
app = ApiGatewayResolver(strip_prefixes=["/some-path"])
@app.get('/')
def my_func():
    return {'execution_status':'success'}

Expected Behavior
Accessing domain.com/some-path would execute my_func().
Current Behavior
Accessing domain.com/some-path returns a 404.  Of note though, accessing domain.com/some-path/ (note the trailing slash) will execute my_func().
Possible Solution
Add an conditional here which will map an empty string to "/".
Steps to Reproduce (for bugs)

Set up the code example above
Set up ApiGateway to use a custom domain, and bind the Lambda to a custom path of some-path
Visit https://CUSTOMDOMAIN/some-path and https://CUSTOMDOMAIN/some-path/ and note the different results

Nice one @BVMiko , i put in a review comment for adding a test case.
Otherwise, i was thinking whether it should be "" instead of "/"? Then this would work when the request is /some-path
from aws_lambda_powertools.event_handler.api_gateway import ApiGatewayResolver
app = ApiGatewayResolver(strip_prefixes=["/some-path"])
@app.get("")
def my_func():
    return {'execution_status':'success'}

As the original request path of /some-path does different from /some-path/. Although @app.get("") does like strange.

I believe that the common standard among other routing frameworks is that all paths should have a leading slash, so @api.get("/") is usually the root path. Unfortunately the Flask and FastAPI docs don't explicitly state this, though all examples are written this way.  I did find this Flask issue describing a related oversight of forgetting to enforce a leading slash though.
As for implementation, Flask and FastAPI both seem to do things differently by building and caching the full paths within each route. However ExpressJS, looks like it handles routing similar to the current ApiGatewayRouter though it does ensure a leading slash.

@michaelbrewer - I updated the testcase and the docs to reflect the new explicit change; let me know if you'd like the wording modified.

Fixed in v1.20.2.


Different values for daily usage

System Health details
System Information

version
core-2024.11.3

installation_type
Home Assistant Container

dev
false

hassio
false

docker
true

user
root

virtualenv
false

python_version
3.12.4

os_name
Linux

os_version
4.15.0-194-generic

arch
x86_64

timezone
Asia/Jerusalem

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

HACS Data
ok

GitHub API Calls Remaining
4988

Installed Version
2.0.1

Stage
running

Available Repositories
1475

Downloaded Repositories
23

AccuWeather

can_reach_server
ok

remaining_requests
33

Home Assistant Cloud

logged_in
true

subscription_expiration
August 17, 2025 at 3:00 AM

relayer_connected
true

relayer_region
eu-central-1

remote_enabled
true

remote_connected
true

alexa_enabled
false

google_enabled
true

cloud_ice_servers_enabled
true

remote_server
eu-central-1-14.ui.nabu.casa

certificate_status
ready

instance_id
a4de4ad502b54059a52696a01502c0d2

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Dashboards

dashboards
4

resources
10

views
42

mode
storage

Recorder

oldest_recorder_run
November 23, 2024 at 4:25 AM

current_recorder_run
November 26, 2024 at 11:39 AM

estimated_db_size
1839.55 MiB

database_engine
mysql

database_version
11.6.2

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues..

Describe the issue
The bezeq extension reports the same value as the app for daily consumption, but the app has another value that is probably from a different api call and seems to be more accurate, so maybe it should be used instead (see attachments - the extension reports 647.920 which rounds to 648 - the value in the graph. But the app has another value 709 for the daily consumption )

Reproduction steps

Check the value in HA
Check the value in the bezeq app

...
Debug logs
N/a

Diagnostics dump
No response

Indeed it comes from different calls:
The monthly usage calculation is take from a call to get Monthly usage (https://my-api.bezeq.co.il/v72.3/api/ElectricityTab/GetElectReportByYear)
While what you see on the tab header is taken from the header of the electricity tab.
It's a good question when it comes to the actual source of truth...
https://my-api.bezeq.co.il/v72.3/api/ElectricityTab/GetElectricityTab
->
  {
            "BillingServiceId": 70,
            "BillingServiceCode": "ElectricityMonthlyUsed",
            "BillingServiceDescription": "צריכת החשמל שלי",
            "CardType": "ElectricityMonthlyUsed",
            "ServiceType": "ElectricityMonthlyUsed",
            "CardDetails": "{\"UsedAmount\":123,\"FromDate\":\"\\/Date(1730412000000)\\/\"}",
            "Makat": null,
            "Quantity": null,
            "SN": null,
            "Mac": null,
            "Link": null,
            "EnterLink": null,
            "ShowMeshMgt": false,
            "Id": 2,
            "Title": "צריכה חודשית מצטברת",
            "SubTitle": null,
            "Picture": null,
            "Order": 2
        },

Now, although this number is indeed higher, but it's not even decimal, I'm really not sure it should be used for this calculation...

Why is decimal important?

I think that decimal point gives an indication of accuracy.

Also - this looks very unreliable, I think
Today I see that the monthly bar is about 250, while the top value says above 500....

Yes, I see that now, that value is 3 times the normal one.
On Wed, Nov 27, 2024, 08:17 Guy Khmelnitsky @.***>
wrote:

Also - this looks very unreliable, I think
Today I see that the monthly bar is about 250, while the top value says
above 500....
—
Reply to this email directly, view it on GitHub
https://github.com/GuyKh/bezeq-energy-custom-component/issues/26#issuecomment-2502999309,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZXKSSSZSJ2VZSE7G7GXVT2CVPYJAVCNFSM6AAAAABSRAE5DCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBSHE4TSMZQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yep. Closing this for now


Check signatures before applying commit

https://github.com/ontola/atomic-data/issues/14
Also relates to #13 and #26
Following implementations required:

[ ] serialize_deterministically to generate a string that can be used to
[ ] validate_signature function on Commit, using serialize_deterministically and a public key from an  Atomic Author.

We need some form of cryptographic signatures. I think the ring library is pretty good - many people use it, and the API is well documented.

In order to validate signatures, the Actor must be available at some endpoint. This Actor should contain a public key. The private key needs to be available for the one creating the commit.

Made some real progress on signing. Deterministic serialization is next!