Shiny Apps from GOV 1005

The purpose of this issue thread is to provide a location for students in GOV 1005 to publicly share their work. Thanks to Amanda Cox for permission to try this experiment.

https://github.com/gbwalker/problem-set-7
https://gbwalker.shinyapps.io/upshot/
Gabe Walker

https://github.com/MirandaLupion/PS_7
https://mlupion.shinyapps.io/PS_7/
Miranda Lupion

https://jack-schroeder.shinyapps.io/ps_7/
https://github.com/jackmschroeder/problem-set-7-jackmschroeder-olivercordeiro
Jack Schroeder and Oliver Cordeiro

Meaghan Townsend, Claire Hotchkin

https://meaghant.shinyapps.io/ps_7/
https://github.com/meaghant/ps_7
Names of team members: Meaghan Townsend, Claire Hotchkin

https://meaghant.shinyapps.io/ps_7/
https://github.com/meaghant/ps_7
Meaghan Townsend, Claire Hotchkin

https://github.com/kbinney/gov1005-pset-7-binney-rokos
https://teresarokos.shinyapps.io/poll_diff/
Katherine Binney and Teresa Rokos

https://maddie-chai.shinyapps.io/pset7/
https://github.com/mchai946/problem-set-7-
Maddie Chai & Allie Russell 🥇

https://mattiharrison1.shinyapps.io/ps_7/
https://github.com/mattiharrison/ps_7_starter
Matti Harrison, Holly Christensen, Justin Hunter

https://junhomoon.shinyapps.io/2018_midterm_2/
https://github.com/dodomoon/pset7
Junho Moon

https://charlieolmert.shinyapps.io/ps_7/
https://github.com/CharlieOlmert/olmert_ps_7
Charlie Olmert

https://github.com/bobbyd9/problem-set-7
https://bobbyd9.shinyapps.io/ps_7/
Bobby Drysdale

https://github.com/macfields/mfields_ps7
https://maclainefields.shinyapps.io/ps_7/
Maclaine  Fields

https://github.com/kakenzua98/ps_7
https://kemi-a.shinyapps.io/ps_7_app/

https://mikecalabro97.shinyapps.io/pset7_app/
https://github.com/mikecalabro97/ps_7
Michael Calabro

https://noahreimers.shinyapps.io/ps_7_reimers_pirrmann_ryan/
https://github.com/noahreimers/ps_7
Noah Reimers, Richie Ryan, John Pirrmann

Github repository: https://github.com/jtopoleski1/ps_7
Shiny app: https://kodiobika.shinyapps.io/ps_7/
Team members: Jordan Topoleski, Richard Qiu, Robert Mckenzie, Kodi Obika

https://hahe211.shinyapps.io/pset7_Hannah/
https://github.com/Hahe211/pset7_Hannah
Hannah Hess

https://kanaparti.shinyapps.io/pset_7/
https://github.com/kreola/ps_7_starter
Shriank K

https://github.com/tauheed7/ps_7_starter
https://tauheed-islam.shinyapps.io/ps_7_app/
Tauheed Islam

https://github.com/tanyarohatgi/gov1005_ps_7
https://tanyarohatgi.shinyapps.io/polling_errors_2018/
Tanya Rohatgi

Github repo: https://github.com/FloodZilla2/Problem-Set-7
Shiny app link: https://cflood.shinyapps.io/PS_7/
Team Members: Charles Flood, Sydney Steel Alex LaPolice

https://github.com/seanhugh/ps_7
https://seanhugh.shinyapps.io/ps_7shiny/
Rana Bansal, Sean Hughes

https://github.com/SelinaDing/ps_7_starter
https://selinading.shinyapps.io/Where_Comes_The_Error/
Cunhonghu Ding

https://github.com/SerhiyS1/ps_7_starter
https://serhiy23.shinyapps.io/PS_7/
Serhiy Sokhan

https://github.com/stevene97/ps_7
https://stevene97.shinyapps.io/ps-7/
Steven Espinoza

https://github.com/sarakvaska/ps_7_starter
https://sara-kvaska.shinyapps.io/ps_7/
Sara Kvaska

https://github.com/annikaengstrom/problem_set_7
https://annikaengstrom.shinyapps.io/problem_set_7/
Annika Engstrom

https://github.com/ben-hb/san-francisco-gunshots
https://benhb.shinyapps.io/Gunshots/


[WIP][Triton-MLIR][BACKEND] Add printf in LLVM backend

Nice, it's actually one of my goals in the next quarter. Since you've already started, it relieves a lot burden from my side.
I wonder if it also makes sense to export the printf function to the frontend?

Nice, it's actually one of my goals in the next quarter. Since you've already started, it relieves a lot burden from my side.
I wonder if it also makes sense to export the printf function to the frontend?

Yeah, that sounds good.
Debugging in triton seems still annoying, for the execution is a black box. Might be a solution, but I think some further design or consideration in Frontend is still needed when exposing, for the behavior is quite different from a python print in heterogeneous scenarios.

How about exposing the assert op to the frontend with cf.assert in the backend?
https://mlir.llvm.org/docs/Dialects/ControlFlowDialect/#cfassert-mlircfassertop

How about exposing the assert op to the frontend with cf.assert in the backend?
https://mlir.llvm.org/docs/Dialects/ControlFlowDialect/#cfassert-mlircfassertop

Yeah, I think the assertion is necessary.
I wonder apart from printf, assert, are there any other best practices for debugging or correctness assurance that could be ported to triton? Here's my guess, a way I used in CUDA dev before is to dump tensors to global memory and use python scripts to further debugging.

Here's my guess, a way I used in CUDA dev before is to dump tensors to global memory and use python scripts to further debugging.

Yeah, I did the same thing. And probably that's it, apart from dumping to global memory, using assert, printing out results, cannot think of additional approaches. You may want to use cuda gdb in corner cases.

Yeah, I did the same thing. And probably that's it, apart from dumping to global memory, using assert, printing out results, cannot think of additional approaches. You may want to use cuda gdb in corner cases.

I agree. These solutions should be a good start.

I think Triton opens the door to much more convenient forms of debugging in the future. At some point in the future we'll have a CPU backend, and this will make debugging kernels much much easier.

Amazing initiative btw -- printf has been requested for a long time :)

Cool, thanks for the prompt actions and I think @qiu93 will be more than happy to see this:)

Continued in https://github.com/openai/triton/pull/825


Fixes the Waypoint: Use Conditional Logic with If Statements with clearer instructions.

Closes #5938
Modified the instructions text to:
Create an if statement inside the function to return "Yes" if testMe is greater than 5 and return "No" otherwise.
as discussed in the issue thread.
Tested locally.

:+1:


280 andrey depth limitations hps

Changed floating point arithmetics to fixed point arithmetics in both: ScaleAndRound function used in multiplication and ScaleAndRound function used in decryption.
Questions:

Do we need to add tests for HPS and other variants? If so the test could be slow, as they require lot's of levels for BFV.
I keep temporary bfv-mult-bug.cpp example from https://github.com/openfheorg/openfhe-development/tree/Erabelli_280-bfv-mult-bug. Do we need some variant of it?
In case of decryption should we use previous method with floating point arithmetics, or move to new with fixed point arithmetics?
Should we rename functions in dcrtPoly, as they could be confusing? Make renamings as a part of another issue?

Do we need to add tests for HPS and other variants? If so the test could be slow, as they require lot's of levels for BFV.

I would add some tests using a small ring dimension, e.g., 16.

I keep temporary bfv-mult-bug.cpp example from https://github.com/openfheorg/openfhe-development/tree/Erabelli_280-bfv-mult-bug. Do we need some variant of it?

I think we want to create a unit test based on it

In case of decryption should we use previous method with floating point arithmetics, or move to new with fixed point arithmetics?

Let us set up a conference call and discuss this in more detail.

Should we rename functions in dcrtPoly, as they could be confusing? Make renamings as a part of another issue?

Better do it as part of another issue

We had a meeting on this. Some code/approaches from this PR will be used to address https://github.com/openfheorg/openfhe-development/issues/280 and https://github.com/openfheorg/openfhe-development/issues/421


Update everything in the build to latest.

AGP 3.0b5, tools, sdk, support libraries.


Rember extension feature intrusive

Rembering an extension should only be presented for unknown extensions, eg. not already registered for an existing language.
It avoids silly things like associating .hx to text syntax when creating a temporary new document and changing its syntax.

How do you change the default syntax for an extension if it has already been added to a syntax then?

Right, it needs something smarter...
Maybe we could have a double menu:
Syntax
+   Change default syntax to >
     +   ActionScript
     +   Haxe
     +   ...
     +   Text
+   ActionScript
+   Haxe
+    ...
+    Text

Yeah, that's what sublime text does, and I really like that UI (mentioned it in #627).

Yeah that would be better. @meychi?

Seems good to me but this should be done for 5.1+.


Fixes tutorial 9 bullet list rendering

Summary
The sentence was not displayed correctly with the bullet sign.
Fixes #153
Details and comments
I have fixed the bullet sign error that was not correctly displayed on the page.

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.514%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
694

Relevant Lines:
831

💛  - Coveralls

@ahmadjubair33 Is this something you still wish to do? Its been a while since the original submission.

Other documentation fixes were needed for tutorials, see #157,  and as there has no been no activity since this was created over 2 1/2 weeks ago and no response to to the above query a week ago on progressing this to conclusion, a fix for the bullet lists is also included in that so I am closing this.


Rename VMAX driver folder to 'dellemc'

Rename VMAX driver folder to 'dellemc' from 'dellemc_storage'
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Is it better to name as dell_emc ?

Renamed to dell_emc

lgtm


Example on using pollInterval query

Is your feature request related to a problem? Please describe.
I'm using graphql: ^4.0.1 package.  Using client.query with FutureBuilder.
I'm tried to set PollInterval but I don't know how does it works. Does it return a Stream so I can use a stream builder to subscribe to it. Currently it returns only Future.
Describe the solution you'd like
Example How to use PollInterval in graphql
Describe alternatives you've considered
Alternative seems to move to graphql_flutter package and use Query Widget. I'm not sure it get updated on the Poll interval
Additional context

pollInterval only does something with client.watchQuery and Query from graphql_flutter – #841. It tells the query manager to refetch from the server on a regular basis and add to the ObservableQuery.stream

Pool interval fails with testing since there timers remaining. how can i solve this.

