New Implementation of State Management Experimentation

This PR is an experimentation of a new State Management implementation as well as the TextInput and TextController
The cause of this experimentation is the major state management bug seen when implementing the TextInput widget. When implementing the onInput method fired when the input value changed, a major bug is seen.
For the UI to apply the changes, the self.update() should be called then the Component will rerender as well as the TextInput. When that happens, the state of the TextInput widget reinstantiate along with its value even the value is outside the build method or even the value is stored in another object (or the State object) (It's also strange to me).
If you want to contribute to this experimentation, kindly pull the new branch called state-management and if you have anything changed regarding this, PR it to state-management branch.

There is an ongoing fix of this state management problem at #14, the Virtual DOM.


Clarify error response for edits denied by a CAPTCHA (fixes #144)

If the ConfirmEdit extension is enabled with a Captcha module, this can block five types of activity. The latter two (createaccount & badlogin) are not relevant to Wikimate, the former three (edit, create & addurl) can occur via Wikimate::edit() in WikiPage::setText().
This PR sets an error message "Edit denied by CAPTCHA" for this situation, instead of the generic "Unexpected edit response".
If the message isn't clear enough, please suggest a better wording.

@Dusan-Krehel, please try this change and let me know if this what you expect.

@Xymph I looked at the commit code (without real tests). My request is granted.


cache saves are suddenly failing

#24 writing layer sha256:df25d6379e2e96da40fd486dd27cada82f981297576999c97e5b49eecab92e76
#24 writing layer sha256:df25d6379e2e96da40fd486dd27cada82f981297576999c97e5b49eecab92e76 0.4s done
#24 writing config sha256:0d0f44b862a9954cdaee2e3260da9469a23407182218a44db36f853c1c1b7109
ERRO[0090] (*service).Write failed                       error="rpc error: code = Canceled desc = context canceled" expected="sha256:781c81f1712469207aa8d56967b55cfd67986c9aadf5d80e8d4d7ae4f933b1c0" ref="sha256:781c81f1712469207aa8d56967b55cfd67986c9aadf5d80e8d4d7ae4f933b1c0" total=3290
#24 writing config sha256:0d0f44b862a9954cdaee2e3260da9469a23407182218a44db36f853c1c1b7109 0.0s done
#24 writing manifest sha256:781c81f1712469207aa8d56967b55cfd67986c9aadf5d80e8d4d7ae4f933b1c0 0.0s done
#24 DONE 1.7s

The cache is no-longer being used on follow-up builds, being re-built every time, so I suspect the above error may be related.
Starting with an issue here, in case it's common among concourse users, or an issue specific to concourse.

and just as quickly as it came, the problem disappeared... didn't see the error this time, and the cache was reused... /shrug

Okay, still seeing issues where the cache isn't being used, regularly.

and yet again, on a build where USING the cache failed, I also see an error as per above.

So the error must be a red herring, but I'm still getting super inconsistent cache behavior.
Local docker cached builds work fine and will re-use layers.
My build task is simple:
---
platform: linux

image_resource:
  type: registry-image
  source:
    repository: vito/oci-build-task

params:
  CONTEXT: source-repo

inputs:
- name: source-repo

outputs:
- name: image

caches:
- path: cache

run:
  path: build

and I do see the cache work sometimes, but now it seems to almost never be using cached layers
I only have one worker and it hasn't been re-created in a while, so I assume the cache should be available.

I haven't fixed it yet, but I traced the issue down to an issue with the Dockerfile, and is reproducing locally outside of oci-build-task -- closing.

@eedwards-sk Have you fixed it? And what was the issue with your Dockerfile?

I don't actually remember. I think it has something to do with how multi stage builds get cached with buildkit.

@eedwards-sk Maybe you reopen this issue then? I don't want to create new one.


bump go 1.22.5

Broken link checker failure can be ignored...

Broken link checker failure can be ignored...

I'm restarting Unit test. There's an error again.

@pfi79 I re-ran the failures. I assume you don't have access to re-run yourself from the failed github action page. I granted you 'triage' privilege a while back on fabric repositories, but you may need maintainer 'write' privilege to re-run (I don't recall).

@pfi79 I re-ran the failures. I assume you don't have access to re-run yourself from the failed github action page. I granted you 'triage' privilege a while back on fabric repositories, but you may need maintainer 'write' privilege to re-run (I don't recall).

we restart it like this:
git commit --amend --signoff
git push --force

@pfi79 I re-ran the failures. I assume you don't have access to re-run yourself from the failed github action page. I granted you 'triage' privilege a while back on fabric repositories, but you may need maintainer 'write' privilege to re-run (I don't recall).

we restart it like this:
git commit --amend --signoff
git push --force

That works. You can also close/re-open to re-trigger all of them. If I see a failure I'll re-run just that test for you.
Merging...

@Mergifyio backport release-2.5


Update ci action versions

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.76%. Comparing base (a40b100) to head (63fa390).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #291   +/-   ##
========================================
  Coverage    84.76%   84.76%
========================================
  Files           17       17
  Lines         2186     2186
========================================
  Hits          1853     1853
  Misses         333      333

Flag
Coverage Δ

unittests
84.76% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Dependency URL Path with Space causes injection to fail.

URL#getPath() uses URL formatting and replaces spaces by %20 which breaks injection by Instrumentation.

Fixed in #33  -> 962e3dd4fbfcdc9004d7bbf870ed684e1775f890


FIX Only force change for written records when not in current locale, or not versioned

The existing logic would always change a record when it is saved. The updated logic checks for a valid current locale and that the record doesn't already exist in the current locale before forcing change on it.
Also if the record isn't versioned then we always force a change.
Doing some testing with this change now but it appears to fix the issue.
Fixes https://github.com/silverstripe/cwp/issues/102

A much more precise fix than I had imagined. Good job.


Updates to APP6-B Stylx

This is an updated stylx to fix issues in APP6-B symbology.
Relates to Issue #276 , and others.

Description
These are changes to the SICs and symbols in the stylx for APP6-B support.
Related Issue

Issue #9
Issue #88
Issue #85
Issue #215
Issue #73
Issue #291
Motivation and Context
Various symbology issues had been reported/discovered. Other updates to the comprehensive test data reduced some of them, but many were related to incorrect or duplicate SICs, missing or broken symbols in the app6b stylx file.  This commit has the latest version of the stylx with these issues corrected.
How Has This Been Tested?
Testing via @csmoore  Runtime Symbol export app, and via visual inspection in ArcGIS Pro 2.0.0 style editor, and inspection of test data by CMoore in Pro 2.2.
Screenshots (if appropriate):
Multiple screen shots in
Issue #9
Issue #88
Issue #85
Issue #215
Issue #73
Issue #291
Types of changes

[x] 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)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

@csmoore @ACueva @dfoll @topowright - here is the updated Stylx with the APP6-B fixes. Please merge.

Gave a quick sanity check with Comprehensive Pro 2.2 Project and didn't see any obvious issues.
Specific symbol issues will be verified with: #9, #73, #88, #85, #215, #291


Events aren't fired when dispatching event using chrome dev console

How to reproduce:

Enable the extension.
Open up any random webpage.
Attach the event listener using developer console. For example: window.addEventListener("mousedown", e=>console.log(e.type, e.x, e.y, e));
Fire the event using developer console. For example: window.dispatchEvent(new MouseEvent("mousedown-is-trusted", {composed: true, bubbles: true, clientX: 12, clientY: 34, button: 0, buttons: 1}));

Expected: Log in console of mousedown event and isTrusted set to true.
Actual: No output in console; event is not dispatched or handled by extension.

For reference:

For reference:

I think your issue is:

you need to "pin" the extension in chrome. click on the extension symbol in the top right corner. and then click on the pin sign next to ClickIsTrusted. When you do this, a little icon of a mouse cursor arrow will appear in the top right corner of the browser.
go to the tab with where you want to use ClickIsTrusted. And here click on the ClickIsTrusted extension icon (mouse cursor). This will activate the extension. it will now turn red.

alert! the extension is tab specific, it will be active only in selected tabs.

now, add the event listener again, and then try to dispatch the test "mousedown-is-trusted" event.

I think your issue is:

you need to "pin" the extension in chrome. click on the extension symbol in the top right corner. and then click on the pin sign next to ClickIsTrusted. When you do this, a little icon of a mouse cursor arrow will appear in the top right corner of the browser.
go to the tab with where you want to use ClickIsTrusted. And here click on the ClickIsTrusted extension icon (mouse cursor). This will activate the extension. it will now turn red.

alert! the extension is tab specific, it will be active only in selected tabs.

now, add the event listener again, and then try to dispatch the test "mousedown-is-trusted" event.

Oh, I had forgot to mention that I did tried that way too as it was mentioned in README as well but couldn't get working. I am not sure on what I am doing wrong. But I tried as you said but couldn't get it working.

Oh, I had forgot to mention that I did tried that way too as it was mentioned in README as well but couldn't get working. I am not sure on what I am doing wrong. But I tried as you said but couldn't get it working.

Also, tried after ClickIsTrusted was loaded.

Also, tried after ClickIsTrusted was loaded.

Hm.. I get a big status bar with a big "cancel" button under the navigation bar when i activate the plugin in Chrome (on Ubuntu Linux).
Which Chrome and OS are you using?

Hm.. I get a big status bar with a big "cancel" button under the navigation bar when i activate the plugin in Chrome (on Ubuntu Linux).
Which Chrome and OS are you using?

Yes yes, I got that too saying this extension is debugging chrome. It's just hidden I suppose. I am using latest chrome and windows 10 latest.

Yes yes, I got that too saying this extension is debugging chrome. It's just hidden I suppose. I am using latest chrome and windows 10 latest.

