Instance metrics config change has no effect

Describe the bug
I create a custom mixer telemetry adapter / handler and deployed it on GKE according to this guide https://github.com/istio/istio/wiki/Mixer-Out-Of-Process-Adapter-Walkthrough. It worked fine for a few days but now if I modify the dimensions that I want to forward to my adapter it doesn't seem to apply the changes. Whichever dimensions I add or remove from my yaml file, the exact same dimensions get forwarded to my adapter.
Is there a way to restart the mixer to force it to reload the config properly? Using

kubectl -n istio-system get instances.config.istio.io i1metric -o yaml

I can verify that my config changes in my yaml also get changed on the istio system config. But still it seems to have no effect on wich dimensions the mixer is forwarding to my adapter. I even deleted my handler, rule and instance but still my adapter continued to receive the same instances with the same old dimensions.
I also tried to follow https://istio.io/help/ops/telemetry/missing-metrics/#verify-there-are-no-known-configuration-errors
but setting up the port-forwarding didn't work for me. When I use my browser to go to https://localhost:9093 I only get

Handling connection for 9093
E1130 10:22:38.550617   15191 portforward.go:316] error copying from local connection to remote stream: read tcp4 <IP_ADDRESS>:9093-><IP_ADDRESS>:50672: read: connection reset by peer

Expected behavior
When I add a new dimension to my yaml, that new dimension should be reported through my custom metric to my custom adapter. When I remove a dimension it should not be reported anymore.
Steps to reproduce the bug
I tried adding and removing dimension from my yaml through adding or removing the hashtags in-front of the corresponding dimenion config line:
handler for adapter mygrpcadapter
apiVersion: "config.istio.io/v1alpha2"
kind: handler
metadata:
name: analytics-cloud
namespace: istio-system
spec:
adapter: analytics-cloud
connection:
address: "analytics-cloud.istio-system:8090"
params:
file_path: "out.txt"
instance for template metric
apiVersion: "config.istio.io/v1alpha2"
kind: instance
metadata:
name: i1metric
namespace: istio-system
spec:
template: metric
params:
value: request.size | 0
dimensions:
reporter: conditional((context.reporter.kind | "inbound") == "outbound", "client", "server")
#request_headers: request.headers
#request_path: request.path | "unknown"
#request_host: request.host | "unknown"
#request_referer: request.referer | "unknown"
#request_size: request.size | 0
#request_total_size: request.total_size | 0
#request_time: request.time
#request_useragent: request.useragent | "unknown"
#response_headers: response.headers
#response_size: response.size | 0
#response_total_size: response.total_size | 0
#response_time: response.time
#response_duration: response.duration
#response_code: response.code | 200
#source_name: source.name | "unknown"
#source_workload_name: source.workload.name | "unknown"
#source_service: source.service | "unknown"
#source_version: source.labels["version"] | "unknown"
#source_ip: source.ip #| "unknown"
#source_port: source.port | 0
#destination_name: destination.name | "unknown"
#destination_workload_name: destination.workload.name | "unknown"
#destination_service: destination.service | "unknown"
#destination_version: destination.labels["version"] | "unknown"
#destination_ip: destination.ip #| "unknown"
#destination_port: destination.port | 0
#connection_event: connection.event | "unknown"
#connection_received_bytes: connection.received.bytes | 0
#connection_received_bytes_total: connection.received.bytes_total | 0
#connection_sent_bytes: connection.sent.bytes | 0
#connection_sent_bytes_total: connection.sent.bytes_total | 0
#connection_duration: connection.duration
#context_protocol: context.protocol
#context_time: context.time

rule to dispatch to handler h1
apiVersion: "config.istio.io/v1alpha2"
kind: rule
metadata:
name: rule-analytics-cloud
namespace: istio-system
spec:
actions:

handler: analytics-cloud.istio-system
instances:

i1metric

Version
kubectl client 1.12, server: 1.9+
Installation
downloaded it from git and build it according to the dev guide for custom adapters
Environment
Google Kubernetes Engine

It is surprising that with rule, handler and instance deleted your adapter is still getting report calls. Would you mind to kubectl get -n istio-system {rule, handler, instance} to make sure there is no garbage config left?

@ksemba my instinct is that there is a configuration error in your updated config. If you are not able to get port-forwarding to work for mixer, can you try querying for the metrics via prometheus? We are interested in values for metrics like mixer_config_rule_config_errors_total and mixer_config_rule_config_errors_total and mixer_config_rule_config_errors_total and mixer_config_handler_validation_error_total (though you might just want to look around at all of the mixer_config_* values).
Also, do you see anything in the logs for istio-telemetry?
I'm not clear on the hashtags bit.
I agree with @bianpengyuan that if you deleted the rule for your instance and were still receiving instances, something is likely up -- which may signify config failures.

@bianpengyuan if I try to delete my custom istio config (that yaml deletes the handler, rule and metric instance but not my adapter) I get the following output:

kubectl delete -f kube_istio_deployment.yaml
handler.config.istio.io "analytics-cloud" deleted
instance.config.istio.io "i1metric" deleted
rule.config.istio.io "rule-analytics-cloud" deleted

So it seems that all three objects are deleted. But the adapter seems to still live:

kubectl get adapters.config.istio.io --all-namespaces
NAMESPACE      NAME                      AGE
istio-system   analytics-cloud   12d
istio-system   mygrpcadapter             13d

--> my rule is gone

kubectl get rules --all-namespaces
NAMESPACE      NAME                     AGE
istio-system   kubeattrgenrulerule      40d
istio-system   promhttp                 40d
istio-system   promtcp                  40d
istio-system   stdio                    40d
istio-system   stdiotcp                 40d
istio-system   tcpkubeattrgenrulerule   40d

--> my instance is also gone:

kubectl get instances.config.istio.io --all-namespaces
No resources found.

@douglas-reid I might need some help on you to query those metrics from prometheus. Since the prometheus service has no external IP and I cannot do port-fowarding I tried to use a busybox to run wget queries against its internal service IP. But:

kubectl exec busybox -- wget 'http://<IP_ADDRESS>:9090/api/v1/query?query=up&time=2018-12-01T20:10:51.781Z'
Defaulting container name to busybox.
Use 'kubectl describe pod/busybox -n default' to see all of the containers in this pod.
Connecting to <IP_ADDRESS>:9090 (<IP_ADDRESS>:9090)
wget: server returned error: HTTP/1.1 503 Service Unavailable
command terminated with exit code 1

The example query I used was taken from here: https://prometheus.io/docs/prometheus/latest/querying/api/
The istio telemetry logs (kubectl -n istio-system logs $(kubectl -n istio-system get pods -lapp=telemetry -o jsonpath='{.items[0].metadata.name}') -c mixer) don't show that much. I tried deleting my rule/instance/handler and re-create them and looked at the telemetry logs but saw nothing interesting. When there is telemetry data to deliver to my adapter, it shows that data. the only somewhat interesting logs I found were these:

2018-12-02T18:04:54.610529Z     info    adapters        Cache sync successful.  {"adapter": "handler.kubernetesenv.istio-system"}
2018-12-02T18:04:54.614374Z     info    Cleaning up handler table, with config ID:56
2018-12-02T18:04:54.614677Z     error   adapters        adapter did not close all the scheduled daemons {"adapter": "handler.kubernetesenv.istio-system"}
2018-12-02T18:04:54.614985Z     info    transport: loopyWriter.run returning. Err: connection error: desc = "transport is closing"
2018-12-02T18:04:55.614945Z     info    Publishing 8 events
2018-12-02T18:04:55.627997Z     info    Built new config.Snapshot: id='58'
2018-12-02T18:04:55.628476Z     info    adapters        getting kubeconfig from: ""     {"adapter": "handler.kubernetesenv.istio-system"}
2018-12-02T18:04:55.628669Z     warn    Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
2018-12-02T18:04:55.630127Z     info    adapters        Waiting for kubernetes cache sync...    {"adapter": "handler.kubernetesenv.istio-system"}
GC forced

But they are from yesterday and I didn't modify my setup yesterday...
The commented lines in my kube_istio_deployment.yaml's dimensions section are just to test to de-/activate various dimensions. I'm trying to push more and more data to my adapter in order to parse and process it...

I tried to use the original YAML file that I created when I setup my istio dev environment (copied from the istio documentation) and with that file everything started working again. I used three different file editors to edit that file over time (vi and Visual Studio under Linux and Notepad++ on Windows) and maybe that messed up the file (-format) somehow. Notepad++ shows the file format as Linux (LF) and UTF-8 and I couldn't spot any Windows-specific characters but that is the best explanation that I have.
Since it is now working I tried expanding the list of dimensions that I send to my adapter. Most of them work but as soon as I add one of these, the adapter only receives the "reporter" dimension but none of the others:

source_port: source.port | 0
connection_duration: connection.duration
context_time: context.time

Not sure what's wrong with these dimensions ...
I also added these two to my YAML but they don't get reported (but also don't break the reporting of all the other, working ones):

request_headers, response_headers

@bianpengyuan @douglas-reid @kyessenov I've been playing with the instances that I forward to my adapter by copying the rule that is used within the prometheus adapter into my YAML file:

apiVersion: "config.istio.io/v1alpha2"
kind: rule
metadata:
name: analytics-cloud-http
namespace: istio-system
spec:
match: context.protocol == "http" || context.protocol == "grpc"
actions:

handler: analytics-cloud.istio-system
instances:

requestcount.metric
requestduration.metric
requestsize.metric
responsesize.metric

That seemed to have messed up my environment. Now my adapter only receives instances that have the request.size as value and the reporter as the only dimension. Even after applying the YAML that I used before (which defines lots of dimensions) I only get one dimension.

received request {[&InstanceMsg{Value:&istio_policy_v1beta11.Value{Value:&Value_Int64Value{Int64Value:0,},},Dimensions:map[string]*istio_policy_v1beta11.Value{reporter: &Value{Value:&Value_StringValue{StringValue:client,},},},MonitoredResourceType:,MonitoredResourceDimensions:map[string]*istio_policy_v1beta11.Value{},Name:i1metric.instance.istio-system,}] &Any{TypeUrl:type.googleapis.com/adapter.analytics.cloud.config.Params,Value:[10 7 111 117 116 46 116 120 116],XXX_unrecognized:[],}<PHONE_NUMBER>4566016383}

