Improving test coverage

As mentioned this:

Adds vfsStream as a require-dev to easier mock any file system interactions
Adds a specific bootstrap script for testing, currently setting UTC timezone
Adds a test for Config class
Corrects scope usage in InterfaceTest

@DannyvdSluijs please open a separate PR for the docker config if this hasn't been pushed by mistake. One PR per feature ;)
You can remove the commit from this PR by resetting the branch to HEAD^ and force-pushing to your branch at origin.

Yes this was accidental. Didn't realize an extra commit would automatically be added to the same pull request without any action from my side
Can I do multiple streams of features by means of an additional (feature) branch? And making pull requests from the feature branch into gitlist-php master branch?
Sent from my iPhone

On 19 Jul 2017, at 23:17, Alexander Haase<EMAIL_ADDRESS>wrote:
@DannyvdSluijs please open a separate PR for the docker config if this hasn't been pushed by mistake. One PR per feature ;)
You can remove the commit from this PR by resetting the branch to HEAD^ and force-pushing to your branch at origin.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Indeed. Create a separate branch foreach feature and open a PR when the feature is finished. However - if the features are related one can put them in one PR (e.g. changing from A to B in classes X,Y,Z may be in one PR instead of three - "Porting to Silex 2" and "Added nginx config" are two separate PR).

@alehaa did as you mentioned. Thanks for the comprehensive guide this made it easy.


Add Contribution Guidelines

Contribution guidelines explaining how to open issues, pull requests etc needs to be added.

Fixed in #29


os laging

almost all operating systems are lagging might be my laptop or something.

See https://github.com/copy/v86/issues?q=label:performance


3.9.0

cc @LewisJEllis @MaxBittker

@LewisJEllis – can you just peruse through and quickly sign off that the built files roughly matches what's added to the changelog? Thanks.


Adding the template for google analytics

after some research on pkgdown docs https://pkgdown.r-lib.org/articles/customise.html, I suggest we just add this to the template

Over to you @MeWu-IDM !


realtime: Bump image to 2.30.22

What kind of change does this PR introduce?
Bump image to 2.30.22

Do not merge, missing realtime authorization flag

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

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 60.163%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.9%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.03%

Covered Lines:
6337

Relevant Lines:
10533

💛  - Coveralls

ok to merge


Version should contain the commit hash that was built

For example:
$ buckaroo version
0.1.0
ccfe143ec63fafe1474148b159cb6df7eb03fd77 (dirty)

This will be very helpful when helping diagnose issues.


The stock span problem

I would like to solve  The stock span problem under LGM SOC'21.
Language: C++
Please assign this issue to me.

@Ayan-thecodeking u can choose any other language

@Ayan-thecodeking u can choose any other language

I can also do in

@Ayan-thecodeking u can choose any other language
I can also do in c language ...
Please assign me now


TFRS - Summary & Declaration page not loading for 2022 and prior compliance reports

Describe the Bug
The Summary & Declaration page is not loading for 2022 and prior compliance reports. This occurs regardless of the data entered into the report: (1) no data, (2) net debit position, or (3) net credit position. Under all scenarios, the system throws and error and fails to load the page.
Expected Behaviour
The page loads when the user clicks on the 'Summary & Declaration' tab
Actual Behaviour
The system throws an error when the user clicks on the 'Summary & Declaration' tab
Implications
Unable to submit a report
Steps To Reproduce
Steps to reproduce the behaviour:
User/Role: BCeID user

Go to a 2022 or prior compliance report in the dev-jan release
Click on the 'Summary & Declaration' tab
See error

Error message:*
Screenshot of error message in UI: Private Zenhub Image
Log of console error messages: tfrs-dev-jan.apps.silver.devops.gov.bc.ca-1696350837031.docx

Note for PO for testing: see card #2625 for additional testing that needs to be done once this bug is resolved. Could not fully test 2625 due to this bug blocking the ability to submit new reports for 2022 and prior.

@AlexZorkin  reminder to create tech debt ticket related to this.

Hey team! Please add your planning poker estimate with Zenhub @AlexZorkin @prv-proton

Please add your planning poker estimate with Zenhub @jig-patel


Added support for "TenantSpecificProperties" in Get-NavContainerAppInfo

Just as the title says.
Btw: I don't know if you prefer the notation tenantSpecificProperties  over TenantSpecificProperties. I usually use uppercase, but you mostly use camelCase, so I changed that.

I have included this in my dev branch, will be checked in today


Update README.md

Correct toml config example to avoid error "Key 'IgnoredVulns.id' has already been defined."
The existing example will produce an error if you try to use it as is to build your config file. Ran into this issue today when implementing it into a project.

/gcbrun

Thanks for the fix!


Downloading .osz files with nothing in them

There are 1582 .osz files which contain no beatmap files, maybe some files got corrupted due to storage malfunctions? (not sure tbh).
v1.0.8
Tried redownloading as well, doesn't work as of 03/18/2022.

This is very odd, the files seem fine on the server side, I will investigate, thank you for reporting

Found the issue, I saved the file size of each map beforehand, but this must have failed and set some of the file sizes to 0. I changed the code to calculate the file size each time and the files are serving properly now


Update webdriver-io docs for new api

See https://github.com/percy/percy-webdriverio/pull/19

Thanks @mfazekas!  I built on top of your docs update and merged it in this PR here: https://github.com/percy/percy-docs/pull/48
I'm not sure if there's a way for me to add commits to your PR directly and merge?  If there is I'd like to learn how to do things that way instead.

@timhaines there is a nice blog post about it
https://blog.spreedly.com/2014/06/24/merge-pull-request-considered-harmful/ it uses hub  but it's a bit complex.
I think you're PR is missing return, at return percy.finalizeBuild()


Webhook initially responds with "Sorry, there was a problem encountered with your request" - warm up problem?

I deployed a simple Python AzureFunction using an HTTP trigger, e.g.:
import azure.functions as func

def main(req: func.HttpRequest) -> func.HttpResponse:
    data = {
        "type": "message",
        "text": "hello from server"
    }

    return func.HttpResponse(
        body=json.dumps(data),
        status_code=200
    )

