Add option to ignore (filter) short lasting nonwear during nighttime

Fixes #1218 by adding functionality to GGIR part 2 to ignore short nonwear during the night. I did it there, such that all GGIR parts will benefit from it. I hope that the documentation updates made in this PR explain sufficiently what it does and how to use the functionality.
Updates include:

Two new parameters to control the nonwear filter
Two new variables in output to monitor whether the nonwear filter was used.
One new function to perform the nonwear filtering which is called from g.weardec inside g.impute.

Checklist before merging:

[ ] Existing tests still work (check by running the test suite, e.g. from RStudio).
[x] Added tests (if you added functionality) or fixed existing test (if you fixed a bug).
[x] Clean code has been attempted, e.g. intuitive object names and no code redundancy.
[ ] Documentation updated:

[x] Function documentation
[ ] Chapter vignettes for GitHub IO
[ ] Vignettes for CRAN

[ ] Corresponding issue tagged in PR message. If no issue exist, please create an issue and tag it.
[x] Updated release notes in inst/NEWS.Rd with a user-readable summary. Please, include references to relevant issues or PR discussions.
[ ] Added your name to the contributors lists in the DESCRIPTION file, if you think you made a significant contribution.
[x] GGIR parameters were added/removed. If yes, please also complete checklist below.

If NEW GGIR parameter(s) were added then these NEW parameter(s) are:

[x] documented in man/GGIR.Rd
[x] included with a default in R/load_params.R
[x] included with value class check in R/check_params.R
[x] included in table of vignettes/GGIRParameters.Rmd with references to the GGIR parts the parameter is used in.
[x] mentioned in NEWS.Rd as NEW parameter

If GGIR parameter(s) were deprecated these parameter(s) are:

[ ] documented as deprecated in man/GGIR.Rd
[ ] removed from R/load_params.R
[ ] removed from R/check_params.R
[ ] removed from table in vignettes/GGIRParameters.Rmd
[ ] mentioned as deprecated parameter in NEWS.Rd
[ ] added to the list in R/extract_params.R with deprecated parameters such that these do not produce warnings when found in old config.csv files.

I now realise that a limitation of this PR is that it will also run the qwindow functionality even if the user is not interested in this.
I think I have a solution for this and will make a commit for it in the next hour:
If the diary name includes the word "filteronly" then qwindow it is reset to its default c(0, 24) after it was used for the filter functionality. This then applies in both part 2+5, and guarantees freedom for the user to do any of the following:

use qwindow without filtering
do not use qwindow but filter
use qwindow and filtering


Need SiteId to be added to a Parameter for Add File

With the addition of sites.selected permissions, its vital to be able to use app integrations that don't have full access to SharePoint Online.
This creates an issue because the issue query of the SiteId requires more permissions that a company would be comfortable providing. As an example this error comes up if you are granting least privilege:
"url": "https://graph.microsoft.com/v1.0/sites/contoso.sharepoint.com:/?$select=id",
"method": "get",
"headers": {
"common": {
"Accept": "application/json, text/plain, /"
},
"delete": {},
"get": {},
"head": {},
"post": {
"Content-Type": "application/x-www-form-urlencoded"
},
"put": {
"Content-Type": "application/x-www-form-urlencoded"
},
"patch": {
"Content-Type": "application/x-www-form-urlencoded"
},
"user-agent": "NONISV|SharePointPnP|CLIMicrosoft365/3.12.0",
"accept-encoding": "gzip, deflate",
"accept": "application/json;odata.metadata=none",
"responseType": "json",
"decompress": true
}
Request error:
{
"status": 403,
"statusText": "Forbidden",
"headers": {
"connection": "close",
"date": "Thu, 26 Aug 2021 01:41:44 GMT",
"content-type": "application/json",
"cache-control": "no-cache",
"transfer-encoding": "chunked",
"vary": "Accept-Encoding",
"strict-transport-security": "max-age=31536000",
"request-id": "d11b9c8e-08b1-457e-8f0d-8e2d688c756d",
"client-request-id": "d11b9c8e-08b1-457e-8f0d-8e2d688c756d",
"x-ms-ags-diagnostic": "{"ServerInfo":{"DataCenter":"North Central US","Slice":"E","Ring":"5","ScaleUnit":"000","RoleInstance":"CH01EPF000076C8"}}"
},
"error": {
"error": {
"code": "accessDenied",
"message": "Access denied",
"innerError": {
"date": "2021-08-26T01:41:45",
"request-id": "d11b9c8e-08b1-457e-8f0d-8e2d688c756d",
"client-request-id": "d11b9c8e-08b1-457e-8f0d-8e2d688c756d"
}
}
}
}
Error: Access denied

I now get a 403 on creating the upload session:
Searching for drive with a URL ending with /Shared%20Documents...
Resolved URL https://graph.microsoft.com/v1.0/sites/company.sharepoint.com,b358b16f-9af3-46d1-bc2d-ff62c2f7aa35,bfebfc4e-1fed-4956-89da-438f3b17092a/drives/b!b7FYs_Oa0Ua8Lf9iwveqNU7867_tH1ZJidpDjzsXCSq7z1r15zC6QKVYT0eNveJi/root:/Jon/MacProfile.xml
Existing access token XXXXXX-r8JO08Ru6XUzOviQ still valid. Returning...
Request:
{
"headers": {
"common": {

  "Accept": "application/json, text/plain, */*"

},

"delete": {},

"get": {},

"head": {},

"post": {

  "Content-Type": "application/x-www-form-urlencoded"

},

"put": {

  "Content-Type": "application/x-www-form-urlencoded"

},

"patch": {

  "Content-Type": "application/x-www-form-urlencoded"

},

"user-agent": "NONISV|SharePointPnP|CLIMicrosoft365/4.2.0-beta.a06522d",

"accept-encoding": "gzip, deflate",

"accept": "application/json;odata.metadata=none",

"authorization": "Bearer

},
"decompress": true,
"responseType": "json",
"url": "https://graph.microsoft.com/v1.0/sites/company.sharepoint.com, /drives/XXXXX/root:/Jon/MacProfile.xml:/createUploadSession",
"method": "post"
}
Request error:
{
"status": 403,
"statusText": "Forbidden",
"headers": {
"connection": "close",

"date": "Tue, 02 Nov 2021 19:14:08 GMT",

"content-type": "application/json",

"cache-control": "no-cache",

"transfer-encoding": "chunked",

"vary": "Accept-Encoding",

"strict-transport-security": "max-age=31536000",

"request-id": "b53f2de5-7c52-49e8-999c-aeb05ca6ce72",

"client-request-id": "b53f2de5-7c52-49e8-999c-aeb05ca6ce72",

"x-ms-ags-diagnostic": "{\"ServerInfo\":{\"DataCenter\":\"North Central US\",\"Slice\":\"E\",\"Ring\":\"5\",\"ScaleUnit\":\"000\",\"RoleInstance\":\"CH01EPF0000B0A0\"}}"

},
"error": {
"error": {

  "code": "accessDenied",

  "message": "Access denied",

  "innerError": {

    "date": "2021-11-02T19:14:09",

    "request-id": "b53f2de5-7c52-49e8-999c-aeb05ca6ce72",

    "client-request-id": "b53f2de5-7c52-49e8-999c-aeb05ca6ce72"

  }

}

}
}
Error: Access denied
From: Waldek Mastykarz @.>
Date: Friday, October 29, 2021 at 1:58 PM
To: pnp/cli-microsoft365 @.>
Cc: Towles, Jonathan @.>, Mention @.>
Subject: Re: [pnp/cli-microsoft365] Need SiteId to be added to a Parameter for Add File (#2655)
CAUTION/ALERT! This message came from outside Freddie Mac – don’t click links or attachments unless you’re sure they’re safe
Hey @jjtowleshttps://urldefense.com/v3/__https:/github.com/jjtowles__;!!EmXWOUHo7x8!aUhkECowxAmRhOUExEHTWxeAvWEAD9yDUaynZarJMamuelKHxf1Zo1P7kXPtK65cdsDM93sr$, we've just released a preview version with the siteUrl option. You can install it by running npm i -g @@.*** Thank you once again for the suggestion 👏
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/pnp/cli-microsoft365/issues/2655*issuecomment-954938741__;Iw!!EmXWOUHo7x8!aUhkECowxAmRhOUExEHTWxeAvWEAD9yDUaynZarJMamuelKHxf1Zo1P7kXPtK65cdputllGi$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AVLJCUNP55CK5G7IKUZDGSTUJLVF5ANCNFSM5C5KVSZQ__;!!EmXWOUHo7x8!aUhkECowxAmRhOUExEHTWxeAvWEAD9yDUaynZarJMamuelKHxf1Zo1P7kXPtK65cdnJvahaV$.
Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.com/v3/__https:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!EmXWOUHo7x8!aUhkECowxAmRhOUExEHTWxeAvWEAD9yDUaynZarJMamuelKHxf1Zo1P7kXPtK65cdkqZaKuU$ or Androidhttps://urldefense.com/v3/__https:/play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!EmXWOUHo7x8!aUhkECowxAmRhOUExEHTWxeAvWEAD9yDUaynZarJMamuelKHxf1Zo1P7kXPtK65cdvqcmzon$.
The information transmitted in this e-mail is for the exclusive use of the person or entity to which it is addressed and may contain legally privileged or confidential information. If you are not the intended recipient of this e-mail, you are prohibited from reading, printing, duplicating, disseminating or otherwise using or acting in reliance upon this information. If you have received this information in error, please notify the sender at Freddie Mac immediately, delete this information from your computer and destroy all copies of the information.

Could you please share with

I pulled the documentation on the Create Upload Session and that’s clearly the issue:
@.***
Jon Towles
Architecture, Sr Tech Lead
Infrastructure Architecture | End User Computing
<EMAIL_ADDRESS>Mobile:<PHONE_NUMBER>
[Freddie Mac logo]
From: Waldek Mastykarz @.>
Sent: Wednesday, November 3, 2021 5:27 AM
To: pnp/cli-microsoft365 @.>
Cc: Towles, Jonathan @.>; Mention @.>
Subject: Re: [pnp/cli-microsoft365] Need SiteId to be added to a Parameter for Add File (#2655)
CAUTION/ALERT! This message came from outside Freddie Mac – don’t click links or attachments unless you’re sure they’re safe
Reopened #2655https://urldefense.com/v3/__https:/github.com/pnp/cli-microsoft365/issues/2655__;!!EmXWOUHo7x8!YVtYBy1KkLD6O3lpf_PnQ9W3TIDVxSt6f4DUYGAvJN_AyTlCJPjs95K1RxgoojTxPk3lb6ai$.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/pnp/cli-microsoft365/issues/2655*event-5560515149__;Iw!!EmXWOUHo7x8!YVtYBy1KkLD6O3lpf_PnQ9W3TIDVxSt6f4DUYGAvJN_AyTlCJPjs95K1RxgoojTxPo8ESIvS$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AVLJCUOTWIN2I2RPNBBPZXTUKD55ZANCNFSM5C5KVSZQ__;!!EmXWOUHo7x8!YVtYBy1KkLD6O3lpf_PnQ9W3TIDVxSt6f4DUYGAvJN_AyTlCJPjs95K1RxgoojTxPnMr9FV7$.
Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.com/v3/__https:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!EmXWOUHo7x8!YVtYBy1KkLD6O3lpf_PnQ9W3TIDVxSt6f4DUYGAvJN_AyTlCJPjs95K1RxgoojTxPjQmkScI$ or Androidhttps://urldefense.com/v3/__https:/play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!EmXWOUHo7x8!YVtYBy1KkLD6O3lpf_PnQ9W3TIDVxSt6f4DUYGAvJN_AyTlCJPjs95K1RxgoojTxPu9Awliu$.
The information transmitted in this e-mail is for the exclusive use of the person or entity to which it is addressed and may contain legally privileged or confidential information. If you are not the intended recipient of this e-mail, you are prohibited from reading, printing, duplicating, disseminating or otherwise using or acting in reliance upon this information. If you have received this information in error, please notify the sender at Freddie Mac immediately, delete this information from your computer and destroy all copies of the information.

Hey @jjtowles, could it be that something was removed in your reply?

Closing due to lack of reply


Separate operator and controller names with _

Description
/ broke using the combined names in label values. Thanks to @maltesander for catching this.

Review Checklist

[x] Code contains useful comments
[x] (Integration-)Test cases added (or not applicable)
[x] Documentation added (or not applicable)
[x] Changelog updated (or not applicable)
[x] Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

bors r+


use Date in TikiSdk and int in request

As a developer, I need to work with native classes for date and time.
All native models should use native Date/Calendar and the request/response objects should convert to epoch milliseconds.
Acceptance criteria
1 - Use Calendar/Date in all objects
2 - Conversion in request and response objects

Needs to write a custom JSON Adapter for converting from/to Long


ARTEMIS-2645 continue refactoring

Make CLI resources more test friendly, and fix some test race
conditions.

I would rename the first Line of the commit to "Make CLI Resources More test friendly"

@clebertsuconic, fair enough.


Fix for "TIMESTAMP format" error

Issue: #145
Description of changes:
Fixes the timestamp formatting code so that it doesn't generate "24" as an hour.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi mate. Thanks for the quick fix!
However, I searched a bit and found this PR with the nicest solution: https://github.com/home-assistant/frontend/pull/11426/files
Solution: use "en-u-hc-h23" as locale instead of en-US
The u-hc-h23 suffix means:

u: subtag for local extension
hc: hourcycle
hc23: the hourcycle wherein midnight is 00 not 24

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/Locale/hourCycle
Wanna change the PR?

Will do! Thanks for looking into this so quickly :)

Done!


SelectableText doesn't follow focus logic

Steps to Reproduce

Execute flutter run on the code sample
Select one region of text
select the other region of text

Expected results:
The first region of text would be unselected
Actual results:
Both remain selected even though unfocus is called

Code sample

import 'package:flutter/material.dart';
import 'package:flutter/gestures.dart';

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

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

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  late FocusNode nodeReg1;
  late FocusNode nodeReg2;

  @override
  initState() {
    nodeReg1 = FocusNode();
    nodeReg2 = FocusNode();

    nodeReg1.addListener(() {
      if (nodeReg1.hasFocus) {
        nodeReg2.unfocus();
      }
    });
    nodeReg2.addListener(() {
      if (nodeReg2.hasFocus) {
        nodeReg1.unfocus();
      }
    });

    super.initState();
  }

  @override
  void dispose() {
    nodeReg1.dispose();
    nodeReg2.dispose();

    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(
          title: Text(widget.title),
        ),
        body: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          crossAxisAlignment: CrossAxisAlignment.stretch,
          children: [
            Center(
                child: SelectableText(
              "This is the first region",
              focusNode: nodeReg1,
              autofocus: true,
            )),
            Center(
                child: SelectableText(
              "This is the second region",
              focusNode: nodeReg2,
              autofocus: true,
            )),
          ],
        ));
  }
}