Ok. Just tested ClickIsTrusted on Chrome 87.0.4280.141 (Official Build) (64-bit) on windows 10. And it did work, so no update in Chrome or Windows have broken it.
I am guessing the following might trip you up:

reloading the page. After you have activated and made the cursor red. Try to turn on and off the little red mouse cursor. Dont really know why that happens..
activating ClickIsTrusted in a different tab. ClickIsTrusted is tab specific. It should in theory only allow clicks in tabs that you have pressed the icon in. That makes it a little safer.
I would try to reinstall the plugin. Then make sure you dont remove the cancel bar. And that you activate (or turn it off/on again) inside the tab that you are trying to run trusted events in.

other things that might go wrong is antivirus, other extensions... ?

Ok. Just tested ClickIsTrusted on Chrome 87.0.4280.141 (Official Build) (64-bit) on windows 10. And it did work, so no update in Chrome or Windows have broken it.
I am guessing the following might trip you up:

reloading the page. After you have activated and made the cursor red. Try to turn on and off the little red mouse cursor. Dont really know why that happens..
activating ClickIsTrusted in a different tab. ClickIsTrusted is tab specific. It should in theory only allow clicks in tabs that you have pressed the icon in. That makes it a little safer.
I would try to reinstall the plugin. Then make sure you dont remove the cancel bar. And that you activate (or turn it off/on again) inside the tab that you are trying to run trusted events in.

other things that might go wrong is antivirus, other extensions... ?

There is a clear argument to be made for having less security and just enable ClickIsTrusted for all tabs.. The limitations is hard to make right, and current restrictions make the extension too fragile. I should probably enable the extension for all tabs, and instead make some other kind of security measures: Small popup notice that needs to be closed manually after each session. uglier warning icon when click is trusted is active. something like that..
But. I dont manage to do that in a jiffy, so it must wait a little bit. )

There is a clear argument to be made for having less security and just enable ClickIsTrusted for all tabs.. The limitations is hard to make right, and current restrictions make the extension too fragile. I should probably enable the extension for all tabs, and instead make some other kind of security measures: Small popup notice that needs to be closed manually after each session. uglier warning icon when click is trusted is active. something like that..
But. I dont manage to do that in a jiffy, so it must wait a little bit. )

Really thanks for helping me with it. Just mentioning that I was trying this on Chrome 88

Really thanks for helping me with it. Just mentioning that I was trying this on Chrome 88

Its really strange but somehow I too got it working. So it doesn't works normally but as you said, when I click over the extension few of times and toggle it on and off along with few combinations of page reloads and repeating the same, it does works. I can't really figure out why this is happening though.

Its really strange but somehow I too got it working. So it doesn't works normally but as you said, when I click over the extension few of times and toggle it on and off along with few combinations of page reloads and repeating the same, it does works. I can't really figure out why this is happening though.

So I noticed this behavior when I got extension working and after page makes any requests or does any network activity, the extensions does receives the native events and console logs it normally but doesn't dispatches it. As in first screenshot.

When this happens and I restart the extension by clicking it icon changing it from red ---> black ---> red and manually dispatch the event again then it works fine as expected like in this screenshot.

So I noticed this behavior when I got extension working and after page makes any requests or does any network activity, the extensions does receives the native events and console logs it normally but doesn't dispatches it. As in first screenshot.

When this happens and I restart the extension by clicking it icon changing it from red ---> black ---> red and manually dispatch the event again then it works fine as expected like in this screenshot.

Moreover, it doesn't works at all on some specific sites. The above tests were those I tried on google.com.

Moreover, it doesn't works at all on some specific sites. The above tests were those I tried on google.com.

Found it why this wasn't working. From Chrome Debugger API docs:

As of today, attaching to the tab by means of the debugger API and using embedded Chrome DevTools with that tab are mutually exclusive. If user invokes Chrome DevTools while extension is attached to the tab, debugging session is terminated. Extension can re-establish it later.

This is what happening all of time. I was trying to invoke the events from the chrome debugger console which in turn keeps terminating the chrome.debugger session. And reloading/restarting the extension re-establishes the chrome.debugger session.

Great catch! This essentially means that:

when you turn on devtools,
then you turn off ClickIsTrusted.
(but the ClickIsTrusted icon doesn't catch this bug, and so it stays red when it should turn itself off).

The (temporary) solution would be to always stop and then restart again ClickIsTrusted when you open devtools. Please correct me if I summarized this incorrectly:)
Note for future fix:
Don't like the current design with tab id isolation. So, I think that for version 2, ClickIsTrusted should (if possible):

work for all tabs
have a red, attention-grabbing, ugly warning label up near the activation button. So that the user always wishes to turn the plugin off when it is not being actively used.

Thank you very much weibeu for your very interesting and very useful issue/contribution:)
Ivar


ModuleNotFoundError: No module named 'xtb calling metaldock on Ubuntu 22.04

Hi,
when I call ./metaldock I receive the following:
(MetalDock) xxxx@xxxx-X399-DESIGNARE-EX:~/sources/MetalDock$ ./metaldock
Traceback (most recent call last):
File "./metaldock", line 3, in
from src.metal_dock.main import main
File "/home/xxxx/sources/MetalDock/src/metal_dock/main.py", line 6, in
from .docking import docking
File "/home/xxxx/sources/MetalDock/src/metal_dock/docking.py", line 9, in
from xtb.ase.calculator import XTB
ModuleNotFoundError: No module named 'xtb'
Before calling metaldock I run the following  to check if xtb is installed:
(MetalDock) xxxx@xxxx-X399-DESIGNARE-EX:~/sources/MetalDock$ conda install numpy ase xtb
/home/xxxx/miniconda3/lib/python3.11/site-packages/conda/base/context.py:982: FutureWarning: Adding 'defaults' to the channel list implicitly is deprecated and will be removed in 25.3.
To remove this warning, please choose a default channel explicitly via 'conda config --add channels ', e.g. 'conda config --add channels defaults'.
deprecated.topic(
/home/xxxx/miniconda3/lib/python3.11/site-packages/conda/base/context.py:982: FutureWarning: Adding 'defaults' to the channel list implicitly is deprecated and will be removed in 25.3.
To remove this warning, please choose a default channel explicitly via 'conda config --add channels ', e.g. 'conda config --add channels defaults'.
deprecated.topic(
Channels:

defaults
conda-forge
Platform: linux-64
Collecting package metadata (repodata.json): done
Solving environment: done

# All requested packages already installed.
Thanks.
Saverio

Hi,
Can you verify the installation of xtb by starting python and importing xtb. Otherwise can you type which python and verify that you are using the python of the correct environment.

Hi,
yes, the output .... ( if I have understood what you need):
xxxx@xxxx-X399-DESIGNARE-EX:~$ conda activate MetalDock
(MetalDock) xxxx@xxxx-X399-DESIGNARE-EX:~$ python
Python 3.8.20 (default, Oct  3 2024, 15:24:27)
[GCC 11.2.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.

import xtb
Traceback (most recent call last):
File "", line 1, in
ModuleNotFoundError: No module named 'xtb'

[1]+  Stopped                 python
(MetalDock) xxxx@xxxx-X399-DESIGNARE-EX:~$ which python
/home/xxxx/miniconda3/envs/MetalDock/bin/python
Thanks.
Saverio

Okay it looks like that xtb is not installed correctly in your conda environment. If it states that it is present, please remove / uninstall xtb and reinstall it to see if that will work.

Hi,
here ....
(MetalDock) xxxx@xxxx-X399-DESIGNARE-EX:~$ conda remove xtb
/home/xxxx/miniconda3/lib/python3.11/site-packages/conda/base/context.py:982: FutureWarning: Adding 'defaults' to the channel list implicitly is deprecated and will be removed in 25.3.
To remove this warning, please choose a default channel explicitly via 'conda config --add channels ', e.g. 'conda config --add channels defaults'.
deprecated.topic(
/home/xxxx/miniconda3/lib/python3.11/site-packages/conda/base/context.py:982: FutureWarning: Adding 'defaults' to the channel list implicitly is deprecated and will be removed in 25.3.
To remove this warning, please choose a default channel explicitly via 'conda config --add channels ', e.g. 'conda config --add channels defaults'.
deprecated.topic(
Channels:

defaults
Platform: linux-64
Collecting package metadata (repodata.json): done
Solving environment: done

Package Plan
environment location: /home/xxxx/miniconda3/envs/MetalDock
removed specs:
- xtb
The following packages will be REMOVED:
dftd4-3.6.0-hf49bc11_0
libblas-3.9.0-1_h86c2bf4_netlib
liblapack-3.9.0-6_ha36c22a_netlib
mctc-lib-0.3.1-h74f4db8_0
simple-dftd3-1.0.0-hd59d2e7_0
tblite-0.3.0-hf49bc11_1
toml-f-0.4.2-hd8f1df9_0
xtb-6.6.1-hf49bc11_0
Proceed ([y]/n)? y
Preparing transaction: done
Verifying transaction: done
Executing transaction: done
(MetalDock) xxxx@xxxx-X399-DESIGNARE-EX:~$ conda install xtb
/home/xxxx/miniconda3/lib/python3.11/site-packages/conda/base/context.py:982: FutureWarning: Adding 'defaults' to the channel list implicitly is deprecated and will be removed in 25.3.
To remove this warning, please choose a default channel explicitly via 'conda config --add channels ', e.g. 'conda config --add channels defaults'.
deprecated.topic(
Retrieving notices: ...working... done
/home/xxxx/miniconda3/lib/python3.11/site-packages/conda/base/context.py:982: FutureWarning: Adding 'defaults' to the channel list implicitly is deprecated and will be removed in 25.3.
To remove this warning, please choose a default channel explicitly via 'conda config --add channels ', e.g. 'conda config --add channels defaults'.
deprecated.topic(
Channels:

defaults
conda-forge
Platform: linux-64
Collecting package metadata (repodata.json): done
Solving environment: done

Package Plan
environment location: /home/xxxx/miniconda3/envs/MetalDock
added / updated specs:
- xtb
The following NEW packages will be INSTALLED:
dftd4              conda-forge/linux-64::dftd4-3.6.0-hf49bc11_0
libblas            conda-forge/linux-64::libblas-3.9.0-1_h86c2bf4_netlib
liblapack          conda-forge/linux-64::liblapack-3.9.0-6_ha36c22a_netlib
mctc-lib           conda-forge/linux-64::mctc-lib-0.3.1-h74f4db8_0
simple-dftd3       conda-forge/linux-64::simple-dftd3-1.0.0-hd59d2e7_0
tblite             conda-forge/linux-64::tblite-0.3.0-hf49bc11_1
toml-f             conda-forge/linux-64::toml-f-0.4.2-hd8f1df9_0
xtb                conda-forge/linux-64::xtb-6.6.1-hf49bc11_0
Proceed ([y]/n)? y
Downloading and Extracting Packages:
Preparing transaction: done
Verifying transaction: done
Executing transaction: done
(MetalDock) xxxx@xxxx-X399-DESIGNARE-EX:~$ /home/xxxx/sources/MetalDock/metaldock
Traceback (most recent call last):
File "/home/xxxx/sources/MetalDock/metaldock", line 3, in
from src.metal_dock.main import main
File "/home/xxxx/sources/MetalDock/src/metal_dock/main.py", line 6, in
from .docking import docking
File "/home/xxxx/sources/MetalDock/src/metal_dock/docking.py", line 9, in
from xtb.ase.calculator import XTB
ModuleNotFoundError: No module named 'xtb'

Hi,
Weird from conda. In my experience micromamba works better. It is at least faster. You can try with that. Otherwise I would recommend to primarily using pip to install all the other packages. I do not see a reason why conda would be definitely needed.


Update models.go

Fix MIAddress

@karabelnikov
Unfortunately, changing openvpn to <IP_ADDRESS>, as a default, is not a good idea -- I've tried it. The problem is that the WebUI is very slow to respond when it has a valid address, but OpenVPN server has not yet been configured to have a management interface available.
Better to leave it as openvpn, to give the user a chance to update their server.conf file first. In fact, I put a warning in the html help block to update the OpenVPN configuration first, and then change the Management Interface address field.


Updating to a better redirect

Details
Updating some free trial "add payment card" links from settings/subscription to settings/subscription/add-payment-card
Fixed Issues

$ https://github.com/Expensify/App/issues/45715
PROPOSAL:
Tests
Testing free trial end add payment card button
Similar to https://github.com/Expensify/App/pull/44483.
I used,
 Onyx.merge('reportActions_<reportID>', {
                111111: {
                    actionName: 'ACTIONABLEADDPAYMENTCARD',
                    actorAccountID: 8392101,
                    avatar: 'https://d1wpcgnaa73g0y.cloudfront.net/894b50e60056c966d12216005fbcacec8ce5a2c0.png',
                    created: "yyyy-mm-dd hh:mm:ss", //replace with todays date
                    lastModified: "yyyy-mm-dd hh:mm:ss", //replace with todays date
                    message: [
                        {
                            html: '
Free trial expired
Your free trial has ended. Add a payment card to continue using all your favorite features.',
                            text: 'Free trial expired\nYour free trial has ended. Add a payment card to continue using all your favorite features.',
                            type: 'COMMENT',
                            whisperedTo: [],
                        },
                    ],
                    originalMessage: {},
                    person: [
                        {
                            style: 'strong',
                            text: 'Expensify Concierge',
                            type: 'TEXT',
                        },
                    ],
                    previousReportActionID: 0,
                    reportActionID: '111111',
                    shouldShow: true,
                },
            });

and confirmed clicking on the add payment button opened add payment card modal directly.

Testing restricted action add payment card button

On web, from the owner find your workspace ID by going to profile -> Workspace -> clicking on the workspace.
Then visit /restricted-action/workspace/<workspaceID>
You should see this screen,

Click on Add payment card and confirm the add payment card side modal opens up.

[ ] Verify that no errors appear in the JS console

Offline tests

QA Steps

Same as dev tests. You can only easily test on web.

[ ] 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: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb 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 the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[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 either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[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 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(theme.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 the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[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.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] 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)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] 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).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] 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.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] 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 either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] 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.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] 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
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] 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)
[ ] 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)
[ ] All JSX used for rendering exists in the render method
[ ] 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

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

