Testing new data science walkthrough

Refs: https://github.com/microsoft/vscode-python/pull/18321, https://github.com/microsoft/vscode-jupyter/pull/8733, https://github.com/microsoft/vscode-python/issues/18139, https://github.com/microsoft/vscode-python/issues/18287

[ ] anyOS @karthiknadig
[ ] anyOS @amunger

Complexity: 5
Authors: @minsa110
Create Issue

Pre-req

The changes are behind experiments so you need to change your settings to "opt-into" the experience. Add the following to your user settings.json file:

  "experiments.override.gettingStarted.overrideCategory.ms-python.python.pythonWelcome.when":"false",
  "experiments.override.gettingStarted.overrideCategory.ms-python.python.pythonWelcomeWithDS.when": "workspacePlatform != webworker",
  "experiments.override.gettingStarted.overrideCategory.ms-toolsai.jupyter.jupyterWelcome.when": "workspacePlatform != webworker"

Install the Python extension insiders build (Python: Switch to Insiders Daily Channel command) and the Jupyter extension insiders build (switch to pre-release version from extensions tab)

Steps

Verify both tiles open on the welcome page (Help: Get Started command)
Note: You may need to X out welcome pages from other extensions (e.g. Getting Started with Docker) since only 2 extension welcome pages show up on the Walkthroughs list

Click on the "Get started with Python development" walkthrough

Most of the tiles have been previously tested here, so we will only test the new ones. Click on the "Create a Python file" tile

Verify the SVG renders appropriately:

Verify the links and buttons work as expected

Click on the "select a Python interpreter" tile

Verify SVG renders appropriately:

Verify clicking on the "select Python interpreter" button shows the interpreter picker

Click on the last tile ("Explore more resources")

Verify the "data science" link on the second line takes you to the "Python for data science" walkthrough:

Verify clicking on the "Jupyter extension" link or the "Search Jupyter extension" opens up the extension view and searches for "ms-toolsai.jupyter"

Click on the "create or open a Jupyter notebook" tile

Verify the SVG renders appropriately:

Verify the "Create new Jupyter notebook" button creates a new notebook to the side
Verify the "open folder" and "clone a git repository" links work as expected

Click on the "open the interactive window"  tile

Verify the SVG renders appropriately:

Verify the "open interactive window" button opens the interactive window

Click on the "find out more!" tile

Verify the SVG renders appropriately:

Verify the links work as expected

navigating to the jupyter walkthrough using the "data science" link in the Python walkthrough works fine. When I go straight to the Jupyter walkthrough from the Getting Started page, The SVGs fail to load:

@amunger thanks for testing!


Account deletion at Mercado Libre and Viva o Linux

See: #348.

@Nightfirecat please tell me if you'll be reviewing or don't find necessary, I intend to squash all MercadoLibre commits.

Oh, I didn't know that forcing push changes to this branch will result in a closed pull request. My mistake. Sorry for that.
I will open a new pull request with the requested changes now.

It shouldn't, I have done it extensively on #333. If you explain what you did we can compare :P

Certainly, it was something I did wrong. I will perform some tests and find out how I can improve this. Git is still something new for me. There is so much I need to learn.

Don't worry, I'm frequently caught by its traps too ;p


Component 99 Review

Component 99 Review using the template provided
Sicheng Liu 220579723
Description
This is the code review of component 99:  all.coffee
Type of change
Please delete options that are not relevant.

[X] Documentation (update or new)

How Has This Been Tested?
This is just a code review. This component's code only has 3 lines and it is a definition. Nothing really needs to be tested.
Testing Checklist:

[X] Tested in latest Chrome

Checklist:

[x] I have made corresponding changes to the documentation

This needs linelint and prettier running on it as its currently failing the tests.

This needs linelint and prettier running on it as its currently failing the tests.

Sorry I did not get it. Would you please be a liitle bit more specific about what the problem is so that I can adjust the missing or wrong part of my work

This needs linelint and prettier running on it as its currently failing the tests.

Sorry I did not get it. Would you please be a liitle bit more specific about what the problem is so that I can adjust the missing or wrong part of my work

Run fernandrone/linelint@master
/usr/bin/docker run --name ed866e9fc6baabfb1b49e080a34cfb1a7873e5_338bc6 --label ed866e --workdir /github/workspace --rm -e "HOME" -e "GITHUB_JOB" -e "GITHUB_REF" -e "GITHUB_SHA" -e "GITHUB_REPOSITORY" -e "GITHUB_REPOSITORY_OWNER" -e "GITHUB_REPOSITORY_OWNER_ID" -e "GITHUB_RUN_ID" -e "GITHUB_RUN_NUMBER" -e "GITHUB_RETENTION_DAYS" -e "GITHUB_RUN_ATTEMPT" -e "GITHUB_REPOSITORY_ID" -e "GITHUB_ACTOR_ID" -e "GITHUB_ACTOR" -e "GITHUB_TRIGGERING_ACTOR" -e "GITHUB_WORKFLOW" -e "GITHUB_HEAD_REF" -e "GITHUB_BASE_REF" -e "GITHUB_EVENT_NAME" -e "GITHUB_SERVER_URL" -e "GITHUB_API_URL" -e "GITHUB_GRAPHQL_URL" -e "GITHUB_REF_NAME" -e "GITHUB_REF_PROTECTED" -e "GITHUB_REF_TYPE" -e "GITHUB_WORKFLOW_REF" -e "GITHUB_WORKFLOW_SHA" -e "GITHUB_WORKSPACE" -e "GITHUB_ACTION" -e "GITHUB_EVENT_PATH" -e "GITHUB_ACTION_REPOSITORY" -e "GITHUB_ACTION_REF" -e "GITHUB_PATH" -e "GITHUB_ENV" -e "GITHUB_STEP_SUMMARY" -e "GITHUB_STATE" -e "GITHUB_OUTPUT" -e "RUNNER_OS" -e "RUNNER_ARCH" -e "RUNNER_NAME" -e "RUNNER_TOOL_CACHE" -e "RUNNER_TEMP" -e "RUNNER_WORKSPACE" -e "ACTIONS_RUNTIME_URL" -e "ACTIONS_RUNTIME_TOKEN" -e "ACTIONS_CACHE_URL" -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/documentation/documentation":"/github/workspace" ed866e:9fc6baabfb1b49e080a34cfb1a7873e5
No configuration file found at ./.linelint.yml (will use default configuration)
[EOF Rule] File "docs/OnTrack/UI Enhancement/Component99 review.md" has lint errors
Total of 1 lint errors!

This needs linelint and prettier running on it as its currently failing the tests.

Sorry I did not get it. Would you please be a liitle bit more specific about what the problem is so that I can adjust the missing or wrong part of my work

Run fernandrone/linelint@master /usr/bin/docker run --name ed866e9fc6baabfb1b49e080a34cfb1a7873e5_338bc6 --label ed866e --workdir /github/workspace --rm -e "HOME" -e "GITHUB_JOB" -e "GITHUB_REF" -e "GITHUB_SHA" -e "GITHUB_REPOSITORY" -e "GITHUB_REPOSITORY_OWNER" -e "GITHUB_REPOSITORY_OWNER_ID" -e "GITHUB_RUN_ID" -e "GITHUB_RUN_NUMBER" -e "GITHUB_RETENTION_DAYS" -e "GITHUB_RUN_ATTEMPT" -e "GITHUB_REPOSITORY_ID" -e "GITHUB_ACTOR_ID" -e "GITHUB_ACTOR" -e "GITHUB_TRIGGERING_ACTOR" -e "GITHUB_WORKFLOW" -e "GITHUB_HEAD_REF" -e "GITHUB_BASE_REF" -e "GITHUB_EVENT_NAME" -e "GITHUB_SERVER_URL" -e "GITHUB_API_URL" -e "GITHUB_GRAPHQL_URL" -e "GITHUB_REF_NAME" -e "GITHUB_REF_PROTECTED" -e "GITHUB_REF_TYPE" -e "GITHUB_WORKFLOW_REF" -e "GITHUB_WORKFLOW_SHA" -e "GITHUB_WORKSPACE" -e "GITHUB_ACTION" -e "GITHUB_EVENT_PATH" -e "GITHUB_ACTION_REPOSITORY" -e "GITHUB_ACTION_REF" -e "GITHUB_PATH" -e "GITHUB_ENV" -e "GITHUB_STEP_SUMMARY" -e "GITHUB_STATE" -e "GITHUB_OUTPUT" -e "RUNNER_OS" -e "RUNNER_ARCH" -e "RUNNER_NAME" -e "RUNNER_TOOL_CACHE" -e "RUNNER_TEMP" -e "RUNNER_WORKSPACE" -e "ACTIONS_RUNTIME_URL" -e "ACTIONS_RUNTIME_TOKEN" -e "ACTIONS_CACHE_URL" -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/documentation/documentation":"/github/workspace" ed866e:9fc6baabfb1b49e080a34cfb1a7873e5 No configuration file found at ./.linelint.yml (will use default configuration) [EOF Rule] File "docs/OnTrack/UI Enhancement/Component99 review.md" has lint errors
Total of 1 lint errors!

Hi, I have edited the file, Would you please have a check again? Thank you so much

Please have a check again, thank you!

Run npm run format:check

format:check
prettier --check "**/*.md"

Checking formatting...
[warn] docs/OnTrack/UI Enhancement/Component99 review.md
[warn] Code style issues found in the above file. Forgot to run Prettier?
Error: Process completed with exit code 1.

Failed tests


remove versioned dependency on rake 10

Kicked off with @gary-rafferty
No need for the hard dependency on v10 of Rake

👍


Failing to assemble .COMs on macOS (13.6)

The order of assembly arguments in the Makefile doesn't work on macOS.  bin/cpmemu $(OBJDIR)/asm.com -pA=$(dir $<) -pB=$(dir $@) ... results in the creation of files named -pA=dir and .pB=dir.  It works if you move the -p options in front of the name of the file to execute: bin/cpmemu -pA=$(dir $<) -pB=$(dir $@) $(OBJDIR)/asm.com ...