I then tried to delete the rule, the instance and the handler but my adapter log

kubectl logs -f $(kubectl get pod -l app=analytics-cloud -o jsonpath="{.items[0].metadata.name}" -n istio-system) -c analytics-cloud-container -n istio-system

still shows new instance data coming in when I refresh the bookinfo web page!
Even though the rule that tells mixer to deliver instances to my handler doesn't exist anymore:

kubectl get rules --all-namespaces                                                                                NAMESPACE      NAME                     AGE
istio-system   kubeattrgenrulerule      41d
istio-system   promhttp                 41d
istio-system   promtcp                  41d
istio-system   stdio                    41d
istio-system   stdiotcp                 41d
istio-system   tcpkubeattrgenrulerule   41d

How is this even possible? Cached config that still lives on in mixer?


Can't revert chunk in file open in PR

Open a PR that edits package.json
Make an edit in package.json
Click the git gutter decoration
I am missing the stage and revert buttons on the peek diff

In another file, they show up and work fine

@alexr00, let me know if you need any assistance from me on this. Thanks!

I'm going to turn off the GitHub Pull Request quick diff today and put it behind an experimental setting. I expect that will solve the problem for now.

Did that happen, or should we put this on the milestone?

I made the change and released a new pre-release version of the extension.

Setting is githubPullRequests.experimental.quickDiff and it is off by default. We have no other users of the quick diff API so it should be safe.

If I enable the GitHub Pull Requests quick diff, and make an edit on a line that was already modified in the PR, I still do not see the quick diff revert icon:


git122 update react ui to 3.5

Signed-off-by: David Radley<EMAIL_ADDRESS>

added as requested


fix broken links in forking.md

Looks like when the forking documentation was moved out of README.md and into its own file all the links weren't copied with it.

Good catch. Thanks!


Support for the Opera browser

Is there any possibility of adding support for the Opera browser?

I found out that I can add it through the Chrome store.


Mocks for input errors

Mocks in progress

@ssandbekkhaug Do you have more details on what is needed here, what is missing

I guess the question is:

For each question type in North Star, are we happy with the error UX?
If not, what should the UX be?

If we're happy with the existing UX, we can close this issue.

@ssandbekkhaug cc @sijiayam my understanding is that for if we trigger an error on any question (required but not filled out, incorrect input type, etc) we use this component above that shows a red box, red vertical line along the question and adds red error text below the helper text with "Error: [specific error]." If that is true, then I'm happy with the existing UX and we can close this issue. If not, let's talk about the smallest possible thing we can do here.

@isabelcshaw Agreed. I think it may be easier to test out the current error messages Greg implemented to see if there is anything that is broken UX? And then prioritize and decide if it is a pre-launch or post-launch issue.
@shreyachatterjee00 @nb1701 Could this be a focus of one of the upcoming bug bashes?
I think we can close this issue.


[bug] using lax.cond with partial effect function gives TypeError

In the master branch, the following script shows the error
TypeError: could not merge true_fun and false_fun output pvals: ((ShapedArray(int32[]), None), JaxTuple(*,0)) and ((ShapedArray(int32[]),ShapedArray(int32[])), JaxTuple(*,*)).

lax.cond(True, (0, 0), lambda x: (x[0], 0), (1, 1), lambda x: x)

While these versions works well:
lax.cond(True, (0, 0), lambda x: (0, 0), (1, 1), lambda x: x)

or
lax.cond(True, (0, 0), lambda x: (x[0], x[1]), (1, 1), lambda x: x)

Turns out this was just a typo. #516 should get it.
(I was in Australia for the past week, but I'm back now!)

Thanks @mattjj !


Braintree integration

Need to meet with Braintree team to finalize what the integration is going to look like between v4 and vzero.

[x] API Agreed on
[x] PR Sent
[x] PR Merged
[ ] Discuss simpler api proposal
[ ] Update developer.paypal.com

PR is merged, with promise support -- calling this done! Can raise more issues for future integration bugs but we're most of the way there.


Set AcceptMessages to isRelevant false

After accepting a Propose to cancel we need to filter out the old accept messages

Showing AcceptMessage in own component as a hidden content object, that can be opened by the user.
Show AgreementDataPanel if new agreementData comes in.

Duplicate of #1652


Unexpected content warning

https://oceanplayground.social/@news_unlisted/109349123070354334
This post was given a content warning of "Death", which is configured this way:
[[content_warnings]]
label = "Death"
phrases = [
    "murder", "killing", "death", "suicide", "die", "kill", "killed", "fatal", "genocide", "shooting", "human remains", "homicide"
]

Here's the text of the description + title:

Source: CBC #news #novascotia

Under fire over claims of political interference, Lucki calls for new rules on #rcmp's relationship with Ottawa

#rcmp Commissioner Brenda Lucki says it's time to bring in better guardrails between the national police force and the federal government to avoid future allegations of political interference.

Haven't looked into this since, but it happened again. Here's the text of the description + title:

Source: CBC #news #novascotia
Man arrested after search warrant conducted at Eskasoni residence
#novascotia #rcmp have arrested a man in Eskasoni after a conducting a search warrant at a residence Wednesday morning.
https://www.cbc.ca/news/canada/nova-scotia/eskasoni-man-arrested-search-warrant-1.6756304?cmp=rss


computeDistanceBetween shows aerial distance or road distance?

Summary:
I am using computeDistanceBetween for calculating distance between two points , but the results is much greater than the original results ie. Result is 12627.6 km where it should be 29.4km . And i guess aerial distance is less than road distance.
Steps to reproduce:
providing an example doc
Expected behavior:
Result is 12627.6 km where it should be 29.4km  ie minimum road distance between two points.
Observed behavior:
distance is much larger than expected
Device and Android version:
one plus2 api23
Screenshots:
Can be created by pressing the Volume Down and Power Button at the same time on Android 4.0 and higher.

@sandeep2242 Can you please provide the values you're using as input to the computeDistanceBetween() method?

I have the same issue here. If i use computeDistanceBetween() function it give me random and in accurate values as distance.

@farhanshahid108 could you please provide the input values you're using, the output you expect, and the output you're actually getting?

Salam,
I am using IONIC 3 google maps plugins from official ionic site. According to index.d.ts file (inside product/node_modules/@ionic-native/google-maps) i have import the following ts file into my main home.ts file and code is below.
let sFrom : LatLng = new LatLng(31.5043105,74.3559769);
let sTo : LatLng = new LatLng(31.519611, 74.411572);
var dis = Spherical.computeDistanceBetween(sFrom,sTo);
console.log(dis);


Existing EC2 clusters drop down should only show "running" clusters

Only show ones that are 'running'? mine 'error' a lot but I can still rerun on them.

closed in #385


Change window manager to compositor

I totally understand if this change is rejected but the pedantic in me wanted to bring it up. Tiling Window Manager should arguably be changed to Tiling Compositor.

Fair enough, I hesitated doing exactly that but decided that people would not know. It is better to be correct though, I will change it ;).


Bug 66684:  set the revocationEnabled as FALSE when crl file is empty

