Bookkeeping commands cannot be trained from API

{"error":"Invalid ThingTalk","detail":"Parse error: unexpected token bookkeeping in state 0, expected (,answer,attimer,edge,executor,filter,monitor,now,policy,timer"}

chef kiss

This is fixed by https://github.com/stanford-oval/thingtalk/pull/89 and will be closed when the new version of ThingTalk is released.


Fix typo getBalances in README

name: Fix typo getBalance in README
about: Typo getBalance in useNativeBalance hook, should be getBalances
New Pull Request
Checklist

[x] I am not disclosing a vulnerability.
[x] My code is conform the code style
[x] I have made corresponding changes to the documentation
[x] I have updated Typescript definitions when needed

Issue Description
Fix documentation typo getBalance to getBalances in useNativeBalance hook.
Solution Description
Change getBalance to getBalances.

:tada: This PR is included in version 1.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Consider publishing the localstack chart to kubernetes/charts ?

Hello,
I am interested in making use of your localstack chart... I would be grateful if it were published in the official kubernetes/charts repo.
Do you have any interest in publishing to that repo, and or, would you mind someone else publishing it on your behalf?

@diclophis This repository is licensed under the MIT license, and the kubernetes/charts is Apache license. I am not a lawyer, but my understanding is that is legitimate. To be polite, I think we should try once more to get explicit permission though.
@everpeace Would you mind if someone republished it in the kubernetes/charts repo under an Apache license?

@StevenACoffman @diclophis sorry for my absent.....

Would you mind if someone republished it in the kubernetes/charts repo under an Apache license?

no.  Please go ahead.  I would be appreciated if you would cite my name Shingo Omura<EMAIL_ADDRESS>as the original developer of the chart.

Hey all, I've started a discussion on localstack and will post a working copy of the chart later once I've finished making it fit helm contribution guidelines, with your terms respected @everpeace. After assessing the work involved, without the localstack support we might have to reinvent docker images for security checks, strict tagging and other technical requirements to get the chart into a stable package. Hopefully we can contribute an incubator in the meantime.
https://github.com/localstack/localstack/issues/882

Nice. Possibly relevant is that I made a local ec2metadata service installable into minikube over at gimme-mini-aws

MIT licensing dictates that it needs to be contributed with the license.

We could add a line to say the MIT license doesn't apply to localstack chart once we approach that problem. Same way you would approach the problem using a chart in helm/chart within a licensed project.

@sidhuko were you able to get a localstack chart created, that fits with the Helm contribution guidelines?

Any progress on moving this over to kubernetes/charts?    kind of an edge case and since it's fairly useful to all.. would be good.
Is there anything community can do to assist here?

I've a draft but needs testing in multiple scenarios: https://github.com/sidhuko/charts/pull/1/files. Some changes on another chart were taking a long time to be accepted so I didn't proceed until it was accepted.
I'll see if I can get a pull request on helm in a few weeks once I've made fixes. Help always welcome.