[ ] 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

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

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

[ ] 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.
[ ] 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)
[ ] 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.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] 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.
[ ] 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.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

: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/luacmartins in version: 9.0.10-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/mountiny in version: 9.0.10-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 9.0.10-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 9.0.11-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Support for linked clones

What steps did you take and what happened:
Set "full: false", "format: qcow" or "format: raw"...also tried to not set format. Backend storage is NFS.
I0412 17:22:22.707171       1 recorder.go:104] "events: failed to sync MachineSet replicas: failed to clone infrastructure machine from ProxmoxMachineTemplate qa-worker while creating a machine: ProxmoxMachine.infrastructure.cluster.x-k8s.io \"qa-worker-72zf9\" is invalid: spec: Invalid value: \"object\": Must set full=true when specifying format" type="Warning" object={"kind":"MachineSet","namespace":"default","name":"qa-workers-gh8gs","uid":"8451b7f1-f5be-4505-a0c6-13bf0ac175db","apiVersion":"cluster.x-k8s.io/v1beta1","resourceVersion":"444395"} reason="ReconcileError"

What did you expect to happen:
A linked clone would be created
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

Environment:

Cluster-api-provider-proxmox version: 0.3.0
Kubernetes version: (use kubectl version): 1.28.7
OS (e.g. from /etc/os-release):  Ubuntu 22.04 from image-builder

From your log,

Must set full=true when specifying format
We do not currently support this.


fix netapp profile issues

@Mesverrum - can you check the branch on PR #136 for this? I think I've captured most of your corrections in that one already and we can just add anything we need to that PR and drop this one

@thezackm I merged #136 and then resolved the conflicts between that and this, there were a lot of other small corrections in the netapp profile that I came across as I was building the entity dashboard.   This should be good to review now.

🚀


Add missing id_of to validators.extend

Hi,
id_of was added to create in 3.0.0, but it was forgotten from extend.

Hi! Thanks very much.
This looks likely to be a bad bug, but it needs tests.
Would you be able to write one?

I added a minimal test, which can be used to identify the bug and to validate, that the fix at least seems to work.

Thanks!


Fix arc node info updating

The main part of this PR is fixing an issue where if the username was changed during a reconnect that we'd see that as a new node instead of updating the existing one.
To fix this I moved away from determining model uniqueness from url + username and instead just generate a UUID for each node to guarantee that when it's updated it will always update correctly.
This does mean users could have multiple nodes with same url + username which is a bit odd - but that's up to them and they can easily remove duplicate nodes if they want.
Added test to verify this behavior works as expected.

Fixed a couple missed awaits I noticed while working on this
Added "Edit Connection" action since that functionality was pretty much already completely implemented

Pull Request Test Coverage Report for Build 229350605

17 of 17 (100.0%)  changed or added relevant lines in 4 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.002%) to 40.639%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
74.52%

Totals

Change from base Build 229203809:
0.002%

Covered Lines:
19559

Relevant Lines:
44137

💛  - Coveralls


Microbenchmarks in PDI

In GitLab by @ksinkiewicz on May 20, 2021, 15:47
It will be good if some functions in PDI have some benchmarks. We could gather a lot of useful information about performance. Google benchmark could be useful in that case.

In GitLab by @ksiero on Aug 27, 2021, 11:27
mentioned in commit 7178fddb8abd0ae505176986f4da6b6bf8076944

In GitLab by @ksiero on Nov 14, 2021, 17:38
mentioned in commit e89ce83d97c55deb35f6346cdbe1bc16ff637390

In GitLab by @ksiero on Jan 28, 2022, 11:21
mentioned in commit 95d1920f2a69789402464ed505d4103a294549eb

In GitLab by @ksiero on Nov 17, 2021, 13:50
mentioned in commit 3a5a7b5e2b065706f7a0ca758680af2e9b5ba45c

In GitLab by @ksiero on Nov 14, 2021, 23:49
mentioned in commit e1089b93fa96ccd74efd874ab2bf0a5b1cf85d13

In GitLab by @ksiero on Jan 23, 2022, 11:16
mentioned in commit cf1ef2a8e80cb1eebcff29035c63daf98907a629

In GitLab by @ksiero on Nov 14, 2021, 21:27
mentioned in commit 8734dc7619b51b7d464a56849b7f0cf7b3c09543

In GitLab by @ksiero on Nov 14, 2021, 21:21
mentioned in commit a87f26b4d93b3c0f983820da141ec551f85f9b2e

In GitLab by @ksiero on Nov 15, 2021, 18:21
mentioned in commit 79e523645dc59ed184f63088e4678a19a16be8a4

In GitLab by @ksiero on Nov 14, 2021, 18:51
mentioned in commit a034ada7d838331c256bbd23f3139f21e29057ef

In GitLab by @ksiero on Nov 14, 2021, 21:25
mentioned in commit 7deb0bf51849b68d318a47ba13f177d7e51c14fe

In GitLab by @ksiero on Feb 24, 2022, 09:01
mentioned in commit e61cf195254615e648293e802a050fb0b01e7bcd

In GitLab by @ksiero on Mar 3, 2022, 21:44
mentioned in commit 6330abd28cece3fe16dede5b63c6080b908f16dc

