Specifying AdditionalDataColumns prevents logging of ASP.Net Core requests

I have only just started working with Serilog and have installed both Serilog.AspNetCore v2.1.1 and Serilog.Sinks.MSSqlServer v5.1.2 to my ASP.NET Core 2.1 project. My configuration is largely as per the documentation.
When my logger configuration includes a ColumnOptions with no configuration, I receive logging as expected during startup and for each HTTP request.
However, as soon as I specify any additional columns - even setting AdditionalDataColumns to an empty collection, I am receiving no logging via SQL Server for HTTP requests post-startup. Console logging continues to work fine.
For example, this code includes the logging for startup and my test use of the additional column. The test log entry is written correctly, but I receive no further logging after startup:
var columnOptions = new ColumnOptions
{
	AdditionalDataColumns = new Collection<DataColumn>
	{
		new DataColumn {DataType = typeof (string), ColumnName = "scheme"}
	}
};

Log.Logger = new LoggerConfiguration()
	.MinimumLevel.Debug()
	.MinimumLevel.Override("Microsoft", LogEventLevel.Information)
	.WriteTo.MSSqlServer(Settings.Value.ConnectionString, "SystemLog", columnOptions: columnOptions)
	.Enrich.FromLogContext()
	.WriteTo.Console()
	.CreateLogger();

try
{
	Log.Information("hi there {@scheme}", "testing");
	Log.Information("Starting web host");
	var webHost = BuildWebHost(args);

	... snip ...

I've enabled Serilog's console debugging logging, but can't see anything in the console which indicates anything is untoward.
Further investigation
My service was exclusively using endpoints created using IdentityServer4 as I am building an OIDC authorisation server.
I have added middleware to log HTTP requests. When I start the service and issue requests against a non-IdentityServer controller, these are all logged. However as soon as I issue a subsequent request against an IdentityServer endpoint, neither that request nor further requests against my controller are logged.
To clarify this behaviour only occurs when AdditionalDataColumns is present (even as an empty collection).
Given that requests to my controller are logged fine in both situations, it looks like something which IdentityServer is logging which is somehow tripping up this sink.
Let me know if you'd like a sample project or any other details.

Before we get into it any deeper, please use one of the 5.1.3-dev packages. There is no support for Microsoft.Extensions.Configuration in 5.1.2, versions prior to 5.1.3 were only compatible with .NET Framework's XML-based ConfigurationManager.
I also have an IdentityServer4 OIDC and ASP.NET Core client sites (all 2.0, we canceled the project before upgrading to 2.1) and I didn't see the issues you're describing, so I can at least reassure you it's possible to have it all working well...

I've just tried with 5.1.3-dev-00202 and all works as anticipated; thanks. Do you have a rough feel on when 5.1.3 will be released?

I don't have any control over packaging, sorry.

No worries. Will wait and see 😄


Common Test Resources - SceneCollections, Browser Sources, Images, etc.

If we had a common sceneCollection file in the test resources, then we could assert that requests such as getSources etc. return the expected objects. Would be a great first step to containerization-based integration tests

@LarsBaertschi I've pushed a start onto topic-30, would you mind taking a look and letting me know if the README and initial instructions make sense and work for you?
If all is well, then I'll add a few more tests to exercise the features E2E.

Hey, I just had a look. I like it, everything is clear to me! It's a very nice way of testing.

Awesome, thank you for your feedback! I'll fill it out more and make a PR at some point.


ROX-11564 replay tests with alerts

Description
This PR aims to provided testing for sensor's re-sync behaviour using pre-recorded k8s events.
The events were recorded using the local-sensor tool.
Create the policies.json file containing the policies:
curl -k -u admin:$ROX_ADMIN_PASSWORD -H "Content-Type:application/json" https://$ROX_CENTRAL_ADDRESS/v1/policies > $tmpf
# and then
# for each id
curl -k -u admin:$ROX_ADMIN_PASSWORD -H "Content-Type:application/json" https://$ROX_CENTRAL_ADDRESS/v1/policies/$id >> $fname

Create the trace.jsonl file containing the k8s events.
go run tools/local-sensor/main.go -record -record-out=trace.jsonl -resync=0s

Create the central-out.bin file containing sensor's output events.
go run tools/local-sensor/main.go -replay -replay-in=trace.jsonl -resync=10s -delay=0s -format=raw -central-out=central-out.bin -with-policies=policies.json

Checklist

[ ] Investigated and inspected CI test results
[x] Unit test and regression tests added
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
Manually testing with:
LOGLEVEL=panic go test -race -count=1 github.com/stackrox/rox/sensor/tests/replay

Disabling the re-sync by changing resyncTime := 1 * time.Second to resyncTime := 0 * time.Second will make the test fail.

Part of this PR is addressed here #2483

I see some rebase is needed here - please ping me for a review once the conflicts are resolved

I see some rebase is needed here - please ping me for a review once the conflicts are resolved

Yeah. The idea is to merge #2483 first and then rebase this one.

/test go-unit-tests-release


Fatal in alias index: word 46631 not exist

[INFO] [2016-06-16 04:22:00] Actual Model capacity: 186 MB, Alias capacity: 301 MB, Delta capacity: 256MB
[INFO] [2016-06-16 04:22:00] INFO: block = 0, the number of slice = 2
[INFO] [2016-06-16 04:22:00] Server 0 starts: num_workers=1 endpoint=inproc://server
[INFO] [2016-06-16 04:22:00] Server 0: Worker registratrion completed: workers=1 trainers=8 servers=1
[INFO] [2016-06-16 04:22:00] Rank 0/1: Multiverso initialized successfully.
[INFO] [2016-06-16 04:22:01] Rank 0/1: Begin of configuration and initialization.
[INFO] [2016-06-16 04:22:26] Rank 0/1: End of configration and initialization.
[INFO] [2016-06-16 04:22:26] Rank 0/1: Begin of training.
[DEBUG] [2016-06-16 04:22:26] Request params. start = 1, end = 162931
[INFO] [2016-06-16 04:22:27] Rank = 0, Iter = 0, Block = 0, Slice = 0
[DEBUG] [2016-06-16 04:22:27] Request params. start = 162932, end = 1099301
[INFO] [2016-06-16 04:22:27] Rank = 0, Alias Time used: 1.77 s
[INFO] [2016-06-16 04:23:01] Rank = 0, Training Time used: 270.57 s
[INFO] [2016-06-16 04:23:01] Rank = 0, sampling throughput: 21328.182632 (tokens/thread/sec)
[INFO] [2016-06-16 04:23:08] doc likelihood : -8.291677e+08
[INFO] [2016-06-16 04:23:08] word likelihood : 2.141731e+09
[INFO] [2016-06-16 04:23:08] Normalized likelihood : -7.755637e+09
[INFO] [2016-06-16 04:23:08] Rank = 0, Evaluation Time used: 59.11 s
[INFO] [2016-06-16 04:23:08] Rank = 0, Iter = 0, Block = 0, Slice = 1
[FATAL] [2016-06-16 04:23:09] Fatal in alias index: word 447 not exist
[FATAL] [2016-06-16 04:23:09] [FATAL] [2016-06-16 04:23:09] [FATAL] [2016-06-16 04:23:09] [FATAL] [2016-06-16 04:23:09] Fatal in alias index: word 429 not exist
Invalid topic assignment 934165176 from word proposal
[FATAL] [2016-06-16 04:23:09] Fatal in alias index: word 1202 not exist
Fatal in alias index: word 2893 not exist
[FATAL] [2016-06-16 04:23:09] Invalid topic assignment 412339115 from word proposal
[FATAL] [2016-06-16 04:23:09] Fatal in alias index: word 46631 not exist
Fatal in alias index: word 2946 not exist
[FATAL] [2016-06-16 04:23:09] Fatal in alias index: word 46631 not exist
[FATAL] [2016-06-16 04:23:09] Fatal in alias index: word 429 not exist

Can't find alias index when training Slice2.

This should be the same problem with issue https://github.com/Microsoft/lightlda/issues/15


Editor not overflowing properly

Lately I've noticed some issues with the overflow of the editor. Tabs won't scroll, the find and replace panel goes off the side of the screen. It almost looks like the editor isn't aware of the width of the project files panel. See the screencap of the find and replace panel as I adjust the files panel and notice how the buttons get pushed way off the side of the screen. The same thing happens with tabs:

I haven't had time to dig into this yet but I'll try and find the properties that are causing this, but wanted to get this out there.
I'm running 1.9.0-beta0

This has been fixed and will be out in the next 1.9.0 beta.

Thanks!


Update docker files

I've installed docker-ce, and pulled the repo.  When I run docker-compose up, I get the following:
nginx-php7.2_1  | In Connection.php line 108:
nginx-php7.2_1  |
nginx-php7.2_1  |   You cannot use the "Symfony\Component\Messenger\Bridge\Amqp\Transport\Conne
nginx-php7.2_1  |   ction" as the "amqp" extension is not installed.
nginx-php7.2_1  |
nginx-php7.2_1  |
nginx-php7.2_1  | doctrine:schema:create [--em EM] [--dump-sql]
nginx-php7.2_1  |
nginx-php7.2_1  | 2021-11-17T13:22:07+00:00 [info] User Deprecated: Since doctrine/doctrine-bundle 2.4: The "connection_override_options" connection parameter is deprecated
nginx-php7.2_1  | Ready 🚀 http://localhost:8100/app_dev.php/

I'm new to docker, so I'm not sure how to fix this.  The fact that it's directing to app_dev.php indicates that it's an older version of Symfony.  Digging around a little bit, I also see that it's copying parameters.yml, also old.
I was also hoping that this whole project could be updated to at least PHP 7.4, but php8.0 would be even better.
The main reason I want to use docker is because I don't have MySQL and Mercure installed locally, so though that this would be a good time to use docker.

That's right, Docker images are outdated. They need to be updated


Support StreamgingIngestionPolicy on both database and tables

Overview
This change adds the support for streaming ingestion policies in delta kusto. Streaming ingestion policies are available on both tables and database, this change supports both entities - db and tables.
Streaming Ingestion policy has two fields - IsEnabled and HintAllocatedRate. Model used here adds both. However, I have observed that sometimes there are more fields in Streaming Ingestion Policy but since I could not find them documented anywhere, I have not considered them when implementing this feature.
More details about streaming ingestion policy: https://learn.microsoft.com/en-us/azure/data-explorer/kusto/management/streamingingestionpolicy
Testing

Local end-to-end testing through Integration Tests (Created a test ADX and 3 test DBs).
Unit Test for Command Parsing and Delta Computation Logic.

Issue #43 - Delta on StreamingIngestion

@vplauzon Looks like I don't have access to see why push-state-to-db and other checks failed. Can you help me with the access or show me the logs/root cause of their failure.

Integrated in 0.14.1.
Thank you for your contribution!


Binding string parameter as PDO::PARAM_INPUT_OUTPUT does not work.

Using string parameters as PARAM_INPUT_OUTPUT produces unexpected results when calling a stored procedure, and in some situations crashes PHP completely. It never works as expected when results are fetched. Here's the table of the behaviour of the string input / output parameter in various situations:

Outcome

Stage
Parameter included in SELECT
Parameter NOT included in SELECT

Before execute()
Unchanged
Unchanged

After execute()
\u0000 added every second character
\u0000 added every second character

With fetch()

After fetch()
Unchanged (back to  original)
Unchanged (back to original)

After closeCursor()
Error 500 (C:\Program Files\PHP\v7.0\php-cgi.exe - The FastCGI process exited unexpectedly)
Unchanged (back to original)

Without fetch()

After closeCursor()
Changed correctly
Changed correctly

The stored procedure:
ALTER PROCEDURE [dbo].[sp_Test]
	@pOutputString nvarchar(255) = NULL OUTPUT,
	@pOutputNumber bigint = NULL OUTPUT,
	@pInputString nvarchar(255) = '',
	@pInputNumber bigint = 0
AS
BEGIN
	SET NOCOUNT ON;
	SELECT @pOutputString = 'You have entered "' + @pInputString + '" as the input.'
	SELECT @pOutputNumber = 666 - @pInputNumber

	SELECT
		'Procedure completed' AS [Result],
		@pInputNumber as [InputNumber],
		@pInputString AS [InputString],

		-- The following two were commented out for 'Parameter NOT included in SELECT' outcomes.
		@pOutputNumber AS [OutputNumber],
		@pOutputString AS [OutputString]
END

And the PHP used for testing:
<?php

header('Content-Type: text/plain');

$dsn = "<our DSN>";
$username = "<user name>";
$password = "<password>";

print("Getting database...\n");
try {
  $pdo = new PDO($dsn, $username, $password);
}
catch(PDOException $e) {
  print("Error: " . $e->getMessage() . "\n");
}

try {
	$sql = "
EXEC sp_Test
	@pOutputString = :out_string,
	@pOutputNumber = :out_number,
	@pInputString = :in_string,
	@pInputNumber = :in_number
";

	$query = $pdo->prepare($sql);

	$in_string = 'Hello world!';
	$in_number = 600;

	$out_string = '';
	// If it's left as an empty string, even weirder things are happening.
	for ($i = 0; $i <= 255; $i++) {
		$out_string .= ' ';
	}
	$out_number = 0;

	print("\n" . json_encode(array(
		'Output string' => $out_string,
		'Output number' => $out_number,
		'Input string' => $in_string,
		'Input number' => $in_number
	), JSON_PRETTY_PRINT));

	$query->bindParam(':in_string', $in_string);
	$query->bindParam(':in_number', $in_number);

	$query->bindParam(':out_string', $out_string, PDO::PARAM_STR|PDO::PARAM_INPUT_OUTPUT, 255);
	$query->bindParam(':out_number', $out_number, PDO::PARAM_INT|PDO::PARAM_INPUT_OUTPUT, 4);

	print("\nExecuting query...");
	$query->execute();

	print("\n" . json_encode(array(
		'Output string' => $out_string,
		'Output number' => $out_number,
		'Input string' => $in_string,
		'Input number' => $in_number
	), JSON_PRETTY_PRINT));

	print("\nFetching results...");
	$results = $query->fetchAll();
	print("\nResults: " . print_r($results, TRUE));

	print("\n" . json_encode(array(
		'Output string' => $out_string,
		'Output number' => $out_number,
		'Input string' => $in_string,
		'Input number' => $in_number
	), JSON_PRETTY_PRINT));

	print("\nClosing cursor...");
	$query->closeCursor();

	print("\n" . json_encode(array(
		'Output string' => $out_string,
		'Output number' => $out_number,
		'Input string' => $in_string,
		'Input number' => $in_number
	), JSON_PRETTY_PRINT));
}
catch(Error $e) {
  print("Error: " . $e->getMessage() . "\n");
}

To produce the table of outcomes, I was commenting out some parts of the code, both in the stored procedure, and in the PHP script.
When the server returns error 500, the catch block is not executed and no errors are logged in the PHP log even though error reporting is set to E_ALL.
I'm using PHP 7.0.8 with sqlsrv extension 4.0.8610.3 (the latest).

@karol-pilch Did you run into the same issue with our PHP 5.6 driver?

Off the top of my head, the 5.6 driver was also filling in the string with
\u0000 characters, but would be updated with the changed string from the
procedure when using fetch(), and most importanty it wouldn't cause the
error 500.
On Friday, 8 July 2016, Meet Bhagdev<EMAIL_ADDRESS>wrote:

@karol-pilch https://github.com/karol-pilch Did you run into the same
issue with our PHP 5.6 driver?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/msphpsql/issues/108#issuecomment-231424799, or mute
the thread
https://github.com/notifications/unsubscribe/AIjBgdr5X2PP15ZuYT1JKk_uqPz9UrIiks5qTowPgaJpZM4JG2Q2
.

@karol-pilch We believe we fixed this issue in our latest release: https://github.com/Azure/msphpsql/releases/tag/4.1.0.
Can you help us validate?

Yep, it looks fine now. As you said in the release notes, the string variable bound to the output parameter only gets substituted after the call to closeCursor(), but at least the script doesn't die.
Thanks very much!


[FEA]:  Progress bar for python client

Is this a new feature, an improvement, or a change to existing functionality?
New Feature
How would you describe the priority of this feature request
Significant improvement
Please provide a clear description of problem this feature solves
The NV-Ingest cli client displays a progress bar that shows real time ingestion progress in terms of files ingested along with metrics like time per file and pages per second when performing ingestion jobs. We now have the ability to ingest multiple files at a time with the python client, but it doesn't display progress like the cli. This becomes an issue for large jobs where it's hard to determine the job's progress.
Describe the feature, and optionally a solution or implementation and any alternatives
I would like for the python client to display similar progress information as the cli when performing multi file ingestion jobs.
Additional context
No response

I'm not entirely sure this is something we want to offload to the client API, since its primary use case is to interact with the ingest service. One thing we may want to look at is to ensure that its easy to query which jobs have completed, so that if someone wants to wrap a batch call with tqdm they can do that easily.


Release 0.1.2a

Coverage increased (+98.2%) to 98.174% when pulling ea2200284ad1faf45d4a22592de87f67e396a34f on release-0.1.2a into dc84ed1d77c7ba02879a22f3c0d89bbe8e191093 on master.


[Bug] Device Time Issue

Hi, thanks for your impressive project. While running the benchmark I encountered with several issues concerning the device time.

'No Internet Connection' because device time is incorrect: It seems that the device time will be initialized before running every benchmark task, and it can cause 'No Internet Connection' in chrome, as the screenshot shows.

Task date mismatch. For some tasks like 'SportsTrackerActivityDuration', a specific time will be assigned to instantiate the task template. However, I often find the generated task objective is inconsistent with the device time. For example, the task here is "How long was my skiing activity October 12 2023 in the OpenTracks app? Express your answer in minutes as a single integer." And in the app the only skiing activity is in yesterday, October 14th(the device time is Oct 15th). This makes the agent confused about where to find the required activity.

So how can I avoid these issues? Thanks!

Hi! Thanks for reporting. we will take a look. Can you please confirm that you’re using a Pixel 6 emulator?

Hi! Thanks for reporting. we will take a look. Can you please confirm that you’re using a Pixel 6 emulator?

Yes, I exactly followed the steps in README and the setup video :)