Next, I created an app using an outgoing webhook for a Teams channel with the Callback URL pointing to my function endpoint. Referencing my named app, I get the expected response ("hello from server"), but not initially if it hasn't been used for a while. In this case, I get an error: "Sorry, there was a problem encountered with your request".
I'm guessing I'm hitting a function "warm up" problem. Can I increase the amount of time before the webhook times out (I think it's set to 5 seconds)? Several people say using an app service plan, as opposed to a consumption plan, would side step this issue - I'm already using an app service plan, so not quite good enough. I might try using C#, instead of Python, for the function. Any other suggestions on how to work around this?

This 5 second timeout is how the service is designed (same thing goes for messaging extensions). There's no configuration or workaround, except building a service that ensures that you don't hit that 5 second load time and is "hot" all the time.


Update hello-minikube.md for M1 mac notebooks

Signed. Please check and merge

This merged PR should solve the issue without adding the warning message https://github.com/kubernetes/website/pull/37383.
Feel free to reopen if I made a mistake.
/close


Style improvements and user history bugs

Made some general style changes

Changed text color for easier reading

Black text is no longer pure black, changed it to #444
White text is no longer pure white, changed it to #efefef

Defaulted font-family to metropolis-medium

This was being used in more places than metropolis-semibold so it makes sense to default to this

User history changes

Down with tables!
Created new css class family item-list for vertical lists of items
Fixed the issue where the layout was wonky on large screens (due to table weirdness)
Fixed link issue when navigating to file page

Can we improve the view when there is no history?  "You don't have anything saved in history yet, go check out some content on LBRY!" ?

Yes good catch


[Release 1.26] Bump latest kube-router version

Backport for #9306

Validated on go mod job
https://github.com/rancher/distros-test-framework/actions/runs/7920539587


Can you add build instructions?

Hi! Sorry to bother, I know this is an unmaintained and very old repo, but do you know if it is still possible to buid the project? And if the answer is yes, how?
I've been trying to do it via command line, eclipse, javafx scenebuilder, and nothing. I just can't understand how to build the it. Can you add the requirements to the Readme? Not even full instruction, just what IDE and which JDK version is recommended.
Thanks in advanced!

Hi! Just use an actual version of Eclipse to build this project. You will also need to add javafx.media.jar (from https://openjfx.io) to the project build path.

Thanks! I could finally build it.
Downloading Openjfx and letting Eclipse to handle the paths did the trick.
It crashes when it runs because I still need to remove references to JXInput in order to make it work under linux, but the jar is being built successfully.

solved


Server side reference

I have set up the client-side successfully and the application is working perfectly fine. I need a server-side reference in order to set up the backend functionality for my application. A reference to a server-side set up will be of great help in developing my application. Any help is highly appreciated.

You can see a python reference for the crypto parts at https://github.com/eyalr0/HashomerCryptoRef/


Make deviations from spec clear

Fixed in https://github.com/LewisGaul/zig-nestedtext/commit/200490952d24600a9f905aeae3065592bc81faab


Clarify forward headers option

We read the documentation a bit to fast an got confused by --forward-headers. We read it as an option to simply forward all headers. I altered the explanation to clarify that you need to specify the individual headers.

@peteruithoven Thanks a lot 👍


fix possession in cluster-deployment instruction

Signed-off-by: Hanqiu Zhang<EMAIL_ADDRESS>

/assign @dockerymick

@hanqiuzh can you also update the weight of install cluster lifecycle so it's below the install klusterlet. Thanks.
Currently, its sitting ontop of Install cluster manager.

/assign @mikeshng

/lgtm


wrong http method for logout

in resources/js/Shared/Layout.svelte(line 62 in the current file) is the 'post'-method used for logout, but it has to be 'delete'. Please see the corresponding Laravel route.

This has been fixed in a recent update. Thanks for reporting it!


Fixes multiple flex issues

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[x] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes following issues:
Closes #12866
Closes #12814
Closes #12803
Closes #12870

Description

@HKuz  fixed all the issues you consolidated.

@ajain17 thank you - I'll review this PR today. I'm putting a 'block' label on it only because I want to run through the other "box" challenges to make sure the divs are all appropriately sized everywhere.

Sorry, I meant to unblock Ayushi's other PR 😅

@ajain17
When you have a moment, can you please update the description above with a closes keyword in front of each issue in separate lines (in place already)?
Like this:
Closes #12866
Closes #12814
Closes #12803
Closes #12870

This is the GitHub syntax for closing multiple issues, nice work BTW!

@HKuz  I modified it.

@ajain17 - apologies for all these comments, I went back and re-read through all the flexbox issues and I want to adjust a couple of the challenges a bit to better incorporate the recent feedback and discussions from today. Instead of making the comments in the diff, here they are in a list:

For Using display: flex with Two Boxes, this challenge doesn't really show anything because the property is put on two separate boxes, instead of a parent element. Here's what I propose:

Change the instructions part (after the
) that currently says "Add the CSS property display to both #box-1 and #box-2 and set each to a value of flex." to say "Add the CSS property display to #box-container and set the value to flex."
Change the seed code so it looks like this:

Change the tests so there's only one, and it checks that #box-container display is flex. (Also update the test message).

In the challenge: Using the Flex-direction Property to Make a Row, I think this is a good place to use the row-reverse value suggested in the issues, so the output does something:

At the end of the 1st paragraph, add the sentence: "Other options for flex-direction are row-reverse and column-reverse."
Change the instructions to read: "Add the CSS property flex-direction to the #box-container element, and give it a value of row-reverse."
Adjust the test to check for row-reverse (don't think you need regex anymore).
The seed code should change the width and height values for the two boxes to 50% for both.

In the challenge Using the Flex-direction Property to Make a Column change the width and height values to both use 50% for both boxes.

In the challenge Aligning Elements Using the Justify-content Property, change the width to 25% and height to 50% for both boxes.

In the challenge Using the Justify-content Property in the Tweet Embed, we have the same issue that no code from the camper passes the test, because of default values. Can you fix the test so it uses regex to check for a justify-content property on header .profile-name (it can be set to any of these five values: center, flex-start, flex-end, space-between, or space-around)

In the challenge Using the Flex-wrap Property to Wrap a Row or Column, change the #box-container height to 100%. Then change all the boxes to use width of 25% and height of 50%

Rest of the "box" challenges can use px, since their specific sizes matter for certain properties 👍
Thanks so much, let me know if anything is unclear!

@HKuz That's a lot of feedback. I'll create a PR asap. :)

Thanks @ajain17 - you can update this PR 😄 (I edited the note above with the git steps). And sorry for the new changes, some of the Flexbox issues were still under discussion when I consolidated the changes for you.

@HKuz That's completely fine. I'll do that asap.

@HKuz  updated the PR. Height 50% looked weird in some places so I changed it to 100%. Let me know if that needs to be changed. 50% looks as follows:

100% looks like this:

Thanks @ajain17 - I'll take a look within the next few hours 👍 For the box sizing, I see your point, I'm trying to address this comment that the sizing should be smaller within the phone output screen. That way it shows the original layout (all the boxes in the phone), then the change in the layout when the flexbox properties are added.

@HKuz I fixed all the CR feedback.

Thanks so much @ajain17 - this LGTM 🎉 🍰
@freeCodeCamp/moderators - I haven't done QA on PRs before, if someone else wouldn't mind sanity-checking this, would be greatly appreciated 😄.

:sparkles: :sparkles: LGTM. :sparkles: :sparkles:
Thanks!


201 can errors cli viewer

The table rendering and program loop code looks good. There's a few Demo files that shouldn't be included in the merge. Use git rm to delete them.
I noticed that information about the error messages (like freshness) is stored in the table itself. This isn't the best design. The table should only be used as an output to represent the internal state of the system. You should have an array / list or similar holding a struct for each message type that includes all relevant fields, then sort that array by freshness and display those structs in the table rows. This will remove functions like re_fresh, SortByColumn, and FindRowString which operate on the table data. After this fix is made, you should (almost) never need to call string.Atoi on any table values - this data should be stored as an integer somewhere else.


<EMAIL_ADDRESS>depends on @kubb/react

What is the type of issue?
No response
What is the issue?
Hi<EMAIL_ADDRESS>now requires @kubb/react as peerDep. Is this really neccessary? For what I can see, it's only used to re-export useSchemas, useOperation and useResolve.
I propose that it should be removed or documented.
Where did you find it?
@kubb/swagger/dist/index.cjs

Are you having issues with building/generating(it should be installed automatically)?
@kubb/react is a experimental package(only internal being used). I will first see if it can do the trick and then update the docs for https://www.kubb.dev/plugins/react.

Yarn does not install peerDependencies automatically (in constrast to npm > v7), it just prints out a warning.
(see https://docs.npmjs.com/cli/v8/configuring-npm/package-json#peerdependencies)
Running kubb without @kubb/react leads to the following crash:
 ✖ Cannot find module '@kubb/react'
Require stack:
-<EMAIL_ADDRESS>- C:\xxxx\xxxx-xxxx\packages\openapi\kubb.config.ts
-<EMAIL_ADDRESS>  Error: Cannot find module '@kubb/react'
  Require stack:
  -<EMAIL_ADDRESS>  - C:\xxxx\xxxx-xxxx\packages\openapi\kubb.config.ts
  -<EMAIL_ADDRESS>


Add Consistency in Branching Guidance

The article "How We Use Git at Microsoft" (https://www.visualstudio.com/learn/use-git-microsoft/) describes hotfixes in the Feature Flag section.  I like how this works, however it's different than what's stated above in this guide. This guide indicates you branch off of release and cherry-pick those changes back down to master. While they are pretty similar, it would be nice to see them consistent. Thanks!

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

ID: fe168fc3-2ed2-b3e6-ce30-cac721165ceb
Version Independent ID: 8d63fbd7-12c4-9a94-92f6-969e7eb41037
Content: Git branching guidance
Content Source: docs/git/concepts/git-branching-guidance.md
Service: unspecified
GitHub Login: @douge
Microsoft Alias: sdanie

@ChristopherMank Sorry for the long delay in responding, we had a backlog of git issues. I'm tracking a work item to make this update - I think I will add a short description and link from the Branching Guidance article to the How we use Gt at Microsoft article and discuss both approaches. I don't have an ETA but I'll make this change. Thank you for your feedback.

Hello, any update on this? What I read on this page is in contradiction to what I hear from Edward (in the embedded video).  I feel that point raised by @ChristopherMank still remains valid and not sure why this discussion has been marked as closed?


Ring radials and Lines are not created as geodetic lines in ArcGIS Pro

From @elinz on December 5, 2016 18:51

The radials are not created as geodetic lines.
We know that at least for 1.2 and 1.3 we can't create geodetic lines so the "Number of Radials" parameter should be hidden/disabled or removed.

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used: ArcGIS Pro 1.3.0 final
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop or mobile): Windows 8.1 Enterprise 64 bit.
Link to your project:

Copied from original issue: Esri/military-tools-for-arcgis#68

From @kgonzago on December 9, 2016 13:41
@elinz Can you test this with Pro 1.4? If issue persists, we will have to remove this functionality from the release.
fyi @lfunkhouser

From @elinz on December 9, 2016 16:8
@kgonzago I installed the latest build Military Tools for ArcGIS that i believe was built on 1.3 SDK and I get the same incorrect results.

Still an issue at Pro 1.4

@nfeuerstein @kgonzago I hid the radials label and text box on range rings and commented out the lines tab. pull request #193

@pHill5136 do we keep this one open to address geodetic lines in a future version of Pro, or close it out along with #181 (which seems a duplicate of this one). I'm for keeping this one open with milestone as Version - Deferred.

@mfunk I think keep this one open. I removed the lines tab and hid the radials dialog in range rings per this thread. So I think we will want someone to verify those fixes...

@pHill5136  I verified the other one. Lines tab is gone and Radials is removed from Rings tab.

Looking in the Pro SDK API Reference there is reference to CurveType enumeration with members "Geodesic", "GreatElliptic", "Loxodrome", and "NormalSection" in ArcGIS.Core.Geometry Namespace.
https://pro.arcgis.com/en/pro-app/sdk/api-reference/#topic7844.html
This guy is used in GeometryEngine's GeodeticDensifyByLength Method:
https://pro.arcgis.com/en/pro-app/sdk/api-reference/#topic8245.html
From the method doc it seems that it basically modifies an existing geometry as one of the four input curve types.

@kgonzago @nfeuerstein I hid fields for creating the radial lines. Noticed that since Pro and Arcmap addins share the dialog view that radials are turned off in ArcMap as well. Do we need to make separate viewmodels for rangerings?

@kgonzago @nfeuerstein Sorry I meant to say separate views for range rings?

@pHill5136 I believe you do. @jmccausland can provide more insight into this issue.

I would bind the views xaml element visible property to a property in the view model.  Then in each view model since you already have one for each product, set those visibility properties in the view models constructor.  I believe we are already doing this somewhere.

ProLineViewModel.cs has an example
public System.Windows.Visibility LineTypeComboVisibility         {             get { return System.Windows.Visibility.Visible; }         }

@nfeuerstein @topowright @elinz @kgonzago @mfunk Fixed geodesic lines and radials in Pro. Should show up in next build...

Build is kicking off right now,  should see the changes in a few minutes.

Going to test now...

So here is our Lines (Geodesic) and Range Ring (with Radials) in Pro with a Mercator projection (no, not Web Mercator). What I am expecting to see is Lines that bend (Geodesic) and Radials that are straight (they define a bearing and are therefore loxodromes). Loxodromes are straight in a Mercator projection.

WOOHOO!
Now if we use WGS 1984....

We get some weirdness going across the dateline.
In another projected ... Robinson (think National Geographic's world maps):

Back to another GCS, this time GRS 1980:

So I don't think this is a problem with our tool, but in how line geometry is handled crossing the dateline in a Geographic Coordinate System. I'm going to close this issue as fixed, and log a new one to investigate this new issue further.

The dateline issue was fixed, but the original problem where radials are not being created as geodetic was not fixed. The dateline issue should have been created as a separate issue. Reopening issue.

Radials are created as Loxodromes so are straight in mercator projections. Should see distortion in non mercator projections....

I guess the question is whether radials should be loxodromes or geodetic lines?

NOT Geodetic. They are intentionally created as Loxodromes which show lines of constant bearing. The whole idea of the radials is that they show a direction from the center to it's outside ring. Instead of defining what bearing to draw for each radial we simply take the number the user wants to draw, divide by 360 to find the interval, then start the first one at 0 degrees north, etc.

lines should be straight in mercator projections. They are loxodromes (constant bearing).

after reading the comments on this, this issue should be closed. The issue is actually opposite of what what originally logged. Opening a new issue to reflect what was originally logged.


HPA issue

Describe the bug
We have two installations for GitHub actions with HPA. When the HPA event triggers repo 1, sometimes it doesn't scale pod. Sometimes if it scales then some other actions pick it and run, But the PR webhook event which triggered the HPA event doesn't get any pod. Please help with this. How we can make sure we autoscale +1 on every queue event and scale down -1 on every completed event. How we can make sure if Repo 1 Webhook triggered event it should run repo 1 action only.
Checks

[ ] v0.20.3

To Reproduce
Steps to reproduce the behavior:

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

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

Controller Version v0.20.3
Deployment Method [e.g. Helm and Kustomize ] Helm
Helm Chart Version [e.g. 0.11.0, if applicable] 0.14.0

Additional context
Add any other context about the problem here.
apiVersion: actions.summerwind.dev/v1alpha1
kind: HorizontalRunnerAutoscaler
metadata:
name:  "{{ .Values.runner.name }}-ci"
namespace: {{ .Release.Namespace }}
spec:
scaleDownDelaySecondsAfterScaleOut: 5
minReplicas: 0
maxReplicas: 100
scaleTargetRef:
name: "{{ .Values.runner.name }}-ci"
scaleUpTriggers:

githubEvent: {}
duration: "20m"

@ajaychoudhary-hotstar Hey. Could you provide full details?

Number of repositories, Names of those repositories
All the RunnerDeployments
All the HRAs

@mumoshu We have around 25+ repos, difficult to share names here,
Two runners deployments
Two HPA Deployments
apiVersion: actions.summerwind.dev/v1alpha1
kind: RunnerDeployment
metadata:
name: "{{ .Values.runner.name }}-ci"
namespace: {{ .Release.Namespace }}
labels:
team: {{ .Values.organization.teamName }}
spec:
template:
metadata:
annotations:
iam.amazonaws.com/role: {{ .Values.iam_role_arn }}
labels:
team: {{ .Values.organization.teamName }}
spec:
terminationGracePeriodSeconds: 120
serviceAccountName: {{ .Values.serviceAccount.name }}
image: "{{ .Values.image.repository }}"
env:
- name: RUNNER_FEATURE_FLAG_EPHEMERAL
value: "true"
resources:
limits:
ephemeral-storage:  {{ .Values.resources.medium.limits.ephemeral_storage }}
requests:
cpu: {{ .Values.resources.medium.requests.cpu }}
memory: {{ .Values.resources.medium.requests.memory }}
ephemeral-storage:  {{ .Values.resources.medium.requests.ephemeral_storage }}
tolerations:
- key: {{ .Values.tolerations.key }}
operator: {{ .Values.tolerations.operator }}
value: {{ .Values.tolerations.value }}
effect: {{ .Values.tolerations.effect }}
dockerdWithinRunnerContainer: true
organization: {{ .Values.organization.organizationName }}
labels:
- {{ .Values.runner.runner_label_ci }}
group: {{ .Values.organization.Group }}
 HPA:

 apiVersion: actions.summerwind.dev/v1alpha1

kind: HorizontalRunnerAutoscaler
metadata:
name: "{{ .Values.runner.name }}-ci"
namespace: {{ .Release.Namespace }}
spec:
scaleDownDelaySecondsAfterScaleOut: 5
minReplicas: 0
maxReplicas: 100
scaleTargetRef:
name: "{{ .Values.runner.name }}-ci"
scaleUpTriggers:
githubEvent: {}
duration: "20m"
Second deployment
apiVersion: actions.summerwind.dev/v1alpha1
kind: RunnerDeployment
metadata:
name: {{ .Values.runner.name }}
namespace: {{ .Release.Namespace }}
labels:
team: {{ .Values.organization.teamName }}
spec:
template:
metadata:
annotations:
iam.amazonaws.com/role: {{ .Values.iam_role_arn }}
labels:
team: {{ .Values.organization.teamName }}
spec:
terminationGracePeriodSeconds: 120
serviceAccountName: {{ .Values.serviceAccount.name }}
image: "{{ .Values.image.repository }}"
env:
- name: RUNNER_FEATURE_FLAG_EPHEMERAL
value: "true"
resources:
limits:
cpu: {{ .Values.resources.small.limits.cpu }}
memory: {{ .Values.resources.small.limits.memory }}
ephemeral-storage:  {{ .Values.resources.small.limits.ephemeral_storage }}
requests:
cpu: {{ .Values.resources.small.requests.cpu }}
memory: {{ .Values.resources.small.requests.memory }}
ephemeral-storage:  {{ .Values.resources.small.requests.ephemeral_storage }}
tolerations:
- key: {{ .Values.tolerations.key }}
operator: {{ .Values.tolerations.operator }}
value: {{ .Values.tolerations.value }}
effect: {{ .Values.tolerations.effect }}
dockerdWithinRunnerContainer: true
organization: {{ .Values.organization.organizationName }}
labels:
- {{ .Values.runner.runner_label }}
group: {{ .Values.organization.Group }}
 HPA:
 apiVersion: actions.summerwind.dev/v1alpha1

kind: HorizontalRunnerAutoscaler
metadata:
name: "{{ .Values.runner.name }}"
namespace: {{ .Release.Namespace }}
spec:
scaleDownDelaySecondsAfterScaleOut: 120
minReplicas: 0
maxReplicas: 100
scaleTargetRef:
name: {{ .Values.runner.name }}
scaleUpTriggers:

githubEvent: {}
duration: "20m"

We have around 25+ repos, difficult to share names here,
Two runners deployments
...
organization: {{ .Values.organization.organizationName }}
...
labels:

{{ .Values.runner.runner_label }}
...
labels:
{{ .Values.runner.runner_label_ci }}

ARC seems to be doing what you asked.
Assuming all your RunnerDeployment specs and HRA specs are configured as you intended,

sometimes it doesn't scale pod. Sometimes if it scales then some other actions pick it and run, But the PR webhook event which triggered the HPA event doesn't get any pod

You might have misunderstood how ARC works.
ARC has no way to "force assignment" of a pending workflow job to a specific (newly created) runner.
ARC adds runners as requested, GitHub assigns runners (like randomly) to queued workflow jobs.
I see duration: "20m", which means you want to keep the added runner replica only for 20 minutes.
If your workflow job occasionally takes more time than 20 minutes it's problematic- try longer duration so that ARC could keep the added replicas until it finishes.
Please read https://github.com/actions-runner-controller/actions-runner-controller/issues/1044#issuecomment-1009741357.
We're living in an eventually consistent world so ARC doesn't rely on the quite untrue fact that webhook delivery is 100% guaranteed.

@mumoshu We set that to 60 min as well but didn't help. How do we make sure we scale on every queue event and scale down on every completed event?

we scale on every queue event and scale down on every completed event?

@ajaychoudhary-hotstar 60 mins should be enough to make it work like that, assuming your workflow job never takes more than 60 minuets in practice.

@mumoshu Job takes a max of 5-6 minutes. Even after making it 60 min, we are facing the same issue.

@ajaychoudhary-hotstar Well, ARC doesn't match runner to workflow job one by one. That's impossible to be done perfectly due to the lack of GitHub APIs for that. What we do instead is to try to calculate the desired number of runners per enteprise/organization/repo depending on the recently received webhook events.
So

When the HPA event triggers repo 1, sometimes it doesn't scale pod.

This is completely valid when e.g. one replica added via a queued workflow_job webhook event while one replica removed due to a capacity reservation expiration at the same time.

Sometimes if it scales then some other actions pick it and run, But the PR webhook event which triggered the HPA event doesn't get any pod

This might be caused by your misunderstanding. ARC should provide approximately enough runners that are demanded at that time. A newly added runner won't be 100% assigned to the latest workflow_job(because that's uncontrollable). It can still be assigned to another, already created runner though.

That said, you'll be able to monitor if your autoscaling setup is working correctly only by tracking the time each workflow_job takes to transition from queued to in_progerss.

we scale on every queue event and scale down on every completed event?

Again, this isn't what you should ask for. The current GitHub Actions API doesn't allow us to implement that correctly.
But ARC's autoscaling does provide the ability to retain the number of desired runners demanded at the time, so your workflow job will get scheduled quickly enough as long as you've configured ARC correctly.

But ARC's autoscaling does provide the ability to retain the number of desired runners demanded at the time, so your workflow job will get scheduled quickly enough as long as you've configured ARC correctly.

How we can configure ARC correctly. Could you please share that? Or current setup looks fine?

@ajaychoudhary-hotstar Yeah, your configuration looks fine alone. In other words, I can't say if your configuration is correct without seeing all your organization, repositories, and workflow definitions.
If it doesn't work as you might have expected, you either misunderstood how ARC works, or have misconfiguration that I can't see from the manifests alone.
If you really see ARC scaling the wrong set of runners, please double-check if you correctly set runner labels on RunnerDeployment and also you have listed correct labels under on of your workflow definitions.
Sorry but I'm closing this, as it doesn't look like a bug even though you used the format of a bug report.
But please feel free to file a bug report if you could come up with a concrete reproduction procedure.

BTW the right place to have community support would be to use GitHub Discussions!


Run test suites and stress test in a debugger

We currently sometimes get segfaults and other crashes when running
our test suites and stress tests via CI. But we don't get any useful
information when these crashes occur.
This commit adds gdb to most of our docker containers and uses either
gdb or lldb to run the test suites and stress tests in CI with
appropriate arguments to print out backtraces, locals, arguments, and
registers prior to quitting with error. While the backtraces aren't
always useful (due to optimizations or lack of symbols), they are
better than nothing.

Before this is merged, all the images will need to be updated in the ponylang repo and pushed and then the values in this PR updated.

I take it that lldb was already installed on the Mac and FreeBSD instances?

I take it that lldb was already installed on the Mac and FreeBSD instances?

i believe clang/lldb/llvm stuff is the default toolchain on those two..

rebased to pick up windows dockerfile/CI image changes

@SeanTAllen not sure why the x86-64 linux musl CI is hanging.. i don't believe they had the same behavior when using gdb.. do you want to switch back to gdb for it or something else?

Is this expected @dipinhora? Its from the x86 musl run. I don't have time at the moment to see if it is happening in others as well.

@dipinhora i restarted the failed musl job. I think it would be worth seeing if gdb has the same issue.

The restarted run is now hung at the same point, but in the debug rather than release build.

Is this expected @dipinhora? Its from the x86 musl run. I don't have time at the moment to see if it is happening in others as well.

Yes, this is expected. lldb cannot set signal handling logic without first starting the process so all of that breakpoint main stuff is to be able to set the signal handling passthrough logic needed to run the apps through lldb while ensuring lldb passes through signals to the application properly.

@dipinhora so i would say, lets see what happens with that image and using gdb

@SeanTAllen Worked fine with gdb..
The code is currently using the dipinhora/ponyc-ci-x86-64-unknown-linux-musl-builder:20220806 image that has gdb installed but it should probably be updated to point to an image in the ponylang docker account..
Let me know once you can build/push an official ponylang image (or copy dipinhora/ponyc-ci-x86-64-unknown-linux-musl-builder:20220806 to the ponylang docker account) and i'll update this PR accordingly..

@dipinhora ponylang/ponyc-ci-x86-64-unknown-linux-musl-builder:20220807 has been pushed

thanks.. updated..

@dipinhora can you open an issue for investigating the LLDB x86 musl problem?

https://github.com/ponylang/ponyc/issues/4176 opened


add zoom_use_ctrl_key editor option

editor.zoom_use_ctrl_key = false will remove the need of pressing Ctrl Key for zooming.

Hello,
Thanks for the PR.
I study the case. Since currently no key can be set.
Also this can be done in a simple way, adding before the start function.
    editor.zoom_enter = function(event, delta) {
        event.preventDefault()
        if(event.deltaY > 0) {
            // Zoom Out
            this.zoom_out();
        } else {
            // Zoom In
            this.zoom_in();
        }
    }


always allow trying mosek_cli

Closes #963
Addresses debugging flow of #959

@whoburg, ready for review.

Failing only because of MOSEK 8. @whoburg this is ready for review.

Still not the right answer. You now find the 32 bit mosek_cli on WIndows 7, but gpkit will fail if it tries to use it. You don't see a failure at the moment because I'm forcing only the cvxopt solver.

@bqpd, waiting to hear a response to @galbramc's concern above before reviewing.
Also, what is the status of Mosek 8 support?

re: MOSEK 8, not yet begun.
re: finding 32bit MOSEK, I'm a little confused why a 32bit MOSEK is in the 64bit Program Files folder on a 64bit operating system... For now, my priority is fixing observed problems from users; making sure that MOSEK 32bit has a friendly error is next, along with testing the other Windows install bugs.

Oops, I had it backwards. Windows 7 has a 32 bit version of python using pythonxy, but mosek is 64bit. That combination does not work. I never upgraded windows 7 to us anaconda after we realized that you could install cvxopt with anaconda.

oh hmm, so it's py32 sys.exec not working with mosek64. updated the issue accordingly...

Yeah I don't know how the mosek_cli works, but from what I recall the mixture of bits don't play nice with each other. Let me turn of the forcing of cvxopt and see what happens.

test this please

Not the error that I recalled, but this is what you get now:
https://acdl.mit.edu/csi/job/gpkit_PullRequest_Unit/buildnode=windows7x64,optimizer=cvxopt/1744/console

Hmm! It looks like it successfully ran the mosek_cli command? But did not like mosek, and so did not set the path...that's a bug. I'll push a fix up.

Looks like it's using mosek_cli just fine!

Apparently. I don't remember what the problem was, but ill update jenkins
to run mosek_cli on Windows 7.
On Thu, 01 Dec 2016, 21:42 bqpd<EMAIL_ADDRESS>wrote:

Looks like it's using mosek_cli just fine!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hoburg/gpkit/pull/966#issuecomment-264358944, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFmotN_dzKItoRFlNlD6ZlJWZYjTg6Wtks5rD4WggaJpZM4LAzHy
.

Test this please

@whoburg, may we merge?

oops, @galbramc, I think it's still finding MOSEK 8

@whoburg the code is ready for review, though tests are pending.

retest this please

Hold on, I'm updating the testing script.

test this please

code is approved but I don't want us to merge until two conditions are true:

all three jobs on the github home page are showing green
all tests pass on this PR

retest this please

(@galbramc, on unit tests but not dependencies still finding mosek8 on win10)

I just reinstalled mosek 7.

test this please

@bqpd, looks good to merge.

It is done!


Add treatLocalhostLike

Make sure the Portal-App only gets an app token (stas_*) from IAM – not a product token (st_*) (caused by the X-Treat-Localhost-Like header of the UI's webpack proxy).
This fixes a deviation from the production env, for a local portal-app running in a local UI.
Outcome of https://infopark.slack.com/archives/G011EJEGKD5/p1721287390884569

Should be fixed on IAM side.


removed println upon Unit creation from_systemctl

This PR has formatting changes and I removed a println statement upon creating a Unit via from_systemctl

Thank you for your contribution,
I just injected a cargo fmt into this crate, and the CI job will now verify contributions follow the standards.
I will provide a new tag

Merci!


Transition from experimental to qa for replatform v1

Summary
Transition away from experimental for app name, app assets, bundle id
Changelog
[General] [Change] - Transition from experimental to qa for replatform v1
Test Plan
Verify app name, icons, splash screen, CI/CD

Codemagic workflow issue addressed. Merging and resolving any further build issues here.


"API_ERROR: {"message":"404 File Not Found"}"

Hi, I'm using netlify-cms since a longtime with no major problems or bugs. But since today, I'm confronted with this message:
"API_ERROR: {"message":"404 File Not Found"}"
Only the last five posts are shown in the Content Manager.
I did also an update to the last versions but that didn't resolve the problem:
    "gatsby-plugin-netlify-cms": "^3.0.10",
    "netlify-cms": "^2.3.3",

I'm using Gitlab.
In the console, I get these three errors:
GET https://www.mywebsite.org/.netlify/git/gitlab/repository/files/content%2Fabout/raw?ref=master&ts=1548432832213 404
netlify-cms.js:14
Error: {"message":"404 File Not Found"}
    at new t (netlify-cms.js:14)
    at new o (netlify-cms.js:14)
    at Ze.z [as parseResponse] (netlify-cms.js:54)

netlify-cms.js:14
Uncaught (in promise) {type: "ENTRIES_FAILURE", error: "Failed to load entries", payload: "API_ERROR: {"message":"404 File Not Found"}", meta: {…}}
/admin/#/collections/post:1
Is anyone experiencing the same issue?

In my case deleting sub-folders inside the content solves the issue!


Update gantt.md

Oh,  I saw two diagram definitions.I thought the second one was a duplication. Sorry about that. I will revert back the change

发件人: Nacho<EMAIL_ADDRESS>发送时间: 2019年9月27日 10:03
收件人: knsv/mermaid<EMAIL_ADDRESS>抄送: lilisha100<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>主题: Re: [knsv/mermaid] Update gantt.md (#960)
@Dunning-Kruger commented on this pull request.

In docs/gantt.mdhttps://github.com/knsv/mermaid/pull/960#discussion_r329166810:

@@ -15,18 +15,6 @@ gantt
Task in sec      :2014-01-12  , 12d
another task      : 24d

-```mermaid

Hey, here you deleted the sample rendered graph of the syntax shown above. Is that for any particular reason?

―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/knsv/mermaid/pull/960?email_source=notifications&email_token=ADA6QOLHDN2WQTSZ7ARY5LTQLY4G3A5CNFSM4I3IQATKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCGGPOPQ#pullrequestreview-294451006>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ADA6QONM7MSSS3476Y4UWDLQLY4G3ANCNFSM4I3IQATA>.

Pull Request Test Coverage Report for Build 1005

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

Totals

Change from base Build 1002:
0.0%

Covered Lines:
2644

Relevant Lines:
4892

💛  - Coveralls


/etcd/etcd-live*: deprecate etcd2 and etcd

Ref: https://github.com/coreos/bugs/issues/1877
Changed to etcd2 to etcd-member.service
The first set of doc changes (missed to create a branch before pushing :-( )

closing this to create a separate branch.

https://github.com/coreos/docs/pull/1132


Use $<INSTALL_PREFIX> in target_include_directories

This is necessary to generate relocatable cmake packages

Relocatable packages are extremely important to create installer in Windows, as the installation prefix typically changes from user to user.


⚠ Samsung has degraded performance

In 9578655, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2294 ms

Resolved: Samsung performance has improved in 5565948.


Add CentOS family and fix CentOS 7 listening port check

CentOS 7 does not have installed ss (at least centos:7 Docker image) so I

add CentOS family awareness and detection
fix CentOS 7 listening port check
add tests for CentOS 7 listening port check

As far as I know, CentOS 7 minimal install has iproute package(including ss) but does not have net-tools package(including netstat).
https://gist.github.com/ozuma/552276c5d8e38b72f91d
And it seems that centos:7 Docker image I'm trying does not have both ss and netstat command.
$ docker run -ti centos:7 /bin/bash
Unable to find image 'centos:7' locally
7: Pulling from library/centos
74f0853ba93b: Pull complete
Digest: sha256:26f74cefad82967f97f3eeeef88c1b6262f9b42bc96f2ad61d6f3fdf544759b8
Status: Downloaded newer image for centos:7
[root@ed95ff48e547 /]# ss
bash: ss: command not found
[root@ed95ff48e547 /]# netstat
bash: netstat: command not found
[root@ed95ff48e547 /]# rpm -qa|grep net-tools
[root@ed95ff48e547 /]# rpm -qa|grep iproute

So it seems that this PR won't help to support centos:7 Docker image.

Hi @mizzy, thanks for your review.
I have inspected my project and found out that I install the net-tools package in my Docker image due to the missing ifconfig command. I did not realize that it also added netstat command. I will add an iproute package to my Docker image to run my tests.


changed the color of navbar and font of the title

Related Issue(s)

CLoses #73

Proposed Changes
The color of the navbar and font of the production page's title has been changed.

Type of Change

[x] Enhancement

Your Idea for Level (GSSoC)

[ ] Level 1
[ ] Level 2
[ ] Level 3

Screenshots (if applicable)

Additional Notes

Bro, do you really think this looks good? 🤔

I thought this will be better.But, if u mention what to change I will change it accordingly

Let's look into other harmonious colours in the palette and and show me.

@Arunjagan12 Stick to colour that will suit dark mode, currently these bright and blue don't LGTM.


Permission to use Crystal logo in a presentation

I'm going to be doing a presentation about mal (https://github.com/kanaka/mal) at midwest.io. Crystal is one of the implementations (by @rhysd) and I would like to use a Crystal logo in the presentation (e.g. this logo:https://github.com/manastech/crystal/blob/gh-pages/images/crystal-v.svg). I searched for guidelines related to use of the logos but didn't find anything obvious. Can I get permission to use the Crystal logo in my presentation (and the subsequent published video/materials from the conference)?

Yes, @asterite  & @waj  should publish some basic guidelines on branding usage. Maybe just a wiki page?

Yes, you can use them. And thank you for doing a presentation about
Crystal! :-)
A wiki page sounds good, but I have no experience in guidelines for
branding usage. Maybe we can reuse an existing guideline?
On Sun, Oct 25, 2015 at 4:43 PM, Jonne Haß<EMAIL_ADDRESS>wrote:

Yes, @asterite https://github.com/asterite & @waj
https://github.com/waj should publish some basic guidelines on branding
usage. Maybe just a wiki page?
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/1822#issuecomment-150962507.

--
Ary Borenszweig         Manas Technology Solutions
[ar.phone]                      5258.5240       #ARY(279)
[us.phone]<PHONE_NUMBER>    #ARY(279)
[email]<EMAIL_ADDRESS>[web]                           www.manas.com.ar

@asterite @jhass Thanks!
FYI, since I've been doing a lot of searching for logo usage guidelines (mal has 41 implementations now), I think the guidelines I like the most are for python: https://www.python.org/community/logos/ They are clear, concise, and link to preferred image sources for people to use.

@asterite I think if you just write up which usages you want to allow in a fairly informal tone and a generic "All usages not explicitly granted here need explicit permission from [...]" it's fine for now. Better have something now and change it to something more correct later if issues arise than to delay it any longer. Maybe also consider explicitly licensing it under a CreativeCommons license.

I second @jhass: add a license for the graphics - Ccreative Commons ones for instance.

It's easy to find one that match your requirements
They're juridically sound - no need to start writing your own license text full with legal loop holes

#triage
This seems to have ended with a single-case permission for use of the logo.
The project is currently licensed under Apache-2.0. I'm no lawyer, but maybe the logo is covered by this license already?

@asterite i think we should put this on wiki and close this issue. WDYT?

This is now answered in https://crystal-lang.org/media/


4.24.2024 - Ananya Jonnakuti

Hello Professor!
I can make a subdirectory--I actually initially organized everything into
python and cpp folders in each chapter, but I wasn't sure if you preferred
them the way they were so I put everything in the cpp subdirectory. Should
I reorganize the code?
Best,
Ananya Jonnakuti
On Fri, Apr 26, 2024 at 12:24 PM Phillip G. Bradford <
@.***> wrote:

Message sent from a system outside of UConn.
@.**** approved this pull request.
Looks good -
I will accept the PR
Would you make cpp (or C++) subdirectories for all the C++ code?
This looks excellent - I just think most users will focus on Python, but
having the C++ in a subdirectory will be very helpful !
—
Reply to this email directly, view it on GitHub
https://github.com/wonder-phil/ChainsThatBindUs/pull/1#pullrequestreview-2025427651,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCNRSTCJ7FNTWQ2RA6Q3WKLY7J5T3AVCNFSM6AAAAABGXFHIPWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMRVGQZDONRVGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes -this sounds good
C++ and Python directories are good


Multi-GPU training fails with CUDA illegal memory access for some examples

When running training examples in multi-GPU, I came across CUDA illegal memory access for examples with LSTM operators
Error Message
        at ai.djl.examples.training.TrainSeq2Seq.runExample(TrainSeq2Seq.java:107)
        at ai.djl.examples.training.TrainSeq2Seq.main(TrainSeq2Seq.java:64)
        Suppressed: java.lang.IllegalArgumentException: Metric name not found: step
                at ai.djl.metric.Metrics.percentile(Metrics.java:135)
                at ai.djl.training.listener.LoggingTrainingListener.onTrainingEnd(LoggingTrainingListener.java:167)
                at ai.djl.training.Trainer.lambda$close$5(Trainer.java:348)
                at java.util.ArrayList.forEach(ArrayList.java:1257)
                at ai.djl.training.Trainer.close(Trainer.java:348)
                at ai.djl.examples.training.TrainSeq2Seq.runExample(TrainSeq2Seq.java:119)
                ... 1 more
        Suppressed: ai.djl.engine.EngineException: MXNet engine call failed: cuDNN: Check failed: e == CUDNN_STATUS_SUCCESS (4 vs. 0) : CUDNN_STATUS_INTERNAL_ERROR
Stack trace:
  File "src/operator/./rnn-inl.h", line 768

                at ai.djl.mxnet.jna.JnaUtils.checkCall(JnaUtils.java:1788)
                at ai.djl.mxnet.jna.JnaUtils.waitAll(JnaUtils.java:466)
                at ai.djl.mxnet.engine.MxModel.close(MxModel.java:161)
                at ai.djl.examples.training.TrainSeq2Seq.runExample(TrainSeq2Seq.java:122)
                ... 1 more
[18:23:19] src/resource.cc:230: Ignore CUDA Error [18:23:19] /codebuild/output/src546137840/src/git-codecommit.us-west-2.amazonaws.com/v1/repos/AWS-MXNet/3rdparty/mshadow/mshadow/./tensor_gpu-inl.h:73: Check failed: e == cudaSuccess: CUDA: an illegal memory access was encountered

[18:23:19] src/resource.cc:279: Ignore CUDA Error [18:23:19] src/storage/./pooled_storage_manager.h:97: CUDA: an illegal memory access was encountered

[18:23:19] src/resource.cc:279: Ignore CUDA Error [18:23:19] src/storage/./pooled_storage_manager.h:97: CUDA: an illegal memory access was encountered

[18:23:19] src/engine/threaded_engine_perdevice.cc:275: Ignore CUDA Error [18:23:19] /codebuild/output/src546137840/src/git-codecommit.us-west-2.amazonaws.com/v1/repos/AWS-MXNet/3rdparty/mshadow/mshadow/./stream_gpu-inl.h:203: Check failed: e == cudaSuccess: CUDA: an illegal memory access was encountered

How to Reproduce?
We can reproduce by running the TrainSeq2Seq example on a machine with more than 1 GPU.
Steps to reproduce
(Paste the commands you ran that produced the error.)

./gradlew :examples:run -Dmain=ai.djl.examples.training.TrainSeq2Seq

It seems the root cause is related to NDManager.create(..., Device dev) APIs, In many places (e.g. MXNDArrayEx), the temporary NDArray is created like: array.getNDManager.create(data), it doesn't explicitly passing device to the function, it will inherit from NDManager rather then the array. In multi GPU cause, NDArray operation across difference device will cause exception.
A proper fix would be, make sure NDArray and it's NDManager's device always in sync.

I verified the crash no longer shows up. closed the issue.


Change icon color

Is there a way to change the color or the icon? Haven't found it yet.
I was trying to make it red (see code), unfortunately without any result.

Code:
type: custom:slider-button-card
entity: cover.ramen
slider:
  direction: left-right
  background: gradient
  use_state_color: true
  use_percentage_bg_opacity: false
  show_track: true
  toggle_on_click: false
  force_square: false
show_name: true
show_state: true
compact: true
icon:
  show: true
  use_state_color: true
  tap_action:
    action: more-info
  icon: mdi:curtains
  styles:
    icon:
      - color: '#FF0000'  # Red color code
action_button:
  mode: custom
  icon: mdi:curtains
  show: false
  show_spinner: true
  tap_action:
    action: toggle
name: Gordijnen

Found it:
    style: |
      :host {
        --icon-color: rgba(255, 255, 255, 1) !important; /* White color */
      }

If you want to do it based on state you can also do this @sanderhoeppermans
        style: |           :host {             --icon-color: {% if is_state('light.main_floor', 'on') %}rgba(255, 0, 0, 1){% else %}rgba(255, 255, 255, 1){% endif %} !important;           }


[Adaptive Cards] Cards not being rendered on Skype

Bot Info

Bot handle: ActmindsEducauseBot
App ID: abc985d3-cc5d-43ce-bf58-b267e80b2ee6
SDK Platform: .NET
SDK Version: 3.11
Active Channels: Facebook Messenger, Cortana Skills, Skype
Deployment Environment: Azure Web App

Issue Description
Adaptive cards are not being rendered on Skype channel.
I'm using Bot builder SDK and Adaptive Cards SKD, both .NET.
I got this message every time:
"Get the latest version of Skype to view this card "
I get redirected to the official download website of Skype.
I've already downloaded the latest version of Skype provided on the website mentioned above
Versions:
Skype desktop: <IP_ADDRESS>
Skype Win10: Last available version on Windows store
Code Example
My Card:
Body = new List()
{
                    new TextBlock()
                    {
                        Text = item.Alias,
                        Weight = TextWeight.Bolder,
                        Size = TextSize.ExtraLarge,
                    },
                    new ColumnSet()
                    {
                        Columns = new List<Column>()
                        {
                            new Column()
                            {
                                Size = "auto",
                                Items = new List<CardElement>()
                                {
                                    new Image()
                                    {
                                        Url = item.Professor.ProfilePicture.Url,
                                        Size = ImageSize.Small,
                                        Style = ImageStyle.Person,
                                        HorizontalAlignment = HorizontalAlignment.Left,
                                    }
                                }
                            },
                            new Column()
                            {
                                Size = "auto",
                                Items = new List<CardElement>()
                                {
                                    new TextBlock()
                                    {
                                        Wrap = true,
                                        Text = item.Professor.Name,
                                        Weight = TextWeight.Bolder,
                                        Size = TextSize.Large
                                    },
                                    new TextBlock()
                                    {
                                        Wrap = true,
                                        Size = TextSize.Medium,
                                        Text = item.Professor.Title,
                                        IsSubtle = true,
                                    },
                                    new TextBlock()
                                    {
                                        HorizontalAlignment = HorizontalAlignment.Right,
                                        Text = " "
                                    }
                                }
                            },
                            new Column()
                            {
                                Size = "auto",
                                Items = new List<CardElement>()
                                {
                                    new FactSet()
                                    {
                                        Facts = new List<Fact>()
                                        {
                                            new Fact()
                                            {
                                                Title = "Room",
                                                Value = item.Room
                                            },
                                            new Fact()
                                            {

                                                Title = "Date",
                                                Value = item.FinalExamDate.ToString("MM/dd/yyyy")
                                            },
                                            new Fact()
                                            {
                                                Title = "Start",
                                                Value = item.Period.Start
                                            },
                                            new Fact()
                                            {
                                                Title = "Duration",
                                                Value = item.ExamDuration
                                            },
                                        }
                                    }
                                }
                            }
                        }
                    },
                    new TextBlock()
                    {
                        Text =" "
                    }
                }

Actual Results
"Get the latest version of Skype to view this card " is shown:

Installed version:

I'm missing something?
Thanks!

I don't think this is the fix but for testing purposes you can try the windows store skype app to see if that thing is working.

@jochem4207
Thanks for the reply.
Same issue:

@BelvedereHenrique: Do AdaptiveCards work with the Cortana channel in your application?

@ViktoriaMedvedenko
No. They are not available on Cortana yet.
The only ones available are: HeroCard, ThumbnailCard, SignInCard and ImageCard.
I verify channelID before rendering the cards.

Thank you for opening an issue against the Bot Framework SDK v3. As part of the Bot Framework v4 release, we’ve moved all v3 work to a new repo located at https://github.com/microsoft/botbuilder-v3. We will continue to support and offer maintenance updates to v3 via this new repo.
From now on, https://github.com/microsoft/botbuilder repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the new Bot Framework v3 repo found here:
https://github.com/microsoft/botbuilder-v3/issues
For Azure Bot Service Channel specific defects or feature requests (e.g. Facebook, Twilio, Teams, Slack, etc.), please create a new issue in the new Bot Framework Channel repo found here:
https://github.com/microsoft/botframework-services/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


[RPC] CallTx not getting added to chain when used from JS libs

Currently am reviewing @NodeGuy 's benchmark application https://github.com/eris-ltd/benchmarks against edb-rc3. The relevant code at the JS level is here -> https://github.com/eris-ltd/benchmarks/blob/master/lib/index.js#L112-L115
After putting a bunch of tracers in the code I get output that looks like this:
Transact
G[94 95 31 217 54 225 47 60 60 172 109 236 180 139 144 74 96 149 20 22]
[]
96 96 96 64 82 96 59 128 96 16 96 0 57 96 0 243 96 96 96 64 82 96 224 96 2 10 96 0 53 4 99 96 254 71 177 129 20 96 36 87 128 99 109 76 230 60 20 96 46 87 91 0 91 96 4 53 96 0 85 96 34 86 91 96 0 84 96 96 144 129 82 96 32 144 243]
1000000
0
SignTx
I'm a call TX
PPubKeyEd25519{0208C10BA9E59522F5EF2E3C134C9BFA552478B7C6E515C6364D7AD051F977E4}
[]
96 96 96 64 82 96 59 128 96 16 96 0 57 96 0 243 96 96 96 64 82 96 224 96 2 10 96 0 53 4 99 96 254 71 177 129 20 96 36 87 128 99 109 76 230 60 20 96 46 87 91 0 91 96 4 53 96 0 85 96 34 86 91 96 0 84 96 96 144 129 82 96 32 144 243]
1000000
0
/5096D9D3C7CD.../
<CallTx{TxInput{5E5F1FD936E12F3C3CAC6DECB48B904A60951416,1,1,/5096D9D3C7CD.../,PubKeyEd25519{0208C10BA9E59522F5EF2E3C134C9BFA552478B7C6E515C6364D7AD051F977E4}} -> :<PHONE_NUMBER>603b8060106000396000f3606060405260e060020a600035046360fe47b1811460245780636d4ce63c14602e575b005b6004356000556
022565b6000546060908152602090f3}
eris-mint's BroadcastTX
<CallTx{TxInput{5E5F1FD936E12F3C3CAC6DECB48B904A60951416,1,1,/5096D9D3C7CD.../,PubKeyEd25519{0208C10BA9E59522F5EF2E3C134C9BFA552478B7C6E515C6364D7AD051F977E4}} -> :<PHONE_NUMBER>603b8060106000396000f3606060405260e060020a600035046360fe47b1811460245780636d4ce63c14602e575b005b6004356000556
022565b6000546060908152602090f3}
After go wire's WriteBinary
vmap[tx:020101145e5f1fd936e12f3c3cac6decb48b904a6095141600000000000000010101015096d9d3c7cdb310bd21a183041c72acc2ae91a0e5eb859cca0ed07973ea40e99c73f9df43d1800b184a3a00a65c28de4048c64143d4e04511c6b5548261560a010208c10ba9e59522f5ef2e3c134c9bfa552478b7c6e515c6364d7ad051f977e4000000000000
0f42400000000000000000014b6060604052603b8060106000396000f3606060405260e060020a600035046360fe47b1811460245780636d4ce63c14602e575b005b6004356000556022565b6000546060908152602090f3]
6invalid character 'x' in literal true (expecting 'r')
XError broadcasting transaction: %vinvalid character 'x' in literal true (expecting 'r')

This seems to imply that somewhere either during the signing of the TX or in go-wire's WriteBinary function that we are not signing or writing for the wire correctly. The tx getting sent is a dead simple send and I cannot find anything within the code that would be a literal true...

The issue may be that the tx argument in BroadcastTx is expected to come as []byte, not pre-encoded to hex.
Eg. in a tendermint test: https://github.com/tendermint/tendermint/blob/master/rpc/test/client_test.go#L66
(see the testTx() function which returns []byte)
Whereas erisdb is encoded to hex first: https://github.com/eris-ltd/eris-db/blob/develop/manager/eris-mint/eris-mint.go#L98
The go-rpc client takes care of encoding to hex so you don't have to: https://github.com/tendermint/go-rpc/blob/master/client/http_client.go#L176

@compleatang is this still a live issue? Can we close?
Is it actually the issue with needing to send quotation marks for hex string?

I'm not sure. I tried to get slams to work, but don't understand the setup well enough to move beyond a bunch of setup errors. @NodeGuy have you tested the slams benchmark against 0.15.6 / 0.12.1 ...?

No I haven't.  I've been focused on fixing bugs in eris-contract.js first.  It's on my to do list to test slams with 0.12.1 and to create documentation for it at the same time.

A bit remote, but this might be fixed by https://github.com/eris-ltd/eris-db/pull/453

Fixed


Configure Azure Pipelines

This configuration builds the application and produces builds for both Debug and Release configurations.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Shamaran sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


DANG-318: Fix Dropbox z-index bug using un-existing class z-100.

JIRA
https://lobsters.atlassian.net/browse/DANG-318

Description
I think Issue was that Dropbox was using a wrong class z-100 which doesn't really exist.. So switched that to z-50

Reviewer Checklist
This section is to be filled out by reviewers
Testing

[ ] This code was tested by somebody other than the developer. Do not merge until this has been done.

Good catch!

Thanks @juanfriss


Installation is broken in 0.0.2

$ mktmpenv
New python executable in tmp-ec9bbfb1a8576f4/bin/python
Installing setuptools, pip, wheel...done.
This is a temporary environment. It will be deleted when you run 'deactivate'.

$ pip install feedfinder2
Collecting feedfinder2
  Downloading feedfinder2-0.0.2.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "/tmp/pip-build-iZmsj6/feedfinder2/setup.py", line 8, in <module>
        import feedfinder2
      File "feedfinder2.py", line 9, in <module>
        import requests
    ImportError: No module named requests

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-iZmsj6/feedfinder2

The requirements are imported before setup.py processes the install_requires section, because of the import feedfinder2 line in setup.py.
The breaking change is: https://github.com/dfm/feedfinder2/commit/bf354c151eb0292b3117ac363ffe3195409475c4#diff-2eeaed663bd0d25b7e608891384b7298R8

@dfm thanks for the quick fix!

This should work now in version 0.0.3. Let me know if you have other problems.

@dfm thanks for the quick reply.
Still doesn't work unfortunately... (on 0.0.3)
$ pip install feedfinder2
Collecting feedfinder2
  Using cached feedfinder2-0.0.3.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "/tmp/pip-build-_RrGYT/feedfinder2/setup.py", line 8, in <module>
        import feedfinder2
      File "feedfinder2.py", line 17, in <module>
        import requests
    ImportError: No module named requests

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-_RrGYT/feedfinder2

Oh duh! I forgot the most important part! Give it another shot...

Works now. Thanks!

I'm having the same issue with 0.0.4
pip install feedfinder2
Looking in indexes: https://pypi.org/simple, https://pypi.ngc.nvidia.com
Collecting feedfinder2
Downloading feedfinder2-0.0.4.tar.gz (3.3 kB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [6 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "D:\tmp\pip-install-wny93au0\feedfinder2_2600a0c3a234420babca44bcb03317d9\setup.py", line 15, in
import feedfinder2
ModuleNotFoundError: No module named 'feedfinder2'
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.


add calico support for DC/OS

High-level description
Changes in this PR:

introduce Calico CNI plugin and calico-IPAM for Mesos UCR containers
add dcos-calico-node service per node as the control panel of Calico network
refactor test_networking.py to included calico container network test
add integration tests and e2e test for calico networking
add vxlan support for calico in DC/OS

exposed configurations:

Parameter
Description

calico_enabled
Indicates whether to enable Calico networking.[Default: 'true']

calico_network_cidr
Subnet allocated for calico. The subnet specified by calico_network_cidr should not overlay with those for VXLAN backends or virtual networks defined for DC/OS virtual networks. Default: '<IP_ADDRESS>/16']

calico_vxlan_enabled
Control, whether IP-in-IP or VXLAN mode is used for calico, by default IP-in-IP, is suggested to be used instead of VXLAN. calico_vxlan_enabled is supposed to set to 'true' for the environment that IP in IP is not supported, like Azure. [Default: 'true']

calico_ipinip_mtu
The MTU to set on the IPIP tunnel device. This configuration works when calico_vxlan_enabled is set to be false. Please refer to this for a suitable MTU configuration. [Default: 1440]

calico_vxlan_port
The UDP port used for calico VXLAN. This configuration works when calico_vxlan_enabled is set to be true. [Default: 64001]

calico_vxlan_vni
The virtual network ID used for calico VXLAN. This configuration works when calico_vxlan_enabled is set to be true. [Default: 4096]

calico_vxlan_mtu
The MTU to set on the VXLAN tunnel device. This configuration works when calico_vxlan_enabled is set to be true. Please refer to this for a suitable MTU configuration [Default: 1410]

Corresponding DC/OS tickets (required)

DCOS-58582 calico plugin installation for Mesos UCR.
DCOS-58583 Installation of Calico Control plane
DCOS-59113 enable l4lb  of calico network container
DCOS-58585 add e2e tests

Related tickets (optional)

DCOS-ID JIRA title / short description.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

rebased from master.

Hey folks, this conversation of PR is too lengthy and to make you better to review, I plan to close this PR and open another one, thanks for your help and understanding.


Add workarounds to support Vlocity local comp in CI

We discovered that using sf.accessToken for persistent orgs is not
compatible with the local compilation of omniscripts, which requires
this workaround. Moreover, vlocity_build does not support passing the
NPM auth token via CLI or environment variable, so we're adding support
for this to the vlocity_pack_deploy task.

@jstvz  what plan are you testing it against?

I'm finding Omniscripts to not be deployable under the api token schema

I'm finding Omniscripts to not be deployable under the api token schema

@jofsky Would you say more about what error you're seeing?


Button and Input components to accept dynamic params ex. onClick funciton

https://stackoverflow.com/questions/71404724/handle-click-with-element-in-react-typescript

https://bobbyhadz.com/blog/typescript-react-onclick-event-type

@MatteoThomas finished this


fix: bump parser-core and avro-parser deps

Description
Bump @asyncapi/parser and @asyncapi/avro-schema-parser deps to latest versions (bot didn't update it yet).

@derberg ping 😏


Wip 253 s3 credentials

Coverage report
 Click to see where and how coverage changed
FileStatementsMissingCoverageCoverage(new stmts)Lines missing

  server/lomas_server
  mongodb_admin.py

  server/lomas_server/dataset_store
  basic_dataset_store.py

  dataset_store.py

  factory.py

  lru_dataset_store.py

  server/lomas_server/private_dataset
  factory.py

  s3_dataset.py

  server/lomas_server/tests
  test_mongodb_admin.py

  server/lomas_server/utils
  collection_models.py

  config.py

Project Total

This report was generated by python-coverage-comment-action

It is nice, just why not handle metadata credentials as the file credentials ? i think all credentials should be kept safely.


Update geth and synapse in e2e image

The E2E image uses outdated versions of geth and synapse. We need to update them in order to test on an environment that is close to the one in which the LC will run.
Tasks

[ ] Update geth to Berlin-compatible version
[ ] Update synapse to the version used in the RSB

We must upgrade the Geth version to 1.10.1 or later since goerli already upgraded to on 17th of March 2021 according to this post

The RSB RC2 has upgraded synapse to v1.27.0 but we are still using  v1.19.1.

The RSB RC2 has upgraded synapse to v1.27.0 but we are still using v1.19.1.

See also https://github.com/raiden-network/raiden-service-bundle/issues/261

@palango Ok they havent released but since they upgraded to 1.29.0 we should too.

@palango the build version here shows 1.27.0 but our guide here tells us to look out for the BUILD VERSION anyways will go with latest since raiden already upgraded. Unless someone has other opinions.

@palango the build version here shows 1.27.0 but our guide here tells us to look out for the BUILD VERSION anyways will go with latest since raiden already upgraded. Unless someone has other opinions.

Sorry, I don't understand what point you're trying to make. Can you explain it differently?

The RSB BUILD_VERSIONS file has a version of synapse that is 1.27.0 but raiden as you already posted has upgraded synapse in their requirements file. and our guide for running the the E2E tests tells us to look at the BUILD_VERSIONS file of the RSB. I hope there is no conflict in that.

The RSB BUILD_VERSIONS file has a version of synapse that is 1.27.0 but raiden as you already posted has upgraded synapse in their requirements file to 1.29.0 and our guide for running the the E2E tests tells us to look at the BUILD_VERSIONS file of the RSB. I hope there is no conflict in that.

Yes, in my opinion we should update to 1.29 directly.

My findings uptil now:
Basically the issue boils down to breaking changes related to broadcast rooms,  to_device messages for services and many other breaking changes  which the python client have introduced. Its difficult to find the version of services and the RSB(for the room_ensurer.py and other setup scripts that we use) in sync with the the raiden version. Until there is some kind of uniformity in the raiden, raiden services, RSB versions that we say ok these versions of software work in sync together it would be difficult integrate the geth and synapse changes till then. Somebody who has indepth knowledge about actually what went in which commit and simulataneously in all the raiden, raiden services and RSB  could actually do this. But someone not so familiar would find it quite difficult. Also that we have breaking changes in the next branch might also be an issue here because the raiden python straightaway introduces breaking changes in their repo and we are much more careful so we need to also think which branch our e2e tests would run against.
@taleldayekh @palango @weilbith Just tagging you since you know these kind of issues much better than me. Would also like your valuable opinion here.

Wait for PC release before continuing on this.

Will be taken care of in https://github.com/raiden-network/light-client/issues/2699


Flaky tests

https://github.com/biscuit-auth/biscuit-go/runs/2529601768
https://github.com/biscuit-auth/biscuit-go/runs/2529523899
Random tests are failing intermittently like:
    datalog_test.go:581:
        	Error Trace:	datalog_test.go:581
        	Error:      	Not equal:
        	            	expected: <nil>(<nil>)
        	            	actual  : *errors.errorString(&errors.errorString{s:"datalog: world runtime limit: timeout"})
        	Test:       	TestWorldRunLimits

I also see TestGenerateWorld failing


Fixed broken link

@DaniEASmith
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@scanum Does the URL need to have localization en-us string? Wouldn't that force readers of different languages to automatically view the English version rather their localized version of the page?

@DaniEASmith, I have checked and the link does not work without the localization en-us string (I am based in Vietnam). I can only assume the URL is not localized.


wishlist: some way of adding async/await to TypeScript (preprocessor, fork of compiler, sweet.js-like thing, whatever)

I know it's on the TypeScript team's roadmap, but it's so painful that you can get async/await and all of ES6 with traceur --experimental, but there's no way (AFAICT) to leverage that while still using TypeScript.
Most of the ES6 features I can wait for TypeScript to add, but async/await is both the biggest draw (IMHO) and the one that appears the furthest out on their roadmap.
Since it looks like the compiler transform and runtime impl are pretty well known at this point, is there any way to get async/await into TypeScript?  I understand that tooling support will be extremely unlikely, but even if it's only supported by the command-line compiler, that would still be a huge win over having to keep waiting for TypeScript to get it.
Thanks!

See https://github.com/Microsoft/TypeScript/issues/1664.

Thanks, @lukehoban !!


Can it support multiple VNC connected to VM？

Is this a BUG REPORT or FEATURE REQUEST?:

/kind enhancement

What happened:Only support one VNC connection per VM.After I tried the second connection, the first connection was automatically disconnected
What you expected to happen: Multiple VNC connected to VM.

We do not support multiple VNC connections yet. Please specify a bit more on your use case please, so that we can understand the need.
/triage needs-information

Hi!
I just found this when using kubevirt, I want to connect to the same vm with a friend, can I modify this part of the code to support this.
https://github.com/kubevirt/kubevirt/blob/main/pkg/virt-handler/rest/console.go#L210-L255

If you want to work on this, feel free to go ahead. We are always looking out for additional hands on the project in several areas :)
Maybe you want to read the contribution guidelines and the getting started guides, starting from here: https://github.com/kubevirt/kubevirt/blob/4f0a9a8470d6ef2fec8e75e1d3a7adec25b2ca10/README.md#to-start-developing-kubevirt
Also, if you need any help, just contact others on either the slack channel #kubevirt-dev, mailing list or join the weekly community meeting happening every Wednesday. More details on that here: https://kubevirt.io/community/

/area virtctl

/kind enhancement

@Jchicode but still the question remains, what exactly do you want to achieve?

@dhiller I created a VM and want to share it with my friends. So want multiple vnc's to connect to it at the same time.
But only one VNC connection is supported per VM.

I haven't looked very close, but from what I understand you would need to work on the virt-handler side to make this work. I see this as a starting point to have a look:
https://github.com/kubevirt/kubevirt/blob/356ddf402a446d81b1a8c9654149907f8449f995/pkg/virt-handler/rest/console.go#L135
I am guessing that you would need to add the handling of several connections there. But the task is way over my head, so I think you should ask around in one of the channels for some help on how to get started on this.
/remove-area virtctl
/area virt-handler

/area handler

I haven't looked very close, but from what I understand you would need to work on the virt-handler side to make this work. I see this as a starting point to have a look:
https://github.com/kubevirt/kubevirt/blob/356ddf402a446d81b1a8c9654149907f8449f995/pkg/virt-handler/rest/console.go#L135
I am guessing that you would need to add the handling of several connections there. But the task is way over my head, so I think you should ask around in one of the channels for some help on how to get started on this.
/remove-area virtctl /area virt-handler

Yes, I found it from VNCHandler.
I found there is an env setting, does this setting make a difference? https://github.com/kubevirt/kubevirt/blob/main/pkg/virtctl/vnc/vnc.go#L306

@dhiller Hi! I have one more question, can I copy local text, paste into the VM via vnc? How should I implement this part.

The guide for developers new to the project starts here: https://github.com/kubevirt/kubevirt/#to-start-developing-kubevirt
After you walked through this, I think you should make yourself familiar with the architecture.
However, if you don't see yourself implementing this, you can still help improving this issue in explaining in more detail what exactly you want to be able to do. I.e. something in the lines of:
<!-- This form is for bug reports and feature requests ONLY!
Also make sure that you visit our User Guide at https://kubevirt.io/user-guide/
-->

**Is this a BUG REPORT or FEATURE REQUEST?**:

/kind enhancement

**What happened**:

I connected to a VM using VNC. After I tried the second connection, the first connection was automatically disconnected.

**What you expected to happen**:

VNC connections from multiple clients to any one VM should succeed.

And of course you can go into more detail than what I just did.

hi all, have you solve this problem?


improvement for readCSV file and code refactoring

Used fs/promises for promise-based file operations.
Improved the construction of the beginSQLInsert statement.
Extracted the validation logic into a separate function (validateRow).
Refactored file existence check into a separate function.
Added .nvmrc file

Hey could you modify your vscode settings so that no unnecessary settings like single  quotes to double or tab spaces appear? That ways, it is helpful to check exact changes🙂

Hey could you modify your vscode settings so that no unnecessary settings like single quotes to double or tab spaces appear? That ways, it is helpful to check exact changes🙂

sure will look into that:)


Cannot read property 'split' of undefined

The following test:
browser.addCommand('test', function () {
  throw {
    errorMessage: 'some error'
  };
});

describe('test', () => {
  it('It should work', () => {
    browser.test();
  });
});

Results in a Cannot read property 'split' of undefined error:
[chrome 54.0 #0a] Running: chrome (v54.0)
[chrome 54.0 #0a]
[chrome 54.0 #0a]   test
[chrome 54.0 #0a]       1) it should work
[chrome 54.0 #0a]
[chrome 54.0 #0a]
[chrome 54.0 #0a] 1 failing (5s)
[chrome 54.0 #0a]
[chrome 54.0 #0a] 1) test it should work:
[chrome 54.0 #0a] Cannot read property 'split' of undefined
[chrome 54.0 #0a] TypeError: Cannot read property 'split' of undefined
[chrome 54.0 #0a]     at Context.<anonymous> (/some/path/to/test.spec.js:7:18)

From the stacktrace, it isn't apparent where the error actually occurs.  It also swallows the actual error that was thrown.  Instead of Cannot read property 'split' of undefined, it would be helpful if the error message was the thrown { errorMessage: 'some error' } object.
(Digging deeper, it looks like the error is thrown at this line.  It is trying to parse a stacktrace that doesn't exist because an object was thrown instead of an Error.)

if the error message was the thrown { errorMessage: 'some error' } object.

Please throw the error correctly like:
browser.addCommand('test', function () {
  throw new Error('some error');
});

Then it should work.

@christian-bromann I agree that it is common practice to throw a new Error object.  However, since any expression can be thrown, I do wonder if other people will also be confused as I was with the Cannot read property 'split' of undefined error.
Here's another example of a test that throws the 'split' error:
browser.addCommand('test', function (func) {
  return func();
});

describe('test', () => {
  it('It should work', () => {
    browser.test.call(null, () => {
      return new Promise((resolve, reject) => {
        reject({});
      });
    });
  });
});

In this case, two things are required for the error to occur:

We are not rejecting a new Error()
We invoke a custom method (browser.test) via .call or .apply

For anyone who stumbles upon this issue, the workaround I found was to not call .call, .apply, or .bind on any custom method.  Additionally, reject with an object that has a stack property like new Error as @christian-bromann comments above.
If you use a custom error, make sure to have a stack:
function CustomError (){
  this.stack = ''; // This is required
};
CustomError.prototype = Object.create(Error.prototype);
CustomError.prototype.constructor = CustomError;

Or alternatively, do:
class CustomError extends Error {}

@christian-bromann did not see your commit. Nice!


Occuring error when executing ./startFabric.sh

Error description:

Error: error getting chaincode code user-chaincode: error getting chaincode package bytes: Error obtaining dependencies for github.com/oraclize/fabric-api: command <go list -f {{ join .Deps "\n"}} github.com/oraclize/fabric-api>: failed with error: "exit status 1"
can't load package: package github.com/oraclize/fabric-api: cannot find package "github.com/oraclize/fabric-api" in any of:
	/opt/go/src/github.com/oraclize/fabric-api (from $GOROOT)
	/opt/gopath/src/github.com/oraclize/fabric-api (from $GOPATH)

Environment：
ubuntu 16.04 TLS
Docker version 19.03.2, build 6a30dfc
docker-compose version 1.24.1, build 4667896b
go version go1.13.4 linux/amd64
git version 2.7.4
node version v4.2.6

go env
GOPATH="/home/ubuntu/go"
GOROOT="/usr/local/go"

And i find that fabric-api has been my gopath directory.
I'm looking forward for your suggestions.

@RookieLinLucy666 This link github.com/oraclize/fabric-api does not exist any more after our rebranding, it should be github.com/provable-things/fabric-api.
Looking forward to fix it.

Sorry, it couldn't fix this error. I altered the package in the user-chaincode.go , and reoperated the go get github.com/provable-things/fabric-api. This problem still exists.
Error: error getting chaincode code user-chaincode: error getting chaincode package bytes: Error obtaining dependencies for github.com/provable-things/fabric-api: command <go list -f {{ join .Deps "\n"}} github.com/provable-things/fabric-api>: failed with error: "exit status 1"
can't load package: package github.com/provable-things/fabric-api: cannot find package "github.com/provable-things/fabric-api" in any of:
	/opt/go/src/github.com/provable-things/fabric-api (from $GOROOT)
	/opt/gopath/src/github.com/provable-things/fabric-api (from $GOPATH)

And i found that something wronged when removing images in ./startFabric.sh. Maybe the documentary forgot to write something?
Ps: I'm sure that the Fabric Network can start successfully.
Error: No such image: dev-peer0.org1.example.com-oraclize-connector-1.0-7765c3fb5c4224a4a2784d8a64a5488e570d39940695306f78f8e54009d89102
Error: No such image: dev-peer0.org1.example.com-user-chaincode-1.0-58b4cc4747da6f30d7cb2cea6511560c9fdad78c58ba6881b33801a2d69aebae

Looking forward to your suggestion. Thank you~


Remove meta property in favor of member access

It seems that check for presence of self._meta doesn't have to be
performed every time self._meta is about to be fetched if it can be
just default constructed on Tree initialization. Avoiding this check
removes unnecessary performance penalty for using this member

It increases the memory footprint, and the performance gain should be minimal.
Did you measure it, or is this a blind optimization?

Not a blind optimization but rather low hanging fruit. I've been recently profiling my project to see what I can do better but couldn't stop my curiosity about internals of lark itself. No doubt, it's quite targetted optimization when propagate_positions is used but nevertheless in my case most of the gains aren't even observed in my code but inside PropagatePositions.__call__. Proposed change reduces time spent there by around a third which leads to around 12-15% less time spent inside lalr_parser:45:parse which I'd say is worth considering.

I would suggest modifying PropagatePositions.__call__ instead. It will have almost the same gains, but without the memory footprint increase when propagate_positions=False. (this can be a lot when the trees are huge)

Thanks for advice. I've given a try in limiting calls to Tree.meta inside PropagatePositions.__call__ instead as suggested and the gains aren't as close but not that far either. In the same profiling scenario gain is around 11-12% which sounds good enough. It seems like a fair compromise since worrying about memory footprint is certainly justified. I've amended the commit and commit message to reflect what's actually reworked.
I'm aware that propagate_positions=True might not be for the best in the first place but that doesn't mean such use-case can't be improved.

Looks good, thanks!


Generator: Update SDK /services/logme

Generated from GitHub run<PHONE_NUMBER>

Unbeautified method, we are working to fix it


Sqlite3 remove usage of "FALSE" for compatibility with older versions

I had the misfortune of running the scheduler with sqlite 3.22.0 and encountering the error:
capnp-rpc [WARNING] Uncaught exception handling Registration.register:
(Failure "Bad row from DB: [TEXT <\"FALSE\">]")

Even though the schema specifies the column as BOOLEAN, this doesn't stop sqlite from writing strings into it. Turns out the aliases for TRUE/FALSE are defined as 1/0 starting in 3.23.0. There is no easy way to setup a "strict mode" for the tables at the moment (they have a draft and the alternative is to sprinkle CHECK(typeof(column) = 'expected_type') everywhere).
Anyway since sqlite 3.23.0 was released in 2018, ... I should probably upgrade my system!

Thanks!


context.h exceptions should be documented  and caught

Describe the bug
context.h can throw exceptions, however they are not documented and not caugth, relying on main.cxx catch all to not segfault, this should be fixed.
To Reproduce
Steps to reproduce the behavior:

comment main.cxx catch all
ctest -R wgl
segfault

Hi, i would like to work on this issue, if is still relevant

It is! Do you know guidance ?

where can I get documentation?

https://f3d.app
https://discord.f3d.app

I thought it is a good issue to get started with

It is! But I wrote it a bit fast, I can add more context


BadRequest when

This is my first attempt at using ElasticMQ and I am running into an error (probably user error Not a bug).
package.json
"aws-sdk": "^2.1692.0",

Here is my code
import AWS from 'aws-sdk';

AWS.config.update({
  accessKeyId: 'notValidKey',
  secretAccessKey: 'notValidSecret',
});

const sqs = new AWS.SQS({ region: 'eu-central-1' });

const sendMessage = (message: unknown) => {
  sqs.sendMessage(
    {
      MessageBody: 'Test Message',
      QueueUrl: 'http://localhost:9324/queue/default',
    },
    function (err: Error, data: unknown) {
      if (err) {
        console.error('sendMessage error', err);
        return;
      }
      console.log('err', err, data);
    },
  );
};

docker-compose.yml
version: '3.9'

services:
  elasticmq:
    image: roribio16/alpine-sqs
    restart: always
    ports:
      - 9324:9324
      - 9325:9325

ElasticMQ startup logs
api-elasticmq-1  | 2024-11-08 04:14:09,310 INFO Included extra file "/etc/supervisor/conf.d/elasticmq.conf" during parsing
api-elasticmq-1  | 2024-11-08 04:14:09,312 INFO Included extra file "/etc/supervisor/conf.d/insight.conf" during parsing
api-elasticmq-1  | 2024-11-08 04:14:09,312 INFO Included extra file "/etc/supervisor/conf.d/sqs-init.conf" during parsing
api-elasticmq-1  | 2024-11-08 04:14:09,312 INFO Set uid to user 0 succeeded
api-elasticmq-1  | Unlinking stale socket /var/run/supervisor.sock
api-elasticmq-1  | 2024-11-08 04:14:09,697 INFO RPC interface 'supervisor' initialized
api-elasticmq-1  | 2024-11-08 04:14:09,698 CRIT Server 'unix_http_server' running without any HTTP authentication checking
api-elasticmq-1  | 2024-11-08 04:14:09,699 INFO supervisord started with pid 1
api-elasticmq-1  | 2024-11-08 04:14:10,727 INFO spawned: 'sqs-init' with pid 13
api-elasticmq-1  | 2024-11-08 04:14:10,745 INFO spawned: 'elasticmq' with pid 15
api-elasticmq-1  | 2024-11-08 04:14:10,752 INFO spawned: 'insight' with pid 17
api-elasticmq-1  | cp: can't stat '/opt/custom/*.conf': No such file or directory
api-elasticmq-1  | 2024-11-08 04:14:11,822 INFO success: sqs-init entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
api-elasticmq-1  | 2024-11-08 04:14:11,822 INFO success: elasticmq entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
api-elasticmq-1  | 2024-11-08 04:14:11,823 INFO success: insight entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
api-elasticmq-1  | 2024-11-08 04:14:11,836 INFO exited: sqs-init (exit status 0; expected)
api-elasticmq-1  |
api-elasticmq-1  | ><EMAIL_ADDRESS>start /opt/sqs-insight
api-elasticmq-1  | > node index.js
api-elasticmq-1  |
api-elasticmq-1  | 04:14:14.351 [main] INFO  org.elasticmq.server.Main$ - Starting ElasticMQ server (0.15.0) ...
api-elasticmq-1  | 04:14:17.221 [elasticmq-akka.actor.default-dispatcher-3] INFO  akka.event.slf4j.Slf4jLogger - Slf4jLogger started
api-elasticmq-1  | Loading config file from "/opt/sqs-insight/lib/../config/config_local.json"
api-elasticmq-1  | Unable to load queues for  undefined
api-elasticmq-1  | Config contains 0 queues.
api-elasticmq-1  | listening on port 9325
api-elasticmq-1  | 04:14:22.929 [elasticmq-akka.actor.default-dispatcher-4] INFO  o.e.rest.sqs.TheSQSRestServerBuilder - Started SQS rest server, bind address <IP_ADDRESS>:9324, visible server address * (depends on incoming request path)
api-elasticmq-1  | 04:14:23.342 [elasticmq-akka.actor.default-dispatcher-4] INFO  o.elasticmq.actor.QueueManagerActor - Creating queue QueueData(dlq,MillisVisibilityTimeout(10000),PT5S,PT0S,2024-11-08T04:14:23.129Z,2024-11-08T04:14:23.129Z,None,false,false,None,None,Map())
api-elasticmq-1  | 04:14:23.453 [elasticmq-akka.actor.default-dispatcher-4] INFO  o.elasticmq.actor.QueueManagerActor - Creating queue QueueData(default,MillisVisibilityTimeout(10000),PT5S,PT0S,2024-11-08T04:14:23.452Z,2024-11-08T04:14:23.452Z,Some(DeadLettersQueueData(dlq,3)),false,false,None,None,Map())
api-elasticmq-1  | 04:14:23.466 [main] INFO  org.elasticmq.server.Main$ - === ElasticMQ server (0.15.0) started in 11924 ms ===
api-elasticmq-1  | Adding consumer for default
api-elasticmq-1  | Adding consumer for dlq
api-elasticmq-1  | 04:16:15.677 [elasticmq-akka.actor.default-dispatcher-7] INFO  o.elasticmq.actor.QueueManagerActor - Creating queue QueueData(newqueue,MillisVisibilityTimeout(30000),PT0S,PT0S,2024-11-08T04:16:15.659Z,2024-11-08T04:16:15.659Z,None,false,false,None,None,Map())
api-elasticmq-1  | Adding consumer for newqueue
api-postgres-1   | 2024-11-08 04:19:08.605 UTC [28] LOG:  checkpoint starting: time
api-postgres-1   | 2024-11-08 04:19:09.278 UTC [28] LOG:  checkpoint complete: wrote 9 buffers (0.1%); 0 WAL file(s) added, 0 removed, 0 recycled; write=0.631 s, sync=0.010 s, total=0.673 s; sync files=8, longest=0.006 s, average=0.002 s; distance=7 kB, estimate=7 kB; lsn=0/7929FF8, redo lsn=0/7929FC0
api-postgres-1   | 2024-11-08 04:29:08.446 UTC [28] LOG:  checkpoint starting: time
api-postgres-1   | 2024-11-08 04:29:09.106 UTC [28] LOG:  checkpoint complete: wrote 7 buffers (0.0%); 0 WAL file(s) added, 0 removed, 0 recycled; write=0.617 s, sync=0.014 s, total=0.661 s; sync files=7, longest=0.011 s, average=0.002 s; distance=7 kB, estimate=7 kB; lsn=0/792BF28, redo lsn=0/792BEF0

I receive the following error when doing a sqs.sendMessage
sendMessage error Error [UnknownError]: Bad Request
  code: 'UnknownError',
  statusCode: 400,
  time: 2024-11-08T04:24:14.269Z,
  requestId: undefined,
  retryable: false,
  retryDelay: 94.1003141202305
}

It doesn't give any indication of what the bad request was. Is there a way I can turn logging on to determine the root cause?
Any help is appreciated!

I found the issue. The version of AWS-SDK I was using 2.1692 only works with JSON. I had to downgrade to 2.1480.0.
https://github.com/aws/aws-sdk-net/issues/3114

Working now


Update link to the Helm project

Signed-off-by: Baiju Muthukadan<EMAIL_ADDRESS>

@sftim Good suggestions! Committed your changes.

/lgtm

/approve


Change in choose a topic functionality

If a root node is selected, it's first node should be the selected topic

@aleffert pl review

Assuming recursion isn't a problem :+1:


new topic RUNNING ASP.NET CORE APPLICATIONS WITH IIS AND ANTARES (AZURE WEBSITES)

[ ] New article focused on Azure websites. Get info from RUNNING ASP.NET CORE APPLICATIONS WITH IIS AND ANTARES (AZURE WEBSITES)

https://docs.asp.net/en/latest/publishing/iis.html could also include some of these details.

@Tratcher There are a few nice troubleshooting adds for sure. I'll open an issue and make a list, including talking about the publish-iis tooling, which still isn't in that doc. Given that people assume it actually publishes apps to IIS, yeah, it probably should get a few words.