Disabling ConfigureConsumeTopology still requires user that can change topology

Contact Details
No response
Version
8.x
On which operating system(s) are you experiencing the issue?
Windows, Linux
Using which broker(s) did you encounter the issue?
RabbitMQ
What are the steps required to reproduce the issue?
We want to integrate with a RabbitMQ queue which is provided by an external partner.
We have been granted credentials which do not allow for setting topology, only for receiving messages.
Connecting to this queue using the RabbitMQ library itself works but we want to use MassTransit as it is being used in other systems.

We use the following code to setup MassTransit using RabbitMQ

services.AddMassTransit(c =>
{
    c.UsingRabbitMq((context, cfg) =>
    {
        cfg.Host(new Uri("rabbitmq://external.partner.queue:5671/vhost"),  hostCfg =>
        {
            hostCfg.UseSsl(ssl => ssl.Protocol = SslProtocols.Tls12);
            hostCfg.Username("...USERNAME...");
            hostCfg.Password("...PASSWORD...");
        });

        cfg.ReceiveEndpoint("integration-queue", endpointCfg =>
        {
            endpointCfg.Consumer<MyConsumer>();
            endpointCfg.ConfigureConsumeTopology = false;
        });
    });
});

the following exception is thrown :
RabbitMQ.Client.Exceptions.OperationInterruptedException: The AMQP operation was interrupted: AMQP close-reason, initiated by Peer, code=403, text='ACCESS_REFUSED - access to queue 'integration-queue' in vhost 'vhost' refused for user '...USERNAME...'', classId=50, methodId=10

### What is the expected behavior?

Bus should start and not throw an error.

### What actually happened?

A RabbitMQ.Client.Exceptions.OperationInterruptedException is thrown.

### Related log output, including any exceptions

```shell
RabbitMQ.Client.Exceptions.OperationInterruptedException: The AMQP operation was interrupted: AMQP close-reason, initiated by Peer, code=403, text='ACCESS_REFUSED - access to queue 'integration-queue' in vhost 'vhost' refused for user '...USERNAME...'', classId=50, methodId=10
   at RabbitMQ.Client.Impl.SimpleBlockingRpcContinuation.GetReply(TimeSpan timeout)
   at RabbitMQ.Client.Impl.ModelBase.QueueDeclare(String queue, Boolean passive, Boolean durable, Boolean exclusive, Boolean autoDelete, IDictionary`2 arguments)
   at RabbitMQ.Client.Impl.ModelBase.QueueDeclare(String queue, Boolean durable, Boolean exclusive, Boolean autoDelete, IDictionary`2 arguments)
   at MassTransit.RabbitMqTransport.RabbitMqModelContext.<>c__DisplayClass19_0.<MassTransit.RabbitMqTransport.ModelContext.QueueDeclare>b__0() in /_/src/Transports/MassTransit.RabbitMqTransport/RabbitMqTransport/RabbitMqModelContext.cs:line 115
   at MassTransit.Util.ChannelExecutor.SynchronousFuture`1.Run() in /_/src/MassTransit/Util/ChannelExecutor.cs:line 283
--- End of stack trace from previous location ---
   at MassTransit.Util.ChannelExecutor.Run[T](Func`1 method, CancellationToken cancellationToken)
   at MassTransit.RabbitMqTransport.RabbitMqModelContext.RunRpc[T](Func`1 callback, CancellationToken cancellationToken)
   at MassTransit.RabbitMqTransport.Middleware.ConfigureRabbitMqTopologyFilter`1.Declare(ModelContext context, Queue queue) in /_/src/Transports/MassTransit.RabbitMqTransport/RabbitMqTransport/Middleware/ConfigureRabbitMqTopologyFilter.cs:line 67

Link to repository that demonstrates/reproduces the issue
No response

MassTransit requires full access to the broker. Changing that property doesn't change the requirement.


Thank you!

Not an issue, but rather a great thank you for creating this add-on!
Kind regards!

Thank you! Great to hear it is useful to someone.


add entrypoint from image where needed

if the image specifies both the image and entrypoint, we need to account for that and preprend the entrypoint to the command.  this only happens if no user command and entrypoint were supplied.
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

LGTM

/lgtm


Better semantic clarity about the categories of assertion that failed

I've not carried changes all the way through the multi-file templates. I suggest we merge this anyway because all the template stuff needs to be re-organized next to produce the actual reports we want to produce on a regular basis, and I want to do that only once :-)

What's the best way for me to test the changes? Do the single file templates work with these changes?

Yes, the single file templates as invoked from the Makefiles are supposed to work. I suggest we merge even if there are some rough edges and file bugs, whether they get fixed in this PR or another doesn't make much difference right now IMHO.


SSO via OAuth sample in C#

Do we have sso ouath sample in C#.
Thanks..

Feature Request
Use the 'Enhancement' query to search for the most popular feature requests.
The Sample tag also lists examples and implementations that have already been requested.
Is your feature request related to a problem? Please describe.

Describe the suggestion or request in detail

Describe alternatives you have considered

Additional context

[Enhancement]

The C# version of SSO is handled through the parent bot/skill bot pattnern:
https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-authentication-sso?view=azure-bot-service-4.0&tabs=csharp%2Ceml
The sample showing this is here:
https://github.com/microsoft/BotBuilder-Samples/tree/master/experimental/sso-with-skills


[v1.x] MyActionItems

[ ] A new Tab is introduced on the 4Minitz main page: "ActionItems".
The old content goes to Tab "Meetings"
[ ] I See Action Items of all Meetings where I am invited & where I am responsible for the AI. (Compare Item-List on each Meeting Series)
[ ] Sort order should be "due date"
[ ] I can filter for open and closed AIs - default is: show only open AIs
[ ] AI display is "read only" (e.g. it's not possible to check AI as done).
[ ] Optional: bring in the filter possibilities of the "Items" tab a meeting series

Hints:

recycle as much of the existing display logic of the existing "Item" tab of a specific meeting series. Do not re-invent the wheel.
The content of action items list does not need to be "reactive". Releoad to refresh is O.K.

If the issue #281 is already implemented the use should be able to switch between: sort by "due date" and sort by priority.

Team estimation: 4d

Done


Feature Request: Switch pre-fader on the mixer

Hi, it would be useful to have a switch on the red outputs to set them on pre-fader mode in order to chain effects that hasn't dry/wet control, what do you think?

sorry I didn't notice that the purples in are inputs cv for mute and solo hahah however the idea it's always the same if u like it

Yeah I had the same idea.  Amazing the simple things you come up with when you actually start trying to use this stuff...  Or I may sacrifice the mute/solo inputs for separate pre-fader outputs, as they aren't as useful I think.

probably they're not very useful but also not completely useless, i suppose it depends on the situation...
however personally i prefer a send and return than a cv for mute and solo

Trying to incorporate 4 aux channel faders per group (A, B etc.) each with a pre fader select button, then there will be four aux outs along the bottom.  Might only be two due to space restrictions, but it might work

Finished.  Give it a try and tell me what you think.

I'm home now. Good... let me see it.

beware, all the level faders can now amplify to 4x so it can really get loud.

I noticed. I like the idea to use 4 aux channels however prefaders doesn't work I don't know why and also the inputs of levels and pans of all blocks, they are cv controls or audio inputs?
by the way prefaders on the aux channels are useless cause
before in the first example, supposed i don't have an effect with dry/wet knob, i need the switches for prefader to block the signal A towards B in order to have:
A sends to Effects in prefader, the knob of A towards B it's closed (dry signal), then effected signal returns to B
in this case a switch for prefader on the aux won't be useful cause i can close the A to B knob while i send the signal on the aux with no problem.
In any case good work, you can remove the switches and fix the inputs (audio or cv) near master level and master pan of blocks.

Levels and pan are CV.  The pre fader works for me, I suppose the one use would be to get the output to the aux channel independent of mute or level.  I am currently making 1x4 and 2x4 mixers, I will look into these issues when I test them.
Try also to initialize the mixer.  Since I added new params they may have conflicted with your old mixer settings.
Thanks for trying this out.

I didn't understand pre-faders referred to level inputs, ok now i figured it out. I tried again the cv inputs the pans works but levels cv not.

whar are cv levels? 0 - 10?

I tried from -10 to +10 but nothing happens to be sure i also tried from -100 to 100.

ok, thanks

Fixed a bunch of stuff, should work better now.

mixer 44 seems to work perfectly
mixer 14 and 2*4 have an issue on channel level knobs cause cause i can hear just the aux outs with prefader switches on.


SPR_isAnimationDone(..) do not correctly report animation state if called from the frame change callback

This is due to the fact that sprite->timer is set to 0 prior the callback call.
Initially timer was set to the frame timer value but then isAnimationDone couldn't correctly report animation state.
Then we changed logic so timer is keep to 0 and only reset to frame timer after the callback call and only if timer was not customized (!= 0) inside the callback.
We need to find a way to correctly report the animation done state both from the main loop :
(frameInd == last) && (!loop || (timer == 1);
And from the callback which is called at frame change period, at this point we have 2 possibles cases where frameInd == 0 :

animation is looping :
(frameInd == 0) && (status & FLAG_LOOPED) ?    --> need to return TRUE
changing animation - SPR_setAnimation(x) :
(frameInd == 0) && (status & FLAG_LOOPED) ?    --> need to return FALSE

Need to find a way to distinguish both cases (if animation does not loop then frame change callback won't be called at the end of animation, that is something to take attention for the developer). Maybe set a bit in status on SPR_setNextFrame(..) when looping, which is cleared as soon we clear NEED_FRAME_UPDATE flag (updateFrame)
Note that sprite->frame is used to distinguish between internally set frame (frameInd) and current displayed frame (*frame) as the display update can be delayed if DMA is overloaded, it's why we can have a discrepancy between the 2 fields.

Fixed.


seek by decimal keys

You will be able to seek videos at MS Streams or TED by pressing the numerical keys: 0-9 with this feature.
I tried to work on Amazon Video, but it did not achieve.

@mkobayashime I do not subscribe to DAZN, so could you add snippets below to dazn.js and check it if you could?
case "0":
case "1":
case "2":
case "3":
case "4":
case "5":
case "6":
case "7":
case "8":
case "9": {
  if (this.config[`keys-decimal`]) {
    decimalSeek({
      media: this.media,
      numericKey: e.key,
    })
    this.callIndicatorCreator({
      type: "text",
      text: e.key,
    })
  }
  break
}

@yudukikun5120
It's not working in DAZN, and I couldn't find any solution for that.
As for Prime Video I changed a few lines to make this work #100.

@mkobayashime I sincerely appreciate the rich change suggestions.

@mkobayashime I implemented the codes for Prime Video and asked you for a code review.

@yudukikun5120
Could you squash misc commits (especially ones simply applying suggestions by me, originally changed in f6d366340dae34d98710498d672d0c8285fb63ee) before I merge them?

@mkobayashime It would presumably be better to use squash and merge because it marks the index of PR (e.g., fix the hoge (#PRIndex)) on the squashed commit message. Could you try it?

@yudukikun5120
I suggeted the squashing because I thought you were intending to merge these changes with multiple (not all-in-one) commits per intension, but I can do the Squash and merge if it's OK for you.

@mkobayashime I appreciate your thoughtfulness. I'm OK with making it in a single commit.


c8y_log_plugin connection closed before message completed

Describe the bug
Requesting a log file from cumulocity leads to an error: connection closed before message completed. The log file request fails. The log output of the c8y_log_plugin includes following lines:
2022-11-07T06:41:27.43706957Z  INFO c8y_log_plugin: Log request received: 522,axcf3152-test1,mosquitto,2022-11-06T07:41:15+0100,2022-11-07T07:41:15+0100,,1000
2022-11-07T06:41:37.831719078Z ERROR c8y_log_plugin: Handling of operation: '522,axcf3152-test1,mosquitto,2022-11-06T07:41:15+0100,2022-11-07T07:41:15+0100,,1000' failed with error sending request for url (https://certification-phoenixcontact.eu-latest.cumulocity.com/event/events/34323/binaries): connection closed before message completed

However if I only request 1 line of the log file the request does succeed.
To Reproduce
Request a log file from cumulocity.
Expected behavior:
The log life request should complete successful.
Environment:

OS: Custom yocto build, glibc 2.35, gcc 11.2
Phoenix Contact AXC F 3152, Custom embedded hardware x86_64 Intel(R) Atom(TM) Processor E3930
System-Architecture: Linux axcf3152 5.4.161-rt66-pxc # 1 SMP PREEMPT_RT Tue Jul 19 13:05:03 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
thin-edge.io version 0.8.0

Here is the corresponding configuration file c8y-log-plugin.toml:
files = [
    { path = '/var/log/tedge/agent/software-*', type = 'software-management' },
    { path = '/opt/plcnext/logs/Output.log', type = 'plcnext-output' },
    { path = '/opt/plcnext/logs/plcnextapp.log', type = 'plcnext-plcnextapp' },
    { path = '/var/log/syslog', type = 'syslog' },
    { path = '/var/log/mosquitto/mosquitto.log', type = 'mosquitto' }
]

@bjoernsauer thanks for the details. Just a few more questions to clarify a few points so we can effectively investigate it:

Do you know roughly are large the log file is that you are trying to transfer?
Is the device network connection stable?

The log file mosquitto.log has approximately 32 Kb.
The device is connected over LAN cable.

Thanks for extra details. We will try to reproduce it, otherwise we will reach out for more details.

I have tackled down the issue. We have a restricted MTU size in our internal test network. I had to reduce the MTU size of the network interface on the device, now it's working.
So the issue was a miss configured network interface on my side. You may close the issue, thank you for your patience.

It seems that we can close this issue without a problem. If you face to the same issue, feel free to reopen it.


[UWP] Renderer solution no longer compiles in VS

Platform

UWP

Version of SDK
master
Issue
Compiling the .sln gives errors in teh mdmerge custom build step in teh UWP renderer:

>XamlBuilder.cpp
2>   Creating library C:\AdaptiveCards\source\uwp\x64\Debug\AdaptiveCardRenderer\AdaptiveCards.Rendering.Uwp.lib and object C:\AdaptiveCards\source\uwp\x64\Debug\AdaptiveCardRenderer\AdaptiveCards.Rendering.Uwp.exp
2>AdaptiveCardRenderer.vcxproj -> C:\AdaptiveCards\source\uwp\x64\Debug\AdaptiveCardRenderer\AdaptiveCards.Rendering.Uwp.dll
2>Performing Custom Build Step
2>MDMERGE : error MDM2018: The type "AdaptiveCards.Rendering.Uwp.TileControl" defined in file C:\AdaptiveCards\source\uwp\x64\Debug\AdaptiveCardRenderer\AdaptiveCards.Rendering.Uwp.winmd was referenced but could not be found.
2>MDMERGE : error MDM2018: The type "AdaptiveCards.Rendering.Uwp.WholeItemsPanel" defined in file C:\AdaptiveCards\source\uwp\x64\Debug\AdaptiveCardRenderer\AdaptiveCards.Rendering.Uwp.winmd was referenced but could not be found.
2>Microsoft(R) Metadata Merge Utility Version 10.0.47.
2>

@paulcam206, could you help get to the bottom of this since it's blocking alberto's PR's as well? Thanks!

@paulcam206, could you help get to the bottom of this since it's blocking alberto's PR's as well? Thanks!

Sure thing! :)


[BUG REPORT] eth-archival-trace endpoints not supported despite revenue generating.

Will be fixed on next rollout of the API code


Incorrect visualization

In def visualize_output(test_images, test_outputs, gt_pts=None, batch_size=10):
    # un-transform the key_pts data
    key_pts = test_outputs.data

should be correctly:
    # un-transform the key_pts data
    key_pts = test_outputs[i].data

Thank you so much for identifying this issue! I've updated the visualize_output function so that it should work properly now.


Hs 5 add dynamic firewalls

Added code for dynamic firewalls in 3 different methods in ContainerThread.py
create_rule(), start_dynamic_firewall(), and end_dynamic_firewall().
Added some simple unit tests. More robust tests will need to be created

Codecov Report

Merging #4 into dev will increase coverage by 6.69%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev       #4      +/-   ##
==========================================
+ Coverage   15.51%   22.20%   +6.69%
==========================================
  Files          17       18       +1
  Lines         754      806      +52
==========================================
+ Hits          117      179      +62
+ Misses        637      627      -10

Impacted Files
Coverage Δ

hpotter/test/test_ContainerThread.py
100.00% <0.00%> (ø)

hpotter/plugins/ContainerThread.py
37.70% <0.00%> (+37.70%)
:arrow_up:

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 1aef40a...620f059. Read the comment docs.

Closing pull request, the last push did not go through

Codecov Report

Merging #4 into dev will increase coverage by 8.2%.
The diff coverage is 86.76%.

@@            Coverage Diff            @@
##              dev       #4     +/-   ##
=========================================
+ Coverage   15.51%   23.72%   +8.2%
=========================================
  Files          17       18      +1
  Lines         754      822     +68
=========================================
+ Hits          117      195     +78
+ Misses        637      627     -10

Impacted Files
Coverage Δ

hpotter/test/test_ContainerThread.py
100% <100%> (ø)

hpotter/plugins/ContainerThread.py
37.7% <75%> (+37.7%)
:arrow_up:

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 1aef40a...0b0f077. Read the comment docs.


Cannot install squido on Ubuntu 22.04

When I try to install squido on a bare bones (+ git) Ubuntu 22.04 install with sudo npm i -g https://github.com/mrvautin/squido.git it gives me a TON of warnings (more than are shown here) and then gives an error about a syscall to spawn sh.
npm warn tar TAR_ENTRY_ERROR ENOENT: no such file or directory, lstat '/usr/local/lib/node_modules/squido/node_modules/date-fns/yearsToMonths'
npm warn tar TAR_ENTRY_ERROR ENOENT: no such file or directory, lstat '/usr/local/lib/node_modules/squido/node_modules/date-fns/yearsToQuarters'
npm warn tar TAR_ENTRY_ERROR ENOENT: no such file or directory, open '/usr/local/lib/node_modules/squido/node_modules/date-fns/typings.d.ts'
npm notice
npm notice New minor version of npm available! 10.7.0 -> 10.8.1
npm notice Changelog: https://github.com/npm/cli/releases/tag/v10.8.1
npm notice To update run: npm install -g<EMAIL_ADDRESS>npm notice
npm error code ENOENT
npm error syscall spawn sh
npm error path /usr/local/lib/node_modules/squido/node_modules/esbuild
npm error errno -2
npm error enoent spawn sh ENOENT
npm error enoent This is related to npm not being able to find a file.
npm error enoent

Full terminal output part 1
Full terminal output part 2
I have node/npm installed with the snap sudo snap install node --classic but tried other configurations too.

It is only when I use npm i -g...
I am able to git clone https://mrvautin/squido, build and then install OR use npx new and npx serve.
Also, I was confused by the How to build a static HTML blog in 10 minutes with squido having # marks before the commands. That made me think npm i -g https://github.com/mrvautin/squido.git needed to be run as root.

Hi @JonathanLindsey it appears recent changes to npm have caused the global install to no longer work.
I've updated the README file with some npx commands which allow you to run squido without globally installing the package.
Let me know how you go.

Thanks. That works.
May I suggest changing the # marks before the commands on How to build a static HTML blog in 10 minutes with squido to $ like it is in the documentation? (I'd offer a PR but it looks like that repo is private.)


Installation failed via cargo

Currently I can't install it via cargo,  this is the log:
$ cargo install lazycli

...
   Compiling lazycli v0.1.15
error[E0599]: no method named `about` found for struct `Arg` in the current scope
  --> /home/savaka/.cargo/registry/src/github.com-1ecc6299db9ec823/lazycli-0.1.15/src/args.rs:20:12
   |
20 |           .about("ignores the first `n` lines of output")
   |            ^^^^^ method not found in `Arg<'_>`

error[E0599]: no method named `about` found for struct `Arg` in the current scope
  --> /home/savaka/.cargo/registry/src/github.com-1ecc6299db9ec823/lazycli-0.1.15/src/args.rs:28:12
   |
28 | ...   .about("frequency of refreshing the content in seconds (values like 0.1 are permitted. Values like 0.001? Also permitted, but you n...
   |        ^^^^^ method not found in `Arg<'_>`

For more information about this error, try `rustc --explain E0599`.
error: failed to compile `lazycli v0.1.15`, intermediate artifacts can be found at `/tmp/cargo-install5hpoM9`

Caused by:
  could not compile `lazycli` due to 2 previous errors

It seems that the Cargo.lock file is ignored and then a version of the clap library with breaking API changes is installed.
You can force Cargo to use the Cargo.lock file using the --locked flag:
cargo install lazycli --locked

Crazy that --locked isn't set by default! I've updated the readme accordingly


🐛 ICustomization for DateOnly causes a hang.

Describe the Bug
When I used the below fixture customization to handle DateOnly properties, the build/create call hangs forever.
Scenario
public sealed class DateOnlyFixtureCustomization : ICustomization {
    public void Customize(IFixture fixture) => fixture.Customize<DateOnly>(x => x.FromFactory<DateTime>(DateOnly.FromDateTime));
}

public static class FixtureFactory
{
    public static IFixture Create() {
        var fixture = new Fixture().Customize(new DateOnlyFixtureCustomization());

        fixture.Behaviors
            .OfType<ThrowingRecursionBehavior>()
            .ToList()
            .ForEach(x => fixture.Behaviors.Remove(x));

        fixture.Behaviors.Add(new OmitOnRecursionBehavior(2));

        return fixture;
    }
}

Then I simply use it like so:
var client = FixtureFactory.Create().Build<Client>()
    .With(...)
    .Create();

Expected Behavior
Assign a DateOnly value to the appropriate properties.
Tasks
I'm using 4.18.0, and from googling it looks like the above is how I'm supposed to make this work

@grosch are you sure the issue is due to the DateOnly customization? I see you've added for some reason the OmitOnRecursionBehavior which is not required for the DateOnly type as it does not have any recursive members, nor any writable properties for that matter.
Could you provide some insight into the Client class you're trying to create?


Docs: improvement

I tried to test collision for my pet project, where shapes gonna use rotation and only after half a day of trying to find another collision libriry, I discovered, that angles are using in radians.
// wrong 45deg rotation
new Box({ x: 100, y: 100 }, 100, 100, { angle: 45 });

// correct 45deg rotation
new Box({ x: 100, y: 100 }, 100, 100, { angle: 0.785398 });

Please, provide more params details, I like your lib, I rly do, but it should be more friendly. I think many ppl have abandoned it after failing to figure out how to use it.

@vadimTestPlatform thank you for opening an issue
of course, I will add this to docs and provide better explanation
I will also add a conversion functions deg2rad and rad2deg for easier use
are there any other improvements I can do?

added those 3 inside readme and api docs since 6.8.0
looking forward to improving further

Body property "isStatic", no description highlights the point of using it

Highlight "Deprecated" paragraph

unclear "isTrigger" property description, "as ghost" as I understood - it doen't trigger collision on check, but it does

@vadimTestPlatform what do you think about https://github.com/Prozi/detect-collisions/blob/dev/README.md I added some improvements you mentioned and then some more

@Prozi looks good, just few moments:

line 222, "although this is very not recommended as the BVH (rbush) broad-phase-search with bounding boxes makes the collision checking a lot more efficient."

"although" -> "Although"
I think phrases "very not recommended" & "a lot more efficient" should be somehow highlighted to emphasize the importance of the possible performance issues.

Although I want to recommend you change Benchmark table output: "index" column is little confusing(it's superfluous) and "value" column, maybe shoud be renamed to "fps" or I event can't understand, what "value" is meaning there )

system method "checkCollision" has unclear param "response'
need to provide the meaning of its use
and I recommend to rename params from "body" & "wall" -> "body1" & body2", cause the "wall" param could be represented with circle etc, that's a little bit confusing

some contribution instructions in the README would also be nice :)

thanks for input.
about renaming wall to body I did it and will ship with next minimal update
about console.table indexes https://developer.mozilla.org/en-US/docs/Web/API/Console/table so I guess I'd have to switch to some different logs for that
about improvements to english in readme thanks will do them
will add some contributions sections too###

improve system "draw" & "drawBVH" methods description
like here: https://prozi.github.io/detect-collisions/classes/Box.html#draw.draw-1
also can't understand the differences between them
maybe should provide more info in readme
at "The BVH can also be drawn to help test" section

hello @Mero-Plaform
since version v8.0.0 the documentation has been extended on this subject,
moreover all the bodies have gotten their own independent drawBVH functions
please see

closing until more changes to docs requested, thanks


Mixins for structure modules

An important feature currently missing from the framework. Just posting this as a reminder to myself to implement eventually.

Implemented as of today. Check out the structure mixin under core/structures/_structure.scss


mklfpk.sh doesn't detect downloading errors

Describe the bug
No rule to make target '/lnx/lib/intel64/libdaal_vmlipp_core.a' error is shown during DAAL build
To Reproduce
Steps to reproduce the behavior:

Setup
1.1 source /opt/intel/compilers_and_libraries/linux/bin/compilervars.sh intel64
1.2 source /opt/intel/tbb/bin/tbbvars.sh intel64
Call make -f makefile daal PLAT=lnx32e
See error

========= Building core =========
make: *** No rule to make target '/lnx/lib/intel64/libdaal_vmlipp_core.a', needed by '__work/lnx32e/daal/lib/libdaal_core.a'.  Stop.

Expected behavior
Build succeeded
Environment:

OS: e.g. Ubuntu 18.04
Compiler: icpc (ICC) <IP_ADDRESS>1 20190815
DAAL Version: 2f4ad00fdfadba45769faf46927f6bd165112224

Hello!
how you did check out?
looks like you missed  step #5 from "Installation from Sources" instruction.
Andrey

Hello @dr-pain,
I've missed the call to the scripts/mklfpk.sh 32e on description, thank you for the notice.
By the way, in my case scripts/mklfpk.sh 32e failed to download via curl due to proxy settings, but the script didn't handle it and proceed to the untaring, failed and finally said that Downloaded and unpacked Intel(R) MKL small libraries to /home/rpovelik/dev/daal/externals/mklfpk :)
To sum up, I didn't notice that curl failed to download libraries.
It would be great if we can detect the error of intermediate steps and fail the build.

we will try to fix mklfpk.sh soon.
we will also try to make diagnostic for this case (now it is "No rule to make target '/lnx/lib/intel64/libdaal_vmlipp_core.a' ") more informative.
Andrey

@dr-pain, thanks!

@PovelikinRostislav issue with mklfpk.sh committed to master.
https://github.com/intel/daal/pull/170

@PovelikinRostislav, could you, please, confirm that the issue is resolved?

Confirmed, fixed

we will also try to make diagnostic for this case (now it is "No rule to make target '/lnx/lib/intel64/libdaal_vmlipp_core.a' ") more informative.
https://github.com/intel/daal/pull/247


docs

Describe the problem
no docs
Describe the proposed solution
write them
Alternatives considered
none
Importance
i cannot use vitest-plugin-playwright-e2e without it

Hey there @dominikg , I just wanted to ask whether this plugin is still being worked on?

I'm quite busy these days.
for e2e i recommend you take a look at using @playwright/test directly to be able to use their advanced features like https://playwright.dev/docs/trace-viewer
for component testing there are efforts from both vitest and playwright that may improve the situation so that you're able to use a real browser env instead of jsdom or happy-dom.
Once the situation has cleared up where vitest and playwright are heading and how this plugin might fit in it'll be picked up.

I'm quite busy these days.
for e2e i recommend you take a look at using @playwright/test directly to be able to use their advanced features like https://playwright.dev/docs/trace-viewer
for component testing there are efforts from both vitest and playwright that may improve the situation so that you're able to use a real browser env instead of jsdom or happy-dom.
Once the situation has cleared up where vitest and playwright are heading and how this plugin might fit in it'll be picked up.

Thanks for the update, appreciate it!


faster-hex v0.8.0 fails to build

The error is:
error[E0425]: cannot find function `vectorization_support_no_cache_x86` in this scope
  --> /home/shnatsel/.cargo/registry/src/index.crates.io-6f17d22bba15001f/faster-hex-0.8.0/src/lib.rs:63:28
   |
63 |         let val = unsafe { vectorization_support_no_cache_x86() };
   |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ not found in this scope

The signature of the missing function is:
// We enable xsave so it can inline the _xgetbv call.
#[target_feature(enable = "xsave")]
#[cfg(any(target_arch = "x86", target_arch = "x86_64"))]
#[cold]
unsafe fn vectorization_support_no_cache_x86() -> Vectorization {

or see it in context here: https://github.com/nervosnetwork/faster-hex/blob/856aba7b141a5fe16113fae110d535065882f25a/src/lib.rs#L72-L76
rustc version:
rustc 1.75.0-nightly (31bc7e2c4 2023-10-30)
binary: rustc
commit-hash: 31bc7e2c47e82798a392c770611975a6883132c8
commit-date: 2023-10-30
host: x86_64-unknown-linux-gnu
release: 1.75.0-nightly
LLVM version: 17.0.3

This issue blocks the compilation of anything that depends on gix, the Git implementation in Rust. That includes cargo itself, numerous Cargo plugins (cargo audit, cargo deny, cargo-semver-checks, cargo edit, cargo release, cargo vet, etc) and other tools such as starship, bacon, onefetch, etc.

This is the problem: https://github.com/nervosnetwork/faster-hex/blob/7ae3424144e578fe9023be8f2c11268d8cf15e16/src/lib.rs#L76 cfg(target_feature = "...") will always be false for cg_clif currently as I haven't implemented CodegenBackend::target_features yet. It looks like faster-hex 0.8.1 has removed this condition, so updating to that version would be a workaround. Shouldn't be too hard to implement CodegenBackend::target_features based the target features specified in the target spec though at least.

Indeed, I can confirm that upgrading to 0.8.1 fixes the issue. And using the cranelift backend cuts the debug build time by 40% for cargo audit :tada:

As of https://github.com/rust-lang/rustc_codegen_cranelift/commit/45d8c121ba02c825379b655d8dd74e1843e98d62 SSE is now reported as enabled on x86_64 by cg_clif.


Temporarily pin libgit2-sys.

There are some issues with the most recent libgit2-sys 0.14.2 not working on Windows (https://github.com/libgit2/libgit2/issues/6453 and https://github.com/libgit2/libgit2/issues/6454). Until we figure out what to do with it, this pins the release to the previous version.

@bors r+
See the discussion on Zulip: https://rust-lang.zulipchat.com/#narrow/stream/246057-t-cargo/topic/libgit2.20issue

:pushpin: Commit 198d2ee4320d4035853acc0a0cfe9a421f3253aa has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 198d2ee4320d4035853acc0a0cfe9a421f3253aa with merge 1a71b3d19ea8c3022a455a240883237f1d4cd0f7...

:broken_heart: Test failed - checks-actions

@bors retry
#6258

:hourglass: Testing commit 198d2ee4320d4035853acc0a0cfe9a421f3253aa with merge a38b83cff28e6e226914ff001dbdb907664d2902...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing a38b83cff28e6e226914ff001dbdb907664d2902 to master...


Added tN & TSV parsing

Go to branch feature-mannycolon-5757 in tC.
rm -rf node_modules
npm i
npm start
Click Check for content updates on the actions button on the top right corner.
Select English from the list (tN tsvs are only available in EN)
Click download
Should download tN correctly

This change is

LGTM


Feature request: A way to save current data from flow and to insert data to flow

Is your feature request related to a problem? Please describe.
Currently there is no simple way to save values from operations to use in later operations
Describe the solution you'd like

A feature to save the current value from the flow to use later in the flow preferable with a nameable variable (Store operation)
A feature to load variables into the current flow (Load operation)
Ability to use the variables as arguments the same way that it is possible with the Register values

The current way I handle this is by using Register to save values and Find/Replace to insert values, example recipe below
Link to recipe

+1


incomplete solution

Hi, I was pretty excited for this solution, particular number 8 on https://aws.amazon.com/about-aws/whats-new/2021/06/new-aws-solutions-implementation-tag-tamer/ to auto-tag more than ec2 instances, snapshots and volumes (which is something we have all already written/re-used a lambda for)
But look as I may I could not find anything in this 1.0 solution. There is no lambda, no Cloudformation for the events from eventbridge.    I was really surprised that this has been promoted as an AWS Solutions.
Maybe you could update step 8 as "Magic Happens here ?"

Hi @ian-ors
We're sorry this issue has caused you problems and the solution has not met your expectations.  We are creating and posting an update that will address this issue.

I look forward to it @billdry , to be clear, the expectations not meet were the ones that you set.


WIP: Add initial meme generator!

WIP, could probably be merged after tested 😃

Can we get this running on Dev1/staging?

@tzarebczan staging can happen after merge, dev1/dev2 would be up to @daovist 😃


Problem using on Chrome

I just tried to use JournalBook through the webpage on Google Chrome and had some issues. When I was trying to write a journal entry, the page kept reloading and wouldn't let me free-write. After every letter I typed the entire page would reload.

@jillianbaker25 How strange! Was this on a Mac/Windows/iPhone/Android? And was it a fresh journalbook or adding to an existing one with previous data?
Thanks!


v8: make it work on Mojave

Fix the following error when installing the "V8" R package on OSX Mojave:
ld: warning: directory not found for option '-L/var/folders/yb/vc950hrs225_j49yhntvv0140000gn/T//build-v8/opt/v8/lib'
ld: library not found for -lv8_base
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [V8.so] Error 1
ERROR: compilation failed for package ‘V8’
* removing ‘/Users/user/Library/R/3.5/library/V8’
* restoring previous ‘/Users/user/Library/R/3.5/library/V8’

It's pulling the wrong brew; ProductVersion for Mojave is now at:
macbookpro:autobrew$ sw_vers
ProductName:    Mac OS X
ProductVersion: 10.14.6
BuildVersion:   18G87

Thanks!

FYI this is not actually supported.

yeah, i went for the quick fix. thanks!


ObjectifyStorageIo.getUser() has weird behavior on correcting user's email

The ObjectifyStorageIo class queries user in two ways: getUser(userId, nullableEmail) and getUserFromEmail(nonNullEmail). It seems they assume users are associated with unique email because getUserFromEmail() never checks duplicated results.
The implementation getUser() is a bit complex. It attempt to hit on userId at first. If such user is found but results in different email, it corrects the email and update the datastore. However, it opens a vulnerability that it duplicates the email on distinct users accidentally or maliciously, due to the fact that email has no unique constraint.

This only place calling getUser(userId, email) is google login page. It queries by both userId given by App Engine's user service and email. Another source of users is local /setpw route, which userIds are randomly generated from getUserFromEmail().
There is (very) small probability that google user service's userId and local random userId collide. However, such collision on is disastrous. getUser(userId, email) will erase other's email and steal the ownership of data.

This would be a problem except for the following:

Google IDs are not UUIDs. Therefore, there will never be a collision (unless Google decides to change to using UUIDs, which would have a whole host of other issues).
We only ever enable Google authentication in production. Local login is only used for development. Therefore only one of the code paths will ever be followed.
Unless Google changes their policy, you cannot reuse an existing Google username, so that combined with 1&2, we don't expect that in production unique constraints are necessary for email addresses.

If you're not running in App Engine, then yes you might have some extremely rare (1:1^248) chances of a collision.
@jisqyv Do you want to weigh in w.r.t. some of the logic around email addresses? I recall that you mentioned cases that prompted the introduction of this logic.

Even with production case, do you implicitly assume users are uniquely determined by emails (without explicit constraint)? I looked into the google login code. It queries the user and potentially discard the original userId.
The storageIo has setUserEmail() defined and is implemented without checking uniqueness. That may breaks getUserFromEmail().

@jerry73204 @ewpatton Once upon a time this code was very clean. Users were always looked up by Google Provided UserId. Google User Ids are long integers, which do not collide with UUIDs, but are unique. However they are only unique to your site. I.e., If you run an instance of App Inventor and look at the Google assigned ID when you login, then setup a second instance (different App Engine App) and login with the same account you will find that Google gives you a different ID.
In those olden days the e-mail address was just used for display purposes, and was updated if Google provided an update.
Then we implemented a stand alone version which didn't rely on Google IDs, and the email address became the primary identifier for an account (and we assigned a UUID, to be used as the primary key within the UserData object). Things started to get messy because in effect we patched on the code for the stand alone system. Some would call this code scar tissue!
Today we have four different ways of identifying accounts and three implementations of the StorageIo interface, only one of which (ObjectifyStorageIo) is in the public sources. And then we have the external login server which we are now using.
Let's see if I can enumerate the four ways you can login:

Google Account login (Google provide ID and Email)
Stand-Alone login, using e-mail address and password
External login (look for references to "token" in LoginServlet.java) where
login is done on another server and sent over in an encrypted token which today just contains the email address and interesting flags. This code is also a version of the code we use in Hong Kong where we integrate with a learning management platform written by a vendor. It handles login, so App Inventor is tightly integrated, including the ability to lock a person into a single project (so project switching is handled by the learning management platform). We use this today in production for our public instance. Logins are handled by login.appinventor.mit.edu, a server outside of App Engine (currently in AWS).
QQ Login, implemented for our Guangzhou stand alone implementation (code written and maintained by staff in Guangzhou).

So its a bit of a mess today. It may be time to rewrite that whole piece of code and get rid of all of the variants of getUser() completely. My advice would be to replace it with something like:
getUser(String eMail, boolean create)
Note: The UUID/UserId is never an input parameter. Accounts are only looked up by e-mail address (and an error generated and logged if two accounts exist with the same e-mail address). The create flag says whether or not to create an account if it doesn't otherwise exist.
I believe all the functionality we need can be handled by the one call, but would require some code re-factoring.
I have this on my radar. However recently I've been swamped with more urgent matters (like first dealing with API 26 requirements, Google Play restrictions and most recently with changes in OAuth authentication required by Google's rapid shutdown of Google+.
-Jeff


Move wasm into data dir

To be included in cosmovisor backup

and junod unsafe-reset-all

Docker failure is spurious. I'm yet to sort the config, apols.


Evaluation segmentation result Bangalore

[x] MATLAB script for loading the segmentation results for all classes from the saved MAT file to binary masks saved in (ordinary) tif files.

[x] Fix the urban and vegetation masks to be  of type uint8 and rename to builtup and nonbuiltup and upload to Beehub.

[x] Use/make the/a source Python notebook for calculating the Jaccard Index (JI) between the ground truth masks and the result masks. See Demo notebook 1_JaccardIndexBinaryMasks

[ ] Make a table with the results and add it to the paper.

Very strange JIndicies (>1)!?

---------------------------------------------------------------------------------------
Loading  ROI1
Visualizing  ROI1
Jacard Index computations for  ROI1
JSS between slum ground truth mask and segmentation result:  1.13443976839
JSS between builtup manual truth mask and segmentation result:  1.84221673512
JSS between non-built-up manual truth mask and segmentation result:  0.311862968548
---------------------------------------------------------------------------------------
Loading  ROI2
Visualizing  ROI2
Jacard Index computations for  ROI2
JSS between slum ground truth mask and segmentation result:  0.369944615735
JSS between builtup manual truth mask and segmentation result:  1.03755905042
JSS between non-built-up manual truth mask and segmentation result:  0.283259629108
---------------------------------------------------------------------------------------
Loading  ROI3
Visualizing  ROI3
Jacard Index computations for  ROI3
JSS between slum ground truth mask and segmentation result:  0.0504021377958
JSS between builtup manual truth mask and segmentation result:  0.94099422707
JSS between non-built-up manual truth mask and segmentation result:  0.557391683695
---------------------------------------------------------------------------------------
Loading  ROI4

/home/elena/.local/lib/python3.5/site-packages/PIL/Image.py:2496: DecompressionBombWarning: Image size (100008360 pixels) exceeds limit of 89478485 pixels, could be decompression bomb DOS attack.
  DecompressionBombWarning)

Visualizing  ROI4
Jacard Index computations for  ROI4
JSS between slum ground truth mask and segmentation result:  0.113874340462
JSS between builtup manual truth mask and segmentation result:  1.76153227317
JSS between non-built-up manual truth mask and segmentation result:  0.637087545879
---------------------------------------------------------------------------------------
Loading  ROI5
Visualizing  ROI5
Jacard Index computations for  ROI5
JSS between slum ground truth mask and segmentation result:  0.205312714519
JSS between builtup manual truth mask and segmentation result:  2.38191806087
JSS between non-built-up manual truth mask and segmentation result:  0.726816109273

Enforcing normalize=True for the scikitlearn jaccard_similarity_score calculation gives the same scores.


Running inference outside of pre-computed datasets

Dear authors,
Congratulations on Uni-Mol! This is truly exciting work 🥇
I'm writing to express interest in running inference outside of the pre-computed datasets. For example:
1.) Running docking from a pdb file + bounding box
2.) Embedding molecules from smiles
3.) Embedding proteins from pdb files
Looking at the code, it wasn't clear to me where to get started on this, but let me know if I can help.
Best wishes,
-Cyrus

Hi Cyrus,
Thanks for your advices, we will add more examples and demo in future updates, especailly for inferencing outiside of pre-computed datasets.
For fast implements, you can change the docking demo on Colab about reading residues for target pockets to select residues in bounding box with rewriting parser and Load_from_CASF functions. And for molecules/pocket embeddings just add a additional output of CLS embedding in unimol model, you can use rdkit to generate conformations as Uni-Mol inputs.  More details you can refer to appendix in our paper.
It will be great and much helpful if you can contribute to Uni-Mol together with us!
Best,

Thank you for your reply @Naplessss! I started with getting the example notebook to run locally, but ran into a couple issues:
1.) My python version wasn't 3.7 so the unicore wheel wouldn't install at first. For myself I added a snippet to check whether the python version was 3.7 and to link to  https://github.com/dptech-corp/Uni-Core/releases if not.
2.) The docking weights are downloaded to the current working directory, but then expected in "/content"
3.) When I fix these issues I get the error that it can't find the "docking_pose" task in unicore:

Do you have advice on what to do from here?
Thanks!
-Cyrus

Do you already set your use dir path with --use-dir?  also you can check if unimol,unicore is installed successfully.

Yes, --use-dir is set in unimol_binding_pose_demo.ipynb. I install unicore from 'https://github.com/dptech-corp/Uni-Core/releases/download/0.0.1/unicore-0.0.1+cu113torch1.12.1-cp37-cp37m-linux_x86_64.whl'.
Below is the output when I import unicore and print the __version__.

