Dist upload failing for PyPI (403 error), but was fine for PyPI Test

Describe the bug
I am ready to launch a package on PyPI, having done so successfully on PyPI Test in the past few hours, but I cannot upload the package due to a 403 error:
twine upload dist/*

...

HTTPError: 403 Forbidden from https://upload.pypi.org/legacy/
The user 'mfarragher' isn't allowed to upload to project 'obsidian-tools'. See https://pypi.org/help/#project-name for more information.

From the link I wondered if the package name was being squatted but I couldn't find the page being in use, therefore I don't have a person to contact or much to say for a PEP 541 request.  As far as I can tell the package name is available, but perhaps there's a block on it being registered? 🤷♂
I have tried different forms of the upload command with no success, different formats of .pypirc, entering login details sequentially in terminal, etc.
In search of alternative ways to upload, I did notice a recent unresolved issue on Python 3.9 where I also get the described blake2_256_digest error when I do python3 setup.py sdist upload: https://github.com/pypa/distutils/issues/25.  That's not the command I'd expect to do but could be related to my issue.
Expected behavior
Expect the wheel to upload successfully on PyPI (like it did for PyPI Test).
To Reproduce
As above and below - for system details, commands and package info.
See the upload on PyPI Test for my package with setup.py.  I installed it successfully from that server and the readme renders OK so the code seems fine.
My Platform
I have had these issues in fresh Miniconda environments (Python 3.9.6) for these operating systems - with wheels rebuilt on each system - via the command line for commands described above:

Fedora 34 (GNOME 40)
Windows 10

Additional context
N/A

You can make a PEP 541 request (https://github.com/pypa/pypi-support/issues/new?assignees=&labels=PEP+541&template=pep541-request.yml&title=PEP+541+Request%3A+PROJECT_NAME)
The current owner of the package is https://pypi.org/user/sfermigier/ (but since then, they have deleted the project, thus the 404)


Absolute paths in systemjs config are not moved to /base

If your systemjs config contains absolute paths, fixing the baseURL will not also fix these paths. They also need to be relocated under /base, where Karma serves them.
System.config({
  baseURL: '/app',
  map: {
    'jquery': '/thirdparty/jquery.js'
  }
});

While these paths could be overridden in the Karma config, this becomes difficult to maintain if your config contains a large number of frequently changing thirdparty paths.

Fixed by #54 which has been merged into master and released with 0.8.1

No it hasn't - I was looking at #52 ...

OK, now #54 has been merged into master, and released with 0.8.2


Default profile with underscore in username is not DNS compliant

If a user's email address contains an underscore, it is not escaped and this is not DNS compliant for creating the default profile during deployment.
INFO[0334] creating Profile/kubeflow-john_smith         filename="kustomize/kustomize.go:447"
Error: couldn't apply KfApp:  (kubeflow.error): Code 500 with message: kfApp Apply failed for kustomize:  (kubeflow.error): Code 500 with message: couldn't create default profile from &{{Profile kubeflow.org/v1alpha1} {kubeflow-john_smith      0 0001-01-01 00:00:00 +0000 UTC <nil> <nil> map[] map[] [] nil [] } {{User<EMAIL_ADDRESS>}} { }} Error: Profile.kubeflow.org "kubeflow-john_smith" is invalid: metadata.name: Invalid value: "kubeflow-john_smith": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

/cc @kunmingg I believe this should already be fixed.

fixed by https://github.com/kubeflow/kubeflow/pull/4309


Add LDAP tests

Moving from the regular product-tests run because these tests need extra resources.

I'll squash the three commits together before merging...

I've seen the checks passed for sprint-43, but there are no traces of ldap tests run in the logs - as if the ldap tests weren't merged to the sprint branch (https://travis-ci.org/Teradata/presto-checks/jobs/174016150). We need to check they pass - I'll trigger a build for sprint-42.

About the JDBC driver - please rebase this branch on feature/overridable_jdbc_driver and drop the JDBC-related commits. I totally forgot about that, sorry :)

After we see the tests run and pass and after this branch is rebased, we'll rebase branches cdh_latest and hdp_latest on top of this, so that we can have the ldap tests run on both distros (also, that's exactly what we're testing the sprint branch against). This will be the equivalent of merging this pull request.
We won't be merging the feature/overridable_jdbc_driver and feature/ldap_tests to presto-checks:master until the needed related changes are in presto:master.

So, the tests passed for both sprint-42 and 43. Please rebase on feature/overridable_jdbc_driver and it LGTM :) After rebasing, please rebase the {cdh,hdp}_latest{-snapshot} branches (4 in total) on this branch and push them -f. Or I can do it :)

Are we planning to test LDAP with the simba driver? If yes, then the simba_jdbc branch should be rebased on top of this branch as well

We do have Simba specific LDAP tests, but I thought we were only going to run them on Jenkins.

Everything that can be run on travis should be run on travis. Where's the build on jenkins? In whch branch are the simba-ldap tests? We should check if and how easily we can run those on presto-checks.

Jenkins build is here: https://jenkins-master.td.teradata.com/job/presto-td-product-tests-ldap/
But it appears to be misconfigured at the moment as it is not actually using the Simba JDBC driver :(.  I'll try to fix that later today.

Do I get it right that the ldap test group (contrary to ldap_cli) tests presto against an ldap server?
I think those tests should be run in this branch, using the openldap container @sanjay990 prepared. I think we had it tested?

ldap test group includes CLI and JDBC.  It should work with AD or openLDAP.

I have completed all the rebasing.

Great! Let's add the openldap tests though. Otherwise there's nothing that makes sure they work.
Do I get it right that ldap-jdbc tests require simba driver? If so, let's run cli_only in this branch and create another one - feature/ldap-simba and base the simba_jdbc branch on feature/ldap-simba

Also, when rebasing the branches, please make sure not to duplicate the commits. cdh_latest-snapshot should base on cdh_latest, and only indireclty on feature/ldap_tests. Sorry for not making this clear in advance

Yes, ldap-jdbc tests do require the simba driver.
My understanding is that the Travis LDAP test uses openldap while the Jenkins LDAP test uses AD, thus both are covered.
When I rebased the -snapshot branches I did it directly on feature/ldap_tests, and I don't see any duplicate commits in the log...but I'll re-do them using the _latest branches.


Use prelude and base fixities from GhclibParserEx

Implement the last piece of https://github.com/ndmitchell/hlint/pull/900#pullrequestreview-373493645.

@ndmitchell Ready for another look.

Looks great, I'll merge tomorrow after the CI has passed.


add-trigger-action

Adding this action: https://github.com/securesign/actions/blob/main/.github/workflows/trigger-konflux-build.yaml

/lgtm
/approve


Array field in PostgreSQL return ArrayExpression instead of array

There is no point to return ArrayExpression for array fields when loading ActiveRecord from database.
ArrayExpression do not have any usefull information per value. Data type and dimension can be obtained from the table scheme.
What steps will reproduce the problem?
Create an array postgresql field. Create ActiveRecord model. Load model from database.
What is the expected result?
Field contain array
What do you get instead?
Field contain ArrayExpression
Additional info

Q
A

Yii version
2.0.14

PHP version
5.6

Operating system
Debian 9

Thank you for opening this issue.
On the first iterations of Arrays support, the AR model's fields got arrays after data population. From some point of discussions, we've decided to create ArrayExpression object after data population to keep things more consistent, but I don't remember the exact use cases that pushed me to change the implementation. I'm not really sticked to the current one, so we can discuss and change it, if it turns out it does not make much sense.
We already have two unexpected behaviors related to this decision. See https://github.com/yiisoft/yii2/issues/15710
@yiisoft/core-developers

Yes, I already discovered two issues with this approach. And I think there might be much more.
But as i remember, one of the reasons of choosing such approach was backward incompatibility with query builder and type casting bacause DB query methods accept field values as an array when implicit type casting is required...

From some point of discussions, we've decided to create ArrayExpression object after data population to keep things more consistent, but I don't remember the exact use cases that pushed me to change the implementation.

This is the matter of lazy loading.
When fetching row with ARRAY or JSON from the database it is unknown whether those fields will be in use anywhere. Thus it is better to leave parsing for the later stage - to the moment it will be in need first time.
Having large JSON or ARRAY data may provide significant impact while being parsing over hunderds of records fetched from DB.
Also not every complex DB type can be converted to an array. For example, while using PostGis extension, its specific type 'geometry' can not be represented as mere array - it should be a specific complex object.
Still, this matter is debatable.

Great feature, but I think ArrayExpression is not a PATCH update, because it broke a lot of model getters, used before for Postgres array fields.

It was not a PATCH update indeed. We have Semver shifted on one group to the right: 2..., so 2.0.14 is a minor update. However, ArrayExpression still breaks compatibility and it was not clearly stated in UPGRADE.md.
@zhdanovartur what is the best solution on your opinion?
We are thinking about adding a flag for Schema class, that allows to return array instead of ArrayExpression, or to return string as it was used before.

When fetching row with ARRAY or JSON from the database it is unknown whether those fields will be in use anywhere. Thus it is better to leave parsing for the later stage - to the moment it will be in need first time.

It would make sense. But for now it parsed on phpTypecast. So it always parsing, no matter need or not.
If we talk about consistency, then the json fields are now returned as an array, not JsonExpresson, but array fileds returned as ArrayExpression. It looks strange to me.

We are thinking about adding a flag for Schema class, that allows to return array instead of ArrayExpression, or to return string as it was used before.

Flag is looking good to me. Unexpected ArrayExpression broke some code in my projects.
And if it were just arrays, then I would not have noticed anything

It would make sense. But for now it parsed on phpTypecast. So it always parsing, no matter need or not.

Laziness can be easily added afterwards and nobody will notice the change ;)

If we talk about consistency, then the json fields are now returned as an array, not JsonExpresson, but array fileds returned as ArrayExpression. It looks strange to me.

But yes, that does not look very consistent, so it is a point of discussion now.

Unexpected ArrayExpression broke some code in my projects.

Could you check, whether it is any better with fix in https://github.com/yiisoft/yii2/pull/15733?

We are thinking about adding a flag for Schema class
Good idea.

Could you check, whether it is any better with fix in #15733?

It not work for in_array
in_array() expects parameter 2 to be array, object given
It's not hard for me to fix my code, but for backwards compatibility (if needed), I think the flag is the only option

Added flag in https://github.com/yiisoft/yii2/pull/15755
Can be configured somehow like this:
'db' => [
    'class' => \yii\db\Connection::class,
    'schemaMap' => [
        'pgsql' => [
            'class' => \yii\db\pgsql\Schema::class,
            'columnSchemaClass' => [
                'class' => \yii\db\pgsql\ColumnSchema::class,
                'disableJsonSupport' => true,
                'deserializeArrayColumnToArrayExpression' => false,
            ],
        ],
    ],
],

Or easier:
[
    'container' => [
        'definitions' => [
            \yii\db\pgsql\ColumnSchema::class => [
                'class' => \yii\db\pgsql\ColumnSchema::class,
                'disableJsonSupport' => true,
            ],
        ],
    ],
]

@tQuant Could you check with code from new PR and an appropriate config?

Fixed by https://github.com/yiisoft/yii2/commit/25d176aa7092ae940b825c6eddfb8784f97b2e20

Does anybody tested this fix in FULL reality?
For me, sample not work.
Config is setup accordingly, both methods, as I see it, even Class \yii\db\Connection receives it, but   ColumnSchema then receives just empty configuration.
I'm testing with Yii2 version 2.0.23 and postgres.

We've spent days to fix our system and about day to figure out why this approach described here don't work.
We neded to move on now so my colegue now tried this again... after long debug of how all this workwe found out basic reason why it wont work...
Problem was CACHE! This setting is THE ONLY ONE, that I know now that is cached in yii cache! Working with it for years aI never had problem with some config directive to not work until this one.
So, if this does'nt work, flush the cache to cvhange values and reread config.


Add documentation for CLI Backup Inspection tools

https://cockroachlabs.atlassian.net/browse/CRDB-7080

This tool displays backup data available outside of the sql shell. A rough draft for high-level topics to be included can be found here: https://docs.google.com/document/d/1tR-IFA-MtPMgUppl8vUQRH81GXaQoKXCEX5ocX0yLAY/edit?usp=sharing


cleanup: Remove error-format special-case in cargo fix

cargo fix had some special handling to deal with ensuring that the --error-format=json flag was passed to rustc. However, cargo was changed in https://github.com/rust-lang/cargo/pull/7450 to always pass that flag. This special handling is no longer necessary, so this PR removes it.

@bors r+
If there is a conflict with the other PR, feel free to r= me when it is resolved.

:pushpin: Commit 88af506bcd9749d624645bea9df4934045c1e1e9 has been approved by epage
It is now in the queue for this repository.

:lock: Merge conflict
This pull request and the master branch diverged in a way that cannot be automatically merged. Please rebase on top of the latest master branch, and let the reviewer approve again.
How do I rebase?
Assuming self is your fork and upstream is this repository, you can resolve the conflict following these steps:

git checkout fix-error-format (switch to your branch)
git fetch upstream master (retrieve the latest master)
git rebase upstream/master -p (rebase on top of it)
Follow the on-screen instruction to resolve conflicts (check git status if you got lost).
git push self fix-error-format --force-with-lease (update this PR)

You may also read Git Rebasing to Resolve Conflicts by Drew Blessing for a short tutorial.
Please avoid the "Resolve conflicts" button on GitHub. It uses git merge instead of git rebase which makes the PR commit history more difficult to read.
Sometimes step 4 will complete without asking for resolution. This is usually due to difference between how Cargo.lock conflict is handled during merge and rebase. This is normal, and you should still perform step 5 to update this PR.

Error message
Auto-merging src/cargo/ops/fix.rs
CONFLICT (content): Merge conflict in src/cargo/ops/fix.rs
Automatic merge failed; fix conflicts and then commit the result.

:umbrella: The latest upstream changes (presumably #13225) made this pull request unmergeable. Please resolve the merge conflicts.

@bors r=epage

:pushpin: Commit 1a8034cc80cec27eeefced834e4943a01b6b9b31 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 1a8034cc80cec27eeefced834e4943a01b6b9b31 with merge ef94adb92704acf3967be8cf73ac0b01d818c47a...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing ef94adb92704acf3967be8cf73ac0b01d818c47a to master...


added notifications settings

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month. End of August
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month. August
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit (including refactoring) and then push new changes, including version updates, in separate commits.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[x] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Breaking change item:
https://msazure.visualstudio.com/One/_workitems/edit/10524993
We had some delay in delivering this version (2021-06-01), the manifest has just been updated last week, and we would like to add additional property to this version. We do not expect any customer using this API version yet. We plan to open the mentioned API version to all customers on October 1st, and until then, we will be on preview mode for few exclusive customers.

There is breaking changes in the existing GA API version please follow the process for breaking changes first. Removing ARM review required for now.

Hi,
@chiragg4u I've already opened a breaking change work item for this:
https://msazure.visualstudio.com/One/_workitems/edit/10524993
We had some delay in delivering this version (2021-06-01), the manifest has just been updated last week, and we would like to add additional property to this version. We do not expect any customer using this API version yet. We plan to open the mentioned API version to all customers on October 1st, and until then, we will be on preview mode for few exclusive customers.
what are the next steps here?


ima.py: increase delay in ima hash check test

During testing, the hash seems to take longer to compute and
add in ascii_runtime_measurements. This increase should take care
of the false negative.

It also adds 30 seconds to the overall test time.
Please try to find a solution with quick polling on the target device and a timeout after 30 seconds.

Updated to try a maximum of 30 times, with 1s delay.

This is likely to run more than 30 seconds because it ignores the time to log in and run the test. It also has additional overhead. I think the best solution would be to run this loop on the device:
ssh ... 'start=$(date '+%s'); while [ $(expr $(date '+%s') - $start) -lt 10 ]; do if grep foo /dev/null; then exit 0; fi; sleep 1; done; exit 1'

From my point of view, the overhead should be low enough to ignore.
Before this, we used a sleep(10), and sometimes it failed. This is why we needed to increase it in the first time.
Using your suggestion (assuming -lt 20 or 30) will only reduce a very small amount of time (not tested, but I guess that less than 2-3seconds) from the proposed solution.
Either way, I do not think a 30 second test is a long test - we have many more which take longer.

I still think the code could be improved to time out after exactly 30  seconds, but as you said, that shouldn't matter in practice, so let's merge it.

Tests failed with some likely unrelated problems, merging anyway.


People in household dropdown: Does it include the caller?

Right now the people in household dropdown is not terribly clear. We should adjust the label language to something more useful.

Paging @lwaldsc @NerdyGirl537 for what would be most helpful here -- thoughts? Is 'People in household (including caller)' okay?

Similar enough to #224, closing


Browser's built-in zoom isn't honored (ignored) after re-rendering

Describe the bug
I want to be able to zoom out a bit in browser to see the whole/more of the page. But as soon as I try to type something after, it will resent/ignore the browser's zoom and reset it as if I never zoomed out.
To Reproduce
Steps to reproduce the behavior (Library test):

Open preview in browser.
Write "Some text" (anything).
Zoom out the preview page in the browser.
Write a whitespace (anything).
Zoom resets and now the page again takes full width of the viewport.

Expected behavior
I can see updated preview with zoomed out view without it resetting after each update.
Package/Software version:
typst-preview version: v0.8.1 (https://github.com/Enter-tainer/typst-preview/commit/bf83af63e62515881b192387b6f0e552f9346c02)
Additional context

Could you elaborate a bit? Since we have tested typst-preview on Windows, Ubuntu, Arch, and macOS, and they looks fine to me. So there should be subtle difference between your OS and that operation systems. I remember you are using PopOS, but I don't have one, sadly :(.

Did you test changing browser's tab zoom in between re-renders? Testing just the preview update itself on multiple OSes isn't enough. I can elaborate, but is there something specific that you don't understand? I think I've described/shown everything clear enough.

I've also now tested in clean Firefox profile and ungoogled-chromium (flatpak). The issue persists.

Okay, I believe I understand it, you are using the builtin zoom of browser instead of ctrl+wheel provided by typst-preview. It is known to broken since initial release. It should be the time to fix it.

It should be the time to fix it.

Yay.

ctrl+wheel provided by typst-preview.

First of all, I tried it with a touchpad — it just scrolls the page up and down. Second of all, since the "main" focus is on the VS Code, it means that the majority of the users are GUI fans. But you have to remember about TUI enjoyers who only every use cursor manipulator when every shortcut fails (including Ctrl+-). So yeah, I use Ctrl+- to zoom out.

touchpad

I did not use touchpad before, but I'm quite curious and tested it just now. It is working on my computer with two fingers pinching in or stretching out. Did you try it?

So yeah, I use Ctrl+- to zoom out.

Could you share more shortcuts? They must be fit my preference. like the ctrl+- binding is quite useful to many users.

Could you share more shortcuts?

Are you sure about this? My Neovim/terminal/FOSS OS knowledge of shortcuts is immeasurable. What exactly do you want to know? You can just read the whole https://support.mozilla.org/kb/Keyboard shortcuts.

My Neovim/terminal/FOSS OS knowledge of shortcuts is immeasurable.

Whoops! But we should at least confirm that all of shortcuts from browsers are working, which is just a simpler step.

Well, there is only Ctrl+0, Ctrl+- and Ctrl+= for zoom. It also works in Okular.

I did not use touchpad before, but I'm quite curious and tested it just now. It is working on my computer with two fingers pinching in or stretching out. Did you try it?

Now, yes. But touchpad a lot of the time isn't supported by default in Linux distros. Pop!_OS has some built-in GNOME extension for touchpad gestures, but I turned it off since it interfered with touchegg touchpad gestures server. In touchegg with empty config nothing will work on pinch, but I added this for firefox:
  <!-- Zoom in/out in Firefox -->
  <application name="firefox-default">
    <gesture type="PINCH" fingers="2" direction="IN">
      <action type="SEND_KEYS">
        <repeat>true</repeat>
        <modifiers>Control_L</modifiers>
        <keys>KP_Subtract</keys>
        <decreaseKeys>KP_Add</decreaseKeys>
      </action>
    </gesture>
    <gesture type="PINCH" fingers="2" direction="OUT">
      <action type="SEND_KEYS">
        <repeat>true</repeat>
        <modifiers>Control_L</modifiers>
        <keys>KP_Add</keys>
        <decreaseKeys>KP_Subtract</decreaseKeys>
      </action>
    </gesture>
  </application>

I know that shindows supports touchpad gestures out of the box.
TL;DR is that there is no positive news, nothing is changed.

Update: typst-preview is a bit joking.
The standalone mode of typst-preview:

respect the built-in zoom but ignoring zoom by ctrl+wheel.
will reset zoom on ctrl+save

Run Open in browser command in VSCode:

don't like the built-in zoom but only zoom by ctrl+wheel.
don't reset zoom on ctrl+save

Anyway, both of the issue is highly related to the topic of this issue.

I thought of design a bit, and come to a major problem that: browser cannot remember your scale setting well, since typst-preview uses a random port to serve preview's webpage.
I then have several points to develop the functionality to remember scale setting:

Don't remember temporary scale settings, that you have changed during previewing documents.
Since that one may open preview multiple times and close them in different order, which could cause a weird scale value. For example, close a document with scale 95% and then close another document with scale 75%, finally typst-preview may remember the setting: 75%.
Please tell me if one have some better idea.

Add a --init-scale option to cli, and add a corresponding vscode setting typst-preview.initScale, whose default value is 100% or 1.
The key is that we should not store the scale setting anywhere, neither on some random configuration file in your personal folders nor in your browser.

Add a metadata label typst-preview-settings-init-scale to your document, which is then get by typst query only once during initialization of preview pages.
The key is that you could store settings and override the configuration at a document level.
Example:
// template.typ
#let my-project-template = [
  #let init-scale = if { get page.width } > a4-paper-width { 0.9 } else { 0.95 }
  metadata({ init-scale }) <typst-preview-settings>
]

// main.typ
#show: my-project-template

Don't use a random port to serve preview's webpage.
But this would affect the software structure of typst-preview.

Note: preview with the 100% scale will adjust the document's width to exact the width of window. preview with the x% scale will adjust the document's width to the width of window but left (1-x)% unoccupied.


added node js sample for tab-external auth

Executed build pipeline:
https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/6734050934

Executed Build pipeline:
https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/6767459581


Add TypeScript/ES6 module instructions

Checklist

[ ] run npm run test and npm run benchmark
[ ] tests and/or benchmarks are included
[x] documentation is changed or added
[x] commit message and code follows Code of conduct

Supersedes by #25


Manta fails with old illumina FASTQ qualities

I had the misfortune to run manta on a couple of outdated FASTQ files with the old illumina (+64) format FASTQ qualities. manta complained about  quality scores >70 (the old illumina qualities ran up to 72) and terminated with an exception (which I found in the manta workflow output, as part of the manta error logs in the structural subfolder of the working directory). manta, although developed by Illumina, does not provide a command line option for old quality formats:

Alignments with basecall quality values greater than 70 are rejected (these are not supported on the assumption that this indicates an offset error) --manta readme

While I know that's possible to specify to bcbio that  inputs have old qualities, I forgot about these particularities of my input data in that run and did not do so. However, when looking into the bcbio sources it seemed as if there were no special provisions for quality score conversion prior to calling manta in either case. So I guess that problem would have occurred even if I had informed bcbio of the old quality format (?)
I got around the issue by converting the input FASTQs to Sanger qualities external to bcbio. But perhaps someone could look into the code and see if I am right - and in that case add some of the conversion code that can be found elsewhere in bcbio to the manta wrapper so that other users won't have to face the same problem? (BTW, although I forgot to specify the quality format in the bcbio configuration,  bwa and vardict just processed the old illumina quality inputs without complaining, probably silently screwing up the results).

Sven-Eric;
bcbio handles conversion of the quality scores during alignment if you specify format: illumina. This either happens during preparation of a bgzipped fastq input:
https://github.com/chapmanb/bcbio-nextgen/blob/master/bcbio/ngsalign/alignprep.py#L66
or in bwa directly:
https://github.com/chapmanb/bcbio-nextgen/blob/master/bcbio/ngsalign/bwa.py#L126
The downstream tools then assume the alignments have the standard format so there won't be anything specific in those.
bcbio should error out in these cases to give you a warning that something is wrong:
https://github.com/chapmanb/bcbio-nextgen/blob/master/bcbio/pipeline/run_info.py#L385
so I'm surprised you didn't get an error message when parsing the input YAML file. Is this using remote fastqs in S3 or something else that would prevent this check being run?
Sorry about the issues and hope this helps.

Yep, from S3. That explains it and thank you for clearing up the matter. In that case everything is fine, closing.


Bump Go version requirement to 1.7

I'm not sure if much else requires 1.7, but context didn't get moved from golang.org/x/net/context to the stdlib as 'context' until 1.7, and it is used as such in the current source.
eg: https://github.com/keybase/kbfs/blob/master/libkbfs/ops.go#L8
ref: https://tip.golang.org/doc/go1.7#context

Thanks, yeah, looks like a few uses of "context" have snuck in. But we have no reason to remain compatible with 1.6 anymore, and we're planning to move all context uses to "context", anyway. Thanks!


Conductor UI Task Logs tab does not show log messages

Describe the bug
Conductor UI Task Logs tab does not show log messages
Details
Conductor UI Task Logs tab does not show log messages.
The problem is - the UI try to get the logs from the following URL - pay attention that "api" appear twice in the URL.
http://my-host:port/api/api/tasks/bd81fae3-d514-42e5-8fbb-3e5bc5433d8b/log

To Reproduce
Steps to reproduce the behavior:
Go to executions and click on some workflow execution.
Click on some task at the diagram. Then click the task's logs tab on the right side.
The logs tab is empty.
Expected behavior
Show log messages added to task by worker

Hi @eransagi1 , thanks for your report, this issue has been fixed, please refer to #2692  and #2630


Move tabs from one place to other

Able to move the tab from one place to another so that user can switch tabs easily instead scrolling through tabs if there are many tabs open

i want to contribute. allow me to contribute in this issue.

Hi @sun-flame,
Sure, thank you for your interest in this issue! I appreciate your enthusiasm and willingness to contribute. I will assign the issue to you – looking forward to your insights and solutions. If you have any questions or need assistance, feel free to reach out. Happy coding! 🚀
Kind Regards,
Shashi Kumar Kasturi


$INCLUDE is not relative to file's location

It seems like the working/excution directory from coredns is used and not something set in the root mw.
Corefile:
  .:5053 {
    root /zones/
    auto {
      directory . (.*)
      transfer *
    }
    errors stdout
    log stdout
  }

Main zone (example.com):
@                   3600 IN SOA     ns.example.com. domains.example.com. (
<PHONE_NUMBER>   ; serial
                                      15m          ; refresh
                                      5m           ; retry
                                      1w           ; expire
                                      12h    )     ; minimum

@                  86400 IN NS     ns.example.com

test IN A <IP_ADDRESS>

; works
$INCLUDE /zones/test.example.com test

; doesn't work
$INCLUDE test.example.com test

from: https://github.com/coredns/coredns/issues/1068

btw: not trivial to fix because Go lacks a good where is this file located API; you need to work from the process' CWD up to the files opened, keeping track of relative and absolute file names used.

Yeah don't have a simple way besides keeping track and using osext.ExecutableFolder(), but that doesn't follow symlinks afaik so not even that predictable. I still think that $INCLUDE in it's current form is a UI bug.
Maybe leave it open as reference for people to find and rework once coredns k8s stuff settles.

Can't we just update parseZone to accept and utilize the file parameter that is already provided by ParseZone? Do you have an example of where something like this would fail?
-			r1, e1 := os.Open(l.token)
+			includePath := l.token
+			if !filepath.IsAbs(includePath) {
+				includePath = filepath.Join(filepath.Dir(f), target)
+			}
+			r1, e1 := os.Open(includePath)
 			if e1 != nil {
-				t <- &Token{Error: &ParseError{f, "failed to open `" + l.token + "'", l}}
+				msg := fmt.Sprintf("failed to open `%s'", l.token)
+				if !filepath.IsAbs(l.token) {
+					msg += fmt.Sprintf(" as `%s'", includePath)
+				}
+				t <- &Token{Error: &ParseError{f, msg, l}}
 				return
 			}
 			if include+1 > 7 {
 				t <- &Token{Error: &ParseError{f, "too deeply nested $INCLUDE", l}}
 				return
 			}
-			parseZone(r1, neworigin, defttl, l.token, t, include+1)
+			parseZone(r1, neworigin, defttl, includePath, t, include+1)

@gibson042 just now seeing your update. I think I tries something like this, but failed.
Let me try your patch.

TL;DR: patch works, I was clearly overthinking it:
example.org -> $INCLUDE include1.org -> $INCLUDE include2.org
% g/src/github.com/coredns/coredns/coredns -dns.port=1053 -conf Corefile.example

% dig @localhost -p 1053 axfr example.org | grep include[1-9]
include1.example.org.	3600	IN	A	<IP_ADDRESS>
include2.example.org.	3600	IN	A	<IP_ADDRESS>

example.org -> $INCLUDE include1.org -> $INCLUDE include2.org
% cd tmp
% ../g/src/github.com/coredns/coredns/coredns -dns.port=1053 -conf ../Corefile.example

% dig @localhost -p 1053 axfr example.org | grep include[1-9]
include1.example.org.	3600	IN	A	<IP_ADDRESS>
include2.example.org.	3600	IN	A	<IP_ADDRESS>

/home/miek/example.org -> $INCLUDE include1.org -> $INCLUDE include2.org

works

example.org -> $INCLUDE /tmp/include1.org -> $INCLUDE include2.org

works.

/tmp/example.org -> $INCLUDE a/include1.org -> $INCLUDE include2.org

works.

@gibson042 want to submit that as a PR?

I'll like to get this fixed in coredns - meaning fixing it here - I'll submit a PR later today.


Assets name collision in dev mode

Version
next
Reproduction link
https://github.com/adrienbaron/nuxt-asset-name-collision-bug
Steps to reproduce

Run the provided Nuxt app in dev mode

What is expected ?

Images should be different (one big, one small)

What is actually happening?

Images are the same

Nuxt seems to copy all the images to /_nuxt/img, but doesn't add a hash of the file at the end of the name. So images with the same name but in different folders collides.

This bug report is available on Nuxt community (#c7617)

Currently Nuxt will flat all built resources by default.
With #3787, you can config build.filenames.img as what you want, for example:
      build: {
        filenames: {
          img: (file) => {
            const dir = path.dirname(file).replace('assets', 'img')
            return `${dir}/[name].[hash:7].[ext]`
          }
        }
      }

@clarkdo Doesn't seem to resolve the issue, the issue only happens in dev mode, in regular build mode the hashing is applied and there is no problem.

It because in dev mode, we removed all hash because it has some problems in https://github.com/webpack/webpack/issues/1914#issuecomment-174171709
If you use the function filename, if will generate images into different directories.

@clarkdo Ok makes sense. Using the path in dev mode seems like a good compromise? Would still store if the user moves files around probably though... Not sure how much this is an issue.
Worse case we could just disallow files with same name in assets and add a warning in the doc?

Fixed by #3789


Feature#492: Enhance LabTest Cards to include more details. Add redirection for 'Read More'.

Feature#492: Enhance LabTest Cards to include more details. Redirect to the Hospitals page when clicked 'Read More'.

The lab test card now contains more details on a lab test like price and ratings.
Redirection to 'Hospitals-around'  page when clicked 'Read More'.

This PR is related to issue #492 .


delete this repository

According to #3 this repo is dead and confusing the world.
Please delete the repository
Thank you

+1

+1 delete this confusing project.


cache=True not working when calling xarray.open_dataset

I think there is a bug in the caching or I am misunderstanding how it should work.  I am creating a dataset as shown below.  Note the arg cache=True.  However every time I do this ds.B8.values it takes a handful of seconds to complete.  To get around this I am creating a new dataset directly from the values with this method.
Additionally, Note below I tried 2 versions of creating the uncached-dataset.  The first I didn't filter by the area first and it took much much longer.  I think this is probably expected but I was a bit surprised since xr.open_dataset is querying over a specific geom.
import ee
ee.Initialize()
import xarray as xr

IC=ee.ImageCollection("COPERNICUS/S2_HARMONIZED")
GEOM=ee.Geometry.Rectangle(-92.38201846776445,34.10974829658343,-92.38097240624865,34.11021909957634)
SCALE=10
EE_CRS='EPSG:3857'

IC=IC.filterDate('2021-01-01','2022-01-01').filterBounds(GEOM).map(lambda im: ee.Image(im).normalizedDifference(['B8','B4']).rename(['ndvi']))

def get_ee_xrr(ic,geom):
    xrr=xr.open_dataset(
        ic,
        engine='ee',
        crs=EE_CRS,
        scale=SCALE,
        geometry=geom,
        cache=True)
    return xrr.chunk().sortby('time')

def cache_ds(ds,bands=['ndvi']):
    attrs=ds.attrs
    coords=ds.coords
    data={}
    for b in bands:
        data[b]=xr.DataArray(
            attrs=attrs,
            coords=coords,
            data=ds[b].values)
    return xr.Dataset(data)

%time ds=get_ee_xrr(IC,GEOM)
# Wall time: 1.38 s

%time ds_cached=cache_ds(ds)
# Wall time: 863 ms

%timeit ds.ndvi.values
# 558 ms ± 57.2 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
%timeit ds_cached.ndvi.values
# 5.79 µs ± 19.8 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

This is huge increase.  Note if I only needed the values once it would be faster not to cache and in this toy example waiting a fraction of a second doesn't make much of a difference but once you start using larger geometries and doing cloud masking accessing the data multiple times becomes a big problem.
Is this expected behavior? Am I misunderstanding the cache param in xr.open_dataset?  Is there another way to keep the downloaded data without explicitly recreating the dataset?
Thanks,
Brookie

Thanks Kel - .load() works!


Issue a warning instead of an error for long Names

Fixes #6151
The PDF specification (cited below) specifies a maximum length of a name in bytes as a minimal architectural limit. This means that PDF writers should not create names that exceed 127 bytes.
It does not forbid PDF readers to accept such names though. These names are only used internally to link PDF objects to other objects. For these use cases, the lengths of the names do not really matter. Hence I have changed the implementation to not treat long names as errors, but warnings.

(7.3.5) The length of a name shall be subject to an implementation limit; see Annex C.
(Annex C.2) Table C.1 describes the minimum architectural limits that should be accommodated by conforming readers running on 32-bit machines. Because conforming readers may be subject to these limits, conforming writers producing PDF files should remain within them.
(Table C.1) name 127 "Maximum length of a name, in bytes."
-- http://adobe.com/content/dam/Adobe/en/devnet/acrobat/pdfs/PDF32000_2008.pdf

/botio test

/botio test

/botio-linux preview

Nice, thanks!

Thanks so much @timvandermeij, @Rob--W @Snuffleupagus and anyone else who took a second on the issue! Love the enthusiasm with which this was addressed. Great example of maintaining a great tool.


Fix Spurious backslash in Update Object Properties test

Fixes #4165
Changed \\' to '.
Tested locally.

Please remove the issue number from the title and I'll merge it right away. The PR guidelines: https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/Pull-Request-Contribute
Thanks!

Awesome!


Fix scroll when moving to Menus or Code of Conduct from another scrolled view

This is solved already in the Slider component, but Menus and Code of Conduct do not use this component and needs to be fixed in some other way.
To reproduce go to Schedule, scroll to bottom and then go to Code of Conduct.

Done in b2f3bfeb960cd3a7baea3781dedefb1d51232ef6


Fixed how to specify minSdkVersion

modified how to specify minSdkVersion in /manifest/uses-sdk because of an error.
error message
Installing "nodejs-mobile-cordova" for android
Failed to install 'nodejs-mobile-cordova': Error: Unable to graft xml at selector "/manifest/uses-sdk" from "/Users/okhiroyuki/Documents/github/RedMobile/platforms/android/app/src/main/AndroidManifest.xml" during config install
    at ConfigFile.graft_child (/Users/okhiroyuki/Documents/github/RedMobile/node_modules/cordova-common/src/ConfigChanges/ConfigFile.js:117:23)
    at PlatformMunger.apply_file_munge (/Users/okhiroyuki/Documents/github/RedMobile/node_modules/cordova-common/src/ConfigChanges/ConfigChanges.js:76:43)
    at PlatformMunger._munge_helper (/Users/okhiroyuki/Documents/github/RedMobile/node_modules/cordova-common/src/ConfigChanges/ConfigChanges.js:219:18)
    at PlatformMunger.add_plugin_changes (/Users/okhiroyuki/Documents/github/RedMobile/node_modules/cordova-common/src/ConfigChanges/ConfigChanges.js:148:14)
    at /Users/okhiroyuki/Documents/github/RedMobile/node_modules/cordova-common/src/PluginManager.js:121:33
    at _fulfilled (/Users/okhiroyuki/Documents/github/RedMobile/node_modules/q/q.js:854:54)
    at /Users/okhiroyuki/Documents/github/RedMobile/node_modules/q/q.js:883:30
    at Promise.promise.promiseDispatch (/Users/okhiroyuki/Documents/github/RedMobile/node_modules/q/q.js:816:13)
    at /Users/okhiroyuki/Documents/github/RedMobile/node_modules/q/q.js:877:14
    at runSingle (/Users/okhiroyuki/Documents/github/RedMobile/node_modules/q/q.js:137:13)
Unable to graft xml at selector "/manifest/uses-sdk" from "/Users/okhiroyuki/Documents/github/RedMobile/platforms/android/app/src/main/AndroidManifest.xml" during config install

Hi, @semidark
I was told that the original was targeted at 21, so I just made sure that there was no error.
https://github.com/JaneaSystems/nodejs-mobile-cordova#requirements
I'm running minSdkVersion at 22, so I didn't notice.
I'll think about it.

cordova 10 confirmed that it clearly states 22.
I will fix it.
Android Version Support Update

The default target SDK version is set to 29.
The minimum SDK version is set to 22.
The minimum supported Android version is 5.1.
NOTE: because Cordova has increased the minimum SDK version to 22, we no longer support or test with Android 5.0 or lower.
https://cordova.apache.org/announcements/2020/06/29/cordova-android-9.0.0.html

@semidark
Thank you for your feedback!
I fixed it.
https://github.com/okhiroyuki/nodejs-mobile-cordova/pull/8


pynq-z2 always restarts automatically

Hello Andre，After compiling the project, when I run 'yolo_img dog.jpg', my pynq-z2 always restarts automatically, I tried different image files, none of them worked. Is this the cause of insufficient resources?

Hello!
First of all, from the screenshot you attached I can see already one problem. You need to run the command yolo_img dog.jpg on the "yolo_pynqz2" folder and not "yolo_pynqz2_data". Forget about the "yolo_pynqz2_data" as this one is for obtaining YOLO metrics. It is not useful after all.
What you should do is, after log in, acess the folder with this command cd yolo_pynqz2 and then yolo_img dog.jpg to test the detections on the example image.
If the PYNQ-Z2 completely crashes as you say, that is probably related to power. I suggested somewhere on the GitBook tutorial that as this application is very demanding in terms of power, you should connect the power using the adapter and not the USB. Take a look at the following image to see what I mean:

If you want to know more specifications about the Power adapter, I wrote some informations on this part of the tutorial. Scroll down a little bit to see it.
On a final note, I should warn you that the DPU might crash in normal circumstances. Not crash the PYNQ-Z2 but the program. It will display a error about "DPU Timeout". I could not find the reason for this but it might not affect your applications.
Thank you very much and I hope this helps! :)

Thanks for your patient answer, the cause of the restart was indeed the power supply, after changing to a DC adapter, the restart issue was resolved and the procedure performed correctly, thank you again and have a good day!  Enjoy the project! :)


Fixed branch selector overlay falling off the screen

Description
Fixes #712 by wrapping the VStack with an ScollView.
Related Issue

#712

Checklist

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[ ] I documented my code
[ ] Review requested

Screenshots

@allcontributors add @matthijseikelenboom for code

@allcontributors add @matthijseikelenboom for bug


Added the ability to chain commands.

Added the ability to chain commands. for a command to be chainable the embed cannot be modified nor deleted (don't know what happens if I try to run that through my fake ctx so lets just not try). A chainable command can have multiple ctx.send() both regular text and embeds, just as long as its done at once. If an embed is the output of a chainable command the fields.values will be extracted and passed on to the next command. It takes the prefix from the context
Examples:

This is literally so scary im merging it as is, im sure ill regret this later


Date picker display issues when used inside table (chrome, firefox)

Describe the bug
When using the date picker inside of a table, some colums inside the date picker are not shown.
How to reproduce

            <input type="date" clrDate>

https://stackblitz.com/edit/clarity-light-theme-v013-6thcd1?file=src/app/app.component.html
Expected behavior
All columns of the month are shown.
Versions
App

Clarity: 0.13 (0.11 also does not work)

Device:

OS: Ubuntu 18.04
Browser Firefox 62, Chrome

The columns are missing in the day view

Yeah, some table cell styles are bleeding into the date picker. This is a bug that we need to address, but the workaround is to avoid using it in a table until then.

The issue is with padding added to table cells.
.table td, .table th {     line-height: .583333rem;      padding: .458333rem .5rem;     text-align: center; }
The workaround is to remove this padding in case of calendar-cell.
Something like below will solve it and will not interfere with anything else.
table.calendar-table td.calendar-cell {     padding: 0; }


[Falcon7b] Tilize attention mask on device, repeat mask across heads on device for prefill

Move attention mask tilize operation from cpu to device for prefill and decode to reduce layout conversion time
For prefill (non-optim mode), remove attention mask broadcast (across heads) from cpu and instead repeat on device to reduce device transfer size
Update Falcon7b perf test targets
Overall e2e perf improvements (single and multi-chip): ~1.7x for prefill, ~1.3x for decode

Post commit model tests: https://github.com/tenstorrent/tt-metal/actions/runs/8927467317
Single-chip model perf tests: https://github.com/tenstorrent/tt-metal/actions/runs/8927449714
Multi-chip model perf tests: https://github.com/tenstorrent/tt-metal/actions/runs/8927444181


Lots of updates

Yay for GitHub API Integration!
Among other things:

Stubbing for octokit and http requests in general with use of webmock

Created classes now verify that the Machine User was added as an owner to the organization.

Machine User invites users joining a class to the class organization.

Header checkbox bug was fixed.

@wpeery @scottpchow23 @shudaniel

Looking like great progress!

Conrad said to just merge so we can test.


feat: change to Java 8

for supporting usage by Java 8.
@markphelps

Thank you @liran2000 !! Very minor changes to get it compatible with Java 8. Unfortunately since our SDK is auto-generated via Fern, these changes would get overwritten on the next version build. Bringing in @dsinghvi from Fern to see if we could make this part of the generation.
@dsinghvi would it be possible to incorporate these changes into the generation process somehow? The removal of Map.of for adding the the headers map one by should be a backward compatible change I think and allow generated SDKs to support older versions of Java (pre 9).
Not sure how the toolchain functionality works, but maybe it could be made optional in the generator properties for Java?
Thank you again @liran2000 for the expertise and work moving this forward! Very much appreciated!!

handling this per the generator now. thanks @liran2000


This PR Should Get Marked As Stale, Manually Commented On, and Automatically Unlabeled

Testing only - do not merge.

Hey FabricBot, I'm still working on this!


ImportError: libcudnn.so.7: cannot open shared object file: No such file or directory

Hello,
My system already has CUDA(v8.0.61) and cuDNN installed.
So I installed nnabla and CUDA extension package using pip as below:
pip install nnabla
pip install nnabla-ext-cuda80
I have also added
export PATH="/usr/local/cuda-8.0/bin:$PATH"
export LD_LIBRARY_PATH="/usr/local/cuda-8.0/lib64:$LD_LIBRARY_PATH"
in .bashrc file.
Then, check nnabla by running:
python -c "import nnabla"
2021-07-08 11:*:*,854 [nnabla][INFO]: Initializing CPU extension...
But when I check nnabla_ext, I got an error message.
python -c "import nnabla_ext.cuda, nnabla_ext.cudnn"
ImportError: libcudnn.so.7: cannot open shared object file: No such file or directory
By the way, I confirmed "/usr/local/cuda-8.0/lib64/",  I could find only "libcudnn.so.6".

Thank you for using nnabla!
Your system cannot find correct cudnn libraries which is used by nnabla-ext-cuda80.
So, if you didn't install cudnn7, please install cudnn7 at first, then setup ldconfig or LD_LIBRARAY_PATH.
But, nnabla currently supports cuda10.0/cudnn7, 10.2/cudnn8 and 11.0/cudnn8.
So, if you can update your system, we recommend to use cuda10 or cuda11.
You can find how to install cuda/cudnn and nnabla from https://nnabla.org/install/

Please let me close this issue.
And please let us know if you still can't run nnabla correctly, we will support you what you want to do as much as possible.


CTRL-290 Fix deadlock of delete

Can you add a short description to the PR, just so in the future we'll have some knowledge about what might've caused this


[in_app_purchase] [iOS] Host platform returned null value for non-null return value

Steps to reproduce
I got PlatformException(null-error, Host platform returned null value for non-null return value., null, null) in Sentry on production after calling SKPaymentQueueWrapper().storefront(). So far the issue occured only for one user.
package: in_app_purchase_storekit 0.3.14
device: iPhone 11
OS: iOS 17.4.1
Expected results
No error
Actual results
PlatformException(null-error, Host platform returned null value for non-null return value., null, null)
Code sample
Code sample
final storefront = await SKPaymentQueueWrapper().storefront();

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel beta, 3.22.0-0.3.pre, on macOS 14.4.1 23E224 darwin-arm64, locale ru-RU)
    • Flutter version 3.22.0-0.3.pre on channel beta at /Users/admin/Application Support/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 87b652410d (3 weeks ago), 2024-04-23 21:41:18 -0500
    • Engine revision b4bfd45986
    • Dart version 3.4.0 (build 3.4.0-282.3.beta)
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/admin/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/admin/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode-15.3.0.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)

[✓] IntelliJ IDEA Ultimate Edition (version 2024.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 79.0.3
    • Dart plugin version 241.15989.9

[✓] VS Code (version 1.89.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (5 available)
    • iPad (Evgeniy) (mobile)         • 00008020-001029592629402E            • ios            • iOS 17.1.2 21B101
    • iPhone 15 Pro (mobile)          • 7FE5FD9E-3748-405C-A6F2-F50CD2509BAD • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-4 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 14.4.1 23E224 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 14.4.1 23E224 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 124.0.6367.208

[✓] Network resources
    • All expected network resources are available.

• No issues found!

https://github.com/flutter/packages/blob/a99c7f5f5710f2bd63e873c5560093787baa9f51/packages/in_app_purchase/in_app_purchase_storekit/darwin/Classes/InAppPurchasePlugin.swift#L119-L122 isn't consistent with the Pigeon definition; both return lines can return nil, but the Pigeon definition says the return type is non-nullable. Either the Pigeon definition needs to change, or any nil value needs to set the out error (which is the reason the output shows as nullable on the native side per this comment).
Converting the generated code to Swift will help avoid this kind of thing since the non-nullable return types will actually become non-nullable since Swift uses throw rather than out errors.

cc @LouiseHsu

Assigning to @LouiseHsu as part of https://github.com/flutter/flutter/issues/119106


Rolling file by date writes to original file

This adds a .seelog_rolling_tracker to track the timestamp of the last log roll.

This change is

Since this change removes the case of the current file being named differently from the original file, we can simplify some of the logic. I haven't done so in case there are other unimplemented cases we want to support that need to handle this case. If you don't think there are, I'll go ahead and do this cleanup.

Testing against my own go project, and it looks like I broke sorted log history, digging into this now.

Hey jonsyu1
Please check our original discussion: basically we can work with your original idea of just using stat because the rollingIntervalDaily is just a deprecated feature.

I see that you already made the filename checking cleanup, so basically we could just modify this pull request to remove the .tracker file. Or you can create another one with your changes and with cleanup, but without the .tracker part, whatever is more convenient for you.
And, anyway, thanks for your contributions, that's appreciated.

Great, I'll go ahead and modify the pull request. Thanks for taking a look. I'm excited to make use of this feature in my project's next release in the coming week!

Hey @goodsign, just checking in to let you know that I have updated the pull request per our discussion. No rush to get it reviewed: we're using my fork in the meantime and are happy to move over once this merges :)

Hey @jonsyu1
I've tried and merged it, sorry it took a while :) Everything seems working well, so thank you for your contribution!


Compatiblity Issue - Safari 10 (macOS Sierra)

The following code works well in Chrome, but it functions differently to me in Safari. Can anyone pls confirm / review my fix?
    EXIF.getData = function(img, callback) {
        if (=>(<=(self.Image && img instanceof self.Image)
            || (self.HTMLImageElement && img instanceof self.HTMLImageElement)=>)<=
            && !img.complete)
            return false;
       ...
   }

I'm trying to use it with Safari / Ionic and I can confirm the headache is a real.


help table is scrollable in y axis

Fixes #117.
@lirsacc I see we have the help table overlay had height: 110% here - I changed it to 100% which feels more correct. Is there a good reason for it to be 110%?

@lirsacc here's the commit with 110% height btw

No reason that I know of, doesn't do anything when changed in dev tools so I guess a typo which remained...

Thanks @lirsacc, I'll get this merged.


Solo.pressMenuItem(int) clicking middle of screen

From<EMAIL_ADDRESS>on February 03, 2015 05:02:43
What steps will reproduce the problem? 1. Have up, delete and done icons in action bar.
2. Populate screen with grid view of buttons.
3. Solo.pressMenuItem(2) presses one of these instead of icon in action bar. What is the expected output? What do you see instead? Solo.pressMenuItem(2) shouls press last icon on action bar i.e. done icon. However a grid item in the GridView is pressed instead. What version of the product are you using? On what operating system? Robotium 5.2.1 in Android Studio on Windows 7. Please provide any additional information below. Solo.pressMenuItem(2) works correctly if no GridView items present. However, as soon as GridCiew populates, one of these is pressed instead.
Original issue: http://code.google.com/p/robotium/issues/detail?id=655

From<EMAIL_ADDRESS>on February 23, 2015 16:30:15
This method was originally introduced for the menu bar. Unfortunately we can't do anything about your case as that will effect backwards compatibility.
Status: WontFix
Owner<EMAIL_ADDRESS>


Clarify state names

Hey Data,
as discussed in Discord, I have found if you are checking for the Plugin state, the output is not very clear:

By counting the different categories you can already select the right one, but to make it better you could add the category somewhere in the name as well, so it is easier to identify the right state to check.

Hm, counting doesn't seem to work unfortunately...

Fixed in the new update !


Support flags enums

We should have a special kind of enum for flags that are meant to be bitwise ORed together:
!flags
  values:
    - none
    - red
    - green
    - blue

The first value will always be 0.
As with enums, you can specify the base type and integer values:
!flags
  values:
    none: 0
    red: 1
    green: 2
    blue: 4

Implemented in #57


Check for new Python version in images

Signed-off-by: Luca Giorgi<EMAIL_ADDRESS>

Assuming this is the test run kicked off yesterday: https://opendatascience-jenkins-csb-rhods.apps.ocp4.prod.psi.redhat.com/job/rhods-ci-pr-test/55/
I see only expected failures, i.e. Git and Python version


Support web fonts (@font-face)

Many websites use web fonts. We should support them.
Implementation Plan

When (just after) a stylesheet is added to the Stylist we should introspect the stylesheet to look for @font-face rules
We should fetch any font URLs found (for now this can be synchronous as with our other networking)
If the font is successfully fetched then it should be registered with the Parley FontContext using https://docs.rs/fontique/0.1.0/fontique/struct.Collection.html#method.register_fonts . For now this can not worry about supporting custom names, matching criteria or deregistering.

I believe this can be closed.


Clean up pointer use in BundleSpawner/BundleInserter

Objective
Following #10756, we're now using raw pointers in BundleInserter and BundleSpawner. This is primarily to get around the need to split the borrow on the World, but it leaves a lot to be desired in terms of safety guarantees. There's no type level guarantee the code can't dereference a null pointer, and it's restoring them to borrows fairly liberally within the associated functions.
Solution

Replace the pointers with NonNull and a new  bevy_ptr::ConstNonNull that only allows conversion back to read-only borrows
Remove the closure to avoid potentially aliasing through the closure by restructuring the match expression.
Move all conversions back into borrows as far up as possible to ensure that the borrow checker is at least locally followed.

I wonder if it would make more sense for bevy_ptr's non-null API to look more like a typed version of `bevy_ptr::Pt

Isn't that just &'a T and &'a mut T? The way I view it is that borrows are lifetimed, mutability-locked, and typed pointers, and bevy_ptr tries to fill the gap between *mut () and &'a mut T by dropping one or more of the constraints on borrows.

Isn't that just &'a T and &'a mut T?

I see it as an &'a mut T with the aliasing requirement lifted --  but I guess that's just &'a UnsafeCell<T>

CI is failing still <3


Android IPv6 Leak

Server: Docker-OpenVPN 2.4 running on Ubuntu 16.04
Server connection: ipv4-only, no ipv6 connection.
Local: Pixel XL - Android 8.0
Local client: Tried all of them - OpenVPN Connect, OpenVPN for Android, VPN Client Pro - all with various settings
Local connection: Mobile Data Plan - LTE - (both ipv4 and ipv6)

Here is my client & server config.
I'm having trouble connecting from my android (on LTE). I can successfully connect to my LAN, but I end up leaking my ipv6, and routing my ipv6 through the VPN only ends up blocking internet access - still connected on LAN though.
I switched devices and tried with both a Mac & iPhone - and it just worked out of the box - and I lost the ipv6 - so no leak.
Other things I have tried:

Not Bypass VPN for local network
Enabling ipv6 & routing gateway
Editing server.conf and adding route-ipv6 200::3 (don't remember exact command)
Editing & Enabling upd4 on both server & client

I just want to block or disable ipv6 (client side, or even server config - couldn't care - just want ipv6 out of the way) - Any feedback or help would be appreciated!
X-POST from reddit
and openvpn forums

For future readers wanting to disable IPv6 - I solved my problem by realizing that as soon as I covered the leak, I ran into another issue with the MTU, as small packets were being delivered, larger ones dropped (in retrospect, makes sense since IPv6 = larger packets)
The solution is adding an mssfix value to the client ovpn file - many guides online to finding that value.


Please Update tModLoaderServer to the new tMod version

Which one?

Problem will be fixed right now after new version released.
Also, this mod will no longer need a special tModLoaderServer.exe.


Add cytokinesis right version by fashrod

Song Name
cytokinesis right version
Artist Name
fashrod
Source
Youtube
Youtube Link
cBDxGINGjns
Direct File Link
https://www.youtube.com/watch?v=cBDxGlNGjns
Song ID
1229116
Start Offset [ms]
4
Extra
Created from Auto Nong.
Created for level: 93754669

reject
invalid YouTube link


Mobile Nerf sRGB and readme fix

Description
Considering the MLP was trained with sRGB values and the Swapchain is SRGB, convert back to linear on shaders to prevent double linear->sRGB conversion.
Right image is with the fix, left is without it:

Also removes a readme line that was mistakenly added on original commit.
Fixes #
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes do not add any regressions
[x] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

Note: The Samples CI runs a number of checks including:

[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[x] I have tested the sample on at least one compliant Vulkan implementation
[ ] If the sample is vendor-specific, I have tagged it appropriately
[ ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ ] Any dependent assets have been merged and published in downstream modules
[ ] For new samples, I have added a paragraph with a summary to the appropriate chapter in the readme of the folder that the sample belongs to e.g. api samples readme
[ ] For new samples, I have added a tutorial README.md file to guide users through what they need to know to implement code using this feature. For example, see conditional_rendering
[ ] For new samples, I have added a link to the Antora navigation so that the sample will be listed at the Vulkan documentation site

Just a minor nitpick, but why not put the sRGB shader functions into a separate file and include instead of duplicating the same code for half a dozen shaders?

@SaschaWillems with the include directive it works fine on Windows but on Android I would always get an error of type "'#include' : Could not process include directive for header name: xxxx".
For reference, what I did was to add "#extension GL_GOOGLE_include_directive: require" on each file that used the include, a shared .h file and ensure the include path was relative (as indicated by the extension spec). I also added all shader files to cmakelists.txt with SHADER_FILES_GLSL, but that didn't help either and it still fails on Android.
I also noticed a few of other samples that make usage of include directive fails to run on Android. At this point I'm not sure if that's my work environment or if there is indeed an issue with the framework/samples.


Add "Code generated by" so github suppresses diffs

Github uses Linguist to determine whether to suppress diffs.
It ignores .go files with Code generated by in the first line
so we can add that to our generated code to suppress diffs.

Awesome. Thanks. Didn't know that was possible


Stop using GC2 DNS proxy and bump AMI

This stops using the DNS proxy in GC2 and also bumps the AMI to the latest version.
This is assuming that we still need this around for a while, but also happy to kill this project :)

:+1:


camera pose file for the real cactus data

Hello Erik, Thanks for your excellent work ! I found that you are obtaining the camera pose from the .npy file in the shape of 3x5 where the the fifth column has height, width and focus which will be essential for camera intrinsics.  I wanted the correct Rotation and Translation matrices. How to get that from this file (.npy)?  Thank You

I hope our data/code can help you! For the real-world scenes (including RealCactus), we used the LLFF data format which is read by this file. Note that the poses arrays are further processed here after the call to _load_data. The camera matrix is essentially computed from the 3x4 matrix left after stripping the last column of the 3x5 matrix containing the height, width, and focus. Note that the axis conventions you use may differ from those used in Ub4D and that would affect how you process the data loaded from poses_bounds.npy.
I would recommend that you visualize the cameras after loading the 3x4 matrices (e.g. after the call to load_llff_data here): the last column is the position and the 3x3 matrix is the orientation of the camera. Visualizing them should give you the understanding necessary to then adapt to your axis conventions. Feel free to reuse the load_llff.py file if that can help.


Testing with oath authentication

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from fastapi import Depends, FastAPI, HTTPException, status

...

client = TestClient(app)

def test_foo(token: str = Depends(oauth2_scheme)):
    response = client.get("/foo")
    assert response.status_code == 200
    assert response.json() == {"Available Endpoints": ENDPOINT_LIST}

Description
I followed the instructions for oath2 with password setup.
My API calls all work and are structured like so:
@app.get("/foo")
async def read_foo(
        token: str = Depends(oauth2_scheme),
):
    return foo()

when I try to run the tests in a different folder and file, the authentication fails, ie i get a 401 response.
from fastapi.testclient import TestClient

client = TestClient(app)

def test_foo(token: str = Depends(oauth2_scheme)):
    response = client.get("/foo")
    assert response.status_code == 200
    assert response.json() == {"Available Endpoints": ENDPOINT_LIST}

Operating System
macOS
Operating System Details
No response
FastAPI Version
0.75.2
Python Version
Python 3.10.4
Additional Context
No response

Is your intention to run test_foo with pytest?
You cannot just toss a Depends in there and expect it to work. When you use Depends FastAPI knows what to do with it but pytest does not. To pytest you are just setting a default parameter.
You have to authenticate with the test client as an api consumer would (sorta like here ) or override the dependency like here

@jgould22 Thanks for the advise. For anyone else with the same issue, I got it working by adding the Bearer token to the request:
def test_get_endpoint_list():
    response = client.get("/", headers={
        "Authorization": f'Bearer {os.environ["BEARER_TOKEN"]}'})
    assert response.status_code == 200
    assert response.json() == {"Available Endpoints": ENDPOINT_LIST}


please surpport send input event to extented display on smartisan nut pro3

adb shell input --ext-display tap/swipe... This commend can sending event to extented display,
so it is very useful for scrcpy --display 2 .
it also can solve the issue "Input events are not supported for secondary displays before Android 10".
Because "adb shell input --ext-display keyevent " can send input to secondary displays on android 9.
it is very useful  for  samrtisan   tnt.
Thank you very much.

@rom1v   Thank you very much.


Determine value of  MSBuildProjectExtensionsPath

dotnet-watch and dotnet-user-secrets assume MSBuildProjectExtensionsPath  is the default value, but in theory users can change this. We should attempt to discover its actual value.
cref https://github.com/aspnet/DotNetTools/pull/243#issuecomment-265599129
See https://github.com/Microsoft/msbuild/blob/052ad30f8f706a87e9fa9431fda1ec9c0b73893b/src/XMakeTasks/Microsoft.Common.targets#L127

Might be possible if CLI were to implement this feature: https://github.com/dotnet/cli/issues/5300. Otherwise, I'm not quite sure how we implement this correctly without opening ourselves to all the problems of using MSBuild API directly.

Depends on dotnet/cli#5300.

Another workaround idea: we could write the dotnet-watch targets to MSBuildUserExtensionsPath instead of MSBuildProjectExtensionsPath.

Another idea: when calling any project, we override CustomBeforeMicrosoftCommonTargets and CustomBeforeMicrosoftCommonCrossTargetingTargets to the location of the path containing dotnet-watch's targets.

Is it not possible for watch to evaluate a csproj to determine what the properties are going to be? (i.e. programmatically figure out something like BaseIntermediateOutputPath). Using Microsoft.Build.Evaluation and Microsoft.Build.Construction seem like they could get us that? Deferring to you to see if you've tried this route before but I have seen it in another project

Is it not possible for watch to evaluate a csproj to determine what the properties are going to be?

@hartmannr76 this was actually the first way we attempted to implement dotnet-watch. It turned out to be very fragile due to https://github.com/Microsoft/msbuild/issues/1097. There is no global installation of MSBuild. dotnet-watch would need to bundle a complete copy of MSBuild (as Omnisharp and VS Code do). Without this, project evaluation will fail anytime a user's project references something our bundled copy of MSBuild is missing. This could be almost anything: docker SDKs, new versions of bundled assemblies (NuGet, MSBuild, BCL, JSON.NET), new targets, etc. It's a huge problem with the way MSBuild APIs are designed.

Thats a bummer... Is there any way to do this without injecting that targets file? (I'm fairly new to the MSBuild props/targets stuff so am still trying to connect the dots on the necessity of it for certain things)

Is there any way to do this without injecting that targets file?

This is the only reliable solution we've found. Our current method of injecting the target causes issues when anyone changes BaseIntermediateOutputPath (a pretty setting to change), so I'm hoping to try other ways of injecting the target that are less fragile.

Right, but what I'm getting at is why have the target file in the first place? A common theme in the JS community is:

Passing in a config file
dotnet watch --config ./.watch run
Placing a file at the root that is picked up at runtime

- Foo.csproj
- .watch
- Program.cs

And it is transparently picked up at runtime.
Personally, I prefer the former because I feel the later is way too elusive; but this gives you the ability to specify what is included/excluded as well as some other items you may want configured that would normally be added into a project file, without coupling this to the msbuild process. In thinking about using/installing this tool at the global level like you specified with dotnet-web, I feel like this makes more sense.
Thoughts?

Understood, but why hook into the msbuild process instead of specifying a config file?

@ricky-hartmann-zocdoc a separate configuration file was considered early on in the design process (see https://github.com/aspnet/DotNetTools/issues/204), but we decided to use the MSBuild file as the source of truth on which files should be watched. The benefit was that that users don't need to yet-another-json-file, and didn't have to manually keep two files in sync (the csproj file and a json file). We found that most users are satisfied with the default list of files given to us from MSBuild. We enabled MSBuild settings to change these defaults, but so far I haven't seen anyone use them. That could be because the settings are an undiscoverable feature, or because the default is good enough. Hard to know.

I tried both workarounds and have this error for many attributes:
error CS0579: Duplicate 'System.Reflection.AssemblyCompanyAttribute' attribute
...

@asfernandes this error appears to be unrelated to this issue. Can you open a separate bug and include repro steps?

Yes, but meanwhile, I'm just starting with dotnet core 2 and want to use watch inside docker at the same time leaving vscode working outside docker. Is there a more direct way to do it or I'm going in the correct path?
My problem is that tools (inside / outside docker) writes absolute paths in files placed in obj.

@asfernandes not trying to be mean, but the scope of your question is beyond what this thread is about. Please open a new issue and tag me. Also, StackOverflow is a good place for these kinds of questions. I'd be happy to explain how to setup dotnet-watch + docker.

Was this moved to a new thread? or SO? If so, could someone share a link, please?


Update README.md

Add Configuration section to the README.md and document the AZDCAP_CACHE environment variable.
Fixes https://github.com/Microsoft/Azure-DCAP-Client/issues/43

@msft-jonnadul Let me know if the current changes from the PR look to be merged.

@ionutbalutoiu your PR looks good to me. Feel free to merge.


CGSpace fails OAI validation at base-search

We fixed OAI support in #57, but now, testing CGSpace's OAI with the base-search validation tool fails with the following single fatal error:
ERROR: No incremental harvesting (day granularity) of ListRecords: Harvest for reference date 2015-02-14 returned no records. Please make sure that the from and until parameters are evaluated.

I've seen several messages on the dspace-tech mailing list about this, and will have to investigate their solutions. //cc @Jayakananthan

Testing with base-search again, we still fail this same check:

According to the discussion in DS-2542 it looks like the problem is in DSpace 4.x - 5.2, with a fix perhaps to be merged in 5.3. DSpace's XOAI module supports from and until dates, but returns no results if you query for one single day, like from=2012-07-17&until=2012-07-17.
Test: https://cgspace.cgiar.org/oai/request?verb=ListRecords&metadataPrefix=oai_dc&from=2012-07-17&until=2012-07-17

DSpace Test was migrated to DSpace 5.5 and now passes validation. Pinging @Jayakananthan if he's still around! Testing from base search with this string:
https://dspacetest.cgiar.org/oai/request?verb=Identify


Fix for broken filter page

Coverage remained the same at 23.873% when pulling c1b845e73d810815484c0757eade614d13a017ab on fix-filter-page into 120e7762a463c980f8145254c8e65192f1774b07 on master.


Enable the Informer's cacheMutationDetector during tests

We should enable the k8s client-go's cache mutation detector during tests.
See https://github.com/kubernetes/kubernetes/blob/07ba609b50d6f00c52acbfd6eaa43e6b527c5c2d/staging/src/k8s.io/client-go/tools/cache/mutation_detector.go#L59-L74

#dibs

Hey @wayilau are you still working on this? If not, I would like to take it.


Parallelize bottleneck creation in retraining script

Hi
It might be possible to parallelize the creation of the bottleneck files in the Inception retraining script. Currently it can only do a few images per second and if you have 100s of thousands of images, it takes forever. I could do it myself also and create a pull request, if that's fine.
Thanks

If you have an improved version, you should just go ahead with a pull request (I'm guessing it's under files in tensorflow/models repo)


feat(ui): persist search criteria for containers, services and stacks

Keep text search criteria during refresh and switching pages.
Fixes #1945

Sure I but can you first check that current way to store search criteria OK?

@deviantony now most of the datatables should be included.
Only ones which I was not able to get working are home page and template list. Any ideas to them?
Codeclimate erros looks to be not related to this one but more likely to existing code. What I should do for them?

Codeclimate is OK.
Endpoint list controller: https://github.com/portainer/portainer/blob/develop/app/portainer/components/endpoint-list/endpoint-list-controller.js#L2
Template list controller:
https://github.com/portainer/portainer/blob/develop/app/portainer/components/template-list/template-list.js#L3
Note that the template list controller should probably be created inside a specific file.

Almost. Now templates list uses filter from endpoint list for some reason. Need to debug more on some other day.

@deviantony is there btw some use case for refresh button on templates page?
I removed it now so search criteria caching works correctly on app pages where refresh button is available.
@yassirh can you maybe test and tell your comments?

@olljanat it's just a matter of consistency and IMO it should not be removed. Looks like you're trying to work-around a problem whereas it should be resolved instead.

@deviantony ok. I just wanted to be sure before waste more time with it...

@deviantony ok. I just wanted to be sure before waste more time with it.
And let me know if you have good ideas what can cause issue.
Afaik it is not directly caused by this change but instead of there is some old code which makes templates page to use same cached search criteria than endpoint list (you can still see it if you check from first commit).

@olljanat what are you saying? Is that a bug in the current develop branch? Do you need help to go further on this topic?

If requirement is that search criteria should we saved on templates page too and removing refresh button is not option then I most probably need help.
I will open conversation to problematic code lines...

@olljanat
I think that this is because both template-list and endpoint-list components are missing the assignement of the table-key binding:

https://github.com/portainer/portainer/blob/6b6a69abfd/app/portainer/views/templates/templates.html#L351
https://github.com/portainer/portainer/blob/6b6a69abfd/app/portainer/views/home/home.html#L41

The binding must also be defined in:

https://github.com/portainer/portainer/blob/6b6a69abfd/app/portainer/components/template-list/template-list.js#L5
https://github.com/portainer/portainer/blob/6b6a69abfd/app/portainer/components/endpoint-list/endpoint-list.js#L7

@deviantony Reverted refresh button disable and added new commit which implements table-key for endpoint and template lists but unfortunately symptoms are still same:

If I write search criteria to templates list and click refresh it disappears.
If I write search criteria to endpoint list and click refresh it is persistent like expected.
If I then switch back to template list it is filtered with search criteria on endpoint list.
Search criteria persistence works correctly on all other pages.

@deviantony Thanks. I didn't see those ones. Now it looks working correctly.
Ready for review.

@deviantony most probably I just forgot to check these after I got this one working. Now all errors should be disappeared.

Ok, LGTM.
Keeping as ready to merge as it impacts a lot of files I'll merge it as one of the last element for 1.20.

Thanks @olljanat :+1:


Adds foreground package 'start' command.

Summary
Adds command to run package in foreground without going through Makefile with yarn start or npm start.
Ticket Link
n/a

My objective is run the package in the foreground easily so whether its in the Makefile or package.json doesn't matter to me. Obviously if I'm the only one interested in doing this then I'll just make an bash alias.

see #1188

Closing.


pipx error while loading shared libraries: libpython3.11.so.1.0

pipx seems to have lost a python context somewhere
Using Ubuntu 24.04
Steps to reproduce:
pkgx install pipx
pipx install pycowsay
pycowsay moo
Error:
~/.local/share/pipx/venvs/pycowsay/bin/python: error while loading shared libraries: libpython3.11.so.1.0: cannot open shared object file: No such file or directory
The pycowsay install does pickup the pkgx python at ~/.local/share/pipx/venvs/pycowsay/bin
python3.11 -> ~/.pkgx/python.org/v3.11.9/bin/python3.11
and there is a ibpython3.11.so.1.0 here:
~/.pkgx/python.org/v3.11.9/lib/libpython3.11.so.1.0
pkgx install python works.

This is an interesting problem. On macOS, it works because it inherits the loader_path:
Load command 14
          cmd LC_LOAD_DYLIB
      cmdsize 80
         name @loader_path/../../v3.11.8/lib/libpython3.11.dylib (offset 24)
   time stamp 2 Wed Dec 31 19:00:02 1969
      current version 3.11.0
compatibility version 3.11.0

does pkgx pipx run pycowsay work? it's makes sense that the package manager (pkgx) running a package manager (pipx) which installs a shim (~/.local/bin/pycowsay) might lose the surrounding environment needed. I wonder if there's a way to solve it without altering pipx internals to change the shim it creates.

yeah, this does work. It is what chatgpt said also. Should pkgx set this up somehow with the pkgx install pipx? I worry that this isn't a global setting so adding it as a global env var could be an issue e.g the python in question is a depedency of pipx through pkgx and not accessible globally.

pkgx pipx run pycowsay does also work.
I did try setting the python location when installing things with pipx using PIPX_DEFAULT_PYTHON. Even doing pkgx install python and using that as a ref, but it got a similar error.

That's exactly the problem. One solution is adding pycowsay to projects/pypa.github.io/pipx/provider.yml in the pantry, which lets us do:
$ pkgx pycowsay moo

  ---
< moo >
  ---
   \   ^__^
    \  (oo)\_______
       (__)\       )\/\
           ||----w |
           ||     ||

the other would be to just package pycowsay into the pantry, so it could be installed like any other pip venv, a la https://github.com/pkgxdev/pantry/blob/main/projects/aws.amazon.com/cli/package.yml.

hmm, yeah, this is a few levels too deep of package managers. I will look at options without pipx. Thanks!


Example of broken on start change

moveOnStartChange is now gone, use <DraggableCore> if you need more control.


dependency injection

Codecov Report

Merging #28 into master will decrease coverage by 2.5%.
The diff coverage is 70.73%.

@@            Coverage Diff            @@
##           master     #28      +/-   ##
=========================================
- Coverage    91.7%   89.2%   -2.51%
=========================================
  Files          28      28
  Lines        1109    1139      +30
=========================================
- Hits         1017    1016       -1
- Misses         58      85      +27
- Partials       34      38       +4

Impacted Files
Coverage Δ

service/http/service.go
87.75% <100%> (-1.34%)
:arrow_down:

service/http/config.go
100% <100%> (ø)
:arrow_up:

service/static/service.go
86.2% <100%> (-0.64%)
:arrow_down:

service/rpc/service.go
100% <100%> (ø)
:arrow_up:

service/static/config.go
94.73% <100%> (+1.4%)
:arrow_up:

service/http/request.go
83.33% <100%> (ø)
:arrow_up:

service/container.go
100% <100%> (ø)
:arrow_up:

service/entry.go
100% <100%> (ø)

service/rpc/config.go
54.54% <28.57%> (-45.46%)
:arrow_down:

service/injector.go
49.01% <49.01%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


Update documentation about AUTH_EXPIRE (IDFGH-922)

Adds an explanation of why the user might be getting AUTH_EXPIRE error:

This can arise while connecting to the network when the AP is out of  range
of the ESP32, i.e., the ESP32 can receive packets  the AP is sending, but the
AP is not able to receive packets that the ESP32 is sending. Try moving the
ESP32  closer to the AP.

The auth is the first phase of the connection process where true bidirectional communication is required. If there is only unidirectional communication, the ESP32 will still find the AP (via the beacons). During the auth phase the ESP32 will keep repeating the auth request for one second, then time out and generate this error.

@balazsracz Thanks for the contribution and very sorry for the slow turnaround. We will look into this PR. Thanks.

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

@balazsracz Thanks again for your contribution, as explained by @liuzfesp , we will close the PR now. Thanks again.


Add support UTF8 emoji

Emoji decoding/encoding was incorrect, so user strings were corrupted

Also performance speedup:


My refactorings and code fixes again, now with .NET 6.0

Merged, thanks! In the future if you can try to keep PRs smaller that would be appreciated 😄 Or at least file an issue first about the refactoring/cleanup you want to do, as it might be easier for me to just do it than go through a PR for big things like this. Also, you might want to use branches on your fork to help make it easier if you ever want to submit multiple PRs, etc. Thanks for the contribution, I'm happy to be on file-scoped namespaces now 😄


Loading indicator

Invision desktop
As we're loading data in order to render the page, we should show a loading indicator so the user understands that the page is loading. To avoid a jarring effect, I think it would be good to show the loading indicator for at least 1 second.. showing it for less kind of defeats the purpose I think.
Implementation note: I think this is best handled if we hook into ui-router state changes.

The easiest way to do this is to just drop angular-sham-spinner or some similar module into the project. Not as configurable, but gets the job done.


error parse library

For example specifying a Library with parameter
Library         SerialLibrary         encoding=ascii

leads to
raise DataError("%s '%s' does not exist." % (file_type, path))
robot.errors.DataError: Library 'SerialLibrary         encoding=ascii' does not exist.

only the library name should be parsed and not the param(s)

Hi embexus,
This library was a prototype/experiment from a couple of years ago and isn't fully maintained.
Thanks for reporting the issue (as well as https://github.com/chrisBrookes93/robotframework-remoterunner/issues/18), give me a couple of days and I'll take a look into this.
Thanks

@chrisBrookes93 Hi, Chris, remoterunner is really good, but I also encountered the problem. Have you found anything on it?

I understand where the problem is now. The regular expression [ IMPORT_LINE_REGEX = re.compile('(Resource|Library)([\s]+)([^[\n\r]*)([\s]+)') ] cannot accommodate library declaration like [Library      SSHLibrary  WITH NAME  SSH]. I will try to fix it. Should I modify on new fork?

I use now gitlab-runner to trigger my Robot Gitlab CI and this plugin is for me not required anymore.

I use now gitlab-runner to trigger my Robot Gitlab CI and this plugin is for me not required anymore.

I actually consider to use CI, but currently I have to stick to it.

Hey @jayvynsong,
I am travelling for 3 weeks and don't have access to a laptop so I will have to pick this up when I return. In the meantime, feel free to issue a fix on a fork.
And apologies @embetrix , I completely lost track of this!
Thanks

Thanks, working on fixing.


x/website/content/static, x/tools/godoc/static: consider reusing repo list from x/build/repos instead of maintaining own hard-coded copy

This is an extension of #36047. /cc @bradfitz
There are hard-coded lists in the static templates for x/website/cmd/golangorg and x/tools/cmd/godoc, e.g.:
https://github.com/golang/website/blob/12a8390500dd3fedb41561111a59df98883b92b9/content/static/packageroot.html#L122-L138
In theory, x/website can be modified to get the list of repos from x/build/repos and pass that list to the template. However, that would require modifying golang.org/x/tools/godoc to either be able to pass data through from x/website, or having golang.org/x/tools/godoc itself get the list from x/build.
There may or may not be a constraint about whether we should add a requirement on x/build to x/tools, since x/tools contains many stable packages, while x/build is more internal and less oriented to be imported by library modules (e.g., see #29935).
I don't think it's worth implementing right now, because it's a lot of forced changes and not a lot of benefit. But there are changes planned in order to resolve #29206, and this may become more viable after that. Filing this issue to track this task.

Rather than have anything depend on x/build, we could also expose a new gRPC endpoint for this info from maintnerd.golang.org (which we could productionize a bit more) and then godoc can get this info at runtime (and cache in memory for like 1-5 minutes) instead.

That is an option to consider, but I think one of the valuable use cases for x/tools/cmd/godoc is to be able to run offline, which isn't very compatible with relying on RPC calls to fetch this information.
It may be a better option for x/website/cmd/golangorg though.

By now, I think this can be done directly in x/website without modifying x/tools/godoc, similar to how it was done in CL 229483.


I can't add one specific blog

All blogs download normally, but for some reason I can't add this one http://shaymeliaa.com/  I can't even click on the add button, so I decided to try writting it like shaymeliaa.tumblr.com and it worked but that's a older blog of the same person, what can I do to download the first one?

Thank you very much, it worked.


Add a code of conduct

adding a code of conduct will improve this repo😊

Hello @Ayush0431
Looking forward to seeing your work.

Hey @GabeNedden I have made a PR for this issue hope you like it .


URDF substitute for Ellipsoid geom objects?

Hi! Was looking into translating a MJCF file into a URDF file, and was wondering what the best substitute would be for the "ellipsoid" type geometry objects.
Would appreciate any tips. Thanks!

Does URDF not support ellipsoids?
If not then I think your best bet is to just create ellipsoidal meshes.
Maybe in MeshLab?