trouble dependency

it's truly great, but now I'm facing trouble facing dependency in firebase while I've more installed another package, one say is play-service-vision (I used that on react-native-camera) and I just excluded that and facing another one for base and auth
below is my android/app/build.gradle file
apply plugin: "com.android.application"

import com.android.build.OutputFile

/**
 * The react.gradle file registers a task for each build variant (e.g. bundleDebugJsAndAssets
 * and bundleReleaseJsAndAssets).
 * These basically call `react-native bundle` with the correct arguments during the Android build
 * cycle. By default, bundleDebugJsAndAssets is skipped, as in debug/dev mode we prefer to load the
 * bundle directly from the development server. Below you can see all the possible configurations
 * and their defaults. If you decide to add a configuration block, make sure to add it before the
 * `apply from: "../../node_modules/react-native/react.gradle"` line.
 *
 * project.ext.react = [
 *   // the name of the generated asset file containing your JS bundle
 *   bundleAssetName: "index.android.bundle",
 *
 *   // the entry file for bundle generation
 *   entryFile: "index.android.js",
 *
 *   // whether to bundle JS and assets in debug mode
 *   bundleInDebug: false,
 *
 *   // whether to bundle JS and assets in release mode
 *   bundleInRelease: true,
 *
 *   // whether to bundle JS and assets in another build variant (if configured).
 *   // See http://tools.android.com/tech-docs/new-build-system/user-guide#TOC-Build-Variants
 *   // The configuration property can be in the following formats
 *   //         'bundleIn${productFlavor}${buildType}'
 *   //         'bundleIn${buildType}'
 *   // bundleInFreeDebug: true,
 *   // bundleInPaidRelease: true,
 *   // bundleInBeta: true,
 *
 *   // whether to disable dev mode in custom build variants (by default only disabled in release)
 *   // for example: to disable dev mode in the staging build type (if configured)
 *   devDisabledInStaging: true,
 *   // The configuration property can be in the following formats
 *   //         'devDisabledIn${productFlavor}${buildType}'
 *   //         'devDisabledIn${buildType}'
 *
 *   // the root of your project, i.e. where "package.json" lives
 *   root: "../../",
 *
 *   // where to put the JS bundle asset in debug mode
 *   jsBundleDirDebug: "$buildDir/intermediates/assets/debug",
 *
 *   // where to put the JS bundle asset in release mode
 *   jsBundleDirRelease: "$buildDir/intermediates/assets/release",
 *
 *   // where to put drawable resources / React Native assets, e.g. the ones you use via
 *   // require('./image.png')), in debug mode
 *   resourcesDirDebug: "$buildDir/intermediates/res/merged/debug",
 *
 *   // where to put drawable resources / React Native assets, e.g. the ones you use via
 *   // require('./image.png')), in release mode
 *   resourcesDirRelease: "$buildDir/intermediates/res/merged/release",
 *
 *   // by default the gradle tasks are skipped if none of the JS files or assets change; this means
 *   // that we don't look at files in android/ or ios/ to determine whether the tasks are up to
 *   // date; if you have any other folders that you want to ignore for performance reasons (gradle
 *   // indexes the entire tree), add them here. Alternatively, if you have JS files in android/
 *   // for example, you might want to remove it from here.
 *   inputExcludes: ["android/**", "ios/**"],
 *
 *   // override which node gets called and with what additional arguments
 *   nodeExecutableAndArgs: ["node"],
 *
 *   // supply additional arguments to the packager
 *   extraPackagerArgs: []
 * ]
 */

project.ext.react = [
    entryFile: "index.js"
]

apply from: "../../node_modules/react-native/react.gradle"

/**
 * Set this to true to create two separate APKs instead of one:
 *   - An APK that only works on ARM devices
 *   - An APK that only works on x86 devices
 * The advantage is the size of the APK is reduced by about 4MB.
 * Upload all the APKs to the Play Store and people will download
 * the correct one based on the CPU architecture of their device.
 */
def enableSeparateBuildPerCPUArchitecture = false

/**
 * Run Proguard to shrink the Java bytecode in release builds.
 */
def enableProguardInReleaseBuilds = false

android {
    compileSdkVersion rootProject.ext.compileSdkVersion
    buildToolsVersion rootProject.ext.buildToolsVersion

    defaultConfig {
        applicationId "com.xxx"
        minSdkVersion rootProject.ext.minSdkVersion
        targetSdkVersion rootProject.ext.targetSdkVersion
        versionCode 1
        versionName "1.0"
        ndk {
            abiFilters "armeabi-v7a", "x86"
        }

        multiDexEnabled true

    }
    signingConfigs {
        release {
            if (project.hasProperty('MYAPP_RELEASE_STORE_FILE')) {
                storeFile file(MYAPP_RELEASE_STORE_FILE)
                storePassword MYAPP_RELEASE_STORE_PASSWORD
                keyAlias MYAPP_RELEASE_KEY_ALIAS
                keyPassword MYAPP_RELEASE_KEY_PASSWORD
            }
        }
    }
    splits {
        abi {
            reset()
            enable enableSeparateBuildPerCPUArchitecture
            universalApk false  // If true, also generate a universal APK
            include "armeabi-v7a", "x86"
        }
    }
    buildTypes {
        release {
            minifyEnabled enableProguardInReleaseBuilds
            proguardFiles getDefaultProguardFile("proguard-android.txt"), "proguard-rules.pro"
            signingConfig signingConfigs.release

        }
    }
    // applicationVariants are e.g. debug, release
    applicationVariants.all { variant ->
        variant.outputs.each { output ->
            // For each separate APK per architecture, set a unique version code as described here:
            // http://tools.android.com/tech-docs/new-build-system/user-guide/apk-splits
            def versionCodes = ["armeabi-v7a":1, "x86":2]
            def abi = output.getFilter(OutputFile.ABI)
            if (abi != null) {  // null for the universal-debug, universal-release variants
                output.versionCodeOverride =
                        versionCodes.get(abi) * 1048576 + defaultConfig.versionCode
            }
        }
    }
}

dependencies {
    implementation(project(":react-native-google-signin")){
        exclude group: "com.google.android.gms" // very important
    }
    implementation 'com.google.android.gms:play-services-auth:15.0.1'
    implementation (project(':react-native-text-detector')) {
        exclude group: 'com.google.android.gms', module: 'play-services-base'
        exclude group: 'com.google.android.gms', module: 'play-services-vision'
    }
    implementation project(':react-native-select-contact-android')
    implementation project(':react-native-firebase')
    implementation "com.google.android.gms:play-services-base:15.0.1"
    implementation "com.google.firebase:firebase-core:16.0.1"
    implementation "com.google.firebase:firebase-messaging:17.1.0"
    implementation 'me.leolin:ShortcutBadger:1.1.21@aar'
    implementation "com.google.firebase:firebase-ads:15.0.1"
    implementation "com.google.firebase:firebase-auth:16.0.2"
    implementation 'com.android.support:multidex:1.0.3'

    implementation 'com.facebook.android:facebook-android-sdk:4.34.0'
    implementation project(':react-native-fbsdk')
    implementation project(':react-native-share')

    implementation project(':react-native-image-picker')
    implementation project(':react-native-bottom-sheet-behavior')
    implementation project(':react-native-svg')
    implementation project(':react-native-navigation')

    implementation fileTree(dir: "libs", include: ["*.jar"])
    implementation "com.android.support:support-v4:${rootProject.ext.supportLibVersion}"
    implementation "com.android.support:appcompat-v7:${rootProject.ext.supportLibVersion}"
    implementation "com.facebook.react:react-native:+"  // From node_modules
    implementation (project(':react-native-camera')) {
        exclude group: "com.google.android.gms"
    }
}
// apply plugin: "com.google.gms.google-services"
// Run this once to be able to run the application with BUCK
// puts all compile dependencies into folder libs for BUCK to use
task copyDownloadableDepsToLibs(type: Copy) {
    from configurations.compile
    into 'libs'
}

apply plugin: 'com.google.gms.google-services'

and this is my android/build.gradle file
// Top-level build file where you can add configuration options common to all sub-projects/modules.

buildscript {
    repositories {
        jcenter()
        mavenCentral()
        maven {
            url 'https://maven.google.com/'
            name 'Google'
        }
        maven { url 'https://plugins.gradle.org/m2/' }
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:3.1.3'
        classpath 'com.google.gms:google-services:4.0.1'
        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }
}
// apply plugin: 'com.google.gms.google-services'

def googlePlayServicesVersion = '15.0.1'

allprojects {
    repositories {
        configurations.all {
            resolutionStrategy {
                // react-native-admob
                force "com.google.android.gms:play-services-ads:$googlePlayServicesVersion"
                // react-native-maps
                force "com.google.android.gms:play-services-base:$googlePlayServicesVersion"
                force "com.google.android.gms:play-services-maps:$googlePlayServicesVersion"
                // react-native-onesignal
                force "com.google.android.gms:play-services-gcm:$googlePlayServicesVersion"
                force "com.google.android.gms:play-services-analytics:$googlePlayServicesVersion"
                force "com.google.android.gms:play-services-location:$googlePlayServicesVersion"
                force "com.google.android.gms:play-services-vision:$googlePlayServicesVersion"
                //force "com.google.android.gms:play-services-places:$googlePlayServicesVersion"
                // react-native-google-sign-in
                force "com.google.android.gms:play-services-auth:$googlePlayServicesVersion"
                // ml kit
                force "com.google.android.gms:play-services-vision:$googlePlayServicesVersion"
                // force "com.google.android.gms:play-services-vision-common:15.0.2"
            }
        }
        mavenLocal()
        maven {url "https://maven.google.com"}
        google()
        jcenter()
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url "$rootDir/../node_modules/react-native/android"
        }
        maven {
            url 'https://maven.google.com/'
            name 'Google'
        }
        maven { url "https://jitpack.io" }
    }
}

ext {
    buildToolsVersion = "26.0.3"
    minSdkVersion = 16
    compileSdkVersion = 26
    targetSdkVersion = 26
    supportLibVersion = "26.1.0"
}

This is the error
> The library com.google.android.gms:play-services-auth-base is being requested by various other libraries at [[15.0.1,15.0.1]], but resolves to 15.1.0. Disable the plugin and check your dependencies tree using ./gradlew :app:dependencies.
> The library com.google.firebase:firebase-analytics is being requested by various other libraries at [[16.0.1,16.0.1], [16.0.3,16.0.3]], but resolves to 16.0.3. Disable the plugin and check your dependencies tree using ./gradlew :app:dependencies.

Thanks

RNTextDetector uses firebase core and firebase vision as you can see here. I would suggest you to exclude both from dependency and inject seperately in your app's build.gradle
    implementation (project(':react-native-text-detector')) {
        exclude group: 'com.google.firebase'
    }
    implementation "com.google.firebase:firebase-core:16.0.1"
    implementation "com.google.firebase:firebase-ml-vision:17.0.0"  // We support ml vision 17+


a11y: Combobox haspopup value should be listbox & should have aria-controls attribute when expanded

Detailed description

Describe in detail the issue you're having.
Current aria-haspopup value is set to true. While this is correct, it should be changed to listbox which is more specific. If we take a look at the template, you can see that aria-haspopup attribute has been passed twice. Initially as listbox then true, hence overwriting listbox to true.

Once the aria-haspopup value has been updated to listbox & the accessibility checker is run, you can see that aria-controls is missing.

Is this a feature request (new component, new icon), a bug, or a general issue?

Bug

Is this issue related to a specific component?

Combobox & Dropdownlist

What did you expect to happen? What happened instead? What would you like to see changed?

Should have correct role & necessary attributes to go with them

What browser are you working in?

Firefox/Chrome

What version of the Carbon Design System are you using?

4.61.0

Making this PR into next since integration for angular 8 is failing for some reason... Most likely has to do with styles package being updated 🤔


Need to make sure that we the max seen cid is recorded in the log header

make sure we get to this.

Incorrect max_log_buffer_id being maintained in the header must be fixed


SwiftUI toolbar and title disappears when CalendarViewRepresentable present

I have tried creating a calendar with exact as from your PR with example.
Works great, but I have noticed that when put inside a NavigationStack, title and toolbar disappears.
I have tried some alternatives and figured out that toolbar and title appears when I remove both monthHeaderItemProvider and dayItemProvider.
WITH month and day providers

WITHOUT month and day providers

Am I doing something wrong? Is there some kind of a workaround or fix for this?

This is really weird - did you ever find a workaround? I'll play around with this later today to see what's up.

small update, tried on iOS15 and it seems to work there (at least partially, the nav bar is shown but without a title), so something changed in iOS16 that breaks this.

Small update, my app is a tabbed app, I've noticed that when I switch to another tab and back to the calendar tab, the nav bar appears. So looks like a timing issue, a hacky workaround is to force a tab change at app start...however, when scrolling to another month (horizontal layout), the nav bar dissappears again for that month. Switching tabs fixes it for that month for good so it looks like the issue appears when first displaying a month.

Any updates on This? Seems like it's still happening in iOS 16

any updates on this? Seems like it pretty much blocks HorizonCalendar from being used in a SwiftUI context

Any updates?

@bryankeller, any updates for this? Using the providers makes the navigation title hidden, which is just like the OP said.

I'm assuming the issue is related to the CalendarViewRepresentable being a ScrollView which messes with the large navigation title or something..? Whatever it is a simple workaround is to remove it from navigation stack hirachy. Simple way of doing that is overlaying it on top of a placeholder view:

I face the exact same problem. When using the days or months providers, the navigation bar disappears. Have you found any solutions?

Same issue here, and it's a real pain. We use the calendar in only 2 views and we have the same issue in both. Are there any updated on the issue?

Probably not the best solution, but I've fixed this in my apps by wrapping each Calendar in an extra NavigationView

Probably not the best solution, but I've fixed this in my apps by wrapping each Calendar in an extra NavigationView

I tried to do that but it did not work. Could you show me the code snippet of your project?


string indices must be integers when I click import from router

Environment

Python version:
Peering Manager version:

Steps to Reproduce

Expected Behavior

Observed Behavior

This issue is supposed to be fixed in 7e6ec86.


 dropdown disappeared abnormally in

Reproduction link
https://stackblitz.com/edit/angular-stxgla?file=src%2Fapp%2Fapp.component.ts
Steps to reproduce
focus at input, dropdown will show, then input a single character, dropdown disappeared.
Note: It occurs after the page first loaded,or after hot update,or validateStatus changed .
What is expected?
auto-complete dropdown does not disappear.
What is actually happening?
auto-complete dropdown disappeared after input a character.

Environment
Info

ng-zorro-antd
13.3.2

Browser
Edge 103.0.1264.62

Closed because there is no response

@LucianaMq What did you mean? If there is no , nz-autocomplete makes no sense.
https://github.com/NG-ZORRO/ng-zorro-antd/pull/7618/files#diff-e581b7fccfce41aa71bddda98f463eb741a10afee0cb9a521e3df720baa2fd59L144
By the way, I found that the problem is with the nz-input, which use hsotView.clear() to remove all its host view just for hidden feedback icon,without checking.To fix that,we only need to use indexOf to remove the specific feedbackRef.

@yunyuyuan Did you find a workaround ? I have the same problem.

@yunyuyuan Did you find a workaround ? I have the same problem.

Unfortunately no.

@yunyuyuan thank you for figuring out this happens when the autocomplete is placed inside an nz-form-control! i am experiencing the same issue and it saved me a lot of time. for now i am simply avoiding the usage of the nz-form-control when working with the autocomplete


Cosmos DB Terraform Module

As a developer, I can now configure and deploy a Cosmos DB instance
All Submissions:

[YES] Have you followed the guidelines in our Contributing document?
[YES] Have you added an explanation of what your changes do and why you'd like us to include them?
[YES] I have updated the documentation accordingly.
[NA] I have added tests to cover my changes.
[NA] All new and existing tests passed.
[YES] My code follows the code style of this project.
[YES] I ran lint checks locally prior to submission.
[YES] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

What is the current behavior?

The ability to create and deploy an instance of Cosmos DB does not exist.

What is the new behavior?

Users can now use terraform to configure and deploy an instance of Cosmos DB.

Does this introduce a breaking change?

[NO]

Looks like we have a failing test. Is this something we can easily run locally? If not, can we get access to the logs, or can someone paste the issue?

@erikschlegel If you get a second, can you send us the logs from the build test that's failing in the pipeline?

@erikschlegel If you get a second, can you send us the logs from the build test that's failing in the pipeline?

Apologies for the confusion here @ericpera-msft . I believe @nmiodice is currently investigating this build failure which seems to be unrelated to the changes in your PR. @nmiodice will confirm.

Hey I checked this out. The root cause was related to a stale deployment we had. Were still learning with Terraform, forgive us :)
I re-ran your build and it checks out

@nmiodice No problem :) Thanks for looking into it!


replaced options with ArgumentParser

fixed django 1.10 warning.

I guess we need to update travis matrix to oficially supported django versions:
https://github.com/ui/django-rq/pull/173


[BOW-31] Implement old media cleanup

NB: doesn't have E2E test coverage

Codecov Report
Patch coverage has no change and project coverage change: +3.55% :tada:

Comparison is base (7104e53) 12.35% compared to head (c611523) 15.91%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #86      +/-   ##
==========================================
+ Coverage   12.35%   15.91%   +3.55%
==========================================
  Files         109        5     -104
  Lines        6709      955    -5754
  Branches      149        2     -147
==========================================
- Hits          829      152     -677
+ Misses       5880      803    -5077

see 108 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


I don't think this works on Ubuntu 12.04.

We're running Puppetmasters on Ubuntu 12.04 and they happen to be mcollective clients as well.  This module is setting the heartbeat option:
plugin.activemq.heartbeat_interval = 30
Which doesn't seem to be working with the version of the ruby-stomp package that's distributed.
mco find

The find application failed to run, use -v for full error backtrace details: Could not connect to ActiveMQ Server: Setting STOMP 1.1 properties like heartbeat intervals require at least version 1.2.10 of the STOMP gem

ruby-stomp
aptitude show ruby-stomp
Package: ruby-stomp
State: installed
Automatically installed: no
Version: 1.2.2-1
Priority: extra
Section: misc
Maintainer: Debian Ruby Extras Maintainers<EMAIL_ADDRESS>Architecture: all
Uncompressed Size: 287 k
Depends: ruby | ruby-interpreter
Breaks: libstomp-ruby (< 1.1.9-1~), libstomp-ruby (< 1.1.9-1~), libstomp-ruby-doc (< 1.1.9-1~), libstomp-ruby-doc (< 1.1.9-1~),
        libstomp-ruby1.8 (< 1.1.9-1~), libstomp-ruby1.8 (< 1.1.9-1~), libstomp-ruby1.9.1 (< 1.1.9-1~), libstomp-ruby1.9.1 (< 1.1.9-1~)
Replaces: libstomp-ruby (< 1.1.9-1~), libstomp-ruby (< 1.1.9-1~), libstomp-ruby-doc (< 1.1.9-1~), libstomp-ruby-doc (< 1.1.9-1~),
          libstomp-ruby1.8 (< 1.1.9-1~), libstomp-ruby1.8 (< 1.1.9-1~), libstomp-ruby1.9.1 (< 1.1.9-1~), libstomp-ruby1.9.1 (< 1.1.9-1~)
Provides: libstomp-ruby, libstomp-ruby-doc, libstomp-ruby1.8, libstomp-ruby1.9.1
Description: Ruby client for the stomp messaging protocol
 Stomp is a text-oriented wire protocol for messaging (MOM/MQ/JMS) type systems. This library provides two useful interfaces, a low- level
 class, Stomp::Connection, which is a basic protocol implementation, and Stomp::Client, which is designed as a higher level convenience
 API.
Homepage: https://github.com/morellon/stomp

It seems that the mcollecitive package and plugins depend on this package.  We have v1.3.5  of the stomp gem installed, but that's not what mco appears to be using.
There's currently no way in the module to suppress the heartbeat option.

hi,
Why do you think this doesn't work? have you tested it and could provide any error messages?

I  hit enter and posted a blank body.  It's been updated.  My bad.

I can second this. The MCollective server, client will not work on Ubuntu 12.04 with SSL activated.
E, [2016-09-27T13:28:42.487939 #26654] ERROR -- : activemq.rb:377:in ssl_parameters' Failed to set full SSL verified mode: RuntimeError: Stomp gem >= 1.2.2 is needed
E, [2016-09-27T13:28:42.488112 #26654] ERROR -- : runner.rb:107:in main_loop' A failure occurred in the MCollective runner. E, [2016-09-27T13:28:42.488282 #26654] ERROR -- : runner.rb:108:in main_loop' Could not connect to ActiveMQ Server: Stomp gem >= 1.2.2 is needed
E, [2016-09-27T13:28:42.488358 #26654] ERROR -- : runner.rb:109:in main_loop' /usr/lib/ruby/vendor_ruby/mcollective/connector/activemq.rb:301:in connect'
/usr/lib/ruby/vendor_ruby/mcollective/runner.rb:164:in receiver_thread'         /usr/lib/ruby/vendor_ruby/mcollective/runner.rb:142:in start_receiver_thread'
/usr/lib/ruby/vendor_ruby/mcollective/runner.rb:140:in initialize'         /usr/lib/ruby/vendor_ruby/mcollective/runner.rb:140:in new'
/usr/lib/ruby/vendor_ruby/mcollective/runner.rb:140:in start_receiver_thread'         /usr/lib/ruby/vendor_ruby/mcollective/runner.rb:71:in main_loop'
/usr/lib/ruby/vendor_ruby/mcollective/unix_daemon.rb:60:in daemonize_runner'         /usr/lib/ruby/vendor_ruby/mcollective/unix_daemon.rb:13:in daemonize'
/usr/lib/ruby/vendor_ruby/mcollective/unix_daemon.rb:5:in fork'         /usr/lib/ruby/vendor_ruby/mcollective/unix_daemon.rb:5:in daemonize'
/usr/lib/ruby/vendor_ruby/mcollective/unix_daemon.rb:20:in daemonize_runner'         /usr/sbin/mcollectived:74
You'll need to upgrade ruby-stomp to get this to work. What about a check on the ruby-stomp version if SSL is enabled?

The ruby-stomp problem has been around for a while. I remember opening issue https://tickets.puppetlabs.com/browse/MODULES-459 and sending PR https://github.com/voxpupuli/puppet-mcollective/pull/121 to allow specifying the ruby-stomp version. We usually set it to "latest".


Battery sensor not updated when location permission is disabled

Home Assistant Android version:
1.8.0-144
Android version:
10
Phone model:
Samsung Galaxy S10 Plus
Home Assistant version:
0.107.0b1
Last working Home Assistant release (if known):
Description of problem:
My family members don't want to be tracked, so I disabled the location permission for Home Assistant. Now, the battery sensors don't update anymore. Battery sensors should still show up in Home Assistant.
Traceback (if applicable):

Screenshot of problem:
Additional information:
I'm willing to grant Home Assistant the location permission, but then I'd need a switch to disable reporting to Home Assistant from inside the App

This is actually as designed.  Location must be enabled for the app in order for sensor updates to be sent to HA.
https://companion.home-assistant.io/docs/core/sensors
You can simply disable the device_tracker entity from the integration screen so you won't get any location updates.

I know that I can disable the entity in Home Assistant, but this would leave me in the power of re-enabling it whenever I want, which I think is suboptimal. But as you stated, this is not a bug on the Android app, so I'm closing this.


P4-16-spec: Consistently consider the 'miss' field in apply_result(T).

Some places this is mentioned, otherwise it is not.
Issue #885.

This is a bug fix, so let's just merge it.


Twitter API/oauth2 client

Relates to:
Using Twitter with API

Risks
Low

Background
What does this PR do?
This pR created a new twitter client that uses API (oauth2) to connect
What kind of change is this?
A new client

We are doing this because I am building a multi agent app that connects to twitter via API for its ease of use. Not many people can work with twitter cookies.
Documentation changes needed?

Testing
I tested this by updating my access tokens in .env.
I also tested this by passing the access token and refresh token in a kubernetes config map along with character file.
Where should a reviewer start?
packages/client-twitter-api
Detailed testing steps

varkrishin

understood @jkbrooks.
This was aimed as a client_secret approach with functions for token_refresh and access tokens.
I will take at the shared pull request  https://github.com/ai16z/agent-twitter-client/pull/3/files and modify it accordingly to support this usecase


Minor typos

Only a few typos but wasn't sure of the etiquette of merging directly.

connected to #133


Review Presto database (RDS) to ensure that updates are administrated

"AS Andy the SCOS System Administrator,
I WANT to keep all components of the Operating System up to date,
SO THAT any security vulnerabilities that arise are patched before they expose the system to risk.
Acceptance Criteria:

Enable semi-automatic service version updating
Enable backups
A method exists which allow new managed services to automatically be provisioned with these policies.
Existing backups are within the new retention and encryption policies
Identify a way to be notified of available updates to these services.
Write cards with what is learned to perform the same actions for Joomla and Lime Survey's RDS databases.

Technical Notes:

Many of these may be implemented or half implemented.
Pick a reasonable (2 weeks), time to keep backups and a reasonable cadence to take backups (every 24 hours)

Note, this came from the pre-groomed list of cards, from meetings with Warner, Johnson, Jessie, Ben B and Rob.
06-24-19: Migrated from SMRT-1339"

Moved to SCOS


Obsoletion request: GO:0046715 active borate transmembrane transporter activity

From 'Structural and functional insights into the mechanism of action of plant borate transporters' PMID:34112901
There are two types of 'borate' transporters:
channels:
Nodulin 26-like intrinsic proteins (NIPs) (https://tcdb.org/search/result.php?tc=1.A.8.12#1.A.8.12)
carriers which are responsible for the efflux of borate:
BOR/Solute Carrier (SLC) 4 family of secondary active transporters (https://tcdb.org/search/result.php?tc=2.A.31)
The term GO:0046715 'active borate transmembrane transporter activity' is confusing and has been used to annotate NIPs and BOR/Solute Carriers and so should be obsoleted.

GO term ID and Label

GO:0046715	active borate transmembrane transporter activity

Reason for deprecation Put an x in the appropriate box:

[x] The reason for obsoletion is that the term is not clearly defined and usage has been inconsistent.
[ ]  The reason for obsoletion is that this term is pre-composed and should be represented as a GO-CAM model.
[ ]  The reason for obsoletion is that this term represent an assay and not a GO process.
[ ]  The reason for obsoletion is that this represents a phenotype.
[ ] The reason for obsoletion is that the data from the paper for which the term was requested can be accurately described using [appropriate GO term].
[ ] The reason for obsoletion is that this term represents a molecular function.
[ ]  The reason for obsoletion is that this term represents a gene product.
[ ] The reason for obsoletion is that this term refers to a class of proteins.
[ ]  The reason for obsoletion is that this term was added in error.
[x]  The reason for obsoletion is that more specific terms were created.
[ ]  The reason for obsoletion is that this term was an unnecessary grouping term.
[x]  The reason for obsoletion is that the meaning of the term is ambiguous.
[]  The reason for obsoletion is that is not known to be catalyzed by any gene product.
[ ]  The reason for obsoletion is that it more specific than the specificity of any known gene product.
[]  The reason for obsoletion is that there is no evidence that this function/process/component exists.

There are 5 EXP annotations to these terms
UniProtKB:Q8LAI1 ; NIP7-1 PMID:21710975
UniProtKB:Q9SAI4 ; NIP6-1 PMID:18952773
UniProtKB: Q9SV84 ; NIP5-1 PMID:16679457
There is a ticket for nodulin 26-like intrinsic proteins (NIPs) https://github.com/geneontology/go-annotation/issues/5156 for when borate channel activity becomes available (https://github.com/geneontology/go-ontology/issues/27487)
UniProtKB:Q8NBS3 ; SLC4A11 PMID:15525507 (https://www.rhea-db.org/rhea/66817)
UniProtKB:Q8VYR7 ; BOR1 PMID:12447444
should be reannotated to
GO:0080139 borate efflux transmembrane transporter activity

Any other information

For borate efflux the 'most likely form of borate is the charged borate [B(OH4)]−form' tetrahydroxoborate CHEBI:41132 (PMID:34112901)
maps to https://www.rhea-db.org/rhea/66817 for mouse and human SLC4A11.
For the NIPs channels it is
CHEBI:33118 boric acid

Checklist for ontology editor
Check term usage and metadata in Protégé

[ ] check term usage in the ontology
[ ] check internal mappings: RHEA, EC, MetaCyc
[ ] check subset usage
[ ] check taxon constraints

Check annotations

[ ] create Annotation Review spreadsheet

Annotation Review template: https://docs.google.com/spreadsheets/d/1izV3q1jnq_Ybg5MJ53O_b5BCTkGtjl_Fi-cOg3-7eBE/edit#gid=0

Instructions: https://wiki.geneontology.org/Annotation_Review_Request

[ ] create annotation review ticket: https://github.com/geneontology/go-annotation/issues/new?assignees=&labels=annotation+review&template=annotation-review-for-obsoletion.md&title=Review+annotations+to+[INSERT+GO+TERM]
[ ] find external mappings (via IEAS), include in obsoletion notice
[ ] tag curators in Annotation Review ticket. Responsible annotator for each group: https://github.com/geneontology/go-site/blob/master/metadata/group-contacts.csv

Notification

[ ] create obsoletion announcement
[ ] announce to GO friends<EMAIL_ADDRESS>[ ] paste the text in the ontology ticket

#27495


K_Warplist Not Implemented

Search before asking

[X] I searched the issues and found no similar issues.

Description
The K_warplist functionality has not been implemented. This feature is required to display or handle a list of available warp points for players.
Use case
Tasks for Implementing K_Warplist

Review the design requirements for the K_warplist feature.
Implement the necessary code to support warplist functionality.
Add a user interface or commands to access the warplist.
Test the feature thoroughly.

Are you willing to work on and submit a PR to address the issue?

[x] Yes I am willing to submit a PR!

It's not an official feature. Officially, SMD files are used for that, but it would be a nice enhancement ;)

It's not an official feature. Officially, SMD files are used for that, but it would be a nice enhancement ;)

Ohh right! but wait, we have this ticket already: https://github.com/ko4life-net/ko/issues/89
and tbls will anyways be in a 2nd db eventually.


Feat/cc payment method linting fixes

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Formatting PR Title

Need to reformat the PR title, hence closing this PR.


Invert diagnostics dependency

See parent https://github.com/pilosa/pilosa/issues/1093

Fixed in #1161


Current database only records the prevalence and cumulative infections at the present

https://github.com/aezarebski/derp-simulation/blob/a4ac52db4f24c5657b10c00f1293b22430e0b686/main.py#L436
The simulated dataset only includes measurements of the prevalence and cumulative infections at the present. This limits our ability to make predictive models that estimate these values through time using this dataset.
Suggested solutions

storing the whole piece-wise constant function in the database might be a bit much and leaves us with complex objects to work with downstream.
storing the piece-wise constant function at a random selection of times could be a sensible approach, it would allow us to do away with storing the whole representation of the R0 function and we would just move to a flat table for the values at different times. This is the simplest but feels a bit arbitrary.
we could store evaluate the relevant functions on a fine mesh and then do linear interpolation to get approximate values at any time. This is the most elegant but has an approximation error and may be overkill.

Option 2 seems the best for now but would involve a slight reorganisation of the database being produced. It would definitely simplify downstream usage though.

https://github.com/aezarebski/derp-simulation/blob/a4ac52db4f24c5657b10c00f1293b22430e0b686/main.py#L336
Need to pass times through here.

The database should have the following structure (eventually)
/Simulations/
    /Simulation_001/
        TemporalMeasurements (dataset with columns MeasurementTime, Prevalence, ReproductionNumber, Etc)
        PickledTree (dataset)
        SimulationXMLConfig (attribute storing the XML as a string)
	PresentTime (attribute storing the time of the last sample)
	TreeHeight (attribute storing the tree height)

Initial work on this issue should just focus on getting the temporal measurements included into the current database. We can reorganise it into the structure above later.


Cannot add or select BIS gearset

I used this plugin to manage the BIS gearsets for all of the jobs but recently this stopped working.
When clicking on the BiS button of a job, there'd be 3 selections: BiS(0), + Add New, and + Search known sets.
If selected + Add New, I'm able to download a set from etro but after clicking "Save", nothing happens.
If selector + Search known sets I'm able to see the list of saved gearsets, including the ones I chose to "make local copy" from etro, but the leading check button (with tooltip "Select gearset") has no functionality.
I've not yet found any mean that I can specify the BIS of a job and have it compared with my current gear as this plugin used to do.

FYI: I have deleted everything in pluginConfigs\HimbeertoniRaidTool and reinstall the plugin (confirmed all data been cleared) but no luck.

The issues with + Search known sets and + Add New are fixed now.


fix(menu): reposition menu if it would open off screen

Closes #1540

Realized I need to change the animation origin point if a fallback is used, so this will need some more work.


Upgrade SDK version

Pull Request Description
Upgrade Java SDK

Future Release Comment
Upgrading to the latest SDK
Breaking Changes:

None

Features:

None

Fixes:

Update libs


Multisig Transactions Not Taking

Issue

We attempt a new transaction.
TX creator shows TX in wallet, pending confirmations.
No one else can see the TX in the multisig to confirm or deny the TX.
After a refresh the TX disappears altogether.
As you can see on the explorer, we have been trying to get a TX to stick all day:
https://explorer.harmony.one/address/0x65f028CDA0FB723b51edDF8DD852375247eE2f9c

Here is our multisig:
https://multisig.harmony.one/#/safes/0x65f028CDA0FB723b51edDF8DD852375247eE2f9c/transactions
None of the TX's are showing that we've tried today.

@hypnagonia can you take a look at this?

The Validator DAO multisig had a similar problem when "validator wallets" are used for signing tx - Not sure if this is related to the issue or the root cause but I believe all 5 wallets approved for the multisig above are validator wallets.

Also when funds were sent to the multisig, they do not show up in the transactions list, only on explorer.


VCRURLConnection can't handle testing multiple requests to the same endpoint

Hi @dstnbrkr!
First of all, thanks for the great tool, it helps us a lot mocking the network communication and therefore speeding up our integration tests. Unfortunately we cannot use VCRURLConnection like it is for all our tests, because we've discovered a non-trivial issue:
Problem
We're dispatching multiple requests to the same endpoint with different response expectations within one test case, but VCR always overwrites the responses with the latest one.
Recording example:

verify that there are 0 issues: GET https://some.domain/issues - returns an array with 0 issues
create an issue: POST https://some.domain/issues - returns issue id 123
get all issues: GET https://some.domain/issues - returns an array with 1 issue
delete the issue: DELETE https://some.domain/issues/123
get all issues: GET https://some.domain/issues - returns an array with 0 issues

Replaying example:

verify that there are 0 issues: GET https://some.domain/issues - returns an array with 0 issues - luckily this succeeds
create an issue: POST https://some.domain/issues - returns issue id 123 - succeeds
get all issues: GET https://some.domain/issues - returns an array with 0 issues - FAILS
...

We're getting the last recorded response for URL https://some.domain/issues every time in this test case.
Desired Behavior
Always get the correct response for the correct request, even when multiple requests to the same endpoint are dispatched.
Proposed Solution
To have a way to record and replay multiple requests to the same endpoint, i.e. instead of an array of VCRRecording objects, introduce a VCRRecordingSet holding a sorted array of VCRRecording objects as well as the main identifiers for the request (method and URL).
Matching the correct request could work similar to how it works right now with the only difference that we can't fetch the response from the VCRRecodingSetright away but would have to fetch it from the sorted array according to a certain index.
This would of course also mean that the VCRCassette or some other instance would need an additional state representing a request counter to read the VCRRecoding entries at the correct index from the sorted array.
Next Steps
I'll probably fork your implementation, if I find time and get the resources, to create a pull request with the suggested solution. But I'm not sure yet.
Have you thought about this issue already or did you specifically decide to go with the current solution for a special reason?

Thanks @Aaang - ordered recordings per url would be a great feature, does sound handy for mimicking state changes. Your solution sounds great - effectively mapping each request to a queue of responses and dequeueing per request during replay. I'd definitely welcome that pull request, thanks for proposing it!

@Aaang ditto on this (noticed the same issue when using the Google API).


I can't use the excavator, it only says "there are Material Container"

when I'm in a resource deposit I can't excavate materials because it says there are Material Container or there are not material container (I always have the red pieces empty when I try to drill) I press O in an attemp for anything to change but it doesn't, then I press O near the colony and still nothing happens and I still can't excavator

if you are in beta you can use the pre-release 3.0.0. if you not are in beta just re enter in the game.

It didn't work. What does "There are Material Container" mean? I make sure to be in a material deposit and to have empty material (red) containers when I try to excavate

it's a text error. should be "There are no Material Container". if you  just re enter in the game should it work


Optimize log printing

Optimize log printing

Codecov Report

Merging #91 into dev will decrease coverage by 0.07%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev      #91      +/-   ##
============================================
- Coverage     29.29%   29.22%   -0.08%
  Complexity      174      174
============================================
  Files            66       66
  Lines          2082     2087       +5
  Branches        128      131       +3
============================================
  Hits            610      610
- Misses         1450     1455       +5
  Partials         22       22

Impacted Files
Coverage Δ
Complexity Δ

...crosssdk/performance/transfer/DagTransferUser.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...nk/wecrosssdk/performance/transfer/DagUserMgr.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

Continue to review full report at Codecov.

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

Well done @chenjiawei514 , why not optimize some logger.debug() logger printing by the way :)
Thx for your help!


Add a syntax to search only changeset comments (tags where key=comment)

As per, #13 we will default to searching only through changeset discussions. Add a syntax to allow user to search only through comments - like comments:foo as the search or so.

Right now we're doing a full text search through comments and discussions after some db optimization. So this is not really required currently. Closing.


[DISTANCE] LOW: UpdateDistanceRequest 1:1:1 - UpdateMoneyRequestTag - Split out into its own command

There will be three pieces to this:

Auth: Using the logic from EditMoneyRequest, create a new command called UpdateMoneyRequestTag following the example in https://github.com/Expensify/Auth/pull/8986
Web-E: Add a new API endpoint called UpdateMoneyRequestTag following the example in https://github.com/Expensify/Web-Expensify/pull/39445
App: Implement the new API command following the example in https://github.com/Expensify/App/pull/30737

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01307ca29530c2a1a3
Upwork Job ID:<PHONE_NUMBER>188379648
Last Price Increase: 2023-11-30

@tgolen 👋 , I'm guessing that for the Auth part we want to reuse this MoneyRequest class and add the tag when we load the transaction here:
https://github.com/Expensify/Auth/blob/0c3ae5c2ee23daf8aada8a4c4c13eba896544a75/auth/lib/MoneyRequest.cpp#L30-L50
does that sound correct?

Made the Web-E and Auth PR ready for review. I'm going to work on finishing the App PR now

@aldo-expensify Can you help with BZ Checklist here?
Also can you assign a BZ here?

Added Bug to get a BugZero member

Not overdue, payment period still open

@Santhosh-Sellavel I have run through the payment checklist https://github.com/Expensify/App/issues/32312#issuecomment-1896237423. You can request via ND.

Requested on ND

$500 approved for @Santhosh-Sellavel based on this comment.


Download only latest?

Is it possible to only download the latest invoice? (
E.g. a o2 cronjob, running every month.

There is no way to get the latest document per se but there are a few possible approaches to achieve that:
For o2, the latest document should always be the first one (id = 0), so
$ document-dl --action download --jq 'contains({id: 0})' o2

will download the latest invoice. But this behaviour might change in the future and other plugins might behave differently as well, so you might not detect when it breaks if you leave the cronjob completely unsupervised.
If you know the date of the previous invoice, you could also filter for all documents newer than that.
E.g. if the current invoice has date = 2021-07-01, then outputting any invoice newer than that will yield the latest one:
$ document-dl --action download --jq 'select(.date > "2021-07-01T")' --jq 'contains({"doctype":"BILL"})' o2

Some plugins also add the unread attribute which can be used to filter new documents.


ability for offline usage

Is your feature request related to a problem? Please describe.
use habra without internet connection
Describe the solution you'd like

ability for save articles for offline reading
setting that will automatically save all articles loaded in the menu for offline reading

use habra without internet connection

Can't you already do that? Check if Service worker is installed in the page (F12 -> Application -> Service Workers). The SW caches online requests, so every article you've loaded in a near past will be available offline.

setting that will automatically save all articles loaded in the menu for offline reading

This is possible to do, although, it would be very intensive on network and app can freeze for some time.


Interpreter Source code

I saw your blog post which I loved, I am also trying to create my PL in python/js, I have seen your lexer and parser source code, but where is the interpreters own.

I believe most of the code for running programs is here: https://github.com/wmww/Pinecone/tree/master/src/Actions, but it's been way to long for me to really remember.

Ok tnx


(1) Include boost/bind.hpp in samples with boost::bind (2) Fixed mujindownloaddebug sample (3) updated taskparameter example

background: https://github.com/mujin/controllerclientcpp/commit/69db0fe7e5a32d5a3a9cee0038865feee187ce59 removed boost/property_tree/ptree.hpp from mujincontrollerclient.h, which caused removing implicit including boost/bind.hpp.
impact: mujindownloaddebug.cpp cannot be compiled.
note: there are other samples using boost::bind, which are safe because of binpickingtask.h's including boost/thread.hpp implicit including boost/bind.hpp. I added including bind.hpp for those files as well.

@rdiankov updated taskparameter example as well because robots argument is deprecated

thanks~


fix(charts,logging-operator): add empty workloadMetaOverrides field i…

…n hosttailer if not specified
The '.spec.workloadMetaOverrides' field is required for all hosttailer resources. Unfortunately if not set the deployment via the chart fails with 'is invalid: spec.workloadMetaOverrides: Required value'. In order to mitigate this, an empty object is injected if not specified.

@csatib02 please don't merge this. Even passing an empty object isn't allowed, too. I think this might need to be fixed in the operator and relaxing the crd.

@csatib02 I created a bug instead, sorry for the turn around.


Vulnerability roundup 97: percona-server-5.6.49-89.0: 1 advisory [9.8]

search, files

[ ] CVE-2020-26542 CVSSv3=9.8 (nixos-20.09, nixos-unstable)

Scanned versions: nixos-20.09: 50c8d1e0deb; nixos-unstable: 83cbad92d73.
Cc @grahamc

The CVE is about the MongoDB Simple LDAP plugin.


Assertion in tutorial code incorrect

In Line 100 we have an assertion that doesn't look right to me:
# Let's create a list of all submodels, to which the AAS has references, by resolving each of the submodel references:
submodels = [reference.resolve(obj_store)
             for reference in aas.submodel]

# The first (and only) element of this list should be our example submodel:
assert submodel is tmp_submodel

That assertion is identical to line 87.
From the comment, the assertion should be
# The first (and only) element of this list should be our example submodel:
assert submodels[0] is tmp_submodel

Good catch! I created #187 to fix this.


[Event Change] codeunit 12 "Gen. Jnl.-Post Line"

Please change OnBeforeCreateGLEntriesForTotalAmountsUnapplyV19 event as below :
    [IntegrationEvent(true, false)]
    local procedure OnBeforeCreateGLEntriesForTotalAmountsUnapplyV19(DetailedCustLedgEntry: Record "Detailed Cust. Ledg. Entry"; var CustomerPostingGroup: Record "Customer Posting Group"; GenJournalLine: Record "Gen. Journal Line"; var TempIDimPostingBuffer: Record "Dimension Posting Buffer" temporary; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Specify encoding (utf-8) in importer.py

This should be done as otherwise I get UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 883: ordinal not in range(128) on my machine. I think "utf-8" is the sane setting.

That seems reasonable. Thank you for the contribution!


Move type annotations from methods and fields closer to the types

What problem are you trying to solve?
I don't like how OpenRewrite moved @Nullable from type to a method.
-  @Nullable String getSlotName();
+  @Nullable
+  @Override
+  String getSlotName();

Describe the solution you'd like
@Nullable is a type annotation, so it applies to String type rather than getSlotName() method.
Since OpenRewrite knows the exact definition of the annotation, it should be able to infer that @Nullable is a type annotation while @Override is not.
So I would like the formatting to be
  @Override
  @Nullable String getSlotName();

The emerging jspecify uses only TYPE_USE, see https://github.com/jspecify/jspecify/blob/ee91db649ce2de81d39b55b52345730b3f0f2531/src/main/java/org/jspecify/annotations/Nullable.java#L142, so users should prefer
    @Nullable String newValue;

rather than
    @Nullable
    String newValue;

At the moment I am not even sure that OpenRewrite knows the full definition of annotation types. While we have type attribution for annotations (like @Nullable) and the type attribution also lists the annotations present on that annotation type (e.g. @Target), we currently unfortunately don't include the annotation parameter values in the type attribution. So we don't know if @Target includes TYPE_USE or not. At parse time of the source this is available, but in the LST (specifically the type attribution) this information is lost.
We have an open issue for this here: #3504. Having a concrete use case for this enhancement is really useful. Should we add that as a comment to the mentioned issue and close this one?

Deduplicating issues sounds reasonable

Thanks! Copied the above there just now

https://github.com/openrewrite/rewrite/issues/3504#issuecomment-2104129320


Add some options for mapping POCO fields into a LiquidHash via .ToLiquid()

Right now it calls .ToLower on everything.

Where is this one up to?
We have a branch that we're using https://github.com/mikebridge/Liquid.NET/compare/master...schotime:case_insensitive_liquid_hash that just does the hash, but it obviously doesn't handle the local variables.


Overview issue [Linghang]

Introduction
My name is Linghang and I'm from the Department of Biotechnology. I like music and sports in my spare time.

Describe your research in 2-3 sentences to someone that is not from your field (please avoid abbreviations)
My research focuses on engineering microorganisms to purify wastewater. More specifically, I am researching an innovative biotechnology developed in TU Delft, and trying to optimize the process control to achieve better wastewater treatment.

My research entails the following aspects:

Research Aspect
Answer

Use/collect personal data (health data, interviews, surveys)
No

Use/collect experimental data (lab experiments, measurements with instruments)
Yes

Collaborate with industry
No

Write/develop software as the main output of the project
No

Use code (as in programming) for data analysis
No

Work with large data (images, simulation models)
No

Other:
N/A

Reflections on the importance of RDM videos
The examples of losing data given in the video are really scary, but they are possible to happen in our lives. These examples make me realize the importance of RDM skill better. Luckily, I don't have a horror story in my past research. But I did find it difficult to find the pictures I needed when I was writing, which can be solved with better RDM skill.

What would you like to learn during this course?
An organized way to back up and synchronize data. More specifically, a better way to organize different types of data (excel, word, pictures).

Checklist assignments

[x] Assignment 1: creating a GitHub issue (the overview issue) (before 30 April, 10:00)
[x] Respond to the GitHub issue #125
[x] Assignment 2: Data Flow Map 1 (share a link in this issue before 7 May 13:00). Link: https://surfdrive.surf.nl/files/index.php/s/Va1VwmGf9tlCLEi
[x] Provide feedback to at least one Assignment 2 from another participant
[ ] Respond to the GitHub discussion #126
[ ] Respond to the GitHub discussion #127
[ ] Assignment 3: Data Flow Map 2 (share a link in this issue before 21 May 13:00). Link:
[ ] Provide feedback to at least one Assignment 3 from another participant
[ ] Assignment 4: Data Management Plan (before 29 May 13:00)
[ ] Respond to the GitHub discussion #128 if you have any questions (optional)
[ ] Assignment 5: Data Flow Map 3: submit your slide (before 30 May, 17:00)

My assignment 2 link: https://surfdrive.surf.nl/files/index.php/s/Va1VwmGf9tlCLEi

Thanks for sharing assignment 2 @linghangdelft! It looks very clear and comprehensive, well done!
I have two questions:

Do you also have a back up of your written lab notes?
In Q3 you mention that you'll back up the project drive on a monthly basis - where is this back up stored?
Other people have also included things such as literature or presentations in their assignment - you don't have to do that of course, but a back up of these files can also be crucial!

Thanks for sharing assignment 2 @linghangdelft! It looks very clear and comprehensive, well done!
I have two questions:

Do you also have a back up of your written lab notes?
In Q3 you mention that you'll back up the project drive on a monthly basis - where is this back up stored?
Other people have also included things such as literature or presentations in their assignment - you don't have to do that of course, but a back up of these files can also be crucial!

Thanks for bringing these questions to discuss. Backup of written notes is indeed an issue I don't have clear idea about. But I have tried to convert those important data and notes to my Excel, which will help a bit. For backup of project data, maybe another drive like SURF drive will help. Literature and presentation are indeed important, I will store them together with my lab data in the project drive.

Hi @linghangdelft ! I don't see a link or comment with your Assignment 3. Did something go wrong in updating the issue? Could you please still share this as soon as possible? Thank you!

Hi @linghangdelft ! I don't see a link or comment with your Assignment 3. Did something go wrong in updating the issue? Could you please still share this as soon as possible? Thank you!

Hi Esther, sorry, I was busy recently so I didn't manage to finish it. I will try my best to finish the assignment these two days

Thanks for sharing @linghangdelft! I'll get back with feedback on Monday!
In the meantime I might suggest using SURFdrive/OneDrive as your active storage location and from there maintaining a backup to the project drive - SURF/OneDrive are a bit more userfriendly and you can work on files offline as well, which is more complicated with the project drive. It is up to you of course!

And as promised, some feedback on assignment 3 @linghangdelft! It looks very clear and comprehensive, well done!
I therefore have little feedback, mostly the question on why you consider your data to be suitable for restricted access? It looks like you're not working with any confidential data so that would mean that any results could be made openly available.  Note that this is one of the requirements before you defend your thesis: to make the code/data underlying the publications and thesis chapters available according to the FAIR principles!

And as promised, some feedback on assignment 3 @linghangdelft! It looks very clear and comprehensive, well done!
I therefore have little feedback, mostly the question on why you consider your data to be suitable for restricted access? It looks like you're not working with any confidential data so that would mean that any results could be made openly available. Note that this is one of the requirements before you defend your thesis: to make the code/data underlying the publications and thesis chapters available according to the FAIR principles!

Thanks, Esther for checking. I think it's because in my field, the data included in published papers is not much different from the original data, but in a more organized way. Therefore, showing the complete data doesn't give more information but might cause more confusion.
Usually, the data amount in my field is small but with a lot of context, so making every detail open is not very helpful but can cost much more time if the data need to be very clear. A common way in our field would be adding an appendix to include extra data, if someone really wants some specific detailed data they can contact the author.
But after taking this course, I will think about how to make my data more open.

Thanks for sharing the assignment, @linghangdelft ! Very clear!


Phaser.Physics.Arcade.Sprite.setSize in Typescript definition cannot set center

Version

Phaser Version: ^3.19.0
Operating system: macOS

Description
In Typescript definition, setSize in types/phaser.d.ts line 57389 define like this:
/**
 * Sets the internal size of this Game Object, as used for frame or physics body creation.
 *
 * This will not change the size that the Game Object is rendered in-game.
 * For that you need to either set the scale of the Game Object (`setScale`) or call the
 * `setDisplaySize` method, which is the same thing as changing the scale but allows you
 * to do so by giving pixel values.
 *
 * If you have enabled this Game Object for input, changing the size will _not_ change the
 * size of the hit area. To do this you should adjust the `input.hitArea` object directly.
 * @param width The width of this Game Object.
 * @param height The height of this Game Object.
 */
setSize(width: number, height: number): this;

As I understand, setSize method come from Size component. That it's this. So that's means setSize in class Sprite should be like this:
setSize(width: number, height: number, center?: boolean)

Additional Information
I'm not sure I understand correctly. If I understand it wrong, please suggest to me. :)

For Arcade Game Objects, Physics.Arcade.Components.Size#setSize() overrides GameObjects.Components.Size#setSize(), but the two methods are actually unrelated (and have different signatures). It is confusing. 🙁
https://github.com/photonstorm/phaser/issues/4213#issuecomment-455898802

@samme Thanks for your clarify. I got really confusing too.

This function has been deprecated, replaced with setBodySize in 3.24.

@photonstorm Thanks for this fixed. :)


Glitchy behavior when animating fitBounds multiple times

Demo: 0.7.3, master.
Triggering code:
map.fitBounds([[-20, -20], [20, 20]]);
map.fitBounds([[-10, -10], [10, 10]]);

Expected behaviour: final state of the map reflects the last call to fitBounds.
Observed behaviour: final state of the map might reflect either of the fitBounds calls, depending on the initial map state.
The effect is reproducible as long as the second fitBounds is called before the first has finished animating, eg:
map.fitBounds([[-20, -20], [20, 20]]);
setTimeout(function() {
  map.fitBounds([[-10, -10], [10, 10]]);
}, 200);

or a user clicking two buttons in quick succession will also show the effect.
Disabling animation on the fitBounds calls prevents the issue, but... the zooms are so pretty! :)
I don't have a simple testcase for it, but in my project, two quick fitBounds calls can also leave the map in weirder states (zoomed out very far) that don't reflect any of the fitBounds calls.
Edit: the issue may not affect other animated view changes. tested and working as expected:

setView
panTo

@uniphil hmm, so this is triggered only when both fitBounds calls have bounds with the same center, right?

@mourner thanks for the quick response!
No, they can have different centres (the linked demos have different centres). However, it's only glitchy if a zoom occurs. changing the demo so that fitBounds only causes a pan produces the expected behavior though:
map.fitBounds([[-20, -20], [20, 20]]);
map.fitBounds([[0, 0], [40, 40]]);

so it seems to be something about the zoom, specifically.

I'm just wondering why it's specifically limited to fitBounds when it just calls setView under the hood...

ah, yeah, probably because I didn't change the zoom level in my setView test :)

glitchy setView

@perliedman I wonder why this reappeared after #2021...

@mourner while they look similar, do you think it's really the same issue? #2021 occured because the transition end event was never fired unless requestAnimFrame was called, causing the map to appear locked up, since drag handlers etc. are disabled during zoom (just recalling from memory, so sorry if some details are incorrect).
With this issue, the map appears to continue to work as expected, but the location and zoom is not (always) the expected. So it has the looks of a race condition, but possibly some other sort of race condition.
Anyway., I'll try to look in to it a bit.

Some notes after inspecting this problem:

The same problem is reproducible with Leaflet 0.6.0, so it's been there for a long time
When animation is enabled, the map's zoom (the value returned by getZoom) isn't updated until the zoom animation finishes
If a zoom animation should be triggered or not is determined by comparing the target zoom level with the current zoom level
If we're at zoom level a, two consecutive calls to change zoom levels, one to a, and one to b, will always initiate a zoom to b, no matter the order of the calls, since the change to zoom b will initiate an animated zoom, setting the zoom to b when it's finished, while the zoom to a will not initiate an animated zoom, since the map is already at this level

I think the last point explains the behavior we're seeing.
I guess the solution is to somehow abort a requested zoom animation if a new call comes in that sets the zoom level to the current zoom level, before the animation has finished (or, as in this case, in fact before it's even started because of the delay by requestAnimFrame).
Hope this explanation makes some sense.

I've seen this bug manifest as well. Some observations, if it helps:

Turning off animation fixes the issue.

To add something new: besides markers, I'm also drawing a polyline on the map, all with GeoJSON. When calling setView with animation, the polyline ends up "stuck" in the old zoom, and often times shifted over.
This also is resolved without animation.

Also seeing this, I used animate: false and it's now ok.
Seems like this will be fixed in 1.1?

Before discovering this issue I created a test case which I wanted to use to submit a new issue.
Adding it here instead http://playground-leaflet.rhcloud.com/recu/edit?html,output

I may be seeing something related: fitBounds will animate only if there's no zoom involved.
I first use fitBounds to set the map's bounds. Then if I pan and run that same fitBounds function, animate: true works and leaflet honors the duration value. If I just zoom, or add a zoom to the pan, fitBounds results in a very fast move to the bounding area, which will appear to be a simple switch to it, unless the change from the bounds area is fairly small, in which case you can see the move take place very quickly. (I assume this is because the tiles are cached or something, so leaflet can play them back as the quick move takes place.)

I'm still seeing my issue with 1.2.0.

@Jmuccigr that is the expected behaviour. fitBounds will only animate if the point is close on the same zoom level (https://github.com/Leaflet/Leaflet/blob/master/src/map/Map.js#L1520) or the zoom level is within a threshold (https://github.com/Leaflet/Leaflet/blob/master/src/map/Map.js#L1575); also see the option [zoomAnimationThreshold] to control this.
If you always want an animation, you can use flyToBounds instead.

The original issue is still reproducible in @theashyster's playground example. I think my explanation here is still a valid description of why this happens, but we need to figure out how.

@perliedman, thanks. I hadn't realized that. Is this something that might usefully go into the docs?

This is still open. Just a suggestion but can we have fitBounds and setZoom return a promise or callback when the animation is complete?

@atlantageek I guess we theoretically could but

we don't use a lot of callbacks for async stuff in other places, so it is sort of against the Leaflet style
it would not fix this issue (as far as I understand)

If you really would like something like that, I suggest opening a separate issue with this suggestion. I do not want to sound harsh, but I would say it is not likely to be implemented unless you can argue for it and are prepared to implement it; Leaflet is quite mature and changing core APIs and/or patterns is not really on any maintainers todo at the moment.

This bug is not resolved yet. The only solution, which worked for me, was to disable the animation.

Hello @bygiro ,
I have the same case, please could you tell me exactly how you solved it?
var map = L.map('map), { center: L.latLng(LATITUD_DEFAULT, LONGITUD_DEFAULT), zoomControl: false, minZoom: 4.50, animate: false }
My map sometimes appears centered on a place where I am not setting the center.

Ok so just looking into an issue on my project and I believe this is linked.  Might get into it and hack a fix if nobody has yet...
move then zoom vector animation is broken (order matter it seems)
this is reproducible here
https://github.com/Leaflet/Leaflet/issues/3249
https://leafletjs.com/examples/geojson/
Steps to reproduce:
Mouse down and start panning
Then while panning  zoom with scroll wheel (or second touch)
This is not so noticeable on desktop but on mobile
Touch down start panning first then touch with second finger and zoom
this happens quite often win users are just moving around with 2 fingers an is more prominent with continuous zoom.
Looks like something is not lining up.  @bygiro's solution does not work on mobile  bot does mitigate desktop all be it at a cost.
The fact that it works properly on mobile if you start to zoom first and ten pan might hint at some solutions
Idea time:
Have the pan action with one finger Emulate 2 touches moving in parallel
Fix the second touch event to properly update the initial position by ending the pan before starting.
Anyone whos been in the guts have inputs? does this sound like its on the right track if so let me know.

Ok so I did some hacking this seems to do the trick  I'm not sure how reliable it is.  The animation still messes up a little bit on mobile but its is more usable and on desktop it works perfect with scroll.
This fixes the above problem by ending the drag pan action before leaflet does the two finger drag or scroll zoom.

let mapDiv = document.getElementById('map')
L.DomEvent.on(mapDiv, 'wheel', (e)=>{
    console.log('wheel')
    if(map.dragging.moving()) {//if its moving
        map.dragging._onDragEnd(e); //end the move action before leaflet handles the scroll
    }
});

L.DomEvent.on(mapDiv, 'touchstart', (e)=>{
    //https://github.com/Leaflet/Leaflet/blob/69f73ed1e8a83724c086ce206e7f9d7babc5d3dc/src/map/handler/Map.TouchZoom.js#L42
    if (!e.touches || e.touches.length !== 2 || map._animatingZoom /*|| this._zooming*/) { return; }
    console.log('touchstart')

    if(map.dragging.moving()) {
        map.dragging._onDragEnd(e);
    }
});
//...initialize map after I believe this makes it so that our event handlers get executed first.

Is the core issue here that Leaflet doesn't cancel previous animations when a new one is triggered? I've noticed a situation I think is related, where I am updating the map bounds as several layers each asynchronously load.
Basically something like:
let layer1, layer2;
loadData('layer_one').then((d) => {
  layer1 = addData(d, map);
  updateBounds();
});
loadData('layer_two').then((d) => {
  layer2 = addData(d, map);
  updateBounds();
});
function updateBounds() {
  if (layer1) map.setView(layer1.getBounds().getCenter(), 12);
  else if (layer2) map.fitBounds(layer2.getBounds());
}

Admittedly a little weird logic, but regardless, what I'm seeing is that by default I don't always get the expected boundaries once both layers have loaded. That is, the map should always be zoomed to the layer1 state once both layers load. But if layer2 happens to load first I do not get that result.
If I add {animate:false} to both calls then it works as expected. So without diving into the code, my theory is that if the map.fitBounds(layer2.getBounds()) animation is still in progress when I request map.setView(layer1.getBounds().getCenter(), 12) the earlier animation keeps progressing too, in parallel with and throwing off the end result of the new animation, perhaps?

Is the core issue here that Leaflet doesn't cancel previous animations when a new one is triggered?

The following code seems to indicate that a previous animation is canceled when a new animation start:
https://github.com/Leaflet/Leaflet/blob/bbd264704b05036a281bd44513cf1760d8336b9d/src/map/Map.js#L174-L180
https://github.com/Leaflet/Leaflet/blob/bbd264704b05036a281bd44513cf1760d8336b9d/src/map/Map.js#L354-L361
https://github.com/Leaflet/Leaflet/blob/bbd264704b05036a281bd44513cf1760d8336b9d/src/map/Map.js#L1259-L1265

I'm running into this or a similar issue when using Map.flyTo/Map.flyToBounds. I have a page where the main content is on the left side of the screen, while the right side shows a map. Depending on where what's in the viewport in the left side, I use flyTo to highlight the relevant area on the map. This works great when browsing the page "slowly", but if you scroll the page fast (e.g. to get to the top of the page) it'll trigger multiple consecutive calls to flyTo.
I've found two workarounds that both fixes the glitchy behaviour:

Calling map.fire('viewreset') just before calling flyTo.
Setting zoomSnap to 0 and calling map.stop() just before calling flyTo.

So adding a line that fires the viewreset event to either Map.flyTo or Map._stop should fix the issue, I believe. However, I'm not familiar enough with the internal Leaflet code to know if that would cause any other issues?

Hello. The fix on mine was to call fitBounds() after the map has been rendered on the page by using a set timeout function for 200ms and turning animations off. My implementation here is a snipper of code from a vueJS application so my bounds are saved in a ref.
     // used to reshow the hidden leaflet map after performing a request.
      pricesLoading.value = false;
      loadingPricesMap.value = false //hide the spinner
      showPricesMap.value = true //show the map

      setTimeout(() => {
        dynamicMap2.value.invalidateSize(); //make sure the map resizes correctly
      }, 100);

      setTimeout(() => {
        if (hadSetBounds.value) {
          dynamicMap2.value.fitBounds(bounds.value, {padding: [60, 60]});
          hadSetBounds.value = false;
          bounds.value = null;
        }
      }, 200);
    });


Emit DECRST 1 sequence to restore keyboard mode on Unix systems on pwsh exit

PR Summary
Dotnet runtime sets keyboard mode to application on Unix systems and does not currently reset on exit.  This change emits the DECRTS 1 sequence on pwsh exit on Unix systems to put keyboard mode back to default so that arrow keys do not emit escape sequences.
PR Context
Fix https://github.com/PowerShell/PowerShell/issues/12268
PR Checklist

[ ] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[ ] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

It is not full fix for #12268
We need to address external applications run. See https://github.com/PowerShell/PowerShell/issues/12268#issuecomment-761724502

I'll update to set and reset in the native command processor

Start-Process?

@iSazonov it seems that if we follow zsh, then the set/reset should be in console host repl loop and not in native command processor nor start-process.

Why do we only Unix fix if the issue was reported as Windows bug?
I still think we could safely skip s_theConsoleHost.IsInteractive check.

@iSazonov it's not a Windows bug, it was WSL.  .NET on Windows doesn't use ANSI escape sequences for keyboard input.  As for why check for interactive, the problem is that if you do something like: pwsh -command 1, then you get back 1ESC[?1l as the output from that command.  Even if you don't explicitly use -noninteractive the console host knows that -command without -noexit is non-interactive so we don't want to emit the ESC sequence that will be part of the stdout output.


[BUG] GPU degreees function does not overflow

Describe the bug
On very large numbers GpuToDegrees will not overflow and produce inf when the GPU version does.  This appears to be an order of operations issues where the CPU version multiplies by 180 and then divides by pi, while we for performance reasons multiply by (108/pi).  It should be a minor fix to make this work.
A python test is being added to reproduce this issue.

The toDegrees in spark is a direct call to the toDegrees in the java math library, but since JDK 9 the java implementation has changed to the same order of calculation as the current plugin (from angrad * 180.0 / PI to angrad * (180.0 / PI)), So the results of spark will differ depending on the JDK version in the runtime environment.
Do we need to add support for higher versions of the JDK to ToDegrees to match spark's output? It can perhaps be implemented by determining the Java version when executing the function.
Some tests: current code is XFAIL on JDK8 and XPASS on JDK11, and following code in this PR is PASS on JDK8 and FAIL on JDK11 for case test_degrees.
case class GpuToDegrees(child: Expression) extends GpuUnaryMathExpression("DEGREES") {

  override def doColumnar(input: GpuColumnVector): ColumnVector = {
    val tmp = withResource(Scalar.fromDouble(180d)) { multiplier =>
      input.getBase.mul(multiplier)
    }
    withResource(tmp) { _ =>
      withResource(Scalar.fromDouble(Math.PI)) { pi =>
        tmp.div(pi)
      }
    }
  }
}

If the JDK itself decided that our implementation is better, then I am inclined to just keep the current one and add docs explaining the difference. Then we can have tests for an exact answer that are skipped if the JDK version is JDK 8 or below.
@sameerz do you have any objection to this?


Add an id property to all test cases

Problem: Tests in the suite are occasionally moved around in the file, or their descriptions are clarified, or they are modified because they have a bug. Given that the intended use of the test suite is that downstream users -- (all implementations, Bowtie, other research use cases) -- be able to run the tests, they do not have a way to "track" a test over time. Specifically, say an implementation has a known issue with a specific test for $dynamicRef -- a test case called $dynamicRef flubs the bub when you bla bla. An implementer who wishes to mark this test as a TODO in their test suite has no reliable way to do so over the long term, because it's name, position within the file, or even file itself may change here upstream for various reasons.
We should therefore introduce a stable ID which we guarantee will never change for a test, even if its description changes, or if the test is moved elsewhere within the file or suite.
A "slug" is likely the simplest choice structure-wise, in that it's derivable from the existing test descriptions. UUIDs or globally incrementing integers are other options (see below for considerations).
This addition should also include a sanity check ensuring that the IDs have the properties we wish of them (i.e. uniqueness, see below).
For context, we had previously added this exact concept to the test case schema in a PR #53 but did not ever add it to tests
Considerations:

The IDs should likely be version-wide unique -- i.e. not simply within one file, though also not necessarily across the whole test suite. This is to ensure we can move tests between files if we ever recategorize them (c.f. e.g. #630 which we ultimately haven't done, but we have in other cases) This would mean a true "global" ID for a test would be the composite ID (version, ID).
We should spend a moment considering whether IDs should be historically unique as well -- i.e. if we delete a test for whatever reason, not reuse the ID we gave it.
If a test was incorrect, we should likely change it's ID, given that anyone referencing it has possibly skipped it because they indeed had the correct behavior -- but this may need a bit more thought.
We may need to be more precise in the future in cases where we modify a test in deciding whether the new test is the "same" and therefore should keep its ID or "new" in which case it should get a new one. In general we should err heavily towards never modifying tests in any non-trivial way, so this hopefully will never come up "in practice" but it undoubtedly has once or twice previously.
We need to consider what to do about tests themselves (rather than test cases) -- one would suspect we could rely on the ordering of the test array as IDs for tests, but again, this means we are not able to reliably reorder tests within a test case without disrupting downstream or forcing reliance on their test names. It seems initially then like we should include IDs for tests as well.
The test case schema will need updating to document and allow this new property. We should initially make the property optional until all tests in the suite have it, at which point we can then require the property.
We should probably write the above considerations down in the README when merging a PR implementing this.
Please do not send a giant PR which does this for all files all at once. Send small PRs, perhaps one file at a time, or even less, to help reviewers be able to review the PR reasonably.

If you'd like to assign me to this, I'll take it on.
I propose the following in response to this:

Use UUIDs for the IDs, and make them globally unique, rather than just unique within the version of the spec.
Using Array index for the tests themselves is extremely brittle - people keep arbitrarily adding tests in the middle of the lists, and it is difficult to enforce. I propose using a UUID for each test within the test suite.
I will write a tool to update the IDs for everything and land a complete update all in one go
If a test changes in any way, I propose that its ID should be changed, and we should document that in the README
Note that with repsect to (4) a test that relies on the metaschema should not be considered to be changed if the metaschema changes.

I originally presumed we would have some sort of human readable ID. UUIDs would be simpler in some respects.
I guess it depends on the intended use case.
If someone has a list of tests they skip, having human readable names which partially suggest the sort of test in question, it could be helpful. However, this wouldn't be the best if we agree that IDs should change if the test changes (which I think is a good idea).
In that case, I wonder if a simpler approach is to just MD5 the test object... although I suspect because of JSON and object ordering, that might not end up the same depending on various things.
All that to say, are we convinced UUIDs as IDs would be the best here?

I did consider an MD5 hash of the test object (and the schema in the case of the test case) but that makes it rather harder to generate while developing the tests, and feels like a barrier to entry.

I don't really have any strong preference on the structure of the IDs -- I think I mentioned I think slugs are better because of the extra benefit (of being user readable and usable for other reasons in isolation) but if I'm not doing the work I definitely don't care.
The only thing I do care about is:

I will write a tool to update the IDs for everything and land a complete update all in one go

I don't personally want to (ever) review large automated PRs (and have said elsewhere that as far as I have seen in "real life" it's simply impossible for any human to do.) -- meaning I'm happy to review the script you write to do this, or else for someone else who thinks it's possible to do the review -- though experience in this repo itself has reaffirmed as far as I'm concerned that this is the case :) -- the last two times I said this (no large automated PRs) and others reviewed it, the PRs indeed introduced subtle test bugs that were fixed over a few months afterwards (not that it took months to fix, it took months to notice, which is often the problem).
So yeah, happy to review the script, or else to say "find any other test suite contributor willing to review the PR".
Otherwise I'm fine with UUIDs if you prefer.


Thorin-Oakenpants you become too soft.

Sorry dude, but I have to say it. Since the beginning of this year, I began to notice that you are remove (disabling) some important pref. I think it's all due to the fact that you're trying to please both the one and the other. But you can not play on both sides, otherwise both will suffer. Hardening (privacy and security) should be for you in the first place. You become too soft. In the end, you will ruin everything for what it was created.

@Angelus35, can you list those "some important pref" that were disabled?

Please remember that this is a "medium" (but I always try to err on the side of security, privacy, anti-FP in that order) TEMPLATE. It's actually almost a pretty damn hardened version (I do not see it as middle-of-the-road at all). Read around the comments in places like reddit FWIW, or use google, and loads of people saying this is too hard, breaks too much stuff. People moaning about TP/SB being turned off (in favor of uBo), people complaining that they suddenly can't log in on sites etc.  However, that is irrelevant when you consider than since day one it can been called a template.
Consider that this "medium" branch (which takes some site breakage into account), is actually, in many ways more hardened than so called hardened versions. Example, cookies denied out of the box. PB mode not used out of the box, which enables users to flick open one off PB mode windows - which is actually even more private than starting in PB mode.
It is a TEMPLATE. As said in the readme (future branches), and in numerous comments here, and on ghacks, we plan on released a "lite/relaxed" version as well as an uber-hardened version.
There are 454 prefs (excluding deprecated, personal section & parrot prefs) in 291 numbered items - getting these so everyone is happy is impossible in this branch. This is subjective, it's not a case black and white. Even if it were, there are still trade-offs to be judged.
Part of the reason for this project is to educate and make it easier for people to get into using this or similar setups. I can easily make this HARD, but then no-one would use it.
If you have an issue about any particular prefs being dropped or made inactive, then by all means speak up. Create an issue such as "What did you disable pref xxx.yyy.zzz?" instead of insulting me. I'd rather you were part of a solution, than a problem.

@Thorin-Oakenpants, please, don't trigger to provocations as this one. It takes energy, which could be spent somewhere else.
@Angelus35 took one minute to freshly create account at github with a single post almost obvious targeted to insult you without any grounds,
I was asking a guy with a one-liner question, to which I am 99% sure, he/she doesn't have an answer, to avoid your time being spend wrongly. ;)

Reply was the for benefit of all readers. Am tempted to mark this as invalid and close it, but, alas, I am not the dark insect overlord I always wanted to be ... it's true, I am soft :cry: :princess:
I promise to be a total bitch in future :fist_oncoming:


remove absolute urls

No worries @rpetrusha. I've also filed a bug for the validation app because it's not working on PRs from forks and also doesn't do a good job on the update to live PR.


Lay out issues with some pages in the Submission workflow

if you go through the submission workflow, you'd note the UI elements in Compliance and Repositories page are squished up together with no vertical spaces between elements.

![screen shot 2017-12-11 at 10 21 11 am](https://user-images.githubusercontent.com/11050142/33843070-5142aa94-de6a-11e7-9576-394fc772d97f.png)

Resolved with PR https://github.com/DataConservancy/pass-ember/pull/81


ReferenceError: Headers is not defined

Hello there,
I'm trying to use this package in one of my projects which is built-in AdonisJS.
When I simply try to list all the invoices await DocumentService.listDocument().then(res => console.log({res}))
It always throws this error:
ReferenceError: Headers is not defined
    at /.../node_modules/@codingsans/billingo-client/lib/client/core/request.js:156:5
    at Generator.next (<anonymous>)
    at fulfilled (/.../node_modules/@codingsans/billingo-client/lib/client/core/request.js:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Same with all the others like:
await PartnerService.createPartner({})
What could be the problem?

Thanks for reporting the issue. It's fixed in version 2.1.0


Wrong window fullscreen positioning with "window.titleBarStyle": "custom"

Issue Type: Bug
When "window.titleBarStyle" is set to "custom" and I activate fullscreen by double-clicking on the top bar, the window is shifted 15px or so down (windows height is correct so 15px are out of view on bottom).
Same issue happens when i move the window from one screen to another while using a dual monitor setup.
VS Code version: Code - Insiders 1.25.0-insider (0f080e5267e829de46638128001aeb7ca2d6d50e, 2018-07-04T14:17:16.571Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.79GB (9.10GB free)

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

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

Ionide-fsharp
Ion
3.22.1

svn-scm
joh
1.29.6

It actually happens once every two times I maximize the window no matter how (moving from one screen to another / double clicking on bar / clicking the maximize button on bar).

This is not a VSCode issue but rather a Windows 10 issue.
The same issue happens to Discord, the newer version of Chrome, Slack I believe (don't quote me on that one), etc.
I'll find a link talking about this exact problem and edit this comment shortly. Probably a screenshot would help too.
A temporary fix for everyone experiencing this issue : once you have maximized VSCode and get this issue, drag vscode so it becomes a windowed app and put it back in full screen.
As @XandruDavid already said, it happens once every two times, so you'll have to do this fix again. (I'm explaining it again just to make this temp fix clear, @XandruDavid ;) ).
Hopefully Windows will get this fixed soon, this problem affects me often times too and it's very frustrating!

From a comment on this reddit thread :

[...] I fixed the issue for discord by activating the windows 8 compatibility mode

@XandruDavid (and everyone having this issue) : Maybe in the meantime Windows fixes this problem (view my previous comment if you don't understand why I talk about Discord and Windows), you could try activating compatibility mode for VSCode. I'm not on my personal computer to try myself unfortunately, will come back here with my personal results once I'll be home.

duplicate of #52265


drop xml-rpc support

(note: this issue is to only be completed after #209 is performed)
After a release containing deprecated marked XML-RPC capabilities (see #209), a subsequent release should remove the various XML-RPC related implementation/support from this extension. This is to help reduce maintenance work for the old control method (see #208).
A quick glace at configuration entries that can be removed include:

confluence_disable_rest
confluence_disable_xmlrpc
confluence_proxy

XML-RPC has been officially dropped from master; v1.2 was the lasted supported revision supporting this feature.


tomtomclub world  stream sport & movie

Addon Manifest URL
https://iptv closed!!
Addon Description
Yeah foks. enjoy this sports movie and series stream
Language of Content
None
Choose Labels

[ ] anime
[ ] asian drama
[ ] bollywood
[ ] cartoons
[ ] debrid support
[ ] http streams
[X] live tv
[ ] metadata
[ ] misc
[ ] movies
[ ] music
[ ] nsfw
[ ] podcasts
[ ] radios
[ ] subtitles
[ ] torrents
[ ] tv shows
[ ] video games

i don't know what this is, but it's not an addon, so i'm closing it


[BUG] - makeTransactionBody duplicates scripts

Internal/External
External
Area
Plutus    Related to Plutus Scripts (Alonzo).
Summary
makeTransactionBody duplicates scripts. This is obviously undesirable due to size and execution unit limits. I have written a version which does not duplicate scripts, but this is not ideal - it has to do a lot of unsafe and gnarly things with Generics to get around the fact that the true constructors for TxBody are not exported (which they should be, please!). Because the constructors aren't exposed, I can't just deduplicate after the fact.
Steps to reproduce
Steps to reproduce the behavior:
I don't have a minimal repro right now. What I would suggest is to create a TxBodyContent with some scripts of a chosen size (say 2000 bytes), spending multiple different inputs from the same script. Call makeTransactionBody, then serialiseToCBOR, then inspect in the CBOR Playground. You should see multiple appearances of the script with the size you chose.
Expected behavior
makeTransactionBody should not duplicate scripts.
System info (please complete the following information):

OS Name: NixOS
OS Version: 21.11
Node version: cardano-node 1.33.0 - linux-x86_64 - ghc-8.10\ngit rev d7427a896fd477706c5803ac422e36d422bf2e98
CLI version: cardano-cli 1.33.0 - linux-x86_64 - ghc-8.10\ngit rev d7427a896fd477706c5803ac422e36d422bf2e98

Screenshots and attachments
Here is the cbor of a transaction with duplicates: https://gist.github.com/chessai/12f9795ab322cc633b1b403995473c55
Note that the scripts are 0'd out. You can load this into the CBOR Playground and you should see that there are two instances of a script with size 2121.
Additional context
I would suggest exporting the constructors of TxBody from an unsafe module, and keeping the (complete) pattern synonym as the default export.

@chessai makeTransactionBody uses an intermediate type for the transaction body, a type which is somewhere between what the ledger code calls a transaction body and a full transaction. And indeed the intermediate type does contain duplicates.  When the CLI creates a full transaction from this intermediate body type, however, it removes the duplication and creates a proper transaction according to the ledger wire specification.

@chessai makeTransactionBody uses an intermediate type for the transaction body, a type which is somewhere between what the ledger code calls a transaction body and a full transaction. And indeed the intermediate type does contain duplicates.  When the CLI creates a full transaction from this intermediate body type, however, it removes the duplication and creates a proper transaction according to the ledger wire specification.

Even if things are fine (i.e. deduplicated) when it comes time to submit a transaction, I want to be able to test my Plutus applications off-chain and make sure that I don't exceed execution units/size such that the test conditions match the real world. Currently I am using a variant of makeTransactionBody to do this. What is the recommended way to do this, if one exists?

I don't honestly know what all tooling exists for testing Plutus scripts off-chain first. Perhaps @Jimbo4350 or @michaelpj knows about something that I am unaware of.
I did write a function in the ledger repo, evaluateTransactionExecutionUnits, which is used by the CLI to count the execution units needed for a given transaction:
https://github.com/input-output-hk/cardano-ledger/blob/f26eea2bcde15fde726d0289f716cc3617715b81/eras/alonzo/impl/src/Cardano/Ledger/Alonzo/Tools.hs#L127-L132
That function is probably not convenient for you, though, given that it needs the protocol parameters, the UTxO (or at least the relevant subset for the transaction), and some time related information.

I don't honestly know what all tooling exists for testing Plutus scripts off-chain first. Perhaps @Jimbo4350 or @michaelpj knows about something that I am unaware of.
I did write a function in the ledger repo, evaluateTransactionExecutionUnits, which is used by the CLI to count the execution units needed for a given transaction:
https://github.com/input-output-hk/cardano-ledger/blob/f26eea2bcde15fde726d0289f716cc3617715b81/eras/alonzo/impl/src/Cardano/Ledger/Alonzo/Tools.hs#L127-L132
That function is probably not convenient for you, though, given that it needs the protocol parameters, the UTxO (or at least the relevant subset for the transaction), and some time related information.

I have a lot of my own tooling for doing this. It seems like a killer feature of Cardano to be able to test off-chain, but this isn't really well-supported by anything open source. I am working toward things of this nature which are general and open source.
For now, does the intermediate representation need those duplicates? Could a fix be as simple as calling some variant of nub? I don't see why not, plus any call to nub will be cheap (at least for now), because tx size/cpu/mem requirements make it so that too many scripts can't exist anyway.


Bug: Updating user information empties password

Describe the bug
If I want to comment or change the role(s) of a user (no matter if it's a root user or a normal database user), the password gets reset. Even though it should not "reset" the password field to an empty string. You can not login with an empty password string and therefore you brick your user.
I was able to detect what was happening via http and sniff the traffic (using Proxyman).

Important to note the PASSWORD "" in the request body.

Root user update request/response:

Normal database user request/response:

To Reproduce
Steps to reproduce the behavior:

Go to 'Authentication'
Click on a 'root' user (which has a password set) or create a new "Database user" with a password.
Verify if login with user, with a separate connection.
Update the user, select a role or add a comment and click on "save"
Try login with root or the database user and it will not be successful because the password was reset.

Expected behavior
The form field states that when leaving empty the password will not be set.
Workaround/Solution to fix
Current workaround I have found is start SurrealDB with the --allow-guests flag and change the password.
Environment:
Platform: Desktop
OS: Darwin
Architecture: x86_64
WebView: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko)
Version: 2.0.6
Flags: featureFlags: false, models_view: true, apidocs_view: true, themes: false, newsfeed: true

Thank you for reporting this issue! This was caused by an oversight on my side, and should be rectified in the next release
My apologies for any inconvenience this may have caused