Unable to add onchange event to each field inside array type in json schema

Description
I have a requirement like one textbox depends on other textbox value inside array of textboxes. I am adding a change event to one textbox dynamically but its not adding.
Minimal Reproduction
JSON Schema:
{

"type": "object",
"title": "Rules",
"properties": {
"LR": {
  "type": "array",
  "maxItems": 9,
  "title": "LR",
  "items": {
    "type": "object",
    "properties": {

      "PMR": {
        "type": "string",
        "title": "PMR",
        "pattern": "^(\\d{1,3})(\\.\\d{0,2})?$",
        "widget": {
          "formlyConfig": {
            "templateOptions": {
              "maxLength":"3",
              "pmrChange":"this.pmrChange($event, field)"
            }
          }
        }
      }
    }
  }

}

}
}

extendFieldConfigs(): FormlyFieldConfig[] {
this.fieldConfig.fieldGroup.map((field) => {

  const key = field.key ? field.key.toUpperCase() : null;
  if (!key) return;
  debugger;
  switch (key) {
    case "LR":

      field.fieldArray &&
        field.fieldArray.fieldGroup.map((fgrpItem) => {
          const childKey = fgrpItem.key ? fgrpItem.key.toUpperCase() : null;
          switch (childKey) {
            case "PMR": {
              fgrpItem.templateOptions.change = Function(
                "field",
                "$event",
                fgrpItem.templateOptions.pmrChange
              ).bind(this);

              break;
            }
          }
        });

      break;
  }
});

return [this.fieldConfig];

}
pmrChange(a, b) {
alert(a);
}
Your Environment

Angular version: 9.0.4
Formly version: 5.5.13

Additional context
When its direct element like textbox its fine , but when we add in type array then event is not working.. pls help.

Related to https://github.com/ngx-formly/ngx-formly/issues/2809 which has been fixed in next version. Fields in array are generated during the build step we've used a getter in v5 (as workaround), the proper way can be found in next version which allows passing a function to the field array.

Next: https://github.com/ngx-formly/ngx-formly/blob/main/src/core/json-schema/src/formly-json-schema.service.ts#L265
V5: https://github.com/ngx-formly/ngx-formly/blob/v5/src/core/json-schema/src/formly-json-schema.service.ts#L265

V5: https://github.com/ngx-formly/ngx-formly/blob/v5/src/core/json-schema/src/formly-json-schema.service.ts#L265
How to use this in my above  code :?
fgrpItem.templateOptions.change = Function(
"field",
"$event",
fgrpItem.templateOptions.pmrChange
).bind(this)

I would suggest using v6 if you can, for v5 use a similar way of the source code but not recommended:
const fieldArray = Object.getOwnPropertyDescriptor(field, 'fieldArray').get;
Object.defineProperty(field, 'fieldArray', {
  get: function () {
    const fgrpItem = fieldArray();
    ....

    return fgrpItem;
  }
}

V6 is still in-progress I think. is it stable..Can I upgrade from 5.10.21?

Still in-progress but the major parts are done, we may still introduce some breaking changes before going to stable.


put www in api

This adds the ability to build the Studio dashboard into static HTML files. Then it bundles those static files with the API server. The end result is that the API server is now capable of serving the frontend, and the pkg for www can be removed.

@0xcadams I think we covered this on Discord, but just to clarify publicly: nah, we'll still use Vercel, this is just for a different use case


Long Build Times

I can't tell you just how awesome this tool is---the only "oh man I wish it were better" for me is how long "1c build" takes even for the simple examples.
Any thoughts on why that is? Any thing I could do in terms of extra memory allocation or some such to make the build take a little less time?
Again thanks for a kick-butt tool---I've been dying for something like this to show up on the server side for a long time now

In #7, @wizawu mentioned:

If you are sure the TypeScript source code is written without syntax error, you can use 1c build --skipJDK to skip checking TypeScript Definition of JDK, which can reduce a lot of compilation time.

Understood.
But I am hoping for a more general solution, similar in my experiences in the past with something like the GWT compiler, where if you have it, you can trade extra memory at compile time for reduced compile times, in cases where i may or may not know if the code is going to compile cleanly or not.

I think this issue is quite related to #9 . The slow compilation is caused by checking those large .d.ts files.
I am also very annoyed by this when I am trying to deploy a new version of daemon/website online. When we used Python/Node/Go before, it took less than one minute. But now we have to wait 2-3 mins for 1c build.
So I will try fix that, along with #9 .

@mfreeman-xtivia @Egoscio
Please try this example with<EMAIL_ADDRESS>// test.ts
java.lang.System.out.println($ENV.PWD)

The compilation time has been reduced from 6.4s to 3.8s.

Great news! I just added fork-ts-checker-webpack-plugin to speed up builds.
I just tried everything ts-loader recommended including

fork-ts-checker-webpack-plugin
happypack
thread-loader/cache-loader

The latter two didn't seem to have significant optimization. Luckily, with fork-ts-checker-webpack-plugin added to 1c, we can already reduce the build time to 50% (122s -> 55s).
But please notice that for small projects (like building single ts file), the build time is still at least 5 seconds or so (Mostly costed by compiler bootstrap).


What about Standby API?

https://www.chromestatus.com/feature/5768542523752448

This was renamed to the Wake Lock API, which is available at https://caniuse.com/#feat=wake-lock


Hiding a line is not persistent with data changes

I have 2 data sets:
const DATA1 = {
  series: [
    [0, 2],
    [1, 1]
  ]
}

const DATA2 = {
  series: [
    [0, 2],
    [1, 1]
  ]
}

Note: the 2 data sets contain the same values but are not the same object.
I inject one of them in a Chartist which draws this chart:

Then I hide the blue line by unchecking the box:

Then I dynamically inject the other set by clicking the "Switch data set" button:

The blue line shows again.

I'm afraid I can't follow, what is the expected result? I'm not sure what exactly you are doing and am not sure if this is intended or not, since i assume you are switching references or something.. Please elaborate and include your code so i can investigate.

Hi! I'll try to be clearer.
The expected result was for the blue line to stay hidden when switching data set. As you can see on the last screenshot, the blue checkbox is unchecked while the blue line is showing. I don't think this should happen.
Here's my code for the 2 line example:

Code
import React, { Component } from 'react'
import ChartistGraph from 'react-chartist'
import ChartistLegend from 'chartist-plugin-legend'

const DATA1 = {
  series: [
    [0, 2],
    [1, 1]
  ]
}

const DATA2 = {
  series: [
    [0, 2],
    [1, 1]
  ]
}

export default class extends Component {
  state = {
    data: DATA1
  }

  _switch = () => this.setState({
    data: this.state.data === DATA1
      ? DATA2
      : DATA1
  })

  render () {
    const opts = {
      showArea: true,
      height: 300,
      plugins: [
        ChartistLegend()
      ]
    }

    return

      <button onClick={this._switch}>Switch data set</button>

        <ChartistGraph
          type='Line'
          data={this.state.data}
          options={opts}
        />

  }
}

And here's a more concrete example to help you understand what my issue is:
I'm injecting some network stats into a chartist graph:

The stats are updated every 10 seconds by injecting a new data set.
When I uncheck a checkbox, the corresponding line disappears, which is fine:

But a few seconds later, on the next data update, the line shows up again:

The expected result would be for the red line to stay hidden until I check the box again.
Like in the 2 line example, on each update, a new object is injected (new reference), I am not updating the previous object.
I hope I've made myself clear :)

All clear now, I agree with you, that should not happen.
Without rigorously checking the code, I assume the issue is the object references the legends uses to determine display gets lost as you use a new object. I'll investigate if this should be implemented different (or can be), or the library should check this, regarding the hopefully soon-to-come change in which it should support dynamic changes with e.g. React, as you're using it.
You might have already seen by the amount of open issues, I'm very, very busy with things besides maintaining this project, so my throughput is really low right now. I'll try to find some time soon.

What happens after looking at the code is quite straightforward. Legend is called at initialization of plugin. Clicking a series element will remove it from the chart data entirely. So updating the chart data will override whatever the plugin has done until now.
Worse, after clicking on the legend you will get old data instead of new data because at initialization it took the data.
Even worse, there is no way to access the plugin variables to manually update them. The plugin should get called on data updates or something. In reality Chartist should allow you to change data and have properties for data like "visible:false" to avoid this entire mess.
Currently (horrible workaround), I check legend elements if they are hidden, recreate chart from scratch and "click" the legend items through javascript, there is a flicker of course.
let lastChart; //global variable?
...
       let hiddens = [];
        if(lastChart){
          document.querySelectorAll(".ct-legend .inactive").forEach((item)=>{
            hiddens.push(item.getAttribute("data-legend"));
          });
        }
        lastChart = new Chartist.Line('#ct-chart', {
          labels: timestamps,
          series:series
        }, {

          plugins: [
              Chartist.plugins.legend({
                  className: 'crazyPink',

              })
          ]
        });
        setTimeout(()=>{
          if(hiddens.length){
            hiddens.forEach(index=>{
              let item = document.querySelector(".ct-legend [data-legend='"+index+"']");
              if(item)
                item.click();
            });
          }
        },100);

Hello,
are there any news on this problem (other than the workaround provided by @msj121 ) ?
It seems this project is quite abandoned (no updates since a while).
Is someone aware of a fork which could be used instead of this project?

If somebody still got issue like me, when you cannot have legend while updating data, here is my solution. 4 years later woow.
In CSS add class:
.ct-hidden { visibility: hidden; }
In JS remove all references to originalSeries and seriesCopy (line 97, 183, 192, 202).
No needed. Definitely could be cleaned much more, but did not want to dig too deep.
Edited/added lines 155, 161, 171, 175.
if (removedSeriesIndex > -1) {
   // Add to series again.
   chart.data.series[seriesIndex].className = chart.data.series[seriesIndex].className.replace('ct-hidden','');    //<--remove class
   removedSeries.splice(removedSeriesIndex, 1);
   li.classList.remove('inactive');
} else {
   if (!options.removeAll) {
      // Remove from series, only if a minimum of one series is still visible.
      if ( chart.data.series.length > removedSeries.length+1) {       //<--if not last
         chart.data.series[seriesIndex].className += ' ct-hidden';   //<--add class
         removedSeries.push(seriesIndex);
         li.classList.add('inactive');
      }
      // Set all series as active.
      else {
         removedSeries = [];
         var seriesItems = Array.prototype.slice.call(legendElement.childNodes);
         seriesItems.forEach(function (item) {
            item.classList.remove('inactive');
         });
         chart.data.series.forEach(function (item) {
            item.className = item.className.replace('ct-hidden','');   //<--remove class from all items
         });
      }
   }
   else {
     // Remove series unaffected if it is the last or not
     chart.data.series[seriesIndex].className += ' ct-hidden';    //<--add class
     removedSeries.push(seriesIndex);
     li.classList.add('inactive');
   }
}

It has one dis/advantage - depending how you watch on it.
The graph will not adjust axis when series is hidden.
But for me, it was actually desired, so will have same still ratio while switching series.
Anyway now you can update data and chart:
data.series[0].data = Array(10).fill().map(() => ({x: Math.round(Math.random() * 10), y: Math.round(Math.random() * 50) }));
chart.update();

Maybe somebody will use just idea and make the code looks better :)


