Don't ask for user input on non-interactive console

Current behavior:
Currently, we do not detect if the console is interactive. We simply try to request user input. If it's not supported on the low level then we return null.
Desired behavior:

The use of UserInputHandler should detect if the console supports interactive mode.
If the console is not interactive then the question is not rendered and no user input is requested.

Implementation:

Console interactive detection needs to work on Unix and Windows.
Console interactive detection needs to work daemon and no-daemon mode.
Console interactive detection likely needs to happen on the client-side and not the daemon side.
At this time we don't have any code in place that can detect interactive mode in console independent of OS.
For now we could probably use ConsoleDetector. If the ConsoleMetaData is not null and standard output is available then we can assume that standard input is available. We could potentially hook that into ConsoleConfigureAction (which already initialized the console) and pass this information along to UserInputHandler.

Test cases:

Existing test cases for interactive user input keep passing.
If the console is not interactive then the question is not rendered. No user input is requested. The method UserInputHandler.askYesNoQuestion returns null. This test cases works on Unix and Windows.

@oehme While implementing a spike I detected more requirements:

Needs to work with Gradle command line in no-daemon and daemon mode
Needs to work with Gradle core's integration testing tooling for in-process and forked mode (and can emulate interactive mode)
Needs to work with the Tooling API in in-process and daemon mode
Needs to work with GradleRunner in embedded and daemon mode

I put together some code that passes the tests and seems to work as expected. However, it feels a bit assumptious and probably needs the slightly different approach.

The interactive flag probably doesn't belong into StartParameter.
The tooling API assumes that the console is interactive if standard input was provided.

Could you please have a look at the code and let me know what you think before I even open a PR?

The interactive flag probably doesn't belong into StartParameter.

I don't have a better place for it either. The other logging options are handled there as well.

The tooling API assumes that the console is interactive if standard input was provided.

I think that's a relatively safe assumption.
The other assumption about the console being interactive if stdOut is available seems like a bigger issue. How does stdOut tell us that it is interactive? Don't all consoles provide stdOut (you need to write something after all)? I guess we are missing interactive detection in native platform?
I'm fine with having such a workaround for now if it works out in practice, but I think we should open a follow-up issue for 4.4 to detect this more reliably.
An implementation hint: Instead if passing the StartParameter into the DefaultUserInputHandler and adding an if-statement on every request it would be nice if we just created a NonInteractiveUserInputHandler and doing the if-statement once in the service registry.

The other assumption about the console being interactive if stdOut is available seems like a bigger issue. How does stdOut tell us that it is interactive? Don't all consoles provide stdOut (you need to write something after all)? I guess we are missing interactive detection in native platform?

I instead changed the implementation to check if System.console() returns null. I think that's probably a safe approach as long as you run that code on the client-side. I also tested the changes on a run of a build on Jenkins that asks for user input.

An implementation hint: Instead if passing the StartParameter into the DefaultUserInputHandler and adding an if-statement on every request it would be nice if we just created a NonInteractiveUserInputHandler and doing the if-statement once in the service registry.

Done in the PR.


[REG2.069.0-rc1] Issue 15251 - std.datetime bug with -inline

https://issues.dlang.org/show_bug.cgi?id=15251
The problem is in CTFE interpreter.
Inlining had introduced cast(void) expression to ForStatement.increment part,
then it had hit following interpret issues.

CastExp.interpret does not work for cast(void) if goal is not ctfeNeedNothing.
ForStatement.interpert does not evaluate its increment part with ctfeNeedNothing.

I fix both of them for the best work, although fixing one of them can fix the original issue,

Shouldn't attempting to interpret cast(void)... when a result is required be an internal error?

@yebblies AFAICS, ctfeNeedNothing is not properly specified in interpreter code.
So, I think it's risky in a regression fix to add an assertion something like assert(goal == ctfeNeedNothing) into CastExp.interpret for void casts, because it might introduce new ICE.

Inlining had introduced cast(void) expression to ForStatement.increment part

Inlining in the frontend is such a kludge.

Could you add an assert to master, at least in debug builds?


Can't see tooltip text when the series color is white.

The tooltip text color is fixed to be white, so it cannot be seen if the series color is white.  We need a setting specify the tool tip text color.

Wait, if your series is white, how can you see it ? Do you have a dark background ?
Could you provide an example of the problem please ? Thanks :-)

You can fix this by styling the text within your container with ! important. Example.

Closing this as no longer happening in v2.


pass full path to script being compiled

Hi,
this is my stab at #493. I took your advice and tracked the calls to this point in Interpreter.scala. But… I'm not quite sure if that is the correct place; or all of them. Also, I left the case for ImportHook.Source.Url unchanged, but maybe it is sensible to have a full url here, too? I thought an url as return value for File() looks strange. And to be honest,  I don't really understand what the url case means.
I tested with the result of amm/test:assembly.

Looks good to me; I don't really know what the Url case should do either, it was an experiment that hasn't really panned out. Fine to leave it unchanged for now.
Thanks!

This bug exists still in version 1.5.x and 1.7.1; did it regress? Perhaps there should be an integration test?

Yep, still looks like this is a problem in version 2.1.4 as well.


BUGFIX: Slim Styling

There was an issue where the first Master Category was overlapped by the header bar. Not a problem for sub-categories.

Closing the PR. I'm seeing different behavior in my browser than @dbaldon is -- I'm actually seeing the overlap when a Master Category is the first row and when a sub category is the first row.
Not really sure what the issue could be here. Hopefully something's not caching though, that's what it sounds like?


AzureRM 3.114.0  deprecated enable_https_traffic_only as  superseded by https_traffic_only_enabled

Community Note

Please vote on this issue by adding a :thumbsup: reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
https://github.com/claranet/terraform-azurerm-storage-account/blob/506e7cb6851fc92e681adcd9b2463750d6c0ac5f/r-storage-account.tf#L20
triggers the deprecation warning:
│ Warning: Argument is deprecated
│...
│   on .terraform/modules/stack_genesis.log_analytics.storage_logs/r-storage-account.tf line 20, in resource "azurerm_storage_account" "storage":
│   20:   enable_https_traffic_only        = var.nfsv3_enabled ? false : var.https_traffic_only_enabled
│
│ The property `enable_https_traffic_only` has been superseded by `https_traffic_only_enabled` and will be removed in v4.0 of the AzureRM Provider.

I'm opening this issue as reminder to update this module - I will try to submit PR after summer holidays period.
New or Affected Resource(s)/Data Source(s)
azurerm_storage_account
Potential Terraform Configuration
No response
References

https://github.com/hashicorp/terraform-provider-azurerm/blob/main/CHANGELOG.md#31140-august-01-2024
https://github.com/hashicorp/terraform-provider-azurerm/pull/26740

I have also ran into this warning and was going to open an issue here to update it.

Internal MR opened :), should be fixed/released this week

Thank you!


Improve Data Seeds

A general need to improve data seeding process has been identified.
The biggest problem is that patch version specific files are currently needed in order to run tests.  The goal is to use the REDCap native installation files instead.
We are working to improve this on this branch:
https://github.com/aldefouw/redcap_cypress/tree/seed-test

This is now completed and pushed to master and v9.1.3.


Provide more context for this new library and its relation with ypy in the documentation

For the sake of transparency and for folks landing on the documentation page directly, it could be useful to know more about the history behind pycrdt and how it compares to ypy.
Maybe summarizing the discussion from https://github.com/jupyter-server/team-compass/issues/55 (or linking to the issue) would be enough.

also missing from discussion in #55 is the "incubation" status of this project.


[scanner] add generic shard scanner

What changed?
Shard scanner is a generic shard scanner.
To use shard scanner you need to provide:

invariant manager - to set-up check and fix actions defined by scanner user.
iterator - shard scanner uses iterator interface to iterate over entities.
config resolver (optional) - scanner users can have their own custom dynamic configs. Config values are recorded for better determinism.

Why?
This shard scanner helps to reduce boilerplate when introducing new types of scanners, no need to reimplement/duplicate common logic.

How did you test it?
Unit tests.

Potential risks
None, this is noop change

Coverage increased (+0.001%) to 68.802% when pulling 8ec70d4d634ba408f7812a5b44a9dd956c220885 on mantas-sidlauskas:shard_scanner into 3eb3aa73894d4c8e15cdd1e0048fa09b46c903fb on uber:master.


Async dependant rules are not validated when calling ValidateAsync

void Main()
{
    var validator = new AsyncValidator();
    validator.ValidateAsync(0).Result.IsValid.Dump();
}

class AsyncValidator : AbstractValidator<int>
{
    public AsyncValidator()
    {
        RuleFor(model => model)
            .MustAsync(async (ie, ct) => {
                await Task.Delay(500);
                return true;
            })
            .DependentRules(dependentRules =>
            {
                dependentRules.RuleFor(m => m)
                    .MustAsync(async (ie, ct) => {
                        await Task.Delay(1000);
                        return false;
                    });
            });
    }
}

Validator result should be false but instead after 500ms it is true. Similar to #253

It looks like the previous patch that was submitted to resolve this doesn't actaully work - it isn't awaiting completion of the dependent rules before returning the results, so the list of failures is empty. I'll try and get this fixed today.

I've pushed out 7.1.1 with a fix for this. Please give it a try and let me know if it's working OK for you.

That was fast. Everything works now as expected.


Update rust-embed version

This fixes #270.

I would argue that we don't need a test for this, since we shouldn't test rust-embed's behavior. Given that this is a bit scary, I could be convinced otherwise though. @casey

I agree. Also, I think if we do #271, i.e. normalize and reject .. in the request handler, and write a test for that, we don't need to write a specific test for this vulnerability.

Also LGTM


Update data-protection.md

Update CMK doc to mention VNET scenarios wont work unless MSI+allow trusted services is used

@enkrumah : Thanks for your contribution! The author(s) have been notified to review your proposed change.


chore: Pin ECS module to sha while module is undergoing changes

Description

Pin ECS module to sha while module is undergoing changes

Motivation and Context

The current module source is pointed at a fork that is undergoing changes based on PR review feedback. This will pin the source to a stable version to avoid user disruption. Once the upstream PR is merged a separate PR will be opened to remove the fork source and instead use a versioned upstream source. The SHA used is from March 13th prior to latest changes

How Has This Been Tested?