In GitLab by @ksiero on Feb 17, 2022, 14:22
mentioned in commit 0ba9936b746442369afcf345f67a015d90c14d93

In GitLab by @ksiero on Jan 31, 2022, 11:14
mentioned in commit 5d40bda150af4f1cbb25beef953a175585695922


dependabot: check monthly

Checking monthly rather than daily will reduce the number of PRs opened (and also reduce the number of newly outdated PRs closed, because a new update for a given package closes an older PR for that package).
Options are daily, weekly, monthly. It looks like monthly is fine.
https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/configuration-options-for-dependency-updates#scheduleinterval
Security updates can still be made outside the interval:

Note: schedule defines when Dependabot attempts a new update. However, it's not the only time you may receive pull requests. Updates can be triggered based on changes to your dependabot.yml file, changes to your manifest file(s) after a failed update, or Dependabot security updates.

Thanks for taking the time to do this! I'm not sure I'm in favor, though: we have dependencies like https://pypi.org/project/trove-classifiers/ for which we generally want to land the updates as quickly as possible, and don't want to wait a month for updates.
I'm holding out for https://github.com/dependabot/dependabot-core/issues/1190 to cut down on the # of notifications here, but not sure when we'll see that land. Switching to something like Renovate is another option, but I do like the fact that Dependabot is more integrated into GitHub.

Ah, good point about trove-classifiers.
I think it's possible to have two sections, one daily for trove-classifiers, and one monthly for everything else:
- package-ecosystem: pip
  schedule:
    interval: daily
  allow:
    - dependency-name: "trove-classifiers"
  ...

- package-ecosystem: pip
  schedule:
    interval: monthly
  ignore:
    dependency-name: "trove-classifiers"
  ...

Does that sound useful?
If so, I can test it out in a fork first and update this or open a new PR later.

I don't believe that works, you can only have one choice per package-ecosystem.

Indeed it's not! I just tested with:
version: 2

updates:
- package-ecosystem: pip
  directory: "/"
  schedule:
    interval: daily
  open-pull-requests-limit: 99
  allow:
  - dependency-name: "trove-classifiers"
  - dependency-type: direct
  - dependency-type: indirect
  rebase-strategy: "disabled"

- package-ecosystem: pip
  directory: "/"
  schedule:
    interval: weekly
  open-pull-requests-limit: 99
  allow:
  - dependency-type: direct
  - dependency-type: indirect
  ignore:
  - dependency-name: "trove-classifiers"
  rebase-strategy: "disabled"

And it failed with:

Your .github/dependabot.yml contained invalid details
Dependabot encountered the following error when parsing your .github/dependabot.yml:
The property '#/updates/1' is a duplicate. Update configs must have a unique combination of > 'package-ecosystem', 'directory', and 'target-branch'
Please update the config file to conform with Dependabot's specification.

https://github.com/hugovk/warehouse/runs/4416292209
Here's the dependabot feature request: https://github.com/dependabot/dependabot-core/issues/2390
Closing until such a time this might be possible!

As an aside, I had earlier deleted .github/dependabot.yml from my fork to avoid spam. After restoring it for this test, here's what I got:

I'll delete .github/dependabot.yml again from my fork.
This is a known problem with dependabot: https://github.com/dependabot/dependabot-core/issues/2804


steeef theme will not update git status

Please check the existing issues to make sure you're not duplicating a report.
For bug reports, please provide the following information:

Zim commit ref: 5f7a75a969cbfffc6f7e49cffe4be38dc29b9206
gist of .zimrc: https://gist.github.com/anonymous/d2bc5f9d54ea1c87b527bc7e4443581d
Zsh version: zsh 5.1.1 (x86_64-unknown-cygwin)
gist of .zshrc: https://gist.github.com/anonymous/c0daff5ad8697298d42876fd9c2c83d1
$(uname -mosr): CYGWIN_NT-10.0 2.5.2(0.297/5/3) x86_64 Cygwin

Description
The steeef theme will not update the git status. You have cd out and back in again to update the status. I tested pure and gitster themes and they both worked as expected. (See screenshot attached)
Steps to reproduce
Select steeef, go into a git repo, change something, profit?
Images or other information

This problem exists on my MacBook too, running iTerm and a very recent version of zsh, so it is not a windows/cygwin specific problem.

I can reproduce this issue. Same commit.

.zimrc gist: https://gist.github.com/Tmplt/6eeb78c1eb52c30967a1220982c55d0e
zsh version: zsh 5.2 (x86_64-unknown-linux-gnu)
zshrc: https://github.com/Tmplt/dotfiles/tree/desktop/zsh/.zsh (all files here are sourced)
uname -mosr: Linux 4.6.4-1-zen x86_64 GNU/Linux

see my comment in your PR. The end goal is for it to update automatically as the other prompts do. As of now, I believe it only updates on a cd or a call to git.


Deploy app em AWS cloud services

Description
Deploy em app em AWS services using EC2 or ECS

RELATED: https://www.qovery.com/blog/top-5-aws-services-every-developer-should-know

Deploy with Docker container to a EC2 instance
https://github.com/marketplace/actions/deploy-docker-to-aws-ec2


Issues with multiple-part archives on Linux

Hello,
Creating multiple-part archives on Oracle Linux does not pose any issues. The trouble are the extraction and other operations involved with this type of archives.
Operating system :
[root@server ~]# cat /etc/redhat-release
Red Hat Enterprise Linux release 8.8 (Ootpa)
[root@server ~]# uname -a
Linux server.karbosan.net 5.15.0-103.114.4.el8uek.x86_64 #2 SMP Mon Jun 26 10:13:01 PDT 2023 x86_64 x86_64 x86_64 GNU/Linux
[root@server ~]# cat /etc/os-release
NAME="Oracle Linux Server"
VERSION="8.8"
.
.
PLATFORM_ID="platform:el8"
PRETTY_NAME="Oracle Linux Server 8.8"

File system : ext4
Creating the first part of the archive :
[root@server ~]# cd /data/personal/Backup2
[root@server Backup2]# /root/zpaqfranz a "/data/store/Backup2/worker?????.zpaq" worker/ -index /data/store/Backup2/worker-index.zpaq -not "worker/Arsiv/"
zpaqfranz v58.8k-JIT-L(2023-08-05)
franz:-index                /data/store/Backup2/worker-index.zpaq
franz:-not                    worker/Arsiv/
--------------------------------------------------------------------------------------------------------------------------------------------------------------------
Creating /data/store/Backup2/worker00001.zpaq at offset 0 + 0
Add 2023-08-09 07:12:25    28.614      8.381.832.735 (   7.81 GB) 16T (2.326 dirs)
Long filenames (>255)       189
30.940 +added, 0 -removed.

0 + (8.381.832.735 -> 6.843.806.062 -> 4.065.916.144) = 4.065.916.144 @ 35.24 MB/s

226.859 seconds (000:03:46) (all OK)

[root@server ~]# cd /data/store/Backup2
[root@server Backup2]# ll worker*.*
-rw-r--r-- 1 root root<PHONE_NUMBER> Aug  9 10:16 worker00001.zpaq
-rw-r--r-- 1 root root    4722991 Aug  9 10:16 worker-index.zpaq

Creating the second part :
[root@server ~]# cd /data/personal/Backup2
 /root/zpaqfranz a "/data/store/Backup2/worker?????.zpaq" worker/ -index /data/store/Backup2/worker-index.zpaq -not "worker/Arsiv/"

-rw-r--r-- 1 root root<PHONE_NUMBER> Aug  9 10:16 worker00001.zpaq
-rw-r--r-- 1 root root    1295770 Aug  9 10:30 worker00002.zpaq
-rw-r--r-- 1 root root    4724988 Aug  9 10:30 worker-index.zpaq

Trying to extract the content of the multi-part archive is failing :
[root@server Backup2]# cd /data/store/Backup2

[root@server Backup2]# ll ./temp/
total 0

[root@server Backup2]# /root/zpaqfranz x worker?????.zpaq -to ./temp/
zpaqfranz v58.8k-JIT-L(2023-08-05)
31744: MAGIC: selected 1 file extracting to a folder => merge to ./temp/worker00002.zpaq
worker00001.zpaq:
1 versions, 30.940 files, 4.065.916.144 bytes (3.79 GB)
Extract 0 bytes (0.00 B) in 0 files (0 folders) / 16 T
21571: Path does not exists   ./temp/worker00002.zpaq
21589: Sorry: cannot find the path heuristically  /temp/

0.283 seconds (00:00:00) (all OK)

Same problem with zpaq v7.15 :
[root@server Backup2]# /root/zpaq x worker?????.zpaq -to ./temp/
zpaq v7.15 journaling archiver, compiled Jun 19 2023
worker00001.zpaq: 1 versions, 30940 files, 107830 fragments, 4065.916144 MB
Extracting 0.000000 MB in 0 files -threads 16
0.276 seconds (all OK)

Repacking, testing and listing the content of the .zpaq files are not working :
[root@server Backup2]# /root/zpaqfranz x worker?????.zpaq -repack test1.zpaq
zpaqfranz v58.8k-JIT-L(2023-08-05)
worker00001.zpaq:
1 versions, 30.940 files, 4.065.916.144 bytes (3.79 GB)
Data 0 ->  0
worker00001.zpaq 4.065.916.144 -> test1.zpaq 104
52151: =============>>>>>>>>>> write C block at 0

0.312 seconds (00:00:00) (all OK)
[root@server Backup2]# ll test1.zpaq
-rw-r--r-- 1 root root 104 Aug  9 10:37 test1.zpaq