MexcTicker model missing return "QuoteVolume"

Describe the bug
MexcTicker model missing return "QuoteVolume"

Please add it. Thanks

Has been added


This happened during the build of cfjump which has been working with the same build process for some time.

I'm unable to reproduce this:
$ cd $GOPATH/src/github.com/pivotal-cf/cliaas &&\
> glide install &&\
> go install github.com/pivotal-cf/cliaas/cmd/cliaas
[INFO]  Downloading dependencies. Please wait...
[INFO]  --> Found desired version locally cloud.google.com/go 675fad27ef35e552d3708858ff337d54b3b6f4d3!
[INFO]  --> Found desired version locally code.cloudfoundry.org/clock 2269160ae1757f96bbb8c6475e6fa36c805e73e0!
[INFO]  --> Found desired version locally github.com/aws/aws-sdk-go 00fb2125993965df739fa3398b03bef3eb2e198f!
[INFO]  --> Found desired version locally github.com/Azure/azure-sdk-for-go 088007b3b08cc02b27f2eadfdcd870958460ce7e!
[INFO]  --> Found desired version locally github.com/Azure/azure-storage-go 7c36d244fc8bf27bcf9785b0b37d6beef2949296!
[INFO]  --> Found desired version locally github.com/Azure/go-autorest ec5f4903f77ed9927ac95b19ab8e44ada64c1356!
[INFO]  --> Found desired version locally github.com/dgrijalva/jwt-go 2268707a8f0843315e2004ee4f1d021dc08baedf!
[INFO]  --> Found desired version locally github.com/go-ini/ini e7fea39b01aea8d5671f6858f0532f56e8bff3a5!
[INFO]  --> Found desired version locally github.com/golang/protobuf 2bba0603135d7d7f5cb73b2125beeda19c09f4ef!
[INFO]  --> Found desired version locally github.com/google/go-genproto 411e09b969b1170a9f0c467558eb4c4c110d9c77!
[INFO]  --> Found desired version locally github.com/googleapis/gax-go 9af46dd5a1713e8b5cd71106287eba3cefdde50b!
[INFO]  --> Found desired version locally github.com/jessevdk/go-flags 48cf8722c3375517aba351d1f7577c40663a4407!
[INFO]  --> Found desired version locally github.com/jmespath/go-jmespath bd40a432e4c76585ef6b72d3fd96fb9b6dc7b68d!
[INFO]  --> Found desired version locally github.com/pkg/errors ff09b135c25aae272398c51a07235b90a75aa4f0!
[INFO]  --> Found desired version locally golang.org/x/net ffcf1bedda3b04ebb15a168a59800a73d6dc0f4d!
[INFO]  --> Found desired version locally golang.org/x/oauth2 7fdf09982454086d5570c7db3e11f360194830ca!
[INFO]  --> Found desired version locally golang.org/x/text f4b4367115ec2de254587813edaa901bc1c723a8!
[INFO]  --> Found desired version locally google.golang.org/api 48e49d1645e228d1c50c3d54fb476b2224477303!
[INFO]  --> Found desired version locally google.golang.org/appengine 56d253d1dd14aa01937e12c73a0971bcfd797ff2!
[INFO]  --> Found desired version locally google.golang.org/grpc b5071124392bfd416c713e6595ac149b387f7186!
[INFO]  --> Found desired version locally gopkg.in/yaml.v2 a3f3340b5840cee44f372bddb5880fcbc419b46a!
[INFO]  --> Found desired version locally github.com/google/uuid 6a5e28554805e78ea6141142aba763936c4761c0!
[INFO]  --> Found desired version locally github.com/onsi/ginkgo 77a8c1e5c40d6bb6c5eb4dd4bdce9763564f6298!
[INFO]  --> Found desired version locally github.com/onsi/gomega 334b8f472b3af5d541c5642701c1e29e2126f486!
[INFO]  --> Found desired version locally golang.org/x/sys 7de4796419dc3b554e6dab3a119a62469569d299!
[INFO]  Setting references.
[INFO]  --> Setting version for github.com/Azure/go-autorest to ec5f4903f77ed9927ac95b19ab8e44ada64c1356.
[INFO]  --> Setting version for github.com/dgrijalva/jwt-go to 2268707a8f0843315e2004ee4f1d021dc08baedf.
[INFO]  --> Setting version for github.com/go-ini/ini to e7fea39b01aea8d5671f6858f0532f56e8bff3a5.
[INFO]  --> Setting version for github.com/googleapis/gax-go to 9af46dd5a1713e8b5cd71106287eba3cefdde50b.
[INFO]  --> Setting version for github.com/jmespath/go-jmespath to bd40a432e4c76585ef6b72d3fd96fb9b6dc7b68d.
[INFO]  --> Setting version for github.com/google/go-genproto to 411e09b969b1170a9f0c467558eb4c4c110d9c77.
[INFO]  --> Setting version for github.com/pkg/errors to ff09b135c25aae272398c51a07235b90a75aa4f0.
[INFO]  --> Setting version for github.com/Azure/azure-sdk-for-go to 088007b3b08cc02b27f2eadfdcd870958460ce7e.
[INFO]  --> Setting version for github.com/jessevdk/go-flags to 48cf8722c3375517aba351d1f7577c40663a4407.
[INFO]  --> Setting version for github.com/aws/aws-sdk-go to 00fb2125993965df739fa3398b03bef3eb2e198f.
[INFO]  --> Setting version for github.com/golang/protobuf to 2bba0603135d7d7f5cb73b2125beeda19c09f4ef.
[INFO]  --> Setting version for github.com/Azure/azure-storage-go to 7c36d244fc8bf27bcf9785b0b37d6beef2949296.
[INFO]  --> Setting version for golang.org/x/text to f4b4367115ec2de254587813edaa901bc1c723a8.
[INFO]  --> Setting version for github.com/google/uuid to 6a5e28554805e78ea6141142aba763936c4761c0.
[INFO]  --> Setting version for github.com/onsi/gomega to 334b8f472b3af5d541c5642701c1e29e2126f486.
[INFO]  --> Setting version for github.com/onsi/ginkgo to 77a8c1e5c40d6bb6c5eb4dd4bdce9763564f6298.
[INFO]  --> Setting version for golang.org/x/oauth2 to 7fdf09982454086d5570c7db3e11f360194830ca.
[INFO]  --> Setting version for gopkg.in/yaml.v2 to a3f3340b5840cee44f372bddb5880fcbc419b46a.
[INFO]  --> Setting version for code.cloudfoundry.org/clock to 2269160ae1757f96bbb8c6475e6fa36c805e73e0.
[INFO]  --> Setting version for golang.org/x/sys to 7de4796419dc3b554e6dab3a119a62469569d299.
[INFO]  --> Setting version for golang.org/x/net to ffcf1bedda3b04ebb15a168a59800a73d6dc0f4d.
[INFO]  --> Setting version for google.golang.org/appengine to 56d253d1dd14aa01937e12c73a0971bcfd797ff2.
[INFO]  --> Setting version for google.golang.org/grpc to b5071124392bfd416c713e6595ac149b387f7186.
[INFO]  --> Setting version for google.golang.org/api to 48e49d1645e228d1c50c3d54fb476b2224477303.
[INFO]  --> Setting version for cloud.google.com/go to 675fad27ef35e552d3708858ff337d54b3b6f4d3.
[INFO]  Exporting resolved dependencies...
[INFO]  --> Exporting github.com/Azure/go-autorest
[INFO]  --> Exporting github.com/dgrijalva/jwt-go
[INFO]  --> Exporting github.com/googleapis/gax-go
[INFO]  --> Exporting github.com/aws/aws-sdk-go
[INFO]  --> Exporting github.com/go-ini/ini
[INFO]  --> Exporting github.com/Azure/azure-sdk-for-go
[INFO]  --> Exporting github.com/Azure/azure-storage-go
[INFO]  --> Exporting golang.org/x/text
[INFO]  --> Exporting github.com/pkg/errors
[INFO]  --> Exporting github.com/google/go-genproto
[INFO]  --> Exporting github.com/google/uuid
[INFO]  --> Exporting github.com/golang/protobuf
[INFO]  --> Exporting github.com/onsi/gomega
[INFO]  --> Exporting github.com/onsi/ginkgo
[INFO]  --> Exporting github.com/jmespath/go-jmespath
[INFO]  --> Exporting github.com/jessevdk/go-flags
[INFO]  --> Exporting golang.org/x/net
[INFO]  --> Exporting google.golang.org/grpc
[INFO]  --> Exporting gopkg.in/yaml.v2
[INFO]  --> Exporting google.golang.org/appengine
[INFO]  --> Exporting golang.org/x/sys
[INFO]  --> Exporting code.cloudfoundry.org/clock
[INFO]  --> Exporting golang.org/x/oauth2
[INFO]  --> Exporting google.golang.org/api
[INFO]  --> Exporting cloud.google.com/go
[INFO]  Replacing existing vendor dependencies
$ echo $?
0

@RamXX we'll close this issue, unless you tell us you still see this happening with the latest cliaas

As of today with latest release, the error is still happening:
go get github.com/pivotal-cf/cliaas && cd $GOPATH/src/github.com/pivotal-cf/cliaas && glide install && go install github.com/pivotal-cf/cliaas/cmd/cliaas
# github.com/Azure/azure-sdk-for-go/arm/examples/helpers
/opt/go/src/github.com/Azure/azure-sdk-for-go/arm/examples/helpers/helpers.go:23: undefined: azure.ServicePrincipalToken
/opt/go/src/github.com/Azure/azure-sdk-for-go/arm/examples/helpers/helpers.go:24: azure.PublicCloud.OAuthConfigForTenant undefined (type azure.Environment has no field or method OAuthConfigForTenant)
/opt/go/src/github.com/Azure/azure-sdk-for-go/arm/examples/helpers/helpers.go:28: undefined: azure.NewServicePrincipalToken

I'll opt for downloading a pre-built binary for cfjump instead of building from source. You are welcome to git clone https://github.com/RamXX/cfjump.git and attempt to reproduce it that way. Maybe I'm missing some package that doesn't get downloaded by go get, or maybe the azure client I already have installed somehow produces conflict. I'll just use the binary going forward.

glide install only installs dependencies to vendor/ of the directory they're for. That compilation error is coming from a dependency in your GOPATH. What version of Go are you using? And what do you have in vendor/?

