pip install does not seem to include some files

Describe the bug
pip install econ-ark does not seem to install HARK.datasets.data
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Mac/Windows/Linux]
Browser [e.g. chrome, safari]
Econ-ARK and Python version [eg. 0.10.6 and python 3.7]

Additional context
Add any other context about the problem here.

Leading to issues in DEMARK https://github.com/econ-ark/DemARK/actions/runs/5085827532/jobs/9139693267

It also does not seem to install dependencies along with it. Just had to correct some issues in https://github.com/econ-ark/DistributionOfWealthMPC because pip installing econ-ark master did not install all the dependencies. @MridulS @sbenthall can you take a look at this before our next meeting?

This is probably coming from https://github.com/econ-ark/HARK/commit/0ad8bb4f4ab5ad4ccef4f2eda75aa44faee424b0 I will take a look.

@alanlujan91 the datasets part should be fixed now. DemARK build is running now.

It also does not seem to install dependencies along with it.

Can you point towards which deps aren't being installed? Current HARK deps are available at https://github.com/econ-ark/HARK/blob/master/requirements/base.txt and these are the ones that should be installed when we do a pip install

See https://github.com/econ-ark/DistributionOfWealthMPC/blob/master/binder/environment.yml
I had to manually add xarray scipy numpy numba pandas because they were not installed with econ-ark

Hmm, was this to run on binder?

yes


Add the option to deploy K8s metrics server

This PR adds the option to deploy the Metrics Server and gives users read-only access to the Kubernetes Dashboard.

Isn't it better to install these components via RADAR-Kuberentes?
Also I remember that there was better alternatives for Kubernetes Dashboard but I don't remember what they were.

Ye, not sure why metrics-server has not been included in RADAR-Kuberentes.

Also I remember that there was better alternatives for Kubernetes Dashboard but I don't remember what they were.

I saw people using Lens and Aptakube but am not familiar with their license and pricing plans. AWS console also provides a limited dashboard view and let's hope EKS can catch up with other competitors on UI improvement soon. This PR only creates an optional deployment (disabled by default) with the "native" k8s dashboard so shall we include both components as a minimum offer to users for now and remove them when RADAR-Kuberentes has a better solution?


folder name option added for DiskLogAdapter.

max file size option added for DiskLogAdapter.
packages updated

also, change buildToolsVersion = '27.0.3' to buildToolsVersion = '28.0.3' in logger/build.gradle

@Guiorgy The buildToolsVersion is already 28.0.3

@Guiorgy The buildToolsVersion is already 28.0.3

You changed a global variable (rootProject.ext.buildToolsVersion) into a constant (28.0.3) in multiple files. I am just saying, that you should revert them back and change the variable value instead.

Hello @Guiorgy
Thanks for suggestions and I accepted your suggestions and committed.

can someone please merge this. we need this for a project

I forked and updated the project : Forkked Project


Projects - left margin

Left margin is indented too far compared to the other layout in list view.


401 Unauthorized on WebSocket, on iOS & macOS Safari when using Apache2 reverse proxy setup

Describe the bug
Followed guide here to set up self-hosted TeslaMate using Apache2 and HTTP Basic Auth.  This works fine on desktop browsers, but on mobile (iOS) the map does not load - endless loading wheel - and going to the Settings page results in the browser constantly refreshing.
If I remove basic HTTP auth completely it works.  Looking in the web server logs shows that accessing the /live/websocket resource on mobile results in a 401 Unauthorized error.  On desktop it returns 200 as normal.
Expected behavior
Map to load, and settings, etc to be available to edit without auto refreshing.
How to reproduce it (as minimally and precisely as possible):

Implement TeslaMate per above link
Visit website on iOS Safari.

Relevant entries from the logs
This is 100% not a TeslaMate bug, however I believe the install instructions here are wrong or are omitting some extra configuration to pass along HTTP Auth data:
Mobile/iOS (Safari) - NOT WORKING
<IP_ADDRESS> - - [28/Sep/2020:14:57:17 +0100] "GET /live/websocket?_csrf_token=KjgzYy8BABoUIToSOw0wBlA4IC5GH1cXHLB3K4-puCBVs4jt7KrY4q8e&baseUrl=https%3A%2F%2Fteslamate.mydomain.net&referrer=&tz=Europe%2FLondon&vsn=2.0.0 HTTP/1.1" 401 381 "-" "Mozilla/5.0 (iPhone; CPU iPhone OS 14_0_1 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0 Mobile/15E148 Safari/604.1"
Desktop/macOS (Safari) - NOT WORKING
<IP_ADDRESS> - - [28/Sep/2020:15:04:19 +0100] "GET /live/websocket?_csrf_token=BwgvBwwYMRk_Sy44OCgeL3hjfB40BgQU2XzXxPYHi9wbAJQjIQL-CeSB&baseUrl=https%3A%2F%2Fteslamate.mydomain.net&referrer=&tz=Europe%2FLondon&vsn=2.0.0 HTTP/1.1" 401 381 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0 Safari/605.1.15"
Desktop/macOS (Firefox) - WORKING
<IP_ADDRESS> - teslamate [28/Sep/2020:14:36:29 +0100] "GET /live/websocket?_csrf_token=PAdxShQhOzJAFHIZNQ9RdhFdDkBWDz5CS05ymLPzxF9wGb32C56v5fQw&baseUrl=https%3A%2F%2Fteslamate.mydomain.net&referrer=&tz=Europe%2FLondon&vsn=2.0.0 HTTP/1.1" 200 - "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:82.0) Gecko/20100101 Firefox/82.0"
Screenshots
Data
N/A
Environment

TeslaMate version: 1.19.4
Type of installation: Docker
OS TeslaMate is installed on: CentOS Linux 7.8
User OS & Browser: iOS 14.0.1 (not working), macOS 10.15.7 (Firefox, working) (Safari, not working)
Others: -

Could be related to https://github.com/adriankumpf/teslamate/issues/415

Have resolved this by following this helpful post - https://github.com/adriankumpf/teslamate/issues/576#issuecomment-618686810
This does make the websocket insecure, but in my case ports 3000 and 4000 are firewalled off anyway and the whole installation runs on my own private VPS.  Wouldn't say this is solved, but perhaps worth adding to the docs.teslamate.org instructions.


Drag and drop?

Any plans to add drag and drop?

Yes.
But it will take me at least 2 weeks.
I'm thinking about introducing it as a plugin, not to contaminate the
source code too much.
On Wed, Jul 20, 2016 at 1:44 AM, Nathan Pearson<EMAIL_ADDRESS>wrote:

Any plans to add drag and drop?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/17, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACUsQzhsZYWoeLteFjZB7Svxrv2YDYz-ks5qXVNGgaJpZM4JQQIj
.

--
Adam Klein
CTO & Co-Founder
Tel. +972-52-747-5633
blog.500tech.com http://blog.500tech.com
meetup.com/angularjs-il

That would be cool. Would be nice to have a native Angular 2 alternative to
jstree.
On Wed, Jul 20, 2016 at 9:27 AM, 500Tech<EMAIL_ADDRESS>wrote:

Yes.
But it will take me at least 2 weeks.
I'm thinking about introducing it as a plugin, not to contaminate the
source code too much.
On Wed, Jul 20, 2016 at 1:44 AM, Nathan Pearson<EMAIL_ADDRESS>wrote:

Any plans to add drag and drop?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/17, or mute
the thread
<
https://github.com/notifications/unsubscribe-auth/ACUsQzhsZYWoeLteFjZB7Svxrv2YDYz-ks5qXVNGgaJpZM4JQQIj
.

--
Adam Klein
CTO & Co-Founder
Tel. +972-52-747-5633
blog.500tech.com http://blog.500tech.com
meetup.com/angularjs-il
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/17#issuecomment-233851431,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQxluCmc0H6C9ti8Dynmf7P4_dh0alEks5qXb_PgaJpZM4JQQIj
.

That's where we're going :)
On Wed, Jul 20, 2016 at 9:41 AM, Nathan Pearson<EMAIL_ADDRESS>wrote:

That would be cool. Would be nice to have a native Angular 2 alternative to
jstree.
On Wed, Jul 20, 2016 at 9:27 AM, 500Tech<EMAIL_ADDRESS>wrote:

Yes.
But it will take me at least 2 weeks.
I'm thinking about introducing it as a plugin, not to contaminate the
source code too much.
On Wed, Jul 20, 2016 at 1:44 AM, Nathan Pearson <
<EMAIL_ADDRESS>wrote:

Any plans to add drag and drop?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/17, or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/ACUsQzhsZYWoeLteFjZB7Svxrv2YDYz-ks5qXVNGgaJpZM4JQQIj

.

--
Adam Klein
CTO & Co-Founder
Tel. +972-52-747-5633
blog.500tech.com http://blog.500tech.com
meetup.com/angularjs-il
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/500tech/angular2-tree-component/issues/17#issuecomment-233851431
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ABQxluCmc0H6C9ti8Dynmf7P4_dh0alEks5qXb_PgaJpZM4JQQIj
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/17#issuecomment-233853461,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUsQ6q9NwuSaUDZhSWyuMhWypBbQlz8ks5qXcMJgaJpZM4JQQIj
.

--
Adam Klein
CTO & Co-Founder
Tel. +972-52-747-5633
blog.500tech.com http://blog.500tech.com
meetup.com/angularjs-il

Hello,
I am searching for a tree view with a drag and drop functionality for angular 2 / typecript since a few days. Now I found this post. I hope this plugin will be realized. It would be cool. ;)

Would be nice to have select box options too. Scenario: I have something I
want to add to multiple categories in my tree, I can select them and click
save.
On Mon, Jul 25, 2016 at 3:34 PM, lemacabre<EMAIL_ADDRESS>wrote:

Hello,
I am searching for a tree view with a drag and drop functionality for
angular 2 / typecript since a few days. Now I found this post. I hope this
plugin will be realized. It would be cool. ;)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/17#issuecomment-234940506,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQxlqCqB9FB9TZMNqgFj39pappxAWTYks5qZK08gaJpZM4JQQIj
.

Hi,

you can achieve that with a custom template.
I intend to add a multi select option to the tree later on (but if you
want checkboxes, then it will still be a custom template)

On Mon, Jul 25, 2016 at 3:54 PM, Nathan Pearson<EMAIL_ADDRESS>wrote:

Would be nice to have select box options too. Scenario: I have something I
want to add to multiple categories in my tree, I can select them and click
save.
On Mon, Jul 25, 2016 at 3:34 PM, lemacabre<EMAIL_ADDRESS>wrote:

Hello,
I am searching for a tree view with a drag and drop functionality for
angular 2 / typecript since a few days. Now I found this post. I hope
this
plugin will be realized. It would be cool. ;)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/500tech/angular2-tree-component/
issues/17#issuecomment-234940506>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
ABQxlqCqB9FB9TZMNqgFj39pappxAWTYks5qZK08gaJpZM4JQQIj>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/17#issuecomment-234944829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUsQ9FLdajbCQRIrq8-vKVB6yrfEUdfks5qZLIPgaJpZM4JQQIj
.

--
Adam Klein
CTO & Co-Founder
Tel. +972-52-747-5633
blog.500tech.com http://blog.500tech.com
meetup.com/angularjs-il

Multi select is implemented in the latest version, you can use node.toggleActivated(true) - the true tells it to do multi select.
Then treeModel.getActiveNodes() will return all the selected nodes.

when drag & drop behaviour in the tree be implemented?

I'm very much looking forward to this functionality, as I would pay for an AngularJS 2 treeview component that supports drag & drop. I see one out there right now, but your implementation has significant advantages with its support for keyboard navigation, customizable field mappings, multi-select, etc.

Thanks for the motivation :) I will definitely push the pedal on this one,
as you are not the only one that requested it.
And don't worry, you don't need to pay - this is an open source project ;)
--

Adam Klein http://il.linkedin.com/in/adamklein500
CTO @ 500Tech.com http://500tech.com/
AngularJS, ReactJS, NodeJS
Development, consulting, training and public speaking
(+972) 52 747 5633

On Tue, Sep 27, 2016 at 8:46 PM, KeithGillette<EMAIL_ADDRESS>wrote:

I'm very much looking forward to this functionality, as I would pay for an
AngularJS 2 treeview component that supports drag & drop. I see one out
there right now https://github.com/valor-software/ng2-tree, but your
implementation has significant advantages with its support for keyboard
navigation, customizable field mappings, multi-select, etc.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/17#issuecomment-249941949,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2SShpT9gFld2E5n1xdi5rG2oeUwDkqks5quVZigaJpZM4JQQIj
.

Added drag & drop in Version 1.3.1:
https://angular2-tree.readme.io/v1.3.1/docs/drag-drop

You rock!
On Tue, Oct 4, 2016 at 1:42 PM, adam klein<EMAIL_ADDRESS>wrote:

Closed #17 https://github.com/500tech/angular2-tree-component/issues/17.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/17#event-811552493,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQxlsXuimV0qJRfLjLr_BPus3XTGX6nks5qwi2egaJpZM4JQQIj
.

:)

It seems like drag and drop does not work on firefox (it is working on chrome)

Hi,
Any event after drop is completed?
like once i drag and drop the element once the tree is updated with new data i need an event to do some more actions.
Thanks


Update official StratusLab images

The following disk images have been updated:

CentOS 6 (5G, 30G)
Ubuntu 14.04 (5G, 15G)
CentOS 7 (5G)


Add RequestId to botocore spans

Is your feature request related to a problem? Please describe.
When opening an issue with AWS Support, they will ask for the Request ID. If you don't have this ID then they won't help you. You are out of luck if you can't reproduce the issue.
Describe the solution you'd like
I would like to add the RequestID somewhere on the span for the boto3 calls. I couldn't find somewhere obvious to put it in the Elastic Common Schema (it doesn't matter to me where its stored on the span). I am mostly interested in getting this ID for errors but it seems simple enough to add it for errors and successful calls. Something like this (from your botocore.py):
from botocore.exceptions import ClientError

def call(self, module, method, wrapped, instance, args, kwargs):
      ...
      with capture_span(
            handler_info.signature,
            span_type=handler_info.span_type,
            leaf=True,
            span_subtype=handler_info.span_subtype,
            span_action=handler_info.span_action,
            extra=handler_info.context,
        ) as span:
            if service in span_modifiers:
                span_modifiers[service](span, args, kwargs)
            try:
                response = wrapped(*args, **kwargs)
                add_response_meta_data(span, response)
                return response
            except ClientError as client_error:
                add_response_meta_data(span, client_error.response)
                raise

def add_response_meta_data(span, response):
    if response:
        request_id = response.get('ResponseMetadata', {}).get('RequestId')
        span.label(RequestId = request_id)

I can submit a PR and tests if that helps.
Describe alternatives you've considered
Its possible to get the request ID if you add very verbose logging (as described here: https://github.com/boto/boto3/issues/1143) but that creates a lot of noise in the logs.

Hmmm, it seems like a no-brainer that we should be collecting this information. For our lambda integration, we do collect the request ID and store it in faas.execution and faas.trigger.request_id. But I don't think we should be doing anything faas-related in non-lambda transactions/spans. I'm going to ask around if other agents think we should get this added to intake -- if not, we can use labels as you've proposed.

I opened https://github.com/elastic/apm-server/issues/7871 to discuss adding span.context.http.request.id to the intake, which I think would be a great location for this Request ID. We could also potentially stick it in span.context.response.headers which is already supported in the intake. Stay tuned.

Thanks I really appreciate it!


Bot won't work anymore

The bot can't detect the URL, I've tried testing other Youtube Live URLS. Somehow, all proxies that I've tested all ends up in Errors.

Close, I realize I put the full URL of the youtube live instead of the ID only.


How do I type variables?

Hello all,
I searched for an answer but it was very hard to figure this out. I know how to get function headers to have types. I don't know how to define variables. For example:
def foo(InputVar: int) -> bool:
will define InputVar as an int. Now I want to do something like this:
def main():
x = 10
b = foo(x)
This will result in an error like (variable) x: Unknown and Type of "x" is unknown Pylance(reportUnknownVariableType)
What is the syntax to define x as an int? I would hope something like x: int = 10. Obviously 10 will result in an int but you get the idea. x could be assigned to a variable, some function, etc or even None. Is there a way to specify the variable as an integer to avoid getting this warning? BTW, I am using the latest version of pylance and vscode.

The syntax for annotating variable types was introduced in Python 3.6 and is described in PEP 526. In short, it looks the same as when annotating a parameter:
x: int = 10

In the vast majority of cases, Pylance can infer the type of a variable based on its assigned value, so explicit variable type annotations shouldn't be required.
If the statement x = <some expression> is resulting in x having an unknown type, then annotating the type of x isn't the first approach I'd take. Rather, I'd look at <some expression> to determine why it is evaluating to an unknown type. For example, if it's making a call to an unannotated function, then it's best to annotate that function. Once you do that, type inference will work, and an annotation for x will be unnecessary. Plus, by annotating the function, you eliminate the source of the "unknown" type, and the type checker can do a better job at finding and reporting bugs that take the form of type consistency violations.

True. In an ideal case, this would easily work. I seem to run into the
problem in a few wierd and specific cases, likely due to some parsing or
syntaxes error. I assumed this was the syntax but for some reason it didn't
seem to work. I will try again and give it more time.
Thank you for the reply. This was a remarkably difficult question to ask
google.
On Thu, Mar 25, 2021, 12:56 PM Eric Traut @.***> wrote:

The syntax for annotating variable types was introduced in Python 3.6 and
is described in PEP 526 https://www.python.org/dev/peps/pep-0526/. In
short, it looks the same as when annotating a parameter:
x: int = 10
In the vast majority of cases, Pylance can infer the type of a variable
based on its assigned value, so explicit variable type annotations
shouldn't be required.
If the statement x =  is resulting in x having an
unknown type, then annotating the type of x isn't the first approach I'd
take. Rather, I'd look at  to determine why it is
evaluating to an unknown type. For example, if it's making a call to an
unannotated function, then it's best to annotate that function. Once you do
that, type inference will work, and an annotation for x will be
unnecessary. Plus, by annotating the function, you eliminate the source of
the "unknown" type, and the type checker can do a better job at finding and
reporting bugs that take the form of type consistency violations.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/pylance-release/issues/1089#issuecomment-807093455,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC5B4RIXYC4UXLVKTH6KXDTFNTMNANCNFSM4ZZWTUDQ
.


New AKS Mariner nodes are missing 10-azure.conflist CNI config

What happened:
Since updating our AKS nodes to AKSCBLMariner-V2gen2-202405.20.0 we observed that the CNI conflist file /etc/cni/net.d/10-azure.conflist is no longer present on the new nodes.
This breaks other CNIs which chain to this as the file is not present. We suspect that the base image previously created this file and recent updates have changed this behaviour.
What you expected to happen:
New nodes should have the /etc/cni/net.d/10-azure.conflist present on boot.
How to reproduce it:
Updated nodes from AKSCBLMariner-V2gen2-202402.07.0 -> AKSCBLMariner-V2gen2-202405.20.0
Remote into machine and observe  /etc/cni/net.d/10-azure.conflist is absent.
Anything else we need to know?:
Previously the file was present on AKSCBLMariner-V2gen2-202312.06.0 so the change occurred between these two versions.
Some examples of the azure-cns deamonset have the dropz initcontainer argument create this file; however this is not present on our clusters (both with and without the new images).
Environment:

AgentBaker version: AKSCBLMariner-V2gen2-202405.20.0
Kubernetes version (use kubectl version): 1.27.9

Would anyone be able to help track down why the conflist file is missing? It seems there was some process previously which was baking this into the base images and it has since been removed. Diffing the repo shows 1000's of changes between those releases so it's difficult to track down where exactly the change happened.
It's not clear if this is intentional and dropz shoudl be generating this file instead (it doesn't with the current daemonset config). Or if thsi file is now deprecated for 15-azure-swift.conflist?

Closing as duplicate of Azure/AKS#4349


net-gateway-api shouldn't re-release Gateway API CRDs

We shouldn't be shipping the Gateway API CRDs
https://github.com/knative-extensions/net-gateway-api/blob/main/config/100-gateway-api.yaml
In places where we need to install Gateway API we should probably use similar steps as istio outlines
https://istio.io/latest/docs/tasks/traffic-management/ingress/gateway-api/#setup
kubectl get crd gateways.gateway.networking.k8s.io &> /dev/null || \
  { kubectl kustomize "github.com/kubernetes-sigs/gateway-api/config/crd?ref=v0.8.0-rc1" | kubectl apply -f -; }

We'll want to update our docs as well

We'll need to tweak serving's e2e testing here
https://github.com/knative/serving/tree/main/third_party/gateway-api-latest

/assign

@dprotaso I don't quite follow what I need to do for serving, if you can explain a bit further

But I don't know if I have to install them for the istio head and latest plus the api gateway test cases. Is that the case?

Just the gateway test cases - we stage those files here
https://github.com/knative/serving/blob/eac93b072bab30ccb9fd6fb49aca4c7e28fa95ab/test/e2e-networking-library.sh#L25-L26
We want to make sure that continues working when we stop shipping the gateway api resource in our net-gateway-api.yaml . We could simplify this a bit and include a separate file in a release with the gateway api crds we want
This could be done here
eg. https://github.com/knative-extensions/net-gateway-api/blob/bd2d2610fab9946be0d63955b5201ea30679fc38/hack/release.sh#L24-L28

I'll leave this issue open (so it doesn't close when your PR merges) until we resolve the serving repo testing


Investigate DLSS support

Did a very large chunk of that work already - however mostly blocked by MSFS2020 not submitting depth to OpenXR. To be picked up in a little while, and hopefully the game will submit depth then.

Will not do since MSFS will have native support.


GMSServices with typescript

when i add
if(application.ios) {
GMSServices.provideAPIKey("PUT_API_KEY_HERE");
}
typescript compiler gave me this error
error TS2304: Cannot find name 'GMSServices'.
how can i add this code in app.ts not app.js
thanks

the problem was the cocoapods was not fully installed


Update getting_started_with_lava.rst

Updated paths to tutorials after directory restructure of core lava repo
Fixes #9

Fixed MNIST path. Build is still broken. @mgkwill ?

Fixed MNIST path. Build is still broken. @mgkwill ?

@drager-intel Thanks. Yeah I think the issue is that @bamsumit has this configured to do autodoc for lava-dl but CI doesn't currently pull the source for lava-dl or for that matter lava, so when the build runs it fails to find source to run autodocs.


Encoding the dataset

Did you generate one giant TFRecord for the Lakh MIDI dataset? Or did you process the data in shards? If the latter, how exactly does one go about sharding the data with the pipelines that you have in place? I'm finding that generating one giant TFRecord using convert_dir_to_note_sequences is too large to load into memory when doing scripts/generate_song_data_beam.py.

Hello Ketan as far as I've observed in my own case (was kind of similar to yours), changing the Encoded files as ENCODED_TFRECORD=youroutputfolder/encoded.tfrecord and calling on terminal as:
python generate_song_data_beam.py --checkpoint="musicvae_checkpoints/cat-mel_2bar_big.ckpt" --input="../../../mnt/d/lakh/clean_midi_build/notesequences.tfrecord" --output=$ENCODED_TFRECORD
worked for me (for some reason) to solve the memory issue and "Killed" problem due to OOM.

Hello Ketan,
I had a similar issue and I've managed to solve it by removing ReShuffle from the pipeline to avoid parallelism at the cost of reducing the speed of the process but at least it doesn't try to load entire data to the memory and runs smoothly (but very slowly)


VS Code window is not responding after update and has black and white formations

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.82.1
OS Version: Ubuntu 20.04

VS Code was working properly while an update popped up from snap store. The update was installed, and when I re-opened vscode the window is now showing almost anything except the ribbon on the top. Whatever I click doesn't do anything, and has white and black rectangle boxes in the window.
I installed manually a previous version of vscode (1.81.1) and it did work but Python extension mentioned that it was not supported and needed  > 1.81

Experiencing the same issue after the latest update
VS Code Version: 1.82.1
OS Version: Ubuntu 20.04

Same here:
VS Code Version: 1.82.1
OS Version: Ubuntu 20.04
Seems to be related to this:
link failed but did not provide an info log
[11819:0913/111619.223859:WARNING:angle_platform_impl.cc(48)] ProgramGL.cpp:989 (checkLinkStatus): Program link or binary loading failed with no info log.
[11819:0913/111619.223884:ERROR:shared_context_state.cc(81)] Skia shader compilation error

/duplicate https://github.com/microsoft/vscode/issues/190437
Please delete the GPUCache as mentioned in https://code.visualstudio.com/updates/v1_82#_update-highlights-for-chromium

Thanks @nicklyk
rm -rf ~/.config/Code/GPUCache

Merging to https://github.com/microsoft/vscode/issues/190437


Add tools for Docker forensics

I added a few tools to perform forensic analysis of docker host machines.
Full disclosure: one is a tool I wrote

The docker-explorer should than be removed from the Disk image handling section.

Thanks!

Right you are. I removed it from the disk image handling part.


Provide method to NOT wrap at any length

Currently, clap wraps long help text lines at 120 chars if no term width can be determined (i.e. Windows). There should be a way for one to say, "Don't wrap at all" such as saying App::set_term_width(0) or something similar
cc https://github.com/rust-lang-nursery/rustup.rs/issues/657

This is added with #628


use_dot_notation = True causes InvalidDocument exception

Mongokit causes a weird issue when I have a document structure below. I think the issue started when I added 3 new fields which are key-value type dictionary.
structure = {
        'strategy_idx': unicode,
        'strategy_type': IS(u"static", u"normal"),
        'strategy_state': IS(u"draft", u"active", u"suspended"),
        'strategy_size_params': {
            unicode: OR(unicode, int, float)
        },
        'strategy_exec_params': {
            unicode: OR(unicode, int, float)
        },
        'strategy_risk_params': {
            unicode: OR(unicode, int, float)
        },
    }

I was able to insert a new record but when I tried to update the same object and call save(), I got an exception : InvalidDocument |  Cannot encode object: {u'FillSize': 20, u'MinSize': 10}
The whole document is below if you want to check for its validity. I tested it using bson.BSON.encode(obj) and it works perfectly.
{u'strategy_size_params': {u'FillSize': 20, u'MinSize': 10},
 u'strategy_type': u'static',
 u'strategy_exec_params': {u'Max': 100, u'Avg': 40},
 u'strategy_idx': u'TEST-77753-ID',
 u'strategy_risk_params': {u'Kind': u'A', u'StopLoss': -60},
 u'_id': ObjectId('5582668f75124f311344eb64'),
 u'strategy_state': u'active'}

This error is with MongoKit <IP_ADDRESS>
mongokit==<IP_ADDRESS>
pymongo==2.7

This error went away when I set 'use_dot_notation = False' It took me long time to figure out that this issues is not with the document but with its handling.
Related issue >
https://stackoverflow.com/questions/22049076/mongokit-nested-structure-save-error/22078430#

I notice that https://github.com/namlook/mongokit/issues/173 is related to this. But the difference is that I did not get this error until I added these kind of fields which are of nested dictionary type
'strategy_size_params': {unicode: OR(unicode, int, float)}

So I guess it must be related to these kind of fields since nothing changed on the system or usage.


Migration error..?

Hi there,
I am trying to migrate V3 chirpstack to newest one, but unfortunately I coudn't do it. It served by docker compose, in containers.
Here is my docker-compose.yml :
version: "3.5"
services:
#**********************************************************************************
#*    chirpstack network server
#**********************************************************************************
chirpstack-network-server:
image: chirpstack/chirpstack-network-server:3.16.8
container_name: chirp-network-server
restart: always
ports:
- 8003:8003
volumes:
- /etc/soitron/configs/chirpstack-network-server:/etc/chirpstack-network-server
- /etc/soitron/configs/chirpstack-network-server/chirpstack-network-server.log
depends_on:
- postgresql
- mosquitto
logging:
driver: "json-file"
options:
max-size: "100m"
max-file: "3"
#**********************************************************************************
#*    chirpstack application server
#**********************************************************************************
chirpstack-application-server:
image: chirpstack/chirpstack-application-server:3.17.9
container_name: chirp-application-server
restart: always
ports:
- 8880:8080
- 8001:8001
volumes:
- /etc/soitron/configs/chirpstack-application-server:/etc/chirpstack-application-server
depends_on:
- chirpstack-network-server
logging:
driver: "json-file"
options:
max-size: "100m"
max-file: "3"
#**********************************************************************************
#*    chirpstack gateway  bridge
#**********************************************************************************
chirpstack-gateway-bridge:
image: chirpstack/chirpstack-gateway-bridge:3.14.8
container_name: chirp-gateway-bridge
restart: always
ports:
- 1700:1700/udp
volumes:
- /etc/soitron/configs/chirpstack-gateway-bridge:/etc/chirpstack-gateway-bridge
depends_on:
- mosquitto
logging:
driver: "json-file"
options:
max-size: "100m"
max-file: "3"
#**********************************************************************************
#*    redis
#**********************************************************************************
redis:
image: redis:6.2.6-alpine
container_name: redis
restart: always
volumes:
- /var/lib/soitron/redisdata:/data
logging:
driver: "json-file"
options:
max-size: "100m"
max-file: "3"
#**********************************************************************************
#*    Mosquitto
#**********************************************************************************
mosquitto:
image: eclipse-mosquitto:2
container_name: mosquitto
restart: always
ports:
- 1883:1883
volumes:
- /etc/soitron/configs/eclipse-mosquitto/mosquitto.conf:/mosquitto/config/mosquitto.conf
logging:
driver: "json-file"
options:
max-size: "100m"
max-file: "3"
#**********************************************************************************
#*    Postgresql
#**********************************************************************************
postgresql:
image: postgres:14-alpine3.15
ports:
- <IP_ADDRESS>:5432:5432
restart: always
volumes:
- /var/lib/soitron/postgreSQL/data:/var/lib/postgresql/data
- /etc/soitron/configs/postgresql/initdb:/docker-entrypoint-initdb.d
environment:
POSTGRES_PASSWORD: xxxxxx
logging:
driver: "json-file"
options:
max-size: "100m"
max-file: "3"
postgres-new:
image: postgres:14-alpine
ports:
- <IP_ADDRESS>:5444:5432
container_name: postgres-new
restart: unless-stopped
volumes:
- /etc/soitron/configs/postgres-new/initdb:/docker-entrypoint-initdb.d
- /var/lib/soitron/postgres-new/data:/var/lib/postgresql/data
environment:
- POSTGRES_PASSWORD=root
logging:
driver: "json-file"
options:
max-size: "100m"
max-file: "3"
chirpstack4:
image: chirpstack/chirpstack:4
command: -c /etc/chirpstack
restart: unless-stopped
volumes:
- /etc/soitron/configs/chirpstack:/etc/chirpstack
- /etc/soitron/configs/chirpstack/chirpstack.log
- /etc/soitron/lorawan-devices:/opt/lorawan-devices
depends_on:
- postgres-new
- redis7
environment:
- MQTT_BROKER_HOST=mosquitto
- REDIS_HOST=redis
- POSTGRESQL_HOST=postgres-new
ports:
- 8888:8080
logging:
driver: "json-file"
options:
max-size: "100m"
max-file: "3"
#**********************************************************************************
#*    redis
#**********************************************************************************
redis7:
image: redis:7-alpine
container_name: redis7
restart: unless-stopped
volumes:
- /var/lib/soitron/redisnew:/data
logging:
driver: "json-file"
options:
max-size: "100m"
max-file: "3"
Firstly I received this error:
2023/11/30 06:17:20 Select users errordial tcp: lookup postgresql on <IP_ADDRESS>:53: server misbehaving
And later
errors like this related to integrations and other some missing collumns
Exec application integrations statement errorpq: invalid input syntax for type uuid: "InfluxDb"
I tried to remove unneccessary integrations and add some collumns to ne Chirp V4 but, I find out this is because old V3 use different data types than new V4. So I stucked here.
Then I try to change some .toml and also docker-compose.yml somehow. But I am back at the start with first error "<IP_ADDRESS>:53: server misbehaving".
I have no other ideas what to do or change. I am writing here because I need some help.
Here is toml files:
chirpstack-application-server.txt
chirpstack-network-server.txt
chirpstack.txt
Thank you for every idea ;)

2023/11/30 06:17:20 Select users errordial tcp: lookup postgresql on <IP_ADDRESS>:53: server misbehaving

This is a configuration issue. E.g. the hostname can't be resoved from where you run the migration tool.

Exec application integrations statement errorpq: invalid input syntax for type uuid: "InfluxDb"

Fixed by the above commit.

It looks like there is more issues with other columns
Or I did some mistake
root@Running-v3-Chirpstack:/home/soitron# ./chirpstack-v3-to-v4 --cs-config-file /etc/soitron/configs/chirpstack/chirpstack.toml-local --as-config-file /etc/soitron/configs/chirpstack-application-server/chirpstack-application-server.toml-local --ns-config-file /etc/soitron/configs/chirpstack-network-server/chirpstack-network-server.toml-local
2023/12/08 19:27:59 Reading ChirpStack configuration file: /etc/soitron/configs/chirpstack/chirpstack.toml-local
2023/12/08 19:27:59 Reading AS configuration file: /etc/soitron/configs/chirpstack-application-server/chirpstack-application-server.toml-local
2023/12/08 19:27:59 Deleting users and tenants from target database
2023/12/08 19:27:59 Migrating users
2023/12/08 19:27:59 Migrating organizations
panic: pq: null value in column "tags" of relation "tenant" violates not-null constraint
goroutine 1 [running]:
main.migrateOrganizations()
/chirpstack-v3-to-v4/main.go:318 +0x373
main.run(0xf33a80?, {0xa9afa7?, 0x6?, 0x6?})
/chirpstack-v3-to-v4/main.go:101 +0x37f
github.com/spf13/cobra.(*Command).execute(0xf33a80, {0xc00001e080, 0x6, 0x6})
/go/pkg/mod/github.com/spf13/cobra@v1.4.0/command.go:856 +0x67c
github.com/spf13/cobra.(*Command).ExecuteC(0xf33a80)
/go/pkg/mod/github.com/spf13/cobra@v1.4.0/command.go:974 +0x3b4
github.com/spf13/cobra.(*Command).Execute(...)
/go/pkg/mod/github.com/spf13/cobra@v1.4.0/command.go:902
main.main()
/chirpstack-v3-to-v4/main.go:79 +0x25
root@Running-v3-Chirpstack:/home/soitron#

Which version of the v3 to v4 migration utility are you using? Are you
using the latest release from yesterday?
On Fri, Dec 8, 2023, 20:42 JozefRuta @.***> wrote:

If I remove one I get another:
@.***:/home/soitron/chirpstack-v3-to-v4#
./chirpstack-v3-to-v4 \
--cs-config-file /etc/soitron/configs/chirpstack/chirpstack.toml-local
--as-config-file
/etc/soitron/configs/chirpstack-application-server/chirpstack-application-server.toml-local
--ns-config-file
/etc/soitron/configs/chirpstack-network-server/chirpstack-network-server.toml-local
2023/12/08 20:30:31 Reading ChirpStack configuration file:
/etc/soitron/configs/chirpstack/chirpstack.toml-local
2023/12/08 20:30:31 Reading AS configuration file:
/etc/soitron/configs/chirpstack-application-server/chirpstack-application-server.toml-local
2023/12/08 20:30:31 Start migration
2023/12/08 20:30:31 Migrating users
2023/12/08 20:30:31 Exec user statement errorpq: duplicate key value
violates unique constraint "user_pkey"
—
Reply to this email directly, view it on GitHub
https://github.com/chirpstack/chirpstack-v3-to-v4/issues/17#issuecomment-1847820552,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABIM6MVVMY6O6TRU2MLQY3YIN3T5AVCNFSM6AAAAABADDMHHCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBXHAZDANJVGI
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Hi,
I had version cloned 8.12 20:30
But to be sure I tried it again

I couldn't find written version number anywhere.

Please see:
https://github.com/chirpstack/chirpstack-v3-to-v4/releases/tag/v4.0.7.
On Sun, Dec 10, 2023 at 12:30 AM JozefRuta @.***> wrote:

Hi,
I had version cloned 8.12 20:30
But to be sure I tried it again
image.png (view on web)
https://github.com/chirpstack/chirpstack-v3-to-v4/assets/65413551/b70612bb-c92d-4032-bdb3-11b20ae9e22f
image.png (view on web)
https://github.com/chirpstack/chirpstack-v3-to-v4/assets/65413551/c469a6ce-16d3-490a-8d5f-d72ed7a1f0cc
I couldn't find written version number anywhere.
—
Reply to this email directly, view it on GitHub
https://github.com/chirpstack/chirpstack-v3-to-v4/issues/17#issuecomment-1848794903,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABIM6NIV26SSN2YBB5JL2TYIT7AXAVCNFSM6AAAAABADDMHHCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBYG44TIOJQGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Hi, I downloaded your precompiled version and untar again. But I have the same error:

Duplicate key value violates unique constraint "user_pkey"

That means you try to migrate the same user twice, the DB constraint guarantees that an user only can exist once. Please see the README, if you want to overwrite / remove all data before re-running the migration you need to use --drop-tenants-and-users.
The latest version introduces many new flags to make it easier to do partial migrations.

Hi,
I really appreciate your help 👍
I have to remove DB volume of V4 postgres...compose up/down it wasn't enough. It is obvious because volume is outside of container and if tried to migrate many times before there was something partialy migrated.
Have a nice day!


Chathuggingface 422 error

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_huggingface import ChatHuggingFace, HuggingFaceEndpoint

llm = HuggingFaceEndpoint(
    endpoint_url="http://<IP_ADDRESS>:9009",
    task="text-generation",
    max_new_tokens=10,
    do_sample=False,
    temperature=0.8,
)
res = llm.invoke("Hugging Face is")
print(res)
print('-------------------')

llm_engine_hf = ChatHuggingFace(llm=llm, model_id = "meta-llama/Meta-Llama-3-8B-Instruct")
res = llm_engine_hf.invoke("Hugging Face is")
print(res)

Error Message and Stack Trace (if applicable)
No response
Description
Using ChatHuggingFace with llm being HuggingFaceEndpoint got 422 error "Unprocessable entity" from huggingface-hub Inference client post function, when using the latest versions of langchain and huggigface-hub 0.24.3. Downgrading to the following version, I got the code to run.
The working versions of packages
huggingface_hub==0.24.0
langchain==0.2.9
langchain-core==0.2.21
langchain-huggingface==0.0.3
langchain_community==0.2.7
System Info
The following versions are what caused problems
langchain-community==0.0.38
langchain-core==0.2.19
langchain-huggingface==0.0.3
langchain-openai==0.1.16
huggingface_hub==0.24.3

I am experiencing the same issue.

I'm getting:
HfHubHTTPError: 422 Client Error: Unprocessable Entity for url: https://yna78eduyn1wumb2.us-east-1.aws.endpoints.huggingface.cloud/v1/chat/completions (Request ID: lIPtPv)
using:
from langchain_core.utils.function_calling import convert_to_openai_tool

dict_schema = convert_to_openai_tool(ArticleAnalysis)
output_msg = chat_model.bind(max_tokens=8192, temperature=0.0).with_structured_output(dict_schema).invoke(chat_sequence)
chat_sequence.append(output_msg)
md_text = output_msg.content
display_markdown(md_text, raw=True)
display_markdown("**Finish Reason**: " + output_msg.response_metadata["finish_reason"], raw=True)
pprint(output_msg.response_metadata)

@Jofthomas could this be linked to this issue? Not sure if the base_url argument is available in the langchain wrapper

@dosubot Downgrading is not a valid solution.


Problem loading fonts

Hello, I am trying to use this font: ypn envypn
My configuration looks like this:
font:
  normal:
    family: ypn envypn
  bold:
    family: ypn envypn
  italic:
    family: ypn envypn
  size: 9.0
  offset:
    x: 0.0
    y: 0.0
  use_thin_strokes: false

But i get the following error:
thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', src/libcollections/vec.rs:1431
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Did i miss something? I tried to adjust the size and the offset to see if there would be any difference but it's the same, is this a bug?

Hmm, would you mind running this again with RUST_BACKTRACE=1 alacritty? Hard to say what the problem is with the short error.

I think that this might be fixed with #590. At least for me, I got the same error when I tried to use bitmap fonts before that was merged. But now it seems to be working nicely :)

Sorry for the delay..
Ok, i recompiled alacritty and now works if i set the font size to 10.0. But the font which alacritty displays is the version with the pixelsize set to 13, I tried to put the pixelsize to 15, but i don't know how.
I tried with a lot of numbers from 10 to 20 and i can't display the correct terminal font size.
Alacritty encountered an unrecoverable error:

	Invalid pixel size

Thank you for the response.

Yeah, I also had that problem. It took a while for me to figure out which size the font wanted.
@jwilm In most other programs I used, it either auto adjusts the size to the nearest correct one, or it prints out the available font sizes. Perhaps alacritty could do something like this?
I would be really happy if it for example printed out this instead:
Alacritty encountered an unrecoverable error:

	Invalid pixel size: 10. Available pixel sizes: 11, 15

With [bitmap] xos4 Terminus font the only working size in range 1.0-50.0 is 9.0 and alacritty says:
$ ./target/release/alacritty
got font path="/usr/share/fonts/terminus/ter-x12n.pcf.gz"

and it only works if dpi is default 96x96. If I set dpi to 48x48 or 192x192, for example, or change size to any other value, alacritty says
$ ./target/release/alacritty
got font path="/usr/share/fonts/terminus/ter-x12n.pcf.gz"
Alacritty encountered an unrecoverable error:

        Invalid pixel size

So I assume the issue is not really about uninformative error message, but about really strange font matching (or https://github.com/jwilm/alacritty/issues/619 should be reopened, if it's a separate issue).
There are multiple font sizes installed here:
]$ fc-list | grep xos4 | grep Regula | sort
/usr/share/fonts/terminus/ter-x12n.pcf.gz: xos4 Terminus:style=Regular
/usr/share/fonts/terminus/ter-x14n.pcf.gz: xos4 Terminus:style=Regular
/usr/share/fonts/terminus/ter-x16n.pcf.gz: xos4 Terminus:style=Regular
/usr/share/fonts/terminus/ter-x18n.pcf.gz: xos4 Terminus:style=Regular
/usr/share/fonts/terminus/ter-x20n.pcf.gz: xos4 Terminus:style=Regular
/usr/share/fonts/terminus/ter-x22n.pcf.gz: xos4 Terminus:style=Regular
/usr/share/fonts/terminus/ter-x24n.pcf.gz: xos4 Terminus:style=Regular
/usr/share/fonts/terminus/ter-x28n.pcf.gz: xos4 Terminus:style=Regular
/usr/share/fonts/terminus/ter-x32n.pcf.gz: xos4 Terminus:style=Regular

so I don't really understand why size 12 is always used, why it can be used with size=9.0 (not 12.0) and why dpi setting affects this at all.

I'm also experiencing this issue with Terminus, but no sizes seem to work for me.

I can confirm that xos4 Terminus is working for me with size: 9.0.

Still having the same issue with ypn envypn font.

Running with the environment variable FC_DEBUG=16777215 prints a lot of info from fontconfig.
BTW, tried freetype2's FT2_DEBUG (i.e. FT2_DEBUG="any:10") but it did not yield anything.
I think that the first issue is that FcFontMatch in fontconfig does not pick the right PCF file. When configuring alacritty for the Terminus font with size 32.0, /usr/share/fonts/terminus/ter-112n.pcf.gz gets picked (size 12). This, regardless of the font size that is passed to the match. I have lots of other ter*.pcf.gz files for other sizes, of course, and fontconfig prints something about them.
The second issue is the font size disagreement inside freetype2 or around alacritty's use of it - I am not sure, have not delved into these pieces of code too much.

Tamzen used with Tamzen for Powerline works with size: 9.0 but nothing else

I've managed to fix this problem in pull request #776, more testers are welcomed.


[UI] Make Contributing to Open Source Page Responsive

Category

[ ] Resource Addition
[ ] Documentation
[ ] Codebase
[x] User Interface
[ ] Feature Addition

Description

@Sriparno08 Can you assign me this issue

Sure, @rnavaneeth992. You're assigned!


fix #254

raise datagrid row-focus event

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree


Fixing case where signingJobs is empty for non-DERIVE_KEY_AND_SIGN requests.

#139  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Nov 26, 8:00 PM EST: A user merged this pull request with Graphite.


Ported module for 1.7

Ported module for 1.7
ping @aleeks @maximebiloe

👻

Hello @kpodemski
Thank you for having turned the problem!!
Do you know where the problem is, I didn't see where and for lower than 5.4... I just make refacto..

Shorthand array

We just released a new version of the module.
Thanks for reporting @kpodemski!


unnecessary-optional-chain lint triggers incorrectly for array indexer access

When accessing values of an array of type Array<T>, undefined may be returned. But flow insists that they are always type T. I understand why this works this way and I am mostly ok with that (from a pure soundness perspective, it is not ok. But that is a different discussion).
The issue is that the unnecessary-optional-chain lint option triggers for defensive code that is actually necessary.
In the example below, a function accesses a property on the first item in the array. If the array is empty and we followed the flow recommendation, this would fail with a null-exception.
// flowlint unnecessary-optional-chain:error

function getFirst(x:$ReadOnlyArray<{| foo: string |}>) : ?string {
  return x[0]?.foo
}

I know there are multiple workarounds, I would just prefer that I did not have to use them.
Please note that this is not about making flow insist that array[index] returns an optional value, it is only about the linter accepting that this is a perfectly valid place to use optional chaining.
Flow version: 0.112.0
Expected behavior
The linter should not trigger.
Actual behavior
The unnecessary-optional-chain lint triggers.
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVxhlomBLAOwBcwBXAggUwGNKBnOgQwCcBPAWjgAci84DGMdtQAWjQgC5KzZnGboo5ar35gA5pSIAxPMzpEAFAA8JAEgBKlRgBMA8gRisAgjMasAPAG8APtjhwJMH1mQjUwbwBfAD4ASjBAgH5g0LBPVDAwZk1SZgIwIwBtAAYAXQSAOih-VAjUIA

Was discussed in flow discord channel. Posting here because related. Have edited with some formatting to make it readable.

TrySound 10/03/2020
Hey guys, I have enabled unnecessary-optional-chaining lint rule and struggle with accessing array by index. Array items can be undefined. And here lint rule tells array[0] is not nullable. Can array index be an exception for this lint rule?
const array: Array<{| a: 1 |}> = []
array[0]?.a

malectro 10/03/2020
beyond the lint rule, it's always worried me that Flow considers indexing with a literal to yield a defined value
Nat 10/03/2020
setting aside the lint rule for a moment, the array indexer return type is a long-standing debate...
let me see if I can dig up a thread
this will likely provide some useful context: https://github.com/facebook/flow/pull/6823
the summary is that Flow decided long ago to accept some unsoundness here for the sake of ease-of-use
the behavior of array indexers in JS leaves us with no good options here
if I were designing JS arrays, I would probably have included a bounds check and made them throw if the array index were out of bounds. that would allow Flow to safely assume that if an array access returns, it returns a non-nullable value (if, of course, the array contents are non-nullable)
unfortunately, because of the way JS arrays behave, we have to choose between two bad options:

Force people to check all of their array accesses for null, or
Unsoundly assume array accesses are in-bounds

the first option is especially bad because people often write their own bounds checks, but Flow does not have the machinery to analyze bounds-checks to conclude that the indices are in-bounds, e.g. in a for loop iterating through the indices of an array
so, in essence people would be forced to check twice
we are moving towards soundness in general, but we have to pick our battles. we believe this change would be pretty unpopular, and we know it would have a significant rollout burden. we may make it eventually, but we don't have any short-term plans to do so
now, to address the lint rule question:
because of the above, as far as Flow is concerned, this is not nullable. we don't have any mechanism by which to indicate that this particular type is untrusted (though we have explored doing a trust analysis, and will likely have something along those lines eventually). so, for now, there's no way to selectively disable the lint for cases like this, and I don't see a reasonable short-term path to such functionality
I know this isn't the answer you were hoping for...
sainati might also have some thoughts on this
sainati 10/03/2020
my suggestion is to add a // $FlowIgnore[unnecessary-optional-chain] comment above that line
but yes Nat summarized the issue well and atm don't expect anything to change here
TrySound 10/03/2020
We currently do length check instead of optional chaining to not hide other errors.
But it's forgotten sometimes
sainati 10/03/2020
you won't hide other errors if you use this suppression, that will only suppress this lint specifically
TrySound 10/03/2020
I mean other chaining errors
malectro 10/03/2020
obviously you guys have thought about this way more than i have so this is probably silly to ask, but was there any discussion around the difference between array[0] and array[i]? the first is something i'd always assume to be possibly undefined – to the point where i wrote a custom first<V>(Array<V>): V | void  utility in our code base to use instead.
sainati 10/03/2020
ah i see
TrySound 10/03/2020
Btw, I saw proposal about array.item(i). Can it be nullable like map.get(key)?
Nat 10/03/2020
malectro the fundamental issues at play aren't really different. your utility is a reasonable thing to add
TrySound I'm not aware of that, but if it's something new I think we could make it nullable
https://github.com/tc39/proposal-item-method
nice, stage 3
yeah, when we add that I think we should add it with a nullable return value

https://github.com/facebook/flow/commit/1cd0a55ec66a330a5af7f77cbdd626eec106f3fa implements the behavior will include void for array indexed access. Under that flag, this lint should be reliable


Fix table on readme

Pull Request Test Coverage Report for Build 3

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

Totals

Change from base Build 1:
0.0%

Covered Lines:
126

Relevant Lines:
135

💛  - Coveralls


Hub cards can have very little text

The top left one have the title and tags overlap

Could consider re-design to make room for more, and/or check the length in the PR contributions.

Hi @flynnduism and @karthik2804
This could be solved in one or more of the following ways:

Limit the amount of text allowed in a title
Reduce the font of title
Increase the layout of the card

I am not sure which one (perhaps a combination of 2 and 3 might be the way to go)


Android Kotlin: 01.1 Requirement of VCS

Description :scroll:

Give a step-by-step tutorial of Version Control System.
Required Video and Audio.

Domain of Contribution :bar_chart:

[x] Android Dev (Kotlin)

References :
https://git-scm.com/book/en/v2/Getting-Started-About-Version-Control
https://guides.github.com/activities/hello-world/
Please Read it Carefully Before Contributing

Changes should be made in Android_Development_With_Kotlin/ directory under Android_Development_With_Kotlin branch.
Inside Android_Development_With_Kotlin/ directory creates a directory and names it according to the topic ( for this issue 09. Dynamic User Interface)
Inside 09. Dynamic User Interface/ directory add your task with subtopic number and name and make changes accordingly.
You have to push your PR to the Android_Development_With_Kotlin branch only.
You can take help to the references but don't simply try to copy it. Just explain in your own word.
This issue is for GWOC'21 contributors only.
You need to push in a single commit. If there exists more than one commit then you need to squash your commits into one commit.
Will assign you on a first come first serve basis.
Please try to complete your assigned task within 7 days. If you need some more time then, please inform mentors and supervisors.
If you have two previously assigned issues on Android-Kotlin pending then please first complete them.

Supervisor @rahulsain

Happy Contributing. Best of Luck.:sparkling_heart:

Hey @rahulsain , Because the previous issue of 01.1 Requirement of Version Control System #453 , is already closed and Audio and video part are remaining .
Assign me this issue for Audio part . I would like to make an "  Audio "  file on this issue .


The hello world example does not work browser for TVs and IE11

Version
3.0.0-rc.9
Steps to reproduce
Just follow the steps here and create a small hello world application:
https://v3.vuejs.org/guide/installation.html
Now, open the URL http://localhost:3000 on your IE11 (or any browser in a TV, like Opera).
What is expected?
It should show the hello world page.
What is actually happening?
It shows only a blank page, without raising any error in the browser console.

The Vue3 should support browsers used in TVs like Sony, Samsung etc., which in general uses Opera as web browser.

IE 11 will be supported but it's not yet. What is the specific opera version you are targeting?

The Opera is of a Sony TV, and it shows only "2015" on its about dialog. :-/

If the browser does not support baseline ES2015 then it won't work. That's expected.
When IE11 is supported, those browsers are likely supported as well.

When Is Vue3 Compatible with Internet Explorer 11?


Update CSS Logical Properties for Chrome 87+

Chrome 87 now supports the properties stated in the notes #3 #4 by default. Specifically:

inset, inset-block, inset-inline
margin-block, margin-inline
padding-block, padding-inline

Reference Chrome Platform Status

Thanks for the test! They seem to be somewhat outdated—The left red one checks for margin-start which I believe never got standardized according to https://github.com/w3c/csswg-drafts/issues/3651, and for historical reasons they work with -webkit -moz prefixes. Many other properties are also missing. Should be revised in one way or another.
Loosely related to #2745.

Yeah, these tests only pick some of the properties... and if there's a gray dotted outline it means it's deprecated or something, but is worth checking for partial support.
Thanks for the update!


mkOverridable: Use functors to allow overridable functions

As an example use case, take NixOS/nixpkgs@96ae5d58bfa3bb64449bef3703238a1cdc76eff0 . Currently we have to override jailbreak-cabal for the whole set, but with this change we could instead override mkDerivation only.

cc @edolstra

Let me see whether I understand this correctly ... with this patch we could make the mkDerivation function overridable, and then we could override the jailbreak-cabal argument to that function instead of replacing the jailbreak-cabal attribute in the package set. Is that right?

Yep. could have mkDerivation = super.mkDerivation.override { jailbreak-cabal = pkgs.haskellngPackages.jailbreak-cabal; } in haskell-configuration-ghc.7.9.x.nix

BTW, makeOverridable is called by callPackage, so with this patch mkDerivation will already be overridable.

Very nice! :+1: from me.

(check out http://nixos.org/nix/manual/#ss-functions , the bit about __functor, for details)

Awesome!

@peti merged

I don't really understand the motivation for this. I mean, you can just say
mkDerivation = args: super.mkDerivation (args // { jailbreak-cabal = ... });

right? So why is the .override needed?

No, mkDerivation is a result of callPackage on a function that returns a function. jailbreak-cabal is an argument to the first function, not the second.

What is super and what is mkDerivation (apparently it's not the usual mkDerivation)? (I have no idea what this code does...)
 For instance, they prevent garbage collection because they keep the arguments to a function alive after the function has been called.

@peti can probably explain super better than I. mkDerivation is the haskell-specific one, see https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/haskell-modules/default.nix#L18-L28.
Do you have an alternative to overrides here?

There is always an alternative to override, namely, call the original function with the right arguments. For instance,
foo = function { x = true; };
bar = foo.override { x = false; };

is equivalent to
bar = function { x = false; };

@edolstra, the haskell-ng code currently doesn't use this feature. We could have used it to implement a hack that worked around some temporary issue, but since the issue has been resolved, that hack is gone, too. All overrides that we currently use are "normal" ones.

@edolstra So should we remove mkOverridable all together?

@edolstra BTW, given the import cache, doesn't the normal use of callPackage (i.e. calling it on a file) prevent GC of the function anyway?

@edolstra, if we define every Haskell build as a function, then we'll end with 7500 functions + 7500 derivations = 15,000 attributes. Is that going to be more efficient that 7500 derivations plus ~200 overrides?

@shlevy, this doesn't look good: http://hydra.nixos.org/jobset/cabal2nix/master#tabs-errors. Hydra.nixos.org cannnot evaluate Nixpkgs. :-(

I have the same error as hydra.nixos.org locally.

@rbvermaa @edolstra Can we update hydra? Regardless of the desirability of this particular feature, it was my impression that master was allowed to use features present in nix stable on the latest stable branch.

I've updated Hydra to 1.8.

Should we document how this is handled for nixos? https://botbot.me/freenode/nixos/2015-01-17/?msg=29836376&page=7


Peek definition works like Go to Definition if it runing from command palette

VSCode Version: 1.9.1
OS Version: MacOS 10.12.3

Steps to Reproduce:

Place cursor on a simbol
Open command palette and choose Peek Definition
Repeat this several times.

Instead of showing a Peek Definition window, the editor opens a file with a definition. It happens in about 50% of cases.

I can reproduce... It seems like the Enter from the command palette reaches the preview widget and then is interpreted as go to definition.

The problem is that the F1 list seems to dispatch on key_down where the default tree controller dispatches on key_up. That means the key_down triggers peek definition and that the key_up of that same keypress selects the definition from the preview tree, therefore revealing it.
@joaomoreno @bpasero Why do the tree widgets behave so different? Why does the tree by default work on key_down except for Enter and Esc?

Easy steps to reproduce

open TypeScript file, select a symbol
F1 > Peek Definition
Press Enter, do not release key
when the preview shows, release key
🐛 definition is revealed

Good old https://github.com/Microsoft/vscode/issues/6146


New Plot Type: Circle

Maybe i just miss something trivial, but I tried this:
const double r = 300;
var func1 = new Func<double, double?>((x) => Math.Sqrt(Math.Pow(r, 2) - Math.Pow(x, 2)));
var func2 = new Func<double, double?>((x) => -Math.Sqrt(Math.Pow(r, 2) - Math.Pow(x, 2)));

formsPlot1.Plot.Clear();

// Add functions to the plot
formsPlot1.Plot.AddFunction(func1, lineWidth: 3, color: Color.Blue);
formsPlot1.Plot.AddFunction(func2, lineWidth: 3, color: Color.Blue);

// Manually set axis limits because functions do not have discrete data points
formsPlot1.Plot.SetAxisLimits(-300, 300, -300, 300);

formsPlot1.Plot.Render();
formsPlot1.Refresh();

then i get this:

Hi @bukkideme,
This is an interesting problem. The gist is that function plots just walk the screen from left to right, and for each pixel column (X) it runs the function and plots the point (Y). The artifacts you circled here are caused by the fact that the X pixel positions don't perfectly line-up with the values your function would return 0 for.
This is the function that walks from left to right and calculates Ys for each X.
https://github.com/ScottPlot/ScottPlot/blob/9c81ea29063051efe6f1cb3110929186229811bb/src/ScottPlot4/ScottPlot/Plottable/FunctionPlot.cs#L46-L79
If you really wish to draw a circle, I think the best way to go would be to create a new plot type for this (something that implements IPlottable). I've been meaning to write a FAQ page for a while describing how to do this (https://github.com/ScottPlot/ScottPlot.NET/issues/10) and this seems like a really easy plot type to make, so I'll leave this issue open to remind me to do this soon.

Hi! Cool, it would be very useful. Just to explain my motivation: here i only wanted the circle to act as a "decoration". I need to make a plot where i show data points inside a circle. So in my case, the circle is just a static decoration.
This leads to my question: what about to make a general "decoration" plottable class? Using this, circles, ellipsis, rectangles, etc could be drawn on a Chart. The default behaviour would be that they scale with real data, so follow zooming, panning, etc. What do you think?

This leads to my question: what about to make a general "decoration" plottable class?

This could work, but envision that class growing larger and larger (infinitely) as new decorations are thought of. A more long-term sustainable and maintainable solution could be to create small classes (plot types) for new decorations as they are thought of, and also make it easy for users to create their own plot types.
However, Ellipse and Rectangle plot types seem useful.
The Arrow and Text plot types are good examples of "decorations" that I think make sense to keep separate

https://scottplot.net/cookbook/4.1/category/plottable-arrow/
https://scottplot.net/cookbook/4.1/category/plottable-text/

Hi @bukkideme,
I just made a FAQ page demonstrating how to make a custom plot type.
It does not require editing any ScottPlot source code 🤯
Do you want to try to use this new guide to make a CirclePlot on your own?
If it works well in your hands you can add it to ScottPlot's folder of plottables as PR and I'd be happy to review it!

forgot the link: https://scottplot.net/faq/custom-plot-type/

Thanks very much, i will have a look!


Implemented clear test

Added tests:

Clear should empty target collection
Isar clear should clear every collection
Clear already cleared collection
Clear should reset auto increment

Looking great!!


Continuous rendering results in random crashes.

For a live renderer i use the following code:
private class ContinuousGlSurface : GLSurface
{
	private const int FrameDelay = 1000 / 60;

	private readonly Stopwatch stopwatch;

	public Action RenderLoop;

	public ContinuousGlSurface(GraphicsMode graphicsMode, int major, int minor, GraphicsContextFlags graphicsContextFlags)
		: base(graphicsMode, major, minor, graphicsContextFlags)
	{
		this.stopwatch = Stopwatch.StartNew();
	}

	protected override void OnDraw(EventArgs args)
	{
		base.OnDraw(args);

		if (this.stopwatch.ElapsedMilliseconds >= ContinuousGlSurface.FrameDelay)
		{
			this.stopwatch.Restart();
			this.RenderLoop();
		}

		this.Invalidate(false);
	}
}

This works flawless.. basically, because it only works properly with either single buffering (due to frameswap glitches), or not using the stopwatch because not using it will results in rendering as fast as your pc can.
So this works till suddenly out of nowhere a random (sometimes after seconds, sometimes after minutes) crash appears:
OpenTK.Graphics.GraphicsContextException: Failed to make context 65537 current. Error: 2004
   at OpenTK.Platform.Windows.WinGLContext.MakeCurrent(IWindowInfo window)
   at OpenTK.Graphics.GraphicsContext.MakeCurrent(IWindowInfo window)
   at Eto.OpenTK.WinForms.WinGLUserControl.MakeCurrent()
   at Eto.OpenTK.Wpf.WpfWinGLSurfaceHandler.MakeCurrent()
   at Eto.OpenTK.Wpf.WpfWinGLSurfaceHandler.UpdateView()
   at Eto.OpenTK.Wpf.WpfWinGLSurfaceHandler.<AttachEvent>b__7_2(Object sender, PaintEventArgs e)
   at System.Windows.Forms.Control.OnPaint(PaintEventArgs e)
   at Eto.OpenTK.WinForms.WinGLUserControl.OnPaint(PaintEventArgs e)
   at System.Windows.Forms.Control.PaintWithErrorHandling(PaintEventArgs e, Int16 layer)
   at System.Windows.Forms.Control.WmPaint(Message& m)
   at System.Windows.Forms.Control.WndProc(Message& m)
   at System.Windows.Forms.ScrollableControl.WndProc(Message& m)
   at System.Windows.Forms.ContainerControl.WndProc(Message& m)
   at System.Windows.Forms.UserControl.WndProc(Message& m)
   at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
   at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
   at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

When searching for that crash, i came across this one: https://github.com/MonoGame/MonoGame/issues/3196
Which looks quite similar.

hm, weird.  Does this happen with the code above (with no GL code), or does it require any GL code to make it happen?


Fixed issue #70

/claim #70

Thanks man for the fix, the goal was to make it customizable whe you use the plugin not to set it in the code


Documentation for config files

We're making good progress on documenting the library functions. I wonder is there is an equivalent way to capture the config file options. There should probably be a documentation page for how to create and use them, at the very least.

This sounds like a great idea!
I'm not entirely sure how to capture the config options unless we standardize the syntax for it. When I work on creating a downloadable version of each script (https://github.com/Nick-Mazuk/jw-lua-scripts/issues/70), I'll try to figure this out. I'm hoping to be able to be able to list every script in the repo on the site, so people can just download the scripts straight from the site without ever touching GitHub. I'll add the config options there.
In a similar vain, would it make sense to create a separate script that allows users to update the configuration files for any of the scripts? That way, they won't need to manually go to the txt file to update the values.
In the meantime, I fully support adding a documentation page for creating and using the configuration files. You just need to add a markdown file to the /docs folder. For instance, /docs/configuration.md.

Does this page resolve this issue?
https://www.finalelua.com/docs/rgp-lua/rgp-lua-configuration

No. That's for RGP Lua. I was thinking about the optional config files in lib/configuration.lua

Ok. Here's one shower thought I had if RGP Lua could set environment variables.
For instance, let's say we had this pseudocode:
local file = "my_script.config.txt"
if process.env.REGISTER_CONFIGURATION then
    for key, value in pairs(config) do
        -- `rgplua` is a namespace specifically for
        -- RGP Lua similar to `finenv` or `finale`.
        rgplua.RegisterConfigParameter(file, key, value)
    end
end

-- Later in the code...
-- If sandbox mode is on, this is not strictly necessary
-- but could be useful if the script is expensive to run
if process.env.RUN_SCRIPT then
    do_main()
end

Then if RGP Lua had a way of setting the configs for a particular script in the plugin itself, it could silently run the script behind the scenes and see which config values were registered. Preferably it would also set sandbox mode to true.
Since all configs must eventually call configuration.get_get_parameters(file_name, parameter_list), we could add this registration to that function behind the scenes.

I'm having trouble understanding exactly the issue you want to address, but in general I think we should keep RGP Lua simple and do things in Lua code to the extent possible.
Could this be an enhancement of the configuration library itself? Or a separate library?
For what it's worth, "sandbox mode" was never actually implemented in the PDK Framework as far as I can tell. Instead, NoStore simply forces a blanket rollback of any changes that were made by the script. Lua scripts can do this as easily as the PDK Framework can.


BUG - @sap-ux/create

Description
In @sap-ux/create dependencies is "@sap-ux/mockserver-config-writer": "0.1.10", but max version only "0.1.9"
Steps to Reproduce
Steps to reproduce the behavior:

npm i @sap-ux/create
See error

Screenshots

@kranthie-sap any idea what went wrong with the publishing to npmjs? https://github.com/SAP/open-ux-tools/blob/main/packages/mockserver-config-writer/package.json shows version 0.1.10.

I created a PR.

Thanks, @devinea.

@devinea Everything is working now, thank you very much!

Great. Thanks @3lodey


Request for Code Documentation

It would be awesome if we could add some comments to the code to make it easier for everyone to understand what's happening. Commented code not only helps us maintain it better but also encourages collaboration from the community.

@Tusharknwl  can you allow me to work on this

@MohdSaquib114 yeah you can work on it.

Thank you @Tusharknwl  for assigning me this task. I will reach to  you  in future if a need your help in this task.

Thanks for giving me the opportunity to work on this issue. I will complete it in some times


fixes bad log ordering with multiple requests

When issuing multiple network calls the notification sometimes comes after the next request, so AFNetworkActivityLogger logs in wrong order

No, the current behavior is correct. Notifications should be sent out after the completion block begins, which means dispatching within the completion group.

The current behavior leads to fail if we have many requests. Let's say we do this:
request A
request B (using some data retrieved from request A)
The process works as expected, but not the log.
Expected
log request A
log data received from request A
log request B
Observed:
log request A
log request B
log data received from request A
...
I created another pull request and moved the postNotification right just before the completionHandler and inside the dispatch_group_async. Tested with the same environment and works as expected
https://github.com/AFNetworking/AFNetworking/pull/2574

@pabloroca What you described is exactly the desired and expected behavior. Notifications are sent off after the completion block fires. If you're sending a new request in a completion handler, that's just what will happen.


Add help message indicating {create,delete}-firewall is not necessary

First time I ran kubehost -h these commands confused me:
  create-firewall <service>  : Create a GCP firewall rule for the hostPort
                               deployment created with 'bind'.
  delete-firewall <service>  : Delete a GCP firewall rule for the hostPort
                               deployment created with 'bind'.

I think they are "not needed" if you're using bind/unbind.
Please consider adding a note to these cmds saying something in the lines of:

Not needed if you use bind/unbind.

Good point.


Removed dead links from unofficial books

I noticed a few links were dead. Removed them.

Thanks @hrishim! I reinstated one link which is now ok and merged the rest of your changes.


Konfigurator unit tests

Added some tests for Konfigurator for completeness

Coverage increased (+20.9%) to 44.776% when pulling f332dfc76409656f8232083d4c8b87bba2d79631 on konfigurator-test into b46f1f7a8421f868865a5e55afa18c13b12531d8 on master.


The endpoint https://graph.microsoft.com/v1.0/external/connections returns 429

Describe the bug
I am trying to retrieve external connections via https://graph.microsoft.com/v1.0/external/connections, but the API returns 429
Not sure why it is not working now because it was working before.
To Reproduce
Steps to reproduce the behavior:

Go to (https://developer.microsoft.com/en-us/graph/graph-explorer)
Click on "Resources" -> External -> Connections -> Get Connections
Click on Run Query and consent with the required permission
Instead of seeing existing connectors I See error 429

Expected behavior
All the existing external connectors should return
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hey @xinzhaozhang1985. I have reproduced this problem on postman. The endpoint has an issue. Repost this question on the Graph Q&A forum.


fix: improve spray proxy error handling

Description

added error severity level for Slack alert messages (it uses different icons in a header of the slack message based on the severity level of an error message).

error when registering a new server to SprayProxy is considered as an error and should require attention
error when unregistering server from SprayProxy is not critical and the message is just informative:

improved spray proxy registering/unregistering flow and error messages

enabled spray proxy registering for e2e-tests rehearsal jobs

Issue ticket number and link
RHTAPBUGS-905
Type of change

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

How Has This Been Tested?
locally against a private slack channel
Checklist:

[X] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/test all

golang proxy server outage
/retest

/retest

golang proxy server outage again
/retest

/assign @xinredhat @Dannyb48

/approve

/lgtm


Create some form of Contributing documentation

As an open-source contributor, one who is not a member of Open Austin, it would be awesome to have some sort of documentation around making contributions to the repo.
Benefits:

allow contributions from the open community
visibility into the Git workflow
opportunity to standardize new issue, commit, and pull request guidelines
potential quicker turn around on issues

:tada:


Refactor SequenceNetwork

Should enable to:

generate network, based on SequenceRecord object
serialization to json
add identity attribute to each edge, based on pairwise alignment
optional calculation of betweenness, centrality
cytoscope visualization

resolved by #79


GEODE-7541: Revert back to SocketCreator in GMSUtil

One of the changes for this ticket is causing unit test failures on
some machines - especially Macs - in the form of
java.lang.RuntimeException: Failed to start locator
Caused by:
org.apache.geode.GemFireConfigException: This process is attempting to join with a loopback address (gzhou-mbpro/<IP_ADDRESS>) using a locator that does not have a local address (/<IP_ADDRESS>:60167). On Unix this usually means that /etc/hosts is misconfigured.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

closing this PR - there is a new one that provides a permanent fix


regenerate Hint.Extension (fixes #34)

I did runghc generate/mk_extensions_mod.hs and a trial build.

Could you mention GHC 8.2 in the message?

I did amend it to mention ghc-8.2rc2 (10 minutes before your comment incidentally).

Ah - was on my phone and I guess I didn't refresh.


Make it possible to use multiple datagrids on one page

More info can be found here https://forums.bagisto.com/topic/379/multiple-datagrids-on-one-page

Hi @ghermans,
I have started working on this PR and I am using the same refactored PR.


[WIP] add cyclotomic square of fp12 element

Add cyclotomic square of fp12 element
Description
This PR attempts to add faster squaring for 6th cyclotomic subgroup elements.
I imagine this would be the first step towards implementing #544
I tried implementing what I described in this comment  but there are obvious gaps in my knowledge which is apparent in the code. Hence, I am hoping to get reviews here to bridge those gaps and clear away my misunderstandings.
Type of change
Please delete options that are not relevant.

[x] New feature
[ ] Bug fix
[x] Optimization

Checklist

[x] Linked to Github Issue
[x] Unit tests added
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

[x] This change is an Optimization

[ ] Benchmarks added/run

As mentioned in Telegram
It's important to mention in document that the paper uses a Fp2->Fp4->Fp12 towering while usually people implement Fp2->Fp6->Fp12.
Formulas need to be adapted like this depending on towering: https://github.com/mratsim/constantine/blob/master/constantine%2Fmath%2Fpairings%2Fcyclotomic_subgroups.nim#L278

Thanks for the reference!
Unless I'm missing something, the paper seems to derive the squaring for Fp6 (by cubic over quadratic, Fp -> Fp2 -> Fp6) and not Fp12. I tried to follow the same train of  thought here in this rundown , which I believe is quadratic over cubic over quadratic (Fp -> Fp2 -> Fp6 -> Fp12). Could you maybe go through that and let me know if it makes sense ?

Unless I'm missing something, the paper seems to derive the squaring for Fp6 (by cubic over quadratic, Fp -> Fp2 -> Fp6) and not Fp12. I tried to follow the same train of thought https://github.com/lambdaclass/lambdaworks/issues/544#issuecomment-1925423017, which I believe is quadratic over cubic over quadratic (Fp -> Fp2 -> Fp6 -> Fp12). Could you maybe go through that and let me know if it makes sense ?

The paper is about cyclotomic subgroups first and foremost, and use Fp6 as an example with Fp->Fp2->Fp6. You're using Fp12, so paper formulas are for Fp2->Fp4->Fp12.
Hence either you match the paper cubic over quadratic or you adapt them.

So should I be deriving the formula for Fp6 via Fp -> Fp3 -> Fp6

No, you forgot about Fp, completely. What matters is how Fp12 is built as a 6th degree extension. And it's built as a 6th degree extension from Fp2.

So should I be deriving the formula for Fp6 via Fp -> Fp3 -> Fp6

No, you forgot about Fp, completely. What matters is how Fp12 is built as a 6th degree extension. And it's built as a 6th degree extension from Fp2.

Okay I see your point. I'll try to revaluate.

This functionality has been added


missing field for nif-version

Just noticed that there is no field in the documentation to specify the nif version.
https://api-dev.freme-project.eu/doc/api-doc/simple.html#!/e-Terminology/e_terminology

Will be added in #294.


instructions

Hi i want to colorize my black and white video i downloaded the model and ckpt_woflow i want to ask how to use this thing. Thanks

Transfer video to frames
Colorize the frames according to the instruction
Transfer fraems to video.


Docs: improved README examples of stats/base/dists/cosine namespace

Resolves #1620
Description

What is the purpose of this pull request?

This pull request:

Improves the README examples for the stats/base/dists/cosine namespace package

Provides a more comprehensive and informative example demonstrating various functionalities of the package

Enhances user understanding of how to use the namespace contents

Related Issues

Does this pull request have any related issues?

This pull request:

Resolves [RFC]: Improve README examples for stats/base/dists/cosine namespace package #1620

Questions

Any questions for reviewers of this pull request?

The updated example covers creating a distribution object, calculating properties, evaluating functions, generating random samples, and calculating moments. Are there any other key functionalities that should be included?

Is the level of detail in the example appropriate, or should it be expanded/condensed?

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

The example uses ES5 syntax as per project conventions

No additional packages were required to create an informative example

Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

[x] Searched for existing issues and pull requests before creating this one.

[x] Updated the example code block in the "Examples" section of the README.

[x] Ensured the example follows existing project conventions (ES5 syntax).

Hi @Planeshifter,
Could you please review the code? I have raised a PR.


Shareable URLs

Great project!
I suspect the project would be used a lot more if it were easy to share a URL for a specific city or location. Something like https://www.openparkingmap.com/#philadelphia that I could embed in an article would be incredibly helpful!

good idea! To make it work for any city I'd have to use a geocoder that might not always hit, but would at least work for major cities, or could hardcode some values in.

Another thought is that if there's a way to pull the latitude / longitude of the center of the screen and the current zoom level then that data could always be encoded in the URL.
Then sharing a specific perspective would be just be as easy as sharing the URL.

good call! I can work on that

FYI @kettle11

@brandonfcohen1 Thank you! This is excellent and makes this website all the more useful.


Update for Konnectivity?

If I understand correctly, aks-link is being replaced by konnectivity.  Might it be worth updating this article to reflect this change, as I suspect the same error would be encountered if there is an issue with the konnectivity-agent pods.

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

ID: 03b2de19-c6a7-5ed3-f0ec-115a1abbcd28
Version Independent ID: 03b2de19-c6a7-5ed3-f0ec-115a1abbcd28
Content: Error from server: error dialing backend: dial tcp - Azure
Content Source: support/azure/azure-kubernetes/error-from-server-error-dialing-backend-dial-tcp.md
Service: container-service
GitHub Login: @DennisLee-DennisLee
Microsoft Alias: v-dele

Thanks for the feedback, @lukemaslany-next. I've updated the article. We'll let you know when the changes go live.

The article update is now live.


Firesquid governance events batch 1

Your util for prawns/governance/src/utils/createCallHandlerFromEventHandler.ts is much nicer than mine. Let me know once you have resolved my comments - will rewrite my PR to use your util once its merged.

I've merged this @markdavison - Feel free to use the dodgy util 😅


Some feedback and confusion after trying use this plugin

How can I make log from tauri-rust-end? It seems this plugin only for front-end's log. I try to re-export log module inside tauri-plugin-log, but not work.😭

When using with_colors, the log file and the FE console.log appears the color code which is unwanted.

console.log

log file

terminal

Check the log message, the log from FE but why the target shows tauri-plugin-log?

Why the terminal show logs from tao package?

// here is my config of using this plugin
.plugin(
  LoggerBuilder::new()
    .rotation_strategy(RotationStrategy::KeepAll)
    .max_file_size(4000)
    .targets([LogTarget::LogDir, LogTarget::Stdout, LogTarget::Webview])
    // .filter(|meta| meta.target().eq("tauri_plugin_log"))
    .with_colors(ColoredLevelConfig {
      error: Color::Red,
      warn: Color::Yellow,
      debug: Color::White,
      info: Color::BrightGreen,
      trace: Color::Cyan,
    })
    .build(),
)

How can I make log from tauri-rust-end? It seems this plugin only for front-end's log. I try to re-export log module inside tauri-plugin-log, but not work.😭

The way rusts logging system works, you can just add the log crate as a dependency. See 4. for more on this.

When using with_colors, the log file and the FE console.log appears the color code which is unwanted.

I agree this is a bit annoying, it's not really easy to get rid if these codes since all LogTargets share one stream of messages, but maybe we can find a solution for this regardless.

Check the log message, the log from FE but why the target shows tauri-plugin-log?

That's where we technically produce the logs from the frontend (rusts logging system has obviously no idea about JavaScript stuff). This is something we can change very easily though 👍🏻

Why the terminal show logs from tao package?

The way rusts logging system works is quite ingenious, any crate can depend on log and the exported logging macros to produce logging messages. But by default this does absolutely nothing, no logs to stdout, nothing.
You have to add another crate, a so-calling logging frontend that accepts logging messages and forward them to some output. tauri-plugin-log is such a frontend (with some tauri specific additions).
This design makes modular logging a possibility. If you need to debug a problem, getting information from lower level crates can be very helpful. But you as the final consumer remain in control of all the messages.

Thank you very much. Your answers are really helpful. 👍👍👍


fix: send chat id with analytics events

Description

sends chat id along with analytics events

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