documentation doesn't articulate core use case / target audience

Hi Bryce =)
Reading the docs here, it seems like implicitly there's an emphasis on web event use cases, funnel analysis, etc, but I don't see that anywhere explicitly.
I've got a use case that's pretty different and I can't quite tell if this package is right for us. we often get activity data in terms of intervals  and many intervals may be missing. this is to say that we have prior expectations about what data we should get and when, and our work tends to have to do with the alignment of the interval data we get with a set of scheduled events which are "fulfilled" by the activity. Missing activity records therefore need to be counted and possibly infilled in specific ways, and they may arrive late and need to be backfilled regularly. In general we are trying to simply measure the overlap between these event windows and the activity of the entities whose activities can be linked to that event (each event will pertain to a set of IOT-style actor entities whose activities can contribute to at most one event at a time)
For us, we use window functions a lot for this, and are just beginning to play with match_recognize in snowflake. I think that approach will work for us as a generalizable pattern, but I wonder if our approach is missing things covered by the canonical activity framework.

Hey @mike-weinberg - thanks for opening this issue! You highlighted a few different topics so going to try to address each:

Reading the docs here, it seems like implicitly there's an emphasis on web event use cases, funnel analysis, etc, but I don't see that anywhere explicitly.

Given the event-like structure of the activity stream and the sequential nature in which activities relate to each other when building datasets, I think I can see where this perception originates. Ultimately, though, the methodology is not unique to web events and funnel analysis. I regularly build current-state datasets from the streams I've implemented. I personally view activity schema as an intermediate transformation standard that coerces downstream consumers to access the data in a consistent fashion. So ad hoc datasets or stable dimensional models can be built on top of it. There are tradeoffs to this approach, of course, and while I have some blogging I intend to do on that subject, I do think the documentation should largely be dedicated to the implementation of the project (which is also why I articulate in the readme that some prior knowledge of activity schema modeling is required). Perhaps I can link to the framework spec there, though, for easy access for folks who don't yet have that context.

we often get activity data in terms of intervals and many intervals may be missing...

If I'm understanding correctly, this is effectively telemetry data you're modeling, yes? In the case of telemetry data, the activities I've produced are activities that represent when an attribute being tracked in the telemetry changes. So, for example, if the telemetry data is tracking temperature and sends an event every 5 minutes, then the activity I'd recommend is "temperature_changed" and only include an activity occurrence each time the temperature has changed relative to the previous logged occurrence. But there are other dynamics at play (e.g. late-arriving events) that could affect the recommendation, and also I didn't totally follow the use case due to lack of domain context - can you provide an example with more specifics? What's an example event? What questions are you trying to answer on behalf of the business? That would help me refine a recommendation here. Happy to chat live if easier.

For us, we use window functions a lot for this, and are just beginning to play with match_recognize in snowflake.

No sales agenda here, and if your team is exclusively trying to address a specific use case isolated to the event data you described, then using something like match_recognize seems like the best path forward. Will fit in nicely with your existing stack (just write a query using match_recognize in a sql file in your dbt project), and doesn't come with the overhead of having to integrate a new dbt package that's highly opinionated about code structure (i.e. comes with a meaningful migration effort). Where activity schema shines brightest is when trying to reconcile different types of data in different formats in different systems. So, if your team is trying to connect user attributes from a CRM to this telemetry data, activity schema (and possibly this package) could be a good fit.
Lmk what you think!

Hey @mike-weinberg doing some issue cleanup - this appears to have gotten stale so closing for now. Feel free to reopen if you'd like to continue the discussion!


Multiple Validator cannot generate Nodeaddress to connect Peers

burrow version(0.25.1+commit.v0.25.0-51-ga11c9a4c+2019-06-08)
go version (go version go1.12.5 darwin/amd64)
operating system details (osx)
what you expected to happen
Trying to connect multiple validator
Using  cmd
   "NODE_0_URL=curl -v -o ${GoPath}/{FolderName}/Node_address.txt <IP_ADDRESS>:20001/network | jq -r '.result.ThisNode | [.ID, .ListenAddress] | join("@") | ascii_downcase'"
