PatchItemAsync failing with HTTP400 when using CosmosPropertyNamingPolicy.CamelCase instead of [JsonProperty("id")

My documents all have the Id property named Id (upper case) as is the convention in C#. To get things to work with CosmosDb, I'm using the CosmosClientOptions.SerializerOptions.PropertyNamingPolicy which I'm setting to CosmosPropertyNamingPolicy.CamelCase to have the CosmosDb SDK write my Id as id:
var cosmosOptions = new CosmosClientOptions()
{
    AllowBulkExecution = true,
    SerializerOptions = new CosmosSerializationOptions()
};
cosmosOptions.SerializerOptions.PropertyNamingPolicy = CosmosPropertyNamingPolicy.CamelCase;
var cosmosClient = new CosmosClient(endpoint, key, cosmosOptions);

This has worked fine for me so far - inserting, upserting, deleting, searching all work just fine. It's on Patch where I ran into an issue. Here's a sample to illustrate:
using Microsoft.Azure.Cosmos;

namespace ConsoleApp1;

internal class CosmosDbTest
{
    string endpoint = "endpoint_url";
    string key = "key";

    internal async Task RunTest()
    {
        try
        {
            var cosmosOptions = new CosmosClientOptions()
            {
                AllowBulkExecution = true,
                SerializerOptions = new CosmosSerializationOptions()
            };
            cosmosOptions.SerializerOptions.PropertyNamingPolicy = CosmosPropertyNamingPolicy.CamelCase;
            var cosmosClient = new CosmosClient(endpoint, key, cosmosOptions);
            var createDb = await cosmosClient.CreateDatabaseIfNotExistsAsync("TestDatabase");
            var db = createDb.Database;
            var createContainer = await db.CreateContainerIfNotExistsAsync(new ContainerProperties("TestContainer", "/id"));
            var container = createContainer.Container;
            var testItem = new PhoneBookCategory
            {
                Name = "test object", Id = Guid.NewGuid().ToString(), SubProp = new TestObject { StringProp = "hello world", IntProp = 10 }
            };
            var createRes = await container.CreateItemAsync(testItem, new PartitionKey(testItem.Id));
            if (createRes.StatusCode == System.Net.HttpStatusCode.Created)
            {
                testItem.Name = "renamed by upsert";
                testItem.SubProp.IntProp = 9;
                var upsertRes = await container.UpsertItemAsync(testItem, new PartitionKey(testItem.Id));
                if (upsertRes.StatusCode == System.Net.HttpStatusCode.OK)
                    Console.WriteLine($"Successfully upserted item {upsertRes.Resource.Id}");
                else
                    Console.WriteLine($"Upsert failed - status: {upsertRes.StatusCode}");

                List<PatchOperation> patchOperations =
                [
                     PatchOperation.Add($"/{nameof(PhoneBookCategory.Name)}", "updated name"),
                     PatchOperation.Replace($"/{nameof(PhoneBookCategory.SubProp)}/{nameof(TestObject.IntProp)}", 5)
                ];
                var patchRes = await container.PatchItemAsync<PhoneBookCategory>(testItem.Id, new PartitionKey(testItem.Id), patchOperations);
                Console.WriteLine($"Item modified with id {patchRes.Resource.Id}");
                var deleteRes = await container.DeleteItemAsync<PhoneBookCategory>(testItem.Id, new PartitionKey(testItem.Id));
                if (deleteRes.StatusCode == System.Net.HttpStatusCode.NoContent)
                    Console.WriteLine($"Successfully cleaned up test object");
                else
                    Console.Write($"Unable to clean up test object - status: {deleteRes.StatusCode}");
            }
            else
                Console.WriteLine($"Unable to create test object - status: {createRes.StatusCode}");
        }
        catch (CosmosException ex)
        {
            Console.WriteLine($"Cosmos DB Exception with Status {ex.StatusCode} Message: {ex.Message}");
        }
    }
}

public class PhoneBookCategory
{
    //[JsonProperty(PropertyName = "id")]
    public string Id { get; set; }
    public string Name { get; set; }
    public TestObject SubProp { get; set; }
}
public class TestObject
{
    public string StringProp { get; set; }
    public int? IntProp { get; set; }
}

This will cause the HTTP 400 error on this line
var patchRes = await container.PatchItemAsync<PhoneBookCategory>(testItem.Id, new PartitionKey(testItem.Id), patchOperations);
The exception message being:

Response status code does not indicate success: BadRequest (400); Substatus: 0; ActivityId: bfc4ab22-b363-4cad-970b-63900522677c; Reason: ();

Now uncomment the line
//[JsonProperty(PropertyName = "id")]
and comment out the line
cosmosOptions.SerializerOptions.PropertyNamingPolicy = CosmosPropertyNamingPolicy.CamelCase;
And it'll work. So it seems we have an issue with Patch and CosmosPropertyNamingPolicy.CamelCase. I'd expect the original sample to work and patch the document (it works on all other operations in this sample after all - the insert, the upsert, even the delete). You can even leave the property naming override ([JsonProperty(PropertyName = "id")]) active, as soon as the PropertyNamingPolicy  is set, you get the issue.
Environment:
Microsoft.Azure.Cosmos 3.42.0
.NET 8.0.7
VS2022 17.10.5

@philipthomas-MSFT can you please take a look?

Seems like it only happens on PatchOperation.Replace. Can repro, so will dig more into the root cause next.
https://github.com/Azure/azure-cosmos-dotnet-v3/pull/4632/files#diff-cceb41a13a0fcd2dfd82902943878d2d699fb966cf72edf2fc2257b4f8802b79

FYR: https://learn.microsoft.com/en-us/azure/cosmos-db/partial-document-update#supported-operations

FYR: https://learn.microsoft.com/en-us/azure/cosmos-db/partial-document-update#supported-operations

Yes, I was going to research that. Thanks. It seems like it makes sense why it fails on Replace so not sure this is something we need to fix. @kirankumarkolli

Path API's by-definition taking string as path contracts. => Application needs to provide the right propertyNames based on serialization options/customizations. It's by-design.
In future when Typed APIs are added then SDK will automatically do the required conversion.

If you replace the patch operations in my sample with just pure Adds, it still fails. This will still fail:
List<PatchOperation> patchOperations =
[
     PatchOperation.Add($"/{nameof(PhoneBookCategory.Name)}", "updated name"),
     PatchOperation.Add($"/{nameof(PhoneBookCategory.SubProp)}/{nameof(TestObject.IntProp)}", 5)
];

Okay, I get it.
But, now run this single patch operation:
List<PatchOperation> patchOperations =
[
     PatchOperation.Add($"/{nameof(PhoneBookCategory.Name)}", "updated name"),
];

It'll pass. If the path is case sensitive, this should trip things up, too.
Also, returning an HTTP 400 with no explanation is imho not a proper way to return errors. "property 'Name' does not exist in document 'xyz') would be a significant improvement. Unfortunately we have this behavior in various places (e.g. trying to save a document that has an Id instead of id also gets you an HTTP 400 without any explanation - you can repro that easily with my sample, just comment out cosmosOptions.SerializerOptions.PropertyNamingPolicy = CosmosPropertyNamingPolicy.CamelCase;)

If you replace the patch operations in my sample with just pure Adds, it still fails. This will still fail:
List<PatchOperation> patchOperations =
[
     PatchOperation.Add($"/{nameof(PhoneBookCategory.Name)}", "updated name"),
     PatchOperation.Add($"/{nameof(PhoneBookCategory.SubProp)}/{nameof(TestObject.IntProp)}", 5)
];

Sorry for providing the initial sample with a Replace, it is indeed not the right syntax to use for this case.

Before we get to a solution, apologies for the delay.
Also, I am not able to reproduce the issue with 2 Add patch operations, just the Replace. I have a branch and PR to support this.

May I propose that the error message be made more useful?

I think that the only way to successfully pull this off, is to have some proactive validation for Patch operations in general instead of reacting to an exception. Especially because the exception is too generic and not explicitly and descriptively pointing out the true error.

And also can you please validate if the shortlink in error/exception has related context to troubleshoot?

I think that Replace operation is similar to Set except it follows strict replace only semantics covers it but is very vague

how about update Path API code document in remarks section about the possibilities?

I can add more to the documentation to help clarify Replace operation is similar to Set except it follows strict replace only semantics.

Not sure if related to this particular issue, but I just logged a ticket for PatchItem where by it doesn't respect my CamelCase serialization, however the CreateItem and UpsertItem do.
#4733


Add new Bundler/GemComment cop

This cop helps our team make sure all gem declarations inside a Gemfile have a comment that describes why this gem was added to the project.
Bad:
gem 'rubocop'

Good:
# Ruby styleguide enforcer.
gem 'rubocop'

The cop is disabled by default so as not to break in codebases that do not enforce this behavior.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

Yay! Thank you for reviewing and merging this! \o/

@sunny how difficult would it be to add an option to this cop for an inline comment versus a comment above the line?
Example:
# This is my gem description
gem 'foo'

versus
gem 'foo' # This is my gem description

When you have a Gemfile that has a long list of gems, adding the comment above can end up making the Gemfile both extremely long, as well as more difficult to parse (inline comments can be lined up so as to be easy to scan).

Does this take into account inline comments?

We use inline comments by copying the first line of the description from rubygems.org.
Example:
gem 'oj', '3.6.8'               # The fastest JSON parser and object serializer.
gem 'patron', '0.13.1'          # Ruby HTTP client library based on libcurl
gem 'pg', '1.1.3'               # Pg is the Ruby interface to the PostgreSQL RDBMS

@rhymes that's pretty cool 😎
I prefer to leave comments addressing why the gem was installed in this project and why we're on a particular branch or version, if applicable. Basically, I try to avoid comments that are duplicative with the gem's readme.
For instance:
# Get results from the FooBar API over HTTP
# 0.14 has a bug (http://github.com/patron/patron/issue/42)
gem 'patron', '0.13.1'

@mikegee that makes perfect sense!


Azure settings: automate the generation of web.config and app.config files

In the Website project and in the devices projects, it would be useful to have a pre-build task that prompt the user for its Azure credentials and proposes a "wizzard" to select the event hub name and connection strings without having to enter them manually in the config files.

fixed in several commits and closed in 86f0be3d53a79ac91f81a99855eb7f75c7f19e58


Is a photo on your website necessary?

It doesn't need to be a professional headshot, but I think including a photo of yourself is very important so that other people can recognise you when they see you in person.

Duplicate of #4


Page takes a very long time to build and deploy - due to installing R dependencies?

See https://github.com/CDU-data-science-team/PatientExperience-QDC/actions/runs/5509202232/jobs/10041579387?pr=18 for example - it takes 28 minutes to install all the R dependencies and publish - can this be streamlined at all? No worries if not

the issue is that the few packages used depend on many others, therefore the long running time.

Ok, that make sense! I'm unclear on why we need to install the whole experiencesdashboard package just to show some images, especially since they are not interactive? maybe screenshots would be more efficient?

any thoughts @ChrisBeeley? Whole experiencesdashboard v0.6 is being installed to show the graphics on this page https://cdu-data-science-team.github.io/PatientExperience-QDC/dashboard/dashboard2.html , this makes the quarto site slow to build/deploy as all R dependencies need to be installed (it's taking 25 mins currently on average)

Definitely suboptimal. Shall we chat about it tomorrow? I have some ideas

To summarise our discussion, we're going to convert the graphs to .png but keep the code and add eval: false to the code chunk so the code (and package imports) are visible but not run

Following this recommendation https://quarto.org/docs/publishing/github-pages.html#freezing-computations
https://github.com/CDU-data-science-team/PatientExperience-QDC/pull/21 froze R computation only to be done locally


traslation revised

Overview
I made some revision in your translation in response to your request.

Thanks for @javadr


dpkg-preconfigure: unable to re-open stdin: No such file or directory

==> default: Running provisioner: shell...
    default: Running: /var/folders/w8/7kxbcq6d5k57r18y0sbsbmfr0000gn/T/vagrant-shell20160121-4415-19kl3p7.sh
==> default: stdin: is not a tty
==> default: Update apt repositories ...
==> default: dpkg-preconfigure: unable to re-open stdin: No such file or directory
==> default: Install requirements ...
==> default: dpkg-preconfigure: unable to re-open stdin: No such file or directory
==> default: Symlink /usr/bin/nodejs to /usr/bin/node ...
==> default: Apply base config for dnsmasq ...
==> default: Compile NodeJS libs and assets of hostkeeper web interface ...
[...]

export DEBIAN_FRONTEND=noninteractive

https://serverfault.com/questions/500764/dpkg-reconfigure-unable-to-re-open-stdin-no-file-or-directory


Change opening to left click

Hi,
I was wondering if there is a way to change the opening of the menu to left click instead of right click. Is this added already? Have I missed it?
Thanks and have a good one

Had a look into the docs :) sorry about that