https://bz.apache.org/bugzilla/show_bug.cgi?id=66684
Recently, we tested the use of the crLFile configuration （in server.xml） in the scenario where two-way certificate authentication is enabled. When the file pointed to by the crlFile configuration item is an empty file (the file exists but the file content is blank), Tomcat cannot provide services and SSL HandShake reports an error（certificate_unknown）. When crlFile is not configured or the content in crlFile is correct, no error is reported.
I'm thinking that don't need to set the revocationEnabled parameter to true (In org.apache.tomcat.util.net.SSLUtilBase#getParameters 498L) when the file content is blank.

This PR will not be merged. See BZ 66684 for details of why.


What is the difference between BaseModel save() and update()?

If I have saved a model with save() and then change the attributes so I can also use save() again. No need for update().

update() will force an update on the table first. If that fails, it resorts to an insert().
A save() will check using ModelAdapter.exists(Model) first before inserting or updating. This is more efficient for Column.COLUMN_PRIMARY_KEY_AUTOINCREMENT because it simply checks the autoincrementing key on the model if its not null and larger than 0. If its not an AI PK, then it uses its ModelAdapter.getPrimaryWhere(Model) to see if it exists.
I hope this helps.

an update() is much more efficient in large quantities than a save() if you know the data already exists.

@agrosner Thank you for explanation.

Maybe this is something for the wiki.


Remove (flask-)mongoengine support?

flask-mongoengine appears to be dead, and isn't compatible with Flask>=2.3.0.
Should we stop supporting it? What're the alternatives? Is pymongo enough?

The dead frameworks should be removed. But I guess flask-mongoengine which uses MongoEngine is high-level abstraction and implemented with a lot of features. If we only stay with PyMongo, it is perfect itself. How about we remove it in the next release?
Until then, currently, we add a warning message to all MongoEngine users to tell them that we are dropping support for FLask-MongoEngine usage from x.x.x version, and please migrate to mongo or stay with the old version of flask-admin?
But I also want to hear others' opinions about this)

I've gone back and forth on this a little bit.
Given flask-mongoengine appears to be unsupported/unmaintained, is incompatible with newer versions of our core dependencies (eg flask/wtforms), and has no obvious popular replacement (there are a couple, but much smaller community presence), I propose that we remove flask-mongoengine support from Flask-Admin in v2.0.0 onwards.
If flask-mongoengine becomes maintained again, or a fork picks up a clear broad base of support and is brought up-to-date, then I'd be glad and keen to add support back in.


Auto reminder mail system using python

Proposed title of article
Auto reminder mail system using python
Introduction paragraph (2-3 paragraphs):
Maybe you are prone to forgetting to send emails to your co-workers, tenants asking for rent or even monthly subscription pay request email to your subscribers. These tasks can be automated by a system that uses python to setup a reminder and sends it at the time specified.
Python comes with the built-in smtplib module for sending emails using the Simple Mail Transfer Protocol (SMTP). This library ensures secure and safe sending of emails. The library is light ans easy to use.  We are going to take advantage of this library to build a mailing system that sends emails upon schedule.
Key takeaways:

Setting up reminders in python
Using STMP to send secure emails in Python
Sending HTML into Python emails
Reminder mail system using python
Sending bulk emails

References:
Please list links to any published content/research that you intend to use to support/guide this article.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

@Babatunde13
Good afternoon and thank you for submitting your article for publishing.
Your article has been entered into our queue and should be reviewed (and published pending any changes) as soon as a peer reviewer is finished with the ones in the queue before it. 🚀
Thank you.

We do already have an article covering this topic using Django. @jamila-laureen
Please feel free to suggest an alternate topic to explore. 🚀


No such file or directory on AWS EC2 instances

I'm unable to use the action on AWS EC2 instances as runners due to the following error:
ENOENT: no such file or directory, open '/home/ec2-user/.docker/config.json'

They are simple AL2 EC2 instances, nothing special about them yet the action gives an error.
Usage looks like this:
....
      env:
        OCI_CLI_USER: ${{ secrets.OCI_CLI_USER }}
        OCI_CLI_TENANCY: ${{ secrets.OCI_CLI_TENANCY }}
        OCI_CLI_FINGERPRINT: ${{ secrets.OCI_CLI_FINGERPRINT }}
        OCI_CLI_KEY_CONTENT: ${{ secrets.OCI_KEY_FILE }}
        OCI_CLI_REGION: ${{ secrets.OCI_CLI_REGION }}
...
      - name: Log into OCIR
        uses<EMAIL_ADDRESS>        id: login-ocir
        with:
          auth_token: ${{ secrets.OCI_DOCKER_USER_AUTH_TOKEN }}
....

Any ideas?

The action assumes it's running on a standard GitHub-provided runner, with all the tools installed as per https://github.com/actions/runner-images?tab=readme-ov-file#software-and-image-support
You need to ensure that your self-hosted runner has the same tools available, or at least the ones you need (like Docker).

@Djelibeybi The strange thing is that this EC2 instance has docker pre-installed. In fact right after the "login" step, it's building a docker image and it works fine. Not sure what could be the issue yet.

@galovics actually, there is a bug. I check if config.json exists, but not if .docker exists. In the meantime, if you manually create $HOME/.docker before this step, it should work. I'll try and get this fixed properly by next week.

@Djelibeybi awesome, thanks for checking it. I haven't had the time to look into the code yet but I'm glad you've found it. ;-)
Don't rush it, in the meantime I switched over to the regular docker-login action, but obviously it would be nicer to use this for OCI registries.


Cluster with ArgoCD crashes

Describe the bug
Lens shows an error shortly after opening a new cluster with ArgoCD installed
To Reproduce
Steps to reproduce the behavior:

Create a new kubernetes 1.21 cluster
Install argocd on it
Add the cluster to lens and wait fo rthe erro

Expected behavior
I would be able to open the cluster as normal
Screenshots

Environment (please complete the following information):

Lens Version:4.03
OS: Linux Mint
Installation method (e.g. snap or AppImage in Linux): Tried Deb and AppImage

Kubeconfig:
Quite often the problems are caused by malformed kubeconfig which the application tries to load. Please share your kubeconfig, remember to remove any secret and sensitive information.
your kubeconfig here

Additional context
Tried 2 other clusters that works fine.

Turns out not to be related to Argocd. Updating Lens and then it worked.


Channels who are online are reported as offline

[INFO] channel is not online, check again in 1 minute(s)
Debian 12
V.0.94 @YamiOdymel

Still this issue also on Ubuntu anr Debian 11

It still works for me, it's most likely an Internet Provider issue, or the Chaturbate is blocking the program with some Protection
rules that based on your internet environment.
There's nothing I can help you with, maybe try https://github.com/lossless1024/StreaMonitor
Also https://github.com/teacat/chaturbate-dvr/issues/31 is watching 10 streams at a time and had the same issue.


drivers: stepper: tmc: create rampgen Kconfig template

create rampstat Kconfig template to enable respective tmc drivers to reuse the common configurations
PR to facilitate #80784

doc: migration guide: 4.1:add deprecation info of STEPPER_ADI_TMC_RAMP_GEN
                         ^
                         '--- missing space

Already reached 75 characters :)

Already reached 75 characters :)

Is that enforced? I think that's more of a suggestion


continue get changed files Action

Works as expected here https://github.com/fireeye/capa-testfiles/pull/67/checks?check_run_id=1883643726#step:7:10


Create Navbar

Description
As an user I would like to have a navbar to navigate in the app
Acceptance criteria

Navbar should be responsive
Create in into Components folder in Next JS
Use Tailwind CSS

Notes

Good first issue, right now navbar is very ugly

Navbar created:
Desktop view

Phone view closed

Phone view open


feat: add cloud device status for LitterBox

Added the Cloud Status sensor for litter boxes. (Sensor already present for feeders as FeederStatus)
State got from "state.pim" key
This information is already used to make the buttons "unavailable" but was not displayed as a sensor

Hi, please revert the changes that you have made to the strings and English translation file as it will break the integration for feeders. Instead, create the device_status (without the on batteries state) as its own key while keeping the feeder_status as it was.

So yes, I chose to reuse "feeder_status" which I renamed to "device_status" to avoid making the code heavier.
There is actually no "battery" status on the Litter Boxes but that is not that problematic.
What do you think?


add more text to wait for poll results to load

see https://github.com/gridcoin/Gridcoin-Research/issues/459
3 time values are mentioned by now on reddit

#497 makes this obsolete. Poll loading is pretty much instantaneous now.


[INFRA] BIDS needs logo

There is no logo for organization, some academic hat in the otherwise beautiful  https://bids-specification.readthedocs.io/ (makes hard to locate the tab with bids among other 10s of tabs ;)).  So someone with artistic skills should give it a shot.  But since it is probably worse to file an issue and not to suggest some version, here is my humble ugly attempt

I think you could guess on what I hoped it to look like, although someone might argue it is more of a burger on a stick ;)

+1 for this idea, and thanks for a draft!
I remember that this issue came up before, see: https://github.com/bids-standard/bids-validator/issues/492
However as far as I know, it never lead to anything serious. Perhaps we can have a collection of logos and then vote on which one we want?
To consider:

we probably can't pay anyone money for this
it should have an appropriate license for us

I have reached out to an artist that I know, perhaps she'll give it a shot just for fun: Then we'd already have two drafts :-)

Following up on this - I have just started a design competition to create our BIDS logo!
The competition can be found here: http://99d.me/c/jmvi
Feel free to pass around if you may know anyone interested in submitting!

We are starting to get some entries. They can be seen here: https://99designs.com/logo-design/contests/design-brain-imaging-data-standard-logo-916110/entries
let me know if there may be thoughts on the designs

Amazing we have so many options so quick!
While many are simply pleasing to the eye, it would be magic if it could actually have some meaning, link to the content/concept of the project.  For that reason, I'm draw to the direction of
https://99designs.co.uk/logo-design/contests/design-brain-imaging-data-standard-logo-916110/entries/12
The brain outline is awful, I'll admit, but imagine a half-hemisphere that quickly reminds you of a brain, and an iconic representation of metadata, i.e. dotted lines emerging from the other half.
If we had something snappy along those lines that'd be cool.

+1! I agree that the file structure needs to be incorporated somehow. I have been submitting feedback to the entries to please integrate that into their design (12 is only one currently to do so, but there will be resubmissions)

yeah, if 12 may be flipped horizontally and had more obvious file hierarchy tree - could be the one.
Which font is that though? we don't want to end up with some proprietary one used/embedded into the end product

the designer resubmitted the design as 42, I think it is getting closer. The file structure seems present (potentially reducing one of the lines more to provide more leveling) the flipping I am not sure.
The font should be fine, we will be the owners of the copyright at the end of it so everything that makes it up should be able to do that.

Design 45 is pretty cool too!
On Thu, May 16, 2019, 11:59 AM Franklin Feingold<EMAIL_ADDRESS>wrote:

the designer resubmitted the design as 42, I think it is getting closer.
The file structure seems present (potentially reducing one of the lines
more to provide more leveling) the flipping I am not sure.
The font should be fine, we will be the owners of the copyright at the end
of it so everything that makes it up should be able to do that.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bids-standard/bids-specification/issues/216?email_source=notifications&email_token=AAB2JJ2P44LHLQ5YHLVWNT3PVWVIDA5CNFSM4HJESWC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVSYDAA#issuecomment-493191552,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB2JJ5RKOZ5LGRDS4I5AR3PVWVIDANCNFSM4HJESWCQ
.

Since this is getting serious, I am making some more people aware of this issue: @bids-standard/bep_leads

I also vote for the idea of have a file structure! 78 for looks even better than 12/42 IMO

I like 78 too! @CPernet

78 better brain, but the file structure seems more elegant in 42 I think

#78 !  better brain, better files, from brain (DICOMs) to files - left to right.  good
I like and dislike (there is nothing special about D) notch on D

re #78 -- is there a way to verify provenance on the brain/font?  I mean if brain is not drawn by the artist, it could be covered by a copyright/license etc

Thank you for your feedback! I agree 42 and 78 are looking very nice.
Regarding the timing - there is about 1 day left of the first round. Then finalist (I believe 6) are put into the final round for 3 days. Then 14 days to pick the winner. Once the winner is selected the handover begins (and the discussion of @yarikoptic concerns of copyrights/licensing)