For UniMol, the notebook is downloading another copy of the repo into the working directory (Uni-Mol/notebooks). It does seem to be able to find this copy of unimol using --user-dir ./unimol.


Fix VlxDevConstant value for PARTIAL_BYPASS_OFF

Unfortunately, i committed a wrong value for this constant.
It is a non critical issue because the value can not be written due to a vallox fw bug. But i would appreciate to get it into the next planned release.
Thank you!

Thanks for the fix and your commitment! I'll release version 1.0.1 soon.

Published with version 1.0.1


Proposal: switching to Vertx 4 already

It seems Vert.x 4 is already in CR1, with a CR2 coming soon.
We're currently "stuck" on Vert.x v 3.x because we need to stay compatible with the Vert.x versions used in Quarkus; however:

Quarkus plans to upgrade ASAP in 2021, expecting migration completion by March (but this very issue is meant to simplify that work)
It seems we only need changes in a single class in Hibernate Reactive to be compatible with both
It would be odd for the Hibernate Reactive community to impose a major upgrade of such a crucial dependency so close to our own 1.0 release.

So, I'd like us to explore being able to "baseline" on Vert.x 4 already: switch the dependency to v4, and document it as the recommended version, use it in examples - while applying some internal magic to keep runtime compatibility with v3 just for a bit longer, only to satisfy integration needs in Quarkus for as long as Quarkus doesn't upgrade as well.
I've already discussed this in chat with @DavideD and he thought this shouldn't be too hard - pending exploration.
This would also allow us to release with the new drivers (MSSQL support?), as I'm assuming we can keep them out of the Quarkus tree.
Sounds good? Asking @gavinking and @blafond in particular, as I already chatted with Davide.
I hope you like the idea, as it would take some barriers away from our velocity, and reduce the complexity of the Vertx upgrade in Quarkus - so helping them as well.

I'm skeptical that this is worth the effort. March is really not far away and there are other things we could spend time on.

I think it's a small investment and will reduce the total volume of work.
So unless you're highly preferring to release with Vertx 3 first, I'd like to give it a shot - we can abort if it gets unnecessarily complicated.

Well I mean I gotta be honest I'm just not really keen on seeing reflection-based code added to the codebase if we're just going to throw it away in four months time when Vert.x 4 is released.

oh I totally agree, but it's only a couple of methods and we'll get rid of them soon as well.

Closing this as it turns out it was a bad idea: too complex.

Closing this as it turns out it was a bad idea: too complex.


support disabling multi-level gateways.

Description:
support disabling multi-level gateways.

Testing done:

Affected area:

Functional Area

New Functionality
[X]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Is this a breaking change? no

Has documentation corresponding to this change been updated in the fsm-docs repo (if applicable)? no

Codecov Report
Attention: Patch coverage is 0% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 47.14%. Comparing base (7f01986) to head (63bbde1).
Report is 34 commits behind head on release/v1.2.

Files
Patch %
Lines

pkg/providers/kube/client.go
0.00%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@               Coverage Diff                @@
##           release/v1.2     #223      +/-   ##
================================================
+ Coverage         46.83%   47.14%   +0.30%
================================================
  Files               199      196       -3
  Lines             15232    12118    -3114
================================================
- Hits               7134     5713    -1421
+ Misses             8052     6361    -1691
+ Partials             46       44       -2

Flag
Coverage Δ

unittests
47.14% <0.00%> (+0.30%)
:arrow_up:

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

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


DO NOT MERGE

JIRA link
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Closing as this was a test


The default memory (4G) for coordinator may not enough to build an application

Could we reduce the dependency of app_frame to make it less memory-intensive?

Related to #1266.
It's time to reduce the intra-dependency complexity between our modules...

And speed up the compilation of UDF apps.

The app build process has moved to engine worker.


Acquire and leak main context in gtk::init()

Fixes partially gtk-rs/gtk-rs-core#186, because gtk4 still has that issue.

The reason for the failed job seems to be unrelated to my change:
2021-07-31T15:59:46.8746626Z meson.build:1:0: ERROR: Value "false" (of type "string") for combo option "Enable GObject Introspection (depends on GObject)" is not one of the choices. Possible choices are (as string): "enabled", "disabled", "auto".
2021-07-31T15:59:46.8747658Z
2021-07-31T15:59:46.8748628Z A full log can be found at C:\gtk-build\build\x64\release\graphene\_gvsbuild-meson\meson-logs\meson-log.txt
2021-07-31T15:59:46.9338617Z Traceback (most recent call last):
2021-07-31T15:59:46.9339631Z    same file (C:\gtk-build\build\x64\release\gettext\build\win32\vs15\gettextlib.vcxproj.filters)
2021-07-31T15:59:46.9340859Z   File "D:\a\gtk3-rs\gtk3-rs\gvsbuild\gvsbuild\utils\builder.py", line 492, in build
2021-07-31T15:59:46.9341530Z     if self.__build_one(p):
2021-07-31T15:59:46.9342584Z   File "D:\a\gtk3-rs\gtk3-rs\gvsbuild\gvsbuild\utils\builder.py", line 618, in __build_one
2021-07-31T15:59:46.9343378Z     skip_deps = proj.build()
2021-07-31T15:59:46.9344038Z   File "D:\a\gtk3-rs\gtk3-rs\gvsbuild\gvsbuild\projects.py", line 587, in build
2021-07-31T15:59:46.9344749Z     Meson.build(self, make_tests=True)
2021-07-31T15:59:46.9365782Z   File "D:\a\gtk3-rs\gtk3-rs\gvsbuild\gvsbuild\utils\base_builders.py", line 66, in build
2021-07-31T15:59:46.9366689Z     self.exec_vs(cmd, add_path=add_path)
2021-07-31T15:59:46.9367442Z   File "D:\a\gtk3-rs\gtk3-rs\gvsbuild\gvsbuild\utils\base_project.py", line 110, in exec_vs
2021-07-31T15:59:46.9368369Z     self.builder.exec_vs(cmd, working_dir=self._get_working_dir(), add_path=add_path)
2021-07-31T15:59:46.9369272Z   File "D:\a\gtk3-rs\gtk3-rs\gvsbuild\gvsbuild\utils\builder.py", line 865, in exec_vs
2021-07-31T15:59:46.9370168Z     self.__execute(self.__sub_vars(cmd), working_dir=working_dir, add_path=add_path, env=self.vs_env)
2021-07-31T15:59:46.9371192Z   File "D:\a\gtk3-rs\gtk3-rs\gvsbuild\gvsbuild\utils\builder.py", line 942, in __execute
2021-07-31T15:59:46.9372049Z     subprocess.check_call(args, cwd=working_dir, env=env, shell=True)
2021-07-31T15:59:46.9372969Z   File "C:\hostedtoolcache\windows\Python\3.7.9\x64\lib\subprocess.py", line 363, in check_call
2021-07-31T15:59:46.9373802Z     raise CalledProcessError(retcode, cmd)
2021-07-31T15:59:46.9377061Z subprocess.CalledProcessError: Command 'C:\gtk-build\tools\python.3.7.7\tools\python.exe C:\gtk-build\tools\meson-fd6585ecc41e1c731224cac08a3f6ef6f49ab184\meson.py C:\gtk-build\build\x64\release\graphene C:\gtk-build\build\x64\release\graphene\_gvsbuild-meson --prefix C:\gtk-build\gtk\x64\release -Dbenchmarks=false -Dintrospection=false --buildtype debugoptimized' returned non-zero exit status 1.
2021-07-31T15:59:46.9379516Z Error: graphene build failed

The reason for the failed job seems to be unrelated to my change:

This probably disappears on a retry... or needs to be reported to graphene.

@ArekPiekarz Thanks, you also do the same for GTK4?

@sdroege Yes, I will try to do that.

FYI now the Windows build failed for a different reason, again probably unrelated to this change:
[577/989] Generating test.gresource with a custom command (wrapped by meson to set PATH)
FAILED: gio/tests/test.gresource
"C:/gtk-build/tools/python.3.7.7/tools/python.exe" "C:/gtk-build/tools/meson-fd6585ecc41e1c731224cac08a3f6ef6f49ab184/meson.py" "--internal" "exe" "--unpickle" "C:/gtk-build/build/x64/release/glib/_gvsbuild-meson/meson-private/meson_exe_glib-compile-resources_c9c94695beae3266dd669148af439d87e97211d7.dat"
../gio/tests/test.gresource.xml: Failed to execute helper program (No such file or directory).
while executing ['C:\\gtk-build\\build\\x64\\release\\glib\\_gvsbuild-meson\\gio\\glib-compile-resources.exe', '--target=gio/tests/test.gresource', '--sourcedir=C:/gtk-build/build/x64/release/glib/gio/tests', '--sourcedir=C:/gtk-build/build/x64/release/glib/_gvsbuild-meson/gio/tests', '../gio/tests/test.gresource.xml']

Just want to confirm one thing: can we still get the context if we want (after calling init I mean)?

@GuillaumeGomez Yes, I tested it with a following small program:
use gtk::glib;

const DEFAULT_CONTEXT : Option<&glib::MainContext> = None;

fn main() {
    gtk::init().unwrap();
    let (_sender, receiver) = glib::MainContext::channel(glib::PRIORITY_DEFAULT);
    let handler = move |event: i32| {
        println!("got event: {}", event);
        glib::Continue(true)
    };
    receiver.attach(DEFAULT_CONTEXT, handler);  // panicked before this change
}

When we go into definition of Receiver::attach(), it calls MainContext::ref_thread_default() that retrieves the default context, asserts that it is its owner, then uses it.

Thanks for the explanations! Would it be worth it to add such a test as well so we don't have a regression again?

@GuillaumeGomez Good idea, I added a unit test for that.

Thanks!

To save you time, Windows build failed again on graphene and "CI / gtk (rust nightly)" on linking Cairo:
note: /usr/bin/ld: /__w/gtk3-rs/gtk3-rs/target/debug/deps/libcairo-5e44ed6c27c39d69.rlib(cairo-5e44ed6c27c39d69.cairo.3157f652-cgu.0.rcgu.o): undefined reference to symbol 'cairo_gobject_extend_get_type'

Not surprised, for whatever reason, windows CI is broken. Just waiting for @sdroege to take a look.

@GuillaumeGomez Good idea, I added a unit test for that.

Actually a bad idea. Not only does it fail on the CI now (because -lcairo-1.0 -lcairo-gobject-1.0 is missing for some reason) but also will it fail in various ways later (because you can initialize only from one thread per process but all tests are running from different threads).
@GuillaumeGomez If you insist on a test, how do you want to solve this? :)

I guess it's not possible unless we run all tests in one thread... Well, let's remove the test and merge this then.

I guess it's not possible unless we run all tests in one thread... Well, let's remove the test and merge this then.

We have some code for doing so in gtk4 if you want to use that here

Would be nice to backport this to gtk4 as well

Backport is already scheduled I think. As for the test, please provide the code! :D

I've been using rusty-fork crate for isolating tests in my apps, but I guess @bilelmoussaoui had TEST_THREAD_WORKER in mind.

Remove the test for now (or comment it out), then we can merge this here and do the test stuff independently.

And open an issue in any case. :p

This is now in 0.14.1 on crates.io


chore: Refactor WDSTableWidget with basic features

/build-deploy-preview skip-tests=true env=release

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true


added some more parameters for metadata transformation

added

Last-Modified
Display-Url
Crawl-Once
Lock

as parameters to be passed through to the metadata/document transformer, so that you can modify metadata based on more information

(1) is fixed
(2) is fixed with as good as I could do it. Is there some rule how to group your imports?

On Jul 16, 2015, at 12:50 AM, Dominik Weidenfeld<EMAIL_ADDRESS>wrote:

(1) is fixed
(2) is fixed with as good as I could do it. Is there some rule how to group your imports?

The rules for grouping and sorting imports are outlined
in the Google Java style guide:
https://google-styleguide.googlecode.com/svn/trunk/javaguide.html#s3.3-import-statements
The import statements were in the correct order and
spacing in the original file, and you neither added nor
removed any.  So simply restoring the imports to their
original arrangement should suffice.
Your pull request also contains numerous changes in
whitespace with no other functional code changes.
Some were changes in indentation, but most were
deleting trailing whitespace on lines.
This is discouraged for several reasons.  It makes
it difficult for us as reviewers to focus on your
actual functional changes.  It also pollutes the
version control history for those lines.
When problems are found with a particular piece of
code, we often use git blame to determine the author
and commit revision where that code was introduced
or modified.  We then examine all the changes for that
particular commit, the log entry, and the code review,
to try to understand why the code was written as it
was.  Gratuitous changes alter that history.
Since more than half of the changes in the code
are non-functional changes (re-ordering the imports
and changing whitespace), you may find it more
convenient to use git reset to restore the files to their
original state, then re-apply just your changes to
the transform() method.
—
Brett M. Johnson

Thanks Brett for the nice explanation.
I've added a couple of comments to the 2nd commit.
Thank you for this work,
PJ

Thank you for your response. I will change the code style and implement the checks as you wanted (maybe beginning next week).
The code style changes has only been made because the IDE optimises the code on save automatically. I will try to revert my changes completely and only add my changes (maybe with a normal text editor).


@Inject(CONTEXT) is undefined

It seems that @Inject(CONTEXT) is not working with typegraphql-nestjs the way it does for @nestjs/graphql.
I am trying to do something like this:
import { Inject, Injectable } from "@nestjs/common";
import { CONTEXT } from "@nestjs/graphql";
import { Recipe } from "./types";

@Injectable()
export default class RecipeService {
  private readonly recipes: Recipe[] = [];

  constructor(@Inject(CONTEXT) context) {
    console.log('Request Headers:', context?.req?.headers);
  }
}

With an app module that looks like this:
import { ApolloDriver } from "@nestjs/apollo";
import { Module } from "@nestjs/common";
import { TypeGraphQLModule } from "typegraphql-nestjs";

import RecipeModule from "./recipe/module";

@Module({
  imports: [
    TypeGraphQLModule.forRoot({
      driver: ApolloDriver,
      emitSchemaFile: true,
      context: ({ req, res }) => {
        return { req, res };
      }
    }),
    RecipeModule,
  ],
})
export default class AppModule {}

However, context is just undefined. A similar example using @nestjs/graphql however does work, mostly following https://docs.nestjs.com/fundamentals/injection-scopes#request-provider.
I have not found anything in the docs that says this should not work or how to achieve this differently, so if this is expected or just works differently please let me know. I think this may be a similar issue as discussed in https://github.com/MichalLytek/typegraphql-nestjs/issues/22, but it was never reproduced or fixed.
Initially I was not sure myself where the issue comes from, so I have created a reproduction with both typegraphql-nestjs and @nestjs/graphql here: https://github.com/ahilke/typegraphql-nestjs-inject-context.
Thank you for your support!

I think you need to use @Ctx() decorator from TypeGraphQL to inject context into resolver method.
You can't inject context into constructor.


Helm Charts Support

Is this a BUG REPORT or FEATURE REQUEST?:
/kind feature
What happened:
Helm is a popular package manager in Kubernetes. For easier deployment and operation, we should develop opensds charts to support helm.
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Hotpot(release/branch) version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

fyi, another agnostic packaging technology is snapd

@noelmcloughlin Looks like a cool installer tool! Can you draft an initial design proposal if you want to implement this feature? Thanks!

@noelmcloughlin BTW, I will close this issue, please be free to set up a new one if you want to add snapd deployment.

@leonwanghui Thanks. Raised Helm issue upstream.
https://github.com/saltstack-formulas/packages-formula/issues/19


jason

Is it possible to provide a json file for instance segmentation？

Instance annotations are currently unavailable on UDD, but you're welcome to contribute!


Update code formatting tools

#91 introduced an ESLint config to validate the ESLint rules, and as a result, this codebase now uses Standard (formatting and linting) as well as ESLint (linting).
https://github.com/xjamundx/eslint-plugin-promise/pull/91#issuecomment-364181144 makes a good point that Standard is ESLint under the hood, so some options I see moving forward:

update the ESLint config to extend eslint-config-standard and stop executing the standard CLI tool altogether
migrate to Prettier for code formatting, and use ESLint strictly for code linting

As a personal preference and having more experience with Prettier, I would choose the latter. I prefer having a file formatted the same way across IDEs and computers, either by saving a file or formatting in a precommit hook - standard doesn't necessarily enforce consistent line length and bracket spacing, from my experience.
Either way, part of resolving this issue would be to introduce lint-staged into the codebase, so that code formatting and linting would run as a precommit hook.
I will leave this open for a bit in case other contributors/maintainers have anything to add.

Yeah I love prettier


returned non-zero exit status 1. Something went wrong

Hi, I'm trying this since yesterday an I got the same results:
subprocess.CalledProcessError: Command '['/usr/bin/python3', '/content/diffusers/examples/dreambooth/train_dreambooth.py', '--image_captions_filename', '--save_starting_step=500', '--save_n_steps=0', '--train_text_encoder', '--pretrained_model_name_or_path=/content/stable-diffusion-v1-5', '--instance_data_dir=/content/data/jsr2', '--output_dir=/content/models/jsr2', '--instance_prompt=', '--seed=96576', '--resolution=512', '--mixed_precision=no', '--train_batch_size=1', '--gradient_accumulation_steps=1', '--use_8bit_adam', '--learning_rate=2e-6', '--lr_scheduler=polynomial', '--center_crop', '--lr_warmup_steps=0', '--max_train_steps=1000']' returned non-zero exit status 1.
Something went wrong