Glad you got it sorted 🙂


Update IDR URL

I grepped & this is the only place we link to the IDR in the Docs.
Staged at https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/system-requirements.html

N.B. I checked Help too but there are no references there.


Update CHANGES.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.71%. Comparing base (760953f) to head (51ba114).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #459      +/-   ##
==========================================
+ Coverage   88.48%   88.71%   +0.23%
==========================================
  Files          60       60
  Lines        4716     4716
  Branches      448      448
==========================================
+ Hits         4173     4184      +11
+ Misses        542      531      -11
  Partials        1        1

Flag
Coverage Δ

unittests
88.71% <ø> (+0.23%)
:arrow_up:

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

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


Web REPL bugs and improvements

#72 Fixes some problems like pasting and interaction, but there are still several issues in the web REPL. These probably require structural changes and a better integration with the xterm.js api.
Problems:

[ ] Backspace does not work properly when not on the end of a line.
[ ] Delete does not work
[ ] Multiline edit does not work

Does #112 fix all problems?

@dlight No, the web repl still has some issues:

Multi-line editing still doesn't work (because newline graphemes aren't handled properly by up, down, left, right, ^left, ^right, home, end, delete and backspace keys).
The implementation also breaks when lines are longer than the terminal screen (deleting/backspacing graphemes in prior lines doesn't cause graphemes in later lines to move back).


shift in byte mode should produce syntax error

This is not a valid syntax that is not recognised by linter
  DATA lv_temp TYPE string.
  lv_temp = 'abc'.
  SHIFT lv_temp BY 1 PLACES LEFT in byte mode.
  WRITE lv_temp.

DATA lv_temp TYPE string.
SHIFT lv_temp BY 1 PLACES LEFT IN BYTE MODE.

@arcanist123 try again, 2.77.5

closing, no reply


Server crashes when a player tries to join

Description
I wanted to use LuckPerms on my "Encrypted_" server, but when i tried to join the server, the server crashed
Reproduction Steps

install a encrypted_ server (https://www.curseforge.com/minecraft/modpacks/encrypted_/files/3957277)
install LuckPerms Forge 5.4.50 on the server
start the server
join the server
server crashes

Expected Behaviour
Nothing, just a player joining the server
Server Details
net.minecraftforge:40.1.73
LuckPerms Version
v5.4.50
Logs and Configs
https://gist.github.com/CrazyDev05/858d042e6bcb49df4e683e68af375059
Extra Details
No response

The latest version of LuckPerms is for Forge 1.19.2. Try downloading the latest 1.18.2 build from CurseForge.

i'll try

works now, thank you


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#749 aa408aa Thanks @threepointone! - y-partykit: call process.on/.off only when available

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Edit UG

Explain format
FAQ qn on getting help (Closes #191 )
List caution boxes (Closes #209 )
Small fixes to other callout boxes and *

Codecov Report

Merging #212 (29cc248) into master (9a5dbb8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #212   +/-   ##
=========================================
  Coverage     64.71%   64.71%
  Complexity      574      574
=========================================
  Files           101      101
  Lines          1879     1879
  Branches        194      194
=========================================
  Hits           1216     1216
  Misses          600      600
  Partials         63       63

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Change Column label propType to node

Changes Column label propType to node
See #642

Added tests
Updated Column docs
Tests are passing
Lint is passing

Looks great! Thanks for being thorough. 😄


Add Basic Completion

Complete the following

Date
Payee
Accounts

completed initial impl
need to check for second ' " ' and not show completions

think this is ready for v1


Add indoor to sirens

Indoor sirens represent alarms, such as fire alarms or burglar alarms.

As far as I can see, the tag emergency=siren is not meant to be used for indoor warning devices, such as fire alarms. See the definition on the wiki:

An outdoor warning siren (also known as a civil defense siren, air-raid siren, or tornado siren) is a […]

Also, according to taginfo the combination emergency=siren + indoor=* is not really common.

I didn't find an established tag for indoor sirens. Please let me know if there is one, or start a tagging proposal to introduce a tagging for these devices. :bow:


Updates migrated from sdk repo

LGTM, but do we know why Travis is red? Looks like the IPv6 workaround isn't working anymore.

Not sure. Maybe it's not needed anymore. I will try commenting out the commands.

Looks like it's still needed.

Needed to set sudo to true.


Add GPL 3 License

It's already licensed MIT, which is more permissive. Why another license?

Just didn't see it there. Was tagging all the other groups.  MIT is fine.
On December 18, 2016 11:05:54 AM EST, James McKinney<EMAIL_ADDRESS>wrote:

It's already licensed MIT, which is more permissive. Why another
license?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/guerrilla-archiving/eotarchive-echo/pull/1#issuecomment-267829517

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


[UIE-75] Upgrade Babel

I was trying to pass type parameters to a generic component. Something along the lines of...
const Component = <T>(props: { value: T }) => null

h(Component as typeof Component<string>, { value: 'foo' })

While TypeScript accepts this, Babel refused to compile it.
SyntaxError: /path/to/example.ts: Unexpected token (3:39)

  1 | const Component = <T>(props: { value: T }) => null
  2 |
> 3 | h(Component as typeof Component<string>, { value: 'foo' })

On the other hand, the Babel REPL would accept this syntax.
Upgrading Babel resolves the compilation error. I tried a more selective upgrade at first, but that didn't work. So I went with a more heavy handed approach: upgrade all @babel/... and babel-... packages in yarn.lock.

I approved, but we should consider an upgrade to SPA itself... at least enough to get the newer babel dependency if not to the very newest SPA offering.

Rebased to resolve merge conflicts.


[BUG]: Mark as Sent not working

Describe the bug

[x] Mark as Sent not working in the courtstaff screen for marking summons sent.
[x] SENT_NOTICE_VIA_POST to be renamed as Send Notice via Post

To Reproduce
https://drive.google.com/file/d/1nEdx03idvwmz1cRgCsvoGVpN2c4tgMpt/view?usp=sharing
V0.3

Hi @devarajd94 , can you try again, use new creds for courtStaff (uatCourt01). Updated the same in the role sheet also.

Verfied


[request] Notifications

Add options to notify about overal execution status or fails only.

Euhm, already exists under Webhooks:

Maybe have a look around inside the application before posting feature requests?

webhooks is not a notifications. i mean basic smtp settings.

webhooks is not a notifications...
I disagree.Yes, my notification does not come to the mail, but to telegram + zabbix, but the webhook in the role of notifications works fine.
Perhaps this option will suit you too

webhooks is not a notifications. i mean basic smtp settings.

I disagree.Yes, my notification does not come to the mail, but to telegram + zabbix, but the webhook in the role of notifications works fine.
Perhaps this option will suit you too


typeError: string indices must be integers, not 'str'

Hi:
I have tried to run some sequences downloaded from NCBI RefSeq, but it produces the following prompt without giving me any result.
Reported 584 pairwise alignments, 584 HSPs.
76 queries aligned.
Traceback (most recent call last):
File "/home/ubuntu24/MeTAfisher/./metafisher/metafisher.py", line 309, in
main()
File "/home/ubuntu24/MeTAfisher/./metafisher/metafisher.py", line 260, in main
for contig, genes in contig_to_genes:
File "/home/ubuntu24/MeTAfisher/metafisher/Function_MetaF.py", line 501, in get_genes_by_contigs
gene.add_domain_info(gene_to_domains[gene_id])
File "/home/ubuntu24/MeTAfisher/metafisher/Object_MetaF.py", line 138, in add_domain_info
toxin_count = [d.domain_info['type_prct']['T'] for d in domains]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu24/MeTAfisher/metafisher/Object_MetaF.py", line 138, in
toxin_count = [d.domain_info['type_prct']['T'] for d in domains]
~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^
TypeError: string indices must be integers, not 'str'
Best
Maria

Hello Maria,
Thanks for your interest in MeTAfisher.
The error seems to be caused by some slight change in some of the names of TADB sequences. I       have updated the TADB stat file and improved the code to prevent the error.
Can you retry with the updated version of the code and tell me if it works on your side?
To update the code you can just do a git pull  in your MeTAfisher code directory.
Best


Update Emulator to Use Electron 4.x

Internal MS builds of Electron 4.x should be available at the end of Jan, 2019

Fixed in #1419


Issue token manually

First of all; thanks for providing this great package! It's simply awesome. Good job.
I am developing an RESTful API, that, amongst others, an iOS-app will communicate with.
Currently, the iOS-app will send a users facebook access token to the API. The API will then look op the facebook users e-mail address, and check if a user with that e-mail exists. If not, it should be created. After this, the user is being authed using Auth::login($user);
Nevertheless, when the user has been authenticated, I wish to generate an access-token for the authed user, and return this to the iOS app. Then iOS app will then send that access-token everytime it needs to do something, on behalf of the user.
However, I cannot seem to find the correct way to implement this feature (I have read throughout the great Wiki).
My code:
				// unnessecary code pasted out

				/**
				 * Save the user data
				 */

					$user->save();

				/**
				 * Auth the user
				 */

					Auth::login($user);

				/**
				 * Generate access token for the user
				 */

					$token = some_method();

				/**
				 * Return the token to the client
				 */

					return [
						'token' => $token
					];

I have stumpled upon: https://github.com/lucadegasperi/oauth2-server-laravel/wiki/Creating-your-own-custom-Grant - is this what I am looking for?

@canfiax  Thank you very much, unfortunately disappointed with @lucadegasperi  response regarding help, but definitely you have done a great job, Keep it up! :+1:


cant mirror channel

Are u having problems with just a specific channel or with all channels? Is it a big server?
Give more details pls

with all channels.It didnt clone any msg

This is weird, are you sure that the user/s have access to both channels and that you correctly set "readChannel" and "writeChannel" in the config file?
It is important to point out that the id of the channels must be informed, not the servers id.
Try to create a server and experiment only with a server dedicated to this, so that after you get the hang of it, you can "clone" the target server.


[it] Fix Markdown format

Change The` code `format to The `code` format
Use correct codenew statement
Change .. quindi" applicare "la risorsa .. to .. quindi `apply` la risorsa ..

Ref: #32897

The change is correct.
Not really relevant since those are drafts, but it makes sense to merge is
/lgtm
/approve


Generate yaml from certs for tls lookup

tls.check_expiring could do this so the map is always up to date.

component._find_tls_crt needs to search for crt, not key. When generating crt, the key exists, but not the crt.


Unhandled exception in createAccountTransaction() if bad tx data is given

If createAccountTransaction is called with a transaction with an invalid address, or an amount is given that can't be sent, the following error is thrown:

Fatal error: Uncaught exception 'GuzzleHttp\Exception\ClientException' with message 'Client error: 422'

Code to reproduce:
<?php
  require_once('vendor/autoload.php');

  use Coinbase\Wallet\Client;
  use Coinbase\Wallet\Configuration;
  use Coinbase\Wallet\Resource\Transaction;
  use Coinbase\Wallet\Value\Money;
  use Coinbase\Wallet\Enum\CurrencyCode;
  use Coinbase\Wallet\Value\Fee;
  use Coinbase\Wallet\Enum\Param;
  use Coinbase\Wallet\Resource\Address;

  $apiKey="apikey";
  $apiSecret="apisecret";

  # a bad address and/or an invalid amount will cause the issue
  $address = 'bad_address';
  $btcvalue = 'amount_greater_than_wallet_balance';

  $configuration=Configuration::apiKey($apiKey, $apiSecret);
  $client=Client::create($configuration);
  $account=$client -> getPrimaryAccount();
  $transaction = Transaction::send([
    'toBitcoinAddress' => $address,
    'amount' => new Money($btcvalue, CurrencyCode::BTC),
    'description' => "bad transaction"
  ]);
  $client->createAccountTransaction($account, $transaction);
  echo $transaction->getId();
?>

More discussion here:
https://community.coinbase.com/t/getting-status-result-php/8427/4
https://community.coinbase.com/t/error-guzzlehttp/7577/3

I think this is the expected behavior when trying to create a transaction with bad arguments.
We can link people to http://docs.guzzlephp.org/en/latest/quickstart.html#exceptions to see how to handle exceptions in their applications. Eg.
try {
    $client->createAccountTransaction($account, $transaction);
} catch (ClientException $e) {
    echo $e->getRequest();
    echo $e->getResponse();
}

@aianus more trouble with this here #77 -- exception handling from this angle doesn't seem to work, or we're not doing it right.


Tensorflow hangs when missing "encoder.params" or "rnn_cell"

The module that handles YAML seems missing error handling features. When "encoder.params" missing, it literally just hang - the process is running but not doing anything, no error message even when verbose level is debug. It took me almost two days to figure out such a simple error.

not an issue....


YouTube images on browse are blurry

I think it's using the thumb version ...

it uses what ever you put in image_small or image_thumb in the csv, there is no youtube logic in the template

I get that better now. In the example data, which most people will likely
follow, the images end up being blurry. So maybe just adjust that.
I also wonder about going away from youtube. I get it, but the parsing of
youtube links seems potentially risky. It might be the best solution; I'm
just wondering if its sustainable. And if there are a limited number of
options to support, maybe making options for each would be good, or using
object_template and object_download more explicitly.
On Wed, May 12, 2021 at 11:35 AM EvanWill @.***> wrote:

it uses what ever you put in image_small or image_thumb in the csv, there
is no youtube logic in the template
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CollectionBuilder/collectionbuilder-csv/issues/1#issuecomment-840009307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACTM4GKX7IXKGHIB6MG3QYLTNLC7JANCNFSM44TIPPMA
.


Issue with /docs/tutorials/federation/set-up-cluster-federation-kubefed/

To delete the Federation Control Plane, you need to switch to host cluster context before issuing the delete ns federation-system command. This is not clearly indicated.

@madhusudancs


closing tag and opening tag combination going to the for loop.

I have test "><" and falling into fatal error.

my original test case qa put was ">>>\_(*L*)_/<<>>>\_(*L*)_/<< "

The problem appears whenever we have an unclosed tag start, so. New testcases draft:
  describe('unclosed tag name', () => {
    it('should work with \'<\' as input', () => {
      const result = html_substring('<', 50);

      expect(result).to.eq('<');
    });

    it('should work with \'<div\' as input', () => {
      const result = html_substring('<div', 50);

      expect(result).to.eq('<div');
    });

    it('should work with \'<div abc\' as input', () => {
      const result = html_substring('<div abc', 50);

      expect(result).to.eq('<div abc');
    });

    it('should work with \'test <div abc\' as input', () => {
      const result = html_substring('test <div abc', 50);

      expect(result).to.eq('<test div abc');
    });

    it('should work with \'hello <div <h1\' as input', () => {
      const result = html_substring('test hello <div <h1', 50);

      expect(result).to.eq('hello <div <h1');
    });

    it('should work with \'<div<h1>\' as input', () => {
      const result = html_substring('<div<h1>', 50);

      expect(result).to.eq('<div<h1>');
    });
  });

Fixed by 8135e2f (starting with version 1.8.0)


Filtering for text-align doesn't show the CSS Logical Properties item, but should

CSS Logical Properties includes  the status of text-align: start and text-align: end, users searching for text-align start won't see the desired information. (Instead, they can search for Logical.)

I believe the results are now as expected: https://caniuse.com/?search=text-align start now shows the support table for "CSS property: text-align: Flow-relative values start and end"


Remote fetching

I wanted to take a stab at fetching a remote. A bunch of things that you might not like (open for suggestions):

I changed Remote to be class (it needs to manage the pointer)
The name / URL properties don't copy anymore, but are backed by the libgit2 calls
I use deinit for memory management
I duplicate the pointer instead of deleting a free call somewhere else
This is based off of the fix-build branch

Thanks for opening a PR! 😄
You're right that I don't like the things you listed. 🙈
My design for SwiftGit2 is based off my experience with https://github.com/libgit2/objective-git in GitHub for Mac. The Objective-Git is similar to what you have here.
That was the source of many headaches because:

You need to think more actively about concurrency. Libgit2 access must be serial within an open repository. This introduces a lot of cognitive overhead to application development.

What looks like a simple property access may be quite slow. It's hard to anticipate performance implications when looking at the code or when running under ideal circumstances. (I have a similar dislike for faulting ORMs like Core Data or ActiveRecord.)

You introduce co-effects all over the place instead of containing them, which makes code hard to test.

So I do think it'd be better to add a Repository.fetch method that looks up the remote by name and then fetches. (Although I do hope to move to some sort of command API #103)

Understood, that makes sense. I see how that would work, will change my PR.

I've changed the code to your suggestions, what do you think?

Looks good. 👍
There's just a conflict that needs to be resolved because I removed trailing whitespace in #112.

I rebased off of master and force-pushed.

🎉

Yay!


Article - author

Author image and name should also include Twitter handle

if they have one :)

@davidpmccormick can you please remove the bio copy from div class="author  author--border-bottom " in cardigan?

@jennpb Done. Can this now be closed?

:1st_place_medal:


Add Storage space flag

Would be nice to add max diskspace limit to avoid disk full condition like #1153
I add a comment there but this thread is old so I make a new.
I have a one drive storage with 1tb and want to backup multiple backups there. So it would be nice to add this flag to limit theax backup size.

@wuast94 Thank you for sharing this concern. I understand it's can be a problem when you're tight on disk space.
However, for practical reasons I'd like to ask you to close this issue, as we already have the one you referenced active, and with more discussion as well. This duplicate one only causes more overhead, which makes for less time to attend other issues (including the one you referenced).
Thanks!

I see you referenced this issue there so I will Close it :)

Given that #1153 has been closed, without adding the "max repo size" flag requested in this issue, can we reopen this one?
That issue was more about how filling the disk would put restic into an inoperable state, which is now fixed as of #3481.
There were a number of participants in #1153 that were interested in something more like a feature (automatically purging old snapshots when necessary to accommodate the new one) than a bugfix (ensuring an operable state in extreme FS conditions).
I'll state my own use case here, but I can move elsewhere/delete it if needed:
Personally, I'd like the backup operation to be able to carve out whatever space is necessary for it to successfully complete.
Right now, I'm compensating for this by using an aggressive "forget" policy, but I'm still afraid of the scenario where my nightly backups are continually failing because my heuristic wasn't aggressive enough and I need to manually intervene now. (And on the flip side, I'm potentially missing out on having a lot more snapshots at my disposal because I need to have such an aggressive policy.)

@kibiz0r Please just create a new issue, that's easier to manage than resurrecting very old issues.


Issue 758 oxog

This adds the basic orientation bias filter as previously used in Broad mutation calls.
This version does the bare filtering and relies on the picard preadapter details tool to be run.
WDL has not yet been integrated but issue filed for that.
Closes #758

Back to @LeeTL1220

@davidbenjamin Back to you.

@LeeTL Congratulations, ready to merge.


Fails to Parse Adobe Source Han Variable Font OTFs

ttf-parser seems to be unable to recognize glyphs in Adobe Source VF OTFs. While I didn't test every combination, the problem seems to be present across both Source Han Sans and Source Han Serif across all language varieties.
I would check what tables the font is using and compare it with the support table in the README for some insight, but FontForge, opentype.js, and Typr.js all also failed to parse the fonts, so I can only assume the fonts are using some very obscure features 😓
For comparison/reference, the corresponding TTF fonts are parsed correctly.

For reference, Windows seems to have no issues parsing and installing the fonts. +1 on assuming the fonts are using some very obscure features, or have a malformed table.

Well, it does state that:

WARNING: Currently Windows 10 and Windows 11 do not support CFF2 variable fonts. Please use the TTF variable instead.

But we do support CFF2. And font2svg example produces the correct output. How exactly are you using ttf-parser? As a library or indirectly?
I've tested SourceHanSans-VF/Variable/OTF/SourceHanSans-VF.otf and SourceHanSans-VF/Variable/OTF/Subset/SourceHanSansJP-VF.otf


Enable Constant Folding on Simplified Eval

Enables constant folding for simplified eval path in fuzzer. The simplified path expects constant exprs never to write into an existing result vector. A failed constant folding attempt must therefore clear any intermediate results from the Expr so that the simplified path does not pass these to constant exprs.

@oerling Closing per offline discussion.


Implement an inline assembly fallback for unsupported instructions.

This won't support all cases (256-bit and 512-bit instructions for sure), but
this should allow for correctness on unknown instructions, even if the generated
output will cause the optimizers to throw their hands up in the air.
The test is somewhat flaky since it passes 16 arguments into printf after the
the printf string, which seems to be more than mcsema expects. It also requires
particular compilation flags to get the aes instructions to compile and run
(that means you need Intel Westmere or newer or AMD Bulldozer or newer).
I used AES for the test case since I figured no one would ever bother
implementing an instruction lifter for these instructions unless it were part
of a generalized intrinsic uplifting technique.

I don't know if you get a notification for this, but I did push an updated patch which should address all the aforementioned issues.


Confused about stopping criterion in cg!

Hello,
Sorry if this not the correct place to post questions. I feel confused about the stopping criterion in cg!.

The help of cg! says the method stops when |r_k| / |r_0| ≤ max(reltol * resnorm, abstol). What is resnorm, |r_k| or |r_0|? Apart of that, I do not think that makes sense. Imagine, for example, that our first iteration is very close to the solution, i.e., r_0=1e-7. If we use a stopping criteria of the form |r_k| / |r_0| < some tolerance, then |r_K| may be too small for the machine precision. It would make more sense to me to employ something like |r_K| < reltol |b| + abstol, which does not depend on the initial condition. Note that Matlab uses this criterion with abstol=0.

In issue #280, it is said that the stopping criterion is in reality norm(residual) <= max(abstol, reltol * initial_residual_norm). However, the problem would be the same than before: a small initial residual may lead to a too small tolerance. For me, since the solution is (theoretically) the same irrespective of the initial condition, then the stopping criterion should be the independent as well of the latter. It does not make sense to me to stop the method at some point if the initial condition is x_1 and to stop it at another different point if the initial condition is x_2 \neq x_1.

Having a quick look at the code, it seems to me that the stopping criterion is actually norm(residual) <= max(abstol, reltol * initial_residual_norm), but I'm not really sure.

I would appreciate very much if you let me know which is the current stopping criterion for CG, as well as if you borrowed it from a specific reference or why you use it.
Apart of that, I suggest to change the stopping criterion to |r_K| < reltol |b| + abstol for the reasons discussed above and/or |\Delta x| < reltol |x| + abstol (which seems to be a more robust option to me, as it provides more control over the variable we are trying to find).

Sorry if this not the correct place to post questions.

This is a good place :)
Yeah, that must be a typo, the criterion used to be relative only, so ||r_k|| / ||r_0|| < tol, and then it got changed to supporting absolute tolerance as well, so ||r_k|| < max(abstol, reltol * ||r_0||), but probably the / ||r_0|| was not removed from the docstring when that change was made.
I don't see why you would want to use ||r_k|| < abstol + reltol * ||r_0|| over ||r_k|| < max(abstol, reltol * ||r_0||)