I like 78 conceptually and aesthetically, but the brain looks a little funny to me—it brings to mind a sponge more than a brain. I think my current favorite is 50, which looks great and IMO is sufficiently abstract that the same graph-y object can be interpreted as both a brain and a filesystem (aesthetically, I like some of the same artist's other submissions even better, e.g., 55, but I think 50 really nails the idea).

I do like the idea that we might brand a conference room window with this.

For me, 45 and 78 go far ahead. It must be that my sponge is blind to that brain...
I think 50 is pretty elegant, but I don't see the filesystem side of it.
Please don't 42 and 55.

it brings to mind a sponge more than a brain

and the difference is (especially as we grow older)...? ;)

I think my current favorite is 50,

I didn't search for them but both 50 and 55 (as many others) gave me the feeling of some used with the brains design pattern already ... to me 50 also doesn't quite actually point to "filesystem" concept unless you take it as your prior to look for.

78 is on point but clunky.  I'm drawn to the brain-out-of-folders (eg 64) but worry it won't read well at varying scales (i.e. especially small ones).
I keep coming back to 45... the brain in a folder... if the folder shape is sharpened I think it'll read well at a variety of scales.  Just don't like the pointy brain stem.

I prefer 45 to 78. Agree re: the pointy brain stem, but I assume once we acquire it, we can do whatever we like, and it would be trivial to remove the pointy part outside the folder.

@franklin-feingold

Once the winner is selected the handover begins (and the discussion of @yarikoptic concerns of copyrights/licensing)

What if the discussion ends quickly with "yes, I copied it without looking at the license, which happens not to allow it"? That would be a big deal.

I suspect fraudulent activity would probably have negative consequences for the designer and we'd get our money back.
Also: Who purchases stock art, the customer or designer? When?

Okay, since the logos don't have any watermarks we can assume they are original, correct?

are we supposed to vote here or on 99designs?

are we supposed to vote here or on 99designs?

@franklin-feingold --> can you answer this question? Also, once we have the six finalists we could consider pinging the mailing list to get some more votes from the community - what do you think?

I'm also a bit confused as to where to vote or find this discussion.
This would be my ranking:
Choice 1: 12 good because the brain is at the letters side, reads good,
simple.
But font can be bit less bold, bit better
Choice 2: 88 / 78
Choice 3: 50 best font
Choice 4: 86 53 49 bit too modernistic/unclear
Best wishes/hartelijke groet,
Henk(-Jan) Mutsaerts, MD PhD
Amsterdam UMC/ UZ Ghent
Phone: +31 6 4390 8284; Skype: hj.mutsaerts
On Sat, 18 May 2019 at 01:25, Stefan Appelhoff<EMAIL_ADDRESS>wrote:

are we supposed to vote here or on 99designs?
@franklin-feingold https://github.com/franklin-feingold --> can you
answer this question? Also, once we have the six finalists we could
consider pinging the mailing list to get some more votes from the community

what do you think?

—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bids-standard/bids-specification/issues/216?email_source=notifications&email_token=AGT42LT27U5YDRPAGCXILIDPV6HLXA5CNFSM4HJESWC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVWH6WI#issuecomment-493649753,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGT42LQM4NQPTTPY6ZR6CXDPV6HLXANCNFSM4HJESWCQ
.

Oh wow this one evolved pretty nicely!
https://99designs.com/logo-design/contests/design-brain-imaging-data-standard-logo-916110/entries/88
On Sat, May 18, 2019, 12:26 PM Henk Mutsaerts<EMAIL_ADDRESS>wrote:

I'm also a bit confused as to where to vote or find this discussion.
This would be my ranking:
Choice 1: 12 good because the brain is at the letters side, reads good,
simple.
But font can be bit less bold, bit better
Choice 2: 88 / 78
Choice 3: 50 best font
Choice 4: 86 53 49 bit too modernistic/unclear
Best wishes/hartelijke groet,
Henk(-Jan) Mutsaerts, MD PhD
Amsterdam UMC/ UZ Ghent
Phone: +31 6 4390 8284; Skype: hj.mutsaerts
On Sat, 18 May 2019 at 01:25, Stefan Appelhoff<EMAIL_ADDRESS>wrote:

are we supposed to vote here or on 99designs?
@franklin-feingold https://github.com/franklin-feingold --> can you
answer this question? Also, once we have the six finalists we could
consider pinging the mailing list to get some more votes from the
community

what do you think?

—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/bids-standard/bids-specification/issues/216?email_source=notifications&email_token=AGT42LT27U5YDRPAGCXILIDPV6HLXA5CNFSM4HJESWC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVWH6WI#issuecomment-493649753
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AGT42LQM4NQPTTPY6ZR6CXDPV6HLXANCNFSM4HJESWCQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bids-standard/bids-specification/issues/216?email_source=notifications&email_token=AAB2JJZTX744WNT64C42LLTPWBJ5DA5CNFSM4HJESWC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVWUQLY#issuecomment-493701167,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB2JJ5H5HYBYUEGM2RCUQ3PWBJ5DANCNFSM4HJESWCQ
.

My favorites are 12/42. Prefer 42, because the type is more interesting. If we get the logo as vector paths instead of type, fonts shouldn't be an issue
I agree that the brain on 78/88 looks a bit off. Prefer 78, because the details in 88 will make it difficult for the logo to work well when reproduced at small sizes)

@chrisgorgo I like #88 but the brain there is a bit weird. The rest is well developed, but don't we think the brain picture there can look a bit squished? :-)

I like #88 but the brain there is a bit weird. The rest is well developed, but don't we think the brain picture there can look a bit squished? :-)

It's reused from some public library of vector images (I saw it before somewhere). We can always ask candidates to refine the design in the second phase. This is the procedure: https://support.99designs.com/hc/en-us/articles/204109459-How-long-do-design-contests-take-and-what-s-involved-

Regarding voting - we can vote here or on 99designs, I am collecting feedback for both. Please feel free to continue listing the numbers I can connect it to the designer. We choose the top 6 designers (not concepts) to move forward to the final round. The top 6 need to be chosen by noon PST Wednesday (5/22).
For choosing the final design we can run a poll to share on our different mediums to collect feedback for 1 week before a final decision is made. We are going to have 3 days to move toward a final design to vote on and it appears we will get the original files so they can likely be touched up if need be.

I like 88, 78, 72, 71, 45 because they relate somehow to files/folders.
I like the colors in 9, 3, and 3.

Cool stuff. I like 89, 88, 78, 73 most. :)

Thank you all for your feedback, please feel free to continue submitting! We have a few more days before choosing the finalist! I have taken a tally and here is where we are at -
We currently have identified 6 designers with potentially multiple different designs:
Johnsmithaps, Nilangle, Feliana, Pixeleiderdown, Design Channel, and Katsuo

The only ones I thought were decent were

nilangle (12, 42, 104), and I preferred the font in 12 to that in 42/104, but liked a little bit the non-uniformity of the "files" in 104.
johnsmithaps (78, 88), though the lines sticking into the brain kind of break the directory structure appearance.

In both cases, the brain could stand a shape and sulcus pattern that looks a little more like a real brain.
103 would make a good logo for an anarchist chat app.

The final round has begun with the 6 finalist I mentioned previously

Johnsmithaps (88, 78, 45), Nilangle (42, 12), Feliana (50, 55), Pixeleiderdown (86, 53, 49), Design Channel (71, 72, 73), and Katsuo (89)

Please feel free to continue giving feedback and comments that I can pass onto the finalist! We have 3 more days to tinker with the design before the final polling begins next week to make the final decision!

FWIW I think, of this lot, there are my favorites: 88, 45 (though still
needs work) and 72.
I like the brain graph of 72 but like the typography in 45 more.  88 is
undeniably the most literal spot-on one (and good typography) ... but
something about the 3 heavy lines and the half brains leaves me not
enthusiastic enough to rule out the others.

Update on the competition - we are ready to vote for our winning design! I have compiled all of our feedback into this survey to capture our voting. In this round we are voting on our favorite design concept, we can still work with the winning designer to make small adjustments.

1 - means "most preferred" and 5 "least preferred" or the other way around?

Yup! 1 - most preferred 5 - least preferred.

Are these designs final or can they be tweeked? If https://99designs.com/logo-design/contests/design-brain-imaging-data-standard-logo-916110/entries/158 wins we should modify it a bit so the folder/file hierarchy would be realistic (right now it shows a folder being a child of a file).

These designs can be tweaked (most of our designer finalists are open to continuing the modifications if selected), the final logo will be modified (if needed) to be realistic

Now we do have a logo, see: https://twitter.com/BIDSstandard/status/1136715977737924608
Next we should

provide the logo as .png and as a vector graphics format
include the logo on our specification (favicon, and others)
put the logo on our slides
...

tracking BIDS logo distribution - https://github.com/bids-standard/bids-specification/issues/244


the /analytics and the /summary links are not working from the dashboard

Describe the bug
the links the /analytics and /summary are not working from the dashboard menu
To Reproduce
Steps to reproduce the behavior:

Go to 'dashboard'
Click on 'analytics/summary from the menu'
See error

Expected behavior
the analytics and the summary links should have redirected to their routes
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS] MacOs
Browser [e.g. chrome, safari]Chrome
Additional context
this might be due to the absence of the required data models to perform labelling analysis. Trying to find that out.
working on it!

well i found out that there is no apparent page where those links might lead to and hence no links have been provided and the best option is to go to the landing page... i might need suggestions as to which pages should these links lead to @Tushar19varshney

@Tushar19varshney can you please guide me as to which pages should these links lead to?

can someone please guide me on this?

Closed via #260


Table colors are broken in Codespaces after initial start

Any ideas what it can be? How do we calculate effective colors, what do they depend on?

Extension version: 0.2.20
Full page reload helps fixing table colors, but breaks icons (no DVC extension icon, no experiment circles, etc):

And only reloading one more time fixes everything 🤷

Closing this, since I doubt we can do something here. Let's wait for a while. And it's not critical.


Evaluate selection (Ctrl+Shift+E) does not parse underscores

Tested versions
v4.3.1.rc.custom_build [d40fc50f0]
4.4 on latest master
System information
Godot v4.3.1.rc (d40fc50f0) - Manjaro Linux #1 SMP PREEMPT_DYNAMIC Wed Aug  7 16:19:28 UTC 2024 - Wayland - Vulkan (Forward+) - dedicated AMD Radeon RX 7900 XT (RADV NAVI31) - AMD Ryzen 9 7900X 12-Core Processor (24 Threads)
Issue description
In the script editor, selecting an expression and pressing the shortcut for Evaluate Selection does just what the name suggests, e.g. selecting 10+24 will transform the selection into 34. However, for larger numbers, or any number where you put underscores for readability, e.g. 500+10_000, all characters starting from the underscore are ignored, which results in 510 instead of 10500.
Steps to reproduce

Type an expression in the script editor, with one number containing an underscore.
Press the Evaluate Selection shortcut (or select it from the context menu).
The result will not consider any characters after the underscore and will simply remove them.

Minimal reproduction project (MRP)
N/A

This is a limitation in the way this class parses things, it's very basic, in the same way "1_000".is_valid_int() returns false, and that's where things would need to be improved for this to work
But this is quite an advanced part of formatting

Worth noting that #97218 uses the expression parser as well, so it will be effected by this as well.


v4: onResult is called on error, but not if setting errorPolicy is none (which should be default)

Describe the bug
When running a query using useQuery you get an object containing the onResult function. Using this function you can register function to be called - well - on a result I thought. Following the typings, I happily went on, until I discovered that the data property sometimes is null ... which - according to the docs (https://www.apollographql.com/docs/react/data/queries/#result) - should only happen if the value is set to another value but none. According to the docs, none is the default value, which seems to be different on this package.
I suspect these lines here to be responsible:
https://github.com/vuejs/apollo/blob/63675b624d44167fe5f01c0636d03ac573019e55/packages/vue-apollo-composable/src/useQuery.ts#L287-L289
'Cause they do have a difference if the property errorPolicy is set or not. I've tested it, and in my tests this property was undefined, which is why I get a call on my function registered at onResult.
To Reproduce
const { onResult } = useQuery(
  gql`
    {
      queryThatDoesNotExistOnTheServer
    }
  `,
  // { errorPolicy: "none" } // <-- Apply this line and you'll see that the function registered at onResult is not called
);

onResult((queryResult) => {
  console.log(queryResult)
});

Expected behavior
The function registered at onResult should only be called if there is a valid result - as described in the manual: https://v4.apollo.vuejs.org/guide-composable/error-handling.html#error-handling
In fact, if I explicitly set the errorPolicy to none, it works as expected.
Versions
vue: 3.2.29
@vue/apollo-composable: 4.0.0-alpha.16
@apollo/client: 3.5.8
Additional context
Is there any way to set a general errorPolicy for all queries?

If someone else stumbles upon this, we're currently using the following work-around:
const { result } = useQuery(
  gql`
    {
      queryThatDoesNotExistOnTheServer
    }
  `
);

watch(result, (value) => {
  console.log(value)
});


Fix X-Registry-Auth header

Current implementation of RegistryAuth doesn't work, because it should be encoded in url_safe base64 variant.
Relevant docs: https://docs.docker.com/engine/api/v1.41/#section/Authentication

*nice

*nice


feedback on architecture diagram

This comes from discussing the architecture diagram on https://azure.github.io/az-hop/ with an IT professional from a customer

If possible, it would be useful to use color to indicate not just what is optional, but also what is part of the public/locked-down scenario to make the security aspect easier to understand

For every component outside the hpcvnet box that is connected via a line to a component inside, we should indicate whether the connection can be made via a private endpoint in the lockdown scenario

The rationale for drawing components outside or inside the hpcvnet box is not 100% clear to me (e.g. why does the box in the bottom left memtion CycleCloud?). I guess the rationale is that prepackaged services are drawn outside the box (but bastion & vpn gateway are also services? + they also have associated public IPs that are not drawn)

Some aspects of the drawing are outdated

My = Mariadb
Lustre & connected services are optional

The connection to the enterprise firewall could mention expressroute (?)

@xpillons if you have a visio diagram of the drawing, it might make sense to add it to the repo as well so people can adapt it to their specific deployment for talks with IT security

Yes good point, I will make the visio file available

@ltalirz please check the visio file here https://github.com/Azure/az-hop/blob/main/docs/files/azhop-public.vsdx

Thanks a lot @xpillons ! I won't make it this week but will check back next week


what is this reducer.options({   payload: false }); doing ?

Sorry, but I didn't understand how to use that.
Can you please give an example ?
Thanks!

Sure. Let's say you do the following:
// This is an action creator
const increment = createAction();
// Which is actually just a function returning the action object when called
const action = increment(42);
// The action will look like
// { type: '[1]', payload: 42}

If we were doing standard redux, you would then do:
function reducer(state, action) {
  switch(action.type) {
    case '[1]': return state + action.payload;
  }
}

But using redux-act, we assume that you want to let the lib handle the type on its own so it will extract the payload and give it to you directly inside the reducer:
const reducer = createReducer({
  (state, payload) => state + payload
})

So you do not have access to the full action object, just the data inside its  payload key. It should be enough in 90% use cases. If, for whatever reason, you need the full action object, you can disable the payload extraction by setting the payload option to false.
const reducer = createReducer({
  (state, action) => state + action.payload
});

reducer.options({ payload: false });

All the samples are performing the exact same operation inside the reducer. Good enough?

Yes, Thank you! now I get it 👍


updated web frontend breaks akai sampler cdrom support

Info
RaSCSI version: 21.10.1 cf9a578
Describe the issue
in the updated web frontend it is no longer possible to load an .iso as a .hda,  since it now automaticaly detects the image type.
for akai samplers this is a problem, loading an iso as a cdrom device does not work.
in the old version i could load any file as any device type, so i simply loaded my akai cdrom .iso file as .hda
i have found a  workarround: just rename the .iso files to .iso.hda

@ewanuno May I ask which version of the RaSCSI software you were using prior to this? Even the previous stable version, 21.05, had device type auto detection based on file ending, so I'm just wondering how far back I should go to see what the old workflow was like.
Anyhow, the Akai scenario is a very unusual one, and we have deliberately decided to not enable any image to be attached as any device type in the Web UI, since it would open up for user error that could easily lead to data loss. Would it introduce a lot of inconvenience for you to use the renaming workaround moving forward?
BTW, just for the record, your workaround is already documented on the Samplers wiki page in the Akai section.

@ewanuno One thing that you can try, if you'd like to help us understand the needs of the Akai platform, is to try mounting the iso as a CD-ROM drive with 512 bytes block size. By default, iso images are mounted with a 2048 bytes block size (with is the standard for CD-ROM). By my understanding, one of the quirks of the Akai is that it expects either 1 or 512 bytes block size, which is non-standard. Do this on the command line:
rasctl -i 0 -c attach -b 512 -f /path/to/iso
The above assumes SCSI ID 0. Change the -i parameter if needed.
If the above allows you to use the iso on the Akai, I can look into a solution in the Web UI.

Closing, since workaround exists, and changing the default behavior would make the UX of the web UI less convenient for most users and open up for incorrect operation.
There is always the option to create a Web UI specifically for Samplers, which is something that has been discussed in the past.

This is a shame, I ran into the same issue. I think a lot of people are using these with Akai samplers. I would have thought having a dropdown box in the GUI with some options for block size of ISOs that defaults to 2048 would be acceptable. Appreciate it is a corner case.
I'd be happy to do a pull request but if you think it's not a good idea I'll leave it alone.

@hayesey So I've gotten conflicting messages on the requirements for Akai. When discussed on Discord last year, someone told me that it's not enough to mount a CD-ROM device with 512 block size and attach the iso image, but that the iso image actually has to be mounted as a hard drive for the host to recognize it. Is this not in line with your experience with Akai devices?

@rdmark this command you posted earlier on:
rasctl -i 0 -c attach -b 512 -f /path/to/iso
Works, I ran it with 6 different Akai Sample CD ISOs earlier on with my Akai S3000xl. Refreshing the web UI shows them as CDROMs.
Could well be different on the older samplers such as 950 or 1000 (in fact I'm not sure those even have SCSI interfaces though).

@rdmark Ok this is good to know.
Just to throw an alternative idea out there: How about we introduce a new "bespoke" CD-ROM profile, with an arbitrary vendor/product name and the block size set to 512?
{
    "device_type": "SCCD",
    "vendor": "RaSCSI",
    "product": "SCSI CD-ROM",
    "revision": "",
    "block_size": 512,
    "size": null,
    "name": "Generic CD-ROM 512 block size",
    "file_type": null,
    "description": "Used to mount iso images to hosts that expect block size 512, such as Akai samplers.",
    "url": ""
},

Then you'd go though the bespoke device UI to generate a properties file for each of the iso images that you use with Akai. Thoughts?

@hayesey Would you be able to test this PR? https://github.com/akuker/RASCSI/pull/808

@rdmark I can confirm that this change works. Creating a Named Drive from an existing uploaded ISO using this template creates a CD-ROM with 512byte block size and it successfully loads on my Akai S3000XL.

Thanks for testing! I'll get this added to the next release.

The change is in develop


refac: replace builder classes with create methods in AppBio QuantStudio

Replace builder classes with create methods in AppBio QuantStudio structure where possible
Add utils to get data from pandas series

Close current PR to create requested smaller PRs


Artist Extra's Artwork assumes perfection to match

While working on beefing up the unit tests, it becomes clear that if artist replacement is in place to fix things, the new artist name may not be in the imagecache.  So there needs to be some way to match the original artist name with the desired one.  But that may not happen until significantly later.  There is also the conjunction problem on top of it.
It may be worthwhile to add another table into imagecache that maps artistnames -> id, and then the actual uses uses that id to identify the artist.

4.0.3 now includes code to write to the imagecache as the normalized artist name in the metadata at the time of execution. Additionally, a specific check for excess the's are checked. Impact, using Discogs as the artist service:

Track Metadata
Discogs
Imagecache
Images Will Work?

Pixies
Pixies
pixies
Yes

The Pixies
Pixies
thepixies
Yes

Prince & The Revolution
Prince and the Revolution
princetherevolution
Not entirely

Prince and The Revolution
Prince and the Revolution
princeandtherevolution
Not entirely

The Prince example also causes the image data to be downloaded twice.
Audio recognition runs before artist extras code, so audio recognition should guarantee some level of consistency.  Musicbrainz artist IDs, as usual, provide the best ability to match for those services that use them.
For 4.1.0, I'm looking at implementing a much more complex matching system involving a secondary index.  Primary imagecache database will use an internal id instead of the normalized name.  The secondary index will map normalized names for all services and musicbrainzids to the internal id.  There is also the possibility to just use MBIDs as the internal id to save a lookup.

Code change is in 4.1.0-rc1


Add readme for filepath-input-tool-showcase example to enable CI.

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
All Promptflow Contribution checklist:

[x] The pull request does not introduce [breaking changes].
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

This change need wait for new release of my-tools-package to pass the ci: samples_flows_standard_filepath_input_tool_showcase_readme


feat #7: Implemented new export in XlsExporter that uses StreamExporter2

Extracted code to minimize code duplication. Implemented a new export that takes Map<String,String> instead of Properties.

Tests pass


Handle github profiles that don't have real name filled out.

Right now we only assume that github user has a real name in their profile. Because not every github user has filled out their real name, we need to also have a column for nickname.
Right now also if the user does not have a real name, sql will throw an error when such user is attempting to register because the column 'name' is not allowed to be nullable

Thank you, Andre,
I think we must do these steps :

add username  to the user and store GitHub nickname as username
make the name property on the users table optional
if your name is empty show username instead ( do this on the model by Attribute )

I think we only need name to be the username and that's it, we'rnt a social platform.

I think we only need name to be the username and that's it, we'rnt a social platform.

I agree with this.


Could not find a declaration file for module '@tauri-apps/api/primitives'.

I'm not sure how that is related to this repository?
Anyway, in your tsconfig.json file try changing the moduleResolution setting to "Bundler" (only required until the next api release).
If that doesn't fix it please post the output of pnpm tauri info


App shows "There's no previous chapter" even if there is a previous chapter.

Steps to reproduce

Open a manga with more than 1 chapter
Scroll up

Expected behavior
The previous chapter should start loading.
Actual behavior
The app says that there is no previous chapter even if there is.
Crash logs
No response
Mihon version
0.16.1
Android version
Android 14
Device
Nothing Phone (2)
Other details
This bug is a mihon exclusive one as tachiyomi 0.15.3 seems to work properly.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.1.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

I've tested more and it doesn't seem to happen in all mangas, only some.

I've tested a bit more and it seems to ignore read chapters, closing issue as it isn't an issue really.


Include version number in README of each crate

The README of each crate should mention the version number at the end. This makes it possible to reverse-lookup into what release a crate version was included.

What's the plan for this? A magic sed is the only way I can think to do it but that feels messy and easily breakable if some one changes the readme format unknowingly.

What's the plan for this? A magic sed is the only way I can think to do it but that feels messy and easily breakable if some one changes the readme format unknowingly.

Hm not sure i get what you mean, so maybe Im oversimplifying it but can we do something like this?
echo "\nThe corresponding Polkadot-SDK release for this version is: $VERSION" >> README.md

I guess that works if we want it right at the bottom of the readme.

I've written it so it appends
## Release

{description}

where description is something like Polkadot SDK v1.2.3.
It's this way because the tool should be repo agnostic so I don't want to hardcode polkadot into the sentence and I generally prefer readme's to be less wordy.
Let me know if this wording works for you.

Yep sounds good, thanks! Then what do we do with the crate description?
crates-io only shows the latest description which is then wrong for old versions. Maybe we should remove it to avoid confusion?


Clients options protocol

Shouldn't we precise both argument available 'http:' and 'https:' for the client options protocol ?

Yes, thanks


UX/Decision Points on Gathering Records

Title of ticket: UX/Decision Points on Gathering Records
Description
Points from Demo on December 6th about Gathering Records
Dependencies
Are there any dependencies?
DOD

[ ] How far down will be go in emails with attachments?
Users have concerns as there are typically multiple layers of emails with attachements with further embedded attachments.
[ ] Do we have a way we can flag documents that are password protected?
This is a current pain point for users and would be helpful if we can flag this in our system and generate a comment tagging the Ministry Assigned Users so they can go back to the division asking for the passwords.
[ ] Resource Ministry has flagged that 500MB is to small for them, what is high end of the limit that we are comfortable with that won't impact the loading speed?
[ ] We need to have an error log generated - similar to file watcher - for users to understand issues so they can troubleshoot

Technical Questions from David Handysides (Could be good to book a more technical meeting with him to ask any questions he has about system, was suggested in Demo that we should off-line with him)

[ ] Does Object Storage Compress everything stored in it?
[ ] If so how do we expect decompression speed to impact users?
[ ] What is the dely we would be looking at based on file size?

This ticket can be closed - al decisions and designs were made in other tickets


Bucket duplication when loading the blimp with water.

Greetings.
I have noticed that when a player loads the blimp with water from a bucket, an extra empty bucket appears in the inventory. This isn't a great issue, but I would like it to be solved (and I did solve it with this one simple trick discovered by a mom).
Solution: remove or comment out line 78 in the file fuel_management.lua in order to fix this behavior.
I couldn't push the changes to this repository on my own for some reason, so I simply opened an issue here.
I hope you'll find this helpful (if the bucket duplication isn't an intended feature).

right, I'll check it when I'm at home tonight! thank you!

Fixed! Thank you very much!


Proxy listening on another localhost address

Is is possible to let the proxy listen on <IP_ADDRESS> instead of <IP_ADDRESS>? When I specify that in emailproxy.config and start emailproxy, it just listens on <IP_ADDRESS> and changes the specification in the config file.
To test if somehow <IP_ADDRESS> is invalid, I tried with netcat (using the same credentials as emailproxy):
netcat -l <IP_ADDRESS> 1024
and that worked. Using telnet I can connect to <IP_ADDRESS>:1024, but not to <IP_ADDRESS>:1024
Reading through emailproxy.py I cannot find where <IP_ADDRESS> is enforced.

Did you read the configuration documentation?
Set local_address = <IP_ADDRESS> and the proxy will listen on the IP address you are wanting.


ROX-18892: increase k8s-client retries

Description
In https://issues.redhat.com/browse/ROX-18892 I encountered a test flake due to connection failure.
Wrapped by: java.io.IOException: Failed to connect to /<IP_ADDRESS>:443
    at orchestratormanager.Kubernetes.ensureNamespaceExists(Kubernetes.groovy:158) [6 skipped]

Based on @gavin-stackrox, the root cause is a GKE CLUSTER_RESIZE:
$ gcloud --project=stackrox-ci container operations list | grep<PHONE_NUMBER>041020928
operation-1693321873247-a322b918-4d74-42b4-b71a-9a396906d157  CREATE_CLUSTER  us-central1-b  rox-ci-qa-e2e-test-1696538173041020928                    DONE     2023-08-29T15:11:13.247538598Z  2023-08-29T15:23:38.082118456Z
operation-1693322622916-bf195eb3-0150-4e20-8d31-6e7434ad02db  UPDATE_CLUSTER  us-central1-b  rox-ci-qa-e2e-test-1696538173041020928                    DONE     2023-08-29T15:23:42.916069472Z  2023-08-29T15:23:43.034643792Z
operation-1693327171096-4287b6e6-63b2-4284-b67b-6735cb9e606f  RESIZE_CLUSTER  us-central1-b  rox-ci-qa-e2e-test-1696538173041020928                    DONE     2023-08-29T16:39:31.096328792Z  2023-08-29T16:44:18.497304437Z
operation-1693329499583-13af7fb1-6d93-446e-9276-b7b2baa93e58  DELETE_CLUSTER  us-central1-b  rox-ci-qa-e2e-test-1696538173041020928                    DONE     2023-08-29T17:18:19.583554519Z  2023-08-29T17:23:23.390490672Z

Therefore increase the k8s-client retries to cover such unavailability events.

Thanks @stehessel Can you do 1 run with ci-all-qa-tests for a bit more coverage? thanks.

/test all

Can you do 1 run with ci-all-qa-tests for a bit more coverage? thanks.

Do I need to do anything other than set the label? It doesn't actually start any more tests.

/test ocp-4-13-qa-e2e-tests

/test ocp-4-10-qa-e2e-tests

/test rosa-qa-e2e-tests

/test aks-qa-e2e-tests

/test osd-aws-qa-e2e-tests

/test osd-gcp-qa-e2e-tests

The failures are known flakes and seem unrelated to the k8s retry to me. In general we might have to add more explicit retries for specific POST calls, I'm not sure they are retried by default. But hopefully this will already help.


Checkbox group disappears

The first time the config page is shown (after an installation) the checkboxgroup is correctly displayed.
If I close and open the config page again, the item is not visible anymore...
Probably related with the \0 separators...

And every items under my checkboxgroup is also hidden (My save button for example)

You are correct. Thanks for catching this


exited: php5-fpm (exit status 70; not expected)

Some quick research pulled up this: https://github.com/kasperisager/php-dockerized/issues/20

2016-06-13 03:10:54,123 CRIT Supervisor running as root (no user in config file)
2016-06-13 03:10:54,123 WARN Included extra file "/etc/supervisor/conf.d/startup.conf" during parsing
2016-06-13 03:10:54,146 INFO RPC interface 'supervisor' initialized
2016-06-13 03:10:54,147 CRIT Server 'unix_http_server' running without any HTTP authentication checking
2016-06-13 03:10:54,148 INFO supervisord started with pid 1
2016-06-13 03:10:55,151 INFO spawned: 'nginx' with pid 8
2016-06-13 03:10:55,154 INFO spawned: 'mysqld' with pid 9
2016-06-13 03:10:55,156 INFO spawned: 'php5-fpm' with pid 10
2016-06-13 03:10:55,248 INFO exited: php5-fpm (exit status 0; not expected)
2016-06-13 03:10:56,250 INFO success: nginx entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2016-06-13 03:10:56,250 INFO success: mysqld entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2016-06-13 03:10:56,252 INFO spawned: 'php5-fpm' with pid 37
2016-06-13 03:10:56,289 INFO exited: php5-fpm (exit status 70; not expected)
2016-06-13 03:10:56,292 CRIT reaped unknown pid 40)
2016-06-13 03:10:58,296 INFO spawned: 'php5-fpm' with pid 41
2016-06-13 03:10:58,322 INFO exited: php5-fpm (exit status 70; not expected)
2016-06-13 03:10:58,324 CRIT reaped unknown pid 43)
2016-06-13 03:11:01,330 INFO spawned: 'php5-fpm' with pid 44
2016-06-13 03:11:01,359 INFO exited: php5-fpm (exit status 70; not expected)
2016-06-13 03:11:01,361 INFO gave up: php5-fpm entered FATAL state, too many start retries too quickly
2016-06-13 03:11:01,362 CRIT reaped unknown pid 46)

