UI works but CLI doesn't with NEJE DK-8-KZ

Hello,
EzGraver 4.0.0-beta works nicely with my DK-8-KZ on Linux Mint 19 Cinnamon with the UI version; however, when I try to run the CLI version, it connects to the DK-8-KZ via ttyUSB0 but is unable to command it via the reset, home, or start commands. I notice that the terminal window feedback differs between the UI and CLI versions - e.g., when I run the reset command in the CLI version, I see the message "transmitting 1 bytes "f9"", whereas when I run the same command in the UI version I see the message "transmitting 4 bytes "ff040100""; this seems to correspond somehow to the corresponding functions in ezgraver.cpp versus ezgraver_v3.cpp. What should I do to get the CLI version working?
Thank you!

I've tried re-compiling the CLI main.cpp file with an updated reference to ezgraver_v3.h instead of exgraver.h, with the hope that this would cause the CLI version to invoke the v3 protocol functions - which, as I mentioned before, seem to work - but the CLI version still invokved the v1 protocol functions, which didn't do anything with the DK-8-KZ. Any ideas as to what I should do to get the CLI version of EzGraver working with the DK-8-KZ?

sleptew, I had (other) issues with the command line as well.
camrein directed me to the unix_cli_fixes branch in this issue:
https://github.com/camrein/EzGraver/issues/45
See his reponse from Apr 10th.
That version of CLI app works with an Ubuntu 16.04 Laptop setup and a Raspbian Jessie setup I have at my pad.
Good luck man.

onebeartoe, thank you for the reference. I've downloaded, compiled, installed, and run the command-line version of the unix_cli_fixes EzGraver branch. The only difference in behavior when I try resetting or homing the DK is that the commands run to completion; the same v1 protocol information is printed in the terminal window, indicating that this branch still uses the v1 protocol as opposed to the v3 protocol which the UI version uses - which DOES work on this machine. I'd like to get the CLI version working for a project, though. :-/ Any other ideas?

Ah okay.  I missed that you are trying to configure a different protocol version.
Maybe try that same code change you mentioned you did in the other branch, if you haven't tried already.
But now I am more interested in what you are doing.  I want to get/poll the status of the printer for this Web app I am writting that uses the CLI as a system command to control the engraver.
https://github.com/onebeartoe/electronics/tree/master/neje-engrave
Does the V3 protocol provide engraver status, like percentage complete of a job?

I've tried the code change mentioned (referencing ezgraver_v3.h instead of ezgraver.h). Didn't make a difference. As I've not been able to get the v3 protocol to work yet in command-line mode, then I have no idea if the engraver provides an ongoing status while it is exercising a job....if I do get it to work, I'll try to update this thread to let you know.

Sounds good, thanks.

any news on this? - i also have this problem.  I also tried a new driver from here for my Mac Pro with Mojave - https://www.mac-usb-serial.com/dashboard/
Is there any logs where I can see where this is getting stuck?


Move reimbursement of medication info from ERD to Gendalf

Medication parameters about reimbursement:

medication_id
wholesale_price_package | number| not null
retail_price_package | number| not null
day_recommended_dose | number| not null
reimbursement_day_dose | number| not null
reimbursement_amount_pack | number | not null
surcharge_pack | number| not null

Also use entity Medication_program_info!

Obsolete requirements.


Fix the issue Wildcard matching should be used in help file search

Fix issue #3967
Summary of the issue:
According to Jason's suggestion, replace the regex expression with wildcard.
Root cause of the issue:
This is an code improvement. The existed test cases are covering the scenario

@lzybkr Thanks Jason, your comment has been resolved


[Bug][Master] Fix when the master or worker down, the alert cann't insert to Db, due to the datasource close

close #5491, close #5578
When an instance receives a Node_REMOVE event from zookeeper,  it will trigger the listener to response for the change.
One listener will remove the node to dead path. Another will insert an alert data to database.
But the listener trigger has no order, because TreeCache use a map to store the listener.
This change save the listener in an ordered list.

Codecov Report

Merging #5497 (37f65d4) into dev (2d71930) will decrease coverage by 0.06%.
The diff coverage is 9.25%.

@@             Coverage Diff              @@
##                dev    #5497      +/-   ##
============================================
- Coverage     45.38%   45.32%   -0.07%
+ Complexity     3690     3687       -3
============================================
  Files           607      608       +1
  Lines         24855    24872      +17
  Branches       2825     2826       +1
============================================
- Hits          11280    11272       -8
- Misses        12489    12514      +25
  Partials       1086     1086

Impacted Files
Coverage Δ

...r/server/master/registry/MasterRegistryClient.java
49.25% <0.00%> (ø)

...er/master/registry/MasterRegistryDataListener.java
0.00% <0.00%> (ø)

...uler/server/master/registry/ServerNodeManager.java
6.66% <0.00%> (-0.36%)
:arrow_down:

...dolphinscheduler/spi/register/ListenerManager.java
0.00% <0.00%> (ø)

...lphinscheduler/spi/register/SubscribeListener.java
0.00% <0.00%> (ø)

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
38.73% <20.83%> (+0.27%)
:arrow_up:

...he/dolphinscheduler/common/enums/SqoopJobType.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (ø)

...eduler/server/worker/runner/TaskExecuteThread.java
59.50% <0.00%> (ø)

Continue to review full report at Codecov.

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

Closed by too many conflicts.


[stable/datadog] Allow providing volumes and mounts in the cluster agent deployment

Signed-off-by: Ivan Ilichev<EMAIL_ADDRESS>What this PR does / why we need it:
Allow providing volumes and mounts in the cluster agent deployment.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

/lgtm


Update sbt-scoverage to 1.6.0

Updates org.scoverage:sbt-scoverage from 1.5.1 to 1.6.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "org.scoverage", artifactId = "sbt-scoverage" }]

labels: semver-minor

Codecov Report

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

@@             Coverage Diff             @@
##           master       #5       +/-   ##
===========================================
- Coverage    82.6%   70.37%   -12.24%
===========================================
  Files           3        4        +1
  Lines          69       81       +12
  Branches        3        3
===========================================
  Hits           57       57
- Misses         12       24       +12

Impacted Files
Coverage Δ

...cala/me/lyh/parquet/avro/examples/ExampleApp.scala
0% <0%> (ø)

Continue to review full report at Codecov.

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


M3-5940: New Linode Logo

Description 📝

Replaces our old logo with our new logo

Preview 📷
Before

After

How to test 🧪

Test the Primary Nav and the logo's behavior

test all collapsed and uncollapsed states

Test the Logo in Storybook

Totally unrelated to this PR, I just happened to notice it now. But I wonder if we can apply a max-width or max-height rule or similar to the logo in Storybook to prevent this from happening when resizing the sidebar:

The way the "Skip to Canvas" link gradually shifts into view is especially strange!


Member Browser is missing

The Member Browser is missing.  The IFS and Object Browsers are present though.

QCCSID: 65535
Features:

git: true
grep: true
tn5250: false
setccsid: true
md5sum: true
GENCMDXML.PGM: true
QZDFMDB2.PGM: true
startDebugService.sh: true
iconv: true
attr: true

SQL enabled: false
Source dates enabled: false

Variants
{
  "american": "%23@$",
  "local": "%23@$"
}

Errors:
[
  {
    "command": "system \"CRTLIB LIB(ILEDITOR) TEXT('Code for i temporary objects. May be cleared.')\"",
    "code": 255,
    "stderr": "CPF2111: Library ILEDITOR already exists.",
    "cwd": "/home/JMORGAN"
  },
  {
    "command": "system \"CHKOBJ OBJ(QSYS/QCPTOIMPF) OBJTYPE(*DTAARA)\"",
    "code": 255,
    "stderr": "CPF9801: Object QCPTOIMPF in library QSYS not found.",
    "cwd": "/home/JMORGAN"
  },
  {
    "command": "system \"CHKOBJ OBJ(QSYS/QCPFRMIMPF) OBJTYPE(*DTAARA)\"",
    "code": 255,
    "stderr": "CPF9801: Object QCPFRMIMPF in library QSYS not found.",
    "cwd": "/home/JMORGAN"
  }
]

The member browser was replaced with the object browser.


[BUG] Install failed on GCP when following instructions from docs

The documentation here provides installation instructions for GCP.
These lines result in a 404
wget https://github.com/dessa-oss/atlas/releases/latest/download/atlas_installer.py
wget https://github.com/dessa-oss/atlas/releases/latest/download/atlas.tgz

because the release is not tagged latest. The workaround to fix this right now is to replace latest with the current release version from here.

Retagged 0.1.1 as latest on Github


doc(README): remove typo

Corrected "demostration" to "demonstration" in [README.md].
This pull request addresses a minor typo found in repository. The typo has been corrected to improve clarity and maintain the quality of the documentation.
This change is purely cosmetic and does not affect functionality.
What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

good job，can you fix the  DCO issues ?


cleanup

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Dear solana-labs team!
Please consider merging this pull request. My client has made some mistakes in publishing their token and the cleanup contains deletions.
Thank you!


Add Angelo and Sai to CODEOWNERS

Update CODEOWNERS file in go-training.

In progress: https://github.com/anz-bank/go-training/pull/124

Merged.


Updated examples

I fixed the tutorial notebook and added a newick file.

Thanks!
Could you also add a read_newick example in the tutorial notebook. I'm imagining another cell that reads the example newick file and displays that dataframe as well.


fireeyeax - Add Python3 support

Run 2to3 on code to convert to Python3
Update requirments
Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[x] Testing of all the changes has been performed (for bug fixes / features)
[x] The manual_readme_content.md has been reviewed and added / updated if needed (for bug fixes / features)
[x] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[x] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[x] Verify all checks are passing.
[x] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[x] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[ ] New App
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[x] Other (please describe): Add support for Python 3 so the app can be used in newer versions of Splunk SOAR

Security Considerations (REQUIRED)

[ ] If you are exposing any endpoints using a REST handler,
please document them in the manual_readme_content.md.
[ ] If this is a new connector or you are adding new actions

[ ] Please document in the manual_readme_content.md all methods (eg, OAuth) used to authenticate
with the service that the connector is integrating with.
[ ] If any actions are unable to run on SOAR Cloud, please document this in the manual_readme_content.md.

[ ] Are you introducing any new cryptography modules? If yes, please elaborate their purpose:
[ ] Are you are accessing the file system? If yes, please verify that you are only accessing paths returned through
the Vault API.
[ ] Are you are marking code to be ignored by Semgrep with nosemgrep?
If yes, please provide justification in an additional comment next to the ignored code.