Here's an example from our app:

    final result = graphqlClient.watchQuery$GetStations(
      WatchOptions$Query$GetStations(
          fetchPolicy: FetchPolicy.cacheAndNetwork,
          errorPolicy: ErrorPolicy.all,
          cacheRereadPolicy: CacheRereadPolicy.ignoreAll,
          pollInterval: const Duration(seconds: 5),
          fetchResults: true,
        ),
    );

    result.stream.listen((event) {
      developer.log("Done fetching stations");
      final parsedData = event.parsedData;
      notifyListeners();
    });

where watchQuery$GetStations was generated using https://pub.dev/packages/graphql_codegen


Create "chromium-for-testing" default user configuration folder on Linux; note to use --user-data-dir

I already had Chromium Version 116.0.5825.0 (Developer Build) (64-bit) from https://download-chromium.appspot.com/dl/Linux_x64?type=snapshots (https://github.com/beaufortfrancois/download-chromium/) running. Downloaded Google Chrome for Testing Version 116.0.5817.0 (Official Build) (64-bit) and launched.
Since this is built using Chromium, a new instance of Chromium was launched, using the same default configuration folder on Linux, Xubuntu distribution, ~/.config/chromium.
I think you need to make sure this is creating a "chromium-for-testing" default user configuration folder, just like if we install Chrome proper the first time we launch the folder "google-chrome-unstable" is created in ~/.config, and so forth for each version.
In the mean time it would be helpful to write conspicuous notices in your blog posts, et al. that the user should launch with --user-data-dir flag to avoid the case where launching Chrome for Testing executable will just launch a new Tab in an open Chromium instance when --user-data-dir flag is not set. I'm rather sure that is not what you are trying to do here.

I appreciate this report, but it’s not an issue with the code in this repository (which concerns the dashboard and JSON API only). Please file it here instead: https://goo.gle/cftbug

Update: I’ve filed it there now: https://bugs.chromium.org/p/chromium/issues/detail?id=1454365