Logs

Analyzing flutter_min...
No issues found! (ran in 1.0s)

[✓] Flutter (Channel stable, 3.3.10, on macOS 13.1 22C65 darwin-arm, locale en-US)
    • Flutter version 3.3.10 on channel stable at /Users/user/LocalFrameworks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 135454af32 (8 days ago), 2022-12-15 07:36:55 -0800
    • Engine revision 3316dd8728
    • Dart version 2.18.6
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/user/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)

[✓] VS Code (version 1.74.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.56.0

[✓] VS Code (version 1.68.1)
    • VS Code at /Volumes/Ventura/Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.56.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.1 22C65 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 108.0.5359.124

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Triage report
I can reproduce this issue with Master (3.7.0-13.0.pre.53). Playing around with autofocus, removing and adding it, doesn't solve the issue.

cc @chunhtai


Make metaTuple backed by []byte

This is a version change since there was a bug in the old code; it
didn't include the numLeaves in the rolling hash.
Towards #2270

It is below the fold (before and after) for csv-import and csv-export
The motivation was consistency


Volume plot updates with VTK 9, how to add to docs

Describe what needs to be documented.
There are significant changes to Volume plot with VTK 9.  @ARSanderson updated the docs in the old branch we were using for the VTK 9 port.
I want to integrate his changes into develop, but am a bit uncertain how to proceed since we are still supporting VTK 8.
Can't yet tie it to a version number (EG Volume plot for Visit xxx and older, Volume plot for Visit xxx and newer)
Suggestions?

Resolved #19042.


Complete testing of Akamai XFF header in test.simplereport.gov.

Completion email sent to Akamai partners on Tuesday, 13 June 2023.


Docker privileged mode is incompatible with user namespaces

Same as described at https://github.com/weaveworks/scope/pull/2161

The --userns=host option is not available with the Docker 1.10 client we are currently bundling, so we need some complex footwork to get round this.
Maybe #2545 ?

Has anyone ever got weave working with docker user namespace remapping?


Key Binding not honored when cursor in search or replace box

Issue Type: Bug
Set up user keybindings with Ctrl+shift+S to execute the Save All command in place of the default keybinding.  Do a Find in Files and, with the cursor in the search field, press Ctrl+shift+S.  The Save As dialog appears rather than saving all open files.
VS Code version: Code 1.21.1 (79b44aa704ce542d8ca4a3cc44cfca566e7720f1, 2018-03-14T14:46:47.128Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-2720QM CPU @ 2.20GHz (8 x 2195)

Memory (System)
7.90GB (1.22GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

numbered-bookmarks
ale
0.11.1

html-class-suggestions
And
1.0.7

django-html
bib
1.2.0

vscode-eslint
dba
1.4.7

python
ms-
2018.2.1

code-settings-sync
Sha
2.9.0

Reproduces without extensions

Does your keybinding have a when clause? Can you post the keybinding here?

// Place your key bindings in this file to overwrite the defaults
[
    { "key": "ctrl+shift+d",            "command": "editor.action.deleteLines",
                                            "when": "editorTextFocus" },
    { "key": "ctrl+alt+shift+d",        "command": "workbench.view.debug" },
    { "key": "ctrl+shift+up",           "command": "editor.action.moveLinesUpAction",
                                            "when": "editorTextFocus" },
    { "key": "ctrl+shift+down",         "command": "editor.action.moveLinesDownAction",
                                            "when": "editorTextFocus" },
    { "key": "ctrl+d",                  "command": "editor.action.copyLinesDownAction",
                                            "when": "editorTextFocus" },
    { "key": "ctrl+alt+shift+b",        "command": "workbench.action.toggleSidebarVisibility" },
    { "key": "ctrl+b",                  "command": "editor.action.addSelectionToNextFindMatch",
                                            "when": "editorFocus" },
    {"key": "ctrl+shift+s",             "command":
    "workbench.action.files.saveAll",
                                            "when":
                                            "editorFocus" },
    {"key": "ctrl+k",                   "command":
    "workbench.action.files.saveAs",
                                            "when":
                                            "editorFocus" },
]

Ah, I see.


Hide the FAB when the menu button is clicked

Hi, i success made the 3 menu button in FAB button based on the tutorial, but i would like to know is there possible to hide the Main FAB when the others sub menu fab is clicked.
final FloatingActionMenu menu1 = (FloatingActionMenu) findViewById(R.id.menu1);
fab1 = (FloatingActionButton) findViewById(R.id.fab1);         fab2 = (FloatingActionButton) findViewById(R.id.fab2);         fab3 = (FloatingActionButton) findViewById(R.id.fab3);
meant when the fab1, fab2 or fab3 is clicked, the menu1 button will hide all the menu button, user no need to click the menu1 button for hide the others.
Thank in advance.

Just do like this for every button inside the menu. You have to set the click listener anyway, just add one more line to close the menu.
fab1.setOnClickListener(() -> {
    menu1.close(true);
    // do what you want here
});

thank you very much, but i have done this yesterday. I think this issue is going to help the others.

Hi @kakai248.. I use your code but not only the fab, the menu button is also gone, user cannot show it again.


Callable ptransform cleanup

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: aaltay@

R: @aaltay  [github tags in front, not after, user name]

LGTM


Support input() function

It is important that Codon supports as many Python's features as possible.
But input() has yet to be implemented in Codon. So I made this PR to add it.
input() in Python: https://docs.python.org/3/library/functions.html#input
in develop branch (before merging this PR):
$ cat input.codon
print(input())
$ codon run input.codon
input.codon:1:7-12: error: name 'input' is not defined

in the branch for this PR, it works as well as Python
(test right under the command is manually input)
$ codon run input.codon
test
test

https://github.com/exaloop/codon/issues/96 is solved by this PR

Closes #96

This is finally added here: https://github.com/exaloop/codon/commit/5e2d04188ebadb06a8d2d36822f7d0f414d99571


External resource management using operator

Type of question
Open question
Question
What did you do?
Hello. This follows a discussion we had during the kubecon 2022 in Detroit with @asmacdo from RedHat.
More than an issue it is a Philosophical question about external resource management using an operator. It could be nice to have a consensus and recommendations in the documentation.
When using a standard operator it is possible to watch resources that are owned and managed by the controller so that reconcile loop is called on the child resource's events (ie: Pod crash)
More and more companies are trying to manage external resources using operators:

Strimzi standalone topic operator manages topics on any Kafka Cluster
AWS controller for Kubernetes provisions resources in AWS such as EC2 instances / RDS databases...
Azure Service Operator is doing the same for Azure cloud.

With external resources it is not possible to trigger the reconcile loop with standard Events to quickly react and reflect your desired state (on system fault for example).
ACK made the choice to stop reconciliation when the desired state has been met and triggers a reconciliation after 10 hours. Therefore if RDS is unavailable during this period (or drifted), no reconcile loop is triggered.
- https://aws-controllers-k8s.github.io/community/docs/community/how-it-works/#drift-detection-and-remediation
- https://github.com/aws-controllers-k8s/runtime/blob/13496a878abc8dd9f784d84c6452638a0d736df3/pkg/runtime/reconciler.go#L880
- https://github.com/aws-controllers-k8s/runtime/blob/13496a878abc8dd9f784d84c6452638a0d736df3/pkg/runtime/reconciler.go#L50
- https://github.com/aws-controllers-k8s/ec2-controller/blob/main/pkg/resource/instance/manager_factory.go#L85
But what if you need to react quickly to such a situation ?
What did you expect to see?

If the resource fails for an unexpected reason (system fault), I want to react quickly and update the Custom Resource status

status:
  phase: Failed
  conditions:
    - type: Ready
      status: False
      reason: Dead
      message: Resource state transitioned from Running to Dead

If the resource fails for an unexpected reason I may want to restart it.

If the resource succeeded I want to update the status:

status:
  phase: Succeeded

Status could be enriched with relevant information such as a log url, an id...

What did you see instead? Under which circumstances?
N/A
Environment
N/A
Additional context
Possible solutions
Force reconciliation
Reconciliation can be forced using RequeueAfter even when the desired state has been met.
Rabbitmq allows to force reconciliation with environment variable SYNC_PERIOD
Use an Agent Pod
The Operator owns Pods.
The Operator launch an Agent Pod to poll the external resource, report interesting information in the Custom Resource Status.
If the External resource fails (or drift), the Agent Pod container exits with a status != 0 allowing the Operator having its reconcile loop triggered in order to take actions (ie: Relaunch the resource and reschedule Pod).
Here 2 sub-possibilities
Agent is only polling
The Operator creates, updates, deletes the external resource.
The Agent Pod monitors the external resource status and reports it in the Custom Resource Status.
Agent is creating the external resource and creates it
The Operator creates, update and delete the Agent Pod.
When starting, the Agent Pod creates/updates the external resource.
The Agent Pod monitors the external resource status and report changes in the Custom Resource Status.
The Agent Pod preStop deletes the external resource

Hi @speedfl, thanks for bringing this up. This is an interesting topic - triggering reconcile from events which are external to the cluster. We had a discussion about this and Michael Hrivnak (@mhrivnak) suggested a solution of using controller-runtime's source.Channel (https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/source#Channel). The idea is use the Channel source and push GenericEvents into the queue that eventually trigger reconciliation. We could have a webhook setup that triggers a generic event during a change happening in an external resource that can then be passed into controller-runtime's watch (This way we need not spend resources in having an agent pod to poll the external resource continuously). Eg:
err := ctrl.Watch(
		&source.Channel{Source: events},
		&handler.EnqueueRequestForObject{},
	)

There is some documentation regarding the same: https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/source#Channel, and some piece of code that explains its usage in simple tests: https://github.com/kubernetes-sigs/controller-runtime/blob/master/pkg/source/source_test.go#L294-L338

Thanks Varsha for relaying that.
A channel source is indeed a good tool for this circumstance. You can either listen for some inbound notification, such as listening on an http endpoint or an external message bus, or you can use a goroutine to poll the state of an external resource. The choice will depend on the capabilities and nature of the external entity you want to watch. Either way, this can happen in your controller process, with no need for an extra Pod.

As it happens I --just-- write up another issue about this.
https://github.com/operator-framework/operator-sdk/issues/6122
Thanks for starting the conversation, folks can feel free to close my issue.

@varshaprasad96 This is a very interesting idea. Have you explored it any further or do you know if there are new insights in how to deal with external state?

I think I have a working mechanism.
In the controller I have to add WatchesRawSource(&source.Channel{Source: Events}, &handler.EnqueueRequestForObject{}). to the builder like this:
import "sigs.k8s.io/controller-runtime/pkg/event"
var Events = make(chan event.GenericEvent)
// SetupWithManager sets up the controller with the Manager.
func (r *MemcachedReconciler) SetupWithManager(mgr ctrl.Manager) error {
	return ctrl.NewControllerManagedBy(mgr).
		For(&cachev1alpha1.Memcached{}).
		WatchesRawSource(&source.Channel{Source: Events}, &handler.EnqueueRequestForObject{}).
		Complete(r)
}

Then in cmd/main.go I kick off a http listener on a go routine that puts GenericEvents on the Events channel based on webhooks.
		typeNamespacedName := types.NamespacedName{
			Name:      "target-memcached",
			Namespace: "target-namespace",
		}
		memcached := &cachev1alpha1.Memcached{}
                // do a get request using the k8s client of the controller-manager
		err := h.Client.Get(ctx, typeNamespacedName, memcached)
		if err != nil {
			<error handling>
		}
		controller.Events <- event.GenericEvent{
			Object: memcached,
		}

Not sure if there are better approaches but I am pretty happy with the result.


Blockchain.GetHeader(Blockchain.GetHeight()).Timestamp returns 0

Is there any way for the debugger to get actual now time?

Please use the latest repo. The timestamp is working there in latest version.


Scale using number of cores or memory with ClusterManager

Fixes #130, Supersedes #136 waiting for https://github.com/dask/distributed/pull/2256.
Based on #183.

I will probably merge this one rapidly too, as #183.


Expand settings

This closes most of #37, and adds a few fields to /settings:

@michaeljosephrosenthal Sorry for delay (you should ping me when I forget to merge the things)! :smile: Testing now.


Impala with kerberos support.

We'd like to know what your use case is, and how we can help that specific use case.

Sure!
We have a Hadoop cluster running and right now the only viable way to query it is using HuE + Impala, the cluster has something like 200+ users.
We can also connect using the Impala ODBC driver for a connection with Python or R.
Having this will help us a lot to streamline the process when developing in Python or R and having the option to query Impala right from the editor.
We have been looking for a solution like this for years! We could also use some kind of connection using the Impala ODBC as the connector (Which I think would be a lot easier to add).
I could provide the querys for describing the DB, showing the tables and so on.
Thanks!


Merge release/2.1 into release/2.2

Should be the same change for release/2.2

@anurse can you merge this?

Can you cite the original 2.1 PR and give an review approval (I actually do need that to merge on these branches ;))?

I can't review my own PR... here is my original PR: https://github.com/aspnet/AspNetCore/pull/11616

I didn't actually verify this on aspnetci yet. If you'd prefer I do that, I can, but I believe it should be exactly the same change.

I can't review my own PR

Lol. Ok, I can override it.

Merging as this is infrastructure only and the change was previously approved for 2.1


New home screen concept

Being the home screen the first impression that the user has of the platform, I considered that it should be more captivating for both new and frequent users. The thumbnails shown are those of up to 150 most popular uploads. In this way, the initial screen would be a kind of incentive to explore the rest of the website's content.
Short demo here:
https://newtube.app/user/carlostojal/jzuB5Ff

Very cool! I will test this on mobile but otherwise I think I can merge that right away, nice work!

Is there a way to be 'zoomed out' of the sphere after the first animation? So you're outside of it? Right now when it starts you begin in the centre of the sphere and the uploads are "inside out"

Yes, it's possible.

Done :)

I'm sure you put some work into it but please provide an env variable to disable this. I don't want it.

@chovy changed it now. Just set HOME_SPHERE_ENABLED to false in .env.settings to get the old home screen.

Let's do it this way:
Let's have a route for app.get(/globe)
That will render that globe/sphere as a pug file
Then we will leave home as it is , if people want to use the globe instead of the default landing home page they can change the res.render on the app.get('/') route to render that template instead of the default one.
I'd probably add an env var for 'default landing page' type of thing but it would require extra work because of how the popular section works atm.

Looks good. I'm gonna check this out and maybe change a couple things but will try and get it merged and deployed to NewTube for when you wake up tomorrow lol nice work

I'm gonna merge this, I think it can still use a bit of polish, it's a bit weird that the text is off to the left, I think it would be better if we could center the text and put the globe beneath it, but overall the concept is very very cool so I will merge it and deploy it to NewTube and maybe we can put some finishing touches on it in the next few days


En site edits

Quick formatting and grammar edits to english front page, per email with Yu Yang

Thanks for patch!


fix: twitter summary card image url

According my todays tweet (https://twitter.com/daviddalbusco/status/1200671798708391937?s=20) it seems that the image provided in the meta tag twitter:image respectively ./logo.jpg wasn't interpreted by Twitter.
For DeckDeckGo I use absolute URL and I double checked DEV.to, which also use absolute.
Therefore I think that the image might need to be provide in such form respectively https://postwoman.io/logo.jpg
P.S.: I didn't modified the og:image because I quickly tested Facebook and it seems they are fine with a relative path

Hey @peterpeterparker,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: dd324fe0-1343-11ea-9fec-2bc1e44a98db

Are you sure Twitter might've cached old summary image.

@liyasthomas GitHub card is fine, https://postwomam.io isn't
Tried to refresh the Twitter card with their validator https://cards-dev.twitter.com/validator, same outcome, no image.

@liyasthomas @TravisBuddy ping me when deployed, I'll be happy to try out again and if necessary send another PR

Just verify later today and ping back. ♥

@liyasthomas
looks like it was deployed so if I had a dummy param to the url, the social card do display the images, otherwise, it still doesn't display it...maybe takes a while until Twitter refresh its cache...

Yeah. Acknowledged. Thanks for the quick PR.


Adding Code Coverage output support and updating to Pester 4.0.6

Adds the Code Coverage input for Invoke-Pester and outputs the XML file.

Just added -CodeCoverage as I'd missed it, no use in generating a report if Pester isn't actually doing the code coverage analysis. It needs a bit more work as it currently just looks up the $ScriptFolder for any *.ps1 and adds those to -CodeCoverage. I'll look into it further and put some tests in too.

In test builds it's correctly generating the xml file and publishing it via the dedicated task for it. I'll look into providing another input to specify location of files being analysed. I'll probably default it to the same location as $ScriptFolder but want to support people having different test steps for different groups of tests, such as unit tests and integrations tests in different folders (as seen on the DSC resources repos).

Nice to see this feature on it's way.
On the pester README.md they have a nice image of what it JaCoCo output brings to VSTS/TFS: https://github.com/pester/Pester#code-coverage

Pester 4.0.7 (and now 4.0.8) is out which is the official release of it so I'll get started on finishing this up a bit soon.
I'll probably rev up to a new major version as the updating to the latest version of Pester will potentially break existing builds/releases.


Xaero's World-Map & Mini-Map support

it would be cool if you could add various snuffle icons for Xaero's World-Map & Mini-Map. currently every snuffle shows up as a dot, due to the missing icons.

This resource pack has been included


Added razorpay to technologies.json

I added Razorpay as a payment processor in technologies.json

I added Razorpay as a payment processor in technologies.json

Thanks... Can you please also add SVG logo in src\drivers\webextension\images\icons. For new technology, we need to add logo so that it shows up in Chrome extension.

Okay, I just did that. Added the svg to icons

Okay, I just did that. Added the svg to icons

Thanks.. You need to update this in technologies.json also with property "icon". Have a look at other examples in the file.

Sure, I'm doing that right now. And also, I haven't add description, should I add that too?

Sure, I'm doing that right now. And also, I haven't add description, should I add that too?

Yes.. try to populate as many attributes as you can.

How about now? I added icon attribute and description attribute

@rohitgulam - Not sure if you picked up on your own OR based on this issue - https://github.com/AliasIO/wappalyzer/issues/4339. Can you please also add check for JS object as mentioned in this issue.

I'll fix this one up, thanks!

I'll fix this one up, thanks!

Thanks.


BUG: Probleme with channel permission and close

Describe the bug
Not every times, everyone have acces to the channel or i can't close the ticket.
To reproduce
Not every times, everyone have acces to the channel or i can't close the ticket.
Expected behavior
No probleme, I always can close the ticket or the channel are always configurated.
Your host
I use repl.it with 512MB of RAM

Seems like a permission is missing -
Are you sure you gave the "Manage channels" permission to your bot?

Yes, it have the admin perm

Okay, is it showing any error, warn or something in the console/logs?

Yes, "Cannot read property 'targetID' of null" when i want to close the ticket. And i have fix the bug with permission, do you want my code ?

I've published a commit that modify where support.targetID. Can you try updating your code to see if it's fixed?

Ok, i have change the version and i will say you if i have the same probleme.


Configurable Tor signaling behavior

Ticket: #14
Type: Enhancement
Changes

Add TorController class

Documentation
We need to be able to configure when the controller sends the signal with as little configuration after instantiation.  First step was to remove the functionality from the TorEngine class.  In addition, we needed an easy way to have the controller be both configurable, and also keep track of each call to it.
Impacted Areas

TorEngine

Checklist

[ ] Add information to the release notes documentation
[ ] Add new feature to the usage documentation
[x] Add tests

This will take a bit of a closer read I think, but whats the higher level user pattern here?
controller = TorController(foo, bar, **baz)
callback = controller.limited_signal
engine = TorEngine(foo, bar)

for url in urls:
     engine.get_page_source(url, None, pre_fetch_callback=callback)

Right?   Is the callback passed every time to get_page_source, or does the user (crawler) have to control when the callback needs to happen?

Something like this should work.  If we don't want to pass in the limiter at the get call, and would rather do it at init of engine, we would probably have to use a shallow copy that includes the same instance of the controller or else we'd have like 5 open connections.  The multiple connections will throw off the underlying is_newnym_available method.
# we could move the default limiters into a different class, but that seems just as awkward
limiter = TorController.time_limited(60)
controller = TorController(signal_limiter=limiter, **kwargs)
engine = TorEngine(foo, bar)

with controller.connection():
    for url in urls:
        engine.get_page_source(url, pre_fetch_callback=controller.limited_signal)

pro tip you can also just combine the two limiters to get a rate limiter like we talked about.
time_limit = TorController.time_limited(60)
call_limit = TorController.call_limited(10)
def rate_limit(): return time_limit() and call_limit()
controller = TorController(signal_limiter=rate_limit, **kwargs)

I'd perfer that all engine-specific config happened at instantiation time (or after instantiation but before currying or calling get_page_source), so more like:
controller = TorController(foo, bar, **baz)
callback = controller.limited_signal
engine = TorEngine(foo, bar, pre_fetch_callback=callback)

for url in urls:
     engine.get_page_source(url, None)

For the sake of the crawlers not being overly complex.  If crawlers get passed an already instantiated engine of some type, and all engine-specific code is at instantiation, then crawlers are totally oblivious to the fact that engines have different types, which they should be.

IMO we should have some documentation with some examples etc.

This wouldn't necessarily have to happen right now

I think once we get this PR wrapped up, and get some agreement on the format/UX of a basic crawler, then user-facing docs for how to setup a crawler with different engines will be the next big hurdle.  I think for v0.1.0, this is good, and I'd like to also merge something like #22 with at least the concept of a crawler included as a module.  Then dogfood things some and get good documentation, testing, UX, etc in v0.2.0, before worrying too too much about crawler scale.
That said, I'm marking this approved, with the caveat of needing a good set of example usages in later versions, once we have a crawler to show it used in.

:+1: Thanks for working on this!


Should support authentication

Feature description
Current recommendation is to disable security and allow anonymous access.  This is not a good idea IMO and this module should implement support for authenticating properly https://micronaut-projects.github.io/micronaut-control-panel/snapshot/guide/#management using client credentials grant or something

Note that this module is only intended to be used during development. Adding security will certainly make it suitable for other environments, but that could be done in a future release


404 on documentation and about pages

I am currently receiving 404's on the documentation page.  I am clicking on the links from the home page.
https://swapi.dev/documentation
https://swapi.dev/about

Thanks! Obviously a configuration issue, will likely to fix it tomorrow...

All right, this was an unexpected side-effect, but fixed now. Thanks!


About documentation / Cli

Hey!
We talked about a possible redesign of the CLI part. Is there any plan / work already started on that?
I think it will be best to document things only after this change if it is about to happen (and not lose time on documenting the existing CLI), otherwise, I can add documentation for the current CLI.
Let me know whether you think it is wiser to wait?

I think it is wiser to wait till one of us find some time to refactor the CLI. I was about to start a week ago and then got caught into a whirlwind of other more urgent stuff :(
I would really like the thing to be auto-documented, hence putting as much as possible examples and documentation inside the CLI help.


Host is being encoded

Running go 1.5 here. Receiving this error when running client.MessageSend.
panic: parse https://mandrillapp.com%2Fapi%2F/1.0/messages/send.json: percent-encoded characters in host

very strange. I looked at the code and in master I am not seeing any URL encoding or query escaping.
Can you make sure you are on master, and also post the stack to this issue. thanks.

Was this ever resolved? I'm running into this issue as well.

Hi @mattbaird, not sure either, it's been a while since the issue first arose. I'd say go ahead and ignore this one, thanks!


python310Packages.reolink-aio: 0.5.0 -> 0.5.1

Diff: https://github.com/starkillerOG/reolink_aio/compare/refs/tags/0.5.0...0.5.1
Changelog: https://github.com/starkillerOG/reolink_aio/releases/tag/0.5.1
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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.

Result of nixpkgs-review pr 217926 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.reolink
python310Packages.reolink-aio
python311Packages.reolink-aio


Eliminate non-inclusive language

Hello, we have a project to eliminate non-inclusive language from the Linux-system-roles. Since the ansible-pcp role is vendored in the metrics role, we are hoping to clean up this role, as well.
Running a utility woke (now it's supported in tox. please install the latest tox-lsr and run tox -e woke), two non-inclusive words are reported - master and whitelist.
Do you think they can be replaced with inclusive words? Or they are hardcoded in the subsystem? If the latter, we have to tell woke to skip the problematic word by adding wokeignore:rule=master or wokeignore:rule=whitelist individually...

I've opened https://github.com/performancecopilot/ansible-pcp/pull/44.
@natoscott, @richm, could you please take a look? Thanks!

@noriko - thanks for working on this, I've added some comments to the PR

@natoscott, thank you for your discussions! Now we can close this issue, too.


Support favorites

It should be trivial to add Favorites support to this app (i.e. export favorites as another RSS feed), the question is whether anybody needs it. if so, please :+1: this post, also I would be interested to hear why you don't use bookmarks instead.

I don't know about "needs," but I would like to have a Favorites feed...

I use bookmarks as a "read-it-later" indicator. Might be a long read, or work-relevant technical stuff. I don't know if it's a favorite or not yet.
I use Favorites as quick feedback on something I like or appreciate, but is not a high priority to revisit. I would occasionally like to scan my Favorites for a remembered meme or quote, etc.

Ah, good point. I was kind of imagining a lazy, browseable archive. Many feeds I follow seem to scroll back forever, but it makes sense to limit the feed & avoid burdening indie fedi servers. Given that Mastodon doesn't export Favorites, I should probably just revise my habits and bookmark the gems for export.


Sisualize alpha index using violin plots

Description
This PR works on visualization of alpha indexes and give more options to visualize directly from the class.
Changelogs

new features for AlphaDiversity class:

visualize method now has two mode: histogram or violin
visualize_groups method allows visualization with violin plot based on metadata grouped by a chosen column

There is now definition of index_col for MetadataParser

Issue(s)
Indicates which issue is related to this Pull Request
Fixes #37
Definition of Done

[x]  New code is tested with unit tests
[ ]  Documentation has been updated
[ ]  Pull Request has been revised by a maintainer of the project

Codecov Report

Merging #42 into master will increase coverage by 0.79%.
The diff coverage is 54.16%.

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
+ Coverage   57.59%   58.38%   +0.79%
==========================================
  Files          47       49       +2
  Lines        1448     1526      +78
==========================================
+ Hits          834      891      +57
- Misses        614      635      +21

Flag
Coverage Δ

#unittests
58.38% <54.16%> (+0.79%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moonstone/analysis/diversity/alpha.py
72.46% <50.00%> (-22.41%)
:arrow_down:

moonstone/plot/graphs/violin.py
55.55% <55.55%> (ø)

moonstone/parsers/metadata.py
100.00% <100.00%> (ø)

moonstone/parsers/counts/taxonomy/qiime.py
100.00% <0.00%> (ø)

moonstone/parsers/counts/taxonomy/kraken2.py
100.00% <0.00%> (ø)

moonstone/parsers/counts/taxonomy/metaphlan2.py
100.00% <0.00%> (ø)

moonstone/parsers/counts/taxonomy/base.py

moonstone/utils/df_reindex.py
100.00% <0.00%> (ø)

moonstone/utils/taxonomy.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Including protocol in upgrade-ops-manager/aws/params.yml's opsman_uri causes errors

When using the protocol in 'opsman_uri' we receive Concourse error message on "wait-opsman-clear":
could not execute "curl": failed to make api request: failed submitting request: token could not be retrieved from target url:  Post https://https/uaa/oauth/token: dial tcp: lookup https on <IP_ADDRESS>:53: no such host
in params.yml example includes http protocol:
Ops Manager Url
eg. https://myopsman.mydomain.com
Should be:
Ops Manager Url
eg. myopsman.mydomain.com

:+1: is also an issue with the vSphere pipeline, although the error is more cryptic for me:
Checking if you are applying an allowed patch upgrade @ https://opsman.localnet
parse error: Invalid numeric literal at line 1, column 6
You are trying to install version:
1.11.5#2017-07-23T23:45:35.944Z
Your currently deployed version is:
.

Pivotal recommends that you only automate
    patch upgrades of PCF, and perform major or minor
    upgrades manually to ensure that no high-impact
    changes to the platform are introduced without
    your prior knowledge.

To upgrade patch releases, we suggest using the following version regex in your params file:
^\\.\..*$

Hi @geoffwa, thanks for creating this issue. We already have a story in our backlog (https://www.pivotaltracker.com/story/show/148112387)  to prioritise renamingopsman_uri and to clarify that a protocol should not be included in the param. The second possibly related issue - please let us know if removing the protocol doesn't fix it and you continue to receive that error.

Closing this issue because we've accepted https://www.pivotaltracker.com/story/show/148112387, which clarifies the param name to indicate to operators not to include the protocol. If you have any feedback, please do not hesitate to reach out. Thanks.


feat: OpenLineage integration

Describe your changes:
Adds connector for collecting lineage using OpenLineage standard. Co-authored with @dechoma

I worked on adding OpenLineage connector because it's a widely used standard for harvesting lineage information and OpenMetadata was missing such integration.
This connector is targeted to users:

who use (or plan to use) OpenLineage to emit lineage information to kafka topic (this is one of two ways OL recommends emmiting lineage information)
use OpenMetadata to store metadata within their projects and needs to enrich it with lineage information collected using OL

Type of change:

[ ] Bug fix
[ ] Improvement
[x] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[x] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@mgorsk1 thank you for the PR, should we mark it as ready for review?

it's ready @TeddyCr

any update on review @TeddyCr ?

all checks are green, can you re-review @TeddyCr @ulixius9 ? I wouldn't want this to hang much longer considering how many rebases this already needed

let's go over the pending topics on a separate PR


Add awesome animated gif to Readme

hmm that looks blurry too wtf tj/stack's one doesn't look bad

nvm retina ones are like 30mb

It's quite blurry on Retina, besides do we really need this?

doesn't matter to me, I kinda like that the node one shows how fast deploys can be with lambda, but otherwise I think it probably works the way people would expect from the docs

I don't think you can center the image if you use a relative path with markdown.

Why not? This does not makes sense. DOM itself is responsible for rendering. It does not not care about src location, just data.

let's revisit if there's a way to make a non-blurry version that isn't multi-megabytes

@turboMaCk http://stackoverflow.com/a/12118349

would work perfectly.

Oh okay, @turboMaCk I didn't know you could do that! Thanks! :smiley:


"cannot set terminal process group (-1): Inappropriate ioctl for device"

SQL Operations Studio Version: sqlops 0.23.6 (8a3011756318a2ded1e6e84332230a2b2bf9c544, 2017-11-05T22:46:14.815Z)
OS Version: Linux x64 4.13.0-19-generic

Steps to Reproduce:

Start terminal in Ubuntu 17.10
Launch sqlops

[joakim:~/sqlops-linux-x64] $ ./sqlops
bash: cannot set terminal process group (-1): Inappropriate ioctl for device
bash: no job control in this shell

Thanks for reporting this. To clarify, SQL Ops Studio still starts and works correctly, correct? I see this message on Ubuntu as well, but the program seems to be working normally otherwise.

Same issue under Fedora 27.  Yes it starts and works no problem.  Installed as per the instructions provided.
Found that:

sqlops & disown

seems to help a little (untieing it from the shell process so that closing the terminal window you launched it from doesn't shut it down), but still get the message.
Would be nice to get it installing with a neat icon and lodged properly so that dnf can install it etc.
(loving the application though, really nice!)


BUG: force matplotlib to use PyQt4 as backend

Solve bug reported in issue #80

@gamaanderson Did you get the following warning as well?
because the backend has already been chosen;
matplotlib.use() must be called *before* pylab, matplotlib.pyplot,
or matplotlib.backends is imported for the first time.```

yes, actually I also get this warning. It got lost in my terminal, so I didn't saw it. I wounder however if this line as any effect at all. The important is probably the second one.

I recall seeing that issues could arise from having PySide instead of PyQt (there were some tests that could be used to call either) when I originally switched the code from Tkinter in January. There are slight differences in a few calls if using PySide vs. PyQt as well. Just for future reference I'll put a link to differences in this thread.

I doubt it causes any issues, but we'll see if it fixes the problem.

After I pulled the latest changes to pyart:
commit 20afb010eeeccaeeabd4457241b2eeb6a4870b7e
Merge: de80c70 05f38f9
Author: Nick Guy<EMAIL_ADDRESS>Date:   Wed Aug 19 08:45:23 2015 -0600
I still get the same error message, shown below.
Also, is it possible to suppress the message "Unable to simultaneously satisfy constraints"?
/usr/local/anaconda/lib/python2.7/site-packages/matplotlib/init.py:1318: UserWarning:  This call to matplotlib.use() has no effect
because the backend has already been chosen;
matplotlib.use() must be called before pylab, matplotlib.pyplot,
or matplotlib.backends is imported for the first time.
warnings.warn(_use_error_msg)
2015-08-19 10:04:44.365 python[79925:2804616] Unable to simultaneously satisfy constraints:
(
<NSAutoresizingMaskLayoutConstraint:0x111063ad0 h=--& v=--& V:[NSView:0x115075970(0)]>,
<NSAutoresizingMaskLayoutConstraint:0x1110d4e40 h=-&- v=-&- V:|-(0)-[FI_TListScrollView:0x115079e80]   (Names: '|':NSView:0x115075970 )>,
<NSAutoresizingMaskLayoutConstraint:0x1110d6690 h=-&- v=-&- V:[FI_TListScrollView:0x115079e80]-(0)-|   (Names: '|':NSView:0x115075970 )>,
<NSAutoresizingMaskLayoutConstraint:0x1129e6310 h=-&- v=-&- V:[NSClipView:0x115079a40]-(15)-|   (Names: '|':FI_TListScrollView:0x115079e80 )>,
<NSAutoresizingMaskLayoutConstraint:0x1129e6130 h=-&- v=-&- V:|-(0)-[NSClipView:0x115079a40]   (Names: '|':FI_TListScrollView:0x115079e80 )>
)
Will attempt to recover by breaking constraint
<NSAutoresizingMaskLayoutConstraint:0x1129e6310 h=-&- v=-&- V:[NSClipView:0x115079a40]-(15)-|   (Names: '|':FI_TListScrollView:0x115079e80 )>
Set the NSUserDefault NSConstraintBasedLayoutVisualizeMutuallyExclusiveConstraints to YES to have -[NSWindow visualizeConstraints:] automatically called when this happens.  And/or, break on objc_exception_throw to catch this in the debugger.
Opening file /scr/eldora2/rsfdata/front/cfradial/spol/qpe/20140520/cfrad.20140520_170440.000_to_20140520_170440.000_SPOL_v0_s00_el0.00_SUR.nc
Traceback (most recent call last):
File "/usr/local/anaconda/bin/artview", line 5, in
artview.main.main(sys.argv)
File "/usr/local/anaconda/lib/python2.7/site-packages/artview/main.py", line 18, in main
artview.run(DirIn, filename, field)
File "/usr/local/anaconda/lib/python2.7/site-packages/artview/scripts/standard.py", line 50, in run
parent=MainMenu)
File "/usr/local/anaconda/lib/python2.7/site-packages/artview/components/plot_radar.py", line 142, in init
self.LaunchGUI()
File "/usr/local/anaconda/lib/python2.7/site-packages/artview/components/plot_radar.py", line 171, in LaunchGUI
self._set_figure_canvas()
File "/usr/local/anaconda/lib/python2.7/site-packages/artview/components/plot_radar.py", line 677, in _set_figure_canvas
self.layout.addWidget(self.canvas, 1, 0, 7, 6)
TypeError: arguments did not match any overloaded call:
QGridLayout.addWidget(QWidget): argument 1 has unexpected type 'FigureCanvasQTAgg'
QGridLayout.addWidget(QWidget, int, int, Qt.Alignment alignment=0): argument 1 has unexpected type 'FigureCanvasQTAgg'
QGridLayout.addWidget(QWidget, int, int, int, int, Qt.Alignment alignment=0): argument 1 has unexpected type 'FigureCanvasQTAgg'

Okay, I added a little snippet in the __init__.py. This takes care of the warning message on my machine. Can you grab the changes and try?

I pulled your change - no difference
/usr/local/anaconda/lib/python2.7/site-packages/matplotlib/init.py:1318:
UserWarning:  This call to matplotlib.use() has no effect
because the backend has already been chosen;
matplotlib.use() must be called before pylab, matplotlib.pyplot,
or matplotlib.backends is imported for the first time.
warnings.warn(_use_error_msg)
2015-08-19 13:54:46.872 python[82951:2958946] Unable to simultaneously
satisfy constraints:
(
<NSAutoresizingMaskLayoutConstraint:0x1150830f0 h=--& v=--&
V:[NSView:0x1129b57f0(0)]>,
<NSAutoresizingMaskLayoutConstraint:0x11509e450 h=-&- v=-&-
V:|-(0)-[FI_TListScrollView:0x1129a9c20]   (Names: '|':NSView:0x1129b57f0
)>,
<NSAutoresizingMaskLayoutConstraint:0x11509e400 h=-&- v=-&-
V:[FI_TListScrollView:0x1129a9c20]-(0)-|   (Names: '|':NSView:0x1129b57f0
)>,
<NSAutoresizingMaskLayoutConstraint:0x11509b7b0 h=-&- v=-&-
V:[NSClipView:0x1129a9910]-(15)-|   (Names:
'|':FI_TListScrollView:0x1129a9c20 )>,
<NSAutoresizingMaskLayoutConstraint:0x11509b760 h=-&- v=-&-
V:|-(0)-[NSClipView:0x1129a9910]   (Names:
'|':FI_TListScrollView:0x1129a9c20 )>
)
Will attempt to recover by breaking constraint
<NSAutoresizingMaskLayoutConstraint:0x11509b7b0 h=-&- v=-&-
V:[NSClipView:0x1129a9910]-(15)-|   (Names:
'|':FI_TListScrollView:0x1129a9c20 )>
Set the NSUserDefault
NSConstraintBasedLayoutVisualizeMutuallyExclusiveConstraints to YES to have
-[NSWindow visualizeConstraints:] automatically called when this happens.
And/or, break on objc_exception_throw to catch this in the debugger.
Opening file /scr/eldora2/rsfdata/front/cfradial/spol/qpe/20140520/
cfrad.20140520_170440.000_to_20140520_170440.000_SPOL_v0_s00_el0.00_SUR.nc
Traceback (most recent call last):
File "/usr/local/anaconda/bin/artview", line 5, in
artview.main.main(sys.argv)
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/main.py", line
18, in main
artview.run(DirIn, filename, field)
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/scripts/standard.py",
line 50, in run
parent=MainMenu)
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/components/plot_radar.py",
line 142, in init
self.LaunchGUI()
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/components/plot_radar.py",
line 171, in LaunchGUI
self._set_figure_canvas()
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/components/plot_radar.py",
line 677, in _set_figure_canvas
self.layout.addWidget(self.canvas, 1, 0, 7, 6)
TypeError: arguments did not match any overloaded call:
QGridLayout.addWidget(QWidget): argument 1 has unexpected type
'FigureCanvasQTAgg'
QGridLayout.addWidget(QWidget, int, int, Qt.Alignment alignment=0):
argument 1 has unexpected type 'FigureCanvasQTAgg'
QGridLayout.addWidget(QWidget, int, int, int, int, Qt.Alignment
alignment=0): argument 1 has unexpected type ‘FigureCanvasQTAgg'
On Wed, Aug 19, 2015 at 1:00 PM, Nick<EMAIL_ADDRESS>wrote:

Okay, I added a little snippet in the init.py. This takes care of the
warning message on my machine. Can you grab the changes and try?
—
Reply to this email directly or view it on GitHub
https://github.com/nguy/artview/pull/81#issuecomment-132743212.

Hmmmm, weird. Can you start python in a shell and run the following and tell me what comes out?
import matplotlib
matplotlib.get_backend()

I would also be interested in the test script I posted in the original issue (#80), because I've run out of ideas.
@nguy, do you know that are those constraints?

I'm not sure what is going on. Done some looking and can't seem to find any information matching the symptoms thus far.
With the above code, I wanted to make sure that MacOS was listed as is on my machine.
I also realized I mistyped this line matplotlib.use('Agg') in __init__.py . It should read matplotlib.use('Qt4Agg').
I pushed a change. This could affect the behavior because 'Agg' is for machines with no interactive GUI.

Python 2.7.10 |Anaconda 2.3.0 (x86_64)| (default, May 28 2015, 17:04:42)
[GCC 4.2.1 (Apple Inc. build 5577)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://binstar.org

import matplotlib
matplotlib.get_backend()
u'MacOSX'

On Wed, Aug 19, 2015 at 2:34 PM, Nick<EMAIL_ADDRESS>wrote:

Hmmmm, weird. Can you start python in a shell and run the following and
tell me what comes out?
import matplotlib
matplotlib.get_backend()
—
Reply to this email directly or view it on GitHub
https://github.com/nguy/artview/pull/81#issuecomment-132773370.

No change after pulling:
l$ artview
2015-08-19 17:08:54.919 python[85448:3058449] Unable to simultaneously
satisfy constraints:
(
<NSAutoresizingMaskLayoutConstraint:0x115081230 h=--& v=--&
V:[NSView:0x11503ed10(0)]>,
<NSAutoresizingMaskLayoutConstraint:0x115080c80 h=-&- v=-&-
V:|-(0)-[FI_TListScrollView:0x115042e60]   (Names: '|':NSView:0x11503ed10
)>,
<NSAutoresizingMaskLayoutConstraint:0x115080c30 h=-&- v=-&-
V:[FI_TListScrollView:0x115042e60]-(0)-|   (Names: '|':NSView:0x11503ed10
)>,
<NSAutoresizingMaskLayoutConstraint:0x115036220 h=-&- v=-&-
V:[NSClipView:0x115042ab0]-(15)-|   (Names:
'|':FI_TListScrollView:0x115042e60 )>,
<NSAutoresizingMaskLayoutConstraint:0x1150361d0 h=-&- v=-&-
V:|-(0)-[NSClipView:0x115042ab0]   (Names:
'|':FI_TListScrollView:0x115042e60 )>
)
Will attempt to recover by breaking constraint
<NSAutoresizingMaskLayoutConstraint:0x115036220 h=-&- v=-&-
V:[NSClipView:0x115042ab0]-(15)-|   (Names:
'|':FI_TListScrollView:0x115042e60 )>
Set the NSUserDefault
NSConstraintBasedLayoutVisualizeMutuallyExclusiveConstraints to YES to have
-[NSWindow visualizeConstraints:] automatically called when this happens.
And/or, break on objc_exception_throw to catch this in the debugger.
Opening file /scr/eldora2/rsfdata/front/cfradial/spol/qpe/20140520/
cfrad.20140520_170440.000_to_20140520_170440.000_SPOL_v0_s00_el0.00_SUR.nc
Traceback (most recent call last):
File "/usr/local/anaconda/bin/artview", line 5, in
artview.main.main(sys.argv)
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/main.py", line
18, in main
artview.run(DirIn, filename, field)
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/scripts/standard.py",
line 50, in run
parent=MainMenu)
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/components/plot_radar.py",
line 142, in init
self.LaunchGUI()
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/components/plot_radar.py",
line 171, in LaunchGUI
self._set_figure_canvas()
File
"/usr/local/anaconda/lib/python2.7/site-packages/artview/components/plot_radar.py",
line 677, in _set_figure_canvas
self.layout.addWidget(self.canvas, 1, 0, 7, 6)
TypeError: arguments did not match any overloaded call:
QGridLayout.addWidget(QWidget): argument 1 has unexpected type
'FigureCanvasQTAgg'
QGridLayout.addWidget(QWidget, int, int, Qt.Alignment alignment=0):
argument 1 has unexpected type 'FigureCanvasQTAgg'
QGridLayout.addWidget(QWidget, int, int, int, int, Qt.Alignment
alignment=0): argument 1 has unexpected type 'FigureCanvasQTAgg'
On Wed, Aug 19, 2015 at 3:24 PM, Nick<EMAIL_ADDRESS>wrote:

I'm not sure what is going on. Done some looking and can't seem to find
any information matching the symptoms thus far.
With the above code, I wanted to make sure that MacOS was listed as is on
my machine.
I also realized I mistyped this line matplotlib.use('Agg') in init.py
. It should read matplotlib.use('Qt4Agg').
I pushed a change. This could affect the behavior because 'Agg' is for
machines with no interactive GUI.
—
Reply to this email directly or view it on GitHub
https://github.com/nguy/artview/pull/81#issuecomment-132790521.

python /tmp/testmp.py
Traceback (most recent call last):
File "/tmp/testmp", line 24, in
win.setCentralWidget(canvas)
TypeError: QMainWindow.setCentralWidget(QWidget): argument 1 has unexpected
type ‘FigureCanvasQTAgg'
I uncommented
matplotlib.use('Qt4Agg’)  - no difference
I uncommented
matplotlib.rcParams['backend.qt4']=‘PyQt4’ - no difference
Here are the versions you wanted:
matplotlib.version
‘1.4.3'
from PyQt4.QtCore import QT_VERSION_STR

QT_VERSION_STR
‘4.8.6'

from PyQt4.pyqtconfig import Configuration

cfg = Configuration()
cfg.pyqt_version_str
‘4.11.3

Python 2.7.10 |Anaconda 2.3.0 (x86_64)| (default, May 28 2015, 17:04:42)
On Wed, Aug 19, 2015 at 3:11 PM, Anderson<EMAIL_ADDRESS>wrote:

I would also be interested in the test script I posted in the original
issue (#80 https://github.com/nguy/artview/issues/80), because I've run
out of ideas.
@nguy https://github.com/nguy, do you know that are those constraints?
—
Reply to this email directly or view it on GitHub
https://github.com/nguy/artview/pull/81#issuecomment-132785804.

I've made a question in Stackoverflow, lets see how that goes.

According to the comment the issue you posted, do you think that by leaving the other
import matplotlib
matplotlib.use('Qt4Agg')
matplotlib.rcParams['backend.qt4']='PyQt4'

in other modified scripts when I added to the init.py it caused this issue?

Sorry, I don’t know.
On Wed, Aug 19, 2015 at 9:33 PM, Nick<EMAIL_ADDRESS>wrote:

According to the comment the issue you posted, do you think that by
leaving the other
import matplotlib
matplotlib.use('Qt4Agg')
matplotlib.rcParams['backend.qt4']='PyQt4'
in other modified scripts when I added to the init.py it caused this
issue?
—
Reply to this email directly or view it on GitHub
https://github.com/nguy/artview/pull/81#issuecomment-132874734.

Sorry @vanandel that last comment was directed at @gamaanderson

@vanandel On the Stackoverflow that was submitted, one comment suggested that importing components multiple times might contribute. I commented out the other additions from this original pull request and only left the first instance I added in the init.py.
Would you pull changes and see if the error changes (or goes away, fingers crossed)?
Also do you still get the matplotlib.use() warning at startup?

sorry, I forgot to answer.
letting just one call of matplotlib.use() in __init__.py remove the matplotlib.use() has no effect warning, but this only cosmetic as it does not solve the problems.
The  constraints, I don't know what they are to being with, have never seen and the internet did not help.

When I added the code to the __init__.py the warning you note went away on my machine. Is it still present on your computer @gamaanderson ?
Yeah this one seems tough, not finding much help online.

you are right, the warning is already gone.

Okay I'll take a small victory.  It must have something to do with how those were opened and in fact imported matplotlib before those were enabled. By sticking the code before anything was loaded initially, it got around that.

From this discussion it may be that the window sizes are causing the issue.

So @gamaanderson you said you were able to recreate the problem?
What did you do to recreate? Are you still unable to open?
I'm wondering after floating around the web a bit more if it has something to do with the menu width and height definitions on startup. The only things I can find have to do with mostly mobile applications with resizable windows.

@vanandel in the test script you could replace
from PyQt4 import QtCore, QtGui

by
from matplotlib.backends.qt_compat import QtCore, QtGui
print (QtCore) # just to make sure this is PyQt

@nguy I can get the same error when I force loading PySide with matplotlib.rcParams['backend.qt4']='PySide'. it opens the file but fails as soon as it create the Display (canvas).

@vanandel You do this in lrose_solopy. Did you see any types of failures before implementing this?

I don’t recall this type of failure.  I was attempting to side-step the
issue of whether the user had PySide or PyQt.
When I add the lines
from matplotlib.backends.qt_compat import QtCore, QtGui
print (QtCore) # just to make sure this is PyQt
The test application works, and prints:
<module 'PySide.QtCore' from
'/usr/local/anaconda/lib/python2.7/site-packages/PySide/QtCore.so’>
Following the advice of
http://matplotlib.org/1.4.0/faq/usage_faq.html#how-do-i-select-pyqt4-or-pyside
$ export QT_API=pyqt
The test application prints
<module 'PyQt4.QtCore' from
'/usr/local/anaconda/lib/python2.7/site-packages/PyQt4/QtCore.so’>
and works.
On Thu, Aug 20, 2015 at 10:48 AM, Nick<EMAIL_ADDRESS>wrote:

@vanandel https://github.com/vanandel You do this
https://github.com/nguy/lrose-soloPy/blob/master/lrose_solopy/QtVariant.py
in lrose_solopy. Did you see any types of failures before implementing this?
—
Reply to this email directly or view it on GitHub
https://github.com/nguy/artview/pull/81#issuecomment-133072894.

@vanandel So you Matplotlib is loading PySide even with the matplotlib.rcParams['backend.qt4']='PyQt4'!? And with the QT_API flag does artview works?
@nguy may be we could QtCore, QtGui from matplotlib? And most important should we do it? I can see some reasons why the may be a bad idea.

Wow, this seems strange because I expected the rcParams line to set the behavior for the session.
I wonder if matplotlib is using this file linked from the official Qt page as their qt_compat? I'm not quite sure how it's deployed, but it appears to look for PySide first (unless I'm misreading the script).
@vanandel just to confirm if you start a fresh python session and type:
import matplotlib
print(matplotlib.rcParams['backend.qt4'])

It comes back with PySide?

@gamaanderson I'm not if using the matplotlib method is a good idea either.
One big concern is that if the plotting in PyArt and subsequently ARTview changes to a GPU-based package. Of course this is problematic throughout the code.
We could I guess try to catch and error and use the matplotlib.rcParams or alert user to set QT_API environmental variable. Not the best solutions in mind either...

OK - my bad -   I had export QT_API=pyside in my .bash_profile, so that’s
why matplotlib was using pyside.
If I don’t have QT_API set, or I set QT_API=pyqt, then artview works.
Since artview doesn’t seem to work with pyside, perhaps it could check if
QT_API is set to pyside, print a helpful error message, and then exit?
Would it be hard to use a compatibility layer, as I did in lrose_solopy?
On Thu, Aug 20, 2015 at 1:48 PM, Nick<EMAIL_ADDRESS>wrote:

Wow, this seems strange because I expected the rcParams line to set the
behavior for the session.
I wonder if matplotlib is using this file
https://github.com/epage/PythonUtils/blob/master/util/qt_compat.py
linked from the official Qt page as their qt_compat? I'm not quite sure how
it's deployed, but it appears to look for PySide first (unless I'm
misreading the script).
@vanandel https://github.com/vanandel just to confirm if you start a
fresh python session and type:
import matplotlib
print(matplotlib.rcParams['backend.qt4'])
It comes back with PySide?
—
Reply to this email directly or view it on GitHub
https://github.com/nguy/artview/pull/81#issuecomment-133151627.

Nice, we have figure it out!
testing would be simple, it is just bad having to do every time we start up. We could test it when installing, but as it is bash dependent some cases can still get thought.
using both could be possible, but we still need to be sure of be using the same one as matplotlib, and god knows what matplotlib could do (e.g. PyQt5).
I must think a little, but this should definitely be addressed somehow.

@vanandel I just looked throught the lrose code and I don't think that script is actually used. All internal calls use PySide.
The test for QT_API as an environmental variable should be fairly easy using os.environ['QT_API']. Though I'm not sure a useful message wouldn't just annoy the user. I don't think testing when installing is a good idea in case system settings change - at which point a reinstallation would be required.
The best probably solution IMO at this time would be to deploy a test on startup and somehow propagate that choice through the code. Not sure how to do that.
We could also test each time it is needed and build a hook with the class something like this. But that seems inefficient.

There’s a discussion here:
https://wiki.qt.io/Differences_Between_PySide_and_PyQt
about compatibility layers that let you support both.  They seem to
recommend python_qt_binding.
On Thu, Aug 20, 2015 at 2:40 PM, Nick<EMAIL_ADDRESS>wrote:

@vanandel https://github.com/vanandel I just looked throught the lrose
code and I don't think that script is actually used. All internal calls use
PySide.
The test for QT_API as an environmental variable should be fairly easy
using os.environ['QT_API']. Though I'm not sure a useful message wouldn't
just annoy the user. I don't think testing when installing is a good idea
in case system settings change - at which point a reinstallation would be
required.
The best probably solution IMO at this time would be to deploy a test on
startup and somehow propagate that choice through the code. Not sure how to
do that.
We could also test each time it is needed and build a hook with the class
something like this
http://stackoverflow.com/questions/8052386/how-can-you-easily-select-between-pyqt-or-pyside-at-runtime.
But that seems inefficient.
—
Reply to this email directly or view it on GitHub
https://github.com/nguy/artview/pull/81#issuecomment-133164068.

Testing the environmental is fairy simple, but may be vulnerable, a more robust check would be
from matplotlib.backends.qt_compat import QtCore, QtGui
isinstance(QtCore, PyQt4.QtCore)

big problem qt_compat is in matplotlib just since may 2014, around version 1.4
given a look in the code there is not clear way to predict or force which one (PyQt4, PyQt4v2, PySide or PyQt5) matplotlib will use.

I hesitate to add another dependent library, these can get away pretty quick.
I'm sure there are people out there using matplotlib <v1.4, so that could cause issues. Having our own qt_compat might be the most effective. But I could see PyQt5 becoming more pronounced in the next couple of years.
I really don't know what the best answer is.
@gamaanderson why is the environmental variable test vulnerable?

it is vulnerable because you are testing what matplotlib is supposed to do, not what it have done. If it fails to use that the variable ask it will try something else and not fail.

Well you could update the environmental variable before matplotlib opens, BUT we really don't want to change a users settings.
This would just allow the info to be passed to user that PySide will fail if we don't support it.


readme file - Handling typos and wording issues

Hello hadrienj,
Thank you for your instructive tutorial on Linear Algebra.
I fixed some typos in the readme file and also improved the grammar.
Please find below the general description of the changes line by line (they are plenty but it does not mean you work is not great):

"syllabus for anyone who want "
Replace want with wants

"The goal of this series is to provide content for beginners who wants "
Replace wants with want

"As much information and knowledge through examples than through general definitions"
Replace than with as: as much ... as

"Coding is a great tool to concretely experiment abstract mathematical notions"
Replace with "Coding is a great tool to experiment with abstract mathematical notions"

Remove the repetition of "Coding is a great tool to concretely experiment abstract mathematical notions"

"linear algebra concepts for data science"
Add a comma

"The syllabus follow exactly"
Replace follow with follows

"and how to use it with Numpy"
make it "and how to use them with Numpy"

"what Gilbert Strang call"
Replace call with calls

"It can be thinks as the length"
Make it "It can be thought of as the length"

"We will see different kind of norms"
Replace kind with kinds

"We will see other type of vectors"
Replace type with types

"express quadratic equations into a matrix form"
Replace ... into a matrix form... with ...in a matrix form..

"Since the beginning of this series, I emphasized"
Remove the comma

"We will see that we can see"
Replace ...we can see... with ...we can look at...
This way you would not use 'see' twice

"We will see the effect of SVD on an example image of Lucy the goose so, keep on reading"
Make it
"We will see the effect of SVD on an example image of Lucy the goose. So, keep on reading"

"In some cases, a system of equation"
Replace equation with equations

"in term of minimizing the error"
Replace ... in term of ... with ...in terms of...

"This content is aimed at beginners but it should be easier for people with at least some experience with mathematics"
Replace the second half as follows
"...but it would be easier to follow if you have at least some experience with mathematics"

"I hope that you will find something interesting in that series"
Replace ..in that series... with ...in this series...

So much for my corrections. Have a great day out there!

@ogryniuk Thanks a lot, this is helpful. I just merged the PR.


Variable names in examples are weird

All variable names in Azure communication services examples are weird. the underscore is at the end of the variable which bothers me reading the code.

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

ID: 1f6c0b6e-a3b1-ed87-f673-4842029e498f
Version Independent ID: 10cb7160-76d2-3f5c-3a6d-b9bbddc75d0e
Content: Quickstart - Add video calling to your app (JavaScript) - An Azure Communication Services quickstart
Content Source: articles/communication-services/quickstarts/voice-video-calling/get-started-with-video-calling.md
Service: azure-communication-services
GitHub Login: @xumo-95
Microsoft Alias: mikben

@MhAllan Thanks for the feedback. We're investigating this and will get back to you shortly.

@MhAllan Thanks for bringing this to our attention. I have submitted a PR to fix the issue that's in review. The changes should reflect post the merge.
We will now proceed to close this thread as resolved, but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Invalid redirect_uri but all client and server are config

i have met an error,  in the developement ,i used ip and port line http://<IP_ADDRESS>:4201
the client send the request is

and the server has the logs

and in my database

and so ,i dont know how to resolve it ? any one help?

That's not a bug - you have a typo..
silent vs slient

thank u very much ,it's a very stupid mistake.


@elderjs/plugin-images - Custom config props not replacing default props properly

Default config array props
widths: [1280, 992, 768, 576, 400, 350, 200],
scales: [1, 2],

are not replaced with custom values from elder.config.js
widths: [1000, 750, 500],
scales: [1],

but updated resulting config values like this
widths: [1000, 750, 500, 576, 400, 350, 200],
scales: [1, 2],

@meigo this will be an issue with lodash’s defaultsDeep.
We’ll need to remove the defaults and if the array received by the init function is empty then we populate a default.
I’m slammed for the next few days but will try and bundle these together. If you wanted to they and pull together a PR I’d gladly code review and merge it.
The markdown plugin had a similar issue.

github.com/Elderjs/plugins/pull/32

@meigo thank you for this. Will push a release today.


Add method WithPersistentVolume

#1132
Add an extension method as described in issue:
var database = builder.AddSqlServerContainer("sqlserver", "abcdefg123_78")
    .WithPersistentVolume("/var/opt/mssql")
    .AddDatabase("appdb");

The method WithPersistentVolume would be a shortcut to WithVolumeMount(string source, string target, VolumeMountType type, bool isReadOnly) with source being generated from the application name and resource name, e.g. $"{builder.Environment.ApplicationName}.{resourceName}.data", and type being VolumeMountType.Named and isReadOnly being false.

Testing
Manually tested in eShopLite sample app:
var catalogDb =
    builder
        .AddPostgresContainer("postgres")
        .WithEnvironment("PGDATA", "/var/lib/postgresql/data/pgdata")
        .WithPersistentVolume("/var/lib/postgresql/data")
        .AddDatabase("catalogdb");

Might be overkill, but I'm still looking for an existing unit test I can copy to get coverage on docker volumes.
Would welcome any tips.


PsTwitterTest.java:42-47: Test to be implemented for...

Puzzle 11-790c0445 in src/test/java/org/takes/facets/auth/social/PsTwitterTest.java:42-47 has to be resolved: Test to be implemented for PsTwitter using a oauth mock library (eg:wiremock). Need to modify PsTwitter to accept url from configurations, so that url can be changed for test and real env accordingly. Response to be stubbed for both token and verify_credentials calls and assertions to be performed for the values returned.... The puzzle was created by Prasath Premkumar on 20-Apr-2015.
If you have any technical questions, don't ask me, submit new tickets instead

@ikhvostenkov please go ahead with this task,it's yours

@yegor256 there is no milestone yet, so I set it to 1.0

@davvd Could you please assign someone else to this task?

@davvd Could you please assign someone else to this task?

@ikhvostenkov 30 points deducted from your rating

@davvd Could you please assign someone else to this task?

@ikhvostenkov got it, someone else will be assigned soon

@yamstranger please go ahead with this task, it's yours

@davvd Thanks

@davvd Please help me. I don't understand this words "Puzzle 11-790c0445" what is "11-790c0445"? Is this some id of puzzle specification, of something else?

@YamStranger yes, it's just an id of the puzzle

@yegor256 Can I get some info about this puzzle, by this id? Where can I find this info, if it is possible?

@YamStranger what kind of info do you need? everything you need (and we have) is in the Java file

@yegor256 Thanks. It was info what I need. This is just id, and it will be not useful for me (I cant use it any way)

@davvd , @yegor256. As wrote @alice, I am not with your team any more. So please assign ticket to someone else. Thanks

@davvd , @yegor256. As wrote @alice, I am not with your team any more. So please assign ticket to someone else. Thanks

@yamstranger OK, I will try to assign someone else

@adko-pl please proceed, it's yours

@davvd Can I get an extension on this ticket? I need to fix code review comments.

@davvd give us more time here pls

@davvd give us more time here pls

@yegor256 of course, thanks for letting me know, take your time

@davvd the code has been merged, please close this ticket

@yegor256 Thanks, closed

@ypshenychka please, review this task for compliance with our quality rules

@adko-pl
According to our QA Rules:

Some of ticket messages must mention all pull requests where the problem was actually fixed.

Please indicate in any of your messages link to corresponding PR.

@ypshenychka I fixed the QA issues in this ticket

@adko-pl Thanks!

@davvd Quality is acceptable.

@davvd Quality is acceptable.

@ypshenychka thanks for the QA review, we'll work better next time

@adko-pl just added 10 mins to @ypshenychka (for QA), payment ID is 59387582. Thank you, I have added 30 mins to your account in payment/transaction "AP-6Y065313U9637230D", time consumed: 604 hours and 34 mins. +30 added to your rating, at the moment it is: -150


Please add huobi data

I also need huobi data, please add it.
Thank you. ASAP

@chinarichman coming soon!

Thanks a lot dude!
2017-12-15 10:24 GMT+08:00 swfbarhr<EMAIL_ADDRESS>
@chinarichman https://github.com/chinarichman coming soon!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SuperDBJ/telegram-coin-bot/issues/1#issuecomment-351895132,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ag8GTqD57siDPf6QJsEJpMxrdfP9AZG-ks5tAdhWgaJpZM4RCBwj
.

@chinarichman done! you can add the testing bot by name coin_baymax_bot and enjoy yourself!

Great thanks a lot mate!
2017-12-19 17:31 GMT+08:00 Destec<EMAIL_ADDRESS>
@chinarichman https://github.com/chinarichman done! you can add the
testing bot by name coin_baymax_bot and enjoy yourself!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CoinQuant/telegram-coin-bot/issues/1#issuecomment-352688189,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ag8GTvAVwLisIeKq7gMECFWX4Efdofs6ks5tB4JrgaJpZM4RCBwj
.


Add option for custom watchDog timer interval

Hi,
actually the watchdog-timer is set to 100ms.
https://github.com/wf9a5m75/phonegap-googlemaps-plugin/blob/master/www/googlemaps-cdv-plugin.js#L2018
Could you please provide a function to override the value or even disable it? (false = disabled, 0 = disbaled, 1 = 1ms, 20 = 20ms... etc)

@bFlood are you in for a bounty? This is a very urgent feature for me. I guess if we both reach 50 bucks masashi is in for doing this (I would do it myself, but I'm busy enough with the app)

Sorry.  I'm really appreciate for you.
I just don't have enough time for my plugin.

No problem. I will send PR in 10 minuntes ;)
I'm finished in a few minutes

Hi @wf9a5m75
just add these lines before setCenter function in your JS.
/**
 * @desc Set watchDogTimer for map positioning changes
 */
App.prototype.getWatchDogTimer = function() {
  var self = this;
  time = self.get('watchDogTimer') || 100;
  return time;
};

/**
 * @desc Set watchDogTimer for map positioning changes
 */
App.prototype.setWatchDogTimer = function(time) {
  var self = this;
  time = time || 100;
  self.set('watchDogTimer', time);

  if(time < 50) {
    console.log('Warning: watchdog values under 50ms will drain battery a lot. Just use for short operation times.');
  }
};

And very close to the end, replace
    window._watchDogTimer = window.setInterval(function() { myFunc(); }, 100);

with this one:
    window._watchDogTimer = window.setInterval(function() { myFunc(); }, _mapInstance.getWatchDogTimer());

You still can't cancel the watchdog (this option really should still be implemented, but for now, I could set a high value like 10000 which will save a lot more battery...
You can set your own value with:
Map.setWatchDogTimer(TIMEINMILLISECONDS);

Value needs to be an integer.

Here as a proof, the benefits are HUGE!
https://www.youtube.com/watch?v=cddcqIibOl8&feature=youtu.be
OnsenUI has a preopen and a postopen, preclose and postclose callbacks for the menu.

Thank you, @Hirbod
It seems looks good!
Please send me a pull request. I will merge it.

#367
Maybe you will find some time for adding the options for enable/disable totally.


CHAD-11494: Add localizations for Chinese join names

@cjswedes this is only possible because the CN environment is segregated from the rest of the global environments. If we do end up with a join name localization strategy in the future, I do not expect it will look like this.


Small improvements

PR improves two things:

It emits entrypoints even without a [hash] in the output name. This is sensible in development to avoid creating hundreds of bundles over time.
When the build fails because of an error, the plugin does not produce an additional error because result.metafile is missing.

Thank you very much! Released as v0.2.0.


Could not fetch templates. See the output Channel for more Details

i don't know what else to do. after hitting the refresh button on Templates, am been greeted with the above error "Could not fetch templates. See the output Channel for more Details"

Indeed, the error details aren't correctly displayed.
However, do you think you can run the following command, in the vscode terminal?
erdpy contract templates

Is there any output / error?

having the same issue. And I tried using "erdpy contract templates" and got an error message "bash: erdpy: command not found". how do I proceed. I can't seem to get past this

Same issue here. On refresh an error says  "Could not fetch templates. See the output Channel for more Details".
Terminal command erdpy contract templates also errors out as a command not found.

well I'm getting the same error in Ubuntu also .. :(

I get the same error on Manjaro. Any fix?

i've the same error but a notification tells me 'Unexpected end of JSON input'


How to use this in a fragment

All your examples are activity related could you please do one with fragments . i.e use a webview in a fragment ?

you can use this library in a fragment just like examples do in activity


Auto-approve Renovate pull requests

Branch protections for main now include the need to approve pull requests before merging.
If this will make things strange for myself when making a pull request, it should be more secure.
When Renovate will open a pull request, it will be automatically approved by GitHub action Bot
Renovate will enable auto-merge automatically so no need to enable it as it is the case for pre-commit-ci bot.
Previous automerge.yml is renamed to better reflect its use
and each bot has its workflow because they don't both need to enable auto-merge.

Lets not using approbation for auto-merge as it looks I will stay the only maintainer for a long time not to say forever


Increase lmax to 15 in Figure 13

I recommend increasing the order of the nonlinear fit from 6 to ~15 in Figure 13. This significantly reduces the error while not increasing the run time that much. With l=15, our run time is about 4x faster than batman and we get 7x smaller error. Also, the statement in the caption of Figure 9 isn't quite right, as the accuracy of the fit keeps increasing above l=6 (although it's hard to see on a linear scale; it would be clearer if you plotted it in log space). Somewhere between l=15 and l=20 this turns over, however, since we hit the roundoff error noise floor.

@rodluger Great!  It also looks like you have improved the computational time significantly as well.  What allowed for that?


Add Terraform templates

What did you implement:
Closes #316

How did you implement it:

How can we verify it:

Todos:

[ ] Write tests
[x] Write documentation
[ ] Fix linting errors
[ ] Make sure code coverage hasn't dropped
[ ] Provide verification commands / resources
[x] Enable "Allow edits from maintainers" for this PR
[x] Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

@mthenw A few notes regarding the latest commit:

the coreos etcd module is a part of Kubernetes setup in Tectonic installer and has some dependencies, like certificates, coreos ignition, dns. I've extracted those dependencies into the etcd wrapper (modules/etcd)
Tectonic Installer repo is under Apache v2 licence, so I've included the NOTICE file
it's possible to use https for the etcd (tls_enabled param), however eg libkv store does not initialize with tls config. I don't think this it a big issue for now, as the etcd cluster runs in private subnets.

@baniol overall looks good. We need few days to review it. Again, thanks for contribution.


compile error with g++ 11.2.1_p20220115

I got the following compile error with latest git code from Mar 28 using g++ 11.2.1_p20220115:
[2/57] /usr/bin/x86_64-pc-linux-gnu-g++ -Dresource_file_EXPORTS -I/usr/local/include  -DNDEBUG -O2 -march=native -pipe -fPIC -Wall -Wextra -Werror -Wno-strict-aliasing -std=gnu++20 -MD -MT CMakeFiles/resource_file.dir/src/Decompressors/System2.cc.o -MF CMakeFiles/resource_file.dir/src/Decompressors/System2.cc.o.d -o CMakeFiles/resource_file.dir/src/Decompressors/System2.cc.o -c /var/tmp/portage/app-arch/resource-dasm-0_p20220329/work/resource_dasm-master/src/Decompressors/System2.cc
FAILED: CMakeFiles/resource_file.dir/src/Decompressors/System2.cc.o
/usr/bin/x86_64-pc-linux-gnu-g++ -Dresource_file_EXPORTS -I/usr/local/include  -DNDEBUG -O2 -march=native -pipe -fPIC -Wall -Wextra -Werror -Wno-strict-aliasing -std=gnu++20 -MD -MT CMakeFiles/resource_file.dir/src/Decompressors/System2.cc.o -MF CMakeFiles/resource_file.dir/src/Decompressors/System2.cc.o.d -o CMakeFiles/resource_file.dir/src/Decompressors/System2.cc.o -c /var/tmp/portage/app-arch/resource-dasm-0_p20220329/work/resource_dasm-master/src/Decompressors/System2.cc
/var/tmp/portage/app-arch/resource-dasm-0_p20220329/work/resource_dasm-master/src/Decompressors/System2.cc: In function ‘std::string decompress_system2(const CompressedResourceHeader&, const void*, size_t)’:
/var/tmp/portage/app-arch/resource-dasm-0_p20220329/work/resource_dasm-master/src/Decompressors/System2.cc:106:20: error: ‘source_types’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
  106 |       source_types <<= 1;
      |       ~~~~~~~~~~~~~^~~~~
cc1plus: all warnings being treated as errors

Modifying the initial value of source_types with uint8_t source_types=0; fixes it.

Thanks - just pushed that fix. It's odd that CI didn't complain about this earlier; the CI builds run with g++ 10, so maybe this error is new in g++ 11.


Resource Matching Interface

As a business owner, I want to find financial support and business advice opportunities relevant to my business.
This story is contingent on the resource matching (RM) project, tracked here: https://waffle.io/codeforamerica/wealthmap
Key steps here:

[ ] Wait for RM API to be completed
[ ] Design an interface
[ ] Implement it in code

Lisa and Mark created sketches together.
Technical plan is now detailed in #182
Closing this story.


[FEAT]: implement getApiUrl in @refinedev/nestjs-query

Is your feature request related to a problem? Please describe.
getApiUrl method is not implemented.
Describe the thing to improve
getApiUrl method should return GraphQLClient's URL.

Hi @alicanerdurmaz please assign this issue to me.

Hello, I would like to contribute in it. Is it still under action ?

Hello @609harsh, I assigned it to you. Thanks.
@userAdityaa @Sarthakagg31 Thanks, I assigned to @609harsh because @609harsh wrote before. if @609harsh is not available, we will communicate again

Due to inactivity in PR #5844, we're looking for new  contributors to implement this feature 🙏