modify rules to enable ask and describe

A recent sparql-specific rules update disallows some ASK queries.  Very likely some DESCRIBEs would also be disallowed.  Modify rules to specifically catch and enable ASK and DESCRIBE.
DESCRIBE is being used by some RDBMS' and might be on the queue for the next SQL standard.  ASK doesn't appear to be in the radar.

ASK completed, and various DESCRIBE taken care of, in 6ec1fd509, tag 1.0.9-3.2.0.  Will open a separate ticket for specific forms of DESCRIBE.


Created base Add Mood page

Closes #

Summary

hello kings it is me
i made a change
please review it
thank u kings

merged dependencies


Add additional features for Tobira integration

This adds most of the additional requirements in #311.
Needs https://github.com/elan-ev/tobira/pull/1225 and https://github.com/opencast/opencast/pull/6091 to work correctly.
Only the last 7 10 commits need to be reviewed here, as the rest is based on #313.
Key changes/features:
(Edit: see updated screenshots in my comment further below)

The event details modal now has a Tobira tab which provides a direct link and list any pages in Tobira that feature the event:

The Tobira tab of the series details modal got the addition of an "edit path" button that lets admins edit paths of series that are already mounted in Tobira (with the condition that the page where the series is mounted does not contain any other blocks)

Selectable pages for adding and moving (i.e. updating the path of) series now feature an additional checkbox that shows which page is currently selected (and selectable at all)

Notable omissions/deviations:

@oas777 suggested to also add editing host-page paths of events. While technically possible, we don't allow choosing a host-page for events via the admin UI in the first place, so that would need to be added first. This needs some discussing and is currently out of scope for this PR.
@dagraf suggested adding a green checkmark when creating new subpages for hosting a series. Clicking that would then confirm the creation and disable editing the title and path. I decided against that since I already added the checkbox as per David's other suggestion, which serves a similar purpose. I feel like adding yet another checkmark to that is unnecessary and may wrongly suggest that the path will already be created when that is clicked.

If these omissions are dealbreakers, we should discuss them in an upcoming meeting.
Note: This PR is based on #313 which adds the Tobira integration. That should definitely be merged prior to this one! (it's only been open for like 4 months..)
I'll mark this as a draft until #313 is merged.
Closes #311

I tried to make the Tobira tabs a little less "matryoschka"-like, as suggested in todays technical Meeting. This does however mean that these tabs won't follow the design examples set by other tabs.
They are still built like russian dolls (I didn't come up with that analogy but it's fitting).
Though I think that should be fixed in another PR.
Some updated screenshots:

Found a page crasher (...)

Hm I can't reproduce that. Is this happening every time you try that, even with a freshly started Admin UI, or are there some actions done before that might possibly provoke this?

This is an event without a series, so "Mount series" does not make sense here? Should it say "Mount event"? Also when clicking on the "Mount series" button and trying to mount, the mount post request seems to fail.

Yeah that shouldn't be there. Mounting events via Admin UI is not a thing (yet).

Hm I can't reproduce that. Is this happening every time you try that, even with a freshly started Admin UI, or are there some actions done before that might possibly provoke this?

Was happening consistently for me.

Quick screencap: Bildschirmaufzeichnung vom 2024-10-29 17-25-18.webm

I have added a commit that might fix the crashing error you reported, but I can't say for sure.

Now crashes with a different error, see below. When creating a news series, this error also occurs immediately upon switching to the tobira tab.
Uncaught TypeError: formik.values.breadcrumbs[(formik.values.breadcrumbs.length - 1)] is undefined
    NewTobiraPage NewTobiraPage.tsx:148
    React 15
    nextPage NewSeriesWizard.tsx:107
    onClick NewAccessPage.tsx:364
    React 15
NewTobiraPage.tsx:148:6

Latest and original error are fixed. With this, all my change requests are adressed and this is good to go from my side.


Update dashboard discovery to use .dll instead of executable

Executable was having issues on Mac, so switching to reference the .dll since Aspire.Hosting can already handle that scenario.
Microsoft Reviewers: Open in CodeFlow

While I'm not against this change, have we understood yet what is the problem with the mac os executable that we are generating? I'm assuming this should be relatively easy to test, right? Correct me if I'm wrong but I think this change is changing how the dashboard is launched everywhere, and that could be risky, especially since it feels more like a bandaid trying to cover the actual issue.

Updated to only special case using the .dll on Mac

/backport to release/8.0-preview3


test_tenant_detach_smoke is flaky

