feat: HahnSeries smul order inequality

Given a Hahn series x and a scalar r such that r • x ≠ 0, the order of r • x is not strictly less than the order of x.  If the exponent poset is linearly ordered, the order of x is less than or equal to the order of r • x.  (Note that the order of a Hahn series is not uniquely defined when the exponent poset is not linearly ordered.) This is a complement to the addition result HahnSeries.min_order_le_order_add.

bors r+


Clipping / insetting

This builds on #325 (so merge that first) and adds the ability to define rasterized polygons that can be used for clipping and other purposes. Most of the work here is by @nithinp7, because it incorporates the changes in #252.

Thanks @kring! This is good to merge, I'll write issues for any remaining small concerns (including the one pointed out above).


Changes to set the loglevel as per the setting 'auth.service.log.level'

Changes to set the loglevel as per the setting 'auth.service.log.level' during server startup and reload
This also needs the Cattle PR https://github.com/rancher/cattle/pull/2838
https://github.com/rancher/rancher/issues/9390

LGTM


docs: fix some broken links

I'm not sure when or how these were broken, but the URLs are cleaner now

I am not familiar with the new syntax that doesn't refer to the name of the file but these links seem to broken now for me when browsing the README on the web. It seems like for other people too: https://amundsenworkspace.slack.com/archives/CGFBVT23V/p1622019021119500


component review for on-long-press

Description
Pull request for the component review:

on-long-press

Type of change

[X] Documentation (new)

How Has This Been Tested?
Testing Checklist:

[x] Tested in latest Chrome
[X] Tested in latest Safari
[X] Tested in latest Firefox

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have requested a review from ... on the Pull Request

This one now has Prettier running on it. I hope it looks good now.


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cbr/ibm (source)
module
minor
1.9.0 -> 1.15.0

terraform-ibm-modules/cos/ibm (source)
module
minor
6.12.2 -> 6.13.0

terraform-ibm-modules/key-protect/ibm (source)
module
minor
v2.3.1 -> v2.4.0

terraform-ibm-modules/resource-group/ibm (source)
module
minor
1.0.6 -> 1.1.0

Release Notes

terraform-ibm-modules/terraform-ibm-cbr (terraform-ibm-modules/cbr/ibm)
v1.15.0
Compare Source
Features

added pre-wired rule for IS (VPC infra) -> COS in fscloud submodule (#302) (2e8521d)

v1.14.0
Compare Source
Features

added pre-wired rule for IKS -> IS (VPC infrastructure) in fscloud submodule (#306) (4026d8c)

v1.13.0
Compare Source
Features

update terraform version constraints to >= 1.3.0, <1.6.0 (#319) (57f862a)

v1.12.2
Compare Source
Bug Fixes

added extra validation to the existing_serviceref_zone and existing_cbr_zone_vpcs variables in the fscloud profile submodule (#303) (f24ab4a)

v1.12.1
Compare Source
Bug Fixes

added a default value for the operations variable to make it backward compatible with recent CBR service updates around api types (#305) (2f89973)

v1.12.0
Compare Source
Features

added pre-wired rule for icd-databases in fscloud submodule (#294) (2d8d25b)

v1.11.0
Compare Source
Features

add activity tracker and logdna to list of custom rules (fscloud submodule)  (#297) (3eb5aa7)

v1.10.0
Compare Source
Features

add fscloud added pre-wired rule for AT -> COS (#296) (0e10af6)

v1.9.2
Compare Source
Bug Fixes

endpoint set to direct for VPC to COS in the fscloud submodule (#283) (4467165)

v1.9.1
Compare Source
Bug Fixes

update module references in README (#284) (321229b)

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v6.13.0
Compare Source
Features

update terraform version constraints to >= 1.3.0, <1.6.0 (#447) (de7ddf8)

terraform-ibm-modules/terraform-ibm-key-protect (terraform-ibm-modules/key-protect/ibm)
v2.4.0
Compare Source
Features

update terraform version constraints to >= 1.0.0, <1.6.0 (#445) (40edc27)

terraform-ibm-modules/terraform-ibm-resource-group (terraform-ibm-modules/resource-group/ibm)
v1.1.0
Compare Source
Features

update terraform version constraints to >= 1.0.0, <1.6.0 (#508) (0a10be9)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

/run pipeline

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


Negative prompt?

Is it possible to implement a negative prompt with this?

I'm not sure, they had lots of handling in the code to make empty negatives to match the positive, which leads me to believe it wouldn't work as intended, can try exposing that at some point though.


fix: Fixing external benchmarks memory problems

Description
Charlie's memory management updates made it so we can't allocate some of our structures in global scope, which broke the benchmarks. This is a fix
Resolves https://github.com/AztecProtocol/barretenberg/issues/548
Checklist:

[x] I have reviewed my diff in github, line by line.
[x] Every change is related to the PR description.
[x] The branch has been merged with/rebased against the head of its merge target.
[ ] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[x] There are no circuit changes, OR a cryptographer has been assigned for review.
[x] New functions, classes, etc. have been documented according to the doxygen comment format. Classes and structs must have @brief describing the intended functionality.
[ ] If existing code has been modified, such documentation has been added or updated.
[x] No superfluous include directives have been added.
[x] I have linked to any issue(s) it resolves.
[x] I'm happy for the PR to be merged at the reviewer's next convenience.

Note https://github.com/AztecProtocol/aztec-packages/issues/1174
Although as this continues to work in barretenberg repo I am taking a bit before looking into it again


No Module Named "Voyager"

I get this error each time I try to run Voyager.

What steps did you do to install it?

Sorry - I got that figured out. Now I just a key error. I will see what I can do about that tomorrow.

The issue has been resolved, therefore marking it as closed.


How can i simulate keyboard (or mouse) hid with socket data

I use Scrcpy Client in Python.
I upgraded to 2.4 -> 2.6.1.
Normal key injection not working on supercell's games and i want to simulate hid, can anyone help me?

Try MYScrcpy with HID Mouse support
https://github.com/me2sy/MYScrcpy
or read how to realize hid mouse in python
https://github.com/me2sy/MYScrcpy/blob/main/src/myscrcpy/core/control.py

Try MYScrcpy with HID Mouse support https://github.com/me2sy/MYScrcpy
or read how to realize hid mouse in python https://github.com/me2sy/MYScrcpy/blob/main/src/myscrcpy/core/control.py

Thank you so much, i will use it


show log with anouncements

great would be a log of the announcements with date, time, type, length, ....
Perhaps adding a new form which can be opened by clicking the station logo.


How to change permissions on sftp file?

Hi,
is there a way to change the permissions of a file on a SSH Server? Set-SFTPFile doesn't seem to provide a permissions parameter. My problem is that the permissions are set to  "rw-r--r--" by default but they should be  "rw-r-----" in my case.
Thank you in advance for your help
Martin

sorry just found out about Set-SFTPPathAttribute...


Wacom stylus support

Description
Could we please get pen pressure support? Artsists don't use mice and setting 1% strength is barely workable, still too much strength for pen input.

Duplicate #290
Use a nightly build, which has raise/lower reduced to 1/10th.


pkg/helm: migrate to secret storage backend in namespace of CR

Description of the change:

Migrates the helm-operator from an in-memory storage backend to a secret storage backend so that release state is persisted to the cluster.
Migrates from an operator-global storage backend to CR-specific storage backends so that release secrets are stored in the same namespace as the CR.

Motivation for the change:
This PR is the first step towards completing #1100. We'll also be able to close #917, whose goal is to migrate to a persistent storage backend.

@LiliC

Not too familiar with the problem, so just curious to learn more.

The background is that for Helm (specifically the Tiller components of Helm) to function properly, there needs to be a storage backend that stores and manages the release state for every release. Helm has a storage interface for managing this.
When starting the helm-operator for the first time, the storage backend is empty. As CRs are added and updated and releases are installed and upgraded, respectively, the storage backend is populated and updated with the latest release state.
We're currently using the in-memory storage backend, which means that all of this state disappears when the operator is restarted, so there needs to be some way to repopulate the storage backend when it comes back up. To do that, we save the release state to the CR status and use the syncReleaseStatus function to make sure that the storage backend is populated at the very beginning of the reconcile function (before running install/upgrade/uninstall tasks)
This PR swaps out the in-memory storage backend for a secret backend, which means that release state will be stored via the storage backend with secrets, rather than in memory. When the operator restarts, it will no longer need to sync release state to its backend because it's all already there, stored in the cluster as secrets.

My only concern is that this is a tempory/transition state, what is the reason behind this?

The catch with the above is that we need to gracefully handle transitioning release state from the CR status to the secret-based backend.
As a follow-on to this PR, we plan to store only the most relevant bits of information about the release (e.g. release name and version) in the CR status and remove the rest of the full release state.
With that in mind, we could plan on leaving this syncReleaseStatus function in place, such that it only runs if the CR status contains the full release state. In the follow-on PR, we'll stop saving the full release state to the CR status, and we can update the syncReleaseStatus function to additionally remove the release state from the CR status once it has been migrated to the new secret-based backend.
Does that make sense? WDYT?

Thanks for the detailed explanation! It makes sense, maybe we should document this for the user, that we create the Secret in the cluster that stores the information about the release state.

With that in mind, we could plan on leaving this syncReleaseStatus function in place...

I agree, we should leave it, if at all possible, mainly since users tend to skip releasses when upgrading.

... , such that it only runs if the CR status contains the full release state.

Can we just always have it run whenever the secrets are not present in the cluster. For example, what if it happens that the secrets get deleted by accident but the CR status doesn't contain the full release state?

what if it happens that the secrets get deleted by accident but the CR status doesn't contain the full release state?

If the secret (but not the release resources) associated with the deployed release gets deleted and there's no release state to sync from the CR, the operator will think that the CR represents a new release that needs to be installed. It will:

Attempt to install a new release, which will fail because the release resources already exist
Uninstall the failed release in an attempt to rollback to the previous state, which deletes the new secret and existing release resources
Return an error from the Reconcile function, causing the CR to be requeued for reconciliation.
During the next reconciliation, attempt to install a new release, which should succeed since the release resources were deleted during the uninstall of the failed release.

This is essentially the same thing that would happen if we didn't have this transition code to sync the CR status into the new secret backend.

But still curious about the question, can the user in a way create the Secret with the release information "manually"?

It would be difficult to do it by hand, since the Helm package responsible for creating the release secrets stores the state as a gzipped protobuf. Theoretically, they could be restored from a backup though.


Docker Images?

Are there any official docker containers for registry?
Keen to start using it with NiFi but I can only find https://github.com/TheBookPeople/hortonworks-registry-docker which seems pretty outdated.

https://github.com/hortonworks/registry/tree/master/docker may be useful.

Hi satishd!
Thanks, I did see that but got a bit overwhelmed. 🤕
My current plan is to pull down https://github.com/TheBookPeople/hortonworks-registry-docker/pull/2 to build the image myself and see how I get on with it.


gupnp-tools: add required icons, rename from gupnptools

This fixes #11690. I was able to run gupnp-universal-cp and find a minidlna instance with it.

Looks good. Could you rebase you changes against latest master? After that I will merge.


Add caveat on PCH errors

Moving files that have generated PCHs causes an error that you cannot clear easily.  Here are the instructions on how.
I was prompted to place this info here via #6797

Awesome, thanks! LGTM in general, but it will need to be reviewed by @mkonicek as he is the grammar master here 💃

What does "Moving after compile mean", and what does "Moving a React Native project" mean? Does this mean moving the whole folder, e.g. cd /tmp; react-native init MyApp; mv MyApp ~/?
Just from reading this documentation I can't tell. Thanks for putting the effort in adding it, please just make it a bit more specific and let's merge it.

@GantMan updated the pull request.

@GantMan updated the pull request.

@GantMan updated the pull request.

@mkonicek I made the message more expressive as you requested.  I feel it reads well, now.
I also removed the italics on e.g. since there was a pre-existing usage of e.g. without italics.  Please let me know if I can help any further.

It's been a while since the last commit was reviewed and the labels show this pull request needs review. Based on the blame information for the files in this pull request we identified @mkonicek as a potential reviewer. Could you take a look please or cc someone with more context?

@GantMan updated the pull request.

@grabbou  - this is kinda aging a bit, was a small commit with a thumbs up from @mkonicek

This reads really nicely, thanks and apologies about the delay, I've been swamped :)
@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


[refactor] checkout activity

Motivación y Contexto
reducir las configuraciones de review and confirm + uso limpio de constantes

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@7ddae00). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff             @@
##             develop     #992   +/-   ##
==========================================
  Coverage           ?   28.91%
==========================================
  Files              ?      294
  Lines              ?    14714
  Branches           ?     1340
==========================================
  Hits               ?     4255
  Misses             ?     9925
  Partials           ?      534

Impacted Files
Coverage Δ

...va/com/mercadopago/core/MercadoPagoComponents.java
28.67% <ø> (ø)

...o/review_and_confirm/ReviewAndConfirmActivity.java
0% <ø> (ø)

...go/review_and_confirm/ReviewAndConfirmBuilder.java
0% <0%> (ø)

...rc/main/java/com/mercadopago/CheckoutActivity.java
0% <0%> (ø)

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 7ddae00...11a03f8. Read the comment docs.


vimPlugins.render-markdown-nvim: add nvim-treesitter dependency

Description of changes
From their README
it seems that treesitter is required.
The rest look like optional dependencies.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

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

2 packages built:

vimPlugins.render-markdown-nvim
vimPluginsUpdater

it should not be too hard to test the plugin. add the dependencies into RTP (since nixpkgs infra doesn't do that yet) , open a markdown and see if if neovim errors out. like
nvim -c"set rtp+=${vimPlugins.nvim-treesitter}" -c "${render-markdown-nvim}" "PreviewMarkdown" -es  +quitall!
Some tests are visible for reference at
https://github.com/NixOS/nixpkgs/pull/343769

it should not be too hard to test the plugin. add the dependencies into RTP (since nixpkgs infra doesn't do that yet) , open a markdown and see if if neovim errors out.

This didn't make it to the PR in the end.
Do you have a plugin examples where those tests are used already ?

this is one of those cases where the plugin doesn't actually depend on nvim-treesitter, but it requires one or multiple parsers to be installed.
In such cases, adding nvim-treesitter as a dependency won't make the plugin work out of the box. Users will still have to specify nvim-treesitter.withPlugins ...

This is one of those cases where the plugin doesn't actually depend on nvim-treesitter, but it requires one or multiple parsers to be installed.
In such cases, adding nvim-treesitter as a dependency won't make the plugin work out of the box. Users will still have to specify nvim-treesitter.withPlugins ... There's no need to add nvim-treesitter as a dependency if the plugin doesn't actually try to require one of its (legacy) modules.

Right, I forgot about this. I will revert it to prevent such noise from remaining in the code base.


Prevent activating nginx config after editing disabled proxy host

Check if proxy host is enabled and add an Nginx configuration only if the host is enabled to make sure it is consistent with the the db entry and the GUI
Fixes #290

Docker Image for build 1 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-903

Docker Image for build 6 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-903


Hide text magic and helper mode behind feature-flag

Feature Description
Temporarily hide text magic and helper mode behind feature-flag
Alternatives Considered

None

Additional Context
From UX Evaluation
Screenshots
The two last icons:

---
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Verified in QA:

Verified in UAT


chore: add .DS_Store to .gitignore

mac users should have this in their .gitignore file, but sometimes don't, as evidenced by https://github.com/leanprover-community/mathlib4/pull/3511, which then unfortunately broke the port-status bot for a day!

Thanks :tada:
bors merge


use tox-lsr 2.2.0

Upgrade to tox-lsr 2.2.0 to pick up fix for improving collection
testing to get rid of black.

Thanks, @richm!


Is it possiable to transfer "Generate Mock Data" feature into Python library?

"Generate Mock Data" is such a awesome feature for me, so I want bring it to my python code and try to generate db data with your way, so I wonder if Is this possiable to transfer "Generate Mock Data" feature into Python library?

The extension is written in typescript, if you want to implement this function, obviously you need to use Python to rewrite,
A total of three steps are required

Query information_schema.TABLES to get the table structure
Create insert sql and mock options based on table information
Merge insert sql and mock data, run final insert sql.


[Alert] -  Alert does not change color

Steps to Reproduce

Open https://vuesax.com/docs/components/Alert.html#animate
Click on tabs color

Expected
The Alert should change color
Result
The Alert does not change color
Testcase
https://vuesax.com/docs/components/Alert.html#animate

Hello I'll love to work on this task please assign It to me


Would be pretty nice if we had better controls for the program, basic stuff like WASD,Arrow keys etc etc to control games.

[x] I have checked that a similar feature request does not already exist.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I agree

I agree

for the time u can search for a python script fo this

for the time u can search for a python script fo this

WASD and arrow keys should work, don't they?

WASD and arrow keys should work, don't they?

WASD and arrow keys should work, don't they?

I'm not aware of such feature existing nor it ever working, can you please show me how they work?

WASD and arrow keys should work, don't they?

I'm not aware of such feature existing nor it ever working, can you please show me how they work?

Hmmm… press an arrow key on your computer keyboard in the scrcpy window, it will inject the corresponding key events.
Some games may ignore them however (if they just expect you to press some widgets buttons in the UI). In that case, what you want is keymapping, which is not implemented in scrcpy (#712).

Hmmm… press an arrow key on your computer keyboard in the scrcpy window, it will inject the corresponding key events.
Some games may ignore them however (if they just expect you to press some widgets buttons in the UI). In that case, what you want is keymapping, which is not implemented in scrcpy (#712).

Hmmm… press an arrow key on your computer keyboard in the scrcpy window, it will inject the corresponding key events.
Some games may ignore them however (if they just expect you to press some widgets buttons in the UI). In that case, what you want is keymapping, which is not implemented in scrcpy (#712).

OK, I actually tried it out in some programs actually did something, However are there any plans to add keymapping without doing a complicated command?

Hmmm… press an arrow key on your computer keyboard in the scrcpy window, it will inject the corresponding key events.
Some games may ignore them however (if they just expect you to press some widgets buttons in the UI). In that case, what you want is keymapping, which is not implemented in scrcpy (#712).

OK, I actually tried it out in some programs actually did something, However are there any plans to add keymapping without doing a complicated command?

However are there any plans to add keymapping without doing a complicated command?

Currently, no. It would be easier with another UI framework to do something more interactive.

However are there any plans to add keymapping without doing a complicated command?

Currently, no. It would be easier with another UI framework to do something more interactive.

However are there any plans to add keymapping without doing a complicated command?

Currently, no. It would be easier with another UI framework to do something more interactive.

Ah, okay, hopefully it can be implemented in the future!

However are there any plans to add keymapping without doing a complicated command?

Currently, no. It would be easier with another UI framework to do something more interactive.

Ah, okay, hopefully it can be implemented in the future!

help me , i play pes mobile in scrcpy , but movement (WASD and arrow) not work, give me solution please

WASD injects WASD keys. If the game handle these keys, then it works, otherwise it does not.


Add scope and basic authentication

Also updates the POST endpoints to include all possible ticket fields
@lukecampbell @FBRTMaka please review/merge

:+1:

@Bobfrat :+1:


Add page usage view and show usage information in status side panel

Fixes #1722.
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

Happy with the functionality here. However, from my testing I'm feeling that there are a few user experience issues that we can't ignore - these really come down to the fact that interpreting database relations in understandable-to-ordinary-human terms is a harder problem than we've acknowledged so far...

Will an editor intuitively know what it means for a page to be "used 3 times"? I'm thinking it would be a good idea to change this wording to "linked 3 times" or "referenced 3 times", as a special case for pages. (Links between webpages are probably one of the few places where the real-world understanding of a database reference is a cross-reference, rather than "belongs to" or "appears inside" - although things like author snippets are more borderline.)
One of the pages I looked at has someone subscribed to comments, which shows up as one usage described as "(Private page subscription)". Obviously, in this case PageSubscription should just be excluded from reference indexing completely, but there are no doubt other relations that we haven't considered that are "internal" to a page or otherwise unnecessary to count (Redirects? Search term promotions?) - not to mention ones that are custom to an individual project. The conversation in #9731 about making reference indexing opt-in feeds into this, although that only really covers excluding the target model from indexing (Page in this case), which isn't what we want here. I'm wondering if it would be reasonable to make a heuristic that any ForeignKeys with on_delete=CASCADE are most likely "dependent" objects that a user would expect to be deleted anyhow, and thus don't need to be recorded in the reference index unless we explicitly opt in to it?

I've pushed a commit to change "used" to "referenced" for pages only. I've also added wagtail_reference_index_ignore = True to PageSubscription. I'm not so sure about the other models, though!


Can't share a Flutter project on multiple machines, because hard-coded absolute paths

I'm building a Flutter app, and I'm using two machines. One macbook at school, and one windows machine at home. Both flutter installations are flawless, except for one thing. When I create a flutter project it hardcodes the paths to different things, like packages and the android SDK. Which leads to the other machine not finding said files since they are in a different location. Packages can be fixed by running "flutter packages upgrade", but the SDK (and possibly more things i havent noticed) are nor fixed. Since flutter can easily find them when creating a project, it should be easy to have it recreate the paths.

If you have specific files in question, that could help.  See also https://github.com/flutter/flutter/issues/9827 and https://github.com/flutter/flutter/issues/9502.

Great question, thanks for the feedback! Sounds like we do want to have "portable" Flutter projects, as much as possible.

Hi @leddy231 I think we can deduce the files you are referring to but just to be sure, which operations/files are causing you trouble?

Sharing projects between machines that have different framework paths should work out of the box if you're using git and haven't modified the initial .gitignore files in the project. Are you perhaps using Mercurial?

Im sharing a flutter project, and im using dropbox to do it. Its usually way easier for me to do use dropbox, and not git. It has worked in the past with regular android studio project (opening the project makes the studio re-find the SDK).
Just ran flutter create windowsvsmactest on both machines and compared all file.
The Dart SDKs seams to be hardcoded in several different files.
.idea/libraries/Dart_SDK.xml
.idea/libraries/Flutter_for_android.xml
android/app/build.graddle
.packages
Just to name a few.
I know dropbox is not the most professional syncing tool out there, but for a student with different machines that likes to just jump in and jump out of coding sessions, its does the task of keeping everything synced.

Hi @leddy231
android/app/build.graddle shouldn't have any hardcoded paths, but rather should have relative paths like apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"
The .idea and .packages files do indeed contain local paths. Typically those would be ignored by source control as controlled by the .gitignore file. I'm afraid we don't have good support for "recalculating" them if you open a project on a machine where they are different.

Note that you can always delete the .packages file, and then regenerate it locally by running flutter packages get.

The pubspec.lock also contains local paths

but it definitely makes sense to check this file in.

Local paths in pubspec.lock were removed a while ago.
Excluding files and directories listed in .gitignore should be enough to avoid this problem.
Are there plans to do more work?

for iOS; In the podfile there is a local path right? under flutter_application_path. Note: I have not yet tried relative path like './childdirectory' or '../siblingdirectory' but the docs make it seem like an absolute path...

If we are using Git, the confidential details we can't upload.
If we are uploading, Google will block our app.
So please share a better solution to work multiple persons want to work on the same project.

Relative path works

Note that you can always delete the .packages file, and then regenerate it locally by running flutter packages get.

This worked for me.  Although I agree that some files will have to be added to .gitignore to make the app (accessed from a repo) portable across platform IDEs.  I was cross developing on a mac and windows laptop.  But this is a good workaround for now.

.gitignore is an option. But we can't share those details with our team through the repository.
For share those confidential details, we have to find another solution.
I am thinking about a single solution, which will run on a local server and through the server, we can share the repositories. Anybody have more information, kindly share.

I migrated an app two ways, one via git repo and one via direct file transfer.

git repo

create /android/local.properties with correct references in sdk.dir and flutter.sdk
create /android/key.properties with correct reference in storeFile
The fact that the .packages file is missing is not really a large issue, as the app won't compile and it should be clear that you need to run flutter packages get .  This would be a compiler issue to notice that .packages is missing.

direct file transfer

delete .packages and run flutter packages get to rebuild
same issues as above, plus
build artifacts in /build directory prevents app from compiling, just deleting the contents of debug and release folders to remove some .d files, these files had hard coded references to the location of sources, which prevented a successful compile, this issue was confusing to me since I expected the build artifacts to be removed on a new compile.  May be a VS Code issue.

So transfer via git repo and definitely the way to go, since the build artifacts will be ignored.  However, the three files still pose an issue.

local.properties is referenced in /android/app/build.gradle and should force a compile error if it is missing, this would give you a prompt to go create it, again this is a compiler issue.
key.properties is referenced in /android/app/build.gradle and should force a compile error if it is missing, this would give you a prompt to go create it, again this is a compiler issue.

I found a simple solution to this using andriod studio. You need to put the .idea file in gitignore. This way this file is not shared. Once you clone your repo open the project in andriod studio and it will create all necessary files in based on current OS.
Pull flutter packages and run the application. It will work

The flutter gitignore template contains many files that cannot be shared across different machines. If there are specific files that are not in the gitignore that should otherwise be, then please file a new specific issue for that item.


Add Integration test with scala_import and a macro import

@chaoren @brendandouglas now that rules_scala has scala_import WDYT about adding an integration test to see that when it's used people can use a filegroup of a jar and it's still identified in intellij?
cc interested people- @gkk-stripe @hmemcpy @greggdonovan

@ittaiz is this still relevant?

I think so.
The goal here was to get confidence IJ integration still works.
Having said that I think this is still costly to implement so U don't know if id put the effort into this right now.


Setup Mapbox to display on the same language of the user device

Currently when using a device set up in Portuguese for example, the map displayes names of countries and regions in English.

Task being closed as it is duplicated of another more detailed task - see #55 .


Update Pages “home”

Automatically generated by Netlify CMS

👷 Deploy Preview for parsimony-dev processing.
🔨 Explore the source changes: 30ec7ee4b9d4e77bce8df0110c7e700bd1d68100
🔍 Inspect the deploy log: https://app.netlify.com/sites/parsimony-dev/deploys/6188225e8b975d0007dcf210


Aggregate + Clean runtime fixed point arithmetic types

Currently we have:

Perbill
Permill
PerU128
Fixed64

Need to add:

[ ] RationalU32
[ ] Clean up PerU128
[ ] Mak the Api and exposed functions of all of them consistent. All should expose normal, checked, saturating etc.

Maybe

[ ] Move to a new sr-arithmetic crate or arithmetic.rs in sr-primitvies.
[ ] Extensive and unified tests for all of them.
[ ] Should probably close ~https://github.com/paritytech/substrate/issues/2558~ https://github.com/paritytech/substrate/issues/2908 along the way.

CC @thiolliere

instead of RationalU32 I would name it FractionalU32 because it can't exceed 1, or maybe we can make that it can exceed one.

PerU128 doesn't implement multiplication which is to distinguish with Perbill and Permill

Those types will be very useful to ink as well. Does it make sense to move them into a standalone fixed point rust crate?


Unexpect behaviour using non-standard HTML tag image

When I set the innerHTML of an element with <image></image>, the tag is transformed to the standard image tag img.
It seems to happen only with innerHTML. To reproduce: (copy-paste from the README.md)
var jsdom = require("jsdom");

jsdom.env(
  "https://iojs.org/dist/",
  ["http://code.jquery.com/jquery.js"],
  function (err, window) {
    console.log("there have been", window.$("a").length - 4, "io.js releases!");

    // Start here
    var elementDiv = window.document.createElement('div');
    elementDiv.innerHTML = '<image></image>';
    console.log(elementDiv.firstChild.outerHTML); // ---> <img>

    var elementImg = window.document.createElement('image');
    console.log(elementImg.outerHTML); // ---> <image></image>
  }
);

If you are wondering why I filled this twisted issue: I use Jest (testing framework based on jsdom) to test a React application. I used to mock my image components by creating dummy "image" component TestUtils.mockComponent(MyImageComponent, 'image'); and since React 0.14, React renders these components with innerHTML instead of createElement: https://github.com/facebook/react/pull/4611/files#diff-c66701e4ca8fce9507233d224556a71dR994
I can easily fix my tests by using an another dummy tag ("myimage"). I just thought you would be interested in knowing this inconsistency

http://stackoverflow.com/questions/11928566/img-vs-image-tag-in-html

@ncuillery This is consistent with browsers and specs. HTML spec says to replace any occurrences of the <image> in the markup for the historical reasons. Meanwhile, then you create element manually via DOM this rules doesn't apply because there is no parsing happening.

OK it makes sense :smile: Thanks for the explanation.


Sorry, you are not allowed to access this service.

Describe the bug
I am not allowed to access this service.
To Reproduce
Expected behavior
Screenshots

Desktop (please complete the following information):

OS: [e.g. Windows]
Browser [e.g. chrome]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

What's the value of the Bing cookies you set?

I am not set Bing cookies this's null

Should i need set Bing cookies to a suitable value?

Yes.

How i get the suitable value?

Open Edge, go to Bing, go to bing chat, add yourself to the waitinglist, wait until activated, chat with the bot there, go to development mode (F12 on win) and find the cookies of the network request or get a cookie extension and copy it that way

Open Edge, go to Bing, go to bing chat, add yourself to the waitinglist, wait until activated, chat with the bot there, go to development mode (F12 on win) and find the cookies of the network request or get a cookie extension and copy it that way
That's the solution to the bug, it can be closed now :)

Thanks for your answer! but i wonder that must choose Bing preset?


Show letter

sorry for the stupid question in the wrong place :)
but how can I actually see this task?

this is all i get in the show-letter branch and I can't open 0 - A

@elbotho https://github.com/serlo-org/serlo-abc#launch-the-storybook. Message me in Slack if the README isn't clear enough :)


"savecursor": true does not work on Windows

Description of the problem or steps to reproduce
Set settings.json "savecursor": true on Windows.
Open file, move cursor down.
Close file.
When trying to reopen same file shows error as "invalid handle".
When checking .config/micro/buffers there is only one file called "C" in that directory.
Specifications
settings.json "savecursor": true does not work on Windows.
Version: 1.1.5-dev.163
Commit hash: 67ec0d3
Compiled on May 04, 2017

Still occurring with latest dev 1.4.2-dev.77

Is this issue still observed with a recent version of micro?


libs: added  cbor serialization

Contribution description
Was added  tinycbor  and added the serialization functions also was add the libs folder.

Issues/PRs references
this PR  fixes the issue #82

no issue linked?

yes i linked the issue #83  @luisan00

yes i linked the issue #83 @luisan00

not, there is no link

ohh little mistake.

ohh little mistake.

I think it can be fixed , change it

ohh little mistake.

Same here:

ohh little mistake.

I think it can be fixed , change it

change ir

Please do a squash before to approve


Functional test

Minor point - let the person who started the conversation be the one who decides if it's resolved ;) Adding reply comments is good


Drop active-support dependency

active-support is already a dependency of most GOV.UK projects, and I'd like to avoid a situation where we depend on a newer/older version in this gem, causing dependency conflicts. It's best practice to avoid any unnecessary dependencies in gems.

Well, I removed activesupport and it turned out the tests weren't failing.


Documentation pipeline fix

only for tests

@drakon660 does this need to be an active PR ? it's spamming workflow runs for CI and email notifications.

@drakon660 it could be made draft

@razvangoga please check if you see this https://tye-v2.github.io/tye2/ correctly and approve this PR.

@razvangoga please check if you see this https://tye-v2.github.io/tye2/ correctly and approve this PR.

it looks very good! great job!

@drakon660 i guess that with having this type of docs (github pages), i could disable the repo's wiki so it no longer shows in the top menu bar? right?


Using many characters from custom TTC font results in not found symbols for several viewers

Perhaps, it is equal Antelle's issue #412.
But, I can't use fontkit branch of PDFKit.
I'm developing with node.js.
How to write "devDependencies" in package.json file to install fontkit branch?
Help me!
Regards!

![foxit](https:

It's correct draw in chrome pdf viewer and foxit reader.

It's err draw in adobe acrobat.
Two images are screen shot of equal pdf file that be maked with pdfkit.

v0.8.0 was just released with fontkit. Please try that.


Fix BigInteger bitwise shifting behaviour

What current issue(s) does this address, or what feature is it adding?
fix https://github.com/CityOfZion/neo-python/issues/951
How did you solve this problem?
convert negative lshift into positive rshift and negative rshift into positive lshift.
How did you make sure your solution works?
added tests cmparing against C# output
Are there any special changes in the code that we should be aware of?
Please check the following, if applicable:

[X] Did you add any tests?
[X] Did you run make lint?
[X] Did you run make test?
[X] Are you making a PR to a feature branch or development rather than master?
[X] Did you add an entry to CHANGELOG.rst? (if not, please do)

Coverage increased (+0.008%) to 85.061% when pulling 5e385079c445bb339ec7abd1093481d00b0d52f3 on ixje:fix-biginteger into e32fcafc0f7290addb04eba087f8fe1f073fdc81 on CityOfZion:development.


Azure SDK Review - [Introduction to IoT Hub Device Provisioning Service]

New SDK Review meeting has been requested.
Service Name: IoT Hub Device Provisioning Service
Review Created By: Paymaun Heidari
Review Date: 05/03/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links:
Description: For the IoT device provisioning service, there is a gap in the python ecosystem specifically a modern data plane service sdk. In this meeting we intend to put forth a proposal for preview.
Detailed meeting information and documents provided can be accessed here

Hi @digimaun,
We'll need the materials for this review added by the end of this week, or we'll have to cancel or push back this meeting.
We'll need a link to Hero Scenarios (these can used pseudo code if the libraries have not been created yet), API View Listings for any libraries that have been created already, links to the API Specs for the service, and links to any core concept docs you feel could give the architects better understanding of the service.

Hi @ronniegeraghty, thanks for the reminder. These materials are understood and intended to be provided by the end of the week, where if we are not ready, we will reschedule / ok to cancel.

Hi @digimaun,
I just wanted to ping again as a reminder that we will need the materials added to this issue by the end of the day tomorrow.

Meeting updated by Paymaun Heidari
Service Name: IoT Hub Device Provisioning Service
Review Created By: Paymaun Heidari
Review Date: 05/03/2023 02:05 PM PT
Hero Scenarios Link: here
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: here
APIView Links: Python,
Description: For the IoT device provisioning service, there is a gap in the python ecosystem specifically a modern data plane service sdk. In this meeting we intend to put forth a proposal for preview.
Detailed meeting information and documents provided can be accessed here

@digimaun - it looks like the APIview is missing the operations.
It looks like the current implementation is wrapping the generated client, and forgot to make the actual operations public.
This shouldn't be necessary, you should be able to expose the generated client directly, and just patch the credentials if you need to.

Thanks @annatisch , I'm curious what is the guidance on when to use the patch pattern vs a wrapping client...for example the azure-servicebus SDK's management client does not appear to make use of patch https://github.com/Azure/azure-sdk-for-python/tree/main/sdk/servicebus/azure-servicebus/azure/servicebus/management

Up until quite recently (last 6 months or so), the "patch" pattern was incomplete and didn't really allow for full flexibility in customizing the generated code. So a lot of earlier SDKs resorted to wrapping the client in order to make the necessary customizations.
However, we've made pretty huge investments in autorest generated code, and now you should be able to customize pretty much anything you need to without needing to maintain a client wrapper :)


Test issue

label should stay since i'm part of the team

label should stay since i'm part of the team

label should stay since i'm part of the team

this is a test comment.

testing 1 2 3

should still be here

should I be here?

another one


Unexpected token .

I installed the tool with sudo npm install -g --unsafe-perm backport.
When I run backport, nothing happens.
When I run npx backport i get:
Unexpected token .

I created both config files according to https://github.com/sqren/backport/blob/main/docs/config-file-options.md

Hey!
It could be related to you running with sudo. I'll try to see if I can reproduce.

I just had the same problem. It was caused by accidentally using an old node version (12). Using node 16 fixed the problem.

Ahh, interesting. Thanks for chiming in @gigerdo .
@ct2034 Is that perhaps the same problem you are running into?

I was using node 8 and saw the same problem. Using node 16 fixed the problem.

@li-boxuan Thanks for chiming in. I might be able to catch this and warn the user with something like:
import semver from 'semver';
import { engines } from './package';

const version = engines.node;
if (!semver.satisfies(process.version, version)) {
  console.log(`Required node version ${version} not satisfied with current version ${process.version}.`);
  process.exit(1);
}


Weak lines ignorance mode

To reduce the computational time, we need the ignorance criterion of weak lines.

Water 334 lines vs 802 lines for Luhman16A:
i.e.  the weak lines below the CIA photosphere do not affect the results.


Multiple middleware

router.get(
  '/users/:id',
  function *(next) {
    this.user = yield User.findOne(this.params.id);
    yield next;
  },
  function *(next) {
    console.log(this.user);
    // => { id: 17, name: "Alex" }
  }
);

When User.findOnde is async,
such as mongoose, which data is returned in a callback,
User.findOnde({_id: id}, function (err, user) {

})

is there a solution to handle the problem?

No clean solution to this at all if you're using mongoose. You'll have to wrap mongoose to return thunks/promises or find a module that does so.

@alexmingoia, thanks, I get it.


error on computed column from dashboard

hi there,
I have an enhanced data table with 3 computed columns, the fomulas for these computed columns are like this
"col3 / col2 * 100.0" (computing percentages).
I'm facing the error "In computed column "col3 / col2 * 100.0", column number 3 does not exist" when including the enhanced table in a dashboard.
The error never happens when displaying the table in Visualisation.
I'm using Kibana 7.8.0 and the version 1.9.1 of enhanced table
Any ideas?
Thanks

Hi,
There is a blocking issue with kibana 7.8 for now: #116
Your issue is the same so I close this issue as it is a duplicate.


ERROR while logging

Hello,
I have completed the instalation and set env variables steps but when is time to authenticate, the program show me this:  I review many times, my password, and my user and is correct, even I can connect to IB directly with the same credentials.
Anyone can explainme why sometimes shows: "Invalid username password combination" and other times: simply "ERROR"?

This is using the gateway of IB only

Thanks

Hey @mcjeikk thanks for reporting your issue and for bearing with me 👍
Unfortunately, I'm as clueless as you are as to why such errors appear in the login page. We've seen in the past that Invalid username password combination is an error that appears randomly even though the credentials provided are correct. Sadly, I'll have to refer you to IBKR support to get more insight on this and suggest they address this issue.

Closing this issue due to lack of activity. Feel free to reopen if you'd like to continue the discussion 👍


font-awesome icons & semantic-ui

For the life of me, I don't get what I should be naming the icon
for example,
fa-floppy-o (http://fortawesome.github.io/Font-Awesome/icon/floppy-o/)
I've tried

and numerous other iterations
what's the magic incantation ? ;)
thanks

For floppy it would be:
<i class="save icon"></i>
You simply search in the semantic ui doku for the icon you want and then open the examples and then you copy it or you insert the name of the icon like I did

so all of the font-awesome icons have been renamed in semantic-ui ?

I don´t think all, but some. Just look in the documentation and you find the names you need.

Icons have generally all been renamed.

just edit site/elements/icon.overrides.import.less and add the following code
/*******************************
         Site Overrides
*******************************/

/*******************************
            Icons
*******************************/

/* new 4.5.0 icons */

/* Web Content */
i.icon.hashtag:before { content: "\f292"; font-family: FontAwesome}
i.icon.percent:before { content: "\f295"; font-family: FontAwesome}
i.icon.shopping-bag:before { content: "\f290"; font-family: FontAwesome}
i.icon.shopping-basket:before { content: "\f291"; font-family: FontAwesome}

/* Brand Icons */
i.icon.bluetooth:before { content: "\f293"; font-family: FontAwesome}
i.icon.bluetooth-o:before { content: "\f294"; font-family: FontAwesome}
i.icon.codiepie:before { content: "\f284"; font-family: FontAwesome}
i.icon.edge:before { content: "\f282"; font-family: FontAwesome}
i.icon.fort-awesome:before { content: "\f286"; font-family: FontAwesome}
i.icon.mixcloud:before { content: "\f289"; font-family: FontAwesome}
i.icon.modx:before { content: "\f285"; font-family: FontAwesome}
i.icon.product-hunt:before { content: "\f288"; font-family: FontAwesome}
i.icon.reddit-alien:before { content: "\f281"; font-family: FontAwesome}
i.icon.scribd:before { content: "\f28a"; font-family: FontAwesome}
i.icon.usb:before { content: "\f287"; font-family: FontAwesome}

/* Payment Icons */
i.icon.credit-card-alt:before { content: "\f283"; font-family: FontAwesome}

/* Video Player Icons */
i.icon.pause-circle:before { content: "\f28b"; font-family: FontAwesome}
i.icon.pause-circle-o:before { content: "\f28c"; font-family: FontAwesome}
i.icon.stop-circle:before { content: "\f28d"; font-family: FontAwesome}
i.icon.stop-circle-o:before { content: "\f28e"; font-family: FontAwesome}

Icons have generally all been renamed.

I think that's a terrible idea. They should have all the same classes from Font Awesome.

The renaming of icons is bad for the following reasons:
FA icon was general in meaning and could be used in all kinds of contexts, semantic icon has single specific meaning. For example: FA "times" icon (X) can mean "remove" as you say, but it can also be "close" and "error".
It is also hard to find icons because you can't search icons by literal meaning, you must translate to some abstract notion. I can't search for "phone" I must browse all icons and find out that it is a "call" icon. I don't want to "call" anybody, I want to display the user's phone information.
Maybe the most important reason is having to learn two sets of icon names.

hello, I'm building a Mobile App with fontawesome and it's has a web admin with react-semantic. I need a class name "translation" table to be able to choose a icon on both interfaces, anyone already done this? something like: {"add to calendar":"calendar-plus-o",...}


About encoding result on random images

Thank you for your great work!!
I have tested the model with some random images from internet, the encoding results and manipulation results is not really good as compared to other images included in your examples folders.

How can I improve the model for better encoding images like those?
For the interpolation, can we adjust the copied style range to interpolate coarse style, middle style and fine style?

By the way, the model trained based on StyleGAN, so can the encoded latent can be fed into a StyleGAN generator?

Faces should be pre-aligned before fed into the encoder. Please refer to this link.
Our encoder is based on a StyleGAN model trained on FF-HQ with 256 resolution. The official model is with 1024 resolution. So, they may mismatch. Furthermore, we use a StyleGAN generator that uses different ws for different layers. This is a little bit different from the original StyleGAN generator that uses the same w for all layers.


It really should support showing/hiding progressbar.

It really makes sence. When there is a "page refresh" (state change to user) we need progress bar, when there is  "command send" (say, a "follow-me" button click), a little spinner would make more sence, although there are all just ajax calls to developers.

This is a duplicate of #113


TensorRT converted models are 2-3 times slower in TF 2.10.0 in comparison to TF 2.9.2

Issue Type
Performance
Source
binary pypi
Tensorflow Version
2.10.0
Custom Code
No
OS Platform and Distribution
Linux Ubuntu 18.04
Python version
3.7
CUDA/cuDNN version
11.2.2 / <IP_ADDRESS>
TensorRT version
7.2.3
GPU model and memory
Nvidia Tesla T4 16GB (aws g4dn.2xlarge)
Current Behaviour?
Model: imagenet_mobilenet_v2_100_224_classification from tfhub.
I converted and run the model using TF-TRT 2.9.2 and 2.10.0.
TF-TRT 2.10.0 is 2-3x times slower than TF-TRT 2.9.2.
Results for imagenet_mobilenet_v2 are below
# imagenet_mobilenet_v2

# TF-TRT 2.9.2
P50 (ms): 0.87

# TF-TRT 2.10.0
P50 (ms): 1.87  (2.15x times slower)

Similar performance issue exist for other models such as inception_v3 and resnet_50 - 2-3x times slower in TF-TRT 2.10.0
Standalone code to reproduce the issue
import tensorflow as tf
from tensorflow.python.compiler.tensorrt import trt_convert as trt
import time
import numpy as np

m=tf.keras.applications.MobileNetV2()
tf.saved_model.save(m, "saved_model")

precision_mode=trt.TrtPrecisionMode.FP32
conversion_params = trt.TrtConversionParams(precision_mode=precision_mode)
converter = trt.TrtGraphConverterV2(
    input_saved_model_dir="saved_model",
    conversion_params=conversion_params)
converter.convert()
converter.save("saved_model_trt_32")

m=tf.saved_model.load("saved_model_trt_32")
f=m.signatures['serving_default']
x=tf.random.uniform(shape=(1,224,224,3), dtype="float32")

# Warmup
N = 10
for i in range(N):
  y = f(x)

N = 100
TTT=[]
for i in range(N):
  t1 = time.time()
  y = f(x)
  t2 = time.time()
  TTT.append(t2-t1)

p50_time = np.percentile(TTT, 50) * 1000
print("P50 (ms):", p50_time)

@bixia1 , @sushreebarsa - just want to check if you were able to reproduce the issue

I found that significant performance degradation in TF-TRT was caused by commit 612a531c9bc729329470e778dfc014d2d978e587 - Excises VirtualPlacer from layout optimizer transposer.
@jkr26 , @Kh4L , @tfeher , @christopherbate , @drivanov , @DEKHTIARJonathan , @kaixih , @jblespiau

@apivovarov I tried to replicate the issue on colab using tf-nightly, and faced RuntimeError: Tensorflow has not been built with TensorRT support. Please find the gist here and confirm the same?
Thank you!

Hi @sushreebarsa,
Below are Colab notebooks to run TF-TRT:

TF2.9.3-TRT7.2.3.ipynb
TF2.11.0-TRT7.2.3.ipynb.ipynb
TF-nightly1.12-TRT8.5.ipynb

I'll give the context I can recall, though I definitely do not remember all of it distinctly.
I believe the possibility of such a regression was discussed in the original change. The issue as I recall it being: the default placement behavior was, in some sense, "accidentally better" for certain purposes, though I do not recall in what circumstances this was expected to be true. It was, however, quite poor in some others--the benchmark from TensorFlow-Federated which yielded the investigation that resulted in this change saw something like a 10X-20X speedup, since the default behavior resulted in data being copied between multiple GPUs while executing, not just a single training loop (so, copying once-per-batch), but while even executing a single forward-backwards pass (so copying multiple times per batch). This benchmark uses internal Google infrastructure, so only runs inside Google and unfortunately I cannot point to it.
The general feeling talking to TF core folks was that the behavior of this virtual placer should not / cannot be relied on downstream, and it did not make particularly strong guarantees about what, precisely, it would do. Further, the code seemed mostly unowned--so there was preference for deleting it.
The regressions we saw in internal Google benchmarks could all be fixed by explicitly annotating the model, which was considered to be 'more correct' than relying on poorly understood behavior of virtual placer, so this was the path that was chosen. I believe there was some feeling that if better placer behavior was needed, it would most properly be implemented in / migrated to 'real placer'. That said, I am no expert on op placement in TF.
Hope this context is useful.

@apivovarov Please refer to the comment above and let us know if that helps?
Thank you!

@sushreebarsa I do not think the comment gives exact solution to return fast TF-TRT inference back.
It seems that the only working solution which we have now is to revert commit 612a531

@apivovarov Could you please confirm if this issue has been resolved?
Thank you!

One of the solution is to reverts commit https://github.com/tensorflow/tensorflow/commit/612a531c9bc729329470e778dfc014d2d978e587
I opened PR to revert it https://github.com/tensorflow/tensorflow/pull/59044 one moth ago.
PR is still open.
The issue still exist in TF 2.10, 2.11 and upcoming 2.12.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The PR which was raised for the revert of the mentioned commit was closed. Please find the PR for the reference.
https://github.com/tensorflow/tensorflow/pull/59044
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


New version: Hellofont.Hellofont version 3.0.3

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.1 schema?


Opt to show cases with missing abstracts

In the current version, if a case has a missing abstract, only the title is displayed plus a DOI if available. Could we make it an option in the settings menu whether or not to use/show cases with missing abstracts? Default should be to use/show.

The user wants to hide the articles with missing abstracts.

@PeterLombaers / @J535D165 @terrymyc would this technically be possible?

I guess technically it would be similar to skipping a record? Then you could do https://github.com/asreview/asreview/issues/328 at the same time.

Technically it is easy to implement.

We don't think this should be stored in the settings menu, because settings are stored in the browser. Therefore, this results in reproducibility issues. Alternative would be a different query strategy (one that moves the records with missing abstracts to the end of the queue).
Model changing options in the future can serve as this setting. I will leave this issue open and add a help-wanted tag to it (to create this query strategy). I will remove the v1 milestone.

I like the idea of developing a new model for this purpose! Would be an excellent First contribution!


remove open collective of footer

This removes the open collective from the footer while we decide when to promote it
Relate: https://github.com/expressjs/express/pull/6064#issuecomment-2444511668

So I believe that we have gotten 👍 from most folks now that we are good to move forward with promoting the OC. We have a small proposal for a "policy" to make sure folks don't feel like they are giving money to us without any plan at all, I will check in to see if we can post that somewhere publicly today. And if so, then we can close this.


Improve the UX of external card links

Currently it's not possible to distinguish external links from internal links on the blog homepage cards and it can be disorienting when these links open in a new tab. This UX is also inconsistent with the blog body content which displays a small external arrow for any outbound links.

Example from Protocol.ai

The approach most consistent with existing visual style would be simply to add the outbound-link icon to the end of the headline for cards that will open in a new window (e.g. everything but blog posts). @jdiogopeixoto, would you be able to fit this in sometime this week, since you know those cards better than anyone else? You can see it illustrated in the below screenshot: the left two cards would have the icon (since they open an off-site link) but the right card wouldn't (since it links to a blog post). Thank you!

Yes, I'm already working on this.
I have two questions:

What should we do when the text is truncated (last card below)? Should we try to fit the icon at the end?

Should the icon always stay in the same line where the text ends? (second card below)

This is happening because I just added the icon to the end of the text, it works but we have those 2 issues. Let me know what you think about this so I can fix it.

Thanks!

Yes - if the title truncates let's fit the icon at the end of the line. If that cuts down the number of characters to truncate at, that's OK.
Good catch. Let's please never have a line with just an icon on it.

Nice thanks! There is a utility to check external links and the icon is included as a global component too ( I think but you may need to check the src), it should be relatively simple :)


🚧(databases) generate databases access credentials for an application

Purpose
We need to normalize and enforce database names because MySQL limits db names to 16 characters by default.
Proposal
Name databases with a pattern: {{ env_type }}-{{ customer }}-{{ app.name }}[{{ extra }}]
where:
customer is max 6 characters
env_type is 1 character (because we are limited to 16 characters, we do not use 2 characters for this information):
    s: staging
    t: pre-production (after s... try/test...)
    p: production
app.name is limited to 6 characters: e.g. richie, marsha, edxapp, etc.
extra is 1 character that can be added if we have several databases for a given customer/env_type/database.

This normalization will allow us to generate interface documents to automate the creation of all our databases. We propose the following format for this document:
# groups_vars/customer/foo/staging/secrets/databases.vault.yml
databases:
  postgresql:
    - release: "9.10"
      databases:
        # Richie staging for customer foo
        - application: "richie"
          name: "s_foo_richie"
          user: "s_foo_richie"
          password: 123456

        # Marsha staging for customer foo
        - application: "marsha"
          name: "s_foo_marsha"
          user: "s_foo_marsha"
          password: 123456

  mysql:
    - release: "5.6"
      databases:
        # Mysql staging for customer foo
        - application: "edxapp"
          name: "s_foo_edx"
          user: "s_foo_edx"
          password: 123456

  mongodb:
    - release: "3.2"
      databases:
        # Mongodb staging for customer foo on Mongodb 3.2
        - application: "edxapp"
          name: "s_foo_edx"
          user: "s_foo_edx"
          password: 123456

        - application: "forum"
          name: "s_foo_forum"
          user: "s_foo_forum"
          password: 123456

This work is an implementation of #153

@jmaupetit @sampaccoud This PR is not in WIP anymore.


drivers: ethernet: w5500: MAC change support

W5500 documentation mentions local-mac-address and zephyr,random-mac-address support, but in reality couldn't find the implementation. This PR adds this functionality.
Updated w5500_set_config() to support ETHERNET_CONFIG_TYPE_MAC_ADDRESS and change the MAC address at runtime as discussed here #39252. Example usage:
struct net_if *iface;
struct ethernet_req_params params;
...
net_if_down(iface);
ret = net_mgmt(NET_REQUEST_ETHERNET_SET_MAC_ADDRESS, iface,
&params, sizeof(struct ethernet_req_params));
net_if_up(iface);

@rpaat new changes must go into the same commit. use git commit --amend and rebase -i

@rpaat new changes must go into the same commit. use git commit --amend and rebase -i

Thank you for the hint. I hope that I managed to do the squashing properly.

@rpaat sorry for the huge delay. We will try to get this reviewed and merged.

Sorry for delayed answer from my side too. It was long time ago when I checked this code. I'm still interested to get into main branch and next release.

I express my interest for the local-mac-address support for this driver. Will be very nice if can be merged for v3.3.0.

I tested the changes in this pull request and it seems to be working.

@jfischer-no


pseudoclass, mark

adds pseudoclass ::selection for  selecting text.
adds mark tag as well

Thank you diomed, that were two useful additions. Thank you for your time. However I think that "lavender" is perhaps not the best color for the Medium.com feels.

yes - well, it's my personal preference, so I put that one.
you change it to whatever you feel is the best one.


Feature Request: Automate Video AG Login

Motivation
Hi, this extension is great in automating some of the logins, but it does not automatically login for RWTH Video.
Required Steps

Recognize that Login is required (e.g. https://rwth.video/19ss-lainf/14125)

Open https://rwth.video/internal/rwthauth
Automatically select RWTH at https://oauth.campus.rwth-aachen.de/login/shibboleth/?q=verify&d=...

Automatically Authorize "Video AG" at https://oauth.campus.rwth-aachen.de/manage/?q=verify&d=... (Submits a hidden form)

Automatically close the OAuth page as it is no longer required and return to rwth.video (reload it to update login state)

Already working on that, not sure whether I will auto-authorize though because some people may have security concerns with that. Maybe I'll add an option for that in the settings which is disabled by default.

Yeah, I do agree with the security concerns.
The main annoyance here is that RWTH Video asks for those permissions every time and not just once.
So maybe you could save if permission was given before and only automatically authorize if that is the case.

It is more difficult though, because I don't see any way to identify the requesting application, other than by name. Good enough, I guess

Yeah, the App Name should be fairly unique considering that every application has to be approved etc.

Implemented in version 1.1.0


fix: handle sync abort, reduce duplicate STATE messages, rename _MAX_RECORD_LIMIT as ABORT_AT_RECORD_COUNT

This is proposed to target PR #1399, either to merge into that PR or replace #1399 and merge to main.
Importantly, this adds handling for how "abort" exceptions should be handled. There are two scenarios: either the stream successfully pauses the sync operation or else it raises a failure exception if pausing is not possible.
Notes:

The private member Stream._MAX_RECORD_LIMIT is renamed to Stream.ABORT_AT_RECORD_COUNT. This rename properly documents the original and valid purpose for this limit - specifically this is a record count after which to trigger abort of the stream sync.
Tests which actually want a sync_all() behavior can call sync_all() directly, as this change reverts the behavior so that sync_all() will once again sync all records. This is the only way to ensure tests will get valid and finalized state messages. Otherwise, the aborted streams will have bookmarks left in an unfinalized, non-resumeable state.
For test scenarios that should silence abort exceptions, run_connection_test() has been refactored to call a new and more generic run_sync_dry_run(), which can accept an arbitrary max record abort threshold. Both run_connection_test() and run_sync_dry_run() suppress the abort exceptions regardless of resumeability. In contrast, sync_all() will trigger the same exceptions if ABORT_AT_RECORD_COUNT is set and breached, but it will not suppress the exceptions as the two test-focused methods would do
When aborting stream sync due to a user or s, we raise an exception that dispositions the caller according to whether the operation is still Singer compliant, namely that we will sync all records "at least once" without data loss.
If we still have a valid stream sync after abort, AbortedSyncPausedException specifies that we are mostly meeting Singer Spec, with the one exception that we left records on the source server.

We are Singer compliant if the user runs again, and repeats until all records are drained.

If we do not have a valid stream sync after abort, due to non-resumable streams setting up an infinite loop on subsequent restarts, then AbortSyncFailedException will be raised.

Other fixes:

A prior PR added a SQLStream implementation with a SQL limit() constraint. This has been updated to be max limit + 1 in order for the SDK to know whether or not there are more records not synced, so we can properly inform the caller whether the sync operation was prematurely canceled or not. (If ABORT_AT_RECORD_COUNT=100 and we have exactly 100 records in the source, then the sync operation can wrap with success. But if there are >=101 records in the source, we will report that the operation was aborted prematurely before syncing all records.
The properties TapTestRunner.tap and TargetTestRunner.target were creating a new tap or target upon each call to the property, which violates the expected behavior of a property returning a pointer to the same object on each subsequent call. I've renamed these to be regular methods, new_tap() and new_target(), since that reflects the behavior as implemented.

Alternatively, we could cache the output of .tap and .target properties, but this seemed likely to leak state across multiple tests that otherwise would be atomic. Hence, I think new_tap() and new_target() as methods probably best describe the desired behavior here.

Due to misapplied modulo math on zero-based record index, we previously were sending a STATE message after every first RECORD message. This PR fixes the modulo math, which breaks pre-recorded tests that expected a STATE message after every first RECORD message.
Previously, some cases would duplicate-send a final STATE message. This PR adds an internal private member Stream._needs_state_flush which is used to track whether one or more RECORD message has been written since the last STATE message being sent. The final duplicate STATE message would no longer to be sent after this PR.

Clarifications:

There was previously some confusion regarding whether or not _MAX_RECORD_LIMIT was applied to certain subclasses of Stream. Given that the base class implementation of Stream._sync_records() was applying the limit (and still does here in this PR), this should fully cover RESTStream, SQLStream, and all other subclasses of Stream.

Future work:

#1409 - would allow us to return a specific non-zero exit code to inform the caller that abort thresholds were met, and that one of these are true, either:

(a) All streams successfully paused their sync operations before shutting down, and further sync invocations should be able to eventually bring in all data which is valid, OR
(b) One or more streams failed to pause their sync operations in a valid state. Rerunning/retrying is not advised, since it will only lead to an infinite loop, where data loss is occurring for one or more streams, due to inability to reach a valid bookmark state.

@aaronsteers this is epic 🙌
I have:

Fixed the failing test by moving the _MAX_RECORD_AGE_IN_MINUTES check onto the _process_record_message and _process_batch_message methods. Previously this check was under _process_state and so was triggering in the target as a side-effect of the redundant STATE message sent by the tap (which you fixed). Now we trigger the check on actual records arriving to the target, which I think is the expected behaviour?
Updated the tap-snowflake test PR to point to this branch, verifying the behaviour and test speedup 🚀
Updated this PR to target main. I agree we can close #1399 in favour of this PR 👍


Exception occurs while assigning the 'XpandListView, ID:Employee_ListView' view to WebWindow:  Object reference not set to an instance of an object.

Hi, I am trying to use the masterdetailmode, but whenever i load the employee page, the attached error prompted. Could you advise if any setting is needed or is there any workaround to use the masterdetailmode?
Thanks in Advance.

Hi, below is the link for a simple project.
https://www.dropbox.com/s/8y0k8qwplybt31m/ExcelImport.rar?dl=0

I am sorry for the late response, from what I see you are using a not supported feature. The framework provides full master-detail support by using dashboard. Please run (step-by-step) this easytest as an example.
https://github.com/eXpandFramework/eXpand/tree/18.2/Demos/Modules/Dashboard/DashboardTester.Module/FunctionalTests/MasterDetail

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


Fix the redirect when resource is not found

I was getting a 500 error when I would do GET http://<IP_ADDRESS>:8000/api/v1/resources/9999. This fixed that to make it a 404 instead.

Fixes #69 (finally)


Markdown layout issues in notebooks

When using the default layout:

depending on the font size / header type the folding indicator shows up with different offsets from the centerline/baseline of the font
there should be sufficient spacing between the folding indicator and the first letter of the header so that the first letter of the header is align with the left border of a code cell

Ok, this one was weird because eventually I realized that I saw it in some windows and not in others. Turns out that I had "notebook.experimental.useMarkdownRenderer": false in my settings which caused this effected when a window with a notebook editor was restored. Once we get rid of old markdown rendering code, this issue is fixed.

We don't use the builtin markdown renderer anymore, the webview based markdown renderer is turned on for all


Fix some unsafe usages of GList

It seems that Go 1.3 has very bad CGo support - it might be possible to fix, but not without a lot of effort. Is it worth trying, or should we simply drop it from Travis?

"Go" for it. ;)


Tracking: support trino PostgreSQL connector

The trino PostgreSQL uses PostgreSQL JDBC connector to access PostgreSQL.
However, when we are currently testing query RisingWave via trino PostgreSQL connector, we gets error when we do select from risingwave table.
The following are some blocking bugs and unsupported features

Failed to getTables with _ in table name: https://github.com/risingwavelabs/risingwave/issues/10018
Failed to getColumns: https://github.com/risingwavelabs/risingwave/issues/10019
When trino tries to fetch the columns information of tables, it calls getColumns on DatabaseMetadata of jdbc connector, and for the PostgreSQL JDBC connector, it involves an batch over window, which depends on

https://github.com/risingwavelabs/risingwave/issues/9124
Bypass the over window by https://github.com/risingwavelabs/risingwave/pull/10024
Frontend panic: https://github.com/risingwavelabs/risingwave/issues/10026

Now we can support running a simple select * from table SQL statement via the trino PostgreSQL connector.
A demo with docker-compose
In the demo workspace, we have the following files:

docker-compose.yml

version: "3.5"
services:
  risingwave:
    image: risingwavelabs/risingwave:latest
    container_name: risingwave
    ports:
      - "4566:4566"
      - "5691:5691"
    command: ["playground"]
  trino:
    image: trinodb/trino:418
    container_name: trino
    volumes:
      - ./etc/risingwave.properties:/etc/trino/catalog/risingwave.properties
    ports:
      - "48080:8080"
  trino-client:
    image: trinodb/trino:418
    profiles: ["client"]
    entrypoint: ["trino", "--server", "trino:8080", "--catalog", "risingwave", "--schema", "public"]
networks:
  default:
    name: risingwave-bagel

etc/risingwave.properties

connector.name=postgresql
connection-url=jdbc:postgresql://risingwave:4566/dev
connection-user=root
connection-password=secret
case-insensitive-name-matching=true

Run docker compose up -d to start the demo cluster.
Create a demo table and insert test data via psql client

psql -h localhost -p 4566 -d dev -U root

# within the psql clent
dev=> create table test_table(id bigint);
CREATE_TABLE
dev=> insert into test_table values(1);
INSERT 0 1
dev=> flush;
FLUSH

Query from trino client

docker compose run -it trino-client

# within the trino client
trino:public> show tables;
   Table
------------
 test_table
(1 row)

trino:public> select * from test_table;
 id
----
  1
(1 row)

A docker compose demo for both presto and trino is in https://github.com/risingwavelabs/risingwave/pull/10201


Curl import changes URL

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
When a curl command is imported, the url is changed by removing the last /
curl 'http://example.com/test/'  turns into http://example.com/test
This affects endpoints that end with /
.bru file to reproduce the bug
No response
Screenshots/Live demo link
https://github.com/usebruno/bruno/assets/14101716/8c417d75-78f6-49ac-a008-737a63840656

When you import cURL from Apidog, you won't encounter this issue. Give it a try; I think it's quite effective.

@jokj624 that line is actually specifically written to remove the / from the end of a url. So seems like they are intentionally stripping of that last backslash for some reason. Not sure why the raw_url field also gets the processed value. That does not seem "raw" to me.

I can't imagine a situation in which I want the request that Bruno sends out to be different from the request cURL sends out. If the user doesn't want a trailing slash they would just delete it from the URL after import (or in the cURL command).
This caused me a huge headache yesterday. I was trying to hit this Python FastAPI endpoint, and I was getting a correct response using cURL on the command line, but importing into Bruno kept giving me a "405 method not allowed" error.

I ended up spinning up a webserver that echoed back the headers, thinking maybe Bruno was attaching a hidden header somewhere? I thought it would be great if Bruno had an "export to cURL" feature, so I could have done a string comparison with the cURL syntax, but I don't believe that exists in Bruno.
15 minutes later, I realized that the trailing / was being stripped off.

@helloanoop do you remember why you chose to strip the trailing slash?

No, I don't remember. Agree that we should not strip the slash if curl command has it.
@sanjai0py Can you raise a PR for this ?


Support applications with multiple packages.

Apktool Latest
OS is linux
Believe it or not , if i decompile an apk and try to recompile it again without any changes in code it fails .
I try using a twitter apk file .
I will leave here the link :
https://www.mediafire.com/file/82w10791m4xrjaf/Twitter.apk/file
You can decompile it , but you are unable to recompile it again even if no changes were made in the code .
Update , forgot to write the logs , here they are :
The log :
----------------------------------Decompiling log--------------------------
I: Using Apktool 2.4.1 on twitter.apk
I: Loading resource table...
I: Decoding AndroidManifest.xml with resources...
I: Loading resource table from file: /root/.local/share/apktool/framework/1.apk
W: Could not decode attr value, using undecoded value instead: ns=android, name=resource, value=0x7e010000
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values */* XMLs...
I: Baksmaling classes.dex...
I: Baksmaling classes2.dex...
I: Baksmaling classes3.dex...
I: Baksmaling classes4.dex...
I: Baksmaling classes5.dex...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
I: Copying META-INF/services directory
---------------------------------------------------------------------------
Recompiling log :
I: Using Apktool 2.4.1
I: Checking whether sources has changed...
I: Smaling smali folder into classes.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes4 folder into classes4.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes5 folder into classes5.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes2 folder into classes2.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes3 folder into classes3.dex...
I: Checking whether resources has changed...
I: Building resources...
W: /opt/testapk/twitter/AndroidManifest.xml:831: error: Error: No resource type specified (at 'resource' with value '@2113994752').
W:
brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1): [/tmp/brut_util_Jar_12790044432587942505.tmp, p, --forced-package-id, 127, --min-sdk-version, 21, --target-sdk-version, 30, --version-code, 28820000, --version-name, 8.82.0-release.00, --no-version-vectors, -F, /tmp/APKTOOL4379167829085885196.tmp, -e, /tmp/APKTOOL8538338056184325237.tmp, -0, arsc, -I, /root/.local/share/apktool/framework/1.apk, -S, /opt/testapk/twitter/res, -M, /opt/testapk/twitter/AndroidManifest.xml]|

Yeah got the same thing :

Best way to check if any changes can be made in an apk is , decompile and try to recompile , if recompile fails without any changes in the code of apk then apktool will never be able to recompile that apk no matter what you do .
I used in twitter.apk the 2.4.1 version but 2.5.0 was also giving the same issues , so the log is from 2.4.1 .

Believe it or not , if i decompile an apk and try to recompile it again without any changes in code it fails .

I would believe it, you are disassembling an application and trying to remake it. That isn't easy but any stretch of the imagination.

@archos-sa - Your error is not related.

If we take the bugged resource.
        <meta-data android:name="com.android.vending.splits" android:resource="@2113994752"/>

It looks bad. That isn't a valid reference.
W: /home/ibotpeaches/Downloads/Apktool/2514/Twitter/AndroidManifest.xml:831: error: '@2113994752' is incompatible with attribute resource (attr) reference.
W: error: failed processing manifest.

So lets check aapt.
      E: meta-data (line=0)
        A: android:name(0x01010003)="com.android.vending.splits" (Raw: "com.android.vending.splits")
        A: android:value(0x01010025)=@0x7e010000
      E: meta-data (line=0)
        A: android:name(0x01010003)="com.android.vending.derived.apk.id" (Raw: "com.android.vending.derived.apk.id")
        A: android:value(0x01010024)=(type 0x10)0x1

Hmm, its a reference to a resourceId. Lets look for it.
➜  2514 aapt d resources Twitter.apk| grep '0x7e010000'
      spec resource 0x7e010000 com.twitter.android.webrtcnative:xml/splits0: flags=0x00000000
        resource 0x7e010000 com.twitter.android.webrtcnative:xml/splits0: t=0x03 d=0x00000bbd (s=0x0008 r=0x00)
➜  2514

We could not decode that resource which explains all of this.
W: Could not decode attr value, using undecoded value instead: ns=android, name=resource, value=0x7e010000

Okay, this is a bug with our handling of multiple package apks. We always assumed manifest resources were on 7F, but these are on 7E. However, we don't properly load enough data into the attribute parser for manifest at that point in the code to have a mapping of pkgId to name, so this will take some changes.

Yeah no way I see easily to resolve this without major research. Multiple packages in one resources.arsc. If we store them differently, they have the same resources. So we'd need to overhaul extraction code to isolate to one package.
I can fix the decoding error and properly resolve the reference, but then we get:
➜  2514 apktool b Twitter --use-aapt2
I: Using Apktool 2.5.1-d1c006-SNAPSHOT
I: Checking whether resources has changed...
I: Building resources...
W: /home/ibotpeaches/Downloads/Apktool/2514/Twitter/AndroidManifest.xml:831: error: resource com.twitter.android.webrtcnative:xml/splits0 not found.
W: error: failed processing manifest.

aapt2 can't build it since its a non-standard framework.

➜  2514 aapt d resources Twitter.apk | grep 'Package Group' -C 10
Package Groups (2)
Package Group 0 id=0x7f packageCount=1 name=com.twitter.android
  Package 0 id=0x7f name=com.twitter.android
    type 0 configCount=1 entryCount=117
      spec resource 0x7f010000 com.twitter.android:anim/abc_fade_in: flags=0x00000000
      spec resource 0x7f010001 com.twitter.android:anim/abc_fade_out: flags=0x00000000
      spec resource 0x7f010002 com.twitter.android:anim/abc_grow_fade_in_from_bottom: flags=0x00000000
      spec resource 0x7f010003 com.twitter.android:anim/abc_popup_enter: flags=0x00000000
      spec resource 0x7f010004 com.twitter.android:anim/abc_popup_exit: flags=0x00000000
      spec resource 0x7f010005 com.twitter.android:anim/abc_shrink_fade_out_from_bottom: flags=0x00000000
      spec resource 0x7f010006 com.twitter.android:anim/abc_slide_in_bottom: flags=0x00000000
      spec resource 0x7f010007 com.twitter.android:anim/abc_slide_in_top: flags=0x00000000
--
        resource 0x7f160025 com.twitter.android:xml/standalone_badge_gravity_top_start: t=0x03 d=0x00000bc1 (s=0x0008 r=0x00)
        resource 0x7f160026 com.twitter.android:xml/standalone_badge_offset: t=0x03 d=0x00000bc2 (s=0x0008 r=0x00)
        resource 0x7f160027 com.twitter.android:xml/sync_data_adapter: t=0x03 d=0x00000bc3 (s=0x0008 r=0x00)
        resource 0x7f160028 com.twitter.android:xml/trends_pref: t=0x03 d=0x00000bc4 (s=0x0008 r=0x00)
        resource 0x7f160029 com.twitter.android:xml/tweet_prefs: t=0x03 d=0x00000bc5 (s=0x0008 r=0x00)
        resource 0x7f16002a com.twitter.android:xml/twitter_external_file_provider_paths: t=0x03 d=0x00000bc6 (s=0x0008 r=0x00)
        resource 0x7f16002b com.twitter.android:xml/twitter_internal_file_provider_paths: t=0x03 d=0x00000bc7 (s=0x0008 r=0x00)
        resource 0x7f16002c com.twitter.android:xml/two_factor_auth_settings: t=0x03 d=0x00000bc8 (s=0x0008 r=0x00)
      config v22:
        resource 0x7f160021 com.twitter.android:xml/shortcuts: t=0x03 d=0x00000c0b (s=0x0008 r=0x00)
Package Group 1 id=0x7e packageCount=1 name=com.twitter.android.webrtcnative
  DynamicRefTable entryCount=1:
    0x7e -> com.twitter.android.webrtcnative

  Package 0 id=0x7e name=com.twitter.android.webrtcnative
    type 0 configCount=1 entryCount=1
      spec resource 0x7e010000 com.twitter.android.webrtcnative:xml/splits0: flags=0x00000000
      config (default):
        resource 0x7e010000 com.twitter.android.webrtcnative:xml/splits0: t=0x03 d=0x00000bbd (s=0x0008 r=0x00)

tldr - need to research dynamic refs more and how these function.

Should we auto split them into different packages / thus resources.arsc during framework install?
Automatically move resources to 7F?
Handle automatic loading of non-framework id packages during recompilation

This works perfectly for Twitter, given the following patch above.

Use this patch.
Install the application itself as a package (rename to 126 - 0x7e)
Decode application
Rebuild application

The way this works is that Apktool extracts needed frameworks (1, 126) as requirements but skips 127 (0x7F) as that is private. So when you try and build you get a framework missing exception. That is why installing the application itself, but lying around the pgkId works. So when aapt2 is looking for resources the -I include parameter includes the old application which has the 0x7e resources to find mapping.
This of course is highly dependent on having the old application which true app builders do not. I've been messing around with Instant apps like the discussion here - https://github.com/iBotPeaches/Apktool/discussions/2524 and trying to figure out how to make Android Studio be very verbose with all its commands to gradle/aapt2 so I can figure out the steps that it does to produce a binary from source.
With that knowledge, I could tweak Apktool to meet that behavior.
TLDR - Ongoing.

For those who want to edit Twitter, you can merge apks with the SAP (Split APKs Packer): https://forum.release-apk.com/viewtopic.php?f=23&t=99023
This tool merges the split apks into one apk. Afterwards, editing with apktool is possible again. Indeed, the tool uses a modified version of apktool to make the merging of apks possible.
However, the tool isn't open-source, so it might include malware. Use at your own risk.

I'm pretty sure the same issue occurs when with the latest Snapchat apk:
https://www.file.io/download/l5kTneQcY7tV

Console output`
java -jar apktool.jar d snapchat-11-21-1-37.apk -o snapps
I: Using Apktool 2.5.1-baf8bb-SNAPSHOT on snapchat-11-21-1-37.apk
I: Loading resource table...
I: Decoding Shared Library (com.snapchat.android.dummy_module_dynamic2), pkgId: 126
I: Decoding Shared Library (com.snapchat.android.native_specs_crypto_lib), pkgId: 123
I: Decoding Shared Library (com.snapchat.android.dummy_module_dynamic2), pkgId: 126
I: Decoding AndroidManifest.xml with resources...
I: Loading resource table from file: C:\Users\sando\AppData\Local\apktool\framework\1.apk
W: Could not decode attr value, using undecoded value instead: ns=android, name=resource, value=0x7b010000
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values */* XMLs...
I: Baksmaling classes.dex...
I: Baksmaling classes2.dex...
I: Baksmaling classes3.dex...
I: Baksmaling classes4.dex...
I: Baksmaling classes5.dex...
I: Baksmaling classes6.dex...
I: Baksmaling classes7.dex...
I: Baksmaling assets/secondary-dex/dummy_module_dynamic2.dex...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
I: Copying META-INF/services directory
java -jar apktool.jar b snapps
I: Using Apktool 2.5.1-baf8bb-SNAPSHOT
I: Checking whether sources has changed...
I: Smaling smali folder into classes.dex...
I: Checking whether sources has changed...
I: Smaling smali_assets folder into assets.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes2 folder into classes2.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes3 folder into classes3.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes4 folder into classes4.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes5 folder into classes5.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes6 folder into classes6.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes7 folder into classes7.dex...
I: Checking whether resources has changed...
I: Building resources...
W: C:\Users\sando\Downloads\snapps\AndroidManifest.xml:403: error: Error: No resource type specified (at 'resource' with value '@2063663104').
W:
brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1): [C:\Users\sando\AppData\Local\Temp\brut_util_Jar_40083337941325849378736354795908562753.tmp, p, --forced-package-id, 127, --min-sdk-version, 19, --target-sdk-version, 29, --version-code, 2163, --version-name, <IP_ADDRESS>, --no-version-vectors, -F, C:\Users\sando\AppData\Local\Temp\APKTOOL2495946303200493142.tmp, -e, C:\Users\sando\AppData\Local\Temp\APKTOOL2018924885346877021.tmp, -0, arsc, -I, C:\Users\sando\AppData\Local\apktool\framework\1.apk, -S, C:\Users\sando\Downloads\snapps\res, -M, C:\Users\sando\Downloads\snapps\AndroidManifest.xml]`

**Apktool Version** - 2.5.1-baf8bb-SNAPSHOT
**Operating System** - Windows
**APK From?** - Playstore
Following the patch you mentioned:
java -jar apktool.jar if snapchat-11-21-1-37.apk
*renaming 127.apk to 126.apk in framework folder
Gave the same output as above

Hi @iBotPeaches
I'm very excited about the patch and can not wait to try it, although it's a draft now.
But I'm getting some difficulty trying it. Could you please help with more details?

Use this patch.

I cannot find any artifact file as apktool in the PR action. Could you show me ?

Install the application itself as a package (rename to 126 - 0x7e)

How should I do this, and how will it help because it won't impact the APK file (which I will decode in the next step)?

Decode application
Rebuild application

I think I will be fine perform these steps. But if you have any tips that would be helpful
Thank you so much

@iBotPeaches whats the status of this bug? Is a fix for this possible on the 3.0.0 milestone horizon? Is there any additional research that can be done from people experiencing this bug? I am experiencing this as well, though unlike @Sunep-source, using the patch https://github.com/iBotPeaches/Apktool/pull/2517 produced the same error as did v2.6.0.
error: '@<some_id>' is incompatible with attribute resource (attr) reference.
W: error: failed processing manifest.

I unfortunately cannot share the apk in question

@iBotPeaches can we do something to help you on this issue ? Is it planned to be fixed in the next version or any upcoming versions ? Thanks.

Hi @iBotPeaches ,
Do you think it makes sense to prioritize this issue?
I think it's kinda critical because 2/5 APK files I tested encounter this issue.

@Nogias9x - I don't see this as something to prioritize, but you are welcome to continue from the research I have to come to a solution. I believe the Android 12/12L issues are more important at this time.

@Nogias9x - I don't see this as something to prioritize, but you are welcome to continue from the research I have to come to a solution. I believe the Android 12/12L issues are more important at this time.

@iBotPeaches what do Support for only Android Manifest disassembling (no rebuild) means? Is it possible apktool has option to decompile AndroidManifest.xml and compile it again? Maybe that reduce the chance of crashing?

That is a feature I have planned for a future version to only disassemble the Manifest without needing to decode the entire resources.asrc file. It however, will not allow compilation. Unsure what that has to do with this ticket though.

@Nogias9x - I don't see this as something to prioritize, but you are welcome to continue from the research I have to come to a solution. I believe the Android 12/12L issues are more important at this time.

@iBotPeaches what do Support for only Android Manifest disassembling (no rebuild) means? Is it possible apktool has option to decompile AndroidManifest.xml and compile it again? Maybe that reduce the chance of crashing?

That is a feature I have planned for a future version to only disassemble the Manifest without needing to decode the entire resources.asrc file. It however, will not allow compilation. Unsure what that has to do with this ticket though.

that would be so amazing when it's happening.
Because I was checking this apk and it asked to reinstall from playstore due to detected crashing.
Upon checking apktool log, it failed to decode smt, that is why being able to disassemble the Manifest only seem to be the solution

@Nogias9x - I don't see this as something to prioritize, but you are welcome to continue from the research I have to come to a solution. I believe the Android 12/12L issues are more important at this time.

hi, any updates to this? I'm facing the same issues right now

Same issue on popular app, @iBotPeaches, I would be very grateful if you could find a solution to this problem :)

Just to pass the error replace
<meta-data android:name="com.android.vending.splits" android:resource="@12345"/>

with
<meta-data android:name="com.android.vending.splits" android:value="base"/>

https://github.com/iBotPeaches/Apktool/issues/2139#issuecomment-539451858

Any update?

is it possible to move resources from splitted package into main one?

in case you stuck with this - I fixed broken resources manually and it works. replaced null with required resource and moved appripriate file into base apk resources

bump?
Is there any movement on this?
Where is this work-around-patch written about in the comments here?  I have been unable to find it.
Thank you!