Thanks @haampie.
For me using ||r_k|| < abstol + reltol * ||r_0|| or ||r_k|| < max(abstol, reltol * ||r_0||) is more or less the same numerically (I wrote the former when I was thinking in the latter). My concern is about the term reltol * ||r_0||. It would make more sense to me to use reltol * ||b||, because (assume abstol=0)

With reltol * ||r_0||, you may have more or less accuracy on your final solution depending on the initial condition. With reltol * ||b||, the expected accuracy should be the same.

If the initial condition is very close to the final one (may happen if the system is being solved inside a nonlinear fixed-point iteration, e.g.), then reltol * ||r_0|| will be very small (e.g., 1e-16) and the method will need a lot of iterations to converge, despite the initial condition is already very good. It may also happen that the method never converges to that tolerances with ill-conditioned and/or very large matrices. In contrast, using reltol * ||b|| should make the method to stop in a few iterations, as I would expect.

Please try the following MWE:
function TestIterativeSolvers(N::Int, RelTol::Float64)

    #Random matrix:
    B       = sprand(N, N, 0.2)
    A       = sparse(1:N, 1:N, Float64.(1:N)) + (N/2)*transpose(B)*B

    #Random solution:
    xsol    = rand(N)
    b       = A*xsol

    #-------------------------------------------
    #Solve with random initial guess:

    println("Random initial guess")

    #Initial guess:
    x0      = rand(N)
    r0_norm = norm(A*x0-b)
    b_norm  = norm(b)
    println("|r_0|=$(r0_norm)")
    println("|b|=$(b_norm)")

    #Solve with |r_k| < tol * |r_0|:
    xv              = copy(x0)
    solver_output   = cg!(xv, A, b, abstol=0.0, reltol=RelTol, log=true, verbose=false)
    println("nIters=$(solver_output[2].iters), |r|=$(solver_output[2][:resnorm][end]), |x-xsol|/|xsol|=$(norm(xv-xsol)/norm(xsol))")

    #Solve with |r_k| < tol * |b|:
    xv              .= x0
    solver_output   = cg!(xv, A, b, abstol=0.0, reltol=RelTol*b_norm/r0_norm, log=true, verbose=false)
    println("nIters=$(solver_output[2].iters), |r|=$(solver_output[2][:resnorm][end]), |x-xsol|/|xsol|=$(norm(xv-xsol)/norm(xsol))")

    #-------------------------------------------
    #Solve again with initial guess very close to the solution:

    println("\nInitial guess close to the exact solution")

    #Initial guess:
    x0      = xsol + 100*RelTol * rand(N)
    r0_norm = norm(A*x0-b)
    b_norm  = norm(b)
    println("|r_0|=$(r0_norm)")
    println("|b|=$(b_norm)")

    #Solve with |r_k| < tol * |r_0|:
    xv              = copy(x0)
    solver_output   = cg!(xv, A, b, abstol=0.0, reltol=RelTol, log=true, verbose=false)
    println("nIters=$(solver_output[2].iters), |r|=$(solver_output[2][:resnorm][end]), |x-xsol|/|xsol|=$(norm(xv-xsol)/norm(xsol))")

    #Solve with |r_k| < tol * |b|:
    xv              .= x0
    solver_output   = cg!(xv, A, b, abstol=0.0, reltol=RelTol*b_norm/r0_norm, log=true, verbose=false)
    println("nIters=$(solver_output[2].iters), |r|=$(solver_output[2][:resnorm][end]), |x-xsol|/|xsol|=$(norm(xv-xsol)/norm(xsol))")