[ ] I have tested and validated these changes using one or more of the provided examples/* projects
[ ] I have executed pre-commit run -a on my pull request

works locally with ref to SHA, squashing as is


too many values to unpack (expected 2)

Tried to run the usage example of xLSTM Block Stack from the readme, got the error: too many values to unpack (expected 2)

I have had the same problem, I think it has to do with some changes introduced in version 1.0.4. By installing version 1.0.3 (pip install xlstm==1.0.3) it works as expected.

I have had the same problem, I think it has to do with some changes introduced in version 1.0.4. By installing version 1.0.3 (pip install xlstm==1.0.3) it works as expected.

You can check issue #24. I modified the code to run through README sample. But further test is not done yet.


Max image size for inline cards is actually not 1MB

We have a chatbot platform, and one of our customers uses our chatbot platform to manage their Skype bot. The documentation says that cards have an image size limit of 1 MB. However, our customer tested it and was able to upload an image of size 1.2MB (which is greater than 1MB). The image still looked fine and worked properly in the Skype bot.
Therefore, the documentation is misleading. Please do update the documentation and remove the image size limit as it has been confusing our Skype bot users.

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

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards Reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

We are not able to repro it from our end, could you please provide more information.

Hi @VaraPrasad-MSFT ,
Sure. Here are the steps to reproduce the bug:

Please use any file/image hosting service to upload a GIF greater than 1MB. You may also use this example which is 1.5MB: https://i.stack.imgur.com/p6tME.gif

Attach this GIF to a card of type application/vnd.microsoft.card.hero with the image_url property. Make the bot send this card to a user.

You will notice that the GIF displays fine in MS Teams or Skype. Technically, it should not display fine as the documentation explicitly states that images within cards must be at most 1MB in size. This was misleading because we restricted our users to shrinks their GIFs to a much lower quality to meet this limit, and it affected the end user's experience.

@laujan : we are able to reproduce this and able upload the size of 1.5mb.


feat(cloudmon): modify  cloudmon cronjob

What this PR does / why we need it:
Does this PR need to be backport to the previous release branch?:
NONE
/cc @zexi
/area cloudmon

/hold


Patch cleanup

This was partly inspired by our code auditors calling out the same issue in multiple patches that change the same code.
Patches 0001, 0002, 0003, 0005, 0008, and 0011 are combined in to the new 0001 patch (this is maybe what you'd call the "core" of Envoy)
The Geneva patch is renamed to 0002
Patches 0010, 0012, and 0013 were combined in to the new 0003 patch (I'm pretty sure this is the stuff that lets us start cronet with a fixed SOCKS proxy)
Patch 0014, the shadowsocks binary, is renamed to 0004, but otherwise is unchanged for now while we're investigating if there's a working shadowsocks implementation we can use
Now that I think about it, I'm not entirely sure why I didn't combine the new 0001 and 0003 patches. It wouldn't be too hard to make that change if anyone thinks it's worth it. Maybe I'll revisit it when we know what we're doing with Shadowsocks. (I think I mentally drew a line "before" and "after" we were applying the Geneva patches, but they don't change the same code, so it shouldn't matter)

There's builds from this branch available here: https://github.com/stevenmcdonald/envoy/releases/tag/patch-cleanup_test1

There's builds from this branch available here: https://github.com/stevenmcdonald/envoy/releases/tag/patch-cleanup_test1

I tested these several times and had no issues so I'll approve the PR.


fix: truncate strings with mb_substr instead of substr

Pull Request
What changed?
When populating plugins, fields are truncated with mb_substr instead of substr
Why did it change?
Using substr cuts off UTF-8 byte sequences in the middle, which makes postgres very sad
Did you fix any specific issues?
none
CERTIFICATION
By opening this pull request, I do agree to abide by
the CODE OF CONDUCT and be bound by the terms
of the Contribution Guidelines in effect on the date and time
of my contribution as proven by the
revision information in GitHub. I also agree that any previous contributions shall be deemed subject to the terms of the
version in effect on the date and time of this pull request, or any future revisions for pull requests I may submit.
Further, I certify that this work is my own, is original, does not violate the intellectual property of any other person
or entity, and I am not violating any license agreements or contracts I have with any person or entity. Finally, I agree
that this code may be licensed under any license deemed appropraite by AspirePress, including but not
limited to open source, closed source, proprietary or custom licenses, and that such license terms neither violate my
rights or my copyright to this code.

@chuckadams does the system give a warning when mb_string extension is not installed in php? I have seen this dependency be missing on many php installs especially in hosting.

The ext-mbstring requirement is in composer.json, so the whole thing will fail to build if it's missing.  IMHO any PHP install missing mbstring these days is a HABS of PHP (Hacked Up Bastardized Simalucra)

My https://newpathconsulting.com/wap plugin has a specific check for this.


Glyph That Exists in Android Missing From Font

Hi there,
the codepoint U+1F973 (partying face) exists in android, which uses Noto Color Emoji for emojis. How come it's missing from the font itself then? I'm just downloading the font from the binary here.
https://emojipedia.org/partying-face/
Help?
Ron

Anyone has any information about this?


Refactor occlient package for extensibility

[kind/Enhancement]

Which functionality do you think we should update/improve?
The basic idea here is to have an interface package called odoclient which can have multiple implementation from which we have one already i.e. occlient package.
Why is this needed?
This functionality is needed to allow extension of occlient package for different platforms like kubernetes and knative

we have kclient package work already in progress, hence closing this issue


MINOR: Add system configuration to zk security exception messages

This patch ensures that relevant system configurations are included in exception messages when zk security validation fails.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@rajinisivaram @omkreddy Does this seem reasonable?


feature(argus): Release Planner

This PR adds a new way to manage releases inside Argus, called
Release Planner. Release Planner replaces current Release Scheduling
mechanism (but not the Duty Calendar for now). Main features include:

Plans are now separate entities (replacing one global release
schedule)
Each plan contains tests, groups and people it will be using for
assignments
Plans can target a specific scylla version to scope the plan to a
point or RC release.
Plans can be edited, deleted and copied.
On copy, plan will offer to replace missing tests/groups if it is
being copied to another release and that release is missing required
tests.
Each plan automatically creates and maintains a view dashboard for
said plan and can be viewed from plan overview page or inside views
overview
The new assignment logic is now applied to all releases not marked as
"endless" (perpetual in the admin panel). Currently the only one that
is marked that way is scylla-master.

Task: https://github.com/scylladb/argus/issues/377
Task: https://github.com/scylladb/qa-tasks/issues/1717

Demo:
https://github.com/user-attachments/assets/c9e47bd5-992a-4dab-ad94-407591ad12dd

@roydahan @fruch Ready for review

@roydahan new demo for the full functionality with explanations:
https://github.com/user-attachments/assets/632bcd89-4d6c-4bd4-8644-a019139ac79c

Looks good. There could be some improvements but I think we can start with it and improve it as we go.

One more thing - for a plan that has tests already selected (in edit or create exec plan mode), should opening Grid View clear that list? Currently it will merge the selection and I think that might be annoying to use.


[flash_ctrl] MuBi Changes to RTL flash_ctrl_host_ctrl_arb.sv Effecting ARB and RMA Tests

Hi @tjaychen
I have been debugging a regression fail isssue in the flash_ctrl_host_ctrl_arb test (which I believe I have solved).
However, when I rebased the code base to the latest, I now see different errors from this test.
After the RMA has completed, the ACK goes On, then one cycle later goes Off again, and the state_q changes from StHw->StSwIdle->StInvalid, and we get a fatal error.
(waves enclosed)
I think this is because the RTL has changed as below (MuBi)
If this is already documented, then I may have missed this.
At the moment I do not know whats is causing this ... but obviously will investigate.
Obviously something due to the changes below
If you can assist, that would be great
RTL File : flash_ctrl_host_ctrl_arb.sv
NEW CODE
  StSwIdle: begin
    // software is still selected to enable access to Fifos
    func_sel = SwSel;

    if (prim_mubi_pkg::mubi4_test_true_loose(disable_i)) begin
      // Do not randomly switch unless idle as it may cause stateful operations to be
      // disturbed
      state_d = StInvalid;
    end else if (hw_req_i) begin
      // if hardware request comes in the middle, wipe fifos and enable
      // switch to hardware interface
      fifo_clr_o = 1'b1;
      state_d = StHw;
    end else if (sw_req) begin
      state_d = StSwActive;
    end
  end

OLD CODE
  StSwIdle: begin
    // software is still selected to enable access to Fifos
    func_sel = SwSel;

    // if hardware request comes in the middle, wipe fifos and enable
    // switch to hardware interface
    if (hw_req_i) begin
      fifo_clr_o = 1'b1;
      state_d = StHw;
    end else if (sw_req) begin
      state_d = StSwActive;
    end
  end

ah yes let me explain. This is due to the recent security review of flash_ctrl, and we basically decided to make the "disable" indication in flash_ctrl more robust.
The basic idea is that after RMA transition, further flash operations are "disabled".  This disabled actually takes multiple forms and is described here.
I will update the documentation to make it more clear the fsm's are also impacted (as you observed), and also make it clear that after RMA entry it is explicitly disabled.
Hope this helps. If you let me know how you are creating this condition, I am happy to take a look to make sure it is working as intended.

see #11827

This issue will be closed whent the rma and host_arb tests are updated to use the new RTL.
This will be done after the remaining V2 tests have been written (within the next couple of weeks).
These tests will fail in the current Flash Ctrl regression because of these RTL changes.

@tjaychen FYI

upon looking into this more, I believe this is a design bug.
The way this behaves right now is that once we reach the end of rma, we are immediately transition to an invalid state due to the disable.
However this means that the rma entry ack will not be held long enough for the lc_ctrl to actually sample.  So even if we fix the block level test to conform to this behavior, it will break in chip level.
I am investigating a fix.


Added a container interop layer from a previous now missing dependency.

This allows for the Illuminate application to be compatible with our factories.

exactly what I was looking for. Thx @bweston92


Appreciation

Just would like to express praise and appreciation for the explanation of Configuring DbContextOptions - I saw earlier on that there seemed to be two ways to do this, but couldn't grok the specifics that are spelled out clearly and simply in a few lines of text. Great job!

Thanks @rhires! 😄

👍


fix(graphql): allow iam private rule to access update mutation when authenticated by cognito

Description of changes
Allow update mutation to respect cognito authentication for IAM private rule.
Issue #, if available
Fixes https://github.com/aws-amplify/amplify-cli/issues/9502
Description of how you validated changes

Manual test
yarn test

Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #9682 (a457d2c) into master (1239734) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #9682   +/-   ##
=======================================
  Coverage   52.90%   52.90%
=======================================
  Files         830      830
  Lines       45948    45948
  Branches     9812     9812
=======================================
  Hits        24308    24308
  Misses      19628    19628
  Partials     2012     2012

Impacted Files
Coverage Δ

...-auth-transformer/src/resolvers/mutation.update.ts
88.23% <100.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 1239734...a457d2c. Read the comment docs.


add contribution.html

add contributor html

check my PR @sudz123

@sudz123 added mufti.thml bro

Make sure the name of the PR is changed accordingly in the future.


Store SampleSets in Float64Array

I expect 3x-5x speedup on all SampleSet operations from this, without any WASM or optimizations.

wow! v exciting!

If we could use something in Stdlib, maybe we could get a bunch of efficient operations for free.

If we could use something in Stdlib, maybe we could get a bunch of efficient operations for free.

Maybe! The main goal here is to always store samples in Float64Array; otherwise we'll have to pay conversion costs, so this is all-or-nothing. I don't expect Stdlib to be especially fast on plain JS arrays, they are too dynamic for low-level optimizations.
Then we could optimize further by passing Float64Array to Stdlib functions, they support those on most or all array functions.

Dropped, didn't work out. Reasons explained in comments to #1223.


Add integration tests

Currently, we mock our exec calls. This makes our tests very fast, but doesn't give us any insights if updtr would actually work under real-world conditions. We should add 2-3 integration tests and run them on Travis against different versions of npm and yarn.

Has been fixed with the rewrite


Fixing Get-ADOPSElasticPool -PoolId

Overview/Summary
Bug fix for Get-ADOPSElasticPool -PoolId url interpolation.
This PR fixes/adds/changes/removes

Fixed PoolId interpolation on Get-ADOPSElasticPool

As part of this Pull Request I have

[X] Checked for duplicate Pull Requests
[ ] Associated it with relevant issues, for tracking and closure.
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.
[X] Verified build scripts work.
[X] Updated relevant and associated documentation.

LGTM!


Close stream gracefully

This bug is very annoying for the customers. You don't really see the streaming is open on another device and I have constant complains on streaming not working.

You should raise a screen message when streaming is busy. However, I see the problem on uv4l side since WebRTC seems not to detect a lost connection nor is there a timeout. And this is where the error mostly occur. Got the same experience on my side.

@Mictronics Agree

Got feedback from uv4l developer. He is pointing to https://www.linux-projects.org/webrtc-signalling/ especially the hangup message. Doesn't work however, because one can not open a new websocket to WebRTC server while still streaming. So now to send a hangup to an old connection. He rar on answer to my questions, though.

@Mictronics well, there are plenty of issues with UV4L. If You will get any useful info from the developer let me know. I hope we manage to run gstreamer + custom signalling server, so we will have full control over video streaming.

@stsdc After some back and forth I got message from the uv4l developer that this connection loss problem will be fixed in next release. Seems he finally understood what the real problem is/was. Let's see...


Filters should be applied in linear rgb space

For the most part this doesn't make that much of a difference. However for feColorMatrix the difference is very much noticeable. Fixing this might also make gaussian blurs more closely match the spec/batik.
Currently colourspaces aren't checked. In particular feColorMatrix always filters in sRGB.

feColorMatrix and feBlend now do this correctly. Other filters will need to follow.


For each Gemini model, restrict allowed values for candidate count to avoid runtime errors

Describe the bug
When using Gemini 1.5 Flash and 1.5 Pro, when trying to set a candidate count larger than 1, we are greeted with the following error:
Error for candidate count
Vertex response generation failed: ClientError: [VertexAI.ClientError]: got status: 400 Bad Request. {"error":{"code":400,"message":"Unable to submit request because candidateCount must be 1 but the entered value was 3. Update the candidateCount value and try again.","status":"INVALID_ARGUMENT","details":[{"@type":"type.googleapis.com/google.rpc.DebugInfo","detail":"[ORIGINAL ERROR] generic::invalid_argument: Unable to submit request because candidateCount must be 1 but the entered value was 3. Update the candidateCount value and try again. [google.rpc.error_details_ext] { message: \"Unable to submit request because candidateCount must be 1 but the entered value was 3. Update the candidateCount value and try again.\" }"}]}}
Error: Vertex response generation failed: ClientError: [VertexAI.ClientError]: got status: 400 Bad Request. {"error":{"code":400,"message":"Unable to submit request because candidateCount must be 1 but the entered value was 3. Update the candidateCount value and try again.","status":"INVALID_ARGUMENT","details":[{"@type":"type.googleapis.com/google.rpc.DebugInfo","detail":"[ORIGINAL ERROR] generic::invalid_argument: Unable to submit request because candidateCount must be 1 but the entered value was 3. Update the candidateCount value and try again. [google.rpc.error_details_ext] { message: \"Unable to submit request because candidateCount must be 1 but the entered value was 3. Update the candidateCount value and try again.\" }"}]}}
    at /Users/nohe/Documents/git/nohe/tripedia-genkit/node_modules/@genkit-ai/vertexai/lib/gemini.js:482:17
    at Generator.throw (<anonymous>)
    at rejected (/Users/nohe/Documents/git/nohe/tripedia-genkit/node_modules/@genkit-ai/vertexai/lib/gemini.js:50:29)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Lack of support for multiple candidates is not a Genkit issue, it's a limitation of the Gemini APIs for some models at this time.
Expected behavior
However, it would be a better developer experience if we had a way to define the allowed values for number of candidates in the model definition so developers don't hit the error at runtime.

No longer relevant


Have some question about generate folder

hi
I run successfully and generate some folders but I don't understand what is conv2d-conv2d_7

Hi @xiongfeiliu
conv2d_7 refers to the 7th 2d convolution layer. It'll be the same name as in the model you created in TensorFlow.

Hi @falaktheoptimist
I mean Alexnet only have 5 conv layers But my result generate another folders named conv2d,cond2d_1.....conv2d_7 I don't understand how they come

They could have been present from before you constructed the actual graph (This happens commonly in notebooks). Use tf.reset_default_graph() at beginning if you're using notebook.

Thank you I will try @falaktheoptimist


ML online endpoint invoke implement optional inputs

Description
request_file is optional in the method but mandatory in code. Additional paramerers input_data and kwargs are ignored in code even if they are present in the method signature. These changes include these optional inputs and actually makes all of them optional
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Thanks for your contribution @posneve!
Are you still hoping to have this reviewed? If so, please update the PR and move from "draft" to "ready for review", and respond to the CLA comment above. Otherwise if you no longer wish to pursue this, let us know and we can close :)


Support a quiet mode for dump-logs

The dump-logs script currently prompts the user for input.
We need a --silent switch that skips any user prompts so that we can execute it programmatically.

Fixed in https://github.com/mtlynch/ansible-role-tinypilot/pull/82


feat: Maintain a working queue to schedule bundling

A working queue of modules to bundle results in a progress meter of what is already done, but the
future benefit would be to parallelize the module bundling using multiple processes/threads.

Coverage decreased (-0.5%) to 98.246% when pulling 719f92a29ce5464746afa9ef32a2b1bedcee93d7 on use-working-queue into f666076c37d80c2e14a85aace2d67333f4f54af6 on master.

Coverage decreased (-0.6%) to 98.137% when pulling ce32e8c1090649066207479f73225273129f6d67 on use-working-queue into f666076c37d80c2e14a85aace2d67333f4f54af6 on master.

Coverage decreased (-1.2%) to 97.59% when pulling 47ba5838427ea2518c0e63cf3a08fc9e46f25443 on use-working-queue into f666076c37d80c2e14a85aace2d67333f4f54af6 on master.

Coverage increased (+0.04%) to 98.802% when pulling 320dc0fb28ddbb032599e22308ef211d86620525 on use-working-queue into e972c4d7af123b6ff862a3941a80f6b44a6398d4 on master.

Coverage increased (+0.04%) to 98.802% when pulling db7f5e20d6aa2241fa6b9fe1668b743c4dd58ac9 on use-working-queue into 5eac08ba11862751e7ce257d6290ba8dd0a62dd2 on master.

Coverage increased (+0.01%) to 98.802% when pulling 1c86fe0de65ebed1764d238a67535385a6554114 on use-working-queue into ed066f22309abe0a45bbc1fa42e208bdbf0eaf81 on master.

Coverage increased (+0.04%) to 98.824% when pulling 1562aca8ee3c3b437eafa1720b377b96238723c3 on use-working-queue into 0ba8cab52b0799b1dc7d1d7d9a1b57d09840f0f4 on master.


Remove unnecessary initialisation of axios

In App.tsx, axios is being imported and the supertokens interceptor is being added to it. But the axios instance is not actually used.. so you don't need to import it and don't need to add supertokens interceptor to it.
This applies to all files on the frontend.

Removed


Add metropolis theme

This adds an (admittedly barebones) theme inspired by the beamer theme at https://github.com/matze/mtheme, suitable for my own needs
Examples of missing features that could be desirable to add in the future:

Alternative reverse colors (light becomes dark and vice versa)
Toggle progress bar and page numbers
Create section pages
Colored boxes for examples/theorems etc (would need typst-slides support)

I'll probably leave that for someone with more time and motivation.
Note: I did not update the Makefile in the theme-gallery, as I'm running Windows and didn't want to start messing around with WSL.
(I only use git on my own private repositories, so it took me forever to figure out how to do a pull request. Hope I'm doing this correctly!)

I'd also love a metropolis theme.
To me it is the ideal slide theme.
Thanks for doing this! I hope it gets merged.


Document configuration sections & keys

It would be helpful if the configuration options were documented. E.g., I don't understand the difference between main.path and main.default_dir_name.
Also, seems like some of it isn't even configuration, but just stored state?

A documentation of config options has been added to the Wiki and state info is now stored separately from v0.6.2.


Create a simpler interface for executing JQ programs

The channel interface was "nice" for my use in jqrepl (where I'm running
with the same input again and again but a different program each time)
but it doesn't make it easy to work out what input produced any given
output.
To make that possible add an "immediate mode" that simple returns takes
the input and returns the values directly.

@arodland This might do what you need


GitHub Project boards for PRs, Issues to keep stale[bot] happy

Suggestion/idea: more active use of project management tools such as Github Project boards (as mentioned in Issue https://github.com/kubeflow/community/issues/345).
There are a bunch of stale issues and PRs across /website and elsewhere, and you may have noticed **the stale[bot] ** has been busy sending notifications recently.
GitHub Project boards can help see the big picture for managers like @jlewi and if folks can’t finish something they can unassign themselves from tasks and ask for help. They are kind of like Kanban boards for better project management.
For example, we already have a v1.1 board that’s quite active here: https://github.com/orgs/kubeflow/projects/36.
(This is related to v1.1 release too - including docs stuff - I think kubeflow/website#1984)
For more information on Project boards see: https://help.github.com/en/github/managing-your-work-on-github/about-project-boards

Project boards are made up of issues, pull requests, and notes that are categorized as cards in columns of your choosing. You can drag and drop or use keyboard shortcuts to reorder cards within a column, move cards from column to column, and change the order of columns.
Project board cards contain relevant metadata for issues and pull requests, like labels, assignees, the status, and who opened it. You can view and make lightweight edits to issues and pull requests within your project board by clicking on the issue or pull request's title.

Let’s discuss Project boards here and how/if they can assist with seeing and attending a bunch of moving parts at Kubeflow across various repos.

@8bitmp3 I think the problem we run into with project boards is permissions. See: kubeflow/community#314. I think write permission is needed in order to add issues to project boards.


Schema Registry - Add Schema type and move Content out of SchemaProperties

Rather than having SchemaContent in the SchemaProperties, we decided to introduce a new Schema type.

Fixed in https://github.com/Azure/azure-sdk-for-net/pull/23969


💪✨ Customers - Implement CSV Download

Have reached out to the SAW team to get some feedback on finishing this feature or removing it entirely.

@matt-goldman - are you still going to work on this?

Re-assigning to @christoment


Update to Angular 11

We should update the library to compile using<EMAIL_ADDRESS>This will update the library to compile using the newest versions of dependencies, tooling, and schema's.
The end user should see no difference since it just affects those trying to compile from source. The end result should still be a package that is compatible with projects based on older releases of angular.
Note: this update is being performed in multiple repositories simultaneously.

Closing this due to the release of Angular 12.


Add EndsWith variant of route filters

Hi, thanks for submitting this pull request. Could you perhaps explain the use case for this handler a little bit? I am asking because there is already a similar discussion in issue #7 and that might even make the use case for routeStartsWith redundant. Would love to better understand where this might come handy so we make the right decisions.

@dustinmoris Ah, I didn't see that discussion. It's probably far better to go that route, sorry about that!

No worries and thanks for the effort though! Feel free to join the other discussion if you have more ideas/suggestions :)


Websocket introspect support

Websocket protocol does not allow to add headers like Authorization. Can we support checking for access_token even in query params? Something on below lines:
function M.has_bearer_access_token()
  local header = ngx.req.get_headers()['Authorization']
  if header and header:find(" ") then
    local divider = header:find(' ')
    if string.lower(header:sub(0, divider-1)) == string.lower("Bearer") then
      return true
    end
  end
  local args = ngx.req.get_uri_args()
  for key, val in pairs(args) do
    if key == "access_token" then
      local token = formatAsBearerToken(val)
      ngx.req.set_header('Authorization', token) // "resty.openidc".introspect - also does not allow query param usage
      return true
    end
  end
  return false
end

Websocket doesn’t have support for native Authorisation, therefore this is
implemented by the Developer on top of the Websocket protocol. Most of
Websocket libraries add this layer already.
In my opinion this is concern of the micro itself and not the API Gateway,
which over Websocket acts purely as proxy.
On Sun, 4 Sep 2022 at 09:12, Rashi Jaiswal @.***> wrote:

Websocket protocol does not allow to add headers like Authorization. Can
we add support to checking for access_token even in query params? Something
on below lines:
function M.has_bearer_access_token()
local header = ngx.req.get_headers()['Authorization']
if header and header:find(" ") then
local divider = header:find(' ')
if string.lower(header:sub(0, divider-1)) == string.lower("Bearer") then
return true
end
end
local args = ngx.req.get_uri_args()
for key, val in pairs(args) do
if key == "access_token" then
local token = formatAsBearerToken(val)
ngx.req.set_header('Authorization', token)
return true
end
end
return false
end
—
Reply to this email directly, view it on GitHub
https://github.com/revomatico/kong-oidc/issues/18, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUVDT2UYA26C4YDTN4IB33V4RKYBANCNFSM6AAAAAAQEHFMMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Most of Websocket libraries add this layer already

You mean something like auth or extraHeaders of socket.io? If yes, then are you suggesting that socket server should instead be responsible for introspecting the token passed via these (and kong-oidc plugin should be bypassed essentially) ?

Yes. You shouldn’t be applying REST fundamentals with Websockets. They are
utterly different. Only thing they actually share is the HTTP as the
transport layer.
REST is about avoid sessions by leveraging authorisation in stateless
calls. With websocket you actually create a duplex session via the
websocket.
On Sun, 4 Sep 2022 at 11:59, Rashi Jaiswal @.***> wrote:

Most of Websocket libraries add this layer already
You mean something like auth
https://socket.io/docs/v3/client-initialization/#auth or extraHeaders
https://socket.io/docs/v3/client-initialization/#extraHeaders of
socket.io? If yes, then are you suggesting that socket server should
instead be responsible for introspecting the token passed via these (and
kong-oidc plugin should be bypassed essentially) ?
—
Reply to this email directly, view it on GitHub
https://github.com/revomatico/kong-oidc/issues/18#issuecomment-1236311762,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUVDT74BU656J5QH47YBNDV4R6J3ANCNFSM6AAAAAAQEHFMMA
.
You are receiving this because you commented.Message ID:
@.***>

Cool... sounds good


Unsynchronised CapsLock indicator.

When used xmodmap to remap capslock indicator gets confused.

This is doesn't occur when running Linux as a virtual machine inside Windows.


Errors in grizzly cloud

resolution_fixed   |    by<EMAIL_ADDRESS>

Rose module is totally borked (issues resolving variables)

Mounting scratch failed - no filesystem type

Errors loading LDAP accounts in puppet (ticket #22) - now the groups aren't found even in the second LDAP run

Issue migrated from trac:38 at 2024-01-31 17:11:38 +1100

<EMAIL_ADDRESS>commented

Web services on the instance can't be accessed

ibc599 commented

Has this all been fixed?

<EMAIL_ADDRESS>commented

NCI ticket regarding web access https://sf.anu.edu.au:8443/Ticket/Display.html?id=64610

<EMAIL_ADDRESS>commented

NCI should be working on the web & ldap issues, have raised them with Ben

<EMAIL_ADDRESS>changed owner from `` to bje900

<EMAIL_ADDRESS>changed status from new to assigned

<EMAIL_ADDRESS>removed milestone (was accessdev v1.0)

<EMAIL_ADDRESS>changed resolution from `` to fixed

<EMAIL_ADDRESS>commented

I'd say so

<EMAIL_ADDRESS>changed status from assigned to closed

<EMAIL_ADDRESS>commented

Milestone accessdev v1.0 deleted


Save network training state.

I have seen from the tests that it is possible to save the network, but is it poossible to save its weights efter it has been trained? I.e I would like to train the net and then store it to flash, so that when I boot up my embedded project the next time it is already trained. Is this possible?
Great project btw!

This is totally doable.  Note the store function of a variety of classes, which all accept streams to write to depending on what exactly you're looking to save.   Good luck with your project!

Thanks for replying Joshua! Unfortunately, I cannot get that to work. Let me just brief you on my project real quick so you know what I would like to achieve. I want to run Fido on an embedded system (that part is done, compile, etc), and that system shall drive an Open AI simulation (CartPole) on a PC via websockets. I am expecting the learning to take quite some time due to limited CPU power on the embedded system. That's why I would like to save and recall the network weights.
Now, I have planned to use the FidoControlSystem() class to run this, as it seems to be the most advanced algorithm, and works with experience replay. Am I right in this? This class does have a store() function, but how do I load the network?
Thanks,
Marcus

Two things:

Looking through the code, it seems like only certain classes currently support loading from streams in their constructor.  If all you wanted to store were the neural net weights, you could simply modify the constructor for FidoControlSystem to have it intake a stream of weights, pass that down the hierarchy until the network is initialized, then initialize the NeuralNet from that stream.  Unfortunately this behavior isn't already implemented in the class, but it should be relatively trivial to accomplish.  This also leaves out additional hyperparameters you'd want to save such as adadelta parameters, uncertainty, etc.  However, once again this should be simple functionality to add if you want it.
If you want to use FidoControlSystem with CartPole, I suggest you run it locally on your computer first to make sure this is even feasible.  Wirefitted q-learning isn't really designed for that sort of task, and I don't know if you'd achieve solid performance.   You might be better off using traditional q-learning with a neural network, which Fido also supports and will provably converge on cartpole.

I'm gonna close this for now.


nextMaintenance field Data Type

What were you trying to achieve?
I was trying to seralize data from the AirQo API
What are the expected results?
I expected all fields to have a constant data type or at most 2 data types
What are the received results?
The nextMaintenance field had about 3 data types i.e null, date and json object

What are the steps to reproduce the issue?
Query for devices on the production environment and compare the different values for nextMaintenance
In what environment did you encounter the issue?

Chrome Browser

Additional context
Any other information you would like to share?
#236

Hi @NoahNsimbe , just started an email thread about this issue. In the same email thread, I have identified devices whose next maintenance dates are missing. After investigations, we shoud be able to tell what caused the issue and make appropriate corrections accordingly.


Remove the deprecated numpy dtypes "int" and "float"

It looks like this has sorta kinda already been merged in but with numpy.int64 replacing numpy.int and numpy.float64 replacing numpy.float.  I think those are the correct sizes for native Python int's and float's so maybe this PR isn't needed?

@dentalfloss1 @league  Any opinions on this PR which would allow lower precision values to be stored in a "standard header"?  I'm currently leaning towards keeping the current behavior in master and not merging this one.

I agree.

Motion carries, closing.


[HOLD] Introducing file based Client Overrides

WithOverrides
WithOverrides allows you to create a set of files on the filesystem that will be watched, read, and mutate the outbound event.
File names will be used as the extension attribute name (only extensions are allowed, no first-class attributes name).
File contents will be used as the value of the extension attribute.
For example:
$ tree cmd/samples/http/overrides/extensions/
cmd/samples/http/overrides/extensions/
├── baz
└── foo

$ cat cmd/samples/http/overrides/extensions/baz
bar

$ cat cmd/samples/http/overrides/extensions/foo
42

baz and foo will be added to the CloudEvents extensions  with "bar" and "42", respectively.
See the overrides sample for a working example.

I want to revert the k8s thing and add symlink support. Pending...

What's the motivation? At first glance it seems like it's outside the remit of a CloudEvents library to  populate the events from external data sources. Danger of scope creep?

What's the motivation? At first glance it seems like it's outside the remit of a CloudEvents library to populate the events from external data sources. Danger of scope creep?

It is slight scope creep, but it is tied into the client section (and totally optional to use) and it is useful enough that I think it can be a general client extension for this lib. The motivation is static decoration is fairly common in client usage, this makes that easier to do for those clients that are using a file to decorate the outbound event.

Ready.

@alanconway
why BinaryV1 as V03()?
case BinaryV1, StructuredV1, BatchedV1:
ca := event.Context.AsV1()


Default markers show on iOS devices.

Summary
I have this issue where default markers are visible, but not always and not for all markers. It's kind of strange I can't really determine when/why it shows up.

Reproducible sample code
<Marker {...markerProps} >
      <Image
        source={sourceImage}
        style={{ width: 55, height: 40 }}
        resizeMode="stretch"
        resizeMethod="resize"
      />
    </Marker>

Steps to reproduce
"react": "18.2.0",
"react-native-map-clustering": "^3.4.2",
"react-native-maps": "^1.4.0",
"react-native": "0.70.6"
Pure react native project with expo modules.
Tested on iPhone 7 (15.1)
Expected result
Should show just my defined markers.
Actual result
Show the default red pins.
React Native Maps Version
1.4.0
What platforms are you seeing the problem on?
iOS (Apple Maps)
React Native Version
0.70.6
What version of Expo are you using?
Not using Expo
Device(s)
iPhone 7 (15.1)
Additional information
No response

Hello!
I've got the same issue and I've managed to make it reproducable.

Have two views that use the same MapView Fragment
Navigate to first view where markers show, markers are loaded as they should
Navigate to another map view where other markers are shown.
Navigate back to first map view - pins are shown.

Seems to work with "image" attribute as well.
            <Marker
              key={`k-${s.id}`}
              coordinate={s.coordinate}
              tracksViewChanges={false}
              image={s.marker.image}
            />

yeah, same things happens for me, ios pin icon is displayed when changing marker coordinates on state changes

I really don't want to switch to google maps on iOS, but I might have no other options. This seems like a major issue. I am great with native Android, but unfortunately my iOS coding skills are lacking - otherwise I might try to find and fix the issue.

Having the same issue here.
"react-native-maps": "^1.4.0"
"react-native": "0.68.1",

I really don't want to switch to google maps on iOS, but I might have no other options. This seems like a major issue. I am great with native Android, but unfortunately my iOS coding skills are lacking - otherwise I might try to find and fix the issue.

I found this https://github.com/react-native-maps/react-native-maps/issues/4214#issuecomment-1132214457 in another issue, it fixed it for me...
Just in case you still need a fix for that.

Duplicate of #4214

Is a patch really the only still the only solution for this?

Is a patch really still the only solution for this?

Very frustrating. My map renders properly at first but after clicking around it loses track of my custom pins and replaces them with unclickable default ones.


Satan v1 new style case tolerances

Printed out the case in PLA on my Ender3 Pro, and it's just a wee bit too tight.
I can shove the PCB in, but it's so tight the buttons don't fit without depressing the 3 switches.
Have you heard any other reports of this? I might try re-printing at ~100.5% in all 3 dimensions.

Printed at 100.7% and it fit well enough

@LSatan


Rename deprecated scoped-properties to settings.

The deprecation message says:
"Store package settings files in the settings/ directory instead of scoped-properties/"
More information: https://atom.io/docs/latest/upgrading/upgrading-your-package

+1 PLEASE merge this & repackage

Please merge!

@jroesch It would be great if this got merged in, as v1.0.0 will completely remove deprecated features. This PR also fixes #15, #17, #18, #20, #22, #24, #25, and #27, and closes #19, so those issues/PRs should be closed when this gets merged in.

+1 I hate those pesky deprecation messages.

:+1

:+1:

:+1: please

:+1:

Please version-bump this, so it'll trigger an update.

Fixed by v1.0.2


Put the package under the @nestjs/ scope

Hey there,
this is exactly what I need. Great job! But what about naming this package like @nestjs/router giving it a more integral role in the whole framework like @angular/router.

Hi @vinnichase
Actually it's not up to me, the Namespace @nest is owned by Kamil, and I have no problem moving it to nestjs repo too since this package is becoming popular more and more like Nest, so let's mention Kamil here
cc @kamilmysliwiec


Update v8 to 13.0

Thanks to @erikcorry for the help with this one!

Pushed the changes to the shared linkage patch here, hope that's ok. The updated patch is cleaner and should be easier to upstream.


Settings page implementation is needed

User needs to be able to adjust the following settings:
Email, Name, Last name, telegram handle, etc.
And also he needs to be able to log out on settings page.

Implemented in https://github.com/Flagro/PhriniFluentFrontend/pull/11


Error in density.default

Hi there,
Just trying this out using the KMC instructions - looks like a great program. I'm getting the following error when running smudgeplot.R:
running	Smudgeplot v0.1.3
######################
INPUT PROCESSING
######################
Error in density.default(.total_pair_cov[.subset], adjust = .adjust) :
need at least 2 points to select a bandwidth automatically
Calls: estimate_1n_coverage_1d_subsets -> get_1d_peaks -> density -> density.default
Execution halted
Any Idea why?

Hi @JHereward, thanks for trying out smudgeplots.
I would guess that the _coverages_2.tsv file is empty, but it's hard to say without any other info. Would you mind giving me some more info:

number of lines of the _coverages_2.tsv file
if the histogram looks sane? (if you would post the kmer histogram here, it would be even better)
L and R values you have chosen for the coverage thresholds
perhaps the commands you have used

If you don't want to share your data publicly, would you mind sharing them via email?

Hi there, thanks heaps for making it - looks really great!
The _coverages_2.tsv file seems OK, two columns of numbers like this:
261	493
275	354
309	377
211	263
200	444
270	399
248	347
I've attached the histograms and they don't look good, this is Miseq paired end 300 data from short read archive SRR2032117 - I guess maybe the lack of a good kmer peak is causing trouble? I tried to clean up the data and error correct it - but still get the same error message.
I couldn't even get genomescope to run on the .hist originally, but if you run jellyfish without the -C option it seems to work. I don't fully understand what the -C option of jellyfish does, but removing it did seem to help. I tried with KMC as well, but the results were the same.

Hmm, this is not very nice kmer spectra. Somehow you can not really see the genome. This kind of kmer spetra one see when

there is too much PCR during library prep
there is too much contamination
it's really old sequencing run (Illumina 10 years ago was quite messy)
Especially first two are very common problems for species with low input DNA (like nematodes).

The reason why this is a problem is that if the variance of converage is too high, we are not able to separate peaks that relate to copy number in the genome.
Anyway, none of this is actually answer why smudgeplot does not work. I am pulling the data now, I will check it tomorrow.
Thanks again for trying smudgeplots.

Hi, yes it was WGA from a single nematode egg, and 600 cycle Miseq, which never really worked well, maybe I can find a better dataset for this species.
Here are the tsv and kmer hist files, happy to send anything else along if it helps?
Reniform.zip

Yes, this kmer spectrum does not have clearly defined peaks. I took a look at the SRA run selector for Rotylenchulus reniformis and found the following datasets:
WGA:
SRR2032116
SRR2032115
SRR2027991
WGS:
SRR2029859
SRR2029858
SRR2029861
SRR2029860
SRR949277
SRR949271
SRR2027999
I was able to download the datasets and use KMC to create histograms combining the data from the datasets (see attached). Unfortunately, even pooling across all these samples, the resulting kmer spectra do not have clearly defined peaks, and GenomeScope is unable to converge. I'm not sure what can be done other than getting higher quality sequencing.
WGA.hist.txt
WGS.hist.txt

Hi @JHereward , Hi Rhyker :)
I think why you get the error. The naive script I wrote for guessing L is searching for the local minima and the local minima of your kmer spectra is more than 200x although I suspect that the genome coverage is ~20x. So if you chosen L using the script, it filtered out basically all the genomic kmers and the kernel smoothing indeed failed because of 0 points with ratio 0.48 - 0.5, which is used for the first draft of genomic coverage estimate. I have to add some safety break in there. Thank you for this.
Now about our species, when I used smaller L, I got super messy smudgeplot as well. This is what happens with PCR, I am sorry. The evidence is very very weak, but in the all the mess smudgeplot guesses it to triploid. Is that possible?

Maybe I could try to run it on the samples @tbenavi1 found.

That's strange. I can not reproduce the error. Could you share me somehow the kmer_pairs_coverages_2.tsv file?

Hi there, here it is:
Reniform.zip
I got 200 and 600 for the Lower and Upper estimates.
Triploid is interesting, we don't really have any idea about this one. We have been looking at this because people tend to use ribosomal genes for phylogenetics in this group - but there are at least two copies - so species get split across different clades. We were looking to design a new marker and wanted to try to understand the reason behind the duplications. Genomescope correctly identified the genome as 195mb, but indicated only 50mb unique sequence.
Perhaps there has been an ancient duplication of a large part of the genome?

Thanks a lot. I realized where the bug is.
L = 200, U = 600 means that the coverage ration of kmer pairs could be 200 / 600 at least. And one of the smudge identification techniques is subseting stripes of kmer ratio coverages for peaks around 1/2, 1/3, 1/4 and 1/5. The subset in 1/5 category was empty therefore the kernel smoothing function have thrown an exception.
Long story short. This problem will fixed in the next commit.
However, got to say that the genome coverage is definitely way way bellow 200x and even if you update your smudgeplot and rerun the analysis, you won't get a meaningful result. Genoscope guesses the genome coverage to 26x, smudgeplot with more sane filtering to 21x. I guess that the genome assembly is quite bad, no?
I gave a brief look at the two other sequencing project linked and by my rough calculations the coverages are approx 25x and 36x. That's bit better, but I am afraid not sufficient if wga was used (which I suppose since the peaks are blended). Both genomescope and smudgeplot are very sensitive to coverage and even more to coverage variance and therefore I am afraid that i must agree with Rhyker - you need better sequencing data for a proper genome characterization.

Resolved in 4abe43fe969b327726159391b2662797487474fe


[WIP] Enhancement/Enable user click interaction for guided tour

Checklist

[ ] I tested the changes and all related features on the test server https://artemistest.ase.in.tum.de.
[x] I documented my source code using the JavaDoc / JSDoc style.
[x] I added integration test cases for the server (Spring) related to the features
[ ] I added integration test cases for the client (Jest) related to the features
[ ] I added screenshots/screencast of my UI changes
[x] I translated all the newly inserted strings

Description
This enhancement allows users to interact with elements in the highlighted section while the guided tour is active. This user interaction is e.g. necessary when the user has to click certain buttons.
If enableUserInteraction is enabled for a tour-step then the Next-Button should be disabled and the user can only navigate to the next step.
This implementation is required for the guided-tour for the code-editor and will be extended there.
Steps for Testing
-- clicking disabled --

Log in to Artemis
Start tour
Click anywhere on the screen, it should not trigger any actions
-- clicking enabled --
tbd.

Screenshots

Created new PR #815


[bug]: if an output already spent error occurs, the client fails and cannot be turned on anymore

tested on 2 setups:

Polar v2.0.0; Docker v20.10.21; Compose v2.13.0 (Win 10); network setup: 1 Bitcoin Core v25.0; 2 LND v0.16.2-beta; 2 Taproot Assets v0.2.0-alpha
Ubuntu 20.04; lncli version 0.16.3-beta; tapcli version 0.2.2-alpha

If the client reaches the state Error evaluating state (SendStateBroadcast): rpc error: code = Unknown desc = transaction rejected: output already spent, the client turns off and cannot be turned on again. When trying to power on again, the following occurs:
2023-07-17 23:02:15.888 [INF] RPCS: Starting RPC Server
2023-07-17 23:02:15.888 [INF] RPCS: RPC server listening on <IP_ADDRESS>:10029
2023-07-17 23:02:15.888 [INF] CONF: Starting HTTPS REST proxy listener at <IP_ADDRESS>:8089
2023-07-17 23:02:15.889 [INF] RPCS: gRPC proxy started at <IP_ADDRESS>:8089
2023-07-17 23:02:15.889 [INF] SRVR: Taproot Asset Daemon fully active!
2023-07-17 23:02:15.889 [ERR] SRVR: Shutting down because error in main method: received critical error from subsystem: rpc error: code = Unknown desc = transaction rejected: output already spent
2023-07-17 23:02:15.889 [INF] RPCS: Stopping RPC Server
2023-07-17 23:02:15.889 [INF] SRVR: Shutdown complete

As part of polar, I got to this state by following the procedure reported in the error https://github.com/lightninglabs/taproot-assets/issues/405
On ubuntu, I managed to get to this state in another way, but I am currently unable to repeat it. However, within this error, the status is then the same, the tapd client cannot be turned on again with the same return

For 0.2.0 on Polar that is kind of expected, as we explicitly fixed this with 0.2.2.
Are you sure on Ubuntu you got into the situation with 0.2.2 and not before updating to that version?
If yes, it would be really useful to know how exactly you got into that situation.

@guggero I quickly 2x repeated the procedure on ubuntu, which I used to get to this state and the result is the same. Procedure (I have a newly initialized tapd client, before the beginning I deleted the .tapd folder and let the client recreate it -> tapd commands, ctrl+c after startup and then start with connection to lnd node):

Minted the asset: tapcli assets mint --type normal --name fantasycoin_test --supply 100 --meta_bytes "fantastic money"
Finalized tapcli assets mint f and waited for the transaction to be included in the block
Now generated 2 addresses with the same parameters for this asset with suply > 50%: tapcli addrs new --asset_id {asset_id} --amt 90
I sent to both of these addresses immediately after each other: tapcli assets send --addr {addr-1} tapcli assets send --addr {addr-2}
After sending the 2nd transaction, the client is either loopy, stuck or something similar - you need to turn off the process using ctrl+C. Regardless of whether or not I interrupt the sending process with ctrl+C, once the next block has been mined the client fails with the expected error and cannot be turned on anymore

Ah okay, thanks for the repro.
This is kind of expected, since when you send out the first transaction, the Bitcoin UTXO that contains the assets is being spent and you need to wait for it to confirm before you can send the second transaction.
So you either need to do a multi-send (the RPC supports sending to multiple addresses at the same time, not sure if we added that to the command line as well).
Basically we need to implement https://github.com/lightninglabs/taproot-assets/issues/261 to give you a proper error message on the second send attempt (something along the lines of "no confirmed asset outputs available, please wait for previous sends to confirm before sending again").
So things should work as expected if you do tapcli assets send ..., wait for a block (or mine one if you're on regtest), then do the second send.
I'm going to close this as a duplicate of https://github.com/lightninglabs/taproot-assets/issues/261 (and also related to https://github.com/lightninglabs/taproot-assets/issues/374) for now.


Odroid: diskLayout might use the wrong case in filter

Describe the bug
A user of my software does not get diskLayout to recognize his disk (https://github.com/MauriceNino/dashdot/issues/111).
I think this is the relevant part of the output (but you can look at the linked issue to get the full output):
{
  "alignment": 0,
  "disc-aln": 0,
  "dax": false,
  "disc-gran": 4194304,
  "disc-max":<PHONE_NUMBER>,
  "disc-zero": false,
  "fsavail": null,
  "fsroots": [null],
  "fssize": null,
  "fstype": null,
  "fsused": null,
  "fsuse%": null,
  "fsver": null,
  "group": "disk",
  "hctl": null,
  "hotplug": true,
  "kname": "mmcblk0",
  "label": null,
  "log-sec": 512,
  "maj:min": "179:0",
  "min-io": 512,
  "mode": "brw-rw----",
  "model": null,
  "name": "mmcblk0",
  "opt-io": 0,
  "owner": "root",
  "partflags": null,
  "partlabel": null,
  "parttype": null,
  "parttypename": null,
  "partuuid": null,
  "path": "/dev/mmcblk0",
  "phy-sec": 512,
  "pkname": null,
  "pttype": "dos",
  "ptuuid": "cbf654cd",
  "ra": 128,
  "rand": false,
  "rev": null,
  "rm": false,
  "ro": false,
  "rota": false,
  "rq-size": 64,
  "sched": "none",
  "serial": "0xaceb3869",
  "size":<PHONE_NUMBER>92,
  "start": null,
  "state": null,
  "subsystems": "block:mmc:mmc_host:platform",
  "mountpoint": null,
  "mountpoints": [null],
  "tran": null,
  "type": "disk",
  "uuid": null,
  "vendor": null,
  "wsame": 0,
  "wwn": null,
  "zoned": "none",
  "zone-sz": 0,
  "zone-wgran": null,
  "zone-app": 0,
  "zone-nr": 0,
  "zone-omax": 0,
  "zone-amax": 0
}

The problem is, that you check for fsType, but it seems to be fstype in the user's output: https://github.com/sebhildebrandt/systeminformation/blob/b8b37f6f899d8e5cd02238f9a030172259f341d4/lib/filesystem.js#L855
All other things in this output seem to be valid.
Environment (please complete the following information):

systeminformation package version: 5.11.16
OS: Odroid
Hardware Odroid N2+

@MauriceNino you are absolutely right. Provided a fix for the filtering. Hope this fixes the bug also on your side. Version 5.11.17 just released.

@sebhildebrandt I think this needs to be re-opened.
Seems like now some false positives go through as well. Another user of my software has created an issue (https://github.com/MauriceNino/dashdot/issues/114) which now detects a lot of drives, but most of them are not really hard drives as it seems.

I asked the user that created the issue and will notify you when I have any updates.

@MauriceNino ... I created a small fix that limits the false positives .... but still we would need the output. Thank you for your help on this one. Version 5.11.18 just released.

@MauriceNino based on the provided output on your side I now made a fix that should work. Version 5.11.19 just released.

Nice, but are you sure it is correct? The fsType seems to be in the wrong case again, or is this intentional?

@MauriceNino oh man ... my fault again ... fixed, version 5.11.20 just released.

Nice, thank you again!


Failed to interact with windows dialogue opened from browser

I need to automate scenario:

Open web site in browser (using ChromeDriver)
On a web site I need to upload a file, click to open file -> file explorer window is opened

I am trying to work with this windows pop-up using WinAppDriver in order to find and upload a file.

I run an automated scenario on a remote machine:

create interactive session:
$session = New-PSSession
Start WinAppDriver in interactive session:
Start-Process -FilePath PsExec -PassThru -ArgumentList "-accepteula -u <user> -p <pass> -i $session "C:\path\to\WinAppDriver.exe"
Run tests:
Invoke-Command -Session $session {cd C:\path\to\jar\with\tests java  -jar -tests.jar }

Expected result:
WinAppDriver can work with windows pop-up while running tests locally and on remote machine
Actual result:
WinAppDriver can work with windows pop-up locally but on executing tests on a remote machine I get issue on detecting windows Open file dialogue:
An element could not be located on the page using the given search parameters. (WARNING: The server did not provide any stacktrace information) Command duration or timeout: 0 milliseconds For documentation on this error, please visit: https://www.seleniumhq.org/exceptions/no_such_element.html Build info: version: 'unknown', revision: 'unknown', time: 'unknown' System info: host: 'xxx', ip: 'xx.xx.xx.xx', os.name: 'Windows 10', os.arch: 'amd64', os.version: '10.0', java.version: '11.0.11' Driver info: io.appium.java_client.windows.WindowsDriver Capabilities {app: Root, javascriptEnabled: true, platform: WINDOWS, platformName: Windows} Session ID: 617199B1-5E45-4EBF-8B50-36B89182BD50 *** Element info: {Using=name, value=Open}
Can you please suggest what can be done in order to be able to run such a scenario on a remote machine?

It will not work with the way you are doing? You need to create a root session and the switch to this opened dialog window. The logic has to be in tests.jar.
With selenium you and just upload the file without opening the window.

I am creating root session and looking for opened dialog window, it is part of tests.  It is working fine locally. But when I run tests on a remote machines - it fails to detect opened window as a part of root session.
It is an example of interactions that requires web and windows dialogues opened from web - we have more scenarios and we need to make it work.

@iuliia123 Please post code example of how do you search windows pop-up


Changing the connect string in the connection pool file

Here are some choices on how to specify the jdbc url for Oracle: https://www.razorsql.com/articles/oracle_jdbc_connect.html
ORDS prompts for everything separately, user, pwd, host, port, sid or servicename.
SQLcl uses:
<connect_identifier> can be in the form of Net Service Name
or Easy Connect.
 @[<net_service_name> | [//]Host[:Port]/<service_name>]

e.g. sql scott/tiger@localhost:1521:xe
Since I store user and pwd separately, I guess I could allow:
<server>:<port>:<sid>
and
<server>:<port>/<service_name>

Then I would construct this in the code to:
jdbc:oracle:thin:@<server>:<port>:<sid>

Should I also make sure that the existing config files don't break? When I find "jdbc:oracle:thin:@" I would just remove it. I could allow both or I could even modify the file on the fly.

I guess I will do it the following way:

if jdbc:... exists ... I will accept the full url as is
if no "jdbc:" is found, I will put it in front and just add the connect string to it.

This way:

the file is not changed
the old string is accepted and can be used
newer and shorter strings will work as well.

and when you turn on debug you will see the different parts that I concatenated.


About using npx to initialize a react-native app

It is recommended officially at the react-native document site.
And installed global react-native-cli package may cause unexpected issues when using react-native-cli with npx.
So I think that we should use npx to initialize react-native apps and stop requiring install 'react-native-cli' globally.

We'll consider this. Maybe we can install react-native-cli to dooboo-cli locally.

Thanks for the issue! This is fixed in 4.0.0


[Error] NotFoundError: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.

Page
https://registry.platformio.org/libraries/knolleary/PubSubClient/installation
Stack
at _ (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:480431)
    at R (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:483106)
    at Suspense
    at div
    at div
    at ee (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:186836)
    at div
    at main
    at ne (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:189728)
    at _ (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:480431)
    at O (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:482456)
    at P (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:640208)
    at _ (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:480431)
    at R (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:483106)
    at B (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:482519)
    at p (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:473400)
    at https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:590373
    at gl (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:437471)
    at gl (https://cdn.registry.platformio.org/static/js/main.3cf424ba.js:2:437471)

Please press Ctrl+R and refresh the page


Activate the linters for the entire code base

When we renamed the command to gt we lost almost the entire linter coverage.
@charlierudolph

Do we also need to list the lib folder?

The lib folder is somehow automatically included when we lint the cmd folder.

Then can we lint just the entry point (cmd/gt) as everything else will be automatically included?

Strangely enough, running the linters on cmd did make them run for a long time, but they didn't seem to lint the lib folder at all. So I have added a call to lint lib directly, and fixed the thousands of linter errors in the previous PR. At least we now have a working linter!


new file Creative Commons Licences

Uploading the Creative Commons licences for use in the DES catalogue.
Started with a licence file from http://registry.it.csiro.au/licence and reduced to just using the CC licences.
File has passed testing in SKOS testing tool and SKOS Explorer.
@KellyVance Is is possible to have this one done by 20 November 2020?

Looks seemingly OK

Let's make this a linked.data.gov.au vocabulary.
This vocab should be a copyright licence vocabulary that contains the types used in Australian works (as a start(.
If DES only wants to display the CC licences in their dropdown, then this can be done in the SPARQL query.

@DavidCrosswellGSQ I'll make the changes to switch to linked.data.gov.au. I'm not going to accept the copyright item, as they aren't really copyright and are meant to be about permitted and licenced use of resources. The list can deal with digital and non-digital. Dublin Core uses the term Licence Document would you be comfortable with that?

@DavidCrosswellGSQ Have moved to linked.data.gov.au and changed to licence-document vocabulary name (which matches Dublin Core for the range of a licence)
Did a bit of clean up to creator and added editoralnote to concept scheme.

@DavidCrosswellGSQ - changes made as per request.


Is there a python api to read perfetto line by line

I have created a robot to analysis the app and system issue automatically base on android systrace. So I converted the perfetto to systrace by traceconv firstly.
I want to konw, is there a python api to read the perfetto line by line as the systrace file rather than convert it firstly and then read line by line.

I strongly recommend you switch approaches to instead use the Python API of Perfetto. the Perfetto format is very much not designed to be parsed in a line-based way. For efficiency, it makes use of a lot of tricks like interning, delta timestamps etc. which makes it impossible to parse line-by-line.
Documentation on the Python API is available at https://perfetto.dev/docs/analysis/trace-processor-python. If you insist on trying to do this with your current approach, converting to systrace will be your best bet but it is not something we have any capability of supporting from our side.


Update index.md

Sorry, just adding some FQN to the classes for clarity and consistency. Hope that's OK! 👍

@ampedweb thank you 👍 !


bemenu: 0.6.15 -> 0.6.16

Description of changes
https://github.com/Cloudef/bemenu/releases/tag/0.6.16
Also adds mainProgram attribute and changes from rec to finalAttrs to allow proper overriding

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 252753 run on x86_64-linux 1

3 packages built:

bemenu
pinentry-bemenu
tessen


Uml insert from text not working

Default Uml insert from text is not working
Arrange -> Insert from Text -> Both Plant UML choices

Works as expected, please be more specific (NOTE: We use CORS headers and limit the use of this feature to draw.io subdomains.)

try insert this from text on the draw.io http://draw.io/ site. Its the default text...

Op 22 jan. 2017, om 18:01 heeft Gaudenz Alder<EMAIL_ADDRESS>het volgende geschreven:
Works as expected, please be more specific (NOTE: We use CORS headers and limit the use of this feature to draw.io subdomains.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jgraph/draw.io/issues/58#issuecomment-274343206, or mute the thread https://github.com/notifications/unsubscribe-auth/AGzoNaOYBlyTmF7YROh3BRXHza-EKJQDks5rU4tZgaJpZM4Lqbtl.

https://draw.io/ will redirect to https://www.draw.io/ where insert plantuml works.


"File" participants running minutes

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.6.0
OS Version: Windows 10 Pro

Steps to Reproduce:
1 Connect to a remote machine using the Kubernetes extension: attach visual studio code to a running pod. I am using the Kubernetes extension version v1.3.3
2. Delete a file in the file explorer or Copy a file using the file explorer
After the operation above, a dialog is shown with "running ... participant":

in the case of folder deletion, the dialog disappeared after a couple of minutes
in the case of file copy, it has never disappeared, even if the file was actually copied.

It would help us if you could identify which save participants are running. I suspect it would be participants contributed by the Kubernetes extension, in which case you should file an issue on that repo. To verify this, you could disable all extension except the Kubernetes one, and see if the issue reproduces.

The "participants" dialog appears also when working locally. In this latter case, it stays open for a shorter time.
I have never seen this dialog before upgrading VSC to v1.6

The save participants are contributed by extensions. You can try an extension bisect to see what extension is causing the long save delay.


Sentry integrations causes FrozenError in actionpack-5.2.0

Failure/Error: require File.expand_path('../config/environment', __dir__)

FrozenError:
  can't modify frozen Array
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/actionpack-5.2.0/lib/action_dispatch/middleware/stack.rb:76:in `insert'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/actionpack-5.2.0/lib/action_dispatch/middleware/stack.rb:76:in `insert'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/sentry-raven-2.7.4/lib/raven/integrations/rails.rb:10:in `block in <class:Rails>'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/railties-5.2.0/lib/rails/initializable.rb:32:in `instance_exec'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/railties-5.2.0/lib/rails/initializable.rb:32:in `run'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/railties-5.2.0/lib/rails/initializable.rb:61:in `block in run_initializers'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/2.5.0/tsort.rb:228:in `block in tsort_each'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/2.5.0/tsort.rb:350:in `block (2 levels) in each_strongly_connected_component'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/2.5.0/tsort.rb:431:in `each_strongly_connected_component_from'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/2.5.0/tsort.rb:349:in `block in each_strongly_connected_component'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/2.5.0/tsort.rb:347:in `each'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/2.5.0/tsort.rb:347:in `call'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/2.5.0/tsort.rb:347:in `each_strongly_connected_component'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/2.5.0/tsort.rb:226:in `tsort_each'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/2.5.0/tsort.rb:205:in `tsort_each'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/railties-5.2.0/lib/rails/initializable.rb:60:in `run_initializers'
# /home/vagrant/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/railties-5.2.0/lib/rails/application.rb:361:in `initialize!'
# ./config/environment.rb:5:in `<top (required)>'

ruby 2.5.1
rails 5.2.0

@marksiemers try to update your rails application completely. I've solved my problem by this gem: https://edgeguides.rubyonrails.org/upgrading_ruby_on_rails.html#bootsnap

Yeah, that seems to have done the trick. Thanks!


Bug fix/issue #9612

Port of fix for issue #9612 from 3.4
Fixes ViewNode executor to not stop on first not-found-in-collection document. As index is eventually consistent - this is expected to happen sometimes. Verified by GTEST test.
Added index reversals for insert/update/remove for RocksDBCollection. Verified by js tests (only server side as test requires failpoints)
Changed index order storage from vector to set to allow indexes reordering. Now indexes that need reversal stored (and executed) after all other indexes to reduce possibility of reversals. Verified by GTEST test.
http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/5796/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/5800/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/5803/


Partial graph

Hi,
I am drawing a line graph of price movements during the day. The thing is, that if I create a graph, for example from 9:00 AM until 11:00 AM, then the graph is only two hours wide. I really would want to draw the graph to be as wide as 9:00 AM to 5:00 PM and the line on the graph gets wider during as the day progresses.
Is there an enhancement that will allow me to draw a full scale with labels but only have data points for a portion of the Y values? I've attached a mockup - a picture speaks a thousand words :-)

Hi @michaelschwinges!
Yes this is actually a very similar case for which i created this library. E.g.

// Simple chart
let data = [(x: 0.0, y: 0), (x: 3, y: 2.5), (x: 4, y: 2), (x: 7, y: 3), (x: 9, y: 2.5)]
let series = ChartSeries(data: data)
series.area = true
chart.xLabels = [0, 3, 6, 9, 12, 15, 18, 21, 24]
chart.xLabelsFormatter = { String(Int(round($1))) + "h" }
series.color = ChartColors.greenColor()
chart.addSeries(series)

this will render the following:

I see now that the xLabels prop is missing from the docs...

Wow cool, thank you for the quick response and the example!


Define the configuraiton options for the DID module

To make the module generic so it can be included in the cosmos-sdk it is important that some key aspects are
configuratble:

did method name (currently cash)
supported verification relationships
enable/disable ephemeral DIDs

did method name configurable in genesis.
In an IBC word the DID method should be unique (not sure)
relationships maybe only in node configuration? it has the risks that the did cannot be transfer through nodes / chains

it has been decided not to implement this feature for the time being, it may be implemented in a future release


bump ScalaTest version

Not sure why this didn't get picked up by scala-steward.

I know now: it's not actually referenced anywhere. Will send a follow-up PR that removes the dependency.


Shuffle serialization

Few more bits pulled out by @pphaneuf

LGTM, once my minor mistake is corrected. :smiley:


Click on "Generate Maven Credentials"

This button is nowhere to be found as described in the docs.

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

ID: f03a84e2-22c7-6899-5ac1-c1ed79c05ba6
Version Independent ID: 405e0a0e-b12b-3fd3-7135-5deb022cd9c3
Content: Use Maven artifact feeds - Azure Pipelines
Content Source: docs/pipelines/artifacts/maven.md
Product: devops
Technology: devops-artifacts
GitHub Login: @chasewilson
Microsoft Alias: phwilson

As @soccerjoshj07 mentioned, we now recommend using the MavenAuthenticate task, which has the correct instructions. For the exact issue here, I think we can update the documentation like this:

Navigate to your Maven feed and click "Connect to Feed". Click on "Maven".
Edit your pom.xml file, and add the repository snippet to the  and  sections.
Add or edit your local settings.xml file and add the  snippet in the  section.
Generate a personal access token with Packaging read & write scopes and paste it into the  tag.

@nerijusk Thank you for your feedback. The updated article will go live shortly.


Apply same rule and results to multiple properties?

I need the same rule for multiple properties, and the same error message, same client-side metadata, etc.
I do it like this:
RuleFor(c => c.Mobile).Must((c, s) => OneContactProvided(c)).WithMessage("Provide at least one contact");
RuleFor(c => c.Telephone1).Must((c, s) => OneContactProvided(c)).WithMessage("Provide at least one contact");
RuleFor(c => c.Telephone2).Must((c, s) => OneContactProvided(c)).WithMessage("Provide at least one contact");
RuleFor(c => c.Fax).Must((c, s) => OneContactProvided(c)).WithMessage("Provide at least one contact");

private bool OneContactProvided(Customer c)
{
    var values = new string[] { c.Mobile, c.Telephone1, c.Telephone2, c.Fax };
    return values.Any(v => v != "");
}

It works. If one of the rules fail, they all fail, and in the UI, each field shows the same error message.
But is there a cleaner builtin way to do this - to specify a rule and error message for multiple properties, and for the same result to apply to all of them?

No that's the best way to do it.


Rewording RichTextLabel::remove_line documentation

As discussed in #34852, the currently used the line-th line might be confusing.
This moves the description of line argument to a separate sentence, and makes it clear that it's an index.

Thanks!


--skip-processes and config inconsistency

Bug Description
When setting multiple values via cli argument / env variable is inconsistent with config value
Argument --skip-processes proc_a;proc_b;proc_c and results in having proc_a, proc_b and proc_c being skipped
But config equivalent is problematic
{
  "skip_processes": "proc_a;proc_b;proc_c" // Dosn't work
}

{
  "skip_processes": ["proc_a", "proc_b", "proc_c"] // Works
}

Steps to Reproduce

Add "skip_processes": "ld;cc;rustc;cargo-watch" to mirrord config
mirrord -f <config file> exec cargo -- watch -x "run"

Backtrace
No response
Relevant Logs
No response
Your operating system and version
macOs 13.2 (22D49)
Local process
cargo-watch
Local process version
No response
Additional Info
No response

@DmitryDodzin Is this actually a problem? They're different formats. You'd also have differences between config.yaml and config.json for example.

@DmitryDodzin Is this actually a problem? They're different formats. You'd also have differences between config.yaml and config.json for example.

I fell for it while trying to debug something, I feel like when it's only one value it should follow the logic of splitting by ;

I'd remove the cli flag, have this only via config file.


nim: 1.2.6 -> 1.4.0

Motivation for this change
Nim 1.4.0 is out. This version was added at bb99a0d9a8f764996b0b719f842ea4af44591d17 but reverted at bce139db2b2709e2e4a4b34744c72d281b21df7b when the source tarball was replaced. The new tarball no longer bundles the nimble utility (missing from nim-unwrapped), so this will need to be package seperately and included with the nim package.
Things done

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

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

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

The hash in this commit seems to be wrong. Did the tarball change again?

@ehmry do you still have the source tarball with sha256 0pch14xkymzhrkmyai8i5fcjaxicgjgbzyk3vph6qzvkdfmj0cdy in your store? If so, can you diff the contents of this and the new one?
If they keep changing it we should switch to pulling the sources directly from a git checkout/tarball maybe? CC @dom96

@petabyteboy I do not. If this is how they are going to do releases then we should stay one version behind.

Hi! @ehmry @petabyteboy is there a chance this will get merged, or not until next version is out (which may be unspecified time in the future)? would there be higher chance if this expression was changed to build from github sources? I kinda miss Nim 1.4, and I wonder if there's a way I (or someone else) could help things advance...

Seems like they haven't switched out the release tarball since I last checked a month ago, so we could probably go ahead with nim. I pushed the new hash to this branch. Still need to package nimble seperately.

Can we leave nimble as is here, and extract it later?

Can we leave nimble as is here, and extract it later?

After this change nimble isn't packaged at all. This would be very confusing to me as a user when I'm used to having nimble installed as part of the nim package.

I think packaging nimble separately is momentarily confusing but inevitable.

Actually not a problem, I will package nimble seperately from the unwrapped compiler and combine it into the wrapper.


Enable Dependabot in sfdx projects

Is your feature request related to a problem? Please describe.
Yes, I can't use Dependabot to get notified about releases of salesforce packages.
What are you trying to do
Use Dependabot with sfdx projects to get nofied about salesforce package releases from public or private registries (salesforce app exchange?)
Use Case
there is a repository controlling releases to Orgs called org, and multiple other repositories for dependencies that this org uses. Maybe these repositories belong to third party developers that publish unlocked packages publicly in Github, like this one https://github.com/jamessimone/apex-rollup. A customer wants to be notified about apex-rollup releases  in its org repo without having to keep an eye on that repo.  Dependabot can be used to achieve that in Github.
org repo that controls releases to a given org. It is a way to centralize the source of thruth for releses. No other repositories can deploy to an Org but this one. With this strategy customers can know exactly how a given Org changed over time.
org
├── sfdx-project.json
├── src
└── ...

Customers own project that is a dependency of his org. This repo doesn't deploy to Org because this would create 2 sources of thruth for releases. It would be hard for a customer to know exactly when v1.1.0 was deployed to his Org from github. So, when package_a 1.1.0 is released, Dependabot opens a Pull-Request in the Org repo. Upon merging this PR, the release workflow is triggered and the package is deployed to this Org. Now the customer knows exactly when he installed v1.1.0 in his Org by just looking at when Dependabot's PR was merged to main.
package_a
└── sfdx-project.json

3rd party dependency that is used by org's repo. When a new package is released, Dependabot will also open a PR in the Org's repo..
package_b //third party project
└── sfdx-project.json

Customer org/sfdx-project.json showing all dependencies for his org
{
    "packageDirectories": [
          {
               "name": "randomPackage",
               "versionNumber": "1.0.0",
               "dependencies": [
                    {
                         "package": "package_a",
                         "versioNumber": 0.1.0
                    },
                    {
                         "package": "package_b",
                        "versionNumber": 1.0.0
                    }
              ]
          }
     ]
    ...
}

Describe the solution you'd like
Check with dependabot/github the possibility of creating custom connectors that could be integrated with Github
https://github.com/dependabot/dependabot-core/issues/6762
Describe alternatives you've considered
publish dummy/empty packages to one of the supported package managers.
Additional context
N/A

This would be hard to achieve because Salesforce Packages do not have a strict common Versioning System. Maybe it could work for those packages that use semver


Add Metrics Service based on Micrometer.

Prometheus Endpoint is exposed based on Micrometer Prometheus.
Add IoTDBRegistry which stores all Metrics as IoTDB timeseries under root._metrics
Added Monitoring at several points in the programm
Added IService Implementation for new Service

This is not final but rather a Work-In-Process where I would like to get Feedback regarding the Architecture.

some suggestions:

use latency rather than duration;

use InsertPlan to write data rather than use SQL;

I think it is better to keep the server module as lightweight as possible;

add a switch to enable/disable collect metrics (please read the 4th suggestion. If the performance impaction can be ignored, we do not need this switch);
add a switch to enable/disable writing metrics into IoTDB
move the HTTP service and Prometheus into a new module, e.g., Prometheus-registry. Then the dependency of micrometer-registry-prometheus can be moved out of the server module.
(1) To implement it, you may have to extract some implement in MicrometerServerService.java as an interface, and just retain the IoTDBRegistry as the default implementation;
(2) In prometheus-registry module, you can add the dependency and reimplement the interface.
(3) suppose we have a new module prometheus-registry, do not let the server module depends on it. If so, we may need using the Java reflect strategy to load the new implementation in (2).

better to attach a report that the performance impaction after adding this feature

So what problem have this PR now?

Hi Julian, please see https://cwiki.apache.org/confluence/display/IOTDB/Monitor+Module
and
https://github.com/apache/iotdb/pull/4194

The new metrics framework has been accepted by the community, and the implementation of this PR has been referenced.
Thanks, Julian! @JulianFeinauer
I will close this PR.

Thanks for the feedback @SteveYurongSu and congrats to the effort!