Boost.Python.ArgumentError in shape_landmark_detection.py

Hello,
I have a problem using shape_predictor for face landmarks detection using python 2.7 in anaconda.
I compiled dlib and verified boost lib path according to these issues:

issue 124
issue 177

I also tried to rebuild Boost from source, but still having the same error:
Traceback (most recent call last):
  File "face_landmark_detection.py", line 66, in <module>
    predictor = dlib.shape_predictor(predictor_path)
Boost.Python.ArgumentError: Python argument types in
    shape_predictor.__init__(shape_predictor, str)
did not match C++ signature:
    __init__(boost::python::api::object, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >)
    __init__(_object*)

Boost must be compiled against the version of python you are using, the one in anaconda.  It's probably being compiled against a version of python installed by your OS instead, leading to this error.
In any case, this happens because you aren't using the same python version while compiling boost and running the compiled output.

Hello: I just faced this issue; see also
Issue 103
issue 124
...
I hope the following is helpful, to others.  :-)

SOLUTION -- GETTING DLIB FACIAL LANDMARKS SCRIPT TO RUN IN P3!
==============================================================

[2016-Oct-25]

PROBLEM:
--------

Getting these errors when running some (not all) Dlib facial landmarks-related Python
scripts in my Python 3.5 virtual environment (p3 venv):

    Traceback (most recent call last):
      File "face_landmark_detection.py", line 66, in
        predictor = dlib.shape_predictor(predictor_path)
    Boost.Python.ArgumentError: Python argument types in
        shape_predictor.__init__(shape_predictor, str)
    did not match C++ signature:
        __init__(boost::python::api::object, std::__cxx11::basic_string, std::allocator >)
        __init__(_object*)

I Googled and tried everything I could think of to solve this.  Here is my solution
(your mileage may vary).

----------------------------------------
1. HOST ENVIRONMENT
----------------------------------------

In my host (Arch Linux x86_64 on Intel i7-4790 CPU-based system; Python 3.5),
non-virtual environment, I have { boost | dlib } "pre-installed:"

    boost:
    ------

        whereis boost

            boost: /usr/include/boost       ## installed via: sudo pacman -S boost

    dlib (installed from source):
    -----------------------------

        # compile dlib c++ example programs:
        # ----------------------------------

        cd /mnt/Vancouver/apps

        git clone https://github.com/davisking/dlib

        cd /mnt/Vancouver/apps/dlib

        mkdir build

        cd build

        cmake ..

        cmake --build . --config Release

        # compile dlib python api:
        # --------------------------

        cd ..

        pwd                         ## /mnt/Vancouver/apps/dlib

        python setup.py install
        # python setup.py install --yes USE_AVX_INSTRUCTIONS
        # use the line above instead, if your Intel CPU supports AVX, per:
        # https://pypi.python.org/pypi/dlib

----------------------------------------
2. PYTHON 3.5 VIRTUAL ENVIRONMENT
----------------------------------------

source activate py35            ## activate Anaconda Python 3.5 venv

Then, in p3 venv, searched,

    pip list | egrep -i 'pkg1|pkg2|...'
    conda list | egrep -i 'pkg1|pkg2|...'

... for all pip- and conda-installed

    { scikit-learn | scikit-image | scipy | boost | boost-python }

... versions, and removed these, if present:

    pip list | egrep -i 'scikit|scipy|boost'

    conda list | egrep -i 'scikit|scipy|boost'

... via

    pip uninstall pkg_name

    conda uninstall pkg_name

Oddly, even though I reset my terminal ("exec bash"), I needed to QUIT the terminal and
open a new one (as even thought there was no apparent trace of dlib (pip list; conda list)
in my p3 venv, I could still launch Python and import dlib).  :-/  Easy "fix."

Finally, in that (new) p3 venv session, I installed (note use of "pip install", even though
I am in a conda venv):

    dlib
    scikit-image
    scikit-learn
    scipy

After doing that, my dlib "find facial landmarks"" Python scripts successfully executed in my
p3 venv, as they also did in my host env:

    python  finding-face-landmarks.py  files/carm_victoria_july_13_2012.jpg
        ...     ## exit code 0  :-)

    python  face_landmark_detection.py  ../files/shape_predictor_68_face_landmarks.dat  \
files/carm_victoria_july_13_2012.jpg
        ...     ## exit code 0  :-)

Scripts mentioned (above):
--------------------------

    face_landmark_detection.py :

        http://dlib.net/face_landmark_detection.py.html
        # see comments in that code for face landmarks .DAT file source location

    finding-face-landmarks.py :

        https://gist.github.com/ageitgey/ae340db3e493530d5e1f9c15292e5c74

        ... via:

        https://medium.com/@ageitgey/machine-learning-is-fun-part-4-modern-face-recognition-with-deep-learning-c3cffc121d78

Q.E.D.  (Whew!)

Yeah, boost.python has an awful build process.  I'll switch to pybind11 instead of boost.python at some point in the future (https://github.com/davisking/dlib/issues/293).  That should make building the python extensions pretty trivial.

Hi @davisking,
I'm still a bit lost on debugging this issue. Could you indicate how we can identify where boost is installed, and potentially force recompiling/using the one installed through conda to make it compatible with dlib (on linux)? Thanks!

There are no standards at all for boost and python packagers.  Practically everyone who posts in the dlib forums has their computer configured in some different way, with boost in some random different folder, or multiple copies compiled against multiple different and incompatible versions of python they have strewn throughout their filesystem.
The entire Python build system (or lack of a build system really) is just a total clusterfuck.   So no, there is no general solution other than you should know what software you install on your computer and try and be consistent about what you install.

Also, I realize that's frustrating.  The best advice I can give is to compile boost yourself.  That way at least you know the copy of boost in the folder you are looking at is compiled against the version of python you are using.

Thanks @davisking for the quick response.
I have tried installing boost through

ubuntu: sudo apt-get install libboost-all-dev
anaconda: conda install boost and
download and manually compile boot's sources: boot/bootstrap.sh; sudo ./b2 --with=all -j $n install

None of these installs generated errors, but dlib (after re-installing it every time with python setup.py install) continues throwing Boost.Python.ArgumentError: Python argument types in     shape_predictor.__init__(shape_predictor, str)
I also tried to remove Ubuntu's python2.7 to ensure that boost would link to anaconda's python, again without success.
I'm not familiar with this kind of issue and boost's documentation is pretty rough. Would you have some hints on how to proceed from here? i.e. perhaps locate an already existing boost ? selecting a specific compiler? check the order of compilation and installation steps?

You have to figure out how to compile boost against the version of python you are using.  Like I was saying, I don't know how to do that for all python configurations.  But this isn't a dlib problem.  Fundamentally, it's happening because you are compiling boost against one copy of python and then running another version of python (the one in anaconda).
Honestly, I would just delete anaconda.  Half the people I see having difficulty compiling python modules are using anaconda.

ok thanks, I'll keep trying

I found a fix for my case (Ubuntu 16.04, python 2.7 with conda).
First, I needed to uninstall Ubuntu's Boost:
sudo apt-get remove libboost-dev
as well as conda's boost:
conda uninstall boost.
After this, dlib's setup.py identified the Boost that I had manually compiled and installed: Boost version: 1.62.0. However, there were some lingering dependencies that remained in anaconda, as indicated by: USING BOOST_LIBS: /home/jrking/anaconda/lib/_libboost_python-mt.so
I manually removed (after a backup) ~/anaconda/lib/libboost_python-mt.a and ~/home/jrking/anaconda/lib/libboost_python-mt.so
and it now seems to works.
I hope this helps others.

Sorry to necro this, but maybe my note will help someone.
I had similar error message on Debian with self-compiled Python, Boost, and dlib:
Traceback (most recent call last):
  File "my_dlib_tests.py", line 102, in <module>
    predictor = dlib.shape_predictor(face_landmarks_path)
Boost.Python.ArgumentError: Python argument types in
    shape_predictor.__init__(shape_predictor, PosixPath)
did not match C++ signature:
    __init__(boost::python::api::object, std::string)
    __init__(_object*)

Based on above discussion I figured I also had compilation issue between different versions of Boost and Python. And I did have those previously due to apt-get giving different libboost version.
After fighting for hours I realized that that error is telling me that the shape_predictormethod expects a string, and I'm feeding it a pathlib.Path object.  Code for clarity:
from pathlib import Path
import dlib

fname = 'shape_predictor_68_face_landmarks.dat'
# This gives the error above:
f_as_path = Path(fname)
dlib.shape_predictor(f_as_path)
# And this is the correct way to call that method:
dlib.shape_predictor(fname)

Lesson: Learn to read Boost.Python errors.

I tried all the solutions written above, but none worked on my machine.
The problem was that python from anaconda kept on find the boost version installed in anaconda.
So I just added anaconda2/pkgs/boost-1.59.0-py27_0/lib to LD_LIBRARY_PATH and it worked.
I also uninstalled boost library but I'm not sure it helped.
sudo apt-get remove libboost-dev
sudo apt-get remove libboost-python-dev
conda uninstall boost

So it's working now then?

@davisking Yes, I commented it for someone with the same problem.

It seems like dlib compilation does not find the right boost library if you have multiple versions installed, and ignores some of the cmake flags that allow you to not use system libraries.
It can be fixed by manually fixing dlib/cmake_utils/add_python_module: set the right BOOST_LIBRARYDIR and remove FindPackage lines corresponding to the system python.


Error: could not download the swagger file

kubectl deprecations --k8s-version=1.23.10

Error: could not download the swagger file https://raw.githubusercontent.com/kubernetes/kubernetes/1.23.10/api/openapi-spec/swagger.json
time="2022-12-13T20:41:14Z" level=error msg="An error has occurred: could not download the swagger file https://raw.githubusercontent.com/kubernetes/kubernetes/1.23.10/api/openapi-spec/swagger.json"

When manually browsing the link, I get a 404 error.
It seems the openapi-specs json files might have moved.
I found an alternative link
https://raw.githubusercontent.com/kubernetes/kubernetes/v1.23.10/api/openapi-spec/swagger.json
Downloaded the file following the kubepug documentation and tried to run it with the --input-file=
`kubectl deprecations  --input-file=swagger.json

time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error converting YAML to JSON: yaml: line 4642: found unexpected end of stream. Skipping to next"
time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal string into Go value of type fileinput.FileStruct. Skipping to next"
time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal string into Go value of type fileinput.FileStruct. Skipping to next"
time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal string into Go value of type fileinput.FileStruct. Skipping to next"
time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error converting YAML to JSON: yaml: line 2: mapping values are not allowed in this context. Skipping to next"
time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error converting YAML to JSON: yaml: line 2: mapping values are not allowed in this context. Skipping to next"
time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal string into Go value of type fileinput.FileStruct. Skipping to next"
time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error converting YAML to JSON: yaml: line 2: mapping values are not allowed in this context. Skipping to next"
time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal string into Go value of type fileinput.FileStruct. Skipping to next"
time="2022-12-13T20:00:28Z" level=warning msg="Found invalid yaml: error converting YAML to JSON: yaml: line 2: mapping values are not allowed in this context. Skipping to next"
RESULTS:
Deprecated APIs:
Deleted APIs: `

Update :
Using the --swagger-dir option works though.
kubectl deprecations --k8s-version=v1.23.10 --swagger-dir=./kubepug
W1214 16:03:10.367006   27608 warnings.go:70] v1 ComponentStatus is deprecated in v1.19+
W1214 16:03:10.608304   27608 warnings.go:70] policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+
RESULTS:
Deprecated APIs:

ComponentStatus found in /v1
         ├─ ComponentStatus (and ComponentStatusList) holds the cluster validation info. Deprecated: This API is deprecated in v1.19+
                -> GLOBAL: controller-manager
                -> GLOBAL: scheduler
                -> GLOBAL: etcd-0

PodSecurityPolicy found in policy/v1beta1
         ├─ PodSecurityPolicy governs the ability to make requests that affect the Security Context that will be applied to a pod and container. Deprecated in 1.21.
                -> GLOBAL: longhorn-psp
                -> GLOBAL: pushprox-kube-controller-manager-client
                -> GLOBAL: pushprox-kube-controller-manager-proxy
                -> GLOBAL: pushprox-kube-etcd-client
                -> GLOBAL: pushprox-kube-etcd-proxy
                -> GLOBAL: pushprox-kube-proxy-client
                -> GLOBAL: pushprox-kube-proxy-proxy
                -> GLOBAL: pushprox-kube-scheduler-client
                -> GLOBAL: pushprox-kube-scheduler-proxy
                -> GLOBAL: rancher-monitoring-alertmanager
                -> GLOBAL: rancher-monitoring-crd-manager
                -> GLOBAL: rancher-monitoring-grafana
                -> GLOBAL: rancher-monitoring-kube-state-metrics
                -> GLOBAL: rancher-monitoring-operator
                -> GLOBAL: rancher-monitoring-patch-sa
                -> GLOBAL: rancher-monitoring-prometheus
                -> GLOBAL: rancher-monitoring-prometheus-adapter
                -> GLOBAL: rancher-monitoring-prometheus-node-exporter

Deleted APIs:

@Turb0Fly can I close this one? Thanks!

Closing this based on the last feedback, feel free to reopen/open a new one in case of errors
/close


make blocked nullable

See https://github.com/timols/java-gitlab-api/pull/236#discussion_r141539662

That failure looks bogus to me; it works locally.


BigQuery: Support Date, Time, and DateTime types in FieldValue

A Date type has been recently added to com.google.cloud.core which can be used here
At the same time as this change, timestamp could use the Timestamp type added recently to com.google.cloud.core
Create new types Time and DateTime in com.google.cloud.core to handle those types

@garrettjonesgoogle  Can we close this issue now?

No, https://github.com/GoogleCloudPlatform/google-cloud-java/pull/1819 was closed out because we decided to use threetenbp types instead of handrolled types. The support for threetenbp types still needs to be implemented.


sql/analyzer: alias projected columns wrapped in converts

Fixes required to work on gitbase. It was caught there because there are way more complex queries in the gitbase test suite.
Still needs some fixes to work on gitbase, because squash rule works a bit weirdly with this.

Don't merge, just found a test case that makes this fail.

All working now


HTTP ERROR 500

Hello,
I have followed all the intructions in the tutorial. Except I have installed a newer Golang version (1.19.5).
Photoprism starts normally but I'm not able to access the Webinterface.
If I try to access the IP <IP_ADDRESS>:2342 I get redirected to http://<IP_ADDRESS>:2342/library/login and get a "HTTP ERROR 500
This page cannot process your request right now".
I also get some weird initrd.img errors in the photoprism logs. I have attached it.
photoprism.log
This is my .env:
PHOTOPRISM_AUTH_MODE="password"
PHOTOPRISM_ADMIN_PASSWORD="photoprism"
PHOTOPRISM_HTTP_HOST='<IP_ADDRESS>'
PHOTOPRISM_HTTP_PORT='2342'
PHOTOPRISM_DEBUG="true"

# PhotoPrism storage directories
PHOTOPRISM_STORAGE_PATH="/var/lib/photoprism"
PHOTOPRISM_ORIGINALS_PATH="/var/lib/photoprism/Originals"
PHOTOPRISM_IMPORT_PATH="/var/lib/photoprism/photos/Import"

# Uncomment below if using MariaDB/MySQL instead of SQLite (the default)
# PHOTOPRISM_DATABASE_DRIVER="mysql"
# PHOTOPRISM_DATABASE_SERVER="MYSQL_IP_HERE"
# PHOTOPRISM_DATABASE_NAME="DB_NAME"
# PHOTOPRISM_DATABASE_USER="USER_NAME"
# PHOTOPRISM_DATABASE_PASSWORD="PASSWORD"

Any errors during the build process?
I found a similar discussion in PhotoPrism's Github: https://github.com/photoprism/photoprism/discussions/3042
It refers to this issue: https://github.com/photoprism/photoprism/issues/2946
There it is suggested that the error is caused by files in the root of the filesystem (directly under /). So that might be something to check.

As far as I watched it, there were no errors when building at all.
I'm actually not quite sure what to look for?

Yeah, those files in the root of the filesystem might be the issue. Specifically, the "initrd" and "vmlinuz" files.
I found some info here that says they are symlinks to your current and previous kernel: https://askubuntu.com/q/1049427
You can confirm by running ls -l  while in the root directory; symlinks will have the letter "l" at the very beginning of the line (directories are marked with "d").
Apparently you can delete the symlinks, but they can be recreated with a kernel update.
What OS are you running? Is this a fresh install or a used system? VM or physical machine?
The PhotoPrism issue I linked to before suggested that the issue is fixed in preview builds. So what you could try is going to the directory where you cloned the PhotoPrism source code and switching to the "preview" branch instead of "release", by running git checkout preview. Then go through all the subsequent install steps. If that doesn't fix it, you can also try the bleeding-edge "develop" branch.

Thanks for the answer.
Here the requested information:

I will try to install the preview branch and report back asap

Also it is a VM and freshly installed. The only thing I did on this os was the tutorial from you

I have now build photoprism with the preview branch and it does work now!
Using this build: 230201-b2ffa67e0

Alright, good to hear you got it sorted.
I'll try and add a note about this to the troubleshooting section.


Point drawing abstraction

Updates gps,navsat,odometry,path, and tf_frame plugins to use the
abstract point drawing class. Also adds the draw laps functionality
which will change the color of the path as it passes a base point for
ease of visibility, currently implemented on gps and odometry plugins.

As a general comment, it looks like there are several files in this pull request that got re-indented and reformatted but are unrelated to the actual changes.  We could probably stand to do a reformatting pass over all of the plugins just to make sure they look pretty, but I would rather have that in a separate PR and have this one only contain the path drawing-related changes.

When testing it, it seems like the "Draw Laps" checkbox clears the buffer when it is checked.  Is this intentional?  It seems like somebody might want to highlight laps on a track that they've already recorded.

Aside from the uninitialized member variables, this looks good to me.  Rebase it down to one commit and I'll merge it in.

I managed to make a big mess out of this rebase, so I'm just going to cherry-pick the individual files and make a new PR at #382.


Clicking "Edit Page" Returns 404 URL links to undefined

Description
On the webpage https://opensource.newrelic.com/projects/newrelic/newrelic-ruby-agent, there is an "Edit page" button on the right side of the page next to the View Repo.  When I clicked on that link, it returns a 404 as the edit button now is pointing to: https://github.com/newrelic/opensource-website/blob/develop/undefined
Steps to Reproduce
Click on the "Edit page" link for this page.
Expected Behavior
Allows me to edit and link to the correct repo for the page.
Your Environment
Chrome
Additional context
This seems to be occurring on new project pages being added to the OSS site as well.

@devfreddy / @tangollama Do you have any ideas why this one page edit button isn't linking to the right location? Other pages seem to work fine.

@jerelmiller I checked into this problem and I didn't see anything wrong with the code for the edit button, However I'm wondering if it's a repo setting, as it seems to only impact 2 repos at this moment:
https://github.com/newrelic-experimental/quickstarts
https://github.com/newrelic/newrelic-ruby-agent

The issue seems to be related to the "slug" in newrelic-newrelic-ruby-agent.json.
Currently, The value is "newrelic-ruby-agent", while the corresponding .mdx is named "newrelic-newrelic-ruby-agent". The same issue may be occurring for the https://github.com/newrelic-experimental/quickstarts button as well.

The issue seems to be related to the "slug" in newrelic-newrelic-ruby-agent.json.
Currently, The value is "newrelic-ruby-agent", while the corresponding .mdx is named "newrelic-newrelic-ruby-agent". The same issue may be occurring for the https://github.com/newrelic-experimental/quickstarts button as well.

@DamonHolland thanks for pointing that out!!  Would you be interested in submitting a fix for it? The project is opensource, so feel free if you want to help out to make the change an file a PR and I can code review it. 😃 , if not no worries!

@DamonHolland thanks for pointing that out!!  Would you be interested in submitting a fix for it? The project is opensource, so feel free if you want to help out to make the change an file a PR and I can code review it. 😃 , if not no worries!

Alright, I will go ahead and try. This is my first open source contribution! I Just wanted to state what I thought the problem was in case I had issues on my end.

Alright, I will go ahead and try. This is my first open source contribution! I Just wanted to state what I thought the problem was in case I had issues on my end.

@DamonHolland cool! Give it shot  and if you need help we are here  to assist.

@DamonHolland cool! Give it shot  and if you need help we are here  to assist.


reduce the use of param_find("XXX")

reduce the use of param_find("XXX") in favor to param_handle(px4::params::XXX)
The pr is replacement to #13076
In case parameter is not in a version, there will be build time error.
My method was to set all parameters to use param_handle(px4::params::XXX) and then find what is not being built.
that mean that every place that there you can still find a param_find("XXX") some build/s been broken.
That mean that those places basically need to be fixed.
on some places the error seems as more critical then others, for example building px4_fmu-v2_test need the parameter FW_PSP_OFF but it doesn't exists. and later there is no check if it exists or not. that mean that in _params_handles_standard.pitch_setpoint_offset there is PARAM_INVALID and later there is garbage when trying to fetch the parameter

We actually need to be more careful here than it first appeared. One difference is the param_find() also marks the parameter active (which then syncs to QGC).
https://github.com/PX4/Firmware/pull/13081#issuecomment-537985271

I must say that in general I don't like the idea of "active" parameter. I think that either there is a parameter on a system or not.
In my custom firmware I added a do_touch_all() in order to set all the parameters as active.
the most problematic issue of having 'active' parameters is that copying a parameter file from drone to drone sometimes misses some parameters, as described on #12432.
the advantage of keep using 'active' is that downloading parameters might take a bit faster.

I must say that in general I don't like the idea of "active" parameter. I think that either there is a parameter on a system or not.
In my custom firmware I added a do_touch_all() in order to set all the parameters as active.
the most problematic issue of having 'active' parameters is that copying a parameter file from drone to drone sometimes misses some parameters, as described on #12432.
the advantage of keep using 'active' is that downloading parameters might take a bit faster.
Or I missing another advantage?

Yes it makes downloading faster, but it's more fundamental in that it's really what makes the system scalable from a user perspective. We can have release binaries with all kinds of controllers and drivers, but the user is only going to see a relevant subset for configuration.
The pattern that should be enforced is that a module should be marking all of its parameters active when it starts. Virtually all the problems have come from doing this lazily or incorrectly.

Seconding here - all what Daniel says its correct. The system is only tractable with this enabled. The same way that C++ requires correct initialization of pointers we require correct initialization of parameters. You can argue that the programming environment should address both, but then really the right starting point is to replace C++ with something a lot more defined, and that is what most people are not debating.
If you go down that path you ought to use purely compile-time configurations and binaries for every single airframe type - and yet PX4 has won against these legacy architectures because it creates an ecosystem that is able to generalize more and move faster.

@dagar @LorenzMeier thanks for the replay, I appreciate how you look of things, and agree. My case study is after I removed most of the modules that are not relevant to me...
setting all parameters by do_touch_all() function that I added is optional, and that why I didn't even added it to the PX4/Firmware, I can if you want.
still, about this pr. there is still need to minimize the use of param_find("XXX"), and use hardcoded parameter name. so how about using param_set_used() in param_handle()?

still, about this pr. there is still need to minimize the use of param_find("XXX"), and use hardcoded parameter name. so how about using param_set_used() in param_handle()?

How about adding a param_find that only takes enum and marks it active? It would have to be c++ only similar to this.
https://github.com/PX4/Firmware/blob/941a3258b6e87640a52365fe003db1ec6bb898cd/src/lib/parameters/param.h#L491-L500
Then the replacement would be even cleaner.
param_find("BAT_SOURCE") -> param_find(px4::params::BAT_SOURCE)

I am not sure that 'param_find()' will be the right term, you don't really search for it, you know it exist.
from what I see param_handle() isn't used almost, any just add param_set_used_internal() won't break anything.

@dagar, should I close this issue, and #12992?


Scope of CancellationToken

I found your library on stackoverflow
you noted this was created to replace HostingEnvironment.QueueBackgroundWorkItem
For a sanity check, will CancellationToken be called whenever the app pool recycles/is shut down?
QueueBackgroundWorkItem gives the users a 30 second window to gracefully close it's stuff before forcing everything to end, so i'm hoping CancellationToken.Register will give me the (now 5 second) same grace period

I  can confirm this to be correct: https://github.com/DalSoft/DalSoft.Hosting.BackgroundQueue/blob/master/DalSoft.Hosting.BackgroundQueue/HostedService.cs
StopAsync is implemented and the cancellation token is honoured.
https://learn.microsoft.com/en-us/aspnet/core/fundamentals/host/hosted-services?view=aspnetcore-8.0&tabs=visual-studio


[3.2.0] deployment.toml config for configuring proxy profiles.

We need to update the documentation with deployment.toml config when configuring proxy profiles
We need to add below lines to the /repository/conf/deployment.toml file to achieve configuring proxy profiles.
[transport.passthru_https.sender.parameters]
proxyProfiles = "<profile><targetHosts>example.com</targetHosts><proxyHost>localhost</proxyHost><proxyPort>8080</proxyPort><proxyUserName>squidUser</proxyUserName><proxyPassword>password</proxyPassword></profile>"

related issue https://github.com/wso2/docs-apim/issues/497


[Feature Request] Leave channel with a timeout when nothing is played

Is your feature request related to a problem? Please describe.
When "stayinchannel = false" there should be another parameter called like "leavechanneltimeout = 360" to apply a delay when the bot plays nothing before leaving.
What is your ideal solution to the problem?
A parameter to set a timeout in second to keep the bot in the channel for (as example) 5 minutes when nothing is played but there is someone in channel so it doesn't live directly, but if after 5 minutes nothing is played then it can leave.
How would this feature be used?
Just a parameter as I described earlier would be enough.
Additional Info
No response
Checklist

[X] I have checked the documentation to make sure this feature doesn't already exist
[X] I have searched for similar feature requests
[X] I have confirmed that my requests is not on the list of things that will not be added
[X] I am running the latest version of the bot:

It has been discussed that this shouldn't be implemented, and that stayinchannel should be left to false if you are concerned about bandwidth.


Modal observeChanges doesn't react to accordion

Hi, I was very happy to find the option observeChanges on the modals, it saves a lot of error prone automatized 'refresh' calls
The only problem so far I have encountered is with accordions, I use them a lot within modals, and unfortunately when they open they dont update the modal position (or scrolling), jsfiddle follows:
http://jsfiddle.net/4rnnmm4f/
What is interesting it works when the window is very small from the beginning.
I dont expect observeChanges to be 100% preceise, but this would be a nice improvement.
The refresh works, so I can fix it calling it on accordion opening, but still would be nice to have it out of box. Especially I have found some issues when calling refresh on the model when another is opened behind.

This is unfortunately a limitation of DOM mutation observers.
They don't update when elements height or offset position change, only if they are removed or inserted. This is a frustrating part of the spec that also affects visibility and sticky quite dearly.

I stumbled upon this problem too.
My workaround is to manually add the scrolling class to both body and the actual modal.
$("#my-modal").modal({
  onShow: function() {
    $("body, #my-modal").addClass("scrolling");
  }
}).modal('show');


www-url-form-encoded parameters are not handed in Spring project

Hi,
First, I'm really enjoying the logbook library and thank you for your effort.
I'm not sure the issue I experienced is a bug or intended behavior, and would like to say sorry if it's not a bug.
Description
I'm building a project using Spring Boot 2.4.1 with logbook-spring-boot-starter version: 2.4.1.
One of my controller methods is defined like below.
@RequestMapping(path = "/xxx/xxx", consumes = MediaType.APPLICATION_FORM_URLENCODED_VALUE)
public ResponseObject method(final RequestObject req)

When I sent requests to the API with x-www-url-form-encoded parameters, req parameter was always null before I changed logbook.filter.form-request-mode to parameter.
Expected Behavior
Request parameters need to be handed to target methods with the default option(body)

This is described in https://github.com/zalando/logbook#form-requests and the preceding paragraph, as well as numerous previous issues around this problem. See #94 and the referenced issues there.


FIX THIS

WARNING: Failed to remove contents in a temporary directory 'C:\Users\genop\OneDrive\Desktop\PySilon-malware-3.7.5\PySilon-malware-3.7.5\pysilon\Lib\site-packages~il'.

Can someone close this ID10T error


940 map param lookup

Closes #940
Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide
[x] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[x] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[x] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue in the Development Section on the right hand side.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done! Much love to your accomplishment!

Hi @yangc71,,
Apologies for late comment, I was on holiday.
Is it possible to get an option to get distinct values from the look up table?
An example is where you use this function to add on PARAMCD only, then create some derived PARAMCDs (BMI, etc).
You may want to keep all the PARAMCD/PARAM values in the look up table, even the derived ones, so when you add on the PARAM, merging by PARAMCD, you only want to keep distinct values of PARAMCD, PARAM from the look-up table.
Currently when I use derive_vars_merged it returns an ERROR saying values are not unique.
The new distinct_add option would be applied after filter_add is applied.
Does above make sense?

Hi @millerg23, thanks for your comments. I originally dropped some arguments from derive_vars_merged to keep the new function simple, but based on your comment above I have added them back. I think we can specify check_type = "none", order = vars(...), mode = "first"/"last" to control which unique records to keep from the lookup table. Can you please give it a try and see if it works as you expect? If not, we can chat more about the distinct_add option you mentioned. Thanks.

Hi @yangc71 , Yes you are correct, I should have looked at the main function more carefully. The options check_type, order and mode worked nicely for the scenario I had.
Many thanks.

Thanks! @millerg23 @bundfussr @bms63 Do we need any further updates for this PR?

@bms63 , we need the docs folder and the pkgdown.yml file to enable linking across different admiral packages. For example, if we have admiral::derive_vars_merged() in the admiralonco documentation, it will link to the admiral documentation.
So please do not remove it.

@bms63 , we need the docs folder and the pkgdown.yml file to enable linking across different admiral packages. For example, if we have admiral::derive_vars_merged() in the admiralonco documentation, it will link to the admiral documentation.
So please do not remove it.

ah is see that it is in devel i was looking at main.  apologies.  we might want to set this up in the admiraltemplate as this is quite useful.


Fix destruction of $apollo in vue-apollo-option

The $apollo instance is not correctly destroyed on component destruction due to a typo.

Good catch! It's a leftover from previous version :p


Durable sesion store

Hi I would like to ask on recommendations regarding durable storage for sessions. I would like all the topics and messages inside of those topics to be durably stored, ideally in similar fashion as it is possible with retained messages and WithStorage option. Please any suggestions how to achieve that?
Which project is your question related to?

Server

Well, you could intercept all messages and connections using the handlers described in the wiki:

https://github.com/chkr1011/MQTTnet/wiki/Server#intercepting-application-messages
https://github.com/chkr1011/MQTTnet/wiki/Server#intercepting-subscriptions
https://github.com/chkr1011/MQTTnet/wiki/Server#validating-mqtt-clients

and store them in a database or file(s)?

Thank you @SeppPenner this was my original thought, nevertheless as this not totally trivial task, when we consider cleaning of the db/store, eventually parallel access, I wanted to know, if there is some out of the box approach. Maybe idea for feature request to have topics backed up by durable store?

I don't think that there is a solution like this available at the moment...

We've done this in our project. It's quite trivial. We've implemented custom ApplicationMessageInterceptor(). Just remember to create scoped DB context from a pool inside (or use DI) and pass the MqttApplicationMessageInterceptorContext.ApplicationMessage.Payload (JSON in our case). We've implemented code that using reflection tries to find method in a given namespace that matches normalized topic (msgType/deviceType/deviceName -> ParseDeviceTypeMsgType()) that handles the payload.

You could also take a look at my project under https://github.com/SeppPenner/NetCoreMQTTExampleCluster. I'm storing incoming messages to a database table (PostgreSQL with TimescaleDB). Specifically, take a look at NetCoreMQTTExampleCluster.Storage and don't be irritated by the clustering stuff which is not necessary for this question.


show values for "onset" and "expires" on card?

Describe the solution you'd like
in case of an alert, the meteoalarm sensor has data for the time range, the alert is effective, like:
onset: '2022-11-04T00:00:00+01:00'
expires: '2022-11-04T21:00:00+01:00'
would it be possible to show this start and end times on the card correcty formatted in some way?
Additional context
also the "description" data of the sensor would be a nice addition, in this case:
description: Regenmengen von 30 bis 50 mm sind zu erwarten.
Regards!
Gerwin.

Displaying any more stuff would be just visual clutter. The design goal is for the card to be as minimalistic as possible. Thanks for your input!


Separate daily integration test so that failure can assign issue to Alex

Is the Hilbert team/machine aware and ready for this?

Yes. @aradnaevinf can confirm.

open source actions are fre

Apparently this is not true to a certain limit. We got charged a lot for sitting on Hilbert's queue for hours. https://github.com/Infleqtion/server-superstaq/pull/2938 needs to go in so jobs don't keep idling.

Apparently https://github.com/Infleqtion/client-superstaq/pull/801 to a certain limit. We got charged a lot for sitting on Hilbert's queue for hours. https://github.com/Infleqtion/server-superstaq/pull/2938 needs to go in so jobs don't keep idling.

Is that true? My memory is hazy, but I thought we looked into it and we weren't actually being billed for an action from the server, not the client.

I thought we looked into it and we weren't actually being billed for an action from the server

You mean that we were being billed for the server? That could be it but i remember that we were overbilled around the time when our public Gituhub actions ran for like 9 hours cause Hilbert wasn't accepting jobs.

Going to try out https://github.com/Infleqtion/client-superstaq/pull/845 first.


Release 3.3.0 for Windows

Replacing Capital Xtra radio with Capital Xtra Reloaded radio.

https://github.com/kkamara/kelvinkamara/releases


How do I pass parameters to snapshot_create?

Was wondering how I'm able to pass label/extra parameters to snapshot.
The following method of adding params doesn't work. Was wondering if it had anything to do with how snapshot_create was written.
params = dict()
params['label] = 'new snapshot'
vultr.snapshot_create(subid,params=params)

Thanks!

Params should be a dictionary of parameters as per the Vultr API.
see: https://github.com/spry-group/python-vultr/blob/master/vultr/v1_snapshot.py#L18
see: https://www.vultr.com/api/#snapshot_create
According to the vultr documentation their API endpoint doesn't accepts a 'label' property.  Although it does take a 'description' property.

@dopry I'm getting the following error:
TypeError: snapshot_create() got an unexpected keyword argument 'params'

Is the way param is assigned causing the problem? Could having params in your function variable and inside the scope causing this issue?

Are you using the current pip release or master? The instructions I linked
you to are for master. You should probably use the master branch instead of
version on pypi. I've been meaning to cut a new release, but haven't had
time to wrap up the testing.
ᐧ
On Thu, Mar 30, 2017 at 3:01 PM, rlam3<EMAIL_ADDRESS>wrote:

@dopry https://github.com/dopry I'm getting the following error:
TypeError: snapshot_create() got an unexpected keyword argument 'params'
Is the way param is assigned causing the problem? Could having params in
your function variable and inside the scope causing this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spry-group/python-vultr/issues/39#issuecomment-290511798,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXqOG9_EFxfEinTQBnuMhbCUGAtN8FMks5rq_v4gaJpZM4Mq8iD
.

--
Darrel O'Pry
The Spry Group, LLC.
http://www.spry-group.com
718-355-9767 x101

Yes I'm using the pypi.
I was hoping for a newer version to be released since it has been a while.

You can just specify the github repo and branch when you pip install. I'll
try to tag and publish a release in the next week.
ᐧ
On Thu, Mar 30, 2017 at 3:06 PM, rlam3<EMAIL_ADDRESS>wrote:

Yes I'm using the pypi.
I was hoping for a newer version to be released since it has been a while.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spry-group/python-vultr/issues/39#issuecomment-290513321,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXqODBQaM2VPiMX1JbYzZBbm70mTdMcks5rq_0ogaJpZM4Mq8iD
.

--
Darrel O'Pry
The Spry Group, LLC.
http://www.spry-group.com
718-355-9767 x101

@dopry sounds good! thanks for your help!


Implement multiscales extension and generate array keys in store init

Codecov Report

Merging #4 into master will decrease coverage by 1.89%.
The diff coverage is 23.52%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   45.45%   43.56%   -1.90%
==========================================
  Files           4        4
  Lines          88      101      +13
==========================================
+ Hits           40       44       +4
- Misses         48       57       +9

Impacted Files
Coverage Δ

napari_lazy_openslide/lazy_openslide.py
43.05% <23.52%> (-2.71%)
:arrow_down:

Continue to review full report at Codecov.

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


[UBS-User. Orders tab view] UBS left panel doesn't have the arrow to collapse the menu

Environment: Chrome (latest).
Reproducible: always
Steps to reproduce

Logged in to GreenCity as a User https://ita-social-projects.github.io/GreenCityClient/#
The tab "Замовлення" in the personal cabinet is opened
Pay attention to the left menu panel

Actual result
There is the arrow before 4 tabs

Expected result
List includes 4 tabs
Mock-up: ITA UBS styles – Figma
User story and test case links
E.g.: "User story #3477
Test case
Labels to be added
"Bug", Priority ("pri: low"), Severity ("severity: trial"), Type ("UI")

Not a bug.
We are waiting for changing requirement.


[JAX] Rewrite the Format of FP8 Meta and Remove unused ShardingTypes.

Description
Reformatted FP8 meta to one set per tensor, removed fp8_max and scale_inv from the set of FP8 meta, and deleted unused functions and types.
Fixes # (issue)
To avoid unnecessary slice of FP8 meta then unblock pipeliner to re-schedule the collectives.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
The existing FP8 checkpoints would not be compatiable.

Changes
Please list the changes introduced in this PR:

Reformat FP8 meta to be one-set-per-tensor.
Remove fp8_max and scale_inv from FP8 meta set.
Remove unused functions in fp8.py.
Remove unused ShardingType and MajorShardingType.

Checklist:

[x] I have read and followed the contributing guidelines
[ ] The functionality is complete
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/te-ci jax

/te-ci jax

/te-ci jax

Can you tell what speed up you see with this PR?

/te-ci jax

@denera As this PR is removed some old API not used, we need to have that documented in the next release. Is there a place we need to add them to be sure to be included in the next releases?

/te-ci jax

/te-ci jax

LGTM  👍. Would be interesting to see the diff in performance if any.

Change to Draft for waiting internal verifiy

/te-ci jax

/te-ci jax


Clustering Algorithms Documentation added

Related Issue:#466

@Niketkumardheeryan @Rutikab12 Kindly review

Styling for fonts in the file is not perfect. Please re-check.

I have renamed and the styling seems fine now.Kindly review.

Styling for fonts in the file is not perfect. Please re-check.

@Rutikab12 I have renamed and the styling seems fine now.Kindly review.

it will be better if you could add images or sample code with doc

Okay I have added sample code but images I couldn't find so I have given plots to show in the code

Styling for fonts in the file is not perfect. Please re-check.

@Rutikab12 I have renamed and the styling seems fine now.Kindly review.

it will be better if you could add images or sample code with doc

@Niketkumardheeryan @Rutikab12 Okay I have added sample code but images I couldn't find so I have given plots to show in the code

one more suggestion - Don't underestimate documentation, work on your documentation wala part


The repository lacks a license.

Hi!
I stumbled across this repo after reading the book. It is nice to have all the code examples in the book available online, but the repo doesn't specify a license to the code. The Publisher's repo does have a license file, specifying the MIT license. Please consider adding a license file if you want to allow people to use the code.

If you can create a pull request, I'll be happy to merge it. Thank you!


Remove ENTITY_TYPE references

See https://github.com/Senzing/community-roadmap/issues/66

Already done


New version: StatsKit v0.3.1

Registering package: StatsKit
Repository: https://github.com/JuliaStats/StatsKit.jl
Created by: @nalimilan
Version: v0.3.1
Commit: 7657ae87a9f17c2e12be66ccbfd44d95ea2c1d43
Reviewed by: @nalimilan
Reference: https://github.com/JuliaStats/StatsKit.jl/commit/7657ae87a9f17c2e12be66ccbfd44d95ea2c1d43#commitcomment-50004813

The package doesn't use version bounds for dependencies since it's a meta-package that is just there to install and load the latest version of all dependencies.

At least julia? See #35311

Good point. I've added bounds for these.


Request new release 1.6.22 for moby-containerd in https://debian.pkgs.org/11/microsoft-prod-amd64

Describe the problem you are experiencing.

Latest version for moby-containerd is 1.6.19. Can you help release new version 1.6.22 for moby-container? 1.6.22 addressed one potential vulnerability issue  by https://github.com/containerd/containerd/pull/8642. Thanks a lot!
https://debian.pkgs.org/11/microsoft-prod-amd64/moby-containerd_1.6.19+azure-debian11u1_amd64.deb.html
Describe any possible solutions that you would like to see.

Additional context.

Thanks for bringing this to our attention. An updated package should be published within the next 24 hours.


lastversion (sometimes) does not correctly fetch a version from github releases

First off, I'd like to extend my sincere gratitude for the development of the "lastversion" tool.
It has been incredibly beneficial for my workflows.
However, I've recently encountered a potential issue which I'd like to report:
When I attempt to fetch the latest version from GitHub releases of https://github.com/CGAL/cgal using the lastversion tool,
it does not provide the correct version.
Steps to Reproduce:
$ lastversion --format tag https://github.com/CGAL/cgal
v5.5.3

$ lastversion  https://github.com/CGAL/cgal
5.5.3

$ lastversion --version
lastversion 3.3.1, up to date

However, the actual latest release for the CGAL/cgal repo is v5.6.

https://github.com/CGAL/cgal/releases/tag/v5.6

minimal docker test code:
docker run --network=host -it --rm python:3.11 \
   bash -c "pip3 install lastversion==3.3.1 && lastversion  https://github.com/CGAL/cgal && lastversion --version"

And my test results:

program
CGAL/cgal version

lastversion==3.0.1
5.6

lastversion==3.1.1
5.5.3

lastversion==3.3.1
5.5.3

I hope this report helps in rectifying the issue.

Thank you for the bug report. This should have been fixed in v3.3.2

Thank you!
it is working !


[BUG] ...Privacy Mode button is now located in Days of Buffering area

Describe the bug
On the recent update of YNAB (June 18) Privacy Mode lock button location is thrown on the Days of Buffering area.
To Reproduce
Enable Privacy Mode
Expected behavior
It should be placed beside the budget name as it was before.
Screenshots

Desktop (please complete the following information):

OS: MacOS Catalina 10.15.5
Chrome 83.0.4103.97
Toolkit Version: 2.24.0

Setting Export
Please replace the text below with your exported Toolkit settings. You can export these by going to the Toolkit Options page, click the Import/Export Settings button at the bottom left and copying the text in the modal which appears.
[{"key":"AccountsDisplayDensity","value":"1"},{"key":"AccountsEmphasizedOutflows","value":false},{"key":"AccountsStripedRows","value":true},{"key":"AutoDistributeSplits","value":false},{"key":"AutoEnableRunningBalance","value":false},{"key":"AutomaticallyMarkAsCleared","value":false},{"key":"BetterScrollbars","value":"0"},{"key":"BottomNotificationBar","value":false},{"key":"BudgetCategoryFeatures","value":true},{"key":"BudgetProgressBars","value":"0"},{"key":"BudgetQuickSwitch","value":false},{"key":"BudgetSpendingGoal","value":false},{"key":"BulkManagePayees","value":false},{"key":"CalendarFirstDay","value":"0"},{"key":"CategoryActivityCopy","value":false},{"key":"CategoryActivityPopupWidth","value":"0"},{"key":"CategorySoloMode","value":"0"},{"key":"ChangeEnterBehavior","value":false},{"key":"ChangeMemoEnterBehavior","value":false},{"key":"CheckCreditBalances","value":false},{"key":"CheckNumbers","value":false},{"key":"ClearSelection","value":false},{"key":"CollapseSideMenu","value":false},{"key":"ColourBlindMode","value":false},{"key":"CompactIncomeVsExpense","value":true},{"key":"ConfirmKeyboardCancelationOfTransactionChanges","value":false},{"key":"CreditCardEmoji","value":false},{"key":"CtrlEnterCleared","value":false},{"key":"CurrentMonthIndicator","value":false},{"key":"CustomFlagNames","value":false},{"key":"DateOfMoney","value":true},{"key":"DaysOfBuffering","value":true},{"key":"DaysOfBufferingDate","value":true},{"key":"DaysOfBufferingHistoryLookup","value":"0"},{"key":"DeselectTransactionsOnSave","value":false},{"key":"DisableToolkit","value":false},{"key":"DisplayTargetGoalAmount","value":"0"},{"key":"DisplayTotalMonthlyGoals","value":false},{"key":"DisplayUpcomingAmount","value":true},{"key":"EasyTransactionApproval","value":false},{"key":"EditAccountButton","value":"2"},{"key":"EnlargeCategoriesDropdown","value":true},{"key":"EnterToMove","value":false},{"key":"FilterCategories","value":false},{"key":"GoalIndicator","value":false},{"key":"GoalWarningColor","value":false},{"key":"GoogleFontsSelector","value":"0"},{"key":"HideAccountBalancesType","value":"0"},{"key":"HideAgeOfMoney","value":false},{"key":"HideClosedAccounts","value":false},{"key":"HideHelp","value":true},{"key":"HideReferralBanner","value":true},{"key":"HideTotalAvailable","value":false},{"key":"HighlightNegatives","value":false},{"key":"ImportNotification","value":"0"},{"key":"IncomeFromLastMonth","value":"0"},{"key":"IncomeVsExpenseHoverHighlight","value":true},{"key":"LargerClickableIcons","value":false},{"key":"LinkToInflows","value":false},{"key":"MonthlyNotesPopupWidth","value":"0"},{"key":"NavDisplayDensity","value":"2"},{"key":"NotesAsMarkdown","value":false},{"key":"POSStyleCurrencyEntryMode","value":false},{"key":"Pacing","value":"0"},{"key":"PrintingImprovements","value":true},{"key":"PrivacyMode","value":"0"},{"key":"QuickBudgetWarning","value":false},{"key":"ReconciledTextColor","value":"0"},{"key":"RemovePositiveHighlight","value":false},{"key":"RemoveZeroCategories","value":false},{"key":"ResizeInspector","value":false},{"key":"ResizeSidebar","value":false},{"key":"RightClickToEdit","value":true},{"key":"RowHeight","value":"0"},{"key":"RowsHeight","value":"0"},{"key":"SeamlessBudgetHeader","value":false},{"key":"SetMultipleFlags","value":false},{"key":"ShowCategoryBalance","value":false},{"key":"SpareChange","value":false},{"key":"SplitTransactionAutoAdjust","value":false},{"key":"SquareNegativeMode","value":false},{"key":"StealingFromFuture","value":false},{"key":"StripedBudgetRows","value":false},{"key":"SwapClearedFlagged","value":false},{"key":"TargetBalanceWarning","value":false},{"key":"ToBeBudgetedWarning","value":false},{"key":"ToggleSplits","value":false},{"key":"ToggleTransactionFilters","value":"0"},{"key":"ToolkitReports","value":true},{"key":"ViewZeroAsEmpty","value":true},{"key":"l10n","value":"0"},{"key":"options.dark-mode","value":false}]

Hey @byronvm  the fix for this should be in our current release! If you notice the issue still exists and you're definitely running the newest version of the toolkit, please reopen! Thanks!
Please note, all browser can take up to an hour to provide updates to all users. You can force an update if it's available for you by going to chrome://extensions and clicking the "Update" button. Firefox users can go to about:addons, click the setting cog, and click "Check for updates...".

I updated to version 2.25.0 It's now located in lower right corner. Isn't it supposed to be beside the budget name? And it's worse than before, it's not working for me. The toggle does nothing. I tried Always On, but still no effect.
I tried removing and reinstalling the extension then importing back my settings. Nothing. I tried resetting the settings and applying again, still nothing. It's as if the privacy mode is dead for me.

hmm, weird that it's not working. I moved it there on purpose (i think it fits better, and there's a new "collapse sidebar" feature coming where it'll make more sense)
I can look into this in the next few days. sorry about that!


Setting up grid for NorESM2-LM (vertical levels equal across variables)

Hi! I'm trying to create a grid object for NorESM2-LM (grid label: gr) which has vertical levels equal across variables:
cat_url = "https://storage.googleapis.com/cmip6/pangeo-cmip6-noQC.json"
col = intake.open_esm_datastore(cat_url)
cat = col.search(table_id='Omon',experiment_id='historical',
             variable_id=['wmo','umo'], grid_label='gr', member_id='r1i1p1f1',source_id='NorESM2-LM')
noresm2_lm = cat.to_dataset_dict(
        zarr_kwargs={'consolidated':True, 'decode_times': True, 'use_cftime': True},
        preprocess=combined_preprocessing,
        aggregate=False)
wmo = noresm2_lm['CMIP.NCC.NorESM2-LM.historical.r1i1p1f1.Omon.wmo.gr.gs://cmip6/CMIP6/CMIP/NCC/NorESM2-LM/historical/r1i1p1f1/Omon/wmo/gr/v20190815/.nan.20190815.resolved.low.https://errata.es-doc.org/static/view.html?uid=5ebabff0-388f-07bc-b2cf-d44dcbb2940f']
umo = noresm2_lm[ 'CMIP.NCC.NorESM2-LM.historical.r1i1p1f1.Omon.umo.gr.gs://cmip6/CMIP6/CMIP/NCC/NorESM2-LM/historical/r1i1p1f1/Omon/umo/gr/v20190815/.nan.20190815.resolved.low.https://errata.es-doc.org/static/view.html?uid=5ebabff0-388f-07bc-b2cf-d44dcbb2940f']
diff = wmo.lev-umo.lev
diff.plot()

umo = umo.rename({'x':'x_g','lon':'lon_u', 'lat':'lat_u'})
ds_subset = xr.merge([umo, wmo], compat='override')
from xgcm import Grid
grid = Grid(ds_subset, coords={'X':{'center':'x', 'left':'x_g'}, 'Z':{'center':'lev'},},)
dw_dz = grid.diff(ds_subset.wmo, 'Z')

`KeyError: 'center'`

Is there a way to pre-format the grid through cmip6_pp before creating the grid object? cc: @jbusecke

I am looking into this right now. Quick comment: you are using the non-QC catalog. This is quite dangerous for 2 reasons:

If the dataset is in this catalog and not in the main one ( "https://storage.googleapis.com/cmip6/pangeo-cmip6.json") there is probably something wrong with it.
Maybe more importantly, our retraction logic only applies filtering to the main catalog, so the data you work with might have been retracted!

I strongly recommend to only use  "https://storage.googleapis.com/cmip6/pangeo-cmip6.json" for your work.

Let me see if @AleksiNummelin knows how to proceeed here.
Hey Aleksi,
to quickly summarize: @jdldeauna is trying to compute tracer budgets from the NorESM models, but for the regridded (grid_label='gr') output, the vertical transport is on the tracer points. Am I assuming rightfully, that both the horizontal/vertical transports have been interpolated onto the tracer point, and as such cannot be used to get exact budgets for the tracer cell (since we usually want the flux terms on the surrounding cell faces)?

Hi Julius, I think we actually had a bit of a email exchange with @jdldeauna earlier this month around some of these issues, but I think this might be a bit distinct. Anyway, here's a bit of a re-iteration of some of the points from the email conversation. I wanted to include also @matsbn here just to double check as he knows the code better than I do (Mats, please correct if I'm talking nonsense here).
NorESM ocean model (https://github.com/NorESMhub/BLOM) works on a potential density coordinate so the 'lev' grid (z-coordinate) is arbitrary and indeed interpolated. wmo is in fact not a model variable, but diagnosed from (accumulated) horizontal transports for a given layer (code starting at https://github.com/NorESMhub/BLOM/blob/master/phy/mod_dia.F#L3863). Now, although the horizontal (umo,vmo) transports are defined at a given level, they are flux variables and depth=0 m is actually the flux between 0-2.5 m depth. Now wmo has the same lev coordinate as umo, vmo (but not lon,lat), but if I interpret the code correctly, it is actually the top interface flux i.e. wmo at depth=0 is really the flux trough depth=0 interface, whereas umo, vmo at depth=0 are really the fluxes trough the edges of a 2.5 m thick cell. Therefore, I think it should still be possible to close the budgets. I had a quick try and (sorry, just pseudo code here, but I think you get the idea)
conservation_error = - wmo.diff('lev').isel(lev=0) - (umo.isel(i=slice(1,nx),j=slice(0,-1))-umo.isel(i=slice(0,-1),j=slice(0,-1))).isel(lev=0) - (vmo.isel(j=slice(1,ny),i=slice(0,-1))-vmo.isel(j=slice(0,-1),i=slice(0,-1))).isel(lev=0)
is still considerable, but probably on the order I would expect and the global sum is tiny. Obviosuly, one should wrap around longitude properly and assume flux is 0 trough the bottom.
Hope this helps. I realize that this might not fit right into the cmip6_preprocessing workflow, but probably it would be easiest to redefine the umo,vmo depth axes.

Thank you for the discussion on this issue, and sorry for the late reply!

I strongly recommend to only use "https://storage.googleapis.com/cmip6/pangeo-cmip6.json" for your work.

I use the no-QC version when I need to use MPI-ESM1-2-HR salinity since it has an errata. It's for locations in the small bays of the Baltic, so I figured it would be fine for my purposes in the North Pacific.

If I understand correctly I could try to add an additional layer of 0's to the bottom of wmo and then treat them as the 'outer' fluxes?

Just to clarify, when would it count as a variable is shifted to the left vs when it should be treated as outer?


Duplicate Section title

https://opencollective.com/sustainoss/events/sustainuk-2018-442ev#budget

good catch @znarf

Fixed in https://github.com/opencollective/frontend/pull/555


Change findIntentsForContext -> findIntentsByContext in example

Method name was incorrect in the example.

Thank you for your pull request and welcome to our community! We require contributors to sign a Contributor License Agreement and we don't seem to have CLAs on file for these contributors to the Pull Request: (@richlinnell). In order for your PR to be reviewed and merged, please follow the directions at the link above.
Project team: please do not merge this Pull Request until Foundation staff have confirmed that a CLA is in place for the new contributor(s) listed above.
If there are any questions, please don't hesitate to get in touch with our Infrastructure Support team. /CC @finos-admin

Thank you for your pull request and welcome to our community! We require contributors to sign a Contributor License Agreement and we don't seem to have CLAs on file for these contributors to the Pull Request: (@richlinnell). In order for your PR to be reviewed and merged, please follow the directions at the link above.
Project team: please do not merge this Pull Request until Foundation staff have confirmed that a CLA is in place for the new contributor(s) listed above.
If there are any questions, please don't hesitate to get in touch with our Infrastructure Support team. /CC @finos-admin

Thank you for your pull request and welcome to our community! We require contributors to sign a Contributor License Agreement and we don't seem to have CLAs on file for these contributors to the Pull Request: (@richlinnell). In order for your PR to be reviewed and merged, please follow the directions at the link above.
Project team: please do not merge this Pull Request until Foundation staff have confirmed that a CLA is in place for the new contributor(s) listed above.
If there are any questions, please don't hesitate to get in touch with our Infrastructure Support team. /CC @finos-admin


Record/display statistics from study mode

Similar to what chess position trainer already has, have a percentage meter next to the opening to show a percentage of how much of the opening you memorized.

43d820715d4ead2de453ad2339298940dd66430e
I've added a new "Evaluate" mode which shows statistics about how well you're studying the lines/positions in a given repertoire. Here's a screenshot:


Add debug logs in PartitionSelectionHelper class

Codecov Report
Base: 73.02% // Head: 14.80% // Decreases project coverage by -58.21% :warning:

Coverage data is based on head (fdbdd35) compared to base (0932a60).
Patch coverage: 39.13% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2316       +/-   ##
=============================================
- Coverage     73.02%   14.80%   -58.22%
+ Complexity    10497     2363     -8134
=============================================
  Files           757      757
  Lines         58241    58261       +20
  Branches       7152     7156        +4
=============================================
- Hits          42530     8627    -33903
- Misses        13350    49118    +35768
+ Partials       2361      516     -1845

Impacted Files
Coverage Δ

...a/com/github/ambry/clustermap/ClusterMapUtils.java
43.09% <39.13%> (-49.15%)
:arrow_down:

.../src/main/java/com/github/ambry/router/Router.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/com/github/ambry/store/Message.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/named/PutResult.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaMode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaName.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/rest/RestMethod.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/KMSConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/SSLConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/router/ByteRange.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 594 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


2 graphql servlets

I'd like to be able to start 2 or more graphql servlets on different paths. This way I'd be able to configure different security settings per servlet with spring security (e.g. give public access to one while secure the other). Is this a reasonable request?
the config would look like:
graphql:
servlet:
mapping: /graphql, /graphql2
Thanks,

Roland

I wouldn't configure Spring Security on servlets like that, but annotate the actual service methods that are being called by the GraphQL resolvers. That way you can authorize differently per query or mutation level, and filter response objects based on the principal.

Thanks oliemansm! Indeed, it would be great to be able to do it that way, but I'm struggling with the configurations. I asked for help here:
https://stackoverflow.com/questions/45959234/authentication-in-spring-boot-using-graphql
Could you help me out answering on stackoverflow?

Can you give me please a spring/spring-boot solution to have some graphql operation that can be anonymously accessed, while others are secured?
If I secure the "/graphgl" endpoint with .antMatchers("/graphql").authenticated(), I can use @PreAuthorize annotations on service methods to restrict existing roles, but anonymous users are not allowed access at all.

Instead of .antMatchers("/graphql").authenticated() try to .antMatchers("/graphql").permitAll(). You're @PreAuthorize annotations should still work while anonymous access is also allowed.
Instead of doing basic auth on the servlet endpoint itself, you can create a mutation to create a token given a username and password, e.g.
mutation createSession(input: {username:"foo", password:"bar"}) {
 token
}

Have the server generate a JWT token for example for this mutation. Use middleware to add this token in the Authorization header with each GraphQL request. Add a authentication filter in the Spring Security Chain to check the token and load the user in the Security Context. You can specify that the session should be stateless in the security configuration.
I was trying to find an example to answer your stackoverflow question, but couldn't find one yet and hadn't come around to it create one myself. Does the above help you?
Best,
Michiel
Groet,
Michiel

From: rolandkozma<EMAIL_ADDRESS>Sent: Tuesday, September 5, 2017 10:08:04 AM
To: graphql-java/graphql-spring-boot
Cc: Michiel Oliemans; Comment
Subject: Re: [graphql-java/graphql-spring-boot] 2 graphql servlets (#29)
Can you give me please a spring/spring-boot solution to have some graphql operation that can be anonymously accessed, while others are secured?
If I secure the "/graphgl" endpoint with .antMatchers("/graphql").authenticated(), I can use @PreAuthorizehttps://github.com/preauthorize annotations on service methods to restrict existing roles, but anonymous users are not allowed access at all.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/graphql-java/graphql-spring-boot/issues/29#issuecomment-327102958, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AM5FQMbBIYMUlAiF1ikbtcMeKyFkNXBBks5sfQFkgaJpZM4PKBiG.

Thank you very much for your suggestions! I'm going to try to implement them right away. I'll let you know about my progress.

I'm happy to let you know that we solved the issue. The filter was not even needed. Spring is able to restore the session from redis if it is persisted as Spring expects. I answered my question on stackoverflow. Thanks a lot for your suggestions!
Is it a best practice to have the login/logout operations as mutations?

Glad to hear it! I think it is, yeah.
Best,
Michiel

From: rolandkozma<EMAIL_ADDRESS>Sent: Wednesday, September 6, 2017 11:07:51 AM
To: graphql-java/graphql-spring-boot
Cc: Michiel Oliemans; Comment
Subject: Re: [graphql-java/graphql-spring-boot] 2 graphql servlets (#29)
I'm happy to let you know that we solved the issue. The filter was not even needed. Spring is able to restore the session from redis if it is persisted as Spring expects. I answered my question on stackoverflow. Thanks a lot for your suggestions!
Is it a best practice to have the login/logout operations as mutations?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/graphql-java/graphql-spring-boot/issues/29#issuecomment-327423237, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AM5FQJUsoPm3EiX8fdL3L-rkKViMSnpsks5sfmDngaJpZM4PKBiG.


/cmd_vel for hover

Hi!
Just something that I would like to ask. I noticed that the twist messages in the topic /cmd_vel are actually used for roll pitch yaw thrust control, while in other platforms this is used for body fixed velocity control.
Wouldn't it be more logical to use this topic for the send hover commands instead? Than it will be consistent with other ROS packages

I think the best would be to only define custom messages that are clearly
documented rather than reusing twist messages:-) I think we did so for the
other cmd. Since we have our own teleop node, it shouldn't be a
compatibility problem.
On Wed, Jul 20, 2022, 08:48 Kimberly McGuire @.***>
wrote:

Hi!
Just something that I would like to ask. I noticed that the twist messages
in the topic /cmd_vel are actually used for roll pitch yaw thrust
https://github.com/IMRCLab/crazyswarm2/blob/8b7444b1858bd26efdb6a6c5f4b6893e8e54341a/crazyswarm2/src/crazyswarm2_server.cpp#L1036-L1049
control, while in other platforms this is used for body fixed velocity
control.
Wouldn't it be more logical to use this topic for the send hover commands
https://github.com/IMRCLab/crazyswarm2/blob/8b7444b1858bd26efdb6a6c5f4b6893e8e54341a/crazyswarm2/src/crazyswarm2_server.cpp#L1099-L1107
instead? Than it will be consistent with other ROS packages
—
Reply to this email directly, view it on GitHub
https://github.com/IMRCLab/crazyswarm2/issues/45, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA3SIPI56MA2IRU7674YAHTVU6ONBANCNFSM54CN2LUA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

true, but perhaps we could consider also having at least a twist based /cmd_vel as well as that will make it easier to connect it to other packages outside of the crazyswarm package.
How about we make a custom message for the RPYT commands that listens to other topic like /cmd_rpty that is more for internal crazyswarm use, and we will have another function that send hover commands based on twist messages on /cmd_vel from external packages

another option, perhaps not something I  am the biggest fan of but perhaps a way to really seperate it from the server, that we make a crazyflie bridge node that turns /cmd_vel commands into /hover_cmd topics, which people can put in the launchfile in case they want to use these external packages.

I think we can keep the functionality in the server, but perhaps name the topic something that avoids users using it, if they have the choice, e.g., cmd_vel_legacy.


an error in polymer should never be a blank screen

 Issue by sigmundch
Originally opened as dart-lang/sdk#19045

Often an error in a polymer application is hard to diagnose. You end up seeing a blank screen an no errors in the console. This happens sometimes in Dartium, sometimes after compiling.
We should improve our feedback and diagnostic messages to avoid these kind of scenarios as much as possible.
There are many scenarios and corner cases we need to investigate and improve. One scenarios that recently came up was:
 * code from 0.10.0-pre.12 which is now deprecated (no application/dart;component=1): this is detected in pub-build, but not detected in Dartium
Separately, we should improve the documentation to help users independently diagnose issues:
 * how to turn on logging
 * list the typical troubleshooting we do when we debug an issue:
     * is '@initMethod' called
     * is the timing right? For example, are elements being upgraded?

 Comment by zoechi

That would be great!

 Comment by sigmundch

Added this to the Later milestone.

 Comment by jmesserly

Removed Area-Polymer label.
Added Pkg-Polymer, Area-Pkg labels.

 Comment by sigmundch

Removed Polymer-Milestone-Later label.
Added PolymerMilestone-Later label.

 Comment by sigmundch

Removed this from the Later milestone.

 Comment by sigmundch

Added Polymer-P-2 label.

 Comment by sigmundch

Removed Polymer-P-2 label.
Added Polymer-Milestone-Later label.


Google map circle is not a perfect circle

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Use google_maps_flutter to render a google map with a circle in flutter web
Expected results
A perfect circle should be rendered on the google map
Actual results
Circle is not perfect
Code sample
Code sample
class Map extends StatelessWidget {
  const Map({super.key});

  @override
  Widget build(BuildContext context) {
    return GoogleMap(
      circles: {
       const Circle(
          circleId:  CircleId('test'),
          center:  LatLng(23.77, 134.484),
          radius: 100,
          strokeWidth: 2,
        )
      },
      initialCameraPosition: const CameraPosition(
        target: LatLng(23.77, 134.484),
      ),
      zoomControlsEnabled: false,
      scrollGesturesEnabled: false,
      onMapCreated: (controller) {},
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.3.10, on macOS 13.4 22F66 darwin-x64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
[✓] Chrome - develop for the web
[!] Android Studio (version 2022.2)
    ✗ Unable to find bundled Java version.
[✓] IntelliJ IDEA Community Edition (version 2022.2.1)
[✓] VS Code (version 1.80.1)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

! Doctor found issues in 1 category.

The circle is drawn by Google Maps; the plugin just passes it information like the center and radius. Issues with the actual rendering would need to be reported to Google Maps.


Navbar icons not aligned properly on mobile devices with screen width 350-400px.

1 - Describe the bug
The navbar icons like the Github icon, dark theme icon and the download icon are not aligned properly on mobile devices. On some devices like the Moto G4, the icons are not even visible.
2 - Expected behavior of what should have happened (if applicable)
The icons should be visible on the top right of the navbar even on mobile devices or can be placed below the logo SButtons but aligned at the center to make it look good.
3 - Screenshots (if applicable)

4 - Additional notes
I would like to work on it.

@pinkman7009 instead of making them aligned in the navbar, can you just hide everything but the theme icon on small devices (less than 400px)?

@shahednasser yes sure. should i keep the theme icon at the right of the navbar like in the desktop view?

Yes please

@shahednasser sure ill get started with this.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 16:15:29 on 2023.05.26
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 4, errors out: 0
Original problem ID: 22581
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=22581

Problem has been resolved in 6m 0s at 16:21:29 on 2023.05.26
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 22581


Question: benchamrk about tcmalloc and memcpy

hi, I am reading the code and have done some benchmark:
https://github.com/guangqianpeng/libaco/blob/master/bench_result
I have two questions:

tcmalloc improves the benchmark results. With aco_amount=1000000 and copy_stack_size=56B, the tcmalloc version achieves 37ns per aco_resume() operation but the default takes 66ns. Why? In this case, aco_resume() does not allocate memory, which is really confusing...
When copying stack, you use %xmm registers to optimize small memory copying. But according to my benchmark, this does not make many differences. I guess memcpy() already takes advantage of these registers. Do you have more benchmark results?

I will be very grateful to you for answering my questions :-)

Hi @guangqianpeng,

tcmalloc improves the benchmark results. With aco_amount=1000000 and copy_stack_size=56B, the tcmalloc version achieves 37ns per aco_resume() operation but the default takes 66ns. Why? In this case, aco_resume() does not allocate memory, which is really confusing...

I think the main reason of such result is because the tcmalloc has many specialized optimizations on memory efficiency and locality especially when the allocation of small objects, which is much better than the vanilla glibc allocator.

When copying stack, you use %xmm registers to optimize small memory copying. But according to my benchmark, this does not make many differences. I guess memcpy() already takes advantage of these registers. Do you have more benchmark results?

Here is actually a knack in the code like __uint128_t xmm0: the gcc would try to use sse to optimize the operations about the __uint128_t data type (whereas the clang does not, as far as I know). So if you want to use such sse optimization in libaco now, you could use gcc to compile the libaco into a static library and then use a linker to link it with any object file you like. Also, you could choose to use objdump to inspect the actual machine code generated by the aco_resume function.
Even when there is no such sse enhancement provided by the compiler, such "very-short memcpy inline" with these general purpose registers are still more efficient than to call the libc memcpy directly. That is because, in the case of such short copy, the cost of a function call is not small enough to neglect anymore. So, there would be some gains anyway.
Maybe in the future we should choose to use the sse directly instead of counting on such compiler's behavior ;-) But I'm afraid that such plan has to be postponed since there is a much more important thing to do now, i.e. the #22.

I will be very grateful to you for answering my questions :-)

All the discussions and questions about libaco would always be welcomed here. Just feel free to open any new issue you like :D

2. Do you have more benchmark results?

I did some benchmark about such conditional memcpy-inline in the past and did get the result I want. But there were no records. I would like to do another test as soon as I get another spare time.

I did use perf tool to check L1 dcache miss rate of the two versions, tcmalloc version achieved about the half miss rate of glibc. I guess there was false sharing or other cache problems and tried to solve it without tcmalloc. But I finally failed :-(

I did have inspect the assembly code of co_resume() and saw such movqd %rbx %xmm1 things, I also trace into the glibc and found that memcpy finally call __memcpy_avx_unaligned(), which use AVX instruction sets. What I didn't think of is that the overhead of  memcpy() call cannot be ignored, especially for small stack.

BTW, the libaco project is great, I am looking forward to your next version (especially the co schedualer).

BTW, the libaco project is great, I am looking forward to your next version (especially the co scheduler).

Thank you very much for your kind encouragement, @guangqianpeng, and I would try to finish the next release as soon as possible ;-)

BTW, the libaco project is great, I am looking forward to your next version (especially the co scheduler).

Thank you very much for your kind encouragement, @guangqianpeng, and I would try to finish the next release as soon as possible ;-)

How about the next release? :)


ImageView load event sometimes not fired

Consecutively loading an image with the same source sometimes doesn't result into the 'load' event being fired.
Working around this problem to show an activity indicator for the cases described below is very hard as it would require manually maintaining the loaded state for the image resource and configuration scenario.

fired the very first time the image is loaded
fired on consecutive loading attempt when image object width and height are set
fired on consecutive loading attempt when image object width and height are not set

Android
✓
✗
✗

iOS
✓
? (fired for some values in the example below, e.g. w250, h250, but not for others, e.g. w220, h220)
✓

Example:
var page = new tabris.Page({topLevel: true});

var button = new tabris.Button({
  centerX: 0, top: 10,
  text: "Show me a llama"
}).appendTo(page);

button.on("select", function() {
  new tabris.ImageView({
    image: {
      src: "https://upload.wikimedia.org/wikipedia/commons/thumb/b/b9/Llama_lying_down.jpg/1920px-Llama_lying_down.jpg",
      // height: 200,
      // width: 200
    },
    centerX: 0, width: 200, height: 200, top: "prev() 10"
  }).on("load", function(widget, event) {
    console.log("Load event fired: " + JSON.stringify(event));
  }).appendTo(page);
});

page.open();

@cpetrov I would not expect a resize of the ImageView to trigger a load event. Would that be your expectation?

After clarifying the issue the following situation will be resolved Android client side for now: https://github.com/eclipsesource/tabris-js/issues/867#issuecomment-260642726

Fixed in the 2.0 stream.


Style the navbar

Current style in style-navbar branch

suggestions from @LoganWilliams1 on Discord channel:

add SLASH logo to the navbar so its always at top of the screen
add SLASH logo to the RequestModal popup so it looks nicer. to be really fancy could try a hacky hardcoded progress bar? make the progress bar animation go for about 20 seconds (average scraper duration)


[General Inquiry]: is unauthenticated user pool secure ?

What is your general inquiry?
AWS CloudWatch RUM leverages AWS Congnito and uses an unauthenticated indentity to "PutRumEvents".
Does it mean that an malicious user of my web application can pretend to be the unauthenticated indentity to assume the IAM role and start abuse the "PutRumEvents" API thus pollute the frontend_logs or metrics collected by RUM ?
Many thanks in advance

We have throttling on the backend service to prevent an abnormal amount of incoming data (this handles the traffic spamming)
We have event schema based validation on the incoming RUM events. As a result, the spammers will have to still create data that follow the predefined event schemas. However, they can still create some arbitrary numbers to send inaccurate data.

That being said, due to the nature of Cognito, we cannot fully prevent a malicious user from spamming data.

an malicious user of my web application can pretend to be the unauthenticated indentity to assume the IAM role and start abuse the "PutRumEvents" API thus pollute the frontend_logs or metrics collected by RUM ?

This is an inherent risk of real user monitoring. The risk of receiving spoofed data may be increased when collecting data from unauthenticated users, but the risk is present even when collecting data from authenticated users. I wrote a blog post titled "How to isolate signed-in users from guest users within Amazon CloudWatch RUM" that discusses this topic.
I'm closing this issue because it's in the domain of fraud prevention, which doesn't fall within the scope of this repository. Because this question is related to the CloudWatch RUM service itself, I recommend going through AWS support channels for more information.


Kubeconfig expiration metric

Description
Changes proposed in this pull request:

Introduces kubeconfig expiration metric
...
...

Related issue(s)
#11

Example visualisation of metrics exposed in this PR


Reintroduce monitoring 45.31.1 for release 2.9

45.31.1 was accidentally removed in charts during the monitoring rebase to 57.x

Rancher-monitoring:104.0.0+up45.31.1 chart is available on for release 2.9 and able to install and successfully.


Error ‘assert len(chunk_encode) == 2’ when eval MMMU_DEV_VAL

When I try to test MMMU_DEV_VAL for LlaVa
The assert len(chunk_encode) == 2 is triggered, after print, I find that len(chunk_encode)=4
Since $inputs$ is:
<|start_header_id|>user<|end_header_id|>
[image]
Determine the area in hectares between the line AB and a meandering stream for offsets taken at a regular interval of 20 m along the line AB (Fig. 12.5). Use both the trapezoidal rule and Simpson's rule.
[Image]
[Image]
A. Use trapezoidal rule,area = 0.5010 hectares;Use trapezoidal rule,area = 0.5560 hectares
B. Use trapezoidal rule,area = 0.5010 hectares;Use trapezoidal rule,area = 0.5460 hectares
C. Use trapezoidal rule,area = 0.5010 hectares;Use trapezoidal rule,area = 0.5260 hectares
D. Use trapezoidal rule,area = 0.5010 hectares;Use trapezoidal rule,area = 0.5360 hectares
Answer with the option's letter from the given choices directly.<|eot_id|><|start_header_id|>assistant<|end_header_id|>
After split input with [image], we would get 4 parts more than 4, and the assert is triggered.

seems that error come from:
https://github.com/open-compass/VLMEvalKit/blob/dc2e6d87e758c7f0004312fe140902ba83ff8c02/vlmeval/vlm/base.py#L151

There are multi-image in the prompt, and prompt is:
Determine the area in hectares between the line AB and a meandering stream for offsets taken at a regular interval of 20 m along the line AB (Fig. 12.5). Use both the trapezoidal rule and Simpson's rule.
[Image]
[Image]
A. Use trapezoidal rule,area = 0.5010 hectares;Use trapezoidal rule,area = 0.5560 hectares
B. Use trapezoidal rule,area = 0.5010 hectares;Use trapezoidal rule,area = 0.5460 hectares
C. Use trapezoidal rule,area = 0.5010 hectares;Use trapezoidal rule,area = 0.5260 hectares
D. Use trapezoidal rule,area = 0.5010 hectares;Use trapezoidal rule,area = 0.5360 hectares
Answer with the option's letter from the given choices directly.
seems that image should be discarded.

seems that error come from:
https://github.com/open-compass/VLMEvalKit/blob/dc2e6d87e758c7f0004312fe140902ba83ff8c02/vlmeval/vlm/base.py#L151
Since Model LLaVA_XTuner does not support interleaved input. Will use the first image and aggregated texts as prompt.

Modifying the 'image' into another name would solve this issue. and such images would be discarded.


Add desktop support

I know saying is easier than done but due to the nature of flutter if you can use dart_vlc as a desktop replacement using platform interface then it would be good, also dart_vlc let's you customize the player controls playback speed, source, etc, etc.

Thanks @prateekmedia  for letting me know about this awesome plugin dart_vlc
I will definitely give it a try
But this plugin doesn't provide support for Android and web platform?
So if we try to implement this plugin, doesn't it effect Android and web platforms..?

@newtaDev Nope, there is something called as platform interface so you will have to divide this repk into two folders pod_player and pod_player_platform_interface. If you want you can also create pod_player_desktop folder.
It will have the logic that for which platform what plugin dependency and code has to be used
You can see how just_player does it.

@newtaDev Nope, there is something called as platform interface so you will have to divide this repk into two folders pod_player and pod_player_platform_interface. If you want you can also create pod_player_desktop folder.
It will have the logic that for which platform what plugin dependency and code has to be used
You can see how just_player does it.

Thank you for sharing.
I will try to implement


Lowered the iOS deployment target from 12.1 to 9.0

The iOS deployment target was set to 12.1. Since this is higher than the react native default (9.0), the package was left out of release builds and, as a result, making them fail.
Fixes #17

Thanks for making this PR!

Thanks for making this PR!

Waiting for this PR!:)

when will this be merged?

+1

What happened to this project?  This issue has consum lots of time of many developers..  And after all these days.  It still there?

Let's get this in

Let's get this in

Thanks a lot, orta

@orta can you please also publish a patch version w/ this?

Any idea when this will be published in a release?

I don't have NPM access, I'd check out who does and ask them 👍

@rafaellincoln @thymikee

Published in v1.0.3

The left out file libRNCPushNotificationIOS.a is stil empty.