end

Then
julia> TestIterativeSolvers(5000, 1e-10)
Random initial guess
|r_0|=9.01977918157597e7
|b|=2.22888144955814e10
nIters=375, |r|=0.00874023139937135, |x-xsol|/|xsol|=1.1989921080773163e-8
nIters=258, |r|=2.1205394486976847, |x-xsol|/|xsol|=2.724872584351992e-6

Initial guess close to the exact solution
|r_0|=220.89982926376743
|b|=2.22888144955814e10
nIters=253, |r|=2.1058125468955393e-8, |x-xsol|/|xsol|=1.0299092193761744e-10
nIters=1, |r|=0.2111130008760191, |x-xsol|/|xsol|=4.956472028094513e-9

As you can see, when the initial condition is random, both choices take a similar number of iterations. However, when the initial condition is very close to the exact one, one should expect the method to need less iterations to converge, but the first choice provides a similar number of iterations. This is because the stopping tolerance is now much smaller.
In my opinion, it would be more robust to set |Delta x| < max(reltol * |x|, abstol) or similar, as it would allow a more direct control of the error in the solution.

I think the example shows that your use case can be supported already, right?
You might want to use:
cg!(xv, A, b, abstol=RelTol*norm(b), reltol=0.0, log=true, verbose=false)
to make it more readable.
We just define reltol to be the relative reduction in the residual norm, and anything else can simply use abstol.
If you need even more control, you can use CG as an iterator and implement your own stopping condition.

I think the example shows that your use case can be supported already, right?

Yes.

You might want to use:

...

to make it more readable.

Yes.

you can use CG as an iterator

I will have a look. Is Delta x the field u of the iterator?

Ok. Thank you very much for your help @haampie


[Packer] Masks are being processed differently from expected creating issues at seams.

The following below is what I posted in the poiyomi discord and I was referred here.
Poiyomi Toon 9.0.61 (Should be present in newer versions)
There is an issue with how the RGBA slots are being processed, causing misalignment on the seams/UVs. If you pack your channels using external applications, there isn't a problem; the issue arises when Poiyomi attempts to interprets them as separate channels and even when merged. For example, if you use a mask in the R channel, it will cause misalignment. However, if you place the mask directly in the main slot, thereby avoiding channel merging, the alignment is correct.
Parts I've verified to have this problem for the channels packing are the following: Details Mask, Shadow Maps,Shadow Masks,Matcap Masks,Rimlight Mask&Bias, Any Global Mask. Realistically it appears to be anything that has slots for channel packing. I'm sure the rest have the same issue but that's the extent I'm willing to test right now.
In these two images, you can see a comparison of the seams at the wrist: one image shows correct alignment, while the other appears misaligned. In the inspector, it’s evident that the mask is being used in the R channel and is processed through your channel packing system, with the cyan color being present. This issue persists even after merging the texture, as it becomes baked into the texture itself.
Final note: It almost seems like the image is being processed and filtered in an unexpected way. I tried matching the filtering settings to get it to align correctly, but it still seems off. This behavior is a bit strange and difficult to pinpoint.
https://github.com/user-attachments/assets/dade680f-1c1c-41bd-8532-f8914976d532

What type of texture are you using? The video does not seem to be working for me.

Default? I've already tried matching the settings with linear and everything else.