I'm using Go 1.8.1. Bear in mind this is the first go app that gets built in the workflow after the installation of the Golang environment and the setting up of $GOPATH and $GOBIN. Indeed the error occurs in go get github.com/pivotal-cf/cliaas. Glide doesn't get involved yet. Since I'm now downloading the final release binary, this is no longer an issue for me, so you may decide to close the issue or explore further. Happy to collaborate either way.

This problem is occurring for you because of two things:

go get by default downloads dependencies of what you want and tries to install them and
you're using && to chain these things together, which doesn't work because of the aforementioned behavior

One solution is to add -d to your go get:
go get -d github.com/pivotal-cf/cliaas

This will inform go get not to install the dependent packages.
However, you probably don't want the rest of the dependencies there anyway because the project uses glide, which means go get will likely break at some point in the exact way it has broken for you here, which is that the version of azure-sdk-for-go that we use is different from HEAD of the repo.
A better thing for you to do would be to do a shallow clone of just cliaas:
curl https://glide.sh/get | sh
mkdir -p $GOPATH/src/github.com/pivotal-cf
git clone --depth 1 https://github.com/pivotal-cf/cliaas $GOPATH/src/github.com/pivotal-cf/cliaas
cd $GOPATH/src/github.com/pivotal-cf/cliaas
glide install
go install github.com/pivotal-cf/cliaas/cmd/cliaas

This workflow works, as the following demonstrates:
$ docker run -ti golang bash
root@c15e6f7cfd7d:/go# curl https://glide.sh/get | sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3242  100  3242    0     0  28661      0 --:--:-- --:--:-- --:--:-- 28946
Downloading https://github.com/Masterminds/glide/releases/download/v0.12.3/glide-v0.12.3-linux-amd64.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   608    0   608    0     0   2208      0 --:--:-- --:--:-- --:--:--  2210
100 3344k  100 3344k    0     0  1701k      0  0:00:01  0:00:01 --:--:-- 2301k
glide version v0.12.3 installed successfully
root@c15e6f7cfd7d:/go# mkdir -p $GOPATH/src/github.com/pivotal-cf
root@c15e6f7cfd7d:/go# git clone https://github.com/pivotal-cf/cliaas $GOPATH/src/github.com/pivotal-cf/cliaas
Cloning into '/go/src/github.com/pivotal-cf/cliaas'...
remote: Counting objects: 1160, done.
remote: Compressing objects: 100% (34/34), done.
remote: Total 1160 (delta 8), reused 0 (delta 0), pack-reused 1114
Receiving objects: 100% (1160/1160), 206.82 KiB | 0 bytes/s, done.
Resolving deltas: 100% (616/616), done.
Checking connectivity... done.
root@c15e6f7cfd7d:/go# cd $GOPATH/src/github.com/pivotal-cf/cliaas
root@c15e6f7cfd7d:/go/src/github.com/pivotal-cf/cliaas# glide install
[INFO]	Downloading dependencies. Please wait...
[INFO]	--> Fetching cloud.google.com/go.
[INFO]	--> Fetching code.cloudfoundry.org/clock.
[INFO]	--> Fetching golang.org/x/text.
...snip...
[INFO]	--> Exporting golang.org/x/sys
[INFO]	--> Exporting cloud.google.com/go
[INFO]	Replacing existing vendor dependencies
root@c15e6f7cfd7d:/go/src/github.com/pivotal-cf/cliaas# go install github.com/pivotal-cf/cliaas/cmd/cliaas
root@c15e6f7cfd7d:/go/src/github.com/pivotal-cf/cliaas# which cliaas
/go/bin/cliaas

This would avoid downloading useless dependencies that you don't need and avoid the compilation error.

I've changed the README to suit: https://github.com/pivotal-cf/cliaas/commit/2eb8287265ef49f3057eec45210f7bf8145e08a6


Implement support to variable STREAM_OUTPUT_BODY inside libmodsecurity

More information about STREAM_OUTPUT_BODY can be found at ModSecurity reference manual: https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual

As of now, we don't have a proper way to handle the content in a stream manner. It won't be different from use others variables that represents the same data. Therefore I am marking this as won't implement. At least for this next version.

Hi @zimmerle,
Any update on this variable as this is holding me to use libmodsecurity as I have many response based rules which are dependent on this directive (STREAM_OUTPUT_BODY). Any alternative for same is also okey for me.


Release the 3.2.0 version

Codecov Report

Merging #294 (a9cf887) into master (62381e8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #294   +/-   ##
=======================================
  Coverage   95.17%   95.17%
=======================================
  Files          12       12
  Lines         725      725
=======================================
  Hits          690      690
  Misses         35       35

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 62381e8...a9cf887. Read the comment docs.


Set minimal permissions to Github Workflows

Closes #3971
Set top level permissions to all workflows and grant write permission on run level
Tests:

CIFuzz (cifuzz.yml): success https://github.com/joycebrum/json/actions/runs/4377439445
Code scanning - action (codeql.yml): success https://github.com/joycebrum/json/actions/runs/4377391716
macOS (macos.yml): success https://github.com/joycebrum/json/actions/runs/4377439453
Publish documentation (publish_documentation.yml): not running but it does not to seem due to missing GITHUB_TOKEN permissions. Is there a way to test it better on the main repo?
Pull Request Labeler (.github/workflows/labeler.yml): success https://github.com/joycebrum/json/actions/runs/4377439245
Ubuntu (ubuntu.yml): success https://github.com/joycebrum/json/actions/runs/4377391705. The coveralls was also able to comment https://github.com/joycebrum/json/pull/4#issuecomment-1462667118
Windows (windows.yml): success https://github.com/joycebrum/json/actions/runs/4377439449/jobs/7660926326

Pull request checklist
Read the Contribution Guidelines for detailed information.

[x]  Changes are described in the pull request, or an existing issue is referenced.
[x]  The test suite compiles and runs without error.
[x]  Code coverage is 100%. Test cases can be added by editing the test suite.
[x]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header files single_include/nlohmann/json.hpp and single_include/nlohmann/json_fwd.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for this kind of bug). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Coverage: 100.0%. Remained the same when pulling bb3c7cbd0a5373db8d06ae215aa3b828258a2b6d on joycebrum:develop into bbe337c3a30d5f6eea418b4aee399525536de37a on nlohmann:develop.

Thanks!


Not working when connected to multiple I2C devices

I have connected the module to an Arduino I2C bus. Although it works if it is the only device in the bus, it doesn't seem to be working when multiple devices (NOT multiple APDS-9900, but multiple I2C devices, one of which is APDS-9900) are connected at the same time. What could be the issue?
It either gives me unexpected values or gives me an reading error.

@dingdinglhz I don't have time to check this right now. I have a ton of exams to give at the university. However, check:

that you're using an APDS-9930 and not an APDS-9900 as you mentioned in your post; this library only works with the 9930, but may work with other models with some adaptations (it's actually a fork of the 9960 library)
that there are no i²c address collisions (in that case you can modify the jumper configuration in the expansion board to select another address; refer to the datasheets)
that both devices work by themselves (and that the other device works when wired together with another device)
that the wiring is correct and that any jumper cables or solder are good
that the Wire (i²c) library you're using is updated: maybe they have a fix for that