if fails quite often in regress debug tests in CI
with scary error message
InternalServerError(fsync of newly created layer files\n']

i.e.
https://neon-github-public-dev.s3.amazonaws.com/reports/pr-3365/debug/3939889733/index.html#categories/8fb3a91ba5aaf9de24cc8a92edc82b5d
IIUC, it is only cfg(testing) , so, can't happen in prod.
But still pretty annoying in CI.

I could reproduce this failure on 9d3992ef4, with 10 test instances running in parallel:
$ grep 'do_gc failed:' test.log
5       2024-11-15 20:37:06.14  ERROR [test_tenant_detach_5.py:35] do_gc failed: fsync of newly created layer files
4       2024-11-15 20:37:06.207 ERROR [test_tenant_detach_4.py:35] do_gc failed: fsync of newly created layer files
1       2024-11-15 20:37:06.236 ERROR [test_tenant_detach_1.py:35] do_gc failed: fsync of newly created layer files
...

The last commit this exact failure is reproduced on is 6b6570b58~1. Starting from 6b6570b58, I can't get "do_gc failed: fsync of newly created layer files" errors, but get "timeline is Stopping" under the same conditions:
$ grep 'do_gc failed:' test.log
6       2024-11-15 20:43:35.438 ERROR [test_tenant_detach_6.py:35] do_gc failed: timeline is Stopping
1       2024-11-15 20:43:37.790 ERROR [test_tenant_detach_1.py:35] do_gc failed: timeline is Stopping
2       2024-11-15 20:43:37.790 ERROR [test_tenant_detach_2.py:35] do_gc failed: timeline is Stopping
...

The later error was considered allowed with 0c7276ae1.
So I believe this issue should be closed as resolved with https://github.com/neondatabase/neon/pull/3378.


inflate more SIMD

needs refinement, and just hardcodes x86 + AVX2.
The general gist here is that we get fewer instructions, but the same number of cycles as a safer version without this much SIMD. But that can be very instruction-set dependent. So we should have more instruction sets before we can really make this decision.

I tried to improve the offset_from_end == 1 case, but it is only faster in the micro benchmark, and actually slower in the inflate of silesia-small.tar.gz


why does it bring white to the edge of the picture?

Could you see if this still happens on the version most recently pushed?

Could you see if this still happens on the version most recently pushed?

Yes it done.

Odd, I can't replicate the behavior. Could you give me a few more details on the environment and possibly how you're calling the cropper or whether you're using it in one of the streamlit container objects?
Here's my environment:
Windows 10
Python 3.11.1
Streamlit 1.16.0
streamlit-cropper 0.2.1
Chrome, Firefox and Edge browsers tested.

Sure
Here's my environment:
Ubuntu (AWS EC2 Server)
Python 3.8.15
streamlit 1.16.0
streamlit-cropper  0.2.0  and 0.2.1 (both tested)
Chrome tested
This issue persisted in 2 different conda environments.
I believe image size has something to do with it. Most of the images I used only have this white area in "Wide Mode".
The following image has it on both settings.
WIDE MODE:

WITHOUT WIDE MODE:

So from what I can tell this happens because Streamlit dynamically stretches the container that the image is in, it's especially noticeable if your computer/browser is in dark mode, because the canvas doesn't know to fill with black. I'm trying to figure out a way to force the component to not resize, but it may take some time.

Yes that is probably what's happening. I don't think theres any need to spend ample time on this. It is seamless in light mode. Thanks a lot for this library !


Provide driver binaries

Would it be possible to provide compiled drivers, for developers who would like
to experiment with client-side code only (and who do not have Visual Studio +
WDK but another compiler?)

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 26 Aug 2014 at 1:01

I would appreciate this. Especially since it seems there might be a regression with the latest version of Windows 10 (1709) in which the driver does not register correctly (eg. with the driver included with http://www.sunnysidesoft.com/virtualtablet/). I would like to track down what causes this http://forum.sunnysidesoft.com/t/driver-not-recognized-properly-on-winpro10-1709-regression/211, as the developer of this application hasn't been responsive since December when this issue got filed.

I would also like this.  I may not get to it soon, but I planned on using the instructions here: https://docs.microsoft.com/en-us/windows-hardware/drivers/develop/signing-a-driver-for-public-release
I have a personal Let's Encrypt key that I can use as a test subject.  If this issue is not resolved by the time I get around to it, then I will update with my results.

Hey @Funmungus. Would you happen to have a pre-compiled binary for this that we can use on our projects?

@rakshith-ravi does we need public CA  for driver singing?
Let's Encrypt only provide 3 month length duration


App dosen't load QR Code screen

Hi,
I've checked your source code it doesn't work, first time only it showing the QR code screen then always it showing the blank screen. After debugging the code I found that second time its loading https://www.whatsapp.com instead of https://web.whatsapp.com. Please fix it as soon as possible.
Thanks

Hi, make sure you have version 1.0.1 installed. The code here is not up to date yet.

Could you please update the source or tell the way to fix the issue
Thanks.

Hi, i can't find whatsapp web to go in the play store anymore :-(
Please give me the download link

Hi @LilyIkhsan
The app got pulled from play store, you can find releases over at the release tab
@ArunVicky001
Sources have been updated

Olá Leonard, você avisará  quando o app estiver funcionando?  Obrigada!  Att, Lisiane


Error while mgear menu generation on startup

Getting this error while startup
Error: TypeError: file C:\fractal\packages\mgear\4.1.0\release\scripts\mgear\rigbits\weightNode_io.py line 135: '>' not supported between instances of 'NoneType' and 'str'

I've fixed this issue on my system by replacing this line...
plugin_utils.load_plugin_with_path(plugin_list, "weightDriver/plug-ins")

with this....
plugin_utils.load_plugin_with_path(plugin_list, "plug-ins\\weightDriver.mll")

This was tested with Maya 2022 and Maya 2024

@araifractal Thanks for the feedback. I will check this ASAP

@araifractal  I think this was fixed here https://github.com/mgear-dev/mgear4/pull/267
Can you confirm on your side?
For the moment I will close this ticket. But please open it again if the error happens again. Thanks


Ignore autogenerated code

This excludes the code that has been auto generated for seccomp and dbus from code coverage.

Codecov Report

Merging #592 (9e28c3f) into main (de6385f) will increase coverage by 9.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #592      +/-   ##
==========================================
+ Coverage   61.18%   70.20%   +9.01%
==========================================
  Files          85       83       -2
  Lines       12553    10910    -1643
==========================================
- Hits         7681     7659      -22
+ Misses       4872     3251    -1621

@tsturzl Does not look that way. Direct vs Yaml

@Furisto Good catch, could you possibly add the integration tests to this as well?

@tsturzl Done


Move export buttons to the top of the page for easier access

Move the export buttons so that they display above the chart instead of below the chart.  That will allow people to access them without scrolling through the long, expanded chart.
Test Notes

Not currently implemented, so no need to test. Handled in #56

maybe above the ID column

Difficult to do because Webcharts looks for the export buttons within the table container.

they'd fit perfectly in the Summarize by: quadrant.

I think we talked to Heather and she mentioned that this wasn't a user request and we shouldn't worry about it? It's a lil fuzzy in my mind


Org users accessing other org users public tables

Users in the same organization should be able to query other users' public tables, right now it's not possible because users are not granted with the same permission publicuser has.
As an initial fix for this the best way is to grant publicuser to :username;.

Ref https://github.com/CartoDB/cartodb-postgresql/issues/98


Add ability to trigger modules directly from the API

What

Expose a trigger endpoint to trigger modules directly from the API POST /trigger/module
Change URL from POST /event to POST /trigger/event

Why
If you want to trigger a module, you now had to push a status that fitted an event that contains the module you want to trigger. This PR exposes an endpoint that allows you to directly trigger a module instead.

Oh, only the server exposes the API. In order to send triggers to the module-clients too, we need to use events. It will not be possible to check up-front if the module is set or not (as connected module clients might or might not have connected several modules). So we will just trigger the trigger events and let the listening module clients handle that.


Device state not changing to away.

Sorry to open another issue so quickly but funnily enough I now have the opposite issue.
After more than 30 minutes a phone device is still reporting as home from this wifi integration.
My device modal is

The device in question is reported as ethernet because it is connected to a mesh wifi and not the wifi of my router. I'm curious if ethernet devices are treated differently to wifi devices?
it is currently 09:27 and the last updated time was just as the phone left the house.

It appears to correctly determine the IP is blank but not that the state is meant to change.

I believe I have resolved this by altering one line in device_tracker.py
from:
self._active = self._device['ip'] is not None

to:
self._active = self._device['ip'] is not None and self._device['ip'] is not ""

Nice catch. I'll fix

@SgtBatten please update and see if it fixed.
Thanks

@shaiu Yep seems to be working fine, tested by putting my phone in airplane mode.
Really responsive which is excellent. Mush faster than the netgear integration which my mesh network is using. So I have two router connections to test simultaneously and this one is superior it seems.
Thanks


added await condition to wait for the consistency test topics to be deleted

Added a loop for all active brokers to wait for the topic to be deleted only when delete.topic.enable is not false (there is a test that is set to false on purpose and it causes troubles because it never deletes the topic).

Fixes #114

Better late than never lgtm


[FEATURE] Sitemap Addition for website {GSSoc'23}

Description
Yes, the feature request is related to a problem. The problem is that the current website lacks a sitemap. Without a sitemap, it becomes challenging for search engines and users to navigate and discover all the pages and content on the site. This can result in lower search engine rankings, decreased visibility, and a poor user experience.
Screenshots

In the attached screenshot, I have highlighted the website name with the expected presence of a sitemap.xml file. However, upon inspection, it is evident that the sitemap is not being displayed. This screenshot underscores the problem of the current website lacking a visible sitemap, further emphasizing the need for its implementation.

@NikhilRaikwar go ahead

@Arnabdaz Hey I'm done my work. Pls review the PR.


breaking output limited to one line

I've seen https://github.com/oasdiff/oasdiff-action/issues/35 but still, what is the purpose of that head?
Breaking output is different than diff or changelog. I would like to use that output in the PR comment. Could this be changed?

The idea is to include a summary in the PR and provide links to the report, as well as to the diff and changelog. The report can contain hundreds of lines. We initiated a discussion about this here. Please feel free to add your feedback.

Thanks for the input.
What's wrong with output having hundreds of lines? If I made that many breaking changes I would like to inspect those and text format in one place might be more convieniet at times (than annotations). And I've seen you already scoping it for that 1mb gh limit in all cases.
Currently diff and changelog have no limits but breaking has it out of the blue, for the same GITHUB_OUTPUT.
My use case is to present the breaking change inspection in the spoiler format in the GitHub comment. Annotations are nice but my use case is to present text format in a comment. This is similar to Spacelift integration commenting its terraform plans on PR.
I understand you want to revamp and enchance how action would work in general though this is the problem I'm facing on the current action version.

@ssukienn could you please take a look at this PR and let me know if it resolves your issue?


Integration with Rails asset pipeline as sprocket preprocessor

Check out this article :)
http://psionides.eu/2012/05/06/extending-asset-pipeline-with-custom-preprocessors/
class RootUrlPreprocessor < Sprockets::Processor
  def evaluate(context, locals)
    data.gsub(%r"\bHOSTNAME\b", "myserver.com")
  end
end

The context is an instance of Sprockets::Context; you can use it to e.g. read the current file’s path or content type, generate paths to asset files or access asset pipeline configuration. You also have access to several asset helpers from ActionView, like javascript_include_tag, image_tag etc.
http://rubydoc.info/gems/sprockets/2.0.3/Sprockets/Context
I think you would need fx content_type and somehow access the full path of the asset and run it through image_optim.
# in config/initializers/sprockets.rb
Rails.application.assets.register_preprocessor('application/javascript',
                                               RootUrlPreprocessor)

Just found an example here: https://github.com/botandrose/sprockets-image_compressor
Maybe you could join forces ;)
Sweet and simple! Yeah :)

Hey @toy
I'm attempting to get image_optim_pack pushed to Heroku. I tried using @mooktakim's build pack but fails while precompiling assets. Do you know of any other methods of getting your bin pack to build on Heroku?

@ghostropic
Do I understand right that image_optim_pack itself also doesn't work? Can you share errors?
One way to get binaries is to build them using Makefile from that gem.
Maybe it is also better to move this to a separate issue for that project.

@toy Sorry for the confusion. Image_optim_pack does work locally, but isn't useful to me unless I can get it working on Heroku, so I thought i'd ask if you had any leads on a build pack.  I suppose my only option though is to learn to write my own.

@ghostropic Ah no, I understood you, but the questions were confusing. As I understand heroku gives you ubuntu servers and image_optim_pack should work on ubuntu. So does it not work on heroku and which errors do you get? And one way to get binaries is to use make on machine you are targeting, so on heroku instance.

@ghostropic As there was already a heroku app to check versions, I've added there a very basic upload with optimisation for checking if everything works, and it seems it does: https://iopack.herokuapp.com/


tests: add integration tests

This pull request includes updates to the Cargo.toml dependencies, modifications to the licenses_report.md to reflect new versions and additions, and the addition of new test cases for the gh_bofh_lib library.
Dependency updates and additions:

Cargo.toml: Added assert_cmd as a development dependency.

License report updates:

licenses_report.md: Updated the counts and versions for Apache License 2.0 and MIT License entries. [1] [2] [3] [4] [5] [6] [7] [8] [9]

Code changes and new tests:

src/gh_bofh_lib/lib.rs: Changed excuses module import to pub use.
tests/test_common_excuses.rs: Added new test for common excuses.
tests/test_modern_excuses.rs: Added new test for modern excuses.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#210
#209
#208  👈 (View in Graphite)
#207
#206
#205
#204
next

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 3, 12:04 AM EST: A user started a stack merge that includes this pull request via Graphite.


Configurable p2p polling interval

Introduce new server config item called syncInterval - this will be the rate at which partyinfo rest calls are made.
Default value is 5000 milliseconds.
The timeout of individual requests also will be configured to be 3/4 of the interval if this syncInterval value is set by user
Details at #960

Codecov Report

Merging #969 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master    #969      +/-   ##
===========================================
+ Coverage      99.3%   99.3%   +<.01%
- Complexity     2155    2158       +3
===========================================
  Files           327     327
  Lines          5897    5904       +7
  Branches        355     357       +2
===========================================
+ Hits           5856    5863       +7
  Misses            5       5
  Partials         36      36

Impacted Files
Coverage Δ
Complexity Δ

...n/java/com/quorum/tessera/config/ServerConfig.java
100% <100%> (ø)
27 <3> (+2)
:arrow_up:

...com/quorum/tessera/jaxrs/client/ClientFactory.java
100% <100%> (ø)
9 <4> (+1)
:arrow_up:

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 8560a4f...9d982ae. Read the comment docs.


Add a "tag" command

Add a "tag" command, for adding names to locally-stored images.
Fixes #106.

 (3) [buildah] Add a way to add names to images in local storage

LGTM

LGTM, assuming the latests test runs come back happy and cheerful.

@rh-atomic-bot r+  	512b192


accessing localhost

My app uses a http service which I run at localhost:8545 of the host instance. The deployed app can't access out of my deployed app. Do some extra options have to be provided to mup.json to enable the docker container to have access to localhost:8545?

Ok I used this to discover my host ip outside of the docker container (<IP_ADDRESS>). Still it would be handy to have access to it from env/settings.


🐛 Support for duplicates event

The current cog system does not support an event to be called within multiples cogs.
File system
- cogs
   - yt_statistics.py
- bot.py

bot.py
class Bot(Client):

    ...

    @Client.event
    async def on_ready(self):
        print(
            "       _____       _ _            _____     _",
            " _____|     |___ _| |_|___ ___   | __  |___| |_",
            "|     |   --| . | . | |   | . |  | __ -| . |  _|",
            "|_|_|_|_____|___|___|_|_|_|_  |  |_____|___|_|",
            "                          |___|" "",
            sep="\n",
        )

YtStatistics
class YtStatistics(commands.Cog):

    def __init__(self, client: Bot):
        self.client = client

    @Client.event
    async def on_ready(self):
        task = TaskScheduler(self.client)

        @task.loop(seconds=2)
        async def update_channels():
            print("Updating channels...")

        update_channels.start()

