backup-operator does not use AWS credentials from EKS annotation

When Rancher Backup Operator is deployed with ServiceAccount with annotation:
serviceAccount:
  annotations:
    eks.amazonaws.com/role-arn: yourRoleArn

then operator code is not using AWS IAM permissions that should be available to that role. It fails with similar error message:
invoking set s3 service client                insecure-tls-skip-verify=false s3-accessKey= s3-bucketName=rancher-*redacted* s3-endpoint=s3.amazonaws.com s3-endpoint-ca= s3-folder=*redacted*/subfolder s3-region=eu-central-1
INFO[2023/01/19 12:52:40] invoking set s3 service client use IAM role
ERRO[2023/01/19 12:52:40] error syncing 'test': handler backups: failed to check if s3 bucket [rancher-*redacted*] exists, error: Access Denied., requeuing

When using alpine sidecar in operator pod, one can verify that:
/ # aws sts get-caller-identity

{
    "UserId": "AROAVWICU52HKI64ABCDE:botocore-session-1674132472",
    "Account": "*Redacted*",
    "Arn": "arn:aws:sts::*redacted*:assumed-role/dev-rancher-01-rancher-backup/botocore-session-1674132472"
}

and listing s3 bucket from sidecar:
/ # aws s3 ls s3://rancher-*redacted*/*redacted*/subfolder

                           PRE subfolder/
/ # aws s3 ls s3://rancher-*redacted*/*redacted*/subfolder/
2023-01-18 10:17:42          0
2023-01-18 10:18:03        982 testfile.yaml

S3 Bucket is located in another AWS Account than Rancher installation. There is bucket policy which allows ListBucket.
Conclusion is that respective Go library used does not use AWS credentials available via EKS Service Account annotation.
Setting log trace: enabled is not giving any more information.

Troubleshooted this till I found correct solution:
So, it seems there is very very much sensitivity to exact policy contents for things to work:
To make backups work cross account, on s3 bucket that is storing backups, add bucket policy:
{
  "Version": "2012-10-17",
  "Statement": [
    {
      "Sid": "",
      "Effect": "Allow",
      "Principal": "*",
      "Action": "s3:ListBucket",
      "Resource": [
        "arn:aws:s3:::yourbucket/*",
        "arn:aws:s3:::yourbucket"
      ],
      "Condition": {
        "StringLike": {
          "aws:PrincipalAccount": "AWS Account ID running Rancher"
        },
        "StringLikeIfExists": {
          "s3:prefix": [
            "",
            "AWS Account ID running Rancher/*"
          ]
        }
      }
    }
  ]
}

On AWS Account which is running Rancher in EKS, create following policy and attach it to service account IAM assumed role:
{
    "Statement": [
        {
            "Action": "s3:ListBucket",
            "Condition": {
                "StringLike": {
                    "s3:prefix": [
                        "AWS Account ID running Rancher/subfolder",
                        "AWS Account ID running Rancher/subfolder/*"
                    ]
                }
            },
            "Effect": "Allow",
            "Resource": "arn:aws:s3:::yourbucket/*",
            "Sid": ""
        },
        {
            "Action": "s3:ListBucket",
            "Effect": "Allow",
            "Resource": "arn:aws:s3:::yourbucket",
            "Sid": ""
        },
        {
            "Action": [
                "s3:PutObjectAcl",
                "s3:PutObject",
                "s3:GetObject",
                "s3:DeleteObject"
            ],
            "Effect": "Allow",
            "Resource": "arn:aws:s3:::yourbucket/AWS Account ID running Rancher/subfolder/*",
            "Sid": ""
        }
    ],
    "Version": "2012-10-17"
}

This combination finally worked. Issue was that s3:prefix condition did not grant access if Listbucket was called without prefix, so s3:prefix variable was actually missing I think, causing condition to fail.
Phew.


sendKeys() only passes partial value

After upgrading to protractor 2.0 my login tests started failing.
In my PO I use the following function to handle input values:
    function _input(parentElement, binding, value) {
        var field = parentElement.element(by.model(binding)).element(by.css("input"));
        if (value === undefined)
            return field.getAttribute('value');
        else
            return field.clear().then(function() {
                return field.sendKeys(value);
            });
    }

I found out that sendKeys only passes the first part of the value (always with a different length, sometimes one character sometimes n-1). Even if I make a conditional wait for the input to contain the whole value, the condition never gets fullfilled. It seems like sendKeys randomly stops sending keys from the value.

Can you tell us your browser version, version of webdriver, and OS?

Sure, I'm using:

current stable chrome (atm 41.0.2272.118)
windows 7
not shure which webdriver-version that is exactly, but webdriver-manager status says chromedriver is up to date

