Added opEquals to InternetAddress

I'm not sure why the Address classes do not have opEquals implemented.  I've added it to the InternetAddress class.  If this get's "approved" then I can add the other opEquals methods to the other Address classes.

Auto-merge toggled on

Thanks, looks good.


Include font size property for text tool #444

Feature
Include font size on text element solves issue #444

Coverage decreased (-0.7%) to 82.567% when pulling b544c7e386c964d99f92a27644ba1a742136401b on M06631:change-font-size-of-text-element#444 into 0617f123c2653b67c4697400724ad0b49cc2b144 on CircuitVerse:master.

@satu0king
yes it persist the size to database
also I refactored it using mutableProperties (BTW it was a great extending feature)

@fluctix Sure

LGTM👍
On Sat, Jan 11, 2020, 9:09 PM M06631<EMAIL_ADDRESS>wrote:

@fluctix https://github.com/fluctix Sure
[image: Screenshot]
https://camo.githubusercontent.com/d73e971920be31b8a0e62f6d9acd5abf13358527/68747470733a2f2f692e696d6775722e636f6d2f654d436251314e2e676966
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CircuitVerse/CircuitVerse/pull/772?email_source=notifications&email_token=AJVJDCRQTZYORVA4KFGPXMLQ5HR33A5CNFSM4J6XVH62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIWEPTI#issuecomment-573327309,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJVJDCUASJ26R67JHN4UEBLQ5HR33ANCNFSM4J6XVH6Q
.

Good Job!


Better example

Need to include an example(s) in the master branch demonstrating things like creating a new player, accessing the _internalPlayer, etc

79c096693de3a4229cbc3fe9a0f146263572a322


Add Support for Carbon Dioxide Cluster 0x040d (TZ-262)

I'm working on first and easy project Air Sensor
Fully working example placed on github. https://github.com/lmahmutov/esp32_c6_co2_sensor
but for complete i need support for Carbon Dioxide Cluster. At this time Co2 reported to custom cluster.

I try to add custom cluster with cluster ID 0x040d, but firmware says "Cluster ID not in avaliable range"

Whem the Carbon Dioxide Cluster 0x040d will be added?

and we :)

it can be realised using native ZBOSS SDK, without ESP-ZigBee-Sdk. But binding and configureation report for cluster not working. Only manual reporting.
Fully working native CO2 cluster example.
https://github.com/lmahmutov/esp32_c6_co2_sensor/blob/esp32_c6_co2_ZBOSS-WIP/main/esp_co2_sensor.c#L43

Hello @lmahmutov @u236 @vke @RadekDymak ,
The Carbon Dioxide cluster has been supported in esp-zigbee-sdk v1.0.0. Could you please try using and testing it.


feat: #657 - important attributes first, and then ordered and filtered attributes

Impacted file:

summary_card.dart: new methods _getMandatoryAttributeIds and _getOrderedAndFilteredAttributes

Just to ensure we are on the same page, is your plan to:
--- Render Nutri card
--- Render Eco card
--- Render mandatory attributes (green/yellow/red dot and attribute label)
--- Render attribute groups (with sorted attributes inside)

Just to ensure we are on the same page, is your plan to:
--- Render Nutri card --- Render Eco card --- Render mandatory attributes (green/yellow/red dot and attribute label) --- Render attribute groups (with sorted attributes inside)

To be honest, it isn't confirmed whether we will display the mandatory attributes before displaying the groups, but I think we can go for it.
We can sort of treat mandatory attributes as a new group, "mandatory attributes group"

@jasmeet0817 If I sum up:

render nutriscore the way it was before
then render ecoscore the way it was before
then display a virtual "mandatory group" that includes only the mandatory attributes (except nutriscore and ecoscore) <= that's new
then display the groups in the predefined order the way it was before...
... except that we don't display there the mandatory attributes (already in the "mandatory group"), the not important attributes (no change here), and within a group we display the very important attributes first, then the important attributes <= that's new

@jasmeet0817 If I sum up:

render nutriscore the way it was before
then render ecoscore the way it was before
then display a virtual "mandatory group" that includes only the mandatory attributes (except nutriscore and ecoscore) <= that's new
then display the groups in the predefined order the way it was before...
... except that we don't display there the mandatory attributes (already in the "mandatory group"), the not important attributes (no change here), and within a group we display the very important attributes first, then the important attributes <= that's new

yes, that sounds perfect.

Done. Just a remark: when you set an allergen as mandatory, it will be displayed in the virtual mandatory group.
The other important allergens will be displayed in the allergen group, with its explicit allergen label.
The problem is that therefore mandatory allergens are not displayed with that explicit allergen label (because they are in the virtual mandatory group, that has no label at all).

Done. Just a remark: when you set an allergen as mandatory, it will be displayed in the virtual mandatory group. The other important allergens will be displayed in the allergen group, with its explicit allergen label.
The problem is that therefore mandatory allergens are not displayed with that explicit allergen label (because they are in the virtual mandatory group, that has no label at all).

Definitely share your concern regarding Mandatory group missing a label, I'll talk to the UX designer to see how we can fix that. Thanks for the change, looks good


[improve][broker] log sessionId with zk-session-watcher for debugging

Motivation
Right now, zk-quorum recovers after leader restart within zk-session-timeout but still it can cause restart for some brokers even if it's not expected. It's really helpful if zk-session-watcher also prints session-id if it has been changed while disconnecting so, it can help admin to configure right synclimit at zk-server configuration or doing additional debugging.
Modifications
Printing sessionId when zk is disconnecting.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Codecov Report

Merging #18291 (89fb565) into master (0866c3a) will increase coverage by 31.81%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #18291       +/-   ##
=============================================
+ Coverage     38.97%   70.79%   +31.81%
+ Complexity     8311      438     -7873
=============================================
  Files           683       26      -657
  Lines         67325     2246    -65079
  Branches       7217      245     -6972
=============================================
- Hits          26239     1590    -24649
+ Misses        38079      483    -37596
+ Partials       3007      173     -2834

Flag
Coverage Δ

unittests
70.79% <ø> (+31.81%)
:arrow_up:

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

Impacted Files
Coverage Δ

...apache/pulsar/proxy/server/LookupProxyHandler.java
57.75% <0.00%> (-0.44%)
:arrow_down:

.../main/java/org/apache/pulsar/PulsarStandalone.java

...n/java/org/apache/pulsar/broker/PulsarService.java

.../pulsar/broker/service/AbstractBaseDispatcher.java

...rg/apache/pulsar/broker/service/BrokerService.java

...ava/org/apache/pulsar/broker/service/Consumer.java

...pulsar/broker/service/PulsarCommandSenderImpl.java

...apache/pulsar/broker/service/TopicListService.java

...sar/broker/service/persistent/PersistentTopic.java

...ransaction/buffer/impl/InMemTransactionBuffer.java

... and 648 more


TLS cert auth with TLS 1.3

Has anyone experience with tlscert auth in combination with TLS 1.3 enabled servers.
It would seem that this is technically not possible (atleast with TLS1.3 and HTTP/2+) and could break the authentication completely.
Only Mozilla addressed this to some extent, for Java clients it may not be easily available.
https://bugs.openjdk.java.net/browse/JDK-8206923
https://bugs.chromium.org/p/chromium/issues/detail?id=911653
https://bugzilla.mozilla.org/show_bug.cgi?id=1511989

I no longer work on EWP, but got the notification and got interested. Just read through the links, and it seems that the problems occur only for post-handshake authentication. I think we don't really need post-handshake auth in case of EWP (the client could present its credentials on the first connection)?

Sorry for the late response.
Yes only post-handshake auth is affected by this.
I am not sure if it is possible to host both auth methods (tls cert and http header signatures) on the same endpoint without post-handshake auth (At least as far as I know with Apache/Nginx.), because in case of the header signatures the client has/needs no certificate to present. Also it would make the separation between APIs which require authentication and those which don't a little bit more complicated.
One possible solution could be to have different endpoints for the different auth methods, but I think this could easily break existing implementations.

As far as I understand it, the actual problem only occurs when a single client wants to use multiple client auth methods:

Perhaps we can ask if any clients actually do that (use more than one auth method per connection).
It seems plausible that the answer is NO. That is, every client already picks "the strongest needed" auth method, and sticks to it.
Otherwise, perhaps, adding an actual formal requirement would be good idea ("If you intend to use TLS auth during your connection, then you MUST do it right from the start").

One possible solution could be to have different endpoints for the different auth methods, but I think this could easily break existing implementations.

Interesting idea, that might work too! If clients query the registry after selecting the auth method (as they should), then it shouldn't break anything.


Convert kubernetes/serve_hostname to gcr.io/google_containers/server_hos...

...tname

Running e2e tests... please do not merge until I verify that everything is OK.
@jlowdermilk @zmerlynn @thockin

LGTM, will wait for your go-ahead to merge :)

I've been trying and failing to run e2e tests. I will give up tonight and try again tomorrow.

I think this is eclipsed by #6268 which is now merged.


Issue with nameSelector, labelSelector for Deployment as trigger resource

Expected behaviour (what you expected to happen):
Trying to execute scenario in which when Deployment with particular label Added , kubernetes job will be created.
Actual behaviour (what actually happened):
Shell-operator skips  nameSelector and/or labelSelector and created Kubernetes job immediately after operator/pod creation.
Steps to reproduce:
Run shell operator/pod yaml
`apiVersion: v1
kind: Pod
metadata:
name: shell-operator
spec:
containers:

name: shell-operator
image: lubowsky/shell-operator:monitor-deployment
imagePullPolicy: Always
serviceAccountName: monitor-pods-acc`

ClusterRole:
kubectl create clusterrole monitor-pods --verb=get,watch,list,create --resource=deployments,jobs
Environment:

Shell-operator version: latest (v1.0.5)
Kubernetes version: v1.21.5-gke.1302
Installation type (kubectl apply, helm chart, etc.): kubectl

Anything else we should know?:
Additional information for debugging (if necessary):
Hook script
#!/usr/bin/env bash
if [[ $1 == "--config" ]] ; then
cat <<EOF
configVersion: v1
kubernetes:

name: execute_on_deployment_with_labels
kind: Deployment
executeHookOnEvent: ["Added"]
nameSelector:
matchNames:

deploy01
labelSelector:
matchLabels:
myLabel: "myLabelValue"
EOF
else
kubectl create -n default -f - <<EOF
apiVersion: batch/v1
kind: Job
metadata:
name: pi
spec:
template:
spec:
containers:

name: pi
image: perl
command: ["perl",  "-Mbignum=bpi", "-wle", "print bpi(10)"]
restartPolicy: Never
backoffLimit: 4
EOF
fi

Logs
{"level":"info","msg":"Create metric counter shell_operator_task_wait_in_queue_seconds_total","operator.component":"metricStorage","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"Hooks dir: /hooks","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"Use temporary dir: /tmp/shell-operator","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"Debug endpoint listen on /var/run/shell-operator/debug.socket","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"Create metric histogram shell_operator_kubernetes_client_request_latency_seconds","operator.component":"metricsStorage","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"Create metric counter shell_operator_kubernetes_client_request_result_total","operator.component":"metricStorage","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"Kubernetes client is configured successfully with 'out-of-cluster' config","operator.component":"KubernetesAPIClient","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"Kubernetes client is configured successfully with 'out-of-cluster' config","operator.component":"KubernetesAPIClient","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"Initialize hooks manager. Search for and load all hooks.","time":"2021-12-06T12:20:01Z"}
{"hook":"deployment-hook1.sh","level":"info","msg":"Load config from '/hooks/deployment-hook1.sh'","phase":"config","time":"2021-12-06T12:20:01Z"}
{"hook":"deployment-hook1.sh","level":"info","msg":"Loaded config: Watch k8s kinds: 'Deployment'","phase":"config","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"start shell-operator","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"queue task EnableKubernetesBindings:::deployment-hook1.sh:EnableKubernetesBindings for hook deployment-hook1.sh","operator.component":"initMainQueue","time":"2021-12-06T12:20:01Z"}
{"binding":"","hook":"deployment-hook1.sh","level":"info","msg":"Enable kubernetes binding for hook","queue":"main","task":"EnableKubernetesBindings","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"Create metric histogram shell_operator_kubernetes_client_rate_limiter_latency_seconds","operator.component":"metricsStorage","time":"2021-12-06T12:20:01Z"}
{"level":"info","msg":"I1206 12:20:02.384324       7 request.go:645] Throttling request took 1.168987581s, request: GET:https://<IP_ADDRESS>:443/apis/batch/v1?timeout=32s\n","source":"klog","time":"2021-12-06T12:20:02Z"}
{"binding":"","hook":"deployment-hook1.sh","level":"info","msg":"Kubernetes bindings for hook are enabled successfully, 1 tasks generated","queue":"main","task":"EnableKubernetesBindings","time":"2021-12-06T12:20:09Z"}
{"binding":"execute_on_deployment_with_labels","event":"kubernetes","hook":"deployment-hook1.sh","level":"info","msg":"Execute hook","queue":"main","task":"HookRun","time":"2021-12-06T12:20:09Z"}
{"binding":"execute_on_deployment_with_labels","event":"kubernetes","hook":"deployment-hook1.sh","level":"info","msg":"job.batch/pi created","output":"stdout","queue":"main","task":"HookRun","time":"2021-12-06T12:20:10Z"}
{"binding":"execute_on_deployment_with_labels","event":"kubernetes","hook":"deployment-hook1.sh","level":"info","msg":"Usage: \u0026{Sys:137.373ms User:338.645ms MaxRss:65484}","queue":"main","task":"HookRun","time":"2021-12-06T12:20:10Z"}
{"level":"info","msg":"Create metric histogram shell_operator_hook_run_sys_seconds","operator.component":"metricsStorage","time":"2021-12-06T12:20:10Z"}
{"level":"info","msg":"Create metric histogram shell_operator_hook_run_user_seconds","operator.component":"metricsStorage","time":"2021-12-06T12:20:10Z"}
{"binding":"execute_on_deployment_with_labels","event":"kubernetes","hook":"deployment-hook1.sh","level":"info","msg":"Hook executed successfully","queue":"main","task":"HookRun","time":"2021-12-06T12:20:10Z"}
{"binding":"execute_on_deployment_with_labels","event":"kubernetes","hook":"deployment-hook1.sh","level":"info","msg":"Unlock kubernetes.Event tasks","queue":"main","task":"HookRun","time":"2021-12-06T12:20:10Z"}

Thanks a lot @diafour , that was not obvious from the first look :)


Cleaned the FAQ code and fixing minor changes.

Related Issue

Information about the related issue
Checking if imports work in deployment.
Closes: #

Describe the changes you've made

Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update.
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist

[ ] My code follows the guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)
Code of Conduct

[x] I agree to follow this project's Code of Conduct

is it better now? @balaji-sivasakthi

is it better no

Refer to the link.


(iOS) framework not found DoubleConversion after Pod Install

I cannot get the project to build after a clean react native project init and following the instructions to install react-native-spokestack with Cocoa pods
Steps to reproduce

react-native init testspokestack --version<EMAIL_ADDRESS>npm install react-native-spokestack --save
Instructions for pod install
Removed jschelpers from Podfile and also removed all of the lib* files from Link Binary With Libraries for my projects target

ld: warning: directory not found for option '-F/Users/Library/Developer/Xcode/DerivedData/testspokestack-bcjyjwnjqjiziycpoearfiucjlzj/Build/Products/Debug-iphoneos/DoubleConversion'
ld: warning: directory not found for option '-F/Users//Library/Developer/Xcode/DerivedData/testspokestack-bcjyjwnjqjiziycpoearfiucjlzj/Build/Products/Debug-iphoneos/Folly'
ld: warning: directory not found for option '-F/Users/Library/Developer/Xcode/DerivedData/testspokestack-bcjyjwnjqjiziycpoearfiucjlzj/Build/Products/Debug-iphoneos/RNSpokestack'
ld: warning: directory not found for option '-F/Users/Library/Developer/Xcode/DerivedData/testspokestack-bcjyjwnjqjiziycpoearfiucjlzj/Build/Products/Debug-iphoneos/React'
ld: warning: directory not found for option '-F/Users/Library/Developer/Xcode/DerivedData/testspokestack-bcjyjwnjqjiziycpoearfiucjlzj/Build/Products/Debug-iphoneos/SpokeStack'
ld: warning: directory not found for option '-F/Users/Library/Developer/Xcode/DerivedData/testspokestack-bcjyjwnjqjiziycpoearfiucjlzj/Build/Products/Debug-iphoneos/glog'
ld: warning: directory not found for option '-F/Users/Library/Developer/Xcode/DerivedData/testspokestack-bcjyjwnjqjiziycpoearfiucjlzj/Build/Products/Debug-iphoneos/yoga'
ld: framework not found DoubleConversion
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Hi Brien, sorry that you ran into trouble building! You've run into a know React Native issue (https://github.com/facebook/react-native/issues/21168). There's a manual fix for this that you can apply: https://gist.github.com/sobstel/99587e397f994d6eb319e8cafd423a57. Hope this helps!

Closing pending further feedback.


How to enable debug mode.

I am hitting the issue lightweight-charts Uncaught Error: Value is null  running version "lightweight-charts": "^3.8.0", installed via

After reading through a heap of the other issues I am pretty sure it is due to one of the following reasons:

some of the data is null
potential duplication on data in timestamp
time should be ordered by time in ASC order

The last one I am doing like so:
const sortedAsc = data3.sort(
              (objA, objB) => Number(objA.time) - Number(objB.time),
            );

Now I have also read that I can get better error messages by running in DEBUG mode, but I can't for the life of me work out how to enable DEBUG mode. I couldn't find any search results for DBEUG in the docs.
I have [installed via](https://tradingview.github.io/lightweight-charts/docs0 npm install --save lightweight-charts Is there a flag I can set in my code, or a different package I install?

Firstly, I think your error is related to the time values within your data. They appear to be too large. The expected value should be seconds since epoch instead of milliseconds, thus you should divide that number by a 1000. (https://tradingview.github.io/lightweight-charts/docs/api#utctimestamp)
As you mentioned, data points should be passed in ascending order (newest values at the end), and there shouldn't be any data points which share a time value (no duplicate times).
You can set the NODE_ENV environment variable to anything besides production and it should load the development version. If you are using a module version of node then that may not work, but you could always go into node_modules/lightweight-charts/package.json and temporarily change the "module" value.
If you are using the standalone version then use lightweight-charts.standalone.development.js instead of lightweight-charts.standalone.production.js.
One of the benefits of using the development version is that the console messages will make a bit more sense since the call stack names won't be minified anymore.

Thanks @SlicedSilver for getting back to me so fast... the below worked a treat for me. I should have searched for development 🤦

but you could always go into node_modules/lightweight-charts/package.json and temporarily change the "module" value.

Also thanks for the heads up on the seconds since epoch instead of milliseconds.


Fixed #21237 - Toggle buttons don't honor [disabled] or .disabled

This commit adds a check for disabled attributes and classes on the button element.

@Johann-S, tests added


Added ability to retrieve rating and review information about an application

This allows for commands like the following:
ratings = app.ratings
[8] pry(main)> ratings.rating_summary.average_rating
=> 3.14
[11] pry(main)> ratings.rating_summary.five_star_rating_count
=> 46
[12] pry(main)> ratings.store_fronts
=> {"NZ"=><Spaceship::Tunes::AppRatingSummary
>, "AU"=><Spaceship::Tunes::AppRatingSummary
>, "GB"=><Spaceship::Tunes::AppRatingSummary
>, "US"=><Spaceship::Tunes::AppRatingSummary
>}
[13] pry(main)> ratings.store_fronts["US"].average_rating
=> 3.06
[14] pry(main)> ratings.reviews("US")
=> [{"id"=>xxxxx, "rating"=>1, "title"=>"Review", "review"=>"Review text", "created"=>1422638100000, "nickname"=>"xxxx", "storeFront"=>"US"}]
[15] pry(main)> ratings.versions
=> {"1234"=>"1.0.3",
 "2345"=>"2.0.11",
 "3456"=>"2.1.0"}
[16] pry(main)> ratings.reviews("NZ", "3456")
=> [{"id"=>xxxx,
  "rating"=>2,
  "title"=>"Title",
  "review"=>
   "Review",
  "created"=>1463887020000,
  "nickname"=>"xxx",
  "storeFront"=>"NZ"}]

I'm not super familiar with ruby so open to any suggestions for how to make this better.

@revile Thanks for the PR! Before we get into reviewing and trying this out, I'd love to see some specs for this feature. Here's a tunes example to look at: https://github.com/fastlane/fastlane/blob/master/spaceship/spec/tunes/app_version_spec.rb

This looks like a great addition to spaceship, thanks for this @revile.
As @asfalcone pointed out, we'll need some tests covering the parsing, and the calculations, check out the other tests to see how you can add the tests. Also check out tunes_stubbing.rb on how to fake web requests.
Additionally to adding tests, this also needs some documentation, please add the docs to iTunesConnect.md in the spaceship directory of this repo.
Please let us know if you need any help doing so 👍

Giving this another thought, the instance variable is fine. Could you add some tests though? 👍

Added tests & updated docs - let me know what you think 👍

This is great, thank you so much for your contribution @revile, this will be in the next release 👍


[circle] downgrade loses all but first workflow

Here is a circle pipeline
version: 2.1
jobs:
  build:
    docker:
      - image: cimg/base:2023.03
    steps:
      - checkout
      - run:
          name: build
          command: echo "this is the build job"
  test:
    docker:
      - image: cimg/base:2023.03
    steps:
      - checkout
      - run:
          name: test
          command: echo "this is the test job"
workflows:
  build:
    jobs:
      - build
  test:
    jobs:
      - test

When converting to downgraded yaml, this is the result
pipeline:
  identifier: default
  name: default
  orgIdentifier: default
  projectIdentifier: default
  properties:
    ci:
      codebase:
        build: <+input>
  stages:
  - stage:
      identifier: build1
      name: build
      spec:
        cloneCodebase: true
        execution:
          steps:
          - step:
              identifier: build
              name: build
              spec:
                command: echo "this is the build job"
                image: cimg/base:2023.03
              timeout: ""
              type: Run
        platform:
          arch: Amd64
          os: Linux
        runtime:
          spec: {}
          type: Cloud
      type: CI

Only the first workflow comes through.
Without downgrading, this is the result
stages:
- name: test
  spec:
    platform:
      arch: amd64
      os: linux
    runtime:
      spec: {}
      type: cloud
    steps:
    - name: test
      spec:
        image: cimg/base:2023.03
        run: echo "this is the test job"
      type: script
  type: ci
version: 1

---
stages:
- name: build
  spec:
    platform:
      arch: amd64
      os: linux
    runtime:
      spec: {}
      type: cloud
    steps:
    - name: build
      spec:
        image: cimg/base:2023.03
        run: echo "this is the build job"
      type: script
  type: ci
version: 1

Each workflow is a separate pipeline delimited by ---. Downgrading should have the same behavior, multiple pipelines should be generated with a delimiter.
pipeline:
  identifier: default
  name: default
  orgIdentifier: default
  projectIdentifier: default
  properties:
    ci:
      codebase:
        build: <+input>
  stages:
  - stage:
      identifier: test1
      name: test
      spec:
        cloneCodebase: true
        execution:
          steps:
          - step:
              identifier: test
              name: test
              spec:
                command: echo "this is the test job"
                image: cimg/base:2023.03
              timeout: ""
              type: Run
        platform:
          arch: Amd64
          os: Linux
        runtime:
          spec: {}
          type: Cloud
      type: CI

---
pipeline:
  identifier: default
  name: default
  orgIdentifier: default
  projectIdentifier: default
  properties:
    ci:
      codebase:
        build: <+input>
  stages:
  - stage:
      identifier: build1
      name: build
      spec:
        cloneCodebase: true
        execution:
          steps:
          - step:
              identifier: build
              name: build
              spec:
                command: echo "this is the build job"
                image: cimg/base:2023.03
              timeout: ""
              type: Run
        platform:
          arch: Amd64
          os: Linux
        runtime:
          spec: {}
          type: Cloud
      type: CI

Fix in https://github.com/drone/go-convert/pull/129


Review

Please review, @jakobsvolba

Description of repository
Please add a three-paragraph description
Tests

[ ] License exists
[ ] Create a fork and start a pull-request
[ ] Pytests fails as expected
[ ] Ruff linter fails as expected

@danielhuppmann: Homework Done


Preliminary rewrite for indexing gale with local ocr content

related to #673
Sharing for feedback on general approach to revise how Gale page contents are indexed with local OCR.

Does this approach seem reasonable?
Does it make sense to add the ocr to json script to corppa?

I'm using a single json file per volume, with page contents accessed based on page number. I'd initially thought to try jsonl but wasn't sure how to handle missing/nonsequential pages. I haven't tested yet with any very large volumes.
I wrote a quick script for testing with a single volume, haven't generalized anything yet, but here's what it looks like:
import glob
import os
import json

# Get all .txt files in the specified directory
txt_files = glob.glob("data/tigerdata_subset/ppa-ocr/Gale/145/CB0127495592/*.txt")

def page_num(filename):
    name, _ = os.path.splitext(os.path.basename(filename))
    # with zeroes, as string
    pagenum = name.split("_")[-1]
    return pagenum[:-1]  # strip trailing zero

txt_data = {}
for filename in txt_files:
    with open(filename, "r") as txtfile:
        txt_data[page_num(filename)] = txtfile.read()

with open("CB0127495592.json", "w") as outfile:
    json.dump(txt_data, outfile)

Overall, this seems like a reasonable approach to me.

Consider using pathlib instead of glob and os.path
I'm not sure I understand the difference between .json vs .jsonl. In both cases you may need to worry about missing pages.
Since the ocr code is all in ppa-nlp, it seems reasonable to also include the ocr-to-json script. In the future this script might also do additional text transformations.

@laurejt thanks for the quick review and useful feedback, I'll keep your recommendations in mind when I circle back to this.
jsonl vs json: I was thinking json lines so we could avoid having the whole file in memory, but that would mean we have to sort the pages before we generate the lines file (or when we load it), and then as we iterate we'd have to check if we were on the correct page - so logic is more complicated. I think these volumes are small enough that we can get by with one file per volume and just have the whole text in memory so we can look up contents by page number.


feat: nix uninstaller for ublue-os systems

It would be quite interesting to have an uninstaller for nix, considering that the ublue-nix-installer does some things that are not specified in nix's official documentation

I've tested it in my personal computer + personal branch and it seems to be working fine, although testing it on a virtual machine shouldn't be super hard


Aqua Exception Design

Description

All error raised by AquaApp will be divided into two types: AquaServiceError and AquaClientError.
Plz checkout oci_excpetion_handler decorator to see how to convert OCI Exception into AquaError
Set logger in AquaApp as attribute to be used by all app.

I made some changes on the exception_handler decorator. Here are my suggestions:

I think we can use a single decorator to handle all exceptions including OCI exceptions.
Whenever there is an error. The API will return either 4XX (client error) or 5XX (server error) code.
The error payload will contain:
** message, this will be a text describing the error code. This is filled automatically by Jupyter
** reason, this is a user friendly message describing the actual error
** service_payload, this will be the payload returned by the service, if any. Or some other payload for debugging purpose.
** traceback, optionally we can return traceback.
The error code will be consistent with whatever error code returned by OCI.
I put my example code here: https://github.com/oracle/accelerated-data-science/tree/qq/aqua_exception
See the aqua.model.AquaModelApp.get() for example on different errors.

Once all the review comments are incorporated, @mingkang111 , could you please let me know? I would like to look at the final code before it is merged. Thanks for bringing this all together!

I played with the exception_handler decorator. Here are my suggestions:

I think we can use a single decorator to handle all exceptions including OCI exceptions.
Whenever there is an error. The API will return either 4XX (client error) or 5XX (server error) code.
The error payload will contain:
** message, this will be a text describing the error code. This is filled automatically by Jupyter
** reason, this is a user friendly message describing the actual error
** service_payload, this will be the payload returned by the service, if any. Or some other payload for debugging purpose.
** traceback, optionally we can return traceback.
The error code will be consistent with whatever error code returned by OCI.

I put my example code here: https://github.com/oracle/accelerated-data-science/tree/qq/aqua_exception See the aqua.model.AquaModelApp.get() for example on different errors.

Thanks QQ! I updated the design based on your suggestions.

Once all the review comments are incorporated, @mingkang111 , could you please let me know? I would like to look at the final code before it is merged. Thanks for bringing this all together!

Hi @mayoor , the comments above are all addressed. Could you please take a look when you have time? Please checkout the confluence page shared in team channel for more sample response. Thank you!

Thanks @mingkang111. Let us merge the PR.

Thanks @mingkang111. Let us merge the PR.

We need to drop the traceback from the payload and replace it with requestId. For now let the requestid be random number. We will finalize the details in a couple of days

Thanks @mingkang111. Let us merge the PR.

We need to drop the traceback from the payload and replace it with requestId. For now let the requestid be random number. We will finalize the details in a couple of days

Replaced the traceback field with request_id.


Add metrics related to watch stream

Description:
Currently, Server collects metrics using Prometheus, aggregating changes, operations, and snapshot storage during synchronization(PushPull API). To enhance real-time collaboration features, we propose the addition of the following metrics:
A. Count of Connections

Purpose: To understand the real-time usage of the service.
Benefit: This would allow for accurate measurement of real-time collaboration feature usage and better prediction of system load.

B. Event Exchange Size

Purpose: To monitor the quantity and size of events exchanged through the stream.
Benefit: This would help optimize network bandwidth usage and improve performance in large collaboration sessions.

Also, if watch stream-related metrics are aggregated by project, it is useful to show graphs on the dashboard later.
Related to https://github.com/yorkie-team/dashboard/issues/162
Why:
With these enhancements, we can better monitor the usage and performance of real-time collaboration features, leading to improved service quality and user experience.

@emplam27 Would you like to try to tackle this issue?


dev: "" "00:00:00:00:00:00" (:0)

looks like implementation of permission acquiring mechanism is incorrect
S22ultra, oneui 6.1, android 14 renders next logs
QObject::connect: Cannot queue arguments of type 'QItemSelection'
(Make sure 'QItemSelection' is registered using qRegisterMetaType().) (:0, )
QBasicTimer::start: Timers cannot be started from another thread (:0, )
QObject::startTimer: Timers can only be used with threads started with QThread (:0, )
QObject::killTimer: Timers cannot be stopped from another thread (:0, )
QBasicTimer::start: Timers cannot be started from another thread (:0, )
dev: "" "00:00:00:00:00:00" (:0, )
Falling back to reverse uuid workaround. (:0, )

Please try the latest release to see if the bug is fixed.

at last, i works!
may i use your work to add headphone support in gadgetbridge?
also I'm unable to set name, looks like packet is ignored as i see hex sent in logs:
send: "aa11ca666f787869652d6d7573696370617773283a" (:0, )
thank you for amazing work <3

may i use your work to add headphone support in gadgetbridge?

Yes, you may. It would be better if you can mention the source of the raw commands as the reference for others.
https://github.com/wh201906/mEDIFIER/blob/main/doc/KnownCommands/W820NB_Double_Gold.txt

looks like packet is ignored.

Are you using W820NB Double Gold? I can set the name of it with mEDIFIER.

Are you using W820NB Double Gold? I can set the name of it with mEDIFIER.
I'm owner of w820nb plus (mint color, but looks like chip is same, tried to reverse engineer it on my own)

I don't know why it doesn't work...

if you tell me what i could do maybe i could help you somehow? i have ability to monitor and create bluetooth packets and fair bit of hardware equipment (cmsis-dap)

Would you mind installing the official Edifier Connect app? You can check the raw Bluetooth packets of it in the Logcat. When setting a name to your device, the corresponding packets will be shown in the log.


[Bug] exercise "en/c/variables/1.md" has an issue

My answer and the solution are the same but the app says that’s wrong and I can’t go forward

Can you update the app to the latest version and try again?
I'm not experiencing any error with the given exercise.

I did update but it gives me the same error


evhtp_parse_query_wflags SIGSEGV: address access protected

Details
I'm experiencing an out-of-bounds write attempt during evhtp_parse_query_wflags. I believe this issue is probably a memory allocation limit but I'm not exactly positive.
Steps or code to reproduce the problem.
Not sure if it is specific data that is currently on my server, more testing to reproduce need to be done. Installing the port seahub on FreeBSD and generate some syncs with seafile-client. In the logs there will be "Seafile kernel: pid 91446 (seaf-server), uid 900: exited on signal 11".
Further debugging I found that this will occur when len is at a higher value like len=1048660

thread #6, name = 'seaf-server', stop reason = signal SIGSEGV: address access protected (fault address:
0x7fffdf5f98e0)
frame #0: 0x0000000800dac8da libevhtp.so.0`evhtp_parse_query_wflags(query="846adf988582db791bbedce08b99706b5e94c917", len=1048660, flags=0) at evhtp.c:3531

Version
FreeBSD: 11.2
Seafile-server: 6.3.4
Libevhtp: 1.2.16
Let me know if there is anything else that may help with debugging this.

* thread #13, name = 'seaf-server', stop reason = signal SIGSEGV: address access protected (fault address: 0x7fffde5f1690)
  * frame #0: 0x00000008003ef3aa libevhtp.so.0`evhtp_parse_query_wflags(query="7fb4f6eb233e228d951082079e3b7c2f988eaf05", len=1048708, flags=0) at evhtp.c:3531
    frame #1: 0x00000008003efa1f libevhtp.so.0`evhtp_parse_query(query="7fb4f6eb233e228d951082079e3b7c2f988eaf05", len=1048708) at evhtp.c:3750
    frame #2: 0x00000008003f8144 libevhtp.so.0`htp__request_parse_fini_(p=0x000000080335b800) at evhtp.c:2043
    frame #3: 0x00000008003ff46e libevhtp.so.0`hook_on_msg_complete_run(p=0x000000080335b800, hooks=0x0000000800403038) at parser.c:333
    frame #4: 0x00000008003fea5a libevhtp.so.0`htparser_run(p=0x000000080335b800, hooks=0x0000000800403038, data="f573e", len=802) at parser.c:2200
    frame #5: 0x00000008003f66b5 libevhtp.so.0`htp__connection_readcb_(bev=0x00000008032e3000, arg=0x000000080270a7c0) at evhtp.c:2267
    frame #6: 0x00000008006462ce libevent-2.1.so.7`bufferevent_run_readcb_(bufev=0x00000008032e3000, options=0) at bufferevent.c:251
    frame #7: 0x000000080064fbe4 libevent-2.1.so.7`bufferevent_trigger_nolock_(bufev=0x00000008032e3000, iotype=2, options=0) at bufferevent-internal.h:411
    frame #8: 0x000000080064f51a libevent-2.1.so.7`bufferevent_readcb(fd=45, event=2, arg=0x00000008032e3000) at bufferevent_sock.c:214
    frame #9: 0x000000080065c658 libevent-2.1.so.7`event_persist_closure(base=0x00000008026f8000, ev=0x00000008032e3010) at event.c:1629
    frame #10: 0x000000080065bbef libevent-2.1.so.7`event_process_active_single_queue(base=0x00000008026f8000, activeq=0x0000000802624640, max_to_process=2147483647, endtime=0x0000000000000000) at event.c:1688
    frame #11: 0x0000000800656069 libevent-2.1.so.7`event_process_active(base=0x00000008026f8000) at event.c:1789
    frame #12: 0x0000000800654e83 libevent-2.1.so.7`event_base_loop(base=0x00000008026f8000, flags=0) at event.c:2012
    frame #13: 0x0000000800400f1f libevhtp.so.0`_evthr_loop(args=0x00000008019947e0) at thread.c:121
    frame #14: 0x0000000801193776 libthr.so.3`___lldb_unnamed_symbol1$$libthr.so.3 + 326

Sorry, this took so long to circle back to, here is a backtrace.

I've finally determined the root cause of this, however, I'm still trying to determine why it is happening.
At [1] memory is allocated to the stack or a buffer depending on the EVHTP_HAS_C99 flag. I'm still not sure how to determine the max stack size, but in this particular case, the stack isn't large enough for the data which is causing a stack overflow.
Compiling with -DEVHTP_HAS_C99=false fixes the error because it will allocate using buffers instead. https://github.com/haiwen/seafile/issues/1928 also reported this bug quite awhile ago.
[1] https://github.com/criticalstack/libevhtp/blob/1.2.16/evhtp.c#L3502


Consider an "eager warning" when configuration is wrong prior to execution

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3542
Originally created by @tayloramurphy on 2022-05-28 01:32:44

As discussed in https://gitlab.com/meltano/meltano/-/issues/3209#note_962955570 we deprioritized "warning eagerly" when there is conflicting configuration.

View 1 previous comment from the original issue on GitLab

Still relevant

Superseded by:

https://github.com/meltano/meltano/issues/7285
https://github.com/meltano/meltano/issues/7459

May be impacted by:

#7400

Presumably Meltano could perform some lightweight linting/validation (covered by the above issues) automatically prior to executing pipelines.


Clearly define who are the project leaders

This PR address the first bullet point in #26

cc @paulmelnikow @johnpneumann

Pushed a couple fixes, PTAL

Did I forgot to add @nicoulaj to that list? I am not sure he wants to deal with prezto issues.
@nicoulaj please let me know if you should be in that list.

I don't think he does, though we should ask!

@paulmelnikow you will have to approve before we can merge this

Was that just turned on?

yeah, someone turn it on.


Support cluster mode

It looks like there hasn't been much thought put into supporting cluster mode in Spark-bench, which may be useful in some cases.  In particular, Spark on Kubernetes currently only works in cluster mode, so this is a precondition for supporting the Kubernetes scheduler.

Thank you for filing this, @showermat!
One way in which this affects spark-bench in particular is that the temporary "split" configs aren't necessarily written to the driver so they don't get passed around properly.
Two possible fixes:

Have the temp files write to HDFS, or
Instead of writing temp files and passing a file path to them, pass the config as one giant child argument to spark-submit.

A workaround for this is putting spark-bench jars in hdfs.


Build with mingw-w64 / msys2

Is it possible to build static libraries (libhedis.a) that can be linked into a mingw-w64 application?

Hi @jeroenooms
if you are interested in stand alone hiredis, you'll find it here https://github.com/microsoft/hiredis.
The one used for Redis on Windows has too many dependencies to be linked in other projects.

Thanks! The question still stands though if that can be built with mingw-w64 / msys2 ?

@jeroenooms
as you may imagine, it's not a supported scenario, but it may work, probably with some fixes here and there.


Perf improvements to Monaco editor with webworkers

Right now we have two options to improve the perf of tokenization in monaco:

Option 1: Web workers with out custom webserver. (to be spiked https://github.com/microsoft/vscode-python/issues/8182)
Option 2: Web workers, using our custom worker proxy (we create a web worker in node and proxy the comms ourselves from monaco to this webworker). The architecture is currently pluggable so we can choose how we interact with these workers.

https://github.com/Microsoft/monaco-editor/issues/1317#issuecomment-468680023
Option 1 is more appealing as this involves lot less work. If that fails we can consider Option 2
@rchiodo @IanMatthewHuff  /cc

I'm not sure how Option 1 is going to work? Isn't this all client side rendering? How do we run the tokenizer on the server?

With option 1, when using a custom webserver, then UI is running over http, instead of file://. At this point webworkers will work. Right now our UI is running on file scheme. Hence no webworkers.

Oh so just because of http, webworkers will be supported in electron? That seems odd.

Not sure this is true:

Right now our UI is running on file

I believe it uses a custom scheme. We don't give VS code an html file, we give a string with the html in it.

This is mostly done and verified that it works. We implemented option 1.

@rchiodo . Yeah this was one costing item that I really had low context on. Was just reading up on web workers myself wrt this. I wasn't sure if the move to http in the webserver would be getting this functionality by default with monaco or if we needed to do something more to enable this. Maybe I'm looking in the wrong spots but I not yet seeing good information on how these work in monaco.

I believe it's an Electron thing. @DonJayamanne  would know more. Web workers aren't supported with file uris, but are for http requests.
This article explains partially why?
https://medium.com/swlh/how-to-run-background-worker-processes-in-an-electron-app-e0dc310a93cc

I think we can move this to validate. As it will work with the web server approach.
If we wanted to support webworkers in non-webhost, we'd have to implement our own class to proxy messages from react layer into our custom webworker implementation (monaco have hooks in place that would allow us to do this).
Though don't see much benefit if we're going down web server path and also with VSC implementing their own native notebook editor. Which would take care of perf completely (i.e. take care = not our problem).

Moving to validate per discussion


enable graph mode for hpu

Add graph mode for hpu. Use Model "BAAI/bge-large-en-v1.5" as the backend model, after enabling this feature, avg inference time drops from 18ms to ~3ms. Pls help review @libinta

@regisss  please help review the PR


Free Services is not listed on page

[Enter feedback here]
Free Services is not listed on page

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

ID: caadbfdd-05b5-dc2e-7fd6-69ff2f9fb2e9
Version Independent ID: f76cad32-0690-b06f-4447-0626b92d3e1f
Content: Avoid charges with your Azure free account
Content Source: articles/billing/billing-avoid-charges-free-account.md
Service: billing
GitHub Login: @amberbhargava
Microsoft Alias: banders

@bizinbarefeet Thanks for the comment! Which part of the article are you referring to for Azure free services? Please provide more details so that we can take appropriate action.
FYI: Here is a full list of free Azure services for 12 months.

@bizinbarefeet Closing this issue for now. You may reopen the issue by providing more details and tag me in your response.


Storybook has a failing case on master

Griddle main > with custom griddle key that doesn't exist
Griddle: Property 'garbage' doesn't exist in row data. Please specify a rowKey that exists in <RowDefinition>
    Error: Griddle: Property 'garbage' doesn't exist in row data. Please specify a rowKey that exists in <RowDefinition>
    at transformData (http://localhost:6006/main.771f17111d3e896ed4f6.bundle.js:6810:11)
    at Module.GRIDDLE_INITIALIZED (http://localhost:6006/main.771f17111d3e896ed4f6.bundle.js:5509:97)
    at GRIDDLE_INITIALIZED (http://localhost:6006/main.771f17111d3e896ed4f6.bundle.js:4216:83)
    at http://localhost:6006/main.771f17111d3e896ed4f6.bundle.js:6602:14
    at http://localhost:6006/vendors~main.771f17111d3e896ed4f6.bundle.js:26792:31
    at callReducerWithBeforeAfterPipe (http://localhost:6006/main.771f17111d3e896ed4f6.bundle.js:6607:10)
    at reducer (http://localhost:6006/main.771f17111d3e896ed4f6.bundle.js:6617:12)
    at p (<anonymous>:1:36402)
    at v (<anonymous>:1:36684)
    at <anonymous>:1:40069

hey @docrinehart, I'm taking a look at this one, I'll try to figure out what the problem is.

hey @docrinehart  this is the expected functionality as far as I can see, when Griddle is initializing there's a reducer which is setting initial Immutable data object, you can take a look at src/utils/dataUtils.js:
  // Validate that the first item in our data has the custom Griddle key
  if (hasCustomRowId && data.length > 0 && !data[0].hasOwnProperty(renderProperties.rowProperties.rowKey)) {
    throw new Error(`Griddle: Property '${renderProperties.rowProperties.rowKey}' doesn't exist in row data. Please specify a rowKey that exists in <RowDefinition>`);
  }

I also took a look at the 1.13.1 tag, and this story is also failing there. so I think you can close this issue.
cc @ryanlanciaux


[MRG] General Image Slider

Add the ability to create a general html slider with a list of figs. This can be expanded for a list of images as well.
This is currently functional. Feel free to take over the pr if it requires non-trivial reworking since I'm a bit occupied with things.

this is useful for visualizing time-series data in html, e.g. topomaps with changing distributions, brain activation over time, a flip book cartoon, etc. :)

I think @teonlamont wants to nerdsnipe you @jaeilepp ;)

https://youtu.be/CSj0lajQBrM?t=18s

That's really cool! Thanks a lot.
Any chance you could add a 'play' button on top of this?

It would also be great to add a routine to make an evoked signals. Currently I'm doing this:
fig_list = []
for time in evoked.times:
    fig_list.append(evoked.plot_topomap(times=time, show=False))
share_clim([ax for fig in fig_list for ax in fig.get_axes()])
report.add_slider_to_section(fig_list, evoked.times)

But this is rather slow. It'd probably be faster to make a single figure, and iteratively draw the imshow values.
But this can be a subsequent PR :)

can you share demo page?
travis is not happy

https://www.dropbox.com/s/dzsj92vrgugijyg/test.html?dl=0
haven't figured out the travis error. it's telling me this:
mne.report.Report.add_slider_to_section arg mismatch: ['captions', 'figs', 'image_format', 'scale', 'section']

it's from test_docstring_parameters, any idea?

can you reproduce locally?
We should have some margins here and there and center things properly

it's complaining that your docstrings are not proper eg in the right order

also the footer says 2013 :(

that's been there for a while. I think @mainakjas added that range from when it was initially created, should we update to go to the current year. it's on all the report....

i think it was because the test was parsing based on : and I had Data: ... in my docstring explanation.

also, the format of the slider is currently using the same as the BEM slider. Should we reformat those to also be centered? currently they are left-aligned. also, they don't have the window frame around it and the header is bigger

fix whatever needs to be fixed and only this :)

jk, I forgot to title the Parameters, Travis will be happy again

I will buy someone a beer if they can figure out how to center the slider

only three things are needed:

[ ] center the slider
[ ] add a play/pause button
[ ] add test

optional:

[ ] write a function for quickly generating a series of photos for the slider

@teonlamont will you be able to do this in the next two weeks? We have a release coming up, so if you want it in, that would need to be the timeline.

yeah, maybe toward the end of the week. I am working through my open pr in priority. if I finally close them out then I will come back to tackle this. I have to brush up on my html, sigh.

Great, thanks for tackling some of the old ones.

@mainakjas if you get a chance, could you check it out again. I created a template but the toggling problem persists.

ready for review

Can you try running the tutorial example to see nothing is broken?

One nitpick, you start indexing the slices from 0. Is that how the BEM sliders are also indexed?

@mainakjas the BEM slider index also starts at 0.

@agramfort, here's a link to the html: https://www.dropbox.com/s/2c2dsyptoxs8omi/testing.html?dl=0, and the script below.
from mne import report
from mayavi import mlab
import numpy as np

# define a mesh
pi = np.pi
cos = np.cos
sin = np.sin
dphi, dtheta = pi / 250.0, pi / 250.0
[phi, theta] = np.mgrid[0:pi + dphi * 1.5:dphi,
                        0:2 * pi + dtheta * 1.5:dtheta]
m0 = 4
m1 = 3
m2 = 2
m3 = 3
m4 = 6
m5 = 2
m6 = 6
m7 = 4
r = sin(m0 * phi) ** m1 + cos(m2 * phi) ** m3 + \
    sin(m4 * theta) ** m5 + cos(m6 * theta) ** m7
x = r * sin(phi) * cos(theta)
y = r * cos(phi)
z = r * sin(phi) * sin(theta)

rep = report.Report()
a = mlab.figure()
mlab.mesh(x, y, z)
b = mlab.figure()
mlab.mesh(y, x, z)
rep.add_slider_to_section([a, b], ['wug', 'life'], 'SLIDER')
rep.save('testing.html', overwrite=True)

@mainakjas which tutorial example?

there is still a centering issue right?

@teonlamont this one: http://martinos.org/mne/dev/tutorials/report.html

@teonlamont can you also update whats_new?

@kingjr does this work for you? @teonlamont can you share a nice example report?

Yep, works fine on my ubuntu chrome & firefox.
Thanks @teonlamont!

@mainakjas there's an updated example here https://github.com/mne-tools/mne-python/pull/2234#issuecomment-144452784.
@agramfort I have it centered according to the thumbnail and it seems to be aligned with the image. before, it was left-justified.

@mainakjas I'll you have a close look and merge if you're happy. You know this code better than all of us.

@teonlamont besides the svg comment, update whats_new and I'm happy :)

@mainakjas for the svg, what to do... :/

On Fri, Oct 2, 2015 at 12:31 AM, Teon L Brooks<EMAIL_ADDRESS>wrote:

@mainakjas https://github.com/mainakjas for the svg, what to do... :/

I would drop svg support for the slider if you can't get it to work. I am
not sure it makes a lot of sense even conceptually

—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/pull/2234#issuecomment-144866802
.

@mainakjas svg support dropped. all is good now. it's ready :)

@mainakjas can you have another look when you have a chance?
@teonlamont can you rebase? Probably a whats_new conflict

@mainakjas has been frozen out of his account :/ something dealing with the student pack expiring.

Okay, I'll try to have a look after you rebase. Can you tell me what I should do to test it? Run an example, hopefully?

here's one: https://github.com/mne-tools/mne-python/pull/2234#issuecomment-144452784

@teonlamont do you think it's worth actually adding an example or modifying an existing one? If you do, please don't use Mayavi unless necessary (less compatible than mpl)

yeah, i will make an example this afternoon. I included mayavi here so we could test out all the subfunction routines. i agree with you on not using mayavi in the example, i had a headache making sure it worked.

Sorry @teonlamont you'll need another rebase, but probably only because of whats_new.py. Let me know when the example is up and I should be able to merge.

ping
we're in release mode.
Otherwise it will wait. It's fine with me.

I noticed that there are no other examples of the report, only the tutorial page. should we make a full example of the report or is the slider sufficient?

Let's start with the slider and incrementally add to it. If you're feeling generous you could even include the new .gif support if there is an easy relevant use case (BEM shells?).

Looks like you already have the basic one, let me know if you plan to add more (as per above) or not. If not, I'll review and merge if it's all good.

I'll ping @kingjr to see how he went about creating the .gif and I could add the a BEM one if it's sufficiently easy. will be away from the computer for a couple of hours

actually, I just realized that a BEM one might be a bit complex. I don't think the user has access to the figures(?). perhaps another type of time-series plot.

I think it's ready to review. I will squeeze in any additional report example when I get the chance.

Cool. (Or you could capture topomaps at multiple time points and cycle through them or something, if you get around to it later.)

FYI I use http://matplotlib.org/api/animation_api.html for simple plots. For list of bitmap, I use image2gif https://code.google.com/p/visvis/source/browse/vvmovie/images2gif.py (new BSD licence)

thanks @teonlamont


Link to Linuxbrew issue tracker, not Homebrew

fix #401

I think I found a couple more instances of linking to homebrew in the docs, so I'll add these in.

Not sure if this instance should be replaced: https://github.com/Homebrew/linuxbrew/blob/5d1ef0f4a670cccb5835327bfbfede40cbb468eb/share/doc/homebrew/Formula-Cookbook.md#bad-makefiles should be changed, bad makefiles will probably affect both homebrew and linuxbrew so I'd assume they be reported upstream.
I'm also iffy on this one: https://github.com/Homebrew/linuxbrew/blob/5d1ef0f4a670cccb5835327bfbfede40cbb468eb/share/doc/homebrew/Formula-Cookbook.md#basic-instructions, should that point to both trackers to check if a bug has already been reported?
And I would assume the same question as above would go to which bug tracker the manpage would link to: https://github.com/Homebrew/linuxbrew/blob/f765cd358e3608cdd3088b71b16868489b73f94e/Library/Homebrew/manpages/brew.1.md#bugs

Actually, could the Formula Cookbook be tossed, and just replaced with "give new softwares to either Homebrew/Homebrew or Homebrew/Homebrew-dupes if they are pre-installed on Mac"?

Perhaps, but I'm disinclined to replace all of share/doc/homebrew/* with stubs pointing to upstream Homebrew. I'm not so concerned that all the documentation be carefully edited to replace Mac terminology with Linux terminology and replace Homebrew links with Linuxbrew links. I'm just not sure it's worth the effort. I'm inclined to do it on a case-base-case basis if a particular piece of documentation causes extreme confusion and havoc with users documented by issues opened by users.

Fixed.


Charts does not render color by using a different culture

Describe the bug
I'm using ChartJs.Blazor on a multi-language UI.
The UI language is chosen by using the CookieRequestCultureProvider.
When using culture "en-US" colors on charts display properly.
When change culture to "it-IT" colors on charts does not render properly: they were display by using a gray scale.
Which Blazor project type is your bug related to?

Server-Side

Version of ChartJs.Blazor
v.1.1.0
Which charts does this bug apply to?
In the repro there is ChartJsPieChart, but this behaviors happen also to the ChartJsLineChart
To Reproduce
Use this repository to reproduce the issue: https://github.com/alessandro-gargiulo/ChartJs-GreyColor-Repro

Switch language to English.
Switch language to Italian.

Expected behavior
The colors are rendered correctly as they are rendered by using English culture.

Thank you for the report and repro project. I'm surprised that not more people have run into this. I discovered it a while back and hope to have fixed it (see 960016ff) but unfortunately, this commit isn't included in the Version 1.1.0 you're using. I suggest you upgrade to latest preview version (2.0.0-preview1) which should fix the issue.
Please report back if it worked :)

Sorry for late response.
I have tried with the 2.0.0-preview1 and this fix the problem. Thank you.


chart data type bug

Hello,
I use superset to create a new virtual dataset and create a new column “account_name”. The column's type is String.
But then I want to create a new line chart and group by the column  “account_name”，it comes out a bug , says "Column account_name is not under aggregate function and not in GROUP BY". I don‘t understand why the line chart  do this ：toStartOfHour(toDateTime(account_name))
U can see the pictures below.
Hope to hear from you ~Thanks

Already solved.


Market Basket Analysis using Apriori Algorithm

There is no such project already existing. So I want to work on this issue.
in this project I will be analyzing different items purchased using Apriori Algorithm.

Is there any prediction in this?

And why have you opened same issue twice?

Is there any prediction in this?

Only analysis

And why have you opened same issue twice?

Actually I am having slow net so by mistake it got created twice


Noise cancellation issue with AudioConfig.fromStreamInput

Noise cancellation issue with AudioConfig.fromStreamInput:
Description
Speech SDK recognizer unable to cancel the noise from the audio stream input
AudioConfig audioInput = AudioConfig.fromStreamInput( wavStream );
SpeechRecognizer recognizer = new SpeechRecognizer(speechConfig, audioInput);
wavestream processing details:
cognitive-services-speech-sdk/samples/java/jre/console/src/com/microsoft/cognitiveservices/speech/samples/console/WavStream.java at b55026a09e2f807db9289acd6cdd3b623f44b3e9 · Azure-Samples/cognitive-services-speech-sdk (github.com)
Also when we are specifying the audio processing mentioned in Use the Microsoft Audio Stack (MAS) - Speech service - Azure AI services | Microsoft Learn we are getting the below error.
Code:
MicrophoneCoordinates[] microphoneCoordinates = new MicrophoneCoordinates[7];
microphoneCoordinates[0] = new MicrophoneCoordinates(0, 0, 0);
microphoneCoordinates[1] = new MicrophoneCoordinates(40, 0, 0);
microphoneCoordinates[2] = new MicrophoneCoordinates(20, -35, 0);
microphoneCoordinates[3] = new MicrophoneCoordinates(-20, -35, 0);
microphoneCoordinates[4] = new MicrophoneCoordinates(-40, 0, 0);
microphoneCoordinates[5] = new MicrophoneCoordinates(-20, 35, 0);
microphoneCoordinates[6] = new MicrophoneCoordinates(20, 35, 0);
MicrophoneArrayGeometry microphoneArrayGeometry = new MicrophoneArrayGeometry(MicrophoneArrayType.Planar, microphoneCoordinates);
AudioProcessingOptions audioProcessingOptions = AudioProcessingOptions.create(AudioProcessingConstants.AUDIO_INPUT_PROCESSING_ENABLE_DEFAULT, microphoneArrayGeometry, SpeakerReferenceChannel.LastChannel);
AudioConfig audioInput = AudioConfig.fromStreamInput( wavStream, audioProcessingOptions);
SpeechRecognizer recognizer = new SpeechRecognizer(speechConfig, audioInput);
Error:
java.lang.RuntimeException: Exception with an error code: 0x5 (SPXERR_INVALID_ARG)
at com.microsoft.cognitiveservices.speech.util.Contracts.throwIfFail(Unknown Source) ~[client-sdk-1.37.0.jar:na]
at com.microsoft.cognitiveservices.speech.audio.AudioProcessingOptions.create(Unknown Source) ~[client-sdk-1.37.0.jar:na]
To Reproduce
Steps to reproduce the behavior:
We are using websockets to receive the audio stream and giving input like below
public void handleBinaryMessage(WebSocketSession session, BinaryMessage  message) throws Exception {
byte[] arr = new byte[message.getPayloadLength()];
message.getPayload().get(arr);
SpeechConfig speechConfig = SpeechConfig.fromSubscription("****************", "");
speechConfig.setSpeechRecognitionLanguage("en-IN");
InputStream bStream = new ByteArrayInputStream(arr);
WavStream wavStream = new WavStream(bStream );
    AudioConfig audioInput = AudioConfig.fromStreamInput( wavStream);

    SpeechRecognizer recognizer = new SpeechRecognizer(speechConfig, audioInput);

    recognizer.recognized.addEventListener((s, e) -> {
        if (e.getResult().getReason() == ResultReason.RecognizedSpeech) {
            System.out.println("RECOGNIZED: Text=" + e.getResult().getText());
        } else if (e.getResult().getReason() == ResultReason.NoMatch) {
            System.out.println("NOMATCH: Speech could not be recognized.");
        }
    });

Expected behavior
When we are chosing the microphone input noise is being cancelled but not with the stream,
Version of the Cognitive Services Speech SDK
1.37 we are using.

@ru4sam326 Thank you for writing this up here for better visibility from our team's Java experts. I don't believe you need to configure a microphone geometry since you're using a pushStream audio input. Could you try this code and see if it works?
AudioProcessingOptions.create(AudioProcessingConstants.AUDIO_INPUT_PROCESSING_ENABLE_DEFAULT);
AudioConfig audioInput = AudioConfig.fromStreamInput( wavStream, audioProcessingOptions);

Duplicate of Azure incident 505180412. Why create 2 issues about the same topic on different forums?


Support 'unknown' param, closes #252.  Tidy and reorg service/lib interface

@juandopazo could you let me know if this works for you?

Looks great! I'll try to set up monitoring to see if we run into those kinds of issues you mentioned with big bundles.


Approval needed for not-installed Yarn package

I am developing locally on OS X with Webpack(er), which has an optional transitive dependency on OS X exclusive package fsevents. yarn install installs it and License Finder thinks it is OK probably because I have whitelisted MIT licenses.
However, on the Travis CI build using Linux:
yarn install v1.3.2
...
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
...
bundle exec license_finder --quiet

Dependencies that need approval:
fsevents, 1.1.3, unknown

So it would appear that due to the package being excluded/not fully installed due to incompatibility, the license information is still checked for approval but not found correctly.
I have resolved the problem by adding a dependency approval specific to the fsevents package, but I would expect License Finder to either (a) ignore the package when it is excluded or (b) still be able to find the license information like it does locally when the package is compatible and installed/included.
This is with license_finder 5.0.0.

I don't know if this is still true? Running yarn licenses list --recursive --ignore-platform does yield all packages, regardless of the current platform (using Yarn 1.22.5). I would request that this is reopened (or I could open another issue).


Add invoke option to ordered_class_elements

This PR adds an invoke option to OrderedClassElementsFixer.
When creating invokable controllers in Laravel I typically want the __invoke() method listed before any helper methods.
Adding this option will allow specifying in which order the __invoke() method appears.

hey @driftingly and thanks for the PR!
can you share if this is some kind-of well established standard for any open source community?

Coverage increased (+0.0007%) to 93.149% when pulling 8c661ba97ed845c4bfba32906a7b554e5a32f37b on driftingly:adc/add-invoke-ordered-class into 333f15e07c866e33e2765e84ba1e0b88e6a3af3b on FriendsOfPHP:master.

isn't __invoke() already be covered by config option magic  anyways?

hey @driftingly and thanks for the PR! can you share if this is some kind-of well established standard for any open source community?

In Laravel you can create single-action controllers .
<?php

namespace App\Http\Controllers;

use App\Http\Controllers\Controller;
use App\Models\User;

class ProvisionServer extends Controller
{
    public function __invoke()
    {
        // ...
    }
}

When these invokable methods have any helpers ideally these would be listed below the invoke() method.  In this capacity invoke() similar to the __contstruct() method.
class ProvisionServer extends Controller
{
    public function __invoke()
    {
        $result = $this->helperMethod();
    }

    private function helperMethod()
    {
        return $result;
    }
}

This doesn't appear to be an established Laravel standard but it is the preference at the agency I work at.

isn't __invoke() already be covered by config option magic anyways?

It is covered by magic, but in the same way  __construct() is, adding a separate option for __invoke() allows greater control in which order it appears.

+1 for this, __invoke is often used as a "handler" method for single action controllers as @driftingly mentioned, and also for things like jobs, event listeners, and actions.
It's similar to __construct and I would always want it at the top of a class (right after __construct if present). Would love to be able to configure that with this PR!

does this not work if you use magic ? and if not, can somebody share an example of that?

Using magic puts them all at the top, including __call, __callStatic, __get, etc. Those I generally still want last, it's specifically __invoke I would want at the top.

That shouldn't be the case, using magic should puts al_ magic methods at the place where you configure these, not just all at the top. I cannot reproduce this either, so an example to reproduce would be needed.
I know however that currently you cannot configure putting invoke in a different location than the other magic methods,
This I don't consider an issue as I know of no standard that recommends this exception of invoke over the other magic methods.

Sorry, yes, that's what I meant—not necessarily at the top but all together. So if you wanted __invoke at the top, the only way to do that currently would be to to put magic at the top, which puts them all there.
You're right, there is no standard that enforces placing __invoke at the top of the class. It's a convention in the Laravel ecosystem, and it's something that we would like to be able to use PHP-CS-Fixer for at work, rather than doing manually. Lots of other existing rules and options aren't standards either but are still very useful. This PR adds support for one more existing formatting convention/preference, which I think is a valuable addition to the tool ☺

What @bakerkretzmar is saying is correct.  We would like to treat the __invoke() method similar to the __construct() method and specify the order outside of it being included with the other magic methods.
With configuration:
['order' => [
    'constant_public',
    'construct',
    'invoke',
    'method_public',
    'magic',
    ]
]

Before:
<?php

class Example
{
    public const EXAMPLE = true;

    public function helperFunction(){}

    public function __invoke(){}
}

After:
<?php

class Example
{
    public const EXAMPLE = true;

    public function __invoke(){}

    public function helperFunction(){}
}

Before:
<?php

class Example
{
    public const EXAMPLE = true;

    public function __toString(){}

    public function someMethod(){}

    public function __construct(){}
}

After:
<?php

class Example
{
    public const EXAMPLE = true;

    public function __construct(){}

    public function someMethod(){}

    public function __toString(){}
}

Thanks for all the details and work so far. IIUC this is only an issue if you have multiple magic methods in your project and you want __invoke at some other location than the other magic methods, yet care for all magic methods.
I'm +0 on it as I don't see a wide use for it, however I see the Laravel use case (cc @GrahamCampbell :)  ).
I would like to see unit tests for using both magic and invoke to see the behavior when using both and another utest when using only this new option.
I assume that invoke as a more detailed option is honored over the more generic magic case.
But lets see if other members and/or maintainers have more ideas on this first :+1:

I think it is too specific. Don't use magic in your order, then you can place these methods where you want (but it will not be enforced).
Maybe we could introduce a more general solution to place methods by name. Like:
['order' => [
    'constant_public',
    'construct',
    ['method', '__invoke'],
    ['method', 'myOtherSpecialMethod'],
    'method_public',
    'magic',
]]

I prefer the more generic solution proposed :+1:

Same, generic solution would provide a lot more functionality.


What is the alternative APIs to use when the GetCurrentConditionsForStation does not have a current temp value?

This is not a bug report. This is a question about the results returned.
Is there another set of NWS APIs to use when the GetCurrentConditionsForStation request returns no value for current temperature?
When I attempt to get the current temperature for i.e. Avon, Colorado, I frequently fail to see a value for TemperatureFahrenheit. Is there another service API I can use to obtain a locations current temperature?

I believe there are other endpoints in the API that you can get other forecasts from other stations. But I don't recall what they are. This library only supports endpoints recommended by the NWS for the latitude and longitude entered.


API refactor LeftAndMain_Menu.ss into individually overridable compon…

@tractorcow I thought about merging this straight in, but confidence. Have tweaked your commit to split out the logo and status.
Replaces https://github.com/silverstripe/silverstripe-framework/pull/5176, which replaced https://github.com/silverstripe/silverstripe-framework/pull/4795

Thanks @jonom !

Thank you! Hope you're feeling better


[React Wrappers] Remove Initial Attributes

Right now the initial attributes are being set directly on the element and when the component is loaded. We should be able to remove the former to create redundancy and make for leaner components.

A fix for this has been deployed in release 1.2.0 (https://github.com/break-stuff/cem-tools/pull/67).


removing subset_channels for stub

Motivation
I think, stub tests should only slice the recording in frames and not channels to be as realistic as possible.
Thus the subset_channels attribute is unnecessary.
In addition, the BaseRecordingExtractorInterface.subset_recording() is useful only when stub_test=True thus its unnecessary to have stub_test as argument to that.
Checklist

[ ] Have you checked our Contributing document?
[ ] Have you ensured the PR description clearly describes the problem and solutions?
[ ] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
[ ] If this PR fixes an issue, is the first line of the PR description fix #XXX where XXX is the issue number?

Closing due to too many merge conflicts


Store and load current state for AggregateProcessor on restart of Data Prepper

As a user of the AggregateProcessor, I would like to save state between restarts of Data Prepper

We could implement this now, but we should consider a more wholistic approach to saving state within Data Prepper first.


Azure devops node api null handling

What problem does this PR address?
Null return from azure-devops-node-api method
Is there a related Issue?
#1579
Have you...

If you are willing, please enable me, as the Repo owner, to edit your fork that contains your PR so I can add any missing items such as the readme.md

@rfennell Ready to look at.

I have tested the code in the PR for the 'happy path' (getting parent and children as normal) and works as expected.
However, I am just struggling to test the failure condition the change protects against. Can I confirm my test process

Have a WI (13) with 2 Child WI (62, 670)
Make a commit associated with the parent WI (13)
Run a YAML CI build for the commit
Using this Task's Test Console generate release notes against the YAML build with the "getParentsAndChildren": "true". I see the parent and the 2 child WIs as expected in the log

Looking for parents and children of WI [13]
Add Child WI 62
Add Child WI 670

Delete WI (670) in the Azure DevOps UI (so it is in the recycle bin)
Re run test console with the same configuration

With current task code (on main) I expect to see an error, I do not, it completes just adding the WI 62 child
With new task code (this PR) I expect to see message NOT adding child WI 670 as it is null. It may have been deleted. I do not.

I am assuming there is an error in my test plan

@rfennell I thought it was because the work items were deleted, but after tracking down a team member with access to the work item, I believe it's because the PAT I generated from my account doesn't have read access to the Area that the work item has been moved to.
To reproduce you would move a work item to an area and restrict a user from being able to see that area here:

Then use that user's PAT to generate release notes in the test console.
I tried, but I couldn't add users to my Azure directory even on my test project and with a free personal test project I have to request permissions to run parallelisms directly from Microsoft.


[Bug] Changing color to hex in picker results in invalid color syntax

Describe the bug
When using color picker to cycle to the hex display format, the replaced value becomes invalid
To Reproduce
Steps to reproduce the behavior:

start off with a property with color hsl(190deg 100% 50%)
click on the swatch to show the color picker
click the button to change color mode to hex
some last characters of the rgb color are appended to the hex color

Expected behavior
color should toggle to valid hex
Screencast
https://github.com/willofindie/vscode-cssvar/assets/5300963/75db914e-f02e-4d02-89e6-41ea93c39a4e
Details (please complete the following information):

OS: Ubuntu 22.04
VSCode version 1.84.2
Extension version v2.6.4

Additional context
What's more, the color syntax that the picker uses is using old, comma separated syntax.

Hi @T3sT3ro
This is not an extension bug. It's an issue from VSCode's inbuilt CSS support.
To verify you can try disabling/removing the extension and verify the same scenario. It should replicate (coz it did for me)


Added possible yushan chapter reorg

Glen's possible outline has been translated and merged to the main branch — would you like to resolve the conflict first so we can merge the edits? Could be as simple as moving the gold card paragraphs into the appropriate section ("Moving Forward")

@vbuterin
Hello 你好，
可以參考這個連結：https://hackmd.io/_doIKl2dStK6BUYnq5WwSA
在金卡部分的引述希望可以作為投稿文，先放在「前行（Forward)」session, 我另撰寫了華文版。
P.S.
第一段落、第三段落 重新編譯的部分比較多-> 請參考
也建議在第二段落，可以增加關於 "Hypercerts" 和  "Tea" 項目的理念。
我們也可以直接用Hackmd協作，謝謝～

superseded by your later reorg.


Add cooldown_interval_ms and internal measurements for profiling

Motivation:
Sometimes running the script several times we get a strong scatter in the results.
from tvm.contrib.debugger import debug_runtime as graph_executor
...
m = graph_executor.create(graph, lib, ctx)
...
time_f = m.module.time_evaluator("run", dev, number=1000, repeat=1)
mean = time_f().mean
print(mean)

First try:

0.0846912 secs/iteration

Second try:

0.105601 secs/iteration

In general, the devices has an acceptable temperature range. If the system heats up to the upper limit for some time, it can force cooling by available ways. For example, by lowering the frequency.
In the following example the temperature reaches 90 degrees and after some time the frequency drops and after cooling is restored.
from tvm.contrib.debugger import debug_runtime as graph_executor
...
m = graph_executor.create(graph, lib, ctx)
...
time_f = m.module.time_evaluator("run", dev, number=1)
one = time_f().results[0]
nums = round(1/one)
time_f = m.module.time_evaluator("run", dev, number=1, repeat=nums*200)
t =  time_f()
mean = t.mean
print(mean)
0.105511 secs/iteration

print(t.get_measurement_sequence())
[0.083143681, ... , 0.147928064, ... , 0.147806464]

the results of device profiling

the plot drawn from t.get_measurement_sequence()

In the proposed change, we use a preemptive stop of the calculations to keep the temperature of the device within comfortable limits. As a result, the temperature does not exceed 70 degrees.
from tvm.contrib.debugger import debug_runtime as graph_executor
...
m = graph_executor.create(graph, lib, ctx)
...
time_f = m.module.time_evaluator("run", dev, number=num_iter, repeat=200,
                                                       min_repeat_ms=1000, run_delay=1000)
t =  time_f()
mean = t.mean
0.0832038 secs/iteration

print(t.get_measurement_sequence())
[0.083007232, ... , 0.083067136, ... , 0.083169024]

the results of device profiling

the plot drawn from t.get_measurement_sequence()

Examples:
from tvm.contrib.debugger import debug_runtime as graph_executor
...
m = graph_executor.create(graph, lib, ctx)
...
time_f = m.module.time_evaluator("run", ctx, number=5, repeat = 2)
t = time_f()
print(t)
print(t.get_measurement_sequence())
t.get_measurement_report()

Execution time summary:
mean (ms)   median (ms)    max (ms)     min (ms)     std (ms)
83.3554      83.3554      83.4615      83.2494       0.1061

[0.083227648, 0.083268608, 0.083121664, 0.082902272, 0.083726592, 0.083296512, 0.083305728, 0.08343168, 0.083588608, 0.08368512]

Repeat #0 had 5 internal runs. Execution time summary:
mean (ms)   median (ms)    max (ms)     min (ms)     std (ms)
83.2494      83.2276      83.7266      82.9023       0.2703
Repeat #1 had 5 internal runs. Execution time summary:
mean (ms)   median (ms)    max (ms)     min (ms)     std (ms)
83.4615      83.4317      83.6851      83.2965       0.1540

from tvm.contrib.debugger import debug_runtime as graph_executor
...
m = graph_executor.create(graph, lib, ctx)
...
m.run(number=100, repeat=2)

Node Name  Ops  Time(us)   Time(%)  Shape   Inputs  Outputs  Times

f1         f1   9451.75    9.822    (2, 2)  3       1        [[9.456, 9.465, ..., 9.463], [9.519, 9.367, ..., 9.347]]
f2         f2   5255.25    5.461    (2, 2)  4       1        [[5.261, 5.308, ..., 5.302], [5.227, 5.225, ..., 5.223]]
...
fn         fn   5214.75    5.419    (2, 2)  4       1        [[5.266, 5.192, ..., 5.199], [5.259, 5.142, ..., 5.147]]

Changes:

Added cooldown_interval_ms parameter to next functions:
python\tvm\runtime\vm.py:benchmark
python\tvm\runtime\module.py:time_evaluator
python\tvm\contrib\graph_executor.py:benchmark
python\tvm\contrib\debugger\debug_executor.py:run
python\tvm\contrib\debugger\debug_executor.py:run_individual
python\tvm\contrib\debugger\debug_executor.py:run_individual_node
The parameter is used to stop measuring/cooling of the device.
For python\tvm\contrib\debugger\debug_executor.py:run number, repeat, min_repeat_ms parameters were added.
Changed python\tvm\runtime\module.py:BenchmarkResult class initializing argument to measurements: Sequence[Sequence[float]]. Measurements is the previous one from which the results are produced.
The next functions have also been added to this class:
get_measurement_sequence() - returns measurements as a sequential one-dimensional array.
get_measurement_report() - prints a detailed report for each repeat using measurements.
Fixed RunIndividual logging.
Removed USE_PROFILER that is no longer in use.

@areusch @tmoreau89 @AndrewZhaoLuo

@Icemist i like the direction here--but, i am wondering if it's possible to put some teeth behind this by allowing an application-specific way to verify the cooldown actually happened. for example, perhaps one could pass a PackedFunc that knows how to check the die temperature and continue cooldown if it is still too hot. what do you think of this? otherwise, it strikes me as a feature that would be hard to use in practice if you weren't already watching this separately.

@Icemist i like the direction here--but, i am wondering if it's possible to put some teeth behind this by allowing an application-specific way to verify the cooldown actually happened. for example, perhaps one could pass a PackedFunc that knows how to check the die temperature and continue cooldown if it is still too hot. what do you think of this? otherwise, it strikes me as a feature that would be hard to use in practice if you weren't already watching this separately.

This cooldown_interval logic already used for RPCRunner, LocalRunner and etc.
The other part of the patch only allows you to verify the quality of the data.
In your suggestion, in theory, you could try to use a ready-made f_preproc. Do 1 repeat, call f_preproc with some code to check and cool the temperature, do the next repeat.
for(i : repeat)
 call(f_preproc) // sleep here until the temperature is right
 timer.start()
 call(pf)
 timer.end()

It seems to me that both options have a place.

cc @tkonolige

@Icemist it'd be great if you could address @tkonolige 's comments.

In your suggestion, in theory, you could try to use a ready-made f_preproc. Do 1 repeat, call f_preproc with some code to check and cool the temperature, do the next repeat.

just wondering, would it be possible to achieve same effect as this PR with that strategy?

@Icemist it'd be great if you could address @tkonolige 's comments.

In your suggestion, in theory, you could try to use a ready-made f_preproc. Do 1 repeat, call f_preproc with some code to check and cool the temperature, do the next repeat.

just wondering, would it be possible to achieve same effect as this PR with that strategy?

It seems to me with that strategy it is more difficult to get a quick and generalized solution.
There may be a need for some general mechanics of adding such functions for different devices by developers familiar with these systems.

@areusch and  @AndrewZhaoLuo, what do you think? I addressed all the remarks. I also followed up on @tkonolige's request.

Leave the rest up to @tkonolige

@tkonolige I added a commit to this PR https://github.com/apache/tvm/pull/11465/commits/3e3b2cdd242089f777d969893e22e9b4d8fab517 where I change the way the RunIndividual and RunIndividualNode functions get results. I think this should solve the issue above.


[FEATURE] create Passport registration

This is an advanced user registration, so they already have the basic Viewer account.
Requirements

Refer to user's name, which should be available from profile
Input fields:

Street Address (text, required)
City or town (text, required)
State (text, required)
Zip code (text, required)
Why are you signing up to the NPDC? (required, textarea, minimum character count)

Submit button

Validation

Could someone please assign me to this issue? Thanks!

Just out of curiosity, is there any reason the zip code should be text? This seems like something that could be confusing to a user when it gets validated.

MVP of this issue is complete, future work on this feature will be outlined as needed in subsequent tickets.


Open Bing Chat in VCode

Is your feature request related to a problem? Please describe.

Describe the solution you'd like:

Describe alternatives you've considered:

Additional context:

Bing Chat isn't integrated in VSCode. However, if you're interested in using an AI assistant like Bing Chat, you might be interested in GitHub Copilot: https://code.visualstudio.com/blogs/2023/03/30/vscode-copilot
For info, though, this repository is related to the Edge DevTools extension in VS Code. It's not related to either Bing Chat or GitHub Copilot. So I'm going to close this issue.


Also another issue

Well, some objects dosnt respond to move and set pos functions in a certain points... like the text input bar, i set the y axis to the bottom of the screen and it stays on the center of the screen or just disppears!

That sounds weird, could you post a minimal example?

Sure

And this happens too, it disappears when its outside of some kind of container...

I would suspect that the problem here is that you are not thinking about the coordinates as relative to the parent of the view. For example, if I write:
auto container = new View(this);
container->setTopLeft(100);
container->setSize(300);

auto child = new View(container);

Now child is positioned at (100, 100) relative to the GUI, because the default top-left coordinates are (0, 0).

I would love to learn to use this library because its beautiful! you should do tutorials about this api, i think that people will like it

I've thought about making some tutorial about it, thanks for the motivation! Will do when I have time.

Or create a full documentation on it... will be amazin would be

Yes, might make some kind of website with documentation when the API is final enough, still making some big changes sometimes...
Did you understand what the problem was with your positioning? When you write getBottom() or getCenterY(), those coordinates are absolute and not relative to the parent of the view you're positioning. Also remember that child views are clipped by their parent, so if a child is outside of its parent, it's invisible. Also you might consider removing the container view completely and directly attach the text field to the GUI.

Lol i didnt thought about that... and thats right, it works right now! thanks you!

Nice!
What you would write to center a view within a container would be:
auto container = new View(this);
container->setSize(400, 300); // whatever

auto child = new View(container);
child->setCenter(container->getSize()/2.f);

// Now, if we move the container:
container->move(100, 200);
// The child will still be in the center of the container!

Haha, it took a long time. The motivation has always been strong though, I made it because I was frustrated that there was no library that fit my "needs". When I first started programming, I made games with SFML. Then I started to make fun little JavaScript projects which I put on my website. I always wished I could make anything in c++, including those types of projects. I searched a lot for libraries but the best I could find was like Qt, which is GIANT and it's not fully open source. I wanted a clean library with a logical API that gives a lot of freedom and can be used to make graphical programs as good looking as what you can make with html, css and javascript. I also wanted the framework to hold for making large applications.
I'm still working towards that goal, mainly by just making stuff with it and thinking about how it could be designed differently to work more smoothly, etc.

You inspired me to make one gui system myself!!! thanks you!


Angular 11 - compatibility

npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@angular/common
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/angular-dark-mode
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!

Thank you for pointing it out!
Would you like to submit a PR? If not I'll get to it in a day or two.


[docs] Schema level computed property

I'm not sure if it's entirely doable or not, but is there a way to have a schema level computed property? My case is that I have a paged list and want to be able to bubble up some commonly used properties to the schema object. Is this doable? See my example below, the hasResults property, I want to do a check if results has anything.
export default abstract class PaginatedResource extends Resource {
    static listShape<T extends typeof Resource>(this: T) {
        return {
            ...super.listShape(),
            schema: {
                results: [this.asSchema()],
                hasResults: undefined,    // <----- ????
                currentPage: undefined,
                pageSize: undefined,
            },
        };
    }
}

https://resthooks.io/docs/guides/network-transform#case-of-the-missing-id
You can do this by overriding the fetch member of your FetchShape. Compute whatever you want there and add fields. The linked example above adds a 'username ' field to the entity itself, but you have control of the entire response, so you can just as well add new things to stuff that only exists in the 'schema' part.

Thank you for pointing that out, I totally overlooked that part of the documentation. That should work perfectly.

You can now use SimpleRecord for this case. Replace the top level object and add a getter to your SimpleRecord class. @johnsoncode


MOBILE ERROR (HELP FAST AS POSSIBLE)

HI, YOUR SHADER IS AMAZING, BUT WHEN I SWITCH PLATFORM TO ANDROID(UNITY) THE SHADER LOOKS RED, PLEASE HELP ME, I AM 16 YEAR OLD STUDENT, AND NOT INTO SHADER THINGS... PLEASE HELP

MY GODNESS,I FIXED IT, BY CLICKING RANDOM GRAPHIC SETTINGS FOR Straight 30 Minutes , THANKS YOU FOR THIS GREAT ASSET.


Create address from xpub and hdpath.

Pull Request Test Coverage Report for Build 1532

0 of 0   changed or added relevant lines in 0 files are covered.
60 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 74.689%

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/address.js
60
75.77%

Totals

Change from base Build 1529:
-0.4%

Covered Lines:
2944

Relevant Lines:
3746

💛  - Coveralls

:tada: This PR is included in version 1.14.5 :tada:
Your semantic-release bot :package::rocket:


Delete TokenBalance from Realm when remove from list

Looks like TokenBalance stays in Realm after removing it from list


Static IP support

Doc says only dynamic IPs are supported
But according to https://docs.microsoft.com/en-us/azure/vpn-gateway/create-zone-redundant-vnet-gateway Azure now supports availability zones for VPN gateways. These require the use of standard tier public IPs, which are static only.
That appears to be a contradiction.

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

ID: 98d7fd30-80a8-f690-6191-d97ae4cac174
Version Independent ID: 3e15f99b-2f2d-7adc-a2d9-0b2184989db7
Content: Azure VPN Gateway FAQ
Content Source: articles/vpn-gateway/vpn-gateway-vpn-faq.md
Service: vpn-gateway
GitHub Login: @yushwang
Microsoft Alias: yushwang

@jtuliani Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Was there ever an update to this, can we now use Static IP for VPNGW ?

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


WIP: EXPRESS course clean up

As per https://github.com/expresslang/expresslang.org/issues/18

@manuel489 the "begin/end notes" identify what were originally lecturer notes. We need to convert them into real content that can be read before removing those markings. Can you help do that?

@manuel489 the "begin/end notes" identify what were originally lecturer notes. We need to convert them into real content that can be read before removing those markings. Can you help do that?

@ronaldtse

Each lecturer note contains a section title that is repeated afterwards.
I assume that we need to remove this repetition.
For some sections, it makes more sense to me to interchange the position of some paragraphs.
But that would mess with the lecturer notes, so I won't do it.

@manuelfuenmayor is this ready?

@manuelfuenmayor is this ready?

Not yet @ronaldtse, sorry. I have resumed my work.

Finally merged. Thanks!


iD preview became unusable

This happened just as I was about to add a driveway with preview.  I had searched for "driveway" in the add feature list, selected that choice, and then pressed R to start repeat mode.  Then, the map got all weird and things were messed up.
That one message just kept repeating on the console.

@BjornRasmussen Thanks for reporting. I accidentally introduced this when implementing the solution for #5701.


python312Packages.pypdf2: normalize pname

Description of changes
part of #245383
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

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

6 packages marked as broken and skipped:

chatgpt-retrieval-plugin
chatgpt-retrieval-plugin.dist
odoo
odoo.dist
odoo15
odoo15.dist

7 packages failed to build:

diffoscope
diffoscope.dist
diffoscope.man
python312Packages.pypdf2
python312Packages.pypdf2.dist
python312Packages.xml2rfc
python312Packages.xml2rfc.dist

14 packages built:

diffoscopeMinimal
diffoscopeMinimal.dist
diffoscopeMinimal.man
krop
krop.dist
maigret
maigret.dist
pdf-quench
pdfposter
pdfposter.dist
python311Packages.pypdf2
python311Packages.pypdf2.dist
xml2rfc (python311Packages.xml2rfc)
xml2rfc.dist (python311Packages.xml2rfc.dist)

build failure is already on master


Conditionally compile libb2 only on x86

This is part of the work for compiling Sorbet on M1 macs
Conditionally compile libb2 only on x86 builds to keep the performance gains and use the portable version everywhere else.
Motivation
We need to use the portable version blake2 instead of the x86 specific libb2 for ARM64 builds.
Test plan
Current tests should cover it.

I switched to the suggest macros, which indeed works. However, I'm still unsure as to why the EMSCRIPTEN macro is no longer defined. Switching to __EMSCRIPTEN__ also did not help. Any ideas?

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_L9qRDyzQvdAzXb
→ https://go/builds/bui_L9qR3Kjq5RG2j4


The file generated by load in progress is removed wrong

Upon restart, the user submits a load job. After completing the load job, the file is removed wrong.
The phenomenon is compaction/upstart/load/gc four procedures. The four procedures will result as the file being removed wrong.