Calculation Failure in new Coverage Planner (mowing are turned at 0,0 out of perimeter area)

With the current version of the mow coverage planner I have the problem that I get with "lines" a mowing area that is projected in the wrong place (see picture). It appears to be rotated beyond the 0 point to the perimeter and exclusion.
If I leave out the exclusion in the calculation, the mowing paths come correctly within the perimeter.

Both still worked correctly with the old PathPlanner.
With "rings" it would even work correctly. With "squares":

At/up to 99° MowAngle everything is still OK. At/from 100° it no longer works.
Issue exists e.g. with
{
"pattern": "lines",
"width": 0.18,
"angle": 134,
"distancetoborder": 1,
"mowarea": "yes",
"mowborder": 2,
"mowexclusion": "yes",
"mowborderccw": "yes"
}

Oh yes, totally forgot about you!
At first make an update to the last verstion

Do you have still the problem with  last version?

Yes, the issue still stays with actual version.
And in the long ArduMower thread there was a second one with the same behavior. But I do not know who and when it the thread it was.

Can you share you log again
Thx

The issue appears at 10:31:00
cassandra.log

Please in DEBUG Level

How to do it in actual versions?

Just start the app with
./app.py --app_log_level "DEBUG"
to log in debug level to journalctl
or
./app.py --app_file_log_level "DEBUG"
to log in debug level to .cassandra/log directory

The sent log file is the output I get with
./app.py --app_log_file_level "DEBUG"
The direct output is the same and should be DEBUG by default as you described in the documentation.

./app.py --app_log_level "DEBUG"

OK, I was successful with
./app.py --app_log_level "DEBUG" >output.txt
The default of the direct app output seems to be INFO, not DEFAULT as described.
and --app_log_file_level seems not changing the level in the cassandra.log file
output.txt

Can you share your
pip list
output

Package                   Version

arandr                    0.1.10
astroid                   2.5.1
asttokens                 2.0.4
automationhat             0.2.0
beautifulsoup4            4.9.3
blinker                   1.6.2
blinkt                    0.1.2
buttonshim                0.0.2
Cap1xxx                   0.1.3
certifi                   2022.12.7
chardet                   4.0.0
charset-normalizer        3.1.0
click                     8.1.3
colorama                  0.4.4
colorzero                 1.1
cryptography              3.3.2
cupshelpers               1.0
dash                      2.9.3
dash-bootstrap-components 1.4.1
dash-core-components      2.0.0
dash-daq                  0.5.0
dash-html-components      2.0.0
dash-table                5.0.0
dbus-python               1.2.16
distro                    1.5.0
docutils                  0.16
drumhat                   0.1.0
envirophat                1.0.0
executing                 2.0.0
ExplorerHAT               0.4.2
Flask                     2.3.1
fourletterphat            0.1.0
gpiozero                  1.6.2
html5lib                  1.1
icecream                  2.1.3
idna                      3.4
importlib-metadata        6.6.0
isort                     5.6.4
itsdangerous              2.1.2
jedi                      0.18.0
Jinja2                    3.1.2
lazy-object-proxy         0.0.0
logilab-common            1.8.1
lxml                      4.6.3
MarkupSafe                2.1.2
mccabe                    0.6.1
microdotphat              0.2.1
mote                      0.0.4
motephat                  0.0.3
mypy                      0.812
mypy-extensions           0.4.3
networkx                  3.1
numpy                     1.24.3
oauthlib                  3.1.0
opencv-python             <IP_ADDRESS>
packaging                 23.1
paho-mqtt                 1.6.1
pandas                    2.0.1
pantilthat                0.0.7
parso                     0.8.1
pexpect                   4.8.0
pgzero                    1.2
phatbeat                  0.1.1
pianohat                  0.1.0
picamera2                 0.3.9
pidng                     4.0.9
piexif                    1.1.3
piglow                    1.2.5
pigpio                    1.78
Pillow                    10.0.1
pip                       23.2.1
plotly                    5.14.1
psutil                    5.8.0
pycairo                   1.16.2
pycups                    2.0.1
pygame                    1.9.6
Pygments                  2.7.1
PyGObject                 3.38.0
pyinotify                 0.9.6
PyJWT                     1.7.1
pylint                    2.7.2
PyOpenGL                  3.1.5
pyOpenSSL                 20.0.1
PyQt5                     5.15.2
PyQt5-sip                 12.8.1
pyserial                  3.5
pysmbc                    1.0.23
python-apt                2.2.1
python-dateutil           2.8.2
python-dotenv             1.0.0
python-prctl              1.7
pytz                      2023.3
rainbowhat                0.1.0
reportlab                 3.5.59
requests                  2.31.0
requests-oauthlib         1.0.0
responses                 0.12.1
roman                     2.0.0
RPi.GPIO                  0.7.0
RTIMULib                  7.2.1
scrollphat                0.0.7
scrollphathd              1.2.1
Send2Trash                1.6.0b1
sense-hat                 2.4.0
setuptools                52.0.0
shapely                   2.0.1
simplejpeg                1.6.4
simplejson                3.17.2
six                       1.16.0
skywriter                 0.0.7
sn3218                    1.2.7
soupsieve                 2.2.1
spidev                    3.5
ssh-import-id             5.10
tenacity                  8.2.2
tflite-runtime            2.11.0
thonny                    4.0.1
toml                      0.10.1
touchphat                 0.0.1
twython                   3.8.2
typed-ast                 1.4.2
typing-extensions         <IP_ADDRESS>
tzdata                    2023.3
unicornhathd              0.0.4
urllib3                   1.26.15
v4l2-python3              0.3.2
webencodings              0.5.1
Werkzeug                  2.3.0
wheel                     0.34.2
wrapt                     1.12.1
zipp                      3.15.0

i've a simular (or maybe the same) behavior.

I have installed slightly fewer packages. Maybe this info will help you a bit
Package                   Version

asttokens                 2.4.0
blinker                   1.6.2
certifi                   2020.6.20
chardet                   4.0.0
charset-normalizer        3.2.0
click                     8.1.3
colorama                  0.4.6
colorzero                 1.1
dash                      2.9.3
dash-bootstrap-components 1.4.1
dash-core-components      2.0.0
dash-daq                  0.5.0
dash-html-components      2.0.0
dash-table                5.0.0
distro                    1.5.0
executing                 2.0.0
Flask                     2.3.2
gpiozero                  1.6.2
icecream                  2.1.3
idna                      2.10
importlib-metadata        6.8.0
itsdangerous              2.1.2
Jinja2                    3.1.2
MarkupSafe                2.1.3
networkx                  3.1
numpy                     1.24.3
packaging                 23.1
paho-mqtt                 1.6.1
pandas                    2.0.1
picamera2                 0.3.9
pidng                     4.0.9
piexif                    1.1.3
Pillow                    10.0.1
pip                       20.3.4
plotly                    5.14.1
Pygments                  2.16.1
pyserial                  3.5
python-apt                2.2.1
python-dateutil           2.8.2
python-prctl              1.7
pytz                      2023.3
requests                  2.31.0
RPi.GPIO                  0.7.0
setuptools                52.0.0
shapely                   2.0.1
simplejpeg                1.6.4
six                       1.16.0
spidev                    3.5
ssh-import-id             5.10
tenacity                  8.2.2
toml                      0.10.1
tzdata                    2023.3
urllib3                   1.26.5
v4l2-python3              0.3.2
Werkzeug                  2.3.6
wheel                     0.34.2
zipp                      3.16.2

I added new branch turn-figure-tests. Pls do the same again and post logfile here.
thanks

I'm not shure if the output worked as wanted. Additionally to the logfile I got the following output in the console:
./app.py --app_log_level "DEBUG" >output.txt
ic| figure: <POINT (0 0)>
ic| figure_rotated: <POINT (0 0)>
ic| figure: <POLYGON ((4.25 -2.63, 4.76 -2.85, 6.27 -2.96, 7.22 -3.28, 7.67 -3.23, 8.29 ...>
ic| figure: <POLYGON ((4.25 -2.63, 4.76 -2.85, 6.27 -2.96, 7.22 -3.28, 7.67 -3.23, 8.29 ...>
ic| figure: <LINESTRING (3.82 -2.35, 1.88 -4.19, 2.17 -5.18, 6.17 -9.33, 7.2 -8.69, 8.56...>
ic| figure_rotated: <LINESTRING (-0.963 4.38, 1.708 4.263, 2.219 5.159, 2.425 10.919, 1.25 11.21...>
output.txt

Pls try now

ic| figure: <POINT (0 0)>
ic| figure.is_valid: True
ic| figure_rotated: <POINT (0 0)>
ic| figure.is_valid: True
ic| figure: <POLYGON ((4.25 -2.63, 4.76 -2.85, 6.27 -2.96, 7.22 -3.28, 7.67 -3.23, 8.29 ...>
ic| figure.is_valid: True
ic| figure: <POLYGON ((4.25 -2.63, 4.76 -2.85, 6.27 -2.96, 7.22 -3.28, 7.67 -3.23, 8.29 ...>
ic| figure.is_valid: True
ic| figure: <LINESTRING (3.82 -2.35, 1.88 -4.19, 2.17 -5.18, 6.17 -9.33, 7.2 -8.69, 8.56...>
ic| figure.is_valid: True
ic| figure_rotated: <LINESTRING (-0.963 4.38, 1.708 4.263, 2.219 5.159, 2.425 10.919, 1.25 11.21...>
ic| figure.is_valid: True
output.txt

I'm slowly running out of ideas about what that could be. Which Python version are you using?

I tried several versions. At the past we compared and tried it with the same version.
I just upgraded to the latest delivery of 64bit bullseye. At the moment I use python 3.9.2

I checked something new to the test branch. Pls test

The error message is "Points of LinearRing do not form a closed linestring" But the exclusion is an exactly closed linestring.
Now, I manually manipulated the perimeter file. I moved the endpoint of the exclusion to 1µm distance to the start point of the exclusion.
And you will not believe it: Now, it works as it shall.
I don't know, what the problem in the algorithm is. But I have a solution for me. I will use the manipulated perimeter file in future. So I'm able to mow lines with the planned mow angle.

Can you share the log file. With new version just points are rotated, it is very strange that you get this error message

This way works, too.
ic| figure: <POINT (0 0)>
ic| figure.is_valid: True
ic| figure_rotated: <POINT (0 0)>
ic| figure.is_valid: True
ic| figure: <POLYGON ((4.25 -2.63, 4.76 -2.85, 6.27 -2.96, 7.22 -3.28, 7.67 -3.23, 8.29 ...>
ic| figure.is_valid: True
ic| figure_rotated: <POLYGON ((-5.357 -1.444, -6.485 -2.454, -7.375 -2.915, -7.651 -3.274, -9.67...>
ic| figure.is_valid: True
ic| figure: <LINESTRING (-4.344 -1.115, -4.32 1.558, -5.234 2.037, -10.997 2.043, -11.25...>
ic| figure.is_valid: True
ic| figure_rotated: <LINESTRING (3.82 -2.35, 1.88 -4.19, 2.17 -5.18, 6.17 -9.33, 7.2 -8.69, 8.56...>
ic| figure.is_valid: True
output.txt


[Test] Syntax highlighting in SQL mode is missing

The test version has no syntax highlighting in the SQL view. This worked in the old version.

fixed with fb1afa3a7567ab4d723307ad7ded022e0d9c7bc6


⚠ SceneTime has degraded performance

In fd26f9e, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 11734 ms

Resolved: SceneTime performance has improved in 9fba4b0 after 5 minutes.


Running Freertos with litex-neor32

Hi,
I’m success to build and flash fpga bitstream using LiteX and neorv32 core. I’m able to see the LiteX bios.
I would like to run freertos application with this soc but I can not find the instruction/examples.
Can you please give me few hint/example on running freertos on LiteX-Nerov32?
Thanh you!

Hey there!

I would like to run freertos application with this soc but I can not find the instruction/examples.

There are no instructions or examples yet. However, running FreeRTOS on a NEORV32-based LiteX system is also on my to-do list :wink:
Here are some basic steps/instructions. You could start with the NEORV32 FreeRTOS port in sw/example/demo_freeRTO.

First, you need to adapt the linker script (sw/common/neorv32.ld) so it matches the memory configuration of your LiteX setup.
Then you need to adapt the HAL so your program uses the GPIO and UART of the LiteX setup and not the default NEORV32 ones. The machine time, which is responsible for the RTOS tick, does not have to be modified as this module is part of the "NEORV32 LiteX Core Complex.


WACK test fail after upgrade to Anniversary SDK (build 14393)

Hello,
I do a release build and WACK test on a regular basis during my development.  You never know!  :-)
I allowed my VS 2015 update 3 to be applied along with the Windows 10 Anniversary SDK (build 14393).
Since allowing those updates my app still runs in debug fine and does do a release build fine but fails the WACK on the prelaunch.  Section from WACK results pasted below:
=======
FAILED
App prelaunch
•Error Found: The app prelaunch validation detected the following errors:◦The app failed prelaunch test - 62323CMarassovich.membershipDB_<IP_ADDRESS>_x86__wxeygqgheeyfp.
•Impact if not fixed:  The app will take a longer time to launch, even when prelaunch is enabled.
•How to fix: In the OnLaunched method implementation of the app, ensure you handle the LaunchActivatedEventArgs.PreLaunch option to be prelaunch event aware.
========
I am on T10 nuget 1.1.11
I thought this will possibly be fixed in the T10 nuget for the Anniversary update but then I thought best to raise this just incase.
Any advice is greatly appreciated as well as your time.
Cheers
Chris ...

Hello all,
I'm having the same problem after updating the WACK tool / installing Update 3 for Visual Studio. I tried to rerun a WACK test with a previous app bundle(from a month ago). This also gave the same result as the above mentioned problem.
Any help on how to fix this would be greatly appreciated.

Well, this error with WACK has come back to haunt us. Perhaps the new WACK has changed how it measures prelaunch in the AU version. Let's see.

Good morning Jerry and thanks for your time and effort to respond.
Your comment on "AU Region":

are you saying that the WACK will evaluate differently based on a region?
You are not able to reproduce it?
:-) Or are you having a dig at an Aussie?  I believe your origins are Down Under also so I will take that Dig at me.  :-)

NOTE: I hope you see the smiley faces on point 3 and see it for the fun it was meant to be.

You guys are far smarter than I and I would never fiddle with stuff I do not understand.  I do not understand Prelaunch so I have not touched it.  My app uses the Hamburger template and I have very much left it stock with the exception of using EFCore and in the App.xaml.cs in the OnStartAsync I have the EFCore db.Database.Migrate() command.  Very stock stuff I think.
I possibly did not make it clear in my first post but I run a release build/WACK often.  Always before and after a major change (I have been caught out with another project).  So my app passed the WACK before the SDK upgrade and right after the SDK upgrade it was failing on Prelaunch.
Just before writing this post response I did a new T10 Hamburger and release build and WACK and it is failing on  Prelaunch.
It is good news from JWH89 that this does not seem impact the Store Submission.
I am happy to provide further information or testing as needed.
Would like to state it again T10 and all you guys Rock!
Cheers
Chris Marassovich

Apparently this is a known issue, got this from the MSDN forums:

This is a known issue with running the test on Win10 (1511) using the latest version of the SDK.  If you use Win10 (1607) it should work correctly.  If the app passes fine using that combination of the OS+SDK, then you can definitely ignore the Win10 (1511) results.

PS: I do not use the OnPrelaunchAsync, that is why I was confused it was giving this error.

Dev from down under as well, also getting this after upgrading to the latest SDK.
I am on Win10 1511 and T10 1.1.11 and not doing any special handling for prelaunch.

@Chris-Marassovich - not AU region, AU version.  There is a new version of the SDK to work with the Anniversary Update. I don't quite understand the problem, though. I'm not handling prelaunch in my projects either, but I'm not experiencing this problem. I can't get the CK to fail the prelaunch test.
Are you using the latest version of T10?

@callummoffat - My mistake on the AU region :-)  Saw AU and jumped to it being Australian settings.  Oops sorry.
As I said in my original post the version of T10 I am using is the Nuget 1.1.11 version.

I'm not seeing this issue. I'm using the default OnPrelaunchAsync implementation, on the Anniversary Update SDK, with T10 v1.1.11. I get the full WACK pass (although, I am running the Anniversary Update, so that could be the reason).

@callummoffat  - Just to be clear in your last sentence there are you saying your Win10 has been updated to the AU version?
If so then that makes a little sense that the AU SDK works a little better for WACK testing on the AU version of the OS.  :-)
I have seen lots of press on the AU version of the OS but very little detail on when we will get it.
Fingers crossed when the AU OS update arrives it solves my issue.
Cheers
Chris ...

Yes, my OS has been updated to the Anniversary Update. That must be it.
On Thu, Aug 18, 2016 at 12:24 PM Chris-Marassovich<EMAIL_ADDRESS>wrote:

@callummoffat https://github.com/callummoffat - Just to be clear in
your last sentence there are you saying your Win10 has been updated to the
AU version?
If so then that makes a little sense that the AU SDK works a little better
for WACK testing on the AU version of the OS. :-)
I have seen lots of press on the AU version of the OS but very little
detail on when we will get it.
Fingers crossed when the AU OS update arrives it solves my issue.
Cheers
Chris ...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Windows-XAML/Template10/issues/1165#issuecomment-240589374,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM3pIznxwe4kPHLGR_47Bk1jLHIhYSzxks5qg6YqgaJpZM4Jfn6N
.

--
Thanks,
Callum

Good morning everyone,
My PC has just updated the OS to the AU version and the WACK pre-launch fail now is a pass.
So to summarise:

If your OS is at a pre-AU version but you install the AU SDK you will get a WACK fail in the pre-launch.
If your OS is at AU version and you have the AU SDK your WACK will not fail on the pre-launch text.
As you suspected @callummoffat this issue was a combo of AU SDK and non AU OS.

Thanks to everyone for helping and posting.
Cheers
Chris ...
PS Not only is T10 a great tool but this repo and samples is making me a better developer.  Thank you!

Just to note: post-AU preview builds will also pass certification when using the AU SDK.


workout-tracker: 1.16.1 -> 1.18.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for workout-tracker is: Workout tracking web application for personal use
meta.homepage for workout-tracker is: https://github.com/jovandeginste/workout-tracker
meta.changelog for workout-tracker is: https://github.com/jovandeginste/workout-tracker/releases/tag/v1.18.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.18.1 in filename of file in /nix/store/dkbd3vjjysb98rpx186sbk2bgm050dpi-workout-tracker-1.18.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

workout-tracker

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dkbd3vjjysb98rpx186sbk2bgm050dpi-workout-tracker-1.18.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A workout-tracker https://github.com/r-ryantm/nixpkgs/archive/89a9d23f8e726bc187556a231c52245fec35e042.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/89a9d23f8e726bc187556a231c52245fec35e042#workout-tracker

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dkbd3vjjysb98rpx186sbk2bgm050dpi-workout-tracker-1.18.1
ls -la /nix/store/dkbd3vjjysb98rpx186sbk2bgm050dpi-workout-tracker-1.18.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

workout-tracker

Maintainer pings
cc @bhankas for testing.

@NixOS/nixpkgs-merge-bot merge


update foreign key object

hi, when i try updating an object with foreign keys,  i only get a drop down option for related foreign key objects but i can't edit the actually foreign key object. is it possible to update foreign key object like we have in django admin?
thanks in advance.

Hey @igabice, the idea is cool, would be really nice to implement this. I'm going to mark it as enhancement and let's see if we can make it posible. Thanks for your words.


MAINT: integrate.qmc_quad: correct behavior of parameter log

Reference issue
None. Might as well just fix it.
What does this implement/fix?
This adjusts the output of qmc_quad when log=True.
When log=False, the mean of the independent integral estimates is an unbiased estimator of the true integral, so when log=True, we should be returning the log of this quantity. But currently, we return the mean of the log of the integral estimates, which is not the same because the order of taking the log and the mean matters. This PR corrects this.
Similarly, when log=True, we currently return the SEM of the log of the integral estimates; this PR would return the log of the SEM of the integral estimates.
Numerically, the difference is small when the independent estimates are accurate, which is why the tests didn't catch it. This PR improves the sensitivity of the relevant test so that it would fail in main.
Additional information

Test scripts for the mean, std, and sem functions defined within qmc_quad:
import numpy as np
from numpy.testing import assert_allclose, assert_equal
from scipy import stats, optimize, integrate
import matplotlib.pyplot as plt
rng = np.random.default_rng(1638083107694713882823079058616272161)
from scipy.special import logsumexp

n_estimates = 20
x = rng.random(n_estimates)

def sum_product(integrands, dA, log=False):
    if log:
        return logsumexp(integrands) + np.log(dA)
    else:
        return np.sum(integrands * dA)

def mean(estimates, log=False):
    if log:
        return logsumexp(estimates) - np.log(n_estimates)
    else:
        return np.mean(estimates)

def std(estimates, m=None, ddof=0, log=False):
    m = m or mean(estimates, log)
    if log:
        estimates, m = np.broadcast_arrays(estimates, m)
        temp = np.vstack((estimates, m + np.pi * 1j))
        diff = logsumexp(temp, axis=0)
        return np.real(0.5 * (logsumexp(2 * diff)
                              - np.log(n_estimates - ddof)))
    else:
        return np.std(estimates, ddof=ddof)

def sem(estimates, m=None, s=None, log=False):
    m = m or mean(estimates, log)
    s = s or std(estimates, m, ddof=1, log=log)
    if log:
        return s - 0.5 * np.log(n_estimates)
    else:
        return s / np.sqrt(n_estimates)

print(np.mean(x))
print(mean(x, log=False))
print(np.exp(mean(np.log(x), log=True)))

print(np.std(x, ddof=1))
print(std(x, ddof=1, log=False))
print(np.exp(std(np.log(x), ddof=1, log=True)))

print(stats.sem(x))
print(sem(x, log=False))
print(np.exp(sem(np.log(x), log=True)))

The relative error is not so different between platforms; it's the absolute error that is varying on the order of 1e-17; bumping it to 2e-17 wasn't quite enough. I wasn't concerned about the relative error because I think it comes from cancellation in the variance calculation. There is little variance, but the mean is nonzero, so we lose some digits when subtracting the mean.

CI failures are unrelated; they're in main.


EnvoyFilter Patch doesn't work per documentation

Bug description
Managing NETWORK_FILTER via EnvoyFilter is not working as expected.  Applying the manifest referenced in the documents https://istio.io/docs/reference/config/networking/envoy-filter/ and https://github.com/istio/istio/wiki/EnvoyFilter-Samples has no effect.
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: xff
  namespace: istio-system
spec:
  configPatches:
  - applyTo: NETWORK_FILTER
    match:
      context: ANY
      listener:
        filterChain:
          filter:
            name: "envoy.http_connection_manager"
    patch:
      operation: MERGE
      value:
        typed_config:
          "@type": "type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager"
          idle_timeout: 299s
          xff_num_trusted_hops: 1

Also tried with context: GATEWAY,  no effect. no logs in pilot.
Expected behavior
idle_timeout and xff_num_trusted_hops values should reflect in filter envoy.http_connection_manager config.
Steps to reproduce the bug
Try any Network filter patch documented in https://github.com/istio/istio/wiki/EnvoyFilter-Samples
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
Istio 1.4.6
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.9", GitCommit:"2e808b7cb054ee242b68e62455323aa783991f03", GitTreeState:"archive", BuildDate:"2020-01-21T20:41:54Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.9", GitCommit:"2e808b7cb054ee242b68e62455323aa783991f03", GitTreeState:"archive", BuildDate:"2020-01-21T20:41:54Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)

kind: EnvoyFilter
metadata:
  name: xff
  namespace: istio-system
spec:
  configPatches:
  - applyTo: NETWORK_FILTER
    match:
      context: ANY
      listener:
        filterChain:
          filter:
            name: "envoy.http_connection_manager"
    patch:
      operation: MERGE
      value:
        typed_config:
          "@type": "type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager"
          xff_num_trusted_hops: 1
          common_http_protocol_options:
            idle_timeout: 299s

Apply this instead when you are using istio >= 1.4

kind: EnvoyFilter
metadata:
  name: xff
  namespace: istio-system
spec:
  configPatches:
  - applyTo: NETWORK_FILTER
    match:
      context: ANY
      listener:
        filterChain:
          filter:
            name: "envoy.http_connection_manager"
    patch:
      operation: MERGE
      value:
        typed_config:
          "@type": "type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager"
          xff_num_trusted_hops: 1
          common_http_protocol_options:
            idle_timeout: 299s

Apply this instead when you are using istio >= 1.4

Error from server: error when creating "test.yaml": admission webhook "pilot.validation.istio.io" denied the request: configuration is invalid: Envoy filter: can't unmarshal Any nested proto *envoy_config_filter_network_http_connection_manager_v2.HttpConnectionManager: unknown field "common_http_protocol_options" in envoy_config_filter_network_http_connection_manager_v2.HttpConnectionManager

i find the same problem in istio 1.3.8 with default envoy 1.12.0.
from envoydocs 1.12.0 also use the common_http_protocol_options.idle_timeout.
envoy recommend config
but i can not set common_http_protocol_options.idle_timeout.

Hi all, in case it helps, with Istio 1.5, this works:
$ cat envoyfilter3.yaml
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
name: hcm-tweaks
namespace: istio-system
spec:
configPatches:

applyTo: NETWORK_FILTER # http connection manager is a filter in Envoy
match:
context: GATEWAY
listener:
filterChain:
filter:
name: "envoy.http_connection_manager"
patch:
operation: MERGE
value:
typed_config:
"@type": "type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager"
xff_num_trusted_hops: 5
max_request_headers_kb: 90

Main difference versus the doc: removed WorkloadSelector, removed sni, added type_config."@type"

I am running into a similar issue trying to get the new envoy filter working in an istio 1.5.4 cluster: I get the following error from the following yaml:
Error [IST0106] (EnvoyFilter envoy-filter-1-6.istio-system) Schema validation error: Envoy filter: unknown field "typed_config" in envoy_api_v2.Listener
envoy-filter-1-6.yaml.zip

Lots of issues here:

First was invalid config, which was updated
Then we had not updated to include filters, which is fixed in #23932
 Envoy filter: unknown field "typed_config" in envoy_api_v2.Listener - I suspect wrong version of istioctl or something... not sure. I tried in my cluster and it works. Can you open a new issue with more details about version, config dump?

i am running on istio 1.6.0 cluster, and get errors:
configuration is invalid: Envoy filter: can't unmarshal Any nested proto *envoy_config_filter_network_http_connection_manager_v2.HttpConnectionManager: unknown field "common_http_protocol_options" in envoy_config_filter_network_http_connection_manager_v2.HttpConnectionManager
and my envoyfilter is follows
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: h2-control
spec:
  configPatches:
  - applyTo: NETWORK_FILTER # http connection manager is a filter in Envoy
    match:
      listener:
        filterChain:
          filter:
            name: "envoy.http_connection_manager"
    patch:
      operation: MERGE
      value:
        typed_config:
          "@type": "type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager"
          stream_idle_timeout: 60s
          request_timeout: 60s
          http2_protocol_options:
            max_concurrent_streams: 100
            initial_stream_window_size: 65536 # 64 KiB
            initial_connection_window_size: 1048576 # 1 MiB
          common_http_protocol_options:
            idle_timeout: 60s # 1 hour

I am having a similar issue with 1.9.x docs. EnvoyFilter applies as expected, but changes nothing in the proxy configs when I inspect them using istioctl
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: custom-protocol
  namespace: istio-config # as defined in meshConfig resource.
spec:
  configPatches:
  - applyTo: NETWORK_FILTER # http connection manager is a filter in Envoy
    match:
      # context omitted so that this applies to both sidecars and gateways
      listener:
        filterChain:
          filter:
            name: "envoy.filters.network.http_connection_manager"
    patch:
      operation: MERGE
      value:
        name: "envoy.filters.network.http_connection_manager"
        typed_config:
          "@type": "type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager"
          common_http_protocol_options:
            idle_timeout: 3600s # 1 hour
          http2_protocol_options:
            max_concurrent_streams: 500
            initial_stream_window_size: 65536 # 64 KiB
            initial_connection_window_size: 1048576 # 1 MiB
          stream_idle_timeout: 300s # 5 mins, must be disabled for long-lived and streaming requests
          request_timeout: 300s # 5 mins, must be disabled for long-lived and streaming requests


Allow spaces in ssoAppName:

On Mac having a space in the App name causes an Unhandled Promise Rejection due to not wrapping the name in quotes. Issue originally reported here:
https://github.com/OfficeDev/generator-office/issues/539

wrap ssoAppName in  quotes for consistency between mac an windows

I can confirm that this fixes new project generation on Mac OSX when there is a space in the project name.

/Azurepipelines run

Thanks for the contribution!  Sorry for the delay in approving it.


Body related parameters don't take alias in consideration

Privileged issue

[X] I'm @tiangolo or he asked me directly to create an issue here.

Issue Content
The Body, File, and Form parameters don't support alias.
The following snippet demonstrates the issue:
from typing import Annotated

from fastapi import Body, Cookie, FastAPI, Form, UploadFile, File, Query, Header, Path

app = FastAPI()

@app.post("/{path}")
def endpoint(
    path: Annotated[int, Path(alias="PathAlias")],
    cookie: Annotated[int, Cookie(alias="CookieAlias")],
    header: Annotated[int, Header(alias="HeaderAlias")],
    query: Annotated[int, Query(alias="QueryAlias")],
    body: Annotated[int, Body(alias="BodyAlias")],
    form: Annotated[int, Form(alias="FormAlias")],
    file: Annotated[UploadFile, File(alias="FileAlias")],
):
    ...

If you look at the generated Swagger you see that alias is only being used for params.Param related fields i.e. Path, Cookie, etc.

People can overcome this right now using validation_alias as follows:
from typing import Annotated

from fastapi import Body, Cookie, FastAPI, Form, UploadFile, File, Query, Header, Path

app = FastAPI()

@app.post("/{path}")
def endpoint(
    path: Annotated[int, Path(alias="PathAlias")],
    cookie: Annotated[int, Cookie(alias="CookieAlias")],
    header: Annotated[int, Header(alias="HeaderAlias")],
    query: Annotated[int, Query(alias="QueryAlias")],
    body: Annotated[int, Body(validation_alias="BodyAlias")],
    form: Annotated[int, Form(validation_alias="FormAlias")],
    file: Annotated[UploadFile, File(validation_alias="FileAlias")],
):
    ...

In any case, this should be fixed in FastAPI.

When I use only validation_alias in Form I can see alias in Swagger, but the validation is not working well.
For the moment I'm using both alias and validation_alias, It works.
I think issue is as FastAPI pass validation_alias to FieldInfo of Pydantic v2 in super().init () inside class Body

Here is one way to fix this issue in FastAPI:

Add support for alias in Body, Form, and File parameters similar to how it is already supported in Path, Cookie, etc.

This would involve:

Updating the Body, Form, and File classes to accept an alias argument.

Plumbing the alias through to OpenAPI/Swagger schema generation.

Updating the documentation to mention that alias is now supported.

Handle conflicts if alias and validation_alias are both provided.

For example, raise an exception if they don't match or take precedence of one over the other.

Add tests to validate the new alias functionality.

Submit a PR with the changes to the main FastAPI repo.

This would fix the issue so alias can be used directly instead of having to use validation_alias as a workaround. It keeps the API consistent across all the different parameter types.

I request a PR 10319 to fix it.
Now I can use alias in Form and File

It looks like this behavior might extend to Param types as well. Here's an example that includes Query, Path, Header, and Cookie, :
from fastapi import Query, Path, Header, Cookie
from pydantic import BaseModel, ConfigDict, Field

class ForbidExtra(BaseModel):
    model_config = ConfigDict(extra="forbid")

class FieldModel(ForbidExtra):
    type_: str | None = Field(default=None, alias="type_alias")

class QueryModel(ForbidExtra):
    type_: str | None = Query(default=None, alias="type_alias")

class CookieModel(ForbidExtra):
    type_: str | None = Cookie(default=None, alias="type_alias")

class HeaderModel(ForbidExtra):
    type_: str | None = Header(default=None, alias="type_alias")

class PathModel(ForbidExtra):
    type_: str | None = Path(alias="type_alias")

field = FieldModel(type_alias="alias")  # This behaves as expected.
# But these raise ValidationError:
query = QueryModel(type_alias="alias")
cookie = CookieModel(type_alias="alias")
header = HeaderModel(type_alias="alias")
path = PathModel(type_alias="alias")

@harol97, the workaround didn't work for me, am I doing something wrong?
    for route in app.routes:
        if isinstance(route, APIRoute):
            # TODO: body params aliases won't work in Swagger - https://github.com/tiangolo/fastapi/issues/10286
            _transform_snake_case(route.dependant.body_params)
            _transform_snake_case(route.dependant.query_params)
            _transform_snake_case(route.dependant.header_params)```

def _transform_snake_case(params: list[ModelField]) -> None:
    for param in params:
        camelized = humps.camelize(param.name)
        param.field_info.alias = camelized
        param.field_info.validation_alias = camelized

@hruzeda  you could first do:
camelized = humps.camelize(param.field_info.alias)

Any updates on this?


⚠ ClamAV has degraded performance

In eea5d47, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 198 ms

Resolved: ClamAV performance has improved in 7249494 after 6 minutes.


[TW#16110] Panic in uart tx ringbuffer

We have a uart with flow control at default baudrate, if the driver is under heavy load in both direction eventually we get a panic in ringbuf, we have seen three different places for the assert but all from tx ringbuffer
idf v2.1
esp-idf/components/freertos/ringbuf.c:345 configASSERT((hdr->flags & iflag_free)==0);
*0  0x40087278 in invoke_abort ()
at esp-idf/components/esp32/panic.c:125
*1  0x40087346 in abort ()
at esp-idf/components/esp32/panic.c:134
*2  0x4008689a in returnItemToRingbufDefault (rb=0x3ffdb158, item=0x3fff50a8)
at esp-idf/components/freertos/ringbuf.c:345
*3  0x40086efa in vRingbufferReturnItemFromISR (ringbuf=0x3ffdb158,
item=0x3fff50a8, higher_prio_task_awoken=0x3ffb0bc8)
at esp-idf/components/freertos/ringbuf.c:623
*4  0x400fe37d in uart_rx_intr_handler_default (param=0x3ffdaf0c)
at esp-idf/components/driver/uart.c:543
*5  0x40081919 in _xt_lowint1 ()
esp-idf/components/freertos/ringbuf.c:272 configASSERT((hdr->len < rb->size) || (hdr->flags & iflag_dummydata));
*0  0x40087288 in invoke_abort ()
esp-idf/components/esp32/panic.c:125
*1   0x40087288 in invoke_abort ()
esp-idf/components/esp32/panic.c:125
*2   0x4008696a in getItemFromRingbufDefault (rb=0x3ffdb1a8, length=0x3ffb0bcc,
wanted_length=0)
at esp-idf/components/freertos/ringbuf.c:272
*3   0x40086ec3 in xRingbufferReceiveFromISR (ringbuf=0x3ffdb1a8,
item_size=0x3ffb0bcc)
at esp-idf/components/freertos/ringbuf.c:582
*4  0x400fe3ad in uart_rx_intr_handler_default (param=0x3ffdaf5c)
at esp-idf/components/driver/uart.c:530
*5  0x40081919 in _xt_lowint1 ()
esp-idf/components/freertosingbuf.c:272 (getItemFromRingbufDefault)- assert failed!
0x40086972: getItemFromRingbufDefault at esp-idf/components/freertos/ringbuf.c
0x40086f2f: vRingbufferReturnItemFromISR at esp-idf/components/freertos/ringbuf.c
0x400fe3aa: uart_rx_intr_handler_default at esp-idf/components/driver/uart.c:919
0x40081916: _xt_lowint1 at xtensa_vectors.o:?

Hi, lollizze, I want to reproduce the problems you encountered. can you provide more details , such as test code?

Hi, here is how i configure the uart
uart_config_t config;
config->baud_rate = 115200,
config->data_bits = UART_DATA_8_BITS;
config->parity = UART_PARITY_DISABLE;
config->stop_bits = UART_STOP_BITS_1;
config->flow_ctrl = UART_HW_FLOWCTRL_CTS_RTS;
config->rx_flow_ctrl_thresh = 100;
espStatus = uart_param_config(0, &uart_config);
espStatus = uart_set_pin(0, GPIO_NUM_1, GPIO_NUM_3, GPIO_NUM_22, GPIO_NUM_19);
uart_driver_install(0,  2048 * 2,  0, 50, uartIsrEventQueue, 0);

the tx task
void uartTxTask()
{
    QMessage msg;
    cbUART_LOG("%s %s\n", __FUNCTION__, " task started");
    while (1)
    {
        if ( xQueueReceive(txQueue, &msg, (portTickType) portMAX_DELAY) )
        {
            uart_write_bytes(0, (cb_char*)msg.buf, msg.size);
        }
    }
    vTaskDelete(NULL);
}

we found this when sending/receiving tcp data

@koobest  yes you are right, i took the wrong version. When experience this i had 2048 as tx ringbuffer size,
Here is more info of how the data is sent

computer sends 1024 bytes at a time
Other side sends tcp data of size 512

module reads all available data from rx ringbuffer but max 2048 bytes
chunks it up into max 512 bytes and sends it on tcp
repeat 2

So, is the wrong version of IDF that caused this problem and will this problem appear now if use a new version of IDF?

This is from the latest release so I wouldn't say it is the wrong version of the IDF
uartIsrEventQueue is of type QueueHandle_t

@lollizze  hi  lollizze , It is a bit difficult for us to reproduce this problem. If you provide your test code, perhaps this problem is easy to reproduce and quickly resolved.

I don't have a small reproducibly code only my whole application


Performace issue

hi, I have tested your latest windows binary and latest wl-amigo's windows binary using one picture.
Your version's performace was around 148GFLOPS (on my rx 470) and wl-amigo's version was around 232GFLOPS.
How so your build is slower by a lot even when you are using latest opencv library?

Not sure, I changed nothing special other than the opencv version and the bin check, maybe the precompiled opencv library offered by them is not that good performing anymore?

I don't see anything different between tanakamura OpenCL version and DeadSix27 OpenCL version
I have a few question:
Did you have OpenCL in your computer?
Did you force the converter use your GPU instead of CPU? (Since newer gen CPU also have OpenCL)
did you facing the downclock glitch on your GPU? https://www.reddit.com/r/Amd/comments/5pg1vp/i_also_have_the_300_mhz_mem_clock_glitch_but_on/
It really weird that my GPU is much weaker than your but I have 300GFLOPS, my GPU is HD 7730

Weird indeed.
I have AMD APP SDK v3.0 installed
It is definitely using my gpu - at the start I can see OpenCL: Ellesmere (thats rx 470)
I am not sure forcing will help when it is already using my GPU
I do not experience such a glitch, also I am aware of downsides of Hynix memory timings. (msi rx 470 4gb)
Am I missing something?

@FluffyDiscord
Download the https://github.com/logos Octocat logo then use the JPEG!
And run the following using my latest w2x binary.
Find GPU: (which in my case is for sure Number 0)
G:\w2x\40>waifu2x-converter-cpp.exe --list-processor
   0: Hawaii                                       (OpenCL    ): num_core=44
   1: Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz     (FMA       ): num_core=8
   2: Intel(R) HD Graphics 530                     (OpenCL    ): num_core=24
   3: Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz     (OpenCL    ): num_core=8
   4: Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz     (OpenCL    ): num_core=8

Then run:
G:\w2x\40>waifu2x-converter-cpp.exe -i Octocat.jpg --processor 0
OpenCL: Hawaii
start process block (0,0) ...
......
process successfully done! (all:2.16128[sec], 763.748[GFLOPS], filter:1.88994[sec], 873.402[GFLOPS])

I then ran the same thing on tanakamura version:
G:\w2x\40\tanaka>waifu2x-converter_x64.exe -i Octocat.jpg --processor 0
OpenCL: Hawaii
process successfully done! (all:3.30762[sec], 499.052[GFLOPS], filter:1.89739[sec], 869.972[GFLOPS])

In which case mine is actually faster.

Tried your method, results are here
deadsix27, first try (I do not get this one)

deadsix27, second try

tanakamura, first try

tanakamura, second try

I see some weird performace drop from the start.
Scores are even with that second try.
So, no performace boost with some random "farts" now and then.

@FluffyDiscord the random performance dips can be anything, you doing something on your pc, your windows install doing something and so on. plus this software isn't flawless.

Alrigthy

Here I have about 930 gflops performance sometimes. R9 280

Codes from tanakamura fork are very outdated and not efficient for OpenCL computing
From the old benchmark, tanakamura only have 291GFLOPS while full performance is about 800 GFLOPS
I only have 300 GFLOPS while full performance is about 600 GFLOPS

@GiRaFa-SM The smaller the picture is, the higher your GFLOPS value will be =] Try to upscale some 4K picture and you will see that you get a lot less.

https://youtu.be/MBroPU2PGmg
600Gflops upscaling an 4k image using your waifu2x cmd+koroshell exe.


Fork/PR

This would be a great contribution back to the main /earth repository, and if it was a fork, it would be clearer to users that it is adapted from that project. Could this be redone as a contributable chunk rather than a cp?

A hat tip to http://hint.fm/wind/ for inventing this type of visualization would be a good idea too.


[Utopia 1178] add cpo review to PI PIA review section

Description
This PR includes the following proposed change(s):
-see title

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Updating Testing Framework(s)
[ ] Version change

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Development Dependency Working Agreement

[x] My code DOES NOT include the importing of new dependencies into the DPIA ecosystem
[ ] My code DOES include the importing of new dependencies into the DPIA ecosystem
If new dependencies are being introduced to the DPIA ecosystem:
[ ] The functionality of the dependency drastically reduces code complexity and makes my changes more easily maintainable and readible
[ ] The dependency being introduced does not contain multiple layers of nested dependencies introducing maintainability complexity to the DPIA ecosystem

Frontend Development Changes

[ ] N/A
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to project documentation or diagrams that reflect my changes
[ ] New and existing unit tests pass locally with my changes
[ ] My code follows Airbnb React Style Guidelines

API Development Changes

[x] N/A
[ ] I have performed a self-review of my own code
[ ] My code follows standards and practices outlined in the BC Government API Development Guidelines
[ ] New and existing unit tests pass locally with my changes
[ ] My changes includes Swagger documentation updates that reflect the changes I am introducing

Definition of Done

still wait backend change to finish, so add a label to not merge

The review part should be fully functional now, please take a look and test it.
for new developers, you must create a full PIA (aka non-delegate) so you can see cpo review section. but also please test the delegate flow to make sure I am not breaking any existing pieces.
in order to test add a new CPO reviewer button, you need to either log in as a different user or you can use a tool like pgAdmin to your local database to change the review column data.
the Known issue I am still working the add new CPO reviewer button need to click twice and it will change to view mode immeditary

A couple of bugs on the Review page:

I can submit to CPO_REVIEW even when MPO has not provided the review
Review can be submitted even when user did not acknowledge through the checkbox


hardcoded accounts

at https://github.com/luibo/KeycardForiPhone/blob/d28b6230bea06e57fd48b98c02cdadbb1323da28/KeycardSwiftIntegration/Controller/SelectWalletController.swift#L151
the accounts discovered on card are replaced by an hardcoded list.

Sorry, this was only for testing purposes. I'm updating the code as soon as
possible.
Il giorno mar 7 lug 2020 alle ore 15:48 Bitgamma<EMAIL_ADDRESS>ha scritto:

at
https://github.com/luibo/KeycardForiPhone/blob/d28b6230bea06e57fd48b98c02cdadbb1323da28/KeycardSwiftIntegration/Controller/SelectWalletController.swift#L151
the accounts discovered on card are replaced by an hardcoded list.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/luibo/KeycardForiPhone/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGICP4PJAJWUN4YIWVUS3BTR2MRUPANCNFSM4OS2X7FQ
.

I just commited the fix, the discovered accounts list is no longer
hardcoded. As I said, that was only for testing purposes, as I needed to
test the APIs with addresses that I was sure had been used for at least one
transaction.
Regarding what we discussed earlier, I understood your explanation, in the
next few days I'm changing the app protocol for card reading, even
providing an option for the user to unpair the card from a device.
One more thing, It often happens that the card loses connection with the
device while it's been read, especially during long NFC sessions. I was
wondering if it's some hardware problem related to the card itself or if it
could be a software problem, related to the SDK or to iOS libraries for NFC
cards reading.
Il giorno mar 7 lug 2020 alle ore 15:50 Luigi Borchia<EMAIL_ADDRESS>ha scritto:

Sorry, this was only for testing purposes. I'm updating the code as soon
as possible.
Il giorno mar 7 lug 2020 alle ore 15:48 Bitgamma<EMAIL_ADDRESS>ha scritto:

at
https://github.com/luibo/KeycardForiPhone/blob/d28b6230bea06e57fd48b98c02cdadbb1323da28/KeycardSwiftIntegration/Controller/SelectWalletController.swift#L151
the accounts discovered on card are replaced by an hardcoded list.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/luibo/KeycardForiPhone/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGICP4PJAJWUN4YIWVUS3BTR2MRUPANCNFSM4OS2X7FQ
.

this is most likely related to iOS itself. We tested on many Android phones and while result varies from model to model once you find the "sweet spot" it seems to be stable with no random drops.
In particular certain operations consume more power and if the iPhone's NFC field is weak the card might no have enough power during these operations, thus losing connection. Of course it is very possible that the software stack has bugs too either in the firmware or in the SDK (but in the SDK there isn't much low-level stuff).
The hardware of the card itself is the same you can find in many bank cards or national IDs (including the antenna and the card body) so I would exclude the possibility that that could be the source of the problem.


action:update should not overwrite params

Is your feature request related to a problem? Please describe.
When setting a param via aio runtime action update my/action --param KEY VALUE all other params are deleted.  I don't think this is expected.
Describe the solution you'd like
Only the specified params should be updated, all others should be left as is
Describe alternatives you've considered
I thought this was an error, so I tried the same thing in the wsk cli and it has the same behavior.
At a minimum we should document that this happens, with a warning.
We could also add a flag to specify the exact behavior we want.  A --force flag might be a better way of indicating that we want to kill all unspecified params.
Additional context
I think the use case to surgically update a single param without changing others is more valuable than the current implementation.
We also need to look at --env, --annotation

docs updated, this is deep in ow api territory and there is no easy way around this, the client cannot pull the params because they are encrypted and once deployed only available to the running action.


Add GetBlock to the list of users

What are you trying to accomplish?
Adding GetBlock, an RPC provider to the list of users.

What approach did you choose and why?

Anything you want to highlight for special attention from reviewers?

Hi @JoGetBlock can you please keep the list ordered alphabetically and add a Changelog entry? Thanks!


Roll stagehand pkg:angular version to beta

blocked on publishing angular beta

SGTM!

@kevmoo - will we be setting the min SDK to 2-dev.65 (or .66 if it is out)?

.65 is good
On Wed, Jun 27, 2018 at 1:21 PM Patrice Chalin<EMAIL_ADDRESS>wrote:

@kevmoo https://github.com/kevmoo - will we be setting the min SDK to
2-dev.65 (or .66 if it is out)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/stagehand/issues/547#issuecomment-400815916,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABCimWZLO1sDqBzSy_yyf-chMNctUI8ks5uA-kzgaJpZM4U4ydt
.

I suppose that we'll be waiting for the ACX beta release too, right? /cc @kwalrath

Yup!
On Wed, Jun 27, 2018 at 4:32 PM Patrice Chalin<EMAIL_ADDRESS>wrote:

I suppose that we'll be waiting for the ACX beta release too, right? /cc
@kwalrath https://github.com/kwalrath
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/stagehand/issues/547#issuecomment-400862621,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABCihSYDX-e3gsRujo--hIFR37imnueks5uBBYQgaJpZM4U4ydt
.


Error: unresolved external symbol PyInit_pydasm

Trying to build Pydasm on Windows 10 with Python 3.5 and Visual Studio 2015. Returns error:
LINK : error LNK2001: unresolved external symbol PyInit_pydasm
build\temp.win32-3.5\Release\./libdasm\pydasm.cp35-win32.lib : fatal error LNK1120: 1 unresolved externals
error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\BIN\\link.exe' failed with exit status 1120

@Starwarsfan2099 Hello, it may be a long time, but I encountered the same problem, did you solve this problem?

@ShadowLL I did not get it to build successfully for Python 3. I used Python 2.7 for the one tool I wanted to use this for instead.


[BACKLOG-10661] Update error dialog size and detail scroll

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f ui/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f ui/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  ui/src/org/pentaho/di/ui/core/dialog/DetailsDialog.java
  ui/src/org/pentaho/di/ui/core/dialog/ErrorWithDetailsDialog.java


Executing shortcut at wrong time

Kwm occasionally gets into a state where it doesn't know that a window is focused (I think. This is not kwms fault. Another app has opened something in another app, but kept the previous app focused). The window is in the foreground, but there is no border. I have kwmc bindsym_passthrough cmd-w press cmd+alt-q stubborn -i which quits the app if it is in the stubborn list. When kwm is in this state and I hit the shortcut, it quits the app regardless of if it is in the list.

Actually the weird state might be a kwm problem. Sometimes when I command tab, for example, from slack to chrome, it moves to the space that the chrome window is on, but the active application is still Slack on the other space. Might be related to #419?

Actually the weird state might be a kwm problem. Sometimes when I command tab, for example, from slack to chrome, it moves to the space that the chrome window is on, but the active application is still Slack on the other space. Might be related to #419?

Doesn't seem to be related to #419. Is this on master or v3.0.7?

I've been seeing this issue for awhile. I'm on master right now, but it was there at 3.0.7 as well.

ok so the issue is being caused by another app (https://github.com/johnste/finicky/issues/29) closing for now.


SML example in R

Jupyter notebook with R code for reading and plotting SML data + R functions in a separate file.

Thanks! Looks good 👍
Just please, put the content of the directory that you created into the directory notebooks. Or move the whole directory there.
It would be great if you could also updated the index.ipynb to contain a link to your SML example.


[SP-5766] Backport of PDI-18923 - Calculator Step: Date A - Date B(ho…

…urs) (9.1 Suite)
Cherry-pick of #7684 into 9.1 branch.
@ssamora @bcostahitachivantara

:white_check_mark: Build finished in 30m 5s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3738, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message


Fix selector for current profile's name

The element is now George Costanza

LGTM


question: Widget size

First of all, lovely animations!
I was using the built in CircularProgressIndicator and shifted to your packages, which looks awesome!
The only problem I have is that the built in Indicator sized itself according to the constraints (as far as I can tell without looking deep into the code).
For your widget, a size has to be set manually. Setting it to a sensible default value leads to "jumping" of entries in my ListView if some entry shows the loading widget and then the actual content.
Often times it is tedious to get the available height and then set the size of your loading widget to it. Is there a leaner way to do this?

Yes CircularProgressIndicator has fixed minWidth and minHeight constraints. For this package this won't work I have so many different types of animations and I need a fixed size 😞 For some animations the provided size is outer bound then there is an innerSize to keep the animation within.

I understand, thanks for the fast answer and the package! ❤


Wrapper is no longer recognizing set() and IsReady() functions for some reasons

Hey.
I'm trying to make a resource manager with raylib-cpp (nice wrapper btw) but I'm meeting a problematic issue : it seems like when I launch my program, no window appears, because the render loop doesn't launch. I just get the console telling me it "successfully" loaded the first sprite it found in the "assets" folder, then it stops.
I launch the GDB debugger and I get directly a segmentation fault, not really from my resource manager code, but from the "Texture.hpp" header file, at the 144th line with the error Use of undeclared identifier "set".. I also get the same error with the below line, with the IsReady() function.
 I don't think I forgot any necessary header files, or making really beginner-level mistakes - I could still be wrong, but it seems like it definitely come from the wrapper files. I am on Arch btw.
If needed, here is the original code.
Thanks in advance for the help.
code.txt

Certainly is strange. Here is the Load() function you're hitting...
    /**
     * Load texture from file into GPU memory (VRAM)
     */
    bool Load(const std::string& fileName) {
        set(::LoadTexture(fileName.c_str()));
        return IsReady();
    }

Since it's loading into VRAM, you will want to make sure the Window is loaded first. Use of undeclared identifier "set" is more curious though, since the set function definitely is part of Texture.

This works...
// Init
std::map<std::string, ptr<raylib::Texture>> textures {std::map<std::string, ptr<raylib::Texture>>()};

// Load
textures.emplace("ps3", std::make_unique<raylib::Texture>("resources/ps3.png"));

// Draw
textures["ps3"]->Draw(10, 10);

I'm on C++17 and it just doesn't work. The IDE has nothing to say when I browse the wrapper files by myself, but start to go nuts when I am trying to debug my program. I noticed I didn't say to you my resource manager is supposed to be static... if that can help.
I also tried your example, the same problem happens, it doesn't look to be related to the unique_ptr object but really with this set() weird problem.
Also, I asked CMake to only search and include the raylib library. Did I do well? Because I saw a Cmake template in the main README with a find_package(raylib_cpp) before including that library to the project.
I don't know the hell how library linking works (and I am not sure I want to) but my problem could be related to a missing link.

Are you able to compile one of the basic examples? Like https://github.com/RobLoach/raylib-cpp/blob/master/examples/core/core_basic_window.cpp ?

Yes, it worked.

Missed click.

I noticed I didn't say to you my resource manager is supposed to be static

Another note...static variables will be destroyed when the program terminates, so may want to be careful with that :wink:

That's intended, I anyways don't want to stock my resources for a longer time than the program lifetime itself :)
Just in case, compiling your example worked, but I still got this weird set() problem... heh, soy.

I really would like to get some help if you didn't give up heh

Could you send a larger snippet of your code? It's difficult to understand what's happening without a use case. Or try to put together an example of it.

Could you send a larger snippet of your code? It's difficult to understand what's happening without a use case. Or try to put together an example of it.

After some readings of my own code, I realized I don't even need to give you my own code : the code example you gave days ago just does the exact same result. With no additions on my side.
As my object is using static containers and functions, I thought it could be related but apparently it's not, because the segfault issue happens without this object being even declared.

I believe it could be because of the static usage. Maybe use a static object pointer instead?

I really don't want to look tough, but you should read my posts entirely xD
Without my static class' header being included in the main.cpp file - implying it's not working by any way - I still get the segfault error.
The only code left in my main.cpp is the code sample you gave me higher in the thread:

// Init
std::map<std::string, ptrraylib::Texture> textures {std::map<std::string, ptrraylib::Texture>()};
// Load
textures.emplace("ps3", std::make_uniqueraylib::Texture("resources/ps3.png"));
// Draw
textures["ps3"]->Draw(10, 10);

Here's the debug callstack I got after the error got noticed.

5 raylib::Texture::Load                                  Texture.hpp  144  0x562447d56aa1
6 raylib::Texture::Texture                               Texture.hpp  66   0x562447d56967
7 std::make_unique<raylib::Texture, char const (&) [88]> unique_ptr.h 1065 0x562447d56d71
8 main                                                   main.cpp     17   0x562447d566ce

It's really as much weird as it's frustrating... I've looked several times through my CMakeLists.txt but everything seems to be well installed, and in other ways, raylib and the wrapper are both perfectly working.

Hmm, interesting. Thanks for sharing the debug log. Is it segfaulting on load? I do know that LoadTexture only works after InitWindow...

Yes, directly on Load(). I'd like to give you more info but sadly the debugger won't give me more.
The only think I can remind you is that the compiler literally denies the existence of set() and IsReady() functions in raylib-cpp files.

These... should likely be part of the constructor...
auto& m = m_Musics[entryName] = std::make_unique<raylib::Music>();
m->Load(filePath);

auto& s = m_Sounds[entryName] = std::make_unique<raylib::Sound>();
s->Load(filePath);

// Use constructor?
auto& m = m_Musics[entryName] = std::make_unique<raylib::Music>(filePath);

auto& s = m_Sounds[entryName] = std::make_unique<raylib::Sound>(filePath);

TraceLog() doesn't give more info, sadly.
But strangely, I maybe did succeed to solve the problem. You see, in my main.cpp file, I was asking my static ResManager to load game resources BEFORE my Engine class (that implements the audio initialization, but also the window creation) was running.
Suddenly, when I did put my loadResources() invokation between my Engine class declaration, and the use of its run() function, everything started to work.
I don't really know to what it's relatedmaybe some internal Raylib mechanics, but at least, the program is working now.

Glad you got it to work! On construction, objects need to understand how to construct its children, and if a Texture tries to load before InitWindow(), things will break :wink:


Use name for directory/file initializing and removed unwanted funcion…

…ality.
Fixes https://github.com/Azure/azure-sdk-for-cpp/issues/1163
Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[x] C++ Guidelines
[x] Doxygen docs
[x] Unit tests
[x] No unwanted commits/changes
[x] Descriptive title/description

[x] PR is single purpose
[x] Related issue listed

[x] Comments in source
[x] No typos
[x] Update changelog
[x] Not work-in-progress
[x] External references or docs updated
[x] Self review of PR done
[x] Any breaking changes?

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage


New version: OmniSci v0.5.3

Registering package: OmniSci
Repository: https://github.com/omnisci/OmniSci.jl
Created by: @randyzwitch
Version: v0.5.3
Commit: f3588e13aff0e05ee56cabc8c5f1f9aa50ffe059
Reviewed by: @randyzwitch
Reference: https://github.com/omnisci/OmniSci.jl/commit/f3588e13aff0e05ee56cabc8c5f1f9aa50ffe059#commitcomment-35770156

Please fix the compatibility bounds for the next release such that it will be automatically merged.


C: Add blake3_hasher_init_derive_key_len

blake3_hasher_init_derive_key_len is an alternative version of blake3_hasher_init_derive_key which takes the context and its length as separate parameters, and not together as a C string.
The motivation for this addition is making it easier for bindings to this C library to call this function without having to first copy over the context bytes just to add one 0x00 byte at the end.
Notice that contrary to blake3_hasher_init_derive_key, blake3_hasher_init_derive_key_len allows the inclusion of a 0x00 byte in the context. Given the rules about context string selection, this byte is unlikely to be used as part of a context string. But if for some reason it is ever given, it will be included in the context string and processed like any other non-alphanumeric byte would. For compatibility with blake3_hasher_init_derive_key, bindings should still check for the absence of 0x00 bytes.
See the relevant issue about the C string version: #115

Note that as described in #115, this is an intermediate solution. Ideally, I think blake3_hasher_init_derive_key_len should replace blake3_hasher_init_derive_key altogether.

@oconnor663 I implemented the renaming you suggested in #115, and I added documentation to make things more clear.
@sneves I added a recommendation about the encoding of these bytes, but, since this is just a recommendation, I went ahead and recommend that the context string be written in a minimal ASCII alphabet. This is because many languages that allow you to write unicode literals directly don't necessarily represent this as UTF-8, but as some unicode abstraction, and moreover, because encoding unicode string literals to UTF-8 necessarily requires a unicode normalization step if you are expecting to use said UTF-8 bytes as cryptographic material. Of course, users are free to use UTF-8 if they want, but to keep things very simple and error-proof, a basic ASCII alphabet is what we should be recommending.
Here's the rendered version of the README: https://github.com/BLAKE3-team/BLAKE3/blob/b205e0efa1a1390dd91e3c015976e3d4ce999fff/c/README.md

Thanks for this, and apologies for the delay. I'll add some edits of my own to the C readme on top of this, and maybe some extra tests.

Great! Thanks for taking the time to consider this change!


editor border cut off

snippet code editor
open F8 error peek or symbol peek
:bug: the border on the left to awol

should in on box.


Doesn't delete without archive and got my account suspended

I was able to delete a good chunk of my old tweets using the archive option I exported a while ago. When I tried to use it without the archive it didn't delete a single post even though it appeared to be working. My account was then banned the next day. The reason given was Inauthentic Accounts, which includes "Unauthorized automation" in the criteria.

Permanent ban? can you provide error message

Same issue here.


Can't find awsutility-cloudformation-commandrunner.zip file

Hi,
Unfortunately I can't find awsutility-cloudformation-commandrunner.zip file in the project.
The register script failed at the stage : "Copying Schema Handler Package to temporary S3 Bucket"
Do you know how to get ( or generate ) this file before the register execution ?
Thanks

The following command solved my issue :
curl -LO https://github.com/aws-cloudformation/aws-cloudformation-resource-providers-awsutilities-commandrunner/releases/latest/download/awsutility-cloudformation-commandrunner.zip
Thanks

solved


How to check the command injected to each motor?

I wanna check the command injected to each motor when setting attitude by command set_attitude.
For example, when one sets attitude as set_attitude(0, 0, 0, 0.1) and rotates a drone (without propellers) so that its attitude becomes 30deg, 0, 0 (roll-pitch-yaw), then one would expect that there are some tendencies of each motor command.
So I'd like to monitor the injected motor commands. Which telemetry should I subscribe?

It's not so easy to debug that because you might see mainly motor outputs to fix roll and pitch as that's the most important thing to control and if there is a tiny error it will still integrate up over time and lead to a control response there.
So I'm asking back, what exactly are you trying to do or test?


Question: How to set state depending on browser?

Hi,
I was trying to set state when some browser is detected, however I can't do that inside browserHandler, there is a problem with "this". Could you please help?

Hi @kejtizuki,
can you please share the snippet of code that throws the exception so I can help you?
I'll post here an example that works fine:
import React, { Component } from 'react';
import BrowserDetection from 'react-browser-detection';

export default class App extends Component {
  state = {
    browser: null,
  }

  browserHandler = {
    default: (browser) => {
      this.setState({
        browser,
      });
      return null;
    }
  }

  render() {
    return (
      <div>
        browser: {this.state.browser}
        <BrowserDetection>
          {this.browserHandler}
        </BrowserDetection>
      </div>
    );
  }
}

Please note that the default handler (so the setState) is executed only once, if you want to execute the handler at every render of the App component you can set the once prop to false:
<BrowserDetection once={false}>
  {this.browserHandler}
</BrowserDetection>

@kejtizuki news about this?


What to do when there is no AWS configuration

Recently #48 was merged. These changes make sure default profiles will not be overwritten.
However, I was wondering what we should do when there aren't any $HOME/.aws/config and $HOME/.aws/credentials configuration files yet.
Maybe we should write the newly created profile as a default profile in case there is no previous aws-cli credentials configuration, and add/update it if there is.
That way people that use only this tool to set up aws-cli configuration will have a usable default profile, and people that had already setup aws-cli configuration will have their default profile kept intact.
The current version will just write an empty (unusable) default profile in addition to the newly generated profile if there is no previous aws-cli config.

I think that sounds like a reasonable solution.

Great! :) I've just created a PR.

Merged!


Training glove model on a custom curpus

Hi
I have a corpus in Hindi and want to train the Glove model on this dataset.
My corpus is in the form of a folder of text documents.
How can I do this? Please provide appropriate code. Thanks.

@spiros are you sure it's right?
because I guess doing so would put the last word of the first document and the first word of the second document in the same window

I misread the question, apologies, thought it was a single large document fragmented into multiple files!


Fix bug in persist() synchronization

There was a typo where persist was previously an async function when it should not have been. This still worked in ordinary function calls, but when running an app that called .persist() in modal app run, it would immediately crash with a cryptic error.
import modal

stub = modal.Stub()

stub.q = modal.Queue().persist("temp-delete-this")

@stub.function
def main():
    modal.container_app.q.put(3)
    print("Hello world!")
    assert modal.container_app.q.get() == 3

$ modal app run repro.py
Traceback (most recent call last):
  File "/opt/homebrew/lib/python3.10/site-packages/modal/cli/app.py", line 42, in run
    stub = import_stub(import_path, stub_name)
  File "/opt/homebrew/lib/python3.10/site-packages/modal_utils/package_utils.py", line 90, in import_stub
    spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/Users/ezhang/Documents/temp/sd/repro.py", line 5, in <module>
    stub.q = modal.Queue().persist("temp-delete-this")
  File "/opt/homebrew/lib/python3.10/site-packages/synchronicity/synchronizer.py", line 410, in proxy_method
    return method(instance, *args, **kwargs)
  File "/opt/homebrew/lib/python3.10/site-packages/synchronicity/synchronizer.py", line 368, in f_wrapped
    return self._run_function_sync(res, interface)
  File "/opt/homebrew/lib/python3.10/site-packages/synchronicity/synchronizer.py", line 245, in _run_function_sync
    raise Exception(
Exception: Deadlock detected: calling a sync function from the synchronizer loop
/opt/homebrew/lib/python3.10/site-packages/synchronicity/synchronizer.py:372: RuntimeWarning: coroutine 'Provider.persist' was never awaited
  raise uc_exc.exc from None
RuntimeWarning: Enable tracemalloc to get the object allocation traceback

Removing the async fixes this issue.
I don't know about synchronicity internals, so I don't know why this code was even working before, that seems strange? It seems unlikely, but are there any other problems that could result from this in places where things aren't synchronized properly? Hopefully it's just an internal unsafe usage.

Thanks for fixing!

Hm actually thinking about it, this might point to something weird with modal app run as well. Will take a look. But let's merge this either way!

So yeah, I think the problem is that modal app run runs user code inside the synchronicity loop, which it probably shouldn't, since:

any blocking code will block the thread (which runs the event loop handling all modal stuff)
the user may need to run things on the main thread (eg install signal handlers or whatever)

I'll add an issue to Linear to write a failing test and also fix this. It's not super hard.

cc @freider

Ah it might be that I broke the run command in this regard in a recent patch. I did add some basic tests for running sync and async stubs/functions but did not catch this. And running sync user code on the synchronicity thread sounds bad :(

Made a quick patch that should prevent user code from running in the synchronicity loop: https://github.com/modal-labs/modal-client/pull/151


Develop

Coverage remained the same at 100.0% when pulling 7c9a94f35711959a725f3d3df9f5665a4e66fe64 on develop into 8e2e06c3c0b3511c03251754011df5553a13cc8e on release.

Coverage remained the same at 100.0% when pulling fd53b5ec3cb03ac7852ffa7471d2107ffbc983f9 on develop into 8e2e06c3c0b3511c03251754011df5553a13cc8e on release.


@google-cloud/language: beginOffset is always -1

@google-cloud/language version 0.10.0

beginOffset is always returned as -1.
languageClient.detectEntities('Stephen of Michigan!', function(err, entities) {
  // entities = [
  //   {
  //     name: 'Stephen',
  //     mentions: [
  //       {
  //         text: {
  //           content: 'Stephen',
  //           beginOffset: -1
  //         },
  //       }
  //     ]
  //   },
  //   // ...
  // ]
});

Even documentation examples always quote it like this.
It should be equal to an offset of tag/entity in provided text.

As documented here,

If EncodingType is not specified, encoding-dependent information (such as beginOffset) will be set at -1.

So, doing the following resolves the issue:
document.annotate({ encoding: 'UTF8'})


Fix some memory leaks in freerdp_settings_free()

freerdp_settings_free() in libfreerdp/core/settings.c should free almost all pointer members of "struct rdp_settings" (aka: rdpSettings). Currently not all members of the setting struct are freed. This PR adds free for some settings.
I did not include the following three settings, because I think they need more careful handling for the free phase
TargetNetAddresses
Password51
DeviceArray

@giox069 thanks.

Cool!! Thanks guys!


Tapo Lighstrips

Are there any plans to support Tapo lightstrips L900 and L920?

Not at the moment as I don't own such a device.
It could be possible that they share the same api as the color bulbs, but I would have to check that. If you want you could give it a try if you own a light strip.

I'd be happy to have a look at this with some guidelines. However it doesn't seam like it exposes a valid response compared to other devices.
`curl -H "Content-Type: application/json" http://<IP_ADDRESS>/app?token
200 OK20:44:37~ $ `

I'd be happy to have a look at this with some guidelines. However it doesn't seam like it exposes a valid response compared to other devices.
`curl -H "Content-Type: application/json" http://<IP_ADDRESS>/app?token
200 OK20:44:37~ $ `

I've tested this today with my L900-5 light strip using the Colorlight type. All works nicely.

I've tested this today with my L900-5 light strip using the Colorlight type. All works nicely.

Not sure how or why but gave it another try now successfully

Does this stiff work for you ? Tried to do that with my L900-5 and it's not working

no luck for me, @capmalo mind sharing your config json? :)

no luck for me, @capmalo mind sharing your config json? :)

Yes, but it is roughly the same as in the example :
"platforms": [
    {
        "name": "Tapo Smart Platform",
        "username": "username",
        "password": "password",
        "platform": "TapoP100",
        "devices": [
            {
                "name": "Lightstrip_1",
                "host": "192.168.0.X",
                "type": "Colorlight",
                "updateInterval": 10,
                "timeout": 5
            }
        ]
    }
]


Release tag and brat

Hello ! Thank you for this plugin. Funny to see this appear the day after i talk about this on discord.
There is a problem with the tag release : to make BRAT function, you need to the release number. You tagged your release as "stable", but it should be "1.7.0" to make BRAT work !
I dunno what about the release for the plugin community but i think it will be good to be BRAT compilant :)

it is now 1.7.0; just out of curiosity how does BRAT work?

I don't have any details, but you can take a look here : https://github.com/TfTHacker/obsidian42-brat


Add GUI for displaying ShortScience summaries

The summaries that are gathered from ShortScience should be displayed graphically to the user. They should be able to query ShortScience from their bibliography.

The easiest way to handle this would be to simply provide the user with a link.


Regexes are silently anchored

I'm seeing this in version 0.17.0.
Suppose __meta_consul_tags value is ,environment:stage,application:webapp,.
In relabel_configs:
    - source_labels: ['__meta_consul_tags']
      regex:         ',application:(.*),'
      target_label:  'application'
      replacement:   '$1'

I would expect this to succeed. However, it fails, and my suspicion is that the regular expressions are anchored silently, because this succeeds:
    - source_labels: ['__meta_consul_tags']
      regex:         '.*,application:(.*),'
      target_label:  'application'
      replacement:   '$1'

If my assumption is correct, my suggestion would be that anchoring be not silently added, so if I truly wanted to anchor it, I'd use the regex anchor ^ to anchor it to the beginning of the string.

This behavior has been changed intentionally in 0.17.0 to the one you are seeing because wanting anchored regexes was seen as the more frequently desired use case.
See also: https://github.com/prometheus/prometheus/issues/1518 and the linked changelog entry.

This was a change we made on purpose, as not everyone is fully aware of how regexes work and we'd seen a lot of patterns in use that could unintentionally overmatch (consul tags being a common one).

That makes sense. I'd suggest that it's worth documenting well in the relabel_configs section to make that clear to the people who are more proficient at regular expressions. If you'd like me to submit a patch, I would be glad to create one.

It's already documented? https://prometheus.io/docs/operating/configuration/#relabel_config

It'd make sense to briefly mention it in the code block comment above the regex field too.

https://github.com/prometheus/docs/issues/370


Alter Configuration draft PR with Single String path and String path separator

Redesigned Configuration path and a separator from the original @ljnelson draft PR.

@radcortez I assume this would satisfy your comment in the original draft PR.

Note that another PR with altering the Loader class using the same single String path and separator comes later. While the single Configuration change is more Option 1, the additional Loader change is Option 2.

@radcortez I assume this would satisfy your comment in the original draft PR.

Better, but I'm not sure if it is a good thing to allow setting the separator (at least per class).
Let's have a look into the API usage:
Proposed:
@Configuration(path = {"my", "prefix"})
interface MyConfig {
}

Loader.load(List.of("my", "prefix"), MyConfig.class);
// or I guess that just passing in the class name should be enough, but I only found the above method in the Option 2 PR
Loader.load(MyConfig.class)

With the separator:
@Configuration(path = "my.prefix")
interface MyConfig {
}

Loader.load("my.prefix", MyConfig.class);
Loader.load(MyConfig.class)
// Or if we allow to load the same class but override the prefix, I think adding the path as second argument works better:
Loader.load(MyConfig.class, "another.prefix");

With a different separator:
@Configuration(path = "my.prefix")
interface MyConfig {
}

@Configuration(path = "another_prefix", separator = "_")
interface AnotherConfig {
}

Loader.load("my.prefix", MyConfig.class);
Loader.load("another_prefix", AnotherPrefix.class);

my.prefix.foo=bar
another_prefix_foo=bar

Is this something we want to support? I think it won't be very clear when you should use dot or any other separator, you will have to go and check each mapping class to know which one to use.
Is there any reason to not adopt dot as a single representation to separate paths to load the configuration?

@radcortez  Thank you for your reviewing it.
The separator is a matter of use in the path. The path might have a different separator than the source (config.properties,...), which we have not discussed yet. So at the moment, this PR assumes the source would be assumed to have a dot separator, whereas the path used for loading it can have a different separator. Strictly speaking, the separator is a matter of the canonical representation  i.e. mapping to the configuration key, as was set in the terminology in this project.
So, even though the source would be my.prefix.foo=bar, you can query the configuration with Loader.bootstrap().separator("/").path("my/prefix").load(MyConfig.class).
The use-case would be to set the separator as ".", even though the source (csv for instance in the future) would have "|", or use menu-like navigation such as separator("->").path("menu->file->save as")
That being said, I assume the usage of a separator would be rare.
You have brought up a good question though. Should the programmatic API recognize the values from the Configuration annotation or not.
If yes, you could have just Loader.load( AnotherPrefix.class);. If not, you would need to set separator, i.e. Loader.load("another_prefix", "_", AnotherPrefix.class); or Loader.separator("_").path("another_prefix").load(AnotherPrefix.class)
I assume you would like to inherit the values set by the annotation?

The separator is a matter of use in the path. The path might have a different separator than the source (config.properties,...), which we have not discussed yet. So at the moment, this PR assumes the source would be assumed to have a dot separator, whereas the path used for loading it can have a different separator. Strictly speaking, the separator is a matter of the canonical representation i.e. mapping to the configuration key, as was set in the terminology in this project.

Yes, I know we discussed keeping loading separate from the source, and that is fine, so a loading separator does not have to be the same as its particular source. Even, multiple sources can have different separators

So, even though the source would be my.prefix.foo=bar, you can query the configuration with Loader.bootstrap().separator("/").path("my/prefix").load(MyConfig.class).
The use-case would be to set the separator as ".", even though the source (csv for instance in the future) would have "|", or use menu-like navigation such as separator("->").path("menu->file->save as")
That being said, I assume the usage of a separator would be rare.

Being rare, should we give the option to change it? Because its is for the loading part, and we can control how to handle the loading, we can specify what should be the separator in the canonical representation of the path on the loading part. At the very least, if we want to make this configurable, let's make it global and not per path segment, so it is clear which separator is being used per application and not per config object.

You have brought up a good question though. Should the programmatic API recognize the values from the Configuration annotation or not.
If yes, you could have just Loader.load( AnotherPrefix.class);. If not, you would need to set separator, i.e. Loader.load("another_prefix", "_", AnotherPrefix.class); or Loader.separator("_").path("another_prefix").load(AnotherPrefix.class)
I assume you would like to inherit the values set by the annotation?

I'm a fan of reusing information we already have :) If the programmatic call is passing the same values already available in the annotation, then I should have an API that uses those instead and another API that could allow me to override them (if there is any sense in providing such API).

I have force-pushed the changes reflecting comments to keep the 2 commits of different committers that are not be merged.

I must be missing something. Only the annotation on the configuration interface needs to be observed for the separator/path (as well as the path), or (if overridden) the very programmatic API call is observed for the separator/path using the Loader object. What do you mean by unclear?

I've got it that this is per config object, but the unclear portion is that by providing such customization, you may end up with a very confusing setting. Consider:
one.object.value=1234
second_object_value=1234

Looking into this definition, is not clear which separator to use (I have to search each object load to learn which one to use). It is ok for implementations to provide such feature, but it shouldn't be included in a specification. Also, if there is a complete split between sources and canonical representations of a path, there is no need to allow for custom separators.

The configuration interface does not need to be annotated by @Configuration so we need the path (and the separator) on the Loader anyway.

Depends. One thing I haven't seen specified or discussed yet is if we will have a way to pre-discover the configuration interface candidates. The annotation could be used to mark such interfaces as being Config aware, or have some manual way to register them (maybe via ServiceLoader). This would allow to pre-validate the Config interface rules and fail the application start instead of throwing a runtime exception mid-flight when calling the loader (right now, I'm assuming that everything happens on the fly, since I haven't seen anything that indicates otherwise. Please correct me if I'm wrong).

In this first try, can we just stay with "." as the separator and then add a separator later if needed?

I've got it that this is per config object, but the unclear portion is that by providing such customization, you may end up with a very confusing setting. Consider:
one.object.value=1234
second_object_value=1234

Looking into this definition, is not clear which separator to use (I have to search each object load to learn which one to use). It is ok for implementations to provide such feature, but it shouldn't be included in a specification. Also, if there is a complete split between sources and canonical representations of a path, there is no need to allow for custom separators.

First of all, yes,

one.object.value=1234
second_object_value=1234

has no clear separator, completely agree. But this is a source definition, which we have not discussed yet and I do not think it would be wise to support multiple separators within a single source.
Second, yes, there would be a complete split between source keys and path. Both can have different separators. This way, the config Spec defines just the default separator, but the user use-case can be different and may want to use another separator. Examples of the sources can be the hard drive folder as a source with an OS-specific path separator, or a CSV file with a semicolon separator. The same would be for the config paths. The customers can use a separator from their domain, and make queries using it directly - similarly to passing SQL commands via Java interface, the customers can pass config queries directly to a Loader through some interface using a separator they are used to, for instance, the space character:
Loader.bootstrap().separator(" ").path("George Washington").load(Person.class);

Depends. One thing I haven't seen specified or discussed yet is if we will have a way to pre-discover the configuration interface candidates. The annotation could be used to mark such interfaces as being Config aware, or have some manual way to register them (maybe via ServiceLoader). This would allow to pre-validate the Config interface rules and fail the application start instead of throwing a runtime exception mid-flight when calling the loader (right now, I'm assuming that everything happens on the fly, since I haven't seen anything that indicates otherwise. Please correct me if I'm wrong).

The discussion so far was to have the main Loader mechanism, and the @Configuration annotation was added later, mainly for the cdi-enabled environments. So the implementation can have pre-discovery using the @Configuration annotations if it wanted, but it should not deny annotation-less dynamic resolves of interfaces. That's my understanding of what has been discussed so far.
What we also have not discussed yet is the mutability of sources. The mutability (if any in the future) would change the possibility of pre-discovery.
Also, with pre-discovery, there are some questions, such as whether or not to validate the interfaces that are marked with the @Configuration interface and are not used within the application. So the "pre-discovery" or "on-the-fly" options are up to the implementations at the moment, I believe.

@OndroMih Thank you for chiming in. We would be happy to see you in our weekly meetings every Thursday to discuss the API.
I have created another PR without the separator, though I am not in favor of it.

Closing in favor of #190


Service messages are barely visible in light and white themes

The messages about connecting are unreadable for non-dark themes:

Nightly v1.0.876 built 2019-01-31T07:32:25.0000000Z

Hey, the message should be readable now in the latest nightly version.


fix bug dropdown 7 items and update examples

Describe pull-request
Describe what the pull-request is about

Fix issue with dropdown showing the whole list when closing it with more the 7 items (scroll).
Added better example with dropdown
Fixed button icon in storybook
Added more dropdowns in html example

Solving issue
Add which issue this pull-request solves by adding # plus the number of the issue (for example #123)
Fixes: #233
How to test
Add description how to test if possible

Go to storybook dropdown
Add more then 7 items
open and close, it will show correct way

Screenshots
If applicable, add screenshots to help explain
Additional context
Add any other context about the pull-request here.

Hi Marcus, I had this issue back in november but postponed it since it was not a priority. Looking back now, I updated the @scania/components package to the latest build (2.1.0) but the problem with the dropdowns persists. I still have the same issues. Is this problem solved and in that case which which npm version should I have in my project?
Regards,
Amanda El Rayes, ECDA


Resources: Add 197-A Plans subsection

This will look just like the LUCATS project subsection. Need data from customer @kerrygathers

Two CSV files uploaded on Slack:

All 197-a Plans adopted
Implementation of 197-a Plans through rezoning initiatives

Both files contain "CD_Show" column for pulling into appropriate profile;
-"Comprehensive Manhattan Waterfront Plan" will show on all Manahttan CDs
-"New Waterfront Revitalization Program" will be shown on all 59 CDs

213-197-a-plans

First pass implemented in f4b19ed7c01b77c7fa08d6300d592568446584a2.  Added another table in carto since the schema was different from the other plans, but may want to combine into a single table for easier maintenance.


Fixes Data Preview Not Showing

@mpallansch can we close this? It looks like the positioning is holding on test and changing to absolute positioning crops the gray area.


Vestfold Fylkeskommune

Institution/School Name
Vestfold Fylkeskommune
Instiution/School Website
http://vfk.no
Email Users
Please place an x between the square brackets if yes

[x] Students
[x] Academic/Teaching Staff
[ ] Other/Support Staff

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[ ] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[x] High School or equivalent
[ ] Elementary School or equivalent

Hi @MatsAnd,
Two things I need you to clarify:

Do teachers use one of the domains you added, and students the other? Or are they both used by both students and teachers?
Does your school do post-secondary (degree level) courses?

Hi,
1: Students use the mail @skole.vfk.no and the teachers use the domain @vfk.no.
2: There is multiple schools that use the domain, so yes and no is actually the answer on the question. Anyways is the majority is upper secondary schools (High School).


ec2-instance module is not getting downloaded

Could not download module "ec2-instance" (module.tf:1) source code from "git::https://github.com/terraform-aws-modules/terraform-aws-ec2-instance?ref=29230f956912d751a01a5694510f684754a93196": error
│ downloading 'https://github.com/terraform-aws-modules/terraform-aws-ec2-instance?ref=29230f956912d751a01a5694510f684754a93196': git must be available and on the PATH.

I have tried with iam module as well same issue

Please check the terraform documentation since this problem is not related to the module but to your local setup.


[TW#25960] esp_ota_write function blocking,but other task is running

Environment

Development Kit: ESP32-DevKitC
Kit version (for WroverKit/PicoKit/DevKitC): v4
Core (if using chip or module): ESP-WROOM32
IDF version (git rev-parse --short HEAD to get the commit id.): d91c4251
Development Env:	visualGDB 5.4p4
Operating System:	Windows
Power Supply:         USB

Problem Description
esp_ota_write function blocking,but ble is running!!!!
Every 10 times or so.
Basically blocked every time, when ble data exchange is frequent
Expected Behavior
Actual Behavior
Steps to repropduce
I tried to remove all code, only uart ota(by xmodem), work ok.
I tried to use the wifi demo, the fault is still
Other modules still work when a fault occurs
I am trying to track the blocking position:
esp_partition_write->spi_flash_write->spi_flash_guard_start
I think that the execution lock could not be obtained at a specific time, causing the function to be blocked.
Code to reproduce this issue
// the code should be wrapped in the ```cpp tag so that it will be displayed better.
#include "esp_log.h"

void app_main()
{
    xTaskCreate(jlog_task, "jlog_task", 4096, NULL, 0, NULL);
i try use 8000 usStackDepth and high uxPriority,The error remains

The ota function is called in the task.

}

// If your code is longer than 30 lines, GIST is preferred.
Debug Logs
Debug log goes here, should contain the backtrace, as well as the reset source if it is a crash.
Please copy the plain text here for us to search the error log. Or attach the complete logs but leave the main part here if the log is *too* long.

Other items if possible

I found that it was caused by esp_ipc_call,
esp_err_t ret = esp_ipc_call(other_cpuid, &spi_flash_op_block_func, (void*) other_cpuid);
I tried to increase the ipc stack, but the fault is still

I tried to add vDelay after the write function, the fault basically disappeared, but it still exists.
Is it because I frequently try to read the buffer and the task time is insufficient?

@jaskle Could you help share some updates, or could yo reliably reproduce this issue? Many thanks.

@jaskle Could you help share if there are any updates for this issue? Thanks.

@jaskle Could you help share if there are any updates for this issue? Thanks.

Closing, please feel to reopen if there is any update.