[BUG][opensearch] cant connect to metrics endpoint

Describe the bug
Im trying to see the metrics for opensearch but it seems that it isn't working.
To Reproduce
serviceMonitor:
  enabled: true

Expected behavior
See all metrics
Chart Name
Specify the Chart which is affected?
Screenshots

Host/Environment (please complete the following information):

Helm Version: 3.15.4
Kubernetes Version: 1.29.6

Additional context
Also, prometheus is alerting that the metrics are unreachable

Same problem here. Thank you for your report.

TBH this looks more like a configuration issue. I have the same problem. Looking into it, probably opensearch is either missing a configuration option or even the prometheus-exporter-plugin

Looking at what port is by default configured (9600) I'd think the idea is to use the performance analyzer which is already included and installed as plugin on the opensearch container, but not enabled for what I can tell
https://opensearch.org/docs/latest/monitoring-your-cluster/pa/index/

Okay, when trying to start it inside the container I just see that the address is already bound but not reachable.
I'm gonna continue troubleshoot this.
sh-5.2$ OPENSEARCH_HOME=/usr/share/opensearch/ OPENSEARCH_JAVA_HOME=/usr/share/opensearch/jdk/ OPENSEARCH_PATH_CONF=/usr/share/opensearch/config/ bin/opensearch-performance-analyzer/performance-analyzer-agent-cli &
[1] 1181
sh-5.2$ 09:05:50.833 [main] ERROR org.opensearch.performanceanalyzer.PerformanceAnalyzerWebServer - Could not create HttpServer on port 9600
java.net.BindException: Address already in use
        at java.base/sun.nio.ch.Net.bind0(Native Method) ~[?:?]
        at java.base/sun.nio.ch.Net.bind(Net.java:565) ~[?:?]
        at java.base/sun.nio.ch.ServerSocketChannelImpl.netBind(ServerSocketChannelImpl.java:344) ~[?:?]
        at java.base/sun.nio.ch.ServerSocketChannelImpl.bind(ServerSocketChannelImpl.java:301) ~[?:?]
        at java.base/sun.nio.ch.ServerSocketAdaptor.bind(ServerSocke
        ```

According to the initial PR, the SM is meant to be used in conjunction with https://github.com/Aiven-Open/prometheus-exporter-plugin-for-opensearch
I'll try to integrate it and make sure it's installed by default when enabling serviceMonitor
Reference: #537 (comment)

with the plugin i managed to access the metrics on port 9200, but the ServiceMonitor manifest still have a port pointing to metrics port name:
 spec:                                                                                │
│   endpoints:                                                                                │
│   - interval: 10s                                                                                │
│     path: /_prometheus/metrics                                                                                │
│     port: metrics

i believe that is why prometheus keeps warning me that the opensearch metrics are unreachable even so i can access it through port forward.

You can configure the port with metricsPort

@eyenx the endpoint shows all metrics being scraped, but it seems like because of both master and headless svc has the same exact labels, its causing some weird interaction and i cant get the metrics on grafana explore.
also, changing the metrics port works but when the port is 9200,  if you take a look at the svcs, it does not show that the metrics port is available:
metricsPort: 9200
serviceMonitor:
  enabled: true
  interval: 30s
plugins:
  enabled: true
  installList:
    - https://github.com/Aiven-Open/prometheus-exporter-plugin-for-opensearch/releases/download/<IP_ADDRESS>/prometheus-exporter-<IP_ADDRESS>.zip

kubectl get svc -n opensearch -o wide :
NAME                                 TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)             AGE    SELECTOR
opensearch-cluster-master            ClusterIP   <IP_ADDRESS>   <none>        9200/TCP,9300/TCP   2d8h   app.kubernetes.io/instance=opensearch,app.kubernetes.io/name=opensearch
opensearch-cluster-master-headless   ClusterIP   None         <none>        9200/TCP,9300/TCP   2d8h   app.kubernetes.io/instance=opensearch,app.kubernetes.io/name=opensearch

maybe because is the same port as the http

i have try to make servicemonitor work for a while now, but i think the implementation is broken. we have
service.metricsPortName and metricsPort
but setting either of those, breaks deployment, example:
service:
  metricsPortName: "http"
metricsPort: 9200

* Service "opensearch-cluster-master-headless" is invalid: spec.ports[2].name: Duplicate value: "http"
* Service "opensearch-cluster-master" is invalid: spec.ports[2].name: Duplicate value: "http"

* Service "opensearch-cluster-master-headless" is invalid: spec.ports[2]: Duplicate value: core.ServicePort{Name:"", Protocol:"TCP", AppProtocol:(*string)(nil), Port:9200, TargetPort: intstr.IntOrString{Type:0, IntVal:0, StrVal:""}, NodePort:0}
* Service "opensearch-cluster-master" is invalid: spec.ports[2]: Duplicate value: core.ServicePort{Name:"", Protocol:"TCP", AppProtocol:(*string)(nil), Port:9200, TargetPort:intstr.IntOrString{Type:0, IntVal:0, StrVal:""}, NodePort:0}

the best way for this to work, would be to leave the users an option to edit the servicemonitor but add defaults to values
example:
  endpoints:
  - targetPort: {{ .Values.serviceMonitor.targetPort }}
    path: {{ .Values.serviceMonitor.path }}
    interval: {{ .Values.serviceMonitor.interval }}
    scrapeTimeout: {{ .Values.serviceMonitor.scrapeTimeout }}
    honorLabels: {{ .Values.serviceMonitor.honorLabels }}
    {{- with .Values.serviceMonitor.endpointAdditionalProperties }}
    {{- toYaml . | nindent 4 }}
    {{- end }}

another issue that i notice, i get duplicate endpoints, beacuse the selector selects both headless and non headless service, so the selector needs to be updated to only one service.

@KristijanL i found out that serviceMonitor needs to use basic auth to properly connect to prometheus endpoint:
  - apiVersion: monitoring.coreos.com/v1
    kind: ServiceMonitor
    labels:
      app.kubernetes.io/component: opensearch-cluster-master
      app.kubernetes.io/instance: opensearch
    metadata:
      namespace: opensearch
      name: opensearch-cluster-master-service-monitor
    spec:
      endpoints:
        - basicAuth:
            password:
              key: password
              name: '{{ .Release.Name }}-initial-admin-password'
            username:
              key: username
              name: '{{ .Release.Name }}-initial-admin-password'
          interval: 30s
          path: /_prometheus/metrics
          port: http
      selector:
        matchLabels:
          app.kubernetes.io/instance: opensearch
          app.kubernetes.io/name: opensearch
          name: opensearch-headless

you can disable the chart default serviceMonitor and put a custom one in the extraObjecs.

That is certainly an option. However, at present, the ServiceMonitor appears to be non-functional and effectively redundant. If it is intended for use, it will need modifications to make it functional, or detailed documentation should be provided on how to use it effectively, as its current purpose is unclear.

totally agree

Related #593 and #594

Hi Team,
Is it required to install the plugin "prometheus-exporter-plugin-for-opensearch" for the opensearch container to expose the metrics?
May be adding below content to values.yaml, will fix the issue? How we can make this plugin to expose metrics on 9600?
plugins:
  enabled: true
  installList: []
  - prometheus-exporter-plugin-for-opensearch

Thanks,
Danish

@muhammeddanish yes see #593 We got prom metrics running with that plugin.


⚠ Turnos 24x7 has degraded performance

In d1e0101, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 5259 ms

Resolved: Turnos 24x7 performance has improved in fd5bda7.


ScottPlot5: WpfPlot as CustomControl

Purpose:
Use Control as base class for WpfPlot instead of UserControl to enable customization. (cf https://github.com/ScottPlot/ScottPlot/issues/2509).
Changes:

update ScottPlot5-wpf Solution Filter to add missed ScottPlot.OpenGL project
WpfPlot use now a template and is based on System.Windows.Controls.Control, very close to ScottPlot4 version (#2526).

I am not totally happy with this control. The method WpfPlot::Replace() has a bad taste to me. It looks weird as the control hold by the new Interaction instance my be another one as the control to which it will be set.
This looks very error prone to me 💣

@KroMignon this is fantastic, thanks so much! I'll make a few small changes to improve mirroring between the WPF and WinForms controls and merge this into main in a few minutes 🚀

The method WpfPlot.Replace() has a bad taste to me

I agree! This strategy was originally created so users could customize the interactivity (swapping what mouse buttons do, changing zoom and pan behavior, etc) but I think the API can benefit from improvement in the future


feat: documentation versioning

Sample it at: https://swarnimarun.github.io/rattler-build
Latest(0.7.0): https://swarnimarun.github.io/rattler-build/latest
Dev(master): https://swarnimarun.github.io/rattler-build/dev
This is still draft as I am currently stuck on fixing the CI to actually work with mike which is harder than it looks.
Also should we setup a gh-pages branch that will reduce the deploy from artifact step. cc: @wolfv
Note: the current issue with CI setup is getting output to a directory, mike the tool mkdocs uses for versioning doesn't support output to a directory, it only supports pushing to a branch.

what the gh-pages branch looks like generated by mike.
https://github.com/swarnimarun/rattler-build/tree/gh-pages

Good stuff :)

Also should we setup a gh-pages branch that will reduce the deploy from artifact step. cc: @wolfv

I don't think I care as long as it works.

Testing the CI on my personal repo is a bit too fiddly, for now take a look and let me know if it looks correct. I have a split the Ci into build, release and dev. For testing, release and master branch docs respectively.
Defaults to latest/newest release.


Update MSRV to 1.58

I think this should get CI passing again -- cfg-expr requires 1.58.

Ok so not so much fix CI as improve CI... I'm not sure how to fix the remaining ones 😕

Thanks, the remaining ones need to be regenerated. Will get to it in a bit.


Remove link to old vim plugin

typescript-tools.vim hasn't been updated into a couple of years.
Tsuquyomi is still active.
@DanielRosenwasser are you ok with this change?
In addition, maybe on typescriptlang we should just have the Vim icon link to tsuquyomi because its README lists all the plugins you might want to install — better than our 1 paragraph.

Seems reasonable 👍


Help: Custom Animations between scenes

Hello,
I'm new to react native technology. I need to add animations between one to another scenes for now here is only Left to Right and Bottom to Top.
Can anybody help me to add custom animations for transition.
please help me with source code, demo or if anyone have useful link please share here.
Thank you.

Hey @tjsaini007 , follow this issue : #2628 , be sure you'll have the informations as soon as possible !

Hello @Blapi , thanks for your instant reply but the provided link #2628 , does not work for me as I am not talking about the default "Left to Right" or "Bottom to Top" animation provided by react-native-router-flux. All I need is to animate two screens with some blur ,fade, focus in, focus out, bounce etc. effects , which are to be customized at our end. So i need help with some source code or demo regarding the same and this issue can not be closed now as the problem still prevails.

Yes I understood your issue, it is the same for me, I need to do custom animations/transitions. If you recheck the issue #2628 , you will see that aksonov replied to it. I'm about to investigate on what he said and will update my own issue.


Play one at the same time

How to make palyer Play music one at the same time?

I know it's a little bit late but I use an array of files and everytime the sound finish (promise get back) I call the play again and increment the index (counter).
Something like this:
const sound = new Sound(`${this.state.audioPath}/test${this.state.played}.aac`, '', (error) => {
  if (error) {
    console.log('failed to load the sound', error);
  } else {
    this.setState({ playing: true });
    console.log('starting playback');

    setTimeout(() => {
      sound.play((success) => {
        if (success) {
          console.log(`successfully finished playing /test${this.state.played}.aac`);

          if (this.state.played + 1 < this.state.recorded) {
            this.setState({ played: this.state.played + 1 });
            this.setState({ playing: false });
            this.play();
          } else {
            this.setState({ playing: false, played: 0 });
            console.log('Playlist finished');
          }
        } else {
          console.log('playback failed due to audio decoding errors');
        }
      });
    }, 50);
  }
});

Appendix:
played: Audios already played
recorded: Total audio files to play
playing: Flag to know if any file is being reproduced
played: Reinitialize the index to start again after finish
Would you mind to change the subject and add "playlist" to it ? I guess it would be helpful for people looking for this. It took me a while to figure it out this!
Thanks!

thanks fou your time, i have solve it buy using global


Door Accessory always in opening state

Hi I'm having an issue getting the door accessory working correctly I have half it working by receiving the value from 0 to 100 to open the door to a certain position  but the problem Im having is that once I stop on a value the door status is alway in opening with the loading wheel spinning and never stops. So I don't know what Im missing Ive attached a image to show the opening state and spinning wheel

This is the code Im using
`var Accessory = require('../').Accessory;
var Service = require('../').Service;
var Characteristic = require('../').Characteristic;
var uuid = require('../').uuid;
var lightState = 0;
////////////////CHANGE THESE SETTINGS TO MATCH YOUR SETUP BEFORE RUNNING!!!!!!!!!!!!!//////////////////////////
////////////////CHANGE THESE SETTINGS TO MATCH YOUR SETUP BEFORE RUNNING!!!!!!!!!!!!!//////////////////////////
var name = "Door1";                                       //Name to Show to IOS
var UUID = "hap-nodejs:accessories:windowDoor1xx";     //Change the RGBLight to something unique for each light - this should be unique for each node on your system
var USERNAME = "00:AA:AA:AA:AA:AA";              //This must also be unique for each node - make sure you change it!
var MQTT_IP = '<IP_ADDRESS>'
var TopicOpen = 'Home/Door1O'
var TopicClose = 'Home/Door1C'
////////////////CHANGE THESE SETTINGS TO MATCH YOUR SETUP BEFORE RUNNING!!!!!!!!!!!!!//////////////////////////
////////////////CHANGE THESE SETTINGS TO MATCH YOUR SETUP BEFORE RUNNING!!!!!!!!!!!!!//////////////////////////
// MQTT Setup
var mqtt = require('mqtt');
var options = {
port: 1883,
host: MQTT_IP,
clientId: '00GAK35243'
};
var client = mqtt.connect(options);
client.on('message', function(topic, message) {
});
//setup HK light object
var lightUUID = uuid.generate(UUID);
var light = exports.accessory = new Accessory(name, lightUUID);
// Add properties for publishing (in case we're using Core.js and not BridgedCore.js)
light.username = USERNAME;
light.pincode = "031-45-154";
//add a light service and setup the On Characteristic
light
.addService(Service.Door)
.getCharacteristic(Characteristic.TargetPosition)
.on('get', function(callback) {
callback(null, lightAction.getState());
});
light
.getService(Service.Door)
.getCharacteristic(Characteristic.TargetPosition)
.on('set', function(value, callback) {
lightAction.setState(value);
console.log("Door " + value);
callback();
});
// here's a fake temperature sensor device that we'll expose to HomeKit
var lightAction = {
//initialize the various state variables
currentState: 0,
//On Characteristic set/get
getState: function() { return this.currentState;},
setState: function(newState){
if(newState != this.currentState ){
console.log("Setting new Door state: " + newState.toString());
if(newState > 0){
client.publish('Home/Door1O', newState.toString());
this.currentState = newState;
}
else if(newState == 0){
client.publish('Home/Door1C', newState.toString());
this.currentState = 0;
}
}
},
//other light setting functions
updateState: function() {
this.currentState = lightState;
},
}
// update the characteristic values so interested iOS devices can get notified
setInterval(function() {
light
.getService(Service.Door)
.setCharacteristic(Characteristic.TargetPosition, lightAction.currentState);
}, 2000);`

You are missing the CurrentPosition characteristic of the door service. This is used to determine the state of the door.
You are also so missing the required characteristic PositionState

@Supereg  Hi thanks for the reply do you have an example code as I have no clue where to put it and how to use it
Thanks


Not working with the Raspberry Pi 4 model (Raspberry Pi OS: Buster)

Agent Version and Platform
Version of your agent? 2.170.1
OS of the machine running the agent? Linux/ARM/v7
What's not working?
I am trying to setup the agent on an ARMv7 (32bit) system (Rasbperry Pi 4 model), but the agent does not to work and throws the following error:
pi@RPi-1:/srv/devops/myagent $ ./config.sh
Could not execute because the specified command or file was not found.
Possible reasons for this include:
  * You misspelled a built-in dotnet command.
  * You intended to execute a .NET Core program, but dotnet-configure does not exist.
  * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.

.NetCore 3.1 has been installed and I verified this with the command:
pi@RPi-1:/srv/devops/myagent $ dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   3.1.301
 Commit:    7feb845744

Runtime Environment:
 OS Name:     raspbian
 OS Version:  10
 OS Platform: Linux
 RID:         linux-arm
 Base Path:   /usr/share/dotnet/sdk/3.1.301/

Host (useful for support):
  Version: 3.1.5
  Commit:  65cd789777

.NET Core SDKs installed:
  3.1.301 [/usr/share/dotnet/sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.App 3.1.5 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.1.5 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

Architecture using on RPi4:
pi@RPi-1:/srv/devops/myagent $ uname -a
Linux RPi-1 4.19.118-v7l+ #1311 SMP Mon Apr 27 14:26:42 BST 2020 armv7l GNU/Linux

Raspbian OS version:
pi@RPi-1:/srv/devops/myagent $ cat /etc/*-release
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)"
NAME="Raspbian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

Used resources:
Agent package: vsts-agent-linux-arm-2.170.1.tar.gz
.NetCore Runtime: aspnetcore-runtime-3.1.5-linux-arm.tar.gz
.NetCore SDK: dotnet-sdk-3.1.301-linux-arm.tar.gz

It indicates that dotnet-configure does not exist, but that command is unknown on any documentation i tried to find for the dotnet CLI. On an Raspberry Pi 3 this seems to work fine. On all RPi4 models I have I get the same issue.
The ./config command also required several dependencies that were not available with the newest Raspbian OS (buster) so I needed to modify the apt source list to use stretch instead and get the dependencies.

After no reply and some further investigation now I got some time to search deeper I found that libcurl3 is missing. I called the install-dependency script, but this packages are not available for the Raspberry Pi OS Buster version.
pi@RPi-1:/srv/devops/myagent $ ./config.sh
./bin/System.Net.Http.Native.so: /usr/lib/arm-linux-gnueabihf/libcurl.so.4: version `CURL_OPENSSL_3' not found (required by ./bin/System.Net.Http.Native.so)
Warning: on some platforms, libcurl3 is required.

Anyone having an idea how to fix this and install the agent on an RPi4 model? Is there an alternative build I could use for this?

Hi @SKO85 starting from 2.177.0 libcurl is no longer needed by default - could you please try it with the last agent version?

@SKO85 is this issue still actual for you?

@SKO85 is this issue still actual for you?

Still an issue but with Rasp 3B
Version: vsts-agent-linux-arm-2.179.0
OS: Raspbian GNU/Linux 10 (buster)
Output:
~/myagent $ ./config.sh
./bin/System.Net.Http.Native.so: /lib/arm-linux-gnueabihf/libcurl.so.4: version `CURL_OPENSSL_3' not found (required by ./bin/System.Net.Http.Native.so)
Warning: on some platforms, libcurl3 is required.

Still an issue but with Rasp 3B
Version: vsts-agent-linux-arm-2.179.0
OS: Raspbian GNU/Linux 10 (buster)
Output:
~/myagent $ ./config.sh
./bin/System.Net.Http.Native.so: /lib/arm-linux-gnueabihf/libcurl.so.4: version `CURL_OPENSSL_3' not found (required by ./bin/System.Net.Http.Native.so)
Warning: on some platforms, libcurl3 is required.

@anatolybolshakov : I will be setting up my new kubernetes cluster with the RPI-4 model and will try to verify this this weekend. Thx for your reply. I will keep you posted.

@anatolybolshakov : I will be setting up my new kubernetes cluster with the RPI-4 model and will try to verify this this weekend. Thx for your reply. I will keep you posted.

@biasync did you try to run bin/installdependencies.sh - to install necessary dependencies?

@biasync did you try to run bin/installdependencies.sh - to install necessary dependencies?

Hi @SKO85 @biasync do you have any updates?

Hi @SKO85 @biasync do you have any updates?

@anatolybolshakov , yes. Sorry for the late answer. I was not able to test with Raspbian OS, but teststed with Ubuntu 20.04 64bit ARM64 version for Raspberry Pi 4. The package as downloaded from the DevOps site worked without issues for me.

@anatolybolshakov , yes. Sorry for the late answer. I was not able to test with Raspbian OS, but teststed with Ubuntu 20.04 64bit ARM64 version for Raspberry Pi 4. The package as downloaded from the DevOps site worked without issues for me.

@SKO85 thanks!
@biasync do you still see this issue with Raspbian OS?

@biasync I'm closing this ticket due to inactivity at the moment - please feel free to ask any further questions.


added functionality for Add to Watch List

also removed "profileContainer" class from div wrapping the movie elements in Profile.js.


Add support for getting IdentityServer error details in ErrorMessage

A little bit about our scenario:
We have many websites integrating via code flow, but, by far, the most common error is that they are using an invalid redirect URI. This generates a lot of work from our side, because AFAIK the only way to diagnose this is looking at the logs. We would like to detect this and provide a better error message so we don't have to look at the logs everytime.
What we've tried
I'm getting the ErrorMessage similar to what is done here, but I just get the unauthorized_client that is set here. I'm wondering if it would be possible to expose extra details about the error.
Is there a way to do this? Could this be supported?

It sounds like you want a developer focused error message. As of now (other than the logging) we don't have that coded throughout IdentityServer so it's a bit of a task to do this. We'll assign this to 2.0.
What you might be able to do in the meantime is check the request id against what is persisted via the event service.

The first step has been added: https://github.com/IdentityServer/IdentityServer4/issues/965


Fine-tuning approach for Bert and GPT2 classifiers

❓ Questions & Help
Hey folks, when we are fine-tuning BERT or GPT2 model for a classification task via classes like GPT2DoubleHeadsModel or BertForSequenceClassification, what is the recommended fine-tuning strategy? I assume all transformer layers of the base model are unfrozen for fine-tuning. Does this result in catastrophic forgetting in practice? Do people use gradual unfreezing (as in ULMFiT)?

AFAIK, BERT doesn't make use of gradual unfreezing. Instead, during fine-tuning all model parameters are trainable. It can result in catastrophic forgetting, if you train it for long enough/ large enough learning rate, which is why we usually fine tune for 1-2 epochs at a low learning rate.
When it comes to doing it yourself, you'll should be able to just tweak the number of epochs/train steps and then find which number gives you the best results. IMO anymore than a couple epochs will result in overfitting/forgetting.
Hope that helps.

This issue can be closed.

Thanks @andrewpatterson2018.


Purview example Accounts_CreateOrUpdate.json is incorrect

The request payload of specification/purview/resource-manager/Microsoft.Purview/stable/2021-07-01/examples/Accounts_CreateOrUpdate.json seems to be incorrect because it doesn't include the identity parameter. Resource creation fails without it.
The schema confirms this. purview.json has identity as part of TrackedResource, and Account has TrackedResource included:
      "allOf": [
        {
          "$ref": "#/definitions/TrackedResource"
        }
      ],

Adding Service team to look into this.


New release of hypothesis breaks schemathesis

packages/schemathesis/models.py", line 9, in <module>
    from hypothesis.searchstrategy import SearchStrategy
ModuleNotFoundError: No module named 'hypothesis.searchstrategy'

hypothesis just released an hour ago...breaks this.
Release notes include:

If you are depending on the internal API (which you shouldn't be,    and which we don't guarantee compatibility on) you may have to rename some   imports.

Hello @kdelee !
Thank you for reporting this! I will take a look soon

https://github.com/HypothesisWorks/hypothesis/releases/tag/hypothesis-python-4.53.2

The fix is released in 0.19.1 - https://pypi.org/project/schemathesis/0.19.1/
Please, let me know if everything works for you


Fixes for mediaplatform models

Since there were no tests, we did not discover that the models could not actually be created since the primary key field was too small :S. Address this issue and fix up some others found along the way. The remaining commit increases how lenient we are in what we accept in the database. The reason for this initial leniency is that our models will be being populated from JWP which itself was populated from the SMS. We cannot guarantee that all fields will always have values so, for the moment, err on the side of caution.
Since migrations were starting to make the coverage reports noisy, remove them from coverage calculations.

Deployed to https://development.media.cam.ac.uk/ but execute migration fails

Deployed to https://development.media.cam.ac.uk/ but execute migration init container fails

I deployed a change including #138 to development earlier so suspect that this might be as simple as rolling back the (later) migration and re-deploying.
That being said, this change is deployed (by way of #142) to development already, so you can try it out there.


The duration provided is less than the minimum bound

When i make pause = false of instance if using sound sprite sometimes i got an error.

Assertion failed: End time is before start time

Uncaught RangeError: Failed to execute 'start' on 'AudioBufferSourceNode': The duration provided (-0.00793651) is less than the minimum bound (0).

Here is calculated negative value:
https://github.com/pixijs/pixi-sound/blob/9f18e27997e5153fe8ef04a36e4489e986b180d9/src/webaudio/WebAudioInstance.ts#L359
Because this._elapsed value is bigger then this._end.
https://github.com/pixijs/pixi-sound/blob/9f18e27997e5153fe8ef04a36e4489e986b180d9/src/webaudio/WebAudioInstance.ts#L297
Here is sample of actual debug values after _update function change this._elapsed value
https://github.com/pixijs/pixi-sound/blob/9f18e27997e5153fe8ef04a36e4489e986b180d9/src/webaudio/WebAudioInstance.ts#L501
now: 4.568526077097506
delta: 0.008707482993196791
this._duration: 12.94
this._elapsed: 5.734936507936508
this._end: 5.727
instance._end - (instance._elapsed % instance._duration) === -0.007936507936507908
It happens when i make pause right at the end of sound sprite and then trying to resume. Sorry for my bad english.

I did some tests and looks like that problem is fixed by 77cf4344a769ea7456e1a88c327525478ad0c694 commit.

Same error again. Probably the problem is here
https://github.com/pixijs/sound/blob/ffedde3388688c8a43784e8bb0f8a2db3f958f31/src/webaudio/WebAudioNodes.ts#L84


Set flutter Native window translucent

Use case
on desktop platforms, allows translucent the flutter Native window, and the translucent Native window can see the content of the main window.
(Note: Instead of making the main window transparent, the main window is opaque and the flutter Native window is transparent, which is different from #59969)
Proposal
I want to draw video content to main window and draw UI using transparent flutter Native window as overlay.
Because most video SDK can only draw video using the window handle.
details as follows:
Video Layer = Main window => C++ call video SDK to draw video
UI Layer = flutter Native window => translucent UI overlay

(Note: Instead of making the main window transparent, the main window is opaque and the flutter Native window is transparent, which is different from #59969)

No, #59969 is requesting the Flutter view be transparent, which is exactly the same as what you are asking for. Many of the people commenting there also want to make the window transparent, but that's not controlled by Flutter, it's controlled by the runner, so there is no issue tracking making the main window transparent.
Closing as a duplicate.

@stuartmorgan
#59969, what he requires is to make a transparent application, as long as the main window is transparent.
I checked fluent_ Acrylic project. He uses SetWindowCompositionAttribute to realize the translucency of windows on windows platform. He can solve it #59969
But with this method, when I render the content to the main window, the translucent window still displays the content behind the main window rather than the content I want to render.
As far as I know, on the windows platform, there is no other way to achieve window transparency unless the updatelayeredwindow function is used after rendering.

#59969, what he requires is to make a transparent application

It's irrelevant what the ultimate goal is. There are two parts of making a transparent application on macOS (which is what that issue is tracking): making the window itself transparent, and making the Flutter view transparent. The former is not in scope for the Flutter engine, as I said above, so the only think being tracked there is the ability to make the view transparent. The fact that some people have made a variety of off-topic comments there does not change what the issue is tracking.

As far as I know, on the windows platform

If you're interested in Windows, #71735 is the tracking bug for a transparent Flutter view. Feel free to comment there about your use case.

@stuartmorgan
Although the title of #59969 is the same as my request, the content of #59969 just wants the main window to be transparent, a transparent application.
This is exactly the same request as #71735, and all the answers are about how to implement main window transparency, and someone has already given a solution.
No one in #59969 and #71735 discusses how to implement native window translucency.
Currently, on windows, I didn't find any way to set the native window transparent in the runner, only the main window can be set transparent through SetWindowCompositionAttribute.
I have tried creating a new win32 video window and setting the main window to be transparent (SetWindowCompositionAttribute), but when I attach the main window to the video window via SetParent, the main window becomes opaque.
My current solution is to not use SetParent , synchronize the size of the two windows, turn off the window animations, and make them look like one window, but this will affect the user experience.
I think fluent should add an IsTransparentLayered parameter to the native window, which when enabled will make the native window transparent and can be attached to any window as an overlay.

all the answers are about how to implement main window transparency

The issue tracker isn't a Q&A forum, it is tracking engineering work. That issue doesn't have "answers", it has comments from people about workarounds for various use cases. Again: "The fact that some people have made a variety of off-topic comments there does not change what the issue is tracking."

No one in #59969 and #71735 discusses how to implement native window translucency.

The fact that the issue isn't fixed, or discussion of a temporary workaround for your use case, doesn't change what it is tracking.

I think fluent should add an IsTransparentLayered parameter to the native window, which when enabled will make the native window transparent and can be attached to any window as an overlay.

That is the functionality being tracked in #71735.

@stuartmorgan
Do you only read the title of the issue and not the content?
#59969 and #71735 are actually about creating transparent applications, rather than using the native window transparent as an overlay.
I'll never get an answer in #59969 or #71735.
Maybe you think that the transparent main window can be used as an overlay, but after my testing, using SetWindowCompositionAttribute After the window is transparent, once attached to any window using SetParent, the main window becomes opaque again. I have mentioned this in my previous reply.
Another api that can make window transparent is UpdateLayeredWindow, my current project is to use skiasharp to draw UI, I have implemented transparent window and can use set parent to attach to main window as overlay, after drawing transparent bitmap is done, use UpdateLayeredWindow to achieve The window is transparent.
I didn't see the specific implementation of the native window in the runner, so I can't understand how flutter uses skia to draw to the window. I think it's just that the runner can't achieve the transparency of the native window.

Do you only read the title of the issue and not the content?

No. I read the content, then I set the title to reflect what engineering work will be tracked as a result of the request that was filed.
I was the technical lead for the desktop embeddings before they were even part of the Flutter project, and wrote significant portions of them. Assuming I don't understand the issue tracker is not a productive way to engage here.

I'll never get an answer in #59969 or #71735.

Please see my comments above. The purpose of the issue is not to provide answers, it's to track engineering work.
I'm the case of that issue, it tracks the work to make the Flutter view support being transparent. For any use case, whether that use case is a transparent application, or a transparent overlay working within a normal application.
I'm going to lock this, as this isn't productive to continue. I suggest following the Windows issuei referenced, and mentioning your use case there, because regardless of whether you choose to believe me or not that is the issue that is teaching the engineering work to allow your use case.


Relax array query params needing to have brackets []

This line in route-recognizer.js requires array query params to have brackets, e.g. /some-route?foo[]=1&foo[]=2. I use an API that recognizes arrays of params without the brackets, e.g. /some-route?foo=1&foo=2. I can't find anything in the HTTP spec requiring brackets or otherwise indicating this API is out of line. So, this PR relaxes things.

A param's value becomes an array if any of the following:

The param name ends with [] at least once
The param name occurs more than once

The [] are still always stripped in the queryParams match object

Compatibility

This maintains backwards compatibility for those using foo[]=1&foo[]=2 notation.
This is a breaking change for anybody who was expecting all but the last value to be overwritten, e.g. for foo=1&foo=2 to parse as { foo: 2 }. It now parses as { foo: [1, 2] }. Not sure why you'd send multiple params only to discard all but the last. Nevertheless, it is a breaking change.
This helps nothing if you are mixing foo=1&foo[]=2 and want to interpret them differently. As with 2., the final foo[]=2 will no longer overwrite the first; you will get { foo: [1, 2] }. But the values are both under the same key. There's no way to distinguish the keys as e.g. { "foo": 1, "foo[]": 2 }.

Open Issues

I've added tests for deserializing query params. Should I be touching route generation? In route generation, query params still all get []s at the end.

Haven't looked too thoroughly at the code yet, but I'm :+1: to the intentions of this PR.
I wonder if it makes sense to make this configurable via an option? Or would the only reason to do that would be for BC reasons? Hm.

+1 for BC reasons
A large portion of the internet is still powered by frameworks not supported the latest notation standard.

Any update? @jmeas @john-kurkowski

No updates on my end. I can't tell who's a maintainer/owner of the repo; are you two? If an owner tells me what changes to make, I'll make them.

I wonder if it makes sense to make this configurable via an option?

If that's what people want, can do! But a config people aren't likely to interact with adds maintenance overhead too. Think of the if/else branching. Devil's advocate: this project isn't 1.0 yet. It could be a 0.1 -> 0.2 change, and will probably affect less than a handful of users.

+1

I'm closing this PR as the correct solution here is to adopt some default inside of route-recognizer (literally doesn't matter what it is) and allow it to be configured by passing in a serialize and deserialize function for query parameters. Anything else will work for some people and be less than ideal for others. There is no pleasing all parties with this one.
See other thoughts of mine on the topic here: https://github.com/tildeio/route-recognizer/pull/93#issuecomment-217610272


Issues with Account format

I'm not sure if it's designed this way, but I'm having an issue with creating a accounting numerical format. I have:
  new Inputmask("(,999){+|1}.00", {
    positionCaretOnClick: "radixFocus",
    radixPoint: ",",
    _radixDance: true,
    numericInput: true,
    placeholder: "0",
    definitions: {
        "0": {
            validator: "[0-9\uFF11-\uFF19]"
        }
    }
}).mask($("#amount_financed"))

It works correctly, but you have to click on the whole number and decimal numbers separately to add them, which is really annoying. Is there not a way for the user to start by entering the decimal places and then follow up with the whole numbers for a nice flow?
Thanks

@twigs67 ,
Can't you use the numeric alias? See readme.

	Inputmask("currency",  {
			numericInput: true
        }).mask("amount_financed");


docs(i18n-id): Provide Indonesian translation for array-refs, async-components, and attribute coercion

Overview
This is a part of #23
This pull request provides Indonesian translation for 3 files in migration section.
Changes
migration:

[x] array-refs.MD
[x] async-components.MD
[x] attribute-coercion.MD

Weird, package-lock.json shouldn't be included, shall I remove it?

Di hapus saja seharusnya tidak perlu kalau hanya menerjemahkan


[FSDE-11701] Migrate Link component to new Vue documentation

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/1/. :white_check_mark:

You can check this PRs instance in https://04e181be0eedd995b9fece6e8f28443944242ec7-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-268-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/2/. :white_check_mark:

You can check this PRs instance in https://2394aaec4e8cdd5bc17a628d2016670325ebc526-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-268-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/3/. :white_check_mark:

You can check this PRs instance in https://ef173747088b7bc4eadb0048d47faa1dcd3d64ed-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-268-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/4/. :x:

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/6/. :x:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/7/. :white_check_mark:

You can check this PRs instance in https://27802a0ef7fe725db5ed0ecc5f684d9784ea1a40-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-268-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/8/. :white_check_mark:

You can check this PRs instance in https://99dd6f6d8c33b969c703a7210b7b3e68df384f44-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-268-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/13/. :white_check_mark:

You can check this PRs instance in https://16e0d8e3a1ca09b0fedd87d6192dba2f0a0f1a06-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-268-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/15/. :x:

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/16/. :x:

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/17/. :x:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/18/. :white_check_mark:

You can check this PRs instance in https://27ea56d75d59f7140f8d23042f6114430c9ea1a1-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-268-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-268/20/. :white_check_mark:

You can check this PRs instance in https://e8d93855c24ec1beb6aba1fbcda3fbfa852be203-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-268-ux-chi.kubeodc-test.corp.intranet/ (internal)


use python to run functional image qc

Hi
I don't have the BIDS format fmri data and my data can't be transformed into BIDS format because I don't have the .json files. So how should I do  If I only to want to get a visual report for functional images QC?
Can I use python to get this? I used the wf=fmri_qc_workflow() function, but I can't find anything in the output_dir.
Thanks.

Please, follow up on #898.


Define Environment Variables for a Container

It is written (here):

This page explains how to use env.

What about envFrom example?

Page: https://kubernetes.io/docs/tasks/inject-data-application/define-environment-variable-container/
/retitle “Define Environment Variables for a Container” does not show example for using envFrom
/language en

In the docs on a different page (https://kubernetes.io/docs/tasks/configure-pod-container/configure-pod-configmap/#configure-all-key-value-pairs-in-a-configmap-as-container-environment-variables)  we do have an example that is using envFrom.
We could reuse/repurpose the same example for Define-environment-variable-container page or we could also provide a link to the existing example page to view usage of envFrom.

Hey, I worked on a similar issue where I had to define "env" and "envFrom" so I would like to work on this.
/assign

so instead of reusing the same examples and explaining the same thing that is already written in the doc can we just provide a link to already existing examples?
because I think using same examples again is sort of bad practice since we already have that explained OR we can move all of those examples and the explanation where we are trying to add the examples, but that would create chaos
So in my opinion I think we should just add a link to the examples
Do let me know if there is anything I am missing or am wrong about.

/triage accepted
/priority backlog


[Amazon AWS] Frecency sorting for Console command

Description
Frecency sorting so that most/recent used service console links appear at the top of search
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

closes https://github.com/raycast/extensions/issues/13025
closes https://github.com/raycast/extensions/issues/13032

Hey @JonathanWbn / @pernielsentikaer , are any of you available to review/merge these changes. Seems to be a gnarly issue that is affecting the Console command.


Parse the JDK-Next version number

See conversation in https://github.com/AdoptOpenJDK/openjdk-build/issues/1712
Currently, our VersionInfo class cannot assign a major version to a JDk-Next binary since there is no version number in the string (e.g. OpenJDK-jdk_x64_windows_hotspot_2020-05-19-03-31.zip). This causes issues with installer jobs as described here

So after doing a little digging, it turns out we are actually parsing JDK next version numbers downstream. The match223 regex successfully picks up the version number and passes it down to the installer. This is not the issue here.


Self-Attention Mask Expansion Issue

Assume a mask of [F, F, F, T, T]. In the encoder, this mask is expanded as follows:
slf_attn_mask = mask.unsqueeze(1).expand(-1, max_len, -1)
This results in the following mask:
[F, F, F, T, T]
[F, F, F, T, T]
[F, F, F, T, T]
[F, F, F, T, T]
[F, F, F, T, T]
The expanded mask is then passed into the scaled dot-product attention module. However, I think this may not be correct, as the fourth and fifth words should not be calculating attention at all.
I think the correct version should be:
[F, F, F, T, T]
[F, F, F, T, T]
[F, F, F, T, T]
[T, T, T, T, T]
[T, T, T, T, T]
Could someone clarify if this is an issue or a misunderstanding by me.

Assume a mask of [F, F, F, T, T]. In the encoder, this mask is expanded as follows: slf_attn_mask = mask.unsqueeze(1).expand(-1, max_len, -1) This results in the following mask: [F, F, F, T, T] [F, F, F, T, T] [F, F, F, T, T] [F, F, F, T, T] [F, F, F, T, T] The expanded mask is then passed into the scaled dot-product attention module. However, I think this may not be correct, as the fourth and fifth words should not be calculating attention at all.
I think the correct version should be: [F, F, F, T, T] [F, F, F, T, T] [F, F, F, T, T] [T, T, T, T, T] [T, T, T, T, T] Could someone clarify if this is an issue or a misunderstanding by me.


hw.device-type: Add revpi-connect-s

What does this implement/fix?
This commit adds support for the Kunbus Revpi Connect S
Any other comments?
Doing this to have the Kunbus Revpi Connect S officially supported by Balena, below you can find the links to the PRs if anyone wants to contribute, Thanks in advance!
https://github.com/balena-os/balena-raspberrypi/pull/913
https://github.com/balena-os/balena-raspberrypi/pull/917

Hi again @floion ! I also did a rebase to update this PR.
Thanks in advance!


[Feature][MOD-180] Show the preprint banner regardless of preprint status

Purpose
Show the preprint banner on projects to contributors regardless of preprint status.
Display the preprint status on the banner.
This is because contributors are used to getting to their preprint from the projects page but it doesn't show up if the preprint is pending or rejected.

Changes

Add new properties to node model
Update node view
Update project.mako template

If preprint provider supports moderation

If preprint provider doesn't support moderation

Side effects

Ticket

https://openscience.atlassian.net/browse/MOD-180

Wording from product:
This project represents a [STATE] [PREPRINT-WORD] submitted to [PROVIDER] [question mark icon thing].
Tooltip on hover over the question mark icon (used elsewhere on the OSF) with the wording shown on moderator detail page.

@sheriefvt, can you upgrade this to a hotfix?

@laurenbarker should the banner display for initial preprints?

@laurenbarker @aaxelb I will create a new branch based on master with another PR and close this one.

@aaxelb @sheriefvt the banner should not display for initial preprints.


Migrate into JuliaPlasma

https://github.com/JuliaPlasma
Ref:
https://github.com/JuliaRegistries/General#how-do-i-transfer-a-package-to-an-organization-or-another-user

Related to https://github.com/JuliaRegistries/General/pull/120849

@adamslc Hi Luke, maybe you could help change the url

Hmm, I also don't seem to have permissions. @michakraus, can you change the repo url?

Done. Also I gave @Beforerr admin rights to his own repo 😉 . Not sure why that wasn't automatic...


Prohibit non-Unit suspend functions

From my point of view returning any value from a suspend function is meaningless, because it's anyway will be effectively unused
Also it doesn't work in current implementation (issue)
Of course it's easy to fix, but IMHO the possibility to declare such a function may be very confusing

I think you are right, and we should prohibit this. Let's add the corresponding change to the proposal

Already documented

It's irrelevant since the new convention https://github.com/Kotlin/kotlin-coroutines/issues/71


[T5A3][W10-B1] Le Ba Hieu Giang

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


feat(node-env): general improvements

:tada: This PR is included in version 7.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix memory leak#12471

Source leaks  conf/conf_def.c
Fixes #12533
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

I think you meant:
Fixes #12471

Travis failures look unrelated to this PR.

And for 1.1.1.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master..

Thanks for fixing..
Merged also to 111


Async validation run twice

I call both .passes() and .fails()  to check the async validation.
Right now I don't find any solution to check both validation states without calling both of them.
This runs the validation twice.
I think we need a method which runs the async function only one time and calls respectively passes/fails callbacks. Or at least a promise implementation (as I proposed in this issue: https://github.com/skaterdav85/validatorjs/issues/127)
For example a possible solution could be to provide a validation function which accept an object with the two callbacks:
validator.validate({
  passes: () => {},
  fails: () => {},
});

We discussed the problem in this issue:  https://github.com/foxhound87/mobx-react-form/issues/214
Thank You.

What about checkAsync(passes, fails)[https://github.com/skaterdav85/validatorjs/blob/master/src/validator.js#L86] ?

Is it already available? I don't see it documented

@foxhound87, Please can you share how did you overcome this problem using checkAsync. Any type of help is appreciated.

@raghavgarg1257 take a look at this commit:
https://github.com/foxhound87/mobx-react-form/commit/b74be11f3d16af53870a55a9e5a26654dbff4b36

@foxhound87 Thanks, for the reply.

Can you write a real example here?
I have this issue and can't figure out how to solve.


add basic BDX PMU register definition

address https://github.com/intel/pcm/discussions/496
Change-Id: I201357cd269f52e887e165ccca2ba9fc22926ebf

the failure is unrelated


"excludeSearchPatterns" not honored

Problem
No matter what files are included or excluded inside of the "omnisharp.json" file, the OmniSharp-Roslyn server seems to ignore the settings and scan everything anyway. This is observed on both Neovim and VSCode.
Steps to Reproduce

Create an "omnisharp.json" file (in the proper directory, such as "~/.omnisharp/omnisharp.json" on linux) and fill it with the following:{
	"fileOptions": {
		"systemExcludeSearchPatterns": [
			"**/node_modules/**/*",
			"**/bin/**/*",
			"**/obj/**/*",
			"/tmp/**/*"
		],
		"excludeSearchPatterns": [
			"**/node_modules/**/*",
			"**/bin/**/*",
			"**/obj/**/*",
			"/tmp/**/*"
		]
	}
}

Build any code with dotnet build.
Check for errors with the editor of your choice. You will see errors such as this:Project/obj/Debug/netcoreapp3.1/File.AssemblyInfo.cs|11 col 1 warning| Unnecessary using directive.

Notice the directory matches the pattern **/obj/**/*, yet it has still been scanned for issues and the issues have been reported.

as documented here https://github.com/OmniSharp/omnisharp-roslyn/wiki/Configuration-Options, this is not how the feature works

Warning: this features allows disabling of project file / solution file discovery not individual .cs files.

duplicate of https://github.com/OmniSharp/omnisharp-roslyn/issues/1460


Front: Generator

Here is some feedback on the UI/UX of version 0.1 (in addition https://github.com/jhipster/jhipster-lite/issues/1397)

Sections

Make clearer sections. For example:

Configuration

Project
Options

Generator.

Where project and options are subsections of Configuration.

UI Elements

Currenlty several UI parts next to each other (for example Project configuration and options, or Language and Build tool. In general it's easier (like in a web shop) to put everything in a form under each other.

"Generator" section

Make this section clearer separate from the configuration section
Make it clearer what generate action can be performed (Now it has big buttons and small buttons, unclear is the difference between the two). Probably better is to put the actions in a list (the list should reflect the configuration section).
Add generate all / generate project button below the list (to perform all actions at once)
Give feedback when an action is completed (or failed). In issue 1397 a toast is recommended. I would avoid
the use of popups/modals. Just give, on the button, a 'green' V when it's succesful or a 'red' X when it failed (with below an error).
Put the download button seperate from others

If needed I can create a mockup for this.

Thanks for your suggestion @assimbly : it's just a 1st version, and indeed, it should be improved, the work is not finished. As another section should be added like Spring Boot Database, Spring Security, Spring Tools, etc. I'm waiting https://github.com/jhipster/jhipster-lite/issues/1288 to do it, because it will impact the work.
@swarajsaaj as you started the toast ticket. What do you think about green/red button instead?

Now it has big buttons and small buttons

I don't see what you're talking about. Can you post a screen about this plz?

Add generate all / generate project button below the list (to perform all actions at once)

For now, I don't want this. If the user want to generate all in once, just use generator-jhipster / jhipster-online to have the full big project. The idea of JH Lite is to give full control to the user, and he/she must understand each generated part.

It's unclear if I need to click on initialization or the other buttons. And should I click one by one? In a particular order?

I totally understand your point of view.
But, I would say:

in a real project, you start by creating an empty folder
how can you add frontend maven plugin ? if you don't have maven in your project, it won't work

So my ideas would be (for now):

expose all possible features in the front page, and don't care about the order
the user will be responsible for what he/she wants to generate in his/her project

The approach of JHipster Lite is different of generator-jhipster:

the user needs to think about what he/she wants to generate
the user needs to understand each part

Yes, the approach is different, but you can be sure that most users that will try JH Lite are still in the JH Classic mindset. That's why the UI must be extra clear that the concept/working is different.

Here are some examples to give feedback on what's going when generating:

Jenkins
Github

This just as inspiration, but some similar UI would work for JH Lite.

So don't hesitate to propose something
Once we agree, you are welcome to contribute @assimbly :)

Yes, I will. This was just sharing my first impressions :)

@assimbly : I'm closing this ticket in favor of https://github.com/jhipster/jhipster-lite/issues/1565
So we can discuss there


Fix(Docs): Fix the missing link in /README.md

Fix the missing link in /README.md
Before fix, a clickable link "How_to_enable_ESP32_AT_Classic_Bluetooth.md" links to
docs/en/Compile_and_Develop/How_to_enable_ESP32_AT_Classic_Bluetooth.md
However, the existing file is
docs/en/Compile_and_Develop/How_to_use_ESP32_AT_Classic_Bluetooth.md
So, the link has been broken.
It must be fixed.

Thanks for the observation. I appreciate.
We have recently been reorganizing the document structure and optimizing the document content, and will make necessary corrections.

Updated


client validation is not working with textArea

Hi
in view

setting form

                    $form = ActiveForm::begin([
                        'id'      => 'form-note',
                        'action' => '#',

                        'options' => [
                            'class'   => 'form-bordered',
                            'data-update' => '.notes-list'
                        ],
                        'fieldConfig' => [
                            'horizontalCssClasses' => [
                                'label' => 'control-label',
                            ],
                        ],
                    ]);

define a texarea

 $form->field($note, 'content')->textArea(['rows' => '6', 'class' => 'form-control'])
                    ->label(false)

In model
public function rules()
    {
        return [
            [['content'], 'string'],
            [['content'], 'required'],
            [['users_id'], 'integer'],
            [['created_at', 'updated_at'], 'safe']
        ];
    }

but client validation is not working with textArea tag, even I added some settings for form
'enableClientValidation' => true,
 'validateOnSubmit' => true,
 'enableAjaxValidation' => false,

Thanks

Any errors in console?

@samdark Nothing in console

it happens on me too

Guys i solved the problem by removing 'id' attribute on the activefield

Looks like this is #7627


Fix of the https://github.com/mbrlabs/Mundus/issues/29

Also, removed space from the log file name.

Thank you!


[WIP] Configure local file server and update tests

What type of PR is this?
/kind feature
What does does this PR do / why we need it:
Configure a local file server for accessing the files instead of over a network.
Which issue(s) this PR fixes:
Fixes NA
PR acceptance criteria:

[ ] Unit test NA

[ ] Integration test

[ ] Documentation NA

How to test changes / Special notes to the reviewer:
In the test script no devfiles should be referred to the github link

This will be done as a sprint task instead of ad hoc task
/close


Jenkinsfile Runner CLI/Docker should fail when Jenkins Embedder fails to startup

Currently Jenkinsfile Runner executions may hang if Jenkins Embedded fails to startup because of plugin conflicts or JCasC loading failure. It would be great to ensure that Jenkinsfile Runner stops and returns error code when such situations happen.
Acceptance criteria:

JFR bootstrap codebase is updated to detect Jenkins startup failures and to fail the build
There is a Unit test for JCasC mis-configuration
There is a Unit or an Integration Test test for Plugin Conflicts (might require a mock plugin manager)

We really need runner to fail and return with an error code, otherwise, the whole process running in our worker hangs without knowing what's happening there
For example if AWS creds or role is missing for AWS Credentials
2021-07-13 06:16:29.855+0000 [id=1]	SEVERE	h.i.i.InstallUncaughtExceptionHandler$DefaultUncaughtExceptionHandler#uncaughtException: A thread (Executing main/1) died unexpectedly due to an uncaught exception, this may leave your Jenkins in a bad way and is usually indicative of a bug in the code.
com.amazonaws.SdkClientException: Unable to load AWS credentials from any provider in the chain: [EnvironmentVariableCredentialsProvider: Unable to load AWS credentials from environment variables (AWS_ACCESS_KEY_ID (or AWS_ACCESS_KEY) and AWS_SECRET_KEY (or AWS_SECRET_ACCESS_KEY)), SystemPropertiesCredentialsProvider: Unable to load AWS credentials from Java system properties (aws.accessKeyId and aws.secretKey), WebIdentityTokenCredentialsProvider: You must specify a value for roleArn and roleSessionName, com.amazonaws.auth.profile.ProfileCredentialsProvider@6b31d8a0: profile file cannot be null, com.amazonaws.auth.EC2ContainerCredentialsProviderWrapper@27a0441d: The requested metadata is not found at http://<IP_ADDRESS>/latest/meta-data/iam/security-credentials/]

I improved the behavior for some cases but not for all of them. Any patches will be appreciated


Report contributions

Report individual contributions made by each team member

UI and UX related

Screen and feature prototype design [#50]
CSS, React Implementation and related bug fixes [#51 #56 #90 #46 #75 #88 #91 #55 #107]
Mobile adaptation [#82]
Container/component distribution decisions.
Starter info (welcome page) [#86]

General

Pull request revision and approval.
Route viewing [#36 #55 #49]
Main feed (social section) [#33 #38 #48]
Friend management [#35 #44]
Route sharing [#52]
Group management [#73 #75]
Route creation, edit and removal [#34 #43 #56 #55]
Code cleaning and style supervision, and project structure organization [#42 #47]
Service base.
Permission management.
Deployment fixes [#45]
GPX importing [#54 #78]
Map features (Google Maps API integration, HTML API ubication #89, route distance measurement #90, #88).
Internationalization.
Jest tests base creation, others supervision and related fixes/unification.
Suggestions for optimizing and improving the Viade Spec.

Documentation

Stakeholders (1.3) [#30]
System Scope and Context - Business Context (3.1) [#31 #24]
Cross-cutting Concepts (8) [#3]
Glossary updates [#32]
General review and corrections.

UI and UX

Container/component distribution decisions
Some mockups and styling sugestions

General and backend work

Pull request revision and approval.
Main feed(social seccion)[#33,#48]
My routes section
Friend managment[#35]
Route Sharing[#52]
Group management[#75]
Internationalziation
ViadeSpec participation
Proyect adaptation to ViadeSpec [#74]
COVID-19 service [#99]
Notifications
Permissions instructions html
Multimedia service backend
User service backend
Comments service backend
Add sharing with notifications to route service

Documentation

Quality goals
Deployment view
General review and errors fix

General

Groups:

Implementation of the functionality and its UI (First approach, later revised by other members).
Internationalization of the related messages.
Jest testing of the involved components.
Issues: [#44] [#73] [#75]

Friends:

Implementation of the functionality and its UI (First approach, later revised by other members).
Internationalization of the related messages.
Jest testing of some of the involved components.
Issues: [#35] [#41] [#44]

Documentation:

Technical context.
Design decisions.
Issues: [#10]

Documentation:

Introduction and Goals. Requirements Overview.
Quality Requirements. Quality Tree and Quality Scenarios.
Glossary updates
Some corrections
ISSUES: #8, #24, #32, #102, #110

Route viewing:

A few contributions in comments and multimedia UI
Components reorganization
ISSUES: #36, #49

JEST Tests:

Developed JEST tests for diferent components
Adapted tests with JEST to Enzyme
ISSUES:#57, #83

Cucumber Tests:

Developed Cucumber tests for Friends/Users and Groups
Fixed Cucumber after changes in the application
ISSUES: #71, #111

Other issues: #27, #53, #58

@alejandroleon98 and I have worked together

User interface

Worked in comments and multimedia (later upgraded by @marcosav ) #58 #53
Some other additions like the point selector in which post a comment

Documentation

Solution Strategy
Risks and Technical Debts
Docs updating when needed

Demo video [#100, #86]

Testing

Acceptance Tests and their maintenance (Cucumber and puppeteer) [#111, #97, #71] Also I tried to add ti to travis but i couldn't so the code coverage is not affected by these test unluckily.
Load tests (Gatling) [#95]
Unitary tests (Jest) [#83, #70, #57]

I've been working most of the time with @uo252137 through discord calls.
Also @marcosav helped us with problems we had to start working in the project.

Documentation

Arquitecture Constraints + Building Block View
Issues [#22], [#13]

Implementation

React classes for route creation
Changed to React hooks
Route creation/edition and fields
Issues [#34],[#43]

Testing

Unit tests using Jest + Enzyme
Issues [#70],[#83]

Deployment

Docker with the solid-node-server for Solid PODs and node images (readme)
Creation of viade_es2a image pushed to DockerHub:

(https://hub.docker.com/r/nereav/viade_es2a)

Issues  [#81]

Recording of meetings

I worked individually and with @264691, and @marcosav also helped us.

Implementation

Route creation/edition and fields
Map treatment for the creation of routes
Route routing (Finally we discard this implementation and delete the branch so that commits are not shown unfortunately, as we already reported in class)
Issues [#34],[#43],[#59 ]

Documentation

RuntimeView
Issues [#14 ]

Testing

Unit tests using Jest + Enzyme
Test maintenance
Issues [#70]

I worked individually and with @259000, and @marcosav also helped us. Also i made two cucumber test for the change of the language,but i had some problem and decided to removed for the final version


Ignore certain tokens / numbers

We should be able to ignore tokens / numbers / visitors, avoiding that other bots start a never ending conversation.

delivered in v1.1.2


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.14.267",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=aeaf67314e4e57df8aff636979a57b5a828a6451"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "aeaf67314e4e57df8aff636979a57b5a828a6451",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "scsi: target: iscsi: Make sure the np under each tpg is unique\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.14.267 by commit aeaf67314e4e57df8aff636979a57b5a828a6451. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000300


Correcting some spelling mistakes and typos

Hacktoberfest contribution, hope it helps!

@aliciaphes Thank you so much for the contributions.

Glad it helped!


card.io not working on marshmallow operating system device

Required Information

card.io Android SDK Version: 5.4.2
Android Version  (6.0 )and Device (Google nexus 5):

Issue Description

when I am launching for scanning it is not opening camera, i can see logs as below
E/card.io: Failed to load native library: dlopen failed: cannot locate symbol "__aeabi_memcpy" referenced by "/data/app/com.dd.cotech-2/lib/arm/libcardioDecider.so"...
I/card.io: Processor not Supported. Skipping camera.

Is there any any restriction on marshmallow

There is no restriction on Marshmallow, this is an issue in the native code for certain versions of Android with specific processor architectures. See #166

I am also facing the same issue:
Card IO SDK 5.4.2
Device: Honor 5x
OS version Marshmellow
MinSDK version 16
It works fine on lollipop devices but in marshmellow device it opens manual entry for credit card.


Add line length counts to obfuscation.RawData

This PR adds data relating to lengths of each line of source code to the obfuscation.RawData struct. This data is useful because obfuscated source sometimes consists of very long lines of code, with the special case of everything being on one line
Signed-off-by: Max Fisher<EMAIL_ADDRESS>

Fixes #515


Easy to grasp, early stages of Neural net

Hi,
What is this supposed to be? It does not contain any part number in its name, the name actually does not match any part.
Additionally, we have submissions for parts 1-5 already.


[AutoPR automation/resource-manager] AzAutomation Swagger for job & runbook

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2466

Codecov Report

Merging #2012 into restapi_auto_automation/resource-manager will decrease coverage by 0.09%.
The diff coverage is 46.19%.

@@                             Coverage Diff                             @@
##           restapi_auto_automation/resource-manager    #2012     +/-   ##
===========================================================================
- Coverage                                     52.28%   52.19%   -0.1%
===========================================================================
  Files                                          4913     4891     -22
  Lines                                        122710   121353   -1357
===========================================================================
- Hits                                          64164    63341    -823
+ Misses                                        58546    58012    -534

Impacted Files
Coverage Δ

...zure/mgmt/automation/models/dsc_compilation_job.py
25% <ø> (ø)
:arrow_up:

...ation/azure/mgmt/automation/operations/__init__.py
100% <100%> (ø)
:arrow_up:

...utomation/azure/mgmt/automation/models/__init__.py
100% <100%> (ø)
:arrow_up:

...re/mgmt/automation/models/job_create_parameters.py
55.55% <100%> (+13.88%)
:arrow_up:

.../mgmt/automation/models/automation_client_enums.py
100% <100%> (ø)
:arrow_up:

...gmt/automation/models/webhook_update_parameters.py
40% <100%> (-5.46%)
:arrow_down:

...gmt/automation/operations/job_stream_operations.py
17.24% <22.22%> (+1.85%)
:arrow_up:

...azure/mgmt/automation/operations/job_operations.py
13.21% <23.07%> (+3.82%)
:arrow_up:

...tomation/operations/linked_workspace_operations.py
26.66% <26.66%> (ø)

...e/mgmt/automation/operations/runbook_operations.py
14.72% <33.33%> (+4.38%)
:arrow_up:

... and 120 more

Continue to review full report at Codecov.

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


SBG_TIME_OUT

Hi,
I'm trying to use your driver on a Jetson TX2, plugging an SBG ellipse micro 2 in RS422.
Will your driver be abble to deal directly with it, or do I need a RS422/USB converter ?
Best regards,
Louis

I have no idea, but I guess it might work directly if you set the "portName" and "baudRate" correctly in the config file, see here:
https://github.com/ENSTABretagneRobotics/sbg_ros_driver/blob/7ea946bde87e7a6712377acf15a06139a3627c3b/config/ellipse_A_default.yaml#L9
Let me know if it works,
Thomas

Thanks for your quick answer,
I did set the baudrate and the port name but nothing happens, I keep getting this time out.
I tried all the serials port of the Jetson to be sure, and I set the Baudrate with the windows API.
I'm really stuck now :/
Louis

What do you mean by "Windows API"?

Have you configured the Ellipse with SbgCenter and set correctly the port & baudrate?
Is the port ID the same as with RS232 (PORT_A)?
Have you tested the serial link with an other device to be sure it works (for instance two Jetson connected throw RS422)?

Hi, sorry for my late answer, I've had a problem with my SBG.
By windows API, I mean SBG center.

I think it's well configurated, I did set the bodrate and the port.

Yes the serial link has been tested

You could try the "minimal ellipse" program included in the SDK from SBG to know if it is an issue with the ROS wrapper or the SDK.

Yes, I will test it today. Thanks for your help !

Could you send the code you used?
Thomas

From: louispetitjeanforssea<EMAIL_ADDRESS>Sent: Monday, November 19, 2018 5:26:43 PM
To: ENSTABretagneRobotics/sbg_ros_driver
Cc: Thomas LE MÉZO; Comment
Subject: Re: [ENSTABretagneRobotics/sbg_ros_driver] SBG_TIME_OUT (#12)
Hi again,
After some tests, I know manage to access the datas of the central with the minimal ellipse program. I think that it is a ROS wrapper problem...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/ENSTABretagneRobotics/sbg_ros_driver/issues/12#issuecomment-439953852, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFFW9V6CyQv6E69rsZ7a5uSZufmHgZByks5uwtvDgaJpZM4YgIMu.

Just the code of the sdk actually, the ellipseMinimal.c, compiled for ubuntu with gcc


Measure the success of the new onboarding flow

Check if our changes to the onboarding journey were successful:
How we can further improve it?
What are the yet unsolved issues?

Evaluate if changing the account ID is commonly adopted

We are still encountering issues with the current funding flow, and need to address this as a part of https://github.com/near/near-wallet/issues/1453. Closing this with the intent to measure the success of the next iteration.


[SPARK-13893][SQL] Remove SQLContext.catalog/analyzer (internal method)

What changes were proposed in this pull request?
Our internal code can go through SessionState.catalog and SessionState.analyzer. This brings two small benefits:

Reduces internal dependency on SQLContext.
Removes 2 public methods in Java (Java does not obey package private visibility).

More importantly, according to the design in SPARK-13485, we'd need to claim this catalog function for the user-facing public functions, rather than having an internal field.
How was this patch tested?
Existing unit/integration test code.

Test build #53174 has started for PR 11716 at commit c5fd485.

cc @cloud-fan one more ...

LGTM

@cloud-fan Had an extra minute - I went ahead and removed analyzer as well.

Test build #53177 has started for PR 11716 at commit 49197aa.

LGTM

Test build #53177 has finished for PR 11716 at commit 49197aa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #53174 has finished for PR 11716 at commit c5fd485.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53177/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53174/
Test PASSed.

Thanks - merging in master.


feat: bump containerd-shim-wws to use wws v1.4.0

Reference: https://github.com/vmware-labs/wasm-workers-server/releases/tag/v1.4.0

Could you please sign your commit @ereslibre ?

Could you please sign your commit @ereslibre ?

Done :)


Curl code fix

@mtdowling, ping

The unit tests passed before this change, so can you provide more details on why this is needed?

Sure. A year ago I noticed, that curl filter doesn't work at all. Mistake was in two packages: guzzle/RingPHP, and guzzle/retry-subscriber. Problem was, that RingPHP doesn't pass curl_result to transferInfo, so guzzle/retry-subscriber can't get it :) I've created MR's to fix both packages. You've merged just one out of two. Issue was described here.

@mtdowling, ping


Bloated XML in schemaXML of a Choices field

If provision a site with Apply-SPOProvisioningTemplate and sometime later run Get-SPOProvisioningTemplate the XML for a Choices field is bloated -

Choice1
Choice2
Choice3
Choice4

Choice1
Choice2
Choice3
Choice4

Choice1
Choice2
Choice3
Choice4

Hi Patrick
I thought Erwin had already fixed this one.
Regards
Nigel


More LTN feature requests

[ ] Allow edits on the perimeter road? (The "proper" fix is to extend the boundary until the perimeter is a main road, and never a residential street that just happens to back up to the edge of a map)
[x] Add a modal filter for school streets / time-dependent closures (just a new icon, no semantic difference)
[x] Be able to draw traffic calming / speed tables / 20mph zones (on any road, including perimeter). Not sure how to visually represent yet. Probably per road segment. And probably allow on perimeter roads too

Closing for now, because the first idea wasn't necessary after extending the Stapleford boundary. It's still an interesting idea, but I think it's not ideal.


Images not displayed in IE11

this is quite some special behaviour...
in IE11, images are linked without the ?raw parameter at the end which result in those images not being displayed.
A workaround is to add the parameter manually (it doesn't hurt the normal rendering):
image::demo.png?raw[]

Do you know what going on here? I can't see a section in the source where the ?raw paramater is not appended...

IE11 support is dropped anyway by bitbucket 6.7


Update  docs to reflect use of exact and strict in a

According to the discussion on   #4791 , which was later merged into #4826 , it is possible to use the exact and strict props on a  as long as it is rendered inside of a .
Hence:
<Switch>      <Redirect from='/' to ='/app' strict /> </Swtich>
will work while:
<Redirect from='/' to ='/app' strict />
Will not.
This behavior is (in my opinion) not intuitive, and also not documented in either the  or the  sections of the docs. I think because its behavior on the  component it should be described in its docs.
If people agree I'll make a pull request with proposed markdown (this is my first github issue still not exactly sure how the process works)

This functionality is documented under the <Switch> https://github.com/ReactTraining/react-router/blob/master/packages/react-router/docs/api/Switch.md#children-node. A <Switch> essentially treats all of its child elements as <Route>s, which is why you can use exact/strict on the <Redirect>.
The reason that it is not in the <Redirect> documentation is that they aren't actually props of the <Redirect>. I actually think that it would make sense to remove the from prop from the <Redirect> documentation and add a link to the <Switch> children props section.

I see. So my first comment that its not in either section of the docs is incorrect. My understanding of the situation now is that a <Switch> must convert all of its children into <Route> components which is what allows the <Redirect> component to use props that are normally only on <Route> such as exact and strict.
My problem with this is that it goes against my understanding of a component. The <Redirect> component itself can take different props depending on where its rendered and what parent component it has. This is a behavior that is unique to  the <Redirect> component itself. Which is why I think there should be some form of mention of it in the <Redirect> section.
Overall I think that any component's props should be the same no matter where it gets rendered. In a perfect world that would mean that the vanilla <Redirect> component would get access to strict and exact without needing to be in a Switch but I also understand that this is probably more work than its worth.

I agree that the setup of <Switch> isn't particularly satisfying. It is similar to how older versions of React Router worked where <Route> components were just glorified objects and not actual components.
I have a package of React Router compatible components that I created called rrc; one of these components is the <ConfigSwitch>, which I feel better expresses how the switch works.
<Switch>
  <Route exact path='/' component={Home} />
  <Route exact path='/about' component={About} />
</Switch>
// vs
<ConfigSwitch
  routes={[
    { path: '/', exact: true, component: Home },
    { path: '/about', exact: true, component: About }
  ]}
/>

It won't work as a fix for this because it only supports actual routes, but I think that it makes more sense.

The question now is, do you think its worth documenting the use of exact and strict under both Switch and Redirect? It would make the docs somewhat redundant but it would also make them easier to read and to follow for people new to the library.


Bug for a spesific Device

Issue Description
<CameraScreen
captureButtonImage={require('../assets/cameras.png')}
cameraFlipImage={require('../assets/flipCamera.png')}
flashImages={{
on: require('../assets/flashOn.png'),
styleOn: {width: 20, height: 20},
off: require('../assets/flashOff.png'),
auto: require('../assets/flashAuto.png'),
}}
onBottomButtonPressed={(event) => onBottomButtonPressed(event)}
hideControls={false}
showCapturedImageCount={false}
styleButton={{
width: 50,
height: 50,
justifyContent: 'center',
alignItems: 'center',
}}
/>
can not invoke function onBottomButtonPressed for spesific Device Samsung S7
but can for Xiomi Pocophone F1
Steps to Reproduce / Code Snippets / Screenshots
[FILL THIS OUT - It will be extremely helpful]

Environment

React Native Navigation version: [React Native 5]
React Native version: React Native Version 0.63.4
Platform(s) (iOS, Android, or both?): Android
Device info (Simulator/Device? OS version? Debug/Release?): Device Simulator android 7

Did you find a solution to this?


Sequel models do not work with custom serializers

Minimal reproduction:
require 'sequel'
require 'surrealist'

FileUtils.rm('test.sqlite') if File.exists?('test.sqlite')
DB = Sequel.sqlite('test.sqlite')
DB.create_table('users') do
  primary_key :id
  String :name
end

class UserSerializer < Surrealist::Serializer
  json_schema {
    {id: Integer, name: String}
  }
end

class User < Sequel::Model
  include Surrealist

  surrealize_with UserSerializer
end

User.create(name: 'Name')

User.first.surrealize
UserSerializer.new(User.first).surrealize

Last two lines both failing with:
Traceback (most recent call last):
	3: from test.rb:25:in `<main>'
	2: from .../gems/surrealist-35c87f58b098/lib/surrealist/instance_methods.rb:55:in `surrealize'
	1: from .../gems/surrealist-35c87f58b098/lib/surrealist/serializer.rb:45:in `surrealize'
.../gems/surrealist-35c87f58b098/lib/surrealist.rb:59:in `surrealize_collection': undefined method `map' for #<User @values={:id=>1, :name=>"Name"}> (NoMethodError)
Did you mean?  tap

The cause is here. Same check is performed in some other places in source code.

Yes, you are right, thank you for the issue and PR! We need to make sure this applies to all orms and edit the error message.


Az.Automation - runbookName does not match expected pattern

Description
When we use the Az.Automation powershell module on Azure Pipeline DevOps agents we get the error:
runbookName does not match expected pattern
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[X] Windows Server 2019
[X] Windows Server 2022

Image version and build link
2023-07-05T15:50:14.7848458Z ##[group]Runner Image
2023-07-05T15:50:14.7848570Z Image: windows-2022
2023-07-05T15:50:14.7848670Z Version: 20230630.1.0
2023-07-05T15:50:14.7848856Z Included Software: https://github.com/actions/runner-images/blob/win22/20230630.1/images/win/Windows2022-Readme.md
2023-07-05T15:50:14.7849082Z Image Release: https://github.com/actions/runner-images/releases/tag/win22%2F20230630.1
2023-07-05T15:50:14.7849233Z ##[endgroup]
Is it regression?
2022-10-12T15:56:35.1076001Z ##[group]Runner Image 2022-10-12T15:56:35.1076181Z Image: windows-2022 2022-10-12T15:56:35.1076340Z Version: 20221002.2 2022-10-12T15:56:35.1076638Z Included Software: https://github.com/actions/runner-images/blob/win22/20221002.2/images/win/Windows2022-Readme.md 2022-10-12T15:56:35.1077046Z Image Release: https://github.com/actions/runner-images/releases/tag/win22%2F20221002.2 2022-10-12T15:56:35.1077286Z ##[endgroup]
Expected behavior
All the runbooks should deploy / trigger successfully regardless of their name. This is a bug that was introduced and is logged as closed in git hub https://github.com/Azure/azure-powershell/issues/20614
However the powershell version has not been rolled out to the devops agents meaning the issue is still present.
Actual behavior
Releases fail with the error: runbookName does not match expected pattern. This is because a bug was introduced into the az.automation module. The bug link is: https://github.com/Azure/azure-powershell/issues/20614
This issue has been fixed in the module but the new version of the module needs to be rolled out to the DevOps Agents.
Repro steps
See https://github.com/Azure/azure-powershell/issues/20614

Hello @BSmeeton,
We will consider updating Az PowerShell.
You can also use AzurePowerShell@5 task with desired version.

Given the stability of Az releases we would not like to update a default Az PowerShell version yet, please install the version via the AzurePowerShell@5 AzDo task.


404 not found while downloading containers

I am using docker on windows environment and try to download it using kitematic but it's shows
(HTTP code 404) no such image - no such image: vulhub/struts2:latest: No such image: vulhub/struts2:lates

Which vulnerability did you test?
Please use the form below to describe your question:

Which environment: [e.g. python/ssti]
Host OS: [e.g. Ubuntu]
OS Version: [e.g. 18.04]
Docker version: [e.g. Docker version 18.04.0-ce, build 3d479c0]
Compose version: [e.g. docker-compose version 1.22.0, build f46880f]
Describe your bug, include the full error message, traceback or screenshots:


Request: Remove tokio & reqwest from client FFI library

Today, the FFI library is responsible for polling the server for updates. It uses tokio for scheduling an interval based chron & reqwest for performing HTTP requests. The problem with this is that it'll ship these dependencies to foreign language clients adding un-necessary weight to them. To get a tangible number I created a small binary w/ these dependencies, using minimal features & configured the build for size optimizations. Seems to be a little over 1MB. You can check it out here.
But size isn't the only issue, such dependencies can also limit platform support, as you can only ship to platforms supported by these libraries.
Removing tokio as a dependency shouldn't too hard, but reqwest might be a challenge. Maybe one can move to a lighter HTTP client like ureq & see if the size works out, but platform support might still be a challenge.
IMO, polling itself should just be moved out of the FFI library & into the foreign client.

Why we have tokio and can't remove it from the clients as they are today:

Tokio is the concurrency framework used by the rust clients where most of the functions are async (due to RwLock on the memory), the FFI segments need to block the threads since C doesn't support threading. We need tokio to help with blocking and syncing operations between these functions
"IMO, polling itself should just be moved out of the FFI library & into the foreign client." It is exactly like this, the implementing language needs to provide the runtime for the FFI library. Tokio merely manages the blocking here and adopts the runtime
"such dependencies can also limit platform support, as you can only ship to platforms supported by these libraries." Can you elaborate on this? What platform doesn't tokio work on?

With regards to reqwest, this is the easier dependency to swap out if platform support can be handled.
What we can do:

Today we import tokio "full" feature set, we can reduce that to:

sync
time
rt
rt-multi-thread

This might cut down the size a bit. If this isn't enough, for platforms where size is a concern it may be better to write clients targeting that particular platform in the most optimal language for that platform. We can look into this as well.

Tokio is the concurrency framework used by the rust clients where most of the functions are async (due to RwLock on the memory), the FFI segments need to block the threads since C doesn't support threading...

Doesn't the standard library provide an RW lock ? That way you could avoid dealing w/ any async business.
Also, IDK if I'm understanding your statement correctly, but C does support threading. In-fact that's how the Linux kernel exposes it. I believe it's the clone() system call. Though you could also use some higher-level POSIX API.

.. It is exactly like this, the implementing language needs to provide the runtime for the FFI library...

I don't think this is accurate, from the code I can see that a runtime is being statically created inside the library itself. Also by asking to move it out, I meant that the implementation should be in the foreign language. That way you could keep the FFI library as light as possible.

... What platform doesn't tokio work on?

Wasn't really talking about tokio, my main concern was reqwest in that regard. tokio already has most of the platforms covered, the only issue is it's size.

With regards to reqwest, this is the easier dependency to swap out if platform support can be handled.

I would disagree here. Rust's standard library provides alternatives for the things you are using tokio for, so it'll be an easy change. On the other hand, finding a drop in for reqwest which solves the issues that I have mentioned seems hard.

Today we import tokio "full" feature set, we can reduce...

As I mentioned, to get an estimate on how much these 2 actually end up adding, I already did an experiment like this. There I am only using time for tokio & json for reqwest, plus some build flags to optimize for size. And even then it's a little over 1MB. Re-linking it here.

...If this isn't enough, for platforms where size is a concern it may be better to write clients targeting that particular platform in the most optimal language for that platform...

This is fine, but I still think that there's value in doing a FFI library with just the business logic. That will aid in sharing the code in such cases.

Doesn't the standard library provide an RW lock ? That way you could avoid dealing w/ any async business.

The standard library RwLock doesn't support async/await. It blocks the thread. Went with tokio since its runtime is more performant, supports async/await primitives and is more popular/well supported. Why not deal with async business? Tokio handles work stealing, thread scheduling and management.

Also, IDK if I'm understanding your statement correctly, but C does support threading. In-fact that's how the Linux kernel exposes it. I believe it's the clone() system call. Though you could also use some higher-level POSIX API.

Its not C that's supporting it, that is the underlying OS. For linux, you can use pthreads or fork() and exec(). It is different for windows. C does need OS level support to achieve this, and its different for different platforms. https://stackoverflow.com/questions/3908031/how-to-multithread-c-code

I don't think this is accurate, from the code I can see that a runtime is being statically created inside the library itself. Also by asking to move it out, I meant that the implementation should be in the foreign language. That way you could keep the FFI library as light as possible.

The tokio runtime that is initialized is mostly for spawning a blocking function or blocking a parallel operation to completion. If you don't run cac_start_polling_update inside a thread, it'll just block your main thread. For the other functions that call block_on, you don't need to thread and can just use it like a normal function. The implementing language can choose to make it async or not. Tokio maybe managing how much work is done here, we can take a look at a better way here.

I would disagree here. Rust's standard library provides alternatives for the things you are using tokio for, so it'll be an easy change. On the other hand, finding a drop in for reqwest which solves the issues that I have mentioned seems hard.

Sure, seems I was wrong in this regard. Though replacing tokio also is a challenge:
https://users.rust-lang.org/t/lightweight-alternative-for-reqwest/33601/23

This is fine, but I still think that there's value in doing a FFI library with just the business logic. That will aid in sharing the code in such cases.

Agreed, but in some cases the constraints may justify another approach.

Maybe we can skip the discussions and just try to implement this?
A good way to check the feasibility is to try and remove tokio and reqwest and see if we can do that without comprising perf and safety. @ShreyBana would you be willing to contribute here?

That is a nice discussion.  I think in general going granular makes sense and exposing the core CAC resolution logic as separate functions makes a lot of sense - @ShreyBana @Datron.
Before we jump into implementation - let us consider the CAC lang implication.  I am thinking we should finalize the CAC lang specifics within about a week and that will have client implications as well.
Thanks for the detailed feedback - @ShreyBana


chore: use correct David badge for dependencies

Current coverage is 100% (diff: 100%)

Merging #1 into master will not change coverage

@@           master    #1   diff @@
===================================
  Files           6     6
  Lines          63    63
  Methods        15    15
  Messages        0     0
  Branches        1     1
===================================
  Hits           63    63
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update eda039b...547aab0


Add docs generation pipeline

Calls docs/generateDocs.ps1, then optionally pushes the generated docs to the
gh-pages branch.
Resolves #140.

I pushed a couple of WIP commits by mistake, will revert.


/* @flow strict */ On all source files

With all=true i can make flow check all files. How do i expand this to strictly checking all files?
My current .flowconfig:
[ignore]
.*/node_modules/.*

[libs]
flow-typed

[options]
all=true
module.use_strict=true

[strict]
nonstrict-import
unclear-type
unsafe-getters-setters
untyped-import
untyped-type-import

The last strict part is not applied unless i type /* @flow strict */ in files.
I want to avoid adding a comment to all files if possible :)

cc @gkz

I'd love to have all=strict!

as far as I understand strict, it's actually just setting all of the lints in [strict] to error. Try changing the [strict] section to [lints] and add =error to the end of each line. So:
[strict]
nonstrict-import=error
unclear-type=error
unsafe-getters-setters=error
untyped-import=error
untyped-type-import=error

That should set all of these lints as errors when flow type checks. Let me know if this isn't sufficient.

I had already set [lints] all=error, would that suffice?


README improvements

I couldn't figure out how to use this module when I looked at the
README. It was quickly going into pretty arcane stuff like "inet
filter" and "ip nat table" which might make sense for the module
authors or people used to nftables/iptables, but are pretty
implementation specific when coming from another networking
background.
Instead, we just explain more clearly what the module does, and
how. We also provide more examples, including some that might seem
obvious ("you need to include nftables first") but were not obvious to
me at all.
I also add a warning about firewalld being stopped which seems
important as well.
Closes: #158

Thanks for the patch!


runner: wrap file compare exceptions in TestError

If filecmp.cmp failed with an exception due to a missing file,
its exception was being lost due to another issue in the runner.
For this case it makes sense to catch this exception and wrap
it in a TestError exception as its most likely a missing file.

Will provide better output for https://github.com/OISF/suricata/pull/6723#issuecomment-998529461.


Static files IIS considerations

Addresses #1074

In theory the native static file handler can be placed above httpPlatformHandler in the web.config and configured to handle specific file types. However, I haven't found any working examples of this yet.

So, I'll update my PR with an example of compression, and if achieve it, with moving the iis native static file handler.
Thanks for the review

@leonardlabat I suggest holding off for today on updating PR. I think we need to have a discussion on what should go in this section if anything given that the original text really isn't relevant any more.

All right!
But this section is really important, especially because of the changes in static file handling due to hosting model update are not yet properly relayed on the internet and that right now, we only find out of date content.

:watch:

@leonardlabat Proposed updates in the comments. Can you make these updates?

@danroth27 Just added a new commit

@leonardlabat Were you able to confirm that compression and caching can be configured to work? If not, then I think we should leave the last comment out.

@danroth27 I can only get the dynamic compression working. Neither caching nor static compression seems to work (as its mentionned here https://github.com/aspnet/IISIntegration/issues/40#issuecomment-162238448).
Should  I remove  this last comment ?
Thanks

@leonardlabat @Tratcher Yeah, let's remove the comment for now to avoid setting the wrong expectations. In the future we might make some improvements in this area, but for now I think it's best to just not have this text. it's good to hear though that you got dynamic compression working.

Ok it's removed.
Hope this helps.
Tahnks

Thanks! c26de39213af4f55a9ffcc30b590fc7992bbd629