Interesting. That's a getopt incompatibility between GNU getopt and BSD getopt --- I'll fix it. Thanks.

This should now be fixed as a side effect of other changes.


Refactor to align with updated style guide

https://github.com/softlayer/ember-style-guide#defining-functions-to-be-called-when-events-are-triggered

willInsertElement() - https://github.com/softlayer/sl-ember-components/blob/v0.11.0/addon/components/sl-progress-bar.js#L106

Taking this


package.spdx.yml are discarded in MoveTK project

Previously for the MoveTK project (https://github.com/heremaps/movetk) the report would show SpdxDocumentFile entries and now the Analyzer logs shows the below:
10:11:59 08:11:59.287 [main] INFO  org.ossreviewtoolkit.analyzer.managers.SpdxDocumentFile - Discarded the following non-project SPDX files: '/workspace/project/src/include/third_party/miniball/package.spdx.yml', '/workspace/project/src/include/third_party/boost_geometry/package.spdx.yml', '/workspace/project/third_party/Catch2/package.spdx.yml', '/workspace/project/third_party/GsTL/package.spdx.yml'

This regression in functionality was introduced in https://github.com/oss-review-toolkit/ort/pull/3846 but per the https://github.com/spdx/spdx-spec/issues/439 this should work.

MoveTK is just the public example - per spec for this it's not required to have project.spdx.yml for a project. Requiring to have one would break some of our and other ORT use cases.

From discussion with @tsteenbe on Friday:
Idea:

before starting resolveDefintionFiles() in the analyzer -> create a new function in PackageManager that runs before this and can implement definition file specific rules
for SpdxDocumentFile this would be -> if project.spdx.yml exist, delete all package.spd.yml, otherwise keep them
then in the SpdxDocumentFile resolveDependencies: delete discarding of the package.spdx.yml and return package.spdx,yml information if it is a definition file.

@tsteenbe do you by any chance still have an analyzer result file from an old MoveTK project that shows how you would expect the results to look?

@neubs-bsi  Sure, here you go movetk-november-30-2020-analyzer-result.json.zip


added contact creation and contact list visualization

Updated functionality of the bot with contact creation and contact list visualization

Ok


Removed unused imports

Closing as there are no unused imports in the code base as of commit f2480a984fec5aaf354b4d179f51bd902ad3e0cc.


[Bug]: package_info_plus cause an error related to "kotlin-stdlib-jdk" when building the APK

Platform
Android
Plugin
package_info_plus
Version
3.0.2
Flutter SDK
3.3.8
Steps to reproduce

Create a new flutter project
In app/build.gradle change jdk7 to jdk8

dependencies {
    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8:$kotlin_version"
}

Add package_info_plus to pubspec.yaml file
Execute flutter pub upgrade
Execute flutter clean (make sure to run this before any flutter build)
Execute flutter build apk --split-per-abi

flutter pub upgrade output

The output of flutter pub upgrade

flutter pub upgrade
Resolving dependencies...
async 2.9.0 (2.10.0 available)
boolean_selector 2.1.0 (2.1.1 available)
characters 1.2.1
clock 1.1.1
collection 1.16.0 (1.17.0 available)
cupertino_icons 1.0.5
fake_async 1.3.1
ffi 2.0.1
flutter 0.0.0 from sdk flutter
flutter_lints 2.0.1
flutter_test 0.0.0 from sdk flutter
flutter_web_plugins 0.0.0 from sdk flutter
http 0.13.5
http_parser 4.0.2
js 0.6.4 (0.6.5 available)
lints 2.0.1
matcher 0.12.12 (0.12.13 available)
material_color_utilities 0.1.5 (0.2.0 available)
meta 1.8.0
package_info_plus 3.0.2
package_info_plus_platform_interface 2.0.1
path 1.8.2 (1.8.3 available)
plugin_platform_interface 2.1.3
sky_engine 0.0.99 from sdk flutter
source_span 1.9.0 (1.9.1 available)
stack_trace 1.10.0 (1.11.0 available)
stream_channel 2.1.0 (2.1.1 available)
string_scanner 1.1.1 (1.2.0 available)
term_glyph 1.2.1
test_api 0.4.12 (0.4.17 available)
typed_data 1.3.1
vector_math 2.1.2 (2.1.4 available)
win32 3.1.2
No dependencies changed.
13 packages have newer versions incompatible with dependency constraints.
Try flutter pub outdated for more information.

build.gradle

build.gradle
buildscript {
    ext.kotlin_version = '1.6.10'
    repositories {
        google()
        mavenCentral()
    }

    dependencies {
        classpath 'com.android.tools.build:gradle:7.1.2'
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
    }
}

allprojects {
    repositories {
        google()
        mavenCentral()
    }
}

rootProject.buildDir = '../build'
subprojects {
    project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
    project.evaluationDependsOn(':app')
}

task clean(type: Delete) {
    delete rootProject.buildDir
}

app/build.gradle
def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}

def flutterRoot = localProperties.getProperty('flutter.sdk')
if (flutterRoot == null) {
    throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
}

def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
    flutterVersionCode = '1'
}

def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
    flutterVersionName = '1.0'
}

apply plugin: 'com.android.application'
apply plugin: 'kotlin-android'
apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"

android {
    compileSdkVersion 33

    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
    }

    kotlinOptions {
        jvmTarget = '1.8'
    }

    sourceSets {
        main.java.srcDirs += 'src/main/kotlin'
    }

    defaultConfig {
        // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
        applicationId "com.doors.app"
        minSdkVersion 21
        targetSdkVersion 33
        versionCode flutterVersionCode.toInteger()
        versionName flutterVersionName
    }

    buildTypes {
        release {
            // TODO: Add your own signing config for the release build.
            // Signing with the debug keys for now, so `flutter run --release` works.
            signingConfig signingConfigs.debug
        }
    }
}

flutter {
    source '../..'
}

dependencies {
    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8:$kotlin_version"
}

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

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

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const Text('Hi'),
    );
  }
}

Logs
flutter build apk --split-per-abi

💪 Building with sound null safety 💪

w: Runtime JAR files in the classpath should have the same version. These files were found in the classpath:
    /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-jdk8/1.6.10/e80fe6ac3c3573a80305f5ec43f86b829e8ab53d/kotlin-stdlib-jdk8-1.6.10.jar (version 1.6)
    /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-jdk7/1.6.10/e1c380673654a089c4f0c9f83d0ddfdc1efdb498/kotlin-stdlib-jdk7-1.6.10.jar (version 1.6)
    /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib/1.7.20/726594ea9ba2beb2ee113647fefa9a10f9fabe52/kotlin-stdlib-1.7.20.jar (version 1.7)
    /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-common/1.7.20/e15351bdaf9fa06f009be5da7a202e4184f00ae3/kotlin-stdlib-common-1.7.20.jar (version 1.7)
w: Some runtime JAR files in the classpath have an incompatible version. Consider removing them from the classpath
e: /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib/1.7.20/726594ea9ba2beb2ee113647fefa9a10f9fabe52/kotlin-stdlib-1.7.20.jar!/META-INF/kotlin-stdlib.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.
e: /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-common/1.7.20/e15351bdaf9fa06f009be5da7a202e4184f00ae3/kotlin-stdlib-common-1.7.20.jar!/META-INF/kotlin-stdlib-common.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.
w: Runtime JAR files in the classpath should have the same version. These files were found in the classpath:
    /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-jdk8/1.6.10/e80fe6ac3c3573a80305f5ec43f86b829e8ab53d/kotlin-stdlib-jdk8-1.6.10.jar (version 1.6)
    /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-jdk7/1.6.10/e1c380673654a089c4f0c9f83d0ddfdc1efdb498/kotlin-stdlib-jdk7-1.6.10.jar (version 1.6)
    /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib/1.7.20/726594ea9ba2beb2ee113647fefa9a10f9fabe52/kotlin-stdlib-1.7.20.jar (version 1.7)
    /home/nidal/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-common/1.7.20/e15351bdaf9fa06f009be5da7a202e4184f00ae3/kotlin-stdlib-common-1.7.20.jar (version 1.7)
w: Some runtime JAR files in the classpath have an incompatible version. Consider removing them from the classpath
Running Gradle task 'assembleRelease'...                           73.6s
✓  Built build/app/outputs/flutter-apk/app-armeabi-v7a-release.apk (5.5MB).

Flutter Doctor
flutter doctor -v
[✓] Flutter (Channel stable, 3.3.8, on Ubuntu 20.04.5 LTS 5.15.0-52-generic, locale en_US.UTF-8)
    • Flutter version 3.3.8 on channel stable at /home/nidal/fvm/versions/3.3.8
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 52b3dc25f6 (5 weeks ago), 2022-11-09 12:09:26 +0800
    • Engine revision 857bd6b74c
    • Dart version 2.18.4
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc4)
    • Android SDK at /home/nidal/Android/Sdk
    • Platform android-33, build-tools 31.0.0-rc4
    • Java binary at: /snap/android-studio/123/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2021.2)
    • Android Studio at /snap/android-studio/123/android-studio
    • Flutter plugin version 70.0.2
    • Dart plugin version 212.5744
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.2)
    • IntelliJ at /snap/intellij-idea-community/390
    • Flutter plugin version 70.0.5
    • Dart plugin version 222.3739.24

[✓] VS Code
    • VS Code at /snap/code/current
    • Flutter extension version 3.54.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 20.04.5 LTS 5.15.0-52-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 107.0.5304.68

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

• No issues found!

Checklist before submitting a bug

[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

It's the same issue with path_provider

keep your app/build.gradle jdk7 as it is.
dependencies {
    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
}

Try this comment
I am closing this issue for now.


Change components tests extension from js to jsx or tsx

Is your feature request related to a problem? Please describe.
Now the tests are .js extension. Inside of them, we have jsx code. We should change their extension to jsx or tsx.
Describe the solution you'd like
Change the tests extension to jsx or tsx.

WIP


BUG: URL broken

@teonbrooks what URL do you want? Currently we get:
generated/_names.rst:11: [broken] https://teonbrooks.github.io: 404 Client Error: Not Found for url: https://teonbrooks.github.io/

ah, it's now https://teonbrooks.com


Export three icons: WhatsAppBrand, SignalBrand, and TelegramBrand

What are you changing?
This exports the icons that were added in https://github.com/guardian/csnx/commit/93468258016dbdf78352f029fb10dea73af97f9c

@sophie-macmillan looks like there's a small change needed in the test too<EMAIL_ADDRESS>


Windows Signing

This is somewhat off-topic for OSX utility, but do you have any plans to work on a similar tool for Windows?  I've written up a tutorial on how to do it here http://verysimple.com/2016/02/19/code-signing-applications-for-microsoft-windows/
I think the process could be automated similarly to how osx-sign and osx-flat have automated the process for OSX.  (Although a flatten tool might not be necessary since that functionality is already available via windows-builder.)

As there have already been issues like https://github.com/maxogden/electron-packager/issues/32 and https://github.com/maxogden/electron-packager/issues/245 with signing on Windows, I guess I could have a try later. Not sure if anyone's already started on similar tasks. However, here are some of my concerns with the automation before anything as I work for most of times with Mac:

[ ] If there is a container/directory like login.keychain from Mac on Windows to look for certs?
[ ] Whether it's necessary for Windows to have something like this as the only executable is the .exe rather than an application bundle ending in .app which literally is a folder containing multiple binaries, each needing signing? (I'll have to check the win Electrons just to confirm.)
[ ] If no need to open a separate node package, a PR could be suitable for electron-packager? However, again, electron-packager does the job of packing, not packing with preparing installer.
[ ] If we could reduce the amount of work to create installer and then to sign installer, it would be quite nice to go for, just as neat as the productbuild on Mac which signs the installer. Yet mentioned earlier, a flatten tool may not be necessary as that's available already.

I'll do some research later as https://github.com/maxogden/electron-packager/issues/32 mentions something slightly different from your walk-through with usage of external DLL.

I can answer some of those I think:

If there is a container/directory like login.keychain from Mac on Windows to look for certs? There is a store which you can look at using certmgr.msc, and the signing tool has a parameter to use a key from the store instead of a file. However I don't know how to access the store programmatically to search.
Whether it's necessary for Windows to have something like this as the only executable is the .exe rather than an application bundle ending in .app which literally is a folder containing multiple binaries, each needing signing? Windows is similar except that instead of an .app bundle, the application is literally just a folder with the main .exe file and a bunch of dll files, all of the content, etc. From my work so far, only the main executable and the installer needs to be installed. However I don't know if Electron apps can be submitted to the Windows App Store at all, I have been unsuccessful figuring out how to package it into the necessary App Store bundle.
If no need to open a separate node package, a PR could be suitable for electron-packager? However, again, electron-packager does the job of packing, not packing with preparing installer. It does seem that maxogden wants all extra functionality to be build as modules. There is already an equivalent to electron-osx-flat called electron-builder. But what is missing it seems is something like electron-win-sign.
If we could reduce the amount of work to create installer and then to sign installer, it would be quite nice to go for, just as neat as the productbuild on Mac which signs the installer. Yet mentioned earlier, a flatten tool may not be necessary as that's available already. Yes I agree, it would probably be electron-win-sign for the executable, then electron-builder to create the installer, then electron-win-sign again for the installer

I'm just barely getting things working on windows, but it does seem to me it would be nice to have consistent tools for building on all platforms.

@jasonhinkle just thoroughly read it. It's nice of Windows to have some kind of a keychain tool where go the certs. I guess we could have a tool designed for Windows for signing. But anyway, it requires manual testing. We could have command interfaces like electron-win-sign for apps and electron-win-sign-inst for installers.
I'll look further into an implementation with the assistance from my virtual machine later after drafting the wiki tonight. Thanks @jasonhinkle!

osslisigncode should be used. Out of this project scope. Use electron-builder but not such low-level tools ;)


add interceptor support for both request and session && alamofire 5.…

add interceptor and request modifier support

I pull the master branch but nothing is working can you help me what just happened :)

Hi @hamedmohammadi, do rebase your branch with master again as it seems that the branches history seems diverged.

Please rebase the branch and reopen the pull request once the conflicts are resolved - thanks!

ok I will do it soon


Migrate AWX utility for slow method logging to DAB

Followup from https://github.com/ansible/django-ansible-base/pull/356
The intent is that AWX will delete this method and import from DAB.

Is this the equivalent in awx  https://github.com/ansible/awx/blob/adc2162bac1c9832e8df39df8dc0a4c04126cc78/awx/main/utils/common.py#L1208 ?

Yes, https://github.com/ansible/awx/pull/15675 would remove it from there and import from DAB


[AI Agent idea] Quantum Computing and Cryptography

Description
AI agent with expertise in quantum computing and cryptography. It can assist researchers in solving complex quantum algorithms and help organizations secure their data using quantum-resistant encryption techniques.
Additional Information
No response

/assign @manmohanhunjan


Spark refusing to work inside docker container running ubuntu

As shown in the image above the route is mapped but according to the logs of the container it says otherwise:
[Thread-0] INFO spark.embeddedserver.jetty.EmbeddedJettyServer - == Spark has ignited ...
[Thread-0] INFO spark.embeddedserver.jetty.EmbeddedJettyServer - >> Listening on <IP_ADDRESS>:1026
[Thread-0] INFO org.eclipse.jetty.server.Server - jetty-9.4.48.v20220622; built: 2022-06-21T20:42:25.880Z; git: 6b67c5719d1f4371b33655ff2d047d24e171e49a; jvm 17.0.5+8
[Thread-0] INFO org.eclipse.jetty.server.session - DefaultSessionIdManager workerName=node0
[Thread-0] INFO org.eclipse.jetty.server.session - No SessionScavenger set, using defaults
[Thread-0] INFO org.eclipse.jetty.server.session - node0 Scavenging every 600000ms
[Thread-0] INFO org.eclipse.jetty.server.AbstractConnector - Started<EMAIL_ADDRESS>(http/1.1)}{<IP_ADDRESS>:1026}
[Thread-0] INFO org.eclipse.jetty.server.Server - Started @969ms
[qtp11388440-21] INFO spark.http.matching.MatcherFilter - The requested route [/] has not been mapped in Spark for Accept: [text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9]
[qtp11388440-16] INFO spark.http.matching.MatcherFilter - The requested route [/base] has not been mapped in Spark for Accept: [text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9]
[qtp11388440-17] INFO spark.http.matching.MatcherFilter - The requested route [/base] has not been mapped in Spark for Accept: [*/*]

I have rebuilt the container and jar file multiple times yet Spark doesn't work. I'm not sure if its an issue with Spark or Docker.
If anyone needs anymore information just ask and I'll respond here with it.

ignore my bad

cloise


Support node es modules resolution

Bug Report
🕗 Version & Regression Information
Versions:

4.3.2
4.1.2

This is a crash
This is the behavior in every version I tried

⏯ Playground Link
Minimal repoducible example
💻 Code
Run tsc (yarn tsc, yarn exec tsc, npx tsc, ./node_modules/.bin/tsc) with nodejs flag --experimental-specifier-resolution=node (NODE_OPTIONS="--experimental-specifier-resolution=node" tsc)
🙁 Actual behavior
internal/process/esm_loader.js:74
    internalBinding('errors').triggerUncaughtException(
                              ^

TypeError [ERR_INVALID_MODULE_SPECIFIER]: Invalid module "file:///path/to/project/node_modules/typescript/bin/tsc"
    at Loader.getFormat (internal/modules/esm/loader.js:117:13)
    at async Loader.getModuleJob (internal/modules/esm/loader.js:242:20)
    at async Loader.import (internal/modules/esm/loader.js:176:17)
    at async Object.loadESM (internal/process/esm_loader.js:68:5) {
  code: 'ERR_INVALID_MODULE_SPECIFIER'
}

🙂 Expected behavior
Build without errors

also, it's not so important, add building to native modules and add path to alternative dist in package.json. Thank you

I assume this is related to https://gist.github.com/weswigham/22a064ffa961d5921077132ae2f8da78?

Not, it's not.
This problem because of bin script definition.
Bin scripts defined as

    {
        "tsc": "./bin/tsc.js",
        "tsserver": "./bin/tsserver"
    }

And when I call yarn tsc, nodejs trying to run node_modules/typescript/bin/tsc. This file hasn't extension and nodejs throws an error.
Why it works in Jest?
Because it use another bin declaration like this:
{
"jest": "./bin/jest.js"
}

For package manager, this script has link with name jest, but nodejs run jest.js, file with extension.
So, solving my problem is adding .js in bin files and in paths in package.json bin. It won't change behavior for exists project by help projects with configurations like mine.
As you can see, another experimental flags for native modules aren't change behavior (Now I run tsc without --experimental-specifier-resolution, but with all another flags in scripts and it works)

I created pull request with resolution. Can somebody approve it?


resolve service alias from region and environment

rancher/rancher#9839

LGTM. Will merge along with other components of the feature.


The String problem

Strings are a problem. They are expensive, not portable, and just generally annoying.
I need a way to hash string's to GUID's and use them for fast comparisons, similar to FName in UE. This will be very useful for the future of working on this engine

We can probably just use std::string for this.... it's a lot of work

We can just use the Guid and entt::hashed_string to solve this problem.


JVM restarts don't restart pod

https://kubernetes.slack.com/archives/C8149MREV/p1529469356000184

I suspect that this is due to the fact that the initial statefulset doesn't have any resource requests or limits. Thus the pod isn't even close to running out of memory when the JVM does.

Looking at a kubectl describe po on the first ES node that I have found with a restart in kubectl get po, I found:
Events:
  Type     Reason                 Age              From                                                Message
  ----     ------                 ----             ----                                                -------
  Normal   SuccessfulMountVolume  4m               kubelet, ip-10-4-32-254.us-west-2.compute.internal  MountVolume.SetUp succeeded for volume "sgconfig
  Normal   SuccessfulMountVolume  4m               kubelet, ip-10-4-32-254.us-west-2.compute.internal  MountVolume.SetUp succeeded for volume "default-token-hbl7m"
  Normal   SuccessfulMountVolume  4m               kubelet, ip-10-4-32-254.us-west-2.compute.internal  MountVolume.SetUp succeeded for volume "certs"
  Normal   Scheduled              4m               default-scheduler                                   Successfully assigned apps-es-0 to ip-10-4-32-254.us-west-2.compute.internal
  Normal   SuccessfulMountVolume  4m               kubelet, ip-10-4-32-254.us-west-2.compute.internal  MountVolume.SetUp succeeded for volume "pvc-8d636427-7032-11e8-800b-0ac45b6949fc"
  Normal   Pulled                 3m (x2 over 4m)  kubelet, ip-10-4-32-254.us-west-2.compute.internal  Container image "kubedb/elasticsearch:6.2" already present on machine
  Normal   Created                3m (x2 over 4m)  kubelet, ip-10-4-32-254.us-west-2.compute.internal  Created container
  Normal   Started                3m (x2 over 4m)  kubelet, ip-10-4-32-254.us-west-2.compute.internal  Started container
  Normal   Killing                1m (x2 over 3m)  kubelet, ip-10-4-32-254.us-west-2.compute.internal  Killing container with id docker://elasticsearch:Need to kill Pod
  Normal   SandboxChanged         1m (x2 over 3m)  kubelet, ip-10-4-32-254.us-west-2.compute.internal  Pod sandbox changed, it will be killed and re-created.
  Normal   Started                1m (x3 over 4m)  kubelet, ip-10-4-32-254.us-west-2.compute.internal  Started container
  Normal   Created                1m (x3 over 4m)  kubelet, ip-10-4-32-254.us-west-2.compute.internal  Created container
  Normal   Pulled                 1m (x3 over 4m)  kubelet, ip-10-4-32-254.us-west-2.compute.internal  Container image "busybox" already present on machine
  Warning  BackOff                1m (x2 over 1m)  kubelet, ip-10-4-32-254.us-west-2.compute.internal  Back-off restarting failed container

It seems like it is possible that the killing is the reason for the restart counter not to increment.


OfflineRecognizerResult missing tokens, timestamps, lang, emotion, event

Hello! Thank you for sharing this go project.
I have noticed that the bindings may not be current. For example, the OfflineRecognizerResult is missing several fields  (Tokens []string, Timestamps []float32, Lang string, Emotion string, Event string)
Would it be possible to add these to the go bindings?

would you like to contribute?

yes, unfortunately i was not able to figure out how to convert the tokens or timestamps
tokens *_Ctype_char -> Tokens []string
timestamps *_Ctype_float-> Timestamps []float32

If you could give me some info on converting those two datatypes, i'd be happy to write the code

unfortunately i was not able to figure out how to convert the tokens or timestamps

Please have a look at
https://github.com/k2-fsa/sherpa-onnx/blob/e34a1a2aa39f1cf8384e1bf327e7edd849dbe739/scripts/go/sherpa_onnx.go#L707-L716
A second example is given below (untested, please test it by yourself)

n := int(p.count);
result.Tokens = make([]int, n);
tokens := (*[1<<28]C.int)(unsafe.Pointer(p.tokens))[:n:n];
for i := 0; i < n; i++ {
  result.Tokens[i] = c.int(tokens[i]);
}

result.Timestamps := make([]float32, n);
// please fill in result.Timestamps by yourself.
// I think you have already got the idea.

Here is the code for processing tokens. Please see also
https://github.com/k2-fsa/sherpa-onnx/blob/e34a1a2aa39f1cf8384e1bf327e7edd849dbe739/scripts/go/sherpa_onnx.go#L1114-L1121
n := int(p.count);

result.Tokens = make([]string, n);
tokens := (*[1<<28]*C.char)(unsafe.Pointer(p.tokens_arr))[:n:n];

for i := 0; i < n; i++ {
  result.Tokens[i] = G.GoString(tokens[i]);
}

Could you close them and change the following file instead?
https://github.com/k2-fsa/sherpa-onnx/blob/master/scripts%2Fgo%2Fsherpa_onnx.go

all set
See new PR here


Uncaught Error: Notification must be created with string message: undefined

After installing markdown-themeable-pdf, on converting to pdf using one of the following ways:

Right-click in the editor area when a file is opened and select Markdown to PDF or
Right-click on a markdown file in Tree-View and select Markdown to PDF or
Press ctrl-shift-E (E = Export) in the editor area when a file is opened

Atom: 1.28.1 x64
Electron: 2.0.4
OS: Ubuntu 18.04
Thrown From: markdown-themeable-pdf package 1.2.1
Stack Trace
Uncaught Error: Notification must be created with string message: undefined
At /snap/atom/167/usr/share/atom/resources/app/src/notification.js:21

Error: Notification must be created with string message: undefined
    at Notification.validate (/snap/atom/167/usr/share/atom/resources/app/src/notification.js:21:19)
    at new Notification (/snap/atom/167/usr/share/atom/resources/app/src/notification.js:16:16)
    at NotificationManager.addError (/snap/atom/167/usr/share/atom/resources/app/src/notification-manager.js:154:39)
    at /packages/markdown-themeable-pdf/lib/markdown-themeable-pdf.js:495:44
    at /packages/markdown-themeable-pdf/node_modules/html-pdf/lib/pdf.js:58:21
    at ChildProcess.respond (/packages/markdown-themeable-pdf/node_modules/html-pdf/lib/pdf.js:126:14)
    at emitTwo (events.js:126:13)
    at ChildProcess.emit (events.js:214:7)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)

Commands
  9x -1:11.9.0 core:backspace (input.hidden-input)
     -0:09.9.0 markdown-themeable-pdf:export (div.line)

Non-Core Packages
busy-signal 1.4.3
gpp-compiler 3.0.7
intentions 1.1.5
linter 2.2.0
linter-eslint 8.4.1
linter-ui-default 1.7.1
markdown-themeable-pdf 1.2.1

@cakebake Is there a way to fix this issue?
Actually, I didn't get the issue as well. Can you please let me know if this issue is related to the atom or my PC?

Can you please let me know if this issue is related to the atom or my PC?

Hello! Thank´s for your feedback, but I also could not reproduce this issue. Currently I am working on version 2. For version 1, there are no more updates. Closing.


loadEnv fails in Ubuntu

Describe the bug
An empty string prefix in Ubuntu causes loadEnv to fail, but works when not passing a prefix at all (since it uses the default).
loadEnv(mode, process.cwd(), "")

Vitest tests pass, but this error occurs when starting the server.
ERROR: Error: Parse failure: Unexpected token (51:30)
At file: /home/duval/Desktop/pages/playground/locations-site/src/templates/static.tsx?update=1699573359919
Contents of line 51:       revLookup[""/home/duval/.local/share/pnpm/pnpm"".charCodeAt(0)] = 63;

Reproduction
N/A
Steps to reproduce
No response
System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.1 LTS 22.04.1 LTS (Jammy Jellyfish)
    CPU: (4) arm64 unknown
    Memory: 2.10 GB / 3.82 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 17.9.1 - ~/.nvm/versions/node/v17.9.1/bin/node
    npm: 8.11.0 - ~/.nvm/versions/node/v17.9.1/bin/npm
    pnpm: 7.9.5 - ~/.local/share/pnpm/pnpm
  Browsers:
    Chromium: 105.0.5195.52
  npmPackages:
    rollup: ^3.28.0 => 3.29.4
    vite: ^4.4.9 => 4.5.0

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

For some reason the environment variable of _ was causing the issue. Filtering it out fixed it.


ssh: add trust checks before sourcing env vars

Proposed Changes

Add ownership test before attempting to source environment variables in SSH module.  If the file is not owned by the current user, its contents should not be trusted.  The default path is predictable and thus subject to abuse.

Since commit ff91c8d4 unnecessary.
From this commit on, the env-file is stored within the XDG_CACHE_HOME directory (alternative in ${HOME}/.cache) and as such isn't be stored in a world writeable location (at least I hope nobody has that location writable for anybody except oneself).
The same goes for the gpg-agent.env which, depending of the configuration, might be used instead of the ssh one.


On the list of housing forms, emphasized the Download option

This emphasizes the Download option when a case worker is viewing a list of housing applications they can download.

Changes the text Download link to a button
Moves the button to be first action in the list of actions.

:rainbow:


[Linear] Change sort order of issues under "assigned issues"

Extension – Linear
Author: @thomaslombart

Describe the feature and the current behavior/state.
Seems like issues are sorted by desc created atm. The actual linear app allows you to sort on whatever attribute you want. I have it sorted by priority, timestamp ordering doesn't do a whole lot for me. It would be nice to be able to select which sorting method you want from the extension settings at a minimum.
Who will benefit with this feature?
Anyone who wants to work on stories based on priority order.
Any Other info.

Thanks for the feature request, it sounds like a useful addition! This seems like a good use-case for dropdowns in the issues views.


Wave3: Adding docs for remaining pack errors and warnings

Working towards: https://github.com/NuGet/Home/issues/6498
This PR adds docs for the remaining pack errors and warnings.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/TOC.md
:white_check_mark:Succeeded
View

docs/reference/Errors-and-Warnings.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5001.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5003.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5004.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5005.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5007.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5008.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5012.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5020.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5021.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5024.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5101.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5106.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5107.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5108.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5109.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5110.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5111.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5112.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5114.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5116.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5117.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5118.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5119.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5120.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5121.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5122.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5123.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/TOC.md
:white_check_mark:Succeeded
View

docs/reference/Errors-and-Warnings.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5001.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5003.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5004.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5005.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5007.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5008.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5012.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5020.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5021.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5024.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5101.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5106.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5107.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5108.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5109.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5110.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5111.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5112.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5114.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5116.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5117.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5118.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5119.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5120.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5121.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5122.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5123.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

cc: @rohit21agrawal


ManageEngine ServiceDesk Plus Cloud app submission

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

@SDPCloud please sign our CLA. you can do so by adding a comment with the below command.
@microsoft-github-policy-service agree company="your company"

@SDPCloud please sign our CLA. you can do so by adding a comment with the below command.
@microsoft-github-policy-service agree company="your company"


How do you add a directory?

ubuntu@base ~/d/runs (main) [1]> dud stage add 2023-07-24
Error: 2023-07-24: yaml: input error: read 2023-07-24: is a directory

If this isn't supported, it seems like a pretty essential feature...

Mh, it works different:

Add your directory to stage file

dud stage gen -o my/dir/ | tee my_dir.yaml

Add stage

dud stage add my_dir.yaml

Do the rest:
dud commit
dud push
git add .
git commit -m "Added dir"
git push

Thanks for the question, @ezhang7423! Sorry for the late response; I was away on vacation for a while.
@thorstenwagner has shown how this is done in Dud; thanks a bunch, @thorstenwagner!
TLDR: stage gen generates/creates a stage, and stage add adds a stage file to Dud's index.


Update LAPACK to 3.9.0

This is just to note that reference LAPACK 3.9.0 was released two days ago.
As usual, this is not simply a drop-in replacement here as OpenBLAS uses modified Makefiles (and own CMakelists) to replace some functions with optimized implementations. There are also some platform-specific changes and header rearrangements that need to be merged during an update.
And I see svillemot has already found some small bugs in the release.

Do you intend to block waiting for bugs to be addressed or let it go as is?

I primarily wanted to acknowledge the release and preempt any well-meaning million lines "delete 3.8.0, insert 3.9.0" PRs that do not address the OpenBLAS-specific adaptions. :)
If I recall correctly, any major bugfixes since 3.8.0 should already have been tracked here so the primary motivation for upgrading would probably be the added functions for QR SVD and Householder reconstruction.


Corrections to align files to TOSCA 1.0/1.1/1.2/1.3 specifications

Added missing

imports
tosca.capabilities.network.Linkable
tosca.relationships.network.LinksTo
tosca.relationships.network.BindsTo
tosca.nodes.network.Network
tosca.nodes.network.Port
required: false
For TOSCA 1.2 and 1.3
renamed tosca.nodes.BlockStorage to tosca.nodes.Storage.BlockStorage
replaced capability: tosca.capabilities.Container by capability: tosca.capabilities.Compute

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


How do I externalise the parameters for the KafkaTrigger?

I need to move (e.g.) UserName and Password to configuration, but cant find the correct syntax.  I have been looking at IBinder and it makes complete sense to me how this works for output bindings - but I cant see how to wire it up to provide the trigger parameters?
I do see that there are Kafka Attributes in the repo, but could do with an example of how to do that (cant find much on this one the web generally)...

I found this:
`
[FunctionName(nameof(SampleConsumer))]
    public void SampleConsumer(

[KafkaTrigger(

        "LocalBroker",

        "%EHTOPIC%",

        ConsumerGroup = "$Default",

        Username = "$ConnectionString",

        Password = "%EventHubConnectionString%",

        Protocol = BrokerProtocol.SaslSsl,

        AuthenticationMode = BrokerAuthenticationMode.Plain)] KafkaEventData<string> kafkaEvent,
ILogger logger)

    {

        logger.LogInformation(kafkaEvent.Value.ToString());

}`

How are you setting these values:
%EventHubConnectionString%

Is that resolving environment variables or what?
And whats this?
"$Default",

Is this from a settings file or how are you setting these values?


add gcc variant

based on the discussion on https://github.com/joyent/docker-node/pull/2#issuecomment-71506264 adding a -gcc variant that could share between language stacks with a C API.
The goal of the variant is to include just enough bytes to cover building native package:

Python C extensions
Node native modules
cgo packages

You still haven't clearly defined where we draw the line for this variant, though.

For example, I would personally be very surprised to find that this doesn't include g++, and that personal expectation is going to be different from user to user.

@tianon will update the description.

@tianon I think adding g++ make senses, since some module might be written in C++ with extern "C" symbol to interop with the runtime.

What about the rest of build-essential that come as recommends (which is just manpages manpages-dev if we ignore the items for making debian packages)?  Where do we draw the line?
This issue is related to #16.

Yes, build-essentials without the debian stuff sounds a fine line to draw for me.
Maybe dropping the --no-install-recommends would make more sense here.

That's great but it's all feelings and impressions.  How do we
unambiguously state that "this is what's OK to be included" without it
being based entirely on our personal feelings for what's "too much" ?

@tianon what do you think for the updated description?

The goal of the variant is to include just enough bytes to cover building native package

The goal should be that if you include a python, node or go package in requirements.txt, packages.json or import that has some C code in it, it should not fail because you miss some elements of the native toolchain.

I think "building native package" is still subjective.  I don't think it's at all far-fetched to imagine Python or some other language growing support for native modules written directly in Go.  Similarly, what does this mean for LLVM and friends?

@tianon maybe a better way to phrase that would be npm install or pip install or go get should not fail because you miss a tool. Many languages have a document C/API, with clear hook in their build system. The goal here is to define the common set of tool they expect to be there to produce native extension.

Closing stale PR.  Most users want something tiny, like an alpine-based image, or want it just to work, like the default language images; shared layers isn't really a concern for most users.


add Bootlint v0.10.0

Thanks as always!
https://github.com/twbs/bootlint/releases/tag/v0.10.0

On it thanks @cvrebert

@cvrebert bootlint.min.js appears to be missing.  Should I just add the bootlint.js instead?

Upstream doesn't include a minified version since Bootlint is only for debugging and isn't perf-critical. Looks like y'all have been minifying it yourselves so far.
We need bootlint/latest/bootlint.min.js to exist due to the bookmarklet, but it's fine if that's not actually minified and it's fine if there's not any bootlint/0.10.0/bootlint.min.js file.

 I will get this up now.

@jdorfman there's a script to create min @ ./scripts/update_bootlint.sh.
Usage:
./scripts/update_bootlint.sh 0.10.0

sheet didn't see that.  I will use that next time.  Thanks @jmervine


speed up crossing load/processing in testing

load only watershed groups of interest
minimize use of UPDATES (replacing updates with lookup tables and populating the values of interest in crossings_vw)

mostly fixed with ec2db00b5d857280670ba7000dba008d45da509c


Support MS2M2 or MS2M3 .mzML format loading

Currently only support .raw.

I tested this with a .raw converted to .mzML and it works as expected. I'm going to close this for now, but if we notice any specific vendor formats not converting to .mzML properly, we can revisit this


index.html structure

I need to structure the index.html to have semantic language (99%) and follow a top to bottom approach.

I have done roughly 65% of the structure build at this point. Header/nav/first article/second article are basic set.

I altered some of the structural set up surrounding the "my work" images. The structure is almost finished.

I added a footer and finished the structural setup for the index.html


Checkout

implemented checking out (no order confirmation), editing cart, deleting from cart, conditional rendering

looks good; resolved all merge conflicts


Dynamic "inputs"

In the launch.json these input options are very useful. For some applications I would prefer to provide the input dynamically. Like I could run a script which returns the values and those would be the ones I can choose from.
I could imagine something like:
  "inputs": [
    {
      "id": "pickSomething",
      "description": "...",
      "type": "pickDynamicString",
      "cmd": "executable",
      "args": [],
      "envs": {},
      "default": ".*"
    }

Of course the executable should generate something which is understandable by the vscode like json or just lines..
Thanks for the consideration.

The command-type inputs can be used in this way, with an extension providing the picker and returning the value
https://code.visualstudio.com/docs/editor/variables-reference#_input-variables

Hello,
I wouldn't mind a little more explanation or maybe an example.. And which extension are you referring to?

Found this: https://stackoverflow.com/questions/58747689/using-a-shell-command-as-vscode-task-variable-value
This works.


If "As Instance" is checked the instances are not exported

if this checkbox in geometry info is ticked the exporter does not export the instances
there's no bin file and here's the gltf
{
    "asset" : {
        "generator" : "Khronos glTF Blender I/O v3.3.32",
        "version" : "2.0"
    },
    "scene" : 0,
    "scenes" : [
        {
            "name" : "Scene",
            "nodes" : [
                0
            ]
        }
    ],
    "nodes" : [
        {
            "name" : "Plane"
        }
    ]
}

is this correct behavior ? or a bug ?

Did you check "Apply Modifier" checkbox in the exporter settings?
https://github.com/takahirox/glTF-Blender-IO-EXT-mesh-gpu-instancing#how-to-use

yes , if i keep this checkbox off it exports correctly

Thanks for the response. And I confirmed the problem.
Probably these lines are suspicious...
https://github.com/takahirox/glTF-Blender-IO-EXT-mesh-gpu-instancing/blob/5ae881c067310abff2f3dd70f21f1c579d6eeb4c/__init__.py#L51-L52
Let me take a look.

@optimus007
Honestly I'm not so familiar with Blender and Geometry nodes. What is the purpose of checking "As instances" in your example?

Made a PR #5 that can resolve your case. But honestly I'm not sure it is an appropriate way so we may need to investigate more before merging tho.

I'm also not exactly sure about it's use case
the blender docs for it is not helpful
things i noticed
When its "off"

if i do Ctrl+A > Make Instances Real
It converts all the toruses into separate meshes and these meshes also have the same geometry node modifier which might result in a crash
(eg if i have 100 boxes instances and i do Make Instances Real the 100 boxes will become separate meshes and each box will again make its own 100 boxes , so too much boxes  )

When its "on"

if i do Ctrl+A > Make Instances Real
All the instances become separate meshes(geometry is shared) and they do not have the geometry nodes modifier on them and i can hide the original geometry nodes mesh and export these duplicate meshes as a generic gltf


Plugin specific level?

Cool library, but I was wondering if there was a way of restricting the log level just for the remote part? In my own logger I have code like this
    consoleLogger(...args);

    // log errors on server
    if (level === 'error') {
      dispatchLoggingAction(args);
    }

Meaning I don't care about logging anything but errors to the server. Is there a way of achieving this in your logger (read the code, but didn't see any trace of it)?

In the current implementation, this is not provided. But I like this idea. I'll deal with this after the end of Js13kGames =)

Done

Cool library, but I  facing this issue loglevel-plugin-remote.js:372 POST https://localhost/logger 404 (Not Found)


infinite cycle of console warnings

something went wrong...

I can't reproduce this error.
https://archilogic-com.github.io/3dio-inspector-plugins works fine for me :/
Can you pls provide step-by-step instructions?


[FEATURE] 96kHz support

Please include 96kHz support for the VST plugin.
I thought the training process was creating bad captures, turns out my DAW project was set to 96kHz which makes all captures sound broken, but when the DAW is set to 48kHz captures sound correct.

Good catch. I'm tracking this with #59, so I'll close this one as a duplicate


Update jetty-client, jetty-server, ... to 9.4.20.v20190813

Supersedes #2791.

Merging because it's the defunct 2.13.0-M5 build that failed.


Insufficient state when the state is sufficient

Reported by @nicbus:
I have tried the updated code in our regtest sandbox, which uses rgb-contracts (with patched dependencies and a quick fix to have a working issuance).
What I do to check for this issue is:
issue an asset (2000)
send some assets (100) from the issuer to a 1st recipient
send some more assets (200) from the issuer to the same recipient
send some assets (250) from the 1st recipient to a 2nd recipient

This way I check that the issuer is able to spend the change allocation, which is created via WitnessUtxo, and then that the resulting allocation is itself spendable.
At the moment, step 4 fails when I call rgb transfer with error:
Error: InsufficientState
Original report: https://github.com/RGB-WG/rgb-wallet/issues/55#issuecomment-1591429377
May be related to: https://github.com/RGB-WG/rgb-core/issues/159

@nicbus did you pay the initial amount (100) and the second one (200) to the same UTXO or different ones? If different, you had to prepare a PSBT which spends both UTXOs, otherwise indeed you won't have enough tokens to pay the invoice (that is what InsufficientState means)

It should be improved. But here you get not an error message with Display but with Debug. Seems like you just unwrap in rgb-lib code. So to have a proper message instead of unwrapping/expecting you need to handle error prints and reporting with some code using Display

here I'm not using rgb-lib but rgb-contracts. specifically, I receive the InsufficientState error after calling rgb transfer

So than error has to be n non-unrapped in rgb-contracts, not here. This would display the full verbose message with explanation.
I can open the issue there - and I assume this one can be closed?

Closed in d90868a769987aa278214d06d13da98e698ea702


Reporting deployment in a monorepo

We using Azure DevOps as a monorepo and I believe you are well (as described here: https://docs.microsoft.com/en-us/azure/devops/learn/devops-at-microsoft/use-git-microsoft)
One issue we have with this is that a a commit is created for User Story 1 related to Application A. This is deployed to Development and so the story now shows that this is deployed to the Application A environment (using Application A pipeline).
A different story is finished with a commit for Application B and this is deployed separately. However as this is a monorepo it is aware of the commit of Application A (even though it isn't interested) and reports back to User Story 1 that it has deployed to Application B environment.
Similarly we have the same issue when comparing releases (commits and work items). Comparing current release to previous release of Application B will show Application A stories even though it's not interested.
I'm just wondering how we can get round this (if it's even possible). We can trigger pipelines and releases based on the path that the code has changed, but will need something similar to explicitly associate a release to a story based on the path of commit.
Have you done anything similar to this?

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

ID: 462be419-4cfd-4bfc-06cc-45800c0d888b
Version Independent ID: 95cb8a7d-c61d-dfac-403f-12fe40ba1c49
Content: See release deployment information from the work item form - Azure Boards
Content Source: docs/boards/work-items/work-item-deployments-control.md
Product: devops
Technology: devops-agile
GitHub Login: @danhellem
Microsoft Alias: kaelli

Any updates on this when using a monorepo?

Sorry, no current plans to invest in the deployments control in 2020.


Allow save/load of inactive characters

My Gloomhaven group has a few party members who don't get brought on every scenario (player alternate characters and guest characters). These party members have equipment and personal goals, both of which reduce the general supply, but they don't fit in the active 4-player roster and are not saved with the campaign manager. When loading our game state to a new copy of the mod, I need to copy the inactive characters as Saved Objects and manually delete their duplicate equipped items from the shop.
Would it be possible for the campaign manager to process inactive (boxed) characters into the save file, and then create them in a boxed state somewhere on the table when the save file is loaded? Perhaps there could be an "Inactive Characters" mat somewhere on the table, and any character boxes sitting on that mat when the user clicked "Save" would be saved with the campaign manager.
(I love the mod, BTW. We've been playing on an old version of Gloomhaven Fantasy UI and I just finished using your mod to migrate our campaign to the freshest version. It's a relief to not have to deal with migrating a few dozen Saved Objects!)

Yeah, that would be a very nice feature. I didn't touch that yet, because it was tricky to do as you can't access objects within a box, so you would have to unpack it, wait until that is done and then check the objects. However, with the knowledge I have now about TTS it should be also possible to do so while the objects are boxed, just hadn't the time yet to fully test it.

Phew, finally got this working. :-)
Inactive characters are now saved and loaded as well. If you have an already packed character in your save, you have to unpack and repack it again. The reason is that the buttons for adding and removing characters are changed through this mod. Only when packing a character with this new version of the buttons can an inactive character be stored. Otherwise it won't be found.


Invalid exception formatting within Chocies.validator

I'm' sorry to spam you with issues like that :^) I noticed a small bug in Choices.validator, you don't pass the value to exception:
python3.4/site-packages/djchoices/choices.py in __call__(self, value)
    118     def __call__(self, value):
    119         if value not in self.values:
--> 120             raise ValidationError('Select a valid choice. %(value)s is not '
    121                                   'one of the available choices.')
    122

ValidationError: ['Select a valid choice. %(value)s is not one of the available choices.']

I'm' sorry to spam you with issues like that
Don't worry about that, better too much input than too little!

Thanks for the report, this looks like a bug indeed.


Bug, input button & select button showing different rows

2 buttons have class col-sm- & col-md-
It's make conflict in css.
Use only one: col-sm- or col-md-

this is not a bug but intended behavior - sm and md both add up to 12. you need to make sure that your container is 100% width and with no padding/margin: http://stackoverflow.com/questions/25178130/why-is-bootstrap-pushing-last-column-on-next-line

We've dropped the ugly bootstrap column usage in v0.9.6.  Closing as dupe of #503.

now allow-clear is broken with theme="bootstrap"...
http://plnkr.co/edit/yzoAL0PRkOyGHeuMbGj2?p=preview


[Feature] flag or config option for disallowing conflicting dependencies

thought: we're going to have to implement an equivalent to the luarocks loader to support multiple versions of the same dependency.
And for people who don't have the loader enabled, we might want to consider adding a flag that disallows conflicting dependencies. Which would mean we'll need a more sophisticated dependency resolution.
Originally posted by @mrcjkb in https://github.com/nvim-neorocks/rocks/pull/74#discussion_r1765708240

This could be a bit of metadata that is read from the tree's lockfile in the future (e.g. a dependency or rock could have an only or singleton flag attached). Good point though.

Copying over from different thread.


GitHub Actions: Package: Run setupmac

Our packaged builds were missing pre-built binary resources.

Double checking the artifact from the build (dmg), I do see the file:
$ file /Volumes/rancher-desktop\ 0.0.1/rancher-desktop.app/Contents/Resources/resources/darwin/minikube
/Volumes/rancher-desktop 0.0.1/rancher-desktop.app/Contents/Resources/resources/darwin/minikube: Mach-O 64-bit executable x86_64


Trying to create a main header

It's using the first sub-header about volumes as the header of the page.  I'm thinking this might fix that?

This change is

wait, i think i might see what the real culprit is here...

Related to #1942 ?

Yes, quite possibly!
On Wed, Dec 14, 2016, 3:41 PM Brian Grant<EMAIL_ADDRESS>wrote:

Related to #1942
https://github.com/kubernetes/kubernetes.github.io/pull/1942 ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/kubernetes.github.io/pull/1955#issuecomment-267150260,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALMeh_mSTybc7i2oznU8brAPCWt61eeks5rIFRhgaJpZM4LNWne
.

The titling issue got fixed by #1942. I'll go ahead and close this to avoid duplicate titling.


[FLINK-10757] [tests] Avoid port conflicts in AbstractTaskManagerProc…

…essFailureRecoveryTest
What is the purpose of the change
Harden AbstractTaskManagerProcessFailureRecoveryTest by avoid port conflicts.
The relative log is https://api.travis-ci.org/v3/job/449439623/log.txt and test fails on

Caused by: java.net.BindException: Address already in use

thus start the cluster with rest port 0 to avoid possible port conflict.
Verifying this change
This change is a trivial rework and it itself is a test.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with (Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

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

cc @zentol @tillrohrmann

Travis doesn't start for over six hours, manually re-trigger.

@zentol travis stalls on status "created" on this pull request. I have no idea on what happened and how to process.

There are 3 more builds in front of this one. It should be done in 4 hours.


InsecureRequestWarning:

What is the fast/best solution for this error?
/usr/lib/python3/dist-packages/urllib3/connectionpool.py:849: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
InsecureRequestWarning)

This may be relevant.
Slack SSL Verification Disabled by Default #2955

We also recommend asking questions on the Gitter channel
https://gitter.im/Yelp/elastalert

Yelp/elastalert is no longer maintained. Please use jertel/elastalert. Questions to the discussion below
https://github.com/jertel/elastalert/discussions


Go back to main screen when Download finished

I'm not sure if I am just dumb or if this is the intended behaviour:
When a download finishes (I tested Arch latest and NixOS minimal), it stays on this screen forever:

I would expect to be able to click a button and go back to the main page, or similar :see_no_evil:

There should be a button at the bottom of the screen :

Ahhh, now I see a button! However, I think the text colour is set to white in the light theme - which makes the text invisible on both the "Cancel Download" and "Close" button:

With Dark Mode, everything is OK.

Thanks! :)


generateUrls() does not work with universal-router/sync

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] other (Please do not submit support requests here (below))

This is pretty easy to replicate:
import UniversalRouterSync from 'universal-router/sync'
import generateUrls from 'universal-router/generateUrls'

const router = new UniversalRouter([])
const url = generateUrls(router)

The above code throws an error with the message An instance of UniversalRouter is expected.
Looking at the source it's pretty easy to understand why. The implementation of generateUrls() checks explicitly for instanceof UniversalRouter, and UniversalRouterSync does not inherit from UniversalRouter.

Thank you for the bug report! The fix is released: v8.3.0 🎉

Thanks for the quick fix!


Implement file_copy

@itdependsnetworks

Definition of methods (work in progress):
I view the higher level operations we want initially is a get_file and a put_file. The long term goal probably being an abstraction that lets you relatively easily do an OS upgrade.
  def get_file

  def put_file

  def _scp_connect

  def _remote_md5

  def _local_md5

  def _compare_md5

  def _remote_space_available

  def _local_space_available

  def _verify_space_available


Task: Public available test plan

We should make all of our automation and manual test plan available at https://github.com/harvester/tests , so adding the test plan can be a part of the requirement for closing the issue.
Ref:
https://github.com/longhorn/longhorn-tests
https://longhorn.github.io/longhorn-tests/manual/
https://longhorn.github.io/longhorn-tests/integration/
https://github.com/longhorn/longhorn/issues/2460#issuecomment-919132422

The tests are available at https://github.com/harvester/tests in the docs folder and the tests are also available and being published on CI at https://harvesterhci.io/tests/ .


UFADE - Universal Forensic Apple Device Extractor

I have added UFADE into readme.md kindly check it out and accept the changes !
click here to know more about it
let me know your thoughts on these changes!

Thank you for the contribution!


Moving .bazelrc according to best practices.

See https://docs.bazel.build/versions/master/best-practices.html#bazelrc

Nice, thanks!


Markdown Cheatsheet

Description
This documentation will cover the syntax of Markdown.
Domain
Open Source
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

@Aryamanz29 I forgot to add this 2.6_Markdown_CheatSheet file in Readme.
So I have created an another PR, Kindly merge it.
Link added in Readme


Required view (...) with ID (...) for method (..) was not found.

Hi!
Having @OnClick(R2.id. ...) annotation in library module crashes the app with java.lang.IllegalStateException Required view (...) with ID (...) for method (..) was not found. when Butterknife.bind(this) is called.
   @BindView(R2.id.willWork)
    lateinit var thisNeedsToBeHere: View

    @OnClick(R2.id.btnInLibrary)
    fun start(){ ... }

    @OnClick(R2.id.willWork)
    fun start2(){ ...}

Generated ViewBinding class looks as follows:

You may notice it does not use R.id.* reference for the btnInLibrary view.
Interesting thing is that if you add BindView annotation with R2.id.willWork resource - it references proper resource.
Tested with kotlin 1.3.31, 1.3.21 and 1.3.20 (which supposedly fixed #974)
Butterknife 10.1.0, AGP 3.4.1
Repro project: https://github.com/mateuszkwiecinski/butternkife_bug

I am also encountering this problem.
This issue looks to be a duplicate of https://github.com/JakeWharton/butterknife/issues/1498.

Seems to be still a kapt issue: https://youtrack.jetbrains.net/issue/KT-29929

I am also encounted this problem.


FW Loiter Radius not working?

http://dash.oznet.ch/view/Ygp8Hg78EniX7oRfS8kcZM

Its not the loiter radius - there is more to it.
Is there a gain parameter which would increase the commanded roll angle?

I'm closing this, since its not really an issue but a logging / plotting / gain tuning thing which confused us.


Feature/slurmdbd

First part of incorporating slurmdbd in the slurm setup.

Hopefully we can build both docker images using one workflow file. Also, uploading the base image to Github packages could make the workflow a bit easier.


[Question] Writing Tests

I have a complex application, with several sub routes, as such when writing tests I render the specific page and write tests for that page rather than rendering the whole application. I have found writing tests with reach router extremely difficult and I'm not sure if I'm missing something.
Here is a code sandbox example, I'd appreciate it if you could point me into the right direction.
https://codesandbox.io/s/reach-testing-oz2l0

Solved.
You must spy on the history object before you render the LocationProvider.
const history = createHistory(createMemorySource("/"));
  jest.spyOn(history, "navigate");

  return {
    ...render(
      <LocationProvider history={history}>
        <Dashboard />
      </LocationProvider>
    ),
    history
  };


Preserve Children Order in case of mixed content.

In current functionality, in order to maintain Chilren Order in case if mixed content like this:
<TailParas fid="td">
	<Para>That capitulation reflects the conundrum facing Mr. Ballmer's successor, <En cat="pe" fpRef="958861" instRef="7" djRef="9589153">Satya Nadella</En>, who took over two years ago, as he runs out of options to bolster <En cat="co" fpRef="27147355" instRef="8" djRef="mcrost">Microsoft</En>'s standing in mobile technology. His new strategy aims to help information-technology managers cope with the increasingly central position of smartphones in corporate networks. The company is betting that corporate customers will want to use <En cat="co" fpRef="27147355" instRef="9" djRef="mcrost">Microsoft</En>'s technology to lock down and control devices on their networks.</Para>
	<Para>In a statement, Mr. Nadella also touted its Continuum feature, which enables a smartphone running Windows 10 to function as a surrogate PC when connected to a monitor and keyboard.</Para>
	<Para>
		<En cat="co" fpRef="27147355" instRef="10" djRef="mcrost">Microsoft</En> must establish its Windows operating system on a substantial portion of smartphones or accept a diminished presence in the technology landscape, where mobile devices outnumber personal computers—and it may not get another chance, said Forrester Research Inc. analyst Julie Ask.</Para>
</TailParas>

you need to pass these flags:
explicitChildren: true, preserveChildrenOrder : true, charsAsChildren: true, includeWhiteChars: true
And these flag generates lots of extraneous json nodes.
With this change I have added a flag called preserveChildrenOrderForMixedContent (default is false). But if it's turned on, it will generate following output. Basically, it's smart enough to detect that it has mixed content and will wrap the object only if required in order to maintain the order.
{
  "$": {
    "fid": "td"
  },
  "Para": [
    {
      "$$": [
        {
          "_": "That capitulation reflects the conundrum facing Mr. Ballmer\'s successor, "
        },
        {
          "En": {
            "_": "Satya Nadella",
            "$": {
              "cat": "pe",
              "fpRef": "958861",
              "instRef": "7",
              "djRef": "9589153"
            }
          }
        },
        {
          "_": ", who took over two years ago, as he runs out of options to bolster "
        },
        {
          "En": {
            "_": "Microsoft",
            "$": {
              "cat": "co",
              "fpRef": "27147355",
              "instRef": "8",
              "djRef": "mcrost"
            }
          }
        },
        {
          "_": "\'s standing in mobile technology. His new strategy aims to help information-technology managers cope with the increasingly central position of smartphones in corporate networks. The company is betting that corporate customers will want to use "
        },
        {
          "En": {
            "_": "Microsoft",
            "$": {
              "cat": "co",
              "fpRef": "27147355",
              "instRef": "9",
              "djRef": "mcrost"
            }
          }
        },
        {
          "_": "\'s technology to lock down and control devices on their networks."
        }
      ]
    },
    "In a statement, Mr. Nadella also touted its Continuum feature, which enables a smartphone running Windows 10 to function as a surrogate PC when connected to a monitor and keyboard.",
    {
      "$$": [
        {
          "_": "\\n            "
        },
        {
          "En": {
            "_": "Microsoft",
            "$": {
              "cat": "co",
              "fpRef": "27147355",
              "instRef": "10",
              "djRef": "mcrost"
            }
          }
        },
        {
          "_": " must establish its Windows operating system on a substantial portion of smartphones or accept a diminished presence in the technology landscape, where mobile devices outnumber personal computers—and it may not get another chance, said Forrester Research Inc. analyst Julie Ask."
        }
      ]
    }
  ]
}

Coverage decreased (-13.7%) to 83.333% when pulling 6841f84cf3fe80c95f3b5bcc999cf5e1df47a370 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

Coverage decreased (-13.7%) to 83.333% when pulling 12cbab1299d305e4d2aafcb0a425aed66b43c637 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

Coverage decreased (-4.5%) to 92.473% when pulling 065216217ad1ef673f6a2667a26f8f08576beb39 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

Coverage increased (+0.3%) to 97.312% when pulling 0f768e0993afffd8c73417a6dc77726992d66bd0 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

:+1:  Change was very useful for my use case. Can we merge this with master branch?

Coverage increased (+0.08%) to 97.067% when pulling e3218af981bfdff4bc241a2151a5c9b2dd657f85 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

Coverage increased (+0.08%) to 97.067% when pulling e3218af981bfdff4bc241a2151a5c9b2dd657f85 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

Coverage increased (+0.08%) to 97.067% when pulling e3218af981bfdff4bc241a2151a5c9b2dd657f85 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

Coverage increased (+0.08%) to 97.067% when pulling e3218af981bfdff4bc241a2151a5c9b2dd657f85 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

Coverage increased (+0.08%) to 97.067% when pulling e3218af981bfdff4bc241a2151a5c9b2dd657f85 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

Coverage increased (+0.08%) to 97.067% when pulling e3218af981bfdff4bc241a2151a5c9b2dd657f85 on chavdam:patch-1 into 129ebba8b313e92389c111c0b477c89fb78b37ad on Leonidas-from-XIV:master.

@Leonidas-from-XIV - Could you please look at the pull request I created and see if there is any possibility of merging this to master ?
Thanks,
Mahesh

@Leonidas-from-XIV  I would also need this, as I am trying to process XLIFF (xlf) files generated by angular2 for merging and difference operations.
@chavdam will this allow round-tripping (e.g. loading and saving back) the example you mentioned initially?

@paumayr  - If I understand it correctly, you are asking if we can have bidirectional conversion to/from xml to json. Right? I should work.

@chavdam Awesome... I currently have the following issue: angular2s ng-xi18n generates an xlf file that contains something like this:
<trans-unit id="aa69f7913cb1d24946f9aae8366cc24bd405dc8d" datatype="html">
   <source>Version: <x id="INTERPOLATION"/>,
BuildInfrastructureVersion: <x id="INTERPOLATION_1"/>.<x id="INTERPOLATION_2"/>.<x id="INTERPOLATION_3"/>.<x id="INTERPOLATION_4"/>,

BuildInfrastructureInformation:

setting page server versions, build infrastructure version field

I writing tools to deal with such files. ng-xi18n currently always generates localizations for all packages, including dependencies. Now, once I load and store such a file, it reorders mixed content like this:
  <trans-unit id="aa69f7913cb1d24946f9aae8366cc24bd405dc8d" datatype="html">
    <source>

     ASDFJKLMVersion: ,
BuildInfrastructureVersion: ...,
BuildInfrastructureInformation:

      <x id="INTERPOLATION"/>
      <x id="INTERPOLATION_1"/>
      <x id="INTERPOLATION_2"/>
      <x id="INTERPOLATION_3"/>
      <x id="INTERPOLATION_4"/>
      <x id="INTERPOLATION_5"/>
    </source>
    <target state="translated" xml:lang="de">

Version: ,
BuildInfrastructureVersion: ...,
BuildInfrastructureInformation:

      <x id="INTERPOLATION"/>
      <x id="INTERPOLATION_1"/>
      <x id="INTERPOLATION_2"/>
      <x id="INTERPOLATION_3"/>
      <x id="INTERPOLATION_4"/>
      <x id="INTERPOLATION_5"/>
    </target>
<note priority="1" from="description">setting page server versions, build infrastructure version field</note>
</trans-unit>

So obviously it causes confusion in the UI with the mixed-up text.

@paumayr - Did you test using my patch to see if it satisfies your requirement ?

@chavdam sorry for the late (really late) reply. I just tested your patch, and I think it works for parsing. But I think the builder requires support for converting $$ back to empty text:
source:
<trans-unit id="aa69f7913cb1d24946f9aae8366cc24bd405dc8d" datatype="html">
    <source>BuildInfrastructureVersion: <x id="INTERPOLATION"/>.<x id="INTERPOLATION_1"/>.<x id="INTERPOLATION_2"/>.<x id="INTERPOLATION_3"/></source>
    <target state="translated" xml:lang="de">
      Build Infrastruktur Version: <x id="INTERPOLATION"/>.<x id="INTERPOLATION_1"/>.<x id="INTERPOLATION_2"/>.<x id="INTERPOLATION_3"/>
    </target>
    <note priority="1" from="description">setting page server versions, build infrastructure version field</note>
</trans-unit>

The result is the following (with pretty printing)
<trans-unit id="aa69f7913cb1d24946f9aae8366cc24bd405dc8d" datatype="html">
    <source>
      <$$>BuildInfrastructureVersion: </$$>
      <$$>
        <x id="INTERPOLATION"/>
      </$$>
      <$$>.</$$>
      <$$>
        <x id="INTERPOLATION_1"/>
      </$$>
      <$$>.</$$>
      <$$>
        <x id="INTERPOLATION_2"/>
      </$$>
      <$$>.</$$>
      <$$>
        <x id="INTERPOLATION_3"/>
      </$$>
    </source>
    <target state="translated" xml:lang="de">
      <$$>
      Build Infrastruktur Version: </$$>
      <$$>
        <x id="INTERPOLATION"/>
      </$$>
      <$$>.</$$>
      <$$>
        <x id="INTERPOLATION_1"/>
      </$$>
      <$$>.</$$>
      <$$>
        <x id="INTERPOLATION_2"/>
      </$$>
      <$$>.</$$>
      <$$>
        <x id="INTERPOLATION_3"/>
      </$$>
      <$$>
    </$$>
    </target>
    <note priority="1" from="description">setting page server versions, build infrastructure version field</note>
</trans-unit>

When disabling pretty-printing and removing <$$> and </$$> tags using something like this after writing I get the results I need:
xml = xml.replace(/(<\$\$>|<\/\$\$>)/g, "");

soo, almost :)..
Could we add an option to write the content of a specific element tag without writing the element tags?


Migrate CLI to typer

Context
Because it's better
Proposal
No response
Updates and actions
No response

Because it's better

That is a wonderful reason! 😝

Ah, but maybe won't work with instance methods because it will read self as an Argument https://github.com/tiangolo/typer/issues/309#issuecomment-894774248


Integrity Check breaking

IC error:
models/48h6e2.edm @ c:/users/XXXX/saved games/dcs/mods/tech/highdigitsams/shapes

Solved by @Auranis.


Do not allocate in TaskItem.GetHashCode

This was causing about 0.3% of allocations (12 MB) of a particular scenario installing NuGet packages.

While I didn't search MSBuild's code base - I did look for every GetHashCode that allocated enough to hit PerfView's threshold, and either fixed them or filed them. There were no more MSBuild ones.

@rainersigwald @jeffkl Can one you merge this in? I don't have permissions.

Looks like the Ubuntu build is failing due to a timeout, unsure if this introduced it?

@dotnet-bot test Ubuntu16.04 Build for CoreCLR please
It passed on your change before, so I don't think it's related.

Thanks @rainersigwald!


Is there a way i can test for choices?

Hi
Sorry to bother you, but I don't know if I'm doing something wrong.
If I wanted to test my logic and verify the fact that given an input, i can expect one or more choices from a dialog, could i do that?
I am able to test the text portion of replies in unit tests but i don't understand if i can test that the length of outputted choices, for a specific channel, are exactly 3.
using AzureMapsToolkit;
using Microsoft.Bot.Builder;
using Microsoft.Bot.Builder.Dialogs;
using Microsoft.Bot.Builder.Testing;
using Microsoft.Bot.Builder.Testing.XUnit;
using Microsoft.Bot.Schema;
using Moq;
using System.Threading;
using System.Threading.Tasks;
using Xunit;
using Xunit.Abstractions;

...
public class TelegramStartDialogTests
{
private MockRepository mockRepository;

private Mock<IAzureMapsServices> mockAzureMapsServices;
private Mock<IBotTelemetryClient> mockBotTelemetryClient;
private readonly IMiddleware[] _middlewares;

public TelegramStartDialogTests(ITestOutputHelper output)
{
this.mockRepository = new MockRepository(MockBehavior.Loose);

this.mockAzureMapsServices = this.mockRepository.Create<IAzureMapsServices>();
this.mockBotTelemetryClient = this.mockRepository.Create<IBotTelemetryClient>();
_middlewares = new[] { new XUnitDialogTestLogger(output) };
}

private TelegramStartDialog CreateTelegramStartDialog()
{
return new TelegramStartDialog(
this.mockAzureMapsServices.Object,
this.mockBotTelemetryClient.Object);
}

[Fact]
public async Task ChooseTroubleAsync_ReceveingValidMessage_ShouldSuggest3Options()
{
// Arrange
var telegramStartDialog = this.CreateTelegramStartDialog();
var testClient = new DialogTestClient(Microsoft.Bot.Connector.Channels.Telegram, telegramStartDialog, _middlewares);

var reply = await testClient.SendActivityAsync<Activity>("anymessage");
Assert.Contains("# Welcome", reply.Text); //Ok

//How do i test that choices are 3 here?
}

This is the part I'd like to test for choices:
        public async Task<DialogTurnResult> MethodNameAsync(WaterfallStepContext stepContext, CancellationToken cancellationToken)
        {
            var data = stepContext.Context.Activity.ChannelData as dynamic;

            string msg = @$"# Welcome {data?.message?.from?.first_name}

How can i help you?";
            var retryText = $"Please choose one option.\n\n{msg}";
            var retryPromptText = MessageFactory.Text(retryText, retryText, InputHints.ExpectingInput);

            return await stepContext.PromptAsync("ChoiceTextDialog", new PromptOptions()
            {
                Prompt = MessageFactory.Text(msg),
                Choices = ChoiceFactory.ToChoices(new List<string> { "option a", "option b", "option c" }),
                RetryPrompt = retryPromptText
            }, cancellationToken);
        }

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

ID: afcaa01f-5e9c-8b11-9dbe-9f9740a45397
Version Independent ID: b24363df-987f-75c3-a8b8-eab8509aa13c
Content: Unit test bots - Bot Service
Content Source: articles/v4sdk/unit-test-bots.md
Service: bot-service
GitHub Login: @gabog
Microsoft Alias: ggilaber

For most channels, a choice prompt sends a message activity that contains suggested actions. The Activity protocol describes the "shape" of activities. See Suggested action for specifics.
We haven't discussed in the docs how the prompts are implemented, but let me know if you'd like pointers into the SDK code.

Debug a bot with inspection middleware might be useful for discovering how individual activities are formed. You could then use that to inform how you construct your tests.

Hi Jonathan
thank you for your answers, i sorted it out.
Basically something breaks but doesn't really throw any Exception when instantiating choices with strings that are too long.
When this happens, no actions get suggested, and during the context.PromptAsync(...) i get back a generic error.
Now that i used shorter strings for choices, everything is correctly working(both in tests and in the live BOT).
Thank you again

Made a suggestion/feature request in the dev repo.

Hi Jonathan
thank you for your answers, i sorted it out.
Basically something breaks but doesn't really throw any Exception when instantiating choices with strings that are too long.
When this happens, no actions get suggested, and during the context.PromptAsync(...) i get back a generic error.
Now that i used shorter strings for choices, everything is correctly working(both in tests and in the live BOT).
Thank you again

Hi @MACEL94 Can you help me with the repro steps for this issue and the generic error you get when choices are too long. I am looking into this to see if we can show a clear exception when choice prompts are too long.

@MACEL94 Hi Francesco, Can you help me with the repro steps for this issue and the generic error you get when choices are too long. I am looking into this to see if we can show a clear exception when choice prompts are too long.

Hi @skommireddi
i changed the code a lot since the first of july when i wrote this issue, and i also updated every package to their latest version.
I tried but i'm not able to reproduce it anymore, so it was either my error or it was fixed.
Thank you for your time and help, and sorry if i wasn't clear enough on the code used.

Thanks @MACEL94 for getting back. Since this cannot be reproduced, I will close the issue 5759- Throw clear exception when choice prompt strings are too long. linked to this for now.