To be more clear.  The MIT license and Apache License are not incompatible.  An MIT licensed project can be completely relicensed to Apache 2 as long as the original MIT license is mentioned.    Other than that, it is a very open license (even open to a project being relicensed so that additional code falls under a new license and not the original MIT.
That’s what I meant by you only need include the original MIT license of the original source.  That doesn’t mean it can’t follow the Apache license from that point forward.

What is the latest on this issue? Are we able to find an resolution to the license problem?

I'm not so familiar with oss licenses.  What kind of actions can I do to help to solve this issue?

I am not a lawyer but have experience interacting with lawyers on these matters.
There's no license problem.  Maybe my original comment wasn't clear.

https://github.com/helm/charts i.e. the "kubernetes charts" repo is Apache 2 licensed.
https://github.com/everpeace/helm-charts (this repository) is MIT licensed

There's two options to resolve this.
Option 1
Random person copies this code to kubernetes charts repository... so what has to happen?
Both licenses are compatible with each other.  If somebody who isn't @everpeace wants to copy all of the code into the kubernetes charts repository, they can.  They just need to include the original MIT license published by @everpeace in this repository to be covered.
Option 2
@everpeace copies this code to kubernetes charts repository... so what has to happen?
Not much, really.  Since you're the original author (assuming no contributors) you can simply copy your code in and re-license it under Apache 2.  You just need to open a pull request.
I do not know what hoops you have to jump through to get it merged since I haven't contributed to kubernetes before.
None of this blocks @everpeace from doing whatever they want with their code (copying to kubernetes charts repository or not).  Hopefully, that makes more sense.

I wouldn't worry about contributors... their contributions are small.


Allow Subdomain identification and Path identification in the same application

It would be great if tenant can be identified by Subdomain or Path in the same application.
So if we create a tenant with subdomain, he will be identified by subdomain.
If it's possible, tenants could choose between path or subdomain.

I said mention this in the pinned issue, not open a new issue. Please move the suggestion to that thread. Thanks!

Any news about this feature?

There isn't anything preventing you from using multiple identification approaches in a single application. You just need the routes to be separately defined.


Constant inlining & evaluate math ops of constants

Inline constant expressions (declared with the const keyword) instead of emitting a variable if possible.

Fixes #58


[FEATURE] Added fibonacci code snippet

Description

Adds a snippet for a "fibonacci" method that generates the fibonacci sequence in a list up to a given nth term
What does your PR belong to?

[ ] Website
[x] Snippets
[ ] General / Things regarding the repository (like CI Integration)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (non-breaking improvement of a snippet)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have checked that the changes are working properly
[x] I have checked that there isn't any PR doing the same
[x] I have read the CONTRIBUTING document.

Missing tests in the test directory. Also duplicate of #38


React native latest master

Description
large merge with master
list of things that need to be tested and verified that correct merge decision was taken
notice: react-native-card-flip doesnt compile, you can temporarily edit its file in node_modules to simply remove the props usage.

[x] add ability to compilte react-native-card-flip or fork module
[ ] Animations/send img_0.svg - pt master, px RN , which unit to use?
[ ] Verify animations jsons and index.js files - verify merge decision
[ ] Logo animation component - verify merge decision
[ ] Spinner checkmark
[x] Inputrounded
[ ] InputText
[ ] ModalTopImage - need to be adapted to SVG component? it uses Image source= with SVG.
[ ] avatar.js - make sure avatar displays correctly in dashboard, my profile/edit profile, in feed card
[ ] circle.js missing box shadow for react native
[ ] Receive summary - biggooddollar line height, 24 or 36?
[ ] Jumping people animation - the margin by isMobileOnly does it look the same on native?
[ ] Feedlistitem.js animation
[ ] Circlebuttonwrapper - verify it shows label/text correctly
[ ] Avatar - verify edit/set works,
[ ]  profile.js (check styles overflow scroll?)
[ ] Profiledatatable - phone input errors style
[ ] Signininfo
[ ] Nameform
[ ] Splash.js
[ ] Shareobject (url, title,message)
[ ] Hasconnectionchanged - verify new logic from master works
[x] Cardflip - solve compilation issue, verify animation works (in claim button).
[ ] base.js - do we need view wrapper? Width height styles 100%?
[ ] SendLinkSummary - left in conflicted state, need to consider native Contacts related changes
[ ] ScanQRButton - need to verify conflict resolution, does it look as expected?

Bugs found

[ ] Phoneinput in edit profile doesnt look good
[ ] My profile - avatar image should have white border
[ ] Edit Profile - avatar image is not centered in white border
[ ] Edit profile - on changing user name throws error "TypeError: Cannot read property 'value' of undefined"

After i applied my fixes to this branch, compiling issue appeared again as proptypes has been reinstalled as nested dependency in the nested node_modules.
I've tried to resolve it through package-lock.json but without any success.
Then, after some investigations i've found the reason: by adding node_modules/react-native-card-flip we're adding all package directory  (including nested node_modules). So, Babel tries to compile prop-types as the source, and throws exception because it's commonJS module without any ES6 exports.
So, I've just excluded node_modules/react-native-card-flip/node_modules from Babel and now the issue with compiling is resolved completely.
Checking how animation works i've met with another issue - there's a missed up variable in the CardFlip's code. I'm not sure why it worked on web before (probably because animation is shown after countdown finished and somehow countdown delay sets rotateOrientation = x before first render), but if to render card flip immediately we'll got  "bYRotation is not defined" exception
I've found this issue on card flip's guthub and the corresponding patch  i've added then to the patch-package
Now card flip animation compiles fine and working.

@sirpy it's seems that card-flip is abandoned by its's author
the critical but very obvious  issue found - any actions won't performed since last commit. Two PRs with solution also won't be merged.
So i think one more issue with this component - and we'll have to find another flipping animation solution or fork it or just rewrite it (taking only those features that we needs in our app)

At the moment I have the next issues:
forceLogLevel: window.location.search not found.
backgroundImage is not a valid property.
Problems with Net info on hasConnectionChange file, NetInfo shouldn't import from react-native, it should import from '@react-native-community/netinfo' and the use changing.

At the moment I see the next issues:
forceLogLevel: window.location.search not found.
backgroundImage is not a valid property.
Problems with Net info on hasConnectionChange file, NetInfo shouldn't import from react-native, it should import from '@react-native-community/netinfo' and the use changing.

@sirpy  Prefer you that I send commit fixing it or I keep testing it? At the moment I still working on the next error: “Can’t autodetect provider for ropsten/” to know that generates it.

yes please fix
ropsten? that's a blockchain network we dont use, it should not happen
unless you env settings are incorrect. NETWORK=fuse when in development env.

@sirpy we're not able to run this branch on our local, it's throwing an error for randombytes (lib used by crypto). we can't reproduce the error in the main react-native branch so there's something in these changes that is conflicting and causing the issue.
we're still investigating so far no luck,  the people that had the same issue weren't able to find a solution and had to move out of the library. did you or any of the other guys faced this issue before?

@mauriguz
first this need to be fixed
SendLinkSummary - left in conflicted state, need to consider native Contacts related changes
then @MathiasMoralesDixtra mentioned NetInfo and createAppContainer
it seems react-native was upgraded to 0.61.5 while in web we use 0.61.3 also react-navigation was upgraded on native to a different version then on web
I cant start it locally at all because of the unresolved conflict

@MathiasMoralesDixtra
you can't just remove code. (FIXME RN)
create a .web .native files
verify the SendSummaryLink resolved correctly with @yaroslav-fedyshyn-nordwhale

Regarding 'Animations/send img_0.svg - pt master, px RN , which unit to use?':
We should use 'px' - but all of these SVGs is empty and even aren't used at all, except RocketShip animation.
So I deleted useless SVG images - and tested if everything works well.

I did refactor for ModalTopImage.js file to render SVG files as React Components and other files (png) to be rendered within Image Component.

I'll look the following issues for the next week:
Avatar - verify edit/set works,
Shareobject (url, title,message)
Hasconnectionchanged - verify new logic from master works

@sirpy @serdiukov-o-nordwhale Jumping people animation - the margin by isMobileOnly does it look the same on native? JumpingPeople isn't being used as far as I can see, instead we're using FlyingPeople. Do we wanna keep it or should we remove it?

@sirpy can you please provide clarification on  profile.js (check styles overflow scroll?) please? I'm not sure what I should check as it isn't a screen with lots of items to overflow

@sirpy @serdiukov-o-nordwhale Jumping people animation - the margin by isMobileOnly does it look the same on native? JumpingPeople isn't being used as far as I can see, instead we're using FlyingPeople. Do we wanna keep it or should we remove it?

@yaroslav-fedyshyn-nordwhale can you clarify which animations we use and remove unwanted ones?

@sirpy can you please provide clarification on profile.js (check styles overflow scroll?) please? I'm not sure what I should check as it isn't a screen with lots of items to overflow

All things in this PR are related to the merging. look at profile.js changes in this PR.

about react-native-image-crop-picker issue - the simplest way to fix is to comment the following lines in
node_modules/react-native-image-crop-picker/ios/QBImagePicker/QBImagePicker/QBAssetsViewController.m
if (@available(iOS 13.0, *)) {
     labelColor = [UIColor labelColor];
}

labelColor = [UIColor labelColor];

seems like we just need to upgrade to latest version
https://github.com/ivpusic/react-native-image-crop-picker/issues/1154


header util.h changed to be accepted by c++ compiler

It's hard to embed ucode in a c++ object, because it's headers are not accepted by a c++ compiler. There are two problems,
ucode/include/ucode/types.h:126:14: error: ISO C++ forbids flexible array member 'name' [-Wpedantic]
         char name[];

and
ucode/include/ucode/util.h:58:48: error: invalid conversion from 'void*' to 'char**' [-fpermissive]
       vec->entries = xrealloc((vec)->entries, sizeof((vec)->entries[0]) * ((vec)->count + UC_VECTOR_CHUNK_SIZE));

The first one can be solved by a wrapper header file using #pragma system_header:
#ifdef __cplusplus
extern "C"{
#endif
#pragma GCC system_header
#include <ucode/types.h>
#ifdef __cplusplus
} // extern "C"
#endif

the second one is solved by this PR

Would the following expression work as well?
vec->entries = (typeof(vec->entries))xrealloc((vec)->entries, sizeof((vec)->entries[0]) * ((vec)->count + UC_VECTOR_CHUNK_SIZE));

No.
ucode/include/ucode/types.h: In function 'void uc_search_path_add(uc_search_path_t*, char*)':
ucode/include/ucode/types.h:254:9: error: 'typeof' was not declared in this scope
  254 |         uc_vector_push(search_path, xstrdup(path));

typeof() is not a c++ operator. It's not a c operator either, but a GCC extension. Unfortunately not a g++ extension,

Could you tell me which toolchain you're using? I utilized typeof() in other parts of the code since it is supported by clang and gcc. I tried a naive C++ usage test by copy-pasting examples/execute-string.c and building it as C++ source:
#include <stdio.h>

extern "C" {
#include <ucode/compiler.h>
#include <ucode/lib.h>
#include <ucode/vm.h>
}

#define MULTILINE_STRING(...) #__VA_ARGS__

static const char *program_code = MULTILINE_STRING(
	{%
		function add(a, b) {
			c = a + b;

			return c;
		}

		result = add(x, y);

		printf('%d + %d is %d\n', x, y, result);

		return result;
	%}
);

static uc_parse_config_t config = {
	.lstrip_blocks = true,
	.trim_blocks = true,
	.strict_declarations = false
};

int main(int argc, char **argv)
{
	int exit_code = 0;

	/* create a source buffer containing the program code */
	uc_source_t *src = uc_source_new_buffer("my program", strdup(program_code), strlen(program_code));

	/* compile source buffer into function */
	char *syntax_error = NULL;
	uc_program_t *program = uc_compile(&config, src, &syntax_error);

	/* release source buffer */
	uc_source_put(src);

	/* check if compilation failed */
	if (!program) {
		fprintf(stderr, "Failed to compile program: %s\n", syntax_error);

		return 1;
	}

	/* initialize default module search path */
	uc_search_path_init(&config.module_search_path);

	/* initialize VM context */
	uc_vm_t vm = { 0 };
	uc_vm_init(&vm, &config);

	/* load standard library into global VM scope */
	uc_stdlib_load(uc_vm_scope_get(&vm));

	/* add global variables x and y to VM scope */
	ucv_object_add(uc_vm_scope_get(&vm), "x", ucv_int64_new(123));
	ucv_object_add(uc_vm_scope_get(&vm), "y", ucv_int64_new(456));

	/* execute compiled program function */
	uc_value_t *last_expression_result = NULL;
	int return_code = uc_vm_execute(&vm, program, &last_expression_result);

	/* release program */
	uc_program_put(program);

	/* handle return status */
	switch (return_code) {
	case STATUS_OK: {
		exit_code = 0;

		char *s = ucv_to_string(&vm, last_expression_result);

		printf("Program finished successfully.\n");
		printf("Function return value is %s\n", s);
		free(s);
		break;
	}

	case STATUS_EXIT:
		exit_code = (int)ucv_int64_get(last_expression_result);

		printf("The invoked program called exit().\n");
		printf("Exit code is %d\n", exit_code);
		break;

	case ERROR_COMPILE:
		exit_code = 1;

		printf("A compilation error occurred while running the program\n");
		break;

	case ERROR_RUNTIME:
		exit_code = 2;

		printf("A runtime error occurred while running the program\n");
		break;
	}

	/* free last expression result */
	ucv_put(last_expression_result);

	/* free VM context */
	uc_vm_free(&vm);

	/* free search module path vector */
	uc_search_path_free(&config.module_search_path);

	return exit_code;
}

Building against revision d0e017fd41476e8d0fec335dd19d1123846f7d96 (current head of the improve-vector-macros branch) seemed to work, but I suppose my implicit C++ standard settings are too lenient:
jow@j7:~/devel/ucode.git/examples$ g++ -v
Using built-in specs.
COLLECT_GCC=g++
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/12/lto-wrapper
OFFLOAD_TARGET_NAMES=nvptx-none:amdgcn-amdhsa
OFFLOAD_TARGET_DEFAULT=1
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Debian 12.2.0-14' --with-bugurl=file:///usr/share/doc/gcc-12/README.Bugs --enable-languages=c,ada,c++,go,d,fortran,objc,obj-c++,m2 --prefix=/usr --with-gcc-major-version-only --program-suffix=-12 --program-prefix=x86_64-linux-gnu- --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-vtable-verify --enable-plugin --enable-default-pie --with-system-zlib --enable-libphobos-checking=release --with-target-system-zlib=auto --enable-objc-gc=auto --enable-multiarch --disable-werror --enable-cet --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-offload-targets=nvptx-none=/build/gcc-12-bTRWOB/gcc-12-12.2.0/debian/tmp-nvptx/usr,amdgcn-amdhsa=/build/gcc-12-bTRWOB/gcc-12-12.2.0/debian/tmp-gcn/usr --enable-offload-defaulted --without-cuda-driver --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
Supported LTO compression algorithms: zlib zstd
gcc version 12.2.0 (Debian 12.2.0-14)
jow@j7:~/devel/ucode.git/examples$ g++ -Wall -o execute-string execute-string.cxx -lucode
jow@j7:~/devel/ucode.git/examples$ ./execute-string
123 + 456 is 579
Program finished successfully.
Function return value is 579
jow@j7:~/devel/ucode.git/examples$

I checked, the toolchain on the box I tested it is a bit outdated:
g++ --version
g++ (Ubuntu 11.4.0-2ubuntu1~20.04) 11.4.0

A minimal program compiled without issues:
int main( void )
{
    int a = 0;
    typeof(a) b = a + 1;

    return b;
}

This compiles, and the exit value of the generated program is 1. I checked the compiler arguments in my project, and the problem is '-std=c++17'. Don't know why it's there. I tried some more recent standards, and they all failed, including c++23.
A more modern compiler (12.3.0) on another box behaves the same.

Yeah, --std=c++17 will select the strict ISO standard mode. I guess a --std=gnu++17 would work to select C++ language level ISO/IEC 14882:2017 with extensions, but not sure if this is viable for your project.
Preferably the code should work in standard mode though, will toy around some more with it.

Okay, so it turns out that -std=c++17 indeed makes g++ stop recognizing typeof() however it seems that __typeof__() will work in both -std=c++17 and -std=gnu++17 modes as well as in  the clang++ compiler.

Superseded by #213


Create a document on how to use MSAM to monitor channels created by MediaLive's workflow wizard

MediaLive recently released a Workflow Wizard that helps create a live streaming channel. Resources (MediaLive channel, MediaPackage channel and endpoints, CloudFront distribution) created by the wizard are tagged with keys and values used by MSAM to automatically create diagrams with these resources for monitoring. We need a doc that outlines how this works.

I made a couple of minor edits for clarity, but very well done and easy to follow. Thanks!

Thanks for reviewing, Jim.


linux: do not clutter config directory with non-config

vscode creates files in .config which are not configuration, like:
./.config/Code - OSS/Backups/workspaces.json
./.config/Code - OSS/CachedData/d2e414d9e4239a252d1ab117bd7067f125afd80a/protocol.workspaceFolders-75c6277d5e20e7d71bb20368814ea86c.code
./.config/Code - OSS/CachedData/d2e414d9e4239a252d1ab117bd7067f125afd80a/workspaceFolders-cf8a46b453ac4453c18bf21321987dd4.code
please move in cache, tmp or whereever. but not in config. there is a standard for this on linux, XDG base directory specification: https://specifications.freedesktop.org/basedir-spec/basedir-spec-0.6.html

/duplicate https://github.com/microsoft/vscode/issues/3884


I don't know how to solve it

"I want to remove the Iron Golem from the Shatter enchantment, but I don't know where to modify it. Also, how to write the disabling potion effects for Tipped, and in which file.

the iron golem should already be blacklisted:
https://github.com/SiverDX/additional_enchantments/blob/877a3524efdb5862108e94bfb074a2df5452638a/src/generated/resources/data/additional_enchantments/tags/entity_types/shatter_aoe_blacklist.json#L1-L6
but it's not checked atm, will fix that
tipped blacklist is here:
https://github.com/SiverDX/additional_enchantments/blob/877a3524efdb5862108e94bfb074a2df5452638a/src/generated/resources/data/additional_enchantments/tags/mob_effect/tipped_blacklist.json#L1-L3
To change either you'll have to use a datapack (https://minecraft.wiki/w/Data_pack) to modify the tags (https://minecraft.wiki/w/Tag)

Thanks, but I meant removing the Iron Golem from the blacklist

That is still done through a datapack
There should be enough tutorials around on how to do this since it's a vanilla feature
The corresponding file is /additional_enchantments/tags/entity_types/shatter_aoe_blacklist.json with the content:
{
  "replace": true,
  "values": [
    "minecraft:villager"
  ]
}

(Note that in this case the tag will only contain the entries you specify here)
I don't know if 1.19.2 has this but in 1.20.1 you can explicitly remove things without having to re-specify every other entry
{
  "remove": [
    "minecraft:iron_golem"
  ]
}

I am very sorry, I shouldn't bother you further, but I found that the data pack I wrote is not working, although it should be correct. Here are the details:
Added MODs: Jade, additional_enchantments Game version: 1.20.1 Data pack path: additional_enchantments/data/additional_enchantments/tags/mob_effect/tipped_blacklist Data pack content: { 'replace': true, 'values': [ 'minecraft:slowness' ] }

Seems like there was an additional issue regarding how tags are handled
It's fixed in the new version currently uploading
Tested it with this:
some_datapack.zip

Now the blacklist can be correctly set, and it works as expected. After such a long period of testing, it turned out that the issue was surprisingly due to a tag error.

Now the blacklist can be correctly set, and it works as expected


FR doc: add markdownTooltip / disableToolTip example

This is kind of a follow-up to #40 where it was implemented and API-documented in the README.
As the README has nice examples for (all?) other attributes: please add a doc entry, maybe the following would be a starting point
## Tooltips

```js
"GitHub": {
    "icon": "github",
    "markdownTooltip": "## Description:\n\n- one\n- two",
},
"GitLab": {
    "icon": "gitlab",
    "disableToolTip": true,
},

Idk. I think README has too much stuff as it is. People are looking at it and go "Aww, hell naw":

Maybe if extension had a dedicated docs page, it would fit there.

I agree that it would be reasonable to have a docs folder and move the samples there.
Something like https://github.com/redhat-developer/vscode-xml (linking to the docs page with all the details).


Cache dynamically added rules with yii\web\UrlManager::addRules()

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#13425

#13425 Cache dynamically added rules with yii\web\UrlManager::addRules()

@SilverFire , this is not quite copy-pasting). Maybe create a separate method is not entirely suitable?
in UrlManager::init():
$cacheKey = $this->cacheKey;
$hash = md5(json_encode($this->rules));
if (($data = $this->cache->get($cacheKey)) !== false && isset($data[1]) && $data[1] === $hash) {
    $this->rules = $data[0];
} else {
    $this->rules = $this->buildRules($this->rules);
    $this->cache->set($cacheKey, [$this->rules, $hash]);
}

in Urlmanager::addRules():
$cacheKey = $this->cacheKey . ':' . md5(json_encode($rules));
if (($data = $this->cache->get($cacheKey)) !== false) {
    $rules = $data;
} else {
    $rules = $this->buildRules($rules);
    $this->cache->set($cacheKey, $rules);
}

This will store each combination of rules in cache for one year - this is really long time. I think that using 24-hour duration for cache will be more practical and will not degrade performance that much.

@rob006 , there is no problem in that the cache value will be stored for a long time. whenever you update the URL rules group, will be the new cacheKey:
$cacheKey = $this->cacheKey . '#' . md5(json_encode($rules));

there is no problem in that the cache value will be stored for a long time

This is a problem - if I change rules 100 times, I will get 99 useless entries in cache. For one year they will take place and slow down some operations, for no reason. And this can be even more problematic if rules will be generated dynamically based on database data - this can change hundreds of times per month.

UrlManager can configure with a different Сache object in which $defaultDuration = 86400

@scriptcube that depends on the project.

@SilverFire , Now caching occurs in yii\web\UrlManager::buildRules(). I hope it is more correct option)

This is a problem - if I change rules 100 times, I will get 99 useless entries in cache. For one year they will take place and slow down some operations, for no reason. And this can be even more problematic if rules will be generated dynamically based on database data - this can change hundreds of times per month.

@rob006, sharply noticed :+1:
This points to a fundamental problem with UrlManager in the first place. Why should rules be continiously built and not a the moment of their first use?
It would be more logical to keep configuration & instantiation separate and to do the caching 'once' on arrays instead of on instances, which is more clean & stable.

I've already asked this question:
https://github.com/yiisoft/yii2/issues/13425#issuecomment-274341903
The problem is that we do not have a unified way to use configs from yii2-extensions in core, except bootstrapping or global event handling.

After a year of thinking (😉) decided that in case when there are 100 addRules() calls, getting 99 useless cache items should not harm against 99 rules builds.

@rob006 I see your thumbs down and I've seen your point above.

if rules will be generated dynamically based on database data - this can change hundreds of times per month

For me this case does not look real. Do you have a good idea why should anybody ever do this when we have route params?

For one year they will take place and slow down some operations, for no reason

There is a common step in release: drop caches to prevent data structure BC issues and clear garbage, ofc.

For me this case does not look real. Do you have a good idea why should anybody ever do this when we have route params?

Actually, I've done something similar in one app - rules are created dynamically based on the list of services in multisite CMS, as a combination of websites slugs and supported languages. New site = a new set of rules. New rules modify old ones, so with solution from this PR I would keep this old useless cache for one year.

There is a common step in release: drop caches to prevent data structure BC issues and clear garbage, ofc.

This has nothing to do with the release - I can have a working app for 2 years with no new release, but with lots of rules generated dynamically. You can't rely on releases to clear trash from app.
On the other hand, I see no reason why not set cache duration to 24 hours. Building rule once per 24-hours should not give any noticeable performance impact and will avoid storing outdated cache for a long time.

In most cases URL rules are mainly static and do not change often, so a long value for caching these is okay. Setting an arbitrary value (24hours) as the default does not really make sense imo, a good value depends on how much they really change, so this has to be fine-tuned on a per application basis.
Imo we can keep using the $defaultDuration setting of the cache and improve the documentation about it. Added a note in ef8f617.


linuxPackages*.xmm7360-pci: unbreak for kernels > 5.14

As of v5.15, the put_tty_driver alias has been dropped in favor of
directly calling tty_driver_kref_put (9f90a4ddef4e). Apply this change
in the xmm7360 driver in order to allow building on recent kernels.
See https://github.com/xmm7360/xmm7360-pci/pull/152

Motivation for this change
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Hold on, this doesn't seem to compile right:
make -C /nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/build M=/build/source modules
make[1]: Entering directory '/nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/build'
  CC [M]  /build/source/xmm7360.o
/build/source/xmm7360.c:1322:16: error: initialization of 'unsigned int (*)(struct tty_struct *)' from incompatible pointer type 'int (*)(struct tty_struct *)' [8;;https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wincompatible-pointer-types-Werror=incompatible-pointer-types8;;]
 1322 |  .write_room = xmm7360_tty_write_room,
      |                ^~~~~~~~~~~~~~~~~~~~~~
/build/source/xmm7360.c:1322:16: note: (near initialization for 'xmm7360_tty_ops.write_room')
cc1: some warnings being treated as errors
make[3]: *** [/nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/source/scripts/Makefile.build:271: /build/source/xmm7360.o] Error 1
make[2]: *** [/nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/source/Makefile:1858: /build/source] Error 2
make[1]: *** [/nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/source/Makefile:220: __sub-make] Error 2
make[1]: Leaving directory '/nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/build'
make: *** [Makefile:9: default] Error 2
error: builder for '/nix/store/fsxn9hy9jjgc70qz7vr0468wcwps3pb2-xmm7360-pci-unstable-2021-07-19.drv' failed with exit code 2;
       last 10 log lines:
       > /build/source/xmm7360.c:1322:16: error: initialization of 'unsigned int (*)(struct tty_struct *)' from incompatible pointer type 'int (*)(struct tty_struct *)' [8;;https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wincompatible-pointer-types-Werror=incompatible-pointer-types8;;]
       >  1322 |  .write_room = xmm7360_tty_write_room,
       >       |                ^~~~~~~~~~~~~~~~~~~~~~
       > /build/source/xmm7360.c:1322:16: note: (near initialization for 'xmm7360_tty_ops.write_room')
       > cc1: some warnings being treated as errors
       > make[3]: *** [/nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/source/scripts/Makefile.build:271: /build/source/xmm7360.o] Error 1
       > make[2]: *** [/nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/source/Makefile:1858: /build/source] Error 2
       > make[1]: *** [/nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/source/Makefile:220: __sub-make] Error 2
       > make[1]: Leaving directory '/nix/store/q8bsakfb9cg8mhdg2d5a10wiag4dbysi-linux-5.14.21-dev/lib/modules/5.14.21/build'
       > make: *** [Makefile:9: default] Error 2
       For full logs, run 'nix log /nix/store/fsxn9hy9jjgc70qz7vr0468wcwps3pb2-xmm7360-pci-unstable-2021-07-19.drv'.


Add the AvoidSemicolonsAsLineTerminators rule to warn about lines ending with a semicolon.  Covered with tests. The rule is disabled by default as it's more of a styling preference. https://github.com/PowerShell/PSScriptAnalyzer/issues/824
PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Change is not breaking
[X] Make sure all .cs, .ps1 and .psm1 files have the correct copyright header
[X] Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

@michaeltlombardi Please review. And we need a docs issue opened for this.

Thank you all.
Could you please advise what should I better do with the PSScriptAnalyzer-CI build failure, got a test failure for WS2022:
Test Directed Graph.Runspaces should be disposed.
Running analyzer 100 times should only create a limited number of runspaces
Expected the actual value to be less than or equal to 14, because Number of Runspaces should be bound (size of runspace pool cache is 10), but got 15.
I could be wrong, but it looks like a flakiness.


don't use std::vector in RefCollection

std::vector uses 3 words with any elements, and we could easily do with 2 (pointers, and two 16 bit sizes).
The grow policy is probably also over aggressive given our memory constraints.

This is for https://github.com/Microsoft/pxt-microbit-core - but no one looks at issue list there

Fixed by Microsoft/pxt-microbit#326


Initial start of removing unwrap from a Result type

💻 Examples

🚨 Test instructions

✔ PR Todo

[ ] Included links to related issues/PRs

I understand what you mean now. Maybe in functions that return a Result I
can use the ? operator to propagate the error. I don't wish to push changes
since I have made a lot and am still reviewing them. But for the
above functions is this sufficient?
let tag = CString::new("maplibre");
let tag = match tag {
Ok(_) => tag,
Err(e) => Err(e),
};
let message = CString::new("maplibre WOORKING");
let message = match message {
Ok(_) => message,
Err(e) => Err(e),
};
On Sun, May 15, 2022 at 6:15 PM Max Ammann @.***> wrote:

@.**** requested changes on this pull request.
In android/src/lib.rs
https://github.com/maplibre/maplibre-rs/pull/97#discussion_r873184316:

let tag = match CString::new("maplibre");

let tag = match tag {

   Ok(tagg) => tagg,

   Err(e) => (e),

};
let message = CString::new("maplibre WOORKING");
let message = match message {

   Ok(mess) => mess,

   Err(e) => (e),

};

I'm quite sure this won't compile because there is an unneccassary match
in the first line.
Also the return types of the match arms do not match.
This use if .unwrap also happens in a dummy-example used to test JNI for
android. This code will eventually be removed soonish.
The general idea was to replace uses of unwrap by propagating the error
type.
—
Reply to this email directly, view it on GitHub
https://github.com/maplibre/maplibre-rs/pull/97#pullrequestreview-973186752,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD64W4INFBGTHYIREJQXYMTVKEII3ANCNFSM5VXQUMBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I don't wish to push changes since I have made a lot and am still reviewing them. But for the

Just go ahead and push the changes, so we can discuss them early on.

But for the above functions is this sufficient?

There is no benefit of avoiding .unwrap() code there because its just a test function I used to prototype JNI support.

let tag = match tag { Ok(_) => tag, Err(e) => Err(e), };

That won't work because the types of the match arms are still different.

Maybe in functions that return a Result I can use the ? operator to propagate the error.

Yes, that's the general idea. We probably should also discuss whether we want to have some helper crate like thiserror. Never checked the pros/cons.

Hello Max,
I have been trying to work out the unwrap features however, I think my
skill set and understanding of Rust are not sufficient for me to continue
on the project. I am sorry for wasting your time. I think I should polish
my Rust concepts before attempting to help again. I thought since it was a
good first issue it would be easier but I admit I am out of my depth. Again
my sincere apologies.
Best regards,
Kevin Mwongera
On Wed, May 18, 2022 at 12:03 PM Kevin Kirimi @.***>
wrote:

Okay understood, will push them as soon as am done going over them once
more.
On Tue, May 17, 2022 at 11:23 AM Max Ammann @.***>
wrote:

I don't wish to push changes since I have made a lot and am still
reviewing them. But for the
Just go ahead and push the changes, so we can discuss them early on.
But for the above functions is this sufficient?
There is no benefit of avoiding .unwrap() code there because its just a
test function I used to prototype JNI support.
let tag = match tag { Ok(_) => tag, Err(e) => Err(e), };
That won't work because the types of the match arms are still different.
Maybe in functions that return a Result I can use the ? operator to
propagate the error.
Yes, that's the general idea. We probably should also discuss whether we
want to have some helper crate like thiserror. Never checked the
pros/cons.
—
Reply to this email directly, view it on GitHub
https://github.com/maplibre/maplibre-rs/pull/97#issuecomment-1128568465,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD64W4NLMF75VJ4N75H3E73VKNJQ7ANCNFSM5VXQUMBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I have been trying to work out the unwrap features however, I think my skill set and understanding of Rust are not sufficient for me to continue on the project. I am sorry for wasting your time. I think I should polish my Rust concepts before attempting to help again. I thought since it was a good first issue it would be easier but I admit I am out of my depth. Again my sincere apologies.

No worries! In fact I have to admit that the issues was also badly written by me. It was very unspecific and too open to serve as a good first issue. Also code has been changing to fast the last weeks that it was difficult.
We'd be happy to receive a future contribution from you!

Okay understood, will push them as soon as am done going over them once
more.
On Tue, May 17, 2022 at 11:23 AM Max Ammann @.***>
wrote:

I don't wish to push changes since I have made a lot and am still
reviewing them. But for the
Just go ahead and push the changes, so we can discuss them early on.
But for the above functions is this sufficient?
There is no benefit of avoiding .unwrap() code there because its just a
test function I used to prototype JNI support.
let tag = match tag { Ok(_) => tag, Err(e) => Err(e), };
That won't work because the types of the match arms are still different.
Maybe in functions that return a Result I can use the ? operator to
propagate the error.
Yes, that's the general idea. We probably should also discuss whether we
want to have some helper crate like thiserror. Never checked the
pros/cons.
—
Reply to this email directly, view it on GitHub
https://github.com/maplibre/maplibre-rs/pull/97#issuecomment-1128568465,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD64W4NLMF75VJ4N75H3E73VKNJQ7ANCNFSM5VXQUMBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Lint Gradle Groovy scripts

See this comment.
┆Issue is synchronized with this Jira Task by Unito

Or convert them to Kotlin, which might make them easier to maintain in Android Studio. 🤔
CC @ben-xD, as this overlaps with the conversation you and I had earlier today. 😁

I'm not sure if switching to Kotlin will provide improved linting for build.gradle files, but the built in Android Studio linter is much more helpful when using the Kotlin DSL for Gradle.

Closing this issue as will end up superseded by https://github.com/ably/ably-asset-tracking-android/issues/350.


Operator Rewrite: Operator base class

What is the expected enhancement?
A base class for several types of Operator classes: MatrixOperator, PauliOperator (system for grouping for commutativity wip), FermionicOperator, etc.
Continuation of https://github.com/Qiskit/qiskit-aqua/issues/311

From my understanding, most of the aspects of the Operator class that make it a mutable object come from changing its representation.
It sounds like this enhancement would place the ability to change representations within Operator as opposed to the child classes MatrixOperator, PauliOperator and FermionicOperator, etc.
I think this would open up the possibility to make each instance of the child classes immutable (also see https://github.com/Qiskit/qiskit-aqua/issues/579).
My premise of the mutability of the Operator class may be wrong though, I haven't used all of the functionality of the Operator class yet.

Operator refactoring has been done by #593


Repetitive code in each module

Is there any chance that this transform will be smart enough to keep it's code in separate module, like "transform-runtime" does?
Of course, it's not necessary to joyfully use it – just to be, you know, and ideal tool =)

It's a tough one to solve since Babel doesn't currently provide a way to do this easily. It is something I'm keeping in mind for the future though. For now, I'd hope that by the time the file is gzipped, the difference in size wouldn't be super noticeable.

We're planning to move this into Babel as babel-plugin-transform-decorators in Babel 7.x, which will mean this code will end up in babel-runtime like the other code Babel stuff, so closing this.

Just noticed the duplication of the code myself. Any word on babel-plugin-transform-decorators in Babel 7.x?

Looks like it might actually be a part of Babel 7 now?
https://www.npmjs.com/package/babel-plugin-transform-decorators
https://github.com/babel/babel/tree/master/packages/babel-plugin-transform-decorators
I guess we could switch to that if we're willing to move to Babel 7?

Hi,
@loganfsmyth
I noticed that babel-plugin-proposal-decorators migrated the helpers to babel-helpers in https://github.com/babel/babel/pull/7017
Any chance this can be done to the legacy also?


Drop card v2

iterating fast, lets play more on the social factor

after observations that price and status are not so relevant, here is updated card

updated in figma handoff drop page: https://www.figma.com/file/JXNtryaNYhY0XJNArzvPQx/Handoff?type=design&node-id=394-6126&mode=design&t=UwOjVaVtynz3fLJf-11
Minting live

the minted/supply indicates its status, if minted = supply, its obvious its minted out
if its 0/supply, it could be scheduled or live drop that has no mints yet

for that there is the right part (collected by part)

for state when its live but no one minted yet

when someone mints, it starts showing bubbles (collected by)

bubbles will have disabled hover for now, and let's see how it works

Scheduled

shows minted/supply and also the date
when the time to drop < 24 hours, it starts to show the hours - like in drop page etc.

Ended

minted = supply
collected by bubbles

Other

one major change is that the preview img should have fixed aspect ratio of 16:9
the bottom part scales accordingly

https://github.com/kodadot/nft-gallery/assets/90852205/08646c57-0533-4cce-998c-6b2fb2f39474
let me know if anything
thanks!

will be done for release? 👀

@iMac7 in drop page its random addresses that minted


Average vs Specific setup option - incorrect comment

It looks like the Comment for Average vs Specific setup option had been swapped.

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c5a40235-e14f-f7c0-1517-d1559a986f81
Version Independent ID: 2a3c68fb-a4e8-5c07-fd84-14c571cdbfff
Content: Setup Best Practices - Costing Method - Business Central
Content Source: business-central/setup-best-practices-costing-method.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: sgroespe

Thank you for your feedback! This will be fixed in the next doc update.

Now live.


[python] GH actions are not testing all supported Python versions

Our setup.cfg says Python versions 3.7+ are supported.  The CI currently only tests 3.8 and 3.9.  We need to add to the matrix 3.7, 3.10 and 3.11

Looks like we'll need to wait on Python 3.11. From #599:
At https://github.com/single-cell-data/TileDB-SOMA/actions/runs/3661926647/jobs/6190605155 we see numba is not yet supported in Python 3.11. cc @thetorpedodog @bkmartinjr @Shelnutt2 .
Collecting numba>=0.41.0
  Downloading numba-0.56.4.tar.gz (2.4 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.4/2.4 MB 146.6 MB/s eta 0:00:00
  Preparing metadata (setup.py): started
  Running command python setup.py egg_info
  Traceback (most recent call last):
    File "<string>", line 2, in <module>
    File "<pip-setuptools-caller>", line 34, in <module>
    File "/tmp/pip-install-0tpo_aju/numba_b1ec8a6b557b450ea19a316e9668617a/setup.py", line 51, in <module>
      _guard_py_ver()
    File "/tmp/pip-install-0tpo_aju/numba_b1ec8a6b557b450ea19a316e9668617a/setup.py", line 48, in _guard_py_ver
      raise RuntimeError(msg.format(cur_py, min_py, max_py))
  RuntimeError: Cannot install on Python version 3.11.0; only versions >=3.7,<3.11 are supported.

See also https://github.com/numba/numba/issues/8304
Looks like they're hoping for a release candidate by end of 2022: https://github.com/numba/numba/issues/8304#issuecomment-1336218610

#620 (merged) implements Python 3.7 support
Python 3.11 is still blocked on the aforementioned numba-dependency issue


Utility function to build a URL from route.id + params

Describe the problem
I'm setting up redirects for a route directory with multiple params (/blog/[slug], /blog/[slug]/[part], /blog/[slug]/[...rest], etc.) and want to build the new URL by replacing just one of them ([slug]).
I ended up doing this naive regex replacement on route.id which doesn't account for all the possible variations of route params in SvelteKit (matchers, optionals, templates etc.):
// routes/blog/+layout.ts

const redirects = {
    'old-slug': 'new-slug'
}

export const load: LayoutLoad = ({ route, params }) => {
    if (redirects[params.slug]) {
        const newParams = {
            ...params,
            slug: redirects[params.slug]
        }

        throw redirect(
            308,
            route.id.replace(/\[(?:\.\.\.)?([^\]]*)\]/g, (_, param) => newParams[param])
        )
    }
}

Describe the proposed solution
Expose SvelteKit's internal resolve_entry function as buildRouteURL in the $app/navigation module, which parses a given route id and builds a URL string with the given params.
Now implementing my LayoutLoad function is as easy as this:
export const load: LayoutLoad = ({ route, params }) => {
    if (redirects[params.slug]) {
        throw redirect(
            308,
            buildRouteURL(route.id, {
                ...params,
                slug: redirects[params.slug]
            })
        )
    }
}

Alternatives considered
Related: #8289 Let ParamMatcher functions return a string to redirect
Importance
nice to have

🎉

Damn that was fast, I was just looking for this type of utility 🙏


[Question] Range slider for timeseries?

I noticed, there is a range slider in the timeseries interactivity example in the main README.md page. How do I enable that? I don't see it in the codepen example?

The scroller was available prior v2.0 but was removed due to memory leaks and other bugs. I decided to remove it until I come up with a better version of it.
Maybe it will come again in the future.


Cannot add to queue while spotify-true-random is working

Hello,
Thanks for creating this.
When I run this for a playlist, I get a full "Next up" queue, which is presumiably true random as intended. Anything currently in the "Next in queue" remains until it is all played but I'm not able to add anything to the next in queue as I normally would when shuffling in the usual spotify way.
Is this intended behaviour?

Hmmm, I'm able to reproduce that issue as well. That's pretty odd... I don't know that I've noticed that before so I wonder if something changed with Spotify here.
I'll investigate a bit


Add a quiet output flag (-q)

I think this could be useful for people who use this cli app on scripts, like me. Thank you for this app too, btw.

+1
A reason: When you use PHP and call cli with shell_execute() I must add  2>&1 1> /dev/null after command, otherwise an warning message will be logged in php on each call.
WARNING: [pool example.com] child 1721038 said into stderr: "vkEnumeratePhysicalDevices failed -3"
A quiet flag could simplify it :)


Can this stream to multiple clients?

I'm using another WS streaming library that only streams to one client at a time. Is it the same with this library, or can you emit a stream to all connected clients?
Thanks,
Andrew

I am using websocket-stream with several connected clients to stream video: no problem. :)

@desaintmartin Do you have any example code you could show for this? i'm trying to stream audio from the web bug can't figure out how to pipe the audio data into the web socket


Add requestsAlwaysUse prop to the UserLocation

Description

Added requestsAlwaysUse prop to the UserLocation component that allows choosing if we want to listen for the user location in the background or not.
It allows us to not ask for background location permission when we don't need it and still use the UserLocation component.
Checklist

[x] I have tested this on a device/simulator for each compatible OS
[x] I updated the documentation with running yarn generate in the root folder
[x] I updated the typings files - if js interface was changed (index.d.ts)
[ ] I added/ updated a sample - if a new feature was implemented (/example)

Screenshot OR Video
None for now.

Thanks much, sounds interesting. Can you actually receive events in background?

I'd actually prefer naming it requestAlwayUse rather than listensToLocationInBackground as I assume receiving location updates is much more difficult, due to the background nature of the task.

There are several additional steps to follow in order to listen to the location in the background.
So yes you're right, requestAlwaysUse seems more appropriate. I will continue working on this next week and keep you posted!

@pablogdcr can you look into test failures

@mfazekas fixed. I also tested on Android and made some adjustments

Thanks @pablogdcr for your work! Hoping for this adjustment to come to the next release version!


Update tapir-cats, tapir-http4s-server, ... to 0.19.0-M14

Updates

com.softwaremill.sttp.tapir:tapir-cats
com.softwaremill.sttp.tapir:tapir-http4s-server
com.softwaremill.sttp.tapir:tapir-json-circe

from 0.19.0-M12 to 0.19.0-M14.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

labels: library-update, early-semver-pre-release, semver-spec-pre-release

Superseded by #83.


Add $TupleReduce destructor

// @flow

declare var union: $TupleReduce<
  [1, 2, 3],
  <Acc, Value, Key>(Acc, Value, Key) => Acc | Value
>;

declare var accumulate: $TupleReduce<
  [1, 2, 3],
  <Acc, Value, Key>(Acc, Value, Key) => [Acc, Value],
  []
>;

union; // 1 | 2 | 3
accumulate; // [[[[], 1], 2], 3]

We are unlikely to pursue this, since we don't want more special builtin types. For highly dynamic functions, a lot of overloads might be better.


Update the -AppFileTypes parametre example usage

@Huios,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Thanks Tanaka! I already changed the example yesterday in response to that thread. Sorry for getting ahead of you on that. Closing this PR as a dup.

Thanks Norm!


ReferenceAssemblyAnnotator causes ReSharper to lose BCL references and highlight everything in red

This seems to plague net35, net40, and net45, but not net472.

I've submitted a bug report through the ReSharper feedback tool.

Tracking at https://resharper-support.jetbrains.com/hc/en-us/requests/2418887, seems to be a private ticket.

I don't think I have seen this with net45. Could it be a matter of cleaning caches?


[Suggestion] Lists and Libraries: include new action buttons

The Lists and Libraries module provides links to SP pages for list settings and permissions which cover many of the operations one would want to do with a list.
However there are some actions not available on the UI but there's an API in JSOM / REST for it, for example hide/unhide a list. It would be great to have some of these actions inline on the tool itself.

That sounds like a very useful option, I'm think it might be a good idea to add some sort of contextual menu with all the option available so far and the ones to come.
http://dev.office.com/fabric#/components/contextualmenu
I'm thinking about adding the delete option as well.

@jquintozamora is thinking about it. No pressure :)


Failing to parse valid JS

I feel like the plugin cannot parse this file. At version 7.0.0 the error is more verbose though:
"use strict";
exports.isArrayLike = (function (x) { return x && typeof x.length === 'number'; });
//# sourceMappingURL=isArrayLike.js.map

I get
[!] Error: Unexpected token
node_modules\rxjs\util\isArrayLike.js (2:78)
1: "use strict";
2: var isArrayLike_1 = function (x) { return x && typeof x.length === 'number'; });
                                                                                ^
3: //# sourceMappingURL=isArrayLike.js.map

If I remove the parenthesis around the function declaration it works.

Hey folks (this is a canned reply, but we mean it!). Thanks to everyone who participated in this issue. We're getting ready to move this plugin to a new home at https://github.com/rollup/plugins, and we have to do some spring cleaning of the issues to make that happen. We're going to close this one, but it doesn't mean that it's not still valid. We've got some time yet before the move while we resolve pending Pull Requests, so if this issue is still relevant, please @ me and I'll make sure it gets transferred to the new repo. :beer:


Sprig templates

The sprig library contains a lot of useful functions that would make the current templates directive more powerful — in particular default, coalesce and some crypto functions like sha256sum.
Anything in particular we should be mindful of if we decided to add these?
I could submit a PR if this sounds like a good idea 😊

Implemented in Caddy 2.


Implicit clears

Instead of explicitly clearing attachments: can we specify the clear values and clear operation in renderpass?
https://github.com/gopro/gopro-lib-node.gl/blob/4e1456401fe96fb12c058e30bcc0d1f398854ec9/libnodegl/gctx.h#L60

Explicit clear functions have been removed since 3259577b13dc8ecc8d90174b133d0d1b8c5d0c3f.


IL-826 Revert "Pass compatibility flag to v2"

Reverts docker/compose-switch#21
Reverting as this introduces different behaviour in Desktop when using docker compose and docker-compose.
Fixes #24

@ndeloof FYI ☝

❤


Distributed events

Feature Request
Currently we have Background Workers where one Job is only given to one Worker.
It would be great if we can fire events where one event is given to all workers.
I need to inform every server running when a specific entity is created.
Describe the solution you'd like
A system that works just like the workers but the event is published to every worker instead of only one.
Describe alternatives you've considered
There aren't really any alternatives in the rust ecosystem.

Sounds like some kind of pubsub requirement (subscriptions via Redis for example) doesn’t it?

Sounds like some kind of pubsub requirement (subscriptions via Redis for example) doesn’t it?

Yes kinda but that both can be used simultaneously and controlled via code.
Just like Messages and Events in other frameworks like spring or symfony.


Extension 'tensorflow/tensorflow.bzl' has errors

System information

**OS Platform and Distribution : Linux Ubuntu 16.04
Python version: 2.7
Bazel version (if compiling from source): 0.16.0
CUDA/cuDNN version: 8.0 / 6.0
GPU model and memory: nvidia GTX 860m 16gb

Describe the problem
When I try to download the Tensorflow-gpu from the source with Bazel, it says:
ERROR: /home/kevin/tensorflow/tensorflow/tensorflow.bzl:582:12: name 'set' is not defined
ERROR: error loading package '': Extension 'tensorflow/tensorflow.bzl' has errors
Source code / logs
$ git checkout r0.12
$ ./configure
~/tensorflow ~/tensorflow
Please specify the location of python. [Default is /usr/bin/python]:
Do you wish to build TensorFlow with Google Cloud Platform support? [y/N]
No Google Cloud Platform support will be enabled for TensorFlow
Do you wish to build TensorFlow with Hadoop File System support? [y/N] No Hadoop File System support will be enabled for TensorFlow
Found possible Python library paths:
/usr/local/lib/python2.7/dist-packages
/usr/lib/python2.7/dist-packages
Please input the desired Python library path to use.  Default is [/usr/local/lib/python2.7/dist-packages]
Using python library path: /usr/local/lib/python2.7/dist-packages
Do you wish to build TensorFlow with OpenCL support? [y/N]
No OpenCL support will be enabled for TensorFlow
Do you wish to build TensorFlow with GPU support? [y/N] y
GPU support will be enabled for TensorFlow
Please specify which gcc should be used by nvcc as the host compiler. [Default is /usr/bin/gcc]:
Please specify the Cuda SDK version you want to use, e.g. 7.0. [Leave empty to use system default]:
Please specify the location where CUDA  toolkit is installed. Refer to README.md for more details. [Default is /usr/local/cuda]:
Please specify the Cudnn version you want to use. [Leave empty to use system default]:
Please specify the location where cuDNN  library is installed. Refer to README.md for more details. [Default is /usr/local/cuda]:
Please specify a list of comma-separated Cuda compute capabilities you want to build with.
You can find the compute capability of your device at: https://developer.nvidia.com/cuda-gpus.
Please note that each additional compute capability significantly increases your build time and binary size.
[Default is: "3.5,5.2"]:
Starting local Bazel server and connecting to it...
INFO: Starting clean (this may take a while). Consider using --async if the clean takes more than several minutes.
Starting local Bazel server and connecting to it...
ERROR: /home/kevin/tensorflow/tensorflow/tensorflow.bzl:582:12: name 'set' is not defined
ERROR: error loading package '': Extension 'tensorflow/tensorflow.bzl' has errors
Building: no action

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
TensorFlow installed from
TensorFlow version
Exact command to reproduce
Mobile device

Can confirm, I have the same bug
patrick@patrick-X470-AORUS-ULTRA-GAMING:~/git/tensorflow$ ./configure
~/git/tensorflow ~/git/tensorflow
Please specify the location of python. [Default is /usr/bin/python]: /usr/bin/python3.6
Do you wish to build TensorFlow with Google Cloud Platform support? [y/N]
No Google Cloud Platform support will be enabled for TensorFlow
Do you wish to build TensorFlow with Hadoop File System support? [y/N]
No Hadoop File System support will be enabled for TensorFlow
Found possible Python library paths:
  /usr/lib/python3/dist-packages
  /usr/local/lib/python3.6/dist-packages
Please input the desired Python library path to use.  Default is [/usr/lib/python3/dist-packages]

Using python library path: /usr/lib/python3/dist-packages
Do you wish to build TensorFlow with OpenCL support? [y/N]
No OpenCL support will be enabled for TensorFlow
Do you wish to build TensorFlow with GPU support? [y/N]
No GPU support will be enabled for TensorFlow
Configuration finished
INFO: Starting clean (this may take a while). Consider using --async if the clean takes more than several minutes.
Starting local Bazel server and connecting to it...
ERROR: /home/patrick/git/tensorflow/tensorflow/tensorflow.bzl:582:12: name 'set' is not defined
ERROR: error loading package '': Extension 'tensorflow/tensorflow.bzl' has errors
Building: no action

It has been 22 days with no activity and the awaiting response label was assigned. Is this still an issue?

This is still an issue by using Bazel.
but i can use this to install v0.12
pip install https://storage.googleapis.com/tensorflow/linux/gpu/tensorflow_gpu-0.12.0-cp34-cp34m-linux_x86_64.whl

0.12 is at this point 2 years old. You have to have the bazel version that is compatible with that version.
@martinwicke should we start deleting these old branches to avoid user confusion?

I'm a big fan of historical things being available. I suppose we can remove the branch and leave the tag. But I don't think it'll save much confusion.
We are not supporting old releases though.


Increase test coverage in data_structures/binary_tree/red_black_tree.py

Current coverage is 74% and though the file is almost 1k LOC it should be possible to get test coverage to >90%.

Hi, "trim trailing whitespace" fails when I run pre-commit on it~
Besides, after I modified pre-commit-config.yaml as "https://stackoverflow.com/questions/59730765/is-it-possible-to-run-mypy-pre-commit-without-making-it-fail" states, "mypy" pass but with lots of error, not sure it's normal or not.
Coverage is 93% now.
I did take a look at the code, but didn't review it very carefully. Since the main target is to enhance coverage, I guess as long as the coverage is consistent as what you claim, that's enough.


Fix default null/empty value

removeLayer attempts to filter:
this.layersToAdd = this.layersToAdd.filter((l) => l.name !== layer.name);

Null value creates undefined error.

@disarticulate thank you for the PR, there is a specific issue related to this ? Is it better to filter it like this or to track the issue that generate the undefined ?

The data property of it starts as an empty array, and I don't see any other uses for it except as an array.


Fix packet exception logging, packet sending and nullref

Fixes packets not being sent when a map is selected.
TODO: Figure out what causes the nullref on MpRequirements
TODO: Implement additional fixes from MpCore.Quest

@RedBrumbler can you merge this so we can get @roydejong to merge https://github.com/roydejong/MultiplayerCore/pull/1 so we no longer have this PR chain?


loading_screen example panics after switching twice

Bevy version
Main (4147422)
What you did
Run the example
cargo run --release --example loading_screen

And press "1", "2" and then press "1", to load the first scene back.
What went wrong
The program panics with
     Running `target\release\examples\loading_screen.exe`
2024-09-01T16:06:13.368716Z  INFO bevy_diagnostic::system_information_diagnostics_plugin::internal: SystemInfo { os: "Windows 11 Home China", kernel: "22631", cpu: "AMD Ryzen 7 7840HS w/ Radeon 780M Graphics", core_count: "8", memory: "15.2 GiB" }
2024-09-01T16:06:13.751715Z  INFO bevy_render::renderer: AdapterInfo { name: "AMD Radeon 780M Graphics", vendor: 4098, device: 5567, device_type: IntegratedGpu, driver: "AMD proprietary driver", driver_info: "24.8.1 (LLPC)", backend: Vulkan }
2024-09-01T16:06:14.298432Z  INFO bevy_winit::system: Creating new window "App" (0v1#4294967296)
thread 'Compute Task Pool (6)' panicked at crates\bevy_render\src\mesh\allocator.rs:674:21:
internal error: entered unreachable code: Slab not found
stack backtrace:
   0: std::panicking::begin_panic_handler
             at /rustc/3f5fd8dd41153bc5fdca9427e9e05be2c767ba23/library\std\src\panicking.rs:652
   1: core::panicking::panic_fmt
             at /rustc/3f5fd8dd41153bc5fdca9427e9e05be2c767ba23/library\core\src\panicking.rs:72
   2: bevy_render::mesh::allocator::MeshAllocator::mesh_index_slice
   3: bevy_render::mesh::allocator::allocate_and_free_meshes
   4: <bevy_ecs::system::function_system::FunctionSystem<Marker,F> as bevy_ecs::system::system::System>::initialize
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
Encountered a panic in system `bevy_render::mesh::allocator::allocate_and_free_meshes`!
 *  Terminal will be reused by tasks, press any key to close it.

Duplicate of #14845 :)


Make Shadow.inline() caching account for language version and compilation environment.

Closes #3419.

Thanks


Fix KinesisIO throwing NullPointerException when progress check made before starting reader

fixes #23868
This has the KinesisReader return unknown if it cannot determine the backlog size due to the start method not previously being called. The progress check can happen at arbitrary times, so it must be accounted for here.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[x] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Part of the stack trace of the issue is:
...
Caused by: java.lang.NullPointerException
        at org.apache.beam.sdk.io.kinesis.KinesisReader.getSplitBacklogBytes(KinesisReader.java:172)
        at org.apache.beam.sdk.io.Read$UnboundedSourceAsSDFWrapperFn$UnboundedSourceAsSDFRestrictionTracker.getProgress(Read.java:983)
        at org.apache.beam.sdk.transforms.reflect.ByteBuddyDoFnInvokerFactory$DefaultGetSize.invokeGetSize(ByteBuddyDoFnInvokerFactory.java:431)
...

From talking with Cham, it sounds like getProgress can be called at arbitrary times, including before reading starts, and in that case, we should just report no progress.

makes sense to me


BKPeriphial Protocol

I am following the example of setting up a peripheral, and I am stuck when adding the BKPeriphial to my view controller class. How do I implement the BKPeriphialController Protocol?

Hi kswain1.
There is nothing in the framework called BKPeripheralController Protocol. What exactly are you referring to?

I had to add the BKPeripheralDelegate functions below to use a peripheral object in my class
// MARK: BKPeripheralDelegate
internal func peripheral(peripheral: BKPeripheral, remoteCentralDidConnect remoteCentral: BKRemoteCentral) {
    Logger.log("Remote central did connect: \(remoteCentral)")
    sendDataBarButtonItem.enabled = true
}

internal func peripheral(peripheral: BKPeripheral, remoteCentralDidDisconnect remoteCentral: BKRemoteCentral) {
    Logger.log("Remote central did disconnect: \(remoteCentral)")
    sendDataBarButtonItem.enabled = false
}


Setting authentication token for request

What is the correct way to set a bearer token when using this package? I am running into an issue where the request is always failing due to being unauthenticated but works when I test it on a server client (.net grpc client) using the same token. Do note that in both cases, I have a proxy sitting in front of the server for handling the grpc-web request.
intercept<Q extends GrpcMessage, S extends GrpcMessage>(request: GrpcRequest<Q, S>, next: GrpcHandler): Observable<GrpcEvent<S>> {
        return this.authService.getTokenSilently$().pipe(
            mergeMap(token => {
                const newMetadata: Metadata = {
                    ...request.requestMetadata,
                    ['Authorization']: `Bearer ${token}`
                };

                const tokenReq = {
                    ...request,
                    requestMetadata: newMetadata
                };
                return next.handle(tokenReq);
            })
        );
    }

I am getting an access token and setting is via initializing a new metadata object within a grpc interceptor. I can confirm the interceptor is registered and triggered for each request. I'm not quite sure if this is correct because no matter what I try, the access token is being refused when being set this way. Is there anything I am missing?

Hi @vrao1020
Do you use ConsoleLogger? If yes, when you expand any request in the browser console you will be able to see the metadata in the second row (first row is request data, third row is response data). Can you find the token being set in the metadata?
If no, then your interceptor does not attach the metadata properly.
If yes, then something should be configured on the proxy side. Which grpc-web proxy / implementatiin do you use?

I am using .Net core with the Grpc.AspNetCore.Web package. It's a .Net core only version of what Envoy proxy does.
I do have console logger enabled and when I have authorization enabled on my server, the call fails with bearer token authorization failing (when called from my angular app).

Note that the request data is present but everything after is not (the server sends a 401 unauthorized via an OPTIONS response which is odd).
As for the token being set in the metadata, it is because when I removed the authorization requirement from my server, below is the flow when I send a request from my angular app.

I used the same token in a console .net core app that uses grpc-web to communicate with the server when authorization is active and receive no errors so I know the token is valid in that regard.

OPTIONS response... CORS is in place. What is weird is that the options request fails while it should not have any metadata yet.
We use bearer token exactly in the way you do with the Authorization header in Golang (native client from improbable-eng) and via envoy proxy as well. The only thing we did to make it work was adding to the envoy config the Authorization to Access-Control-Allow-Headers.
To me the issue seems to be somewhere in .NET and CORS configuration. Ideally, you should omit CORS whenever possible, it is hard to maintain and always bites the back. It is fairly easy to set a proxy in angular.json for local development and some sort of load balancer to separate the frontend and grpc by some paths and then everything is on the same domain. Probably I should make an example of this in the docs in the future.

I don't believe CORS is an issue as it works fine when I send the request without trying to authenticate it. My CORS policy also allows any header from specific domains so it can't be that the auth header is being blocked (I inspected request server side without authentication and see the auth header come through). If I have my request setup correctly as you stated, it must be something else on the server side being unable to parse the metadata for the request correctly - I'll open a ticket for that with the library I'm using on the server side to see if I can get anywhere with this. Thanks for your help!

You are welcome, hope it gets resolved! Please post the solution if you find it, maybe it helps somebody in the future :)

I guess the issue is resolved, feel free to reopen in case not


Extension switches back to Jedi when reloading after setting Pylance as the default language server

Environment data

VS Code version: 1.49.3
Extension version (available under the Extensions sidebar): v2020.9.114305
OS and version: Windows 10 2004
Python version (& distribution if applicable, e.g. Anaconda): 3.8.6
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions:
Relevant/affected Python-related VS Code extensions and their versions: Pylance (v2020.10.0)
Value of the python.languageServer setting: Jedi

Expected behaviour
Changed the language server to Pylance, and it should stay Pylance.
Actual behaviour
Pylance extension suggests me to change to it's language server.

Click 'Yes, and reload'
settings.json

Python extension changes the language server back to Jedi and asks me to reload

I go check settings.json and to my surprise:

Steps to reproduce:

Install Python extension and Pylance extension.

Create Python 'project' with these settings in .vscode

settings.json:
{
    "python.pythonPath": "C:\\Users\\tiago\\AppData\\Local\\Programs\\Python\\Python38\\python.exe"
}

launch.json
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Python: Current File",
            "type": "python",
            "request": "launch",
            "program": "${file}",
            "console": "externalTerminal"
        }
    ]
}

Python code:

print('hello')

Can you check your "Python Language Server" log window to see if Pylance is starting? And could you provide the logs from the Python output window?

Do you happen to have any workspace settings that could be overriding it?
As far as I know, the fallback to Jedi should happen transparently if needed without changing the setting, but I could be wrong.

Can you check your "Python Language Server" log window to see if Pylance is starting? And could you provide the logs from the Python output window?

Do you happen to have any workspace settings that could be overriding it?
As far as I know, the fallback to Jedi should happen transparently if needed without changing the setting, but I could be wrong.

@jakebailey
The output for "Python Language Server"
[Info  - 20:54:26] Pylance language server 2020.10.0 (pyright d1f950ac) starting
[Info  - 20:54:26] Server root directory: c:\Users\summo\.vscode\extensions\ms-python.vscode-pylance-2020.10.0\dist
[Error - 20:54:26] stubPath c:\Users\summo\Desktop\testbench.py\typings is not a valid directory.
[Error - 20:54:27] stubPath c:\Users\summo\Desktop\testbench.py\typings is not a valid directory.

(this was after reloading from changing to Pylance, meanwhile the message to reload the window to switch between language servers appeared)
Output for "Python"
User belongs to experiment group 'ShowPlayIcon - start'
User belongs to experiment group 'DebugAdapterFactory - experiment'
User belongs to experiment group 'PtvsdWheels37 - experiment'
User belongs to experiment group 'UseTerminalToGetActivatedEnvVars - control'
User belongs to experiment group 'LocalZMQKernel - experiment'
User belongs to experiment group 'CollectLSRequestTiming - control'
User belongs to experiment group 'CollectNodeLSRequestTiming - experiment'
User belongs to experiment group 'EnableIPyWidgets - experiment'
User belongs to experiment group 'RunByLine - experiment'
User belongs to experiment group 'CustomEditorSupport - control'
User belongs to experiment group 'pythonaacf'
> conda --version
> pyenv root
> python3.7 c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python3.6 c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python3 c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python2 c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> py -3.7 c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> py -3.6 c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> py -3 c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> py -2 c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> ~\AppData\Local\Programs\Python\Python38\python.exe c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> conda info --json
Starting Pylance language server.
Python interpreter path: ~\AppData\Local\Programs\Python\Python38\python.exe
Error 2020-10-08 20:54:25: Failed to get interpreter information for 'C:\Users\summo\AppData\Local\Microsoft\WindowsApps\python3.exe' Error: Command failed: "C:\\Users\summo\AppData\Local\Microsoft\WindowsApps\python3.exe" "c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py" "c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\interpreterInfo.py"
Python was not found but can be installed from the Microsoft Store: https://go.microsoft.com/fwlink?linkID=2082640
    at ChildProcess.exithandler (child_process.js:304:12)
    at ChildProcess.emit (events.js:223:5)
    at maybeClose (internal/child_process.js:1021:16)
    at Socket.<anonymous> (internal/child_process.js:430:11)
    at Socket.emit (events.js:223:5)
    at Pipe.<anonymous> (net.js:664:12) {
  killed: false,
  code: 9009,
  signal: null,
  cmd: '"C:\\\\Users\\summo\\AppData\\Local\\Microsoft\\WindowsApps\\python3.exe" "c:\\Users\\summo\\.vscode\\extensions\\ms-python.python-2020.10.289937622-dev\\pythonFiles\\pyvsc-run-isolated.py" "c:\\Users\\summo\\.vscode\\extensions\\ms-python.python-2020.10.289937622-dev\\pythonFiles\\interpreterInfo.py"'
}
Error 2020-10-08 20:54:25: Failed to get interpreter information for 'C:\Users\summo\AppData\Local\Microsoft\WindowsApps\python.exe' Error: Command failed: "C:\\Users\summo\AppData\Local\Microsoft\WindowsApps\python.exe" "c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py" "c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\interpreterInfo.py"
Python was not found but can be installed from the Microsoft Store: https://go.microsoft.com/fwlink?linkID=2082640
    at ChildProcess.exithandler (child_process.js:304:12)
    at ChildProcess.emit (events.js:223:5)
    at maybeClose (internal/child_process.js:1021:16)
    at Socket.<anonymous> (internal/child_process.js:430:11)
    at Socket.emit (events.js:223:5)
    at Pipe.<anonymous> (net.js:664:12) {
  killed: false,
  code: 9009,
  signal: null,
  cmd: '"C:\\\\Users\\summo\\AppData\\Local\\Microsoft\\WindowsApps\\python.exe" "c:\\Users\\summo\\.vscode\\extensions\\ms-python.python-2020.10.289937622-dev\\pythonFiles\\pyvsc-run-isolated.py" "c:\\Users\\summo\\.vscode\\extensions\\ms-python.python-2020.10.289937622-dev\\pythonFiles\\interpreterInfo.py"'
}
Error 2020-10-08 20:54:25: Detection of Python Interpreter for Command python3 and args  failed as file  does not exist
Error 2020-10-08 20:54:26: Detection of Python Interpreter for Command py and args -2 failed as file Python 2 not found!

 -3.8-64 does not exist
Error 2020-10-08 20:54:26: Detection of Python Interpreter for Command py and args -3.6 failed as file Python 3.6 not found!

 -3.8-64 does not exist
Error 2020-10-08 20:54:26: Detection of Python Interpreter for Command py and args -3.7 failed as file Python 3.7 not found!

 -3.8-64 does not exist
> ~\AppData\Local\Programs\Python\Python38\python.exe c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import jupyter"
> ~\AppData\Local\Programs\Python\Python38\python.exe c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import jupyter"
> ~\AppData\Local\Programs\Python\Python38\python.exe c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import notebook"
> ~\AppData\Local\Programs\Python\Python38\python.exe c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py -c "import notebook"
> conda --version
> ~\AppData\Local\Programs\Python\Python38\python.exe c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py flake8 --format=%(row)d,%(col)d,%(code).1s,%(code)s:%(text)s c:\Users\summo\Desktop\testbench.py\test.py
cwd: c:\Users\summo\Desktop\testbench.py
> ~\AppData\Local\Programs\Python\Python38\python.exe c:\Users\summo\.vscode\extensions\ms-python.python-2020.10.289937622-dev\pythonFiles\pyvsc-run-isolated.py flake8 --format=%(row)d,%(col)d,%(code).1s,%(code)s:%(text)s c:\Users\summo\Desktop\testbench.py\test.py
cwd: c:\Users\summo\Desktop\testbench.py
##########Linting Output - flake8##########
21,1,W,W293:blank line contains whitespace
22,5,F,F841:local variable 'wait' is assigned to but never used
27,1,W,W391:blank line at end of file

The workspace settings are the ones you can see above.

settings.json:
{
    "python.pythonPath": "C:\\Users\\tiago\\AppData\\Local\\Programs\\Python\\Python38\\python.exe"
}

Is python.languageServer set to a specific value in your User settings?

Is python.languageServer set to a specific value in your User settings?

The issue is all about that, isn't it?
I may change it to Pylance, but as soon as I open a Python file, the extension changes it back to Jedi.

The key thing is knowing where the setting is set, because it can be set in many places. The global user config, in a workspace settings.json, in a code-workspace file. We've had bugs in the past where we were only considering one and not the other, setting it in the wrong place, etc, so eliminating any additional locations helps greatly.

But, judging by this behavior, this could be the settings migrator. It's likely that the languageServer setting has been around long enough that we could consider dropping it and not worrying about many many month old configs.

The setting is set in the User settings. The ones you set in one of these options! And yes, in the User tab if using the UI option.

Can you check that file to see if there's a setting python.jediEnabled, even commented out?

That was it! python.jediEnabled was set to true and causing the extension to set it as the Language Server.
Thank you!

Thanks for confirming. For reference, it's this way because before the languageServer setting was the only setting, jediEnabled needed to be set explicitly to false to get any LS to work. Now jedi is "just another LS setting", but for compatibility we had to roll people forward; explicitly setting jedi to true meant that we were forcing it to be the LS.
There's a TODO to remove this, as at this point I'm not expecting there to be many machines configured this way intentionally anymore.


remove_lcr: what does this setting do?

Hi,
I've just come across a setting that I wasn't familiar with: remove_lcr.
It appears here in the documentation: https://bcbio-nextgen.readthedocs.io/en/latest/contents/somatic_variants.html?highlight=remove_lcr#review-parameters-in-the-yaml-file
and nowhere else, as far as I can see.
Sounds like it has something to do with removing low complexity (repetitive) regions from the variant calling analysis. But how exactly does it work? Where are the low complexity regions defined/described?
Is this setting only usable for somatic variant calling? Or for germline variant calling as well?

Hi @amizeranschi !
Yes, it removes low complexity regions from variant calling, both germline and somatic.
The following code: https://github.com/bcbio/bcbio-nextgen/blob/fe239ee2a09adae18bdddc1af615bf02e3df22c4/bcbio/pipeline/shared.py#L143
is looking for a resource in genome resources, for example
genomes/Hsapiens/hg38/seq/hg38-resources.yaml :
  lcr: ../coverage/problem_regions/repeats/LCR.bed.gz

i.e. the bed file is in
genomes/Hsapiens/hg38/coverage/problem_regions/repeats/LCR.bed.gz
The file is installed by https://github.com/chapmanb/cloudbiolinux/blob/b913e075deb6678eaf27a086d3fcdb58450e1024/ggd-recipes/hg38/coverage.yaml
from here: https://github.com/lh3/varcmp
There is an article explaining the motivation:
https://www.ncbi.nlm.nih.gov/pubmed/24974202
S.

Hi @naumenko-sa,
Thanks a lot for the information and the citation, this is really helpful!

Should this be added to the docs?

@hackdna IMO this could be helpful.

@naumenko-sa
Do you know of any other potential sources for LCR BED files, for non-human organisms?
I'm interested in bovine and I noticed that UCSC has some datasets for tandem repeats, microsatellites and tandem repeats. Would it make sense to convert these to BED and concatenate them?
http://hgdownload.soe.ucsc.edu/goldenPath/bosTau6/database/microsat.txt.gz
http://hgdownload.soe.ucsc.edu/goldenPath/bosTau6/database/nestedRepeats.txt.gz
http://hgdownload.soe.ucsc.edu/goldenPath/bosTau6/database/simpleRepeat.txt.gz

We documented remove_lcr: https://bcbio-nextgen.readthedocs.io/en/latest/contents/configuration.html#coverage-information
Thanks for facilitating this improvement!
The original Li2014 publication used mdust to generate 2% LCR regions.
I think mdust is agnostic regarding the reference (you may run it on bosTau), but I can't open the page anymore.
I don't have an idea of how to tackle LCRs in bosTau. If you don't have a validation dataset like NA12878 for H.sapiens, you can't perform validations similar to Li2014. I'd rather take the annotation approach - to annotate variants with repetitive regions instead of not calling them at all. Then you can filter variants downstream.

Alright, thanks for following up on this. Can you recommend any tool or approach for annotating variants with repetitive regions read from a BED file?

FTR, remove_lcr is an alias of a specific option of exclude_regions.

@amizeranschi
https://github.com/brentp/vcfanno
https://bcbio-nextgen.readthedocs.io/en/latest/contents/germline_variants.html?highlight=vcfanno#variant-annotation

@naumenko-sa thanks a lot for mentioning it.
I guess the easiest thing to do is to set up a BED file with LCRs and subtract this from variant_regions.bed. It would be helpful if bcbio_setup_genome.py could automatically set up the BED file within the genome resources (and same for a VCF file with known variants for BQSR). I've opened up a separate feature request for this.


Various version updates & have an option to apply torch.compile

Making this as draft for now, because we need to confirm backward compatability across all workloads.

I will update the code in #405


where is 'v5-PilotNet_v2-009-0.67221.h5' file?

I run inference.ipynbls, an error occurred:
%ls $model_path
/Users/Rand/europilot/model: No such file or directory

Hey Alpslee,
As of now pretrained models are not uploaded. But I just added a dataset of 17G, which you can download here.

Appreciate it!
2017年10月9日 下午6:18，"Gyuri Im"<EMAIL_ADDRESS>
Hey Alpslee,
As of now pretrained models are not uploaded. But I just added a dataset
of 17G, which you can download here
https://drive.google.com/file/d/0B42sVbnSOCJ4bnZhWF80b0xUY28/view?usp=sharing
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/marshq/europilot/issues/7#issuecomment-335117356, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANm29j6npBHemG--T1m5OjRQxM-LUFdHks5sqfLpgaJpZM4PyBb6
.


Videojs v5 upgrade for tests

videojs-contrib-ads master is incompatible with videojs v4

We've still got a lot of testing to do to make sure we work OK with video.js 5, so we're not quite ready to accept this yet.

Ok,
Additionally skippable state change event not firing and "video continues after ad was skipped" failed.
player.ima.addEventListener(google.ima.AdEvent.Type.SKIPPABLE_STATE_CHANGED, function() {
      var adManger = this;
      adManager.skip();
    })

LGTM, thanks! If you're still seeing the skip problem can you open a separate issue? Thanks!


Change auto-suffixing behavior of data-test attrs on rad-drawer children

The child components of rad-drawer take the data-test values passed into them and ALWAYS auto append -target and -content to the supplied strings.
This was intended to make it easy to pass a single root data-test attr into the component and have it auto-propagate down through the children so you didn't have to do custom attrs on them.
However, due to how this feature was implemented, custom data-test attrs on the children always receive the suffixes.
This just needs to be reimplemented to happen within the parent template instead of as JS logic in the children, so that custom data-tests take precedence.

Fixed in v1.5.1


Migration to microservice architecture

MERGE ASSIGNMENT 3 FIRST
Refactoring changes for microservices:

Enforce the names of the services to be in plural form throughout, for consistency. e.g. "users_function" instead of "user_function"

The term "functions", like in "user_functions.py", has been changed to "controller", to more accurately describe its purpose

Ports used:

API gateway: 8000
users api: 8001
questions api: 8002
users db: 5001
questions db: 5002

Session authorization will be checked at API Gateway level (instead of e.g. question_functions). Hence, microservice APIs no longer takes session_id as args.

This is because session authorization will require a call to the /sessions api, so just do the check at API gateway instead of throwing calls back and forth

Should be merged onto assignment-3 branch instead because it uses assignment-3 as its base.


vendor: update swagger-ui to v3.24.3

This PR also improve the related make task to only package needed file.
The update of swagger-ui fix various security bug :

https://github.com/swagger-api/swagger-ui/releases/tag/v3.18.0
https://github.com/swagger-api/swagger-ui/releases/tag/v3.23.11
https://github.com/swagger-api/swagger-ui/releases/tag/v3.24.1

I just unvendored and updated swagger-ui in https://github.com/go-gitea/gitea/pull/9714 so I fear this PR might be obsolete.

@silverwind sorry I haven't see your PR.

@silverwind I feel after reviewing your PR that we should be better to keep this outside js build chain and stick to provided releases by swagger-ui.
I may miss one advantage to use the custom webpack build. What do you think building the swagger js add compared to using the released version ?

I primarily want the dependency managed by npm for easy updating.
We could use the pre-built module swagger-ui-dist and just copy it to public, but with that, we'd have to inline the initialization in the go template again which I'm not a fan of because that code will never be linted or polyfilled for our supported browsers.


All the Tags are not visible under Tags/API categories section in the devportal UI

Description
When the Devportal UI is loaded and Tags/API categories section is selected, can only see the 25 Tags.
Steps to Reproduce
Steps to reproduce

Start APIM 4.0.0 GA or latest pack
Create API and add more than 25 tags.
Publish the API.
Go to the Devportal -> and select  Tags/API categories section.
You will see only 25 tags.

Affected Component
APIM
Version
4.0.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Hi @Tharanidk ,
Can we get an update on this issue as the best case estimation is already exceeded and reaching the Most Likely Estimate?
Regards,
Supun


Aggregate PySpark DataFrame before Pandas conversion

Change
Aggregate the PySpark DataFrame before converting it to a Pandas DataFrame. This helps reduce the likelihood of an OOM error when the PySpark DataFrame is too large to fit into memory.
Test
TO ADD

@xinrong-meng besides tests, please also try if the example.ipynb works after the changes.


As a developer, I want a Code of Conduct so that my interaction with the community is good

Expected behavior
Actual behavior
Steps to reproduce the behavior

Progress update: Emails are still being made by Bradley, event sponsorship form still be made by Teri, GitHub pages site still not live, still up in the air when this will be able to be live.


Run make to update template-coq/gen-src/cRelationClasses.mli.orig

Is this change expected?  Is this file supposed to be checked in?  What's going on here?

It looks like the extraction changed somehow? The .orig file doesn't need to be checked in, you're right.

Can we let the patch command fail if the patch is not applicable? If not, it might make sense to keep the orig file and fail if it is not correct anymore
I hunted a problem for about an hour the other day where the issue was that extraction changed and the patch silently failed, resulting in a compilation error that I couldn't make sense of

Does changing
https://github.com/MetaCoq/metacoq/blob/ca64d7236c22a438496d0360385f62216ef0a2ca/template-coq/update_plugin.sh#L22-L23
with
-    patch -N -p0 < extraction.patch
-    patch -N -p0 < specFloat.patch
+    patch -N -p0 < extraction.patch || exit $?
+    patch -N -p0 < specFloat.patch || exit $?

work?

Is that different to || exit? But the idea seems to be the right one to enforce a failure on a failing patch

Is this still relevant?

The .orig file still exists, but it seems to no longer be the case that make changes it, so, idk, maybe not?