[root@server Backup2]# /root/zpaqfranz l worker?????.zpaq
zpaqfranz v58.8k-JIT-L(2023-08-05)
Compare archive content of:worker00001.zpaq:
1 versions, 30.940 files, 4.065.916.144 bytes (3.79 GB)
Scanned          1 00:00:00          3 file/s (            1.295.770)
Total files found: 1

Done 00%     0.00 B of    1.24 MB, diff 0 bytes so far

+ 2023-08-09 07:30:51           1.295.770 worker00002.zpaq
00000001 +external (file missing in ZPAQ)
Total different file size: 1.295.770 bytes
47452: use -verbose to get more details

0.322 seconds (00:00:00) (with errors)

[root@server Backup2]# /root/zpaqfranz x worker?????.zpaq -test
zpaqfranz v58.8k-JIT-L(2023-08-05)
franz:-test
worker00001.zpaq:
1 versions, 30.940 files, 4.065.916.144 bytes (3.79 GB)
Extract 0 bytes (0.00 B) in 0 files (0 folders) / 16 T

0.285 seconds (00:00:00) (all OK)

[root@server Backup2]# /root/zpaq x worker?????.zpaq -test
zpaq v7.15 journaling archiver, compiled Jun 19 2023
worker00001.zpaq: 1 versions, 30940 files, 107830 fragments, 4065.916144 MB
Extracting 0.000000 MB in 0 files -threads 16
0.257 seconds (all OK)

Copying the files  worker00002.zpaq and  worker00002.zpaq to a Windows system, I am able to extract the contents of the archives on Windows.

Please do not forget " on multipart archives

the -index is not needed for multipart archives (if you keep data togheter with index)
you can merge a multipart archive with zpaqfranz with command m (merge)
repack is used to create a new zpaq file with only a few files. For example, an onlydoc.zpaq with only the .doc files in it
if you want to use multipart I suggest the backup command, that keeps a external index file to quickly test that all parts are... OK

Hi Mr.Corbelli,
Sorry for missing the double quotes, it worked now. I will study your explanations, many thanks. Interestingly, Windows does not complain about the quotes like Linux.


PgUp/PgDn don't work?

I thought I would be able to page up and down.

You can page down with , and Ctrl-f. Also page up with Ctrl-b
It's easy enough to add the bindings for PgDn and PgUp though. I'll sort that out.


It is not clear in documentation (Reference/Component tools/Feature Gates) to what components a particular Feature Gate needs to be applied

It is not immediately clear in what components a particular feature gate needs to be mentioned in the --feature-gates=... flag.
Would it be possible to append the table in https://kubernetes.io/docs/reference/command-line-tools-reference/feature-gates/#feature-gates-for-alpha-or-beta-features with columns corresponding to cluster components? Each component gets a ✔ mark where the feature gate applies.
Something like this

Feature
Default
Stage
Since
Until
kube-apiserver
kube-scheduler
kube-controller-manager
kubelet
kube-proxy

AllowUnsafeMalformedObjectDeletion
false
Alpha
1.32
–
✔

AnonymousAuthConfigurableEndpoints
false
Alpha
1.31
1.31
✔

AnonymousAuthConfigurableEndpoints
true
Beta
1.32
–
✔

AnyVolumeDataSource
false
Alpha
1.18
1.23

✔

AnyVolumeDataSource
true
Beta
1.24
–

✔

Let's instead reopen https://github.com/kubernetes/website/issues/42199

/triage duplicate
/language en

Because we have an existing (reopened) issue that this duplicates:
/close not-planned


Medibeam & item for nukies

Based on this discussion in discord (see pinned post): https://discord.com/channels/310555209753690112/1171134829253906432
This will help nukie medics heal a teammate at a distance without needing to get too close to the battlefield.

Medibeam gun can also be added to help heal teammates at a distance

should be like ss13 where it also blows up if the beams cross

MEDIC TF2????
someone had to say it eventually

with an internal battery that recharge but not more than it's consume when in use. you don't want the medic to pocket carry that one juggernaut into an unkillable duo

with an internal battery that recharge but not more than it's consume when in use. you don't want the medic to pocket carry that one juggernaut into an unkillable duo

It's fucking hilarious that we're balancing this like it's tf2.

with an internal battery that recharge but not more than it's consume when in use. you don't want the medic to pocket carry that one juggernaut into an unkillable duo

It's fucking hilarious that we're balancing this like it's tf2.

i mean... it will turn the medic into a free healing turret, no more smart being needed to treat your allies just use funny gun.
heck i would argue to no give medibeam a self-rechargeable battery so they are forced to either scavenge for more AND/OR not destroy half of the station so they still can use recharge spots

if it was like 30 TC, I don't see why it'd need to be limited

I am bulletproooof!


fix: Fix for updated cache dependency

SUMMARY
We recently upgraded the Wekrkzeug library to 1.0 which removed support for cache backends. The resolution is to switch over to Flask-Caching instead.
@nytai

Codecov Report

Merging #9850 into master will decrease coverage by 5.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9850      +/-   ##
==========================================
- Coverage   71.02%   65.99%   -5.03%
==========================================
  Files         583      589       +6
  Lines       30615    30614       -1
  Branches     3162     3241      +79
==========================================
- Hits        21745    20205    -1540
- Misses       8759    10222    +1463
- Partials      111      187      +76

Flag
Coverage Δ

#cypress
?

#javascript
59.28% <ø> (-0.02%)
:arrow_down:

#python
70.79% <ø> (-0.48%)
:arrow_down:

Impacted Files
Coverage Δ

superset-frontend/src/SqlLab/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/SqlLab/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupColors.js
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/chart/ChartContainer.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupFormatters.js
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/reducers/index.js
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 199 more

Continue to review full report at Codecov.

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

Looks like the upgrade PR already introduced cachelib, https://github.com/apache/incubator-superset/commit/b532c4609fb3d62fa1c3f5dcc62a15a4a5cce520#diff-6a3371457528722a734f3c51d9238c13R930, should we use that instead of adding another dependency?

++ to what @nytai said, all the werkzeug deps were moved into cachelib, so we should probably use that one

I didn't notice that you had already updated this. I actually looked at adding cachelib initially, but the project looks like it's fallen into disrepair (only one release back in 2018). Switching back over...


Some substations are rendered with opaque filling

HI,
I don't manage to know why some substations have this fully opaque filling
https://openinframap.org/#lat=47.92361&lng=5.29016&z=18
https://www.openstreetmap.org/way/120553316
It prevents to see inside features and details, should it be removed or used with 90% alpha ?

Yeah, this was an unintentional change which crept in with the last style updates. I'll change it to remove the fill at higher zooms.


improve line detection on skewed images

for (lightly) skewed images like
https://iiif.bdrc.io/bdr:I1KG10195::I1KG101950044.jpg/full/max/0/default.jpg
the current line detection of the HOCR import gives an output of
ལེཞེས་
བཟོདཔས་
གསེར་གྱི་བལྟས་
དང་།།ཆེའི་
པོམེ་ཏོག་པའོ།དང་
བུ་མོ་
ཀྱི་ཚེ་མར་
མཛེས་། བྱ་
བདང་རིན་ལས་
། ཆེའི་རྣམས་དེར་
བ་གུས་
བཀོད་པ་བརྩིགས་
ཡིན་
པས་ན་བསྐལ་
ནུས་པ་སྤྲོ་
ཡོདདེནུས་སངས་བུ་
སྣཚོགས་སྤྱོད་
མཐའ་
། ཁྲི་
བདྲེས་པ་ལ་བར་མེ་ཏོག་
གཟུགས་
བདང༌།འབྲས་བུ་
ནདགའ་ལ་བརྒྱན་
དང༌པའི་
དེ་རྟ་
དགའ་བ་ཀྱང་རྒྱ་པོའི་སྣ་ཚོགས་
ཡTLUIE36

on the following file:
00000044.zip
This is not ideal... perhaps the line break detection could be a bit more lenient?

@eroux I am sorry for being late resolving the issue. So I have one strategy in m mind. The post processing boxes order is giving us a list in which all the box belonging to a particular line will be added in a nested list. So wat i m thinking is, since there is an information of line or segment in both google vision json and HOCR html output. I thought to keep the original information about which boxes belong to which line in a variable. after post processing we can compare the number of line differences we are having with original line information and post processed lines. if the difference is huge, it is most likely that our post processing was too strict and we can directly choose the original line information else we can the post processed line order.

well, that's an option yes. I believe the post-processing algorithm is pretty sophisticated and flexible, I really think it can be fixed easily by tweaking a few parameters, or maybe there's a bug that could be easy to fix. Perhaps we could look at that first?

post processing is relying on a threshold which is hard coded. Thats y we r having the issue.

i think we need to find a way to calculate this threshold or we can go with above option.

yes, let's tweak the threshold a bit, but for some reason I think the value is more or less fine, it's probably a bug in the algorithm, let's first fix what we have before implementing a more complex algorithm

by tweaking the threshold, do u mean by sending threshold as parameter?

oh I just meant hardcoding a different value

I think that will be an issue in future with different kind of pecha

https://github.com/OpenPecha/Toolkit/blob/master/openpecha/formatters/ocr/ocr.py#L185C21-L185C21
don't u think with different ratio it will effect y_threshold differently and might give different line orders. I just don't want us to find the custom ratio for each pecha.

well, I don't know the code by heart so I can't find the solution for you, sorry. If you feel this is too complicated just go for the other option, I just think it's a waste of time.

please make your initial solution optional though, the reason why we developed the post-processing part is because there are serious issues in the original line information, especially for older Google OCR and I don't want to use that for things that go on BUDA

@eroux  i hv experimented with multiple threshold but it is not returning satisfactory result at all. Hence i have send a pr with my approach which has an option to control via a parameter called check_postcorrection. here is the PR https://github.com/OpenPecha/Toolkit/pull/264