The issue is that PHP FPM is daemonizing, which it must not with supervisord.
This'll fix it:
sed -i 's/;daemonize = yes/daemonize = no/g' /etc/php5/fpm/php-fpm.conf

This worked, has anyone had a chance to push this the repo?


Can't save palettes since Sketch 49 update

Hi! Love your plugin. I use it frequently and more frequently since I've been creating a design system for my company. Thank you so much.
Since Sketch updated to version 49, I'm no-longer able to save palettes. I can still load them and clear them. Any ideas?

Yeah, I just noticed this as well. I think it only happens for palettes with gradients. Regardless, I plan to fix it soon.

Thanks so much man. I appreciate your quick response, and the work you’ve
put in. Great plugin
On Tue, Mar 13, 2018 at 11:26 AM Andrew Fiorillo<EMAIL_ADDRESS>wrote:

Yeah, I just noticed this as well. I think it only happens for palettes
with gradients. Regardless, I plan to fix it soon.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/andrewfiorillo/sketch-palettes/issues/62#issuecomment-372770155,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYSoQZgSXl-UXaBDWf_FJiIY7EpE3QhXks5teA9mgaJpZM4SpNjZ
.

--
Roy Alnashef
818-669-2723
royalnashef.com

I just pushed a new version which should ix this issue. You can update it in Sketch or grab it from the link below. Let me know if you still have any issues!
https://github.com/andrewfiorillo/sketch-palettes/releases/latest


[ENH] window based change point detection adapter #2826

Issue #2826
What does this implement/fix? Explain your changes.
I have implemented a Window based change point detection algorithm in the form of an adapter for the ruptures Window class
Does your contribution introduce a new dependency? If yes, which one?
The contribution creates a new soft dependency as such, pyproject.toml and check_soft_dependencies.py have been updated as well.
What should a reviewer concentrate their feedback on?
As there is no template chosen yet for annotation/change point/segmentation I went ahead and chose what I thought was best and was to inherit from BaseSeriesAnnotator class.
I can refactor any changes if needed or if any other template would be better I will make the changes. Suggestions are more than welcome!
PR checklist
For all contributions

[x ] I've added myself to the list of contributors.
[ ] I've added unit tests and made sure they pass locally.
[ x] The PR title starts with either [ENH], [MNT], [DOC], or [BUG] indicating whether the PR topic is related to enhancement, maintenance, documentation, or bug.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

Great start! It seems the test is failing when ruptures internals are trying to call .reshape probably expecting a np.ndarray but getting pd.Series. Might be worth looking into what  annotation/tests/test_all_annotators.py::test_output_type[WindowBasedChangePoint] is actually doing.


First steps towards importing maps in the cloud

See https://github.com/a-b-street/abstreet/issues/326#issuecomment-832096798 for context. When certain parts of the code change, we have to rebuild all maps. I want to stop running data/regen.sh and frying my poor laptop. Instead, I want to import each city in parallel in the cloud.
This is a simple first step to get the importer and updater working inside a Docker container. Running ./cloud/start_batch_import.sh some-new-map-feature creates a Docker image with local code changes. That container has a script that'll eventually run, import_one_city.sh some-new-map-feature jp/hiroshima that'll download some existing input data and build Hiroshima. Some next steps are to upload that to a temporary place in S3 (not the dev subdirectory that everything live uses), farm out the AWS Batch jobs, shuffle all the results into the right place in S3, and get the net data/MANIFEST.json changes committed to git.

I guess this LGTM, though it feels like a very narrow slice of a pretty large system that I'm not familiar with.

Thanks for reviewing -- all of the questions you asked are about things I'm not really happy with, but not sure how to improve without more effort than feels useful yet. I still dream of describing the import process as some big dependency graph with clearly defined input/output files that can be regenerated from scratch or cached, and nicely ran in the cloud, and synced up to whatever people/UIs want different pieces... but I've yet to see anything like that. :)


Automatic language classification for Untitled files

We could do automatic language classification for Untitled files so users do not have to explicitly choose what language mode to use.
There are already modules for this in Python https://github.com/yoeo/guesslang
However it would be cool to have something in Javascript. One idea is to use TensurFlowJS and try to reuse the guesslang model.
Nice demo by Tyler:
https://user-images.githubusercontent.com/1926584/111809976-d18ca880-88d5-11eb-8e74-6c278aa2fd38.mov

Some related issues: https://github.com/microsoft/vscode/labels/languages-guessing

I would actually also use this for the CLI support for reading from stdin. Currently we create a tmp txt file that contains the contents that are piped into VS Code, but it would be awesome if we could change the language mode based on contents 👍

@bpasero cool, glad there are more use cases.
Did not know we have this label :)

@TylerLeonhardt did a great experiment, we plan to continue on this next milestone thus assigning to April
This PR has a checklist and discussion points https://github.com/microsoft/vscode/pull/119325

@bpasero I think for your request, if we fix this: https://github.com/microsoft/vscode/issues/41614
Then what your requesting could come "for free" if we add the logic to run the model on untitled files as I did in my prototype.

Yeah unfortunately opening an untitled file from piping is currently not an option because we have no good way of talking from the one process directly into an untitled file. But you are probably right that this would be the right solution for that issue.
However, do we consider to support language detection also for plain text files? I wonder if the detection should also run when you open a txt file without having a language mode set. If we think that makes sense, it would solve the case for piping too.

In my eyes a .txt file is a plaintext file so I don't really think that scenario needs to be addressed but if we were to run the detect, we'd need to:

change the file extension as well
maybe run it less often (we said once every 100ish chars for untitled) since it's less likely that the user wants the detect logic than in the untitled case (then again running the model might not be intensive enough to need to do this)

I am not sure the file extension needs to change, we can change the language today in any file without the need to change the file extension.

I am not sure the file extension needs to change, we can change the language today in any file without the need to change the file extension.

I'm not sure all language runtimes can handle non-expected file extensions (PowerShell can't, for example) so if you try to F5 debug a file, and that file is a .txt file even though the language mode is set to Python/PowerShell/TypeScript, the debugger might get confused.
I'd want the language mode to change and then F5 debugging/running in their terminal (node foo.js) to all work flawlessly and leaving it as a txt might prevent that/cause confusion.

Yeah I think that is fine, I actually think https://github.com/microsoft/vscode/issues/41614 is doable with some file watching tricks so I guess it is another good reason to look into it eventually.

Assigning also @TylerLeonhardt since I know he is interested and half of June and most of July I will be on vacation.

A couple of pre-reqs:

[ ] Figure out how to convert the native model to the tfjs model https://github.com/tensorflow/tfjs/issues/4838#issuecomment-859258920
[ ] Figure out how to get tfjs to load the model over something other than http https://github.com/tensorflow/tfjs/issues/4838#issuecomment-859275362

Also if you'd like to play with the experience today (not fine tuned, but works...) the code is published as an extension here:
https://marketplace.visualstudio.com/items?itemName=TylerLeonhardt.auto-language

Moving this to July as I think there are few things we are blocked on above that I don't think we'll figure out in the next week or so.

Is it possible to increase the accuracy using language heuristics data from github/linguist?

@4086606 Good idea. However that library is a ruby library so we would have a dependency on ruby, which is a no-go for us, since we want this working in the browser. Unless there is a js alternative?

Should be able to compile the entire gem to WASM, but I do wonder about size

If you succeed do let us know how it went and the size. Though I see this as step 2, only in the case that the initial model classification does not prove super accurate.

It seems like we will need to take the YAML files from that library and reimplement the heuristics logic. You're right that it should be a step 2 seeing as it's rather blind and not as accurate as it could be..

@isidorn didn't work out - github/linguist can only disambiguate file extensions. Guesslang should be able to use the linguist samples however

I was able to address the pre-reqs and published a new version of my extension that doesn't make a single network request:
https://marketplace.visualstudio.com/items?itemName=TylerLeonhardt.auto-language
Next order of business is to understand how to bring this into Core.

Closing in favor of https://github.com/microsoft/vscode/issues/129004


Interactive output unreadable

I'm working with jupyter notebooks. The code syntax highlighting in the Python Interactive pane is making my eyes bleed. If there is a way to set the color scheme (or just disable it) please tell me.
Environment data

VS Code version: 1.29.1
Extension version (available under the Extensions sidebar): 2018.11.0
OS and version: linux 4.19
Python version (& distribution if applicable, e.g. Anaconda): Python 3.6
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: jupyter (any)

Expected behaviour
Code readable in "Python Interactive" pane.
Actual behaviour

Steps to reproduce:

Create somescript.py:

#%%
import os
from matplotlib import pyplot

b = 17. + 27.
print("This is just some python code:", b)
print(undefined_variable)

Run by cicking the codelens "run cell" thing.
Try debugging it.

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
[nothing]

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)
[Extension Host] Python Extension: Execute for import pandas as pd
import numpy
%matplotlib inline
import matplotlib.pyplot as plt
from matplotlib import style
style.use('dark_background') silently finished.
(4)
console.ts:134 [Extension Host] Python Extension: Execute for %cd "/[path/to/secret/project]" silently finished.