Release Notes (REQUIRED)

Add support for Python 3
Update dependencies

What is the current behavior? (OPTIONAL)

Application does not support Python 3

What is the new behavior? (OPTIONAL)

Application supports Python 3

Other information (OPTIONAL)

N/A

Pay close attention to (OPTIONAL)

N/A

Screenshots (if relevant)

N/A

Thanks for contributing!

Hi @moshekaplan,
We have made the required changes in the code based on the best practice standards, including in the connector file. Can you please verify the changes from your end?
We have also bumped the major application version.

LGTM, it passed my basic testing of detonating a URL


Support of gRPC integration on baremetal

We can enable the gRPC feature to test Quarkus application using the @QuarkusApplication(grpc = true) annotation. This way we can verify purely gRPC applications using the GrpcService service wrapper:
@QuarkusScenario
public class GrpcServiceIT {

    static final String NAME = "Victor";

    @QuarkusApplication(grpc = true) // enable gRPC support
    static final GrpcService app = new GrpcService();

    @Test
    public void shouldHelloWorldServiceWork() {
        HelloRequest request = HelloRequest.newBuilder().setName(NAME).build();
        HelloReply response = GreeterGrpc.newBlockingStub(app.grpcChannel()).sayHello(request);

        assertEquals("Hello " + NAME, response.getMessage());
    }
}

Internally, the test framework will map the gRPC service of our Quarkus application using a random port.
This does not work for OpenShift/Kubernetes deployments as it requires to enable HTTP/2 protocol (more information in here).

This needs a rebase though!


EPEL7 python-pip package is called python2-pip; puppet-python won't recognize that it is installed

In EPEL7, the python-pip package is called python2-pip.  It is aliased to python-pip, so it can be installed with yum install python-pip, but then running yum list python-pip returns No matching Packages.
This causes some problems when puppet-python tries to install the pip package:
Error: Execution of '/bin/yum -d 0 -e 0 -y list python-pip' returned 1: Error: No matching Packages to list
Error: /Stage[main]/Python::Install/Package[pip]/ensure: change from absent to present failed: Execution of '/bin/yum -d 0 -e 0 -y list python-pip' returned 1: Error: No matching Packages to list
Notice: /Stage[main]/Python/Anchor[python::end]: Dependency Package[pip] has failures: true
Warning: /Stage[main]/Python/Anchor[python::end]: Skipping because of failed dependencies

Is there a workaround for this?  I tried to figure out if there was a way that I could force puppet-python to look for python2-pip instead of python-pip but I couldn't see that as a configurable option.
I'm running CentOS 7.2.

I fixed this by setting python::version to python27 and python::provider to scl.

Sounds like we need to change the name of the package.


Some clippy lints

bors r+

bors retry

@kjeremy I'll disable windows gating. Would you be interested in looking into why this test fails on windows? It should pass I think, so there might be a real issue hidden there!

@matklad annoyingly it passes on my machine.

@kjeremy just to sanity check, you run with RUN_SLOW_TESTS=1?

@matklad I had a hypothesis that the windows workers, probably being multi-tenant, are a bit overworked and thus not the best place for timing sensitive tests. Why the situtation on linux, or especially macOS, is better, is beyond me. Maybe the windows scheduler is significantly less fair?

The tests test that we do O(1) of work, so it is not time-senstive in the success path

@matklad It's diagnostics_dont_block_typing that's failing right? That's timing sensitive in my mind 🤷♂. But I don't see why it should be failing unless the workers are incredibly congested.

Right. The server.request::<OnEnter> call should complete "immediately", I'd expect elapsed to be bellow 16ms. Ie, we should not wait at all, and should do a relatively small amount of CPU work. 2000ms is well over the budget we have here.
OTOH, the background work we need to do, without blocking OnEnter, is well over two seconds, so 2 seconds seems like a good arbitrary cutoff point which should not give any false positives or false negatives.

In other words, it is time-senstive in an absolute sense, but we have like 3 orders of magnitude error margin here in both directions

