proposal: crypto/x509: support code-constrained roots

CertPool and Certificate.Verify currently only support roots constrained using X509 semantics. For most cases this is fine, but with the introduction of the golang.org/x/crypto/x509roots/fallback package (#43958), which plans to use the Mozilla NSS root list, there is a need to implement additional constraints.
The Mozilla NSS list encodes additional trust constraints for specific root certificates, such as only trusting certificates issued before a specific date (e.g. what is happening with TrustCor, or what previous happened with Symantec), etc. In order to support these constraints we need a new API which allows for attaching additional checks to certificates which chain to particular roots.
The API suggested below, CertPool.AddCertWithConstraint, would allow attaching a callback to a root, which Certificate.Verify would call during chain building. There would be no other externally visible API changes, as the root->constraint mapping would be a private structure in the CertPool, accessible only by Certificate.Verify.
As always the name is up for debate. The meaning of 'Constraint' in X509 is perhaps overloaded, AddCertWithAdditionalChecks could also work, but is possibly a little verbose. Another question is if the function should accept a single callback, or a list of callbacks (i.e. should the caller compose all of their constraints into a single callback, or should we allow passing multiple callbacks that we check ourselves serially).
// AddCertWithConstraint adds a certificate to the pool with the additional constraint.
// When Certificate.Verify builds a chain which is rooted by cert, it will additionally
// pass the chain to constraint to determine its validity. If constraint returns a non-nil
// error, the chain will be discarded.
func (*CertPool) AddCertWithConstraint(cert *Certificate, constraint func([]*Certificate) error)

cc @FiloSottile @golang/security

This allows adding constraints to individual certs as they're added to the pool, but what if you're starting with the system pool and want to layer additional checks on top of existing certs/system verifier checks?

I don't think we want to support adding additional arbitrary constraints to platform verifiers. Platforms have their own ways of applying constraints to trust stores (i.e. both Windows and macOS allow you to constrain certificates in complicated ways), and I think we'd prefer that the platform verifier, that can more properly understand those constraints, apply them themselves.
If users really want to apply their own constraints purely in code, that can be done by wrapping calls to Verify with the logic that excludes specific chains.
For platforms that don't allow applying additional constraints (namely Linux), the root pool can be loaded manually, and constraints can be added at that point (we've considered exposing the logic used to load Linux trust stores, which perhaps we should do to make this simpler).

What is the motivation for this, and how does it affect the feasibility of the algorithms for building the chain? Do we have to build all possible chains ending at a given root to try to make the constraint happy?
And why is it only about roots? That makes the combinatorics better (but not great), but nothing in the API suggests it would only be about roots.
Is there something simpler we can do, like allowing a parent cert to disallow a direct child?

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

The motivation as described in the parent comment is that there are trust stores that encode additional trust constraints outside of those that can be defined in X509. In particular with the introduction of the golang.org/x/crypto/x509roots package, there are roots in the NSS list that contain additional constraints around trust based on issuance dates. Currently we just ignore roots with these extra constraints, but it'd be more valuable for users to attempt to match behavior of NSS itself, and apply the additional constraints consistently.
This has no meaningful impact on the chain building algorithm, it can be implemented in a similar way to how EKU chaining checks are applied. We already build all chains from a leaf.
It is unlikely you'd want to apply constraints to intermediates (the only other anchor type feasibly in a CertPool), but there is not really any extra complexity to check both roots and intermediates (either during the path building process, or once it is completed).

Spoke to @rolandshoemaker about this. The code already generates all possible chains (paths from the leaf back to a known root), and while that is potentially exponential in the number of intermediate certs, you can't get an signed, trusted intermediate cert without essentially being a CA, so the opportunity does not really exist for a malicious peer to create a large enough graph problem to cause issues. Things are also helped by the relevant TLS message being fairly limited in size.
This new API locks us in to that decision, but some other semantics of certficate chain validation arguably do too, and since the attack surface seems not to exist, using this API seems okay to me.

Have this all concerns about this proposal been addressed?

I believe so.

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group


Protocol should all be documented

Lobby Quick is now fully documented

Lobby Registered mode is now fully documented

General commands are now fully documented. \o/ Only Game commands remaining

Everything is now documented properly :+1:


fix: HSA-wrw9-m778-g6mc vulnerability with bl

#minor
Description
This PR updates the bl versions grater than 4.0.0 to avoid the HSA-wrw9-m778-g6mc vulnerability.
Specific Changes

Updated bl to versions greater than 4.0.0 in yarn.lock file.

Testing
The following image shows the bl versions installed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

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 84.048%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
20351

Relevant Lines:
22912

💛  - Coveralls

fixed in https://github.com/southworks/botbuilder-js/pull/464


Reduce the amount of audit messages

Task linked: QF-1837 Do not audit changes on last_login, last_used_at at etc


Design an avatar component

All of our systems have user integration in some way. Therefore it would be nice to have a streamlined representation of a user with a profile picture.

It is possible to display the avatar of a user
If the user has no avatar, the first letters of the first and last name should be displayed

Inspiration

This is the representation of myself in one of our systems:


Requests to /api/v1/series are not cached correctly

Thanks for your work on Trickster, it's made our dashboards a lot more pleasant to use.
Some of our Grafana dashboards use label_values() to populate templating variables. That function calls /api/v1/series[]match={label_name}&start=... and we've been seeing some strange behaviour when that endpoint is served by Trickster.
Trickster does not appear to use match[]/start/end in the cache key. Results for this endpoint are always cached with the key {origin}/api/v1/series regardless of what data is being requested from Prometheus.
This should be pretty easy to reproduce with any setup. I've been the simple docker-compose config in this gist to look into the issue.
# Against trickster
$ curl -w '\n' -g 'http://localhost:8000/api/v1/series?match[]=prometheus_build_info'
{"status":"success","data":[{"__name__":"prometheus_build_info","branch":"HEAD","goversion":"go1.10.3","instance":"localhost:9090","job":"prometheus","revision":"188ca45bd85ce843071e768d855722a9d9dabe03","version":"2.3.1"}]}

# Against trickster
$ curl -w '\n' -g 'http://localhost:8000/api/v1/series?match[]=this_is_a_fake_key_that_does_not_exist'
{"status":"success","data":[{"__name__":"prometheus_build_info","branch":"HEAD","goversion":"go1.10.3","instance":"localhost:9090","job":"prometheus","revision":"188ca45bd85ce843071e768d855722a9d9dabe03","version":"2.3.1"}]}

# Against Prometheus directly
$ curl -w '\n' -g 'http://localhost:9090/api/v1/series?match[]=this_is_a_fake_key_that_does_not_exist'
{"status":"success","data":[]}

I was considering opening a PR, but I'm not sure of the best approach.

Should the cache be bypassed for any endpoint that Trickster is not specifically configured to handle, or should some ordered set of all match[] values be included in the cache key?
What about start/end? Those don't appear to be used for other cache keys. I imagine that's related to the step normalisation?

👍 I've seen this behavior as well.
I'm in favor of not caching explicitly handled API endpoints.

Yup, confirmed, our public dashboard now works correctly.
https://monitor.gitlab.net/dashboard/db/unicorn-controller
It would be good to add correct cashing of this API endpoint. :-)


Make variable exclude test less strict

For #6525

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

I got a run of the nightlies going on this. So I won't submit until I've confirmed that the test is fixed there as well.


Dynamic JSON schema

We are about to finish the initial ticket for static JSON #81  schema and now is the right moment to continue working on this topic and implement dynamic JSON schema.
I think that we can start looking at this class that returns us all needed Metrics, Filters and Decorators: src/BasicRum/CollaboratorsAggregator.php
Let's outline how we could implement this:

Contain the needed logic for the dynamic JSON schema in separate class
Write bunch of unit tests in order to confirm that our current implementation works. Also this will prepare us for future refactoring because at some point we will reorganize where some files are stored in the files system.

I also expect a lot of discussions here because this task is a huge step and it will challenge the current software architecture of Basic RUM.

Hello @ceckoslab
I am currently working on a class - SchemaOrchestrator. This is kind of test schema builder implementation. Once it is done I hope to see what parts it has in common with DiagramOrchestrator and others.
My idea is that then we can decide if we should incorporate schema builder into diagram builder or leave it as a standalone part

Hello @korzol
I like the idea for having a schema builder as a standalone part.
I am curious if during this task we will have software design changes. Are you planning to add new methods and do changes in Metrics, Filters and Layout classes? If yes, then could you share some details. Also did you play with Unit Test for the new class SchemaOrchestrator?
Have a nice day.

I'd like not to change anything currently existing. For now. If there will be any changes required I will do it and let you know, so we can discuss it.
As of Unit Tests - not yet, still did not come this far.
Thanks
You too have a nice day

Hello @ceckoslab
Could you please provide some default JSON. Probably we need some default filter in global section as well as some default filter and metric into segment section. What do you think ?

Hello @korzol ,
The defaults will be:

Globa/default filter:

period: {
    type: 'moving',
    start: '130',
    end: 'now'
}

Default filter for 1st segment:

device_type:
 - condition: is
 - search_value: 1

Default metric for 1st segment:

technical_metrics - first_paint
data_flavor - histogram

Hello @ceckoslab
Thank you. However there is one problem with provided Global/default filter
Here is a piece of code we have currently in use:
{
                title: 'Desktop OS distribution',
                global: {
                    presentation: {
                        'render_type': 'distribution'
                    },
                    data_requirements: {
                        period: {
                            type: 'moving',
                            start: '30',
                            end: 'now',
                        },
                        filters: {
                            device_type: {
                                search_value: 2,
                                condition: 'is'
                            }
                        }
                    }
                },

As you can see period and filter can exists simultaneously. Probably it's just wrong copy/paste.
Kindly clarify
P.S. Meantime I use the following for default/global filter
filters: {
    device_type: {
        search_value: 2,
            condition: 'is'
        }
}

Hello @korzol
I looked at the PR: https://github.com/basicrum/backoffice/pull/115/files
I suggest 2 things:

We stop writing code for 1-2 days and we try to represent in a diagram how the filters, metrics and data flavors are connected. Please try this link for drawing the diagram: https://app.diagrams.net/#G1ndBLyW4H5qYqaVHx3KfZyks4IUGCJAgF

If the link doesn't work please create your own diagram here https://www.diagrams.net/ and share it later.
After we finish with the diagram we check thing number 2.

We don't use JSON directly in the Backend files but we use multi dimensional arrays.

This:
$json = "
            {
                title: 'Diagram Title',
                global: {
                    presentation: {
                        'render_type': 'distribution'
                    },
                    data_requirements: {
                        period: {
                            type: 'moving',
                            start: '30',
                            end: 'now',
                        }
                    }
                },
...

becomes:
$json = [
                "title" => "Diagram Title",
                "global" => [
                    "presentation" [
                        "render_type" =>  "distribution"
                    ],
                    "data_requirements" [
                        "period" => [
                            "type" => "moving"
                            "start" => "30"
                            "end" => "now"
                        ]
                    ]
                ]
...
]

Hello @ceckoslab
Till this moment I already implemented technical metrics to JSON schema. However this is done in simple way - concatenation. Here is a piece of code from src/BasicRum/Metrics/Collaborator.php
public function getDataFlavor($renderType): string
    {
        $dataFlavor = '';
        if ( $renderType == 'time_series' )
        {
            $dataFlavor = '"data_flavor": {
                                "type": "object",
                                "properties": {
                                    "percentile": {
                                        "enum": [50],
                                        "type": "integer"
                                    }
                                }
                            }';
        }

        elseif ( $renderType == 'plane' )
        {
            $dataFlavor = '"data_flavor": {
                                "type": "object",
                                "properties": {
                                    "histogram": {
                                        "type": "object",
                                        "properties": {
                                            "bucket": {
                                                "enum": [200],
                                                "type": integer
                                            }
                                        }
                                    }
                                }
                            }';
        }
        elseif ( $renderType == 'distribution' )
        {
            $dataFlavor = '"data_flavor": {
                                "type": "object",
                                "properties": {
                                    "count": {
                                        "type": "boolean"
                                    }
                                }
                            }';
        }

        return $dataFlavor;
    }

    public function getDataMetrics($renderType): string
    {
        $segmentMetricsPart = '
                            "technical_metrics": {
                                "type": "object",
                                "properties": {
            ';
        foreach ($this->technicalMetricsClassMap as $key => $class)
        {
            $entry = new $class;
            $segmentMetricsPart .= '
                    "'.$key.'": {
                        "type": "object",
                        "properties": {
                            '.$this->getDataFlavor($renderType).'
                        }
                    }';
        }
        $segmentMetricsPart .= "
                }
            }
            ";

        return $segmentMetricsPart;
    }

Right now want to implement similar thing into BusinessMetrics/Collaborator.php
Also I think it would be a good idea to deploy some php class which can convert php array into JSON schema, if there is any. So for example in getDataFlavor function I can simply give it an array and accept a piece of schema. But still not sure if there already exist any or I have to write one new.
Once I done with business metrics I will check

Hello @ceckoslab
Kindly review the latest commit to PR #115
The work is still in progress. And new changes will be added

Kindly check this attached diagram. Did you mean something like this ?

Hello @korzol
I looked at the code and my feeling is that we need to discuss things about the software architecture and which logic stays where. I suggest that we do a peer codding session at some point. If possible this Friday ... perhaps you may take take Monday as a day off.
Here are the things that I have specific comments:

I still see a lot of JSON strings concatenation but it will be better to have arrays where we use array merge in order to glue different parts of the JSON schema. When we want tore return the schema to the browser we can do json_encode() - reference: https://www.php.net/manual/en/function.json-encode.php

It will be better we have arrays but not JSON strings concatenation now because later it will be more harder to refactor and maintain JSON strings.

Did you think about unit tests? If not I can support you by showing you from where to start.

The diagram is getting in shape and it's cool but it looks more like a class diagram. Could you think about a diagram that explains the relation between diagram types and data flavors, because for example time series diagram doesn't support data flavor - histogram.

I urge you to stop implementing new code but to do point 1. and to work on point 3. Feel free to explore the internet about publications for dynamic schemas, you can watch again the Vega Lite video ... feel free to do more research.

Hello @ceckoslab
This is just prove of concept. For me it was easier to start with concatenation and finish it in same style. Once it is done and works, now I can rewrite it using arrays. Actually there is/are part(s) which already use arrays.

Sorry I did not understand what kind of diagram did you mean

Also I am not going to implement any new code. I was going to put existing one in order

Here is a diagram which we worked on.
We still have to find out how/if we can make data_flavor to be dependent on global.presentation.render_type. For example:
if render_type is distribution it can't be histogram. etc..
Also global.data_requirements.period's parameters - type, start, end - still have to be added onto diagram

Hello @ceckoslab
Here is latest updates to diagram. It is not done yet, so any thoughts are welcome
Kindly review

Hello @ceckoslab
Just pushed fresh code update.
Kindly review it and let me know.
Meantime going to work on tests

Hello @ceckoslab
Just pushed tests as well.
Kindly review and let me know your thoughts

As of diagram from this post.
I am not sure what to do about:

TechnicalMetrics: DownloadTime and LastBlockingResource
I think DownloadTime can be percentile and histogram
And not sure about LastBlockingResource. Would be great if we can discuss it

BusinessMetrics - I think they rely on data from RT plugin, right?
Here as well would be great if we can discuss it more in details

Kindly clarify

Hello @korzol

DownloadTime and LastBlockingResource can be a Histogram and Percentile.

The business metrics depend on data from the RT plugin but not directly. For example the Bounce Rate is not calculate because bounce rate metric is send on a beacon but because we check how many pages a client has visited. At one moment we will calculate how long a visitor stays on a page and this again can be done with the help of RT plugin metrics. In conclusion I would say that the RT plugin is used for providing visitor session information.

Hello @ceckoslab
Just pushed latest updates.
Here is a problem:
I am trying to figure out solution for this thing.  As you may noticed I use "definitions" schema feature for Json segments:
segments: {
        1: {
            presentation: {
                name: '',
                color: '#ff6023',
                type: 'bar'
            },
            data_requirements: {
                filters: {
                    device_type: {
                        search_value: 1,
                        condition: 'is'
                    }
                },
                technical_metrics: {
                    first_paint: {
                        data_flavor: {
                            histogram: {
                                bucket: '200'
                            }
                        }
                    }
                }
            }
        }
    }

In the example above I have only one segment - 1. However it is possible to add more segments 2, 3, 4 etc.. accordingly.
And in schema I use $ref feature like this:
"segments": {
        "type": "object",
        "properties": {
            "1": {
                "$ref": "#/definitions/segment"
            }
        }
}

Now, I am not sure how to make schema works for segments 2, 3, 4 etc..
I see 2 possible solutions:

hardcode predefined amount of segments. Lets say 4 segments should be enough
Currently "segments" section is object. If its possible to convert it to array then we can make schema more compact by using the following:
https://json-schema.org/learn/miscellaneous-examples.html
See "Arrays of things" section

Let me know, what you think

Here is latest changes to diagram

Hello @korzol
I am answering on:
I see 2 possible solutions:

1. hardcode predefined amount of segments. Lets say 4 segments should be enough
2. Currently "segments" section is object. If its possible to convert it to array then we can make schema more compact by using the following:
https://json-schema.org/learn/miscellaneous-examples.html
See "Arrays of things" section
Let me know, what you think

Could you explore option 2? I think that it will be good because the number of segments will vary.

Hello @ceckoslab
With latest update I've pushed implementation of second option - schema segments object to array.
Ran tests locally and it went fine. Travis as well reported tests went fine
Kindly review and let me know

I also tested if we can use array instead of object for segments on dashboards diagrams. And they worked well

Here is latest changes to diagram


Add a step-by-step user guide on adopting Falco

/area documentation
What would you like to be added:
Clear instructions on adopting Falco. How it integrates with other tools. What are the dependencies? What are the expectations?
Why is this needed:
To set clear expectations for user about Falco. What other tools you need to get most of Falco? Concrete step-by-step guide on:

Getting started - show the value of Falco (what questions it addresses/problems it solves?). Smooth installation and up and running is a prerequisite.
Provide good adoption path: What do we do once we have Falco?

### Tasks
- [ ] Add getting started landing page
- [ ] Add a quickstart for Falco
- [ ] Add scenario walkthrough
- [ ] Add concepts and architecture

/assign mikegcoleman

Over the past two days we met with @LucaGuerra and @nate-double-u to review our plan for the getting started section of the website and it looks like we have agreement, so I wanted to post an outline here.
Getting started is intended to be a new technical user's first contact with Falco. Think of it as “prework” for users to be ready to start planning PoC or Production deployments.
Our goals are to

Show Falco’s value as quickly as possible
Ensure awareness of next steps
Provide full and complete instructions for core hands-on exercises

After getting started users should know

What falco does
What are its core components
The value it brings / Key use cases
What to do next.

There four different page associated with the getting started journey, and they should be visited in this order (Ideally)
Landing page:

Provides a quick overview of Falco’s value proposition
Needs to describe the end-to-end workflow with Falco outputting to Sidekick
Does not need to go deep on anything (What not how)

Quick Start:

Designed to show the core Falco features
Walks through to install Falco on Linux, and show some simple output
Basically a revamp of the current "try Falco on Ubuntu" walkthrough on our current docs site

Scenario Walk Through:

More detailed walk through
Represents an end-to-end scenario that shows how Falco fits into a workflow.
Could be Kubernetes Audit Logs -> Falco -> Prometheus
Covers Plugins and Sidekick

Concepts and Architecture:

Show users the core Falco components
Educate on trade-offs and considerations for production deployments
Could include Glossary

After progressing through these four pages, practitioners should understand what falco does, a bit about how it does it, and be ready to start planning for a POC or production deployment.

I added the deliverables as trackable tasks. Thanks Mike!

This PR is related: https://github.com/falcosecurity/falco-website/pull/1072


Idea: Curated/Personalized Weekly Email

Send out an optional curated/personalized email on a weekly basis featuring popular blog posts.

Would the email be composed of popular blog posts on sites the user doesn't subscribe to? That could be a useful alternative to suggested feeds, although I think I'd prefer an in-app list of suggested feeds showing a selection of curated/personalized posts from each feed.

@mlapierre We're not sure just yet. It's something that we are still planning out. What would you prefer? We are rolling out an in-app suggested feeds component early the coming days. I'll ping you here when we release the feature.

Well, I know some people prefer email to RSS so it might be useful for them, but I would prefer something within the app.

Cool. Stay tuned!


Add an Task Bar Icon to the Application

AFTER #46

Maybe we should design a logo first. Maybe discuss in a meeting. @qmascarenhas


The KayakPro Genesis Port workout doesn't seem to show power readings lately

Not sure why this happens, but the Genesis Port (#73) should communicate Instantaneous Power, so I need to see why it is 0 lately. I see that the test data contains the power.

Upon further checking I was wrong. The recorded KayakPro workouts do have power data. I need to double check if it is displayed properly during workout, but I suspect yes. Maybe I was tricked by some seconds of delay. Also note that Strava and SUUNTO omits the power data on their mobile app / web app, while MapMyFitness (Under Armour) and Training Peaks keeps it and displays it. This can be a sport specific behavior for those former portals (Strava and SUUNTO).


Fold ADD with TRANSPOSECONV in tflite converter.

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template

This is a feature request and also relative to performance issue.

System information

TensorFlow version (you are using):
tf-nightly '2.1.0-dev20191126'
Are you willing to contribute it (Yes/No):
Yes

Describe the feature and the current behavior/state.
For generating .tflite file with TFLiteConverter, when model contains Conv2DTranspose layers, bias cannot fold into Operator TRANSPOSECONV. It will result with extra Op ADD following Op TRANSPOSECONV.
But with other CONV-like layers (Conv2D, DepthwiseConv2D), bias will be fold into CONV layer.

[ ] bias should fold into Op TRANSPOSECONV in tflite.

Reproduce current behaviors with example code and model graph:
def get_keras_model():
    input_0 = tf.keras.layers.Input(shape=[100,100,3])
    # Conv2D with bias
    conv_0 = tf.keras.layers.Conv2D(filters=3, kernel_size=[3, 3],
                                    use_bias=True, padding="same",
                                    kernel_initializer=tf.random_uniform_initializer,
                                    bias_initializer=tf.random_uniform_initializer)(input_0)
    # Conv2D without bias
    conv_1 = tf.keras.layers.Conv2D(filters=3, kernel_size=[3, 3],
                                    use_bias=False, padding="same",
                                    kernel_initializer=tf.random_uniform_initializer)(conv_0)
    # DepthwiseConv2D with bias
    depthwise_conv_0 = tf.keras.layers.DepthwiseConv2D(kernel_size=[3, 3],
                                                       use_bias=True, padding="same",
                                                       kernel_initializer=tf.random_uniform_initializer,
                                                       bias_initializer=tf.random_uniform_initializer)(conv_1)
    # DepthiseConv2D without bias
    depthwise_conv_1 = tf.keras.layers.DepthwiseConv2D(kernel_size=[3, 3],
                                                       use_bias=False, padding="same",
                                                       kernel_initializer=tf.random_uniform_initializer)(depthwise_conv_0)
    # TrasposeConv with bias
    transpose_conv_0 = tf.keras.layers.Conv2DTranspose(filters=3, kernel_size=[3,3],
                                                       use_bias=True, padding="same",
                                                       kernel_initializer=tf.random_uniform_initializer,
                                                       bias_initializer=tf.random_uniform_initializer)(depthwise_conv_1)
    # TrasposeConv without bias
    transpose_conv_1 = tf.keras.layers.Conv2DTranspose(filters=3, kernel_size=[3, 3],
                                                       use_bias=False, padding="same",
                                                       kernel_initializer=tf.random_uniform_initializer)(transpose_conv_0)

    keras_model = tf.keras.models.Model(inputs=[input_0], outputs=[transpose_conv_1])

    return keras_model

def gen_tflite():

    keras_model = get_keras_model()
    keras_model.compile(loss=tf.keras.losses.categorical_crossentropy,
                        optimizer='adam',
                        metrics=['accuracy'])
    converter = tf.lite.TFLiteConverter.from_keras_model(keras_model)
    tflite_quant_model = converter.convert()
    open('conv.tflite', 'wb').write(tflite_quant_model)

gen_tflite()

As the graph below shows, TFLiteConverter will fold bias with Conv2D and Depthwise2D, but not fold bias with TransposeConv:

Will this change the current api? How?
It will not affect any TF python api.
It will affect tflite as bias added.
Who will benefit with this feature?
Users of TensorFlow, as

consistent implementation/behavior between conv-like layers

Performance boost when tflite running on device, as TRANSPOSECONV + ADD folded into TRANSPOSECONV

Any Other info.
N/A

@liyunlu0618 Sorry to bother you, I saw your update with INT8 TRANSPOSE_CONV reference kernel.
Do you mind to have a look of this issue? I'm wondering why transpose_conv doesn't come with bias in tflite Op kernel. Meantime, conv and depthwise_conv does.
I'm trying to contribute regarding this issue, so that transpose_conv will has bias inside. And tflite graph transformer will behave the same for {conv, depthwise_conv, transpose_conv}, which fuse bias add with "conv" Op.
It will be very helpful if you can give some suggestions. Thanks.

@gadagashwini @haozha111 Hi guys, sorry to bother you. Can you give some suggestions?

Pull request https://github.com/tensorflow/tensorflow/pull/34903 trying to resolve this issue.

thanks!

Is this issue resolved??
It does not seem to work in tf versions: 2.4.0-dev20200916, 2.3.0 !!!
An extra shape node seems to be created ...


fiori-tools-proxy disables npm module shims in its' default config

Description
The issue was initially reported at https://answers.sap.com/questions/13461279/ui5-project-shims-not-working.html?childToView=13479438.
When a service is added using the odata-service template. the fiori-tools-proxy middleware is added to the ui5.yaml
server:
  customMiddleware:
  - name: fiori-tools-proxy
    afterMiddleware: compression
    configuration:
# ...
      ui5:
        path:
        - /resources # <-- this line makes the proxy intercept everything with the prefix /resources
        - /test-resources
        url: https://ui5.sap.com
        version:  # The UI5 version, for instance, 1.78.1. Empty means latest version

If a developer then wants to using the shimming functionality and adds a configuration like
specVersion: '2.1'
kind: extension
type: project-shim
metadata:
  name: thirdparty # this can be your project shim name
shims:
  configurations:
    lodash: # name as defined in package.json
      specVersion: '2.1'
      type: module # Use module type
      metadata:
        name: lodash
      resources:
        configuration:
          paths:
            /resources/thirdparty/lodash/: ''  # never getting here because the proxy intercepts it before

It doesn't work because the shimming capability requires resources not to be generically proxied, as it relies on a subdir of resources to make the npm module available at runtime.
Steps to Reproduce
Steps to reproduce the behavior:

Run the tests in the fiori-freestyle template
Enhance the one of the generated projects in the test-output folders with the shim configuration above
Try reaching any file in /resources/thirdparty/lodash
See 404 not found error

Expected results
Actual file is returned.
Actual results
404
Screenshots
-n/a-
Version/Components/Environment
Add any other context about the problem here
OS:

[x] Mac OS
[x] Windows
[x] Other

Root Cause Analysis
Problem
See description
Fix
Different options

Change the generated ui5.yaml to only proxy requests start with sap

ui5:
  path:
    - /resources/sap*

Leave the template as-is and report the issue to the fiori-tools-proxy team
Leave it as is and document that users that require shimming have to change the default settings

Why was it missed
This feature is not covered in a test case.
How can we avoid this
Create a test case covering it.
pinging @petermuessig @tobiasqueck as FYI

@Klaus-Keller what do you think? Is this something that should be fixed in the fiori-tools-proxy or should we update the templates as @vobu suggested as fix option 1?
@petermuessig is there anything not starting with sap at https://ui5.sap.com that we could/would/might need during the preview?

@petermuessig, for custom middleware we have a defined execution order and can also influence this execution order, for instance using
afterMiddleware: compression

Does such an option also exist for project shims? Couldn't find a way to influence execution order in https://sap.github.io/ui5-tooling/pages/extensibility/ProjectShims/

@RandomByte could you help us here?

Hi @RandomByte, it is a good idea, but unfortunately it doesn't work as expected. It is due to the fact that the http-proxy-middleware (the middleware that we are using in the fiori-tools-proxy) doesn't really support this feature (please see Issue248 for reference).
I can call next(), but then I get an exception Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client, because res.end() was called as soon as 404 was received.

Hi all, and sorry for joining the party that late. Basically, the UI5 resources of today are typically in the resources/sap* location but there are some special cases for the jQuery plugins, the ui5loader.js or ui5loader-autoconfig.js.
I like the suggestion of Merlin but this would mean the proxy middleware would require a response stub which is used to intercept the calls like setting the headers and res.end() - I did this in Java for some Servlets in the past and this may also work for NodeJS. Maybe just put a Proxy around and intercept the required functions like setHeader or end.
Cheers, Peter

Update
We decided to go for the idea where the fiori-tools-proxy calls the next available middleware (next()), if the proxy target returns 404. The fix is currently being implemented. Afterwards it will undergo quality assurance. If it all goes well we should have it released in 3-4 weeks.

Update
The fix didn't make it in the next release. Now it should be released mid of January.

Update
Fix was submitted to the main branch. The fix will now undergo quality assurance and if all goes well it will be released at the end of this month.

Fixed with<EMAIL_ADDRESS>

Hello,
we needed to revert the fix, because the fix was not stable. There were multiple reports of the error
Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client.
At this point in time we will not provide a fix for this. Please refer to https://answers.sap.com/questions/13461279/ui5-project-shims-not-working.html?childToView=13479438 for the workaround.
Best regards,
Zdravko

well, the workaround in https://answers.sap.com/questions/13461279/ui5-project-shims-not-working.html?childToView=13479438 is not really a workaround as a pattern like
ui5:
        path:
        - /resources/sap*

will not proxy ui5-loader* or any jquery* resources.

The sources responsible have been published by now and are at https://github.com/SAP/open-ux-tools/tree/main/packages/ui5-proxy-middleware.
@vobu any proposal how to fix it? Should we open the issue?
Btw, wouldn't the config below solve the issue?
ui5:
        path:
        - /resources/sap*
        - /resources/ui5-loader*
        - /resources/jquery*

Hello, we needed to revert the fix, because the fix was not stable.

Hi just enquiring if a fix is still being considered for this or if the issue has been moved to a new discussion?

Hi @proehlen, currently we are not considering to provide a fix for this issue.


after dao creation, clicking close button crashes app

i can

@please add proper steps to reproduce in the description


Make mqtt optional in the standard address space

To reduce footprint in cases where MQTT is not needed, it would make sense if mqtt components are not deployed when the mqtt endpoint is not configured in the address space config.

done


Test: simple fullscreen support

https://github.com/Microsoft/vscode/issues/26445
Complexity: 2

[ ] macOS
[ ] macOS

A new setting window.nativeFullscreen can now be set to false to disable the native macOS fullscreen support and use a custom one instead. The custom one should still do the same thing (bring the window into fullscreen), but will not create a separate space on the desktop (macOS Space).
Verify:

with this setting configured to false, everything works as it does with normal fullscreen
however you cannot bring the menu to fullscreen by using the window controls top right
you are not getting a separate macOS space created
entering and leaving fullscreen works without issues
fullscreen is also restart after a restart if you configure window.restoreFullscreen

switching back from fullscreen and the control buttons are gone

@a-eid are you using any custom settings, I cannot reproduce that. How did you enter fullscreen?

also you have to use keyboard shortcuts to switch back from fullscreen.
Or the menu, but yeah the window buttons do not toggle fullscreen because native fullscreen gets disabled with that setting.

I use command-option + f to toggle fullscreen.
I don't use any custom settings that might affect that, I will check again.

@a-eid that sounds like the keybinding for switching to native full screen no? Can you try Ctrl+Cmd+F instead?

Works nicely, I will actually use this setting in my daily selfhosting

@bpasero  yes I meant Ctrl + Cmd + F sorry..

@bpasero yes the new update fixed the issue I had, thank you.


Add drums

Add support for wii_drums (also black on every .py file, and that was maybe not needed).
Maybe you should only accept my change to code.py and the file I add.
"black" as a different style that what you did... and destroyed the harmony in boot.py.

Can you submit a new one that doesn't modify the boot.py? I am fine with black being ran on the other files though.

Done.

I did not submit a new one as you asked, I just modified my branch restoring the original boot.py.
So it should be fine since then.


Remove superfluous commas

Probably a typo and doesn't look nice in the home assistant interface with the comma

Sorry I needed to fix the CI scripts, Thank you for contribution!


Programatically launch the data viewer for specified dataframe

Hi folks,
One of the biggest RStudio time savers is the ability to open a dataframe (let's say "df") in the data viewer by simply typing View(df).
It would be incredibly handy to similarly have a python command which would do the same thing in VScode.
Thank you

Thanks for the suggestion. Our team will discuss it at our upcoming triage meeting.

Thank you @joyceerhl !

Is this issue better suited in vscode-python?

This is the right place, . We prioritize issues based on upvotes and community engagement. Unfortunately this hasn't received any upvotes.
I'll bring this up once again in our triage meeting.

@DonJayamanne , people using Jupyter files should use way less this feature than those which uses .py files in the Interactive Window (which is my case), that's why I asked if this is the right place. But good to know it is.
I wish there was some engagement but I believe only users coming from R using .py files would fully understand how useful this feature is.
Thank you for the feedback.

Thanks for understanding, by the way, we do have a large number of users using .py files in Interactive window that use the Jupyter extension.
Hopefully they'll upvote this.

To do this:

You'll need to create a custom IPython magic , such as %%showdataviewer xyz
When you handle the magic, you can send back a response with some special text
The Jupyter extension could monitor this special output and display the data frameviewer

Optionally you can try to use the widget framework and send custom messages via a custom channels (however that would require a bit more work).

Again, this is something you'll have to do in Python as well as a custom copy of the Jupyter extension, and its not likely the solution we'd implement. (If we were to implement this within the product, one of the best ways to implement this in a generic manner would be for the Jupyter extension to dynamically register a magic if that's at all possible).

Thanks a lot for the explanation, @DonJayamanne .


Error: failed to read index: 'C:\Users\username\Desktop\test_repo.git' no longer exists     at Error (native) errno: -1

Hey guys, I am new to nodegit and am struggling to understand what this error means.
I've expected addByPath to take the path of my file as a parameter and that it will then add it to the files to be committed. In other words "git add hey.txt".
But so far, I keep getting

Error: failed to read index: 'C:\Users\username\Desktop\test_repo.git' no longer exists     at Error (native) errno: -1

Here is the code you need to replicate the issue. You will need to create a repository on your desktop called test_repo and have an untracked file called hey.txt to replicate the issue.  Also, I'm on Windows.
Any help greatly appreciated!
var Git = require("nodegit");
var fs = require('fs');

Git.Index.open("C:\\Users\\firstname lastname\\Desktop\\test_repo\\.git\\index").then(function(index) {
    console.log(index.path());
    return index.addByPath("C:\\Users\\firstname lastname\\Desktop\\test_repo\\hey.txt");
  }).then(function(result) {
    console.log(result);
  }).catch(function(reasonForFailure) {
  console.log(reasonForFailure);
});

I guess after reading the issues guidelines that this is not the place to ask this. So I will close it.


chat_postMessage method is sending duplicate messages

Hello,
I'm using the slack API to send notification to users. I used the Python apscheduler package to send the message, say once a week. But the message I received every time is duplicated, like:

I'm not sure why I got two same messages when I use the chat_postMessage method once.
My code is like:
def notification():
    slack_bot_client = WebClient(token=****)
    slack_bot_client.chat_postMessage(channel=***,text='notice you')

if __name__ == '__main__':
    scheduler = BackgroundScheduler(timezone='Asia/Shanghai')
    scheduler.add_job(remind_await,'cron',day_of_week=0,hour=15,minute=25)
    scheduler.start()
    ### use socket mode to run the app to listen and respond to other user requests
    SocketModeHandler(bot_app,app_token).start()

Hello @RachelDong,
Can you share more of your application code? What you shared is insufficient to understand the full functioning of your app, in particular, the remind_await function/module, and any other code that connects remind_await to the notification() method.

Hi @RachelDong, the most likely cause would be that two running processes are running for some reason. Can you try the following and see how it goes?

Create a new app-level token with connections:write scope
Start a new app using the token
Terminate all the existing processes you are aware of
Disable the existing app-level tokens

After doing this, I am guessing that the issue should no longer arise.
As a preventive measure for this kind of situation, I would recommend a more robust approach such as having an external scheduler rather than as the code within each application instance. For example, if your app exposes an internal web endpoint to post a message in Slack and you have a stable scheduler somewhere else (ideally, managed service), managing the activities of the triggers and handler invocations would be much easier.
I hope this helps!

Hi @seratch ! Thank you for the advice! That helps me resolve the problem!


Fix #81

Refer to this link for build results (access rights to CI server needed):
http://guh.guru:8080/job/guh-webinterface-pull-request/56/


Fix List.compareWith case

From List.comparewith to List.compareWith.
Ref: https://msdn.microsoft.com/en-us/visualfsharpdocs/conceptual/list.comparewith['t]-function-[fsharp]

Thanks!


Refactor Navigation Include to use ESI

See https://github.com/adobe/helix-helpx/pull/3/files#r194838365 – note that this depends on adobe/project-helix#171

The refactoring must consist into:

remove the js code that "collect the nav": https://github.com/adobe/helix-helpx/blob/master/src/default.pre.js#L44-L63
refactor the htl template to use the ESI command to include /summary.html (line https://github.com/adobe/helix-helpx/blob/master/src/default.htl#L15)

Up-coming questions / issues:

/summary.html will be rendered with the default template... infinite loop!!
we'll then need a nav.htl template that renders only the required HTML (would also get rid of the commiters data and other useless stuff). summary.md would then need to be changed to define the template! Or template needs to be solved differently!

Pre-requisites:

ESI support in Petridish - https://github.com/adobe/project-helix/issues/171

actually comparing the original solution with the rendering directly in the default.htl and the separate  nav.htl solution, i am not sure I like the ESI include better.
it seems that it introduces quite a bit of complexity for little apparent reason. I would generally agree that if the nav component was reference from a number of places this could be useful, but in this specific case, looking at the amount of source code added, it seems to just complicate things.

I disagree.
Having a clean and clear split makes the code much readable and the structure of the content more transparent. In the previous solution, all the nav handling was done in the default.pre.js including the "fetch" of the SUMMARY.md file.
A third option could be to use a cq:include-like htl command to include the nav.htl inside the default.htl which I do not like neither because still means that some code in the default.pre.js must "fetch" of the SUMMARY.md file and enhance the resource with the parsed md.
While I did not like the use of another "technology" at the beginning (introduce a new esi tag in the htl template), I now like the fact that this makes transparent the fact that the nav is managed in an other markdown (fragment ?).
Imagine now the nav requires a little of rendering logic (a more complex htl code), same for the header, footer... this will all end up in the default.htl template which will become huge while the esi model allows to delegate.

i totally agree that separating concerns into the seperate resources is cleaner.
however, just looking at the lines of code, the additional concept of ESI that i need to understand, just makes things more complicated. the duplication of much of the .pre.js code doesn't help, and i appreciate that we could (and should) hide that.
your statement that the rendering code could be more complicated is absolutely correct, but it is not in this very example.
as for ESI, in the case of the navigation that contains the same markup on all pages, one could argue that ESI is not the right solution either but instead it would be a separate resource (json, js, html, md?) that is requested directly from the client and therefore cached in the browser.
of course i agree that there is a place for ESI (and client side includes and htl-includes), i just think that in this very example, it is a lot easier for someone who is new to the project to just have to understand how default.htl and the default.pre.js work and be done.
i think we have a tendency to over-engineer things because we are anticipating additional complexity and extension needs or we are striving for abstraction and with that make the initial learning curve steeper.

@kptdobe, as i was reading #8, i just noticed that we possibly may have different goals for this example project.
i think you are looking at this as a project to try out a variety of solutions for correctly anticipated real-life problems, and i am looking at it in a way where i think how can we keep it as simple as possible to show how little complexity you will need to master to get started.
i think both goals are fine, and we should probably do both. maybe separately.

Correct and I sensed that already ;)
But my main goal is more to be... convinced and convincing.
With AEM with started the same way (roughly "dump the whole code in the JSPs") and it was simple and easy at the beginning but in the end we never really convinced the developer eco-system (messy code organisation, maven, clientlibs, to much to learn...). Simplicity is key I agree but not to the price of code maintenance, testing, performance...
So I agree to have "master" branch for simplicity and other branch(es) for real-life approach
Regarding ESI, I can revert the changes from master and move them to the "other" branch for now. I do not say ESI is THE solution but the previous code is pretty ugly (during the rendering of the md, you fetch the nav md, execute the lambda function to render it, compute some metadata you do not need... not so simple and smart).

As agreed, I reverted the changes: the "nav" is generated via code in the default.pre.js file.
See commit: https://github.com/adobe/helix-helpx/commit/dbbef1ee2e2d2751a601dbace5a1de351309d16e

Still WIP: moving code to new "experiments" branch

(roughly "dump the whole code in the JSPs")

i things are slightly different here than they were, in two ways:

sling and all it's predecessors always came with per resource script resolution that always split up your code into dozens of small files almost by default.

the separation of concerns that you are referring to in my books in mainly related to mixing display logic with business logic. i would argue that this is supported in a clean way through the .pre.js

i completely agree with you on the question of the code being maintainable, and i don't know a more real-world project would yield a different code organization, but i would like to make sure that we keep simple projects simple and provide the necessary means, for a bigger more complex project to modularize things as needed.


how to use a link inside a modal window to close and open that link on background

i have some galleries and they have under each image a title with a link to an anchor ID.
When i click the link the modal window should close and scroll to the anchor link.
How to obtain that? - Thanks
Links to new pages works - but to scroll to an anchor of the parent-page (page that contains the popup link) does not work.

means - open the anchor link on background page an close modal window.


Unable to compile from source for latest build on github

Ubuntu 18.04
gcc 7.5.0
llvm-10.0.0
cuda 10.1
root@asd-TUF-GAMING-FX504GM-FX80GM:/opt/softwares/tvm/build# locate libcublas.so
/usr/lib/x86_64-linux-gnu/libcublas.so
/usr/lib/x86_64-linux-gnu/libcublas.so.10
/usr/lib/x86_64-linux-gnu/libcublas.so.<IP_ADDRESS>
/usr/lib/x86_64-linux-gnu/stubs/libcublas.so
/usr/local/cuda-10.1/doc/man/man7/libcublas.so.7

-- Found TVM_LLVM_VERSION=100
-- Build with LLVM
-- Set TVM_LLVM_VERSION=100
-- Build with contrib.random
-- Build with contrib.sort
-- Build with contrib.hybriddump
-- Git found: /usr/bin/git
-- Found TVM_GIT_COMMIT_HASH=3734d5f7f8475a2a7897f239b9942c913256fc96
-- Building with TVM Map...
-- Build with thread support...
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
CUDA_CUBLAS_LIBRARY
    linked by target "tvm_runtime" in directory /opt/softwares/tvm
    linked by target "tvm" in directory /opt/softwares/tvm

-- Configuring incomplete, errors occurred!
See also "/opt/softwares/tvm/build/CMakeFiles/CMakeOutput.log".
See also "/opt/softwares/tvm/build/CMakeFiles/CMakeError.log".

Looks like cmake failed to find your CUBLAS. You may provide the CUBLAS manually in config.cmake:USE_CUBLAS.
For future issues like this, please post them to the discuss forum: https://discuss.tvm.apache.org/t


Question - "util" lib?

Where is the "util" lib? Don't see it in package.json nor anywhere in "generated/" parent.
EDIT: Sorry, forgot to mention the file, it's https://github.com/qteatime/crochet/blob/main/source/generated/crochet-grammar.ts#L5
ps. Big fan of your art and all your projects!

Ah, totally forgot about that. Thanks.


"TypeError: Cannot read property 'forEach' of undefined" in IE 11.

Bug Report
Ionic Info
Ionic:
ionic (Ionic CLI)             : 4.2.1
Ionic Framework               : @ionic/angular 4.0.0-beta.16
@angular-devkit/build-angular : 0.10.6
@angular-devkit/schematics    : 7.0.6
@angular/cli                  : 7.0.6
@ionic/angular-toolkit        : 1.2.0
System:
NodeJS : v8.11.3 (C:\Program Files\nodejs\node.exe)
npm    : 5.6.0
OS     : Windows 10
Describe the Bug
Ionic element such as ion-input use with ngModel will cause "TypeError: Cannot read property 'forEach' of undefined" in IE 11. (Polyfill for IE 11 has been enabled)
Steps to Reproduce
Steps to reproduce the behavior:

add a ngModel in ion-input <ion-input type="text" [(ngModel)]="message">
ng serve and view in IE 11
See error "TypeError: Cannot read property 'forEach' of undefined"

Related Code
setIonicClasses
Additional Context
To fix the problem, Transform DOMTokenList to array in ion-input, then the problem can be fixed.

Add the polyfills to your polyfills.ts - this is related to Angular, not Ionic

Fixed by https://github.com/ionic-team/ionic/commit/e76544d0f4e4bb31664d83d335156418b1fcea98


file update - 1. load (*_new) 2. delete 3.rename

don't delete, then try to load new version :-)

Wip

done: used "new-" as a prefix instead of a postfix, all rest as requested.


COOPR-703 Fix upgrade script to work on MySQL

Upgrade script was previously tested against MS SQL, but we need MySQL.

:+1: LGTM

Err.. could you do this against release/0.9.9 also?

Oops.. will do.


add stable diffusion example

What does this PR do?
This PR adds an example of finetuning a diffusion model for the text to image task.

@JingyaHuang can you please review?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Hi @prathikr, it's awesome to add stable diffusion example! Thanks!
I left some nits, and when testing the script I got the following error:
Traceback (most recent call last):
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/onnxruntime/training/ortmodule/_utils.py", line 316, in get_exception_as_string
    raise exception
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/onnxruntime/training/ortmodule/_graph_execution_manager.py", line 425, in _get_exported_model
    torch.onnx.export(
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 506, in export
    _export(
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 1548, in _export
    graph, params_dict, torch_out = _model_to_graph(
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 1117, in _model_to_graph
    graph = _optimize_graph(
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 665, in _optimize_graph
    graph = _C._jit_pass_onnx(graph, operator_export_type)
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 1901, in _run_symbolic_function
    raise errors.UnsupportedOperatorError(
torch.onnx.errors.UnsupportedOperatorError: Exporting the operator 'aten::scaled_dot_product_attention' to ONNX opset version 15 is not supported. Please feel free to request support or submit a pull request on PyTorch GitHub: https://github.com/pytorch/pytorch/issues.

It seems to be an opset issue while exporting the ONNX graph. Can you confirm the version of onnx to use and probably add it to the requirements? Thanks.

@JingyaHuang this example currently required PyTorch

Hi @prathikr, it's awesome to add stable diffusion example! Thanks!
I left some nits, and when testing the script I got the following error:
Traceback (most recent call last):
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/onnxruntime/training/ortmodule/_utils.py", line 316, in get_exception_as_string
    raise exception
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/onnxruntime/training/ortmodule/_graph_execution_manager.py", line 425, in _get_exported_model
    torch.onnx.export(
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 506, in export
    _export(
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 1548, in _export
    graph, params_dict, torch_out = _model_to_graph(
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 1117, in _model_to_graph
    graph = _optimize_graph(
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 665, in _optimize_graph
    graph = _C._jit_pass_onnx(graph, operator_export_type)
  File "/home/onnxruntimedev/miniconda3/lib/python3.9/site-packages/torch/onnx/utils.py", line 1901, in _run_symbolic_function
    raise errors.UnsupportedOperatorError(
torch.onnx.errors.UnsupportedOperatorError: Exporting the operator 'aten::scaled_dot_product_attention' to ONNX opset version 15 is not supported. Please feel free to request support or submit a pull request on PyTorch GitHub: https://github.com/pytorch/pytorch/issues.

It seems to be an opset issue while exporting the ONNX graph. Can you confirm the version of onnx to use and probably add it to the requirements? Thanks.

@JingyaHuang please retry with PyTorch and ONNX Runtime nightly packages. I added a note in the readme about requiring nightly packages.

Hi @prathikr, thanks for the updates.
Tried with ort nightly, the script works fine (though massive log info looks a little scary, but guess it might be intended as it's still experimental). Left two nits before merging it.
Also,I think of adding it to our ORT training doc, we can add it in a separate PR (are you interested in it?)

@JingyaHuang yes, let's follow up offline and I can work on a documentation PR.


Show the state of containers when making a selection

This affects the rancher exec command.
Showing the current state of containers in the selection dialog improves the workflow of the rancher exec command.
Example
rancher exec site -ti /bin/bash
Containers:
[1] service-site-1 (<IP_ADDRESS>, stopped)
[2] service-site-1 (<IP_ADDRESS>, running)
Select:

Very stale. Closing. Thanks! Please open a new PR if there is still a relevant change.


Cannot GET data/demo.nc file error

No detail provided, so all I can say is that check your web server configuration if you are running this application locally

Sorry sir for the incomplete info.
I am running the application locally and creating an http server using lite-server and it is showing this error in the terminal

and this in the browser

What is the root directory of your lite-server? It should be the path to the cloned repo folder(like the following screenshot).

It seems that you are using the subfolder 'Cesium-3D-Wind' as the root directory

Yes sir you are absolutely correct. I have now used live-server and it is working just fine.
Thank you for your guidance , i ll close the issue now.


add ran ratio vs skipped. adjust math.

adds new section to report summary with ran ratio

also should close #382

@tsiq-karold could you please review the fix?


Error committing to github

I'm working on a (non-beta) MakeCode Arcade extension in one browser window (Edge for Mac). I'm using the extension in a project in a second browser window. Sometimes when I commit my changes in the first browser window, I get an error that something bad has happened and I should try again. This erases my changes (uncool). I can't just recommit, I have to refresh the browser, reload MakeCode, edit the code again before I can commit successfully.
This happens after committing successfully in the first browser window, then refreshing the second browser window to force it to reload the changes in the extension, and then going back to the first browser window to make additional changes. If I don't refresh the first browser, I get the error on the next commit.
 Please try again.

Multi tab is not really supported on the current live site, but peli did fix this when he added the multieditor page - so this is fixed in /beta


Added DoorCode backend

Created schema and routing for DoorCode API and wrote test cases for the routing

I'd say we wait until it is merged in since we are not in a rush, but there are merits in making the limit variable right now. I would like to hear the other's suggestions before moving forward.

I think we can implement the variable limit later because I don't believe it is a relevant issue right now. Also i think it will be easier to implement after we get some frontend components set up.


HTTPS Mirror Webserver Issue with Spaces/Character Encoding in URL path

Hello!
I've found that the https mirror is rendering a 404 for files that include spaces in their name.
Example:
The following ways of linking to a file work:
dat://39389303/files/file name with spaces.mp3
dat://domain.name/file name with spaces.mp3
However this does not:
https://domain.name/file name with spaces.mp3
It returns a 404 file not found.
I'm not entirely sure where to troubleshoot this. My first inclination is to dig into server.js as opposed to server environment so I thought I'd create an issue. I'll continue to investigate locally but if it's really issue, here is a ticket for it :)

Thanks for filing. Should just need to handle percent-encoded paths at https://github.com/beakerbrowser/homebase/blob/master/lib/vhosts/dat.js#L87

Hi @pfrazee, thank you for taking the time to reply and to point me the right direction.
I applied a local fix on line 103 of dat.js that solved the issue.
            path = decodeURI(path)
I don't know if this is the best spot, or the best approach, for handling percent-encoded paths but I'll see how other path checks/manipulations are done here and maybe look into a better solution. (or have it better solved by more capable hands 😺 )

Seems fine with me if you want to make a PR

Fix deployed as 2.0.8, thanks @rreinhardt


Support packaging nativefier applications into Squirrel-based installers

Squirrel is "an installation and update framework for Windows desktop apps ".
This PR adds electron-squirrel-startup, allowing to package nativefier applications
into squirrel-based setup installers. Squirrel require this entrypoint to perform
desktop and startup menu creations, without showing the UI on setup launches.

References: https://github.com/mongodb-js/electron-squirrel-startup

Resolves electron-winstaller and electron-installer-windows support of desktop / startup menu shortcuts for nativefier packaged applications.

The electron-squirrel-startup entrypoint has no effect on both Linux and Darwin, only on Windows

Supporting it directly inside nativefier avoids having to "hack" around the existing main.js
and including dependencies from electron-squirrel-startup in an intermediate package
to be included in a third layer for the final installer executable

The following script based on both nativefier and electron-winstaller templates
represents a portable proof of concept for this merge request :

var nativefier = require('nativefier').default;
var electronInstaller = require('electron-winstaller');

var options = {
  name: 'Web WhatsApp',
  targetUrl: 'http://web.whatsapp.com',
  platform: 'windows',
  arch: 'x64',
  version: '0.36.4',
  out: '.',
  overwrite: false,
  asar: false,
  counter: false,
  bounce: false,
  width: 1280,
  height: 800,
  showMenuBar: false,
  fastQuit: false,
  userAgent: 'Mozilla ...',
  ignoreCertificate: false,
  ignoreGpuBlacklist: false,
  enableEs3Apis: false,
  insecure: false,
  honest: false,
  zoom: 1.0,
  singleInstance: false,
  fileDownloadOptions: {
    saveAs: true
  },
  processEnvs: {
    GOOGLE_API_KEY: '<your-google-api-key>'
  }
};

nativefier(options, function(error, appPath) {
  if (error) {
    console.error(error);
    return;
  }
  console.log('App has been nativefied to', appPath);

  resultPromise = electronInstaller.createWindowsInstaller({
    appDirectory: 'Web WhatsApp-win32-x64',
    outputDirectory: './',
    authors: 'Web WhatsApp',
    exe: 'Web WhatsApp.exe'
  });

  resultPromise.then(() => console.log('It worked!'), e => console.log(`No dice: ${e.message}`));
});

@AdrianDC okay, got it, thanks for the explanation! Merging and adding docs.

@AdrianDC okay, got it, thanks for the explanation! Merging and adding docs.

That will do for now: https://github.com/jiahaog/nativefier/blob/master/docs/api.md#packaging-squirrel-based-installers . Thanks again @AdrianDC, and :pray: for the delay.

missing ../screen is still an issue for me @AdrianDC

How to use this?


Breadcrumbs: questionable use of hover

Hovering in the breadcrumbs tree usualy duplicates the name of field and adds in the brackets something like (property). I argue this is not useful and we shuold be consistent with the suggest widget and not show a hover here. People are already familiar with our suggest widget icons, and the hover brings no other useful value
Hovering over the the actual breadcrumbs I get the path hover (feels like duplication since we show the actual path) and over the names of classes duplicates the class name. I find this only useful when there is limited horizontal space.

Yeah, that's a leftover from the shrink experiment... I'll remove for now


Error on Unity3d while building for iOS

Hello, I use Tensorflow.NET to process some data in a unity3d mobile application.
It runs in Unity3d editor, but when I want to build it for iOS it stops with error.
Macbook pro 16 2019
Unity3D 2021.3.18f1
 Exception: Unity.IL2CPP.Building.BuilderFailedException: Build failed with 0 successful nodes and 1 failed ones Annotation: IL2CPP_CodeGen /Users/myuser/Downloads/CustomProject/Library/Il2cppBuildCache/iOS/buildstate/artifacts/il2cpp_conv_x4og.traceevents Cmdline: "/Applications/Unity/Hub/Editor/2021.3.18f1/Unity.app/Contents/il2cpp/build/deploy/il2cpp" --convert-to-cpp --directory="/Users/myuser/Downloads/CustomProject/Temp/StagingArea/Data/Managed" --data-folder="/Users/myuser/Downloads/CustomProject/Library/Il2cppBuildCache/iOS/il2cppOutput/Data" --generatedcppdir="/Users/myuser/Downloads/CustomProject/Library/Il2cppBuildCache/iOS/il2cppOutput" --symbols-folder="/Users/myuser/Downloads/CustomProject/Library/Il2cppBuildCache/iOS/il2cppOutput/Symbols" --additional-cpp="/Users/myuser/Downloads/CustomProject/Library/Il2cppBuildCache/iOS/additionalCppFiles/UnityClassRegistration.cpp" --additional-cpp="/Users/myuser/Downloads/CustomProject/Library/Il2cppBuildCache/iOS/additionalCppFiles/UnityICallRegistration.cpp" --emit-null-checks --enable-array-bounds-check --code-generation-option=EnableInlining --stats-output-dir="/Users/myuser/Downloads/CustomProject/Library/Il2cppBuildCache/iOS/il2cppStats" --dotnetprofile=unityaot-macos --cachedirectory="/Users/myuser/Downloads/CustomProject/Library/Il2cppBuildCache/iOS" --profiler-report --profiler-output-file="/Users/myuser/Downloads/CustomProject/Library/Il2cppBuildCache/iOS/buildstate/artifacts/il2cpp_conv_x4og.traceevents" ExitCode: 255 Stdout:  Error: IL2CPP error for method 'System.ValueTuple2<Tensorflow.Tensor,Tensorflow.Tensor> Tensorflow.image_ops_impl::non_max_suppression_padded_v2(Tensorflow.Tensor,Tensorflow.Tensor,Tensorflow.Tensor,System.Single,System.Single,System.Boolean,System.Boolean,System.Int32)' in assembly '/Users/myuser/Downloads/CustomProject/Temp/StagingArea/Data/Managed/TensorFlow.NET.dll'
System.ArgumentException: System.Object is not an array type (Parameter 'typeReference')
at Unity.IL2CPP.ArrayUtilities.ArrayElementTypeOf(ResolvedTypeInfo typeReference) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/ArrayUtilities.cs:line 18
at Unity.IL2CPP.MethodBodyWriter.ProcessInstruction(Node node, InstructionBlock block, ResolvedInstruction ins) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/MethodBodyWriter.cs:line 1770
at Unity.IL2CPP.MethodBodyWriter.GenerateCodeRecursive(Node node, ReadOnlyDictionary2 instructionBlocks) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/MethodBodyWriter.cs:line 424    at Unity.IL2CPP.MethodBodyWriter.Generate() in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/MethodBodyWriter.cs:line 245    at Unity.IL2CPP.CodeWriters.CodeWriterExtensions.WriteMethodWithMetadataInitialization(IGeneratedMethodCodeWriter writer, String methodSignature, Action2 writeMethodBody, String uniqueIdentifier, MethodReference methodRef, Boolean writingMethodBody) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/CodeWriters/CodeWriterExtensions.cs:line 153
at Unity.IL2CPP.MethodWriter.WriteMethodDefinition(AssemblyWriteContext context, IGeneratedMethodCodeWriter writer, MethodReference method) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/MethodWriter.cs:line 63
at Unity.IL2CPP.SourceWriter.WriteTypesMethods(SourceWritingContext context, IGeneratedMethodCodeWriter writer, TypeWritingInformation& writingInformation, NPath filePath, Boolean writeMarshalingDefinitions) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/SourceWriter.cs:line 89
at Unity.IL2CPP.SourceWriters.TypeMethodsSourceWriter.WriteItem(SourceWritingContext context, IGeneratedMethodCodeWriter writer, TypeWritingInformation item, NPath filePath) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/SourceWriters/TypeMethodsSourceWriter.cs:line 64
at Unity.IL2CPP.SourceWriters.SourceWriterBase2.WriteItem(StreamWorkItemData2 data) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/SourceWriters/SourceWriterBase.cs:line 92
at Unity.IL2CPP.Contexts.Scheduling.Streams.FileLevelParallelStreamManager3.WorkerWriteItemsToFile(WorkItemData2 data) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/Contexts.Scheduling.Streams/FileLevelParallelStreamManager.cs:line 36
at Unity.IL2CPP.Contexts.Scheduling.PhaseWorkScheduler`1.WorkerLoop(Object data) in /Users/bokken/build/output/unity/il2cpp/Unity.IL2CPP/Contexts.Scheduling/PhaseWorkScheduler.cs:line 339
at il2cpp.Program.DoRun(String[] args, RuntimePlatform platform, Il2CppCommandLineArguments il2CppCommandLineArguments, BuildingOptions buildingOptions, Boolean throwExceptions) in /Users/bokken/build/output/unity/il2cpp/il2cpp/Program.cs:line 339
UnityEditorInternal.Runner.RunProgram (UnityEditor.Utils.Program p, System.String exe, System.String args, System.String workingDirectory, UnityEditor.Scripting.Compilers.CompilerOutputParserBase parser) (at /Users/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline/BuildUtils.cs:129)
UnityEditorInternal.Runner.RunNetCoreProgram (System.String exe, System.String args, System.String workingDirectory, UnityEditor.Scripting.Compilers.CompilerOutputParserBase parser, System.Action1[T] setupStartInfo) (at /Users/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline/BuildUtils.cs:91) UnityEditorInternal.IL2CPPBuilder.RunIl2CppWithArguments (System.Collections.Generic.List1[T] arguments, System.Action1[T] setupStartInfo) (at /Users/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline/Il2Cpp/IL2CPPUtils.cs:817) UnityEditorInternal.IL2CPPBuilder.ConvertPlayerDlltoCpp (UnityEditor.Il2Cpp.Il2CppBuildPipelineData data) (at /Users/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline/Il2Cpp/IL2CPPUtils.cs:801) UnityEditorInternal.IL2CPPBuilder.Run () (at /Users/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline/Il2Cpp/IL2CPPUtils.cs:639) UnityEditorInternal.IL2CPPUtils.RunIl2Cpp (System.String tempFolder, System.String stagingAreaData, UnityEditorInternal.IIl2CppPlatformProvider platformProvider, System.Action1[T] modifyOutputBeforeCompile, UnityEditor.RuntimeClassRegistry runtimeClassRegistry) (at /Users/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline/Il2Cpp/IL2CPPUtils.cs:279)
UnityEditor.iOS.PostProcessiPhonePlayer.CrossCompileManagedDlls (UnityEditor.iOS.PostProcessiPhonePlayer+BuildSettings bs, UnityEditor.iOS.PostProcessiPhonePlayer+ProjectPaths paths, UnityEditor.AssemblyReferenceChecker checker, UnityEditor.RuntimeClassRegistry usedClassRegistry, UnityEditor.Build.Reporting.BuildReport buildReport) (at /Users/bokken/build/output/unity/unity/PlatformDependent/iPhonePlayer/Extensions/Common/BuildPostProcessor.cs:932)
UnityEditor.iOS.PostProcessiPhonePlayer.PostProcess (UnityEditor.iOS.PostProcessiPhonePlayer+BuildSettings bs, UnityEditor.iOS.PostProcessiPhonePlayer+ProjectPaths paths, UnityEditor.RuntimeClassRegistry usedClassRegistry, UnityEditor.Build.Reporting.BuildReport buildReport) (at /Users/bokken/build/output/unity/unity/PlatformDependent/iPhonePlayer/Extensions/Common/BuildPostProcessor.cs:755)
UnityEditor.iOS.PostProcessiPhonePlayer.PostProcess (UnityEditor.iOS.PostProcessorSettings postProcessorSettings, UnityEditor.Modules.BuildPostProcessArgs args) (at /Users/bokken/build/output/unity/unity/PlatformDependent/iPhonePlayer/Extensions/Common/BuildPostProcessor.cs:695)
UnityEditor.iOS.iOSBuildPostprocessor.PostProcess (UnityEditor.Modules.BuildPostProcessArgs args) (at /Users/bokken/build/output/unity/unity/PlatformDependent/iPhonePlayer/Extensions/Common/ExtensionModule.cs:45)
Rethrow as BuildFailedException: Exception of type 'UnityEditor.Build.BuildFailedException' was thrown.
UnityEditor.iOS.iOSBuildPostprocessor.PostProcess (UnityEditor.Modules.BuildPostProcessArgs args) (at /Users/bokken/build/output/unity/unity/PlatformDependent/iPhonePlayer/Extensions/Common/ExtensionModule.cs:49)
UnityEditor.Modules.DefaultBuildPostprocessor.PostProcess (UnityEditor.Modules.BuildPostProcessArgs args, UnityEditor.BuildProperties& outProperties) (at /Users/bokken/build/output/unity/unity/Editor/Mono/Modules/DefaultBuildPostprocessor.cs:28)
UnityEditor.PostprocessBuildPlayer.Postprocess (UnityEditor.BuildTargetGroup targetGroup, UnityEditor.BuildTarget target, System.Int32 subtarget, System.String installPath, System.String companyName, System.String productName, System.Int32 width, System.Int32 height, UnityEditor.BuildOptions options, UnityEditor.RuntimeClassRegistry usedClassRegistry, UnityEditor.Build.Reporting.BuildReport report) (at /Users/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline/PostprocessBuildPlayer.cs:370)
UnityEditor.EditorApplication:Internal_CallGlobalEventHandler() (at /Users/bokken/build/output/unity/unity/Editor/Mono/EditorApplication.cs:462)
`

up

Could you please provide the C# language version and dotnet version (and any other possibly useful information)? I have little knowledge about Unity but I'd like to try to help you to find the reason of this error.


Get-DbaCpuUsage

Is this a feature OR bug:
Enhancement
https://trello.com/c/gJhptCU0
Simply returns a present value of records from the ring buffer (T-SQL) that holds the CPU usage for SQL Server. Don't have blog post reference for code I've used but can return SQL Server CPU usage, System Idle, and then calculated "Other CPU usage". Which the last column would be CPU usage for anything else running on the system.
Additional comments

I have a runspace script that handles it but if you want to convert over for dbatools, I do not mind at all.  I have not had the time due to a change/enhancement of duties.

@wsmelton - you mentioned having code for this. Can you paste me the T-SQL and I can create this command?

Get-DbaCpuUsage now exists 👍


add support for PowerScale topology

Is this a new chart?
No
What this PR does / why we need it:
enable PowerScale topology
Which issue(s) is this PR associated with

#334: https://github.com/dell/csm/issues/334

Special notes for your reviewer:
This code change has been verified along with changes in karavi-topology PR#116
Checklist:
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[ ] Variables are documented in the chart README.md
[ ] Title of the PR starts with the chart name (e.g. [charts_dir/mychartname]) if applicable

Can these changes point to the karavi-observability-release branch instead of main. We've recently bumped the versions in Chart.yaml as well. Thank you!

Can these changes point to the karavi-observability-release branch instead of main. We've recently bumped the versions in Chart.yaml as well. Thank you!

Thanks, @shaynafinocchiaro ! Done.

I signed the commit and had to push it again. @sharmilarama @shaynafinocchiaro please help review to approve it again, thank you!


Remove prospector in favor of running pylint directly

Codecov Report

Merging #87 into dev will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev      #87      +/-   ##
==========================================
- Coverage   99.05%   99.04%   -0.02%
==========================================
  Files          11       11
  Lines        1063     1046      -17
==========================================
- Hits         1053     1036      -17
  Misses         10       10

Impacted Files
Coverage Δ

tbmodels/_tb_model.py
98.76% <100.00%> (-0.02%)
:arrow_down:

tbmodels/io.py
100.00% <0.00%> (ø)

tbmodels/_cli.py
100.00% <0.00%> (ø)

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 ead15d2...dd3a3ae. Read the comment docs.


Fix CI/CD Pipeline

We need to fix the CI/CD pipeline so that the linting test can pass.

@micimize from what I can tell, it seems we have also not had a release since the end of August, can you confirm this?

It seems the linting test is failing because of flutter analyze detecting use of deprecated document field of QueryOptions. This has been replaced by DocumentNode by the new AST Implementation. For this test to pass, we need to replace all instances of document (49 of them) with DocumentNode. In some cases like examples and tests, it's straight forward but since the functionality is deprecated and not removed, in instances it's not even possible to remove that.

@micimize from what I can tell, it seems we have also not had a release (beta) since the end of August, can you confirm this?

I have discovered the reason for this is due to failing tests. I hope PR #462 fixes the issue.

@HofmannZ Sync beta with master is still failing due to a dependency resolution issue while release to beta is failing with the following message:


GH-3243: Ensure expected order of writes to Console by flushing after writes

Before this PR:

After this PR:

Task("build")
    .Does(() =>
{
    Information("Hello from {0}! This is written to stdout (1).", "Cake");
    Error("Hello from {0}! This is written to stderr (1).", "Cake");

    Information("Hello from {0}! This is written to stdout (2).", "Cake");
    Error("Hello from {0}! This is written to stderr (2).", "Cake");

    Information("Hello from {0}! This is written to stdout (3).", "Cake");
    Error("Hello from {0}! This is written to stderr (3).", "Cake");

    Information("Hello from {0}! This is written to stdout (4).", "Cake");
    Error("Hello from {0}! This is written to stderr (4).", "Cake");
});

RunTarget("build");

Closes #3243

Hopefully this will also solve the problem with GitHub Actions logging sometimes entering a "blackout" period.


Implement RFC0015: Distribute Buildpacks via Docker Hub

RFC
Summary
In addition to the current Google Container Registry (GCR) distribution channel, buildpacks should be distributed via Docker Hub.

[x] Setup paketobuildpacks DockerHub account
[x] Setup paketocommunity DockerHub account
[x] Publish buildpacks to Dockerhub

[x] Go
[x] Ruby
[x] .Net Core
[x] NodeJS
[x] PHP
[x] Python
[x] Java
[x] Rust
[x] Web Servers

[x] Update package.toml references in language-family buildpacks

[x] Go: https://github.com/paketo-buildpacks/go/pull/600
[x] Ruby: https://github.com/paketo-buildpacks/ruby/pull/773
[x] .Net Core: https://github.com/paketo-buildpacks/dotnet-core/pull/709
[x] NodeJS: https://github.com/paketo-buildpacks/nodejs/pull/581
[x] PHP: https://github.com/paketo-buildpacks/php/pull/540
[x] Python: https://github.com/paketo-buildpacks/python/pull/463
[x] Java
[x] Rust
[x] Web Servers: https://github.com/paketo-buildpacks/web-servers/pull/63

[ ] Update README references on all buildpacks
[ ] Update documentation references on website

Hi, I think Docker OpenSource Program should be applied for paketobuildpacks and paketocommunity in order to get unlimited pulls.

I thought something like this should be displayed.

It took a little while but the Java and Rust Buildpacks should be on Docker Hub this week when we publish on Friday.
That should close out this issue.

This is mostly a documentation project at this point. I believe we still have a number of locations across the project where we reference our image distributions by their GCR location. We should update these to point to Docker Hub.

Oh, I forgot to update. On the badges I asked Chris at the CFF about this and he said they are still working on getting this straightened out with Docker. Images published under paketobuildpacks/ should not have rate limits, cause we are a qualified OSS project, but we don't have the purple badge.


Infer: do format guessing

pwalsh wrote:

I've noticed in OpenSpending over the last months how useful it would be to guessing full type + format, and not just type, so this is good to capture in an issue (cc @akariv FYI)

MERGED
#105


GPT4 cutoff date is September 2021 - how did this impact evals?

Any new API info would not be in GPT4 training.
How much impact do you think this has with respect to relative performance between GPT4 and Gorilla?
Did you do any eval on APIs that existed prior to 09/21 versus those after?
I reviewed the paper but could not find any discussion on this.  https://arxiv.org/abs/2305.15334
To be clear, I am not saying this invalidates the ideas, which I think were a fantastic contribution to OS LLMs, but rather that it would be good to understand the precise reason for the superior performance.

@qrdlgit It's basically based on RAG method so don't need to be contained within training set.

Yes, but the paper claims superior performance to GPT4.
I have consistently found that GPT4 hallucinates less on data that it has been trained on.  When you add vector retrieval it even does a better job.
For APIs that were added after the cutoff date, it wouldn't be surprising that GPT4 hallucinations would increase.
This is not a complaint.  I just want to understand.

They used APIs that has been quite stable a while and I believe not much has been changed after the cut-off of GPT4 pre-training. so the benchmark seems to me fair enough.

Don't take this personally, but I'm not sure you are familiar with these details.
eg, from https://github.com/ShishirPatil/gorilla/blob/main/data/apibench/huggingface_train.json
I found microsoft/xclip-base-patch16-zero-shot which had an initial commit  in the last 9 months.

@qrdlgit Thank you for your comments! One thing we need to clarify:
We don't require GPT-4 to output exactly same API here, as long as the API from GPT-4's output has the same functionality, we count as correct. See the script from here: https://github.com/ShishirPatil/gorilla/blob/main/eval/eval-scripts/ast_eval_hf.py. This has been very consistent from the very beginning.

That answers the question, but not in the way you probably intended -  Ie, evals were not done with API dates in mind.
Again, the gorilla is still a great idea and paper.  A lot of good takeaways for sure.
However, in the future you probably want to be more careful about data leakage / data contamination issues.   This is a problem I'm seeing in a lot of papers coming out recently.

Thank you for your question and insightful discussion @qrdlgit and @fritzprix! When it comes to the issue of data contamination, we are completely aligned. We have been cautious to ensure that Gorilla doesn't encounter any of the test set data during its training phase. However, we are unable to provide any comment on the training/test data for models that are closed-source.
Your point about splitting APIs before and after 09/2021 is well taken. As @fritzprix pointed out, we would ideally like to believe that an oracle retriever can address the issue concerning the cut-off date as effectively as possible.
To validate this hypothesis, you can conduct a straightforward experiment - given that our training and evaluation datasets are open-sourced, it should be relatively simple to filter out APIs published post 09/2021 and validate this experiment. If you do end up doing it, please feel free to share the results. We would certainly appreciate such a contribution!


Docker swarm support

Does it support Docker Swarm API?

Yes, just take this in account: https://docs.docker.com/swarm/api/swarm-api/

Updated link: https://docs.docker.com/swarm/swarm-api/


Basic: make the member specifiers match the declaration

Visual Studio does not like the member being declared as const and then used
as constexpr.
include\swift\Basic\ImmutablePointerSet.h(352): error C2370: 'swift::ImmutablePointerSetFactory<T>::AllocAlignment': redefinition; different storage class
include\swift\Basic\ImmutablePointerSet.h(185): note: see declaration of 'swift::ImmutablePointerSetFactory<T>::AllocAlignment'
include\swift\Basic\ImmutablePointerSet.h(352): error C2059: syntax error: '='
include\swift\Basic\ImmutablePointerSet.h(353): error C2059: syntax error: 'alignof'

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

const and constexpr don't imply each other anymore, so you probably want both.

Made it constexpr const.  This required moving the definition back inline.

@swift-ci please test and merge

@swift-ci please test and merge

@swift-ci please test macOS platform

@swift-ci please test macOS platform

Build failed
Swift Test OS X Platform
Git Sha - 4ef18d56a715875a4c033fb60b263ac1999c3010

@swift-ci please test macOS platform

Build failed
Swift Test OS X Platform
Git Sha - 4ef18d56a715875a4c033fb60b263ac1999c3010

@jrose-apple , hmm, seems that it doesn't.  Sorry, not following what you mean by a separate template here.  Template what over what?

Like, move AllocAlignment out into a separate struct, which can be defined after ImmutablePointerSet is complete.

@swift-ci please test

@jrose-apple, this seems to work for Visual Studio for me.

Build failed
Swift Test Linux Platform
Git Sha - 61298b919c5896d2391623837566889373422cb2

Build failed
Swift Test OS X Platform
Git Sha - 4ef18d56a715875a4c033fb60b263ac1999c3010

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - c69ffc76a11ce69be9906038751c5217a5ab3987

The nested template is a little redundant but I guess it doesn't hurt anything in practice.
@swift-ci Please smoke test macOS

@swift-ci Please smoke test macOS platform

Seems that VS no longer needs this?  If it builds, I'm happy without this.


make multicluster secret controller handle updates

The multicluster secret controller was silently ignoring updates to
secrets. Users had to delete and re-add secrets if they wanted to
change the credentials for remote clusters. This PR adds update
support and also improves the unit tests.
fixes https://github.com/istio/istio/issues/18708

LGTM

@ayj: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
024d10bc0cb7b2b4b2f09e0133e832936127c16d
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
024d10bc0cb7b2b4b2f09e0133e832936127c16d
link
/test gencheck_istio

unit-tests_istio
024d10bc0cb7b2b4b2f09e0133e832936127c16d
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Writing ConfigStoreCache to ../config/aggregate/fakes/config_store_cache.gen.go... pilot/pkg/model/config.go:206: running "counterfeiter": exit status 1

TestFixedWindowAlgorithm also failed which seems unrelated.
/retest

Its broken by an accidentally reverted revert, should be fixed in ~30 min
or so once CI is done
On Thu, Mar 5, 2020 at 9:30 AM Jason Young<EMAIL_ADDRESS>wrote:

Writing ConfigStoreCache to
../config/aggregate/fakes/config_store_cache.gen.go...
pilot/pkg/model/config.go:206: running "counterfeiter": exit status 1
TestFixedWindowAlgorithm also failed which seems unrelated.
/retest
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/21839?email_source=notifications&email_token=AAEYGXJ6HBWOKZN7GVVCPXLRF7OSTA5CNFSM4LCAFOMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEN6FQZY#issuecomment-595351655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXLYQF64MDY67K6OBHTRF7OSTANCNFSM4LCAFOMA
.

@ayj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
024d10bc0cb7b2b4b2f09e0133e832936127c16d
link
/test unit-tests_istio

gencheck_istio
024d10bc0cb7b2b4b2f09e0133e832936127c16d
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Do I need to rebase, or is the issue in the test infra itself?

In the infra, waiting then /retest will fix it.
You should almost never need to rebase by the way, as the tests will rebase
before running.
On Thu, Mar 5, 2020 at 9:35 AM Jason Young<EMAIL_ADDRESS>wrote:

Do I need to rebase, or is the issue in the test infra itself?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/21839?email_source=notifications&email_token=AAEYGXP6DAZXU65RMY3VU7LRF7PH5A5CNFSM4LCAFOMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEN6GFNQ#issuecomment-595354294,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXLWQRBPMMQSW7AOGLTRF7PH5ANCNFSM4LCAFOMA
.

@ayj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
024d10bc0cb7b2b4b2f09e0133e832936127c16d
link
/test gencheck_istio

unit-tests_istio
024d10bc0cb7b2b4b2f09e0133e832936127c16d
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest
should be fixed

@ayj: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
024d10bc0cb7b2b4b2f09e0133e832936127c16d
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

--- FAIL: TestStreamSecretsUpdateFailures (0.51s)
sdsservice_test.go:762: unexpected metric totalPush: expected 3 but got 1

/retest

In response to a cherrypick label: new pull request created: #21869


Keeping upload failed

I am pretty sure I have correct parent folder id and all setting were just fine,
but once I upload my file to file cabinet, webstorm shows me:
'You have entered an Invalid Field Value XXX for the following field: url.
I have try reset my project setting, and reinstall the netsuite tool, but the error keeps showing.
And ideal?

Yeah that's a NetSuite error. The plugin just catches errors thrown by NetSuite when data is uploaded. I would check your url data type in your code that's throwing the error and make sure it is valid. For example sometimes I have a syntax error and when I upload, I'll get an error stating the issue.


feat: review the code of the generated results

Implemented the Logger class to show the logs.
Support jupyter notebook environment as well as simple text editor IDE.
Show the code of the generated result by the datahorse.

Feeling amazing to see you again with useful contribution, Let me review, Thanks.


URL that doesn't exist displays blank page

Description of problem
If you navigate to a url that doesn't exist, a blank page will display
Component (web, API, etc)
web
Version or commit hash (if applicable)
0.1.0
How often reproducible
always
Steps to Reproduce

Specify a url that doesn't exists, like /welder#/nope

Actual results
A blank page displays
Expected results
Some type of 404 message
Additional info

@andreasn or @larskarlitski
Could you include a link or example of how you handle this in cockpit?

There is a Not Found message and icon when an unknown page is loaded now.


edited documentation

Pull Request Test Coverage Report for Build 373

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 68.02%

Totals

Change from base Build 372:
0.0%

Covered Lines:
2212

Relevant Lines:
3252

💛  - Coveralls


Replace uses of set_sample_data

[skip changeset]
Do not merge before appsignal/appsignal-elixir#713.
Replace uses of set_sample_data
Since set_sample_data/3 is deprecated, usages of it have been replaced with set_params/2, set_environment/2 and set_session_data/2.
Point to the integration's branch
TODO:

[ ] Rebase out this commit before merging, after the deprecate-set-sample-data branch has been merged to main.

Closing. See appsignal/appsignal-elixir#713.


New-AzureRmADApplication parameter replacement

The latest version of the CmdLet New-AzureRmADApplication does not support the parameters KeyValue and KeyType, these have been replaced by CertValue

Hi @Desmopowerrr, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@Desmopowerrr : Thanks for your contribution to the Azure documentation! The author, @tfitzmac, has been notified to review your proposed change.

@Desmopowerrr - thanks for making this change.

#sign-off