After more extensive testing, I think I’m closer to identifying the issue. In this case, I’m working with an 8K texture resized down to 4K using a bilinear filter. The issue seems directly related to resizing, as it only impacts the 8K texture resized within Unity. In contrast, another texture, “Original mask_4k,” was downscaled in an external editor using bilinear filtering and doesn’t exhibit this problem, likely because it bypasses resizing. (I've also tested Mitchell)
I suspect that the resizing algorithm in use here may differ from Unity’s standard options or could be bugged. It would be helpful to support both resizing algorithms that Unity offers and to reference the original texture’s filter setting.
Please disregard the fact that I was using an 8K texture; it was purely for testing purposes. I was testing upscaling and inspecting seams, which is how I noticed this issue. My primary goal is to bake the texture to new UVs and clean it up as needed. I suspect this resizing issue could occur at other resolutions, though I haven’t tested extensively.
https://github.com/user-attachments/assets/8acae07f-c291-4e8d-a799-3b7366fed482

What type of texture are you using? Is it a png, a jpg, a tga?

png

Set your filtermode on your 8k texture to Billinear or Trilinear instead of Point. That should help with the resizing during the packing

Just a heads up—the issue still persists in my version of the packer provided with Poiyomi. I’ve created a test package that replicates the problem in a streamlined way. Interestingly, the issue can be reproduced without resizing the texture; it appears related to the filtering setting in the advanced options, specifically when set to either bilinear or trilinear. It seems as though it’s filtering twice, which gives the appearance of offsetting the texture.
I’m moving on from this, as I’m confident I’ve demonstrated that the issue exists. It may be considered negligible, but the issue is definitely present. Good luck!
Issue-ThryPackerTest-UnityPackage.zip
https://github.com/user-attachments/assets/1e085201-1780-4a53-811b-cb51cb97c50a

Issue is reproduceable with provided assets


Auto indent in json doesn't work that well

VSCode Version: Code - Insiders 1.14.0-insider (15b579d13dbf6817bb493feff6f36cec87577929, 2017-06-27T05:11:33.154Z)
OS Version: Linux x64 4.10.0-24-generic
Extensions:

Extension
Author (truncated)
Version

EditorConfig
Edi
0.9.3

lorem-ipsum
Tyr
1.0.0

sort-lines
Tyr
1.3.0

vscode-svgviewer
css
1.4.2

tslint
eg2
0.15.0

git-project-manager
fel
1.3.2

md-navigate
jri
0.0.1

vscode-scss
mrm
0.6.2

seti-icons
qin
0.1.3

vim
vsc
0.9.0

(1 theme extensions excluded)

#29492

Move down:

Move up:

This is because we don't have indentation rules for JSON. Now add a simple version which just checks brackets.


flake8 doesn't recognize magic command

Hey! I'm getting the following error when I'm using the %pylab inline command:
Foo.ipynb:cell_2:1:1: F821 undefined name 'pylab'

Is it expected?
Steps to recreate this error:
notebook:
%pylab inline

pylab.plot([1, 2, 3], [1, 2, 3])

ndqa:
nbqa flake8 Foo.ipynb

Hey @subpath
Yes, it looks like pylab magically imports matplotlib objects, but that's not something that can be detected by a static analysis tool like flake8 which doesn't run your code.
So I don't think there's much we can do here
Thanks anyway for the report, much appreciated, please do let me know if you run into anything else!


linalg functions accepting float16

Currently some linalg functions accepts float16, which should fail as in NumPy.

https://github.com/cupy/cupy/pull/2509#discussion_r331863033
https://github.com/cupy/cupy/pull/2526#discussion_r335804411

Just sharing some knowledge from @IvanYashchuk as it seems relevant here https://github.com/chainer/chainer/pull/7995#issuecomment-547777423.


Fix #8162 Spark timeout in BDC cluster is higher than client-side, causing successful sessions to show as failed

@Charles-Gagnon yes I do. Long term, I think we can delete file but given the very tight deadline I'll be super conservative and just fix in both places today.


Change Hover#range to be optional

The docs of Hover#range in monaco.d.ts and modes.ts state:

When missing, the editor will use the range at the current position or the current position itself.

This implies that the range field should be optional as it can be missing. When looking at the references to Hover#range, we find out that there might pop up warnings in modesContentHover.ts when vscode is compiled with strictNullChecks later.

Looks good, merging but as said earlier: It is unlikely that monaco.d.ts. will properly support strict-null-checks because a good chunk of it is taken directly from our sources


may unexpectedly block event loop

'use strict'

var from = require('from2')
var tick = 0
var stream = from.obj(function (size, cb) {
  cb(null, {tick: tick++})
})

stream.once('data', function (data)  {
  console.log(data)
})

setTimeout(function () {
  process.exit()
}, 1000)

Expected behavior to print out {"tick": 0} and  for the process to exit after 1000 milliseconds, instead the process never exits because the (infinite) from stream is blocking the event loop (even though we use once)

This is how readable streams work (unfortunately). Once you add a data listener it enters flowing mode. After that it is your own responsibility to .pause/.resume the stream. Streams run in the same tick for perf reasons.


README Link is not added and other links are not functional

Link: https://github.com/secure-device-onboard/pri-fidoiot/blame/master/README.md
Line No 214, 313, 390,500:Can ReadMe link can be added.
Line No 375,437 : Link(https://verify.epid-sbx.trustedservices.intel.com) is not working
Line No 376,438 : Link(https://verify.epid.trustedservices.intel.com/) is not working.
Line No 376,438: Link (https://localhost:1180) is not working.

https://verify.epid-sbx.trustedservices.intel.com, https://verify.epid.trustedservices.intel.com/ are the URL for the hosted verification service. If you want to check if the service is up and running you need to run the healthcheck API, simply clicking on the url is not enough.
Healthcheck API for verification service: http://verify.epid-sbx.trustedservices.intel.com/health, http://verify.epid.trustedservices.intel.com/health respectively.
For the localhost URL to work the onprem verification service should be running and appropriate healthcheck API should be run to see the service status. Read the README files includes in the RV onprem package for more info.

Updated in PR#72


Fail to run project/hn-sinc-nsf-9/00_demo.sh on Google Colaboratory

Hello. I tried to run project project/hn-sinc-nsf-9/00_demo.sh on Google Colaboratory but  I failed. The contents of  log_err are as follows;
/usr/local/lib/python3.6/dist-packages/torch/functional.py:516: UserWarning: stft will require the return_complex parameter be explicitly  specified in a future PyTorch release. Use return_complex=False  to preserve the current behavior or return_complex=True to return  a complex output. (Triggered internally at  /pytorch/aten/src/ATen/native/SpectralOps.cpp:653.)
  normalized, onesided, return_complex)
Traceback (most recent call last):
  File "main.py", line 170, in <module>
    main()
  File "main.py", line 114, in main
    trn_set, val_set, checkpoint)
  File "/content/project-NN-Pytorch-scripts/core_scripts/nn_manager/nn_manager.py", line 336, in f_train_wrapper
    epoch_idx, optimizer, normtarget_f)
  File "/content/project-NN-Pytorch-scripts/core_scripts/nn_manager/nn_manager.py", line 153, in f_run_one_epoch
    loss_computed = loss_wrapper.compute(data_gen, normed_target)
  File "/content/project-NN-Pytorch-scripts/project/hn-sinc-nsf-9/model.py", line 957, in compute
    pad_mode="constant")
  File "/usr/local/lib/python3.6/dist-packages/torch/functional.py", line 516, in stft
    normalized, onesided, return_complex)
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!

It seems that torch.hann_window in class Loss returns the Tensor on cpu, which is incompatible with outputs (on CUDA) at line 955 in  project/hn-sinc-nsf-9/model.py.   I uploaded the jupyter notebook file which I used to gist.github.com[1] so please see it for detail.

https://gist.github.com/taroushirani/a263d1c33a23191ad0cedacf76f36409

@taroushirani Thanks for pointing it out.
My mistake -- torch.window() should be used with dtype and device so that the tensor of the window can be created on device.  Please check the changed files in the linked pull request.
It did not cause problem when window length != FFT_points, which is the case for all the model.py with default configurations.  In this case,  "window will be padded on both sides to length n_fft before being applied." https://pytorch.org/docs/stable/generated/torch.stft.html.  There, I guess that the padding operation will automatically convert the tensor to GPU.
Anyway, I should have set dtype and device in torch.window().
Please have a try!

@taroushirani Thanks for pointing it out.
My mistake -- torch.window() should be used with dtype and device so that the tensor of the window can be created on device.  Please check the changed files in the linked pull request.
It did not cause problem when window length != FFT_points, which is the case for all the model.py with default configurations.  In this case,  "window will be padded on both sides to length n_fft before being applied." https://pytorch.org/docs/stable/generated/torch.stft.html.  There, I guess that the padding operation will automatically convert the tensor to GPU.
Anyway, I should have set dtype and device in torch.window().
Please have a try!

Thank you for your rapid response.  I tried the master-debugging branch and confirmed the problem is fixed.
I may have misunderstood your reply, but I used the hn-sinc-nsf-9/model.py with the default configulations and I met this problem.  I also tried this model several months ago but I did not encounter any problems, so I suspect that the difference of versions of pytorch(1.6.x->1.7.1) causes this problem.
Anyway the problem is solved. Thank you very much.

Thank you for your rapid response.  I tried the master-debugging branch and confirmed the problem is fixed.
I may have misunderstood your reply, but I used the hn-sinc-nsf-9/model.py with the default configulations and I met this problem.  I also tried this model several months ago but I did not encounter any problems, so I suspect that the difference of versions of pytorch(1.6.x->1.7.1) causes this problem.
Anyway the problem is solved. Thank you very much.

Thank you for the information!
I used pytorch 1.6.  I haven't tried 1.7

Thank you for the information!
I used pytorch 1.6.  I haven't tried 1.7


[Bug] Trim trainling spaces not done in comment parts

Hello,
On the last version at this day, commented php part are not properly processed to trim trailing spaces.
Exemple:
        $this->view->query = $query;
        $this->view->listing = $listing;
        $this->view->ajax = $ajax;
        $this->view->paginator = $paginator;
        $this->view->areas_select = $areasSelect;
        $this->view->medias_select = $mediasSelect;
        $this->view->brands_select = $brandsSelect;
        $this->view->targets_select = $targetsSelect;
<= here OK
        /*
                // Search (query)
                $query = $request->get('query');
                <= here KO
                if (!empty($query))
                {
                    $news->addConditionParam('title LIKE ?', "%{$query}%");
                    $news->addConditionParam ('description LIKE ?', "%{$query}%", 'OR');
                    $filters[] = "query={$query}";
                }
                <= here KO
                // Pagination
                $filters[] = "page={$request->get('page')}";
                <= here KO
                // $filters_string = '?' . implode('&', $filters);
        */