This is an existing issue: #3107. Or at least that issue will address your problem when we finally get to it.
You can try using the default dark theme. That will be slightly better.


Figure out enum serialization and deserialization

Right now having an enum in the deserialization is failing

If we set serialization type of enum to str, we pass test_polymorphism_get_and_put_missing_discriminator in test_complex but not test_enum_not_exapandable in test_string_tests
If we set serialization type of enum to the enum type, it's the opposite

We are now always serializing enums as strings. This is different from the past autorest behavior


【emergency】short&buy can't math right now in bitUSD:bts market

i have reloaded many times

Another screenshot here: #1498 .
The top 'sell order' is a short order.
The reason is same to #1495 .
A workaround could be short with a higher limit_price or without limit_price.

Duplicate: #1495


Specify contributing rules and release workflow

Main branch get major change.
Release/vx as release branch
Last release branch as default branch
All release commits should be merged in main
bugfix/vx-bugfix-name
feature/vx-feature-name
squash-merge for easy cherrypick a fix from a release to another releases if adaptés
Follow a git message format for be able to generate changelog
Tag each release
Github action on tag to build ans deploy package in gh packages registry

https://www.conventionalcommits.org/en/v1.0.0/


Clarification about collective key switching

I don't understand why collective key switching must be performed rather than generating keyswitch keys.
Can't a similar procedure be used to generating relinearisation keys collectively?
This approach seems to introduce unnecessary communication costs.
Furthermore, there does not seem to be a description of how galois keys can be generated. However, can't one generate Galois keys by just permuting the coefficients of the secret key? This is not a problem even if we are dealing with shared secrets. So it is easy to publish galois keys, is it not? What other purpose is there for collective key switching?

Hello Jon-Chuang,
Yes you are right, the parties could also generate in advance a keyswitching key and have someone else carry the operation using only public operations. Using such an approach does however introduces a larger noise amount than just having each party partialy re-encrypt the ciphertext. There are pros and cons for each approaches.
The galois keys are an encryption of a permutation of the coefficients of the collective secret-key, so yes they are as "easy" to generate as the collective public-key.
What do you mean by "What other purpose is there for collective key switching?" ?

To add up to @Pro7ech 's answer, the CKS protocol enables the parties to allow decryption or, re-encryption under the receiver's key, of the computation result (and only the computation result) to be performed. Indeed, this needs to be an interactive functionality (because of @Pro7ech's Edit1).
So, in our terminology, we differentiate the notions of key-switching (s1 -> s2) and relinearization (s^2 -> s).

Hi all, I understand this better.
Essentially, collective keyswitching is meant as a barrier so that the parties who have a stake in the data can collectively agree to allow a single party to decrypt the data, even if they do not know the secret key of that party (PCKS), or that party has shared it with them collectively through secret sharing (CKS).
However, I still feel there is a missing piece of the puzzle, which is zero knowledge. This would incur some overhead (100x?) but may be what an application developer wants to have more control over their data.

You are correct about the need for ZK primitives when more guarantees are needed and we would be happy to push this discussion further. As for the HE scheme you are proposing, this seems like an interesting path to investigate. However, since it does not concern the code-base directly, please use the Lattigo mailing list<EMAIL_ADDRESS>

Closing: discussion taken offline.

The above scheme I described does not exist, as I just figured out today.
Hence I think zero knowledge is the only viable alternative we known of.


[18.06] luci-app-wol: fix "Host to wake up" to select

This removes two undefined option in the menu.
Signed-off-by: Chuck<EMAIL_ADDRESS>(cherry-picked from commit coolsnowwolf/luci@c28e337b3e8ae261454f2a7980030825af084a78)

NAK, as this removes the ability to enter custom MAC addresses. The underlying undefined should be fixed instead.


makePropGetter always results in fresh properties causing render thrash

I'm not completely sure that this is a bug so much as a problem where the best fix might end out being in react-table.
Due to the nature of makePropGetter, you can guarantee that repeated calls to methods such as getToggleRowSelectedProps will generate new objects. Where the top level values are primitives then this tends to not cause problems since the results are often destructured and passed to a component, but when they are objects, such as styles, this fails simple equality checking and leads to a spam of re-renders.  Non-memoized callbacks have the same problem.
Not quite sure yet whether this is best handled in the table or by memoizing the UI components, but thought it was worth bringing up.

I'll look into this as part of the stable release 👍

After some deep thinking on this issue, I think I've come to the conclusion that this is going to be the best we can do for now. As of today, all if not most uses of makePropGetter are going to return fairly stable props other than inline callbacks created or composed style objects. In those cases that styles or callbacks are created, I think this is in line with the React docs and common practices as it is highly unusual that inline style objects and inline callbacks actually result in overrendering and when they do (as you pointed out), there are tools like useMemo or React.memo that let you break out of that. Alternative ways of constructing these props would not be simple or maybe not even possible, given that the nature of react hook memoization is tied to the instance of the parent component and not to something variable like the amount of rows. We would have to build our own memoization solution that could operate on a variable amount of items across the entire library. So while it is technically possible, I think the pros don't justify the cons here. I am totally open to discussing this as the library grows though and use cases arise. For now I don't think this is a blocker to get v7 out though. I really appreciate your attention to detail here though. It's helped me validate some of the core architecture of the library.


PermissionsAndroid permission not granted on APIs 22 and below

Is this a bug report?
Yes

Have you read the Contributing Guidelines?
Yes.
Environment

react-native -v: 0.46.4
node -v: 8.0.0
npm -v: 5.0.2
yarn --version: 0.24.6

Then, specify:

Target Platform: Android

Development Operating System: MacOS

 Android SDK

Build tools: Gradle

Steps to Reproduce

(Write your steps here:)

Create an async function to request permission. PermissionsAndroid.PERMISSIONS.WRITE_EXTERNAL_STORAGE
Call this function inside onPress prop of TouchableOpacity
The function returns a prompt asking for the user to agree or deny the permission like an android app does based on the new permission model. But while testing on a device which has Android 5.1.1 installed the request function does not return the string "granted". Therefore the if statement which checks for the condition based on this returned string value returns false.

Expected Behavior

In Android 5.1.1 or APIs before that PermissionsAndroid returns true for check and request calls. This means if I write my actual functionality inside an if statement with the condition PERMISSIONS.RESULT.GRANTED === permission, it should pass as true.
Actual Behavior

Whenever the async function is called, the IF statement which checks if the request function returned the string value "granted" returns False. This happens in devices with Android Lollipop installed or previous versions.

Did you get any solution for this?


Move OpenShift Route to a new module named 'openshift'

$subject

Codecov Report

Merging #307 into master will decrease coverage by 0.03%.
The diff coverage is 64.28%.

@@             Coverage Diff              @@
##             master     #307      +/-   ##
============================================
- Coverage     78.69%   78.65%   -0.04%
  Complexity      689      689
============================================
  Files            65       65
  Lines          3018     3022       +4
  Branches        289      290       +1
============================================
+ Hits           2375     2377       +2
- Misses          505      506       +1
- Partials        138      139       +1

Impacted Files
Coverage Δ
Complexity Δ

.../processors/openshift/OpenShiftRouteProcessor.java
68.57% <ø> (ø)
5 <0> (ø)
:arrow_down:

...va/org/ballerinax/kubernetes/KubernetesPlugin.java
78.88% <ø> (ø)
16 <0> (ø)
:arrow_down:

...rs/openshift/OpenShiftBuildExtensionProcessor.java
73.33% <0%> (-5.24%)
3 <0> (ø)

...etes/handlers/openshift/OpenShiftRouteHandler.java
90% <100%> (ø)
7 <0> (ø)
:arrow_down:

...andlers/openshift/OpenShiftBuildConfigHandler.java
91.07% <100%> (ø)
5 <0> (ø)
:arrow_down:

...ernetes/processors/AnnotationProcessorFactory.java
87.09% <100%> (ø)
12 <0> (ø)
:arrow_down:

...andlers/openshift/OpenShiftImageStreamHandler.java
82.14% <100%> (ø)
4 <0> (ø)
:arrow_down:

...g/ballerinax/kubernetes/utils/KubernetesUtils.java
73.25% <62.5%> (-0.09%)
55 <2> (ø)

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 10c0f5f...b1cdebc. Read the comment docs.


Fix(project template) Force bindings regen, if out dir empty.

Running tns build ... --bundle runs the clean gradle target and
we can end up deleting bindings *.dex files while the presence of
*.txt files in the generator dir will prevent us from regenerating
them.
The fix checks the output dir and forces regeneration if it has no
files.

Related to NativeScript/nativescript-cli#2110

Ping @Plamen5kov

Looks :+1: for invalidating the gradle's up-to-date status.
A definite fix to the problem with SBG not triggering when project's been cleared can be to output interfaces-names.txt, and potentially bindings.txt inside the project's build dir, or make a custom generateBindings task. @Plamen5kov

@hdeshev I think a better solution would be to make our clean command take care of deleting the generated file in build-tools folder. That way we won't need additional logic to regenerate bindings. Maybe the template platforms/android/build.gradle can call the :asbg:clean command on the subproject. And the subproject can clean it's on generated files.

IMO the "right" solution to this problem is to generate all bindings-related .txt files inside the build subfolder that will get properly cleaned and recreated.
I did a half-hearted attempt at getting that to run, but couldn't get it to work in reasonable time. Perhaps somebody who knows the gradle infrastructure better than me can have a shot.

I'll take a look.

closing in favor of: https://github.com/NativeScript/android-runtime/pull/598


Encrypt the Lambda function envirornment variables created using buildkite elastic stack

The lambda function environment variables need to be encrypted as per the security standards.
While creating the Elastic CI stack we can force the encryption of the lambda environment variables with the default or the customer-managed key.

hey there @RahulMahale! i wasn't aware that lambda had the capability for encrypting envars in transit, that's a nifty feature - docs here, for future travellers.
what security standards are you using that would require this? would you be interested in submitting a PR to implement this feature?