New driver for Mx510 series Canon Printers

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Sorry for not including cask's name and version: canon-pixma-mx510-printer-driver <IP_ADDRESS>

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

Welcome, and thank you for the submission. In the future, please use x in checklists (no spaces inside the brackets) and Github will make them clickable. Y and N makes it hard to read.

There was no need to close the issue. The submission itself was fine and I corrected the template.


enhancement add retry for broker pwrite

Proposed changes
Issue Number: close #xxx
Problem summary
Problem:
We got following error frequently while SELECT xxx INTO OUTFILE:
ERROR 1064 (HY000): RpcException, msg: Fail to write to broker, broker:TNetworkAddress(hostname=a.b.c.d, port=8111) failed:write() send(): Broken pipe
Reason:

we cache broker thrift client in BE;
thrift client check connect isOpen only return cached flag, not care the real socket is opened or closed;
after we get client from cache, the socket may already closed, then pwrite will failed.

How to fix:
Other interfaces such as open and close, will reopen and retry again, but pwrite do not retry.
As there are write offset inside pwrite, and the broker(server) side also will check the write offset, it is safe to retry pwrite.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[x] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.39 seconds
load time: 459 seconds
storage size:<PHONE_NUMBER>1 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221124123031_clickbench_pr_52241.html


sensingkit (android app) only sends data while databox app is visible in foreground