I think for file coherence it should be done in this part too. Usualy when "trim trainling spaces" features are applied on a file, it is on all the file.
I should be applied in the case of mixed content too (html/php). I don't think it can break HTML code (HTML space are important because it can change behaviour of how diplayed, but as this is empty line, the empty line will act like a space for hmtl displaying). Expert could confirm ?
Thanks.

for the space in comments you can enable the no_trailing_whitespace_in_comment rule

I use @PhpCSFixer and i have not modified this rule.
Here is the ruleset:
    ->setRules([
        '@PhpCsFixer'                         => true,

        // one space when concatenation
        'concat_space' => ['spacing' => 'one'],

        // force class element order alphabetically
        'ordered_class_elements'              => ['sortAlgorithm' => 'alpha'],

        // if views are fixed better to activate those
        'no_alternative_syntax'               => false, // this one do the bug
        'no_short_echo_tag'                   => false,
    ])

So rules doesn't seems to work ?

I think this is the same issue as https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/3819 , the no_trailing_whitespace_in_comment remove the trailing space but braces re-adds it

closing as duplicate

I confirm same, thanks. Sorry for polluting with this issue.

no problem! :)

fixed in https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/3820

Thanks again for your fast fix : )


create github action file for CI/CD

create github action file for CI/CD

It's working on normally! Thanks :)

This ticket has been closed @bart2001


I cannot run any sample code. What is required?

I've tried to run the sample code on Linux and Windows using python 3.10 but I could not.
I think there are some requirements that need to be installed first.
I installed them but the result is the same, sample codes do not run properly, they give errors.
I think there are some undocumented requirements, what are they?
Even though I followed each line of the book, sample codes do not run.

Hi @gokhankocak, could you specify which code you are trying to run? Then, I will be able to help :)


Staff page update

Thomas Greer appointed as Media Manager

Source: https://community.vatsim.uk/news/marketing/appointment-media-manager-r334/


Reverse AutoMapper mapping in MappedEntityDomainManager throws if not defined

The MappedEntityDomainManager class makes this call in various places:
Mapper.Map<TData, TModel>(data, model);

This implies a mapping from a data transfer object to an EF entity. We never do that, it is bad practice, and even Jimmy Bogard speaks out against reverse mapping (he made AutoMapper). StackOverflow is full of explanations why this is a bad idea.
(And even if I want to do this I can't, as the destination entities have navigation properties, so the mapping makes no sense and won't work... which is incidentally why I'm using this class to begin with, because I thought it is supposed to support non 1:1 mapping.  :confused:)
So, we define a mapping for TModel -> TData, but not for TData -> TModel. Thus whenever that call is made, it throws.  :sob:
Is there a workaround?

BTW I know this is happening because, for example, if I want to delete an entity, then in Postman I can see the response contains `exceptionMessage: "Missing type map configuration or unsupported mapping.\r\n\r\nMapping types:\r\nFooDTO -> Foo\r\nsome.namespace.Models.FooDTO -> some.namespace.Models.Foo"`.

Actually a workaround is to override that method in my subclass, and not make that call.
BUT. Can someone please explain why it's there? I have soft delete on, so execution enters that block of code. What is that mapping supposed to do?
Can I safely remove it?

I'm closing this, as the bigger issue is discussed in my issue here.


Cannot download complete road network when given a polygon boundary

I used the ox.graph_from_polygon function to extract road network using a polygon boundary yet when I examined the graph, half of the area does not have any network. I checked OSM's original website and there exists data for the missing section. I am wondering what might be the issue here and how to solve it. Thanks so much!

Please reread the new issue template and completely fill it out. Incomplete issues are automatically closed.


Fix: Removed type hint from documentation for closures from database queries

Based on http://forum.codeigniter.com/thread-74714.html
Using an example from the documentation, the developer might not understand that the BaseBuilder type hint in the closure is a reference to the Codeigniter\Database\BaseBuilder class.
By specifying a BaseBuilder type hint without using "use Codeigniter\Database\BaseBuilder", a developer will receive a type error.
Checklist:

[x] User guide updated

I think it is better to mention the use statement instead of removing type hint.

I think it is better to mention the use statement instead of removing type hint.

@samsonasik In fact, this type hint does not affect anything. Only those who use the IDE can feel its benefits. Such developers will be able to add a type hint to their code themselves.

@samsonasik In fact, this type hint does not affect anything. Only those who use the IDE can feel its benefits. Such developers will be able to add a type hint to their code themselves.

Using type hint doesn't change much here, it's true - but I feel like we should promote good practices, and using type hint is one of them.

Using type hint doesn't change much here, it's true - but I feel like we should promote good practices, and using type hint is one of them.


Hotfix: Icon, Title, Deployment

WHAT DOES THIS PR DO?

Go to https://cs-472-group-2-project.vercel.app/ and test the API endpoints

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (c2f9e49) 91.56% compared to head (007db75) 91.56%.
Report is 5 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #164   +/-   ##
=======================================
  Coverage   91.56%   91.56%
=======================================
  Files          29       29
  Lines         830      830
  Branches        6        6
=======================================
  Hits          760      760
  Misses         70       70

Flag
Coverage Δ

web
100.00% <ø> (ø)

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

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


Version tags for docker images

Currently the only tag available for the docker image is latest.
Would it be possible to have tags for each release?

Yes this can be done using tags. This action is nearly what you want. It just uploads to dockerrhub instead of ghcr
https://github.com/openmc-dev/openmc/blob/develop/.github/workflows/dockerhub-publish-release.yml#L13

I was just looking at this ;-)


rendering is broken in Firefox after v7.4.0

Hey, thanks for a fantastic library!
blockquote rendering seems to have broken sometime between v7.3.2 and v7.4.0 and is still broken on the latest release.
The bug only affects Firefox (v122), and does not affect Chrome or Safari.
This can be reproduced the demo ( https://markdown-to-jsx.quantizor.dev/ ) in Firefox:
Enter the following:

hello

The following is rendered:

hello
</blockquote>

In other browsers the blockquote is rendered correctly.

Should be fixed with https://github.com/quantizor/markdown-to-jsx/releases/tag/v7.4.3
Sorry about that!


Add support for multiple RPC providers

Currently supports a single choice of RPC provider. To add support for failing over from a primary to a secondary RPC provider, based on an error response/no response from the primary

One could do this localy by creating multiple Providers but we can create it also in the lib./
I'm marking this as only dust, his should be simple to do. we could call it fallbackRpcProvider or it could be implemented inside existing rpc provider?

Ethers.js has a rather complex logic regarding this IIRC, we could try and go with something simple


Register Phys Flows - warn if canceling / moving away

@kamransaleem

An example of how someone else approached the same.
http://www.markcampbell.me/tutorial/2013/10/08/preventing-navigation-in-an-angularjs-project.html
He uses root scope, but am sure that could be abstracted into a service


What's wrong! I can capture nothing except skybox?

I build a simple scene in Unity2017.1.0f3 to test capture something ,but when capture started,the camera renderering nothing except skybox,finally i only have a skybox vedio.What happened?What should i do?
The attachment is the simple scene,please take a look at it when you have free time,thank you!
CaptureTestScene.zip

Nice catch. I found a small but critical bug in the camera capture component.
The bug has been already fixed. Please try v0.1.2.
https://github.com/keijiro/FFmpegOut/releases
I'm closing this issue. Please feel free to reopen if you find further issues.

thank you！！！


compilation failure due to switch from fabs to std::abs

Compiling today, I ran into a problem:
src/glop/basis_representation.cc: In member function ‘operations_research::glop::Fractional operations_research::glop::BasisFactorization::ComputeInverseOneNorm() const’:
src/glop/basis_representation.cc:555:51: error: call of overloaded ‘abs(__gnu_cxx::__alloc_traits<std::allocator<double> >::value_type&)’ is ambiguous
       column_norm += std::abs(right_hand_side[row]);
                                                   ^
In file included from src/base/logging.h:18:0,
                 from src/glop/basis_representation.h:18,
                 from src/glop/basis_representation.cc:15:
/usr/include/stdlib.h:735:12: note: candidate: int abs(int)
 extern int abs (int __x) __THROW __attribute__ ((__const__)) __wur;
            ^
In file included from /usr/include/c++/5.4.0/ext/string_conversions.h:41:0,
                 from /usr/include/c++/5.4.0/bits/basic_string.h:5249,
                 from /usr/include/c++/5.4.0/string:52,
                 from /usr/include/c++/5.4.0/bits/locale_classes.h:40,
                 from /usr/include/c++/5.4.0/bits/ios_base.h:41,
                 from /usr/include/c++/5.4.0/ios:42,
                 from /usr/include/c++/5.4.0/ostream:38,
                 from /usr/include/c++/5.4.0/iostream:39,
                 from src/base/logging.h:19,
                 from src/glop/basis_representation.h:18,
                 from src/glop/basis_representation.cc:15:
/usr/include/c++/5.4.0/cstdlib:166:3: note: candidate: long int std::abs(long int)
   abs(long __i) { return __builtin_labs(__i); }
   ^
/usr/include/c++/5.4.0/cstdlib:174:3: note: candidate: long long int std::abs(long long int)
   abs(long long __x) { return __builtin_llabs (__x); }
   ^

Seems to be caused by commit fe66d48cee746408f2093a083b073aa8d60cec4c
re-ran "make third_party" with no change in outcome.  I'm currently cleaning that out and redoing again just in case, and will continue to try to track it down on my end.
using  gcc (GCC) 5.4.0 on slackware linux 14.2 (current)

Following some hints from similar issues on the internet, and looking at the above error message, I did the following patch to fix the issue:
diff --git a/src/base/logging.h b/src/base/logging.h
index 2fdd6f613..ad995a112 100644
--- a/src/base/logging.h
+++ b/src/base/logging.h
@@ -16,6 +16,7 @@

 #include <assert.h>
 #include <stdlib.h>
+#include <cmath>
 #include <iostream>  // NOLINT
 #include "base/commandlineflags.h"
 #include "base/integral_types.h"

It seems like bad practice to put "#include " into a file called logging.h and have that fix compilation problems related to the use of absolute value.  So there is probably a better fix.  In any event, logging.h is called enough that it gets the include cmath in where it needs to go I guess.

Hi,
Can you just try by adding #include  in the .cc file?
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2017-03-28 22:25 GMT+02:00 James Marca<EMAIL_ADDRESS>
Following some hints from similar issues on the internet, and looking at
the above error message, I did the following patch to fix the issue:
diff --git a/src/base/logging.h b/src/base/logging.h
index 2fdd6f613..ad995a112 100644
--- a/src/base/logging.h
+++ b/src/base/logging.h
@@ -16,6 +16,7 @@
#include <assert.h>
#include <stdlib.h>
+#include
#include   // NOLINT
#include "base/commandlineflags.h"
#include "base/integral_types.h"
It seems like bad practice to put "#include " into a file called
logging.h and have that fix compilation problems related to the use of
absolute value. So there is probably a better fix. In any event, logging.h
is called enough that it gets the include cmath in where it needs to go I
guess.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/356#issuecomment-289893580,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17cFEYdgG-dSrWdj1mFCsf9uBBsveks5rqWzTgaJpZM4MsD-c
.

Okay.  Did that.  So far I've needed to add #include <cmath> into the following cc files:
Unstaged changes (4)
modified   src/glop/basis_representation.cc
modified   src/glop/lu_factorization.cc
modified   src/glop/markowitz.cc
modified   src/glop/update_row.cc

Each time I added the line, compilation moved on to the next file, I added that line, etc.
Now there is a new error:
src/sat/boolean_problem.cc: In function ‘void operations_research::sat::UseObjectiveForSatAssignmentPreference(const operations_research::LinearBooleanProblem&, operations_research::sat::SatSolver*)’:
src/sat/boolean_problem.cc:236:78: error: ‘fabs’ was not declared in this scope
                           fabs(static_cast<double>(objective.coefficients(i))));
                                                                              ^
src/sat/boolean_problem.cc:240:60: error: ‘fabs’ was not declared in this scope
         fabs(static_cast<double>(objective.coefficients(i))) / max_weight;
                                                            ^
make: *** [makefiles/Makefile.gen.mk:1711: objs/sat/boolean_problem.o] Error 1

Because the patch I ref'd above seemed to swap out "fabs" for "std::abs"  I am going to do that and see if I can't move on some more.
(Please be aware, I'm a competent programmer but the last time I really used C++ was back in 1992, using Stroustroup's first book on the language.  I'm remembering by doing here!)

Okay, that worked.  I created a branch on my clone:
https://github.com/google/or-tools/compare/master...jmarca:bug/compilation_failure
Shall I make a pull request?  It is a pretty simple fix.

I think this is fixed by https://github.com/google/or-tools/commit/c36066c39cb79108293ee0bd83cfc4227a4eb380
which adds <cmath> in ortools/lp_data/lp_types.h


Question about setting is_training in normalizer params

Generally, we use a placeholder for the is_training flag of normalizer_params for batch norm to toggle between training and test mode. When using morph net regularization though, we can't set is_training to a placeholder since we see the following error:
Traceback (most recent call last):
  File "gamma_prune.py", line 47, in <module>
    network_regularizer = latency_regularizer.GammaLatencyRegularizer([out.op], gamma_threshold=gamma_scale, hardware='K80')
  File "/Users/vashishtmadhavan/Documents/Code/morph-net/morph_net/network_regularizers/latency_regularizer.py", line 90, in __init__
    force_group=force_group, regularizer_blacklist=regularizer_blacklist)
  File "/Users/vashishtmadhavan/Documents/Code/morph-net/morph_net/framework/op_regularizer_manager.py", line 126, in __init__
    self._op_handler_dict[op.type].assign_grouping(op, self)
  File "/Users/vashishtmadhavan/Documents/Code/morph-net/morph_net/framework/grouping_op_handler.py", line 35, in assign_grouping
    input_ops, op_reg_manager)
  File "/Users/vashishtmadhavan/Documents/Code/morph-net/morph_net/framework/op_handler_util.py", line 73, in get_ops_without_groups
    op_slices = op_reg_manager.get_op_slices(op)
  File "/Users/vashishtmadhavan/Documents/Code/morph-net/morph_net/framework/op_regularizer_manager.py", line 437, in get_op_slices
    size = op_handler_util.get_op_size(op)
  File "/Users/vashishtmadhavan/Documents/Code/morph-net/morph_net/framework/op_handler_util.py", line 505, in get_op_size
    shape = op.outputs[0].shape.as_list()
  File "/Users/vashishtmadhavan/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/tensor_shape.py", line 904, in as_list
    raise ValueError("as_list() is not defined on an unknown TensorShape.")
ValueError: as_list() is not defined on an unknown TensorShape.

What's the appropriate strategy here? Would we just set is_training to False for both training and testing or True?

The short answer is to use  is_training=True.
This must be so as you want batch norm params to be updated and network use batch statistic during training. However not freezing stats during test will result in a more noisy test accuracy (possibly lower), this is not really a problem as you don't care too much about the accuracy during the structure learning phase where the MorphNet regularizer is on. During the retraining phase, you of course do not need the network regularizer at all and you will be able to pass is_training as a place holder with no problem.
BTW, is there a specific reason you are using is_training as a place holder?

I see that makes sense. I found a work around where I'm just separating the training and validation so i removed the need for a placeholder. thanks for the help


updated node version

Happily we don't need to enfore this version. Anyway thanks for looking into it.


Added instructions for configuring nltk

I'm not 100% sure that "stopwords" is the best corpus, so feel free to ask me to update it to something else.

Thank you for contribution :+1:

Stopwords is enough for the project requirements.


PO - Define SLI's in cooperation with SRE

Having delivered one SLI on charge master data, lets continue the monitoring by identifiying and implementing SLI's with SRE.
This is a placeholder for touch bases with SRE, with the intention of defining three SLIs;

Defined Service Level Indicators measuring rejections of charge RSM's (ie. charges, charge prices, and charge links)
SLIs measurement is implemented for reporting on the Service level of the past 90 days
Here and now indication of the Service Level (Past one hours performance of the SLI)

Tech note:
** we are also interested in logging.

Will call for a meeting, once team volt has estimated the user story.

Configuration of solution, e.g. function names, connections, would like to be included in solution, prior to volt involvement.
Having this in place, volt are prepared to identify and implement these.


How to send "/me" message?

I am having trouble learning how to do this. Thanks in advance

I think I found it. action_reply?

Yes, you are correct that http://www.rubydoc.info/gems/cinch/Cinch/Message#action_reply-instance_method can work. There is also http://www.rubydoc.info/gems/cinch/Cinch/Target#action-instance_method so you can do Channel('#test').action('waves') or User('joe').action('waves') and the bot will wave in-channel or in PM to that user.

Thank you!


Modifications when batch_size of data and label are not the same

Modifications for the python training code, when the of the batch_size of input data and label are not the same. Related to issue #214. Modifications for fast-rcnn training.
For now we can call model.fit(X=your_data_iter) to train the network where your_data_iter has different batch_size for the image and label.

Can you work around this by reshaping the label into (data_batch_size, label_batch_size/data_batch_size, ...)

@piiswrong Can I add a flatten layer for the label and used the flattened label as input for the loss layer?

Yes you can add a flat layer to flat labels (see the current symbol constructed for char-LSTM for example). There is however one caveat for that: once you start applying operators on the label, during prediction time, if you want to bind an executor without providing label shape (because there is no label), it will fail because the shape inference for the label pipeline will fail. Of course you can always use a slightly different symbol without the whole label pathway. But just to let you know of this potential issue.

Which value of the batch_size for optimizer's rescale_grad=(1.0/batch_size) should be set in this case? Is it the batch_size of label?

What should the rescale_grad set in this situation?


Update License

Update license year.

@SuryaVamsi I am closing this PR since we merged #1332. Thanks again.


Color blocks could show the resulting color

It was suggested that color blocks could show the resulting colors on the block itself, instead of waiting until the program is run.

This was an idea that came up during a testing session last night.  I'm not actually sure that I like it, though.  It might be a bit noisy UI-wise, and #194 might be the better approach for getting immediate feedback.
(On the other hand, you could argue that giving immediate feedback about intermediate values like this is an even greater opportunity than merely doing a continuous update of the final program.  You could also argue, then, for preview options for other types as well.  This would be an amazing direction to go, eventually.)

Are you referring to color blocks such as blue, green and so forth? If so maybe a picture of a paint bucket in the correct color might be a compromise?

Yes, I definitely like the idea of color literals showing the color.  This also has the advantage that it doesn't sound as hard.
The more interesting idea was to have color expression blocks - such as mixed, light, etc., also show their resulting color, computed on the fly based on their arguments.  This extrapolates to other types as well: perhaps picture-type blocks could have a mini-picture on them.  Or number blocks could show the values of intermediate expressions.
Of course: (a) this would only work for expressions without local variables, and (b) it would get very noisy, so if it were implemented, maybe it should be moved to a "Preview" option in the context menu, or some such thing.  This is certainly not a priority, but I could see it being a valuable debugging tool.

I'll split the preview thing into a separate issue, and leave this one for the literal color icons.


query hits not match

The run log show get 120 hits，but use dsl from trace log to query ES，the actual hits is 240？
How to solve it？

Try adding
query_delay:
  minutes: 5


Add debounce to drawing

added a debounce of 1.5s.
I don't know if it's from my side, but the image manipulation following the debounce takes longer.
test if from your end and let's see

Another fantastic piece of work! Thank you! I tried it out on my end and it seems alright time wise. I think the debounce makes it feel like the image manipulation is taking longer but it's really that it waits 1.5s for debounce and then the image manipulation takes another 2-3s to do its thing.
We can always lower the debounce time in the future if needed but I think the big thing it needs is a spinner letting the user know that some thinking is going on when the image manipulation kicks in. I'll make a new issue for it for the future.

alright. I get it👌


PowerShell does not accept paths with spaces in -File argument

Steps to reproduce

Create a folder with space in it (for example my folder)
Create a simple script inside this folder (script.ps1)

Write-Output "Hello, World"

run with command pwsh -File "/Users/username/my folder/script.ps1"

Expected behavior
Script executes successfully:
Hello, World

Note: script does execute correctly this way if run under PowerShell Desktop under Windows
Actual behavior
➜ /Users/username pwsh -File "/Users/username/my folder/script.ps1"
The argument 'my' is not recognized as the name of a script file. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Usage: pwsh[.exe] [-Login] [[-File] <filePath> [args]]
                  [-Command { - | <script-block> [-args <arg-array>]
                                | <string> [<CommandParameters>] } ]

Same under Windows and PowerShell 6.2.4
C:\work\src>pwsh -File "my folder\script.ps1"
The argument 'my' is not recognized as the name of a script file. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0
PSEdition                      Core
GitCommitId                    7.0.0
OS                             Darwin 19.3.0 Darwin Kernel Version 19.3.0: Thu Jan  9 20:58:23 PST 2020; root:xnu-6153.81.5~1/RELEASE_X86_…
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Windows part
Name                           Value
----                           -----
PSVersion                      6.2.4
PSEdition                      Core
GitCommitId                    6.2.4
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

It sounds like you installed PowerShell as .NET global tool (dotnet tool install powershell -g)
Sadly, a PowerShell version installed this way uses a shim executable whose argument parsing is fundamentally broken  - please see #11747
By contrast, a regularly installed PowerShell 7  version isn't affected.