@kleinerroemer are you using a non-English keyboard or remote display or any other configuration that might confuse the keyboard mappings?  Chromedriver has had some problem configurations where it drops characters out of sendKeys.  See https://github.com/angular/protractor/issues/698 and http://stackoverflow.com/questions/18483419/selenium-sendkeys-drops-character-with-chrome-driver  and https://code.google.com/p/chromedriver/issues/detail?id=1037 (the non-English keyboard bugs seem to be fixed recently, though.)
Alternatively, the bug might be in the code after your _input method (if that is not waiting for the sendKeys to complete somehow.

my keyboard layout is set to german but switching to english layout didn't change anything. sendKeys always skips the keys at a different point. I don't use remote display.
I doubt that the bug is in my method, since it worked just fine before upgrading. My current workaround looks like this:
	function _completeInput(field, value) {
		return field.getAttribute('value').then(function(v) {
			if (v.length < value.length) {
				return field.sendKeys(value.substring(v.length, value.length)).then(function() {
					return _completeInput(field, value);
				});
			}
		});
	}

It looks like we have a similar issue here.
On the install of a colleague (windows 7, protractor 1.5, chrome 42.0.2311.90) everything is fine.
On my machine setup (ubuntu 13.10, protractor 2.0.0, chrome 42.0.2311.90 (64-bit)) the exclamation mark (!) will not be entered by sendKeys().
I did a downgrade to protractor 1.8.0, but the problem was still occurring.
I did a second downgrade to 1.5 (to match my colleague's version).
My colleague did an upgrade to protractor 2 (on his windows machine).
From what we could observe, the problem does not seems to happen on a windows machine (there's always a first time) but on ubuntu, whatever the version is, it happens all the time.
I hope it will help to spot the issue.

I am still getting this problem, particularly in chrome. It seems totally random - sometimes it works and sometimes it doesn't.
eg:
element(by.id('message')).sendKeys(Date.now());

will not output the current timestamp, but something like 1142

I experience this is well. In my case, the tested application is based on Angular 2 and it happens only with the input elements that are bound to a model [(ng-model)]. If they are not bound, everything is ok. My current workaround is (in TypeScript):
export function sendKeys(element:protractor.ElementFinder, string:string):void {
  "use strict";

  if (!string) {
    return;
  }
  let match:string[] = /(.)(.*)/.exec(string);
  element.sendKeys(match[1]);
  sendKeys(element, match[2]);
}

but it significantly slows down the tests.

I'm also experiencing this issue now.
Upgrading tests from 1.8.0 to 2.5.1. I have some tests randomly failing now that used to work. These are on a page where browser.ignoreSynchronization = true;
I will do something like:
input1.sendkeys('abc');
input2.sendkeys('def');
I will either see this:
Input1: 'ab'
Input2: 'cdef'
or
Input1: 'abcdef'
input2; ''
Using Chrome: Version 46.0.2490.86 m

@brocsk @radek-holy are either of you able to create a reproducible case we can test against using http://www.protractortest.org/testapp/?

No, sorry, I am not able to reproduce it against the Testapp. Here is the simplest reproducer I can come up with: http://plnkr.co/edit/f0LQ0zLvtr8HEodySl35 . For some reason, it's necessary to call sendKeys twice to hit the issue. I'm sorry if this is another issue (e.g. Angular 2 specific)...

I've encountered a similar issue with 48.0.2564.109 (Official Build) m (32-bit) on Windows 10 x64. I've switched to Firefox for running the tests in the meanwhile, and didn't encounter this issue yet.
I'm running my tests so far on non-angular pages with browser.ignoreSynchronization = true;
I'm currently working on reproducing the problem on the testapp.

:+1: Thank you for the workaround @pgrm!
This problem was hitting me probably about 8 times out of 10 on the first test in my suite.
FWIW, my platform below.
    Build info: version: '2.51.0', revision: '1af067d', time: '2016-02-05 19:15:17'
    System info: host: 'MacPro-3362', ip: '<IP_ADDRESS>', os.name: 'Mac OS X', os.arch: 'x86_64', os.version: '10.10.5', java.version: '1.8.0_45'
    Driver info: org.openqa.selenium.chrome.ChromeDriver
    Capabilities [{applicationCacheEnabled=false, rotatable=false, mobileEmulationEnabled=false, chrome={chromedriverVersion=2.21.371459 (36d3d07f660ff2bc1bf28a75d1cdabed0983e7c4), userDataDir=/var/folders/qv/5y3z61z52ng0tgtffc5fmc24zcqrgw/T/.org.chromium.Chromium.4RbZiA}, takesHeapSnapshot=true, databaseEnabled=false, handlesAlerts=true, hasTouchScreen=false, version=49.0.2623.87, platform=MAC, browserConnectionEnabled=false, nativeEvents=true, acceptSslCerts=true, locationContextEnabled=true, webStorageEnabled=true, browserName=chrome, takesScreenshot=true, javascriptEnabled=true, cssSelectorsEnabled=true}]

I'm experiencing the same issue as @radek-holy described on Angular 2 with input fields.

I spent a whole day trying to figure out why my tests were suddenly failing. This post fixed all my troubles. Thank you !

I am not using Angular 2. I have protractor 3.0.0. My input fields on login screen no more work! SendKeys() don't seem to work at all. even clear() does not work on the inputs. I get an error invalid element state: element must be user-editable in order to clear it. I actually click() the input before trying to clear it.
The second issue with clear() is secondary. I am also facing the same issue with sendKeys() not working. MY chrome recently updated to version 50.0.2661.102 m. Unfortunately this workaround does not work for me.

Hi,
Try with below example,
inputUsername.sendKeys(protractor.Key.chord(protractor.Key.CONTROL, "a"));
inputUsername.sendKeys(protractor.Key.chord(protractor.Key.CONTROL, "x"));
browser.sleep(1000);
inputUsername.sendKeys(username);
Here instead of clear(), we can use "inputUsername.sendKeys(protractor.Key.chord(protractor.Key.CONTROL, "x"));"

@juliemr
I have the similar problem, and I am able to consistently reproduce it in the testapp link you mentioned above
In the elementexplorer, I typed command:

I expected the character 5 will be appended after "Anon" (to be "Anon5"), but in fact, it deleted the last character of "Anon":

Then I tried sending another string and got this:

It looks like the character 5 is misinterpreted to backspace... and the similar misinterpretation happened to charactor 6 (move cursor to the next button or textbox, similar to the function of tab).
Here is the environment I used:
Chrome: 50.0.2664.102 (64-bit)
ChromeDriver: 2.21.371461
Selenium: 2.45.0
Protractor: 2.0.0

Splitting up the string seems to fix the issue for me, for example:
emailAddress.split('').forEach((c) => emailField.sendKeys(c));

All issues here are either a duplicate of https://github.com/angular/protractor/issues/3003 or https://github.com/angular/protractor/issues/3196 - I'm closing this issue to concentrate effort in those two.

Two years later and @sdsanders' hotfix is still the way to go

2.5 years and we are still waiting.... it is f$%#! not usable like this.

Just implemented @sdsanders hotfix on an Angular Material form. It seems to be the only stable solution.

Just in case anybody it's still facing this problem.
I'm using Selenium, Chrome and Mac but having the same problem.
After quite a few trial-and-error approaches what worked for me was introducing a tiny delay before entering the test.
Even if I tried char by char it didn't work for me.
It seems that the problem has to do (in my case) with these inputs:

I could see that once I began typing there were a refresh were the placeholder reappeared, then cleared the input field and wrote the rest of the text
The slower you wrote the easier was to see this effect.
So I introduced this before start typing:
getClock().pauseFor(timeInMilliseconds);

Animations seem to be a problem.
I had this issue whenopening a modal dialog (sendKeys only wrote the first 1 to 4 letters of my text). I noticed that disabling animations by using NoopAnimationsModule fixed the input issue.
Since I want to have animations I ended up with a fix where I assert that the dialog is ready for input by testing for the visibility of the modal without the ng-animating class on it (Xpath: //mat-dialog-container[not(contains(@class, 'ng-animating'))]).


themes/content: no such file or directory

Fresh installed hugo and the theme.
run hugo server got this:
My_site_path/themes/content: no such file or directory

Hugo Static Site Generator v0.54.0-B1A82C61 linux/amd64 BuildDate: 2019-02-01T09:40:34Z

Hi @jacobsun Sounds like you may have an issue aside from the theme. Can you please ask your question to the community? https://discourse.gohugo.io/
Thanks!


${workspaceFolder} is not resolved on VSC start

Brief Issue Summary
I have a workspace with a settings.json with:
"cmake.cmakePath": "${workspaceFolder}\\Tools\\cmake\\bin\\cmake.exe"
When starting VSC, the following error are displayed:
[proc] Executing command: ${workspaceFolder}\Tools\cmake\bin\cmake.exe --version
[proc] The command: ${workspaceFolder}\Tools\cmake\bin\cmake.exe --version failed with error: Error: spawn ${workspaceFolder}\Tools\cmake\bin\cmake.exe ENOENT
[proc] Executing command: ${workspaceFolder}\Tools\cmake\bin\cmake.exe -E capabilities
[proc] The command: ${workspaceFolder}\Tools\cmake\bin\cmake.exe -E capabilities failed with error: Error: spawn ${workspaceFolder}\Tools\cmake\bin\cmake.exe ENOENT

If I replace ${workspaceFolder} by the "real" path, there are no error. But I need to use ${workspaceFolder} so that it work for any coworker using the projet in its own directory
After start, it work but having error displayed on start is confusing, especially for coworker not familiar with CMake
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

@ANauzet Thanks for posting this issue! We should be supporting this macro, so this is definitely a bug.

@ANauzet PR opened here #3658


Add and implement bit shift left and right opcodes

Closes #145.
Note to self: add tests

At least getType tests and Lua tests should be added.


Question Regarding This Plugin

Hi there,
when I use Source->Format is it also fomatting using rules which is imported via this plugin?


Ignore tagged template literals

Tagged template literals are quite often used for things like SQL where curly quotes are not desirable. Also, by ignoring expressions like
String.raw`Don't change my quotes here please!`

there is now an escape hatch where you can use straight quotes (for things like code snippets) without having to clutter your codebase with eslint-disable comments.

Thank you for the PR!


UndefPhase detected when one phase has mode

When creating a Role with two Phase children, the UndefPhase antipattern is detected even though one of the phases has a Mode, which is one of the suggested refactoring plans in the OntoUML documentation.
A screenshot of the situation:

The UndefPhase antipattern is also detected when the Parent has a <<Characterization>> Mode which is used to decide between phases.
Screenshot:

This error was in November and it cannot be reproduced anymore.


Ability to specify timezone in query

Request for enhancement to specify timezone in query, instead of the default GMT / Zulu.
Example would be to group by date using CST instead of GMT/Zulu.
The workaround now might be to use INTERVAL like:
💪 $ select first_name, last_name,last_visit,  (last_visit - INTERVAL("PT5H")) as US_East, (last_visit - INTERVAL("PT6H")) as US_Central from patients
Query time: 0.1s
 first_name | last_name  | last_visit            | US_East               | US_Central            |
------------|------------|-----------------------|-----------------------|-----------------------|
 Jude       | Wilkerson  |  2013-04-08T08:07:42Z |  2013-04-08T03:07:42Z |  2013-04-08T02:07:42Z |
 Theo       | Joseph     |  2014-09-22T10:15:40Z |  2014-09-22T05:15:40Z |  2014-09-22T04:15:40Z |
 Grisel     | Branch     |  2012-09-06T15:20:25Z |  2012-09-06T10:20:25Z |  2012-09-06T09:20:25Z |
 Ollie      | Shepherd   |  2015-05-08T09:09:42Z |  2015-05-08T04:09:42Z |  2015-05-08T03:09:42Z |
 Jonathan   | Riley      |  2016-01-16T13:14:34Z |  2016-01-16T08:14:34Z |  2016-01-16T07:14:34Z |
 Vaughn     | Wilkins    |  2013-11-22T11:17:21Z |  2013-11-22T06:17:21Z |  2013-11-22T05:17:21Z |
 Kayla      | Bradshaw   |  2007-04-22T14:00:29Z |  2007-04-22T09:00:29Z |  2007-04-22T08:00:29Z |
 Devon      | Jensen     |  2016-10-13T13:37:47Z |  2016-10-13T08:37:47Z |  2016-10-13T07:37:47Z |
 Jean       | Santiago   |  2013-08-31T08:59:46Z |  2013-08-31T03:59:46Z |  2013-08-31T02:59:46Z |
 Mee        | Shaw       |  2006-11-11T12:35:20Z |  2006-11-11T07:35:20Z |  2006-11-11T06:35:20Z |

Is there a better / more standardized way of doing this?  The example above would not survive daylight savings transitions and would need to be updated twice a year.

I am not sure exactly what you are asking for. Can you find equivalent functionality in Postgres so we can simply replicate the corresponding Postgres function?

@jdegoes, something like AT TIME ZONE ?  Postgres Example

@damonLL That looks good to me. Is this for paying customer, demos, prospects, or a user?

@jdegoes paying customer

@zeeiyerWork Let's get this into the June milestone.

It's looking like this will be possible after the solution to #2564, though there will need to be extra work to add syntax to SQL^2 for it.


Added some changes in the readme.md file to see how fork works

This is to check how fork works in large open source projects

Could you maybe test this on your own repos, or with real changes that can provide value to the project?
Creating a PR notifies the maintainers of a project, possibly taking time out of everyone's day/week. Please be mindful of this.


Apache configuration items

so we can add our own config files

modified the branch, making my own PR


Sandbox Example Please

Could you please put an example in the readme of how to get this running from scratch?
it's very confusing to have to backtrack through all the story code to try and get a basic popover up and running.
The reason I ask is I am getting _reactDom2.default.createPortal is not a function error and I have no idea why but I'm sure it's my understanding of how to get setup.
Thanks

I'm getting this error as well. Component:
import * as React from 'react';
import Popover  from 'react-popover'

class ReportTile extends React.Component<{ name: string }, { isOpen: boolean }> {
  constructor(props) {
    super(props);

    this.state = {
      isOpen: false
    }
  }

  togglePopover = () => {
    this.setState({ isOpen: !this.state.isOpen })
  }

  render() {
    return (

        <Popover
          body={

test3

          }
          isOpen={this.state.isOpen}
          onOuterAction={this.togglePopover}
        >

{this.props.name}

        </Popover>

    )
  }
}

export default ReportTile;


After expansion fills, MGSwipeButtons underneath render incorrectly

This issue started occurring after updating to MGSwipeTableCell from 1.4.3 to 1.5.1.
I'm using these settings.
self.rightExpansion.threshold = 1.1f;
self.rightExpansion.fillOnTrigger = YES;
self.rightSwipeSettings.showAnimation.duration = 0.2f;
self.rightSwipeSettings.hideAnimation.duration = 0.2f;
self.rightSwipeSettings.transition = MGSwipeTransitionDrag;

The cell unfolds the buttons correctly initially, showing the two that ought to be presented, as such.

Then I swipe more to the left, activating fillOnTrigger. After I try to unfold the buttons again, this is the result.

The buttons are now overlapping.
Thanks a lot!

Hi Joe,
I have tried to reproduce the issue using the MailApp demo but no luck. I have same settings, check this branch with the changes in the maill app demo: https://github.com/MortimerGoro/MGSwipeTableCell/commit/356da16a0bd86cc83a5666c452d13775d0abc3e7
Any more differences/ideas to reproduce it?

I pulled your changes, but it's not exactly the same as mine. In your case, the buttonIndex is set to something that's destructive, so the row disappears. In my case, it doesn't disappear. I modified your code, and I still wasn't able to reproduce it though.
In my code, I added [cell refreshButtons:NO] at the end of - (BOOL)swipeTableCell:(MGSwipeTableCell *)cell tappedButtonAtIndex:(NSInteger)index direction:(MGSwipeDirection)direction fromExpansion:(BOOL)fromExpansion and that seems to have solved my issues for now. Hope that might be hint at something? ¯_(ツ)_/¯

I am getting some thing similar. and it as to do with the "expansionSettings.fillOnTrigger = YES;"
on mile what happens is that the next time I swipe a cell the start of the button layout for the right side is the leading side of the parent view. feels as if some how the view content for the buttons is not getting the correct layout constraint.
image before full swipe right to left swipe

and this is happens intermittently post doing a full right to left swipe


[Ansible Installer] Add Kiali as addon

Including Kiali on Ansible Installer in order to match #5683

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: gbaufake
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @gbaufake. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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

/ok-to-test

@gbaufake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test istio-pilot-e2e

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test e2e-bookInfo

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test e2e-bookInfo

prow/e2e-bookInfoTests-v1alpha3.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test e2e-bookInfo-envoyv2-v1alpha3

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test e2e-bookInfo

prow/e2e-bookInfoTests-v1alpha3.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test e2e-simple

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

@gyliu513 can you review the new changes?

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

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
f72430573eeef08dadbb1f9746cbfd2d13050f6b
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
017b9fcaf6b2506660461e5caa31fcbd0106f36c
link
/test istio-pilot-e2e

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: gbaufake, gyliu513
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @sdake @linsun @costinm
Can you help check if we can merge? Thanks.

@sdake @linsun  or @costinm
Can you review it?

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gbaufake, gyliu513
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [gyliu513]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gbaufake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
017b9fcaf6b2506660461e5caa31fcbd0106f36c
link
/test istio-pilot-e2e

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


Error installing preferences service

Following the steps, from a clean environment, preferences pods got in a error loop.
Details can be found here:
https://gist.github.com/lucasponce/28acb79f6f10fbefce32c9a0e53a8294

Which istio version are you using? 0.4.0?
I had problems installing preferences services before, but it was problem of image registry.
Also, I used istio 0.5.0 but had lots of troubles on installation and came back to 0.4 version.
Last and a little bit mean, I found myself running all the command of the tutorial really fast (bulk mode) and I realized that stopping among commands helps to better installation. Can't explain why accurately.

It seems that you not build the prerence image using minishift docker daemon: eval $(minishift docker-env). Please re-open if you still facing the issue.


ci:  run sdk tests for weekly build

What type of enhancement is this?
Other
What does the enhancement do?
To identify any bugs, such as #2578, we can execute the Erlang SDK test cases with the latest greptimedb build.
Additionally, we can also run the Java SDK and Golang SDK test cases with the latest weekly build to ensure that any bugs are exposed as soon as possible.
Implementation challenges
No response

It serves as a black box test, addressing scenarios that our unit tests and integration tests may not encompass.


SSH info in webportal behind pylon is not replaced.

REST server part in pylon should also replace hdfs ips, now only nm/rm ips.

should this becomes a hotfix to 0.8.y?

      should this become a hotfix to 0.8.y?

OK I will patch it to 0.8.y after testing it on master


Framework - Update to be compatible with backend

This can be verified through Github auth workflow functioning correctly.

Closed in #50


Macro editing does not resize

Enabling macro editing causes the macro editor to no longer scale with resizing the edit dialog.
As an example:

Compared to built-in editor:

Another macro editor that changes the text space to a better contrast option (black on white, with green as comments, etc.)
https://github.com/ardittristan/VTTMacroEditor


[Issue-3108] Extension - Add highlight button Export multi account on Select account screen

Related issue: #3108

🚀 Deployed on https://pr-3113--sw-web-runner.netlify.app


[21.05] thunderbird, thunderbird-bin: 91.1.1 -> 91.2.1,  thunderbird: mark 78.14.0 insecure

Motivation for this change
#139840, #140810, and #142638 for release-21.05.
This also mark 78.14.0 insecure. 78.14.0 is the final 78.x release, and Firefox 78.15 fixes some vulnerabilities. 78.x should be removed from the next release branch.
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

This bump broke icon installation for me.

@OPNA2608, I have opened a PR #145858.


cmd/torusctl: fix error & help messages for torusctl peer

Fix error message for "torusctl peer remove" to "couldn't remove peer from ring", as it's a remove action.
Also improve help messages a little for "torusctl peer add" and "remove", to indicate that they need a parameter either ADDRESS or UUID.

LGTM


Package creation guide

[ ] Change the node distributable crate to include the HTTP gateway and local development node
[ ] Publish a new version of locutus with the executable, and stalone cli tool
[ ] Publish the locutus stdlib crate
[ ] Publish the locutus typescript std lib in npm
[ ] Make guide using gitbook and pin @sanity so he can check it.

Is Gitbook preferable to Github Wiki for this?

Is Gitbook preferable to Github Wiki for this?

I think there are several advantages (I really dislike documentation on wikis, from my corporate experience I guess):

Wiki: you are stuck with the vendor/service you are using. If you have to migrate your repository/web is not easy and basically you need to recreate everything.
Documentation as code.

Documentation can be versioned along with the code.
Released/built with CI pipelines.
Rendering and writing are 2 different processes (like with TeX) of assembling documentation which makes...

Hosting and presenting in different styles/formats easier, modifying easier, etc.
Easier or more natural to integrate with generated documentation (also from CI/publishing), which I hope we will have plenty and well documented eventually.

Sorry, I meant mdbook, not gitbook.

Sounds good, I assume doc generation can be automatic via Github Actions?

Sounds good, I assume doc generation can be automatic via Github Actions?

Yes, that's right.

Possible points to improve:

Detail the configuration of the locutus.toml file, for example how to specify the file to use as initial state.
Link to the application examples available in the repository.
A page with the standard development flow could be fine (how to make changes in the frontend and backend, dependencies usage...)

Despite this, I have been able to create, compile, publish and execute the contracts with the documentation.

After talking with @netsirius  I think we are good for now. As we improve the development workflow or add functionality we will update the documentation.


Infer type from a Sibling Prop

I did not find any information on Stack overflow or in existing issues so creating a new issue. Sorry if this is a duplicate.
I am not sure if this is a compiler limitation but this is what I am trying to achieve. I am creating a grid component that takes rows and columns. For example
interface Row {
   field1: string;
   field2: number;
}

interface Column<R> {
   field: keyof R;
   value: R[keyof R]; // This gives 'string | number '
}

Now if I create a columns array
const columns: Column<Row>[] = [
  { field: 'field1', value: 2 }, // There is not compilation error here which is expected because of 'R[keyof R]'
  { field: 'field2', value: 3 }
];

Is there a way I can modify value: R[keyof R] type so that in { field: 'field1', value: xyz } xyz can only be a string (Row['field1']).
Thanks.

To make it work you will need something like the following
interface Row {
   field1: string;
   field2: number;
}

interface Column<R, F extends keyof R = keyof R> {
   field: F; // for Row as R produce the set: 'field1' | 'field2'
   value: R[F]; // for Row as R produce the set: string | number
}

// for Row as R produce the set: Column<Row, 'field1'> | Column<Row, 'field2'>
type AnyRowField<R = Row, F extends keyof R = keyof R> = F extends keyof R ? Column<R, F> : never

And use it like this
// type checkes
const columns: AnyRowField[] = [
  { field: 'field1', value: '2' },
  { field: 'field2', value: 3 }
]

// doesn't type check
const columns3: AnyRowField[] = [
  { field: 'field1', value: 2 }, // number not compatible with string
  { field: 'field2', value: '3' } // string not compatible with number
]

Example - https://bit.ly/2GG5nk2

Thank you so much. It worked perfectly.


0.8.0 requires "resolver" due to Cargo bug

The Cargo.toml published with 0.8.0 contains the "resolver" key which is not present in the original Cargo.toml suggesting that the package was published using a Cargo version affected by https://github.com/rust-lang/cargo/issues/10954
Could you publish a semver-compatible release that does not contain the "resolver" key? This would allow the rust-numpy crate to update its dependency on ahash without changing MSRV, c.f. https://github.com/PyO3/rust-numpy/actions/runs/3073552488/jobs/4965760601

Friendly ping. Just reuploading with a bumped version and using a current Cargo would really help. Thanks!

I am working on 0.8.1 which will also have some other improvements.

I just released 0.8.1, but it has the same problem. Is this caused because I released using nightly?

I just released 0.8.1, but it has the same problem. Is this caused because I released using nightly?

It should not be the case when the current nightly is used, only if you have pinned the version to one still affected by the above mentioned bug.

This appears to have been fixed with the 0.8.2 release. I will release using stable from now on as a precaution. Closing for now.

Thanks, the resolve issue is now fixed indeed.
Sadly, we cannot bump to 0.8.x as the atomic-polyfill feature shadows a dependency which the Rust 1.48 toolchain that is shipped by Debian does not yet support, so we'll have to stick to 0.7.x for now after all, c.f. https://github.com/PyO3/rust-numpy/actions/runs/3442951690/jobs/5744015218.
If I could have a wish, adding the rust-version key to this crate's Cargo.toml would be nice. (I would make a PR but I did not find MSRV information.)


Dropdown menu not working on touch devices

A few people seem to have raised this issue on the Forum but no feedback from anyone from Zurb.
Here is a codepen to illustrate the problem (on 6.2.4):
http://codepen.io/Byzantium/pen/oYMbmx
On pc/mac the submenu items are clickable and work as you would expect (except giving a 404 within codepen)
On an ipad, if you

Tap one of the parent menu headings the submenu opens
Tap one of the submenu items the submenu closes with no other action

I have only tested on an ipad as that's all I have, but others have used other touch devices and had the same issue.
I'm still learning foundation, so if the problem is with my code then my apologies. I can't see any reason why it should not work, but it does not.

I'm seeing this behavior as well.  Comparing 6.2.3 to 6.2.4 I removed the following lines (lines 3909-3911 in foundation.js).:
if (_this.options.closeOnClickInside) {               _this._hide($elem); }
The menu doesn't hide on click anymore, but at least the link is followed.

It looks to me that this change: https://github.com/zurb/foundation-sites/pull/9472/files
incorporated into the 6.3 release will fix this. I'm not an insider - this is just my personal observation, but it looks like it should do the trick and since 6.3 is out real soon (tomorrow I believe), I'll wait to try that.

@gitwebdevseo v6.3.0 is released. Is it still an issue ?

I encountered this issue in https://github.com/zurb/foundation-sites/issues/9293 and thought it was later worked out. The interim fix was to add a "data-close-on-click-inside" => "false" to menus. At one point the issue was fixed before 6.3 was released. I just updated to 6.3 and took out this interim fix and the menus aren't working. I just put the fix back in. Really surprised.

@ncoden sorry for the delay in replying - been travelling. I have tried it with a fresh local install of 6.3 and it no longer seems to be a problem - i.e. dropdown menu submenu items are selectable on touch devices - well on the ipad I tried with anyway. I would put up a codepen to illustrate but the jsDelivr CDN links (referenced on this page: https://foundation.zurb.com/sites/docs/installation.html) are not yet actually there, so I can't. It would be good to get those CDN links up!

Actually - I will raise a separate request to get the CDN links populated.

I'm having this issue too with the 6.3.0 version. I have created two codepen's:

http://codepen.io/sinspiral/pen/ObqbLG

This is using v6.3.0 which is broken. On touch devices, the Items 1A-1C don't navigate to its links.

http://codepen.io/sinspiral/pen/BQbQjX
This is using v6.2.1. This version was the last one it was working.

I'm using the same example as on the docs.
As workaround, the only thing I found it works for me is adding to the '.menu' element the attribute data-close-on-click-inside (see http://codepen.io/sinspiral/pen/zoboPp).
Hope it helps!!

Yes - contrary to what I said in my previous post, I agree with @sinspiral that it is still broke. Did my own codepen here:
http://codepen.io/Byzantium/pen/jVXrpR
clicking on submemu items on pc/mac gets a 404 (as expected / intended) but tapping on submenu items on a touch device produces no action except that the submenu gets closed.

thanks @memoht that fixed the issue.

I experienced this problem using v.6.3.0 on an iPad and data-close-on-click-inside="false" came to rescue.

been facing the same issue. i'm using foundation sass through the cli so it's the latest version (6.3.0) and the dropdown menu items are not clickable on my mobile phone. the data-close-on-click-inside="false" solution mentioned by @memoht worked as a temporary fix for now.

Referencing the same error on android firefox and chrome with 6.3.0
data-close-on-click-inside="false"
changes nothing

Added custom handler for parent menu items
$(".topmenu li:has(ul) > a").click(function(e) {
	e.stopPropagation();
    	e.preventDefault();

		if($(this).hasClass("oneTouch")){
  			$(this).removeClass("oneTouch");
  			window.location = $(this).attr("href");
  		}
		else{
			$(this).addClass("oneTouch");
		}
	});

@sashabeep are you seeing this in 6.3.1? I believe @sinspiral's PR addressed this

@kball 6.3.0, 6.3.1 not yet tested without my custom handler.
Facing the problem on touchscreen laptop, now i can't access to this machine

@kball @sashabeep 6.3.1 has fixed this issue for me.

This issue has been mentioned on Foundation Open Source Community. There might be relevant details there:
https://foundation.discourse.group/t/drop-down-menu-disappears-in-tablets/4358/1


Update cord.rb

Just noticed that this wasn't working and wanted to help.
Nathan

Awesome - thank you!

Jeez that was quick.


Update to solana 2.0 sdk

Can't use the latest release with the latest solana-* 2.0.0 crates

can i take on this?

What is there to take on.... you change the number from 1 to 2... https://github.com/luca992/helius-rust-sdk/commit/9a78c68c8cfc0d8dfe9f0689342bf1cf4a44f8e5

Can be considered closed by #80


Add workaround for quick button inconsistency

WHY
BEFORE - What was wrong? What was happening before this PR?
Reported in #5421
Following a simple tutorial that should "just work", was not "just working" 🤷♂
I've debugged the issue and the culprit is the fact that we ->studly($buttonName) to get the access string. So a button with the name comment would have the access string Comment.
AFTER - What is happening after this PR?
It checks both operation name strings. First the regular one and as a fallback the camelCased version of the operation.
HOW
How did you achieve that, in technical terms?
I could have just changed the stub and define the operation button with access => Str::studly($operationName) or just removed the studly() in the quick.blade.php file, but that would be a BC at the moment in any of those cases. But it's what I've planned for v7.
So I made it fallback to a camelCased version of the name as last resort to try to find access.
I don't think this would ever be a BC the way I implemented because it wouldn't be possible for you to have a moderate and a Moderate operations, or a SuperModerate and superModerate operations at the same time.
Is it a breaking change?
I don't think so, no.
How can we test the before & after?
Follow the tutorial in Backpack website https://backpackforlaravel.com/docs/6.x/crud-operations#creating-a-new-operation-with-a-form-1 . It will fail previously, it will work after.

After our talk we decided to do this change on the blade file itself so it only concerns the quick button.
It's better to keep hasAccess() clean from string manipulations. Whatever string it receives, is the string it checks if there is access to.
We would risk introducing security issues if we tried to guess access for operations at this level, so we should be better with the solution for quick button in the blade file..


Hide attribute intellisense for a taghelper

what if i don't want to show attribute intellisenses for a taghelper eventhough i have set HtmlAttributeName attribute to the corresponding class properties?
For example:
[HtmlAttributeName("testTag")]
public Class TestClass: TagHelper{

  [HtmlAttributeName("testProp")]
  public string testProp{get;set;}
}

Here for "testTag" element i don't want to show "testProp" atttribute and some more similar attributes.
The element tag only should present with

If you want to hide HTML IntelliSense for a TagHelpers attribute or tag name you can decorate the corresponding TagHelper class/property with [EditorBrowsable(EditorBrowsableState.Never)]. Mvc does this for the UrlResolutionTagHelper.

In my case, i have two tagHelper elements on same class. I don't want to show the properties as attributes for one taghelper element but for other taghelper element it should show the properties as attributes.

You'd have to split that into two separate classes to get that effect.

Now, i have created a separate class with multiple target element..
For instance lets take the below code example.
`
[HtmlTargetElement("CustomTag1",ParentTag ="ParentTag"),
HtmlTargetElement("CustomTag2", ParentTag ="ParentTag"),RestrictChildren(" "ChildCustomTag1",new string[] { "ChildCustomTag2" })]
public class TestCollection{
}
[HtmlTargetElement("ChildCustomTag2",ParentTag ="CustomTag2")]
public class ChildCustomTag2{
}
`
So in view page under "CusomTag1" element ,"ChildCustomTag2" is showing in intellisense..its wrong.. Actually the intellisense should not have shown since the parentTag of "ChildCustomTag2" is not "CustomTag2" or RestrictChildren attribute should be applicable for each target element instead of to the class/

@alansangeeth In the example you provided the current tooling does not support the ParentTag/RestrictChildren hiding of IntelliSense. We do have fixes for these in RC2 however. Hopefully with the next release you'll see your scenario light up as you expect in IntelliSense.

@NTaylorMullen  Thanks for the information. I need another clarification regarding implementing TagHelper in derived class. I have a derived class which is derived from another class, not an interface and now how to implement Taghelper to derived cass since we cannot inherit more than one class.

@alansangeeth you can implement ITagHelper yourself. The TagHelper base is just for convenience.

You can see how simple the TagHelper class is here.

Any further action required here or are the current questions answered?

If i define TagHelper for a class with Multiple constructors, it throws exception as "Multiple constructors accepting all given argument types have been found". Is there a way to avoid this with multiple constructors?

@alansangeeth Only way would be to make one private and create an alternative static method like MyTagHelper.Create(x,y,z) that delegated to a private ctor.


possible memory leak

Hi
There seems to be an issue with this code - that stops the scene from correctly deInitialising
I have modified your sample code by moving all of the joystick setup code out of the didMoveToView function into a separate function.
I added some print statements so I could see when initialisation and reinitialisation were happening
Additionally I added  willMoveFromView and  deinit functions as shown below
override func didMoveToView(view: SKView) {

    print ("***** \(__FUNCTION__) *****")

    /* Setup your scene here */
    backgroundColor = UIColor.whiteColor()
    physicsBody = SKPhysicsBody(edgeLoopFromRect: frame)

    setupJoysticks() \\comment out to remove joystick from scene

    addApple(CGPointMake(CGRectGetMidX(frame), CGRectGetMidY(frame)))

}
override func willMoveFromView(view: SKView) {
    print ("***** \(__FUNCTION__) *****")

    self.removeAllChildren()
    self.removeAllActions()

    for sv in view.subviews {
        sv.removeFromSuperview()
    }
}

deinit {
    print ("***** \(__FUNCTION__) *****")
}

So that I could get the scene to reload I changed the default clause in the touchesBegan to -
        default:
            //addApple(touch.locationInNode(self))

            let scene = GameScene()
            scene.scaleMode = .ResizeFill
            let transition = SKTransition.moveInWithDirection(SKTransitionDirection.Right, duration: 1)
            self.view?.presentScene(scene, transition: transition)

This means whenever you touch the screen - a new scene is loaded
With the setupJoysticks() code commented out (in didMoveToView) I get the following trace
***** didMoveToView *****
***** touchesBegan(_:withEvent:) *****
***** didMoveToView *****
***** willMoveFromView *****
***** deinit *****
With the setupJoysticks() code enabled (in didMoveToView) I get the following trace
***** didMoveToView *****
***** touchesBegan(_:withEvent:) *****
***** didMoveToView *****
***** willMoveFromView *****
The deinit NEVER happens.
This means that every time you instantiate a new scene - there is a memory leak
I am happy to send the demo project to you if it helps.
Best Regards
Richard

Sorry for my english
Hi Richard!This is because we use the strong references (GameScene has strong reference to AnalogJoystick.In turn AnalogJoystick has strong reference to GameScene in closures).
I use self here
moveAnalogStick.startHandler = {

   guard let aN = self.appleNode else { return }
   aN.runAction(SKAction.sequence([SKAction.scaleTo(0.5, duration: 0.5), SKAction.scaleTo(1, duration: 0.5)]))
}

// && here

moveAnalogStick.trackingHandler = { data in

    guard let aN = self.appleNode else { return }
    aN.position = CGPointMake(aN.position.x + (data.velocity.x * 0.12), aN.position.y + (data.velocity.y * 0.12))
}

ARC not reset the reference to the object  && he (self) will never be removed
There are three ways to solve the

Use unowned (in demo project).Ex:

moveAnalogStick.startHandler = { [unowned self] in

   guard let aN = self.appleNode else { return }
   aN.runAction(SKAction.sequence([SKAction.scaleTo(0.5, duration: 0.5), SKAction.scaleTo(1, duration: 0.5)]))
}

[unowned self]  - Changing self to unowned won't increase self 's retain count by 1, therefore allowing to ARC to deallocate it properly at the correct time.
2. Use unowned (in demo project).Ex:
moveAnalogStick.startHandler = { [unowned self] in

   guard let aN = self.appleNode else { return }
   aN.runAction(SKAction.sequence([SKAction.scaleTo(0.5, duration: 0.5), SKAction.scaleTo(1, duration: 0.5)]))
}

[unowned self]  - Changing self to unowned won't increase self 's retain count by 1, therefore allowing to ARC to deallocate it properly at the correct time.
3. A few days later I upload version of the supported protocol
Happy coding Richard

Thanks for the fast response - I will try some of above and let you know if I have any issues.

Please note : I had to use [weak self] to get this to work properly.
When I used [unowned self] I got random errors when the new scene loaded


Permit creation of records at apex.

To support creation of records at the apex, an empty string needs to be sent in the JSON during record creation.

LGTM. As discussed, we may want to consider the use of pointers for fields.
Just one thing: I'd shrink the test and remove anythink we don't need for this specific use case, specifically any assertion that is not directly related to the empty value or the API method returning an error.
We don't really care about asserting what we get as returned data, as we already do it in the other general test.


requisitions

Is it possible to use GPU to speed up requests? Would there be any simpler demonstration like just hello world?

Sorry, I don't understand your question.

Is it possible to use GPU to speed up requests?

Which part of the request do you want to speed up?

all parts, from routes to assembly of responses

Ok, I'm not sure there is any example of this to be honest.
You should ask your question (and provide more details) here: https://github.com/NVIDIA/triton-inference-server

The name of this repository might be a bit misleading by the way; this is still doing the HTTP processing on the CPU side; but offloading the deep learning inference to the GPU.

So I was wrong about the purpose of the project, I thought GPU REST was about speeding HTTP requests

Can I seriously speed up HTTP responses using GPU? If yes, there is a project about it.

 But if you're interested in network processing offload with GPUs, there is this the NVIDIA Aerial product for 5G packets:
https://developer.nvidia.com/aerial-sdk


[macos] Aliasing occurs when the image is zoomed out

flutter doctor:
[✓] Flutter (Channel master, 3.1.0-0.0.pre.2448, on macOS 12.5 21G72 darwin-x64, locale zh-Hans-CN)
[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[!] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage
        on the Dart side.
        Without CocoaPods, plugins will not work on iOS or [macOS.]
        For more info, see https://flutter.dev/platform-plugins
      To install see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.2)
[!] Proxy Configuration
    ! NO_PROXY is not set
[✓] Connected device (3 available)
[✓] HTTP Host Availability

Original Image:
https://up.enterdesk.com/edpic/6d/3d/d5/6d3dd596e928f1c41a61cbfd7e8cf874.jpg
my code:
   return Scaffold(
      backgroundColor: Colors.black,
      body: Center(
        child: Image.network("https://up.enterdesk.com/edpic/6d/3d/d5/6d3dd596e928f1c41a61cbfd7e8cf874.jpg"),
      ),
    );

this is demo:

The picture is seriously distorted

Hello @zhangwen0510, Thank you for filing this issue. There is a similar issue here. Is this what you're experiencing? Although the issue mentioned is specific to web, it looks like the same thing you are experiencing.

No, this problem occurs on the macos desktop platform I use, other platforms may also have this problem

Could you confirm if this is the case? If so, this issue could be closed as a duplicate of https://github.com/flutter/flutter/issues/109637

I'm not sure because the platforms used are different @exaby73

I cannot use the image you provided on MacOS. Getting the following error

Code Sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(home: ImgFilterTest());
  }
}

class ImgFilterTest extends StatelessWidget {
  const ImgFilterTest({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const Scaffold(
      body: Center(
        child: Image(
          image: NetworkImage(
            'https://up.enterdesk.com/edpic/6d/3d/d5/6d3dd596e928f1c41a61cbfd7e8cf874.jpg',
          ),
          width: 600,
          filterQuality: FilterQuality.medium,
        ),
      ),
    );
  }
}


Backend Migration: Remove PPL Table Explore-only restriction (condition is in frontend)

Done with https://github.com/grafana/opensearch-datasource/issues/372, https://github.com/grafana/opensearch-datasource/issues/290 and https://github.com/grafana/opensearch-datasource/issues/275

Reopening as this was reverted and needs to be reenabled.

Will be done in https://github.com/grafana/opensearch-datasource/issues/407


Support for Linksys AE1200

Provides support for the Linksys AE1200, a USB dongle that enables WiFi.
Essentially, on normal distros the dongle does not work on boot. It must be physically disconnected and plugged back in to be recognized. Upon closer examination, there seems to be a mismatch between the device ID on boot (13b1:0bdc) and the value that brcmfmac (the kernel module) accepts (13b1:0039). After physically plugging it for the 2nd time, lsusb states the correct ID.
Having this udev rule fixes the issue and WiFi works on boot. Merging this would remove the need to keep the spare .rules file.

We use Semantic PR commit messages as outlined here
https://universal-blue.org/CONTRIBUTING/#commit-messages
other than that the changes look good

Hello. I don't know if there's a cleaner way, so I will close this PR and create another one.


Conda-forge package

As mentionned in issue #32, it would have been useful to submit simhash-py to https://conda-forge.github.io/
so that it could be simply installed with
conda install -c conda-forge simhash-py

without the need to compile the C++ code.
Once a Pull Request with the setup is submitted at https://github.com/conda-forge/staged-recipes and accepted, it would create a repository https://github.com/conda-forge/simhash-py-feedstock that would have to be updated for every new release. This should allow to create a pre-built Python package for Linux and possibly Mac OS (Windows won't install won't be available for now cf #31 ). The questions I was wondering about are,

who should make that PR submission (I can do it, if necessary)
who should be added to the maintainers of that conda-forge/simhash-py-feedstock setup ?
there is still the naming issue, discussed in issue #32, the package would be called simhash-py but would be imported as simhash, since the simhash package already exist at https://github.com/conda-forge/simhash-feedstock . Not sure if there is a better way of doing it?

@b4hand What do you think?

Let me discuss the naming concern with a few devs here, and I'll try to see what we can do.

I made a request to create a simhash-py conda-forge package ( https://github.com/conda-forge/staged-recipes/pull/2656 ) which would be imported as import simhash. Please let me know if you want to change the package name or if someone form SeoMoz should be added to the list of maintainers there (which can always be done later).. Thank you.

The conda-forge setup for simhash-py was created at https://github.com/conda-forge/simhash-py-feedstock
simhash-py 4.0.0 can now be installed for Python 2.7, 3.5, 3.6 on Linux and MacOS using conda with,
conda config --append channels conda-forge
conda install simhash-py

which does not require Cython or a C++ compiler.


(Edge case) Handling of collaborative projects with complicated ACL

Describe
(Reporting here for future design improvements.)
Currently, in order to see and select a folder for annotation, the user first must be able to see to a project by having edit access on the project.
In this tricky case, there is a collaborative project involving two labs, one Owner and the other Contributor. The Contributor lab is granted local edit permissions for specific folders within the project, but they don’t actually have edit access on the entire project (Owner lab would prefer Collaborator lab not to have edit access to other data by default, and local settings on every folder are hard to manage).
When Collaborator lab goes to DCA, they can't see the specific folder that they are collaborating on and want to annotate because they don't even see the project. To resolve this, we did just add them with edit permissions on the project, but this might not be what's really wanted/acceptable in other cases.
I think this is related to an existing suggestion somewhere along the lines of "can we add the curator link a folder directly, and if the person has access they can just do the annotation?"
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.
Priority (select one)

[X] Minor ⬇
[ ] Major 📢
[ ] Critical 🆘

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (if applicable, please complete the following information):

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

Additional context
Add any other context about the problem here.

@AmyHeiser for visibility.

Thanks @anngvu.  For the example you provided, would the user have edit access to the individual folder?  you also mentioned that would be hard to manage so want to confirm that this is a desired state with access granted at folder level.


Recent ADO Server versions missing from lookup page

/[collection]/_home/About on our company's Azure DevOps Server instance shows "Version Azure DevOps Server 2022 (AzureDevOpsServer_20230217.5)". The highest About help page version in the Version numbers table is "(AzureDevOpsServer_20220720.1)". Please:

Update the Version numbers table.
Establish a process to update the Version numbers table whenever new Azure DevOps Server versions are released.

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

ID: be9ba678-8bac-f411-06c9-1547f56b7f66
Version Independent ID: be9ba678-8bac-f411-06c9-1547f56b7f66
Content: Look up your Azure DevOps platform and version - Azure DevOps
Content Source: docs/user-guide/lookup-platform-version.md
Service: azure-devops
Sub-service: azure-devops-new-user
GitHub Login: @chcomley
Microsoft Alias: chcomley

Thanks so much for your feedback. I've replaced the table with links to various release notes for each version, which will always be up-to-date. This update should go live shortly. Thanks again!


Add basic test structure and tests for config_flow

Add basic skeleton to support tests (e.g. conftest.py, requirements*)
Add tests for config_flow.py as a first demo test (test coverage for this file -> 100%)
Simplify config_flow.py a bit for readability (including removing checks for mqtt, which is unnecessary since the dependency is in manifest.json).

For the record, output:
(~/src/frigate-hass-integration) $ pytest --cov-report=term-missing --cov=custom_components/frigate tests/
Test session starts (platform: linux, Python 3.8.5, pytest 6.2.4, pytest-sugar 0.9.4)
rootdir: src/frigate-hass-integration
plugins: forked-1.3.0, cov-2.10.1, xdist-2.1.0, aiohttp-0.3.0, test-groups-1.0.3, timeout-1.4.2, sugar-0.9.4, homeassistant-custom-component-0.3.2, requests-mock-1.8.0
collecting ...
 tests/test_config_flow.py ✓✓✓                                                                                100% ██████████

----------- coverage: platform linux, python 3.8.5-final-0 -----------
Name                                         Stmts   Miss  Cover   Missing
--------------------------------------------------------------------------
custom_components/frigate/__init__.py           58     37    36%   38-75, 83-86, 90-93, 98-111, 116-117
custom_components/frigate/api.py                59     39    34%   23-24, 28-29, 33-36, 40-43, 47-48, 52-53, 59-98
custom_components/frigate/binary_sensor.py      68     68     0%   2-128
custom_components/frigate/camera.py            109    109     0%   2-209
custom_components/frigate/config_flow.py        29      0   100%
custom_components/frigate/const.py              24      0   100%
custom_components/frigate/media_source.py      209    209     0%   2-586
custom_components/frigate/sensor.py            167    167     0%   2-299
custom_components/frigate/switch.py             75     75     0%   2-157
custom_components/frigate/views.py             153     99    35%   25-26, 30, 36-42, 55-81, 93-94, 98, 104-110, 123-149, 161-162, 166-175, 181-188, 201-226, 233-269, 274-288
--------------------------------------------------------------------------
TOTAL                                          951    803    16%

Results (0.30s):
       3 passed

@blakeblackshear If you have a moment, can I get a quick signal from you on this approach? I'd like to contribute by continuing to expand the test coverage for some of the other entities in this integration, but before I spend more time on it, would really appreciate a quick thumbs up or thumbs down as to whether or not test coverage/pytest is something you'll be happy to review/merge in the fullness of time. If yes, I'll likely start down the path of preparing a bunch of additional PRs similar to this one to drive the coverage upwards.
If I'm barking up the wrong tree, let me know. Thanks (and thanks for Frigate in general, fantastic work)

This would be great. I would really like to ultimately get this added to the official Home Assistant repo, so I would appreciate if you could try and be consistent with testing/linting patterns that would be expected there to make that process easier.

Absolutely, I am quite familiar with those requirements (you may notice some refactoring in config_flow.py in this PR for that reason).
Great news on the eventual migration to HA Core, I would also definitely be up to help with that having done it before. There are some things I already spotted here that may be structural incompatibilities (e.g. api.py will need to be moved to its own library outside of this repo).
Anyway, thanks for the signal. I'll start slowly stacking up PRs to show my progress, and you can review as time allows. This PR will need to be merged first in line, since it has the testing "skeleton".


Microsoft.AspNet.TestHost.TestServer application base path mismatch

Hey guys,
I'm in the middle of adding a unit-tests for my ASP.NET Core application, and I'm struggling with some difficulty using the TestServer.
My project is based on an N-Tier architecture, so I got Project.Core, Project.Infrastructure etc. Likewise, my ASP.NET is located at Project.Web.UI and my unit tests is under Project.UnitTests.
My Startup.cs class used to read some files, such as configurations (appsettings.json) and certificate (for IdentityServer). For instance:

var builder = new ConfigurationBuilder()
                .AddJsonFile("appsettings.json")

Now, when I'm trying to create a new server, using:
var server = new TestServer()
                  .UseStartup<Project.Web.UI.Startup>();

I'm getting an exception states that the appsettings.json file is missing, since the base path the TestServer is looking for is C:...\Project\Project.UnitTests instead of C:...\Project\Project.Web.UI .
I did manage to bypass this exception, by setting a custom environment name to the test server (e.g. "UnitTests") and then loading the file based on its value:
if (env.EnvironmentName == "UnitTests")
{
    var builder = new ConfigurationBuilder()
                    .AddJsonFile(Path.Combine(env.ApplicationRootPath, "../Project.Web.UI", "appsettings.json"));
}
else
{
    var builder = new ConfigurationBuilder()
                    .AddJsonFile("appsettings.json");
}

However this  approach should be app-wide and be implemented in each case, so I don't think that this is the way to go.
Is there any option to define the application base path in TestServer?

@YahavGB This is normal behavior and was true with Owin self-hosting used for testing in the past when settings existed in an application configuration. Usually one would have a different configuration for test servers executing as part of an integration test (such as running on different port or using a different db) so that you can write tests that are non-destructive to real-world infrastructure.
As an example, I execute integration tests as part of deployment -- and I run destructive integration tests (tests that create or delete data) against a test db.

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


when I train ,  why it becomes another people?

when I train ,  why it becomes another people?

It seems like you're facing an issue during the inference process (using pre-trained authors provided model)

input as a boy image but output as girl


[TO BE TESTED] - (Maybe) fix recycle bin menu opening in standard folder (#1283)

PR to test a fix #1283.
@tsvietOK I'll need your help (as usual 😁) to test if this solves the issue.

Sorry, but i can't reproduce this issue anymore.

@tsvietOK It's ok, I was getting this veery rarely too.
Perhaps @Jaiganeshkumaran according to #1305 you were you having this issue as well? Are you able to repro this with any consistency and check if this PR helps?

Closing this PR now as no one can reproduce the issue.

@tsvietOK The bug is still present although it appears to be very rare #1483
Should we reopen this? But I still have no way of testing if this solves the issue as I can't reproduce it consistently :(

This PR will be reopened only when someone can consistently reproduce the issue and test fix for it.


no entries found

Same. Tried previous versions of 2.9.1 and 2.9.0. Both have save error. Seems like it might be related to the site and how it's parsing the data as a whole. Maybe the site is trying to block attempts at apps like this one from working as intended. Pure speculation on my part.

https://github.com/Dar9586/NClientV2/issues/433


sizing scss

you'll need #735

reviewed this again and I'm not sure what it's for. the set of sizes is now provided by tokens output as
refer to it directly

not needed - was done here:
https://github.com/Vonage/vivid-3/pull/773


Should we use cranelift filetests?

This issue is for discussion of migrating part of our testsuite to cranelift filetests.
Some prerequisite:
Here are some questions we asked in previous sync.
Investigate whether wasmtime filetests allow us to:

Test wasm features that we care about
Test correctness of generated zkasm code
See the diff of codegen to spot weird behaviors during code review
How expensive is it to adopt this solution?

Discussion
Procs:

Filetests allow to have source code not only in wasm, but also in clif. It can be useful for testing some helper-instructions we make, but wasm don't have

Cons:

Cranelift filetests don't supposed to have wasm source code. It is more about testing compilation from clif for isa and other parts of the compiler. For us it mean, that highly likely we will need to have all spectests in clif
This migration don't looks like very obvious and will take time
Speaking about building something what will not deviate cranelift design, simply adding wasm into the filetests will deviate cranelift design even more. We will need to have clif generated from wasm spectest

My answers for questions from the meeting:

Test wasm features that we care about: yes, but will take time to build.
Test correctness of generated zkasm code: same as above.
See the diff of codegen to spot weird behaviors during code review: from my understanding only in CI job log, not in changed files.
How expensive is it to adopt this solution? I think if we want avoid hacks and trully and fully adopt our tests to filetest infra it will take a lot of work:

generate clif from wasm spectest
add zkasm processor as runner to filetest infra
any pitfalls???

And, with this approach, we will lose:

diff between generated code in PR review (i think it will terrible to read huuuge CI job log and find diffs). Or we can add this diff, but it will be hacky and deviate cranelift design
any wasm files from repository

I'm not looking at simply adding wasm to filetest infra, since it is still hacky and need to be rewrited.
My suggestions for now

keep majority of testing infra outside filetest. We can do much less work and still build something which will ok for us, until we will want to merge our code to bytecode alliance repo.
for handwritten tests: if it can be written in wasm -- use our infra. If can't, use filetests.
Also, my suggestions won't work if we are planning merge our code to bytecode alliance in nearest future.

Maybe I forgot some reasons we should use filetests? But in general I think, our own infra can do all tasks we want, and will be much faster to build

@akashin, @nagisa, @mooori, WDYT?

Cranelift filetests don't supposed to have wasm source code. It is more about testing compilation from clif for isa and other parts of the compiler. For us it mean, that highly likely we will need to have all spectests in clif

Can you elaborate why you think this claim holds true? If you look at cranelift/filetests/filetests/isa/*/wasm you can see quite a few tests that generate machine code from input wat already. They are even generated with a script – just like we have one to split up the spectest wast into single modules. So intuitively it would seem quite alright to have wasm-based tests and filetest already has the necessary pieces for this.

until we will want to merge our code to bytecode alliance repo.

My suspicion is that we are not going to pursue this goal approximately ever. At the very best we might provide an externally developed & externally loadable backend. So any of the decisions we make should not be based on the prospects of a merge to upstream.
However we should still make sure we have an easy time integrating with upstream, which may or may not mean that it should remain straightforward to merge upstream/master into our fork.

@MCJOHN974 , Given that we decided to use filetests, can we close this?

Closed by https://github.com/near/wasmtime/pull/215


chain-state-db-size-mb = 8192，this means that the EOS cannot support large data volume applicaiton???

this means that the EOS cannot support large data volume applicaiton???.... since in the end, the state_db will reach the limit...... is there any plan to swap to another DB engine? e.g. put some data onto mongodb instead?? I think that this is actually very serious problem.... as a lot of application requires much more space than 8GB, 1TB, or 4TB.
As far as I know, some of the history data are also stored in state DB.

our dapp had already reached 8GB limit... we tried to increase RAM for the nodeos, but eventually it will fill up again.

chain-state-db-size-mb is a configurable parameter for the size of the shared_memory. The actual limit is around 66GB in the mainnet as of now and it grows with a speed of 1KB/block. EOS can support arbitrary size of memory up to the machine limit. However since the RAM resource is limited, dapp developers need to buy RAM before they can use it. The alternative way is to store data in block logs (or memo, context free actions, etc) which doesn't consume RAM.

we had build a private network ,not use  mainnet ,and mem is 8G.

does mongodb can help solving this problem?can anyone point out.that confuse me a lot.

taokayan, what do you mean by "The alternative way is to store data in block logs (or memo, context free actions, etc) which doesn't consume RAM.",    how can I store data in block logs instead of "state"...
As far as I know, there are two storage tiers in blockchain, one is "state" database, the other is "block" database, or "log" database.

memo, context free actions, etc

/root/.local/share/eosio/nodeos/data/block/block.log is where to store data in disk,but how  to store date in it but not in state?

any update, please?

I guess the original question is already answered.  If you still have the problem and wish to discuss further, please post it in https://eosio.stackexchange.com.


Contact DVSA URL correction

Corrected 'contact DVSA' URL to direct users straight to details. GOV.UK style improvements.

Hi @johnploughman, unfortunately, we cannot accept changes to content as we have an internal process for changing content, including reviewing the impact that any content changes will have. Rest assured that we do assess the content of the application regularly.


Vulkan validation layers for debugging

Added vulkan validation layers.
Value for property debug.geavrf.vulkan determines enabling of validation layers.
debug.geavrf.vulkan 0  [OpenGL]
debug.geavrf.vulkan 1  [Vulkan]
debug.geavrf.vulkan 2  [Vulkan with validation layers]

Getting a build error:
* What went wrong:
Execution failed for task ':framework:transformNativeLibsWithStripDebugSymbolForDebug'.
> A problem occurred starting process 'command 'ndk-bundle/toolchains/mips64el-linux-android-4.9/prebuilt/linux-x86_64/bin/mips64el-linux-android-strip''

Had to add the following to the fw gradle file:
android {
    packagingOptions {
        exclude 'lib/mips/*'
        exclude 'lib/mips64/*'
    }
}

With validation layers, gvr-complexscene crashes:
06-26 07:02:07.838  4507  4619 D vulkan  : vulkan instance layer version (5/5) : x��!!^�� x���!�!I��0x��	 830.76.2944/406847942
06-26 07:02:07.838  4507  4619 E gvrf    : Instance Layer not found: VK_LAYER_LUNARG_image
06-26 07:02:07.838  4507  4619 E gvrf    : VK_CHECK Failure

@liaxim It is actually because of your phone's driver not supporting it. I will log a proper message instead of just saying "Instance layer not found"

Planning to merge on Friday


Introduce Langium 3.0.0

changes the imports to Langium (ESM modules with scope of grammar, lsp, ...)
changes the imports to local files (appending .js)
ESBuild was also updated
ESBuild script files were modernized

Fixed. Thanks @spoenemann !


Use the native useId hook when React 18 is being used

Currently we are using a custom implementation of useId. In version 18 React introduced their own version, we should use this when possible.

how do we handle api differences across major react versions? Do I just do a conditional check for presence of useId?
How do I treat typescript errors warning me useId is not exported by React? Do I update the dependency to 18, or silence them?
How will various projects across the org with lower major versions of react use this lib? Do they need to upgrade, or will the tooling just ignore our version of react?

I got couple questions about the current implementation:

why cant we use the original implementation server side? Since React.useId is meant to be isomorphic afaik.
Why is setDefaultId it in useEffect instead of useState initialisation function?

export function useId(idOverride?: string): string {
  const [defaultId, setDefaultId] = useState(idOverride);
  const id = idOverride || defaultId;
  useEffect(() => {
    if (defaultId == null) {
      // Fallback to this default id when possible.
      // Use the random value for client-side rendering only.
      // We can't use it server-side.
      setDefaultId(`uitk-${Math.round(Math.random() * 1e5)}`);
    }
  }, [defaultId]);
  return id as string;
}

cc @joshwooding


Render Darkvision as Greyscale

https://discordapp.com/channels/170995199584108546/517504137403826177/517513141551366165
Have a “toggle grayscale” for the player vision, so it can better represent Darkvision (D&D). Ideally, though, players with Darkvision enabled would have a separate vision circle for dim and dark parts of the map.

Could this be closed due to the release of version 10 ? https://foundryvtt.com/releases/10.283.
@arbron @aaclayton

I think we can close this in favor of the issue #1166 that was opened.


Fix extra CI checks on macOS

Alternative to https://github.com/astral-sh/uv/pull/2729 since we're having problems with the Homebrew Python.

lovely, seems like this might be the same issue i was looking into just now?
Run python3 scripts/check_system_python.py --uv ./uv
INFO: Checking that `pylint` isn't installed.
WARNING: Package(s) not found: pylint
INFO: Installing the package `pylint`.
error: The interpreter at /opt/homebrew/opt/python@3.[12](https://github.com/astral-sh/uv/actions/runs/8488022849/job/23256549997?pr=2636#step:7:13)/Frameworks/Python.framework/Versions/3.12 is externally managed, and indicates the following:

(i was confused since the yaml file seems to install 3.8)

Yeah sorry about the churn something changed about the runners it seems?

Yeah sorry about the churn something changed about the runners it seems?

no worries - seems like github may not support pinning the runner version/image (https://github.com/actions/runner-images/issues/2894) 😢


cloud remote launching cannot run blue and gold from the same machine

It's just roslaunching but it has several failures.
It has tap0 hard coded.  Which gives the ROS_HOSTNAME and ROS_MASTER_URI incorrectly
It uses the same temp file for the launch file.

I've found an issue that network.py is crashing from ap_network. It looks like it needs to be passed the network device.
[rospy.client][INFO] 2017-01-13 03:01:07,572: init_node, name[/network], pid[16043]
[xmlrpc][INFO] 2017-01-13 03:01:07,572: XML-RPC server binding to <IP_ADDRESS>:0
[rospy.init][INFO] 2017-01-13 03:01:07,573: ROS Slave URI: [http://<IP_ADDRESS>:46097/]
[xmlrpc][INFO] 2017-01-13 03:01:07,573: Started XML-RPC server [http://<IP_ADDRESS>:46097/]
[rospy.impl.masterslave][INFO] 2017-01-13 03:01:07,573: _ready: http://<IP_ADDRESS>:46097/
[xmlrpc][INFO] 2017-01-13 03:01:07,573: xml rpc node: starting XML-RPC server
[rospy.registration][INFO] 2017-01-13 03:01:07,574: Registering with master node http://localhost:11439
[rospy.init][INFO] 2017-01-13 03:01:07,673: registered with master
[rospy.rosout][INFO] 2017-01-13 03:01:07,673: initializing /rosout core topic
[rospy.rosout][INFO] 2017-01-13 03:01:07,681: connected to core topic /rosout
[rospy.simtime][INFO] 2017-01-13 03:01:07,685: /use_sim_time is not set, will not subscribe to simulated time [/clock] topic
[rosout][CRITICAL] 2017-01-13 03:01:07,717: Couldn't establish IP addressing information
[rospy.core][INFO] 2017-01-13 03:01:07,788: signal_shutdown [atexit]
[rospy.impl.masterslave][INFO] 2017-01-13 03:01:07,797: atexit

I think it can be resolved by passing through the net_dev with the --device command line argument in ap_master's master.launch. And we need to differentiate between the ocu network device and the autopilot network device.

PR for above fix here: https://gitlab.nps.edu/sasc/autonomy-payload/merge_requests/11

I was able to work around swarm_commander in ed6da478973f608af96a20e6dfbb64e62436c166 and resolved a typo which seemed to be triggering the invalid network device


Feature: Make the chart available with a chart repository

It would be useful to add an index.yaml to this repo, so tools like Kustomize could make use of this chart.
A GitHub Action like Chart Releaser Action would automate this easily.

thanks for the suggestion @philipsens. I will find time for this, but PRs are also welcome.


Module not appearing

Hi,
it seems the MMM-CalendarExt module is not loading, I cannot see anything.
I'm using the Raspberry Pi 1, with Node version > 10. Master branch is checked out.
Config is the example config. There are no errors during the start.
Any idea how to debug further or why it is not loading?
Thanks,
Rene

Can you share me the ical file or url and the configuration?

Thanks for the fast response!
I attached you the config file.
config.txt

Hmmm.. Did you really want to see url: "http://mlb.am/tix/mariners_schedule_full"? That URL was just a sample. It is out-dated so there is no event to show. (You can see that information on your log)
Try another URL what you really want to use.

Finally a little bit of success. I looked for another calendar module which is up to date. When I connect from my computer in Google Chrome the calendar shows up!
However, on the raspberry the module is still not loading. Im using the kweb browser, maybe there are some features in MMM-CalendarExt which are currently not supported by this browser? Tomorrow I will check with chromium or another browser. Or do you have any other idea?

It's a publicy available .ics file for public holiday in Germany:
http://feed.kleiner-kalender.de/ical/c48-feiertage-in-deutschland.ics

Well, I used your configuration with your url, (I removed other modules, because I have no private key/token for using them) and add show:["monthly"] because there is no near german holiday in these days.

And it shows everything as expectation.

I doubt your other modules(which I removed) might have some errors/bugs to prevent executing my module.

Nevermind... It justed need some pretty long time (2-5 minutes) until it shows up. So we can close this issue. I really appreciated your help!

It means your REAL calendar is somewhat huge or in the slow server, so parsing of events is not finished before trying to draw at first turn. It will retry to draw on the next interval.

Ok, makes sense. But that's find I can accept this :-)


Make "sort_keys" option false by default

The "sort_keys" option may be useful to some people, but it's dangerous because it changes the file from its original structure. If we need to check if the file generated is correct, and when the keys are sorted, we get a different file from what is generated.

If we need to check if the file generated is correct,

Not an issue. The file generated will be correct. This plugin has been thoroughly tested and while it has bugs, correctly parsing compliant JSON is not one of them.

when the keys are sorted, we get a different file from what is generated

Yes, but every language implements JSON objects in such a way that the order of the keys does not matter. Sorting keys ensures consistency because it means that if you have two JSON objects, they will always have the same string representation.


Updatet credits - The most unnecessary pull request in the history of the plugin

Replaced my old name with my new one

Now you're twice there 🤡

Double the quality. 🤡
Didn't have your change, on the credit file, already enabled on my github.


remove DisruptorProvider#onData(final Consumer function)

Fixes #2803
delete method of DisruptorProvider#onData(final Consumer function).
this method not called in anywhere.

Codecov Report

Merging #2804 (ef174ce) into master (387f0f9) will decrease coverage by 0.48%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2804      +/-   ##
============================================
- Coverage     60.20%   59.72%   -0.49%
+ Complexity     4465     4436      -29
============================================
  Files           692      692
  Lines         19952    19950       -2
  Branches       1783     1783
============================================
- Hits          12013    11915      -98
- Misses         6924     7026     +102
+ Partials       1015     1009       -6

Impacted Files
Coverage Δ

...n/alibaba/dubbo/cache/AlibabaDubboConfigCache.java
43.75% <0.00%> (-30.00%)
:arrow_down:

...gin/apache/dubbo/cache/ApacheDubboConfigCache.java
43.03% <0.00%> (-27.85%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
87.50% <0.00%> (-12.50%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
54.25% <0.00%> (-9.58%)
:arrow_down:

...apache/shenyu/admin/utils/ShenyuDictH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...ache/shenyu/admin/utils/PluginHandleH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...va/org/apache/shenyu/common/timer/TimingWheel.java
83.33% <0.00%> (-7.15%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

... and 28 more

Continue to review full report at Codecov.

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


fix(cmd): log format error

Log line isn't correctly formatted.
FTI-4801

I think that we may have more log format errors in cmd, could we have a complete check?

I think that we may have more log format errors in cmd, could we have a complete check?

Checked. No others in cmd.


Community member page: file uploads

https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=1315%3A18469