this is databox 0.5.2 (plus some fixes) and android app 0.5.7.
the sensingkit client in the phone app only sends data when the databox app is in the foreground and screen is on. If the app is moved to the background or the screen is off then it does not send data. (i've tested with light, step count & acceleration)
(It does send data with lock screen visible and the app underneath it.)

I think Kevin shoudl know this.

I do know this. I wrote it that way. Both iOS and Android are making it increasingly difficult to access sensors and data in the background, like location. It was simpler to only have it run in the foreground, and worry about running it in the background in the future.

There are some ways to do that in iOS. For example, using Location or Microphone (audio recording) in the background and with the user’s permission you can access all sensors (only exception is iBeacon broadcasting).


Ignoring Task entities with no step key

Change description
Some productions might be using tasks without SG task entity step key. This will change will not raise the missing key and therefore it will allow to sync tasks into project.
Testing steps:

Not sure how to reproduct such a workflow. Perhaps just create task in root of the project and sync it manually?

During manual sync I got these errors:
Traceback (most recent call last):
File "C:\Users\Mila\code\ayon-shotgrid\services\processor\processor\processor.py", line 243, in start_processing
handler.process_event(
File "C:\Users\Mila\code\ayon-shotgrid\services\processor\processor\handlers\sync_projects.py", line 34, in process_event
hub.synchronize_projects(source=sync_source)
File "C:\Users\Mila\code\ayon-shotgrid\services\shotgrid_common\ayon_shotgrid_hub_init_.py", line 277, in synchronize_projects
match_shotgrid_hierarchy_in_ayon(
File "C:\Users\Mila\code\ayon-shotgrid\services\shotgrid_common\ayon_shotgrid_hub\match_shotgrid_hierarchy_in_ayon.py", line 118, in match_shotgrid_hierarchy_in_ayon
ay_entity = _create_new_entity(
File "C:\Users\Mila\code\ayon-shotgrid\services\shotgrid_common\ayon_shotgrid_hub\match_shotgrid_hierarchy_in_ayon.py", line 259, in _create_new_entity
sg_ay_dict["task_type"],
KeyError: 'task_type'
2024-09-16 10:25:53.128 | processor | start_processing | INFO: Event has been processed... setting to finished!

is this ignoring them or actually syncing them? the title says it's ignoring but the description says:

This will change will not raise the missing key and therefore it will allow to sync tasks into project

What's the intended behaviour? if you don't set a task type on the Ayon entity what does it happen?

is this ignoring them or actually syncing them? the title says it's ignoring but the description says:

This will change will not raise the missing key and therefore it will allow to sync tasks into project

What's the intended behaviour? if you don't set a task type on the Ayon entity what does it happen?

Good point, @fabiaserra. It should ignore them and not produce an error message. We had a case where a client was using SG for development ticket submission. These tasks were in the CG project but didn't have any steps (task types).
There are two solutions:

Ignore them so they aren't synchronized into AYON at all.
Assign them a default task type that AYON offers.
Offer a settings toggle to choose between the two options mentioned above.

@fabiaserra or @robin-ynput, do you have an opinion on this?

There are two solutions:

Ignore them so they aren't synchronized into AYON at all.
Assign them a default task type that AYON offers.
Offer a settings toggle to choose between the two options mentioned above.

@fabiaserra or @robin-ynput, do you have an opinion on this?

Having the settings control is nice but I think most people are confused with the amount of settings control and it kind of beats the purpose. I think I would probably expect them to sync and use the task name for type in order to sync from SG to AYON and for AYON to SG if there's no step that exists for that type just not set the step when creating the task

is this ignoring them or actually syncing them? the title says it's ignoring but the description says:

This will change will not raise the missing key and therefore it will allow to sync tasks into project

What's the intended behaviour? if you don't set a task type on the Ayon entity what does it happen?

Good point, @fabiaserra. It should ignore them and not produce an error message. We had a case where a client was using SG for development ticket submission. These tasks were in the CG project but didn't have any steps (task types).
There are two solutions:

Ignore them so they aren't synchronized into AYON at all.
Assign them a default task type that AYON offers.
Offer a settings toggle to choose between the two options mentioned above.

@fabiaserra or @robin-ynput, do you have an opinion on this?

My understanding is that while Pipeline Step are optional in SG, Task Type are mandatory in AYON.
So I'd recommend to always sync tasks but either set them as Generic or a new Unknown type in AYON.

Yeah, if pipeline steps are optional in SG, we need to make a reasonable fallback choice. I like the unknown type actually.

Yeah, if pipeline steps are optional in SG, we need to make a reasonable fallback choice. I like the unknown type actually.

Here is the solution added to the current state of this PR. Settings are having new option under Compatibility Settings section for defining a default task type.


Unable to see log in button on iPad

The login buttons were not showing on my ipad. When I moved to my laptop I could see them.
per one of me end user

I'm not seeing login in buttons on mobile either.


fix(SelectTree): shoud set the default active node by Value on first render

shoud set the default active node by Value on first render
Summary of the changes (Less than 80 chars)
简单描述你更改了什么, 不超过80个字符；如果有关联 Issue 请在下方填写相关编号
Description
fixes #5001
Regression?

[ ] Yes
[ ] No

[If yes, specify the version the behavior has regressed from]
[是否影响老版本]
Risk

[ ] High
[ ] Medium
[ ] Low

[Justify the selection above]
Verification

[ ] Manual (required)
[ ] Automated

Packaging changes reviewed?

[ ] Yes
[ ] No
[ ] N/A

☑ Self Check before Merge
⚠ Please check all items below before review. ⚠

[ ] Doc is updated/provided or not needed
[ ] Demo is updated/provided or not needed
[ ] Merge the latest code from the main branch

Summary by Sourcery
Bug Fixes:

Fixes an issue where the active node in the SelectTree component was not set correctly on the initial render.

@sourcery-ai review


MSB4181 The "CompileXaml" task returned false but did not log an error.

Hello, i am building an app with Unity and Visual Studio 2019 for the HoloLens. The most recent feature that i added is the ability to connect the application to an Azure database. When I run it on Unity, everything works as expected, but when I try to compile the project in Visual Studio in order to upload it to the HoloLens, this is one of two errors that I am getting. The other one is "MSB4181 The "C1083 Cannot open include file: 'HolographicAppRemoting/StreamerHybridPlayer.h': No such file or directory"
Does anyone know how to solve it? Everything was working well until I added that database feature.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Jobs playerinfo doesn't work for unlearned archive jobs

*Detailed description of the issue:
The command /jobs playerinfo <player> <job> doesn't work for unlearned jobs: It used to show the earnings related to the /jobs archive level. Currently, the jobs archive level for the job is ignored and for unlearned professions the info for level 1 is always displayed.

Jobs version:
<IP_ADDRESS>
Server Type (Spigot/Paper/etc):
Paper
Server Version (using /ver):
Paper-368 (MC: 1.19.3)
Relevant plugins (Delete if this isn't needed):
<IP_ADDRESS>

Can't recall when it was showing stats from archived jobs, but well, I can add that.
e6c476f756ae6823bfab7ef31270fcc4ec8ec12a

Thanks! 🎉


android:uiOptions="splitActionBarWhenNarrow" in Activity not working

android:uiOptions="splitActionBarWhenNarrow" gives a split ActionBar when placed in the application node,
but fails to do so when placed in an Activity node.
Switching my app to use native ActionBar support confirms a split ActionBar is shown when placed in the Activity node.
The activity is a FragmentActivity that contains one fragment. This fragment adds the action items to the ActionBar.
I'm using ABS 4.0 Beta 5.

Can you paste your full manifest? Or at least the root tag, application tag, and the activity tag in question (and all their attributes)?

Just had a look into the ABS code and I can see what is happening.
The name attribute of my activity node is just my class name. ABS code is looking for the . at the start of the name to decide if it
is within our package. My name attribute didn't have the . and so it didn't use the uiOptions.
If I put a . at the start of the class in my activities node name then I get the split ActionBar.
Is this still a bug as the system allows unqualified class names in the activity's name attribute?

The activity name is supposed to be either a fully qualified class name or a relative name from your manifest package (hence the leading period). I suppose that we should also account for ones without it, though, since the Android OS does the same. I'll leave the bug open as a reminder.

I have implemented a simple fix for this locally.
Want me to fork and put in a pull request with it in?


[Dubbo-7054]消费者端Executor调整为“全局共享”、“按提供者隔离”2个可选模式

What is the purpose of the change
这是在 #7109  基础上进一步将消费者端Executor调整为“全局共享”、“按提供者隔离”2个可选模式，默认工作在全局共享，可以通过dubbo:consumer的share.threadpool进行设置。
<dubbo:consumer>
    <dubbo:parameter key="share.threadpool" value="false"/>
</dubbo:consumer>

备注：share.threadpool参数命名沿用自2.6.9
Brief changelog

如果share.threadpool=true，DefaultExecutorRepository在createExecutor时返回包装后的ShareableExecutor。
AbstractClient在close时，如果executor为ShareableExecutor这不处理executor.shutdown。

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #7120 (06ea259) into master (bd43fa7) will decrease coverage by 0.49%.
The diff coverage is 75.51%.

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

Codecov Report

Merging #7120 (06ea259) into master (bd43fa7) will decrease coverage by 0.49%.
The diff coverage is 75.51%.

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

duplicat of #7109


fix: infinite loading with invalid account route

Details

fix: infinite loading with invalid account route
Fixed Issues

$ https://github.com/Expensify/App/issues/20854
PROPOSAL: https://github.com/Expensify/App/issues/20854#issuecomment-1594532769
Tests

Login with any account
Open any profile route that has invalid route number, ie: https://staging.new.expensify.com/a/undefined
Verify that Profile Page doesn't have infinite loading.

[x] Verify that no errors appear in the JS console

Offline tests
Same as Tests.
QA Steps

Login with any account
Open any profile route that has invalid route number, ie: https://staging.new.expensify.com/a/undefined
Verify that Profile Page doesn't have infinite loading.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots

Web

https://github.com/Expensify/App/assets/16502320/57712bec-f75b-4917-80a6-d391fe4b3274
Mobile Web - Chrome

https://github.com/Expensify/App/assets/16502320/27913f31-2e86-41e3-b35b-266829cedd48
Mobile Web - Safari

https://github.com/Expensify/App/assets/16502320/83103c56-b546-4483-8800-4fdc6fd2dc19
Desktop

https://github.com/Expensify/App/assets/16502320/845af7c6-ff1e-42da-8aae-49ab9ecd8a4c
iOS

https://github.com/Expensify/App/assets/16502320/9808cb40-c7fe-41dd-a18d-cfcf3032f63e
Android

https://github.com/Expensify/App/assets/16502320/27913f31-2e86-41e3-b35b-266829cedd48

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/arosiclair in version: 1.3.32-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.3.32-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


fix(Keyboard): setting selectedIndex when creating rows

Description
setting the selectedIndex for the Row when creating a new keyboard on toggle
References
LUI-1320
Testing

migrate to a keyboard that has multiple formats (example: keyboard-qwerty)
click one of the keys that toggle a keyboard change (such as symbols, shift etc)
notice how the focus stays on that key

Automation

Checklist

[ ] all commented code has been removed
[ ] any new console issues have been resolved
[ ] code linter and formatter has been run
[ ] test coverage meets repo requirements
[ ] PR name matches the expected semantic-commit syntax

:tada: This PR is included in version @lightningjs/ui-components-v2.20.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Landscape View in Flip Page not working

Device in Landscape view images are not display.

Can you make it more clear? Using our sample to try or your own project? What happened? black screen or mess picture on screen?

Hello, @eschao!
Thank you for the wonderful page-flip implementation.
I wasn't able to run the sample in Android Studio, so I started a new androidx project, copied all the sample's sources, did the "implementation" of the required libraries in build.gradle.
Vertical orientation works fine. Horizontal - no, does not work.
Black screen. Overlaying page numbers. Samsung, Android 10.
But I didn't look through the code yet.

Comments following lines in SampleActivity is working for me
if (Build.VERSION.SDK_INT < 16) {
            getWindow().setFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN,
                    WindowManager.LayoutParams.FLAG_FULLSCREEN);
        } else {
            mPageFlipView.setSystemUiVisibility(
                    View.SYSTEM_UI_FLAG_FULLSCREEN |
                            View.SYSTEM_UI_FLAG_LAYOUT_STABLE |
                            View.SYSTEM_UI_FLAG_LAYOUT_HIDE_NAVIGATION |
                            View.SYSTEM_UI_FLAG_IMMERSIVE |
                            View.SYSTEM_UI_FLAG_HIDE_NAVIGATION);
        }


[FEATURE] Add a loader component to the website

Detailed description
The web page doesn't have a loader component when the data is being fetched from the backend.
Adding it will make the UI more appealing.
Context
All the websites have a loader component which makes the frontend more fascinating to the users and it represents that data is being fetched which takes some time.
Possible implementation
No response
Additional information
No response
Checklist

[X] I have read the Contributing Guidelines
[X] I have checked the existing issues
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

@abhinav-m22
I have assigned you the issue. Please proceed with it, and don't hesitate to tag me if you require any assistance.
Please make sure, you add a loader component with proper conventions and according to the theme of this repository.

@abhinav-m22
I'm closing this issue due to inactivity. Please feel free to reopen it if you'd like to contribute or have further discussions.


Wanted: GPS pseudo-attitude calculations

Anyone happen to have stumbled across code that does GPS pseudo-attitude calculations? Thus far, all I can find are academic papers on the subject and they can be tough to decipher.
GPS pseudo-attitude has been around for more than 15 years and is how Garmin and other vendors depict a "six pack" on a portable GPS.
I'd like to tinker with implementing a GPS based version of @bagarre Pi2D2.

Not sure if this will help as it's way over my head, but check out the source code they mention here:
http://diydrones.com/profiles/blogs/705844:BlogPost:16708
It uses AHRS as well as GPS and I think you're looking for an all-GPS solution, but this code is supposedly well documented so maybe it can give some ideas.

Thinking about the Garmin six-pack, we're halfway there.
Airspeed: Use GPS ground speed.
Altimeter: Use GPS altitude.
Heading indicator: Use GPS track.
For the rest of the instruments, turn coordinator and VSI are straightforward. Pseudo-attitude would be a function of those two. The challenge, if there is one, is ensuring that the GPS data is fast enough and noise-free enough to have useable inputs. I'll look at writing and testing an implementation this weekend.
Turn coordinator: Turn rate is time derivative of heading. GPS heading is stable and we want to react quickly to changes, so I'd calculate the slope of heading vs time over no more than 3-5 points... so two seconds (assuming 1 Hz GPS sampling), or the last second (if sample rate is 5 Hz or higher).
VSI: Vertical speed is time derivative of altitude. GPS altitude (especially non-WAAS) can drift a few meters around the actual altitude, so filtering would be needed, especially since this will be fed into the pseudo pitch calculation. I'd try to use at least 5, preferably 10 points depending on GPS refresh rate. Report only if GS is greater than 30 knots (?)
Attitude indicator - pitch: Approximate as the flight path, so this simplifies to atan(VS/GS) as long as vertical speed and ground speed are converted to the same units. Report only if GS is greater than 30 (?) knots.
Attitude indicator - roll: Assuming coordinated flight, turn radius is a function of true airspeed and bank angle. Rate of turn is a function of true airspeed and turn radius. I'd have to write out and substitute the equations / convert units, but this means bank angle should be be atan(c*turn rate * GS), where c is some constant I need to solve for. Report only if GS is greater than 30 (?) knots.

Proof of concept (along with a few other enhancements) : https://github.com/AvSquirrel/stratux/commit/bf8e3763e8df81dfbca4ca9f8921575ac0dd858a (Release notes)
Hope I don't have any showstopper bugs -- haven't had a chance to test in a moving vehicle, let alone in flight. Not planning to spin an image, but feel free to 'git clone' and give it a whirl.

It needs a little work, but this has promise.
Data below is an 11-second excerpt from a ground-based test (45-55 MPH on a curvy suburban highway). Reception was only fair, with 8 satellites in solution and no WAAS.
The raw output of simple dh/dt calculation (blue diamonds) is pretty bouncy at low speeds -- at 50 knots, it only takes about 0.4° apparent course change in 0.1 seconds to swing indicated roll by ± 10°. However, I've been toying with a couple of simple filtering algorithms that would greatly reduce this noise without introducing lag.
These can be seen as the four line fits through the diamonds. The thin black line is a 1-second moving average, which I'm considering to be a baseline. It has fairly good noise rejection (±2°, vs ±15-20° in the unfiltered data) during the left turn, but lags by about 0.5 seconds.
The gold line is a complementary filter. I'm ruling this one out since it takes a relatively low gain (0.1) to get noise below the level of the moving average filter, and this pushes the lag up to about 1 second behind the heading change.
Two methods of local linear regression both look very promising. These methods involve for each time t_i, find the slope and intercept of roll vs time over the past 1.0 seconds, then calculate filtered roll = m*t_1+b. The orange line takes this data, and applies a moving average filter to the last 0.3 seconds of regression-filtered roll. The green line ("double filtered") instead of applying a moving average filter, runs the local linear regression on the regression-filtered roll.
Both of these methods seem to have about as much noise as the 1-sec moving average filter, but they both converge almost instantly to changes in direction.

As soon as I implement a least squares method to calculate m and b, I'll roll this into the proof-of-concept code.

Impressive work! I would expect the more filtered result of the green line is still quite valuable and not misleading.

I'm close to a pull request on this. Adding locally-weighted linear regression (first iteratation: https://github.com/AvSquirrel/stratux/commit/68f8d5e88e9aa084dbefd8f5bd5ac9e4456b044a) really smoothed out the bouncing I saw on my January prototype. This is an excerpt from a 15-minute drive I took tonight. Even at 30-35 miles per hour, pitch and roll tracked smoothly and instantly to turns, and even tracked hill climbs and descents.

I've since made three other commits for cleanup, and to add support for SIRF (not tested yet). I also need to flight test this to validate the physics in the air. Latest and greatest here: https://github.com/AvSquirrel/stratux/tree/gpsattitude

What would you like for flight testing? I have a test-setup with GPS (no-AHRS). I'm currently running Avare on a Nexus 7 for testing.

I need to verify that the synthetic attitude matches reality. E.g.:

Climb at Vx and see a 8-12° pitch up
Descend in the pattern and see 3-4° pitch down
Roll angle matches what the wings are doing in coordinated turns across a wide range (from less than standard rate, up to 60° bank)
Output is smooth and responsive, with minimal lag or noise

I'm hoping to fly it at some point this week, but external feedback would be helpful.
I've already quashed one showstopper bug, and new commits submitted this evening to solve a random index out-of-bound panic. Commit a3f0c40b or later should be stable.

45-minute flight test performed wonderfully this evening -- standard rate, 30°, 45°, and 60° turns all indicated properly on WingX. Vy climbouts showed 8-10° nose up, and descents in the pattern were 3-5° nose down.
This excerpt shows performance through a pair of clearing turns, then a series of 360° turns. In sequence: 45° bank left, 45° bank right, 55-60° bank left, 55-60° bank right, and standard rate left.

Current commit is now 2f56c8a to incorporate a 1090 TIS-B bugfix. Update package: update-stratux-v0.8r1-2f56c8ac73.sh.zip

Pull request #340 submitted at commit AvSquirrel@f7a9479.
This version includes detection of GPS measurement rate and adapting the regression window to fit the sample rate. In other words, it filters / averages over a longer period of time if the GPS is reporting at 1 Hz than if it was reporting at 10 Hz.

I hope these results are not being passed to the EFB's since some of
them use their own smoothing methods.  Compounding errors and all that.

This version includes detection of GPS measurement rate and adapting
the regression window to fit the sample rate. In other words, it
filters / averages over a longer period of time if the GPS is
reporting at 1 Hz than if it was reporting at 10 Hz.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/136#issuecomment-198164702

@skypuppy, the method I used is basically a reverse-looking Lowess filter. Much less lag than a moving average filter of the same smoothness. Its responsiveness is similar to the green line in my plot from 1/17, and both ForeFlight and WingX indicate quick roll entries with a couple tenths of a second.
I really should put together a side-by-side video showing how this method, a Stratus 2, a vacuum AI, and the actual outside horizon all compare.

@bradanlane - try out http://updates.stratux.me/builds/update-stratux-v1.0r1-ahrs-ebfb671142.sh.
GPS attitude is calculated using @AvSquirrel's implementation automatically when a GPS device is detected. It's looking good.


Release artifacts for release v0.0.14

Description of changes:
Update release artifacts to version v0.0.14
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Snap Build 2.1.0

This issue is to change the version to 2.1.0 and build release binaries

We should look into Travis releasing to GitHub releases. Also note appveyor needs to be setup for Windows.

Let's open a new issue to introduce automated builds later. let's just get these done manually for now?

:+1:

Is this fork releasing some sort of binaries of at least documentation on how to build? Am I just missing it? :)

We will be releasing binaries "soon" (when someone has time to get auto deployment to work).
There are Linux build instructions at https://github.com/nylas-mail-lives/nylas-mail/blob/master/docs/guide/contributing/installation.md
We don't currently have documentation on building on Windows or Mac (the documentation just hasn't been written yet)

Great! I might want to contribute, for instance with some auto deployment and it infrastructure, and then move on to backend/frontend, how do I sign up? ;)

I would start by forking this repository and finding an issue https://github.com/nylas-mail-lives/nylas-mail/issues that needs a contributor and work on that. If you work on anything related to the client (99% of it), you'll need to get the code to compile and run before making changes. https://github.com/nylas-mail-lives/nylas-mail/issues/40 may be something you may help contribute to.
I would also get on Slack; you can find the invite url on the README so you can ask these kinds of questions instead of here on the issues

https://github.com/nylas-mail-lives/nylas-mail/releases/tag/2.1.0


Allow unused imports

These don't seem to actually be necessary (as the mod semver and mod sha2 will use them when these features are enabled)?

Those are re-exports. Tey are there to reduce the length of the import path - when the sha2 feature is enabled, all the required trait implementations will be in scope without requiring users to add use inventory::sha2.

Hmm interesting -- however, I was using the library without those re-exports (and using the features that'd require trait implementations) without any compile errors. Not sure exactly who though?
I've pushed a new commit to allow these (and make the linter happy), and renamed the PR accordingly.

Curious, lets merge for now to make the linter happy and investigate seperately. :) Thanks!


BLE Fuel data parsing

Hi Man,
when the devices sends fuel data from a BLE Sensor( AVL IO ID : 270 ) it send it as 0x0000 yet when i connect the same device to another platform it display the actual fuel litres, clearly the issue is with how the script is parsing the fuel data.
according to teltonika the type of data for this AVL IO ID 270 is Unsigned meaning data should be a positive int.
any ideas ?

Hello again,
AVL 270 is not being parsed at all by the script, it is sent as RAW hex value. You can check from line 485 how many elements have functions and you are free to add more.
There are too many different AVLs to add and there is no reason to add them all.
If you have Raw codec 8e packet which according to you has avl 270 value but script does take it, please provide it to me and I will check.
If codec structure is being parsed incorrectly then I will fix it.
Thank you


Load the test dataset locally

Hello, thanks for your excellent work!   I'd love to try your work.
However, I can't access kaggle or Huggingface from my server because of network problems. I can only download the dataset locally first.
How do I load the dataset locally instead of downloading it from the network? Do I need to process this dataset? For example, extract the answers in a certain format. Question equivalence?
If possible, can you give a process and method that can be referenced?
Thank you very much for your help!

Hello,
If I understand correctly, you can manually copy files to your server but you can't download from the internet. MultiMedEval will skip downloading from the internet if the datasets are already present on the specified path. Here are suggestions to get the files locally for Kaggle and for Huggingface.
Manual download for Kaggle:
The Kaggle API that I use simply downloads and unzips the dataset, you can do the same manually by going to the page of the dataset.
Once you have the zip file you can add it to your server and unzip it. You can then specify the correct path in the SetupParams.
Manual download for Huggingface
For Huggingface, it is not as straightforward because the API does some processing on the data. I would suggest loading the datasets by running MultiMedEval locally (with the following code for example) and then uploading the folder where the datasets are stored (in this case data) to your server. You can then specify the appropriate path on your server and HF will skip the network call.
from multimedeval import MultiMedEval, SetupParams
engine = MultiMedEval()

setupParams = SetupParams(MedQA_dir="data/")
tasksReady = engine.setup(setupParams=setupParams)

Note that this solution would also work for the Kaggle dataset.
I hope this helps.

Thank you very much for your timely reply!
But I also have a question.
Do you mean to initialize the local path here ?

which I screenshot from the .SetupParams.

Roger that! Thank you very much for your help!
Besides, I am working on multimodal medical treatment, and I hope to cooperate with you if I have the opportunity. A reliable, effective and universal evaluation framework is what I need most now!
Finally, congratulations on such an amazing job!

Thank you very much, I'm glad you appreciate it.
Don't hesitate to contact me via email :)


Rearrange the order/styles of buttons

Signed-off-by: harkiratsm<EMAIL_ADDRESS>Description
This PR fixes #5912
Notes for Reviewers

Signed commits

[x] Yes, I signed my commits.

😆


[Doc] How to achieve SNAT

This is a [Help Needed] first, and maybe a [Doc Enhancement] later.
as we know, native LVS will not do SNAT,  but only DNAT.
Usually, we will have to set the default route to VIP from realserver .
We use another tricky way (iptables -t nat -I POSTROUTING -d <IP_ADDRESS>/0   -j MASQUERADE)
But handling SNAT  seems not being mentioned in kube-keepalived-vip documents any where.
Did I miss anything?
if not , who is not familiar with LVS may suffer from question like why my vip:port is not achievable outside LVS node..

I found we enable net.ipv4.vs.conntrack=1 in the code,
but when running a native keepalived service(yum install then service keepalived start), this kernel parameter is not required.
This also confuses me .

conntrack=1 is required if we use MASQUERADE to achieve SNAT.
But not catching what's the recommendation way  kube-keepalived-vip officially suggest to achieve SNAT .


smoke: support node image

Relevant Issue (if applicable)
Part of #1460.
Details
Add test for node:19.8 in benchmark.
Types of changes
What types of changes does your PullRequest introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Go over all the following points, and put an x in all the boxes that apply.

[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

The nydus-integration-test error always happens, I think it is not caused by this pr. cc @imeoer


Update memory.hpp

Updated for patch 14.6

is this a fix for the bug thats been going on lately?

how to put it?

sexy

Updated for patch 14.6

can you tell us how to swap the memory files with renz's code
cause ive been trying for 30 minutes and i cant figure it out

update for 14.8


Move free functions in fake_etcd.cc to the anonymous namespace.

Rationale for this, lest people think I'm a bit odd: I'm doing big refactoring in there, and I'd like the compiler to tell me when something isn't used anymore.


WHMCS Password not getting passed to Pterodactyl

Hello there!
While troubleshooting with a colleague, I noticed the Add-On doesn't really pass the password generated by WHMCS onto the Pterodactyl panel. We fixed this by simply adding this line:
'password' => $params['password']

to the following code block:
if($userResult['meta']['pagination']['total'] === 0) {
                $userResult = pterodactyl_API($params, 'users', [
                    'username' => pterodactyl_GetOption($params, 'username', pterodactyl_GenerateUsername()),
                    'email' => $params['clientsdetails']['email'],
                    'first_name' => $params['clientsdetails']['firstname'],
                    'last_name' => $params['clientsdetails']['lastname'],
                    'external_id' => $params['clientsdetails']['id'],
                    // Added the line here
                ], 'POST');
            }

Now, this seemed like a very easy and trivial fix to a very complex situation, so I'd like to know if this isn't already featured in the source code because of a security issue or if it was simply a mistake.
Best regards, keep up the awesome work ^^.

Please fill out the information when you submit an issue. With no information on what versions you are running then we cannot identify where the actual issue is.
I will be closing this out now.

Or I read this and thought it was for pterodactyl and not whmcs. I apologize.

You're not suppose to be-able to pass the password.
The user password in WHMCS is hashed & salted, and having it plain text transfer over to pterodactyl will be a massive security issue.

@Lunaversitay From what I can tell, it is not the client's password. It's the per order password where it autogenerates a random one.
Regardless it is at best to leave the password to be made through Pterodactyl's "Setup your Account" page instead of having WHMCS create a random password.

iirc, Pterodactyl sends the user an email when the account is created for user to create their own password.

Here's the problem with this guys. We don't want to sit there and send the clients 20 emails before they can access their account. The client needs to be able to access all the data needed for their server in one email (one we can easily modify as needed). If we shouldn't be passing the password variable over... then how do we get the server to respond back with the link the client can use to create their password?

In addition, if the data is encrpyted (as it should be when communicating with the Ptera API), why should this be a security risk passing the password over? Plenty of other modules pass the password over...

iirc, Pterodactyl sends the user an email when the account is created for user to create their own password.

Yes, pterodactyl sends an email and the user sets up their account that way.

In addition, if the data is encrpyted (as it should be when communicating with the Ptera API), why should this be a security risk passing the password over? Plenty of other modules pass the password over...

lol what app transfers a users' password to another app? I haven't heard of something like that because things like oauth exist. Also it doesn't send "20 emails" it sends 1 email for the client to setup their account on pterodactyl and that's the end of it.

Isn't it just best to allow the user to set their own password in the first place? As I'm sure 80% of people change their password from the randomly generated ones anyways. One less  stop by the end user.

iirc, Pterodactyl sends the user an email when the account is created for user to create their own password.

Yes, pterodactyl sends an email and the user sets up their account that way.

In addition, if the data is encrpyted (as it should be when communicating with the Ptera API), why should this be a security risk passing the password over? Plenty of other modules pass the password over...

lol what app transfers a users' password to another app? I haven't heard of something like that because things like oauth exist. Also it doesn't send "20 emails" it sends 1 email for the client to setup their account on pterodactyl and that's the end of it.
(also side note: the passwords are not encrypted they're hashed & salted - two very different things)

Tell that to the people who developed the WHMCS CPanel and Virtualmin module lol. I don't disagree with the idea of having a link the user can click to change the password. We just feel like it could be implemented better. Maybe there is a way to include the link in the welcome email WHMCS sends? At the end of the day, we are going to integrate the panel with OpenID so the user has one set of login credentials for all the apps... just trying to see if there is something that could help those who aren't going to be using OpenID in the future.

Hi,
This behavior is intentional.
The password and username fields are not used by the module because individual servers don't have a username or password. For the same reason changing password is also disabled.
Even if I had the username and password of the actual account, it would be useless as I don't have any way to track it e.g. if the user changes their username/password. This also would be a security risk as passwords shouldn't be stored in plaintext just to allow this.
Pterodactyl handles getting password to the customer and won't cause major confusion to the customer when he receives a password to his account that isn't valid (due to them already having an account before, leaving them with 2 different service passwords) or getting it changed just to have a valid password sent with the service activated email.

iirc, Pterodactyl sends the user an email when the account is created for user to create their own password.

Yes, pterodactyl sends an email and the user sets up their account that way.

In addition, if the data is encrpyted (as it should be when communicating with the Ptera API), why should this be a security risk passing the password over? Plenty of other modules pass the password over...

lol what app transfers a users' password to another app? I haven't heard of something like that because things like oauth exist. Also it doesn't send "20 emails" it sends 1 email for the client to setup their account on pterodactyl and that's the end of it.
(also side note: the passwords are not encrypted they're hashed & salted - two very different things)

Tell that to the people who developed the WHMCS CPanel and Virtualmin module lol. I don't disagree with the idea of having a link the user can click to change the password. We just feel like it could be implemented better. Maybe there is a way to include the link in the welcome email WHMCS sends? At the end of the day, we are going to integrate the panel with OpenID so the user has one set of login credentials for all the apps... just trying to see if there is something that could help those who aren't going to be using OpenID in the future.

None of those apps send passwords to another app? Implementing WHMCS SSO would require editing the pterodactyl panel, while it's 100% possible (you can check wisp.gg for example) I don't believe it'll be something natively added to Pterodactyl because they want to be as agnostic as possible.
I really don't see the problem either with having an email to setup the panel, you only get the email once and once your account is setup you never have to set it up again.

iirc, Pterodactyl sends the user an email when the account is created for user to create their own password.

Yes, pterodactyl sends an email and the user sets up their account that way.

In addition, if the data is encrpyted (as it should be when communicating with the Ptera API), why should this be a security risk passing the password over? Plenty of other modules pass the password over...

lol what app transfers a users' password to another app? I haven't heard of something like that because things like oauth exist. Also it doesn't send "20 emails" it sends 1 email for the client to setup their account on pterodactyl and that's the end of it.
(also side note: the passwords are not encrypted they're hashed & salted - two very different things)

Tell that to the people who developed the WHMCS CPanel and Virtualmin module lol. I don't disagree with the idea of having a link the user can click to change the password. We just feel like it could be implemented better. Maybe there is a way to include the link in the welcome email WHMCS sends? At the end of the day, we are going to integrate the panel with OpenID so the user has one set of login credentials for all the apps... just trying to see if there is something that could help those who aren't going to be using OpenID in the future.

None of those apps send passwords to another app? Implementing WHMCS SSO would require editing the pterodactyl panel, while it's 100% possible (you can check wisp.gg for example) I don't believe it'll be something natively added to Pterodactyl because they want to be as agnostic as possible.
I really don't see the problem either with having an email to setup the panel, you only get the email once and once your account is setup you never have to set it up again.

If sending the passwords over is a security risk, then we will not do that. If it means the user has to read two emails for now, we are fine with that. One extra email shouldn't hurt :) Appreciate the time you guys took to look over this. Thank you!


Upgrade Linkerd to 2.13.6

Last version of Linkerd includes some perf regressions introduced in 2.13.
It'd be nice to upgrade soon.
https://github.com/linkerd/linkerd2/releases/tag/stable-2.13.6

all done.


Never loads on the M preview

Crashes on load on the M Dev preview update. Running this on a Nexus 6. Screen starts black and then crashes.

@Hixie was using an M preview for a while.  I'm not sure if he was using the official Dev preview or not.

@eseidelGoogle wish I had more info for you but I'm used to my chrome Dev tools (; not so much on debugging android apps.

I wonder if this is fixed by the new way of creating the GL interface for Skia.  We should try again after the next release.

@jgw96 Would you be willing to try this again now that Sky 0.0.24 has been released?  You should be able to upgrade your version of Sky by running pub upgrade in the same directory as your pubspec.yaml file.

@abarth sorry I may have submitted this issue in the wrong repository. I encountered this issue when trying the sky demo app available in the play store. I don't have sky setup on any of my machines. Although I could if needed?

Ah, that makes sense.  We should update the Sky Demo app in the play store.  We'll do that today.  Thanks!

@abarth. No problem! Love the work you guys are doing! We use a lot of phonegap at work but I would love to be able to use something like sky instead.

@eseidelGoogle published a new version of the APK today.  It might be worth trying again with v21.

I believe this will be fixed in v22 by 3e7ab3a338b413bdde1ea2d7cf68f869106f1520

@abarth currently downloading to test, sorry have been very busy lately.

No worries.  We haven't published the version with the fix yet.  We'll probably publish it later this week.


network policy type: ipBlock not working

Creating ipBlock of type ipv4 or ipv6 does not work because it tries to create an ipset of both ip address.
When creating an ipBlock with an ipv4 range, the error is:
Aborting sync. Failed to sync network policy chains: failed to perform ipset restore: ipset v7.6: Error in line 9: Syntax error: cannot parse <IP_ADDRESS>: resolving to IPv6 address failed
When creating an ipBlock of an ipv6 range (/126), the error is:
Aborting sync. Failed to sync network policy chains: failed to perform ipset restore: ipset v7.6: Error in line 10: Syntax error: '126' is out of range 0-32

But I will keep this issue open just in case

Still same error:
Mar 15 17:36:17 ip-10-0-10-40 k3s[678123]: E0315 17:36:17.837424  678123 network_policy_controller.go:253] Aborting sync. Failed to sync network policy chains: failed to perform ipset restore: ipset v7.6: Error in line 10: Syntax error: cannot parse <IP_ADDRESS>: resolving to IPv6 address failed

This is what I tried:
https://github.com/manuelbuil/PoCs/blob/main/2021/ipv6-poc/httpd-ipv6.yaml
And the network policy:
https://github.com/manuelbuil/PoCs/blob/main/2021/network_policy_podSelector-httpd-ipBlock.yml

It's fixed now


not compatible with the current version of VSCode (version 1.65.0).

ERR Can't install 'antonreshetov.masscode-assistant' extension because it is not compatible with the current version of Visual Studio Code (version 1.65.0).: Incompatible: Can't install 'antonreshetov.masscode-assistant' extension because it is not compatible with the current version of Visual Studio Code (version 1.65.0).
    at U.checkAndGetCompatibleVersion (vscode-file://vscode-app/Applications/Visual%20Studio%20Code.app/Contents/Resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:48:12147)
    at async U.installFromGallery (vscode-file://vscode-app/Applications/Visual%20Studio%20Code.app/Contents/Resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:48:3200)

Because min version is 1.66.0 https://github.com/massCodeIO/assistant-vscode/blob/master/package.json#L7


Command-line tool?

readme.md doesn't seem to suggest that there's a command line tool built on top of this library, but I'd love to have one. Is it not implemented or just not mentioned in the readme?

There is no command line tool, and as far as I know there are no plans to add one. It should however be relatively straightforward to write your own tool using this library.


fix: add shell: true to spawn options to add Windows support in duel.js (#48)

Hi @davidensinger. I don't have a windows machine so am curious if this fix is working for you? I've got a similar thing in place on this branch but the workflow is failing on the windows runner: https://github.com/knightedcodemonkey/duel/actions/runs/11113719373/job/30878672727

I've actually opened up #52 to address this. If you want to take a look at that for me that would be great. I skipped the test for running as a CLI on windows. I think it has to do with quotes around the args or file permissions or both. Not sure since I don't have a windows machine at my disposal.

Windows should be supported in v2.0.0-rc.2.


feat(controller): Use S3 for node offload + workflow archiving. POC

Signed-off-by: Alex Collins<EMAIL_ADDRESS>Checklist:

[x] My organization is added to USERS.md.

Testing Instructions
You need to run argoproj/workflow-controller:off-poc with additional configuration:
      # Storage: s3, mysql, or postgres maybe configured.
      #
      # If mulitple are configured then the following rules are used:
      #
      # 1. For node status offloading, S3 is most preference, for archiving, least prefered.
      # 2. MySQL is prefered to PostgreSQL
      #
      # S3 is most prefered (and therefore implicitly recommeded) as it can be expected to be faster,
      # and relies less on the querying capability of an SQL database.
      #
      # S3 is least prefered for workflow archiving. While is is easier to set-up that an SQL database, S3 does not
      # support (out of the box) indexed backed (i.e. fast) querying capabilities for listing archived workflows.
      # List operations will be slow when any significant number of workflows are archived.
      # Unlike MySQL or Postgres, S3 enforces TTLs by setting them on the bucket.
      # You MUST configure different buckets for the archive and offloads if you wish to TTL archived workflows.
      s3: |
        nodeStatusOffload: # ... as artifactRepository.s3
        archive: # ... as artifactRepository.s3

See #4560

PR fail is due to crdgen.sh. Passes locally.

Closing PR so it does not distract me.


backmerge 16/12/24

Follow root level naming convention (https://github.com/scaffold-eth/scaffold-eth-2/pull/1006)
Fix useScaffoldEventHistory duplicated events (https://github.com/scaffold-eth/scaffold-eth-2/pull/1014)
feat: disable vercel telemetry (https://github.com/scaffold-eth/scaffold-eth-2/pull/1012)
Optional chainId config in Scaffold hooks (https://github.com/scaffold-eth/scaffold-eth-2/pull/931)
Foundry improvements (https://github.com/scaffold-eth/scaffold-eth-2/pull/1011)
make useScaffoldWriteContract & useDeployedContractInfo backward compatible (https://github.com/scaffold-eth/scaffold-eth-2/pull/1015)
fix: move warnings to useEffect (https://github.com/scaffold-eth/scaffold-eth-2/pull/1016)

To test:

Clone create-eth repo
Switch to this branch:

git pull && git switch backmerge-16-12-24

Start the dev server:

yarn dev

Test instance:

yarn cli

This one looks good to me too 🙏:

make useScaffoldWriteContract & useDeployedContractInfo backward compatible (https://github.com/scaffold-eth/scaffold-eth-2/pull/1015)

Merging this thanks all!


Adds Dependency Injection to WebAPI Action Filters using Property Injection

Summary
This adds property injection for WebAPI filters using the current Dependency Injection model. It mimics similar techniques that are common in the Unity Dependency Injection library by creating a new BuildUp() extension method to manage the Property Injection.
Why
Suppose you have an ActionFilter that decorates one of your WebAPI methods and you need to leverage dependency injection to access a specific service. In ASP.NET WebAPI you can't use constructor injection due to how the filters are generated in the pipeline. It is considered a best practice to use Property Injection instead of the standard constructor injection.
This can be dangerous for us to adopt into the platform as I can see property injection and the BuildUp() method being abused. I would like to take a closer look at the AttributeUsage and see if we can tightly-couple the [Dependency] attribute just to ActionFilter. I do not want and do not think we should support property injection in DNN, this is a discussion I have had with @mitchelsellers and I am not trying to bring it back up with this PR. I am using property injection here as it appears to be required to get Dependency Injection support in ActionFilters
Example
public class MyCustomFilter : ActionFilterAttribute
{
    [Dependency]
    public IMyCustomService MyCustomService { get; set; }

    public override void OnActionExecuted(HttpActionExecutedContext actionExecutedContext)
    {
        // When this method executes MyCustomService will be injected and ready for use
    }
}

Steps to Merge

Task
Status

Unit Tests
✔ Success

Test Installer
✔ Success

XML Documentation
✔ Success

@mitchelsellers Thanks for the review, do you have any opinions on performance implications of this PR?

@mitchelsellers Thanks for the review, do you have any opinions on performance implications of this PR?

At this point in time, nothing concrete necessarily.  There would be some impact, but traditionally Action Filters will not have many members/properties so it shouldn't be too big of a deal.
I think I'd want to try and profile this one

My biggest concern is action filters that run before or during a request. If I remember correctly from some enterprise projects I worked on years ago a blocking thread through cause performance issues with the ApiController.
@mitchelsellers

I think I'd want to try and profile this one

I agree and I think we should test the following items prior to accepting this PR

Performance Profile
DotNetBenchmarking Results (if they are easy to obtain)
Memory Leak Analysis - I don't want another DI memory leak to come up when we should be able to load test it prior to merging

Are we in agreement that we should move forward and turn this PR into a mergable PR?

I made changes as requested in the initial review and fixed the failing unit tests. This should generate a build for us to take a closer look at and see how and if this all works. After investigating the unit tests I think we can add quite a few unit tests around this new code to ensure it is thoroughly tested. I would like to do that prior to merging it

@ahoefling is our current status that we're waiting on unit tests for this one?  Should I mark this as On Hold until those come in?

Correct we are waiting on a few things for this PR

Unit Tests
Sample Module
Test PR installer

I haven't had a chance to work on those yet. If we want to get this in for 9.8.0 I can find time tonight

I have updated the accessor of the BuildUp API to internal and added unit tests.
If there is no other feedback on this, I think it is good to go

@bdukes

I believe this will be in 9.8.1 or 9.9.0, since 9.8.0 RC has already started

Thanks for the update on when this will be merged. If the maintainers are happy with how this PR is looking I am thinking of starting a 2nd PR for MVC filters.

:+1: go ahead on the PR for MVC filters


JS slugify (with unicode enabled) leaves invalid characters in the slug

Issue Summary
The JS slugify function (used when cleaning a manually-entered slug) fails to strip out certain spacer/combining characters that are disallowed by Django slug validation.
Steps to Reproduce

On a site with WAGTAIL_ALLOW_UNICODE_SLUGS = True, create a new page
Paste the text "উইকিপিডিয়ায় স্বাগতম!" into the slug field
Defocus the field; it will be rewritten to "উইকিপিডিয়ায়-স্বাগতম"
Attempt to save the page; this returns the error "Enter a valid “slug” consisting of Unicode letters, numbers, underscores, or hyphens."

The expected behaviour is that the browser-side slug cleanup step should result in a slug that is accepted by the Django validation. However, the output contains the character U+09BF which is disallowed, being a spacing combining character.
Note that the bug only arises for slugs entered manually into the slug field; pasting "উইকিপিডিয়ায় স্বাগতম!" into the title field results in the slug "উইকপডযয-সবগতম" which is valid. This is because the title-to-slug conversion uses a more complex function which also performs transliteration when unicode slugs are disabled (which wouldn't be appropriate for manually-entered slugs); however, the simplified function used for cleaning manually-entered slugs is overly simple:
https://github.com/wagtail/wagtail/blob/84b3bf70349171e44f283085f1d89ccd9deed40e/client/src/utils/slugify.ts#L10-L13
since it's only stripping out a designated short list of punctuation characters, where it would be better to strip out everything that is not classed as a letter/number or explicitly allowed punctuation (just hyphen and underscore I think). Would be useful to compare this against the behaviour of the more advanced title-to-slug conversion, and Django's django.utils.text.slugify.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.12.4.
Django version: 5.1.1
Wagtail version: 6.3a0
Browser version: Chrome 128

Working on this

Anyone can contribute to this. View our contributing guidelines, add a comment to the issue once you’re ready to start.

Hello, I am new contributor and I want to contribute to it.

Hey @gasman could you review the latest PR and tell me where it needs improvement.

@Atif2077  This is one of my model, In here how can i use slugify functions :class WebStory(Page):
alternative_title = models.CharField(
    max_length=100,
    blank=True,
    null=True,
    help_text='Overwrites the default title',
)
credit = models.CharField(
    max_length=50,
    blank=True,
    null=True,
    help_text="Give Credit of where those photos are collected",
)
story_thumbnail = models.ForeignKey(
    "mediamodels.CustomImage",
    blank=False,
    null=True,
    related_name="+",
    on_delete=models.SET_NULL,
    help_text="Use full screen, portrait images with good resolution (720w x 1280h)",
)
watermark = models.ImageField(blank=True, help_text='For Ads only 1200*100 PNG')
content = StreamField(

    [

        ("richtext", blocks.SimpleRichtextBlock()),
        ("web_story", blocks.WebStoryBlock()),

    ],
    null=True,
    blank=True,
    use_json_field=True
)

categories = ParentalManyToManyField("blog.BlogCategory", blank=True)
subcategories = ParentalManyToManyField("blog.SubCategory", blank=True)
author = models.ForeignKey("blog.BlogAuthor", blank=True,
                           null=True, related_name="+", on_delete=models.SET_NULL,)

# Defining a function to get random id every call

tags = ClusterTaggableManager(through='tags.TaggedWebStory', blank=True)

def get_form(self, request, *args, **kwargs):
    form = super().get_form(request, *args, **kwargs)
    # Adding data-parent-category attribute to subcategory options
    categories = self.categories.all()
    subcategories = SubCategory.objects.all()
    for subcategory in subcategories:
        form.fields['subcategories'].widget.choices.queryset = subcategories.filter(parent__in=categories)
        form.fields['subcategories'].widget.attrs.update({
            'data-parent-category': subcategory.parent.id
        })
    return form
content_panels = Page.content_panels + [
    FieldPanel("alternative_title"),
    FieldPanel("story_thumbnail"),
    InlinePanel('web_gallery_images', label="Web Gallery images"),
    FieldPanel("content"),

    FieldPanel('tags'),

    MultiFieldPanel(
        [
            FieldRowPanel(
                [
                    FieldPanel("categories", widget=forms.CheckboxSelectMultiple),
                    FieldPanel("subcategories", widget=forms.CheckboxSelectMultiple),
                ]
            ),
        ],
    heading="Categories"
    ),

    MultiFieldPanel(
        [
            FieldPanel("author"),

        ],
        heading="Author"
    ),
    FieldPanel("watermark"),
]
subpage_types = []

COMMON_PANELS = (
    FieldPanel('slug'),

    FieldPanel('seo_title'),
    FieldPanel('search_description'),

)
promote_panels = [
    MultiFieldPanel(COMMON_PANELS, heading="Common page configuration")
]

Hi @MdShohoan - please use Stack Overflow or the #support channel in the Wagtail Slack for support queries.

@gasman this bug will fixed in wagtail next update?

@MdShohoan Yes, this fix will be in Wagtail 6.3 (due for release at the start of November).


feat: templating batch 1

Adds an initial master template and templates for Homeseer and HomePro devices.

@AlCalzone Lint doesn't like the master template. It seems to be ignoring the ../ Thoughts?

@AlCalzone Fixed and ready to go


Make RSA_generate_multi_prime_key() not segfault if e is NULL.

This is not a big problem for higher level keygen, as these set e beforehand to a default value. But the logic at the lower level is incorrect since it was doing a NULL check in one place but then segfaulting during a later BN_copy().

Checklist

[ ] documentation is added or updated
[x] tests are added or updated

Merged to master. Thank you for your contribution!


default Docker image is not working.

When trying to spin-up the default provided container, this one seems not to start.

@azurekid thank you for the bugreport. I'm looking into it.

Please let me know if I can provide you more information 🙏🏼

It was actually quite easy to reproduce and fix. Thanks again for reporting.
I've just pushed 0.3.0a3 with the fix and I'm awaiting the dockerhub build now to verify that all is well.

Fix confirmed. @azurekid you'll probably have to run docker pull eclecticiq/opentaxii to get docker to pull the latest image.


Update: Navigator injected on TvShowCatalogPresenter

Navigator is now injected into the TvShowCatalogFragmentPresenter.  The presenter's method "onTvShowThumbnailClicked" is now using navigator to move the user into the corresponding tvshow details.
All the navigation logic has been moved into the Navigator, even the fragment related one. The main problem here is that you need to make Navigator know everything about potentially available Fragments in the current injected Activity context.
Aproach PR made for discussion!

As you says, now your Navigator should know about your fragments implementation details. But this is not a problem!
Think about navigation, inside your system the navigation should be implemented like a boundary. To improve this code Navigator should be an interface and be implemented by TvShowsNavigator or AndroidNavigator or whatever you want for your app. Implementation details of the same boundary, the android application boundary in your case, can be coupled.
To me this code is an improvement :) Thanks @JorgeCastilloPrz. Next time, take care about the code style :P


Hello

thanks your good code
but running bug

What is your cmake version? I just realized our use FetchContent_Declare depends on cmake 3.24 features. Please follow instructions here: https://askubuntu.com/questions/355565/how-do-i-install-the-latest-version-of-cmake-from-the-command-line to install latest cmake. We'll edit the README to reflect this requirement

@Hs293Go
Thank you for your response. I am planning to pass the path generated from https://github.com/uzh-rpg/agile_autonomy to fsc-autopilot to operate the aircraft. I would appreciate it if you could let me know if there will be any issues.
Let me know if you'd like any adjustments!

If you are working with RPG code, be careful of its convention quirks, aka they use [w, x, y, z] quaternions but we use [x, y, z, w] quaternions in compliance with Eigen and scipy.spatial.transform.Rotation. For planning, I'd refer you to my colleague Chao Qin's award-winning work - https://github.com/FSC-Lab/TOGT-Planner. He is actively developing his code to make it compatible with our autopilot.


Default button type submit and not button

Describe the bug
The docs stated that the Buttons default native type should be button, but it's actually submit. The doc was updated, but the default should still be button.
https://nextui.org/docs/components/button#button-props

Thanks for reporting it @Keyruu 🙌🏻

Is it ok if I do the fix for this issue?

For sure @TokiyaHorikawa


Build error. Maybe because of missing tap

julia> Pkg.build("Homebrew")
INFO: Building Homebrew
WARNING: Could not tap staticfloat/juliadeps!
==============================[ ERROR: Homebrew ]===============================

LoadError: LoadError: pipe is already initialized
while loading /Users/andreasnoack/.julia/v0.5/Homebrew/src/Homebrew.jl, in expression starting on line 353
while loading /Users/andreasnoack/.julia/v0.5/Homebrew/deps/build.jl, in expression starting on line 1

================================================================================
ERROR: Build process failed.
 [inlined code] from error.jl:21
 in build! at pkg/entry.jl:515
 [inlined code] from dict.jl:469
 in build at pkg/entry.jl:534
 [inlined code] from promotion.jl:230
 in anonymous at pkg/dir.jl:31
 in cd at file.jl:22
 in cd at pkg/dir.jl:31
 [inlined code] from essentials.jl:116
 in build at pkg.jl:52

Ping @staticfloat

I have a suspicion this was caused by https://github.com/JuliaLang/julia/pull/13506, can someone with a mac bisect and confirn?

I'm unable to reproduce this.  I also tried rm -rf ~/.julia/v0.5/Homebrew/deps/usr/ and Pkg.build("Homebrew") and it worked fine (although it did give a deprecation warning about WARNING: src::AbstractCmd |> dest::AbstractCmd is deprecated, use pipeline(src,dest) instead).  I also tried checking out the master version of Homebrew and it worked fine.  What am I missing?

I'm seeing this too:

So for some reason after a failed build with homebrew if I do
Pkg.build("Homebrew")
Pkg.build("PackageThatFailedToInstall")

Everything goes through and works without a problem


Setting 'tag_keys' dynamically

In the work that I am doing, I do not know in advance the tags to be used in tag_keys. For every packet that is received I can look at the data and decide what the tags could be. Is there any way that I can set the 'tag_keys' per record that I am will be sending to influxDB?
For example, I could get a packet which could transform to {'interface-name': 'abc', 'interface-bandwidth': 1234, }. For this the tag should be 'interface-name'. Next I could get a packet which could transform to {'arp-ip': '<IP_ADDRESS>' and 'arp-timeout': 34}. Here the index should be 'arp-ip'. All of the packets are coming at the same time, so the timestamp would be the same and these should of to the same measurement. Above records are just example to simplify the use case.

:+1:
I would like to dynamically set tags, too. It would be possible by combining rewrite_tag_filter and forest plugins but it is tough. The auto_tags option can work sometimes but I feel it is hard-to-use because I need to carefully choose fields and to check a type of filed is not changed by passing a record thorough some plugins.
A simple solution would be adding an option like "tag_key" (confusing name though...) and then post the value of the "tag_key" field as tags.
For example, if tag_key is set to tags, you can send records like:
{ 'interface-bandwidth': 1234, 'tags': { 'interface-name': 'abc' } }
{ 'arp-timeout': 34, 'tags': { 'arp-ip': '<IP_ADDRESS>' } }

yeah, saidie's way is one approach.
Fluentd can't judge which field is tag or not because Fluentd is not human.
So need parameter or rule for dynamic tags.

Or if we send like below, can we use the 'tag_keys' in the message?
{ 'interface-bandwidth': 1234, 'tag_keys': [ 'interface-name' ] }
{ 'arp-timeout': 34, 'tag keys': [ 'arp-ip'] }
The code modification is minimum and has backward compatibility as well.
Downside is 'tag_keys' will become a reserved key.

👍 for @gaddevijay 's proposal.
An user defined tag_keys field contains a list of fields need to be tagged for current record, is pretty practical.

Maybe, adding tag_keys_field parameter is better for me.


Filamesh always using 16 bit index buffer

Describe the bug
The filamesh MeshWriter seems to use a 16 bit index buffer even if there are more than 2^16 vertices.
This is probably related to the following check in MeshWriter.cpp
bool MeshWriter::serialize(ostream& out, Mesh& mesh) {
    const uint32_t maxint = numeric_limits<uint32_t>::max();
    const bool hasIndex16 = mesh.vertexCount < maxint;
    [...]
}

Shouldn´t this check for mesh.vertexCount < numeric_limits<uint16_t>::max() ?
To Reproduce
Steps to reproduce the behavior:

Write a mesh with more than 2^16 vertices

Expected behavior
32 bit indices should be used if required.

Thanks! Looks like the bug was introduced when filamesh was split in libfilameshio and filamesh. Uploading a fix soon.


Fixing db timeout typing

Fixes: #461

Type casting the NAUTOBOT_DB_TIMEOUT environment variable to int.

@nniehoff Can you please update the change log (docs/release-notes/version-1.0.md) before we merge this? Thanks!


Add benchpark logo to the repository and docs

Adding the benchpark logo to the repository for use in the docs. Once this PR is merged I'll work on adding the logo to the README.rst. (We'll need it in the repo first to get the raw permanent link.)

@alecbcs Do you want to also add the single square logo without the benchpark name?

Added all of the files. I'd kinda planned to wait on the solo icons for part two where I got everything working but they didn't really need to wait.


Option to skip checkout step

I would like to use this action in a workflow that has already checked out and built the application.  Running the checkout action again clears a bunch of files which requires me to install dependencies again.

Hi @veelci thanks for issues, if you done with it let me know and create PR thanks for your contrib!


Reduce polygon stroke opacity to <1

Right now, if you load a ton of small polygons they
a) End up on the light basemap
b) Have 1px polygon stroke that is white and full opaque
So you end up not being able to see anything on the map unless you know the trick.
I'd say something like 0.5-0.8 could fix it, but worth experimenting

I find this pretty interesting. Should the default wizard change depending on the data ?

Would be the best, yeah.
On Mar 18, 2015 7:51 AM, "javi santana"<EMAIL_ADDRESS>wrote:

I find this pretty interesting. Should the default wizard change depending
on the data ?
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb/issues/2810#issuecomment-82801837.

There is even a wiki page talking about that :)
Famous Map recommendation API :)

On 18 Mar 2015, at 11:09, Andrew W. Hill<EMAIL_ADDRESS>wrote:
Would be the best, yeah.
On Mar 18, 2015 7:51 AM, "javi santana"<EMAIL_ADDRESS>wrote:

I find this pretty interesting. Should the default wizard change depending
on the data ?
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb/issues/2810#issuecomment-82801837.

—
Reply to this email directly or view it on GitHub https://github.com/CartoDB/cartodb/issues/2810#issuecomment-82867425.

totally, we should do that. but while we wait.... just drop the line
opacity, or change it black or something :)
On Wed, Mar 18, 2015 at 7:34 AM, saleiva<EMAIL_ADDRESS>wrote:

There is even a wiki page talking about that :)
Famous Map recommendation API :)

On 18 Mar 2015, at 11:09, Andrew W. Hill<EMAIL_ADDRESS>wrote:
Would be the best, yeah.
On Mar 18, 2015 7:51 AM, "javi santana"<EMAIL_ADDRESS>wrote:

I find this pretty interesting. Should the default wizard change
depending
on the data ?
—
Reply to this email directly or view it on GitHub
<https://github.com/CartoDB/cartodb/issues/2810#issuecomment-82801837
.

—
Reply to this email directly or view it on GitHub <
https://github.com/CartoDB/cartodb/issues/2810#issuecomment-82867425>.

—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb/issues/2810#issuecomment-82917935.

--
andrewxhill http://twitter.com/andrewxhill
http://cartodb.com

I think that with line-width: 0.5 we fix this temporarly.

Are we talking about making 0.5 the default Polygon Stroke Opacity?

Or changing the line-opacity CartoCSS property?

polygon-stroke :)

I've seen that we share that configuration with other wizards (Category and Color) too. Should we change it everywhere or just in this particular case?

all defaults.


[bug]: Image gallery fetches thumbnails twice

Is there an existing issue for this?

[X] I have searched the existing issues

OS
macOS
GPU
mps
VRAM
No response
What version did you experience this issue on?
3.0.0rc1
What happened?
Chrome hides this from you, but FF shows NS_BINDING_ABORTED errors in network tab for each occurrence. Looks like this is due to the gallery rendering twice when images are added to the list. maybe we can improve this.
Screenshots
No response
Additional context
No response
Contact Details
No response

Hi Millu, Is this still happening with v3.7 or latest v4.0 Release Candidate?

This is a non-issue. The errors are just indicating that the same image was requested twice in succession. This happens during normal operation.


Update programmatic-usage.mdx

Description
Just documentation site fix. Correcting the link to the core configuration object type
Related #8261
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Checklist:

[x] I have followed the CONTRIBUTING doc and the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Thanks!


Minor issue (probably atoms/affine/diag.py misses an import)

import cvxpy as cvx; x = cvx.Variable(1,10); cvx.diag(x)
gives this error

NameError                                 Traceback (most recent call last)
 in ()
1 x = cvx.Variable(1,10)
----> 2 cvx.diag(x)
/Users/enzo/anaconda/lib/python2.7/site-packages/cvxpy/atoms/affine/diag.pyc in diag(expr)
43         # Convert a row vector to a column vector.
44         else:
---> 45             expr = reshape(expr, expr.size[1], 1)
46             return diag_vec(expr)
47     elif expr.size[0] == expr.size[1]:
NameError: global name 'reshape' is not defined

shouldn't it get
from cvxpy.atoms.affine.reshape import reshape
?


Workspace: Add useLiveRegion hook to application where needed

Context
ARIA live regions are being used to announce updates to screen readers
Summary
Adds ARIA live regions in the following places:

Search component
Dropdown component

If there are any more needed I'm happy to add them there too
Relevant Technical Choices

The ticket specifies that the Snackbar should be updated to use ARIA live regions, however it already announces itself because it has an aria-role of status. Happy to change it to use a live region instead, but it seemed unnecessary since the snackbar message already announces itself to the user
The ticket wanted a live region added to the 'style' panel to announce contrast problems. However, there's no contrast check on that panel yet. Added a different ticket for this: https://github.com/google/web-stories-wp/issues/7230

Looked through the design system to see if any other components needed this, but it seemed to me like none did. If y'all know of any content that changes suddenly that should be announced to the user, let me know!
To-do
none
User-facing changes

Search
Dropdown

Testing Instructions
Need to test live regions for the search and dropdown. Turn on voiceover. If on mac that can be done with Cmd + Shift + F5
Search

Go to dashboard
Click on search, should see nothing about results until you type
As you type and list gets shorter, you'll see announcements like x results found
If you type something that doesn't find any stories, you'll see No results found

Dashboard

Go to dashboard settings
Click on dropdown at the bottom of the page
You can't interact with any dropdowns by typing in the app at the moment, so this is as much as we need to test.

QA

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:
Turn on voiceover. If on mac that can be done with Cmd + Shift + F5
Search

Go to dashboard
Click on search, should see nothing about results until you type
As you type and list gets shorter, you'll see announcements like x results found
If you type something that doesn't find any stories, you'll see No results found

Dashboard

Go to dashboard settings
Click on dropdown at the bottom of the page
You can't interact with any dropdowns by typing in the app at the moment, so this is as much as we need to test.

UAT

[x] UAT should use the same steps as above.

Reviews
Does this PR have a security-related impact?
no
Does this PR change what data or activity we track or use?
no
Does this PR have a legal-related impact?
no
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #6059

Created a ticket for adding the ARIA live region to the styles panel in the event of low contrast style changes: https://github.com/google/web-stories-wp/issues/7230

Thanks a lot for adding these ❤
I hope this is the first step for us to be more aware of aria-live and its benefits, so that we use it whenever appropriate.

re: "3. As you type and list gets shorter, you'll see announcements like x results found"
These announcements aren't appearing

@csossi are you on mac and using voiceover? Or are you on windows and using a different kind of screen reading/accessibility software?

Verified in QA


Don't emit "executable" JSON field for non-executables.

The "executable" field of JSON artifact messages was accidentally filled (with the path to index.html) when documenting a binary target. This fixes it so that it is null.

r? @Eh2406
(rust-highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 6a79008cad57cb6c3d10910dade8653593b7d7c1 has been approved by alexcrichton

:hourglass: Testing commit 6a79008cad57cb6c3d10910dade8653593b7d7c1 with merge a4cdd757e5d3f7f717dddddb2a6ddcaa045fb5df...


AEM-Eclipse plugin is not able to recognize the archetype11 projects.?

Why AEM archetype11 projects are not able to show in Add resources when i try to add in AEM Server in Eclipse Neon for the Sync.

And if i have only the projects open in Eclipse Neon that have been created by archetype11 (and all other aem archetype10 projects closed) then i can see a popup as shown below.

Did you import just the root project or also all the sub-projects?

Yes  have imported all sub-projects,
Even with brand new aem-project-archetype:11 imported project, it wont be able to show up. attaching the Document to reproduce the issue, comparing aem-project-archetype:11 vs aem-project-archetype:10
Eclipse Neon.docx

I just tested and it "works for me". Do you have any entries in the Eclipse error log?

Yes, It gives the below Error.
https://dev.eclipse.org/recommenders/community/aeri/v2/#!/submissions/SJXeVEhrpB5GXRgPwjkVTQVjTC9wxaSjP8PJ78ijODhsPyh92Q06BfxfgG6gqLlp

This was an issue with the archetype, see #78


[bug]:  Module 'cloud_firestore' not found

Is there an existing issue for this?

[X] I have searched the existing issues.

CLI Version
0.2.7
Firebase Tools version
12.5.2
Flutter Doctor Output
[✓] Flutter (Channel stable, 3.16.0, on macOS 14.1.1 23B81 darwin-x64, locale en-IN)
• Flutter version 3.16.0 on channel stable at /Users/mt11-1065/Library/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision db7ef5bf9f (3 days ago), 2023-11-15 11:25:44 -0800
• Engine revision 74d16627b9
• Dart version 3.2.0
• DevTools version 2.28.2
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
• Android SDK at /Users/mt11-1065/Library/Android/sdk
• Platform android-33, build-tools 33.0.1
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 15A507
• CocoaPods version 1.14.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
[✓] VS Code (version 1.84.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.76.0
[✓] Connected device (3 available)
• Pushpendra’s iPhone (mobile) • adbca9d21d5fc744767a19b0660a6a87425777a8 • ios            • iOS 16.7.2 20H115
• macOS (desktop)              • macos                                    • darwin-x64     • macOS 14.1.1 23B81 darwin-x64
• Chrome (web)                 • chrome                                   • web-javascript • Google Chrome 119.0.6045.159
! Error: Pushpendra’s Apple Watch needs to connect to determine its availability. Check the connection between the device and its companion iPhone, and the
connection between the iPhone and Xcode. Both devices may also need to be restarted and unlocked. (code 1)
[✓] Network resources
• All expected network resources are available.
• No issues found!
Description
CLI work fine
but when we build using Xcode then getting error like "Module 'cloud_firestore' not found"
Steps to reproduce
all ok
Expected behavior
all ok
Screenshots

Additional context and comments
No response

This is nothing to do with FlutterFire CLI. Closing out. You might try running the project from VS code first before running in Xcode. i.e. flutter run.


Resolving dependency conflicts

Pipfile specifies a set of constraints on dependencies, and therefore as newer versions of dependencies release, how they resolve will change.
My understanding is that, as a result of this, some time ago our pipenv lock stopped functioning. This has been bothering me for a while, but finally I bit the bullet and resolved it.
To prevent this problem further, I'm also committing Pipfile.lock, which stores the results of dependency resolutions. This way, we can guarantee that we always have a set of versions that satisfies constraints.
As noted in the comment, it is possible that this can create different problems around platform dependent stuff. But so far, we don't seem to have that issue.

Summary by CodeRabbit

Documentation

Updated comments in the Pipfile to provide guidance on managing dependencies and resolving conflicts.

Chores

Removed outdated comments and entries related to pipenv and Pipfile.lock from the .gitignore file.

Bug Fixes

Adjusted version constraints for setuptools, importlib-metadata, and more_itertools to prevent conflicts with newer versions.
Added click package to the development dependencies.

New Features

Enhanced dependency management in the workflow by adding a step to generate a Pipfile.lock for consistent package versions across environments.
Introduced a command to list installed packages, aiding in dependency resolution.

I'm requesting multiple reviews to make sure this doesn't have unintended consequences

Time to cut the loss.
I thought I got this working, then it stopped working. I don't understand how that behavior difference is happening. This is already a way bigger yakshaving. Too bad I couldn't prevail.


Query parameters and hash lost when switching environments

https://<preview_host>/?foo=bar will turn into https://<live_host>/ and vice versa

:tada: This issue has been resolved in version 3.23.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Go toptions

Added warnings if too few genes are submitted, as GO enrichment will fail due to this.
Note: Bacteria fail on cafe go module for this reason, so best to test on a larger dataset.
Added new cml options for pvalue cutoff, pvaue type (e.g. bonferoni) and number of bars on the GO enrichment results (default 10 per go category, MF< CC, BP)

I checked that the nf-tests are still working, they are, minus an updated snp on download

According to nextflow got_cutoff should be  a number instead of a string (by deafault value is "0.05").


Release new version of EP

@samtstern The leak was fixed since 0.4 and it would be nice to have a new version of EP released. 😄

Released 0.4.1.

@samtstern Awesome! 👍