Add files via upload

@minhngoc25a Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

You can provide us with any of the following as the proof:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@minhngoc25a You might miss my previous comment. Please react.

@minhngoc25a I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Tremulus RPG

Changes / Comments
Added a sheet for the Tremulus RPG, simply named "Tremulus".  This is another one that is powered by the Apocalypse game engine.  The game's publisher's site is here:
https://realityblurs.com/shop/catalog/setting/tremulus/
This sheet was not added to resolve a bug request or fulfill another user request.
There are no changes to any existing sheets, so no player data needed tested.
To the best of my knowledge/ability this conforms to the character sheets standards.  This is my first attempt at making one, so my apologies in advance for anything I overlooked.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Thanks for adding the sheet for a great game. You'll make it to allow translations?

My players found a problem with some of the character attributes, so I'll be in the code anyway.  I read over the documents from roll20 on using i18n for translation, so I'll try to work that in as well.  It doesn't seem complicated, just tedious.  I've never done it before, so hopefully I don't mess it up.

Greetings @JBeighel
You code has been merged but "Tremulus.json" will need to be renamed to sheet.json in order for it to appear in the drop down list.
Thanks,
Cassie

Sorry about that name, I can change that easily enough.  I'll set up another pull request to get that updated.
Thanks for setting me straight!


Reimbursement Enum

I got this error, changed the code and now seems to be working
ValueError: 'RI' is not a valid Code
Pr for fix:
https://github.com/csingley/ibflex/pull/66

Closing as PR was merged


POSIX deviation: test arithmetic operands must support decimal only

toybox test 10 -eq 010

returns false and
toybox test 08 -lt 09

gives an error.
POSIX does require operands to be treated as decimal which means it's an undocumented deviation.
The fact that [ arithmetic operators accept only decimal in bash/zsh while [[ ... ]] accepts any arithmetic expression (including octal constants if not disabled globally), is a reason why [ is often recommended over [[ or (( in those shells (in particular [[ $attacker_controlled_variable -lt 10 ]] and (( attacker_controlled_variable < 10 )) are command injection vulnerabilities there).
It's useful to be able to rely on decimal handling when handling 0-padded number such as date/time components as day, month, hour, minute, second are usually 0-padded as in:
if [ "$(date +%m)" -le 6 ]; then
  echo first semester
fi

(toybox date supports date +%-m but that's not standard/portable)
If you decide to keep the POSIX deviation, I would suggest that at least sequences of digits that contain 8 and 9 digits be treated as decimal (both 010 and 08 to be considered as number 8) so we can still do date manipulation without having to strip the leading 0s manually.
POSIX doesn't specify the behaviour for test 0xf -eq 15 or test 1c -eq 1 or test 1000 -eq 1kd so it's fine in that regard to return true instead of an error.

toybox test 10 -eq 010

returns false and
toybox test 08 -lt 09

Still better than GNU behavior...
$ /bin/test 08 -lt 09; echo $?
0
$ toybox test 08 -lt 09; echo $?
test: not integer: 08
2

08 isn't valid octal. So what exactly is GNU test being accurate to?

POSIX does require operands to be treated as decimal which means it's an undocumented deviation.
The fact that [ arithmetic operators accept only decimal in bash/zsh while [[ ... ]] accepts any arithmetic expression (including octal constants if not disabled globally), is a reason why [ is often recommended over [[ or (( in those shells (in particular [[ $attacker_controlled_variable -lt 10 ]] and (( attacker_controlled_variable < 10 )) are command injection vulnerabilities there).

Toybox aliases [ to [[, so they are both functionally the same thing.
Another cool thing you can do by parsing numbers with atolx: test 100M -lt 10G

It's useful to be able to rely on decimal handling when handling 0-padded number such as date/time components as day, month, hour, minute, second are usually 0-padded as in:
if [ "$(date +%m)" -le 6 ]; then
  echo first semester
fi

(toybox date supports date +%-m but that's not standard/portable)

Not portable to what? GNU and busybox have it, FreeBSD also has it and therefore MacOS does too, but admittedly OpenBSD doesn't looking at their strftime manpage. Which means we are not portable to a OpenBSD environment where toybox test is being used, but toybox date isn't.
A quick hack to fix this is to swap out atolx() with regular atol() (Line 82 toys/posix/test.c is where the magic happens). But to keep true POSIX and Bash compatibility there would need to be some way to detect if we are calling under "[[", and only do the octal numbers and such if so.
The patch is on the list, but I have my doubts on Rob applying it due to the loss of functionality.
If you ever are working on a OpenBSD box and trying to interoperate with zero padded month numbers that you can't turn off with OpenBSD strftime, but can use in toybox test. And it's really super security critical (How exactly is a system supposed to be attacked with this minor bug? Then again I could say that about most of the modern CVE's bugs), but also speed critical enough to not pipe the numbers through grep -o "[1-9][0-9]*" to remove the trailing 0's. I'd suggest applying it yourself

Still better than GNU behavior...
$ /bin/test 08 -lt 09; echo $?
0
$ toybox test 08 -lt 09; echo $?
test: not integer: 08
2

08 isn't valid octal. So what exactly is GNU test being accurate to?

That's the whole point of this issue. 08 and 09 are also decimal numbers and must be interpreted as such, not octal.
That's not just GNU test. That's virtually all test implementations in existence including the test builtin of most shells including mksh.

POSIX does require operands to be treated as decimal which means it's an undocumented deviation.
The fact that [ arithmetic operators accept only decimal in bash/zsh while [[ ... ]] accepts any arithmetic expression (including octal constants if not disabled globally), is a reason why [ is often recommended over [[ or (( in those shells (in particular [[ $attacker_controlled_variable -lt 10 ]] and (( attacker_controlled_variable < 10 )) are command injection vulnerabilities there).

Toybox aliases [ to [[, so they are both functionally the same thing.

Note that [[...]] is initially a shell construct from the Korn shell inside which a separate micro-language is recognised to avoid the shortcomings of the [ utility. It's not meant to be implemented as a standalone utility.

Another cool thing you can do by parsing numbers with atolx: test 100M -lt 10G

As I mentioned in my initial report, it's fine to keep that as that's outside POSIX scope (behaviour unspecified).

Not portable to what? GNU and busybox have it, FreeBSD also has it and therefore MacOS does too, but admittedly OpenBSD doesn't looking at their strftime manpage. Which means we are not portable to a OpenBSD environment where toybox test is being used, but toybox date isn't.
[...]

My view is rather that a portable script will use [ "$(date +%m)" -le 6 ] as that's standard and guaranteed to work by POSIX, and using a $(date +%-m) work around should not be necessary and is not even standard.

Note that [[...]] is initially a shell construct from the Korn shell inside which a separate micro-language is recognised to avoid the shortcomings of the [ utility. It's not meant to be implemented as a standalone utility.

We can do both since one is a complete (Apparently, mostly complete) subset of the other.

Another cool thing you can do by parsing numbers with atolx: test 100M -lt 10G

As I mentioned in my initial report, it's fine to keep that as that's outside POSIX scope (behaviour unspecified).

The problem is that there is no easy way from my knowledge to keep one and not the other, I may be wrong though.

My view is rather that a portable script will use [ "$(date +%m)" -le 6 ] as that's standard and guaranteed to work by POSIX, and using a $(date +%-m) work around should not be necessary and is not even standard.

Like I said, you can pipe numbers through grep -o "[1-9][0-9]*" to strip the trailing zeros

FWIW, date +%-m works in neither OpenBSD nor NetBSD.

Note that [[...]] is initially a shell construct from the Korn shell inside which a separate micro-language is recognised to avoid the shortcomings of the [ utility. It's not meant to be implemented as a standalone utility.

We can do both since one is a complete (Apparently, mostly complete) subset of the other.

They have incompatible syntax and feature set in all shells that have both.
[...]
The problem is that there is no easy way from my knowledge to keep one and not the other, I may be wrong though.
It would be enough to do:
if (operand-has-only-digits[-and-whitespace])
  convert-from-decimal;
else
  use-atolx;

I would think (the and-whitespace is not a POSIX requirement but is a nice to have as most test implementations handle leading and trailing whitespace).

My view is rather that a portable script will use [ "$(date +%m)" -le 6 ] as that's standard and guaranteed to work by POSIX, and using a $(date +%-m) work around should not be necessary and is not even standard.

Like I said, you can pipe numbers through grep -o "[1-9][0-9]*" to strip the trailing zeros

grep -o is a non-standard GNUism, and that fails for 00.
You need something like:
hour=$(date +%H)
short_hour=${hour#0}
if [ "$short_hour" -lt 12 ]; then
  echo "$hour is before noon.
fi

Re: your patch
Note that there's no security issue in toybox' test (that I know). I mentioned zsh/bash's security issue with their own [[...]] construct (not [[ utility, they don't have a [[ builtin) which treat operands as arithmetic expressions as a note to stress how we usually rely on [ doing only decimal parsing for its arithmetic operators in those shells.

if (operand-has-only-digits[-and-whitespace])
  convert-from-decimal;
else
  use-atolx;

"if (operand-has-only-digits[-and-whitespace])" is a oversimplification of what that code would
actually look like, you'd need a for loop going over each element and checking it every time
you went over a number. Which adds a lot of code and slows this down by a fair amount.
You could check the first character and if that's a zero don't process the octal, but that
removes hexadecimal support.
This is something that can be done, but it adds complexity, adds executable size,
and slows the code down, and gives using function pointers will warnings because
of mismatches between a "char *" and a "const char *" (sigh).
I may be wrong, and the support for prefixes and hex numbers in test is actually
something less important than I thought. It's not my decision to make
Maybe there should be some way to turn off octal support for atolx. I just found
out that there isn't prefix support in expr, and doing something like expr 100M + 64k + ...
to automatically get the number of bytes would be useful. But adding in octal support
would bring this problem.
atolx is used in commands like head and tail too, I originally thought it was how lib/args.c parsed
numbers too. So this problem isn't just in test, it's a large number of places.
Octal is obsolete almost everywhere except file permissions. I'd suggest removing it from atolx entirely, but doing that has the same problems of detecting a octal number and parsing it as decimal, but not a hexadecimal one
And of course, similar discussion about this problem for dd is on the list:

I noticed a problem with numeric arguments to dd e.g. ibs=999, skip=1234 or
count=0000987
The 0.6.1 implementation assumes that the number is decimal.
0.7.5 assumes that a leading 0 indicates octal, this is different to both the
posix standard and the implementation in standard linux.

Indeed. I had common code to handle the suffixes but it also does the c-standard
base detection. Which is a side effect you don't want. Hmmm...

On Posix Being the be-all, end-all of implementation details:

The dd on ubuntu and as specified in the posix spec
(http://pubs.opengroup.org/onlinepubs/9699919799/) says the numbers are decimal

The posix spec says that the command supports ebcdic to ascii conversions. I've
been taking a somewhat loose interpretation of this one. :)

Also:

("This broke my script" is a different issue from "the spec says
you should do this even though nobody seems to have used it in living memory".)

(This is something I was considering suggesting):

I think that the problem is caused by the use of atolx() as a common numeric
input function for many dissimilar command line utilities.
Maybe an extra param is needed to indicate the expected base, e.g. like the base
param to strtol.

I'm reluctant to add an argument to a library function that's currently used 57
times without it. If the problem is just leading zeroes, a trivial wrapper to
"while (*str=='0') str++;" in this command should fix it?

And Finally:

I agree that octal is obsolete (outside of file permissions), and if posix says
it's decimal then leading zeroes changing that is surprising and should probably
be fixed. I'm less convinced about the x thing (which means not supporting
hexadecimal, which is not obsolete...)

After more consideration, removing octal support in atolx seems like a better solution.
The use of zero padded numbers probably vastly outweighs any modern use of Octal, so
confusing one for another isn't that good of a idea.
Patch 2.0 is on the list

I'm always a little confused about the "do less" reports. I have a pending one in sed on my todo list too.
I can see leading zeroes indicating octal being considered confusing to a modern audience, now that 6 bit systems are obsolete and the only fossil use for it that comes to mind is unix permissions. That said, strtol() detects that when it detects 0x and that's libc plumbing, I didn't write that, just used it.
Haven't decided what to do about it yet, the reporter of the sed one hadn't actually hit a case where it caused a problem, they were just checking compliance corner cases. Did you have a use case here, or were you just doing compliance checking too?

I'm always a little confused about the "do less" reports. I have a pending one in sed on my todo list too.
I can see leading zeroes indicating octal being considered confusing to a modern audience, now that 6 bit systems are obsolete and the only fossil use for it that comes to mind is unix permissions. That said, strtol() detects that when it detects 0x and that's libc plumbing, I didn't write that, just used it.

Yes, strtol() is meant to handle 0123 as octal and that should remain as so. If atolx() is used in places where strtol() is meant to be used like in printf %d 0123, then 0123 handling should not be removed from there as that would break scripts that expect that octal handling.

Haven't decided what to do about it yet, the reporter of the sed one hadn't actually hit a case where it caused a problem, they were just checking compliance corner cases. Did you have a use case here, or were you just doing compliance checking too?

I'm not a (direct) user of toybox myself. I came across it whilst researching for this answer to a question about find -size (where handling 0123 as octal is also a non-conformance but likely less of a problem there as 0-padded decimals are less likely).
Having said that, questions about why (( zero_padded_decimal < 10 )) or [[ zero_padded_decimal -lt 10 ]] don't work properly in bash (one of the Korn-like shells that haven't removed 0123 handling as octal yet in non-sh mode)  are common on usenet or stackexchange (and zero-padded numbers are common in date/time components at least) and the usual answer in that particular case is: use [ instead which POSIX guarantees operands to arithmetic operators to be treated as literal (and is standard as opposed to ((...)), [[...]] so can be used in sh scripts).
In the general case, in Korn-like shells 10#$zero_padded_decimal in arithmetic expressions works but not for negative numbers.

Yes, strtol() is meant to handle 0123 as octal and that should remain as so. If atolx() is used in places where strtol() is meant to be used like in printf %d 0123, then 0123 handling should not be removed from there as that would break scripts that expect that octal handling.

GNU printf doesn't do octal, although busybox does:
$ for i in printf 'busybox printf' 'toybox printf'; do $i "%d\n" 0123; done
123
83
83


vcvt_f32_s32 is not correct in visual studio

I run and debug neon intrinsic by using simde in Visual Studio. When using  simde_vcvt_f32_s32 to convert a  slice of int number ,and it seems like the result not correct, the vector are all 12851, but the result of vcvt_f32_s32 is -nan(ind), 12851.0, 12851.0, ...  Why the first of results is -nan(ind). By the way the specific intrinsic is vcvt_f32_s32(vmovl_s16(vget_low_s16(v_in))). Only the result of this intrinsic is not correct, vcvt_f32_s32(vmovl_s16(vget_high_s16(v_in))) is true.

Hello @xxxxxxLD and thank you for sharing your issue
For which architecture are you compiling for?
Which version of MS Visual Studio are you running?
What compiler options are you using?
https://github.com/simd-everywhere/simde/blob/14311d60539303ca8bad6204dcbc6a29f51b0e09/simde/arm/neon/cvt.h#L1090
I'm also surprised to see the result you are reporting.

I

Hello @xxxxxxLD and thank you for sharing your issue
For which architecture are you compiling for? Which version of MS Visual Studio are you running? What compiler options are you using?
https://github.com/simd-everywhere/simde/blob/14311d60539303ca8bad6204dcbc6a29f51b0e09/simde/arm/neon/cvt.h#L1090
I'm also surprised to see the result you are reporting.

I'm just compile it in Visual Studio 2019, and see the result by debuging the code step by step.Only the first result of vcvt_f32_s32(vmovl_s16(vget_low_s16(v_in))) is not correct.And the vcvt_f32_s32 is explained by HEDLEY_STATIC_CAST(T,expr) (static_cast(expr)) in LINE 854 of hedley.h.


docs(md-elevation): request for feedback on approach

Bug, feature request, or proposal:
proposal
What is the expected behavior?
That .md-elevation-zX classes are documented.
What is the current behavior?
No documentation exists other than within closed PR and issue comments. There are some design documents, but they aren't really designed for developers using the APIs.
What is the use-case or motivation for changing an existing behavior?
To avoid repeated questions and rejected PRs like https://github.com/angular/material2/pull/2212.
Is there anything else we should know?
I would like to help with this documentation, but I would like some feedback on how the team would like this to be done before I submit a PR.
Should there be a separate doc/guide for elevation that covers examples of multiple components?
Should there be examples of using elevation in each component that it applies to?
Is there a list of such components?

It would be great to have a short and simple guide on elevation in the guides/ directory.

agreed would be fabulous! // @EladBezalel (thx @Splaktar!) I like the simplicity of MaterializeCSS docs http://materializecss.com/shadow.html

The mixins should also be documented like @include md-elevation($zValue); (the design documents this is mdElevation($zValue))

Now we should use   @include mat-elevation($zValue);

@Splaktar Do you have a Pr in progress that I could help with or anything like that?

@M-a-c no sorry. I had other priorities jump up and haven't had a chance to get back to this. Thank you for helping out!

I have a problem with hundreds of elevation directives being injected in for EVERY component.  Every components has CSS styles like this:
.mat-elevation-z0 {
  box-shadow: 0px 0px 0px 0px rgba(0, 0, 0, 0.2), 0px 0px 0px 0px rgba(0, 0, 0, 0.14), 0px 0px 0px 0px rgba(0, 0, 0, 0.12); }

.mat-elevation-z1 {
  box-shadow: 0px 2px 1px -1px rgba(0, 0, 0, 0.2), 0px 1px 1px 0px rgba(0, 0, 0, 0.14), 0px 1px 3px 0px rgba(0, 0, 0, 0.12); }

.mat-elevation-z2 {
  box-shadow: 0px 3px 1px -2px rgba(0, 0, 0, 0.2), 0px 2px 2px 0px rgba(0, 0, 0, 0.14), 0px 1px 5px 0px rgba(0, 0, 0, 0.12); }

.mat-elevation-z3 {
  box-shadow: 0px 3px 3px -2px rgba(0, 0, 0, 0.2), 0px 3px 4px 0px rgba(0, 0, 0, 0.14), 0px 1px 8px 0px rgba(0, 0, 0, 0.12); }

.mat-elevation-z4 {
  box-shadow: 0px 2px 4px -1px rgba(0, 0, 0, 0.2), 0px 4px 5px 0px rgba(0, 0, 0, 0.14), 0px 1px 10px 0px rgba(0, 0, 0, 0.12); }

.mat-elevation-z5 {
  box-shadow: 0px 3px 5px -1px rgba(0, 0, 0, 0.2), 0px 5px 8px 0px rgba(0, 0, 0, 0.14), 0px 1px 14px 0px rgba(0, 0, 0, 0.12); }

.mat-elevation-z6 {
  box-shadow: 0px 3px 5px -1px rgba(0, 0, 0, 0.2), 0px 6px 10px 0px rgba(0, 0, 0, 0.14), 0px 1px 18px 0px rgba(0, 0, 0, 0.12); }

.mat-elevation-z7 {
  box-shadow: 0px 4px 5px -2px rgba(0, 0, 0, 0.2), 0px 7px 10px 1px rgba(0, 0, 0, 0.14), 0px 2px 16px 1px rgba(0, 0, 0, 0.12); }

.mat-elevation-z8 {
  box-shadow: 0px 5px 5px -3px rgba(0, 0, 0, 0.2), 0px 8px 10px 1px rgba(0, 0, 0, 0.14), 0px 3px 14px 2px rgba(0, 0, 0, 0.12); }

.mat-elevation-z9 {
  box-shadow: 0px 5px 6px -3px rgba(0, 0, 0, 0.2), 0px 9px 12px 1px rgba(0, 0, 0, 0.14), 0px 3px 16px 2px rgba(0, 0, 0, 0.12); }

.mat-elevation-z10 {
  box-shadow: 0px 6px 6px -3px rgba(0, 0, 0, 0.2), 0px 10px 14px 1px rgba(0, 0, 0, 0.14), 0px 4px 18px 3px rgba(0, 0, 0, 0.12); }

.mat-elevation-z11 {
  box-shadow: 0px 6px 7px -4px rgba(0, 0, 0, 0.2), 0px 11px 15px 1px rgba(0, 0, 0, 0.14), 0px 4px 20px 3px rgba(0, 0, 0, 0.12); }

.mat-elevation-z12 {
  box-shadow: 0px 7px 8px -4px rgba(0, 0, 0, 0.2), 0px 12px 17px 2px rgba(0, 0, 0, 0.14), 0px 5px 22px 4px rgba(0, 0, 0, 0.12); }

.mat-elevation-z13 {
  box-shadow: 0px 7px 8px -4px rgba(0, 0, 0, 0.2), 0px 13px 19px 2px rgba(0, 0, 0, 0.14), 0px 5px 24px 4px rgba(0, 0, 0, 0.12); }

.mat-elevation-z14 {
  box-shadow: 0px 7px 9px -4px rgba(0, 0, 0, 0.2), 0px 14px 21px 2px rgba(0, 0, 0, 0.14), 0px 5px 26px 4px rgba(0, 0, 0, 0.12); }

.mat-elevation-z15 {
  box-shadow: 0px 8px 9px -5px rgba(0, 0, 0, 0.2), 0px 15px 22px 2px rgba(0, 0, 0, 0.14), 0px 6px 28px 5px rgba(0, 0, 0, 0.12); }

.mat-elevation-z16 {
  box-shadow: 0px 8px 10px -5px rgba(0, 0, 0, 0.2), 0px 16px 24px 2px rgba(0, 0, 0, 0.14), 0px 6px 30px 5px rgba(0, 0, 0, 0.12); }

How do I turn this off?

@megamindbrian You are probably indirectly including mat-core in each component

@willshowell Thanks for the response!  I only included mat-core 1 time in my "layout.scss".  I've added "layout.scss" to angular-cli.json:
{
    "apps": [
        {
            "name": "my-app",
            ... main, polyfills, etc
            "styles": [
                "layout.scss"
            ]
        }
    ]
}

I've just confirmed layout.scss is not referenced anywhere else in the project, only angular-cli.json.
Here is a screenshot of the inspector, as you can see there are A DOZEN <style> tags.


FunctionExpression assignment with explicit return type fails to transpile

Using version 0.7.27
The following TS code fails to transpile:
export function main() {
  var times2 = function(x: number): number {
    return x * 2;
  };
  console.log('5 * 2 = %s', times2(5));
}

Attempting to transpile this yields the following output:
fn.ts:1:1: Could not format because the source could not be parsed:

line 1, column 36: Expected to find ')'
 main ( ) { var times2 = num ( num x ) { return x * 2 ; } ; console . log ( "5 * 2 = %s" , times2 ( 5 ) ) ; }
                                   ^
line 1, column 32: Expected to find ';'
 main ( ) { var times2 = num ( num x ) { return x * 2 ; } ; console . log ( "5 * 2 = %s" , times2 ( 5 ) ) ; }
                               ^^^
line 1, column 36: Expected to find ';'
 main ( ) { var times2 = num ( num x ) { return x * 2 ; } ; console . log ( "5 * 2 = %s" , times2 ( 5 ) ) ; }
                                   ^
line 1, column 38: Expected an identifier
 main ( ) { var times2 = num ( num x ) { return x * 2 ; } ; console . log ( "5 * 2 = %s" , times2 ( 5 ) ) ; }
                                     ^
line 1, column 38: Expected to find ';'
 main ( ) { var times2 = num ( num x ) { return x * 2 ; } ; console . log ( "5 * 2 = %s" , times2 ( 5 ) ) ; }
                                     ^
line 1, column 38: Unexpected token ')'
 main ( ) { var times2 = num ( num x ) { return x * 2 ; } ; console . log ( "5 * 2 = %s" , times2 ( 5 ) ) ; }

Note that explicitly specifying times2s type still fails to compile.
This does compile, however, if:

You remove the explicit return type from times2.
You turn times2 into a declaration rather than an expression.

Note that you would probably want to use arrow functions in almost all cases.


Filter out null nodes in nodeToHostNode

Fixes https://github.com/wojtekmaj/enzyme-adapter-react-17/issues/42
This filters out null values from node arrays in the adapter. The problem is that some React component trees render as an array with possible null values when using React fragments, e.g.
<>
  <EmptyFragmentComponent />
  <ConditionallyNullComponent />
  <SomeComponent />
</>

where EmptyFragmentComponent renders an empty fragment (it exists to run useEffect hooks, for instance) and ConditionallyNullComponent may render a null value explicitly based on the state of the application (e.g. if not authenticated, don't show a banner). SomeComponent renders a div and so is not null in the nodes array.
In the resulting nodes value from nodeToHostNode, the output is [null, null, Object], which causes nodes[0] to return null and give the error TypeError: Cannot read property '__reactFiber$ul8w880oi5m' of null during testing.

Hmmm, this change doesn't seem to break anything - that's great. Would it be possible for you to write a test that breaks on main and passes on your code? That would be super helpful.

Thanks!

Has this been released to npm yet?

Nope, no release yet.

Do you have a timeframe when it might be released? Thanks.

v0.6.7 released.

Thank you.

@wojtekmaj thanks! Sorry I didn't respond, was on vacation for a bit, thanks for adding the unit test in.


New Interface For the website

Description
Made new UI for the Website
Fixes:  #538

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@Abh1shekSingh i am giving lvl 1 i think its fair enough

I am working on real stuff and you giving me lvl 1 and Other who are copy
pasting the stuff, clones , they are getting lvl 3. It's great
On Sun, Apr 3, 2022, 3:15 PM Arpit Jain @.***> wrote:

@Abh1shekSingh https://github.com/Abh1shekSingh i am giving lvl 1 i
think its fair enough
—
Reply to this email directly, view it on GitHub
https://github.com/arpit456jain/Cool-Front-End-Templates/pull/570#issuecomment-1086824665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARFFTF3VMTDEITZMFK6MCLTVDFSETANCNFSM5SMQUKCA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Create leetcode2325.cpp

Not accepting CPP/C DSA codes
Alternative ->
1)Create a page using HTML & CSS Add DSA problem statement and solution in that
2)You can contribute in python and java OR
Ask for an issue assign
3)Contribute to Web Project


Remove "Add New Tag" from Tag Notebook page

The "Add New Tag" button is redundant with the one in the Tag modal, and makes it sounds like the user clicks there to add a tag to the spot. In addition, Change "Add New Tag" in the Tag modal to "Create New Tag". Clearer, I think.

Closed with commit ab96cf4


entries with same name in different blocks fail

The following simple code fails:
  $group1 = 'group1'
  $group2 = 'group2'

  sshd_config {"ChrootDirector ${group1}":
    ensure    => present,
    condition => "Group ${group1}",
    value     => '/home/group1/%u',
  }

  sshd_config {"ChrootDirectory ${group2}":
    ensure    => present,
    condition => "Group ${group2}",
    value     => '/home/group2/%u',
  }

Here is the error message:
Notice: /Stage[main]/Main/Sshd_config[ChrootDirector group1]/ensure: created
Error: /Stage[main]/Main/Sshd_config[ChrootDirector group1]: Could not evaluate: Failed to save Augeas tree to file. See debug logs for details.
Notice: /Stage[main]/Main/Sshd_config[ChrootDirectory group2]/ensure: created
Error: /Stage[main]/Main/Sshd_config[ChrootDirectory group2]: Could not evaluate: Failed to save Augeas tree to file. See debug logs for details.

That is expected, you need different titles with a varying name parameter.
See the doc at
http://augeasproviders.com/documentation/examples.html#sshdconfig-provider
On Mon, 20 Apr 2015 17:41 Vadym Chepkov<EMAIL_ADDRESS>wrote:

The following simple code fails:
$group1 = 'group1'
$group2 = 'group2'
sshd_config {"ChrootDirector ${group1}":
ensure    => present,
condition => "Group ${group1}",
value     => '/home/group1/%u',
}
sshd_config {"ChrootDirectory ${group2}":
ensure    => present,
condition => "Group ${group2}",
value     => '/home/group2/%u',
}
Here is the error message:
Notice: /Stage[main]/Main/Sshd_config[ChrootDirector group1]/ensure: created
Error: /Stage[main]/Main/Sshd_config[ChrootDirector group1]: Could not evaluate: Failed to save Augeas tree to file. See debug logs for details.
Notice: /Stage[main]/Main/Sshd_config[ChrootDirectory group2]/ensure: created
Error: /Stage[main]/Main/Sshd_config[ChrootDirectory group2]: Could not evaluate: Failed to save Augeas tree to file. See debug logs for details.
—
Reply to this email directly or view it on GitHub
https://github.com/hercules-team/augeasproviders_ssh/issues/9.

Hmm, they do have different titles

Oh, I see now, I missed key attribute. Never mind, sorry for the noise. Thanks


xml

The color matching of the xml file code is the same color as the background when it is displayed, hoping to provide solutions and improvements to the template

I see, well it's hard to know which variable you change with your commit messages, did you change the text color variables or the scss files?
By default it works, check: https://sylhare.github.io/Type-on-Strap/2019/11/02/Tech-stuff-example.html

Closing for now

I changed and submitted the article directly after downloading, or I will try again


build: upgrade to Go 1.13 and Debian 10

See: https://golang.org/doc/go1.13

Codecov Report

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

@@           Coverage Diff           @@
##           master      #61   +/-   ##
=======================================
  Coverage   88.43%   88.43%
=======================================
  Files          72       72
  Lines        5785     5785
=======================================
  Hits         5116     5116
  Misses        420      420
  Partials      249      249


Add postcss-loader when postcssLoaderOptions is present

Specifying postcssLoaderOptions should trigger postcss-loader to be included in webpack.
Example use case: This would allow configs to directly include postcss plugins and reduce the amount of config files in a project.

This is a very useful feature as some postcss plugins can't be used without webpack context and postcss.config.js doesn't allow to access it.
For example, this one -> https://github.com/jantimon/iconfont-webpack-plugin

@ruslansavenok this should be good now.

@tamagokun seems to be. But I'm not affiliated with zeit and don't have permissions to merge this PR :)

darn 🤔

Fixed conflicts :+1:

Hey just want to add my support here -- it's a very small and simple fix and would be very useful.

I'm going to close this in favor of https://github.com/zeit/next.js/issues/8626
Thanks for your contribution 🙏


Command failed: osascript

I'm getting the below error when trying to turn speakers on and off on iTunes 12.10. Any idea why?
'[2020-10-22T16:23:57.649Z] - ::ffff:<IP_ADDRESS> - PUT /airplay_devices/40-3c-fc-05-7f-38/off 200 123.164 ms - 207b'
Error: Command failed: osascript -l JavaScript -e 'var old = console.log; console.log = function () { Array.prototype.unshift.call(arguments, "<brandonhorst:node-osa>"); Array.prototype.push.call(arguments, "</brandonhorst:node-osa>"); old.apply(console, arguments); }; JSON.stringify((function (id, selected){    itunes = Application('\''iTunes'\'');    id = id.replace(/-/g, '\'':'\'');    foundAirPlayDevice = null;    deviceData = {};    airPlayDevices = itunes.airplayDevices();    for (i = 0; i < airPlayDevices.length; i++) {      airPlayDevice = airPlayDevices[i];      if (airPlayDevice.networkAddress() == id || airPlayDevice.name() == id) {        foundAirPlayDevice = airPlayDevice;        break;      }    }    if (foundAirPlayDevice) {      foundAirPlayDevice.selected = selected;      if (foundAirPlayDevice.networkAddress()) {        deviceData["id"] = foundAirPlayDevice.networkAddress().replace(/:/g, '\''-'\'');      } else {        deviceData["id"] = foundAirPlayDevice.name();      }      deviceData["name"] = foundAirPlayDevice.name();      deviceData["kind"] = foundAirPlayDevice.kind();      deviceData["active"] = foundAirPlayDevice.active();      deviceData["selected"] = foundAirPlayDevice.selected();      deviceData["sound_volume"] = foundAirPlayDevice.soundVolume();      deviceData["supports_video"] = foundAirPlayDevice.supportsVideo();      deviceData["supports_audio"] = foundAirPlayDevice.supportsAudio();      deviceData["network_address"] = foundAirPlayDevice.networkAddress();    }    return deviceData;  })("Computer",false));'
0:1496: execution error: Error on line 1: Error: Parameter error. (-50)

    at ChildProcess.exithandler (child_process.js:308:12)
    at ChildProcess.emit (events.js:314:20)
    at maybeClose (internal/child_process.js:1051:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:287:5) {
  killed: false,
  code: 1,
  signal: null,
  cmd: `osascript -l JavaScript -e 'var old = console.log; console.log = function () { Array.prototype.unshift.call(arguments, "<brandonhorst:node-osa>"); Array.prototype.push.call(arguments, "</brandonhorst:node-osa>"); old.apply(console, arguments); }; JSON.stringify((function (id, selected){    itunes = Application('\\''iTunes'\\'');    id = id.replace(/-/g, '\\'':'\\'');    foundAirPlayDevice = null;    deviceData = {};    airPlayDevices = itunes.airplayDevices();    for (i = 0; i < airPlayDevices.length; i++) {      airPlayDevice = airPlayDevices[i];      if (airPlayDevice.networkAddress() == id || airPlayDevice.name() == id) {        foundAirPlayDevice = airPlayDevice;        break;      }    }    if (foundAirPlayDevice) {      foundAirPlayDevice.selected = selected;      if (foundAirPlayDevice.networkAddress()) {        deviceData["id"] = foundAirPlayDevice.networkAddress().replace(/:/g, '\\''-'\\'');      } else {        deviceData["id"] = foundAirPlayDevice.name();      }      deviceData["name"] = foundAirPlayDevice.name();      deviceData["kind"] = foundAirPlayDevice.kind();      deviceData["active"] = foundAirPlayDevice.active();      deviceData["selected"] = foundAirPlayDevice.selected();      deviceData["sound_volume"] = foundAirPlayDevice.soundVolume();      deviceData["supports_video"] = foundAirPlayDevice.supportsVideo();      deviceData["supports_audio"] = foundAirPlayDevice.supportsAudio();      deviceData["network_address"] = foundAirPlayDevice.networkAddress();    }    return deviceData;  })("Computer",false));'`

So, it turns out this was due to sending a change volume request too quickly after a play playlist request against the latest version of iTunes. This was also compounded by some other issues with iTunes failing to turn on some speakers in time and needing to retry pressing play again after some time. I ended up making an app daemon that chooses the speakers and adjusts the volumes when playing a playlist and then will retry pressing play every 10 seconds until it works. Now I no longer get this error.
So if anyone else has this problem, they could use my app daemon playlist launcher and airplay speaker manager (let me know and I will clean it up for use by the general public as it's kind of a dumpster fire right now although it works).  You can find it here.
The right way to fix this would be for itunes-api to operate more synchonously and queue messages to itunes such as playing a playlist until after volume changes of speakers had processed, and retrying playing a playlist if it errors out due to airtunes timeouts. Unfortunately despite having looked at the code quite a lot, I don't understand the applescript side of it well enough to do this. I'm unfortunately not a good enough programmer to do this myself. So I will leave this open..


QuantumRegister indexing is recursive and shouldn't be

Information

Qiskit Terra version: 0.7.1
Python version: 3.5.6
Operating system: Windows 7

Steps to reproduce the problem
q = QuantumRegister(1)
def inf_recurse(qarg):
    print( qarg[0] )
    inf_recurse( qarg[0] )

What is the current behavior?
When indexing a QuantumRegister, indexing can be done recursively and therefore infinitely.
> QuantumRegister(1, 'q0')
> (QuantumRegister(1, 'q0'), 0)
> QuantumRegister(1, 'q0')
> (QuantumRegister(1, 'q0'), 0)
> QuantumRegister(1, 'q0')
> (QuantumRegister(1, 'q0'), 0)
> QuantumRegister(1, 'q0')
> (QuantumRegister(1, 'q0'), 0)
> QuantumRegister(1, 'q0')
> (QuantumRegister(1, 'q0'), 0)
> File "qiskit-terra\qiskit\circuit\register.py", line 77, in __getitem__
     self.check_range(key)
   File "qiskit-terra\qiskit\circuit\register.py", line 59, in check_range
     if j < 0 or j >= self.size:
 RecursionError: maximum recursion depth exceeded in comparison

What is the expected behavior?
> QuantumRegister(1, 'q0')
> Qubit('q0_0')
> TypeError. Qubit object is not subscriptable

Suggested solutions
My first suggestion would be to restructure the QuantumRegister as a list of some inner Qubit object. Since these objects are only used for book-keeping on the qubits, it would make more sense to separate the collection functionality and the qubit functionality into two classes.

See also #1647, which raises a similar concern with the usability of QuantumRegister.

@kdk The fixes that would address this issue would also address my issue. Should I close/delete this then? They're not exactly the same but the fix for #1647 would also fix mine.

@Woody1193 Great, I'll close this then.


Replace variables.Agent.OS with variables['Agent.OS']

In "Switch based on platform" the three conditions for the steps are broken right now. Instead of variables.Agent.OS you have to use variables['Agent.OS'], otherwise the variable evaluates to NULL which will not match the string.

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

ID: 797838e7-f32b-868e-8766-8d96973bf18f
Version Independent ID: 20c9e62d-56ec-f5a0-c3f9-6f72bf96252e
Content: Cross-platform scripting - Azure Pipelines & TFS
Content Source: docs/pipelines/scripts/cross-platform-scripting.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

The example script has been updated via. https://github.com/MicrosoftDocs/vsts-docs/commit/1a246f2ad9faa5693fc00366ed10ec47d77d8c13.


apply code highlighting to WCF docs

Summary
This is a continuation of https://github.com/dotnet/docs/pull/4991
Add language designation (c#, svc etc) to code blocks in the following portions of the WCF docs:
WCF :: Basic :: Services
WCF :: Basic :: Routing Services
WCF :: :Basic :: Management
WCF :: Basic: :: Discovery
WCF :: Basic :: Contact

@rpetrusha I’ll make those changes soon and update the PR.
One note: in my previous PR, it was specifically requested to use svc for those files, so I used it here to be consistent. https://github.com/dotnet/docs/pull/4991#discussion_r183850464


missing installation of piven library

try with pip install piven   but didnt work kindly mention installation steps

This library is not on pypi so installing it from pip won't work unless you use the following:
pip install git+https://github.com/godatadriven/piven.git


Change font of mobile version to same as desktop version

Change the font style of mobile version, make it same as the desktop site. Also change the font color of "AnimePedia" to dark grey or black and "All Anime Characters" to something familiar.

i would like to work on this

Ok I will be assigning you the issue. Kindly mention the issue no. in the description of your PR

@iamartyaa bro iam also working in this issue already

Is it so @FixxooXD ? I didn't see any open issue regarding it. Anyways you can continue working ok it and open a PR.

@iamartyaa I cant see any change in the font style in mobile version i had changed the colour of "ANIMEPEDIA" can i put a pr
?

I would like to work on this issue.


Improve user friendliness to newcomers

Based on tests, I believe the tutorial needs to be more guided and the directions more specific.

[x] Teach that the "look" verb is used for everything look related by default: reading, examining, etc. Just use "look".
[x] Be specific about how the other key verbs work.
[x] Make "help" work in the lobby.
[ ] Show the lobby menu after cancelling commands (and after using the rest of commands that don't take you out of the lobby?)
[x] In the tutorial curtain room: Tell them that to go through it, he has to write "go "
[ ] Make a list of probable worlds he may refer to. If he writes "exit", maybe he means "exits",
[x] In the last room, tell the players to look at items and hint all commands that have to be used,
[x] Make the go and look verbs say "you don't find X here" instead of "I don't understand that" when the object is not found.
[ ] Repeat "\ to cancel" in every interactive verb prompt.
[x] Alias the name of some help topics, like "build" for "building".
[x] If you are in the lobby and enter a wrong verb: tell you can use "r" to show the lobby menu again.

Keep an eye on new sessions and test users to find what can be improved.

Some time has passed, I think I should look into this again.

I close this issue, since it is so generic it could remain open forever.


input_bam.fofn contents

Dear Jason,
Hello!
Can you briefly explain input_bam.fofn contents ?
How can I generate it ?
Thank you.
Won

it is just a list of files to the PacBio BAM files that contains the pulse level signals.

Then how can I generate it? BAX2BAM?

if you only have BAX files. you will need to generate from BAX2BAM.

Or can I generate from fastq file?
Won

No. Those fastq files do not have enough information.

Hi @pb-jchin :
Can you please explain a little bit what is "the PacBio BAM files that contains the pulse level signals" that you mentioned above?
I have another two questions:

during I run fc_unzip.py, I find there is a sorted .bam file under ../3-unzip../../blasr/ directory, can we let FALCON-unzip use this BAM to perform consensus calling? If we can, then how?

for my own data, I also have 50X Illumina reads, can I map these reads to p_contigs.fasta that is generated from FALCON using like BWA, then use the BAM file (probably after sorting, indexing) to feed FALCON_unzip for consensus calling? Thanks.

@wsuplantpathology please refer to http://pacbiofileformats.readthedocs.io/en/3.0/BAM.html for the BAM format. If you have older data, you might have the information stored in hdf5 file format. In such case, you will need to run bax2bam to convert it to the (unaligned) BAM file with extra tags for PacBio specific information.
FALCON-Unzip does have utility to do haplotype specific consensus as we describe in the publication.  It is initiated with fc_quiver.py after the unzip step.
For 2, you can align Illumina read to p_contigs.fa, but you have to be aware BWA does not "know" you genome is a diploid genome. You will have to examine the alignment and probably do some proper filtering for the downstream processes that you like to do.

Hi wsuplantpathology,

No, the sorted bam file to which you are referring (e.g. 3-unzip/0-phasing/000000F/blasr/000000F_sorted.bam) has reads from both haplotypes. If you called consensus on this particular bamfile, it would result in a haploid consensus sequence, effectively nullifying the unzipping you just performed. This bam file is the sorted alignment file from which SNPS are identified and reads are subsequently segregated by phase. You should use fc_quiver.py for the easiest unzipped assembly consensus calling.

FALCON unzip will not take a random bamfile for consensus calling, it has to be the original machine generated raw bamfile. You can definitely use 50x Illumina reads to correct, however be aware of the caveat jchin noted above. Also, you would probably need to refer to pilon or some other short read based polishing tool.

Thanks @pb-jchin @pb-jchin for clarification. Probably I'll try bax2bam later.

@pb-jchin Do we need additional "pulse features (QVs)" such as DeletionQV,DeletionTag,InsertionQV,IPD,MergeQV,SubstitutionQV,PulseWidth,SubstitutionTag ?
Thank you!

bax2bam default should load all necessary values.

Thank you.

I wanted to try PacBio + ONT reads to assemble with Falcon+FalconUnzip. But I am worried for input_bam.fofn file. Is there any way to create it ?

Pacbio supports the use case of FALCON with PacBio reads.
For other situations, you're welcome to use the software - but you will be on your own as they are unsupported.


Adding num_streams to chttp2_transport

Adding num_streams to chttp2_transport. This decouples checking of number of streams from the stream map. Also solves the race condition uncovered in issue #12201

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Let's find a way to do this without the considered closed bit getting
added: it's likely redundant information that will lead to bugs down the
road
On Wed, Sep 6, 2017, 6:17 PM gRPC Jenkins<EMAIL_ADDRESS>wrote:

[microbenchmarks] No significant performance differences
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/12414#issuecomment-327654812, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJpudWmdQ93EaOLSzq3dC_owIjefbTrDks5sf0QOgaJpZM4PO0CB
.


Ignore a bower_components folder

My project language is incorrectly identified as javascript(it should be java), because I'm using a lot of bower dependencies in my project, with a ton of javascript files.
I just want linguist to ignore all the files in these folders and their subfolders
I've tried two lines in my .gitattributes file, all of which failed, and the false one actually inflated the number of javascript files recognized.
src/main/resources/web/bower_components/* linguist-vendored=false
src/main/resources/web/bower_components/* linguist-vendored

Is it a public repository? Could you link to it? :-)

https://github.com/tchoulihan/torrenttunes-client

I ran Linguist against your repository: https://gist.github.com/pchaigno/7c1130a3a57ca48a2fa2.
I looks like you still have some big JavaScript files which aren't in .gitattribute ;)

What should my line be to ignore this folder?
src/main/resources/web/bower_components/

Maybe src/main/resources/web/js/? Have you looked at the file I linked? It's the details of the language statistics in your repo.

Oops thanks for linking.
What would the line be if I wanted to exclude/ignore a certain file?

The full path to the file with linguist-vendored.

Okay thanks! I'll reopen this then if I have problems.


Faster Version of RMQ?

iirc the current RMQ can be too slow on some suffix array problems

Added in b61cf39 (though I am curious whether there are any other really simple optimizations that can be made)


Make WebResponseExceptionTranslator generic to ease custom error formats for resource servers

The OAuth 2.0 specification defines how the authorization server error responses must be, but it does not force resource servers to adopt the same format.
While adding OAuth 2.0 security to our services, we found it confusing for our clients to have to deal with two different error formats : one for "business errors" the other for authorization errors.
Making WebResponseExceptionTranslator<T> generic allows Resource Servers to easily implement their own error format on top of framework classes, instead of having to copy a bunch of classes :
AbstractOauth2SecurityExceptionHandler, etc.
On the Authorization Server side, the only change is that components
explicitely define that the error type should be OAuth2Exception :
WebResponseExceptionTranslator<OAuth2Exception>

BTW the change is backward compatible. Only people who have already overriden related classes will get "generic type" warnings.

@michaeltecourt Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@michaeltecourt Thank you for signing the Contributor License Agreement!

LGTM!

Hi @jgrandja, I've rebased this PR against the master, the build is back to green.
Is there anything I can do to help this feature get merged ? a bunch of people have asked about it in issue #605.
We have been using this code in production for the past two years, and I'd love to remove the duplicated classes in our codebase :)

@michaeltecourt I'll take a look at this when I return from the holiday break Jan 8.

Thanks for the PR @michaeltecourt ! This is now in master.


Dynamic ingress rules for SecurityGroup

Currently, the ingress rules are created statically within the factory - these should come from args so specific rules can be added by the provider.

this should also include dynamically generating our rule for eks rather than it always being included


Long Running Docker Jobs

I'm wondering if now can handle long running tasks and not just short web services tasks? Think serverless docker.
Here is the example I setup.
Code
Dockerfile
 FROM ubuntu:latest
RUN apt-get update -y
RUN apt-get install -y python-pip python-dev build-essential
COPY . /app
WORKDIR /app
RUN pip install -r requirements.txt
ENTRYPOINT ["python"]
EXPOSE 5000
CMD ["app.py"]

app.py
from flask import Flask
import time
app = Flask(__name__)

@app.route('/')
def hello_world():
  time.sleep(2)
  return 'Label On!'

if __name__ == '__main__':
  app.run(debug=True,host='<IP_ADDRESS>')

requirements.txt
Flask==0.10.1

My Test
Then I deployed to now and scaled the deployment.
now
now scale <url> 1 auto

Lastly I sent a bunch of requests in the hope they would occur in parallel.
curl <deployment-url> & curl <deployment-url> & curl <deployment-url> & curl <deployment-url> & curl <deployment-url> & curl <deployment-url> & curl <deployment-url> & ...

But the requests were just processed sequentially :(
I also increased the scale of my deployment but didn't see any changes.
now scale <url> 5

This is definitely how it works. How are you determining that they happened sequentially?

The result I'm getting is...
Label On!
<2 seconds pause>
Label On!
<2 seconds pause>
Label On!
<2 seconds pause>
Label On!
<2 seconds pause>
Label On!
<2 seconds pause>

And if the requests were all hitting different instances, running the example above, I would expect...
<2 seconds pause>
Label On!
Label On!
Label On!
Label On!
Label On!


AssimpKit can not display all the animations of the FBX model

AssimpKit can not display all the animations of the FBX model, only a portion of the animation. I would like to ask what is the reason for such a result

Please attach a model file that demonstrates the problem
Does your file contain multiple animations? e.g.; walk, run, jump
Does a single animation not play completely is what you mean when you say only a portion of the animation.
Please attach the relevant code that loads/plays these animations

This is a demonstration model.
2.FBX.zip
This model is a swinging couplet, but this model is only swingable by the left pair.
The sample code is as follows:

(void)viewDidLoad
{
[super viewDidLoad];
NSString *filePath = [[NSBundle mainBundle] pathForResource:@"duilian1" ofType:@"FBX"];
self.modelFilePath = filePath;
self.animFilePath = self.modelFilePath;
SCNAssimpScene *scene =
[SCNScene assimpSceneWithURL:[NSURL URLWithString:self.modelFilePath]
postProcessFlags:AssimpKit_Process_FlipUVs |
AssimpKit_Process_Triangulate];
// Load the animation scene
if (self.animFilePath)
{
SCNAssimpScene *animScene =
[SCNScene assimpSceneWithURL:[NSURL URLWithString:self.animFilePath]
postProcessFlags:AssimpKit_Process_FlipUVs |
AssimpKit_Process_Triangulate];
NSArray *animationKeys = animScene.animationKeys;
// If multiple animations exist, load the first animation
if (animationKeys.count > 0)
{
SCNAssimpAnimSettings *settings =
[[SCNAssimpAnimSettings alloc] init];
settings.repeatCount = 3;
      NSString *key = [animationKeys objectAtIndex:0];
      SCNAnimationEventBlock eventBlock =
      ^(CAAnimation *animation, id animatedObject,
        BOOL playingBackward) {
          NSLog(@" Animation Event triggered ");

          // To test removing animation uncomment
          // Then the animation wont repeat 3 times
          // as it will be removed after 90% of the first loop
          // is completed, as event key time is 0.9
          // [scene.rootNode removeAnimationSceneForKey:key
          //                            fadeOutDuration:0.3];
          // [scene.rootNode pauseAnimationSceneForKey:key];
          // [scene.rootNode resumeAnimationSceneForKey:key];
      };
      SCNAnimationEvent *animEvent =
      [SCNAnimationEvent animationEventWithKeyTime:0.1f
                                             block:eventBlock];
      NSArray *animEvents =
      [[NSArray alloc] initWithObjects:animEvent, nil];
      settings.animationEvents = animEvents;
      settings.delegate = self;

      SCNScene *animation = [animScene animationSceneForKey:key];
      [scene.modelScene.rootNode addAnimationScene:animation
                                            forKey:key
                                      withSettings:settings];
  }

}
// retrieve the SCNView
SCNView *scnView = (SCNView *)self.view;
// set the scene to the view
scnView.scene = scene.modelScene;
// allows the user to manipulate the camera
scnView.allowsCameraControl = YES;
// show statistics such as fps and timing information
scnView.showsStatistics = YES;
// configure the view
scnView.backgroundColor = [UIColor blackColor];
scnView.scene.rootNode.position = SCNVector3Make(0, 0, -50);
scnView.playing = YES;
}

This is a demonstration model.
duilian.FBX.zip
This model is a swinging couplet, but this model is only swingable by the left pair.
The sample code is as follows:
(void)viewDidLoad
{
[super viewDidLoad];

NSString *filePath = [[NSBundle mainBundle] pathForResource:@"duilian" ofType:@"FBX"];
self.modelFilePath = filePath;
self.animFilePath = self.modelFilePath;
SCNAssimpScene *scene =
[SCNScene assimpSceneWithURL:[NSURL URLWithString:self.modelFilePath]
postProcessFlags:AssimpKit_Process_FlipUVs |
AssimpKit_Process_Triangulate];
// Load the animation scene
if (self.animFilePath)
{
SCNAssimpScene *animScene =
[SCNScene assimpSceneWithURL:[NSURL URLWithString:self.animFilePath]
postProcessFlags:AssimpKit_Process_FlipUVs |
AssimpKit_Process_Triangulate];
NSArray *animationKeys = animScene.animationKeys;
// If multiple animations exist, load the first animation
if (animationKeys.count > 0)
{
SCNAssimpAnimSettings *settings =
[[SCNAssimpAnimSettings alloc] init];
settings.repeatCount = 3;
  NSString *key = [animationKeys objectAtIndex:0];
  SCNAnimationEventBlock eventBlock =
  ^(CAAnimation *animation, id animatedObject,
    BOOL playingBackward) {
      NSLog(@" Animation Event triggered ");

      // To test removing animation uncomment
      // Then the animation wont repeat 3 times
      // as it will be removed after 90% of the first loop
      // is completed, as event key time is 0.9
      // [scene.rootNode removeAnimationSceneForKey:key
      //                            fadeOutDuration:0.3];
      // [scene.rootNode pauseAnimationSceneForKey:key];
      // [scene.rootNode resumeAnimationSceneForKey:key];
  };
  SCNAnimationEvent *animEvent =
  [SCNAnimationEvent animationEventWithKeyTime:0.1f
                                         block:eventBlock];
  NSArray *animEvents =
  [[NSArray alloc] initWithObjects:animEvent, nil];
  settings.animationEvents = animEvents;
  settings.delegate = self;

  SCNScene *animation = [animScene animationSceneForKey:key];
  [scene.modelScene.rootNode addAnimationScene:animation
                                        forKey:key
                                  withSettings:settings];

}
}
// retrieve the SCNView
SCNView *scnView = (SCNView *)self.view;
// set the scene to the view
scnView.scene = scene.modelScene;
// allows the user to manipulate the camera
scnView.allowsCameraControl = YES;
// show statistics such as fps and timing information
scnView.showsStatistics = YES;
// configure the view
scnView.backgroundColor = [UIColor blackColor];
scnView.scene.rootNode.position = SCNVector3Make(0, 0, -50);
scnView.playing = YES;
}

After I tried to change the AssimpKit source code, I solved the animation problem but did not know if it was correct and the changes were as follows:
No change before :

After the change :

Thanks for providing the file. Comparing it with FBX review tool, I can see only 1 vertical block on the left is animated using AssimpKit.
Btw, your code change is wrong and unnecessary. You should probably read how to load animations.
The gist is: you load the model scene, then load the animations (from the scene you just loaded or another scene if they are defined externally) and add the animation to the model scene. The above docs referred to use the same scenarios.

So AssimpKit loading FBX animation is a problem right?
The correct way is as follows:

(void)viewDidLoad
{
[super viewDidLoad];
NSString *filePath = [[NSBundle mainBundle] pathForResource:@"duilian" ofType:@"FBX"];
SCNAssimpScene *animScene =
[SCNScene assimpSceneWithURL:[NSURL URLWithString:filePath]
postProcessFlags:AssimpKit_Process_FlipUVs |
AssimpKit_Process_Triangulate];
NSArray *animationKeys = animScene.animationKeys;
// If multiple animations exist, load the first animation
if (animationKeys.count > 0)
{
SCNAssimpAnimSettings *settings =
[[SCNAssimpAnimSettings alloc] init];
settings.repeatCount = 3;
  NSString *key = [animationKeys objectAtIndex:0];
  SCNAnimationEventBlock eventBlock =
  ^(CAAnimation *animation, id animatedObject,
    BOOL playingBackward) {
      NSLog(@" Animation Event triggered ");

      // To test removing animation uncomment
      // Then the animation wont repeat 3 times
      // as it will be removed after 90% of the first loop
      // is completed, as event key time is 0.9
      // [scene.rootNode removeAnimationSceneForKey:key
      //                            fadeOutDuration:0.3];
      // [scene.rootNode pauseAnimationSceneForKey:key];
      // [scene.rootNode resumeAnimationSceneForKey:key];
  };
  SCNAnimationEvent *animEvent =
  [SCNAnimationEvent animationEventWithKeyTime:0.9f
                                         block:eventBlock];
  NSArray *animEvents =
  [[NSArray alloc] initWithObjects:animEvent, nil];
  settings.animationEvents = animEvents;
  settings.delegate = self;

  SCNScene *animation = [animScene animationSceneForKey:key];
  [animScene.modelScene.rootNode addAnimationScene:animation
                                        forKey:key
                                  withSettings:settings];

}
// retrieve the SCNView
SCNView *scnView = (SCNView *)self.view;
// set the scene to the view
scnView.scene = animScene.modelScene;
// allows the user to manipulate the camera
scnView.allowsCameraControl = YES;
// show statistics such as fps and timing information
scnView.showsStatistics = YES;
// configure the view
scnView.backgroundColor = [UIColor blackColor];
scnView.scene.rootNode.position = SCNVector3Make(0, 0, -20);
scnView.playing = YES;
}

Yes, hence I have labeled this issue as a bug.

Thank you @dmsurti  for sharing, looking forward to your update

Hi!
Any updates?

I'm banging my head against the wall on similiar issue downstream with swift port
https://github.com/eugenebokhan/AssetImportKit/issues/14

UPDATE - I got the library loading fbx + animations with swift / via carthage. The eugenebokhan/AssetImportKit seems to not work. https://github.com/johndpope/swift-2


Slow running time for scrapping all subs for an author

Hi. I'm trying to use pmaw to iterate a list of usernames and scrape all their submissions and comments.
It turns out to be very slow.
Using pmaw:
gen = api2.search_submissions(author='seeellayewhy').
list(gen)

It took 7s to scrape 300+ submission

Using pmaw, it took several minutes to finish it:
from pmaw import PushshiftAPI

api = PushshiftAPI(num_workers=40, )

submisions = api.search_submissions(author='seeellayewhy', limit=None)
submission_list = [sub for sub in submisions]

WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:325 result(s) available in Pushshift
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 40 - Batches: 1 - Items Remaining: 325
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 80 - Batches: 2 - Items Remaining: 323
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 120 - Batches: 3 - Items Remaining: 318
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 161 - Batches: 5 - Items Remaining: 300
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:1 result(s) not found in Pushshift
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 201 - Batches: 6 - Items Remaining: 271
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 241 - Batches: 7 - Items Remaining: 267
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 281 - Batches: 8 - Items Remaining: 252
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 321 - Batches: 9 - Items Remaining: 238
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:Checkpoint:: Success Rate: 100.00% - Requests: 361 - Batches: 10 - Items Remaining: 221
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 361 - Batches: 10 - Items Remaining: 221
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
WARNING:pmaw.PushshiftAPIBase:Not all PushShift shards are active. Query results may be incomplete.
INFO:pmaw.PushshiftAPIBase:Total:: Success Rate: 100.00% - Requests: 400 - Batches: 11 - Items Remaining: 0

Is there something I'm not doing it right, or it's not recommended to do this kind of task with pmaw? From the logging, it seems like the each requests are requesting only several records? Does it has to do with search windows? If so, how to configure for this kind of tasks.

Hi @ReichYang can you try running again with the default number of workers (10)? For a small number of results using more workers can slow the completion time for multiple reasons. Aside from the overhead of multithreading, fewer submissions are retrieved in each request, and there is more competition between workers, causing requests to fail.
If you are expecting <1000 results for a query, using less workers (maybe 2-5) may yield faster completion times. You can also refer to the benchmarks to see how pmaw performance improves as the number of results increases.

@mattpodolak Hi. Please see my screenshots. It is still considerably slower than just requesting pushshift.io API or using psaw.
Using default number of workers.

Using 2 workers.

Using only one worker.

Since I don't know the number of results for each author, is there a way to dynamically update the workers? I guess most will have only a few results but there might be some active users that post a lot on Reddit.


Set proper rev for processed deleted doc

View changes needs to return the "winning" rev to the client. With
default style, it does this by looking up the rev in the seq tree. Prior
to this commit, when a document was being deleted, the default rev (ie.
{0, []}) would be added to the seq tree. This commit fixes the problem
by manually setting the proper rev in the deleted #doc record before
it's processed.

+1
@rnewson ?


SSHExecutor returns ModuleNotFoundError for cloudpickle despite being installed in remote machine

SSHExecutor yields ModuleNotFoundError for cloudpickle. To reproduce this error, run the quantum machine learning tutorial doc/source/tutorials/machine_learning/classical_quantum_svm.ipynb under covalent

A workaround for this issue is to specify the python path of the active Conda environment in the remote machine when instantiating the SSH executor.
ssh_exec = SSHExecutor(
    username="",
    hostname="",
    remote_dir="/tmp/covalent",
    ssh_key_file="/Path/to/key_file",
    python3_path="/Path/to/python3"
)

If this issue is encountered it is very likely an issue with resolving which python binary is to be used, particularly when using virtualenvs.
If using conda the recommended action is to use the conda_env parameter to reference the named conda environment that was created (this is now supported as of https://github.com/AgnostiqHQ/covalent-ssh-plugin/pull/28)
Otherwise using the python3_path (now called python_path) to point to the exact python binary to be used resolves the issue.

@venkatBala @Emmanuel289 This issue should now be resolved. Can you guys confirm if this still exists? Now you can specify the conda environment with conda_env, so no need to mess around with python paths.


RLT Support ?

General Information

NKVPhonePicker Version:

iOS Version(s):

Swift Version:

Device or Simulator:

Did you check an example:

What happened?
What did you expect to happen?

The worst description ever.


Allow additional properties in project schemas

Having additionalProperties set to false in schemas for project files makes backwards/future compatibility a problem. For example, a tool written to handle project files according to version 1.6 will fail to validate files created by tools for the newer version 1.7 if the latter has added new properties to some objects. This can be seen as unnecessarily strict, since added properties rarely is a breaking change.
Contributed by STMicroelectronics

Disallowing additional properties is important for YAML syntax highlighting in IDE editors.
It's an immediate feedback when a node does not exist, is incorrectly placed or mistyped.
I would suggest to bring the matter to a wider audience at the weekly discussions.

The tools should be able to enable/disable the schema check as required by the user.

Closing this. We will patch this internally when/if needed.


Release/Fix unread count when moving file between nodes

Purpose:
When moving a file to a different component, the node field for all of its comments get updated to the destination node. This updated the date_modified field, which resulted in all of the comments appearing as if they were unread.
Requires: https://github.com/CenterForOpenScience/osf.io/pull/5254 (removes the auto-update of date_modified)
Changes
Update the user.comments_viewed_timestamp so that user.comments_viewed_timestamp[destination_node._id]['files'][root_target_id] == user.comments_viewed_timestamp[source_node._id]['files'][root_target_id]

Pass finished. Looks good to merge when the build passes. :octopus:
As discussed in person, this change will be unnecessary if we change comments_viewed_timestamp to be keyed on GUID rather than node ID. Saman will send a follow-up PR implementing this refactor.

The follow-up PR is #5268

Replaced by #5268


storage: Improved block location API

The Loose variant of BlockLocation can carry the hash that was sought.
This removes a redundant, error-prone parameter of block_read_location.
Free functions in crate storage that have Storage as the first
parameter are replaced with methods on Storage. The existing functions
are marked deprecated:

block_location
block_read_location
block_read

BlockLocation is also deprecated in favor of BlockLocation2, which will
be renamed back to BlockLocation in the next breaking release.

we don't need to assure compatibility just yet, BlockLocation2 is not really needed at this stage. we can just change BlockLocation directly for now.


Refactor docker compose arm64 config

Was not able to verify the test run.
The Chromium console shows the following error and the test runs into a timeout

Firefox shows a different error:


how Can I use german language ?

If I want a Rasa bot in German language then I need to define all the domains, entities, text response in actions in German? or it can be works if I define in English language & chat in German language?

No, you need to create a separate bot with German training data, domain etc.

Ok thank you


Allow header values to be selected from selector

It would be very useful to be able to select header values within chains.
E.g. (taking the example from the docs)
chains:
  username:
    source: !file
      path: ./username.txt
  password:
    source: !file
      path: ./password.txt
  auth_token:
    source: !request
      recipe: login
      # selector: $.token instead of JSONPath, there should be some method of obtaining header values.
      selector: headers.token

requests:
  # This returns a response like {"token": "abc123"}
  login: !request
    method: POST
    url: "https://myfishes.fish/login"
    body: |
      {
        "username": "{{chains.username}}",
        "password": "{{chains.password}}"
      }

  get_user: !request
    method: GET
    url: "https://myfishes.fish/current-user"
    authentication: !bearer "{{chains.auth_token}}"

Either enable selector to be able to access header values or something akin to header_selector which allows for this to happen.

My preference for design on this would be to add an additional field to the !request chain source type. I'll using component as a placeholder here, I don't love that but it essentially means "which part of the response do you want to pull from". This should be an enum (i.e. there's a fixed set of values), so it's represented by a YAML tag rather than a plain string. It'll default to !body to keep current behavior, but you could override it to !headers (or potentially other values in the future, like !cookies). An important decision is whether headers is implicitly converted to a JSON object, or whether you have to specify a specific header that you want to pull from, then operate on that value. For these examples, let's assume the response headers look something like:
Content-Type: application/json
token: asdf

Treat header as an object
chains:
  auth_token:
    source: !request
      recipe: login
      component: !headers
    selector: $.token

Pros

Allows you to operate on multiple headers
Easy to access all or specific values of a repeated header (e.g. Set-Cookie)

Cons

Feels untuitive - you're implicitly building a JSON object. It's very rare (in fact I don't think I've ever seen it) to have a JSON value in a header, so you're basically wrapping raw strings in a JSON object which is odd
Implementation would be kinda grody, which I think is indicative of a clumsy design

Specify header and treat it as a string
chains:
  auth_token:
    source: !request
      recipe: login
      component: !headers token

Pros

Simpler, more clear
Doesn't require converting to JSON

Cons

Can't access multiple headers in the same chain

I don't think this is a huge deal, typically headers don't need to interact with each other

No means for accessing multiple values of a header (if the header is repeated)

Implementation
For either design, the work involved would be roughly:

Add the new field to the Request variant of ChainSource: models.rs

I want to emphasize that the name component is a placeholder, I'm not sure of a good name for it
This would need a new enum to define the possible values, to begin with just Body and Headers

Implement the filtering in render.rs
Add tests in template.rs
Add docs in chain_source.md

Other Thoughts
I'm not sure if it was you @Sammo98 or someone else who commented about this on Reddit, but I believe the original use case was to be able to operate on cookies. Using this design you could get at a returned cookie response, by nesting this chain inside a shell command that parses the cookie. That would be very clunky though so I 100% would like to have first class cookie support in Slumber, I just don't think it needs to be an explicit goal of this ticket.

@LucasPickering Thanks for the response, it was me on Reddit as well, probably should have given the context of wanting to grab the cookie.
Nonetheless I can see this as a useful extension to slumber for certain systems that may want to store headers. Take for instance grabbing a CSRF token (I may be wrong about this being in the headers, maybe first class cookie support would solve this).
Whatever the case, thank you for the pointers, I'm happy to take a crack at this myself if that works?

@Sammo98 Yeah go for it. And CSRF is typically sent via the X-Csrf-Token header, so this would handle that

@LucasPickering
Linked an initial basic attempt at this, it is working but still need to add tests, will try and get to that later. Please let me know if any obvious issues.
(Note, this would be of my first OSS contributions so apologies if something looks awry!)


[iOS] Accessibility: provide a "allowsDirectInteraction" property to Semantics

I'm currently writing a music application in Flutter. My app shows drum pads on the screen which should react directly, also when voice over is enabled. On iOS I use the following properties on an UIView to achieve that:
self.accessibilityElementsHidden = NO;
self.accessibilityViewIsModal = NO;
self.isAccessibilityElement = YES;
self.accessibilityTraits = UIAccessibilityTraitAllowsDirectInteraction;

In Flutter, I did not find any possibility to configure Semantics to achieve the same effect. Please provide an allowsDirectInteraction property on Semantics that realizes the same effect.
Here is a related issue:
https://github.com/flutter/flutter/issues/3814
Thanks, Gabriel

/cc @chunhtai

can you provide an example app for your use case? based on the use case, I may be able to find a way to derive the trait based on semantics node without adding a new field in semantics property

Hi @chunhtai. Here you find a simple flutter app demonstrating a "Drum Pad" that changes its color on pointer down and pointer up:
https://github.com/gatzsche/flutter_semantics_allows_direct_interaction_example
Particularly main.dart demonstrates what is needed: A flag that tells the system to make the "Drum Pad" behaving the same way no matter if VoiceOver or TalkBack is enabled or not.

Note that upstream TalkBack does not offer this possibility on Android.   Because of this, all third party Braille apps or Piano apps need to have TalkBack manually  toggled off by users.   I submitted a ticket for TalkBack like 10 years ago, but it never went anywhere.   It still would be nice to have it in TalkBack...
But while waiting for Flutter to fix this for VoiceOver, where should I look to find the UIView that I could set UIAccessibilityTraitAllowsDirectInteraction on from a ffi plugin, if I have the Flutter Stack widget that should get the treatment?

As this is pretty silent topic, I wonder if I'm correct in assuming that it would be very hard to fix this in Flutter even just for iOS?    I would be happy to fork flutter for this, if the fix would be relatively small.  But after very short incursion into flutter sources, it seems that skia/impaler/uiview interaction means there is no clear uiview that should have the allowsDirectInteraction set.  I'm happy to be proved wrong though.
In the meanwhile, I'll investigate if a platform view could be used, although I'm not looking forward to that either, meaning ~1000 lines conversion from flutter/Dart to Swift and later maintenance for me.


Not working with guard, and unclear documentation.

First of all the Readme.md is confusing on how normal operation is expected to work.  The usage section starts with "You could also use the notifier directly." Also?  Also as apposed to what?  You can't start by saying 'also', you have to first state what is the normal usage, then you can say "also" to add additional usage instructions.
On top of this the examples don't mean anything.  What is the context in which TerminalNotifier::Guard.notify('Hello World') should appear?  Not on the command line, not in irb, not in the guard file.  Where?  Did you forget to specify a requires line, is this a rake task?
So the question is:
Is it supposed to 'just work'(tm)?  If that is the case then I have a bug in functionality to report.
or
Am I supposed to do something to my guard file to get it to work?  In that case I have a documentation bug to report.
Here are the steps to reproduce the issue I'm having.
Mac OS X 10.10.2 in terminal.

> brew install terminal-notifier

> terminal-notifier
terminal-notifier (1.6.2) is a command-line tool to send OS X User Notifications.
...

Added  gem 'terminal-notifier-guard', '~> 1.6.1' to Gemfile.

> bundle install
Fetching gem metadata from https://rubygems.org/...........
Resolving dependencies...
...
Installing terminal-notifier-guard 1.6.1
...

bundle exec guard
Guard file looks like:
guard :shell, first_match: true do
  watch(/(.*)(?<!\.nn|\.y).go$/) {|m| `make` }
  watch(/(.*).nn$/) {|m| `make` }
  watch(/(.*).y$/) {|m| `make` }
end

I get expected / typical guard output, but no notification appears.

However, if I run > terminal-notifier -message "test" I get a notification as expected.

Am I missing a step or is there a bug?
I've tried to read through various issues(*) but it quickly turns into a game of portal as each issue refers to other which refer to still other issues which then refer to pull requests which then refer back to the starting issue.  It is un-clear which, if any, of these should be followed.  I tried a couple of suggestions, and a PR, but none of those changed the behavior described above.
*Issues
#22
#13
#12
#15
guard/guard#626
alloy/terminal-notifier#106
...

Looks like adding this to Guardfile does the magic:
notification :terminal_notifier
Seems like a documentation bug.

@mjtko didn't help me.

@NARKOZ does terminal-notifier -message "test" work?
Adding that notification line to my Guardfile was the only other thing I needed to do (other than brew install terminal-notifier && brew linkapps terminal-notifier)

@mjtko no. It freezes and I need to kill it via Ctrl+C
$ which terminal-notifier
/usr/local/bin/terminal-notifier

Ah - need to make that work first then!  Perhaps this will help? https://github.com/alloy/terminal-notifier/issues/115

I use zsh and rvm. Guard does not touch terminal-notifier when I start it without bundle exec. But it works fine if I start it with bundle exec guard.

This is for anyone else who runs into this issue. I needed to add a require 'terminal-notifier-guard' — I was using underscores earlier. Looks like the require uses hyphens whereas the gem uses underscores.
With the require statement corrected, I am able to display OS X notifications in irb as well as in the Guardfile

I ran into this issue as well. As @debajit stated, adding require 'terminal-notifier-guard' to the top of my Guardfile resolved my issues.

I ran into this issue as well. It turns out things were failing for me because of tmux. Here is a fix for using terminal-notifier-guard with tmux.

Thanks all for reporting. I updated the README with some extra clarification on these and other matters.
Cheers!


PAC4J exceptions doesn't trigger spring security mechanism

Hello,
I have a problem using your lib. In the ClientAuthenticationFilter, client.getCredentials(context) throws pac4j exceptions and spring security's cannot handle them properly.
The exceptions have to be AuthenticationException in order to be catch in the AbstractAuthenticationProcessingFilter.

Yes, it makes sense. I will review your pull request. Thanks for submitting it.
I plan to upgrade the spring-security-pac4j library very soon with the new pac4j v1.8 and cut a 1.4.0 release. This fix would got with it.

BTW, how did you find ou this problem?

I'm using google oauth and a CredentialsException is thrown when the java session timeout on the account choice screen. This screen appear when you are logged to multiple google accounts. You can reproduce the problem by removing your cookies (JSESSIONID) before selecting an account.
I didn't find any clean way to catch this exception so I start looking in spring-security authentication failure mechanisms.

OK. I see.


Latest updates to descriptors

Brief description of the proposed changes:
Check reserved bits are set according to the relevant standard
Update and improve output formatting
Include a common routine to tabulate hexadecimal values (tsTabluateVector)

Hi Paul,
Just one question about the TabulateVector functions. Is there a reason why you did not use one of the UString::Dump() static methods? They are quite versatile and used everywhere hexa/ascii/binary dump is required.
Moreover, in the DisplayDescriptor methods, you can also use specialised methods from the TablesDisplay instance: displayPrivateData() or displayExtraData(). They call UString::Dump() to format the dump but they add some encapsulation for presenting descriptor or section data.
Before merging this one, I would like to make sure that it does not introduce redundant code in the main library.

I did not look deeply into the UString::Dump() mostly because I knew what I needed to to and that it was already done that way in several of my descriptors. In fact TabulateVector mostly just takes the existing code and moves it, but it does also add the vector<boolean> option.
I had seen UString::Dump being used for private data and thought that I did not need all the line and byte numbering provided there.
For me, tsTabulateVector() seems to be more useful for a set of values after the label, i.e....
      LCEVC stream tag:  01 23 45 17 18 19 20 21
                         22 23 24 25 25 26 27 28
                         28 30

I could look into using UString::Dump() if you prefer!

I could look into using UString::Dump() if you prefer!

Also have a look at TablesDisplay::displayPrivateData(). It is used in many descriptors to display stuff similar to the "LCEVC stream tag" you mention.
For the sake of code consistency, I would prefer to keep a common set of tools to handle the signalisation.
Let me know what is missing, such as vector<boolean>, and I will add them.

Or, alternatively, since your new functions are bound to TablesDisplay, I would prefer to move them as methods of TablesDisplay, under the name displayVector(), as specialised companions to displayPrivateData(). This seems more consistent than having independent functions.
This is maybe the most appropriate compromise.

I am happy to change from TabulateVector() ..
    - Descriptor 2: MPEG-2 Extension (0x3F, 63), 46 bytes
      MPEG extended descriptor: LCEVC linkage (0x18, 24)
      LCEVC stream tag:  01 23 45 17 18 19 20 21
                         22 23 24 25 25 26 27 28
                         28 30 41 42 43 45 46 46
                         46 47 48 49 45 51 52 53
                         54 54 56 45 45 99 98 97
                         96 95 94 93

to UString::Dump() ..
    - Descriptor 2: MPEG-2 Extension (0x3F, 63), 46 bytes
      MPEG extended descriptor: LCEVC linkage (0x18, 24)
      LCEVC stream tag: 01 23 45 17 18 19 20 21 22 23 24 25 25 26 27 28 28 30 41 42 43 45 46 46 46 47
48 49 45 51 52 53 54 54 56 45 45 99 98 97 96 95 94 93

For the boolean output, I was looking for something like this....
channel map: X..X...X.XXX.XX...XX
             ....X..XX...X..X..X.
             XXX...X.XXX..X..XX.X

Or, alternatively, since your new functions are bound to TablesDisplay, I would prefer to move them as methods of TablesDisplay, under the name displayVector(), as specialised companions to displayPrivateData(). This seems more consistent than having independent functions.
That would be a good solution

That would be a good solution

OK, we have a deal :) I let you update the code and the PR accordingly.


Add computations of kinematic regressors

Thanks @jcarpent for adding such a quite cool feature! Do you happen to have any reference or pointer for the derivation of this regressor?

@traversaro. I developed the derivatives on my side. If you think it would be worth to write a document/paper on it, we may proceed.

@traversaro. I developed the derivatives on my side. If you think it would be worth to write a document/paper on it, we may proceed.

I have no need to look into that at the moment to be honest, do not worry. Mine was just a curiosity, as most kinematic calibration algorithm that I know typically write the geometric model in some form with a reduced number of parameters (such as Denavit Hartenberg, see for example Section 6.2 https://doi.org/10.1007/978-3-319-32552-1_6 ), while in your case if I got correctly you model the changes that each transform can experience as a 6D left-trivialized derivative (see https://github.com/stack-of-tasks/pinocchio/pull/1332/files#diff-373a0e4d5f39761421b0412c3e3a0ce767985fb27c1482f64375c801d97f63f0R68), that is something I never saw.

Yes, I do agree. DH seems the most used conventions for robot
callibration. But here, working directly on SE(3) seems more convenient
I would say and I think efficient/generic.
Best,
Justin
On 02/11/2020 16:21, Silvio Traversaro wrote:

@traversaro <https://github.com/traversaro>. I developed the
derivatives on my side. If you think it would be worth to write a
document/paper on it, we may proceed.

I have no need to look into that at the moment to be honest, do not
worry. Mine was just a curiosity, as most kinematic calibration
algorithm that I know typically write the geometric model in some form
with a reduced number of parameters (such as Denavit Hartenberg, see
for example Section 6.2 https://doi.org/10.1007/978-3-319-32552-1_6 ),
while in your case if I got correctly you model the changes that each
transform can experience as a 6D left-trivialized derivative (see
https://github.com/stack-of-tasks/pinocchio/pull/1332/files#diff-373a0e4d5f39761421b0412c3e3a0ce767985fb27c1482f64375c801d97f63f0R68),
that is something I never saw.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stack-of-tasks/pinocchio/pull/1332#issuecomment-720537249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL6UVI4M3Y4OE2BH4ZOEUDSN3E7PANCNFSM4TGV2I6Q.

@traversaro. I developed the derivatives on my side. If you think it would be worth to write a document/paper on it, we may proceed.

I have no need to look into that at the moment to be honest, do not worry. Mine was just a curiosity, as most kinematic calibration algorithm that I know typically write the geometric model in some form with a reduced number of parameters (such as Denavit Hartenberg, see for example Section 6.2 https://doi.org/10.1007/978-3-319-32552-1_6 ), while in your case if I got correctly you model the changes that each transform can experience as a 6D left-trivialized derivative (see https://github.com/stack-of-tasks/pinocchio/pull/1332/files#diff-373a0e4d5f39761421b0412c3e3a0ce767985fb27c1482f64375c801d97f63f0R68), that is something I never saw.

By the way, does pinocchio rely on the Denavit-Hartenberg standard to parse the robots URDF and compute the Jacobian matrix?


[Key Vault] Support API version 7.5-preview.1

Description
This adds support for the newest preview API version in each library, and also makes small API changes to Administration and Keys. The former library now has Managed Identity support for backup operations, and the latter has a new property on keys.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-keyvault-administration

Is there any way to separate the refactoring from the 7.5-preview.1 support? It's very hard to review this way. Can't tell what is new vs. what is refactoring.

Is there any way to separate the refactoring from the 7.5-preview.1 support? It's very hard to review this way. Can't tell what is new vs. what is refactoring.

@heaths I could try to split things up if necessary, but the relevant changes only come from two commits:

hsm_platform support
use_managed_identity support in backup/restore operations

If it's not too inconvenient, would reviewing based on those commits be possible? If not, I can try to split things into a regeneration PR and functionality PR.

Is there any way to separate the refactoring from the 7.5-preview.1 support? It's very hard to review this way. Can't tell what is new vs. what is refactoring.

@heaths I could try to split things up if necessary, but the relevant changes only come from two commits:

hsm_platform support
use_managed_identity support in backup/restore operations

If it's not too inconvenient, would reviewing based on those commits be possible? If not, I can try to split things into a regeneration PR and functionality PR.

I gave my approval - though I had a couple nits on one commit - based only on these commits. I'll leave it to @annatisch and/or @lmazuel or whoever about the other "refactorings".


Missing dropdown of Foreign Account and Own Account fields

All the other fields' dropdown lists work just fine. On the Foreign and Own account they just never appear, though they did a couple of days ago. I've deleted the app's cache and data, logged out and then logged in again, and even deleted the app, reinstalled and restarted my phone. None worked, what could be causing the issue here?
Phone is Samsung Galaxy S20 FE, running One UI 5.1 on Android 13.

Thank you it now works. Sorry I should have searched better for it!


Device Timeout Waiting Response

Hi everyone and thanks in advance for taking the time to read my issue.
I am developing a little VB.NET program that connects to an ANVIZ VF30 Pro in order to save all the user record data and integrate it with an ERP.
I have been doing many tests: saving users, deleting them, enroling finger prints, getting data, etc. Everything was going perfectly until today, that suddenly, the Connect function is giving me the "Device timeout waiting response" error.
The device is correctly configured in the LAN and there is a succesfull Ping from my PC's command window.
I am a little bit frustrated since there is nothing that I have changed since the last time I made some tests and everything was working correctly.
I will leave my piece of code, but it is so simple that I doubt it makes any difference:
        Dim L_Manager As AnvizManager
        L_Manager = New AnvizManager
        '-------------------------------------------------------------------------------------------
        L_Manager.ConnectionUser = "0"
        L_Manager.ConnectionPassword = "12345"
        '-------------------------------------------------------------------------------------------
        Dim L_Device As AnvizDevice
        '-------------------------------------------------------------------------------------------
        Try
            L_Device = Await L_Manager.Connect("<IP_ADDRESS>")
        Catch ex As Exception
            MsgBox(ex.Message)
        End Try

Any help will be appreciated, thank you.

Hi,
If you have set a device password for the communication protocol, you must also set L_Device.AuthenticateConnection to true.
It could also be that another device on your network is stealing its IP address.
Does the official software work?

Hi! Thanks for your reply!
I kept trying different alternatives and I think that the problem was that the Wi-Fi mode was active, instead of only Ethernet, I'm not totally sure about this though.
Regarding to the line you mention, L_Device.AuthenticateConnection = True, it seems that in my case the connection is only working if I don't add this line to my code.
I'm not really sure if any of my reasons are true or if it was just a matter of bad luck, but apparently the connection is working again, so I will take that!
Thanks again for the fast response!


modified send to require a connection be passed in to enable connecti…

…on reuse.

Definitely.  Bad push because I have that locally.  Will correct
On Feb 13, 2017 8:20 AM, "Clay Austin"<EMAIL_ADDRESS>wrote:

@caustin-usgs commented on this pull request.
In cw/init.py
https://github.com/USGS-EROS/lcmap-change-worker/pull/3#pullrequestreview-21516928
:

@@ -40,7 +40,14 @@

def send(cfg, message):

return messaging.send(cfg, message)

conn = None
try:

   conn = messaging.open_connection()

   return messaging.send(cfg, message)

based on your mods to messaging.send, dont you need to pass the newly
created connection object (conn) as the 3rd arg?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/USGS-EROS/lcmap-change-worker/pull/3#pullrequestreview-21516928,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALl2xK4b4eKzpOCx06ljnYR36_e-7d4fks5rcGa4gaJpZM4L-S6T
.