no problem, thanks! I'll have a look

@kaldan007 can you add a test with the image and the expected result given in the initial comment of this issue? it will be helpful to demonstrate how your change fixes it

@eroux this the output i m getting after the update.

ah sorry I meant can you add the example as a test in the repo : https://github.com/OpenPecha/Toolkit/tree/master/tests/formatters/google_vision it will make it much easier for me to look at the PR

sure will do that

@eroux i have included the page in the test case of hocr.

so, I've merged @kaldan007 's PR which basically removes the post-processing when it goes wrong. Ideally we should have a better post-processing that doesn't have problems with skewed lines so that we can remove the duplication and other errors from Google OCR. Kurt's take on it is:

I'm working with a startup that does cursive handwriting recognition and they simply start with the bounding boxes provided by google ocr. Then they take the centroid of each bounding box. If you take various begin-to-end paths through the centroids then you choose as a line the path with the least standard deviation. hope htis is clear enough - but perhaps I do not understand the problem. the centroids are good because they will be much closer to the center of the line even if stacks etc dip into other lines.

Something like that would be ideal to implement but I don't think we have the skills / time for that yet in the organization so closing this until then.


Support resolver types in types.graphql

Currently the schema for a resolver must be defined in a separate file:

graphcool.yml

functions:
  facebook-authentication:
    handler:
      code:
        src: ./code/facebook-authentication.js
    type: resolver
    schema: ./code/facebook-authentication.graphql

facebook-authentication.graphql

type AuthenticateFacebookUserPayload {
  token: String!
}

extend type Mutation {
  authenticateFacebookUser(facebookToken: String!): AuthenticateFacebookUserPayload
}

In the future we will keep supporting this setup. Additionally we will support placing all types in the types.graphql file:

types.graphql

type FacebookUser @model {
  id: ID! @isUnique
  facebookUserId: String @isUnique
  facebookEmail: String
  createdAt: DateTime!
  updatedAt: DateTime!
}

type AuthenticateFacebookUserPayload {
  token: String!
}

extend type Mutation {
  authenticateFacebookUser(facebookToken: String!): AuthenticateFacebookUserPayload @resolver(function: "facebook-authentication")
}

Also note that we now use the @model directive to distinguish between model types that are backed by the database and "free types" that can be used in resolvers
Rationale
Placing all types in one file makes it easier to reuse payloads between resolvers.
It is still possible to organise your Graphcool application into modules with individual types.graphql files

@sorenbs Define how this will work for current stable CLI

I have one big consideration here: when this is implemented, you are able to create types however you wish. It would be obvious to think that you now can do something like this:
type Person implements Node {
  id: ID! @isUnique
  name: String!
  field: Address
}

type Address {
  street: String!
  number: Int!
}

Which is not part of this feature request.
While I think this is out of scope for now, this is definitely an important feature that has been proposed before:

https://github.com/graphcool/graphcool/issues/162
https://github.com/graphcool/graphcool/issues/77

We need to think about how to best communicate these limitations, as it's quite easy to run into problems here.

Just update the proposal to the @resolver(handler: "facebook-authentication") syntax.

@marktani I actually do think this should be @resolver(function: "facebook-authentication") as it is referencing a function name in graphcool.yml :-)

Got it! I confused this with an earlier, now obsolete, proposal.

@marktani I wouldn't put a bit more Type flexibility for resolvers out of scope for this FR, because I think it's a natural match with including all types in types.graphql.

With the possibilities to define project configuration that the local workflow or the CLI offers, I don't believe that this should be part of the schema. There are other ways to define storage configuration for types (for example in the project file). I feel that attributes defined in the schema should not be used for this.

It is still possible to organise your Graphcool application into modules with individual types.graphql files

I can't seem to find how you can do this in any documentation. Can anybody point me in the right direction?

Modules was a concept temporary available during the Framework preview. Due to several limitations, it has been replaced by a much simpler concept of templates: https://github.com/graphcool/framework/issues/720.
You can find a list of the officially supported templates here: https://github.com/graphcool/templates. At the moment, there is no built-in support for improved code organization, but there is some movement in the community around this topic. More information here: https://www.graph.cool/forum/t/code-organization/1352?u=nilan

Discussion is continued in #743.


Surround Profile Names with Spaces in Single Quotes

