Running Cargo miri test on cargo, an error occurred

I am using miri to run on cargo and it shows an error:
test util::context::tests::disables_multiplexing ... error: unsupported operation: can't call foreign function `OPENSSL_init_ssl` on OS `linux`
    --> /root/.cargo/registry/src/szv-open.codehub.huawei.com-f5aae4094eaf2133/openssl-sys-0.9.92/src/lib.rs:122:13
     |
122  |             OPENSSL_init_ssl(init_options, ptr::null_mut());
     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't call foreign function `OPENSSL_init_ssl` on OS `linux`
     |
     = help: if this is a basic API commonly used on this target, please report an issue with Miri
     = help: however, note that Miri does not aim to support every FFI function out there; for instance, we will not support APIs for things such as GUIs, scripting languages, or databases
     = note: BACKTRACE on thread `util::context::`:
     = note: inside closure at /root/.cargo/registry/src/szv-open.codehub.huawei.com-f5aae4094eaf2133/openssl-sys-0.9.92/src/lib.rs:122:13: 122:60
     = note: inside closure at /root/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/sync/once.rs:149:41: 149:60
     = note: inside `openssl_sys::openssl::init` at /root/.cargo/registry/src/szv-open.codehub.huawei.com-f5aae4094eaf2133/openssl-sys-0.9.92/src/lib.rs:121:9: 123:11
     = note: inside `libgit2_sys::openssl_init` at /root/.cargo/registry/src/szv-open.codehub.huawei.com-f5aae4094eaf2133/libgit2-sys-0.17.0+1.8.1/lib.rs:4330:5: 4330:24
     = note: inside closure at /root/.cargo/registry/src/szv-open.codehub.huawei.com-f5aae4094eaf2133/libgit2-sys-0.17.0+1.8.1/lib.rs:4303:9: 4303:23
     = note: inside closure at /root/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/sync/once.rs:149:41: 149:60
     = note: inside `libgit2_sys::init` at /root/.cargo/registry/src/szv-open.codehub.huawei.com-f5aae4094eaf2133/libgit2-sys-0.17.0+1.8.1/lib.rs:4302:5: 4324:7
     = note: inside `git2::init` at /root/.cargo/registry/src/szv-open.codehub.huawei.com-f5aae4094eaf2133/git2-0.19.0/src/lib.rs:755:5: 755:16
     = note: inside `git2::Config::open_default` at /root/.cargo/registry/src/szv-open.codehub.huawei.com-f5aae4094eaf2133/git2-0.19.0/src/config.rs:93:9: 93:22
note: inside `util::network::proxy::http_proxy`
    --> src/cargo/util/network/proxy.rs:22:5
     |
22   |     git2::Config::open_default()
     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
note: inside `util::network::proxy::http_proxy_exists`
    --> src/cargo/util/network/proxy.rs:38:5
     |
38   |     http_proxy(http).is_some()
     |     ^^^^^^^^^^^^^^^^
note: inside `util::context::disables_multiplexing_for_bad_curl`
    --> src/cargo/util/context/mod.rs:2955:8
     |