On the i²c side, the library uses basic i²c read/write commands. The library is just a wrapper around all the complicated binary commands that you actually need to send to get the sensor up and running.
I'm quite sure it's not related to this library itself but to something else, but I'll leave this issue open as I'm not sure and cannot test at the moment.
If you have the possibility, try the two devices with some other i²c master (e.g a more powerful AVR MCU - Arduino Mega and similar - or a Linux computer: your PC probably does have an i²c port, it's just a bit hidden 😉 http://davideddu.org/blog/posts/graphics-card-i2c-port-howto/ )
I made a Python library for both the APDS 9930 and the i²c LCD backpack for Linux. Check my blog ( https://davideddu.org/blog/posts/apds-9930-python-module/ ) and my GitHub @Depaulicious
Keep me up to date :)

Sorry I forgot to reply, it was quite a while ago actually.
I discovered in the end that it was an address collision. I did not check the address carefully and confused myself between hex and dec representation.
And later I also replaced it with a APDS-9930 (for a different issue), and discovered that these two model have similar interface, but the gives quite different reading values. For my application the 9930 gives more stable and reliable readings.
And thank you very much for the work on this library! ヾ(◍°∇°◍)ﾉﾞ

That's fine :)
Thanks for contributing, feel free to send pull requests if you notice any issue.


e1f7bf commit breaks get_buffer_name calls

environment

vim: neovim 0.1.5
vim-airline: master (2016-02-07)
OS: Linux
if you are using terminal:
terminal: gnome terminal
$TERM variable: xterm
color configuration (:set t_Co?):

actual behavior
Opening a file prompts a display of errors warning about insufficient numbers of parameters provided to get_buffer_name due to calls in extensions/tabline/buffers.vim to that function.
expected behavior
No errors are displayed.

I have the same problem on:

vim 7.4.1967
vim-airline: master (2016-02-07)
macOS
iTerm2

+1 same problem here

this is my error msg

+1 same problem

+1 with:
let g:airline_powerline_fonts=1
let g:airline_theme='bubblegum'
let g:airline#extensions#tabline#enabled = 1
let g:airline#extensions#tabline#show_buffers = 0
let g:airline#extensions#tabline#show_splits = 1
let g:airline#extensions#tabline#show_tabs = 1
let g:airline#extensions#tabline#show_tab_nr = 0
let g:airline#extensions#tabline#show_tab_type = 0
let g:airline#extensions#tabline#show_close_button = 0
let g:airline#extensions#tabline#fnamemod = ':t'
let g:airline#extensions#hunks#enabled = 0

disable tabline by set
let g:airline#extensions#tabline#enabled = 0
will fix it temporarily

@drselump14 this bug will be fixed, no need disable tabline feature.

Sorry for breaking this.
https://github.com/vim-airline/vim-airline/pull/1205 should fix it.

But you should be aware of that all those callers do not benefit from the actual fix from #1189: if you have e.g. 100+ buffers open, the formatter might be quite expensive CPU wise.

FWIW: I've stopped using airline myself after trying to fix performance bottlenecks, and #1189 was one of them.

+1 same problem

Please try #1205.

And btw: screenshots instead of text for tracebacks are only a bit more useful than +1 comments.

Works again, thanks. :+1:


Add config file authentication to oci object storage state store

Description
A very common way for providing configuration settings to tools (CLI, Terraform, Ansible, SDKs) and applications working with Oracle Cloud Infrastructure is through a configuration file (typically called config and located in ~/,oci/config) that contains the tenancy details and credentials for connecting to OCI (APIs). With this PR, the OCI ObjectStorage state store allows this type of configuration method.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[1428]
https://github.com/dapr/components-contrib/issues/1428
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[X] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[2073]

Fixed documentation in https://github.com/dapr/docs/pull/2165

Thanks @lucasjellema. Were you able to test the component to ensure no regressions? asking because we don't (yet) have conformance tests for it.

Thanks @lucasjellema. Were you able to test the component to ensure no regressions? asking because we don't (yet) have conformance tests for it.

I did and there is no regression. I use a local integration test , that I will commit as well (however - you will need an OCI account to run it). Do you have instructions on creating a conformance test? I am willing to try to create one.

You can refer to other conformance tests to create it.

LGTM overall.
/cc @yaron2 as it is still alpha, we can merge it and add conformance tests later, WDYT?

Agree

Thanks for the contribution @lucasjellema

@lucasjellema please create an issue to track conformance tests.

@lucasjellema please create an issue to track conformance tests.

Done: https://github.com/dapr/components-contrib/issues/1475


Double encoding in JWKS endpoint

Description
Hi Team,
We're encoding the certificate thumbprint which is already encoded. This is the same issue identified in https://github.com/wso2/product-is/issues/14899. Need to port the IAM fix to the APIM side.
Thanks,
Methma
Steps to Reproduce
Below is the my WSO2 APIm 4.2.0 certificate fingerprints in the keystore,
Alias name: wso2carbon
Creation date: Feb 22, 2023
Entry type: trustedCertEntry
Owner: CN=localhost, OU=WSO2, O=WSO2, L=Mountain View, ST=CA, C=US
Issuer: CN=localhost, OU=WSO2, O=WSO2, L=Mountain View, ST=CA, C=US
Serial number: 63f65a03
Valid from: Wed Feb 22 23:38:03 IST 2023 until: Tue May 27 23:38:03 IST 2025
Certificate fingerprints:
MD5: 44:CA:B5:1B:F3:48:DE:D0:B6:FB:21:79:D7:7B:55:4E
SHA1: 81:7C:FA:10:C0:38:E0:F2:02:C9:BA:26:B4:AA:6E:8B:2F:E1:5A:7A
SHA256: 1B:24:29:B1:6E:F5:83:01:B7:4F:F1:62:53:65:5A:E8:55:37:E3:A5:BF:D1:59:08:5A:5A:8E:23:E5:A3:B2:2A
Signature algorithm name: SHA256withRSA
Subject Public Key Algorithm: 2048-bit RSA key
Version: 3
So now when we comparing the SHA256 fingerprint in my keystore with the the x5ts256 field of the JWKS:
I take the SHA256 fingerprint, remove the ":" and convert to lowercase:
SHA256: 1b2429b16ef58301b74ff16253655ae85537e3a5bfd159085a5a8e23e5a3b22a
Then I base64 encode it twice, I can get to the same value as the x5t#S256 value in the JWKS.
This is the x5t#S256 value from the JWKS,
"x5t#S256":"TVdJeU5ESTVZakUyWldZMU9ETXdNV0kzTkdabU1UWXlOVE0yTlRWaFpUZzFOVE0zWlROaE5XSm1aREUxT1RBNE5XRTFZVGhsTWpObE5XRXpZakl5WVE",
And this is the value of the SHA256 fingerprint with encoded twice,
"TVdJeU5ESTVZakUyWldZMU9ETXdNV0kzTkdabU1UWXlOVE0yTlRWaFpUZzFOVE0zWlROaE5XSm1aREUxT1RBNE5XRTFZVGhsTWpObE5XRXpZakl5WVE="
Affected Component
APIM
Version
4.2.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

This issue does not affect the master branch.


Get rid of tiles.py and image_filter.py

Can we delete these 2 files from the repo?


Fixed link in CSV description for UPI installs

The link for the VPC and subnet requirements were broken in the CSV after refactoring the documentation.

/assign @thejasn

/lgtm

/label docs-approved
/label px-approved
/label qe-approved


Sharing some numbers

I figure those are for the "project page" here on GitHub?

Yes these are the page views for the README

Thanks for sharing. It would be great if there was a way to make the information public but after doing some research it looks like Google Analytics only supports template sharing or exporting via an API =(


Fix Exception "Collection was modified"

It leads to Exception "Collection was modified; enumeration operation may not execute." if during execution of InvokeAsync, RemoveHandler is called.
This is the case if we use RxMqttNet.

I assume we have to lock the access to the internal list because it might happen that the collection gets removed when the array is being constructed and all items are being copied over. This might be a very rare case but I am afraid this will happen. So in my opinion we should lock the access to the list, create a copy and then iterate over the copy. All other actions like removal or adding must also be locked.

Another solution would be an ImmutableArray. Which you would prefer?

I am wondering how to get from the mutable list to an immutable one. This always requires locking in my opinion. I will create a branch to show exactly what my idea is.

@wdichler Please checkout https://github.com/dotnet/MQTTnet/pull/1515. I found a solution which is lock less when reading the handlers and will not create a copy every time.


The meaning of the grad on the LieTensor.SO3

📚 The doc issue
After performing autograd on a LieTensor SO3 tensor T, is the resulting T.grad always a LieTensor as well? When the optimizer apply the update to the LieTensor, is the update operation on the manifold?
I ran into such a situation after running loss.backward():
(Pdb) p T_init
SO3Type LieTensor:
LieTensor([[-1.4035e-03, -1.2569e-04,  1.1781e-02,  9.9993e-01]],
          device='cuda:0', requires_grad=True)
(Pdb) p T_init.grad
tensor([[-1.3532, -0.0474, -0.4073,  0.0000]], device='cuda:0')

Where the T_init.grad's data doesn't look like a quaternion any more. Is this value directly added on the T_init's data, or is it converted to a quaternion first (something like SO3(T_init.grad)) then multipled to the T_init?
Suggest a potential alternative/fix
No response

You need to call LieTensor.add to update lietensor by grad. Here is more detailed mathematical definition for gradient of Lie Groups


sensors: move mag voter to new VehicleMagnetometer WorkItem

This PR removes the magnetometer aggregation and voting from the sensors module main loop and puts it in a new isolated WorkItem that's scheduled with the publication of the current primary magnetometer.
@MaEtUgR I think this would be an acceptable place to do additional optional mag processing as it won't directly interfere with the IMU pipeline to the estimator. I wasn't going to pursue this until post v1.11, but opening as a PR now to possibly enable other related work.

Continued in https://github.com/PX4/Firmware/pull/15235.


Dependency resolution does not work for on-premise TFS private feed

I encountered with the same issue as described in #1798 but for TFS. TFS has the same bug with package dependencies as VSTS (I create issue for this here).
I found this code to workaround for VSTS:
https://github.com/fsprojects/Paket/blob/master/src/Paket.Core/NuGetV2.fs#L682
and this:
https://github.com/fsprojects/Paket/blob/master/src/Paket.Core/NuGetV2.fs#L296
There are IF conditions for visualstudio.com.
But I have URLs something like this:
"http://vc-tfs-server:8080/tfs/DefaultCollection/_packaging/default/nuget/v3/index.json" - in first case (L682) and:
"http://vc-tfs-server:8080/tfs/DefaultCollection/_packaging/default/nuget/v2" - in second case (L296)
Can you fix it for me? Or I can fix this for both cases and send you pull request. What do you think about it?

I'd prefer a pr
Am 31.03.2017 2:53 nachm. schrieb "drwatson1"<EMAIL_ADDRESS>
I encountered with the same issue as described in #1798
https://github.com/fsprojects/Paket/issues/1798 but for TFS. TFS has
the same bug with package dependencies as VSTS (I create issue for this
here
https://developercommunity.visualstudio.com/content/problem/38370/package-management-server-does-not-return-package.html
).
I found this code to workaround for VSTS:
https://github.com/fsprojects/Paket/blob/master/src/Paket.
Core/NuGetV2.fs#L682
and this:
https://github.com/fsprojects/Paket/blob/master/src/Paket.
Core/NuGetV2.fs#L296
There are IF conditions for visualstudio.com.
But I have URLs something like this:
"http://vc-tfs-server:8080/tfs/DefaultCollection/_
packaging/default/nuget/v3/index.json" - in first case (L682) and:
"http://vc-tfs-server:8080/tfs/DefaultCollection/_
packaging/default/nuget/v2" - in second case (L296)
Can you fix it for me? Or I can fix this for both cases and send you pull
request. What do you think about it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2213, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNFqO_NxIBAIb7zu5mHCgNYbv3cHQks5rrPdEgaJpZM4MvoRe
.


Add riak and airship

Will squash these two commits, if / when it builds.

@tmcgilchrist note that I've disabled riak, since riak-protobuf still has issues with ghc 8.2.
https://github.com/fpco/stackage/commit/313dc7cd3c55ea6dad1ed51af596b523acc15d13
https://github.com/riak-haskell-client/riak-haskell-client/issues/98


Swallow fingerprint from css asset urls

Partly closes #47.
This PR makes the CSS compiler regex swallow the fingerprint after ? in asset urls.
Benchmark indicates a 3% slowdown.

Not enough to fully fix the issue.
Currently using this issue-47 branch from https://github.com/rails/propshaft/pull/50.
This is what I get in my error logs:
Unable to resolve 'fa-brands-400.svg#fontawesome' for missing asset 'fa-brands-400.svg#fontawesome' in page.css
Unable to resolve 'fa-regular-400.svg#fontawesome' for missing asset 'fa-regular-400.svg#fontawesome' in page.css
Unable to resolve 'fa-solid-900.svg#fontawesome' for missing asset 'fa-solid-900.svg#fontawesome' in page.css
Unable to resolve 'et-line.svg#et-line' for missing asset 'et-line.svg#et-line' in page.css
Unable to resolve 'et-line.svg#et-line' for missing asset 'et-line.svg#et-line' in page.css
Unable to resolve 'fa-brands-400.svg#fontawesome' for missing asset 'fa-brands-400.svg#fontawesome' in page.css
Unable to resolve 'fa-regular-400.svg#fontawesome' for missing asset 'fa-regular-400.svg#fontawesome' in page.css
Unable to resolve 'fa-solid-900.svg#fontawesome' for missing asset 'fa-solid-900.svg#fontawesome' in page.css
Unable to resolve 'et-line.svg#et-line' for missing asset 'et-line.svg#et-line' in page.css
Unable to resolve 'et-line.svg#et-line' for missing asset 'et-line.svg#et-line' in page.css

By using the main branch:
Unable to resolve 'fa-brands-400.eot?#iefix' for missing asset 'fa-brands-400.eot?#iefix' in page.css
Unable to resolve 'fa-brands-400.svg#fontawesome' for missing asset 'fa-brands-400.svg#fontawesome' in page.css
Unable to resolve 'fa-regular-400.eot?#iefix' for missing asset 'fa-regular-400.eot?#iefix' in page.css
Unable to resolve 'fa-regular-400.svg#fontawesome' for missing asset 'fa-regular-400.svg#fontawesome' in page.css
Unable to resolve 'fa-solid-900.eot?#iefix' for missing asset 'fa-solid-900.eot?#iefix' in page.css
Unable to resolve 'fa-solid-900.svg#fontawesome' for missing asset 'fa-solid-900.svg#fontawesome' in page.css
Unable to resolve 'et-line.eot?#iefix' for missing asset 'et-line.eot?#iefix' in page.css
Unable to resolve 'et-line.svg#et-line' for missing asset 'et-line.svg#et-line' in page.css
Unable to resolve 'et-line.eot?#iefix' for missing asset 'et-line.eot?#iefix' in page.css
Unable to resolve 'et-line.svg#et-line' for missing asset 'et-line.svg#et-line' in page.css
Unable to resolve 'themify.eot?-fvbane' for missing asset 'themify.eot?-fvbane' in page.css
Unable to resolve 'themify.eot?#iefix-fvbane' for missing asset 'themify.eot?#iefix-fvbane' in page.css
Unable to resolve 'themify.woff?-fvbane' for missing asset 'themify.woff?-fvbane' in page.css
Unable to resolve 'themify.ttf?-fvbane' for missing asset 'themify.ttf?-fvbane' in page.css
Unable to resolve 'themify.svg?-fvbane#themify' for missing asset 'themify.svg?-fvbane#themify' in page.css
Unable to resolve 'fa-brands-400.eot?#iefix' for missing asset 'fa-brands-400.eot?#iefix' in page.css
Unable to resolve 'fa-brands-400.svg#fontawesome' for missing asset 'fa-brands-400.svg#fontawesome' in page.css
Unable to resolve 'fa-regular-400.eot?#iefix' for missing asset 'fa-regular-400.eot?#iefix' in page.css
Unable to resolve 'fa-regular-400.svg#fontawesome' for missing asset 'fa-regular-400.svg#fontawesome' in page.css
Unable to resolve 'fa-solid-900.eot?#iefix' for missing asset 'fa-solid-900.eot?#iefix' in page.css
Unable to resolve 'fa-solid-900.svg#fontawesome' for missing asset 'fa-solid-900.svg#fontawesome' in page.css
Unable to resolve 'et-line.eot?#iefix' for missing asset 'et-line.eot?#iefix' in page.css
Unable to resolve 'et-line.svg#et-line' for missing asset 'et-line.svg#et-line' in page.css
Unable to resolve 'et-line.eot?#iefix' for missing asset 'et-line.eot?#iefix' in page.css
Unable to resolve 'et-line.svg#et-line' for missing asset 'et-line.svg#et-line' in page.css
Unable to resolve 'themify.eot?-fvbane' for missing asset 'themify.eot?-fvbane' in page.css
Unable to resolve 'themify.eot?#iefix-fvbane' for missing asset 'themify.eot?#iefix-fvbane' in page.css
Unable to resolve 'themify.woff?-fvbane' for missing asset 'themify.woff?-fvbane' in page.css
Unable to resolve 'themify.ttf?-fvbane' for missing asset 'themify.ttf?-fvbane' in page.css
Unable to resolve 'themify.svg?-fvbane#themify' for missing asset 'themify.svg?-fvbane#themify' in page.css

The ones with the number sign without the question mark are not taken into account.

Updated to add the # test. @HLFH could you check again please?

@brenogazzola Things running well for me now!
Sample with propshaft (lastest release with ASSET_URL_PATTERN override): http://localhost:3000/assets/fonts/bootstrap-icons-1ca5e8c7d2fb8e9d60ad1a1feb2a46e98c248a3d.woff2 => 200 OK
Sample with sprockets: http://localhost:3000/assets/fonts/bootstrap-icons-4aee98800746975593dace855d8dc1b9101f7b66df718a65812d1caae06579a0.woff2?524846017b983fc8ded9325d94ed40f3 => 200 OK
Thx

@brenogazzola Works well. Thank you.


added a dependency on "formsy-react"

Thanks for the PR. I needed to modify the version range to support formsy-react 0.14.1 for React 0.13 support (primarily for Meteor), so added this manually


Update fee_balancer.py

Time spent on this PR: 0.2 days
Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[x] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

The current code has several style and type safety issues:

Uses float division for fee calculations
Lacks proper type hints
Has formatting issues with long lines

What is the new behavior?

Changed float division (/) to integer division (//) for more precise fee calculations
Added proper type hints to get_total_balance_of_relayers function for better type safety
Improved code formatting by breaking long lines according to PEP 8 guidelines
Enhanced readability by properly structuring function parameters

These changes improve code quality and maintainability without changing the core functionality.

@ClementWalter
Fixed integer division (//)
Updated function params and type hints


Count of total matches is incorrect when cpo-c is not present

repro instructions:

:set cpoptions-=c
5iab
/abab
N

observe the count reads [4/2] rather than [4/4]

I've looked into that and it seems like there's no way to get [4/4] result. The problem is that vim-searchindex uses :%s/pattern//n Ex command internally, and this command does not provide appropriate option. But this is also the only search command that works efficiently on large line ranges, so it cannot be replaced.
I can however implement a partial fix, which will at least make numbers self-consistent. I could make all match counts work as if cpoptions contained c. This means that search counts will be inconsistent with cursor jumps, displaying [2/2] in your case - which still makes more sense than [4/2].
What do you think?  Sorry that there's no better fix, but Vim's searching functions are a mess, and much of complexity in vim-searchindex is working around their quirks.

Hmm, would it be sensible to file an upstream feature request with Ex?
I slightly prefer [4/2] to [2/2] as the result of the above instructions. If we used nnn instead of N, I think it is better to have the first number increment with each press.
Also in practice I don't think cpo-c has ever made a difference in my Vim usage. I suspect very few users will ever come across this bug unless they contrive a situation to check for it like I did.

Vim is a fragile legacy codebase (still written in pre-ANSI C), and does not have maintainers to implement such feature requests. So, I don't consider changing it a realistic option.
Since we can't solve this problem, I can show you another fun thing in exchange:
set shortmess+=s
/some_non_existing_text<CR>

Now error message is two-line long, requiring you to press enter. How does it work, since "shortmess" seems unrelated? I have no idea :)


Enable MariaDB read-write splitting tests

Summary
Enable MariaDB read-write splitting tests

Will reopen against awslabs/main


Preferred way of subscribing to events

Hello,
I'm writing because I'm wondering what's the preferred way of subscribing to item-change events. I can see two methods:

To go via Amplify.DataStore.publisher as described here
Get notified via Amplify.Hub.listen, specifically HubPayload.EventName.DataStore.syncReceived, as described here

Currently, my project uses both as I believe method 1. was available before method 2. However, that seems like unnecessary overhead and I wonder whether there's any disadvantages just going with method 2, as I'll need Amplify.Hub.listen anyways.
Can any of the two methods be configured such that they don't fire on self-initiated events? Suppose, I call Amplify.DataStore.delete, then I get an event shortly afterwards (via both methods). Currently, I'm using a hand-written filter around the events to distinguish which ones are self-made and which do actually come from another client. However, this doesn't seem right...
Any hints are greatly appreciated.
Thank you!

Hi @anatomybook,
yes you're correct, we added support for DataStore hub events later.
I believe there's no advantage of one over the other, but it's more a matter of APIs availability and app architecture.
Amplify.DataStore.publisher is built on top of Apple's Combine thus is only available on iOS 13 or higher, while Amplify.Hub.listen is available from iOS 11 (there's also a Amplify.Hub.publisher Combine version).
So based on your app availability you may decide to use one or the other.
Amplify.Hub provides a central place to handle events across different Amplify categories, so you may decide to have a centralized logic to handle events vs a "distributed" approach or use both based your app architecture.
You should also consider the type and granularity of events you're interested in.
Unfortunately there's no configuration to filter out self-initiated events, but it'd be helpful to know what's your use case.

Thanks for your answer @diegocstn
Then I'll probably go the Amplify.Hub way.
My case is the following:
First I should add, that my DataStore structure is a tree, i.e. every user has a root object, that contains all the other items.
After trying out various abstraction layers, I found that keeping extra layers in sync is a pain and not worth the benefit. Hence, my SwiftUI-Interface rests on a single DataStore root item, that is @Published. If the UI needs refresh, I re-query that item and everything re-draws. So far, so convenient.
In order to avoid unnecessary refreshes, I need to filter self-initiated events. So from my point of view, I'm only interested in events, that originate from other clients - since only then the GUI needs to be made aware of underlying changes.
Does this make sense?
Thanks again for your answer.


WIP: Dockerfile for ray-node arm64

Summary
Add a new Docker file that builds ray-node container for linux/arm64 platform
Details and comments
The base image of the container should be for linux/arm64 platform.  Ray project started building one in March.

I'll update the Make file to select the correct Dockerfile for the local system.

@Tansito Thanks for review.  Yes, I saw the issue in Ray project.  So I put 2.3.0 in the Dockerfile for the m1 image.  I'll try if I can upgrade for the amd64, too.


Measuring the force a joint or another body exerts on a body

Hiya, is it possible to find the force that was applied from body a to b or joint a to body b on the last tick?
Thanks.

Sorry but it is not currently possible to do that.

Given that I have the whole source, would it be that hard for me to add this myself?  Could you point me in the right direction?

With this commit, I have added methods Joint::getReactionForce() and Joint::getReactionTorque() to retrieve the reaction force/torque between two bodies of a joint.
This is currently included in the "develop" branch but will be part of version v0.9.0 of the library.

The above proposed fix is now part of version v0.9.0 that has just been released.


Make releases faster than before and resolved #490

I have tested github.com/microsoft/vscode. First releases page spent from 5s to 2s now. Of course, it's not the final optimization. We will improve this later.

LGTM

LGTM


WIP

For #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #6323 (7df787f) into main (e5b1854) will decrease coverage by 0%.
The diff coverage is 72%.

@@          Coverage Diff           @@
##            main   #6323    +/-   ##
======================================
- Coverage     72%     71%    -1%
======================================
  Files        401     400     -1
  Lines      27063   27040    -23
  Branches    3988    3999    +11
======================================
- Hits       19543   19354   -189
- Misses      5877    6038   +161
- Partials    1643    1648     +5

Impacted Files
Coverage Δ

src/client/common/application/types.ts
100% <ø> (ø)

src/client/datascience/commands/exportCommands.ts
35% <0%> (ø)

...rc/client/datascience/commands/notebookCommands.ts
62% <ø> (ø)

...ence/interactive-common/intellisense/conversion.ts
61% <ø> (ø)

...ient/datascience/interactive-ipynb/nativeEditor.ts
69% <0%> (-1%)
:arrow_down:

...ience/interactive-ipynb/nativeEditorProviderOld.ts
76% <ø> (+2%)
:arrow_up:

src/client/datascience/notebook/types.ts
100% <ø> (ø)

...ce/notebookStorage/notebookModelSynchronization.ts
31% <0%> (-56%)
:arrow_down:

...science/notebookStorage/notebookStorageProvider.ts
79% <ø> (-12%)
:arrow_down:

src/client/datascience/openNotebookBanner.ts
80% <ø> (ø)

... and 58 more


A "secret" encoder/decoder

ZX spectrum anyone? The code exists already.

I'll leave this open in case we want to add SCR encoding, but I'm pretty happy with that we have now.
If we have SCR encoding we'd need resizing too, but that's a pretty important preprocessor.

*cough*


Remove version: Stevenaaus.ScidvsPC version 4.24

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/scidvspc/files/windows/Scid vs PC-4.24.exe/download
OK
(Automated message - build 923)

Close with reason: Package still available;


Crashes with Sodium 0.4.9

I updated indium to 1.0.13 (the version that says it works with sodium 0.4.9) but my game crashes soon after startup
crash-2023-02-06_21.02.45-client.txt

Updating fapric API fixed it!


No README on npmjs.com

Describe the bug

To Reproduce
Go to https://www.npmjs.com/package/mjml
Expected behavior
I should see the README.md of the repository on the npm package page.

Hi, thanks for reporting this, it will be fixed in next release

It's good now ! Thanks for that 👍


Exception thrown: cannot schedule new futures after interpreter shutdown

Exception thrown: cannot schedule new futures after interpreter shutdown

Still working on this...


moonrise / moonset.

To me the values make no sense, currently what is shown
Sunrise : 7:49
Sunset :  17:59
Moonrise : 5:05
Moonset : 15:07
Surely the moonrise and moonset need to be swapped around.
Basically I would like to know what time the moon will rise (above the horizon) and what time it will set (below the horizon).
type: custom:horizon-card
moon: true
refresh_period: 120
southern_flip: true
fields:
  sunrise: true
  sunset: true
  dawn: true
  noon: true
  dusk: true
  azimuth: true
  sun_azimuth: true
  moon_azimuth: true
  elevation: true
  sun_elevation: true
  moon_elevation: true
  moonrise: true
  moonset: true
  moon_phase: true

No, what you see is correct. Moonrise means "Moon goes above the horizon" and moonset means "Moon goes below the horizon". There are times of the year when the Moon is indeed above the horizon during the day (as happens to be now). Sometimes you can even see the Moon during the day if the conditions are right - position of the Sun, clouds, etc.

at 21h00 would the moon be visible or not ?

Obviously not, since at 15:07 it will go below the horizon. I suggest you play with the setting now in the card config, it allows you to set a fixed time (the Sun and the Moon won't move but will show that exact time), for example (adjust the timezone +0300 to match yours):
Compare this:
now: 2023-07-14T21:00:00+0300

To this:
now: 2023-07-30T21:00:00+0300

Hey @extremeshok, was your question answered? Perhaps you want to propose an addition to the text shown here? https://github.com/rejuvenate/lovelace-horizon-card#visibility-fields
Otherwise please close the ticket. Thanks!


Don't set up css transition callback if there's a js callback.

Fixes #1241
Leave doesn't set up the css callback if there's a js callback, so I did the same for enter.

:+1: nice work


feat: Updated Azure.APIM.MultiRegion

PR Summary
Fixes #2787
Updated Azure.APIM.MultiRegion to improve detection for non-premium SKUs.
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] Change is not breaking
[x] This PR is ready to merge and is not Work in Progress
Rule changes

[x] Unit tests created/ updated
[x] Rule documentation created/ updated
[x] Link to a filed issue
[x] Change log has been updated with change under unreleased section

Other code changes

[ ] Unit tests created/ updated
[ ] Link to a filed issue
[ ] Change log has been updated with change under unreleased section

Hi @BernieWhite. What do you think?


Jormungandr 0.3.9999 > 0.5.0

Issue Number

Overview

[ ] Jormungandr 0.3.9999 > 0.5.0

Comments
There's another release. Apparently there's breaking change so regenerated genesis block.
Integration tests fail though. It seems that all tx are in pending state on the node:
---
- fragment_id: 628fd59e245983e97d6fa30256576beae3cf09c4409686e8204bf81b6c8de558
  last_updated_at: "2019-09-23T21:27:55.236096002+00:00"
  received_at: "2019-09-23T21:27:55.236095959+00:00"
  received_from: Rest
  status: Pending

bors try

bors try

bors try

It wasn't creating blocks, because I haven't updated secret.yaml before.

bors r+

bors r+


Unify transactions and block structures between core and wallet packages

This would save us from duplicate code, and conversion issues when the wallet package is externalized.

This is needed for #47, as it makes the transition a lot smoother.


Fix use of glXChooseFBConfig.

Based on https://www.opengl.org/sdk/docs/man2/xhtml/glXChooseFBConfig.xml
the last parameter of the attributes array must be None (which == 0).
Confusingly, this is different from GLX_NONE, which is non-zero.
This fixes webgl context creation on some drivers, in particular nVidia proprietary drivers but probably others too.

Wow, good catch.
I found this GLX_NONE/None behaviour completely surprising, and this explains Servo's buildbots crashing... Probably Mesa drivers do ignore invalid attributes while nvidia's reject them...
Thanks! :)


My9074/unit test

Current coverage is 45.90% (diff: 100%)

Merging #180 into master will increase coverage by 1.91%

@@             master       #180   diff @@
==========================================
  Files            16         16
  Lines          8925       8925
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3926       4097   +171
+ Misses         3988       3789   -199
- Partials       1011       1039    +28

Powered by Codecov. Last update 087b215...3083312


NF90_HDF5 is deprecated

In pFIO we have:
https://github.com/GEOS-ESM/MAPL/blob/561852aeaafa2f1361c51686b776bdae61d5f42b/GMAO_pFIO/NetCDF4_FileFormatter.F90#L136
But per netCDF:

cmode
: The creation mode flag. The following flags are available: NF90_CLOBBER, NF90_NOCLOBBER, NF90_SHARE, NF90_64BIT_OFFSET, NF90_NETCDF4, and NF90_CLASSIC_MODEL. (NF90_HDF5 is deprecated, use NF90_NETCDF4 instead).

So, we should do so.

I'm assigning just about anyone. The main thing is to make sure it doesn't croak running GEOS. I can hopefully test and get a PR soon.

Realized I have a MAPL develop build around. I can test pretty quickly


More flexibility for defining the locations of assets packaged in an FLX

Add a map of relative to absolute paths for assets located outside the app's
source directory
If a "packages" directory exists, obtain assets from there instead of
using the packages/ prefix to indicate package map lookup

LGTM


Some simple improvements/fixes to the parser

This doesn't improve the situation much, but should help a tiny bit.
As you recognised, there is something wrong with the anonymous functions. I played a bit around with the ordering of the items in "expression" and "expressionItem". Things that can be cheaply rejected should come first, but then order also matters. I didn't dare to actually reorder some of the rules, because I am missing unit tests that would tell me exactly what parts of the langue I break.
One thing that is certainly slowing down the grammar are large overlapping productions like "anonymousFunction" and "simpleAnonymousFunction". They require to read a significant amount of input to be able to decide which function to take. Combining those into one with only the "->" and "=>" as the choice could help.
Similarly, the huge choice switches don't help for speed. Also I am not sure, if they implement what you intend your grammar to do? In fact they parse expressions from left-to-right, no mater the precedence of the mathematical operators. In any case, I suggest that you stack your checks either using the pattern described here (https://github.com/petitparser/dart-petitparser/blob/master/test/core_test.dart#L1334) or use the expression builder as part of your grammar (https://github.com/petitparser/dart-petitparser/blob/master/test/core_test.dart#L1372). In either case you will end up with a much more efficient grammar.

Thanks so much!


Empty nodes are wrongly serialized as null

Describe the issue
GLTF allows nodes to be empty, but not to be null. But tinygltf serializes empty nodes as a json::null object. The underlying reason is that the default constructor for detail::json (be it nlohmann or rapidjson) initializes the json object to a null object.
To Reproduce

OS: all
Compiler, compiler version, compile options: all
Code to reproduce

    tinygltf::Model m{};
    m.nodes.push_back({});
    m.scenes.push_back({});
    m.scenes.front().nodes.push_back(0);
    tinygltf::TinyGLTF writer;
    writer.WriteGltfSceneToFile(&m, "emptynode.gltf", false, false, true, false);

The resulting gltf is invalid:
{
  "asset": {
    "version": "2.0"
  },
  "nodes": [
    null
  ],
  "scenes": [
    {
      "nodes": [
        0
      ]
    }
  ]
}

Expected behaviour
The empty nodes should be serialised as an empty json object, i.e. {} instead of null.
Additional context:
Validation error message:
Incorrect type. Expected "object".
Matches a schema that is not allowed.
Type mismatch. Property value null is not a 'object'.glTF Validator(TYPE_MISMATCH)
Node

A node in the node hierarchy. When the node contains `skin`, all `mesh.primitives` **MUST** contain `JOINTS_0` and `WEIGHTS_0` attributes. A node **MAY** have either a `matrix` or any combination of `translation`/`rotation`/`scale` (TRS) properties. TRS properties are converted to matrices and postmultiplied in the `T * R * S` order to compose the transformation matrix; first the scale is applied to the vertices, then the rotation, and then the translation. If none are provided, the transform is the identity. When a node is targeted for animation (referenced by an animation.channel.target), `matrix` **MUST NOT** be present.

Good catch!
This reminds me this issue: https://github.com/syoyo/tinygltf/pull/295
Probably we also need to apply this fix to other glTF objects: https://registry.khronos.org/glTF/specs/2.0/glTF-2.0.html#reference-gltf

It seems an empty scene causes the same problem.

You can contribute a fix! PR is much appreciated.

Will do.

🙏


python3Packages.icontract: fix build by bumping dependencies

Description of changes
Alternative PR for https://github.com/NixOS/nixpkgs/pull/264772.
Bumped some packages until I got python3Packages.icontract to build again.
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

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

@ofborg build python39Packages.astroid python310Packages.icontract python311Packages.icontract

@ofborg build python39Packages.pylint

From the OfBorg logs this needs lots more work to be done. Will close this PR.


NEXT-13735 Fix DBAL Entity Hydration when one entity is hydrated multiple times with different associations loaded

1. Why is this change necessary?
It is possible that one entity is being hydrated multiple times with differently loaded associations. In these cases the current caching strategy fails to hydrate the entities correctly, because it caches the hydrated entities ONLY by its unique identifier, DISREGARDING the loaded associations of the entity. In situations, where we hydrate the same entity (same ID) two ore more times with the different associations, the entity of the first hydration is REUSED. This can lead to missing association data in the loaded entity.
Use Case
Think of the following situation
PickupPoint
|- id
|- warehouseId
|- warehouse
|- |- id
|- |- zipcodeId
|- |- zipcode
|- |- |- id <--- ID 123
|- |- |- countryId
|- zipcodeId
|- zipcode
|- |- id <--- ID 123
|- |- countryId
|- |- country
|- |- |- id
|- |- |- iso

In the use case above the pickup point and the warehouse share the same zipcode (which in this use case is an entity, not a string). While the Search Query DOES NOT LOAD the country association of the zipcode entity associated with the warehouse, it does load the associated country of the zipcode entity associated with the pickup point.
The above use case is currently not hydrated correctly, since the first time the zipcode is being hydrated, the country data is missing, the country data is also missing for the second zipcode entity, since it is being reused, using the current caching strategy.
2. What does this change do, exactly?
The change changes the way we generate the caching key within the entity hydrator, differentiating between entities that have loaded associations and those who do not.
3. Describe each step to reproduce the issue or behaviour.
See supplied Unit Test
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-13735
5. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hey @AlexBachmann, thanks for your contribution 💙  Your generateCacheKey function will most probably take too much time.
After an internal discussion, we decided that you can take out the caching here. That would be much better performance wise and also fix the issue.

I took out the object caching and added a changelog file

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-13852
Please use this issue to track the state of your pull request.

Hey @AlexBachmann, thanks for your contribution 💙


Add exceptions to team

Codecov Report
Base: 50.33% // Head: 50.24% // Decreases project coverage by -0.08% :warning:

Coverage data is based on head (02ce9c9) compared to base (282d6a6).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #121      +/-   ##
============================================
- Coverage     50.33%   50.24%   -0.09%
  Complexity      406      406
============================================
  Files           100      102       +2
  Lines          1808     1811       +3
  Branches        183      183
============================================
  Hits            910      910
- Misses          858      861       +3
  Partials         40       40

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/model/ModelManager.java
53.33% <0.00%> (ø)

.../java/seedu/address/model/team/UniqueTeamList.java
18.75% <0.00%> (ø)

.../model/team/exceptions/DuplicateTeamException.java
0.00% <0.00%> (ø)

...s/model/team/exceptions/TeamNotFoundException.java
0.00% <0.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Download and convert model

@dkurt

Please keep in the description some code snippet with an example how to use proposed API

see https://github.com/openvinotoolkit/open_model_zoo/pull/1352

please close this PR


use promise-pattern to call commit and rollback

I did clean this up a bit since it was a bit unclear where things ended. I would have been smart to use an http library that supported promises. The mix of callbacks and promises makes this look far worse than it is.


Web - Removing distance default category "Updated" message displayed Instead of "removed"

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

Version Number: v1.4.62-1
Reproducible in staging?: y
Reproducible in production?: y
Issue found when executing PR: https://github.com/Expensify/App/pull/39643
Issue reported by: Applause - Internal Team
Action Performed:

Navigate to /settings/workspaces//distance-rates
Tap Settings
Change the default category
Now uncheck the selected category
Navigate to #admin report of collect Workspace
Note the policy change log message says updated the Distance default category from "..." to "..." instead of "removed distance default category"

Expected Result:
Message "removed distance default category" should be displayed after removing distance default category
Actual Result:
Removing distance default category "Updated" message displayed Instead of "removed"
Workaround:
n/a
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/fcaea6b2-b7a1-41e7-93ee-6eeffff4ac0c

View all open jobs on GitHub

We think that this bug might be related to #wave-control

@stephanieelliott FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

I think this behavior makes sense, I know the " " could seem off but in this context I think it communicates a null value (which it is). So I don't think this is a bug per se and since it's affecting only one platform I don't think it is worth changing.


Want to have log printed with Date

Want to have log printed with Date, current it just print the time ,not print the date+time.

I'm thinking of making the Brolog instance an EventEmitter, and it will emit logs so that the user can use a listener to select how to print their log message with maximum flexibility.
Code might look like this:
log.on('verbose', (date, module, message) => {
  console.info(date, module, message) // any format as you want.
})


User Story #3 - logical structure independent of styling and positioning

GIVEN a webpage meets accessibility standards
WHEN I view the structure of the HTML elements
THEN I find that the elements follow a logical structure independent of styling and positioning

The HTML elements are logical


UTXO details screen- transaction status doesn't update.

Steps to reproduce the behaviour:
wallet A- Desktop wallet
wallet B- Android wallet

Open wallet A and copy the address.
Close wallet A.
Send transaction from wallet B to wallet A(status in wallet B "waiting for receiver").
Open UTXO details screen in wallet B(status "in progress").
Open wallet A.
Expected: updating status in"UTXO details" screen in wallet B once the transaction has received by wallet A.(spent)
Actual: Transaction in wallet A moved to "received" status. Wallet B moved to "sent" status too, but in open "UTXO details" screen transaction status doesn't update(status "in progress").

checked


videogular-youtube - Cannot read property '0' of undefined - onSourceChange(newVal[0].src);

I have installed videogular-youtube and am faced with the following:
TypeError: Cannot read property '0' of undefined
    at Object.fn (vg-youtube.js:121)
    at Scope.$get.Scope.$digest (angular.js:14308)
    at angular.js:14505
    at completeOutstandingRequest (angular.js:4924)
    at angular.js:5312

Note I have even used just the example files provided, (with modification include the correct css file for the theme) but still the same.
I have also tried with code from the latest commits and from the pull requests but faced with the same.
I am using the latest stable versions of angular and videogular.
Is videogular-youtube up to date?

Issue was due to using outdated videogular tags. Seems like it works nicely when using vg-media and not vg-video as per the demo templates.

+1, thanks for the info!

This doesn't work, using the vg-media tags lets me play videos but if I have a youtube url and regular video link it breaks after changing sources and playing the videos.

@JamesBaxta, are you getting the same error as in this issue?
What you are describing sounds more like issue #19 which is linked to pull #20.
Could you post here your config and error?

Docs updated.
Closing this.


Don't mark fields as updated if value didn't change in subscriptions

Is your feature request related to a problem? Please describe.
When updating a node without actually changing the fields, update subscriptions are triggered, and the updatedFields variable contains fields.
Example
subscription:
subscription {
  user(where: {
    mutation_in: [CREATED, UPDATED, DELETED]
    node: {
      id: "cjilqw49tzlp30a86h5to2ogj"
    }
  }) {
    node {
      id
      name
    }
    updatedFields
  }
}

mutations
mutation a {
	createUser(data: {
    name: "Nilan"
  }) {
    id
  }
}

mutation b {
	updateUser(
    data: {
      name: "Nilan"
    }
    where: {
      id: "cjilqw49tzlp30a86h5to2ogj"
    }
  ) {
    id
  }
}

after mutation b, the subscription will trigger this event:
{
  "data": {
    "user": {
      "node": {
        "id": "cjilqw49tzlp30a86h5to2ogj",
        "name": "Nilan"
      },
      "updatedFields": [
        "name"
      ]
    }
  }
}

Describe the solution you'd like
Subscription events should not be triggered if no field of a node changed.
Describe alternatives you've considered
An alternative is to add a flag to the subscription filter to let the user decide whether or not those subscriptions should be fired.
Additional context
Initial Forum discussion.

Naturally I would also expect this to be the default behavior without having to do additional filtering at the subscription level.
Hopefully this will become the default behavior once this issue is considered.

I totally agree, a thing that I have to do manually within my mutations is to check if DateTime fields are updated (since the input string might differ from the actual iso string from the database, for example they might include timezones that prisma automagically takes care of), in order to not trigger any field updates on those fields.
Also when no fields changed at all, the UPDATED subscription still gets called when upserting data, i.e.
...
"mutation": "UPDATED",
"updatedFields": [],
...

I think this causes me to unnessesarly write a lot of extra logic in order to not trigger the mutations (in my query, and in my resolvers to check for DateTime value changes).


Testing pyav

Let's see when it breaks

It seems that other test is causing video ones to fail:

test_io_opt seems to be the culprit for the failure of the tests, I don't know what is the intent of that test

The only effect that set_video_backend('video_reader') has over test_io.py is related to the codecs used to encode/decode the test videos:
https://github.com/pytorch/vision/blob/217e26fcb168aeea00a6341c2a478237f7e75808/test/test_io.py#L45
That means that under the current circumstances test_io is only testing the video_reader backend, rather than the PyAV one.

@andfoy this is great debugging, thanks!
So, test_io_opt is there to test the video_reader backend of read_video, without having to resort to copy-pasting all the tests. If it is negatively interfering with test_io, we should do something to change that.
It is great that we now know the reason why this wasn't working before, now we need to understand why the partial reading of files is not working with the video_reader backend.
@bjuncek you have used the video_reader backend for partial reads recently, right? Can you check again if those tests are passing or not in your system?

@fmassa I pulled your branch and run the tests in my PR. They are passing locally for me but that has always been the case so lets see what CI says

I think we can close this one


Replace FS with Req

https://github.com/wojtekmach/req seems to cover everything we need from that abstraction. Just make sure that we can create an S3 adapter that is able to access protected content.

The whole FS module has been removed.


Combine Wind & Solar Target Generation Charts to One

Hey @nofurtherinformation !
Just created a quick PR related to your ask in slack! Should be a quick one, but let me know if you have any issues / concerns!

@swatland610 - there's some linting failures, and also should we show a number for the amount of renewables already present above the graph, like we do for EVS?

Power
Cars

Lastly if your other PR is now no longer relevant, can you close it?

When enough wind turbines or solar is made, it makes the wattage needed is negative instead of zero

https://deploy-preview-111--state-decarbonize.netlify.app/iowa

hey @vkoves ! sorry about that! I'll fix the linting, but as for your main question, I opted to keep things grounded in installed capacity or (MWs) as that's how most folks doing in depth research on this communicate it.
Also, I feel really good about the math we did to get the current numbers. But if we try adding in figures of how many wind turbines & how many solar panels, the math gets incredibly complex because we'd need to consider where all of these wind turbines and panels need to go and (where they actually can go).
Given that our end goal visualization is just a bar chart, I think sticking with MWs is most appropriate.
Let me know if that's not quite what you meant!

@SuragNuthulapaty good catch! I noticed that two and this current PR of combining these two graphs should fix those issues.
Also, I adjusted the raw data so values shouldn't be going below zero. So once this gets merge, we should be all good!

@swatland610 - I do actually still see the negative value issue on Idaho using the latest Netlify so I think something is still off there:

Maybe Idaho in particular has something weird

And as for current values, I meant more so if we could show the value underlying the graph, as in a paragraph after the amount we need to build saying:
"State already has _ MWs of solar and _ MWs of wind, _% of what we need".
This is directly modeled off of the vehicles section, since I think writing out the value is helpful to get a sense of how far a state has moved in a more direct way.

@vkoves you do mean Iowa right? I just checked and Idaho looks good

Yes, sorry, I screenshoted and linked the right thing but commented the wrong one 😂

Thanks y'all! I promise I'm on it. Just traveling back to the states over the next few days. I'll tackle this early next week!
TLDR; there's a few states like Iowa that are already in good shape as it comes to wind.
So I think your suggestion @vkoves of altering the copy for those is a good one! I can take a stab and working that out and share when I'm done!

@swatland610 - I've fixed up some stuff, including a clamp for the amount of solar and wind to build (so if it's supposedly negative we lock it at zero, since that just indicates we've built plenty) and also added some text indicating the current solar and wind capacity.
I also figured out the root cause of why the Idaho graph was full when it shouldn't have been (which I don't think I realized before) - we were adding two percentages together and subtracting them from 100, but that means that hitting both goals would gets us to 200%. To be more specific, we were doing totalRemaining = 100 - (percSolarTarget + percWindTarget), which at 100% solar and 100% wind yields 200%, or in Idaho's case, 100% wind makes the total 100%.
I now think everything is good but I think one point of confusion is that since we're mapping these two renewable goals, states with similar percentages of fossil fuel emissions show different graphs. Illinois, for comparison, only gets 8.6% of it's energy from renewables, but we get 60% of our energy from nuclear (total 68.6% non-fossil power), so I would say we are a little further ahead at Idaho at 58% renewables and 6% nuclear (total 63.7% non-fossil power). Instead though the graph indicates that Idaho is way ahead of Illinois, since it has much more solar and wind specifically:

Iowa
Illinois

@nofurtherinformation and @derekeder - I'd love your thoughts on this too. @nofurtherinformation I think your graph labelled this as a fossil vs non fossil graph but I'm not sure if you had updated the graph to pull different data to make that happen.

Actually, reading more I think PR #99 will still be useful and needed, since it indeed switches us from just solar/wind to fossils non-fossil, it'll just have to be re-worked a bit once this PR is merged. So I think this PR is ready to go, but someone else should review it since I made some tweaks

Note from Juan (perhaps for the future): Nobody knows what a MW is, so we should probably specify number of turbines and number of solar panels/sq. feet/acres of solar

@vkoves to recap, you'd like to review and merge in this PR and then reopen PR #99 after, is that correct? any way we could bring in what we want from #99 here?

@derekeder - based on our discussion I think this PR alone is fine, since Juan was basically saying we care about building the solar and wind we need, not the overall green-ness of the state


[Feature] Make forceCreatingSnapshot a toggle option for paimon

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
PaimonSink Commit has a logic that if no data is written during a checkpoint cycle, no snapshot will be generated.
Let me give you an example:
If I read a kafka stream and write it to Paimon, there may be filters in the middle, and flink watermarks are generated at the same time. Then each snapshot of Paimon will have watermark information. I can judge my writing progress through the watermark of the latest snapshot. , if the watermark is advancing in a checkpoint interval, but the filter logic filters out the data, a snapshot will not be generated, but the watermark should be advancing.
Currently, forced snapshot generation only takes effect when the automatic tag creation configuration is enabled and certain conditions are met. I think we should make this a switchable option.
Solution
Make forceCreatingSnapshot a toggle option for paimon
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

@JingsongLi  WDYT? Can you give some advice

This requirement looks good to me!
@huyuanfeng2018 Feel free to open PR, I will take a review.

This requirement looks good to me! @huyuanfeng2018 Feel free to open PR, I will take a review.

Thank you for your reply, I will open a PR later.


[BP-1.17][FLINK-32262] Add MAP_ENTRIES support

What is the purpose of the change
This is an implementation of MAP_ENTRIES in apache:release-1.17
Implement the map_entries method to transform a map into an array of key-value structs.
Description: The current implementation of the map_entries method in the Flink library does not provide a way to transform a map into an array of key-value structs. This enhancement aims to add this functionality, allowing users to convert a map into a more structured format for further processing.
Brief change log
MAP_ENTRIES for Table API and SQL

Syntax:
map_entries[map: Map[K, V]] -> Array[Struct<K, V>]

Arguments:
map: The input map to be transformed

Returns: An array of key-value structs obtained from the input map. Each struct consists of two fields: key of type K and value of type V.

Examples:
Transforming a map into key-value structs:

input_map = [1: 'apple', 2: 'banana', 3: 'cherry']
 map_entries[input_map]
 Output: [{'key': 1, 'value': 'apple'}, {'key': 2, 'value': 'banana'}, {'key': 3, 'value': 'cherry'}]
Handling an empty map:
empty_map = {}
map_entries[empty_map]
Output: []

See also: spark:https://spark.apache.org/docs/latest/api/sql/index.html#map_entries
Verifying this change

This change added tests in MapFunctionITCase.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure


feat(egyptian-api): update egyptian api

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sensor offset-calibration during self-test

Currently, sensor offset calibration is always performed before starting the breathing cycle.
This has the disadvantage that starting always takes a couple of seconds.
Should the calibration be moved to the self-test? And/or only at first start after power-on?

Sensor offset calibration will be done upon power-on. This is implemented in 24688c59780a139f89cb0384a6277e4ae10477c6.
If a future firmware implements a proper self-test, that would be a good place to do offset calibration as well..


fix(autoware_smart_mpc_trajectory_follower): fix unusedFunction

Description
This is a fix based on cppcheck unusedFunction warnings.
control/autoware_smart_mpc_trajectory_follower/autoware_smart_mpc_trajectory_follower/src/proxima_calc.cpp:29:0: style: The function 'd_tanh' is never used. [unusedFunction]
Eigen::VectorXd d_tanh(const Eigen::VectorXd & v)
^

control/autoware_smart_mpc_trajectory_follower/autoware_smart_mpc_trajectory_follower/src/proxima_calc.cpp:47:0: style: The function 'd_relu' is never used. [unusedFunction]
Eigen::VectorXd d_relu(const Eigen::VectorXd & x)
^

Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@masayukiaino
It is ok to remove these functions?

It is ok to remove these functions?

I checked it is ok


init Readium

i added readium-js_all.js to my build.
now how to initialize readium object to my web app.

This is the RequireJS single-bundle of AMD modules. You can get inspiration from this example:
https://github.com/readium/readium-js/blob/develop/dev/index_RequireJS_single-bundle.html
...although I would recommend checking how it's done in a "real", such as readium-js-viewer:
https://github.com/readium/readium-js-viewer/blob/develop/dev/index_RequireJS_single-bundle.html
The above application "bootstrappers" are for development mode, using the exploded source code and application assets (images resources, CSS stylesheets, fonts, etc.). If instead you need to deploy a distributable package, you may take a look at:
https://github.com/readium/readium-js-viewer/blob/develop/src/cloud-reader/index.html
...and more importantly, read the build instructions in the README:
https://github.com/readium/readium-js-viewer/blob/develop/README.md
I hope this helps. Regards, Daniel


--builddir not recognized?

Steps to reproduce:
Run bash <(curl -f -L -sS https://ngxpagespeed.com/install) --nginx-version 1.6.2 --builddir /root/new-pagespeed-installation-auto/
Note: bash <(curl -f -L -sS https://ngxpagespeed.com/install) --nginx-version 1.6.2 works fine.
Expected result: Runs smoothly and everyone's very happy.
Actual result: outputs 63: unrecognized option '--builddir' and the help page.

Update: worked fine with -b instead of --builddir for whatever reason. Tried to find in the installation shell script a hidden character breaking it, but wasn't able to. Weird.


Support OS native mouse cursor

General Summary
Support using the OS native mouse cursor instead of a custom rendered cursor.
Motivation
Currently it seems that Enso renders a custom circular mouse cursor instead of using the native cursor provided by the OS.
On Windows 10 with the latest Enso IDE (2.0.0-alpha.3), the custom mouse cursor is very sluggish and feels much less responsive than the native cursor, which is very detrimental to productivity, as navigating the UI with the mouse is extremely frustrating.
Solving these issues for the custom cursor for every OS and especially on lower-end hardware is likely to be very difficult. As such, supporting the use of the native cursor for people who wish to use it seems like a reliable and easy solution.

Was this resolved? Rejected? Why was this issue closed?


HDR color spaces

Any plans to add lookup tables for HDR-supporting color spaces like scRGB?

A BT.2020 LUT would be pretty cool too.


Can Empress._to_dict be made public?

The empress.core.Empress._to_dict method is really handy for serializing an empress object, and we would like to do this on https://github.com/biocore/microsetta-public-api/ .
As noted by @wasade on https://github.com/biocore/microsetta-public-api/pull/84 it is not great to call a private method. Is there any reason this method should not be used as, or made into a public method of the core.Empress object?

I don't see any issue with making Empress._to_dict() public. The original purpose of the empress python object was to create the html page so there was no need to expose the Empress._to_dict() function as using it would be outside the scope of the object. However, it might be better to create a wrapper function for it as Empress._to_dict() returns a dict which contains references to internal variables. So before we return the dict, we would want to duplicated the objects first.

+1, this is a good idea and there shouldn't be any issues with that.
On (Dec-17-20|12:48), kwcantrell wrote:

I don't see any issue with making Empress._to_dict() public. The original purpose of the empress python object was to create the html page so there was no need to expose the Empress._to_dict() function as using it would be outside the scope of the object. However, it might be better to create a wrapper function for it as Empress._to_dict() returns a dict which contains references to internal variables. So before we return the dict, we would want to duplicated the objects first.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://urldefense.com/v3/__https://github.com/biocore/empress/issues/470*issuecomment-747693137__;Iw!!Mih3wA!XsFXxWIF_3dQXd1OlUPlZlngiF-O9WISdHXiHHf_51r3gulMfemtt45ciIj_kV4$

So before we return the dict, we would want to duplicated the objects first.

Maybe something like a copy=True

Looks like #472 addressed this, so I think we can close this (pls yell at me if not)


[Template Submission]

This is a request in regards to the dev.to hackathon, here is my blog post url - https://dev.to/joro550/placeholder-title-414l
What's the use case for the template?
Right now the world has it's mind on COVID-19 how can you help? But looking beyond and forward how do we help the homeless the starving, how do we fix global warming. This project is to help people who want to help with all of these causes and more, oganizations can create events that they are currently running i.e. Donate to fix a curch roof, and have a quick "call to action" that people can use to donate to help that event.
Updates can be published to events to share how progress is coming along so that people are informed with how their contribution helped with the cause.

What's the programming language?
C# backend
C# (webassembly/Blazor) frontend
Template GitHub URL:
https://github.com/joro550/Artemis
Author Information:

Author name (or organization if submitted on behalf of a company): Mark Davies
Author GitHub organization/user: Mark Davies
GitHub username you used to sign the Agreement: joro550

Additional dependencies:
Sql Server (for database)
Checklist:

[x] I've filled out the form at ahoy.twilio.com/code-exchange-community
[x] My sample application is open source and has a permissive license such as MIT or Apache-2
[x] There are no known security vulnerabilities and I'm following security best practices
[x] I've read Twilio's best practices coding guidelines for my project
[x] I agree and adhere to the Code of Conduct
[x] The template has been authored by me / or my organization and will be kept up-to-date. If the template will not longer be maintained, I'll notify this project.

Thank you @joro550! Please make sure you update your DEV.to post in time for the hackathon submission deadline. Anything submitted through DEV.to as part of the hackathon will automatically be considered to also be added to CodeExchange.
I'll close this issue in the meantime. Feel free to re-open it when you finished your submission or want to submit an application for CodeExchange outside of the framework of the hackathon.