@mklement0 yes, that's indeed how I installed it. This issue can be closed as a duplicate then.
Thank you

Closed as a duplicate of #11747


Still Getting 2FA error

Still Getting 2FA error. I'm still getting "Could not get auth token" error, even after login/logout in my account and running the program again:
C:\comix-main>python comix.py list
Getting auth token . . .
{"response": {"challenge": {"challenge_reason": "HandleOnWebView", "uri": "https://www.amazon.com/ap/challenge?arb=370d12b1-17b0-4720-b6be-c2f4d779cef3"}}, "request_id": "01cd1df43kilokh521548c0132066"}
Amazon needs to reconfirm your identity with 2FA. Please logout and login again in your browser.
Traceback (most recent call last):
File "C:\comix-main\comix.py", line 152, in
cmx = Cmx(None)
^^^^^^^^^
File "C:\comix-main\comix.py", line 27, in init
raise Exception("Could not get auth token")
Exception: Could not get auth token

Solved: I've just deleted the device_ID file from comix-main and then hit the "python comix.py list" again.


CB-12963: npm-link all dependencies

Platforms affected
What does this PR do?
What testing has been done on this change?
Checklist

[x] Reported an issue in the JIRA database
[x] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
[ ] Added automated test coverage as appropriate for this change.

@alsorokin would you please take a look?


Type narrowing failure?

(Filed per request from @concavelenz after internal discussion:)
Compiling
/**
 * Add all reachable objects to a list, recursively.
 * @param {*} node JavaScript value to search.
 * @param {!Array<!Object>} objects Array to add objects to.
 */
function add(node, objects) {
  if (!node || (typeof node !== 'object' && typeof node !== 'function') ||
      objects.includes(node)) {
    return;
  }
  objects.push(node);
  var names = Object.getOwnPropertyNames(node);
  for (var i = 0; i < names.length; i++) {
    add(node[names[i]], objects);
  }
}

with -O=ADVANCED_OPTIMIZATIONS gives some unexpected warnings:
input0:8: WARNING - actual parameter 1 of Array.prototype.includes does not match formal parameter
found   : *
required: Object
      objects.includes(node)) {
                       ^^^^

input0:11: WARNING - actual parameter 1 of Array.prototype.push does not match formal parameter
found   : *
required: (Object|undefined)
  objects.push(node);
               ^^^^

input0:12: WARNING - actual parameter 1 of Object.getOwnPropertyNames does not match formal parameter
found   : *
required: Object
  var names = Object.getOwnPropertyNames(node);
                                         ^^^^
0 error(s), 3 warning(s), 100.0% typed

I say "unexpected" because usually the compiler is very good at type narrowing based on conditionals, so I was expecting that that after evaluating !node || (typeof node !== 'object' && typeof node !== 'function') || it would infer that node must be an object in the call objects.includes(node), and similarly in the call to .push if it is reached.
Of course, it is possible that the compiler has correctly inferred that node could be an object but not an Object—but this seems to be contrary to its usual tendency to conflate x being non-primitive with  x instanceof Object, so I'm not sure why it isn't doing that here.
(Interestingly, if I change the declaration at the top to @param {!Object|string|number|boolean|undefined|null|Symbol} node, then the error changes to:
input0:8: WARNING - actual parameter 1 of Array.prototype.includes does not match formal parameter
found   : (Object|boolean|number|string)
required: Object
      objects.includes(node)) {
                       ^^^^

which suggests that the problem is that it has not understood that typeof node !== 'object' && typeof node !== 'function' will evaluate to true for booleans, numbers and strings.)

Looks like the compiler doesn't do any type narrowing for the case where typeof <something with all type> !== 'something' is false (so your last comment is correct). Not just for function/object but for the primitive types as well. Relevant code is here:
https://github.com/google/closure-compiler/blob/6cac56177d9ccac725a784dc6e215f6f125ad797/src/com/google/javascript/jscomp/type/ChainableReverseAbstractInterpreter.java#L622
For example:
/**
 * @param {*} node
 */
function f(node) {
  if (typeof node !== 'boolean') {
    // the compiler doesn't narrow `node` down from *
    var /** null */ n = node; // found:  *, required: null
  }
}

debugger


Clone is not defined svg-morpheus.js:231

I am trying to morph an svg
 var myIcons = new SVGMorpheus('#iconset')
 myIcons.to('icon1')

I am seeing the following error:
svg-morpheus.js:231 Uncaught ReferenceError: clone is not defined
I figured maybe this was jQuery's clone method you are trying to use so I included jQuery, but I am still having the same issue.

The clone method is in helpers.js.


Running Cycle GAN error after epoch3

Hi After epoch 3, There is a call to this fuunction
callbacks=[
ModelSaver(),
ScheduledHyperParamSetter(
'learning_rate',
[(100, 2e-4), (200, 0)], interp='linear'),
PeriodicTrigger(VisualizeTestSet(), every_k_epochs=3), #Value changed from 3 just for training
],vvvv
which throws error. I am not sure why this error comes, But I am guessing it is because of dataflow function in preprocess.py called  "PrefetchDataZMQ" Object has no attribute socket. I am not sure what this really means. I am attaching the error below.
W tensorflow/core/kernels/queue_base.cc:295] _0_QueueInput/input_queue: Skipping cancelled enqueue attempt with queue not closed
Traceback (most recent call last):
File "tensorpack/examples/GAN/CycleGAN.py", line 234, in
GANTrainer(config).train()
File "/home/NOVA/akhil.singh-rana/MyWorkspace/MyProjects/tensorpack/env/lib/python2.7/site-packages/tensorpack/train/base.py", line 136, in train
self.main_loop()
File "/home/NOVA/akhil.singh-rana/MyWorkspace/MyProjects/tensorpack/env/lib/python2.7/site-packages/tensorpack/train/base.py", line 241, in main_loop
self._callbacks.trigger_epoch()
File "/home/NOVA/akhil.singh-rana/MyWorkspace/MyProjects/tensorpack/env/lib/python2.7/site-packages/tensorpack/callbacks/base.py", line 151, in trigger_epoch
self._trigger_epoch()
File "/home/NOVA/akhil.singh-rana/MyWorkspace/MyProjects/tensorpack/env/lib/python2.7/site-packages/tensorpack/callbacks/group.py", line 91, in _trigger_epoch
cb.trigger_epoch()
File "/home/NOVA/akhil.singh-rana/MyWorkspace/MyProjects/tensorpack/env/lib/python2.7/site-packages/tensorpack/callbacks/base.py", line 151, in trigger_epoch
self._trigger_epoch()
File "/home/NOVA/akhil.singh-rana/MyWorkspace/MyProjects/tensorpack/env/lib/python2.7/site-packages/tensorpack/callbacks/trigger.py", line 43, in _trigger_epoch
self.cb.trigger()
File "/home/NOVA/akhil.singh-rana/MyWorkspace/MyProjects/tensorpack/env/lib/python2.7/site-packages/tensorpack/callbacks/base.py", line 160, in trigger
self._trigger()
File "tensorpack/examples/GAN/CycleGAN.py", line 200, in _trigger
for iA, iB in self.val_ds.get_data():
File "/home/NOVA/akhil.singh-rana/MyWorkspace/MyProjects/tensorpack/env/lib/python2.7/site-packages/tensorpack/dataflow/prefetch.py", line 171, in get_data
dp = loads(self.socket.recv(copy=False).bytes)
AttributeError: 'PrefetchDataZMQ' object has no attribute 'socket'[0906 17:50:24 @input_source.py:203] EnqueueThread QueueInput/input_queue Exited.
I have changed the visualiyation to after epoch 193, Hope that helps me atleast train the model ?

Hi could you resolve visualization on test set after every 3 epochs.
Thanks,
Akhil

It should be fixed 14 days ago.

Hi thanks I will try again.
Cheers


ExtAuthz request with duplicated headers

Hi, I’m using gRPC extauthz filter in Envoy. Does anyone know how envoy parses headers into the CheckRequest sent to extauthz cluster? For example, suppose I have a client that sends multiple duplicated Host headers, what does the host field look like?  Will Envoy comma concatenate them?
cc: @dio

In general, I think https://github.com/envoyproxy/envoy/blob/dcdda09c0871be7a13fff8be4a644e0cebd16bf6/api/envoy/service/auth/v3/attribute_context.proto#L116 answers this.
The host header has specific rules around this, see https://httpwg.org/specs/rfc9112.html#request.target.

Thanks @htuch, is it possible to make this more explicit to note to which RFC does the handling of duplicated header names conform? Searching on the internet, I assume the value will be a comma separated list of values based on RFC7230?

Right. Feel free to propose proto comment or documentation updates in a PR to clarify for future readers.

@qixuan-li1 are you planning on working on this documentation improvement?

Sure, will have a PR on this.


Go OpenTracing Error Logging

Go's OpenTracing error logging uses different naming than that recommended by the  OpenTracing semantic conventions. Any reason for the discrepancy? or why it can't be reconciled?

@rnburn the naming predates the standard – this is basically just a bug, certainly no good reason for it to stay as-is.

Ok. Thanks. I'll put in a PR to update it.


Create New file4branch101

this is a testing branch

why need this one

it is because wanna have a test


Fix for xESMF 0.6.1

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes issue #xyz

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)
[ ] I have added my relevant user information to AUTHORS.md

What kind of change does this PR introduce?:
Minimal change to adapt to new xESMF 0.6.1. In that version, the weights attribute of the Regridder object is a DataArray, instead of a scipy sparse matrix. The change is retrocompatible.

Does this PR introduce a breaking change?:
No

Other information:

Are the non-implemented errors expected ?

Not at all and I can't see how my PR would have triggered those?

@ellesmith88 Any idea on what could be causing this ?

@aulemahal @huard From a quick look, this is to do with the newer releases of pandas and xarray. This error appears when using pandas>=1.3.0 and xarray>0.18.2. Seems to be to do with this PR https://github.com/pydata/xarray/pull/5359
Also see https://github.com/roocs/clisops/issues/185

@aulemahal @huard The suggested workaround for this for now is https://github.com/roocs/clisops/commit/c21275fd962ec7fbf48ec28c70b422a5bb37da11.
I've tested the fix locally in your branch and all tests passed for me. The fix is in a clisops branch - dask-error-workaround ( https://github.com/roocs/clisops/tree/dask-error-workaround), if you merge this in the issue should be resolved.
The PR looks good otherwise.

Thanks !


Explain the behavior of the  Requested Claim and Mandatory Claim

Explain the behavior of the  Requested Claim and Mandatory Claim. Current explanation is bit confusing .
Refer mail thread

Clarification regarding Requested Claims

[1] https://docs.wso2.com/display/IS540/Configuring+Claims+for+a+Service+Provider

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.