DataGrid: Add missing labels for filter value fields in data grid column filter (#9319)

This also fixes the alignment issue of fields for these types.

Description
Fixes #9319
It was found that labels for Value fields of types Enums, DateTime and bool were missing. This caused the fields to mis-align in the data grid filter popover for these datatypes.
The labels were added to fix this issue.
Before:

After:

How Has This Been Tested?
visually tested

Type of Changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[ ] I've added relevant tests.

Good catch. It's caused by the missing label from #8540. For consistency with the others I believe the placeholder should additionally be "Filter value":
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Added placeholders for consistency

Noticed that placeholders were also missing for these datatypes in else block of the code (which handles display for ColumnFilterRow mode). Have added those as well for consistency.

Thanks


[RFC: ListView] Default rowHasChanged for DataSource, initial rows value via constructor

Similar to how we provided a default prop differ with requireNativeComponent it would clean up code a bit and reduce the cognitive overhead of people just getting into React Native to provide a default rowHasChanged function that defaults to a reference equality check.
The current minimal example from the docs:
getInitialState: function() {
  var ds = new ListView.DataSource({rowHasChanged: (r1, r2) => r1 !== r2});
  return {
    dataSource: ds.cloneWithRows(['row 1', 'row 2']),
  };
},

render: function() {
  return (
    <ListView
      dataSource={this.state.dataSource}
      renderRow={(rowData) => <Text>{rowData}</Text>}
    />
  );
},

This would become
getInitialState: function() {
  var ds = new ListView.DataSource();
  return {
    dataSource: ds.cloneWithRows(['row 1', 'row 2']),
  };
},

render: function() {
  return (
    <ListView
      dataSource={this.state.dataSource}
      renderRow={(rowData) => <Text>{rowData}</Text>}
    />
  );
},

Additionally, we could clean up the constructor so that we can pass in some initial data rather than having to instantiate the DataSource and then call cloneWithRows on it:
getInitialState: function() {
  return {
    dataSource: new ListView.DataSource({rows: ['row 1', 'row 2']});
  };
},

render: function() {
  return (
    <ListView
      dataSource={this.state.dataSource}
      renderRow={(rowData) => <Text>{rowData}</Text>}
    />
  );
},

If this looks reasonable, I can go ahead and make either or both of these changes and update docs accordingly.

I'm definitely a fan of your second suggestion for initial data - for the comparison function, the thinking was that we wanted people to make sure they thought about the perf instead of just doing things blindly, but I don't think that's really working out since people just copy-pasta anyway, and it's usually not even that big a perf opportunity to customize that function anyway, so I saw go for both.  Closing this assuming you'll put up a PR :)

I like the initial data constructor too. For the differ, I bet (r1, r2) => true will work better for most people as a default. It matches React's default for shouldComponentUpdate and is what you want when working with mutable data objects.

:guitar: :rocket: thanks for the feedback! @ide - excellent point, will try that as the default


Support io.js

It would be great if the plugin could support IO.js as well as NodeJS.

Will look at this. Have not tested IO.js yet, but it looks interesting.

Maybe this is irrelevant now. It looks like io.js is rejoining the Node Foundation and converging with NodeJS: https://medium.com/node-js-javascript/io-js-week-of-may-15th-9ada45bd8a28

Closing this since NodeJS and IO.js will merge.


Repo libs-releases-local outdated, not containing artifacts after 6.1.1

Looks the last updated date for https://repo.gradle.org/gradle/libs-releases-local/org/gradle/ is 24-Jan-2020 for 6.1.1 release. After that, nothing has been pushed there. Plugin developers depending on APIs of Gradle subprojects (e.g., protobuf-gradle-plugin using kotlin-dsl's extension APIs) are not able to upgrade.
There is an API change for kotlin-dsl made in 6.8, NamedDomainObjectContainer<T>.invoke(kotlin.Function1). But protobuf-gradle-plugin is not able to migrate away from it, as no artifact containing the changed API definition can be found.
(#15508 reverted the API deletion, but I guess Gradle is still suggesting to migrate.)
Expected Behavior

Be able to find the latest version of Gradle subproject artifacts (e.g., org.gradle:gradle-kotlin-dsl:6.8.3).
Current Behavior

Third-party plugins are not able to migrate usages of APIs from Gradle subprojects.
Context

NA
Steps to Reproduce

NA
Your Environment

NA
Build scan URL:

Gradle API should be consumed by adding the embedded dependency to gradleApi() in dependency declaration.
There is no immediate plan to publish these binaries as they expose way more than the public API.
Can you be more specific as why there is a use case where gradleApi() is not enough?

Specifically, the protobuf-gradle-plugin has an API for users to configure NamedDomainObjectContainer, something like
protobuf {
   plugins {  // <== NamedDomainObjectContainer<PluginOptions>
      id("grpc") {
        artifact = "io.grpc:protoc-gen-grpc-java:1.37.0"
      }
   }
}

In Groovy, this is just delegated by ConfigureUtil.configure(Closure, T). In Kotlin DSL, it uses <T : Any, C : NamedDomainObjectContainer<T>> C.invoke(configuration: NamedDomainObjectContainerScope<T>.() -> Unit).
However, Gradle is changing the Kotlin DSL API to <T : Any, C : NamedDomainObjectContainer<T>> C.invoke(configuration: Action<NamedDomainObjectContainerScope<T>>) as in https://github.com/gradle/gradle/commit/593001e74f21a67ecc8d6f6c6174818f02eab007. Although #15508 added the T.() -> Unit overloading back, I guess in the long term Gradle is still suggesting plugins to migrate.
So, protobuf-gradle-plugin would need compile-time access to the <T : Any, C : NamedDomainObjectContainer<T>> C.invoke(configuration: Action<NamedDomainObjectContainerScope<T>>) API, which is provided by org.gradle:gradle-kotlin-dsl.  gradleApi() doesn't seem to provide this.

This issue looks like Gradle 8.0 will break all builds which use protobuf-gradle-plugin but does not provide any means for plugin authors to migrate to the recommended API. I assume that "do not upgrade to Gradle 8" will be the official answer of the Gradle team? Or is there a way for a community to resolve this issue? Either here or in protobuf-gradle-plugin repo?

The specific issues around protobuf-gradle-plugin have been addressed by https://github.com/google/protobuf-gradle-plugin/pull/577 (a part of v0.9.0).
The "official" Gradle API is the one produced by gradleApi() and not the published libraries on repo.gradle.org.


Page.WaitForRequestAsync and Page.WaitForResponseAsync [DO-NOT-MERGE]

Closes #497

Done @Meir017


Looking for docker-compose complete example w/ rev. proxy

What I've got isn't hooking in to your compose file; could you provide a complete example using compose?
`version: "2"
services:
nginx:
restart: always
image: nginx
container_name: nginx
ports:
- "80:80"
- "443:443"
volumes:
- "/etc/nginx/conf.d"
- "/etc/nginx/vhost.d"
- "/usr/share/nginx/html"
- "./volumes/proxy/certs:/etc/nginx/certs:ro"
nginx-gen:
restart: always
image: jwilder/docker-gen
container_name: nginx-gen
volumes:
- "/var/run/docker.sock:/tmp/docker.sock:ro"
- "./volumes/proxy/templates/nginx.tmpl:/etc/docker-gen/templates/nginx.tmpl:ro"
volumes_from:
- nginx
entrypoint: /usr/local/bin/docker-gen -notify-sighup nginx -watch -wait 5s:30s /etc/docker-gen/templates/nginx.tmpl /etc/nginx/conf.d/default.conf
letsencrypt-nginx-proxy-companion:
restart: always
image: jrcs/letsencrypt-nginx-proxy-companion
container_name: letsencrypt-nginx-proxy-companion
volumes_from:
- nginx
volumes:
- "/var/run/docker.sock:/var/run/docker.sock:ro"
- "./volumes/proxy/certs:/etc/nginx/certs:rw"
environment:
- NGINX_DOCKER_GEN_CONTAINER=nginx-gen
networks:
proxy-tier:
external:
name: nginx-proxy`

Here's what we use with great success.
version: '2.1'

services:
  redacted-nginx-proxy:
    image: nginx-proxy:938d638e5c4d0dd98105bb15838cb5481de70a80
    container_name: redacted-nginx-proxy
    ports:
      - 80:80
      - 443:443
    volumes:
      - ./conf/vhost.d:/etc/nginx/vhost.d
      - ./conf/html:/usr/share/nginx/html
      - ./conf/certs:/etc/nginx/certs:ro
      - ./logs:/var/log/nginx
      - /var/run/docker.sock:/tmp/docker.sock:ro
    environment:
      - ZABBIX_HOSTNAME=redacted-nginx-proxy
    networks:
      - proxy-tier
    restart: always
    labels:
      - "com.github.jrcs.letsencrypt_nginx_proxy_companion.nginx_proxy=true"

  redacted-letsencrypt-companion:
    image: jrcs/letsencrypt-nginx-proxy-companion
    container_name: redacted-letsencrypt-companion
    volumes_from:
      - redacted-nginx-proxy
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock:ro
      - ./conf/certs:/etc/nginx/certs:rw
    restart: always

networks:
  proxy-tier:
    external:
      name: nginx-proxy

Thanks. And just before I pull the trigger, does this make sense
version: '2'

services:
  freepbx-app:
    container_name: freepbx-app
    image: tiredofit/freepbx
    ports:
     #### If you aren't using a reverse proxy
      - 80:80
     #### If you want SSL Support and not using a reverse proxy
     #- 443:443
      - 5060:5060
      - 5160:5160
      - 18000-18100:18000-18100/udp
    volumes:
      - ./certs:/certs
      - ./data:/data
      - ./logs:/var/log
      - ./data/www:/var/www/html
     ### Only Enable this option below if you set DB_EMBEDDED=TRUE
     #- ./db:/var/lib/mysql

    environment:
      - VIRTUAL_HOST=agent.solvent.ca
      - VIRTUAL_NETWORK=nginx-proxy
     ### If you want to connect to the SSL Enabled Container
     #- VIRTUAL_PORT=443
     #- VIRTUAL_PROTO=https
      - VIRTUAL_PORT=80
      - LETSENCRYPT_HOST=agent.solvent.ca
      -<EMAIL_ADDRESS>
      - ZABBIX_HOSTNAME=freepbx-app

      - RTP_START=18000
      - RTP_FINISH=18100

     ## Use for External MySQL Server
     #- DB_EMBEDDED=FALSE

     ### These are only necessary if DB_EMBEDDED=FALSE
      #- DB_HOST=freepbx-db
      #- DB_PORT=3306
      #- DB_NAME=asterisk
      #- DB_USER=asterisk
      #- DB_PASS=asteriskpass

     ### If you are using TLS Support for Apache to listen on 443 in the container drop them in /certs and set these:
     #- TLS_CERT=cert.pem
     #- TLS_KEY=key.pem

    restart: always
    networks:
      - proxy-tier
    ### These final lines are for Fail2ban. If you don't want, comment and also add ENABLE_FAIL2BAN=FALSE to your environment
    cap_add:
      - NET_ADMIN
    privileged: true

  freepbx-db:
    container_name: freepbx-db
    image: tiredofit/mariadb
    restart: always
    volumes:
      - ./db:/var/lib/mysql
    environment:
      - MYSQL_ROOT_PASSWORD=password
      - MYSQL_DATABASE=asterisk
      - MYSQL_USER=asterisk
      - MYSQL_PASSWORD=asteriskpass
    networks:
      - proxy-tier

  freepbx-db-backup:
    container_name: freepbx-db-backup
    image: tiredofit/db-backup
    links:
     - freepbx-db
    volumes:
      - ./dbbackup:/backup
    environment:
      - ZABBIX_HOSTNAME=freepbx-db-backup
      - DB_HOST=freepbx-db
      - DB_TYPE=mariadb
      - DB_NAME=asterisk
      - DB_USER=asterisk
      - DB_PASS=asteriskpass
      - DB_DUMP_FREQ=1440
      - DB_DUMP_BEGIN=0000
      - DB_CLEANUP_TIME=8640
      - COMPRESSION=BZ
      - MD5=TRUE
    networks:
      - proxy-tier
    restart: always

networks:
  proxy-tier:
    external:
      name: nginx-proxy

edit: uncommented 80

Looks like you are going to use the embedded DB, so you'll want to uncomment the part where it says:
### Only Enable this option below if you set DB_EMBEDDED=TRUE
     #- ./db:/var/lib/mysql

You'll want to get rid of the containers (freepbx-db and freepbx-db-backup) though. I use the external DB (non embedded) and added it the internal DB feature "because I could", but don't have much experience with using it in this docker image. Personally I like to seperate them.

I see. I guess I should set that up and stick with what works, then. Thanks +1+1

Good luck! We're succesfully using this on a development server (calling working, webrtc phone working, with a planned usage case of over 500 extensions in the next 60 days, so updates to this image will come in bursts as I add more settings. I'm currently on the hunt on how to get LDAP authentication to work for the user manager and plan to have an ability to inject custom themes / tweaks without having to modify the image shortly. Also, I usually build complicated images in Debian first and then switch over to Alpine after the fact. The Alpine image is 80% complete and 1/4 of the size..


"Please run 'make slim' to fix" but target does not exist

What happened:
$ make metal
make --directory=docker build-image
make[1]: Entering directory '/home/myon/gardenlinux/docker'
WARNING: There is no gardenlinux/slim on this box. Since this is the builder script, it will pull no binary docker images from public repos.

Since gardenlinux/slim is needed for almost all images we pull debian/testing-slim and rename it to gardenlinux/slim so we avoid the Chicken-and-Egg problem temporarily

Please run 'make slim' afterwards
...

$ make slim
make: *** No rule to make target 'slim'.  Stop.

Environment:

Debian bullseye, gardenlinux aa23f546f58

Deprecated by #1390 #1391 which will provide the container image within public registries and provides gardenlinux-slim (even within the internal build pipeline).


Improve install scripts

[ ] Detect PowerShell Constrained Language Mode
[ ] Improve OS detection
[ ] Move language mode and OS detection to run first and return remark to use manual install instructions
[ ] Improve visibility of dot sourcing

Users who use corporate laptops may have PowerShell Constrained Language Mode enabled which will stop the install script from executing correctly and closes the terminal window leaving the users unable to continue. In this scenario, we should detect if the mode is enabled and provide a friendly message to direct users to use the manual install instructions via a short link that targets to https://learn.microsoft.com/microsoft-cloud/dev/dev-proxy/get-started?tabs=manual&pivots=client-operating-system-windows
Error message: Method invocation is supported only on core types in this language mode

We can have a if statement wrapping our install script ?
if (!$ExecutionContext.SessionState.LanguageMode -eq "ConstrainedLanguage")

Obsolete in favor of #11


Add OBO credential type to Azure Identity README

Related to https://github.com/Azure/azure-sdk-for-net/pull/29096
Summary of changes

Add missing OnBehalfOfCredential type to the README's "Credential Classes" section.
Alphabetize the credential types in the tables, for easier scanning.
Link credential types to their respective API ref pages.

API change check
API changes are not detected in this pull request.


panicked at 'unknown name section chunk type: 7'

At the current version I get "thread 'main' panicked at 'unknown name section chunk type: 7', C:\Users\score.cargo\registry\src\github.com-1ecc6299db9ec823\cargo-web-0.6.26\src\wasm_context.rs:642:41" when building using
 cargo screeps build
Is there a solution to this?

Which version of rust are you using? I think beyond something like 1.47 cargo web is busted. There's an effort to convert to bindgen but in the meantime you may just want to pin your rust version for the project.

yeah that solved it, ty


Calls do not return android sound channel back

This bug is polluting all my devices.
Say before call I was listening to music. When I change volume while playing music, the volume changer has a little note icon like the one in the middle:

It corresponds to "multimedia" channel. When a call goes through the music will stop.
During the call the volume chager has another icon. It's like an old phone. and it changes volume ony during this call.
Normally, when a call ends music will start again and so is the volume changer, but it isn't the case with riot.im. The sound goes back to normal if riot.im app is killed from task manager.
Also, if I use a bluetooth headset, it doesn't work either. The app uses phone's speakers an mic.
I have android 5.0 on my phone and 5.1.1 on tablet.
May be related to #1709 and #1699.

@ylecollen
I sure can. Which branch should I use? Latest master?

I tried building latest master. Both debug and release fail with:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':vector:processAppfdroidDebugGoogleServices'.
> File google-services.json is missing. The Google Services Plugin cannot function without it.
   Searched Location:
  /home/user/projects/riot-android/vector/src/appfdroid/debug/google-services.json
  /home/user/projects/riot-android/vector/src/debug/appfdroid/google-services.json
  /home/user/projects/riot-android/vector/src/appfdroid/google-services.json
  /home/user/projects/riot-android/vector/src/debug/google-services.json
  /home/user/projects/riot-android/vector/google-services.json

This problem persists in last Play Market release (0.7.03).
Also sometimes in event of two consecutive missed calls the gui for answering a call wont pop up.


Is there anyway to get a MAPBOX_TOKEN for pc without a comma device?

Describe the bug
When I debug the map_renderer.py. I got the following error message:
[ INFO ]  "{unknown}[General]: GPU Identifier: NVIDIA GeForce RTX 2080 Ti/PCIe/SSE2"
[ ERROR ]  "{unknown}[Setup]: loading style failed: HTTP status code 403"

However, when I use "https://api.commadotai.com/#comma-api-spec"  to get a jwt token.
export MAPBOX_TOKEN=eyJ0eXxxxxxxxxxxxxxxxxxxxxxx

I got the 401 error.
[ INFO ]  "{unknown}[General]: GPU Identifier: NVIDIA GeForce RTX 2080 Ti/PCIe/SSE2"
[ ERROR ]  "{unknown}[Setup]: loading style failed: HTTP status code 401"

Then, If I use my own MAPBOX token from "https://account.mapbox.com/". I got the following error.
[ INFO ]  "{unknown}[General]: GPU Identifier: NVIDIA GeForce RTX 2080 Ti/PCIe/SSE2"
[ ERROR ]  "{unknown}[General]: The resource `https://api.mapbox.com/styles/v1/commaai/ckvmksrpd4n0a14pfdo5heqzr` not found"
[ ERROR ]  "{unknown}[Setup]: loading style failed: HTTP status code 404"

It seems that the error comes from the MAPBOX_TOKEN issued from comma.ai. Is there anyway to debug/test the function without buying a comma three device?
OS Version
Ubuntu 20.04
openpilot version or commit
No response
Additional info
No response

The token is from Mapbox, not us.

No. The problem is not the MAPBOX-token.  I cann't access the style files of comma. How do I deal with it?
See my previous comments.
[ INFO ]  "{unknown}[General]: GPU Identifier: NVIDIA GeForce RTX 2080 Ti/PCIe/SSE2"
[ ERROR ]  "{unknown}[General]: The resource `https://api.mapbox.com/styles/v1/commaai/ckvmksrpd4n0a14pfdo5heqzr` not found"
[ ERROR ]  "{unknown}[Setup]: loading style failed: HTTP status code 404"


"Center" documentation tweak on web site

On the web page at http://flutter.io/widgets/, it says:
Center Center the child widget within the space occupied by this widget.
I propose that this text be updated to note that it's centered both horizontally and vertically. This was definitely not intuitively obvious to me.

Done.


is_connected_to_host  should only work if connection is actually connected to a real tcp server

3.1, w10, 64-bit
i don't know if this is the same on linux (if sum1 can plz test, please do!, thanks!)
extends Control

var socket = StreamPeerTCP.new()

func _ready():
	if socket.connect_to_host("<IP_ADDRESS>", 9305) == OK:
		print("Connected!") # not connected though

var c = 0
func _process(delta):
	if c > 0.1:
		print(socket.is_connected_to_host())
		c = 0
	c += delta

for example, i'm using a fake ip and fake port (there is no tcp server running) but  the is_connected_to_host still prints true.

Duplicate of #18323 see https://github.com/godotengine/godot/issues/18323#issuecomment-383325914 specifically.

is_connected_to_host returns true also when the status is connecting, if you want to know if the status is connected use get_status() == StreamPeerTCP.STATUS_CONNECTED. As said, it could be renamed to is_active() for clarity, but documenting it should be enough.

If we want to change that behaviour it's an API breaking change, and should be done after 3.1.
connect_to_host returns OK if there was no network error. The fact that there is no listening server on the other end is irrelevant most of the times, unless a TCP packet to that destination generates an ICMP packet in return (e.g. the destination is unreachable during connection, or the remote server has a REJECT policy that sends ICMP Destination Unreachable: RFC-792 when that port is closed).
I think we should at least better explain to use get_status() and not is_connected_to_host() in the docs.

We might address this in 4.0 when/if we decide to refactor low level networking with breaking changes.

@Faless confirmed on chat that this is still pending and would be good to change.


Having a done keyword, to auto complete an added task

So now that notes have been added, I have started to use the todolist to log things I have done and involves me doing:
t a fixed x server
t c [id]

It would be nice to have this wrapped into one command:
t done did some work on x server

so under the hood it would auto complete the item

Oh yes, I like it.

I think Ill take a crack at this later.

So now this is not only a todolist software, but can also be a work journal.  Good job!

thats kinda my use case for it atm.


add calls for PrimalRanging and DualRanging

this has a lot of data copying.  i was considering adding [][]float64 and [][]int versions of the helper functions, and maybe using these helpers more throughout the package.  not sure thoughts on this -- most of the existing functions copy 1 or 2 slices. this has need of up to 7 now...

this has a lot of data copying. i was considering adding [][]float64 and [][]int versions of the helper functions, and maybe using these helpers more throughout the package. not sure thoughts on this -- most of the existing functions copy 1 or 2 slices. this has need of up to 7 now...

In your code, the []float64 slices can be used directly (no copy in/out) as follows:
status := C.primal_ranging(s.model, C.int(n), (*C.int)(cWhich),
	(*C.double)(&valueIncrease[0]), (*C.int)(cseqInc),
	(*C.double)(&valueDecrease[0]), (*C.int)(cseqDec))

The []int slices require a copy in/out because a Go int is commonly a different size from a C int (64 vs. 32 bits).  However, the allocation can be performed in Go, which will enable the data to be garbage-collected.

Done.  7958c4324310e216c765e2b858b1e12904fffea6 eliminates copying of the []float64 slices and uses pure Go for the []int slices.
Thanks for the pull request, @jonreiter.


Right to left languages

I remember requests for this showing up in the sublime text forums every now and then. The backend makes use of Go's unicode rune and string features, but I don't know if this is enough for right to left languages or if there's more to it than that.
Does it work already in the termbox and qml frontends? Is it just displayed left to right, or is it stored like that also? Does the Region's A and B values make sense? How about the backspace key being bound to the "left_delete" command, does it work as expected with right to left languages?
@zoli, am I correct in that you can speak and write Farsi? Have you seen anything that is obviously broken with regards to supporting Farsi?
Not critical to solve for 0.1, but I would like to have it in 1.0 and if there are issues it's good to start discussing so that we have plenty of time to think of solutions :)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/2548334-right-to-left-languages?utm_campaign=plugin&utm_content=tracker%2F282001&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282001&utm_medium=issues&utm_source=github).

Movet to backend https://github.com/limetext/backend/issues/110.


AJAX GET errors for missing sound files.

Hi.
In dev console an error is thrown for each missing extension whenever I play a file.
// include 2 formats only.
A.mp3
A.wav

GET file:///path/to/A.ogg?1444081896506 net::ERR_FILE_NOT_FOUND
GET file:///path/to/A.mp4?1444081896508 net::ERR_FILE_NOT_FOUND
GET file:///path/to/A.aac?1444081896510 net::ERR_FILE_NOT_FOUND
 - p.load    ion.sound.min.js:7
 - p.reload    ion.sound.min.js:8
 - p.error    ion.sound.min.js:10

Isn't this essentially forcing users to include all file formats of each sound? The documentation implies adding more versions is recommended but optional.
Is there a way to suppress these errors/skip checks? I would prefer to skip unnecessary checks/errors and including files when I do not need wide-browser support.

Hello, i think i will add this option in future, to predefine extension.


Feature compare nested

Added features:

Added Equals func for comparing two Nested objects and unit-tests for it
Update readme

Added features:

Added Equals func for comparing two Nested objects and unit-tests for it
Update readme


Support to use RPC to get object and improve the resolver and builder of python

What do these changes do?
With it, rpc_client and ipc_client can get/put the value as the same way.

Don't touch the submodule, if you are not actually changing it...


Minor: Pass container target with quote-selection event

In testing https://github.com/github/github/pull/151548 I remembered that this event exists 🤦🏻♀ .
However, since this event is fired on the parent container, it'd benefit the listeners to send the target quote region as well. The quote region is found through the scopeSelector. Listeners shouldn't have to re-query the region and risk getting a different target than the one quote-selection found.
We use this container to determine which textarea the quote should go to.

me 12 hours ago:

this event is fired on the parent container,

This is not true. 🤦🏻♀ The event is fired on container already.


Is it possible not to send the report?

Sending a long string of characters every day will affect normal alarms？

The configuration file is commented

as show in your second snapshot, you can set schedule to none to disable the report.

as show in your second snapshot, you can set schedule to none to disable the report.

Already modified to none, no effect

could you please show us your configuration file? the sla configuration should be below settings part

find your configuration content above, the right configuration shoule be
settings:
  sla:
    schedule: none

Can't it?

I deleted all the sla for a period of time, and I saw the log after restarting, or will it be sent? Why?

I deleted all the sla for a period of time, and I saw the log after restarting, or will it be sent? Why?我删除了一段时间的所有sla，重新启动后看到了日志，还是会发送？为什么？

Jul 12 09:50:45 ansible easeprobe[160456]: time="2023-07-12T09:50:45+08:00" level=info msg="Scheduling daily SLA reports at 00:00 UTC time..." Jul 12 09:50:45 ansible easeprobe[160456]: time="2023-07-12T09:50:45+08:00" level=info msg="The SLA report will be schedule at 2023-07-13 00:00:00 UTC"

find your configuration content above, the right configuration should be
settings:
  sla:
    schedule: none

pay attention to the indentation in YAML configuration file


Convert old sruveyValues to the new sampleQuestions

This is much easier to do in PG, so now there is no reason to keep two sets of calculations.


Using x86 instructions instead of software emulation

I’ve noticed you use lookup tables and bit tricks in performance-critical parts of the code, for functionality that’s implemented in hardware.
Intel introduces BSR and BSF instructions in 80386, i.e. they support them since 1985.
POPCNT is slightly newer, but it’s reasonable to assume it’s present, introduced in SSE4a by AMD in 2007, supported by Intel since 2008. Most people don't use 12 years old computers.
This pull request implements them for vc++, gcc and clang compilers. For unknown ones it falls back to your original software implementation.
Please let me know if you’re willing to accept the PR, is yes I should probably implement a couple of tests, so far I've only built with vc++ but haven't tested yet. Also you wrote you want me to sigh something.
If you’re not interested, I’ll just keep these changes in my private branch.

Thanks for the PR. This is a change that will likely need discussion among the TSC so we should get this onto an upcoming meeting agenda. Your PR has come at a time of the year when we're otherwise engaged with the Siggraph conference and Summer vacations that has disrupted our regular TSC meeting schedule. Bear with us and we'll get back to you when we can.

We might consider first doing a separate commit to change all the static const tables (everywhere) into static constexpr, removing the #ifdef _MSC_VER since we have since moved on to require C++14, even on Windows.

We might consider first doing a separate commit to change all the static const tables (everywhere) into static constexpr, removing the #ifdef _MSC_VER since we have since moved on to require C++14, even on Windows.

Technically we're still intending to support C++11 until the end of the year as we're currently maintaining compatibility for the last two years worth of the VFX Reference Platform (ie as far back as 2017). That being said, we did briefly discuss at Siggraph about adjusting this policy to deprecate based on when the next major release of VDB is released. I understood that to mean that once we've pushed out ABI 7 + VFX 2020, we'd soon thereafter drop support for ABI 5 + VFX 2017. However, even if we change that, it'll likely still be Q4 before we can start using C++14 features in the core library unfortunately.

Sure, but using static constexpr is probably C++11 and doesn't require C++14.

Thank you again for this PR.  We had a chance to discuss it at the last TSC meeting.
POPCOUNT is a troublesome one as a surprising number of such old CPUs are still around.  2008 is also when CPUs stopped getting faster,  so stopped turning into bricks every two years. (In particular, if I recall properly, because of a strange interaction of sse 4.1, 4.0a, and 4.2, you end up having to set 4.2 as your minimal platform to gain popcount, which causes the compiler to start generating all the 4.2 instructions, thereby disabling the early AMD Phenoms.)  All that said, there were no objections to changing VDB to default to SSE 4.2+ only.
We would very much appreciate, however, if you could run a quick performance test on your favorite CPU/OS to verify that these intrinsics are faster than the equivalent C++.  While a glance at Agner's tables suggests BSR and POPCOUNT aren't some of them, the x86 architecture is lousy with intrinsics that are slower than the equivalent assembly, so we would like to see some wall-clock numbers to justify the additional maintenance of supporting the intrinsics.
As for signing something, there are two things that need to be signed.

Each commit has to be amended with a Signed-off-by: line to meet the requirements of the DCO.  The --signoff command can be used for this.  Unfortunately, it must be in all commits, so I sometimes find it easier to squash and resubmit as a single commit with the right --signoff than try to amend, but you may be more skilled at Git than I.
You will need to sign a Contributor License Agreement.  I think this is still in the air as we haven't turned on the system yet.

@jmlait In VC++, when you use intrinsics like __popcnt(), the compiler just emits the instruction you want. No need to change any project-wide configs. But the code will crash in runtime when not supported, with “invalid instruction” runtime error (happened to me once, years ago, for an SSSE3 instruction not being supported by some old AMD my customer was running).
Gcc and clang are different in that regard, the __builtin_popcount can compile into popcnt if allowed by compiler options, if not allowed the compiler will fall back to bit tricks, probably very similar ones to what you have in the master branch. E.g. -march=native will emit popcnt if building on a CPU newer than 12 years. There’s also a separate option in gcc to enable just the popcnt, -mpopcnt
OK, I’ll try to make a small performance benchmark later this week.
BTW, I have no idea who uses vdb, but I would be very surprised if more than 0.1% of users are using CPUs without hardware popcnt support. To find one, you need something like Core 2 Quad Q6600, or Athlon 64 X2. Even Intel Atoms support it for many years, since Silvermont microarch.

It likely is the GCC case I was thinking of then.  I do recall that to enable using the intrinsics at all you had to enable the compiler flag. And as soon as you did that, the compiler would start using the intrinsics itself in random parts of your code.  Which is frustrating as we are then unable to have runtime switches to enable those intrinsics on certain critical code paths.   And POPCOUNT in particular needs 4.2 because I don't think you can get the correct intersection of 4.0a and 4.1 otherwise.  Coincidentally, just heard from someone unable to move from 16.5 on their laptop because of the 4.2 requirements :>

I happen to have been in a similar part of the code recently and found an opportunity to run a quick test as I was curious. On my particular hardware, I found that the lookup table was actually around 10-20% faster in the typical case where you're doing a lot of repeated lookups (ie once the table is in the cache). That appears to be supported by other benchmarks such as this one - http://dalkescientific.com/writings/diary/archive/2011/11/02/faster_popcount_update.html.
It does look like there may be a faster implementation around if you use SSE intrinsics carefully, but it's not trivial and I wasn't able to find such an implementation in the time I put towards this. There are also some copyright considerations with some of these other methods too.
In summary - we still need some convincing here otherwise I think we'll stick with the current implementation and close out this PR. We do appreciate you submitting this though as this is a highly critical routine that we would be very happy to find a faster alternative for, so well worth investigating.

@danrbailey It’s your project, not mine. Do whatever you see fit. Meanwhile I’m still using my private branch.

On my particular hardware, I found that the lookup table was actually around 10-20% faster in the typical case

I’m pretty sure you have used software fallbacks implemented by gcc/clang. Hardware popcnt can’t possibly be slower than lookup table. Intel says the instruction has 3 cycles latency, and for repeated computations it’s even faster because the throughput is just a single CPU cycle. A lookup table is slower than that by an order of magnitude, the typical latency of L1 data cache is 4 cycles / read, and you need multiple reads, e.g. 4 reads for 32-bit popcnt with 256 values in the table.
The article you’ve linked doesn’t include the hardware implementation I’m using in the PR. Here’s what it says about it:

if you have a chip with the POPCNT instruction built-in then use it.

Also note the PR it’s not just about popcnt. The first/last set bit algorithm has similar performance characteristics of hardware versus software.

Yes, you're right, I was accidentally using the software fallbacks when testing. Thanks for the info and the articles. It is faster, around 40% on my hardware which is very exciting.
Here are some suggestions about how we can tweak your implementation to try and get this into OpenVDB.
Firstly, as @jmlait has already outlined, there may be people out there who are still using VDB with older hardware so at the very least I think this should be exposed through a CMake compile option so it can be toggled on/off, something like (using GCC as an example here):
if (OPENVDB_USE_SSE42)
  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -msse4.2")
  add_definitions(-DOPENVDB_USE_SSE42)
endif()

An additional reason for including this compiler toggle - for most studios, this code will be compiled on a build machine and then used by others on potentially quite different hardware. This rules out checking bit manipulation flags in CPUID feature bits or relying on software fallbacks.
For the sake of getting this PR in, I'd recommend that we default it to off at first, then we discuss whether to switch it on in our TSC meetings by default as a subsequent change. Perhaps we'd even take a very cautious approach by leaving it off for a while, enabling it in a CI build then changing the default at a later date.
I think we should put any logic that includes headers related to intrinsics into openvdb/Platform.h, that feels like the right place for this to go.
Because this is a fairly significant change, I suggest we focus this on just four methods at first:

CountOn(Byte)
CountOn(Index64)
FindLowestOn(Byte)
FindLowestOn(Index64)

These are the ones that will have the biggest impact. The Index32 variants and FindHighestOn() are only used for an old legacy file format which I don't believe we are even testing in our CI.
As a result, I think we should include ifdef branching for these four methods in NodeMasks.h rather than introducing new header files as that will keep all the code in one place. It would be nice, but I don't think it's necessary to include new unit tests as these methods are called fairly directly from NodeMask and are tested pretty extensively there.
@pcucka - please comment on my suggestions above if you think we should go about this differently.
Now on to the question of DCO and CLA. To be able to accept this PR, all of your commits have to be signed using a DCO which Jeff mentioned above:

Each commit has to be amended with a Signed-off-by: line to meet the requirements of the DCO. The --signoff command can be used for this. Unfortunately, it must be in all commits, so I sometimes find it easier to squash and resubmit as a single commit with the right --signoff than try to amend, but you may be more skilled at Git than I.

You can either go back and edit every commit in the chain and sign them, collapse the commits and sign that and then force push to your branch or just re-submit the PR with signed commits.
The next step is the CLA. We now have a system called EasyCLA turned on, any more commits to this PR and you'll get a new GitHub CLA check that will run with a link to allow you to sign a corporate or individual CLA.
Thanks again for the contribution and your patience with explaining the details of this change! :)

Hi @Const-me, we have discussed this PR in TSC meeting and are happy to move forward with introducing some of the specific hardware instructions you're proposing. However, we would like to see this go into the next release of OpenVDB which is coming up in the next few weeks. Would you either have time to prepare this PR for merging and we'll guide you through the process or otherwise would you be happy to hand this off for us to complete (and we'll credit you for the idea in the change notes)?

@danrbailey Don’t have much time lately but I hope it won’t take too long.
As you see I’ve squashed the commits and added “Signed-off-by”.
I’ve signed the CLA, got the e-mail with a PDF which says “Your signed document is attached to this email” but this issue wasn’t updated. Can forward you the PDF if it helps. When I try again, it doesn’t offer me to sign, redirects back to this PR.

@danrbailey Don’t have much time lately but I hope it won’t take too long.
As you see I’ve squashed the commits and added “Signed-off-by”.
I’ve signed the CLA, got the e-mail with a PDF which says “Your signed document is attached to this email” but this issue wasn’t updated. Can forward you the PDF if it helps. When I try again, it doesn’t offer me to sign, redirects back to this PR.

The EasyCLA check doesn't appear to have been updated, perhaps an empty commit will trigger it? (This system is still pretty new).

@danrbailey Nothing changed. The CLA web site has remembered my github account, they know I have already signed and they just redirect me back. New commits don’t change anything because the PDF says “all Contributions to the Project made by You”, it doesn’t list commits nor defines a time interval.

@danrbailey Nothing changed. The CLA web site has remembered my github account, they know I have already signed and they just redirect me back. New commits don’t change anything because the PDF says “all Contributions to the Project made by You”, it doesn’t list commits nor defines a time interval.

I think you're confusing the system because you appear to have two email addresses you're using here:

<EMAIL_ADDRESS>(the DCO email)
<EMAIL_ADDRESS>(the Git author email)

Perhaps only one of these is linked to your GitHub account?
It looks like b2a61d7399cbeb4a657b811721893f6a200b6e48 did pass the EasyCLA check (so you're close!), but when you subsequently force-pushed, it then failed again.
I'd recommend using one email address for your DCO, Git author and GitHub registered emails and trying again? Thanks for your patience.

Looks like everything is working now, great! I've just pushed up a PR that switches on a conditional flag to enable SIMD from our CMake:
https://github.com/AcademySoftwareFoundation/openvdb/pull/557
The intention is to use this to gate your changes by #ifdef OPENVDB_USE_SSE42 to add the popcount intrinsic.

I feel your pain.  I was stuck in that limbo for a couple of weeks before I
got all addresses verified.  I had to link the email used for the DCO with
the github account; that was a bit tricky as its hostname wasn't actually
an email address :>  Fortunately our IT could invent it for the duration.
Thank you for persevering through this!
On Thu, Nov 14, 2019 at 6:44 PM Konstantin<EMAIL_ADDRESS>wrote:

@danrbailey https://github.com/danrbailey Nothing changed. The CLA web
site has remembered my github account, they know I have already signed and
they just redirect me back. New commits don’t change anything because the
PDF says “all Contributions to the Project made by You”, it doesn’t list
commits nor defines a time interval.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AcademySoftwareFoundation/openvdb/pull/493?email_source=notifications&email_token=AALXQL4Q6WMJWMDXONRD32LQTXPG3A5CNFSM4IG7X552YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEDXQSQ#issuecomment-554137674,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALXQL3T5QLACWWGBNXXMF3QTXPG3ANCNFSM4IG7X55Q
.

I've just merged a PR (#557) that adds OPENVDB_SIMD CMake support.
To complete this PR, we'd need you to merge master back, fix the conflicts, add gated conditionals using ifdef OPENVDB_USE_SSE42 and to remove all but the following intrinsic changes:

CountOn(Byte)
CountOn(Index64)
FindLowestOn(Byte)
FindLowestOn(Index64)

We'd like to get this into our upcoming release which is a few weeks out, but as mentioned before I'm happy to help finish this for you if you don't have the time to do so.

Hi @Const-me,
I've just pushed up a PR from my own fork with the changes I was proposing on this PR (#574). Due to the big benefits in performance this represents, we're trying to get this into the 7.0 major release that we'll be pushing out in the next week or so, which means we've had to rather rush to get this ready. I appreciate you may not have had time to look at addressing my feedback yet so apologies for having to take this over to get it ready in time. If you do have a few moments to review my PR (#574) and verify I haven't made any mistakes in pulling your logic across, it would be much appreciated.
I'm going close down this PR now. Thanks again for the contribution, it's exciting to get these changes into OpenVDB! :)


bump jinja2 to v3

Jinja2 depends on Markupsafe but the constraint is a minimum version with no upper bound. Markupsafe recently removed a deprecated function which causes the following error:
ImportError: cannot import name 'soft_unicode' from 'markupsafe'

The problem has been reported by a few Jinja2 users, e.g. https://github.com/pallets/jinja/issues/1585. The Jinja2 dev team recommends upgrading Jinja2 - they refuse to patch the v2 releases with an upper bound to their Markupsafe constraint as it's considered "unsupported".
This PR bumps out Jinja2 requirement to Jinja2 = "^3.0.3". We only use Jinja2 for templating the run config files - I see no breaking changes and the tests passed locally.

@ajstewart I think this is better than adding Markupsafe to our dependencies since we don't directly depend on Markupsafe, Jinja2 does.
If you agree, I suggest merging this into dev then rebasing #627 onto dev, and removing the Markupsafe pin in #627.

@ajstewart I think this is better than adding Markupsafe to our dependencies since we don't directly depend on Markupsafe, Jinja2 does.
If you agree, I suggest merging this into dev then rebasing #627 onto dev, and removing the Markupsafe pin in #627.

Yep makes sense, I'm happy with that.


Support node.js VMs

This is admittedly a hyper-specific use-case. I'm using cargo-web to make a bot for the game Screeps, which uses node.js VMs to run user code. It currently has two modes user code can be running in:

Shared, which uses https://nodejs.org/api/vm.html
Isolated, which will soon become the default and uses https://github.com/laverdet/isolated-vm

In both of these situations, the generated .js file will incorrect use fetch (and fail). Even if it did still detect node, it would try to use fs which does not work in this environment. In Screeps specifically the WASM file has to be required.
Currently my workaround is to run this powershell file after every build:
$js = Get-Content target\wasm32-unknown-unknown\release\mycrate.js
$js = $js.replace('typeof window === "undefined" && typeof process === "object"', 'true')
$js = $js.replace('const fs = require( "fs" );', '')
$js = $js.replace('const path = require( "path" );', '')
$js = $js.replace('const wasm_path = path.join( __dirname, "sair.wasm" );', '')
$js = $js.replace('const buffer = fs.readFileSync( wasm_path );', 'const buffer = require("mycrate")')
$js = $js.replace('typeof window === "undefined" && typeof process === "object"', 'true')
Set-Content target\wasm32-unknown-unknown\release\mycrate.js -Value $js

This is something I've been thinking about fixing soon. If cargo-web would support generating a .js runtime which you'd have to manually initialize would that work for you? Basically you'd fetch and instantiate the .wasm file yourself; the generated runtime would only provide you with the imports object (to use when instantiating the .wasm) and an initialization function to which you'd pass the .wasm module you've instantiated.
(See also https://github.com/koute/cargo-web/issues/76, https://github.com/koute/cargo-web/issues/35, https://github.com/koute/cargo-web/issues/71)

Yes that would definitely work!

Cool! I'll try to whip something up tomorrow and/or this weekend depending on how much time I have.

This is long supported with the --runtime option; closing.


Grünstromindex HTTP 429

Describe the bug
I have configured CO2 tariff and yesterday i recognized that no CO2 values are available anymore.
Running systemctl status evcc shows:
Mär 25 10:15:52 pv evcc[3780238]: [gsi   ] ERROR 2024/03/25 10:15:52 unexpected status: 429 (Too Many Requests)
Mär 25 10:15:52 pv evcc[3780238]: [main  ] ERROR 2024/03/25 10:15:52 failed configuring co2 tariff: cannot create tariff type 'grünstromindex': unexpected status: 429 (Too Many Requests)
Who is running those API-calls? Is it my evcc instance or do you run those centralized? Is there anything I can do about it?
Steps to reproduce

Configure CO2 tariff
Check if UI shows CO2 or just the cost per kWh

...
Configuration details
tariffs:
  currency: EUR
  grid:
    type: fixed
    price: 0.3197
  feedin:
    type: fixed
    price: 0.1051
  co2:
    type: grünstromindex
    zip: <schnippschnapp>

Log details
[main  ] INFO 2024/03/25 10:30:31 evcc 0.124.10
[main  ] INFO 2024/03/25 10:30:31 using config file: /etc/evcc.yaml
[main  ] INFO 2024/03/25 10:30:31 starting ui and api at :7070
[db    ] INFO 2024/03/25 10:30:31 using sqlite database: /root/.evcc/evcc.db
[mqtt  ] INFO 2024/03/25 10:30:31 connecting evcc-194965399 at tcp://<IP_ADDRESS>:1883
[mqtt  ] DEBUG 2024/03/25 10:30:31 tcp://<IP_ADDRESS>:1883 connected
[mqtt  ] DEBUG 2024/03/25 10:30:31 tcp://<IP_ADDRESS>:1883 subscribe SMA-PV/status/3015604826
[gsi   ] ERROR 2024/03/25 10:31:22 unexpected status: 429 (Too Many Requests)
[main  ] ERROR 2024/03/25 10:31:22 failed configuring co2 tariff: cannot create tariff type 'grünstromindex': unexpected status: 429 (Too Many Requests)
[semp  ] WARN 2024/03/25 10:31:23 SEMP_BASE_URL unspecified, using http://<IP_ADDRESS>:7070 instead
[site  ] INFO 2024/03/25 10:31:23 site config:
[site  ] INFO 2024/03/25 10:31:23   meters:      grid ✓ pv ✓ battery ✓
[site  ] INFO 2024/03/25 10:31:23     grid:      power ✓ energy ✓ currents ✓
[site  ] INFO 2024/03/25 10:31:23     pv 1:      power ✓ energy ✓ currents ✗
[site  ] INFO 2024/03/25 10:31:23     battery 1: power ✓ energy ✓ currents ✗ soc ✓ capacity ✓
[site  ] INFO 2024/03/25 10:31:23   vehicles:
[site  ] INFO 2024/03/25 10:31:23     vehicle 1: range ✓ finish ✓ status ✓ climate ✓ wakeup ✓
[site  ] INFO 2024/03/25 10:31:23     vehicle 2: range ✓ finish ✓ status ✓ climate ✓ wakeup ✓
[site  ] WARN 2024/03/25 10:31:23 interval <30s can lead to unexpected behavior, see https://docs.evcc.io/docs/reference/configuration/interval
[site  ] DEBUG 2024/03/25 10:31:23 ----
[site  ] DEBUG 2024/03/25 10:31:23 pv power: 2494W
[site  ] DEBUG 2024/03/25 10:31:23 battery soc: 49%
[site  ] DEBUG 2024/03/25 10:31:23 battery power: -2216W
[site  ] DEBUG 2024/03/25 10:31:23 grid meter: 1W
[site  ] DEBUG 2024/03/25 10:31:23 grid currents: [0.356 0.242 0.297]A
[site  ] DEBUG 2024/03/25 10:31:23 site power: -2115W
[main  ] DEBUG 2024/03/25 10:31:27 vehicle status: B (VW e-up!)

What type of operating system are you running?
Linux
Version
0.124.10

I get
failed configuring co2 tariff: cannot create tariff type 'grünstromindex': unexpected status: 500 (Internal Server Error)
too.

That's different than mine. "Internal Server Error" is most likely due to a problem on Corrently's site while "Too many requests" means that more calls to Corrently's API than allowed have been issued.
Buty maybe they are related? Let's wait for the Dev's feedback.

Due to the server outage in Strassbourg Datacenter the free  version of the GrünstromIndex is currently not available. We hope that out data center provider fixes the issues shortly. Meanwhile we could only provided limited free access.

Those Strassburg datacenter don't seem to be a good choice. Fire at OVH in 2021 and now water in a battery room at Server4You...

Those Strassburg datacenter don't seem to be a good choice.

We are using several... but for a free service there is no budget for the more expensive ones. However, it might be good to look for some sponsors as there are thousands of requests to our free service per hour.

Update 12:30 UTC: Limited service is restored for free service allowing 1000 instead of 10 requests per minute. Hope that this is enough for the evcc community and ioBroker users.

Don't get me wrong, I'm not critizising. Instead I'm grateful for these services. Thanks for the update.

@zoernert we're very happy with using fly.io


Getting 401 unauthorized with insufficient_error

I haven't been able to get pass the point of 401 unauthorized access when doing a PUT/POST command, it keeps on saying "authentication required".
I tried getting the bearer token from GET request call to https://registry-1.docker.io given the scope and service and was able use the provided token to authenticate any GET method api to get list of tags... etc.
I tried doing the same with PUT method by using Basic authentication with username and password, or getting token in the same way described above. I'm receiving 401 unauthorized error with insufficient_error tag on still.
What am I doing wrong here?
Trying to get it work with Postman.
My request:

PUT /v2/{image}/manifests/{newtag} HTTP/1.1
Host: registry-1.docker.io
Content-Type: application/vnd.docker.distribution.manifest.v2+json
Authorization: Bearer eyJhbGciOiJFUzI1NiIsInR5cC....
Attached manifest.json as raw text in body

Response:

Content-Length →244
Content-Type →application/json; charset=utf-8
Date →Wed, 20 Dec 2017 00:17:10 GMT
Docker-Distribution-Api-Version →registry/2.0
Strict-Transport-Security →max-age=31536000
Www-Authenticate →Bearer realm="https://auth.docker.io/token",service="registry.docker.io",scope="{image}:pull,push",error="insufficient_scope"
{"errors":[{"code":"UNAUTHORIZED","message":"authentication required","detail":[{"Type":"repository","Class":"","Name":"{image}","Action":"pull"},{"Type":"repository","Class":"","Name":"{image}","Action":"push"}]}]}

@shenred176 were you able to figure it out?


Include thrift idl files in repo

Instead of only including the generated C++ bindings, could you please include the Thrift IDL files in the repository? This is both a more concise reference to program to and also allows developers to generate bindings w/ custom parameters (e.g. the Cob-style c++ RPC clients).

+1

Isn't this what you are looking for? https://github.com/evernote/evernote-thrift

Absolutely; somehow overlooked that. Thanks!


pandoc-include: 1.2.1 -> 1.3.0

Description of changes

Changelog: https://github.com/DCsunset/pandoc-include/blob/master/CHANGELOG.md#130-2024-02-10
Things done

Built on platform(s)

[x] 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

[x] 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
[x] 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.

Thanks for the comments. I've updated the commit accordingly.


bump utp-rs to 0.1.0-alpha.4

What was wrong?

incorrect uTP ACK num initialization upon SYN-ACK
unexpected sequence number in uTP STATE packets

How was it fixed?
bump utp-rs to 0.1.0-alpha.4 to include fix
To-Do

[ ] Add entry to the release notes (may forgo for trivial changes)
[ ] Clean up commit history

it would be nice if we could run the portal hive test suite before merge


Is it possible to run buildkitd as container in containerd system-namespace?

nerdctl needs user to install buildkitd. So is it possible to support system-plugin as container running in specific container namespace? If so, the user doesn't need to care about buildkitd, nerdctl can help to install buildkitd as running container.

Yes, this should be possible for OCI worker. This might be hard for containerd worker, but I guess we don't really need to use containerd worker.

We are going to recommend using containerd worker (https://github.com/containerd/nerdctl/pull/858), so let me close this one


Hi I need some assistance can it work with ardupilot

Pls tell me does it work with ardupilot

Hi. We don't support ArduPilot specifically. This packages target PX4 primarily.
But some of the features (like getting telemetry and others) might work.

Hi. We don't support ArduPilot specifically. This packages target PX4 primarily.
But some of the features (like getting telemetry and others) might work.

So d u know something that would work with ardupilot like a service that u
made is there any service like same pls tell  pls tell me I need it
On Thu 7 Jan, 2021, 8:48 PM Oleg Kalachev<EMAIL_ADDRESS>wrote:

Hi. We don't support ArduPilot. These packages target PX4 primarily.
But some of the features (like getting telemetry and others) might work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CopterExpress/clover/issues/296#issuecomment-756179904,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARCQFBPAN7OZ44ROLPVJHYTSYXGFNANCNFSM4VX6UYNQ
.

So d u know something that would work with ardupilot like a service that u
made is there any service like same pls tell  pls tell me I need it
On Thu 7 Jan, 2021, 8:48 PM Oleg Kalachev<EMAIL_ADDRESS>wrote:

Hi. We don't support ArduPilot. These packages target PX4 primarily.
But some of the features (like getting telemetry and others) might work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CopterExpress/clover/issues/296#issuecomment-756179904,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARCQFBPAN7OZ44ROLPVJHYTSYXGFNANCNFSM4VX6UYNQ
.

Some of features will work. mavros package itself support Ardupilot, for example.

Some of features will work. mavros package itself support Ardupilot, for example.

Can u pls tel me how to program drone ardupilot with opencv rpi pls help me
On Tue 12 Jan, 2021, 6:05 PM Oleg Kalachev<EMAIL_ADDRESS>wrote:

Some of features will work. mavros package itself supports Ardupilot, for
example.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CopterExpress/clover/issues/296#issuecomment-758627714,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARCQFBOPCPWEQCDQNS2ZT4LSZQ623ANCNFSM4VX6UYNQ
.

Can u pls tel me how to program drone ardupilot with opencv rpi pls help me
On Tue 12 Jan, 2021, 6:05 PM Oleg Kalachev<EMAIL_ADDRESS>wrote:

Some of features will work. mavros package itself supports Ardupilot, for
example.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CopterExpress/clover/issues/296#issuecomment-758627714,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARCQFBOPCPWEQCDQNS2ZT4LSZQ623ANCNFSM4VX6UYNQ
.


mapkit tile overlay CGImageSourceCreateWithData error

I can't find many answers to this error on the web...
Here's the error that I get:
[8225] : ImageIO: CGImageSourceCreateWithData data parameter is nil
It looks like in SDK8, the tile overlay generates errors... but is it the mbxmapkit or the apple MapKit? and is this error has been reported somewhere?
Thx!
Brian

I used pod 'MBXMapKit' in my podfile and I can't seem to find the MBXRasterTileRenderer.
Is there something I should check?
I get the : ImageIO: CGImageSourceCreateWithData data parameter is nil both in iPhone 6 simulator and iOS 8.1 iPhone 6 device.
Also, can I avoid the refresh that is doing when I drag the map around? I should use caching to improve performance?
Another problem is that we don't have access to set the accesstoken, will this introduce problems using a custom layout to MapKit?

@gtasko This is not yet released (see #136; it's close) so you'll have to reference the 132-renderer branch in your Podfile in order to pull this in.

OK, thanks, I'll wait for the release then.
Do you know about the setAccessToken question? Why the method is not exposed to the interface file?

Yes, it's related. See the changelog. It will come back very soon with support for v4. Right now v3 will automatically be used.

OK, thanks for the response, we will wait for the next release version.

How to calculate the size of the selected area of the map before downloading i am using MBXMapKit SDK any one can help me how to calculate the size of the map before downloading the map.


Artefacts in render of simple object

The render shows artefacts of simple triangulated surface

Tested with Cycles and RenderMan, neither show this problem. File was converted from CAD to stl.  Running Alpha 7 of LuxCore on Linux Mint 18.3, GTX 970 and AMD Ryzen 1700.

Forgot to apply the scale on the file I provided in the zip, but I did apply it on the test scene I used to produce the image in the initial post. Sorry, tired.

Min and max epsilon are currently not exposed in BlendLuxCore.
I have this issue about it: https://github.com/LuxCoreRender/BlendLuxCore/issues/24
Is there some algorithm we can use to automatically find good values for min/max epsilon?

I can confirm that the source of the problem is the object scale:
Object name: Mesh_HDDrack_140595502187016000
BBox[Point[-0.00199997, -0.119998, 0.00154629], Point[0.121998, 0, 0.150544]]

The object is very small for LuxCore, it has a width of about 10cm. If I set the epsilon value to:
scene.epsilon.min = 0.001
scene.epsilon.max = 0.001

I get this error free rendering:

I don't know how Blender unit management works but, looking at the Blender file, the object seems to be scaled down by "0.001" and so it is very small (1000 times smaller than usual) for LuxCore. @ajmurrell, you  can fix the problem by changing the object scale or by adapting the epsilon value used as listed above (after the support has been added to LuxCore Blender plugin).
P.S. Simon, there isn't a way to adapt epsilon min/max values. The problem is intrinsic in the nature of the vary low precision (and variable resolution) of 32bit floating point numbers.

Yes the scale is screwed up on the blend file I provided, but the original render was scaled down and the scale applied.  There is an issue with scale coming from the CAD package I used to blender's units, though they are both metric, blender adds a few digits (3) to the actual measurements. But from what I am gathering, there is a minimum object size for a clean render? Are we talking that we need object sizes in the meter range?

But from what I am gathering, there is a minimum object size for a clean render? Are we talking that we need object sizes in the meter range?

Yes and no.
If we say that the floating point number 1.0 always equals 1 meter, then yes.
But floating point numbers do not have inherent units, so depending on the scene you can redefine them and say that 1.0 equals 1mm or 1km.
Related:

https://github.com/LuxCoreRender/BlendLuxCore/issues/97
https://forums.luxcorerender.org/viewtopic.php?f=4&p=1733#p2064

It is something I have to fix in the addon, but it is a larger refactor.

There is a covenant in Lux, it assumes 1 is 1 meter and the default values of some parameter is picked according this convention.
If you use a scale significantly far from covenant (like 1000 times), you have to set the other parameters according the new scale (like what is considered an "epsilon" or bump mapping scale, etc.).
This is true for any computer application approximating real numbers with (very) finite precision of 32 bit floating numbers. It is true for all rendering programs, CAD, etc. It may be different where the problem starts to be noticeable (10, 100, 1000, 10.000 scale) but you are always going to hit a wall.
As you have noticed GPUs has often a lower than CPUs precision. This is not due to some intrinsic limit but more likely to some short cut taken by vendors in the implementation of OpenCL standard (it shouldn't happen but it does and quite often).

Ok, what should I put as sensible min, max and description?

With min/max I meant the minimum and maximum values the user is allowed to set for the max. epsilon.
For the min. epsilon I chose min=1e-6, max=0.1.

The only rule is "max. epsilon >= min epsilon", after that there are just too many factors to have a fixed rule.
P.S. it may be better to use min=1e-5 as default value, it is a bit more GPU friendly.

P.S. it may be better to use min=1e-5 as default value, it is a bit more GPU friendly.

It is the default.
Ok it's in: https://github.com/LuxCoreRender/BlendLuxCore/commit/3a871345b34d6d7db638873c07d1f8a6a0fbcbf0


[DNM]separate read/write ready

Signed-off-by: Jay Lee<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
Proposal: xxx
What's Changed:
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
Please add a release note.
If you don't think this PR needs a release note then fill it with None.

/release

/release make dist_release

/release make dist_release

/release

/release

/release make dist_release

Can this PR be closed?


ci: use reusable workflows for agreements and release notes

Github introduced reusable workflows: https://docs.github.com/en/actions/learn-github-actions/reusing-workflows so we can leverage them by replacing agreements and other popular simple workflows to use 1 centralised version.

Codecov Report

Merging #119 (1f842e2) into main (72fd9eb) will increase coverage by 0.05%.
The diff coverage is n/a.

:exclamation: Current head 1f842e2 differs from pull request most recent head 73e022a. Consider uploading reports for the commit 73e022a to get more accurate results

@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
+ Coverage   87.49%   87.54%   +0.05%
==========================================
  Files          23       23
  Lines        1831     1831
==========================================
+ Hits         1602     1603       +1
+ Misses        229      228       -1

Impacted Files
Coverage Δ

solnlib/file_monitor.py
91.22% <0.00%> (+1.75%)
:arrow_up:

Continue to review full report at Codecov.

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

This is already tested here: https://github.com/splunk/addonfactory-splunk-conf-parser-lib


Fresh install of Home-brew causes oh-my-zsh permission errors when using zsh

What you were trying to do (and why)
I have a fresh copy of macOS Catalina, which is using zsh as the default shell. I have installed oh-my-zsh too.
I just finished installing Homebrew, and now, every time I open a new Terminal session, I see permission warnings from oh-my-zsh.
I appreciate I can add ZSH_DISABLE_COMPFIX to my .zshrc file, but this doesn't seem like a good idea.
What happened (include command output)
New Terminal windows now show the following warning:

Command output

[oh-my-zsh] Insecure completion-dependent directories detected:
drwxrwxr-x  3 jack  admin   96 Apr 13 20:08 /usr/local/share/zsh
drwxrwxr-x  4 jack  admin  128 Apr 13 20:11 /usr/local/share/zsh/site-functions
[oh-my-zsh] For safety, we will not load completions from these directories until
[oh-my-zsh] you fix their permissions and ownership and restart zsh.
[oh-my-zsh] See the above list for directories with group or other writability.
[oh-my-zsh] To fix your permissions you can do so by disabling
[oh-my-zsh] the write permission of "group" and "others" and making sure that the
[oh-my-zsh] owner of these directories is either root or your current user.
[oh-my-zsh] The following command may help:
[oh-my-zsh]     compaudit | xargs chmod g-w,o-w
[oh-my-zsh] If the above didn't help or you want to skip the verification of
[oh-my-zsh] insecure directories you can set the variable ZSH_DISABLE_COMPFIX to
[oh-my-zsh] "true" before oh-my-zsh is sourced in your zshrc file.

What you expected to happen
There should not be a warning, Homebrew should install its completions with the correct permissions.
Step-by-step reproduction instructions (by running brew commands)
 /bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install.sh)"
brew help
brew update
brew doctor
brew config

Output of brew config and brew doctor commands

▶ brew doctor
Your system is ready to brew.

▶ brew config
HOMEBREW_VERSION: 2.2.13
ORIGIN: https://github.com/Homebrew/brew
HEAD: 3d9cf83fec45a75af61551f53d25383abe009d31
Last commit: 7 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: ae7ff1293bc1c8a62946d4aaa107651321c33baa
Core tap last commit: 22 minutes ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_EDITOR: nano
HOMEBREW_MAKE_JOBS: 8
CPU: octa-core 64-bit kabylake
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
Clang: 11.0 build 1103
Git: 2.24.1 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 10.15.4-x86_64
CLT: <IP_ADDRESS>.1.1583798712
Xcode: 11.4

Can you reproduce this with plain zsh (i.e. not with oh-my-zsh)?

I haven't tried it, but logic would suggest it can't be reproduced on plain zsh.
ZSH_DISABLE_COMPFIX and the specific error message output are all contained in ~/.oh-my-zsh/lib/compfix.sh, meaning they are part of the oh-my-zsh installation.

In that case given it doesn't apply to software shipping with macOS: we'll accept a PR to Homebrew/install to fix this but don't consider it our bug, sorry.


Support xxHash

xxHash is very fast. Consider supporting it.

I'll mull it over, but the road map for the hash lib is to tackle Blake next which will open up implementing Argon2 PBE (among other things).
Will leave this open for the time being.


Install vue3 migration build

Followed https://v3-migration.vuejs.org/migration-build.html
In development, the Vue components now generate a lot of warnings but should still be working.  This should help us migrate to Vue 3.
Fixes # (link to Jira or GitHub issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] I have updated the CHANGELOG
[x] I have requested at least 1 reviewer for this PR
[ ] I have made corresponding changes to the documentation

@FrantisekMichalSebestyen Thank you for a thorough review! The PopoverGroupController is now working. The problem was that it was using this.$scopedSlots.default({}), but in the Blade template there was no v-slot (because it was unnecessary), so the default slot did not get created.


PHP standards

Hello, thanks for working on this!
Seems a few common PHP standards are not followed in this package:

PSR-0 / PSR-4 autoloading structure
No Composer support, requires 1.
PSR-2 / PSR-12 coding style
PDS skeleton guidelines
unit tests, mutation tests
static analysis

Do you feel this is something you're interested in as a general direction?

Thank you for the information, we will take this into account if it has not already been implemented or is in process


Fixing Cosmos DB Javascript template code

This change addresses the correct property

@ealsur,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


ARROW-3053: [Python] Pandas decimal conversion segfault

Codecov Report

Merging #2429 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #2429      +/-   ##
=========================================
+ Coverage   85.47%   85.5%   +0.02%
=========================================
  Files         300     300
  Lines       45849   45857       +8
=========================================
+ Hits        39190   39208      +18
+ Misses       6585    6579       -6
+ Partials       74      70       -4

Impacted Files
Coverage Δ

python/pyarrow/tests/test_convert_pandas.py
94.99% <100%> (+0.03%)
:arrow_up:

go/arrow/memory/memory_sse4_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/uint64_sse4_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/float64_sse4_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/int64_sse4_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/float64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/uint64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/int64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/math_amd64.go
36.84% <0%> (+5.26%)
:arrow_up:

go/arrow/memory/memory_amd64.go
42.85% <0%> (+14.28%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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

I see no fix here. Did you manage to reproduce ARROW-3053? Is it required to go through HDF5?

Oops missed @pitrou same comment

The roundtrip through HDF5 is required to reproduce the problem. If you make a copy of the DataFrame after the roundtrip, the problem goes away.

Hmm... and did someone investigate why that was?

The segfault occurs here, I'm not sure why yet
#3  0x00007fffe3a49834 in arrow::py::NumPyConverter::ConvertDecimals (this=0x7fffffffad58) at ../src/arrow/python/numpy_to_arrow.cc:789
789	      RETURN_NOT_OK(max_decimal_metadata.Update(object));
(gdb) l
784	  internal::DecimalMetadata max_decimal_metadata;
785	  Ndarray1DIndexer<PyObject*> objects(arr_);
786
787	  if (type_ == NULLPTR) {
788	    for (PyObject* object : objects) {
789	      RETURN_NOT_OK(max_decimal_metadata.Update(object));
790	    }

This script segfaults for me with 0.9.0
import decimal

import pandas as pd
import pyarrow as pa

import pandas.util.testing as tm

data = {
    'a': {0: 'a'},
    'b': {0: decimal.Decimal('0.0')}
}
df = pd.DataFrame.from_dict(data)

nthreads = 1
table = pa.RecordBatch.from_pandas(df, preserve_index=False,
                                   nthreads=nthreads)

result = table.to_pandas(nthreads=nthreads)

tm.assert_frame_equal(result, df, check_dtype=True)

It does not fail against master. So at least we should nix all the hdf5 stuff in this patch until we understand the underlying problem

I found the problem. The "object : objects` logic is implemented incorrectly for NdarrayIndexer:
  int64_t size() const { return PyArray_SIZE(arr_); }

  T* data() const { return data_; }

  T* begin() const { return data(); }
  T* end() const { return begin() + size() * stride_; }

In this particular case, the stride is 2, so it is reading past the end of the buffer:
(gdb) p objects.size()
$5 = 1
(gdb) p object
$6 = (PyObject *) 0x16532d0
(gdb) p objects.stride_
$7 = 2

I removed the STL iterator protocol methods from NdarrayIndexer<T> for now as it does not work correctly. Removed the hdf5 stuff also

FYI @adshieh and others who are interested, the reason why HDFStore is in the mix is because of a bug in HDFStore causing columnar / non-strided columns to be returned as strided. I recently reported this as a bug
https://github.com/pandas-dev/pandas/issues/22073

+1


Fix #7611: Add support for String Literal Types in find all refs and occurances

Fixes #7611

Can you add a test where an ordinary string literal (of type string) don't get affected?

@DanielRosenwasser any more comments?

I meant a test in which contextually typed strings are considered references, but others of the same content are not.
Other than that and my last feedback, looks great :+1:


Performance tracker

If I understand correctly, the primary motivation behind Sixten's design of having data unboxed by default is cache performance. As documented in the Readme, this approach creates some runtime overhead as functions need to be passed the sizes of the types, as Sixten doesn't do monomorphization by default. It would be nice if we could add some small benchmarks comparing Sixten to other functional languages, to have a concrete sense of how much the net benefit is.
For the monomorphization case, it would be useful to compare to other imperative languages too and check if it is similar where it ought to be. Since this ought to have similar cache performance, we could also isolate the runtime overhead by comparing the monomorphized benchmark with the ordinary benchmark.
An enhancement of this feature would be keeping track of running times over commits automatically. This would be useful when adding and optimizing new features such as records.

Agreed, this would be great to have!

I would be really interested in this, since if the difference is insignificant, there is probably no reason to use unboxed representation instead of boxed / lazy one.

Contributions would be very welcome!


invoices: fix log messages

This PR fixes incorrect log messages.

@bhandras Is this still a draft PR?

@bhandras Is this still a draft PR?

No, sorry forgot to push the button.


Update openssl path for poky branch daisy

By following the instructions in the README, the daisy branch of poky that is pulled from Yocto's repository has the required openssl recipe filename of openssl_1.0.1m.bb instead of openssl_1.0.1j.bb, which meta-debian attempts to find (link).

Thank you very much for your contribution.
I also modified openssl recipe (58bd3ab4b13f82c9fb6f56d2586f1a2e594bc271) to handle the update to 1.0.1m. After updated to 1.0.1m, some openssl patches in poky/meta are changed. (http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?h=daisy&id=159f66aea7860b1008b7dcd60d81dfa12d7546d9) The patches cannot be applied to jessie's source tree (1.0.1k). This mismatch was solved in the above commit.


Fix return value of msc_rules_merge()  and fix typos

Hi all,
Regarding to the documentation msc_rules_merge() should
return the number of merged rules instead of 0 in all cases.
Fix typos in documentation.

Hi @soonum,
Thank you for the patch ;)

Merged!


How can I gain access to a camera of the camera manager to set an android.hardware.camera.PreviewCallback to it?

I need to take a picture of the whole image and not the cropped version of BarcodeResult.getBitmap(). The only way I see is too attach a PreviewCallback on a camera so I can use that to do camera.takePicture(previewCallback) but all of the camera instances in the CameraManager class is private. How do I do this? Thanks in advance

I'm busy with a refactor to expose more internals like the Camera instance, but it may take a while before it's ready for release.

Can you let me know once it's done? Thank you, and very nice plugin!

BarcodeResult#getBitmap() now returns the entire image (instead of the cropped version) in v4.0.2.


Feat 986 change external links to list of dicts

closes #986
Metadata.external_links input changed: List[Dict[Platform, str]] --> Dict[Platform, List[str]]
This might impact how data is drawn from the metadata service / how data is indexed, so @helen-m-lin and @mekhlakapoor should asses those effects.

This shouldn't affect the metadata-service

This is technically a breaking change. If users have code expecting this field to be in a certain shape, then they'd need to update their code base. If we implement a stricter version control release cycle, then this is something that would need to be discussed. I'll leave it up to Saskia and David to decide on whether to merge this. If we haven't released v1.0, then maybe we can add this before then.


Better selector

After reviewing #68, I've made a few tweaks to the selector engine ( Init and find ) and getting nearly 2-3 times the speed of jQuery and the original cash for almost every test.
http://jsperf.com/shshaw-vs-jquery/7

This greatly resolves the after test slowness that was initially shown by @simeydotme

Outstanding

:+1: