multiple people's Bone detection

Plugin Version or Commit ID
v0.14.0
Unity Version
2022.2.15f1c
Your Host OS
Windows10
Target Platform
Windows Standalone
Description
Can multiple people's bones be recognized simultaneously? How to set and modify?
Code to Reproduce the issue
No response
Additional Context
No response

How about setting numPoses when running the PoseLandmarker?
https://github.com/homuler/MediaPipeUnityPlugin/blob/f5ab41d387bc2a3547beaa1d6a330aaa14715dc6/Packages/com.github.homuler.mediapipe/Runtime/Scripts/Tasks/Vision/PoseLandmarker/PoseLandmarkerOptions.cs#L48-L51

After testing the changes, the image still only shows the bone information of one person, and cannot draw the bone information of multiple people at the same time

Thank you for your reply! After testing the changes, the image still only shows the bone information of one person, and cannot draw the bone information of multiple people at the same time

When the camera captures two people, it recognizes them here, but there is an empty and null list


[$1000] Emoji is not selected when clicking on it the first time

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

Action Performed:

Login with any account
Go to any report, open the emoji picker, try to tap an any icon, open emoji picker again, tap an any icon then Press send.
Repeat step 2.
Notice that sometimes you need to tap 2 times to choose the emoji.

Expected Result:
Emoji should be selected after user tap on it.
Actual Result:
Sometimes you need to tap 2 times to choose the emoji.
Workaround:
unknown
Platforms:

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

[ ] iOS / Safari

Version Number: 1.2.67-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/218229901-dce93847-be23-4e84-8e90-4d0c2f3fffe8.MP4
https://user-images.githubusercontent.com/43996225/218229912-20886f1f-d6cf-4bce-bb5e-3ad70dd4160a.mp4
Expensify/Expensify Issue URL:
Issue reported by: @hungvu193
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1675418567037039
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0110e6876ed5e355ff
Upwork Job ID:<PHONE_NUMBER>580985856
Last Price Increase: 2023-02-13

Triggered auto assignment to @CortneyOfstad (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

I was able to kind of get this to work on my mobile device using the Safari browser. The first time I did it, I had to click the emoji option twice, but every time after that worked without issue.
https://user-images.githubusercontent.com/10926327/218503870-f0025717-bf5c-4a1a-bbb4-6f3210906428.MP4

Job added to Upwork: https://www.upwork.com/jobs/~0110e6876ed5e355ff

Current assignee @CortneyOfstad is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (External)

Triggered auto assignment to @iwiznia (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Yes, I can fix this issue

Sorry I missed that. I think I agree on doing nothing. The issue seems a (weird) decision of iOS/safari, it is not reliably reproducible (ie: sometimes things work correctly) and I think the solutions are hacks like @parasharrajat said.
Let's close.


TolaAdmin: Limit access to all but 6 functions

If we can scope admin access to a user's country, then we plan to give "basic" (non-super-user) TolaActivity admins access to the following limited set of functions:

Assign user permissions. (Doesn't include super-user permissions.)
Manage offices.
Manage country strategic objectives.
Manage program objectives.
Manage disaggregations.
Add program.

In order fo this work, the following needs to happen:
A super admin needs to configure the "Admin User" group in the admin interface to have the following permission:
auth | user | Can change user
indicators | disaggregation label | Can add disaggregation label
indicators | disaggregation label | Can change disaggregation label
indicators | disaggregation label | Can delete disaggregation label
indicators | disaggregation type | Can add disaggregation label
indicators | disaggregation type | Can change disaggregation label
indicators | disaggregation type | Can delete disaggregation label
indicators | Program Objective | Can add objective
indicators | Program Objective | Can change objective
indicators | Program Objective | Can delete objective
indicators | Country Strategic Objective | Can add  strategic objective
indicators | Country Strategic Objective | Can change  strategic objective
indicators | Country Strategic Objective | Can delete  strategic objective
workflow | Office | Can add office
workflow | Office | Can change office
workflow | Program | Can add program
Once the User Admin group is set u with the above permissions,  then any user who needs to have admin access will need to be a member of this group  and  the"Staff  Status" checkbox selected.

@mahmoodkhan, @jennyhavah  On demo, when I update the tolatest user's country access from my own account, I can see the changes when I look directly at the database.   and then log in as tolatest from a different browser, the database values revert back to where they were before the initial change.  Couldn't replicate locally.

@mahmoodkhan Because of the issues Sanjuro described above, I paused testing until they are resolved and did not ask other Tola team members to review.

@sanjuroj @jennyhavah this is working as expected. Tola-activity-demo is behind cosign and part of of the cosign module functionality is to assign the country you belong to in cosign added to your Tola-activity profile.

Tested in tola-activity-demo using Tolatest logins, permissions include the 6 outlined in this ticket, but include several others under the "Workflow" heading that are not outlined (e.g. checklist items).
However I think we may however wish to retain many of these anyway - such as management access to sites, stakeholders. @jennyhavah did you mean to exclude those?

Screenshot of test view:

Tested in dev with the same tolatest logins, admin now correctly shows only 6 options:

@sanjuroj Can we close and archive this?

Archived and closed.


Mouse wheel scrolls too much in Firefox (custom delta)

Scrolling using mouse wheel in Firefox isn't level by level but several zoom levels at once. It is caused by custom wheel delta which results in high values returned from getWheelDelta() (e.detail was 18 in my case). It is because of:

Settings panel for my mouse (Logitech SetPoint) contains settings "Scrolling Size" which is set to "3 Lines by default" but I changed it to "6 lines".
To further increase scrolling I increased Firefox property mousewheel.default.delta_multiplier_y from 100 to 300.

I do not think this is something desirable. Both Google Maps and local maps (mapy.cz) work even with my settings. I have no idea why delta is even read but wouldn't moving from obsolete DOMMouseScroll to wheel solve this? https://developer.mozilla.org/en-US/docs/Web/Reference/Events/wheel

Can you check if https://github.com/Leaflet/Leaflet/pull/3653 fixes the issue?

I can test it if you tell me how I do get that version. I've no idea how to check it out from Git and how to build it. And what tools I do need to for that.
If your fix is already applied in the master branch, then the bug is still there. (tested with this: http://leaflet-cdn.s3.amazonaws.com/build/leaflet-master.zip on FF 39.0)

Try with this JS: https://www.dropbox.com/s/r0ftoe11q8yow2w/leaflet.js?dl=0

Thanks for the link. I tested it and it works fine now on FF 39.0. However the problem appears in Chrome now (tested with v43.0.2357.134 m).

Closing in favour of #3653.

I don't think #3653 has resolved the issue, reopen?
I'm using Firefox 68.0.2 and I have increased my mousewheel.default.delta_multiplier_y, because I prefer an overall faster scrolling speed:

This is how leaflet behaves on a single mousewheel up/down, it is basically unusable for me:

All other map frameworks don't have this problem in Firefox.


Adding SQL service control cmdlets

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[x] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[x] Pester test is included

Purpose
Adding control over SQL Server services:

Ability to start/stop/restart
Processing services in the correct order (DBEngine stops/starts last/first)
Filters based on service type and instance name
Changing startup mode

Approach

Added 3 new public functions and 3 new internal functions to support the functionality
Modified Get-DbaSqlService to include methods and addition object properties to support pipelines
Re-wrote some of the commands in Get-DbaSqlService (code re-use, Write-Message and indentation)

Commands to test
Included in pester tests, but here are some examples:
Stop-DbaSqlService -ComputerName $instances -Type SqlAgent
Start-DbaSqlService -ComputerName $instances -Type SqlAgent
$services = Get-DbaSqlService -ComputerName $instances -InstanceName $instances -Type SqlAgent,SqlServer
$services | Restart-DbaSqlService

Screenshots
http://imgur.com/a/Jx5Ig

@nvarscar You will need to redo this PR for a few reasons:

Your PR should be based on the development branch
When you develop new commands you need to use the development branch as your starting point.
We need to have each command in a separate PR.

If you have like a public and an internal associated with it, that is fine to put into one PR since there is a dependency.
As well there is already a PR submitted for Get-DbaSqlService so it will be a huge conflict to your version. That command was rewritten in #1887 so you will need to wait until that is merged and then perform your changes on that version.

hey @wsmelton - I know we usually request a smaller number of files, but I'm okay with the higher number this time. @nvarscar can you fix the psd1 again? we rebased you to development and there is a conflict.
as for the Tests, @wsmelton actually suggested mocking for those. Shawn, would you be able to write a functioning mock and I can create the tests for the other commands or if @nvarscar wants to, he can (but not a requirement for this PR? I'd like to get it merged.)

gonna commit to this new branch so that I can modify it, thank you!


Old version of sharp causes installation error

Environment
Node: v21.5.0
npm: 10.2.5
Reproduction
This issue isn't related to ipx itself, but rather to its sharp dependency.
Describe the bug
ipx currently depends on<EMAIL_ADDRESS>This sharp version (and other 0.32.x versions) utilizes a problematic custom install script that may fail on some machines (see sharp#3833 and  sharp#3750). Any sharp versions upwards of 0.33.0 would have fixed this problem.
For reference, this is one of the errors that may happen during installation:
npm ERR! code 1
npm ERR! path /home/user/.local/lib/node_modules/ipx/node_modules/sharp
npm ERR! command failed
npm ERR! command sh -c (node install/libvips && node install/dll-copy && prebuild-install) || (node install/can-compile && node-
gyp rebuild && node install/dll-copy)
npm ERR! sharp: Downloading https://github.com/lovell/sharp-libvips/releases/download/v8.14.5/libvips-8.14.5-linux-x64.tar.br
npm ERR! sharp: Please see https://sharp.pixelplumbing.com/install for required dependencies
npm ERR! sharp: Installation error: Request timed out

Additional context
No response
Logs
No response

Right, I didn't notice that pull request (#190). Sorry for that.
Nevertheless, I think I will keep this issue open so that people that encounter this same problem will understand why.
For those who came here because of the above error: TL;DR, one of ipx dependencies is too old and uses a weird install script. There is a pull request addressing this problem, though. For now, you can use the ipx@next version to avoid this problem.

@Kemystra Has the PR been released to npm? I am on latest ipx 2.0.2 and the sharp dependency is ^0.32.6.
I come here because using Bun sharp is not getting installed. This has been fixed with 0.33.0 of sharp, latest version is 0.33.1

@MickL as of just now, no the PR (#190) is still in draft state (and npm installation is still failing). Also seems like Bun installed ipx@latest and<EMAIL_ADDRESS>just fine, so that's nice. Bun really does handle the script properly.
And yeah, this problem is cascading. I came here in the first place from netlify/cli cuz I can't install that too.

Yes it works fine in Bun with latest sharp 0.33.1. When using @nuxt/image just run bun add sharp which will add 0.33.1 and everything works.
Hope sharp version will be updated soon in unjs/ipx.

https://github.com/unjs/ipx/releases/tag/v3.0.0
(due to potential breaking changes, it is released as a major version in IPX same as sharp)


Using the OWASP dependencyCheck to generate separate reports for java modules instead of one large report

I'm trying to break the dependency check report generated by my Jenkins CI pipeline into multiple reports (one per module) because having one giant report can get rather large and hard to read. This is the code I have so far to accomplish this:
if(fileExists('pom.xml')) {    def pom = readMavenPom file: 'pom.xml'    pom.modules.each {      dependencyCheck additionalArguments: "--scan ./${it} --format CSV --out owasp-reports/${it}-dependency-check-report.csv", odcInstallation: "${env.DEPENDENCY_CHECK}"      dependencyCheck additionalArguments: "--scan ./${it} --format JSON --out owasp-reports/${it}-dependency-check-report.json", odcInstallation: "${env.DEPENDENCY_CHECK}"      dependencyCheck additionalArguments: "--scan ./${it} --format HTML --out owasp-reports/${it}-dependency-check-report.html", odcInstallation: "${env.DEPENDENCY_CHECK}"    } }
I'm getting files generated with the appropriate names, but no dependencies are scanned. However, this appropriately generates a large report containing all the child modules
dependencyCheck additionalArguments: '--scan ./ --format XML --format JSON --format HTML --format CSV --out owasp-reports', odcInstallation: "${env.DEPENDENCY_CHECK}"
Is it possible to use this approach or something close to it?  There are going to be lots of projects and asking each project owner to throw the maven plugin in their poms isn't ideal.

I have also tried this to no avail:
pom.modules.each { 			dir("${it}"){ 				//dependencyCheck additionalArguments: "--scan ./ --format CSV --out ../owasp-reports/${it}-dependency-check-report.csv", odcInstallation: "${env.DEPENDENCY_CHECK}" 				//dependencyCheck additionalArguments: "--scan ./ --format JSON --out ../owasp-reports/${it}-dependency-check-report.json", odcInstallation: "${env.DEPENDENCY_CHECK}" 				//dependencyCheck additionalArguments: "--scan ./ --format HTML --out ../owasp-reports/${it}-dependency-check-report.html", odcInstallation: "${env.DEPENDENCY_CHECK}" 			} 		}

I would highly recommend using the Maven or gradle plugin instead of Jenkins to do the scans. I have provided a high level outline of how you could accomplish this in #2470.


Complete the four items from Angel for Unit Testing that are supposed to be in the team report

you have some nice diagrams, but I'm not sure about this du path tests?

I was going to do full-path testing where I can and state/branch coverage where I can not. I am working on it now and should have something in the testing document as well as a write up in the actual document sometime today.

On Apr 3, 2016, at 11:47 AM, Josh Sherman<EMAIL_ADDRESS>wrote:
you have some nice diagrams, but I'm not sure about this du path tests?
—
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub https://github.com/PennStateSoftwareTest/againstHumanityTest/issues/8#issuecomment-205008745


pdhg_generic branch Attribute error running regularisers.py

Running regularisers.py returns
Traceback (most recent call last):
  File "regularisers.py", line 89, in <module>
    f.primal_var()
  File "regularisers.py", line 81, in primal_var
    return ImageData(np.zeros(self.op.domainDim()), geometry = self.op.geometry)
  File "C:\Users\ofn77899\Documents\Projects\CCPi\GitHub\vaggelis\Wrappers\Python\ccpi\optimisation\operators.py", line 96, in domainDim
    return ImageData(geometry = self.geometry).shape
  File "C:\Apps\miniconda3\envs\cil35\lib\site-packages\ccpi\framework.py", line 737, in __init__
    channels  = geometry.channels
AttributeError: 'TV' object has no attribute 'channels'

The error comes at this line.
The problem is actually at this line
class TV(Function):

    def __init__(self, alpha, memopt = False):
        super(TV,self).__init__()
        self.alpha = alpha
        self.memopt = memopt
        self.op = gradient(geometry=self)

gradient is getting the geometry as self which is the TV instance, not a geometry. Hence the error:  TV object has no attribute channels that comes out when creating a new ImageData from the ImageGeometry

from email:
If I do something like
ig = ImageGeometry(voxel_num_x=3, voxel_num_y = 2)
f = TV(3, geometry=ig)
z = f.primal_var()
print(z.as_array())

it works, but I would like not to define geometry in TV, but rather inherits its geometry from gradient.

Yes, but you define gradient  in the TV's __init__
If you'd pass the gradient Operator to TV you'd be able to.

@epapoutsellis  is this still open?

i reimplemented TV, it's called mixedL12norm in https://github.com/vais-ral/CCPi-Framework/blob/831f6826c479549abe972f858a65fe29643cebc6/Wrappers/Python/ccpi/optimisation/working_with_CompDataContainers/functions.py#L170


W-select issues

Been trying to implement a select but having numerous issues and then docs are actually kind of missing on the specifics.
I get errors. Think it as to do with model-value but do not even understand that part of the docs properly.
The javascript examples(as few as they are) dont show the template usage.

Don't know what to do. Some clarification on how to set them up would be appreciated

Hi @R0N1n-dev,
I've created a Stackblitz for ya here: https://stackblitz.com/edit/waveui-3-wpxs6i?file=src%2FApp.vue where you can see a working w-select with Vue 3 and composition API.
If this doesn't help you, please fork and update the above reproduction link and let me know.
In the future, please share your full code in a reproduction link. That will help me help you faster. ;)

https://stackblitz.com/github/R0N1n-dev/nuxt-issues for repro.
Still have issue with my array. Urs works. Mine does not. No idea why
It gives following error

BTW it also gives the same error on attempting to use w-icon component always.

Wave 3 docs really removed alot of js stuff. mostly see pug templates. This is just weird and dont see the reason for it..


⚠ TiendaEnLinea has degraded performance

In 342deae, TiendaEnLinea (https://www.fahorro.com/) experienced degraded performance:

HTTP code: 200
Response time: 5907 ms

Resolved: TiendaEnLinea performance has improved in ff4e53b.


fix: fix iam permissions to retrieve logs from aws s3

Signed-off-by: Lukas Heppe<EMAIL_ADDRESS>Fixes #TODO
Please do not open a pull request until you have checked ALL of these:

[x] Create the PR as draft .
[ ] Run make pre-commit -B to fix codegen and lint problems.
[x] Sign-off your commits (otherwise the DCO check will fail).

[ ] "Fixes #" is in both the PR title (for release notes) and this description (to automatically link and close the issue).
[x] Add unit or e2e tests. Say how you tested your changes. If you changed the UI, attach screenshots.
[x] Github checks are green.
[ ] Once required tests have passed, mark your PR "Ready for review".

If changes were requested, and you've made them, dismiss the review to get it reviewed again.
Description
When using the IAM policy provided in the docs I couldn't view the logs via UI. By allowing the ListBucket Policy on the bucket instead of the objects within, I was able to retrieve those.
The prinicple is also described in the AWS Blog post https://aws.amazon.com/de/blogs/security/writing-iam-policies-how-to-grant-access-to-an-amazon-s3-bucket/.

It's the same problem as in the SO post below. You are allowed to perform actions on the resources (objects) within the bucket (marked by the wildcard *), but you are not allowed to list the bucket itself.
If you want to access the logs of some worklfow via UI, the S3 client seems to try to perform a "ListBucket" operation, but it is not allowed to do so. Therefore, this policy explicitly whitelists this action. I will try to find the respective line later.
https://stackoverflow.com/questions/38774798/accessdenied-for-listobjects-for-s3-bucket-when-permissions-are-s3

Thanks for clarifying!

 Thank you providing such nice projects :+1:


[WIP] test: container package

ToDo:

podmanengine.go
container.go

ToDo:

podmanengine.go
container.go

Let me know when the PR is ready to be merged


improve sriov nic selector in tests

If the sriov is not able in the kernel for intel nic the totalVF will be 0 so we skip the device. That is not the case for Mellanox devices that will report 0 until we configure the sriov interfaces with the mstconfig package
Signed-off-by: Sebastian Sch<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.07%) to 25.659%

Files with Coverage Reduction
New Missed Lines
%

controllers/sriovibnetwork_controller.go
2
67.33%

Totals

Change from base Build<PHONE_NUMBER>:
0.07%

Covered Lines:
1909

Relevant Lines:
7440

💛  - Coveralls

Can one of the admins verify this patch?


remake design with open-props and radix primitives

remake complete design and CSS with open-props, @emotion/css and radix primitives


refactor(helm): allow using helm built-in namespace

The change is fully backward compatible and allows the user to decide whether he uses the namespace from the values.yaml or the built-in namespace of helm (--namespace).
The change is made possible thanks to the ability to exclude namespaces by name and not only by annotation as was the case until PR #79.
Now you can set the namespace to null and use the built-in namespace, i.e:
helm install k8tz k8tz/k8tz --namespace=myothernamespace --create-namespace --set namespace=

The namespace in the values can be still used with existing namespace by setting createNamespace to false, i.e:
helm install k8tz k8tz/k8tz --set namespace=existingnamespace --set createNamespace=false

And in the future if we want, we can always make breaking changes by removing namespace completely from values.
Fix #91

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

Comparison is base (de22f7c) 67.66% compared to head (2ac987a) 67.66%.

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

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   67.66%   67.66%
=======================================
  Files           6        6
  Lines         838      838
=======================================
  Hits          567      567
  Misses        234      234
  Partials       37       37

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


nixos/nginx: add gitweb sub-service

Motivation for this change
Similarly to services.lighttpd.gitweb. Should be a better match for nginx than cgit, since it supports FastCGI natively.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
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 pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

👌

@wmertens Thanks!


Don't report whitelisted CVEs again

Currently, a whitelist rule matches are atomic. This means if there are some CVEs for a given package covered by a whitelist rule and some are not, the whole set of CVEs is reported again. This goes against users' expectations.
For example, see NixOS/nixpkgs#42882 - exiv2 got a few new CVEs but most of them have already been covered in previous vulnerability roundups and have been added to the whitelist.
It would be better to report non-whitelisted CVEs only.

Fixed in cce172006af8edcd4ddc53fa442e7c4bc0c18e24


Analytics label

add analytics_label
#580

I have already done the feature, i'm just waiting for the merge of the analytics branch because i've based my branch on it :)

Hey, thanks for doing this! :+1: Is there a release coming soon with this commit included?


adguardhome: 0.107.54 -> 0.107.55

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for adguardhome is: Network-wide ads & trackers blocking DNS server
meta.homepage for adguardhome is: https://github.com/AdguardTeam/AdGuardHome
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.107.55 with grep in /nix/store/i2cnm7znm63kj9hs4j08rsydklga9s95-adguardhome-0.107.55
found 0.107.55 in filename of file in /nix/store/i2cnm7znm63kj9hs4j08rsydklga9s95-adguardhome-0.107.55

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

adguardhome

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/i2cnm7znm63kj9hs4j08rsydklga9s95-adguardhome-0.107.55 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/73133dbd4372e53dcd322bb951a167d34f9e17c8#adguardhome

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

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

x86_64-linux

:white_check_mark: 1 package built:

adguardhome

Maintainer pings
cc @numkem @iagocq @rhoriguchi for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Same as #367144 but not formated


fix(app): fix modified dropdown label for removable tags

https://wandb.atlassian.net/browse/WB-15710
Test:
uma-wandb/long_run_name/table?workspace=user-jofang
For long names:
https://github.com/wandb/weave/assets/103448723/ac98ae54-0140-44a7-bb8e-cf4959ec525d
For short names:
https://github.com/wandb/weave/assets/103448723/c4fe8b1d-3058-4858-88b9-c791d24517d6
Things I didn't address, but noticed:

placeholder value in modified dropdown is not centered
no popuover is rendered on hover because that is not implemented yet...will be implemented in a future PR

@vwrj or @tssweeney since i saw you both on this PR  that introduced this logic, can I get your eyes that this still works for the panel plot config? If you could link to a board/panel with this, that'd be great

looks fine - maybe try: https://weave.wandb.ai/?exp=get%28%0A++++%22wandb-artifact%3A%2F%2F%2Ftimssweeney%2Fweave%2Fshareable_oai_example%3Alatest%2Fobj%22%29

@tssweeney thank you - verified change is good on weave as well


BTC not showing up in ASGARDEX

have 10 confirmations, wallet address is correct.   first time have seen this issue.

Your Environment
Windows 10 desktop

ASGARDEX desktop version:
v0.3.0
Operating system and version:

Transaction hash(es):
trx id 82e16e50819783bbda7942d24feb8f67b66cceba7825e4740fb0b947ec974f4a
Wallet address(es):
bc1quy2ahqmjue3rjkmwsqyz80s7zlaq2304j9yz6t

@Zackieboy Two questions:

Transaction hash(es):
trx id 82e16e50819783bbda7942d24feb8f67b66cceba7825e4740fb0b947ec974f4a

What's the transaction about? Is it a swap, deposit , withdraw (pls share a tx hash for such txs). Or is it "just" a "send" transaction from one account to another?

Wallet address(es):
bc1quy2ahqmjue3rjkmwsqyz80s7zlaq2304j9yz6t

Is your BTC wallet address an address you got from ASGARDEX desktop by creating a new wallet before?

just a send from another account to ASGARDEX
yes, BTC address in ASGARDEX, have used to receive BTC many times before.
Thx

I have the same issue..
On 6/7/21 in the THORswap discord was announced the cap was raised.   Using ASGARDEX-0.2.1 I tried to add to the pool with $120 bitcoin and RUNE.   Gave me code 400, which I've not learned the codes yet.   I tried a second time, got code 400.
Next morning I was going to transfer some more bitcoin (still 0.2.1) and no value presented..  it shows my RUNE still, but not the BTC.   I thought maybe a glitch.
I opened the app today, see the update notice.  I've uninstalled 0.2.1 and installed 0.3.0 ..  imported my wallet.
Still shows no BTC.   I've only done (3) transactions.

ok. update to my issue..   in ASGARDEX 0.3.0 I opened the BTC pool window again and now it shows this:

However, I'm unable to open the page linked.   getting connection timeout message.

figured the link issue..  clicking the 'open recovery tool' link takes you to https://asgard.exchange/deposit-sym-recovery/  it's not the testnet.asgard.exchange..  copied that and page opens.

I'll need some help with the connecting my wallet to HTTPS://TESTNET.ASGARD.EXCHANGE/DEPOSIT-SYM-RECOVERY ..  it gives options to create keystore, or use keystore file.   which I didn't make a file.  I have the phrase words, but no file.   I did try making a text file and using that, but then it doesn't like my secure password.

However, I'm unable to open the page linked. getting connection timeout message. I tried to look for that code 400 in the documentation. couldn't find it.

@Zunthras That's a bug you have just recovered. The correct link for the Recovery Tool is https://app.asgard.exchange/deposit-sym-recovery (ASGARDEX 0.3.0 is missing app that url). We will fix it asap (see #1521) and ship it with next release.

I tried to look for that code 400 in the documentation. couldn't find it.

@Zunthras 400 code means a BAD request. It might happened in case of high traffic while raising the cap (as similar as you have described in https://github.com/thorchain/asgardex-electron/issues/1520#issuecomment-857437588). By using Recovering Tool you can withdraw pending BTC back to your wallet. Following step are needed:

In ASGARDEX desktop export keystore in Settings -> Export keystore
Goto https://app.asgard.exchange/deposit-sym-recovery and import the keystore from step 1
Select BTC pool
Click Withdraw All
Pls let me know if it works for you - thx!

1. just a send from another account to ASGARDEX
2. yes, BTC address in ASGARDEX, have used to receive BTC many times before.

@Zackieboy It seems underlaying API of sochain.com ASGARDEX is using have not caught it up properly, it still returns zero amounts for balances https://sochain.com/api/v2/get_address_balance/BTC/bc1quy2ahqmjue3rjkmwsqyz80s7zlaq2304j9yz6t. However, balances are already in your account by checking https://blockstream.info/address/bc1quy2ahqmjue3rjkmwsqyz80s7zlaq2304j9yz6t Maybe sochain does need more time to catch up everything or has some issues recently - we will have an eye on that.

Hi Veado,
Thank you very much for stepping through the process. I didn't realize that was a link in the settings since it is different, I just figured it was a label.
I did as explained and received the following success message:
"Submitting the withdraw transaction was a success. Please wait for the transaction to go through."
In gratitude and light.
Scott
P.S.
Have you heard about Cloudcoin? If you'd like to learn more about it click here:
https://cloudcoin.global/
If you're interested in video entertainment like TV & Movies:
https://storyteller.americanstoriesentertainment.com/affiliate-r/8kltva92b4
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Wednesday, June 9, 2021 8:16 AM, veado @.***> wrote:

However, I'm unable to open the page linked. getting connection timeout message. I tried to look for that code 400 in the documentation. couldn't find it.

@Zunthras That's a bug you have just recovered. The correct link for the Recovery Tool is https://app.asgard.exchange/deposit-sym-recovery (ASGARDEX 0.3.0 is missing app that url). We will fix it asap (see #1521) and ship it with next release.

I tried to look for that code 400 in the documentation. couldn't find it.

@Zunthras 400 code means a BAD request. It might happened in case of high traffic while raising the cap (as similar as you have described in #1520 (comment)). By using Recovering Tool you can withdraw pending BTC back to your wallet. Following step are needed:

In ASGARDEX desktop export keystore in Settings -> Export keystore

Goto https://app.asgard.exchange/deposit-sym-recovery and import the keystore from step 1

Select BTC pool

Click Withdraw All
Pls let me know if it works for you - thx!

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks everybody, I will check later this evening.
On 6/9/2021 1:38 AM, veado wrote:

|1. just a send from another account to ASGARDEX 2. yes, BTC
address in ASGARDEX, have used to receive BTC many times before. |

@Zackieboy https://github.com/Zackieboy It seems underlaying API of
|sochain.com| ASGARDEX is using have not caught it up properly, it
still returns zero amounts for balances
https://sochain.com/api/v2/get_address_balance/BTC/bc1quy2ahqmjue3rjkmwsqyz80s7zlaq2304j9yz6t
https://sochain.com/api/v2/get_address_balance/BTC/bc1quy2ahqmjue3rjkmwsqyz80s7zlaq2304j9yz6t.
However, balances are already in your account by checking
https://blockstream.info/address/bc1quy2ahqmjue3rjkmwsqyz80s7zlaq2304j9yz6t
https://blockstream.info/address/bc1quy2ahqmjue3rjkmwsqyz80s7zlaq2304j9yz6t
Maybe sochain does need more time to catch up everything or has some
issues recently - we will have an eye on that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thorchain/asgardex-electron/issues/1520#issuecomment-857504141,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUNDIBHFW7M7REGGQO5TJLLTR4SB5ANCNFSM46LJAODQ.

So there's a little issue now.
On the web version, https://app.asgard.exchange/ , I'm seeing the bitcoin.
[ASGARDEX-Trbl3.jpg]
But the desktop version does not show the BTC.
[ASGARDEX-Trbl4.jpg]
I did check, BTC addresses are the same (desktop vs web page).
I do the see (est) on the web page. Is the transaction still processing?
I'll check later when I wake up. I work nights.
In gratitude and light.
Scott
P.S.
Have you heard about Cloudcoin? If you'd like to learn more about it click here:
https://cloudcoin.global/
If you're interested in video entertainment like TV & Movies:
https://storyteller.americanstoriesentertainment.com/affiliate-r/8kltva92b4
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Wednesday, June 9, 2021 11:07 AM, Zunthras @.***> wrote:

Hi Veado,
Thank you very much for stepping through the process. I didn't realize that was a link in the settings since it is different, I just figured it was a label.
I did as explained and received the following success message:
"Submitting the withdraw transaction was a success. Please wait for the transaction to go through."
In gratitude and light.
Scott
P.S.
Have you heard about Cloudcoin? If you'd like to learn more about it click here:
https://cloudcoin.global/
If you're interested in video entertainment like TV & Movies:
https://storyteller.americanstoriesentertainment.com/affiliate-r/8kltva92b4
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Wednesday, June 9, 2021 8:16 AM, veado @.***> wrote:

However, I'm unable to open the page linked. getting connection timeout message. I tried to look for that code 400 in the documentation. couldn't find it.

@Zunthras That's a bug you have just recovered. The correct link for the Recovery Tool is https://app.asgard.exchange/deposit-sym-recovery (ASGARDEX 0.3.0 is missing app that url). We will fix it asap (see #1521) and ship it with next release.

I tried to look for that code 400 in the documentation. couldn't find it.

@Zunthras 400 code means a BAD request. It might happened in case of high traffic while raising the cap (as similar as you have described in #1520 (comment)). By using Recovering Tool you can withdraw pending BTC back to your wallet. Following step are needed:

In ASGARDEX desktop export keystore in Settings -> Export keystore

Goto https://app.asgard.exchange/deposit-sym-recovery and import the keystore from step 1

Select BTC pool

Click Withdraw All
Pls let me know if it works for you - thx!

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

But the desktop version does not show the BTC

@Zunthras It seems these are issues with underlying API (see previous message to Zackieboy https://github.com/thorchain/asgardex-electron/issues/1520#issuecomment-857504141). We are still on it ...

I have the same issue. BTC is not showing up in the wallet and I definitely have some available. Very disappointing that I missed the $9 Rune.

We are currently preparing a hotfix release 0.3.2 to fix issue of BTC balances, which will be published within next few hours.


Upgrade marked to 0.3.9 to fix vulnerability

https://npmjs.com/advisories/531

Hi! Thanks for taking care of security.
Could you please verify that everything still works after updating to 0.3.18?
Github marked the package with another vulnerability at https://github.com/markedjs/marked/commit/b15e42b67cec9ded8505e9d68bb8741ad7a9590d.
That one is fixed in the specified version, and seems related to the one you just pointed out.
Thanks in advance!

Sorry, button misclick.

Well, I tried 0.3.19, and that had more vulnerabilities fixed in >=0.6.2, so I just went all the way to 0.7.0.
I'm not sure how to fully test, as you don't have any tests to run in your project, but I am using https://github.com/surebert/docker-ffmpeg-service and I used my updated 0.7.0, and it still rendered the readme page.

Hey, I tried the 0.7.0 with my project and it seems to work just fine. Any chance it will be merged soon?


doubly_linked_list.cpp

Code contains the explanation of it

Code contains the explanation of it

Write alongside code, check contribution guide for example

updated


Task/modal integrantes

What's this PR do?
It changes the way in which we add and edit integrantes of a family.
Where should the reviewer start?
captura/views.py
familias/forms.py
familias/tests.py
captura/test_views.py
check the static files, including the js.
How should this be manually tested?
npm install
run the tests.
log as a capturista and try adding/editing integrantes.
Any background context you want to provide?
The main purpose of this PR is to allow the creation and modification of integrantes in modals, via ajax. Instead of three different forms for integrante, alumno and tutor, there's just one that inherits from the modelform of integrante.
This template was adapted ~stolen~ from Quickleft/Sprint.ly

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@880b3ad). Click here to learn what that means.
The diff coverage is 99.38%.

@@            Coverage Diff             @@
##             develop      #76   +/-   ##
==========================================
  Coverage           ?   97.43%
==========================================
  Files              ?      126
  Lines              ?     3196
  Branches           ?        0
==========================================
  Hits               ?     3114
  Misses             ?       82
  Partials           ?        0

Impacted Files
Coverage Δ

captura/urls.py
100% <ø> (ø)

jp2_online/settings/base.py
96.66% <ø> (ø)

captura/test_views.py
100% <100%> (ø)

captura/views.py
93.67% <100%> (ø)

familias/tests.py
100% <100%> (ø)

familias/forms.py
97.4% <96.15%> (ø)

Continue to review full report at Codecov.

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

👍


Support for http basic authentication.

Hello,
Do you guys think it could be easy to implement http basic authentication in redbird ?
like for example can we access the http-proxy object in order to extend it.
https://github.com/http-auth/http-auth

Maybe you could do that manually with a resolver but: #150

Hello!
Anyone else finds themselves wondering about this might hack together something like this. (I found no easy way to return a response from a resolver. Nor to specify * or <IP_ADDRESS> as host, but for my use case losing host-based routing is not a big problem, I explicitly wanted to disable that anyway.)
#!/usr/bin/env node

const proxy = require('redbird')({port: 8111});
const http  = require('http');

const target = "http://<IP_ADDRESS>:6688";

const server = http.createServer(function(req, res) {
    res.statusCode = 401;
    res.setHeader('WWW-Authenticate', 'Basic realm="secure stuff"');
    res.end('Need some creds');
});

let serverPort = 0;

server.on('listening', function () {
        serverPort = server.address().port;
    });

server.listen();

const auth = function(host, url, req) {
    const authH = req.headers['authorization']
    if (!authH) {
        return 'http://<IP_ADDRESS>:' + serverPort;
    } else {
        const tmp = authH.split(' ');
        const buf = Buffer.from(tmp[1], 'base64');
        const plain_auth = buf.toString();

        if (plain_auth != 'admin:putyoursecr3tp4sswordHERE') {
            return 'http://<IP_ADDRESS>:' + serverPort;
        } else {
            return target;
        }
    }
};

auth.priority = 100;

proxy.addResolver(auth);

proxy.register(".", target);


Does Sentiment feature support Chinese (zh-CN) ?

I tried but always received "null" in this field when feeding Chinese audio.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 73fb7e76-fb80-c420-2efe-e3ca3a27b749
Version Independent ID: 07a42557-deb3-e4b3-3f14-f26a27be49c2
Content: How to use Batch Transcription - Speech Service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/batch-transcription.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @PanosPeriorellis
Microsoft Alias: panosper

@wukun2015
Thanks for the feedback! We are currently investigating and will update you shortly.

@wukun2015
Hi Wuk, thanks for reaching out to us. We are supporting the following languages:
zh-CN | Chinese (Mandarin, simplified)
zh-HK | Chinese (Cantonese, Traditional)
zh-TW | Chinese (Taiwanese Mandarin)
A gentle remind here: A standard subscription (S0) for Speech Services is required to use batch transcription. Free subscription keys (F0) will not work. For additional information, see pricing and limits.
Please check your subscription and let us know if you are in Standard Subscription but still in trouble. Thanks.

Glad to hear that !

@wukun2015
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


Use new vscode-gradle API

In preparation for a new version of vscode-gradle with a new API being released.
Tests will be broken until I upgrade the vscode-gradle package.

https://github.com/badsyntax/vscode-gradle/releases/tag/3.0.0

https://github.com/badsyntax/vscode-gradle/releases/tag/3.0.1
I really need to invest the time to write proper tests for vscode-gradle :(


added type meta in config and preference structs

What is the purpose of this change? What does it change?

added a metav1.TypeMeta in the LocalConfig and Preference struct
Was the change discussed in an issue?
fixes https://github.com/redhat-developer/odo/issues/1365

How to test changes?
# run
odo preference set timeout 20
# check the ~/.odo/config.yaml for Kind and ApiVersion

# run
odo config set memory 500MiB
# check the <current-dir>/.odo/config.yaml for Kind and ApiVersion

/test unit

Unit tests have failed can you ptal?

resolved @mohammedzee1000

:white_check_mark: odo build 1928 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/f82b7b59b1 by @girishramnani)

\hold cancel

/hold cancel

/refresh

/meow


Foscam support

The generic IP camera implementation did not work for my Foscam FI9821W because the Foscam cameras return a small HTML page containing the image instead of the raw image itself. This implementation makes the first request to snap the picture, then parses through the HTML to find the generated image and finally, makes a second request for the raw image.

Can you add it to .coveragerc ? After that it looks good to merge!

I was already in the code, did it myself. Thanks for your contribution!


upgrade to wp 5.5

@thetwopct did I forget to copy anything over from the cncfci WP 5.5 upgrade?  Or is this good to deploy?


.NET 4.7 Regex replace returns wrong string

Dear Devs,
the following code is used:
'''
[TestClass]
public class DebugTests
{
[TestMethod]
public void OneSpaceStringTest()
{
var oneSpaceString = Regex.Replace("x    x", "[ ]{2,}", " ");
Assert.AreEqual("x x", oneSpaceString);
}
}
'''
one would expect to get "x x" but returns in the FULL framework "xx" instead.
.NET Core returns the correct string.
Expectation is based on the test i did with RegexR
https://regexr.com/3rtdn
Martijn

How can folks repro this? I just ran Regex.Replace("x x", "[ ]{2,}", " ") on .NET Framework 4.7.2 and got "x x" as a result.

The server was running 4.7.1 and not 4.7.2
@jnm2 4.7.2 runs like a sharm. Thanks


GitHub Action and Encoding logic.

Issue #, if available: N/A
Description of changes:

Ported the original implementation to another branch called docker.

Now the main branch only holds Rust code.

Added a GitHub Action to auto PR the generated WASM to react-website branch.
Added the Session Generation logic in rust.
Change the Readme.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

Comparison is base (4014486) 0.00% compared to head (3642df1) 0.00%.

Files
Patch %
Lines

src/encode.rs
0.00%
39 Missing :warning:

src/lib.rs
0.00%
27 Missing :warning:

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #12   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          2       3    +1
  Lines        119     178   +59
  Branches     119     178   +59
=====================================
- Misses       119     178   +59

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


Behaviour change: CORSPolicy middleware returns 404 on preflight request

In the latest release, CORSConfig was deprecated in favour of using CORSPolicy. This has introduced a behaviour change where preflight requests will now return a 404 instead of a 200. I think this may cause some (all?) browsers to reject preflight requests, but I'm not sure.
In 0.23.1 was doing the following:
val service: HttpRoutes[F] = ???

private val corsConfig = CORSConfig.default
    .withAllowedMethods(Some(Set(Method.POST, Method.GET, Method.OPTIONS)))
    .withAllowedHeaders(Some(Set("Authorization", "Content-Type", "Accept")))
    .withAllowCredentials(false)

EmberServerBuilder
      .default[F]
      .withHost(host)
      .withPort(port)
      .withHttpApp(CORS(service, corsConfig).orNotFound)
      .build

and then an OPTIONS request with headers "Access-Control-Request-Method" ="POST" "Origin"="http://example.com",  "Access-Control-Request-Headers"="authorization,content-type,accept" would return 200 with the appropriate CORS headers.
Following the deprecation of CorsConfig in 0.23.2, I replaced corsConfig like so:
private val cors = CORS.policy
    .withAllowMethodsIn(Set(Method.POST, Method.GET))
    .withAllowHeadersIn(Set(ci"Authorization", ci"Content-Type", ci"Accept"))
    .withAllowCredentials(false)

EmberServerBuilder
      .default[F]
      .withHost(host)
      .withPort(port)
      .withHttpApp(cors(service).orNotFound)
      .build

But now the same preflight request returns a 404. I can see that this is because in the implementation of CorsPolicy#apply, service: HttpRoutes[F] is being applied to the incoming request and the preflight headers added to the response, here:
https://github.com/http4s/http4s/blob/5ebfb746ff98817959cbcce3ab2520342253c608/server/src/main/scala/org/http4s/server/middleware/CORS.scala#L413
So because my service doesn't have an endpoint defined for an OPTIONS request, it falls through and I get a 404.
In the previous implementation an options response was created, as seen here:
https://github.com/http4s/http4s/blob/5ebfb746ff98817959cbcce3ab2520342253c608/server/src/main/scala/org/http4s/server/middleware/CORS.scala#L251
which would always return a 200.

Ouch.  A fix is my top priority today.

Some CORS implementations have a setting to allow a passthrough to an underlying OPTIONS implementation.  This would be nice too, but I'm going to focus on the ass-on-fire bug first.

I'm going to lunch, but anyone affected is welcome to try that out locally: should just be a copy-and-paste into your own package.  Once approved, I'll start a release train.

Fixed.  Releases working their way through as 0.21.28, 0.22.4, 0.23.3, and 1.0.0-M26.


Bump d2l-colors version to fix BSI build

@dbatiste ?
Unable to find a suitable version for d2l-colors, please choose one by typing one of the numbers below:
    1) d2l-colors#~2.1.0 which resolved to 2.1.0 and is required by jquery-vui-change-tracking#0.5.3
    2) d2l-colors#^2.1.0 which resolved to 2.2.0 and is required by brightspace-integration, d2l-dropdown#3.0.0, d2l-icons#2.13.0, d2l-image-action#0.5.3, d2l-link#3.1.2, d2l-menu#0.2.3, d2l-navigation#0.14.6, d2l-page-load-progress#1.1.1, d2l-table#0.3.0, jquery-vui-more-less#1.2.1, vui-breadcrumbs#2.1.1, vui-dropdown#0.7.2, vui-input#1.6.3, vui-list#1.1.1, vui-validation#1.1.2
    3) d2l-colors#^2.0.0 which resolved to 2.2.0 and is required by d2l-button#3.0.6, d2l-loading-spinner#4.0.0, d2l-my-courses#0.18.5, vui-field#1.2.0
    4) d2l-colors#^2.2.0 which resolved to 2.2.0 and is required by d2l-typography#5.1.1

Coverage remained the same at 95.402% when pulling 29c981f73243efc2354306e131ba61f9cad11559 on bump-d2l-colors-version into fc9d3ce882fa2a876d99d0920fb40deef66c1e3e on master.

@dlockhart ?

Thanks Anthony.  Yeah, I am going through the components to update d2l-colors.  There are others that are being fixed as well.


[FLINK-27844] Support column type evolution in file meta

The column type may be updated when table store read data from file meta, it should convert the data from old column type to new type.
For example, there're four fields with types as follows
1->a int, 2->b int, 3->c int, 4->d int
After column type evolution, the new types are as follows
1->c bigint, 2->a float, 3->d double, 4->d bigint
When table store reads data from file meta according to the old column type, it should convert the data to the new column type.
The main changes in this PR are as followed

Added DataValueConverter to convert value from old LogicalType to new type.
Added SchemaEvolutionUtil.createConvertMapping to create converter mapping between table and data fields
Updated FieldStatsArraySerializer to get new type value from old type value
Updated AppendOnlyFileStoreScan and KeyValueFileStoreScan to create converter mapping

The main tests are as followed

Added AppendOnlyTableColumnTypeFileMetaTest to test column type evolution for file meta in append-only table
Added ChangelogValueCountColumnTypeFileMetaTest to test column type evolution for file meta in changelog value count table
Added ChangelogWithKeyTableColumnTypeFileMetaTest to test column type evolution for file meta in changelog with key table

Hi @JingsongLi I have updated this PR according to the cast executor, please help to review it again, thanks

Hi @JingsongLi I have rebased from master and update the codes, please review it again, THX


migrate logging to new framework

Lots to be done here - if someone new to hyperion wants to contribute, this is a tedious but worthy task!
There are 178 std::cout calls in the code as show here:
/hyperion.ng# find src libsrc -type f -exec grep -n std::cout /dev/null {} ; | wc -l
178

no problem, i just need a code pattern /howto that could be used from me and for docu that devs could understand it.
regarding include, loglevel variants and how i call them

please also take printf, std::cerr and qDebug() in account ;-)

info for current head of master regarding new logger:
you can add commandline options for loglevel:
view debug messages:
hyperiond --debug

(more see help)

Call for help! I wrote some doc about logging in the wiki:
https://hyperion-project.org/wiki/Logging
Would be great if somebody can migrate logging. We don't need migrating logging at once - component by component is absolutly ok. Ifquestions ... I will assist

Good idea, i will have a look. For sure i could do that boring part :)

Starting with the blackborder files are nice to practice

What about banning commits of files that don't use the new logging framework ?

Not sure what you mean in specific. Remove all code which has no logger implemented? This break Hyperion?

Dont accept pr's when new logger oisnt used

Of course.

I have a question regarding the GetInstanceOf. This result in? another logoutput? a phrase like "EffectEngine" before the message itself?

a new logging object if it doesnt exist.
If it does, you get a handle to the existing logging object

This results in: you no longer see which part of Hyperion has thrown this error? If the message itself contain no "identifier"

Sorry, I missed part 2 of your question.
The name of the logging object IS printed

try to explain:
first get a logger:
log = logger::getinstance("effects")
this will create a logging object with logging scope "effects", if not exists, otherwise it returns existing object.
on log messages:
Info(log, "my msg")
a specific (named) logger is neccessary. This will print out the desired message and the logging scope. More than that, the logger knows exactly the position in source code (function, file and line number) where the message occurs.
Debug messages will print this information too, while info-error don't print those extra infos. This is because normal user would e a bit confused. with those info.

the "solid" principles for good code:
https://en.wikipedia.org/wiki/SOLID_(object-oriented_design)
https://de.wikipedia.org/wiki/Prinzipien_objektorientierten_Designs#SOLID-Prinzipien

just for statistic:
search for outputs to convert to new logging:
find src libsrc -type f -exec egrep -n "std::cout|qDebug|qWarning|std::cerr" /dev/null {} \; | wc -l
result: 138-1 (logger.cpp has a cout, but this is neccessary ...)
getting the files:
find src libsrc -type f -exec egrep -n "std::cout|qDebug|qWarning|std::cerr" /dev/null {} \; | cut -d: -f1 | sort | uniq
result:
libsrc/blackborder/BlackBorderDetector.cpp
libsrc/blackborder/BlackBorderProcessor.cpp
libsrc/boblightserver/BoblightClientConnection.cpp
libsrc/bonjour/bonjourserviceregister.cpp
libsrc/effectengine/EffectEngine.cpp
libsrc/grabber/dispmanx/DispmanxFrameGrabber.cpp
libsrc/grabber/framebuffer/FramebufferFrameGrabber.cpp
libsrc/grabber/osx/OsxFrameGrabber.cpp
libsrc/grabber/v4l2/V4L2Grabber.cpp
libsrc/grabber/x11/X11Grabber.cpp
libsrc/hyperion/Hyperion.cpp
libsrc/hyperion/LinearColorSmoothing.cpp
libsrc/jsonserver/JsonClientConnection.cpp
libsrc/jsonserver/JsonServer.cpp
libsrc/leddevice/LedDeviceLightpack.cpp
libsrc/leddevice/LedDeviceLightpack-hidapi.cpp
libsrc/leddevice/LedHIDDevice.cpp
libsrc/protoserver/ProtoConnection.cpp
libsrc/protoserver/ProtoServer.cpp
libsrc/webconfig/CgiHandler.cpp
libsrc/webconfig/QtHttpClientWrapper.cpp
libsrc/webconfig/StaticFileServing.cpp
src/hyperion-aml/hyperion-aml.cpp
src/hyperion-dispmanx/hyperion-dispmanx.cpp
src/hyperiond/main.cpp
src/hyperion-framebuffer/hyperion-framebuffer.cpp
src/hyperion-osx/hyperion-osx.cpp
src/hyperion-remote/hyperion-remote.cpp
src/hyperion-remote/JsonConnection.cpp
src/hyperion-v4l2/hyperion-v4l2.cpp
src/hyperion-x11/hyperion-x11.cpp

checking led devices:
false positive:
LedDeviceLightpack.cpp (just 2 std:cout commented out)
LedDeviceLightpack-hidapi.cpp (just 2 std:cout commented out)
LedHIDDevice.cpp (commented out printf and std:cout)
positive in brackets:
LedDevicePiBlaster.cpp (there are some printf but the majority is done)
positive:
LedDeviceWS2812b.cpp (printf, fatal and so on)

piblaster is fine - those printfs are how hyperion writes to the piblaster daemon

@redPanther
if i am right you removed the "hyperion ambient light" notice and the version including the build number from the log, for sure this was not your initial intention :)
And the BB detection needs a "moved" logging or i just rename it to debug level.

I migrated Proto and Json, but needs a review and help for 2 lines.

removed the "hyperion ambient light" notice

it's there:
https://github.com/hyperion-project/hyperion.ng/blob/master/src/hyperiond/main.cpp#L106
message has to be moved to logger .... but I will make a more general way to get hyperion version

That's debug, and main.cpp is never used for the "log", but as you said needs some adjustment.
I move bb to debug, as long this notice is so annoying :)

main.cpp is never used for the "log"

what you mean? there is the "MAIN" logger defined. There are log messages, e.g.:
https://github.com/hyperion-project/hyperion.ng/blob/master/src/hyperiond/main.cpp#L93
I moved the version output to --version ... if we have a good fucntion for the version number, the hyperion class will make a info message for version number.
in normal circumstances we should keep terminal output as clean as possible, just the errors and warnings (--debug and --verbose are for bug hunt / development)

in normal circumstances we should keep terminal output as clean as possible

yes, : if someone uploads "a log" eg from webinterface it does not contain a version, this is no problem if we handle this like the following
workaround with a paste name (eh hyperion version gathered via JSON RPC).
http://pastebin.com/api
Hm, okay if everyone using the logupload we know always what we have here.
:)

as i said, if we have a good func for version number, this info is printed to log. I didn't migrate it, because such a func didn't exists atm.
later on we can make a log upload from webinterface to pastebin as you suggest

pastebin (or another paste service)? yes, i want to stop the log postong without quote tags at the forum :)

something wiped your list? :o

Are we ready here? Lost the overview ...

hyperion-remote.cpp are std::cerr for output and ws2812b some printf

From memory Hyperion remote was ok - they were printing to the console as they should

I took a look ... mission is done. we can close the issue

Yay :)


backend: Fix default env vars on main

Making sure make first-run is successful

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

Project coverage is 81.38%. Comparing base (9125660) to head (f14c6c6).
Report is 18 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #429      +/-   ##
==========================================
- Coverage   86.46%   81.38%   -5.09%
==========================================
  Files         178      198      +20
  Lines        6873     8210    +1337
==========================================
+ Hits         5943     6682     +739
- Misses        930     1528     +598

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


Immutable size / length ??

in server/src/main.js, i try to log the size of an Immutable List<BinData>.
console.log('SAVING BIIIIIIIIINS !!!', data.action.bins.length, data.action.bins.size);
I get a TS warning: server/src/main.ts(46,60): error TS2339: Property 'length' does not exist on type 'List<BinData>'.
The length outputs the correct length, while size outputs undefined...

blargh. Probably wrong tsd :-/

No, what's weird is i should be able to output the size, since it's an Immutable... I don't really care about the TS warning.
It's as if my Immutable List acted as a regular Array...

hmm... Try Array.isArray(data.action.bins) ?


Introduce permissions into Ngrok Ingress Controller webapp.

Description
Currently, my production and development domains are both running on the same Ngrok account, which may not be ideal. Recently, a developer accidentally deleted my production domains, and I had to go through the lengthy process of restoring them. This incident has highlighted a crucial need for role-based permissions within the Ngrok web app. Having this feature would help prevent accidental modifications or deletions in the production environment by limiting access and actions based on user roles.
Use Case
The primary use case for this feature is to provide administrators with the ability to assign specific permissions to different users, especially in environments where multiple domains (both production and development) are managed within the same account. By implementing user roles and permissions, it would be possible to restrict access to critical configurations and prevent accidental changes or deletions in production, thus minimizing downtime and enhancing security.
Related issues
None known at this time.

@nsgoneape thanks for the issue. Adding true role based access control to the ngrok platform is currently on our roadmap, which would help solve for this issue. I don't have an ETA of when it will be available, but I will add you to our list of alpha/beta users and will reach out as soon as we have something to test out. Thank you!

Hey @russorat, how are you doing?
Is there an update on this feature? My organization is making great use of your product. But as I am adding more people, the riskier running my operations via ngrok is becoming.


What's the version of the demo? TypeError: Cannot read property '287' of undefined

that's my error i get by using the xlsjs.readFile function.
I've tested the xls-file with your demo and there works fine.
so what's the version of xlsjs in the demo? Would help me a lot thanks.
Here is the complete error-stack:
TypeError: Cannot read property '287' of undefined
at __readInt32LE (.../node_modules/xlsjs/xls.js:129:49)
at make_sector_list (.../node_modules/xlsjs/xls.js:1619:8)
at parse (..../node_modules/xlsjs/xls.js:1437:19)
at Object.readSync as read
at Object.readFile (.../node_modules/xlsjs/xls.js:7131:38)

Related: https://github.com/SheetJS/js-xlsx/issues/569
I believe the fix there also resolves this issue.  Please try against js-xlsx -- it has the same API and exposes XLS as well as XLSX browser variables.


build: skip duplicate commit entries in changelog

Ensures that commits are not showing up multiple times in the changelog.
Commits can show up multiple times, if a changelog has been generated on a publish branch and has been copied over to "master". In that case, the changelog will already contain the commits that have been cherry-picked into the publish branch. These shouldn't be added twice.
Closes #12915

@jelbourn The logic is kind of simple, but should do the job. I've also added a log message so that we can easily see if something shouldn't be removed.


Handle parameters on the format r#type

A new version of my previous non working PR, this seems to actually do what I intended.
When you have a struct with a member using a reserved word as a name, it will be specified as r#type. For Query structs, this is probably quite common. Previously they got hte name "r#type" in the openapi spec, so I just added a strip_prefix r# to handle this case.

Hi, is it possible to add a test so we prevent any regressions?

Test added


When Skipping a Question it is frequently reselected

place recent questions in a penality loop. Don't select questions from this penalty loop! The penalty loop for skipped questions should be as long as the current selection list for the next question.

It is important that skipped questions don't reappear BEFORE all other questions in the current entropy scope were covered.


Variable replacement of the form: ${config}[browser] fails for indexed pandas Series

While variable expansion like this:
${config['browser']}
works fine on my ${config} object, it fails when indexed like this:
${config}[browser]
with error:
robot.errors.ExecutionFailed: Series '${config}' used with invalid index 'browser'.
I would have expected this to work

Can you create a concrete, but simple, example demonstrating the problem?

*** Test cases ***
X
    ${X}    Evaluate    pandas.Series(data={'a': 1, 'b': 2, 'c': 3}, index=['a', 'b', 'c'])
    Log     ${X['a']}   console=True
    Log     ${X}[a]     console=True    # This fails currently

Result from above is:
PS C:\Users\a480438\s> robot .\Pandas.robot
==============================================================================
Pandas
==============================================================================
X                                                                     .1
X                                                                     | FAIL |
Series '${X}' used with invalid index 'a'. To use '[a]' as a literal value, it needs to be escaped like '\[a]'.
------------------------------------------------------------------------------
Pandas                                                                | FAIL |
1 test, 0 passed, 1 failed
==============================================================================
Output:  C:\Users\a480438\s\output.xml
Log:     C:\Users\a480438\s\log.html
Report:  C:\Users\a480438\s\report.html

Thanks, I was able to reproduce the problem with the example above.
In my case the failure was a bit worse, though, and I got this error:

AttributeError: 'NoneType' object has no attribute 'islower'

The above error occurred in code that was trying to get the name of the Series object to be used in the error message you got. Not sure why you didn't get that, but I suspect it has something to do with Robot or Pandas version. Anyway, that particular problem I already fixed locally and now I get the exact problem you see.
The reason for the "invalid index" error is that [Series]https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.Series.html) is not technically a mapping but an array. More precisely, it doesn't implement collections.abc.Mapping and thus Robot's is_dict_like returns False. It's detected to be a list by is_list_like, though, and thus the code handling ${x}[y] assumes y to be an index, not a key.
Although Series isn't a Mapping, it still supports accessing values by keys and thus that not being possible using the ${x}[y] syntax is unfortunate. Some ideas that we could possibly do:

When we detect that something is a list and items are accessed with something that's not an index (or slice), try to access the item with the given value anyway. This ought to solve the issue with Series and, if we do this extra step only when index detection fails, this ought to be backwards compatible as well.
Detect that the object supports "mixed mode" and in that mode try both key and index access. I'm afraid this kind of detection is not possible in generic manner, but we possibly could special case Series and then other similar objects in the future.

I'll prototype the approach 1. above and see how it could work. If it works nicely and isn't overly complicated, it could be still included into RF 4.1. The aforementioned fix for getting the type name will nevertheless be included.

I much prefer option 1.  Better to ask forgiveness and all that
I already have another weird difference (both error situations, so doesn't matter so much) with this syntax and a custom object (so in this case I can customize the object behavior) - the point being that adding Series specifically won't help general use of this nice feature.

Supporting "mixed mode" wasn't overly complicated. I try to get RF 4.1 rc 1 out tomorrow. Hopefully you @VernonCrabtree can test that it works with Pandas as you expect.


Connecting to github broken due to outdated libssh2

Github has deprecated a number of crypto protocols that are used in the 5 year old version of libssh2 (@ficfa55) referenced by objective-git.  See https://githubengineering.com/crypto-deprecation-notice/
This is causing Unable to exchange encryption keys errors preventing the ssh session from being established.  This is likely the root cause of the issue for #641 as well.  (In fact, I believe he may be experiencing problems with the same app that I am, passforios https://github.com/mssun/passforios.
I don't program for iOS so I don't have the toolchain to provides a pull request.

#643 has updated libssh2 to 1.8.0, which works against the "new" GitHub cryptosuite. Unless you find it's still not working, I'm closing this. Thanks for the report !


[Tracker] Port to a c++ module for master

Would you be interested in a c++ module for this?
I could try porting it.

Yeah, please do.

I see if I can find some effort when I get stuck on my gltf2 task to do the port to godot engine master. It'll be a non-core c++ module and be less accessible but still workable.

[ ] Basic evaluation of what dependencies are needed
[ ] Convert to c++

I have been meaning to contribute to godot for a while now. I can try porting it to master.
For the dependencies, what are the new ways to draw immediate geometry in godot4? I dont see an ImmediateGeometry node in godot4.
The VisualServer should probably used anyway since it doesnt have the overhead of immediate geometry.
How do you think this should be implemented?

Godot 4 uses ImmediateMesh. I'll research your other questions.

https://github.com/V-Sekai/godot-verlet-rope is my branch.
I'm having some trouble it. Can you review? Then I can port to C++.

The mesh looks very thin.

Here's a non godot example:
https://twitter.com/R_Nikaido/status/1274142361635876866?s=20

Do you think the bug is that you need to go from vector3 to Transform3D?

That is a known problem. In editor mode there is no way to get the position of the editor camera, and so the rope cannot always "billboard" in the direction of the camera.
But in play mode it looks correct. If godot 4 has a way to get the position of the editor camera a bit of code should be able to fix it.

if godot 4 returns the editor camera when calling get_viewport().get_camera_3d() then it should work in editor too.

I wasn't able to get this to work.
func _calculate_rope_orientation_with_camera() -> void:
	var camera: Camera3D = get_viewport().get_camera_3d()
	if Engine.is_editor_hint():
		camera = EditorInterface.new().get_edited_scene_root().get_parent()

In terms of what can be improved in my current implementation, here are some things.

Being able to select and move the start/end points directly from the editor viewport. Currently you need to navigate through the scene tree and find the one you're looking for. Maybe a system like the bezier curves would be nicer?
Adding forces to a particle or group of particles from script. Imagine explosions affecting the rope.
I wasn't able to get normal mapping working, it would give some depth to the rope.
Better collisions
If performance allows, having a rope with volume would be nice. 4-16 vertices with smooth shading.

Oh thats good. But it looks a bit hacky.

Will evaluate the list.
Here's the video of the current results.
https://user-images.githubusercontent.com/32321/146817364-c1592a36-f516-4760-bbf1-f4334cffd179.mp4

Being able to select and move the start/end points directly from the editor viewport. Currently you need to navigate through the scene tree and find the one you're looking for. Maybe a system like the bezier curves would be nicer?

This is entirely doable. We added a gizmos (plural) system for skeleton transforms on a single node, it should work here too.

Ye the twisting problem really isnt solvable if its being rendered with quads. at some point one quad is going to get pinched. Ropes with volume would solve it but was a little heavy for gdscript to handle.

Want to break this apart into the Godot Engine proposal format?
Being able to select and move the start/end points directly from the editor viewport. Currently you need to navigate through the scene tree and find the one you're looking for. Maybe a system like the bezier curves would be nicer?
Adding forces to a particle or group of particles from script. Imagine explosions affecting the rope.
Get normal mapping working, it would give some depth to the rope.
Better collisions

If performance allows, having a rope with volume would be nice. 4-16 vertices with smooth shading.

Do you want me to port to c++? Are you ok with requiring the dependency. It's unclear if Godot Engine 4 will accept this but I can make the port.
Rigidbody interactions like hanging a ball from the rope. Probably using pin joints?

https://user-images.githubusercontent.com/40386587/146821708-b72cd2c7-3458-4dcb-abd7-d7b340f07a68.mp4
I did a bit of testing if my current implementation can be used with rigidbodies and pin joints, and it doesnt look too bad.

Are you ok with requiring the dependency dependency on what?
By a port to c++ you mean as a module or into godot4? I would prefer if we can get this into godot4.

@2nafish117 Can you write a proposal or point to an existing proposal on godot-proposals?

ive created a propsal. https://github.com/godotengine/godot-proposals/issues/3704
From reading the main readme i get the feeling this fits more into a module rather than core.


Driver not found but installed

Describe the bug
On a mysql connection, I have installed the mysql driver from the vscode marketplace, but, after a reboot, the driver is not recognized.
To Reproduce
Steps to reproduce the behavior:
1 - Install the driver with the new way
2 - Reboot your computer
3 - Try connect to existing mysql connection
4 - The popup message Driver MySQL is not installed. is shown
Expected behavior
The driver should be recognized even after a reboot
Screenshots

Desktop:
Version : 1.45.1
Commit : 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
Date : 2020-05-14T08:27:22.494Z
Electron : 7.2.4
Chrome : 78.0.3904.130
Node.js : 12.8.1
V8 : 7.8.279.23-electron.0
OS : Linux x64 5.6.14-300.fc32.x86_64

I have exact the same issue

Ubuntu 18.04 (updated)

Same thing, v0.22.3 introduces this with postgres even with the driver installed.

For Postgres, I had to roll back both the SQLTools extension and the postgres driver to the version from 4 days ago.

The exact same problem.
The driver does not work.
MySQL, Postgresql, Oracle, SQLite.
He does not see them.
SQLTools 0.22.3
Version: 1.45.1 (user setup)
Commit: 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
Date: 2020-05-14T08: 27: 35.169Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363

Same problem: SQLTools v0.22.3, MysqlDriver 0.0.2
Also MysqlDriver v0.0.1 staill working fine.

Same problem: PostgreSQL/Redshift Driver 0.0.2
SQLTools 0.22.3
Ubuntu 20.04 LTS (tried with vscode installed from both snapcraft and MS store)

@Fmstrat How do I roll back the postgres driver version exactly?

Also Same Problem:

SQLTools v0.22.3
PostgreSQL/Redshift Driver 0.0.2

    Virtualization: vmware
  Operating System: CentOS Linux 7 (Core)
       CPE OS Name: cpe:/o:centos:centos:7
            Kernel: Linux 3.10.0-957.21.3.el7.x86_64
      Architecture: x86-64

@Fmstrat How do I roll back the postgres driver version exactly?

I had to role back the SQLTools extension 0.21.7 works for me.  You can right click on the extension and find the older versions there.

@Fmstrat How do I roll back the postgres driver version exactly?

I had to role back the SQLTools extension 0.21.7 works for me. You can right click on the extension and find the older versions there.

Yes, this works!

It seem to work now, Thanks @mtxr !


Time simulation for user accounts

To make the banking simulator an actual simulator, it needs a function that can calculate the interest return on ALL open accounts with funds greater than 0 within a specified time (in months).
The function will have to be called via the main_menu and will have to provide an extra entry in the info() function for it.

Added functionality to simple_interest_calc() in interest_calcs.py. Just need to code compound interest and add conditional statement(s) to time_simulation(self) in account.py. The conditional should let the user select what type of interest calculation they want to do on their account.

Complete! Added function call in main.py, added the simulation info to help() in general_functions.py.


Weaviate RM should NOT assume that "content" contains the text of the passage

Hi @CShorten
File: dspy.retrieve.weaviate_rm, line 74
Give the flexibility to pass the the name of the text field as a parameter along with the collection name. Something like below:
WeaviateRM(
weaviate_collection_name= "<collection_name>",
weaviate_collection_text_field="",
weaviate_client=weaviate.Client("")
)

Hey Gopi! Thanks for bringing this up, this will resolve it!
https://github.com/stanfordnlp/dspy/pull/360


Use AutoValidateAntiforgeryToken in Templates

From @ardalis on February 20, 2017 18:34
Instead of explicitly using ValidateAntiForgeryToken on every HttpPost method in controllers that accept posts, instead use AutoValidateAntiforgeryToken attribute at controller level (e.g. https://github.com/aspnet/Templates/blob/dev/src/Rules/StarterWeb/IndividualAuth/Controllers/ManageController.cs) or consider adding it globally in Configure when MVC is configured.
Currently [ValidateAntiForgeryToken] appears about 18 times between AccountController and ManageController. This could be reduced down to 2 controller-level attributes or 1 global filter. Personally I would recommend the controller-level attribute, as it keeps the behavior visible to developers working in these controllers, while still demonstrate the better practice of applying the policy broadly, rather than on a one-off basis (which easily be forgotten when the next POST action is added).
Copied from original issue: aspnet/Templates#824

No plans to do this for MVC templates. Razor Pages already does this automatically.


Can not find duckdb headers

Code: 	https://github.com/duckdb/duckdb.git
Open with vscode + remote ssh + clangd extension on remote host, result is:

remote host OS: ubuntu 20.04
cmake version: 3.16.3
clang version: Ubuntu clang version 13.0.1-++20220112083154+b9a243d1cac2-1~exp1~20220112203239.53
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
local OS: macOS Monterey
But it works fine on my local macbook pro.
Thanks for your reply.

Have you built the project and generated a compile_commands.json for clangd? (See https://clangd.llvm.org/installation#project-setup.)
If you've generated a compile_commands.json and it still does not work, please share clangd logs.

@HighCommander4 Thanks for you reply. compile_commands.json is there using cmake command and compile this project is fine. Here is the clangd output
I[15:33:05.304] clangd version 13.0.0 (https://github.com/llvm/llvm-project d7b669b3a30345cfcdb2fde2af6f48aa4b94845d) I[15:33:05.305] Features: linux+grpc I[15:33:05.305] PID: 3802102 I[15:33:05.305] Working directory: /data1/zzb/code/duckdb I[15:33:05.305] argv[0]: /home/zhangzhibiao/.vscode-server/data/User/globalStorage/llvm-vs-code-extensions.vscode-clangd/install/13.0.0/clangd_13.0.0/bin/clangd I[15:33:05.305] Starting LSP over stdin/stdout I[15:33:05.311] <-- initialize(0) I[15:33:05.312] --> reply:initialize(0) 1 ms I[15:33:05.327] <-- initialized I[15:33:05.327] <-- textDocument/didOpen I[15:33:05.328] --> textDocument/publishDiagnostics I[15:33:05.328] Failed to find compilation database for /data1/zzb/code/duckdb/src/include/duckdb/parser/statement/insert_statement.hpp I[15:33:05.328] ASTWorker building file /data1/zzb/code/duckdb/src/include/duckdb/parser/statement/insert_statement.hpp version 1 with command clangd fallback [/data1/zzb/code/duckdb/src/include/duckdb/parser/statement] /usr/bin/clang -resource-dir=/data1/zzb/.vscode-server/data/User/globalStorage/llvm-vs-code-extensions.vscode-clangd/install/13.0.0/clangd_13.0.0/lib/clang/13.0.0 -- /data1/zzb/code/duckdb/src/include/duckdb/parser/statement/insert_statement.hpp I[15:33:05.329] --> textDocument/clangd.fileStatus I[15:33:05.342] --> workspace/semanticTokens/refresh(0) I[15:33:05.342] --> textDocument/clangd.fileStatus I[15:33:05.357] --> textDocument/publishDiagnostics I[15:33:05.357] --> textDocument/clangd.fileStatus I[15:33:05.448] <-- reply(0) I[15:33:05.560] <-- textDocument/documentSymbol(1) I[15:33:05.560] <-- textDocument/codeAction(2) I[15:33:05.560] <-- textDocument/documentLink(3) I[15:33:05.561] --> reply:textDocument/documentSymbol(1) 0 ms I[15:33:05.561] --> reply:textDocument/codeAction(2) 0 ms I[15:33:05.561] --> reply:textDocument/documentLink(3) 0 ms I[15:33:05.561] --> textDocument/clangd.fileStatus I[15:33:05.897] <-- textDocument/documentLink(4) I[15:33:05.897] --> reply:textDocument/documentLink(4) 0 ms I[15:33:05.897] <-- textDocument/documentSymbol(5) I[15:33:05.898] --> reply:textDocument/documentSymbol(5) 0 ms I[15:33:05.898] --> textDocument/clangd.fileStatus I[15:33:05.992] <-- textDocument/codeAction(6) I[15:33:05.992] --> reply:textDocument/codeAction(6) 0 ms I[15:33:05.992] --> textDocument/clangd.fileStatus I[15:33:06.422] <-- $/setTrace I[15:33:06.422] unhandled notification $/setTrace I[15:33:06.422] <-- $/setTrace I[15:33:06.422] unhandled notification $/setTrace I[15:33:07.211] <-- textDocument/codeAction(7) I[15:33:07.211] <-- $/cancelRequest I[15:33:07.211] --> reply:textDocument/codeAction(7) 0 ms, error: Task was cancelled. I[15:33:07.211] --> textDocument/clangd.fileStatus I[15:33:07.211] <-- textDocument/codeAction(8) I[15:33:07.211] --> reply:textDocument/codeAction(8) 0 ms I[15:33:07.211] --> textDocument/clangd.fileStatus [Error - 3:33:07 PM] Request textDocument/codeAction failed. [object Object] I[15:33:08.151] <-- textDocument/codeAction(9) I[15:33:08.151] <-- $/cancelRequest I[15:33:08.151] --> reply:textDocument/codeAction(9) 0 ms I[15:33:08.151] --> textDocument/clangd.fileStatus I[15:33:08.151] <-- textDocument/codeAction(10) I[15:33:08.151] --> reply:textDocument/codeAction(10) 0 ms I[15:33:08.151] --> textDocument/clangd.fileStatus I[15:33:08.151] <-- $/cancelRequest I[15:33:08.151] <-- textDocument/codeAction(11) I[15:33:08.151] --> reply:textDocument/codeAction(11) 0 ms I[15:33:08.151] --> textDocument/clangd.fileStatus

The log shows:

Failed to find compilation database for /data1/zzb/code/duckdb/src/include/duckdb/parser/statement/insert_statement.hpp

What is the full path of the compile_commands.json file?

compile_commands

@HighCommander4 Thanks for your reply. Full path is
/data1/zzb/code/duckdb/build/compile_commands.json

@zhanglistar Can you try symlinking it into /data1/zzb/code/duckdb and see it that helps?

@zhanglistar Can you try symlinking it into /data1/zzb/code/duckdb and see it that helps?

@HighCommander4 It does no help.

Still got the error:
I[18:24:39.543] <-- textDocument/documentSymbol(45) I[18:24:39.543] Failed to find compilation database for /data1/zzb/code/duckdb/src/include/duckdb/parser/statement/extension_statement.hpp I[18:24:39.544] ASTWorker building file /data1/zzb/code/duckdb/src/include/duckdb/parser/statement/extension_statement.hpp version 1 with command clangd fallback [/data1/zzb/code/duckdb/src/include/duckdb/parser/statement] /usr/bin/clang -resource-dir=/data1/zzb/.vscode-server/data/User/globalStorage/llvm-vs-code-extensions.vscode-clangd/install/13.0.0/clangd_13.0.0/lib/clang/13.0.0 -- /data1/zzb/code/duckdb/src/include/duckdb/parser/statement/extension_statement.hpp I[18:24:39.545] --> textDocument/clangd.fileStatus I[18:24:39.553] --> workspace/semanticTokens/refresh(1) I[18:24:39.553] --> textDocument/clangd.fileStatus

Very strange. Clangd looks for the compile_commands.json in the ancestor directories of the file being opened, so if you're opening /data1/zzb/code/duckdb/src/include/duckdb/parser/statement/extension_statement.hpp, it should look in /data1/zzb/code/duckdb/src/include/duckdb/parser/statement/ and then /data1/zzb/code/duckdb/src/include/duckdb/parser/ and so on and eventually in /data1/zzb/code/duckdb/. I'm not really sure what goes wrong in this case.

Maybe try using the --compile-commands-dir command line argument to clangd, as discussed in this FAQ entry?

@HighCommander4 Thanks for you reply, so how do I pass the argument of --compile-commands-dir to clangd?

In vscode settings, something like:
{
  "clangd.arguments": [
    "--compile-commands-dir=/data1/zzb/code/duckdb/build"
  ]
}

@HighCommander4 This works! So where is wrong?

@HighCommander4 This works!

Yay!

So where is wrong?

I'm not sure, sorry. Perhaps someone else can offer a hypothesis or suggest a way to investigate the original setup further.

In vscode settings, something like:
{
  "clangd.arguments": [
    "--compile-commands-dir=/data1/zzb/code/duckdb/build"
  ]
}

Thanks. It works well in wsl.

May it caused by following reason?

clangd will look in the parent directories of the files you edit looking for it, and also in subdirectories named build/. For example, if editing $SRC/gui/window.cpp, we search in $SRC/gui/, $SRC/gui/build/, $SRC/, $SRC/build/, …

ref

Because duckdb sets the compile_commands.json path, just change it.

or use make clangd to generate a compile_commands.json file at build/clangd


[UX][Store] Production keys access token title shows as "Generate a Test Access Token"

Description:
Not fulfilling checklist items:
Aesthetic and minimalist design - Is the system contains information which is relevant and needed? (no irrelevant or rarely needed info)
Related task:
Subscribe to an application
Issue & Solution:
Key generated view of Production keys shows a title as "Generate a Test Access Token". Since this key is used for production the title needs to rename to "Generate a Access Token".

Suggested Labels:
UX, Improvement
Affected Product Version:
2.1.0

In here the word Production has used to indicate the API endpoint that the request will be routed. That doesn't mean that you have to generate a production access token from here to use in production and generate a sandbox key otherwise.
The generate key functionality is provided here to test API using either production or sandbox access tokens.
So IMO, the usage of the word Test in the context of Generate a Test Access Token is valid here.
@chanaka3d , @tgtshanika , @bhathiya any comment on this ? Because we have implemented the test key UI in 3.x with the same notion

Yes, the key generated here is not supposed to be used in real production. It's used to "Test" the production endpoint.


Explain how to set webhook URL

Added a new step explaining how to set the webhook URL in the Twilio dashboard
Proposed changes:

...

Status (please check what you already did):

[ ] added some tests for the functionality
[x] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@erohmensing Could you please review? thanks!

Thanks for submitting a pull request 🚀 @rgstephens will take a look at it as soon as possible ✨

Oy vey rebasing a fork is impossible, we'll merge this to master and cherry-pick the commits


Should change the way "version" is detected

This works, as is, but I think we could design the "version check" stuff better. I'd like to make it so that the version is detected once at config time, then maybe during "reload" ? Then we could switch the "checks" to just compare the version from config.
Originally posted by @TJM in https://github.com/jfrog/artifactory-secrets-plugin/pull/45#pullrequestreview-1329619107

@alexhung - I agree. @danielmkn and I are about to do something similar in the Terraform provider and stores the version in the context.

@alexhung 19 hours ago
You probably can add a field to the Backend and then call InitializeFunc to fetch the version, then store it in the new field.

Fixed in #47


Frames do not write, display window absent.

Hi Ahmet. I ran your code straight out the box on ubuntu 18.04 with all necessary prerequisite installed, it displayed this:
Writing frame 1 / 7717
Writing frame 2 / 7717
Writing frame 3 / 7717
Writing frame 4 / 7717
Writing frame 5 / 7717
Writing frame 6 / 7717
Writing frame 7 / 7717
Writing frame 8 / 7717
Writing frame 9 / 7717
Writing frame 10 / 7717
Writing frame 11 / 7717
Writing frame 12 / 7717

but infact did not write the frames to system. No display windows showed without bounding box around faces either.

fixed, please check it again..


Ripple effect suggestion

Hi is it has any prop for ripple effect ?

No prop, the ripple effect would be the inner layout of a cardview typically. You could apply a ripple drawable to the cardview but currently it's not supported with this plugin.


80 Meter Dial Freq

Currently the 80 meter dial freq is 3568600 Hz and FSKview ver 1.0.5 shows a dial freq of 3592600 Hz.

Hi @larryiweaver thanks for pointing that out! I drew my WSPR frequencies from those listed on http://wsprnet.org/drupal/node/218 - Do you have a suggestion for an "official" source of WSPR frequencies I can refer to that may reflect the most updated and accurate list?
Note to self: WSPR band frequencies are defined here
https://github.com/swharden/FSKview/blob/4f0a6e23afc57bf618d5d84314f41ca11f43dfaa/src/FSKview/WsprBands.cs#L17-L29

... answering my own question, I found a list of frequencies on the front page of http://wsprnet.org/ so I'll make this change now. Thanks again for reporting this issue @larryiweaver!


add intial release branch ci

Notion ticket link

Ticket Name

Implementation description

Work in progress
CI portion of production infrastructure

Steps to test

What should reviewers focus on?

Checklist

[ ] My PR name is descriptive and in imperative tense
[ ] I have run the linter
[ ] I have requested a review from the PL, and/or other devs who have background knowledge on this PR or who will be building on top of this PR

ima put this to draft until this is ready for review


2120: fix issue with umount arg list too long on uninstall

Signed-off-by: Patrick Force<EMAIL_ADDRESS>Proposed changes

Fixing issue where the install.sh script umount calls end up with an arg list that's too long. Just split up the calls.

Types of changes

Bugfix

Verification
Testing the do_unmount method continues to operate as expected in all cases. The updated function(s) were tested in isolation to make sure they still work correctly.
Linked Issues

https://github.com/rancher/k3s/issues/2120

This should be a oneline change; replace
umount ${MOUNTS}

with
xargs -n 1 umount <<< ${MOUNTS}

@brandond updated per the request