Event listeners with REST API

I am trying to add an onclick listener to button on the page. Also, I am using REST API to fetch html from Plasmic. In componentProps query documentation there is a line:

This should be the props to be passed to the component, with this you can customize which variants are going to be active, define the value of expected props and override slots content (with text only).

And indeed, when I try to pass function as a prop to api, it fails with code 500. Does that mean that it is not possible to use event listeners with plasmic rest api?

That's right, there's no way to specify event listeners via the REST API. It's unclear what this would mean. For something like React, you can do this, since you are working within a JavaScript runtime, can pass enclosures that can actually interact with other parts of your application, etc. What are you looking for here?

That's unfortunate. I am trying to combine Plasmic and native Web Components (for easier implementation I use Lit). So I've created a simple component, which fetches html from Plasmic via REST and renders it. It works fine, but I guess if adding event listeners is not possible I'll have to do it theother way. Basicly, I want to use full functionality of Plasmic without any framework like React. Is this possible?

Update here. Eventually I've implemented simple querySelector("button[class*=nameOfComponentInPlasmic]") call after html from plasmic is loaded. Works ok so far, but it defenetly would be nicer to have some easier and less error-prone approach to adding event listeners with REST.

Closed by accident :(

Hi @andrewlevada! 👋
Have you try calling with the hydration option?
According to the doc, it looks like what you would like to achieve:

"Since event listeners aren’t going to be attached when only the API result is used, you may have to perform hydration, this way your pages will become responsive to user interaction. You can indicate it with the following query params."

@arenzel thanks for the response. Unfortunately, I couldn't find a way to make hydration of specifically event listeners work with rest api. @yang also pointed out that this is in fact intended.
I don't think that behavior I want to achieve is possible, unless there is a way of manually calling hydration function on client after getting code with rest.


question regarding minifcation

do you plan on exposing all the uglify options?
or do you recommend doing a bundle, then using uglifyjs itself to do something like, say, drop console.logs?

The plan is not to provide every minification option, just the useful ones. For custom minification, certainly handling it separately is recommended.


is this package dead?

there is no update since 2018, is this package dead? we are using this on our old projects, if this package no longer maintained, could you please suggest me similar package?

I've updated the project for personal use and would love to share but I'm not allowed to make a committee.

@iannortnau did you test your version using RN 0.67?

Not yet but if it's of interest to people I can test it


IE8 support

Don't know if IE8 compatibility is supposed to exist, but bridge.js doesn't work with IE8. Object doesn't support property or method 'create' is the error, denoting the absence of ES5 method Object.create. A few lines somewhere about browser support would be helpful, in case I did not just miss seeing it. Would ES5-shim do it?

Perhaps because HTML5 is explicitly stated, I'm asking the wrong question?

Reporting that bridgedotnet works in IE8 with the following polyfills: html5shiv (for HTML5 support in IE8), es5-shim and es5-sham included in that order in the HEAD section of the document.

We should include that in the documentation. Maybe someone will find it helpful, though IE8 is legacy.

Thank you very much for the feedback, @kenshinthebattosai.
We didn't really mention IE8 as not supported as we required HTML5 and not all of this is supported on IE8. But really, your directions on overcoming the issue are appreciated.
In the other hand though, other features we have do not as well support IE8. jQuery2 for example, dropped -- at least at a point -- support for the browser. Although there might be workarounds, fixes and patches, the trend now is to get it harder and harder to support without compromising the support of new features.

In most for(var in obj) loops, hasOwnProperty check has to be done.
In the loop in line number 1465,
IE8 does not include 'constructor' in the enumerated properties for the loop,
so an additional check such as
            // IE8 Fix
            if (window.isIE8) {
                if (prop.constructor) {
                    prototype['$constructor'] = prop.constructor;
                    ctorCounter++;
                }
            }

should be added

Just adding comments here if people would find it useful

Hello @kenshinthebattosai!
I'm taking this time to thank you for sharing your solution, this is invaluable for other users which come across the same issue you're encountering and this will be very useful. We'll remember this when someone else raises a question about IE8 support and workarounds.
Thank you very much!

Your feedback regarding #1050 is appreciated.
IE8 support might be dropped in the next release, milestone 1.12.


Add network manager

This PR is based on #74

Add network manager to manage network in single place
Integrate network manager for components

Summary of this PR:

Created a NetworkManager interface (along with its NetworkStorageManager and SingleNetworkManager class instances) to allow the waalletBackgroundProvider to directly retrieve the current networkActive {id, chainId, node, bundler} values from storage.

Note: This enables retrieving the ethers.JsonRpcProvider's url, provider.call(), provider.getCode() using node, and accessing methods such as getSupportedEntryPoints() and getChainId() using bundler.

Previously, the waalletBackgroundProvider would hardcode a set of node and bundler during instantiation, whereas now it retrieves them from storage.

Suggestion:
Considering the similarity in names between the JsonRpcProvider class in bundler (BundlerProvider class) and ethers.JsonRpcProvider class in node (NodeProvider class), it might be advisable for future PRs to rename them, such as BundlerRpcProvider or BundlerJsonProvider.


[FLINK-29363][runtime-web] allow fully redirection in web dashboard

What is the purpose of the change
Remove operator related configs from flink runtime config, so that users will not see any operator related config in web ui.
Brief change log

remove operator configs before deployments in Flink services.

Verifying this change
This change is a trivial rework / code cleanup with unit test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changes to the CustomResourceDescriptors: (no)
Core observer or reconciler logic that is regularly executed: (no)

Documentation

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

I've started to build an image to test the feature end-t-end. Let's see how it goes...
Additionally plz change the heading which is Flink PR compliant: [FLINK-29363][runtime-web] ...

@flinkbot run azure

I presume a rebase to the latest master is needed since second time the following error arrived:
Sep 22 08:19:47 	Suppressed: java.lang.AssertionError: Test failed Error while running command to get file permissions : java.io.IOException: Cannot run program "ls": error=1, Operation not permitted

Yes, this issue with ls is a known build failure

@rmetzger thanks for the help in advance!
I think the current change set is not going to work with same-origin policy.
Let's hear what @HuangZhenQiu thinks about this.

@gaborgsomogyi @rmetzger
Let's say proxy server is served in domain A. If the token/cookie times out, requests need to be redirected to domain B. In this case, Users need to configure CSP and CORS as below for security considerations.
For the security considerations, users need to set
Content-Security-Policy: sandbox, allow-form, allow-scripts, allow-same-origin
Access-Control-Allow-Origin: A, B
CSP guarantee the app runs in an isolated environment and also make sure cookies are attached to request to the allowed domain. Putting B in the Access-Control-Allow-Origin will make the redirection work, otherwise the redirection will be blocked by browser due to the cross origin access.

If I understand it correctly then the proxy would add CORS, right? If we can test it in a real environment then I'm fine w/ the actual code.

I've tested it manually on live cluster and it works.
@HuangZhenQiu could you do a rebase to the latest master where tests are more stable?

Thanks for manually testing it.

A rebase would be nice just to make sure there haven't been major changes on the UI side since then.

@rmetzger now it's all green and good to go :)

Hi all, what's the current status of this PR? Looks like it's ready to be merged, but hasn't been merged

You see it well...

Merging change ...


Support for BCH?

Quick update for BCH support?

Using CryptocoinService type, you can already communicate with a BCH node without any issue (as BCH is a recent fork of BTC):
            var CoinService = new CryptocoinService(UtxoNodeUri, UtxoRpcUsername, UtxoRpcPassword,
                null, UtxoRpcTimeoutInSeconds);

Closing this, based on @milkeg's comment. Feel free to reopen this, or add comments to it, if needed.


feat: Manifest for v4.14.0

Pending testing

Aesop doesn't build. Strange

it builds after rebasing on top of #31

it builds after rebasing on top of #31

Not on x86_64-linux

Solved building problem on x86_64-linux. Should be ready to merge.


[FEAT]: Implement build pre-commit hooks

Description
Preferably husky
https://typicode.github.io/husky/
Additional Information
No response

To maintain best practices and reduce load on getting back and forth on server builds.

Hello! I have worked with Husky before so I would love to work on this. Can you please elaborate on what exactly you have in mind?


FEATURE: Implement Filter/Sort UI for Allowance Holdings Data

System Increment Definition of Ready (DoR)
Before being accepted into a Program Increment each feature:

[x] Meets the INVEST criteria (Independent, Negotiable, Valuable, Estimate-able, Small, Testable)
[x] Has a clearly defined user
[x] Has an estimate
[ ] Has acceptance criteria
[x] Has sufficient context, such as

-User Research

Wireframes/UI Design

Data Conversion (completed)

System Requirements (in progress)

[x] Traceable to feature, epic or PI

[ ] Dependencies are identified

[ ] Component reusability assessed

Feature Benefit
Provide front-end user interface to access Allowance Holdings data
The following are notes for discussion. The functionality described in the user story is always the source of truth.
Context
Wireframes from 3/30/21
Allowance Holdings Filter/Sort UI Requirements
Questions:
Will the site use the EPA header/footer? Will it use the go wide template? - Waiver process has begun. The header and content are fairly defined. Footer is TBD. The development will initially focus on main content, filter sort criteria, as template work continues in parallel. Pages should be responsive to screen size.
Assumptions:

Blank parameter values will return all records
If multiple parameters are selected in a single query (TX, general account), then only values that match all parameters will be returned (AND)
If multiple values are sent for a single parameter (e.g. TX, MD), then all data that matches the values is returned (OR).
Download limits will be implemented in future and are out of scope for initial development
Users must select data type and data subtype before other fields are available
The site will be responsive to different desktop screen sizes (not mobile compatible)
Processing the file download will be handled by the browser

UI Parameters
Data Type
Button/ Drop down
Value:  Emissions, Allowance, Compliance, default blank,
Required
Single select
Data Sub-Type
Drop down
Value: Allowance Holdings,  Allowance Transactions, Account Information, default blank
Required
Single Select
Vintage Year
Text field. Allows ranges and individual years (YYYY-YYYY, YYYY, YYYY)
Format: YYYY
Optional
Valid date range: 1995 to current year
Multi select (multiple 2019,2020, range 2017-2020, and indiv. Years 2015)
Breadcrumb display: Vintage year (no years displayed)
Program
Drop down, default blank
Values: All active programs (ARP, CSNOX, CSOSG1/2, CSSO2G1/2, TXSO2)
Optional
Multi select
Out of Scope: If account is already selected, then search is limited to programs applicable to the account state
Account Type
Drop down, default blank
Values: Facility, Unit, General, Reserve, Surrender, All
Optional
Multi select
Account
Account Name
Dynamic search
Search is a contains search on buyer and seller accounts Does allowance holdings filter on buyer and seller accounts?
Results are displayed in Account Name (Account Number) format
Results populate in a list, parameters populates when a value is selected from the list  	(results list should stay at top of page/visible in current view, possible use pagination to keep window small)
Search is not case sensitive
Out of scope: If state is already selected, then search is limited to accounts in the selected state
If no results are found, the system will indicate so. “No accounts match your search.”
Multi select
Account Number
Dynamic search
Search is a contains search on buyer and seller accounts
Results are displayed in Account Name (Account Number) format
Results populate in a list, parameters populates when a value is selected from the list 	(results list should stay at top of page/visible in current view, possible use pagination to keep window small)
Search is not case sensitive
Out of scope: If state is already selected, then search is limited to accounts in the selected state
If no results are found, the system will indicate so. “No accounts match your search.”
Multi select
Facility
Facility Name
Dynamic search
Search is a contains search
Results are displayed in Facility Name (ORIS Code)
Results populate in a list, parameters populates when a value is selected from the list  	(results list should stay at top of page/visible in current view, possible use pagination to keep window small)
Search is not case sensitive
Out of scope: If state is already selected, then search is limited to facilities in the selected state
If no results are found, the system will indicate so. “No facilities match your search.”
Multi select
ORIS Code
Dynamic search
Search is a contains search
Results populate in a list, parameters populates when a value is selected from the list
Results are displayed in Facility Name (ORIS Code) format
Out of scope: If state is already selected, then search is limited to facilities in the selected state
If no results are found, the system will indicate so. “No facilities match your search.”
Multi Select
Owner/ Operator
Owner
Dynamic search
Search is a “beings with” search on buyer and seller accounts - Confirm if this should be a begin with search.. Requirements say contains search
Results populate in a list, parameters populates when a value is selected from the list 	(results list should stay at top of page/visible in current view, possible use pagination to keep window small)
Search is not case sensitive
If no results are found, the system will indicate so. “No accounts match your search.”
Multi select
Operator
Dynamic search
Search is a “beings with” search on buyer and seller accounts
Results populate in a list, parameters populates when a value is selected from the list 	(results list should stay at top of page/visible in current view, possible use pagination to keep window small)
Search is not case sensitive
If no results are found, the system will indicate so. “No accounts match your search.”
Multi select
State
Dynamic search
Search is a “begins with” search
Results populate in a list, parameters populates when a value is selected from the list
Results display the full state name (Texas, not TX)
Out of scope: If Facility/ORIS Code or Account Name/Number is already selected, then state is automatically populated with the state (or states) associated to the facility/account (or facilities/accounts)
Multi Select
Change Criteria
The user will have the ability to clear one, many, or all filter criteria
If the user changes data type or sub data type, then:
The filter criteria displayed updates accordingly
No previously entered filter criteria is retained
If a user changes data type or sub data type, then filter criteria is cleared
If the user chooses to clear one or more filter criteria before preview is selected, then the system clears the criteria from the filter and updates any remaining dependent criteria accordingly.
If the user chooses to clear one or more filter criteria after preview is selected, then the system clears the criteria from the filter AND download is disabled until data is previewed.
Data Preview
Trigger: A user has entered, at a minimum, the required parameters, and selected the option to preview data.
For each record that matches the criteria, the system will display in a tabular format:
ACCOUNT_NUMBER
ACCOUNT_NAME
ORISPL_CODE
PRG_CODE
VINTAGE_YEAR
TOTAL_BLOCK
START_BLOCK
END_BLOCK
STATE
EPA_REGION (2 digit number)
OWN_DISPLAY
ACCOUNT_TYPE
Data Labels-
Refer to the CAMPD_ DataLabels_AllowanceCompliance file here
Results will display record count
Results are limited to 100 records
Results in preview will have a scroll functionality to show all columns/rows
NOTE: NEED TO ID COLUMNS FOR HOVER TEXT AND DISPLAY TEXT
OUT OF SCOPE: Results may be sorted in alphabetical ascending and descending order
No additional filtering: CAN THEY BE FILTERED? IF SO BY WHAT CRITERIA?
OUT OF SCOPE: Results are paginated
File Selection
Trigger: The user has at least one record that matches the search criteria, and has previewed the results
The user has the option to select the file output in JSON or CSV format. CSV is the default file type
API will provide the data in the requested format based on user selection in UI
Download
Trigger: The user has selected a file output type
When the user selects the option to download, the system creates a file in the format specified  containing all records that match the parameters and downloads the file
After a user has chosen to download data, then the system will display the preview of the downloaded results.
Error Conditions
API Timeout
Default API timeout = TBD
When default time out is reached, API will generate error condition. Display message in UI. SAMPLE TEXT: The data you requested is too large to download. Refine your criteria to limit your results, or visit the Datasets page .
Bad request
Internal server error/network outage
New FC = 3 (4)
Existing FC = 1 (3)
Preview = 2
Filter Tags = 2
Download = 3

@JanellC I think we are missing a ticket for the Vintage year filter. Not seeing it under this epic. Thanks!

@tlageman-epa Thank you for pointing this out. For some reason AH: Filter Custom Data Download By Vintage Year #1102 was closed. I reopened the issue and the feature should reflect the change.

@JanellC Same as your other question. Buyer/seller accounts are not applicable here. This data is holdings data for every allowance account for all programs. This is not transaction data.


X.509 certificate isuue when executing ks init

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind feature

What happened: while exercising the tutorial guestbook application I get the following error
ks init guestbook --context ks-dev
INFO  Using context 'ks-dev' from the kubeconfig file specified at the environment variable $KUBECONFIG
ERROR Get https://api.github.com/repos/ksonnet/parts/commits/master: x509: certificate signed by unknown authority
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
exercise the tutorial steps starting from the beginning but the mac terminal (from where the commands are executed) is behind the enterprise proxy
Anything else we need to know?: The cluster is in AWS east and the KubeConfig points to the cluster
Environment:

ksonnet version (use ks version): 0.9.2
Kubernetes version (use kubectl version): client 1.9.0, Kubernetes version 1.8.3

I have an update on this issue. I have traced the issue to the proxy. For some reason the init command (or one of the libraries) is not getting the proxy information and hence fails.
I have worked around by cloning the repo and performing a local build which uses the local environment variables during compilation.
I believe the binaries need to have a flag to get the proxy variables passed through to all the libraries.

I'm having this issue as well when following these instructions on the main page(https://ksonnet.io/).
These instructions:
 # View all available commands
  ks --help

  # Create an app directory (and reference your cluster with $KUBECONFIG)
  KUBECONFIG="${KUBECONFIG:-$HOME/.kube/config}"

  ks init ks-example    # Add '--api-spec=version:v1.8.0' to this command if you're running Kubernetes 1.8!
  cd ks-example

  # Autogenerate a basic manifest
  ks generate deployed-service guestbook-ui \
    --image gcr.io/heptio-images/ks-guestbook-demo:0.1 \
    --type ClusterIP

  # Deploy the manifest to your cluster, and view results
  ks apply default
  kubectl get svc guestbook-ui

  # Teardown (Don't worry, this only removes resources defined in your ksonnet app,
  # and should not affect any other resources in your 'default' namespace)
  ks delete default

My Session
bash-3.2$ ks init ks-example --insecure-skip-tls-verify
INFO Using context "aws" from kubeconfig file "/Users/kuberchaun/.kube/config"
INFO Creating environment "default" with namespace "default", pointing to "version:v1.10.3" cluster at address "https://63AA204DEA02A6647A0A60577038B340.yl4.us-east-1.eks.amazonaws.com"
ERROR Get https://raw.githubusercontent.com/kubernetes/kubernetes/v1.10.3/api/openapi-spec/swagger.json: x509: certificate signed by unknown authority
To undo this simply delete directory 'ks-example' and re-run `ks init`.
If the error persists, try using flag '--context' to set a different context or run `ks init --help` for more options
bash-3.2$

The only thing of interest that I have going on is we are using AWS EKS and using heptio authenticator aside from that nothing too strange.

@bobhenkel this issue was resolved in #832 and is available if you build master. Once you have done that, you can pass the new —tls-skip-verify flag to avaoid the certificate errors stemming from the proxy.


Fix AccessKey.iam reconciliation

Description of your changes

Depends on: https://github.com/crossplane/upjet/pull/417
If the MR API has a spec.forProvider.status field and there are sensitive attributes, then fieldpath.Paved.ExpandWildcards complains instead of expanding as an empty slice, which breaks the reconciliation.
Please refer to https://github.com/crossplane/upjet/pull/417 for more context on the issue.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Manually verified AccessKey.iam now successfully reconciles and its connection details secret contains all the keys from the v1.4.0 version of the provider: attribute.secret, attribute.ses_smtp_password_v4, password, username
Via uptest run here: https://github.com/crossplane-contrib/provider-upjet-aws/actions/runs/9399021904
Also validated that the current spec.initProvider behavior is not broken with the uptest run of User.mq resource here: https://github.com/crossplane-contrib/provider-upjet-aws/actions/runs/9399082530

/test-examples="examples/mq/v1alpha1/user.yaml"

/test-examples="examples/iam/v1beta1/accesskey.yaml"


[Fix-2806] [Flink] The job parameters are not effective when the set parameters key and value contain single quotes

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

close #2806


Adding IsConnected() convenience function to session

Hey thanks for the PR, looks great so far. I agree with the comments that @alkchr made, if possible could you look into making the suggested changes?

Hey guys thanks for the feedback.

Made change in comment style based off of the rest of the comments in library
Merged the two if statements into one (session.cluster==nil and session.closed)
Will return value of session.cluster.IsConnected() if session.cluster is not nil

Awesome :+1: Will merge when I release v1.3.0. Thanks again for the contribution.


Copy relevant try-convert contents to UA repo

Our PRs into the try-convert project are gated on contributors,
while that project is openly stated as unmaintained.
Since we were previously pinning to an exact commit in t-c anyway,
we have decided to move forward with copying the portions of t-c
which UA utilizes directly into the try-convert extension area of
the project.
This will allow for faster iteration on fixes/extensions that require
updates to try-convert's functionality and decouple us from a
deprecated project.
Fixes #1242

Only diffs are expected ones


[Feature] The "Chart.getDataURL()" method does not contain a background image

What problem does this feature solve?
I expect to get the base64 address of the trend chart with background pictures, but the base64 I get through getDataURL does not contain background pictures
What does the proposed API look like?
I expect to get the base64 address of the trend chart with background pictures, but the base64 I get through getDataURL does not contain background pictures

If you are expecting to get the background of the container of your chart, it's impossible. You need to set the background of the chart via the backgroundColor option.


Standardize time in seconds.

BP uses two types of time - seconds and milliseconds - and there's a conversion factor of 1000 thrown around to keep things going in a dozen places in the code.
This is a trap.  We should just use Python standard times - seconds - everywhere.

Going through stuff assigned to me... where are you at on this? Looks like you've already started.

I assigned this to you?! Er...
I did nothing with it except make sure that you can grep for 1000.0.  :-D
This isn't really a 100% important thing, but it'll make for better
consistency.
On 5 March 2017 at 00:11, Adam Haile<EMAIL_ADDRESS>wrote:

Going through stuff assigned to me... where are you at on this? Looks like
you've already started.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/104#issuecomment-284191707,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsjBMcU4p6G6n-yzEg6dbQvaiKfMXks5rie-LgaJpZM4MARsN
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000

It's dual assigned.
On Sat, Mar 4, 2017 at 6:18 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

I assigned this to you?! Er...
I did nothing with it except make sure that you can grep for 1000.0. :-D
This isn't really a 100% important thing, but it'll make for better
consistency.
On 5 March 2017 at 00:11, Adam Haile<EMAIL_ADDRESS>wrote:

Going through stuff assigned to me... where are you at on this? Looks
like
you've already started.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/104#issuecomment-
284191707>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAPdsjBMcU4p6G6n-
yzEg6dbQvaiKfMXks5rie-LgaJpZM4MARsN>
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/104#issuecomment-284192132,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6nE0FjAFnVk9S1gFu4Bu9rNnhjHBks5rifFMgaJpZM4MARsN
.

@adammhaile I unassigned you from this.
I'll hit this at the end once all other things are done.  I think it's somewhat important, if only to prevent people from getting "off by 1000" errors - but it's independent of everything else and if you had some sentimental reason not to do this...


The API docs is missing the attributes included in System.Text.Json.

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem (if available).
Why isnt this just attached to every page already?
Link to article:
https://docs.microsoft.com/en-us/dotnet/api/system.text.json?view=netcore-3.0
Problem:
Attribute documentation is missing. I'm trying to find the equivalent for something like this...
[System.Text.Json.JsonProperty("Ip", Required = System.Text.Json.Required.Default, NullValueHandling = System.Text.Json.NullValueHandling.Ignore)]

but the best thing I can find is just JsonPropertyName and nothing that handles the required or null handling.

Where are you seeing this attribute @StingyJack?

JsonPropertyName ?
It's one of the attributes in the System.Text.Json assembly. There are a few others that are missing from the reference docs.

@tdykstra @ahsonkhan can you please look at this feedback?

There is no equivalent in System.Text.Json to the Newtonsoft.Json required property attribute.  We have an article about to be published explaining which features S.T.J. has that are equivalent to Newtonsoft.Json features and which it doesn't have.  For some of the missing ones, including required property attribute, writing a custom converter is a possible workaround. Until the how-to-migrate article is published, see issue 38492 .

I can dig that the "required" part of the functionality isnt available. Can the other attributes get some docs though?

The attribute docs are here:
https://docs.microsoft.com/en-us/dotnet/api/system.text.json.serialization?view=netcore-3.0#classes
For more detailed information, most of those API reference docs link to this doc:  https://docs.microsoft.com/en-us/dotnet/standard/serialization/system-text-json-converters-how-to?view=netcore-3.0#register-a-custom-converter

Then there is just some link missing from here  to those attribute documents then.

I'll add a link to the Remarks section.

Then there is just some link missing from here to those attribute documents then.

That link covers the System.Text.Json namespace. The attributes you are looking for are in the nested S.T.Json.Serialization namespace (see link @tdykstra provided), so I am not sure what the fix for this should be. Are you suggesting that the root namespace provide a link to a nested namespace? It's not just attributes that don't show up, JsonConverter doesn't either (because it is not in the S.T.Json namespace).
Should System.Text have a link to all the other sub-namespaces (S.T.Json, S.T.Encoding, etc.)? Should it only go one level deep?
Or, do we think that namespaces that convey a single "concept" should have links to each other, to help with discovery?

The things that are in the same assembly should be linked together.


Rewrite greeting activity trigger in order to work with web chat and directline

Is your feature request related to a problem? Please describe.
We found that if we use a bot created using Bot Framework Composer in Web Chat control using DirectLine channel, the greeting is displayed twice. The second time is after user sends first message to the bot. The reason is clear, is related to the double event OnConversationUpdateActivity that is fired when the conversation between bot and user starts, the first when bot is added to conversation and the second when is added the user.
We know that it is a "knowed problem" on BotFramework with directline that is solved in most of templates with this two lines of code:
foreach (var member in membersAdded)
if (member.Id != turnContext.Activity.Recipient.Id)
Describe the solution you'd like
I'd like that Bot Framework Composer editor Greeting works by default like in DialogAndWelcome Bot example of Bot Framework SDK 4.
Describe alternatives you've considered
Our first try was to use a distinct value on:
OnConversationUpdateActivity.condition
toLower(turn.Activity.membersAdded[0].name) != 'bot'
->
substring(toLower(turn.Activity.membersAdded[0].name), 0, 3) != 'bot'
Because we detect that name is not always "bot" in our case was "bot-xxxx..."
But it seems that condition on OnConversationUpdateActivity is not really working: we print the condition on message to user and it send 'True' and 'False' but is showing twice.
We found a workarround following this steps:
· blank trigger OnConversationUpdateActivity condition (because is not working)
· adding "Loop: for each item" with itemsProperty = turn.Activity.membersAdded
· adding a "Branch: if/else" with condition = string(dialog.foreach.value.id) != string(turn.Activity.Recipient.id)
· Send welcome message only if condition is True

  {
      "$type": "Microsoft.OnConversationUpdateActivity",
      "$designer": {
        "id": "376720"
      },
      "actions": [
        {
          "$type": "Microsoft.Foreach",
          "$designer": {
            "id": "887482",
            "name": "Loop: for each item"
          },
          "itemsProperty": "turn.Activity.membersAdded",
          "actions": [
            {
              "$type": "Microsoft.IfCondition",
              "$designer": {
                "id": "129743",
                "name": "Branch: if/else"
              },
              "condition": "string(dialog.foreach.value.id) != string(turn.Activity.Recipient.id)",
              "actions": [
                {
                  "$type": "Microsoft.SendActivity",
                  "$designer": {
                    "id": "789374",
                    "name": "Send a response"
                  },
                  "activity": "- Welcome, I'm Bot Composer!"
                }
              ]
            }
          ]
        }
      ]
    }

This workarround fixes the bug and we think is the equivalent to Bot Framework standard Welcome Bot:
foreach (var member in membersAdded)
if (member.Id != turnContext.Activity.Recipient.Id)
· We use IDs instead of Names.
· Foreach is better than single condition, because we thinks this will work on other channels with more than user at conversation with the bot like in Teams.
Additional context

Same as #1632

addressed in https://github.com/microsoft/BotFramework-Composer/pull/1794


Support for compound words in predictive mode

In swedish, and possibly other languages compound words exist where words consist of two or more words joined together. For example the word for fever thermometer in swedish is "febertermometer" consisting of the word "feber" for fever and "termometer" for  thermometer.
Since it would not be feasible to add all possible combinations of words that make up these compound words to the dictionary it would be good to be able to mark that input has finished for one part of the word and that a new word is starting without a space being added in predictive mode.
Suggested flow:

Enter first part of word 'feber'
Press d-pad "Down" to mark that the first part of the compound word is finished
Enter last part of word 'termometer'
Press "OK" or 0 for space
Enjoy your beautiful new compound word and possibly press * to add it to the dictionary if used often.

References:
http://nds2.webapps.microsoft.com/files/support/apac/phones/guides/Nokia_5210_APAC_UG_en.pdf page 20.

Well, you can type compound words even now. Just type the first word, hit backspace to erase the extra space and type the second word.
Or, if you don't want automatic spaces added, just turn them off from Settings -> Keypad -> Predictive Mode section -> Automatic Space. This way, OK will not add an extra space and you would be able to choose whether to add one with 0-key or not.
Now, if you still want automatic spaces only after punctuation, it is possible to split the setting and have something like "Automatic space after words" and "Automatic space after punctuation". This way, you would have even finer control. And... I think it's the best I can do. You can't have auto correction on and off at the same time... :slightly_smiling_face:
Finally, TT9 recreates the Nokia experience on modern Android devices, but it is never going to be exactly the same as Nokia 5210 or 3310 or any other. You can't expect that. It will always be somewhat different from your past favorite phone.


Add Discord Clone

Here is a link:  https://discord-clone-rajdeep019.vercel.app/dashboard

Hi @Rajdip019      you can work on this issue ur project should have a working link and you should add that in the main website too once done with creating it .
Deadline -  5-6 days

@Rajdip019 deadline passed and someone already make a discord clone, So I'm closing this issue


How do I know which runtime version can be installed with a certain provider?

There isn't really any way to do that. The SCL provider is limited to specific versions based on what SCL packages it knows exist. The other providers can theoretically install any version. The system provider can only check during the install process because it needs to look at the actual package version. The best answer I can give is "whatever the underlying system supports, I support".


Minor Typo on home page

Replace "creats" with "creates".  Super minor.

Thanks, it's been updated!


chore(settings): remove emoji to fix settings bot

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

GitHub release
v1.3.0

Your semantic-release bot :package::rocket:


Make the measurements param an array

Proposed changes
the m param is actually an array so fixing this
Checklist

[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[x] I have run make fmt and formatted my code

@themantissa this is only used by cli so I'll merge this away

Makes total sense but thank you for the tag/info @gssbzn !


Set user roles

This is just a nitpick, it wouldn't prevent a merge, but it would be nice if both Roles and Effectives Roles were in the same sort order. It makes lists easier to compare. There's no reason to actually compare these lists though, haha, it's just an instinct, so it's a little distracting.

I did a very quick smoke test of this and it seems to work. Please go ahead and merge it whenever you're ready.
I do have two questions. First question, why when I sign in with the testcafe user do I only see FMDB and IAMDRUPAL clients? Didn't it used to show the full list of clients, that is, including the default Keycloak clients? For a test, I added a new client SOMECLIENT, and it doesn't show up in the Applications list, not even when I assign a role to the user I'm viewing.
Second question, and probably related. Can we use the User Management console to assign another user user-management/realm-management permissions? Or can that only be done in the Keycloak Admin Console?


Ability to patch/overwrite missing/wrong declarations

Search Terms

correct wrong declaration, fix declaration, overwrite module declaration, fix module type
Suggestion

Sometimes we encounter an npm module with missing declarations and incorrect declarations in its types file. Wish we have this ability patch/correct its declaration for temporary using before PR a patch and have it's released.
Current behavior & Workaround
Consider this situation, module moduleWithIssues indeed exports itemExistedWithoutDeclaration but its declaration file doesn't contain it, and has a incorrect declaration itemWithWrongDeclaration
import {
    foo, bar,
    itemExistedWithoutDeclaration,   // report 'itemExistedWithoutDeclaration' doesn't exist
    itemWithWrongDeclaration
} from 'moduleWithIssues'

// 'itemWithWrongDeclaration' is number type but declared as string, TS report type error
console.log(Math.abs(itemWithWrongDeclaration))

At present, I found a workaround is adding a local module declaration for itemExistedWithoutDeclaration and assert itemWithWrongDeclaration as its correct declaration
import { foo, bar, itemExistedWithoutDeclaration, itemWithWrongDeclaration } from 'moduleWithIssues'

declare module 'moduleWithIssues' {
  const itemExistedWithoutDeclaration: number
}

const itemCorrected: number = itemWithWrongDeclaration as any

It works in *.ts file but not in *.d.ts file. An idea patching solution should be re-declare some items of moduleWithIssues in a *.d.ts file in the project. like below:
// interfaceInProject.d.ts
declare module 'moduleWithIssues' {
  const itemExistedWithoutDeclaration: number
  const itemWithWrongDeclaration: number
}

unfortunately this patch module will shadow original module declaration of moduleWithIssues  package.
import { foo, bar, itemExistedWithoutDeclaration, itemWithWrongDeclaration } from 'moduleWithIssues'

foo and bar are reported non-existent.

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. new expression-level syntax)

It seems like the current behavior is actually ideal, because otherwise you'd have no way to remove an incorrect declaration?

Maybe we need a way to partially change declarations. Many patched declarations are not very correct.

Any news? I also want to override incorrect declaration of document.open (lib.d.ts)
Because this is wrong typing
open(url?: string, name?: string, features?: string, replace?: boolean): Document;

correct is:
open(url: string, name: string, features: string): Window;
open(type?: string, replace?: string): Document;

https://developer.mozilla.org/en-US/docs/Web/API/Document/open

No news

This really is an annoying limitation. Eg. relay-compiler just updated from version 6 to 7, so while I tried to update the relay-compiler-language-typescript plugin I needed to modify the existing generate function on an interface (in definitelytyped), to add another parameter to it. Instead of being able to make a local @types/relay-compiler file and make the override there, I have to use give the interface another name and use that throughout the code. Logically thinking you should be allowed to override from local definition file.

In my case, the @types/chai library declares:
interface Object {
  should: Chai.Assertion;
}

And this infection affects how I use elasticsearch (that actually has should properties). I wish I could just force the Object interface back to its default state.

I'm not sure if this is a correct topic, but I'm having related issues:
typeorm module
export declare function Entity(options?: EntityOptions): Function

local global.d.ts
import * as typeorm from 'typeorm'

declare module 'typeorm' {
	export function Entity(options?: typeorm.EntityOptions): ClassDecorator;
	export function Entity(name?: string, options?: typeorm.EntityOptions): ClassDecorator;
}

Doesn't work:

nvm, it just miraculously worked now

Sometimes when working with changes on .d.ts files, restarting the TS Server fixed the issues, maybe that was your case?

I am creating a library contains a class extends from eventemitter. I create declaration for some event.

I think something like this would be awesome. Fastify (a JS library with their own .d.ts declaration files) have some interfaces that could benefit from this. For example I'm trying to replace the FastifySchema interface that accepts unknown as values to actual schema definitions with TSchema from typebox, but I'm unable to replace their unknown types.


OrchardRedis - RedisDatabaseExtensions, use of keys in script.

The RedisDatabaseExtensions (https://github.com/OrchardCMS/Orchard/blob/1.9.2/src/Orchard.Web/Modules/Orchard.Redis/Extensions/RedisDatabaseExtensions.cs) are using the KEYS command during script for counts, and deleting by prefix?
The KEYS command is pretty bad, in fact the redis documentation warns explicitly against it in production. (http://redis.io/commands/KEYS) as it can effectively dos your redis server. SCAN is recommended over KEYS. I also think KEYS won't work properly ran like this (from script) in a redis server set up in a cluster (eg. the new azure redis premium tiers), as KEYS targets a single server. (though the same can be said of SCAN) reference - https://github.com/StackExchange/StackExchange.Redis/blob/master/Docs/KeysScan.md
The RedisOutputCacheProvider (https://github.com/OrchardCMS/Orchard/blob/1.9.2/src/Orchard.Web/Modules/Orchard.Redis/OutputCache/RedisOutputCacheStorageProvider.cs) actually implements this much better but only in one place, see line 125). This uses the .Keys method on the StackExchange.Redis client api, which will use SCAN instead of KEYS if the server supports it (older versions of redis don't). It also handles the cluster scenario by looping through each endpoint. Unfortunately in other places the provider is using the databaseextensions previously mentioned.
The extensions probably want updating to use the .Keys method on the StackExchange.Redis client api and to loop through the available endpoints.

Would you mind providing a PR?

@sebastienros - sure, will try and look over it more this weekend and get a pr over

Going to test this a bit more tomorrow but i essentially have a pr once i'm happy with it

This might have been some sort of merge error, but RedisConnectionExtensions.cs is still in the repo on 1.10.x, but not added to Orchard.Redis.csproj. The commit fixing this issue actually removed RedisDatabaseExtensions.cs (still in the repo and the csproj) in favor of RedisConnectionExtensions.cs.
I think this basically means that this bug is NOT fixed on 1.10.x.

Please file another issue and tag it 1.10.2

Done!


Add DRink! as backend for ink-wrapper-types.

This PR adds the support for DRink! backend to ink-wrapper-types. It also removes the support for aleph_client - see CHANGELOG.md for explanation.
NOTE: As stated in the CHANGELOG -- the only compatible runtime for DRink! integration is MinimalRuntime.
Closes:
https://cardinal-cryptography.atlassian.net/browse/A0-3910
https://cardinal-cryptography.atlassian.net/browse/A0-3908
https://cardinal-cryptography.atlassian.net/browse/A0-3907

I've tested latest commit with all my latest work on common-amm (10+ different drink-based tests in two different crates) and it works correctly.


C# http template returns JSON / XML instead of plain text

Not sure if this was a deliberate change, but raising it in case. Previously, HttpTriggerCSharp was returning text. Now, when I run it from portal, the response has JSON, e.g.
"Hello Azure"

What I don't like is that if you run copy the function your and run it from new tab, you get XML, e.g.
<string xmlns="http://schemas.microsoft.com/2003/10/Serialization/">
Please pass a name on the query string or in the request body
</string>

That feels pretty heavy for what's supposed to be the simplest template. That may be a runtime issue rather than template.

Also, it's inconsistent with the node http template, which returns plain text.

+1 on making these consistent. Regarding the returned content type - for C# we pretty much return the response the user created w/o modifying it, so likely not a runtime issue.

@mathewc on the runtime side, maybe it's possible to tweak WebAPI to always default to JSON? It just sucks that simply pasting the URL into a browser gives you XML :(

The behavior change was probably introduced with the template update as it was changed from creating an instance of an HttpResponseMessage to use the CreateResponse method on the request.
I do like the current behavior as it is correctly negotiating the content type, though.
Regarding the default, it is already JSON, but you're likely making the request using Chrome, and the browser is sending an accept header that includes application/xml and not application/json and since there is a registered formatter that knows how to satisfy that, it will be used.
If you try using Edge (which doesn't include application/xml), you should get a JSON response as there will be no formatter to satisfy the accept header media types and will fallback to JSON.
I can change it so that it always use JSON in the template. But I don't think we should change the runtime as it is behaving correctly.
Thoughts?

Thanks, I guess that makes sense. I guess let's not worry about this now, as it's way below the beta bar.

The current functionality is correct. We should honor the "Accept" header as-is.


Action executions are suspended

I also got an email from github that notified me that I burnt through all my free build minutes for January 🤦🏻♂ A few of the Acrionsflow builds lasted hours before failing 🤦🏻♂ Ouch

I also got an email from github that notified me that I burnt through all my free build minutes for January 🤦🏻♂ A few of the Acrionsflow builds lasted hours before failing 🤦🏻♂ Ouch

Workaround: I made a little change into my .github/workflows/actionsflow.yml to downgrade to old act version.
       - name: Setup act
         uses: actionsflow/setup-act-for-actionsflow@v1
+        with:
+          version: 0.2.17

And, I also set Github Actions timeout to 3min just in case.
 jobs:
   run:
     runs-on: ubuntu-latest
+    timeout-minutes: 3

Workaround: I made a little change into my .github/workflows/actionsflow.yml to downgrade to old act version.
       - name: Setup act
         uses: actionsflow/setup-act-for-actionsflow@v1
+        with:
+          version: 0.2.17

And, I also set Github Actions timeout to 3min just in case.
 jobs:
   run:
     runs-on: ubuntu-latest
+    timeout-minutes: 3

Looks like the issue is in actionsflow/setup-act-for-actionsflow — when it initializes act, it's not creating the ~/.actrc file. It should create the file and write something in it, based on one of the three choices (large, medium or micro): https://github.com/nektos/act/blob/56ec36726b384e2f860bbc5b5a9170a3d0c9f511/cmd/root.go#L96-L101
Note that I think the last entry for the medium option is missing the -P prefix

Looks like the issue is in actionsflow/setup-act-for-actionsflow — when it initializes act, it's not creating the ~/.actrc file. It should create the file and write something in it, based on one of the three choices (large, medium or micro): https://github.com/nektos/act/blob/56ec36726b384e2f860bbc5b5a9170a3d0c9f511/cmd/root.go#L96-L101
Note that I think the last entry for the medium option is missing the -P prefix

Downgrading the default version to 0.2.18, and I'll fix the 0.2.19 issue~

Downgrading the default version to 0.2.18, and I'll fix the 0.2.19 issue~

It's ok now, I have fix the act 0.2.19 issues, https://github.com/actionsflow/setup-act-for-actionsflow/issues/1
The v1 has already alias to the latest release, so you don't need do anything.
Feel free for further issues, thanks~

It's ok now, I have fix the act 0.2.19 issues, https://github.com/actionsflow/setup-act-for-actionsflow/issues/1
The v1 has already alias to the latest release, so you don't need do anything.
Feel free for further issues, thanks~


Replace defunct JCenter with Maven Central

Following up on #12 and #14, this PR should complete the work of publishing RR as a reusable library.
As JCenter was turned into a read-only archive in 2021, RR needs a new distribution repository. Using Maven Central has the added benefit that developers will not have to configure anything to consume the artifact, but just add a dependency to their project. The standard way to do this is to use Sonatype OSSRH (OSS Repository Hosting).
With this PR, the build should set all fields required by the OSSRH guidelines. The approach taken is based on this article.
Before merging this, do the following:

Set up an OSSRH account and repository following the official guide.
OSSRH may tell you to use a different server. If that's the case you need to adapt releaseRepo and snapshotRepo in the Gradle file accordingly.
Put your credentials for GPG (signing.keyId, signing.password and signing.secretKeyRingFile) and OSSRH (ossrhUsername and ossrhPassword) into ~/.gradle/gradle.properties.
Perform a publish with a snapshot version. Log in to OSSRH to verify the artifacts were uploaded.

Note: while snapshots can be consumed by developers (via the special OSSRH snapshots repository), unlike releases they do not get pushed to Maven Central.

After merging this, you should be able to publish a master build (non-snapshot versions) to OSSRH's Nexus. There, you can log in, find the staging repository for your build and trigger the release process which includes automated requirements validation. If successful, the build will be propagated to Maven Central in a matter of hours.
Let me know if you need any assistance with this.

Great stuff. Thank you very much for your guidance, it worked exactly as you described it. The only thing that I had to add was signing the webapp.


feat: better diagnostic report for harmony residual dependencies

Summary
Related to #7336
This PR solves the problem of partial harmony dependencies not aligned with webpack.

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

@h-a-n-a cc


SETD2-related neurodevelopmental disorder without or with macrocephaly/overgrowth

Preferred gene-related syndrome label
SETD2-related neurodevelopmental disorder without or with macrocephaly/overgrowth
Synonyms
Parent term (use OLS, or your favorite ontology browser)
Intellectual Disability
Definition
Please write the definition in the format: Any [parent class] in which the cause of the disease is a mutation in the [gene name] gene. For example: Any congenital myopathy in which the cause of the disease is a mutation in the TPM3 gene.
A neurodevelopmental disorder caused by heterozygous variants in SETD2 and characterized by intellectual disability or developmental delay, motor delay, speech delay, hypotonia, autism spectrum disorder, attention deficit disorder, and sometimes features such as macrocephaly, overgrowth, and dysmorphic features. In the literature, the disease entities reported with these phenotypes have been separated into Luscan-Lumish syndrome (MIM#616831) and intellectual developmental disorder-70 (MIM#620157). Given current evidence, these entities are believed to fall on a single spectrum of disease and are both included as a part of SETD2-related neurodevelopmental disorder without or with macrocephaly/overgrowth.
This isn't the standard format for gene-related conditions, let me know if you want me to submit this under the regular "new term" ticket template instead.
**Definition source (Please give PubMed ID, if applicable, in format PMID:#######)
Children terms (if applicable) Should any existing terms be re-classified as children underneath this new proposed term?

SETD2-related microcephaly-severe intellectual disability-multiple congenital anomalies syndrome
Intellectual Disability Disorder 70 (MIM#620157 - https://www.omim.org/entry/612778?search=setd2&highlight=setd2). There is no MONDO# for this term currently. While we do not need have to have this term listed as a child, if you would like us to add this to be consistent with OMIM, please let us know.

Your nano-attribution (ORCID) or URL for a working group
If you don't have an ORCID, you can sign up for one here
https://clinicalgenome.org/affiliation/40006/
@ErinRiggs

Sounds good. Thanks!


Purpose of Test "CheckQosScripts"?

Looks like the Test "CheckQosScripts" is checking if "qos-scripts" was not installed,
what was the purpose of this test? should it run before some other tests to make sure "qos-scripts" was not installed?

There were some people who made some openwrt builds that wanted to make sure qos-scripts package was not installed because it interfered with some things they cared about.
Feel free to remove that from your own test suite, you probably don't need it.  You can add or remove whatever tests you want by changing testsuites.cfg: https://github.com/qca/boardfarm/blob/master/testsuites.cfg

Thanks, @mbanders


autoFindInSelection does not work in replace mode

The setting editor.find.autoFindInSelection doesn't work as it should when entering replace mode. One work around is to press Ctrl+F Ctrl+H.
It was reported casually in the thread 27083 but I couldn't find an Issue or acknowledgement from developers.

VSCode Version: 1.19.2
OS Version: Windows 8.1

Steps to Reproduce:

Select text
Press Ctrl+H

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

Already fixed ;)

This is not fixed, can still reproduce the same error in.
VSCode Version 1.21.1
OS Version: Windows 10 Pro, 1709, build 16299.309

Doesn't work at all when only a single line is selected.


cosign copy no longer copies signatures by default in 2.2.1

Description
We recently started using cosign at v2.2.0. We have a CI step that runs cosign copy temp-image prod-image which would copy the container image and signatures.
After the cosign-installer action got updated in our repo this week, I noticed I could no longer verify signatures on prod images because the signatures were missing from the prod registry.
I tested with -d using 2.2.0 and 2.2.1 and confirmed 2.2.1 doesn't copy or look for existing .sig artifacts without setting -only sign. 2.2.0 does copy without needing extra params.
I think this change was introduced by #3247. Should there be some default tags if nothing is explicitly requested?
Version
v2.2.1

Hi folks, we are also observing the same behavior. Not only does it not copy signatures by default, but it also skips all attestations and sboms.
And according to cosign copy -h, it should still copy everything by default if no -only= flag is set:
$ cosign copy -h
Copy the supplied container image and signatures.

Usage:
cosign copy [flags]

Examples:
  cosign copy <source image> <destination image>

  # copy a container image and its signatures
  cosign copy example.com/src:latest example.com/dest:latest

  # copy the signatures only
  cosign copy --only=sign example.com/src example.com/dest

  # copy the signatures, attestations, sbom only
  cosign copy --only=sign,att,sbom example.com/src example.com/dest

  (...)

That was unintentional to make any breaking changes, the behavior should be the same as before. I can take a look at a fix later, though feel free to propose a PR with the fix.

https://github.com/sigstore/cosign/pull/3409 for the fix


Cannot use inline delete button to delete when the records _id property is not an ObjectId

Description:
When an object's _id property is not an ObjectId (in this case it was a string), and you try and use the inline delete button, you get an error '{"message":"objectId must be an instance of ObjectId"}'.

Steps to reproduce:

Insert a new object into a collection. Instead of a Mongo ObjectId() for the _id property, use a regular string.
Do a find() query on the collection, find the object and hover over it to show the hover menu.
Click on the 'X' button to try and delete the record.

Can we assume that records will always have an _id property?

Can we assume that records will always have an _id property?

MongoDB by default will insert an _id field, but still this behavior can be changed. So we cannot assume this.
So, if Mongotron finds an _id it can proceed with the deletion. Otherwise, it should deny it because it doesn't have a unique identifier of the document it tries to delete and that may lead to deleting something that the user wasn't intent to.
MongoDB also supports _id fields that are not instance of ObjectID. In that case, it should do the deletion regularly since the _id index is unique even if it doesn't contain the default ObjectID instances.

thanks @kbariotis. Just looked at the Mongo docs and looks like all docs must have a unique _id.
https://docs.mongodb.org/manual/reference/glossary/#term-id
So I just got rid of the check that it's an ObjectId. Commit is here:
https://github.com/officert/mongotron/commit/1965f46bfaac768d3291f10ff1ec07692c5c730e


fix compile warnings for Elixir 1.4

There are a couple Erlang 19 warnings as well, but I was less sure about changing them:
warning: crypto:rand_bytes/1 is deprecated and will be removed in a future release; use crypto:strong_rand_bytes/1
  lib/verk.ex:92

warning: random:uniform/1: the 'random' module is deprecated; use the 'rand' module instead
  lib/verk/retry_set.ex:38

Coverage remained the same at 83.709% when pulling a8e20677241f524f5cfabd977c815eea19bad6d9 on mitchellhenke:1.4-warnings into 78264ed730d53e4d5d5cc869ce48feeefc95ef80 on edgurgel:master.

Thank you! I will have a look at them soon. I will also add erlang 19 to .travis.yml. Cheers!


can you make a explain for isShared and maxWaiting in readme

i'm confused, can you make a explain for isShared and maxWaiting in readme ?

The isShared property was asked by me in https://github.com/nlhuykhang/meteor-publish-join/issues/2 and maxWaiting is a bit more complicated.
The join will re-run every interval-time excepted for if the last publish is still ongoing. The option maxWaiting is about to consider how long the plugin should wait until it considers the last publish as stale and runs another one - if I interpret the code right.
Since the codebase isn't really big, I also encourage you to read through it and understand what's actually going on here under the hood.

nice, thank for your explains. it's very clear and helpful.


Added screenshots for some examples

Context:
Adds screenshots of examples.
I didn't manage to get the same window theme as seen in the calculator.
I took the pictures on win 10, I hope it's fine (at least it proves the lib is cross-platform :)
Fixes #318

Thank you. I think the basic ideas was to embed the images like the screenshot on the README. Would be nice if you can adjust it.

Images looks very very strange because there is not new line before and they seems to be aligned randomly

Yes, I see now.
I have checked the result in Visual Studio with a markdown plugin.
Seems like GitHub and the plugin render the markdown test differently.


Provide RxJava 2.x variant

I'm migrating my app to use RxJava 2 and it would be nice to use a variant of Whorlwind that uses RxJava 2, instead of relying on RxJava2Interop to use Whorlwind in my app.

Is there any update on this?

Change it merged. Want to figure out #16 before releasing.


Change the font of the text in submission module

Change the font of the text in the submission module from default to Arial. Submission module is given as sub.html in the html folder.

Hey, @tejas-44 I would like to work on it please assign this issue to me.

Hey, @tejas-44 I would like to work on it please assign this issue to me.

Hey if you are registered as a participant at https://csirait.in/opencode/
you must have got a mail where there is the discord invite for the OpenCode server, there you will get all the announcement.
If you have any doubts related to any issues of this project, we can chat in the text channel allotted to this project there in the discord sever.
Moreover I think the contribution phase starts from tomorrow..

I haven't registered for any type of Open source program. I just found this issue while searching.

Hey, @tejas-44 I would like to work on it please assign this issue to me.

Hey if you are registered as a participant at https://csirait.in/opencode/ you must have got a mail where there is the discord invite for the OpenCode server, there you will get all the announcement. If you have any doubts related to any issues of this project, we can chat in the text channel allotted to this project there in the discord sever. Moreover I think the contribution phase starts from tomorrow..

Hey, @tejas-44 I would like to work on it please assign this issue to me.

Hey if you are registered as a participant at https://csirait.in/opencode/ you must have got a mail where there is the discord invite for the OpenCode server, there you will get all the announcement. If you have any doubts related to any issues of this project, we can chat in the text channel allotted to this project there in the discord sever. Moreover I think the contribution phase starts from tomorrow..

I haven't registered for any type of Open source program. I just found this issue while searching.

Then why don't you try registering and then contributing to this issue? The registration is completely free and there are lot of stuff like certificates and goodies in the room for participants..!

Hey, @tejas-44 I would like to work on it please assign this issue to me.

Hey if you are registered as a participant at https://csirait.in/opencode/ you must have got a mail where there is the discord invite for the OpenCode server, there you will get all the announcement. If you have any doubts related to any issues of this project, we can chat in the text channel allotted to this project there in the discord sever. Moreover I think the contribution phase starts from tomorrow..

I haven't registered for any type of Open source program. I just found this issue while searching.

Then why don't you try registering and then contributing to this issue? The registration is completely free and there are lot of stuff like certificates and goodies in the room for participants..!

Yeah, I just registered on the website that you mention above. Thanks for the information 😄

Hey @tejas-44, Please assign me this issue under the OpenCode event.

@tejas-44, Thank you for assigning me this issue.
I have resolved the issue and sent a pull request please review and merge it and also let me know if there are any mistakes you found. 😃

@tejas-44, Thank you for assigning me this issue. I have resolved the issue and sent a pull request please review and merge it and also let me know if there are any mistakes you found. 😃

Merged your contribution, Great Work and keep contributing!


fix ctypes.wintypes import error on Linux

remove platform checking in import section
add crossplatform CHECK_CMD
some PEP8 changes with long lines

Hello! I've moved ctypes.wintypes import in init_windows_clipboard(). Now pyperclip import doesn't raise an exception on Linux. This change help remove if statements in import section with IS_WINDOWS and IS_UNIX.

This refers to #45

This looks great :+1:

Much better! :8ball:


save the model under caret

For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
Environment info
Operating System: Ubuntu 16.10
Compiler: gcc
Package used (python/R/jvm/C++): R
R version 3.3.2 (2016-10-31) -- "Sincere Pumpkin Patch"
Copyright (C) 2016 The R Foundation for Statistical Computing
Platform: x86_64-pc-linux-gnu (64-bit)
xgboost version used: 0.6-4
If installing from source, please provide

The commit hash (git rev-parse HEAD)
Logs will be helpful (If logs are large, please upload as attachment).

If you are using python package, please provide

The python version and distribution
The command to install xgboost if you are not installing from source

If you are using R package, please provide

The R sessionInfo()

R version 3.3.2 (2016-10-31)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.1 LTS
locale:
[1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8     LC_MONETARY=en_US.UTF-8
[6] LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8       LC_NAME=C                  LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] psm_0.0.1          matrixStats_0.51.0 assertthat_0.1     scales_0.4.1       ggplot2_2.2.1      digest_0.6.11      stringr_1.1.0      xgboost_0.6-4
[9] dplyr_0.5.0
loaded via a namespace (and not attached):
[1] Rcpp_0.12.8       lattice_0.20-34   plyr_1.8.4        grid_3.3.2        R6_2.2.0          gtable_0.2.0      DBI_0.5-1         magrittr_1.5
[9] stringi_1.1.2     lazyeval_0.2.0    data.table_1.10.0 Matrix_1.2-7.1    tools_3.3.2       munsell_0.4.3     colorspace_1.3-2  knitr_1.15.1
[17] tibble_1.2

The command to install xgboost if you are not installing from source

Steps to reproduce

do parameter sweeping with caret for xgboost
save the caret train object to file
load the caret train object
4, xgb.save(caret_train$finalModel)

the R session crashes while executing xgb.save
What have you tried?

the caret train object finalModel is a list containing additional fields. I removed these fields before calling xgb.save, but it still crashes

Why do I need that?
I train the model in R, and I need to load the model using the python api for predictions.

Can you provide a small runnable script for this?

I will work that out
best,
ged
On 01/10/2017 04:20 PM, Qiang Kou (KK) wrote:

Can you provide a small runnable script for this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/issues/1955#issuecomment-271740317,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACO_-qB1j7t8RniI_zu0-ed4nlha0Rb1ks5rRCBQgaJpZM4LgCre.

Here is the script.
Best,
ged
On 01/10/2017 04:20 PM, Qiang Kou (KK) wrote:

Can you provide a small runnable script for this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/issues/1955#issuecomment-271740317,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACO_-qB1j7t8RniI_zu0-ed4nlha0Rb1ks5rRCBQgaJpZM4LgCre.

Hi, if you reply this email with attachment. I am afraid I can't see it.
Best,
On 01/10/2017 08:09 PM, Georges Dupret wrote:

Here is the script.
Best,
ged
On 01/10/2017 04:20 PM, Qiang Kou (KK) wrote:

Can you provide a small runnable script for this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/issues/1955#issuecomment-271740317,
or mute the thread

https://github.com/notifications/unsubscribe-auth/ACO_-qB1j7t8RniI_zu0-ed4nlha0Rb1ks5rRCBQgaJpZM4LgCre.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/issues/1955#issuecomment-271748508,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABebVbHI5qk46i9goVUtZE3C4Wb3Banrks5rRCvkgaJpZM4LgCre.

yes, I added a R script and two some small data files to reproduce the problem. https://dl.dropboxusercontent.com/u/7001766/crash_save.tgz
Best

@gdupret  It's not specific to caret. If you save any xgboost model as an R object and then load it back, it would have neither an actual internal xgboost model in memory (only its saved raw binary dump) nor a valid handle/pointer to it. The xgb.save is currently not handling this use case properly.
As a temporary workaround, you can do
bst <- xgboost:::xgb.Booster.check(xgb_tune$finalModel, saveraw = FALSE)
xgb.save(bst, fname = "model.xgb")

indeed! Your workaround is what I needed. Thank you Vadim!


Update Docker-compose documentation

Docker image must be in lower case => https://github.com/DRuggeri/alertmanager_gotify_bridge#docker-compose
ghcr.io/druggeri/alertmanager_gotify_bridge:latest

OK, sure - this is done.

@DRuggeri please, replace the tag master with latest. master  does not exists.


Pinned all dependencies

:wave: Hello!
We’re all trying to keep our software up to date, yet stable at the same time.
This is the first in a series of automatic PRs to help you achieve this goal. It pins all of the dependencies in your package.json, so you have complete control over the exact state of your software.
From now on you’ll receive PRs whenever one of your dependencies updates – in real time. This way you get all the benefits of up-to-date dependencies, while having them pinned at the same time. This means:

No more surprises because some of your dependencies didn’t follow SemVer. Your software is always in a state you know about, regardless of when someone does $ npm install.
If you have continuous integration set up for this repo, it’ll run automatically. Ideally, it will pass and you'll stay up to date with the press of a button. If the updated dependency does break your software, it won’t affect your users, because their dependencies are still pinned to the working state.
You can immediately identify which dependency updates break your software, because each one is tested in isolation. You’ll also have a branch ready to work on, so adapting to new APIs is way faster.

Merge this PR and you’ll have up-to-date software without the headaches :sparkles:

This pull request was created by greenkeeper.io.
It keeps your software, up to date, all the time.

Tired of seeing this sponsor message? Upgrade to the supporter plan!
You'll also get your pull requests faster :zap:

Thanks @hemanth for setting it up. Will check it out soon.

This is not perfect as it still requires everyone to update and bump a release. There's lot of edge cases that might affect only a small parts of our users that can be fixed by sub-dependency.
Semver is not perfect, but this system is not better.

@SBoudrias Agree and they are working on it.


How to keep terminal running after execution?

Hi, I want the terminal to keep running after the execution is finished.
And it will be nicer to make the plugin doesn't close the old tab and create new terminal every time it runs, rather just past the file type command with absolute path and run it, and keep the terminal open which let me see the previous output and compare with the old one.
In short, I want to make it function like the one in vscode.

Of course you can achieve this, simply use my other plugin for the handling of the terminal
https://github.com/CRAG666/betterTerm.nvim
  vim.keymap.set("n", "<leader>e", function()
    require("betterTerm").send(require("code_runner.commands").get_filetype_command(), 1, false)
  end, { desc = "Excute File" })

This is just what you need, assign the combination you like the most
The ISSUE closes if this was helpful if we cannot segir iterando
@MustafaLinux thanks

Thank you, I didn't know about that.
But although it keeps the terminal running, it's still refreshing every time I run it.
Any idea how I can prevent that?

If there will be a problem

@MustafaLinux A better terminal has already been implemented natively, so this is the solution to this problem


CaSyn optimization needs to check sender CaSyn threshold too

Per @siboehm there is a bug where we skip updating on recv if sender spiked -- but also need to make sure sender didn't skip updating due to CaSyn being below threshold!

https://github.com/emer/axon/blob/master/axon/prjnparams.go#L238-L240


Fix broken external links

Follows up on #1066

Sorry for the random comments that might be less related to your main change 😅
but while you are at the area and the changes I suggested are fairly small, I thought we might as well clean it up along the way

Always down for a cleanup! I made the suggested changes and pulled the styling out to its own file.


Mesh Chunk Generation Questions

I'm essentially trying to make a minecraft clone to work off of but I've come across some confusion around mesh deletion. I have a voxel of cubes as shown in the picture and I want to delete one the of the cubes from the mesh when I mine it in the game and then update he mesh again to show the changes. Another question I have is how should I generate a chunk full of blocks? Should it be one mesh? Should each block be their own separate game object so I don't have to worry about meshes in general? But what about performance? I'd like the terrain to be fairly deep so the player can have fun exploring below the surface of the planet.
This video explains how to cut holes in the plane of a mesh. You can also google "unity mesh vertices deletion" for more results.

Marc (metavoxel)#0069 from The Programming Hangout gave a very good insight on this.
https://www.quora.com/How-are-the-worlds-in-open-world-games-created-in-terms-of-programming
https://answers.unity.com/questions/1644923/optimization-how-can-i-optimize-this-voxel-terrain.html
https://discord.gg/theprogramminghangout


User Account Deactivation

Endpoint: /api/users/deactivate
Method: POST
Description: Allows users to deactivate their accounts, ensuring their data is handled appropriately.
Authorization Required: Yes
Feedback:
Success: Returns account deactivation confirmation
Failure: "Unauthorized" or "Account not found"

Please assign me to this issue


Android Studio Flamingo 2022.2.1 corrupted - unable to reinstall

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
I have already fielded a report - twice - on both Stackoverflow - and - on the official Google issue tracker in regards to this issue but I am getting no response and nobody seems interested in solving the problem aside from some initial update with advice that didn't yield any results - at all. A faulty Kotlin update message from the application and subsequent manual installation of the former has completely messed up and corrupted the install process, making it impossible to reinstall the application itself. This has now gone on for several weeks with no solution in sight.
https://stackoverflow.com/questions/76266649/android-studio-flamingo-2022-2-1-corrupted-unable-to-reinstall/76292143#76292143
https://issuetracker.google.com/issues/283537109
This is a scandalous support from Flutter's side. I do not understand how an application, that cannot be reinstalled no matter what is done isn't rated at the highest possible level with the most serious bug rating in order to get the application to install and work again asap.
If I would depend on this application and its functioning on a professional basis, I'd go bankrupt and lose my job, clients etc. How is this still not being taken seriously after all this time???
Expected results
The app will reinstall without issues, be usable and work without further issues.
Actual results
The app still doesn't reinstall nor work.

Code sample
Code sample
[Paste your code here]

Not possible to reproduce code in this scenario!
Screenshots or Video

Screenshots / Video demonstration
[]

Logs
Logs
[Paste your logs here]

Unable to produce logs in this scenario!
Flutter Doctor output
Doctor output
[Paste your output here]

Unable to produce Flutter Doctor output in this scenario! (see earlier attached screenshot)

This is a scandalous support from Flutter's side.

Android Studio is not part of the Flutter project. (Given that you appear to have already filed your issue in the correct place, based on the link above, I'm not sure why you're going it here.)
Closing as out of scope.

Well, thanks a lot for helping out in solving this. I posted it here as Google is supporting the Flutter project and Flutter is used in conjunction with Android Studio - primarily. The Kotlin update request popped up due to me using a Flutter project within Android Studio and was requested due to updates to that very same project. I posted it here since nobody seems to take it seriously (as I already pointed out in my initial message) within the other official bug report services. Where else should I post this but here and on those services?
You could also have shot a message to the relevant Android Studio team instead of spending the same time on writing, that you got nothing to do with it and hence, get things going.


[WIP] Make typing work with numpy stubs

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue
Update typing to use numpy stubs. So far, np.ndarray were considered Any

How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CONTRIBUTING).
[ ] All tests passed, and additional code has been covered with new tests.

To do:

solve tests
solve mypy strict mode
add numpy stubs in CircleCI pipeline


Upgrade TypeScript to 3.9.2

Blocked on https://github.com/DefinitelyTyped/DefinitelyTyped/issues/33015

Love using the latest TS!

Closing due to inactivity.


bash to sh

The docker image has no bash. Use sh instead.

Please target branches at master

In master we handle this by adding bash to the actions container
https://github.com/sifive/wit/blob/9fdc590b4af08bb5094842df868fbc8aa9391597/actions/wit/Dockerfile#L4

By the way, this change also won't work correctly on all systems, since I used bash-specific syntax (the [[ ]]) which are not part of the POSIX standard. I know for a fact that dash installed as /bin/sh on Ubuntu-based systems will not support that syntax.
If you just use the wit action targeting the latest commit on master, you should get bash installed in the Docker container and it should work around this issue.

By the way, this change also won't work correctly on all systems, since I used bash-specific syntax (the [[ ]]) which are not part of the POSIX standard. I know for a fact that dash installed as /bin/sh on Ubuntu-based systems will not support that syntax.
If you just use the wit action targeting the latest commit on master, you should get bash installed in the Docker container and it should work around this issue.

I see. Seems better way is to apk add bash


Connection drops, LMP Response Timeout

Thanks to the support for the Asus BT-400 I'm rewriting my program to keep the connection to several peripherals open.
Currently I am only using one peripheral but something strange happens. I connect to the peripheral and set a notify callback on one of the characteristics. Whenever the characteristic value is updated on the device the callback is called and I can read the new value. This works however only for about 30 seconds, then the connection with the peripheral is disconnected.
Using the debugger on the peripheral I found the reason for the disconnect is "LMP Response Timeout" I have no clue what to do with this and Google won't help me much further.
Can you tell me if this has something to do with Noble? Do I need to adjust something to keep the connection alive? Or is this a issue with the peripheral?
I pasted the relevant code (based on the Pizza example and heavily commented) for the notify and callback here: http://pastebin.com/YKx9nuHa

@PimButton please provide an HCI dump capture: sudo hcidump -t -x
Also, which kernel, linux distro, and BlueZ version do you have installed?

@sandeepmistry
I'm on windows so hcidump is not available. I just set debug=true instead. The output is here: http://pastebin.com/mCrAWrYT Most of the output is from receiving a new value (orientation reading from an accelerometer)
This seems to be the relevant piece:
  hci-usb ACL Data In: 40200b00070004001b0b004f523a34 +1s
  hci onSocketData: 0240200b00070004001b0b004f523a34 +2ms
  hci   event type = 2 +1ms
  hci           cid = 4 +1ms
  hci           handle = 64 +1ms
  hci           data = 1b0b004f523a34 +1ms
Nieuwe acc data: OR:4
  hci-usb ACL Data In: 40200b00070004001b0b004f523a36 +2s
  hci onSocketData: 0240200b00070004001b0b004f523a36 +3ms
  hci   event type = 2 +1ms
  hci           cid = 4 +1ms
  hci           handle = 64 +0ms
  hci           data = 1b0b004f523a36 +1ms
Nieuwe acc data: OR:6
  hci-usb HCI event:<PHONE_NUMBER>13 +654ms
  hci onSocketData:<PHONE_NUMBER>0013 +3ms
  hci   event type = 4 +1ms
  hci   sub event type = 5 +0ms
  hci           handle = 64 +1ms
  hci           reason = 19 +1ms
Connection disconnected

Just after the last 'New acc data: OR:6'  the connection is disconnected with handle 64 and reason 19. Does that provide some insight? I'm using Windows 7 64 bit and the Asus BT-400 dongle.

@PimButton could you please provide full debug logs?
I'm wondering if your hitting the conditions discussed in https://github.com/sandeepmistry/noble/issues/269#issuecomment-159769073.

 I screwed up using the NRF debugger and got the wrong disconnect reason. Instead of 0x22 (LMP Response Timeout) it's 0x16 which is LOCAL_HOST_TERMINATED_CONNECTION. So the issue is still there (and apparently my peripheral is the one that is disconnecting the connection) but it has nothing to do with LMP so I will close this issue and continue with the great debugging done by @george-hawkins :)


Error with the code when ran with chunk number = 9

nannyml version: 0.4.1
Python version: 3.7.12
Operating System: Linux (Kaggle Kernel)

Description
Trying to run the code in the Chunking data section. It specifically gives error when chunk number =9, but works fine for chunk number 8 and 10
What I Did
import pandas as pd
import nannyml as nml
reference, analysis, _ = nml.datasets.load_synthetic_binary_classification_dataset()
metadata = nml.extract_metadata(reference, model_type='classification_binary', exclude_columns=['identifier'])
metadata.target_column_name = 'work_home_actual'
cbpe = nml.CBPE(model_metadata=metadata, chunk_number=9, metrics=['roc_auc']).fit(reference_data=reference)
# Estimate on concatenated reference and analysis
est_perf = cbpe.estimate(pd.concat([reference, analysis]))
est_perf.data.iloc[3:5,:7]

Traceback
/opt/conda/lib/python3.7/site-packages/nannyml/chunk.py:191: UserWarning: The resulting list of chunks contains 1 underpopulated chunks. They contain too few records to be statistically robust and might negatively influence the quality of calculations. Please consider splitting your data in a different way or continue at your own risk.
  f'The resulting list of chunks contains {len(underpopulated_chunks)} underpopulated chunks. '
/opt/conda/lib/python3.7/site-packages/nannyml/chunk.py:191: UserWarning: The resulting list of chunks contains 1 underpopulated chunks. They contain too few records to be statistically robust and might negatively influence the quality of calculations. Please consider splitting your data in a different way or continue at your own risk.
  f'The resulting list of chunks contains {len(underpopulated_chunks)} underpopulated chunks. '
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
/tmp/ipykernel_32/3099037028.py in <module>
      6 cbpe = nml.CBPE(model_metadata=metadata, chunk_number=9, metrics=['roc_auc']).fit(reference_data=reference)
      7 # Estimate on concatenated reference and analysis
----> 8 est_perf = cbpe.estimate(pd.concat([reference, analysis]))
      9 est_perf.data.iloc[3:5,:7]

/opt/conda/lib/python3.7/site-packages/nannyml/performance_estimation/confidence_based/_cbpe_binary_classification.py in estimate(self, data)
    205                     **self._estimate(chunk),
    206                 }
--> 207                 for chunk in chunks
    208             ]
    209         )

/opt/conda/lib/python3.7/site-packages/nannyml/performance_estimation/confidence_based/_cbpe_binary_classification.py in <listcomp>(.0)
    205                     **self._estimate(chunk),
    206                 }
--> 207                 for chunk in chunks
    208             ]
    209         )

/opt/conda/lib/python3.7/site-packages/nannyml/performance_estimation/confidence_based/_cbpe_binary_classification.py in _estimate(self, chunk)
    215         estimates: Dict[str, Any] = {}
    216         for metric in self.metrics:
--> 217             estimated_metric = _estimate_metric(data=chunk.data, metric=metric)
    218             estimates[f'confidence_{metric}'] = self._confidence_deviations[metric]
    219             estimates[f'realized_{metric}'] = _calculate_realized_performance(chunk, metric)

/opt/conda/lib/python3.7/site-packages/nannyml/performance_estimation/confidence_based/_cbpe_binary_classification.py in _estimate_metric(data, metric)
    248 def _estimate_metric(data: pd.DataFrame, metric: str) -> float:
    249     if metric == 'roc_auc':
--> 250         return _estimate_roc_auc(data[NML_METADATA_PREDICTED_PROBABILITY_COLUMN_NAME])
    251     elif metric == 'f1':
    252         return _estimate_f1(

/opt/conda/lib/python3.7/site-packages/nannyml/performance_estimation/confidence_based/_cbpe_binary_classification.py in _estimate_roc_auc(y_pred_proba)
    301     tpr = TP / (TP + FN)
    302     fpr = FP / (FP + TN)
--> 303     metric = auc(fpr, tpr)
    304     return metric
    305

/opt/conda/lib/python3.7/site-packages/sklearn/metrics/_ranking.py in auc(x, y)
     89         raise ValueError(
     90             "At least 2 points are needed to compute area under curve, but x.shape = %s"
---> 91             % x.shape
     92         )
     93

ValueError: At least 2 points are needed to compute area under curve, but x.shape = 1

Hey Kishan,
sorry for being somewhat slow to reply.
What went wrong
This exception is due to the "chunking" of the given datasets. The size of the combined reference and analysis data is 100000 records and the desired chunk number is 9. This leads to 9 chunks of size 100000/9 or 11111, but there is still one leftover record that will currently be placed in a tenth chunk. When trying to calculate auc on that last chunk containing the single record, the exception is raised. You can check the relevant chunking implementation here.
When using a different chunk size we still have a 'leftovers chunk', but it won't have just a single record.
When using a different metric with chunk size 9 this also doesn't occur.
How to work around it for now

Don't use chunk number 9 or metric roc_auc :smile:
Drop the last record before running the estimate function

How to solve it

Add some clearer exception logging when this specific issue occurs
Review the "leftover" chunk behavior and provide alternatives ways of dealing with them:

drop any leftover records (in worst case losing (total_number_of_records / chunk_count) - 1 records)
append leftover records to the last complete chunk

I'm not sure if both make sense, so I'll run this question on the community Slack.
What are your thoughts @NeoKish ?

Issue no longer applies as of version 0.5.3
Updated code used to check:
import pandas as pd
import nannyml as nml
reference, analysis, _ = nml.datasets.load_synthetic_binary_classification_dataset()

# initialize, specify required data columns, fit estimator and estimate
estimator = nml.CBPE(
   y_pred_proba='y_pred_proba',
   y_pred='y_pred',
   y_true='work_home_actual',
   timestamp_column_name='timestamp',
   metrics=['roc_auc'],
   chunk_number=9
)
estimator = estimator.fit(reference)
estimated_performance = estimator.estimate(analysis)

estimated_performance.data
estimator.previous_reference_results


Make the number of containers configurable for both from-ambari and from-source

For from-source, the number of containers has already been configurable.


Exit Qualification Optimizations

The exit qualification fields need to be parsed as fast as
possible, and reading the VMCS on each sub-field is not
a good idea, so this patch provides the ability to
read the sub-fields once the fields have been read.
Signed-off-by: “Rian<EMAIL_ADDRESS>

@connojd / @brendank310 passed, ready to merge


Generate machine-readable list of tools

Would it be possible to generate a machine readable list of tools in addition to the markdown documentation ?
I help maintain a project called nixvim that maps neovim lua configurations to nix, and to support nicely this plugin we extract from the files the metadata and export it to json (exporter and metadata).
This is a bit brittle, because the tool could break at any time due to upstream changes, and as there already is a generator could it be possible to add a json (or other format) document that lists all the available tools ?

Correct me if I'm wrong, you would like a json file detailing all the linters and formatters associated to a language?
Something similar to the defaults.json file but just for all languages?

Exactly!

It could be ordered by tool, or by language, it only needs to be exhaustive & machine readable, then anything can be done with it! (by language would be the simplest)

Hmm yeah I think its a nice idea, I could just make it the same as the defaults file 🤔

@traxys let me know if this will be enough for your nix generation, or if there are some improvements needed.

I think it should be enough, I'll implement the mapping from JSON to nix and tell you how it went!

I implemented the nix generation, it worked fine!

Nice!


IndexError: list index out of range

Hi,
I just installed the pluging in Sublime 3 build 3176 and I get this error in Sublime's console every time mypy tries to run:
SublimeLinter: ERROR: Unhandled exception:
Traceback (most recent call last):
  File "/home/doronc/.config/sublime-text-3/Installed Packages/SublimeLinter.sublime-package/lint/backend.py", line 111, in execute_lint_task
    errors = linter.lint(code, view_has_changed) or []
  File "/home/doronc/.config/sublime-text-3/Installed Packages/SublimeLinter.sublime-package/lint/linter.py", line 925, in lint
    return self.parse_output(output, virtual_view)
  File "/home/doronc/.config/sublime-text-3/Installed Packages/SublimeLinter.sublime-package/lint/linter.py", line 946, in parse_output
    return self.parse_output_via_regex(output, virtual_view)
  File "/home/doronc/.config/sublime-text-3/Installed Packages/SublimeLinter.sublime-package/lint/linter.py", line 966, in parse_output_via_regex
    error = self.process_match(m, virtual_view)
  File "/home/doronc/.config/sublime-text-3/Installed Packages/SublimeLinter.sublime-package/lint/linter.py", line 1047, in process_match
    start, end = vv.full_line(m.line)
  File "/home/doronc/.config/sublime-text-3/Installed Packages/SublimeLinter.sublime-package/lint/linter.py", line 88, in full_line
    start = self._newlines[line]
IndexError: list index out of range

Closing due to lack of response. If you find a reliable way to reproduce this with the latest SublimeLinter version, let me or the SublimeLinter devs know.


Etda 1741

Upgraded to ruby2.6.6.  This fixes the issue with rails restart (shutting down the container when ran)

@whereismyjetpack  I updated the ruby version to 2.6.6 which caused the startup scripts to fail.  then, I wiped the vendor/bundler directory and now it works.  Not sure if upgrading ruby or wiping the vendor/bundle fixed the rails restart issue, but it works now so... 🤷♂ .
I also added public/packs to the dockerignore.  Its a big directory (1.6G) and I don't think this needs to pulled in on build since it is compiled by webpacker.

Will merge and deploy during maintenance window 6/3


cannot beautify php code under Yii2 frame work

Description
The results of beautification are not what I expect.
Expected Results
The beautified code should have looked like:
<INSERT CODE HERE>

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: https://gist.github.com/e4afe8b4fdb15722121cc37690928991
Checklist

[ ] I have tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[ ] I have reloaded (or restarted) Atom to ensure it is not a caching issue
[ ] Searched for existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[ ] Generated debugging information and added link for debug.md Gist to this issue

Please provide more information.  This issue can be reopened once we have the appropriate information to help you.

Please provide more information.  This issue can be reopened once we have the appropriate information to help you.


"math" blockquote type not supported

Trying to use Marker (which uses scidown) to write GitLab compatible equations, I ran into a problem. scidown and Marker expect equations to be surrounded by $$, whereas GitLab expects math after a triple backtick

.

It's also unfortunate that the same-line syntax is also incompatible, GitLab expects a dollar sign followed by a backtick to open the formula, Marker expects the formula to be surrounded by $ signs. It makes extra "math" stylised backticks appear around the formulas.


setup.py fail because of moto lib - Quick fix

Fix for the issue : Jinja2 2.7.2 is installed but Jinja2>=2.8 is required by set(['moto'])
Moto now depends on Jinja2 2.8 - Consider updating to the latest version of Jinja2 2.9?

Hi,
I was facing an issue due to a recent update of moto lib. They require Jinja2 >= 2.8 to be installed.
Since I did not wanted to risk installing a newer version, I have enforced security monkey to depend on version 0.4.25.

Thanks @AngeloCiffa
I just merged this.  I'll investigate upgrading jinja.


Fragment caching of gon

Is it possible to place this statement inside of the cache block:
- cache(key) do
- call_code_to_set_up_gon
= include_gon(camel_case: false)
- end

And then to setup the gon variables inside the cache block.
Thanks.

Thank you for sending Issue.

Is it possible to place this statement inside of the cache block:

I do not know gon works intently because I think gon is not supposed to this situation.

Thank you for sending Issue.

Is it possible to place this statement inside of the cache block:

I do not know gon works intently because I think gon is not supposed to this situation.


Cleanup

This pull request inclues a variety of cleanup tasks. See individual commits for details:

5b356c59 Cleanup trailing whitespace
17fc3b4c Add shebang line to Rakefile
5dff69a2 Run cukes by default
fc9f7890 Loosen gem dependencies
aef2968 Cleanup warnings in cukes

Erik Cleanupbot strikes again. Thanks man :)


Fix issues in top consumers dashboard

Signed-off-by: Shirly Radco<EMAIL_ADDRESS>

Hi @machadovilaca , Can you please review my updated json flie?


Make the "bug with several xxx in the last 3 weeks" emails go out weekly instead of daily

These lists don't change much from day to day, so they end up being pretty redundant. Lowering the frequency to once per week would be less spammy.

+1


refactor(ast)!: rename IdentifierReference::reference_flags field

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5028
#5027
#5026
#5025
#5024  👈
#5023
main

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

Join @overlookmotel and the rest of your teammates on  Graphite


fix colab notebook

Hi!

removed tf.enable_v2_behavior() as it no longer needed (and now it raises an AttributeError)
prettified a little (according pep8)
edited link to colab notebook from repo, instead of gdrive in README

@jacksonwb can you take a look? thanks!

@jacksonwb can you take a look? thanks!