If I turn on 'Set time automatically' in the system setting, it will be able to connect to the internet. But the time will be reset to October 2023 when running evaluation.

For the sports tracker issue, I see a skiing activity on Thursday (5th one down). Is that not on October 12?

@StarWalkin , I have submitted a fix for the browser date time mismatch. Now for browser tasks, we set the date time as the current date time, which will remove that Chrome warning.
Regarding the Sports Tracker issue, as @clink42 mentions, in this case the answer should be the 5th row labeled "Outdoor Adv...", so this appears to be working as intended. Please let us know if you still believe there is a bug. Thank you.

@crawles @clink42 I'm very sorry for mistakenly raising part of the issue due to my carelessness. The time-related problems have all been resolved now. Thank you very much for your prompt response and fix!

So glad to hear and no worries!


fix: lower a bit the market filters breakpoint

Description
Lowering the breakpoint from XL to LG so we don't add an extra click to some in-between screen sizes

Issue (if applicable)

follow-up of https://github.com/shapeshift/web/pull/8082#pullrequestreview-2416983146
Risk

High Risk PRs Require 2 approvals

What protocols, transaction types, wallets or contract interactions might be affected by this PR?

Testing
Go to the market page and play with the screen width to see when the filters are aggregated into a single dropdown

Engineering
n/a

Operations
n/a

[ ] :checkered_flag: My feature is behind a flag and doesn't require operations testing (yet)

Screenshots (if applicable)

No longer seeing the wrapped dropdown text, however just pointing out that One Click DeFi Assets now wraps where previously it didn't, and the breakpoints for the card size seem different where we now display less larger cards at certain breakpoints instead of more slightly smaller cards like previously.

I don't see any differences with cards?
The wrapping you're talking about is on the top left text right? Let's give product a few time to check it, seems like particularly on a specific screen size, we might want a few beard oil if needed
merging this and happy to do anything as follow up if @shapeshift/product feels like


Adding another API style

Meta issue: discussion so far seems to have unearthed two possible usage patterns.
Using dataframe columns as a namespace
This was the original design and is what the documentation and examples focuses on. In this mode, we expect all data relating to a given index to reside in one dataframe (e.g. sparse pairwise index for an assignment problem, with data for costs and limits). It then seems natural to create new variables as additional columns so that they live with relevant data which can be used when building constraints.
Using python variables as a namespace
In this mode, various series of vars and data exist separately as different python variables. To create constraints, the user needs to align them as dataframes before using the accessors. In this situation, convenience functions which create constraints from aligned series are a better pattern.
Relevant discussions on other issues:

https://github.com/Gurobi/gurobipy-pandas/issues/39
https://github.com/Gurobi/gurobipy-pandas/issues/42

There are example of both styles in the docs. For reference:

Project-team allocation shows creating all variables as series using the free functions
L1 norm regression shows building all variables and constraints in the method chaining style using accessors

There are example of both styles in the docs

OK, but earlier you wrote this:

what's left is to add a page of documentation for each style to show pros/cons (typing support, data/variable separation, method chaining, maintaining series alignment)

Does that exist?  If not, do you have an issue to track it?

Yes, there is a section in the reference covering typing, and the usage page covers both styles. If you feel something is missing, feel free to raise an issue or PR.

Yes, there is a section in the reference covering typing, and the usage page covers both styles. If you feel something is missing, feel free to raise an issue or PR.

Took a look, and I think what you have is good for now.  It will be interesting to see how people react to this new package.


Easier Select Old Years in Calendar

This is a (multiple allowed):

[ ] bug
[x] enhancement
[ ] feature-discussion (RFC)
Framework7 Version: 1.4.2
Platform and Target: All

What you did
Want to select date for birthdate using calendar for an old date 50+. Have to click the previous year button 50+ times
Expected Behavior
Have a way to select the year and then replace the calendar with a selection of years and previous next for other years. Something similar to the first example of the datepicker at http://www.eyecon.ro/datepicker/
Actual Behavior
There is no problem with the current operation just time consuming when entering older dates.

