hi please help figure it out thanks

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
please help i want play youtube video links via firefox with addon play with via vlc via yt dlp smtube can do it how do it outside smtube thanks for help
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

If you are talking about what I think you're talking about (my remote telepathy game is not too strong), then you might find this enlightening: https://github.com/mpv-player/mpv/issues/4727#issuecomment-788367843


Please add Magnet

"Every time you want to copy content from one app to the other, compare files side by side or multitask in any other way, you need all the windows arranged accordingly. Magnet makes this process clean and simple. In just one drag to the edge, you snap any window into left, right, top or bottom half of your screen"

Magnet on the Mac App Store || https://itunes.apple.com/us/app/magnet/id441258766?mt=12

What alternative do the downvoters recommend? I find Spectacle a good open source alternative, but Magnet still has more features, and I would list both.

I've been using BetterTouchTool for this (this actually used to be part of BetterSnapTool).
Yet another alternative.

@rubenvarela I would like to use the Chinese and English form of the switch.

All in the list. Closing.


[ OT ] Developing Chrome OS apps using Dart under CDE!

I can see that the CDE is still not at v1, but, when it does get there, will there be a document or some material which would show newbies how to create apps for Chrome OS using Dart under CDE?
I am on a Chromebook, and am itching to get started with Dart, just can't because all the tutorials and books (currently) focus on desktop oriented tools for Dart development.

This isn't an issue but a question so you'd be better off asking this in the CDE G+ Community: https://plus.google.com/u/0/communities/110934803273940338474
However, why not just add a project in CDE and choose Dart Web App or Dart Chrome App?


Fungible Assets CREATE Event

An event handler for indexing the creation of a fungible event from a chain, such as (RMRK/USDT) will be developed.

Closed in #199


Fixes function destructured parameters

Fixes: #94

Great job! Thanks.

You're welcome, and thanks for this lib.
Are you planning any release soon?

Yes. 0.4.9 just released.


Add New Filter or Where Condition in  Job Status = Open

Currently there is one Condition for Job search i.e Open Jobs.
I want to add another condition Job Status = Open,
I want to display the job list where   Job Status = Open,
Please Guide me .
Not other Status
Current behavior
Not Checking BullHorn  Job Status,

Expected behavior
Need to Check BullHorn  Job Status where Job Status = Open

Steps to replicate behavior

Screenshots

Current
Expected

Hello,
By default the portal looks at the Open/Closed field rather than the job status field as there could be possibly multiple statuses applicable when a job is open and accepting candidates.   We recently added a feature where you are able to access more than the standard set of fields to filter on or add fields to your job.  To enable additional fields you will need to modify your portal to add the fields into the search query, and you or an authorized support contact will need to contact Bullhorn Support to add the field as well.
Please let me know if you have any questions.


fix(audit_logs)!: channel_id can be None due AutoMod blocking nickname changes

Summary

Fixes #1038. There's a possibility that channel_id for being None when a nickname change was blocked by AutoMod and this has been logged in the AutoMod log channel.
Also removes some repetition in audit_logs.py
Testing code: https://paste.nextcord.dev/?id=1682809496392755
Checklist

[x] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.
[x] I have run task pyright and fixed the relevant issues.

[x] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

This is needed since it also leads to hard crashes once an audit_log_entry_create event fires without a channel id.


Include styling and allow external overriding

In this PR

Basic styling for AnnotationBlock elements
Include CSS/SCSS stuff in Webpack and ignore with Jest
Give all elements (container, body, label, buttons) CSS class names
Add classes to readme

@rlskoeser Oh right! I'd meant to, but forgot. Should be fairly simple if you wanted to; just need to add the data-placeholder attribute when initializing the label element, and then add the rule to the css.


optimize needs to be defined for multiple args

@mrocklin this could use your review when you get a chance

Looks good otherwise.

turns out Not wasn't implemented for pyfunc either

@mrocklin added unaryop and Not implementation to pyfunc
any other comments?


[Snyk] Upgrade react-router-dom from 6.3.0 to 6.8.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade react-router-dom from 6.3.0 to 6.8.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 60 versions ahead of your current version.
The recommended version was released a month ago, on 2023-02-27.

Release notes

Package name: react-router-dom

6.8.2 - 2023-02-27

6.8.2-pre.3 - 2023-02-23

6.8.2-pre.2 - 2023-02-22

6.8.2-pre.1 - 2023-02-22

6.8.2-pre.0 - 2023-02-21

6.8.1 - 2023-02-06

6.8.1-pre.0 - 2023-02-02

6.8.0 - 2023-01-26

6.8.0-pre.1 - 2023-01-24

6.7.1-pre.0 - 2023-01-24

6.7.0 - 2023-01-18

6.7.0-pre.5 - 2023-01-18

6.7.0-pre.4 - 2023-01-17

6.7.0-pre.3 - 2023-01-13

6.7.0-pre.2 - 2023-01-13

6.7.0-pre.1 - 2023-01-11

6.6.3-pre.0 - 2023-01-11

6.6.2 - 2023-01-09

6.6.2-pre.0 - 2023-01-04

6.6.1 - 2022-12-23

6.6.1-pre.1 - 2022-12-22

6.6.1-pre.0 - 2022-12-22

6.6.0 - 2022-12-21

6.6.0-pre.0 - 2022-12-16

6.5.0 - 2022-12-16

6.5.0-pre.1 - 2022-12-14

6.5.0-pre.0 - 2022-12-13

6.4.5 - 2022-12-07

6.4.5-pre.2 - 2022-12-06

6.4.5-pre.1 - 2022-12-06

6.4.5-pre.0 - 2022-12-05

6.4.4 - 2022-11-30

6.4.4-pre.2 - 2022-11-29

6.4.4-pre.1 - 2022-11-28

6.4.4-pre.0 - 2022-11-23

6.4.3 - 2022-11-01

6.4.3-pre.2 - 2022-10-28

6.4.3-pre.1 - 2022-10-25

6.4.3-pre.0 - 2022-10-21

6.4.2 - 2022-10-05

6.4.2-pre.1 - 2022-10-04

6.4.2-pre.0 - 2022-09-30

6.4.1 - 2022-09-21

6.4.1-pre.0 - 2022-09-20

6.4.0 - 2022-09-13

6.4.0-pre.15 - 2022-09-08

6.4.0-pre.14 - 2022-08-31

6.4.0-pre.13 - 2022-08-18

6.4.0-pre.12 - 2022-08-10

6.4.0-pre.11 - 2022-08-02

6.4.0-pre.10 - 2022-07-22

6.4.0-pre.9 - 2022-07-14

6.4.0-pre.8 - 2022-06-22

6.4.0-pre.7 - 2022-06-10

6.4.0-pre.6 - 2022-06-09

6.4.0-pre.5 - 2022-06-08

6.4.0-pre.4 - 2022-06-08

6.4.0-pre.3 - 2022-06-06

6.4.0-pre.2 - 2022-05-23

6.4.0-pre.0 - 2022-05-23

6.3.0 - 2022-03-31

    from react-router-dom GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report

Merging #1 (4f81047) into dev (4dfd648) will increase coverage by 13.53%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff             @@
##              dev       #1       +/-   ##
===========================================
+ Coverage   50.97%   64.50%   +13.53%
===========================================
  Files         482      416       -66
  Lines       11223    13358     +2135
  Branches     3188     1273     -1915
===========================================
+ Hits         5721     8617     +2896
+ Misses       5502     4318     -1184
- Partials        0      423      +423

Flag
Coverage Δ

unittests
64.50% <ø> (+13.53%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 898 files with indirect coverage changes


[windows] Process event data stream

Initial revision, starting to get PEDS working on windows.
What does this PR do?
On windows, implements the network consumer for the process event data stream.
Using the network consumer, gets the APM tags (service/version/env) from the process' directory if configured by web.config or datadog.json so that the tags will match the APM tags.
Motivation
Make tagging consistent between APM & USM
Additional Notes
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes
To validate that tagging is happening properly.

install iis

install agent

install a properly formatted datadog.json or app.config into  C:\windows\System32\WindowsPowerShell\v1.0.  This allows powershell commands to be tagged.

stop process agent

this is so that you can be sure you're getting all the connections.  At this point, you have 20min to complete the test, which should be no problem

Execute a web request against your IIS using powershell, such as: (start-process powershell -passthru -ArgumentList "-C iwr -usebasicparsing -disablekeepalive -outfile test.txt http://localhost")

Collect the connections as process agent would:

PS C:\windows\System32\WindowsPowerShell\v1.0> $res = ((iwr -UseBasicParsing -DisableKeepAlive http://localhost:3333/net work_tracer/connections).content | ConvertFrom-Json)

note the assignment to a variable; this is important because you're going to post-process them to validate the results.

the tags are stored separately.  Get a list of tags included in this payload:

service:powershell
env:staging
version:somever
http.iis.app_pool:DefaultAppPool
http.iis.sitename:Default Web Site
service:service1
version:1.0-prerelease
http.iis.site:1```

Somewhere should be the tags that you set in the `datadog.json` or `web.config`.  Note that this is an ordered list.

Now, get the connections.  Using this command, you can filter just requests that the remote port was 80, which will hopefully be only one:

`$res.conns | where-object {$_.laddr.port -eq "80" }`
Note the `tags` property should have indices into the list of tags you printed above, and should contain the expected string.

Note:  a subsequent PR will contain some automated testing for this feature.

/merge


Resize of persistent disks fails on recent Jammy stemcells - systemd mount namespacing issue?

We are experiencing the same symptoms as https://github.com/cloudfoundry/bosh-linux-stemcell-builder/issues/195
Namely, resize of a persistent disk is failing with message:
Remounting persistent disk as readonly: Shelling out to mount: Running command: 'mount /dev/sdc1 /var/vcap/store -o ro', stdout: '', stderr: 'mount: /var/vcap/store: /dev/sdc1 already mounted or mount point busy

This is using the following vSphere stemcell: bosh-vsphere-esxi-ubuntu-jammy-go_agent/1.329
Outside of a resize operation, we can reproduce this on a BOSH VM by SSHing to it, and, in a root shell, running the commands:
# restart either auditd or chronyd
service auditd restart

# stop all bosh services
monit stop all
sleep 10 # or wait for all services to stop

# unmount persistent disk
umount /var/vcap/store

# try to remount it
mount /dev/sdc1 /var/vcap/store -o ro
# Output (ERROR):
# mount: /var/vcap/store: /dev/sdc1 already mounted or mount point busy.

# CLEANUP:
# mount as read-write - this will succeed
mount /dev/sdc1 /var/vcap/store
monit start all

(credit to @bg-govau for finding the above steps)
A more generic way to reproduce this, that does not require stopping BOSH services or messing with the real persistent disk, is as follows:
# create a temp file for our block device
tmpfile=$(mktemp)

# needs to be a decent size to format as XFS
truncate -s 50M "${tmpfile}"

# make a loop devices that points to this file
loopdev="$(losetup --find --show "${tmpfile}")"

# format it as XFS
mkfs -t xfs "${loopdev}"

# make somewhere to mount it
mntdir="$(mktemp -d)"

# mount it as read-write
mount "${loopdev}" "${mntdir}"

# restart auditd
service auditd restart

# now unmount it
umount "${mntdir}"

# now try to re-mount it, but as read-only (rw will work)
mount "${loopdev}" "${mntdir}" -o ro
# Output (ERROR):
# mount: /tmp/tmp.PMLRvWX6wP: /dev/loop0 already mounted or mount point busy.

# CLEANUP:
service auditd restart
losetup --detach "${loopdev}"
rmdir "${mntdir}"
rm "${tmpfile}"

However try as I may, I am unable to reproduce that 2nd method on an AWS standard Ubuntu Jammy VM - and I can't tell why! But it does happen every time on our stemcells, hence raising the issue here (rather than with systemd).
Here's what we've noted so far:
auditd and chronyd have their own mount_namespace
If we run pstree -S we see that some services started by systemd have their own mnt (mount) namespace:
systemd─┬─VGAuthService
        ├─agetty
        ├─auditd(mnt)─┬─audisp-syslog
        │             └─2*[{auditd}]
        ├─chronyd(mnt,uts)───chronyd
        ├─cron
        ├─dbus-daemon
        ├─networkd-dispat
        ├─rsyslogd───10*[{rsyslogd}]
        ├─runsvdir─┬─runsv─┬─bosh-agent───11*[{bosh-agent}]
        │          │       └─svlogd
        │          └─runsv─┬─monit-actual───{monit-actual}
        │                  └─svlogd
        ├─sshd───sshd───sshd───bash───sudo───bash───pstree
        ├─systemd───(sd-pam)
        ├─systemd-journal
        ├─systemd-logind(mnt,uts)
        ├─systemd-network(mnt)
        ├─systemd-udevd(mnt,uts)
        └─vmtoolsd───3*[{vmtoolsd}]

Mount propagation does not appear to be working properly
If these services are started before the persistent disk is mounted, then there are no issues. However if they are restarted after /var/vcap/store is mounted, then they can see those, and can continue to see those after the device is unmounted.
For example:
monit stop all
# wait for all to stop

# restart auditd to demonstrate the issue
service auditd restart

# verify it is currently mounted:
ls -l /var/vcap/store
# Output:
# total 0
# drwxrwxr-x 3 vcap vcap 18 Sep 28 01:23 mongod
# drwxr-xr-x 2 root root  6 Sep 28 01:23 mongodb_exporter
# drwxr-xr-x 2 root root  6 Sep 28 01:23 node_exporter

# unmount it
umount /var/vcap/store

# try to mount as read-only
mount /dev/sdc1 /var/vcap/store -o ro
# Output:
# mount: /var/vcap/store: /dev/sdc1 already mounted or mount point busy.

# verify that it isn't mounted:
ls -l /var/vcap/store
# Output:
# total 0

# use nsenter to demonstrate that `auditd` still sees this as mounted in its own namespace:
nsenter --target $(cat /run/auditd.pid) --all ls -l /var/vcap/store
# Output:
# total 0
# drwxrwxr-x 3 vcap vcap 18 Sep 28 01:23 mongod
# drwxr-xr-x 2 root root  6 Sep 28 01:23 mongodb_exporter
# drwxr-xr-x 2 root root  6 Sep 28 01:23 node_exporter

# restart auditd again, it now forgets about the old mount
service auditd restart

# and we can now successfully mount it as read-only
mount /dev/sdc1 /var/vcap/store -o ro

And this explain why we are affected. One of our BOSH jobs has a pre-start script which modifies the auditd config, and restarts it.
The systemd documentation is really confusing
The configuration for auditd (which is standard) includes:
### Security Settings ###
MemoryDenyWriteExecute=true
LockPersonality=true
ProtectControlGroups=true
ProtectKernelModules=true
ProtectHome=true
RestrictRealtime=true

See https://www.freedesktop.org/software/systemd/man/latest/systemd.exec.html for docs on these, some of which imply that those options will disable mount propagation.
If we comment them all out, then the issue clears up. And interestingly upstream auditd has removed some of these (2 years ago!) but those are not backported into Jammy.
However this isn't the full story. If we create a VM in AWS, using their standard Jammy image (ubuntu/images/hvm-ssd/ubuntu-jammy-22.04-amd64-server-20231207), and run those same reproduction steps above (the generic version), the mount propagation does work as expected (umount in host propagates into slave auditd process).
Therefore I suspect that this should work with the standard security options, however perhaps something else in the stemcell is undoing this?
What is different between the vSphere stemcell and the AWS Jammy image?
Not much as far as I can tell. The versions of auditd and systemd are the same.
When I print out all config options systemctl show --all the only difference I noted that looked significant is that the BOSH one shows:
Tainted=cgroupsv1

And AWS did not. This felt like a solid lead, and I found:
https://github.com/cloudfoundry/bosh-linux-stemcell-builder/blob/c3e6a15dc4e0a9d2ae65b871947a53bf1801b0ac/stemcell_builder/stages/image_install_grub/apply.sh#L87-L89
I tried setting the same in my AWS VM and restarted. After doing so it also showed Tainted=cgroupsv1 however the issue still wouldn't reproduce.
I can't easily rebuild the stemcell (without this setting) to test from the other direction.
What I have noted is that the output of mountinfo differs on each system. On the vSphere BOSH system we see:
cat /proc/$(cat /run/auditd.pid)/mountinfo | grep "${loopdev}"

# Output:
# 475 474 7:0 / /tmp/tmp.3V7N9qPucV rw,relatime shared:211 - xfs /dev/loop0 rw,attr2,inode64,logbufs=8,logbsize=32k,noquota

And on AWS Jammy:
cat /proc/$(cat /run/auditd.pid)/mountinfo | grep "${loopdev}"

# Output:
# 524 194 7:8 / /tmp/tmp.jAtl1twL1v rw,relatime shared:337 master:346 - xfs /dev/loop8 rw,attr2,inode64,logbufs=8,logbsize=32k,noquota

Note that vSphere shows: shared:211 and AWS shows shared:337 master:346.
I think that means the AWS one will receive updates from master:346 whereas the vSphere one will not receive updates from anything - and I think that matches what we are seeing. However I can't pick where the systemd config differs.
And that's as far as we've got. Does anyone have an idea what is going on to make our systemd behave differently to the AWS Jammy one?

I found this thread on the internet which describes how to restore systemd to the default before their change (12 years ago).
On a hunch, I ran grep -r private /etc on the BOSH VM, and sure enough, found this file:
https://github.com/cloudfoundry/bosh-linux-stemcell-builder/commit/914b37862104cad6b49b3630be4570c90ef6be20#diff-60e3485cb060306454b1646b6184caeaf2390bd16907601ce9b4bedbb98a5591
From the commit comment (6 years ago):

Remount root drive as rprivate
Systemd will make / shared by default, this is different than what we
had on trusty and breaks some of the garden interactions with containers
and mounts.
We chose to revert back to rprivate since that avoids having to make too
many changes too several different projects.
#153888006
Signed-off-by: Chris Dutra<EMAIL_ADDRESS>
And the linked tracker says:

it appears that bosh-warden-cpi-release fails with mysterious mounting problem: https://github.com/cppforlife/bosh-warden-cpi-release/issues/14. host vm was on a xenial stemcell. container is on trusty warden stemcell.
timebox to several hours.

The linked issue in bosh-warden-cpi-release may in fact have been fixed in https://github.com/cloudfoundry/bosh-warden-cpi-release/pull/13
And sure enough, if I delete that file, restart the BOSH VM, then my test-cases above both succeed, and outputs match what see on AWS.
Can we consider reverting (for Jammy) that change from 7 years ago (which was copied from the Bionic -> Xenial -> Jammy code) away from the default that systemd uses on a standard VM?
Related issues this may resolve:
https://github.com/cloudfoundry/bosh-linux-stemcell-builder/issues/195
https://github.com/cloudfoundry/bosh-agent/issues/252 (matches many of the same keywords not sure if solved with this)


Add spatial query

this is a simple example of getting all points within a given bounding box.
sta locations --bbox upperleftx, upperlefty, lowerrightx, lowerrighty

another example is to specify a shapefile or geojson file that defines the geometry
sta locations --within myboundary.geojson

all the spatial relationships defined at https://developers.sensorup.com/docs/#query-filter should be available e.g.
sta locations --intersects myboundary.shp

added the ability to use STATE:COUNTY with within.
currently only NM is supported.
sta locations --withn NM:Socorro

same as
sta locations --within Socorro

retrieves the county boundaries from https://reference.geoconnex.us/collections/counties/items


OutOfMemoryError when using ImageView.setContentUri() with large files

When I try to create the PhotoView object using image captured from camera (large image >2MB) it throws an OutOfMemoryError exception.
The sample code I am using to test the library is:
        mPhotoView = new PhotoView(this);
        mRlImageContainer.addView(mPhotoView);
        mPhotoView.setScaleType(ImageView.ScaleType.CENTER_CROP);
        mPhotoView.setImageURI(Uri.fromFile(imageFile));

Am I doing something wrong here?

Does this occur if you do this on a normal ImageView?

Yes, it does when I put the full resolution image. The problem is that the library should do some kind of subsampling in order to use less memory. To use this library for low resolution images is not quite logical, zooming is required when you work with large files.

That is beyond the scope of the library, at least for now.

Android has a limit that it cannot display images larger than 4096*4096. if the image is bigger in one dimension it throws OutOfMemoryError. #373 should fix this.


[Desktop][Mobile] NeedHelp|GiveHelp Question1 screen load time

env: staging
Related to the #770
Could we improve load time when opening Question 1 screen.
Steps: Home screen -> NeedHelp|GiveHelp -> Question1 screen
Transitions from Question 1 ->2->3 screens are faster then transition Home -> Question 1

This isssue has been resolved.


update possible javascript/typescript regex flags from dgimsuy to dgimsuvy in in textmate grammars

fixes #206579
I changed every dgimsuy I could find to dgimsuvy to support the relatively new v regex flag (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/unicodeSets).

@microsoft-github-policy-service agree

Should make the pr to the upstream repo https://github.com/microsoft/TypeScript-TmLanguage

Ah, of course there's also already an issue for it in that repo: https://github.com/microsoft/TypeScript-TmLanguage/issues/1006
@RedCMD Any way I could have found out about this earlier, in case I want to avoid this next time? I see there is https://github.com/microsoft/vscode/blob/main/extensions/javascript/syntaxes/Readme.md but is there any general overview of all the files in the main vscode repo that are not "dynamically" linked or included through npm or git submodules but rather "manually" copied over by some script?
Whatever, I'll close this here and see if I find the motivation to do the same PR at the other repo...


Issue 416 - eagerly expose broker endpoints on startup to allow existing client to reconnect

(PR is based on @robobario's spike)
Initially we bind the broker ports for port-per-broker to map to the upstream bootstrap address. If we receive connections on these broker ports then we discard the first message from the client and request metadata. Once we have reconciled the metadata response we close the channel. Now subsequent connections should have an up to data binding.
Note: what if the upstream bootstrap address happens to be the same as a broker address, we want to make sure we don't keep doing this behaviour.
Type of change
Select the type of your PR

Bugfix
Enhancement / new feature
Refactoring
Documentation

Description
Please describe your pull request
Additional Context
Why are you making this pull request?
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Make sure all Sonar-Lint warnings are addressed or are justifiably ignored.
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

On the thundering herd competing for the reconciliation. I wonder if a reconciliation is in progress, could further connections on one of these "ports-with-no-upstream" close immediately instead of queuing up further metadata requests and reconciliations. Maybe a small delay before connection close would be a good idea too to slow down reconnects.
I guess we shouldn't go too deep into bulletproofing it, can start with getting it going :)


Querying multiple tags; how to change the width of the "unread/total" field in the feedlist

Currently I have a query for one tag "query:Journal:tags # "(Journal)"" meta
I want to add more tags to that same query but I do not know how to do it.

You can use operators "and" and "or" to combine filters. For example, if you want articles from feeds that are tagged with "Journal" or "News", you should write:
query:Example1:tags # \"Journal\" or tags # \"News\""

Similarly, if you want articles from feeds that are tagged both with "News" and "Asia", you should write:
query:Example2:tags # \"News\" and tags # \"Asia\""

You can find a few more examples in https://newsboat.org/releases/2.27/docs/newsboat.html#_filter_language
Does that answer your question?

It works, but there is a problem.
I put "query:Example:tags # "tag1" and tags # "tag2"" and as you say it only presents feeds that contain both tags.
The problem is with how it looks when opening newsboat:

So I want the query to look as "Example 2". The problem I have is that "Example 2" does not do what the first query does (I mean to retrieve feeds that have both tags, it retrieves feeds as it would have and "or" instead of an "and").
The "Example 2" query is this: "query:Example2:unread = "yes" and tags # "tag1"" and tags # "tag2""
Other little thing: Do you know how to do in config to increase the parenthesis in the right part? As you can see in the image, it does not have any parenthesis on the right side. It only happens when it is more than 100.

The problem is with how it looks when opening newsboat:

Looks like your configuration is different from the default. Can you tell me what Newsboat version you use, and show your config file? The lines in the urls file that surround the query would help too.

it retrieves feeds as it would have and "or" instead of an "and"

I couldn't reproduce this with Newsboat 2.27. Perhaps it'd become clear once I find out what Newsboat version you're running.

Do you know how to do in config to increase the parenthesis in the right part?

Yes. You have to change the width dedicated to those numbers. It's configured via feedlist-format. Apparently you already configured it to decrease the width (the default is 11 columns, you seem to have 8), so just increase it back by replacing %8u with %11u (or any other number bigger than 8).

Newsboat version (latest): 2.27-1 (from arch linux official repository).
Config file:
config.txt
And url lines:
#starred articles a shift-a mark all as read, alt-l
ctrl-e to flag, s-enter to star it, to remove, ctrl-e delete flag s
"query:Unread:unread = "yes"" ~Unread (Unread)
"query:Starred:flags # "s"" ~Starred (Starred)
"query:(Other)(tag)+Word:tags # "(tag1)"
"query:Other:unread = "yes" and tags # "strenght"" and tags # "(Journal)""
"query:Example2:unread = "yes" and tags # "tag1"" and tags # "tag2""
"query:Example:tags # "tag1" and tags # "tag2""
#macro ,,
Then I start with the urls (that work fine). That is the first part of the urls file.
And thanks for your help.

When I quit that double quotes the results offered are correct, but the looks are wrong.
So, what should I put in feedlist-format?

This is also why "Example" and "Example2" look differently. You configured feedlist-format to display feeds without tags the way "Example" is displayed, and feeds with tags as "Example2" is displayed.
I do not understand that. I had tags in both of them (in "Example" and "Example2") so I am not being able to figure out what is the trouble. The only thing I know is what already said. That when changing that double quote the results are correct, but the looks change (to worse). Do you have the same queries format?

I had tags in both of them (in "Example" and "Example2")

True, you have tags in filter expressions for these feeds, but there are no tags on feeds themselves. From your urls file:
"query:Starred:flags # \"s\"" ~Starred (Starred)
"query:Example:tags # \"tag1\" and tags # \"tag2\""

Here, the first feed has two tags ("~Starred" and "(Starred)"), and the second one has none. You should add a tag to the second feed like so:
"query:Example:tags # \"tag1\" and tags # \"tag2\"" "just a tag"

Well, maybe I expalined wrong. I said, "That is the first part of the urls file". Meaning that is only the first part. I have also lots of feeds tagged, but I did not want to publish them here. So, there are actually feeds with tags, that are included in my queries.

So, did tagging the query itself work?

Yes, i simply put a tag called query after the query. so "query:Example:tags # "tag1" and tags # "tag2"" "query"


Cannot use the provided models.

Before modeling my own rigged hand, as a Blender newbie I tried to open the models included in this plugin in order to understand how they were made and figure out the path I have to follow (since it's unfortunately impossible to re-import the working JSON file in Blender).
But none of the models here seems to work (using Blender 2.75a and Three.js r71):

/models/Game Quality Hand/Arm_Hand: not the same model as the one actually used by the plugin (longer forearm). The .dae files can't be imported (the scene remains empty, I see no error) and the geometry of the .fbx files is screwed up (especially around the thumb).
/models/Game Quality Hand/Handsolo: seems to be the right model, but the "Leapmotion_Handsolo_Rig_RIght.dae" file is completely unrelated to the left one (another model, both hands, not rigged). The .fbx files look right, but the bones are messy (then again, most notably in relation to the thumb). Couldn't use them back in Three.js (using various settings: fingers are all displayed with weird angles, the palm is upside-down, etc.).
/models/Revamped Hand: not the same model (nothing beyond the wrist). I tried to export it to Three.js using various settings, but then again it doesn't fit in the data model. The "handrevamp_hipoly_rig_left.dae" file is almost twice as large as the right one, which looks suspicious (missing data?).
/models/Test Rig: not the same model. The "Leapmotion_Basehand_Rig_Left.dae" file is almost empty (less than 5Kb). The other file (as well as both the .fbx files) look good, but then again, once exported, I always fail to use them in Three.js (wrong angles, the geometry tends to be torn apart, etc.).

What am I doing wrong? Are there some steps and settings I'm missing?
Thanks in advance for your advice.

Looks like it's been awhile since this was in active development, we might have bitrot on our hands.  @pehrlich, any insights?

Hi, thanks to have a look at this issue.
My motive was also to fix the original model as I noticed a tiny "hole" between the middle and the ring finger (here in the center of the red circle):

It would be beneficial for the community to have a clean "template" to work from. I would be pleased to contribute to it, btw.
I tried to use a hand built with MakeHuman, but the task of making it fit in the Leap Motion data model has been insurmountable so far :(

Interesting.  Well, if you can fix the issue yourself then certainly we can accept and merge a pull request.  That little hole you have selected looks like a gap in the mesh; I'm not quite sure of the best way to fix something like that.
And yes, rigging a human hand from our data can be pretty tough, it's a lot of glue logic.

That's precisely why I tried to "get my hand" on the original model :)
Fixing a model is something I can handle, but the whole rigging thing is still way beyond my knowledge right now...
I was hoping that the working JSON data could be imported back in Blender, but it seems this feature is not going to happen soon.

I'd have to defer to Peter on this one, he's the current maintainer.

Let me first direct your attention to the wiki, which may help while I attempt to repro the same process myself. https://github.com/leapmotion/leapjs-rigged-hand/wiki/Creating-Rigged-Hands

Hi Peter, I tried many times to follow the steps described in the wiki page (starting from "Maya -> Blender -> THREE.js"), but the resulting JSON file never behaves as the original one. So either I'm missing a step, or the DAE file is not the proper one.

Just a quick update - I was able to load up the left hand .dae /models/Game Quality Hand/Handsolo/Leapmotion_Handsolo_Rig_Left.dae in to Blender, and successfully export it to THREE.js and use it in the project.  Getting some errors going from Maya/FBX to Blender/DAE - something to look in to on monday.
I've also noticed that with the Bones checkbox checked in the THREE.js exporter, there are base quaternions and positions for every bone - perhaps that can be used by the JS to accept skeletons in more natural rest poses.

Hi Peter, thanks for the update.
I also managed to import and export the left hand, using the following settings when importing the DAE file:

In the wiki page you mention "Make sure that the import units box is checked", but it looks like that the two Armature Options need to be checked too.
 Tada!
And for the export:

Blender keeps forgetting my settings, even if I press "Save Settings", so I may have missed something in the course of the numerous tests and trials I made the past few days.
Now I'll try to fix the mesh and recreate the right hand from the left one (will it be easy? I have no idea...).

See https://github.com/leapmotion/leapjs-rigged-hand/pull/19

Good stuff!  I've pushed to master with a few corrections:

The Left hand fbx no longer has references to unavailable assets.
The right.dae now actually is as named raw link
The pair of hands misnamed as right hand, have been moved in to
other.

@LePhasme any further feedback?

Hi, everything is fine! Many thanks!


Add Makefile

I'd like to have a Makefile in the repo, being the source of truth for how to install the app.
This will allow me and everyone else use $ make to compile the binary and $ make install to install it.

That looks convenient, thanks for adding!


Fix the Towncrier philosophy link

What do these changes do?
These changes fix a broken link to towncrier's philosophy.
Are there changes in behavior for the user?
Link from CHANGES/README.rst to towncrier's philosophy will be fixed.
Related issue number
Similar PR:

aio-libs/frozenlist#574

Checklist

[x] I think the code is well written
[x] Documentation reflects the changes

Best regards!

Thanks!


Should we allow multiple path segments or  @key properties in a model definition?

This came up as part of #110
Currently, multiple path segments designated on a model is disallowed.  Is there a need to allow multiple path segments for a key:

For composite keys (i.e. when there are multiple keys, but no parent resource)

possible example: Compute VM Gallery
How do we express ordering in the path segments without a route string (for autoroute)?

If there are alternate paths to accessing the resource

this is a common pattern for ODATA APIs

Additionally, should we separate '@key' from path segments, and should we allow multiple `@key' properties in a model

This comes up in arrays of complex types.  The 'id' of a particular instance may be made up of multiple properties (e.g. firstName, lastName)

should we allow this
Is there any important ordering in keys (last name, first name, for example) that we want to preserve?

links to specs with this issue

Need to collect data on how many specs would benefit from this

Examples:

Device update today -> Potential idea
General scenarios


What's the input size of CNN

Hello,
I try to print the input size of each layer, take the Cnn14 model code for example:

use function librosa.load to load audio wav. [1, 32000]
spectrogram_extractor: [1, 1, 1001, 513]
logmel_extractor: [1, 1, 1001, 64]

I have three questions:

Different audio has different length, for example, some audio may be [1, 32000], others may be [1, 294198], so they have different size after spectrogram_extractor. Why can you input different size of tensor into CNN? Or have you reshape them into the same size?
How do you input a (1001, 64) size( not the same width and length) into CNN?
I test your model , the accuracy is really high. I try to extract audio features using mfcc, and train the audioset on VGGNet, but the accuracy is about 50%. So how do you improve your model‘s accuracy?

Looking forward to your reply. Thank you.

Hi! Here are the answers:

All audios are padded or truncated to 10 seconds (32000 * 10 samples) before they are input to the CNN.
In training, all input has shapes of (1001, 64). In inference, the input can be arbitrary size (L, 64), because there is a global pooling layer after the CNN.
MFCC should work, but is slightly worse than log mel spectrogram. In PANNs the training is simple, using log mel spectrogram + CNN14 should achieve a good performance. I have not looked into VGGNet yet.

It really helps. Thank you so much for your reply.


Python Visualization Fix for python 3

This is a follow-up for #439. I found current python visualization impl has a problem in python3.
I've fixed the problem and added travis test for python 3. Also, fixed setup.py which prevents the installation on python 3. Not sure python 3 is going to be supported officially, current test looks passed.

Thanks! It is great to be able to test python3 compatibility automatically as well.
I merged another PR in to fixed the print in setup first.
Can you pull on master and update the PR so it can be automatically merged? Thanks!

Sure. Rebased.


The bot does not place orders on the site

The bot does not place orders on the site
th pricelist is correct and in the console i see
2020-10-24 22:37:29 info: Getting TF2 schema...
2020-10-24 22:37:30 info: Starting bot...
2020-10-24 22:37:30 info: Setting up pricelist...
2020-10-24 22:37:30 info: Signing in to Steam...
2020-10-24 22:37:31 info: Signed in to Steam!
2020-10-24 22:37:32 info: Initializing bptf-listings...
2020-10-24 22:37:33 info: Getting Steam API key...
2020-10-24 22:37:34 info: tf2-automatic v3.5.2 is ready! 5 items in pricelist, 0 listings on www.backpack.tf (cap: 85)
2020-10-24 22:37:34 warn: Enabling autorelist! - Consider paying for backpack.tf premium instead of forcefully bumping listings: https://backpack.tf/donate
THE LOG
{"message":"You are not running the bot with PM2! If the bot crashes it won't start again, see the documentation: https://github.com/Nicklason/tf2-automatic/wiki/PM2","level":"warn","timestamp":"2020-10-24T19:37:28.061Z"}
{"message":"Connecting to PM2...","level":"debug","timestamp":"2020-10-24T19:37:29.324Z"}
{"message":"Getting TF2 schema...","level":"info","timestamp":"2020-10-24T19:37:29.348Z"}
{"message":"Starting bot...","level":"info","timestamp":"2020-10-24T19:37:30.656Z"}
{"message":"Calling onRun","level":"debug","timestamp":"2020-10-24T19:37:30.662Z"}
{"message":"Setting poll data","level":"debug","timestamp":"2020-10-24T19:37:30.666Z"}
{"message":"Setting login attempts","level":"debug","timestamp":"2020-10-24T19:37:30.666Z"}
{"message":"Setting up pricelist...","level":"info","timestamp":"2020-10-24T19:37:30.666Z"}
{"message":"Getting key price...","level":"debug","timestamp":"2020-10-24T19:37:30.669Z"}
{"message":"Got key price","level":"debug","timestamp":"2020-10-24T19:37:30.984Z"}
{"message":"Signing in to Steam...","level":"info","timestamp":"2020-10-24T19:37:30.986Z"}
{"message":"Signing in using login key","level":"debug","timestamp":"2020-10-24T19:37:30.987Z"}
{"message":"Signed in to Steam!","level":"info","timestamp":"2020-10-24T19:37:31.847Z"}
{"message":"Waiting for web session","level":"debug","timestamp":"2020-10-24T19:37:31.848Z"}
{"message":"New web session","level":"debug","timestamp":"2020-10-24T19:37:32.343Z"}
{"message":"Initializing bptf-listings...","level":"info","timestamp":"2020-10-24T19:37:32.347Z"}
{"message":"Getting inventory...","level":"debug","timestamp":"2020-10-24T19:37:32.348Z"}
{"message":"Initializing bptf-listings...","level":"debug","timestamp":"2020-10-24T19:37:32.349Z"}
{"message":"Getting Steam API key...","level":"info","timestamp":"2020-10-24T19:37:33.406Z"}
{"message":"Getting max friends...","level":"debug","timestamp":"2020-10-24T19:37:33.407Z"}
{"message":"Creating listings...","level":"debug","timestamp":"2020-10-24T19:37:33.799Z"}
{"message":"Removing all listings","level":"debug","timestamp":"2020-10-24T19:37:33.799Z"}
{"message":"Checking listings...","level":"debug","timestamp":"2020-10-24T19:37:33.800Z"}
{"message":"Count didn't change","level":"debug","timestamp":"2020-10-24T19:37:34.107Z"}
{"message":"We have no listings","level":"debug","timestamp":"2020-10-24T19:37:34.107Z"}
{"message":"Checking all","level":"debug","timestamp":"2020-10-24T19:37:34.107Z"}
{"message":"Checking listings for 5 items...","level":"debug","timestamp":"2020-10-24T19:37:34.109Z"}
{"message":"Done checking all","level":"debug","timestamp":"2020-10-24T19:37:34.110Z"}
{"message":"tf2-automatic v3.5.2 is ready! 5 items in pricelist, 0 listings on www.backpack.tf (cap: 85)","level":"info","timestamp":"2020-10-24T19:37:34.110Z"}
{"message":"Enqueueing sort job","level":"debug","timestamp":"2020-10-24T19:37:34.112Z"}
{"message":"Ensuring TF2 GC connection...","level":"debug","timestamp":"2020-10-24T19:37:34.112Z"}
{"message":"Not playing TF2","level":"debug","timestamp":"2020-10-24T19:37:34.112Z"}
{"message":"Checking friends count","level":"debug","timestamp":"2020-10-24T19:37:34.114Z"}
{"message":"Friends to remove: -223","level":"debug","timestamp":"2020-10-24T19:37:34.114Z"}
{"message":"Checking group invites","level":"debug","timestamp":"2020-10-24T19:37:34.114Z"}
{"message":"Checking account info","level":"debug","timestamp":"2020-10-24T19:37:34.115Z"}
{"message":"Connected to socket server","level":"debug","timestamp":"2020-10-24T19:37:34.404Z"}
{"message":"Enabling autorelist! - Consider paying for backpack.tf premium instead of forcefully bumping listings: https://backpack.tf/donate","level":"warn","timestamp":"2020-10-24T19:37:34.443Z"}
{"message":"Enabled autorelist","level":"debug","timestamp":"2020-10-24T19:37:34.443Z"}
{"message":"Authenticated with socket server","level":"debug","timestamp":"2020-10-24T19:37:34.992Z"}
{"message":"Sending sort request","level":"debug","timestamp":"2020-10-24T19:37:35.001Z"}
{"message":"New login key","level":"debug","timestamp":"2020-10-24T19:37:36.808Z"}
{"message":"Queue is empty","level":"debug","timestamp":"2020-10-24T19:37:45.015Z"}
{"message":"Done processing queue","level":"debug","timestamp":"2020-10-24T19:37:45.015Z"}
{"message":"Queue finished","level":"debug","timestamp":"2020-10-24T19:37:45.015Z"}
{"message":"Checking account info","level":"debug","timestamp":"2020-10-24T19:39:03.676Z"}
{"message":"0 incoming offers (0 on hold), 0 outgoing offers (0 on hold)","level":"verbose","timestamp":"2020-10-24T19:40:12.549Z"}
{"message":"Checking account info","level":"debug","timestamp":"2020-10-24T19:40:33.789Z"}
{"message":"Checking account info","level":"debug","timestamp":"2020-10-24T19:42:03.840Z"}
{"message":"0 incoming offers (0 on hold), 0 outgoing offers (0 on hold)","level":"verbose","timestamp":"2020-10-24T19:42:51.936Z"}
{"message":"Checking account info","level":"debug","timestamp":"2020-10-24T19:43:33.775Z"}
{"message":"Checking account info","level":"debug","timestamp":"2020-10-24T19:45:03.795Z"}
{"message":"0 incoming offers (0 on hold), 0 outgoing offers (0 on hold)","level":"verbose","timestamp":"2020-10-24T19:45:31.677Z"}

d


Feature: improve splash screen manager logic by using ini helper intensively

Codecov Report
Attention: 20 lines in your changes are missing coverage. Please review.

Comparison is base (ee6def5) 70.97% compared to head (eb7b4fc) 67.41%.

:exclamation: Current head eb7b4fc differs from pull request most recent head c8d8aea. Consider uploading reports for the commit c8d8aea to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #384      +/-   ##
==========================================
- Coverage   70.97%   67.41%   -3.56%
==========================================
  Files          43       43
  Lines        2453     2449       -4
  Branches      522      521       -1
==========================================
- Hits         1741     1651      -90
- Misses        540      617      +77
- Partials      172      181       +9

Files
Coverage Δ

...s_deployment_toolbelt/profiles/qgis_ini_handler.py
68.18% <100.00%> (+3.68%)
:arrow_up:

qgis_deployment_toolbelt/jobs/job_splash_screen.py
64.81% <66.66%> (-3.05%)
:arrow_down:

qgis_deployment_toolbelt/profiles/qdt_profile.py
73.93% <51.51%> (+2.42%)
:arrow_up:

... and 6 files with indirect coverage changes


"Rudeni", "Valea Calului" are listed in the drop-down menu

Description:
When user writes "Rudeni" and "Valea Calului" the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Rudeni"

Expected results
The location appears in the drop-down menu as "Sat Rudeni, ARGEȘ (Șuici)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.


GPIO INPUT_PULLUP is not work.

GPIO INPUT_PULLUP is the same as a INPUT.

Need to change the document as well.


impl From<core::convert::Infallible> for $error_type

All error types should implement this conversion to make error handling easier.

Please provide more context on this issue.

For each error type we should implement this:
impl From<core::convert::Infallible> for Error {
    fn from(never: core::convert::Infallible) -> Self {
        match never {}
    }
}

Probably best to define and use a macro

Probably best to define and use a macro

alright. I'm defining the macro in internals/src/errors.rs because I think error-handling macros should exist here. Is it okay?

Yep, that sounds good.

@tcharding it makes it easier to compose the code. Examples:
fn foo() -> Result<(), BitcoinError> {
    // ...
    // write_all from push_bytes
    vec.write_all(bytes)?; // works! Although something like .infallible() could be argued to be better
    // ...
}

A stronger reason
trait Foo {
    type Error;
    // ...
    // IIRC there's something like this in futures or tokio.
    fn convert_err<E2>(self) -> ConvertErr<Self, E2> where Self::Error: Into<E2>;
}

With this if something expects an object that returns specific error you can also pass any object that returns infallible and use conversion helpers.

Hi, I've opened a PR to hopefully close this issue https://github.com/rust-bitcoin/rust-bitcoin/pull/2516.
Cheers


websocket connections will be closed once a while

the websocket connection keep getting closed after few seconds, i looking through the logs and found this line
time="2023-06-08T07:13:33Z" level=error msg="vulcand/oxy/forward/websocket: Error when copying from backend to client: websocket: close 1006 (abnormal closure): unexpected EOF"

This likely occurs because of idle timeout checks. You can disable them by setting --idle-connection=false. You can also ensure that your websocket is properly being pinged in order to not hit the timeout instead.


AxisAuto() crashes with InvalidOperationException

When setting the PlotScatter.xs' and '.ys' properties to double[0], an initially well behaved PlotScatterinstance crashes with said Exception when callingPlotScatter.AxisAuto()` method, complaining about some "sequence without elements".
I guess, the error originates in a call to Linq.IEnumerable.Min().
Setting the data fields to an empty array is an allowed operation and should not invalidate the instance. Therefore, this is definitely not the intended behaviour.
see #309

Hi @lefi7z, thanks for pointing this out! To re-state this problem, after plotting the user can assign to a scatter plot's xs and ys in such a way as to cause an exception (because validation only occurs when these data are provided in the constructor).
This topic (externally modifying xs and ys to produce a bad state) was recently discussed in #145, so I think it deserves some attention.
A good solution here may be to intercept assignments to these two properties and perform error checking to ensure they are not null, they contain elements, and they contain the same number of elements. Perhaps a better solution is to make these properties private and add methods to update them and perform validation (like SignalConst). These rules should also apply to errorbar data.
Thanks again for raising this issue @lefi7z, I'll see what I can do here...

Hi @swharden, thanks for the quick answer. I just read #415 and like the idea of a ScatterList() plotitem. I see two use cases where someone might be tempted to set the .xs property in the first place:

(fast) update data in a ConstSignal() plot (like an oscilloscope does).
append data one point at a time to a time-series plot.

For point 1. I don't know if it actually makes it any faster setting the property than simply creating a new ScatterPlot() instance. I remember that point 2. has been discussed before and that there is another solution to this. But adding elements to a List<> would be a very language-compliant way to achieve this.

The jump from ScottPlot 4.0 to 4.1 will introduce breaking changes, and I think the best way to handle plottable data mutability is to make all plottable data arrays private and have UpdateData() methods which perform validation. SignalConst already does this, albeit for advanced reasons.
Validation could be done on the existing code by intercepting the assignments to public properties, but I think requiring the user to call a method is a better opportunity to provide documentation (in the form of argument names and summary strings) and will result in more readable code in the long run.
I'll close this issue for now, but this topic will be in the forefront of my mind as I internally begin to refactor the plottable modules 👍


[ENH] Contribute A-DBSCAN

I'm planning on contributing A-DBSCAN, a method introduced in this paper (accepted, so working paper and online version will come out shortly and will update this link), as I think it'll be a nice addition to the library. I have two questions:

Whether you think it'd make a good addition or it might be better as a separate package? It's written following the sklearn API for DBSCAN where possible and, in theory, it could be used in non-spatial contexts, but I think it's best when used with spatial points.
If it makes sense, where do you think it'd work best? My sense is in esda as it's an exploratory method, but I could be wrong.

What do you think? I'd like to press with the addition so it can ideally make it to the January release of the meta package #naiveoptimist 😂

very cool!
spopt is the successor to region, where we have the other clustering algos, so it might make the most sense there? Obviously, we'll probably want to include it in geosnap as well. We're planning to make a pretty substantial push on spopt at UCR over the next couple months, so the timing could mesh well (though it could also delay getting this function into the metapackage if you definitely want it included in a hurry)

Without knowing exactly what the code does/looks like, I would agree with @knaaptime about it probably fitting best into spopt.

I did think about spopt but my sense is that it is not really a spatial optimisation; it's more of an extension to the traditional DBSCAN that provides a method for inference on the "blurriness" of boundaries. In that sense, it's a bit like a LISA for points (being grossly wrong in many senses but hopefully gets the point).
Yet another option would be pointpats?
I'm definitely cool with whatever works for the long term. If we settle on esda, I'd love to get it in before the Jan release, but if we decide it works best somewhere else, I'm happy letting it wait a bit longer so it finds the best home.

yeah i see the tension here... on the one hand, it isnt really an optimization problem. But on the other hand, conceptually it kinda makes sense to keep the clusterers together. I don't feel terribly strongly one way or the other

Indeed. It stretches between sklearn and pysal, and within the last, in between spopt and esda/pointpats. My vote would go for either of the latter two as it's not an optimisation and it's not an exhaustive "clustering" in that it identifies clusters but also noise points (in many cases those are the majority) so it's closer to a LISA type of cluster rather than a AZP type of cluster...

I would vote for esda!

I also think it belongs in esda.
Centralizing an implementation would help another package in writing using
what sounds like a similar heuristic! Happy to contrib stuff from my
implementation if that's useful
Sent from Mobile
On Mon, Nov 18, 2019, 14:47 Wei Kang<EMAIL_ADDRESS>wrote:

I would vote for esda!
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/pysal/pysal/issues/1144?email_source=notifications&email_token=AARFR47BOTHLD6U73X3RRLLQULWNFA5CNFSM4JOWFCQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEELVZWQ#issuecomment-555179226,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARFR47XIN5KRLEIDNJWUNTQULWNFANCNFSM4JOWFCQQ
.

toward that end, do we have a date scheduled for the next esda release? I'd like to get the geosilhouettes into geosnap
also, should we leave smaup in esda, or would it make more sense in tobler?

For the A-DBSCAN, I would vote for putting it in esda.

toward that end, do we have a date scheduled for the next esda release? I'd like to get the geosilhouettes into geosnap
also, should we leave smaup in esda, or would it make more sense in tobler?

I think we leave smaup in esda, unless there is a really pressing case to move it to tobler. I think it could have wide scope beyond the focus of tobler?

First of all, cool method! I would also vote for esda. I used esda and DBSCAN for my urban tree pattern analysis and would find it great if A-DBSCAN is also accessible via esda, from a users perspective.

Super! I'll prepare a PR and move discussion over there! Thanks everyone for chipping in!!!

I’m a little late, but also vote for ESDA, which worked well as good topic
to introduce DBSCAN when teaching this semester.
On Tue, Nov 19, 2019 at 6:44 AM Dani Arribas-Bel<EMAIL_ADDRESS>wrote:

Super! I'll prepare a PR and move discussion over there! Thanks everyone
for chipping in!!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pysal/pysal/issues/1144?email_source=notifications&email_token=AB553TPTHBUBEKPH2WK4BCDQUPGTDA5CNFSM4JOWFCQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEN5FLI#issuecomment-555471533,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB553TKMLKGCMK6FT6BCV3LQUPGTDANCNFSM4JOWFCQQ
.

OK, I've just opened a PR over at esda#94 with the functionality discussed. I'd probably close this issue and move the discussion over there? (feel free to reopen if needed)


json: add hand-rolled json sanitizer

Commit Message: Creates a hand-coded json string sanitizer, functionally equivalent to creating a protobuf and serializing to JSON, but is 13x faster when there are escapes, and 60x faster without escapes, which is going to be most of the time for stats.
---------------------------------------------------------------------------
Benchmark                                 Time             CPU   Iterations
---------------------------------------------------------------------------
BM_ProtoEncoderNoEscape                1089 ns         1089 ns       547657
BM_JsonSanitizerNoEscape               17.6 ns         17.6 ns     39777516
BM_StaticJsonSanitizerNoEscape         18.6 ns         18.6 ns     37789666
BM_ProtoEncoderWithEscape              1308 ns         1308 ns       533386
BM_JsonSanitizerWithEscape             96.0 ns         96.0 ns      7291029
BM_StaticJsonSanitizerWithEscape       96.7 ns         96.7 ns      7236032

Additional Description: This is intended to be adapted into the JSON renderer introduced in https://github.com/envoyproxy/envoy/pull/19693
Risk Level: low for now, this is not used by anything yet. when we integrate it into a flow it should be 'medium' risk.
Testing: Just the new unit-test and benchmark
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

Looks great. Thanks.

The benchmark result looks very good!
BTW, not sure it is a common optimization for protobuf -> json encoder, if it is, we can upstream it as well.

potentially, sure. I think if you already have a protobuf structure and want to generate JSON from it, it might perform OK. But in our case we had a C++ structure, and replicated that as a protobuf, building up a huge parallel structure, and then serialized it.
The serializing of JSON is I think mostly trivial if you have a sanitizer you can trust, so this is kind of a building block for a better way to translate C++ structures to json.

/assign-from @envoyproxy/senior-maintainers

@jmarantz how hardened is this security-wise? I imagine we might want to use it in JSON logs as well as admin, so would like to get some clear understanding of how to think about threat model when looking at this code.

RE hardening, I was thinking about adding fuzz tests. Differential fuzz tests are possible by comparing sanitized results with the protobuf json encoder. However, the protobuf encoder generates error messages and yields an empty sanitized result whenever an invalid utf8 input is given. So I'm not sure if we can do that much better than the unit tests. What do you recommend?

converting to draft temporarily while I iterate a little on a fuzzing solution.

After some iteration I decided that to do differential fuzzing against protobuf-serialization I needed to do utf8-scanning, and there was no portable non-deprecated library to do that. It's really only about 20 lines of code, and allows us to get the same results as protobuf-serialization for all valid 2-char utf8 points. This is fully tested in the unit test, and also in the differential fuzzer.
I expected this change to reduce performance for escaped input a bit, which it does. However it also reduces performance for unescaped input, but it's still 60x faster than protobufs. I may continue to poke around to understand the cause of that, but that could be addressed in a follow-up tweak. We should be able to do an iteration in 11 or 12 ns rather than 17 ns.

I found some incorrect behavior with invalid input. Iterating again.

Thanks to @wbpcode who showed me a fast utf-8 decoder without branches that I think could crash on invalid input...making me wonder about how this sanitizer would do on invalid input. It does help to go ahead and test that :)
Also I borrowed the 'branchless' concept to short-circuit the 99% case where nothing needs to be escaped, bringing it back to 128x the protobuf serializer performance in the fast case.

I think the prod code is correct, though I'm able to run the fuzzer now beyond my own corpus and it finds discrepancies, so I'll go back to draft mode till those are sorted.

I was able to get a local asan-fuzzer built and run it for a few hours, finding no problems, so I think this is ready for review again.
The fuzzer didn't previously find production problems but it did find utf-8 sequences producing what appear to be valid unicode values that protobufs reject, making differential fuzzing find discrepencies rather quickly. We now have a map of which unicode ranges cannot be protobuf-serialized, and a mechanism to re-compute that map if needed in the future.

closing this PR for now and will  open a new one that just uses nlohman as the slowSanitizer impl after the test changes are sorted out.
I think I just have to have the tests generally do unicode unpacking when testing equivalence against protobuf sanitization, or we could just punt all that and assume nlohmann is correct due to its own tests.

This PR is superseded by https://github.com/envoyproxy/envoy/pull/20637 which uses the nlohmann json library for the slow sanitization path -- it's about 2x slower than what we have here but still significantly faster, and more correct, than protobuf sanitization.


xcode 8 build error

"_CTFramesetterCreateWithAttributedString", referenced from:
"_CTFramesetterSuggestFrameSizeWithConstraints", referenced from:
clang: error: linker command failed with exit code 1 (use -v to see invocation)

i used Xcode8.3.3 for a long time, haven't met it. How to reproduce it?

@lionnner  do you add CoreText.framework on your project?


Fail ingesting query usage

Affected module
BE
Describe the bug
I’m using OMD v.0.13.2 ,
At v0.13.1  I succeeded ingesting Athena query usage with metadata ingest -c <path.yaml>  command,
but at v0.13.2 it fails with error message below with same csv and yaml.
Also there are ‘schema_name’ field and values in csv file
[2023-02-15 06:01:15] INFO     {metadata.Ingestion:workflow:116} - Service type:ServiceType.Database,query-log-usage configured
[2023-02-15 06:01:16] INFO     {metadata.Utils:logger:165} -
Error initializing USAGE: 'QueryLogUsageSource' object has no attribute 'schema_field'

To Reproduce
metadata ingest -c <path.yaml>
Expected behavior
ingest successfully
Version:

OpenMetadata version: 0.13.2

Additional context
I asked about this issue at slack
https://openmetadata.slack.com/archives/C02B6955S4S/p1676441146216039

I also have this error @ulixius9
Error initializing USAGE: 'QueryLogUsageSource' object has no attribute 'schema_field'

I have the next csv
query_text,database_name,schema_name
"create table dbo.t1 as select * from dbo.t2 join dbo.t3 on t2.id = t3.dsb_id ",default,dbo

and the next yaml
source:
  type: query-log-usage
  serviceName: default
  sourceConfig:
    config:
      type: DatabaseUsage
      queryLogFilePath: "/tmp/queries.csv"
processor:
  type: query-parser
  config: {}
stage:
  type: table-usage
  config:
    filename: "/tmp/query-usage"
bulkSink:
  type: metadata-usage
  config:
    filename: "/tmp/query-usage"
workflowConfig:
  openMetadataServerConfig:
    hostPort: "http://<IP_ADDRESS>:8585/api"
    authProvider: openmetadata
    securityConfig:
      jwtToken: "~~~"


Add auto-save to clin when variants added

Auto save anytime a flagged variant is added (set up a watch on flagged variants?)
Auto save when gene list changes


Update translations

This pull request was created automatically. Please check new translations, approve and deliver them.

Superceded by https://github.com/bigcommerce/checkout-js/pull/2108


System Testing of Installation,Ingestion & Input Data,Processing & Output Data,Visualization

Checking the end-to-end cQube_4.0 Installation(Manual Tc's)

[ ] Azure (Devaraja)
[ ] S3 (Inchara)
[ ]  On-premise (Chetan)

Checking end-to-end for Ingestion (Manual Tc's)

[ ] Azure (Devaraja)
[ ] S3 (Inchara)
[ ]  On-premise (Chetan)

3.Checking end-to-end Data Processing(Manual Tc's)

[ ] Azure (Devaraja)
[ ] S3 (Inchara)
[ ]  On-premise (Chetan)

4.Checking end-to-end Output data (Manual Tc's)

[ ] Azure (Devaraja)
[ ] S3 (Inchara)
[ ]  On-premise (Chetan)

5.Checking the all the functionalities for visualization(Manual Tc's)

[ ] Azure (Devaraja)
[ ] S3 (Inchara)
[ ]  On-premise (Chetan)

6.Upgradation From 3.7-to 4.0 (S3) (Manual Tc's)(Manjula)

[ ] Installation of 3.7
[ ] Upgradation to 4.0
[ ] Ingestion & Processing of data
[ ] Visualization

7.Upgradation From 3.7-to 4.0 (On-premise) (Manual Tc's)(Tanushree)

[ ] Installation of 3.7
[ ] Upgradation to 4.0
[ ] Ingestion & Processing of data
[ ] Visualization

The test case documentation is up in the repository.


Release 0.38.0

Target release date is the weekend of October 3rd/4th.

After the release, we need to add the following job back in to pr.yml.
  validate-gnu-docker-release-image-builds:
    name: Validate GNU Docker release image builds
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v1
      - name: Docker build
        run: "docker build --pull --file=.dockerfiles/release/x86-64-unknown-linux-gnu/Dockerfile ."

The release is going to be moved to when https://github.com/ponylang/ponyc/pull/3653 lands.
We already had a PR land this week that is a "triggers release" issue and #3653 is another that I want to get in with it as it fixes a mess of edge cases. Even with a delay to get #3653 in, the release should still happen before the original target date.

Release is underway.

Release is done. PR opened to turn back on release image building check:
https://github.com/ponylang/ponyc/pull/3662


[macos] Cython not found

I'm trying to install on macos.
Building Eigen3ToPython fails with the following error messages.
Traceback (most recent call last):
  File "/Users/kanehiro/workspace/build/Eigen3ToPython/python3/RelWithDebInfo/setup.py", line 12, in <module>
    pkg_resources.require(pkg)
  File "/usr/local/lib/python3.11/site-packages/pkg_resources/__init__.py", line 966, in require
    needed = self.resolve(parse_requirements(requirements))
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pkg_resources/__init__.py", line 827, in resolve
    dist = self._resolve_dist(
           ^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pkg_resources/__init__.py", line 868, in _resolve_dist
    raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'Cython>=0.2' distribution was not found and is required by the application
ninja: build stopped: subcommand failed.
CMake Error at /Users/kanehiro/workspace/mc-rtc-superbuild/cmake/scripts/cmake-with-prefix.cmake:20 (execute_process):
  execute_process failed command indexes:

    1: "Child return code: 1"

Cython 0.29 is installed.
% brew info cython
==> cython: stable 0.29.35 (bottled) [keg-only]
Compiler for writing C extensions for the Python language
https://cython.org/
/usr/local/Cellar/cython/0.29.35 (325 files, 6.8MB)
  Poured from bottle using the formulae.brew.sh API on 2023-05-29 at 00:32:36
From: https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/cython.rb
License: Apache-2.0
==> Dependencies
Required<EMAIL_ADDRESS>✔
==> Caveats
cython is keg-only, which means it was not symlinked into /usr/local,
because this formula is mainly used internally by other formulae.
Users are advised to use `pip` to install cython.

If you need to have cython first in your PATH, run:
  echo 'export PATH="/usr/local/opt/cython/bin:$PATH"' >> ~/.zshrc
==> Analytics
install: 24 (30 days), 124 (90 days), 24,289 (365 days)
install-on-request: 13 (30 days), 86 (90 days), 18,741 (365 days)
build-error: 0 (30 days)

Hi @fkanehiro
Can you give it a try with the latest version? It seems Cython is keg-only and brew recommends installing it via pip so I have modified superbuild to do that.

Thanks, @gergondet
After manually installing coverage and pytest, I could complete the compilation of Eigen3ToPython.

But now sch-core is failing with cython related messages.
Error compiling Cython file:
------------------------------------------------------------
...
#
# Copyright 2012-2019 CNRS-UM LIRMM, CNRS-AIST JRL
#

from eigen.c_eigen cimport Vector3d
from sva.c_sva cimport *
^
------------------------------------------------------------

sch/c_sch.pxd:6:0: 'sva/c_sva.pxd' not found

Error compiling Cython file:
------------------------------------------------------------
...
from sva.c_sva cimport *

from c_sch cimport *

cdef extern from "sch_wrapper.hpp" namespace "sch":
  void transform(S_Object & obj, const PTransformd&)
                                      ^
------------------------------------------------------------

sch/c_sch_private.pxd:11:39: 'PTransformd' is not a type identifier

Error compiling Cython file:
------------------------------------------------------------
...
#

cimport c_sch
cimport c_sch_private
cimport eigen.eigen as eigen
cimport sva.sva as sva
       ^
------------------------------------------------------------

sch/sch.pyx:10:8: 'sva/sva.pxd' not found

Error compiling Cython file:
------------------------------------------------------------
...
    if self.__own_impl and isinstance(self, S_Object):
      del self.impl
  def __cinit__(self):
    self.__own_impl = True
    self.impl = NULL
  def transform(self, sva.PTransformd trans):
                     ^
------------------------------------------------------------

sch/sch.pyx:22:22: 'PTransformd' is not a type identifier

Error compiling Cython file:
------------------------------------------------------------
...
      del self.impl
  def __cinit__(self):
    self.__own_impl = True
    self.impl = NULL
  def transform(self, sva.PTransformd trans):
    c_sch_private.transform(deref(self.impl), deref(trans.impl))
                                             ^
------------------------------------------------------------

sch/sch.pyx:23:46: Invalid operand type for '*' (Python object)
Compiling sch/sch.pyx because it changed.
[1/1] Cythonizing sch/sch.pyx
Traceback (most recent call last):
  File "/Users/kanehiro/workspace/build/sch-core-python/python3/RelWithDebInfo/setup.py", line 88, in <module>
    extensions = cythonize(extensions)
                 ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/Cython/Build/Dependencies.py", line 1115, in cythonize
    cythonize_one(*args)
  File "/usr/local/lib/python3.11/site-packages/Cython/Build/Dependencies.py", line 1238, in cythonize_one
    raise CompileError(None, pyx_file)
Cython.Compiler.Errors.CompileError: sch/sch.pyx
ninja: build stopped: subcommand failed.
CMake Error at /Users/kanehiro/workspace/mc-rtc-superbuild/cmake/scripts/cmake-with-prefix.cmake:20 (execute_process):
  execute_process failed command indexes:

    1: "Child return code: 1"

Hi @fkanehiro
This should be fixed by updating SpaceVecAlg (or run make update-SpaceVecAlg in superbuild) and having the correct PYTHONPATH is the install prefix is not /usr/local

having the correct PYTHONPATH is the install prefix is not /usr/local

Thank you for pointing this out. At some point, python3 was upgrade from 3.9 to 3.11.
I updated .zshrc.
At the end of installation of  SpaceVecAlg, the 7th test failed with the following messages.
________________ ERROR collecting tests/test_sva_containers.py _________________
ImportError while importing test module '/Users/kanehiro/workspace/build/SpaceVecAlg/binding/python/Python3/RelWithDebInfo/sva/tests/test_sva_containers.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/local/Cellar/python@3.11/3.11.3/Frameworks/Python.framework/Versions/3.11/lib/python3.11/importlib/__init__.py:126: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/test_sva_containers.py:10: in <module>
    import sva
sva/__init__.py:1: in <module>
    from .sva import *
E   ModuleNotFoundError: No module named 'sva.sva'

@fkanehiro thanks for reporting this. I have updated SpaceVecAlg to fix this but I think it will fail next in RBDyn/Tasks/mc_rtc and I'll update the submodule to circumvent that there as well
The underlying problem is that we were generating .dylib files for the bindings in OSX but those are not loaded by Python on this platform (only .so are considered)

I'm closing this. The macOS CI of superbuild now includes the Python bindings and it has been running steadily.
Since then I have also introduced fixes for:

building the bindings with the recently release Cython 3.0
working around an issue introduced in recent CMake versions


[OTHER] remove package-lock.json

What would you like to share or ask?
We are using pnpm for this project & it's recommended to use one package manager for a particular project. So, we need to remove the package-lock.json file & also need to add it to the .gitignore file.

We need to mention to the docs to use only pnpm for this project.
also, we need to mention that if your system hasn't installed pnpm, you need to install it using the npm install -g pnpm

I want to work on this issue.

Additional information
No response

I would like to work on this one.

Assign me to do this work


Feature/use known option

Closes #203

Thank you, will take a look tomorrow

There are three changes in this request:
1. docs

2. help margins

3. practice options changed

The last I do not support for reasons described above (also: I plan to probably add more decks, e. g. english, spanish, french, deutsh, math, musical, chemical...; I plan to probably add a writing mode, where letters are mirrored, add an option to switch on/off higher probability of occurence of letters that user typed in incorrectly; this all seems to be more logical to control within a single screen, which is accessible in one click and icon is BIG and MEANINGFUL (who knows what's behind "other options"?)).
Let's raise a discussion? @braille-systems/learn-braille-team

This PR solves a particular problem. Not going to redesign anything here.

This PR solves a particular problem. Not going to redesign something general here.

It solves a problem in a strange fashion.
And, besides, brings a new problem: "change deck" menu is not accessible directly from practice

there are basically two types of features:

Proof-of-concept. These needn't bee super-convenient, polished and reliable
Usability improvement. Here we should do our best.
This one is of the second type. So I'm going to try an alternative design myself.

here is my vision (though not fully functional, but design is ~ as I would like to see)

I prefer the way @winter-yuki implemented it tbh. Its easily accessed from where its needed the most - right from the practice with one click.
But maybe(?) it would be useful to have the same switch in general settings, thinking of it as a place where all the preferences can be set.

@kaustika thank you. But in my version these settings are accessible in one click, too. This menu pops up on “change decks” button click

I trust in good judjement and common sense of @kaustika and @winter-yuki so let's for now it be like this.
@winter-yuki what do you think about keeping the switch in general settings? I suppose it's a good idea, maybe you could do it?

@winter-yuki please, return the switch (or decide not to return) and let me know when you're done, I will change the help message accordingly and merge this request.

I will delete the corresponding branch.
But will save this code in my clone of this repo: https://github.com/zuevval/learn-braille/tree/feature/use-known-option
Just in case.


Sending Characteristic between threads

Hey, I was trying to create a separate thread to send notifications as a server, but was unable to, because BLECharacteristic is not Send and not Sync. Hence I have a question: what is the point of wrapping characteristics in Arc Mutex if you can't share them between threads?

Please use Arc::clone.
See https://doc.rust-lang.org/book/ch16-03-shared-state.html#atomic-reference-counting-with-arct for details.
If it still doesn't work, please paste the source code.

Something like this for example does not compile. BLECharacteristic doesn't implement Sync and Send because Vec<ble_gatt_sdc_def> does not, since ble_gatt_sdc_def contains *mut c_void.
use esp32_nimble::*;
use std::sync::Arc;

let ble_dev = BLEDevice::take();

let server = ble_dev.get_server();

let service = server.create_service(utilities::BleUuid::from_uuid16(9234u16));
let notify_characteristic = service.lock().create_characteristic(
    utilities::BleUuid::from_uuid16(1642u16),
    NimbleProperties::NOTIFY | NimbleProperties::READ,
);
notify_characteristic.lock().set_value(&[125]);

let char_clone = Arc::clone(&notify_characteristic);
let service_clone = Arc::clone(&service);

std::thread::spawn(move || {
    char_clone.lock().notify();
});

Thanks a lot! You should also mark BLEService as Send, as it has the same problem.


[improve][doc] Use Text Blocks improve too long string in ServiceConfiguration doc annotation

Motivation
I found that there are many long strings joined in ServiceConfiguration doc annotation, we can use Text Blocks to improve readability.
In earlier releases of the JDK, embedding multi-line code snippets required a tangled mess of explicit line terminators, string concatenations, and delimiters. Text blocks eliminate most of these obstructions, allowing you to embed code snippets and text sequences more or less as-is.
Modifications
use Text Blocks instead of long strings joined.
Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[ ] doc-not-needed
(Please explain why)

[x] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

@Technoboy-
could you please review this PR from the technical perspective? Thank you!

could you please review this PR from the technical perspective? Thank you!

This patch should not change any word but only reformatting.


tfsec: 0.58.1 -> 0.58.4

Motivation for this change
Update to latest upstream release 0.58.4
Change log: https://github.com/aquasecurity/tfsec/releases/tag/v0.58.4
Things done

[x ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 133831 run on x86_64-linux 1

1 package built:

tfsec


[FR] Add tags to images anywhere

Hi, TomNCatz. I hope to be able to add tags to images directly in markdown files. This will be more comfortable.
Now, it should only be possible to add tags to images in the gallery or in the meta files of the images. I believe this is not in line with the usual practice. This is because the tags for images often depend not only on the information contained in the image but also on the context in which it is placed. For example, in an article about astronomy, images often require additional tags related to astronomy as a profession. Furthermore, managing tags in a gallery is generally done after the fact, and it is not as intuitive and effective as real-time management. Therefore, I hope to make it easy to add tags to images embedded in Markdown. The gallery should serve more as a tool for image retrieval and maintenance.

I think I have a reasonable fix for this coming out in a few minutes. In 1.1.1 if you right click on a supported embed it should open the meta leaf to the right and you can access and edit info there.

I have to open gallery to use this method.  Is this the original intention of the design?

I version 1.1.1 you should be able to right click on an embedded image or video anywhere in your vault and it should open the side panel to that image's meta file. If it has no meta file it should create one. It currently has a bug I am working on for images with spaces in their path, which I hope to get a fix out for today.
None of that should require you to open the gallery though, just the side panel, and then you should be able to add and remove tags in the side panel.

.If I haven't opened the gallery, the error mentioned above will appear when right-clicking. After opening the gallery, it can be used normally.


SO Charts not working in java 8

I am using java 8 for my project so i run vaadin 14 project with SO Charts got error java version problem. i am not able to update my java because all of my project will be collapsed give me some idea....

This add-on requires Java 11+.
Switching from 8 to 11 should be straight forward (switching to 9 or 10 is not recommended though). I don't think you will have any compatibility issues.
Actually, updating up to Java 15 is not at all a problem. However, if you want to switch to Java 16, you need to take care of few issues.


Cannot go back to react-native from android native activity

I implemented android native module from Bambora SDK and displayed its native view(activity) by getCurrentActivity().startActivity(intent);
It works like a charm but finish() makes whole app quit instead of going back. Here are some of my codes.
public class ToastModule extends ReactContextBaseJavaModule {
    @ReactMethod
    // Calling module function from ReactJS code and works well
    public void goToRegisterCardView(Callback callback) {
        Intent intent = new Intent(getCurrentActivity(), NativeCardRegistrationActivity.class);
        getCurrentActivity().startActivity(intent); //start Activity successfully
    }
    ...
}

public class NativeCardRegistrationActivity extends AppCompatActivity implements ICardRegistrationCallback {
    @Override
    public void onRegistrationSuccess(CreditCard creditCard) {
        finish();// Exit whole app instead of going back.
    }
    ...
}

So I'm sure it's not a reason from Bambora SDK and finish() kills NativeCardRegistrationActivity. The whole app is killed because there is no activity in Activity Stack.
Because I'm not familiar with Java code, I'm not able to figure out what the reason is here.
Please let me know what's wrong and how I can go back to original react-native activity.

Would you mind open-sourcing this? I know a lot of other people (me included) are building the same thing, might be better to do it together instead.

Which source do you mean? For Bambora? If so, I can produce new project with Bambora but unfortunately, I'm not able to share the source now because it is on-developing. 🙏

Yeah, like a react-native-bambora package. Dont worry if its still under development, if you open source it you can probably finish faster since more people can help build it.

There isn't anything obvious in your code.
Can you share how you've declared both your standard RN activity and the NativeCardRegistrationActivity in your AndroidManifest.xml? Maybe you're accidentally configuring your activities to clear the history or reset the stack after NativeCardRegistrationActivity is opened. Or you are using some additional flags in your intent that's not shown in your sample code.
You can also try adb to dump your activity stack and see if your RN activity is gone after your open your  NativeCardRegistrationActivity.

Sorry, it was my own fault. I tested with emulators and it worked fine. So I checked the device developer options and found that "Do not keep activities" is enabled.
Disabling it solved my problem.
I'm now closing this issue and thanks very much for reaching.

FYI That does not sound right. You should have been able to recreate the activity when you navigated back. "Do not keep activities" is not intended to kill the activities in the stack permanently, it is to destroy them to make sure you can recreate with the bundle arguments, saved instance state etc. https://stackoverflow.com/questions/22400859/dont-keep-activities-what-is-it-for


Yaml separator fix

Hi there.  I was working on a demo of the shinyproxy operator for a client and noticed a minor type in one of the yaml files docs/deployment/clustered/shinyproxy2.yaml.
Looks like the separator was added in the wrong  place:
...
roleRef:
  kind: Role
  name: shinyproxy-apps-role
  apiGroup: rbac.authorization.k8s.io
apiVersion: openanalytics.eu/v1alpha1
---
kind: ShinyProxy
metadata:
  name: example-shinyproxy
  namespace: yanamespace
...

The apiVersion for the Shinyproxy ended up in the roleBinding object. It should be:
...
roleRef:
  kind: Role
  name: shinyproxy-apps-role
  apiGroup: rbac.authorization.k8s.io
---
apiVersion: openanalytics.eu/v1alpha1
kind: ShinyProxy
metadata:
  name: example-shinyproxy
  namespace: yanamespace
...

Seems like this operator has potential, however I was not able to get it working yet. Seems like the shinyproxy Pod couldn't create the app Pods, but I digress....

Thanks the heads-up, I just fixed it!
Regarding you other problem: usually ShinyProxy gives some error when it cannot start an app. If the error is too generic, you can have a look at the events of the pod (e.g. using kubectl describe -n shinyproxy pod/... or something like https://k8slens.dev/ )
If you can't figure it out, feel free to open an issue!


Get fingerprint from a sql?

How to get the fingerprint from a specified sql? Is there any implemention in JSqlParser?

Greetings,
there is no specific implementation (because it is not really our concern), however you can safely create a distinct fingerprint by:

parsing your SQL with JSQLParser
de-parsing it (in order to mangle white space and Upper-/Lower-Case spelling)
hashing it with your favorite algorithm (e. g. SHA-1)

First you need to specify what you mean by fingerprint. @manticore-projects suggestion is right, if all of your SQL is relevant. But you could also replace concrete values by a e.g. question mark and then
select * from mytable where id = 5

would by equal to
select * from mytable where id = 6

@wumpz is right of course: For the purpose of execution plans, when Deparsing, you could rewrite all Values (String, Date, Double) with Parameters, before you hash.

@roseduan 请问获取指纹的问题，您解决了么？

@roseduan Excuse me, have you solved the problem of obtaining fingerprints?

Closed, since no further question has been asked.


Add graphql-mesh-gateway search index

Description
This PR adds the Algolia index for the new AdobeDocs/graphql-mesh-gateway repo/site.
Site deployed to production today.
https://developer.adobe.com/graphql-mesh-gateway/
Related Issue
N/A
Motivation and Context
Customers need to search the commerce-frontend-core index.
How Has This Been Tested?
N/A
Screenshots (if appropriate):
Confirmation of index in Algolia:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

LGTM


using geojson.io in an iframe redirect the page!

I added index.php in an iframe src and it redirects url

in index.php
top.location = self.location;
how can set top.location in another way to prevent redirecting page

Did you manage to visualize geojson.io in iframe?


Indicate some Storage methods as must-use

This is the best that can be done for the time being since Roslyn doesn't have an attribute of its own.

Hang on a sec. It seems like [Pure] may be able to be used.


Publish to pypi

Can it be published to PYPI?
It would be very helpful.

The problem is that STPyV8 build workflow builds Google V8 (C++) from source and makes use of the boost library which is installed by default on each selected platform. For this reason releasing a universal wheel on PyPi would be extremely hard (impossible?) at the moment

Can't it be published for linux at least?


feat: implement password input masking

allows users to unmask their password inputs and reveal the hidden text
adds hardcoded list of sensitive model provider configuration fields to ensure sensitive inputs are marked as passwords

https://github.com/user-attachments/assets/565345e8-b4ef-4b84-ba43-adcdacb52e0d

Here's my alternative, non-hardcoded, pass at this: https://github.com/otto8-ai/otto8/pull/824

replacing with https://github.com/otto8-ai/otto8/pull/825 because ollama's host field should not be masked


Make ucws tests skipped when DATABRICKS_ACCOUNT_ID is present

Changes

Added skip_for_env function and make all tests with ucws skip when DATABRICKS_ACCOUNT_ID is present

Tests

[ ] make test run locally
[ ] make fmt applied
[ ] relevant integration tests applied

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (86ca043) 51.62% compared to head (f43f3bf) 51.62%.

:exclamation: Current head f43f3bf differs from pull request most recent head b682085. Consider uploading reports for the commit b682085 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #448   +/-   ##
=======================================
  Coverage   51.62%   51.62%
=======================================
  Files          38       38
  Lines       22266    22266
=======================================
  Hits        11495    11495
  Misses      10771    10771

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add 'KeyboardInterrupt' handlers to all applications/commands

The PyKOB applications/commands (Time.py, News.py, etc.) should gracefully handle the KeyboardInterrupt to exit.
Currently, many of the apps/commands produce a Traceback when ^C is used to exit.

I agree this should be done. The only reason I never handled Ctrl-C more gracefully is due to laziness on my part and lack of requests from users.

Well, you've got two picky users now who've been spoiled by long exposure to proper ^C handling... :-)

Actually, just one picky user is all it takes to convince me!  :-)
On Tue, Apr 28, 2020 at 6:30 PM Patrick Dirks<EMAIL_ADDRESS>wrote:

Well, you've got two picky users now who've been spoiled by long exposure
to proper ^C handling... :-)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MorseKOB/pykob-4/issues/15#issuecomment-620940498,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APHISTJ6QK4ETIE4LQ7EKPLRO57LVANCNFSM4MRZK47Q
.

In writing commercial software (at IBM, TRIRIGA, Oracle, PeopleSoft, Calico) we were never allowed to display a stack-trace to an end user. That is considered a security vulnerability. If testing or users ever get a stack-trace it is a priority 1 (critical) issue and must be fixed immediately. We even had a function in our logging that would 'scrub' a stack-trace before it was written to a logfile to remove PII and other information.
I know this isn't commercial software, but I would like to take care of these as we have a chance to. It shouldn't be hard (an encompassing try on the user processing code.

@pwdirks I noticed updates to many files (as conflicts to my issue #12). Do those changes complete/close this issue, or are there still some apps to update?

Yeah - sorry; wrapping large bodies of code in a try ... except block unfortunately changed the indentation on a lot of code you must've changed for #12 as well.  If you can do a comparison ignoring blank space changes you should find the differences almost negligible but I don't know if the merge tool works strictly on whole lines or is smart enough to merge in only the extra indentation I had to add.
In any case, that merge completed this issue.


AmplitudeEmbedding normalization not behaving as expected

This simple example:
import numpy as np
import pennylane as qml
from pennylane.templates.embeddings import AmplitudeEmbedding

dev = qml.device('default.qubit', wires=3)
@qml.qnode(dev)
def circuit(data):
    AmplitudeEmbedding(data, wires=[0, 1, 2])
    return qml.expval.PauliZ(0)

data = np.ones(shape=(8,)) / np.sqrt(8)
circuit(data)

gives the error
TypeError: unsupported operand type(s) for +: 'Variable' and 'Variable'

This seems to be due to the normalization check within the definition of AmplitudeEmbedding.
As a further point, the normalization check is probably too strict. Instead of checking if the normalization is exactly not equal to one, we should allow for some small numerical tolerance by using np.allclose()

I agree, ran into the problem of normalisation condition being too strict too. Will replace norm == 1 check with isclose check.
With regards to the error, is it wanted behaviour that amplitude encoding encodes trainable parameters?

With regards to the error, is it wanted behaviour that amplitude encoding encodes trainable parameters?

That's a good point. In my (limited) experience I would expect it to encode raw data only, and so would then have to pass it via a keyword argument to avoid the Variable + Variable error @co9olguy got.

A use case where variables have to be encoded is if there is classical pre-processing that scales the inputs by trainable parameters. So @co9olguy is right, we need to consider Variables beeing fed in.

I will work on it!

@mariaschuld Let me know if I can help in any way.

Fixed in PR #376


chore: bdd-nh-1586287607 to 0.0.1

chore: Promote bdd-nh-1586287607 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
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

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


Sort ascending/descending

When I choose the Sort dropdown, the only choice it seems to give me is an ascending sort. Is it possible to get a descending sort, too?? (For example, in Windows Explorer, the sort toggles between ascending and descending.)

We discussed this... we decided this wasn't the kind of sort that required this functionality.


project_preferences tutorial_completed_at field

Could we add a tutorial_completed_at field?
I'd like to store it as a date so we can tell the user if a tutorial has changed since they last went finished it.
Defaults to null.

Can this be stored in the user's project_preferences?

In the preferences hash?

Sure, looks like I can't write arbitrary stuff there though: Error: found unpermitted parameter: tutorial_completed_at

We should move the preferences controller to use json schema and allow writes to the preferences json object.

I was hoping to write arbitrary data here too, in my case for reusing annotation grids with Old Weather


Voice support for multi prompt

For one project they are interested in having different voice responses for the multi-prompt endpoints. Right now voice response is only configured by the router prompt (I believe). The project wanted to use different endpoints for different characters, so they should have a unique voice.

Are they using the API? Your mention of "endpoints" lead me to think that they are

No, sorry. That's just the term they were using for the leaf nodes of the multi-prompt. As in the end of the chain or something

No, sorry. That's just the term they were using for the leaf nodes of the multi-prompt. As in the end of the chain or something


App blank page on next launch when closing with an illustration open

Might relate to #146 and #148.
Steps

Open PixivDeck.
Click on an illustration to see its enlarged version.
Cmd+Q to quit PixivDeck without closing the enlarged image.
Reopen PixivDeck. A blank page will be shown. Cmd+R will not resolve it.

I can forcibly resolve this every time by deleting ~/Library/Application Support/PixivDeck, however it wipes all storage so it's not an ideal solution.
Versions

PixivDeck 0.16.3
Electron 1.7.8
darwin x64 15.6.0

Fixed #165


Add short version of common used flags, like -v for --version

In version <=0.0.14 we have only commands

 -h, --help      help for lets
     --version   version for lets

As you can see, flag version is does not support short abbreviation -v.
I consider, lets should support both variants of all common used flags

#45


Client should yield self

Small change to allow using the client with block syntax:
    Riemann::Client.new do |c|
      c.tcp << {
        :state => 'ok',
        :service => 'test',
        :time => Time.now.to_i,
      }
    end

Also includes change to test suite to allow specifying an alternate host/ip and port via env vars to support testing against a riemann running somewhere besides localhost/std port (eg: vm, container).
RIEMANN_IP=<IP_ADDRESS> RIEMANN_PORT=12345 bundle exec spec/client.rb

@aphyr updated to handle exceptions

:+1: looks good

Thanks all! :)