setup = YtStatistics

Raises
pincer.exceptions.InvalidEventName: The event on_ready has already been registered or is not a valid event name.
This may be improved by forwarding every event of the same name one after an other?

~I'll start working on this Monday.


Duplicate census tract IDs

The nyc_acs5-2018_census.csv file has duplicate census tracts for different counties. E.g census tract 200.
nyc_tax_lien_sale_list_geocoded_benchmark4vintage410.csv on the google drive also reflects this.
When merging data it can't be done by census tract alone it has to be done by census and county or by GEOID.

@lhdatalab - Is there sufficient data to do all the joins required? i.e. does each dataset that identifies by the shorter tract ID  (e.g. 200 instead of 360541200) have the data to create the full long ID?

Made a file nyc_tax_lien_sale_list_processed_acs.csv.gz file from nyc_tax_lien_sale_list_geocoded_benchmark4vintage410.csv including a column for GEOID which are unique. Also merged it with nyc_processed_2017_to_2019_20210225.csv for predictive modeling use. Added it to _volunteer_created_datasets.

closed by: https://github.com/datakind/Mar21-housing-insecurity/pull/29
documentation updated on:


GOROOT issue using fish shell

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.8.3 linux/amd64
What operating system and processor architecture are you using (go env)?
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"

What did you do?
I'm using fish shell and the GOROOT is seted in my config.fish file
➜  kubernetes git:(master) cat $HOME/.config/fish/config.fish
set TERM xterm-256color

set GOPATH $HOME/go
set PATH $GOPATH $PATH
set GOROOT $HOME/.programs/go

set PATH $GOROOT/bin $PATH
set PATH $GOPATH $PATH
set PATH $GOROOT $PATH

# kubernetes development environment
set working_dir $GOPATH/src/k8s.io
set user walteraa

I have verified if GOROOT is defined correctly, as you can see below
➜  kubernetes git:(master) echo $GOROOT
/home/walteralves/.programs/go

I have a similar configuration in my .bashrc, as you can see below
export GOPATH=$HOME/go
export GOROOT=$HOME/.programs/go
export working_dir=$GOPATH/src/k8s.io
export user=walteraa

What did you expect to see?
When I am in bash shell, and a run go env, everything looks good
walteralves@carvalheira-pc-198:~/go/src/k8s.io/kubernetes$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/walteralves/go"
GORACE=""
GOROOT="/home/walteralves/.programs/go"
GOTOOLDIR="/home/walteralves/.programs/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build463961936=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

What did you see instead?
But, when I am in the fish shell, my GOROOT isn't updated correctly, but the GOPATH is
➜  kubernetes git:(master) go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/walteralves/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build753890953=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

There some way to fix it in my fish environment?

Notice: I did a workaround to make my environment work(I have created a symbolic link to my GOROOT in the /usr/local folder.

If go env reports the incorrect GOROOT value then fish is not exporting the
correct value of GOROOT. It's probably not working for GOPATH, but the
default value for GOPATH is the same as the one you're setting, so you
can't see that it didn't work.
On Fri, Aug 4, 2017 at 6:39 AM, Walter Alves<EMAIL_ADDRESS>wrote:

Notice: I did a workaround to make my environment work(I have created a
symbolic link to my GOROOT in the /usr/local folder.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/21298#issuecomment-320083665, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAcA1Iqqs5__ith3hpXI7xD5u7JRUg4ks5sUjASgaJpZM4Os9UK
.

You are not using set correctly: https://fishshell.com/docs/current/commands.html#set
Use set -gx GOROOT $HOME/.programs/go instead.

Closing based on @zegl 's comment.

Thank you @zegl


Adding "scroll-target" option

Adding a simple string attribute for targeting the scrolling element. Defaults to $document.

Doesn't this introduce a jQuery dependency?

jQuery (or a "jQuery light" version, at least) is already part of Angular. There are no special requirements to use it in this fashion.

jqLite doesn't support general CSS selectors like jQuery does and you are depending on the window global $ (thinking of this line) which is not available without jQuery. In angular all dependencies should be injected and not depend on globals.
It's a good functionality you're adding, thanks :+1: – but I'm not comfortable adding a jQuery dependency. If you could refactor I'd be happy to merge.

Also please reverse bumping version numbers/js builds and squash commits, then I'm happy :smile:

As always for Angular, $ is available (in it's limited fashion, granted) as a global. If the developer wants to extend the available functionality (to get complex CSS selectors, for example), they can choose to include a full version of jQuery. But there is no forced dependency on it.
I appreciate your efforts on this, but I don't have the inclination nor time to modify this library further as these changes suit the needs of my client. Best of luck and thanks again for your work.


[Feature request] Dark Theme Switcher

Option to switch website into light and dark theme

I would like to work on it.

Sure @Anamika810

any update? @Anamika810

In the process.It must be done by tomorrow, itself. @lakshyatyagi24

May I work on this? @lakshyatyagi24

As there is no update from @Anamika810 i am assigning this to you

any update @zorione

Got busy with the exams. Will do it soon.

ok

Just a small query. Do u only need the switch to toggle or want the website in dark mode even? @lakshyatyagi24

What was the problem Sir?

Hey, @zorione toggle button was not working, and also it has disturbed the UI of the page, the whole page was shifted downwards. If you make changes make sure to put ss for the same first.

Oh okay. I will make the necessary changes!

Hey @akanksha-raghav can I try this .?

yes sure @chinmaym07

is the issue still opened for the feature asked by @lakshyatyagi24 ?

Yes @harikanani

is the issue still open ? @akanksha-raghav @lakshyatyagi24 ?

is the issue still open ? @akanksha-raghav @lakshyatyagi24 ?

yes @Palaksharma23

hello @akanksha-raghav I would like to work on this

is the issue still open ? @akanksha-raghav @lakshyatyagi24 ?

yes @Palaksharma23

hello @akanksha-raghav I would like to work on this

Hey, I am Abhishek Khairnar . How can I contribute to this open source project for my open source class? Please assign me a task.
Contact Me<EMAIL_ADDRESS>GitHub: Abhishekcuso


.NET Core is now available in Nitrous quickstarts! This allows more users to try out C# on Linux, albeit without highlighting.

https://github.com/rogersachan/DotNet-Core

Linux users on .NET Core can now demo and use console applications and websites via Nitrous.io . This allows direct demos of .NET Core all set up and gives them dozens of hours of use without having an install on their primary machine.

lack of interest = closed issue


Handle non-hash input/output values

We currently assume that input/output have to be a JSON/hash payload, however while looking at some map examples it seems like it is possible for simple strings as well (and I assume by extension integers, etc...).
Example:
https://docs.aws.amazon.com/step-functions/latest/dg/tutorial-use-inline-map.html
Input to Map state:
{
  "foo": "bar",
  "colors": [
    "red",
    "green",
    "blue",
    "yellow",
    "white"
  ]
}

So with "ItemsPath": "$.colors" the input to the "ItemProcessor" would be e.g. "red" and the output would be a simple UUID string.

Not sure In understand - it seems like the input to Map is an array always.  Or are you saying the contents of the Array itself can be anything (in this case an Array of Strings as opposed to an Array of Inputs/Hashes)?

Or are you saying the contents of the Array itself can be anything

^ this
In the step-functions examples the input to Map is ["red", "green", et...] which means the input to the sub-workflow is just "red"

Also I think there isn't anything to prevent you from passing e.g. "red" as the top-level workflow input (if you don't have a Map state that is)

Can we solve this by removing the JSON parsing?
Instead we can put that parsing into manageiq-workflow-providers, exe/floe, and possibly in Task.
I've noticed that a lambda (task) can return a string (not json) and it will work. The basic guide stated some advantages of this. Not sure if we want to support this behavior. Think we throw away the input if it is not json.

Can we solve this by removing the JSON parsing?

Oh that's interesting, yeah that might be the way to go

I really like this PR

Just wanted to add some clarifications here (as discussed with @agrare in PM).  Strings, integers, etc are JSON.  I've updated the title to say non-hash (and Adam updated his commit before it was merged)
[1] pry(main)> require "json"
=> true
[2] pry(main)> "test".to_json
=> "\"test\""
[3] pry(main)> JSON.parse(_)
=> "test"
[4] pry(main)> 1.to_json
=> "1"
[5] pry(main)> JSON.parse(_)
=> 1


react tooltip did not match className err

Hello, I'm working on [react-tooltip]. I'm using it with next.js because it with next.js. The behavior works fine, but [id not match className] error appears. Is there any way?

Hi @arsendev, can you try the new release and let us know if it continue.


Corrected Selma, Georgia Example

Original said "Five me driving directions" but it should be "Find me driving directions".

@wjohnson : Thanks for your contribution! The author, @v-geberr, has been notified to review your proposed change.

@wjohnson  Hi Will, great catch. Thank you for taking the time to fix this. #sign-off


No need to check *_FOUND because REQUIRED already specified

When REQUIRED used in find_package there is no need to check *_FOUND
variable since processing already will stop with error.
From documentation (https://cmake.org/cmake/help/v2.8.3/cmake.html#command:find_package):
The REQUIRED option stops processing with an error message if the package cannot be found.

Is it something that you've tested? I know that some packages won't raise a fatal error if not found, especially those -- like X11 or XCB -- that also search many extra libraries..
And the extra copyright notice seems a little bit too much.

Is it something that you've tested?

Yes. Ubuntu 14.04 and CMake 2.8.3.

like X11 or XCB -- that also search many extra libraries

"X11 not found" was the error I've experienced. Generation stops immediatelly.

And the extra copyright notice seems a little bit too much.

I did surf the code, test it and hack it. So what is the problem exactly?

I did surf the code, test it and hack it. So what is the problem exactly?

It is your right to enforce a copyright on your work, but although there's no explicit rule regarding this part of contributions (there should definitely be one), I don't think we'll accept that. There are many many contributors on SFML source code, often with small patches, and we don't really want to deal with the mess of maintaining copyright information, and most of all contacting all the involved authors, whenever something changes in source code.
I don't know what the rest of the team thinkgs about it though, so just take it as my personal point of view for now :smiley:

So you remove nine, add two lines and the whole file should fall under your copyright?

the whole file should fall under your copyright?

No. I just add my name (feel free to add yours). That's what I usually do and it's the only things left after I spend some of my time for doing investigate and improves.
Feel free to close if you don't see it's appropriate. I see no problems here.

The problem is not you leaving your name. We do want authors to be rewarded for their work; they appear in the commit history and we could even edit a public list of contributors upon each release.
The point is, it's not just about your name. If you're part of the copyright, it means that we can't modify this whole file (because nobody will remember which exact lines you edited) without your permission. This is pretty much unmaintainable.
Not so long ago, we've asked the libsndfile author if he could change its license (or at least make an exception) to allow static linking on iOS. The answer was "I don't care, but there are too many copyright holders and I can't track them all to get their approval". The consequence is that we've got rid of that library and rewrote all its features ourselves.  If you had written a brand new part of SFML we could leave your name in the copyright, but not here.

 If you had written a brand new part of SFML we could leave your name in the copyright, but not here.

This is certainly how I feel about this scenario. For such a small change there is absolutely no reason to write a copyright notice, especially when you only added a small piece of functionality (arguably a bugfix) to existing code. This is why the git history is so important, as it tracks exactly who made what changes to a file.
@SFML/sfml maybe this should be clarified in the contributing docs that small changes will require surrender of copyright to SFML? And that only new large features/rewrites will not require a surrender (at the desecration of the SFML team).

one... two... three... deep breath...

 If you have any kind of respect to developers please never mention such stupid arguments.

If you're part of the copyright, it means that we can't modify this whole file

I'm not a lawyer but as far as I know this copyright note (or absent of that!) literally means nothing until somebody sign some papers digitally or physically. Anyway that is something that you should care about, not me.
One more time I just add my name (feel free to add yours). That's what I usually do, I did it for:

Qt
Boost
spdlog
or even CMake itself

I think that this talking can be continued for:

we can discuss technical details of pull request
I can update copyright notice in any form you think it will be better

Any other discussion make no sense to me. You have enough courage to start this crap, please have enough courage to close this pull request yourself.
Cheers, Ruslo

You added it so you must explain your reasons. If "it doesn't involve anything" and "you're not the one who should care about it", then don't see any reason to add this notice in first place. If you add your copyright you must have valid reasons, and I can only see two: are you trying to protect something, or is it just to have your name in SFML?
If you're trying to protect something, then it's not clear what, and as I said, it is not something we want to deal with in the long term. In any case you should explain your intention.
If it's just to show your name, then there are other ways to do so that we prefer, and that are less ambiguous.

Dear @LaurentGomila, I already explain everything. I have repeated it twice and will repeat it third time: if you don't like this pull request (for any reason) then please close it, stop wasting my time. Thank you.

No, you never explained the reason behind this copyright. Which is quite important in my opinion.
So now it's up to you: either you can't leave without your name in SFML sources and close this PR, or you just want to contribute to the project because you've found something useful and remove this copyright notice.

No, you never explained the reason behind this copyright. Which is quite important in my opinion.

Just like I said - that what I usually do, no more, no less. If you're finding it offensive or suspicious it is really your problem. If you want to protect yourself from something you should do it in another way.

I'm not a lawyer but as far as I know this copyright note (or absent of that!) literally means nothing until somebody sign some papers digitally or physically.

This is where you are wrong, @ruslo. Especially, adding "All rights reserved." IS a constraint on the code. I talked to a lawyer specialised in IP a while back and it was very clear for him that this kind of © gives you every right on the material.
And again, as @LaurentGomila and @zsbzsb said, @SFML/sfml cannot afford this kind of maintenance burden for obvious reasons that were, again, illustrated above.
@zsbzsb: we could indeed update the contribution guideline to include that, although it should be quite obvious from the current state of the source code. Beside, people don't read this document anyway (or we would be discussing those issues/PRs on the forum)...

@ruslo
At first: Thank you for your contribution.
Your claim of copyright and all rights on the file is illegal, because others are holding a valid copyright already, and you can't just void that without explicit permission from the respective copyright holders.
So in order to submit your pull request, that text has to be removed. You automatically hold copyright on your changes, and you'll be listed in the contributors list by GitHub automatically, which hopefully gives you enough credit. (We don't maintain a contributors file ourselves.)
@zsbzsb @mantognini
The contribution guidelines (or rather: contributor agreement) should indeed cover that, because copyright still remains with the original authors. One could argue that the active zlib license, which only lists Laurent Gomila as copyright holder, takes care of that. But I doubt that's the case. In order to transfer copyright, contributors must explicitly agree to that (at least in most European countries; this library holds most copyright in France I guess).
So, even when @LaurentGomila said that we are not going to make the same mistake libsndfile did, we already have.. ;-) In case we plan to change from zlib to something else, the only option we have is sub-licensing, but not replacing the license. Replacing requires to contact every single contributor.
Good thing is: A license change is not really required, is it? :-)


Add support for OPTIONS-provided username and password

Presently:
def __init__(self, server, params, username=None, password=None):
    ...
    self._username = os.environ.get('MEMCACHE_USERNAME', username)

Proposed:
def __init__(self, server, params, username=None, password=None):
    ...
    self._username = os.environ.get('MEMCACHE_USERNAME', username or params.get('username'))

And the same for password.
It seems strange to me to have to poke things into environment from settings.

Fixed back in https://github.com/django-pylibmc/django-pylibmc/commit/e1c265f5f2fcd2f13eeafff7f5ca13a6887f9c89


remove unfilteredSearchResults toggle

Who is this for?
Devs having to stop fathoming between filtered and unfiltered search
What is it doing for them?
Removes the unfiltered catalogue search toggle.

@gestchild could you pass you 👁 over this again please, I just added the /collections path back, and just need some assurance.


Make patch argument optional in both production and patch manager recipes.

At the moment you are required to supply both arguments, even if they are identical.
Change code so the second argument, patch, is optional and package is copied to patch if patch is not supplied.

https://github.com/Honestpuck/PatchBotProcessors/commit/42d1af1fe4aec229a77ca6a5825b1d366f11d868


Still getting "Library not loaded<EMAIL_ADDRESS>error after upgrading tool to 0.50700.1

Description
I upgraded BartyCrouch to the latest swift-syntax release but it's still failing with dyld[15423]: Library not loaded<EMAIL_ADDRESS>– how do we need to link it?
I followed all instructions in the release, but it doesn't work ... help would be appreciated! @ahoppen
Either something is wrong with the release, or instructions of how to depend on SwiftSyntax properly should be updated.
Steps to Reproduce
As you can see on my Homebrew PR's CI fail here, running my tool which includes the latest release, e.g. via bartycrouch update results in this error. You should be able to simply build my BartyCrouch app locally via make install, then run /path/to/bartycrouch update inside a folder with a .swift and a .strings file to reproduce (see also the test function).

rdar://100842452

The problem is that bartycrouch links against lib_InternalSwiftSyntaxParser.dylib dynamically but you aren’t vendoring that library as part of barycrouch. If you copy lib_InternalSwiftSyntaxParser.dylib next to bartycrouch, the issue disappears. If you don’t want to put the library into a different directory (e.g. lib instead of bin), you could try adding an rpath to the binary that points to the directory that contains lib_InternalSwiftSyntaxParser.dylib.
You might also find https://github.com/keith/StaticInternalSwiftSyntaxParser/ interesting, which contains a static version of that parser library, maintained by @keith.

If this continues to be an issue for you, feel free to re-open the issue.

@ahoppen Thank you very much for your help. I am now trying to use the static parser provided by @keith and already made a new release with the appropriate changes. Hope it goes through this time. If not, I might ask more questions. Here's the PR: https://github.com/Homebrew/homebrew-core/pull/112557

Update: It worked, the Homebrew PR went through without problems this time. Thank you again @ahoppen!


Add ppc64le as a supported architecture

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: mdafsanhossain
Once this PR has been reviewed and has the lgtm label, please assign andreagit97 for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @mdafsanhossain! It looks like this is your first PR to falcosecurity/syscalls-bumper 🎉

@FedeDP Should I add ppc64le in release.yml?

Nope! I mean, until you are interested in having syscalls-bumper docker image pushed on ppc64le too, but i don't think so right?
Well, we could add s390x,riscv64,ppc64le since they are supported by docker/setup-qemu-action but i can do that in a follow up pr, since it is unrelated with ppc64le support ;)

LGTM label has been added.  Git tree hash: 040aef97cd3450da306f69da290dbd896858dba1

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, mdafsanhossain
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Don't skip testing on OSX

Investigating what is wrong with the OSX image.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Unable to start TruckersMP

Trying to get TruckersMP set up for the first time, keeps throwing me this error:

Proton:** Missing or invalid openvrpaths.vrpath file! [Errno 2] No such file or directory: '/home/robert/.config/openvr/openvrpaths.vrpath'

as well as

wine: cannot find L"C:\windows\system32\winemenubuilder.exe"

I'm new to Linux so unsure if this is my doing or an issue with the script.

Since I also got this problem tho. I recommend you to look at this solution. https://github.com/lhark/truckersmp-cli/issues/34#issuecomment-504429293 This might help for you since it did for me. In your home folder look for a hidden folder called .local and go into the Steam folder. Grom there search for the folder legacycompat and copy all files over to the TruckerMP-CLI directory. I recommend to watch the solution before you ask for help.

I did have a look and that before posting, but none of the commands worked and couldn't find the folders/files.
While typing this I did notice that the 'Steam' folder under home just has one folder called 'logs' the actual steam folder under home is called '.steam'. I also noticed within truckersmp-cli I dont have a 'Euro Truck Simulator 2'  folder hence why the  "$XDG_DATA_HOME/truckersmp-cli/Euro Truck Simulator 2/prefix/pfx/drive_c/Program Files (x86)/Steam/" command wasn't working. Something weird has gone on. Going to just remove any changes and reinstall truckermp-cli.
So I'm missing the same files you were which is what's causing the same error but I wonder if Im also missing some other stuff.

Did you install Steam first before installing Truckersmp-cli? And what operating system are you using to game on?

What you can do is open a terminal and install mlocate. After installing mlocate update their database by typing sudo updatedb. And then type mlocate legacycompat. Can you try this for me?

You're likely affected by bug #32 which gets fixed by #33 or the new python port.
Change the wrong folder inside the script to the actual steam path or merge the listed changes into your local copy.

I had installed steam and played single player Euro Truck Sim 2  before trying this. I'm using Elementary OS, which I'm pretty sure is built off Ubuntu.
Was typing this while going through solutions, Lucki was spot on with the issue, I merged the listed changes and it works! Thank you both for the timely responses and the help. Also apologies didn't want to be that guy who posted an issue when there was already a thread on the issue, thought my problem was related to #34. Thanks you both once again, no way would I be able to solve that on my own.


option --superimpose seems failed

hi, thanks for your great jobs. Everything seems work well except I add the param --superimpose no matter run with tmalign and lddt-align.

wish for your help, Thank You.

Hi @imSeaton
Could you share here the full python command you tried? I cannot see it from your screen prints.
Cheers,

Thank you for your reply, My commands are:
python prepare_templates.py --target examples/H1142/casp15_predictions/unbound_chain_A.pdb \     examples/H1142/casp15_predictions/unbound_chain_B.pdb \     --template examples/H1142/H1142.pdb \     --output_dir AF_models/H1142 \     --align \     --superimpose \     --align_tool tmalign
and
python prepare_templates.py --target examples/H1142/casp15_predictions/unbound_chain_A.pdb \     examples/H1142/casp15_predictions/unbound_chain_B.pdb \     --template examples/H1142/H1142.pdb \     --output_dir AF_models/H1142 \     --align \     --superimpose \     --align_tool lddt_align
By the way, I want to know what's the difference and usage scenarios between the two commands
python prepare_templates.py --target examples/H1142/casp15_predictions/unbound_chain_A.pdb \     examples/H1142/casp15_predictions/unbound_chain_B.pdb \     --template examples/H1142/H1142.pdb \     --output_dir AF_models/H1142 \     --align
and
python prepare_templates.py --target examples/H1142/casp15_predictions/unbound_chain_A.pdb \     examples/H1142/casp15_predictions/unbound_chain_B.pdb \     --template examples/H1142/H1142.pdb \     --output_dir AF_models/H1142 \     --align \     --superimpose.
It seems they both align monomers structure to the multimer sturcture, no matter whether they are remote homologous.

Hi @imSeaton , I was able to reproduce your issue and I have changed the code in order to fix the issue. Could you do a pull to update your repository and try again?

By the way, I want to know what's the difference and usage scenarios between the two commands

The --align command only causes the pdb_hits.sto alignment files to be written out so that AlphaFold knows how to map the template atoms onto the target sequence
The --superimpose flag makes is to that the target PDB chains themselves are rototranslated onto the template, and use the those chains as template. The difference then will be that you are using a different set of mmCIF files when running AlphaFold: if you don't superimpose, then you will use the template as is. If you superimpose, then you will use the target chains as they were superimposed onto the template.
Hope this clarifies it and let me know if the issue is solved on your end

This looks like an issue with the code from lDDT_align git repository. I will take a look as soon as I can and let you know when it's fixed

@imSeaton as it turns out, the necessary lDDT_align code was on the wrong branch that was not merged into main. I have done that now, so if you pull the new lDDT_align code and reinstall it with pip install . from within your AF_unmasked environment everything should work as expected


Detection IoU

What does this PR do?
Fixes #659
Continues the work from https://github.com/Lightning-AI/metrics/pull/761 and adds IoU, GIoU, DIoU and CIoU
Things I did so far:

Implementation follows naming convention
The input and its validation is the same as for mAP so all detection metrics can be used in a MetricsCollection
Threshold is set to None by default to enable proper progress logging, especially for IoUs with negative values
Use .diag().mean() for single value calculation if all labels are equal, .mean() and all permutations otherwise
Implemented additional metrics via inheritance
Check for correct labels is supported and enabled by default
IoUs are aggregated per image first and then averaged

Things to do:

[x] There is still no checking whether the labels are correct, how to handle them in calculation?
[x] Decide what to do with multiple boxes per ground truth, usually in object detection this is handled by NMS
More test cases are required
[x] Currently IoU calculation takes all boxes and doesn't evaluate on a per image basis. Should it rather calculate on a per image basis or keep evaluation per box?
[x] is .diag().mean() the correct behaviour if the number if ground truth boxes != predicted boxes?

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃
cc: @SkafteNicki @jscottcronin @tkupek @Borda

@mibaumgartner  mind having a look as well (as my goto detection guy :D )

@Borda The difference imo is that they are dedicated metrics, the difference in logic is just hidden inside of torchvision. But I'm just a sucker for inheritance and one could argue that they are all some kind of IoU so it probably comes down to preference.

@Borda Python 3.7 is not compatible with your suggested literal type hint. Is there a deprecation plan for 3.7 yet, should I just remove it or do you prefer a conditional import?

Seems like the tests got canceled due to a timeout, but apart from that everything else imo looks fine now

Apparently the tests fails for DDP, but I have no idea why as I don't understand the error message

Apparently the tests fails for DDP, but I have no idea why as I don't understand the error message

@justusschock could you pls help here? :rabbit:


Printed text when --aggregate-output is used doesn't have colour

When --aggregate-output is used the printed text loses its colour. That's the only downside of this option.

Open to PRs addressing this. Curious to know, is there a behavioral difference from upstream here?

I just tested with the latest version on Windows native cmd and WSL and it seems to work fine with colors.


onThumbPress / onThumbRelease

Hello,
would it be possible to make these two events available: when the user presses the scroll thumb and when she releases it.
Or is it possible already?

I created a PR recently that could handle that:
https://github.com/malte-wessel/react-custom-scrollbars/pull/157


For y axis fill color direction is wrong

For the y-axis, fill color is starting from top to bottom.
But to use it as a volume controller it should be bottom to top.
Any props or attribute to change this behavior?

Currently, I am doing in this way, Kind of a reverse thing,
<Slider
  axis="y"
  ystep={0.01}
  ymin={0}
  ymax={1}
  y={1 - this.state.volume}
  onChange={({ y }) => this.setState({ volume: parseFloat(1-y) })}
  styles={{
    track: {
      backgroundColor: "#5e72e4",
      height: "100px",
    },
    active: {
      backgroundColor: "#ddd",
    },
  }}
/>;

Just added reverse support in 5.1.5 https://github.com/swiftcarrot/react-input-slider#props


Allow non-tight degree bound

Reverts the degree adjustment part of #436. As @jimpo pointed out, the adjustment complicates security by allowing rational functions of the form poly(x) / x.
In hindsight a tight degree bound shouldn't be necessary. Ultimately we want to check that some witness function f(x) exists satisfying (simplified) c(f(x)) = Z_H(x) q(x).  We only need f(x) to be low-degree because that allows us to use polynomial identity testing. With PIT we don't care about exact degree bounds; a negligible degree change will have a negligible effect on PIT soundness.

The whole issue is also discussed in my summary on FRI. The first construction (Section 4.1.1. therein) is a commitment scheme for polynomials of degree <= k-1 (where k is the rate of the Reed-Solomon code used in the proximity test) and therefore needs a degree correction factor, whereas the "optimized" scheme (Section 4.1.2) does it without that factor and hence is for degree <= k polynomials.
Anyway, I agree with  @dlubarov : We don't need a tight degree bound for polynomials in the circuit satisfiability statement, and this situation does not change when leaving the realm of unique decoding. (It is good enough for DEEP-ALI to work with the simple evaluation quotients - no degree correction factors needed. See the soundness argument in my write-up.)


Update README.md

@chrisghill is importing the new template into the CLI so customers get it to. It'll include PARAMS:START fences as well so that @jaketf's python script can keep users params, connections, and artifact docs up to date in the readme.
New template is pretty tight -> https://github.com/massdriver-cloud/aws-vpc

@coryodaniel I'm a bit confused. On Friday I thought this was supposed to be the "go forward" bundle repo template. that aws-vpc readme does look dope is there a PR for that template @chrisghill is using somewhere? I can work on getting the python script into our workflow but I wanna make sure I'm starting in the right place :)

He’s been working on putting the new template together. We rolled it to all the pre-existing bundles. They have the PARAMS:START tokens in them. He’s replacing the README here w the same template.
We’ll put the Python script in precommit (existing and this template for in the cli) so it keeps those PARAMS and CONNECTIONS sections updated

@WillBeebe seems we should we close this PR then.
What was the point of the scramble Friday tracked in this sheet to update READMEs with this template? Seems we wasted effort on opening over a dozen PRs that now near useless because they were using the wrong readme template.
Some of those PRs also updated massdriver.yamls / pre-commits though so now we should till through them and make them only scoped to those changes.f
cc @mclacore

Yeah this PR is no good anymore. I opened it as tracking for the readme edits. I thought this would be reviewed, approved, merged, then we'd copy over to the bundle repos.
Every bundle PR I opened for public launch has Readme edits so I'll toss those and just keep the massdriver yaml edits


Append nulls for missing values in Parquet.

Parquet only calls converts for which it found the values. The missing
values are not reported. The BlockBuilder must be appended with
nulls for the missing values based on fieldIndex of the currently
read value by Parquet. Fixes #3699

@saileshmittal can you please add some unit tests for your changes? @dain what's the best place to add such tests?

@dain, @nezihyigitbasi, Could you tell where I can add these unit tests?

@saileshmittal I guess #3716 looks like a good place to add such tests once it's merged.

This should be covered by TestHiveFileFormats

@nezihyigitbasi , @dain, Thanks for pointing out the tests. I have added a couple that cover this patch.

@dain @nezihyigitbasi Do you have any comments?

Hi @dain , saw some changes in parquet reader. Does that solve this issue as well?

@saileshmittal I don't know.  I have been very busy lately and haven't had time to track this.

FWIW, we're using this patch at Slack for our parquet/presto data, and would love to have it merged upstream.

Merged, thanks!


fix(MD-2944): move system image out of extenstion and into PaddedLabel

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


updating use statements from recent rust-cocoa module changes

Builds are a mess right now. As I untangled pending PRs, I noticed this still dangling.
When servo/rust-cocoa#83 is merged, a new version of glutin_cocoa will need to be published. This change updates glutin after servo/rust-cocoa#81 so that publish will be unblocked.

Thanks @ryanstew!
@tomaka, servo/rust-cocoa#83 was merged so could you publish an update to glutin_cocoa now?

Done


Regex matching broken for File Ignore and Project Ignore filters (settings)

This issue was fixed and it will be available in the next release(v3.6)


Where do I get bpe_models/wikitext-2.bpe.32000?

I'm trying to run train_lm.py to replicate Table 5 in your paper. For the simple baseline experiment python train_lm.py --gpu_id 0 --beta2 0.98 I am missing the file bpe_models/wikitext-2.bpe.32000. How do I download or generate this file? Also the default --dataset is 'wikitext103', not 'wikitext2', is this compatible with bpe_models/wikitext-2.bpe.32000?
Thanks! And cool project!

Hey @darkmatter08,
Thank you for your interest in our paper!
The .bpe file refer to the tokenizer used for generating dataset-dependent tokens, as first described in https://arxiv.org/abs/1508.07909.
You will need to build the tokenizer yourself against the dataset you use. To do so, you can run the following snippet:
import youtokentome as yttm
vocab_size = 32000
train_data_path = ".data/wikitext-103/wikitext-103/wiki.train.tokens"  # Path to the torchtext dataset
model_path = f"bpe_models/wikitext-103.bpe.{vocab_size}"  # Where to save the tokenizer

yttm.BPE.train(data=train_data_path, vocab_size=vocab_size, model=model_path)

The .bpe file necessary to run our Transformer will be saved automatically at model_path.
There is indeed a mismatch in the default configuration: you should use a tokenizer (.bpe) that matches the dataset you wish to use. (Mismatching the tokenisers might work, but will probably lead to subpar performance.)
Thank you for noticing, we will fix that :).

Thanks! For other readers, I downloaded the files from here and unzipped them into .data/wikitext-103/, in the dfa-scales-to-modern-deep-learning/paper-experiments/natural-language-processing/ directory. I then executed @slippylolo 's script, and then was able to run python train_lm.py --gpu_id 0 --beta2 0.98. I also had to create an empty folder the script expected to exist.
Separately, I also had to massage the requirements.txt file to get it to install on my system, and I had to use python3.6 to get things running (python3.5 didn't work).


Openshift Deployment Discussion

Hey Again
The default SecurityContext settings in values.yaml sets the runAsUser to 1000.
https://github.com/tremor-rs/tremor-k8s-helm/blob/main/charts/tremor/values.yaml#L136
This causes the following error in an OpenShift Deployment.
pods "tremor-install-6d9b6f6456-" is forbidden:
unable to validate against any security context constraint:
[spec.containers[0].securityContext.runAsUser:
Invalid value: 1000: must be in the ranges: [1000660000,<PHONE_NUMBER>]]

This is because each namespace/project in OpenShift has an allocation of UIDs.
1000660000,<PHONE_NUMBER> in this case.
OpenShift by default will set the correct nonroot userid by default so the option doesn't need to be supplied in that scenario.
A side effect of this is that a user may try to resolve this by allowing the pod to run as a privileged user which is not a desirable outcome.
Would you be open to adding an isOpenShift variable to the values.yaml and if it is set to true a conditional in the template will unset the runAsUser property?
Also other projects I have worked on have set the following securityContext properties
allowPrivilegeEscalation: false
runAsNonRoot: true

Can I suggest we add those to the template as well?

Hey Anton, this is a great insight / enhancement request and we'll look into it tomorrow. Thanks for going the extra mile and teaching us about OpenShift so we can make this OS compatible 👌

Hey,
I have created PR with a few improvements. The values in values.yaml file are just default and you can override them without changing the helm chart.
You can just simply comment out the line and it won't set  runAsUser option.
I have tested that:
...

       mountPath: /etc/tremor/config
              readOnly: true
          securityContext:
            allowPrivilegeEscalation: false
            runAsNonRoot: true
          resources:

...

Would that work for you?
Thank you


Eufy Motion Sensor - Binary sensor states

One thing I noticed while trying to set up an automation using the binary switch motion detector entity, checking the logs, it goes from "false" (no motion detected) to "unavailable". I don't see any records of "true" or any motion detected, even though it shows up in the Eufy app.
Originally posted by @boctok in https://github.com/fuatakgun/eufy_security/issues/15#issuecomment-885754190

can you confirm if changes are states happening in similar times? it can be an issue of mapping of eufy states into HA states.
let's assume it is False now as there is no motion, is it switching into unavailable instead of True when there is motion?
of vice versa?

I will check it now and get back to you shortly.

Maybe not. It didn't change from 'false'. I guess that would mean that it simply isn't getting any state from the sensor.
Here is the info from the sensor state in developer tools:
name: Front Motion Sensor model: T8910 serialNumber: T8910P0020150C3D hardwareVersion: 0.0.1 softwareVersion: <IP_ADDRESS> stationSerialNumber: T8010P13203100C9 batteryLow: false motionDetected: false motionSensorPIREvent:<PHONE_NUMBER>000 motionDetection: false friendly_name: Front Motion Sensor Motion Sensor device_class: motion
compared with the state from the doorbell motion sensor:
name: Doorbell model: T8210 serialNumber: T8210P00204134E3 hardwareVersion: T8210 softwareVersion: <IP_ADDRESS> stationSerialNumber: T8010P13203100C9 enabled: true state: 1 battery: 73 batteryTemperature: 38 lastChargingDays: 11 lastChargingTotalEvents: 923 lastChargingRecordedEvents: 512 lastChargingFalseEvents: 411 batteryUsageLastWeek: 17 motionDetected: false personDetected: false personName: '' ringing: false autoNightvision: true ledStatus: true motionDetection: true watermark: 2 wifiRSSI: -45 pictureUrl: https://security-app.eufylife.com/v1/s/g/ayK1Szi0s friendly_name: Doorbell Motion Sensor device_class: motion

please let your motion sensor with my eufy account so I can debug.
I also need you around to initiate motion events.
I am located in Luxembourg, +2 timezone. I have next one hour or tomorrow 20:00 to 21:00.

How do I share it?

apparently, only cameras are shareable between accounts, I will try to improve this blindly, hopefully we can make it

Would the states from the MQTT entities be of any help? I did see that those are "on" or "off", but that may just be within that integration. Those do trigger, by the way, but I know nothing about how to set up MQTT automations etc.

Please enable excessive logging following last section here: https://github.com/fuatakgun/eufy_security#troubleshooting
later, share your logs. Try to generate some motion events on your sensor and share your logs later on, we can make it work

I was able to trigger the binary sensor for motion on my 2k indoor camera using the settings below. The sensor stays “true” for 10 seconds and then takes 3 minutes to “re-arm” ignoring any motion during that window. My cameras are also enrolled in HomeKit which triggers the motion sensor with no re-arming delay, which is how I was able to verify this.

In eufy app for camera settings (gear), enable motion detection and notifications for motion on each desired camera
In eufy app for security (guard mode), select a mode that has notifications checked.

I hope there is a way to do this without notifications, because when cameras are “disarmed” there is no option to enable notifications, which differs from the HomeKit behavior wherein motion sensing is always on regardless of guard mode. Also perhaps the three-minute delay is a self-imposed window to prevent notifications overflow, or hopefully there is another way to eliminate or shorten this.

I am using like this;

notifications are enabled on main and guest account
use guest account in home assistant
disable notifications in Android permission settings on my phone

Hey just chiming in here to mention my motion sensor doesn't work either and can help with any information you require as well.
I am trying to have the motion camera turn on lights in a room when it detects motion but there seems to be no way to do this with a Eufy Motion sensor currently. I can't even get it to show up in Google Assistant (and it's officially supported), only the Homebase shows up there.

I need you to enable excessive logging for the integration and trigger some events from motion sensor and share your logs from home assistant regarding. This will be first step.

That log corresponds to a single motion sensing trigger. I have no idea why it has multiple "false" trigger events and one "true", but that is one trigger that generated all those.

I think the true values may indicate that "motionDetection" is enabled, which appears to be different from "motionDetected", which is always showing false.

Ah yes I do see there is a difference there. Unclear though why it would be indicating that is a "property change" as the property was not changed by me. Also weird how it keeps throwing motion detection "false" despite it sending a notification to the contrary.
Looking at my other logs of motion triggers it looks like it is the same pattern every time. motionDetected = false, and motionDetection = true
So for some reason it seems like the motionDetection flag is the one indicating whether it is detecting movement. Because in some cases motionDetection is showing as false. Whereas motionDetected never seems to change from false.

thanks for investigation, so individual motion sensor should use motionDetection rather than motionDetected, now it is clear, but I believe this is a mistake. actions;

create an issue on eufy-security-client to fix the event name
come up with a solution for existing situation

Yes that seems correct and it does seem to point to a naming error coming from the eufy-security-client

I don't think it is a naming error. The phone app allows for motion detection to be turned off or on. There is also a timeout period after a detection event in which it will not register another motion event. I believe that it is simply indicating if motion detection is enabled, then indicating that the camera or sensor has entered the period in which detection is disabled, and subsequently, when it was re-enabled.

@bropat , it would be nice to get your opinion on this. when motion sensor receives an motion event, motionDetection is toggling between True and False but motionDetected is always False.
This looks contradicting with camera but you know better internal dynamics.

@boctok In my case I have 0 second delay set for the motion triggering cooldown so this may explain why the log messages are all happening at the same time then.

Another interesting thing I noticed while looking through the logs is that sometimes, the URL for the snapshot of the last activity is https://security-app.eufylife.com/imageName and at other times, it is https://zhixin-security-pr.s3.us-west-2.amazonaws.com/ etc.

I don't think it is a naming error. The phone app allows for motion detection to be turned off or on. There is also a timeout period after a detection event in which it will not register another motion event. I believe that it is simply indicating if motion detection is enabled, then indicating that the camera or sensor has entered the period in which detection is disabled, and subsequently, when it was re-enabled.

I would rely on enabled/disabled attribute rather than motionDetection but @bropat would answer better.

please get latest version of codebase and retry and share your results. resolve if possible.

Looks like the "motionDetected" reported a 'true'...
2021-08-10 23:49:01 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - set_event_for_entity - T8114P02202745B6 motionDetected True - {'name': 'temp', 'model': 'T8114', 'serialNumber': 'T8114XXXXXXXXXXX', 'hardwareVersion': 'eufy2_v', 'softwareVersion': '2.7.4', 'stationSerialNumber': 'T8010XXXXXXXXXXX', 'enabled': True, 'state': 1, 'battery': 87, 'batteryTemperature': 32, 'lastChargingDays': 0, 'lastChargingTotalEvents': 2, 'lastChargingRecordedEvents': 2, 'lastChargingFalseEvents': 0, 'batteryUsageLastWeek': 0, **'motionDetected': True,** 'personDetected': False, 'personName': '', 'antitheftDetection': False, 'autoNightvision': True, 'ledStatus': True, 'motionDetection': True, 'rtspStream': True, 'watermark': 2, 'wifiRSSI': -33, 'pictureUrl': 'https://security-app.eufylife.com/image', 'motionDetectionType': 2, 'motionDetectionSensivity': 20, 'microphone': True, 'speaker': True, 'speakerVolume': 92, 'audioRecording': True, 'powerSource': 0, 'powerWorkingMode': 0, 'notificationType': 3, 'type_raw': 9, 'type': 'DEVICE_TYPE.CAMERA2', 'category': 'CAMERA'} 2021-08-10 23:49:01 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - on_message - {'type': 'event', 'event': {'source': 'device', 'event': 'motion detected', 'serialNumber': 'T8114XXXXXXXXXXX', 'state': True}}

I checked the state within developer tools, since I have this one camera sitting on my desk waiting to be mounted and sure enough, I can trigger the state between off and on by moving my hand in front of the camera.

@bropat , it would be nice to get your opinion on this. when motion sensor receives an motion event, motionDetection is toggling between True and False but motionDetected is always False.
This looks contradicting with camera but you know better internal dynamics.

@fuatakgun That is a typing error on my part. It will be fixed in the next release.
I only implemented the motion sensor by reverse engineering the code without really being able to test it.
If someone has such a device and could share it with me for a short time, I could check everything and implement missing features.

I asked this earlier but user was not able to find sharing option and i thought it is not shareable

It isn't shareable individually, like the cameras. In order to share it, I'd have to share everything and that's not a good option.

Two entries are created in the log when I trigger the motion sensor. One concerning motionDetection and the other motionDetected:
2021-08-11 08:18:36 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - set_event_for_entity - T8910P0020150C3D motionDetection False - {'name': 'Front Motion Sensor', 'model': 'T8910', 'serialNumber': 'T8910P0020150C3D', 'hardwareVersion': '0.0.1', 'softwareVersion': '<IP_ADDRESS>', 'stationSerialNumber': 'T8010P13203100C9', 'batteryLow': False, 'motionDetected': False, 'motionDetection': False, 'motionSensorPIREvent':<PHONE_NUMBER>000, 'type_raw': 10, 'type': 'DEVICE_TYPE.MOTION_SENSOR', 'category': 'MOTION_SENSOR'} 2021-08-11 08:18:36 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - on_message - {'type': 'event', 'event': {'source': 'device', 'event': 'property changed', 'serialNumber': 'T8910P0020150C3D', 'name': 'motionDetection', 'value': False, 'timestamp':<PHONE_NUMBER>714}}
2021-08-11 08:18:36 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - set_event_for_entity - T8910P0020150C3D motionDetected False - {'name': 'Front Motion Sensor', 'model': 'T8910', 'serialNumber': 'T8910P0020150C3D', 'hardwareVersion': '0.0.1', 'softwareVersion': '<IP_ADDRESS>', 'stationSerialNumber': 'T8010P13203100C9', 'batteryLow': False, 'motionDetected': False, 'motionDetection': False, 'motionSensorPIREvent':<PHONE_NUMBER>000, 'type_raw': 10, 'type': 'DEVICE_TYPE.MOTION_SENSOR', 'category': 'MOTION_SENSOR'} 2021-08-11 08:18:36 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - on_message - {'type': 'event', 'event': {'source': 'device', 'event': 'motion detected', 'serialNumber': 'T8910P0020150C3D', 'state': False}}

Hi,
this will be fixed in eufy-security-client version 1.1.0 (to be released today).

It isn't shareable individually, like the cameras. In order to share it, I'd have to share everything and that's not a good option.

If there is anything I could do to pull info from it and post it, I would be happy to.

Yeah you must release the station where the unit is registered.

@boctok In my case I have 0 second delay set for the motion triggering cooldown so this may explain why the log messages are all happening at the same time then.

Can you test latest codebase and share your findings?

As far as I know, I have the latest. I don't see any notification that there is a new one.

so, you should have an entity named Front Motion Sensor Motion Sensor, right? Do you see any activity when you trigger motion event?

2021-08-12 07:37:45 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - set_event_for_entity - T8910XXXXXXXXXXX motionDetection False - {'name': 'Front Motion Sensor', 'model': 'T8910', 'serialNumber': 'T8910XXXXXXXXXXX', 'hardwareVersion': '0.0.1', 'softwareVersion': '<IP_ADDRESS>', 'stationSerialNumber': 'T8010XXXXXXXXXXX', 'batteryLow': False, 'motionDetected': False, 'motionDetection': False, 'motionSensorPIREvent':<PHONE_NUMBER>000, 'type_raw': 10, 'type': 'DEVICE_TYPE.MOTION_SENSOR', 'category': 'MOTION_SENSOR'} 2021-08-12 07:37:45 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - on_message - {'type': 'event', 'event': {'source': 'device', 'event': 'property changed', 'serialNumber': 'T8910XXXXXXXXXXX', 'name': 'motionDetection', 'value': False, 'timestamp':<PHONE_NUMBER>024}}
2021-08-12 07:37:45 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - set_event_for_entity - T8910XXXXXXXXXXX motionDetected False - {'name': 'Front Motion Sensor', 'model': 'T8910', 'serialNumber': 'T8910XXXXXXXXXXX', 'hardwareVersion': '0.0.1', 'softwareVersion': '<IP_ADDRESS>', 'stationSerialNumber': 'T8010XXXXXXXXXXX', 'batteryLow': False, 'motionDetected': False, 'motionDetection': False, 'motionSensorPIREvent':<PHONE_NUMBER>000, 'type_raw': 10, 'type': 'DEVICE_TYPE.MOTION_SENSOR', 'category': 'MOTION_SENSOR'} 2021-08-12 07:37:45 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - on_message - {'type': 'event', 'event': {'source': 'device', 'event': 'motion detected', 'serialNumber': 'T8910XXXXXXXXXXX', 'state': False}}

but I do not see any event here which arrives as True. This is contradicting with this: https://github.com/fuatakgun/eufy_security/issues/22#issuecomment-896192283

That was from one of the cameras, which had also, up to that point, not given a result of true.

From the camera Model# T8114
2021-08-12 08:18:23 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - set_event_for_entity - T8114XXXXXXXXXXX motionDetected False - {'name': 'temp', 'model': 'T8114', 'serialNumber': 'T8114XXXXXXXXXXX', 'hardwareVersion': 'eufy2_v', 'softwareVersion': '2.7.4', 'stationSerialNumber': 'T8010XXXXXXXXXXX', 'enabled': True, 'state': 1, 'battery': 100, 'batteryTemperature': 22, 'lastChargingDays': 1, 'lastChargingTotalEvents': 6, 'lastChargingRecordedEvents': 6, 'lastChargingFalseEvents': 0, 'batteryUsageLastWeek': 0, 'motionDetected': False, 'personDetected': False, 'personName': '', 'antitheftDetection': False, 'autoNightvision': True, 'ledStatus': True, 'motionDetection': True, 'rtspStream': True, 'watermark': 2, 'wifiRSSI': -29, 'pictureUrl': 'https://security-app.eufylife.com/v1/s/g/lLhAIKurt', 'motionDetectionType': 2, 'motionDetectionSensivity': 20, 'microphone': True, 'speaker': True, 'speakerVolume': 92, 'audioRecording': True, 'powerSource': 0, 'powerWorkingMode': 0, 'notificationType': 3, 'type_raw': 9, 'type': 'DEVICE_TYPE.CAMERA2', 'category': 'CAMERA'} 2021-08-12 08:18:23 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - on_message - {'type': 'event', 'event': {'source': 'device', 'event': 'motion detected', 'serialNumber': 'T8114XXXXXXXXXXX', 'state': False}}

My apologies. I thought the issue was with motion detection in general, because until a few days ago, none of my cameras were providing any motion detection events either. Again, I apologize.

no worries, do you still have these issues regarding to non-functional motion sensors for cameras? Person & motion detection is well working for me.

They kind of worked before, but until a few days ago, I couldn't use them as triggers for automations. I will stick to the motion detector from now on.

The motion sensor will initially produce this:
2021-08-15 20:10:06 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - set_event_for_entity - T8910XXXXXXXXXXX motionDetection True - {'name': 'Front Motion Sensor', 'model': 'T8910', 'serialNumber': 'T8910XXXXXXXXXXX', 'hardwareVersion': '0.0.1', 'softwareVersion': '<IP_ADDRESS>', 'stationSerialNumber': 'T8010XXXXXXXXXXX', 'batteryLow': False, 'motionDetected': False, 'type_raw': 10, 'type': 'DEVICE_TYPE.MOTION_SENSOR', 'category': 'MOTION_SENSOR', 'motionSensorPIREvent':<PHONE_NUMBER>000, 'motionDetection': True} 2021-08-15 20:10:06 DEBUG (MainThread) [custom_components.eufy_security] eufy_security - on_message - {'type': 'event', 'event': {'source': 'device', 'event': 'property changed', 'serialNumber': 'T8910XXXXXXXXXXX', 'name': 'motionDetection', 'value': True, 'timestamp':<PHONE_NUMBER>657}}
so, it is registering something, but when I check states, I'm wasn't sure which value it was using, because the state was always false.
name: Front Motion Sensor model: T8910 serialNumber: T8910XXXXXXXXXXX hardwareVersion: 0.0.1 softwareVersion: <IP_ADDRESS> stationSerialNumber: T8010XXXXXXXXXXX batteryLow: false motionDetected: false type_raw: 10 type: DEVICE_TYPE.MOTION_SENSOR category: MOTION_SENSOR motionSensorPIREvent:<PHONE_NUMBER>000 motionDetection: false friendly_name: Front Motion Sensor Motion Sensor device_class: motion
Was it using motionDetection or motionDetected?
So, I went and pulled it down and brought it to my desk so that I could refresh the state while I was causing a motion event. It is clearly changing the motionDetection attribute, but the state is still false, so maybe it is using motionDetected as the state attribute, but that isn't the one being updated. See image:

@boctok
This was a BUG that was fixed in version 1.1.0 of the driver (eufy-security-client).
The correct attribute is in any case motionDetected.
In any case, you should use eufy-security-ws version 0.4.1.

As far as I can see, eufy-security-ws version 0.4.1 isn't available in HA yet. I'll keep an eye out for it.

That is on me, i need to test it throughly and probably include newly added
versions, please follow it closely but do not expect very soon, on vacation
:-)
On Mon, Aug 16, 2021, 18:33 boctok @.***> wrote:

As far as I can see, eufy-security-ws version 0.4.1 isn't available in HA
yet. I'll keep an eye out for it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fuatakgun/eufy_security/issues/22#issuecomment-899606089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUSN7RJUJK2UYZRSDOYS4LT5EVTXANCNFSM5BAOJDXQ
.

Enjoy yourself!

So if I wanted to use this motion sensor to do something like turning on a light in a room and keeping that light on while it detects motion say every 5 minutes, I'd have to have notifications on for this? Is there any workaround or way around having to have the devices notifications on? This works fine for cameras for me as they don't detect motion nearly as often as I will want this motion sensor detecting it.

That's the way it seems to work for mine. I tried with "React to motion" unchecked and the stater never changed. I tried it with "React to motion" checked, but "Push notification" unchecked and the state never changed. It will only change the state if "React to motion" and "Push notification" are both checked. I'm not sure if @fuatakgun can work some sort of magic and make it only update the state on HA, but I kind of doubt it, because it's a push notification. HA and your phone get the same notification.

Maybe you can have one account to hold all cameras and motion sensors to use with HA and have a separate one for your mobile app without motion sensor.
I have just built the bridge between eufy security ws and HA, magic comes from @bropat

Yesterday I bought a motion sensor. As soon as it arrives I will take a closer look at it to understand if I can do some magic. ;)

I just installed this a couple days ago, I have the 2k Battery Doorbell from Eufy and motion sensor is not showing correctly. It does not produce any updates in HA regarding motion detection.

I found out that with the wireless doorbell the motion sensor entities do work however, it depends on how you setup your Eufy device in the App. If set to all motion, only the motion sensor works and the person detector sensor doesn't. If you set it to Persons only then the normal motion sensor doesn't work. I was hoping that both would work regardless of the setting in the app you so you could have a combination of motion or motion and person simultaniously.

FYI: this thread is about physical motion sensor by Eufy, not about motion and person sensing features of cameras/doorbells.

@sjikkervisje  , thanks for your findings, can we continue over here: https://github.com/fuatakgun/eufy_security/issues/49
I believe they are directly related

That's the way it seems to work for mine. I tried with "React to motion" unchecked and the state never changed. I tried it with "React to motion" checked, but "Push notification" unchecked and the state never changed. It will only change the state if "React to motion" and "Push notification" are both checked. I'm not sure if @fuatakgun can work some sort of magic and make it only update the state on HA, but I kind of doubt it, because it's a push notification. HA and your phone get the same notification.

back to here, we know that our integration relies on push notifications for events, if we are not able to receive any push notifications, there is no way around this. @bropat , do you foresee a solution here other than adding the motion sensor into integration specific eufy account?

After I received the motion sensor, I checked how it works and unfortunately I have to inform you that the motion notification can only be integrated via the push notifications. There is no other interface that is openly accessible, not even via the P2P interface to the station.
I would not buy this sensor (have returned it).

Thanks for update, i will close the issue, @boctok , please go with a separate account if you are worried about number of push notifications.


ENH: carpet plot tweaks

This PR is a mix of fixes and enhancements to viz.plot_carpet. Most of the enhancements are based on personal preference so they can be dropped if desired.
Fixes

Legend should now show up if legend = True

The legend argument was not getting passed from carpet_plot() to _carpet()

Title should now show up if title argument is specified

Enhancements

The nskip argument should now work for NIfTI inputs

Looks like it's trivial to implement this for CIFTI as well, but I don't use them, so I didn't touch it

Tick formatting

If the xticks are frame numbers, then they will be printed as integers instead of decimals
Xticks now use the default font size

This prevents a clash with user-defined matplotlib configuration

The input data can now be a nibabel-image. A path (as a string) is still acceptable.

This is convenient because I can pre-process the data and then plot it directly without the i/o overhead

Hello @notZaki, thank you for submitting the Pull Request!

In the file niworkflows/viz/plots.py:

Line 154:73: W291 trailing whitespace

To test for issues locally, pip install flake8 and then run flake8 niworkflows.


[feat] Bypass need for PMU with itimer?

Currently we cannot use rbspy in many cases because we don't broadly have PMU access in GCP.
It looks like this is what fails:
https://github.com/javierhonduco/rbperf/blob/a6aac37715a1aabbbe84b7b4a72eb4d644214be2/src/events.rs#L49-L50
I notice that we are still able to run async-profiler for JVM profiling, because they add the option to support itimer as the clock source rather than perf:
https://github.com/async-profiler/async-profiler/blob/cdb87041561e0cf8b0597c3263eb3c09a3d49b5c/src/profiler.cpp#L882
I wonder if the same thing can be done with rbperf?
It looks like syscall profiling should still work without this and would be useful, but because of #61 I can't give that a go either

Since the profiling is triggered by a hardware event, it might be more difficult to have it be triggered by a software event.
Maybe it is possible to do something like set up itimer in userspace, and add a uprobe for when this itimer is triggered, but initiating it from another userspace process probably has pitfalls. I wonder if there is a software equivalent in perf...
For now though, it looks like there is a hard requirement on PMU access, which will make this trickier to run on some public clouds that don't provide access to it.

Hi Dale!
Thanks for bringing this up, this is definitely something that should be fixed!

Maybe it is possible to do something like set up itimer in userspace, and add a uprobe for when this itimer is triggered, but initiating it from another userspace process probably has pitfalls. I wonder if there is a software equivalent in perf...

As a last resort, we could do something like this but it shouldn't be necessary. The timing here is funny because I'm currently finishing a write-up on signal/itimer based profiling 😃.

Since the profiling is triggered by a hardware event, it might be more difficult to have it be triggered by a software event.

PERF_COUNT_SW_CPU_CLOCK should work without PMU support I think and it's emulated by the kernel similarly to how itimer works underneath. Something that stands out is the type field. That's most definitely wrong!! Let me address it and hopefully this should unblock you

Oops, didn't intend to close this issue. Feel free to close once we verify this is fixed.

Dale, I assume this is sorted out now?

sorry i'll give this a go after lunch, but in theory it should be - i thought there was a perf-based software timer just wasn't sure. If that's what you used i assume it solves it! I'll try it out on one of our clusters without PMU access to be completely sure 👍

I can confirm on the latest master I can actually collect samples now 🎉
Unfortunately I think we're seeing the same issue as in https://github.com/javierhonduco/rbperf/issues/61#issuecomment-1440434502
I see a lot of:
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=148 and received=129 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=145 and received=132 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=145 and received=132 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=145 and received=132 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=145 and received=132 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=145 and received=132 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=148 and received=129 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=148 and received=129 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=148 and received=129 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=176 and received=173 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=176 and received=170 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=184 and received=183 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=185 and received=184 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=185 and received=184 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=185 and received=184 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=224 and received=222 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=224 and received=222 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=224 and received=222 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=224 and received=222 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=234 and received=232 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=235 and received=233 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=234 and received=233 frame count
[2023-02-22T19:00:24Z ERROR rbperf::rbperf] mismatched expected=234 and received=233 frame count

I got a flame graph, but it is basically invalid because far more samples errored than were successful:
Got 10616 samples and 869805 errors

Though prior to your patch in #63, rbperf could not run at all so this is still progress, and if I understand that other issue, it is caused by hash collisions? This process is also a production rails process so it is pretty pathological in terms of stack depth etc (I've seen some callchains that are over 500 frames long!!!).
Anyways I think we consider this "Fixed", with a caveat being per my comment in https://github.com/javierhonduco/rbperf/pull/63#issuecomment-1440617379 we might want to either prefer hardware timing events if it is available, or make this an option to toggle between hardware / software clock source

mismatched expected=185 and received=184 frame count

Uh, oh! That's an interesting exciting new error. Never seen it before! We can compute the expected stack frame size ahead of time. It's what we use to know when to stop unwinding the stack and we also use this in userspace to know whether stacks are truncated. This is the check that's failing.
Will look into it.

Though prior to your patch in https://github.com/javierhonduco/rbperf/pull/63, rbperf could not run at all so this is still progress, and if I understand that other issue, it is caused by hash collisions? This process is also a production rails process so it is pretty pathological in terms of stack depth etc (I've seen some callchains that are over 500 frames long!!!).

Part of the problem is key/hash collisions, will hopefully get this improved in some days.

This process is also a production rails process so it is pretty pathological in terms of stack depth etc (I've seen some callchains that are over 500 frames long!!!).

Woah! Glad that now rbperf bumped the max frames above that! 😄

Anyways I think we consider this "Fixed", with a caveat being per my comment in https://github.com/javierhonduco/rbperf/pull/63#issuecomment-1440617379 we might want to either prefer hardware timing events if it is available, or make this an option to toggle between hardware / software clock source

Sounds good! More than happy to accept a PR for this feature 😄


Color Picker for HTML

VSCode Version: 1.16 Insider
OS Version: macOS

Steps to Reproduce:
We already support color picker in builtin CSS extension. This can be done in the same way for CSS in HTML.
@aeschli let me know if you think it's a good idea to roll in this feature to styles immediately for gathering feedback or you want to wait for the stabilization/more enhancements of Color Picker.

Sure, makes sense. I'll add the color provider to the HTML extension.


build for windows

adding a windows build to make build

@kidbrax
I'm looking into this and I'm not sure that an extra build item is completely necessary to get a Windows build to compile. The current build action will take into consideration the $GOOS & $GOARCH when doing the actual build. Compiling the config-lint package on a Windows system should already provide a way to create this binary based on the GOOS=windows & GOARCH=amd64 values. I don't think hardcoding these values is the proper way to go about building this specific executable.
The Go docs state that the $GOOS & $GOARCH variables default to the values of the $GOHOSTOS and $GOHOSTARCH https://golang.org/doc/install/source#environment So I think if you compile config-lint on a Windows system, it should create a binary that can run on Windows.

Closes #84
@kidbrax I went about this change/fix a bit differently. I think the better way to make this change is to create an actual Windows release.zip file with each future release that we create here with the project. Let me know your thoughts.


perf: incremental build chunk graph

Summary

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

!bench