Full screen map not shown in v3.0.0

If I define the map with
 <ContentView height="240" width="240">
the map is correctly shown.
However if I set <ContentView> to have a full screen map, the map is not shown.
This was working on previous versions. Is there any change in v3.0.0 I should take into account?

Could be changed indeed. Is this iOS?
How did you previously defined the fullscreen map?

Yes, it is IOS.
I did it in this way
<GridLayout rows="*,auto,auto,auto" columns="*">
   <ContentView row="0" col="0">
      <map:MapboxView xmlns:map="nativescript-mapbox"
                <!-- my map definition -->
      </map:MapboxView>
   </ContentView>
   <!-- rest of rows and columns -->
</GridLayout>

And what does it look like?

Now,
[image: Imágenes integradas 1]
Before (in the new one I have removed all the others elements)
[image: Imágenes integradas 2]
2017-04-28 14:24 GMT+02:00 Eddy Verbruggen<EMAIL_ADDRESS>
And what does it look like?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EddyVerbruggen/nativescript-mapbox/issues/107#issuecomment-297984504,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMmIGfpDfUTQcAwvfHm4ZmJU_8kvDdJEks5r0dp6gaJpZM4NLZrm
.

Thx, I have an idea why this changed.

@j99ht if you're using angular2:
import {screen, device} from 'platform';
then in the your component define the height of your screen like this:
let mapHeight : number = 0;     constructor() {         this.mapHeight = screen.mainScreen.heightPixels;     }
In your <ContentView [height]="mapHeight"></ContentView>
Let me know if that helps.

I was just looking at an app that had the following alomost-fullscreen layout working nicely on iOS:
<StackLayout>
    <Button text="load markers" (tap)="loadPoints()"></Button>
    <ContentView height="100%" width="100%">
      <Mapbox
          accessToken="the_token"
          mapStyle="dark"
          latitude="50.467735"
          longitude="13.427718"
          hideCompass="true"
          zoomLevel="18"
          showUserLocation="false"
          disableZoom="false"
          disableRotation="false"
          disableScroll="false"
          disableTilt="false"
          (mapReady)="onMapReady($event)">
      </Mapbox>
    </ContentView>
</StackLayout>


Need settings for text classification

Can you improve ur library so that you can do text classification?

unfortunately, that is out of scope of the framework.


Cyclomatic complexity not correct

Hi,
I noted that some function would be spotted as having a too high cyclomatic complexity if local functions are used. Putting the local functions outside (as non local) solves the issue. In my opinion, this is erroneous. Locally defined functions should not be counted in cyclomatic complexity of the enclosing function.
Pierre-Yves

+1! Moving inner helper methods outside does not improve code quality. So both method length and cyclomatic complexity should be configurable to allow for a more functional style.

@rklaehn @wookietreiber @pysa Can you test this with the latest snapshot, and tell me if this is better for you please?
Thanks.

@matthewfarwell Is there an easy way to test the latest snapshot using sbt scalastyle plugin?

Yes, there is, but since I've just released 0.8.0, you can test it with that.


[Scopes] can't select an icon to display in scope selector

default scope selector uses the scope to "define the css class for the scope".

But the move to material has changed the way icons are rendered:

So you end up with the code of your scope written in a undefined font.

linked to https://github.com/KleeGroup/focus-components/issues/552 i believe

Could be, though,I'd go a little farther : instead of creating your own font, why isn't there a system where you could feed the path to an icon? Or define a css class with a background image pointing to that icon.
Either way, the fact the the code of the scope defines the icon seems wrong to me. The code of the scope defines the scope : in Linkit, its values are project or job, and I don't want to have to create a font where these words match some icons, I'd rather leave these codes as is and have another value in the scope config for the icon?.
I mean, technically, the getActiveScopeIcon does nothing: you feed it a code, it searches the code in the scope config and returns... the code.

Duplicate of https://github.com/KleeGroup/focus-components/issues/552

https://github.com/KleeGroup/focus-components/blob/develop/src/search/search-bar/scope.js#L114

We need to be able top provide other props to the icon.

_renderScopeList() {
        const {scopesId} = this;
        const {list: scopeList, value} = this.props;
        return (
            <ul className={`mdl-menu mdl-menu--bottom-left mdl-js-menu mdl-js-ripple-effect`} data-focus='search-bar-scopes' htmlFor={scopesId} ref='scopeDropdown'>
                {0 < scopeList.length && scopeList.map(scope => {
                   const {code, label, ...otherScopeProps} = scope;
                    const scopeId = uniqueId('scopes_');
                    const isActive = value === code;
                    return (

                            {scope.code &&
                                <Icon name={code}  ...otherScopeProps/>
                            }
                            {label}

                    );
                })}
                {0 === scopeList.length &&

                        {this.i18n('scopes.empty')}

                }

        );
    }

Fixed in the scope list, not in the selected scope


CI should pull logs if test fails

Pull logs if a test fails (like Jenkins)
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/ci-kubernetes-charts-gce/17407

/assign @mattfarina @viglesiasce

Fixed with the new chart testing lib.

Don't have the cycles to address this, so I'm closing. Probably still a good idea to pull pod logs, however.


AWS repository delete document + unit test

Start for #250
Left puzzle for continuing implementation in future ticket.

Coverage decreased (-0.02%) to 78.935% when pulling 464cc55aa82c777d8b15b7401f53b19061c6bb84 on 250 into bba4f4df01e75df682e1e7b3a6ae9dd4aea869e7 on master.

@rultor merge it ...

@rultor merge it ...

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge it ...

@amihaiemil Done! FYI, the full log is here (took me 2min)


🐛 Bug Report: Gitlab Scaffolding Action PipelineTrigger can't trigger with variables

📜 Description
You can't pass variables to a pipeline/trigger as per:
https://docs.gitlab.com/ee/api/pipeline_triggers.html#trigger-a-pipeline-with-a-token
👍 Expected behavior
Allow you to create step actions that can trigger a pipeline with specific variables
👎 Actual Behavior with Screenshots
There is no option/mechanism to pass variables into the trigger.
👟 Reproduction steps

Setup backstage with a Gitlab account
Import/Load https://gitlab.com/backstage5212121/scaffolding-templates/-/blob/main/gitlab-pipeline-trigger/template.yaml
Run the scaffolder job and see that no variables occur in the pipeline outputs.

📃 Provide the context for the Bug.
Not sure if it's a bug or feature.
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

I've already got a fix for this and tested it, Just need to spend some time doing all the necessary paperwork involved and submit the PR.

PR #25466

It looks like pull request #25480 has been merged 🎉 , can this issue be closed now @veenarm ?


It's time to update this page

I hope this page will eventually be updated, now that MSAL is no more in preview. Moreover, as stated in this answer on stackoverflow "Azure Mobile Apps" are something still supported, but the new kid in town is "App Center", so maybe this page should be demoted in favour of the App Center way of doing it.

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

ID: 16864cab-cbfa-98f3-c1da-7ed88cfe2579
Version Independent ID: c1731783-e6da-af46-3fce-5e4bcc50dd61
Content: Integrating Azure Active Directory B2C with Azure Mobile Apps - Xamarin
Content Source: docs/xamarin-forms/data-cloud/authentication/azure-ad-b2c-mobile-app.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @ncarandini
This page/sample has been updated internally, and is undergoing review/feedback incorporation, prior to publication. Therefore I'll close this issue.


Last Week in Pony - August 5, 2018

Pony 0.24.4 has been released. Upgrading is recommended: https://www.ponylang.org/blog/2018/07/0.24.4-released/

Perhaps useful for Last Week's Playground:
Isolates and Tuples
Working with isolates and tuples can be challenging given the extreme restriction of an iso capability. This playground covers putting isolates into a tuple, as well as extracting them back out.

As an aside: Perhaps it would be useful to keep a list of such playgrounds on the website, or create something similar for the tutorial. This way they won't get lost overtime in old Last Week in Pony posts, as they are not labelled in anyway as far as I can tell.
Actually, I can see Last Week's Playground being transitioned to the tutorial, as a way to have such examples which don't exactly fit in with the flow of the tutorial. In the case of the above, I thought first to expand the tutorial, but including it in the tuples section felt too complicated (given refcaps are not yet introduced), and including it in any refcap section felt out of place.
Long story short:
Having a Playgrounds section in the tutorial with categories / labeled playgrounds, would make for better procurement and management of such playgrounds, and provide the tutorial with a way to expand on any feature or combination thereof without overloading the learner with inlined examples or out of context notes. Last Week in Pony could still feature interesting playgrounds procured there.

Do you use the Sublime Text Pony module? You might want to update it. Stefan Achatz has added a couple fixes in the last few weeks. Nothing huge, but... its an update. Didn't realize there was Sublime Text support? There is... feel free to contribute more fixes yourself at https://github.com/ponylang/sublime-pony.

The Ponylang emacs mode got an update recently thanks to spreadLink. There's been no official release with the fix yet, but if you are running latest, you can take advantage of it now.
A note from Sean, if you are interested in helping him as a maintainer of ponylang-mode, drop him an email<EMAIL_ADDRESS>


jsonGraph.json for other languages

Pls support Chinese language and other. seems Netflix support it. don't know how to scrape from netflix.com

chrome F12 tool filter https://www.netflix.com/api/shakti/vdf992f93/pathEvaluator api call


ButtonStyle merge does not merge

In my example, https://gist.github.com/NebulaFox/172e7ac344370935b3d12b8aa5fc0400, I have defined a default/base style for ElevatedButton, which is styled based on the _toggle property of the state class. When using the merge method to change the base style, it appears the style is not being merged in.
Expected results:
The button should toggle between two different styles
Actual results:
No change in style
[✓] Flutter (Channel beta, 1.23.0-18.1.pre, on Mac OS X 10.14.6 18G6032 x86_64,
    locale en-GB)
    • Flutter version 1.23.0-18.1.pre at /usr/local/flutter
    • Framework revision 198df796aa (5 weeks ago), 2020-10-15 12:04:33 -0700
    • Engine revision 1d12d82d9c
    • Dart version 2.11.0 (build 2.11.0-213.1.beta)

Hi @NebulaFox
Looks like overriding existing parameter in the merge, it is  used to fill unspecified fields not override existing ones https://api.flutter.dev/flutter/material/ButtonStyle/merge.html
For example, here i added different in merge that are present in default style,  it works fine
    ButtonStyle defaults = ElevatedButton.styleFrom(
      padding: EdgeInsets.all(10),
      textStyle: TextStyle(
        color: Colors.black,
      ),
    );
    ButtonStyle style;

    if (_toggle) {
      print('Toggling $_toggle');

      style = defaults.merge(
        ElevatedButton.styleFrom(
            elevation: 10,
            shape: RoundedRectangleBorder(
                borderRadius: BorderRadius.all(Radius.circular(20)))),
      );
    }

code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

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

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  bool _toggle;

  initState() {
    super.initState();

    _toggle = false;
  }

  ButtonStyle _buildStyle(BuildContext context) {
    print("_buildStyle _toggle:$_toggle");

    ButtonStyle defaults = ElevatedButton.styleFrom(
      padding: EdgeInsets.all(10),
      textStyle: TextStyle(
        color: Colors.black,
      ),
    );
    ButtonStyle style;

    if (_toggle) {
      print('Toggling $_toggle');

      style = defaults.merge(
        ElevatedButton.styleFrom(
            elevation: 10,
            shape: RoundedRectangleBorder(
                borderRadius: BorderRadius.all(Radius.circular(20)))),
      );
    }

    print(style);

    return style ?? defaults;
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            ElevatedButton(
                style: _buildStyle(context),
                child: Text("TOGGLE"),
                onPressed: () {
                  setState(() {
                    _toggle = !_toggle;
                  });
                })
          ],
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel beta, 1.23.0-18.1.pre, on macOS 11.0.1 20B29 x86_64, locale
    en-GB)
    • Flutter version 1.23.0-18.1.pre at /Users/tahatesser/Code/flutter_beta
    • Framework revision 198df796aa (5 weeks ago), 2020-10-15 12:04:33 -0700
    • Engine revision 1d12d82d9c
    • Dart version 2.11.0 (build 2.11.0-213.1.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.10.0

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

[✓] Android Studio (version 4.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
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (4 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E            • ios
      • iOS 14.2
    • iPhone 12 (mobile)     • B46BCEDD-9601-4CD0-8D0B-CE4EF96ACD93 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-2 (simulator)
    • Web Server (web)       • web-server                           •
      web-javascript • Flutter Tools
    • Chrome (web)           • chrome                               •
      web-javascript • Google Chrome 86.0.4240.198

• No issues found!

I was under the impression that it would follow similar behaviour to IconThemeData.merge, https://api.flutter.dev/flutter/widgets/IconThemeData/merge.html. ButtonStyle.merge breaks continuity already existing merge methods (other theme data classes). If this is the desired behaviour, it would be better to renameButtonStyle.merge to something like ButtonStyle.override?

You are correct that Flutter's merge methods are not consistent and that ButtonStyle.merge() behaves differently then TextStyle.merge which itself is different than ListTileTheme.merge(), which differs from IconTheme.merge(), and etc. It turned out that - for ButtonStyles - a "merge" operation that just reset fields that were null, i.e. that just added default overrides, was useful.
For your example, it looks like you want is copyWith:
...

style = defaults.copyWith(
  elevation: 10,
  shape: RoundedRectangleBorder(
    borderRadius: BorderRadius.all(Radius.circular(20)))),
);

I'm going to close this issue however if you discover a use-case that doesn't seem to fit, please reopen it.

@HansMuller The reason I ended up using merge is because I was doing this
 style = style.merge(ElevatedButton.styleFrom(
        primary: theme.accentColor.withOpacity(0.12),
        onPrimary: theme.disabledColor,
      ));

I was creating a style through ElevatedButton.styleFrom, which cannot be done via a copyWith since ButtonStyle.copyWith does not have the parameters primary and onPrimary. I do understand I can look into code for ElevatedButton.styleFrom and find what parameters it sets on ButtonStyle, but if there is any custom logic, I will have to copy that as well. I would be simpler to just use merge. Instead, I've had to do something like this
    ButtonStyle style ;
    if (selected) {
      style = ElevatedButton.styleFrom(
        primary: theme.accentColor.withOpacity(0.12),
        onPrimary: theme.disabledColor,
      );
    } else {
      style = ElevatedButton.styleFrom(
        primary: theme.disabledColor.withOpacity(0.12),
        onPrimary: theme.accentColor,
      );
    }

    return style.merge(ElevatedButton.styleFrom(
      padding: this.contentPadding,
      elevation: 0.0,
    ));

Little bit further removed from just having a defaults and merging any changes that I want in. Would it be better for me to create a issue that points out the inconsitencies of merge methods on *Style and *Theme classes?

There's no need to open a new issue, I think I understand the use case that's causing trouble now.

flutter doctor -v
[✓] Flutter (Channel beta, 1.24.0-10.2.pre, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-IN)
    • Flutter version 1.24.0-10.2.pre at /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 022b333a08 (6 days ago), 2020-11-18 11:35:09 -0800
    • Engine revision 07c1eed46b
    • Dart version 2.12.0 (build 2.12.0-29.10.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

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

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (4 available)
    • SM A260G (mobile)                   • 5200763ebcfa861f
      • android-arm    • Android 8.1.0 (API 27)
    • iPhone SE (2nd generation) (mobile) • 6C85835D-FBFD-4AB3-8DE8-B4FAD35E5367
      • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-0 (simulator)
    • Web Server (web)                    • web-server
      • web-javascript • Flutter Tools
    • Chrome (web)                        • chrome
      • web-javascript • Google Chrome 87.0.4280.67

• No issues found!

I am able to reproduce the issue on the latest master channel. The merge properties do not reflect on the button.
  void _incrementCounter() {
    if (!selected) {
      style = ElevatedButton.styleFrom(
        backgroundColor: Colors.red,
        foregroundColor: Colors.white,
      );
    } else {
      style = ElevatedButton.styleFrom(
        backgroundColor: Colors.blue,
        foregroundColor: Colors.white,
      );
    }
    // none of the properties are applied
    style!.merge(ElevatedButton.styleFrom(
      padding: EdgeInsets.all(_counter.toDouble()),
      elevation: 5,
    ));
    setState(() {
      selected = !selected;
    });
  }

code sample
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        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);
  final String title;
  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;
  void _incrementCounter() {
    if (!selected) {
      style = ElevatedButton.styleFrom(
        backgroundColor: Colors.red,
        foregroundColor: Colors.white,
      );
    } else {
      style = ElevatedButton.styleFrom(
        backgroundColor: Colors.blue,
        foregroundColor: Colors.white,
      );
    }

    style!.merge(ElevatedButton.styleFrom(
      padding: EdgeInsets.all(_counter.toDouble()),
      elevation: 5,
    ));
    setState(() {
      selected = !selected;
    });
  }
  ThemeData? theme;
  bool selected = false;
  ButtonStyle? style ;
  @override
  Widget build(BuildContext context) {
    theme = Theme.of(context);
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            ElevatedButton(
              style: style,
              onPressed: () {}, child: Text('tap me')),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

flutter doctor -v (mac)
[✓] Flutter (Channel master, 3.6.0-6.0.pre.33, on macOS 12.6 21G115 darwin-arm64,
    locale en-IN)
    • Flutter version 3.6.0-6.0.pre.33 on channel master at
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a9858ec524 (72 minutes ago), 2022-11-21 10:58:11 -0500
    • Engine revision 46a6b54295
    • Dart version 2.19.0 (build 2.19.0-406.0.dev)
    • DevTools version 2.19.0
    • If those were intentional, you can disregard the above warnings; however it is
      recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc4)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0-rc4
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

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

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

[✓] Android Studio (version 2021.2)
    • 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.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

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

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 026D5789-9E78-4AD5-B1B2-3F8D4E7F65E4 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64
      • macOS 12.6 21G115 darwin-arm64
    • Chrome (web)           • chrome                               • web-javascript
      • Google Chrome 107.0.5304.110

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

• No issues found!

[✓] Flutter (Channel stable, 3.3.9, on macOS 12.6 21G115 darwin-arm, locale en-IN)
    • Flutter version 3.3.9 on channel stable at /Users/mahesh/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b8f7f1f986 (24 hours ago), 2022-11-23 06:43:51 +0900
    • Engine revision 8f2221fbef
    • Dart version 2.18.5
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc4)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0-rc4
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

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

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

[✓] Android Studio (version 2021.2)
    • 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.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

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

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 026D5789-9E78-4AD5-B1B2-3F8D4E7F65E4 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.6 21G115 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 107.0.5304.110

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

• No issues found!


Migrate keywords data structure to Postgres ARRAY type

🚨 Note: Data migration contained within
Does not drop existing column, but removes references to it.
Once merged, no new Releases will populate their keywords column, only the keywords_array column will be filled.

I haven't read the entire PR yet, but you probably want this split over multiple deploys, currently what's going to happen is:

Column will be added, data migration will be run.
New deployment will occur with the new code.
Old deployment will be spun down, shutting down old code.

The important bit of that is that the old code will still be running after the data migration has completed, so anything uploaded in that window of time via the old code will have keywords populated, but not keywords_array.
So I would refactor this into multiple PRs, that we can deploy in separate deployments:

Adds the keywords_array column, sets up the upload code to write both columns.
Does the data migration, guarding not to run against rows that (1) would have already written, sets up the code to only use the keywords_array column.

Optional: This could also drop the keywords column from the code, but not from the database, making SQLAlchemy completely ignore that column.

If the keywords column has been dropped from the code, drop it from the database as well.

probably want this split over multiple deploys

Yep! Going to dismantle and rebuild, now that all the pieces are known.


cant change icon for modified buffers

i tried to change the "+" symbol into a little icon to indicate that modifications have been made to the file.
i did it the following way:
" Show icon and change the /+ for unsaved changes to be within the same block with diff icon
function! FileNameWithIcon() abort
  let filename_with_icon = expand('%:t') !=# '' ? expand('%:t') : 'Txt'
  let modified = &modified ? '  ' : ''
  return WebDevIconsGetFileTypeSymbol() . ' ' . filename_with_icon . modified
endfunction

and than added that to the statusbar via:
let g:lightline.active = {
        \ 'left': [ ['mode'], [ 'filename_with_icon' ], ['git_branch'] ],
        \ 'right': [ ['lineinfo'],[ 'percent' ], [ 'unix_logo' ] ],
        \ }

it worked for a bit but then i changed something else in my lightline config (tried to integrate github a bit more) and it stopped working. tried the exact same version where it worked again and it still doesnt work anymore. can u spot a mistake or give me an idea what else i could try?
or is it possible a different plugin or command is intefering with the function?

What method did you use to integrate GitHub features? Or are they Git features instead?

Do you have config in component_function? Please describe the issue with your entire config.

i tried to integrate github into lightline via vim fugitive. it didnt work in the end the way i wanted it to work so i went back to my old config.
heres my complete lightline config (but theres a little bit of config for lightline-bufferline inbetween).
" Lightline -------------------------------------------------------------------------------------
"       _       _     _   _ _
"| |   (_) __ _| |__ | |_| (_)_ __   ___
"| |   | |/ _` | '_ \| __| | | '_ \ / _ \
"| |___| | (_| | | | | |_| | | | | |  __/
"|_____|_|\__, |_| |_|\__|_|_|_| |_|\___|
"         |___/

" Lightline shows the mode -> get rid of vim's default
set noshowmode

" Show second tabline on top
set showtabline=2

let g:lightline = {
        \ 'colorscheme': 'deus',
        \ }

" Custom Functions-----------------------------------------------------------------

" Show icon and change the /+ for unsaved changes to be within the same block with diff icon
function! FileNameWithIcon() abort
  let filename_with_icon = expand('%:t') !=# '' ? expand('%:t') : 'Txt'
  let modified = &modified ? '  ' : ''
  return WebDevIconsGetFileTypeSymbol() . ' ' . filename_with_icon . modified
endfunction

" Show filetype with icon and display an icon for unsaved changes
function! FileTypeWithIcon() abort
  let filetype_with_icon = expand(&ft) !=# '' ? expand(&ft) : 'Txt'
  let modified = &modified ? '   ' : ''
  return WebDevIconsGetFileTypeSymbol() . ' '
endfunction
" return WebDevIconsGetFileTypeSymbol() . ' '  . filetype_with_icon  . modified

" Fugitive Integration
function! Git_branch() abort
  if fugitive#head() !=# ''
    " return  fugitive#head() . "  "
    " return  fugitive#head() . "  "
    return  fugitive#head() . "  "
  else
    return " "
  endif
endfunction

" Define lightline look-----------------------------------------------------------

let g:lightline.active = {
        \ 'left': [ ['mode'], [ 'filetype_with_icon' ], ['git_branch'] ],
        \ 'right': [ ['lineinfo'],[ 'percent' ], [ 'unix_logo' ] ],
        \ }

let g:lightline.separator = { 'left': " ", 'right': " " }
let g:lightline.subseparator = { 'left': " ", 'right': " " }
let g:lightline.tabline_separator = { 'left': " ", 'right': " " }
let g:lightline.tabline_subseparator = { 'left': " ", 'right': " " }
" let g:lightline.tabline_subseparator = { 'left': " ", 'right': " " }
" let g:lightline.subseparator = { 'left': " ", 'right': " " }

" let g:lightline.separator = { 'left': " ", 'right': " " }
" let g:lightline.subseparator = { 'left': " ", 'right': " " }
" let g:lightline.tabline_separator = { 'left': " ", 'right': " " }
" let g:lightline.tabline_subseparator = { 'left': " ", 'right': " " }
"

" Tabline-------------------------------------------------------------------------

let g:lightline.tabline = {
        \ 'left': [ [ 'vim_logo'], [ 'buffers' ] ],
        \ 'right': [ [ 'buffer' ], [ 'fileencoding' ] ]
        \ }

let g:lightline.tab = {
        \ 'active': ['filename_with_icon'],
        \ 'inactive': ['filename_with_icon']
        \ }

" Custom lightline components-----------------------------------------------------

let g:lightline.tab_component = {}
let g:lightline.tab_component_function = {
        \ 'artify_filename': 'lightline_tab_filename',
        \ 'filename': 'lightline#tab#filename',
        \ 'modified': 'lightline#tab#modified',
        \ 'readonly': 'lightline#tab#readonly',
        \ 'tabnum': 'lightline#tab#tabnum',
        \ 'filename_with_parent': 'FileNameWithParent'
        \ }

let g:lightline.component = {
        \ 'filename_with_icon': '%{FileNameWithIcon()}',
        \ 'filetype_with_icon': '%{FileTypeWithIcon()}',
        \ 'vim_logo': "\ue7c5 ",
        \ 'unix_logo': "  ",
        \ 'filename_with_parent': '%t',
        \ 'buffer': 'buffers %n',
        \ 'git_branch': '%{Git_branch()}',
        \}

" Tabline Settings-------------------------------------------------------------------

" Enable bufferline
let g:lightline.component_expand = {'buffers': 'lightline#bufferline#buffers'}
let g:lightline.component_type   = {'buffers': 'tabsel'}

" Tabline Icons
let g:lightline#bufferline#enable_devicons = 0
let g:lightline#bufferline#filename_modifier = ':t'

" Tabline colors
let s:palette = g:lightline#colorscheme#deus#palette
let s:palette.tabline.tabsel = [ [ '#d6f4fd', '#3e4452', 252, 66, ] ]
" let s:palette.tabline.tabsel = [ [ '#d6f4fd', '#3e4452', 252, 66, 'bold' ] ]

unlet s:palette

You did not have put functions in component_function.

Adding to g:lightline.component instead of component_function is fine. But you're trying to show the component in tabs? You're FileNameWithIcon() implementation does not work within tabs. Please consider g:lightline.tab_component_function, and the function accepts one argument; the tab number.

yes. i basically have the normal statusbar which shows the mode, the filetype etc and then above a have the lightline bufferline which shows on the right side the number of buffers and on the left side it has tabs with the filenames which are currently open which also shows if a buffer has unsafed changes.
i know for fact that it once worked with the icon cuz ive seen it with my own eyes. only after i changed something else it stopped working.
so if i understand u correct its not supposed to work but if i switch basically the filetype with icon and filename with icon places it should work (filename with icon being in the normal statusbar)?

yes. i basically have the normal statusbar which shows the mode, the filetype etc and then above a have the lightline bufferline which shows on the right side the number of buffers and on the left side it has tabs with the filenames which are currently open which also shows if a buffer has unsafed changes.
heres a picture to better understand what i mean:

i know for fact that it once worked with the icon cuz ive seen it with my own eyes. only after i changed something else it stopped working.
so if i understand u correct its not supposed to work but if i switch basically the filetype with icon and filename with icon places it should work (filename with icon being in the normal statusbar)?

If I'm correct you're using lightline-bufferline plugin. Please read the Integration which mentions on the modified indicator.

correct. its that plugin. i found the answer how to do it. thanks


[Run] remove KillPowerLauncher pre-build event

Summary of the Pull Request
What is this about:
Remove a pre-build event that was added to workaround a code bug that was causing PowerLauncher.exe to keep running after quitting PowerToys.exe.
What is include in the PR:
Deleted src/modules/launcher/Directory.Build.targets
How does someone test / validate:
Build the solution, clean the solution, rebuild the solution: no error should occur.
Quality Checklist

[x] Linked issue: https://github.com/microsoft/PowerToys/issues/7474
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Actually, I find it useful while running only PT Run from VS so I do not have to open Task Manager and kill it.

Actually, I find it useful while running only PT Run from VS so I do not have to open Task Manager and kill it.

@mykhailopylyp
if you run it from the debugger, you should not have PowerToys.exe running as well, since it will mess up things.

@mykhailopylyp
if you run it from the debugger, you should not have PowerToys.exe running as well, since it will mess up things.

We agreed that killing a running process before building the solution is not a good approach.
You may be running PowerToys and do some testing while building a different branch, randomly killing processes is not a good idea.

We agreed that killing a running process before building the solution is not a good approach.
You may be running PowerToys and do some testing while building a different branch, randomly killing processes is not a good idea.


[Proposal / Proof of Concept] Protocol Buffers as a Firestore schema

Proposal for an easy way to sync our schema and model objects across clients:

Create lightweight converters in Kotlin and TypeScript which take a Map<Long, Any> and produce protobufs and vice-versa, where keys in Firestore correspond to a protobuf field number, and values primitive types supported by proto.

Example survey data before:
{
  "title": "My survey",
  "description": "An example survey",
   //...
}

Example survey data after:
{
  1: "My survey",
  2: "An example survey",
  ...
}

This has several advantages:

Generated code ensures a consistent schema across web, Android, and Cloud Functions.
Using field IDs instead of names allows field names can be changes without breaking clients.
Storaging values as Firestore values rather than binary allows Firestore query filters to work without duplicating values in "index" fields, and it allows data to be viewed and debugged using standard tools.
The implementation is simple, easy to write and debug.
Using field numbers instead of names allows this sol'n to work with Android "lite" protobufs in which name bindings aren't available at runtime.
Writing directly to generated code

This has a few disadvantages as well:

Maintaining backwards compatibility with existing test surveys is very difficult. We would probably want to zap the dbs and start from scratch.
Indirection in key names in Firestore; would need to check protobuf def to understand which field is which.

@sufyanAbbasi @shobhitagarwal1612 @scolsen @rfontanarosa @amysorto @nwkotto Thoughts?

Move to #2049


Vim9 script functions defined in Vim script cannot take references to script local functions whose names start with a lower case letter

Issue:
Only in Vim9 script functions defined in Vim script files, defining functions with s:foo style names (beginning with a lower case letter) are allowed, but it is not possible to get references to them by using the function() or by assigning variables.
The current behavior is inconvenient in terms of compatibility with Vim script, and I can't find the specifications in document.
# in def block
var Ref = s:foo
var Ref2 = function('s:foo')

Proposal:
Allow Vim9 script functions defined in Vim script to reference functions in s:foo format.
Advantage:

It will be convenient to replace some of the plugin processes written in Vim script with Vim9 script without changing the function names.
Improved compatibility with conventional Vim script.
The current specification is unnatural because it can be processed normally in other cases

Reproducible code:
This code is added to the test.
function! s:func()
  echo "s:func() was called"
endfunction

def! s:C1()
  s:func()
enddef

def! s:C2()
  var Ref2 = function('s:func')
  Ref2()
enddef

def! s:C3()
  var Ref3 = function(s:func)
  Ref3()
enddef

def! s:C4()
  var Ref1 = s:func
  Ref1()
enddef

call s:C1()
" Error ↓
call s:C2()
call s:C3()
call s:C4()

Issue:
Only in Vim9 script functions defined in Vim script files, defining functions with s:foo style names (beginning with a lower case letter) are allowed, but it is not possible to get references to them by using the function() or by assigning variables.
The current behavior is inconvenient in terms of compatibility with Vim script, and I can't find the specifications in document.
# in def block
var Ref = s:foo
var Ref2 = function('s:foo')

Proposal:
Allow Vim9 script functions defined in Vim script to reference functions in s:foo format.
Advantage:

It will be convenient to replace some of the plugin processes written in Vim script with Vim9 script without changing the function names.
Improved compatibility with conventional Vim script.
The current specification is unnatural because it can be processed normally in other cases

Although I would prefer always using an uppercase letter for user
functions, in this case it was already possible to define the function,
so we might as well allow for referencing it.
--
`When any government, or any church for that matter, undertakes to say to
its subjects, "This you may not read, this you must not see, this you are
forbidden to know," the end result is tyranny and oppression no matter how
holy the motives' -- Robert A Heinlein, "If this goes on --"
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Thank you very much for checking and receiving my PR!


Device restart_service fails silently

Replication steps
>>> from balena import Balena
>>> balena = Balena()
>>> credentials = {<EMAIL_ADDRESS>'password': '******' }
>>> balena.auth.login(**credentials)
>>> balena.models.device.restart_service('uuid', 1234)

Nothing is returned from the last command, and the service is not restarted on the dashboard.
Tested with the latest balena-sdk version (14.4.0)

Fixed in v14.4.4 with https://github.com/balena-io/balena-sdk-python/pull/359


Any way to pass a feedable placeholder for num_spectrogram_bins  in linear_to_mel_weight_matrix?

I need to run auido->spectrogram->mel graph for three different frame sizes. Ideally, I would have frame_size input tensor of the stft function be a placeholder and feed it in a loop. However, when computing the mel scale, tf does a value check for the number of bins, which according to my understanding, means that the number of bins needs to be specified during the graph construction.
Any suggestions?
Below is the link to the problematic code
https://github.com/tensorflow/tensorflow/blob/f7cbb757cf51143a1c7d9db5a812ac165941adf4/tensorflow/contrib/signal/python/ops/mel_ops.py#L72
Edit: I am referencing the official tutorial.
https://www.tensorflow.org/api_guides/python/contrib.signal#Computing_log_mel_spectrograms
The very first line essentially begins the problem chain.
Edit: I just commented out the line below that initiates the value checks, and everything ran smoothly. I was able to feed in frame_length.
https://github.com/tensorflow/tensorflow/blob/f7cbb757cf51143a1c7d9db5a812ac165941adf4/tensorflow/contrib/signal/python/ops/mel_ops.py#L151

Nagging Awaiting TensorFlower: It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

Nagging Awaiting TensorFlower: It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

@siryog90 @reedwm I take a look at the implementation of linear_to_mel_weight_matrix. It looks that num_spectrogram_bins is only passed to math_ops.linspace, which already performs validations in both shape function and kernel. For that I think it makes sense to remove the validation in python, so that non python int values could be used.
Create a PR #17404 for the fix. Please take a look.


add back the color='red' tags, which represent marked nodes

Great work, thanks for your contribution 🐧


[Unit Tests] components-contrib: Gate PRs to ensure no PR reduces test coverage

In what area(s)?

/area test-and-release
Describe the feature

Enable PR gating on unit test coverage to ensure we do not regress on test coverage. For now the agreed upon rule is that no PR should reduce the test coverage. We may expand these rules later.
Part of dapr/dapr#2226
Release Note

RELEASE NOTE: ADD Added codecov rules to ensure that no PR reduces unit test coverage.

/assign


can you update to tensorflow v1.x

Hi,
can you update to tensorflow v1.x. That will help a lot.

Check out https://github.com/SivanKe/Attention-OCR/pull/1
Also fixes bugs, adds gradient clipping, etc.


Please add scale option to image cropping

I tried to use this code to generate a splash screen, the logo placed in the middle of the 2732x2732 px, quite small compared to the space around it, and even so, it cut out parts of it in the generated resolutions.

@hembachrterran Added --transform-splash thanks to latest community PRs, wich will probably solve your issue.
You can try it with:
npm run capacitor-resources --transform-splash cover
or
npm run capacitor-resources --transform-splash crop
And check which works out best for you ;)


Fix download

Currently the download uses the puppet command.  This breaks when used against hosted forge repos.  This change modifies the code to use the download feature in the puppet forge library.

The code no longer builds on 1.9 ruby as the latest version of json_pure which is pulled in by hiera won't install on less than ruby 2.0.

This needs a rebase now.

The library now uses puppet module install to download. Given its age, I'm closing this.


Move memoize to single file in containers

Pull Request Test Coverage Report for Build 232

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

Totals

Change from base Build 231:
0.0%

Covered Lines:
414

Relevant Lines:
422

💛  - Coveralls


HIP for changing consensus group deselection dynamics

This HIP introduces a change to the consensus election dynamics on the Helium L1 blockchain to provide additional tunability for the deselection scoring of current consensus group members. The aim is to more quickly and reliable eject poor performers from the group.
Code for this change has already been merged as blockchain-core PR 1468 https://github.com/helium/blockchain-core/pull/1468

Hi @PaulVMo, does this need to be a HIP if it's already deployed? Please correct me if I have that wrong.

It's deployed but behind a chain variable that's bound to the 1.15.x release. I believe it's a fair thing to have a HIP and a quick vote on (at least by validator operators) since it changes election dynamics.

Okay. I likely see others having the same question I just asked. Can we put that in the context of this HIP? I added this as two comments to be added to the Summary portion. Otherwise looks great and, we'll get it out!

@vincenzospaghetti great questions and feedback. I updated the summary to address the points you brought up. Please let me know if you have any more questions.

I’d be supportive of this coming to a full discussion (for a few days), some education from the author and the foundation, and a relatively quick vote. Ideally we could activate this variable around Nov 1.

All of that sounds good @abhay. I see this definitely needing some education from you and @PaulVMo. How can the Foundation help education? If you have a proposal of that we can add in the comments here that would be ideal.
Let's go ahead and open a channel.


Added custom snippet

added a custom snippet to align the playlists and folders on the left sidebar to all the other categories in the sidebar.

The first part of the code can be replaced with anything shorter, but for some reason when I remove the line, it reverts the last few lines.

made the changes in the latest commit

Please include the preview image file itself

Please include the preview image file itself

alright, inside the same folder as all the others?

I hope it is fixed now

Yo @kyrie25, it says it's block because you still have requested changes. Are we good now?

All good.


RescueEnsureAlignment and IndentationWidth don't work properly together

An indented begin rescue expression does not indent the inner begin section correctly nor the end

Expected behavior
    result = begin
      yield
    rescue Exception => e
      exit(1)
    end

should be reformated to be
    result = begin
               yield
             rescue Exception => e
               exit(1)
             end

Actual behavior
it reformats to
    result = begin
      yield
             rescue Exception => e
               exit(1)
    end

where both the yield and the end are not indented.
Steps to reproduce the problem
The above code should reproduce
RuboCop version
➜ rubocop -V
0.76.0 (using Parser <IP_ADDRESS>, running on ruby 2.5.3 x86_64-darwin18)

We have this issue too, we just have the cop disabled because alignment is much worse when this cop is turned on. would love a nice fix to it someday

Subscribing in case this is related to / fixed by #7531
My use case: https://github.com/rubocop-hq/rubocop/pull/7531#issuecomment-602966770

This is still an issue...

I think this is now addressed by enabling the Layout/BeginEndAlignment cop with EnforcedStyleAlignWith: start_of_line

I think this is now addressed by enabling the Layout/BeginEndAlignment cop with EnforcedStyleAlignWith: start_of_line
I tested this against out application and saw little-to-no issues anymore. Quite nice actually. So you can consider my comment resolved. Might be able to just close this issue

Ah yes I meant to test this. If someone else has tested then I think we can close it. Thanks for the fix!


.travis.yml: sudo false for Docker containers

Enables new Docker container infrastructure.

Coverage decreased (-2.32%) to 89.63% when pulling 03397fb578d81e5e9879cdbe637c6e1968254bd4 on msabramo:patch-4 into 337c05f95cc3b5435f9a7718bbd62debbf282d31 on jakubroztocil:master.

Coverage decreased (-2.32%) to 89.63% when pulling 03397fb578d81e5e9879cdbe637c6e1968254bd4 on msabramo:patch-4 into 337c05f95cc3b5435f9a7718bbd62debbf282d31 on jakubroztocil:master.

Coverage decreased (-2.32%) to 89.63% when pulling 13f27b821ec8d9ede1e54c357cac2d162c6447b6 on msabramo:patch-4 into 337c05f95cc3b5435f9a7718bbd62debbf282d31 on jakubroztocil:master.

Coverage decreased (-0.08%) to 91.87% when pulling 398dc2edc48b505109cfc953804c1fa4bf409791 on msabramo:patch-4 into 337c05f95cc3b5435f9a7718bbd62debbf282d31 on jakubroztocil:master.

Coverage decreased (-0.08%) to 91.87% when pulling 398dc2edc48b505109cfc953804c1fa4bf409791 on msabramo:patch-4 into 337c05f95cc3b5435f9a7718bbd62debbf282d31 on jakubroztocil:master.

Coverage remained the same at 92.75% when pulling 55fa975ae5e84083a7fccc4c4d5ecc45a04f37b4 on msabramo:patch-4 into e6e94398ae548aead53b20bc26ad7f0723b95b03 on jakubroztocil:master.

Hmmm, pypy and pypy3 timed out.

Thanks!
The timeout is related to #308.

That's a pretty cool feature btw. I'm wondering if the caching this change enables works for pip packages as well.

Yeah I haven't tried the caching yet. On other projects, usually just setting this speeds up the builds a bit though. It might be interesting to cache necessary wheels.


bit.ly clone is much, much faster

like, much faster. please 🙏 try it: https://github.com/mutalis/bitly
could it be easily ported here? 🌚
this would certainly benefit way over 80% of people and no, it can't "be this fast" in other ways, i believe. 👶
also, ✋ i'm under the impression that whatever you're doing to redirect under the hood is a bit archaic and can lead to breaking history when hitting the back button and going to the redirect page which enters into a back and forward redirection loop, you know. yesterday it happened to me, today i couldn't reproduce this.
(now, i'm also wondering if it'd be possible to also clone bitly stats generation somehow... or at least slowly get its features, starting with a "simple" click counter...) :P 🌞
cheers! 🍕

well, it was all already considered and elaborated.
i hope this can be useful to someone in the future, even if it gets closed.
cheers!

This feels spammy


[BEAM-564] Updates Python source API to allow reporting consumed and remaining number of split points

With this update Python BoundedSource/RangeTracker API can report consumed and remaining number of split points while performing a source read operations. Java SDK source API already supports reporting these signals.
These signals can be used by runner implementations, for example, to perform scaling decisions.
This provides a slightly simplified API compared to previous PR https://github.com/apache/incubator-beam/pull/881.
Main differences compared to https://github.com/apache/incubator-beam/pull/881 are following.
(1) set_done()/done() methods were removed from the RangeTracker interface.
Downside is that RangeTracker will be unable to provide the signal that all records have been consumed. I think this signal is unnecessary since a runner can detect that anyways since the reader loop of the source ends at that point.
(2)
Callback between BoundedSource and RangeTracker was changed from reporting remaining number of split points to reporting unclaimed number of split points. This makes the implementation of the callback simpler for source authors.

R: @robertwb

Fixed conflicts.
Robert, please take a look when you have time.

Changes Unknown when pulling 2ce563eaf8ac146d88a5a33d5dd29b02822ff814 on chamikaramj:limited_parallelism_updated into ** on apache:python-sdk**.

Changes Unknown when pulling 2ce563eaf8ac146d88a5a33d5dd29b02822ff814 on chamikaramj:limited_parallelism_updated into ** on apache:master**.

This PR was automatically closed (and cannot be reopened) after python-sdk branch was deleted. Recreated a new PR against master: https://github.com/apache/beam/pull/1890


Fix minor doc issue

Automatic CI verification build not done, please verify manually.

Hi @attilamolnar
I know this is trivial fix, but unfortunately our policy is to not accept contributions without a CLA.
To accept the contribution and any future contribution we will need a Contributor’s Licence Agreement (CLA) signed by yourself or by whoever has the authority to do so. You can find an agreement to sign here, which can be signed and returned to us, or you could create an mbed account and accept a slightly different agreement here with a click through if this is a personal contribution.
Thanks in advance for your understanding.

Hi,
This is a personal contribution so I have created an mbed account (https://developer.mbed.org/users/attilamolnar/) and accepted the contributor agreement.

Thanks for responding so quickly! I have now merged the fix.


How to modify AlignMtcnn.py on images containing multiple faces?

How to use alignmtcnn for dataset containing images and in each image there are multiple faces?

Something is wrong, so I suggest using the MTCNN from the facenet library on a single image. Pick a good image, with a clearly identifiable face. Then tweak the parameters until you can get something working...
i.e. perhaps your min face size was too small...
What is the resolution of the images?
I've made a script to plot the MTCNN results on detecting faces, which may be useful in your case just to tweak the parameters...
https://github.com/cjekel/cjekel.github.io/blob/master/assets/2017-05-01/faceDetect.py
with a short description of the script here > http://jekel.me/2017/How-to-detect-faces-using-facenet/

https://github.com/davidsandberg/facenet/blob/master/src/align/align_dataset_mtcnn.py
bounding_boxes, _ = align.detect_face.detect_face(img, minsize, pnet, rnet, onet, threshold, factor)
#actually, this line will detect all faces in the image, the following code will only select one face in the middle of the image.
nrof_faces = bounding_boxes.shape[0]
if nrof_faces>0:
det = bounding_boxes[:,0:4]
det_arr = []
img_size = np.asarray(img.shape)[0:2]
if nrof_faces>1:
if args.detect_multiple_faces:
for i in range(nrof_faces):
det_arr.append(np.squeeze(det[i]))
else:
bounding_box_size = (det[:,2]-det[:,0])(det[:,3]-det[:,1])
img_center = img_size / 2
offsets = np.vstack([ (det[:,0]+det[:,2])/2-img_center[1], (det[:,1]+det[:,3])/2-img_center[0] ])
offset_dist_squared = np.sum(np.power(offsets,2.0),0)
index = np.argmax(bounding_box_size-offset_dist_squared2.0) # some extra weight on the centering
det_arr.append(det[index,:])
else:
det_arr.append(np.squeeze(det))

Thank you ,very useful


Configure Reverse Proxy handler

The reverse proxy must be updated still to handle the preferred URL.

Actually not a DNS problem--updated to reflect. The reverse proxy must be configured since the url will be a subpath of the existing domain name, not a subdomain.

Resolved--added CNAME record and reverse proxy configuration.


VMware exporter error

What did you do?
Hello team, I need some helps because it's impossible for me to scrape my vcenter and I don't understood.
my vmware config.yml :
default:
  vsphere_host: "fr-vcenter-vdi-01"
  vsphere_user<EMAIL_ADDRESS>  vsphere_password: "XXXXXXX"
  ignore_ssl: True
  specs_size: 5000
  fetch_custom_attributes: False
  fetch_tags: False
  fetch_alarms: True
  collect_only:
    vms: True
    vmguests: True
    datastores: True
    hosts: True
    snapshots: True

Pip list :
root@fs01m00prom01:~# pip3 list
Package           Version
----------------- --------------
attrs             21.4.0
Automat           20.2.0
certifi           2020.6.20
cffi              1.15.0
chardet           4.0.0
constantly        15.1.0
cryptography      37.0.2
httplib2          0.18.1
hyperlink         21.0.0
idna              2.10
incremental       21.3.0
pip               20.3.4
prometheus-client 0.0.19
pyasn1            0.4.8
pyasn1-modules    0.2.8
pycparser         2.21
pycurl            <IP_ADDRESS>
PySimpleSOAP      1.16.2
python-apt        2.2.1
python-debian     0.1.39
python-debianbts  3.1.0
pytz              2022.1
pyvmomi           7.0.3
PyYAML            6.0
reportbug         7.10.3+deb11u1
requests          2.25.1
service-identity  21.1.0
setuptools        52.0.0
six               1.16.0
Twisted           22.4.0
typing-extensions 4.2.0
urllib3           1.26.5
vmware-exporter   0.18.3
wheel             0.34.2
yamlconfig        0.3.1
zope.interface    5.4.0

What did you expect to see?
No response
What did you see instead? Under which circumstances?
Need to collect my vsphere info
System information
Linux 5.10.0-13-amd64 x86_64
Prometheus version
root@fs01m00prom01:~# prometheus --version
prometheus, version 2.35.0 (branch: HEAD, revision: 6656cd29fe6ac92bab91ecec0fe162ef0f187654)
  build user:       root@cf6852b14d68
  build date:       20220421-09:53:42
  go version:       go1.18.1
  platform:         linux/amd64

Prometheus configuration file
- job_name: "vcenter"
    metrics_path: '/metrics'
    scrape_interval: 60s
    scrape_timeout: 60s
    static_configs:
      - targets:
        - 'fr-vcenter-vdi-01.akawan.corp'
    relabel_configs:
      - source_labels: [__address__]
        target_label: __param_target
      - source_labels: [__param_target]
        target_label: instance
      - target_label: __address__
        replacement: localhost:9272

Alertmanager version
No response
Alertmanager configuration file
No response
Logs
No response

Thanks for your report. It looks as if this is actually a question about usage and not development.
It makes more sense to ask questions like this on the prometheus-users mailing list or our forums rather than in a GitHub issue. In our community channels, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


[Improvement] Block azure.functions module collided trigger names

Investigative information
Please provide the following:

Timestamp: 5/6/2020
Function App name: fa-clipy
Function name(s) (as appropriate): http
Core Tools version:

Repro steps
Provide the steps required to reproduce the problem:

Create a python function app
Create an HTTP trigger with the name "http"
Deploy the function app to a Linux Dedicated app.
Hit /api/http endpoints

Expected behavior
Provide a description of the expected behavior.
The function should go through and execute.

Actual behavior
Provide a description of the actual behavior observed.
The function app throws a 500 (functions not found in azure module) due to a name collision with azure.functions.http module.

Known workarounds
Provide a description of any known workarounds.
Rename trigger name "http" to "httptrigger"

Contents of the requirements.txt file:
Provide the requirements.txt file to help us find out module related issues.
azure-functions
Related information
Provide any related information

Links to source
Bindings used

cc: @divyagandhii

V2 programming model addresses this issue.


Apply maximize direction to pruning example scripts.

Apply maximize direction to pruning example scripts. Even though Optuna supports maximize direction since v0.9.0, some pruning example scripts still use minimize returning 1.0 - accuracy in their objective functions, e.g., https://github.com/pfnet/optuna/blob/master/examples/pruning/xgboost_integration.py#L57

Merged: #526


Polish characters are garbled

Hi,
I want to add chapters with Polish characters in it (żółćęśąźń). Unfortunately when I do it using mp3chaps it becomes garbled, probably some issues with proper character conversion.
I attached a txt file and mp3 file disguised as txt, as mp3 upload is not permitted but it is a tiny 10 seconds mp3 file that has less than 100kB.
I based my report on #6 because it was a very well formed bug report.
When Polish characters are added to the test.chapters.txt file (where test is the name of the mp3), and the mp3chaps -i test.mp3 is used, all Polish characters become garbled in the chapter text.
Steps to recreate

Create a valid mp3 file called test.mp3
Create a text file called test.chapters.txt
Insert the following line into the text file:

00:00:00.000 żółćęśąźń
00:00:09.000 zolcesazn

Run the following command:

mp3chaps -i test.mp3

Full output from result
λ mp3chaps -l test.mp3
Chapters:
Å¼Ã³Å‚Ä‡Ä™Å›Ä…ÅºÅ„
zolcesazn

Expected result
test.mp3 should have a two chapter markers called żółćęśąźń and zolcesazn, and running mp3chaps -l test.mp3 should show:
λ mp3chaps -l test.mp3
Chapters:
żółćęśąźń
zolcesazn

Machine details

Windows 10 (build number: 19041.450)
mp3chaps vLatest from pip3 (at time of reporting)
Python 3.8.5

test.chapters.txt
test.mp3.txt

Any chances on looking into this?


Abbreviate mode line if on small display

I think something like this is accomplished by https://github.com/mrkkrp/cyphejor? Or maybe we should consider https://github.com/Malabarba/smart-mode-line.

Not really an issue since our mode line is a lot shorter now.


[Tutorial Question]BUILD file not found in any of the following directories

run: bazel build //ios-app:ios-app
ERROR: /tutorial/ios-app/BUILD:14:16: no such package 'Pods/AgoraAudio_iOS': BUILD file not found in any of the following directories. Add a BUILD file to a directory to mark it as a package.
 - /tutorial/Pods/AgoraAudio_iOS and referenced by '//ios-app:ios-app_objc'
ERROR: Analysis of target '//ios-app:ios-app' failed; build aborted: Analysis failed
INFO: Elapsed time: 22.698s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (36 packages loaded, 128 targets configured)
    currently loading: @local_config_cc//

├── README.md
├── WORKSPACE
├── appengine.BUILD
├── backend
├── bazel-bin -> /private/var/tmp/_bazel_cc/fe3145c93c0cfda8271c29935d89fd9a/execroot/__main__/bazel-out/darwin-fastbuild/bin
├── bazel-out -> /private/var/tmp/_bazel_cc/fe3145c93c0cfda8271c29935d89fd9a/execroot/__main__/bazel-out
├── bazel-testlogs -> /private/var/tmp/_bazel_cc/fe3145c93c0cfda8271c29935d89fd9a/execroot/__main__/bazel-out/darwin-fastbuild/testlogs
├── bazel-tutorial -> /private/var/tmp/_bazel_cc/fe3145c93c0cfda8271c29935d89fd9a/execroot/__main__
└── ios-app
    ├── BUILD
    ├── Pods
    │   └── AgoraAudio_iOS
    │       ├── AgoraAIDenoiseExtension.framework
    │       ├── AgoraCore.framework
    │       ├── AgoraRtcKit.framework
    │       ├── AgoraSoundTouch.framework
    │       ├── Agorafdkaac.framework
    │       └── BUILD.bazel
    └── UrlGet
        ├── AppDelegate.h
        ├── AppDelegate.m
        ├── UrlGet-Info.plist
        ├── UrlGetViewController.h
        ├── UrlGetViewController.m
        ├── UrlGetViewController.xib
        └── main.m

ios-app/BUILD

load("@build_bazel_rules_ios//rules:app.bzl", "ios_application")

objc_library(
    name = "UrlGetClasses",
    srcs = [
         "UrlGet/AppDelegate.m",
         "UrlGet/UrlGetViewController.m",
         "UrlGet/main.m",
    ],
    hdrs = glob(["UrlGet/*.h"]),
    data = ["UrlGet/UrlGetViewController.xib"],
)

ios_application(
    name = "ios-app",
    bundle_id = "Google.UrlGet",
    families = [
        "iphone",
        "ipad",
    ],
    minimum_os_version = "9.0",
    infoplists = [":UrlGet/UrlGet-Info.plist"],
    visibility = ["//visibility:public"],
    deps = [
        ":UrlGetClasses",
        "//Pods/AgoraAudio_iOS"
        ],
)

Pods/AgoraAudio_iOS/BUILD

load("@build_bazel_rules_ios//rules:framework.bzl", "apple_framework")

apple_framework(
    name = "AgoraAudio_iOS",
    platforms = {"ios": "8.0"},
    vendored_dynamic_frameworks = [
        "AgoraAIDenoiseExtension.framework",
        "AgoraCore.framework",
        "AgoraRtcKit.framework",
        "AgoraSoundTouch.framework",
        "Agorafdkaac.framework",
    ],
    visibility = ["//visibility:public"],
)

If i move WORKSPACE to ios-app
run bazel build :ios-app in ios-app
ERROR: /ios-app/Pods/AgoraAudio_iOS/BUILD.bazel:3:16: every rule of type apple_framework_packaging implicitly depends upon the target '@build_bazel_rules_ios//rules/framework:framework_packaging', but this target could not be found because of: error loading package '@build_bazel_rules_ios//rules/framework': Unable to find package for @rules_python//python:defs.bzl: The repository '@rules_python' could not be resolved.
ERROR: Analysis of target '//:ios-app' failed; build aborted: Analysis failed
INFO: Elapsed time: 0.173s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (2 packages loaded, 2 targets configured)

Pods/AgoraAudio_iOS/BUILD.bazel

load("@build_bazel_rules_ios//rules:framework.bzl", "apple_framework")

apple_framework(
    name = "AgoraAudio_iOS",
    platforms = {"ios": "8.0"},
    vendored_dynamic_frameworks = [
        "AgoraAIDenoiseExtension.framework",
        "AgoraCore.framework",
        "AgoraRtcKit.framework",
        "AgoraSoundTouch.framework",
        "Agorafdkaac.framework",
    ],
    visibility = ["//visibility:public"],
)

Can you link to what tutorial exactly you are following? This doesn't seem to be related to cocoapods-bazel but a general iOS bazel problem.
That said, noticed this in the error msg:
- /tutorial/Pods/AgoraAudio_iOS and referenced by '//ios-app:ios-app_objc'

and that path doesn't exist in the folder structure you shared above, maybe your deps should look like this instead?
    deps = [
        ":UrlGetClasses",
        "//ios-app/Pods/AgoraAudio_iOS"
        ],

Also, the WORKSPACE file should always be at the root

@thiagohmcruz
Hi, tutorial from https://docs.bazel.build/versions/main/tutorial/ios-app.html
deps = [
        ":UrlGetClasses",
# Can't change it like that, because this library might depend on other libraries
        "//ios-app/Pods/AgoraAudio_iOS"
        ],

like generated BUILD by cocoapods-bazel
load("@build_bazel_rules_ios//rules:framework.bzl", "apple_framework")

apple_framework(
    name = "Bagel",
    srcs = glob([
        "iOS/Source/*.h",
        "iOS/Source/*.m",
    ]),
    platforms = {"ios": "8.0"},
    public_headers = [
        "iOS/Source/Bagel.h",
        "iOS/Source/BagelBaseModel.h",
        "iOS/Source/BagelBrowser.h",
        "iOS/Source/BagelCarrierDelegate.h",
        "iOS/Source/BagelConfiguration.h",
        "iOS/Source/BagelController.h",
        "iOS/Source/BagelDeviceModel.h",
        "iOS/Source/BagelProjectModel.h",
        "iOS/Source/BagelRequestCarrier.h",
        "iOS/Source/BagelRequestInfo.h",
        "iOS/Source/BagelRequestPacket.h",
        "iOS/Source/BagelURLConnectionInjector.h",
        "iOS/Source/BagelURLSessionInjector.h",
        "iOS/Source/BagelUtility.h",
    ],
    visibility = ["//visibility:public"],
    deps = ["//Pods/CocoaAsyncSocket"],
)

May be need move WORKSPACE to ios-app/
but, there is an error:
ERROR: //ios-app/Pods/AgoraAudio_iOS/BUILD.bazel:3:16: every rule of type apple_framework_packaging implicitly depends upon the target '@build_bazel_rules_ios//rules/framework:framework_packaging', but this target could not be found because of: error loading package '@build_bazel_rules_ios//rules/framework': Unable to find package for @rules_python//python:defs.bzl: The repository '@rules_python' could not be resolved.
ERROR: Analysis of target '//Pods/AgoraAudio_iOS:AgoraAudio_iOS' failed; build aborted: Analysis failed
INFO: Elapsed time: 0.144s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (1 packages loaded, 0 targets configured)

fixed by import rules_python and move the WORKSPACE to ios-app/
git_repository(
    name = "rules_python",
    remote = "https://github.com/bazelbuild/rules_python.git",
    tag = "0.3.0",
)

Closing, sounds like the issue is resolved, doesn't appear related to this repo.


Bump GitHub Actions Checkout

This avoids deprecation of Node 16 which is likely to come soon given that it is now beyond end of life.

Redundant since 131952248c056c4a80a815870a7a6923969029a1


EULA: update the service code to avoid deadlocks

In certain cases, when showing a EULA dialog is impossible (the current thread holds a read or a write lock), we'll show a corresponding notification.
An exception may still be thrown in those cases, but this shouldn't matter: after accepting the EULA, everything should start working correctly. Or, in the worst case, after accepting EULA and restarting the IDE.
There are new internal actions (-Didea.is.internal=true) to show the notifications (for QA purposes).
For cases when something's completely wrong and the user still cannot accept the EULA or see the notification, there are new non-internal actions: Show TF Client EULA, Show TF SDK Agreement.
The users will be able to invoke them manually if required.
PropertyServiceImpl is now more thread-safe.

Closes #539.

Tests ok.


Performance with CPU

I notice you put the model on the Gradio demo, and it seems to be running nicely. However, when I attempt to "dockerize" the model and run it in the cloud with the following configuration: 4 vCPU and 16GB RAM, it remains frozen or extremely sluggish (5 minutes per picture).
Could you please give the infrared configuration on the Gradio demo? Is there anything I did wrong?

I try to modify the model structure to the same configuration as the cord-v2 public model on the Hugging face.
However, CPU run time still costs between 20 and 60 seconds. I tested it on Docker images with four Intel Core i7 7700k vCPUs and the Amazon EC2 environment: 4 vCPUs and 16 GB RAM.
I believe the issue stems from my vocabulary and the complexity of my prediction scenario. I'm working with menu image data and Vietnamese.

I ran the DocVQA Task1 model locally(CPU) and tested it on Hagging Face demo. In both cases, the result took around 90 sec per question.


[Feature] Allow reality block and statue block to be cleared/reset.

Describe your idea
I think this pretty self explanatory; but, a way to reset the reality and statue block. Otherwise removing them causes all of them to no longer stack for all of eternity.
Additional context

Linked Issues
No response

Oh, this sounds like a nice idea, I will do it after finish #18
I guess, default logic with crafting clears nbt data should be enough

With 0.0.9, you able to cleanup any internal data. Just put flexible reality anchor or flexible statue to craft panel


⚠ Bofrost.fr has degraded performance

In 6a2ccb9, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1315 ms

Resolved: Bofrost.fr performance has improved in 6fe3df9 after 6 minutes.


incorrect handling of array.shape with nopython=True

Each of the following variations on the same code results in numba raising a 'Tuple index out of range' error regarding (it seems) array.shape[1].
The checks on len(array.shape) and array.ndim are completely ignored.
Code to reproduce:
from numba import jit
import numpy as np
import traceback

@jit(nopython=True)
def test_1(array: np.ndarray):
    if array.ndim == 2:
        if array.shape[1] == 2:
            return 1
    return 0

@jit(nopython=True)
def test_2(array: np.ndarray):
    if array.ndim == 2 and array.shape[1] == 2:
        return 1
    return 0

@jit(nopython=True)
def test_3(array: np.ndarray):
    if len(array.shape) == 2:
        if array.shape[1] == 2:
            return 1
    return 0

@jit(nopython=True)
def test_4(array: np.ndarray):
    if len(array.shape) == 2 and array.shape[1] == 2:
        return 1
    return 0

array = np.empty((1,))

for test in (test_1, test_2, test_3, test_4):
    try:
        test(array)

    except:
        traceback.print_exc()

Tracebacks:
Traceback (most recent call last):
  File "C:/Users/admin/Documents/Projects/scraps/numba_bug_test.py", line 40, in <module>
    test(array)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\dispatcher.py", line 350, in _compile_for_args
    error_rewrite(e, 'typing')
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\dispatcher.py", line 317, in error_rewrite
    reraise(type(e), e, None)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\six.py", line 658, in reraise
    raise value.with_traceback(tb)
numba.errors.TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Internal error at <numba.typeinfer.StaticGetItemConstraint object at 0x000001BF103C9EB8>:
--%<----------------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\errors.py", line 627, in new_error_context
    yield
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\typeinfer.py", line 343, in __call__
    value=ty, index=self.index,
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\typing\context.py", line 300, in resolve_static_getitem
    return self.resolve_function_type("static_getitem", args, kws)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\typing\context.py", line 204, in resolve_function_type
    res = self._resolve_builtin_function_type(func, args, kws)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\typing\context.py", line 221, in _resolve_builtin_function_type
    res = defn.apply(args, kws)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\typing\templates.py", line 207, in apply
    sig = generic(args, kws)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\typing\builtins.py", line 611, in generic
    ret = tup.types[idx]
IndexError: tuple index out of range

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\typeinfer.py", line 144, in propagate
    constraint(typeinfer)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\typeinfer.py", line 351, in __call__
    self.fallback(typeinfer)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\contextlib.py", line 130, in __exit__
    self.gen.throw(type, value, traceback)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\errors.py", line 635, in new_error_context
    six.reraise(type(newerr), newerr, tb)
  File "C:\Users\admin\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\six.py", line 659, in reraise
    raise value
numba.errors.InternalError: tuple index out of range
[1] During: typing of static-get-item at C:/Users/admin/Documents/Projects/scraps/numba_bug_test.py (9)
--%<----------------------------------------------------------------------------

File "numba_bug_test.py", line 9:
def test_1(array: np.ndarray):
    <source elided>
    if array.ndim == 2:
        if array.shape[1] == 2:
        ^

This is not usually a problem with Numba itself but instead often caused by
the use of unsupported features or an issue in resolving types.

File "numba_bug_test.py", line 16:
def test_2(array: np.ndarray):
    if array.ndim == 2 and array.shape[1] == 2:
    ^

This is not usually a problem with Numba itself but instead often caused by
the use of unsupported features or an issue in resolving types.

File "numba_bug_test.py", line 24:
def test_3(array: np.ndarray):
    <source elided>
    if len(array.shape) == 2:
        if array.shape[1] == 2:
        ^

This is not usually a problem with Numba itself but instead often caused by
the use of unsupported features or an issue in resolving types.

File "numba_bug_test.py", line 31:
def test_4(array: np.ndarray):
    if len(array.shape) == 2 and array.shape[1] == 2:
    ^

This is not usually a problem with Numba itself but instead often caused by
the use of unsupported features or an issue in resolving types.

Thanks for the report, I can reproduce. What you are seeing is due to two things interacting:

Numba, by default, has to compile all branches as if the code were static (there is a caveat, will explain below).
As part of compilation, Numba tries to turn get/set with a constant index into a special "static get/set item" node in the intermediate representation it uses when compiling. It does this as it leads to easier analysis/better performance. The assumption made though is that the constant index is a valid index.

As a combination of 1. and 2., the branch which looks dead still needs compiling and then because the array.shape of a 1D array is a length one tuple, accessing index 1 is out of bounds,  hence the error.
Switching off the rewrite passes (one of which is the static get/set item rewrite) makes the code work as expected:
from numba import jit
import numpy as np

@jit(nopython=True, no_rewrites=True)
def test_1(array: np.ndarray):
    if array.ndim == 2:
        if array.shape[1] == 2:
            return 1
    return 0

array = np.empty((1,))
print(array.shape)
print(array.ndim)
test_1(array)

How to fix this? In the context of the Numba compilation pipeline there is currently a compiler pass which prunes dead branches based on comparison to None, i.e. this pattern:
@jit(nopython=True)
def dead_branch_prune(a, maybe_none):
    if maybe_none is None:
        return a
    else:
        return a + maybe_none

print(dead_branch_prune(10, None))
print(dead_branch_prune(10, 20))

this code would suffer the same problem as in the original post, in that there's a branch that could be taken return a + maybe_none where, if maybe_none is actually None then trying to statically compile it would lead to an error (there's no way to add numba.types.Integer and None). To get around this Numba analyses the code, spots that there's a comparison between an input argument and None and removes whichever branch won't be taken. I think it might be possible to apply a similar scheme to the case above, in that array.ndim is a compile time constant being compared to a user defined compile time constant (2), so the dead branch could be pruned.

https://github.com/numba/numba/pull/4113 fixes the first two cases, the latter two would be better off waiting for true constant propagation. Hope this helps.


Update the section on creating an identity

The command in the section Create an identity on Azure for retrieving the client ID will not work.
The example given is: az ad sp show --id http://secrets-store-test --query 'appId' -otsv
According to https://docs.microsoft.com/en-us/cli/azure/ad/sp?view=azure-cli-latest#az-ad-sp-show the value of --id would not work in this instance.   "az ad sp list" might be an option to get the client  ID

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

ID: 48500978-a4c7-4fd8-2c5a-3fca53607600
Version Independent ID: d87865b6-266a-a2fb-631a-d7f66b62e304
Content: Kubernetes Secrets Store CSI Driver integration with AKS on Azure Stack HCI - AKS-HCI
Content Source: azure-stack/aks-hci/secrets-store-csi-driver.md
Service: azure-stack
Sub-service: aks-hci
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@cybersmithio Thanks for your feedback. We will update shortly.
#in-progress

@cybersmithio
Thanks for the feedback!
As @mattbriggs the document update is in-progress.

Updated.
#please-close


build.prop list

Hello friend, where is the list with build.props, I wanted to download and use this list in a project.
It would be possible?

There is no list of build.prop values. The only list you'll find here is the list of certified device fingerprints (in the /common directory).

@Didgeridoohan Thank you very much, colleague.


WIP ROX-259330: use scl postgres upstream images

Description

The downstream version of the product uses the downstream SCL Postgres images owned by Red Hat.  The upstream pulls postgres from an RPM registry and installs it via RPM.  This creates a unnecessary divergence in how to process an upgrade.  As it creates 2 separate paths we must upgrade from.  It is far simpler if we simply use the same general base images for central-db in both the upstream and downstream.
Plus from an investigation standpoint if the new target images are the SCL images it would be best to have them in the upstream to more easily use them for debugging and investigation to see what utilities are installed.
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

change me!

/test ocp-4-12-operator-e2e-tests


Custom Tour Directories

This PR fixes #173.
I added a setting codetour.customTourLocation and logic to push the value into the SUB_TOUR_DIRECTORIES set. The intention is that I can add a value in .vscode/settings.json to enable tour discovery in any path needed.

Checking in with work on approvals for the CLA.

Checking in with work on approvals for the CLA.

LGTM!

@lostintangent PR is good to go.


Troubleshooting openSSL

Use content from debian 9 walkthrough
cc @mtbc

Staged at http://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/troubleshooting.html#openssl-version.


Listener Satet not supported

Betriebsystemname	Microsoft Windows 10 Home
Version	10.0.18363 Build 18363
would be kind if smn can help me.
Greetings!

Renew rdpwrap.ini to support 10.0.18362.267
https://github.com/asmtron/rdpwrap/blob/master/binary-download.md

Renew rdpwrap.ini to support 10.0.18362.267
https://github.com/asmtron/rdpwrap/blob/master/binary-download.md

thank you bro! no it says accses denied when i try to login to my user.

thank you bro! no it says accses denied when i try to login to my user.


Add the steps to setup gbq integration testing to the contributing docs

Pull requests from forked repositories cannot access the existing secure Google BigQuery credentials which are on Travis at pydata/pandas. In order for contributors to run Google BigQuery integration tests on Travis, they need to create a pull request on their forked repository of pandas with their own secure Google BigQuery credentials.
I've updated the contributing documentation to include the steps required to run Google BigQuery integration tests on a forked repository of pandas.

any easier way to do this?

Current coverage is 85.27% (diff: 100%)

Merging #14144 into master will not change coverage

@@             master     #14144   diff @@
==========================================
  Files           139        139
  Lines         50506      50506
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits          43067      43067
  Misses         7439       7439
  Partials          0          0

Powered by Codecov. Last update 0323336...b3a7031

any easier way to do this?

The easiest way is to use the existing encrypted credentials that are already on Travis, but we would need to create a branch from master each time. If we don't use existing credentials, then we need to ask contributors to create their own encrypted credentials. Should I consolidate text and reduce the number of steps?

The easiest way is to use the existing encrypted credentials that are already on Travis, but we would need to create a branch from master each time.

what does this mean? most branches are forks of master

Sorry that I wasn't clear.  I meant to say that we would need to create a branch from pydata/pandas which is not on a fork.
For example: For this PR, gbq integration tests wouldn't be skipped if the branch was pydata:add-gbq-integration-test-setup-to-docs , but would be skipped with parthea:add-gbq-integration-test-setup-to-docs

ok I c. another (maybe easier option), is to have the travis secure key embedded in a file, then we could have a script which a user could run to take there credentials and update the encoded credentials file and the key very easily. (and of course make sure to exclude them from being added to the repo).

is this the 'name' of the secure key that we have? "$encrypted_1d9d7b1f171b_iv"

Yes, that is the name of the environment variables encrypted_1d9d7b1f171b_iv and encrypted_1d9d7b1f171b_key which contain the private iv and key.  which is only available on the travis repo that encrypted the file.

You're right. It makes sense to automate these steps. I'll put together a script in order to reduce the steps.

@parthea Good to document this!
Some general comments:

can you clarify that we only expect to do this from contributors if they are doing a PR related to the gbq module
I would make it a separate section (and maybe put it at the end), and then refer to this section from the sections on running tests

this is not mergeable ATM / see above comments

@jreback @jorisvandenbossche All gbq tests passed at parthea/pandas#3. I've moved gbq integration testing to a separate section in the contributing docs. Please take a look.

lgtm.

Small further question, otherwise looks good!

Here is a link to the build from my branch which has my encrypted credentials so that gbq test are not skipped.  https://travis-ci.org/parthea/pandas/jobs/158060278

@parthea Nice, thanks a lot!


Reconsider ratified

I am very concerned that ratified is a dangerous word to use, even in the code, but especially in anything that an end user might see (including casual end users that might look at github issues).
What alternatives were considered? Why wasn't approved sufficient?

Why are you concerned over ratified?  I chose this word as yc.o incorporated other SDOs that had different, well, ratified states.  Approved didn't ring right as a WG adopts a document and that is a form of approval.  I wanted a word that conveyed ultimate consensus from the governing body.

I tend to agree with @xorrkaz to be consistent among SDO

As I read my comment it might come across defensive.  Not my intent.  Genuinely curious.  And if there is a better word, I'm all for it.

The point is about isse #13 so search only on published documents and not drafts. I have suggested to revert the meaning to use 'draft' rather than [ratified|published|standard]

Consistent among SDOs includes consistent with IETF, and this is an IETF supported project, and IETF is the principal user.
ratified has no meaning in the IETF. I really thing using this term will cause confusion in the IETF process later on. (Though it didn't bother @mcr when this was discussed at #13).
I am surprised that approved also triggered confusion worry for @xorrkaz, but will take that as a strong indication that that particular word isn't going to work both inside and outside the community.
Will Éric's suggestion of reversing the sense work better?
Sorry if this looks like a bike-shed argument, but naming is hard, and adding a word to the vocabulary about the status of IETF documents in particular makes me very uncomfortable.

Totally agree that naming is hard.  I settled on "ratified" way back before this was more official IETF software.  I like Éric's suggestion.

Consistent among SDOs includes consistent with IETF, and this is an IETF supported project, and IETF is the principal user.
ratified has no meaning in the IETF. I really think using this term will cause confusion in the IETF process later on. (Though it didn't bother @mcr when this was discussed at #13).

While ratified doesn't mean anything at the IETF, I think that most IETFers would understand it.
I am not attached in anyway to that word though.  approved is okay, but as Erick suggests, we could also go the opposite direction and say, draft or work-in-progress or even not approved...
What I think is that when someone does a search (particularly, when a search engine visits, which maybe we don't want), that they should not get a confusion of work-in-progress YANG modules by default unless they specifically ask for that.
I know what it's all about, and I found it really difficult to find the approved/ratified YANG module that I wanted.
If the result set for "not draft" is empty, then by all means, give the user a note, "There are WIP documents that match"

Yes, naming is hard, especially with cross-SDOs project.
I recall the "ratified" was discussed with the BBF people at the time (and IEEE if not mistaken)
Regards, Benoit

In the communication for Release notes v5.9.0, we did not agree on the change, so we leave it with the label Include Drafts.
In case you will find out better naming please open new Github issue - I am closing this one now.