2955 |     if network::proxy::http_proxy_exists(http, gctx) && http.multiplexing.is_none() {
     |        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
note: inside `util::context::tests::disables_multiplexing`
    --> src/cargo/util/context/mod.rs:3006:13
     |
3006 |             disables_multiplexing_for_bad_curl(curl_v, &mut http, &gctx);
     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
note: inside closure
    --> src/cargo/util/context/mod.rs:2975:31
     |
2974 |     #[test]
     |     ------- in this procedural macro expansion
2975 |     fn disables_multiplexing() {
     |                               ^
     = note: this error originates in the attribute macro `test` (in Nightly builds, run with -Z macro-backtrace for more info)

note: some details are omitted, run with `MIRIFLAGS=-Zmiri-backtrace=full` for a verbose backtrace

error: aborting due to 1 previous error

error: test failed, to rerun pass `--lib`

Looking at the error message, miri needs to support the common API on Linux? I would like to know what is the progress of API support on current linux?

You can't call arbitrary C libraries like OpenSSL or libgit2 from inside miri. Miri has shims for most common libc api's, but for anything else you did either have to remove the usage or patch miri to add your own shims. It seems like you are trying to run cargo inside miri. You could try using gitoxide and rustls for git and tls accesses. Cargo already has support for this, but it is disabled by default. I believe there are a couple of env vars you can set. Also I think miri doesn't have any networking support yet.

Yeah, Miri does and will not not support calling OpenSSL - sorry. Miri can only check bugs in Rust code, and all C code that you call needs to be re-implemented inside Miri. We do this for some basic platform APIs, but OpenSSL is too big to qualify for that.

I wonder if we should have a specific error message for "known to be deliberately not supported" functions that says -- sorry but this is out of scope.

Also I think miri doesn't have any networking support yet.

FWIW that is definitely in scope, I plan for us to have networking support at some point. See https://github.com/rust-lang/miri/issues/3449.

Sorry, my thoughts are: if the problem is due to unsupported calls to APIs such as OpenSSL, it seems like it should be ignored for now and continue with the program later and give a unified reason later.
Also, I'm just getting started with miri and it's not going very well for me (maybe I'm just too dumb), should the user experience be enhanced?
It's just a personal thought, don't take it seriously.

There is not really a way to ignore it. Miri doesn't know what it could return as dummy value without causing the program to crash immediately or corrupt state in such a way that it will error later in a seemingly unrelated way. Nor does miri know if any of the input pointers will be modified by the call or not.

If you are just getting started with Miri, why are you trying to run the cargo test suite inside Miri? That is certainly not the normal mode of use.
The normal mode of use is to install Miri, navigate to your own project, and do cargo miri test there. Miri is made to test libraries that internally use unsafe code, not fully integrated end-user applications like cargo.


Fix: Fixed an issue where dragging or opening multiple items would use the wrong sort mode

Resolved / Related Issues

Closes #8534
Closes #15308

Steps used to test these changes
These changes make the behavior of opening or dragging and dropping multiple files consistent with Files Explorer.

Prepare 6 video files, named 1-6
Select them in any order
Click play all on tool bar or enter on keyboard
The 6 files open in the player in the same order as they appear in the file list
Set sorting to "by name ascending"
Select the 6 files in any order
Right-click on file 3, choose "Open" from the menu
The 6 files open in the player in the order: 3 -> 4 -> 5 -> 6 -> 1 -> 2
Select the 6 files in any order
Drag file 3 and drop on a Windows Media Player window
The 6 files open in the player in the order: 3 -> 4 -> 5 -> 6 -> 1 -> 2

@Qin-shihuang thank you! I previously attempted to fix this but couldn't get the logic quite right, really appreciate your help on this!


Chore: a simple template change

And remove duplications in some tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.591%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1337

Relevant Lines:
1659

💛  - Coveralls

This is just a unittests refactoring, so merged directly.


feat(copy-button): sync with @carbon/react v11

Related Ticket(s)
#10074
Description
Update copy-button to sync with react version
Changelog
New

copy component button
now cds-copy-button uses cds-copy
add animation classes

Changed

updated code-snippet with the new copy-button, but will still need to be updated with the sync process
updated typos with slot should be span

Removed

{{removed thing}}

@ariellalgilmore looks good! One fix: the icon color is not changing in the dark theme, looks like the token needed is $icon-primary

@ariellalgilmore looks good! One fix: the icon color is not changing in the dark theme, looks like the token needed is $icon-primary

updated! thanks!


[SDPA-4062] limit max width of body content copy

JIRA issue: https://digital-engagement.atlassian.net/browse/SDPA-4062
Changed

Added max-width to Markup p and headings element
Added max-width to Markup Quotation
Added max-width to DocumentLink component

Screenshots

Markup Quotation and Embedded document:

Callouts

Search results

Form

Note: Still waiting for Duncan's advice on what to do with Web Form when there is no sidebar. This has been raised by Jess during testing
cc @tim-yao @MdNadimHossain

@tim-yao @MdNadimHossain @dylankelly this has been tested and reviewed before. Merging this as requested by Jess.


Fix: Broken image link to MeshMate logo

Description
This PR fixes issue #1108
Notes for Reviewers
Screenshot of fixed image link to MeshMate logo

Signed commits

[x] Yes, I signed my commits.

LGTM  // @Chadha93 @leecalcote
https://github.com/meshery/meshery.io/issues/1108#issuecomment-1500933652 @AbhishekGMV will you please reference the PR fixing the logo in README

LGTM // @Chadha93 @leecalcote
#1108 (comment) @AbhishekGMV will you please reference the PR fixing the logo in README

Thanks, made changes and fixed the broken images in REAME file too.


docs: Update Screenshot in README.md

What do you think?

What did you do to these files?

Ah i see these are the old logos


Split bill - "Payment details not found" for a split with 1 person

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Login to NewDot
Click + -> Split Bill
Input amount and click continue
Select 1 attendee
Submit Split Bill
Click in "split $XX for description" link in chat

Expected Result:
Payment details should be displayed (or link should be not clickable)
Actual Result:
"Payment details not found" error page is displayed
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No, in provided build
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/cc809f9f-04f7-47f8-808b-c0eb4da4dba9
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Issue is not reproductible in provided build https://18652.pr-testing.expensify.com/
https://github.com/Expensify/App/assets/93399543/8db88031-1392-4680-b82a-dad097bec084

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Reproduced in production:

I think this is backend issue, with split request, the IOUReportID is not found inside originalMessage object.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Split bill - "Payment details not found" for a split with 1 person
What is the root cause of that problem?
We haven't defined what to show when performing a split bill with 1 person.
Previously, the split text isn't clickable and now we are making it clickable on purpose, have a look at this PR.
What changes do you think we should make in order to solve the problem?
According to the PR, for the split bill, we'll show the participant page at the moment.
Then we can modify this line
https://github.com/Expensify/App/blob/45c3212061983394e0919839248512874c2af83f/src/components/ReportActionItem/IOUAction.js#L72-L79
To
    const isBillSplit = lodashGet(props, 'action.originalMessage.type') === 'split';
    const hasMultipleParticipants = props.chatReport.participants.length > 1;
    const onIOUPreviewPressed = () => {
        if (hasMultipleParticipants || isBillSplit) {
            Navigation.navigate(ROUTES.getReportParticipantsRoute(props.chatReportID));
        } else {
            Navigation.navigate(ROUTES.getIouDetailsRoute(props.chatReportID, props.action.originalMessage.IOUReportID));
        }
    };

Also determining isSplitBill with room participants' length isn't reliable, it will not true if we split bill with only one person. Then we can change:
https://github.com/Expensify/App/blob/45c3212061983394e0919839248512874c2af83f/src/components/ReportActionItem/IOUAction.js#L109
To:
                    isBillSplit={isBillSplit}

What alternative solutions did you explore? (Optional)
N/A

Not sure if this is going to be gone after the refactor is over for manual requests.

This is the page that is supposed to be opening but for some reason, it opens the wrong one with only one participant:

@aldo-expensify that comment makes sense, but we are already addressing it here

@luacmartins ahh, thanks! I'll close this one then.


feat: bump hdwallet native packages to latest

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #3123   👈

PR #3124

This comment was auto-generated by Graphite.


how do i logout of the playground?

when i reopen it automatically logs me in

using this button, but I think it will be easier if this button is on the navbar beside the balance (no one will expect that it's inside)

Definitely, needs to be in the navbar

Verified on Devnet, 9227221.
New logout button was add beside the balance in the navbar


Write a detailed usage guide

Similar to the one for reactstrap https://reactstrap.github.io/components/alerts/
It should show valid prop values, the rendered output and example code.

Ace, in that case we could just add some links to relevant storybook docs.

resolved by https://github.com/resin-io-modules/resin-components/pull/37


Parallel tasks within a SERVICE or WORKER

I create a new request of type "SERVICE" or "WORKER".
When I deploy 3 tasks within that request, each with a unique deploy id - each new task kills the previous task before spawning itself. Is this expected behaviour?
I was looking for a "parallel task within same request" behaviour, which did not kill other running tasks.
I understand there is a feature called "instances", but I do not want to spawn all tasks right away; and each task have slightly different docker environment variable parameters.
How can I go about it?
Thanks!

Could this be because I have disabled health checks, and Singularity killed it because it thought the other tasks were unhealthy?

That is expected behavior in Singularity. A deploy is a new configuration for all tasks running under that request. So, you have an 'active deploy' that is the current configuration for all running tasks under that request. When you create a new deploy, new tasks (corresponding to number of instances) are created with the new configuration. When those new deploy tasks are healthy, the new deploy becomes the 'active deploy' and tasks from the old deploy are shut down. More docs on this here.
If you need different tasks with different configuration running in different timings, I would suggest creating separate requests for them. Then, they are completely independent and a deploy of one will not impact the others.

Great thanks, closing this.


Confliction between atan and atan2

With angle unit defaulted as deg, the built-in function of atan and atan2 gives out different angle units, the codes are as follows:
atan(1)
atan2(1; 1)
b = atan2(sqrt(3); 1)
b = atan2(sqrt(3); 1)|deg
atan2(1; sqrt(3))

@idealkindom thank you for reporting this.
Yes, unlike other trig functions, atan2 is not unit-aware. It returns the answer always in radians.
There was a reason for that, but now i cannot remember why we did so. I will try to recall.
In all cases, it is good to know that.

@idealkindom thank you for reporting this.
Yes, unlike other trig functions, atan2 is not unit-aware. It returns the answer always in radians. There was a reason for that, but now i cannot remember why we did so. I will try to recall. In all cases, it is good to know that.

How about reconcider my earlier suggestion that let radian or 1 be default for trigometric function, why deg is a unit that must be typed for recognition.

OK. I made the atan2 function units aware to match the others.
https://github.com/Proektsoftbg/Calcpad/releases/tag/v6.1.7
I cannot find any reason why we have not changed it so far. Thank you for reporting this.
About making radians to be default and using  degrees explicitly, you can set it on your computer and work like this. There is no need to change anything in the program now. Also it is not good to restrict everybody to this only option. Lets users to be able to choose. For me personally, it is more convenient to have degrees by default.


Downloading a dictionary contents does not include imported dictionary contents

Create two dictionaries
Put some words in both
Make one dictionary 'import' the other
Download contents of that dictionary
It doesn't include the contents of the other, I would expect it to if the dictionary is to act as if the imported contents are an extension of it's own...

Apparently this is the correct behaviour....I'll get me coat


(Feature) Package go-feature-flag in an API to use it with different language

Requirements
Package go-feature-flag with an API on top to make it usable for other languages.

Create an API on top of the SDK.
A docker package should also be available to use as a sidecar container.
It should probably be in another repository.

What about RPC? 🤷♂
RPC can be used both locally and over a network, maybe that makes sense?

For information I am working on a new repository called go-feature-flag relay proxy that will expose go-feature-flag with a rest API.
There is also a plan to extent it with GRPC support.
As soon as the repository is ready I will link this new software here.


fix: use external_url in alert source

Issue
Closes #581.
Closes https://github.com/canonical/alertmanager-k8s-operator/issues/206.
Not sure why we were using the route_prefix from the self.external_url (from ingress), but adding it to the self.internal_url (which is always the FQDN).
Solution
Use the external URL instead of the internal one.
Screenshots
Before:

After:

Testing Instructions
juju deploy alertmanager-k8s --channel=latest/edge alertmanager
juju deploy prometheus-k8s --channel=latest/edge prometheus
juju deploy traefik-k8s --channel=latest/edge traefik

juju relate prometheus:alertmanager alertmanager
juju relate prometheus:metrics-endpoint alertmanager
# observe the watchdog alert's source link (internal url)
juju relate prometheus:ingress traefik
# wait for things to settle
# observe the same alert's source link again and try to click it!

Upgrade Notes
Anything that refreshes the Pebble plan will activate this change, so the upgrade should be seamless.

A unit test has been fixed :)
I also skipped the tests we were XFail-ing, because they'll just waste time in integration tests without contributing :)


SoundLibrary.add takes AudioBuffer as source type

Resolves

Resolves #184

Test Coverage

[x] code lint
[x] unit test

Nice work! Thank you.


ArkOS frequent reboots just to get to the dash, exact issue as issue #504

after buying several samsung and sandisk cards, the issue remains, arkos boots to the arkos screen, just sits there.  i have to reboot several times sometimes up to 8 times before it loads emulation station. i suspect it has something to do with the sdcard reader driver.
this is rg353v.  Jelos and stock work perfect
after it finally booted i was able to update to the latest version 10062023
but still problem persists.
even if i take the second sd card out , the one with the roms, the problem is the same.

How did you image the card?

Any update on my question?

Apologies for the delay.  I first wipe all partitions from the card, then I formatted the card at exfat. The i tried using Rufus, win32 disk imager. I also tried balena.  Yesterday I received my second rg353v( my 10 year old wanted one) and I'm having the same issues.

Don't use Balena Etcher.  Try this tool and see if you have any better results.
https://www.alexpage.de/usb-image-tool/download/
Also be sure you're using the updated image I uploaded yesterday.

Are there any logs we can look at to see where it's failing? Also when I'm re imaging the card some times it takes 2 or 3 reboots before the process can start, same issue on both rg353v and different sd cards, but both systems work fine with jelos and stock

Only when you get to the OS.  From what you're describing, the unit doesn't get to the OS.

It gets to the os after several reboots

i cant find a donwload link for the new version you menitoned you posted yesterday. can you post the link?

They are located here: https://github.com/christianhaitian/arkos/wiki#download-links

got it, downloading now, will try that along with our imaging app, will report back soon.
than you  :)

tried it, its now stuck on the arkos screen

rebooted, 3rd time, it now says arkos 2.0 (10062023)
umount: /roms: not mounted, just stuck there.  there is no sd card on the second slot

rebooted again, now its gone further to the bluscreen with white window easyroms partition expansion, seems to be stuck again.
rebooted again, not its continuing the process,

At that point, it's in the OS.  It's just not able to continue for some reason without issues.  Have you fully charged the device?

ran thru all the white letters, fast scrolling, rebooted itself, stuck on arkos screen again,, rebooted it, messaga about arkos with little dots, then black screen, froze at black screen. rebooted, arkso 2.0 (10062023) blinking dots, stuck with blinking dots

I left it at the blinking dots, but then after a few minutes, i got a black screen.  just stuck at the black screen

Have you charged the device?

rebooted again, now it loaded arkos, im at the dash, shutdown, turned on again, emulation station loading, black screen again. the device is at 92% battery

rebooted again, dash loaded again.

maybe we can look at logs since it loaded?

just to test, i hit quit , restart, this time it loaded emulation station, shutdown to test, turned on again , stuck on arkos screen again, its inconsitent, pretty much same simptoms as before.

Are you sure you sd cards are authentic Samsung or Sandisk cards?  I don't experience any of this on my 3 rg353v devices.

yes, i have 2 sandisk and 1 samsung, i able to reflash stock and jelos on both and they all work perfectly, and im having the same issue on 2 different rg353v consoles. i just restarted again, and its stuck on arkos again. could it be something with the card reader driver?

same driver used in JELOS.  again, I use the same image on my 3 rg353v without these issues and there are others who do as well.  Just some devices are compatible with ArkOS and some are not for one reason or another.  Although, I wonder if disabling the fake overclock may help in your case since you're able to boot sometimes.

Another difference, both JELOS and the Stock OS don't enable rumble support.  Perhaps that could be a source of instability for some units as well.  Going to try disabling that and the fake overclock give you an updated dtb to place in the fat32 BOOT partition and see if t makes a difference for stability for you

Try replacing the existing rk3566-OC.dtb file in the fat32 BOOT partition with the one size the attached zip file.  You can do this with the sd card inserted in your computer.  Then try booting ArkOS with the sd card and see if it makes a difference with stability.
rk3566-OC.zip

Will try that by the way I rebooted again since it froze after I clicked on Wi-Fi, I rebooted and it got stick on the blinking dots , I click volume down and it looks like it's doing something but it's stuck on some kind of loop. Will try to upload a photo now

If I click the down volume again it brings me back to this

now its freezing up on the dots, :(

going to re image the sandisk, that was the samsung, will report back

Froze there. This is using the re images Sandisk card

Going to head to Best Buy to buy another sd card

Rebooted and now have this message

Looks like it's failing something during boot froze here

That's with the file you gave me

Well, I don't know unfortunately.  Another idea is to try a non quality sd card see if that acts differently.

Interesting, I just saw a report in the RGH Discord where someone was having a similar issue as you.  Stock OS worked fine but ArkOS was having mounting issues and UnofficialOS (old fork of JELOS).  The person decided to order replacement unit through Amazon and that unit works just fine with both OS.

ok, so picked up a new sandisk from BestBuy, same steps but same results.  i guess these 2 units have something different that make it not work with arkos, i got mine china seller.  :( ill just stick to stock os for now i guess.
thanks for all your help. your OS looks amazing, but i have not tried it yet, maybe one day I will lol

not sure why, mut jelos latest version runs perfect, and so does stock.

would you mind sending me an older version of your os?  i read on reddit , another person had the same issue, but when he used 1.7 it worked perfect.  I just want to test to see whats going on

There was never a 1.7 for this unit.  1.7 existed on rk3326 devices last on the rg351p.  Be careful what you read on reddit.  There's a lot of misinformation on there.  Thanks for your time and sorry we couldn't figure this out.

do you have another version older? id like to give it a try if you dont mind, the first version for the rk3326? and no worries, I apreciate your time

That will not work on this device.  The oldest version I have is the previous version you already tried.  Sorry.

ok, :(


IGNITE-20995 Add more integration tests for tx recovery on unstable topology

https://issues.apache.org/jira/browse/IGNITE-20995

There are no tests for parallel recovery cases, but they seem to be important to me

Parallel tx recoveries happen on two replicas of commit partition on different nodes, both processes were started at a moment when the corresponding replica was the primary one. This also shouldnt break anything.
There are two parallel recoveries on commit partition initiated by tx state requests and a commit process initiated by coordinator that is already dead. Transaction is committed. Both recovery processes get correct commit timestamp and end up with correct write intent resolution (one recovery process receives commit timestamp from exception thrown by raft client, another should receive it from the completed finish future of local FINISHED state).


run_plugin_tests does not copy out xmlrunner reports

[x] main
[x] 4-2-stable

These results (*.xml) are usually in /var/lib/irods/test-reports at the time the tests are finished.

Ready for checkboxes and closing, I think?

Let's reopen this to fix the confusing error messages

Nevermind, see #35. 🤡


Update example for creating alias for param

Wrapping alias_method into the class << self, because alias_method :dependency, :param does not work for me and produces such error:
Failure/Error: alias_method :dependency, :param

NameError:
  undefined method `param' for class `BaseService`'

thanks!


datagrid clr-dg-action-overflow is not shown until the page is clicked on

[X] bug
[ ] feature request
[ ] enhancement

Expected behavior
The icon for the actions should be shown as soon as items are rendered in the datagrid - even if the data is loaded from a server.
Actual behavior
Currently data is loaded from the server, then the datagrid is populated and rows are created. When the data is added to the grid, the rows do not initially show the actions icon. The icons do not render until the page is clicked on.
Reproduction of behavior
https://plnkr.co/edit/0l3Mq1rIVBGJF5z0Zb7n?p=preview
Environment details

Angular version:
4.0.X

Clarity version:
0.9.X

OS and version:
macOS Sierra v10.12.4 (16E195)

Browser:
Chrome v57.0.2987.133 (64-bit)

https://github.com/vmware/clarity/commit/505a3ad4438c3e5fb2d7a5ded917ee49bcab8071 fixes the column alignement issue, and I'm thinking it might fix the action icons not showing at the same time. Let me check this today.
The commit in question will be part of this week's release.

@youdz : Did you get a chance to look at this?

So on a (possibly) related note, when trying to use the clr-dg-action-overflow component while using *ngFor rather than * clrDgItems to iterate over my list of items, it totally breaks things and causes the following error: ERROR Error: ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: 'false'. Current value: 'true'. This seemed to be the case with both Clarity 0.9.0 and 0.9.1.
You can see an example of this in the plunker below:
https://plnkr.co/edit/j9QrmOGYrn2fqRscQ2Cp?p=preview
Please let me know if you want me to create a new issue for this.
Unfortunately the original case where I found the necessity of clicking the page to see the clr-dg-action-overflow correctly has evolved to requiring me to use server supplied data and now I need to iterate over my items with *ngFor. This site is in production so even a suggested work around for now would be highly appreciated.

Hi guys, any updates on this? I think my additional comment above may be an even more important bug/issue. Trying to use the clr-dg-action-overflow while iterating with an *ngFor totally breaks the datagrid. We're gearing up for a release soon and unfortunately I'm relying on this feature.

Hi @jbgarr, sorry this got lost. I'm literally looking at it right now, investigating the cause and potential solution. I'm not sure I'll have a good answer tonight, so I just wanted to post to at least tell you we were working on it. Hopefully my next post will have more information. 😛

Quick update: I identified the root cause, and I have two fixes working:

An ugly-ish hack with setTimeout(), which is simple but makes a few things asynchronous. If it impacted only our unit tests, I'd be okay with that, but I'm afraid it might introduce asynchronicity in consumer's unit tests.
A slightly more complex cycle that re-triggers change detection on rows when actions or expandables rows (we have the same issue with the caret as with the ellipsis) change in a specific way. Much cleaner, stays synchronous, but involves more work and triggering a second change detection on each displayed row of the Datagrid might impact performances.

So I'm trying to evaluate the risk of each, maybe share with the team for a sanity check, and I'll submit a PR as soon as possible with the chosen fix.

@youdz Great, thanks for the updates on this. I'll stay tuned for the incoming fix.

I am able to reproduce this issue. As a workaround I've added datagrid action buttons instead of using the menu, but the menu would make things more compact and elegant. I'll keep an eye as well. Thanks for keeping us updated.

any update on this?

We're still working on this, it impacts more than just this issue and there is a lot of research involved to produce a maintainable fix (to this and half of the other "Expression has changed after it was checked" errors for Datagrid out there).

@youdz FYI, my plugin sample is affected by this bug as well.  --Laurent

I just wanted to note that this issue also seems to crop up if the expandable rows clr-dg-row-detail component is used as well. Unfortunately in that case there is no simple work-around that I can think of. Any suggestions until a full fix is implemented?
For more details please see this plunker:
https://plnkr.co/edit/KhiYZrCPqbTCGaAgTrJ5?p=preview

@jbgarr for a work-around you could try using ChangeDetectorRef as shown in this plunker:
https://plnkr.co/edit/NRxLPSAzGlDoHNbTz2ID?p=preview

@TechRazor Awesome! That takes care of it for now. Thanks for the quick fix!


bad URI(is not URI?)

I'm setting PGSLICE_URL as
postgres://user:password@localhost:5432/database
But when running command
pgslice prep table_name column_name period
i got error:
 bad URI(is not URI?): postgres://user:password@localhost:5432/dbcounter2 (URI::InvalidURIError)

How to fix it ?

Make sure there are no spaces or special characters in the URL. If your password contains non-URL safe characters, be sure to use percent-encoding to encode them.


JSON array of object saved as ORC format

Hi
In some situations Kafka messages may contains JSON array of objects. Each item on the list have the same schema. This patch allows to read messages like this and save each list item as a separate records.
This patch should not have performance implications as root JSON element type was checked before.
It was expected to receive JsonObject but in our scenario it is JsonArray.

It looks like the travis build is failing for this PR: https://travis-ci.org/github/pinterest/secor/builds/683900104
The error doesn't seem related to your PR, and probably a transient error.
Can you do a dummy checkin (e.g adding a comment in the code) to trigger the rebuild?

Now it passed: https://travis-ci.org/github/pinterest/secor/builds/684173873

Great, I will merge the PR, thanks.

No problem, thanks!


Push changes to hex.pm

Hi, could you please consider pushing the changes to Hex.pm? I'd greatly appreciate it 😄 .
Greetings ✌🏻

Pushed v0.1.1 to Hex.pm. Take care!


Not prefixing 'cursor' property

Some cursor properties like 'grab' and 'grabbing' don't yet work in webkit without a prefix. Prefixfree doesn't seem to add these yet.

https://github.com/LeaVerou/prefixfree/pull/6125
added it in https://github.com/LeaVerou/prefixfree/commit/055df27a44f036980fd5d2e31aa81ffbd13a15c9


feat: draw hyper graph by graphviz

draw hyper graph by graphviz
Proposed changes
Issue Number: close #13750
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

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

Has unit tests been added:

[x] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[x] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

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

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

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 39.13 seconds
load time: 593 seconds
storage size:<PHONE_NUMBER>0 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221028033048_clickbench_pr_35084.html

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.54 seconds
load time: 559 seconds
storage size:<PHONE_NUMBER>0 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221028035022_clickbench_pr_35076.html


feat: add vector support for scripts

closes: https://github.com/FuelLabs/fuels-rs/issues/712, https://github.com/FuelLabs/fuels-rs/issues/655
Scripts now support vectors as arguments.
This PR is the joint work of @iqdecay, @segfault-magnet  and me. I decided to open the PR ASAP as this feature is highly requested by users.
This PR is based on https://github.com/FuelLabs/fuels-rs/pull/716 as we needed logs for debugging. I recommend we merge the logs PR first and than this one.

I  marked this PR as draft until we merge https://github.com/FuelLabs/fuels-rs/pull/716

@iqdecay found a bug regarding how we calculate the script offset. Until now we were creating it form default of the ConsesnsusParameters. If the max_inputs field was set to something else than the default our scripts could not be called ass the offset are wrong. I updated the code to take into consideration the actual consensus_parameters and added a test for contract_calls

I also had to change the ScriptCallHandler to accept UnresolvedByets instead of the actual script data. This was done so that I can querry the provider and get the current consensus_parameters.


Whitelist private endpoint for reform-scan

Closed in favour of https://github.com/hmcts/cnp-jenkins-config/pull/580


goog.module.ModuleManager - incorrect error when an execOnLoad callback throws an error

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=232

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


BatchCollection LaserMode "spectrum" broken on SiG

Seems to be running afoul of RamanModeFeature's automated attempts to disable acquisition_laser_trigger_enable.
The whole thing is a mess.
Basically we need to re-factor BatchCollection to provide much more direct control of WasatchDevice / FeatureIdentificationDevice, rather than trying to do everything through WasatchDeviceWrapper and "settings".
Maybe there's a short / simple fix to this for now, but fundamentally BatchCollection needs redesigned with lower-level timing control.

CC ran into this yesterday...

fixed in #398


CI pass even when there are errors during build/test run

Shouldn't Travis run tests located in tests folder so we know when something is broken in a commit?

Also, it passes when there's a build error
1 error generated.
[796/1396] Building CXX object contrib/poco/Data/CMakeFiles/Data.dir/src/StatementImpl.cpp.o
ninja: build stopped: subcommand failed.
+date
Mon Jul  9 21:06:43 UTC 2018
The command "utils/travis/normal.sh" exited with 0.

@alexey-milovidov oh I see. I tried to break the test and the build passed so I though it's not run.
Probably the issue is just error code 0 when there is an error

We have two Travis jobs for each commit.
This is for your pull request:
https://travis-ci.org/yandex/ClickHouse/builds/401968092?utm_source=github_status&utm_medium=notification
Second job is failed as expected:
https://travis-ci.org/yandex/ClickHouse/jobs/401968094
Although the first job passed. As I remember, it is designed as a "dry run" for different environment.


Remove the explicit 'table' part from Table Coordinate

The puzzle 31-fe4a3a60 from #31 has to be resolved:
https://github.com/nergal-perm/java-decita/blob/5671fb31397698a44e9e8dc750de4fc24d950778/src/main/java/ru/ewc/decita/ComputationContext.java#L102-L104
The puzzle was created by Евгений Терехов on 20-Jan-24.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd the puzzle #36 is still not solved.

@0pdd the only puzzle #36 is solved here.

The puzzle 31-fe4a3a60 has disappeared from the source code, that's why I closed this issue.


Add static site docs and sample CSV for patient roster upload

Background
The only guidance on our site for a patient bulk import is in a (pretty hidden) PDF on the static site. As we open up the feature to end users, we should have more documentation and examples for users.
Action requested
Create a page on the static site with the content from the bulk upload PDF, and attach the sample spreadsheet sent out by the support team (with any updates necessary).
Acceptance Criteria
Bulk upload guide and sample spreadsheet exists on the static site as a page, along with any content updates necessary to open the feature to users.

@lauraykerr does this ticket capture the content/eng work needed for patient bulk uploads on the static site? Feel free to edit if not!

@emmastephenson This is perfect, thank you!

Staged, doing some final QA.

@emmastephenson Just putting in a note about the sample CSV that we talked about at sprint planning — can you share when you have a chance, and I'll plan to upload to the static site? Thank you!!

@lauraykerr sure thing! Here's a sample in the private repo, let me know if you need access: https://github.com/cdcent/simplereport_docs/blob/main/runbooks/csv_operations/patient_upload_final.csv
Please note that all of this is completely fake data! Generated using the faker library, so there's no actual PII here. You may want to take off the numbered middle name, I just added that to make development testing a little easier 😄

@lauraykerr giving you repo access is more complicated than I thought it would be, so I've just emailed you the file directly!


Add Ruby Koans

Hi @rfair404
I think link is not working at least on my side because web site it is not with https so please remove that part and add it after Microsoft Virtual Academy. Thank it will be ok.
Thanks !


types: support app.component to be defineComponent

fix CI issue https://github.com/vuejs/ecosystem-ci/actions/runs/6925726707/job/18836836250
Probably because the type is just to complex to resolve as Component
I've seen this issue, other way to fix is to offload the component to a variable
// Too complex
awesomeFunction(defineComponent(/* ... */))

// works
const c = defineComponent(/* ... */)
awesomeFunction(c)

/ecosystem-ci run


Find out why some tests fail when targeting the iOS Simulator

Some unit tests succeed when targeting macOS but fail when targeting the iOS simulator.
They are marked with // TODO: Test fails on iOS Simulator.

Just verified on an actual iPhone; fails as well.

Here's a list of all the APIs I've found to SIGSEGV up until now:
System.Type.GetType
System.String.IndexOf
System.DateTime.Now
System.Globalization.CultureInfo ctor
System.Version ctor
System.Version.TryParse
System.Convert.ToInt32
System.Reflection.Assembly.GetName
System.Decimal.TryParse

Setting InvariantGlobalization in the csproj to true works around the crashes but makes tests using a specific culture fail.

Can also be fixed by having <RuntimeHostConfigurationOption Include="ICU_DAT_FILE_PATH" Value="icudt.dat" /> in the csproj and including a copy of icudt.dat in the bundle.
The file can be found in the NuGet cache (ie. ~/.nuget/packages/microsoft.netcore.app.runtime.nativeaot.ios-arm64/8.0.0-preview.6.23329.7/runtimes/ios-arm64/native/icudt.dat)

Unfortunately that doesn't work in a XCTest bundle because the runtime locates the file by using -[NSBundle mainBundle].
See https://github.com/xamarin/xamarin-macios/issues/18471#issuecomment-1634287195

Might be able to work around this by calling AppContext.SetData("ICU_DAT_FILE_PATH", "<path>") early on in startup.

Using System.AppContext to set the icudt.dat file path at runtime does indeed work.
See https://github.com/royalapplications/beyondnet/commit/f2d4f2b6d51641ac43412eb70bae9c19d4b11ab8.


Create connection from a string

As mentioned on Reddit, I'm planning to add support for migrations for Cassandra in https://github.com/Keats/dbmigrate
Would it be possible to add a method to cdrs to get a connection from a string like cassandra://username:password@host:port/keyspace?

@Keats It seems it makes sense to provide an option as a method of CDRS non-ssl mode. Obviously it should use password authenticator.
So, it might look like:
CDRS::from_string("cassandra://username:password@host:port/keyspace")

There is one think here I don't understand at the moment it's how this keyspace will work. From one hand, to apply this space to each query by default we'll need to keep it somewhere inside CDRS. From other hand, it can be set from:

this connection string;
set keyspace query
query itself SELECT * from keyspace.query

To have this feature (keyspace) we may want to have some smart merge strategy for that.
Until it is clear we could provide from_string that could accept a connection string without keyspace cassandra://username:password@host:port

It would be also cool if one could select keyspace in advance so it would not repeat in future queries.

e.g.: select * from keyspace_rembered.table ..

@ernestas-poskus It's already possible via set keyspace query.

Any update on that? I don't know about the keyspace itself as I've never used Cassandra myself, it seems similar to a schema in SQL databases?
I was just looking at https://github.com/mattes/migrate/tree/master/driver/cassandra as a reference, maybe the Golang driver can be used to see how they implement it?

@Keats : cassandra://username:password@host:port assumes that we have only one host; cassandra in production would have an array of hosts. It is a ring topology. At any point in time a node can die or a node would be added into the cluster. the client would ( not supported by cdrs currently)  poll the cluster periodically to see which nodes are up/down and add/remove nodes ip from it's memory !!!
Say we have  a 4 node cluster (<IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>) with a replication factor of 2. so <IP_ADDRESS> data would be copied on <IP_ADDRESS> and similarly <IP_ADDRESS> and <IP_ADDRESS> are pairs.   we have provided cassandra://cdrs:cdrpassword@<IP_ADDRESS>:9042 as intial cassandra connection string  there is no real guarantee that <IP_ADDRESS> would be alive for forever; but since the data on <IP_ADDRESS> 's copy is on <IP_ADDRESS> cassandra server would serve the data out of 11 and the client application using the driver shouldn't be worried about this fact as the driver would abstract this transition behind the scenes.
I know I have gone into a totally different tangent with my explanation; but does this explanation make any sense?

I see, and I guess you will need to have the cluster to agree on a schema (I guess?) so you probably need to have special handling to wait till the that happens before running the next one (in my case of a cli to run schema migration). Thanks for the explanation

yes precisely  !!!  providing an array of hosts (I deal with  cassandra cluster with 50 nodes) in a string seems to be less ergonomic.
Fun fact
on a side note every node knows about all other nodes; so mentioning a single node would be enough; to execute cql statements.  there I contradicted myself  !!!
Developers who wants to run application in production should not have a single node configuration; period ! if it a tool which run once in a while; if it fails we can change the host and retry.
I don't know how could we incorporate these 2 orthogonal features into the same library.

doesn't the driver discover the other nodes in the ring from the initial node(s) it connects to?

@cmsd2 Not yet, so far there is an option in excluding nodes which went down basing on server events


Make sure the macros don't require use statements

After converting a benchmark to use criterion (I have an extern crate statement but no use statement), I get the following error:
criterion_group!(benches, mybench);
Use of undeclared type or module `Criterion`

I know how to fix it, but I think the macro should only reference absolute names.

Hey, thanks for the suggestion! You're right, it should.


fix: bug in prime number

Description of Change

Checklist

[ ] Added description of change
[ ] Added file name matches File name guidelines
[ ] Added tests and example, test must pass
[ ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[ ] Relevant documentation/comments is changed or added
[ ] PR title follows semantic commit guidelines
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes:

Improve the documentation

That has to be done in another PR, not in this one.

It would be nice if you could improve the tests for this file

IMO, that should be done in another PR, as this PR is just to fix a bug in the algorithm. But if you folks want it like that, then we can request the PR author to do that. Thanks anyways. 🙂

It would be nice if you could improve the tests for this file

IMO, that should be done in another PR, as this PR is just to fix a bug in the algorithm. But if you folks want it like that, then we can request the PR author to do that. Thanks anyways.

If there is a bug then the author must add tests that cover the cases of that bug. Just to ensure that now things are proper.

It seems this is a duplicate of #1537. @ayaankhan98, @mishraabhinn, what should we do? 🤔

It seems this is a duplicate of #1537. @ayaankhan98, @mishraabhinn, what should we do? 🤔

Accept the pr who raised first :monocle_face:

Duplicate of #1537.


Implement search for basic attributes

Both in schema and generated model code, please enable Search for basic search argument:
SearchArg{
  field: String
  operator: String
  value: String
  searchArgs: [SearchArg]
}```

In the model provide an adequate resolver.

Done.
Example with person model:
curl -XPOST http://localhost:3000/graphql -H 'Content-Type: application/graphql' -d '{ searchPerson(input:{field:email, value:{value:"%science%"}, operator:like}){ firstName lastName}}'


feat: use peerid mismatch error type

Uses the go-libp2p PR in https://github.com/libp2p/go-libp2p/pull/2451.
Currently not mergeable as:

It fails with QUIC connections since quic-go hides the underlying error type as described in https://github.com/libp2p/go-libp2p/pull/2451#issue-1831773396
The go-libp2p PR is just a PR

Update: These were both fixed and included in go-libp2p v0.31.0

Codecov Report

Merging #29 (ce085a2) into main (486ce55) will increase coverage by 1.27%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
+ Coverage   42.23%   43.50%   +1.27%
==========================================
  Files           4        4
  Lines         322      308      -14
==========================================
- Hits          136      134       -2
+ Misses        154      143      -11
+ Partials       32       31       -1

Files
Coverage Δ

lib/identify.go
60.00% <100.00%> (+7.19%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


1.20.1 - Question About the Auto Repair perk

Hello,
I have to ask about the auto repair skill for the Crystal AIOT.
Is it meant to automatically fully repair the tool after a specified amount of time or does it repair the tool little by little over a specific time?
I sunk 50 skill points into Auto Repairing and it seems that it isn't repairing the AIOT, it also voided an enchantment from the indestructible enchantment mod when I reset the tool's skill points.
I have also put 50 points into the infinite unbreaking skill and it seems that the tool still gets damaged by the same amount of durability even at 50 percent.
Please let me know if this is intended or is a bug, thank you for a great mod!

It is meant to repair the tool slowly over time. It doesn't work while the item is in your hand. I did just mess with the though so I might have broken something.
I will look into the infinite unbreaking skill to make sure that is working properly.
The tools shouldn't be enchantable in the first place. I can check to see if that is a problem on my end or a problem with the other mod.
Thanks for the report!

I fixed a small bug related to durability and tools taking damage and made tools not be enchantable. I'm not sure about the auto repair and unbreaking issues, I tested those and they seemed to both be working fine with me, it might be some weird mod interaction but I'm not sure. If you are still seeing the issues I can maybe try to look into some interactions to see if I can find some issues.

Heya, got a small update that I wanted to share with you.
I tried out the unbreaking skill again and it works perfectly now, no damage is done to the tool at all.
I have however run into another problem with this mod.
I think it could be because of some new update with NeoForge or Forge itself but the crystal pickaxe has a very slow mining speed like breaking deepslate for regular stone.
I've tried disabling several mods but this one seems the be the cause of the issue, no other tools are affected as far as I know.
If you could check and make sure the tier of the crystal pickaxe is correct, I'd hope this would clear up any issue.

Sure I will check, can you let me know which version of Forge or NeoForge you are running?

NeoForge version is 47.1.79 - Tested and confirmed to have block breaking issue
Forge version is 47.2.0 - Also tested and found the same block breaking problem
I've tried all versions of Crystal Tools available for 1.20.1

I tested on both of those versions and was unable to reproduce the issue. What other mods are you using?

I had to upload several screenshots in order to capture all of the mods in my folder.
I hope this helps!


add '--syncdeps', remove pkg cache, squash docker layers

'--syncdeps' param added, so pacman will install missing dependencies
pacman cache has been cleaned
docker layers has been stripped

new uncompressed image size 796MB vs 872MB

Thanks for this PR @sfesenko! I was testing it out locally and I'm getting an error trying to build the docker image. Is this something you've seen?
error: failed to initialize alpm library:
(root: /, dbpath: /var/lib/pacman/)
could not find or read directory

No, I don't have this error.
> docker version

Client:
 Version:           20.10.8
 API version:       1.41
 Go version:        go1.16.6
 Git commit:        3967b7d28e
 Built:             Wed Aug  4 10:59:01 2021
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true

Server:
 Engine:
  Version:          20.10.8
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.16.6
  Git commit:       75249d88bc
  Built:            Wed Aug  4 10:58:48 2021
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          v1.5.5
  GitCommit:        72cec4be58a9eb6b2910f5d10f1c01ca47d231c0.m
 runc:
  Version:          1.0.2
  GitCommit:        v1.0.2-0-g52b36a2d
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

build as:
> docker build --no-cache -t nubs/arch-build . | tee arch_build.log

arch_build.log

Merged in all of your changes with the exception of the wheel/sudo stuff. I'd rather keep the user as unprivleged. Thanks for the PR!

with the exception of the wheel/sudo stuff

With --syncdeps makepkg will ask for build use password, that is not set:
We trust you have received the usual lecture from the local System
Administrator. It usually boils down to these three things:

    #1) Respect the privacy of others.
    #2) Think before you type.
    #3) With great power comes great responsibility.

[sudo] password for build:

Ah, thanks for following up. I added a new target to the buildfile for syncdeps. Would this work for your needs?
docker build -t nubs/arch-build-syncdeps --target syncdeps .

Pushed to docker hub as nubs/arch-build:syncdeps

makepkg doesn't refresh pacman cache, so if image is old more than day - this whole approach with --syncdeps won't work
So, It seems, that I'll need to re-implement it


Exclude only unnecessary files from being staged

Currently all files except files with .php extension are excluded from being staged in make-release.sh. This has been changed to stage all files except unnecessary files.

Currently all files except files with .php extension are excluded from being staged in make-release.sh. This has been changed to stage all files except unnecessary files.

files other than .php extension like schema.ser in HtmlPurifier, which was causing the error file_get_contents(HTMLPurifier/ConfigSchema/schema.ser): Failed to open stream: No such file or directory.


Review all the titles for the Parasol theme

No more "Streamlining insurance claims with OpenShift AI"

should be done. in #28


Question: How to use multiple orm's together?

I can't figure out how to use multiple orm's together. We initially used only ActiveRecord, and database_cleaner worked great.
Recently we introduced mongodb to the stack, but am unable to figure out the configuration to have mongoid and active record using database cleaner.
Our current database_cleaner.rb looks like this -:
RSpec.configure do |config|
  config.use_transactional_fixtures = false

  config.before(:suite) do
    DatabaseCleaner.clean_with(:truncation)
  end

  config.before(:each) do
    DatabaseCleaner.strategy = :transaction
  end

  config.before(:each, js: true) do
    DatabaseCleaner.strategy = :truncation
    FactoryGirl.reload
  end

  config.before(:each) do
    DatabaseCleaner.start
  end

  config.append_after(:each) do
    DatabaseCleaner.clean
  end
end

i'm in the same situation :/

@pratik60 did you read "How to use with multiple ORM's" in the docs? ( https://github.com/DatabaseCleaner/database_cleaner#how-to-use-with-multiple-orms )
Set up all your connections via DatabaseCleaner[:orm, { :connection => :something }]

@eduzera @pratik60 Did you try with the example mentioned in the documentation? Please let me know.

Updated it to this
RSpec.configure do |config|
  config.use_transactional_fixtures = false

  config.before(:suite) do
    DatabaseCleaner[:active_record].strategy = :truncation
    DatabaseCleaner[:mongoid].strategy = :truncation
  end

  config.before(:each) do
    DatabaseCleaner[:active_record].strategy = :truncation
    DatabaseCleaner[:mongoid].strategy = :truncation
  end

  config.before(:each, js: true) do
    DatabaseCleaner[:active_record].strategy = :truncation
    DatabaseCleaner[:mongoid].strategy = :truncation
    FactoryGirl.reload
  end

  config.before(:each) do
    DatabaseCleaner.start
  end

  config.append_after(:each) do
    DatabaseCleaner.clean
  end
end

and in each of them I now get
12) DpaController Product Feed fetch product feeds
      Failure/Error: Unable to find matching line from backtrace
      NoMethodError:
        undefined method `default_session' for Mongoid:Module
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/database_cleaner-1.4.1/lib/database_cleaner/mongoid/truncation.rb:30:in `session'
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/database_cleaner-1.4.1/lib/database_cleaner/moped/truncation_base.rb:26:in `collections'
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/database_cleaner-1.4.1/lib/database_cleaner/moped/truncation_base.rb:14:in `clean'
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/database_cleaner-1.4.1/lib/database_cleaner/base.rb:86:in `clean'
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/database_cleaner-1.4.1/lib/database_cleaner/configuration.rb:79:in `block in clean'
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/database_cleaner-1.4.1/lib/database_cleaner/configuration.rb:79:in `each'
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/database_cleaner-1.4.1/lib/database_cleaner/configuration.rb:79:in `clean'
      # ./spec/support/database_cleaner.rb:28:in `block (2 levels) in <top (required)>'
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/rspec-retry-0.4.4/lib/rspec/retry.rb:53:in `block (3 levels) in apply'
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/rspec-retry-0.4.4/lib/rspec/retry.rb:44:in `times'
      # /Users/pratikbothra/.rvm/gems/ruby-2.2.2/gems/rspec-retry-0.4.4/lib/rspec/retry.rb:44:in `block (2 levels) in apply'

@etagwerker I believe it is an issue with mongoid 3.2 as well

Seems fixed after upgrading to 1.5.1.
Can we do a new release as well?


Update OrientDB driver

Description
I can't connect to the latest OrientDB server with the default driver because version 1.7 is too old.
DBeaver Version
DBeaver 24.3.0
Steps to reproduce

Download portable database version from https://orientdb.org/download

Extract files from the archive

Run server .\server.bat or .\server.sh from the bin directory and set the default password for the root user. See more here http://orientdb.org/docs/3.0.x/gettingstarted/Tutorial-Run-the-server.html#starting-the-database-server

Open DBeaver and create a new connection to OrientDB

Press the Test Connection ... button - the connection fails due to the driver error.

Stop the server http://orientdb.org/docs/3.0.x/gettingstarted/Tutorial-Run-the-server.html#stop-the-server

verified


Disable zooming when pan in progress

This fixed #316

Sorry, I missed this earlier. Thank you for the PR


NTR: alpha-ketoglutarate reductase activity

Hi,
Here is some background for the NTR
In the following paper:
Saccharomyces cerevisiae Forms D-2-Hydroxyglutarate and Couples its Degradation to D-Lactate Formation via a Cytosolic Transhydrogenase. PMID: 26774271
the yeast gene DLD2, is demonstrated to function as an D-2-hydroxyglutarate (D-2HG) dehydrogenase, converting D-2HG to 2-oxoglutarate (aka alpha-ketoglutarate). In GO there is a term for 2-hydroxyglutarate dehydrogenase activity (GO:0047545) with synonym alpha-ketoglutarate reductase activity (important for what follows), and a term for (R)-2-hydroxyglutarate dehydrogenase activity (GO:0051990), a better fit since its specific for the R/D enantiomer.
Later in the paper they show that in yeast both SER3 and SER33 function as alpha-ketoglutarate reductases, catalyzing the conversion of alpha-ketoglutarate to D-2HG. This is the reverse reaction to the one described above. I posted this scenario to go-discuss and was advised to request a new term “alpha-ketoglutarate reductase activity” and request the removal of this string as exact synonym for 2-hydroxyglutarate dehydrogenase activity (GO:0047545), as this is the reverse activity.  I have attached figure 13 from the paper, the proposed yeast metabolic network involving these genes, in case this helps

I would therefore like to request the following new function term:
Term: alpha-ketoglutarate reductase activity
Definition: Catalysis of the reaction: alpha-ketoglutarate + NADH = D-2-hydroxyglutarate + NAD+
Synonyms: 2-oxoglutarate reductase activity, 2-ketoglutarate reductase activity; D-2HG reductase activity
Instance of GO:0016657 oxidoreductase activity, acting on NAD(P)H, nitrogenous group as acceptor
Definition reference: PMID: 26774271
E.C. number: I cannot find the appropriate EC number sign ExPASy
In addition, please remove the exact synonym “alpha-ketoglutarate reductase activity” from term GO:0047545 2-hydroxyglutarate dehydrogenase activity, as this term describes the reverse activity.
Cheers,
Rob

Hi Rob,
We discussed this on a recent call and decided to keep the term representing the union of the forward and reverse reactions. So what I have done is made two separate terms for each of the forward and reverse reactions. Here are the new terms:
[Term]
+id: GO:0061758
+name: 2-hydroxyglutarate dehydrogenase activity, forward reaction
+namespace: molecular_function
+def: "Catalysis of the reaction: (S)-2-hydroxyglutarate + acceptor -> 2-oxoglutarate + reduced acceptor." [EC:<IP_ADDRESS>, GOC:dph, MetaCyc:2-HYDROXYGLUTARATE-DEHYDROGENASE-RXN]
+synonym: "alpha-ketoglutarate reductase activity, reverse reaction" EXACT [GOC:dph]
+xref: Reactome:REACT_24926 "(S)-2-hydroxyglutarate + FAD => 2-oxoglutarate + FADH2, Homo sapiens"
+xref: RHEA:21253
+is_a: GO:0047545 ! 2-hydroxyglutarate dehydrogenase activity
+created_by: dph
+creation_date: 2016-03-07T08:28:58Z
+
+[Term]
+id: GO:0061759
+name: alpha-ketoglutarate reductase activity
+namespace: molecular_function
+def: "Catalysis of the reaction: 2-oxoglutarate + reduced acceptor -> (S)-2-hydroxyglutarate + acceptor." [EC:<IP_ADDRESS>, GOC:dph, PMID:26774271]
+synonym: "2-hydroxyglutarate dehydrogenase activity, reverse reaction" EXACT [GOC:dph]
+synonym: "2-ketoglutarate reductase activity" RELATED [GOC:dph]
+synonym: "2-oxoglutarate reductase activity" RELATED [GOC:dph]
+synonym: "D-2HG reductase activity" RELATED [GOC:dph]
+xref: RHEA:21254
+is_a: GO:0047545 ! 2-hydroxyglutarate dehydrogenase activity
+created_by: dph
+creation_date: 2016-03-07T08:38:57Z


extend GPU DYAMOND run length

I accidentally merged https://github.com/CliMA/ClimaCoupler.jl/pull/659 without increasing the longrun sim length. We should see if the simulation can run for longer (50 days?), or if it will fail

update: the simulation fails after ~3 days, so we won't extend the simulation length right now (see failed build here). It tries to exceed the available memory, so we can revisit this after we lower the allocations in ClimaCoupler


Delete CNCI when it is no longer in use.

If we do not need the CNCI because the tenant has deleted all of their instances, delete the CNCI as well in order to keep it from hanging around forever.
Fixes: #1307

Coverage decreased (-0.1%) to 65.658% when pulling efb1e504e4e400d9f7cf0b3d8b34e755442c9d5d on kaccardi:topic/cnci-cleanup into 4e8babb9be6ef28eda040231ddeefb62610adb8f on 01org:master.

The semaphore failure actually looks real.
base_bat_test.go:88: Starting an instance should increase instance count on node &{9ced9217-c20c-422e-bdae-afcacabc238b 0001-01-01 00:00:00 +0000 UTC READY 7983 5801 60921 52549 1 4 1 1 0 0} &{9ced9217-c20c-422e-bdae-afcacabc238b 0001-01-01 00:00:00 +0000 UTC READY 7983 6524 60921 52721 1 4 1 1 0 0}


Can we override a file based on environment

Like having two files ex: config.dev.xml and config.prod.xml
config.dev.xml is used with dev env and replace the current etc same with prod file
is there a way to do so ?

Pretty sure that is not supported in a way you describe, however there is a very similar feature, which could answer your needs - appConfigs - https://renative.org/docs/overview/introduction#appconfigs.  As you can see in docs it allows configuration of platforms, plugin overrides, asset file injections and more, so hopefully this resolves what you need


Add option "-r"

If there is "-r" option fullGitHistory will return history object.
Add tests for this option.

fullGitHistory always return undefined, but callback has args (error, history), if there is option "-r".

Done. Options "-o", "-no", and "-r" operate independently of each other.


Renaming puzzles should rename Discord channel

Actually this looks to be not an issue, think I got confused

Actually this looks to be not an issue, think I got confused


Disable built-in link provided

monaco-editor version: 0.20.0
Browser:: Whatever ships with Electron 9.1.0
OS:: macOS 10.14
Playground code that reproduces the issue:
monaco.editor.create(document.getElementById("container"), {
	value: 'http://foo.unknownasd',
	language: "text"
});

There seems to be a built-in link provider for all languages, which detects a link in that sample snippet of text. How do I disable it? Possibly only just for a particular language.
I need this because I'm providing my own link detection logic so there seems to be some unnecessary work going on currently, plus I don't think a link should be detected at all in that snippet of text because "unknownasd" is not a IANA TLD.

Something like this seems to work:
Monaco.languages.registerLinkProvider ( { language: config.id, exclusive: true }, data.providers.links );

Despite of what TypeScript think should work.

We have a links editor option, but I believe that turns off all link providers.


5,5',6,6'-Tetrahydroxy-3,3'-biindolyl (CHEBI:174128)

This is a ring assembly of two identical components so I presume the IUPAC name should be 1H,1′H-3,3′-biindole-5,5′,6,6′-tetrol.
ChemSpider 10209313 also gives the ACD/Index Name, [3,3′-bi-1H-indole]-5,5′,6,6′-tetrol.
Pleased add a crossref to patent EP3148648. Here it is called "3,3′-bi-1H-indole-5,5′,6,6′-tetrol".

now added. thanks


XML Encoder/Decoder

First off: Awesome package! My Praises!
I had a question regarding extensibility: Is it possible with de Encoder/Decoder to return XML documents in this format as well? if so how? Just some guidance would help a lot.

Hey! Glad you like the package and sorry for taking a bit of time to reply.
In theory this would be possible as the encoder/decoder are interfaces.
You'd need to write an XML encoder that implements this interface:
https://github.com/neomerx/json-api/blob/v1.x/src/Contracts/Encoder/EncoderInterface.php
And a decoder that implements this interface:
https://github.com/neomerx/json-api/blob/v1.x/src/Contracts/Decoder/DecoderInterface.php
For you to tie this into my extensions to the neomerx/jsonapi package. you'd need to return an array from the decoder that is in the expected JSON API document format. (I.e. your array must look like it would if it had come in via JSON rather than XML.)
If you get to the point of having an encoder and decoder that work behind these interfaces, let me know and I'll give you some guidance as to how to wire it in. It's possible I might have to make one or two minor changes to this package to do that, but it totally should be possible for you to provide your own encoder/decoder so I'd be happy to make any adjustments that are needed.

@Nizari no problem! yes that packages are quite abstract underneath. I'm actually planning to merge my cloudcreativity/json-api package into this one, as we now don't use it for anything except in Laravel - hoping that merging them means we can simplify the code a bit.
I'll close this issue for the moment but do feel free to open issues with follow up questions once you start digging into things.

When I have written a custom Encoder, e.g. for CSV, where would be the place the set this encoder? custom ContentNegotiator with something like this?
...
protected $encoding = [
        'application/csv' =>  \App\Encoders\CsvEncoder::class
    ];
...

@fibis, you should try it.
By docs: i supposed to do it like this:
(custom ContentNegotiator with something like this)
`
...
protected function decodingsForResource(?Media $media): DecodingList
{
$decoder = new FileDecoder();
    return $this
        ->decodingMediaTypes()
        ->when(is_null($media), Decoding::create('multipart/form-data', $decoder))
        ->when(is_null($media), Decoding::create('multipart/form-data; boundary=*', $decoder));
}

...
`

@fibis yes the instructions in the Media Types (Content Negotiation) chapter are the ones to follow now if you want to do XML encoding. You'll need to look at the Custom Encoding section.

@Warchiefs @lindyhopchris Thank you both very much for the response, i will give it a try. I was only a little bit confused that the decoding part was more detailed than the encoding part.


Snake demo does not work with Mill 0.10.2

It looks like the fastOpt file has been renamed from out.js to main.js...?

Followed up on this: Mill 0.10.3 is also not working but it's now a problem with the way Mill handles module kinds (ESModule/CommonJSModule).
Indigo has a PR ready to go that includes a less brittle method of finding the right file to use from the right location as described in the original comment.
@lolgab has a fix ready to go, just needs to be released.
Everyone should use Mill 0.10.0 until then.

This will be fixed once https://github.com/com-lihaoyi/mill/pull/1851 is merged

Mill 0.10.3-36-e038bb contains the fix. Can you try to see if it solves your problem?

Will do - will give it a shot this evening. 👍

@lolgab Works perfectly 🎉
Sorry about the delay coming back to you.

No worries :)

Fixed on main branch by upgrading to Mill 0.10.4, will go out with the next release.


TF modules for Zone_base firewall,feature policy and IPS policy . modifiy main.tf

@rrahimm  please review and share the review comments

@rrahimm  review comments incorporate.

@rrahimm  added the dependency in device templates for security policy

Hello @danischm
Can you please review and approve

hello @tzarski0
Can you review and approve this.


Addition of FastAPI Project | Issue #336

Pull Request for PyVerse 💡
Requesting to submit a pull request to the PyVerse repository.

Issue Title
Please enter the title of the issue related to your pull request.
Addition of FastAPI Project | Issue #336

[X] I have provided the issue title.

Name
Please mention your name.
Enter your name here.
Shariq

[X] I have provided my name.

GitHub ID
Please mention your GitHub ID.
Enter your GitHub ID here.
Shariq2003

[X] I have provided my GitHub ID.

Email ID
Please mention your email ID for further communication.
Enter your email ID here.

[ ] I have provided my email ID.

Identify Yourself
Mention in which program you are contributing (e.g., WoB, GSSOC, SSOC, SWOC).
Enter your participant role here.
GSSOC Ext

[X] I have mentioned my participant role.

Closes
Enter the issue number that will be closed through this PR.
*Closes: #336

[X] I have provided the issue number.

Type of Change
Select the type of change:

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

Checklist
Please confirm the following:

[X] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[X] I have commented my code, particularly wherever it was hard to understand.
[X] ] My changes generate no new warnings.
[X] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

@UTSAVS26 please merge this.


The initialization of default value should work even if the value is initialized before fetching the list

The values in my selector are not getting initialized. The scenario is that I have a saved object in the database that I am fetching but my lists are getting populated after I fetch the item.
I think the watch on options should make share that the correct selected values are displayed. Below is the plunk for the behaviour.
http://plnkr.co/edit/D2wLxtgF2oOjTUVvdYrX?p=preview
In this my model browser1 is initalized prior to the list browsers1 is populated. As a result, the selected value does not get reflected in the list.
If I initialize the value after the population of the list, then it works fine. Below is the plunk depicting the correct behaviour.
http://plnkr.co/edit/UZIoYByVqSheTW2L8Eiq?p=preview
For now I have done a workaround in my code so that this works correctly. But is there any other way of doing this?

Hi @kanikagupta13, this should be handle using remote and remote-validation properties.
Give me 5 minutes to write a plunker, in the meantime you can read about these properties in the README file. :)

Check out this plunker and let me know if it fits your needs :)
http://plnkr.co/edit/PcbDYNZqdNxKoEnFWv63?p=preview

hi @indrimuska . Thank you for the solution. :)
I was thinking if it is possible to use the remote or some other way without promises as well, then it will be great as I will have to inject promises everywhere in my app even for lists that are being loading synchronously. But for now, I will use this. Please let me know if there is any future scope of handling this without the use of promises.
Thank you for the quick solution. :+1: :)

Well, I don't think that it makes sense to have a remote property without a Promise.
My advice for you is to understand the power of promises and the design pattern around them. You can also change your options without using a promise, but your code won't surely become so easy to handle and manage.
If you want to try, you can start using this plunker that doesn't make use of remote property:
http://plnkr.co/edit/uyNiH8ozCW20PQ9tYoZT?p=preview
Good luck! :)


docs: Add choices with html example

Hi @makseq!
This PR was created in response to a PRs in upstream repo:

https://github.com/heartexlabs/label-studio-frontend/pull/1208

It's docs only.


Add FTP Service

Is your feature request related to a problem? Please describe.
Uploading multiple files to a new plate via the File Editor can be tedious. We have already added the uncompressed zip option to facilitate this a little bit. The upload of zip file still requires unpacking of the archive using the unzip command.
I discovered a new repository that implements a SimpleFTPServer  on ESP32 and Arduino. This seems like a good fit to add to openHASP. It makes it a lot easier to upload and download files from the plate.
Describe the solution you'd like
Add SimpleFTPServer  as an openHASP service, like telnet or http.
Describe alternatives you've considered
unzip command is still not very user friendly. FTP is a well known file transfer protocol with multi-platform client support.
Additional context
Some issues still need to be resolved:

[x] Arduino v2 uses LittleFS instead of LITTLEFS
[ ] ⚠ Reading of /config.json poses a security issue
[x] The service can not be stopped (so service stop ftp won't work)
[x] Improve logging when transferring files

Is ftp really easier than curl with the http server that's already in there?
❯ curl http://roomPlateIpv4Here/edit --form name=data --form<EMAIL_ADDRESS>
This will upload the pages.jsonl file from $PWD to the device. You can wrap this in a simple for ... in ... loop:
❯ for i in *.jsonl; do
echo uploding $i
curl http://roomPlateIpv4Here/edit --form name=data --form data=@$i
done

or similar

It depends, if you're an admin or developer that's familiar with curl, by all means... I hadn't even thought of using that.😃
But for a regular user who's accustomed to using a GUI application, I think FTP will be easier.

Fair enough. Browsers don't support FTP anymore so I didn't see a huge difference between installing curl and installing another FTP program :/

I use SFTP fairly often, so already have a client for that. It seemed quite useful and suitable for the project.
SFTP/SSH is a bit much for the ESP32, but on a LAN it could be OK.

FTP only works if the client and the plate are on the same network. Impossible to connect through FTP from a different subnet.
Filezilla default settings (max connections 1):
Status:    Connection established, waiting for welcome message...
Status:    Plain FTP is insecure. Please switch to FTP over TLS.
Status:    Server does not support non-ASCII characters.
Status:    Logged in
Status:    Retrieving directory listing...
Command:    PWD
Response:    257 "/" is your current directory
Command:    TYPE I
Response:    200 TYPE is now 8-bit binary
Command:    PORT xxx,xxx,xxx,xxx,xxx,xxx
Response:    200 PORT command successful
Command:    MLSD
Response:    425 No data connection
Error:    Failed to retrieve directory listing

Passive mode:
Status:    Connection established, waiting for welcome message...
Status:    Plain FTP is insecure. Please switch to FTP over TLS.
Status:    Server does not support non-ASCII characters.
Status:    Logged in
Status:    Retrieving directory listing...
Command:    PWD
Response:    257 "/" is your current directory
Command:    TYPE I
Response:    200 TYPE is now 8-bit binary
Command:    PASV
Response:    227 Entering Passive Mode (0,0,0,0,195,89)
Command:    MLSD
Error:    The data connection could not be established: ECONNREFUSED - Connection refused by server
Response:    425 No data connection
Error:    Failed to retrieve directory listing

If this is implemented deliberately as a kind of security measure, it should rather be disregarded. In secure environments IoT devices should be placed in separate subnets to be able to isolate their traffic, and management has to be possible to be done from user networks through normal routing policies - and FTP by design supports that - I think there's some extra constraint here which is contraproductive imho.

Looking at the code of the FTP library, it uses a static port for PASV which is 50009. The error above complains that the client can't connect to this port, Connection refused by the plate.

https://github.com/xreef/SimpleFTPServer/issues/8

FTP now only works if the client and the plate are on the same network.

This this really the case? If yes, how can I disable/turn this off?  I deploy my openHasp devices to a subnet that is different from the one where my workstation resides.  I would prefer not writing a SNAT rule on my router to trick the openHasp devices into thinking that the FTP traffic from my workstation is coming from another device on the openHasp subnet...

Read the issue above. The SimpleFTPServer has a bug, it reports <IP_ADDRESS> as the PASV IP address...

Works great. It allows for a simple and effective way to do backups off all config. Excellent.

Thanks for this nice feature! With this was able to upload all my files at once.
Just one hint: There is a lack of documentation which FTP username and password needs to be used. Actually, on the openhasp homepage there is no documentation about FTP at all.
Anyway, I found the answer in the code here https://github.com/HASwitchPlate/openHASP/pull/284/files#diff-b07f16599331cf75146b1a06431eaf8a01488bb416807415c703958cf742782dR113. The HTTP username and password is used. So I needed to set is first. In Filezilla I activated the passiv mode and limit the numbers of connection to 1.
Maybe this information helps others to use the great FTP access.

@AlbrechtL Thanks!
The link with the HTTP credentials was just a test to see if the library was working.
Now that this feature is stable, I've updated the library and added specific FTP configuration options in the Web UI.
We are working on the 0.7.0-dev docs, and this service will be documented there eventually.


kap cluster config value should be "cluster" ?

URL for relevant page?
======
In section "influxDB Configuration" The configuration section for InfluxDB has a new option subscription-mode which should be set to server when running Enterprise Kapacitor as a cluster. This allows each server within the cluster to create its own subscriptions to InfluxDB so that each member receives all the data..
I think this value should say subscription-mode should be set as cluster which is set by default, correct?

Do you have the link to the specific page?

The comments in the code `services/influxdb/config.go:26-28 are as follows:
        // ClusterMode means that there will be one set of subscriptions per cluster.
        ClusterMode SubscriptionMode = iota
        // ServerMode means that there will be one set of subscriptions per server.
        ServerMode

So this implies that for each server within the cluster to create its own subscription subscription-mode should be set to server.  However, I don't currently have a clustered deployment on which to verify this.

The quoted docs are correct, while running an Enterprise Kapacitor cluster you want to set the subscription mode to server. Otherwise there will only be one subscription for the entire cluster and only one of the nodes will receive the data.

Closing based on @nathanielc comment.


chore(types): Drop password property from UserJSON

Description
There is no reference in our FAPI responses (/v1/me) about password field.
The password is only used as part of the UpdateUserParams type, so i moved it from UserJSON to that type.
Also there is no reference in our Frontend API docs: https://reference.clerk.dev/reference/frontend-api-reference/users/user
Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[x] (If applicable) JSDoc comments have been added or updated for any package exports
[x] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[x] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[x] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Mac OSX? cant install

There are several plugin folders in theOS  Kicad app. directory - this plugin does not seem to work from any of them.
How to fix?

I can confirm that it does not work in MAc OS Catalina while other plugins placed in the same folder do work.
The plugin is not even shown in the plugins menu.

I can't get it to work as a plugin but it does work. from the command line.
In pcbnew :
Save your pcb as dsn file
File->export ->Spectra DSN
from command line:
java -jar /jar/jar3/FreeRouting.jar -di
or
java -jar /jar/jar3/FreeRouting.jar -de filename.dsn
*note -di for a file dialog -de if you're providing the path and filename of the dsn file.

This is most likely because our freerouting macOS executable is not signed.
We have a discussion about it here, if you are interested.
@ScaredyCat's solution should work though, and you might also need to add the -do parameter for the output file.
java -jar freerouting-<IP_ADDRESS>.jar -de input_filename.dsn -do output_filename.ses
The current freerouting-<IP_ADDRESS>.jar can be downloaded from here: https://github.com/freerouting/freerouting/releases


Fixed some bugs, added Russian translation

Clear text passwords need to have no limit in the methods of authorization, but work with it was broken.
Two lines in project was'nt have translation.
Added Russian translation.
Fixed displaing national symbols. Default encoding in gettext is now UTF-8

As mentioned previously in #35, we do not want to reintroduce the clear field.
If you want to submit just the Russian localization, this is more than welcome. Bug fixes and code style fixes are more than welcome. But the clear field is definitely not.
Also, bug fixes that are isolated from each other have better chances to be accepted into the repository. For example, I would accept the missing quotes fix to adminuserblocksubmit.php without thinking much, but I'm not so sure about setting locale to *.UTF-8 explicitly.


Cloudstack 4.13v unable to 2nd Zone

@wido
ISSUE TYPE

Other

COMPONENT NAME

cloudstack-agent.service

CLOUDSTACK VERSION

Cloudstack 4.13

CONFIGURATION

OS / ENVIRONMENT

Centos7, KVM, NFS
SUMMARY

I've two cloudstack setups, when I try to add 2nd zone, going with Advance zone setup,
Everything goes in flow till adding host, when it reaches adding host, it starts failing, simultaneously cloudstack-agent.service fails and doesn't start or restarts again.
[root@acs4 ~]# service cloudstack-agent status -l
Redirecting to /bin/systemctl status  -l cloudstack-agent.service
● cloudstack-agent.service - CloudStack Agent
Loaded: loaded (/usr/lib/systemd/system/cloudstack-agent.service; enabled; vendor preset: disabled)
Active: activating (auto-restart) (Result: exit-code) since Wed 2020-01-29 15:43:22 IST; 8s ago
Docs: http://www.cloudstack.org/
Process: 17938 ExecStart=/usr/bin/java $JAVA_OPTS $JAVA_DEBUG -cp $CLASSPATH $JAVA_CLASS (code=exited, status=66)
Main PID: 17938 (code=exited, status=66)
Jan 29 15:43:22 acs4.omnion.in systemd[1]: Unit cloudstack-agent.service entered failed state.
Jan 29 15:43:22 acs4.omnion.in systemd[1]: cloudstack-agent.service failed.
[root@acs4 ~]#
STEPS TO REPRODUCE

Goto->Infrastructure -> Zones -> Add New Zone
Give the IP address and all other required info.

EXPECTED RESULTS

Successfully add zone and create VM's

ACTUAL RESULTS

fails while adding host, cloudstack-agent.service also enter fail state

@prashant2594 we have the same issue - did you find a fix?

Please help me same error form my. this is my kvm centos 7.7. cloudstack agent not start.

Thanks.

@alamintech try to remove and add the host agains, check in the agent.properties file is there any public nic defined. For example, in my local setup I see:
# cat /etc/cloudstack/agent/agent.properties | grep network
private.network.device=cloudbr0
domr.scripts.dir=scripts/network/domr/kvm
guest.network.device=cloudbr0
public.network.device=cloudbr0

@rhtyd  Thanks for your reply!
Now fix, I am using MS version is 4.14 and kvm agent is 4.13. so kvm agent and libvirt is ok, now.
If I am use this any problem ?
when I am create instance so I am arise new problem Error "unable to start a vm due to insufficient capacity cloudstack"
Please help me..
Thanks again.

You're using 4.13 agent with 4.14 mgmt server. Your setup is incorrect, you must use the same software version for all components.

ok I am trying again.

@rhtyd Many many thanks.
version 4.14 all is ok using ceph storage primary and secondary is nfs.
Now two question for you.

How can I use open vm console using LAN ip without WAN interface?
How can I use secondary storage as ceph pool?

Thanks.

@alamintech Ceph can not be used as Secondary Storage.

ok, you know Q. 1 ans. Please help me.
Thanks.

all working is fine.
I see primary storage is two think one is my ceph storage and one is my host kvm location is
/var/lib/libvirt/images

And another question.. I see all time local storage is double. It is my problem or not. I am not understand.
Please help me.
Thanks.


[draft] first attempt to fully-qualify object names for purposes of rolling updates

add modified helper-template with full-name function that composites the following fields:
release-name + release-version + service-name + service-version + kind
applied this strategy to every name attribute, in all likelihood too much so.
additionally, we should verify reference-names are not broken by this change.
apiVersion: apps/v1
kind: Deployment
metadata:
  name: release-name-0.0.1-mds-policy-author-latest-deployment

note: tests will need to be updated prior to commit
reference: https://www.youtube.com/watch?v=9cwjtN3gkD4&feature=youtu.be&t=470
PR Checklist

[ ] simple searchable title - [mds-db] Add PG env var, [config] Fix eslint config
[ ] briefly describe the changes in this PR
[ ] mark as draft if should not be merged
[ ] write tests for all new functionality

Impacts

[x] Provider
[x] Agency
[x] Audit
[x] Policy
[x] Compliance
[x] Daily
[x] Native
[x] Policy Author

Please retarget this PR at @lacuna-tech/mds-core


bug(*): Update documentation woth infromation from Angular 10

I am not sure how out of date it is, but I found it is true for entryComponents.
From https://update.angular.io/#9.0:10.0l3:

If you have specified any entryComponents in your NgModules or had any uses of ANALYZE_FOR_ENTRY_COMPONENTS, you can remove them. They are no longer required with the Ivy compiler and runtime.

Expected Behavior
https://material.angular.io/cdk/portal/overview (and dialog and possibly others)
Used to create a portal from a component type. When a component is dynamically created using portals, it must be included in the entryComponents of its NgModule.
Actual Behavior
I would expect it not to be there as it clearly is not true anymore. (just tested for dialog)
Environment

Angular: 10
CDK/Material: 10
Browser(s): *
Operating System (e.g. Windows, macOS, Ubuntu): Windows

entryComponents aren't required if you're running Ivy which is the default now, but they're still necessary for ViewEngine. Since we support switching between the two, we can't remove the entry from the docs yet. Once core removes ViewEngine completely, we'll remove the mentions of entryComponents from our docs as well.


Request fix optimizeImport module support carbon-icon-component v12

Since Svelte 4 was released, carbon-component-svelte was also upgraded a few days ago, I would like to request we upgrade the component to support icon v12.
For now in code only validate package icon prefix v11. When meet v12 it fell to an old icon component link and broken site.

Fixed in v0.10.0.


Error in v1.6 dataclasses breaks Authorize messaging

When making an Authorize request from the Charge Point to the Central System, the request is an instance of call.AuthorizePayload:
@dataclass
class AuthorizePayload:
    id_tag: str

Note that the id_tag is of type str, and the OCPP-J v1.6 spec says it should be an IdToken type. Since this is basically the same as a string but constrained to a 20 character case-insensitive string, this is fine.
The response must be an instance of call_result.AuthorizePayload:
@dataclass
class AuthorizePayload:
    id_tag_info: Dict

Where there is a dataclass for id_tag_info (though not yet added to the above dataclass) defined in datatypes.py:
@dataclass
class IdToken:
    """
    Contains the identifier to use for authorization. It is a case
    insensitive string. In future releases this may become a complex type to
    support multiple forms of identifiers.
    """

    id_token: str

@dataclass
class IdTagInfo:
    """
    Contains status information about an identifier. It is returned in
    Authorize, Start Transaction and Stop Transaction responses.

    If expiryDate is not given, the status has no end date.
    """

    status: AuthorizationStatus
    parent_id_tag: Optional[IdToken] = None
    expiry_date: Optional[str] = None

The spec says that parent_id_tag is also an IdToken. However in this IdTagInfo dataclass it's an IdToken dataclass, with an id_token str attribute inside. Therefore, in a call_result.AuthorizePayload it gets serialised to json like this:
{
  "idTagInfo": {
    "status": "Accepted",
    "parentIdTag": {
      "idToken": "123"
    },
    "expiryDate": "2022-01-07T20:22:02.735181"
}

When this is received back at the Charge Point, it then fails the json schema validation because it's expecting the idToken to be a str as defined in the json schema. If schema validation is skpped, only then do you get a helpful exception:
Traceback (most recent call last):
  File "/Users/vince/projects/cloud-services/services/ocpp/ocpp-beam/tests/chargepoint/./charge_point.py", line 109, in <module>
    asyncio.run(main())
  File<EMAIL_ADDRESS>line 44, in run
    return loop.run_until_complete(main)
  File<EMAIL_ADDRESS>line 642, in run_until_complete
    return future.result()
  File "/Users/vince/projects/cloud-services/services/ocpp/ocpp-beam/tests/chargepoint/./charge_point.py", line 102, in main
    await asyncio.gather(cp.start(), cp.send_authorize())
  File "/Users/vince/projects/cloud-services/services/ocpp/ocpp-beam/tests/chargepoint/./charge_point.py", line 45, in send_authorize
    response2 = await self.call(request2)
  File "/Users/vince/.cache/pypoetry/virtualenvs/ocpp-beam-ChE6HI8P-py3.9/lib/python3.9/site-packages/ocpp/charge_point.py", line 285, in call
    validate_payload(response, self._ocpp_version)
  File "/Users/vince/.cache/pypoetry/virtualenvs/ocpp-beam-ChE6HI8P-py3.9/lib/python3.9/site-packages/ocpp/messages.py", line 194, in validate_payload
    raise ValidationError(f"Payload '{message.payload} for action "
ocpp.exceptions.ValidationError: Payload '{'idTagInfo': {'status': 'Accepted', 'parentIdTag': {'idToken': '123'}, 'expiryDate': '2022-01-07T21:19:11.794097'}} for action 'Authorize' is not valid: {'idToken': '123'} is not of type 'string'

Failed validating 'type' in schema['properties']['idTagInfo']['properties']['parentIdTag']:
    {'maxLength': 20, 'type': 'string'}

On instance['idTagInfo']['parentIdTag']:
    {'idToken': '123'}

I'm not sure why this only gets raised when skip_schema_validation is enabled on the on decorator - unfotunately it seems to not be surfaced as otherwise the websocket disconnects instantly:
Traceback (most recent call last):
  File "/Users/vince/projects/cloud-services/services/ocpp/ocpp-beam/tests/chargepoint/./charge_point.py", line 109, in <module>
    asyncio.run(main())
  File<EMAIL_ADDRESS>line 44, in run
    return loop.run_until_complete(main)
  File<EMAIL_ADDRESS>line 642, in run_until_complete
    return future.result()
  File "/Users/vince/projects/cloud-services/services/ocpp/ocpp-beam/tests/chargepoint/./charge_point.py", line 102, in main
    await asyncio.gather(cp.start(), cp.send_authorize())
  File "/Users/vince/.cache/pypoetry/virtualenvs/ocpp-beam-ChE6HI8P-py3.9/lib/python3.9/site-packages/ocpp/charge_point.py", line 125, in start
    message = await self._connection.recv()
  File "/Users/vince/.cache/pypoetry/virtualenvs/ocpp-beam-ChE6HI8P-py3.9/lib/python3.9/site-packages/websockets/legacy/protocol.py", line 552, in recv
    await self.ensure_open()
  File "/Users/vince/.cache/pypoetry/virtualenvs/ocpp-beam-ChE6HI8P-py3.9/lib/python3.9/site-packages/websockets/legacy/protocol.py", line 929, in ensure_open
    raise self.connection_closed_exc()
websockets.exceptions.ConnectionClosedOK: received 1000 (OK); then sent 1000 (OK)

I think the fix here is to remove the dataclass for IdToken and make the IdTagInfo be type annotated like so:
  parent_id_tag: Optional[str] = None

I tested this by adding this id_tag_info to a call_result.AuthorizePayload which works, despite it obviously going against the dataclass definition/type hinting.
@on(Action.Authorize)
def on_authorize(self, id_tag: str, **kwargs) -> call_result.AuthorizePayload:
    # Do some checking on the requested id
    id_tag_info = IdTagInfo(
        status=AuthorizationStatus.accepted,
        expiry_date=(datetime.now() + timedelta(days=30)).isoformat(),
        parent_id_tag=IdToken("123")
    )
    return call_result.AuthorizePayload(id_tag_info.__dict__)

Hi,
What this seems to request is a way to constrain the str data type, something that pydantic does: https://pydantic-docs.helpmanual.io/usage/types/#constrained-types
But, we have the schema that does the validation later on, so pydantic can be skipped.
Anyway, according to the spec and the schema IdToken is a str Type, so that is what we should use. Thanks for your analysis  ;)

Fi


Add node info

add more information to node

Since we already have devnet and betanet, we should finish this PR (just make sure it still shows something useful for the testnet)
There is still an issue of misreporting (https://github.com/nearprotocol/nearcore/issues/2306) the telemetry data to a wrong network, so we should also add a condition checking the chain ID to match the one that is relevant for the alias (domain name).

I see the link you put, but still not know what to do

I thought this PR was adding more info to the Nodes List page, that is why I thought it is a draft. We should just land it then.


How to get the the file on other datasets just like mini_imagenet_part_prior.pickle

Thanks for your interesting work.
We reproduce the results on the MiniImagenet , and then we want to reproduce the experiments on other datasets just like tieredImagenet in the paper. Hovewer ,we meet a problem how to get the file(part_prior.pickle) of other datasets.
Looking forward to your reply.

TieredImageNet is also constructed from ImageNet. The construction process of the file (part_prior.pickle) is similar to miniImagenet. The code can be directly applied to TieredImageNet.