Traceback (most recent call last):
File "/content/diffusers/examples/dreambooth/train_dreambooth.py", line 694, in
main()
File "/content/diffusers/examples/dreambooth/train_dreambooth.py", line 439, in main
tokenizer = CLIPTokenizer.from_pretrained(args.pretrained_model_name_or_path, subfolder="tokenizer")
File "/usr/local/lib/python3.7/dist-packages/transformers/tokenization_utils_base.py", line 1748, in from_pretrained
_commit_hash=commit_hash,
File "/usr/local/lib/python3.7/dist-packages/transformers/utils/hub.py", line 420, in cached_file
local_files_only=local_files_only,
File "/usr/local/lib/python3.7/dist-packages/huggingface_hub/file_download.py", line 1022, in hf_hub_download
cache_dir, repo_folder_name(repo_id=repo_id, repo_type=repo_type)
File "/usr/local/lib/python3.7/dist-packages/huggingface_hub/utils/_validators.py", line 92, in _inner_fn
validate_repo_id(arg_value)
File "/usr/local/lib/python3.7/dist-packages/huggingface_hub/utils/_validators.py", line 137, in validate_repo_id
"Repo id must be in the form 'repo_name' or 'namespace/repo_name':"
huggingface_hub.utils._validators.HFValidationError: Repo id must be in the form 'repo_name' or 'namespace/repo_name': '/content/stable-diffusion-v1-5'. Use repo_type argument if needed.
Traceback (most recent call last):
File "/usr/local/bin/accelerate", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.7/dist-packages/accelerate/commands/accelerate_cli.py", line 43, in main
args.func(args)
File "/usr/local/lib/python3.7/dist-packages/accelerate/commands/launch.py", line 837, in launch_command
simple_launcher(args)
File "/usr/local/lib/python3.7/dist-packages/accelerate/commands/launch.py", line 354, in simple_launcher
raise subprocess.CalledProcessError(returncode=process.returncode, cmd=cmd)
subprocess.CalledProcessError: Command '['/usr/bin/python3', '/content/diffusers/examples/dreambooth/train_dreambooth.py', '--image_captions_filename', '--save_starting_step=500', '--save_n_steps=0', '--train_text_encoder', '--pretrained_model_name_or_path=/content/stable-diffusion-v1-5', '--instance_data_dir=/content/data/jsr2', '--output_dir=/content/models/jsr2', '--instance_prompt=', '--seed=96576', '--resolution=512', '--mixed_precision=no', '--train_batch_size=1', '--gradient_accumulation_steps=1', '--use_8bit_adam', '--learning_rate=2e-6', '--lr_scheduler=polynomial', '--center_crop', '--lr_warmup_steps=0', '--max_train_steps=1000']' returned non-zero exit status 1.
Something went wrong

Did you accept the terms at https://huggingface.co/runwayml/stable-diffusion-v1-5 ?

yes

there is not. Do I have to create it manually? I have to put the model inside?
I tried the same notebook yesterday and it worked. this changed?

when you put your huggingface token in the "download model" cell and run it, it should download the model, you don't have to do it manually, did you run the cell ?

yes, i followed all the steps

there should be a model folder, try a clean run

and make sure you use the latest  colab notebook

How can I check if it is the latest? The link is not the same?
https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast-DreamBooth.ipynb

yes that is the link, just avoid saving the notebook to your gdrive

Ok, yes, it seems I was using an outdated notebook. I'm trying it now and I see that the folder with the images doesn't appear anymore under "data". it is me, or it changed? Where are the images now?

Yes, everything is moved to your gdrive to allow resuming training easily after disconnecting from a session

Excellent! Thank you Ben! I really appreciate it ☕😉

happy to help


Textgame checkInput method

checkInput uses
command  = sc.next()   (Roughly line 32)
instead of sc.nextLine();
Thus, two word inputs like "check inventory" are inaccessable and will never work.
(I don't know who contributed to this or how to check)

I'm going to place this issue in the testing documentation writeup instead of on the issues page, since it can't be proplerly tested with Junit


Add rustbot prioritize command

Would be nice if we have @rustbot prioritize command that assigns I-prioritize label and also creates a new topic on Zulip's WG-prioritization stream requesting the working group to prioritize the in question issue.
Read more about the discussion about this feature request in this Zulip topic.

cc @rust-lang/wg-prioritization

Please also document on this issue the format for the topic (e.g., should it link to the issue?)
A sample topic in Zulip would be good, too.


fix chrome options configuration from capabilities

Coverage increased (+0.08%) to 90.115% when pulling 6767ee3b761a04674809bfa3056cdd4ca6e75fe2 on fix/chrome_options into 8abc76b84001f68428792e27219221b3522c8fc6 on master.


[material-ui][List] On applying high contrast themes, the focus indicator on the list items under "Movie" combo box is not visible in all the modes

Steps to reproduce
Link to live example:

Steps:
1.Open URL: heuristic-boyd-zw8xv7 - CodeSandbox
2.Sandbox editor page appears.
3.Use tab key to reach to "Movie" combo box and select it.
4.Navigate list items under "Movie" combo box using tab keys and observe whether the keyboard focus indicator is visible or not on the list items after applying high contrast themes.
Current behavior
On applying high contrast themes, the focus indicator on the list items present inside the "Movie" combo box is not visible in all the modes.
Observation:
In normal mode while navigating the list items there is an indication of focus (Represented with hash color) but, on applying high contrast themes no focus indicator is visible.
The issue is with all the four (Aquatic, Desert, Night sky and Dusk) modes.
Expected behavior
On applying high contrast themes, the focus indicator on the list items present inside the "Movie" combo box should be clearly visible in all the modes.
Context
Color Sensitive users who rely on High contrast modes would be impacted if the focus indicator or controls are not visible on applying high contrast themes. Due to this they cannot access the page efficiently and face difficulty in accessing the control.
Your environment
Windows 11 22H2 (OS Build:23620.1000)
Component: @mui/material/Autocomplete
Search keywords: high contrast themes

Thanks for reporting the issue.


refactoring

I am not convinced that it's a good idea to remove the white space on empty lines. If you would auto indent with Xcode, those will be changed back to their original state.

There is an Xcode option that is currently broken that will trim trailing white space . I hope they fix it soon.

Another reason why I think it makes sense to keep the current white space indentation. If you revert the indentation change for whitespace only lines, I could accept the other changes.


Optimize rendering performance

Kotlin version of WorldWind works noticeably slower then original Android and JS code-bases.
Especially it is visible in JS examples due to huge Kotlin to JS translation overheads like type checking, Java-like Hash Map implementation, equals and hash-code processing etc.
One general reason why Kotlin version feels slower is the re-projection of Mercator Image Tiles on the go. Default examples use EPSG:3857 tile sources from popular 2D map tile services like Open Street Maps, Google Maps, etc. and re-project them to EPSG:4326 required by 3D globe. Re-projection is a per-pixel modification of image 256x256 pixels size. This operation consumes much CPU especially in JS. Android caches re-projected tiles in GeoPackage and read re-projected tiles, but JS cache original tiles in the browser cache and re-project them each time on load. This problem can be avoided by using WMS EPSG:4326 tile sources.
The next most noticeable place in the code profiling is AbstractTiledElevationCoverage.scanHeightLimits() used by Tile.getExtent() function. This functionality recalculates map tile volume size (min and max height) each time when Elevation Coverage change its timestamp (new elevation tile is downloaded). For some reasons it is very CPU consumable operation and comparing to JS code-base it executes too much times. May be we need to recalculate tile heights not so often somehow. Need help to investigate this issue.
And the most wired thing is Kotlin JS overheads due to Java-like Hash Map implementation based on equals and hashCode functionality. There are to many calls of getStringHasheCode, Long.compareTo, Angle.CompareTo functions. Most of this calls appears due to the core of WorldWindKotlin engine is a RenderResourceCahce Hash Map, which is used to store Image Textures, Buffers,  Shader Programs etc. The access to this cache requires to calculate hash code of Image Tile URLs and other key objects to get or put resources into cache. In JS profiling this operation appears among top CPU consumers.
Another issue related to previous one is specific Angle class implementation. Most geometry classes are based on Latitude and Longitude values which are represented by the Angle class - a wrapper over Double degrees value with special angle calculation logic in it. This approach was taken from Java code-base and is very comfortable during code development, but Angle value is immutable and each arithmetic operation with angles creates new Angle object instance, plus each comparing of angles uses Java-like equals and compareTo functionality, which is very noticeable in JS profiling.
And one more consumption is that Surface Path rendering uses heavy logic to project tile on terrain, because it loops over all terrain tiles. This is the reason why PathPolygonsLablesActivity example has so poor performance.
Dear community, we need your help with performance investigation. Feel free to give us an advice.

One of the main issues of performance is too much tiles in a view  to horizon.
We need to speed up subdivision of tiles when looking into horizon and make frustum far plane dependent on camera altitude, because we do not need to render 160km distance when camera is 10m above the terrain.

We download all upper level tile textures and load them into resource cache, not only the required level. This performance impact was done to avoid blinking of dark holes when you zoom out or rapidly swipe to the side and nothing to display until required level is loaded.

Tiled image level of details in Kotlin version was made the same as in Google Maps - one pixel of tile is equals one dp. Original Android repo had details 4x worth which caused 4x less tiles in memory.

Probably we need to extend buffer size of elevation and image tiles cache, to avoid unnecessary generation and bounding box calculations.

When any new terrain tile is loaded all bounding boxes invalidate and whole scene of tiles and surface objects recalculates min and max elevation from scratch.

Each image layer and surface shape or image calculated its own bounding box and makes frustum intersection instead of just checking if it intersect current terrain, which already intersected frustum in this frame. Surface objects cannot be outside terrain.

All placemarks transfer own leader line buffer into graphical memory by separate command instead of having one buffer with offsets and transfer it once per frame.

We have taken into account the cos of angle between viewing vector and tile normal vector during subdivision and it decreased significantly amount of terrain and image tiles on horizon. After applying this optimization some other become not relevant:

Downloading of all upper level tile textures and loading them into resource cache may be kept, as it does not influence performance any more as tiles count decreased from 400 to 50.

Tiled image level of details in Kotlin version may be kept the same as in Google Maps - now it produces 18-50 image tiles on the scene.

The buffer size of elevation and image tiles cache was decreased instead of increasing as amount of tiles becomes significantly lower.

Invalidation of only those tiles which intersects with just loaded elevation tile instead of invalidation of whole scene may be extracted in a separate feature request, but now it is not urgent.

Intersection of surface objects with terrain tile sectors should be extracted to a separate feature request. Currently frustum culling is more performance effective then checking n(m) times each renderable with each terrain tiles sector.

Calculation of min and max elevations was optimized by decreasing height limit samples amount from 64 to 32. It makes algorithm which determines height rand to use the same level of details as further terrain rendering. This approach causes some mountains to be miss-picked on the very top points, but it gives significant voost in performance, so we can live with this.

Determination of altitude for placemarks clamped or relative to ground located on terrain outside frustum was optimized bu caching last 50 000 of request of elevation by coordinates. It significantly increased Placemarks rendering.

All placemarks transfer own leader line buffer into graphical memory by separate command instead of having one buffer with offsets and transfer it once per frame. This issue has a separate topic on big tracker and should be redesigned in future.

Subdivision of tiles is dependent on field of view and in case of very narrow FoV tiles become very narrow and this situation produces abnormal amount of terrain and image tiles. But after considering cos to tile normal the efficiency of dropping tiles LoD makes this issue obsolete. It does not influence performance any more.


DynamicFab on Android extended images not seen because of background/image color is same

Any image without a background color specified will be displayed as white which is same as the page background.
As a result, you cannot see these images displayed so nothing appears when the + is clicked.

I assume you have a light theme on your device.


PP-2714 Create payment_requests table

Created PaymentRequestEntity;
Created PaymentRequestDao;
Created test for Dao;
Created PaymentRequestEntityFixture;
Changed ChargeService#create to write to payment_requests table after writing into charges, inside the same transaction;
Added test for the new insert for the ChargeService;
Added migration for the new table;

with @alexbishop1

Can I request you put a bit more about why you're doing this/context into the PR  and commit message? Good practice, allows us to reason about changes later, and allows a greater range of people to confidently review changes


migrator: compare sub-kind when checking if two diff items are equivalent to avoid dropping information. rdar://32431567

@swift-ci please smoke test


Default flash area

Select flash memory area by default

That is indeed a useful feature. But I'd offer even more - when clicking on "flash" start programming immediately with flash memory area selected by default. And only, say, having CTRL or ALT pressed at the same time show the enhanced flash menu. Same for the build process.
Recently I was debugging an attiny10 project  and that's exactly what consumed time - click "build" at the bottom of the display, move cursor to the menu on the top of display, click "build" again, move cursor to the bottom, click "flash", move cursor to the top, click "flash, ok" etc.
ESP32 extension offers, for example, single "build and flash" button which is very useful and demands no other actions.

This feature was requested from the Marketplace page. I did both: added a new shortcut command and selected "flash" area by default when using "Flash" command. I still need to retest error handling before release.
In the meantime, to speed up your example workflow, one could use Ctrl Shift P to open the command list instead of clicking status bar items.
Feel free to suggest a good name for the new command and the new status bar item.

ESP-IDF has just an icon for this action:

published v2.2.0


misc: link from Authoring to Contribution guide is broken

from:

https://www.codeshiftcommunity.com/docs/authoring

links to:

https://www.codeshiftcommunity.com/docs/authoring/docs/contribution

when should link to:

https://www.codeshiftcommunity.com/docs/contribution

tried fixing it but haven't used mdx before so dunno how, it didn't work, so just fyi.

Done!


'1 hours ago' shown as time for some feeds

It should either be 'an hour ago' or '2 hours ago'. This is not for all feeds. Some feeds correctly show 'an hour ago', so it is problem for some corner cases of time.

Fixed in commit 65c25e8


[Merged by Bors] - feat(Analysis/InnerProductSpace/Calculus): Add theorem that abs on ℝ is not differentiable at 0

Adds a theorem which states that the absolute value function on ℝ is not differentiable at zero.

This was suggested by @kmill as an exercise at the recent Hausdorff School on Formal Mathematics and Computer-Assisted Proving in Bonn.
The differentiability of the absolute value function everywhere else is already an immediate consequence of DifferentiableWithinAt.norm.

Ok I think some filter expert could probably shrink this more (maybe with some more API etc) but to me this seems good enough for now, more golfing can happen in a follow up if anyone is motivated to do so.
maintainer merge

Argh I forgot to include @alexjbest as a co-author. I thought the bot would do it automatically since Alex co-authored many of the commits. Sorry 😢

It's ok! I'm just happy to be of assistance 😊


Upgrade OpenRewrite 2.21.1 -> 2.22.0

This PR contains the following updates:

Package
Type
Update
Change

OpenRewrite
import
minor
2.21.1 -> 2.22.0

Release Notes

openrewrite/rewrite-recipe-bom (OpenRewrite)
v2.22.0: 2.22.0
Compare Source
What's Changed

OpenRewrite v8.42.2 https://github.com/openrewrite/rewrite/releases/tag/v8.40.2
rewrite-gradle-plugin v6.27.1 https://github.com/openrewrite/rewrite-gradle-plugin/releases/tag/v6.27.1
rewrite-maven-plugin v5.45.0 https://github.com/openrewrite/rewrite-maven-plugin/releases/tag/v5.45.0

Full Changelog: https://github.com/openrewrite/rewrite-recipe-bom/compare/v2.21.0...v2.22.0

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade OpenRewrite 2.21.1 -> 2.22.0

See:
- https://github.com/openrewrite/rewrite-recipe-bom/releases/tag/v2.22.0
- https://github.com/openrewrite/rewrite-recipe-bom/compare/v2.21.1...v2.22.0


Change authentication-local to check the isVerified field.

If authentication-local checks the isVerified field in the user record (if it exists), then the isVerified hook will not be needed.

https://github.com/feathersjs/feathers/issues/1103


Remove Chocolatey packaging

Too much time was wasted trying to fix this. Once Chocolatey 1.0.16 gets
released, it will be worth trying again.

For anyone who wants to take a look, I have a working Docker project that builds choco and fwup from source in a container. The hard part is integrating this work with existing tooling. https://github.com/RickCarlino/build_choco_and_fwup_docker


Fix bad block detection

Currently every block that fails state transition is marked as invalid. If block comes too late/too early if will be marked as invalid and we won't be able to include it ever.
Not sure about best solution^^

Handle out of order blocks. Ignore blocks with unknown/incorrect previous state.

Check if block.parentRoot matchessigningRoot(state.latestBlockHeader, config.types.BeaconBlockHeader)


Support maxReturnNodes optionIn for trajectory

Summary
If a client requests maxReturnNodes=n then the number of trajectory stations MUST be limited to n results before returning.
The query MUST [else error -402] specify a whole number greater than zero
Implementation Detail

in translateQueryResponse method of com.hashmapinc.tempus.witsml.valve.dot.DotClient class if optionsIn contains a key maxReturnNodes = n then you must only take the first n elements from the trajectoryStations array.
add a validation query in com.hashmapinc.tempus.witsml.server.api.StoreValidator to validateGetFromStore to make sure that the value of maxReturnNodes is > 0, if not return -402

Soap UI Trajectory CRUD::maxReturnNodes tests can be used to verify the functionality requested


No Genre option in file name save. {genre}

Until 1.8.x version, i can save videos with genre name in it.
But since new version 2.x, it's now removed. The title is kinda useless because it can very long text.
I need only genre name so i can see what genre it has in each video. Other options are good.
Oh, genre is same as category btw. {category} is also good.
Maybe new API has no option for it?

From the release notes:

Filename wildcards {game} and {fps} have been removed (not available in new API)


[Improve][Manager] Abstract operations on sink configuration to reduce repetitive cod

Description
Abstract operations on sink configuration to reduce repetitive code.
Referring to the AbstractSourceOperation, suggested creating a new abstract class and naming it AbstractSinkOperator.
InLong Component
InLong Manager
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please assign this issue to me, thx.

Ok, assigned.

Please assign this issue to me, thx.
Assigned. Thanks for your contribution.


New node to join the network

Hi I understand that we have to define the all the the makers, voters node in the storage section of the genesis json file.
Is it possible to create a new and late coming node and join an existing quorum network that is actively running and was previously initialized based on the genesis json file which doesn't contain the storage info of the new node?
Thank you.

@alexpee that was possible in QuorumChain. Please note that it has been replaced with Istanbul BFT in Quorum v2.


Update Containerfile which tries to use the nonexisting requirements.txt file

Description
This PR fixes the side effect of https://github.com/road-core/service/pull/66 where the requirements.txt file is deleted.
Type of change

[ ] Refactor
[ ] New feature
[x] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[ ] CI configuration change
[ ] Konflux configuration change

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing
For testing, just try to build the app image and validate if it works. I was able to test this in the fork project ansible/ansible-chatbot-service

Figure the same just now :sweat: By the way, this should work as well: https://github.com/road-core/service/pull/78
@tisnik feel free to close mine if you think this one is the appropriate approach

@mabulgu, #78 got merged


Retry operators

As per contribution guidelines, I'm opening an issue for discussion regarding the implementation of Retry.
Per Documentation:

The Retry operator responds to an onError notification from the source Observable by not passing that call through to its observers, but instead by resubscribing to the source Observable and giving it another opportunity to complete its sequence without error. Retry always passes onNext notifications through to its observers, even from sequences that terminate with an error, so this can cause duplicate emissions (as shown in the diagram above)

I have an implementation of Retry on my for, as linked below
reference: http://reactivex.io/documentation/operators/retry.html

Implemented in https://github.com/ReactiveX/RxGo/pull/215 as part of the v2


Etcdv3 pr686

As requested by  @shashidharatd
this merge request is based on https://github.com/kubernetes-incubator/external-dns/pull/686

@jremond, you need to sign Linux foundation CLA

@shashidharatd if https://github.com/kubernetes-incubator/external-dns/pull/686 works, you can decline this PR and move forward with yours