to generate Node_address.
'what actually happened'
Facing Error when i run above cmd.
'output'
   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                               Dload  Upload   Total   Spent    Left  Speed
       0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying <IP_ADDRESS>...
      * TCP_NODELAY set
      * Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 26656 (#0)
      > GET /network HTTP/1.1
     > Host: <IP_ADDRESS>:26656
     > User-Agent: curl/7.54.0
     > Accept: */*
     >
   { [34 bytes data]
     * Recv failure: Connection reset by peer
     * stopped the pause stream!
     100    68    0    68    0     0  50221      0 --:--:-- --:--:-- --:--:-- 68000
     * Closing connection 0
      curl: (56) Recv failure: Connection reset by peer

This looks like a connection issue with the chain. Could we pick this up in https://chat.hyperledger.org/channel/burrow


the upstream receive double /

when the httproute config with urlrewrite from envoy-gateway like follows:
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  name: backend
spec:
  parentRefs:
    - name: eg
  rules:
    - backendRefs:
        - group: ""
          kind: Service
          name: backend
          port: 3000
          weight: 1
      filters:
        - type: URLRewrite
          urlRewrite:
            path:
              replacePrefixMatch: /
              type: ReplacePrefixMatch
      matches:
        - path:
            type: PathPrefix
            value: /om/test

the upstream receive double /

envoy doc

reference https://github.com/envoyproxy/gateway/issues/1044

Please file this issue with Envoy Gateway and debug there. Thanks.

Please file this issue with Envoy Gateway and debug there. Thanks.

its envoy case.
eg1：prefix = /prefix，prefix_rewrite = /rewrite
Request path = /prefix/abc
The upstream received path = /rewrite/abc
eg2：prefix = /prefix，prefix_rewrite = /
Request path = /prefix/abc
The upstream received path = //abc, /prefix replaced by prefix_rewrite value "/", the upstream received double "/"

@mattklein123  pls view this capture
like envoy doc configured:

upstream receive double "/"

OK. Can you open a fresh issue that gives a pure Envoy repro? Thanks.

OK. Can you open a fresh issue that gives a pure Envoy repro? Thanks.

@mattklein123 #26055


Add audit records to setup and installation

We need to add audit subrecords to the setup and installation records to ensure we know who changed what when.

Added the audit records to setup and installation records. Audit is visible for the detailed setup of directories but is available on all setup and installation record.


Missing JFrog Licenses in SARIF format

Describe the bug
None of the JFrog licenses are included in the SARIF format.
Current behavior
None of the JFrog licenses are included in the SARIF format.
Reproduction steps

Create 2 reports (1 JSON and 1 SARIF)
jf audit --licenses --format json > foss_result.json
jf audit --licenses --format sarif > foss_result.sarif
Compare the outputs of the reports

Expected behavior
The SARIF format contains the same license information as the json output.
JFrog CLI version
2.52.9
Operating system type and version
Docker
JFrog Artifactory version
No response
JFrog Xray version
No response

Related to: https://github.com/jfrog/jfrog-cli/issues/2063 (open since July 2023)

Based on https://github.com/jfrog/jfrog-cli/issues/2270 it seems like https://github.com/jfrog/jfrog-cli/issues/2063 has been partially addressed, but nothing regarding licenses yet.


[BUG] Face API library in Maven repository is outdated

Describe the bug
The azure-faceapi-0.0.1-beta.jar and azure-cognitiveservices-faceapi-1.0.0-beta.jar files in the Maven repository are outdated (the latter was uploaded 20180518). As a result, classes like FaceAPIManager, used to authenticate to the Face service, are not available.
Also, in https://github.com/Azure/azure-sdk-for-java/blob/master/README.md, in the section Cognitive Services/Vision/Face API, the Sources and Download links are outdated. The former gives me a 404. The latter gives me search results in the Maven repository that only include azure-faceapi-0.0.1-beta.jar.
Exception or Stack Trace
Test.java:7: error: cannot find symbol
            FaceAPI client = FaceAPIManager.authenticate("westus", "subscriptionKey");
                             ^
  symbol:   variable FaceAPIManager
  location: class Test
1 error

To Reproduce

Save code snippet as Test.java.
Download azure-cognitiveservices-faceapi-1.0.0-beta.jar from https://search.maven.org/search?q=azure faceapi.
Open a command prompt.
Run:
javac Test.java -cp azure-cognitiveservices-faceapi-1.0.0-beta.jar

Code Snippet
import com.microsoft.azure.cognitiveservices.vision.faceapi.*;
import com.microsoft.azure.cognitiveservices.vision.faceapi.models.*;

public class Test {
    public static void main(String[] args) {
        try {
            FaceAPI client = FaceAPIManager.authenticate("westus", "subscriptionKey");
// Do something with client
        } catch (Exception e) {
            System.out.println(e.getMessage());
            e.printStackTrace();
        }
    }
}

Expected behavior
The code compiles.
Screenshots
N/A
Setup (please complete the following information):

OS: Windows 10 Pro 1803
IDE: None
Version of the Library used: 1.0.0-beta

Additional context
I am able to make my code compile by doing the following.

Clone/download: https://github.com/Azure/azure-sdk-for-java
Install Maven (https://maven.apache.org/).
Add C:\Program Files\Maven\bin to your PATH environment variable.
At a command prompt or git-bash shell, browse to [download location]\azure-sdk-for-java-master\cognitiveservices\data-plane\vision\faceapi
Run:
mvn package -f ./pom.xml
That generates azure-cognitiveservices-faceapi-1.0.2-beta.jar in a folder named "target".
Compiling Test.java using this .jar file works as expected.

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

Thank you for opening this issue! We are routing it to the appropriate team for follow up.

@milismsft @ryogok

@joshfree any updates for this issue?

Still not fixed as of January 2021. Any idea when this is going to be fixed ? Are there any other alternative ways to get the up-to-date sdk ?
Thanks a lot

Still not fixed as of January 2021. Any idea when this is going to be fixed ? Are there any other alternative ways to get the up-to-date sdk ?
Thanks a lot

@samvaity could you please follow up on this issue?

Hello @v-jaswel, you can find the sources and download here.
Do let me know if you are still facing the issue.
cc: @thomasburguiere

Hello @v-jaswel, you can find the sources and download here. Do let me know if you are still facing the documentation issue.
cc: @thomasburguiere

I go into the mentioned url (https://search.maven.org/artifact/com.microsoft.azure.cognitiveservices/azure-cognitiveservices-faceapi/1.0.0-beta/jar) and download the dependency but the problem still persists, the FaceAPIManager class is still not inside the jar

Hello @RobertoSD22, you are right, the FaceAPIManager class was missing from the previous release. We did release this package again recently and you can find the updated jar here https://mvnrepository.com/artifact/com.microsoft.azure.cognitiveservices/azure-cognitiveservices-faceapi/1.0.1-beta

Hello @RobertoSD22 / @v-jaswel, This seems to be working with the updated jar here https://mvnrepository.com/artifact/com.microsoft.azure.cognitiveservices/azure-cognitiveservices-faceapi/1.0.1-beta
Closing this, do reopen if this is still an issue.


Lerna managed app - styles errors

Hi. I don't think this is a bug, but definitely it's not working as it should or maybe i do not know use Material-UI library well. So... there is a lerna app managed that contains two lerna packages: app and components. Material UI is installed and used in both places. When i use makeStyles in a component in components package and use (low level in dom tree hierarchy),  the styles somehow goes also up in the three (see image): jss1 should be ONLY for the 2 span elements

the styles are applied only for the span
Can you help me out ?
const useStyles = makeStyles(() => ({
  root: {
    color: 'blue',
  },
}));

function Comp({ parameter }) {
  const classes = useStyles();
  return ({parameter});
}

Thank you

Do you have a warning in the console? Is this related to https://material-ui.com/getting-started/faq/#i-have-several-instances-of-styles-on-the-page?

@oliviertassinari i do not have a warning or error in the console. I'm reading that page. Not sure, but probably will fix my issue... Coming back here soon to tell the conclusion.

@alexandru-tomescu I'm closing, I hope you will find a solution, please report the solution when you do. We gonna move to https://www.styled-components.com/. The root issue will be solved once it's completed.

@oliviertassinari will styled-components take the advantage of the m-ui theme ?

What do you mean by advantage?

@oliviertassinari i mean... if u can have and inject a theme object, and can provide it when styiling the components ...


Unable to switch to Powerapps form in Tasks list- After Site Upgraded to Modern

Category
[*] Bug
[ ] Enhancement
Problem Area
[ ] Page Transformation: Error during the setup/use of the Page Transformation UI solution (did you check our troubleshooting guide?)
[ ] Page Transformation: Error during the use of page transformation from PnP PowerShell
[ ] Page Transformation: Error during the use of page transformation from .Net
[ ] Page Transformation: Page is not looking correct after transformation
[* ] Modernization Scanner: something went wrong...
After Upgrading a site to Modern, When a new Tasks list is created,unable to switch the form to Powerapps form using form settings.

Hey @InfosysAdmin ,
A task list is classic by design, it will not show a modern UI as Microsoft has not built a modernized the task list and web part (we do have planner for modern task management). Given the UI is forced to classic there's no option to use PowerApps as that only works for lists showing in modern.
This is not an issue in page transformation, so will close it but hope the answer I gave will clarify things for you.

The problem is not only on migrated sites. We have sites that are in Classic Experiences, but in the new task lists it is no longer possible to use SharePoint Integrated PowerApps forms or even standard SharePoint forms. The cause is other.
In the Modern Experience view the SharePoint integration should be available.
https://powerusers.microsoft.com/t5/Building-Power-Apps/Customize-Sharepoint-Online-Form-Parent-Child-Lookup-field/td-p/245171


gpu-compute: Fix for failing assertion in GPUCoalescer

Commit #200 introduces colaesced atomics to same addr from within a WF; atomicLog was part of it to record the changes on datablock sequentially; However, for non-returning atomics, this log isn't cleared, which manifests as a failing assertion.
Fixes issue #508, adds to PR #397
Change-Id: Ib2aa470120163552488c3e69913d87b70b293c6e

Due to the fallthrough case in the change, with my compiler, I get the implicit-fallthrough warning as an error -- which can be fixed by modifying the SConstruct file like so:
diff --git a/SConstruct b/SConstruct
index e7fa4b53b7..47bed3014b 100755
--- a/SConstruct
+++ b/SConstruct
@@ -452,6 +452,7 @@ for variant_path in variant_paths:
         env.Append(CCFLAGS=['-Werror',
                              '-Wno-error=deprecated-declarations',
                              '-Wno-error=deprecated',
+                             '-Wno-error=implicit-fallthrough',
                             ])

Should I add this change as well?

There should only be 1 commit. Whoever merges should use squash merge!

I assume, but I want to double check... This is unrelated to the memory leak discussed in #514 (https://github.com/gem5/gem5/pull/514#issue-1967109979). Please let me know one way or the other.

Would this conflict with CHI now clearing the atomic log within the protocol here in #514?

I think this is also related to #449. Should the same fix be applied to Sequencer.cc?

Since this issue is specifically related to GPU atomics, there could be a similar memory leak in the CPU, but it would have to be for something other than GPU atomics.  Where do you want us to look in the Sequencer?

Would this conflict with CHI now clearing the atomic log within the protocol here in #514?

In theory since this commit is about GPU atomics and CHI does not support GPUs, it should be similar concept but unrelated codewise?

I assume, but I want to double check... This is unrelated to the memory leak discussed in #514 (#514 (comment)). Please let me know one way or the other.

I believe it is the same memory leak, so it would have to be handled for other atomic no return requests (I guess in Sequencer?)
Maybe another fix would be to have an additional boolean parameter for DataBlock and WriteMask atomic methods which indicates whether or not to populate the log at all and default to false. The protocols would have to explicitly set to true for atomics with return though which may be a lot of changes.  I don't know how many protocols actual use atomics though.

@powerjg -- I'm not sure what's needed in Sequencer.cc
https://github.com/gem5/gem5/blob/8b63299ad61f6ea5b3ae50cc830e632b17d8430e/src/mem/ruby/system/Sequencer.cc#L749 -- this is where we could clear the data->m_atomicLog -- but I'm not sure. Do you have any insight on this?

@abmerop said the following.

I believe it is the same memory leak, so it would have to be handled for other atomic no return requests (I guess in Sequencer?)

If there is a memory leak that is in both Sequencer and in GPUSequencer, it should be fixed in both.
I am not an expert on this code. @bujset contributed the original atomic log code. If this introduced a memory leak, we need to fix it. #200
Also, it would be great if you could answer this specific question: Is the memory leak resolved in the change the same as the memory leak mentioned in #514. If you need more information about #514, then talk to Giacomo in that PR.

@abmerop said the following.

I believe it is the same memory leak, so it would have to be handled for other atomic no return requests (I guess in Sequencer?)

If there is a memory leak that is in both Sequencer and in GPUSequencer, it should be fixed in both.
I am not an expert on this code. @BujSet contributed the original atomic log code. If this introduced a memory leak, we need to fix it. #200
Also, it would be great if you could answer this specific question: Is the memory leak resolved in the change the same as the memory leak mentioned in #514. If you need more information about #514, then talk to Giacomo in that PR.

This specific problem will not be fixed by #514 because 514 only updates the CHI protocol, which is not used in GPUs.  So there is a separate fix needed here.
I am not sure where exactly @abmerop is envisioning a fix being done in Sequencer.cc as the atomic log -- I assume it's related to https://github.com/gem5/gem5/blob/8b63299ad61f6ea5b3ae50cc830e632b17d8430e/src/mem/ruby/system/Sequencer.cc#L721?

When I wrote the changes to GPUCoalescer, I had noted my assumption that Store and AtomicNoReturn requests follow the same path. However, looking through the VIPER protocol files, it seems that AtomicNoReturn and AtomicReturn requests are treated the same (so logs are created and populated even on AtomicNoReturn requests). And I'm guessing this assumption doesn't hold in other protocols as well.
I think the correct fix will require changes to the cache protocols to treat AtomicReturns and AtomicNoReturns differently, rather than changing the GPUCoalescer. As discussed in #508, the RubyRequestType already has the information needed to differentiate these, so I think you would just need to have separate action in the protocols themselves. I think not create the logs if you don't need them is a cleaner solution than creating logs in both cases, and deleting them when not needed.

My suggestion was to modify the WriteMask atomic method to add an additional parameter whether or not the log should be used, with false as the default.  The log only seems to be necessary for GPU, so we would set the parameter to true in VIPER. Then there are no changes to CHI, Sequencer, nor GPUCoalescer to fix the leak

Seems like at the cache closest to the core (TCP), the accepted request has a RubyRequestType, and when it passes on to the next cache (TCC), it switches to CoherenceRequestType which does not distinguish between RETURN and NO_RETURN. However, interestingly, it has 2 types: Atomic (used ubiquitously) and AtomicWriteBack -- which is unused. We could rename this for those atomics needing a return (=log), else don't use log. How does that sound?

The GLC bit determines if it is return or no return and I believe that is already propagated, so we could use that

Alright then, I can add in the changes to `performAtomic

Closing in favor of https://github.com/gem5/gem5/pull/546.


Prep initial charts indexing

Fixes #7, Fixes #8

[x] Cherry-pick commits from the one PR that was merged since the code was copied https://github.com/helm/charts/pull/23506 (see #13 for context)
[x] Deprecate charts/prometheus-operator (see #8 and #9 for context)
[ ] Remove prow OWNERS files (see #12 for context)

Charts that need to be prepared for initial helm index

[x] charts/prometheus
[ ] charts/prometheus-adapter
[ ] charts/prometheus-blackbox-exporter
[ ] charts/prometheus-cloudwatch-exporter
[ ] charts/prometheus-consul-exporter
[ ] charts/prometheus-couchdb-exporter
[ ] charts/prometheus-mongodb-exporter
[ ] charts/prometheus-mysql-exporter
[ ] charts/prometheus-nats-exporter
[ ] charts/prometheus-node-exporter
[ ] charts/prometheus-postgres-exporter
[ ] charts/prometheus-pushgateway
[ ] charts/prometheus-rabbitmq-exporter
[ ] charts/prometheus-redis-exporter
[ ] charts/prometheus-snmp-exporter
[ ] charts/prometheus-to-sd

Suggestion: per the steps in the plan at https://github.com/prometheus-community/community/issues/28#issuecomment-670230344 (which has been approved for the prometheus-community), for this PR review and comments let's discuss only what should happen in the intial step. To be clear, this PR is the first step, before "Step 2. Collaborate with prometheus devs to refactor/rename and simplify charts as appropriate." I outlined what I think we need in #7, but please let me know if I'm missing anything.
📊  Maintainers poll
How do you all feel about the new charts/prometheus README update in b324685? I Needed to update references to stable repo, but took the opportunity to reorganize, fix and simplify. Is this a good pattern for the other chart READMEs? Note major differences are:

Added Helm 2 and Helm 3 commands, with links to Helm 3 command reference links (from there, users can select Helm 2 from the version dropdown if they wish)
Removed the options table that's duplicated from (often out of sync with) values.yaml in favor of helm inspect (Helm 2) and helm show values (Helm 3)

LGTM, I think it makes sense to drop the table of values from the README.md, it's often hard to keep it in sync with reality.

So far it sounds like we may be able to compromise with helm upgrade [RELEASE_NAME] [CHART] --install. Welcome other opinions here as well (though not trying to bike-shed). Looking for feedback on the readme reorganization overall as a pattern for the rest of the chart updates. Thanks!

I'm ok with using the upgrade command. Maybe the long version (--install instead of -i) of the arguments should be used to be clear.
Rest of the readme looks good to me and could be used as a template for the other charts.

@monotek @steven-sheehy & gang, I put the Helm 3 commands above Helm 2. And instead of using helm upgrade for installation instructions, I took suggestions above for making helm install use an explicit release name, then added helm upgrade commands in a similar way to the "Upgrading chart" section so there's no confusion. I also reorganized the upgrading MAJOR version headers slightly to be more clear to end users, depending on the version they're upgrading to. This has worked out well for other community charts. The pattern is like this:
## Upgrading Chart
(helm commands)

## To v6.0
(instructions)

## To v5.0
(instructions)

## To v4.0
(instructions)

## From `foo` legacy chart located <HERE> (if any)
(instructions)

Does this look reasonable? I can apply this same pattern to the other charts.

I've now resolved all prometheus chart maintainer feedback, so will apply this pattern to the other charts so we can get this merged and expedite the repo transfer ⏳

Tests were failing for prometheus-couchdb-exporter (example lint-test GH Actions job). Had to fix k8s 1.16 deprecated PodSecurityPolicy in the extensions/v1beta1 API version. See https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/ I'm guessing this chart hasn't been modified in a while 😛

Update: All initial chart readme changes have been pushed. Waiting on tests. Please give this a once over review. Once this is approved, we'll merge and move ahead 😅

PS re-running gh action job. This appears to be an API blip:

✖ prometheus-consul-exporter => (version: "0.1.6", path: "charts/prometheus-consul-exporter") > Error validating maintainer 'timm088': 502 Bad Gateway

Just tested locally and it validates fine:
$ docker run --rm -it -v $(pwd):/app:cached quay.io/helmpack/chart-testing:v3.0.0
$ / #
$ / # cd /app
$ /app # ct lint --charts /app/charts/prometheus-consul-exporter/
...
1 chart(s) linted, 0 chart(s) failed
------------------------------------------------------------------------------------------------------------------------
 ✔ prometheus-consul-exporter => (version: "0.1.7", path: "/app/charts/prometheus-consul-exporter/")
------------------------------------------------------------------------------------------------------------------------
All charts linted successfully

However, now getting similar errors:

✖ prometheus => (version: "11.12.1", path: "charts/prometheus") > Error validating maintainer 'gianrubio': 429 too many requests

According to GitHub everything is fine https://www.githubstatus.com/ 🤔
We may have to just try to run the job again 🤦 ⏳

😑 https://github.com/scottrigby/prometheus-helm-charts/runs/1007003791
I've honestly no idea which GH user GITHUB_TOKEN ties to for GH API rate limits. Don't seem to see this in the docs for this token in actions. It doesn't seem to be my user though:
X-RateLimit-Limit: 5000
X-RateLimit-Remaining: 4992
X-RateLimit-Reset:<PHONE_NUMBER>

Fine, I'll go back to sleep and re-run later 😴
PS: If anyone wants to join us at the Deep Dive Into Helm session today, it starts in a little over 2 hours 🕣 See some of you then!

Woot! tests have passed! Let's get to the next steps. Not all maintainers of these charts have participated yet, but they may or may not be available. The goal of this PR is to change at least the charts readmes so the initial helm repo indexing will contain only updated info, and relevant to the new repo (we had to fix things along the way only because some of these charts have not been touched for some time). In future PRs, they will only be merged once at least one maintainer of the affected charts approve the PR. For that reason I recommend future PRs try to touch only one chart at a time like we do in the stable and incubator git source repo (github.com/helm/charts).

Update: Maintainers please check https://github.com/scottrigby/prometheus-helm-charts/issues/11 for another poll to see if there are any remaining blockers for the repo transfer to prometheus-community :shipit:
There are also testing steps in that issue, if anyone wants to try it out now 🚀


Add new device intel wireless bluetooth USB\VID_8087&PID_0AAA

Hi, can you add support fot this intagrated device - intel wireless bluetooth USB\VID_8087&PID_0AAA ?

Feel free to comment this related ticket:
https://github.com/abandonware/node-bluetooth-hci-socket/issues/9

Feel free to comment this related ticket:
https://github.com/abandonware/node-bluetooth-hci-socket/issues/9


The rangeCategory filtering should take into account height/verticality and altitude already present in the specification

Now, in psgeoDomainDefault there is already a variable pointing to the user specified metric in JSON.
Code needs to be implemented.


Android Copy Paste broken for some implementations of tab navigator

Current behaviour
I have identified an issue with copy/paste on android where it will not work with some instances where the removedClippedSubviews is set to true.
found inside the ResourceSavingScene in the drawer and tabs views.
https://github.com/react-navigation/tabs/blob/44b895fa114465fedafba217c27a2918ed38367d/src/views/ResourceSavingScene.js#L35-L39
I have filed a seperate comment on an existing issue on the RN repo regarding the removedClippedSubviews
https://github.com/facebook/react-native/issues/18926#issuecomment-494271934
Expected behaviour
Copy paste should not be effected by the usage of removedClippedSubviews which I believe is a RN core issue, but I am unsure if these performance improvements are required on android as it seems based on the code comments that these were for iOS. So hopefully setting removedClippedSubviews to false is a viable fix for this issue.
Code sample
import React from "react";
import { View, TextInput } from "react-native";
import { createAppContainer, createBottomTabNavigator } from "react-navigation";

function Input() {
  return (
    <View
      style={{
        flex: 1,
        justifyContent: "center",
        padding: 40,
        backgroundColor: "gray"
      }}
    >
      <View style={{ elevation: 1 }}>
        <TextInput placeholder="broken copy/paste" style={{ backgroundColor: "white" }} />
      </View>

      <View style={{ borderWidth: 1 }}>
        <TextInput placeholder="broken copy/paste" style={{ backgroundColor: "white" }} />
      </View>

      <View style={{ backgroundColor: "white" }}>
        <TextInput placeholder="broken copy/paste" style={{ backgroundColor: "white" }} />
      </View>

      <View>
        <TextInput placeholder="Can Copy/Paste" style={{ backgroundColor: "white" }} />
      </View>

      <TextInput placeholder="Can Copy/Paste" style={{ backgroundColor: "white" }} />
    </View>
  );
}

const Navigator = createBottomTabNavigator({
  Input
});

const Navigation = createAppContainer(Navigator);

export default Navigation;

What have you tried
Setting removedClippedSubviews={false}  in the ResourceSavingScene fixed this issue on android.
Your Environment
Android
React native: 0.59.8
react-navigation: 3.11.0
node: 10.15.1
yarn: 1.16.0

I was using createMaterialBottomNavigator from react-navigator which caused this issue for me.
I spent to whole night trying to fix this. The problem is the removeClippedSubviews is set as true. After setting it to false the clipboard appeared.
react-navigation uses react-native-paper which is also has this setting that prevents the clipboard options  from appearing.
I solved this by setting it to false in four files.
Path to files:

\..\node_modules\react-navigation-material-bottom-tabs\node_modules\react-navigation-tabs\src\views\ResourceSavingScene.js

\..\node_modules\react-navigation-tabs\src\views\ResourceSavingScene.js

\..\node_modules\react-navigation-drawer\dist\view\ResourceSavingScene.js

\..\node_modules\react-navigation-material-bottom-tabs\node_modules\react-navigation-tabs\dist\views\ResourceSavingScene.js

\..\node_modules\react-native-paper\src\components\BottomNavigation.js

--
removeClippedSubviews={
                 // On iOS, set removeClippedSubviews to true only when not focused
                 // This is an workaround for a bug where the clipped view never re-appears
                 Platform.OS === 'ios' ? navigationState.index !== index : true  //<--  set this to false
               }

Yes @AzizStark  You are right. It worked for me. I have one question here. We don't push node moudles to github so if someone gets pull he will not be able to get our updates. So how can we fix that.
Also if we deploy our code to server we dont deploy node modules so on server it will not be fixed.
So is there any solution for it.

@PrakashRNale Use patch-package https://github.com/ds300/patch-package

Same problem. Please, fix it.


Add DoubleEndedIterator to U32Digits and U64Digits

Since U32Digits and U64Digits do not implement DoubleEndedIterator, there is currently no easy way to iterate over the digits in a big-ended fashion.

Since U32Digits and U64Digits do not implement DoubleEndedIterator,

Did you explore what it would take to add that implementation instead?

I'd thought about it a little but went with this instead -- doing a quick mockup, it looks like that won't be as difficult as I first thought, so I'll give it a go.

It's definitely a lot cleaner than I expected; I think we should go that route instead.

@cuviper Thoughts on the new implementation?

It looks good to me, thanks!
bors r+

Hmm, that failure with arbitrary is not your fault -- I'll see what I can do about this...

bors r+


Remove unnecessary check for existing cert, due to certbot handles by…

The code sample can be found here:
https://github.com/certbot/certbot/blob/7f0fa18c570942238a7de73ed99945c3710408b4/certbot/certbot/_internal/main.py#L288
I have to make this change, due to AWS Eb resetting Nginx conf each time.
Since certbot handles cert check by itself, I think it's useful to move the decision to outside.

Btw, it won't solve the problem of triggering rebuild after changing any EB config. For instance, changing the env variable would lead to a rebuild, with the following nginx.conf removal. But there is no hook involved in that type of operation.
https://stackoverflow.com/questions/64244878/how-to-call-script-on-every-aws-elasticbeanstalk-change-attempt

@kaldown Certbot does handle that, but I wouldn't call it unnecessary since the main purpose was to log. Also, I'm not quite sure what you mean by nginx.conf removal? Use an nginx.conf override if the default doesn't work and you need a config to fit your needs. Below is a resource for that, but I haven't looked into whether it applies to only Java and Go environments.
https://docs.aws.amazon.com/elasticbeanstalk/latest/dg/platforms-linux-extend.html
https://stackoverflow.com/questions/23709841/how-to-change-nginx-config-in-amazon-elastic-beanstalk-running-a-docker-instance


Add name and phone number to quote converter test data

Updates test to account for changes made in commit 1c354a47.

Can you attach this to the ticket that broke it?

I don't think there was one. @nicolenorman do you know if there was a ticket which prompted that change?

Can you attach this to the ticket that broke it?

I don't think there was one. @nicolenorman do you know if there was a ticket which prompted that change?

This change was made as a part of the Apple Pay work

Noting Nicole's comment above, I guess we can say that this is related to CHK-4371.


feat: add vscode extension

Description
Here is the code that provides the language an extension in VSCode. You can take a look here for more information.
https://marketplace.visualstudio.com/items?itemName=FormKL.formkl
Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

@all-contributors
please add @imrim12  for author.
please add @tailtq for extension.
please add @quangtudng for maintainer.
please add @bongudth for documentation.

@all-contributors
please add @imrim12 for maintenance.
please add @tailtq for plugin.
please add @quangtudng for maintenance.
please add @bongudth for doc.

@all-contributors please add @quangtudng for maintenance.


Is it possible to wrap component with Fragment, without additional div?

See question in topic.
Thanks!

Yes, you can!

How can I do this?
Could you please share some option or prop to achieve this?
Thanks!

I've provided className for the container, but now it turns all children to be animated separately, but I need to have only parent element animated.


Could not find method compile()

When I try to implement this module I get compile errors:

FAILURE: Build failed with an exception.

Where:
Build file '/Users/bomalley/Documents/React/Acthar/android/build.gradle' line: 9

What went wrong:
A problem occurred evaluating root project 'Acthar'.
Could not find method compile() for arguments [project ':ReactNativeDropdownAndroid'] on >org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler_Decorated@2f98635e.

Here's the build.gradle:
// Top-level build file where you can add configuration options common to all sub-projects/modules.

buildscript {
    repositories {
        jcenter()
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:1.3.1'

        compile project(':ReactNativeDropdownAndroid')

        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }
}

allprojects {
    repositories {
        mavenLocal()
        jcenter()
    }
}

Not sure how to proceed.  Any ideas?

My bad.  I made the changes in the wrong build.gradle.  I put them in android/build.gradle instead of android/app/build.gradle.
All good!


What's the difference between the installation options?

Thanks for making salad ui! I think it'd be helpful to clarify what the difference between the two installation options. Why would I choose one over the other?

Thanks @jamauro I should update document to clarify the different.

salad.init --as-lib configure all necessary config to use SaladUI as a library. Before release v0.10 you have to manually copy tailwind config, update app.css, app.js, ... yourself.

salad.init would do all works as salad.init --as-lib does, it also copies helper files, setup component directories, ... then you can use salad.add <component> to install (copy) selected component to your project. This way you can customize the component the way you want because those components are parts of your project source code.

Why I make salad.init default to what I explained above. When I started developing SaladUI, I intended to create a collection of component that user just copy and use whichever component they need.

Interesting. If I'm understanding correctly, it sound like when using --as-lib you wouldn't be able to customize the component at all. Good to know. For my use case, it sounds like I don't want to use --as-lib.
Here's what my experience was:
I tried both ways of installing to see if I could understand what the difference was. I tried --as-lib first. After it was done installing, it instructed me to add the components one by one with mix salad.add. My intuition was that with --as-lib I wouldn't need to add each component one by one. It also seemed to be a bit slow when adding each component. It took ~5 seconds to add each one.
So then I tried the other way and it was much faster, nearly instantaneous when adding each component.

oh my bad, I should update the instruction for mix task. When using --as-lib you don't need to add anything. Just import and use the component, ex: import Salad.Checkbox

and you can customize the class

document updated in #81


Bad performance on NLVR2

Hi, thanks for releasing your code! I'm not able to reproduce your fine-tuning result on NLVR2. I followed your instructions by downloading the pre-trained model, downloading the image features, pre-processing the nlvr2 JSON files, and running the nlvr2_finetune.bash script as is. However, I get the following results, which are much lower than the result you reported. Do you know why this might be happening?
Epoch 0: Train 52.32
Epoch 0: Valid 50.86
Epoch 0: Best 50.86
Epoch 1: Train 50.50
Epoch 1: Valid 49.14
Epoch 1: Best 50.86
Epoch 2: Train 50.56
Epoch 2: Valid 49.31
Epoch 2: Best 50.86
Epoch 3: Train 54.83
Epoch 3: Valid 51.65
Epoch 3: Best 51.65

Many thanks for running the experiment and point this issue out!!!
I am now running a verified experiment and would let you the result tomorrow morning.
PyTorch Version
My initial guess is the PyTorch version. Could you help to try torch==1.0.1? Installation command:
pip install --force torch==1.0.1

I found that I used an old virtualenv with an old PyTorch version. I was supposing that PyTorch should be backward-compatible in computing gradients but it seems not the case.
By the way, here is a full list of my virtualenv. I believe that the only difference might be the torch version.
Package         Version
--------------- ---------
boto3           1.9.205
botocore        1.12.205
certifi         2019.6.16
chardet         3.0.4
docutils        0.14
idna            2.8
jmespath        0.9.4
numpy           1.17.0
pip             19.2.1
python-dateutil 2.8.0
requests        2.22.0
s3transfer      0.2.1
setuptools      41.0.1
six             1.12.0
torch           1.0.1
tqdm            4.33.0
urllib3         1.25.3
wheel           0.33.4

If so, it's really strange but I will update requirement.txt first.
Raw Feature
And could you also try to use the raw feature from our server in replace of the feature from zip files with the command:
wget nlp.cs.unc.edu/data/lxmert_data/nlvr2_imgfeat/train_obj36.tsv -P data/nlvr2_imgfeat
wget nlp.cs.unc.edu/data/lxmert_data/nlvr2_imgfeat/valid_obj36.tsv -P data/nlvr2_imgfeat

In case there are some broken zip files.

Hi, I got the result of accuracy 74.39% (within the range 74.0% to 74.5% in README) with the same command in README last night.
Here is a snapshot of the results:

I would recommend still trying torch==1.0.1 if possible.

Thanks for the update! Yes, it does look like the torch version was the issue. (So far I have re-trained for 1 epoch using pytorch 1.0.1 and got a validation accuracy of 67.86.) Previously, I was using version 1.1.0.


Add logging support

Allows for debug, info, warning, error, fatal logging with various loggers

@jeeyoungk @Nathan-Fenner

LGTM. can you try to see whether the build works on square's internal repo before merging?


IPTT 500 error and program page failure: SAFAL II program

Program page error
I'm getting an "ValueError at /program/457/" error page when I try to load the program page, for that same program - https://tola-activity-qa.mercycorps.org/program/457/
Here's the time stamp: Mon, 15 Jul 2019 11:31:09 -0700
IPTT error
I'm getting a 500 Internal server error when I try to load the Recent progress report for SAFAL II: Sustainable Access to Finance and Livelihoods:
https://tola-activity-qa.mercycorps.org/indicators/iptt_report/457/timeperiods/?frequency=6&start=0&end=2

Initial assessment:  This seems to be restricted to numbers with 1 number followed by one letter (e.g. 2a).  Changing the output_field and/or the casting in IndicatorSortingQSMixin fixes the error, but may also mess up the sorting that is output.  Because the conditions that lead to this error are so specific, it affects very few programs.  Was able to confirm that program id 340 has the same issue, because of numbers like 2a associated with some indicators.

I am able to view the SAFAL program page now.


Images are out of date

The images for the Time To Live page appear to be from an older version.
Current Screen

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

ID: ae1b1c71-a8ca-2bba-202e-f74a9414ced4
Version Independent ID: 65ce6b13-2e3d-39ea-167a-ebdc1864f66b
Content: Configure and manage Time to Live in Azure Cosmos DB
Content Source: articles/cosmos-db/sql/how-to-time-to-live.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @anfeldma-ms
Microsoft Alias: anfeldma

@Mjanofsky Thank you for your feedback. We are looking into this and will get back shortly.

@anfeldma-ms Please see customer feedback above. We don't have access to update image in documentation. Could you please help us with PR.

#reassign:@SnehaGunda

@mjanofsky The above PR fixes the screenshot and some text. Changes will go live in a day. We will close this issue.
#please-close


binder Frontmatter field is ignored by thebe

Right now, it's not easy to specify a custom mybinder URL, e.g. the obvious:
version: 1
project:
  binder: https://binder.opensci.2i2c.cloud/
  github: https://github.com/2i2c-org/demo-icepyx

Instead, we have to write:
version: 1
project:
  jupyter:
    binder:
      repo: https://github.com/2i2c-org/demo-icepyx
      url: https://binder.opensci.2i2c.cloud/

  github: https://github.com/2i2c-org/demo-icepyx

This seems to be because we don't listen (at all?) to the project.binder Frontmatter entry for anything.

yes, correct I think we should remove that. Historically, way back(TM), project.binder was being used to display a classic [launch | binder] badge in the frontmatter, prior to even thebe being integrated IIRC. The theme's don't support it so we should drop it.


FEAT-#2087 Added benchmarks test suite

Benchmarks added for the following functions:

DataFrame.sum()
DataFrame.agg(sum)
DataFrame.agg("sum")
GroupBy.sum()
GroupBy.agg(sum)

Signed-off-by: Gregory Shimansky<EMAIL_ADDRESS>
What do these changes do?

[x] commit message follows format outlined here
[x] passes flake8 modin
[x] passes black --check modin
[x] signed commit with git commit -s
[x] Resolves #2087
[x] tests added and passing

@gshimansky , are there any thoughts on expanding of the benchmarks?

@gshimansky , are there any thoughts on expanding of the benchmarks?

Sure, this commit is just the start. Right now I have plans for merge, mul, mask based indexing, maybe more will emerge while benchmarks performance is investigated.

Okay, I am just interesting on how will benchmarks architecture be organized. There changes looks good to me. @devin-petersohn , @anmyachev , any comments?

Okay, I am just interesting on how will benchmarks architecture be organized. These changes look good to me. @devin-petersohn , @anmyachev , any comments?

Currently I plan to add them to the same file. It is still very small. It is grows big, we can split it based on whatever criteria that we like, but this moment is quite far from now.

@gshimansky , please, answer this question.

@gshimansky , please, answer this question.

If you mean regular CI runs, I don't have plans for benchmarks so far. If you mean irregular runs to test performance, you can try to run these benchmarks anywhere you run other Modin applications.

As I see, we will track performance manually for now. It would be good to think of automatic checking the performance in future.

I will let @anmyachev take a final pass to verify prior comments are resolved.


Retieving the path to liblldb from the lldb executable

Theoretically, we could parse the lldb executable's ELF header and acquire the path to liblldb. Doing so could eliminate the need to separately config the path to executable and the path to liblldb and thus simplify the configuration.

"Use Alternate Backend..." command does that.  Not by parsing ELF headers, but to the same effect.


cmd ref: document releasing of locks in dvc run

Context: https://github.com/iterative/dvc/pull/2584

What do you mean by "us"? Thanks

Hi there! @efiop can you confirm the given context is still up to date and/or copy the parts that now affect repro and exp run? Thanks

Hello again. Is this the what we're talking about? From https://github.com/iterative/dvc/pull/2584#issuecomment-564391845:

Adjusted the schema and made rwlock reentrant, so @rwlocked is now placed on public methods of Stage, which is much nicer than what it was before.

I don't think any user has ever needed to know about this so it sounds like an implementation detail we don't need to document.
Please reopen if I'm incorrect.

I think it's about the user visible behavior - now you could run other DVC commands when dvc run itself is running. Before it was not possible. Since this behavior is more optimistic it is not clear if we need any explicit notes. Only in places (if we have it) where we are clearly stating that no commands could run in parallel, or we discuss locks somewhere.

Hmmm well in general the repo is supposed to be locked during any DVC operation or at least that's implied by https://dvc.org/doc/user-guide/troubleshooting#lock-issue . Is that outdated?
We could separately expand on the description of what .dvc/tmp/lock does in https://dvc.org/doc/user-guide/project-structure/internal-files . Other than that I could not find any text mentioning of this limitation so that should be enough to close this.
I started #3251 for this.

the repo is supposed to be locked during any DVC operation or at least that's implied by https://dvc.org/doc/user-guide/troubleshooting#lock-issue . Is that outdated?

Hi @dberenbaum (me again 🙂 ) do you know if this info. is outdated? Would be great to get clarification from the core team on what needs to be updated here, if anything (otherwise please close this issue). Thanks

the repo is supposed to be locked during any DVC operation or at least that's implied by https://dvc.org/doc/user-guide/troubleshooting#lock-issue . Is that outdated?

Hi @dberenbaum (me again 🙂 ) do you know if this info. is outdated? Would be great to get clarification from the core team on what needs to be updated here, if anything (otherwise please close this issue). Thanks

There is a big improvement here for users, because when the user cmd is being executed in a stage, other dvc commands may be run. The repo will still be locked before when dvc is setting up and and after when collecting results of the stage.
However, I don't see any changes needed here because the docs don't address when the repo will or won't be locked, so I think let's close the issue. If we ever put in an in-depth locking guide, we can revisit.

Good to know, and sounds good. Thanks


Webpack config file.

Making dist webpack file sourcemap debuggable for end users who build on
top of react-storybook.
This code review addresses issues of #457
Since the react-storybook is used as a toolkit for developers to build custom react components on top of it, changing this option allows a relatively light weight debug friendly version for other developers to build and debug their separate components built/demoed on react-storybook.

Hi @jennylia
The dist directory contains transpiled code. Can you change files in the src directory:
https://github.com/kadirahq/react-storybook/tree/master/src/server/config

@mnmtanish Thank you for your comment.
I have reverted my previous commit and added a commit that changes in the webpack config file. I have tested this against "npm run prepublish" and verified that it indeed change the webpack.config file for the dist directory.

We covered this with: https://github.com/kadirahq/react-storybook/pull/577

Thanks for the PR.


feat: Use domain-qualified finalizer

Revert 1580b68 'Revert the default policy server finalizer in use (#468)'
Description

Bring back https://github.com/kubewarden/helm-charts/pull/460

Test

Tested on the original PR.

Additional Information
Tradeoff

Potential improvement

@viccuad the diff looks strange, only the chart version is changed (going backwards). Can you explain what is going on please?

@flavio Indeed, the version bump is wrong. It is also missing the actual revert, as it already is in main.
Closing as not needed.


Exporter blocked on prod due to unacked messages in rabbitmq

to-do:

[x] get list of submissions currently in exporting status on prod

[ ] delete prod exporter

[ ] purge ingest.terra.experiments.new queue in rabbitmq prod

[ ] redeploy exporter

[ ] re-export the submissions that were in exporting status

current submissions in exporting state on prod:
https://contribute.data.humancellatlas.org/submissions/detail?uuid=96e500d8-11e9-4d25-a2e5-817e0628199f
https://contribute.data.humancellatlas.org/submissions/detail?uuid=ae4b73ba-7b74-4fc2-b36c-13fed3fa7ca4
https://contribute.data.humancellatlas.org/submissions/detail?uuid=e86165dd-28a1-4791-8a78-bc7e9fbda7df
https://contribute.data.humancellatlas.org/submissions/detail?uuid=7a81ac3e-e824-4c03-9bb5-f862ced23860
https://contribute.data.humancellatlas.org/submissions/detail?uuid=2e43c12e-30fe-4c77-8d35-829063049d3c
https://contribute.data.humancellatlas.org/submissions/detail?uuid=7b4cd093-bfa5-477e-9c95-69bafc1cb6bf
https://contribute.data.humancellatlas.org/submissions/detail?uuid=bf3116e5-1af1-46c2-8bbd-44dac49d1e7f

deleted deployment using command
kubectl delete deployment ingest-exporter

before purging queue on prod rabbitmq:

after purging using rabbitmq ui:

redeployed exporter using make deploy-app-ingest-exporter image=quay.io/ebi-ait/ingest-exporter:prod-1ba52fe9-19-05-2021.1621445881


hotkey to move to next/previous error should not move NVDA's focus outside the editor

VSCode Version: Code - Insiders 1.20.0-insider (f4ed56b0d61050f6a94cfa033a058fabfdc0d651, 2018-01-09T05:13:28.256Z)
OS Version: Windows_NT x64 10.0.15063
Extensions (2)

Extension
Author (truncated)
Version

vscode-markdownlint
Dav
0.12.0

markdown-preview-enhanced
shd
0.3.2

Steps to Reproduce:

Create a file that has a lint error. The example I used was:

# Heading 1

* HTML
- LaTeX

## heading with trailing space

Save the file as "test.md". With NVDA running and vscode-markdownlint, press f8 or shift+f8.

Expected result:
The error/warning is read. Keyboard focus stays in the editor, and I move to the offending line.
Actual result:
Keyboard focus exits the text area and lands on the text of the error. For e.g, NVDA would say
[markdownlint] MD004/ul-style: Unordered list style [Expected: asterisk; Actual: dash]

Focus is now on the text of the alert. I can quickly move to the edit area and press enter to enter forms mode, and focus is placed on the originating line.
It would be neat if the focus stayed inside the editor though to save this extra step.

This works fine for me on macOS.
The focus stays in the editor and the error is read out.
@Neurrone can you still reproduce this with latest vscode?

@isidorn yup, this is still happening. If I press f8, my focus moves outside the textarea to where the error message is.

@Neurrone aha, only when the screen reader is detected we actuall move focus.
Would you prefer that we read out the content of the error using an aria.alert.
Since currently we move focus only so the screen reader would read the content without us using aria.alert.
Or do you have some other recommendation?

@isidorn yes, having it use alerts would be ideal.
However, when focus moves to the source of the error, I don't know if that is currently being announced. I.e, the alert might interupt the focus movement announcement, if one is present.

Currently it would read the word where it navigates to.
I have changed that now we use aria.alerts.
Can you please try vscode insiders tomorrow and let me know how it behaves? And if you like it or not.
Thank you very much for your help
fyi @jrieken

@isidorn I've just tested with latest build.
When pressing F8 to move to the next error for a markdown file, this is what I'm hearing:
MD032/blanks-around-lists: Lists should be surrounded by blank lines (occurred 2 times)
alert  MD032/blanks-around-lists: Lists should be surrounded by blank lines

The reading of the first sentance is interupted with the alert.
Would be ideal if we could suppress the redundant announcement.

@Neurrone thanks for your feedback.
I have pushed a fix for this. You can try it out tomorrow.
I verified and now it behaves well for me.
We are no longer using an alert, but just aria-livereagion which just reads out the content without stealing focus. So seems to work nice from my persepctive.

@isidorn thanks a lot, this is working perfectly now :)

Glad to hear, adding verified label


Upgrading express to 4.x to remove use of deprecated dependency

Received the following warning when running 'npm install teslams':
npm WARN deprecated<EMAIL_ADDRESS>connect 2.x series is deprecated
Upgrading express to 4.x fixes this problem

Thanks for the PR


Add shortcuts section to the help box

Is your feature request related to a problem? Please describe.
It is not clear what shortcuts are available to use while annotating.
Describe the solution you'd like
Add a concise shortcuts guide to the help box.
Describe alternatives you've considered
Perhaps a shortcuts section in the docs.
Additional context
Add any other context or screenshots about the feature request here.

For example, a user mentioned that they didn't know that they could delete labels using the double-click. Maybe we should add this to the shortcuts help?

@nataliaElv would be nice to dress a list from the main shortcuts that need to be more explicit threw the app, then let's see if we can categorize them by type, and define an information architecture.
I would push the main ones into the help box and publish the extended list into the docs (linked to the help box)


@conda-forge-admin, please add user @grst

Additional comment:
No response

@gtca, could you merge this one pls?
Also the mudata one: https://github.com/conda-forge/mudata-feedstock/issues/7


Unused e2e jobs removed, email notifications disabled

@jmelis @riuvshin  could you have a look?

[test]


Does not build on GHCJS

Is it meant to? (ghcjs 0.2.0, ghc 8.0.2)
src/TextShow/Data/Text.hs:25:1: error:
Failed to load interface for ‘Data.Text.Foreign’

Hi @Saulzar. I've never been able to successfully build text-show with GHCJS (or most large Haskell projects, for that matter), so I'm certainly not surprised to learn that text-show doesn't build with GHCJS. If you know how to fix the issue, I'd gladly accept a patch.

OK, with the help of the folks at the #ghcjs IRC channel, I was able to get further than before. In fact, I was able to build the entirety of text-show-3.7.4 (the library, at least) with ghcjs-<IP_ADDRESS>, so perhaps this is simply an issue with an old version of ghcjs? This means I'd likely need to patch around the issue with some sort of CPP, but I don't know what that would be. If anyone reading this is familiar with ghcjs and knows the answer, your input would be appreciated.

Hmm. I don't quite understand the error, maybe it's got nothing to do with text-show. Looks like that module should be present - but the interface files are missing in my environment (I'm using the reflex-platform).
In this enviornment the text version is <IP_ADDRESS> which definitely has this module, and I can import it just fine compiling with ghcjs --make.

I managed to build a project using text-show without any problems using stack and ghcjs-<IP_ADDRESS>08011_ghc-8.0.2. But as I'm switching to reflex-platform I ran into the described problem as well. So I'd assume it's a problem on the reflex-platform side.

There's now an issue in the reflex-platform project: https://github.com/reflex-frp/reflex-platform/issues/466

OK. I'm not familiar with how reflex-platform does things. Is the necessary patching that has to be done to text-show something that can be upstreamed into this repo?

My current understanding is that nothing needs to be done in text-show but only in reflex-platform. text-show as it is compiles with ghcjs.
reflex-platform (when used with ghcjs) uses a fork of the text package that doesn't expose Data.Text.Foreign. And within reflex-platform there is already a place to adjust packages that are dependent on the forked text package. I'm currently looking into this and hopefully get a PR ready soon.

Are these patches tied to particular Hackage releases? My concern is that as text-show makes new releases, this reflex-platform patch will quickly become out-of-date and have to be maintained separately.

Here is the PR I made for reflex-platform (https://github.com/reflex-frp/reflex-platform/pull/468). Ryan from the reflex project is asking if you'd be interested in taken it. Maybe just have a look.

@RyanGlScott I think it's really on reflex-platform to make this work, but if you would be willing to have a flag that chops out the Foreign stuff, that would definitely make our lives easier!
For what it's worth, the reason we use a non-standard text package is that, in our test cases, it saved almost 50% of the total memory of a running program compared with the standard text implementation.  Browsers are much better at dealing with their own strings than at letting us reimplement them in terms of typed buffers!

Judging from the patch, it looks like the only part that's giving reflex-platform trouble is the I16 instance? To be honest, I have no particular reason for having that; I just blindly added it in an attempt to provide a TextShow instance to mirror every publicly accessible Show instance in the text. But I16 is really an internal component of text, and given a choice between workaround around the issue with CPP and just removing the I16 stuff altogether from text-show, I'm inclined to pick the latter (and wait until someone complains before adding it back). Does that sound like a reasonable course of action?

@RyanGlScott Thanks!  I really appreciate you sorting this out :)
@tgass Want to strip out the patch in reflex-platform?

@ryantrinkle Sure. I'll just revert the commit and open a PR.

I've uploaded text-show-3.8 to Hackage with these changes.

@RyanGlScott Awesome; thanks!


clean code with react cache

Current dependencies on/for this PR:

master

PR #7   👈

This stack of pull requests is managed by Graphite.


Implement new PoseBroadcaster controller

This PR implements a new broadcasting controller based on the PoseSensor semantic component introduced in https://github.com/ros-controls/ros2_control/pull/1775.
As described in the PoseSensor PR, my primary use-case for this is the publishing of tcp poses from robot arms. This can be more accurate than calculating forward kinematics, as robot calibrations etc. can be considered by the robot. Settings on the robot side concerning tool offsets etc. can also be included by this (at least for UR and kuka RSI). The ROS 1 ur_robot_driver used to publish this transform directly from within the hardware interface, but in ROS 2 there is not yet a standard way for providing this information. A PR for the UR ROS 2 driver already exists at https://github.com/UniversalRobots/Universal_Robots_ROS2_Driver/pull/1108.
This controller publishes poses directly as geometry_msgs::msg::PoseStampeds and as tf transforms (can be disabled).

Would you see use of this being a chainable controller at the same time?

I don't have much experience with chaining controllers, but this sounds like a good extension. I can work on that and extend this PR.

@bmagyar This should probably be backported? The semantic component has already been backported.

@mergifyio backport humble iron


Upgrading from 2.x to 4.x fails on database migrations

Environment

PHP Version: 8.1.3
Database Driver & Version: pgsql (Postgresql 14.5)
Actions Version: 4.1.0
Laravel Version: 9.41.0

Issue description
This likely won't impact many people at this stage, but I am in the process of upgrading from Laravel-Migration-Actions 2.x to Laravel-Actions 4.x, and ran into an issue because the BaseChangeColumn migration uses $this->config->table to determine the table name.
However, by 4.x, the configured default table name is 'actions' rather than 'migration_actions', and because of that, the changes in BaseChangeColumn are not actually applied.
The result was that while php artisan migrate appeared to work, but on the first run of php artisan actions, it did not recognize that any of the actions had already run and tried to run them again. An SQL error then arose after the first action completed, because it could not save the action name/batch number into the database since the 'migration' field no longer existed:
[previous exception] [object] (PDOException(code: 42703): SQLSTATE[42703]: Undefined column: 7 ERROR:  column \"action\" of relation \"actions\" does not exist
LINE 1: insert into \"actions\" (\"action\", \"batch\") values ($1, $2)

My solution was to add an additional migration in the app itself to make those changes -after- the rename was completed.
<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

return new class extends Migration {
    public function up(): void
    {
        Schema::table('actions', function (Blueprint $table) {
            $table->renameColumn('migration', 'action');
            $table->unsignedInteger('batch')->change();
        });
    }
};

Steps to reproduce
php artisan migrate followed by php artisan actions after upgrading from 2.x to 4.x

First you need to upgrade to version 3 and then to version 4.
At the same time, do not forget to execute the commands at each stage:
2 to 3: php artisan migrate:actions:upgrade
3 to 4: php artisan actions:upgrade
More detailed steps for upgrading can be found in the documentation: https://actions.dragon-code.pro/prologue/upgrade-guide/


src/createStore

Would love to have a discussion around this in the high-level context of the entire redux process/flow. There were already a lot of comments in the file, which I found extremely helpful in interpreting this code.

Great work!


Do not use cpuid.h on Windows

According to msdn docs __cpuid/__cpuidex/_xgetbv are provided by intrin.h and should be used on windows

this works equally with mingw/clang-cl. On the contrary, if intrin.h was already included cpuid.h won't work with mingw

In general, we try to include the smallest usable header file to speed up the compilation... When including headers on Windows, it's not always possible, so we have to include header that pulls in a lot of unnecessary lines or files.
However, supporting more than just Visual C++ has been our plan from very beginning and we have gradually improved support for MinGW, MSYS and Cygwin.


Zappa 0.3.1 doesn't work with node 0.6.0

Zappa depends on express 2.4.x which depends on node (>= 0.4.1 < 0.5.0).
express 2.5.0 is available for node 0.6.0, so perhaps zappa's dependency can be updated?

It always takes a couple of days to shake out dependency updates, no doubt Zappa will be 0.6.0 ready in no time.
I haven't read through node 0.6.0 updates yet, any significant changes?

Figures the day I learn about zappa, I can't use it :(

Quick work around:
(tested with node v0.6.1)
$ mkdir myproj
$ cd myproj/
$ npm install zappa --force

With the --force flag<EMAIL_ADDRESS>is installed (check with npm ls in the myproj directory), despite the engine version mismatch indicated in its package.json. I created a test server.coffee, fired it up, and it seems to work.  Obviously, a better solution is for zappa to be updated/upgraded and tested with respect to all its dependencies.

I've just been testing zappa with the latest version of all its dependencies, on node 0.6.1 and it seems to all be working well.

Published v0.3.3 (v0.3.2 had a publishing error) which works with express 2.5.x and node 0.6.x. Had to fix two minor issues for the tests to run (a38d2f1d9c43045e32c1379dd7640bc5a3c6e92a and 24e626c43752e3789bedf0beaeec100b1103c682). If you find anything else, feel free to reopen.


Bug - Accessibility TalkBack : Header Navigation

Prerequisites

[X] I have searched the backlog for duplicate or closed feature requests

Your test device
Samsung S20
OS Version
Android 11
App version
1.0
Describe the issue
Activate Talkaback
Launch the application
Go to Colour
Try to navigate by heading : (By swiping up and down)
Talkabck say "Aucun élément de titre suivant"
Expected Behavior
with header navigation (swiping up and down) Talkabck should navigate to the next item in the screen
For the screen "Colour " :  The user must be able to navigate by sections from Core to functional to supporting without vocalising all items (colours).
Suggestion:   https://a11y-guidelines.orange.com/en/mobile/android/development/vocal-navigation/#enable-header-navigation

Header navigation ok for this screen : I can navigate between different items correctly


Set image size?

When I attach a large image (iOS screenshot), and then press send, it asks if I want to change the image size to make the email smaller.
Question: Is there a way to programmatically set this to "medium" before the mail dialog is presented?
Thanks for the excellent package, it works great.

Never mind! I just resized the UIImage before attaching, works great


Failed to download file: Invalid API response from server: type mismatch

think the API has been changed?

oops the server was just down


Usage with buffer?

Can I use this without touching the file system? Is there a way to pass pdf as buffer to input, and get image buffer from the output?

+1

I would also be interested in knowing this.


Enhancement: Zoom amount and ZoomTo

Added optional amount parameter to delegate’s zoomIn and
zoomOut methods
Added zoomTo method to delate

:+1: thank you


"Recent Objective / Efforts" should not display any efforts that do not currently belong to the report author's advisor organization.

Condition:

The report author previously created engagement reports and assigned efforts to engagement reports.
The report author's advisor organization no longer has any efforts assigned.
The report author creates a new engagement report.

Result:

The "Efforts" data field does not display any efforts. (Good)
The "Recent Objective / Efforts" displays efforts from previous engagement reports.  (Bad)

Correct Functionality:

"Recent Objective / Efforts" should not display any efforts that do not currently belong to the report author's advisor organization.

This ticket can be closed.
Only ANET administrators will still see "Recent Objective / Efforts" displayed.


vm: fix the error of 'Too many open files' when do 'vm image list --all'

Fix #3649
lsof -p <p-id> showed the active socket number peeked at less than 100 vs 1000 before the change.
A whole live query returns 4000+ images after around 50 seconds. Supplying either --publisher or --offer take way less time at < 10 seconds.
General Guidelines

[x] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #3662 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3662      +/-   ##
==========================================
- Coverage   72.38%   72.37%   -0.01%
==========================================
  Files         422      422
  Lines       26299    26301       +2
  Branches     3977     3978       +1
==========================================
  Hits        19036    19036
- Misses       6032     6036       +4
+ Partials     1231     1229       -2

Impacted Files
Coverage Δ

...azure-cli-vm/azure/cli/command_modules/vm/_help.py
100% <ø> (ø)
:arrow_up:

...re-cli-vm/azure/cli/command_modules/vm/_actions.py
75% <0%> (ø)
:arrow_up:

...zure-cli-vm/azure/cli/command_modules/vm/custom.py
77.01% <0%> (-0.17%)
:arrow_down:

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
72.1% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Encoder reverse direction setting not the same from main menu to sub-menus

I'm on the 1.4.18 firmware.
Encoder direction is set to 'off'. Turning the encoder in the main patch view results in a different direction of scrolling than in the sub-menus. They are reversed.

Thank you for reporting this. I'm experiencing the same issue.
The main menu to select a component to Add to the patch scrolls normally, but as soon as I enter the Inputs menu, the scroll direction is backwards and confusing.

Please retest on 1.99 and let me know your thoughts.

Hi @duddie !
Just tested this again using the latest 2.0 firmware and I'm still having some issues.
✅ It seems the scroll direction has been fixed for the main patch view and when scrolling through the parameter groups while editing a patch device's inputs (🙌🏼!)
❌ However, one level deeper when selecting a single parameter at a time, the scroll direction is still backwards.
Here's a video showing the scroll behavior. The backward scrolling starts at about 14 seconds in the video:
https://user-images.githubusercontent.com/4134679/190942089-b93906db-a335-4cec-b8ae-1affcb6fe637.mov

OK, will fix.
You are aware you do not need to get into individual settings now? The 4 components of inputs can be changed by using LVL1-4 and ATT1-4 to change value and input source respectively.
Please check user-manual.pdf :)
But yeah, I will patch it today.

The latest version should have all encoder reverse issues resolved. Also there are 4 settings for reversing encoder now.


Update readme.md - [closed]

In GitLab by @wknet123 on Jan 15, 2018, 09:29
Merges dev-gogs-test -> dev

In GitLab by @wknet123 on Nov 13, 2019, 10:24
closed


fixed attachment bug in 8.8 server

We should create a story to have an automated test for attachments (perhaps in the MediaCo tests) added soon. @4manasa

We should create a story to have an automated test for attachments (perhaps in the MediaCo tests) added soon. @4manasa

okay John.


feat(wip): wasm impl

Re: #94
Creates flipt-client-wasm engine that we can use in a new browser capable SDK that will resolve #84
Still need to package it correctly and implement the browser capable SDK that will resolve the Flipt state and pass it to the WASM engine
TODO

[ ] create flipt-browser SDK to consume this library
[ ] add ITs to our dagger IT suite
[ ] publish flipt-browser SDK

Codecov Report
Attention: Patch coverage is 0% with 55 lines in your changes are missing coverage. Please review.

Project coverage is 82.91%. Comparing base (8f96af6) to head (786b97b).
Report is 2 commits behind head on main.

Files
Patch %
Lines

flipt-engine-wasm/src/lib.rs
0.00%
54 Missing :warning:

flipt-evaluation/src/lib.rs
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
- Coverage   84.26%   82.91%   -1.35%
==========================================
  Files          10       11       +1
  Lines        3050     3131      +81
==========================================
+ Hits         2570     2596      +26
- Misses        480      535      +55

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


How to trigger reset event ?

How I used?
I used this inside the form.
  <form>
   {{dropzone event1=(action 'actionName') }}
 </form>

Problem or Requirement
I don't know how to trigger the reset event. I am supposed to reset the elements in the form to their
initial status.
I did not find anything related inside the docs.

Closed due to inactivity


Cannot open file ios/Runner.xcodeproj/project.pbxproj

Steps to Reproduce

Flutter Run

Logs
command
flutter run
exception
FileSystemException: FileSystemException: Cannot open file, path = '/Users/christian/Documents/coding/spirit4/spirit4/ios/Runner.xcodeproj/project.pbxproj' (OS Error: No such file or directory, errno = 2)

#0      _File.open.<anonymous closure> (dart:io/file_impl.dart:368:9)
#1      _rootRunUnary (dart:async/zone.dart:1132:38)
#2      _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#3      _FutureListener.handleValue (dart:async/future_impl.dart:129:18)
#4      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:642:45)
#5      Future._propagateToListeners (dart:async/future_impl.dart:671:32)
#6      Future._completeWithValue (dart:async/future_impl.dart:486:5)
#7      Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:516:7)
#8      _rootRun (dart:async/zone.dart:1124:13)
#9      _CustomZone.run (dart:async/zone.dart:1021:19)
#10     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#11     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#12     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#13     _runPendingImmediateCallback (dart:isolate/runtime/libisolate_patch.dart:115:13)
#14     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:172:5)

flutter doctor
[✓] Flutter (Channel beta, v0.9.4, on Mac OS X 10.13.6 17G2307, locale en-TH)
    • Flutter version 0.9.4 at /Users/christian/documents/coding/flutter
    • Framework revision f37c235c32 (2 weeks ago), 2018-09-25 17:45:40 -0400
    • Engine revision 74625aed32
    • Dart version 2.1.0-dev.5.0.flutter-a2eb050044

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.1)
    • Android SDK at /Users/christian/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.0, Build version 10A255
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 27.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] VS Code (version 1.28.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.19.0

[✓] Connected devices (1 available)
    • iPhone XS Max • F701DB9C-85B5-425E-A5B8-9749A44CA6D2 • ios • iOS 12.0 (simulator)

• No issues found!

Pubspec:
dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^0.1.0
  scoped_model: "^0.3.0"
  http: "^0.11.3+16"
  shared_preferences: "^0.4.3"
  rxdart: "^0.18.1"
  map_view: "^0.0.12"
  location: "^1.4.1"
  image_picker: "^0.4.5"
  mime: "^0.9.6"
  url_launcher: "^3.0.2"

dev_dependencies:
  flutter_launcher_icons: "^0.6.1"

I had this error today and I found that the file in question was actually in the .ios directory instead of ios. So I flutter createed to repair the project and removed the .ios directory and now it works.

I have the same issue.
@qtbeee What do you mean by "flutter createed"?  flutter create is not a command I can run to repair anything, just to create new projects.

@JonasJW you can indeed use flutter create . to "repair" the current project to a limited degree.
If the ios/ or android/ folders are missing, they are created.
You can use this to switch native language Java/Kotlin, ObjC/Swift or to get the generated files updated to match newer Flutter versions by deleting the directories and running flutter create . [options]. You need to re-apply previously made manual changes yourself though.

@ChristBKK did @qtbeee's suggestion apply to your problem as well?

flutter create . indeed solved the problem for me. Thank you!

@JonasJW  flutter create . works for me. Thanks!

Delete the flutter cache folder located at /Users/username/Documents/flutter/bin/cache and run flutter doctor, it will download all dependencies again. Worked for me.

flutter create . 解决了问题，非常感谢

The only solution for me is to remove the build folder: rm -rf build/


Setup Appveyor

Win32 and Win64 builds
MSVC 2013/2015/2017
I'll settle for just one build, but a build matrix would be great.

Thanks @AnotherFoxGuy
our first builds: https://ci.appveyor.com/project/psi29a/xlengine


compile 'com.commit451:PhotoView:1.2.4' not working

i can't see your lib classes  with this gradle in my code .

Are you saying it won't resolve at all or just you cannot see the source?

When i try to call your classes after syncing gradle  compile 'com.commit451:PhotoView:1.2.4'
i can't see any.
I used this one instead compile 'com.github.chrisbanes.photoview:library:1.2.2' and it worked fine

+1 Gradle build does not complain about not being able to find the library, but PhotoView classes are not accesible after compile.

I use this :
compile 'com.github.chrisbanes.photoview:library:1.2.4'
And it works properly for me. But perhaps this is not the right way to do.

It is possible that you are include mavenCentral and not jcenter. This dependency now requires you to use jcenter

I added the line maven { url "https://jitpack.io" } to the root build.gradle file but doesn't work for me Error:Could not find com.github.chrisbanes:PhotoView:1.3.0.

allprojects {
    repositories {
        jcenter()
        maven { url "https://jitpack.io" }
    }
}

worked for me!

right link is
implementation 'com.github.chrisbanes.photoview:library:1.2.4'

Author should change link in README

Thikn's :pavel-ismailov

implementation 'com.github.chrisbanes.photoview:library:1.2.4'

Looks like the good answer but I don't know in what file I'm supposed to change it, could you be more specify?

implementation 'com.github.chrisbanes.photoview:library:1.2.4'

Looks like the good answer but I don't know in what file I'm supposed to change it, could you be more specify?

Here ../node_modules/com-sarriaroman-photoviewer/src/android/photoviewer.gradle
Replace all the 'compile' with 'implementation'

@sashakalapusha: I tried but still same error.

@ddesignercoder I've replaced all the 'compile' with 'implementation' and re-open Android Studio.
Now photoviewer.gradle looks:
`repositories{
jcenter()
}
dependencies {
implementation 'com.commit451:PhotoView:1.2.4'
implementation 'com.squareup.picasso:picasso:2.71828'
}
android {
}`

Hi all, I encountered this issue also. May I know if these works after you add in the specific .gradle file in your android folder?
I would like to ask.. where's did you input these implementation in your gradle file

Configure project :app
WARNING: DSL element 'dexOptions' is obsolete.
It will be removed in version 8.0 of the Android Gradle plugin.

FAILURE: Build failed with an exception.

Where:
Script '/Users/runner/work/1/s/node_modules/com-sarriaroman-photoviewer/src/android/photoviewer.gradle' line: 6

What went wrong:
A problem occurred evaluating script.

Could not find method compile() for arguments [com.commit451:PhotoView:1.2.4] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

implementation 'com.github.chrisbanes.photoview:library:1.2.4'

Looks like the good answer but I don't know in what file I'm supposed to change it, could you be more specify?

Here ../node_modules/com-sarriaroman-photoviewer/src/android/photoviewer.gradle Replace all the 'compile' with 'implementation'

I tried this, but it still threw the same err. Hence, I updated the code in

I tried this, but it still threw the same err. Hence, I updated the code at,
'platforms\android\com-sarriaroman-photoviewer<app-name>-photoviewer.gradle'
This worked for me. The above path was available after build ('ionic cordova platform add android').

I tried this, but it still threw the same err. Hence, I updated the code at,
platforms\android\com-sarriaroman-photoviewer<app-name>-photoviewer.gradle
This worked for me. The above path was available after build (ionic cordova platform add android).


We have a muscle-controlled cyclist!

Likes to steer to the right a bit too much. Potentially a bug in the geometry of the musculotendons, or it could be that the initial positive roll rate interacts with the delay in the (very simple) controller in a way that means our cyclist can't damp out the initial perturbation. I think an improved controller will help here, but this is good enough for now, it at least shows that they rider can stay upright on the bike.

I pulled it and it wasn't stable for me, but I did a little refactoring and just have it checking whether a simple pulse in muscle activation causes counter steering and it does. I've pushed some changes to this branch.

Here it is:

https://github.com/brocksam/sympy-biomechanics/assets/276007/9a4c4900-0717-45a4-8267-5ffeee0cd3f7

I had to swap the sign on the control (still confused about that). Need to investigate.

Merging this, as it runs.


add/update trait/method/scale definitions via BRAPI

From @rfinkers on July 29, 2016 12:53
Currently, there is no means to update / add new observations via the API. Only get. Having such functionality would be useful.
Copied from original issue: plantbreeding/Documentation#18

From @jeback1 on July 29, 2016 14:22
There are a few calls that have an update method.  I suggest that full CRUD should be considered for version 2 but would be more than we would want to take on for version 1.


The API would be something like .list.slice(<start>, <length> or .list.slice(<start>, <end>). It seems that different query engines expose different APIs so it's not clear which API would be more appropriate.

Closing with https://github.com/Eventual-Inc/Daft/pull/2479


Update GHA actions versions for happy stuff

Summary:

What: Updates the various GHA actions (not the Happy version, the underlying action)
Ticket: None
Env: None

Notes:
Changes made per discussion with Alex, see Slack here
https://chanzuckerbergteam.slack.com/archives/C01C310V70X/p1670027223992869?thread_ts=1668438062.542359&cid=C01C310V70X
Checklist:

[x] I merged latest trunk

For posterity, this is using the latest version as of right now for each action from this release manifest: https://github.com/chanzuckerberg/github-actions/blob/main/.release-please-manifest.json


BrowserslistError: Unknown browser query browsers

The example simply fails with this error:
BrowserslistError: Unknown browser query `browsers`

Is this a known issue?

"browsers" in the example is a list of browsers, not the word "browsers". Here's the relevant explanation in the Autoprefixer docs.

Ah, now I see that row explaining what browsers is. Thanks! :+1:


Memory crash on start

🔥 Something broke!
Minecraft crashed with memory issue
In the crash-report
Methane 1.7
crash-2023-07-15_15.38.48-client.txt
Does not happen when Methane uninstalled

Oh I'm sorry
I provided wrong crash report

I'll send a new one in a few minutes

I'm going to assume that this issue is... fixed? Make a new issue/discussion if you do need help.

I'm sorry, I was wrong in reporting this issue, this was not your mod bug

all good, no worries!


make breaking on macos m1

clang -v
Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1)
Target: arm64-apple-darwin22.4.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

terminal output
make
* Detected `kqueue`
* Detected `sendfile` (Apple)
* Detected 'tm_zone' field in 'struct tm'
* Detected native socket API, without additional libraries
* No compatible SSL/TLS library detected.
* Detected Little Endian byte order.
* Compiling lib/facil/fio.c
        1.00 real         0.95 user         0.03 sys
* Compiling lib/facil/tls/fio_tls_missing.c
        0.04 real         0.03 user         0.00 sys
* Compiling lib/facil/tls/fio_tls_openssl.c
        0.03 real         0.01 user         0.01 sys
* Compiling lib/facil/fiobj/fio_siphash.c
        0.03 real         0.02 user         0.00 sys
* Compiling lib/facil/fiobj/fiobj_ary.c
        0.06 real         0.05 user         0.00 sys
* Compiling lib/facil/fiobj/fiobj_data.c
        0.09 real         0.07 user         0.01 sys
* Compiling lib/facil/fiobj/fiobj_hash.c
        0.08 real         0.07 user         0.00 sys
* Compiling lib/facil/fiobj/fiobj_json.c
        0.13 real         0.11 user         0.01 sys
* Compiling lib/facil/fiobj/fiobj_mustache.c
        0.16 real         0.14 user         0.01 sys
* Compiling lib/facil/fiobj/fiobj_numbers.c
        0.04 real         0.02 user         0.00 sys
* Compiling lib/facil/fiobj/fiobj_str.c
        0.09 real         0.07 user         0.00 sys
* Compiling lib/facil/fiobj/fiobject.c
        0.08 real         0.07 user         0.00 sys
* Compiling lib/facil/cli/fio_cli.c
        0.08 real         0.07 user         0.00 sys
* Compiling lib/facil/http/http.c
        0.30 real         0.28 user         0.01 sys
* Compiling lib/facil/http/http1.c
        0.18 real         0.16 user         0.01 sys
* Compiling lib/facil/http/http_internal.c
        0.15 real         0.13 user         0.01 sys
* Compiling lib/facil/http/websockets.c
lib/facil/http/websockets.c:99:27: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes]
static ws_s *new_websocket();
                          ^
                           void
lib/facil/http/websockets.c:305:14: warning: a function declaration without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
static ws_s *new_websocket(intptr_t uuid) {
             ^
lib/facil/http/websockets.c:99:14: note: a function declaration without a prototype is not supported in C2x
static ws_s *new_websocket();
             ^
2 warnings generated.
        0.12 real         0.10 user         0.01 sys
* Compiling lib/facil/redis/redis_engine.c
        0.15 real         0.13 user         0.01 sys
* Linking...
Undefined symbols for architecture arm64:
  "_main", referenced from:
     implicit entry/start for main executable
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
        0.05 real         0.03 user         0.01 sys
make: *** [build] Error 1

clang -v
Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1)
Target: arm64-apple-darwin22.4.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Hi @AbhimanyuAryan ,
Please note that the facil.io library doesn't have a main function... the main function is the program itself, which is up to you to write.
There is an example app (a possibly good starting point) at https://github.com/boazsegev/facil.io/tree/master/examples/boiler_plate, it's the boiler plate for the facil.io framework new/app script.
I hope this helps.
Good Luck!

sorry @boazsegev I am not very familiar with making files. I tried this and still fails
MAIN_ROOT=examples/boiler_plate/

Hi, I used MAIN_ROOT=examples/boiler_plate/src/ in https://github.com/boazsegev/facil.io/blob/7b8e5fcf5e945d07df92aca7728909c5887a80fc/makefile#L49 and still get this error
make
* Detected `kqueue`
* Detected `sendfile` (Apple)
* Detected 'tm_zone' field in 'struct tm'
* Detected native socket API, without additional libraries
* No compatible SSL/TLS library detected.
* Detected Little Endian byte order.
* Compiling examples/boiler_plate/src//cli.c
examples/boiler_plate/src//cli.c:76:5: warning: 'http_url_s' is deprecated [-Wdeprecated-declarations]
    http_url_s info =
    ^
lib/facil/http/http.h:958:20: note: 'http_url_s' has been explicitly marked deprecated here
    __attribute__((deprecated("use fio_url_s instead")));
                   ^
lib/facil/fio.h:234:28: note: expanded from macro 'deprecated'
#define deprecated(reason) deprecated
                           ^
1 warning generated.
        0.09 real         0.03 user         0.04 sys
* Compiling examples/boiler_plate/src//http_service.c
        0.05 real         0.03 user         0.01 sys
* Compiling examples/boiler_plate/src//main.c
        0.05 real         0.02 user         0.01 sys
* Linking...
Undefined symbols for architecture arm64:
  "_main", referenced from:
     implicit entry/start for main executable
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
        0.04 real         0.03 user         0.01 sys
make: *** [build] Error 1


FunctionID: Ability to change limits

For now a user cannot change fidb limits. Related: https://github.com/NationalSecurityAgency/ghidra/issues/1205

In addition, it would be helpful/handy if byte sequences could be added/remove from functions being matched at specific location distances from both ends of said function so that the hash could pick up simple variations.
For example, I have a library DLL from which I've created an fidb, and I've noticed the code I'm analysing has practically the same code, byte-for-byte, apart from an unrequired/unremoved PUSH DS/POP DS prologue/epilogue sequence.

In addition, it would be helpful/handy if byte sequences could be added/remove from functions being matched at specific location distances from both ends of said function so that the hash could pick up simple variations.
For example, I have a library DLL from which I've created an fidb, and I've noticed the code I'm analysing has practically the same code, byte-for-byte, apart from an unrequired/unremoved PUSH DS/POP DS prologue/epilogue sequence.


[1.0 Cherry Pick] Add Chef Color Temperature Light (#22956)

Fixes #23149

Add Chef Color Temperature Light

Add new chef device : colortemperaturelight

Restyled by whitespace

Restyled by clang-format

Remove Chef ColorTemperatureLight Clusters.matter

Remove unused Clusters.matter
zzz_generated/chef-rootnode_colortemperaturelight_hbUnzYVeyn/zap-generated/Clusters.matter

Remove Scene Cluster which is not supported in 1.0

Restyled by whitespace

Restyled by clang-format

Remove unnecessary clusters fro chef colortemperature device

The current chef Color Temperature Light device contains several unnecessary clusters such as Switch, LocalizationConfiguration, , TimeFormatLocalization.
Putting a Switch cluster in the EP0 (root node) is wrong. While we don't need to localize text/time for a light. So we remove them all.

Fix Chef color temperature light zzz_generated

After removing Switch/LocalizationConfiguration/TimeFormat clusters, the codes in zzz_generated are also regenerated and should be committed

Fix chef colortemperaturelight matter new format

Fix examples/chef/devices/rootnode_colortemperaturelight_hbUnzYVeyn.matter which defines new matter format "fabric_scoped"

Remove FixedLabel & UserLabel which doesn't apply here

Restyled by whitespace

Restyled by clang-format

Co-authored-by: Restyled.io<EMAIL_ADDRESS>

@andy31415 Sorry I've closed this PR due to wrong commit and replaced it by new PR 23191. Please help to review and approve again. Thanks