Front End Certificate Will Not Generate

I completed the items marked with a * located before the front end certificate, but then when I completed the "Claim Your Front End Development Certificate" challenge, it did not create a certificate even though it said that it was successful. Am I missing something?

Interesting.  We'll look into it.
@BerkeleyTrue - got another one here.

I tried reproducing this issue...ii could complete the front end certification challenge even though the bonfire and zip lines are not completed. And the certificate does not appear ..But the challenge should be completed before the previous challenges are done

I ran through the challenge again today and it produced a certificate so this can be closed. The challenge looked slightly different (word Sample on certificate of step one and different verbiage in instructions) so it may have been an update the fixed the issue.


Upgrade default tool version of PMD

Update toolVersion to latest available version which is compatible with Java 8: 6.7.0.
PMD 6.0 has deprecated the default rule set the plugin has been using: java-basic. It can still be used, but will print a big block of deprecation warnings such as:
> Task :pmdMain
Use Rule name category/java/errorprone.xml/AvoidBranchingStatementAsLastInLoop instead of the deprecated Rule name rulesets/java/basic.xml/AvoidBranchingStatementAsLastInLoop. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/AvoidDecimalLiteralsInBigDecimalConstructor instead of the deprecated Rule name rulesets/java/basic.xml/AvoidDecimalLiteralsInBigDecimalConstructor. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/AvoidMultipleUnaryOperators instead of the deprecated Rule name rulesets/java/basic.xml/AvoidMultipleUnaryOperators. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/AvoidUsingOctalValues instead of the deprecated Rule name rulesets/java/basic.xml/AvoidUsingOctalValues. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/BrokenNullCheck instead of the deprecated Rule name rulesets/java/basic.xml/BrokenNullCheck. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/CheckSkipResult instead of the deprecated Rule name rulesets/java/basic.xml/CheckSkipResult. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/ClassCastExceptionWithToArray instead of the deprecated Rule name rulesets/java/basic.xml/ClassCastExceptionWithToArray. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/DontUseFloatTypeForLoopIndices instead of the deprecated Rule name rulesets/java/basic.xml/DontUseFloatTypeForLoopIndices. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/JumbledIncrementer instead of the deprecated Rule name rulesets/java/basic.xml/JumbledIncrementer. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/MisplacedNullCheck instead of the deprecated Rule name rulesets/java/basic.xml/MisplacedNullCheck. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/OverrideBothEqualsAndHashcode instead of the deprecated Rule name rulesets/java/basic.xml/OverrideBothEqualsAndHashcode. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/ReturnFromFinallyBlock instead of the deprecated Rule name rulesets/java/basic.xml/ReturnFromFinallyBlock. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/errorprone.xml/UnconditionalIfStatement instead of the deprecated Rule name rulesets/java/basic.xml/UnconditionalIfStatement. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/multithreading.xml/AvoidThreadGroup instead of the deprecated Rule name rulesets/java/basic.xml/AvoidThreadGroup. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/multithreading.xml/DontCallThreadRun instead of the deprecated Rule name rulesets/java/basic.xml/DontCallThreadRun. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/multithreading.xml/DoubleCheckedLocking instead of the deprecated Rule name rulesets/java/basic.xml/DoubleCheckedLocking. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/bestpractices.xml/AvoidUsingHardCodedIP instead of the deprecated Rule name rulesets/java/basic.xml/AvoidUsingHardCodedIP. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/bestpractices.xml/CheckResultSet instead of the deprecated Rule name rulesets/java/basic.xml/CheckResultSet. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/codestyle.xml/ExtendsObject instead of the deprecated Rule name rulesets/java/basic.xml/ExtendsObject. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/codestyle.xml/ForLoopShouldBeWhileLoop instead of the deprecated Rule name rulesets/java/basic.xml/ForLoopShouldBeWhileLoop. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/performance.xml/BigIntegerInstantiation instead of the deprecated Rule name rulesets/java/basic.xml/BigIntegerInstantiation. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/performance.xml/BooleanInstantiation instead of the deprecated Rule name rulesets/java/basic.xml/BooleanInstantiation. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/design.xml/CollapsibleIfStatements instead of the deprecated Rule name rulesets/java/basic.xml/CollapsibleIfStatements. PMD 7.0.0 will remove support for this deprecated Rule name usage.
Use Rule name category/java/design.xml/SimplifiedTernary instead of the deprecated Rule name rulesets/java/basic.xml/SimplifiedTernary. PMD 7.0.0 will remove support for this deprecated Rule name usage.

Thus, in order to avoid printing these warnings when a user uses the plugin's default configuration, we have to pick one of the newly introduced categories:

Best Practices
Code Style
Design
Documentation
Error Prone
Multithreading
Performance

There is no category that contains all rules from java-basic. 13 of the 24 rules are contained in "Error Prone", though.

@adangel @oowekyala @jsotuyod As PMD maintainers, do you have any advice for us here?

@marcphilipp that's right. The best practice is for developers to write their own rulesets and tailor which rules are right for them.
We are in the process of defining some quickstart rulesets for different kind of projects, that developers can use as starting points for their own rulesets, but they are still unshipped.
In the meantime, if you want something that works out of the box you have 2 alternatives:

bundle a custom ruleset with Gradle (which may include the rules from the old java-basic ruleset if you want) and reference that by default
default to a category that is broad enough / basic enough to not be polemic.. possible bestpractrices or errorprone are the best fit.

If you can, please contact me by email (my email is public) so we can discuss this in detail / better collaborate to get the Gradle integration up to speed with all the PMD features not currently supported:

CPD
Support for Java 9, 10 and 11
Support for other languages Gradle supports. ie: PMD can also apply rules to xml files, and CPD is supported not only forJava, but among languages Gradle supports it can also work on:

Groovy
Scala
C
C++
Objective-C

Incremental analysis

@jsotuyod Thanks for the feedback!
The quickstart rule sets sound exactly like what we would need. Any idea when those will be shipped?
I don't think we should be bundling our own ruleset, because it needs to work across all supported versions of PMD (4.x - 6.x). The approach I'd take is to use a category by default (e.g. errorprone), and fall back to java-basic for PMD 5.x and basic for 4.x.
Regarding adding missing features to the Gradle integration, I'm afraid we don't have a lot of time to work on it at the moment. Would you or someone else from the PMD team/community be willing to take on that work? If so, there are two possible ways: (1) contribute to Gradle by submitting PRs, (2) fork/create a new community plugin and eventually deprecate the plugin in Gradle core.

Support for Java 9, 10 and 11

The update to PMD 6.7.0 should be sufficient for that, right?

@marcphilipp ok, that sounds good.
Yes, we are willing to contribute to Gradle, as we have done in the past. However, the last time I tried to bump PMD and support the analysis cache, the PRs got stalled for months and then reverted before release. So I'd really appreciate being able to talk to someone from Gradle and coordinate the work to avoid wasting efforts.
As for Java 9, 10 and 11... Yes, PMD 6.7.0 supports those, but you will also need to update Gradle's TargetJdk

The TargetJdk is currently only used for PMD < 5.x:
https://github.com/gradle/gradle/blob/2622baaf86d959685f94c947302b5fb1b26f4ac5/subprojects/code-quality/src/main/groovy/org/gradle/api/plugins/quality/internal/PmdInvoker.groovy#L46-L49
Do we have to specify language/version for PMD 6.x to work with Java 9/10/11 or will it detect that on its own?

It would be totally awesome if there were CPD support built into Gradle. I do want to make sure it supports language plugins. For example, for Kotlin. I'm planning on publishing this plugin when I get a chance: https://github.com/JLLeitschuh/pmd-kotlin

PMD will always default to the latest supported Java version, which may not be actually the version used by the Gradle project. This may result in having rules enabled / disabled which shouldn't be for the project.
That targetJdk attribute was removed back in PMD 5, as PMD moved from a Java-only tool to a multilanguage tool. The way to do that properly is using a child node in the task:
<sourceLanguage name="java" version="1.8"/>

Supported versions are:
        addVersion("1.3", new JavaLanguageHandler(3), false);
        addVersion("1.4", new JavaLanguageHandler(4), false);
        addVersion("1.5", new JavaLanguageHandler(5), false);
        addVersion("1.6", new JavaLanguageHandler(6), false);
        addVersion("1.7", new JavaLanguageHandler(7), false);
        addVersion("1.8", new JavaLanguageHandler(8), false);
        addVersion("9", new JavaLanguageHandler(9), false);
        addVersion("10", new JavaLanguageHandler(10), false);
        addVersion("11", new JavaLanguageHandler(11), true);

https://github.com/pmd/pmd/blob/e3018c3892bb34805efff9d9a2e4c6acde682887/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/JavaLanguageModule.java#L20-L28

I have a initialized a default value for the variable to avoid NPE and overriding the value on certain if condition.  But pmd check throws a warning " 'DD'-anomaly for variable 'x' (lines '16'-'18') ".
Ex:
List<String> x = new ArrayList<>();
if(  condition ) {
  x = <some value retrieved from different object>;
}

This is more similar to original problem.
I can remove the "DD anomaly" warning by following code:
List<String> x;
if(condition) {
  x = <some value from object>;
} else {
 x = new ArrayList<>();
}

I simply initialized the default value ahead instead of defining in else part.
Should I open different ticket or can it be reopened?
Most of the DataFlow anomaly are reporting false warning.

@sudharmohan This isn't the place to report issues/questions about PMD. Please refer to the README for the PMD project to ask question:
https://github.com/pmd/pmd


Can't get Basic Auth to work

I'm having some troubles with the authentication layer. No matter what I try, I can not use the laravel 4.2 basic auth functionality.
I protected my route in the constructor and this is the response after making a post request with postman:
{"message":"Invalid authentication credentials.","status_code":401}
I'm using 0.8.2

Can you login using the same credentials with a simple Auth::attempt somewhere to make sure that you can successfully authenticate.

I can successfully authenticate using Auth::attempt().
If I try to access a protected route and if I try to access the corresponding URI from the browser it won't even try to authenticate, it will just fail whereas laravel's filter does ask for a password

Hi @Ortix92,
I experienced the same thing. How does your webserver configuration look like? It might be that the authorization headers are not sent. I'm using Apache and had to add the following for this to work
RewriteCond %{HTTP:Authorization} ^(.*)
RewriteRule .* - [e=HTTP_AUTHORIZATION:%1]

I'm using nginx and php-fpm. The auth headers are sent when using the regular laravel basic auth. But as soon as dingo comes into play it breaks down. Weird..

How do your routes look like?

Hi, I have the same problem, i am using dingo with jwt-auth. Always got:
{"message":"Failed to authenticate because of bad credentials or an invalid authorization header.","status_code":401}

@ruanyl This might be the same issue as yours tymondesigns/jwt-auth#16

@tymondesigns Yeah! I got it solved :smile:  Thank you for the information!

Hi @Ortix92,
You might want to add the following to your .htaccess file:
RewriteCond %{HTTP:Authorization} ^(.)
RewriteRule . - [e=HTTP_AUTHORIZATION:%1]
As mentioned also by everybody here, the http headers are not set and the above will correct that.
Sacha

Sounds like this is resolved? I shall close it then. Cheers to all who helped.

It turned out to be this line in the config:
    'auth' => [
        'basic' => function ($app) {
            return new Dingo\Api\Auth\BasicProvider($app['auth'], 'username'); // second parameter defaults to email.. i did not know that
        },
    ],

I had no idea it was defaulting on email..

Good to hear @Ortix92! I remember now I also changed that setting :)

'auth' => [
'jwt' => 'Dingo\Api\Auth\Provider\JWT'
],
result

It should add below to ***.conf
<VirtualHost *:80>
   ...
   ...
    # Fix authentication headers
    RewriteEngine On
    RewriteCond %{HTTP:Authorization} ^(.*)
    RewriteRule .* - [e=HTTP_AUTHORIZATION:%1]

    ...
</VirtualHost>

But add this to .htaccess don't work

I did all ways that described but none of them does not work and when sent requests to protected endpoint 404 Not Found error occures

add this to your config/api.php
'auth' => [
        'jwt' => 'Dingo\Api\Auth\Provider\JWT'
    ],


Change timezone and admin for task info

Can one of the admins verify this patch?

Current coverage is 94.51% (diff: 72.22%)

Merging #46 into master will decrease coverage by 0.40%

@@             master        #46   diff @@
==========================================
  Files            31         31
  Lines           985       1003    +18
  Methods           0          0
  Messages          0          0
  Branches         42         43     +1
==========================================
+ Hits            935        948    +13
- Misses           42         47     +5
  Partials          8          8

Powered by Codecov. Last update bd46f79...31428c6


CLI: add get-registration cmd

As in https://github.com/encointer/encointer-worker/blob/3be977ba4d8de76ac724f69c7928e67f9dd375d3/stf/src/cli.rs#L377

The participant index type can be get from trustedGetter:ceremonies_aggregated_account_data


withBarValueLabel value not displaying in correct positions

This issue was addressed in issue #363, but still not working properly:

So, I'll cross-post the issue in mantine to resolve the issue with them.

Posted in @Mantine/mantine#7156

Fixed by #7160.
They added a new prop valueLabelProps.


Issue with setting up Hough_voting_cuda

Hi Chris,
I am using uois2d branch and I followed the installation steps and I am getting an error when running "python setup.py install"
Environment is activated
Using Amazon Linux sagemaker p3.8 instance
The error I'm getting is this :
(uois) [root@ip-172-16-48-121 hough_voting]# python setup.py install
No CUDA runtime is found, using CUDA_HOME='/usr/local/cuda'
running install
running bdist_egg
running egg_info
writing hough_voting.egg-info/PKG-INFO
writing dependency_links to hough_voting.egg-info/dependency_links.txt
writing top-level names to hough_voting.egg-info/top_level.txt
/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/torch/utils/cpp_extension.py:381: UserWarning: Attempted to use ninja as the BuildExtension backend but we could not find ninja.. Falling back to using the slow distutils backend.
  warnings.warn(msg.format('we could not find ninja.'))
reading manifest file 'hough_voting.egg-info/SOURCES.txt'
writing manifest file 'hough_voting.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_ext
Traceback (most recent call last):
  File "setup.py", line 15, in <module>
    'build_ext': BuildExtension
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/setuptools/__init__.py", line 153, in setup
    return distutils.core.setup(**attrs)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/dist.py", line 966, in run_commands
    self.run_command(cmd)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/dist.py", line 985, in run_command
    cmd_obj.run()
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/setuptools/command/install.py", line 67, in run
    self.do_egg_install()
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/setuptools/command/install.py", line 109, in do_egg_install
    self.run_command('bdist_egg')
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/dist.py", line 985, in run_command
    cmd_obj.run()
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/setuptools/command/bdist_egg.py", line 164, in run
    cmd = self.call_command('install_lib', warn_dir=0)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/setuptools/command/bdist_egg.py", line 150, in call_command
    self.run_command(cmdname)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/dist.py", line 985, in run_command
    cmd_obj.run()
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/setuptools/command/install_lib.py", line 11, in run
    self.build()
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/command/install_lib.py", line 107, in build
    self.run_command('build_ext')
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/dist.py", line 985, in run_command
    cmd_obj.run()
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/setuptools/command/build_ext.py", line 79, in run
    _build_ext.run(self)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/distutils/command/build_ext.py", line 340, in run
    self.build_extensions()
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 404, in build_extensions
    self._check_cuda_version()
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 777, in _check_cuda_version
    torch_cuda_version = packaging.version.parse(torch.version.cuda)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/pkg_resources/_vendor/packaging/version.py", line 56, in parse
    return Version(version)
  File "/home/ec2-user/anaconda3/envs/uois/lib/python3.7/site-packages/pkg_resources/_vendor/packaging/version.py", line 275, in __init__
    match = self._regex.search(version)
TypeError: expected string or bytes-like object
(uois) [root@ip-172-16-48-121 hough_voting]#

I have tried looking for answers but didn't find any. Hope you can help with this.

Hi,
I am not sure what the cause of this is. However, as mentioned in the README, one tip is to make sure that cudatoolkit in env.yml matches your native CUDA version (under /usr/local/cuda/). This was done because cudatoolkit as installed by anaconda did not come with the CUDA compiler. Nowadays, I believe it does,  so an alternative method is to configure it to use the compiler installed with cudatoolkit. Hopefully one of those two options should work.


Review subscription in case there are multiple functions to subscribe to

In case we need to subscribe from multiple functions at the same time, sometimes the functions get updated at a different pace, so we need to establish a proper subscription order to response to all changes

[ ] Balance service:

[ ] system.account pallet

[ ] Earning service:

[ ] Pool positions
[ ] Reward


change default connection timeout 1000ms

Motivation:
Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.
Modification:
Describe the idea and modifications you've done.
Result:
Fixes #https://github.com/sofastack/sofa-rpc/issues/776
If there is no issue then describe the changes introduced by this PR.

Codecov Report

Merging #777 into 5.5 will decrease coverage by 0.12%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                5.5     #777      +/-   ##
============================================
- Coverage     70.73%   70.61%   -0.13%
+ Complexity     1152     1147       -5
============================================
  Files           386      386
  Lines         16062    16058       -4
  Branches       2589     2584       -5
============================================
- Hits          11362    11339      -23
- Misses         3350     3369      +19
  Partials       1350     1350

Impacted Files
Coverage Δ
Complexity Δ

...java/com/alipay/sofa/rpc/module/LookoutModule.java
52.38% <0%> (-33.34%)
3% <0%> (-1%)

...ofa/rpc/registry/consul/internal/TtlScheduler.java
77.77% <0%> (-5.56%)
7% <0%> (-1%)

...fa/rpc/registry/consul/internal/ConsulManager.java
46% <0%> (-5%)
11% <0%> (-2%)

...alipay/sofa/rpc/api/context/RpcContextManager.java
47.82% <0%> (-4.35%)
4% <0%> (ø)

...fa/rpc/registry/consul/model/HeartbeatService.java
29.41% <0%> (-2.95%)
4% <0%> (-1%)

...m/alipay/sofa/rpc/codec/jackson/JacksonHelper.java
77.77% <0%> (-2.78%)
7% <0%> (-1%)

...ofa/rpc/registry/zk/ZookeeperOverrideObserver.java
39.34% <0%> (-1.64%)
0% <0%> (ø)

.../sofa/rpc/bootstrap/DefaultClientProxyInvoker.java
90.32% <0%> (-1.62%)
0% <0%> (ø)

.../java/com/alipay/sofa/rpc/ext/ExtensionLoader.java
79.6% <0%> (-0.5%)
0% <0%> (ø)

...om/alipay/sofa/rpc/metrics/lookout/RpcLookout.java
75.23% <0%> (-0.24%)
14% <0%> (ø)

... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d15f8a...8c9aba9. Read the comment docs.


Fixed spec for cowboy_req:reply/4

Hi Everybody,
I found that cowboy_req:reply/4 has invalid spec (seems like some typo). As it should support all allowed types of #http_req.resp_body, I added missed variants.
Without this change Dialyzer warns about mismatches like :
The call cowboy_req:reply(200,Req2::cowboy_req:req()) contains an opaque term as 2nd argument when terms of different types are expected in these positions

It's possible however there are no more planned 1.x releases so this will not be merged. Leaving open for a few days before closing to give a chance for other people to see this patch. Thanks!

I think we can close it now. Thanks!


[main] Upgrade to latest dependencies (#71)

upgrade to latest dependencies
bumping knative.dev/eventing ea81855...29ac3ee:

29ac3ee [main] Upgrade to latest dependencies (# 7170)
8e9ab2a [main] Update community files (# 7171)
742c45e Create new trigger rekt helper functions (# 7135)
1b1b690 Test ApiServerSource with Broker as sink (# 7151)
bumping knative.dev/pkg 26b41ba...4f651e0:
4f651e0 Bump gomodules.xyz/jsonpatch/v2 from 2.2.0 to 2.3.0 (# 2789)
6519aff Update community files (# 2794)
bumping gomodules.xyz/jsonpatch/v2 93a4daf...13a9e4a:
13a9e4a Merge pull request # 36 from howardjohn/v2/improvements
62e0c90 Update repository config (# 33)
6d5c3df Fully remove dead code
0c26413 Update repository config (# 32)
45186ed (hackily) remove inefficient optimization
e0f6c24 Fix known broken case
c616a48 Fix marshaling invalid chars
2c05a51 Fix inputs with empty keys
1f7eb13 Improve fuzz tests
471fa30 Fix CI
4844bbf Add fuzzing test
87f8876 add benchmarks
a81e531 Go.mod for go 1.20
fc9e957 Update to Go 1.20
bumping knative.dev/hack 3403e35...044c02b:
044c02b Update community files (# 309)

Signed-off-by: Knative Automation<EMAIL_ADDRESS>

OCF Webhook is merging this PR


scrollbars not hearing mouseup

hey @marijnh,
dunno if this is just an issue with the scrollbar demo, but it looks like the mouseup handler isnt firing, so the scrollbar is stuck to the cursor after dragging ends. thought it was an issue of mouseup firing outside the bar itself; was gonna suggest delegation to cm's wrapper or even document, but it looks like even when mouseup is inside the scrollbar, it doesnt fire.
EDIT: tested in FF and IE11

There actually was no mouseup handler at all, since Webkit browsers set a sane which property on mousemove events that allows us to detect button release in a much more reliable way, but I guess other browsers don't. Fixed in attached patch.


Question, Area Chart - How set min and max scroll limit?

Subj,
plunker
So, when zoom max ( first and second frame on my attach ) i see spaces left and right sides, it`s okay?
But i hope view full filled area of chart

And i found bug,
try zoom to max and drag to left many time ( so about 10 times )

P.S. thank you for you library, it is best solution) like
Edited,
how about this ( Zoom to Domain ) sample ,
So, how set min and max zoom value?
similar
d3.zoom()
    .scaleExtent([1, Infinity])

So, i understand what zoom depended from plotData
https://github.com/rrag/react-stockcharts/blob/master/src/lib/ChartCanvas.jsx#L595

About bug, i find it and fix, wait the pull request from me, maybe tomorrow

the gap in the data is due to weekends, as market is closed on weekend there is no data to show. You should use a discontinuous scale to remove weekend breaks.

What about bug?

So, i need help please
how set max zoom in
how implement don`t disable render when pan enabled, i found you disabled render graph when pan started
how create custom indicator for only last point on chart?
how set min and max view domain range when pan, i want see max 50% domain width when pan left or right end of chart
how set animation for updating data component

These are different types of feature requests, kindly file a separate issue for each explaining with pictures if possible. I have questions on all of these, we can discuss them in those issues

how set max zoom in ( i like zoom as v 0.3.1 )
how implement don`t disable render when pan enabled
how set min and max view domain range when pan, i want see max 50% domain width when pan left or right end of chart
how set animation for updating data component


⚠ AGEG website (redirect) has degraded performance

In 1e80867, AGEG website (redirect) (https://ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 3160 ms

Resolved: AGEG website (redirect) performance has improved in 2d98bed.


Create MWE python to run inference

We need a python script that:

takes one set of inputs
runs inference on a given model (1x1/3x3/global)
saves / prints output

this is related to #31


Saving a temp file with file extension

Hi,
I'm attempting to process some image files with image. However, it infers the image type from the file extension (kinda annoying imo, but hey, makes sense). Is there any way to save files to a temp directory with a file extension? I noticed you can change the folder path, but what about the file name?
Thanks,
Nicholas

It's not a great idea to infer the file type based on extension for untrusted input anyway, honestly. I think image does verify its guess from the extension against the file header for the format so it's curious why it doesn't just do that instead. I guess extension + magic bytes for double verification or something.
Ideally you could pass an open file to load() and it would guess the format and decode the image but it doesn't do that either, you have to pass the format. Fortunately it does also provide guess_format() which you can link together into something like this:
let mut file = BufReader::new(File::open(multipart_file)?);
let format = image::guess_format(file.fill_buf()?);
let image = image::load(file, format)?;

Makes sense, thanks!


[PE-D][Tester E] Add command nil fields

You might want to make it such that fields that are omitted are treated as nil so increase the convenience for the users as they right now have to type a pretty long command for adding a fosterer.

Labels: type.FeatureFlaw severity.Medium
original: jonasongg/ped#14

Thank you for the feedback, we will write this in the planned enhancements.


cyan not found in chalk, Vite-Svelte and pnpm

Hi, so I added unplugin to a repo, and it broke everything for some reason. The error is this

I installed chalk to mitigate this issue, but it wouldn't go away, and hence, I can't turn on the dev server
Here's the repo https://github.com/PuruVJ/ms-todo-svelte
Thanks
Puru

chalk doesn't provide  esm, so we need to change its usage.

Or maybe this? Seems compatible with chalk https://www.npmjs.com/package/nanocolors

uhmm, maybe it will better to change the dependency, good catch man

Yeah, would also save us some kbs, and seems imple considering it's used only once in the codebase

updated

Superb!!

@PuruVJ
With autoInstall works with your fork with 0.12.9. We're changing to use picocolors instead nanocolors, it seems it will be closed in favor of picocolors (no esm support yet).
I'm checking on local with picocolors on your repo.

Wait, so yur saying autoInstall is not working due to nanocolors? 🤯

No, autoInstall  works like a charm man, I'll change to use picococlores since nanocolors has been deleted/deprecated, see PR #83 comments

The  autoInstall is an experimental thing to allow use the icons without the package installed, then unplugin-icons will install for you on dev, see the logs on console.
The [listID].svelte file looks like this:

Upps, the PR has no comments, you can see on the nanocolors repo or in #82 issue

We've already bikeshedded this quite a bit, but would you consider using kolorist for this task @antfu ?
It's small, deps free, fully ESM, and most importantly, by the preact guy @marvinhagemeister, so it's reliable as an army tank.
Here's all the code it has https://github.com/marvinhagemeister/kolorist/blob/main/src/index.ts
Would love to make a PR


Reintroduce click-log

Looks like passing the root logger could have solved our issue with it last time. This is also now the default too. 🎉

Any comment son this, @untitaker (I'm asking just in case you think I missed something).

Lgtm


[Bug] Burning Damage Support affects poison damage dealt by Fire -> Chaos conversion

https://pastebin.com/pTTd7g74
The more modifier on Burning Damage Support is applying to poison.

This is a known issue, but it is rather difficult to fix, as "Burning Damage" is synonymous with "Fire Damage over Time", and the program's damage calculation code cannot allow "Fire Damage" and "Damage over Time" to apply without also allowing "Fire Damage over Time" to also apply.
Fixing it would require Burning Damage to exist as a distinct stat, which is quite doable on the input side, but including that stat in the damage calculations would be a major pain.
However, I feel that Burning Damage actually should apply in this case, as having Fire Damage and Damage over Time apply without also applying the combined Fire Damage over Time doesn't really make sense. Probably worth running past GGG, but I haven't gotten around to it yet.


[Tokenomics] Probabilistic Proofs

Objective
Add support for probabilistic proofs, following the protocol specification proposal.
Origin Document
See: Probabilistic Proofs
Goals

Reduce on-chain resource utilization.

Deliverables

[ ] A PR for each new parameter needed:

[ ] ProofRequestProbability
[ ] ProofRequirementThreshold
[ ] ProofMissingPenalty

[ ] A PR which updates tokenomicskeeper.Keeper#SettlePendingClaims() to require a proof according to the probabilistic proofs spec.

Non-goals / Non-deliverables

...

General deliverables

[ ] Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
[ ] Testing: Add new tests (unit and/or E2E) to the test suite.
[ ] Makefile: Add new targets to the Makefile to make the new functionality easier to use.
[ ] Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @bryanchriswhite

Duplicate of #579


BSOD memory management error

Microsoft PowerToys version
0.63.0
Installation method
GitHub
Running as admin
No response
Area(s) with issue?
General
Steps to reproduce
just installed it, within an hour got BSOD, turned on pdf preview in setting as well, no more configuration.
PowerToysReport_2022-10-26-11-42-04.zip
✔ Expected Behavior
computer to keep working
❌ Actual Behavior
BSOD
Other Software
No response

Thank you for the report. Could you please send us the minidump of the BSOD located in "C:\Windows\Minidump"?
/needinfo

minidumps.zip
I had a few more BSOD since then, so I am adding all the dumps I had since yesterday.

it is now resolved, the issue was a failing RAM module. Sorry for wasting time.


Support "array by table subquery" constructor

From the spec:
<array value constructor> ::=
    <array value constructor by enumeration>
  | <array value constructor by query>

<array value constructor by query> ::=
  ARRAY <table subquery>

If <array value constructor by query> is specified, then
a) The <query expression> QE simply contained in the <table subquery> shall be of degree 1 (one). Let
ET be the declared type of the column in the result of <query expression>.
b) The declared type of the <array value constructor by query> is array with element type ET and maximum
cardinality equal to the implementation-defined maximum cardinality IMDC for such array types.

The result of <array value constructor by query> is determined as follows:
a) QE is evaluated, producing a table T. Let N be the number of rows in T.
b) If N is greater than IMDC, then an exception condition is raised: data exception — array data, right truncation.
c) The result of <array value constructor by query> is an array of N elements such that for all i, 1 (one) ≤ i ≤ N, the value of the i-th element is the value of the only column in the i-th row of T.
NOTE 167 — The ordering of the array elements is effectively determined by the General Rules of Subclause 7.13, “<query expression>”.

In simple terms,
SELECT ARRAY (SELECT x FROM t)

is equivalent to
SELECT (SELECT array_agg(x) FROM t)

This functionality is supported in BigQuery as described here.
One use case is to simplify queries that process arrays. Instead of doing a cross join, you can use the ARRAY function on a subquery. For example, you can do this:
select d.id,
       ARRAY(SELECT CAST(ROW(
          date_trunc('day',s.timestamp),
          s.user_id,
          s.path_id,
          s.accept_source
         ) as ROW(occurred_at TIMESTAMP,
                  id varchar,
                  path_id varchar,
                  accept_source varchar))
         FROM UNNEST(ARRAY_CONCAT(
               previous_states,
               current_state) s
       ) AS all_states
FROM my_data d

instead of this:
WITH arrayed_data AS (
    SELECT d.created_at, d.id,
           ARRAY_CONCAT(d.previous_states,
               current_state) as accepted
    FROM my_data d
)
select all_a.id,
    ARRAY_AGG(CAST(ROW(
          date_trunc('day',a.timestamp),
          a.user_id,
          a.path_id,
          a.accept_source
         ) as ROW(occurred_at TIMESTAMP,
                  id varchar,
                  path_id varchar,
                  accept_source varchar)))
    FROM arrayed_data as all_a
    CROSS JOIN UNNEST(all_a.accepted) AS a
    GROUP BY all_a.id

Below are a few standalone tests that show the current and desired behavior.


Robotics results using my voice

Hello, thanks for the amazing job you have done.
I have tried your model with my own voice and I am getting poor results, I have attached both audios (don't clone my voice please) so you can have an idea about what is happening.
These are the commands I am runing:
from utils.prompt_making import make_prompt

### Alternatively, use whisper
make_prompt(name="henry", audio_prompt_path="examples/henry.wav")

from utils.generation import SAMPLE_RATE, generate_audio, preload_models
from scipy.io.wavfile import write as write_wav

# download and load all models
preload_models()

text_prompt = """
Hey, Traveler, Listen to this, This machine has taken my voice, and now it can talk just like me!
"""
audio_array = generate_audio(text_prompt, prompt="henry")

write_wav("henry_cloned.wav", SAMPLE_RATE, audio_array)

Any information about how to improve the results?
henry.zip

I found the cloned voice in your attached file very nice and clear. Which aspect do you think the synthesized voice is poor and robotic?

Probably is because your recorded audio is in stereo while the cloned voice is in mono. The script will make a prompt with only the first channel of your input prompt if it contains more than one channel, which is probably the reason why you found the voice not to resemble yourself.


error: comparison of unsigned expression in '>= 0' is always true [-Werror=type-limits]

I tried to test DOLMA in Ubuntu20.04, but came across an ERROR after scons compiling. It seems to be blocked owing to safe checking? All helps will be greatly appreciated!

I tried to modify the SConstruct in dolma's repository root to ignore the error, but it doesn't work.

// added in ~/DOLMA/SConstruct
main.Append(CCFLAGS=['-Werror'])
main.Append(CXXFLAGS=['-Werror'])


ES2016 skeleton broken 'out of the box' ?

I've done the following:
git clone git@github.com:aurelia/skeleton-navigation.git
cd skeleton-navigation
cd skeleton-es2016
npm install
npm install -g gulp jspm
jspm install -y
gulp watch

Then I opened the browser in an incognito window, to http://localhost:9000 and got this error:
TypeError: Aurelia is not a function
    at http://localhost:9000/jspm_packages/npm/aurelia-bootstrapper@1.0.0-beta.1.0.1/aurelia-bootstrapper.js:96:21
    at h (http://localhost:9000/jspm_packages/npm/core-js@1.2.6/client/shim.min.js:1773:66)
    at http://localhost:9000/jspm_packages/npm/core-js@1.2.6/client/shim.min.js:1778:19
    at MutationObserver.e (http://localhost:9000/jspm_packages/npm/core-js@1.2.6/client/shim.min.js:1958:55)

I'm using Chrome Version 47.0.2526.111 (64-bit) on Linux Mint KDE
$ npm --version
3.5.1
$ node --version
v5.2.0

The skeleton I'm using is at git commit e0a0e3e9d3c8ebdca6fcd543f510004f2cb5f248 aka tag 1.0.0-beta.1.0.5.

I have now tried at tag 1.0.0-beta.1.0.4 and followed the same steps (except the difference in directory structure made me need to skip the cd skeleton-es2016 step).
That works perfectly, it seems. At least the few times I've cleared my cache and tried again.

I am in the process of doing a git bisect between .4 and .5 to see if I can figure out where my issue entered..

git bisect tells me that 2ca81bbc964f0c7f7381fa6a98386e7ad811dc6f is the first bad commit:
2ca81bbc964f0c7f7381fa6a98386e7ad811dc6f is the first bad commit

When I check out at 1.0.0-beta.1.0.5 and revert this line in config.js, the error goes away.
It seems the root cause of my problem is in the aurelia-bootstrapper, at v 1.0.0-beta.1.0.1.

Switching to<EMAIL_ADDRESS>makes the problem go away.

Fixed in latest bootstrapper.

We need to make the latest bootstrapper part of the skeleton.. I've done this in #267 ... :shipit: :wink:


Create contract request#2180

https://github.com/edenlabllc/ehealth.api/issues/2180

Coverage decreased (-10.3%) to 81.107% when pulling 9f1f09952ec5914baa0b2ed085435e1abdcbbfb4 on create_contract_request#2180 into 9269c07934d360bb411666dc936537f49b039381 on contracts#2176.


Web UI has interactive console to containers (kubectl exec)

(Via Slack conversation in K8s #tilt channel)
@epotex suggested a feature in the Web UI where you can easily jump into a console to debug issues in your running containers, equivalent to kubectl exec:

Usually when I  jump into a containers it would be to debug process or network related issues, for example I need to check connectivity to aother service(ping/telnet/nmap/etc) or maybe a response from api with a quick and dirty curl, a real life example from today I had a  Django service that was hanging after starting where ps aux | grep python will show that it listen to the port but netstat/nmap  confirm  that the service actually is not listening(hang)  so I needed to kubectl exec  into  the container to find all of that

A command-line shorthand, like tilt exec <resource name> would not be helpful to address this workflow:

to me it’s all boiled down to developers efficiency, that’s the focus for me as I build our dev tooling - less context switching is better - i’m not sure that the tilt exec will be useful since if I’m already switching to the console i’ll just use the kubectl exec so it’s just going to be a redundant command if it makes sense, thanks for considering it!

Thanks for documenting it!

It would be neat if the command-line UI had this feature as well. I could foresee being frustrated by a web console experience as they always have some issue or another. But being able to get an embedded shell right from within my terminal using my actual terminal to run it would be very nice.
I personally would see value in a tilt exec command as well. A lot of my potential users would definitely prefer to have to look up the full k8s name if I can provide them a consistent shorthand name consistent with what they see in tilt.

Similar to this what could be awesome would be to have the ability to pass in arguments to a specific local_resource from the UI. See comment in slack here
eg ( In the scenario where we wanted to pass a filter into a running tests command we could do that )


Share config

This doesn't quite work, I thought we supported remote urls for logos, but we don't yet!
Can hold off on merging until we get that fixed.

This is blocked by:

https://github.com/executablebooks/mystmd/issues/1296


Make error messages more compact, remove blank lines

https://github.com/elm-lang/elm-compiler/issues/963
I appreciate the attempts to make the errors helpful!
I would like to suggest what I think would further improve the messages:
The blank lines don't really help, my $0.02 is that I would prefer not to have blank lines emitted. It just ends up making the error message too long. e.g.  Removing them and it fits.

Some progress made. I'm going to close this in favor of https://github.com/elm-lang/elm-compiler/issues/936 though.


Problem with Esphome firmware upload

Which programmer do you use to upload the Esphome firmware?
CP210x USB to UART converter?
Ch340g USB to UART converter?
Esp-prog JTAG Downloader?
I'm using the "web.esphome.io" site and a "CP210x USB to UART" converter. The Dingtian board is the developer version.
A serial connection with the card begins but the firmware upload does not start.
A suggestion?
Thanks

The ESP32 will enter the serial bootloader when GPIO0 is held low on reset. Otherwise it will run the program in flash.

Also ESP32 is 3.3V device.

But in my case I did use ch340 and ftdi chip. It should'n make any difference which usb conveter you will be using as long is working.

Thank you very much. I solved.
the problem was that GPIO0 was not kept low upon reset


Should we flatten substance/model?

While we had the feeling it would be good not to clutter the root folder, I start disliking to write substance/model/transform.
We could either pull model/data and model/transform out or flatten it into substance/model.
Personally I would prefer substance/data and substance/transform
@michael what is ur opinion?

We can do substance/data and substance/transform. I have no objections.

Maybe that's one step too far: substance/model -> substance/document ?

Probably. I mean nothing holds us back from bringing the node-modules we had previously into the root folder but not particularly sure about this one. What would remain in model then?

Looking at substance/model it contains mainly Document related stuff. Some others such as Fragmenter could go into util.

Mhh i guess i would just miss this model vs ui thing..  mhh but that's gone anyway now we have transform on top level.

We should at least sleep one night over it ;)

Or we do it later on...

I'm also not sure regarding the collab folder. We have a server vs. client dimension now also. And the structure in test is not consistent anymore. Maybe we should dedicate a day next week for restructuring.

I am flattening tests right now, because there we have a lot of ../../../../../ which is b***t.
For tests I want that ../.. is root, so it is only one level deeper than the regular source files.

+1

For the top-level folders we can wait.

BTW, collab is not only about server, as you could do p2p and make a client a master.

btw don't you want to get rid of qunit?
it's really boring. i can't see any reason to use it except already written
tests...
On Fri, Jun 3, 2016 at 3:40 PM Michael Aufreiter<EMAIL_ADDRESS>wrote:

+1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223568078,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-lHeNVs7hGM8LAUTGsYm6NpkdlhXks5qICCwgaJpZM4Itarp
.

I agree. Let me finish this iteration. Then we start planning how to migrate.
@Integral have to time to figure out how to do it with mocha? Just a clean-room example somewhere.
We need to have that UI driven test suite, plus nodejs (which is obvious with mocha).
Also we need to know how to run the mocha driven UI suite from within karma.
That's unfortunately a bunch of requirements...

@michael another thing to think about: should tests for a package not live in the package, too?

well... first of all, as I understand, QUnit is just an outdated thing. not
sure, but I have a feeling that there is no features in QUnit which you
cant find on other modern frameworks.
There is no problem to execute mocha within karma. Don't know nothing about
mocha driven UI test suites, but pretty sure that it is possible.
if you want i could prepare some small package, like with 2 tests for basic
and UI things.
regarding dependencies... you already drown inside dependencies ;) but
that's a dev stuff, isn't it?)
On Fri, Jun 3, 2016 at 3:45 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

I agree. Let me finish this iteration. Then we start planning how to
migrate.
@Integral https://github.com/Integral have to time to figure out how to
do it with mocha? Just a clean-room example somewhere.
We need to have that UI driven test suite, plus nodejs (which is obvious
with mocha).
Also we need to know how to run the mocha driven UI suite from within
karma.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223569158,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-iHH7yOyhhpIZmbSX5R-DznRB7lXks5qICH5gaJpZM4Itarp
.

That would be great!
Yep, devDependency -- we can have as many we need ;)

Please also evaluate tape. It's from the browserify maker.
https://github.com/substack/tape
https://medium.com/javascript-scene/why-i-use-tape-instead-of-mocha-so-should-you-6aa105d8eaf4#.kjrsjvt50

tape looks interesting. Very simple.
Also found this: https://ci.testling.com/guide/tape
It seems that it comes without UI, though.

will play with both in a branch
On Fri, Jun 3, 2016 at 4:22 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

tape looks interesting. Very simple.
Also found this: https://ci.testling.com/guide/tape
It seems that it comes without UI, though.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223576980,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-if5HQXC3fKnGeHryUsLuwDnbBReks5qICqEgaJpZM4Itarp
.

Cool! Thx!

But before, let's just summarise what we need (pls, add something or
correct me):

test code with linter
test code with node
test code in different browsers
test code in different devices (desktop, tablets, phones etc)
run tests directly in browser and receive report
test coverage report
run tests on CI servers

is it correct?
On Fri, Jun 3, 2016 at 7:32 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

Cool! Thx!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223627860,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-qdltyPWUhneIH0-D8vZkhqYBUcdks5qIFcvgaJpZM4Itarp
.

Linter is working already - so no need for that.
yes
yes (we were using karma for that particularly only testing for FF and Chrome on Travis)
future (so no prio)
yes, because that's the env we are actually fixing things
yes
yes (but ok if only Chrome and FF, Edge is just too big an effort --  would need a different CI

2 vs 3: there are some ui tests we exclude when running in node.. but this actually done from within the test by checking inBrowser.

5: and it should run in like browserified served setup... debugging in karma is not an option.

finally: would be good to start the automated tests from gulp, while 5) should still be just served by server.js.

so you want to run tests and debugging inside concrete browser, but still
want to have some tool which will emulate browsers and run all tests inside
them one after another, right?
On Fri, Jun 3, 2016 at 8:10 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

finally: would be good to start the automated tests from gulp, while 5)
should still be just served by server.js.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223636916,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-iMG0vRasebdIUwvsNCNcHF5YzbLks5qIGAGgaJpZM4Itarp
.

correct :)

plus of course: running the tests in nodejs to make sure the server side stuff (including server side rendering) is working.

Well, didn't try anything yet, but looks like tape could be very useful
here as you can bundle test (browserify) and then debug it in browser.
Regarding automated browser tests looks like there is no better way then
karma... There are couple of other tools like zuul (
https://github.com/defunctzombie/zuul) or smokestack (
https://github.com/hughsk/smokestack), but karma works with biggest number
of browsers... Of course there are external tools like browserstack or
sauce labs, but I think we must do it firstly ourselves...
On Fri, Jun 3, 2016 at 8:49 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

correct :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223640171,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-prfFDOWvFO06bD3RaKEff8JfQBVks5qIGMKgaJpZM4Itarp
.

Ok!
Do u think that tape bundle can be run in karma?
As I understand karma needs kind of an adapter which is starting the test and providing the result to the karma API.. but don't know for sure.

There are karma adapters for tape, no worries!
But I mean running test suite and debugging inside individual browser. E.g.
automated tests will tell you that something wrong with chrome and then you
could open special test bundle in chrome and debug! At least I hope so ;)
On Fri, Jun 3, 2016 at 9:27 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

Ok!
Do u think that tape bundle can be run in karma?
As I understand karma needs kind of an adapter which is starting the test
and providing the result to the karma API.. but don't know for sure.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223656316,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-gDvt5XEjU5Wfska9Uw2CZ3W9HmXks5qIHIDgaJpZM4Itarp
.

Yep. That is essential :)

Btw, do you know anything about this thing?
http://seleniumhq.github.io/selenium/docs/api/javascript/
On Fri, Jun 3, 2016 at 9:34 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

Yep. That is essential :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223658199,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-i-eYnT9uTd9_XWi8BlrlN7TBualks5qIHO5gaJpZM4Itarp
.

I am not sure... but selenium is a brutal heavy thing. But if u think that is feasible than take a look.

I don't think we would pull that in as dev dependency, just to run it on CI.

maybe... https://www.npmjs.com/package/selenium-standalone

As I understand, karma is there to offer a more lightweight alternative. Still, I would not say karma is light-weight. And I also understand, that karma is not helping us much to produce real browser events, as a user would produce. Maybe selenium is able doing that. (which is one thing I really miss, because right now we can't really test the ui as we can't produce native UI events.)

Can't say anything. Never used it. But agreed that it is a brutal thing :)
On Fri, Jun 3, 2016 at 10:05 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

As I understand, karma is there to offer a more lightweight alternative.
Still, I would not say karma is light-weight. And I also understand, that
karma is not helping us much to produce real browser events, as a user
would produce. Maybe selenium is able doing that. (which is one thing I
really miss, because right now we can't really test the ui as we can't
produce native UI events.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223665928,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-nXXIXvrRzk-k89WLUwvXlYk1q5kks5qIHrcgaJpZM4Itarp
.

interesting: https://github.com/juliangruber/tape-run
On Fri, Jun 3, 2016 at 10:32 PM<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Can't say anything. Never used it. But agreed that it is a brutal thing :)
On Fri, Jun 3, 2016 at 10:05 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

As I understand, karma is there to offer a more lightweight alternative.
Still, I would not say karma is light-weight. And I also understand, that
karma is not helping us much to produce real browser events, as a user
would produce. Maybe selenium is able doing that. (which is one thing I
really miss, because right now we can't really test the ui as we can't
produce native UI events.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223665928,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-nXXIXvrRzk-k89WLUwvXlYk1q5kks5qIHrcgaJpZM4Itarp
.

also: https://theintern.github.io/intern/#what-is-intern
https://github.com/jquery/testswarm/wiki
On Fri, Jun 3, 2016 at 11:39 PM<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

interesting: https://github.com/juliangruber/tape-run
On Fri, Jun 3, 2016 at 10:32 PM<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Can't say anything. Never used it. But agreed that it is a brutal thing :)
On Fri, Jun 3, 2016 at 10:05 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

As I understand, karma is there to offer a more lightweight alternative.
Still, I would not say karma is light-weight. And I also understand, that
karma is not helping us much to produce real browser events, as a user
would produce. Maybe selenium is able doing that. (which is one thing I
really miss, because right now we can't really test the ui as we can't
produce native UI events.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223665928,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-nXXIXvrRzk-k89WLUwvXlYk1q5kks5qIHrcgaJpZM4Itarp
.

and finally local testling: https://github.com/substack/testling (but it
looks like abandoned...)
On Fri, Jun 3, 2016 at 11:56 PM<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

also: https://theintern.github.io/intern/#what-is-intern
https://github.com/jquery/testswarm/wiki
On Fri, Jun 3, 2016 at 11:39 PM<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

interesting: https://github.com/juliangruber/tape-run
On Fri, Jun 3, 2016 at 10:32 PM<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Can't say anything. Never used it. But agreed that it is a brutal thing
:)
On Fri, Jun 3, 2016 at 10:05 PM Oliver Buchtala <
<EMAIL_ADDRESS>wrote:

As I understand, karma is there to offer a more lightweight
alternative. Still, I would not say karma is light-weight. And I also
understand, that karma is not helping us much to produce real browser
events, as a user would produce. Maybe selenium is able doing that. (which
is one thing I really miss, because right now we can't really test the ui
as we can't produce native UI events.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223665928,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-nXXIXvrRzk-k89WLUwvXlYk1q5kks5qIHrcgaJpZM4Itarp
.

testlng really seems to be stalled. No option.

what about a mocha based solution? do u think that is ok as well?

Yep. This would be ok, there is karma adapter for mocha of course.
But tape seems to be much lightweight. I would say that the challenge is to
replace karma with somewhat easier...
Btw, do need installed browsers for karma?
On Sat, Jun 4, 2016, 03:05 Oliver Buchtala<EMAIL_ADDRESS>wrote:

what about a mocha based solution? do u think that is ok as well?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223715129,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-j5FNZyjwdNqr2O6lo1OufFPubQbks5qILMdgaJpZM4Itarp
.

karma needs the browsers installed yes.
Actually, I don't have much hope to get rid of karma.
Do you think we can achieve all we said with karma and tape?
I would then opt for this combi

There is https://github.com/tmcw-up-for-adoption/karma-tap
but this project isn't maintained anymore and it's not clear whatever or
not it will work
Regarding karma... if everything we want from karma is ability to run tests
in browsers than it might help us: https://github.com/juliangruber/tape-run
On Sat, Jun 4, 2016 at 1:47 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

karma needs the browsers installed yes.
Actually, I don't have much hope to get rid of karma.
Do you think we can achieve all we said with karma and tape?
I would then opt for this combi
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223749237,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-p1xg2iQsUZrbOrRlUVEVIN5_Q9Jks5qIVfBgaJpZM4Itarp
.

Alright, let's try tape-run :)

And if that not working we can go back to karma - i understance adapters are not difficult: https://developers.soundcloud.com/blog/writing-your-own-karma-adapter

Yep, and there is already adapter for tape here:
https://github.com/tmcw-up-for-adoption/karma-tap/tree/master/src
but look, they do it with tap parser which is soooo big:
https://github.com/tmcw-up-for-adoption/karma-tap/blob/master/src/tap-parser.js
On Sat, Jun 4, 2016 at 3:20 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

And if that not working we can go back to karma - i understance adapters
are not difficult:
https://developers.soundcloud.com/blog/writing-your-own-karma-adapter
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223752696,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-qs79rhUBNDv1hxqwjmmlPNSbVFuks5qIW18gaJpZM4Itarp
.

The size is actually not a problem, I think.

Mmmm, it is a problem actually! Just imagine that you need to find a bug
inside parser which consists of 16k of lines! Ahhh. It would be just a
black hole in terms of understanding what's going there.
On Sat, Jun 4, 2016 at 4:41 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

The size is actually not a problem, I think.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223756280,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-rxia4wJJqf9UMLdiTXLHEogTjtXks5qIYCGgaJpZM4Itarp
.

But... less talk, more work... I mean, let's try!
On Sat, Jun 4, 2016 at 5:36 PM<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Mmmm, it is a problem actually! Just imagine that you need to find a bug
inside parser which consists of 16k of lines! Ahhh. It would be just a
black hole in terms of understanding what's going there.
On Sat, Jun 4, 2016 at 4:41 PM Oliver Buchtala<EMAIL_ADDRESS>wrote:

The size is actually not a problem, I think.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/620#issuecomment-223756280,
or mute the thread
https://github.com/notifications/unsubscribe/AALG-rxia4wJJqf9UMLdiTXLHEogTjtXks5qIYCGgaJpZM4Itarp
.

Hey!
Looks like I ported whole test suite to tape.
Checkout test-env, do npm install and then gulp test:tape.
Of course there is a room for some improvements... But will describe later, now i need to switch for other things. Woooo! ;)

Whoot!

let's continue in separated issue (#625)
sorry for such a big off topic :)

I think we are not motivated anymore.


Internally handle server port range

This PR internally handles server port range for the HTTP transport. On further reflection of https://github.com/micro/go-micro/pull/49 it was clear that PortRange is a use case that needs to be solved but is very specific to the HTTP implementation. Adding a PortRange option did not feel like the right approach. Parsing the address allows existing --server_address or MICRO_SERVER_ADDRESS options to be set with "host:minPort:maxPort" without modifying any other code.

:+1:


Added basic settingsMenu with no real functionality

Can be merged without problems, as no original files are changed.


Flat trim for Bebop

Hi,
I'm currently working with a Bebop v1 using pyparrot installed through pip.
A major problem that I'm facing is that the drone will start drifting off to the side after a while. I've been previously working with katarina (switched for performance reasons) and i found out that this problem is solved by performing a flat trim on the drone once after booting. (In practice this meant, booting up the drone, connecting with katarina, issuing a flat trim command, disconnecting and connecting with pyparrot)
However, i'm missing a flat trim command in pyparrot for this purpose. I tried writing an extension method along the lines of:
def flat_trim(self: Bebop):
    command_tuple = self.command_parser.get_command_tuple("ardrone3", "Piloting", "FlatTrim")
    self.drone_connection.send_noparam_command_packet_ack(command_tuple)

but something still seems to be wrong.
I've previously successfully experimented with sending other commands to the bebop and i'm not really doing anything differently here.
Any help would be highly appreciated,
Cheers
Jonas

Hi,
def flat_trim(self):
command_tuple = self.command_parser.get_command_tuple("ardrone3", "Piloting", "FlatTrim")
self.drone_connection.send_noparam_command_packet_ack(command_tuple)
Works well for me.
Just add the function in the Bebop.py file at Bebop class level.

Hi,
so after trying all kinds of weird hacks, i found adding self.ask_for_state_update() before and after above mentioned function call actually fixes the problem for me.
No idea what is actually going on there. I have a feeling it might have to do with trying to flat trim directly after connecting and something important has not happened yet.

I tried this script :
from pyparrot.Bebop import Bebop

bebop = Bebop("Bebop")
bebop.connect(10)
bebop.flat_trim()
bebop.disconnect()

And the flat trim works well for me.

Ah, i suppose it could have something to do with the fact that you're working with a Bebop 2 while i'm using the Bebop (v1).

I also use a bebop v1

This is perfect timing!  I'm doing demos in about 2 hours and I was trying to figure out why it was drifting.

Glad to help!

@mayerjRRR Have you tested the latest version of pyparrot that should normally fix le problem?

I just demoed and it worked well :)

I am flying a Bebop 2 though.

On my bebop 1 it works very well too :)


[ip6] add tests for ipv6 fragmentation

Codecov Report

Merging #1 into ip6-fragmentation will increase coverage by 2.38%.
The diff coverage is 85.75%.

@@                  Coverage Diff                  @@
##           ip6-fragmentation       #1      +/-   ##
=====================================================
+ Coverage              72.06%   74.45%   +2.38%
=====================================================
  Files                    292      298       +6
  Lines                  38718    37575    -1143
=====================================================
+ Hits                   27901    27975      +74
+ Misses                 10817     9600    -1217

Impacted Files
Coverage Δ

src/core/coap/coap_secure.cpp
91.46% <ø> (ø)
:arrow_up:

src/core/api/message_api.cpp
80.45% <ø> (ø)
:arrow_up:

examples/apps/ncp/main.c
63.15% <ø> (-1.85%)
:arrow_down:

src/core/common/settings.hpp
93.33% <ø> (ø)
:arrow_up:

examples/apps/cli/main.c
84.21% <ø> (-0.79%)
:arrow_down:

src/core/common/locator-getters.hpp
100% <ø> (ø)
:arrow_up:

src/core/api/channel_manager_api.cpp
100% <ø> (ø)
:arrow_up:

include/openthread/ip6.h
100% <ø> (ø)
:arrow_up:

examples/platforms/posix/alarm.c
87.8% <ø> (+0.3%)
:arrow_up:

include/openthread/udp.h
100% <ø> (ø)
:arrow_up:

... and 143 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8682b90...ea1243f. Read the comment docs.


Generated GraphQL declarations not written to project config source directory

Describe the bug
It appears that the generated GraphQL declarations are always written to a directory called src instead of respecting the sourceDir config attribute that appears in amplify/project-config.json. This is a nested config attribute under javascript.config.
Steps to reproduce the behaviour:

Create a new amplify js project
Choose e.g. the Ember framework option (which defaults to app for sourceDir) or manually enter app at the prompt.
Add the api category and create a GraphQL schema (default one should be fine)
See that the generated queries, mutations, subscriptions and schema files are written to src/graphql instead of app/graphql

Expected behaviour
Since we've declared that source files live in app we would expect the graphql files to also live there.

This is a duplicate issue and we're tracking it as a part of #916.


Remove the pingInterval

Since we don't use the timeout strategy in the publication-client, we
can remove the pingInterval in favor of our own ping/pong
implementations.

@ghmeier bump


rtc: Add data model for "Cesiones Periodo" entries

Add a data model for "Cesiones Periodo" entries.
This commit was taken from:

https://github.com/fyntex/lib-cl-sii-api-python/commit/0bafb09711cfe1695e27f82eeeb8686b8f8626d3
https://github.com/fyntex/lib-cl-sii-api-python/commit/d7dd76d713cd429ca1d3c8e08da5f7ede2c95649
https://github.com/fyntex/lib-cl-sii-api-python/commit/60eda63de7d8fd54ddf681ad54709f3752bc8e7c

Additional changes:

Rename files.
Fix imports.
Add and remove some empty lines.

Ref: https://github.com/fyntex/lib-cl-sii-api-python/issues/209.

Codecov Report

Merging #172 (694e260) into develop (a7a78e1) will decrease coverage by 1.37%.
The diff coverage is 55.04%.

@@             Coverage Diff             @@
##           develop     #172      +/-   ##
===========================================
- Coverage    78.14%   76.77%   -1.38%
===========================================
  Files           26       27       +1
  Lines         1725     1834     +109
  Branches       279      308      +29
===========================================
+ Hits          1348     1408      +60
- Misses         256      282      +26
- Partials       121      144      +23

Impacted Files
Coverage Δ

cl_sii/rtc/data_models_cesiones_periodo.py
55.04% <55.04%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a7a78e1...694e260. Read the comment docs.

It would be nice to convert them to pydantic dataclasses and be able to remove most of the awful type checking code, don't you agree?

It would be nice to convert them to pydantic dataclasses and be able to remove most of the awful type checking code, don't you agree?

@glarrain The AEC and generic Cesion data models are like that. After they're finished it'll be the turn of CesionesPeriodoEntry.


export iterator.valid()

The valid() method could be helpful when we want to know if an iterator is positioned at a kv pair after calling seek().

LGTM, thanks for the contribution!  I'll cut a new version 0.3.2 that contains these changes now!


"Sourcepath" doesn't get included

HI!
While using the tool, I was possible to generate the configuration.xml file.
However, this file did not include a very important part - SourcePath. I had to manually add it.
I've pointed it out when using the tool, and generated an installation file with no embedded installation files.
Can you please verify?
Thanks,
Dvir

@Dviros  which tool are you talking about?  The XML Editor?

Hey @alex182, it's the "Microsoft Office ProPlus Install Toolkit"

@Dviros We've tested the most recent version of the Toolkit and it adds the SourcePath if you specify it. If you don't specify a SourcePath and choose to embed the files, the operation will still work because the ODT will look in the current directory for the source files. These files get extracted to C:\Windows\Temp\OfficeProPlus.


Resolving a dependency by path

Purpose

Capability of overriding a dependency by giving its balo path

[dependencies]
"wso2/twitter"= { path = "/path/to/twitter.balo" }

Fixes https://github.com/ballerina-platform/ballerina-lang/issues/17854
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[X] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[X] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #18055 into packerina-dev will increase coverage by 6.03%.
The diff coverage is n/a.

@@                Coverage Diff                @@
##           packerina-dev   #18055      +/-   ##
=================================================
+ Coverage           14.2%   20.23%   +6.03%
=================================================
  Files                 51       51
  Lines               1598     1334     -264
  Branches             260      211      -49
=================================================
+ Hits                 227      270      +43
+ Misses              1358     1050     -308
- Partials              13       14       +1

Impacted Files
Coverage Δ

tool-plugins/vscode/src/debugger/index.ts
0% <0%> (ø)
:arrow_up:

...-plugins/vscode/src/project-tree-view/activator.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/extension.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/diagram/activator.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/core/preferences.ts
100% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/project/activator.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/overview/activator.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/overview/renderer.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/core/messages.ts
100% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/api-editor/activator.ts
0% <0%> (ø)
:arrow_up:

... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6017a66...84bd376. Read the comment docs.


BeeTable: It's not possible to remove the Last column of a GIVEN / EXPECT group

When attempting to delete the last column in a given or expect group, the delete option is currently disabled. Instead, the delete option in the context menu should be enabled, and upon deleting, a new empty column should replace the removed one

@kbowers-ibm I already fixed that in a local branch, I'll create a PR soon.

Nice! Just converting some of the items from the list in the epic to issues so I can skip them for testing purposes 👍


rsz: print progress

Adds:

adds number of violating endpoints to setup repair (to mirror the behavior of the repair hold).
adds a -verbose to repair_timing to print the status of the repair.

Motivation:
repair_timing can run for a very long time on some designs and it would be helpful to be able to see what progress if any is actually being made.
Current format:
Iteration | Resized | Buffers | Cloned Gates | Pin Swaps |   WNS   |   TNS   | Endpoint

printed every 10 repairs (could increase of decrease that). It doesn't seem to impact the runtime (atleast from what I could tell).
If there are suggestions for alternative headings or phrasing I think that would be helpful, since I just grabbed the first set that came to mind.
@maliberty @openroadie I noticed that the pin swap goes negative (this seems counter intuitive to me, I assume that is not correct).

My guess on the negative swaps is that undos are not properly counted

10 repairs sounds too low.  ca53 has > 3k hold buffers so that is a lot of messages.  Maybe 100?


Still add log-point if not changing role

fixes weird interaction with used_invites where it would store a log pointer to bad log point
i don't think UserLog should lie about latest sig-meta if there is a change_membership entry that doesn't change role

cc @mlsteele can you take a look? this seems scary but I'm fairly confident it's the right thing to do here.


Merged master into release 1.16

@daminisatya , Looking at the error above  -- Unverified commit signature. Have you looked into this?

@kbhawkey I see the commit messages are proper. Not sure what is failing - https://github.com/kubernetes/website/pull/17313/commits

Let me try again this merge


Automatically deploy if contract changed (#75)

Automatically deploy on testnet if contract changed

Merging to main,  already approved on develop.


Zend_Mail_Header_HeaderValue::isValid is not valid with special characters

I just encountered an error in an older script (using Zend 1):
I do realise this is ZF1 code, but I checked and the validation check in ZF2 is exactly the same.
This is part of an email:

    --_008_AM2PR07MB0610CE26A6AB7483C300DDC0F4620AM2PR07MB0610eurp_
    Content-Type: application/vnd.openxmlformats-officedocument.wordprocessingml.document;
            name="=?utf-8?B?VmVydHJhdWxpY2hrZWl0c2Vya2zDpHJ1bmdfTGllZmVyYW50ZW5fYmxhbmtv?=
     =?utf-8?Q?.docx?="
    Content-Description: =?utf-8?B?VmVydHJhdWxpY2hrZWl0c2Vya2zDpHJ1bmdfTGllZmVyYW50ZW5fYmxhbmtv?=
     =?utf-8?Q?.docx?=
    Content-Disposition: attachment;
            filename="=?utf-8?B?VmVydHJhdWxpY2hrZWl0c2Vya2zDpHJ1bmdfTGllZmVyYW50ZW5fYmxhbmtv?=
     =?utf-8?Q?.docx?="; size=29798;
            creation-date="Wed, 10 Dec 2014 15:04:14 GMT";
            modification-date="Wed, 10 Dec 2014 15:04:14 GMT"
    Content-Transfer-Encoding: base64

But my code is failing with this:
    exception 'Zend_Mail_Exception' with message 'Invalid header value detected [application/vnd.openxmlformats-officedocument.wordprocessingml.document; name="Vertraulichkeitserklärung_Lieferanten_blanko.docx"]' in /usr/share/php/Zend/Mail/Header/HeaderValue.php:133
    Stack trace:
    #0 /usr/share/php/Zend/Mail/Part.php(600): Zend_Mail_Header_HeaderValue::assertValid('application/vnd...')
    #1 /usr/share/php/Zend/Mail/Part.php(151): Zend_Mail_Part->_validateHeaders(Array)
    #2 /usr/share/php/Zend/Mail/Part.php(289): Zend_Mail_Part->__construct(Array)
    #3 /usr/share/php/Zend/Mail/Part.php(353): Zend_Mail_Part->_cacheContent()
    #4 /root/gmail_backup/Searcher.php(241): Zend_Mail_Part->countParts()

As you can see the name of the file is with a special character (ä).
my "iconv.internal_encoding" was set to ISO-8859-1, I am setting it now to UTF-8, but the error still appears.
The invalid character is at position 99 (so the ä) which has an ordinal of 195. (UTF8 of ä is 0xC3 0xA4). I think it has to do with taking the next byte ( via ord($value[$i])) instead of taking the next character by using mb_substr.

Closing as duplicate

ZF2: https://github.com/zendframework/zend-mail/pull/28
ZF1: https://github.com/zendframework/zf1/issues/572


Designate our markdown files as our source code

This parallels a change submitted by @i-ky on our learn repo, as a way to attract the right participation.  Our repo currently looks more like a JavaScript package, which may partially explain several recent issues opened with the expectation that this is a published JavaScript project. Our JavaScript is really just our test environment for our Markdown.

This doesn't require TSC approval and has a very limited impact, so I'm going to merge it with one approval and take a look at whether GitHub shows our source code correctly to verify.

Yup- now shows as mostly markdown with only a tiny bit of JavaScript, which is accurate!

:tada: Thanks @handrews (and @ralfhandl )


Web – Personal information– Weird conversion from English to Spanish

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the "Settings" menu.
Click on "Workspaces."
Navigate to "Bank Account."
Click on "Connect bank account."
Proceed through all the steps until you reach the "Personal information" section (step 3 of 5).
On the first button two texts are written “Passport” and “Photo Page” in English, on conversion to Spanish by google which should be “pasaporte” and “página de fotos”.
Now convert the langusage to Spanish
Notice "Passport" is correctly translated to "pasaporte," but "Photo Page" is incorrectly translated to "Página del pasaporte con su foto," which means "Passport page with your photo." This translation is not accurate.

Expected Result:
The text "Photo Page" should be accurately translated to "página de fotos."
Actual Result:
The "Photo Page" is incorrectly translated to "Página del pasaporte con su foto," which means "Passport page with your photo."
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.43-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/29d058b6-fafd-4a40-b971-2be7f8414c0f
Expensify/Expensify Issue URL:
Issue reported by: @usmantariq96
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689843862152859
View all open jobs on GitHub

Can i work on this?

Proposal
Problem
Personal information– Weird conversion from English to Spanish
Root Cause
default translation for button detail is Página del pasaporte con su foto,
https://github.com/onfido/onfido-sdk-ui/blob/master/locales/es_ES/es_ES.json#L218
Proposed Changes
To address this issue, we have to pass desired translation for doc_select.button_passport_detail here. https://github.com/Expensify/App/blob/d77494013cc2b449573150e17e5771a69e60e08c/src/components/Onfido/BaseOnfidoWeb.js#L111C13-L113C15

Going to check this one in BZ from our Spanish speakers!

https://expensify.slack.com/archives/C01GTK53T8Q/p1690215562111699

Closing, since the translation is coming from Onfido, this is not something we would fix!


Document the use of androidx.startup explicitly

I'm using this library and was completely unaware that it uses androidx.startup.Initializer to initialize the PhoneNumberUtil instance. Since I was not using the androidx.startup anywhere in my app, I went ahead disabling the same while following the WorkManger: Custom Configuration  guide, which resulted in an app crash due to IllegalStateException raised here
it's pretty neat that library takes care of initialization itself 👍
(No more "put these lines in your Application's onCreate()" instructions 🙂)
and documenting the same would make it even clearer.

Done!


Fixes scenarios of generating declaration file when first type argument is generic function type

Handles two scenarios:

When the type is inferred, checker tracks if its writing first type argument so it can parenthesize generic function type.
When the type is written in a file but isn't parenthesized but has used space to separate the type parameter from type argument's <. Declaration emitter after looking up previous emitted character to be <, parenthesizes the function type with type parameters

Fixes #8105

:+1:


[Pharmacy Module] Check  button of ARV drug is not working as it should

[Actual Results]: The system does not displays any field.
[Expected results]: The system should display the terapeutical line field, regimen field and the ARV plan field.
[Steps to reproduce]: Go to Pharmacy Module- Search for a patient- click on the prescription tab- click on the prescription check

@Benilde in which server this is happening?

@valvijo Dev server

@Benilde i think its working on dev. Can you check again?

@ynurmahomed . It is working


WIP: Drugbank update

Importer modified by commenting out the def self.get_version in order to import current drugbank contents into db before merge. Code will be reverted back once versioning has been fixed in the respective scripts (which I'm working on but just haven't got working up until this point).

Importer reverted back to original after python scripts were updated and working.


Add support for HTTP server and client metrics

It'd be awesome if support for HTTP server/client metrics could be added to the OTel Pipeline as well to track worker request processing time (maybe a flag indicating if we'd want the entire worker wall time or just the time it took to handle a request and return a response) and the same for outbound fetch requests:
https://opentelemetry.io/docs/specs/semconv/http/http-metrics/

Hey @stebet, while in theory it might be possible to add those metrics, in practice that is a bit tougher than it might seem, but more importantly they would not be very useful. Unless you have a massive amount of traffic, or extremely localised clients, your requests will be so spread out over different machines that you will get basically one datapoint per request.
You are better off using a service that can calculate those metrics based on the traces that we do enable. I know Honeycomb is able to do that, but there might be others too..


Update auth check

Switch to checking user info instead of loading to stop flickering when navigating

Maybe Closes https://github.com/weaveworks/weave-gitops/issues/1900

Indeed, thanks @jpellizzari! :)

Maybe Closes https://github.com/weaveworks/weave-gitops/issues/1900

I think so. We could refactor the whole AuthCheck thing to useQuery too where isLoading behaves a bit more in-align w/ what we're after. This is a bit of a quick-fix so we can get a release out the door. 😬

Yup, I've got that on the list - make more use of useQuery (was going to work on it as part of the refactoring on https://github.com/weaveworks/weave-gitops/pull/1983) ;)


Add keys to jump more than a single file at once when browsing the files list

I don't get this issue @soupi, brick's list widget already supports page up, page down, home, and end.

I didn't know. It's not stated anywhere in the program.


Try map failure

A follow-up of #1923

Codecov Report

Merging #1924 into master will decrease coverage by 0.07%.
The diff coverage is 91.66%.

@@             Coverage Diff              @@
##             master    #1924      +/-   ##
============================================
- Coverage     98.06%   97.99%   -0.08%
- Complexity     5070     5080      +10
============================================
  Files            89       89
  Lines         11457    11553      +96
  Branches       1513     1522       +9
============================================
+ Hits          11235    11321      +86
- Misses          113      114       +1
- Partials        109      118       +9

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/javaslang/collection/Traversable.java
96.8% <ø> (ø)
112 <0> (ø)
:arrow_down:

javaslang/src/main/java/javaslang/control/Try.java
93.35% <91.66%> (-0.8%)
87 <11> (+11)

...src/main/java/javaslang/concurrent/FutureImpl.java
83.54% <0%> (-2.54%)
21% <0%> (-1%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8da6f2f...108a1e2. Read the comment docs.

Note: Calls to mapFailure will be unchecked in 2.1.0 because

Try in an interface
=> we can't make mapFailure() final
=> we can't mark mapFailure as @SafeVarargs

Solution: make Try an abstract class (see #1825)


BufferedSourceJsonReader.lenient is a private var but doest not expose API to mutate its value

Version
3.7.4
Summary
BufferedSourceJsonReader.lenient is a private var but doest not expose API to mutate its value
Steps to reproduce the behavior
When the server returns a malformed JSON, Use JsonReader.setLenient(true) to accept malformed JSON was thrown but the error message didn't indicate a fix in practise
Logs
(Your logs here)

Hi!
Indeed, the message is misleading since there is no way to do that!
On the other hand I'm not sure it would be wise to add the option as it would be preferable for the backend to emit valid JSON in all cases. I'd rather replace the error message by "Invalid JSON" or something. For the sake of curiosity, could you share which part of the JSON is triggering the error?

In my synthetic test cases, the following can trigger Use JsonReader.setLenient(true) to accept malformed JSON exceptions
            {
              "errors": [;]
              "data": null
            }

I see, thanks! The semicolon and missing comma.
To help us better understand, does this also happen with actual payloads coming from the backend or is it limited to tests?
In any case I think it will make more sense to not have a lenient mode at all in the parser.

It happened with the actual payloads, but we need to wait for a week for our app to be released so that we can capture the actual payload.

Thanks for clarifying! Please update the issue if/when you have access to the payload, just so we're sure that there isn't an actual issue with the parser.

From our logging, we either have

Body as an empty string
Body length exceeding 16000KB which is forbidden to be fully logged in our observability tools

The query name aligns with what's reported https://github.com/apollographql/apollo-kotlin/issues/4570
Our guess is that in those cases, error and data are both present, and data section is too large to be reported.

The only useful stacktrace for the non-empty Body I see is, where I think is the stream is closed before fully parsing

Caused by: okhttp3.internal.http2.StreamResetException: stream was reset: CANCEL
	at okhttp3.internal.http2.Http2Stream$FramingSource.read(Http2Stream.kt:54)
	at okhttp3.internal.connection.Exchange$ResponseBodySource.read(Exchange.kt:14)
	at okio.RealBufferedSource.read(RealBufferedSource.kt:4)
	at okio.RealBufferedSource.exhausted(RealBufferedSource.kt:21)
	at okio.InflaterSource.readOrInflate(InflaterSource.kt:51)
	at okio.InflaterSource.read(InflaterSource.kt:6)
	at okio.GzipSource.read(GzipSource.kt:321)
	at okio.RealBufferedSource.indexOfElement(RealBufferedSource.kt:35)
	at com.apollographql.apollo3.api.json.BufferedSourceJsonReader.nextQuotedValue(BufferedSourceJsonReader.kt:4)
	at com.apollographql.apollo3.api.json.BufferedSourceJsonReader.nextString(BufferedSourceJsonReader.kt:94)
	at com.apollographql.apollo3.api.json.-JsonReaders.readAny(JsonReaders.kt:101)
	at com.apollographql.apollo3.api.json.-JsonReaders.readAny(JsonReaders.kt:89)
	at com.apollographql.apollo3.api.json.-JsonReaders.readAny(JsonReaders.kt:89)
	at com.apollographql.apollo3.api.ObjectAdapter.fromJson(Adapters.kt:39)
	at com.apollographql.apollo3.api.ListAdapter.fromJson(Adapters.kt:5)
	at com.apollographql.apollo3.api.ListAdapter.fromJson(Adapters.kt:1)
	at com.apollographql.apollo3.api.NullableAdapter.fromJson(Adapters.kt:26)
	at

Thanks a lot for reporting back!
It looks like they're all network errors, broadly speaking - that explains why the JSON parsing could fail.
In #4656 I removed the misleading lenient messages, I will merge it then.

#4556 has been released as of v3.7.5 - the misleading error messages have been removed.


file name returned by function call is having temp in the last and file save by normal name

I have created a db backup cronjob. previously it was working fine but now its giving error
for (let index = 0; index < dbOptions.length; index++) {
                let date = new Date();
                currentDate = this.stringToDate(date);
                const element = dbOptions[index];
                if (element.autoBackup) {
                        con = {
                                host: element.host,
                                user: element.user,
                                password: element.pass,
                                database: element.database,
                        }
                        fName = element.database + '-' + currentDate.getHours() + '_' + currentDate.getMinutes() + '-' + currentDate.getFullYear() + '-' + (currentDate.getMonth() + 1) + '-' + currentDate.getDate() + '_dump.sql.gz';
                        name = backupDirPath + '/' + fName;
                        console.log('starting bakup',name);
                        await mysqldump({
                                connection: con,
                                dumpToFile: name,
                                compressFile: true,
                        }),then(async (res)=>{
                                await this.uploadFile(fName, element.bPath);
                        }).catch(error=>{
                                console.log(error)
                        });
                        console.log('end backup');
                        console.log('upload start');
                }
        }
        return;

exports.uploadFile = async (fileName, key) => {
        // Read content from the file
        console.log('reading file');
        const fileContent = fs.readFileSync(backupDirPath + '/' + fileName);
        console.log('file readed');
        // Setting up S3 upload parameters
        const params = {
                Bucket: bucket,
                Key: key + '/' + fileName, // File name you want to save as in S3
                Body: fileContent
        };
        // Uploading files to the bucket
        console.log('uploading...');
        await s3.upload(params, function (err, data) {
                if (err) {
                        throw err;
                }
                console.log(`File uploaded successfully. ${data.Location}`);
        }).promise();
        console.log(fileName);
        console.log('uploaded!');
};

I mean... Nothing has changed from this side (no releases in a while), so if it's recently just stopped working - it's likely something on your end?

It was working fine but now it broken. I am not able to identify why this is happening now
you can check complete code here
https://github.com/Devshail/Database-backup-scheduler

again - we haven't done a release in over 6 months. So if something has started happening recently - likely either something on your end.
there's absolutely no way for me to reproduce your dev environment - so there's no way for me to reproduce this for you, so you'll have to help debug and look into this.

can you please do this in Linux environment  I am facing issue over there

I do not have access to a linux environment.

you'll have to help debug and look into this.

I observed the same thing 2-3 weeks ago. We are using an alpine based docker image and installing the latest node 14.x version in it. Since I cannot reproduce this issue on my dev system (outside of a docker container) I tend to assume that alpine is the issue here.
But since we are migrating away from MySQL we didn‘t invest in tracking down the root cause but  implemented the workaround to use uncompressed backups and then stream compress them on the fly while uploading to S3...

I have this issue too, on my localhost (Mac, node v15.6.0) and my production server (Ubuntu) as of 5 days ago. It has been working for a while but I just noticed the problem.
node:events:356
      throw er; // Unhandled 'error' event
      ^

Error: ENOENT: no such file or directory, open './db_20210222.sql.gz.temp'
Emitted 'error' event on ReadStream instance at:
    at emitErrorNT (node:internal/streams/destroy:188:8)
    at emitErrorCloseNT (node:internal/streams/destroy:153:3)
    at processTicksAndRejections (node:internal/process/task_queues:81:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: './db_20210222.sql.gz.temp'
}

Update: The issue is with the compressFile option. If I omit this param (default: false) then the dump to .sql performs without error.

I noticed this issue on my side as well, by reusing the function compressFile in my code. The issue @bradzacher is that in the deleteFile function, sometimes for some reason the file to delete does not exists, so the unlink fails, and for some reason I don't understand, the catch does not work. So my suggestion is to first check the existence of the file before the unlink:
from currently:
function compressFile(filename: string): Promise<void> {
//...
  const deleteFile = (file: string): void => {
    try {
      fs.unlinkSync(file);
    } catch (_err) {
      /* istanbul ignore next */
    }
  };
// ...
}

change to:
function compressFile(filename: string): Promise<void> {
//...
  const deleteFile = (file: string): void => {
    try {
      if (fs.existsSync(file)) { // <-- FIX IS HERE
        fs.unlinkSync(file);
      }
    } catch (_err) {
      /* istanbul ignore next */
    }
  };
// ...
}

Regards,

Same issue as #111

The reasons why the error is not caught in the catch block could have been given in this comment:
https://github.com/nodejs/node/issues/9455#issuecomment-258315799
The comment is dated from Nov. 2016. I don't know if something has been done since then, but apparently, it is originated in the 'stream' library (probably called internally by the fs.unlinkSync method) which would try to stat the file after it has been deleted. Anyway, wrapping this line with the if condition might be a good solution for now.