tesflight command fails reporting that app cannot be found

New Issue Checklist

✅ Updated fastlane to the latest version
✅ I have read the Contribution Guidelines

Issue Description
Attempting to push a gym built .ipa to Testflight by specifying the testflight step in my Fastfile fails with the error:
[!] Couldn't find app 'com.company.AutomatedTestingiOS' on the account of<EMAIL_ADDRESS>on iTunes Connect

The app, however, does exist in my iTunes Connect.
Complete output when running fastlane, including the stack trace and command used
NOTE: I've replaced sensitive information (like emails and last names of people) with placeholder text (e.g., email@company.com). Let me know if this somehow prevents anyone from helping with this issue.
Captured Output
Command Used: beta
Output/Log

[16:45:26]: -------------------------------------------------
[16:45:26]: --- Step: Verifying required fastlane version ---
[16:45:26]: -------------------------------------------------
[16:45:26]: Your fastlane version 1.111.0 matches the minimum requirement of 1.111.0  ✅
[16:45:26]: ------------------------------
[16:45:26]: --- Step: default_platform ---
[16:45:26]: ------------------------------
[16:45:26]: Driving the lane 'ios beta' 🚀
[16:45:26]: -----------------------
[16:45:26]: --- Step: cocoapods ---
[16:45:26]: -----------------------
[16:45:26]: Installing Ruby gem 'cocoapods'...
[16:45:27]: Successfully installed 'cocoapods' to '~/.fastlane/bin/'
[16:45:27]: $ bundle exec pod install
[16:45:29]: ▸ Analyzing dependencies
[16:45:29]: ▸ Downloading dependencies
[16:45:29]: ▸ Using email-Player-SDK (5.2.0)
[16:45:29]: ▸ Generating Pods project
[16:45:29]: ▸ Integrating client project
[16:45:30]: ▸ Sending stats
[16:45:30]: ▸ Pod installation complete! There is 1 dependency from the Podfile and 1 total pod installed.
[16:45:30]: ------------------
[16:45:30]: --- Step: cert ---
[16:45:30]: ------------------

+---------------+-------------------------------------------------+
|                     Summary for cert 1.4.4                      |
+---------------+-------------------------------------------------+
| development   | false                                           |
| force         | false                                           |
| username      |<EMAIL_ADDRESS>                |
| team_id       | RP7C3H2395                                      |
| keychain_path | /Users/nfuller/Library/Keychains/login.keychain |
+---------------+-------------------------------------------------+

[16:45:31]: Starting login with user<EMAIL_ADDRESS>[16:45:32]: Successfully logged in
[16:45:34]: Certificate KR38Z522N4 (Megan Lastname) can't be found on your local computer
[16:45:35]: Certificate BQCNH2PE82 (Megan Lastname) can't be found on your local computer
[16:45:36]: Found the certificate 8N46S8VP24 (Megan Lastname) which is installed on the local machine. Using this one.
[16:45:36]: Verifying the certificate is properly installed locally...
[16:45:36]: Successfully installed certificate 8N46S8VP24
[16:45:36]: Use signing certificate '8N46S8VP24' from now on!
[16:45:36]: ------------------
[16:45:36]: --- Step: sigh ---
[16:45:36]: ------------------

+-------------------------------------+--------------------------------+
|                       Summary for sigh 1.12.0                        |
+-------------------------------------+--------------------------------+
| adhoc                               | false                          |
| development                         | false                          |
| skip_install                        | false                          |
| force                               | false                          |
| app_identifier                      | com.company.AutomatedTestingiOS   |
| username                            |<EMAIL_ADDRESS>|
| team_id                             | RP7C3H2395                     |
| ignore_profiles_with_different_name | false                          |
| cert_id                             | 8N46S8VP24                     |
| skip_fetch_profiles                 | false                          |
| skip_certificate_verification       | false                          |
+-------------------------------------+--------------------------------+

[16:45:36]: Starting login with user<EMAIL_ADDRESS>[16:45:37]: Successfully logged in
[16:45:37]: Fetching profiles...
[16:45:39]: Certificate for Provisioning Profile 'com.company.AutomatedTestingiOS AppStore' not available locally: KR38Z522N4, skipping this one...
[16:45:40]: Found 1 matching profile(s)
[16:45:40]: Downloading provisioning profile...
[16:45:41]: Successfully downloaded provisioning profile...
[16:45:41]: Installing provisioning profile...
/Users/nfuller/Developer/iOS/Automated Testing iOS/AppStore_com.company.AutomatedTestingiOS.mobileprovision
[16:45:41]: Setting Provisioning Profile type to 'app-store'
[16:45:41]: -----------------
[16:45:41]: --- Step: gym ---
[16:45:41]: -----------------
[16:45:41]: Using legacy build system - waiting for radar to be fixed: https://openradar.appspot.com/radar?id=4952000420642816
[16:45:41]: Installing provisioning profile...
[16:45:42]: $ xcodebuild -list -workspace AutomatedTestingiOS.xcworkspace
[16:45:43]: $ xcodebuild clean -showBuildSettings -workspace AutomatedTestingiOS.xcworkspace -scheme AutomatedTestingiOS

+---------------------------+----------------------------------------------------------------------+
|                                      Summary for gym 1.12.1                                      |
+---------------------------+----------------------------------------------------------------------+
| use_legacy_build_api      | true                                                                 |
| scheme                    | AutomatedTestingiOS                                                  |
| workspace                 | AutomatedTestingiOS.xcworkspace                                      |
| provisioning_profile_path | /Users/nfuller/Developer/iOS/Automated Testing iOS/AppStore_com.b... |
| export_method             | app-store                                                            |
| destination               | generic/platform=iOS                                                 |
| output_name               | AutomatedTestingiOS                                                  |
| clean                     | false                                                                |
| output_directory          | .                                                                    |
| silent                    | false                                                                |
| buildlog_path             | ~/Library/Logs/gym                                                   |
| xcode_path                | /Applications/Xcode 8.1.app                                          |
+---------------------------+----------------------------------------------------------------------+

[16:45:44]: $ set -o pipefail && xcodebuild -workspace AutomatedTestingiOS.xcworkspace -scheme AutomatedTestingiOS -destination 'generic/platform=iOS' -archivePath /Users/nfuller/Library/Developer/Xcode/Archives/2016-11-30/AutomatedTestingiOS\ 2016-11-30\ 16.45.44.xcarchive archive | tee /Users/nfuller/Library/Logs/gym/AutomatedTestingiOS-AutomatedTestingiOS.log | xcpretty
[16:45:47]: ▸ Building Pods/Brightcove-Player-SDK-BCOVPUIResources [Release]
[16:45:47]: ▸ Check Dependencies
[16:45:47]: ▸ Processing ResourceBundle-BCOVPUIResources-Info.plist
[16:45:47]: ▸ Copying Brightcove-Player-SDK/ios/dynamic/BrightcovePlayerSDK.framework/bcovpuiiconfont.ttf
[16:45:47]: ▸ Touching BCOVPUIResources.bundle
[16:45:47]: ▸ Building Pods/Pods-AutomatedTestingiOS [Release]
[16:45:47]: ▸ Check Dependencies
[16:45:47]: ▸ Processing Info.plist
[16:45:47]: ▸ Compiling Pods-AutomatedTestingiOS-dummy.m
[16:45:47]: ▸ Compiling Pods-AutomatedTestingiOS-dummy.m
[16:45:47]: ▸ Compiling Pods_AutomatedTestingiOS_vers.c
[16:45:47]: ▸ Compiling Pods_AutomatedTestingiOS_vers.c
[16:45:47]: ▸ Copying Pods-AutomatedTestingiOS-umbrella.h
[16:45:47]: ▸ Touching Pods_AutomatedTestingiOS.framework
[16:45:47]: ▸ Building AutomatedTestingiOS/AutomatedTestingiOS [Release]
[16:45:47]: ▸ Check Dependencies
[16:45:47]: ▸ Running script '[CP] Check Pods Manifest.lock'
[16:45:49]: ▸ Linking AutomatedTestingiOS
[16:45:49]: ▸ Linking AutomatedTestingiOS
[16:45:49]: ▸ Compiling LaunchScreen.storyboard
[16:45:50]: ▸ 2016-11-30 16:45:50.586 xcodebuild[15603:646286] CoreSimulatorService connection interrupted.  Resubscribing to notifications.
[16:45:51]: ▸ Compiling Main.storyboard
[16:45:51]: ▸ Processing Info.plist
[16:45:51]: ▸ Generating 'AutomatedTestingiOS.app.dSYM'
[16:45:51]: ▸ Running script '[CP] Embed Pods Frameworks'
[16:45:52]: ▸ Running script '[CP] Copy Pods Resources'
[16:45:52]: ▸ Touching AutomatedTestingiOS.app
[16:45:54]: ▸ Signing /Users/nfuller/Library/Developer/Xcode/DerivedData/AutomatedTestingiOS-hkqgapbpvelyioanejggouwxrazv/Build/Intermediates/ArchiveIntermediates/AutomatedTestingiOS/InstallationBuildProductsLocation/Applications/AutomatedTestingiOS.app
[16:45:55]: ▸ Touching AutomatedTestingiOS.app.dSYM
[16:45:55]: ▸ Archive Succeeded
[16:45:56]: Legacy build api is enabled, the `export_method` value will be ignored
[16:45:56]: $ /usr/bin/xcrun /tmp/PackageApplication4Gym -v /Users/nfuller/Library/Developer/Xcode/Archives/2016-11-30/AutomatedTestingiOS\ 2016-11-30\ 16.45.44.xcarchive/Products/Applications/AutomatedTestingiOS.app -o '/var/folders/9v/944t6qp92s50d9pbl4m8d86n7d4hl8/T/gym_output20161130-15504-d0pfht/AutomatedTestingiOS.ipa' exportFormat ipa --embed '/Users/nfuller/Developer/iOS/Automated Testing iOS/AppStore_com.company.AutomatedTestingiOS.mobileprovision'
[16:46:00]: $ zip --recurse-paths '/var/folders/9v/944t6qp92s50d9pbl4m8d86n7d4hl8/T/gym_output20161130-15504-d0pfht/AutomatedTestingiOS.ipa' SwiftSupport > /dev/null
[16:46:03]: Compressing 1 dSYM(s)
[16:46:03]: $ cd '/Users/nfuller/Library/Developer/Xcode/Archives/2016-11-30/AutomatedTestingiOS 2016-11-30 16.45.44.xcarchive/dSYMs' && zip -r '/Users/nfuller/Developer/iOS/Automated Testing iOS/AutomatedTestingiOS.app.dSYM.zip' *.dSYM
[16:46:03]: ▸ updating: AutomatedTestingiOS.app.dSYM/ (stored 0%)
[16:46:03]: ▸ updating: AutomatedTestingiOS.app.dSYM/Contents/ (stored 0%)
[16:46:03]: ▸ updating: AutomatedTestingiOS.app.dSYM/Contents/Info.plist (deflated 52%)
[16:46:03]: ▸ updating: AutomatedTestingiOS.app.dSYM/Contents/Resources/ (stored 0%)
[16:46:03]: ▸ updating: AutomatedTestingiOS.app.dSYM/Contents/Resources/DWARF/ (stored 0%)
[16:46:03]: ▸ updating: AutomatedTestingiOS.app.dSYM/Contents/Resources/DWARF/AutomatedTestingiOS (deflated 70%)

[16:46:03]: Successfully exported and compressed dSYM file
[16:46:03]: Successfully exported and signed the ipa file:
[16:46:03]: /Users/nfuller/Developer/iOS/Automated Testing iOS/AutomatedTestingiOS.ipa
[16:46:03]: ------------------------
[16:46:03]: --- Step: testflight ---
[16:46:03]: ------------------------
[16:46:03]: Login to iTunes Connect<EMAIL_ADDRESS>[16:46:04]: Login successful
[16:46:05]: Variable Dump:
[16:46:05]: {:DEFAULT_PLATFORM=>:ios, :PLATFORM_NAME=>:ios, :LANE_NAME=>"ios beta", :CERT_FILE_PATH=>"/Users/nfuller/Developer/iOS/Automated Testing iOS/8N46S8VP24.cer", :CERT_CERTIFICATE_ID=>"8N46S8VP24", :SIGH_PROFILE_PATH=>"/Users/nfuller/Developer/iOS/Automated Testing iOS/AppStore_com.company.AutomatedTestingiOS.mobileprovision", :SIGH_PROFILE_PATHS=>["/Users/nfuller/Developer/iOS/Automated Testing iOS/AppStore_com.company.AutomatedTestingiOS.mobileprovision"], :SIGH_UDID=>"06838ce0-21be-454c-b13d-179af00c3331", :SIGH_UUID=>"06838ce0-21be-454c-b13d-179af00c3331", :SIGH_PROFILE_TYPE=>"app-store", :IPA_OUTPUT_PATH=>"/Users/nfuller/Developer/iOS/Automated Testing iOS/AutomatedTestingiOS.ipa", :DSYM_OUTPUT_PATH=>"/Users/nfuller/Developer/iOS/Automated Testing iOS/AutomatedTestingiOS.app.dSYM.zip", :XCODEBUILD_ARCHIVE=>"/Users/nfuller/Library/Developer/Xcode/Archives/2016-11-30/AutomatedTestingiOS 2016-11-30 16.45.44.xcarchive"}
[16:46:05]: Couldn't find app 'com.company.AutomatedTestingiOS' on the account of<EMAIL_ADDRESS>on iTunes Connect

+------+-------------------------------------+-------------+
|                     fastlane summary                     |
+------+-------------------------------------+-------------+
| Step | Action                              | Time (in s) |
+------+-------------------------------------+-------------+
| 1    | Verifying required fastlane version | 0           |
| 2    | default_platform                    | 0           |
| 3    | cocoapods                           | 4           |
| 4    | cert                                | 6           |
| 5    | sigh                                | 5           |
| 6    | gym                                 | 21          |
| 💥    | testflight                          | 2           |
+------+-------------------------------------+-------------+

[16:46:05]: fastlane finished with errors

[!] Couldn't find app 'com.company.AutomatedTestingiOS' on the account of<EMAIL_ADDRESS>on iTunes Connect

✅ fastlane environment ✅
Stack

Key
Value

OS
10.11.6

Ruby
2.3.0

Bundler?
true

Git
git version 2.9.3 (Apple Git-75)

Installation Source
~/.rvm/rubies/ruby-2.3.0/bin/fastlane

Host
Mac OS X 10.11.6 (15G1004)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.3.0/lib

OpenSSL Version
OpenSSL 1.0.2h  3 May 2016

Is contained
false

Xcode Path
/Applications/Xcode 8.1.app/Contents/Developer/

Xcode Version
8.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
# Customise this file, documentation can be found here:
# https://github.com/fastlane/fastlane/tree/master/fastlane/docs
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "1.111.0"

ENV['FASTLANE_ITC_TEAM_ID'] = '59730801'

default_platform :ios

platform :ios do
  before_all do
    # ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."
    cocoapods

  end

  desc "Creates an app in iTunes Connect"
  lane :create do
      produce(
        username<EMAIL_ADDRESS>        app_identifier: 'com.company.AutomatedTestingiOS',
        app_name: 'AutomatedTestingiOS',
        language: 'English',
        app_version: '1.0',
        sku: 'com.company.AutomatedTestingiOS',
        team_name: 'Oscar Lastname|499325643'
      )
  end

  desc "Runs all the tests"
  lane :test do
     scan(scheme: "AutomatedTestingiOS",
          workspace: "AutomatedTestingiOS.xcworkspace")
  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # match(type: "appstore") # more information: https://codesigning.guide
    cert
    sigh
    gym(use_legacy_build_api: true,
        scheme: "AutomatedTestingiOS",
        workspace: "AutomatedTestingiOS.xcworkspace") # Build your app - more options available
    testflight
    # sh "your_script.sh"
    # You can also use other beta testing services here (run `fastlane actions`)
  end

  desc "Deploy a new version to the App Store"
  lane :release do
    # match(type: "appstore")
    # snapshot
    gym(scheme: "AutomatedTestingiOS") # Build your app - more options available
    deliver(force: true)
    # frameit
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Platforms.md
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md

# fastlane reports which actions are used
# No personal data is recorded. Learn more at https://github.com/fastlane/enhancer

`./fastlane/Appfile`
app_identifier "com.company.AutomatedTestingiOS" # The bundle identifier of your app
apple_id<EMAIL_ADDRESS># Your Apple email address

team_id "RP7C3H2395"  # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Appfile.md

fastlane gems

Gem
Version
Update-Status

credentials_manager
0.16.2
✅ Up-To-Date

fastlane_core
0.57.2
✅ Up-To-Date

spaceship
0.38.4
✅ Up-To-Date

cert
1.4.4
✅ Up-To-Date

deliver
1.16.0
✅ Up-To-Date

frameit
3.0.0
✅ Up-To-Date

gym
1.12.1
✅ Up-To-Date

sigh
1.12.0
✅ Up-To-Date

match
0.11.0
✅ Up-To-Date

pem
1.4.0
✅ Up-To-Date

pilot
1.12.1
✅ Up-To-Date

produce
1.3.2
✅ Up-To-Date

scan
0.14.2
✅ Up-To-Date

screengrab
0.5.6
✅ Up-To-Date

snapshot
1.16.4
✅ Up-To-Date

supply
0.8.0
✅ Up-To-Date

fastlane
1.111.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.0.0

executable-hooks
1.3.2

bundler-unload
1.0.2

rubygems-bundler
1.4.4

bundler
1.13.6

io-console
0.4.5

CFPropertyList
2.3.4

i18n
0.7.0

json
1.8.3

minitest
5.9.1

thread_safe
0.3.5

tzinfo
1.2.2

activesupport
<IP_ADDRESS>

public_suffix
2.0.4

addressable
2.5.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander
4.4.0

security
0.1.3

credentials_manager
0.16.2

excon
0.54.0

gh_inspector
1.0.2

multi_json
1.12.1

plist
3.2.0

rubyzip
1.1.7

terminal-table
1.4.5

fastlane_core
0.57.2

multipart-post
2.0.0

faraday
0.10.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20161129

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.10.1

fastimage
1.6.8

multi_xml
0.5.5

claide
1.0.1

dotenv
2.1.1

mini_magick
4.5.1

rouge
1.11.1

xcpretty
0.2.4

net-ssh
3.2.0

net-sftp
2.1.2

krausefx-shenzhen
0.14.11

slack-notifier
1.5.1

xcpretty-travis-formatter
0.0.4

jwt
1.5.6

little-plugger
1.1.4

logging
2.1.0

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
<IP_ADDRESS>

hurley
0.2

mime-types-data
3.2016.0521

mime-types
3.1

uber
0.0.15

representable
2.3.0

retriable
2.1.0

google-api-client
0.9.20

terminal-notifier
1.7.1

word_wrap
1.0.0

xcode-install
2.0.9

nanaimo
0.2.2

xcodeproj
1.4.1

generated on: 2016-11-30

Additional Details
For what it's worth, it does seem there is an issue with iTunes Connect not displaying newly created apps at the "My Apps" grid. A few people are reporting this issue here.

I removed the cert and sigh steps from my beta lane, and specified the actual .mobileprovision profiles after disabling automatic provisioning in Xcode 8.1, but I am still getting the same problem.

we also expierence the problem of apps not showing up in itc. or atleast taking ages.
see this radar: http://openradar.appspot.com/radar?id=5024333961363456
i think aslong as it does not show up in the dropdown - pilot will fail

Ah perfect. Thanks for linking that radar. The new app was there when I logged on this morning and then the testflight step worked fine. I'll close this out as it seems this is an issue with ITC and not Fastlane.


Add instructions for Docker

Feature
The project already has a Dockerfile and can be run locally.
It might be a nice idea to A. Publish an image to Docker hub and B. include instructions in the README how to run with Docker.

Hi I would like to work on this issue , notify me when the docker image is published  I will start working on the instructions  then .

If you create an dockerhub ou quay account and set the credentials as secrets in this repo, we can provide an action to build and push it using github actions.
gitub actions recognize when an action has a published image and use it instead of building it everytime the action is run.

Example of docker configuration:
jobs:
  upload_episode:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - name: Upload Episode from YouTube To Anchor.Fm
        uses: docker://quay.io/weltonrodrigo/youtube-to-anchorrm:episode_path
        timeout-minutes: 10

Hey @weltonrodrigo
Thanks for providing the example
Is this something you tested out? did it work?

On this note
I don't like to use github actions as from what I understand it violates the TOS
Are you aware of a platform that can be used instead?
something that'll run the docker container either on a schedule or through a trigger

Hey @weltonrodrigo
Thanks for providing the example
Is this something you tested out? did it work?

This is how I use this code. I have my own image built at quay.io and use it that way on my action.
As far as I know, this is a supported way to use a GitHub action hosted as a docker image. They have it on their GitHub workflows manual.
So, for a normal action run on a commit, I suppose this is completely covered under the TOS.
Running this as as scheduled action is not covered under the TOS, as that is simply using GitHub as a computation means.

I'll explain what I mean by "its against GitHubs TOS"
Quoting from https://docs.github.com/en/site-policy/github-terms/github-terms-for-additional-products-and-features#actions

Actions should not be used for:
...
• if using GitHub-hosted runners, any other activity unrelated to the production, testing, deployment, or publication of the software project associated with the repository where GitHub Actions are used.

Using this project in a GitHub action doesn't enhance the repos codebase it instead bring a 3rd party gain (publishing an episode on amchorFM)
I hope this help clarify what I meant

I suggest we continue the conversation on our discord server


Update release workflow

Codecov Report

Merging #36 (dd6cfde) into main (911fc57) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head dd6cfde differs from pull request most recent head c608602. Consider uploading reports for the commit c608602 to get more accurate results

@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage   78.66%   78.66%
=======================================
  Files           5        5
  Lines          75       75
  Branches        4        4
=======================================
  Hits           59       59
  Misses         15       15
  Partials        1        1

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix apache-rat license header check.

Motivation
Fix license header check error by apache-rat.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (no)

/cc @wolfstudy please help review the changes, i deleted empty README.md file in pulsar-function-go, because can not pass apache-rat check.

run cpp tests


Replace CommandLineArgumentProvider with ListProperty

In Gradle 4.6 we introduced a List< CommandLineArgumentProvider> on the compile and exec tasks. This could be replaced with the more generic ListProperty<String>, which would cover all cases:

adding simple String arguments (thereby also superseding getJvmArgs)
adding a single lazy String
adding a lazy set of Strings

I'm not sure what you mean. Do you mean using ListProperty<String> (this we have), or are you suggesting introducing ListProvider<String> as a new thing? Should users extend either ListProperty or this new ListProvider somehow? How would that type be instantiated?
Notice that a single CommandLineArgumentProvider can produce 0 or more String arguments, so I guess you'd need List<ListProvider<String>> to replace List<CommandLineArgumentProvider>.
We could have that, I guess, but I think CommandLineArgumentProvider has a very specific purpose, and it's better to capture it as a thing itself: we are not just producing Strings here, but Strings to be passed on the command-line that need to have their own annotated inputs and outputs. If we had a thing like ListProvider<String>, CommandLineArgumentProvider could extend that interface, but I would keep CommandLineArgumentProvider even then.

Sorry I meant ListProperty. You can add a Provider<List<String>> to that ListProperty if you want to compute multiple arguments at once. Providers are instantiated using project.objects.provider.
I don't see a need for a special-purpose type, it just makes the DSL less streamlined in my opinion. //cc @big-guy since you wanted to push Providers into more of our plugins.

I'm still not sure how this would work. A good exercise would be to show how JacocoAgent would be implemented with such a setup.

Okay so the problem is that you want it to be a @Nested object with annotated input properties which are used for up-to-date checking instead of the computed value. This is indeed not possible with Providers right now as far as I can see. I think it should be though, otherwise I fear we'll reinvent this pattern several times. I.e. command line arguments are not the only thing that should be computed from more structured data. Jar manifest entries come to mind as a different example.

Yes, this is exactly what CLAP was designed to be. I think it's alright to have these types, too. What would we gain if we had this thing as a ListProvider<String>?

Consistency. Why have two properties List<String> jvmArgs + List<CLAP> jvmArgProviders when we could have one ListProperty<String> jvmArgs?
I mean do we really want to create the equivalent of CLAP for every property that needs to be turned into a list if Strings and could be derived from richer data? Clearly for the Jar manifest case the name "CommandLineArgumentProvider" doesn't make sense. Wouldn't it be much better to have one type for this that we can use everywhere? I think that's exactly what ListProperty should be.

The jvmArgs property was left in place solely for backwards compatibility. We would have done the same if we introduced it as List<ListProperty<String>>. (Again, a single level ListProperty<> wouldn't work, as you want to be able to have multiple providers that each can provide lists of arguments.)
I think it's okay to create the equivalent for CLAP for some concepts. Command-line arguments is one of those things, as they have multiple uses and capture a real-world concept. We already use them in multiple places (Exec task parameters, jvmArgs, options for the Java compiler), and expect to use it in other command-line tool invoking tasks, e.g. native compilers, linkers etc.
I just saw that we have ListProperty<String> compilerArgs in AbstractNativeCompileTask – we'll need to change that somehow to be able to pass rich command-line arguments to native compilers.
Can you elaborate on how this collides with your expectations about Jar manifest handling?

@oehme I think one important piece is still missing from the discussion: We expect plugin authors to implement CommandLineArgumentProviders, while the Provider/Property interfaces should never be implemented by them.
The whole point of the interface is that a plugin author can implement it. The lazyness is not important.
We could have added a ListProperty<CommandLineArgumentProvider>, but we decided against it, since the whole task does not use providers at all and we were not sure how ready ListProperty is for prime time.
I guess the Provider in CommandLineArgumentProvider is misleading - a CommandLineArgumentProvider is not a Provider. Maybe we should come up with a better name. How about CommandLineArguments?
There also was some discussion on the design doc.

We would have done the same if we introduced it as List<ListProperty>. (Again, a single level ListProperty<> wouldn't work, as you want to be able to have multiple providers that each can provide lists of arguments.)

I think you misunderstand how ListProperty works. You can totally add a Provider to it that computes multiple arguments at once. As I mentioned earlier, ListProperty supports adding T, Provider<T> and Provider<List<T>>

I just saw that we have ListProperty compilerArgs in AbstractNativeCompileTask – we'll need to change that somehow to be able to pass rich command-line arguments to native compilers.

That's why I'm saying we should improve ListProperty and Provider so that users can treat them as @Nested.

Can you elaborate on how this collides with your expectations about Jar manifest handling?

Well according to your argument we would have to introduce ManifestEntryProvider similar to CommandLineArgumentProvider. My point is that both are just lazy lists of Strings and we should use ListProperty<String> for such lists.

The whole point of the interface is that a plugin author can implement it.

I don't see a good reason why we wouldn't allow some way to create a provider with nested fields. You currently pass a Callable to project.objects.provider, that Callable could have annotated fields.
I'd really like to hear @big-guy or @adammurdoch take on this.

I don't see why it would be a problem if we introduced a ManifestEntryProvider. If the use case is to produce manifest entries, I think you'd actually want a different API, something like:
interface ManifestEntryProvider {
    Map<String, String> asManifestEntries();
}

I'm still curious how you imagine something like JacocoAgent or a manifest entry provider to be implemented by a plugin developer or a build author with Providers only.

JacocoAgent implements Callable<String> {
  @InputFiles
  FileCollection jacocoPath

  String call() {
    //compute agent path here
  }
}

Test {
  //This isn't supported yet, but should in my opinion. Instead of snapshotting the return value of the providers it would snapshot the input properties of each provider added to this list.
  @Nested
  ListProperty<String> getJvmArguments()
}

test.jvmArguments.add(objects.provider(new JacocoAgent(configurations.jacoco))

I raised this generally in the preliminary discussion about CommandLineArgumentProvider. I thought we should look to reuse providers, but I now think that's wrong. However, I still think that a myriad of orthogonal bespoke types for every case (in reality, the ones we consider important enough to do this extra work for) is the wrong direction.
I think we should aim to simplify things to three concepts:

Providers - only provide execution integration machinery (e.g. lazy computation, task dependencies)
Value types - RegularFile, Directory etc.
Type representations/transformations/coercions (i.e. what CommandLineArgumentsProvider is) - needs a catchy name

We have, mistakenly in my opinion, complected (1) and (2) with things like ListProvider and RegularFileProvider. Where's my LockableListProvider? Where does this stop? I think provider should be always be absolutely value agnostic and be only concerned with execution mechanics. But that's another issue.
CommandLineArgumentsProvider should be replaced by something like Value<T> or  As<T>. This would have a single no arg method that returns the logical thing as T. So CommandLineArgumentsProvider becomes As<List<String>>. We would provide various out of the box type coercions, either via objects or via the value types themselves, e.g.
objects.asAbsolutePath(regularFile) // returns `As<String>`, but implementation has appropriate `@InputFile` etc annotations.

// or on regularFile

regularFile.asAbsolutePath() // like above

Provider would not implement As. They are different concerns. But you could have a Provider<As<List<String>>>.
This highlights a hurdle to this generic approach, in that many will find it not aesthetically pleasing to have so many <> flying around. Moreover, declaration site covariance and contravariance exacerbates this problem with Provider<As<? extends List<String>>> being required in a few places.
However, if we and users can accept this, I think we get a lot. The benefit of a simple, universal and composable programming model would go a long way. With the proliferation of Provider implementations and types like CommandLineArgumentsProvider I think we are headed in the opposite direction of further complecting the programming model by introducing too many bespoke concepts.

I like this separation of concerns, and is in-line with making CommandLineArgumentProvider more generic iteratively. In 4.5 we had CompilerArgumentProvider that has been extended to the more general CommandLineArgumentProvider in 4.6. I can see us taking it to this new place eventually. My only concern is not to end up with a deep type system that we try to model with Java's generics.

My only concern is not to end up with a deep type system that we try to model with Java's generics.

I am concerned about this too, but I would like to really explore the benefits before writing it off. My intuition is that we could come up with something very, small, simple and powerful with this approach and it might in the end outweigh the emotional barrier of nested parameterised types.

We should also think about what this would look like in Kotlin. Maybe some of its features would make what Luke is proposing look more pleasing.


Document summaries cannot contain predicates

Describe the bug
When a summary of type predicate is added to a document-summary, all queries fail (even those that do not request any presentation.summary).
To Reproduce
Steps to reproduce the behavior:

Used sample application Predicate Search.
Added the following at line 20 to src/main/application/schemas/ad.sd:

  document-summary adsummary {
    from-disk
    summary title type string {
      source: title
    }
    summary target type predicate {
      source: target
    }
  }

After deploying and feeding data, the response to the application test is:

{
   "root":{
      "id":"toplevel",
      "relevance":1.0,
      "fields":{
         "totalCount":3
      },
      "coverage":{
         "coverage":100,
         "documents":4,
         "full":true,
         "nodes":1,
         "results":1,
         "resultsFull":1
      },
      "errors":[
         {
            "code":10,
            "summary":"Backend communication error",
            "message":"Error response from rpc node connection to vespa-container:19106: Connection error"
         }
      ],
      "children":[
         {
            "id":"index:adserver/0/85cecfac5cdb8b0c17bbe05c",
            "relevance":0.0017429193899782135,
            "source":"adserver"
         },
         {
            "id":"index:adserver/0/382f7509e4bed774eb3c57ee",
            "relevance":0.0017429193899782135,
            "source":"adserver"
         },
         {
            "id":"index:adserver/0/3e26ff81baa990bebf73395a",
            "relevance":0.0017429193899782135,
            "source":"adserver"
         }
      ]
   }
}

Expected behavior
{
   "root":{
      "id":"toplevel",
      "relevance":1.0,
      "fields":{
         "totalCount":3
      },
      "coverage":{
         "coverage":100,
         "documents":4,
         "full":true,
         "nodes":1,
         "results":1,
         "resultsFull":1
      },
      "children":[
         {
            "id":"id:sampleapp:ad::http://example.com/sled",
            "relevance":0.0017429193899782135,
            "source":"adserver",
            "fields":{
               "sddocname":"ad",
               "subqueries(target)":-1,
               "documentid":"id:sampleapp:ad::http://example.com/sled",
               "title":"ACME Rocket Sled",
               "target":"('name' in ['Wile E. Coyote'] or 'age' in [14..25])\n"
            }
         },
         {
            "id":"id:sampleapp:ad::http://example.com/boulder",
            "relevance":0.0017429193899782135,
            "source":"adserver",
            "fields":{
               "sddocname":"ad",
               "subqueries(target)":-1,
               "documentid":"id:sampleapp:ad::http://example.com/boulder",
               "title":"ACME Dehydrated Boulders",
               "target":"('name' in ['Wile E. Coyote'] or 'age' in [10..20])\n"
            }
         },
         {
            "id":"id:sampleapp:ad::http://example.com/tornado",
            "relevance":0.0017429193899782135,
            "source":"adserver",
            "fields":{
               "sddocname":"ad",
               "subqueries(target)":-1,
               "documentid":"id:sampleapp:ad::http://example.com/tornado",
               "title":"ACME Do-It Yourself Tornado Kit",
               "target":"('name' in ['Wile E. Coyote'] or 'age' in [20..40])\n"
            }
         }
      ]
   }
}

Additional context
Removing target from adsummary or adding any other new field of any other type yields the expected result.

confirmed, this is a bug (in config-model).
workaround: use the old syntax for putting the field in another summary, which is
        field target type predicate {
            indexing: attribute | summary
            summary {to: default, adsummary}

instead of declaring it inside the document-summary adsummary block.

I just wanted to say thanks for the detailed bug description @albertmolinermrf!

workaround: use the old syntax for putting the field in another summary

Thank you very much!


Button Enhancements

issues tab report preview buttons

sizes change when flipping between 30-60-90 and referral schedule previews

As a (type of user)
I want (goal)
so that (reason/ why/ value)
Acceptance Criteria
Scenario 1:
GIVEN ... (you or your condition)
WHEN ... (what you do)
THEN ... (what you see)
Scenario 2:
xxxxxx
Validation Rules?
(If yes, list here)
Screenshot
(Link above will always be the most up-to-date version.)

@djnunez-aot
not working for Issue Tab, should work the same way as on the Status Tab


Support concurrent EM Engine

Today, due to some EnvoyProxy limitations, Envoy-Mobile can not run more than one Engine at a given moment. For the YouTube App, only one Engine is not viable - it needs at least two concurrent Engines.
TODO: upstream tests with tear down of A first, tear down of B first, different logging paths.

Ok @goaway @junr03 the upstream part is "done enough" insofar as if you bump main it should work (and if not, it definitely works with envoy.restart_features.no_runtime_singleton = true, as that's what is regression tested)

I am going to close this issue (which is actually a duplicate of #332). I forgot that original one existed. Now that all the Envoy pieces are in, the more relevant context is @goaway's description in #332. Thanks for all the work @alyssawilk, as discussed in the community meeting on thursday, the team at Lyft will prioritize resources to work on this.


[DO NOT MERGE] Test some stuff around Date serialisation

This is me experimenting with approaches to https://github.com/wellcomecollection/wellcomecollection.org/issues/8364

This works on my minimal example page – does it work everywhere?
I tried running yarn tryAllContentPages in the prismic-model directory to see if there are any pages passing other props that don't get through the stricter removeUndefined method.
I got two interesting 500 errors (plus a few 404s for exhibition guides because I don't have the toggle):
!!! 500 http://localhost:3000/pages/Wvl1wiAAADMJ3zNe
!!! 500 http://localhost:3000/pages/WwQHTSAAANBfDYXU

In both cases they're passing a Moment value through the Props, which superjson doesn't know how to serialise. Presumably we could change those to Date and we'd be okay?

Size Change: +6.1 kB (+1%)

Oof.

So I think I've got all the pages working now, but removing Moment from the props has broken a bunch of other things.
Before I go any further, I wonder what other people think of this. Does this plugin/approach seem like a sensible or plausible idea, or am I barking up the wrong tree entirely?

(I've temporarily reverted the changes to Moment in the props, because they're muddying the diff and I don't think they're useful for assessing the impact of the broader change.)

Closing as superseded; I got a working version of this in #8378.


⚠ Processors has degraded performance

In a3429a7, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 78 ms

Resolved: Processors performance has improved in 351d68b.


Add support for draco compression to Core

This PR add support for draco compression to the Core lib.
For the purpose, we divided the DracoCompression class into an Encoder and a Decoder, keeping the old API with a deprecated flag.
We now have a

DracoCommons file which encompass all the standard declarations and interfaces
DracoEncoder for encoding purpose
DracoDecoder for decoding purpose.
DracoCompression has been kept for backward compatibility, but greatly reduced as kind of decorator.

NO regression has been observed using GLTF Loader with KHR_draco_mesh_compression
Test has been Also conducted by compressing and decompressing mesh on the fly using
  // encode the mesh
  var encoder = new BABYLON.DracoEncoder();
  return encoder.encodeMeshAsync(mesh).then((encoded)=> {
          // once done decode the mesh
          var decoder = new BABYLON.DracoDecoder();
          return decoder
              .decodeMeshAsync(encoded.data, encoded.attributes)
              .then((babylonVertexData) => {
                  // and apply the geometry to new mesh
                  const name = mesh.Name + "Dracoized"
                  const babylonMesh = new BABYLON.Mesh(name, scene);
                  babylonVertexData.applyToMesh(babylonMesh)
                  // finally return the mesh
                  return babylonMesh;
              })
              .catch((error) => {
                  throw new Error(`${error.message}`);
              });

from playground
Note the above playround will not be functional until the PR will be merge.
Test has been conducted changing the Draco wasm configuration to local
    let baseEncoderUrl = "http://localhost:1337/";

    const codecConfiguration  =  {
        decoder: {
            wasmUrl: baseEncoderUrl + "draco_wasm_wrapper_gltf.js",
            wasmBinaryUrl: baseEncoderUrl + "draco_decoder_gltf.wasm",
            fallbackUrl: baseEncoderUrl + "draco_decoder_gltf.js",
        },
        encoder: {
            wasmUrl: baseEncoderUrl + "draco_encoder_wrapper.js",
            wasmBinaryUrl: baseEncoderUrl + "draco_encoder.wasm",
            fallbackUrl: baseEncoderUrl + "draco_encoder.js",
        },

    };
    BABYLON.DracoEncoder.Configuration = codecConfiguration.encoder;
    BABYLON.DracoDecoder.Configuration = codecConfiguration.decoder;

cc @pandaGaume to see if he has a few cycles to finish this one? Thanks a million!

@pandaGaume, do you think you ll be able to finish this one ? or should we flag as draft until someone can pick up where it is ?

@pandaGaume I'm closing this PR for now. We can reopen if it becomes active again.


Remove frameworks from repo

Remove all frameworks that from Xcode

Fixes #2

Thank you! :)

@yycking Would you be interested in being a maintainer here? Just asking since you've done some good pull requests. No worries if not.


Add EUC Nordvest

Homepage: http://eucnordvest.dk/
The following is an academic education (HTX, tekniske skole), where students choose a "path". Several of these paths include "programming" and "Communications and IT". These educations last 3 years.
http://eucnordvest.dk/tgt/
For reference on the danish education system, please go to: https://en.wikipedia.org/wiki/Education_in_Denmark
(The educations at EUC Nordvest, are all "vocational secondary educations", and start at age 15-16)

@stou Pull request merged. Thank you!


Move Bootstrap 4 to Maintenance LTS

The 5/26/2020 date has already passed. This patch fixes the table, moving v4 to Maintenance LTS.
Also, I statd that v5 has not yet been released.

Thanks, we have already moved the dates.


server client mode IPSec & force-udp-encapsulation mode

Get and expose backend config for preferred-server
Implement multiple connection modes in libreswan
Implement the CE_IPSEC_FORCEENCAPS flag

@skitt I added an extra commit to this so you guys don't need to rebase a follow up PR while I'm not around, I suspect that you are not around at this time, so I will talk to tom about removing that commit, merging, and opening a new PR.

@skitt I added an extra commit to this so you guys don't need to rebase a follow up PR while I'm not around, I suspect that you are not around at this time, so I will talk to tom about removing that commit, merging, and opening a new PR.

The extra commit is fine, I’ll leave my approval.

Minor comments - pushed https://github.com/submariner-io/submariner/pull/1224/commits/56bd6864fb4f9e7eadeec8290365c5f980ad2a2e to address them.

Minor comments - pushed 56bd686 to address them.

I fixed up the original commits using that commit and added you as co-author, so that the history still makes sense.

Minor comments - pushed 56bd686 to address them.

I fixed up the original commits using that commit and added you as co-author, so that the history still makes sense.

Thanks!


Idea: Track Xcode version in archive digest identifier

One of my apps failed submission because the frameworks specified iOS 10.2 rather than 10.1.

Looking online it seems that the error I got from iTunesConnect matches the ones of people that have been trying to submit apps using Xcode Beta versions.
I am not 100% sure but I think this might be due to the framework being build on a Mac with Xcode 8.2 and uploaded, but the CI box that submitted the build to the app store is running Xcode 8.1.
If the version of Xcode had been used as part of the identifier of the archive this might not have happened.
Would it make sense though to do this? Or would it get in the way?
I can see this creating friction when Xcode updates but with no changes to the min sdk version of iOS.
Anyway... Just wanted to put this here to start the discussion 👍

I am sorry to hear that. Carthage cache currently uses a digest of the Cartfile.resolved plus the version of Swift. I think I assumed that if Xcode version changed the Swift version would change but that does not seem to be the case.
This is really annoying (thanks Apple for that).
Can you corroborate that the problem is the different version of XCode? If this is the case I think we could add the xcode version as part of the archive digest identifier.
This is quite weird because if this compiles is actually an iTunes issue because if the swift compiler did not change everything should work.
Making the change is really simple is basically adding
`xcodebuild -version`.match(/Xcode ((\d+\.)?\d+\.\d+)/)[1]

to this function https://github.com/guidomb/carthage_cache/blob/master/lib/carthage_cache/carthage_resolved_file.rb#L35
But I don't want to add this unless is really necessary.

Hi, sorry for the intermittent feedback.
I'm still not 100% sure where the issue is, sorry 😞 . I'll keep you posted on this.

I've been using carhage_cache we both Xcode 7.2.1 and Xcode 8.2.1 and did not find any issues. I am closing this for now because it seems there is nothing to be done. Feel free to re-open it if you have any news. Thanks!


Error 500 with web hook with Gitlab

Hello,
I try to configure continuous deployment with a perso gitlab server. I follow the "Setting up continuous deployment using manual steps" of the page https://github.com/projectkudu/kudu/wiki/Continuous-deployment.
When I push something on my gitlab repository, Azure receive the hook but the deployment has an exception.
In the POST_deploy log I have an error 500 :
"Value cannot be null.
Parameter name: url" stackTrace="   at LibGit2Sharp.RemoteCollection.Add(String name, String url, String fetchRefSpec)
at Kudu.Core.SourceControl.Git.LibGit2SharpRepository.FetchWithoutConflict(String remoteUrl, String branchName)
at Kudu.Services.ServiceHookHandlers.ServiceHookHandlerBase.Fetch(IRepository repository, DeploymentInfo deploymentInfo, String targetBranch, ILogger logger)
at Kudu.Services.FetchHandler.<PerformDeployment>d__e.MoveNext()
I don't understand which url is wrong : I don't find already the xxxHandlerFact used.
Can someone help me?
Thanks a lot.
Best regards.
Mikaël PLOUHINEC

Maybe the payload that gets sent is not compatible with what Kudu expect for some reason. See https://github.com/projectkudu/kudu/blob/16d3fa55140f15fe8517515f5fbc4f89481196ed/Kudu.Services/ServiceHookHandlers/GitlabHqHandler.cs#L10-L40 for what's expected, and compare with what's getting sent in your scenario (you can use a site like http://requestb.in/ to capture it if needed).

Just try it and it seems to work.   As @davidebbo suggested, do capture it with http://requestb.in and share us the payload.

Hello,
Where can I change the url? Is that the web hook the problem or the kudu request to my gitlab?
This is the web hook send by my gitlab :
{
"before":"f18bed37cdcee5cf93835cc6a071e7221b3d0b62",
"after":"f90a1c8322164053981bc2a48329cdcdf7e5b37f",
"ref":"refs/heads/master",
"checkout_sha":"f90a1c8322164053981bc2a48329cdcdf7e5b37f",
"user_id":143,
"user_name":"Mikael Plouhinec",
"project_id":266,
"repository":{
"name":"kitdefacilitation.serveur",
"url":"git@gitlab-toulouse.sqli.com:vfruchet/kitdefacilitation.serveur.git",
"description":"Kit de facilitation - Serveur NodeJS",
"homepage":"http://gitlab-toulouse.sqli.com/vfruchet/kitdefacilitation.serveur"
},
"commits":[
{
"id":"f90a1c8322164053981bc2a48329cdcdf7e5b37f",
"message":"Ajout d'un fichier README.md\n",
"timestamp":"2015-10-26T22:58:12+01:00",
"url":"http://gitlab-toulouse.sqli.com/vfruchet/kitdefacilitation.serveur/commit/f90a1c8322164053981bc2a48329cdcdf7e5b37f",
"author":{
"name":"MikeSQLIdeployment",
<EMAIL_ADDRESS>}
},
{
"id":"c38fa062c78c4368e4535e12a02262fb6311196a",
"message":"Correction de bugs\n",
"timestamp":"2015-10-26T17:18:05+01:00",
"url":"http://gitlab-toulouse.sqli.com/vfruchet/kitdefacilitation.serveur/commit/c38fa062c78c4368e4535e12a02262fb6311196a",
"author":{
"name":"vfruchet",
<EMAIL_ADDRESS>}
},
{
"id":"f18bed37cdcee5cf93835cc6a071e7221b3d0b62",
"message":"Merge branch 'master' of https://kitdefacilitation.scm.azurewebsites.net:443/kitdefacilitation\n\n# Conflicts:\n#\tserver.js\n",
"timestamp":"2015-10-26T14:40:29+01:00",
"url":"http://gitlab-toulouse.sqli.com/vfruchet/kitdefacilitation.serveur/commit/f18bed37cdcee5cf93835cc6a071e7221b3d0b62",
"author":{
"name":"vfruchet",
<EMAIL_ADDRESS>}
}
],
"total_commits_count":3
}
Thanks

Your payload is missing git_http_url and git_ssh_url, which Kudu expects. I'm guessing you're running on old version of GitLab. Those fields were added around February. See https://github.com/projectkudu/kudu/issues/1470. @randx can confirm.

My GitLab Server is 7.7.2 version.
I try with a gitlab server in version 7.11.0 : the payload contains the visibility_level, git_http_url, git_ssh_url.
The Gitlab server need an upgrade I think.
Thanks a lot for your help.
Mike

@davidebbo one idea is we could consider do a small targeted Kudu fix for this (fallback to repository's url).   The other missing info is whether this is public or private repository.   We will always assume whatever defined by repository's url.   Thought?

For information, the fields were added here. https://github.com/gitlabhq/gitlabhq/commit/92434b29cc45677fe72bb6a8a5bd09d5ead8d138


Needs not in word list

Add the word needs to the word list

This question has been answered in https://github.com/jason-chao/wordle-solver/issues/2
All plurals are not loaded into the solver by default.  The word needs is on the word lists though it is considered the plural form of the noun need.
If you really want to include plurals in your search, please check the include plurals box in the web UI or pass --plurals True in the argument to the CLI.

This question has been answered in #2
All plurals are not loaded into the solver by default. The word needs is on the word lists though it is considered the plural form of the noun need.
If you really want to include plurals in your search, please check the include plurals box in the web UI or pass --plurals True in the argument to the CLI.

But I was using the web version of the solver, as I don't have python, so it doesn't work...
Maybe I'll fork it

@DaCuteRaccoon  In the web version, the include plurals checkbox is in the top right corner.   I hope it helps.


Bug fix for logical interconnect.

Description
Added missing structure for logical interconnect
Issues Resolved
[List any issues this PR will resolve. e.g., Fixes #01]
Check List

[X] New functionality includes testing.

[X] All tests pass for go 1.11 + gofmt checks.

[ ] New functionality has been documented in the README if applicable.

[ ] New functionality has been thoroughly documented in the examples (please include helpful comments).

[ ] Changes are documented in the CHANGELOG.

LGTM


Added error handling to getWorkflow and getWorkflows in workflow.js

Test before pulling.

This seems that it will just move the error to the next part of the calling function, e.g. if getWorkflows fails, PysssssWorkflows.load will then fail on if(this.workflows.length) as workflows is undefined
What problem were you trying to solve here, or was it just to log the response error better?

:thinking: I was hoping to have it fail gracefully if it doesn't get what
it wants.
On Sun, Apr 7, 2024 at 8:08 AM pythongosssss @.***>
wrote:

This seems that it will just move the error to the next part of the
calling function, e.g. if getWorkflows fails, PysssssWorkflows.load will
then fail on if(this.workflows.length) as workflows is undefined
What problem were you trying to solve here, or was it just to log the
response error better?
—
Reply to this email directly, view it on GitHub
https://github.com/pythongosssss/ComfyUI-Custom-Scripts/pull/225#issuecomment-2041482902,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC36LN7JVM3BWGN22FSLE2TY4FHNJAVCNFSM6AAAAABFSUXC7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBRGQ4DEOJQGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


[Bug] Tooltip on icons in the icon selector dropdown moves away with the scrolling

Description
Tooltip on icons in icon selector dropdown moves away with the scrolling, please find attached the screenshot and screen recording.
Steps to reproduce the behaviour:

Add an icon button
Click on the icon button
Click on icon selector dropdown
Mouse over any of the icons until tooltip appears and then scroll down
the tooltip floats out of the icon selector dropdown to the top of the screen

Important Details

Version: Cloud
Browser: chrome, safari
Environment: production

https://user-images.githubusercontent.com/5444055/132344864-f09aaf00-eaee-4605-b916-3e8aff23d508.mov

I can fix this issue, assign it to me. please.

Hi @ipankajchaudhary, it;s all yours go for it!

Maybe its time to re-assign or close this? @Nikhil-Nandagopal

Property pane is docked now, This solves the problem


Is a package missing to do coverage?

https://github.com/coderefinery/software/blob/main/environment.yml

pytest-cov ?

pytest-cov=4.0.0 for the (now updated) environment files.  Then the following command can be run locally: pytest --cov-report "xml:coverage.xml"  --cov=.
The Actions file already has it explicitly.  We don't ask peolpe to run this locally, but it can be with this: pytest --cov-report "html:coverage-report" test_functions.py --cov=.
Should this be added to environment.yml or not?


Add terminal renderering engine

New:

first proposal to fix #35

When the size of the terminal is changed and the program run, the size is adapted and in case of an overflow add an ellipsis in the end


As Dr. Bob, I want to see recall data on a national scale, to identify and validate patterns and trends in food recalls across the country.

The Map should show the number of recalls affecting each state.

This is now the default map view.


Nodes added via EditorPlugin should have more direct file path access

It seems to make more sense for nodes that are added as a new node type to have an easier way of retrieving their script resource.
Currently, if I had a node at res://addons/my_plugin/path/to/my/node.gd, I have to actually supply the entire path to the node in the plugin just to extend its functionality with a script (seems pretty extreme). I'm not sure how possible it would be to get syntax highlighting / compilation off of the node's name in GDScript (does adding it to the ClassDB allow for that at all?). However, even if that weren't possible, there's gotta be something we can do. Even if it's just adding a Globals method (or adding a class to the Globals) that takes a string and returns the corresponding path to the script file. Something like...?
# Given a 'MyNode' new node type
extends script("MyNode")
var MyNode = preload(script("MyNode"))

This is fixed with the introduction of class_name


Show starred repositories

It was nice to show them on the profile page after the repositories :star:

An extra item to show Stars that takes the user to a RepositoryListScreen of all their starred repos? I like it, my only concern is maybe 4 items on the user profile might be cluttering things up a bit?? More than happy to see how it looks before anyone starts adding the logic 🙌

It will look like this:

With more then a thousand we can do something like:

I think it still looks good and not to cluttered, the clickable areas are still easily touchable.

Nice agreed that does look good 👌🏾

If it's not too difficult, let's also add navigation functionality so the user can click it and view what another user has starred as part of this ticket, hopefully the API makes this straightforward 🙌🏾

Has anyone worked on this? I am more than interested to do so.

@adrianhartanto0 fine! We will be only happy if you do it :smiley:

@adrianhartanto0 Would be awesome if you do it 😉
This can probably help you:
https://stackoverflow.com/questions/30636798/get-user-total-starred-count-using-github-api-v3

Good link @RolfKoenders! Wish GitHub would return explicit counts, though.

Getting an error saying ```
"Provide a valid path to the desired application bundle.
Print: Entry, ":CFBundleIdentifier", Does Not Exist".

@andrewda I haven't checked the v4 api yet. Maybe its in there?


Add private and private test network types

The latest alpha SDK adds private and private test networks
https://github.com/nemtech/symbol-sdk-typescript-javascript/blob/dev/src/model/network/NetworkType.ts#L45
We should add those to the wallet, for example, the network type selectors:

At least:
https://github.com/nemgrouplimited/symbol-desktop-wallet/blob/main/src/views/pages/settings/AboutPage/AboutPage.vue#L38
https://github.com/nemgrouplimited/symbol-desktop-wallet/blob/main/src/core/utils/NetworkTypeHelper.ts#L26
About page should use the NetworkTypeHelper, NetworkTypeHelper could enum the values from SDK's NetworkType

I have another opinion. For me for public release should be visible only MAINNET and TESTNET. Others can confuse users.

Closing due to scope change. cc @fboucquez @cryptoBeliever


Strict Write-Output

Summary of the new feature / enhancement
Write-Output description excerpt:

In PowerShell, it is generally not necessary to use the cmdlet in instances where the output is displayed by default.

This is a very nice feature but at the other hand could cause quite some confusion for a novice PowerShell scripter or with complex (recursive) functions as it implies that unwanted objects (including $null) could be accidently dropped into the pipeline by an included command (function, method or cmdlet) without notice.
Besides, if it known that a custom function leaks objects into the pipeline,  it might still be a hassle to actually track them down.
See StackOverflow issues along with:

PowerShell Pipeline Pollution
Powershell ArrayList magically inserts a zero
Why does Range.BorderAround emit “True” to the console?

Proposed technical implementation details (optional)
Despite the reasons to avoid the use of the Write-Output cmdlet, it would be nice to have a Set-StrictMode version that enforces the use of Write-Output or Return. Meaning, if anything is placed on the pipeline that doesn't come from Write-Output or Return, a "strict" error should be raised pushing for a fix either way:

If it was the intention to add something to the pipeline, use the Write-Output cmdlet.
if it was not the intention to add anything to the pipeline, waste the results by using the [Void] initiator or assigning it
(e.g. $Null = ...)

Caveats

I have no clue how difficult it is to determine whether a function or method always returns nothing or for a cmdlet (like Export-Csv) always returns a System.Management.Automation.Internal.AutomationNull.
I suspect that that external commands should always be voided ([void](NotePad)) or assigned ($Null = NotePad)
As Set-StrictMode is also applied to the child scopes, I presume that custom child functions can just be checked for the existence of Write-Host or return statement to determine whether something is returned.

Actually,  at second thought, the above caveats ar probably not relevant as the strict check (whether Write-Output or Return is used when something is added to the pipeline)  should take place at runtime.

Set-StrictMode can be set from within a function but could also be set (temporary for trouble shooting) at the prompt level (or even at a global scope with Set-PSDebug -Strict command). The indiscriminate implication of this request would be that every expression (including a simple expression-mode calculation as 2 * 3 or .\MyTestFunction.ps1 invocation)  would require a Write-Output (as: 2 * 3 |Write-Output or .\MyTestFunction.ps1 |Write-Output) unless exceptions are made at the top/shell-level.

The pitfall of accidental output is real, but I wonder if it is better addressed with a debugging feature rather than a new strict mode, which would in essence take away a fundamental PowerShell feature (implicit output). Also, as a fundamental problem, the lack of lexical scoping makes any strict mode liable to break unrelated code.
Therefore, I wonder if enhancing, say, Set-PSDebug is a better way forward, allowing users to locate all lines in their script that produce output in a given call.
# HYPOTHETICAL example.
# As an aside: -Trace is currently [int]-typed  - switching to an *enum* with descriptive symbolic value names
# would make sense.
# 3 == hypothetical new output-tracing flag
PS> Set-PSDebug -Trace 3; ./script.ps1

DEBUG:    8+  >>>> $arrayList.Add($foo)
0

That is, the above indicates that the statement on line 8 produced output.

As a stopgap  you can use the following technique to print the locations and source code of output-producing statements:
PS> ./script.ps1 | % {
  Write-Verbose -vb "Output from $(($entry=(Get-PsCallStack)[1]).Location): $($entry.Position.Text)"; $_
}

VERBOSE: Output from script.ps1: line 8: $arrayList.Add($foo)
0

@mklement0, thanks for the comments, I indeed also like the Set-PSDebug idea.

a new strict mode, which would in essence take away a fundamental PowerShell feature (implicit output). Also, as a fundamental problem, the lack of lexical scoping makes any strict mode liable to break unrelated code.

I agree, the idea is quite controvert. Yet avoiding accidental pitfalls is exactly why you would prefer strict programming (modes).

Re Set-PSDebug: Glad to hear it, @iRon7.
Re introducing new strict modes in general: I am wary:

The lack of lexical scoping of strict modes makes their use treacherous.
Even if lexical scoping were to be introduced, the dynamic scoping of the existing strict modes would have to be retained for backward compatibility, which would invite confusion and subtle bugs.
Generally, strict modes make it harder to reason about a given piece of code.

As for the specific new mode proposed:

While such a mode would undoubtedly help, to me the philosophical question is whether carving out a "non-PowerShell niche" inside of PowerShell that subscribes to a different output philosophy is worth doing.

Therefore, my personal preference would be to help prevent this pitfall via debugging and linting features (PSSA, also as an opt-in).

@iRon7, I assume you meant to post this comment here instead:

@mklement0,

Even if lexical scoping were to be introduced, the dynamic scoping of the existing strict modes would have to be >> retained for backward compatibility, which would invite confusion and subtle bugs.

A different way to implement lexical scoping is by excluding using SuppressMessageAttribute class in the cmdlet binding similar to the approach of PSScriptAnalyzer.

That would certainly fix the scoping issue (it could even be a property of the existing CmdletBinding attribute) and make code easier to reason about.
However, requiring the use of an attribute - undoubtedly an advanced technique -  could prove too high of a hurdle for the intended target audience, novice scripters.

The Engine WG reviewed this issue today. We don't think introducing a new strict mode is the right way to help prevent this issue because that could easily break other scripts. If a script uses strict mode, then most likely it's using Set-StrictMode latest, which means new strict mode like the proposed one will be applied to it, and thus potentially break the script.
We believe PowerShell Script Analyzer should be the right tool to help, by emitting warnings for expressions that generating output directly to pipeline. Feel free to open an issue in that repo.

Sorry, I was on holidays and couldn't immediately respond.

If a script uses strict mode, then most likely it's using Set-StrictMode latest, which means new strict mode like the proposed one will be applied to it, and thus potentially break the script.

This argument is more related to the implementation of Set-StrictMode and how the scope is linked to a version rather than a lexical scoping using a different parameter or cmdlet annotations (using the System.Diagnostics.CodeAnalysis Namespace?) instead. See also: #16388 -Mode parameter for Set-StrictMode.

We believe PowerShell Script Analyzer should be the right tool to help, by emitting warnings for expressions that generate output directly to pipeline. Feel free to open an issue in that repo.

Afaik, PowerShell Script Analyzer is not able to catch runtime errors/situation knowing that a (custom) function/cmdlet could potentially return an object but effectively never does during runtime...
Anyways, I am happy to give it a try if you think otherwise.

Related: #3669 Allow advanced functions to only return objects to the pipeline explicitly


Use less None as default

In the code None is used as argument default while there is better default.
For example:

Repo, could have path='' and config_file='.howfairis.yml'
Config, could have config_file='.howfairis.yml'
Compliance could have False for repository, license, registry, citation and checklist arguments
Checker, could have  self.readme = self._get_readme()  in constructor.
.howfairis.yml, could have False for force_* options.

item 3 was implemented as part of PR #156

item 4 is not allowed by a linter we were using in the past. It said, if your instance has methods that change members of the instance, define them in the (body of the) constructor. This is to avoid introducing surprise members from methods such as self._get_readme()

item 5 works different than what you may expect, users can force it either false or true, or let the algorithm do its thing without any forcing (hence you need the 'unset' value ~)

Found another use of None in Readme.py constructor, it could use required arguments.

Option 5 is handled in #164

All options have been fixed already or split off into own issues.


DO NOT USE AOS

AOS is an incomplete project that will inevitably mess up your workflow and cause you all sorts of problems. Don't make the same mistake I did and install this product. Will take me less time to uninstall and find a complete solution than to fix the issues that exist in AOS.
AOS breaks frequently, items triggering too early, too late, or not at all.
There is some issue deep in the AOS library that most likely will never get fixed. Delete AOS from your projects and move onto the next scroll-trigger software.

I've had no such issues.
Also, if you know what's wrong with a repository, the nature of GitHub allows anyone to submit Pull Requests and fix those issues or submit improvements. There's no need to behave like a spoilt 5 yr by opening these types of issues and bitching about your inability to use it. I'm sure there's plenty of others who use this Repo without issues.

I've had no such issues.

Read through the 129 open issues and you can see some core commonalities.
This product is not production ready, and should not be presented as such.

I've been using AOS for years on dozens of projects with no issues. There are several threads outlining common issues and solutions for most problems. I'm not sure where this flippant aggression is coming from, but if you explain the issue you're having, many of us would be happy to help.

I don't know why pointing out that there's 129 open issues is going to do. Many projects, have a lot more open issues, with people outside of that group working to improve projects, hence, the nature of Open Source and GitHub allowing anyone to help.
@TheIllestt you are more likely to get a positive response when keeping the above in mind.

Alalalalalala

very unnecessary issue dude

Well, don't be that harsh. I myself is having issues with AOS recently. AOS doesn't like overflow:hidden for sure.

I agree a Little bit,
Often I see issues with Aos, animation that triggered few moments Later or earlier, ecc.

AOS comes with a lot of CSS rules, that's the point of a library. That's why it can break things with your project CSS.
Depending on your project, AOS CSS rules can be:

too specific
not specific enough
can cause conflict with one of your project CSS rules

When you import AOS (or any library, actually), you have to be sure of its implementation in your project (and you have to be sure of how your project's CSS implementation aswell: surely, AOS won't handle well hacky CSS with negative margin, zoom, etc).
So, well, that's not an issue, and cannot be considered as it.

I'm voting to close and lock-down this issue, as it isn't an issue at all.

This post has served its point to caution developers who aren't looking for a "basket case" software. If I can save one person the frustration and wasted time, then this post was worth it. Seems a few people may have benefited from my cautionary tale.
As for the 129 issues -- the point is that you can see there are many issues that have no resolution, and there are some core similarities. The number doesn't matter, but unresolved and piling up issues in the framework do.
I've found another scroll-trigger solution. You can put your pitchforks and torches away now.
Good luck y'all, stay safe.

This post has served its point to caution developers who aren't looking for a "basket case" software. If I can save one person the frustration and wasted time, then this post was worth it. Seems a few people may have benefited from my cautionary tale.

You get the point: that's not an issue, that's what people tells you. It won't last 5 minutes on Stack Overflow before getting closed by a moderator. If your goal is to warn rookie web dev, just start a Medium post, this is not the purpose of an issue.

As for the 129 issues -- the point is that you can see there are many issues that have no resolution, and there are some core similarities.

React too, will you start the same issue on its repo?

The number doesn't matter, but unresolved and piling up issues in the framework do.

This is basically why the issue tab exists.

I've found another scroll-trigger solution.

You're more than welcome to share it.
One thing you never said, which might actually help someone: what version of AOS are you using?


(feat) added vertical padding to total height in _suggestion_builder

Dear @maheshmnj,
I trust this email finds you well.
Firstly, I would like to extend my gratitude to you for the recent improvements you've made to the SearchField package. The addition of padding has greatly enhanced user interaction and brought a superior aesthetic quality to the feature. Your attention to detail and dedication to providing valuable contributions are both commendable and appreciated.
I wanted to bring to your attention that I have further refined the package by adding vertical padding to the total height in the suggestion_builder. I am attaching two videos that demonstrate the before and after effects of this enhancement for your reference.
Before
https://github.com/maheshmnj/searchfield/assets/66751960/98b748af-9e8d-449f-ae87-b8692da2ce41
After
https://github.com/maheshmnj/searchfield/assets/66751960/9940823c-8844-495a-99d9-bc30ec22f0ed
However, during my testing process, I noticed an issue related to animation jumping. While the overall functionality is not hindered, the jumping does have an adverse impact on user experience, given that fluid animation contributes significantly to a user-friendly interface.
Your expertise and insights would be of great help in resolving this issue. Could you please assist me in debugging this problem to ensure smoother animations? Your assistance in this matter would be greatly appreciated.
Looking forward to your positive response.
Best Regards,
Andrii

Thanks for the PR @AndriiReverchuk I will review it soon.

@maheshmnj  Thank you for your feedback. I've added the tests as you requested. Apologies for the delay in adding them; I've been pressed for time. Let me know if there are any further changes needed.

Hi Andrii,
Thank you for your thoughtful contribution and the kind words in your description. It's clear that you put significant time and effort into improving the Searchfield package, and I appreciate your dedication to enhancing the user experience.
I've reviewed the changes, and they look fantastic to me. I'll be merging this PR as it adds valuable enhancements to the package.
I understand the concern about animation jumping, and I'll make sure to look into it post-merge to ensure that we provide the best user experience possible. Your input is invaluable in this process.
Thanks again for your contribution, and I look forward to collaborating with you in the future.
Best,
Mahesh

Released in v0.8.7

Dear @maheshmnj,
Thank you for your prompt response and positive feedback regarding my work on the Searchfield package. I'm glad that my efforts proved beneficial and contributed to the project's advancement.
I'm pleased that you addressed the animation issue. I'm confident that together we can find the best solution to this matter.
I look forward to further collaboration and am ready to assist in improving the package. If you have any additional questions or comments, please let me know.
Best regards,
Andrii Reverchuk.


[Help] Issues compiling on latest Ubuntu

I was hoping you could help me with this.
I was trying to compile https://github.com/be5invis/Iosevka by myself but I fail at compiling otfcc for it, as I'm not exactly keen with compilers and environments.
There were other errors before, which I was able to solve by installing libc6-dev-i386.
==== Building otfccdump (debug_x32) ====
Creating obj/x32/Debug/otfccdump
otfccdump.c
../../src/cli/otfccdump.c: In function ‘main’:
../../src/cli/otfccdump.c:55:7: warning: variable ‘no_bom’ set but not used [-Wunused-but-set-variable]
  bool no_bom = false;
       ^
stopwatch.c
Linking otfccdump
bin/x32/Debug/liblibotfcc-tables.a(glyf.o): In function `coord_to_json':
/home/dev/otfcc/build/gmake/../../src/tables/glyf.c:356: undefined reference to `roundf'
bin/x32/Debug/libexternals.a(json-builder.o): In function `json_measure_ex':
/home/dev/otfcc/build/gmake/../../dep/extern/json-builder.c:672: undefined reference to `floor'
collect2: error: ld returned 1 exit status
otfccdump.make:138: recipe for target '../../bin/Debug-x32/otfccdump' failed
make[1]: *** [../../bin/Debug-x32/otfccdump] Error 1
Makefile:98: recipe for target 'otfccdump' failed
make: *** [otfccdump] Error 2

Running 16.04 LTS (Xenial), uname -a is:
Linux dev-vm 4.4.0-21-generic #37-Ubuntu SMP Mon Apr 18 18:33:37 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

Looks like missing lm when linking.

Is it working now?

@eiszfuchs Is the latest version works?

Sorry the delay! Yes, this seems to work now after a pull!
Thank you!
==== Building otfccdump (debug_x32) ====
otfccdump.c
../../src/cli/otfccdump.c: In function ‘main’:
../../src/cli/otfccdump.c:56:7: warning: variable ‘no_bom’ set but not used [-Wunused-but-set-variable]
  bool no_bom = false;
       ^
Linking otfccdump
==== Building otfccbuild (debug_x32) ====
Creating obj/x32/Debug/otfccbuild
otfccbuild.c
stopwatch.c
Linking otfccbuild

dev@dev-vm:~/otfcc/bin/Debug-x32$ ./otfccbuild --version
This is otfccbuild, version 0.1.5.
dev@dev-vm:~/otfcc/bin/Debug-x32$ ls -l
total 1612
-rwxrwxr-x 1 dev dev 926372 Apr 29 12:18 otfccbuild
-rwxrwxr-x 1 dev dev 711684 Apr 29 12:18 otfccdump

Calling make linux-release-x64 worked, too (but it had more warnings passing by while building). That's much easier than the steps in the README. :beers:
dev@dev-vm:~/otfcc/bin/Release-x64$ ls -l
total 656
-rwxrwxr-x 1 dev dev 349696 Apr 29 12:24 otfccbuild
-rwxrwxr-x 1 dev dev 316896 Apr 29 12:24 otfccdump

OK, closed.


VQE does not respect input variational form

Information

Qiskit Aqua version: latest
Python version:
Operating system:

What is the current behavior?
In going through this tutorial here: https://qiskit.org/documentation/tutorials/chemistry/03_ground_state_solvers.html it is clear that the VQE solver does not actually use the input variational form.  For example, I can do
num_qubits = 1
tl_circuit = TwoLocal(num_qubits, ['h', 'ry'], 'cz',
                      entanglement='linear', reps=2, parameter_prefix = 'th')
callback = Callback()

#backend = Simulators.aer_manhattan_simulator
backend = Aer.get_backend('qasm_simulator')
another_solver = VQE(var_form = tl_circuit, optimizer=SPSA(maxiter=250), callback=callback,
                     quantum_instance = QuantumInstance(backend, shots=2048))

tl_circuit.draw()

which yields:
     ┌───┐┌───────────┐┌───┐┌───────────┐┌───┐┌───────────┐
q_0: ┤ H ├┤ RY(th[0]) ├┤ H ├┤ RY(th[1]) ├┤ H ├┤ RY(th[2]) ├
     └───┘└───────────┘└───┘└───────────┘└───┘└───────────┘

However this is not what is actually used.  Calling

calc = GroundStateEigensolver(transformation, another_solver)
res = calc.solve(driver)

works just fine, and the callback function gives the parameter vector for a 4 qubit variational form.
Now my guess is that this is because it needs a 4Q circuit, but then the solver should raise an exception rather then implicitly recast things.
Steps to reproduce the problem
What is the expected behavior?
Suggested solutions

I removed the portion about the transformation class as I do not think it was actually valid.

try to set the number of qubits on the variational form, if possible
https://github.com/Qiskit/qiskit-aqua/blob/master/qiskit/aqua/algorithms/minimum_eigen_solvers/vqe.py#L274
This seems a current specification.
I think this is not user friendly, but we need to discuss changing the behavior.

The reason this was done was to allow the var form to be passed like a template if you will. In many cases a user will not know the actual number of qubits until the problem is computed and turned into some operator. So the design allows the var form num qubits to be updated when applicable. So for example work when you want to say solve an optimization problem you can pass the optimization capabability an algo like VQE, configured with some flexible var form that can span multiple qubits, like TwoLocal. Now if the var form supports setting of the num qubits, like TwoLocal does then it will automatically do that - if not then it will fail if the number of qubits of the var form does not match the operator.

But again then the routine should raise rather than modify implicitly.  I would also say that the tutorial should then just pass the ansatz as a template rather than specified by num_qubits which is just overridden anyway.

@nonhermitian Since this relates to the tutorial can you transfer this across there to have it improved (I have no permission) so it removes an explicit num qubits setting
As mentioned, in regard of updating num-qubits, this is what the Blueprint circuits are about - templates that can later instantiated. Upfront a user will often not know what the problem size is, in terms of qubits, but they can specify a template circuit that the algorithm will use. Since the same circuit template can be used for different problems it just sets the template circuit to the size that it needs.

@nonhermitian Since this relates to the tutorial can you transfer this across there to have it improved (I have no permission) so it removes an explicit num qubits setting
As mentioned, in regard of updating num-qubits, this is what the Blueprint circuits are about - templates that can later instantiated. Upfront a user will often not know what the problem size is, in terms of qubits, but they can specify a template circuit that the algorithm will use. Since the same circuit template can be used for different problems it just sets the template circuit to the size that it needs.


Speed up image stuff

Partially contributes to #199, as it makes three main things:

replaces reading from a File with a reading from a BufReader to increase speed
replaces image::open() with image::image_dimensions() to increase speed
makes image crate optional to increase compilation speed

@xamgore
Thanks for the PR.
I am getting some kind of error, but I will merge it.
(I will fix the error if I can resolve it on my end.)


Make reboot_pending? look for CBS RebootPending

reboot_pending should look for CBS RebootPending instead of RebootRequired
Ref: https://gallery.technet.microsoft.com/scriptcenter/Get-PendingReboot-Query-bdb79542

:+1:

/cc @chef/client-windows

@randomcamel, is this something you worked on in the past? Would be good to get another +1.

@adamedx @jeremiahsnapp I'm not an expert in the registry keys, but this change matches the behavior of opscode-cookbooks/windows, so :+1: for that.


python-packages: mps-youtube 0.2.6 -> <IP_ADDRESS>

Motivation for this change
Update mps-youtube to newest release
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Since this is included in #18635, I close this one and let the mps-youtube update be part of #18635

Could you please reopen this pull request. I have fixed the error with my commit in the other pull request #18635. It is much better because of fitting the title to the content of the pull request.
thx

Github do not let me re-open it, yould you re-open a new PR ?
Thanks


Bug patches and docs improvements for v1.0

This PR is an accumulation of bug patches and other small improvements that I made while thoroughly testing the code in prep for v1.0.
This PR:

[x] Clarifies that Fig3103 x-axis is relative pixel position, not detector pixel position
[x] Fixes the load_old_fitparams function to work for spectroscopic fits
[x] Fixes a major bug with the PyMC3 code where the GPeval function would return a constant 1.0 if no GP was fitted, which would result in a lightcurve with a mean of 2.0.
[x] Makes some other minor fixes to the PyMC3 code that I made while trying to determing the cause of the previous issue
[x] Fixes a bug with masked data in the differentiable GPModel
[x] Adds a better warning if Rs set to independent (which was previously the template default, but is no longer permitted)
[x] Adds emphasis that users should not fit all ramp parameters at the same time. This resolves #375
[x] Adds exoplanet_first option to nuts fitter, similar to the lsq_first option for the emcee fitter

[x] Explains this new feature in the docs

[x] Fixes plot title issue with Figs 3504, 3505, and 3507 (resolves #683)
[x] Resolves #389 with a small tweak to environment.yml

This PR is still a draft - once I've completed it, I'll request a review

@kevin218, want me to also increment the version number in the installation.rst docs page to v1.0 in prep for the release?

Yes please!


version printing is broken

See below
$ drive version
drive version: 0.3.0
Commit Hash: '9ef0072fc67d5a1f703ab791a309c00e05646121'
Go
Version: go1.5
OS: linux/386
BuildTime: 2015-10-09 17:00:36.738571295 -0600 MDT

Addressed by PR https://github.com/odeke-em/drive/pull/422


Support for amazon acceleration endpoint

Output of restic version
restic 0.3.1 (v0.3.1-2-g75f21f2)
compiled at 2016-12-15 17:18:53 with go1.6.3 on linux/amd64

Expected behavior
Restic working successfully with Amazon S3 Accelerated Endpoint.
Actual behavior
When using restic to backup or restore to Amazon S3 Accelerated Endpoint, we receive a following error:
unable to open repo at s3:name.s3-accelerate.amazonaws.com/name: minio.New: Amazon S3 endpoint should be 's3.amazonaws.com'.

here is the environment variable:
export RESTIC_REPOSITORY=s3:name.s3-accelerate.amazonaws.com/name

Note. Amazon about acceleration:
Amazon S3 Transfer Acceleration makes data transfers into and out of Amazon S3 buckets as much as 300% faster, and only charges if there is a performance improvement. Simply enable this feature for buckets you want to accelerate and update the bucket name in your applications. Please visit the Speed Comparison Tool to view estimated performance improvements by region.
Endpoint:
name.s3-accelerate.amazonaws.com

Actual name is omitted here.
Steps to reproduce the behavior

Enable acceleration in S3 Amazon Control Panel.
Set RESTIC_REPOSITORY env to accelerated endpoint.
Try to work with it.

Thanks for your report. We're using the minio s3 Go library to access S3. It expects s3.amazonaws.com as the host name for accessing data on S3, if a bucket was created in another region, it follows the redirect from the Amazon servers. I'm raising an issue there.

I tries to pass into env value like s3:http://......:80 due to Restic documentations permits this. But i received the same error, so it can be supposed that it will not work with anything except Amazon standart endpoint.

The documentation states that this form can be used "For an S3-compatible server that is not Amazon". If you use it with an Amazon URL, that won't work and you get the error.
The great people from Minio are working on supporting Amazon Acceleration Endpoints, that's tracked here: https://github.com/minio/minio-go/issues/585

Thanks!

Fixed upstream: https://github.com/minio/minio-go/issues/585#issuecomment-270563165

Bump. I'm still getting this error when using a repository like s3:mybucket.s3-accelerate.amazonaws.com/mybucket/folder
Fatal: unable to open config file: Stat: The authorization header is malformed; the region 'us-east-1' is wrong; expecting 'us-west-2'
This is with the restic 0.9.1 released binary (Linux amd64).

That's expected: while an accelleration endpoint is now supported in minio-go, no code for using it has been added to restic. If someone is up for a PR (and can test this!) I'd be willing to help reviewing and getting a PR merged. Until that happens, it won't be supported by restic :)

If someone is up for a PR, please take a look at https://github.com/minio/mc/pull/1946/files for an example implementation.


[Bug] Code Execution dataset loading error

from datasets import Dataset, DatasetDict, concatenate_datasets, load_dataset
data = load_dataset("livecodebench/execution-v2", split="test")

datasets version: 3.0.1
get the error
hon3.9/site-packages/datasets/utils/info_utils.py:68, in verify_splits(expected_splits, recorded_splits)
     66     return
     67 if len(set(expected_splits) - set(recorded_splits)) > 0:
---> 68     raise ExpectedMoreSplitsError(str(set(expected_splits) - set(recorded_splits)))
     69 if len(set(recorded_splits) - set(expected_splits)) > 0:
     70     raise UnexpectedSplitsError(str(set(recorded_splits) - set(expected_splits)))

ExpectedMoreSplitsError: {'train'}
```

+1 any follow-up on this?

You can modify the README

Hi. Could you please provide more details about modifying the README? Were you able to manage the problem?

The README has been updated to fix this issue. Thanks for the report!


Algorithm Implementation-3 (Easy)

Solve Happy Ladybugs, Manasa and Stones and Cavity Map

I want to do it in C++

Assign me in Java

I can do them in Python

Can I work on it in C++ or python?


When ingesting, user should be able to choose timeout in case of big file / slow setup / ...

When ingesting, user should be able to choose timeout in case of big file / slow setup / ...

intentionally skipped list and delete, but forgot about the AsyncIngestionClient, thanks!


Download opencv error.

Type of issue
Typo
Description
By the tips, it means that the links error. Like opencv, the success is "https://github.com/opencv/opencv/releases/tag/4.8.0/...", but it shows "https://github.com/opencv/opencv/archive/4.8.0.tar.gz" when I run the code ".\vcpkg.exe install opencv:x64-windows --triplet x64-windows-custom".
It's error like '7z | powershell | ...'.
What I downloaded is version-2024.02.14.

Page URL
https://learn.microsoft.com/zh-cn/vcpkg/get_started/overview
Content source URL
https://github.com/Microsoft/vcpkg-docs/blob/main/vcpkg/get_started/overview.md
Document Version Independent Id
c038c76a-550b-9b30-215f-a2747465590b
Article author
@AugP
Metadata

ID: c038c76a-550b-9b30-215f-a2747465590b
Service: vcpkg

Hi @devil-ycl, sorry you're having trouble with vcpkg!
This looks like an issue when using vcpkg and not a problem with the documentation. Please file a bug on the microsoft/vcpkg repository instead.


Fork kantan.codecs.resource into its own project

The resource package currently uses Result, which would make kantan.codecs a dependency of kantan.resource.
Better to wait until after #17 is done.


Role assignments must be "named" with UUIDs

The classic provider supported automatic generation of role assignment names. In contrast, the nextgen provider requires a roleAssignmentName. This results in boilerplate RandomUuid creation like in the following:
// <https://docs.microsoft.com/en-us/azure/role-based-access-control/built-in-roles#managed-identity-operator>
const managedIdentityOperatorUuid = 'f1a07417-d97a-45cb-824c-7a7467783830';

new azng.authorization.v20150701.RoleAssignment(
    'AKS cluster’s kubelet identity is a Managed Identity Operator',
    {
        properties: {
            principalId: kubeletIdentity.objectId,
            roleDefinitionId: `/subscriptions/${azng.config.subscriptionId}/providers/Microsoft.Authorization/roleDefinitions/${managedIdentityOperatorUuid}`
        },
        scope: pulumi.interpolate`/subscriptions/${azng.config.subscriptionId}/resourceGroups/${nodeResourceGroup.name}`,
        roleAssignmentName: new RandomUuid(
            'AKS cluster’s kubelet identity is a Managed Identity Operator'
        ).result
    }
);

(The full context of the above example can be found in the repository at https://github.com/MerelyAPseudonym/MC-issues.)
I believe restoring the capability to automatically generate UUIDs for names would be an improvement.

Part of #5


Patch: Support more character literals

Yosys does not currently support all of the required string literal characters.  IEEE Std 1800-2012, 5.9.1, states that \v, \f, and \a are to be supported.  These are characters not currently supported by yosys.  The attached patch adds support for these characters as well as support for the carriage return character.
Dan

Thanks! Merged in e060375.


[PE-D][Tester C] Missing index number on a contact detail

If I want to addnote for a person named bca, I will have to firstly use command find bca followed by a command addnote 1 n/note1. After this, the contact's list will be back to the full list, and I won't be able to locate the index of this person named bca. If I want to add multiple notes to him/her, I may need to keep using find bca and addnote 1 n/someNote to do this, which is troublesome. A nicer way is to show the index number on the detail part at right.

Labels: type.FeatureFlaw severity.Medium
original: jian7490/ped#3

Duplicate of #366


panic on dep ensure

One of my project (which imports a lot of dependencies) randomly ends up with a panic, about half of the time.
The other half of the time it will finish and write correctly all the dependencies, as expected.
Here is the trace:
(114) ✗<EMAIL_ADDRESS>depends on k8s.io/api with release-1.9, but that's already selected at kubernetes-1.10.0
panic: runtime error: index out of range

goroutine 1 [running]:
github.com/golang/dep/gps.(*solver).findValidVersion(0xc420362120, 0xc4209b7540, 0xc42115bb00, 0x2e, 0x2e, 0x0, 0xc42115bb00)
    /Users/bvandewa/golang/src/github.com/golang/dep/gps/solver.go:957 +0x521
github.com/golang/dep/gps.(*solver).createVersionQueue(0xc420362120, 0xc4201e9060, 0x10, 0x0, 0x0, 0xc42115bb00, 0x2e, 0x2e, 0x0, 0x0, ...)
    /Users/bvandewa/golang/src/github.com/golang/dep/gps/solver.go:914 +0x955
github.com/golang/dep/gps.(*solver).solve(0xc420362120, 0x17e9fe0, 0xc420016090, 0x0, 0xc420314240, 0xc4203142a0)
    /Users/bvandewa/golang/src/github.com/golang/dep/gps/solver.go:517 +0x5b0
github.com/golang/dep/gps.(*solver).Solve(0xc420362120, 0x17e9fe0, 0xc420016090, 0x1845c50, 0xc42001811f, 0x20, 0xc42037a3f0)
    /Users/bvandewa/golang/src/github.com/golang/dep/gps/solver.go:457 +0xe4
main.(*ensureCommand).runDefault(0xc420159ef6, 0xc420065080, 0xc4200101d0, 0x0, 0x0, 0xc4200650e0, 0x17ed540, 0xc420065440, 0xc420018104, 0x3b, ...)
    /Users/bvandewa/golang/src/github.com/golang/dep/cmd/dep/ensure.go:301 +0x168
main.(*ensureCommand).Run(0xc420159ef6, 0xc420065080, 0xc4200101d0, 0x0, 0x0, 0x0, 0x0)
    /Users/bvandewa/golang/src/github.com/golang/dep/cmd/dep/ensure.go:219 +0x8f4
main.(*Config).Run(0xc420064f60, 0xc420064f60)
    /Users/bvandewa/golang/src/github.com/golang/dep/cmd/dep/main.go:210 +0x11cb
main.main()
    /Users/bvandewa/golang/src/github.com/golang/dep/cmd/dep/main.go:52 +0x1fc

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


fix: check if options are empty

Typeahead plugin gives error on some instances:
i) No option found, []
ii) On switching from linebreak in quote node to heading node
https://user-images.githubusercontent.com/64399555/180824717-377b0d41-4828-4838-a2b5-72971542c6ef.mp4

@tylerjbainbridge How does this look?

Thanks @im-adithya!


Issues with sticky header and footer

Hi,
I am currently implementing this library in a project of mine. First of all great work!
I am trying to make the header and "footer" sticky, but in the documentation the sample code of “basic” and “sticky” is the same (also confirmed it in the SvelteKit code, the file is always the same).
The CSS under “tips” also does not work for me.
It would be wonderful if you could explain how the sticky header and footer is meant to be implemented.
Thank you in advance

Hi,
Sure, thanks for asking. I will complete the documentation to be clearer.
Actually the table must be embed into a parent element to make it works
I made a live example in the svelte REPL:
https://svelte.dev/repl/bfc8bfc170bb4e3d9af2070dbffb3677?version=3.56.0
There may be limits to explore. I set a height to the parent element but don't know the behavior in case of relative height.
If it still doesn't work for you, can you send me an example of your code?

Thanks for the response and the effort!
I will look into it tomorrow and respond here if it worked or not..

So, as you already mentioned, it works different with a relative height and that was my issue, but I found a workaround for that..
Thanks again for reaching out.. now it works 😎


⚠ ClamAV has degraded performance

In 48f7583, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 210 ms

Resolved: ClamAV performance has improved in aa45255.


How to add Bold style to words with hashtags ?

Hi, i would like to bold words with hashtag instead of changing color... How to do it ? Thank you

I had same questions. And I fork the repositories. https://github.com/ShawnBaek/ActiveLabel.swift

ShawnBaek, how to easily add the Bold style to a hashtag in the native version of ActiveLabel please ? Or how to get your fork version with cocoapod (what to write to my pod file to get your version...) Thank you

I'm sorry @ShawnBaek  but i really didn't find the stuff i was looking for in your code... What's the simplest way to add bold style to hashtag please ? Can you show me the code please. Thank you so much

@sebastienboulogne
Sorry for late. Today I received the email from you. I just updated my fork version -> https://github.com/ShawnBaek/ActiveLabel.swift-master-2
It is working fine with Xcode 8.
I have time to remind how to set bold. My tip is you can use attributedText. I will let you know this weekend.

@ShawnBaek i sent you an email asking for an example please on how to set bold style to a hashtag ;) Thx

Pending PR https://github.com/optonaut/ActiveLabel.swift/pull/192

Please include this feature.


Correct version number

Looks like a version number got lost in the rebase. This is the one that cur expects, and must have been correct before the rebase.

Sorry, this happened as part of the cherry picking. 0.4.9 is the right version number, because the unicode commit got included with the 0.3.x changes.
I rewrote this cur commit to accommodate it.
Sorry for the confusion.

Oh okay. So cur should now depend on 0.4.9?

Oh okay. So cur should now depend on 0.4.9?

yes


"Vârșii Mari", "Vârșii Mici" are listed in the drop-down menu

Description:
When user writes "Vârșii Mari" and "Vârșii Mici" the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Vârșii Mari"

Expected results
The location appears in the drop-down menu as "Vârșii Mari, Alba (Bistra)"

Step 3
Write in the search bar "Vârșii Mici"

Expected results
The location appears in the drop-down menu as "Vârșii Mici, Alba (Bistra)"


Make the 2.2.0 release

[x] Check for any open issues in the milestone.
[x] Update the compatibility table.
[x] Make the release.
[x] Update the release notes so that they make sense.
[x] Close this task and the milestone.

Forgot to close this...