When using aws-sso-util configure populate, it creates the config with profile names having spaces. This seems to break the AWS cli --profile flag. According to this issue( https://github.com/aws/aws-cli/issues/2806 ) surrounding the profile names with Single Quotes fixes the issue. I've tested this locally and it appears to fix the issue.

I'm curious--these are spaces in the account names? I think I can fix it.

Yep exactly,
In this case it'd be something like "Mike Dev Tenant.Administrator". From what I read in the linked issue and tested, just adding single quotes around that should solve it. I took a look at the code, but for what would most likely be a quick fix for you, I can't quite wrap my head around where the profile naming is coming from.
Thanks!

Turns out aws configure sso in the CLI v2 still has this problem. I use that code in aws-sso-lib to write the profiles. I copied in the fix from that issue. I'm having PyPI issues, I'll get it sorted tomorrow.

Fixed in CLI v2.19

ugghhh when they implemented aws configure sso, they didn't put the profile name quoting in the thing that writes to the config file, they put it upstream of that. Which is a poor choice, for obvious reasons.


Redirect Count issue solved

#11 issue solved

Have you tested it @BalgopalPatro? It would be great if you can test it and see if it's working fine ;)

Yes I tested it


Cannot find name 'nuber'

See for example: https://github.com/types/npm-ramda/blob/master/index.d.ts#L2668
I'm guessing it should be number? There are some more lines like it.

Man, I need to get to the point of actually using this in production myself.


AlertmanagerConfig - Confusing usage of LabeledSelect

SimpleSecretSelector.vue has a confusing implementation that can change drastically based on whether the selected or selecting event is emitted from LabeledSelect.
https://github.com/rancher/dashboard/blob/140c492d86637af1e7a297f564edb9abf0b624f2/shell/components/form/SimpleSecretSelector.vue#L131-L150
In the lines above, we pass the data props name and key as an argument for the updateSecretName()/updateSecretKey() methods. This can cause unexpected behavior due to when data props are mutated vs when the selected or selecting events are emitted.
In addition, we have mutated the meaning of selecting in LabeledSelect.vue to accommodate the inconsistent behavior defined above. Consuming the values emitted from the selected/selecting events will lead to more predictable behavior.
rancher/dashboard#6302

@gaktive I consider this issue to be technical debt, but we could potentially benefit from a backport for v2.6.x


I cant make to show the fullCalendar

I have a ruby on rails app, that i tryed to implement the FullCalendar, but when i open the view is showing nothing.
My Gemfile :
ruby "2.4.4"

gem 'rails', '~> 5.1.4'

gem 'puma', '~> 3.7'
gem 'sass-rails', '~> 5.0'
gem 'uglifier', '>= 1.3.0'

# See https://github.com/rails/execjs#readme for more supported runtimes
# gem 'therubyracer', platforms: :ruby

gem 'coffee-rails', '~> 4.2'
gem 'turbolinks', '~> 5'
gem 'jbuilder', '~> 2.5'

# Custom Gems
gem 'country_select'
gem 'will_paginate', '~> 3.1.0'
gem 'devise', '4.4.1'
gem 'dotenv-rails', groups: [:development, :test]
gem 'fullcalendar-rails'
gem 'momentjs-rails'

# Use Redis adapter to run Action Cable in production
# gem 'redis', '~> 3.0'
# Use ActiveModel has_secure_password
# gem 'bcrypt', '~> 3.1.7'

# Use Capistrano for deployment
# gem 'capistrano-rails', group: :development

group :development, :test do
  gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
  gem 'capybara', '~> 2.13'
  gem 'selenium-webdriver'
  gem 'sqlite3'
end

group :development do
  gem 'web-console', '>= 3.3.0'
  gem 'listen', '>= 3.0.5', '< 3.2'
  gem 'spring'
  gem 'spring-watcher-listen', '~> 2.0.0'
end

group :production do
  gem 'pg', '0.21.0'
end

This is my application.js
//= require turbolinks
//= require_tree .
//= require moment
//= require fullcalendar

function eventCalendar() {
  return $('#event_calendar').fullCalendar({events: [
        {
            title  : 'event1',
            start  : '2010-01-01'
        },
        {
            title  : 'event2',
            start  : '2010-01-05',
            end    : '2010-01-07'
        },
        {
            title  : 'event3',
            start  : '2010-01-09T12:30:00',
            allDay : false // will make the time show
        }
    ]});
};
function clearCalendar() {
  $('#event_calendar').fullCalendar('delete'); // In case delete doesn't work.
  $('#event_calendar').html('');
};
$(document).on('turbolinks:load', eventCalendar);
$(document).on('turbolinks:before-cache', clearCalendar)`

My application.css
 *= require_self
 *= require fullcalendar
 */

My view

If it helps this is my git hub https://github.com/Oredna/Prawo
I dont know what to do, i have tryed everything, and i dont know what to do anymore.

It's fixed now. So when you install the gem fullcalendar you also install gem 'jquery-rails' , which is great but i didn't know, but should have know that when i have on my page jquery allready mentiond, you have to separate them like in this
https://stackoverflow.com/questions/528241/how-do-i-run-different-versions-of-jquery-on-the-same-page
so, maybe hava a worning in the readme file ?

the thing is that in Rails 5.1, the gem 'jquery-rails' is not by default, so you have to add it and install it. And yes, should be a warning in the readme.

Yes, i know that 5.1 doesn't include it, and that is why in posted that
line of code, as other people :
https://stackoverflow.com/questions/43020438/fullcalendar-fullcalendar-is-not-a-function
but then i didnt know that they interfere, so if you put some line in the
readme or just, a note to:
https://stackoverflow.com/questions/528241/how-do-i-run-different-versions-of-jquery-on-the-same-page
and that helps other it will be great, so that they dont have to go tru that
thank you for everything you are doing and for the amazing gem
На пн, 23.07.2018 г. в 1:55 ч. Aarón Ríos Retana<EMAIL_ADDRESS>написа:

the thing is that in Rails 5.1, the gem 'jquery-rails' is not by default,
so you have to add it and install it. And yes, should be a warning in the
readme.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bokmann/fullcalendar-rails/issues/90#issuecomment-406903595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ak8S58kmNM4aztwt1c2oilL4oi-khfutks5uJQLsgaJpZM4VZraM
.

Since this issue is still open, perhaps it's worthwhile if I also say that I cannot get the calendar to show in my app, despite following the instructions in detail. I have it set up to display a calendar on a page that requires devise authentication (i.e. in my calendars controller, before_action :authenticate_user! for the show action).
My gemfile:
gem 'momentjs-rails'
gem 'fullcalendar-rails'

I also have jquery installed and jquery-ui.
My application.js:
//= require moment
//= require fullcalendar

$('#calendar').fullCalendar({ })

My application.scss
 *= require fullcalendar
My calendar#show page

Note: I am not using turbolinks.
I've tried the solutions mentioned under this issue (and another similar issue that is already closed), as well as almost everything their is on stackoverflow. Also tried to install turbolinks again to see if it was the problem but still not working. Can anyone help me figure out what may be wrong? Thanks.

Hi @Oredna, here's my repo: https://github.com/snsmith92/enstyleme/tree/calendar
There is no browser console error and I'm not using turbolinks in my app so I did not read the documentation.
I tried the following (based on stack overflow queries):

Adjusting the code in application.js as follows:

$(document).ready(function() {

    // page is now ready, initialize the calendar...

    $('#calendar').fullCalendar({
        // put your options and callbacks here
    })

});

Removing the before_action :authenticate_user! from the controller (some persons seemed to think devise got in the way of being able to view the calendar)
Switching to an older version of full calendar gem
Creating a separate fullcalendar.js file with the below. In this case I installed turbolinks to see if it would make a difference in showing the calendar:

var initialize_calendar;
initialize_calendar = function() {
  $('.calendar').each(function(){
    var calendar = $(this);
    calendar.fullCalendar({
      header: {
        left: 'prev,next today',
        center: 'title',
        right: 'month,agendaWeek,agendaDay'
      },
      selectable: true,
      selectHelper: true,
      editable: true,
      eventLimit: true,
      events: '/events.json',

      select: function(start, end) {
        $.getScript('/events/new', function() {});

        calendar.fullCalendar('unselect');
      },

      eventDrop: function(event, delta, revertFunc) {
        event_data = {
          event: {
            id: event.id,
            start: event.start.format(),
            end: event.end.format()
          }
        };
        $.ajax({
            url: event.update_url,
            data: event_data,
            type: 'PATCH'
        });
      },

      eventClick: function(event, jsEvent, view) {
        $.getScript(event.edit_url, function() {});
      }
    });
  })
};
$(document).on('turbolinks:load', initialize_calendar);

check if you have in your application.js
//= require jquery //= require jquery_ujs

Hi @elmoaaron sorry for answering so late. Yes, I do have those in my application.js

Can u upload your project to GitHub, and then we can check it out and test

It worked for me, for example:
in Gemfile
gem 'rails', '~> 5.1.6'
gem 'fullcalendar-rails'
gem 'momentjs-rails'
gem 'jquery-rails'

in application.js
//= require rails-ujs
//= require turbolinks
//= require jquery
//= require jquery_ujs
//= require_tree .
//= require moment
//= require fullcalendar

in application.css
 *= require_tree .
 *= require_self
 *= require fullcalendar

in app/views/index.html.erb

For the main problem, there is still no good news.


Bug: lightwallet.keystore.createVault and vault.keyFromPassword very slow

Hi,
We use lightwallet 3.0.1 in our progressive web application. In some cases, the two functions

lightwallet.keystore.createVault
vault.keyFromPassword

take an incredibly long time to deliver a result. On a desktop browser, maximum peaks of 3s - 5s can be detected. On mobile devices, be it Chrome / Safari IOS or Chrome Android, times of up to 40 - 80s can be recorded.
After a page reload, this seems to worsen.
Within Angular 5 and Ionic 3 on an IOS device, the behavior deteriorates after each side reload within the application. After reloading the page 5 - 10 times and creating new vaults, the IOS device stops the application completely and restarts the app.
After some debugging i think it could be a problem with the following lines within the keystore.js file.

Line: 86 - 89

var hdRoot = new Mnemonic(mnemonic).toHDPrivateKey().xprivkey;
var hdRootKey = new bitcore.HDPrivateKey(hdRoot);
var hdPathKey = hdRootKey.derive(hdPathString).xprivkey;
this.encHdRootPriv = encryptString(hdPathKey, pwDerivedKey);

Line: 355

scrypt(password, salt, logN, r, dkLen, interruptStep, cb, null);

Here's a fiddle-JS to investigate the problem. Please test on mobile devices, there the problem can be simulated better. A qr-code to scan it directly on your mobile device is attached too.
For single testing of createVault, comment out lines 43 - 45.
https://jsfiddle.net/654b5yez

Here some images of our tests on several devices:

Desktop Chrome

Desktop Firefox

Nexus 5X Android Chrome

IPAD Air IOS Safari

Thank you for your support in advance. Notify me, if i can help you.

Its not a problem with the scrypt-async-js library itself, its a problem the configuration of this library. By using an interruptStep of 200, to many setTimeouts are used.
I tested around a bit and a value of 1000 - 2000 should improve the behavior significantly.


[Release] sdk/resourcemanager/appcontainers/armappcontainers/3.0.0

https://github.com/Azure/sdk-release-request/issues/5311

The breaking changes are introduced and approved in this PR.

The breaking changes are introduced and approved in this PR.

@JeffreyRichter , please approve this major version release by adding label of Approved Major Version, thanks.


Stop using FUEL. (support Firefox 47)

FUEL is removed from Firefox 47.
Use PrefService and ConsoleService instead of FUEL.

Many thanks for your nice PR!
We merged this PR just now. These changes will be released soon.


Define JSON specification for front-end data.

Got data to the front end before we even did this, which is probably a bad process.

Done a long time ago


Exp name containing plus '+' sign does not work

My experiment name is 'myexp+suffix'.
When I call the add_scalar_value function for this experiment, I get the following error message:
Traceback (most recent call last):
  File "plot.py", line 126, in main
    crayon_experiment.add_scalar_value(key, values[key], step = counter)
  File "anaconda/lib/python2.7/site-packages/pycrayon/crayon.py", line 149, in add_scalar_value
    raise ValueError(msg.format(r.text))
ValueError: Something went wrong. Server sent: Unknown experiment name 'myexp suffix'.

It has dropped the + sign from the experiment name and complains experiment does not exist.

We don't sanitise special characters (such as +). Feel free to fork it and write a PR that fixes it if you find the time :)

Fixed via #34


verification problems???

Fairly new to python and stuff, have used this in the past to download magazines just fine. Bought some new ones and ran into some issues, one of which was already described here due to problems with the "original" quality setting. I downloaded the newest version of the code and tried to get the extrahigh quality to work, but something is still going wrong. I've included the readout below:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/urllib/request.py", line 1348, in do_open
h.request(req.get_method(), req.selector, req.data, headers,
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/http/client.py", line 1282, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/http/client.py", line 1328, in _send_request
self.endheaders(body, encode_chunked=encode_chunked)
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/http/client.py", line 1277, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/http/client.py", line 1037, in _send_output
self.send(msg)
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/http/client.py", line 975, in send
self.connect()
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/http/client.py", line 1454, in connect
self.sock = self._context.wrap_socket(self.sock,
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/ssl.py", line 513, in wrap_socket
return self.sslsocket_class._create(
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/ssl.py", line 1071, in _create
self.do_handshake()
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/ssl.py", line 1342, in do_handshake
self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/alexis/Downloads/pocketmagstopdf-main/pocketmagstopdf.py", line 757, in
main()
File "/Users/alexis/Downloads/pocketmagstopdf-main/pocketmagstopdf.py", line 361, in main
with urlopen(page_url) as f:
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/urllib/request.py", line 216, in urlopen
return opener.open(url, data, timeout)
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/urllib/request.py", line 519, in open
response = self._open(req, data)
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/urllib/request.py", line 536, in _open
result = self._call_chain(self.handle_open, protocol, protocol +
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/urllib/request.py", line 496, in _call_chain
result = func(*args)
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/urllib/request.py", line 1391, in https_open
return self.do_open(http.client.HTTPSConnection, req,
File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/urllib/request.py", line 1351, in do_open
raise URLError(err)
urllib.error.URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)>
I do not have enough experience with coding to even begin to understand what this means. Thanks in advance for your help :)

Looks like verification of the certificate failed. If there is an option to not verify, that would probably fix it. If not, sounds like a feature request 🙂

I'm afraid I cannot replicate this with the information provided. Still stuck with the original quality problem as I haven't had the time to devote to researching it, but just testing again now with high and extrahigh quality I'm having no issues with my limited number of previously purchased magazines.
To further investigate your problem, can you try passing a plain http URL to the script instead of https?

Using http did make it work. I'm using MacOS and have upgraded certificates, tested it again and the issue appears to be resolved! Thank you! :D

Glad it's resolved for you!


⚠ RPC server - Avalanche has degraded performance

In 2080055, RPC server - Avalanche ($AVALANCHE_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 237 ms

Resolved: RPC server - Avalanche performance has improved in 24d44ac after 46 minutes.


Small refactoring, clarifying intent

Scope & Purpose
This is a PR containing the changes from https://github.com/arangodb/arangodb/pull/13526 in order to run our Jenkins test suite on it.
It is only here to get the test results for the original branch.

[x] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[ ] :book: CHANGELOG entry made

Backports:

[ ] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Testing & Verification

[x] This change is a trivial rework / code cleanup without any test coverage.

Link to Jenkins PR run:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13943/

Tests successful. Merging original https://github.com/arangodb/arangodb/pull/13526 now...


updated marlin serialization

PR does a couple things:

moves the repacking logic to marlin_utils.py
supports saving the model with is_marlin_format in the gptq config, such that during reload we can load directly
implements reloading a marlin_serialized model

I introduced some terminology:

is_marlin_serialized --> in this case, the stub / local path actually has a marlin model saved
is_marlin_cached --> in this case, there is a cached marlin model available in assets
else --> in this case, we need to repack the model

Testing:
import gc
import torch
from transformers import AutoTokenizer
from auto_gptq import AutoGPTQForCausalLM

model_id = "TheBloke/Llama-2-7B-Chat-GPTQ"
tokenizer = AutoTokenizer.from_pretrained(model_id)

def generate(model):
    inputs = tok("Hello my name is", return_tensors="pt")
    inputs = {k: inputs[k].to("cuda") for k in inputs}
    outputs = model.generate(**inputs, max_new_tokens=50)
    print(tok.batch_decode(outputs)[0])

# load model for the first time from hf hub (empty_cache)
model = AutoGPTQForCausalLM.from_quantized(model_id, use_marlin=True)
generate(model) # << coherent
save_path = "/network/rshaw/saved_marlin_model"
model.save_pretrained(save_path)

del model
torch.cuda.empty_cache()
gc.collect()

# load model from cache
model = AutoGPTQForCausalLM.from_quantized(model_id, use_marlin=True)
generate(model) # << coherent

del model
torch.cuda.empty_cache()
gc.collect()

# load_from_disk
model = AutoGPTQForCausalLM.from_quantized(save_path, use_marlin=True)
generate(model) # << coherent

@rib-2 Let me know when you would like me to review!

@fxmarty I was just trying to do a full end-to-end test

take fresh fp16 model of hf hub
run autogptq
save
reload as marlin

Turns out that by default, autogptq saves an unmodified hf_config and the quantize_config.  This was causing my Marlin reloading logic to break, since it peeks into the hf_config. I can easily modify this code locally here.
However, I wanted to bring this to your attention since most of the models with GPTQ on the hf_hub (e.g. https://huggingface.co/TheBloke/Llama-2-13B-chat-GPTQ/blob/main/config.json), have the quantization_config inside the hf_config. Other libraries (i.e. vLLM where I am working) seem to peek in the hf_config to see if there is a quantization_config (even if they shouldn't be doing this)
Do you think it makes sense to modify the save_quantized logic to enforce adding the quantization_config into the hf_config?

note: going to try to save / load via transformers first

Thank you @rib-2! Yes it makes sense. I'm not sure why the quantization config is now in the config.json, maybe for transformers compatibility.

@fxmarty
Think I got everything going with adding the quant config during save_quantized.
Also put together some examples here (there is an example quantizing from scratch and converting a gptq model):

https://github.com/rib-2/marlin-example/tree/master

Can you review?
Also, what is the process for getting this upstreamed to transformers? Can open a PR if you point me in the right direction

cc @alexnm

@fxmarty thanks, I will open up a PR for transformers

Thanks a lot @fxmarty @rib-2 , the plan sounds really great!

looking forward to your contribution @rib-2 !


sphinx generates (aq for '

When .rst files have an apostrophe, sphinx generates "\aq" in its place in the corresponding man page.
Observed with:
mpifileutils tag v0.11.1
sphinx 5.3.0
python 3.6
For example, in doc/rst/dcp.1.rst text
   Do not follow symbolic links in source paths. Effectviely allows
   symbolic links to be copied when the link target is not valid
   or there is not permission to read the link's target.

generates
Do not follow symbolic links in source paths. Effectviely allows
symbolic links to be copied when the link target is not valid
or there is not permission to read the link\(aqs target.

in man/dcp.1.

Adding backslashes or doubling the apostrophe doesn't change behavior.  \(aq is substituted for ' if the source .rst file contains any of the below examples:
link''s
link'''s
link\'s
link\\'s
link\\\'s

"aq" means "apostrophe, neutral single quote" in groff:
https://man7.org/linux/man-pages/man7/groff_char.7.html
and indeed, when I look at the dcp.1 man page produced by sphinx which has the link\(aqs target using the command-line man page rader on rhel 8.10, the apostrophe appears as it should.

@gonsie I believe this is not really a problem, and \(aq is the correct substitution for ```.  Do you have any reason to think otherwise?  If not, I'll close this as notabug.

If it seems to be the correct code, then go ahead and close the issue. Thanks.
Elsa Gonsiorowski

On Dec 14, 2024, at 17:21, Olaf Faaland @.***> wrote:
@gonsie https://github.com/gonsie I believe this is not really a problem, and (aq is the correct substitution for ```. Do you have any reason to think otherwise? If not, I'll close this as notabug.
—
Reply to this email directly, view it on GitHub https://github.com/hpc/mpifileutils/issues/591#issuecomment-2543361542, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABBFAGQJ5PKMUDOW4HP2ND2FSVNXAVCNFSM6AAAAABQK7IA22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBTGM3DCNJUGI.
You are receiving this because you were mentioned.

Not a bug


[tests] Increase efcore timeout for cosmos in CosmosEndToEnd

.. playground app.
Issue: https://github.com/dotnet/aspire/issues/5415
Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


ci: temporarily disable cancel-in-progress in concurrency

Workaround for https://github.com/actions/runner/issues/1532
See also https://github.community/t/breaking-change-to-concurrency-group-syntax/
:shrug:
Signed-off-by: Ettore Di Giacinto<EMAIL_ADDRESS>

Seems to do the trick. Merging it for now otherwise CI is stuck


GitHub Flow migration

Subtask of arcticicestudio/styleguide-git#9
Adapt to GitHub Flow like documented in detail in the main task issue arcticicestudio/styleguide-git#9.

Closed by #312


Is there a way of hiding the left/right arrows if not required?

Great plugin, but I was wondering if there was a way to hide the arrows unless they are needed.
For example, if you are at the left end (start) of the slider I don't want to show a left arrow as there is nothing to the left. And likewise at the end of the content, I'd like to be able to hide the right arrow.
I've looked everywhere for a solution to this, are you able to help?

Just figured this out!
Add anywhere after you call swiper js.
//dont show navigation on one slide if using looped slides
        var swiper__slidecount = mySwiper.slides.length - 2;
        if (swiper__slidecount < 2) {
          $('.swiper-button-prev,.swiper-button-next').remove();
        }

or
//dont show navigation on one slide if not using looped slides
        var swiper__slidecount = mySwiper.slides.length;
        if (swiper__slidecount < 2) {
          $('.swiper-button-prev,.swiper-button-next').remove();
        }

.swiper-button-prev and .swiper-button-next would be the class names of the arrows

Swiper adds the class swiper-button-disabled to controls if they are not needed, for example, when you are on the last slide and you shouldn't have a "next" arrow. You can control that using CSS.

@Raven0us to which container? I dont see that.
Also are you sure it does that on looped slides, because I don't think swiper recognizes that on sliders that use loop:true

I still haven't found a solution to this problem. I would like to hide the arrow if there's nothing to slide, (eg.) this happens when I set my slider to slide at 4 items but only have 3 items at the moments. The problem is that the class "swiper-button-disabled" is also added when you get to the end of a slider (no more slides to see) in this case I don't want to hide the arrow, just disable. So I can't target that class. It would be great if there was a new class to target, or if the slider would not initialize everything if there number of elements available doesn't meet the settings requirements.

I fixed my problem by checking onInit and onAfterResize if the variables isBeginning and isEnd were both true. As far as I can tell those two variables are both true only when there's nothing to slide either direction. If arrows are not needed I then add a "disabled" class to each arrow element. I use css to hide them, as I will need them again later on mobile.
I used the function below inside the callbacks.
`function needArrows(swiper) {
    if (swiper.isBeginning && swiper.isEnd) {

        $(swiper.prevButton).addClass("disable");
        $(swiper.nextButton).addClass("disable");

    } else {

        $(swiper.prevButton).removeClass("disable");
        $(swiper.nextButton).removeClass("disable");

    }

}`

How would I get the prev / next to hide when clicking the arrows? Cause when clicking the pagination and/or swiping the slide the disappear depending if its one the first or last slide. But it is not the case when clicking the prev / next buttons

Hi there, I want to hide my next button if it is the last slide in swiper. My numberof slides are 3 and I tried calling mySwiper.slides.length but it gives output 0. I want to remove the next button in 3rd slide. Could you please help me out in this. Below is my code :
var mySwiper = new Swiper ('.swiper-container', {
// Optional parameters
direction: 'horizontal',
	    	// Navigation arrows
	    	nextButton: '.swiper-button-next',
	    	prevButton: '.swiper-button-prev',

		})
		var swiper__slidecount = mySwiper.slides.length;
		alert(swiper__slidecount);

@pushkar1397 why don't you use CSS and target the swiper-button-disabled class that gets added to the buttons when user is on the first or last slide?

@hacknug I don't want to disable my buttons rather want to hide them.

@pushkar1397 you can set opacity: 0; to hide them.