[CXF-7562] correction for truncated flag

Hi, thanks for this PR, it looks good, can you please sync it with the current master, also note a minor typo in the new function name, thanks.


model_downloader: merge different precision variants of the same model

This eliminates a lot of information duplication within the config file,
as the different variants are the same in everything but the files.

CC @kchechil @eizamaliev
I was going to combine this change with a more thorough refactoring of the config file, but then I realized that I don't have to, so I submitted it early.

Jenkins please retry a build

@zmaslova @snosov1 I suspect there are no comments so merging this.


Develop

Create tournament participants with token balance taken from blockchain.

Pull Request Test Coverage Report for Build 905

52 of 55 (94.55%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 85.671%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tradingdb/relationaldb/models.py
3
6
50.0%

Totals

Change from base Build 883:
0.1%

Covered Lines:
2858

Relevant Lines:
3336

💛  - Coveralls


Error: spawn dev_appserver.py ENOENT

I got some error logs when I have tried to run the site,
My environment is Ubuntu 14.04:
root@user:/home/user/WebFundamentals/# gulp
......
......
[05:20:04] Finished 'pretty-print-gae-info' after 1.66 ms
[05:20:04] Finished 'start-gae-dev-server' after 42 ms
[05:20:04] Starting 'dev-watch-tasks'...
[05:20:14] Finished 'dev-watch-tasks' after 9.87 s
[05:20:14] Finished 'develop' after 10 min
[05:20:14] Starting 'default'...
[05:20:14] Finished 'default' after 27 μs
events.js:142
throw er; // Unhandled 'error' event
^
Error: spawn dev_appserver.py ENOENT
at exports._errnoException (util.js:856:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:179:32)
at onErrorNT (internal/child_process.js:345:16)
at nextTickCallbackWith2Args (node.js:455:9)
at process._tickCallback (node.js:369:17)

I also seeing the same error, but on a Mac.

Hi, Actually logs were appeared because I have missed to following a step before Running The Site. I am hurrying to run gulp command after I have finished the 9th step of Installation Guide npm install. It mean I did not through. But generally, if we develop web app using npm we usually running gulp or grunt serve after npm install right?
I have missed to following:
10th Get the App Engine SDK and unzip into the google_appengine folder inside the project root. Add it to your path accordingly (in bash, $ PATH=./google_appengine:$PATH)
So, Probably It does't caused by WebFundamentals Platform. Now I can run the site on my localhost!


Multiple lexers inside StyleNeeded

I am trying to use multiple lexers inside StyleNeeded, but as soon as i use the second lexer the styles gets overriden from the previous one.
What i basically want is to call Cpp lexer, then call Html lexer for specific ranges, then do a custom lexer and lastly do Cpp lexer again for special ranges. Its the same process like doing PHP with HTML - but with more layers. Unfortunatly the Styles[] array seems to be persistent and the indices for each language always start at zero (Default) as well, meaning that Style.Cpp.Default is the same as Style.Html.Default.
Is there a way around that? Can i add a style offset of some sort - so it gets unique for each language?
Currently i have this, which only works for one lexer and one custom lexer :-(
Font editorFont = new Font(FontFamily.GenericMonospace, 14.0f, FontStyle.Regular);
editor.StyleResetDefault();
editor.Styles[Style.Default].Font = editorFont.Name;
editor.Styles[Style.Default].Size = (int)editorFont.SizeInPoints;
editor.StyleClearAll();

// Use container lexer for calling StyleNeeded
editor.Lexer = Lexer.Container;

editor.StyleNeeded += (s, e) =>
{
	Scintilla thisEditor = (Scintilla)s;
	EditorState editorState = (EditorState)thisEditor.Parent.Tag;

	int startPos = thisEditor.GetEndStyled();
	int line = thisEditor.LineFromPosition(startPos);
	startPos = thisEditor.Lines[line].Position;
	int endPos = e.Position;

	// Highlight C++
	thisEditor.Lexer = Lexer.Cpp;
	thisEditor.Styles[Style.Cpp.Default].ForeColor = Color.Black;  // This overwrites the default style :-(
	thisEditor.Styles[Style.Cpp.Identifier].ForeColor = Color.Blue;
	thisEditor.Styles[Style.Cpp.String].ForeColor = Color.Green;
	thisEditor.Colorize(startPos, endPos);

	// Find intersecting comment blocks
	Match[] commentMatches = GetCommentMatches(thisEditor, startPos, endPos);
	foreach (Match commentMatch in commentMatches)
	{
		int matchStart = commentMatch.Index;
		int matchEnd = matchStart + (commentMatch.Length - 1);

		// Highlight Html
		thisEditor.Lexer = Lexer.Html;
		thisEditor.Styles[Style.Html.Default].ForeColor = Color.Silver; // This overwrites the previous C++ default style :-(
		thisEditor.Styles[Style.Html.Tag].ForeColor = Color.Blue;
		thisEditor.Colorize(matchStart, matchEnd);

		// Highlight Doxygen
		thisEditor.Lexer = Lexer.Null;
		thisEditor.Styles[30].ForeColor = Color.Red; // 30 Seems to be a good index for a custom style
		thisEditor.Styles[31].ForeColor = Color.Yellow;
		CustomHighlightDoxygen(thisEditor, matchStart, matchEnd);
	}

	// Switch back to container lexer, so StyleNeeded is called again
	thisEditor.Lexer = Lexer.Container;
};

I dont want to write the lexers myself, because each lexer by itself works beutifully.

You ever get this working?  I'm also curious to see how your GetCommentMatches() works.

You ever get this working? I'm also curious to see how your GetCommentMatches() works.

Initially i was using pure regex to detect comments, to detect the comment blocks which may contain doxygen commands.
But now i am fully switched to a 100% custom lexing & parsing of C/C++/Doxygen/Html - which means i just use "Container" for everything and never use any integrating lexers nor styles.
Regarding your actual question, detecting the comment matches are easy - you need to find all comment blocks, on every change and save the regions in a list or something. GetCommentMatches() iterates from all blocks and does simple start/length lookup to search for intersections. Thats it.


Pipeline status not updated when runAfter is used with a condition`

Say we have a pipeline with two pipeline tasks A and B. A is a conditional task that only runs if a condition is true while B is dependent on A using either from or runAfter.
Expected Behavior
If task A is skipped due to a condition failure, task B should not run. The pipeline status should be set to success since both A and B are skipped
Actual Behavior
Task A is skipped due to condition failure and Task B is also skipped due to it being dependent on a condition failure. But the pipeline status is never updated from Running to Successful
Steps to Reproduce the Problem

Run this pipeline: https://gist.github.com/dibyom/761199a1af98f435951fe1c8c2c8a08c
Run kubectl get prs -o yaml to see the status is not updated properly even after the first task is skipped.

Additional Info
/kind bug
/assign

@dibyom adding this to 0.6 milestone as it will ship conditionals :angel:


Fixing issue with recursive dependency on VisualFocus

This commit fixes the following bug: if appOutlineStyle is set to a non-null value in the AppEnvironment (after Aria Templates is loaded but before any template is loaded), and if the packaging of Aria Templates is such that aria.utils.VisualFocus is available synchronously when aria.utils.environment.VisualFocus is loaded, aria.utils.VisualFocus does not get a correct reference to aria.utils.environment.VisualFocus when it is loaded, leading to the undefined is not a function exception when executing ariaUtilsEnvironmentVisualFocus.getAppOutlineStyle() in https://github.com/ariatemplates/ariatemplates/blob/master/src/aria/utils/VisualFocus.js#L39

:white_check_mark: 22278a7 is fine for me.


Upgrading gem from 0.5.5669 -> 0.5.5848  produces 58 new errors

Input
Upgrade Sorbet gem from 0.5.5669 -> 0.5.5669 in a Rails project. All other gems are up-to-date. Full gemfile-lock diff below:

Please note
Although Sorbet-rails looks like it was simultaneously upgraded I actually did that prior to Sorbet (and should have checked it in to make the diff clearer) so the errors are not coming from Sorbet-Rails. After it was upgraded I ran srb tc and everything ran with no errors. After upgrading the Sorbet gems, srb tc began producing the errors below.
Also, and this may not be relevant but just so as you know, I also upgraded OSX in between the two gemsets. So the original RBI file generation was done on Mojave and the project is now running on Catalina. The errors also occur when running the srb tc on an image on CircleCI as well though.
Observed output
Running  srb tc results in ~ 58 errors where none were previously present. About 5 of them are actually related to my own codebase but the rest all seem to be dealing with some sort of gem issues.
FWIW the errors that are connected to my codebase picked up the fact that a file that was typed: true and which had sig methods did not in fact have the extend T::Sig line at the top of it.
lib/analytics/slack.rb:36: Method sig does not exist on T.class_of(T.class_of(Analytics::Slack)) https://srb.help/7003
    36 |      sig{params( user: T.nilable(User),
    37 |                  event_name: String,
    38 |                  props: Hash).returns(Hash)}

The error above was welcome as I'd forgotten to exten T::Sig but the errors below are harder to know what to do with:
Below are a selection of some of the other errors:
sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:198: Duplicate type member Elem https://srb.help/4011
     198 |    Elem = type_member(fixed: T.untyped)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L1920: Also defined here
    1920 |  Elem = type_member(fixed: T.untyped)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:318: Method Gem::BasicSpecification.default_specifications_dir redefined without matching argument count. Expected: 1, got: 0 https://srb.help/4010
     318 |    def self.default_specifications_dir; end
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2044: Previous definition
    2044 |  def self.default_specifications_dir(*args, &block); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:710: Method Gem::DependencyInstaller#available_set_for redefined without matching argument count. Expected: 1, got: 2 https://srb.help/4010
     710 |    def available_set_for(dep_or_name, version); end
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2269: Previous definition
    2269 |  def available_set_for(*args, &block); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:720: Method Gem::DependencyInstaller#find_gems_with_sources redefined without matching argument count. Expected: 1, got: 2 https://srb.help/4010
     720 |    def find_gems_with_sources(dep, best_only = T.unsafe(nil)); end
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2279: Previous definition
    2279 |  def find_gems_with_sources(*args, &block); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:722: Method Gem::DependencyInstaller#find_spec_by_name_and_version redefined without matching argument count. Expected: 1, got: 3 https://srb.help/4010
     722 |    def find_spec_by_name_and_version(gem_name, version = T.unsafe(nil), prerelease = T.unsafe(nil)); end
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2281: Previous definition
    2281 |  def find_spec_by_name_and_version(*args, &block); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:778: Duplicate type member Elem https://srb.help/4011
     778 |    Elem = type_member(fixed: T.untyped)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2302: Also defined here
    2302 |  Elem = type_member(fixed: T.untyped)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:858: Method Gem::Ext::Builder#build_error redefined without matching argument count. Expected: 2, got: 3 https://srb.help/4010
     858 |      def build_error(build_dir, output, backtrace = T.unsafe(nil)); end
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2351: Previous definition
    2351 |  def build_error(output, backtrace=T.unsafe(nil)); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:858: Method Gem::Ext::Builder#build_error<defaultArg>1 redefined without matching argument count. Expected: 2, got: 3 https://srb.help/4010
     858 |      def build_error(build_dir, output, backtrace = T.unsafe(nil)); end
                                                               ^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2351: Previous definition
    2351 |  def build_error(output, backtrace=T.unsafe(nil)); end
                                              ^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:1047: Method Gem::Installer#unpack redefined with argument directory as a non-splat argument https://srb.help/4010
    1047 |    def unpack(directory); end
              ^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2473: The corresponding argument args in the previous definition was a splat argument
    2473 |  def unpack(*args, &block); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:1049: Method Gem::Installer#verify_gem_home redefined without matching argument count. Expected: 0, got: 1 https://srb.help/4010
    1049 |    def verify_gem_home(unpack = T.unsafe(nil)); end
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2475: Previous definition
    2475 |  def verify_gem_home(); end
            ^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:1458: Duplicate type member Elem https://srb.help/4011
    1458 |      Elem = type_member(fixed: T.untyped)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L1682: Also defined here
    1682 |  Elem = type_member(fixed: T.untyped)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:1657: Method Gem::RemoteFetcher#fetch_size redefined with argument uri as a non-splat argument https://srb.help/4010
    1657 |    def fetch_size(uri); end
              ^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2578: The corresponding argument args in the previous definition was a splat argument
    2578 |  def fetch_size(*args, &block); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:2251: Method Gem::Resolver::ActivationRequest#initialize redefined without matching argument count. Expected: 2, got: 3 https://srb.help/4010
    2251 |      def initialize(spec, request, others_possible = T.unsafe(nil)); end
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L2971: Previous definition
    2971 |  def initialize(spec, request); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:2501: Duplicate type member Elem https://srb.help/4011
    2501 |        Elem = type_member(fixed: T.untyped)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L3193: Also defined here
    3193 |  Elem = type_member(fixed: T.untyped)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:2905: Duplicate type member Elem https://srb.help/4011
    2905 |      Elem = type_member(fixed: T.untyped)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L3552: Also defined here
    3552 |  Elem = type_member(fixed: T.untyped)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:3445: Duplicate type member Elem https://srb.help/4011
    3445 |    Elem = type_member(fixed: T.untyped)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L3851: Also defined here
    3851 |  Elem = type_member(:out)
            ^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:3812: Method Gem::Specification#rubyforge_project= redefined with argument rubyforge_project as a non-splat argument https://srb.help/4010
    3812 |    def rubyforge_project=(rubyforge_project); end
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L1154: The corresponding argument args in the previous definition was a splat argument
    1154 |  def rubyforge_project=(*args, &block); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/sorbet-typed/lib/ruby/all/gem.rbi:4010: Method Gem::StreamUI#debug redefined with argument args as a splat argument https://srb.help/4010
    4010 |    def debug(*args, &block); end
              ^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/rubygems.rbi#L1820: The corresponding argument statement in the previous definition was not a splat argument
    1820 |  def debug(statement); end
            ^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/hidden-definitions/hidden.rbi:13382: Method Module#undef_method redefined with argument _ as a splat argument https://srb.help/4010
       13382 |  def undef_method(*_); end
                ^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/core/module.rbi#L1581: The corresponding argument arg0 in the previous definition was not a splat argument
    1581 |  def undef_method(arg0); end
            ^^^^^^^^^^^^^^^^^^^^^^

sorbet/rbi/hidden-definitions/hidden.rbi:21853: Method Psych.load_file redefined without matching argument count. Expected: 2, got: 1 https://srb.help/4010
       21853 |  def self.load_file(path); end
                ^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/514434c85b65104cbc70bbba24ab110ad0aae5fa/rbi/stdlib/psych.rbi#L501: Previous definition
     501 |  def self.load_file(filename, fallback: false); end
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Expected behavior
Either no errors or if the errors require re-running any config scripts then an instruction on what to do

One further quirk I've noticed - after rolling the gemfile back to the previous version of the Sorbet gems, the errors still persisted. I ran bundle clean --force to get rid of the newer gem versions and the errors went away.
I then thought that maybe the errors were happening because for some reason sorbet was picking up both versions of the gem. I reinstalled the updated versions of Sorbet and ran bundle clean --force again to clean out the old versions but the errors persisted. So it was the presence of the new gems that was causing it (and not simply having both gems present).
I always just used the command srb tc to run typechecking but I guess you need to run bundle exec srb tc if you want to tie it to the particular version. It may be worth makign this clearer?

Sorbet introduces new errors regularly as bugs are fixed, types for the standard library are improved, and new features are added.
Sorbet does not have any stability guarantees. Upgrades nearly always come with new errors.
If you believe there is a bug in Sorbet as a result of an upgrade, feel free to open individual issues with concrete steps to reproduce the issue in isolation.

I hit the same problem on another upgrade but then discovered that it could be fixed by re-running sorbet init. I guess maybe it needed to reload the RBI files for various updated gems?


[Compute] New API for Disk Attachments

👋
Currently to attach a Data Disk to a Virtual Machine requires submitting this as a part of the Virtual Machine payload (e.g. a appending to the list of data disks).
Whilst this works when you want to define the Virtual Machine definition in a single payload, unfortunately this makes it challenging when breaking this apart into multiple payloads to provide a better user experience - which means that we've had to fake this via calling the API sequentially using locks to ensure multiple disk attachments happen sequentially.
This functionality already exists for the DevTest Labs Service - which has an endpoint to Attach/Detach Data Disks - in short we're looking for the same thing for regular Virtual Machines.
As such would it be possible to introduce an API for managing Data Disk Attachments, similar to the ones provided by AWS and Google?
Thanks!

Thanks for the feedback. I have moved this over to UserVoice feedback channel so that it is captured along with the other feature requests. Please up vote this item: https://feedback.azure.com/forums/216843-virtual-machines/suggestions/39952540-api-for-disk-attachments-that-does-not-require-upd

I'd also like to up vote this item. Unfortunately the link to the forum in the comment above is broken.

https://feedback.azure.com/d365community/idea/97a67488-bf25-ec11-b6e6-000d3a4f0f1c
Link for anyone wanting to upvote.


fix: swap makes vms out of disk space report false positive

MiqExpressions convert nil to a floating point number. In ruby this is a 0.
nil <= 20
nil.to_f ==> 0
0 <= 20 == true

Swap partitions have a nil free_space / free_space_percentage.
so free_space_percentage <= 20 shows true for swap
So any vm with a swap partition will say it is low on disk space.
Fix is to ask if used space is over 80%
nil >= 80
nil.to_f ==> 0
0 >= 80 == false

Now, there are no false positives for running out of disk space.
https://bugzilla.redhat.com/show_bug.cgi?id=1686281

Hammer backport details:
$ git log -1
commit 331132f8c8821c8d1042047aa36f6c63ea9241b6
Author: Jason Frey<EMAIL_ADDRESS>Date:   Wed Apr 3 15:32:44 2019 -0400

    Merge pull request #18623 from kbrock/disk_usage_report

    fix: swap makes vms out of disk space report false positive
    (cherry picked from commit 629474ca44471d34b19f0ec4682eaece5f79394e)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1696420


Add support for iDEAL Instant Component

Describe the bug
When I want to pay via iDEAL on Android, I have the following error : "Unsupported payment method ideal".
On iOS, the same code works.
Merchant account : supports the new iDEAL configuration (mandatory for April 1st 2025)
Environment : Environment.test
Country code : NL
To Reproduce
I reproduced the bug using the exemple project.
In InstantAdvancedComponentScreen, you can replace klarna with ideal.
Then I click on iDEAL button : on Android, an error is thrown by the SDK.
Expected behavior
On iOS, iDEAL's webview is displayed as expected.
Screenshots
Android (Pixel 4 API 34 emulator) :

iOS (iPhone 15 Pro Max - iOS 17.5 simulator):

@Robert-SD Do I have to use Drop-In to add iDEAL to the payment method list even if I want to use the old flow ? Instant Component doesn't seem to support the old flow too for Android. I want to support iDEAL for January 2025.
Thanks a lot for your help.

Hello @MickaelTitecaTAO,
we've looked into the code and opened a PR to add the new iDAEL flow to our instant component on Android. When merged, it will be part of the new release in the beginning of 2025. Our current release does not support iDEAL as a component whether it is the new or old flow. Drop-In is therefore the only way and supports the old and new flow.
You could also try out the new solution by using our Github repo directly. But please keep in mind that this is not recommended for your production app because we actively work on the branches.

Hi @Robert-SD,
I've tested commit 90b81b2646451104312fda6c3ed0be6acb61973c and Instant Component works on Android on my project. My merchant account has the old flow. I had to update Kotlin version to 2.0.0 in order to run the app.
I'm waiting for a stable release to use it.
Thanks a lot for putting iDEAL support as one of your top priorities for the next release !

Awesome, thanks for testing it! 👍
Indeed, the Kotlin version as well as Gradle will need an updated because we also updated to the latest native Android SDK v5.8.0. Kotlin v1.8.22 will be the new minimum version together with AGP 8.1 and Gradle v8.


Create falco-0-30-0.md

Add falco-0-30-0.md Chinese document, easy to read.
Signed-off-by: wanhui<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: gogolovefish
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval by writing /assign @jasondellaluce in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Please, use the template for the PR and uncomment the relevant fields. That will help classify the pull request.
/kind content
/area blog

@mstemm @vjjmiras  hello , i have changed the  comment format ,please +1,thanks


Name, motivation and clear goal

We need:

A catchy name for that project. 😉

"Catchyness" is one thing - it needs to have nice name / shortcut for a command line interface.

Motivation and rationale - why we created it in the first place?

Section in README.md and maybe longer explanation on the wiki if needed.

Clear goal - with what we want to help via this tool?

Section in README.md and maybe longer explanation on the wiki if needed.

Possible options:

Name from a particular type of clouds - I really like this idea, that I've picked it for naming releases / milestones.
Weather related: Weather, Pressure, Atmospheric Front, Nephology (science about clouds).
Entities related: Thumper (in meaning of giant), Colossus.
Deities related:

Slavic: Perun, Triglav, Dazhbog (deities of sky, wind, storm).
Nordic: Njord (deity of wind).

perun +1

It looks like there is a competition - https://github.com/search?utf8=✓&q=perun&type= (most known would be https://perun.io).

Maybe something like CFPerun or PerunCloud?


Homework #3

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
I completed homework #3 for submission.
What did you learn from the changes you have made?
I learned how to aggregate, order and group data using SQLite
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Some of the queries required a lot of trial and error but eventually I got the answer I needed.
How were these changes tested?
I used SQLite to test the changes.
A reference to a related issue in your repository (if applicable)
Checklist

[ X] I can confirm that my changes are working as intended

HW3: 7/8 AGGREGATE Q2 missing ORDER BY
Added ORDER BY


Mikrotik RouterOS script syntax highlighting

Language
RouterOS has a scripting and console language for their routers, switches and access points.
Even though Mikrotik is still kind of a niche player in the networking hardware, they have very large market shares in certain sectors and countries. VScode has two plugins providing this, and they have been downloaded ~20k times each
Additional resources
Homepage mikrotik.com
To download a virtual router to run in a virtual machine, visit their download page and scroll down to "Cloud Hosted Router".
Plugin for VScode that adds syntax highlighting there

+1
My personal reason why I want MikroTik Scripting language support is highlighting in Obsidian 😉
Maybe this 3rd party code can help with this https://github.com/gmougeolle/prismjs-mikrotik

Same here, want it in obsidian. There are at least two working addons for vcode making good syntax highlighting for MikroTik Script

+1 for syntax highlighting in Obsidian 🤩

Friendly bump <3

+1
Me too, I'd love to have mikrotik syntax highlighting in obsidian 🙇

+1
I'd love to have mikrotik syntax highlighting in obsidian.

+1
I'd love to see it in Obsidian.

+1

+1 it would definetely be awesome!

+1

+1

Would be cool if this was added +1


init cargo

Fixes
Fixes WB-NNNN
Fixes #NNNN
Description
What does the PR do?
Testing
How was this PR tested?
Checklist

[ ] Include reference to internal ticket "Fixes WB-NNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Ensure PR title compliance with the conventional commits standards

🤖 Generated by Copilot at ec81dd6

Table of Contents
A blank line for clarity
Autumn leaves falling

closing as stale


Project is abandoned.

No merged pull requests for a year, no responses to issues. If it's abandoned, are you looking for maintainers?

closed

@jamiesonbecker Should we fork? How does miniS3 sound?

It's got a nice API, but my feeling is that the API is the real value - not the underlying code, since there's still a lot of security concerns (or outright gaping holes!) in this code.
It'd probably be safer/easier/faster/cleaner and last longer if miniS3 was a nice wrapper on top of boto that just provides something similar to the tinyS3 API.
And, yep, good name. :+1:

@jamiesonbecker I like safer/easier/faster/cleaner! I'll get started on this tonight :smile:

Hi guys, I apologize again about this project abandonment.
Let's talk over email about changing maintainers.

Contact me at<EMAIL_ADDRESS>

I'm at my first dot last @ userify..

could somebody please update this issue with the current maintainer status?

@shlomiatar can you accept pull request #56 just updating the README? Thanks!


Scrolling stutters over notebook outputs

Open my-endgame.github-issues in VS Code stable and VS Code insiders
Run all cells in each
Scroll quickly up and down through the notebook
:bug: in VS Code insiders scrolling over notebook outputs is not smooth anymore; scrolling appears to stutter and halt over notebook outputs

I took performance profiles in stable and insiders but devtools won't let me save them unfortunately. Nothing stood out though in terms of memory usage or a particular function call increasing in percentage across profiles. LMK if I can provide more info

Is this with a trackpad? I might be seeing this. Could be a regression from https://github.com/microsoft/vscode/issues/174913

I've the same issue with a trackpad on Windows.


Enable no-wgl and include EGL DLLc in cargo packager

While cargo run on Windows works fine, it will need several DLLs to make the actual binary work.
In the final package, following DLL sources should be included:
https://github.com/servo/servo/blob/main/python/servo/build_commands.py#L365

Its mostly libgstreamer right or are there any other deps that are needed?

We will use cargo packager in the future. Probably worth looking into it to see if it has option to include custom dlls.

Ill check.  My first go to was to just add a build.rs file that uses pkg-config but I am sure that cargo packager has its own stuff for that.

Rename the issue because it's WGL by default now. But since ANGLE is more stable, it's still better to enable it when we build with cargo packager in the future.

I'll handle this in https://github.com/versotile-org/verso/issues/85


(Goodreads) icons-request

A Goodreads icon would be good.
(I'm sorry I don't know how to add the "icon-request" label)

No worries, just added ;-)
Like the idea too!


[WIP] Add remaining data for the SK launch

#256
(Sorry - forgot to create only a draft pull request...- this is WIP yet)

I don't know slovak, but other than that it looks good! 😄Could you add an image to be used on social media when people share the site? Should be located in /static/sk/social1.png

Ready to merge? :)

Ready to merge? :)

no no, it's an early draft yet, WIP :-) I will need to properly go through all the views in the app and I expect there are lot of mistakes and inaccurate translations. Also the privacy policy needs some tweaking, etc... So please ignore this PR until i remove the [WIP] prefix (i forgot to create this as a draft PR :-| ). Thank you!

Fixed your merge conflict

I'm overwriting @adriaanvanrossum review and pressing the merge button. The review has no changes requested anymore but Github didn't pick that up.

Perfect


Use unique file name for TTX dump

Description
When FB is ran in parallel on the same file, the runs step on each other because the TTX dumps use the same file name.
This change makes the XML file of each run have a unique random name.
To Do

[ ] update CHANGELOG.md
[ ] wait for all checks to pass
[x] request a review

@felipesanches this is ready for review

thanks!


LibCore: Fix compilation of EventLoopImplementationWindows.cpp

Replaces the use of a variable length array with a vector. Adds maybe_unused to unimplemented function signatures to allow for compilation.

Actually ensure capacity seems better here as this doesn't work. The template argument would need to be known at compile time. Or I can overallocate to whatever MAXIMUM_WAIT_OBJECTS is. Which one would be better?

What do you mean it doesn't work? Setting the inline capacity to event_counts? That's expected, we are not creating a VLA anymore.
It's probably fine to put 64 ints on the stack. But you could also set this value to something smaller and still keep the ensure_capacity, Vector will move from its inline capacity to an external memory location if needed.

Yeah, for a second I though event_count was const or constexpr, but it is runtime. I think I'll just leave it with MAXIMUM_WAIT_OBJECTS. Though those are pointers not ints. So it will  be 64 pointers.


chore: enable migration tests again

PR has been merged. https://github.com/lnbits/lnbits-extensions/pull/300

closes #2414


[DOC] Immutable indices setting is not documented

What do you want to do?

[ ] Request a change to existing documentation
[X] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request.
Immutable indices are needed for security products that require a guarantee of integrity. We provide integrity for plaintext logs with checksums. But in a database, it is not always the case. And we need to ensure logs are not tampered.
I created a forum topic about the issue and a forum user with a nickname pablo, managed to find that it is already there. It is also possible to see in the full list of all settings provided in another issue.
How it works, default values, relationship with indice management policies -such as "would a DELETE action be applied to an immutable index?"- needs to be clarified.
All versions are affected from the lack of documentation.
What other resources are available?
Nothing but the source code itself:
https://github.com/opensearch-project/security/search?q=SECURITY_COMPLIANCE_IMMUTABLE_INDICES

Thanks for the issue submission @zbalkan.

@cwillum - Does this look like something you can take on?

@here Sure.

@scrawfor99 - Is this a setting that was missed? https://opensearch.org/docs/latest/install-and-configure/configuring-opensearch/security-settings/
https://github.com/search?q=repo%3Aopensearch-project%2Fsecurity+SECURITY_COMPLIANCE_IMMUTABLE_INDICES&type=code

Hi @hdhalter, yeah seems like this was missed when I had gone through the settings before. My bad! Thanks for the catch. We should add this as well. John should be able to I suspect.

Hi @hdhalter, yeah seems like this was missed when I had gone through the settings before. My bad! Thanks for the catch. We should add this as well. John should be able to I suspect.

Thanks, can you please add it to his queue?

@hdhalter I can take this one


asyncronous execution

This change adds a callback parameter to the generateMarkdown function. This hack allows the module to execute in a nominally asyncronous manner.

Thanks @isysd!
@HowardWolosky or @emackey any thoughts on this?

Could this be a promise instead of a callback?
Also, clearly the callback is being used synchronously here.  Are there plans to make this function async in the future?  What's the advantage of making it look async-ish now, if it's not?

Agreed with @emackey on this. I'm not sure what advantage we're getting with this specific change given that it's not being consumed asynchronously.

@isysd do you think we should close this one?  Perhaps revisit later if needed?

Closing for now since there is no update.
Feel free to pick this up again if it becomes a priority.


index.d.ts provides incorrect typings for connect

As for now, the following typing is provided:
export declare function connect<TStateProps, TDispatchProps, TOwnProps>(
    mapStateToProps?: MapStateToProps<TStateProps, TOwnProps> | MapStateToPropsFactory<TStateProps, TOwnProps>,
    mapDispatchToProps?: MapDispatchToProps<TDispatchProps, TOwnProps> | MapDispatchToPropsFactory<TDispatchProps, TOwnProps>,
    mergeProps?: MergeProps<TStateProps, TDispatchProps, TOwnProps>,
    options?: Options
): ComponentDecorator<TStateProps & TDispatchProps, TOwnProps>;

Problem lies in the following type:
type MapDispatchToProps<TDispatchProps, TOwnProps> =
    MapDispatchToPropsFunction<TDispatchProps, TOwnProps> | MapDispatchToPropsObject;

This makes it impossible for typescript compiler to deduce TDispatchProps from the second parameter of connect function.
Essentially, we need to do the following:

remove MapDispatchToPropsObject and MapDispatchToPropsFunction
change MapDispatchToProps type to the following:

type MapDispatchToProps<TDispatchProps, TOwnProps> = TDispatchProps;

Please note that it only will automatically deduce types if component has no TOwnProps. That will change when either typescript/#4183 or typescript/#13470 will land.

These typings aren't on this repo. Please open this on the repo where you obtained them from.


Fix build on latest nightly

I'm not at all sure this is how this should be fixed, but on the off chance that this is useful to you, I thought I'd make a pr out if my local fixes anyway. Feel free to ignore it, though.
Fixes #103

Thanks for the PR. I ended up going a different route for this though. I opted not to expose more generics to the end user and to simplify the SockOpt trait.


Databases created using restore dialog should show in OE without needing to refresh

SQL Operations Studio Version: 0.24.1

Steps to Reproduce:

Use restore dialog to create a new database

Expected: Database is shown in OE tree once restore task completes
Actual: OE tree needs to be manually refreshed to show new DB.

@abist I'll move this to March milestone, but if we can get it in for Feb that would also be great.  Thanks!

@kburtram I have a PR out for this. If we get time to go over it, we can get it in for Feb or March is fine.


Error in plot.new() : the base graphics system is not registered

Okay, another weird graphics bug. Certain unrelated julia functions will trigger the following error when an R graphics function is subsequently called
Error in plot.new() : the base graphics system is not registered

The following reliably triggers it on the second invocation of plot(1:10) on my system (OS X, v0.3.7) when run at the REPL (though seems fine when in a script, or via include).
using RCall

rprint("png('/tmp/aa.png')")
rprint("plot(1:10)")
rprint("dev.off()")

io = IOBuffer()
write(io,"1,2,3")
seek(io,0)
readall(io) # this seems to be the trigger

rprint("png('/tmp/aa.png')")
rprint("plot(1:10)")
rprint("dev.off()")

Hmm, if I insert a rprint("gc()") call in the middle, that seems to fix things. Does this mean it's a memory leak somewhere?

The code runs well on my OS X 10.10.3 with Julia 0.3.7, RCall 0.1.2.

I get the message at random. Not sure how to reproduce it. Your code runs well some time but not another.
Error in plot.new() : the base graphics system is not registered

It is related to the variable baseRegisterIndex in C
unsafe_load(cglobal((:baseRegisterIndex,RCall.libR),Int32))

The normal value of baseRegisterIndex should be >=0. However, due to unknown reason, the value was changed to -1, which lead to the error.
A way to change it to -1 that I know of is
ccall((:Rf_KillAllDevices,RCall.libR), Int32, ())

I think the problem may have been that the finalizer was being called prematurely. Hopefully this works correctly now. Please reopen if there are further problems.

Good catch. It should be the reason. As Rf_endEmbeddedR calls Rf_KillAllDevices which sets baseRegisterIndex to -1.

Yeah, I guess the crazy nature of the bug should have been a clue it was gc related.


Increase default checks timeout to 5 seconds for "instant" checks

High-level description
Attempts to reduce the relatively common occurrence of timeouts in checks that have no real way to time out by increasing the timeout to 5 seconds in case load is preventing a cluster from being able to run the check in the normal 1 second.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS-53742

Checklist for all PRs

[ X] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[X] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[X] Read the DC/OS contributing guidelines
[X] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot label ready for review


WIP: Migrating freehand tool to v3, and working on API utility for more complicated tools.

Notes: I left 'isAwaitingMouseUp' with its current name for now, as there are a lot of dev branches, I'll review and tidy up the eventDispatcher when we have all the tools in.
J

Preview URL: https://deploy-preview-488--cornerstone-tools.netlify.com/examples/00-test/

I'm wondering if it's possible for it to work without adding/removing event listeners
I'm a little torn, because I think it makes it more efficient
Is the logic/use case just too complex to use callbacks?
Or is it more about locking in to that event loop once you've started to use the tool?
I'm going to take break from reviewing this.
I really, really, really like the end product, but I think there are some points at least discussing before merging.

Hey @dannyrb, I know we already talked on slack, but airing it more publicly here. The main reason of adding and removing listeners is to lock into the event loop once we start using it. This is actually very similar to manipulators that other tools import, which do the same thing, only here I do this explicitly in the class.
There are also some things that could probably use shared functionality (e.g. the textBox), but a lot of the functionality is too bespoke to generalise. I'm happy for any ideas if anyone has them! :)

P.S. All comments here are now addressed and pushed.

@JamesAPetts just a heads up, I don't need anything else from you on this PR. I'm tidying up the dispatchers and working in some changes to support the flexibility your tool needs. After I'm at a good point, I'll address merge conflicts and pull these changes in.
Didn't want to leave you hanging (: Thanks again for all of your hard work 👍

Alright, @JamesAPetts. I've cleaned up everything that was blocking me from taking a harder look at this. I think our biggest hurdle for a merge is going to be the best way to handle the mouseToolEventDispatcher.js changes. The ones I see are:

👍 Adding a callback to mouseMove
🤷♂ Modify mouseDown to identify target/active tool in a new way:

Tools with or without data
Tools with a mouseDownCallback
Tools w/ an alternative way to indicate they're a validTarget other than pointNearTool

🤷♂ Modify mouseDown to call a tool's mouseDownCallback instead of default logic
🤖  Add implementation for mouseDoubleClick
🤖  setIsAwaitingMouseUp

🤷♂ - The problem/solution we need to focus on
👍 - I agree, and we should keep this implementation
🤖 - I agree, but have provided an alternative implementation in a different PR. Curious to hear your thoughts.
I'll create a separate PR to add the 👍, and also to address 🤷♂ . We can discuss 🤖 further if you'd like. After we agree on these details, we can merge this PR, but blow out the dispatcher changes.

@dannyrb , thanks for taking the time to go through the mouseToolEventDispatcher changes thoroughly. I look forward to seeing your PR. As for your 🤖 comments, do you mean these will be present in your new PR, or there are already alternatives out in the wild I can look at?

Closing as will rebuild in updated eventDispatch scheme.


CI failures/issues on current master

https://jenkins.konduit.ai/blue/organizations/jenkins/konduitai%2Fdeeplearning4j/detail/PR-284/2/pipeline/
linux-ppc64le-cpu
[2020-03-03T22:54:01.306Z] Tests in error:
[2020-03-03T22:54:01.306Z]   testTransform[0: backend(org.nd4j.linalg.cpu.nativecpu.CpuBackend)={1}](org.nd4j.linalg.dataset.NormalizerStandardizeLabelsTest): test timed out after 30000 milliseconds

linux-x86_64-cuda-10.0

[2020-03-03T22:51:56.647Z]   EmbeddingLayerTest.testEmbeddingWeightInit:774 expected: org.nd4j.linalg.jcublas.JCublasNDArray<[[    0.0324,   -0.0465,   -0.0584,  ...   -0.2255,    0.1262,    0.0089]]> but was: org.nd4j.linalg.jcublas.JCublasNDArray<[[    0.0324,   -0.0465,   -0.0584,  ...   -0.2255,    0.1262,    0.0089]]>

macos-x86_64-cpu-avx2
Failed tests:   testRandomBinomial[0: backend(org.nd4j.linalg.cpu.nativecpu.CpuBackend)={1}](org.nd4j.linalg.api.rng.RngTests)

windows-x86_64-cuda-10.0

JVM crash - unclear what test this is (must not extend BaseNd4jTest as test name is not logged before starting - which is another issue in and of itself...

windows-x86_64-cuda-10.1
looks like libnd4j deadlock - look at timestamps here - stuck for an hour in OneOffTests.test_conv2d_nhwc_failed_1
[2020-03-03T22:25:56.614Z] [ RUN      ] OneOffTests.test_assert_scalar_float32_2
[2020-03-03T22:25:56.614Z] [       OK ] OneOffTests.test_assert_scalar_float32_2 (1 ms)
[2020-03-03T22:25:56.614Z] [ RUN      ] OneOffTests.test_pad_1D_1
[2020-03-03T22:25:56.614Z] [       OK ] OneOffTests.test_pad_1D_1 (1 ms)
[2020-03-03T22:25:56.614Z] [ RUN      ] OneOffTests.test_conv2d_nhwc_failed_1
[2020-03-03T23:29:05.295Z] Sending interrupt signal to process
[2020-03-03T23:29:11.675Z] Sending interrupt signal to process
[2020-03-03T23:29:19.890Z] script returned exit code 127

Everything except a windows CUDA issue here is fixed. Will open a new issue for that one.


Oauth proxy

Summary
http://github.com/openshift/client-go/image
Why use imagestreams

An image stream and its associated tags provide an abstraction for referencing container images from within {product-title}. The image stream and its tags allow you to see what images are available and ensure that you are using the specific image you need even if the image in the repository changes.

Need to continue investigating if the imagestream is reliable for us to get oauth-proxy. the oauth-proxy imagestream is created by clusterversion
  ownerReferences:
    - apiVersion: config.openshift.io/v1
      kind: ClusterVersion
      name: version
      uid: e451973e-62f9-44c3-a3ed-441c013da25f
      controller: true

Related issue(s)
Fixes https://issues.redhat.com/browse/ACM-12538

/override ci/prow/sonarcloud
it is hard to cover the watch

/override "SonarCloud Code Analysis"

/lgtm


Topic/add velocity constraint

There is only needed to format the code. I suggest you to run the clang command before committing the work, so we avoid boring commits in the history.

Thanks @PepMS :)


VD16, VD17, VD18 and VDLied issues

On the explorer the title appears first, while on the VRE it appears last.
The “Languages” field on the explorer appears as “Language” on the VRE.
The “Size” field on the explorer appears as “Dimensions” on the VRE.
The “Contributors” field on the explorer appears as “edpoprec:contributors” on the VRE.

Solved.


Add azure-iot-common package installation...

The current docs don't include the required azure-iot-common package installation.

Hi @BretStateham, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@BretStateham - Hi, thanks for your contribution. We are in the process of refactoring the Azure technical articles, so we can’t merge this PR. When the refactoring is complete, we can give you a link to the new location of the article you updated so you can resubmit your update, if you want to.

@BretStateham - Thanks for your contribution to the Azure docs. On November 16th, we announced the availability of our Azure technical documentation in a new location on docs.microsoft.com. You can read about it on our team blog. This change means the article source lives in a new GitHub repository. Because there are some changes in the source markdown as a result of the move, we can’t pull your contribution over to the new repo without creating a ton of merge conflicts. We invite you to resubmit the change (if you are willing) to the new version of the article at https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-node-node-c2d. Thanks for your understanding.


SVHN bugs

which_set='splitted_train' and which_set='valid' fail because split_train_valid returns four values, not two. There's an easy fix, which is to simply turn the first two as a tuple and the last two as a tuple, because split_train_valid used to do this and apparently the relevant code still expects this.
Also, this looks totally suspicious and I suspect that which_set='train_all' has incorrect labels.

First part was fixed in #1371, thanks to @sandeepkbhat.
The second part is indeed suspicious.

I saw that, but it looks like it got merged away or something -- it's not in master (current link as of the time of this comment).

I just merged it, so the first part should be fixed now.
For the second part, I opened #1535.


Removed horizontal scroll by simple hack

Modified style.css file to edit the body tag. That is, set the "overflow-x" attribute to hidden.

"3 Solving Problems By Searching" page still had the issue of horizontal scroll, so it needed a fix.

This hides the problem without trying to understand why the problem is happening or trying to fix the actual cause of it.

@redblobgames You're right. Thanks for the advice. I'll try to identify the root cause and fix that.


Refactor LoggerState, use ZSTD for compression

Combine https://github.com/commaai/openpilot/pull/1098 https://github.com/commaai/openpilot/pull/1951 into one.

[ ] C++ LoggerState

[x] use shared_ptr instead of handles array
[x] use ZSTD for compression

[ ] test routine for LoggerState

We've thought about moving to zstd, but it's not something we're prepared to spend time on for the work that needs to be done outside of openpilot. However, we're thinking of offloading the compression to the DSP if you want to look into that.


updating maintainers to include vsoch

Signed-off-by: Vanessa Sochat<EMAIL_ADDRESS>This pull request will update maintainers to include myself, @vsoch, developer of Singularity Python (spython). The version is not changed, but I'm not sure if I should bump the build number or not, as there is no change to the metadata other than addition of my username.
Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Do I also need to run the re-rendering? Apologies for all the questions, new smithy here :)

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Ah ok, so the re-render instructions are for the conda-forge-admin bot. Noted!

All done now... but please use your newly acquired powers wisely 😉


✨[Feature Request]: Adding an FAQ section

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I want to work on adding an FAQ section to the website. I will make sure that the page is responsive as well.
Use Case
The users might have so many questions in their mind so this section will be helpful. Here is one of the template.

@Anishkagupta04 can you please assign this to me?
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

assigning


Bump k8s version to 1.16.4

Proposed Changes

Bump k8s version to 1.16.4

/assign @ n3wscott

/assign @n3wscott

/lgtm
/approve


Update service-connect-to-microsoft-sustainability-calculator.md

It's little confusing to our customers since the ACM and logic app are not coming in exact order.

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

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

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


Els static analysic js [delivers #158375524]

Description
The DoD requires that we do security linting and suggested that we use eslint-lugin-security. Create react app doesn't allow configuration of eslint, but it is possible to use react-app-rewired to avoid ejecting, so that has also been set up. In addition the warnings it produced have been evaluated and either addressed or disabled on a case by case basis.
Reviewer Notes
The primary warning that needed to be addressed was detect-object-injection which fires anytime variable[key] is used. See article below for why this is a warning. TL;DR: These need to be checked to make sure they don't have user input as the key. That was never the case for our code, so in most cases I have disabled the warning using
// eslint-disable-next-line security/detect-object-injection

or in some cases by disabling the warning for a block of code like this:
 /* eslint-disable security/detect-object-injection */
  const cacheKey = pattern;
  const cache = patternCache[cacheKey] || (patternCache[cacheKey] = {});

  if (cache[pattern]) return cache[pattern];

  const compiledGenerator = pathToRegexp.compile(pattern);

  if (cacheCount < cacheLimit) {
    cache[pattern] = compiledGenerator;
    cacheCount++;
  }
  /* eslint-enable security/detect-object-injection */

There were several cases where mapStateToProps was using formData: state.form[formName]. I've replaced those usages with redux-form's getFormValues as this prevents any risk of modifying state.
Code Review Verification Steps

[ ] All tests pass.
[x] There are no build warnings.

References

Pivotal story for this change
The dangers of square bracket notation explains more about the primary warning.

Do you know if pre-commit will run eslint? I looked through https://github.com/transcom/mymove/blob/master/.pre-commit-config.yaml and didn't find anything.

eslint runs as part of make client_build in a way that we can't tease out -- create react app has it baked in. Do we want it to run twice? Right now if there are warnings, the build will fail.

Yeah, no need to run it twice. Looks good to me.


Pow registration controller, sending options to new.html.eex and edit.html.eex. Further passing options to app.html.eex layout.

If I use Pow's registration controller, how can I send some variables to app.html.eex layout?
Do I need to use custom controllers?

What variables do you want to use? All assigns should be available in app.html.eex.

How do I assign, say @profile to edit.html.eex, to later use it in app.html.eex?

What data will @profile have? You could do it with a plug in router.ex, assigning the value there.

It's a struct

But where do you pull it from? Is it part of @current_user?

I pull it from Profiles context. I use @current_user.id to get Profile and want to pass it to app.html.eex

You can load it in a plug similar to the one in the sync user guide.
defmodule MyAppWeb.LoadProfilesPlug do
  @doc false

  @spec init(any()) :: any()
  def init(opts), do: opts

  @doc false
  @spec call(Conn.t(), atom()) :: Conn.t()
  def call(conn, _opts) do
    case Pow.Plug.current_user(conn) do
      nil ->
        conn

      user ->
        profile = MyApp.Repo.get!(MyApp.Profiles.Profile, user_id: user.id)

       Plug.Conn.assign(conn, :profile, profile)
    end
  end
end

thanks a lot! I will work on this and other issue I opened and get back with results. And close the issues.

Works! Thanks a lot!


Socket termination should trigger onError for streams before RS.onClose

@benjchristensen

I had various subscriptions over a ReactiveSocket, then terminated the socket. Instead of the subscription streams receiving an 'onError' with an exception about the socket/connection closing, they received 'onComplete' and behaved as if everything had completed successfully.
It is my view that a stream should only receive an 'onComplete' if the origin on the server actually sent an onComplete and that ReactiveSocket itself can only terminate a stream via an 'onError'

TODO

confirm against java behaviour
update https://github.com/ReactiveSocket/reactivesocket/blob/master/Protocol.md
implement errors

Behaves as expected in rs-java
https://github.com/ReactiveSocket/reactivesocket-java/blob/52a670f4d7fd8426927a10c97de35da02ed829ac/reactivesocket-core/src/main/java/io/reactivesocket/internal/Requester.java#L1008

https://github.com/ReactiveSocket/reactivesocket-cpp/pull/127


Bringing the plugin to a working state.

Fixed bug with unchanged balance.
Added administrative commands to change the balance. For the administrative command to deposit a player's balance has been added cache, which for 5 days gets the data that could not be processed. This will replenish the balance of those players who have not yet logged on to the server, but bought game currency (for example on the server site).
Improved the /pay command. Now you can specify the currency.
Optional support for LocaleAPI has been implemented. The plugin now supports localization. Players will see messages in the language selected in their game client, provided that the plugin has a language config for this language. If no config is found, the default language config will be used. If there is no LocaleAPI plugin, all messages will be in English (this text is entirely in the plugin code).
In the process of building the plugin, a driver will be added to connect to the MySQL database. This will avoid errors when trying to establish a connection to the MySQL server.
Changed encoding of created tables in the database. Now utf8mb4_unicode_ci is used. I did not have the encoding you specified earlier and the database was not created. I have the latest version of MySQL server MariaDB(Btw. I use Arch. :smile: ).

I didn't try to design the code nicely. You're going to want to design it your way anyway.

Hey @SawFowl, first of all I very much appreciate your continued interest in contributing to Total Economy. At the moment, the codebase is not in stable condition as I'm rewriting it all to make unit and integration testing possible. As well as that, the rewrite is database focused compared to file based which will make data storage much easier to work with. Overall just making the codebase easier for everyone to work with and contribute to.
Also, because of your comment about the encoding I did some more digging and realized it wasn't necessary to specify it since the value I used was the default value in MySQL 8. If you want, double check on your end if you're using MySQL 8 to see if you're still running into any issues. Again, thank you for the continued contributions and I'll be sure to make it more clear when I'm ready to start taking on PRs.


Added option to build the libraries statically

As it is stated in CMake docs SHARED parameter is given to add_library(), when variable BUILD_SHARED_LIBS is ON. So now as before CMake will configure the Makefile to build shared libraries by default and if we want to build them as static, we just have to pass -DBUILD_SHARED_LIBS=OFF to CMake command line.
Also quote from add_library() docs:

For SHARED and MODULE libraries the POSITION_INDEPENDENT_CODE target property is set to ON automatically.

I know nothing about CMake so have to rely on your expertise.  How confident are you that this change will not adversely affect other users?

Perhaps @noviv should review/comment?

I'll configure and build both the original and my forked version and compare the Makefiles and hashes of install trees and let you know.

Ok, so I've compared the original and the forked builds and they turned to be identical.
I've configured both builds using this command line:
cmake <src_path> -DDESTDIR=<install_path>
Generated Makefiles only differ for the source and install paths, which is natural:
https://kpaste.net/02c2be5/diff
sha256sum of Install directory trees does not differ:
https://kpaste.net/058b8a5d/diff
I think this evidence is enough to confirm, that my change does not mess up the build process.
I'd still suggest you to run your regressions if possible, because there is never enough testing right? ;)

How kind of you to think I do regression testing.  I have no problem with this, however I'll leave it for a day or so in case @Noviv wants to comment (thanks to @guymcswain for the suggestion).


android studio not installed and android toolchain

i already installed the android studio but i keep having this problem

Hey @SarausAlZoren , could you please check your version of Android Studio and post the output of flutter doctor -v

Hi @SarausAlZoren
This was reported in #67986, should be fixed soon in the stable channel with cherry-pick release
Thank you

Could possibly related and duplicate of 67986 but to confirm this, please specify the Android Studio version you have installed.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Content managers can configure modals that appear based on conditions in the challenge mode

To discuss:

How do we deal with conditions for single select (value field / label field dropdown?)

Acceptance criteria:

Conditions that can be used are == < > =< => !=
Conditions can be based on interactive elements (sliders and single select) and dashboard results

nationaal & lokaal
condition is placed with the slider

Multiple conditions can be combined for a single modal and are always and condition
The modal can be styled:

Color (red, green and orange)
Optional image (gif)
Additional buttons in buttongroup (default button = 'ga door'), additional buttons with page links are possible

The modal resizes vertically based on the content to minimal size if no image / subtext is present
Modal is shown after the calculation of kpi's is finished
Only the last modal is shown
Ga door of naast klikken sluit de modal

A start has been made:

One modal that consistst of three themes (green, orange, red)

modal themes:

green,
orange,
red

parameters:

localcosts
localnetload
localself_sufficiency
localsustainability
nationalcosts
nationalnetload
nationalself_sufficiency
nationalsustainability
interactive_input (one of the sliders / inputs)

operators:

parameter is lower then value: lower
parameter is lower or equal then value: lowerequal
parameter is bigger or equal then value: biggerequal
parameter is bigger then value: bigger
parameter is equal to value: equal
parameter is not equal to value: notequal

Frontend has been made:


Wrong link in the readme file

Hi!
There's a typo in the readme:
gem 'refinerycms-videojs', git: 'https://anitagraham/refinerycms-videojs.git'
instead of
gem 'refinerycms-videojs', git: 'https://github.com/anitagraham/refinerycms-videojs.git'

Looks like this should go to one of our forks.


link to 1200 contributors on https://www.rust-lang.org/community.html is dead

It looks like the AUTHORS.txt file has been removed from the Rust repo. The GitHub Contributors page for the project only includes the top 100 contributors, which is obviously not enough. Should the file be reintroduced to the project, or some other option introduced?

Hm. It's in the git history, but that's not exactly easy to link to...

Looks like it was removed in this commit.
As the commit message says, the time necessary to maintain the file was deemed to be too much to handle. It may be good to hear from @brson about what sort of manual steps were necessary after running the script, and see if something can be done to bring the file back and improve the situation. If not, there may be another option we could sort out based on his input.

It is a shame to lose this link.
The maintenance was mostly about keeping the mailmap file updated, which for me usually involved resetting and running script to update the authors file repeatedly. When authors change their email preferences you needed to update AUTHORS.txt with their new email manually. The way the maintenance script worked was to merge the git authors into the existing AUTHORS.txt file.
It is great to acknowledge every contributor on the website. I don't know that there's  much better way to maintain the list than what we were doing before. It wasn't a huge amount of work, so maybe its worthwhile to maintain it periodically. Updating it for releases felt like busywork.
We might use a different process for updating an authors list. Previously we were generating it automatically from git log. You'd think this would be an easy way to maintain the list, but actually you spend a lot of time maintaining the mailmap file.
We publish a list of contributors to every release of Rust on the blog, which is cleaned up to look sane before publications. If we maintained the AUTHORS.txt file by just merging that clean list in once, then take one human pass to look for obvious problems, it would be simpler than regenerating the list from scratch.
We could also not maintain the mailmap. Merge the new authors in once, clean up all the obvious junk as well as possible in one pass and publish it.


appservice: update short summary of appservice group to exclude webapp

The appservice web to be gone in June, removing it causes less confusions
General Guidelines
~- [ ] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).~
Command Guidelines
~- [ ] Each command and parameter has a meaningful description.~
~- [ ] Each new command has a test.~
(see Authoring Command Modules)

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3239   +/-   ##
=======================================
  Coverage   70.35%   70.35%
=======================================
  Files         381      381
  Lines       24780    24780
  Branches     3786     3786
=======================================
  Hits        17434    17434
  Misses       6237     6237
  Partials     1109     1109

Impacted Files
Coverage Δ

...vice/azure/cli/command_modules/appservice/_help.py
100% <ø> (ø)
: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 4cb2e59...6507dcb. Read the comment docs.


Redirect trailing slashes so they work

This PR is based on @mloberg's original work but adds in unit tests for the new middleware. I originally PR'd against his repo, but this brings in their code plus the tests and rebases on top of the latest master code at the time this PR was created.

Thanks @dstockto


Desktop: fix test DesktopFontTest.findTypeface for Windows and Linux

Currently it works only on macOs.
It disabled for Windows/Linux here

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Add capability to disable TLS warning

Hi 👋
I'm running a spicedb cluster, and I'm getting a warning in the Status.Conditions of the cluster because TLS is not configured. I would like to be able to remove this warning, I'm running spicedb internally, nothing is exposed to the outside world, there should be no security issue without tls.
Here's the output of describing the cluster:
Name:         spicedb-mycoach-infrastructure
Namespace:    default
Labels:       app.kubernetes.io/managed-by=Helm
Annotations:  meta.helm.sh/release-name: mycoach-infrastructure
              meta.helm.sh/release-namespace: default
API Version:  authzed.com/v1alpha1
Kind:         SpiceDBCluster
Metadata:
  Creation Timestamp:  2023-03-23T17:26:40Z
  Generation:          2
  Managed Fields:
    API Version:  authzed.com/v1alpha1
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          f:meta.helm.sh/release-name:
          f:meta.helm.sh/release-namespace:
        f:labels:
          f:app.kubernetes.io/managed-by:
      f:status:
        f:conditions:
        f:currentMigrationHash:
        f:image:
        f:migration:
        f:observedGeneration:
        f:secretHash:
        f:targetMigrationHash:
        f:version:
          f:attributes:
          f:channel:
          f:name:
    Manager:      spicedb-operator
    Operation:    Apply
    Subresource:  status
    Time:         2023-04-04T08:03:28Z
    API Version:  authzed.com/v1alpha1
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .:
          f:meta.helm.sh/release-name:
          f:meta.helm.sh/release-namespace:
        f:labels:
          .:
          f:app.kubernetes.io/managed-by:
      f:spec:
        .:
        f:channel:
        f:config:
          .:
          f:datastoreEngine:
          f:replicas:
        f:secretName:
        f:version:
    Manager:         Go-http-client
    Operation:       Update
    Time:            2023-03-23T17:57:48Z
  Resource Version:  332941291
  UID:               3fd90ca2-f928-4fc1-bb16-326def7e6ae6
Spec:
  Channel:  stable
  Config:
    Datastore Engine:  mysql
    Replicas:          3
  Secret Name:         spicedb-mycoach-infrastructure
  Version:             v1.18.0
Status:
  Conditions:
    Last Transition Time:  2023-03-23T17:26:40Z
    Message:               no TLS configured, consider setting "tlsSecretName"
    Reason:                WarningsPresent
    Status:                True
    Type:                  ConfigurationWarning
  Current Migration Hash:  n698hch68ch65h544h67fh9ch6q
  Image:                   ghcr.io/authzed/spicedb:v1.18.0
  Migration:               add_caveat
  Observed Generation:     2
  Secret Hash:             n645h599h694hd6h96h547h695h688q
  Target Migration Hash:   n698hch68ch65h544h67fh9ch6q
  Version:
    Attributes:
      migration
    Channel:  stable
    Name:     v1.18.0

And the status condition:
Status:
  Conditions:
    Last Transition Time:  2023-03-23T17:26:40Z
    Message:               no TLS configured, consider setting "tlsSecretName"
    Reason:                WarningsPresent
    Status:                True
    Type:                  ConfigurationWarning

Thanks

It's perhaps worth noting here that TLS is not really optional for some clients, due to https://github.com/authzed/authzed-py/issues/89

Is it basically removing the warning from here: https://github.com/authzed/spicedb-operator/blob/main/pkg/config/config.go#L350


chore(deps): update common-dev-assets digest to 1d3b343

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
23a510d -> 1d3b343

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 3.1.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Is it possible to run a script using TSDX?

Apart from the single entry point i would like to have a demo of the library as well. Is is possible to use tsdx like ts-node?

\Projects>npx tsdx create try_02_tsdx
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.
npm ERR! notarget
npm ERR! notarget It was specified as a dependency of '@babel/preset-env'
npm ERR! notarget
npm ERR! Roaming\npm-cache_logs\2020-06-30T14_31_06_175Z-debug.log
Install for [ 'tsdx@latest' ] failed with code 1

The issue template was removed here and the description is quite short, so it's difficult to tell what the intent here is.
Is your library a bin script? A demo wouldn't need to use ts-node as the library is already transpiled. Unless you're saying a demo of the app inside of a TypeScript application. In either case, it sounds like you'd want an example/ directory similar to the React template but modified for a script or a Node app instead.
This is outside the scope of TSDX the library but could be an idea for another template TSDX could have.

Is is possible to use tsdx like ts-node?

TSDX does not run your code per #478, it just builds the library (and can also test, lint, etc it). If you're looking to run a Node app instead of a Node library, then this would be a duplicate of #654 . It's unclear what the intent of that statement was, so I'm not sure how else to answer.

@agilgur5 i think you answered perfectly. ✋ yes, the intent is to have something similar to the example folder, because so many time you want to test the lib before you actually build it.
Thanks for your time


add JavaScript 5 formatter for Eclipse

@Mmdixon @tavianator @freeone3000 I'll be using this formatter when writing the JavaScript code for Native Dialogue.  It looks similar to our Java 8 formatter.  If you disagree with any of the style choices I've made, please comment on this PR.

is there a way to run this without eclipse?

Idk.  For Java, we run the formatter through maven.  Maybe we can get the thing that builds NativeNlp to run it for us?

Yea we should use something like eslint and just make it part of the build step.  http://eslint.org/

Good idea.  But I still want a formatter in Eclipse in addition to that, so I can hit Ctrl+Shift+F instead of having to build the entire project (which I would probably have to do on the command line).


Add important / demo tests to package and release pipeline

Per @skhorasganiTT , we will add:

single-card device perf
t3000 demos

to release and package.
@skhorasganiTT pls open future issues to add more pipelines. You're also more than welcome to try doing it yourself.

@skhorasganiTT waiting on approval from infra team


Add Waze

Description
Add Waze icon
Type of change

:x: Bug fix (non-breaking change which fixes an issue)
✅ : Icon addition (non-breaking change that adds/modifies Lawnicons's icons)
:x: General change (non-breaking change that doesn't fit the above categories like copyediting)

Icons addition information
Icons added

Waze (com.waze)

Thanks for contributing!


a replication of one chunk translation workflow using Agently development framework

INSTRUCTION
This dir translation_agent_Agently is a replication of one chunk translation workflow using Agently development framework with some extra features:

You can switch models easily by changing the settings in ./app.py to test the workflow
Clearly expression of workflow using Agently Workflow in ./workflow.py
You can edit prompt yaml files in ./prompt_yamls to optimize prompt instructions without coding, these prompt yaml files will be loaded into model request when the workflow start
Add a command line user input interface (enter 3 times to ensure programe know source text input is over)

HOW TO RUN
git clone git@github.com:Maplemx/translation-agent.git
cd translation-agent/src/translation-agent-Agently
pip install -r requirements.txt
python app.py

Hi @Maplemx , thank you for suggesting this new application. We are currently working on integrating a gradio app that has a similar functionality. I invite you please to checkout PR #30 and provide feedback if there is something missing from that application. Thanks!

Hi @Maplemx , thank you for suggesting this new application. We are currently working on integrating a gradio app that has a similar functionality. I invite you please to checkout PR #30 and provide feedback if there is something missing from that application. Thanks!嗨，感谢您推荐这个新应用程序。我们目前正在努力集成具有类似功能的 gradio 应用程序。我邀请您查看 PR #30 并在该应用程序中缺少某些内容时提供反馈。谢谢！

Hi @j-dominguez9 , thank you for inviting me to checkout PR #30 and that's a great work. In fact Agently is a development framework more like LangChain than Gradio or Streamlit. I think this PR can provide two extra features for the applicaiton:

Extend more model support for those models did not adjust OpenAI request format  like Ernie, MiniMax
Separate those key prompts from workflow codes into YAML files to enable users to customize their translation workflow actions


Provide ReactiveMongoOperations criteria extensions

Using infix to have a SQL like syntax, can be maybe be moved to Spring Data later.

Superseeded by https://github.com/spring-projects/spring-data-mongodb/pull/622.


Large runner (16 core, 64G memory) and Singularity to run CI tests

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

@nf-core-bot fix linting

@gq1 : why did you remove the tests on Nextflow 22.10.1 ? It's there to guarantee that the minimal version we advertise is indeed compatible (because locally we tend to test with a later version)

save money? put it back now.

save money? put it back now.

Thanks. I think that money is worth it :)


[Bug]: Feed-in despite feed-in being deactivated

Contact Details
No response
VRM portal ID
c0619ab2c843
Country / region
Germany (de)
B max
2.2
TB max
1.2
FB max
1.2
TG max
0
FG max
10
Battery costs
0.04
Buy price
(p+0.02+0.13)*1.19
Sell price
0
feed-in possible
no
feed-in possible
yes
Version
0.1.6
What happened?
After I updated Venus to 3.10 today and started Dynamic ESS Node Red and another test, I discovered that after a certain amount of time the battery is no longer being charged with the maximum possible power, but is being fed into the network. However, I deactivated feeding into the power grid in Node Red. The error is only fixed again when I set mode 0 and then mode 4 again.
Relevant log output
No response
Screenshots
![DESCRIPTION](LINK.png)

Can you also set feed_in_control_on to false? Not sure if it has to do with it, but it makes not a lot of sense to have this one enabled and the feed_in disabled.

So far I have understood that feed in possible and feed-in control have different uses. The first is for feed in to grid and the second is because the battery discharge can be planned and if the prices are negative the battery is not used. Is it false?
Now i have feed-in control disabeld. I'll get back to you tomorrow at the latest to see if it's of any use
I'll get back to you tomorrow at the latest to see if it's of any use.

What I noticed now that I changed it and the PV no longer produces enough is that the Dynamic ESS somehow limits the power from the battery. Watch in the picture here:

Although in the Dynamic ESS dashboard it looks as if he is not planning to make any purchases:

Hello, yes, I can confirm this issue via beta VRM portal. I think it is related to ESS mode being stuck.
I also think, that this problem is much wider in its philosophy and such issues are complicated to solve and debug with native tools (VRM, console, HW reboot, ...).
I will elaborate further...
Some Modbus settings change console settings (let's call them internal regulation variables - which should NEVER be allowed to change by BMS persistently).
External control should always only substitute persistent setting variables (user defined via console) and should be valid and effective ONLY when communication works OK.
However, when communication with BMS is lost, then reverting to default (fallback) internal settings should happen immediately (does not happen NOW - now last communicated values are pernament).
Therefore, in such cases we see weird behaviour, because simply turning off Modbus, rebooting, reconfiguring does not work - ie. ESS mode can not be set via console - so BMS (3rd party company) can effectively break our installation.
I suggest implementing a heart-beat variable for Modbus. All settings made by BMS (or higher level apps and functions) should not be persistent and after comms are lost then heart-beat variable does not change it's value (ie every 30s) and then Cerbo should fallback to internal (default user defined) values and drop everything that was substituted via BMS.
Unless this is solved, I think implementing more complex d.bus and Modbus apps will be very problematic and not transparent.

I'm noticing the same problem when enabled DynESS. This morning it started buying power at a way more expensive rate then it did sell an hour before. The battery was 50% so it could've used the battery but it did not.

The energy prices differ a lot between buy/sell because I ignore the 'saldering'

So even if i have a 50% charged battery it is going to buy expensive power

@Martijnth what is your vrm id?

@dirkjanfaber 48e7da870ed3

@dirkjanfaber Mr. Hoor, if you would be kind enough, I am trying to make my ESS work again - disabling DynESS does not help, I even tried changing stuck ESS mode via Modbus TCP Client - I changed it successfully, but it did not help either :-(
Any support is appreciated at this time. Otherwise, I will have to reconfigure everything from scratch and cross my fingers :-)
This site has been OK until 500 -> 506 upgrade and then enabling DynESS (I wanted to test with fresh FW...).
Thank you.
Regards,
Robert
ID: c0619ab1eb6e

@kyros32 :As far as I can see, your issue is not Dynamic ESS related. I suggest posting it on the community forum instead in a new post.

@kyros32 :As far as I can see, your issue is not Dynamic ESS related. I suggest posting it on the community forum instead in a new post.

Hi @dirkjanfaber . I want to add to this as I think it is somehow DynESS related. I have tested previous version and disabling it was ok - like deploying disabled flow and then work normally as before DynESS test. I have current version (working with SOC) and I fear that its different story. Tried like 3 days ago to disable it. Do disabled DynESS flow and changed ESS mode to charging. And nothing happened - setting was totally ignored. Then I have tried to turn off nodeRED and still no charging. So now I am stuck. DynESS work otherwise ok I think although there are still some questionable timeframes. But I dont knnow what internal values was changed which are not reverted back after disabling dynESS
So I think that behavior kyros32 described when system is ignoring standard setup after DynESS was installed is valid.
So there is maybe some bug or there are some specific steps to disable it in case its needed. I hope there are still previous configured values stored somewhere.

You can check the mode the system is operating in.
Checking the mode your system is in can be done by using the following piece of flow:
[{"id":"2fd11441aaae5df8","type":"debug","z":"15669bac1d63a856","name":"dynamic ess mode","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"payload","targetType":"msg","statusVal":"","statusType":"auto","x":710,"y":1020,"wires":[]},{"id":"df9c15a37ea63581","type":"victron-input-custom","z":"15669bac1d63a856","service":"com.victronenergy.settings","path":"/Settings/DynamicEss/Mode","serviceObj":{"service":"com.victronenergy.settings","name":"com.victronenergy.settings"},"pathObj":{"path":"/Settings/DynamicEss/Mode","name":"/Settings/DynamicEss/Mode","type":"number"},"name":"","onlyChanges":false,"x":370,"y":1020,"wires":[["2fd11441aaae5df8"]]}]

@dirkjanfaber @khostri Hello, as for my problem, only resolution is sticking to this manual:
https://www.victronenergy.com/media/pg/Cerbo-S_GX/en/reset-to-factory-defaults-procedure.html

Do Cerbo GX reset described above.
Set everything up again
Works like a charm - no need to reconfigure inverters over VE.Bus
After re-enabling DynESS this problem has not occurred yet - maybe fixed or it's a problem with initial set-up

So I really believe this should be looked into before going into production. Definitely some left-over persistent settings that blocks ESS.

@dirkjanfaber @khostri update:
now, when I set DynESS to Auto, it behaves the wrong way like before. However, when I set to Off then ESS starts working again normally. Definitely worth investigating.
-Regards,Robert

Yesterday I updated to beta firmware v3.20~7 since then it seems to be working better. What surprises me a little is that the battery charge is limited and feed into the grid. As a result, the battery is not full. Why is that? I have a Pylontech US2200C, what should the battery capacity be set to 2.2 or 2.4 kWh? Maybe that has something to do with it?

@dirkjanfaber Thanks for flow - its ok for check. I will try to test if mode=0 works.
But regarding bug feeding when should not is there in my instalation also:

disabling feedin manually (or through flow) does not help:

Looking back at the data of the original post of this issue (as there is quite a lot of noise in this issue).
The system calculates every few minutes how quickly it should (dis)charge to reach the target SOC at the end of the hour. If it falls behind (because your load is higher), it will speed up a bit, if it gets there too fast, it will slow down. If the battery capacity is overestimated, it will typically slow down towards the end. If the battery capacity is underestimated, it will typically speed up towards the end.
Battery manufacturers are not always exact in the battery capacity. You may have to tweak it a bit.
If you stop and start the dynamic ess, it forces a re-calculation.


Update go to 1.11.5

There was a serious security flaw found in all versions prior to go 1.11.5: https://github.com/golang/go/issues/29904
So it'd be a good idea to upgrade xgo as many projects depend on it which now still also have this vulnerability because of them being built with Go 1.11.1.

Also there are two prs ready to be merged which could be modified to use go 1.11.5: #149 and #145

@karalabe Any updates on this?

Two (user-closed) PR's have done this:

https://github.com/karalabe/xgo/pull/153
https://github.com/karalabe/xgo/pull/154

how about 1.12

Published 1.11.2, 1.11.3, 1.11.4, 1.11.5 and 1.12. Docker hub will probably take a whole lot (1-2) hours to crunch through the whole queue. They also seem to be having some problems + changed a few things so I had to reconfigure the builders to work around some new errors.


Add a editor FileSystem dock action to open a terminal in selected folder

This is useful to enter some commands without having to open a separate terminal and cd to the project folder.
Tested on:

Windows: PowerShell, cmd, Hyper
macOS: Terminal.app, iTerm2
Linux: Konsole, kitty

Paths with spaces have been tested in all scenarios listed above and are confirmed to be working.
Testing on more Linux terminal emulators is welcome.

This closes https://github.com/godotengine/godot-proposals/issues/6994.

Preview

I force-pushed to fix a typo in the commit message; no other changes were made.

Can confirm that this works on Windows 11 with PowerShell, both for folders and files (open containing folder)

Thanks!

I'm a bit late to the party, but I would have suggested adding another really useful option for Windows:
Showing the shell contextual menu of certain folder/file.
It would be pretty useful, as we would have access to all Windows shell extensions just right there.
This idea comes from TortoiseGit:

The function to call would be SHCreateDefaultContextMenu ( https://learn.microsoft.com/es-es/windows/win32/api/shlobj_core/nf-shlobj_core-shcreatedefaultcontextmenu ) but, after having a look into it, seems that it is not an easy task.


feat: add link between updated and created transactions

Closes #2

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 75 lines in your changes missing coverage. Please review.

Project coverage is 2.59%. Comparing base (247b601) to head (ae67f78).

Files with missing lines
Patch %
Lines

pkg/firefly/client.go
0.00%
45 Missing :warning:

internal/webhooks.go
0.00%
18 Missing :warning:

internal/handlers.go
0.00%
12 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##            main     #10      +/-   ##
========================================
- Coverage   2.78%   2.59%   -0.20%
========================================
  Files         14      14
  Lines        610     656      +46
========================================
  Hits          17      17
- Misses       592     638      +46
  Partials       1       1

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


<EMAIL_ADDRESS>task returning buffer string representations instead of strings

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: SSH
Environment

Server - Azure Pipelines or TFS on-premises? Azure Pipelines

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:

Account name: charli-ai
Team project name: CharliAI
Build definition name/build number: Definition ID is 21, name is Backend release

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name: fv-az215

Issue Description
The SSH@0 task no longer returns output streams (out/err) to the pipeline log. In its place are strings representing buffers, eg.
<Buffer 50 75 6c 6c 20 5b 2f 6f 70 74 2f 63 68 61 72 6c 69 5d 20 69 6d 61 67 65 73 20 66 6f 72 20 5b 53 74 61 67 69 6e 67 5d 20 65 6e 76 69 72 6f 6e 6d 65 6e ... >
<Buffer 46 72 6f 6d 20 73 73 68 2e 64 65 76 2e 61 7a 75 72 65 2e 63 6f 6d 3a 76 33 2f 63 68 61 72 6c 69 2d 61 69 2f 43 68 61 72 6c 69 41 49 2f 63 68 61 72 6c ... >
<Buffer 48 45 41 44 20 69 73 20 6e 6f 77 20 61 74 20 38 66 65 33 66 35 32 20 4d 65 72 67 65 64 20 50 52 20 35 30 39 3a 20 61 64 64 20 73 71 6c 20 73 63 72 69 ... >
<Buffer 41 6c 72 65 61 64 79 20 6f 6e 20 27 6d 61 73 74 65 72 27 0a>

This issue only started when the SSH task was updated to 0.165.0. The previous version (0.151.1) worked correctly.
Error logs
No error logs, but the task is behaving improperly.

At least one other user is impacted as well: https://developercommunity.visualstudio.com/content/problem/928519/ssh-plugin-no-longer-working.html

I'm guessing the problem started here: https://github.com/microsoft/azure-pipelines-tasks/pull/12249/files#diff-2874428db3637eb5645ad56ab629b767R83
tl._writeLine called process.stdout.write, which can accept buffers. console.log() will just print the object.

I also had the problem - I have a work around.  Just add a utf8 conversion to the output stream to ssh2helpers.js  and it should work.  This should be done for all SSH console.log messages.
         defer.resolve('0');
        }
    }).on('data', (data) => {
        /* bad code below
         * console.log(data);
         */
        console.log(data.toString('utf8'));

I just discovered that we started suffering from this regression between 2/21 & 2/24.

Does anyone have a good way to revert to the old task?  My team is using this SSH task to support an EMR that has code written before some of us were born, and this is forcing us to do some manual processes.
Unrelated --- I'm in my mid-30s.

This has been fixed, let me know if you're still experiencing issues - sorry for the inconvenience

Thanks for the quick fix @damccorm. Problem's resolved for us.

@damccorm Thanks for quick fix! The SSH task works for us now too.
I don't see any commits attached to this issue, that is why I would like to ask just this quick follow up question: were there some tests added to check for this so it won't happen in future? :) This was very critical bug for us, as we use output of SSH task to read Terraform plan phase output, when we deploy changes in our Azure infra.

@stazz , I noticed #12441, #12442, and #12443 were completed in short order during the issue; looking through those I did not find any tests.

@MarkIannucci Thanks for finding out the completed issues! Yeah I didn't see any tests either in those. I guess one just has to hope this same thing won't happen in future... 😱