Hi @amaster507,
my dirty solution was create two more buttons like "<<" and ">>".
Then when a user click on "<<", i call:
$$('.left10').on('click', function() {
          for (i = 0; i < 10; i++) {
![calendar](https://cloud.githubusercontent.com/assets/12820154/20632404/633b9e08-b324-11e6-8b64-2b580f48a7c5.png)

            myCalendar.prevYear();
          }
        });

And when click on ">>" I run:
$$('.right10').on('click', function() {
          for (i = 0; i < 10; i++) {
            myCalendar.nextYear();
          }
        });

.right10 and .left10 are classes attached to respective buttons.
This is for 10 years, but you can adjust to yours needs.

@7h3m4dS0u1 could you open a PR perhaps?

I also like the idea from http://www.eyecon.ro/datepicker/


Add CBMC proof of initializeLocalBuffer

Add CBMC proof of initializeLocalBuffer function
Description
This PR adds files required for the CBMC proof of initializeLocalBuffers function. Since the test function initializeLocalBuffer only changes the fields of the otaAgent struct, there is no requirement to have a proof for such function but to have completeness in regards to the CBMC proofs in OTA library and also to avoid any future memory unsafe additions to the function; this proof is added to the repo.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is formatted using Uncrustify.
[ ] I have read and applied the rules stated in CONTRIBUTING.md.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Couldn't this be covered in a proof of OTA_Init?

Couldn't this be covered in a proof of OTA_Init?

Yes, it could've been covered in that. The proof of this function is included just for the sake of completeness.


Release v4.0.7-1

Looking at release notes, I think it might be worth adding that Electron 27 upgrade implies that "macOS 10.13 (High Sierra) and macOS 10.14 (Mojave)" are no longer supported.


Problems with bytea performance

I have the below simple query:
return client('models')
    .first('glb')
    .where({
        assets_id: assetId
    })
    .then(function (row) {
        if (defined(row)) {
            return row.glb;
        }
    });

glb is a binary type (bytea since I'm using postgres 9.5) and contains a single row where the glb column has an entry that is 28630077 in bytes (~27MB).  The query takes 13305 ms to run (time until then is called) and the Node process (not the DB process) maxes out the CPU while the query is running). If I query for the assets_id column instead of the glb column, it only takes 2 ms.
Running the same query with the same data from the psql command line completes almost immediately:
time psql -A -c "select glb from models where assets_id=16" master3 postgres > out.glb

real    0m0.679s
user    0m0.000s
sys     0m0.031s

Here's the table setup for completeness
return knex.schema.createTable('models', function (table) {
    table.integer('assets_id').primary().references('assets.id').onDelete('CASCADE');
    table.binary('glb').notNullable();
});

Am I doing something wrong?  Or is there a performance issue with knex and bytea column types?  I'd be happy to debug this myself if someone can point me in the correct direction.  This is using the latest released version of knex.  Thanks.

Could you please check out if the query is also really slow when using node-postgres driver directly?

Thanks.  I can confirm that using node-postgres directly has the same issue and running time.
I decided to also  check pg-native  while I was at it and it's quite a bit faster (~440ms), but from what I can tell, it's no longer possible to use pg-native with knex (and I'm not sure if that's a viable strategy to begin with).
Does this mean I'm out of luck? Any guidance you can provide would be greatly appreciated.

Thanks for checking.
I think that better place to discuss it would be node-postgres issue tracker. This sounds like one of the rare cases where using pg-native is actually reasonable (if node-postgres cannot be fixed to work faster).
Support for pg-native was dropped, because it didn't support all API's that knex uses... maybe that decision could be re-evaluated...

Thanks, I opened https://github.com/brianc/node-postgres/issues/1286 to hopefully get some guidance from them.  Feel free to close this unless you want to use it as a discussion point for re-adding pg-native support.
Assuming pg-native ends up being my only option, I would be happy to help re-add native support, otherwise I'm not sure what my options are.

@elhigu, great news.  I was able to track down the problem and open a PR: https://github.com/brianc/node-packet-reader/pull/3  Assuming that gets merged and published, it looks like it will be a significant improvement for node-postgres read performance all-around.

@mramato I just read the node-postgres issue through, thats just plain awesomeness that you were able to track it down! Hope that it ends up to upstream soon. I'm closing this now, but let me know if there is anything more I can do for this.

Thanks, and thanks for all of the help.

This has been merged:

https://github.com/brianc/node-postgres/issues/1286
https://github.com/brianc/node-packet-reader/pull/3

It's now fixed in node-pg 6.2.2 - any chance we can get a version bump on the dependency in knex soon?

@hansef there's no need for a knex version bump because pg is an assumed sibling dependency, not a direct one.  Just make sure you depend on the latest pg in your own application and you will pick up the change.  knex only has a dev dependency on "pg" for it's own unit tests and it's not part of the production app.


Add HowLongToBeat Plugin

JavaScript Plugin. Uses HowLongToBeat API wrapper to quickly check their times

This can now use the new nodejs env setup automation. I will create a merge request to the repo soon.

@Tueska PR ready here https://github.com/Tueska/fl-HowLongToBeat/pull/2 please merge, afterwards I will also merge this in.

@Tueska PR ready here Tueska/fl-HowLongToBeat#2 please merge, afterwards I will also merge this in.

Merged your PR. Thanks a lot, didn’t know it was possible that way!

Yeah feature was released recently, I haven't got around to update the docs yet.


Updating README to mirror documentation for NEARStudio

I suggest we update the readme to mirror the documentation found here: https://docs.nearprotocol.com/quick-start/near-studio-ide-intro
Most people aren't going to want to run NEARStudio locally unless they're looking to make a contribution, so we should move the run locally instructions to a separate README or to a contributors guide or something.

I think README should include these docs vs just mirroring, so that on top there are user-oriented docs and on the bottom it's stuff for contributors, credits, etc
BTW, it's also possible to automate snippet extraction. E.g. there is such script in package.json, but for now only used for 1 of templates.
    "templates:build-template-docs": "mdinject -w --root=./templates/. --docsroot=./templates/. --sourceext=\".ts|.js|.html\" --targetext=\".md\"",


seller property of runAdAuction argument

For the moment I'm not planning to implement this because it seems to be redundant, but filing an issue so this is on our radar in case the spec changes later.

For buyer_signals.seller will you provide the origin of decision_logic_url?

I assume you mean browser_signals.seller? I was going to say yes, but then I realized at least for the initial implementation round there won't be a decision_logic_url because all the decision algorithms are checked into the repo and the caller will just provide an algorithm name. So I guess at least initially that will be no.
When the flag is enabled to use fetched logic, then it could make sense to provide the origin of decision_logic_url. When we add reporting, it would be possible to provide the origin of that URL, but I'm not sure that would actually make sense.
Sellers can identify themselves through auction_signals, but the browser can't vouch for the correctness of the identity provided there, so it doesn't belong in browser_signals.
Maybe we should file a bug against the FLEDGE spec to get clarification on what's supposed to be going on there? Is that spec being actively worked on right now?

Sorry, yes, browser_signals.seller in generate_bid arguments.
The origin of decision_logic_url is still relevant, for reporting.  I think that's enough?


Order "Rank breakdown" best rank (language) first

Is your feature request related to a problem? Please describe.
I find it hard too see in the percentage list what my best languages are.
Describe the solution you'd like
Use my pre-written Ruby code to properly order the Codewars profiles language ranking percentages.

Additional context

┆Issue is synchronized with this Clickup  by Unito

Seems to be fixed now. Thank you !!


Tweak peer disconnect logic

When the node is above its max peers limit, it disconnects from randomly selected addresses and adds them to the restricted peers list. There is no reason to restrict these peers as they haven't misbehaved in any way.

Rebased.

The restriction is intentional, so that peers with misbehaving logic will not intentionally reconnect immediately, perhaps in an attempt to DOS the sync nodes. Note that the restriction is only in place for a few minutes.


Update WebTools tech. overview and setup

Is there an existing issue for this?

[X] I have searched the existing issues

What kind of idea is it?
Add or revise existing page
Who is it for?
Administrator
Which area (SuperOffice entity) is this related to?
Document
Description

SignalR protocol replaced
ProtocalLogFile no longer in use

Update WebTools tech. overview and setup


create token

resolves #187

@arekkubaczkowski there are still some differences. Can you please make sure that both platforms have the same amount of params? And address details are not mapped on iOS. PTAL

Android

{
  "livemode": false,
  "bankAccount": null,
  "card": {
    "address": {
      "postalCode": "77063",
      "country": "US",
      "line2": "Texas",
      "state": null,
      "line1": "1459  Circle Drive",
      "city": "Houston"
    },
    "last4": "3220",
    "funding": "Credit",
    "expMonth": 12,
    "currency": null,
    "expYear": 2022,
    "brand": "Visa",
    "country": "IE"
  },
  "type": "Card",
  "created": "1621435180000",
  "id": "tok_1IsqYiG7gfqYEkPOw3a7kUfC"
}

iOS

{
  "card": {
    "expYear": 2022,
    "brand": "Visa",
    "name": null,
    "currency": null,
    "expMonth": 12,
    "last4": "3220",
    "country": "IE",
    "address": {
      "city": null,
      "country": null,
      "phone": null,
      "postalCode": null,
      "line2": null,
      "email": null,
      "line1": null,
      "name": null,
      "state": null
    },
    "funding": "Credit"
  },
  "id": "tok_1IsqVoG7gfqYEkPOu0HufCwa",
  "created": "1621435000000.0",
  "livemode": false,
  "bankAccount": null,
  "type": "Card"
}

Hi tem, Is create token params wrong from this site? https://stripe.dev/stripe-react-native/api-reference/interfaces/card.createtokenparams.html


IRGen: Don't drop method descriptors for vtable-elided internal methods.

We still use the method descriptor for key path identity and other uses. Fixes rdar://problem/66280170.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - c4032c94b00265d8e32fb8290bc5e061166d67b7

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - c4032c94b00265d8e32fb8290bc5e061166d67b7

Build failed
Swift Test OS X Platform
Git Sha - c4032c94b00265d8e32fb8290bc5e061166d67b7

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - cbe07c9795aecf6d8bdf71bc10b81327f8317716

@rxwei @dan-zheng I noticed a problem with derivative functions' method descriptors here. They were getting mangled the same as the method descriptor for the original method, and silently colliding before (and now crashing when they end up becoming distinct definitions instead of aliases after this patch). I disabled emitting method descriptors for derivative functions here, since I figured you probably don't need them to work with library evolution yet. Is that OK?

Build failed
Swift Test OS X Platform
Git Sha - cbe07c9795aecf6d8bdf71bc10b81327f8317716

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - f386f06b3ee94cb903196b0a7d82ca4c075d32cd

Build failed
Swift Test OS X Platform
Git Sha - f386f06b3ee94cb903196b0a7d82ca4c075d32cd

I disabled emitting method descriptors for derivative functions here, since I figured you probably don't need them to work with library evolution yet. Is that OK?

That sounds good, thanks! I think there's also a duplicate derivative function method descriptor TBDGen issue that we haven't looked into yet: SR-13125.

@swift-ci Please test

@dan-zheng I'm having trouble finding the AutoDiff/validation-test/class_differentiation.swift test that's still failing in CI here. Is it in a different repository from the compiler?

The test exists on master branch: test/AutoDiff/validation-test/class_differentiation.swift. It may have been added to master branch after this PR was created, you should be able to access the test if you pull from master.

Build failed
Swift Test OS X Platform
Git Sha - e0a232f7af13f27a760d27f46b769309cb1abf30

Build failed
Swift Test Linux Platform
Git Sha - e0a232f7af13f27a760d27f46b769309cb1abf30

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - e0a232f7af13f27a760d27f46b769309cb1abf30

Build failed
Swift Test OS X Platform
Git Sha - e0a232f7af13f27a760d27f46b769309cb1abf30

@dan-zheng Since this test was added recently, would it be OK if I disabled it temporarily, so that we can get this correctness fix in for other code?

That sounds good! If you have any ideas on what's causing the test failure and how to eventually fix it, that would be quite helpful.

Sure, I'll look into it once I land this.

Ultimately, I think you'll want to feed the derivativeIdentifier from the SILDeclRef through into the LinkEntity for the derivative's MethodDescriptor and MethodOverrideDescriptor, so we can include the identifier in a distinct mangling for these method entries.

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - e0a232f7af13f27a760d27f46b769309cb1abf30

Build failed
Swift Test Linux Platform
Git Sha - e0a232f7af13f27a760d27f46b769309cb1abf30

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 4b8cf4ee950efff4866ed6e14f9bdaf7c524837a

Build failed
Swift Test OS X Platform
Git Sha - 4b8cf4ee950efff4866ed6e14f9bdaf7c524837a

@dan-zheng I think I worked out an approach that keeps the existing AutoDiff test behavior working for now, while still fixing the problem I'm trying to address for other code.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 7e76b1c9bf104ee0b1a8f927acb8ab1b9d3913f3

Build failed
Swift Test OS X Platform
Git Sha - 7e76b1c9bf104ee0b1a8f927acb8ab1b9d3913f3


Route53 DNS01 Validation does not work in split horizon dns setups.

Describe the bug:
if --dns01-self-check-nameservers= is used, only a portion of the code uses the specified DNS server.
There's a step that gets the authoritative NS for the zone and uses those dns servers to check if TXT record exist, in environments where access to internet DNS servers is not allowed, DNS01 validation doesnt work.
Expected behaviour:
Cert-manager uses provided dns01 servers for all the validations.
Steps to reproduce the bug:
Deploy cert-manager on a VPC w/o access to public dns and set --dns01-self-check-nameservers= to point to an internal DNS server.
Anything else we need to know?:
Our infrastructure is as follows.
foo.bar Route53 Private zone on account A.
foo.bar Route53 Public zone on account B.
Access to internet on port 53/UDP/TCP is not allowed in our vpcs / networks.
Workloads DNS records are written into foo.bar private zone.
In order to make dns01 validation to pass.

Deployed a DNS server that can resolve public foo.bar zone. certmanager writes TXT records there.
This DNS server will answer any A/AAA query to foo.bar with a bogus ip (<IP_ADDRESS>) I will call this records workload dns records, as the hostnames resolve to valid IPS in the private zone, lets use app.foo.bar for example purpose.

I identified a few problems with DNS01-Self validation and split horizon.

In a split horizon DNS setup, the workload dns records are not written to the public zone, the first check that certmanager does is wait for DNS to resolve app.foo.bar using the dns01 self check dns, and of course since dns01 server resolves the public zone, this first step will fail, I wonder if this step is valid, or can be skipped if --dns01-self-check-nameservers= is used.

Once app.foo.bar is present in the DNS server, it writes the TXT record once propagation is done, it determines the authoritative DNS servers for public zone foo.bar and tries to directly connect to those servers  https://github.com/jetstack/cert-manager/blob/master/pkg/issuer/acme/dns/util/wait.go#L95-L125 , and here's where completely fails when no access to outside dns servers.

In my own build I changed https://github.com/jetstack/cert-manager/blob/master/pkg/issuer/acme/dns/util/wait.go#L92 to use the  --dns01-self-check-nameservers=  and it works like a charm.
If this the correct way to fix this problem, if yes I can submit a PR.
Thanks!.
Environment details::

Kubernetes version (e.g. v1.10.2): 1.9
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): Tectonic AWS/Baremetal.
cert-manager version (e.g. v0.4.0): master.
Install method (e.g. helm or static manifests): helm.

/kind bug

@munnerz / @mikebryant any insight on this?
Thanks!.

Interesting scenario. The current behaviour is working for us. We have split horizon DNS, but the internal deployments have access to contact public DNS servers to check the validation. (And we use --dns01-self-check-nameservers to talk directly to google's dns, so we can bypass our internal split horizon.
The dns validation needs to check the correct TXT record has propagated to the public internet. The spec says Clients SHOULD NOT respond to challenges until they believe that the server's queries will succeed..
If we make all requests go to the specified server instead of the authoritative one, that could introduce extra delays (in that you may now need to wait for that recursive resolver to expire TTLs).
If we're not actually checking the real thing, maybe this is again back to the discussion of just disabling the self-check entirely, and just using a set delay?

I've opened #903 to try and track and consolidate some discussion on this.
Split horizon is pretty prickly right now. I'm keen to come to some resolution 😄

Re-Activating this issue as is not solved yet.

Closing this now, as the v0.6.1 release should not have any issues now and should properly honour the recursive-nameservers-only flag too.


Jison-based grammar

@ryngonzalez: Ready for your review!

Actually nevermind, I'm not yet satisfied with how the parsers are stacking up performance-wise.

@ryngonzalez: Ready for review again.

Could you rebase this?

TLDR:

Jison grammar definition is easier to read
:+1: to the speed improvement

LGTM after rebase!


Response: add Error storage, retrieval, conversion

This allows for robust creation of Responses directly from Errors, with error
capture for future reference, and retrieval via error() -> Option<&Error>.
Refs: #169
Refs: http-rs/tide#546
Refs: http-rs/tide#532
Refs: http-rs/tide#452
For posterity, see discussion in discord following: https://discordapp.com/channels/598880689856970762/649056551835009097/718227631421784084

Optionally also commit 2 (feel free to merge without if commit 2 is controversial) (https://github.com/http-rs/http-types/pull/174/commits/777f4637633941a4dc053718ffe1c5d560d4be29)

Backed out commit 2 after discussion: https://discordapp.com/channels/598880689856970762/649056551835009097/718265585602920468

Backed out 752e7e4 in favor of Clone with error: None + docs.

I've added set_error() here due to the discussion resulting in https://discordapp.com/channels/598880689856970762/649056551835009097/722181265435918469
This should cover all our bases. @yoshuawuyts Please take a look again when you are able to, I'd like to get this merged so that PRs on Tide (and soon Surf) can progress.


Update vessel version

Should the version be 0.6.4 here?
https://github.com/dfinity/motoko-base/blob/c174fc4bfc0287ccbfbd5a4b55edbc8669e3f2f3/.github/workflows/ci.yml#L11

Yeah! Thanks!
$ perl -pi -e s/v0.6.3/v0.6.4/g .github/workflows/ci.yml .github/workflows/package-set.yml


fix: install starship from edge channel

Starship moved to an edge-only release on Snap.
See: https://github.com/starship/starship/issues/4954

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


Upgrade cert-manager to v1.14.4

Trivial version change.
https://github.com/cert-manager/cert-manager/releases/tag/v1.14.4
Alternative: since we remove the webhook, can we drop it?

It was removed.


add extended attributes constants on NetBSD

Add three EA-related constants on NetBSD:

XATTR_CREATE
XATTR_REPLACE
EXTATTR_NAMESPACE_EMPTY

And include sys/xattr.h in the test.

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

Thanks! @bors r+

:pushpin: Commit 1422cee067e25f6336e730f7b71c75030659357b has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 1422cee067e25f6336e730f7b71c75030659357b with merge d99e55fd3dfb9621e805c21413ee2aa554ac9fcd...

:broken_heart: Test failed - checks-actions

Docker Linux Tier2 (sparc64-unknown-linux-gnu) timed out

@bors retry

:hourglass: Testing commit 1422cee067e25f6336e730f7b71c75030659357b with merge f7e6930ae8fbf97a0065e52884ee7c4eb920e77e...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing f7e6930ae8fbf97a0065e52884ee7c4eb920e77e to master...


[pre-sve] TC-DM-1.2 Accepted Command List Missing AddTrustedRootCertificate

Problem
Pre-SVE Week 2 Commit: 530e325359009cadbfc2752e16f588de436c90e1
~/chip-tool operationalcredentials read accepted-command-list 1 0
[1655165173.203615][2440:2445] CHIP:TOO: Endpoint: 0 Cluster: 0x0000_003E Attribute 0x0000_FFF9 DataVersion:<PHONE_NUMBER>
[1655165173.203738][2440:2445] CHIP:TOO:   AcceptedCommandList: 8 entries
[1655165173.203786][2440:2445] CHIP:TOO:     [1]: 0
[1655165173.203825][2440:2445] CHIP:TOO:     [2]: 2
[1655165173.203860][2440:2445] CHIP:TOO:     [3]: 4
[1655165173.203898][2440:2445] CHIP:TOO:     [4]: 6
[1655165173.203938][2440:2445] CHIP:TOO:     [5]: 7
[1655165173.203976][2440:2445] CHIP:TOO:     [6]: 9
[1655165173.204014][2440:2445] CHIP:TOO:     [7]: 10
[1655165173.204052][2440:2445] CHIP:TOO:     [8]: 11
[1655165173.204203][2440:2445] CHIP:EM: Sending Standalone Ack for MessageCounter:36998582 on exchange 36433i

Same test in with Pre-SVE Week 1 commit: 425f1fb916ca154deeb75cd82cd5196778aba152
[1655165133.194502][2432:2437] CHIP:TOO: Endpoint: 0 Cluster: 0x0000_003E Attribute 0x0000_FFF9 DataVersion:<PHONE_NUMBER>
[1655165133.194625][2432:2437] CHIP:TOO:   AcceptedCommandList: 9 entries
[1655165133.194671][2432:2437] CHIP:TOO:     [1]: 0
[1655165133.194710][2432:2437] CHIP:TOO:     [2]: 2
[1655165133.194747][2432:2437] CHIP:TOO:     [3]: 4
[1655165133.194784][2432:2437] CHIP:TOO:     [4]: 6
[1655165133.194817][2432:2437] CHIP:TOO:     [5]: 7
[1655165133.194853][2432:2437] CHIP:TOO:     [6]: 9
[1655165133.194891][2432:2437] CHIP:TOO:     [7]: 10
[1655165133.194929][2432:2437] CHIP:TOO:     [8]: 11
[1655165133.194966][2432:2437] CHIP:TOO:     [9]: 12
[1655165133.195129][2432:2437] CHIP:EM: Sending Standalone Ack for MessageCounter:8620394 on exchange 61674i

System Configuration
DUT: EFR32
SDK SHA: 530e325359009cadbfc2752e16f588de436c90e1

@yinyihu-silabs AddTrustedRootCertificate has id 0x0b, so 11.  It's right there in the list.
Id 12 was RemoveTrustedRootCertificate, which has been removed from the spec, so it's gone from the implementation as well.


localDevEnv.ps1 Error: Cannot find path ... because it does not exist.

Roughly 3 to 4 days ago, the use of localDevEnv.ps1 to build a local docker environment started to fail.
This script will create a docker based local development environment for your project.

NOTE: You need to have Docker installed, configured and be able to create Business Central containers for this to work.
If this fails, you can setup a cloud based development environment by running cloudDevEnv.ps1

All apps and test apps will be compiled and published to the environment in the development scope.
The script will also modify launch.json to have a Local Sandbox configuration point to your environment.

Error: Cannot find path 'C:\BMI Projects\OfficeProducts-AppSource\OfficeProducts-AppSource' because it does not exist.
Stacktrace: at ReadSettings, C:\Users\Michael\AppData\Local\Temp\tmp957E.tmp.ps1: line 526
at <ScriptBlock>, C:\BMI Projects\OfficeProducts-AppSource\.AL-Go\localDevEnv.ps1: line 58
at <ScriptBlock>, <No file>: line 1

The last part of the path is supposed to be "OfficeProducts" rather than "OfficeProducts-AppSource".  Has something changed recently that would now throw the calculation of the project name off?

If you can add me to the repo, then I can have a look at the structure of the repo to understand why this happens, thanks

Thanks and sorry - have a little too many things on my plate... - will have a look.

I have hotfixed the issue, should work again.

Thank you Freddy!  I can confirm this works.


window.tauri is undefined

Describe the bug
I have set up a tauri app with a rust backend, and it seems to be working fine. I can use window.__TAURI__.promisified to communicate with the backend. However, window.tauri is undefined, and I can't seem to find a good reason why it should be this way
Platform and Versions (please complete the following information):
[tauri]: running info
Operating System - Linux(5.4.0-60-generic) - linux/x64
Node.js environment
Node.js - 10.19.0
tauri.js - 0.14.0
Rust environment
rustc - 1.49.0-nightly
cargo - 1.48.0-nightly
tauri-bundler - 0.9.4
Global packages
NPM - 6.14.4
yarn - Not installed
App directory structure
/dist
/node_modules
/src
/src-tauri
App
tauri.rs - 0.10 (no lockfile) (outdated, latest: 0.11.0)
mode - embedded-server
build-type - bundle
CSP - default-src blob: data: filesystem: ws: http: https: 'unsafe-eval' 'unsafe-inline'
distDir - ../dist/ui
devPath - http://localhost:4000

We just released new versions. Can you please update everything?
tauri.rs - 0.10 (no lockfile) (outdated, latest: 0.11.0)

Note the outdated bit there...

We just released new versions. Can you please update everything?
tauri.rs - 0.10 (no lockfile) (outdated, latest: 0.11.0)

Note the outdated bit there...

updated to 0.11, same result

updated to 0.11, same result

I was probably being stupid, and should most likely be using import { emit, listen } from 'tauri/api/event' rather than the global object anyhow. Closing.

I was probably being stupid, and should most likely be using import { emit, listen } from 'tauri/api/event' rather than the global object anyhow. Closing.

As of Jan 2022 the imports are now import { app, fs} from "@tauri-apps/api" etc. This got me too the first time 😊

How are we supposed to deal with this ? I'm use readDir and window is not defined while I do not use window there

So what's the fix?

As of Jan 2022 the imports are now import { app, fs} from "@tauri-apps/api" etc. This got me too the first time 😊

How do you do this? Can yo give example?

    startBtn.onclick = () => {
      window.Tauri.invoke("start_tshark")
        .then((response) => {
          output.textContent = response;
        })
        .catch((error) => {
          output.textContent = error;
        });
    };

how do i fix window.Tauri not available in browser context?


scraping is very slow, performance issue?

What TMM version are you using?
tmm_3.0-SNAPSHOT_074c6f3_win.zip
release, pre-release, nightly, or directly from GitHub/branch?
nightly
What is the actual behaviour?
I need greater than 10 minutes to scrap a tv show with 168 episodes (e.g: "Ein Käfig voller Helden").
-- I allocated already, max memory -> 2560 MiB and it is still very slow.
What is the expected behaviour?
scrap a tv show lesser than 1 minute
Additional
Have you attached the Logfile from the day it happened?
Your Operating system? (win/mac/linux? version?)
Windows 10 x64, VDSL 100k

What scraper do you use?
Tvdb is sometimes very slow with their new api...

tvdb, but i do not think it's a tvdb problem because i tested it with an other tool and it works fast there.
I think it's a problem with collecting and  transfering the thumbnail images, mayby it is better to collect all thumbnails urls first, download the thumbnails all together at the end (e.g. with multi download options so that two or more images are downloaded at the same time) and then move the thumbs from the tmp folder to the specific series folder?

works here like a charm?
images are collected AFTER everything else has been done (async) - so there should be no problem with images

well I finally found the issue..
tvdb offers a single web service endpoint for all episodes now so we do not need to call 1 endpoint for every episode..
but the internal cache did not get fired, so we still needed to fetch every episode for its own.. should be fixed with the build tomorrow

The season images being downloaded very fast, but the thumbnails not. Look all sequential for me. Need over 1 Minute to scrap 24 Episodes. Here a video, that shows what i mean,
https://youtu.be/d5w4FtPzgLg


qemu: Use virtio-net-pci for our network driver

Signed-off-by: Samuel Ortiz<EMAIL_ADDRESS>

lgtm

Coverage decreased (-0.4%) to 67.268% when pulling b267d759c6230b49878229d215cee0605b0f770c on topic/networking into cd58b9082b15d1917a952773d81bd2f84eca9bc4 on master.


Cleanup the codebase, add documentation, explanations, etc...

Description in progress...

Some work-in-progress information got added by mistake, it will be completed later.

@nc0fr btw, do not care about the 60+ edited files. Just review the markdown files when they are updated. Thank you!

😭

Ship it to production 🚀🚀


about train

latents = vae.encode(pixel_values).latent_dist.sample()
running here error
RuntimeError: Expected weight to be a vector of size equal to the number of channels in input, but got weight of shape [128] and input of shape [128, 512, 512]
what causes this?
thanks

Sorry,I didn't serious look at the code. It was a framework issue


paymentItems is not reactive

BUG
If you change the value of paymentItems on a PayButton it doesn't update it internally.
Example
This Widget returns a ApplePayButton that will charge priceInCents.
However, If the parent widget changes the price (E.G. The customer selects a different tip amount) the payment will still show the original amount.
class PaymentButton extends HookConsumerWidget {
  const PaymentButton(
      {super.key,
      required this.priceInCents,
      this.onPressed,
      this.onSuccess,
      this.onError});
  final int priceInCents;
  final VoidCallback? onPressed;
  final VoidCallback? onSuccess;
  final void Function(Object?)? onError;

  @override
  Widget build(BuildContext context, WidgetRef ref) {
    final paymentItems = [
      p.PaymentItem(
          label: "Donation",
          amount: (priceInCents / 100).toStringAsFixed(2),
          type: p.PaymentItemType.total,
          status: p.PaymentItemStatus.final_price)
    ];

    return p.ApplePayButton(
      onPressed: onPressed,
      paymentConfiguration:
          p.PaymentConfiguration.fromJsonString(defaultApplePay),
      type: p.ApplePayButtonType.donate,
      onPaymentResult: (result) {
        runApplePay(result, amountInCents: priceInCents);
        if (onSuccess != null) onSuccess!();
      },
      paymentItems: paymentItems,
    );
  }
}

https://github.com/google-pay/flutter-plugin/assets/75931499/ee77d93a-1026-4060-a001-4c429475a82b
Temp Workaround
Assign the button a UniqueKey and change it when the priceInCents changes.
This will force Flutter to rebuild the entire ApplePayButton
(This is using flutter_hooks, but you could use something else to do this.)
 final buttonKey = useMemoized(() => UniqueKey(), [priceInCents]);

This is because onMethodCallHandler is only called once.
https://github.com/google-pay/flutter-plugin/blob/af2e70d121778337bf4bda9b5ef12ffbe4df664d/pay_ios/lib/src/widgets/apple_pay_button.dart#L153-L162

PR https://github.com/google-pay/flutter-plugin/pull/236 fixes this.
Please upvote

Published under pay_ios:1.0.11 and pay:1.2.0-beta02.


Remove extraneous installations in uv sync by default

Summary
First step of https://github.com/astral-sh/uv/issues/4358. uv sync will now remove any extraneous installations by default.

As a reference point, Poetry makes this opt-in, with poetry install --sync. However, exact sync by default with an opt-out seems like the correct default for uv sync.

When you omit the --sync option, you can install any subset of optional groups without removing those that are already installed. This is very useful, for example, in multi-stage Docker builds, where you run poetry install multiple times in different build stages.


Feature/mapping for mongodb odm

support mapping for mongodb

Can't move configs to module.config.php?

Needs rebase

What's the status?

MongoDB Mapping needs isEmbeddedDocument = true instead of isEmbeddedClass = true.

See also: https://github.com/zfbrasil/doctrine-money-module/pull/20

This PR can get closed, see https://github.com/zfbrasil/doctrine-money-module/pull/20 instead.


⚠ Turnos 24x7 has degraded performance

In 13a1031, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1561 ms

Resolved: Turnos 24x7 performance has improved in e0ccb10.


one-way-number-mask types one or many "0" characters in odd circumstances

I've written these two failing tests for one-way-number-mask-test.js.
  test('It does not input a zero when hitting "."', async function(assert) {
    await render(hbs`<OneWayNumberMask />`);
    await fillIn('input', '.');

    assert.dom('input').hasValue('');
  });

  test('It does not input a zero when hitting "m"', async function(assert) { // This also happens with the letter `k`
    await render(hbs`<OneWayNumberMask />`);
    await fillIn('input', 'm');

    assert.dom('input').hasValue('');
  });

Maybe the reason is the "shortcut" option. If you type "m", it's replaced by "000000". I cannot turn it off passing @options={{hash shortcuts=null}} though.


Option "mapFallbackValue" ignored when inherited to subcommand

With an @Option defined like this:
@Command(
    name = "main"
    subcommands = {
        SubCmd.class,
        CommandLine.HelpCommand.class
    }
)
public class Main {
    @Option(
            names = "-D",
            paramLabel = "<key=value>",
            mapFallbackValue = Option.NULL_VALUE,
            scope = ScopeType.INHERIT
    )
    private final Map<String, String> javaOpts = new HashMap<>();
}

@Command(name = "sub")
public class SubCmd {
    @ParentCommand
    private Main parent;
}

Running with args like ["sub", "-Dxyz"] results in error message:
Value for option option '-D' (<key=value>) should be in KEY=VALUE format but was xyz

This seems like it might be because the ArgSpec Builder copy constructor doesn't copy the "mapFallbackValue" field in picocli 4.7.3:  https://github.com/remkop/picocli/blob/v4.7.3/src/main/java/picocli/CommandLine.java#L9496

Thank you for raising this, and thank you for the analysis!
It’s very plausible that that’s the cause.
Will you be able to provide a pull request for this?

PR #2036 opened


database-ops.md created

duplicate #5 we can close one of these.

duplicate #6 we can close one of these.

not a duplicate. this one is a db guideline, the other is testing guideline.


Armenian translations of national indicator titles

The Armenia-specific indicators still need to be translated. They are actually available here so it may be just a matter of copy/paste. They need to be adjusted in this file.

@SDGArmenia This may be a simple copy-paste job, but I don't know enough with Armenian to know what to copy-paste. This will need to be done by someone on the team, or a translator.


Support generation of unsigned IPAs

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
🔑 I'm doing a blog series on creating a "build-once" deployment pipeline using fastlane. As part of this process, the signing of the IPA happens at deployment time, at the moment the configuration is known, rather than at build.
I'd like to propose the addition of an "unsigned" option for gym.
tl;dr I'm happy to contribute the PR, which would simply add "CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY=''" to the archive command when the "unsigned"option is true
As for why it's needed: It's currently possible without a flag, but hoops need to be jumped. Namely, xcodebuild's archive command requires a SIGNING_IDENTITY value, even if it's blank. However, providing a "signing_identity" option as a blank string also causes it to trigger the "--signed" argument for the export command.
The feature could also be implemented by having the export archive command generator deal with blank signing_identity values, but I feel that an explicit option would better express intent.
Thoughts?

I assume the build environment has no signing identity in the keychain at all. Is that the case?
Otherwise, it doesn't really matter how you sign the app initially, you can resign it afterwards, e.g. using sigh resign command.

@mgrebenets Correct, the idea is to have no signing identities involved at all until deployment, when resign is called.
But my issue is that fastlane actually makes it difficult to create an unsigned IPA for this purpose. It's still possible, via the workaround I mentioned above, but it would be cleaner to have an unsigned flag supported by gym.

I haven't tried it myself yet, what happens if you pass CODE_SIGNING_REQUIRED=NO to gym's xcargs option?
I've looked through the source (build_command_generator.rb).
If I read it correctly, using codesigning_identity does not set CODE_SIGNING_REQUIRED to YES, so using blank identity plus CODE_SIGNING_REQUIRED=NO added to xcargs should result into unsigned build.
Adding new unsigned option would be a convenience shortcut instead of using xcargs.

@mgrebenets That would indeed work, but the exporting half of gym sees a (blank) value for signing_identity and supplies a --sign value to xcodebuild exportArchive, which fails.
I did mention this in the original post, apologies if it wasn't clear.

@richardszalay I can't figure out what you workaround is.
I'm setting xcargs to "CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY=", but the exportArchive command fails.
I would, by the way, very much interested in this addition to gym.

@jjochen Blog series incoming (when I can get around to finishing it)
Are you passing anything at all as signing_identity to gym? The option needs to be absent. Failing that, what error are you seeing?
Feel free to create a stack overflow question, or email me<EMAIL_ADDRESS>

I've finally published my blog series, which might give some context as to why I wanted to do this - https://blog.richardszalay.com/2016/08/22/ios-deploy-pipeline-1-introduction/

@richardszalay Awesome! Thanks so much for sharing.


ll_cam.c:384:34: error: 'CONFIG_CAMERA_DMA_BUFFER_SIZE_MAX' undeclared (first use in this function)

I believe I followed your instructions correctly.  But I get this when trying to compile:
ll_cam.c:384:34: error: 'CONFIG_CAMERA_DMA_BUFFER_SIZE_MAX' undeclared (first use in this function)
     size_t dma_half_buffer_max = CONFIG_CAMERA_DMA_BUFFER_SIZE_MAX / 2 / cam->dma_bytes_per_item;
                                  ^
sketch/ll_cam.c:384:34: note: each undeclared identifier is reported only once for each function it appears in
exit status 1
'CONFIG_CAMERA_DMA_BUFFER_SIZE_MAX' undeclared (first use in this function)

Note, the zip I downloaded was last updated two dates ago (tree is at this commit: https://github.com/espressif/esp32-camera/tree/6a9497bbe909165663d958986e621e98dabcf994)

I notice that is defined here:
https://github.com/espressif/arduino-esp32/blob/master/tools/sdk/esp32/include/config/sdkconfig.h
Maybe that header isn't included but needs to be included?

according to these search results:
https://github.com/espressif/esp32-camera/search?q=CONFIG_CAMERA_DMA_BUFFER_SIZE_MAX
that CONFIG_CAMERA_DMA_BUFFER_SIZE_MAX is used but not defined in espressif/esp32-camera

it compiles if I copy
#define CONFIG_CAMERA_DMA_BUFFER_SIZE_MAX 32768
and paste it into the top of ll_cam.c
Maybe some of the driver code has changed.

and then it successfully compiles and I can flash to my ESP32-CAM and I can access the stream in my browser!

as an aside, this is how my test came out: https://www.youtube.com/watch?v=6Md69RYzKV8
(I have to admit I'm a little disappointed that it feels like a whole second from when a visual change in the camera propagates to my end browser.  I guess that is a result of how buffering is handled.)

Hi there. I hope you understand that this whole "procedure" that I concocted is just a dirty hack, not a proper way to include a submodule. It will have to potentially change every time Espressif changes something in there code.
As far as I am aware, unfortunately, this is so far the only way to make it work on Arduino with multiple clients, so we will have to continue to find ways to hack through new updates.
Thanks for finding this one. I will looking into it. Which esp32 board did you use?
Last comment: if the latest drivers do not work, you can always use my repo as-is. It works as of whatever date I last tested it.

ESP32-CAM written on front and ESP32-S written on back (purchased from https://www.amazon.com/gp/product/B08MJJTFN9)
My arduino IDE is set wtih:

when you say "make it work on Arduino" do you mean specifically "Arduino IDE"?  Are you suggesting it may be better to just forgo the use of Arudino IDE and instead use esp-idf?  (In that case I'm going to try with a clean fresh git repository but I will point to this and credit your code as an initial commit cause I don't know how to transfer git commits to a fresh repo)

No, I mean it is meant to work with Arduino Core. There is a lot more to it than just IDE.
And yes, if you want to use IDF environment, it is completely different, but please go ahead!


Fix array on no copy

When copy=False in array method, it changes shape of an original array. I fixed the method to use a view.
fix #2392

LGTM!


EXLM-2163 Personalized content placeholder block

Please always provide the Jira Issue your PR is for, as well as test URLs where your change can be observed (before and after):
Jira ID: EXLM-2163
Test URLs:

Before: https://main--exlm--adobe-experience-league.hlx.live?martech=off
After: https://exlm-2163--exlm--adobe-experience-league.hlx.live?martech=off

Please check the code quality check.


Fix settings render being called w/o component

Overview
To provide an in-settings guide for users, I used MarkdownRenderer.renderMarkdown() with markdown to more easily created the guide contents. However, I didn't fully understand the call and was getting errors:
pp.js:1 Error: Plugin "obsidian-jtab" is not passing Component in renderMarkdown. This is needed to avoid memory leaks when embedded contents register global event handlers.
at t.render (app.js:1:1276278)
at t.renderMarkdown (app.js:1:1276059)
at Il.display (plugin:obsidian-jtab:249:2362)
at t.openTab (app.js:1:2741344)
at HTMLDivElement. (app.js:1:2739783)
I'm not certain about this problem, but I guess it's not actually causing these bugs. It might be just some unrelated other issue with jTab? But I'm not 100 % sure this doesn't cause the actual problem.

Changes
Create a MarkdownRenderChild with the parent div so that the div can be cleaned up when the settings are closed.

This issue has been published with v1.2.3 package and is available for download/update in Obsidian Settings under Community Plugins.


Save the Baby Dinos: On sprite of kind overlaps otherSprite block's help is empty

Describe the bug
Save the Baby Dinos: On sprite of kind overlaps otherSprite... block's help is empty
Steps to reproduce the behavior
1.Navigate to  https://arcade.makecode.com/beta--skillmap#dino or https://arcade.makecode.com/beta--skillmap#beginner
2.Save the Baby Dinos-> Start.
3.Click Sprites Category in Toolbox.
4.Right-click on sprite of kind overlaps… block- >Help
Actual behavior:
On sprite of kind overlaps… block's help shows error
Expected behavior:
Shows help documentation.
Screen Recording
Beta version:

Live version:

Additional context:
1.OS: Windows 11
2.arcade version: 1.13.49
3.Microsoft MakeCode version: 9.3.2

The issue also reproduced in Traffic Dodger in https://arcade.makecode.com/--skillmap#dino, on sprite of kind overlaps otherSprite... blocks

This bug has different behaviors on live/beta version.


[docs] remove

[v] I have followed (at least) the PR section of the contributing guide.

@marsen The current demo looks correct. Why change it?

Ok, it looks like we don't need to mention the theme argument in this demo. TypeScript can infer the correct type. I'm not 💯 sure it's better, but we could probably document this independently 🤔

I don't understand the value of the change. Is it really simplifying that much the example? Developers can now provide any type to the spacing, like number, array etc.

OK closing

@marsen Thanks for the effort
@mnajdova my assumption is that type inferance was enough.


fix: don't warn on unused struct that has an abi annotation

Description
Problem
Part of #6250
Summary
Also renames a method to have a clearer name.
Additional Context
Documentation
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Ah, I forgot to do the same for globals, but I'll do it in a separate PR given that this was already reviewed and ready to go.


Error?

https://pastebin.com/2BHLx9MB

/ce verify shows no errors, latest version of CE and Paper 1.20.4

Doesn't provide any informative messages.


Migrate plugin to use new @nativescript/core imports

Make sure to check the demo app(s) for sample usage
Make sure to check the existing issues in this repository
If the demo apps cannot help and there is no issue for your problem, tell us about it
The current version of the plugin works perfectly with NS 7 because of the webpack alt loading modules. But just upgraded a project to NS 8 / Webpack 5 and the plugin no longer works. Webpack fails to compile:
ERROR in ./node_modules/nativescript-couchbase-plugin/couchbase-plugin.ios.js 4:12-51
Module not found: Error: Can't resolve 'tns-core-modules/utils/types' in '/Users/peter/Projects/SwyftCG/node_modules/nativescript-couchbase-plugin'
 @ ./app/shared/db-manager.ts 2:0-92 46:22-31 181:29-53 187:29-53 193:29-53 515:29-53 521:29-53 527:29-53 533:29-53 549:29-53 555:29-53 561:29-53 584:29-53 590:29-53 596:29-53 720:21-33
 @ ./app/ sync .(xml|js|ts|s?css)$ ./shared/db-manager.ts
 @ ./app/__@nativescript_webpack_virtual_entry_typescript__ 3:16-92

ERROR in ./node_modules/nativescript-couchbase-plugin/couchbase-plugin.ios.js 5:9-48
Module not found: Error: Can't resolve 'tns-core-modules/file-system' in '/Users/peter/Projects/SwyftCG/node_modules/nativescript-couchbase-plugin'
 @ ./app/shared/db-manager.ts 2:0-92 46:22-31 181:29-53 187:29-53 193:29-53 515:29-53 521:29-53 527:29-53 533:29-53 549:29-53 555:29-53 561:29-53 584:29-53 590:29-53 596:29-53 720:21-33
 @ ./app/ sync .(xml|js|ts|s?css)$ ./shared/db-manager.ts
 @ ./app/__@nativescript_webpack_virtual_entry_typescript__ 3:16-92

Which platform(s) does your issue occur on?

iOS/Android/Both
iOS/Android versions
emulator or device. What type of device?

Please, provide the following version numbers that your issue occurs with:

CLI: (run tns --version to fetch it)
Cross-platform modules: (check the 'version' attribute in the
node_modules/tns-core-modules/package.json file in your project)
Runtime(s): (look for the "tns-android" and "tns-ios" properties in the package.json file of your project)
Plugin(s): (look for the version numbers in the package.json file of your
project and paste your dependencies and devDependencies here)

Please, tell us how to recreate the issue in as much detail as possible.
Describe the steps to reproduce it.
Is there any code involved?

provide a code example to recreate the problem
(EVEN BETTER) provide a .zip with application or refer to a repository with application where the problem is reproducible.

Hey @PeterStaev it's under  @triniwiz/nativescript-couchbase here is for v6 support all v7+ plugins are here

Sorry, missed the note in the Readme...


gh: add re-triage workflow

updates https://github.com/honeycombio/telemetry-team/issues/143

Looks like you're already working on this, but noting just to make sure we don't lose sight of it.

@JamieDanielson we can merge this in, I'll open an issue to add the org secret in so workflows can run

scratch that -- issues aren't even enabled


Doesn't work?

I have the latest version of Firefox and Firebug and also the firepicker.
And I was trying to change color and nothing appear.
I have about 5 plug ins to Firebug and  more plug ins to firefox is it possible the compability?

Hello @Tekylla,
can you please post your Firefox, Firebug and operating system versions?

No update from the submitter, closing.


Unable to Understand Header values. bellow is code i have used. Please suggest solution

  oauthswift.client.request(
"api.tumblr.com/v2/blog/One_Touch_Share.tumblr.com/post",
method: .POST, parameters: ["type":"text"], headers: ["Authorization":??????????????]
, success: {
                responce in
                print("Success..")
                },
 failure: { error in
            print("Failed.....")
            }
)

please don't remove the template...
headers: ["Authorization":??????????????] you don't have to fill header with "Authorization"
it is automatic if you are authentified
It's for additional headers that oauth provider need

Dear,
you don't have to fill header with "Authorization"
it is automatic if you are authentified -- i am using

oauthswift.authorizeWithCallbackURL ( )  and then
oauthswift.client.request ( ) methods.
Now getting error "HTTP Status 404"
Please suggest solution,

Thanks for your reply.
On Thu, Sep 29, 2016 at 2:08 PM, Eric Marchand<EMAIL_ADDRESS>wrote:

please don't remove the template...
headers: ["Authorization":??????????????] you don't have to fill header
with "Authorization"
it is automatic if you are authentified
It's for additional headers that oauth provider need
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OAuthSwift/OAuthSwift/issues/282#issuecomment-250405345,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT3byTuDC3mC9xG3j4dts_j5dFzVHac_ks5qu3jpgaJpZM4KJqFS
.

I can help more without more information
404 is a generic http error, in web view or in log?

please add a breakpoint and see the full content of error if its an error in callback
your callback url is ?

Thank you for your reply.
I have sort out that problem.
Thanks,
Mahadev
On Thu, Sep 29, 2016 at 6:03 PM, Eric Marchand<EMAIL_ADDRESS>wrote:

I can help more without more information
404 is a generic http error, in web view or in log?

please add a breakpoint and see the full content of error if its an
error in callback
your callback url is ?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OAuthSwift/OAuthSwift/issues/282#issuecomment-250453038,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT3bya18IMdVaobYKviTYe8ZTM5qeAn2ks5qu7AfgaJpZM4KJqFS
.

Hi @phimage Im having the same issues and I get this as a response:
Response-Body=Optional("{\"meta\":{\"status\":400,\"msg\":\"Bad Request\"},\"response\":{\"errors\":[\"Post cannot be empty.\"]}}")}]
my code is the following
 @IBAction func postTextBlog(_ sender: AnyObject) {
        print(oauth1swift?.parameters)

        let headers = ["Accept": "application/json", "Content-Type":"application/json"]

        let parameters = [ParameterKeys.BlogType:ParameterValues.MarkdownType, ParameterKeys.Body:textBlog.text] as [String : Any]

        oauth1swift?.client.request(TMClient.sharedInstance().tumblrURL(withPathExtension: Methods.PostText), method: .POST, parameters: parameters, headers: headers, success: {
            (data, error) in
            let json = TMClient.sharedInstance().convertToJSONObject(data: data)
            print(json)
            print("hello")

            }, failure: {
                error in
                print(error)
        })

    }

(TMClient.sharedInstance().tumblrURL(withPathExtension: Methods.PostText) is essentially this:
URL: https://api.tumblr.com/v2/blog/idelfonsog2.tumblr.com/post
you can also see it below
OAuthSwiftError.response=<NSHTTPURLResponse: 0x60000022dc80> { URL: https://api.tumblr.com/v2/blog/idelfonsog2.tumblr.com/post } { status code: 400, headers {
    Connection = "keep-alive";
    "Content-Type" = "application/json; charset=utf-8";
    Date = "Tue, 18 Oct 2016 02:01:00 GMT";
    P3P = "CP=\"Tumblr's privacy policy is available here: https://www.tumblr.com/policy/en/privacy\"";
    Server = openresty;
    "Set-Cookie" = "tmgioct=5805825c60e2e80081398740; expires=Fri, 16-Oct-2026 02:01:00 GMT; Max-Age=315360000; path=/; domain=.tumblr.com; HttpOnly";

@idelfonsog2 please open a new issue
this one is closed and I see anything that can make us conclude easily that the issue is the same...
then I commit a tumblr test in demo app
a GET not a POST request (so not same way to sign request)
a POST could be added (like /user/follow & /user/unfollow or /user/like & /user/unlike)


[BUG] EvolvePokemon task only run once at the start of the bot

Expected Behavior
EvolvePokemon task run during the normal task loop of the bot.
Actual Behavior
EvolvePokemon task only run once at the start of the bot. See tho logs for the actual behavior
Steps to Reproduce
Install latest version from dev branch with provided config
Other Information
OS: masOS 10.11.6
Git Commit: 0f2bddd26675c5232b2e9f4e83dc6525c41a842d
Python Version: Python 2.7.12
Config
{
	"auth_service": "",
	"username": "",
	"password": "",
	"location": "",
	"gmapkey": "",
	"tasks": [
		{
			"type": "HandleSoftBan"
		},
		{
			"type": "CollectLevelUpReward"
		},
		{
			"type": "IncubateEggs",
			"config": {
				"longer_eggs_first": true
			}
		},
		{
			"type": "NicknamePokemon",
			"config": {
				"nickname_template": "{name}_{iv_pct}"
			}
		},
		{
			"type": "TransferPokemon"
		},
		{
			"type": "EvolvePokemon",
			"config": {
				"evolve_all": "Caterpie, Weedle, Pidgey, Rattata, Spearow, Psyduck, Poliwag, Slowpoke, Drowzee, Goldeen, Staryu, Magikarp",
				"first_evolve_by": "iv",
				"evolve_above_cp": 0,
				"evolve_above_iv": 0,
				"logic": "and",
				"evolve_speed": 20,
				"use_lucky_egg": false
			}
		},
		{
			"type": "RecycleItems",
			"config": {
				"item_filter": {
					"Pokeball": {
						"keep": 50
					},
					"Greatball": {
						"keep": 50
					},
					"Ultraball": {
						"keep": 50
					},
					"Masterball": {
						"keep": 50
					},
					"Potion": {
						"keep": 0
					},
					"Super Potion": {
						"keep": 0
					},
					"Hyper Potion": {
						"keep": 50
					},
					"Max Potion": {
						"keep": 50
					},
					"Revive": {
						"keep": 10
					},
					"Max Revive": {
						"keep": 10
					},
					"Razz Berry": {
						"keep": 50
					}
				}
			}
		},
		{
			"type": "CatchVisiblePokemon"
		},
		{
			"type": "CatchLuredPokemon"
		},
		{
			"type": "SpinFort"
		},
		{
			"type": "MoveToFort",
			"config": {
				"lure_attraction": true,
				"lure_max_distance": 2000
			}
		},
		{
			"type": "FollowSpiral",
			"config": {
				"diameter": 5,
				"step_size": 70
			}
		}
	],
	"forts": {
		"avoid_circles": true,
		"max_circle_size": 50
	},
	"websocket_server": false,
	"walk": 12.5,
	"action_wait_min": 1,
	"action_wait_max": 4,
	"debug": false,
	"test": false,
	"health_record": false,
	"location_cache": false,
	"distance_unit": "km",
	"reconnecting_timeout": 5,
	"evolve_captured": "NONE",
	"catch_randomize_reticle_factor": 1,
	"catch_randomize_spin_factor": 1,
	"catch": {
		"any": {
			"catch_above_cp": 0,
			"catch_above_iv": 0,
			"logic": "or"
		}
	},
	"release": {
		"any": {
			"keep_best_cp": 1,
			"keep_best_iv": 2
		}
	},
	"vips": {
		"any": {
			"catch_above_cp": 1200,
			"catch_above_iv": 0.9,
			"logic": "or"
		},
		"Lapras": {},
		"Moltres": {},
		"Zapdos": {},
		"Articuno": {},
		"Mewtwo": {},
		"Dragonite": {},
		"Snorlax": {},
		"Mew": {},
		"Arcanine": {},
		"Vaporeon": {},
		"Gyarados": {},
		"Exeggutor": {},
		"Muk": {},
		"Weezing": {},
		"Flareon": {}
	}
}

Logs
2016-08-09 16:28:22,185 [       cli] [INFO] PokemonGO Bot v1.0
2016-08-09 16:28:22,187 [       cli] [INFO] No config argument specified, checking for /configs/config.json
2016-08-09 16:28:22,188 [       cli] [INFO] Configuration initialized
2016-08-09 16:28:22,196 [PokemonGoBot] [INFO] [set_start_location] Setting start location.
2016-08-09 16:28:22,197 [PokemonGoBot] [INFO] [x] Coordinates found in passed in location, not geocoding.
2016-08-09 16:28:22,197 [PokemonGoBot] [INFO] [location_found] Location found: 52.011775, 4.359331 (52.011775, 4.359331, 0.0)
2016-08-09 16:28:22,197 [PokemonGoBot] [INFO] [position_update] Now at (52.011775, 4.359331, 0)
2016-08-09 16:28:22,197 [PokemonGoBot] [INFO] [login_started] Login procedure started.
2016-08-09 16:28:25,840 [PokemonGoBot] [INFO] [login_successful] Login successful.
2016-08-09 16:28:26,971 [PokemonGoBot] [INFO]
2016-08-09 16:28:26,971 [PokemonGoBot] [INFO] --- BottBob ---
2016-08-09 16:28:26,972 [PokemonGoBot] [INFO] Level: 25 (Next Level: 176525 XP) (Total: 723475 XP)
2016-08-09 16:28:26,972 [PokemonGoBot] [INFO] Pokemon Captured: 2232 | Pokestops Visited: 2718
2016-08-09 16:28:26,973 [PokemonGoBot] [INFO] Pokemon Bag: 148/250
2016-08-09 16:28:26,974 [PokemonGoBot] [INFO] Items: 300/350
2016-08-09 16:28:26,974 [PokemonGoBot] [INFO] Stardust: 886 | Pokecoins: 0
2016-08-09 16:28:26,974 [PokemonGoBot] [INFO] PokeBalls: 49 | GreatBalls: 48 | UltraBalls: 50
2016-08-09 16:28:26,974 [PokemonGoBot] [INFO] RazzBerries: 49 | BlukBerries: 0 | NanabBerries: 0
2016-08-09 16:28:26,974 [PokemonGoBot] [INFO] LuckyEgg: 6 | Incubator: 1 | TroyDisk: 6
2016-08-09 16:28:26,974 [PokemonGoBot] [INFO] Potion: 0 | SuperPotion: 0 | HyperPotion: 50 | MaxPotion: 20
2016-08-09 16:28:26,975 [PokemonGoBot] [INFO] Incense: 9 | IncenseSpicy: 0 | IncenseCool: 0
2016-08-09 16:28:26,975 [PokemonGoBot] [INFO] Revive: 10 | MaxRevive: 0
2016-08-09 16:28:26,975 [PokemonGoBot] [INFO]
2016-08-09 16:28:26,975 [PokemonGoBot] [INFO] Found encrypt.so! Platform: darwin Encrypt.so directory: /Users/bobsoppe/Programming/pokemongo-bot-bottbob
2016-08-09 16:28:26,975 [PokemonGoBot] [INFO]
2016-08-09 16:28:29,257 [PokemonGoBot] [INFO] [bot_start] Starting bot...
2016-08-09 16:28:29,863 [CollectLevelUpReward] [INFO] [level_up_reward] Received level up reward: []
2016-08-09 16:28:29,864 [IncubateEggs] [INFO] [next_egg_incubates] Next egg incubates in 0.43 km
2016-08-09 16:28:31,355 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Goldeen renamed to Goldeen_49
2016-08-09 16:28:33,557 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Weedle renamed to Weedle_36
2016-08-09 16:28:34,861 [EvolvePokemon] [INFO] [pokemon_evolved] Successfully evolved Goldeen with CP 501 and IV 0.49!

2016-08-09 16:28:58,925 [EvolvePokemon] [INFO] [pokemon_evolved] Successfully evolved Goldeen with CP 350 and IV 0.31!
2016-08-09 16:29:20,456 [EvolvePokemon] [INFO] [pokemon_evolved] Successfully evolved Poliwag with CP 293 and IV 0.24!
2016-08-09 16:29:36,279 [EvolvePokemon] [INFO] [pokemon_evolved] Successfully evolved Psyduck with CP 365 and IV 0.24!
2016-08-09 16:29:56,822 [EvolvePokemon] [INFO] [pokemon_evolved] Successfully evolved Weedle with CP 204 and IV 0.36!
2016-08-09 16:30:21,063 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.10km (attraction of lure 0.29km)
2016-08-09 16:30:22,618 [IncubateEggs] [INFO] [next_egg_incubates] Next egg incubates in 0.33 km
2016-08-09 16:30:24,695 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Poliwhirl renamed to Poliwhirl_24
2016-08-09 16:30:26,462 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Seaking renamed to Seaking_31
2016-08-09 16:30:28,353 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Golduck renamed to Golduck_24
2016-08-09 16:30:30,278 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Kakuna renamed to Kakuna_36
2016-08-09 16:30:32,285 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Seaking renamed to Seaking_49
2016-08-09 16:30:32,956 [TransferPokemon] [INFO] [keep_best_release] Keeping best 2 Kakuna, based on cp and iv
2016-08-09 16:30:33,682 [TransferPokemon] [INFO] [pokemon_release] Exchanged Kakuna [CP 220] [IV 0.36] for candy.
2016-08-09 16:30:37,417 [TransferPokemon] [INFO] [keep_best_release] Keeping best 3 Golduck, based on cp and iv
2016-08-09 16:30:38,161 [TransferPokemon] [INFO] [pokemon_release] Exchanged Golduck [CP 838] [IV 0.24] for candy.
2016-08-09 16:30:39,963 [TransferPokemon] [INFO] [keep_best_release] Keeping best 3 Poliwhirl, based on cp and iv
2016-08-09 16:30:40,696 [TransferPokemon] [INFO] [pokemon_release] Exchanged Poliwhirl [CP 520] [IV 0.24] for candy.
2016-08-09 16:30:43,188 [TransferPokemon] [INFO] [keep_best_release] Keeping best 3 Seaking, based on cp and iv
2016-08-09 16:30:43,982 [TransferPokemon] [INFO] [pokemon_release] Exchanged Seaking [CP 781] [IV 0.31] for candy.
2016-08-09 16:30:47,467 [TransferPokemon] [INFO] [pokemon_release] Exchanged Seaking [CP 1119] [IV 0.49] for candy.
2016-08-09 16:30:57,035 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.05km (attraction of lure 0.05km)
2016-08-09 16:31:06,454 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort *** (***)
2016-08-09 16:31:07,152 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Poliwag appeared! [CP 152] [Potential 0.47] [A/D/S 4/10/7]
2016-08-09 16:31:11,401 [PokemonCatchWorker] [INFO] [threw_pokeball] Used Pokeball, with chance 53.30 (48 left)
2016-08-09 16:31:12,036 [PokemonCatchWorker] [INFO] [pokemon_caught] Captured Poliwag! [CP 152] [Potential 0.47] [4/10/7] [+210 exp]
2016-08-09 16:31:12,037 [PokemonCatchWorker] [INFO] [gained_candy] You now have 204 Poliwag candy!
2016-08-09 16:31:17,676 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.11km (attraction of lure 0.28km)
2016-08-09 16:31:25,845 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort *** (***)
2016-08-09 16:31:26,510 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.10km (attraction of lure 0.27km)
2016-08-09 16:31:30,715 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Poliwag renamed to Poliwag_47
2016-08-09 16:31:37,899 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort *** (***)
2016-08-09 16:31:38,618 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.09km (attraction of lure 0.26km)
2016-08-09 16:31:47,275 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.07km (attraction of lure 0.25km)
2016-08-09 16:31:57,344 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Pidgey appeared! [CP 168] [Potential 0.53] [A/D/S 7/6/11]
2016-08-09 16:32:01,523 [PokemonCatchWorker] [INFO] [threw_pokeball] Used Pokeball, with chance 47.34 (47 left)
2016-08-09 16:32:02,171 [PokemonCatchWorker] [INFO] [pokemon_caught] Captured Pidgey! [CP 168] [Potential 0.53] [7/6/11] [+210 exp]
2016-08-09 16:32:02,172 [PokemonCatchWorker] [INFO] [gained_candy] You now have 1338 Pidgey candy!
2016-08-09 16:32:07,187 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.06km (attraction of lure 0.24km)
2016-08-09 16:32:16,162 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.04km (attraction of lure 0.23km)
2016-08-09 16:32:21,415 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Pidgey renamed to Pidgey_53
2016-08-09 16:32:29,763 [  SpinFort] [INFO] [spun_pokestop] Spun pokestop ***. Experience awarded: 50. Items awarded: {u'Pokeball': 3}
2016-08-09 16:32:33,013 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.07km (attraction of lure 0.22km)
2016-08-09 16:32:42,241 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.06km (attraction of lure 0.21km)
2016-08-09 16:32:44,687 [IncubateEggs] [INFO] [next_egg_incubates] Next egg incubates in 0.22 km
2016-08-09 16:32:51,667 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.04km (attraction of lure 0.19km)
2016-08-09 16:33:01,114 [  SpinFort] [INFO] [spun_pokestop] Spun pokestop ***. Experience awarded: 50. Items awarded: {u'Pokeball': 2, u'Revive': 1}
2016-08-09 16:33:04,351 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.07km (attraction of lure 0.18km)
2016-08-09 16:33:14,722 [RecycleItems] [INFO] [item_discarded] Discarded 2x Pokeball (maximum 50).
2016-08-09 16:33:15,456 [RecycleItems] [INFO] [item_discarded] Discarded 1x Revive (maximum 10).
2016-08-09 16:33:16,160 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Poliwag appeared! [CP 342] [Potential 0.4] [A/D/S 2/2/14]
2016-08-09 16:33:19,748 [PokemonCatchWorker] [INFO] [threw_pokeball] Used Pokeball, with chance 35.29 (51 left)
2016-08-09 16:33:20,387 [PokemonCatchWorker] [INFO] [pokemon_capture_failed] Poliwag capture failed.. trying again!
2016-08-09 16:33:23,108 [PokemonCatchWorker] [INFO] [threw_pokeball] Used Pokeball, with chance 35.29 (50 left)
2016-08-09 16:33:23,731 [PokemonCatchWorker] [INFO] [pokemon_caught] Captured Poliwag! [CP 342] [Potential 0.4] [2/2/14] [+210 exp]
2016-08-09 16:33:23,731 [PokemonCatchWorker] [INFO] [gained_candy] You now have 207 Poliwag candy!
2016-08-09 16:33:29,257 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.08km (attraction of lure 0.11km)
2016-08-09 16:33:39,675 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.06km (attraction of lure 0.10km)
2016-08-09 16:33:43,560 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Poliwag renamed to Poliwag_40
2016-08-09 16:33:50,491 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.05km (attraction of lure 0.09km)
2016-08-09 16:33:59,785 [  SpinFort] [INFO] [spun_pokestop] Spun pokestop ***. Experience awarded: 50. Items awarded: {u'Pokeball': 3}
2016-08-09 16:34:02,386 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.09km (attraction of lure 0.09km)
2016-08-09 16:34:12,848 [RecycleItems] [INFO] [item_discarded] Discarded 1x Pokeball (maximum 50).
2016-08-09 16:34:12,862 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.07km (attraction of lure 0.07km)
2016-08-09 16:34:21,486 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.06km (attraction of lure 0.06km)
2016-08-09 16:34:31,542 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.04km (attraction of lure 0.04km)
2016-08-09 16:34:41,604 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort *** (***)
2016-08-09 16:34:42,230 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Psyduck appeared! [CP 320] [Potential 0.31] [A/D/S 1/1/12]
2016-08-09 16:34:45,159 [PokemonCatchWorker] [INFO] [threw_pokeball] Used Pokeball, with chance 43.22 (49 left)
2016-08-09 16:34:45,822 [PokemonCatchWorker] [INFO] [pokemon_caught] Captured Psyduck! [CP 320] [Potential 0.31] [1/1/12] [+210 exp]
2016-08-09 16:34:45,822 [PokemonCatchWorker] [INFO] [gained_candy] You now have 317 Psyduck candy!
2016-08-09 16:34:51,644 [  SpinFort] [INFO] [spun_pokestop] Spun pokestop ***. Experience awarded: 50. Items awarded: {u'Razz Berry': 2, u'Pokeball': 1}
2016-08-09 16:34:54,256 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.10km (attraction of lure 0.25km)
2016-08-09 16:34:58,654 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Psyduck renamed to Psyduck_31
2016-08-09 16:35:07,527 [RecycleItems] [INFO] [item_discarded] Discarded 1x Razz Berry (maximum 50).
2016-08-09 16:35:07,529 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort *** (***)
2016-08-09 16:35:08,221 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.09km (attraction of lure 0.24km)
2016-08-09 16:35:17,669 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.07km (attraction of lure 0.23km)
2016-08-09 16:35:26,469 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.06km (attraction of lure 0.22km)
2016-08-09 16:35:35,148 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.04km (attraction of lure 0.20km)
2016-08-09 16:35:44,951 [  SpinFort] [INFO] [spun_pokestop] Spun pokestop ***. Experience awarded: 50. Items awarded: {u'Revive': 2, u'Greatball': 1}
2016-08-09 16:35:48,230 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.08km (attraction of lure 0.19km)
2016-08-09 16:35:57,380 [RecycleItems] [INFO] [item_discarded] Discarded 2x Revive (maximum 10).
2016-08-09 16:35:57,396 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.06km (attraction of lure 0.18km)
2016-08-09 16:36:05,889 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.05km (attraction of lure 0.17km)
2016-08-09 16:36:15,578 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Magikarp appeared! [CP 56] [Potential 0.64] [A/D/S 12/11/6]
2016-08-09 16:36:18,472 [PokemonCatchWorker] [INFO] [threw_pokeball] Used Pokeball, with chance 70.08 (49 left)
2016-08-09 16:36:19,106 [PokemonCatchWorker] [INFO] [pokemon_caught] Captured Magikarp! [CP 56] [Potential 0.64] [12/11/6] [+210 exp]
2016-08-09 16:36:19,107 [PokemonCatchWorker] [INFO] [gained_candy] You now have 105 Magikarp candy!
2016-08-09 16:36:24,764 [  SpinFort] [INFO] [spun_pokestop] Spun pokestop ***. Experience awarded: 100. Items awarded: {u'Razz Berry': 3, u'Pokeball': 3, u'Greatball': 1}
2016-08-09 16:36:28,675 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop *** - 0.07km (attraction of lure 0.16km)
2016-08-09 16:36:32,466 [NicknamePokemon] [INFO] [rename_pokemon] Pokemon Magikarp renamed to Magikarp_64
2016-08-09 16:36:33,103 [TransferPokemon] [INFO] [keep_best_release] Keeping best 3 Magikarp, based on cp and iv
2016-08-09 16:36:33,844 [TransferPokemon] [INFO] [pokemon_release] Exchanged Magikarp [CP 56] [IV 0.64] for candy.

Hope this will be fixed soon.

It's a new feature... and there is no config switch for this...

Not a bug.
"evolve_captured": "NONE",
Configure evolve captured in the same way as evolve_all

"evolve_captured": "NONE" solved the problem!

Transfer pokemon ?


Standardize on lower-case profile names?

I don't like the case sensitivity of our profiles right now. I'm thinking we standardize on lower-case.
cis-rhel-6-l1-scored.yaml
Thoughts? The current version is so easy to typo.

+1 (I hate typing caps)

Do it.
On Fri, May 6, 2016 at 1:53 PM, Gabriel Miu<EMAIL_ADDRESS>wrote:

+1 (I hate typing caps)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/HubbleStack/Nova/issues/125#issuecomment-217542630

I'm checking but there may be some constraints on Windows (although we can make exceptions as needed for these).

windows is case-insensitive, isn't is? Keep us posted if you run into any problems, but I don't expect there to be any.


stack deploy cancel

deploy a stack with wrong image, this will result in super slow failure detection:
I, [2017-08-20T05:55:23.453582 #1]  INFO -- Kontena::Workers::ImagePullWorker: pulling image: registry.kontena.rocks/lol-x:latest
W, [2017-08-20T05:56:33.957457 #1]  WARN -- Kontena::Workers::ImagePullWorker: image pull failed: No such image: registry.kontena.rocks/lol-x:latest
. Retrying still for 9 times.
W, [2017-08-20T05:57:38.566828 #1]  WARN -- Kontena::Workers::ImagePullWorker: image pull failed: No such image: registry.kontena.rocks/lol-x:latest
. Retrying still for 8 times.
W, [2017-08-20T05:58:49.106052 #1]  WARN -- Kontena::Workers::ImagePullWorker: image pull failed: No such image: registry.kontena.rocks/lol-x:latest
. Retrying still for 7 times.
W, [2017-08-20T05:59:59.681150 #1]  WARN -- Kontena::Workers::ImagePullWorker: image pull failed: No such image: registry.kontena.rocks/lol-x:latest
. Retrying still for 6 times.

workaround: remove the stack and try again.

looks like the workaround doesn't work if the service gets scheduled on the same node, it will continue to pull for the missing image?
workarounder: restarted the node to force scheduling to another node

...okay was a side effect from https://github.com/kontena/kontena-plugin-vagrant/issues/48
not sure how long this would take with a working dns - maybe that many retries are okay

If DNS works and registry responds without too much delay -> total delay should be ~2s.
But probably we could do less retries here.

#2516 for the slow failure on image pull, and the stack rm workaround probably also invokes #2623
Canceling a stack deploy would be easy to implement on the server, but harder on the agent.

Workaround to cancel a stack deploy is to kontena service stop, because this bypasses the deploy queue and updates the service desired_state  - this will abort any running deploys on the server. If there's multiple deploys queued up for a service, it may take up to 30s for each of them to start and abort themselves.
#2516 fixes the agent to allow canceling the wait_for_port by re-deploying/stopping the service, because it explicitly checks for service/container changes during the wait.
Issues like the #2516 image pull retry loop will still block the agent, and issues like #2623 seemingly have those blocking image pulls queue up. Stopping the service won't help clear up those issues.

I think the service deploy queueing on the server should be changed to abort any running or queued deploy if a newer deploy is created, and thus only run the most recent deploy. This would effectively be the same thing as the ability to cancel deploys. The kontena service stop/start/restart should also really happen as deploys.
Reasoning: because the services are not revisioned, each deploy will effectively only deploy the most recently updated service configuration. Changing the service configuration mid-deploy and queueing up newer deploys just means that the currently running deploy will deploy the remaining instances with the newer service configuration, and then each remaining queued up deploy will just deploy the newest service configuration.