Issue with the agent on Server 2012 R2 after TLS 1.2 on 1/31

After the update this morning for Azure Devops to enforce TLS 1.2, it seems a few (not all) of my older Server 2012 servers (yes we're in the process of upgrading them) cannot talk to the azure pipelines anymore. The error from the agent is below. I made sure the machine is only using TLS 1.2 with TLS 1.0,1.1 and SSL 3 disabled. dotnet framework registry is set to use only strong encryption as well for both x64 and x32.
 [2022-01-31 17:32:50Z INFO LocationServer] Unable to connect to https://dev.azure.com/********/. [2022-01-31 17:32:50Z ERR  LocationServer] System.Net.Http.HttpRequestException: The SSL connection could not be established, see inner exception.  ---> System.Net.InternalException: Exception of type 'System.Net.InternalException' was thrown. -2146892963    at System.Net.SecurityStatusAdapterPal.GetSecurityStatusPalFromInterop(SECURITY_STATUS win32SecurityStatus, Boolean attachException)    at System.Net.Security.SslStreamPal.InitializeSecurityContext(SafeFreeCredentials& credentialsHandle, SafeDeleteContext& context, String targetName, ArraySegment1 input, Byte[]& outputBuffer, SslAuthenticationOptions sslAuthenticationOptions)    at System.Net.Security.SecureChannel.GenerateToken(Byte[] input, Int32 offset, Int32 count, Byte[]& output)    at System.Net.Security.SecureChannel.NextMessage(Byte[] incoming, Int32 offset, Int32 count)    at System.Net.Security.SslStream.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)    at System.Net.Security.SslStream.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)    at System.Net.Security.SslStream.ReadFrameCallback(AsyncProtocolRequest asyncRequest) --- End of stack trace from previous location where exception was thrown ---    at System.Net.Security.SslStream.ThrowIfExceptional()    at System.Net.Security.SslStream.InternalEndProcessAuthentication(LazyAsyncResult lazyResult)    at System.Net.Security.SslStream.EndProcessAuthentication(IAsyncResult result)    at System.Net.Security.SslStream.EndAuthenticateAsClient(IAsyncResult asyncResult)    at System.Net.Security.SslStream.<>c.<AuthenticateAsClientAsync>b__65_1(IAsyncResult iar)    at System.Threading.Tasks.TaskFactory1.FromAsyncCoreLogic(IAsyncResult iar, Func2 endFunction, Action1 endAction, Task1 promise, Boolean requiresSynchronization) --- End of stack trace from previous location where exception was thrown ---    at System.Net.Http.ConnectHelper.EstablishSslConnectionAsyncCore(Stream stream, SslClientAuthenticationOptions sslOptions, CancellationToken cancellationToken)    --- End of inner exception stack trace ---    at Microsoft.VisualStudio.Services.Common.VssHttpRetryMessageHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)    at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)    at Microsoft.VisualStudio.Services.WebApi.VssHttpClientBase.SendAsync(HttpRequestMessage message, HttpCompletionOption completionOption, Object userState, CancellationToken cancellationToken)    at Microsoft.VisualStudio.Services.WebApi.VssHttpClientBase.SendAsync[T](HttpRequestMessage message, Object userState, CancellationToken cancellationToken)    at Microsoft.VisualStudio.Services.Location.Client.LocationHttpClient.GetConnectionDataAsync(ConnectOptions connectOptions, Int64 lastChangeId, CancellationToken cancellationToken, Object userState)    at Microsoft.VisualStudio.Services.WebApi.Location.VssServerDataProvider.GetConnectionDataAsync(ConnectOptions connectOptions, Int32 lastChangeId, CancellationToken cancellationToken)    at Microsoft.VisualStudio.Services.WebApi.Location.VssServerDataProvider.ConnectAsync(ConnectOptions connectOptions, CancellationToken cancellationToken)    at Microsoft.VisualStudio.Services.Agent.LocationServer.ConnectAsync(VssConnection jobConnection)

I have the same issue with 2012 R2. What worked for me is enabling all the Cipher Suites using Nartac (1st screenshot below). 2nd screenshot below shows my SChannel settings.

I have the same issue with 2012 R2. What worked for me is enabling all the Cipher Suites using Nartac (1st screenshot below). 2nd screenshot below shows my SChannel settings.

This worked for me too.  It is quite the sledgehammer approach, but it got my pipelines back up and running until we can get this server upgraded.

Same issue earlier, had to enable the last 2 of the 4 supported Ciphers by dev.azure.com after an SSLLabs scan and make sure Diffie-Hellman KE was set. This said, it appears Microsoft has rolled back the change as the SSLLabs scan I just ran shows the old ciphers and TLS 1.0/1.1 are enabled again.
Cipher Suites
TLS 1.2 (suites in server-preferred order)
TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384
TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256
TLS_DHE_RSA_WITH_AES_256_GCM_SHA384
TLS_DHE_RSA_WITH_AES_128_GCM_SHA256

@jackwhitcomb Could you please try to use the solution that is provided by @kotslee and check if it will help you to bypass the issue.
Generally, situation with disabling of TLS 1.0 and TLS 1.1 is expected and here Microsoft's blog describes the reasons for it - https://devblogs.microsoft.com/devops/deprecating-weak-cryptographic-standards-tls-1-0-and-tls-1-1-in-azure-devops/.

@jackwhitcomb Could you please try to use the solution that is provided by @kotslee and check if it will help you to bypass the issue.
Generally, situation with disabling of TLS 1.0 and TLS 1.1 is expected and here Microsoft's blog describes the reasons for it - https://devblogs.microsoft.com/devops/deprecating-weak-cryptographic-standards-tls-1-0-and-tls-1-1-in-azure-devops/.

We followed the guidance, but what was unexpected AND undocumented are the cipher suites used by dev.azure.com. Best practices dictate we turn the following two cipher suites off, but they seem to be required on a 2012 R2 server to access this endpoint.
TLS_DHE_RSA_WITH_AES_256_GCM_SHA384 (0x9f) DH 2048 bits FS 256
TLS_DHE_RSA_WITH_AES_128_GCM_SHA256 (0x9e) DH 2048 bits FS 128

I have the same issue with 2012 R2. What worked for me is enabling all the Cipher Suites using Nartac (1st screenshot below). 2nd screenshot below shows my SChannel settings.

Thanks, this was a life saver for us ;)


enabling usage of minio

https://github.com/water-tech-repos/wat-hydrograph-stats-py/blob/9fe0463abf3429aedc724519023761a46b410333/tests/aws_integration_test.py#L17
In the other sdk's we enable an env parameter called S3_MOCK where if true we set aws force path style to true, we set the endpoint, and we set the protocol to http instead of https.
https://github.com/USACE/wat-java-sdk/blob/b214177bece2a528d3a970336384303f8d9b5a77/src/main/java/usace/wat/plugin/Utilities.java#L131
https://github.com/USACE/filestore/blob/6484811f7b135c982f7484d6f9bc924fc7de7626/filestore.go#L97
that way we can test with minio locally

ok sorry i missed that detail. i see it now.
you arent getting a cors? i would suspect the way you connect to minio would be http rather than https, which needs to be set in the aws configruation i think.

No worries. The tests pass; not getting a CORS error. When the tests are run with the integration_tests.sh script (which spins up the docker-compose.test.yml and runs the tests), that S3_ENDPOINT is set with an environment variable to point to the minio container (http://minio:9990):
https://github.com/water-tech-repos/wat-hydrograph-stats-py/blob/main/docker-compose.test.yml#L30

yeah i saw that test, and how you set the endpoint. cool im glad it is all working. sorry for the confusion

No problem! Closing this issue, then.


spdlog: reformat expression arguments, and comment about fmt_9

Motivation for this change
Explain in a comment why we use fmt_9, and make that visible in the expression, and not in all-packages.nix.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Now the hal-hardware-analyzer evaluation error should be fixed.

I don't understand why ofborg fails .. hal-hardware-ananlyzer evaluates on my machine... I will try a new PR.

https://github.com/NixOS/nixpkgs/pull/240730


Fix documentation formatting

https://github.com/rust-lang/docs.rs/issues/23#issuecomment-442009197

Really thanks!


juicebox_commands: "Unable to send command to juicebox before current_max and current_max_charging values are set"

Simple issue:
"current_max" and "current_max_charging" should default to the lower of (current_rating (m80 in my case) or the unit's offline current rating (TBD? I can't recall if it's published in the telemetry)). The max field value is 80, and sending an out-of-bounds value will not cause issues; the system has checks to not exceed its unit rating deep internally) when the variable is not otherwise set. A command response must be sent to every incoming telemetry string, so there can't be any reason it won't send a reply.
Seems to be caused by this (juicebox_mitm.py):
    # Send a new message using values on mqtt entities
    async def send_cmd_message_to_juicebox(self, new_values):

       if self._mqtt_handler.get_entity("act_as_server").is_on():

          # Max current values must be define to send messages to device
          if self.is_mqtt_numeric_entity_defined("current_max") and self.is_mqtt_numeric_entity_defined("current_max_charging"):
              cmd_message = self.__build_cmd_message(new_values)
              if cmd_message:
                  _LOGGER.info(f"Sending command to juicebox {cmd_message} new_values={new_values}")
                  await self.send_data(cmd_message.encode('utf-8'), self._juicebox_addr)
          else:
              _LOGGER.warn("Unable to send command to juicebox before current_max and current_max_charging values are set")

I'm having trouble following the process flow around to fix this - we would want to "else" that, if not defined, the value current_rating should be inserted into both values and then proceed normally. (This is how the Enel/eMW system handles a "never seen before" station entity - it writes the unit rating into both fields in the database and then replays it until told otherwise by external configuration)
Additionally, only number.juicebox_max_charging_current seems to be published to my HA dashboard, so there seems to be no way to satisfy this.
(edit: After switching to @ivanfmartinez latest branch, both values are published and I'm able to naturally work-around this by setting both values in HA. Still, this conditional should be reworked to always send a command and set the default values :) )

I selected to only send back the value that are ALREADY configured on the unit. Thats why I only send the command after the unit values are know.
Considering that are many devices with different currents, the basic code dont know what device will be communicating. The sequence that things happens now in juicepassproxy needs for example that the Entityes are created BEFORE the first message came from the juicebox. For this happen and set the max value to be used we need MAX_CURRENT. The default are now 48, in my case that my device supports 32A I set it on docker.
I used a safe approach to make it work on many devices. Can refuse to give some responses, but after getting all data from device messages it will start responding to the device.

Fixed in latest commit 😄 Now it sets offline current after 6 minutes of not sending commands (so the box "hints" its present offline setting). Clever, and it works!


Execute native script written inside the yaml document

Execute inline scripts. The full syntax for cmd is now changed to this:
cmd:
    - command
    - |
    codeblock
    - python:
        - pyscript1
        - |
        pycodeblock
        - pyscript2

That is:

[ ] cmd now accepts type: list[str, dict[Literal['python', 'bash'], str|list[str]]
If given as a raw string, the command/script shall be executed as follows:

[ ] If a shebang is not provided, the command shall be executed using the current shell environment.
[ ] If a shebang is provided, attempt to run the command using the shebang target:

[ ] If the shebang target is valid, use the target to run the command
[ ] If the target doesn't exist, exit and raise error

[ ] If given as a dictionary, the key must be one of the supported scripting languages - python and bash atm. Errors will be raised if invalid.
The value shall be a string or a list of strings, composed of commands to be executed:

[ ] each string value can be a simple command or a full-script.
[ ] for now, values will be interpolated using the syntax $() from the previous iteration.
[ ] if a variable is referenced inside the script or string, it must be valid w.r.t the language syntax, or it must reference a namespace variable defined using the $() syntax.

A future update shall address different modes of variable interpolation, other programming languages, pre-commit hooks, other python versions and virtual environments

Environment Variable Syntax is address in #20


Global setting keys don't show up in autocomplete (sbtVersion)

(using 0.13.5-RC5)
akka > sbt
sbtBinaryVersion    sbtClearOnFailure   sbtDependency       sbtPlugin           sbtPopOnFailure     sbtResolver         sbtStashOnFailure
akka > sbtV
{invalid input}
akka >

An interesting factoid: 78 upvotes for how to check sbt version.

This works in 0.13.11, and it also works in 0.13.5.

Also, completion of build-level keys is better in 0.13.12+

Also, completion of build-level keys is better in 0.13.12+: https://github.com/sbt/sbt/pull/2469

I've just encountered this issue when trying to do autocompletion on parallelExecution. I can confirm it works for sbtVersion, but it doesn't for parallelExecution, which I believe is also scoped in Global.
Can we reopen this issue and aim at getting autocompletion that fully accounts for delegation?

Sure. My guess is that it works with sbtVersion because it's Def.derive-d.


RuntimeWarning: coroutine 'sleep' was never awaited while calling get_data

I very frequently see the mentioned RuntimeWarning above while trying to get_data from miners. The line of code that is responsible for it is pyasic/miners/base.py", lineno 432 api_command_task = asyncio.sleep(0). It would be nice to see this refactored to avoid the spam of RuntimeWarnings.

Should be improved if not fixed in 0.57.6, give that a test.

It's much better thanks!


Log when we download a new ChromeOS ARM image

This is crucial for our ARM users to understand a new Widevine was installed from a specific ChromeOS image.
Having this will make it much easier for users to idenfity the cause of their recent problems, as well as help us identify the affected ChromeOS image in case of issues.

We tend to log as little as possible already, but this event happens maybe once every 31 days so I think this is warranted.

I'm fine with logging it, but in general I prefer ".format" over "%". So I'd do something like "...".format(**arm_device).

lgtm, I'll merge it.


Is this typo intentional? PackageMan(a)gerTests

https://github.com/vidstige/jadb/blob/312cd0235a82faee2928920c2c6077c1538f9bbb/test/se/vidstige/jadb/test/integration/PackageMangerTests.java#L15

It is not! Pull requests open for business if you've got the time. 🙇 Thanks for taking the time to report this.


Unify "reset" password approach

Remove PasswordResetPage for now
unify the API calls

@adammertel please check on the test subdomain, should be ok

https://github.com/DISSINET/InkVisitor/pull/1497


Cherry Pick missing 2.0.1 commits

All Submissions:

[ ] Have you added a note in the CHANGELOG wiki for your change?
[ ] Have you added unit tests for your change?

Retest this please

LGTM


Upgrading to API V5

Please proceed with care. Its got a lot of changes as the API also changed dramatically.
I have put up what I think its a good code base to work from.
Most stuff is already implemented and working.

Ok, I don't have much time to properly test it, so I will just trust you on this one. I still think the docs could be improved a bit by better displaying the options though.
But thanks a lot for the contribution :+1:

@goferito I will work improve the docs. You have my word I will. Well it was not working, now most stuff is working :)

I'll publish it now as v3 on npm. Thanks again.

closes #24


Feature Request - Search by multiple tags

This app is so great!
But this app does not seem to support searching by multiple tags. I think it would be more convenient to support it.

Intuitively, I would expect C to be returned.


fix: #2129 - SafeArea for preferences with sliver

Impacted file:

user_preferences_page.dart: SafeArea for sliver

What

Preference pages with sliver started from the very top of the screen, including the device status bar.
Added a SafeArea to fix that for all concerned preference pages.

Screenshot

Fixes bug(s)

Fixes: #2129

Codecov Report

Merging #2133 (da0d012) into develop (2ea0da3) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #2133      +/-   ##
==========================================
- Coverage     8.86%   8.85%   -0.01%
==========================================
  Files          161     179      +18
  Lines         6623    8692    +2069
==========================================
+ Hits           587     770     +183
- Misses        6036    7922    +1886

Impacted Files
Coverage Δ

...kages/smooth_app/lib/widgets/attribute_button.dart
0.00% <0.00%> (-92.00%)
:arrow_down:

...s/smooth_app/lib/data_models/user_preferences.dart
12.64% <0.00%> (-19.67%)
:arrow_down:

packages/smooth_app/lib/themes/smooth_theme.dart
64.10% <0.00%> (-18.88%)
:arrow_down:

...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart
19.60% <0.00%> (-14.61%)
:arrow_down:

...mooth_app/lib/data_models/product_preferences.dart
26.47% <0.00%> (-4.96%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/welcome_page.dart
0.00% <0.00%> (-3.13%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/scan_example.dart
0.00% <0.00%> (-2.28%)
:arrow_down:

packages/smooth_app/lib/main.dart
15.74% <0.00%> (-2.16%)
:arrow_down:

...s/smooth_app/lib/pages/onboarding/next_button.dart
2.22% <0.00%> (-1.78%)
:arrow_down:

...p/lib/pages/onboarding/consent_analytics_page.dart
0.00% <0.00%> (-1.57%)
:arrow_down:

... and 157 more

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 8ebda96...da0d012. Read the comment docs.


Template isn't reloaded until rack app is restarted.

Hello!
I'm trying to use dry-view in my latest project. It works find, however there is one inconvenience.
While I'm developing after I modify template I need to restart rack app in order to have view output updated (First render is fine, subsequent renders would use cached template instead of reparse updated template).
It would be cool if there was either setting to parse template on each request, or a method that could be called or something.
Is this is expected behavior?
EDIT:
At first I was thinking that it might be some weird hamlit issue, but I could reproduce same behavior with slim
EDIT 2:
For now I swapped dry-view to hanami-view since I'm just starting out (plus to see if the problem is not related, how I set up my app). with hanami-view I don't have this issue. Would still prefer to find a way to use dry-view.

Probably this is about the configuration of your code reloading mechanism and has nothing to do with dry-view. Have you added haml and slim file extensions to it?

Hello!
I do have file extensions .html.slim or .html.haml depending on used engine.
I've created small repo that can be used to reproduce this issue: https://github.com/graudeejs/dry-view-issue-134-code

this is not an issue with dry-view, something like code reloading must be handled either by you or a framework that you use. you can simply reload the server whenever a file is changed. I use entr for that, but there are also ruby solutions likes guard or shotgun.

But I am reloading code. Is there any specific stuff that I need to be able to reload views (or more precisely templates)?

So, to work this around, had to create Dry::View.reset_cache! method
  require 'dry/view'
  require 'dry/view/renderer'
  require 'dry/view/path'
  require 'dry/view/scope_builder'
  require 'dry/view/part_builder'
  module Dry
    class View
      def self.reset_cache!
        [self, Renderer, Path, ScopeBuilder, PartBuilder].each do |klass|
          klass.instance_variable_set(:@__cache__, ::Concurrent::Map.new)
        end
        nil
      end
    end
  end

Technically it seems enough to just reset cache of Dry::View::Renderer, however for correctness of environment after reload I decided to reset all of it (Might change that in future)

@graudeejs yep, this is what's required to make things work. In the future we'll have a more unified caching mechanism across all the gems, so resetting caches will be very simple. I also plan to start dry-rails project that would provide support for reloading OOTB (amongst other things).


ErrorException in 131f65c111715449dcfef46681fad83bd0d079c4.php line 13: Undefined variable: data in Laravel 5.2

I have added Nayjest/Grids in laravel 5.2 with the following instructions.
I have added nayjest/grids and laravelcollection/html using the below composer command.
composer require nayjest/grids laravelcollection/html
Updated composer.json with the following code.
"require": {
    "nayjest/grids": "^1.1",
    "laravelcollective/html": "^5"
},

Added the following code in config/app.php providers
/*
*	For Database Grid
*/
'Nayjest\Grids\ServiceProvider',
'Collective\Html\HtmlServiceProvider'

In config/app.php aliases, added the below code.
// For Grid
'Input' => 'Illuminate\Support\Facades\Input',
'Form'  => 'Illuminate\Html\FormFacade',
'HTML'  => 'Collective\Html\HtmlFacade',
'Grids'     => 'Nayjest\Grids\Grids'

Also in my controller, I have added the below code.
use Grids;
use HTML;
use Illuminate\Support\Facades\Config;
use Nayjest\Grids\Components\Base\RenderableRegistry;
use Nayjest\Grids\Components\ColumnHeadersRow;
use Nayjest\Grids\Components\ColumnsHider;
use Nayjest\Grids\Components\CsvExport;
use Nayjest\Grids\Components\ExcelExport;
use Nayjest\Grids\Components\Filters\DateRangePicker;
use Nayjest\Grids\Components\FiltersRow;
use Nayjest\Grids\Components\HtmlTag;
use Nayjest\Grids\Components\Laravel5\Pager;
use Nayjest\Grids\Components\OneCellRow;
use Nayjest\Grids\Components\RecordsPerPage;
use Nayjest\Grids\Components\RenderFunc;
use Nayjest\Grids\Components\ShowingRecords;
use Nayjest\Grids\Components\TFoot;
use Nayjest\Grids\Components\THead;
use Nayjest\Grids\Components\TotalsRow;
use Nayjest\Grids\DbalDataProvider;
use Nayjest\Grids\EloquentDataProvider;
use Nayjest\Grids\FieldConfig;
use Nayjest\Grids\FilterConfig;
use Nayjest\Grids\Grid;
use Nayjest\Grids\GridConfig;
use Nayjest\Grids\DataProvider;

While in controller, when I tried the below code, it works.
public function getExample1()
{
	$cfg = [
		'src' => 'App\Users',
		'columns' => [
			'id',
			'login_type',
			'uid',
			'uname',
			'email',
			'xp'
		]
	];
	echo Grids::make($cfg);
}

But when I use the below code in my view, I am getting the error.
<form>
<?php
use Nayjest\Grids\Grid;
use Nayjest\Grids\DataProvider;

/**	@var Nayjest\Grids\DataProvider $data **/
/**	@var Nayjest\Grids\Grid $grid **/
?>
<table class="table table-striped" id="<?= $grid->getConfig()->getName() ?>">
<?= $grid->header() ? $grid->header()->render() : '' ?>
<?php # ========== TABLE BODY ========== ?>

<?php while($row = $data->getRow()): ?>
    <?= $grid->getConfig()->getRowComponent()->setDataRow($row)->render() ?>
<?php endwhile; ?>

<?= $grid->footer() ? $grid->footer()->render() : '' ?>
<?php # Hidden input for submitting form by pressing enter if there are no other submits ?>
<input type="submit" style="display: none;" />

</form>

Hi!
How you changed default grid view template?

Oh, you should not use that code in your page view, you need just pass grid to view variables and print it inside view:

My grid:

<?= $grid ?>

Hi!
Have you found a solution for this issue?
Any case, it's better to render grid in controller, save output into variable and just print it in view.

// in controller:
$grid =  Grids::make($cfg)->render();
// in view
<?= $grid ?>


slingshot/spacesuit: add SignedInteger::checked_add, checked_mul

Adds checked_add and checked_mul to check overflow on SignedInteger
Part of #91

Please also see some checked arithmetic for ScalarWitness in #98 including my comments.

Can we avoid checked_* and simplн define Add with Add::Output=Option<SignedInteger>? So any attempt to add signed integers will be always checked.


feat: allow quiz answer value to have string type

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

The value of QuizQuestion answer is currently a number. This matches the challenge schema in /learn, but not in the exam environment, where the answer values are strings (the app uses answer IDs for values, and answer IDs are strings).
This PR updates QuizQuestion and Quiz answer to accommodate for string values.

Are we definitely using these for the exams? The problem is, right now, a lot of what these components do is concerned with giving feedback about which answer is correct. However, for the exam, I don't believe the client will have access to that information. It should be getting a list of questions and potential answers, but no information about which of those is correct.

Yes we only make use of the QuizQuestion component not the Quiz component.


chore(main): release 2.10.0

:robot: I have created a release beep boop
2.10.0 (2024-08-08)
Features

Feature 140 popup when dashboard is under maintenance (WIP)  (#150) (c1f1c40)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ai4os/ai4-dashboard/releases/tag/v2.10.0 :sunflower:


How to generate a column data depend on other column data

Summary
I am trying to insert a record, for example, Total runs and highest scores. In this case, total runs can not be less than highest scores.
Versions

Version

PHP
7.3.0

fzaninotto/faker
^1.9

use Faker\Generator as Faker;

$factory->define(App\Player::class, function (Faker $faker) {
return [
        'total_runs' => $faker->randomNumber(4),
        'highest_scores' => $faker->randomNumber(3),
     ];
});

Is there a direct method other than this?
$total_runs = $faker->randomNumber(4);
$highest_scores = $faker->randomNumber(3);
if($highest_scores > $total_runs){ $highest_scores = $total_runs - 1;}
return[
         'total_runs' => $total_runs,
         'highest_scores' => $highest_scores,
];

I've tried searching on google and looked in previous issues but could not found anything related to this. Please help.

Maybe it works to you.
$total_runs = $faker->randomNumber(4);
$highest_scores = $faker->numberBetween(0, min($total_runs, 999));


How does search bar work?

I can type there, but it does not search, nor opens a new tab;  does not react to ENTER etc.

You need to use prefix and then type your search query. For example, to search "what is docker" using google search you would type /g what is docker.
Please visit README file for reference.

My bad, I did not check the github main page.

Notification with error Prefix not found will now be displayed if prefix is incorrect or missing.


docs(tabs): add base theme demo

What I did

Adds base theme demo

Closes #1336
Testing Instructions

Notes to Reviewers

"Theme", as we've seen, is a loaded and amorphous concept, one we're working very hard to simplify and define for our user's benefit. Does it help our users to have multiple, subtly different, current-task-dependent definitions of the word theme in our DS?
I suggest we mark this attribute as deprecated and instead document how to use the component tokens to set custom colours.

@coreyvickery @marionnegp this also doesn't exist in the current rh-tabs figma file or UX dot, was this an intentional drop from XD -> Figma transistion?  Maybe it's something worth depreciating and implementing as @bennypowers is suggesting.


⚠ Roblox has degraded performance

In 40e97bd, Roblox (https://www.roblox.com) experienced degraded performance:

HTTP code: 200
Response time: 2609 ms

Resolved: Roblox performance has improved in 84aacbf.


Add option to use EGL rendering for Linux systems when disp=False.

By default, the Environment class was using openGL hardware accelerated rendering. By switching to EGL rendering if using a Linux system, you can get an approximately 10x speedup in each call to the env.step method (9.83x to be precise.) This patch also fixes a visual bug in the spatula URDF which was missing a material definition.
Tested on my Ubuntu 20.04 machine with a RTX 2080ti:

Renderer
Mean
Std dev

Default openGL
0.2252595923267878
0.03675561883686646

EGL
0.02291664930695262
0.0037472464227214205

Speedup = 9.83x

@ayzaan Would be awesome if this could get this merged!

Hi Kevin, yes when I tried running this internally it didn't work out of the box. I can take a look again and see if I can get it running properly.

@ayzaan Done, made a separate PR for the spatula bugfix #17.


Add logic connecting the character name to the existing logic for the search endpoint

A match will be found based on the cross referencing of show with cast/character name data. A console log should be generated with either a character not found message flagged as false results, or a character name and actor name with true results flagged.

Duplicated in previous work


Cannot call (gdscript) object's methods from lua

When creating a class in godot, I cannot use lua.globals to make the class accessible from lua
Reproduction
Create and run a new scene with the below script:
extends Node2D

class SomeClass:
	func _init(from_gd=false):
		if from_gd:
			print("Init was called from gd")
		else:
			print("Init was called from lua")

	func some_method():
		print("Some other method was called")

func _ready():
	var lua = LuaState.new()
	lua.open_libraries()

	var c = SomeClass.new(true)
	lua.globals["SomeClass"] = c

	var result = lua.do_string("""
	SomeClass()
	SomeClass:some_method()
	""")

	if result is LuaError:
		printerr("Error in Lua code: ", result)
	else:
		print("LUA result is ", result)

When running the code, the following error is seen:
Init was called from gd
  Error in Lua code: [string "..."]:2: attempt to call a RefCounted value
  stack traceback:
  	[C]: in global 'SomeClass'
  	[string "..."]:2: in main chunk

Expected behavior
Calling SomeClass() from lua would call the init method, and calling SomeClass:some_method() would call the object's some_method method.
Environment:

Godot version: v4.2
Module version: v0.1.0 from https://godotengine.org/asset-library/asset/2330

Additional details
The init function can be called when setting lua.globals["SomeClass"] = c._init, however, when using this, some_method is not recognized as valid:
Init was called from gd
Init was called from lua
  Error in Lua code: [string "..."]:3: attempt to call a nil value (method 'some_method')
  stack traceback:
  	[string "..."]:3: in main chunk

Hey @al0fdf , thanks for the report.
When you do lua.globals["SomeClass"] = c, in Lua SomeClass doesn't refer to your class, but rather to c, which is an instance of your class that you created in the line above var c = SomeClass.new(true).
That's why the error message in your first attempt is attempt to call a RefCounted value, because you're trying to call a value that is actually an instance of your class and not the class itself.
When you do lua.globals["SomeClass"] = c._init, now SomeClass refers to the Callable c._init. Callables don't have a definition for some_method, that's why you get the next error attempt to call a nil value (method 'some_method').
If you want to properly create instances from Lua, you do just like in GDScript: call SomeClass:new(...).
This only works if the class name is registered in ClassDB, when ClassDB.class_exists("SomeClass"). I don't know if nested classes, like SomeClass in your GDScript there, would work. In case it doesn't , moving the class to a separate script and giving it a class_name should do the trick.
Another thing about object initialization: just like in GDScript, you never call _init yourself, it gets called automatically from new(...).

I'm closing this issue, since the problem is in your script. Feel free to open a new Issue or Discussion for other problems/questions.


core build task lacks status indicator

@tyriar this is probably what you were seeing.

I was wrong - I just waited a bit and it ultimately showed up.

Pls reopen if you see this


WIP: solr: 4.10.3 -> 7.5.0, refactor service to reflect major changes in version bump

Fix for issue #33876 by updating solr from an ancient version to the most recent.
I have only tested that solr actually starts running, but I have not indexed anything yet. Looking for feedback from anyone who is familiar with solr. I'll be asking a colleague at work who is familiar with solr to test this out and then report back. Will be adding a NixOS test when I get a chance.
Motivation for this change
Security issues on ancient version of solr. Version bump to latest for fix.
Things done

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

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

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

@domenkozar are you still using familiar with solr? This is trying to resurrect the whole thing with a test. If you have any input that would be appreciated.

@andir Given this entirely breaks compatibility with previous module/package, but previous module/package is now entirely unsupported by upstream (and has CVEs against it...) what is the plan as far as which versions of NixOS this will land in?

@aanderse I'd say master and then the next release until there is a good reason to replace the old (& broken) module on 18.09.

@andir sounds good.
@GrahamcOfBorg test solr

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: tests.solr
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: tests.solr
Partial log (click to expand)
machine: exit status 1
syncing
machine: running command: sync
machine: exit status 0
test script finished in 41.47s
cleaning up
killing machine (pid 597)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/hn8in1czvbr5j3qmzpz9n4h1r5gni6xf-vm-test-run-solr

@aanderse I did rebase, and moved a few changes around. I also had to bump the module id since something else was merged with 307 already.
@GrahamcOfBorg test solr
When the tests complete this is good for me.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: tests.solr
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

@aanderse would you like writing a changelog entry for the (re-)introduction of solr? :)

Success on x86_64-linux (full log)
Attempted: tests.solr
Partial log (click to expand)
machine: exit status 1
syncing
machine: running command: sync
machine: exit status 0
test script finished in 36.85s
cleaning up
killing machine (pid 597)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/4piih9b917n00k6snghscz9v5ar9nb2k-vm-test-run-solr

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: solr
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: solr
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0/bin
patching script interpreter paths in /nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0
/nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0/server/scripts/cloud-scripts/zkcli.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/bash"
/nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0/server/scripts/cloud-scripts/snapshotscli.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/bash"
/nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0/example/exampledocs/test_utf8.sh: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
/nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0/contrib/prometheus-exporter/bin/solr-exporter: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
/nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0/bin/.post-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/bash"
/nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0/bin/.solr-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0...
/nix/store/8lmj2yjns5cpgjcdxckxia1jgf96xw8m-solr-7.5.0

@andir I hope this entry will suffice. If not please let me know. Thank you.

Can you cleanup the git history a bit like I did a few days before you pushed your latest changes. I think the merge of master should go away and also a few of those commits should be stashed together.

@andir Well I've definitely made the situation worse, and I'm sorry but I don't know how to fix it. I'll try to find some time to study up on git some more.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: solr
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: solr
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0/bin
patching script interpreter paths in /nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0
/nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0/server/scripts/cloud-scripts/snapshotscli.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/bash"
/nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0/server/scripts/cloud-scripts/zkcli.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/bash"
/nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0/bin/.post-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/bash"
/nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0/bin/.solr-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/bash"
/nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0/example/exampledocs/test_utf8.sh: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
/nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0/contrib/prometheus-exporter/bin/solr-exporter: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0...
/nix/store/d2kgqgl08navv2npc2fw2p5s55lcbq61-solr-7.5.0


Cant navigate within Webmin Servers

Hi,
After the last update (13.00 and 13.02) I could not navigate to other webmin servers. I use "Login via Webmin" method..

So... I connect from my main server to another server and I get doubled sidebars

If I press anything from the side pannel it sends me back to the main server information page

Thanks

I got it! Will fix it in 13.03 very soon. After update please report here if it's working or not! Thanks for reporting!


[bug]: Unable to construct funding tx/derived public key does not match bip32 derivation info public key

Background
We are using two lnd instances (remote signer feature) and trying to open a channel:
lncli openchannel --node_key 02015cb10a74914fe6c126e591b1a4bda66662ca807e97a94cf973ffe408259f74 --local_amt 3000000 --sat_per_vbyte 1
[lncli] rpc error: code = Unknown desc = unable to construct funding tx: error signing with remoteinstance: remote signer returned invalid key spend signature: signature too short with 0 bytes

The relevant logs on the watch only instance are:
2022-10-11 20:43:02.050 [INF] SWPR: Manual fee rate input of 250 sat/kw is too low, using 253 sat/kw instead
2022-10-11 20:43:02.050 [INF] FNDG: Initiating fundingRequest(local_amt=0.03 BTC (subtract_fees=false), push_amt=0 mSAT, chain_hash=000000000019d6689c085ae165831e934ff763ae46a2a6c172b3f1b60a8ce26f, peer=02015cb10a74914fe6c126e591b1a4bda66662ca807e97a94cf973ffe408259f74, min_confs=1)
2022-10-11 20:43:02.050 [WRN] LNWL: Web API does not have a fee rate for target=3, using the fee rate for target=2 instead
2022-10-11 20:43:02.050 [INF] CHFD: Performing funding tx coin selection using 253 sat/kw as fee rate
2022-10-11 20:43:02.065 [WRN] CHFD: Unable to find funding output for shim intent: unable to create witness script, no funding keys
2022-10-11 20:43:02.082 [INF] FNDG: Target commit tx sat/kw for pendingID(c2c839c49b0cc1ea2f1c8aa67c7e979c29f4cc3c219af6c48e50f9cf828f9699): 1275
2022-10-11 20:43:02.082 [INF] FNDG: Dust limit for pendingID(c2c839c49b0cc1ea2f1c8aa67c7e979c29f4cc3c219af6c48e50f9cf828f9699): 0.00000354 BTC
2022-10-11 20:43:02.082 [INF] FNDG: Starting funding workflow with lrttbpewpyf4w4ecprhgfih4ymhhssbnqmrcyjzbksjmwhxhc5ra63yd.onion:9736 for pending_id(c2c839c49b0cc1ea2f1c8aa67c7e979c29f4cc3c219af6c48e50f9cf828f9699), committype=tweakless
2022-10-11 20:43:03.311 [INF] FNDG: Recv'd fundingResponse for pending_id(c2c839c49b0cc1ea2f1c8aa67c7e979c29f4cc3c219af6c48e50f9cf828f9699)
2022-10-11 20:43:03.313 [ERR] FNDG: Unable to process contribution from &{{[2465652 16775426 43306903 33684062 40264394 6893417 40786203 20945668 17462417 22316]} {[59556220 3875047 1227694 23039521<PHONE_NUMBER>8236 64646312 30028152 54437282 974180]}}: unable to construct funding tx: error signing with remoteinstance: remote signer returned invalid key spend signature: signature too short with 0 bytes
2022-10-11 20:43:03.313 [INF] FNDG: Cancelling funding reservation for node_key=02015cb10a74914fe6c126e591b1a4bda66662ca807e97a94cf973ffe408259f74, chan_id=c2c839c49b0cc1ea2f1c8aa67c7e979c29f4cc3c219af6c48e50f9cf828f9699
2022-10-11 20:43:03.313 [ERR] RPCS: unable to open channel to NodeKey(02015cb10a74914fe6c126e591b1a4bda66662ca807e97a94cf973ffe408259f74): unable to construct funding tx: error signing with remoteinstance: remote signer returned invalid key spend signature: signature too short with 0 bytes
2022-10-11 20:43:03.313 [ERR] RPCS: [/lnrpc.Lightning/OpenChannel]: unable to construct funding tx: error signing with remoteinstance: remote signer returned invalid key spend signature: signature too short with 0 bytes

If we look at the remote signer logs, the only relevant line is:
[WRN] BTWL: SignPsbt: Skipping input 0, derived public key 02464e51e1eede9721d0c97bc15cbfea5c38a6f6b820c69ac14623c971932464d3 does not match bip32 derivation info public key 02cd0efa576d888e03623a311e3192de0f2df06a88a0493ee64463c91954b3a075

The node is "identity_pubkey": "030e7d77f4012c809e4bcd0701e7d50b3a0869679f328e325e222ae56acabc74b2" which has dozens of channels. The only recent change was the update from 0.15.1 to 0.15.2 and some channels that are being force-closed.
Environment
Both instances are using 0.15.2-beta
Backend is a full bitcoin core 22 in neutrino mode (-txindex=1 -blockfilterindex=1 -peerblockfilters=1)

So I tried --reset-wallet-transactions and it did nothing to change the error.
Then I tried to open a smaller channel and got an err=Synchronizing blockchain
(While always getting synced_to_chain=true and synced_to_graph=true on lncli getinfo and active=true and synced=true on lncli neutrino status)
Then I switched to bitcoin core rpc and now everything is working. So this must be an issue with neutrino.
What's weird is that this setup ran flawlessly for many months and nothing has changed recently besides the upgrade to 0.15.2-beta

Just for clarity, you were running bitcoin core yourself serving block filters and using them from your lnd instance?

Just for clarity, you were running bitcoin core yourself serving block filters and using them from your watch-only lnd instance and switching over to using bitcoind directly as the backend fixed it for you?

Exactly. This is a kubernetes cluster, all the instances are of different pods communication through the internal network.

I took a quick look as well to make sure there's no new issue with remote signing setups.
This sounds very weird to me. I cannot really see how the chain backend would have an influence on the derivation paths. Is it possible the above error was just a temporary problem in general? Are you using some sort of load balancing and perhaps the watch-only node contacted/connected to the wrong remote signing instance?

I took a quick look as well to make sure there's no new issue with remote signing setups. This sounds very weird to me. I cannot really see how the chain backend would have an influence on the derivation paths. Is it possible the above error was just a temporary problem in general? Are you using some sort of load balancing and perhaps the watch-only node contacted/connected to the wrong remote signing instance?

There are no other signing instances on this cluster and the error really seemed to be persistent.
I tried to change back again to neutrino and the only error I'm able to get is the err=Synchronizing blockchain that happened before while trying to open a small channel. I don't have much on chain balance so I may try later with more funds to open a big channel.
So right now neutrino seems to be really broken. At least for channel opening purposes.

Could it be that you got the err=Synchronizing blockchain from the remote peer? Or did you try both times with the same remote peer?

Could it be that you got the err=Synchronizing blockchain from the remote peer? Or did you try both times with the same remote peer?

Both times the err=Synchronizing blockchain come while trying to open a channel to 1ML. Looking at the comments on https://github.com/lightningnetwork/lnd/blob/b0e9442ba5e5c145c9725907a18b887d16247e24/lnwire/error.go#L18 it seems that yes, it could come from the remote peer.

Could it be that you got the err=Synchronizing blockchain from the remote peer? Or did you try both times with the same remote peer?

Both times the err=Synchronizing blockchain come while trying to open a channel to 1ML. Looking at the comments on
https://github.com/lightningnetwork/lnd/blob/b0e9442ba5e5c145c9725907a18b887d16247e24/lnwire/error.go#L18
it seems that yes, it could have come from the remote peer.

I can confirm, I've faced the same error and debug logs says
2022-10-17 20:55:52.252 [ERR] RPCS: unable to open channel to NodeKey(0217890e3aad8d35bc054f43acc00084b25229ecff0ab68debd82883ad65ee8266): received funding error from 0217890e3aad8d35bc054f43acc00084b25229ecff0ab68debd82883ad65ee8266: chan_id=1ecd05ec14ec1462f77665239fb53341f58a7c82f9b6e03f9856ec86ddef16f0, err=Synchronizing blockchain

It's happening again but now using the RPC:
Watch Only:
2022-10-27 16:57:10.864 [INF] FNDG: Recv'd fundingResponse for pending_id(1299ae4a6fb9d4c2d20b6c5f79b8513f245d63eb3456926fcfc8d44dc2c57c83)
2022-10-27 16:57:10.867 [ERR] FNDG: Unable to process contribution from &{{[42673605 2107050 10768122 29318847 38745622 56734640 50472538 59755664 26394517 2918186]} {[4150870 10742057 66321416 1250398 32948464 14909017 31841400 22939989 60274405 1463876]}}: unable to construct funding tx: error signing with remoteinstance: remote signer returned invalid key spend signature: signature too short with 0 bytes
2022-10-27 16:57:10.867 [INF] FNDG: Cancelling funding reservation for node_key=02b21ca992bf95e3f324302265ad86cec24f36166fd7afca44efa0809aaa8b25c5, chan_id=1299ae4a6fb9d4c2d20b6c5f79b8513f245d63eb3456926fcfc8d44dc2c57c83
2022-10-27 16:57:10.867 [ERR] RPCS: unable to open channel to NodeKey(02b21ca992bf95e3f324302265ad86cec24f36166fd7afca44efa0809aaa8b25c5): unable to construct funding tx: error signing with remoteinstance: remote signer returned invalid key spend signature: signature too short with 0 bytes
2022-10-27 16:57:10.867 [ERR] RPCS: [/lnrpc.Lightning/OpenChannel]: unable to construct funding tx: error signing with remoteinstance: remote signer returned invalid key spend signature: signature too short with 0 bytes

Signer:
2022-10-27 16:57:10.867 [WRN] BTWL: SignPsbt: Skipping input 0, derived public key 02663bf07bc2f47108f0c9e9aa63f0ef4d31fb3b44141b3d21321049133379d6f7 does not match bip32 derivation info public key 039b4c16e5f41a8ff887a7ab6ac6cb50667a1380a299a6e0fa55053f96a3af4c6a

Any ideas for a workaround?

Also I found out I can't move these coins. Same error

Can you compare the output of lncli wallet accounts list of both the watch-only and the signer node? The only difference should be the watch_only flag and the name of some accounts. But there should be the same number of accounts with exactly the same xpubs.

Yes, the only differences are the name, watch_only, external_key_count and internal_key_count (these counts are always zero on signer). The quantity and values of xpubs and derivation_paths are the same.

Also I found out I can't move these coins. Same error

I'm unable to use the total but I was able to transfer a fraction

Interesting... Thanks for the info. I'll see if I can reproduce this.

I'm unable to use the total but I was able to transfer a fraction. My hunch is that the error happens when it tries to include a p2tr output. If it only uses a p2wpkh address it seems to work.

Another hunch is the following:
I've read somewhere that LND won't let you spend all your funds because it requires some reserved funds. So instead of telling you that you can't spend most of funds, because of this watch-only/signer segregation, it is hitting a bug on this corner case and the result is this weird error instead of the correct message.

Gave this another attempt at reproduction. Your hunch sounded very plausible but it seems like that is not the case. I tried with lnd v0.15.4-beta and I got a proper error ([lncli] rpc error: code = Unknown desc = not enough witness outputs to create funding transaction, need 0.00198425 BTC only have 0.001923 BTC  available) when attempting to open a channel with a size that would leave insufficient balance for the anchor reserve.
Can you reproduce the issue reliably or does it only happen sometimes? Would you be willing to share a bit more info about your setup and your UTXOs (perhaps on Slack for more privacy)?

Gave this another attempt at reproduction. Your hunch sounded very plausible but it seems like that is not the case. I tried with lnd v0.15.4-beta and I got a proper error ([lncli] rpc error: code = Unknown desc = not enough witness outputs to create funding transaction, need 0.00198425 BTC only have 0.001923 BTC available) when attempting to open a channel with a size that would leave insufficient balance for the anchor reserve.
Can you reproduce the issue reliably or does it only happen sometimes? Would you be willing to share a bit more info about your setup and your UTXOs (perhaps on Slack for more privacy)?

Yes, it's very reliable. I've joined the slack


Silences are randomly sorted

On the /#/silences endpoint, it seems like silences are in a different order every time the page is refreshed. Is it possible to have them sorted e.g. by start or expiration time?

Related to #812. Would you be up for submitting a PR for this?

That depends... do you think the sorting should be done in the frontend or the API?

@mpitt I would vote for sorting on server side. I don't think this should be a big performance issue. @stuartnelson3 @fabxc what do you think?

I would say server side as well.

Also in favor of server side. Most folks probably have less than 2000 silences, and sorting that should be pretty quick.
I'm not sure how we should sort. We ran into the issue at soundcloud where folks had expirations set years in the future, so there was always noise at the top of the list with things that weren't expiring until 2025. Thoughts on sorting by expiration, soonest expiring first?

Maybe configurable via UI? :slightly_smiling_face:

We have three lists of silences, that I think should have different sorting rules:

Active — soonest expiring first (asc expiration date)
Pending — soonest starting first (asc start date)
Expired — soonest expired first (desc expiration date)


Implement async variants of methods that make HTTP requests

Currently we make blocking calls when we make HTTP requests to the API server, which makes this library unusable from async contexts, common in most projects that make network requests. We should implement async variants of methods on our service structs, which will require moving away from our ureq dependency which does not support async. We should probably use Tokio and reqwest.

We would like to address this in the future – as it is important for most contexts. We're open to a PR that introduces this.


Simplify Orientation angle check (try 2)

@kecnry, like this?
xref #27
for spacetelescope/jdaviz#2179

In theory, but that seems to fail when calling add_orientation manually.

Gah, okay. At least I tried. Thanks!


Tests and Monthly Automated Tag

Describe the bug
In the testing_pr.yml file the tests are run for python 3.8, 3.9, 3.10, 3.11, 3.12.
In the monthly-tag.yml only 3.7 and 3.8 are tested.
While in the ReadMe compatibility is ensured for 3.7,3.8, 3.9, 3.10.
Expected behavior
I expect PINA to run for python 3.8, 3.9, 3.10, 3.11, 3.12 so the ReadMe and the monthly-tag.yml should be changed accordingly.

Also consider to update the PyPl page for Python >= 3.8


Implement okOr/okOrElse for PlainOption

Fix https://github.com/karen-irc/option-t/issues/215
Fix https://github.com/karen-irc/option-t/issues/214

This change is

bors r+


New version: Elgato.WaveLink version <IP_ADDRESS>6

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

It installs correctly for me, it just needs reboot of the PC on which it was installed.

Automatic Validation ended with:

Installation failed with exit code -1978334967 2024-01-19 13:05:47.029 [CLI ] ShellExecute installer failed: 3010 CAQuietExec:  Error 0x80070001: Command line returned an error. CAQuietExec:  Error 0x80070001: QuietExec Failed CAQuietExec:  Error 0x80070001: Failed in ExecCommon method CustomAction ClearFirewall returned actual error code 1603 but will be translated to success due to continue marking Property(S): DRIVERINSTALLFAILED_MESSAGE = Driver installation failed. Please reboot and start setup again. Property(S): DRIVERINSTALLFAILED_AUTO_MESSAGE = Driver installation failed. Please reboot and Setup will be started again. Property(S): ErrorDialog = ErrorDlg Property(S): SecureCustomProperties = DELETE_APPDATA_FILES_ACCEPTED_VALUE;DRIVERINSTALLFAILED_AUTO_MESSAGE;DRIVERINSTALLFAILED_MESSAGE;EXE_FILE_NAME;INSTALLDIR_PRODUCT_FAKE;INSTALLER_FILE_NAME;IS_DRIVER_UP_TO_DATE;IS_ELGATO_WAVE_DEVICE_CONNECTED;KEEP_SETTINGS_ALERT_MESSAGE;NETFRAMEWORK45;NEW_FIRMWARE_VERSION;OLD_COMP_PATH;OLDER_VERSION_DELETING_REGISTRY_APPDATA_ON_UPGRADE;OLDER_VERSION_WITH_ELGATO_PLUGINS;OLDERVERSIONBEINGUPGRADED;UNPLUG_MESSAGE;USAGE_STATISTICS_ACCEPTED_VALUE;WIN10FOUND;WIX_DOWNGRADE_DETECTED;WIX_UPGRADE_DETECTED;WIXUI_INSTALLDIR MSI (s) (D8:D4) [13:05:46:702]: Windows Installer installed the product. Product Name: Elgato Wave Link. Product Version: <IP_ADDRESS>6. Product Language: 1033. Manufacturer: Corsair Memory, Inc.. Installation success or error status: 0.

(Automated response - build 615.)

Hi @krystofkrticka,
Driver installation failed. Please reboot and start setup again.
Despite having a failing error message, this package appears to install and launch normally. Also, there wasn't any interactive installer after rebooting the VM. Thoughts?

Automatic Validation ended with:

Installation failed with exit code -1978334967 2024-01-19 13:05:47.029 [CLI ] ShellExecute installer failed: 3010 CAQuietExec:  Error 0x80070001: Command line returned an error. CAQuietExec:  Error 0x80070001: QuietExec Failed CAQuietExec:  Error 0x80070001: Failed in ExecCommon method CustomAction ClearFirewall returned actual error code 1603 but will be translated to success due to continue marking Property(S): DRIVERINSTALLFAILED_MESSAGE = Driver installation failed. Please reboot and start setup again. Property(S): DRIVERINSTALLFAILED_AUTO_MESSAGE = Driver installation failed. Please reboot and Setup will be started again. Property(S): ErrorDialog = ErrorDlg Property(S): SecureCustomProperties = DELETE_APPDATA_FILES_ACCEPTED_VALUE;DRIVERINSTALLFAILED_AUTO_MESSAGE;DRIVERINSTALLFAILED_MESSAGE;EXE_FILE_NAME;INSTALLDIR_PRODUCT_FAKE;INSTALLER_FILE_NAME;IS_DRIVER_UP_TO_DATE;IS_ELGATO_WAVE_DEVICE_CONNECTED;KEEP_SETTINGS_ALERT_MESSAGE;NETFRAMEWORK45;NEW_FIRMWARE_VERSION;OLD_COMP_PATH;OLDER_VERSION_DELETING_REGISTRY_APPDATA_ON_UPGRADE;OLDER_VERSION_WITH_ELGATO_PLUGINS;OLDERVERSIONBEINGUPGRADED;UNPLUG_MESSAGE;USAGE_STATISTICS_ACCEPTED_VALUE;WIN10FOUND;WIX_DOWNGRADE_DETECTED;WIX_UPGRADE_DETECTED;WIXUI_INSTALLDIR MSI (s) (D8:D4) [13:05:46:702]: Windows Installer installed the product. Product Name: Elgato Wave Link. Product Version: <IP_ADDRESS>6. Product Language: 1033. Manufacturer: Corsair Memory, Inc.. Installation success or error status: 0.

(Automated response - build 656.)

Hi @krystofkrticka,
The installation isn't unattended. Is there an installer switch to have the package install silently?
(Automated response - build 656.)

@stephengillie Oh this might help since it uses wix installer which is compiled to .msi file so I could try usual switches.

@stephengillie just tried adding /quiet /qn /norestart but it still returns code 3010 which means that it installed successfully but restart is needed. When I tried promptrestart it just rebooted the computer without asking.

@stephengillie So I guess I have to probably add expected response code in manifest.

@stephengillie I have added switches for installer and expected return code but it still says that installer failed because the reboot is needed to finish.

Automatic Validation ended with:

Installation failed with exit code -1978334967 2024-01-25 15:22:35.929 [CLI ] ShellExecute installer failed: 3010 CAQuietExec:  Error 0x80070001: Command line returned an error. CAQuietExec:  Error 0x80070001: QuietExec Failed CAQuietExec:  Error 0x80070001: Failed in ExecCommon method CustomAction ClearFirewall returned actual error code 1603 but will be translated to success due to continue marking Property(S): DRIVERINSTALLFAILED_MESSAGE = Driver installation failed. Please reboot and start setup again. Property(S): DRIVERINSTALLFAILED_AUTO_MESSAGE = Driver installation failed. Please reboot and Setup will be started again. Property(S): ErrorDialog = ErrorDlg Property(S): SecureCustomProperties = DELETE_APPDATA_FILES_ACCEPTED_VALUE;DRIVERINSTALLFAILED_AUTO_MESSAGE;DRIVERINSTALLFAILED_MESSAGE;EXE_FILE_NAME;INSTALLDIR_PRODUCT_FAKE;INSTALLER_FILE_NAME;IS_DRIVER_UP_TO_DATE;IS_ELGATO_WAVE_DEVICE_CONNECTED;KEEP_SETTINGS_ALERT_MESSAGE;NETFRAMEWORK45;NEW_FIRMWARE_VERSION;OLD_COMP_PATH;OLDER_VERSION_DELETING_REGISTRY_APPDATA_ON_UPGRADE;OLDER_VERSION_WITH_ELGATO_PLUGINS;OLDERVERSIONBEINGUPGRADED;UNPLUG_MESSAGE;USAGE_STATISTICS_ACCEPTED_VALUE;WIN10FOUND;WIX_DOWNGRADE_DETECTED;WIX_UPGRADE_DETECTED;WIXUI_INSTALLDIR MSI (s) (88:48) [15:22:35:758]: Windows Installer installed the product. Product Name: Elgato Wave Link. Product Version: <IP_ADDRESS>6. Product Language: 1033. Manufacturer: Corsair Memory, Inc.. Installation success or error status: 0.

(Automated response - build 657.)

@wingetbot waivers Add Validation-Installation-Error

Despite returning an error, this package installs and launches normally in a Windows 10 VM.

That's exactly what I saw when testing it with Windows Sandbox which is opened by YamlCreate.ps1 script.


Fix TorchText examples

Stack from ghstack:

#74
#73
-> #72

There is a bug that HTTPResponse doesn't have correct read method. We need to use __iter__ to get data. Then, the bytes are also needed to be decoded as string.
cc: @Nayef211
Differential Revision: D31797766

@ejguan has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


AWS Glue JOB Keep Running While after giving result using pydeeque

Team,
When I am using pydeeque in glue job. after giving results. the job keeps running.

code:
import sys
import os
from awsglue.transforms import *
from awsglue.utils import getResolvedOptions
from pyspark.context import SparkContext
from awsglue.context import GlueContext
from awsglue.job import Job
from pyspark.sql import SparkSession, Row
import pydeequ
from pydeequ.checks import *
from pydeequ.verification import *
args = getResolvedOptions(sys.argv, ['JOB_NAME'])
sc = SparkContext()
glueContext = GlueContext(sc)
spark = glueContext.spark_session
job = Job(glueContext)
job.init(args['JOB_NAME'], args)
df = spark.sparkContext.parallelize([
Row(a="foo", b=1, c=5),
Row(a="bar", b=2, c=6),
Row(a="baz", b=3, c=None)]).toDF()
check = Check(spark, CheckLevel.Warning, "Review Check")
checkResult = VerificationSuite(spark)
.onData(df)
.addCheck(
check.hasSize(lambda x: x >= 3)
.hasMin("b", lambda x: x == 0)
.isComplete("c")
.isUnique("a")
.isContainedIn("a", ["foo", "bar", "baz"])
.isNonNegative("b"))
.run()
checkResult_df = VerificationResult.checkResultsAsDataFrame(spark, checkResult)
checkResult_df.show()
job.commit()
Logs:

I just ran into the same thing and resolved it by following wrapping up section


Setting the rate parameter in VLC URL

I have a question about Vlc.DotNet.
Generic information

Vlc.DotNet version : (<IP_ADDRESS>)
Vlc.DotNet project used : (Web Application)
libvlc version : (x86)
.net version : (4.5)
Project language : (C#)
Project build architecture : (x86/x64)
Operating system : (Windows ...) (x86/x64)

Summary
I have a Url which is working fine in vlc command line      vlc --rate=4 "rtsp://username:password@ipAddress:port/someUrl"
But I want to use this in my one of core c# project where I am using VLC dll.
I have tried some ways those are :

                       string Url= "rtsp://username:password@ipAddress:port/someUrl";

                       string[] options = { "--rate="+ rate };
                      _videoControl.Play(new Uri(fastForwardUrl), options);

2
                      string Url= "rtsp://username:password@ipAddress:port/someUrl";

                      string[] options = { ":rate="+ rate };
                      _videoControl.Play(new Uri(fastForwardUrl), options);

                    string Url= "rtsp://username:password@ipAddress:port/someUrl";

                    _videoControl.SetMedia(new Uri(fastForwardUrl));
                    PlaybackRateVal = (float)rate;
                   _videoControl.Rate = PlaybackRateVal * 1.0f;
                   _videoControl.Play();

I tried all these ways, but I am not able to set the rate of the video, basically, I want to do fast forward in my Playback video.

VlcMediaPlayer.Rate =... should work. Provide full logs please.

also try "--rate", "2"

also try "--rate", "2"

I have tried this, I am not getting any error in the log, but when I play my recorded video on fast forward mode, it stopped the video.
If you have any sample code, can you please share

You created the issue, so you should share your full code and your full verbose logs (even if you see no error)


Retention policy schedules too many operations

As you can see in the logs, operations are consecutively scheduled to execute in a very short period.
The retention config:
retention:
  period: 3 days
  cutoff_reference: publish_timestamp
  schedule: "* 1 * * * *"

Interesting, isn't it? :)
2022-10-15T18:01:00.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:00.101Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=898.286024ms
2022-10-15T18:01:01.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:01.058Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=941.039419ms
2022-10-15T18:01:02.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:02.056Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=943.73079ms
2022-10-15T18:01:03.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:03.053Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=946.186865ms
2022-10-15T18:01:04.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:04.052Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=947.44181ms
2022-10-15T18:01:05.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:05.101Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=898.168433ms
2022-10-15T18:01:06.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:06.051Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=948.238708ms
2022-10-15T18:01:07.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:07.052Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=947.082531ms
2022-10-15T18:01:08.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:08.053Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=946.707415ms
2022-10-15T18:01:09.002Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:09.056Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=943.95755ms
2022-10-15T18:01:10.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:10.066Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=933.944936ms
2022-10-15T18:01:11.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:11.055Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=944.979769ms
2022-10-15T18:01:12.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:12.069Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=930.012833ms
2022-10-15T18:01:13.002Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:13.059Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=940.276412ms
2022-10-15T18:01:14.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:14.057Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=942.718308ms
2022-10-15T18:01:15.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:15.065Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=934.24559ms
2022-10-15T18:01:16.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:16.102Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=897.70154ms
2022-10-15T18:01:17.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:17.056Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=943.719544ms
2022-10-15T18:01:18.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:18.062Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=937.844451ms
2022-10-15T18:01:19.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:19.066Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=933.652239ms
2022-10-15T18:01:20.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:20.052Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=947.539197ms
2022-10-15T18:01:21.000Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:21.054Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=945.158588ms
2022-10-15T18:01:22.001Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-execute-operation index_id=quickwit
2022-10-15T18:01:22.060Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit" scheduled_in=939.891696ms

I skipped some lines but at the end, we finally get the right scheduling:
2022-10-15T18:01:59.059Z  INFO quickwit_janitor::actors::retention_policy_executor: retention-policy-shedule-operation index_id="quickwit-fts-staging" scheduled_in=3540.940524636s

After investigating this, I have noticed the cron crate iterator semantic is not correct. So calling next() during the execution period yield the same period (the difference makes up in millis). That's why it stabilizes after a minute.
There is an interface to get the interval after(datetime) but you need to know how much to advance otherwise it could skip. I have opened an issue and trying to see how I can fix this.

@fmassot I think the scheduler is just doing what is requested. With the help of @guilload this is correct for the schedule * 1 * * * *  meaning it should run every 1st minute or every hour.

Ok... thank you for the feedback @evanxg852000 and I believe you're very right. I'm closing this issue, I will be more careful when reporting new issues.

@fmassot Well, reporting is a good thing even with the smallest suspicion. It helps makes sure we are doing the correct. I also learned more about parsing cron expressions, and how to convert & schedule them by exploring some crates.


Add support for proxy and other features in TelegrafOptions

@alexeynobody Good PR 🙏, thx! It's gonna take me a little time to look at the code. I'll answer you soon.


add linked_list and linked_list_from

Closes #3. Note that I preferred using "push_back" instead of "push_front".

Looks good, thanks!


Fixed missing "border" parameter in TexImage2D & TexImage2DAsync (WebGLContext)

Fixes Issue https://github.com/BlazorExtensions/Canvas/issues/50
The current implementation of texImage2D throws an exception on call because its missing the "border" parameter.
Since border is specified to always be 0 it could be set by default but I did not want to deviate from the parameter count used in standard WebGL.

I'm getting
WASM: Microsoft.JSInterop.JSException: Failed to execute 'texImage2D' on 'WebGLRenderingContext': parameter 9 is not of type 'ArrayBufferView'.
Now from using your branch.

could someone please merge this already?

2ndwolf :  In my case, I was using a byte[] for the data, which was passed as a string to CompanyContextManager.ts deserialize.  I added else if (typeof (object) === "string" &&
(method === "bufferData" || method === "bufferSubData" || method === "texImage2D")) {
and that got past that issue, still working thru other rendering setup.

I'm experiencing the same issue as @2ndwolf. I would also really like this missing parameter issue sorted, but I think more work is needed than what's in this PR.


Request charset defaults to UTF-8, Response charset defaults to ISO_8859_1

Both the request and response objects should default to the same charsets.  This causes very hard to troubleshoot bugs with repeated requests.
from com.google.api.client.http.HttpResponse
public Charset getContentCharset() {
    return mediaType == null || mediaType.getCharsetParameter() == null
        ? Charsets.ISO_8859_1 : mediaType.getCharsetParameter();
}

from com.google.api.client.http.AbstractHttpContent
protected final Charset getCharset() {
    return mediaType == null || mediaType.getCharsetParameter() == null
        ? Charsets.UTF_8 : mediaType.getCharsetParameter();
}

Ideally both will use Charsets.UTF_8, as Charsets.ISO_8859_1 only supports a subset.

Per the HTTP 1.1 spec, if the server does not supply a charset in the HTTP headers, then ISO-8859-1 is assumed. Thus on the request, I think we should return Charsets.8859_1 instead of Charsets.UTF_8.

This issue may no longer be relevant due to its age. Feel free to re-open.


Use ESMF::ESMF CMake target

ESMF's FindESMF.cmake now has a CMake target called ESMF::ESMF which has the benefit of avoiding ambiguity between library name and target name when invoking target_link_libraries(). If MAPL can move to using this target name, then when UFS Weather Model/GOCART imports MAPL, we can avoid the spurious -lESMF flag we've been seeing because of a mismatch in the case of the target name.
I'm happy to help however I can with making this happen, but I don't know MAPL or its build system very well.

@mathomp4 as discussed

@AlexanderRichert-NOAA I'm working on it now. There will be a series of steps needed, but hopefully can get a release of MAPL 2.44 out soon (tomorrow?). We have a hotfix first needing to go in, so that wins.


Add synthesis inner join tests

Summary: Adds the spark inner join tests to check if there's difference between velox and spark
Reviewed By: funrollloops
Differential Revision: D32458856

Can anyone kindly approve and merge this change? Thanks cc @funrollloops @mbasmanova

Can anyone kindly approve and merge this change? Thanks cc @funrollloops @mbasmanova

This change has been merged some time ago. Not sure why GitHub doesn't reflect that. I'd just close this PR.
git show 9ef16dc
commit 9ef16dc28ab5b29824a946cdaf554cccaa85a10b
Author: Yating Zhou<EMAIL_ADDRESS>Date:   Fri Nov 19 18:50:20 2021 -0800

    Add synthesis inner join tests

    Summary: Adds the spark inner join tests to check if there's difference between velox and spark

    Reviewed By: funrollloops

    Differential Revision: D32458856

    fbshipit-source-id: a9bade61555a6d1a4c1948b6266567f1e55342b0

diff --git a/velox/exec/tests/utils/OperatorTestBase.h b/velox/exec/tests/utils/OperatorTestBase.h
index a23e977..d06dd55 100644
--- a/velox/exec/tests/utils/OperatorTestBase.h
+++ b/velox/exec/tests/utils/OperatorTestBase.h
@@ -102,6 +102,12 @@ class OperatorTestBase : public testing::Test {
     return vectorMaker_.rowVector(children);
   }

+  RowVectorPtr makeRowVector(
+      const std::vector<std::string>& childNames,
+      const std::vector<VectorPtr>& children) {
+    return vectorMaker_.rowVector(childNames, children);
+  }
+
   template <typename T>
   FlatVectorPtr<T> makeFlatVector(
       vector_size_t size,


Attacking with multiple labels

Hi, how may we use your attack with the textattack api to attack text that contains more than the 0,1 label? For example, if we have text that corresponds to the labels 0,1,2,...etc.

figured it out via setting model config


Amazon Alexa Reviews Analysis (NLP)

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
I've added the related notebook and have also updated the README.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
This PR is with reference to issue #6818.

This looks good to me but can you please add the link to the dataset in a markdown cell of the notebook at the beginning of it. Rest seems perfect.

Please check now. @bhav09


Disorder of SCWatermarkOverlayView color space

Hi! @rFlex
Long time no contact.
I add subview in SCWatermarkOverlayView initWithFrame function.
        UIView *view = [[UIView alloc] initWithFrame:CGRectMake(0, 0, 100, 100)];
        view.backgroundColor = [UIColor redColor];
        [self addSubview:view];

But the redColor will be change like blueColor,  maybe add UIImageVIew to SCWatermarkOverlayView will be change the image color space.
I guess is a problem of color space, this due with function:
https://github.com/rFlex/SCRecorder/blob/master/Library/Sources/SCAssetExportSession.m#L146
and this line:
https://github.com/rFlex/SCRecorder/blob/master/Library/Sources/SCAssetExportSession.m#L155
Change color picture:

I am also eager to find a solution to this issue, I am trying to overlay text+emojis and am finding that on export the emojis colours are slightly different.

Hi！@twomedia
How to solve?
Jack

Haven't found a solution yet @xhzengAIB :(

solve~
http://stackoverflow.com/questions/9920836/color-distortion-in-cgimagecreate

Thank You @fuchen2624! That fixed it!  :D

Merged #207
Can you confirm the issue is fixed?

@rFlex I can confirm this is fixed, colors are now rendering correctly.

Thanks @twomedia @rFlex


Add custom visit tracking to front-end

Problem Description:
SiteImprove is not correctly tracking activity in LUX.
Expected Behavior/Solution:
Add code to front-end to allow custom event tracking.   https://help.siteimprove.com/support/solutions/articles/80000448441-siteimprove-analytics-custom-visit-tracking
Requirements:
Analytics Requirements
Needed for promotion:
If an item on the list is not needed, it should be crossed off but not removed.

[ ] ~Wireframe/Mockup~
[ ] ~Committee discussions~
[ ] Feasibility/Team discussion - Sarah
[ ] ~Backend requirements~
[ ] Frontend requirements- TBD
[ ] Questions
List of questions for discussions. Answers should be documented within the issue.

UAT/LUX Examples:

TBD

Dependencies/Blocks:

Blocked By: Issues that are blocking the completion of the current issue.
Blocking: Issues being blocked by the completion of the current issue.

Related Github Issues:

Issues that contain similar work but are not blocking or being blocked by the current issue.

Related links:

#51

Wireframe/Mockup:
Place wireframe/mockup for the proposed solution at end of ticket.

closing. dupe of #124


Withdraw 22.2.4 from stable channel

For  an upgrade regression. 23.0.1 is not affected.
Update 21.0.8 is not affected either, only 22,

this is outdated


Improve support for Windows High Contrast Mode and Internet Explorer 11

This builds on work by @chris-gds for the accordion component refresh in #2257.
What
Add media feature -ms-high-contrast to media queries that change styles for Windows High Contrast Mode.
Add CSS property -ms-high-contrast-adjust where property forced-color-adjust is used.
For example, see draft PR #2290
Why
We can port fixes for High Contrast Mode such as improved focus states for radios and checkboxes (https://github.com/alphagov/govuk-frontend/pull/2264) that currently work in Edge / Chrome / Opera but not Internet Explorer.
According to the 2018 WebAIM survey of users with low vision, a higher percentage use Internet Explorer than Edge.
Who needs to know about this
Devs
Done when

[ ] GOV.UK Frontend has support for Windows High Contrast Mode and Internet Explorer 11
[ ] Documented how to add tweaks for Windows High Contrast Mode and other forced colour modes in future

It would be nice if autoprefixer dealt with this for us, however it looks like this functionality won't be added: https://github.com/postcss/autoprefixer/issues/1351
We could add a mixin for Windows High Contrast Mode (as suggested by @chris-gds), however even in that case we would need to be careful when using it to make sure we include prefixed CSS properties such as forced-color-adjust/-ms-high-contrast-adjust.
Also, there may be scenarios where we want to use the forced-color media query but not the -ms-high-contrast media query.


ListRow with with multiple item rows

Hi, i'm using RowsSupportFragment to have multiple rows on a fragment with presenters.
For one of the rows i want it to be like a verticalgridfragment (multiple rows and columns of items).
The effect i want is the one in row3:
|----------------------------|
|    Row 1                   |
|    Item Item ... Item      |
|                            |
|    Row 2                   |
|    Item Item ... Item      |
|                            |
|    Row 3                   |
|    Item Item ... Item      |
|    Item Item ... Item      |
|    Item Item ... Item      |
|    Item Item ... Item      |
-----------------------------

I've seted the num of rows of the List row presenter to 6, it presents multiple rows, but when i scroll down the fragment does not scroll to show the rows that are not visible, like it does when i scroll to a different ListRow.
Is there anyway to do it?
Thanks in advance

Hey Jose, I'm not quite sure what you mean. If the issue is that Row 3 is so large that it overflows on the bottom when in focus, then you can call RowPresenter.setFacet[1] to center it on the screen but likely that won't be enough since 6 rows will be too large to fit on a screen.
If what you are trying to do is nested vertical scrolling... I would recommend not to do that from a UX perspective. In theory it should be possible if you set the height of Row 3 to something smaller than its actual height.
[1] https://developer.android.com/reference/android/support/v17/leanback/widget/Presenter.html#setFacet(java.lang.Class<?>, java.lang.Object)

Hi @owahltinez , thanks for your answer .
I'll make some tests using the setFacet method you suggested.
What i'm trying to do is to have the same effect of having 6  or more different rows, of the same object type, but without the complexity of creating multiple adapters and having the navigation like a grid.

I want to do the same thing!

It sounds like this is more of a general implementation question than a question with these specific examples. StackOverflow (http://stackoverflow.com/questions/tagged/android-tv) is a better forum for those types of questions because you'll have a much larger audience to offer help, so I'm going to close this out.


Do not skip Coveralls job, ever

This rolls back a part of 8722d689955a859d6b7a81b7369b3f81fcd4cedf. The
problem is that on.push.paths-ignore doesn't just skip the job, it
doesn't even start it (that's a documented behaviour, nothing wrong
here). This doesn't play well with the fact that Coveralls job is
required, i.e. a PR can't be merged without it.
The default job.<name>.if mechanism would let us skip the job
properly, but it doesn't provide a ready-made way to check which paths
changed. Third-party actions don't seem to cover this either.
I ran out of interest looking into this, so I'm just rolling back the
changes. Perhaps I'll revisit this later.

This is prompted by #1692 and #1693, and is related to #1660.

Coverage remained the same at 59.123% when pulling 1fdaef0b8b7231d83948a966ea076ccfdb8acc6d on bugfix/roll-back-gha-skips into 7d6c83493f2296b455e87d5335afdb23a3d36c52 on master.


Clicked option passed on change plus option click event

Selected option will be passed as extra data to the change event function. In addition the option click event will be triggered as well in case there are listener for that option. Useful for validations.
Example: Have to disable/enable certain options depending on the selected one, either by adding a click listener to one option or using the second parameter in the change event should be easier to test and validate what other options should be disabled or enabled.

This breaks the multiple select. With your solution, when you select an option, the dropdown closes

Fixed it using $option.triggerHandler("click") instead of $option.click(). I did a little example here.


embed DroidSans font

This uses the same method as ImGui.

I improved premake to set the working directory automatically. So loading the font is easier now.


proposal: explicit interface implementation

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.10.3 linux/amd64
Does this issue reproduce with the latest release?
N/A
What operating system and processor architecture are you using (go env)?
N/A
What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
Tried to implement two different interfaces with the same type, where the two interfaces share a method name, but require that name to have different semantics.
What did you expect to see?
That is impossible
What did you see instead?
That is possible.
The easiest way to do this is the same as done in C#: allow an interface to be implemented explicitly, so the method is only available when cast to the interface type.

Actually it is possible, by providing a second method in the interfaces that is different, to make it clear. It can even be done like this:
type Fooer interface {
   Foo()
   ImplementsFooType1()
}

type Fooer interface {
  Foo()
  ImplementsFooType2()
}

Sorry, this is an explicit design feature of Go and isn't going to change.
The number of types that implement both Cowboy and Artist in the wild and get confused by a Draw() method are super rare.


Hyperlinks for samples

Add the option to add hyperlinks for samples to i.e., HuBMAP Data Portal datasets.
Would work better if the sample information is not a hover, but a click for expanding window.

Perhaps we could enable this on a more generic level by allowing createRowHref and createColHref functions to be provided to the component? This would avoid specifically tying this to HuBMAP and enable more configurability for users.
e.g. for the HuBMAP datasets case, we would be able to pass in
createRowHref={(row) => `portal.hubmapconsortium.org/browse/datasets/${row}`}

We'd be able to pass in a similar function to link to cell types' ontology definitions:
createColumnHref={(col) => `https://www.ebi.ac.uk/ols4/search?q=${col}&ontology=cl`}


Removed inaccurate docstring from dispatcher.py and deprecated an unused...

... argument from disconnect.

Buildbot, test this please.

My opinion is that a deprecation period doesn't offer much benefit in a case like this as you can remove the weak parameter from an existing Django project without breaking backwards compatibility with older versions of Django.

You implemented a special fast deprecation (deprecated in 1.9, removed in 2.0). If our dear release manager accepts it, we could then start the deprecation in 1.8 to have a "normal" deprecation period.

Updated patch in https://github.com/django/django/pull/3981.


Add GitHub starter workflow

Add the default buf-ci.yaml workflow as a starter workflow. See: https://docs.github.com/en/actions/using-workflows/creating-starter-workflows-for-your-organization

Does checking this in actually make this work somewhere? By my reading of the docs you linked, for a user to take advantage of these files, they need to create a .github repository in their organization and copy these files there. Do you have a different understanding?
If I am correct, it seems like we actually want is documentation how to do that rather than just checking in these files.

It should become part of the workflow templates on clicking new workflow in the GitHub actions UI: https://github.com/bufbuild/buf-action/actions/new
I'm not sure if it's available outside of the buf org, that may require some manual step from GitHub.

Can you please research this a bit more, and verify what actually needs to happen to make this work? Everything I can find points to this will not work as is.

The docs clearly state that if you want to share a workflow within an organization, it needs to defined inside of a .github repository (not in a .github/ directory inside of a GitHub action).

Starter workflows can be created by users with write access to the organization's public .github repository. These can then be used by organization members who have permission to create workflows.

It seems like this is the repo you would need to submit a PR to if you want this to appear to other orgs: https://github.com/actions/starter-workflows

Sorry, this was my misunderstanding. Closing needs more research.


[FEATURE REQUEST] Worldborder to keep players inside the map

[FEATURE REQUEST] Worldborder to keep players inside the map
Description: Add a worldborder to the map to prevent players from going outside the boundaries of the playing zone. This ensures that players stay within the playing area and that the game is fair.

Thank you for the suggestion! This feature will be included in the upcoming v2.0.1 release, where player movement will be restricted using a PlayerMoveEvent listener instead of a world border.
Commit: e8c3bdc

Thanks for implementing the requested functionality! Seems like a good solution. I look forward to seeing this feature soon in the upcoming v2.0.1 release.


[Endpoint] API Endpoint to Fetch Patient List for Each Station

We need an API endpoint that returns a list of patients assigned to a specific station. This endpoint will be used on the frontend to display the patient list for each station.
URL: /stations/{station_id}
For the time being, I have hardcoded the patient data under "frontend/views.py ---> def station_patient_list(request, id)" to facilitate frontend development, but we will need an API endpoint to retrieve the actual data from the backend.

Also add a parameter checked_today if the patient has been categorised today.


Segment.java

This algorithm is used to find the maximum in given range query.

Thank you for contributing to @AllAlgorithms✨ Make sure that you ⭐ the project 🤓, and continue helping us!
It seems you have opened a pull request on the wrong repository, you have implemented an algorithm in Java, and according to #1 Go is the programming language for this repository.
You can always feel free to open a new pull request on the Java Algorithms.


Age of Empires 2 + Userpatch

Hi. Work perfect, but with UserPatch (https://userpatch.aiscripters.net/) after start new game, game change resolution and vsync is disabled. Becouse resolution main menu fixed (800*600). i am about full screen mode.

So everything works fine when the game changes the resolution but just vsync is disabled?
Can you create a log file and upload it please: https://github.com/CnCNet/cnc-ddraw/issues/44

yes

The problem could be that vsync is skipping some frames. It's actually running at 59 FPS and not 60.
Try this instead:
nonexclusive=true
vsync=false

Did some improvements related to vsync, here is the latest build:
ddraw.zip
Closing this now


feat: add EntityPresentationApi implementation using simple icons a…

…nnotation

Created PR to use EntityPresentationApi in catalog-graph, see https://github.com/backstage/backstage/pull/25852


removed dependency on react-static-container

closes #384

Thanks 👍

Any idea when this will make it into the next release?


(PA-1169) Add checks for hiera config files that are version 5

@johnduarte updated

These tests seems like they should go in puppet-agent, not puppet, since its puppet-agent that does this deployment

CLA signed by all contributors.

@MosesMendoza Ok, I'll take a look at moving the test to puppet-agent


[TE] Added helm deployment for ThirdEye

Description
Creating a helm application for ThirdEye to work with pinot quickstart
right off the box. The current change installs MySQL with the initial
set of tables. This creates thirdeye frontend and backend pods
Caveats

The initdb.sql needs to be updated regularly which is an overhead. To be refactored later.
Using the create-schema.sql is not possible at the moment since the script has errors with MySQL 5.7.

No backward incompatible changes.

This is good. I had a few additional comments:
It would probably make sense to have 3 types of installation :
frontend, worker and scheduler : So users should be able to install these 3 kinds of nodes either by changing something in values.yaml or by passing params to helm. This is convenient as each of these 3 node types could live in different VPC's or be provisioned in different env's.
Could the mysql initialization be treated as an independent concern -- because its a one-time thing. It may not be a bad thing to assume that the db is already available and initialized and not treat DB upgrades via helm at all. DB install / backup / schema migration is an independent and mature concern with its own tools like liquibase. In theory, thirdeye should be able to work with  any database, maybe even NoSQL ones.
The bulk of operation load (IMHO) comes because we want to scale or change the config of some existing cluster, we want to increase the size of the back end, add new config variables, override old ones, support new datasources or detection algos. This would help upgrade / scale up a cluster to keep pace with code changes and load requirements.

Hi @devinay
I like all your comments. And I think we are aligned on most of the issues. Think of this as the alpha version of the deployment. As we go forward, we'll streamline configuration and focus of scalability.
Some thoughts:

The intent of this distribution is to help new/existing users with a template to kickstart ThirdEye with the minimal amount of effort
Going forward, the goal is to make this distribution ready for production
Regarding MySQL:  We want people to be able to plugin their existing SQL service in the future. However, currently we are just trying to make things simple for new users so that they can jumpstart their application with minimal effort.
Regarding scheduler, worker: Agreed. This is an alpha version. We'll be enhancing this further. PR's welcome :) However, long term, we are also looking at architectural simplifications that could make TE more reliable. For example, having a distributed scheduler.