Here is the output of in my machine:  (I dbg the elapsed time
running 1 test
[crates\ra_lsp_server\tests\heavy_tests\main.rs:532] elapsed = 13.8147ms
test diagnostics_dont_block_typing ... test diagnostics_dont_block_typing has been running for over 60 seconds
test diagnostics_dont_block_typing ... ok

It used 13.8147ms, so it is quite fast.
However, it used over 60 seconds to shutdown the test.

@matklad it still passes locally with RUN_SLOW_TESTS=1

@edwin0cheng yeah, this is exactly the expected behavior: we test that we quickly complete "foreground" job, while background tasks slowly completes in background.

Okay, I tried kind of reproduce this bug by setting the CPU Affinity to 1 in Task Manager:
[crates\ra_lsp_server\tests\heavy_tests\main.rs:535] elapsed = 929.6164ms
test diagnostics_dont_block_typing ... test diagnostics_dont_block_typing has been running for over 60 seconds

Maybe github windows VM is single core ?

Interesting! So that means we need to set thread priorities on windows
somehow?
On Mon, 13 Jan 2020 at 19:31, Edwin Cheng<EMAIL_ADDRESS>wrote:

Okay, I tried kind of reproduce this bug by setting the CPU Affinity to 1
in Task Manager:
[crates\ra_lsp_server\tests\heavy_tests\main.rs:535] elapsed = 929.6164ms
test diagnostics_dont_block_typing ... test diagnostics_dont_block_typing has been running for over 60 seconds
Maybe github windows VM is single core ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/pull/2825?email_source=notifications&email_token=AANB3MY5RKXKYK6UJVYIGF3Q5SXQFA5CNFSM4KGFA76KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIZZD4A#issuecomment-573805040,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANB3M4IX32WALI36WZ2EQTQ5SXQFANCNFSM4KGFA76A
.

Seems related: https://www.reddit.com/r/rust/comments/eo5bs2/comment/fea2dlf
On Monday, 13 January 2020, Aleksey Kladov<EMAIL_ADDRESS>wrote:

Interesting! So that means we need to set thread priorities on windows
somehow?
On Mon, 13 Jan 2020 at 19:31, Edwin Cheng<EMAIL_ADDRESS>wrote:

Okay, I tried kind of reproduce this bug by setting the CPU Affinity to 1
in Task Manager:
[crates\ra_lsp_server\tests\heavy_tests\main.rs:535] elapsed = 929.6164ms
test diagnostics_dont_block_typing ... test diagnostics_dont_block_typing has been running for over 60 seconds
Maybe github windows VM is single core ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/pull/2825?email_source=notifications&email_token=AANB3MY5RKXKYK6UJVYIGF3Q5SXQFA5CNFSM4KGFA76KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIZZD4A#issuecomment-573805040,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANB3M4IX32WALI36WZ2EQTQ5SXQFANCNFSM4KGFA76A
.

Interesting! So that means we need to set thread priorities on windows
somehow?

I changed the code and did some tests on my Windows machine:

Force it to run in cpu Affinity = 1

#[test]
fn diagnostics_dont_block_typing() {
    if skip_slow_tests() {
        return;
    }
    unsafe {
        let process = winapi::um::processthreadsapi::GetCurrentProcess();
        let mask = 1;
        winapi::um::winbase::SetProcessAffinityMask(process, mask);
    }

Print out the total elapsed time:

#[test]
fn diagnostics_dont_block_typing() {
    if skip_slow_tests() {
        return;
    }

   .....

    let elapsed = start.elapsed();
    assert!(elapsed.as_millis() < 2000, "typing enter took {:?}", elapsed);
    dbg!(elapsed);
}

Print out the actual time used in that processing thread:

    pool_dispatcher
        .on_sync::<req::CollectGarbage>(|s, ()| Ok(s.collect_garbage()))?
        .on_sync::<req::JoinLines>(|s, p| handlers::handle_join_lines(s.snapshot(), p))?
        .on_sync::<req::OnEnter>(|s, p| {
            use std::io::Write;
            let _ = writeln!(&mut std::io::stderr(), "start!");
            let start = std::time::Instant::now();
            let r = handlers::handle_on_enter(s.snapshot(), p);
            dbg!(start.elapsed());
            r
        })?

The result is quite interesting :
start!
[crates\ra_lsp_server\src\main_loop.rs:485] start.elapsed() = 1.0432ms
[crates\ra_lsp_server\tests\heavy_tests\main.rs:537] elapsed = 1.1332555s

You can see the actual processing time is quite fast (1.0432ms), but the total elapsed time is around 1 second, which means we are blocking on the main loop !!
So I added a SetThreadPriority call in our main_loop :
pub fn main_loop(
    ws_roots: Vec<PathBuf>,
    client_caps: ClientCapabilities,
    config: ServerConfig,
    connection: Connection,
) -> Result<()> {
    ....
    unsafe {
        let tid = winapi::um::processthreadsapi::GetCurrentThread();
        winapi::um::processthreadsapi::SetThreadPriority(tid, 1);
    }

    log::info!("server initialized, serving requests");
    {
        let task_sender = task_sender;
        let libdata_sender = libdata_sender;
        loop {
            log::trace!("selecting");
            let event = select! {
   ...

And it is the result :
start!
[crates\ra_lsp_server\src\main_loop.rs:485] start.elapsed() = 1.3183ms
[crates\ra_lsp_server\tests\heavy_tests\main.rs:537] elapsed = 98.3874ms

Fascinating! Opened https://github.com/rust-analyzer/rust-analyzer/issues/2835


[WIP] Fix Index.to_series to work properly

Resolves #1305

before fixing

>>> pidx = pd.Index([1, 2, 3, 4, 5])
>>> kidx1 = ks.Index([1, 2, 3, 4, 5])
>>> kidx2 = ks.Index(pidx)
>>> kidx3 = ks.from_pandas(pidx)
>>> kidx1.to_series() == kidx2.to_series() == kidx3.to_series()
Traceback (most recent call last):
...
AssertionError: (1, 0)

after fixing

>>> pidx = pd.Index([1, 2, 3, 4, 5])
>>> kidx1 = ks.Index([1, 2, 3, 4, 5])
>>> kidx2 = ks.Index(pidx)
>>> kidx3 = ks.from_pandas(pidx)
>>> kidx1.to_series() == kidx2.to_series() == kidx3.to_series()
5    True
1    True
3    True
2    True
4    True

ah... more complicated than i thought.
i'll take a look at this in next week since really crazily busy this week 😭

Yeah, I roughly looked into this, and found it complicated.
Actually I don't think this PR fix the root cause. Let me take a look at the issue.

@ueshin yeah i think so, too. then let me close this for now. Thanks! :)


Allow users to redefine "input parameters" for "Simple Run"

This is important to be able to migrate some applications who require a deployment only to be able to redefine input parameters (like the Wordpress example).


fix(exr): suppress exr attributes that interfere with our hints

Our exr writer uses some special hints starting with "openexr:" to control aspects of the writing, and the reader sets these to communicate some things about the file it's reading.
But what happens if... the actual exr file has metadata of tha same name. Oops, those get carried along and mess with things. We had previously accounted for this by suppressing arbitrary metaname named "openexr:lineOrder" if we find it in the input (the actual lineorder comes from a header field, not named metadata, but OIIO uses the name "openexr:lineOrder" to communicate it back as metadata.
Turns out that there are two more that we need to proactively suppress: "openexr:levelmode" and "openexr:roundingmode". There's no good reason for those to ever be in an exr file (like I said, those names are NOT how openexr itself sets those in a file). And yet, we found at least one example in the wile where it was.

I just replaced this with an entirely different solution that is simpler and that I like better, the fix is now specific to make_texture.

You pointed out a good issue, which is that currently there is a corner case where the user wanted something called "openexr:levelmode" as metadata but not for it to trigger any special behavior on our end. I just want to acknowledge that possibility and kick the can down the road, as this ticket is the wrong place to tackle such a thorny (but also obscure?) problem.
But I'll review our presumptions here:

Metadata that starts with the prefix "oiio:" is only intended for OIIO signaling -- information that OIIO sets to be informative to readers, or hints to writers, or other internally carried state.
Metadata starts with the name of a file format itself, like "openexr:" is meant to give hints from or to that format reader/writer, and should definitely be ignored if encountered by any other format writers.
Some other prefixes are semi-special, for example ones starting with "Exif:" are exif fields that will be honored by any file format that supports Exif in some fashion.

It is also worth noting that I think OpenEXR is the only format we regularly deal with that truly allows "arbitrarily named" metadata. Most formats have a fixed list by their specification and/or the structure fields of their headers. So really, any conflict between "openexr:foo" and intentional user data is an odd case for sure. It still gnaws at me a little, though, so maybe there is a possible future revision about the rules for how we handle hints vs oiio-specific metadata vs "user" metadata to eliminate any rare edge cases.


Prevent copying of BsModule by default

It doesn't make sense to allow for Module to be copied, or moved, really, as it represents a singleton instance. Copying or moving such an object would ultimately end up pointing at the same instance.

Makes sense, thanks. Merged.


cli should show count of unsafe operators

Technically any operation from an IO point of view is unsafe. Just counting the total number of JS operations doesn't make sense. We can provide a summary of all the type of resolvers via check command ie — const, http, grpc, garphql and js, but that should be a separate issue.


(node:32072) UnhandledPromiseRejectionWarning: Error: Cannot autolaunch D-Bus without X11 $DISPLAY

soulseek login
? Login username1
? Password [hidden]
(node:32072) UnhandledPromiseRejectionWarning: Error: Cannot autolaunch D-Bus without X11 $DISPLAY
(Use node --trace-warnings ... to show where the warning was created)
(node:32072) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:32072) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hey @aloysiusantioch,
Thanks for reporting this bug. From what I have found, it's related to the underlaying lib to save the credential to the OS keystore (https://github.com/atom/node-keytar). This lib seems to need X11.
I found this related issue: https://github.com/atom/node-keytar/issues/224#issuecomment-540459884. They recommend to install xvfb. Can you give it a try please? Thanks a lot

I'm running into the same error.  I'm running in WSL Ubuntu 18.04.4 LTS.  Installed xvfb but still getting the same error:
soulseek login
? Login myusername
? Password [hidden]
node:internal/process/promises:225
          triggerUncaughtException(err, true /* fromPromise */);
          ^

[Error: Cannot autolaunch D-Bus without X11 $DISPLAY]

Hey @aloysiusantioch and @geoff-maddock,
I added some instructions in the README to run soulseek-cli in a headless environment: https://github.com/aeyoll/soulseek-cli#on-headless-linux
Can you give it a try? Thanks!


Group prefix not applied for script tasks

Describe the bug
When I use the following folder structure, no prefix is applied to the unit task.
mise
└── tasks
    └── test
        └── unit

✓ mise tasks
Name  Description  Source
unit               /tmp/mise/tasks/test/unit

According to the documentation, I expect the name of the task to be test:unit. Note that the main folder is mise, if I use .mise instead, it works as expected.
I'm not sure if this is expected and it's just a documentation issue, but I would prefer to put the tasks in mise for more visibility.
To Reproduce
install -D -m 755 /dev/null /tmp/mise/tasks/test/unit
cd /tmp/mise
mise tasks

Expected behavior
The task is grouped according to the folder structure.
✓ mise tasks
Name       Description  Source
test:unit               /tmp/.mise/tasks/test/unit

mise doctor output
version: 2024.6.2 linux-x64 (2024-06-07)
activated: yes
shims_on_path: no

build_info:
  Target: x86_64-unknown-linux-gnu
  Features: DEFAULT, NATIVE_TLS
  Built: Fri, 7 Jun 2024 11:48:31 +0000
  Rust Version: rustc 1.78.0 (9b00956e5 2024-04-29)
  Profile: release

shell:
  /usr/bin/zsh
  zsh 5.8 (x86_64-ubuntu-linux-gnu)

dirs:
  data: ~/.local/share/mise
  config: ~/.config/mise
  cache: ~/.cache/mise
  state: ~/.local/state/mise
  shims: ~/.local/share/mise/shims

config_files:
  ~/.config/mise/config.toml

backends:
  cargo
  core
  go
  npm
  pipx
  spm
  ubi

plugins:
  bun     (core)
  deno    (core)
  erlang  (core)
  go      (core)
  java    (core)
  node    (core)
  python  (core)
  ruby    (core)
  zig     (core)

toolset:
<EMAIL_ADDRESS>

env_vars:
  MISE_SHELL=zsh

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf_compat = false
  cargo_binstall = true
  color = true
  disable_default_shorthands = false
  disable_tools = []
  experimental = true
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  http_timeout = 30
  jobs = 4
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  node_compile = false
  not_found_auto_install = true
  paranoid = false
  plugin_autoupdate_last_check_duration = "7d"
  python_default_packages_file = "<redacted>/.default-python-packages"
  python_pyenv_repo = "https://github.com/pyenv/pyenv.git"
  raw = false
  trusted_config_paths = []
  quiet = false
  verbose = false
  yes = false
  ci = false
  debug = false
  trace = false
  log_level = "info"
  python_venv_auto_create = false

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false

No warnings found
No problems found

This might be an oversight. Documentation states that tasks can reside in either .mise/tasks, mise/tasks or .config/mise/tasks/ but implementation does not yet contain the mise/tasks use-case.


Memdb: Unify revisions between Namepaces and Relationships

Memdb has a separate changelog table for namespaces and relationships. Our other datastores use a unified revision space between those operations, which means that you can pass a revision from a namespace operation to a future relationship operation (e.g. watch).
Memdb should have its revision generation (and likely changelog implementation) unified to bring it in line with other datastores.

Fixed by #319


Remove InvokeCommand overrides with ICommand

Do you want to request a feature or report a bug?
feature
What is the current behavior?
InvokeCommand currently takes a ICommand or a ReactiveCommandBase<TParam, TResult>. Unfortunately, as all ReactiveCommand objects implement ICommand it becomes problematic when you InvokeCommand a ReactiveCommand that has mismatched input parameters.
What is the expected behavior?
This should easily be a compile time check but since ReactiveCommand extends from ICommand it falls back to that method override and provides no static time checks. Then, when you run the app and a input of bool invokes a command with a TParam of Unit the app crashes :/
What is the motivation / use case for changing the behavior?
Better experience with ReactiveUI.
Which versions of ReactiveUI, and which platform / OS are affected by this issue? Did this work in previous versions of ReativeUI? Please also test with the latest stable and snapshot (http://docs.reactiveui.net/en/contributing/snapshot/index.html) versions.
Latest: 7.4.0.
Other information (e.g. stacktraces, related issues, suggestions how to fix)

I sympathise with what you're saying, but two points:

removing the overload means there is no way to InvokeCommand against a non-reactive command. That would be a big breaking change.
the crash you get when the types mismatch results in a very clear and concise message telling the developer exactly what they've done wrong: Command requires parameters of type {0}, but received parameter of type {1}.

Is using InvokeCommand against a non-reactive command a large use case? It would seem like if you're using ReactiveUI you're trying to stay within the reactive family - just my assumption. Regardless, while the error message is definitely better than nothing, it seems silly to not use static checking to our advantage here. If the breakage is as big as you say, perhaps my suggestion should be altered to, instead of removing an override, create a new InvokeReactiveCommand extension with just ReactiveCommand arguments.
This, of course, is not a huge issue because I can create a version of InvokeReactiveCommand in my own app in 1 line of code. My concern is merely a quality of life one. I'd rather see ReactiveUI make it hard to write misbehaving apps and this linkage with InvokeCommand seems odd.

Don't get me wrong - I 💯 agree with you. It's just that breaking changes like this are generally frowned upon. I personally don't use ICommand directly at all, but I can't speak for the community.
I was thinking along the same lines as you, except in reverse. Instead of InvokeReactiveCommand I was thinking InvokeICommand, but whatever. Regardless, it may be tricky to come to some consensus. In my experience, you can raise awareness through as many channels as you like (Slack, GitHub, Twitter) and many people don't notice until they update to that release, then they complain that their code has been broken by the update.
Let's see what feedback comes through on this. Thanks for raising it @thedillonb.


Clarity around fees

When doing a loop in, the fee that is presented may not be the actual fee that is paid.

In this example, the fee is quoted as 104,809 sats which seems incredibly expensive.
In reality though the on actual onchain transaction was broadcast with a fee of 0.00009517 BTC (62.4 sat/vB) - much more reasonable.
Suggest to present fee as "maximum fee" so that it's clear that this is an upper estimate and may not be the actual price.
Also, rather than only presenting the upper fee scenario could we present other (probably more likely) scenarios?
Also suggest showing the full breakdown of the fees so that it's easier to understand where the cost is going.

Thanks for the feedback 👍

Suggest to present fee as "maximum fee" so that it's clear that this is an upper estimate and may not be the actual price.

This is a good point, and it's an easy enough fix.

Also, rather than only presenting the upper fee scenario could we present other (probably more likely) scenarios?

I'm not 100% sure I understand fully. What do you mean by more likely scenarios? Currently, when we request a quote for a loop via gRPC, loopd will return the max onchain fee (htlcPublishFeeSat) and the service fee (swapFeeSat). I'm not sure I'm following how we can display a different scenario in the UI here.

Also suggest showing the full breakdown of the fees so that it's easier to understand where the cost is going.

Yes, this is definitely something I've also wanted to add. I thought we had an issue open for this but I cannot find it at the moment.

Hey @jamaljsr  My understanding is that the fee that loop provides in its quote is 100x the current onchain fee in order to cover the case where there is a sudden fee spike. But this is an exception and not the rule. In most normal cases the actual fee paid would be considerably lower than that quoted so I would say that a "more likely scenario" is one where the fee equals the current fee estimate rather than the 100x buffer case.


Tabbar is moved out of the screen alongside the cells

hello，If there is tabbar, tabbar, how can we not follow the move?

Can you elaborate a bit more?

I use is on the home page, the home page, there is a uitabbar. When I click on the home page of the cell into the details and then return. Uitabbar also will follow the animation. I want to return when UITabBar does not need the animation.

Uhm, the tab bar animates? That's odd, can you include a video capture of the app by any chance?

Click this link to see the effect, I do not want the bottom part of the blue don  https://github.com/AllLuckly/AllLuckly.github.io/blob/master/12.gif?raw=true

Oh, I'm an idiot, I was thinking about another library :grimacing:
Ok, can you show me your visibleCells method?
You are pushing the new controller in the navigation controller's stack, right?

yes，then back，Sorry, my English is poor.

No problem. Can you post your visibleCells method implementation?

(id)navigationController:(UINavigationController )navigationController
animationControllerForOperation:(UINavigationControllerOperation)operation
fromViewController:(UIViewController)fromVC
toViewController:(UIViewController*)toVC
{
if (operation != UINavigationControllerOperationNone) {
  return [AMWaveTransition transitionWithOperation:operation];

}
return nil;
}

(NSArray*)visibleCells {
return [self.tabView visibleCells];
}

(void)dealloc
{
[self.navigationController setDelegate:nil];
}

Looking at your demo writing.

Looks good. Mmmm, it might be a bug, the tabbar should remain in place. I'll take a look at it.

oh，thanks，The effect is very good

Hey, have any news on this bug?


Create Log In/Sign Up Cards

Sign up cards should include both email/password options along with different providers, like Gmail.
Call our User Auth lib to authenticate and change user state

i can take this


191 SIM-Mock Tutorial execution

This PR includes a new tutorial store that mocks most of the node calls for the tutorial execution

Code editor step is broken

2. Can't get further than the "Deploying Contracts" step
3. When refreshing, the opened contracts are closed

merged with latest and tested @cristiam86

Was merged in another branch / PR https://github.com/yeagerai/genlayer-simulator/pull/240


Deletion of old resource when spec.Identifier field is updated

There should be a standard defined across ACK service controllers on how to handle reconciliation of a resource if the spec.Identifier field is updated.
For Example, Currently if K8s user updates the Name of s3 Bucket resource, ACK reconciler will create a new bucket and not delete the old bucket.
As part of this issue, it should be decided,
a) Is it ACK service controller's responsibility to delete the old dangling resource?
b) If yes, how to store the information about old identifiers?
c) How to handle the deletion of old resource in a standard way across all ACK service controllers?
Note: This issue originated from a discussion here

Yeah I'd love to see an open discussion on what a user would expect, here.
My assumption would be that it would delete a previously created resource, since an "update" to the name would be the equivalent of taking the current resource and recreating it with a new name (deleting the old one in the process). But whenever dealing with automated deletion of resources, I agree we need to take care. Perhaps this could be gated by an annotation as well.

/lifecycle frozen


Need ability to represent VRF systems

Opening a new issue to bring attention to a previously discussed topic: https://github.com/BrickSchema/Brick/pull/344
Fan Coil Unit/ Indoor Unit
The term "Indoor Unit" is equivalent to Brick's existing "Fan Coil Unit". We can also improve our ability to represent the various types of Indoor Units/ Fan Coil Units by defining the following:

Wall [Mounted] Fan Coil Unit

Ceiling [Mounted] Fan Coil Unit

Concealed/ Duct Fan Coil Unit

Floor [Mounted] Fan Coil Unit

Condensing Unit/ Outdoor Unit
The term "Outdoor Unit" is also referred to as a "Condensing Unit", which needs to be added to Brick. Condensing Unit is not to be confused with Condenser, which is a core component of a Condensing Unit along with a compressor and, in some cases, a reversing valve.
We can categorize Condensing Units as follows:

Heat Pump Condensing Unit
Cooling Only Condensing Unit

Branch Selector
It is worth mentioning that up to this point, all concepts can be used interchangeably with conventional cooling and heating systems and VRF systems. The distinction often lies in the presence of a "Branch Selector" between the FCUs and Condensing Units. Therefore, Brick will also need to introduce a Branch Selector Box class.

If "Outdoor Unit" and "Indoor Unit" must be represented, we should make these an alias to "Condensing Unit" and "FCU" respectively.  In summary, I propose introducing:

Condensing Unit
Branch Selector
Wall Fan Coil Unit
Ceiling Fan Coil Unit
Floor Fan Coil Unit
Duct Fan Coil Unit

Additionally, when it comes to distinguishing between air-sourced and water-sourced condensing units, we can usually infer this based on its connection to a chilled water system. However, it might be worthwhile to have a discussion on whether we should explicitly define air-sourced and water-sourced condensing units, or if inferring their type based on their relationships within the system is sufficient.

Wall mounted doesn't look like that. Looks like:

Ceiling Unit is typically called "Cassette".

The outdoor units should be categorized as Heat Recovery, Heat Pump, and Cooling only. Heat Recovery entails more than just the branch selectors diverting refrigerant. The outdoor unit is a special unit that is capable of mixed mode operation sending hot gas to the indoor and outdoor coils simultaneously.
Here is a good video explanation:
https://youtube.com/watch?v=r-NQEoPidX8

@rmartinez-avila Thanks for the feedback and yes CW, not CHW. Cassette sounds good. Here's the updated proposal:
.
├── Branch_Selector
├── Condensing_Unit
│   ├── Cooling_Condensing_Unit
│   ├── Heat_Pump_Condensing_Unit
│   └── Heat_Recovery_Condensing_Unit
└── Fan_Coil_Unit
    ├── Cassette_Fan_Coil_Unit
    ├── Duct_Fan_Coil_Unit
    ├── Floor_Fan_Coil_Unit
    └── Wall_Fan_Coil_Unit

I realize it is common to use Indoor and Outdoor Unit to describe VRF equipement, do you have any opposition to using Condensing Unit and Fan Coil Unit?

"Ceiling Mounted Cassette" would be complete.

I think Condensing Unit and Fan Coil Unit are fine. Those terms are used for VRV/VRF all over the place and I've even seen them in some of their manuals.

PS Your original picture (labelled "wall mounted") is what's often called a bulkhead system.

@david-waterworth,
I appreciate your input regarding the bulkhead system. I must admit that my choice of illustration may not have been ideal, as my intention was primarily to demonstrate the operational concept of a wall-mounted FCU. I acknowledge that the illustration may not accurately represent a wall-mounted FCU, and I believe that rmartinez's image clarifies a wall mounted FCU most accurately.
I am open to receiving concrete feedback and possible concerns regarding the definition of an FCU.

The AHRI Performance Rating of Variable Refrigerant Flow (VRF) Multi-split Air-conditioning and Heat Pump Equipment has definitions that might be helpful here. For example...

3.2.21 Indoor Unit. A separate assembly of a Split System, excluding a service coil, that includes all of the following
features:

An arrangement of refrigerant-to-air heat transfer coil(s) for transfer of heat between the refrigerant and the
indoor air
A condensate drain pan
An air temperature sensing device
An integrated indoor blower such as a device to move air including the device’s associated motor

3.2.29 Outdoor Unit. A separate assembly of a Split System that transfers heat between the refrigerant and the outdoor
air or refrigerant and water, and consists of an outdoor heat exchanger, compressor(s), an air moving device, and in
addition for Heat Pumps, can include a heating mode expansion device, reversing valve, or defrost controls, or any
combination thereof. VRF Water-source Heat Pumps do not have an air movement device.


Add --user command line option to aerys

AFAIK, and correct me if this is wrong, it's only possible to pass the user, from which workers will run, through a constant called AERYS_OPTIONS. Ex.:
const
    AERYS_OPTIONS = [
        "user" => "marcio",
        "defaultContentType" => "application/json",
        ...
    ]
;

This is fine if you run from the same unprivileged user on all your dev and prod servers. But that can be a PITA if you need to set different users for different servers - because we obviously can not getenv("MY_API_USER") on a constant declaration in PHP.
I'd like to suggest a -u and --user option to aerys command line interface for that purpose. The rational is simply convenience - at least I noticed the "user" option is the only one that will certainly vary and can't be getenved like the exposed port.
Thoughts?

You can use getenv when using define instead of const, but a command line option is better here I guess.


Option to delay convert until method called

As far as I'm aware, there's currently no way delay a media conversion, it just starts as soon as the recording has stopped. Could there be a way to delay the conversion? Say player.record().convert(); This would allow the user to first review the video prior to converting.

sounds like a good enhancement.

For anyone looking for this in the meantime, I've just implemented it directly for now. Referencing the examples provided at https://github.com/ffmpegwasm/ffmpeg.wasm and converting this.player.recordedData

Thanks for the update @thijstriemstra


Make the code more legible

In an attempt to wrap my head around createElement function, I swapped the arrow functions for named functions.  I would argue that this slighly less clever approach is more readlable to an outsider.

Oh yes, very good. And now I think the createElement function should probably return a type Element which would return Render function (or Renderer).


fix(ci): apply lessons learned to monthly roadmap reminder cross-repo

Signed-off-by: heitorlessa<EMAIL_ADDRESS>Issue number: #4074
Summary
Main lesson learned is that when using cross-repo workflows that use local scripts, we must change the checkout in the target workflow to explicitly checkout itself.
What do I mean by that?
Take the following job in mind
jobs:
    call-workflow-passing-data:
        permissions:
            contents: read
            pull-requests: read
            issues: write  # create monthly roadmap report
        uses: aws-powertools/actions/.github/workflows/monthly_roadmap_reminder.yml@main
        secrets:
            token: ${{ secrets.GITHUB_TOKEN }}

There are two implicit designs that aren't easy to understand at first and docs don't call that out:

aws-powertools/actions is where the workflow is
monthly_roadmap_reminder.yml is the workflow name
The workflow runs in this repository

With that in mind, the target workflow naturally would try to do a code checkout as its first step. However, that code checkout will checkout this repository (Python source code), not the shared actions we want.
For this to work, the target workflow owned by aws-powertools/actions must explicitly pass which repository it will checkout code -- it can't be via ${{github.repo}} because it would resolve to powertools-lambda-python.
jobs:
  build:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@9bb56186c3b09b4f86b1c65136769dd318469633  # v4.1.2
        with:
            # NOTE Since it's called from another repository
            # we must checkout the repository where the github-script is (aws-powertools/actions)
            # otherwise the caller check out their own repository code (e.g., Python/TS), where the github-scripts aren't available
            repository: aws-powertools/actions

A nicer benefit of this is that we don't need to control from the target who can call it and try to abuse creating GitHub issues in another place, because it's the caller repository that runs the workflow (it's not a fire-n-forget).
Changes

Please provide a summary of what's being changed

User experience

Please share what the user experience looks like before and after this change

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Don't merge - I forgot two things:
PR review status column, and point to main over SHA until we have actions releases


Move geo concerns

Make the geo_concerns the root directory. Move experiments and use cases into into misc. Closes #13.

Ping @jamesrgriffiniii @drh-stanford.

:+1: @eliotjordan  All tests pass within my CentOS 6.6 environment using ruby-2.2.1.

Thank you for this @eliotjordan and @drh-stanford.
For reference, I have triggered the generation the following RubyDocs:
http://www.rubydoc.info/github/projecthydra-labs/pcdm-geo-models/master
I shall review these as I proceed with other matters.


Events not displayed on week view (only on month view)

Hey,
From some reason Im fetching events from server and pass them to vue-cal, the events are displayed correctly in month view but on week view they not displayed.
On week view, the specific day that should display the events is just empty.
See images for more info:

Hey! I also have this problem. Have you found a solution?

sorry, I put the time and now it works :)

Hi @netanelwebninja,
From your screenshot I can see you are using the time option, all the events must have a time defined in the start and end properties - like in your event in position 0.
The event in position 1 does not. Also the event one has no duration, it might be there but with a height of 0.
Let me know.
@enzonotario, can you also share your code, so I can see how to help?


Support plugin only request during plugin initialization

connect #219

@ben181231, thanks for your PR! By analyzing the annotation information on this pull request, we identified @rickmak to be a potential reviewer

I will leave more comments


Function (built in to load experiment) to interpret loaded experiment and output a summary.

For this I think we should try and have it output the code used to set up said experiment. For example, in demo nb 1 it would output something like:
"""
devices = ps.ItemAttribute()
devices.v1 = ps.TestVoltage() # Device 1
devices.v2 = ps.TestVoltage() # Device 2
devices.v3 = ps.TestVoltage() # Device 3
def get_voltage_data(expt):
"""
Reads the voltage from v1, v2, and v3 devices. Also adds a calculated value vsum.
"""
runinfo = expt.runinfo
devices = expt.devices

d = ps.ItemAttribute()

d.v1_readout = devices.v1.voltage
d.v2_readout = devices.v2.voltage
d.v3_readout = devices.v3.voltage

d.vsum = d.v1_readout + d.v2_readout + d.v3_readout

return d

runinfo = ps.RunInfo()
runinfo.scan0 = ps.PropertyScan({'v1': ps.drange(0, 0.1, 1)}, prop='voltage', dt=0.01)
runinfo.measure_function = get_voltage_data
expt = ps.Experiment(runinfo, devices, time=True)
expt.start_thread()  # or alternatively expt.run()
"""
I think this could offer a quick overview of how the experiment was designed and be valuable to experimenters wanting to understand old experiments.


pydrive.auth's TimeoutError

Hi there,
I am having issues authenticated via pydrive.auth.
When I execute my basic code, api_practice.py file:
from pydrive.auth import GoogleAuth
from pydrive.drive import GoogleDrive
gauth = GoogleAuth()
gauth.CommandLineAuth()
drive = GoogleDrive(gauth)
f = drive.CreateFile({'title': 'test.txt'})
f.SetContentString('test pydrive')
f.Upload()

in the same path, My settings.yaml file contains:
client_config_backend: settings
client_config:
  client_id: 51***************f.apps.googleusercontent.com
  client_secret: Z1***************

save_credentials: True
save_credentials_backend: file
save_credentials_file: credentials.json
get_refresh_token: True
oauth_scope:
  - https://www.googleapis.com/auth/drive.file
  - https://www.googleapis.com/auth/drive.install

and follow the output url, and grant permissions (via the web browser),
input Enter verification code,then there is a error:
Traceback (most recent call last):
  File "api_practice.py", line 7, in <module>
    gauth.CommandLineAuth()
  File "C:\ProgramData\Anaconda3\lib\site-packages\pydrive\auth.py", line 125, i
n _decorated
    self.Auth(code)
  File "C:\ProgramData\Anaconda3\lib\site-packages\pydrive\auth.py", line 495, i
n Auth
    self.Authenticate(code)
  File "C:\ProgramData\Anaconda3\lib\site-packages\pydrive\auth.py", line 508, i
n Authenticate
    self.credentials = self.flow.step2_exchange(code)
  File "C:\ProgramData\Anaconda3\lib\site-packages\oauth2client\_helpers.py", li
ne 133, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "C:\ProgramData\Anaconda3\lib\site-packages\oauth2client\client.py", line
 2054, in step2_exchange
    http, self.token_uri, method='POST', body=body, headers=headers)
  File "C:\ProgramData\Anaconda3\lib\site-packages\oauth2client\transport.py", l
ine 282, in request
    connection_type=connection_type)
  File "C:\ProgramData\Anaconda3\lib\site-packages\httplib2\__init__.py", line 1
514, in request
    (response, content) = self._request(conn, authority, uri, request_uri, metho
d, body, headers, redirections, cachekey)
  File "C:\ProgramData\Anaconda3\lib\site-packages\httplib2\__init__.py", line 1
264, in _request
    (response, content) = self._conn_request(conn, request_uri, method, body, he
aders)
  File "C:\ProgramData\Anaconda3\lib\site-packages\httplib2\__init__.py", line 1
187, in _conn_request
    conn.connect()
  File "C:\ProgramData\Anaconda3\lib\site-packages\httplib2\__init__.py", line 1
050, in connect
    raise socket_err
  File "C:\ProgramData\Anaconda3\lib\site-packages\httplib2\__init__.py", line 1
011, in connect
    sock.connect((self.host, self.port))
TimeoutError: [WinError 10060] 接続済みの呼び出し先が一定の時間を過ぎても正しく
応答しなかったため、接続できませんでした。または接続済みのホストが応答しなかった
ため、確立された接続は失敗しました。

I have tried two method to solute this problem,but Useless
a. maybe it's a proxy problem,I tried add proxy setting into basic code→same err
b. I tried make a new client_id and client_secret→same err
Does anyone have any ideas what is going on here?

Hi there. It seems pydrive is using the deprecated oauth2client library. I would encourage you to file this bug on the pydrive repo, and I'd be happy to help them with the migration to this library.

I think I got it.
Storage Server seems limited by company network,
I have cleaned this problem on my private PC.
Thank you.


fix(ldap-plugin) cannot be applied on a consumer

Full changelog

LDAP Plugin cannot be applied to a specific consumer.

Issues resolved
Fix #2235

doubles with #2237


Unable to Import COCO, PASCAL, YOLO annotations - Error 500 [Urgent Support Requested]

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

When I try to import annotations for a small dataset, CVAT online as well as the locally hosted version throws an error 500 with the error
datumaro.components.errors.datasetnotfounderror: failed to find dataset at '/home/django/data/projects/1/tmp/tmptsnzb3r6'\n"..

in the console.
Steps to Reproduce (for bugs)

Create a Project "XYZ" with 5 labels.

Create a Task and upload images directly (no archive) with local storage option and 100% image quality.

Open the job and upload annotations (archive with just annotation files or JSON file) - I have coco json, yolo, and pascal voc xml annotations - which need to be verified and updated.

The YOLO data are in the format:
images
| 1.jpg
| 2.jpg
...
labels
| 1.txt
| 2.txt
...
data.yaml
data.names
obj.data
train.txt

The COCO JSON data is in the same format except that the images folder has a file named _annotations.json, and there are no labels, data.yaml, etc files and folders.

The PASCAL VOC XML data is in the same format except that the images folder has 1.xml, 2.xml etc files and there are no labels, data.yaml, etc files and folders.

CVAT will take the annotations in all formats, and throw the same error after some time and ask to check console.

This is happening with many datasets, also sometimes for 5-7 images it works!

Annotations are not flawed - model trains, tested on roboflow, makesense, and labelimg and it works! Want to use CVAT for better processing as always.

Expected Behaviour

Old behaviour was that when I used CVAT earlier (a couple of months), with the same dataset, it used to take the old yolo format archived labels folder with data.yaml. This does not work.
Expected behaviour is that with the correct dataset for all formats, it should accept and display the annotations.

Current Behaviour

error  500

datumaro.components.errors.datasetnotfounderror: failed to find dataset at '/home/django/data/projects/1/tmp/tmptsnzb3r6'\n"..

OR

Possible Solution

No Idea!

Context

Annotating data for a project, need urgent support.

Docker
Operating System and version (e.g. Linux, Windows, MacOS):
MacOS 13.3 - 2.6 GHz 6-Core Intel Core i7
Code example or link to GitHub repo or gist to reproduce problem:
NA
Other diagnostic information / logs:

[2023-04-05 05:40:54,471] WARNING django.request: Unauthorized: /api/auth/password/change
[2023-04-05 05:40:55,150] WARNING django.request: Not Found: /api/user-agreements
[2023-04-05 05:41:16,658] WARNING django.request: Not Found: /api/functions/requests/
[2023-04-05 05:43:36,643] INFO cvat.server.project_1: label:create Label id:1 for spec:OrderedDict([('name', 'Ready'), ('type', 'any'), ('color', '#689981')]) with sublabels:[], parent_label:None
[2023-04-05 05:43:36,654] INFO cvat.server.project_1: label:create Label id:2 for spec:OrderedDict([('name', 'empty_pod'), ('type', 'any'), ('color', '#a38aca')]) with sublabels:[], parent_label:None
[2023-04-05 05:43:36,666] INFO cvat.server.project_1: label:create Label id:3 for spec:OrderedDict([('name', 'germination'), ('type', 'any'), ('color', '#dff59f')]) with sublabels:[], parent_label:None
[2023-04-05 05:43:36,678] INFO cvat.server.project_1: label:create Label id:4 for spec:OrderedDict([('name', 'pod'), ('type', 'any'), ('color', '#4a8cf4')]) with sublabels:[], parent_label:None
[2023-04-05 05:43:36,690] INFO cvat.server.project_1: label:create Label id:5 for spec:OrderedDict([('name', 'young'), ('type', 'any'), ('color', '#7ea09e')]) with sublabels:[], parent_label:None
[2023-04-05 05:43:37,276] WARNING django.request: Not Found: /api/functions/requests/
[2023-04-05 05:44:48,373] INFO cvat.server: create task #1
[2023-04-05 05:46:17,244] INFO cvat.server: Found frames 1510 for Data #1
[2023-04-05 05:46:17,245] INFO cvat.server: New segment for task #1: idx = 0, start_frame = 0,             stop_frame = 1509
[2023-04-05 05:46:18,390] INFO cvat.server.task_1: get repository request
[2023-04-05 05:46:28,773] WARNING django.request: Not Found: /api/functions/requests/
[2023-04-05 05:46:31,671] INFO cvat.server.task_1: get repository request
[2023-04-05 05:47:26,321] ERROR django.request: Internal Server Error: /api/jobs/1/annotations
[2023-04-05 05:47:51,966] WARNING django.request: Not Found: /api/user-agreements

Hi, please make sure the uploaded annotations and dataset use the file layouts described here.

Would it need exact naming convention?
obj_train_data, and so on?

Image file names can be arbitrary, given they are listed in the corresponding subset file (train.txt, valid.txt). Everything else should be the same as in the description. This file layout was borrowed from the YOLO training framework, meaning it should be ready-to-use in the practical use cases. Could you please specify which elements you would like to be optional in the file structure?

If anyone has the same problem, i was able to solve it for the YOLO case by including the images alongside the annotation files.


pep8: Fix E128.

Automated message from Dropbox CLA bot
@bulat22101, it looks like you've already signed the Dropbox CLA. Thanks!

I merged 94b28e0afb0ee97a4302dfbadbc49806d4c0c097, which implements this change; it looks like the two of you did basically the exact same thing, which is at least a nice verification that this was done properly.  Thanks @bulat22101!!


[MXNET-514] Add clip_global_norm(row_sparse_grad). Fix row_sparse_param.save(). Fix trainer init_kvstore

Description
As title. Test cases are updated accordingly.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@eric-haibin-lin I have this test failing in my build, is it flaky or an actual failure you reckon?
test_trainer_reset_kv

@ThomasDelteil let me look into it

It's a flaky one. I don't see it fail on master build. Maybe it's to do with param.load/save on windows...

https://github.com/apache/incubator-mxnet/issues/11353

http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-11379/1/pipeline


arm64/pinephone: Add driver for PinePhone LTE Modem (Quectel EG25-G)

Summary
This PR adds the PinePhone Driver for Quectel EG25-G LTE Modem. When the driver is started, the LTE Modem accepts AT Commands over UART3 and /dev/ttyS1.
PinePhone's LTE Modem is explained in this article.
New Files in boards/arm64/a64/pinephone
src/pinephone_modem.c, pinephone_modem.h: Driver for Quectel EG25-G LTE Modem
configs/modem/defconfig: New Build Configuration pinephone:modem that enables the LTE Modem at UART3 and /dev/ttyS1
Modified Files in boards/arm64/a64/pinephone
Kconfig: Added Kconfig Setting PINEPHONE_MODEM to enable the LTE Modem at "Board Selection > Board-Specific Options > LTE Modem"
src/pinephone_bringup.c: Start the LTE Modem if PINEPHONE_MODEM is selected
src/Makefile: Added LTE Modem Driver to Makefile
Updated Documentation
platforms/arm/a64/boards/pinephone/index.rst: Added Build Configuration pinephone:modem that enables the LTE Modem
Impact
With this PR, NuttX Apps will be able to control the PinePhone LTE Modem with AT Commands over UART3 and /dev/ttyS1.
There is no impact on existing code because the driver is not used currently.
Testing
We tested the LTE Modem Driver on PinePhone by sending AT Commands. For Regression Testing, we tested PinePhone with the LTE Modem Driver disabled.
LTE Modem Test
To test the LTE Modem Driver, we selected the pinephone:modem Build Configuration:
tools/configure.sh pinephone:modem
make

We ran a Modem Test App that sends AT Commands to make phone calls and send SMS messages:

Modem Test App: hello_main.c
(As explained here)

The modem responds correctly to our AT Commands:

Modem Test Log

Regression Test
For Regression Testing, we selected the pinephone:lvgl Build Configuration that disables the LTE Modem:
tools/configure.sh pinephone:lvgl
make

NuttX boots correctly and starts the LCD Display, with the LTE Modem disabled:

Regression Test Log

Fixed the UART3 configuration. Sorry about that.

Sorry I shall take some time to rethink and redo this PR:

Create a Generic Modem Interface for NuttX that will support Modems for any SoC. It shall support PinePhone's Quectel EG25-G LTE Modem, including AT Commands over UART, AT Commands over USB, GNSS (NMEA 0183) over USB. And potentially other features like Direct-IP, USRSOCK, MBIM and QMI.
It's probably my mistake to call this PR a "Modem Driver". All it does is to Power Up the Quectel LTE Modem by toggling some GPIOs. I wished we had a better way to do this without creating a Generic Modem Interface.
Disable UART2 and /dev/ttyS2 for PinePhone, so that UART3 will map to /dev/ttyS3 for the Quectel LTE Modem. This will make it easier for NuttX Apps to access the LTE Modem.

If anyone has started working on the Generic Modem Interface for NuttX, I'm happy to join you :-) Meanwhile I'll check out whether Zephyr OS has something similar.


📘 DOC: Arabic Layout theme have an issue

Preview:
https://user-images.githubusercontent.com/54203033/144560112-1bcc051f-1f34-4d30-9193-9a5c41cc1267.mp4
There is an issue with arabic theme, some MarkDown go after footer and Display as LTR not sure why.

Nuh, It's solved.
I will close it.


Updated AI Builder Form Processing lab sample documents

Added new sample documents for Form Processing in AIBuilder_Lab.zip
Added new zip file with only Form Processing sample documents

Hi @Dean-Haas, can you please review this PR? Thanks!


Enhance poetry related logging and add new flag

Distinct logging whether the requirements.txt file is generated from poetry.lock vs pyproject.toml.
New boolean flag: requirePoetryLockFile - When set to true, fail the build when poetry.lock is missing. This helps with creating reproducible builds where you want to have a fixed poetry.lock file instead of one generated on the fly.

Follow up to https://github.com/serverless/serverless-python-requirements/pull/639

And I forgot - it would be great to cover that functionality with a test case 💯

Hey @FinchPowers, thanks a lot 🙇 As for testing, it's using tape, but it's pretty barebones as you've noticed and kinda looks like a custom implementation. I don't have much experience with it as well, but I think it would be best to avoid mocking, so something similar to your suggestion 2 would be best 👍 I don't think you will have to copy poetry anywhere, but you should be able to just use that "fixture" as each test is run separately and in isolation (see setup and teardown methods). If you have any questions, please let me know 💯


Disable "misleading indentation" warning on GCC

This warning has been bugged in GCC for the past several versions: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89549
Aside from that, this warning isn't relevant to Godot anymore, since we enforce braces now in the master branch.
Bugsquad edit: Fixes #26443.

Thanks!

Cherry-picked for 3.2.3.

All right, starting with GCC 11 we have a new option (-flarge-source-files) that should address this issue:
https://gcc.gnu.org/onlinedocs/gcc/Preprocessor-Options.html#index-flarge-source-files
Can you please try enabling it?


Missing Pagination for List of Directs

Describe the bug
I am fetching directs via client.users.by_user_id(user_id).direct_reports.get(). The list is incomplete for users with many directs. Browsing through the code a bit, it seems that this API ignores the nextLink property and only returns the first page.
Expected behavior
The list of returned directs should be complete.
How to reproduce

create user user_id with many directs
fetch direct reports of this user with client.users.by_user_id(user_id).direct_reports.get()
confirm not all results are returned.

SDK Version
1.11.0
Latest version known to work for scenario above?
none.
Known Workarounds
No response
Debug output
Click to expand log
```

</details>

### Configuration

- OS: Ubuntu 22.04
- arch: x86

### Other information

_No response_

@samwelkanda @baywet -- tagging some regulars here, are you guys monitoring the issues list? Thanks!

Hi @temporaer
Thank you for using the SDK and for reaching out.
You can use the page iterator to page through the different pages automatically.
Unfortunately, there's no documentation sample for Python yet, but using the other languages and reading the source you should be able to extrapolate. CC @jasonjoh

@baywet , why is this not a bug? Why isn't pagination implemented without user intervention like for other list operations?

@temporaer depending on the application and requirements, some scenarios might require a full enumeration of all the elements (e.g. auditing data sets), some might need to stop mid-way through (e.g. user clicking a UI and paging through), and others might only need the first page (e.g. recap of the last few elements).
We cannot assume that every application out there is going to need all items of the collection, that'd put unnecessary burden on the service and the client applications for scenarios where it's not needed.
Let us know if you have any additional comments or questions.

@baywet , you could return an iterator instead of a list, as is common practice. Then, each application can decide whether to fetch the next page, without adding unnecessary burden on the service.

We did discuss that in the early design of this new generation of SDKs and decided to keep things simple for now.
https://github.com/microsoft/kiota/issues/1569

@baywet , I see. That's a very different argument though. In this case, please make sure this limitation is clearly documented, and a (potentially more involved) way to list all directs is provided. Thank you!

I'm not super sure about the linked context -- it fseems that there, the work was completed, but it doesn't seem to explain why it wasn't picked up in the newer version of the SDK. Is there generally no pagination support, or are there certain list operations that you left out intentionally?

Ah sorry, I assumed the context had been shared from internal discussions.
The gist being: we needed a way to generate paging for the CLI, as layering CLI calls is more complex than in any other language.
We were faced with a choice between building that in a bespoke way for the CLI only, or making it generic for any language that kiota supports.
In the end, to prioritize the release of the CLI, it was decided to take the shortest path, and NOT make things generic for other languages to include an additional "getAll" method (in addition to get) which would include paging.
Let us know if you have any additional comments or questions.


Is function "process_gen" and "process" used in trainer.py?

And could you please give me some hints or examples on how to train the model? By reading class "Trainer" I see the structure of method mentioned in FICE paper, but I still don't know how to get the train start.
I'm a student who is new to deeplearning and very interested in your research, please help me if it's convenient.

Hi.
This repository only provides trained models for inference purposes.
Would you like to train the models from scratch?

Thanks for your so rapid reply. Yes, I'd like to train the model from scratch using some other datasets. It'll be quite helpful if you could provide me some training codes.

I see. This seems to be a popular request. I will make sure to provide the training code in the near future.

Thank you very much. I'm looking forward to that. Thanks for your hard work and enthusiasm.

And may you please send some training codes(Unorganized ones is okay) to my educational email(zhangsc@mail.hfut.edu.cn)? This will also help a lot.

Done.
On Fri, 10 Feb 2023 at 05:28, zsMoonshine @.***> wrote:

And may you please send some training codes(Unorganized ones is okay) to
my educational @.***)? This will also help a lot.
—
Reply to this email directly, view it on GitHub
https://github.com/MartinPernus/FICE/issues/3#issuecomment-1425158592,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIZIL5YRYW4F3IVOTFSLZWDWWW7XLANCNFSM6AAAAAAUWHAIUQ
.
You are receiving this because you commented.Message ID:
@.***>

Thanks for your reply and share. And I need to spend some time reading them.
I think I will ask questions via email after that. Thanks for your patience and ethusiasm.

Sorry to bothering, I am a student who is working on Multimodal domain. I greatly admire your creative work in the field of image editing and want to train this model from scratch. Cound you please send the training codes to my<EMAIL_ADDRESS>if it doesn't bother you?

Commit 7354216 adds additional instructions about training from scratch!

Thanks a lot👍


openstack-crowbar: use onadmin_rebootcloud for MU testing

Call onadmin_rebootcloud instead of relying on the reboot_node
ansible role when testing maintenance updates. The onadmin_rebootcloud
functionality from qa_crowbarsetup.sh has additional logic built
into it which ensures that controller nodes are rebooted correctly
when HA is enabled, whereas the reboot_node ansible role does not.

Is it possible to implement the same logic into the ansible role?
Otherwise, LGTM.

included in #3527


Example of a OrchestrationStep Type ReviewScreen

Hi, can you please include a working example of a User Journey which uses an orchestrationStep with Type = "ReviewScreen"
I keep getting a 500 error when trying to submit it so im not sure what I am doing wrong
Thanks

Hello.
I am from the product team. As you may know we have these features under private preview and we are eager to get your feedback which helps us get this service ready for broader availability.
I assume you are getting the generic "500 internal server error."
Can I also assume that you already have other custom (uploaded) policies working correctly? and the problem seems to be restricted to the "ReviewScreen" orchestration step type?
and lastly, have you used the UserJourney  Recorder to be able to see where the error is coming up?
Are you seeing this in the browser after invoking the b2c custom policy?

Hi rojasja,
I get the 500 error when trying to submit with the review screen.  I think it is because my datauri in content definition is type selfasserted which fails if you dont submit any claims, but with review screen I dont get to wrap it in a technical profile so I cant supply claims.
is there a specific datauri type for review screens.
Yes i have tried the user journey recorder which points me towards the specific datauri solution
Thanks


Removed some patterns from gitignore

A lot of patterns seems to be useless for this project

And added slash where relevant

This is template file, don't see the point in touching


Think harder about how task ownership works

See https://youtu.be/btCLcFLvV2I?t=2881
Why does Player1 own Player2's task (4 BuyCard?)?
I think I decided that the task owner is who has the right to reify the task, so that is wrong.

For 1.0 we won't have anything that allows out-of-turn decisions. So let's say the task owner both:

chooses when to perform the task
chooses how to reify the task

I think the REPL and relevant APIs should require the task owner to be the one performing the task.
The mechanism for kicking off a turn in blue mode should probably require the engine to do exec GiveTurn<Player2>; then somehow Player2 needs to be the owner of the resulting task.

basically fixed, and #8 can rethink it a bit if necessary.


Skip tests for now

Tests are being flaky

Well, seems like they passed after a few tries so no need anymore


Automated cherry pick of #3699: fix: cpuCmtbound sync for zstack

Cherry pick of #3699 on release/2.11.
#3699: fix: cpuCmtbound sync for zstack

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@1f81c03). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #3719   +/-   ##
==============================================
  Coverage                ?   7.13%
==============================================
  Files                   ?     480
  Lines                   ?   77984
  Branches                ?       0
==============================================
  Hits                    ?    5565
  Misses                  ?   71953
  Partials                ?     466

Flag
Coverage Δ

#aFlag
7.13% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/hosts.go
0.38% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


 This is Github infra having trouble and the only workaround I saw so far was to trigger more commits

@andy31415  Problem not with build of new pull request, it just blocked because of absence docker tag 0.5.98 which wasn't builded by next commit https://github.com/project-chip/connectedhomeip/commit/5c2334a0d4ab173347cac2caad155fbb2f32d933 for any vendor.
If you check build of docker images you will see that it in progress (hung up) almost 5 days.

Maybe we can increase version number here "integrations/docker/images/chip-build/version" without any other changes, just to trigger again docker images build?

The docker images are manually built. I started a 0.5.98 build now.


Added failing test for re-renders with context

Children receive context on the initial render, but not on subsequent renders.
This PR adds a test with a failing assertion that demonstrates this behavior.

Note: this test passes on the master branch, so there is some regression introduced in dev.

@rtsao thanks for this, I'll get a fix for it as soon as I have some time today.


Loading austraits data is throwing strsplit bibtex error

library(austraits) austraits
$sources
Error in strsplit(tools:::BibTeX_entry_field_db[[bibtype]], "|", fixed = TRUE) :
non-character argument
Is this the R package's fault or the build? To be investigated

Might require "RefManager" to be loaded. Does that change anything?

Hooray!


Add builtin id(:any), returns a string (the address) of the arg.

There is a discussion, #14374, about using a class or enum as a dict key.
Consider java's HashMap and IdentityHashMap.
This PR provides a way to have an identity dict using dict or list or
object as the key. See below for an example comparing the difference. Without
something like this, it is impossible to create an efficient, or even close to
efficient, identity map.
I went with a short name, id, as is my wont. Maybe identity is better.
No help/tests yet, looking for feedback first.
job and channel could be included or anything that has a refcount. LMK.
This should be considered an initial implementation and probably should be
extended, or supplanted, in the future. Some issues are

Must manually keep a reference to an item used as a key. The id() call
does not increment the refcount.
Could add an option to id() to take a reference. But this requires
considerable consideration. For example need a decref and avoid SEGV.

Returns empty string if not VAR_OBJECT/VAR_DICT/VAR_LIST.
vim9script

var d1: dict<any>
var d2: dict<any>

d1 = {a: 1}
d2 = {a: 2}

var d: dict<any>
d[string(d1)] = d1
d[string(d2)] = d2

echo d

d->filter((_, _) => false)
d2.a = 1
d[string(d1)] = d1
d[string(d2)] = d2
echo d

# Now the same thing with "id()"

d->filter((_, _) => false)
d1 = {a: 1}
d2 = {a: 2}
d[id(d1)] = d1
d[id(d2)] = d2
echo d

d->filter((_, _) => false)
d2.a = 1
d[id(d1)] = d1
d[id(d2)] = d2
echo d

using string()
Equality is used as the index.
{'{''a'': 2}': {'a': 2}, '{''a'': 1}': {'a': 1}}
{'{''a'': 1}': {'a': 1}}

using id()
Identity is used as the index.
{'0x5e9614976fc0': {'a': 2}, '0x5e961497ab60': {'a': 1}}
{'0x5e9614976fc0': {'a': 1}, '0x5e961497ab60': {'a': 1}}

Went ahead and added blob/job/channel.

@zeertzjq Do those messages mean you approve this PR ;-)

Rebased.

Thanks, I think this is useful. However I think we should emphasize that this is machine dependent and is not portable (e.g. when using sessions, etc).

Did some wordsmithing to the help text, and added a simple test.
I need some clarification. I don't understand sessions (I've never knowingly used them), I scanned :he Session but I don't see how this interacts with that.
Is there further text and/or changes required to address any concerns?

I need some clarification. I don't understand sessions (I've never knowingly used them), I scanned :he Session but I don't see how this interacts with that.

Sessions may save global variables, if 'sessionoptions' contains "global" and if the variable starts with an uppercase letter, see :h 'sessionoptions'. But according to the documentation this is only supported for string and number variables. I am pretty sure I contributed a patch to fix this several years ago.... Oh wait, it wasn't for session files, but for the viminfo file, see :h 'viminfo'. So if you include the flag ! in your viminfo setting, Vim will store global variables (consisting only of upper case letters, grml, that is quite inconsistent...) in the viminfo file. It works for Dicts and Lists at least but not for deeply nested variables I think and I have actually no idea, how it behaves for the new types channels, jobs, blobs, etc.
So long story short, I suppose this won't work properly if such a variable is stored in the viminfo file. But I see you addressed this already in the updated documentation, so this is probably good enough.

Maybe you should also document how it treats null_dict etc.?

Thanks, I missed that entirely. Currently it returns "(nil)" for any of the null_*.
@zeertzjq, @chrisbra What about returning "null_list"/"null_dict"/... or whatever?

A few anomalies with b92408660, I think:
echo printf('"%s"', id(null_string))
echo printf('"%s"', id(""))

try
    echo id(null_class)
catch
    echo v:exception
endtry

A few anomalies with b92408660, I think:
echo printf('"%s"', id(null_string))
echo printf('"%s"', id(""))

try
    echo id(null_class)
catch
    echo v:exception
endtry

The exception with id(null_class) is expected. Notice that get(null_class, ''), or has(null_class), or len(null_class) all do the same thing.

My bad, I read ‘If the item is not a permitted type, then an
empty String is returned’ from the documentation for id()
into the output of id(null_class).
I would argue for id() to return either an empty string for
invalid arguments (and any null*s) or ‘an address’ string.

And let id(null_class) raise an exception.

Agree with the empty string

Also return an empty string for a null type.

I played with 5539ca a little: took adt.vim from here and
abc_gen.vim from here and rewritten the original test.vim:
vim9script

# Generate A-Z Letter implementations and declare "g:abc_src".
source abc_gen.vim

execute 'import "' .. g:abc_src .. '" as abc'
import './adt.vim'

# Create a Set (i.e. dict<number>) of abc.Letter elements.
const set: adt.Set = adt.Set.newFromList(
    # Populate this Set with elements from a list<abc.Letter>.
    abc.LETTERS,

    # Provide a lambda expression to map elements to dictionary keys.
    # (Available as set.ToStringer().)
    (o: abc.Letter): string => id(o),

    # Provide a lambda expression to map dictionary keys to elements.
    # (Here, partially apply a generated list that maps its indices to
    # object identities.)
    # (Available as set.FromStringer().)
    ((ii: list<string>) => (id: string): abc.Letter =>
			abc.LETTERS[index(ii, id)])(abc.LETTERS
					->mapnew((_, v) => id(v))))

# Recover all original objects.
for object in set.Elements()
    echo string(object) object is abc.ABC.Get(object.AsString())
endfor

echo set.FromStringer(set.ToStringer(abc.ABC.Get('a')))
echo set.ToStringer(abc.ABC.Get(tolower('A')))
echo set.Contains(abc.ABC.Get('Ω'))

The id() function behaves as advertised.

thanks all!


Code coverage skips files without tests

Seems like the code coverage is only reporting on files that actually have a corresponding spec file - I would expect this include all source files.

Really? That was an issue back in the angular 1. I need to investigate it, PRs welcome.

at the time i have made this commit https://github.com/preboot/angular-webpack/commit/78eb561c524f18515d7f0975c7a8f4da46c30a54
maybe can fix this too.

Hi,
I restart this topic because actually, the code coverage is only for .spec.ts files and not all files.
It's nice to have a very good percent but the result is worth !
I have try to modify karma.conf.js with this reporter with "includeAllSources" params but it doesn't work :
if (!isTestWatch) {
    _config.reporters.push("coverage");

    _config.coverageReporter = {
      dir: 'coverage/',
      includeAllSources: true,
      reporters: [{
        type: 'json',
        dir: 'coverage',
        subdir: 'json',
        file: 'coverage-final.json'
      }]
    };
  }

Thank's for your answer.
Nicolas

I have no idea how to do that.


Helm dashboard v2

The style might need changes when new items will be added into the project
I added all my components inside a different folder hope that's ok I also added models folder
and the hlp popup will need z-index changes

@YossiMorgen please tag the issue this solves

@YossiMorgen why did u change the backend code?


Authorize MIS using api-key only

As a MIS
I want to be able to access some endpoints (that doesn't belong to ehealth-user) having api-key only without authorization token
So that my code won't be dependent on human-factor and user availability (currently user should pass auth flow every two weeks) and I won't be forced to simulate user presence and store his credentials.
Kong plugin

[ ] add api-key-endpoint as analog of current url-template works with api-key only
[ ] change kong corresponding routes  to use api key verification or token verification
Mithril
[X] check if token exists in headers use token verification
[X] check api-key if no token exists in headers

test-cases

@AlexKovalevych please help  with kong

Before test, please ensure client has broker_scope legal_entity:write

It shouldn’t be dependent on the broker scopes. Client type scopes should be checked instead

@lymychp according to the code broker_scopes + scopes will be checked. Are you definitely sure to return empty broker scope?

https://github.com/edenlabllc/mithril.api/pull/388

Yes, in case if we do authorize using api-key, only client-type scopes should be checked

@svetavedmed please update test collection for the below endpoints api-key should be used instead of token
GET /api/events/id
GET /api/events
POST /api/legal_entities

on update legal_entity_v1 error
{
                "entry": "$.updated_by",
                "entry_type": "json_data_property",
                "rules": [
                    {
                        "description": "can't be blank",
                        "params": [],
                        "rule": "required"
                    }
                ]
            }

tested on dev


Doc Feedback: Inconsistent font emphasize

Location : https://apim.docs.wso2.com/en/latest/Learn/CICDPipeline/initializing-projects-using-the-cli-tool/

The content has be re-written, https://apim.docs.wso2.com/en/latest/Learn/APIController/getting-started-with-wso2-api-controller/


implement removeAttribute option and correcting soup_src identation at display.py

So, we can remove attributes from tags, principally 'disabled' attribute from button or input button tags in order to toggle their states.
I'm building a GUI where the user can only click certain buttons after a state is reached, such as a file upload is completed.
So, buttons start disabled and then, after the file upload suceeds, they get enabled in order to allow user to make their subsequent actions.

Also, soup_src = content inside if file block at Class Window's (Neutron/init.py) display method was misindented.
So, when users tried to display a window through html file from an exe bundle, soup_src wasn't properly assigned, and then, an UnboundLocalError is raised as it is:
File "Neutron_init_.py", line 128, in display
UnboundLocalError: local variable 'soup_src' referenced before assignment
so, bringing soup_src a tabulation (4 spaces) back corrects this issue


New and Updated Items Are Not Showing in the Grid

Hello.
I really love this plugin a lot, but I'm having trouble getting the grid to reload with added or updated items. It looks like the item exists because the row remains, but there aren't any values in the row.
During the edit:

And after I save, it does get updated on my back end, but the grid info disappears:

Any ideas? Have you seen this behavior before? It was working till I added the toastr plugin, but I removed it and now I'm getting these blank rows.
I'd appreciate any help.

Could you share your grid config?

Ah ... yes ... I did change that to return a success or error message. My whole config is pretty enormous so I'll just put the updateItem method:
updateItem: function(item) {
    var url = "{{ route('cards.update', ['id' => ':id']) }}";
    var url = url.replace(':id', item.id);

    return $.ajax({
        headers: {'X-CSRF-TOKEN': $('meta[name="token"]').attr('content')},
        type: 'POST',
        url: url,
        data: item,
        success: function(json) {
            if (json['type'] == 'success') {
                toastr.success(json['message'], json['title']);
            } else {
                toastr.error(json['message'], json['title']);
            }
        }
    });
},

How can I return a message from my controller without throwing off the grid? Can I return the item and the message in the same object and just utilize the message parts I need? Or is there a way to get the grid to ignore the returned message and just update as it does without it?
Thanks for your help.
-V

Ok nevermind man, I got it sorted. I just stored the item and returned it in the success method of the ajax command.
Thanks a lot.

Ok, I spoke too soon.
It works for updateItem but not for insertItem.
My insertItem code looks like so:
insertItem: function(item) {
    $item = item;

    $.ajax({
        headers: {'X-CSRF-TOKEN': $('meta[name="token"]').attr('content')},
        type: 'POST',
        url: "{{ route('cards.store') }}",
        data: item,
        success: function(json) {
            if (json['type'] == 'success') {
                toastr.success(json['message'], json['title']);

                return $item;

            } else {
                toastr.error(json['message'], json['title']);
            }
        }
    });
}

Thanks.

Have you tried to return the inserted item from insertItem?
You are talking about insertItem, does this mean that updateItem now works as expected?

No answer, closing for now.


[refactor]: Move to downloader

Instead of doing manual reqwest calls we may want to use an abstracted library like downloader, which give progress callbacks and handy utilities.

our CLI is decent enough as is, this should be considered way past 1.0 (maybe even 2.0 :p)


Discussion tab

Hi,
I have some questions about the repo (like: the docs are slim and not contains a microservice example / cons of the library),
but dont know where to ask them..
I don't think that issues are suitable. (but discussion tab do)
It looks like a very easy task:
https://docs.github.com/en/discussions/quickstart
Any chance to open a discussion section ?

I'm not opposed. @jezdez can you enable discussions tab for this repository? Thanks!

I think he really busy..
Would it will be ok to bump this in a month or two ?

Sure. I don't really mind questions being asked in issues since people can more easily search for answers and bugs simultaneously. Especially since this package is more strict when it comes to implementation, there are many solutions that appear in the issues search engine rather than a new discussion tab. Again, not opposed, but also idrc since we already have a questions tag :P

I'm a fresh graduate, that learn django alone and now I starting to learn about microservices and JWT is a term that I heard once or twice.
Did some research and found this library, but I still dont know how to use it.
And in the Docs you dont have an example like in channels (for example)
I read the Docs but its not clear to me how it used in a microservices..
So any chance to see some example ? (maybe one?)
Another question is what kind of attacks this library durable to and for which it doesnt ?

The doc's introduction is the example itself. Create a path to the SimpleJWT views and now you can retrieve tokens. The library assumes you understand what JWT is in the first place. Something that helps many people, in fact most people who come to this library, is by reading a tutorial. Many people who come to this library are using tutorials to teach them the ropes around JWT and stateless authentication.
For security, learn the difference between stateless and stateful sessions. Then, you'll understand what JWT is and how it is used in mobile authentication. It's also used in React apps, but never plain vanilla HTML. The reason for this is because most Django developers serve their React apps on a separate domain from their Django server, which means a session cookie cannot be properly set to HttpOnly. HttpOnly is a flag that marks that a cookie cannot be altered by JavaScript, preventing XSS attacks from stealing session ids.
No library is durable to any attack. This library is designed for one simple purpose: generate access and refresh tokens, similar to OAuth 2.0 tokens. There are features to assist in some secure sense such as blacklisting tokens, but security itself is inherent in the design of stateless authentication. In other words, you would want to add throttling to our views by subclassing our views and adding decorators such as from django-ratelimit. At your stage, I wouldn't worry about security. Remember, Facebook stored all passwords plaintext; not that you should, but it means you should be caring more about your product than the security of it to grow first. Security is only good if you have the knowledge to build with security first. Another instance of security not really being this library problem is whether you use http or https. Simple using HTTP means people can steal your client's session. This is for both cookies and web tokens passed through POST requests.

I'm going to close since I don't see a particular use case with discussion tabs for the size of this repo. Bump @jezdez if you can still open it; I don't mind, but it's not a top priority.


Automated Package Upgrade

:tada: This PR is included in version 2.2.224 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: