Patch Release: V6.1.1

Added

Adds automated release notes

Fixed

filename can be used again with ASFProduct.Download() method (ignored if multiple files are to be downloaded)

Codecov Report

Merging #214 (09c75aa) into stable (a7406b0) will increase coverage by 1.10%.
The diff coverage is 100.00%.

:exclamation: Current head 09c75aa differs from pull request most recent head 330efdd. Consider uploading reports for the commit 330efdd to get more accurate results

@@            Coverage Diff             @@
##           stable     #214      +/-   ##
==========================================
+ Coverage   88.76%   89.86%   +1.10%
==========================================
  Files          52       52
  Lines        1958     1964       +6
==========================================
+ Hits         1738     1765      +27
+ Misses        220      199      -21

Flag
Coverage Δ

unittests
89.86% <100.00%> (+1.10%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

asf_search/ASFProduct.py
94.11% <100.00%> (+34.44%)
:arrow_up:


Code | Symbol Pattern 61

I would like to work on this as part of SWOC.
Could you please assign it to me?

Do follow the naming convention for your code files and take care of all the standards that need to follow.
Nmaing convention is: symbolpattern61.py
While making PR, tage the issues as follows:
fixes #issuenumber


[CI] update travis distrbution

trusty is too old to use newer software stack(cmake, gcc)
let's upgrade to binoc
Signed-off-by: Yuan Zhou<EMAIL_ADDRESS>What changes were proposed in this pull request?
update travis to use bionic so we could use newer cmake and gcc
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
travis based unit testing

@zhixingheyi-tian @winningsix  need to use newer cmake/gcc for the native-sql project. Please help to take a look if this would make trouble for you

cc @jikunshang , @xieqi

looks like bionic will run to failures, let's check if xenial works

@zhouyuan
You can also update os version in travis matrix job confs.
Thanks

@zhixingheyi-tian thanks, I'll update the native-sql job then.


Stars visible in master flats

Stars appear visible in some master flats. Not good!

Looking at the LSST flat field code:
https://github.com/lsst/pipe_drivers/blob/d00fb1e517e6e177fc9fbe9285921bb8515ad3ae/python/lsst/pipe/drivers/constructCalibs.py#L644
It looks like it calls only ISRTask, which performs the steps outlined here:
https://github.com/lsst/ip_isr/blob/535a0ce822ab4f2679c4e32ff5a3617ff778b991/python/lsst/ip/isr/isrTask.py#L1229
So there is no explicit source masking. There is a default sigma-clipping algorithm applied during combination:
https://github.com/lsst/pipe_drivers/blob/d00fb1e517e6e177fc9fbe9285921bb8515ad3ae/python/lsst/pipe/drivers/constructCalibs.py#L91
But that doesn’t seem to be working as intended. Suggested solution is to modify FlatTask.processSingle to include a multi-scale filter masking procedure.

Suggestion for implementing multi scale filtering is to use MaskObjectsTask several times with different smoothing lengths:
https://github.com/lsst/pipe_drivers/blob/d00fb1e517e6e177fc9fbe9285921bb8515ad3ae/python/lsst/pipe/drivers/background.py#L770


Add "flags: {}" to default Elm.Main.init in index.html

TLDR:  when elm make src/main.elm generates index.html, have it add in a default flags:{} argument.  Then users who wish to use Browser.Document to make a single page with a title can do so by handling an empty flags arg in their init function.  If the user chooses Browser.sandbox, the extra flags arg is just ignored.
Motivation:
I want to make a simple self contained one page app - all code in index.html.  I'd like to set the title, so that means using Browser.Document.
The problem is that document requires an init function that takes a flag: argument.  Index.html does not supply this.
I have the option of editing in an argument in the generated index.html after every compile, or going to a separate index.html and main.js, which I don't want to do for this no-server page.

This is not needed, just use () as your flags type.
For example:
module Main exposing (main)

import Browser
import Html exposing (Html, text)

type alias Model =
    {}

main : Program () Model msg
main =
    Browser.document
        { init = always ( {}, Cmd.none )
        , subscriptions = always Sub.none
        , update = \msg model -> ( model, Cmd.none )
        , view = always { title = "title", body = [ text "hello" ] }
        }

You can compile it to index.html and test it.

I agree that it would be nice if it was explicitly described in the flags section for example.


Leaflet 0.7 - Added autoSpiderfyOnMaxZoom and preventUnspiderfyOnClick.

Additional options for preventing the spiderfied clusters to close.

Sorry I don't want to add new features in to the 0.7 version.
If they went to the master version first, then we could consider adding them to the 0.7 version (as a backport)


IphoneX FastImage performance

Describe the bug
FastImage component causes little performance drops (transitions is not smooth enough).
I have 2 screens with one image on each. When i perform transition i see some transition freezes (not critical). But when i remove FastImage from scene - transitions became butter smooth.
what is really interesting - this drop occurs only on > IphoneX devices. Maybe it's some fs interactions issue, idk.
My images is optimized and minified.
To Reproduce
Steps to reproduce the behavior if possible, or a link to a reproduction repo:

Create two scenes (i use react-navigation)
Add Fastimage on each scene
Perform transition
See micro transition freezes

Expected behavior
Clear and butter smooth transition
Dependency versions

React Native version: ^0.59.10
React version: ^16.8.6
React Native Fast Image version:^6.0.2

experiencing the same right here on an iPhone 11 Pro. Default <Image> component from react-native performs way better.
The <FastImage> component looks a little pixelated, and re-renders these pixels on transitions instead of just smoothing the image like the <Image> component. In other words; The <Image> component looks slightly blurred instead of high-detail-pixelated, and I think that's what's causing the performance drops. Hopefully that explanation was understandable..
Anyone knows why this is only happening on iPhone X and above?

Since this issue is really old, I'd like to bump this.
Noticeable performance lags still occur on
"react-native-fast-image": "^8.1.5",
"react-native": "0.62.2",
"@react-navigation/native": "^5.4.0",
"@react-navigation/stack": "^5.3.7",

Hi, @mrousavy since there is no answer on this issue, did you find a good alternative to cache images on React Native ?
On iOS with a large flatlist and the original  component I don't have good results with cache:"force-cache" :-/

@manuhook did you actually compare the native  component to the  component? Was FastImage noticeably slower for you?

@mrousavy yes same transition issue as described here, however it's seems to perform better when using react-native-navigation

@manuhook I'd love to use react-native-navigation as well, awesome work from the guys at wix. Unfortunately I've got the Apple's app of the day shared element transition in my app, which I can't replicate with the current capabilities of react-native-navigation yet :/
Also to answer your question; No, I didn't find a better solution to this. You can either use <FastImage> or <Image> with force-cache.

The performance on ios is terrible, only helps on Android. On ios you may want to use the built-in Image component from React Native.


Task/create estudio socioeconomico

What's this PR do?
It connects the button on the interface to the creation of a study.
Where should the reviewer start?
In captura/views.py
How should this be manually tested?
Log in as a capturista, click the crear nuevo estudio button, and check that it appears on the screen,
Any background context you want to provide?
When this pull request gets accepted, the rest of the
This template was adapted ~stolen~ from Quickleft/Sprint.ly

Codecov Report

Merging #46 into develop will increase coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop      #46     +/-   ##
==========================================
+ Coverage    96.36%   96.46%   +0.1%
==========================================
  Files           91       94      +3
  Lines         1649     1699     +50
==========================================
+ Hits          1589     1639     +50
  Misses          60       60

Impacted Files
Coverage Δ

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

...ioeconomicos/migrations/0007_auto_20170314_0445.py
100% <100%> (ø)

familias/models.py
94.11% <100%> (ø)
:white_check_mark:

estudios_socioeconomicos/models.py
100% <100%> (ø)
:white_check_mark:

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

familias/migrations/0009_auto_20170314_0445.py
100% <100%> (ø)

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

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 119a859...1b6f72b. Read the comment docs.


Priority in the Queue

Is there any options where I could set a priority in the queues. Sometime I have a large queue and I need to process one item fast and it gets in the bootle of the queue. Would like to know if such an option is possible where it would to go the top and be next.

@getuliojr no, this is not possible. We consider queues an exceptional situation needed only in case when cluster is temporarily overloaded. In normal operation mode there should be no queues.

If you think selenoid running test cases I imagine that. But when it is used to crawl data in internet and other systems your start to take cases where one item in queue can take from 30seconds to 10 minutes processing time and with that in mind priority in queues make sense. Thats is our case.

I will try with GGR and quotas for user. That might work as an alternative. Thanks.

@getuliojr JFYI Ggr distributes requests randomly over a set of hosts and also has no queues. So it does not prevent the cluster from overloading.


[Question] Question about bash scripts and custom tap

My team has multiple public and private tools that we install using a custom homebrew tap. I was hoping to replicate these on scoop to enable Windows users to also use those same tools. The tools are written in either bash or python.
I had two questions:

Can I use scoop to install custom public or private tools, similar to a custom private homebrew tap? Or does scoop only work with some global list of packages?

Can scoop allow me to run a tool written using bash with the regular cmd.exe or PowerShell? For example, my team uses this tool (https://github.com/AlJohri/kar) but the core of it is written in bash. Can I create a scoop formula/package, that will allow the user to use this command line tool in PowerShell or cmd.exe completely transparent to them which language it is written in?

Thanks! Apologies if there's any silly questions here- I'm trying to re-learn windows.

Can I use scoop to install custom public or private tools, similar to a custom private homebrew tap? Or does scoop only work with some global list of packages?

Sure, you can either create your own bucket or specify a path to a local or remote JSON manifest and have it installed directly:
# A bucket is a Git repository with JSON manifests in its root or in a `bucket/` subdirectory.
scoop bucket add <bucket URL>
scoop install <package>

# Or...
scoop install <URL or local file path>

Thank you so much for the detailed response! I will give this a shot.


Fix error parsing query string twice

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/82b409dbc333eafb064a8ae5417a2286e18bed92/
Edited pages:

search.html

Hello! Thank you for contributing to CockroachDB this year. As a token of our appreciation, we would like to send you a gift. Please DM me on our community Slack @Amruta so I can send you a link. (If you don’t want a gift, we also have a charitable contribution choice.) All orders need to be in by December 13, so please contact me as soon as possible :)

@mbesada LAST DAY TO ORDER!!!
Hi Martin. Thank you for contributing to CockroachDB this year. As a token of our appreciation we would like to send you a gift. We tried to reach you via GitHub, but if you didn’t see the message please DM me on our community Slack @lisa so I can send you a link to order your gift. (If you don’t want a gift, we also have a charitable contribution choice.) All orders must be in by December 13 (that’s tomorrow!), so please send this asap. :)

A little late I just saw the message 2 years later LOL!


Update launchpad page to use holesky instead of goerli

Goerli has been shutdown and therefore made our docs outdated. Creating this issue so we update the examples and links to point to the holesky testnet that has replaced goerli.

@m4sterbunny, I believe this is the page that needs to be updated: https://docs.web3signer.consensys.io/development/tutorials/load-launchpad-keystores

https://github.com/Consensys/doc.web3signer/pull/267/

@gfukushima pls confirm whether scope is just current docs or backward through versioned docs

@m4sterbunny looked at your PR, I'd suggest you replace sepolia by holesky as @joaniefromtheblock has suggested. Other than that it lgtm.


Add link to the Kubescape survey

Until the Message of the Day work is complete, we should display a link to our project survey at the bottom of every time through the kubescape CLI.

Please fill a 3 question survey to help the Kubescape project! https://kubescape.io/project/survey/

I suggest you use the following logic:

if connected to a provider, display the provider text
else, display the survey link

Can I work on this?

Are we supposed to put the link somewhere around here?

Hi @prady0t Craid said instead of the provider text, so it should replace :
View results
────────────
Now, take your scan results to the next level with actionable insights on ARMO Platform.

Sign up for free here - https://cloud.armosec.io/repository-scanning/74ec6f9c-f2f2-47fa-b755-6b88e8a1289a

Hey @matthyx. Can you refer to areas in the codebase from where the provider text is coming from. I found out this function https://github.com/kubescape/kubescape/blob/9e6efd04ce12bd2147d3719b18e5766e2eb2ad2f/core/pkg/resultshandling/reporter/v2/reporteventreceiver.go#L269
Is responsible for printing the provider text.

https://github.com/kubescape/kubescape/blob/master/core/core/initutils.go#L75

@matthyx Can I work on this?

sure, thanks!

Looks like the PR to close this wasn't actually merged?

Looks like the PR to close this wasn't actually merged?

@slashben said we no longer need this, do you have a different opinion?

not one as important as that, I just wanted to check it wasn't a flaw of automation


[Bug] [SuperUrlBar] Latest Zen Update broke Blur for non-internal pages

Affected Theme

Name: SuperUrlBar
Version: v1.4.2

Affected Theme Settings

Enable a Background Blur when the Url Bar is focused & select its Intensity

Zen Version: 1.0.2-b.0 (Firefox 133.0)
System OS: Both macOS and Linux
Describe the bug
When the URL Bar is focused and active, blur does not apply to non-internal pages anymore.
To Reproduce
Steps to reproduce the behavior:

Install and Enable SuperUrlBar
Enable Enable a Background Blur when the Url Bar is focused & select its Intensity
While in Zen Settings, activate the URL bar and start typing. Notice that the background is correctly blurred
Open a new tab and go to Google.com

Expected behavior
Background is correctly blurred like it is on the browser-internal page.

Screenshots/Videos
Background is not blurred.

Additional context
This was working on a previous version. I noticed it break on macOS first last week, and now on my Linux machine today.
It also appears to be off-center, but I didn't check to see if there was an existing bug for that yet.

I hadn't realized it, but I had the now-deprecated official Floating Url Bar extension installed and enabled at the same time. That was the issue :grimacing:

Just kidding. It seems to be intermittent... It's very odd.

Yeah, it's a long onging issue that's related to "clip-path" (used to create the rounded browser corners). The problem with clip-path is that it uses some performance (should be unnoticable though) so they tried to find ways to create the corners without clip-path and suceeded, but now it seems they're kinda using sth like that again so blur isn't working outside browser internal pages.
If you know how to edit your userChrome.css paste the following code to fix it:
  /* Scuffed af solution to make blur work (need to find better solution) */
  :root:not([inDOMFullscreen='true']):not([chromehidden~='location']):not([chromehidden~='toolbar']) {
    clip-path: content-box;
  }

I probably won't update this officially soon since the SuperUrlBar Mod needs an "Overhaul" after the beta update but i'm rather busy at the time and not really motivated, sry


add encrypt route

First time even touching go. Implemented encryption of PDFs for qpdf and pdfcpu. Pdftk also supports encrypting, but behaviour was weird, so i dropped it for now.
Api description was extended, but no tests so far, because i have no idea how they work in go.
Please give feedback :)

Codecov Report

Merging #496 (094fc8e) into main (d8f3f85) will decrease coverage by 2.51%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##             main     #496      +/-   ##
==========================================
- Coverage   93.77%   91.25%   -2.52%
==========================================
  Files          35       36       +1
  Lines        3050     3203     +153
==========================================
+ Hits         2860     2923      +63
- Misses        121      204      +83
- Partials       69       76       +7

Impacted Files
Coverage Δ

pkg/gotenberg/mocks.go
85.71% <0.00%> (-14.29%)
:arrow_down:

pkg/modules/libreoffice/pdfengine/pdfengine.go
92.85% <0.00%> (-7.15%)
:arrow_down:

pkg/modules/pdfcpu/pdfcpu.go
66.66% <0.00%> (-33.34%)
:arrow_down:

pkg/modules/pdfengines/multi.go
68.75% <0.00%> (-31.25%)
:arrow_down:

pkg/modules/pdftk/pdftk.go
91.48% <0.00%> (-4.07%)
:arrow_down:

pkg/modules/qpdf/qpdf.go
69.23% <0.00%> (-26.52%)
:arrow_down:

pkg/modules/pdfengines/routes.go
75.47% <16.12%> (-24.53%)
:arrow_down:

pkg/modules/chromium/chromium.go
91.92% <75.75%> (-1.11%)
:arrow_down:

pkg/modules/chromium/stream.go
79.16% <79.16%> (ø)

pkg/modules/pdfengines/pdfengines.go
97.50% <100.00%> (+0.03%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

regarding the routes: is it possible (or a good idea) to pass the multiPDFEngines to the chrome/libre routes? not sure atm how to access the encrypt parts from within the chrome/libre routes

Both the Chromium and LibreOffice modules have a PDFEngine:

https://github.com/gotenberg/gotenberg/blob/main/pkg/modules/chromium/chromium.go#L66
https://github.com/gotenberg/gotenberg/blob/main/pkg/modules/libreoffice/libreoffice.go#L20

For the Chromium module, you should update the convertURL method (it is used by other methods): https://github.com/gotenberg/gotenberg/blob/main/pkg/modules/chromium/routes.go#L327.
For the LibreOffice module, you should update the convertRoute method https://github.com/gotenberg/gotenberg/blob/main/pkg/modules/libreoffice/routes.go#L16.
Also, it seems the following packages do not have all required tests (I suspect most of them being "stupid" test but still):

So i added encryption to every route that handles PDFs (all chromium, libreoffice and pdfengine). Restructured the openapi.yaml a little bit, so it's easier to maintain. Tested all routes locally. I hope i got all the tests, but it's hard to tell them all apart :D

any feedback?

Hello @vidschofelix,
Sorry for the delay, but I don’t have enough time to properly review it.
There are many implementations available, so finding the correct interface for all modules is not straightforward.
Also, having an encrypt method will trigger a need for a decrypt route.
I keep this PR open until I find some time to work on it.

Ok, thank you :).

It's a very useful feature, I hope I can use it soon.


blast repo changes: auto-publish, github-actions, no-response

This PR contains changes created by the blast repo tool.

auto-publish
github-actions
no-response

coverage: 98.642%. remained the same when pulling 6a185359147f85bcbc1d6546b09b927ab4e390e0 on blast_repo-2023_07_11T04_26_36 into de1b0ccf24a89315fe76116090d9fe1b8bd774bb on main.

The bulk of this PR is adding publishing automation. The packages here still have to be published once manually before the automation is useful (and some toggles configured in the pub admin page for each package).

Thanks @devoncarew !


sentry: Ignore DisallowedHost messages.

Testing Plan: Set up ALLOWED__HOSTS locally to not be *, enabled Sentry, and set a REALM_HOSTS so that we could check the realm bits.  Bad realms got 404's, bad hosts didn't get to Sentry.

CI failures look to be for tests of the DisallowedHost code path.

Just noticed you pushed here; is this ready for a review?

Yup!

Merged as the series ending with 28c627452fbcc5366bdeb410dec2470881576959 after adding a comment in the second commit (and resolving some easy merge conflicts).  Thanks @alexmv!
I think a natural follow-up would be to replace get_realm_from_request with just accessing request.realm; and perhaps more generally audit use of get_subdomain for whether we could just be accessing request.realm set by this middleware instead?  I think that could present a cleanup of basically of those references since they're largely in non-API contexts (in authenticated API context, one can just access user_profile.realm).


Get KeyVault name for sp_check from matrix data

Context
KeyVault name is stored in GitHub secret (which hasn't been set in a number of repos.  Reducing GitHub secrets reduces management overhead and makes this workflow easier to generalise.
Changes proposed in this pull request

Use matrix data to store KeyVault
Pinned major version of rtCamp/action-slack-notify

Guidance to review
The same changes were made to find-a-lost-trn and tested here
Checklist

[x] Attach to Trello card
[x] Rebased main
[x] Cleaned commit history
[ ] Tested by running locally

 Check Service Principal Key Vault Values


No space before :sup: in the reStructeredText output

Hello, I use vancouver-superscript.csl when converting from orgmode to rst. The reference list and entries look great. However, I obtain citations in my rst output such as this one:
Blah blah.:sup:1
Whereas for rst editors and programs to understand it correctly, there needs to be a space before the first colon. So, it should be:
Blah blah. :sup:1
I was wondering if I am missing something or if there can be anything done about this.
Thanks in advance,
Steve

I believe it's a pandoc issue, not a pandoc-citeproc issue.  But
first, what version of pandoc are you using?  Recent
versions seem to insert the necessary \  before the
:sup:.
+++ stvharman [Aug 12 15 19:07 ]:

Hello, I use vancouver-superscript.csl when converting from orgmode to
rst. The reference list and entries look great. However, I obtain
citations in my rst output such as this one:
Blah blah.:sup:1
Whereas for rst editors and programs to understand it correctly, there
needs to be a space before the first colon. So, it should be:
Blah blah. :sup:1
I was wondering if I am missing something or if there can be anything
done about this.
Thanks in advance,
Steve
—
Reply to this email directly or [1]view it on GitHub.
References

https://github.com/jgm/pandoc-citeproc/issues/157

Thank you, John. Here are my versions:
$ pandoc --version
pandoc <IP_ADDRESS>
$ pandoc-citeproc --version
pandoc-citeproc 0.7.2

Can you give me something sufficient to reproduce the
problem?  A small Markdown sample, bibliography, and
the stylesheet you're using (or its name if it's from
the standard csl repository).
+++ stvharman [Aug 13 15 10:42 ]:

Thank you, John. Here are my versions:
$ pandoc --version
pandoc <IP_ADDRESS>
$ pandoc-citeproc --version
pandoc-citeproc 0.7.2

Reply to this email directly or view it on GitHub:
https://github.com/jgm/pandoc-citeproc/issues/157#issuecomment-130776041

Sure. Try this from the command line:
pandoc --filter pandoc-citeproc --bibliography  mybib.bib --csl vancouver-superscript.csl   test.org -s -o test.rst

You can download mybib.bib from here http://shelah.logic.at/mybib.bib  (or really use any bib file)
The content of test.org can be only one line:
#+BEGIN_SRC
Here is my reference [@small]
#+END_SRC
Then it will produce the following test.rst
#+BEGIN_SRC
Here is my reference:sup:1
.. raw:: html

.. raw:: html

Freely I. A small paper. The journal of small papers. 1997;-1.

.. raw:: html

.. raw:: html

#+END_SRC

Verified. It works. Thanks John.


Fix redirect issue in password_expired_controller

The code was doing the opposite of what it was intended to do.
It was redirecting to root when the resource is present and when the
resource needs a password change.
Also, this caused redirect to be called multiple times in the same
action because calling redirect_to :root will not terminate
execution of the action. It will continue to execute the rest of the
code even if the original condition returns true. To stop execution,
you have to add and return or use a guard clause, as I did here.
I also added tests for the update action, and configured the dummy
Rails app to support the tests.

:+1:  LGTM

I'm closing this PR because it looks like a fix was already made. However, that fix is still not fully tested, so I'll submit a new PR with more tests.


pkg/bpf: Use the other directory when /sys/fs/bpf is not BPFFS

When Cilium is running inside container, it has /sys/fs/bpf
directory mounted from the host. However, if BPPFS is not mounted
on the host, then containerized Cilium should be aware of that and
try to mount BPPFS in the other directory - /run/cilium/bpffs.
Fixes #3717
Signed-off-by: Michal Rostecki<EMAIL_ADDRESS>Remount /sys/fs/bpf if its filesystem is not BPFFS

This change is

test-me-please

test-me-please

test-me-please

@tgraf @aanm @joestringer It's ready for review. It works both on Vagrant (as a systemd servise running on host, which uses /sys/fs/bpf mount) and on minikube (where /sys/fs/bpf/ is an empty mount and Cilium fallbacks successfully to /run/cilium/bpffs and makes the mount by itself).
Logs from Vagrant:
https://paste.opensuse.org/19728735
Logs from minikube:
https://paste.opensuse.org/60298312

test-me-please

tc has an option to pass the mount point via env, but it also is able to auto-detect a BPF fs mount from /proc/mounts; just making sure the latter works fine in this setting. What happens if you have, say, another BPF fs instance in some different location like /tmp/foo. Are the maps/progs still populated at the right location under /run/cilium/bpffs when /tmp/foo is listed first or last?

That's a good question, I didn't check that. But I think that any autodetection mechanism which exists outside of pkg/bpf/bpffs.go file can be harmful and every map should rely on mapRoot variable.
I will try to dig into tc code. But if you already know better how it works, can you point me to the code which autodetects BPFFS mount for tc? Thanks a lot!

@borkmann That's a good question, but I think to ensure that everything will be working well, I will just pass the proper mapRoot via env variable to tc.

@borkmann I added the BPF_ENV_MNT variable for init.sh and join_ep.sh scripts. I skipped the run_probes.sh script, because it's executed before BPFFS mount. I also skipped scripts from test/ directory.

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Percentage/count of accidents per {gender, age, time of day, day of week, month, severity, injury type, road user type, vehicle type} relative to other geo buckets

Description:

Create the data query
Create the data plot
Drill down to account for statistical methodology.

normalization
significance
deeper analysis to understand data source with respect to visualization type
etc.

this will be done as part of use case investigations


Backend on IBMQ devices

Although it is possible to run PennyLane program on IBMQ public version, it seems currently not able to run on IBMQ services that are exclusively to institution subscriptions!
According to the attached link, when we want to access the institute-exclusive IBMQ services, it is not only token but also URL are needed to login. Is it possible to extend this capability to the plugin?
ref:
https://qiskit.org/documentation/install.html#access-ibm-q-systems

Hi, are you using this plugin: https://github.com/carstenblank/pennylane-qiskit? It needs some updates and I am looking into it.
They have instructions on how to use your token to run the institute exclusive services:

Tests on the IBMQ device can only be run if a ibmqx_token for the IBM Q experience is configured in the PennyLane configuration file. If this is the case, running make test also executes tests on the ibmq device. By default tests on the ibmq device run with ibmq_qasm_simulator backend and those done by the basicaer and aer device are run with the qasm_simulator backend. At the time of writing this means that the test are "free". Please verify that this is also the case for your account.

You can see the documentation here:
https://pennylane-qiskit.readthedocs.io/en/latest/gettingstarted.html
Here is a code snippet
import pennylane as qml
dev = qml.device('qiskit.ibmq', wires=2, ibmqx_token="XXX")

Please let me know if this works as I am trying to update the plugin.

We could have something like:
dev = qml.device('qiskit.ibmq', wires=2, ibmqx_token="XXX", url="xyz.ibm")

and make a check in the init for the device being "qiskit.ibmq".

From IBMQ document, the institute exclusive service should login in the following format:
from qiskit import IBMQ IBMQ.save_account('MY_API_TOKEN', 'MY_URL')
it seems there is currently no fields in pennylane-qiskit to fill the "MY_URL".

Thanks @ycchen1989 for the report of this missing functionality. We will work with IBM to get the appropriate information necessary to remedy it as soon as possible.

Hi @ycchen1989. Apologies, I'm not so familiar with the IBMQ institutional access. I had a look on the link you posted, and it looks like once you save the token and URL in your Qiskit configuration, it should be automatically used the next time Qiskit has loaded:

Just confirming that you have followed the steps above, and then used PennyLane-qiskit to run an IBMQ device? Does this still not work (and if not, what error message do you get?)

Also, I think you have to enable the account:
IBMQ.enable_account("token")

Right now I am facing some issue with connecting to the server even though I have accepted the license online.
ConnectionError: Couldn't connect to IBMQ server: error during login: License required. You need to accept the License from Web Platform. https://quantumexperience.ng.bluemix.net

It seems after following these steps, one can use the new URL login API.

Install qiskit and run the followig code (before runnign the pennylane plugin) to enable your token and url on the machine (I still do not know where IBM stores these).

from qiskit import IBMQ
IBMQ.save_account(token, url)
IBMQ.enable_account(token)

Go to the folder from which you want to run pennylane-qiskit code and set the path variable IBMQX_TOKEN as,

export IBMQX_TOKEN=token

This should make the pennylane-qiskit plugin work. I am able to successfully connect and run the basic tests.
If you get some error like me regarding the license. Then goto: https://quantumexperience.ng.bluemix.net and accept their license agreement.
I hope this solves this issue, @ycchen1989 please let us know if this helps and I can add this bit to the documentation of the pennylane qiskit plugin.

It seems after following these steps, one can use the new URL login API.
1. Install qiskit and run the followig code (before running the pennylane plugin) to enable your token and url on the local machine (I still do not know where qiskit stores these).

from qiskit import IBMQ
IBMQ.save_account(token, url)
IBMQ.enable_account(token)

1. Go to the folder from which you want to run pennylane-qiskit code and set the path variable IBMQX_TOKEN as,

export IBMQX_TOKEN=token

This should make the pennylane-qiskit plugin work. I am able to successfully connect and run the basic tests.
If you get some error like me regarding the license. Then goto https://quantumexperience.ng.bluemix.net and accept their license agreement.
I hope this solves this issue, @ycchen1989 please let us know if this helps and I can add this bit to the documentation of the pennylane qiskit plugin.

I just tried this. However, when I ran the first part to save the account, it show the license issue which should not be because I already clicked such things in the IBMQ web console. I am wondering whether it send authentication to correct remote servers since that message do not appear when I use pure qiskit.
The following is the code I used with my pure-qiskit code and can run without any issue.
IBMQ.enable_account(token, url)

I just tried this. However, when I ran the first part to save the account, it show the license issue which should not be because I already clicked such things in the IBMQ web console. I am wondering whether it send authentication to correct remote servers since that message do not appear when I use pure qiskit.

Even I faced this issue. There was a different (perhaps older site) where you had to login and accept it:
https://quantumexperience.ng.bluemix.net
Could you try this and see if it works. It did for my case.

Interestingly, I encountered the same problem when using the Qiskit plugin yesterday, and as @quantshah mentioned, this seems to be due to the new IBM Q Experience platform. But if I remember it correctly, then calling IBMQ.save_account('MY_API_TOKEN', 'MY_URL') before I load the device did the job.
I think this could be solved by simply introducing a kwarg for the url and an entry for it in the configuration file.

I think this could be solved by simply introducing a kwarg for the url and an entry for it in the configuration file.

Is this now fixed in XanaduAI/pennylane-qiskit#39?

This should now be fixed in XanaduAI/pennylane-qiskit#44


Weird interval between every idle motion play.

Already set the motionFadingDuration and idleMotionFadingDuration to 0, but it doesn't work like the idle motion interval in Cubism Viewer, which is continuous and would not stop the previous idle animation before the next idle animation play.

I don't get it, that sounds like it's working as expected, since setting the fading duration to 0 means there will be an instant transition between two motions.

The idle animations should not have any transition between it, I think. It should loop smoothly, instead of looping with transition between every individual animation.

I'm still not clear, hmm, isn't "smoothly" equivalent to "with transition"? Or perhaps, did you mean to let the model loop a single idle motion instead of looping between multiple idle motions?

Ummm, yeah that's probably what I mean, there is a weird interval the model just stop animating when transition between the idle motion(one idle animation only)

Example

Example of looping single idle motion without interval using offical cubism viewer

OK I get it, this is a problem indeed, I'll investigate it.

Well, I realized setting the duration to 0 can actually fix this problem. The reason it doesn't work in your case is probably because the durations are overwritten by the ones that the idle motion defines. You can check if the idle motion in your model's settings file looks like this:
{"file": "abc.mtn", "fade_in": 2000, "fade_out": 2000}

If so, removing the fade_in and fade_out should fix it.
And if it's a model of Cubism 3 or 4, the keys will instead be FadeInTime and FadeOutTime.

Ok, I will look into that,thanks!

OK, so I open up the motion.json, and the FadeInTime and FadeOutTime are both 0.0.

You need to check the model.json as well, because the durations will only be read from the model.json, and those in motion.json will be ignored. This is basically another bug that I'll fix later.

I didn't see that two parameters in my model.json , hmmm.

That's strange, I've tested various models and they all behaved well.
Could you share the model you used so I can test with it? If it's a proprietary model, you can replace the textures with solid black images before sharing, in order to prevent any potential abuse.

left_sen.zip
I've also test the models with the viewer in the readme.md, the problem stills happen.

Thanks.
During the test, I can still see that it makes difference when setting 0 durations, as the picture below, the left model gets 0 durations applied and the right model remains default.

Here's how the config should be set up, did you do it like this?
import { config } from 'pixi-live2d-display';

config.idleMotionFadingDuration = 0;

Yes, that's what I did.
This set of models' motion changes expression(not using the expressions in Live2D), so some models would stop at the end of the motion and give a weird face in the time between the motions.

I'm not sure not this is the correct usgae
const live2d = PIXI.live2d;
and use it as
live2d.config.motionFadingDuration = 0;  live2d.config.idleMotionFadingDuration = 0;  live2d.config.expressionFadingDuration = 0;
I did not use nodeJS.

https://github.com/dannylin0711/SDVX-Wallpaper
This is the project, the code is open to public, but not the model.

Okay so I actually made some idiot error
I initialize the model before I change the config
BEFORE
let modelData = await live2d.Live2DModel.from(jsonPath, { idleMotionGroup: 'Idle', motionPreload: "ALL" });
...
live2d.config.motionFadingDuration = 0;
live2d.config.idleMotionFadingDuration = 0;
live2d.config.expressionFadingDuration = 0;

AFTER
live2d.config.motionFadingDuration = 0;
live2d.config.idleMotionFadingDuration = 0;
live2d.config.expressionFadingDuration = 0;
...
let modelData = await live2d.Live2DModel.from(jsonPath, { idleMotionGroup: 'Idle', motionPreload: "ALL" });

I firstly thought that model have the duration time set already, so I should edit the time after I initialize the model, however I just fiqure out while I was taking a shower that the config show be changed before I initialize the model if the model didn't specify a fadein/out value.

Hope that the fade in/out timing would implement in the future, good work and thanks for helping

Even I didn't think of this possibility, it's good to see it works.

Thanks!


Plugins that require variables

Some plugins require variables at install time, how do we handle this?

Good question. More of UI/UX question I guess.


Support all sentence transformers in vectorize workflow

Please test here

Vectorize

Both the Colab Vectorize and Vectorize_params.ipynb notebooks now support the following model_id for sentence-transformers.
config = {
  "dataset_id": "advanced_search_example",
  "model_id": "sentence-transformers/paraphrase-MiniLM-L6-v2",
  "encode_type": "text",
  "fields": [
    "product_title"
  ],
  "region": "xxx",
  "project": "xxx",
  "api_key": "xxx",
  "authorizationToken": "xxx:xxx:ap-southeast-2:xxx",
}

If sentence-transformers/ detected in model_id, will run the following snippet to vectorized instead of the current Vectorize workflow.
    install_package("sentence-transformers==2.2.0")

    print(f"Encoding with {config['model_id']} ... ")
    ds = client.Dataset(config['dataset_id'])
    ds.vectorize_text(fields=config['fields'], models=[config['model_id']])

RelevanceAI version updated to 2.3.2 to support ds.vectorize_text

Great PR thank you


plugin mistake my site not being a next.js site

Describe the bug
my problem is quite similar to #219 but im using yarn with pnp, i tried setting the yarn version to berry, using next build and then next export,  just using next build, un gitignoring .yarn/cache, and everything else i can think about how to fix this
To Reproduce
Steps to reproduce the behavior:

Create a next.js project using yarn with pnp
try to deploy site on netlify
plugin says its not a next.js site and told me to run yarn next on the repo

Expected behavior
plugin work and my site get deployed
Versions

Next.js: v10.1.3

@MeztliRA hello! apologies you're running into issues with PnP :( i believe this is a duplicate of https://github.com/netlify/netlify-plugin-nextjs/issues/169 so i'm going to close this, but if you believe it's not, feel free to re-open!!!


All exporters and importers on network suddenly stop working

Not sure of the exact cause, it is just something that happens after the server has been up for a bit. Breaking and replacing controller does not help. Breaking and replacing the exporters/importers one by one does - they will start working again once placed. If I shutdown the server and bring it back up they will also start to work again. Neither is a really great workaround. This issue does not seem to affect external storage at all - that continues to work when the importers and exporters stop.
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: build 2185
Refined Storage: 1.2.14

Does this issue occur on a server? yes

Likely related to #370

fixed in 1.11. might get backported to 1.10

Definitely hoping this gets backported to 1.10.  It's a frequent hitch currently.

Given that SkyFactory is huge right now, it'd be nice to have the backport for the sake of everyone's sanity :)

In case anyone stumbles onto this post months after the fact like I just did, I fixed the issue by moving around some hard drives in the disk storage system.


Basic JukeBox WebApp

Enter room code
Search for songs
Queue songs

Read through this https://cli.vuejs.org/guide/browser-compatibility.html#browserslist

Not in a room

Host page (get code)

After sign in

Check the queues (nerd)


dep ensure -add should not depend on Go code

What version of dep are you using (dep version)?
dep:
 version     : devel
 build date  :
 git hash    :
 go version  : go1.9
 go compiler : gc
 platform    : darwin/amd64

What dep command did you run?
dep ensure -add github.com/square/go-jose

I ran this in an empty project, so I could start my main.go file.
What did you expect to see?
The usual warning about this dep being ephemeral.
What did you see instead?
no dirs contained any Go code

dep ensure -add is already a "stop trying to be smarter than me, I mean what I say" command, it shouldn't complain about an empty project.

eek, that's definitely an oversight. it makes sense that the requirements would be different for dep ensure -add.

dep ensure -add is already a "stop trying to be smarter than me, I mean what I say" command, it shouldn't complain about an empty project.

while i think i know the general sentiment behind this - the essential model of "keeping things in sync" instead of allowing arbitrary state mutations - if there are any specific things you're chafing against, please do let us know. while we're settled on the base model, we're now increasingly entering a phase of sanding off the particularly onerous edges.

if there are any specific things you're chafing against, please do let us know

I'm fairly at peace with dep ensure -add, modulo #1313.

I want to use dep ensure before I've even started a project or when I'm working with projects that depend on me installing Go dependencies without actually having Go code in them.

dep ensure -add won't work if the project can't build, either:
all dirs contained build errors

dep ensure -add won't work if the project can't build, either:

this is usually a bit of a different case, as having build errors in all your dirs usually occurs when you're mid-refactor. the problem with continuing through in that case is that to skip all of those packages with errors would mean skipping all the imports that you probably picked up in previous dep ensure runs...which likely means that you'd end up with a Gopkg.lock and vendor directory that's basically empty except for what you -added. i don't think that's generally the desired outcome.
(it's important to note that it's fairly hard to reach this case, as there must be Go files that fail validation prior to the end of the import statements, as we use ImportsOnly|ParseComments)

This is true for all calls, even for additional and dep ensure -add, so you will only be able to add a single dependency before actually importing it

you can pass multiple arguments to dep ensure -add, but...

(This could be solved in a separate issue)

that's the only option. anything else would require hanging on to additional state in a sidecar somewhere, which means more files on disk. if you want to permanently guarantee that something not in your imports is still included, you need a required entry in Gopkg.toml.

Still it feels correct to allow running dep ensure on an empty project.

eh, i could be convinced either way? on the one hand, it is a harmless no-op; on the other, what's the benefit of having it not complain to the user when we know the inputs are nonsensical? that seems like a useful guardrail that could help inform the user as to dep's intended use pattern.

IMHO the log output from the no-op explains way more than the short error message. Of course, than the fix could be a better error message as well ;)
I don't really have a strong opinion on this, but I took this as a chance to get my hands dirty on the code. I made a small PR out of it but can't judge my self if it should go through.

I'm running into a case where this would be beneficial (to run dep ensure on an empty project).
I'm using a docker multistep build and building a base image which is then be used in each of my microservices, and it would be helpful if I could simply add the Gopkg.toml and Gopkg.lock to the base image, and then run dep ensure to get my packages into that base image.
This way, dep ensure only needs to be run once, then each of the microservices just copies in everything from that base image and continues from there. As it currently stands I have to copy in my code from that base image and then run dep ensure for each of the 10+ microservices.

@trushton that's a slightly different case, and should be covered by dep ensure -vendor-only.

An additional data point that the docs contradict what the current functionality is (emphasis mine):

At this point, our project is initialized, and we're ready to start writing code. You can open up a .go file in an editor and start hacking away.  I'd forgotten that we hadn't fixed this yet when i wrote that. thank you for calling attention to it.
if you wanted to submit a PR with a parenthetical caveat link to this issue, similar to elsewhere in the docs when there's an outstanding bug... i wouldn't cry about it  🙏🎉

If the project won't build, maybe it will build after the dependency is added/sourced? (shrug) lol


Export location of nearby missing pictures to GPX file

The "Nearby missing pictures" map exists, but many people would probably prefer to see the location of their favorite maps app such as OsmAnd/Maps.me/etc.
So, it would be great to have an "Export to GPX" button that asks you for the desired radius, and generates a GPX file containing all of the latitudes/longitudes. KML/KMZ formats could be a nice addition.
The KMZ file for the whole world (generated by my script https://github.com/nicolas-raoul/wikidata-missing-pictures-offline) takes 100MB, so a default radius of 100km sounds reasonable.
Another advantage is that the file can then be used offline, for instance if you only have WiFi at the hotel or in your home country. Being able to select the center would be a nice addition, for instance letting you download all of Cuba's points while preparing your trip at home in Pakistan.
I tag "beginner-friendly" because while it takes some time implementing, I believe there is nothing really difficult technically. The Wikidata query is already present in the code, and GPX format can be copied from the GitHub project above.
KMZ file seen on Maps.me:

Hi, can I work on this enhancement.

@nsaiisasidhar Awesome thanks! Go ahead with it. :)

@nsaiisasidhar Awesome thanks! Go ahead with it. :)

Hello, I notice this is unassigned but may be being worked on.  I have worked in Geo systems in the past and this is of interest to me.  Could I take this or is this still in active development?

 ;-)

--bump-- ;-)

@nsaiisasidhar Yes, if you get the results as CSV then feel free to use regular expressions to make that into GPX. Purists might argue that you should get JSON and parse it, but that would take more memory, so CSV is OK I guess.
Indeed, please use a radius query if you can, thanks! :-)
Sorry for my late reply. I guess this is your current code https://github.com/nsaiisasidhar/apps-android-commons/commits/953-Export-location-of-nearby-missing-pictures-to-GPX-file ?
In what state is your code? Is it ready, or in progress?
If in progress: Do you want to continue working on it, or do you can @mashawan continue your work?
Thanks!

I don't think this issue is relevant now. Please review and close this issue

@Rishavgupta12345 This issue is very relevant, we still need this feature.

Hi @Rishavgupta12345 , could you elaborate on why you feel this issue is irrelevant now?

Hey @nicolas-raoul, Can I work on this issue ?

Shared a pull request here https://github.com/commons-app/apps-android-commons/pull/5645
I am open to suggestions 🙌


On MP3 decoding

There's no need to justify for this feature. Also I think in 2k18 the technology has already come to a point where MP3 is available for ordinary mortals. Whatever people say against it.
So there is libmpg123, and as I see it's LGPLv2, so it can be used commercially and requires dynamic linking. But libnyquist is built as a staticlib. Btw, why exactly libnyquist (and LabSound) only support staticlib build?
May be we could arrange it the other way around. Currently, your AudioDecoder is very rigid. If it allowed some external tampering, I could simply add my own decoder in the middle of runtime.
Taking the "external" approach even further, I could send the decoded MP3 data in some intermediate format... For example with ".mp3dec" extension, and there will be the decoded data and metadata in a specified format. But this is the least desired way.

Hi @raub - some people (like myself) prefer static linkage models -- less dynamic libraries to copy around. libnyquist was created out of frustration with the LGPL'd libsndfile to start with. Runtime extensibility is a nice feature but beyond the scope of what I had intended to provide here.
If I add MP3, it would be through this library in the public domain: https://github.com/lieff/minimp3

I'm ok with pretty much any. It all comes down to the fact the domos, I'm trying to port from web, all use mp3 resources. And currently LabSound (using libnyquist decoders) is incapable of mp3, so for me anything is better then nothing.

Hi @raub - implemented in acaf19d97e028fc499a316942bfa62c5934bc957


Add troubleshooting information for fisher module

Hi, as per #733 I've updated the readme.md with some troubleshooting for the fisher overlay. I'll try and keep the troubleshooting doc updated with more common problems if they arise.

Ahh sorry for forgetting about this! I've added an FAQ into 'docs` and added a link on the main readme.md to it. I've moved the Common workarounds and solutions section into the new FAQ and updated it for ngld's OverlayPlugin. I included a few of the common errors I've seen on the FFXIV ACT Discord too, and added the fishing module info underneath. Your thoughts on these changes?


Lock/unlock mouse cursor to window

todo
https://msdn.microsoft.com/en-us/library/windows/desktop/ms648383(v=vs.85).aspx
https://github.com/d954mas/Defold-lock-cursor/blob/master/lock/extension/src/LockMouse.cpp

Locking cursor to center of the window, used in FPS shooters where you want the user to be able to continuously turn around with the mouse. mouse dx and mouse dy are used to get rotation. Generally you want to show a graphic which is centered in the screen and hide the mouse cursor. If you don't hide the mouse cursor as you turn around you'll still see the mouse dx/dy distances from center which is weird.
is_cursor_locked
lock_cursor
unlock_cursor
Clips the cursor to the borders of the current window so that it cannot exit the window bounds. Useful in games like Diablo where you might want to play in window mode with lots of fast clicking and not accidentally click outside of the game window.
is_cursor_clipped
clip_cursor
unclip_cursor
Generally if playing in window mode if the user presses ESC you want to open a pause overlay menu of some kind and unclip the cursor so that it can leave the window bounds.
For FPS game you want similar behavior where pressing ESC displays the normal cursor again which is allowed to move around the window freely while the overlay is visible..

The way d954mas's Lock extension works is it just sets the mouse position back to some preset position every time you call the function and reports dx and dy. So all you really need is a set_mouse_position function. You can get dx and dy from input events normally, and DefOS already has the show/hide cursor functions.
"Capture"ing the mouse to a window should be done with this I think?

I think we can close this, as it’s been implemented for all platforms in #57 and #60 (minus clipping for HTML5, but as far as I know that’s not possible).


”Brădet”, ”Floroaia”, ”Scrădoasa” are listed in the drop-down menu

Description:
When user writes ”Brădet”, ”Floroaia”, ”Scrădoasa” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Brădet"

Expected results
The location appears in the drop-down menu as "Sat Brădet, COVASNA (ÎNTORSURA BUZĂULUI)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Floroaia"

Expected results
The location appears in the drop-down menu as "Sat Floroaia, COVASNA (ÎNTORSURA BUZĂULUI)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Scrădoasa"

Expected results
The location appears in the drop-down menu as "Sat Scrădoasa, COVASNA (ÎNTORSURA BUZĂULUI)".


Cannot draw out of the viewport when scale is less than 1

Prerequisites

[X] Which version are you using? (Is it latest?) 5.1.4 latest
[X] Are you reporting to the correct repository?
[X] Did you search existing issues? (Were any related?)

Description
Cannot draw out of the viewport.
Steps to Reproduce the issue

The element size is 500x500 while scale is set to 0.5, so that it occupies 250 in width and height. Display correctly
Open the code sandbox below. Select length tool.
Draw a line out of the region of center 250x250, it disappears.

Expected behavior: (What you expected to happen)
I wish to be able to draw outside the viewport. Why limit the drawing to the viewport?
Actual behavior: (What actually happened)
When draw outside the viewport region, it disappears.
CodeSandbox With Reproduction of Issue:
https://codesandbox.io/s/lengthtool-yhypz?file=/src/components/SimpleExampleViewer.vue

Basic Template

I


"Viișoara" is listed in the drop-down menu

Description:
"Viișoara" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Viișoara" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Viișoara, TELEORMAN (Viișoara)" and "Comuna Viișoara, TELEORMAN".


Replace design assets/links references to MM

Review the whole project and update assets, links, strings and anything that may refer to MM

replace icons to https://www.dropbox.com/sh/f808p8o7eln1x8o/AACn8Znq5r0CdZHKUzg4JgB1a?dl=0
replace name to Nifty Wallet

@igorbarinov
There are missing icons:

64 × 64
512 x 512

Please ask to add them too.


[Security] Edit Customer

Admin should not able to edit any customer's details. This is a big security bug.

I would like to work on this

Why shouldn't they?

On WordPress, admins are able to change all of the information provided by a user except the username. So what is the big security bug?

On WordPress, admins are able to change all of the information provided by a user except the username. So what is the big security bug?

Closing due to inactivity. Will re-open if required.


New documentation system/format (based on md)

Here, I am starting to copy new documentations using md format. By this pull request, first, I can check how the new format works on github, second, some of the new documentation texts will become publicly available.

is this pull request supposed to be from master into your sh-documentation? if so you can just merge this request since its from the master into your branch, otherwise please close this one and change the direction

closed because of wrong direction


BUGIX Altering calls to methods that are not deprecated

BUGFIX not BUGIX @stojg :D

Holy crapbags!


python_repl_ast and package import (ModuleNotFoundError and NameError)

System Info
langchain == 292
Who can help?
@hwchase17
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Here is my code:
agent_analytics_node = create_pandas_dataframe_agent(
                                                        llm,
                                                        df,
                                                        verbose=True,
                                                        reduce_k_below_max_tokens=True,
                                                        max_execution_time = 20,
                                                        early_stopping_method="generate",
                                                    )

tool_analytics_node = Tool(
                                name='Analytics Node',
                                func=agent_analytics_node.run)

tools = [tool_analytics_node]

chat_agent = ConversationalChatAgent.from_llm_and_tools(llm=llm, tools=tools)

executor = AgentExecutor.from_agent_and_tools(
                                                        agent=chat_agent,
                                                        tools=tools,
                                                        memory=memory,
                                                        return_intermediate_steps=True,
                                                        handle_parsing_errors=True,
                                                        verbose=True,
                                                    )
with st.chat_message("assistant"):
 st_cb = StreamlitCallbackHandler(st.container(), expand_new_thoughts=False)
 response = executor(prompt, callbacks=[st_cb])

here output from the agent:```

Entering new AgentExecutor chain...
Thought: The question seems to be asking for the sentiment polarity of the 'survey_comment' column in the dataframe. The sentiment polarity is a measure that lies between -1 and 1. Negative values indicate negative sentiment and positive values indicate positive sentiment. The TextBlob library in Python can be used to calculate sentiment polarity. However, before applying the TextBlob function, we need to ensure that the TextBlob library is imported. Also, the 'dropna()' function is used to remove any NaN values in the 'survey_comment' column before applying the TextBlob function.
Action: python_repl_ast
Action Input: import TextBlob
Observation: ModuleNotFoundError: No module named 'TextBlob'
Thought:The TextBlob library is not imported. I need to import it from textblob module.
Action: python_repl_ast
Action Input: from textblob import TextBlob
Observation:
Thought:Now that the TextBlob library is imported, I can apply it to the 'survey_comment' column to calculate the sentiment polarity.
Action: python_repl_ast
Action Input: df['survey_comment'].dropna().apply(lambda x: TextBlob(x).sentiment.polarity)
Observation: NameError: name 'TextBlob' is not defined

### Expected behavior

agent should be able to install python packages

In my case TextBlob is installed in python environment

met the same issue and the bot answer is not corret as I think. I have installed it in my env.


remove O_NONBLOCK from stderr

step1: start a container without tty
hyper run ubuntu
step2: get the fdinfo of the stderr
cat /proc/self/fdinfo/2
pos:	0
flags:	04001
mnt_id:	8
it shows that the stderr has the O_NONBLOCK flags,
we should remove it.
Signed-off-by: Lai Jiangshan<EMAIL_ADDRESS>

see https://github.com/torvalds/linux/blob/master/include/uapi/asm-generic/fcntl.h#L38
#define O_NONBLOCK	00004000


Feat: Context Provider and Object

Here is the current plan, feed the Hooks into a context provider for people to import and Use.

onConnect can pass the Username, Secret and Project ID into the hooks for setting up.
Then is exports state which can be passed into Socket or Components with the prop names lining up perfectly


Week 2 review ticket

HTML Fragments
Time Box on home page and dark mode throughout blog
Markdown Table Post
Dictionaries, Lists, Iterations
Tags
CSP Notes Additions
I think I deserve a 2.7 plus/3.0 because I met all the requirements and I added extra such as putting all the loops and adding extra stuff to my CSP Notes, made a post with extra markdown table, etc.

Crossover Review: 2.7+/3.0
Notes:

Table is present, though it doesn't utilize liquid
Search and Tag functions present; reflection on changing website theme present
Demonstrated 3 of 2 loops and made a python quiz


Tests, mailbox fixes, etc

Quite a lot going on in this PR 😅 To summarize:

Tests for the OneForOneStrategy
Re-added exception handling in mailbox, for cases where an actor throws an exception instead of returning a failed task
Send system messages to mailbox statistics
Moved SystemMessage into mailbox so Suspend/ResumeMailbox can inherit from it
Moved all test fixtures into a shared project

I'll merge this, but please see the inline comment on message expect abstraction.
Maybe we could create something similar to Akka testkit, but not tied to any specific test framework, only helper abstractions for the built in primitives.


No Cloud SDK error in run config window needs to be more descriptive

Now that you can save an invalid Cloud SDK path and the deployment run config validates this path is valid before deploying, we should make it more clear where the problem is by updating the error language. Here's what it currently says:

... and the problematic field is:

Selecting the ellipses button on the right opens the following dialog:

We should improve the experience here to make it more clear from the run config window where the issue is and how to fix it.

Just a note that a convention we use is that the message should not include any positional references (e.g. click on the server field above) so that the messages are decoupled from the UI.


bug in frame utils?

https://github.com/waymo-research/waymo-open-dataset/blob/a19ba044e0ad42c49e95f09298b491db5d8ed05c/waymo_open_dataset/utils/frame_utils.py#L318
this should be rolling_shutter_direction ?

Yes, thank you for pointing this out. We will fix soon and update the validation server accordingly. For now, you can act as though that field reads "ROLLING_SHUTTER_DIRECTION" instead.

@sabeekpradhan thanks! Could you also fix
#305 otherwise the camera projection module is still not usable

Yes they will be in the same PR.

Fixed in https://github.com/waymo-research/waymo-open-dataset/commit/1f7525229e2b2853b94a92ce44a7961cddf9aa71


Extend OE for Neuropixels

The Neuropix-PXI plugin saves TTL in a states.npy. Extended the list of _possible_event_stream_names to reflect this.

In some OE versions, the units field in the structure.oebin is blank, but the gain is always to uV. So this is automatically inferred in case it's missing

OK for me.
@JuliaSprenger @apdavison : do you want to have a look.

Yes, looks also good to me. @alejoe91 Thanks for the improvements!


TSDB: Ever growing txRing in 2.27.1

we also occur same problem at 2.27.1
{
"status": "success",
"data": {
"version": "2.27.1",
"revision": "db7f0bcec27bd8aeebad6b08ac849516efa9ae02",
"branch": "HEAD",
"buildUser": "root@fd804fbd4f25",
"buildDate": "20210518-14:17:54",
"goVersion": "go1.16.4"
}
}

Originally posted by @ilixiaocui in https://github.com/prometheus/prometheus/issues/7120#issuecomment-905322514

@ilixiaocui could you graph the low watermark and see if it is changing at all or if it got stuck for good? If the latter, is there anything that happened at that moment with your Prometheus? (Like a long running or failed query or anything interesting logged.)
If you restart your Prometheus, does the problem promptly recur, or does it take a while?

Closed as duplicate of #9253

@ilixiaocui could you graph the low watermark and see if it is changing at all or if it got stuck for good? If the latter, is there anything that happened at that moment with your Prometheus? (Like a long running or failed query or anything interesting logged.)
If you restart your Prometheus, does the problem promptly recur, or does it take a while?

It is changing.
In order to keep the scene，I have not rebooting. No query operation was performed when the memory suddenly increased .


Cocoapods won't get the newest version of SwiftCharts

Hi,
For some reason when I try to get SwiftCharts 0.5 with my pod file, it refuses.  I think somehow Cocoapods only knows about 0.4.  I don't know why this is.  Any help in getting the podfile to get the most recent version would be helpful.  Thanks.
Sincerely,
Sean

Hi,
for now please use master as described in the readme (it's stable). Last time I tried there were some issues creating the pod and don't have time to investigate this. Will come back to this later.

Hi,
Any hints on how to upgrade 0.4 to Swift 3 or Swift 2.3?  I've upgraded to XCode 8 and I need the library I'm working with to work with one of those Swift versions.  Thanks for your consideration of this matter.

Yes use master, which is 3.0. In the readme you find all you need.

0.5(.1) is now available in Pods, sorry for the delay!


[DOC] Links to github not working

I think the links in the https://xgcm.readthedocs.io/ are not working properly.
The github icon seems to be directing everything to pangeo-forge.

I submitted a fix in #512. I think the warning on the 'raise issue' is just persisting from the 'suggest edit' issue before. When I x out the warning and click on raise issue it works fine for me.


Permission policy to the end-user app blocks the bot

Several clients ask us to manage access to applications through policies, especially for end-users.
But we find ourselves with the situation where the permission policies are applied and the bot is blocked. The reproduction of the case is as follows:
We create a "Setup policy" that installs the end-user app.

We hope that this policy is replicated and verified that the application was installed and works for this user.
We apply a "Permission policy" to the previous user that allows the end-user app to be used and blocks the others.

We wait for this policy to be applied and stop receiving messages getting the error "The admin tenant has blocked this bot"
This is a desired functionality since the installation of the client application will be carried out for thousands of users.

@dbernalf : Can you confirm the App version whether it is V3 or V4.
V3 using same bot id for user app and author app. So, allowing one app in app permission policy leading to issue.
So, Both apps are separated in V4.

Hi, is V3. I'm pending test with V4, as well as you said with diferent App regsitration for both i think this work correctly. Thanks for all

Hello @dbernalf,
We are closing this issue.
Let us know if issue still persists.
Thanks


Handling basic auth with a middleware or a request annotation

Does it already exist, du you want a pull request for that?

Security is not a part of aiohttp and perhaps will never be included.
There is aiohttp_security library that tries to solve auth/autz issues. It's almost ready but still not released yet.
The code looks good but I need write documentation for the library.


Endless loop of sorts in suggest widget on master

open a new untitled file
type "hey hey"
vscode freezes
change the settings to "editor.quickSuggestions": false
type "hey hey"
vscode does not freeze

Trait#set being called in a loop...

This got introduced with the in-order-event delivery... I pushed a change to make that opt-in but we should still look into the list and why that happens. Ideally, in-order-event-delivery is the default, not the exception

This should actually stop the recursion: https://github.com/Microsoft/vscode/blob/master/src/vs/editor/contrib/suggest/suggestWidget.ts#L576
hm..

Yeah, problem was that a nested call to fire would return before the event have been dispatched. With that state wouldn't be updated and stuff like this would happen...


feat: add new AnalyzeText API

Related Issues/PRs

#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?

[ ] I have written tests (not required for typo or doc fix) and confirmed the proposed feature/bug-fix/change works.

Does this PR change any dependencies?

[ ] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[ ] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

/azp run

Codecov Report

Merging #1760 (1bf2f5b) into master (4a25954) will decrease coverage by 7.73%.
The diff coverage is 92.45%.

@@            Coverage Diff             @@
##           master    #1760      +/-   ##
==========================================
- Coverage   86.51%   78.78%   -7.74%
==========================================
  Files         273      275       +2
  Lines       14420    14571     +151
  Branches      769      782      +13
==========================================
- Hits        12476    11480     -996
- Misses       1944     3091    +1147

Impacted Files
Coverage Δ

...zure/synapse/ml/cognitive/text/TextAnalytics.scala
37.84% <25.00%> (-46.22%)
:arrow_down:

...re/synapse/ml/cognitive/language/AnalyzeText.scala
95.83% <95.83%> (ø)

...apse/ml/cognitive/language/AnalyzeTextSchema.scala
100.00% <100.00%> (ø)

...m/src/main/scala/com/microsoft/lightgbm/SWIG.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...pse/ml/lightgbm/params/ClassifierTrainParams.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...e/synapse/ml/lightgbm/StreamingPartitionTask.scala
0.00% <0.00%> (-97.11%)
:arrow_down:

...zure/synapse/ml/lightgbm/dataset/SampledData.scala
0.00% <0.00%> (-90.00%)
:arrow_down:

...ure/synapse/ml/cognitive/speech/AudioStreams.scala
0.00% <0.00%> (-87.88%)
:arrow_down:

...azure/synapse/ml/lightgbm/LightGBMClassifier.scala
10.75% <0.00%> (-80.65%)
:arrow_down:

.../synapse/ml/cognitive/speech/SpeechToTextSDK.scala
18.03% <0.00%> (-73.34%)
:arrow_down:

... and 35 more

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

/azp run

/azp run


Request: Option to disable auto-output of closing markup

Pretty simple: I use bulleted lists a lot and I find it annoying that typing * results in * * in the editor.
Other open/close markup doesn't bother me as much. I'd be happy with an option to disable the closing markup generation altogether, but the ideal option would be options per type of markup (underscores, backticks, etc.)

I support request - sometime programmer must ask himself: would his "help" really help??


"Customer Service Analyst - Opera Cloud" job from Oracle is duplicated

URL: https://admin.peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 11
STEPS TO REPRODUCE:

Click on "Companii".
Select "Oracle".
Click on "Vizualizeaza joburi".
Go to the "Customer Service Analyst - Opera Cloud" jobs and check their IDs.

EXPECTED RESULTS:
"Customer Service Analyst - Opera Cloud" jobs have different IDs.
ACTUAL RESULTS:
Two "Customer Service Analyst - Opera Cloud" jobs have the same ID 244691.

Jobul specificat de tine nu există în validator. Sunt 2 joburi cu același nume dar au “job idetification “ unice.


packet.py doesn't work

Hi Liz,
I got the following error, could you help to figure out what was wrong?
root@iubuntu2004:~/ebpf-beginners# python3.9 packet.py
bpf: Attaching prog to eth0: Operation not supportedTraceback (most recent call last):
File "/root/ebpf-beginners/packet.py", line 32, in
b.attach_xdp(dev="eth0", fn=b.load_func("hello_packet",
File "/usr/lib/python3/dist-packages/bcc/init.py", line 726, in attach_xdp
raise Exception("Failed to attach BPF to device %s: %s"
Exception: Failed to attach BPF to device b'eth0': Success

same problem

Hi @peiqiaoWang @wangsu502, just change eth0 with your actual network interface, for example: ens3.


BigInt divide (/) operator should return a BigInt

Dart VM version: 2.2.0-dev.0.0 (Mon Nov 19 15:10:42 2018 +0100) on "windows_x64"

I'm working on factoring large numbers and I came across a problem. When I divide a BigInt by another BigInt, the result is a double. Which means it gets truncated.
BigInt upper = BigInt.parse('9426890448883247745626185743057242473809693764078951663494238777294707070023223798882976159207729119823605850588608460429412647567360000000000000000000000');
BigInt divisor = BigInt.from(99);
print(upper / divisor); //9.522111564528534e+151
print((upper ~/ divisor) + upper.remainder(divisor)); //95221115645285330763900866091487297715249431960393451146406452295906132020436604029120971305128576967915210612006146064943562096640000000000000000000000

If I do an integer-division and add the remainder, it works as expected.

CC @lrhn

In Dart, integer division uses the ~/ operator and always returns a non-fractional value.
The / division may return a fractional result and is always a double.
That's also true for non-big integers.
We can't make / return a bigint because BigInt.one / BigInt.two cannot be represented exactly as a big-int. Dart does not have a "big decimal" type with arbitrary precision and a fractional part, which could otherwise have been the type of /.
So, as you recognized, you just have to use ~/.


Rayon and indicatif

Will something like the following cause pb to lock and eventually run sequentially?
    let pb = ProgressBar::new(size as u64);
    let found_r = (0..size).into_par_iter().map(BigUint::from).find_any(|r| {
        pb.inc(1);
        // Check if g^r == a
        if g.modpow(r, &p) == a {
            true
        } else {
            false
        }
    });
    pb.finish_with_message("done");

indicatif has native support for rayon iterators, maybe check that out?

 I saw 100% utilization from all the cores but my question was mostly focusing on whether this is efficient. What does it mean to have built in support for rayon? Doesn't it still need to be locked and unlocked all the time in order to be updated? In my understanding this would result in a lot of overhead. I'm asking to try to understand how it works.

I think the 100% utilization is to be expected and is not necessarily an issue. There are no delays in the code.

Thank you, that makes sense.


[HOLD for payment 2024-05-03] Unable to load categories in the workspace settings

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

Version Number: v1.4.65-2
Reproducible in staging?: Y
Reproducible in production?: N
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
Expensify/Expensify Issue URL:
Issue reported by: @trjExpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1713951624767449
Action Performed:
Break down in numbered steps

Have a collect policy on the beta
This policy should have categories set up
Sign into this account
Go to the workspace settings
Go to Categories page

Expected Result:
Describe what you think should've happened
the categories list should load without issues
Actual Result:
Describe what actually happened
Infinite spinner is showing
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub

reverting https://github.com/Expensify/App/pull/40496/files fixes the issue

This is a known problem https://github.com/Expensify/App/pull/39132#discussion_r1571085835 It seems that https://github.com/Expensify/App/pull/40375 didn't fix it

cc @hayata-suenaga

PR was merged. requesting cherry pick -> https://expensify.slack.com/archives/C07J32337/p1713970530252449

@abekkala I'm OOO 2024-05-03 to 2024-05-06 so adding a BZ buddy to issue payment if I'm out when it becomes due

@sonialiap - what is the payment summary?
@mountiny it looks like you provided the fix and @s77rt did the PR review, correct? Was that a $250 PR review or some other amount?

No payment is needed here since it was a regression

Handing this one back to you @sonialiap as you're back from ooo

This can be closed


Remove notification from travis config

Any reason ci failed ? a cache issue ?

It looks like cloning the branch failed because Travis was not looking for it in my fork.


fix: set cookie time

Branch to have consistent happo screens

@AntBush to ensure screenshot consistency, you'll want to wait until the mutations complete. Asserting that Last saved is displayed in the status component before calling happo should do it.

Asserting that Last saved is displayed in the status component before calling happo should do it.

True, I checked the happo diffs as well and the cookie set that I did wasn't recognized/sent along. The happo diffs show that it isn't being set to Oct 10 2022.  @matthieu-foucault Any thoughts on this? I don't see anything immediate that is wrong here especially since

Are we

deploying the mocks schema before running our end-to-end tests?
running our server with the correct env variables?


Update getServer to handle %referral and rwhois urls.

This PR addresses the issue raised in Issue 49
It also includes a getServer unit test to check that I didn't break the existing behaviour, with whois responses in testdata files, as that seemed cleaner than including the text inline in whois_test.go?

Hello @gavincarr
Thanks for your works.

If you think this repository is helpful, please share it with friends, thanks.


Make GCloud artifacts page show up in Prow directly

Description
Also style this and other pages like Prow.
Checklist

[ ] Investigated and inspected CI test results

None of these are needed:

[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed

[ ] Checked how the output looks in Prow and shared the screenshot in this PR.

An early look (before adjustment) is available here https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/stackrox_stackrox/5603/pull-ci-stackrox-stackrox-master-ocp-4-10-operator-e2e-tests/1644055638038286336
Waiting for more recent results.


epinio chart and cli ignore the image.{bash,awscli}.* settings.

Describe the bug
This is a sub issue for #1248.
Rancher requires chart support to support airgapped installation and the like. A chart has to specify images in a specific format rancher will then use to redirect all image requests from the chart to its own registry.
While the epinio chart contains most basic support for the awscli and bash images the server uses during staging this support is incmplete. The values.yaml only contains the necessary repository and tag settings. Missing:

The chart does not use these settings, i.e. they are not stored in cluster resources, and
The epinio binary actually has the images baked into it, i.e. is not reading the information from cluster resources.

Expected behavior
The epinio server has to read the image information from cluster resources, and the chart has to pass the settings into whatever cluster resources the reads from.
Proposed change

Pass the image references into the same config map the server reads the staging scripts from.
Extend the epinio server to read the image references at the same time it reads the scripts.

There is also this one:
internal/manifest/manifest.go:  DefaultBuilder = "paketobuildpacks/builder:full"

Yep.
Chart PR = https://github.com/epinio/helm-charts/pull/148
Epinio PR = https://github.com/epinio/epinio/pull/1311


DRILL-5039: NPE - CTAS PARTITION BY ()

Moved varchar newPartitionValue functions to NewValueFunctions template:
a. took advantage of code generation for varchar newPartitionValue functions;
b. all newPartitionValue functions will be in one file.
Fixed logic for reassigning nullable varchar holders during newPartitionValue function execution (previously resulted in NPE if varchar partition column contained nulls).
Added alltypes_optional.parquet (each type contains at least one null value) in test resources and updated alltypes_required.parquet to have the same types as in alltypes_optional.parquet.
Added new unit test testPartitionByForAllTypes to cover all types and cases (with and without nulls) during CTAS with partitioning.
Updated minMaxEmptyNonNullableInput test to reflect changes in alltypes_required.parquet.

Is this the same bug as DRILL-4872 ?

+1


CLI Testing placeholder

Spoke to Ken today we discussed there will be some testing needed for the CLI wallet, "sending transactions like in the stacks wallet" with more specific use cases to come.
I have created this issue as a place holder to flush out details of the test @yknl currently testing is blocked and we are not able to test. @yknl Please let me know when this task is ready for QA, we can track the progress here.

Hey guys just following up, are there any updates on testable functionality that I can get started with this week?

@timstackblock I will write up a document today outlining how to properly test the explorer :)


Move templates/plugins ipc calls from main to separate services files

This is related to https://github.com/kubeshop/monokle/issues/1093

Done in https://github.com/kubeshop/monokle/pull/1534


construct the struct contains null from DataChunk::from_pretty

Is your feature request related to a problem? Please describe.
Following example will fail.
let test = DataChunk::from_pretty(
            "<i,i,i>
             (.,1,.)
            "
        );

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

fixed by #14497


[All platforms][CalendarView] IsGroupLabelVisible="True" isn't working with Material and Cupertino Styles

Current behavior
For the CalendarView, IsGroupLabelVisible="True" isn't working with Material and Cupertino Styles

UWP :  App crashing with this error " The parameter is incorrect"
Other platforms : App + calendar launching correctly but the goup label is not visible

Expected behavior
IsGroupLabelVisible="True" should normally work on all platforms with Material and Cupertino styles without any errors or crashes.
How to reproduce it (as minimally and precisely as possible)
<CalendarView IsGroupLabelVisible="True"
                          Style="{StaticResource MaterialCalendarViewStyle}" />

OR
<CalendarView IsGroupLabelVisible="True"
			  Style="{StaticResource CupertinoCalendarViewStyle}" />

Environment
Nuget Package: Uno.UI
Package Version(s): 3.8.0-dev.439
Affected platform(s):

[X] iOS
[X] Android
[X] WebAssembly
[X] UWP
[X] MacOS

Anything else we need to know?
We should have a discussion regarding the guidelines for Material and Cupertino for the group label.

Not fixed by unoplatform/uno#6220


Double-check all functionality works in Linux

Per #37 we may be completely compatible with most Linux distros, but I'd love for Linux users to check it out for me.
Tag @jyrkidn, @PHLAK, and @sinnbeck who all commented on that issue about Linux.

I'll do my best to check this out in the next couple of days.

I'm on Pop!_OS 20.04 LTS and can confirm that I can enable and disable docker containers through takeout. I'm going to start using it daily now, if i should encounter any issues, I'll create a new one.

I just tried on Linux Mint 20 and was able to start/stop and enable/disable a MySQL container with takeout.

I'm using this on Ubuntu 20 and everything is working great.

Thanks everyone! Gonna call this good!


[Feature Request]: add organization id to OpenAIEmbeddingFunction

Describe the problem
For openai accounts with multi organization, the only way to manage the api source is using the
Describe the proposed solution
Add optional organization_id to OpenAIEmbeddingFunction, this parameter is used if the user sends it; otherwise, openai will assume the default organization.
Alternatives considered
No response
Importance
nice to have
Additional Information

The same parameter added to type script version but not the python class.
I prepare the solution and ready to merge.

Thanks! Will merge when tests pass

@mik-laj yea https://github.com/chroma-core/chroma/blob/main/chromadb/utils/embedding_functions.py#L53


Extend make fetch-att to also fetch image data

This should be useful for hacking on policies now that https://github.com/enterprise-contract/ec-cli/pull/821 is merged.
Currently it doesn't include the "signatures" key which shows up in the ec input. Not sure how to extract that easily.

I also notice the "extras" key is not there, but not sure what to do about that.

It would be nice, at some point in the future, to use ec-cli to fetch this data. Maybe we can introduce a command like ec fetch image that does this.

Comparing to a "real" ec input file:
json-diff /tmp/ecp_input.585651778/input.json input/input.json
 {
   attestations: [
     {
-      extra: {
-        signatures: [
-          {
-            keyid: "SHA256:IhiN7gY+Z3uSSd7tmj6w5Zfhqafzdhm3DZjIvGc6iYY"
-            sig: "MEUCIQDcgZIwEkLFqD7U9HrobgEC8Jo7wm+xJ5AoyO3qg+aj8QIgb9xDpjYGRMmpVk+QATeVKlHonzBiu51HtT3J+lQXPXc="
-          }
-        ]
-      }
     }
     {
-      extra: {
-        signatures: [
-          {
-            keyid: "SHA256:IhiN7gY+Z3uSSd7tmj6w5Zfhqafzdhm3DZjIvGc6iYY"
-            sig: "MEYCIQDKSihaAR/zAhJhR5GCqleDvfUUtvRw61vk0YeTBAnOSQIhAKa09B4yEfaSJronmWBFbu5cVPNxm17CMl/PElEz1POa"
-          }
-        ]
-      }
     }
   ]
   image: {
-    signatures: [
-      {
-        keyid: ""
-        sig: "MEUCIQD86lmOqCovYZDPKm0XxxsLgDQcFIFAv+QZxrFSHmCvQAIgTd1I005ox8MfABqsAen6PZEyg2MCEQNBCx1NLS3V0JQ="
-      }
-    ]
   }
 }


persist patterns thru game quit / start

this is an obvious thing and might be related to the import/export features #48 as well as improved LIF library importing, #41

not actually sure how much persistence we want here. to have every pattern ever made be in the world forever is probably not desirable. possible use cases are:
only patterns that were busy growing should persist. this could mean that only the grow_queue is kept.
other patterns that players want to keep should maybe be handled by import / export. or maybe patterns should be checked off as "persist" which could equate to a "favorite settings" feature that would capture the rules and the starting conditions of patterns selected by players as favorites.
otherwise, if we persist everything, we will need a delete mode so players can flush or individually delete patterns and that doesn't seem like the ideal way to do it.

this is adequately handled by chat command #51


Medaka dependency issue

Hi,
I have been trying to run Natrix2 on the dev branch with Nanopore data with ./pipeline.sh. During the initial run, at the creation of conda environments, I encountered a package not found error. Medaka 1.1.2 is indeed not available through bioconda for osx-64. There is a gap in available versions between 1.0.3 and 1.8.0. Trying to install 1.8.0 runs into other dependency issues. Trying to run with 1.0.3 is accepted and runs perfectly fine until the medaka step where Natrix exits with an unspecified non-zero exit code in the medaka_polishing rule.
Do you have an idea of a potential workaround?
Log from initial missing medaka package: 2024-09-05T143442.833687.snakemake.log
Log from error in medaka step: 2024-09-05T150213.857076.snakemake.log

Hello @theo-krueger,
since you are using osx-64, it is often best to run bioinformatics workflows in a Linux environment to avoid potential compatibility issues. One way to solve your problem is to use Docker to create a container that allows you to run the workflow in a Linux environment. This way, all dependencies, including those that may not be available for osx-64, are properly managed.
Natrix2 has specific version requirements for some packages, and manually changing these requirements can sometimes lead to conflicts. For example, Medaka is often not available for osx-64, which could contribute to the problems you are having.
There will be an official Docker image update for Natrix2 in the next few weeks. In the meantime, you can create your own Docker container. Alternatively, you can also use a Linux virtual machine (VM) to get a fully compatible environment.
If you have access to a workstation or an external server, I recommend using it as it might provide better performance and stability for running these workflows.
If you have any further questions please feel free to contact us.
Best,
Dustin

Hello Dustin,
I will try that and I'm looking forward to trying the official relase.
Thanks a lot for your help :)

Hello Dustin, I will try that and I'm looking forward to trying the official relase. Thanks a lot for your help :)

Great, I'm glad to hear that.
Have a great day. :)


[pkg/errors] Add Gatherer DoneOrFailed(Ctx)

Closes #181
Need this for testing, @RmbRT also seems to appreciate it.

@ggwpez Where you able to figure out the reason for linter error ?
I am also getting the same error on my PRs as well. But when I try to run the linter locally, I do not get this error.

@ggwpez Where you able to figure out the reason for linter error ?
I am also getting the same error on my PRs as well. But when I try to run the linter locally, I do not get this error.

It looks like the function that is used got deprecated.
https://staticcheck.io/docs/checks#SA1019
Would it be possible to rewrite that code without using the deprecated function?

Update: I think we can just modify the panic message:
panic(fmt.Sprintf("Expected both interfaces to be zero or both non-zero, got %v, %v", left, right))

Yes. Its just weird that i cannot locally reproduce the error, just as @manoranjith said.
I will create a new PR for the fix.


[BUG] Sometimes crash on joining world

Describe the bug
A clear and concise description of what the bug is.
Sometimes, crash on joining world. and after crash, it crashes again and again when I joined world.
To Reproduce
Steps to reproduce the behavior:
Sorry, Idk how to reproduce it.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
No screenshots.
Desktop (please complete the following information):

OS: Windows
Minecraft 1.18.2
Forge Loader version 40.1.0
Mods : Rubidium 0.5.2a / Rubidium Extra 1.3.1
Version : D. light 1.3.1
Branch mc1.18

Additional context
Add any other context about the problem here.
crash-2022-04-30_00.48.04-client.txt - This one for first crash
crash-2022-04-30_00.48.49-client.txt - and this one for second crash
two other crashes happened.

Duplicate of #1


Enable verbose logging for image-builder Ansible tasks

Enable verbose logging for all Ansible tasks that run during OS image builds. Users can set the verbosity from 1 to 6.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/close


Weighted average is not used in FedAvg

Thanks for your valuable time for cheking this issue.
Describe the bug
Weighted average is not used in FedAvg (see fedavg.py)
Details
My understanding on this package.

global_update() is performed in the server.

Then we show the details.
First, we found that the server inherite the SyncServerHandler without more modifications (see line 13 fedavg.py).
Then we see that FedAvg use a unweighted average on the model parameters at lines 106-109 in basic_server.py.
We note that class FedAvgClientTrainer() at lines 25-32 in fedavg.py is rewriten with a new function global_update(), yet this contradicts with my understanding 1. So maybe the function global_update() should be added in the class FedAvgServerHandler().
In addition, we note that the global learning rate (see ref [1,2]) has been widely adopted in FedAvg, so it would be better to add the global learning rate in FedAvg.
[1] Karimireddy, S. P., Kale, S., Mohri, M., Reddi, S., Stich, S., & Suresh, A. T. (2020, November). Scaffold: Stochastic controlled averaging for federated learning. In International Conference on Machine Learning (pp. 5132-5143). PMLR.
[2] Wang, J., Charles, Z., Xu, Z., Joshi, G., McMahan, H. B., Al-Shedivat, M., ... & Zhu, W. (2021). A field guide to federated optimization. arXiv preprint arXiv:2107.06917.

Thank you very much for your careful code review.

weight aggregation can be launched by passing the weights of clients. Please see the code.
Global iteration with learning rate has been implemented in our code base, e.g., SCAFFOLD.

Thanks again for your attention and patience. We appreciate your efforts to improve our repo. Besides, we are also open to code contributions.

Thanks for your response.
For clarity, we use weight to denote the weight of each client's mode parameter when making aggregation (or average) and model parameters to denote the model parameters.

Yes, we can use weighted aggregation on the model parameters by using the code. Yet, we note that for fedavg.py, the repo does not use weighted average (vanilla average with each client's weight equal to each other is used here), since the repo does not pass the clients' weights to the function fedavg_aggregate (see lines 106-109 in basic_server).
For the 2-nd point, I get it well. Yet, I still think it need to be added into FedAvg.

To validate this issue. I modify lines 13-15 in fedavg.py with the code below.
class FedAvgServerHandler(SyncServerHandler):
    def global_update(self, buffer):
        parameters_list = [ele[0] for ele in buffer]
        weights = [ele[1] for ele in buffer]
        serialized_parameters = Aggregators.fedavg_aggregate(
            parameters_list, weights)
        SerializationTool.deserialize_model(self._model, serialized_parameters)

And I get that (the first row is for the original code and the second row is for the modified code) with the unbalanced data.

You can customize these ServerHandler. For vanilla FedAvg, you can use sample number to average the model weights. The code you showed above is what we used for our own experiment, actually.